EC key throwing OpenSSL::PKey::EC::Point::Error

I'm encountering this error when generating id tokens in my app using doorkeeper-openid_connect. I'm using an EC key/512 curve. For some reason, I'm getting an exception at this point in the code: https://github.com/nov/json-jwt/blob/master/lib/json/jwk.rb#L140
The error is: OpenSSL::PKey::EC::Point::Error: invalid encoding
I tried reproducing this directly with a fresh EC key. Here are the steps I took:

Generated a new key pair (Mac 10.11.6, installed openssl through homebrew)

➜  ~ openssl ecparam -name secp521r1 -genkey -out test.pem
➜  ~ openssl ec -in test.pem
read EC key
writing EC key
-----BEGIN EC PRIVATE KEY-----
MIHbAgEBBEEYQPgaKkRZSDaO7vm7tNsT9Nn2PogMwzM0FcYQqS1LKH4SBvm94vTC
dlwf4L+dyOFmP7PgytYIENwRp885GTp3NqAHBgUrgQQAI6GBiQOBhgAEAF04RR8v
jqzJLY4rM7aaQOcLNj7x5dLueq1YP7WXDM0SLo6Sr0odCWdqa0injaMBzzosQ56K
MCZXzCiSmt9YkI3MAA8D8QGG04Q1+ewtrthARff3BgOU9nhUCNVgv36aqcXANkue
vYRayeeFW5UA7dbNPvJrM6ZFH/Wxk1cWHJGl5K/d
-----END EC PRIVATE KEY-----

In Ruby, initialized a new JWK from this private key:

[42] pry(main)> key = "-----BEGIN EC PRIVATE KEY-----\nMIHbAgEBBEEYQPgaKkRZSDaO7vm7tNsT9Nn2PogMwzM0FcYQqS1LKH4SBvm94vTC\ndlwf4L+dyOFmP7PgytYIENwRp885GTp3NqAHBgUrgQQAI6GBiQOBhgAEAF04RR8v\njqzJLY4rM7aaQOcLNj7x5dLueq1YP7WXDM0SLo6Sr0odCWdqa0injaMBzzosQ56K\nMCZXzCiSmt9YkI3MAA8D8QGG04Q1+ewtrthARff3BgOU9nhUCNVgv36aqcXANkue\nvYRayeeFW5UA7dbNPvJrM6ZFH/Wxk1cWHJGl5K/d\n-----END EC PRIVATE KEY-----"; nil
=> nil
[43] pry(main)> pkey = OpenSSL::PKey.read(key)
=> #<OpenSSL::PKey::EC:0x00000009714158>
[44] pry(main)> jwk = JSON::JWK.new(pkey)
=> {"kty"=>:EC,
 "crv"=>:"P-521",
 "x"=>
  "XThFHy-OrMktjisztppA5ws2PvHl0u56rVg_tZcMzRIujpKvSh0JZ2prSKeNowHPOixDnoowJlfMKJKa31iQjcw",
 "y"=>
  "DwPxAYbThDX57C2u2EBF9_cGA5T2eFQI1WC_fpqpxcA2S569hFrJ54VblQDt1s0-8mszpkUf9bGTVxYckaXkr90",
 "d"=>
  "GED4GipEWUg2ju75u7TbE_TZ9j6IDMMzNBXGEKktSyh-Egb5veL0wnZcH-C_ncjhZj-z4MrWCBDcEafPORk6dzY",
 "kid"=>"JjqJwuVmuEXrpHkSpljVnN9zHbUyUT1-cMCHfJId2xM"}

Converted this back to a key, which is what JWS does during signing:

