ngMock contains unannotated functions

ngMock contains unannotated functions, i.e.
https://github.com/angular/angular.js/blob/d67e999dfbdf47b79fdb3830a04f4f4010a98b98/src/ngMock/angular-mocks.js#L2527
https://github.com/angular/angular.js/blob/d67e999dfbdf47b79fdb3830a04f4f4010a98b98/src/ngMock/angular-mocks.js#L2652
This prevents angular-mocks.js from being minified.
The problem was conditional (angular.mock.module({ mockedService: ... }) syntax was incompatible with minification before this commit), now it throws

[$injector:modulerr] Failed to instantiate module function (e){l.$providerInjector=e} due to:
Error: [$injector:unpr] Unknown provider: e
http://errors.angularjs.org/1.5.0-beta.2/$injector/unpr?p0=e

unconditionally.

Probably happens because ngMock isn't usually minified, because it should only run in tests, where minification isn't really needed. But we should probably fix it anyway.

Indeed I don't think it is intended to be minified. (In fact we are not producing a angular-mocks.min.js as part of our build.)
Why minify it ?

@gkalpak I just modified our Gulp file to prevent minification of angular-mocks.js as a workaround for this issue. Previously, we just added angular-mocks.js to a list of all external JS files (but only if ENV === "test") that later all ended up as a single vendor.js file, which is loaded in index.html. It was just much easier to load it like this, especially for e2e tests.

OK, from a quick search, this has been brought up #3534 and #4448 before.
The last comment from @IgorMinar sums it up nicely, I think:

[...] We have no way to test it right now, so if we break it in the future, we'll fix it when someone complains.

So, @szimek, did you just complain ? :smile:

@gkalpak I just wanted to explain why people minify it :smile:  I found a workaround for it, so it's fine for me as it is, but I suspect there are a lot more people who just add angular-mocks.js to their standard pipeline that minifies everything and will complain later on :smile:

@gkalpak It is rare scenario but it exists, I've experienced problems after 1.5.0-beta.2. I test here that class detection will fail in transpiled and minified code on some condition, while it will pass on another condition. Without mgMock being fixed, I have to revise the way the specs are being bundled.

@bisubus, @szimek, could you try out this angular-mocks.js and see if it works for you ?

@gkalpak, we had the same issue as OP, and I have just tried your build and it is working for us :+1: Thanks! It would be nice to see this in 1.5.1

Thx, @messo, for the feedback. If it doesn't make it into v1.5.1 it will sure make it into v1.5.2 (but let's think positive :stuck_out_tongue: )


Updated Readme.md Mern Stack

Description:
Please include a summary of the change and which issue is fixed. List any dependencies that are required for this change.
Fixes #71 :
Replace issue_no with the issue number which is fixed in this PR
Have you read the Contributing Guidelines on Pull Requests?

[x] Yes
[ ] No

Type of change:
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[x] My code follows the style guidelines(Clean Code) of this project
[x] I have performed a self-review of my own code
[x] My changes generate no new warnings
[x] I have added tests/screenshots(if any) that prove my fix is effective or that my feature works.

Screenshots:

Can you also add the other libraries used. Refer the package.json files to check out all the major libraries and have a talk to @Sanyam2000-dot if any doubt persists.

Should i add in one row only or like "Other Libraries Used:" then badges. Also, no respond from mentioned person.

Libraries used then their badges. Please ask the doubt how would you expect me to know that you are facing any.

Should i add in one row only or like "Other Libraries Used:" then badges. Also, no respond from mentioned person.

The latter one libraries used then badges. And please ask the doubt first else how would you expect me know that you are facing any.

This is good enough, i think.

This is good enough, i think.

Well Done!

@Muditxofficial you have not added the libraries section. Add it.

@ADRE9 Ok i will send a new pr and you can just merge it .

I created a pr. Now you can merge it. Apologies for this :)


How to map spacers using dart sass with use instead of import

Discussed in https://github.com/twbs/bootstrap/discussions/35302

Originally posted by TannerS October 29, 2021
When using node-sass with import, we would use bootstrap with setting the spacers as such within our scss file.
$spacer: 1rem !default;

$spacers: (
  0: 0,
  // 2px
  1:
    (
      // 4px
      $spacer * 0.25
    ),
  2: (
    // 8px
    $spacer * 0.5
  ),
  3: $spacer,
  4: (
    // 24px
    $spacer * 1.5
  ),
  5: (
    // 32px
    $spacer * 2
  ),
  6: (
    // 48px
    $spacer * 3
  ),
  7: (
    // 64px
    $spacer * 4
  ),
  8: (
    // 80px
    $spacer * 5
  ),
  9: (
    // 100px
    $spacer * 6.25
  )
);

@import "~bootstrap/scss/bootstrap";

While switching to dart-sass (sass), they are slowly moving to replace import with use, so while doing that, I did it like this
@use "~bootstrap/scss/bootstrap"; but that doesn't work and the spacers don't exist, when I tried doing
@import "~bootstrap/scss/bootstrap" as *;

I get this error
Error: Module build failed (from ./node_modules/sass-loader/dist/cjs.js):
SassError: This module and the new module both define a variable named "$spacer".
   ╷
42 │ @use "~bootstrap/scss/bootstrap" as *;
   │ ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
   ╵
  src/client/app.scss 42:1  root stylesheet

Any idea how to properly do this using use?  Please and thanks in advance and sorry about my naiveness in the matter.

Unfortunately there's not much we can help with here because we're not using Dart Sass's modules system yet. I've had very little experience with it, other than a couple test runs during early v5 development. Closing this out and keeping it as a discussion. Perhaps give it a bump in the Bootstrap Slack?

Never knew you guys had a slack, but thanks for the reply!
On Mon, Nov 1, 2021, 8:12 PM Mark Otto @.***> wrote:

Closed #35312 https://github.com/twbs/bootstrap/issues/35312.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/twbs/bootstrap/issues/35312#event-5552224912, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ACDUUDZLVBXVHKQEUWLCUFTUJ43GLANCNFSM5HEJJJLA
.
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.

@mdo hey, what is the bootstrap slack?

There's a link in our docs, at least in the footer community section: Bootstrap Slack.


thttpd CGI configuration fails

When trying to serve image files, I'm getting 403 errors because of the CGI security setting in THTTPD.  The documentation says it is possible to specify a different CGI_PATTERN from the command line using the '-c' option, which I'm doing as follows:
FROM lipanski/docker-static-website:latest

# Copy your static files
COPY . .

