Periodic releases

Hi @suzuki-shunsuke. Thanks for building this helpful tool.
I noticed that renovate and renovate-approve bots are enabled here. I saw there are recent merges by these bots, however the last release is Nov 2023.
Could you enable periodic releases using goreleaser and github actions ? Perhaps it can release a new minor version once a month if there are renovatebot merges ?

I checked the update between the latest version v0.2.0 and the main branch.
https://github.com/suzuki-shunsuke/tfprovidercheck/compare/v0.2.0...main

Update Go 1.21.3 to 1.23.0
Update Go directive 1.21.3 to 1.22.2
Update some Go libraries in go.mod
Update Cosign v1.13.1 to v2.4.0
Update Goreleaser v1.21.2 to v2.2.0

Do you need any update?
The idea of periodic release is interesting, but I'm not sure if it is good and necessary.

Do you know any project doing periodic releases?

I know that Renovate creates releases when the main branch is updated (example), though this isn't the periodic release

Thanks for responding quickly. I look at recent releases to ensure if a repository is maintained to integrate it into the CI process. If there hasn't been a release in almost a year then that's usually one less reason to use a tool. While I don't technically need a release to use the tool, the client I work for does require a recent release.
Periodic releases aren't necessary. I do know that goreleaser has this feature under their "nightly builds" docs.
https://goreleaser.com/blog/nightly-actions/#implementing-nightly-builds
Of course, no need to build nightly. Judging by the cadence of renovate PRs merged, it could be released monthly if you're thinking about automating it.
For now, a single manual new release, even a patch, would go a long way.
Thank you for considering

Thank you for your explanation.

If there hasn't been a release in almost a year then that's usually one less reason to use a tool. While I don't technically need a release to use the tool, the client I work for does require a recent release.

I see. That makes sense.

For now, a single manual new release, even a patch, would go a long way.

I'll create a release soon.

Released v1.0.0.
https://github.com/suzuki-shunsuke/tfprovidercheck/releases/tag/v1.0.0
I close this issue for now.

Thank you very much!


testing: setup go tests for future development

including Artur's suggestion for putting https://app.clickup.com/ into a global constant, swapping with mock API when needed.

#9
Commit: 756ae55f8ffa61f4e6a34089d49ce870082dc9d1


Replace Call Stack Graph Implementation With Its Enhanced Version

🚀 Motivation and context
Over the past several months, we've developed an enhanced version of the Trace Call Stack Graph implementation. This version includes numerous enhancements and new features, as detailed in the description section.
In response to requests from several developers and users, we're porting the enhanced version back to the GitHub repo.
Description
The enhanced version of Call Stack implementation has introduced multiple improvements over the current GitHub version. We highlight several major enhancements as follows:
1.     Handling zero-duration events: The open-source implementation doesn’t handle zero-duration events correctly and may construct an incorrect graph. The enhanced version fixed this issue.
2.     Integrating call stack graph with the Trace DataFrame: This allows call stack information to be integrated with Trace filtering and advanced statistics.
3.     Connecting backward and forward threads: While we construct the call stacks on a per-CPU threads basis, ML engineers have requested to combine both the forward and backward threads into the same graph for practical trace analysis.
4.     Simplifying the API for Call graph search: We have added several APIs to the CallGraph class to simplify the search of Call Stack for a given event node.
5.     Call Stack graph for partial trace: Constructing the entire call graph takes a considerable amount of time. The enhanced version supports constructing a Call Graph object for a DataFrame slice.
6.     Optimizing call stack graph construction: We made numerous performance improvements on the call stack construction.

Alternatives
No response
Additional context
No response

This is awesome. Looking forward to it @fengxizhou!

Closed.

Closed.


The "SoftLayer_Billing_Order_Quote::verifyOrder" method fails when try to order a VSI with ruby client

When trying to order a virtual server instance using the ruby client the "SoftLayer_Billing_Order_Quote::verifyOrder" method fails and raises the following exception:
"An invalid price was submitted with this order"
The exception is raised despite valid parameters are set and it is possible order using the same quote from UI and also using the php client.
To reproduce the problem use below code
require 'rubygems'
require 'softlayer_api'
require 'pp'
SL_API_USERNAME = "set-me"
SL_API_KEY = "set-me"
client = SoftLayer::Client.new(
:username => "#{SL_API_USERNAME}",
:api_key => "#{SL_API_KEY}")
quoteId = 1242802
order_templates = client['SoftLayer_Billing_Order_Quote'].object_with_id(quoteId).getRecalculatedOrderContainer()
order = order_templates['orderContainers'][0]
order['complexType'] = 'SoftLayer_Container_Product_Order_Virtual_Server'
order['quantity'] = 1
order['location'] = '154820'
order['hardware'] = [{'hostname'=> 'server01', 'domain'=> 'example.com'}]
begin
receipt = client['SoftLayer_Billing_Order_Quote'].object_with_id(quoteId).verifyOrder(order)
pprint.pprint(receipt)
rescue => error_reason
puts "The order could not be verified  #{error_reason}"
end

OK.  Let me get with the ordering guys and see if we can figure out why this might fail.

Are you placing the order through Ruby using the same User account that created the quote?

Yes, I used same account to create the quote and try to perform the order... please note that this problem happens when try to order virtual servers.. It works when bare metal server are ordered from quotes

This may seem silly... but where you have:
order['hardware'] = [{'hostname'=> 'server01', 'domain'=> 'example.com'}]
Have you tried using virtualGuests in the place of hardware?
From the documentation it doesn't seem that it should matter... but I'm curious to know if it does.

Yep, I tried with "virtualGuests" instead "hardware"... got same results.

I have a strong suspicion that this may be related to Issue #49
It appears that when you get back the order template from the system, the hash contains a lot of key/value pairs which have the empty string ("") as the value.  I suspect that in other contexts (like the PHP example you mentioned) those nil values are handled better.
A hardware order might not have a key that is rejected when it comes back with "" instead of nil.
I had some success with running through the order template and simply removing any key value pairs where the value was "".  Unfortunately, since that is a nested structure, a more thorough "sanitizing" mechanism might be necessary in the general case.
Alternatively you might be able to get away with creating an all new order template and only filling out the fields that "make sense", copying over values from the order obtained from the quote as needed.
None of these is a satisfying solution, however.  Internally will have to pursue the resolution to Issue #49
(on an unrelated note  your SoftLayer_Container_Product_Order_Virtual_Server constant should probably be SoftLayer_Container_Product_Order_Virtual_Guest)

After updating the 'SoftLayer_Container_Product_Order_Virtual_Guest' value that you mentioned.. I did not get the previous message I reported. However the order cannot be verified and the "Invalid data on the order for property: presetId. Invalid preset id: 0" exception is raised, and probably this problem is related to the issue you commented.
Anyway I think in the case to order a Virtual guest the "preset id" parameter should not be mandatory, please let me know your comments

This also fails from Python and PHP. It's a problem in the SoftLayer_Billing_Order_Quote  api.
You have to do it like this:
require 'softlayer_api'

# Credentials to the SoftLayer API are grabbed from the config file by default.
# See https://github.com/softlayer/softlayer-ruby/blob/master/lib/softlayer/Config.rb#L11-L44
client = SoftLayer::Client.new

QUOTE_ID = 1234

quote = client['Billing_Order_Quote'].object_with_id(QUOTE_ID)
order = quote.getRecalculatedOrderContainer['orderContainers'][0]

order['quantity'] = 1
order['virtualGuests'] = [{ 'hostname' => 'quotetest', 'domain' => 'example.com' }]
order.delete('hardware')

pp client['Product_Order'].placeOrder(order) # or verifyOrder()

The problem with not be


Just don't mess with resolve.root at all

Don't seem to need anything but loaders from cli's node_modules?

Between this, #458, and #459, um, thanks for picking up after me! 😟😟😟

Together we take on webpack and we shall prevail! 🎏

(Please let me know if I'm doing anything dumb here... 😁 )


Compose miniblog popup widget

When user clicks on the compose button, it should open a pop-up compose window that lets user create a mini-blog. Add a widget for the compose window.
Check the propose UI in the readme for referenece.

Hey @vedantbahel
Shall I take this up?

Sure!


EspMdns::add_service fails if the hostname is not set

When trying to create a new mDNS service using following code, add_service method fails with EspError(0x102) - ESP_ERR_INVALID_ARG
let mut mdns = EspMdns::take().unwrap();
mdns.add_service(Some("test"), "_http", "_tcp", 80, &[("hello", "world")]).unwrap();

However, if you set the hostname before adding a service, the code proceeds without a panic, yielding the wanted result:
let mut mdns = EspMdns::take().unwrap();
mdns.set_hostname("hostname").unwrap();
mdns.add_service(Some("test"), "_http", "_tcp", 80, &[("hello", "world")]).unwrap();

Could you add a check for that or at least put that into a documentation?
Thanks

@ivmarkov that probably still an issue?

@ivmarkov that probably still an issue?

Yes. Though I'm not really sure what we are supposed to do here? It is just how it works - and - it fails at runtime anyway, and does not panic.
The poster is wrong that it panics - it does so only only because he unwraps.
I would close this.

When trying to create a new mDNS service using following code, add_service method fails with EspError(0x102) - ESP_ERR_INVALID_ARG
let mut mdns = EspMdns::take().unwrap();
mdns.add_service(Some("test"), "_http", "_tcp", 80, &[("hello", "world")]).unwrap();

However, if you set the hostname before adding a service, the code proceeds without a panic, yielding the wanted result:
let mut mdns = EspMdns::take().unwrap();
mdns.set_hostname("hostname").unwrap();
mdns.add_service(Some("test"), "_http", "_tcp", 80, &[("hello", "world")]).unwrap();

Could you add a check for that or at least put that into a documentation? Thanks

There is already a check. Why should we add a check on top of the check coming from ESP IDF and resulting in an EspError anyway?

If you feel like, you can add a short docu string that the host name should be set first, though that would be a dip in the sea of undocumented APIs we are currently having
:)


Can share any Nodes. thanks

Can share any Nodes. thanks

We welcome any kind of contribution since blockchain belongs to the community.


add contextPath before api url

if confluence instance have a contextPath,the client rest api url should start with context path

see also: https://confluence.atlassian.com/doc/configuring-the-server-base-url-148592.html


Please add new term: FastQC

FastQC
https://github.com/s-andrews/FastQC
FastQC aims to provide a simple way to do some quality control checks on raw sequence data coming from high throughput sequencing pipelines. It provides a modular set of analyses which you can use to give a quick impression of whether your data has any problems of which you should be aware before doing any further analysis.
GNU GENERAL PUBLIC LICENSE Version 2

Assigned SWO:1100144


Is anything suppose to appear when you open index.html after downloading?

The control instructions appear in the bottom right but there is no scene, just black screen. Just checking is this expected behavoir or if there should be a simple test scene to start?
Demos look great btw :)

Yes! serve index.html with a localserver and you should see whatever is coded in createScene.js
Let me know if you see any error


Use error log in PersistentApplicationEventMulticaster if listener not found

In my opinion, Modulith should not fail silently but should produce an error log.
resolves #549

That's merged, thanks!


Points additive blend fix

Following https://github.com/tangrams/tangram/pull/583
This PR fixes additive blending for points in various ways:

Antialias points with the same code that antialias overlay blended points
Fixes outline-fill region
Removes discard for additive blended points

These are screenshots of Mapnik, post PR and pre PR:

Basically it uses a #if to switch the blending and avoid the usage of discard.
I don't know much about that discard, but if it tries to be a "lower-quality substitute" for performance reasons, keep in mind that using discard degrades performance considerably, in a similar way to traditional blending, and for similar reasons (but with alpha blending anti-aliasing can be implemented).

Regarding the discard, my recollection is that it was there less for performance reasons, and more for the case where you want to use Tangram's order pipeline to interleave the points with the rest of the 3D space (different from overlay blend more behavior). That said, it's very old code and it is a good time to re-evaluate that behavior (and possible divergence from Tangram ES), so we will do that.