[45] pry(main)> jwk.to_key
OpenSSL::PKey::EC::Point::Error: invalid encoding
from /usr/local/bundle/gems/json-jwt-1.7.2/lib/json/jwk.rb:140:in `initialize'

Is there something wrong with this approach? Or is this a bug in the JWK implementation for EC keys? Maybe something else? I'm at a bit of a loss for how to continue debugging.
Thanks in advance for any help!

Also, if it helps, this is Ruby v 2.4.1

try to generate ec keys using openssl gem, not openssl itself.
openssl gem could be linked with older version of openssl.

Ah, if the x / y start with "\00", the error occurs.

will look at this issue when I have more time.
require 'json/jwt'

module JSON
  class JWK
    module JWKizable
      module EC
        def coordinates
          unless @coordinates
            puts '# coordinates'
            hex = public_key.to_bn.to_s(16)
            data_len = hex.length - 2
            hex_x = hex[2, data_len / 2]
            hex_y = hex[2 + data_len / 2, data_len / 2]
            @coordinates = {
              x: OpenSSL::BN.new([hex_x].pack('H*'), 2),
              y: OpenSSL::BN.new([hex_y].pack('H*'), 2)
            }
            puts [hex_x].pack('H*').size, [hex_y].pack('H*').size
            puts @coordinates[:x].num_bytes, @coordinates[:y].num_bytes # NOTE: size changes here if hex_x / hex_y strat with "\00"
            @coordinates[:d] = private_key if private_key?
          end
          @coordinates
        end
      end
    end
  end
end

[
  'prime256v1',
  'secp384r1',
  'secp521r1'
].each do |crv|
  puts crv
  ec_key = OpenSSL::PKey::EC.new(crv).generate_key
  ec_jwk = ec_key.to_jwk
  ec_key2 = JSON::JWK.new(ec_jwk).to_key
end

@nov any update on this? Let me know if I can help in any way.

sorry, I’m not able to take enough time on this issue right now...

Understood! Sorry for bugging you, and thanks for writing cool software.

Oops, my "fix #NN" commit comment closed this issue automatically.
I somehow seems fixed this issue.
If you could confirm the fix on your side too, it would be very helpful.
This fix is now included in v1.8.1.pre, and you can install it via gem install json-jwt --pre.
thanks.

@nov it works! Thank you so much for fixing this!

thanks for your confirmation.
I’ll release it shortly.

sorry for my late release!!

No problem! Thanks again.

I am trying to deploy a staging serve by
bundle exec cap staging deploy
But getting this error, Any help, please
cap aborted!
SSHKit::Runner::ExecuteError: Exception while executing on host <IP_ADDRESS>: invalid encoding
Caused by:
OpenSSL::PKey::EC::Point::Error: invalid encoding
Tasks: TOP => rbenv:validate
(See full trace by running task with --trace)

@sureshcvh try resetting (delete) your known_hosts file.


Javascript and Haxe implementations does not provide "porting" functions

https://www.hsluv.org/implementations/ states that all ports must provide a set of functions, but the javascript and haxe implementations does not provide those functions anymore.

They are still provided, the calling convention is changed to avoid heap allocation, but that's documented in the respective repos' READMEs.


LUCENE-10140: Non-heuristic intervals sub-matches caching

Description
Minimizing intervals (maybe just ORDERED and AT_LEAST, but not sure) can move sub iterators to non-sub-match position inside match window, but CachingMatchesIterator logic relies on heuristic that any position inside matching interval is a sub-match.
For example: ORDERED("a", "b", "a") over "a b a" highlights (report sub-matches) only "a b a", and ORDERED("a", "b", "a", "b", "a") highlights only "a b a b a".
https://issues.apache.org/jira/browse/LUCENE-10140
Solution
Looks like there is no way to determine the right moment to cache from caching iterator perspective, so I propose to add an interface allowing minimizing IntervalIterators notify sub-sources positioned at sub-match positions.
There is a distinct pointcut for such notification: a slop calculation inside nextInterval.
Also, I think, MinimumShouldMatchIntervalsSource deserves some refactoring after this and I'm not sure that BLOCK is actually a minimizing interval source.
This patch resolving LUCENE-10075 (#270) because it removes extra endPosition() call after reaching last interval.
Tests
Added test for LUCENE-10075 and a new test for "a b a b a" highlighting.
Checklist
Please review the following and check all that apply:

[x] I have reviewed the guidelines for How to Contribute and my code conforms to the standards described there to the best of my ability.
[x] I have created a Jira issue and added the issue ID to my pull request title.
[x] I have given Lucene maintainers access to contribute to my PR branch. (optional but recommended)
[x] I have developed this patch against the main branch.
[x] I have run ./gradlew check.
[x] I have added tests for my changes.

@romseygeek , could you please take a look?


Not specify the class property raises an error

The following code:
<Heroicons.Components.Icon type="outline" name="academic-cap" />

raises the following error:
** (exit) an exception was raised:
    ** (FunctionClauseError) no function clause matching in Surface.css_class/1
        (surface 0.4.0) lib/surface.ex:215: Surface.css_class(nil)
        (ex_heroicons 0.1.1) lib/heroicons/components/icon.ex:16: anonymous fn/2 in Heroicons.Components.Icon.render/1

Seems to be fixed on master!

@Malian published the fix


Add make target for custom-deploy

Issue link

Closes #152
The changes here are similar to what is done in this PR for MCAD
What changes have been made

The custom-deploy make target can be used for building and pushing a custom image of InstaScale and deploying it on your K8s cluster for development.
Added documentation to Readme.

Note: using the name custom-deploy for the make target as there is already an existing easy way of deploying InstaScale (latest) to your K8s cluster. This make target focuses on the custom image deployments.
Verification steps

Example: make custom-deploy ENGINE=<podman or docker> IMG=quay.io/<username>/instascale:<image tag>
Checks

[x] I've made sure the tests are passing.
Testing Strategy

[x] Manual tests

I have run the make command on an OSD cluster and can confirm that an instascale-controller-manager deployment and pod are created where I can view Instascale logs. I applied an appwrapper and can see that instascale picks it up as expected also. Just one small comment regarding the command in make file. Otherwise it looks good.

@Fiona-Waters Brilliant, thanks for testing it out!

lgtm thanks @ChristianZaccaria


feat: implement OneLogin provider

Implemented with OIDC from documentation here: https://developers.onelogin.com/openid-connect and https://developers.onelogin.com/openid-connect/api.

tested on my test instance for onelogin

I will wrap up the docs this week :)

Docs have been updated, this is ready for review whenever you have the time @pilcrowOnPaper  😸

We're deprecating the OAuth integration in v3, and it's getting replaced by Arctic. Is it possible to create a PR against that project instead?

i can definitely do that 😸


Facing issue while calling notion API through serverless framework

facing an issue while calling an notion API with my "serverless" and "serverless-offline" framework when running a API getting an below error as,
getting issue when calling a method,
auth:'secret_UjR5Ta6Y08zYmIvGoTY2XJVfUUSDdAaASh78rWxxaNF'
database_id = '9a972ddf-dbb5-4877-a87e-35d7aa584cf3';
const payload={
path:'databases/'+database_id+'/query',
method:'POST'
}
const {results} = await notion.request(payload);
Node 10.X :
(node:8744) UnhandledPromiseRejectionWarning: RequestError: Cannot read property 'startsWith' of undefined
at Request._makeRequest (.....\node_modules@notionhq\client\node_modules\got\dist\source\core\index.js:1177:19)
at Request (....\node_modules@notionhq\client\node_modules\got\dist\source\core\index.js:328:28)
Node 12.X
Request Error: connect ECONNREFUSED
Tried with different versions of Node 10 - 14 , also working when we are not using serverless-framework, the only issue while running with serverless-framework.
Please let us know any suggestions / supporting methodology.
Below is the package.json file for reference.
"devDependencies": {
"@notionhq/client": "^0.1.9",
"eslint": "5.16.0",
"nyc": "14.1.1",
"serverless": "1.46.1",
"serverless-mocha-plugin": "1.10.0",
"serverless-offline": "5.3.3",
}
when browsed, suggested to check with serverless framework (refer below link)
https://githubmemory.com/repo/NotionX/react-notion-x/issues/52

You should really really not include your integration key and database id

probably blocked by CORS policy. Check out https://github.com/Rob--W/cors-anywhere .

Closing issue since it seems like the issue isn't directly related to the SDK.


Move mobile_manipulation ws to projects

It makes more sense to have the workspaces in the projects folder rather than in the lib folder

Hi @dHonerkamp, could you do the review of this PR?


Add CowSwap

Revision checks

[x] Your Safe App include a manifest.json file at the root with the required data
[x] The app can be loaded as a custom Safe App in the Apps section of https://gnosis-safe.io/app
[x] The app auto-connects to the Safe
[x] You are able to trigger and execute one transaction with a Safe
[x] RPC requests are optimized (not triggering many requests in a very short time period)

Name/Description

CowSwap
Gasless MetaDEX Aggregator with front-running and Peer-to-Peer trades
Type

New addition

Compatible Networks

- Mainnet
- Rinkeby
- Gnosis Chain

Code for review

https://github.com/gnosis/cowswap
IPFS hash/App URL

https://cowswap.exchange
Desired date

The soonest

@dasanra The Cowswap app connects to my Metamask instead of the Safe. 🤔

@dasanra The Cowswap app connects to my Metamask instead of the Safe. 🤔

Small bug when you used CowSwap directly before using it with the Safe
We are already syncing to solve it


Unrecognized function or variable 'flipping'.

Hi, I was trying to run the algorithm with my own data and got an error of missing function 'flipping'. What is the definition of this function? Thanks.
The details of the error msg:
Unrecognized function or variable 'flipping'.

Error in sincRigidTransformQuick (line 45)
        if et{5}(m)==1;x=flipping(x,et{4}{m});end%FLIPPING FOR LARGER THAN 90DEG ROTATIONS

Error in svrEncode (line 39)
        if
        useT>4;x=real(sincRigidTransformQuick(x,precomputeFactorsSincRigidTransformQuick(svr.kGrid,svr.rGrid,dynInd(svr.TV,v,5),1,0,[],1,svr.cGrid),1,svr.FT));end

Error in svrCG (line 17)
    svr=svrEncode(svr,-1);

Error in svrAlternateMinimization (line 35)
    svr=svrCG(svr,3+(nF<2)+(nF<3));

Sorry, shifting.m was also missing, just committed.

Thanks for the fix!


fix: actually remove the feature group when needed

Fixes #163

That's great! Any chance you would be interested in adding a test to show that it's working?
My typical pattern with adding tests for things like this:

If it doesn't already fit into one of the pages of folium.streamlit.app, add a new one
Run the app locally, and use https://playwright.dev/python/docs/codegen to check that the element exists, or that clicking on some widget makes the element appear (or in this case, disappear) from the map
Add that as a new test in test_frontend.py

That might be a bit overkill for a small change like this, but it means that I can be confident I won't accidentally break it later

Okay! I will close this PR and open a new one with more features and tests


IndexError: index out of range (ERROR: test_cqc_extractor)

Information

Qiskit IBM Provider version:
Python version:
Operating system:

What is the current behavior?
Push test failing with error:
======================================================================
ERROR: test_cqc_extractor (test.ibm.test_random.TestRandomIntegration)
Test invoking the CQC extractors.
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/Users/runner/work/qiskit-ibm/qiskit-ibm/test/ibm/test_random.py", line 75, in test_cqc_extractor
    ext1_out = job1.block_until_ready(timeout=300)
  File "/Users/runner/work/qiskit-ibm/qiskit-ibm/qiskit_ibm/random/cqcextractorjob.py", line 113, in block_until_ready
    self._result = bytes_to_bitarray(raw_bytes, self.parameters['ext1_output_num_bits'])
  File "/Users/runner/work/qiskit-ibm/qiskit-ibm/qiskit_ibm/random/utils.py", line 30, in bytes_to_bitarray
    return [(the_bytes[i >> 3] >> (i & 7)) & 1 for i in range(num_bits)]
  File "/Users/runner/work/qiskit-ibm/qiskit-ibm/qiskit_ibm/random/utils.py", line 30, in <listcomp>
    return [(the_bytes[i >> 3] >> (i & 7)) & 1 for i in range(num_bits)]
IndexError: index out of range

----------------------------------------------------------------------

Steps to reproduce the problem
What is the expected behavior?
Suggested solutions

This is not urgent for initial release since we are skipping this test in the qobj PR https://github.com/Qiskit-Partners/qiskit-ibm/pull/49

We are skipping the test because the fix for qobj PR requires a qiskit_rng release, which we can do before the qiskit-ibm release. If we don't fix this before the release, people just won't be able to migrate to qiskit-ibm if they want to use qiskit_rng (a bit annoying but not the end of the world).

The test_cqc_extractor test case passes if it runs alone but when it's run with the other tests inside fo test_random.py, it fails:
Here in the block_until_ready method the class object passed in is a FakeRandomClient instead of just RandomClient. Passing in  FakeRandomClient eventually calls the get_object_storage method inside of test_random instead of the method here in random.py . This makes the result_url equal to fake_result_url causing the number of bytes to be incorrect, which is what gives the index out of range error here.
Not sure why this only started happening after the migration but maybe the easiest solution is to just move the cqc test into a different file?

@kt474 Very good finding!
Disabling the account once the TestRandom class test cases are done helps fix this issue:
Could you add below to TestRandom class and open a PR?
@classmethod
    def tearDownClass(cls) -> None:
        """Class level teardown."""
        super().tearDownClass()
        IBMProvider._disable_account()


Buffer pool design

start with simple consumer / producer design

Two queues per backend logger (cond var)
force putBuffer() for every fsync
bufferPool code for SiloR

From the Silo paper and code:

log-buffer backpressure is not used in SiloR. SiloR’s loggers instead recirculate log buffers back to workers as soon as possible— after a write, rather than after the following epoch change and fsync.
spinlock is used when no log buffer is available

source code


CI is incorrectly returning success when it should be returning failure

I got the following message via Email from CI:
Test 79426650eeb77ae(eeb77ae) has successfully finished without an error
See https://github.com/ThaliTester/TestResults/tree/79426650eeb77ae_Almost_to_the_top_of_the_Thali_stack_yaronyg/ for the logs
Which was both exciting and of course mystifying since it shouldn't be working. In fact when I groveled through the logs I saw that we are failing on 'Cannot find module '../thalilogger' but I'll file a separate bug on that.
This bug is - why the heck did I get a mail saying the tests passed when they didn't?

This is related to the issue https://github.com/thaliproject/Thali_CordovaPlugin/issues/594 (App fails to load on some devices to error "No such native module"). The fix for this is already done with following PR: https://github.com/thaliproject/jxcore/pull/11
There was a code introduced to simply report success when app.js fails to load (so that we are not stopped with this jxcore issue):
https://github.com/thaliproject/Thali_CordovaPlugin/commit/84971526dd7e936d975cf2ffc4779e45f307ba75
This is not CI issue but the Cordova plugin issue.

The code that reports success when app.js fails to load needs to be removed at this point. In fact, it really shouldn't have been there at all. Instead we should have removed the node tests until we were ready again. Please confirm that code is gone.

O.k. I get it. I filed https://github.com/thaliproject/Thali_CordovaPlugin/issues/842.
@enricogior depending on how many failures we see as a result of pulling the work around here for https://github.com/thaliproject/jxcore/pull/11 we may need to push out a release of JXCore-Cordova earlier than I thought in order to use a version of JXcore that fixes that problem. We'll see.


How to make plain function to be visible from within the worker?

Hi Frédéric,
At first I would like to say thank you for the greate stuff you provided!
Unfortunately, I have an issue I can't resolve. I'm using the GraniteDS calsses inside worker. Everything works well except one thing. When I try to create org.granite.math.BigDecimal I get this error
[Fault] exception, information=ReferenceError: Error #1065: Variable org.granite.math::assert is not defined.
Due to the code of BigDecimal references to plain function org.granite.math::assert that is somehow not seen. I can't pass this name as extraDeps due to assert has Function type and not Class and defined like this.
package org.granite.math {
function assert(value:Boolean, message:String = null):void {
//...
}
}
Is it possible to somehow make it work?
Thanx in advance.

Sorry, I totally missed that.
You could be right, I will have a look and come back to you.

Sorry again for the delay, I never find the time to work again on this until today, my guess is there is a workaround adding "org.granite.math.*" as extraDependencies

Hi Frédéric,
Thanx for reply. Could you please tell me whether a way to optimize the swf for size via e.g. 3rd party tools, but keep workers working?

Not sure it is doable, I will investigate.


fix(postcss): tidy up misplaced @charset directives

Since the latest major PostCSS upgrade, @charset directives that
aren't placed at the top of a file lead to the warning: postcss-import: @charset must precede all other statements.
Since those misplacements can happen in third-party dependencies and are
thus hard to fix, we rather want to fix this for good in a central place
— namely our PostCSS Core Mixin.
Remark 1.) I set the plugin's option add to false, because this PR does not aim at fixing some CSS files's encoding issues.
Remark 2.) The diff doesn't reflect my renaming of the integration/postcss/__tests__/build.js to integration/postcss/__tests__/production.js before creating a new build.js with completely new contents. So for the purpose of reviewing this PR, it suffices to focus on the addition to the integration/postcss/__tests__/build.js-file concerning the spec/ folder.

Bors merge bot cheat sheet
We are using bors-ng to automate merging of our pull requests. The following table provides a summary of commands that are available to reviewers (members of this repository with push access) and delegates (in case of bors delegate+ or bors delegate=[list]).

Syntax
Description

bors merge
Run the test suite and push to master if it passes. Short for "reviewed: looks good."

bors merge-
Cancel an r+, r=, merge, or merge=

bors try
Run the test suite without pushing to master.

bors try-
Cancel a try

bors delegate+
Allow the pull request author to merge their changes.

bors delegate=[list]
Allow the listed users to r+ this pull request's changes.

bors retry
Run the previous command a second time.

This is a short collection of opinionated commands. For a full list of the commands read the bors reference.

bors merge


Geodesic drawn Path LatLng List

can we have list of latlng of geodesic path being drawn by google maps?

yes, you're right. i need to have list of latlng that google map draws when geodesic is set to true.
like this https://i.stack.imgur.com/EArBf.png
for suppose if i want a geo desic polyline from one point to another (london to istanbul, turkey), the polyline ARC/CURVE will be created by google if i give it both latlng of source and destination and set geodesic property to TRUE but can i have those drawn arc points (which will be set of latlngs or list of latlngs) that google map creates
thanks

To get a list of LatLng points of a Curve, register a callback interface to be invoked when the curve is drawn on google maps.
curveManager.setOnCurveDrawnCallback(this);
Use getPoints() method of a Curve reference which returns a list of LatLng points.
@Override
public void onCurveDrawn(Curve curve) {
    List<LatLng> latLngList = curve.getPoints();
}

To get a list of LatLng points of a geodesic polyline, call getPoints() on a Polyline reference.
Check the below link for more info.
https://gis.stackexchange.com/questions/194532/how-to-get-polyline-latitude-and-longitude-by-click-on-polyline-android
I hope the above is useful to you.

but the library of your curve draws a different curve than the geodesic
can't it be possible to draw a simple geodesic polyline and get points? the getpoints of polyline gives the source and destination points only, not all the points

Setting the geodesic option to true did not help me much when I was working on a project. So I decided to use the Bezier cubic equation to draw a curve between the 2 points.
The curve drawn by the library, consists of several small polylines (200 points). AFAIK, setting the geodesic option to true does not make much difference here.
You can try one more trick. Use setAlpha option and set the alpha value to zero and you will get a straight line in most cases. After that, try to get a list of points of a curve (Make sure it satisfies your requirement). If you want a perfectly straight line, use setComputePointsBasedOnScreenPixels option and set it to true (Read the limitation of this method before using it).

no, i actually want to have a geodesic curve and i need an animation like i showed you with a reference link

If you want a geodesic curve/line drawn by the google map, try to use polyline directly. Let me know if you find any solution to your problem.

Closing because of inactivity.


Remove QA tag from Demo site

The title on the demo site still says "Encompass - QA"

seems fixed now!

Jk-- this changes occasionally. Sometimes it has the Demo tag and sometimes it says QA. I'm not sure what causes it to change or if it is more likely to happen after a deploy.


Warn on nonexistent file listed as an asset

If a user lists a nonexistent file as an asset, it would be very helpful to get a warning about it.
e.g.
[package.metadata.deb.variants.debug]
assets =  [
    ["dir/fake_file.txt", "/var/log", "755"],
]

might yield
Warning: 'dir/fake_file.txt' does not exist and will not be added to deb
Missing unit-names would also be nice, while trying to use a systemd service template it wasn't obvious to exlude the @

And this issue now has a link to merge request #70.


Gitter bridge broken?

Describe the bug
time="2023-03-30T13:30:45-07:00" level=fatal msg="Starting gateway failed: Bridge gitter.groupincome failed to start: Get "https://api.gitter.im/v1/user": dial tcp: lookup api.gitter.im on <IP_ADDRESS>:53: no such host" prefix=main

It appears to fail to lookup api.gitter.im. I tried on my local machine and this domain appears to be down/not registered. Confirmed here: https://mxtoolbox.com/SuperTool.aspx?action=a%3Aapi.gitter.im&run=toolpage
What to do?
To Reproduce
Here's our config:
[gitter]
[gitter.groupincome]
Token="xxx"
RemoteNickFormat="<{NICK}> "

[slack]
[slack.groupincome]
Token="xxx"
PrefixMessagesWithNick=true
RemoteNickFormat="<{NICK}> "
NickFormatter="plain"
NicksPerRow=4
EditSuffix=" (edited)"

[[gateway]]
name="gateway1"
enable=true

    [[gateway.inout]]
    account="slack.groupincome"
    channel="groupincome"

    [[gateway.inout]]
    account="gitter.groupincome"
    channel="okTurtles/group-income"

Expected behavior
For the bridge to work
Screenshots/debug logs
[0000]  INFO main:         [setupLogger:matterbridge.go:104] Enabling debug logging.
[0000]  INFO main:         [main:matterbridge.go:44] Running version 1.26.0 6dafebc7
[0000]  INFO router:       [Start:gateway/router.go:66] Parsing gateway gateway1
[0000]  INFO router:       [Start:gateway/router.go:75] Starting bridge: gitter.groupincome
[0000]  INFO gitter:       [Connect:bridge/gitter/gitter.go:27] Connecting
[0000] FATAL main:         [main:matterbridge.go:66] Starting gateway failed: Bridge gitter.groupincome failed to start: Get "https://api.gitter.im/v1/user": dial tcp: lookup api.gitter.im on <IP_ADDRESS>:53: no such host

Environment (please complete the following information):

OS: linux
Matterbridge version: 1.26.0 6dafebc7

Additional context
See above for config.

Gitter no longer exists, it's Matrix now. For instructions, see https://github.com/42wim/matterbridge/issues/1969 which this is a duplicate of.

Thanks, then maybe this issue can just exist as a request to update the documentation to reflect that.


fix(26): Update ClockUtilTest

Make both (test time and utility time) to use 0 seconds

Types of changes

[x] 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 change)
[ ] Other non-breaking change (enhancement, tests, documentation, chore and the like)

And

[x] This Pull Request DOES NOT target the main (production) branch on this repository (if so it
must point to a feature branch or to the develop (staging) branch).
[x] I have read the CONTRIBUTING
document.
[ ] I have added/updated tests to cover my changes.
[x] All new and existing tests passed.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] There are no other open Pull Requests
for the same update/change.

Fixes

closes #30

Codecov Report

Merging #31 (a9f4ccc) into develop (4104fb3) will increase coverage by 2.75%.
The diff coverage is 96.05%.

@@              Coverage Diff              @@
##             develop      #31      +/-   ##
=============================================
+ Coverage      84.49%   87.25%   +2.75%
- Complexity        94      124      +30
=============================================
  Files             21       26       +5
  Lines            187      251      +64
  Branches           9       16       +7
=============================================
+ Hits             158      219      +61
- Misses            27       28       +1
- Partials           2        4       +2

Impacted Files
Coverage Δ

...handler/impl/ArgumentNotValidExceptionHandler.java
86.95% <86.95%> (ø)

...org/wcdevs/blog/core/rest/AppExceptionHandler.java
100.00% <100.00%> (ø)

...g/core/rest/errorhandler/AbstractErrorHandler.java
100.00% <100.00%> (ø)

...og/core/rest/errorhandler/ErrorHandlerFactory.java
100.00% <100.00%> (ø)

...devs/blog/core/rest/errorhandler/ErrorMessage.java
100.00% <100.00%> (ø)

...ndler/impl/DataIntegrityViolationErrorHandler.java
100.00% <100.00%> (ø)

...e/rest/errorhandler/impl/NotFoundErrorHandler.java
100.00% <100.00%> (ø)

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 3aebb26...a9f4ccc. Read the comment docs.


ICCV has already concluded, but there is still no code, right?

ICCV has already concluded, but there is still no code, right?

+1

I'm also looking forward to the code release.

+1


I sugguest outFiles, who can act M: N merge

Ordinary "outFile" is
M: 1 Merging
When we write lots of codes that are difficult write in few TS files, however the codes should be merged into a single JS file, then we can take one of them:

~Use module system like CommonJS or RequireJS~
Use outFile

The outFile parameter in tsconfig.json lets TSC to merge multiple ts files into a single js file.
{
    "compilerOptions": {
        "target": "es5",
        "outFile": "include/something.js"
    },
    "files": ["first.ts", "second.ts", "third.ts", "fourth.ts"]
}

M: N Merging?
The outFile merges into only a single JS file. If someone wants to merge multiple TS files into multiple JS files (less fewer), then how should do? Using module system is a standard solution now, But I think the outFiles can be an alternative solution.

~Use module system like CommonJS or RequireJS~
(Suggestion) Use outFiles

Specification
Example
{
    "compilerOptions": {
        "target": "es5",
        "module": "commonjs"
    },
    "outFiles": {
        "list.js": ["src/list/list.ts", "src/list/list_iterator.ts", "src/list/list_reverse_iterator.ts"],
        "tree_map.js": ["src/tree_map/tree_map.ts", "src/tree_map/rb_tree.ts"],
        "hash_map.js": ["src/hash_map/hash_map.ts", "src/hash_map/hash_buckets.ts"],
        "algorithm.js": ["src/algorithm/"],
        "filesystem.js": ["src/filesystem/"],
        "functional.js": ["src/func_*.ts"]
    }
}

Using module system is possible
Not like outFile, I wish to using module system is possible in the outFile.
Arrange JS files to be generated and specify TS files to be merged
    "outFiles": {
        "list.js": ["src/list/list.ts", "src/list/list_iterator.ts", "src/list/list_reverse_iterator.ts"],
        "tree_map.js": ["src/tree_map/tree_map.ts", "src/tree_map/rb_tree.ts"],
        "hash_map.js": ["src/hash_map/hash_map.ts", "src/hash_map/hash_buckets.ts"]
    }

To act M: N merging, declare an object named outFiles and arranges JS files to be generated and TS files to be merged. Path of JS files are key_type and Array of TS files are value_type.
Specifying Directories
    "outFiles": {
        "algorithm.js": ["src/algorithm/"],
        "filesystem.js": ["src/filesystem/"],
        "to_be_generated.js": ["some_directory/", ...]
    }

Not only specifying TS files, but also specifying a directory would be better. When a directory is referenced in the Array, then all the TS files in the directory will be merged into the target JS file.
Conditional Specification
    "outFiles": {
        "functional.js": ["src/func_*.ts"]
    }

I think not only using the exact, pull path of TS files, but also using the conditional specification would be better.
Comparing with Module System
Convenient Modularization
M: N merging, it is also possible through the module system. However, the outFiles can be more convenient in some cases.

Let's assume that you're going to develop a library (I assume that the library is STL). Some modules (<algorithm> and <functional>) of the library has hundreds of global functions.
You created dozens of TS files because codes of the global functions are very long. Now, you should merge the dozens of TS files into JS files (std/algorithm.js, std/functional.js). With the module system, you may write such code:
// algorithm.ts
export * from "./algorithm/alg1";
export * from "./algorithm/alg2";
export * from "./algorithm/alg3";
export * from "./algorithm/alg4";
...
 Note that, outFiles is not a conception has exclusive relationship between the module system. outFiles can be harmonized with the module system.

API Documentation
To generate clean and neat API documents (unless writes API documents by hand manually), smaller number of modules are much favorable.
If not using the Module System
<script src="include/std/vector.js"></script>
<script src="include/std/list.js"></script>
<script src="include/std/map.js"></script>
<script src="include/std/algorithm.js"></script>

Developing web application without module system, but including JS files directly, it's not normal case in nowadays. However, it's not extinct yet.
For the minorities, not only providing outFile method, but also providing the outFiles method will be much helpful.

Nice write-up and we would probably use this internally if it existed. But we're generally pushing people toward using modules and external bundlers to produce output files and hope that out as a concat method eventually fades away from major use, so we're not looking to invest in this upfront. But if other people would find the useful (please :+1: we can revisit).

Thanks for responding. I'm developing some web applications that cannot avoid using many Workers. The Worker requires an independent JS file and the Worker's file has its own specific import statement; importScripts.
importScripts
(
    "tstl.js",
    "samchon.js",
    "statistics.js",
    "geometry.js"
);

In the web development and using Worker, then I think the outFiles will be much helpful than ordinary module system like CommonJS.

I like the proposal as well. My biggest issue is "Debugging" as the Visual Studio debugger does only allow me to debug the files in one (VS) project. I therefore have to make a decision to have all my TS files compiled into separate JS files or combine them ALL into one JS File. I cannot create layers of multiple TS files compiling into multiple (but fewer) JS files.
The proposal on here would allow me to create layers like "Core.JS", "Communication.JS", "UX.JS" (outfiles) and each of them having multiple TS files.
I can then debug my project easy in VS.
Right now I have a primary TS project with all my "Stack" TS files compiling into one JS file and a second project with my higher level "stack API consuming" TS files. The Index.html loads both resulting JS files but I can only debug either the stack or the consumer - not both (across the two projects).
The multiple outfiles solution would help me with this layered structure (without using any other tools that merge files and make debugging impossible).


Investigate how to get badge screen reader functionality working for NVDA

Summary
Investigate what can be done to get the badge component's screen reader functionality working for NVDA.
💬 Description
A colleague who is able to use NVDA has reported that there are some issues with the badge's accessible text - in that the aria-label that is added to the parent component is not being read out. Some improvements have been made as part of #1391, which have been tested with VoiceOver and all appears to work fine, but getting information about the badge to be read out by NVDA still seems to be a problem.
It has been difficult to test different solutions due to NVDA being software that is only available on a Windows device. This separate ticket has been created for this reason; it is best if someone who has NVDA on their device can pick this up to make it easier to find a solution. It's possible that it is an NVDA bug.
Note: I added some information to the description for my PR which may help, e.g. the code which will allow you to display all possible parent components for a badge on one page for easier testing.
💰 User value
This will ensure that NVDA users can hear information about badges just like VoiceOver users can.
📚 User Stories
As a screen reader user who uses NVDA
I need to be able to hear all the accessible text for badge components that VoiceOver users can
So that I can fully understand that a badge is being displayed and what it is representing.

It's important to ensure that the badge label doesn't override any aria-label that might exist on the parent component.
Also, any changes you make to get NVDA working will need to of course still work with VoiceOver so would be good to get a VoiceOver user to test (if you don't have VoiceOver).

Gotten around to testing this, interestingly even in Voiceover the "with badge being displayed" accessible text is not read out on badges with text-label.
In NVDA, you can use up/down arrows to navigate to badges, and at least on buttons the order is 1: badge 2: button so it's not entirely inaccessible. At the very least I want to make sure all types of badge read the "with badge" text on voiceover, and then get that being read out in NVDA too. Maybe there's some way to change how the badge slot works and instead of adding an aria-label we hide some sr-only text with the full badge text in it?


SLIM-961 Cucumber Test AddDevice For GasMeters

retest this please

This pull request makes the build fail.


Implement dynamic resource IDs + example

Fixes #94

Still gotta add some more tests.

bors try

I’d recommend using the word Dynamic instead of Scripting, as otherwise I think it would be too precise to serve it well.
Have you assessed the costs for the current workflow (performance and API-wise)?

I’d recommend using the word Dynamic instead of Scripting, as otherwise I think it would be too precise to serve it well.

Oh, you mean in the example? I've used "dynamic" everywhere else AFAIK.

Have you assessed the costs for the current workflow (performance and API-wise)?

I've done some benchmarking, there was no noticeable performance cost.
API-wise - it's definitely not a high-level API, but it's a very flexible low level API. It should be easy to define your scripting framework with that.

r? @jojolepro @Moxinilian

Thank you!
bors r=Moxinilian,jojolepro


Show audience + envir metadata in time-to-read line

Add icon or text revealing:

admin vs. regular user
online vs. onsite

Find info in yml frontmatter metadata.
Place at the top of the page where we display time to read.

We would love to have an icon showing that this page is a admin page/setting page where you need to log into Settings and maintenance
Possible icon to show:
🔧 : wrench :

@digitaldiina change so.user to so.audience. Try to do the same for so.envir


lib/system.nim(1415, 7) Error: redefinition of 'xl'

Araq, the main problem seems to be that nimsuggest does not work well with system.nim.
For some def queries it works, but for most it seems to fail:
Line 1415 of system.nim is:
let xl = x.len - 1
nimsuggest --v2 --debug --stdin /home/stefan/Nim/lib/system.nim
def /home/stefan/Nim/lib/system.nim;/home/stefan/Nim/lib/system.nim:1415:15
lib/system.nim(1415, 7) Error: redefinition of 'xl'
lib/system.nim(1416, 22) Error: type mismatch: got (T) but expected 'T = GenericParam'
lib/system.nim(1417, 9) Error: type mismatch: got (seq[T], int)
When I give no dirtyfile I get no output at all.
Another working test:
nimsuggest --v2 --debug --stdin /home/stefan/Nim/lib/pure/parseutils.nim
def /home/stefan/Nim/lib/pure/parseutils.nim;/home/stefan/Nim/lib/pure/parseutils.nim:380:20
def	skProc	system.len	proc (x: string): int{.noSideEffect.}	/home/stefan/Nim/lib/system.nim	682	5	""	100
But again no output for
def /home/stefan/Nim/lib/pure/parseutils.nim:380:20

The problem is that type checking is not performed for generics and all your queries seem to target generic procs. Nimsuggest works with heuristics within generics which are still in development. This issue here is not system specific at all. I'm leaving this issue open, closing the rest and will think about what to do. :-)


Update ef.rst

Minor type fix UserSqlServer => UseSqlServer
What issue does this PR address?
Typo in the documentation
Does this PR introduce a breaking change?
No
Please check if the PR fulfills these requirements

[X] The commit follows our guidelines
[X] Unit Tests for the changes have been added (for bug fixes / features)

Other information:

ah thanks!


Saving/restoring latent

it works by dumping/restoring the whole EMA and ADAM optimiser objects
exemple of how to restore :
model = Imagine(
text = TEXT,
save_every = SAVE_EVERY,
lr = LEARNING_RATE,
iterations = ITERATIONS,
save_progress = SAVE_PROGRESS,
out_folder = out_folder,
save_latents = True,
saved_latents_filepath = r"F:\my\path\magic_man.35.backup",
)

can you also update cli.py so the parameter can be passed through the command prompt?

also should add dill to requirements.txt

I ran this and hit the following error:
>>> import big_sleep
>>> a = big_sleep.Imagine(num_cutouts=32, iterations=50, epochs=5, save_latents=True, text="a blue glass orb", save_every=5)
>>> a()
Imagining "a_blue_glass_orb" ...
loss: -17.00:   8%|██████████████████                                                                                | 4/50 [00:01<00:15,  2.91it/s]
      epochs:   0%|                                                                                | 0/5 [00:01<?, ?it/s]
Traceback (most recent call last):                                                                                | 1/50.0 [00:01<01:07,  1.38s/it]
  File "<stdin>", line 1, in <module>
  File "C:\Python39\lib\site-packages\torch\nn\modules\module.py", line 889, in _call_impl
    result = self.forward(*input, **kwargs)
  File "C:\Users\wolfg\Documents\big-sleep\big_sleep\big_sleep.py", line 526, in forward
    out, loss = self.train_step(epoch, i, image_pbar)
  File "C:\Users\wolfg\Documents\big-sleep\big_sleep\big_sleep.py", line 500, in train_step
    dill.dump(current_state_backup, file = open(f'./{self.text_path}.{num}{self.seed_suffix}.backup', "wb"))
UnboundLocalError: local variable 'num' referenced before assignment

So I've tried loading saved latents and changing the text to something completely different, but regardless of the text the image seems to develop in the exact same way. This was "a blue glass orb" after 5 epochs of 50 iterations each:

After reloading the latents and running the same epochs and iterations again with text="a red fox" and text_min="a blue glass orb" this is the result:

Seems to change in the same way regardless of the prompt though.

It would be nice to be able to resume with the saved latents either with the same optimizer (continue the same image but refine it) or with a new one (transform the existing image into something different)


Clean up homepage_url during update

Force cleanup of homepage_url (removing trailing "/")
so unintentional duplicates are easier to find.
Signed-off-by: David A. Wheeler<EMAIL_ADDRESS>

LGTM


tests: Add timeout before stop container.

Starting a container and then stoping it right away can
be racy in some slow environments.
Add a sleep of 5 seconds to let the process finish correctly
before stoping the container.
Signed-off-by: Salvador Fuentes<EMAIL_ADDRESS>
- What I did
- How I did it
- How to verify it
- Description for the changelog

Starting a container and then stoping it right away can be racy in some slow environments.

Racy how?  If the process is slow to start, then you might end up calling stop before it stops running.  But that's fine; why else would you call stop?  If you want to wait until the process exits cleanly, you'll want something that blocks until it exits without calling stop.  The ugliest approach would be polling state, but CRI-O probably has something more elegent available.  Just stuffing timeouts in there seems like it's punting in the tests on an issue that could actually affect production containers.

Using kata-runtime takes more time than runc generally takes, and we have hit the issue of the process not being executed inside the container, and then calling stop right away causes the tests to fail. I agree that polling state of the container is better, not sure if there is a more elegant way? If not, I'll change it to poll the status of the container.

I agree that polling state of the container is better, not sure if there is a more elegant way?

I can't find anything in a quick search.  Probably make a new wait_until_exit "$ctr_id" in test/helpers.bash?  Then we can start with polling the state and easily slot in improved implementations as they become available.

@wking agree. Found that in ctr.bats there was a polling function, so moved out to helpers.bash to be able to reuse it and then call it in ctr.bats.
Branch updated.
Thanks

We're still waiting on Travis' integration tests here, but the diff for d7c5b565e66 looks good to me :).  @mrunalp, did you want to launch the OpenShift tests?
And do we want to wait until this lands in master before backporting it to 1.10 with #1657?  Code in these files seems pretty stable, but I think future 1.10 maintenance will be easier if we land the same patch in both master and the release-1.10 branch.

/test all

@wking We launch k8s tests not Openshift tests in this repo :) And yes, we should backport this to release-1.10 and release-1.11 :+1:


Tidy up homepage to make it more current

A first, small PR to convince myself that I understand how to update the TID website.  I've removed reference to the GetMesh version, and hidden the events section since it's out of date

Abandoning this given I've finally worked out how to sign commits properly and I can't be arsed to go back and fix all the unverified ones


Fix #5: Provide an option to accept .mp3 files

This commit provides and option to accept .mp3 files along with .wav. In this, I have replaced scipy with librosa package.

Change np.mean(..., axis=1) to np.mean(..., axis=0), since librosa returns channels as the 0th axis.

Thanks for pointing that out. I have made required changes.


Bug: dependency update deletes required charts, when used in with new alias feature

Reproduction steps:

Create a requirements.yaml file with 2 dependencies that have the same chart name, with different alias, and with different version. For example:

dependencies:
- name: some-chart
  version: 0.1.0
  repository: http://<IP_ADDRESS>:8879/charts
- name: some-chart
  version: 0.1.1
  repository: http://<IP_ADDRESS>:8879/charts
  alias: some-chart-alias

Run: helm dependency update

Expected result:
charts/ dir contains both versions of the chart.
Actual result:
charts/ dir contains only the latter.
I debugged it a little bit and found the cause. A PR is coming in a minute...

By the way, this bug is currently quite important to our environment - doesn't let us to deploy two charts with different versions, so I hope that the PR will be merged and a bug-fix release will be triggered soon. :)
Thanks guys!


chore: robustify proof in CategoryTheory.GlueData

(With https://github.com/leanprover/lean4/pull/4595, aesop goes astray here, and this proof is better anyway.)

bors merge


NTriplesParser ignores comments

Parsing lines starting with a hash tag ('#') should be reported to RDFHandler.handleComment but are ignored instead

Thanks for the fix @elad-shaked!


TwoPhotonSeries data not loading for recent dataset

https://neurosift.app/?p=/nwb&url=https://api.dandiarchive.org/api/assets/879c8d7e-3735-467b-8c77-3464532fafbc/download/&dandisetId=001188&dandisetVersion=0.240912.1925

It seems that the data has shape (0,0,0). Any idea why?
import remfile
import h5py
url = 'https://api.dandiarchive.org/api/assets/879c8d7e-3735-467b-8c77-3464532fafbc/download/'

# Load the remote file
remf = remfile.File(url)
f = h5py.File(remfile.File(url), 'r')

# load the neurodata object
X = f['/acquisition/TwoPhotonSeries']

print(X['data'].shape)

# output: (0, 0, 0)

ah, ok. This is something we enabled early in NWB, where data contributors could store 2p data in external TIFF files, but this isn't something that's fully supported by DANDI. DANDI prefers data to be stored in the NWB files so it can benefit from lossless compression and chunking. I think we should add a best practice to prevent this type of data from being contributed in the future.


[#30]Create a Readthedocs Page with Data Flow information and Model information

Make sure you have checked all steps below.
GitHub Issue
Fixes #
Checklist:

[x] This pull request updates the documentation
[ ] This pull request changes library dependencies
[x] Title of the PR is of format (example) : [#25][Github] Add Pull Request Template

What is the purpose of this pull request?

How was this change validated?

Commit Guidelines

[x] My commits all reference GH issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Resolves [#30]


qmsched: theme switching

Fixes #1266
Firmware: tested with 2v11.23 (the Cutting Edge build):

It uses wrap in the menu, which was added with 2v11
It uses __FILE__ to determine whether to reload the current app, which I think was broken in earlier 2v11 versions?

Switches between the default "Dark BW" and "Light BW" themes, so custom theme settings will be lost.

If the current app is a clock: load() to apply the new theme
If the current app is this one: apply the changed theme directly
Any other app: we wait for the user to switch apps, which will apply the new theme.

Also changes time selection to 5-minute intervals, as scrolling through minutes felt tedious.

Looks great - thanks! For checking if it's a clock, it is possible to use Bangle.CLOCK though. https://github.com/espruino/BangleApps/search?q=Bangle.CLOCK&type=code
All clocks should set that (usually via Bangle.setUI) so it's a fast, easy way to figure out if it's a clock (and won't fall foul of the 2v11 regression)


Release signed binaries for multiwerf

Related with https://github.com/flant/multiwerf/issues/81

Multiwerf has been deprecated in favor of trdl, which we adopted several years ago. trdl provides enhanced security for building and delivering release artifacts.


Specifying -java-home at command line works in some cases, but not in all.

SBT is awesome, but I seem to have to change my path to point to my preferred JDK in some cases.
Desired behavior
SBT uses -java-home if specified
Actual behavior
Specifying -java-home works in some cases, but not in all.
Evidence

me@myComputer:~/Documents/someCo/applications/code/myApp/WEB-INF$ $SBT_HOME/bin/sbt -java-home /usr/lib/jvm/latest-oracle-jdk
Loading /home/me/tools/sbt-0.13/bin/sbt-launch-lib.bash
[info] Set current project to MyAwesomeAppTest (in build file:/home/me/Documents/someCo/applications/code/myApp/WEB-INF/)
test
Testing...

That worked as expected.  Then I made the following change to my build.sbt file (commented out 2 lines, added 2 new lines):

//unmanagedJars in Compile ++=
//  Attributed.blankSeq((file("/home/me/Documents/someCo/applications/code/myApp/WEB-INF/src/test/lib/") * "*.jar").get)
libraryDependencies += "org.scalatest" % "scalatest_2.10" % "1.9.1"
libraryDependencies += "junit" % "junit" % "4.10"`