CMD ["/thttpd", "-D", "-h", "<IP_ADDRESS>", "-p", "3000", "-d", "/home/static", "-u", "static", "-l", "-", "-M", "60", "-c", "\"**.png\"']

I've tried different variations of this approach, but nothing works.
I've also gone as far as replacing the "config.h" file prior to thttpd compilation in the base image to #define the CGI_PATTERN directly in the binary image, but nothing seems to work.

@lipanski  Thanks for that tip. After struggling for several hours and knowing that it was a file permission issue ( since the http root was getting displayed but not the files) , I gave correct permissions for the data file
RUN chmod 644 /var/www/http/index.html  ( running thttpd with alpine base)
tried to test a larger static web app deployment, but started to get the 403 error again. I tired adding recursive file and folder permissions but doesn't seems to work. Shall have to do more permission tweaks to get it working.

BTW , I decided to use thttpd with alpine base after reading this article.

So I ran a simple one line index.html with both lighttpd & thttpd  and was surprised to see the results.
Container size with lighttpd : 4.47 MB & with thttpd its 2.7 MB
in an ideal mode , with no requests coming in, lighttpd uses less than 1 MB of RAM
whereas thttpd uses a whooping 160 MB !

So I did wanted to make my containers smaller but there is a huge trade-off in terms of Memory !
Usage stats : lightttpd :

Usage stats : thttpd :

@pushpdeep I had my eye on this problem for a while but couldn't figure out how to contain it.
but I did solve the problem by replacing thttpd with BusyBox httpd instead. the image comes with a significantly lower memory footprint (I'm getting <1MB on my pet projects) and it's actually smaller in size as well (154KB). on top of that, it can do a bunch of other things - check the FAQ. if you want to give it a try, the transition should be seamless (unless you overwrote the CMD line).


OIDC session expires with ID token expiry and refresh token is never used.

Description
In OIDC flow ID token expiry time was used to set cookie expiry time. Because of this cookie was getting expired at same time as ID token and alternate flow in isValidCookie function to fetch new ID token using refresh token is not getting triggered as cookie itself is expired.
Category
Bug fix
Why these changes are required?
What is the old behavior before changes and new behavior after changes?
This code was correct in version 2.11.1, issue is only in 2.13.0 version. I haven't verified 2.12.0 version.
Issues Resolved
[List any issues this PR will resolve (Is this a backport? If so, please add backport PR # and/or commits #)]
Testing
[Please provide details of testing done: unit testing, integration testing and manual testing]
Check List

[ ] New functionality includes testing
[ ] New functionality has been documented
[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Hi @Alankarsharma thanks for the PR! Just providing some initial feedback here - I was the person who made the PR that changed the OIDC cookie expiry - initial PR is here. I am not an OIDC expert, so maybe I got some things wrong, but we can discuss here and get to the bottom of this (I still have seen some complaints about auto logouts in the public slack, and I am actually trying to replicate and dive deep on this myself - see conversation here: https://opensearch.slack.com/archives/C051Y637FKK/p1712521239951639).
Some basic background is that there is actually two checks that go into seeing if the cookie is still valid - the place you modified in this PR, but also here. In my initial PR in which I consolidated the expiryTime variables, and used the expiry from the token response is because from my understanding for external IDP login setups we should not be blindly giving a session length of TTL, and should instead rely on the timeout from the credentials given by the IDP. Is that not correct/can you explain a little more the reasoning behind why you think this PR would fix the issues?

@derek-ho I checked two code location that you mentioned. No code change is required there.

@derek-ho I will check test case for refresh token, will create a separate pull request for that.

@Alankarsharma Thank you for this PR! I branched off of this and added a test to verify the refreshToken workflow with this fix included: https://github.com/opensearch-project/security-dashboards-plugin/pull/1990
I believe there's one more area where the expiryTime of the credentials needs to be updated instead of the cookie here: https://github.com/opensearch-project/security-dashboards-plugin/blob/9abc57d415e8eedfc96d1bb399cfe118500a4d21/server/auth/types/openid/openid_auth.ts#L296-L300
The changes in this PR will work for the first refresh, but would fail on subsequent refresh since it would be setting the expiry time of the cookie to the expiry time of the access token after first refresh.

@Alankarsharma will be closing this out since the change was merged with @cwperks PR. Thank you for your contribution! I hope to see more from you in the future :)


Won't launch at all. problems with libcurl

Lampray won't launch at all.
I tried running it both as a normal and sudo, but everytime I try to do it, i get this error
./Lampray: /usr/lib64/libcurl.so.4: no version information available (required by ./Lampray)
./Lampray: symbol lookup error: ./Lampray: undefined symbol: SDL_SetTextureScaleMode
I tried rebooting my machine too. I can't get it to even OPEN. Anyone know what's up and how I can fix this?

Probably by installing SDL2.

Probably by installing SDL2.

Do I have to install SDL2 to get it working? It doesn't seem to be available for the current version of my Linux distro.

Do I have to install SDL2 to get it working

It’s listed as a requirement.

It doesn't seem to be available for the current version of my Linux distro.

That actually sounds very unlikely.


Along with tokens/sec and memory consumption (mentioned in issue #106) it would be great if we can put at least One most popular benchmark result (example: MMLU), so that users can understand the tradeoff between

performance degradation
speed
memory consumption

Including all these key metrics might able to provide an overall picture and understand the potential tradeoffs.

Closing this issue, since memory profiling support is provided by PR #160


Move all router access to page-level components

We should only be accessing the URL (via the router) from page-level components, not individual components.
This improves the composability of the app.
Cleanup progress:

[x] Paratime Dashboard
[x] Token Dashboard
[x] Account details
[x] Other places

I think we should also move most data loading into the same page-level components in order to reduce redundancy in the code, but I will do that in a separate PR.

A technically interesting detail: we can also use our tabs (wrapping React Router's Outlet) to pass down some context, so we can also eliminate the repetitive loader configuration from the routes.


Readthedocs

This PR gives us this : http://gnuplot.readthedocs.org/en/readthedocs/ .
I refactored the README into different files (one per section) to make it look good.
The doc can still be consulted from github.

nice!!!


Preserve attrs with imports_granularity=Item

Closes #5030
On the road to #4991

Thanks for the PR!
looks like a pretty simple fix to just pass the attributes along, which is great. When you have a chance, could you add test cases for the other variants: Preserve, Crate, Module, Item, One.

#5030 doesn't concern the other variants, which have harder-to-fix bugs. I'll see if I can work on them but it'll be in a separate PR.

Thank you for the PR! This seems to be a duplicate of #5031 to me though, and the interactions with the other variants was one of the outstanding questions still unresolved over there (https://github.com/rust-lang/rustfmt/pull/5031#discussion_r753617083)
As such, I think it' be best to either extend the resolution as suggested in #5031 or as @ytmimi suggested, add some cases for other variants to make sure there's no unanticipated side effects.
I don't recall offhand whether the diff is currently on a call path that's only reachable with the Item variant, but this portion of the codebase gets refactored frequently enough that the tests are still valuable even if only as a preventative/defensive measure

Closing in favor of #5314


Excuting "gulp watch" error!

OS :        Windows7 64bit
Branch :  dev
I enter ui and do "npm install" then get two WARN:
D:\MyWorkSet\Github\conductor\ui>npm install
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules\chokidar\node_modules\fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for fsevents@ 1.0.17: wanted {"os":"darwin","arch":"any"} (current: {"os":"win32","arch":"x64" })
then "gulp watch" error:
D:\MyWorkSet\Github\conductor\ui>gulp watch
[14:36:57] Requiring external module babel-register
[14:37:17] Using gulpfile D:\MyWorkSet\Github\conductor\ui\gulpfile.babel.js
[14:37:17] Starting 'watch'...
[14:37:17] Starting 'clean'...
[14:37:17] Finished 'clean' after 12 ms
[14:37:17] Starting 'set-env'...
[14:37:17] Finished 'set-env' after 4.73 ms
[14:37:17] Starting 'serve'...
Stats : undefined
[14:37:17] Starting 'watchTask'...
[14:37:17] Finished 'watchTask' after 99 ms
[14:37:17] Starting 'public'...
[14:37:17] Finished 'public' after 13 ms
[14:37:17] Starting 'styles'...
[14:37:17] Finished 'styles' after 9.82 ms
[14:37:17] Starting 'fonts'...
[14:37:17] Finished 'fonts' after 1.67 ms
[14:37:17] Starting 'lint'...
[14:37:18] Finished 'lint' after 317 ms
[14:37:18] Starting 'images'...
[14:37:18] Finished 'images' after 2.8 ms
events.js:160
      throw er; // Unhandled 'error' event
      ^
Error: spawn D:\MyWorkSet\Github\conductor\ui\node_modules\optipng-bin\vendor\optipng.exe ENOENT
    at exports._errnoException (util.js:1022:11)
    at Process.ChildProcess._handle.onexit (internal/child_process.js:193:32)
    at onErrorNT (internal/child_process.js:359:16)
    at _combinedTickCallback (internal/process/next_tick.js:74:11)
    at process._tickCallback (internal/process/next_tick.js:98:9)
Maybe conductor doesn't supports Win7?

what version of node and npm are you running?

@v1r3n
C:\Users>node -v
v6.9.5
C:\Users>npm -v
3.10.10

@FreeceLian node and npm versions are fine.  Do you have access to Linux, Mac or later versions of Windows where you can try this out?  I will have to get hold of a win 7 machine to try this out.

@v1r3n OK , I'll try this at CentOS Linux 7.2 . But I think maybe using docker is a better way? BTW,my own work PC is Win7(cannot change...) and cannot support docker.... so I hope the problem can be solved.

Met the same problem on Win 7, and I'm a corporate proxy.
I tried to install pngopt-bin, but getting the following errors. I'm not sure the problem is related to proxy, since most modules can be well installed.
`D:\workspace\java\conductor\ui>npm install --save optipng-bin

<EMAIL_ADDRESS>postinstall D:\workspace\java\conductor\ui\node_modules\opti
png-bin
node lib/install.js

‼ tunneling socket could not be established, cause=getaddrinfo ENOTFOUND 3128
3128:80
‼ optipng pre-build test failed
i compiling from source
× RequestError: tunneling socket could not be established, cause=getaddrinfo
ENOTFOUND 3128 3128:80
at ClientRequest. (D:\workspace\java\conductor\ui\node_modules\go
t\index.js:74:21)
at ClientRequest.g (events.js:291:16)
at emitOne (events.js:96:13)
at ClientRequest.emit (events.js:188:7)
at ClientRequest.onError (D:\workspace\java\conductor\ui\node_modules\tunnel
-agent\index.js:178:21)
at ClientRequest.g (events.js:291:16)
at emitOne (events.js:96:13)
at ClientRequest.emit (events.js:188:7)
at Socket.socketErrorListener (_http_client.js:310:9)
at emitOne (events.js:96:13)
<EMAIL_ADDRESS>D:\workspace\java\conductor\ui
`--<EMAIL_ADDRESS>npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules\ch
okidar\node_modules\fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for fsevents@
1.0.17: wanted {"os":"darwin","arch":"any"} (current: {"os":"win32","arch":"x64"
})`

@FreeceLian @niwy any chance you can try this outside the corporate network and possibly on a *nix machine?

@v1r3n I'm afraid not.I can only try it on a Win7 machine with free network or other OS machines with corporate proxy.  Now I'm trying it on a CentOS machine , but i meet the certification problem because gradle doesn't support NTLM proxy well.

@FreeceLian @niwy some related discussion on the similar error, might be helpful:
https://github.com/npm/npm/issues/3354
http://stackoverflow.com/questions/33162560/error-tunneling-socket-could-not-be-established-cause-connect-econnrefused-10

@v1r3n Thanks for the reference link. I have tried many configuration methods to set proxy, however none of them could solve the "gulp watch" problem. I will test the UI server at home without proxy later.

@v1r3n Thanks. But my problem isn't the npm proxy setting.  I can always keep my network active in my Win7 machine, but meet the "gulp watch" problem. Or i can use a CentOS machine behind proxy, i'm sure the network is connected, but i meet the gradle problem:"unable to find valid certification path to requested target". I think my npm is fine because i can do the "npm install".

@v1r3n Now I have solved the proxy problem and try it at CentOS 7.2 but meet the same problem as Win7. I wonder if i get a wrong step :

git clone git@github.com:Netflix/conductor.git

modify conductor/gradle/wrapper/gradle-wrapper.properties
distributionUrl=gradle-2.14.1-bin.zip
because I use a modified gradle to replace the original one due to the proxy problem

cd server
../gradlew server
It works.
8044 [main] INFO  com.netflix.conductor.server.ConductorServer  - Kitchen sink workflows are created!

install node
[root@SZX1000138239 wrapper]# node -v
v6.10.0

cd ui
npm install
get a lot of WARNs during the install
npm WARN deprecated<EMAIL_ADDRESS>use uuid module instead
npm WARN deprecated<EMAIL_ADDRESS>Please update to minimatch 3.0.2 or higher to avoid a RegExp DoS issue
npm WARN deprecated<EMAIL_ADDRESS>Please update to minimatch 3.0.2 or higher to avoid a RegExp DoS issue
npm WARN deprecated<EMAIL_ADDRESS>graceful-fs v3.0.0 and before will fail on node releases >= v7.0. Please update to<EMAIL_ADDRESS>as soon as possible. Use 'npm ls graceful-fs' to find it in the tree.
npm WARN deprecated<EMAIL_ADDRESS>to-iso-string has been deprecated, use @segment/to-iso-string instead.
npm WARN deprecated<EMAIL_ADDRESS>Jade has been renamed to pug, please install the latest version of pug instead of jade
npm WARN deprecated<EMAIL_ADDRESS>Please update to minimatch 3.0.2 or higher to avoid a RegExp DoS issue
npm WARN prefer global<EMAIL_ADDRESS>should be installed with -g
After the install , execute npm install again, it shows
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules/chokidar/node_modules/fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"linux","arch":"x64"})

install gulp: npm install gulp -g
some WARNs:
npm WARN deprecated<EMAIL_ADDRESS>Please update to minimatch 3.0.2 or higher to avoid a RegExp DoS issue
npm WARN deprecated<EMAIL_ADDRESS>Please update to minimatch 3.0.2 or higher to avoid a RegExp DoS issue
npm WARN deprecated<EMAIL_ADDRESS>graceful-fs v3.0.0 and before will fail on node releases >= v7.0. Please update to<EMAIL_ADDRESS>as soon as possible. Use 'npm ls graceful-fs' to find it in the tree.
After the install, execute gulp -v   (dir: conductor/ui)
[17:00:30] Requiring external module babel-register
[17:00:30] CLI version 3.9.1
[17:00:30] Local version 3.9.1

gulp watch
Then I get the completely same error as Win7.

Running into a similar error.  I am running Centos7.  I have had to manually install many of the dependencies with npm.  Finally after all dependency errors were no longer being thrown:
$ node -v
v7.6.0
$ npm -v
4.1.2
$ gulp watch
[13:41:08] Requiring external module babel-register
process.argv =====>  [ '/usr/local/bin/node', '/bin/gulp', 'watch' ]
DEBUG =====>  true
WATCH =====>  true
[13:41:13] Using gulpfile ~/Development/conductor/conductor/ui/gulpfile.babel.js
[13:41:13] Starting 'watch'...
[13:41:13] Starting 'clean'...
[13:41:13] Finished 'clean' after 12 ms
[13:41:13] Starting 'set-env'...
[13:41:13] Finished 'set-env' after 4.01 ms
[13:41:13] Starting 'serve'...
Stats : undefined
[13:41:13] Starting 'watchTask'...
[13:41:13] Finished 'watchTask' after 25 ms
[13:41:13] Starting 'public'...
[13:41:13] Finished 'public' after 8.35 ms
[13:41:13] Starting 'styles'...
[13:41:13] Finished 'styles' after 4.97 ms
[13:41:13] Starting 'fonts'...
[13:41:13] Finished 'fonts' after 1.24 ms
[13:41:13] Starting 'lint'...
[13:41:13] Finished 'lint' after 18 ms
[13:41:13] Starting 'images'...
[13:41:13] Finished 'images' after 1.55 ms
events.js:161
throw er; // Unhandled 'error' event
^
Error: write EPIPE
at exports._errnoException (util.js:1028:11)
at WriteWrap.afterWrite (net.js:804:14)

@FreeceLian @cquon can you try npm install browser-sync and then try again?  I was able to reproduce this once on a container.  If that works for you, will try to fix it.

@v1r3n
[root@SZX1000138239 ui]# npm install browser-sync
npm WARN deprecated<EMAIL_ADDRESS>use uuid module instead
<EMAIL_ADDRESS>/opt/conductor/ui
└─┬<EMAIL_ADDRESS>  └─┬<EMAIL_ADDRESS>    └─┬<EMAIL_ADDRESS>      ├─┬<EMAIL_ADDRESS>      │ └──<EMAIL_ADDRESS>      └──<EMAIL_ADDRESS>npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules/chokidar/node_modules/fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"linux","arch":"x64"})
After this, execute gulp watch, the error is still the same.

Same for me as well
$ npm install --save-dev browser-sync
npm WARN deprecated<EMAIL_ADDRESS>use uuid module instead
<EMAIL_ADDRESS>/home/cquon/Development/conductor/conductor/ui
\u2514\u2500\u252c<EMAIL_ADDRESS>\u2514\u2500\u252c<EMAIL_ADDRESS>\u2514\u2500\u252c<EMAIL_ADDRESS>\u2514\u2500\u2500<EMAIL_ADDRESS>npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules/chokidar/node_modules/fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"linux","arch":"x64"})
[cquon@coreycentos7 ui]$ gulp watch
[11:18:39] Requiring external module babel-register
process.argv =====>  [ '/usr/local/bin/node', '/bin/gulp', 'watch' ]
DEBUG =====>  true
WATCH =====>  true
[11:18:44] Using gulpfile ~/Development/conductor/conductor/ui/gulpfile.babel.js
[11:18:44] Starting 'watch'...
[11:18:44] Starting 'clean'...
[11:18:44] Finished 'clean' after 8.53 ms
[11:18:44] Starting 'set-env'...
[11:18:44] Finished 'set-env' after 2.12 ms
[11:18:44] Starting 'serve'...
Stats : undefined
[11:18:44] Starting 'watchTask'...
[11:18:44] Finished 'watchTask' after 30 ms
[11:18:44] Starting 'public'...
[11:18:44] Finished 'public' after 9.7 ms
[11:18:44] Starting 'styles'...
[11:18:44] Finished 'styles' after 5.72 ms
[11:18:44] Starting 'fonts'...
[11:18:44] Finished 'fonts' after 1.39 ms
[11:18:44] Starting 'lint'...
[11:18:44] Finished 'lint' after 26 ms
[11:18:44] Starting 'images'...
[11:18:44] Finished 'images' after 1.53 ms
events.js:161
throw er; // Unhandled 'error' event
^
Error: write EPIPE
at exports._errnoException (util.js:1028:11)
at WriteWrap.afterWrite (net.js:804:14)

Running into similar error on executing docker-compose build command.
Is there any fix/workaround for this issue?

I'm running into the same issue with the ui docker container. I'll note that I am also behind a corporate proxy.

We are running to the same error. Any resolution or workaround? Thanks!
[10:21:13] Finished 'images' after 1.51 ms
events.js:160
throw er; // Unhandled 'error' event
^
Error: spawn C:\dev\garnet\netflix\conductor\ui\node_modules\optipng-bin\vendor\optipng.exe ENOENT
at exports._errnoException (util.js:1020:11)
at Process.ChildProcess._handle.onexit (internal/child_process.js:197:32)
at onErrorNT (internal/child_process.js:376:16)
at _combinedTickCallback (internal/process/next_tick.js:80:11)
at process._tickCallback (internal/process/next_tick.js:104:9)

For me it was simply the missing libpng on my Ubuntu box.
So installing libpng-dev solved the issue for me.


TdDialogService should take an optional validator as an input

Feature Request
TdDialogService should allow putting a validator while we enter the text.
I am using it to rename files and would need to disable the okay button in the case the name already exists.

I think this will cover your need https://github.com/Teradata/covalent/pull/606

Ill get the pr done during the weekend! :muscle:


Fix unportable type in lib.rs

This type may dismatch under certain architecture.

Thank  you! Merged.
Should fix #7 as well.


cleanup of several CVEs via dependency updates

Clean up CVEs by updating dependencies.
Next steps - update license and and owasp check list
Outcome of the first commit:
Total: 137 (UNKNOWN: 0, LOW: 4, MEDIUM: 64, HIGH: 56, CRITICAL: 13)
down to:
Total: 92 (UNKNOWN: 0, LOW: 2, MEDIUM: 34, HIGH: 43, CRITICAL: 13)
second commit down to:
Total: 89 (UNKNOWN: 0, LOW: 2, MEDIUM: 32, HIGH: 42, CRITICAL: 13)

Closing this as the there is an ongoing effort with the cleanup based on version 28.0.1.


datepicker doesn't update if the current date value in the input field was entered with the datepicker

I noticed that the datepicker only updates after date changes made in the input field directly if the first date was entered via keyboard (or javascript). If the first date was selected from the datepicker, any subsequent changes made directly in the input field do not update the datepicker. Manually calling $(selector).datepicker('update') also doesn't work (which I presume is the same issue).
So to replicate:

instantiate datepicker as component with date-format="yyyy-mm-dd"
load form with empty date field
select "2012-10-03" from datepicker
change this to "2012-10-01" by keyboard directly in the input field
datepicker still shows 2012-02-02

Awesome work by the way!

Closing this issue for now, please comment or reopen with an jsfiddle example if this should still be an issue!


Git log with custom format - incorrect escaping

This command
wslgit.exe log --name-status --format="%x3c%x2ff%x3e%n%x3cr%x3e %H%n%x3ca%x3e %an%n%x3ce%x3e %ae%n%x3cd%x3e %at%n%x3cp%x3e %P%n%x3cs%x3e%n%B%n%x3c%x2fs%x3e%n%x3cf%x3e" -n200 --follow -- packages/backendjs/src/processing/portal/chatHandlers/IgnoreChat.ts

Results to:
fatal: bad revision '"--format=%x3c%x2ff%x3e%n%x3cr%x3e %H%n%x3ca%x3e %an%n%x3ce%x3e %ae%n%x3cd%x3e %at%n%x3cp%x3e %P%n%x3cs%x3e%n%B%n%x3c%x2fs%x3e%n%x3cf%x3e"'

VSCode GitLens plugin is using such custom formats for log & stash commands

Yes, without spaces it works normally (after merging rev-parse fix)

@asvetliakov , in your specific computer/wslgit setup, are you using the advanced feature where you have to set either WSLGIT_USE_INTERACTIVE_SHELL and/or BASH_ENV ?
I ask because I have isolated a behavior related to it and the escaping of spaces.

@diablodale Yes, i'm using WSLGIT_USE_INTERACTIVE_SHELL=false. However this doesn't seem to change anything

I found some issues and I've made a fix and pull request.
@asvetliakov , compile the code in my PR that should link here. Or if you don't have that ability and trust, here is a release compile of the same code.
wslgit.zip
I tested this with your original test case above - it works now.
I also tested this against six combinations including spaces in working directory and parameters.
I would appreciate feedback if this works better for you both.

It fixed it. You're my hero!

@diablodale Thank you for the fix, works great so far.


New first contract

This is a different take of the first contract where less of the ERTP choregraphy happens in the code and more happens on the client-side
I kept things minimalist as far as explaining ERTP itself and delegate it to the ERTP dedicated documentation

I added warnings about tutorial steps being hard sometimes
I'm re-requesting a review because the wording i chose sounds a bit like a commitment
If that's find with you as is, you can merge the PR
I'm creating an issue label called Contract developer experience. It will be used to track anything that is lived as annoying by contract devs. Issues with this label probably will link to follow-up issues in other repos

I don't mind committing to continuing to work to make things better, since that's what we plan to do.
I don't like the phrasing "the steps in this tutorial may seem unnecessarily complicated". The work-around I pointed at is actually simpler than the right way to do things, so it's overly simplified. I think this would be better covered by an annotation at the point that we tell them to overwrite a file, saying that this is a hack that saves them work, and that better solutions are being developed. (As you've noted, they may even be in place quite soon, but they're certainly not ready now.)

Closed in favor of #23


Set base path for Flutter Web example

The Flutter web example should have a different base path because it is hosted at a non root location.
See https://flutter.dev/docs/development/ui/navigation/url-strategies#hosting-a-flutter-app-at-a-non-root-location
The Flutter Web example is hosted at https://getsentry.github.io/sentry-dart/

Because of the problem mentioned above, there are errors while processing as seen in https://sentry.io/organizations/sentry-sdks/issues/2308253501/?project=5428562&query=is%3Aunresolved


🔨 forcibly removing DOI citation lookup & insertion

Trying to address issue #196 here.
Currently, this is intended for discussion on how to best provide an option in frontmatter.

@fwkoch so would adding a frontmatter field, and checking that in these two locations, adding the transformers only when true be the best way forward?
convert_doi_links: true|false

?

We can certainly come back to this! My main hesitation at this stage is we don't have any centralized options yet for controlling parse behaviour. I think this is an obvious thing we need to add, but it will take some work. :)


Feature/fix date between months

Critical Changes
When using date_between function, for the options start_date and end_date , changed the "m" for months to be "M" uppercase, per Snowfakery documentation.
Changes
Issues Closed
#142

I changed all 5 recipes but I could only test the 4 in NPSP folder in NPSP environment.  Somebody (@eehjunggnujhee ? @samanthashain ?) who knows more about PMM should please test the remaining one,  pmm_1_program-service.recipe.yml .  Also mentioning @nai888, just FYI, because we had worked together on Corporate_Donors_npsp.recipe.yml during the last Sprint.

hi @programmer2coder
Yes, PMM recipe has to be updated as well. Im troubleshooting again after Allison reported a bug to Paul/ Aaron.


SNMP- Send trap from BMC and verify

Automation:

Induce a trap by generating error on BMC
Capture and verify trap on SNMP manager

@prkatti1 anything pending here ?

https://gerrit.openbmc-project.xyz/c/openbmc/openbmc-test-automation/+/23065


controller.sh_not_found

Dear Andres Murillo Piedrahita,
Hope you are doing well.
I tried simulating this environment but unfortunately the making the fran folder, the ./controller.sh file is not found in the repository, as a results, Unable to contact the remote controller at <IP_ADDRESS>:6633.
Also when making the paper is says; invalid user:  'mininet:mininet'
Kindly find the screenshots attached.

Hello,
Sorry for that. I just added the scripts required to run. Could you try again?

Dear Andres Murillo, that solved the issue, thank you very much

Ok, closing this issue


dockerTools.streamLayeredImage: make /tmp

Description of changes
Make /tmp directory while building a Docker image with pkgs.dockerTools.streamLayeredImage.
(see https://github.com/NixOS/nixpkgs/issues/257172#issuecomment-2039171878 for the motivation)
Things done

Built on platform(s)

[ ] x86_64-linux
[x] 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

[ ] 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
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 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.

@roberth Unlike the files like /etc/{group,passwd} that is created with shadowSetup, the existence of /tmp is considered essential in POSIX/Linux systems.
Here are some facts:

The /tmp directory shall exist on POSIX-conforming systems, according to IEEE Std 1003.1-2017^1
Linux Foundation stipulates that the /tmp directory must be made available for programs that require temporary files. in File System Hierarchy 3.0^2

So in my opinion it isn't one of such feature requests but a bug fix.
Here, two questions seem to be clarified:

a) Should pkgs.dockerTools continue to ship POSIX-incompatible Docker images lacking the /tmp directory?
b) If it is undesirable that all the image have the /tmp directory in the perspective of security, should #260063 be also reverted?

Containers are not required to comply with anything except their runtime's requirements and the requirements of the application.

a) Should pkgs.dockerTools continue to ship POSIX-incompatible Docker images lacking the /tmp directory?

Yes, but you are responsible for the shipping part. streamLayeredImage is just a tool, which is at the bottom of the "stack", so it should be flexible rather than opinionated.

b) If it is undesirable that all the image have the /tmp directory in the perspective of security, should

Thank you for bringing this up. I would prefer for it to be removed. I'll cross-post.

Okay, then I'll closing this. Let's keep the discussion in #257172.
Anyway, thank you for your review and being a good shepherd!


0.13 breaks arrays

I have the following in my values.yaml:
  # @schema
  # required: false
  # @schema
  args:
    - start

helm-schema version prior to 0.13 produced this schema which works fine with helm itself:
"args": {
  "items": {
    "anyOf": [
      {
        "type": "string"
      }
    ]
  },
  "title": "args"
}

helm-schema 0.13 produces this:
"args": {
  "items": {
    "anyOf": [
      {
        "required": [],
        "type": "string"
      }
    ],
    "required": [],
    "type": "array"
  },
  "required": [],
  "title": "args"
}

The produced schema does not accept the original values.yaml file since it expects the values of args to be of type array but they are of type string. I can fix this myself by specifying the type of the items like this:
  # @schema
  # required: false
  # type: array
  # items:
  #   type: string
  # @schema
  args:
    - start

Should be fixed in 0.13.1-2

yep that works fine - thanks a lot for the quick response!


Update the docs website

docs.phinx.org is massively out of date. Can we get that updated to use the latest docs?

Done! And a fresh release <IP_ADDRESS> made to ensure that all the necessary stuff (website, docs, composer) all reflect the current version.

Thanks. Thats great.

@shadowhand if your doing releases be sure to follow the http://semver.org/ guidelines. E.g: 3 components for a version instead of 4.
I would of done 0.4.3.

@robmorgan sorry, it seemed silly to use 0.4.3 when there were no functional changes. I thought semver had allowance for "hotfix" points, but it seems not. Future releases will follow accurately.

@shadowhand yeah sure! I've always followed the X.Y.Z but I've normally grouped several commits into a release and only updated the Y (minor) when I thought it was really necessary.


.friction needs to be gated to a number between 0 and 100

will involve turning it into a private property with a wrapping getter and setter


Does monitoring get run?

Hey @TylerSheffels. Looks like in my somewhat mechanical refactoring, I moved the reference to the monitoring role here https://github.com/dimagi/commcarehq-ansible/blob/master/ansible/deploy_db.yml#L45-45, and since there's no host called monitoring, I think it doesn't get run. So that's my bad, but fixing it raises the following question, which probably has a simple answer.
What machine is this supposed to be run under? If it has to be run on the same machine as redis, then calling it "monitoring" seems wrong, and "redis_monitoring" seems better.

@dannyroberts Initially I was thinking there was more monitoring that would occur.  Currently I think this should be switched to redis_monitoring and only run on the redis server. #55

Awesome, thanks!


[BUG] Error: Expected a Runnable, function or object.

Hi
when I follow up with youtube demo, a error occured. like below.
Flowise is 1.8.1

BR,

Should be able to work, try this
Vector Upsert Chatflow.json

you can load it from the top right:


Ask

How can resolve this problem with imsi catcher
“Python can not open file ‘simple imsi -catcher .py’
Why this error appear ?

What command did you try ?

when I use this command (sudo python simple_IMSI-catcher.py --sniff) or
(sudo python simple_IMSI-catcher.py)
when i use the two commands (all them )  appear to me the python can not
open the file simple  IMSI_catcher .py  error
how I can resolve it
في الأربعاء، 12 يونيو 2019 في 10:18 ص تمت كتابة ما يلي بواسطة Oros <
notifications@github.com>:

What command did you try ?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Oros42/IMSI-catcher/issues/73?email_source=notifications&email_token=AKLH2DK3U62KJMUJR347KM3P2EVYFA5CNFSM4HP6TIPKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXRFN5Q#issuecomment-501372662,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKLH2DPCDR3VATXVYLAK3C3P2EVYFANCNFSM4HP6TIPA
.

Why did you have space in the simple_IMSI-catcher.py name ?
And use python3.


No effects with Hyperion-remote

Hi, neighbours and friends...
I use latest hyperion in an OpenELEC distro and a 52 leds LighBerry pack with no major issues. Always works as should. But when I try to run a script effect from command line wiht hyperion-remote.sh script I get no effect at all. It works fine on changing colors (-c red | -c blue | -c ffffff, and so on), but when I try to run any effect (i.e: ./hyperion-remote.sh -e "Rainbow swirl fast", nothing happens, I get a message that stands "effect: xxx" as if it was running, but nothing happens to my leds.
When Hyperion starts, the effect configured in /storage/.config/hyperion.config.json is played, and if I restart Hyperion, the effect runs again.
Any clue?
Thank you!!

I was using the file names, not the effect name in " "...
i.e. it is
"Knight rider" not knight-rider


Fancy process table

Functionality relating to the process table.

#94
#89
#87

All done. Nice work @dalonsoa.


Web sign-in note outdated?

The Web sign-in Warning under EnableWebSignIn that it only supports Temporary Access Pass as an authentication method may be outdated. The latest information on the Web sign-in page says the capabilities have been expanded with Windows 11:
"With the release of Windows 11, the supported scenarios and capabilities of Web sign-in are expanded.
For example, you can sign in with the Microsoft Authenticator app or with a SAML-P federated identity."
(https://learn.microsoft.com/en-us/windows/security/identity-protection/web-sign-in/)

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 845fc177-23cf-0b2c-4b2b-5ac3966763a6
Version Independent ID: ef4248eb-9b9f-24ec-631e-d65ae2ae3ecf
Content: Authentication Policy CSP - Windows Client Management
Content Source: windows/client-management/mdm/policy-csp-authentication.md
Product: windows-client
Technology: itpro-manage
GitHub Login: @vinaypamnani-msft
Microsoft Alias: vinpa

Thanks @CBrasga for the feedback!
@paolomatarazzo and @vinaypamnani-msft can ya'll tag team to determine if this note is an editable section of the CSP article, and if so how it should be handled?

@CBrasga thank you for the feedback, I'll update the CSP documentation.


#394 Added CSGO Round Winner Classifier

Description
This is a classification problem where we we predict who wins individual snapshots of rounds.
Fixes #394
Type of change

[ X] Added a new machine learning frameworks, libraries or software.
[ ] Documentation update

Checklist:

[X ] My code follows the style guidelines of this project
[ X] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[X ] My changes generate no new warnings

@sanjay-kv
The files that im adding in this PR are already added accidentally by my other branch
I'd request you to delete those files then merge this PR


Pull in transaction receipts only when necessary

When processing blocks considered final, we fetch the triggers only to drop them and re-process them from the transaction receipts. This is a wasteful process, particularly of disk space because we cache hundreds of GBs worth of transaction receipts we don't really need.
This PR makes it so that we only request and store transaction receipts when they are actually necessary, which is when processing blocks that may still be reorged. blocks_with_triggers now returns richer information instead of dropping it, the return type was changed from EthereumBlockPointer to EthereumBlockWithTriggers.
One thing that makes this change tricky is that we may need to re-process a block for dynamic data sources, at which point we again need to know if the block is final or not, and need extra information for non-final blocks. This is encoded in the BlockFinality enum and the trigger re-processing is encapsulated triggers_in_block. Some code got moved from the block stream to the ethereum adapter so that it could be shared between the block stream and the instance manager.
Since this touched some of the block stream, I took the opportunity to refactor it a bit. We no longer have special logic for advancing through an empty range, instead we insert a dummy block for to with no triggers, and use the normal processing logic to advance. The special casing was in some sense an optimization, but it was optimizing the fastest possible case which is an empty range, so it was considerable complexity for no real gain. In doing this, transact_block_operations got a bit simpler, it was only used to advance one block a time when it's perfectly capable of skipping blocks. Also it no longer takes a block_ptr_from, it doesn't make sense for that to be anything other than the current block pointer, so we just assume that. Also, some of the types in the block stream state machine got simplified a bit.
Sorry if this is a lot in one PR! Please ask if some change needs clarification or more comments. I tested this with Moloch and Betoken, both synced fine. I avoided any non-trivial change to reversion code, any changes there are just formatting and minor refactors.

Some comment threads are still on going but I've responded to everything. I did a sample of the size of an ethereum_blocks row before and after this change, and there was a 10x size reduction, from around 300k to around 30k. Yay!

@Jannis rebased and fixed the message.


feat(web3Hub): list tabs in grid

✅ Checklist

[ ] npx changeset was attached.
[ ] Covered by automatic tests.
[ ] Impact of the changes:

...

📝 Description
Replace this text by a clear and concise description of what this pull request is about and why it is needed. Be sure to explain the problem you're addressing and the solution you're proposing.
For libraries, you can add a code sample of how to use it.
For bugfixes, you can explain the previous behavior and how it was fixed.
In case of visual features, please attach screenshots or video recordings to demonstrate the changes.

❓ Context

JIRA or GitHub link: https://ledgerhq.atlassian.net/browse/LIVE-13189

🧐 Checklist for the PR Reviewers

The code aligns with the requirements described in the linked JIRA or GitHub issue.
The PR description clearly documents the changes made and explains any technical trade-offs or design decisions.
There are no undocumented trade-offs, technical debt, or maintainability issues.
The PR has been tested thoroughly, and any potential edge cases have been considered and handled.
Any new dependencies have been justified and documented.
Performance considerations have been taken into account. (changes have been profiled or benchmarked if necessary)

Could be close if we don't think we'll be able to merge it soon

I'll try to review soon but we still need to rebase it in the meantime

one last remark, i dont see a changeset file. can you add it if this PR is a candidate to merge ?
Thanks in advance

We need to update and merge this at some point


Use Core CLR version of System.Data.Common and System.Data.SqlClient

...once the NuGet packages are ready.

Just blocked by getting the native SNI.dll into the SqlClient package.
@bricelam can help out with packages etc.


Smart RGB bulbs

Is there someone that has any good cheap RGB bulbs that you can connect with homebridge??

Yeelight RGB Bulb

@iCarl Do you use them? Are these really wifi bulbs and are they any decent quality? I'm seeing them online for less than $20 so I'm curious to know if they're any good.


[HIGH] ManageProject_CyclesTab

Display/Open Cycles tabs

[ ] Add functionality in ARA
[ ] Add NRT

Moved to another project


KNX-IP Lite Version

Hi @envy,
Great Library!!! I was able to connect it to Home Assistant very fast and It is working great!!! :+1: Thanks
I have a feature request.
Can be modified a little your library so as to be just the KNX-IP Interface without the Webserver? Like an option to turn off the webserver and eeprom features, and to set manually the address and callbacks?
This request is because for using your library in a project that has its own webserver, the integration is very difficult.
If you can have the webserver and eeprom as optional, it can be easily integrated on ESPEasy, Sonoff-Tasmota, Espurna, etc.
What do you think?

Hi,

Can be modified a little your library so as to be just the KNX-IP Interface without the Webserver?

Yes, I can add this.

Like an option to turn off the webserver and eeprom features, and to set manually the address and callbacks?

Without EEPROM should already be possible, just don't call knx.load().
EEPROM is only accessed there and by the webserver code.
However, there is no function yet for assigning callbacks from code, I will add this.

So I made it possible for start() to accept a nullptr which disables the webserver part.
I also added a callback_assign function to assign grozp addresses to callbacks. Together with the config_set_type functions that should be enough to configure the library without using the web UI.
You can now achieve a webserver-less and EEPROM-less library by calling knx.start(nullptr) instead of knx.start() and removing knx.load(). Configuration is then done using callback_assign and config_set_type or the default value of config items.
Please test the changes.

wow!!! so fast!!! I will test it. Thanks a lot :+1:

Hi, works great! :+1:
I made small changes for make it work.
Added few functions to the library:
// Physical Address Set and Get:
config_set_physical_addr ( area, line, member );
config_get_physical_addr ();

// The Reverse of GA_to_address ( area, line, member ):
area_from_address ( address_t );
line_from_address ( address_t );
member_from_address ( address_t );

// Address set by area, line, member on GA and Callbacks>
config_set_ga ( id, area, line, member );
callback_assign ( id, area, line, member );

Also, I made a Sonoff_DHT11.ino as example of manual set of address on the code. With some comments on using the Start() function
I'm making a Pull Request with this changes for you to review.

The Start(server) function to use an already running webserver is a very good idea also with root_prefix :+1:
An idea:
When Root_Prefix is different than "", a Home or Return Button should show up on the KNX Config Page. What do you think?

Sorry, but I'm going to reject all those pull requests.
#23 You added a bunch of addresses specific to your use-case and there is also a DHT sensor added. The example is meant to show the feature-set of the library in a minimal way.
#22 I don't want to move the files into a subfolder. AFAIK it is fine for Arduino libraries to have the files in the root folder
#21 See above.
#20, #19, #18 I added the physical address functions myself, but named them differently (17c390d8ae8d71b1bc7e99226c2dea21f8ddf7c6).
I also extended the address_t union to include ga and pa members making it possible to read out area, line and member easily without extra functions.
Lastly, I want the public API to expect address_t and not have overloads for every function with uint8_t area, uint8_t line, uint8_t member. That's why GA_to_address exists, to make this easier. But now, you can even do
address_t a;
a.ga.area = 1;
a.ga.line = 2;
a.ga.member = 3;

or address_t a = {.ga={line, area, member}} (Yes, you need to switch line and area) (432439a69c518d4eb46d5c1e84bdd66dacb23fd9)

Hi,
No problem with those PR. I make those pull requests just as ideas to be discussed and as an example of setting addresses by code :+1:
I like your approach for the tranformation of GroupAddress to area, line and member. It is more elegant.
I will re make my tests with your modifications.
As the ESP_KNX_IP library is intended to be a library to be included on other proyects, what do you think of:
1- Posibility of adding buttons with links (as you said) and buttons to callbacks?
2- Posibility of turning off the buttons EEPROM SAVE, EEPROM RESTORE and REBOOT?
3- Posibility of changing the webpage title?
4- Posibility of setting the webpage text (so as a language translation be posible by code)
5- Posibility of a custom arrange of all the objects on the webpage. Then, I will post here an example.
If you want, I can help you with the code and with the Wiki.
Thanks a lot.

Made the PR #26 #27 #28 to:

Added #ifndef ROOT_PREFIX to have the option to set the Root Prefix before calling the #include <esp-knx-ip.h>, so as to eliminate the need of modifying the library.

Changing the name of ntohs to __ntohs to eliminate conflict when using ESP8266 LWIP library.

Hi, I created some issues to track some ideas seperately. (#36, #37, #38)

Posibility of adding buttons with links (as you said) and buttons to callbacks?

What do you mean with "buttons to callbacks"? Buttons to issue callbacks from the webui?

Posibility of a custom arrange of all the objects on the webpage. Then, I will post here an example.

How would you specify arrangement of objects? xy-coordinates? Some kind of grid?

What do you mean with "buttons to callbacks"? Buttons to issue callbacks from the webui?

Yes! exactly :+1:

How would you specify arrangement of objects? xy-coordinates? Some kind of grid?

Can be in the order you register them on the main application code. Also can be everything centered.
#36

Closing this issue as all the questions I had were answer. Thanks :+1:
New topics continue in their specific issues.


Download link point to download PoshTool dev14 in dev12

Repro step:
Make sure there is no PoshTool installed

Open .ps1 file with VS
There is a tip tell you that “Powershell Tools for Visual Studio are available”. Click "Install Now"

Expect:
Link point to download poshtool for dev12
Acutal:
Link point to download PoshTool for dev14. Details please see attachment.

There is a link on Dev14 page that points to Dev12


chore: Add code split for application deploy flow

Description

Add code split to create an extension point for module instances

Create an interface for application publishing service to be implemented by module instances

Out of scope: Move existing entities to this interface
PR fixes following issue(s)
Fixes https://github.com/appsmithorg/appsmith/issues/29183
Media

A video or a GIF is preferred. when using Loom, don’t embed because it looks like it’s a GIF. instead, just link to the video

Type of change

Chore (housekeeping or task changes that don't impact user perception)

Testing

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Also list any relevant details for your test configuration.
Delete anything that is not relevant

[ ] Manual
[ ] JUnit
[ ] Jest
[ ] Cypress

Test Plan

Add Testsmith test cases links that relate to this PR

Issues raised during DP testing

Link issues raised during DP testing for better visiblity and tracking (copy link from comments dropped on this PR)

Checklist:
Dev activity

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] PR is being merged under a feature flag

QA activity:

[ ] Speedbreak features have been covered
[ ] Test plan covers all impacted features and areas of interest
[ ] Test plan has been peer reviewed by project stakeholders and other QA members
[ ] Manually tested functionality on DP
[ ] We had an implementation alignment call with stakeholders post QA Round 2
[ ] Cypress test cases have been added and approved by SDET/manual QA
[ ] Added Test Plan Approved label after Cypress tests were reviewed
[ ] Added Test Plan Approved label after JUnit tests were reviewed

Force merging since last set of changes are just a refactor of unused code.


TileSet::duplicate(false) breaks original TileSet resource

Godot version
4.2.dev2
System information
Godot v4.2.dev2 - Windows 10.0.19044 - Vulkan (Forward+) - dedicated NVIDIA GeForce GTX 1080 (NVIDIA; <IP_ADDRESS>79) - AMD Ryzen 5 3600 6-Core Processor (12 Threads)
Issue description
The mere invocation of TileSet::duplicate(false) breaks the original TileSet resource. In particular it looks like the TileSetSources of the original TileSet are getting removed, but I can only speculate.
Example A
In this example both TileMaps are using the same TileSet resource, but one of them has the script attached; invoking TileSet::duplicate(false) causes both of them to lose their tiles.
https://github.com/godotengine/godot/assets/5732925/0b362a46-e88d-43ce-a028-58daceca267e
Example B
This example is the same as above, except the first tilemap gets the newly duplicated TileSet, and the second one keeps the original TileSet:
https://github.com/godotengine/godot/assets/5732925/3cc57bfb-b692-4685-b786-6a0ad1619e9d
Steps to reproduce

Create a TileSet resource and attach it to a TileMap
At runtime call duplicate(false) on the TileSet resource
Observe that the TileMap is now blank

Minimal reproduction project
TileSetDuplicate.zip

AFAIK tile sources are tied to a TileSet. Doing duplicate(false) duplicates the TileSet and assigns the old sources to the new TileSet, so they are transferred. Not sure if this is fixable.

Yes, as Kobewi's says, TileSet have internal resources that needs to be duplicated with the TileSet if you want it to keep working. I guess we could mention that somewhere in the the docs.

This is documented partially here and here but should be added more clearly about duplication too

My intuition tells me shallow duplicating a TileSet (or any resource for that matter) shouldn't adversely modify the original resource merely because of an implementation detail. Shouldn't the TileSetSources be duplicated as well, no matter what?

It would be pretty involved and complicated to force partially deep duplication on TileSet, and I don't think it's a good idea
We could make it error if you try shallow duplication as well as documentation, it should be as simple as:
	virtual Ref<Resource> duplicate(bool p_subresources = false) const override {
		ERR_FAIL_COND_V_MSG(!p_subresources, Ref<Resource>(), "TileSet cannot be duplicated without duplicating subresources");
		return Resource::duplicate(true);
	}


Duplication of atlas names in cli help text

If you specify a wrong atlasname while using the cli, you get the following message:
atlasreader: error: argument -a/--atlas: invalid choice: 'destrieuxx' (choose from
    'AAL', 'Desikan_Killiany', 'Destrieux', 'Harvard_Oxford', 'Juelich',
    'Neuromorphometrics', 'aal', 'desikan_killiany', 'destrieux', 'harvard_oxford',
    'juelich', 'neuromorphometrics', 'all')

As you can see, this shows all atlases once in normal and once in lowercase version. I recommend to get rid of the duplication (in the help text).

Hmm... I just went through the code and I see the issue. If we want to accept upper and lowercase names for the atlases, we need to leave it as it is. I.e. the error message will always include both versions.
Fine for me than. @rmarkello - except if you see another solution, you can close this issue again.


[FLINK-28218] Add liveness probe to k8s operator

What is the purpose of the change
This PR introduces a liveness probe mechanism for the Flink Kubernetes Operator that aims to asses whether the operator can access namespaces and can reconcile resources properly.
The liveness probe can be enabled from the Helm chart as shown in the example:
operatorHealth:
 port: 8085
 livenessProbe:
   periodSeconds: 10
   initialDelaySeconds: 30
 startupProbe:
   failureThreshold: 30
   periodSeconds: 10

Brief change log

Add Health endpoint to the opeartor
Add flexible Helm configuration to enable startup/liveness probes

Verifying this change
[TODO]
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 changes to the CustomResourceDescriptors: no
Core observer or reconciler logic that is regularly executed: no

Documentation

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

cc @wangyang0918 @tweise

I will add some tests and docs for the helm changes tomorrow morning

@morhidi , I have cleaned up naming and added the missing test and helm docs.

@morhidi , I have cleaned up naming and added the missing test and helm docs.

LGTM, the port 1234 might be blocked in certain environments, we could try to use an ephemeral port instead.


Removing redundant concept of pre and post kernel steps

Fairly non-controversial. Removing the redundant lines of code pointed out in https://github.com/opesci/devito/issues/121

What do you mean redundant lines?

I like removing stuff. I'm happy with this


docs: add Celestia-ContributorGitPOAP sections

Add Celestia Contributor GitPOPA sections to CONTRIBUTING
Close https://github.com/celestiaorg/celestia-core/issues/1155

[ ✅] Tests written/updated
[ ✅] Changelog entry added in .changelog (we use
unclog to manage our changelog)
[ ✅] Updated relevant documentation (docs/ or spec/) and code comments

I completely agree with you!!Thank you for your kind correcting me.
I think it would be better to add the GitPOAP explanation to the README, so I made a new pull request to the README to it.
https://github.com/celestiaorg/celestia-core/pull/1157


What the algo form PHP for use this wraper?

Thank you for develop encryb/simplecrypto easy to use WebCryptoAPI.
Please tell me the algorithm/function in PHP  for encrypt and what is the encrypted in
decryptWithPassword: function(password, encrypted, onError, onSuccess)
Thank you very much

I found another lib which looks good and is openssl compatible. Should be easy to use with php I think...?
But if I try to increase data to encrypt it works fine with simplecrypto and fails with https://github.com/safebash/opencrypto. See issue https://github.com/safebash/opencrypto/issues/5 with example...


Configure CO2 levels and buzzer sound via web interface

Wishes:

Configure warn and critical CO2 level via web interface (instead of using compile-time definitions CO2_WARN_PPM and CO2_CRITICAL_PPM from config.h)
Allow to switch buzzer sound on/off via web interface (nice to have: beep duration, tone pitch, etc.)

Idea:

Store configuration values in non-volatile memory (EEPROM or internal flash)
Read configuration values from non-volatile memory during start-up
Allow reconfiguration of those values via web interface (store in non-volatile memory and also use those values without a re-start)

Fixed by latest commits.

Fixed by latest commits.


Fix typo datase => database

Hi folks,
Here is a tiny fix for a typo in a comment :)
Btw, learning a lot from this repo for multi tenant deployments. Thanks for this project!

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Thank you for the fix - merged.


Feat/subspecs

Why?
Enable installing separately each format type

HI there 👋.
Is someone still maintaining this framework?


Poormatches with processing DTU data

Hello! I used the following code to process the images in the DTU dataset：
ns-process-data images  --data data/sdfstudio-demo-data/dtu-scan65/image  --output-dir data/scan65_test --matching-method exhaustive --num-downscales 1
and got the following output
(  ●   ) Copying images...[08:39:56] 🎉 Done copying images.                                                             process_data_utils.py:151 [08:39:57] 🎉 Done downscaling images.                                                         process_data_utils.py:209 [08:40:01] 🎉 Done extracting COLMAP features.                                                       colmap_utils.py:531 [08:40:06] 🎉 Done matching COLMAP features.                                                         colmap_utils.py:545 [08:41:08] 🎉 Done COLMAP bundle adjustment.                                                         colmap_utils.py:567 [08:41:09] 🎉 Done refining intrinsics.                                                              colmap_utils.py:576 ────────────────────────────────────────────── 🎉 🎉 🎉 All DONE 🎉 🎉 🎉 ──────────────────────────────────────────────                                                 Starting with 49 images                                                                                We downsampled the images by  and 2x                                                                                Colmap matched 2 images                                                                               COLMAP only found poses for 4.08% of the images. This is low.                                   This can be caused by a variety of reasons, such poor scene coverage, blurry images, or large exposure changes.      ────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────
May I ask why this happened?
Thank you！

Hi, the DTU dataset provided camera poses and you don't need to use colmap.

Thank you for your reply，I found the problem in my Colmap version. Now I get the result:
`( ●    ) Copying images...[15:34:09] 🎉 Done copying images.                                                             process_data_utils.py:151
[15:34:10] 🎉 Done downscaling images.                                                         process_data_utils.py:209
[15:34:15] 🎉 Done extracting COLMAP features.                                                       colmap_utils.py:531
[15:34:19] 🎉 Done matching COLMAP features.                                                         colmap_utils.py:545
[15:35:17] 🎉 Done COLMAP bundle adjustment.                                                         colmap_utils.py:567
[15:35:24] 🎉 Done refining intrinsics.                                                              colmap_utils.py:576
────────────────────────────────────────────── 🎉 🎉 🎉 All DONE 🎉 🎉 🎉 ──────────────────────────────────────────────
Starting with 49 images
We downsampled the images by  and 2x
Colmap matched 49 images
COLAMP found poses for all images, CONGRATS!
────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────'

@HanakoAA Hello, I had the same problem as you: not only the DTU dataset but my custom dataset as well. May I ask how you solved it? Thank you!

May I ask if you have solved this problem? I had the same problem.

@HanakoAA Hello, I had the same problem as you: not only the DTU dataset but my custom dataset as well. May I ask how you solved it? Thank you!

May I ask if you have solved this problem? I had the same problem.

Hello! I solved the problem by changing my Colmap version from 3.7 to 3.8. You can have a try. Sorry for replying so late.

@HanakoAA Hello, I had the same problem as you: not only the DTU dataset but my custom dataset as well. May I ask how you solved it? Thank you!

Hello! I solved the problem by changing my Colmap version from 3.7 to 3.8. You can have a try. Sorry for replying so late.


fixed some typos & a dead link in the doc

Come across some typos & a dead link and have them fixed. Also have a few expressions re-written.

Thanks!

No problem!


implemented lasted checked in, favorite places, and history

Team Android, please check if it works for you guys.

LGTM


Reorganize Gradle Module Metadata spec documents

Keep a 1.0 spec
Add a latest spec, now at 1.1

Proposal is that upon creation of 1.2, latest is copied with a 1.1 name
and then latest is edited to become 1.2.
This enables having a stable link for the latest version and historical
versions in direct access. Maintenance should be minimal as older
version have no reason to change anymore.

@jjohannes This is a proposal, let me know what you think. In any case, we at least need to rename the file as it says 1.0 but contains 1.1.

@jjohannes Updated following your comments. Please have another look.


WIP: Add TLD and Dictionary permutation types

Related to #6
The following PR aims to include the TLD and dictionary permutations methods. Given that this is a library, we want to avoid runtime dictionary files being passed around. Instead the PR will include a new build.rs that will compile the raw txt dictionaries in static arrays at compile time.
Special ref. https://dev.to/rustyoctopus/generating-static-arrays-during-compile-time-in-rust-10d8

Unfortunately forgot to open as draft PR. Will merge when complete.


ChromaVectorStore throws NullPointerException on document add

Bug description
When adding a document to ChromaVectorStore, a NullPointerException is thrown if the add was successful. The document ends up in Chroma, but application is left to handle the NPE.
When a document is successfully added, the body of the response is simply null (I've confirmed this by using curl to POST documents to the upsert endpoint). But the code at https://github.com/habuma/spring-ai/blob/main/vector-stores/spring-ai-chroma/src/main/java/org/springframework/ai/chroma/ChromaApi.java#L309-L315 tries to bind that to a Boolean, but then it returns null to the caller which then tries to use it as a Boolean value...then you get a NullPointerException. More completely, here's the first few lines of the stack trace I got:
java.lang.NullPointerException: Cannot invoke "java.lang.Boolean.booleanValue()" because "success" is null
	at org.springframework.ai.vectorsore.ChromaVectorStore.add(ChromaVectorStore.java:106) ~[spring-ai-chroma-store-0.8.0-SNAPSHOT.jar:0.8.0-SNAPSHOT]
	at com.example.aichat.UploadController.upload(UploadController.java:49) ~[classes/:na]

I also suspect that it will throw a different exception (possibly ClassCastException) if the addition fails, although I've not recreated that case. If the document add fails, you get a JSON error structure back...something like: {"error":"IndexError('list index out of range')"}. I suspect that will fail because it won't be able to parse that JSON into a Boolean.
Environment
Spring AI version 0.8.0-SNAPSHOT
Steps to reproduce
Simply call the add() method on ChromaVectorStore.
Expected behavior
If the document add is successful, the method should not throw any exception. If it is unsuccessful, it should throw a RuntimeException, as described in the code.

@habuma, By specification the upsert endpoint is expected to return a boolean: https://docs.trychroma.com/js_reference/Collection#returns-8
Furthermore the existing IT is successfully verifying this behaviour: https://github.com/spring-projects/spring-ai/blob/2f891243501d77bdd620d27419c2f0e07dfa0db5/vector-stores/spring-ai-chroma/src/test/java/org/springframework/ai/chroma/ChromaApiIT.java#L88-L90
So I'm puzzled by the statement?

(I've confirmed this by using curl to POST documents to the upsert endpoint)
Could it be that we are running different Chroma version (the ITs are based on the chroma:0.4.15)?

I'm running 0.4.22. (Started by Spring Boot's Docker Compose support via a docker-compose.yaml).
I just now used curl to POST a few broken JSON files to the upsert endpoint and get responses like this one:
{"error":"IndexError('list index out of range')"}

(In this case, I had two IDs, but only 1 document and 1 set of embeddings in the body.)
If there's a JSON error (e.g., a missing comma) I get a response like this:
{"detail":[{"type":"json_invalid","loc":["body",21],"msg":"JSON decode error","input":{},"ctx":{"error":"Expecting ',' delimiter"}}]}

But, if the body that I'm posting is correct and if the document is added successfully, I get this:
null

Per the Swagger documentation at http://localhost:8000/docs, the response for a success should be an any. The response for a validation error should be a JSON response similar to what I show above for a JSON error. (The Swagger doc doesn't mention the other JSON response, but I can assure you that I get it.)
Moreover, what even led me to this was much higher level. I have the following code that exhibits what I'm seeing:
    @Bean
    ApplicationRunner go(VectorStore vectorStore) {
        return args -> {
            try {
                vectorStore.add(List.of(new Document("99999", "Rich Purnell is a steely eyed missile man.", Map.of("test", "doc"))));
            } catch (Exception e) {
                System.err.println("Error adding documents: " + e);
                // ignore errors because of problem with Chroma vector store
            }
        };
    }

When run, I see this in the console:
Error adding documents: java.lang.NullPointerException: Cannot invoke "java.lang.Boolean.booleanValue()" because "success" is null

Followup: I changed my docker-compose.yaml to force 0.4.15 and I do get true when a document is successfully upserted. So, something changed between 0.4.15 and 0.4.22 (perhaps a bug in Chroma?).

@habuma thanks for clarifying this.
Maybe they did some breaking changes in the API.
Will investigate further with the 0.4.22

@habuma i can confirm that Chroma has changed their API, so that the upsert doesn't return boolean anymore.
Their JS client docs still claims that it is possible , but the Python counterpart suggest None as return value.
Also noticed another change in the getEmbedding as well. The where filter doesn't doesn't support the complex operators (like with the query) but only simple matches.
A fix for the above issues will follow shortly.


[stable/oauth2-proxy] config secrets

@desaintmartin @Miouge1
What this PR does / why we need it:
Changes the config file to a Secret, which is important for settings like basic_auth_password. Preserves compatibility by still accepting a configmap at existingConfig, and adding a new setting existingConfigSecret to allow users to set a secret.
While we're at it, modifies the configFile value so that in addition to accepting a string describing the literal contents of the file, it can also accept a Map, which it then renders to the config format accepted by oauth2-proxy. This is useful in environments where helm's deep merging of multiple values files is used.
Special notes for your reviewer:
Checklist

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

Thanks!
Could you add/edit a test file in the ci directory about this?
/assign

ok, added a test so that configFile is tested in both string and map forms. i don't see a way to test existingConfigSecret though since that would require a secret external to the helm chart


Fill resume content

Fill with main information divide by primary and secondary section about myself

our main content is here!


Installation fails, with restKit Pod!

How to install it along with RestKit pod?

Please help...

you need to fork AUMediaPlayer and change dependency in AUMediaPlayer.podspec

@emilwojtaszek ok, I forked AUMediaPlayer. Now to to install the forked library to my project.
And secondly, would AUMediaPlayer work with AFNetworking v1.3.4 ?

in AUMediaPlayer.podspec replace s.dependency 'AFNetworking', '~> 2.3' with s.dependency 'AFNetworking', setup AUMediaPlayer as development pod and try install

Sorry, to say that I don't know how to setup the development pod.
As, I am ne to pods...

I am confused, what to type in the podflie
Should it be pod 'AUMediaPlayer’, :git => ‘git@github.com:sahil263/AUMediaPlayer.git'
But, this throughs error...
[!] Unable to find a specification for AUMediaPlayer’, :git => ‘git@github.com:sahil263/AUMediaPlayer.git``

@emilwojtaszek
How to install the forked library to my project?
Please help...

@emilwojtaszek  Or could you update your .podspec file and push the changes?


spring-cloud-starter-netflix-eureka-client dev adds jsr311-api-1.1.1.jar which is in conflict with  jakarta.ws.rs-api-2.1.6.jar

Hello, Spring cloud newbie here.  I recently modified my spring boot app to read config from the config server and register with discovery server.  I successful configure my boot app and register.  However my application endpoints are failing due to jsr311-api-1.1.1.jar being added to the classpath when I add spring-cloud-starter-netflix-eureka-client.  My app uses jakarta.ws.rs-api-2.1.6.jar and I am getting a NoSuchMethodError since jsr311 is added ahead of Jakarta in the classpath.  Problem detailed at https://stackoverflow.com/questions/69804665/adding-spring-cloud-config-to-boot-app-causes-java-lang-nosuchmethoderror-by-add
How can I fix this problem?  Is there another Eureka dependency I should be using?
Thanks,
Sufie

I have added an exclusion of the jar in the pom.xml and this has resolved my issue.


Bank Viewer Addon

An addon to view your bank while you are not at a bank.
Implementation would go like this. When the player opens the bank the addon will store all the items on the client. When the user clicks the addon tab, the items will be displayed.

Duplicate of #4343


Issue converting latitude and longitude to spatial coordinates

I'm creating a "Locate Me" function and I have the lat/longs but I'm seeing inconsistent results when converting them to spatial coordinates.
For testing I'm using the lat/longs for the Bentley's Austin Texas office for all tests:
Lat: 30.36936664900865 Long: -97.74329476841908
I can only get it to work in the following sandbox:
https://www.itwinjs.org/sandbox/DerekHoltman/Derek Testing
Using cartographicToSpatialFromEcef and the Metrostation model I get the correct results:
1134495.7502543565, -1951322.9502983307
Using cartographicToSpatialFromGcs and the Metrostation model I get the wrong results:
1198251.9361718288, -2025549.2996983863
This is north of San Antonio and affects the current sessions orientation of the model which is seen when selecting Top of the cube.
Note: I do not know how to  check the itwin version in the sandbox.
I also have the following test app that has the same code:
https://dev.azure.com/bentleycs/AssetWise - Energy Infrastructure/_git/powerbi-report-grouping?version=GBUpgraded-grouping-mapping-widget-to-0.8.2-package-lock
It is using itwin 3.2.8 but I have seen the following results with itwin 3.5.1 in another app.
Using cartographicToSpatialFromEcef and the QA-RockyMountCircuits-LatestChangeset model I get the wrong results:
-1155569.9807568148, -202005.6185020086	South Carolina
Using cartographicToSpatialFromGcs and the QA-RockyMountCircuits-LatestChangeset model I get the wrong results:
-1187585.3929570804, -205278.5211527112	Pacific Ocean
To start test app:
npm install
npm start
To test in both sandbox and test app:
A decorator tab has been added to the right side panel.
select decorate to run the code above and watch debug output for coordinates.
There is a useEcef constant that is used to control which code it hits.
Note: the imodels are geolocated.
Does this have something to do with the imodels? my environment? or something else?

@mdastous-bentley do you have any suggestions?

Couple of points:

The testing point is located in Texas while the dataset is in North Carolina, that's far away.  Do we get the same result if the testing point is location near the model data?  ECEF transformation should only be used near the imodel data location.
I see this iModel is geo located (EPSG:3404), so based on the documentation cartographicToSpatialFromGcs should be used to get accurate result from the backend, I really don't understand why its not the case with the sandbox.
I would suggest testing with another iModel generated using the MicroStation connector, using the same area and GCS, that way we can check if it's an iModel data issue.

Using hard coded NC Englewood Recreation Park location:
Lat: 35.954177 Long: -77.829353
Which is in the same area as the data.
cartographicToSpatialFromEcef and cartographicToSpatialFromGcs both return the correct location:
715204.1659459114,<PHONE_NUMBER>6102668
I also tried using cartographicToSpatialFromGcs going to the Austin Office on the following model that was generated from the MicroStation Connector: http://localhost:3000/?iTwinId=cb6b9001-96ff-4af8-aa4d-8571973a6e45&iModelId=cb64c9a2-8296-4aef-bcd3-787912e91aa1
It is using the same area and GCS.
It also returned the coordinates in the Pacific: -1187585.3929570804, -205278.5211527112

IModelConnection.cartographicToSpatialFromGcs is mostly a wrapper on top of getIModelCoordinatesFromGeoCoordinates RPC.
@pmconne Who maintains this API?

@mdastous-bentley it has no dedicated maintainer.

@DRHoltman Not 100% sure on the timing -- but we are now internally aware of the issue you are seeing and are asking around.  We'll update this item with findings.

Hey Derek,
Myself and @AlainRobertAtBentley looked into this.
First thing:
Your iModel's GCS is set to EPSG:3404, which domain of validity look like this:

You cannot expect a point located in Texas to be accurate with this GCS.  In my tests, the point was located near Austin, but not exactly where it should be.
You might consider setting your iModel's GCS to something like EPSG:5070 which cover the United States:

Another point,
I noticed some display issues with decorations when the Globe mode of your view was set to 'Ellipsoid' you might consider using 'Plane' for now by using something like this:
 activeViewport?.view.displayStyle.changeBackgroundMapProps({globeMode: GlobeMode.Plane});

Thanks guys for looking into this...
I'm not sure if we can require our users to use this GCS so we are going to add a check to see if the lat/long is within the GCS extents (converted to lat/longs)  and error out only if outside the area.
I guess what is still confusing, is why this worked in v2.
Here is a link to V2 working on the same area:
https://openutilities-dts.qa.aw-au.bentley.com/852f17d6-5a8b-423b-9ee9-d2cb17722de4/viewer/https%3A%2F%2Fopenutilities.qa.aw-au.bentley.com%2F%3Fproject%3D852f17d6-5a8b-423b-9ee9-d2cb17722de4%26isEmbedded%3Dtrue%26resultId%3D425ba9f6-7b81-40d7-99ec-cc0182e0cee5

Do you know which method you were using to translate your lat,long coords?  I cannot see any request to the backend.

I just ran it. 8:10AM Central time. It should be using cartographicToSpatialFromEcef.

That's interesting, this tool uses Bing services... and calls ViewState3d.lookAtGlobalLocationFromGcs to adjust the view, would be interesting to see if call directly this method would work too.

Can this item be closed?

Yes as I found a work around.


Add initial basic CA issuer implementation

This PR now includes e2e tests that verify a signing keypair can be verified, and also ensures a Certificate can be issued successfully.

This needs docs adding, which will come in a follow up PR along with docs for all other current functionality.

Merging now to try and get some feedback on it's use, and it also has better test coverage than the ACME implementation anyway...


Add --ts flag to properly support projects with mixed .js and .ts files.

Properly addresses issues https://github.com/tapjs/node-tap/issues/549 and https://github.com/tapjs/node-tap/issues/526

Oh!  Maybe what you're describing only happens when there are two different instances of ts-node in play?  I copied ts-node to another folder, and ran it this way.  This is an interesting problem:
$ node -r ts-node-duplicate/register -r ts-node/register load-ts.js

/Users/isaacs/dev/js/tap/node_modules/ts-node/src/index.ts:240
    return new TSError(diagnosticText, diagnosticCodes)
           ^
TSError: ⨯ Unable to compile TypeScript:
ok.ts:2:23 - error TS2304: Cannot find name 'exports'.

2 Object.defineProperty(exports, "__esModule", { value: true });
                        ~~~~~~~
ok.ts:3:9 - error TS2580: Cannot find name 'require'. Do you need to install type definitions for node? Try `npm i @types/node`.

3 var t = require("./");
          ~~~~~~~

    at createTSError (/Users/isaacs/dev/js/tap/node_modules/ts-node/src/index.ts:240:12)
    at reportTSError (/Users/isaacs/dev/js/tap/node_modules/ts-node/src/index.ts:244:19)
    at getOutput (/Users/isaacs/dev/js/tap/node_modules/ts-node/src/index.ts:360:34)
    at Object.compile (/Users/isaacs/dev/js/tap/node_modules/ts-node/src/index.ts:393:11)
    at Module.m._compile (/Users/isaacs/dev/js/tap/node_modules/ts-node/src/index.ts:439:43)
    at Module.m._compile (/Users/isaacs/dev/js/tap/node_modules/ts-node-duplicate/src/index.ts:439:23)
    at Module._extensions..js (internal/modules/cjs/loader.js:770:10)
    at require.extensions.<computed> (/Users/isaacs/dev/js/tap/node_modules/ts-node-duplicate/src/index.ts:442:12)
    at Object.require.extensions.<computed> [as .ts] (/Users/isaacs/dev/js/tap/node_modules/ts-node/src/index.ts:442:12)
    at Module.load (internal/modules/cjs/loader.js:628:32)

Interestingly, if I do what the error message says, and npm i @types/node, it fixes the problem in that case as well.  Of course, not as efficient to double-transpile it since the second one has nothing to do, but still, not a crash.

It seems like a cleaner way to address this would be to treat --ts or --no-ts in the same way as the --esm flag.  Ie, it's a flag that just means "do -r ts-node/register for any /\.tsx?$/ files".  If you want to provide your own mechanism, you'd do tap --test-arg=--require=my-ts-node/register --no-ts test/foo.ts.  To make it even easier, you could put this in your package.json:
{
  "tap": {
    "ts": false,
    "test-arg": [
      "-r",
      "my-ts-node/register"
    ]
  }
}

Would that address the problem?

@kyegupov Can you check out c66309c and see if that would give you what you need?


Special Tokens Are Wrong

For instance, counting the special token "<|end_of_text|>" is wrong.
See Llama 3's tokenizer config for more details about special tokens.
It might be better to also visualize the token id in the website.

For instance, counting the special token "<|end_of_text|>" is wrong.

Are you inserting the string "<|end_of_text|>" into text and expecting it to be parsed as a special token? I have not implemented special tokens in that manner. I wrote into the README an example of how to work with special tokens. Here is the example from README adapted into this particular token:
const tokens = []
tokens.push(llama3Tokenizer.getSpecialTokenId('<|end_of_text|>'))
console.log(tokens.length) // Count is one

I implemented it this way because I am not sure what to do with all these special tokens, and the official LLaMA 3 repo has similar Python implementation: https://github.com/meta-llama/llama3/blob/main/llama/tokenizer.py#L202-L229
Thinking more about this, maybe I should also parse the special tokens from text?

It might be better to also visualize the token id in the website.

Thank you for the suggestion. I can see how there would be a need for this, though the website is just a quick demo for the library.

Got it! Makes much more sense now~!

I have now added the features you requested:

Visualize token ids on the demo website
Allow passing special tokens inside normal input text, no longer requiring any special function calls

@belladoreai  Awesome! Thanks so much, I appreciate it!


Udf

javasctipt udf implementation without schema support.

Codecov Report

Merging #5243 into user-defined-functions will increase coverage by 0.14%.
The diff coverage is 86.7%.

@@                     Coverage Diff                      @@
##             user-defined-functions    #5243      +/-   ##
============================================================
+ Coverage                     82.57%   82.71%   +0.14%
- Complexity                    11037    11206     +169
============================================================
  Files                          1331     1341      +10
  Lines                         45030    45717     +687
  Branches                       5642     5719      +77
============================================================
+ Hits                          37182    37815     +633
- Misses                         5226     5256      +30
- Partials                       2622     2646      +24

Impacted Files
Coverage Δ
Complexity Δ

sql/src/main/java/io/crate/plugin/SQLPlugin.java
98.21% <ø> (ø)
12 <0> (ø)
:arrow_down:

...o/crate/analyze/DropFunctionAnalyzedStatement.java
100% <100%> (ø)
5 <5> (?)

...c/main/java/io/crate/analyze/DataTypeAnalyzer.java
60% <100%> (+6.15%)
6 <3> (+2)
:arrow_up:

.../src/main/java/io/crate/executor/task/DDLTask.java
100% <100%> (ø)
2 <1> (ø)
:arrow_down:

.../java/io/crate/analyze/CreateFunctionAnalyzer.java
100% <100%> (ø)
3 <3> (?)

...crate/analyze/CreateFunctionAnalyzedStatement.java
100% <100%> (ø)
8 <8> (?)

...ava/io/crate/metadata/sys/SysClusterTableInfo.java
100% <100%> (ø)
4 <0> (ø)
:arrow_down:

...arser/src/main/java/io/crate/sql/SqlFormatter.java
86.19% <100%> (+0.82%)
7 <0> (ø)
:arrow_down:

sql/src/main/java/io/crate/metadata/Functions.java
98.18% <100%> (+0.3%)
14 <9> (+4)
:arrow_up:

...nce/information/InformationRoutinesExpression.java
100% <100%> (ø)
1 <0> (ø)
:arrow_down:

... and 104 more

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 1f536f3...02ee8b3. Read the comment docs.

is superseded by PR of udf -> master

DO NOT DELETE udf BRANCH!!!


Adding hover effect on links.

Adding hover effect on links in navbar.
Something like this https://codepen.io/jstn/pen/zuDst

I want to work on this issue. Please assign it to me.

@SauravMukherjee44 Can I work on this issue

please assign this to me


Fragment template sets phx-target to :live_view

Describe the bug
The application consists of a LiveView and a LiveComponent. The component renders a button in a separate function whose event is to be processed in the component.  Instead, the event is sent to the LiveView so that an error is thrown:
** (UndefinedFunctionError) function TestWeb.PageLive.handle_event/3 is undefined or private
How to reproduce it
defmodule TestWeb.PageLive do
  use Surface.LiveView
  alias TestWeb.Component
  def render(assigns) do
    ~F"""

      <Component id="my-comp"/>

    """
  end
end

defmodule TestWeb.Component do
  use Surface.LiveComponent
  def render(assigns) do
    ~F"""

component

      {button(assigns)}

    """
  end

  def button(assigns) do
    ~F"""
    <button :on-click="click"></button>
    """
  end

  def handle_event("click", _, socket) do
    IO.inspect "clicked :D"
    {:noreply, socket}
  end

end

The behavior you expected
I expect the LiveComponent that renders the button to also receive the event.
The bug can be worked around if the entire template is inside the render/1 function.
Your Environment
Surface: v0.7.0
LiveView: v0.17.5
Elixir: v1.12.1

I'm also seeing something similar where an event from a stateful component isn't respecting the target given, and is instead attempting to be handled by the LiveView.
With a set up of LiveView -> ComponentA -> ComponentB, inside ComponentB I have a button tag that has :on-click={"select-item", target: "#component-a-dom-id"}. Prior to upgrading to Surface v0.7 (from 0.6.x), the handle_event function defined on ComponentA would process the event, but after upgrading the handle_event function on the parent LiveView is being called instead, causing the LiveView to crash.

I'm also seeing something similar where an event from a stateful component isn't respecting the target given, and is instead attempting to be handled by the LiveView.
With a set up of LiveView -> ComponentA -> ComponentB, inside ComponentB I have a button tag that has :on-click={"select-item", target: "#component-a-dom-id"}. Prior to upgrading to Surface v0.7 (from 0.6.x), the handle_event function defined on ComponentA would process the event, but after upgrading the handle_event function on the parent LiveView is being called instead, causing the LiveView to crash.

In the above scenario, the HTML generated by v0.6 is as follows:
<button phx-click="select-item" phx-target="#component-a-dom-id" class="block w-full h-full" phx-value-item-id="x" phx-value-area-id="y">
for v0.7 it generates:
<button phx-click="[["push",{"event":"select-pupil","target":"#component-a-dom-id"}]]" class="block w-full h-full" phx-value-item-id="x" phx-value-area-id="y">

I'm also seeing something similar where an event from a stateful component isn't respecting the target given, and is instead attempting to be handled by the LiveView.
With a set up of LiveView -> ComponentA -> ComponentB, inside ComponentB I have a button tag that has :on-click={"select-item", target: "#component-a-dom-id"}. Prior to upgrading to Surface v0.7 (from 0.6.x), the handle_event function defined on ComponentA would process the event, but after upgrading the handle_event function on the parent LiveView is being called instead, causing the LiveView to crash.

In the above scenario, the HTML generated by v0.6 is as follows:
<button phx-click="select-item" phx-target="#component-a-dom-id" class="block w-full h-full" phx-value-item-id="x" phx-value-area-id="y">
for v0.7 it generates:
<button phx-click="[["push",{"event":"select-item","target":"#component-a-dom-id"}]]" class="block w-full h-full" phx-value-item-id="x" phx-value-area-id="y">

Hi @Menkir and @elliotb!
That's the expected behavior. Surface can only set the target automatically as @myself when the event is defined inside a live component, i.e., inside the render/1 function of a module using SurfaceLive.Component. In the provided example:
  def button(assigns) do
    ~F"""
    <button :on-click="click"></button>
    """
  end

The event "click" is defined inside an ordinary function, which has no knowledge of any assign called @myself.  That function can be called by any other component, live component or liveview so we can not blindly assume it will have that assign or not. For cases like this, you have to pass the target explicitly. Like:
defmodule TestWeb.Component do
  use Surface.LiveComponent

  def render(assigns) do
    ~F"""

component

      <.button target={@myself}/>

    """
  end

  def button(assigns) do
    ~F"""
    <button :on-click={"click", target: @target}></button>
    """
  end
  ...
end

Ideally, to could make the function even more flexible and pass the event as a whole:
defmodule TestWeb.Component do
  use Surface.LiveComponent

  def render(assigns) do
    ~F"""

component

      <.button click={"click", target: @myself}/>

    """
  end

  def button(assigns) do
    ~F"""
    <button :on-click={@click}></button>
    """
  end
  ...
end

Thank you @msaraiva  for the explanation!

Thanks @msaraiva.
In my case, it was that using the DOM ID as the target wasn't working correctly.
I've refactored ComponentB to have an event prop for select_item, and then in ComponentA I set this to {"select-item", target: @myself} and this then works as expected. As a bonus it's neater too as ComponentB no longer needs to have knowledge of the DOM ID of ComponentA.


14822: CVUT coverage: Update source code of the PBIVIZ tools in order to fix the watching mode

14822: CVUT coverage: Update source code of the PBIVIZ tools in order to fix the watching mode
Power BI Visuals Tools 1.4.3

Hi @ignatvilesov, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Ignat Vilesov (Akvelon)). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;


Emacs like search file buffer for mini.pick

Contributing guidelines

[X] I have read CONTRIBUTING.md
[X] I have read CODE_OF_CONDUCT.md

Module(s)
mini.pick
Description
I'd love there to be a way to customize/ theme mini.pick for it to look like the Emacs search buffer.

Thanks for the suggestion!
This is certainly possible. Take a look at how source can be defined. In this particular case the easiest approach would be to precompute items yourself (maybe based on some CLI tool) and modify choose (and maybe preview).
If you want it to be a file explorer (that is, show also directories and navigate inside them on choosing), that would be a harder task. There is a pickers.explorer picker in 'mini.extra' to have a similar feel (but showing only basename without extra information). Or just use 'mini.files' :)
Closing as not planned, as this is likely won't be built-in.