removes use of  inside of nix output

Because the  import was in the middle of the source it was not
possible to pin the version by passing it as an argument.

Looks good. Thanks!


Added clarifications to README.md

Describe the changes you have made in this PR: Added clarifications as described in issue #720

Coverage decreased (-0.06%) to 82.507% when pulling a7cd092566fb546966427ca7e51f5fbf89e9d9b5 on nrs1729:new-readme into 2c01dad8e0b59646b9625313a0da5801dde1e221 on CircuitVerse:master.


[SPARK-39693][INFRA] Do Not Execute tpcds-1g-gen for Benchmarks Other Than TPCDSQueryBenchmark

What changes were proposed in this pull request?
Currently tpcds-1g-gen runs for any benchmarks even that do not require TPC-DS data on Github Actions.
This PR proposes to skip running tpcds-1g-gen if the benchmark class does not contain TPCDSQueryBenchmark or * based on the discussion on #37020
Why are the changes needed?
This PR should save time to launch benchmarks on Github Actions
Does this PR introduce any user-facing change?
No
How was this patch tested?
Tested on Github Actions.

late LGTM +1

Thank you all


Extra Tree Memory Corruprion

i cloned and installed  lgb from the source last week and i am using  python wrapper.
i tried  extra trees  in our project, it crashed for a couple of runs.
the error messages :
free(): invalid next size (normal)
another error  :  double free or corruption (!prev)
i think  they are  C level errors.

@davutpolat could you try the latest master branch? we have several fixes recently.

@guolinke  it seems  solved, thanks

@davutpolat great, glad to hear it!


Tool bar icon not displaying

I cannot get the toolbar icon to display correctly on startup? every time I load unreal it just comes in as a blank button?
Installed using pip all icons within the editor are working correctly

I have the same now, think unreal removed that icon or changed icon logic. I ll look into it


Mismatched ratbagd / ratbagctl versions cause errors

When testing upgades to ratbagd 0.4, I noticed that running ratbagctl 0.4 with an already-running ratbagd 0.2 produces
event6:    None

and show-device produces
Traceback (most recent call last):
  File "/usr/bin/ratbagctl", line 202, in <module>
    main(sys.argv[1:])
  File "/usr/bin/ratbagctl", line 199, in main
    cmd.func(r, cmd)
  File "/usr/bin/ratbagctl", line 92, in show_device
    capabilities = [caps[c] for c in d.capabilities]
TypeError: 'NoneType' object is not iterable

It would be nice if ratbagctl queried ratbagd’s version and complained appropriately.

I'm not sure if this even applies anymore, just moved it from libratbag/ratbagd.

Yeah, this is really old. We don't support it anymore and we have introduced an API to avoid this.


Realtime updates

Hi @jmelosegui nice work :+1:
I have been playing with this sample here can you please share me some guidance on how to show realtime updates while truck moving. I have tried with signalR 2
Also, how to check bounding box, car/truck is out of box area.
thanks
Papi

SignalR should work but that is an interaction between the server and the javascript on the client and has less to do with this MVC control.
Regarding the bounding box check out the FitToMarkersBounds sample.
Thanks for your feedback.


Shorthand for the officially supported buildpack

Replacing the Github link with the Heroku shorthand defined for the officially supported buildpack.

Description

Replacing the Github link with the Heroku shorthand defined for the officially supported buildpack.
Checklist:

[x] I've made sure that tests/test_cookiecutter_generation.py is updated accordingly (especially if adding or updating a template option)
[x] I've updated the documentation or confirm that my change doesn't require any updates

Rationale

Opting to use the officially supported short hand as defined in the official documentation.

@raybesiga Good point. Merged 🚢. Thank you.

You are welcome @luzfcb. 👍🏾


Take a look at ng-zorro

An enterprise-class UI components based on Ant Design and Angular.
Here is the link: https://github.com/NG-ZORRO/ng-zorro-antd

Thanks for opening the issue, happy to add it. Closing.


Search field does not work on touch based iOS devices

The demo located at http://ugrose.com/content/demos/kingtable/complex-filters.html works fine on desktop browsers, but when I view it with Safari on the iPad, the search field does not seem to work properly.
The first character is displayed, but after that it does not accept any new character.
Other functionality, such as sorting, does not seem to be affected.

Hi Maurice, thanks for the heads-up, I will dedicate time to this as soon as possible.

@MauriceS, quite surprisingly, it seems the bug was caused by a CSS rule. I tested using an Apple iPad Mini 16Gb (MD531FD/A), reproducing the same error you described. After removing this CSS rule user-select: none, now the page is working properly on my iPad. Since currently I have only this single device to test, I don't close this issue yet.

The fix has been tested successfully with different mobile devices running iOS.


[GLUTEN-2822] [VL] Add native memory manager to avoid recreating new Velox memory pool instances for one Spark memory consumer

Introduce "MemoryManager" concept to hold all memory resources in native, and bind with Spark memory consumer to count and report native memory usage.
And implement VeloxMemoryManager to create memory pool when first initialization, then same child instance from one Spark task could reuse memory pool, e.g. all shuffle reader stream will use same memory pool to reduce peek memory usage.
And hide NativeMemoryAllocator under NativeMemorManager.
Fixes: #2822

@zhztheplayer

Ignore old file/class/comments for now.

Most parts of CI has passed.

I've edited the title to make it more accurate to describe the changes here.

I've edited the title to make it more accurate to describe the changes here.

Thanks! I update title and description further.

I've edited the title to make it more accurate to describe the changes here.

Thanks! I update title and description further.

Umm. I suggest to use title:
[VL] WIP: Reuse a single Velox memory pool for one Spark operator (or something like that).
And put information about the new interface MemoryManager in PR description. So that we know how much the Velox mem pool scope is reduced by this patch.

I've edited the title to make it more accurate to describe the changes here.

Thanks! I update title and description further.

Umm. I suggest to use title:
[VL] WIP: Reuse a single Velox memory pool in one Spark operator (or something like that).
And put information about the new interface MemoryManager in PR description. So that we know how much the Velox mem pool number is reduced by this patch.

I think reuse memory pool is one benefit of this PR, but we could get more clearly memory management interface and debug convenience by this.

I've edited the title to make it more accurate to describe the changes here.

Thanks! I update title and description further.

Umm. I suggest to use title:
[VL] WIP: Reuse a single Velox memory pool in one Spark operator (or something like that).
And put information about the new interface MemoryManager in PR description. So that we know how much the Velox mem pool number is reduced by this patch.

I think reuse memory pool is one benefit of this PR, but we could get more clearly memory management interface and debug convenience by this.

Can you use
[VL] Add native memory manager to avoid recreating new Velox memory pool instances for one Spark operator

?
As long as we describe the actual functionality change in the title.

Velox backend / velox-ubuntu2004-test-spark32 (pull_request) Failing after 84m

Due to cpp micro benchmarks.
Velox backend / velox-ubuntu2004-test-spark33 (pull_request) Failing after 85m

Due to issue #2564

I will do performance benchmark in our environment, just wait..

@Yohahaha Thanks. And would you like to share the reduced Velox memory pool number during query execution after your test is done? Thanks.

No performance regression, and 10% performance boost since #2802 merged in our test.

@zhztheplayer #2802 has reduce most parts of velox memory pool of shuffle reader stream, thus I dont count decreased number. After this patch we can ensure only one memory manager(velox memory pool) will be used by all instances which created by same name.

After this patch we can ensure only one memory manager(velox memory pool) will be used by all instances which created by same name.

Agreed. Merging. Thanks for your work!

query
log/native_master_08_24_2023_time.csv
log/tpch_vanilla_spark322_execution-time_sf2t_date_decimal.csv
difference
percentage

q1
42.97
571.86
528.895
1330.93%

q2
24.29
62.65
38.364
257.94%

q3
36.78
113.36
76.578
308.22%

q4
41.66
81.33
39.660
195.19%

q5
69.82
216.88
147.062
310.63%

q6
6.32
18.90
12.575
298.89%

q7
85.04
209.16
124.128
245.97%

q8
79.03
211.69
132.664
267.87%

q9
115.95
297.94
181.993
256.96%

q10
45.98
90.61
44.631
197.06%

q11
18.87
49.99
31.130
265.01%

q12
20.70
62.00
41.294
299.44%

q13
47.87
94.96
47.087
198.36%

q14
15.90
32.49
16.590
204.37%

q15
26.33
76.73
50.402
291.41%

q16
15.05
28.32
13.269
188.16%

q17
118.79
442.32
323.531
372.36%

q18
159.59
639.34
479.752
400.62%

q19
10.28
37.59
27.308
365.58%

q20
23.09
60.43
37.339
261.68%

q21
229.40
350.61
121.215
152.84%

q22
15.56
33.61
18.048
215.97%

total
1249.27
3782.79
2533.515
302.80%


Witness Demo Instructions not working

See error image attached below. Following exact instructions for Rebase demo app, wrangler expects some script file.

Based on the information I have, I suspect that wrangler dev was not run in the rebase/demo/witness directory, or that a wrangler.toml was not created at that directory with the needed contents, but without more info, I can't speculate further. The wrangler dev documentation mentions that no arguments are needed if the wrangler.toml is correctly configured.
The current Rebase documentation is about to receive a major overhaul so hopefully the exact steps needed to run the demo will be clearer after the open PR (#40) merges. Because Rebase is not published to crates.io (nor is it's major dependency: ssi), it's not the easiest library to work with currently. We should be reaching that point soon, so stay tuned for more details there!
In the meantime, feel free to make use of our deployed witness worker at https://rebasedemo.spruceid.workers.dev you should be able to hit /instructions, /statement, and /witness as expected, also take a look at https://rebase.pages.dev which has a graphical front-end pointed to the worker linked above.
I'm going to close this for now, but if you still have problems after #40 closes, please feel free to re-open. Thanks for the interest!

I can confirm that I am running this from inside the witness directory, copied the example wrangler.toml file from the example (which is parsed just fine by wrangler, it throws out some warnings about unnecessary variables set), following the directions as written. It still expects some entrypoint file, which I do not see in the project. I will keep an eye on #40 merge


⚠ Bofrost.lu has degraded performance

In efa351c, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1415 ms

Resolved: Bofrost.lu performance has improved in 3d105c3 after 10 minutes.


java.sql.SQLException: No suitable driver

Hello,
I am unable to execute my code as Spark fails due to lack of a suitable driver. Is there any particular configuration I need to set in my Config()? I'm using SQL Server.
Am I supposed to set the driver class manually? And is there any particular format for the URL?
Thanks

Yes, you have to add "driver" -> "com.microsoft.sqlserver.jdbc.SQLServerDriver" to the Config.
I think the library should do this automatically. Since it makes no mention of having to set it manually and it's already partly hardcoded.

@saikiranbandaru Did this solve it for you?

Thank you @Jasper-M for your suggestion; I hoped it helped the OP. Closing this due to lack of response.


No persistence for created personas

On use of app, and entering bad persona (text)
Your personas:
0 paul
Would you like to use an existing persona? [y/n]: y
Select a persona to use: l
Traceback (most recent call last):
File "/Users/paul/code/python/./black_lager_app.py", line 101, in
wallet = PersonaWallet()
File "/Users/paul/code/python/persona_wallet.py", line 42, in init
persona_selection = int(input("Select a persona to use: "))
ValueError: invalid literal for int() with base 10: 'l'

oh ... always asks for new persona ... but maybe it is persisted

Parsing error ... persona are persisted


Test Move widget import service to core

Closing for now, I'm looking at bringing over the other import/export services at the same time


How to trigger XN handover

I want to know how to create two gnb
corenet@corenet-virtual-machine:~/PacketRusher$ sudo ./packetrusher multi-ue -n 1 --xnh 1000
INFO[0000] Loaded config at: /home/corenet/PacketRusher/config/config.yml
INFO[0000] PacketRusher version 1.0.1
INFO[0000] ---------------------------------------
INFO[0000] [TESTER] Starting test function: Testing registration of multiple UEs
INFO[0000] [TESTER][UE] Number of UEs: 1
INFO[0000] [TESTER][GNB] gNodeB control interface IP/Port: <IP_ADDRESS>/38412
INFO[0000] [TESTER][GNB] gNodeB data interface IP/Port: <IP_ADDRESS>/2152
INFO[0000] [TESTER][AMF] AMF IP/Port: <IP_ADDRESS>/38412
INFO[0000] ---------------------------------------
WARN[0000] [TESTER] We are increasing the number of gNodeB to two for handover test cases. Make you sure you fill the requirements for having two gNodeBs.
INFO[0000] [GNB] SCTP/NGAP service is running
INFO[0000] [GNB] Initiating NG Setup Request
FATA[0000] Error incannot assign requested address

Hi @LiuYu97,
Thanks for using PacketRusher :-)
To use Xn Handover with the command and the configuration you mentioned, you need to have both <IP_ADDRESS> and <IP_ADDRESS> IP address configured on your virtual machine.
Can you try adding <IP_ADDRESS> as the second IP of your VM's Ethernet interface and see if it works?
eg. sudo ip addr add <IP_ADDRESS> dev <INTERFACE>
where  is the interface with the <IP_ADDRESS> IP.
Make sure <IP_ADDRESS> is not used by another of your VM as well.
When you use PacketRusher Xn Handover feature, the second gNodeB will automatically use the IP next to the one configured for the first gNodeB (eg. <IP_ADDRESS> in your config.yml).
We'll try to improve this so the second gNodeB can be configured from the config.yml, we'll also make the error "Error incannot assign requested address" clearer in what IP it was unable to bind to. (cc: @Raguideau)
Thanks a lot!
Valentin