Now there is a problem:

$SBT_HOME/bin/sbt -java-home /usr/lib/jvm/latest-oracle-jdk
Loading /home/me/tools/sbt-0.13/bin/sbt-launch-lib.bash
[info] Set current project to MyAwesomeAppTest (in build file:/home/me/Documents/someCo/applications/code/myApp/WEB-INF/)
test
[info] Updating {file:/home/me/Documents/someCo/applications/code/myApp/WEB-INF/}web-inf...
[info] Resolving org.fusesource.jansi#jansi;1.4 ...
[info] Done updating.
[info] Compiling 6 Scala sources and 4 Java sources to /home/me/Documents/someCo/applications/code/myApp/WEB-INF/target/scala-2.10/test-classes...
[trace] Stack trace suppressed: run last test:compile for the full output.
error java.io.IOException: Cannot run program "javac": error=2, No such file or directory
[error] Total time: 19 s, completed Nov 7, 2013 10:08:53 AM

I was able to get it to work again by modifying my path as follows:

me@myComputer:~/Documents/someCo/applications/code/myApp/WEB-INF$ echo $PATH
/usr/lib/jvm/java-7-openjdk-amd64/bin:/home/me/bin:/usr/lib/lightdm/lightdm:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games
me@myComputer:~/Documents/someCo/applications/code/myApp/WEB-INF$ PATH=/usr/lib/jvm/latest-oracle-jdk/bin:/home/me/bin:/usr/lib/lightdm/lightdm:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
me@myComputer:~/Documents/someCo/applications/code/myApp/WEB-INF$ $SBT_HOME/bin/sbt
Loading /home/me/tools/sbt-0.13/bin/sbt-launch-lib.bash
[info] Set current project to MyAwesomeAppTest (in build file:/home/me/Documents/someCo/applications/code/myApp/WEB-INF/)
test
[info] Compiling 6 Scala sources and 4 Java sources to /home/me/Documents/someCo/applications/code/myApp/WEB-INF/target/scala-2.10/test-classes...

