[Windows] HotReload auto discovery sometimes doesn't work

Hi Andrei,
For some reason after a long period that everything was working great, the Hot Reload stopped working last week.
I was working with an old version 1.2.4 but I think since the extension was updated by itself things stopped working.
I tried about everything including:

updating to the newest version 1.4.10
reinstall the extension tool
Adding the adb path to the environment path
forwarding: adb forward tcp:8000 tcp:8000 after I made sure the output log said it was 8000
Changes start to run:   HotReloader.Current.Run(this);

I'm running on a device, on windows and it worked on that device for quite some time until last week.
Is there a way to go back to 1.2.4 with prior extension?
Thanks!

I think this is a fairly recent breaking change.
Everything was working fine yesterday.
This morning default ports changed from the 15X range to 17502 - not sure what's going on, but super disappointed that lately this plugin is costing more time than it is saving.
Also windows VS 2017 + android emulator

Just want to go back to how it was 1.2.4, works great and saved a lot of time.
Now I wasted two days to try to make in work again.

@AlonRom  if you want to roll back - have a look here at #73 - there's a link to an old version of the extension there (and you need to turn off auto-update for the extensions as well)

Thanks @InquisitorJax ! I will try

@AlonRom did you install 1.3.5 version of extension?

@AndreiMisiukevich yes!

@AlonRom Changes start to run: HotReloader.Current.Run(this);
but what does extension show you? what port value ?

15000

@AlonRom try to disable firewall

@AndreiMisiukevich  Where? how?
I don't think it's the best solution and can't apply it in the work routine

@AlonRom i can explain how does it work:

Extensions opens UDP receiver on that port, which it shows in alert
Reloader build url "http://{deviceIp}:{DeviceUrlPort}" and broadcast it via UDP client to that port (ExtensionAutoDiscoveryPort).
Then extension sends updates to your device/emulator by that url

Well, try to find your PC/LAPTOP ip (NOT DEVICE IP!)
and
ExtensionIpAddress = IPAddress.Parse("YOUR LAPTOP IP HERE") (e.g. <IP_ADDRESS>)
It's property of configuration
It will force to send device's url directly to your PC instead of broadcasting.

@AlonRom do you work on Windows?

@AndreiMisiukevich yes

@AlonRom just had a call with @InquisitorJax
Sooo, it's strange thing, but could you please try:

Enable extension (it will show you a port value)
put this port value to config (if it isn't 15000 for sure, if it is, you may skip this step)
run your application
after running the application make this (find that message and make adb forward yourself)

@AlonRom just in case you may try to
adb forward tcp:8000 tcp:8000
adb forward tcp:8001 tcp:8001
adb forward tcp:8002 tcp:8002
adb forward tcp:8003 tcp:8003

@AlonRom does it help?
Okay, anyway @stanbav  and me decided to release old extensions as separate extension. So, everybody will be able to decide which one to use.
Does it sound good?

@AndreiMisiukevich everything you mentioned above I already did with no success.
But why use ports 8001,2,3?
When will you provide the option to go back to older extensions?

@AlonRom today i think

but i suggest you to try on REAL android device:
            HotReloader.Current.Run(this, new HotReloader.Configuration
            {
                DeviceUrlPort = 4981, //or other values
                ExtensionAutoDiscoveryPort = {value from alert},
                ExtensionIpAddress = IPAddress.Parse({your PC ip here}
            });

@AndreiMisiukevich
I'm using a real device all the time, the same one I used when it worked:

DevicePort - I put 8000 as the output suggested:

HOTRELOAD STARTED ON DEVICE's PORT: 8000

ExtensionAutoDiscoveryPort  - I put 15000 as the alert window suggested.

@AlonRom tell me please your android device IP

<IP_ADDRESS>

@AlonRom ... https://www.speedguide.net/ip/<IP_ADDRESS>
it's not LOCAL ip address. Your device and your PC/laptop must be at the same local network

Now changed to:<IP_ADDRESS>
Just tell me where to look and what to do and I will provide the details.

Now changed to:<IP_ADDRESS>
Just tell me where to look and what to do and I will provide the details.

<IP_ADDRESS>
<IP_ADDRESS>

is it possible to put device and your pc to the same addresses range
for example to 172.18.8.{value} ?

This was it!!! working:)

@AndreiMisiukevich is this something users need to be aware of and fix themselves - or can this be remedied with a hotreload update?

@InquisitorJax not sure that there is possibility to change device's IP
So, i think it makes sense to put more info in Readme. Do you want to send PR with info :) . ?

It has stopped working on VS 16.1.3 as well.  HotReload extension 1.3.5 and Nuget package 1.4.10
My VS port is 15000, app.cs has following after initialize()
#if DEBUG
HotReloader.Current.Run(this);
//optionaly you may specify EXTENSION's port for auto discovery and device's port
#endif
This extension has been very unstable, ever since the auto detect IP feature (BTW which is great) was launched.

@IoTFier look through the discussion. We have already resolved this question.
also you may use old extensions with newest nuget package https://github.com/AndreiMisiukevich/HotReload/tree/master/old_extension_packages

@AndreiMisiukevich after our discussion yesterday, it stopped working again, and resetting the adb port forwarding didn't help.
I'll try again later (and also the old extensions) to see if anything changes.

@AndreiMisiukevich , thank you for your response.  I use iOS simulator and Android device.
I did read the discussion and will definitely read it again, if I missed any step. My VS extn is configured to update automatically.
The point I was making earlier is - new updates have taken toll on productive time when using this extension. New improvements are intended for great purpose, but they have destabilized the core functionality. I will feel, we should at least know the IPs it is connecting to.
Regarding port # besides 15000; do we have to change the code every-time to provide the port number if it is not 15000 ?
Thank you

@IoTFier I had the same feedback ... before setting up IP numbers was a once-off. With the new changes - that port 15X could change every time you connect - which means changing code - which doesn't really help in a team setting.
I prefer the old extension - but that's disappointing, because that means it's not going to be getting new features.

@IoTFier are you sure that Windows extension ever works with iOS simulators at all?
As for Android device
Can you confirm that your device IP and your PC ip are in the same IP range ?

@InquisitorJax - I am not sure if it make sense to go backward by losing features and functionality for getting stability.  I truly thank @AndreiMisiukevich & co to provide us with this great help. My request is to not compromise on basic usability of this tools when launching / adding new features / bug-fixes.

@IoTFier old extension is stable. It just doesn't have autodiscovery feature.

@AndreiMisiukevich perhaps consider keeping the old IP dialog - but add a checkbox that enables auto-IP discovery? That way if there's a problem with auto-discover, it's a lot easier to uncheck that feature, rather than installing older extensions?

@IoTFier are you sure that Windows extension ever works with iOS simulators at all?
As for Android device
Can you confirm that your device IP and your PC ip are in the same IP range ?
Yes, it was working before this port # feature was brought in. Ever since that I have lost the functionality... Let me put it this way .. it is hit and miss

My android device IP and PC ip are in the same range.
I use HotReload more on the iOS simulator than Android.

@IoTFier old extension is stable. It just doesn't have autodiscovery feature.
@AndreiMisiukevich - It is certainty a great planned feature;
My request is to display those connected IPs, and OS version , and device / simulator name (old extn++? )

@AndreiMisiukevich perhaps consider keeping the old IP dialog - but add a checkbox that enables auto-IP discovery? That way if there's a problem with auto-discover, it's a lot easier to uncheck that feature, rather than installing older extensions?

need to discuss with @stanbav

@IoTFier have you tried this as well? (put your PC ip there)

@InquisitorJax as for entering Extension port. We can easily solve this problem. Reloader may send its ip to range of possible IP, so you won't need to copy-paste anything

1.4.12 nuget package
new extensions will be available soon. Read readme again please
@InquisitorJax no need to fill extension's port anymore. Just ignore alert message :)

Hi @AndreiMisiukevich I have latest extension and nuget package, tried above suggested step by you.
No luck. It is still not working.. Can you please keep this issue open?
Thanks

@IoTFier reach me in skype please csi.andreymisyukevich

@IoTFier case is WINDOWS + iOS simulator :)

thank you @AndreiMisiukevich
I am seeing issue on both iOS simulator and Android emulator as well.
My Android emulator has IP address in a different subnet, even though android emulator is running off my PC.
Also, searching of "$"### HOTRELOAD STARTED ON DEVICE's PORT:" in VS output resulting in nothing. Am I missing any step in the android config for the HOTRELOADER to not to get fired off when app.xaml.cs is loaded ?
With old extn I could enter IP address of my iOS simulator and Android device; and it worked fine. I had never tried Android simulator. That's my next move until the latest ext 1.3.6 is ready to use.
Thank you for your help.

@AndreiMisiukevich it seems that old extn 1.2.1 is not compatible with VS 2019 when I tried to install it...
Is that right ? How do I configure / install it for VS 2019?

No idea, should work. Try to restart VS, after removing

I have no instance of VS 2017/2019 running.
FYI..

Figured out... just disabling the extn for installing old one doesn't work. We need to uninstall the new extension before the old one recognizes a need to install it on that VS. Not sure if it make sense fully... VS should provide an option for choosing the version # it is already installed ...

@AndreiMisiukevich ,
the old extension works like a charm !

@IoTFier i think, nice to hear it. Do you use <IP_ADDRESS> for iOS simulator reloading?

I managed to run the hotreload with the latest version in emulators, so it was necessary to map the folder of adb.exe, close the visual studio and reopen, enabling "Enable it" the hotreload.
The settings I left were these in App.Xaml.cs.
HotReloader.Current.Run(this);

I managed to run the hotreload with the latest version in emulators, so it was necessary to map the folder of adb.exe, close the visual studio and reopen, enabling "Enable it" the hotreload.
The settings I left were these in App.Xaml.cs.
HotReloader.Current.Run(this);

Yeah :) as described in documentation :)

I managed to run the hotreload with the latest version in emulators, so it was necessary to map the folder of adb.exe, close the visual studio and reopen, enabling "Enable it" the hotreload.
The settings I left were these in App.Xaml.cs.
HotReloader.Current.Run(this);

Yeah :) as described in documentation :)

However for it to work according to the instructions it is necessary to close and reopen Visual Studio otherwise it does not work

@IoTFier what IP do you use for iOS simulator on Windows?

@IoTFier , sorry for my delay.. trying to get you the info... but the meantime the extn got upgraded ., I did uncheck the box of auto upgrade.. reinstalling it will let you know

@AndreiMisiukevich - my apologies for the delay again. The IP Address of my iOS Simulator is same as the IP address of my Mac machine.
The issue I was facing about text was .. .just unchecking the upgrade of the extn wasn't enough. Along with that it is also necessary to disable that (uncheck) under tools -> options -> environment -> extensions .. see the pic below..
Otherwise, if you were to open another VS instance, it keeps upgrading the extension from 1.2.1 to the latest one

@IoTFier so, let's imagine that your PC ip is <IP_ADDRESS>
You enters http://<IP_ADDRESS>:8000 to extension box and it works well, right?

@AndreiMisiukevich - yes, that's what seems to be correct.

the port get changed for me when Hotrelaod is connected, but I close the solution and open a new one (now 15001). On closing a solution Hotreload should disconnect automatically.
Maybe this helps others with connection issues.

This is working great for me in Android emulator - thanks!
In iOS simulator (running in Mojave VM) it takes about 45 seconds for any changes to appear. Any ideas?

@Tyri0n wow. Weird. Do you work on virtual machine?
anyway try to disable your firewall

I code in VS 2019 under Windows 10. Mojave runs in VM on same Windows host. Mojave firewall is off.
For some reason it seems a bit better now. Sometimes saving a xaml change is causing the app to jump to a different page and the change is visible when navigating back.
I suspect it's a simulator issue - I usually have to double tap buttons to change page whereas it works fine on device.
Anyways, thanks for the quick response.