In my  VM, there are two network interfaces, and the IP are <IP_ADDRESS> and <IP_ADDRESS> respectively.
Can you give me 'config.yml' demo for Xn handover?

You cannot add two gNodeB in the config for now, you must have only a single gNodeB: https://github.com/HewlettPackard/PacketRusher/blob/main/config/config.yml
When using the --xnh or the --ngh option, PacketRusher automatically creates a second gNodeB with the same configuration as the first gNodeB, but with the first IP next to the first gNodeB: if the first gNodeB has IP <IP_ADDRESS>, the second gNodeB created automatically by PacketRusher will have IP <IP_ADDRESS> and this cannot be configured differently for now.
Using the configuration I shared, the first gNodeB will have IP <IP_ADDRESS>, and the second gNodeB will automatically have <IP_ADDRESS>.
As such, I would recommend you to use two sequential IP for your two network interfaces.
Thanks and cheers!
Valentin

I think i have understood, thank you very much for your response.
Thank you.

Great to hear! Feel free to reopen an issue if you need help with something else :-(


[EventRequest]Table 570 G/L Account Category

Due to Polish Localization App building, we need four publishers in T570. We need to override default calculation of category balance, moving and presentation order updating mechanism.
Function: UpdatePresentationOrder
IF "Entry No." = 0 THEN
EXIT;
// START
OnBeforeUpdatePresentationOrder(Rec,IsHandled);
IF IsHandled THEN
EXIT;
// STOP
GLAccountCategory := Rec;
LOCAL [IntegrationEvent] OnBeforeUpdatePresentationOrder(VAR Rec : Record "G/L Account Category";VAR IsHandled : Boolean)

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


Class 'Google_Client' not found

Hi,
I followed the instructions and now get:
Fatal error: Uncaught Error: Class 'Google_Client' not found in /var/www/html/user/plugins/google-auth/plugin.php:36
Please help?

I think I've fixed it. I did not user composer, and only included /vendor. I've know included the complete contents and updated plugin.php line 21:
require_once __DIR__ . '/google-api-php-client-2.4.1/vendor/autoload.php';


Terra Hyperlink update icons from css to jsx

Feature Request
Enhance hyperlink to update variant icons (external, audio, video, image, document) from css to jsx
Description
Update the Terra-Hyperlink component to switch from themeable icons provided by CSS to imported terra-icon component and inline content with JSX. This will allow the icons to be able to take the current-color of the text for all states (hover, active, visited, inactive, etc.).
Other Items to Include:

Determine if space between text and icon needs to be themeable
Additional theming color value visited+focus state.
The underline-hidden version needs to add additional text color theme values for all states (e.g. so when re-theming and basic hyperlink shows visited-state as purple, the underline-hidden version stays its own unique color and doesn't show as purple when visited).

Planning to hold off on this change until around the end of Q2/beginning of Q3 to minimize major version bumps in the Terra UI ecosystem.

@neilpfeiffer
What do we expect the color of visited+focus state to be in all the themes?

This will mainly be a tech-only update (swap the icons and add additional theming variables).

Work needed is for:

https://github.com/cerner/terra-core/pull/2666#discussion_r334440023: new variable for space between icon

https://github.com/cerner/terra-core/pull/2666#discussion_r334434956: all theme colors remain same are current, need to add theme variables for these states
[ ] visited and focus (color),
[ ] underline-hidden and normal (color),
[ ] underline-hidden and hover (color),
[ ] underline-hidden and focus (color),
[ ] underline-hidden and visited (color),
[ ] underline-hidden and visited and focus (color),
(default theme has 1 WCAG color change: noted here)
(clinical-theme: https://zpl.io/bew3P5D )

Removing MVP label as this will be update to internal construction and not a major version.


Just a heads up!

Hey Daniel!
I know you haven't worked on musikcube in a long time, and don't really expect you to accept the pull request, but I just wanted to let you know I've been hacking on the old code for a while now. I ended up changing jobs a while back, so I no longer have a conflict of interest. And I always wanted to finish musikcube.
For better or worse, I ended up stripping a bunch of the functionality I wasn't personally using to make the code base smaller and easier to work with it -- things like the XML/HTTP server were removed for the time being, and the entire UI layer (including win32cpp) was also deleted. I'd like to add the client/server model back at some point soon, but using something more efficient, maybe protocol buffers, and take advantage an existing boost-compatible HTTP server library instead of a home grown one to make maintenance easier.
I added a multi-platform, curses-based frontend and implemented output devices for Linux (using ALSA) and OS X (using CoreAudio). I was also able to refactor the audio engine to support true gapless playback, and fixed a handful of obscure race conditions along the way.
Also, because your version of the repo is old, I was thinking about promoting mine to a top-level (i.e. non fork) so I have things like ownership of the wiki. Not sure if you'd be cool with this, though, so let me know. :)
Best,
Casey

Awesome!
You should promote your repo as the master if you are working on it. I just converted the repo from google code to github so that we would not loose any of the old code 😄
I can delete this repo if you like - or maybe add a README that directs to your repo instead?

Or - https://help.github.com/articles/about-repository-transfers/

Oh, neat. A repository transfer sounds good -- that documentation suggests my fork promoted to the "master" fork, and yours would then show up as a fork of mine. Then you wouldn't have to update yours in an anyway, or delete it.
Am I understanding properly? Should we give it a go? :)

No..Didn't work. You already have that repo

Guess I can rename it first and then transfer.
How do I promote yours?

Yeah, hmm. It looks like I have to delete my instance of the project, renaming it won't be enough (see https://github.com/stevenvachon/broquire/issues/3).
What would you think about merging my pull request (I can create another one with more up to date changes), then I can delete my repo and you can transfer ownership then? At that point you can probably fork mine, then revert my changes and force push to your instance if you'd like to preserve your current state of the world.
Or would you rather keep yours around without any of my code? I'd understand if that's the case.

Just let me know when I can transfer 👍

Alright, give it a shot, I just deleted my fork!

Worked 👍

Awesome, thank you!  As always, feel free to play with things if you ever get bored. :)


Optional extendable GUI

Hello,
I've added the option to use an user extendable GUI.
Therefore I also did some changes to the Machine.plot method to also plot some intermediate steps.
There is also a slight BC that verifies that the loaded 'json' is an Machine instance.
Best regards,
Sebastian

Thank you for your contribution :)
I particularly like how you reorganized the machine plot.

As a side note, it would be interesting to check how the "plot" part of the extended GUI behave at each step of the machine GUI (starting from a new machine) since for now most of the parameter are set to None at the beginning of the process. Or we can go forward on the "defaut value" approach for the GUI

Actual the plot of a new machine will cause an error (TypeError 'None'). I think thats okay for now.
This also led me to the question, what is the purpose of the check methods. Maybe they could be utilized to check if a specific machine part can be plotted.
"Default values" are a nice to have feature and i think we should keep that in mind. But in my opinion there are some other issue with a higher priority, i.e. Material Library (I will open an issue.) and iron losses.

Indeed the check method enables us to know which part of the current machine are correctly set. We use them to know which step of the nav_step are available. So we can use it to know which part of the machine we can plot.


release: 0.21.0

https://github.com/leptonai/leptonai/actions/runs/9474306336

https://github.com/leptonai/leptonai/actions/runs/9476211553


Fetching Devices, no devicesa are published to HomeKit - takes very long

What issue do you have? Please be as thorough and explicit as possible.
Since today, after a restart of the homebrige, no eWeLink devices are published to homekit anymore. After a while the message: [01/09/2022, 08:19:46] [eWeLink] Unable to reach eWeLink, retrying in 30 seconds [getHost() - ECONNABORTED]. is displayed and the devices will be discovered an provided to homekit. The eWeLink App is running fine instead.
Details of your setup.

Do you use (1) Homebridge UI-X (2) Homebridge CLI or (3) HOOBS?
(1) Hombrige UI-X

Which version of Homebridge/HOOBS do you have?
1.5.0

Which platform do you run Homebridge/HOOBS on (e.g. Raspberry Pi/Windows/HOOBS Box)? Please also mention your version of Node.js/NPM if known.
Raspberry Pi 4B 4 GB

Which version of this plugin (homebridge-ewelink) do you have? Has the issue started since upgrading from a previous version?
homebridge-ewelink v10.0.0

Which eWeLink devices do you have that are causing issues? Please include product models if applicable.
All models

Please paste any relevant logs below.

[01/09/2022, 08:11:01] [eWeLink] Fetching devices from eWeLink home/id [Wxxxxx] [xxxxxxxxxxxxxxxxxxxxxx].
[01/09/2022, 08:19:46] [eWeLink] Unable to reach eWeLink, retrying in 30 seconds [getHost() - ECONNABORTED].

Hi,
Closing this just as been a little overwhelmed with issues recently.
If this is still an issue or a feature you would like implemented then please do re-open this ticket!
Thanks for your understanding!


Completed changes for GroundTruth theme.

@tim-wgbh Let me know what you think!

Looks, good, except for one thing on you, and maybe one thing on them.  When you add a new font, you need to specify it in settings/index.js. You'll notice that the caption font is not correct - looks like Times. The other thing is that on the square audiogram, I think it would look better with centered text, but that would be something to bring up with them.

@tim-wgbh entry added to index.js! Thank you!


kvstore-perf core mask.

Currently the tool only supports N number of cores, counting from core 0.  We need to be able to support a range of specific cores.  They should be of the format..
1,2,3,6-10,12-40

I see a single range is supported, but no multiple?