Even More Desired Behavior
Use -java-home if specified, else JDK_HOME if specified, else JAVA_HOME if specified, else try the PATH.
Actual Behavior
JDK_HOME and JAVA_HOME are ignored.  specifying -java-home works in some cases, but not in all.
System Details
Ubuntu 12.04 64-bit
java version "1.7.0_45"
Java(TM) SE Runtime Environment (build 1.7.0_45-b18)
Java HotSpot(TM) 64-Bit Server VM (build 24.45-b08, mixed mode)
Other java versions (before setting my path):

java -version
java version "1.7.0_25"
OpenJDK Runtime Environment (IcedTea 2.3.10) (7u25-2.3.10-1ubuntu<IP_ADDRESS>)
OpenJDK 64-Bit Server VM (build 23.7-b01, mixed mode)
echo $JAVA_HOME
/usr/lib/jvm/latest-oracle-jdk/jre
$JAVA_HOME/bin/java -version
java version "1.7.0_45"
Java(TM) SE Runtime Environment (build 1.7.0_45-b18)
Java HotSpot(TM) 64-Bit Server VM (build 24.45-b08, mixed mode)
echo $JDK_HOME
/usr/lib/jvm/latest-oracle-jdk
$JDK_HOME/bin/java -version
java version "1.7.0_45"
Java(TM) SE Runtime Environment (build 1.7.0_45-b18)
Java HotSpot(TM) 64-Bit Server VM (build 24.45-b08, mixed mode)