can be closed i think


Reload newly subscribed stream

When a user is not subscribed to a stream, they do not receive live updates for events in that stream (e.g., new messages).
Thus, if a user subscribes to a stream while their message view includes that stream (e.g., by clicking the "Subscribe" button at the bottom of an unsubscribed stream view), we should immediately refresh the view to avoid confusion due to missing messages.

@alya Hey Alya, I am a student at Carnegie Mellon University that is starting a project where me and my group have to contribute to an open source project. We aren't all the most experienced programmers but I think we can complete a few good-first-issue tasks over the next 2-3 weeks or so.
This is a large repository with a lot of issues and contributors so I figured I would reach out to somebody, do you have any ideas for good issues to contribute to that we can take over? Sorry for commenting on a random issue, wasn't sure how else to ask a question. This issue does look interesting though, we might be able to fix it!

@pkaufhol Welcome! Please start by reading the contributor guide; it has advice on how to pick an issue, as well as how to ask for help.

@zulipbot claim

Hi @alya !
I implemented a page reload after clicking the "Subscribe" botton. However, this will cause the "Unsubsribe" botton useless as it seems not appearing anywhere else.

@zulipbot claim

@Beanpen Good question! I think it's OK to drop the "Unsubscribe" button; if you accidentally subscribe to a stream, it's not too hard to undo in other ways... Since the button will never appear, we should probably remove the code for it.

However, if it's somehow possible to keep the button without too much gymnastics, that would be preferable.

Hi @alya !
Thanks for the reply! I agree with you to drop the "Unsubsribe" button on the page after clicking on the "Subscribe" botton as other subscribed message pages also don't have the "Unsubsribe" button shown. This revision is made in #23717. I accidentally submitted two fixes in one PR. I will split them into two PR to avoid confusion.
In addition, I tested a couple scenarios to see whether the "Unsubsribe" button will appear after making this change. I found one situation when it will appear. To reproduce it, we need to open the message page for that stream, and "Subscribe" and "Unsubscribe" using this interface. Then, go back to the message page and the "Ubsubscribe" botton willl appear here.

@zulipbot claim

@Beanpen sorry for reclaiming it. I'll unassign myself since you have a working solution. I was also thinking wouldn't it be better to refresh the view instead of the whole page? I was thinking you could maybe do this with the rerender_view function in message_list.js.

@zulipbot the link is invalid. Could you send it again?

@zulipbot claim

@zulipbot claim

@zulipbot unclaim

@zulipbot claim


Fix tests

what

Fix tests

why

Unblock community PRs

/terratest

@kevcube I am still working on it.

/terratest

/terratest

/terratest

/terratest

/terratest

/terratest

/terratest

/terratest

/terratest

/terratest

/terratest

/terratest


fzf preview not works

preview works well on fzf's built-in function, what debug info do you need to help

I also encountered this issue.  I think it has to do with :
https://github.com/ojroques/nvim-lspfuzzy/blob/7896597f54e0f7b90acda26468f5bbe97544b330/lua/lspfuzzy.lua#L82-L87
could try let g:loaded_fzf_vim = 1  in init.vim as a workaround.

Hello!
Yes I've very recently (yesterday) changed how fzf.vim is detected because fzf.vim has just added the global variable g:loaded_fzf_vim` see https://github.com/junegunn/fzf.vim/commit/636a62f140181f80c8e7460a76ae6a5d2c5d97b2. So make sure to update fzf.vim and it should work.

@wgurecky @ojroques
let g:loaded_fzf_vim = 1 or update fzf.vim both works, thanks, closed.


Missing resize_text_pos_embed in clip_model.py

Hello,
It seems that the method resize_text_pos_embed is missing in your CLIP model. Can you provide it, please ?
Or maybe it's some dead code left because we never enter in this condition?
Thanks again for your time and your hard work for this paper.
Mathias.

resize_text_pos_embed is a method to adjust the CLIP model text embedding, it is only called when you need to change the default 77 maximum text sequence length, in the IRRA framework, we do not change the maximum text sequence length, so this method will not be called.

Thanks!


[WIP] Add nodesets from modulesync_config

The intent is that we can use these in modules rather than to modulesync them all the time.

I'm thinking of releasing this as a separate gem so we can still only list this gem in the release group.

Going to build something based on beaker-hostgenerator instead


Use @brandFontName for brand icon's font-family CSS rule

Bug Report
The @brandFontName vatiable is not consumed in icon.overrides as it should be.
Steps
I have an open PR with the fix.
Expected Result
If I want to use different icon libraries aside from those offered by SUI, overriding the @brandFontName variable should render the icons that I want.
Actual Result
Overriding the @brandFontName variable does not render my third-party icons.

Sorry, but we don't handle CSS issues there, you should move it to the main repo: https://github.com/Semantic-Org/Semantic-UI
As I see, it also fixed in Fomantic UI: https://github.com/fomantic/Fomantic-UI/blob/7128a9324a835796d625b77507a7c8881ba913a9/src/themes/default/elements/icon.overrides#L1888


Zend_Http_Client-Adapter_Socket - check transfer-encoding header is a string (and not an array)

I saw this error reported in some server logs -
PHP Fatal error:  Uncaught TypeError: strtolower(): Argument #1 ($string) must be of type string, array given in ... /Zend/Http/Client/Adapter/Socket.php:362
Most http headers can be a single stringy value or an array of strings ... I'm not sure what the correct behaviour should be if there is more than one transfer-encoding (it feels like it's an error on the server my code connected to?).
Anyway, after this, it'll just throw a Zend_Http_Client_Adapter_Exception instead - which feels like the only option?

Thanks for merging!


docs: remove obsolete samples

Will wait until we've got absolute confirmation on this before merging.


Parametrise target

In certain runtimes (AWS Lambda in my case), we are quite restricted on where we can write files. When we extract the compressed binaries and write them to the filesystem, they need to be written under /tmp in Lambda, as that's the only writeable filesystem.
So, here I'm parametrising the target path.

Sorry this stacked incorrectly with the other PR, abandoning and resubmitting.

@ross-mcnairn-dev, merged your ARM branch. Thanks for that and sorry for the delay.

@ross-mcnairn-dev, merged your ARM branch. Thanks for that and sorry for the delay.

Hey, no worries at all! Happy to be able to make a small contribution here :). I'll clean this one up and open it tonight.

OK, @JSv4 no rush on this, but this is ready for review when you've got a few this week

Hey @JSv4 , do you think you'd have time to review this this week?

Thanks for the reminder @ross-mcnairn-dev

This is great @ross-mcnairn-dev @JSv4


Error in user.service.ts

(TS) Type 'Observable<boolean>' is not assignable to type 'Observable<UserRegistration>'.

Type 'boolean' is not assignable to type 'UserRegistration'.
(tsconfig or jsconfig project)
\src\app\shared\services\user.service.ts	42	Active

You should be using HttpClient and not Http anyway, so calling .json() is not necessary.


Fix: ToASM Performance Issues

Move away from string concat and to string builder.

Codecov Report

Merging #99 (71b68a0) into master (18b76ae) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #99      +/-   ##
==========================================
+ Coverage   83.75%   83.76%   +0.01%
==========================================
  Files          35       35
  Lines        3668     3671       +3
==========================================
+ Hits         3072     3075       +3
  Misses        428      428
  Partials      168      168

Impacted Files
Coverage Δ

bscript/script.go
62.33% <100.00%> (+0.51%)
:arrow_up:

Continue to review full report at Codecov.

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


Support more distinct syntaxes

Fixes #1654.
Support more distinct syntaxes, which are as follows:
SELECT DISTINCT(col1) FROM tbl_name
SELECT COUNT(DISTINCT col1 + col2) FROM tbl_name
SELECT AVG(DISTINCT col1) FROM tbl_name
SELECT COUNT(DISTINCT col1), SUM(DISTINCT col1) FROM tbl_name
SELECT col1, COUNT(DISTINCT col2) FROM tbl_name GROUP BY col1

Pull Request Test Coverage Report for Build 6109

19 of 38 (50.0%)  changed or added relevant lines in 6 files are covered.
13 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.07%) to 73.545%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

sharding-core/src/main/java/io/shardingsphere/core/rewrite/placeholder/AggregationDistinctPlaceholder.java
0
2
0.0%

sharding-core/src/main/java/io/shardingsphere/core/rewrite/SQLRewriteEngine.java
12
19
63.16%

sharding-core/src/main/java/io/shardingsphere/core/executor/sql/execute/result/AggregationDistinctQueryResult.java
0
10
0.0%

Files with Coverage Reduction
New Missed Lines
%

sharding-core/src/main/java/io/shardingsphere/core/parsing/parser/clause/SelectListClauseParser.java
13
84.47%

Totals

Change from base Build 6108:
-0.07%

Covered Lines:
9555

Relevant Lines:
12992

💛  - Coveralls


Cleanup join memory accounting in CrossJoin and NestedLoopsJoin

Which issue does this PR close?

Closes #.
Rationale for this change

Follow on to #6170
What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?

cc @korowa

Thank you for the review @korowa !

Thank you everyone -- I will merge this PR and make a follow on with some proposed renamings


Debugging issue with

Does this occur consistently?
Repro steps:

Action: appService.StartRemoteDebug
Error type: ECONNRESET
Error Message: socket hang up
Version: 0.18.0
OS: win32
OS Release: 10.0.18362
Product: Visual Studio Code
Product Version: 1.47.3
Language: en

Call Stack
new RequestError extension.bundle.js:225:1107125
Request.callback extension.bundle.js:225:1105766
Request.RP$callback [as _callback] extension.bundle.js:225:1105212
r._callback.r.callback.r.callback extension.bundle.js:201:4468
Request.emit events.js:203:13
Request.onRequestError extension.bundle.js:201:13754
ClientRequest.emit events.js:208:15
TLSSocket.socketOnEnd _http_client.js:433:9
TLSSocket.emit events.js:208:15
endReadableNT _stream_readable.js:1168:12
processTicksAndRejections task_queues.js:77:11

Please provide repro steps


EKF2_RNG_A_HMAX, EKF2_RNG_A_VMAX missing units in parameter meta data

https://discuss.px4.io/t/units-m-of-ft-for-range-aid-params/11464

@priseborough The units for EKF2_RNG_A_HMAX and EKF2_RNG_A_VMAX is just metres right?


store/proxy: Deduplicate chunks on StoreAPI level. Recommend chunk sorting for StoreAPI + Optimized iter chunk dedup. (#2710)

Cherry picked: https://github.com/thanos-io/thanos/pull/2710

Deduplicate chunk dups on proxy StoreAPI level. Recommend chunk sorting for StoreAPI.

Also: Merge same series together on proxy level instead select. This allows better dedup efficiency.
Partially fixes: https://github.com/thanos-io/thanos/issues/2303
Cases like overlapped data from store and sidecar and 1:1 duplicates are optimized as soon as it's possible.
This case was highly visible on GitLab repro data and exists in most of Thanos setup.
Signed-off-by: Bartlomiej Plotka<EMAIL_ADDRESS>
Optimized algorithm to combine series only on start.

Signed-off-by: Bartlomiej Plotka<EMAIL_ADDRESS>
Optimized chunk comparision for overlaps.

Signed-off-by: Bartlomiej Plotka<EMAIL_ADDRESS>
Optimized deduplication for deduplicated chunk on query level as well.

Never use proto .String() in fast path!
Signed-off-by: Bartlomiej Plotka<EMAIL_ADDRESS>

We seen huge improvement on our setup:

We seen huge improvement on our setup:

What does up_custom_query_last_duration tracks?

Basically it runs heavy analytical queries against any Prometheus API endpoint.


Removing modal from dom removes all instances

I am using selectize.js inside modal and initialize it inside modal. After closing modal just because all elements removed instance of selectize also is removed. So it s not working.
https://selectize.github.io/selectize.js/

Hi @jamilalisgandarov can you please provide an example of the issue you're having so we can help debug it? I believe I understand, but your exact setup might be different to my own approach. Once you're able to provide some more context, and code examples, we might be able to help you figure out a solution if you haven't already.

Closing this due to age, and lack of context and examples to the issue.


NAME IT

What's it's grown-up name?

migurski: Innobasics™

Mirguski: thebasics
Cyd: DialTone

cc @danhon @Jaime-Alexis

GovBuilder
Civic Builder
Civic CMS

I really, really don't want the word "civic" in it. I feel like that's a super saturated word.
I also feel like it would be good for the overall DFD branding if it felt like it was related to, or rather, inside of the overall DFD practice.

Civic Platform
On Fri, Feb 13, 2015 at 10:49 AM, Jaime-Alexis<EMAIL_ADDRESS>wrote:

GovBuilder
Civic Builder
Civic CMS
—
Reply to this email directly or view it on GitHub
https://github.com/codeforamerica/ceviche-cms/issues/42#issuecomment-74305006
.

--
Help local governments deliver better services to communities. Donate now.
https://secure.codeforamerica.org/page/contribute/default?source_codes=staff
Meghan Reilly
CFO & CAO, Code for America
c:<PHONE_NUMBER>
@mfreilly

City Platform
City Platform Services
Clearly I like Platform ;)
On Fri, Feb 13, 2015 at 10:50 AM, Frances Berriman<EMAIL_ADDRESS>
wrote:

I really, really don't want the word "civic" in it. I feel like that's a
super saturated word.
I also feel like it would be good for the overall DFD branding if it felt
like it was related to, or rather, inside of the overall DFD practice.
—
Reply to this email directly or view it on GitHub
https://github.com/codeforamerica/ceviche-cms/issues/42#issuecomment-74305184
.

--
Help local governments deliver better services to communities. Donate now.
https://secure.codeforamerica.org/page/contribute/default?source_codes=staff
Meghan Reilly
CFO & CAO, Code for America
c:<PHONE_NUMBER>
@mfreilly

If we're only talking about the Content Management System, I would consider
GovCMS, GovContent+, GovWeb
I love the idea of Civic Web Stack -- as it's clear and people get it. DFD
for folks I think takes some time to understand, but it's already out
there. But, I also hear what Frances is saying.
On Fri, Feb 13, 2015 at 12:49 PM, Jaime-Alexis<EMAIL_ADDRESS>wrote:

GovBuilder
Civic Builder
Civic CMS
—
Reply to this email directly or view it on GitHub
https://github.com/codeforamerica/ceviche-cms/issues/42#issuecomment-74305006
.

--
Luke D. Norris | Director, Government Relations | Code for America
http://codeforamerica.org/ | c:<PHONE_NUMBER> | @norrisluke

@phae understood. What would help signal that it's for municipal govs? Or help is show up if someone is searching an oversaturated word like 'civic'?

Tributary
OneVoice
Public People Power

@Jaime-Alexis the language within the DFD project signals that it's for municipal govs. It could also easily come from a snappy tagline that goes beneath. Just for example (this, but not this, but like this): "thebasics: a simple CMS for municipal governments", you know?

Quick search on Google Trends

@phae for sure. Only flag..."simple" and "basic" may signal 'it's not important enough or robust enough to do what we want...'? Something we could test though and get some perceptions around.

Yes, that was an example. They may be the wrong words, I just meant that we want to have a fun, unique name that ties it to DFD and rely on a good, snappy, slogan/tagline to satisfy the understanding and search-ability, such as these real examples:
Perch: A really little content management system
Slack: A platform for team communication: everything in one place, instantly searchable, available wherever you go.
Wordpress: The best place for your personal blog or business site.
etc. etc.

They named HAL by letter shifting IBM – we could letter shift CMS and get BLR (Blur?) or DNT (Dent?)

Migurki: Lintel (as in, supporting structures around [digital] front doors)

Chime (as in a digital front doorbell)

I really like Chime. I want it now. How much??

+1 on Chime.

Comes with a theme song! https://www.youtube.com/watch?v=dDdq5_ZzBRw

agreed! chime is great.

+1 on Chime

:dancers:

Btw, if we want, we could do some market testing a bit with a few names...just a thought.

The next port of call for this ticket is a conversation with Dan and Debs.
We may well have something to test after that - most specifically
short-copy snippet - I'll open new tickets as appropriate and ask for help
on those. To stop the sprawl, I think we'll consider this closed for now
unless we hit a problem with it. Thank you everyone. :)


Error when initializing datetimepicker with defaultDate and inline:true

Hi there,
When initializing datetimepicker with defaultDate and inline:true, it generates the following error:
"Uncaught TypeError: Cannot read property 'trim' of undefined" at "bootstrap-datetimepicker.js:1492".
Probably because it attempts to test the value of input while it does not have an input in inline-mode.
Best regards,
Christophe

Hi @christophe - this is the repository for Moment.js, not a a date/time picker. Perhaps you meant to post this here?

Yes, indeed. Sorry for the mix-up.


🙉Commercial Usage of okhttp

Can we use OkHTTP for commercial purpose.
Our start up is using okhttp java version.

Yes.  Without restrictions.
See https://github.com/square/okhttp/blob/master/LICENSE.txt


Include yarn.lock in distributed files

Hey 👋
Would it be possible to add yarn.lock to the files in package.json?
In a world where supply chain security becomes more prevalent, it would be nice to be able to track exactly what the action was built on. For example, if I wanted to know what libraries were used in release v1.4.0 I'd have to look at the history of package.json in order to know what content was used to create this release.
This happens because, the way this action is released (create separate branch, move only some of the files) makes it difficult to trace back what the release was built on.
If yarn.lock was included in the files in package.json, and it would be included in the distributed files (tar.gz) and therefore, it would be possible to take a tag (e.g. v1.4.0), look at the yarn.lock and know what exact versions of libraries were used.
I'm happy to create a PR if you agree.
Thanks

I know this is not a tamper-proof solution, but at least it gives me some way of knowing what was included. Looking at the actions surely works, but I'm looking for an automated solution that gives me most of what I want with minimal effort :)
In particular, I'm not looking to ensure nobody has fiddled with the release script and thereby included a malicious dependency. And I am aware that tags could be updated to point to something completely different. What I am looking for is knowing what version of what is included such that I can tell if there is a vulnerability in this action.

I consider the current process simple and secure and I'm not willing to spend time working on or reviewing something to change this. I'm thus closing this.


GL Container Images: Replace image in build pipeline

Replace image in build pipeline
Currently, the build pipeline is based on Debian testing slim. As soon as the newly created gardenlinux-base container image is published to public registries, we may replace the images within the build pipeline by this one. Additionally, it should fix the make slim Makefile target.

This issue depends on #1390 and can only be proceeded afterwards.

todos left for this:

change need slim script to use appropriate Garden Linux container
update all container/*/Dockerfile to add debian testing apt repo if required
remove container that we do not need anymore for builds with new builder (e.g. build-image)


[Feature] Don't eagerly load the data that is only required in inactive tabs

Currently, queries that provide data to widgets in inactive tabs are also loaded on page load. However, this data is not actually needed until that tab is selected. This can be a good area of optimizing the number of queries that run on page load.
More reasoning behind this (and an implementation using existing Appsmith features): https://gist.github.com/tazmaniax/584447ab527e1198d3fbc72ac70d5b7e (full credits to the amazing @tazmaniax 😄)

In my case the tabs are not loaded on page load but when a customer is selected and it relies on local storage to capture what customer data is currently displayed in what tab. The issue is the local storage is retained across page loads and this means if the same user is loaded prior to the page load the tabs won't lazy load the tab data because it thinks that customer data is already loaded. Need a way to clear this specific local storage state on page load. Could this be an onPageLoad trigger handler to keep it super generic and applicable for other use cases?

@sharat87 in the end I fixed my issue with the local storage by clearing the tab customer state variables in question when a customer is selected as this is the natural point when everything needs to be reloaded anyway. So for this scenario no extra support for clearing local storage at page loaded is needed. I've updated the gist to include the additional logic again to help anyone in a similar position, https://gist.github.com/tazmaniax/584447ab527e1198d3fbc72ac70d5b7e. Thanks for all your help!

@tazmaniax thank you for the detailed description! Queries in inactive tabs are loaded by default but this can be changed by simply marking the query onPageLoad as false in the settings section. We do run these queries because it is possible that another section of the app is expecting to use the data in an inactive tab since inactivity only represents a selection state. I think the correct solution for these is to not persist the storeValue and flip the onPageLoad for these Queries as false as you've mentioned in your gist. Adding a JS editor will also make writing this logic easier. I guess I want to close this issue because by default appsmith is designed to show as much data as quickly as possible but if developers don't want this, they are free to turn it off.


R1.22 review: qualitative assessment

Minor improving updates and incorporation of the R1.22 review.

@renelabounek can you upload in this thread a PNG output -- that would help me review your changes. Thx

Old version:

Reviewed version:


asyncio error messages after CancelledError with Python 3.5 / Ubuntu 16.04

Our environment

Ubuntu 16.04
ROS 2 Bouncy
colcon version-check

colcon-argcomplete 0.2.2: newer version available (0.2.3)
colcon-bash 0.3.1: up-to-date
colcon-cmake 0.2.4: up-to-date
colcon-core 0.3.13: newer version available (0.3.14)
colcon-defaults 0.2.1: up-to-date
colcon-devtools 0.2.1: up-to-date
colcon-lcov-result 0.2.0: up-to-date
colcon-library-path 0.2.0: up-to-date
colcon-metadata 0.2.1: up-to-date
colcon-mixin 0.1.0: newer version available (0.1.1)
colcon-notification 0.2.5: up-to-date
colcon-output 0.2.3: up-to-date
colcon-package-information 0.2.1: up-to-date
colcon-package-selection 0.2.1: up-to-date
colcon-parallel-executor 0.2.2: up-to-date
colcon-pkg-config 0.1.0: up-to-date
colcon-powershell 0.3.3: up-to-date
colcon-python-setup-py 0.2.1: up-to-date
colcon-recursive-crawl 0.2.0: up-to-date
colcon-ros 0.3.5: up-to-date
colcon-test-result 0.3.0: up-to-date
colcon-zsh 0.3.1: up-to-date

build command: colcon build --merge-install

After we had a code error like this:
error: ‘m_pcl_sub’ was not declared in this scope
     const auto & loaned_message = m_pcl_sub->take();

colcon printed out a whole bunch of errors:
Starting >>> euclidean_cluster
--- stderr: apex_tf
/home/user/1816/apex_ws/src/apex_app/common/apex_tf/src/local_tf.cpp: In member function ‘virtual void apex_app::common::apex_tf::LocalTf::TfSubTask::task_function()’:
/home/user/1816/apex_ws/src/apex_app/common/apex_tf/src/local_tf.cpp:142:35: error: ‘m_pcl_sub’ was not declared in this scope
     const auto & loaned_message = m_pcl_sub->take();
                                   ^
make[2]: *** [CMakeFiles/apex_tf.dir/src/local_tf.cpp.o] Error 1
make[1]: *** [CMakeFiles/apex_tf.dir/all] Error 2
make: *** [all] Error 2
---
Failed   <<< apex_tf	[ Exited with code 2 ]
Finished <<< lifecycle [9.53s]
Aborted  <<< velodyne_driver
Aborted  <<< euclidean_cluster
Aborted  <<< lidar_integration
Aborted  <<< motion_model
Aborted  <<< performance_test
Aborted  <<< udp_driver

Summary: 169 packages finished [1min 52s]
  1 package failed: apex_tf
  6 packages aborted: euclidean_cluster lidar_integration motion_model performance_test udp_driver velodyne_driver
  1 package had stderr output: apex_tf
  25 packages not processed
[116.165s] ERROR:concurrent.futures:exception calling callback for <Future at 0x7f7aeed4a9e8 state=finished returned NoneType>
Traceback (most recent call last):
  File "/usr/lib/python3.5/concurrent/futures/_base.py", line 297, in _invoke_callbacks
    callback(self)
  File "/usr/lib/python3.5/asyncio/futures.py", line 442, in _call_set_state
    dest_loop.call_soon_threadsafe(_set_state, destination, source)
  File "/usr/lib/python3.5/asyncio/base_events.py", line 532, in call_soon_threadsafe
    handle = self._call_soon(callback, args)
  File "/usr/lib/python3.5/asyncio/base_events.py", line 506, in _call_soon
    self._check_closed()
  File "/usr/lib/python3.5/asyncio/base_events.py", line 334, in _check_closed
    raise RuntimeError('Event loop is closed')
RuntimeError: Event loop is closed
[116.166s] ERROR:concurrent.futures:exception calling callback for <Future at 0x7f7aeed4a128 state=finished returned NoneType>
Traceback (most recent call last):
  File "/usr/lib/python3.5/concurrent/futures/_base.py", line 297, in _invoke_callbacks
    callback(self)
  File "/usr/lib/python3.5/asyncio/futures.py", line 442, in _call_set_state
    dest_loop.call_soon_threadsafe(_set_state, destination, source)
  File "/usr/lib/python3.5/asyncio/base_events.py", line 532, in call_soon_threadsafe
    handle = self._call_soon(callback, args)
  File "/usr/lib/python3.5/asyncio/base_events.py", line 506, in _call_soon
    self._check_closed()
  File "/usr/lib/python3.5/asyncio/base_events.py", line 334, in _check_closed
    raise RuntimeError('Event loop is closed')
RuntimeError: Event loop is closed
[118.025s] ERROR:concurrent.futures:exception calling callback for <Future at 0x7f7aeee97198 state=finished returned NoneType>
Traceback (most recent call last):
  File "/usr/lib/python3.5/concurrent/futures/_base.py", line 297, in _invoke_callbacks
    callback(self)
  File "/usr/lib/python3.5/asyncio/futures.py", line 442, in _call_set_state
    dest_loop.call_soon_threadsafe(_set_state, destination, source)
  File "/usr/lib/python3.5/asyncio/base_events.py", line 532, in call_soon_threadsafe
    handle = self._call_soon(callback, args)
  File "/usr/lib/python3.5/asyncio/base_events.py", line 506, in _call_soon
    self._check_closed()
  File "/usr/lib/python3.5/asyncio/base_events.py", line 334, in _check_closed
    raise RuntimeError('Event loop is closed')
RuntimeError: Event loop is closed
[118.026s] ERROR:concurrent.futures:exception calling callback for <Future at 0x7f7aeed1bfd0 state=finished returned NoneType>
Traceback (most recent call last):
  File "/usr/lib/python3.5/concurrent/futures/_base.py", line 297, in _invoke_callbacks
    callback(self)
  File "/usr/lib/python3.5/asyncio/futures.py", line 442, in _call_set_state
    dest_loop.call_soon_threadsafe(_set_state, destination, source)
  File "/usr/lib/python3.5/asyncio/base_events.py", line 532, in call_soon_threadsafe
    handle = self._call_soon(callback, args)
  File "/usr/lib/python3.5/asyncio/base_events.py", line 506, in _call_soon
    self._check_closed()
  File "/usr/lib/python3.5/asyncio/base_events.py", line 334, in _check_closed
    raise RuntimeError('Event loop is closed')
RuntimeError: Event loop is closed
[118.026s] ERROR:concurrent.futures:exception calling callback for <Future at 0x7f7aeeb2fbe0 state=finished returned NoneType>
Traceback (most recent call last):
  File "/usr/lib/python3.5/concurrent/futures/_base.py", line 297, in _invoke_callbacks
    callback(self)
  File "/usr/lib/python3.5/asyncio/futures.py", line 442, in _call_set_state
    dest_loop.call_soon_threadsafe(_set_state, destination, source)
  File "/usr/lib/python3.5/asyncio/base_events.py", line 532, in call_soon_threadsafe
    handle = self._call_soon(callback, args)
  File "/usr/lib/python3.5/asyncio/base_events.py", line 506, in _call_soon
    self._check_closed()
  File "/usr/lib/python3.5/asyncio/base_events.py", line 334, in _check_closed
    raise RuntimeError('Event loop is closed')
RuntimeError: Event loop is closed
[118.027s] ERROR:concurrent.futures:exception calling callback for <Future at 0x7f7aeeb2f780 state=finished returned NoneType>
Traceback (most recent call last):
  File "/usr/lib/python3.5/concurrent/futures/_base.py", line 297, in _invoke_callbacks
    callback(self)
  File "/usr/lib/python3.5/asyncio/futures.py", line 442, in _call_set_state
    dest_loop.call_soon_threadsafe(_set_state, destination, source)
  File "/usr/lib/python3.5/asyncio/base_events.py", line 532, in call_soon_threadsafe
    handle = self._call_soon(callback, args)
  File "/usr/lib/python3.5/asyncio/base_events.py", line 506, in _call_soon
    self._check_closed()
  File "/usr/lib/python3.5/asyncio/base_events.py", line 334, in _check_closed
    raise RuntimeError('Event loop is closed')
RuntimeError: Event loop is closed
[118.869s] ERROR:concurrent.futures:exception calling callback for <Future at 0x7f7aeeaf1518 state=finished returned NoneType>
Traceback (most recent call last):
  File "/usr/lib/python3.5/concurrent/futures/_base.py", line 297, in _invoke_callbacks
    callback(self)
  File "/usr/lib/python3.5/asyncio/futures.py", line 442, in _call_set_state
    dest_loop.call_soon_threadsafe(_set_state, destination, source)
  File "/usr/lib/python3.5/asyncio/base_events.py", line 532, in call_soon_threadsafe
    handle = self._call_soon(callback, args)
  File "/usr/lib/python3.5/asyncio/base_events.py", line 506, in _call_soon
    self._check_closed()
  File "/usr/lib/python3.5/asyncio/base_events.py", line 334, in _check_closed
    raise RuntimeError('Event loop is closed')
RuntimeError: Event loop is closed
[118.869s] ERROR:concurrent.futures:exception calling callback for <Future at 0x7f7aeeaf1a58 state=finished returned NoneType>
Traceback (most recent call last):
  File "/usr/lib/python3.5/concurrent/futures/_base.py", line 297, in _invoke_callbacks
    callback(self)
  File "/usr/lib/python3.5/asyncio/futures.py", line 442, in _call_set_state
    dest_loop.call_soon_threadsafe(_set_state, destination, source)
  File "/usr/lib/python3.5/asyncio/base_events.py", line 532, in call_soon_threadsafe
    handle = self._call_soon(callback, args)
  File "/usr/lib/python3.5/asyncio/base_events.py", line 506, in _call_soon
    self._check_closed()
  File "/usr/lib/python3.5/asyncio/base_events.py", line 334, in _check_closed
    raise RuntimeError('Event loop is closed')
RuntimeError: Event loop is closed
[138.736s] ERROR:concurrent.futures:exception calling callback for <Future at 0x7f7aeead00f0 state=finished returned NoneType>
Traceback (most recent call last):
  File "/usr/lib/python3.5/concurrent/futures/_base.py", line 297, in _invoke_callbacks
    callback(self)
  File "/usr/lib/python3.5/asyncio/futures.py", line 442, in _call_set_state
    dest_loop.call_soon_threadsafe(_set_state, destination, source)
  File "/usr/lib/python3.5/asyncio/base_events.py", line 532, in call_soon_threadsafe
    handle = self._call_soon(callback, args)
  File "/usr/lib/python3.5/asyncio/base_events.py", line 506, in _call_soon
    self._check_closed()
  File "/usr/lib/python3.5/asyncio/base_events.py", line 334, in _check_closed
    raise RuntimeError('Event loop is closed')
RuntimeError: Event loop is closed
[138.737s] ERROR:concurrent.futures:exception calling callback for <Future at 0x7f7aeead09e8 state=finished returned NoneType>
Traceback (most recent call last):
  File "/usr/lib/python3.5/concurrent/futures/_base.py", line 297, in _invoke_callbacks
    callback(self)
  File "/usr/lib/python3.5/asyncio/futures.py", line 442, in _call_set_state
    dest_loop.call_soon_threadsafe(_set_state, destination, source)
  File "/usr/lib/python3.5/asyncio/base_events.py", line 532, in call_soon_threadsafe
    handle = self._call_soon(callback, args)
  File "/usr/lib/python3.5/asyncio/base_events.py", line 506, in _call_soon
    self._check_closed()
  File "/usr/lib/python3.5/asyncio/base_events.py", line 334, in _check_closed
    raise RuntimeError('Event loop is closed')
RuntimeError: Event loop is closed
[138.754s] ERROR:asyncio:Task was destroyed but it is pending!
task: <Task pending coro=<_fd2callback() done, defined at /usr/local/lib/python3.5/dist-packages/colcon_core/subprocess.py:220> wait_for=<Future pending cb=[_chain_future.<locals>._call_check_cancel() at /usr/lib/python3.5/asyncio/futures.py:431, Task._wakeup()]> cb=[_wait.<locals>._on_completion() at /usr/lib/python3.5/asyncio/tasks.py:414]>
[138.755s] ERROR:asyncio:Task was destroyed but it is pending!
task: <Task pending coro=<_fd2callback() done, defined at /usr/local/lib/python3.5/dist-packages/colcon_core/subprocess.py:220> wait_for=<Future pending cb=[_chain_future.<locals>._call_check_cancel() at /usr/lib/python3.5/asyncio/futures.py:431, Task._wakeup()]> cb=[_wait.<locals>._on_completion() at /usr/lib/python3.5/asyncio/tasks.py:414]>
[138.755s] ERROR:asyncio:Task was destroyed but it is pending!
task: <Task pending coro=<_fd2callback() done, defined at /usr/local/lib/python3.5/dist-packages/colcon_core/subprocess.py:220> wait_for=<Future pending cb=[_chain_future.<locals>._call_check_cancel() at /usr/lib/python3.5/asyncio/futures.py:431, Task._wakeup()]> cb=[_wait.<locals>._on_completion() at /usr/lib/python3.5/asyncio/tasks.py:414]>
[138.755s] ERROR:asyncio:Task was destroyed but it is pending!
task: <Task pending coro=<_fd2callback() done, defined at /usr/local/lib/python3.5/dist-packages/colcon_core/subprocess.py:220> wait_for=<Future pending cb=[_chain_future.<locals>._call_check_cancel() at /usr/lib/python3.5/asyncio/futures.py:431, Task._wakeup()]> cb=[_wait.<locals>._on_completion() at /usr/lib/python3.5/asyncio/tasks.py:414]>
[138.755s] ERROR:asyncio:Task was destroyed but it is pending!
task: <Task pending coro=<_fd2callback() done, defined at /usr/local/lib/python3.5/dist-packages/colcon_core/subprocess.py:220> wait_for=<Future pending cb=[_chain_future.<locals>._call_check_cancel() at /usr/lib/python3.5/asyncio/futures.py:431, Task._wakeup()]> cb=[_wait.<locals>._on_completion() at /usr/lib/python3.5/asyncio/tasks.py:414]>
[138.755s] ERROR:asyncio:Task was destroyed but it is pending!
task: <Task pending coro=<_fd2callback() done, defined at /usr/local/lib/python3.5/dist-packages/colcon_core/subprocess.py:220> wait_for=<Future pending cb=[_chain_future.<locals>._call_check_cancel() at /usr/lib/python3.5/asyncio/futures.py:431, Task._wakeup()]> cb=[_wait.<locals>._on_completion() at /usr/lib/python3.5/asyncio/tasks.py:414]>
[138.755s] ERROR:asyncio:Task was destroyed but it is pending!
task: <Task pending coro=<_fd2callback() done, defined at /usr/local/lib/python3.5/dist-packages/colcon_core/subprocess.py:220> wait_for=<Future pending cb=[_chain_future.<locals>._call_check_cancel() at /usr/lib/python3.5/asyncio/futures.py:431, Task._wakeup()]> cb=[_wait.<locals>._on_completion() at /usr/lib/python3.5/asyncio/tasks.py:414]>
[138.755s] ERROR:asyncio:Task was destroyed but it is pending!
task: <Task pending coro=<_fd2callback() done, defined at /usr/local/lib/python3.5/dist-packages/colcon_core/subprocess.py:220> wait_for=<Future pending cb=[_chain_future.<locals>._call_check_cancel() at /usr/lib/python3.5/asyncio/futures.py:431, Task._wakeup()]> cb=[_wait.<locals>._on_completion() at /usr/lib/python3.5/asyncio/tasks.py:414]>
[138.755s] ERROR:asyncio:Task was destroyed but it is pending!
task: <Task pending coro=<_fd2callback() done, defined at /usr/local/lib/python3.5/dist-packages/colcon_core/subprocess.py:220> wait_for=<Future pending cb=[_chain_future.<locals>._call_check_cancel() at /usr/lib/python3.5/asyncio/futures.py:431, Task._wakeup()]> cb=[_wait.<locals>._on_completion() at /usr/lib/python3.5/asyncio/tasks.py:414]>
[138.756s] ERROR:asyncio:Task was destroyed but it is pending!
task: <Task pending coro=<_fd2callback() done, defined at /usr/local/lib/python3.5/dist-packages/colcon_core/subprocess.py:220> wait_for=<Future pending cb=[_chain_future.<locals>._call_check_cancel() at /usr/lib/python3.5/asyncio/futures.py:431, Task._wakeup()]> cb=[_wait.<locals>._on_completion() at /usr/lib/python3.5/asyncio/tasks.py:414]>

Google search didn't turn out anything similar. Any idea where to start digging?

The stacktraces are coming from an unclean shutdown of asyncio tasks. Since these can be very timing specific can you please share exact steps for a reproducible example.

@dirk-thomas: It happens with every build that has a build error so it is not spurious.  I can try to reproduce it with the ROS 2 master codebase.

@deeplearningrobotics I doesn't happen for me on Ubuntu 18.04. So I need a reproducible example in order to be able to investigate it. I would expect the packages in the workspace to be unrelated to this (except that obviously one needs to fail in order for the packages processed in parallel to be aborted).

I confirm seeing this as well on 16.04— it happens on failure, and there's a variation of it when I ctrl-c out of a running build as well:
Aborted  <<< gtsam
[54.978s] ERROR:asyncio:Task exception was never retrieved
future: <Task finished coro=<_fd2callback() done, defined at /home/administrator/colcon_env/lib/python3.5/site-packages/colcon_core/subprocess.py:216> exception=RuntimeError('cannot reuse already awaited coroutine',)>
Traceback (most recent call last):
  File "/home/administrator/colcon_env/lib/python3.5/site-packages/colcon_parallel_executor/executor/parallel.py", line 67, in execute
    loop.run_until_complete(future)
  File "/usr/lib/python3.5/asyncio/base_events.py", line 375, in run_until_complete
    self.run_forever()
  File "/usr/lib/python3.5/asyncio/base_events.py", line 345, in run_forever
    self._run_once()
  File "/usr/lib/python3.5/asyncio/base_events.py", line 1276, in _run_once
    event_list = self._selector.select(timeout)
  File "/usr/lib/python3.5/selectors.py", line 441, in select
    fd_event_list = self._epoll.poll(timeout, max_ev)
KeyboardInterrupt

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/lib/python3.5/asyncio/tasks.py", line 239, in _step
    result = coro.send(None)
RuntimeError: cannot reuse already awaited coroutine
[54.980s] ERROR:asyncio:Task exception was never retrieved
future: <Task finished coro=<_fd2callback() done, defined at /home/administrator/colcon_env/lib/python3.5/site-packages/colcon_core/subprocess.py:216> exception=RuntimeError('cannot reuse already awaited coroutine',)>
Traceback (most recent call last):
  File "/home/administrator/colcon_env/lib/python3.5/site-packages/colcon_parallel_executor/executor/parallel.py", line 67, in execute
    loop.run_until_complete(future)
  File "/usr/lib/python3.5/asyncio/base_events.py", line 375, in run_until_complete
    self.run_forever()
  File "/usr/lib/python3.5/asyncio/base_events.py", line 345, in run_forever
    self._run_once()
  File "/usr/lib/python3.5/asyncio/base_events.py", line 1276, in _run_once
    event_list = self._selector.select(timeout)
  File "/usr/lib/python3.5/selectors.py", line 441, in select
    fd_event_list = self._epoll.poll(timeout, max_ev)
KeyboardInterrupt

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/lib/python3.5/asyncio/tasks.py", line 239, in _step
    result = coro.send(None)
RuntimeError: cannot reuse already awaited coroutine
[54.993s] ERROR:concurrent.futures:exception calling callback for <Future at 0x7f5a5e55e710 state=finished returned NoneType>
Traceback (most recent call last):
  File "/usr/lib/python3.5/concurrent/futures/_base.py", line 297, in _invoke_callbacks
    callback(self)
  File "/usr/lib/python3.5/asyncio/futures.py", line 442, in _call_set_state
    dest_loop.call_soon_threadsafe(_set_state, destination, source)
  File "/usr/lib/python3.5/asyncio/base_events.py", line 532, in call_soon_threadsafe
    handle = self._call_soon(callback, args)
  File "/usr/lib/python3.5/asyncio/base_events.py", line 506, in _call_soon
    self._check_closed()
  File "/usr/lib/python3.5/asyncio/base_events.py", line 334, in _check_closed
    raise RuntimeError('Event loop is closed')
RuntimeError: Event loop is closed
[54.994s] ERROR:concurrent.futures:exception calling callback for <Future at 0x7f5a5e55eb00 state=finished returned NoneType>
Traceback (most recent call last):
  File "/usr/lib/python3.5/concurrent/futures/_base.py", line 297, in _invoke_callbacks
    callback(self)
  File "/usr/lib/python3.5/asyncio/futures.py", line 442, in _call_set_state
    dest_loop.call_soon_threadsafe(_set_state, destination, source)
  File "/usr/lib/python3.5/asyncio/base_events.py", line 532, in call_soon_threadsafe
    handle = self._call_soon(callback, args)
  File "/usr/lib/python3.5/asyncio/base_events.py", line 506, in _call_soon
    self._check_closed()
  File "/usr/lib/python3.5/asyncio/base_events.py", line 334, in _check_closed
    raise RuntimeError('Event loop is closed')
RuntimeError: Event loop is closed

Summary: 2 packages finished [54.5s]
4 packages aborted: ...
3 packages had stderr output: ...
nnn packages not processed
[55.204s] ERROR:asyncio:Task was destroyed but it is pending!
task: <Task pending coro=<_fd2callback() done, defined at /home/administrator/colcon_env/lib/python3.5/site-packages/colcon_core/subprocess.py:216> wait_for=<Future pending cb=[_chain_future.<locals>._call_check_cancel() at /usr/lib/python3.5/asyncio/futures.py:431, Task._wakeup()]> cb=[_wait.<locals>._on_completion() at /usr/lib/python3.5/asyncio/tasks.py:414]>
[55.205s] ERROR:asyncio:Task was destroyed but it is pending!
task: <Task pending coro=<_fd2callback() done, defined at /home/administrator/colcon_env/lib/python3.5/site-packages/colcon_core/subprocess.py:216> wait_for=<Future pending cb=[_chain_future.<locals>._call_check_cancel() at /usr/lib/python3.5/asyncio/futures.py:431, Task._wakeup()]> cb=[_wait.<locals>._on_completion() at /usr/lib/python3.5/asyncio/tasks.py:414]>

The situation is slightly better if I use colcon from a py3.8 venv (deadsnakes), but still lots of spurious output, including a similar warning about the event loop being closed (and needing to issue a final ctrl-c as well):
Failed   <<< rosmaster  [ Exited with code 1 ]
--- stderr: std_msgs
Traceback (most recent call last):
  File "/home/administrator/colcon_env/lib/python3.8/site-packages/colcon_core/subprocess.py", line 176, in _async_check_call
    done, _ = await asyncio.wait(callbacks, return_when=ALL_COMPLETED)
  File "/usr/lib/python3.8/asyncio/tasks.py", line 426, in wait
    return await _wait(fs, timeout, return_when, loop)
  File "/usr/lib/python3.8/asyncio/tasks.py", line 523, in _wait
    await waiter
asyncio.exceptions.CancelledError

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/administrator/colcon_env/lib/python3.8/site-packages/colcon_core/executor/__init__.py", line 91, in __call__
    rc = await self.task(*args, **kwargs)
  File "/home/administrator/colcon_env/lib/python3.8/site-packages/colcon_core/task/__init__.py", line 92, in __call__
    return await task_method(*args, **kwargs)
  File "/home/administrator/colcon_env/lib/python3.8/site-packages/colcon_ros/task/catkin/build.py", line 72, in build
    rc = await extension.build(
  File "/home/administrator/colcon_env/lib/python3.8/site-packages/colcon_cmake/task/cmake/build.py", line 85, in build
    rc = await self._reconfigure(args, env)
  File "/home/administrator/colcon_env/lib/python3.8/site-packages/colcon_cmake/task/cmake/build.py", line 172, in _reconfigure
    return await check_call(
  File "/home/administrator/colcon_env/lib/python3.8/site-packages/colcon_core/task/__init__.py", line 147, in check_call
    rc = await run(
  File "/home/administrator/colcon_env/lib/python3.8/site-packages/colcon_core/subprocess.py", line 81, in run
    rc, _, _ = await _async_check_call(
  File "/home/administrator/colcon_env/lib/python3.8/site-packages/colcon_core/subprocess.py", line 183, in _async_check_call
    for task in done:
UnboundLocalError: local variable 'done' referenced before assignment
---
Failed   <<< std_msgs   [ Exited with code 1 ]

Summary: 16 packages finished [1min 21s]
  4 packages failed: gtsam rosmaster std_msgs xmlrpcpp
  5 packages had stderr output: aruco gtsam rosmaster std_msgs xmlrpcpp
  654 packages not processed
[81.366s] WARNING:asyncio:Loop <_UnixSelectorEventLoop running=False closed=True debug=False> that handles pid 31742 is closed

^CError in atexit._run_exitfuncs:
Traceback (most recent call last):
  File "/usr/lib/python3.8/concurrent/futures/thread.py", line 40, in _python_exit
    t.join()
  File "/usr/lib/python3.8/threading.py", line 1011, in join
    self._wait_for_tstate_lock()
  File "/usr/lib/python3.8/threading.py", line 1027, in _wait_for_tstate_lock
    elif lock.acquire(block, timeout):
KeyboardInterrupt

The situation is slightly better if I use colcon from a py3.8 venv (deadsnakes), but still lots of spurious output, including a similar warning about the event loop being closed (and needing to issue a final ctrl-c as well):

Does that happen all the time for you or only sometimes? Do you happen to have reproducible steps?

UnboundLocalError: local variable 'done' referenced before assignment

You might want to give #245 a try which should resolve this problem.

I will only be able to look into it when it is reproducible on 18.04 (or soon also 20.04) with the default Python interpreter. Don't because other environments are not something we use / test with.

No worries! We probably won't be looking into it either— this just clarifies for us that a colcon migration is not in the cards for us until we're no longer targeting 16.04.

@mikepurvis It might be worth to try this case again with recent changes landed which addressed some problems around the CancelledError handling (see #284 and #285).

@mikepurvis Friendly ping.

Yup, just re-tested now and the issue on 16.04 is now fixed. Thanks!


fix sketchy tests

Scope & Purpose
Fixed evil thread creation and make it safe

[x] Bug-Fix for devel-branch (i.e. no need for backports?)
[ ] Bug-Fix for a released version (did you remember to port this to all relevant release branches?)
[ ] Strictly new functionality (i.e. a new feature / new option, no need for porting)
[x] The behavior in this PR can be (and was) manually tested (support / qa / customers can test it)
[x] The behavior change can be verified via automatic tests

Testing & Verification
This change is already covered by existing tests, such as arangodbtests.
http://<IP_ADDRESS>:8080/view/PR/job/arangodb-matrix-pr/8535/

http://<IP_ADDRESS>:8080/view/PR/job/arangodb-matrix-pr/8543/


PHP code snippets for Rackspace CDN getting started guide

Also, fixing placeholders in curl code snippets.
Attn: @jamiehannaford, @rdodev

LGTM :shipit:

@ycombinator expressiveness vs. consistency via placeholders is a fine line to toe. And to be honest, the argument can go either way. I don't have a definite answer, sorry :(

@ycombinator @rdodev Based on what Shaunak said, I have no problem with the sample data - I agree it's more expressive than placeholders here. LGTM :+1:

I see @rdodev's :shipit: and @jamiehannaford's :+1: and I raise them a merge.


Custom Data Source

It would be great to add sources from custom TableProvider. Also, leveraging sources in https://github.com/datafusion-contrib/datafusion-table-providers would allow to reuse the work done on datatype conversions and connectors.
The current API  relies on SessionManager to start an HTTP server. Could there be an example to add tables or a possible change in the API to allow that?

Thank you very much for your advice. I greatly appreciate your suggestion regarding the datafusion-table-providers repository.
The insights from datafusion-table-providers have been extremely valuable. I am considering actively utilizing the TableProvider trait of DataFusion, especially for reading operations. However, I believe that DataFusion’s TableProvider is most powerful when dealing with intelligent systems like RDBMS.
In this project, datafusion-server, our goal is to achieve seamless SQL operations, including both read and write directions, which somewhat diverges from the original concept of Apache DataFusion.
For example, a DML like this.
INSERT INTO table1@pg1 (id, name)
SELECT id, name FROM superstore WHERE "State" = 'CA';

UPDATE table1@mysql1
SET profit = (SELECT MAX(profit) FROM store WHERE "State" = 'CA')
WHERE state = 'us-west';

I recognize that the current integration between the SessionManager and the Axum HTTP server is not ideal for enabling seamless queries across different systems, and this is an area I plan to refactor in the future.
For now, we rely on the Python data source connector plugin to allow users to add arbitrary data sources. However, I am aware that this approach has limited extensibility as a Rust crate.
In the future, and especially towards the release of Version 1.0, I aim to decouple the HTTP server, session management, transport, data sources, and queries (including DML) as much as possible to improve modularity and make the system more robust as a collection of Rust crates.
Thank you again for your valuable input. I look forward to any further suggestions or discussions on this topic.


DnsAddressResolverGroup does not respect java.net.preferIPv4Stack=true

For async DNS queries using
DnsAddressResolverGroup resolverGroup = new DnsAddressResolverGroup(EpollDatagramChannel.class, DnsServerAddresses.defaultAddresses());

wanted to only connect to ipv4 addresses. The jvm flag java.net.preferIPv4Stack=true passed is not respected
netty 4.1.4.Final

Was fixed in 4.1.5.Final:
https://github.com/netty/netty/issues/5657


Geofence-String wont work

Hi,
Geofence_id, Geofences and Geofence_string dont show anything under objects, when tracker is in homezone. The fences are registered inside the device but it wont work. On traccar-WebUI I can see the logged in geofence, but not under the iobroker object. There is still Null or [] .

Please provide a debug log

2023-06-21 10:05:22.409 - debug: traccar.0 (30400) Send ping to server
2023-06-21 10:05:22.411 - debug: traccar.0 (30400) Receive pong from server
2023-06-21 10:05:27.774 - debug: traccar.0 (30400) Incomming message: {}
2023-06-21 10:05:52.411 - debug: traccar.0 (30400) Send ping to server
2023-06-21 10:05:52.412 - debug: traccar.0 (30400) Receive pong from server
2023-06-21 10:05:52.523 - debug: traccar.0 (30400) Incomming message: {"devices":[{"id":2,"attributes":{"deviceImage":"device.png"},"groupId":0,"name":"Derbi GP1 125 Low Seat","uniqueId":"350317172823217","status":"online","lastUpdate":"2023-06-21T08:05:52.444+00:00","positionId":3483,"phone":null,"model":null,"contact":null,"category":"motorcycle","disabled":false,"expirationTime":null}]}
2023-06-21 10:05:52.990 - debug: traccar.0 (30400) Incomming message: {"positions":[{"id":4652,"attributes":{"priority":0,"sat":20,"event":0,"ignition":false,"motion":false,"rssi":4,"io200":0,"io69":1,"pdop":1.0,"hdop":0.6000000000000001,"power":11.943,"battery":4.071,"io68":0,"operator":26203,"odometer":29552,"distance":0.0,"totalDistance":8167.11,"hours":732990},"deviceId":2,"protocol":"teltonika","serverTime":"2023-06-21T08:05:52.444+00:00","deviceTime":"2023-06-21T08:05:48.000+00:00","fixTime":"2023-06-21T08:05:48.000+00:00","outdated":false,"valid":true,"latitude":49.758933,"longitude":9.7544266,"altitude":233.0,"speed":0.0,"course":114.0,"address":null,"accuracy":0.0,"network":null,"geofenceIds":[1]}]}
2023-06-21 10:05:52.991 - debug: traccar.0 (30400) ============= Process attributes start ================
2023-06-21 10:05:52.992 - debug: traccar.0 (30400) objID: devices.device.priority, val: 0
2023-06-21 10:05:52.992 - debug: traccar.0 (30400) objID: devices.device.sat, val: 20
2023-06-21 10:05:52.992 - debug: traccar.0 (30400) objID: devices.device.event, val: 0
2023-06-21 10:05:52.992 - debug: traccar.0 (30400) objID: devices.device.ignition, val: false
2023-06-21 10:05:52.993 - debug: traccar.0 (30400) objID: devices.device.motion, val: false
2023-06-21 10:05:52.993 - debug: traccar.0 (30400) objID: devices.device.rssi, val: 4
2023-06-21 10:05:52.993 - debug: traccar.0 (30400) objID: devices.device.io200, val: 0
2023-06-21 10:05:52.994 - debug: traccar.0 (30400) objID: devices.device.io69, val: 1
2023-06-21 10:05:52.994 - debug: traccar.0 (30400) objID: devices.device.pdop, val: 1
2023-06-21 10:05:52.994 - debug: traccar.0 (30400) objID: devices.device.hdop, val: 0.6000000000000001
2023-06-21 10:05:52.996 - debug: traccar.0 (30400) objID: devices.device.power, val: 11.943
2023-06-21 10:05:52.996 - debug: traccar.0 (30400) objID: devices.device.battery, val: 4.071
2023-06-21 10:05:52.996 - debug: traccar.0 (30400) objID: devices.device.io68, val: 0
2023-06-21 10:05:52.997 - debug: traccar.0 (30400) objID: devices.device.operator, val: 26203
2023-06-21 10:05:52.997 - debug: traccar.0 (30400) objID: devices.device.odometer, val: 29552
2023-06-21 10:05:52.997 - debug: traccar.0 (30400) objID: devices.device.distance, val: 0
2023-06-21 10:05:52.998 - debug: traccar.0 (30400) objID: devices.device.total_distance, val: 8167.11
2023-06-21 10:05:52.998 - debug: traccar.0 (30400) objID: devices.device.hours, val: 732990
2023-06-21 10:05:52.998 - debug: traccar.0 (30400) ============== Process attributes end =================
2023-06-21 10:06:22.412 - debug: traccar.0 (30400) Send ping to server
2023-06-21 10:06:22.415 - debug: traccar.0 (30400) Receive pong from server
2023-06-21 10:06:22.774 - debug: traccar.0 (30400) Incomming message: {}
7

Traccar 5.8 change:
List of current geofences moved from devices to positions

What do u mean with that comment ?

Before Traccar V 5.8 GeofenceIds was in devices (API), in V 5.8 it is in Position. You can see this in your Log. I use my own Script and I have the same Problem, but I fixed it.

If you use the Adapter and can Not fix it by your self, someone else have to make it. Good to Open this issue! May be you are the First one with this problem, but over time not the last one IMO.

@Larson-Sei180LX please test the dev branch, I don't have traccar in use anymore so I can't test it.
https://github.com/o0shojo0o/ioBroker.traccar/tree/%23482

Hey - perfect - it works. Installed dev branch and now I can see the geofence area....
Thank you - maybe u could update in master ?!


Update documentation.md

Updated the spelling errors, added full stops wherever necessary, and corrected the formatting of tables.
Description

The documentation is read by everyone visiting this repository. So, certain spelling mistakes and missing punctuation might make it look incomplete. So, I have made all the necessary changes after raising this issue.

Resolves: #930
Checklist

[ ✓] Have you adhered to the repository's defined coding convention rules?
[ ] Have you updated the 'documentation.md' file with the method/function documentation?
[ ] Have you sent a message along with the result or response?
[ ] Have you used the try-catch technique?
[ ] Has the method/class been added to the documentation (md file)?

Screenshots

Correction of the word 'Challenges'.

Grammatical and punctuation corrected.

'Latest' spelling corrected.

I certify that I have carried out the relevant checks and provided the requisite screenshot for validation by submitting this pull request.
I appreciate your contribution.

Great work @Subhoshri  !!
Do not forget to

Share your contribution on socials and tag Clueless 🫂
Give this repository a star ✨
Follow Clueless Community on GitHub ✅

Thank you @nikhil25803 ! Looking forward to working upon other issues as well.


Sync with Upstream

This pull request updates the fork to match the latest changes from the upstream repository.

Accidentally opened by my fork, closing


Documentation for mobile framework support

Add a doc to address questions we get around mobile support

@jasonnutter  open to suggestions, maybe we can link from there but it wasn't clear where this belonged.

@jasonnutter  open to suggestions, maybe we can link from there but it wasn't clear where this belonged.

I would be up for a repo-level FAQ (which could also then link to FAQs for each library).

I would be up for a repo-level FAQ (which could also then link to FAQs for each library).

I would be up for a repo-level FAQ (which could also then link to FAQs for each library).

+1 for this suggestion

I would be up for a repo-level FAQ (which could also then link to FAQs for each library).

+1 for this suggestion


Source Code download for Dnn 9.5.0 still has version 9.4.4 on solutioninfo file

Just downloaded the source code for Dnn 9.5.0 and noticed that it was building 9.4.4 version. Did it just missed the update to the new version number, or is this the actual 9.4.4 version and the link for Dnn 9.5.0 is incorrect.

Upon build, the version get's set by the build scripts. We went from hard coded versions to fully automated versions to a hybrid solution. Right now whatever version is in the source code would get replaced at build time for the correct one, but we plan on improving the process so the source code at least matches the last or next release number or some such. This is still a work in progress and there is another issue on the plate for this. But right now, if you read the build documentation, it is automated for releases and for people needing to specify the version manually, you can create a local settings file for it also. See https://github.com/dnnsoftware/Dnn.Platform/blob/develop/.github/BUILD.md


Use cases of Dream.path

The function Dream.path has been marked as deprecated. I understand that structured routing is preferable, but its removal blocks use cases for which routes can't be computed upfront. For example:

Dream.static depends on the internal equivalent to path to map paths to the file system; it should be possible to implement such a handler with Dream's public API as well.
I'm currently writing a CGI handler, and I needed path to forward paths to CGI scripts.

If path were removed, implementing such handlers would become an exercise in recursive application of Dream.router to consume the entire path that wouldn't even provide any static information to parent scopes. Simple access to the path feels to me too foundational to remove, even if it competes with structured routes. (Would access to query parameters be removed as well if they were made routable?)
If widespread access to path is problematic for routing, maybe path could return a value only when the innermost route handler ends with a wildcard, and raise an exception otherwise. This would make handlers like static or my CGI handler fail unless mounted directly under a wildcard, which I think is reasonable.

Will/has the deprecation of this be reversed?

We use Dream.path in two cases:

.../build/latest/**  where it redirects to .../build/:build/** preserving the tail of the path
file/** which looks up in a database to find the data corresponding to that file, somewhat similar to Dream.static except with a indirection through a database.

Thank you all. It looks like it's best to undeprecate it, but probably also consider routing in more detail.


Why is jnp.nonzero not jittable?

I'm trying to use jnp.nonzero(a) inside a jitted function. The array a depends only on the input array shapes, so I expected it to work, since the output shapes are constant for a given input shape. But apparently jnp.nonzero always throws an error.
For example,
@jit
def example():
    a = np.array([0, 1, 2])
    return np.nonzero(a)

throws an error when called.
Is this the intended behaviour? If so, are there any workarounds?

Yes, this is intended.
The reason this is not jittable is because under JIT, the shapes of returned arrays must be static, i.e. dependent on the shape of the inputs rather than their contents. The nonzero function returns a value whose shape depends on the contents of the inputs, so it is not compatible with JIT.
There are no workarounds for the simple case you show above, but depending on your actual application you can often use the three-argument version of jnp.where instead.  For example, here is a non-jittable function:
def f(a):
   return a[jnp.nonzero(a)].prod()

and here is an alternative version of what is essentially the same function, but that is compatible with JIT:
def f(a):
  return jnp.where(a != 0, a, 1).prod()

Thanks!
I'm confused about the static shapes requirement: jnp.nonzero doesn't work even when embedded in a larger function where the shapes of the returned arrays are static. The thing I'm trying to do is return the indices of the off-diagonal elements of a square matrix, that is
def offdiagonal_idx(matrix):
    n = matrix.shape[0]
    return np.nonzero(~np.eye(n, dtype=bool))

The output only depends on the shape of the input matrix, but still throws an error. Also, this worked in a previous version of jax, so there must have been a decision at some point to not allow this anymore.

if you've done import numpy as np, that should work without any issues: that's the approach I would recommend here:
import numpy as np
import jax.numpy as jnp
from jax import jit

@jit
def offdiagonal_idx(matrix):
    n = matrix.shape[0]
    return np.nonzero(~np.eye(n, dtype=bool))

print(offdiagonal_idx(jnp.ones((3, 3))))
# (DeviceArray([0, 0, 1, 1, 2, 2], dtype=int32), DeviceArray([1, 2, 0, 2, 0, 1], dtype=int32))

Thanks a lot, this resolves my problem perfectly. The 'just use numpy for static computations' trick is super useful (might be worth adding to the docs, even though it seems obvious in retrospect). Also thanks for the ref to #3370.
Cheers!


Help setting up a Golang adapter

Hi @rcarriga,
Great work on this plugin 🚀, really excited to see how it evolves.
I've been trying to get it working with go and have been having some trouble, having read the explanation in the README and also dug through the code of this repo and the existing adapters.
I've got the project over here https://github.com/akinsho/neotest-go (happy to move it under a more general namespace if you intend to create a neotest org since I'm sure other people might want to contribute eventually, and I might stop working with go 🤷🏿)
I've added a sub folder (neogit_go) within the repo which adds two simple examples of tests and am trying to get things working using those.
Reference: https://pkg.go.dev/testing
Issues

[x] Detecting multiple test functions doesn't always work, sometimes it only works if there is one test in the file, and sometimes only works if there are multiple
[x] It's not entirely clear how results should be returned, I assumed as a list of neotest.Result[] but tests seem to still show as pending when I return this list, so something seems to be missing
[ ] How to handle test cases specified using a table or a list or struct (map) e.g.
(I'm guessing this case might not be possible, since it's not clear how to consistently determine what is a test case in the snippet below)

package add
import "testing"
func TestAdd(t *testing.T) {
    cases := []struct {
        desc     string
        a, b     int
        expected int
    }{
        {"TestBothZero", 0, 0, 0},
        {"TestBothPositive", 2, 5, 7},
        {"TestBothNegative", -2, -5, -7},
    }
    for _, tc := range cases {
        actual := add(tc.a, tc.b)
        if actual != tc.expected {
            t.Fatalf("%s: expected: %d got: %d for a: %d and b %d",
                tc.desc, actual, tc.expected, tc.a, tc.b)
        }
    }
}

Any advice or pointers would be much appreciated

Hey thanks for looking to create an adapter! I've considered a separate org for them and will probably do that, just need to get some time to do so, hopefully this week. Probably should have that before releasing but to be honest, I didn't expect people to be so fast to start developing adapters :sweat_smile:
To answer your questions:

I believe this was due to a small bug in the core when multiple adapters were present. You should see this fixed now in the latest commit.
The return type of results should be a table<string, neotest.Result> (see here https://github.com/rcarriga/neotest/blob/master/lua/neotest/adapters/interface.lua#L32) which maps the test IDs to results. Otherwise the core has no way to no what result corresponds to what result.
Had the same issue of dynamic tests for neotest-plenary and I suspect this will be a problem in other languages too. I know VSCode's Jest integration will actually show the dynamic tests as separate tests but that requires a very strong integration which I think is overkill. Instead what I did for neotest-plenary was to check where the dynamic test was created (using the debug module) and matched it to the range of a static test. When constructing results the result object allows for multiple error messages so that multiple dynamic tests results can be aggregated in one. So for your example the result for TestAdd would contain the errors for TestBothZero, TestBothPositive and TestBothNegative. Of course this is just a suggestion and nothing about the interface requires that. Also not sure how possible that is without integrating with go. I've looked at wrapping go's test module before and it is a bit of a beast)

Thanks for the fix and the pointers @rcarriga, regarding point 2 on the return for results, is the correct way to handle this then to iterate the tree somehow and match each result output from the test runner to a node of the tree? how do you account for ways that the tests might have been run? do you need to?

I ended up using the tree:iter method and mapping each test name to each node which seemed to work. Not sure if that would work as well for larger test scopes like the whole suite etc.

@rcarriga seems to work well enough now, so if you're ok with it will open a PR to add it to the list of adapters (it's still pretty beta though). One thing I'd still like some help with is I'd like to change the output content. I use the -json flag, so the test runner gives me easily parsable output, but that means that opening up the test output just shows JSON which isn't that readable. I'm able to collect and parse the test output, so I can re-generate the string that you'd otherwise get from a normal run of go test but not sure how to feed it into neotest for the output function
e.g.

Other things I'll eventually want to tackle are

table/list based test cases
Different ways of running tests other than just test, file. I'd like to add an easy way to test packages not sure exactly how to expose that command

Great to hear it's working! Happy to add it to the list.
For showing readable output you could create a tempfile using tempname(), write the reconstructed data to it and then set the output field of the result objects. It's set automatically by the core client if not present to the output of the process.

I'd like to add an easy way to test packages not sure exactly how to expose that command

Not too familiar with go beyond the basics, can multiple packages share the same directory (meaning that running a directory wouldn't work)? It may require an additional position type to be added to core to represent it if so

@rcarriga that worked re. saving output to a different file and then using that path 👍🏿. One case where I'm not sure if this works is when for whatever reason no test file is found, in this case there is no result to point my output file to, so it just uses the fallback. Tbh the case I'm explaining is an odd one since it should definitely not be saying there is no test, but I'll dig into that, I'm just wondering if there's a way to change the fallback output file to one of my own making even if there are no results
Re. the packages, I believe that is invalid in go and all files from a package must be within that folder, so I guess just using the directory runner would work.

if there's a way to change the fallback output file to one of my own making even if there are no results

You can return a result object for any position including namespaces, files and directories. By default they're just aggregated from child tests. So just iterate through the tree and set the result output for each one to your custom one.

Re. the packages, I believe that is invalid in go and all files from a package must be within that folder, so I guess just using the directory runner would work.

Good to hear :+1:

@rcarriga thanks for your help, now the project is up I'll close this out. If I have any issues with getting other stuff working, I'll tag you in an issue on that repo if that's alright. Hopefully shouldn't be frequent.

Sounds good to me, thanks for contributing!


Fix shellcheck tests

See https://travis-ci.org/yakshaveinc/linux/builds/573118833 for the rest of what needs to be done.

Done.


Incorrect exercise: [courseId=grammar-intro_eng][lessonNumber=10][exerciseId=3267]

Description
If possible, please provide a brief description of what is wrong with this exercise.
I am using my phone to type the answers. It seems the apostrophe differs from a single quote. Is it possible to include an apostrophe along with single quotes {be’o} vs {be'o}?
Diagnostics information (do not remove)

Exercise identifier: [courseId=grammar-intro_eng][lessonNumber=10][exerciseId=3267]
Exercise payload: {"sentences":[{"text":"I like your name, which is beautiful.","lojbanic":false}],"title":"Translate this sentence","type":"typing"}
Attempted solution: {"text":"mi nelci lo cmene be do be’o noi melbi"}
Attempted solution was classified as correct: no
Validation payload: {"canonicalAnswer":"mi nelci lo cmene be do be'o noi melbi","correct":false}

Thank you, good catch! I will make code changes so that apostrophes and single quotes are treated identically.

Amazing. I’m learning so much.


API: Handler request url & header parameters. Hook recipients

Coverage decreased (-1.007%) to 93.666% when pulling a5ce32195c17c1fcd50d1d2e3f1bf39f045bc8b4 on enhacement/small_endpoints_api into 436876bd0c4f6f694724b01aa1ac302027b981f0 on master.


Add containers which responsively switch between being fluid or fixed-width

E.g. there are no classes like container-md. I wrote a gist to work around the issue.
It seems odd that the framework allows column styles to be invoked only at certain screen widths, but not for containers (and rows).

Containers are max-width right now, so they're always "responsive" in v4. Not sure I see much value in replacing the existing grid and container system with this—feels over engineered to me.

@mdo Yes, containers are already responsive but they cannot switch between fluid and fixed-width which provide very different layouts, especially on larger screens. The issue was not about making them responsive, but about allowing the same flexibility for containers as is done for columns.

I should've written that clearer—it's less about any particular behavior and more about the over-engineered aspect. The columns within those containers behave the same though—column widths are all set in percentages. A pixel-based max-width vs a width: 100% doesn't matter much.
The kind of control seems ripe just for overrides rather than a rethinking of a core part of the grid system at this point in v4's development process.


Tweak renderLights()

Tweaked renderLights() to save ~74 bytes :

Reordered switch case in descending time.
Used switch-statement fallthrough (with extra comments), to combine common code.
Replaced renderLights_Shown() function, with inline fillRect.

This has been play tested and is not expected to alter the UX in anyway. Just some code shuffling to shave off bytes, while (hopefully) maintaining readability. :-)


Training error about detectron2

Hi,
During the training, I've encountered the error below:
2021-06-02 19:47:08.050507: I tensorflow/stream_executor/platform/default/dso_loader.cc:48] Successfully opened dynamic library libcudart.so.10.1
2021-06-02 19:47:10.647322: I tensorflow/stream_executor/platform/default/dso_loader.cc:48] Successfully opened dynamic library libcuda.so.1
2021-06-02 19:47:14.161774: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1716] Found device 0 with properties:
pciBusID: 0000:3b:00.0 name: Tesla V100-PCIE-32GB computeCapability: 7.0
coreClock: 1.38GHz coreCount: 80 deviceMemorySize: 31.75GiB deviceMemoryBandwidth: 836.37GiB/s
2021-06-02 19:47:14.163223: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1716] Found device 1 with properties:
pciBusID: 0000:5e:00.0 name: Tesla V100-PCIE-32GB computeCapability: 7.0
coreClock: 1.38GHz coreCount: 80 deviceMemorySize: 31.75GiB deviceMemoryBandwidth: 836.37GiB/s
2021-06-02 19:47:14.164597: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1716] Found device 2 with properties:
pciBusID: 0000:86:00.0 name: Tesla V100-PCIE-32GB computeCapability: 7.0
coreClock: 1.38GHz coreCount: 80 deviceMemorySize: 31.75GiB deviceMemoryBandwidth: 836.37GiB/s
2021-06-02 19:47:14.165981: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1716] Found device 3 with properties:
pciBusID: 0000:af:00.0 name: Tesla V100-PCIE-32GB computeCapability: 7.0
coreClock: 1.38GHz coreCount: 80 deviceMemorySize: 31.75GiB deviceMemoryBandwidth: 836.37GiB/s
2021-06-02 19:47:14.166042: I tensorflow/stream_executor/platform/default/dso_loader.cc:48] Successfully opened dynamic library libcudart.so.10.1
2021-06-02 19:47:14.168723: I tensorflow/stream_executor/platform/default/dso_loader.cc:48] Successfully opened dynamic library libcublas.so.10
2021-06-02 19:47:14.170526: I tensorflow/stream_executor/platform/default/dso_loader.cc:48] Successfully opened dynamic library libcufft.so.10
2021-06-02 19:47:14.171492: I tensorflow/stream_executor/platform/default/dso_loader.cc:48] Successfully opened dynamic library libcurand.so.10
2021-06-02 19:47:14.175216: I tensorflow/stream_executor/platform/default/dso_loader.cc:48] Successfully opened dynamic library libcusolver.so.10
2021-06-02 19:47:14.177022: I tensorflow/stream_executor/platform/default/dso_loader.cc:48] Successfully opened dynamic library libcusparse.so.10
2021-06-02 19:47:14.182611: I tensorflow/stream_executor/platform/default/dso_loader.cc:48] Successfully opened dynamic library libcudnn.so.7
2021-06-02 19:47:14.192325: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1858] Adding visible gpu devices: 0, 1, 2, 3
2021-06-02 19:47:14.193076: I tensorflow/core/platform/cpu_feature_guard.cc:142] This TensorFlow binary is optimized with oneAPI Deep Neural Network Library (oneDNN)to use the following CPU instructions in performance-critical operations:  AVX2 AVX512F FMA
To enable them in other operations, rebuild TensorFlow with the appropriate compiler flags.
2021-06-02 19:47:14.205182: I tensorflow/core/platform/profile_utils/cpu_utils.cc:104] CPU Frequency:<PHONE_NUMBER> Hz
2021-06-02 19:47:14.206884: I tensorflow/compiler/xla/service/service.cc:168] XLA service 0xa3fe5b0 initialized for platform Host (this does not guarantee that XLA will be used).  Original config value was 0.
2021-06-02 19:47:16.816159: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1402] Created TensorFlow device (/job:localhost/replica:0/task:0/device:GPU:0 with 30132 MB memory) -> physical GPU (device: 0, name: Tesla V100-PCIE-32GB, pci bus id: 0000:3b:00.0, compute capability: 7.0)
2021-06-02 19:47:16.819248: W tensorflow/core/common_runtime/gpu/gpu_bfc_allocator.cc:39] Overriding allow_growth setting because the TF_FORCE_GPU_ALLOW_GROWTH environment variable is set. Original config value was 0.
2021-06-02 19:47:16.819287: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1402] Created TensorFlow device (/job:localhost/replica:0/task:0/device:GPU:1 with 30132 MB memory) -> physical GPU (device: 1, name: Tesla V100-PCIE-32GB, pci bus id: 0000:5e:00.0, compute capability: 7.0)
2021-06-02 19:47:16.822083: W tensorflow/core/common_runtime/gpu/gpu_bfc_allocator.cc:39] Overriding allow_growth setting because the TF_FORCE_GPU_ALLOW_GROWTH environment variable is set. Original config value was 0.
2021-06-02 19:47:16.822118: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1402] Created TensorFlow device (/job:localhost/replica:0/task:0/device:GPU:2 with 30132 MB memory) -> physical GPU (device: 2, name: Tesla V100-PCIE-32GB, pci bus id: 0000:86:00.0, compute capability: 7.0)
2021-06-02 19:47:16.824747: W tensorflow/core/common_runtime/gpu/gpu_bfc_allocator.cc:39] Overriding allow_growth setting because the TF_FORCE_GPU_ALLOW_GROWTH environment variable is set. Original config value was 0.
2021-06-02 19:47:16.824781: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1402] Created TensorFlow device (/job:localhost/replica:0/task:0/device:GPU:3 with 30132 MB memory) -> physical GPU (device: 3, name: Tesla V100-PCIE-32GB, pci bus id: 0000:af:00.0, compute capability: 7.0)
2021-06-02 19:47:16.827988: I tensorflow/compiler/xla/service/service.cc:168] XLA service 0x3a36fae0 initialized for platform CUDA (this does not guarantee that XLA will be used). Devices:
2021-06-02 19:47:16.828018: I tensorflow/compiler/xla/service/service.cc:176]   StreamExecutor device (0): Tesla V100-PCIE-32GB, Compute Capability 7.0
2021-06-02 19:47:16.828030: I tensorflow/compiler/xla/service/service.cc:176]   StreamExecutor device (1): Tesla V100-PCIE-32GB, Compute Capability 7.0
2021-06-02 19:47:16.828041: I tensorflow/compiler/xla/service/service.cc:176]   StreamExecutor device (2): Tesla V100-PCIE-32GB, Compute Capability 7.0
2021-06-02 19:47:16.828058: I tensorflow/compiler/xla/service/service.cc:176]   StreamExecutor device (3): Tesla V100-PCIE-32GB, Compute Capability 7.0
Total Params 2559576
Img Model 2405676
Text Model 153900
Loading Saved Model
  0%|                                                                                | 0/2528 [00:00<?, ?it/s]2021-06-02 19:47:48.137599: I tensorflow/stream_executor/platform/default/dso_loader.cc:48] Successfully opened dynamic library libcublas.so.10
  2%|███▍                                                                                | 48/2528 [01:08<48:47,  1.18s/it]Traceback (most recent call last):
  File "trainer_scipt.py", line 232, in <module>
    train_joint_model()
  File "trainer_scipt.py", line 156, in train_joint_model
    train_model(epoch)
  File "trainer_scipt.py", line 85, in train_model
    z_img, z_t_match, z_t_diff = combined_model(img, text_match, text_diff, batch, seq_len_match, seq_len_diff,
  File "/home/engine210/MMFinal2/venv/lib/python3.8/site-packages/torch/nn/modules/module.py", line 722, in _call_impl
    result = self.forward(*input, **kwargs)
  File "/home/engine210/MMFinal2/COSMOS/model_archs/models.py", line 51, in forward
    img = self.maskrcnn_extractor(img, bboxes, bbox_classes)
  File "/home/engine210/MMFinal2/venv/lib/python3.8/site-packages/torch/nn/modules/module.py", line 722, in _call_impl
    result = self.forward(*input, **kwargs)
  File "/home/engine210/MMFinal2/COSMOS/model_archs/image/image_models.py", line 43, in forward
    targets = [annotations_to_instances(bbox.cpu().numpy(), bbox_class.cpu().numpy(), img_shape) for
  File "/home/engine210/MMFinal2/COSMOS/model_archs/image/image_models.py", line 43, in <listcomp>
    targets = [annotations_to_instances(bbox.cpu().numpy(), bbox_class.cpu().numpy(), img_shape) for
  File "/home/engine210/MMFinal2/COSMOS/utils/img_model_utils.py", line 23, in annotations_to_instances
    target.classes = classes
  File "/home/engine210/MMFinal2/detectron2/detectron2/structures/instances.py", line 61, in __setattr__
    self.set(name, val)
  File "/home/engine210/MMFinal2/detectron2/detectron2/structures/instances.py", line 76, in set
    assert (
AssertionError: Adding a field of length 11 to a Instances of length 1

My environment is

CentOS 7
CUDA 11.0/10.1
Python 3.8.1

I installed detectron2 v0.3 (commit 4841e70) with the modified code provided in this repo. I think it's the problem with detectron2 version.
May I ask what version (or more specifically which commit) of detectron2 should we use in this project?

It's weird that you're getting the error after 2% being trained. What batch size are you using?
For sanity check, could you try running on tiny subsets of the dataset, say just 1 sample, then 32 samples, and so on, and see where it breaks.

Closing the issue due to inactivity


v21: signing issue

elliptic's sign method sometimes produces r & s with less than 32 bytes. eosjs needs to pad these.

Fixed by #664


feat(wasmx): DB entities and admin API

This is still WIP and not ready for a final review, but feel free to share feedback in the meantime.
For the sake of simplicity, this implementation bundles the entire filter chain array into a JSON-serialized array. We can break filter entities off into their own table/entity with a FK relationship if need be (should decide on this before release).
changes

new wasm_filter_chains entity

schema
migrations
DB tests
admin API tests

todo

[ ] filter registration/metadata/schema integration

[ ] validate wasm_filter_chains.filters[].name against known filters
[ ] validate wasm_filter_chains.filters[].config

Were we not going for a single plugin that takes in the lists of filters to execute (and their configurations) to start small?

@hbagdi No, we ultimately reversed on that decision because the execution flow between Lua plugins and Wasm filters (each happening on their own Nginx module) imposes some constraints that wouldn't be correctly represented if Wasm filters were encapsulated by a plugin.

Were we not going for a single plugin that takes in the lists of filters to execute (and their configurations) to start small?

That was the 2022 Tech Preview model.

I've been banging my head on the wall about filter config schema validation for a minute now and still don't have it fully functional and sane, so I'm punting on that for the moment. I'd rather we just get this PR merged as-is so that everyone is unblocked.


Fix TDE protector key length

The key length is measured in bits, not bytes

@rbgnk : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@rbgnk Thanks for the feedback, good catch!

#sign-off

@VanMSFT I have approved this PR, any further action required from me on this?

Hi @shohamMSFT - Approved pull requests in this repository are manually merged during Redmond standard business hours, FYI. 👍🏼


FixReclaim resources held by finished tasks

Proposed changes
What happened
Failure to explicitly release resources used by completed tasks can cause memory to remain in use until the task is cleared.
Affected versions
2.1.0 +
Changes
add closeOrReleaseResources method
make sure that resources are properly cleaned up when they are no longer needed. This improves code readability and maintains a consistent approach to resource management.
test
Testing on my test cluster

run buildall


[#2933] Removed [FIRLibrary configureWithApp:]

[Closes #2933]

Removed all calls to [FIRLibrary configureWithApp:]
Removed the configureWithApp: method from the FIRLibrary protocol

⚠ Breaking Changes: Based on the issue, this PR should land in Firebase v9.

Still need to confirm with external clients. Also there is a usage in FIAM at https://github.com/firebase/firebase-ios-sdk/blob/master/FirebaseInAppMessaging/Sources/FIRInAppMessaging.m#L71

Still need to confirm with external clients. Also there is a usage in FIAM at https://github.com/firebase/firebase-ios-sdk/blob/master/FirebaseInAppMessaging/Sources/FIRInAppMessaging.m#L71

Hey @yakovmanshin - thanks for the PR. After some internal discussions, we're going to leave this as is for now as to not disrupt downstream dependencies. It's not worth the breakage at the moment and we don't know who else would be impacted, plus it's not performance intensive or a ton of code so we're okay with it sitting there for another little while (potentially indefinitely). I'll close this for now but will re-open if we decide to move forward with removing it. Thanks again and sorry about that!