Done on commit ID 0fe872f3d04548718e5825f2dcdf8db7d71e2b04. Pasting in comments about this change below:
kvstore-perf: core mask supports ranges, singles

    general:
    - behavior of the --cores input argument has changed (see issue #35)
    - previously, --cores=N would run tasks on cores 0 to N-1
    - now, cores input arg supports arbitrary masks made up of single cores
      (separated by commas) and ranges (separated by '-' character)
    - example: --cores="1,2,3,6-10,12-40" will run tasks on cores 1, 2, 3, 6 to 10, and 12-40
    - example: --cores="8" will run tasks on core 8


Two factor authentication issue?

I've enabled two factor authentication in RainLoop . It's working perfectly fine, but I have one concern. On login page there is a check box that states "Do not ask for code for two weeks". But if I don't log off for two weeks RainLoop still keeps my session. Shouldn't RainLoop force automatic log off after two weeks?

These are different things and they are not related.

Can you elaborate? For what this check box is used?

This checkbox used for two factor code only.
You don't need to enter this code each time you enter the webmail.

So basically this check box is useful when you login and log off all the time to RL. Checking it will prevent asking for two factor code for two weeks. And if you don't log off then in fact it doesn't matter if you check this box or not?
From security point of view wouldn't be a better to force log off user after some time? Just to make sure that the person using the mail is still the mail owner.

Do you mean "Remember Me" functionality?

Yes, I mean if my two factor code is valid for two weeks then "remember me" perhaps should also work only for two weeks?

Well, signme token has limitation in 1 month.
Of course I can reduce it but I'm not sure it's necessary.

No need to. I think now I understand how "remember me" and "do not ask for code" are linked.
Thanks for explanation!

You're welcome.


IPLDT Scan Fixes

 - Changed references to be: Intel® Manager for Lustre* software
 - Changed references to be: Intel Corporation
 - The official names can be found here: namesdb.intel.com
 - Re-scanned the repo and ensured that the TM errors went away.

This change is

I will load this up on my cluster and check the registered symbols.

  File "setup.py", line 22
SyntaxError: Non-ASCII character '\xc2' in file setup.py on line 22, but no encoding declared; see http://www.python.org/peps/pep-0263.html for details
make[1]: *** [tarball] Error 1
make[1]: Leaving directory `/var/lib/jenkins/workspace/manager-for-lustre/arch/x86_64/distro/el7/chroma-agent'
make: *** [chroma-agent] Error 2
Build step 'Execute shell' marked build as failure

Good catch Will.

File "setup.py", line 22
SyntaxError: Non-ASCII character '\xc2' in file setup.py on line 22, but no encoding declared; see http://www.python.org/peps/pep-0263.html for details
make[1]: *** [tarball] Error 1
make[1]: Leaving directory `/var/lib/jenkins/workspace/manager-for-lustre/arch/x86_64/distro/el7/chroma-agent'
make: *** [chroma-agent] Error 2
Build step 'Execute shell' marked build as failure

Thanks for confirming, that was what I was wondering.

I loaded this build on my vagrant adm node successfully.
I clicked on the "Admin" tab and at the bottom, the footer shows:
Intel® Manager for Lustre* software Build jenkins_manager_for_lustre_1581 is Copyright © 2017 Intel Corporation. All rights reserved. About Intel® Manager for Lustre* software System Status
So far, so good. Still clicking and testing.

Just in case it's not clear, not all help strings come from the IML repo.

Thanks. That makes sense. I was just checking to see if anything was obviously broken and so far I am not seeing anything from the initial install.


Suggestion: cut with plane

Hello, Thank you for your effort. I like it how easy to use this is. I was working with cut_with_plane and notice that it only returns one mesh and discard the other although vtk has the option to return either. I made some changes to the code so that the user can control which mesh to return.  Hopefully this improves on already a great repo.
`
def cut_with_plane2(mesh, origin=(0, 0, 0), normal=(1, 0, 0), direction=0):
    """
    Cut the mesh with the plane defined by a point and a normal.
    Parameters
    ----------
    origin : array
        the cutting plane goes through this point
    normal : array
        normal of the cutting plane
    direction: bool
        0,1 which cut to keep
    Example:
        .. code-block:: python
            from vedo import Cube
            cube = Cube().cut_with_plane(normal=(1,1,1))
            cube.bc('pink').show()
    .. hint:: examples/simulations/trail.py
        .. image:: https://vedo.embl.es/images/simulations/trail.gif
    Check out also:
        ``crop()``, ``cut_with_box()``, ``cut_with_cylinder()``, ``cut_with_sphere()``
    """
    s = str(normal)
    if "x" in s:
        normal = (1, 0, 0)
        if "-" in s:
            normal = -np.array(normal)
    elif "y" in s:
        normal = (0, 1, 0)
        if "-" in s:
            normal = -np.array(normal)
    elif "z" in s:
        normal = (0, 0, 1)
        if "-" in s:
            normal = -np.array(normal)
    plane = vtk.vtkPlane()
    plane.SetOrigin(origin)
    plane.SetNormal(normal)

    clipper = vtk.vtkClipPolyData()
    clipper.SetInputData(mesh.polydata(True))  # must be True
    clipper.SetClipFunction(plane)
    clipper.GenerateClippedOutputOn()
    clipper.SetValue(0)
    clipper.Update()

    cpoly = clipper.GetOutput(direction)

    if mesh.GetIsIdentity() or cpoly.GetNumberOfPoints() == 0:
        mesh._update(cpoly)
    else:
        # bring the underlying polydata to where _data is
        M = vtk.vtkMatrix4x4()
        M.DeepCopy(self.GetMatrix())
        M.Invert()
        tr = vtk.vtkTransform()
        tr.SetMatrix(M)
        tf = vtk.vtkTransformPolyDataFilter()
        tf.SetTransform(tr)
        tf.SetInputData(cpoly)
        tf.Update()
        mesh._update(tf.GetOutput())

    return mesh

`

Hi @saurabhpre
Thanks!
In principle the user can decide which one to keep by flipping the signs of normal,  I understand that maybe it is something that one does not think of , in the sense that it may not be intuitive enough..
That is why other similar methods have an invert=True keyword .. maybe cut_with_plane() should have the same thing even if it's redundant.
You are welcome to make a PR if you wish!
Another interesting option i'm thinking of right now is to return the same (uncut) mesh with an additional boolean flag in mesh.pointdata.


Testing organizational changes

Overview
We should consider moving integration tests that use mocks to functional tests.
Reason: These tests are great in that they validate some expected behavior and increase the confidence of the code.
They are not like the integration tests though and do not need to run X times where X is the number of project types.
Also, it is a good thing that they use in mocks, in my opinion, because it allows them to be like unit tests and focus on the one bit of logic they aim to test.
The integration tests should stick to project-type based testing.
Additionally, we should structure the functional tests to align wit the structure of the source code.
Reason: Code auditing.  Unit tests are great for auditing code bases, especially for newcomers.
It makes it easier to change background-functionality for util methods, manager classes, etc. It allows us to know which cases we are testing for, which can sometime be indicative of which bugs we have encountered. It allows contributors to confidently change method implementations (upgrading python, for example).
Also, integration tests take a long time to run and are best suited for running in CI/CD. However, at this time, we don't have too many functional tests. So, part of this is to test the manager classes more thoroughly.
Specification
Proposed structure:
tests/
   integration/
      cli/
         test_accounts.py (no mocks)
         ....
   functional/
       ape/
          test_utils.py (could contain what is in test_ether.py currently as well
          managers/
             test_accounts.py
       ape_accounts/
         test_cli.py (mocks! some of the tests that were in integration/cli can be moved here)
      ....

Notice that the functional tests follow the codebase structure for auditing purposes.
Notice that there is a test_cli.py in the functional tests that can still use the CliRunner however it mocks all of the ape stuff to be more like unit tests. We already have tests like this that can be moved there. Testing for error conditions or output formatting are good candidates here.
Dependencies
Include links to any open issues that must be resolved before this feature can be implemented.

I do agree with pushing as much logic out of the _cli modules as possible, and into managers or wherever.

Closing for now.
Lets make good tests all the way around.


feat(zbugs): useWatchQuery

This is WIP but I want some feedback.
This creates a new hook that takes a callback that gets called with the changes that come out at the end of the pipeline.
This is based on the ideas I had last week (or the week before that?) when I first worked on adding the emoji change callbacks.
Usage:
useWatchQuery(z.query.comment.where('issueID', id), change => {
  if (change.type === 'add') {
    console.log(change.row, 'was added');
  } else if (change.type === 'remove') {
    console.log(change.row, 'was removed');
  }
});

Things that are not obvious/clean.

Name of the hook?
The implementation waits for the transaction end but this information is lost in the callback. We could call the callback with an array of all the changes instead.
What to do about child changes since at this level there is no memory of the previous children?
Should the hook have an option to also be called for non complete results or maybe add that info to the change objects?

Your perlow contact in the channel had an amazing idea: to propagate
mutation name and args to client and drive effects off that. There was a
long thread.
a (phone)
On Sat, Dec 21, 2024 at 8:35 AM Greg Baker @.***> wrote:

@.**** commented on this pull request.
In apps/zbugs/src/hooks/use-watch-query.tsx
https://github.com/rocicorp/mono/pull/3307#discussion_r1894669487:

flush() {
if (!this.#queryComplete) {

 return;

}

for (const change of this.#pendingChanges) {

 for (const cb of this.#subscribers) {

   cb(toChange(change));

 }

}
this.#pendingChanges = [];
}
+}

+function changeViewFactory<
Hmmm I wonder if we can reasonably push this down a layer so that it can
be used in non-React environments like Cloudflare-DO, NodeJS, vanilla JS.
—
Reply to this email directly, view it on GitHub
https://github.com/rocicorp/mono/pull/3307#pullrequestreview-2518804418,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAATUBDE5IWMJNVWCGJCZA32GWYHRAVCNFSM6AAAAABTSGDWCSVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDKMJYHAYDINBRHA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Make compatible with Ghost 1.x

Resolves #1
Fixed errors thrown by Ghost when trying to add theme.

Changes only seemed to be required for partials/post-single.hbs and author.hbs, specifically needed to use updated image helper.

Hi, thanks! Is this backwards-compatible with 0.7.x ?

Ohhhhh.... good question. I hadn't tried. Since their are no other changes necessary (so far?) perhaps this deserves a major version bump?

Ohhh gosh wish I have seen this earlier... I have done the same fix on this theme for Ghost 2.x, just the same as Macrofig's and the fix still works.
Consider merge this PR?  I believe almost nobody uses Ghost 0.x now.....@belen-albeza


Further deduplicate code for organization-level icon uploads

From https://github.com/zulip/zulip/pull/13038#issuecomment-523680033:

[x] Rather than having the {{theme_mode}}-settings class on the button, we could just use a .closest("realm-logo-section"), so that only the outer ID differs between the two widgets.
[x] I think the whole upload_realm_icon code path could likely also be deduplicated with upload_ream_logo in a similar way (just with an additional case in the big conditionals).
[ ] I'm less sure about this, but I think ultimately we could just have an image_upload_widget.js code and template, and share that logic and code with the user's personal avatar as well.  That would likely be significantly more messy on the CSS/template end, since there's fairly different styling, but the JS logic should be possible to deduplicate (importing the better UX we have on the avatar of the "click and widget" and "X icon" rather than big upload buttons in the process).  Something to work towards.

Tagging as as priority because duplicated code tends to be expensive in terms of future bugs.

@zulipbot claim

@Jagansivam28 @clarammdantas @sahil839 is there more we should do here before closing this as complete?
It feels like there may still be some more code cleanup opportunities, so maybe worth you all taking a quick look.

@zulipbot claim

Yes, I have some thoughts about the image_upload_widget.scss. I think we should be defining attributes such as position, height, and weight of most of the elements in specific files, like settings.scss, so it can give more flexibility when one does use this widget. For example, the .image-block having the height and weight of 200x200 pixels is quite big to use in the registration form, which I wanted it to be 150x150. And if I want to use a smaller size for the .avatar-block, I would have to adjust the position of a couple of elements like .image_upload_spinner and the texts of "Delete profile picture" and "Upload profile picture".

@zulipbot claim

Currently Working on this !


Runtime error: node rpsls-cli.js command not found

What is happening?
All of my autograder tests for rps are working, but I'm getting errors for every rpsls test. However, both work when I test them locally in my terminal.
What does the error say is happening?
node-rpsls is not found, node-rpsls: command not found
What do you think the error means?
I think it means something is off with my node or the way I installed/exported the packages.
Command string
n/a
Code snippet
n/a
Screenshots

Error log
/bin/sh: 1: node-rpsls: not found
temp.sh: line 1: node-rpsls: command not found
Node.js version
v19.4.0
Operating system
macOS Big Sur 11.2.3

This is probably because the rpsls-cli.js file is not executable. Make sure that it has the permissions set correctly. If you descend into the directory where it lives and do ls -lah and the permissions look different than those for rps-cli.js then you need to do chmod +x rpsls-cli.js and it should work after that.


display issue

I am able to download to only m slide manager and I can load pictures, but I have no option to play or display.
my JW Library works just fine.
I've

uninstalled and reinstalled and shut down and restarted my computer.
Same result.

Hey sdavis5299,
the main tool is OnlyM. This is the tool, to display images, videos, pdf-files and websites. Also it can display SlideShows you generated using the OnlyM Slide Manager. So please, have a look to the main tool, I guess this solves your problem ;-)
Many love and regards,
Benjy