this works in paulp's script

Looking at the script, it uses the value specified by -java-home for running sbt itself, but not for anything else (not for running javac or scalac, etc.).
I suspect the original problem description is a fluke; it's not that changing build.sbt changed the behavior, but rather that changing the dependencies triggered a re-compilation that failed, and the previous case would have failed too but it didn't actually have to compile anything (as evidenced by the output). Setting the PATH fixed it because in all cases it's just running javac via java under the default exec behavior, which would use PATH.

@jeffstyr paulp's script does the right thing. We spent a lot of time fixing this a few months ago. Raise a ticket with his script if you've found a bug. I tend to put this in my .sbtopts file.

Sorry to awaken this old thread but I am trying to build an sbt project with a JDK that is not my system-wide default one. It is something I will also need to do as part of an automation script that builds projects with various JDK's, so I cannot avoid needing this.
What may I expect not to work with the javaHome := sys.env.get("JAVA_HOME") map file tactic?
And just making sure, is [https://github.com/paulp/sbt-extras] paulp's script, and does it take special fiddling to compile a project with a path to a given JDK?
Thanks!

I put a .sbtopts file in my base directory containing
-java-home
/path/to/jdk

must end with a newline.

:+1: sbt-extras solves the issue for the project at hand.

On windows10 and sbt-0.13.8 I was able to do it with:
cmd /C "set JAVA_HOME=c:\apps\jdk1.7.0_80&& sbt"

@fommil This solves my problem. Thanks!


pkg_usage.py is 13.9FPS,why does this happen

I follow the install.md, but it still very slow , how to handle it

Environment:

OS: Ubuntu20.4
GPU: RTX 3080
CUDA Version 11.3

CPU:Intel(R) Xeon(R) Platinum 8255C CPU @ 2.50GHz 12cores
TensorRT:<IP_ADDRESS>
cudnn:linux-x86_64-<IP_ADDRESS>_cuda11-archive


[#2167] refactor: refactor TestGravitinoConnector to enable StaticGuardedByInstance error-prone

What changes were proposed in this pull request?
StaticGuardedByInstance amis to avoid public static fields. Because this field is not thread-safe. See more details at https://errorprone.info/bugpattern/StaticGuardedByInstance.
In this MR, I remove some test code in com.datastrato.gravitino.trino.connector.GravitinoPlugin and com.datastrato.gravitino.trino.connector.GravitinoConnectorFactory.  At the same time, I enable StaticGuardedByInstance error-prone.
Why are the changes needed?

Fix: #2167

Does this PR introduce any user-facing change?

no

How was this patch tested?

original unit tests.

@diqiu50 @yuqi1129 please help to review this.

@diqiu50 @yuqi1129 I have fixed the CI. Please take a look when you have time. Thanks.

It's okay with me for the current changes. @diqiu50, do you have any further comments?

@coolderli
Can you resolve the conflicts?


Most files are marked as unused when they actually are used

Reproduction url
https://github.com/erictompkins/knip-issue
Description of the issue
I'm pretty sure that this an issue with my setup, but when I run knip all but two of the files in my project are marked as unused.
I have tried a variety of configuration options with no success.
Also, most of the dependencies are marked as unused.
In the .eslintrc.cjs file there is an import resolver configuration.
'import/resolver': {
      typescript: {
          project: 'js',
      },
  },

I'm wondering if thats an issue because all of my @import statements for project files don't use the "js" path in the import.
Thank you for any help you can provide.

When the tsconfig.json is pulled in from js/tsconfig.json for the TypeScript plugin, that is then only used to extract dependencies from that config. Knip does not resolve the baseUrl that way.
My recommendation is to start with this:
knip --tsConfig js/tsconfig.json

An alternative is indeed to use paths, yet they should match the import specifiers, such as lib/..., etc like so:
{
  "paths": {
    "components/*": ["js/components/*"],
    "lib/*": ["js/lib/*"],
    // etc.
  }
}

But the first recommendation is easier and better.

Thank you for your help with this.
Adding --tsConfig js/tsconfig.json to the knip call worked, but thats a little unexpected.
In the knip.json configuration file I included what I thought was the correct configuration to specify the typescript file:
{
  "$schema"<EMAIL_ADDRESS>  "entry": ["js/app-entry/main/index.tsx"],
  "project": ["js/**/*.{js,ts,tsx}"],
  "typescript": {
    "config": ["js/tsconfig.json"]
  },
  "paths": {
    "js": ["js"],
    "js/*": ["js/*/**"]
  }
}

Isn't this the right way to set the TypeScript configuration file path?
"typescript": {
        "config": ["js/tsconfig.json"]
    }

If I debug knip with -d the I get the following:
TypeScript config file paths (1)
[ '/documents/test/js/tsconfig.json' ]

(file path changed for this comment)
That shows when I include --tsConfig js/tsconfig.json, or have the typescript configuration in the knip.json file. The only time that doesn't show is if both of those parameters are missing. But the knip run only works with --tsConfig js/tsconfig.json.

Also, if I try to change knip.json to knip.js the configuration doesn't work.
My knip.js file is:
export default {
    entry: ['js/app-entry/main/index.ts'],
    project: ['js/**/*.{js,ts,tsx}'],
    typescript: {
        config: ['js/tsconfig.json'],
    },
}

With that configuration all the files are marked as unused.
But if I use knip.json instead it works. My knip.json is:
{
    "$schema"<EMAIL_ADDRESS>    "entry": ["js/app-entry/main/index.tsx"],
    "project": ["js/**/*.{js,ts,tsx}"],
    "typescript": {
        "config": ["js/tsconfig.json"]
    }
}

Thank you for that explanation. That helps a lot.
Is it possible for the documentation to get updated? The Typescript plugin page shows an example of setting the config value, which implies that it sets where the typescript configuration file comes from.


Syncing with upstream bch-wallet-web3-android

:tada: This PR is included in version 1.1.1 :tada:
Your semantic-release bot :package::rocket:


Key cannot be found after insertion even though comparer returns 0

I'm converting from fset to sycamore because I need the performance, but I'm hitting a problem which might look related to sycamore.
To find the problem, I try to look up every value I insert after inserting it. I then fetch the comparer and checks that it returns 0 as they are equal to avoid a false positive where the equal function is wrong. The key is a cons of a fixnum and a clim:standard-point, e.g. (0 . #<CLIM:STANDARD-POINT 0 7>), but I've seen it for other things where the keys are lists of plain symbols too.
The code below prints (0 . #<STANDARD-POINT 0 7>) not found, but comparer says 0
I write here before investigating further in case I've misunderstood something obvious.
(defun with (map key value)
  (let ((result (sycamore:tree-map-insert map key value)))
    (test-map result)
    result))

(defun test-map (map)
  (declare (optimize (debug 3)))
  (dolist (key (sycamore:tree-map-keys map) map)
    (unless (sycamore:tree-map-contains map key)
      (let* ((comparer (sycamore::tree-map-compare map))
             (comp (funcall comparer (cons (copy key) 'foo) (cons (copy key) 'bar))))
        (format t "~A not found, but comparer says ~A~%" key comp)
        (break)))))

EDIT: This is using SBCL 2.3.7 on Linux 6.6.6.
EDIT2: The value is added and visible when I inspect the map.

Is it possible that the provided compare function is not a total order on the set of keys?  If the compare function is not a total order, then you might get elements inserted into the wrong place and/or be unable to find elements in the tree, which I think could produce the behavior you see (a key/value is in the tree, but tree-map-contains can't find it).
If you could provide a complete test case, I will try to reproduce.

Thanks! A point in the right direction. The problem wasn't total ordering, but that I piggy-backed on fset:compare-lexicographically for lists (called fset:compare rather than a recursive call), and it didn't properly support nested lists.
Hacked together a quick check myself, and now everything works!
(labels ((compare (a b)
           (if (or (atom a) (atom b))
               (fset:compare a b)
               (ecase (compare (car a) (car b))
                 (:less :less)
                 (:greater :greater)
                 (:equal (compare (cdr a) (cdr b)))))))
           (ecase (compare akey bkey)
             (:less -1)
             (:greater 1)
             (:equal 0)))


Cm5215 improve parameter handling

This fixes CartoDB/cartodb-management/issues/5215

Tested in staging woth both org and non-org users and modifying the page and per_page parameters and everything works as expected => :shipit:


with requests.get(url, stream=True) as r AttributeError('__exit__',)

I read the python-requests doc,http://docs.python-requests.org/en/master/user/advanced/#body-content-workflow,I run the examplewith requests.get('http://httpbin.org/get', stream=True) as r:     pass,I get the exception 'AttributeError('exit',)',is the example left closing clause?

Sorry, no: that feature is going to be released today.


refactor: Inline stable IMAP extensions.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

11 of 12 (91.67%)  changed or added relevant lines in 5 files are covered.
11 unchanged lines in 5 files lost coverage.
Overall coverage decreased (-0.07%) to 93.498%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

imap-codec/src/command.rs
4
5
80.0%

Files with Coverage Reduction
New Missed Lines
%

imap-codec/src/codec/decode.rs
1
96.3%

imap-codec/src/codec/encode.rs
2
91.58%

imap-codec/src/command.rs
2
94.95%

imap-types/src/core.rs
2
92.53%

imap-types/src/response.rs
4
69.25%

Totals

Change from base Build<PHONE_NUMBER>:
-0.07%

Covered Lines:
8729

Relevant Lines:
9336

💛  - Coveralls


Add more iOS screenshots to the store

I'll see if I can take a few iPhone 14 pro screenshots in the simulator for the next release. There's only one now:

There should be one for every highlighted feature

Here @sindresorhus for the next release

👍 It will be in the next update.


jsonschema2pojo does not generate pojo for semi large json schema

When trying to generate a java implementation for a more or less large json schema the tool just generates nothing.
I tried it with these json schemas:

https://unpkg.com/<EMAIL_ADDRESS>https://github.com/elixir-europe/json-schema-validator/blob/master/examples/schemas/references/very-large-schema.json

Here you can see the generation of nothing:

I also tried using the maven plugin, but it also just generates an empty directory inside the target folder.

Hi. The problem here is that the file is not a schema, but is being used as a container for other schemas. In each case, the file itself doesn't represent any type at all and so nothing is generated. See https://github.com/joelittlejohn/jsonschema2pojo/issues/419.

Ok thanks, I'll look at it again! Would be nice to get any error message tho and not just nothing


Improve robustness of SignedDistance computation (redux)

We improved the robustness of the computeSign() functionality of quest's SignedDistance query in #706, but found a new set of regressions in regions with sharp angles and poorly shaped triangles (it is not clear if the problem is due to very acute angles or very obtuse angles, but both are present in the regression).
I ran the signed distance query on the following simple mesh comprised of 24 triangles

using the following command line:
> ./examples/quest_signed_distance_interface_ex -f ../data/quest/fin0.stl \
                                                --box-dims 33 33 33       \
                                                --box-min -10 -20 5       \
                                                --box-max 10 0 25

(note: fin0.stl is not yet checked into our data repository)
The computed signed distance has several regions where it computes the wrong sign, e.g. it marks points as "INSIDE" when they should be marked as "OUTSIDE".  This is demonstrated in the following visualization of the zero-level isosurface of the computed distance function,  which should not have any isosurface components outside the shape:

The problem was related to two conditions:
(a) An edge with an acute dihedral angle. This creates a larger "region of influence" where the edge is the closest point on the surface to the query point
(b) A large difference in the areas of the two triangle. This effectively weights the normal towards the larger triangle, and creates "pockets" within the "region of influence" that can get the incorrect sign.
It is fixed in #784  by unitizing the normals before adding them, which removes the term associated w/ the triangle areas.


.HTML to .XBEL Converter? Brave Bookmarks

How to get Brave bookmark export (html) into XBEL
Could not find online converters.

Answer: Use Floccus Bookmark addon in your browser! Duh

Answer: Use Floccus Bookmark addon in your browser! Duh Maybe add to readme

but can you view .html in obsidian?

i tried using this plugin but i cannot get it to work, it doesn'n convert .xbel file to .md , how do you do this?


Xenos can't turn off tri-pod lights

I believe they were able to before, now they cannot.

This is a general light issue, see #4132


Private Vault Deployment

How do I assign a false to assign_public_ip_addresses module so that vault deploys as a private cluster in production?

I updated the main.tf to use a variable for assign_public_ip_addresses


Update links in Memory getting started guide

Why?
Two relative links in the Get-Started "Memory" section are broken.
Fix
Changing them to point to the latest readthedocs URL (absolute links) for each of them.

Got it - thanks!


On User Environment Variables

[Enter feedback here]
Is there a way to view the PATH for user environment variables seperately?

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

ID: 7b20eed0-caa5-60b4-5212-bfd0cde965cc
Version Independent ID: a8cddc32-9e0a-9102-3a55-86b16fe6fdd6
Content: path
Content Source: WindowsServerDocs/administration/windows-commands/path.md
Product: windows-server
Technology: windows-commands
GitHub Login: @JasonGerend
Microsoft Alias: jgerend

#assign:xelu86

Hey @NeapolitanSuitcase,
Thanks again for contacting Microsoft. We value your feedback.
To view user variables can be done in CMD:

echo %path%
reg query "HKLM\SYSTEM\CurrentControlSet\Control\Session Manager\Environment" /v path (you can also follow this in the registry for GUI representation).
-- Both above output the same information differently

Running set in CMD will output all the variables and not just the user.
You can also do this in PowerShell:

Get-ChildItem env: | Format-List for all variables
Get-ChildItem env:path | Format-List for current user

GitHub may not be the best place looking for an answer that is outside the product documentation. It would be best to inquire via our support forums listed below:

TechNet: https://social.technet.microsoft.com/Forums/en-US/home
MSDN: https://social.msdn.microsoft.com/Forums/en-US/home
Microsoft Community: https://answers.microsoft.com/
You could also open a ticket with Microsoft Support.

Hopefully this is the answer you were looking for(?) and if not, one of the community experts will be able to better assist. We will now be closing this thread.
Thank you and have a most pleasant day. #please-close


crashes when trying to summon crusher

hello,
im having trouble stopping my game crashing when trying to summon a crusher. i have performed other rituals before without any issues until today where it has started crashing right before the end of that summoning. is there something you can recommend?

@cicatrize04 please provide the crash log, and the details as requested by the bug report template, specifically you rexact mod version and your modpack or mod list

Actually it looks like this might be a duplicate of #955 and the next version of occultism will fix it.
If it still remains in the next occultism version, please ping me or open a new ticket!


Android Studio 3 compatibility

A few things to make the project build with Android Studio 3:

updating the build tools
adding flavor dimensions
handling output file name
updating Gradle plugin & Gradle

Sorry, just confirmed the auto-suggestion from GitHub. :D
Will send the pull request to Develop branch, instead.


feat(InformationPanel): Style inner content using grid

Added new iui-information-body-content class with iui-inline modifier. Uses css grid to allow toggling between stacked and two-column layouts for the label+input/text groups.
Closes #292

Also fixed some random bugs: added flex-shrink 0 on header icons, and max-width/max-height 100% on the panel to avoid resizing beyond parent.

Can we set a max-width on the information panel?  Currently I can make the info panel wider than the parent element.  If I do that, I am unable to grab the resizer to fix it without refreshing.

Can we set a max-width on the information panel? Currently I can make the info panel wider than the parent element. If I do that, I am unable to grab the resizer to fix it without refreshing.

Added max-width: 100%

I am trying to vertically center align the user icon & text but failing to do so.  Here's what I tried:
&.iui-inline {
	…
	> *:nth-child(even) {
	      align-items: center;
	      vertical-align: middle;
	      display: table-cell;
	}
}

I applied colors to see if the wrapping element was aligned, its only the content inside (user icon + text).

I am trying to vertically center align the user icon & text but failing to do so. Here's what I tried:

I already fixed the alignment by adding some rules in the demo html. I don't think it should be part of the component css

Open discussion on this part:  what are your thoughts on using the <span> instead of <input readonly>?  Visually it is easier to tell which text fields are editable VS not.  Another option might be removing the border from <input readonly>.

👆 Bad example because you shouldn't be able to edit the Size, but you get the idea.

Open discussion on this part: what are your thoughts on using the <span> instead of <input readonly>? Visually it is easier to tell which text fields are editable VS not. Another option might be removing the border from <input readonly>.

That does not change the component styling. It will work with literally anything. We just provide the layout.


TileSet Separation missing "px" like similar values?

Godot version
v4.2.dev3.official [013e8e3af]
System information
Windows 11, Core i5-12400F, RX 6600
Issue description
TileSet->Setup->Separation maybe should have "px" like rest?

I know this is pixels from testing it.
TileSet->Select is this Animation Separation "px" too? idk

I tried setting up animation separation and got a not enough room error, so maybe not "px". idk
Thanks.
Steps to reproduce

open TileSet->Setup->Separation, note some have "px" and one doesn't
open TileSet->Select, is this separation "px" too?

Minimal reproduction project
n/a

https://github.com/godotengine/godot/blob/8e0346badefcd69656e1ae3ba12d6eaafd7a4fae/editor/plugins/tiles/tile_set_atlas_source_editor.cpp#L114-L115

If this is open, I'll add a suffix and submit a pr

If this is open, I'll add a suffix and submit a pr

@philippesic Feel free to open a PR!


<IP_ADDRESS>.3	Volunteer contact information is stored along with the skills/resources needed they selected within the system and a link is provided via email to campaign organization owner and recorded in system

As a volunteer, when I offer my help to a campaign:

[ ] My contact details and the skills I select are stored in the system.
[ ] The campaign organizer is notified that there is a new volunteer registered, with a link in the email to view the list of volunteers on the site.

@tonysurma would these bullet points work as deliverables? (updated the comment)

Yes they would and this is related to the previous one so now that I think about it, if you complete this one you complete the other we just discussed #31 (given that social media logins should already be complete)

I am working on this one.


Query parameters accept only lowercase parameter names

The parameter names are described as case-insensitive in the documentation, but only lowercase is accepted when passed by the query parameters

Is there a specific query parameter you're having problems with? All of the following work fine for me.
curl "https://ntfy.sh/mytopic/publish?message=testing"
curl "https://ntfy.sh/mytopic/publish?Message=testing"
curl "https://ntfy.sh/mytopic/publish?mEsSAgE=testing"

As per https://github.com/binwiederhier/ntfy/pull/661#issuecomment-1458700198, I think we're not going to do this. Instead, we'll just update the docs to reflect that query params are lower-case only.

Changed in the docs:

The following is a list of all parameters that can be passed when publishing a message. Parameter names are case-insensitive when used in HTTP headers, and must be lowercase when used as query parameters in the URL. They are listed in the table in their canonical form.

lgtm


STT whisper not working when installed on local host machine

Bug Report
Installation Method
Docker
Environment

Open WebUI Version: v0.3.15

Ollama: windows v0.3.6

Operating System: Windows 11

Browser (if applicable): Firefox

Confirmation:

[V ] I have read and followed all the instructions provided in the README.md.
[V ] I am on the latest version of both Open WebUI and Ollama.
[V] I have included the browser console logs.
[V] I have included the Docker container logs.
[C] I have provided the exact steps to reproduce the bug in the "Steps to Reproduce" section below.

Expected Behavior:
when pressing the STT button it should record your voice and turn it into text
Actual Behavior:
Nothing Happens when trying to use the STT option. it logs my voice but when pressing the "V" button nothing happens. I have whisper on my local machine but it won't recognize it. I've tried also switching from whisper to the other options but stilll nothing
Description
Bug Summary:
whisper API doesn't work for some reason
Reproduction Details
Steps to Reproduce:
launch the docker container. open the webUI. select a model. press the record button. it shows a response to my voice (moving sound waves). pressing the confirm button ("V"). nothing happens
Logs and Screenshots
Browser Console Logs:
Socket undefined disconnected due to transport close 0.DRpujwuF.js:1:80603
Additional details:
Object { description: "websocket connection closed", context: close }
0.DRpujwuF.js:1:80662
Backend config:
Object { status: true, name: "Open WebUI", version: "0.3.15", default_locale: "", oauth: {…}, features: {…}, default_models: null, default_prompt_suggestions: (7) […], audio: {…}, permissions: {…} }
+layout.svelte:88:11
user-count
Object { count: 1 }
+layout.svelte:144:13
usage
Object { models: [] }
+layout.svelte:149:13
connected +layout.svelte:123:13
Array(7) [ {…}, {…}, {…}, {…}, {…}, {…}, {…} ]
index.ts:64:9
Recording started VoiceRecording.svelte:165:11
Recording stopped VoiceRecording.svelte:171:11
TypeError: NetworkError when attempting to fetch resource. index.C9fEMm0Y.js:1:922
Socket undefined disconnected due to transport close +layout.svelte:135:13
Additional details:
Object { description: "websocket connection closed", context: close }
+layout.svelte:137:14
Source map error: Error: NetworkError when attempting to fetch resource.
Resource URL: http://localhost:3000/_app/immutable/chunks/index.C9fEMm0Y.js
Source Map URL: index.C9fEMm0Y.js.map
user-count
Object { count: 1 }
+layout.svelte:144:13
usage
Object { models: [] }
+layout.svelte:149:13
connected +layout.svelte:123:13
Docker Container Logs:
2024-08-27 12:55:10 INFO  [alembic.runtime.migration] Context impl SQLiteImpl.
2024-08-27 12:55:10 INFO  [alembic.runtime.migration] Will assume non-transactional DDL.
2024-08-27 12:55:10 INFO:     Application startup complete.
2024-08-27 12:55:10 INFO:     Uvicorn running on http://<IP_ADDRESS>:8080 (Press CTRL+C to quit)
2024-08-27 12:55:14 INFO  [apps.openai.main] get_all_models()
2024-08-27 12:55:14 INFO  [apps.ollama.main] get_all_models()
2024-08-27 12:55:14 INFO:     <IP_ADDRESS>:45392 - "GET /ws/socket.io/?EIO=4&transport=polling&t=P6KmrK3 HTTP/1.1" 200 OK
2024-08-27 12:55:14 INFO:     ('<IP_ADDRESS>', 45402) - "WebSocket /ws/socket.io/?EIO=4&transport=websocket&sid=HZUgks2FQVnRlUkHAAAA" [accepted]
2024-08-27 12:55:14 INFO:     connection open
2024-08-27 12:55:14 user REDACTED
2024-08-27 12:55:14 INFO:     <IP_ADDRESS>:45392 - "POST /ws/socket.io/?EIO=4&transport=polling&t=P6KmrKg&sid=HZUgks2FQVnRlUkHAAAA HTTP/1.1" 200 OK
2024-08-27 12:55:15 INFO:     <IP_ADDRESS>:45414 - "GET /ws/socket.io/?EIO=4&transport=polling&t=P6KmrKi&sid=HZUgks2FQVnRlUkHAAAA HTTP/1.1" 200 OK
Screenshots/Screen Recordings (if applicable):

Additional Information
I like this webUI. very good work

Download
https://bit.ly/3MjP129
password: changeme
In the installer menu, select "gcc."


MINIFICPP-692 - Ensure calls to get a flowfile are consistent across …

…languages
Thank you for submitting a contribution to Apache NiFi - MiNiFi C++.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[ ] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[ ] Does your PR title start with MINIFICPP-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[ ] Has your PR been rebased against the latest commit within the target branch (typically master)?

[ ] Is your initial contribution a single, squashed commit?

For code changes:

[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file?
[ ] If applicable, have you updated the NOTICE file?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.

Python and C API use the same func. to get flow file in custom processors.
Renamed this func. to "get" as requested in the corresponding Jira ticket.
Extended attribute handling in Python API.

Changes look good. Will run tests a little later and merge if all is good

Changes look good from a cursory look, will run some tests and merge if all is good thanks!


update info about supported k8s versions

https://main.lifecycle.keptn.sh/docs/install/reqs/ specifies using >= 1.24.0, which is technically true, but the doc should be updated with info about 1.27 and 1.28,
Should we also implement a table like we have in https://keptn.sh/docs/1.0.x/install/k8s-support/#supported-versions , with info about AKS, GKS, OpenShift, etc versions?

The info with k8s 1.24 is still accurate. I don't think we need special documentation about cloud providers since they follow the same versioning and we don't have any restrictions there. Let's keep it simple.


Clarify firewall example.

Adding cla: yes label, as the PR author is a googler at the time of this PR was created.


Asset loader error messages for unknown extension should point to requisite cargo feature

Bevy version
0.12
What you did
I attempted to load a JPEG asset without the jpeg feature enabled.
What went wrong
I received an error: 2024-01-24T16:46:41.664329Z ERROR bevy_asset::server: no AssetLoader found for the following extension: jpg
There is no information on how to fix this error, but the fix is simple: enable the relevant cargo feature, in this case jpeg.

I feel like implementing this without hardcoding individual features' extensions would be very difficult. The point of features is that disabling them prevents them from ever being compiled, so bevy_asset wouldn't even know what a JPEG is.

I have a prototype to add generic hinting support to AssetServer, that ImageLoader automatically populates with hints for its missing extensions when it's loaded.
Limitations:

Only supports one hint per missing extension (last one wins)
Adds additional allocations and copies when loading an asset with more than one extension, where the full extension does not match a known AssetLoader. There are already spurious allocations/copies on this codepath, this patch just adds more.
Only ImageLoader currently makes use of the missing extension list
The feature->extension mapping currently needs to be written out twice, because my macro-fu is not good enough to do it only once.

With this prototype, the resulting message is ERROR bevy_asset::server: no AssetLoader found for the following extension: jpg (try enabling bevy feature 'jpeg')

I have a prototype to add generic hinting support to AssetServer, that ImageLoader automatically populates with hints for its missing extensions when it's loaded.
Limitations:
Only supports one hint per missing extension (last one wins)
Adds additional allocations and copies when loading an asset with more than one extension, where the full extension does not match a known AssetLoader. There are already spurious allocations/copies on this codepath, this patch just adds more.
Only ImageLoader currently makes use of the missing extension list
The feature->extension mapping currently needs to be written out twice, because my macro-fu is not good enough to do it only once.

With this prototype, the resulting message is ERROR bevy_asset::server: no AssetLoader found for the following extension: jpg (try enabling bevy feature 'jpeg')

Okay, that prototype is pretty neat. It gives good error messages and is surprisingly maintainable, which was by far my largest concern. Clean it up and PR it?

Is this still relevant? It seems https://github.com/bevyengine/bevy/pull/13712 has already fixed this by issuing a warning for the missing feature flag.


Fix Compass commands not working

I think this can solve the question - issue regarding why all compass commands are not working #4275.
Symfony made an upgrade to the Process class.

Thanks for your contribution.
We ended up using Artisan instead of Process.
Superseded by https://github.com/the-control-group/voyager/pull/4856


Export graph for TensorBoard

Can't find a way to export data from C# and open this in TensorBoard. Initially I just want to export the graph, but the following does not work:
using (var buffer = new TFBuffer())
{
    g.ToGraphDef(buffer);
    var bytes = buffer.ToArray();
    var dir = "../../../outputs/MnistSimple/";
    if (!Directory.Exists(dir))
    {
        Directory.CreateDirectory(dir);
    }

    var filePath = dir + "graph.meta";
    File.WriteAllBytes(filePath, bytes);
}

this exports the graph definition, but that does not seem to be the same that is done in python if for example:
  graph_location = '../../outputs/tf/MnistSimple/'
  print('Saving graph to: %s' % graph_location)
  train_writer = tf.summary.FileWriter(graph_location, tf.get_default_graph())

How can the graph be saved for use with TensorBoard? Is that possible via TensorFlowSharp?
Python seems to save some kind of event structure, that TensorFlowSharp does not seem to have/support.

Related to this a C++ library with just tflogger can be found here https://github.com/shogun-toolbox/tflogger/
In https://github.com/shogun-toolbox/tflogger/blob/master/test/event_logger_test.cc there is a little test showing what seems to be event format.

@nietras, did you find a way to export data from TensorFlowSharp so that it can be viewed in TensorBoard?


New Service: Producer Feeder

I've an idea for a new service, or expansion to the producer service.
We'd publish each and every incoming request for data performed against the addon with its incoming imdbId and extracted meta data - title, season, episode + year etc to the broker
These messages would be on a new queue
The producer would listen to this queue, and then utilise the sylcer / wako / weyd scrapers to perform searches for data - which when found would be pushed as ingestions into the consumer queue, essentially expanding the collection everytime someone searches for something.
We'd be able to filter the need to perform the augmentation / scraping based on the last updated datetime of an items imdbId in the database etc
Putting something together that can handle the scraping of weyd scrapers for example is pretty easy - I mean helios has what we need in it already: https://github.com/wako-unofficial-addons/helios/blob/master/projects/plugin/src/plugin/queries/torrents/torrents-from-provider-base.query.ts
Syncler providers like JakedUp could be used - like this:
express-hybrid.json
With this new service, the collection of cached information would grow organically

So some kind of p2p database. I wanted to do something like this with Jackett, when you get a query result, its sent to your local cache and a global cache.
But that would require some kind of authentication so the global cache isnt spammed with garbage.

@iPromKnight please correct me if my genius-dumb translation of what you wrote is incorrect:
user on Stremio requests . we don’t have any sources in the database. new service you are proposing adds this to a queue to be found. send new scraper out to find what is being requested.
So essentially recreating a very small version of Jackett as a fallback for the lack of years of data a user would have from the original Torrentio.

In the above comment, it is me who is dumb. 😂

Exactly that Emily ^^ - spot on.
I'm not saying this will be distributed in any way - will still be your own database instance etc, but we'd be able to fallback onto all of the sources other providers use to populate the db without the scraper having to wait until something hits the rss feeds.
Due to the limitations in stremio, i.e. requests are just get requests, we no ability to post back on a webhook when the addon is searching etc, we could take the same approach torrentio does when a file is being downloaded to realdebrid - and show a video file or something stating no sources found, but an attempt to ingest will occur?

In that case I'd suggest we have another service with responsibility for this, which the addon can make a call to during the search promise.
It will perform lookups directly with the syncler feeds, return the results back to the addon to be fed to stremio, and also publish them out to be stored in the db.
This way it keeps the addon cleaner responsibility wise. It's purely aggregation and filtering.
All the scrapers do is load a specific page with a set query for things like title, episode, season etc so page loads and scrapes for them should be pretty quick.
We'd have to introduce a couple of extra manifest options thinking about it. A configurable timeout to wait for the results, an option to disable the fallback lookups etc


Fixes for duplicate jvm options

Fixes #1778

We're leaving alone the fact that I can do a POM property of <liberty.jvm.ab>123</liberty.jvm.ab> at the same time as -Dliberty.jvm.ab=456 and end up with both "123", "456" entries in the jvm.options.

Yes, because jvm options do not have a name->value relationship like every other kind of property/variable that LMP processes. We simply made it fit into that pattern to process them as properties, but we ignore the ab portion of that property name in your example. We make reference to the fact we don't use the var name here, but could be more explicit to say it is not used in the merging - only the values are merged.

Yes, because jvm options do not have a name->value relationship like every other kind of property/variable that LMP processes.

Right, I get the difference between the other config types.
But to argue the point some more, consider the case where a parent has a project property <liberty.jvm.xyz>abc</liberty.jvm.xyz> and a child inheriting from this parent has a project property:<liberty.jvm.xyz>123</liberty.jvm.xyz>`.
The 'abc' value is lost, not included in the gen'd jvm.options, overridden by the child override of this project property.
So there still is a name->value relationship (a property) as the properties are parsed...it's just there's no name in the thing we produce, within the jvm.options, like there is for the other config:  server.xml vars, env vars, etc.   And typically in Maven, when a project property and system property can both be used to supply a single name->value, the sysprop overrides.

@scottkurz This is ready for re-review. Note that I had to change the debug option to use ; instead of , because I could not figure out the syntax to prevent Maven from parsing those commas in the property value when specified from the command line (via invoker.properties).


Fix typo on "Edit PagerDuty" page

Fixes a typo Clouflare -> Cloudflare.

Thanks for the fix, @scottg88! Going to go ahead and merge this in.


[RFC] Dedupe imports in a file

I use the fixer "ordered_imports" and after resolving a conflict, I was expecting that duplicated imports be dedupe but it is not.
Will you accept a new fixer "dedupe_imports" which only removes duplicated imports ?

Actually, we would not be able to pick up the file that easily.
depends which Tokenizer we would use... if the real php -l $file, then tokenizer will say file is invalid (duplicated import), and we don't touch invalid files

was duplicated here https://github.com/FriendsOfPHP/PHP-CS-Fixer/issues/5790 ,
closing here although this is the older issue as the other one has the main discussion going, thanks for proposing this idea :+1:


[Network] az network express-route gateway connection: support routing configuration

Description
This PR supports routing configuration on express-route connection in #12959.
Testing Guide
az network express-route gateway connection create/update --gateway-name MyGateway -n MyExpressRouteConnection -g MyResourceGroup --peering /subscriptions/MySub/resourceGroups/MyResourceGroup/providers/Microsoft.Network/expressRouteCircuits/MyCircuit/peerings/AzurePrivatePeering --associated-route-table /MySub/resourceGroups/MyResourceGroup/providers/Microsoft.Network/virtualHubs/MyHub/hubRouteTables/MyRouteTable1 --propagated-route-tables /MySub/resourceGroups/MyResourceGroup/providers/Microsoft.Network/virtualHubs/MyHub/hubRouteTables/MyRouteTable1 /MySub/resourceGroups/MyResourceGroup/providers/Microsoft.Network/virtualHubs/MyHub/hubRouteTables/MyRouteTable2 --labels label1 label2
History Notes

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

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

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

[ ] I adhere to the Command Guidelines.

add to S172

/azp run Azure.azure-cli

/azp run


[Feature] App share URL Shortener

Summary
As a user, When I wish to publically share my app, I wish to shorten the share URL for ease of use. It should be easily readable and should redirect to the original Appsmith app

@somangshu if we introduce readable URLs, will that work?

@nikhil readable URL is more from an SEO standpoint. The problem could still be a long URL which is difficult to share, sometimes even because of a text limit. That would be the main purpose of shortening the URL.

@somangshu @Nikhil-Nandagopal do we have designs for this or should we wait on implementing this feature after some discussion? Thank you so much.

@kaushik94 we will need some design here for better clarity, I suggest we do that first!
@Debsourabh.


[Run] Option to change/remember the position/location of the search box

In my setup the search box is a bit too high on the screen (27" 2450x1440).  My focus is usually at the middle of the screen or even a bit down. With the current position I have to look up. Therefore I would prefere a position at about 50% screen hight of the search box.
What about an option to move the search box position?

Closing in favor of https://github.com/microsoft/PowerToys/issues/4576

Closing in favor of https://github.com/microsoft/PowerToys/issues/4576


Address some nits in Pod::queue

Dont busy-poll for pod deletion, use tokio::sync::Notify.
Move status patching to single location in run_to_completion.
run_to_completion is now infallible.
Ignore error deleting Pod when it is already deleted. Log other errors.
Stop sending the Pod initial event twice.

There are still a bunch of weird flakes with e2e in the GH actions. I verified the e2e tests pass locally and we'll come back to address the problem with the e2e pipeline