@sdavis5299 as mentione by @yjenb you need to use OnlyM to display the slides.


Broaden scope in which local_path mechanism can be used for Files

It does not need to be restricted to specific URI schemes - any staging
mechanism should be able to specify an overridden local path.
This comes from a review of #771 which does something similar but not
quite the same as the URI-based staging system

@AndrewLitteken notes in https://github.com/Parsl/parsl/pull/771#issuecomment-472230993 that this PR is merged into PR #771 (which is what I originally created this PR for)


Incorrect result from indexOnly isActive() with nested IndexRoute

With a route configuration that contains an IndexRoute at the top level (/) and another IndexRoute nested within a child Route (/child), isActive('/', {}, true) returns true even when the current path is /child.
I was expecting the correct behaviour would be for isActive('/', {}, true) to return false and isActive('/child', {}, true) to return true.
Below is a sample route:
  <Router>
    <Route path='/' component={ Home }>
      <IndexRoute component={ TopIndex }/>
      <Route path='content' component={ TopContent }/>
      <Route path='child' component={ Child }>
         <IndexRoute component={ ChildIndex }/>
         <Route path='content' component={ ChildContent }/>
      </Route>
    </Route>
  </Router>

And a example in CodePen to illustrate the issue: http://codepen.io/anon/pen/wKXjxE
I am using react-router 1.0.0-rc1.

This was fixed in a later release.


Issue on converting yolo to tflite with bazel-bin toco

System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Ubuntu 18.10
TensorFlow installed from (source or binary):
TensorFlow version (or github SHA if from source):  master

Provide the text output from tflite_convert
I tensorflow/lite/toco/import_tensorflow.cc:1335] Converting unsupported operation: ExtractImagePatches
2019-03-10 16:38:02.722378: I tensorflow/lite/toco/graph_transformations/graph_transformations.cc:39] Before Removing unused ops: 211 operators, 370 arrays (0 quantized)
2019-03-10 16:38:02.727060: I tensorflow/lite/toco/graph_transformations/graph_transformations.cc:39] Before general graph transformations: 211 operators, 370 arrays (0 quantized)
2019-03-10 16:38:03.193935: I tensorflow/lite/toco/graph_transformations/graph_transformations.cc:39] After general graph transformations pass 1: 97 operators, 189 arrays (0 quantized)
2019-03-10 16:38:03.195613: I tensorflow/lite/toco/graph_transformations/graph_transformations.cc:39] Before Group bidirectional sequence lstm/rnn: 97 operators, 189 arrays (0 quantized)
2019-03-10 16:38:03.196705: I tensorflow/lite/toco/graph_transformations/graph_transformations.cc:39] Before dequantization graph transformations: 97 operators, 189 arrays (0 quantized)
2019-03-10 16:38:03.198405: I tensorflow/lite/toco/allocate_transient_arrays.cc:345] Total transient array allocated size: 68550208 bytes, theoretical optimal value: 66453504 bytes.
2019-03-10 16:38:03.202350: E tensorflow/lite/toco/toco_tooling.cc:456] We are continually in the process of adding support to TensorFlow Lite for more ops. It would be helpful if you could inform us of how this conversion went by opening a github issue at https://github.com/tensorflow/tensorflow/issues/new?template=40-tflite-op-request.md
 and pasting the following:

Some of the operators in the model are not supported by the standard TensorFlow Lite runtime. If those are native TensorFlow operators, you might be able to use the extended runtime by passing --enable_select_tf_ops, or by setting target_ops=TFLITE_BUILTINS,SELECT_TF_OPS when calling tf.lite.TFLiteConverter(). Otherwise, if you have a custom implementation for them you can disable this error with --allow_custom_ops, or by setting allow_custom_ops=True when calling tf.lite.TFLiteConverter(). Here is a list of builtin operators you are using: CONCATENATION, CONV_2D, MAXIMUM, MAX_POOL_2D, MUL, PAD. Here is a list of operators for which you will need custom implementations: ExtractImagePatches.
We are continually in the process of adding support to TensorFlow Lite for more ops. It would be helpful if you could inform us of how this conversion went by opening a github issue at https://github.com/tensorflow/tensorflow/issues/new?template=40-tflite-op-request.md
 and pasting the following:

Some of the operators in the model are not supported by the standard TensorFlow Lite runtime. If those are native TensorFlow operators, you might be able to use the extended runtime by passing --enable_select_tf_ops, or by setting target_ops=TFLITE_BUILTINS,SELECT_TF_OPS when calling tf.lite.TFLiteConverter(). Otherwise, if you have a custom implementation for them you can disable this error with --allow_custom_ops, or by setting allow_custom_ops=True when calling tf.lite.TFLiteConverter(). Here is a list of builtin operators you are using: CONCATENATION, CONV_2D, MAXIMUM, MAX_POOL_2D, MUL, PAD. Here is a list of operators for which you will need custom implementations: ExtractImagePatches.

Also, please include a link to a GraphDef or the model if possible.
Model is retrainer yolo
Any other info / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached.

@antonino-tocco Could you add more details on the context of the issue and the process you followed leading to the bug? Please provide a  code to reproduce the bug. Thanks!

Hi @jvishnuvardhan, thanks for your help.
I follow this tutorial for retrain yolo: https://www.tooploox.com/blog/card-detection-using-yolo-on-android.
I train my model, convert it to protobuf with darkflow and then i try to convert it to tflite for embedd it on my Android app with baze-bin toco. I receive issue for non tflite operators such as ExtractImagePatches.

@antonino-tocco Could you provide full command of tflite_convert? Thanks!

Sure.
/bazel-bin/tensorflow/lite/toco/toco   --input_file=yolo-obj.pb   --output_file=foo.tflite    --input_format=TENSORFLOW_GRAPHDEF   --output_format=TFLITE   --input_arrays=input   --input_shapes=1,416,416,3
Thanks for your help.

Hi, any news?

It's missing an ExtractImagePatches op in TensorFlow Lite.
@talumbau could you take a look to see should we implement it as a TensorFlow Lite builtin op?
Meanwhile, we have an experimental feature that you can use TensorFlow kernel in TensorFlow Lite.
 It would be helpful if you could inform us of how this conversion went by opening a github issue at https://github.com/tensorflow/tensorflow/issues/new?template=40-tflite-op-request.md
and pasting the following:
Some of the operators in the model are not supported by the standard TensorFlow Lite runtime. If those are native TensorFlow operators, you might be able to use the extended runtime by passing --enable_select_tf_ops, or by setting target_ops=TFLITE_BUILTINS,SELECT_TF_OPS when calling tf.lite.TFLiteConverter(). Otherwise, if you have a custom implementation for them you can disable this error with --allow_custom_ops, or by setting allow_custom_ops=True when calling tf.lite.TFLiteConverter(). Here is a list of builtin operators you are using: ADD, CONCATENATION, CONV_2D, LEAKY_RELU, MAX_POOL_2D, MUL, RESHAPE, RSQRT, TRANSPOSE. Here is a list of operators for which you will need custom implementations: BatchNormalization, VariableV2.
We are continually in the process of adding support to TensorFlow Lite for more ops. It would be helpful if you could inform us of how this conversion went by opening a github issue at https://github.com/tensorflow/tensorflow/issues/new?template=40-tflite-op-request.md
and pasting the following:
Some of the operators in the model are not supported by the standard TensorFlow Lite runtime. If those are native TensorFlow operators, you might be able to use the extended runtime by passing --enable_select_tf_ops, or by setting target_ops=TFLITE_BUILTINS,SELECT_TF_OPS when calling tf.lite.TFLiteConverter(). Otherwise, if you have a custom implementation for them you can disable this error with --allow_custom_ops, or by setting allow_custom_ops=True when calling tf.lite.TFLiteConverter(). Here is a list of builtin operators you are using: ADD, CONCATENATION, CONV_2D, LEAKY_RELU, MAX_POOL_2D, MUL, RESHAPE, RSQRT, TRANSPOSE. Here is a list of operators for which you will need custom implementations: BatchNormalization, VariableV2.
I had a same issue with yolo.
I have used  command /bazel-bin/tensorflow/lite/toco/toco   --input_file=/newhd/dev/darkflowToTensorflow/data/yolo-voc.pb   --output_file=foo.tflite    --input_format=TENSORFLOW_GRAPHDEF   --output_format=TFLITE   --input_arrays=input   --input_shapes=1,416,416,3   --output_arrays=yolov2/convolutional23/BiasAdd    --target_ops=TFLITE_BUILTINS,SELECT_TF_OPS

Can you try rerun the tutorial from the "Convert .weights to a frozen graph" step? This might have been fixed in by https://github.com/tensorflow/tensorflow/commit/0f486fc67070ba888204741c404a55a5f1a41fbc.

Hi, thank you to all for the help. I can retry on this weekend. I will update you

@himanshurobo Hi, I have the same issue with you. I also have added --target_ops. But it's still not working. Have you fixed it? Thanks

Hi!
I've download tf folder from here: https://github.com/tensorflow/tensorflow and I run this command
'
$ cd tensorflow-master
$ bazel run tensorflow/lite/toco:toco --
--input_file=my.pb
--output_file=out.tflite
--input_shapes=1,416,416,3
--input_arrays='input_1'
--output_format=TFLITE
--output_arrays='output_0','output_1'
--inference_type=QUANTIZED_UINT8
--std_dev_values=128 --mean_values=128
--default_ranges_min=-6 --default_ranges_max=6
--change_concat_input_ranges=false
--allow_custom_ops
'

Hi @antonino-tocco ! I converted the yolo frozen graph file using TFlite converter api. Attaching gist for reference. Thanks!


added script for datadog script to docs resolves #25411

Hey, I just made a Pull Request!
Added docs for fixing error with dd rum installatin
:heavy_check_mark: Checklist

[ ] All your commits have a Signed-off-by line in the message. (more info)

resolves #25411

Hey! Back from vacation now, sorry for the delay. Just two small suggestions and then happy to merge! Thanks for your efforts.

Hey! Back from vacation now, sorry for the delay. Just two small suggestions and then happy to merge! Thanks for your efforts.

Thanks alot for the review and suggestions, I have made the changes


Unwanted YAML and properties changes with 4.41.0 - compared to 4.39.0

Hi,
I'm not sure if it's a problem with OpenRewrite itself or the Rewrite Maven plugin but after an upgrade to 4.41.0, the Rewrite Maven plugin starts pushing unwanted changes to my YAML and properties files.
When rewriting https://github.com/quarkiverse/quarkus-github-app with our Quarkus upgrade script, I end up with the following diff between a rewrite with 4.39.0 and a rewrite with 4.41.0:
(the private key is a test one that isn't used anywhere, no worries here)
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
index 84703f7..b2452e3 100644
--- a/.github/workflows/build.yml
+++ b/.github/workflows/build.yml
@@ -30,6 +30,7 @@ jobs:
       fail-fast: false
       matrix:
         java:
+
           - {
             version: "11"
           }
diff --git a/.github/workflows/downstream.yml b/.github/workflows/downstream.yml
index fdfd744..3a918c0 100644
--- a/.github/workflows/downstream.yml
+++ b/.github/workflows/downstream.yml
@@ -31,6 +31,7 @@ jobs:
       fail-fast: false
       matrix:
         project:
+
           - {
             organization: "quarkusio",
             repository: "quarkus-github-bot"
-quarkus.github-app.private-key=-----BEGIN RSA PRIVATE KEY-----\
-MIIEowIBAAKCAQEAtsCKYkMzgqSsbkcPThTjeYnd6r5RAS4q5SyiOyIf34nYH5D0\
-jy8Dlt1s8u69KmBoM9Neesqf2Yh80b/UNUtea9knpgR4ME+MIg1LdsLnUWoZuPBl\
-lLaMfwRkN4+ZbTJ7Uo98UUlxoTcCjZVzopSoIazuUF5u+01sx6QEcHB312s42FVL\
-gkd8T36wvIywseOPKr2oyCMQ2AdcxL/cEHwx2hcQAe+eTipsxNy4atIhbDWrML5W\
-v7cYlyn9/sKgxkXtW3eInP3VI3FS+IH+cjwO4FS6ez8TT6USfaCjDb4LgdEKt/de\
-Ds4TbqwM2vjqedLNBhE7t4V2Uf3STdFq+cKHwwIDAQABAoIBAE07MUA1hh8/2F8C\
-SMWGrls+mDoME8+b4yTWp/i1gyLE7kDo0XFxPOMU0GYZ3nd6Jo9AVD0wRD16IMXD\
-e7rKDy0kqEzQtroz49TAKZQW6grN+/DcJxGh094ZzQBt/zjWjKdnW/I+R3cJ+Fo9\
-PpEGoccZfd0ZC23IWqBEAFxEK2Eth7CbJW5eMJElyqSZa9jiF6wdnryH4Xww38iE\
-zv5tL2Ieg03lxj0mdeyzFyIzJTk3PEAFLIjIiducIhSWyJHltpKQEZ3iFMzKfx7l\
-U0KGUYECgYEA5xcnO9nv58QdcMXobHWethh+a2QTWmEXbE/Lnunqgq+bTgJUbgXR\
-Fi+awb+6GI8lJh16oBbzSRGe2pFQcQa9S84+CnAyVVyras4WC5bcl38q46wh0IRW\
-2sqvJfAI/CJZxgLVceELJ4hn9t8/7Ucoh6ZYYSsVnwtvvrmzMsC1z/0CgYEAynOC\
-v0O+FC5MI+p6HQXdK8JWNecIMEoat5RpmF6nvYSRmHliggoeAV5Of7b6Y0JBRgGm\
-+wECWCKMfaJ1BdCOncci+YkLguSBeJGbDkr65En/zzPxtuvYqcg05x50RWdbLQ5N\
-PF6FHSC8FILpfpJ5f/W8rfNNmU4eTKS4DxUr4r8CgYEA4vfqoO48ovYLtGetEFm1\
-uEP2ZqO0HmCeENOOulYk7pZrgwLmyekMoy2+Ye1daiGt6vGpLvNbn7ievS1cRKbJ\
-5Vp7tOTditmpww0GuftCTcmo5lR6IcLZS6smu6w2Ju3WHpVJ7r+JpRpkgiRjNTle\
-pVzMESOv6LXi2wCo8IA2EkECgYAOyzQRr+yS4vMzaK31svj/epr8I17I0JF1OsYg\
-mUIeqjJNdwlIwV6B8RdBY+iWGkBU0kgWbXNzZ0rm31k3zI6vXt7iZy5NKU+AtPsk\
-pzwANJwZ0wzltgRGG9gpz2Lls3DJMRNZxvppL3wu74YKdr+kJxvbhjz0Z+304dCF\
-YaGsVwKBgA6KSf1pPD93W/v5tV0WnhBZGKb3sX34JLQoCJPS5w6nTjHnuwpP9g0K\
-C8HXKULXuX/rb/gz99PNcPQ9GqB0Rw/ho1KhO3VLJjBiSh/hiASKuFGs2y59UXYt\
-mvqX3jAqi6ksGD5WPG6G2CdL/n30e5/lc10hq7SiwV7z9njW/8Pu\
------END RSA PRIVATE KEY-----
+quarkus.github-app.private-key=-----BEGIN RSA PRIVATE KEY-----MIIEowIBAAKCAQEAtsCKYkMzgqSsbkcPThTjeYnd6r5RAS4q5SyiOyIf34nYH5D0jy8Dlt1s8u69KmBoM9Neesqf2Yh80b/UNUtea9knpgR4ME+MIg1LdsLnUWoZuPBllLaMfwRkN4+ZbTJ7Uo98UUlxoTcCjZVzopSoIazuUF5u+01sx6QEcHB312s42FVLgkd8T36wvIywseOPKr2oyCMQ2AdcxL/cEHwx2hcQAe+eTipsxNy4atIhbDWrML5Wv7cYlyn9/sKgxkXtW3eInP3VI3FS+IH+cjwO4FS6ez8TT6USfaCjDb4LgdEKt/deDs4TbqwM2vjqedLNBhE7t4V2Uf3STdFq+cKHwwIDAQABAoIBAE07MUA1hh8/2F8CSMWGrls+mDoME8+b4yTWp/i1gyLE7kDo0XFxPOMU0GYZ3nd6Jo9/I+R3cJ+Fo9PpEGoccZfd0ZC23IWqBEAFxEK2Eth7CbJW5eMJElyqSZa9jiF6wdnryH4Xww38iEzv5tL2Ieg03lxj0mdeyzFyIzJTk3PEAFLIjIiducIhSWyJHltpKQEZ3iFMzKfx7lqLIczsyLRouc64+T7lW1PO6SQshHlvc6CtJaHHf98iFMDwdDlSolF/PfNLE5C6lAU0KGUYECgYEA5xcnO9nv58QdcMXobHWethh+a2QTWmEXbE/Lnunqgq+bTgJUbgXRFi+awb+6GI8lJh16oBbzSRGe2pFQcQa9S84+CnAyVVyras4WC5bcl38q46wh0IRW2sqvJfAI/CJZxgLVceELJ4hn9t8/7Ucoh6ZYYSsVnwtvvrmzMsC1z/0CgYEAynOCv0O+FC5MI+p6HQXdK8JWNecIMEoat5RpmF6nvYSRmHliggoeAV5Of7b6Y0JBRgGm+wECWCKMfaJ1BdCOncci+YkLguSBeJGbDkr65En/zzPxtuvYqcg05x50RWdbLQ5NPF6FHSC8FILpfpJ5f/W8rfNNmU4eTKS4DxUr4r8CgYEA4vfqoO48ovYLtGetEFm1uEP2ZqO0HmCeENOOulYk7pZrgwLmyekMoy2+Ye1daiGt6vGpLvNbn7ievS1cRKbJ5Vp7tOTditmpww0GuftCTcmo5lR6IcLZS6smu6w2Ju3WHpVJ7r+JpRpkgiRjNTlepVzMESOv6LXi2wCo8IA2EkECgYAOyzQRr+yS4vMzaK31svj/epr8I17I0JF1OsYgmUIeqjJNdwlIwV6B8RdBY+iWGkBU0kgWbXNzZ0rm31k3zI6vXt7iZy5NKU+AtPskpzwANJwZ0wzltgRGG9gpz2Lls3DJMRNZxvppL3wu74YKdr+kJxvbhjz0Z+304dCFYaGsVwKBgA6KSf1pPD93W/v5tV0WnhBZGKb3sX34JLQoCJPS5w6nTjHnuwpP9g0KC8HXKULXuX/rb/gz99PNcPQ9GqB0Rw/ho1KhO3VLJjBiSh/hiASKuFGs2y59UXYtmvqX3jAqi6ksGD5WPG6G2CdL/n30e5/lc10hq7SiwV7z9njW/8Pu-----END RSA PRIVATE KEY-----

So OpenRewrite is somehow:

adding new lines (with trailing spaces) to my YAML
reformatting my properties, removing the new lines in values

That's not something I want so I'm wondering what is going on.
Let me know if it's more of a core issue that should be moved there.
Thanks.

Thanks a lot!

I think I found out what the issue is.
My recipes targeting singleSource with wildcards are applied to all the files of the project, totally ignoring the applicability, AFAICS:
I get:
[WARNING] Changes have been made to runtime/src/main/java/io/quarkiverse/githubapp/runtime/github/GitHubConfigFileProviderImpl.java by:
[WARNING]     io.quarkus.updates.core.quarkus30.Quarkus3
[WARNING]         io.quarkus.updates.core.quarkus30.JavaxToJakartaDocumentation
[WARNING]             org.openrewrite.text.FindAndReplace: {find=javax.json.bind.config., replace=jakarta.json.bind.config.}
[WARNING]                 org.openrewrite.Recipe$AdHocRecipe
...

while I have for io.quarkus.updates.core.quarkus30.JavaxToJakartaDocumentation the following declaration:
type: specs.openrewrite.org/v1beta/recipe
name: io.quarkus.updates.core.quarkus30.JavaxToJakartaDocumentation
applicability:
  singleSource:
    - org.openrewrite.FindSourceFiles:
        filePattern: "**/*.adoc"
recipeList:
  - org.openrewrite.text.FindAndReplace:
      find: javax.json.bind.config.
      replace: jakarta.json.bind.config.

So this rule should only be applied to *.adoc files of my project but it is applied to Java files, properties, YAML..., thus modifying files that shouldn't have been modified (and me noticing it).
The upgrade starts being extremely slow as it applies all our text transformations to all the files instead of targeting only the ones in the applicability.singleSource.
I bissected and AFAICS the regression was introduced in OpenRewrite 7.37.0 (and it is still a problem in 7.37.3). 7.36.1 is working as expected.
So unless I'm doing something incorrect (but that was working before), a regression was probably introduced in 7.37.0 regarding applicability singleSource.
For reference, our recipes are hosted here: https://github.com/quarkusio/quarkus-updates/blob/main/recipes/src/main/resources/quarkus-updates/core/3alpha.yaml

Thanks a lot both for bisecting where the behavior changed as well as providing the additional context around the recipe & applicability test used. It looks like you're using a pattern similar to what we have in the documentation, but curiously that's also the only place I see FindSourceFiles used in that way. 🤔  That kind of makes me doubt if that's the correct way to limit recipe execution for text replacements to certain files. For instance: elsewhere we use HasSourcePath to limit recipes to certain files, which could be helpful here as well.
Would you mind giving this a try with HasSourcePath? That could indicate this is a "bug" in the documentation instead.

So this was because applicability tests didn't apply correctly and was fixed here (which is in the commit range of differences that @timtebeek tagged in his earlier comment):
https://github.com/openrewrite/rewrite/commit/d974a7d7975bec7980d358cdb123b1875223df22
Applicability tests were inappropriately selecting sources under the single source case, especially, which is what led to your issue here. If I were you, I'd try this again on the newest maven plugin and you should have much better luck.

@gsmet, I missed that on the first read; apologies about missing that!

@timtebeek the issue is that I cannot use HasSourcePath in my YAML descriptor directly as it is not a Recipe, it's a TreeVisitor.
My understanding is that using FindSourceFiles was the blessed way to apply recipes to a bunch of files.

@timtebeek any news? Asking that because I'm hitting another issue with 7.36.0 that seems to be fixed with the latest and I'm in an uncomfortable situation.
Thanks!

Understandable that you're looking to see this resolved; I don't have any immediate answers, but I've asked @kunli2 if he can chime in to see what (if any) changes need to be made to get you on the latest versions again.

I started with Open Rewrite 4.40.0 and did a migrate Java 11 with it. Wanted to do a migrate Java 17 but got the "[OpenRewrite: The contents of a quark are unknown, so the charset is unknown] bug. This was fixed in 4.41.1, so upgraded. Could now do the migration. Also noticed that in the Yaml files all new lines were indeed removed and now its all on 1 line.

I would bet it could be related to https://github.com/openrewrite/rewrite/commit/9b72d9abd9dfed058abc3b48bb8b990ab18fef20 .
I will try the version before this commit then the version after and report back.

After some bisecting, I can confirm that https://github.com/openrewrite/rewrite/commit/9b72d9abd9dfed058abc3b48bb8b990ab18fef20 is the culprit.

Thanks a lot for your efforts to really pin this down; hope @kunli2 is able to use that information for a fix or workaround.

Thanks for all the useful findings so far, I start working on this now.

this is fixed by https://github.com/openrewrite/rewrite/pull/2962, I have verified the https://github.com/quarkiverse/quarkus-github-app  and didn't see any unexpected changes.

Thanks @kunli2 , I tested latest main and AFAICS the issue is gone!
Looking forward to having a version of the Maven plugin coming with the fix!


Added eight new functions to diplib

Hi Cris,
I didn't follow all your changes when I was writing these. For example, 'Process' is not be changed to 'Dimensions'. If you want me to do this please feel free.
Yan

I've made a few minor changes, fixed the copyright notices, and committed here: dce2838e6cca26f28a362f01b9c6bc30ab8fd4eb
Many thanks!


Problem with create/update namespace responders

I'm trying to track down this problem with rspec testing namespaced IR controllers.  I thought it could be with Responders but I've tried with an without them.
Note: These work in development environment in a browser it just the test cases that fail.
# in app/controllers/abc/encounters_controller.rb
Abc::EncountersController < Abc::ApplicationController
  inherited_resources
end

Here is the RSpec2 controller test (note... I'm only including one failing case. Only the update/create actions fail).  These are basically the scaffold test cases.
# in spec/controllers/abc/encounters_controller_spec.rb
describe Abc::EncountersController do
  login_user    # from devise -- this seems to work with the rest of my tests

  describe "PUT update" do
    describe "with invalid params" do
       it "re-renders the 'edit' template" do
          encounter = Encounter.create! valid_attributes
          # Trigger the behavior that occurs when invalid params are submitted
          Encounter.any_instance.stub(:save).and_return(false)
          put :update, :id => encounter.id, :encounter => {}
          response.should render_template("edit")
      end
   end
end

Now the error I'm getting is undefined method encounter_url' for #Smd::EncountersController:0x00000005dd2c50` as its not correctly prepending the 'abc_' namespace to the redirect url.
Here is the portion of the stack trace I think is critical:
     # /home/jonlhouse/.rvm/gems/ruby-1.9.2-p0/gems/actionpack-3.1.3/lib/action_controller/metal/instrumentation.rb:59:in `redirect_to'
     # /home/jonlhouse/.rvm/gems/ruby-1.9.2-p0/gems/actionpack-3.1.3/lib/action_controller/metal/responder.rb:135:in `redirect_to'
     # /home/jonlhouse/.rvm/gems/ruby-1.9.2-p0/gems/actionpack-3.1.3/lib/action_controller/metal/responder.rb:193:in `navigation_behavior'
     # /home/jonlhouse/.rvm/gems/ruby-1.9.2-p0/gems/actionpack-3.1.3/lib/action_controller/metal/responder.rb:162:in `rescue in to_html'
     # /home/jonlhouse/.rvm/gems/ruby-1.9.2-p0/gems/actionpack-3.1.3/lib/action_controller/metal/responder.rb:160:in `to_html'
     # /home/jonlhouse/.rvm/gems/ruby-1.9.2-p0/gems/responders-0.6.4/lib/responders/flash_responder.rb:93:in `to_html'
     # /home/jonlhouse/.rvm/gems/ruby-1.9.2-p0/gems/actionpack-3.1.3/lib/action_controller/metal/responder.rb:153:in `respond'
     # /home/jonlhouse/.rvm/gems/ruby-1.9.2-p0/gems/actionpack-3.1.3/lib/action_controller/metal/responder.rb:146:in `call'
     # /home/jonlhouse/.rvm/gems/ruby-1.9.2-p0/gems/actionpack-3.1.3/lib/action_controller/metal/mime_responds.rb:238:in `respond_with'
     # /home/jonlhouse/.rvm/gems/ruby-1.9.2-p0/gems/inherited_resources-1.3.0/lib/inherited_resources/base_helpers.rb:262:in `respond_with_dual_blocks

Again its only a problem in RSpec and only on POST and PUT actions that are namespaced.  Thanks.

Have you resolved this issue?  I'm running into the exact same problem.

I am experiencing the same issue. The only work-around I have found is to simply disable the tests. Interestingly, everything seems to work on the actual site, i.e. the "edit" template does get rendered.
Yet, response.should render_template("edit") (for update) and response.should render_template("new") (for create) tests fail.


Use OpenAIDiscreteVAE as pretrained model

I notice the forward function in the OpenAIDiscreteVAE class is not implemented.  I am interested in using this model as a pretrained model and train it on a medical dataset.  What is the best approach to achieve this?

implement the forward function
Is it possible to initialize the parameters for DiscreteVAE class such that we can load the Openai discrete vae model in with load_state_dict( )?

Any suggestion is welcome!

@richcmwang hmm, I don't have any plans to do so. I think the onus would be on OpenAI to provide that

Understood.  A question on the second point: For the current DIscreteVAE code, is there a way to load OpenAIDiscreteVAE encoder/decoder in?  Or the two models have a different design so loading pretrained is not feasible?

@richcmwang there are some slight differences, so it won't be loadable, unfortunately. one thing to note is that it is coming to light that the VQVAE from Taming Transformers is actually much better than the OpenAI one (despite the latter's scale)
https://github.com/CompVis/taming-transformers Their repository should be more approachable for training your own VAE (or fine tuning your data from their checkpoints)

Thanks for pointing to the taming-transformers.   The reconstruction images contain a lot more details! Can the VQVAE pass directly into your train_dalle.py through --vae-path?

@richcmwang oh oops, never documented, but you just have to omit the --vae-path and pass in --taming to use the trained taming VQVAE
there is an issue open for using your own trained VQVAE, and i'll have to put some thought into that before writing that feature out in code

Thanks!  will give it a try!


Disable TLS verification k3d cluster

My config file:
File name: cluster.yaml
apiVersion: k3d.io/v1alpha5
kind: Simple
metadata:
name: mycluster
k3d cluster create mycluster --api-port 6445 --servers 1 --agents 2 -p "80:80@loadbalacer"
servers: 1 # same as --servers 1
#-- insecure-skip-tls-verify: true
agents: 1 # same as --agents 2, before k3d used to call them worker nodes
kubeAPI:
host: "localhost"
hostIP: "<IP_ADDRESS>"
hostPort: "6445"
image: docker.io/rancher/k3s:v1.21.5-k3s1
ports:

port: 80:80 # same as --port '80:80@loadbalancer'
nodeFilters:

loadbalancer
registries:

create: #same as --registry-create demo-registry
name: demo-registry
host: "<IP_ADDRESS>"
use:
 - k3d-demo-registry:61919

options:
k3d:
wait: true # wait for cluster to be usable ; same as --wait (default: true)
timeout: "60s" # wait timeout before aborting; same as --timeout 60s
disableLoadbalancer: false # same as --no-lb
loadbalancer:
  configOverrides:
    - settings.workerConnections=2048

kubeconfig:
updateDefaultKubeconfig: true
switchCurrentContext: true
Question:  How and where do I implement insecure-skip-tls-verify: true or --insecure-skip-tls-verify in the above configuration file e to disable TLS?

cluster create -c cluster.yaml

+1


Perceptor Tab repeats itself in release

Type of Issue
What types of issue have you found with hypertrons-crx?
Put an x in the boxes that apply

[x] Bug
[ ] Feature Request
[ ] Performance issue
[ ] Help Wanted
[ ] Documention incomplete
[ ] Test Improvement
[x] Raise a Question

Introduction ot the issue
Perceptor Tab repeats itself repeatedly while switching tabs (for release)

Probably has something to do with the Github token but this is not seen in the build version
Process

Go to any repo
Go in deep file structure or sometimes not even that
Perceptor tab repeats itself and the tab eventually crashes

Expected result
This thing does not happen at all in the dev build,
Current result
Perceptor tab repeats itself and the tab eventually crashes
Possible Fix
This probably got fixed in the later updates but I couldn't track where it was

Hi @DantrazTrev , this problem was mentioned in #145, and in next release(v1.0.1) it will be solved.

Thanks for the info @tyn1998


Do queries affect annotations?

Description
I'm trying to fuzz an API where the POST request must be completed first with some parameters like name, category, etc.
Then a GET request must be performed using those parameters as queries.
All the other requests (PUT, DELETE, etc) depend on getting the ID from the GET request.

I tried to write an annotation for getting the ID from the GET request as something like this:

{
	    "producer_endpoint": "/setup",

            "producer_method": "GET",

            "producer_resource_name": "id",

            "consumer_param": "setID"
}

But there's no change in the grammar file after compiling. I've enabled GET producers in the config file as well.

Is it possible to add the dependency of the GET request getting the query parameters from the POST?
The parameters are user generated and only present in the request body.

Hello @mohona,
I believe everything you are asking for should be possible to configure.  Just to make sure I understand the API pattern:
POST /product {name = "Name"; category = "Food"}
-> returns 200 OK?  or 201 with Location etc. ?
GET /product?name=Name&category=Food
-> returns 200 OK {"id" = 12345; name = "Name", category = "Food"...}
PUT /product/12345  <-- /product/{setID} is the path as declared in OpenAPI
Can you please confirm this is the setup?  I believe the easiest way to troubleshoot would be to have an OpenAPI spec sample to reproduce the issue.   I am attaching an example with a dependency both as in your (1) and (2).
Config:
{
  "SwaggerSpecFilePath": [
    "../swagger.json"
  ],
  "DataFuzzing": true,
  "AnnotationFilePath": "annotations.json"
}

Annotation file:
{
  "x-restler-global-annotations": [
    {
      "producer_endpoint": "/setup",
      "producer_method": "GET",
      "producer_resource_name": "id",
      "consumer_param": "setID"
    },
    {
      "producer_endpoint": "/setup",
      "producer_method": "POST",
      "producer_resource_name": "/name",
      "consumer_param": "product_name",
      "consumer_endpoint": "/setup",
      "consumer_method": "GET"
    }

  ]
}

Swagger and resulting grammar with generated dependencies attached.
If there is something different about your Swagger/OpenAPI spec that makes the annotations not take effect, please send me your version of the spec that reproduces the issue.  Or, please let me know what fixes you applied to make i
grammar.py.txt
t work (perhaps there is a doc improvement we can make to better describe such examples).
swagger.json.txt
Thanks,
Marina

Hi, these annotations really helped. I couldn't find any information in the documentation about how to retrieve a parameter from the body (eg. '/name').
However, I've run into another issue: one of the parameters needs to be sent as a string in the GET query, but it is in the form of an array in the POST request.
For example:
POST /product {name = "name", models = ["abcd", "efgh"]}
GET /product?name="name"&model="abcd"
Is there a way to configure restler to extract the values from within the array?

I tried to workaround this by manually changing the grammar to include the brackets and the request is giving a 200 status code however it's rendered invalid due to a parser failure.


[Ubuntu] Sort tool names in README

Description
Sort tool names in Alphabetical order inside each category in README of Ubuntu

Related issue:
#1665
Check list

[x] Related issue / work item is attached
[x] Documentation is updated (if applicable)
[ ] Changes are tested and related VM images are successfully generated

Browsers section is not sorted

Browsers section is not sorted

@AlenaSviridenko, It's expected. It is not sorted to keep the browsers and its associated web drivers in pairs.

@AlenaSviridenko, It's expected. It is not sorted to keep the browsers and its associated web drivers in pairs.


[SUPPORT] How to get record key and partitionfields using hudi API ？

I load a base path using HoodieTableMetaClient， but how to get key and partition feilds？
My Version： 0.7.0

@shenbinglife you can just look for the columns _hoodie_record_key and _hoodie_partition_path

@shenbinglife Closing this ticket due to inactivity. Please re-open if you need further assistance.


[5] Prerequisites - amateur formatting

This is so unprofessional guys - you've left in the guidelines on how to add in a URL on github in the documentation... not cool.

RiverCatch requires the following Python packages:
FOR URL: square brackets for title, parentheses for URL
NumPy - makes use of NumPy's statistical functions
Pandas - makes use of Panda's dataframes

Sometimes we just need things for us <3


⚠ ClamAV has degraded performance

In d35a7d6, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 69 ms

Resolved: ClamAV performance has improved in 8d3571e after 8 minutes.


v5 breaks support for functional components with hooks in accessories

🐛 Bug Report
v5 breaks support for rendering components with hooks in props such as accessoryLeft due to the implementation of: https://github.com/akveo/react-native-ui-kitten/blob/next/src/components/devsupport/components/falsyFC/falsyFC.component.tsx
FalsyFC should not make the assumption that components are plain functions. It should properly use React.createElement so hooks continue working.
To Reproduce
const MyHookComponent = () => {
        const [name, setName] = React.useState('');
        React.useEffect(() => {
              setName('test');
        });

        return <Text>{name}</Text>;
};

const MyApp = () => (
        <TopNavigation
                accessoryLeft={MyHookComponent}
                title={"test"}
          />
);

Expected behavior
For FalsyFC to render components with hooks by properly using React.createElement.
UI Kitten and Eva version

Package
Version

@eva-design/eva
2.0.0-alpha.1

@ui-kitten/components
5.0.0-alpha.1

@Photonios thanks for reporting and pointing for the possible solution.
As a workaround, you may create a function that renders a function component: https://snack.expo.io/AAtTzHId3


Cool project, we have similar ideas

Hi,
I just wanted to say that this is a cool idea. I had a similar idea a while ago and wrote something similar. Gitsentry watches repos and reports changes in Slack, which fit my usecase at the time. Here it is: https://github.com/DandyDev/gitsentry
You can close this issue, just wanted to let you know of my alternative.

Nice! Love the simplicity.
I initially wanted to set it up on a dedicated server too and do email or RSS notifications.


defaut value should be False for locked and reset fields

done here https://github.com/CounterpartyXCP/counterparty-core/pull/2041/commits/155a27256adf17d5072e96059680219b1af038c4


Support import time framework selection

Currently to use txaio we have to either call use_twisted() or use_asyncio() at the start of a python file, while it works great, it doesn't look good. flake8 doesn't like that as well since we add more imports after the use_twisted() call.
It would be good if the library had a way to select the framework on import, something along the lines
from txaio import with_twisted as txaio

yeah, I agree, would look nicer to have it hidden in the import.
to silence flake8 with the existing method:
import txaio
txaio.use_twisted()  # noqa


failed to get tv camera

[x] I have read the FAQ.
[x] I have searched in existing issues.

Environment

OS: [ Windows]
scrcpy version: [1.18]
installation method: [ Windows release]
device model:HUAWEI THAL-560
Android version: [10]

Describe the bug
In cmd console, I inputed scrcpy.exe -Nr test.mp4, it failed. And I also used scrcpy version 1.21, it returned with the same error message.
scrcpy-win64-v1.21>scrcpy.exe -Nr test.mp4
scrcpy 1.21 https://github.com/Genymobile/scrcpy
scrcpy-win64-v1.21\scrcpy-serve... file pushed, 0 skipped. 13.0 MB/s (40067 bytes in 0.003s)
[server] INFO: Device: HUAWEI THAL-560 (Android 10)
INFO: Recording started to mp4 file: test.mp4
ERROR: Recording failed to test.mp4
WARN: Device disconnected
[server] ERROR: Exception on thread Thread[main,5,main]
android.media.MediaCodec$CodecException: start failed
at android.media.MediaCodec.native_start(Native Method)
at android.media.MediaCodec.start(MediaCodec.java:2131)
at com.genymobile.scrcpy.ScreenEncoder.internalStreamScreen(ScreenEncoder.java:89)
at com.genymobile.scrcpy.ScreenEncoder.streamScreen(ScreenEncoder.java:65)
at com.genymobile.scrcpy.Server.scrcpy(Server.java:93)
at com.genymobile.scrcpy.Server.main(Server.java:309)
at com.android.internal.os.RuntimeInit.nativeFinishInit(Native Method)
at com.android.internal.os.RuntimeInit.main(RuntimeInit.java:359)

it is ok after use scrcpy -m 1920

Could you test this version please: https://github.com/Genymobile/scrcpy/pull/2947 (without any additional parameter like -m1024)
Does it work correctly? What is the full console output?


Issue while building Botsing

Characteristics

Issue Type: [bug]
Reproducibility: [always]
Severity: [major]
Tool/Service/Component: [Botsing, v1.0.4, Commit 7fad258b430d6c2c4476b6c6eb49bd9a43bd3288 (currently master HEAD)]
Execution Environment: [Ubuntu, 18.0.4]
Reporter: [Mael Audren<EMAIL_ADDRESS>
Description
I've not managed to build Botsing following the documentation.
I've followed the documentation (Build Botsing paragraph) to build Botsing and I've run the following command:
mvn install:install-file -Dfile=botsing-reproduction/evosuite-client-botsing-1.0.7.jar -DgroupId=org.evosuite -DartifactId=evosuite-client-botsing -Dversion=1.0.7 -Dpackaging=jar
The output is Erreur : impossible de trouver ou charger la classe principale org.codehaus.plexus.classworlds.launcher.Launcher
In English Error : Impossible to load main class org.codehaus.plexus.classworlds.launcher.Launcher
Steps to reproduce
1/ Clone Botsing
2/ Run mvn install:install-file -Dfile=botsing-reproduction/evosuite-client-botsing-1.0.7.jar -DgroupId=org.evosuite -DartifactId=evosuite-client-botsing -Dversion=1.0.7 -Dpackaging=jar

I've discovered the same issue for the module botsing-maven.

Hi @MaelAudren, it is working fine on my setup:
Apache Maven 3.5.2
Maven home: /usr/local/Cellar/maven/3.5.2/libexec
Java version: 1.8.0_162, vendor: Oracle Corporation
Java home: /Library/Java/JavaVirtualMachines/jdk1.8.0_162.jdk/Contents/Home/jre

Could you check your Maven installation with mvn -version?

Thanks Xavier, it seems that the issue comes from my environment and is not linked to Botsing.

I updated maven and it fixed the issue.


Add column order for workbooks

To be added as workbooks inputs:
"add_comment_column": true,
"summary": "dias",
"reorder_columns"="CHROM POS END CNVLEN ID ALT QUAL CSQ_SYMBOL CSQ_Feature CSQ_VARIANT_CLASS CSQ_EXON CSQ_INTRON CSQ_STRAND GT CN NP QA QS QSE QSS" ,
"exclude_columns"="REF FILTER CSQ_Allele CSQ_Consequence "

Also exclude IMPACT


"FasterRCNN: 'CLIPResNet is not in the models registry'"

when i use the "python -m torch.distributed.launch tools/train.py --config configs/singledataset/clip_end2end_faster_rcnn_r50_c4_1x_coco.py --work-dir output --gpus 8 --launcher pytorch" to train model on singledataset(coco) i found this error . How to solve it? i run setup.py but this action don't solve. Thanks!

same issue here
python demo/image_demo.py demo/demo.jpg \     configs/inference/clip_end2end_faster_rcnn_r50_c4_1x_lvis_v0.5.py \     ckpt/epoch_12_end2end_coco.pth

May I ask you finally solved this problem ?I also met the same problem as you.

pip install -v -e .     I solved the bug.


Pluggable Hedera Consensus Service for Hyperledger Fabric

Checklist for Hyperledger Lab Proposal PR
The Pluggable HCS Lab proves the ability of a Hyperledger Fabric network to connect to the Hedera Consensus Service running on a publicly accessible testnetwork.

@donaldthibeau : Can you follow the steps here to fix the DCI issues.

@christo4ferris : Can you confirm sponsorship?

Can these two repos be combined into one? or is the expectation that there will be two separate repos in labs:

pluggable-hcs
pluggable-hcs-samples

My preference would be a single repo with a subdirectory for samples.

I'll merge first-sample into fabric-hcs repo so we just need the pluggable-hcs repo in labs.

Think we are waiting on @christo4ferris to confirm sponsorship

@vipinsun  did you get @christo4ferris confirmation? I don't see it in the comment thread of this PR and the PR reviewers panel doesn't indicate his approval.


RSDK-8714: Input Controller wrappers

-input controller wrapper + associated tests
-added kotlinx coroutines library as a requirement in gradle for channel use in streamEvents

Putting on ice for now -- need to take a look at removing the kotlin dependency


fix: don't reopen process manager once it is closed

Description
TODO
So far the first commit just:

beefs up the tests a bit more so we can have some confidence in what we want to change
switches various tests that run a child to completion to not use the script that takes 5s. This changes the runtime for this module's unit tests from 5.14s to 0.14s

Testing Instructions
TODO
Closes TURBO-1494

Closing in favor of #6258 6258


[BUG] JSQLParser Version : 4.0 : failing parse postgresql &&

Always check against the Latest SNAPSHOT of JSQLParser and the Syntax Diagram
Failing SQL Feature:
net.sf.jsqlparser.JSQLParserException: net.sf.jsqlparser.parser.ParseException: Encountered unexpected token: "&&" "&&"
at line 1, column 26.
Was expecting one of:
"::"
"["
"^"
<EOF>
<ST_SEMICOLON>

SQL Example:

Simplified Query Example, focusing on the failing featureselect array['1'] && array['1']

Greetings.
This works well with JSQLParser 4.8 Snapshot and you can test it online here.

No further questions asked.


SymmetricEigenAnalysisImageFilter incorrect python wrapping

The python implementation of SymmetricEigenAnalysisImageFilter is returning 6 values per pixel instead of 3, and more than 3 of the 6 values is non-zero. Since there are only 3 eigenvalues for a 3D Hessian (3x3) eigenvalue decomposition, this functionality must be incorrect.
I asked about this on the forum and the comment was that it was probably wrapped incorrectly.
"Maybe the problem is in wrapping specification? The test instantiates the filter with <SSRT<3>, FixedArray<3> >, whereas the wrapping instantiates with <SSRT<3>, SSRT<3> >."
https://discourse.itk.org/t/computing-hessian-eigenvalues-in-python-how-to-interpret-output-of-symmetriceigenanalysisimagefilter/6517

Description

Steps to Reproduce

Use a test image (ideally a CT scan) with size >= 100^3
Run this code:

import itk
#Load the sample image, ideally a CT scan
hessian_image = itk.hessian_recursive_gaussian_image_filter(
            itk_image, sigma=2.0
        )
#hessian_image size = (6, 188, 217, 259)
eigen_values_image = itk.SymmetricEigenAnalysisImageFilter(hessian_image)
#eigen_values_image size = (6, 188, 217, 259)

eigen_values_array = itk.GetArrayFromImage(eigen_values_image, keep_axes=True)
print( eigen_values_array[:,100,100,100] )

Expected behavior
You should see an array printed out that has 6 values. Similar to this, but the numbers obviously will not be the same
array([ -3.0928779 , 1.86561006, 0. , -11.97670084,
6.26783045, 0. ])
Actual behavior

Reproducibility
Unknown
Versions
5.3.0
Environment
python:
4.9.18 (main, Feb 13 2024, 10:56:47)
[GCC 12.2.0]

Taking a second look, itk.SymmetricEigenAnalysisImageFilter is the object-oriented syntax. You should have a .Update(), or use snake case like for hessian_recursive_gaussian_image_filter.

Taking a second look, itk.SymmetricEigenAnalysisImageFilter is the object-oriented syntax. You should have a .Update(), or use snake case like for hessian_recursive_gaussian_image_filter.

Object oriented approach doesn't work either
   ```

eig_filter = itk.SymmetricEigenAnalysisImageFilter.New(hessian_image)
eig_filter.Update()
eig_image = eig_filter.GetOutput()

[ -0.56114415   0.56627178   0.         -28.24620925   5.85857641                                            │···························································
   0.        ]

It is a bit strange that your original code worked (due to syntax).
But I guess that problem is in the wrapping.

It is a bit strange that your original code worked (due to syntax).
But I guess that problem is in the wrapping.

Yeah, it ran without error but the values at that pixel were different.