CDTQIndexManager has weak reference to CDTDatastore

Checklist

[x] Tick to sign-off your agreement to the Developer Certificate of Origin (DCO) 1.1
[x] Added tests for code changes or test/build only changes
[x] Updated the change log file (CHANGES.md|CHANGELOG.md) or test/build only changes
[x] Completed the PR template below:

Description
Fix lifecycle issues regarding proper opening and closing of resources across CDTDatastore, CDTQIndexManager, CDTDatastoreManager.
Note that the primary motivator here is that the associated object relationship between CDTDatastore and CDTQIndexManager is currently incorrect when facading through index/query methods, and that it is currently possible to obtain multiple CDTQIndexManager instances for the same underlying SQLite database, which leads to incorrect behaviour.
For more details see "Approach" which gives an overview of the design issues.
Fixes #435
Approach
Lifecycle of key objects
CDTDatastore -> CDTQIndexManager
1-to-1 relationship, strong reference via objc_{get,set}AssociatedObject in
CDTDatastore+Query.m.
Each index manager is constructed on demand.
CDTQIndexManager -> CDTDatastore
1-to-1 relationship, weak reference via property datastore, therefore no retain cycles. Because
reference to CDTQIndexManager can't 'leak' (all methods facade through CDTDatastore+Query), this
reference will always be valid.
Note that if CDTQIndexManager is constructed using +managerUsingDatastore or
-initUsingDatastore, the above doesn't hold and it is possible for the reference to datastore to
be invalid. Even worse, a user could then use a mixture of API calls through the constructed index
manager and facaded through the datastore. Therefore this usage is discouraged (all of our current
documentation shows index/query usage facaded through datastore).
For the above reason, add a comment in CDTQIndexManager to clarify that it's not public API.
CDTDatastoreManager -> CDTDatastore
1-to-many relationship, strong references via the openDatastores property, which is a map whose
keys are the datastore name, and values the datastore itself.
Note that the only way of obtaining a new or existing datastore instance is through the manager.
This is a change to existing functionality and guarantees users will obtain the same reference when
multiple calls to -datastoreNamed are made. This is essential for the CDTDatastore ->
CDTQIndexManager to work properly. As of now, this is not the case, and multiple calls to
-datastoreNamed will result in a new and different CDTDatastore to be constructed each time
(although they all point to the same underlying TD_Database). This causes errors as the different
index managers will all try to simultaneously open the same underlying sqlite database.
Note that a user could still have a 'dangling' reference to a CDTDatastore, that is they can
assign the pointer of a previously constructed datastore, then dealloc its manager. Now they have
a datastore they can't delete (they need the manager for this), and worse, they can now obtain a
different pointer for the same underlying datastore by constructing a new manager and calling
-datastoreNamed. I think we have to be pragmatic and decide we can't protect users from this API
abuse.
CDTDatastore -> CDTDatastoreManager
1-to-1 relationship. This is currently held as a strong reference via the property _manager, but
will cause retain cycles when combined with the relationship described in the above para. Is not
currently used so can be downgraded to a weak reference or removed.
Schema & API Changes
Add new method closeDatastoreNamed on CDTDatastoreManager to allow users with many datastores open to avoid resource (file handle) starvation.
Security and Privacy
No change.
Testing
See added test class CDTLifecycleTests. Test approach as follows:
New instances

Calls to CDTDatastore+Query.m methods should always obtain the same CDTQIndexManager instance
for a given CDTDatastore instance.
Calls to CDTDatastoreManager -datastoreNamed should always obtain the same CDTDatastore
instance for a given datastore name.

Retain count reaching zero

When dealloc is called on CDTDatastore, dealloc should be called on the CDTQIndexManager
held as an associated object.
When dealloc is called on CDTDatastoreManager, dealloc should be called on all
CDTDatastores in openDatastores.
Should we have tests for TD_Database retain count?

Explicit close

Explicit close supported and tested via new method call on  CDTDatastoreManager

Explicit delete

When deleting extensions, dealloc should be called on CDTQIndexManager to allow it to close
its underlying database for the extensions directory is deleted. (In deleteDatastoreNamed call
[_openDatastores removeObjectForKey:name] first). Note that not doing this is the cause of the
following message, as reported by customer:

Other test changes
Updated some existing tests which now need an explicit close before re-opening database to reproduce expected behaviour.
Tests run via Rake now run in release mode, as the original bug report shows certain failure modes are only exhibited when run in release (as opposed to debug) mode.
Removed some #ifdef(DEBUG) guards to allow test code to compile in release mode.
Monitoring and Logging
TODO - some debug NSLogs have been left in, these need to be evaluated and turned into "proper" log statements if necessary.

@ricellis suggested log-grepping strategy in e8472c404fc337ad1a3c201f803c74373a1d19f4 - freopen stderr to a temp file, then mmap the file to check contents.

@tomblench looks good, as discussed I think it is worth updating the sample app with the close call if it is relevant there.
Other query tests likely need some updates too, or at least an issue to track those updates.


[i2c,dv] Add csr write excl to OVERRIDE csr

Writes can affect hardware in a way that causes interrupts (sda/scl_interference) to re-assert continuously, preventing the end-of-test clear_interrupts routine from completing.

CHANGE AUTHORIZED: hw/ip/i2c/data/i2c.hjson

CHANGE AUTHORIZED: hw/ip/i2c/data/i2c.hjson


Search history bugged

Microsoft PowerToys version
0.65.0
Installation method
GitHub
Running as admin
None
Area(s) with issue?
PowerRename
Steps to reproduce
nothing special in steps
✔ Expected Behavior
when history turned off, the suggestions should be off
❌ Actual Behavior
, when searching for names to replace in power rename, previous searches appear even though the history is turned off.
Other Software
No response

Is this still an issue in v0.74.1? /needinfo


Support Request:

Describe Your Problem
I’ve had the plugin working before but I installed a fresh copy of homebridge and now it doesn’t work. Thank you
Relevant log output
Devices 2: TypeError: Cannot convert undefined or null to object

Config for homebridge-august
No response
Screenshots
No response
Device & Model
I have 12 locks. 1 Yale Conexis 11 Yale connected
Node.js Version
V18.17.1
NPM Version
9
Homebridge Version
V1.61
Homebridge August Plugin Version
V1.21
Homebridge Config UI X Plugin Version
4.60.6
Operating System
Pi

Also getting this error. Did not do a re-install. Happy to do more debugging.

I have the same error :(
I have 3x Yale Conexis locks

Can I get more logs?

This is all that is showing up in the live log. Where else would there be more logs?
[August] Discover Devices 2: TypeError: Cannot convert undefined or null to object

Same error as well. Please let us know how to generate more logs. Thanks.

Go to config ui > advance setting, change logs to debug

Go to config ui > advance setting, change logs to debug

Sorry, can't see how to do that. Only have UI Settings and no debug option there. I set debug under the plugin itself and under Home Bridge settings, Environment Variables there is DEBUG however there is an empty text box next to it.

This is all of it (passwords etc changed)
[02/11/2023, 22:13:39] [August] Initializing August platform...
[02/11/2023, 22:13:39] [August] [DEBUG] Using Config Logging: debug
[02/11/2023, 22:13:39] [August] [DEBUG] Finished initializing platform: August
[02/11/2023, 22:13:39] [August] Platform Config: {"logging":"debug"}
[02/11/2023, 22:13:39] [August] [DEBUG] Using Default Refresh Rate (5 minutes).
[02/11/2023, 22:13:39] [August] [DEBUG] Config OK
[02/11/2023, 22:13:39] Loading 1 accessories...
[02/11/2023, 22:13:39] [DS1019+] Initializing synology accessory...
[02/11/2023, 22:13:39] [DS1019+] Synology DS1019+ finished initializing!
[02/11/2023, 22:13:39] [August] [DEBUG] Executed didFinishLaunching callback
[02/11/2023, 22:13:39] [August] [DEBUG] augustId<EMAIL_ADDRESS>installId: 6xxxxxxec5-4ec7-8ebxxxxxx33552, password: xxxxxxcfi4, isValidated: true
[02/11/2023, 22:13:39] [August] [DEBUG] August Credentials: {"config":{"apiKey":"xxxxxxx-47721289d1d9","pnSubKey":"sub-c-10xxxxxxx619f8945a4f","installId":"6axxxxxxx8eb2-8b54272xxxxxx","idType":"email","augustId":"si@hotmail.com","password":"hxxxxxx4","countryCode":"US"}}
[02/11/2023, 22:13:40] Homebridge v1.6.1 (HAP v0.11.1) (Homebridge D29C) is running on port 51873.
FetchError: Status 401 (undefined): undefined
[02/11/2023, 22:13:40] [August] Discover Devices 2: TypeError: Cannot convert undefined or null to object

Same error as well. Please let us know how to generate more logs. Thanks.

Here is mine:
[11/2/2023, 3:46:08 PM] [August] [DEBUG] Using Config Logging: debug
[11/2/2023, 3:46:08 PM] [August] [DEBUG] Finished initializing platform: August
[11/2/2023, 3:46:08 PM] [August] Platform Config: {"logging":"debug"}
[11/2/2023, 3:46:08 PM] [August] [DEBUG] Using Default Refresh Rate (5 minutes).
[11/2/2023, 3:46:08 PM] [August] [DEBUG] Config OK
[11/2/2023, 3:46:08 PM] [August] [DEBUG] Executed didFinishLaunching callback
[11/2/2023, 3:46:08 PM] [August] [DEBUG] augustId<EMAIL_ADDRESS>installId: bXXXXXXXX-XXXX-XXXX-XXXX-XXXXXXXXXXX, password: XXXXXXXXX, isValidated: true
[11/2/2023, 3:46:08 PM] [August] [DEBUG] August Credentials: {"config":{"apiKey":"7XXXXXXXX-XXXX-XXXXX-XXXX-XXXXXXXXXXX","pnSubKey":"sub-c-1XXXXXXXXX-XXXX-XXXX-XXXX-XXXXXXXXXX","installId":"bXXXXXX-XXXX-XXXX-XXXX-XXXXXXXXXXXX","idType":"email","augustId":"rightedges@gmail.com","password":"XXXXXXXXX","countryCode":"US"}}
[11/2/2023, 3:46:08 PM] Homebridge v1.6.1 (HAP v0.11.1) (August) is running on port 43273.
[11/2/2023, 3:46:09 PM] [August] Discover Devices 2: TypeError: Cannot convert undefined or null to object

I also need both of your configs to make sure you have everything set up correctly.

Click JSON Config and copy your Config. Please remove any sensitive info.

Here you go:
{     "name": "August",     "credentials": {         "augustId"<EMAIL_ADDRESS>        "password": "XXXXXXXX",         "contryCode": "US",         "installId": "bXXXXXX-XXXX-XXXX-XXXX-XXXXXXXXXXXX",         "isValidated": true,         "notice": "Keep your tokens a secret!"     },     "options": {         "logging": "debug"     },     "_bridge": {         "username": "0E:XX:XX:XX:XX:XX",         "port": 43273     },     "platform": "August" }

I'm sure the "me too" comments aren't much help. Here is my log. Just added this plugin for the first time but could t see any devices. Checked the logs and saw this.

successfully
[17/11/2023, 21:14:03] [August] [DEBUG] Using Config Logging: debug
[17/11/2023, 21:14:03] [August] [DEBUG] Finished initializing platform: August
[17/11/2023, 21:14:03] [August] Platform Config: {"logging":"debug"}
[17/11/2023, 21:14:03] [August] [DEBUG] Using Default Refresh Rate (5 minutes).
[17/11/2023, 21:14:03] [August] [DEBUG] Config OK
[17/11/2023, 21:14:03] [August] [DEBUG] Executed didFinishLaunching callback
[17/11/2023, 21:14:03] [August] [DEBUG] augustId: xxx, installId: xxx, password: xxx, isValidated: true
[17/11/2023, 21:14:03] [August] [DEBUG] August Credentials: {"installId":"xxx","augustId":"xxx","password":"xxx","countryCode":"US"}
[17/11/2023, 21:14:03] [August] [DEBUG] August Credentials: {"config":{"apiKey":"xxx","pnSubKey":"sub-c-xxx","installId":"xxx","idType":"email","augustId":"xxx","password":"xxx","countryCode":"US"}}
[17/11/2023, 21:14:04] Homebridge v1.6.1 (HAP v0.11.1) (August) is running on port 59244.
FetchError: Status 401 (undefined): undefined
[17/11/2023, 21:14:04] [August] Discover Devices 2: TypeError: Cannot convert undefined or null to object

I’m really sorry I know it doesn’t help anyone. I’m just so desperate to get the plug in working again. I just thought if there’s lots of people with the same problem it may get looked quicker. I’m very happy to make a donation if that helps
Thanks
Adrian
Sent from my iPhone
On 18 Nov 2023, at 02:23, Scott Meves @.***> wrote:

I'm sure the "me too" comments aren't much help. Here is my log. Just added this plugin for the first time but could t see any devices. Checked the logs and saw this.
successfully
[17/11/2023, 21:14:03] [August] [DEBUG] Using Config Logging: debug
[17/11/2023, 21:14:03] [August] [DEBUG] Finished initializing platform: August
[17/11/2023, 21:14:03] [August] Platform Config: {"logging":"debug"}
[17/11/2023, 21:14:03] [August] [DEBUG] Using Default Refresh Rate (5 minutes).
[17/11/2023, 21:14:03] [August] [DEBUG] Config OK
[17/11/2023, 21:14:03] [August] [DEBUG] Executed didFinishLaunching callback
[17/11/2023, 21:14:03] [August] [DEBUG] augustId: xxx, installId: xxx, password: xxx, isValidated: true
[17/11/2023, 21:14:03] [August] [DEBUG] August Credentials: {"installId":"xxx","augustId":"xxx","password":"xxx","countryCode":"US"}
[17/11/2023, 21:14:03] [August] [DEBUG] August Credentials: {"config":{"apiKey":"xxx","pnSubKey":"sub-c-xxx","installId":"xxx","idType":"email","augustId":"xxx","password":"xxx","countryCode":"US"}}
[17/11/2023, 21:14:04] Homebridge v1.6.1 (HAP v0.11.1) (August) is running on port 59244.
FetchError: Status 401 (undefined): undefined
[17/11/2023, 21:14:04] [August] Discover Devices 2: TypeError: Cannot convert undefined or null to object
—
Reply to this email directly, view it on GitHubhttps://github.com/donavanbecker/homebridge-august/issues/75#issuecomment-1817331810, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AO7OJ7GLK4PH2UB3AOC3SATYFAL2TAVCNFSM6AAAAAA54SXDVKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMJXGMZTCOBRGA.
You are receiving this because you authored the thread.Message ID: @.***>

Same issue, anything you need from me to assist in troubleshooting please let me know. I have an August 4th gen wifi lock. This is on a fresh install of home bridge, latest and greats node and npm and fresh install of your plugin.

this is an API issue.

Ok. Thank you. I appreciate your time for looking into it for me
Thanks
Adrian
Sent from my iPhone
On 22 Nov 2023, at 17:59, Donavan Becker @.***> wrote:

august-apihttps://github.com/hufftheweevil/august-api
—
Reply to this email directly, view it on GitHubhttps://github.com/donavanbecker/homebridge-august/issues/75#issuecomment-1823236206, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AO7OJ7AAJHZ6YNCBPJ6AG2TYFY4RFAVCNFSM6AAAAAA54SXDVKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMRTGIZTMMRQGY.
You are receiving this because you authored the thread.Message ID: @.***>

So I'm still having this problem. "Discover Devices 2: TypeError: Cannot convert undefined or null to object" There have been 2 updates to this plug-in but the error continues. Was there a fix that I missed? Any advice?

@scottgreiner keep your eyes peeled on https://github.com/hufftheweevil/august-api/issues/6, as that library provides the interface to the August API for this plugin. You can hit the button to get notified if there is any activity on the issue.

The issue is still occurring for myself and not resolved

The first time I install, it was working but after I had to restart Homebridge for another Plugins it started with the message, I dont know if it is a problem from August side after trying to reconnect.

sorry everyone. I am working on redoing the august-api module. If I am successful then I will be able to use it with this plugin.

Can you try the latest beta:

Click here to find out how to install Beta

Release v2.0.0, which is a major change. I am going to close this, if you are still having issues please open a new issue with logs. Thanks!


Item Filter (Client Side only)

I was wondering if its possible to have this clientside only and use a hotkey to open the menu rather then needing the mod on both the server and the client. (I'd rather not introduce a hard mod that players need to have to be able to join)

While I do understand that this would be useful, it's not really feasibly to implement as client-side functionality, as the server fundamentally decides when Item entities get picked up - the client has no say over this.
As such, I'll be closing this issue as wontfix
Cheers


IDVA6-1177 acsp menu navigation bar

Link to Jira:
https://companieshouse.atlassian.net/browse/IDVA6-1177
Changes:

Page header updated
Menu navbar added
Vulnerability in dependencies fixed

CI: No security warnings found

CI: No security warnings found


How-to-NPM Nodeschool module workaround needed

Lesson 10 of Node School's how-to-npm challenge doesn't work anymore. I think one of the dependencies changed.
We need to confirm that older version of the how-to-npm package works, or better yet, patch how-to-npm itself. Here's the (long open) issue: https://github.com/nodeschool/discussions/issues/995

I just got this error, going to investigate.. here's the error:
user@computer$ npm dist-tag add<EMAIL_ADDRESS>'wonderful'

npm ERR! Darwin 14.5.0
npm ERR! argv "node" "/usr/local/bin/npm" "dist-tag" "add"<EMAIL_ADDRESS>"wonderful"
npm ERR! node v0.12.7
npm ERR! npm  v3.5.2
npm ERR! code E404

npm ERR! 404 missing : -/package/how-to-npm-workshopper/dist-tags

npm ERR! Please include the following file with any support request:
npm ERR!     /Users/me/Documents/learning/nodeschool/how-to-npm/src/npm-debug.log

I suspect it has something to do with --tag being changed to dist-tag by npm recently and the workshopper just needs an update.

Still the same problem. The wokshopper is using --distag but still keep error ERR! 404 missing : ….

+1


fix(dapui): opening float element requires a debug session

Fix Float Element Can Be Open While There Is No dap Session Alive #398
This fix will detect debug session before opening a float element when executing require("dapui").float_element(). If there is no debug session found it will stop execution and prompt a warning "No active debug session"

Thanks!


Include test gutter icons in sticky scroll

Moved from https://github.com/Dart-Code/Dart-Code/issues/4610.
It'd be really nice for CodeLens and test icons to show up in sticky scroll:

I see there's an existing issue about CodeLens at https://github.com/microsoft/vscode/issues/182949, so this request is for the test icons to appear there too (red arrow).

Hi thanks for filing the issue. There are indeed many issues concerning the addition of decorators into sticky scroll : decorators in the form of indentation lines, gutter icons, code lens text, background colors.  In other words making the sticky scroll lines resemble the editor lines. The idea is on the roadmap, but I am not sure when we will be addressing it.

Hi @aiday-mar ,
I just noticed the same issue with my Bookmarks extension, which adds gutter and line/background decorators. Good to know it is on the roadmap.
Thank you for your feedback!

/duplicate of https://github.com/microsoft/vscode/issues/186166


settings.xml file from Neuropixels, Open Ephys

Hello everyone, I'm using spikeinterface to analyze Neuropixel (1.0) data from Open Ephys.  However, I'm missing the ProbeInterface probe object.
Perhaps I'm supposed to use the settings.xml generated by OE but not sure how.
settings.txt (I had to change the format from xml to txt in order to attach it)
I've also tried looking in ProbeInterface public library. And I'd like to avoid generating it from scratch. Does anyone know where I can find it?
Thank you! I'm new to this so your support is deeply appreciated.

Hi @phanhuynh.
Thanks you for this file.
I don't see any obvious section in this file that would contain the conatct contact position on NP probe.
Could you dig a bit into the open-ephys doc and send us a link on how to retrieve this locations please ?

I think that the channel locations are not stored in the settings for openephys. @jsiegle can you confirm this? In that case we can store all the locations for np1 and np2 probes in probeinterface and select the channels based on the settings

As of version 0.3.3 of the Neuropixels PXI plugin, the channel locations (in microns from the tip) are stored in the settings.xml file. There's also a button within the the Neuropixels settings interface that directly exports the probe settings in ProbeInterface JSON format.
See this page for full release notes.

@jsiegle, thank you!  I got the json file.
@alejoe91 , it's version 0.3.4-API7
Quick follow up:  what's the python code to set the probe in spikeinterface?  I tried:
probe = read_probeinterface('probe.json')
set_prob(probe)
Which doesn't work.  I feel like this should be obvious but I just can't find it online.  Thank you again.

What's the full error?

NameError                                 Traceback (most recent call last)
~\AppData\Local\Temp/ipykernel_9584/4189760159.py in
1 probe = read_probeinterface('probe.json')
----> 2 set_prob(probe)
NameError: name 'set_prob' is not defined

Note, however, that read_probeinterface does work.

Ah! yeah there is a typo!
This is what you should do:
import spikeinterface.full as si
from probeinterface import read_probeinterface

recording = si.read_openephys("...")

probe = read_probeinterface('probe.json')
recording.set_probe(probe, in_place=True)

si.plot_probe_map(recording)

Unfortunately, it gives me this error:

AssertionError                            Traceback (most recent call last)
~\AppData\Local\Temp/ipykernel_9584/2040178160.py in
1 probe = read_probeinterface('probe.json')
----> 2 recording.set_probe(probe, in_place=True)
~\anaconda3\envs\si_env\lib\site-packages\spikeinterface\core\baserecording.py in set_probe(self, probe, group_mode, in_place)
266         Wrapper on top on set_probes when there one unique probe.
267         """
--> 268         assert isinstance(probe, Probe), 'must give Probe'
269         probegroup = ProbeGroup()
270         probegroup.add_probe(probe)
AssertionError: must give Probe

Ah I see, it's a probe group. Use recording.set_probegroup(...) instead

I don't think I'm using a probe group (it's neuropixels), but here's the new error:
(note that there's a typo in the error message: set_proce)
Exception                                 Traceback (most recent call last)
~\AppData\Local\Temp/ipykernel_7760/3053197239.py in
1 probe = read_probeinterface('probe.json')
----> 2 recording.set_probegroup(probe, in_place=True)
~\anaconda3\envs\si_env\lib\site-packages\spikeinterface\core\baserecording.py in set_probegroup(self, probegroup, group_mode, in_place)
272
273     def set_probegroup(self, probegroup, group_mode='by_probe', in_place=False):
--> 274         return self.set_probes(probegroup, group_mode=group_mode, in_place=in_place)
275
276     def set_probes(self, probe_or_probegroup, group_mode='by_probe', in_place=False):
~\anaconda3\envs\si_env\lib\site-packages\spikeinterface\core\baserecording.py in set_probes(self, probe_or_probegroup, group_mode, in_place)
344         if in_place:
345             if not np.array_equal(new_channel_ids, self.get_channel_ids()):
--> 346                 raise Exception('set_proce(inplace=True) must have all channel indices')
347             sub_recording = self
348         else:
Exception: set_proce(inplace=True) must have all channel indices

Can you share both the settings.xml and the probe.json?

Hi @alejoe91 , here is the json file (I had to convert to txt for github):
probe.txt
The settings.xml file can be found in my first post in this issue (also converted to txt).
Thank you, again, so so much for all your attention :)

Hello @alejoe91 and @jsiegle , I just wanted to make sure that you're not waiting on me to provide anything.  Thank you for your help!

@phanhuynh can you try this?
probegroup = read_probeinterface('probe.json')
probe = probegrouup.probes[0]
recording_w_prb = recording.set_probe(probe)
print(recording_w_prb) # this should have one channel less than 'recording'

or:
probegroup = read_probeinterface('probe.json')
recording_w_prb = recording.set_probegroup(probegroup)
print(recording_w_prb) # this should have one channel less than 'recording'

The problem is you cannot use in_place because the open-ephys NPIX recording has an extra channel which is the SY channel. The suggest approach instead should work because it selects a subset of channels.

Hi @alejoe91 , I tried both:

For both, when I apply the sorters (tridesclous and spykincircus), the error says "ValueError: There is not Probe attached to recording. use set_probe(...)" :
ValueError                                Traceback (most recent call last)
~\AppData\Local\Temp/ipykernel_11252/1173793720.py in
5
6 # parameters set by params dictionary
----> 7 sorting_TDC_5 = ss.run_tridesclous(recording=recording, output_folder='tmp_TDC_5',
8                                    **default_TDC_params, )
~\anaconda3\envs\si_env\lib\site-packages\spikeinterface\sorters\runsorter.py in run_tridesclous(*args, **kwargs)
410
411 def run_tridesclous(*args, **kwargs):
--> 412     return run_sorter('tridesclous', *args, **kwargs)
413
414
~\anaconda3\envs\si_env\lib\site-packages\spikeinterface\sorters\runsorter.py in run_sorter(sorter_name, recording, output_folder, remove_existing_folder, delete_output_folder, verbose, raise_error, docker_image, singularity_image, with_output, **sorter_params)
65                                     delete_output_folder=delete_output_folder, verbose=verbose,
66                                     raise_error=raise_error, with_output=with_output, **sorter_params)
---> 67     return run_sorter_local(sorter_name, recording, output_folder=output_folder,
68                             remove_existing_folder=remove_existing_folder,
69                             delete_output_folder=delete_output_folder,
~\anaconda3\envs\si_env\lib\site-packages\spikeinterface\sorters\runsorter.py in run_sorter_local(sorter_name, recording, output_folder, remove_existing_folder, delete_output_folder, verbose, raise_error, with_output, **sorter_params)
86     SorterClass.set_params_to_folder(
87         recording, output_folder, sorter_params, verbose)
---> 88     SorterClass.setup_recording(recording, output_folder, verbose=verbose)
89     SorterClass.run_from_folder(output_folder, raise_error, verbose)
90     if with_output:
~\anaconda3\envs\si_env\lib\site-packages\spikeinterface\sorters\basesorter.py in setup_recording(cls, recording, output_folder, verbose)
169             all_params = json.load(f)
170             sorter_params = all_params['sorter_params']
--> 171         cls._setup_recording(recording, output_folder, sorter_params, verbose)
172
173     @classmethod
~\anaconda3\envs\si_env\lib\site-packages\spikeinterface\sorters\tridesclous\tridesclous.py in _setup_recording(cls, recording, output_folder, params, verbose)
86
87         # save prb file
---> 88         probegroup = recording.get_probegroup()
89         prb_file = output_folder / 'probe.prb'
90         write_prb(prb_file, probegroup)
~\anaconda3\envs\si_env\lib\site-packages\spikeinterface\core\baserecording.py in get_probegroup(self)
398             positions = self.get_property('location')
399             if positions is None:
--> 400                 raise ValueError('There is not Probe attached to recording. use set_probe(...)')
401             else:
402                 warn('There is no Probe attached to this recording. Creating a dummy one with contact positions')
ValueError: There is not Probe attached to recording. use set_probe(...)

Hi,
You have to run sorter with the recording_w_prb object.

Yes, thank you!  It seems to be working so far!  You guys are amazing!

Thank you!! :) :) :)


[Question] Can you provide register-specific instruction data with the help of GPT-4V?

Question
hi, author,
Thank you for your contribution. Could you please provide this dataset?

Hi Mary,
The dataset is here: https://huggingface.co/datasets/mucai/ViP-LLaVA-Instruct/tree/main
It is called vip-llava_stage3_mix.json, you can parse the id of each dictionary in the list, and you will find the GPT-4V instruction data!

Hi Mary,
The dataset is here: https://huggingface.co/datasets/mucai/ViP-LLaVA-Instruct/tree/main It is called vip-llava_stage3_mix.json, you can parse the id of each dictionary in the list, and you will find the GPT-4V instruction data!

OK! Thanks for your reply.


FileNotFoundError: [Errno 2] No such file or directory: 'parsing/pobp.csv'

Hello,
While using your Dataset class, I encountered a FileNotFoundError for parsing/pobp.csv. Here's the traceback for the error:
FileNotFoundError: [Errno 2] No such file or directory: 'parsing/pobp.csv'
This file seems crucial to properly initialize the Dataset class as it loads information into pobp_mapping.
Can you kindly look into this problem or provide guidance about acquiring the file?
Thanks in advance for your assistance!

Hey,
Sorry, these files apparently got excluded by an over-eager .gitignore. I have included them all in the last commit :)
Best Wishes,
Robin

Thanks for the response, I will close this issue.


texlive.bin.dvisvgm: texlive 2.8.1 -> upstream 2.11

Motivation for this change
Closes: #104847
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[x] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build tests.texlive.dvisvgm

@veprbl how?

@spacekitteh I thought you could use it the way you describe in https://github.com/NixOS/nixpkgs/issues/104847#issuecomment-733445591 and see if it produces broken svg. I don't know how you use nixpkgs, but perhaps you could use
nix-shell -p texlive.bin.dvisvgm -I nixpkgs=https://github.com/NixOS/nixpkgs/archive/9ef277209a5c7151da308244bbabc50b80864453.tar.gz

to get dvisvgm from this PR in a new shell environment.

Hmm, I'm not sure if this error is due to it being in nix-shell or not, but:
warning: kpathsea: configuration file texmf.cnf not found in these directories: /nix/store/f8skxd8hjzyh19l8r4pmvljb108m84nl-texlive-dvisvgm.bin-2.11/bin:/nix/store/f8skxd8hjzyh19l8r4pmvljb108m84nl-texlive-dvisvgm.bin-2.11/bin/share/texmf-local/web2c:/nix/store/f8skxd8hjzyh19l8r4pmvljb108m84nl-texlive-dvisvgm.bin-2.11/bin/share/texmf-dist/web2c:/nix/store/f8skxd8hjzyh19l8r4pmvljb108m84nl-texlive-dvisvgm.bin-2.11/bin/share/texmf/web2c:/nix/store/f8skxd8hjzyh19l8r4pmvljb108m84nl-texlive-dvisvgm.bin-2.11/bin/texmf-local/web2c:/nix/store/f8skxd8hjzyh19l8r4pmvljb108m84nl-texlive-dvisvgm.bin-2.11/bin/texmf-dist/web2c:/nix/store/f8skxd8hjzyh19l8r4pmvljb108m84nl-texlive-dvisvgm.bin-2.11/bin/texmf/web2c:/nix/store/f8skxd8hjzyh19l8r4pmvljb108m84nl-texlive-dvisvgm.bin-2.11:/nix/store/f8skxd8hjzyh19l8r4pmvljb108m84nl-texlive-dvisvgm.bin-2.11/share/texmf-local/web2c:/nix/store/f8skxd8hjzyh19l8r4pmvljb108m84nl-texlive-dvisvgm.bin-2.11/share/texmf-dist/web2c:/nix/store/f8skxd8hjzyh19l8r4pmvljb108m84nl-texlive-dvisvgm.bin-2.11/share/texmf/web2c:/nix/store/f8skxd8hjzyh19l8r4pmvljb108m84nl-texlive-dvisvgm.bin-2.11/texmf-local/web2c:/nix/store/f8skxd8hjzyh19l8r4pmvljb108m84nl-texlive-dvisvgm.bin-2.11/texmf-dist/web2c:/nix/store/f8skxd8hjzyh19l8r4pmvljb108m84nl-texlive-dvisvgm.bin-2.11/texmf/web2c:/nix/texmf-local/web2c:/nix/store:/nix/store/share/texmf-local/web2c:/nix/store/share/texmf-dist/web2c:/nix/store/share/texmf/web2c:/nix/store/texmf-local/web2c:/nix/store/texmf-dist/web2c:/nix/store/texmf/web2c.
WARNING: map file ps2pk.map not found
WARNING: map file pdftex.map not found
WARNING: map file dvipdfm.map not found
WARNING: map file psfonts.map not found
WARNING: none of the default map files could be found
pre-processing DVI file (format version 2)
WARNING: PostScript header file tex.pro not found
WARNING: PostScript header file texps.pro not found
WARNING: PostScript header file special.pro not found
WARNING: PostScript header file color.pro not found
PostScript error: undefined in TeXDict

I'll try with texlive.combined.scheme-full.

Yup! It works! <3

And by works it means also addresses the #104847?

Yup.

\o/


enable display of metrics and metadata in KFP UI

This PR

enables display of Elyra metadata in the KFP UI
enables display of user-provided metrics in the KFP UI
requires merge of https://github.com/elyra-ai/kfp-notebook/pull/64
closes #1033

Developer's Certificate of Origin 1.1
   By making a contribution to this project, I certify that:

   (a) The contribution was created in whole or in part by me and I
       have the right to submit it under the Apache License 2.0; or

   (b) The contribution is based upon previous work that, to the best
       of my knowledge, is covered under an appropriate open source
       license and I have the right under that license to submit that
       work with modifications, whether created in whole or in part
       by me, under the same open source license (unless I am
       permitted to submit under a different license), as indicated
       in the file; or

   (c) The contribution was provided directly to me by some other
       person who certified (a), (b) or (c) and I have not modified
       it.

   (d) I understand and agree that this project and the contribution
       are public and that a record of the contribution (including all
       personal information I submit with it, including my sign-off) is
       maintained indefinitely and may be redistributed consistent with
       this project or the open source license(s) involved.

Example output in KFP UI for a Python script node:

The Elyra specific output includes:

Heading "Inputs for <path/to/script_or_notebook_name>"
Link label:
Link url: <KFP_URL_from_runtime_config>/minio/<BUCKET_NAME_from_runtime_config>/<experiment_name>, aka the location on COS where the node's tar.gz file is stored and the output artifacts are located.  (The link will probably not work for third-party COS, such as IBM COS. However, it is probably likely that many KFP deployments utilize minio because it is installed as part of KFP.)

This PR requires additions to the documentation:

describe how users can generate output/metadata that is displayed in the UI

See https://github.com/elyra-ai/kfp-notebook/pull/64 for test coverage

Related doc issue: https://github.com/elyra-ai/elyra/issues/1056

I pushed one more fix
WCD = os.getenv('ELYRA_WRITABLE_CONTAINER_DIR', '/tmp').rstrip('/').strip()

should have been
WCD = os.getenv('ELYRA_WRITABLE_CONTAINER_DIR', '/tmp').strip().rstrip('/')

to normalize the path specification to "/mywritabledir" for these cases:

"/mywritabledir/"
"/mywritabledir"
"/mywritabledir/"
"/mywritabledir"

Interesting enough pathlib.Path doesn't remove whitespaces (e.g. Path('/tmp ') resolves to /tmp ) so maybe we shouldn't either in the unlikely event that the user's path does intentionally end with  .

The test failure is not caused by the changes in this PR:
URL: https://download.cypress.io/desktop/5.6.0?platform=linux&arch=x64
Error: Failed downloading the Cypress binary.
Response code: 503
Response message: Service Unavailable


Tampermonkey scripts' settings don't get saved

Not sure if that's the right place to post it, as I am a fairly casual user, but still.
I'm using a few Tampermonkey scripts and their settings reset after I reload the browser. I'm not exactly sure how Tampermonkey works and if there's a workaround that doesn't involve manually saving the settings as JSON and then manually reading those settings from JSON, a functionality that the scripts I use provide. My theory is that I'll have to enable cookies, but I'm worried that it might lower my privacy. Is there a good solution to my problem?

I wanna work on this project please assign me

You can fork the project, work on code and sumbit a pull request with your changes. There's no need for you to be assigned to anything. This also applies to #822


perf(Examples):Change GPIO settings in MAX32670 LP example to improve current consumption

Sets all unused GPIO pins to input mode with pull-down enabled.

/clang-format-run


feat: add getBySomeTags, rename getByTags to getByEveryTag

Types of changes

[ ] Chore (a non-breaking change which is related to package maintenance)
[ ] Bug fix (a non-breaking change which fixes an issue)
[x] New feature (a non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to change)

Description

Adds getBySomeTags and getAllBySomeTags. They query for documents with at least one of the given tags.
Renames getByTags and getAllByTags to getByEveryTag and getAllByEveryTag. They query for documents with all of the given tags.

Checklist:

[x] My change requires an update to the official documentation.
[x] All TSDoc comments are up-to-date and new ones have been added where necessary.
[x] All new and existing tests are passing.

This hasn't been documented yet right?
https://prismic.io/docs/technical-reference/prismicio-client?version=v6


build failure with "file RPATH_CHANGE could not write new RPATH"

I tried building 13.0-6 release aomp on Open Build Service but got error during installation:
[ 4384s] CMake Error at lib/Transforms/HC/PromotePointerKernArgsToGlobal/cmake_install.cmake:55 (file):
[ 4384s]   file RPATH_CHANGE could not write new RPATH:
[ 4384s]
[ 4384s]     $ORIGIN:$ORIGIN/../lib:$ORIGIN/../lib64:/home/abuild/rpmbuild/BUILD/aomp_13.0-6/lib
[ 4384s]
[ 4384s]   to the file:
[ 4384s]
[ 4384s]     /home/abuild/rpmbuild/BUILD/aomp_13.0-6/lib/LLVMPromotePointerKernArgsToGlobal.so
[ 4384s]
[ 4384s]   No valid ELF RPATH or RUNPATH entry exists in the file;
[ 4384s] Call Stack (most recent call first):
[ 4384s]   lib/Transforms/HC/cmake_install.cmake:47 (include)
[ 4384s]   lib/Transforms/cmake_install.cmake:58 (include)
[ 4384s]   lib/cmake_install.cmake:59 (include)
[ 4384s]   cmake_install.cmake:70 (include)
[ 4384s]

when used with AOMP_COMPONENTS="roct rocr project libdevice extras openmp pgmath flang flang_runtime comgr rocminfo vdi hipvdi ocl rocdbgapi rocgdb"
and
[ 4645s] -- Set runtime path of "/home/abuild/rpmbuild/BUILD/aomp_13.0-6/bin/llvm-cfi-verify" to "$ORIGIN:$ORIGIN/../lib:$ORIGIN/../lib64:/home/abuild/rpmbuild/BUILD/aomp_13.0-6/lib"
[ 4645s] -- Installing: /home/abuild/rpmbuild/BUILD/aomp_13.0-6/lib/libLLVMCFIVerify.a
[ 4645s] -- Installing: /home/abuild/rpmbuild/BUILD/aomp_13.0-6/bin/llvm-cov
[ 4645s] CMake Error at tools/llvm-cov/cmake_install.cmake:55 (file):
[ 4645s]   file RPATH_CHANGE could not write new RPATH:
[ 4645s]
[ 4645s]     $ORIGIN:$ORIGIN/../lib:$ORIGIN/../lib64:/home/abuild/rpmbuild/BUILD/aomp_13.0-6/lib
[ 4645s]
[ 4645s]   to the file:
[ 4645s]
[ 4645s]     /home/abuild/rpmbuild/BUILD/aomp_13.0-6/bin/llvm-cov
[ 4645s]
[ 4645s]   No valid ELF RPATH or RUNPATH entry exists in the file;
[ 4645s] Call Stack (most recent call first):
[ 4645s]   tools/cmake_install.cmake:66 (include)
[ 4645s]   cmake_install.cmake:77 (include)

with default components.
Here's full logs:
aomp-13.0-6_install-fail.log
aomp-13.0-6_install-fail_defcomponents.log

I'm getting the same error when compiling on openSUSE Leap 15.2 and 15.3 but openSUSE Tumbleweed works fine.

Try replicating build for aomp 14.0-2 .  I dont even see those cmake files in the amd-stg-open branch.

Try replicating build for aomp 14.0-2 . I dont even see those cmake files in the amd-stg-open branch.

Yep, doesn't happen.
I managed to make a build but OpenCL component doesn't seem to be there. Was it removed ?


canonical coordinates maps for background

How do you handle the background when estimating the canonical coordinates maps? As the background is empty, how do you estimate their canonical coordinates?

Hello, the background is randomly set to a value between 0 and 1.
More specifically, during the finetuning process of the UNet on the 64 x 64 x 4 latent space, the input consists of the CMM (64 x 64 x 3, which is blended using alpha blending with the random background) and the alpha channel (64 x 64 x 1). We will provide additional details about this process in the paper to clarify it further.
Hope this helps :)

thanks!


Linux version?

Is it already available for Linux? Or it will be if you can fix the existing issues?

What is your current issue so I can try to fix it?

@JVenberg You mean it's already available?

@ViBE-HU I just installed it on Ubuntu 16.04 without any issue. It can run properly.

Hm, first when I red the instructions I thought brew is an OSX only tools so I did not try to install it. Btw there is no clear information about installation and workarounds. I thought Linux is not supported. Now I tried to install and I get some red text while install. But it finished and here is the output when try to run it:
> electron main.js

sh: 1: electron: not found

npm ERR! Linux 4.4.0-31-generic
npm ERR! argv "/usr/bin/nodejs" "/usr/bin/npm" "start"
npm ERR! node v4.2.6
npm ERR! npm  v3.5.2
npm ERR! file sh
npm ERR! code ELIFECYCLE
npm ERR! errno ENOENT
npm ERR! syscall spawn
npm ERR<EMAIL_ADDRESS>start: `electron main.js`
npm ERR! spawn ENOENT
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start script 'electron main.js'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the PokemonGoBot package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     electron main.js
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs PokemonGoBot
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls PokemonGoBot
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /home/userPokemonGo-Bot-Desktop/npm-debug.log

and here is the log:
0 info it worked if it ends with ok
1 verbose cli [ '/usr/bin/nodejs', '/usr/bin/npm', 'start' ]
2 info using<EMAIL_ADDRESS>3 info using<EMAIL_ADDRESS>4 verbose run-script [ 'prestart', 'start', 'poststart' ]
5 info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>6 silly lifecycle<EMAIL_ADDRESS>no script for prestart, continuing
7 info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>8 verbose lifecycle<EMAIL_ADDRESS>unsafe-perm in lifecycle true
9 verbose lifecycle<EMAIL_ADDRESS>PATH: /usr/share/npm/bin/node-gyp-bin:/home/user/PokemonGo-Bot-Desktop/node_modules/.bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games:/snap/bin:/usr/lib/jvm/java-8-oracle/bin:/usr/lib/jvm/java-8-oracle/db/bin:/usr/lib/jvm/java-8-oracle/jre/bin
10 verbose lifecycle<EMAIL_ADDRESS>CWD: /home/user/PokemonGo-Bot-Desktop
11 silly lifecycle<EMAIL_ADDRESS>Args: [ '-c', 'electron main.js' ]
12 info lifecycle<EMAIL_ADDRESS>Failed to exec start script
13 verbose stack Error<EMAIL_ADDRESS>start: `electron main.js`
13 verbose stack spawn ENOENT
13 verbose stack     at ChildProcess.<anonymous> (/usr/share/npm/lib/utils/spawn.js:17:16)
13 verbose stack     at emitTwo (events.js:87:13)
13 verbose stack     at ChildProcess.emit (events.js:172:7)
13 verbose stack     at maybeClose (internal/child_process.js:821:16)
13 verbose stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:211:5)
14 verbose pkgid<EMAIL_ADDRESS>15 verbose cwd /home/user/PokemonGo-Bot-Desktop
16 error Linux 4.4.0-31-generic
17 error argv "/usr/bin/nodejs" "/usr/bin/npm" "start"
18 error node v4.2.6
19 error npm  v3.5.2
20 error file sh
21 error code ELIFECYCLE
22 error errno ENOENT
23 error syscall spawn
24 error<EMAIL_ADDRESS>start: `electron main.js`
24 error spawn ENOENT
25 error Failed at the<EMAIL_ADDRESS>start script 'electron main.js'.
25 error Make sure you have the latest version of node.js and npm installed.
25 error If you do, this is most likely a problem with the PokemonGoBot package,
25 error not with npm itself.
25 error Tell the author that this fails on your system:
25 error     electron main.js
25 error You can get information on how to open an issue for this project with:
25 error     npm bugs PokemonGoBot
25 error Or if that isn't available, you can get their info via:
25 error     npm owner ls PokemonGoBot
25 error There is likely additional logging output above.
26 verbose exit [ 1, true ]

It seems you need to build this module first https://github.com/electron/electron

Now I get this:
sudo npm install electron-prebuilt -g
loadDep:wrappy → network  ▀ ╢██████████████████████████████░░░░░░░░░░░░░░░░░░░░░/usr/local/bin/electron -> /usr/local/lib/node_modules/electron-prebuilt/cli.js

><EMAIL_ADDRESS>postinstall /usr/local/lib/node_modules/electron-prebuilt
> node install.js

sh: 1: node: not found
npm ERR! Linux 4.4.0-31-generic
npm ERR! argv "/usr/bin/nodejs" "/usr/bin/npm" "install" "electron-prebuilt" "-g"
npm ERR! node v4.2.6
npm ERR! npm  v3.5.2
npm ERR! file sh
npm ERR! code ELIFECYCLE
npm ERR! errno ENOENT
npm ERR! syscall spawn

npm ERR<EMAIL_ADDRESS>postinstall: `node install.js`
npm ERR! spawn ENOENT
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>postinstall script 'node install.js'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the electron-prebuilt package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node install.js
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs electron-prebuilt
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls electron-prebuilt
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /home/user/PokemonGo-Bot-Desktop/npm-debug.log

Ok, did you successfully do brew install python node? The nodejs is missing.
Try
curl -sL https://deb.nodesource.com/setup_6.x | sudo -E bash -
sudo apt-get install -y nodejs

(It may cause further issues, I should really save my terminal commands. I don't remember which tutorial I was using. Try to google linux install nodejs v6.)


Basic tests needed

I need to write basic unit tests for the construction of the 'factor' class.  I should especially be sure to document and test its invariants.

Tests should include:

exception for cardinality / value mismatch
permutation of unsorted scope variables
exception for empty scope
exception for empty values
comparison of constructed values
simple test of is_close()


IOS rotation effect

I use this plugin for changing orientation device and locking it. But the transition between landscape mode and portrait mode isn't good looking.
Is there a way to simulate the native rotation screen effect ? (particularly on Ipad IOS 8).
Thanks.

Bump!

Bump!

Unfortunately there is no control over the transition as iOS doesn't even support the ability to trigger a rotation in a view (this plugin is a hack).  I have looked into this before and have been unable to find a solution.  More than happy to accept pull requests or pointers to where others have addressed this issue.

I just had another look and it seems that iOS has changed the game again... It does look terrible now on 8.3, it was quite smooth on 8.0.  Will see what I can find.

Any update here?

Hi, recently I was changing orientation to play video on page, and the view usually breaks. I'm using ios 9.2 on iphone 5c, any suggestion?

1+

@mathieu94, @chineseneo - @gbenvenuti has released 1.4.2 which fixes some iOS issues. Unfortunately, I don't think this is going to address the "ugly orientation change" issue, but will you try 1.4.2 and provide an update, please?

Migrated to Apache Cordova JIRA CB-11361


Cannot create profile using Ledger

I get the following error when trying to create a profile:

Failed to create profile, reason: [object Object]

Platform: Ubuntu 21.04
Symbol wallet version: 1.0.2
Using with Ledger Live (v. 2.26.1) and Ledger Nano S (firmware 2.0.0). Both are latest version at time of writing.
What I did

Fresh install on Ubuntu 21.04
Ledger Live unlocked, incl. Symbol app open on Ledger Nano S (optional)
Open Symbol Wallet and select "Access Ledger"
Complete information as per screenshot. Click "Next"

Same error occurs also without step 2. I included this just in case this is supposed to be required.

Fixed in integration branch.


Secor is leaking memory through DeleteOnExit

Secor is exhausting the while heap over time and either crashes or starts spinning due to GC trying to free memory very frequently. Memory is accumulated in the DeleteOnExitHook by DeleteOnExit method. We are running Secor v0.25 and the related code seems to be the same in the latest version (which is v0.27 at the moment of writing).

We are getting the same problem - every few days secor crashes with identical Memory and CPu patterns. Filed under #964

any updates ?

This is interesting.  If you can reproduce the problem, can you turn on the debug logging category for FileRegistry.java which will prints out all the files added for deleteOnExit.
It would be interesting to see why there will be so many files added, we have never seen the memory problem on DeleteOnExitHook before and DeleteOnExitHook doesn't have a method to clear the map either.  One suspicion I have is the file size is set to too small (secor.max.file.size.bytes and secor.max.file.age.seconds) so it ends up creating too many files.

I think I have the same problem.
In my case, I have custom MessageParser implementation, which adds multi-level partitions, something like: /owner=1/workspace=2/event=click/dt=2020-07-07 in that case, we can get really big number of files and storing them in compressed parquet ads up to even biger memory consumption.
Any ideas on how to approach this problem?
Maybe we should pause Kafka consumers, when we reach X number of files, upload those files and resume? But in that case we'll potentially have many small files :thinking:.  We could also partition kafka topic by same values as we partition our data on  destination system, in that case consumed data would appear on same consumer thread and that would help us to have smaller number of files?

Looks good to me, I left a small comment in the code for backward compatible behavior.   And I don't think we should sort the files since that would change the file uploading order, some people might not like that.

@giner @dovka @yehoshoua Do you guys want to try https://github.com/pinterest/secor/pull/1443?

Thank you Henry.
is #1443 parameter going to limit the throughout?
Which release it will be available?

I haven't cut any release yet since this PR is just landed.  I think you can just pull that PR's classes into your local build to try it out, it's a very simple PR.   I don't think it will affect the upload throughput since most of your files are probably small files and you can tune the upload threshold parameter.

I'm wondering why using deleteOnExit at all? It could be fine if there was a limited number of files created during the process lifetime hover it's not the case with Secor and deleteOnExit doesn't release memory until the process is restarted.

Yes, my change solves only one part of the problem, it helps to keep FileRegistry HashMaps sizes under controll:
private HashMap<TopicPartitionGroup, HashSet<LogFilePath>> mFiles;
private HashMap<LogFilePath, FileWriter> mWriters;
private HashMap<LogFilePath, Long> mCreationTimes;

that was enough for me, to keep secor running without OOM for now.
It does not affect DeleteFileOnExit though.
It would be nice to use FileRegistry on the hook, so we have single list of active files, but since the hook is static, we cannot access that list :(.
What is the idea behind DeleteOnExit tracking list of files? Wouldn't it be sufficient to delete all staged files on exit?


Enable multiple dataset in validation_step

Allow validation step to use multiple datasets.
Need to also decide how it will be called (especially to handle a case where the two datasets aren't the same length.
Option A:
for batch_a in dataset_a:
    model.validation_step(batch_a, batch_nb, dataset_index)

for batch_b in dataset_b:
    model.validation_step(batch_b, batch_nb, dataset_index)

Option B:
for batches in zip(dataset_a, dataset_b):
    # use both
    # new dynamic signature validation_step(batch_a, batch_b, batch_n, batch_nb)
    model.validation_step(*batches, batch_nb)

Option C:
(I think this is the only real generic way). The user would have to make sure datasets are the same length or be ok with iterating only as far as the shortest one.
import itertools

for batches in zip(*itertools.chain(dataset_a, dataset_b, ..., dataset_n)):
    # use both
    # new dynamic signature validation_step(batch_a, batch_b, batch_n, batch_nb)
    model.validation_step(*batches, batch_nb)

@cinjon @ppwwyyxx

Anyone interested in implementing this?
Will need to make sure it's supported with distributed wrappers. Specifically, the DistributedDataset wrapper.

Another option would be to just concat the datasets with torch.utils.data.ConcatDataset.
It's kind of a quick fix but then I'm pretty sure that then the existing DistributedDataset wrapper should handle it the same.

@sidhanthholalkere yeah, actually that might be a better option.
I like deferring this stuff to PyTorch.
So to use 2 datasets the user would (according to this issue):
class ConcatDataset(torch.utils.data.Dataset):
    def __init__(self, *datasets):
        self.datasets = datasets

    def __getitem__(self, i):
        return tuple(d[i] for d in self.datasets)

    def __len__(self):
        return min(len(d) for d in self.datasets)

train_loader = torch.utils.data.DataLoader(
             ConcatDataset(
                 datasets.ImageFolder(traindir_A),
                 datasets.ImageFolder(traindir_B)
             ),
             batch_size=args.batch_size, shuffle=True,
             num_workers=args.workers, pin_memory=True)

My issue with that method is that some samples in the larger dataset will be left out. The only benefit would be that you get to validate a batch from both datasets at the same time, but in my experience, only the full validation loss(on whole validation set) matters(not per batch).
Now if the user wants separate losses for each set, i think the best option would be:
for dataset in datasets:
    for batch in dataset:
        model.validation_step(batch_a, batch_nb, dataset_index)

If they just want the validation on both sets combined, I still think that using the built in torch.utils.data.ConcatDataset here is better because it handles some error checking and would essentially do the same thing as option B and C

that looks great. Want to give it a shot and submit a PR?

I've started implementing this locally. I originally started by having validate() take in  dataset_index so validation_step() could have access to dataset_index so the user can name the outputs accordingly, ie:
return {'val_loss_{}'.format(dataset_index}: whatever}
Looking back, it feels like this adds unnecessary complexity because the user has to decide what to do with dataset index.
Now, I'm trying a new approach where validate itself can take in the list of val_dataloaders (instead of previously having to enumerate through them and passing dataset_index) and then just append whatever dataset_index to each of the result keys, ie:
output = {key+str(dataset_index) : value for key, value in output.items()} if len(dataloader) > 1 else outputs

With this new method, there's no need to enumerate through val.dataloader when calling validate AND the user doesn't need to handle dataset_index.
What do you think of this new method?
Now for writing tests, how should I write the tests?
Do you want me to just check if model.nb_val_batches is correct and if model.validate() works(since that is all i'm changing, at least in the new method)?
Also, for creating the test with two val_dataloaders, should I just use the default get_model() and override val_dataloader() to
@ptl.data_loader
def val_dataloader(self):
    print('val data loader called')
    return [self.__dataloader(train=False) for i in range(2)]

Interesting suggestion but I like the version with the dataset index, just pass it after batch_nb. This way it’s intuitive, users don’t have to read docs, and it remains fully flexible.
so the gist is:
for dataset in val_datasets:
for batch in dataset:
out = model.validation_step(batch, batch_nb, dataset_i)
It’s also backwards compatible and what a researcher would do if they had to implement it on their own.

I've added support for multiple val_dataloaders on my fork.
Is there a specific way you want me to write tests?
Also, unfortunately I won't be able to run most of the other tests locally as I don't have a multiple gpu machine/apex working for now.

awesome contribution!
for tests, can you add multiple val dataloaders to exampleModel used in tests?  then modify what’s returned to be a accuracy and val loss indexed by the dataset.
add a separate test to trainer where it inits dataloaders, then check that the loaders are correct (we don’t have such test yet).
make sure to also check that all val dataloaders are being wrapped in distributed dataset (there’s a warning for that).
i can run in gpus once you submit.

when you say exampleModel, do you mean the template model(lightning_module_template.py)?
Also, how would you want me to init multiple dataloaders, I could make a child of LightningTemplateModel where val_dataloader returns 2 dataloaders instead of one.
For the warning about val_dataloaders being wrapped, should I write another exception in trainer.py similar to the one that checks if ddp is used & if the tng_dataloader is a DistributedSampler.

@sidhanthholalkere

this is exampleModel https://github.com/williamFalcon/pytorch-lightning/blob/master/pytorch_lightning/testing/lm_test_module.py.

return 2 dataloaders from the  module above. That way some of the tests  use  a different model which returns 1 loader and other tests use this model which returns 2.
return [ds1, ds2]

just use

import warnings

warnings.warn('something to warn about')

4.in fact,  while you're there, could you remove the exception about distsampler and turn it into a warning?    that would solve: #81.

Ok, I've made the changes, here is a summary of what I've done:

Multiple val_dataloader support in trainer.py
Added 2 val_dataloaders for lm_test_module.py(its just the same one twice
Added an output to validation_step (if batch_i % 4 == 0) that has the losses/accuracies indexed by dataset
Warning for if val_dataloaders are not DistributedSamplers and ddp is selected
Test fit a model with multiple val_dataloaders and check if the length of the trainers val_dataloader is 2(not sure about this test)

Let me know if anything should be changed(its on my fork if you want to check)
Quick nitpick, in lm_test_module.py and in validation_step(), the acc and loss are named val_acc and loss_val, could I change it so the naming is consistent?
Also, in your test_models.py, multiple comments say "traning complete" so I can fix that in another PR

@sidhanthholalkere awesome! probably easier to make edits and comments on the PR itself!

@sidhanthholalkere 4 was taken care off in a different PR by someone yesterday

Any update?
I actually need to use two data loaders for validation and take the mean of the logits. I tried to return a list of loaders for val_dataloader but it does not work:
TypeError: 'DataLoader' object is not subscriptable.

@sidhanthholalkere

Fixing some errors, should be finished soon

@lorenzoFabbri @sidhanthholalkere merged! was not super trivial to verify haha


User CSV Import Not Functioning as Expected

🐛 Summary
I attached my CSV via the import file link but nothing happened.  And there's no button to proceed with import, or anything like that.  In fact, if I didn't hover of the import link, I would have had no indication the file was even uploaded :(

I suspect the import code was updated with the Oct2023 package because prior to the update, I had to convert the csv to xlsx for the UI to accept it.  Now it only sees csv.
To reproduce
Steps to reproduce the behavior:

Log in and "export" the current users - This step is only needed to create a CSV file in the proper upload filename and format
Edit the CSV file (remove the existing users and add the new ones)
Import the edited file (DON'T change the file name - it's just safer that way - trust me)

Expected behavior
I expected the Crossfeed to process the import file and add users - but nothing happened - nothing at all.
Any helpful log output or screenshots
Paste the results here:

Add any screenshots of the problem here.
I captured this screenshot while working with @nickviola and @cduhn17:

@aloftus23 - Can you confirm that the functionality to upload a list is working, there is just no UI/UX component for the User to understand the state of the upload. This may determine the priority of this bug.


Cleanup UnionArray Constructors

Is your feature request related to a problem or challenge? Please describe what you are trying to do.

The UnionArray constructors are somewhat at odds with the pattern followed for other array types.
Describe the solution you'd like

I would like the signature to be updated to be
fn try_new(fields: UnionFields, type_ids: ScalarBuffer<i8>, offsets: Option<ScalarBuffer<i32>>, children: Vec<ArrayRef>)
    -> Result<Self, ArrowError>

into_parts should then be updated to match
Describe alternatives you've considered

Additional context

label_issue.py automatically added labels {'arrow'} from #5585

label_issue.py automatically added labels {'arrow-flight'} from #5623


TypeError: _tokenize() got an unexpected keyword argument 'add_prefix_space'

Hi
In the preprocess, when we use albert model in 5_dump_features, we get this error:
TypeError: _tokenize() got an unexpected keyword argument 'add_prefix_space'
but it works fine for roberta model

Hi,
I hav the same problem as mohammadi.
I am using  bert in 5_dump_feature, I have the the error: _tokenize() got an unexpected keyword argument 'add_prefix_space'.
it works well with roberta.

Hi
In the preprocess, when we use albert model in 5_dump_features, we get this error:
TypeError: _tokenize() got an unexpected keyword argument 'add_prefix_space'
but it works fine for roberta model

I think your error comes from line 353:  sub_tokens = tokenizer.tokenize(token, add_prefix_space=True)
you can try to overwrtie it by:  sub_tokens = tokenizer.tokenize(token)


Export _NavigationDestinationInfo as public API

This exports _NavigationDestinationInfo as public API
This closes the Issue:
#139038
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[ ] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making, or this PR is test-exempt.
[ ] All existing and new tests are passing.

Hello! It is not clear to me what you can build with this class being public. Can you give some examples? Also, it would be good to cover this in a test prevent us from ever making this private again.
Can you also take a look at the failing analyze check above?

Hi @goderbauer! Thanks for your review.
Here is a rudimentary intended NavigationBar UI that can be easily accomplished with that widget being public (and waaay hard [if not impossible] without it)

The code for every component would be as this:
class _CustomDestination extends StatelessWidget {
  const _CustomDestination(this.icon, {super.key});

  final IconData icon;

  Widget build(BuildContext context) {
    final NavigationDestinationInfo info =
        NavigationDestinationInfo.of(context);
    return InkWell(
      onTap: info.onTap,
      child: Column(
        mainAxisAlignment: MainAxisAlignment.center,
        verticalDirection:
            info.index.isEven ? VerticalDirection.down : VerticalDirection.up,
        children: [
          Icon(icon),
          FadeTransition(
              opacity: info.selectedAnimation,
              child: Text(
                  'label ${info.index + 1}/${info.totalNumberOfDestinations}'))
        ],
      ),
    );
  }
}

I also added a test in navigation_bar_test.dart and this very example in navigation_bar.3.dart files. As a separate commit I added the example to the documentation of the published widget.
I have no idea on how to fix the failing analyze check though. Any help on that would be appreciated

@Rodsevich Looks like this is still failing some checks. Do you still have plans to come back to this and clean it up?

I tried fixing it but had no clue on how to debug the issue. Then life
happened and had no more time on finishing it. Is there any chance you can
continue by yourserf? If not, a clue on how to tacxle the issue will be
welcomed.
El vie, 15 de mar de 2024, 14:45, Michael Goderbauer <
@.***> escribió:

@Rodsevich https://github.com/Rodsevich Looks like this is still
failing some checks. Do you still have plans to come back to this and clean
it up?
—
Reply to this email directly, view it on GitHub
https://github.com/flutter/flutter/pull/139429#issuecomment-2000149408,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAORZBDDA3GMTQTAAYYW7Z3YYMXT7AVCNFSM6AAAAABAEH62D6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMBQGE2DSNBQHA
.
You are receiving this because you were mentioned.Message ID:
@.***>


fix: state serialization time

Motivation
There was a regression of v1.22 due to #7042 but it only affect n-historical-states configuration. This happens when we persist checkpoint state at every epoch

state serialization time was spiked because:

BufferPool is not used correctly
the metrics are recorded after state bytes are persisted to db

Description

release Uint8Array back to BufferPool with "using" key word
record the state serialization time before persisting state bytes to db
also refactor the metric to be used for both archive_state and persistent_checkpoints_cache_state flow

will follow up with Dashboard fix when this is merged
a prerequisite for #7104
Testing
It now takes less than 300ms to serialize a mainnet state

:tada: This PR is included in v1.23.0 :tada:


Support operator with no input

Describe your changes and why you are making these changes
One way to enable this is to make module-level variables for the dag and api_client objects. The following change seems to work.
Related issue number (if any)
Checklist before requesting a review

[x] I have created a descriptive PR title. The PR title should complete the sentence "This PR...".
[x] I have performed a self-review of my code.
[ ] I have included a small demo of the changes. For the UI, this would be a screenshot or a Loom video.
[ ] If this is a new feature, I have added unit tests and integration tests.
[ ] I have run the integration tests locally and they are passing.
[x] I have run the linter script locally (See python3 scripts/run_linters.py -h for usage).
[ ] All features on the UI continue to work correctly.
[x] Added one of the following CI labels:

run_integration_test: Runs integration tests
skip_integration_test: Skips integration tests (Should be used when changes are ONLY documentation/UI)

@kenxu95 addressed comments. I had to add dag variable reset logic to integration tests for some of them to pass. I think we had an assumption that when the client gets destroyed the dag var will be reset, which is no longer true, so I had to manually reset. I noticed that when I ran the integration test locally, "test_check_failure_with_varying_severity" is failing with a preview error, but it's not showing here. Not sure why.

@kenxu95 added an integration test.

@kenxu95 addressed all comments.


collect critical metrics

critical metrics

request duration histogram
request count
request error count
...

reference

https://github.com/grpc-ecosystem/go-grpc-prometheus
https://github.com/grpc-ecosystem/go-grpc-middleware?tab=readme-ov-file#observability

May you try to add some descriptions and related code?

I'd like to do this issue.

@JetSquirrel The issue description is updated with more information. Assigned and please go ahead :D

@yuanbohan  I instrument some metrics, do you think this metrics is enugh or we need add more?
And how about the format of metrics , it's as expected or need to modify?
$ curl localhost:4010/metrics

# HELP greptime_client_go_request_count
# TYPE greptime_client_go_request_count counter
greptime_client_go_request_count 3
# HELP greptime_client_go_request_duration_histogram
# TYPE greptime_client_go_request_duration_histogram histogram
greptime_client_go_request_duration_histogram_bucket{le="0.001"} 0
greptime_client_go_request_duration_histogram_bucket{le="0.01"} 3
greptime_client_go_request_duration_histogram_bucket{le="0.1"} 3
greptime_client_go_request_duration_histogram_bucket{le="0.3"} 3
greptime_client_go_request_duration_histogram_bucket{le="0.6"} 3
greptime_client_go_request_duration_histogram_bucket{le="1"} 3
greptime_client_go_request_duration_histogram_bucket{le="3"} 3
greptime_client_go_request_duration_histogram_bucket{le="6"} 3
greptime_client_go_request_duration_histogram_bucket{le="10"} 3
greptime_client_go_request_duration_histogram_bucket{le="20"} 3
greptime_client_go_request_duration_histogram_bucket{le="60"} 3
greptime_client_go_request_duration_histogram_bucket{le="+Inf"} 3
greptime_client_go_request_duration_histogram_sum 0.006428089
greptime_client_go_request_duration_histogram_count 3
# HELP grpc_client_handled_total Total number of RPCs completed by the client, regardless of success or failure.
# TYPE grpc_client_handled_total counter
grpc_client_handled_total{grpc_code="OK",grpc_method="Handle",grpc_service="greptime.v1.GreptimeDatabase",grpc_type="unary"} 3
# HELP grpc_client_msg_received_total Total number of RPC stream messages received by the client.
# TYPE grpc_client_msg_received_total counter
grpc_client_msg_received_total{grpc_method="Handle",grpc_service="greptime.v1.GreptimeDatabase",grpc_type="unary"} 3
# HELP grpc_client_msg_sent_total Total number of gRPC stream messages sent by the client.
# TYPE grpc_client_msg_sent_total counter
grpc_client_msg_sent_total{grpc_method="Handle",grpc_service="greptime.v1.GreptimeDatabase",grpc_type="unary"} 3
# HELP grpc_client_started_total Total number of RPCs started on the client.
# TYPE grpc_client_started_total counter
grpc_client_started_total{grpc_method="Handle",grpc_service="greptime.v1.GreptimeDatabase",grpc_type="unary"} 3

@JetSquirrel Thanks a lot for your time and commitment.
These metrics are enough for the first version. But what about change the name of some metrics?
We can discuss the following concerns:

all metrics names SHOULD BE started with a same prefix, like: greptimedb_ingester_go_ (or a better option)
unary or stream SHOULD BE a label in metric
greptime_client_go_request_duration (which is a Prometheus histogram) and greptime_client_go_request_duration_count contains the totol request. Is greptime_client_go_request_count needed?
what's the actual meaning message_sent and message_received?
what's the purpose of _started_total and _handled_total?

By the way, we should:

an option to export the /metrics endpoint or not (users may have already instrumented their apps already, and do not want sdk to export a port)
a Go function to get the metrics (users may have already instrumented their apps already, and want to merge sdk metrics into their own)

maybe we can discuss the metrics before you implement them. 👍

@JetSquirrel By the way, if you have time to acquire more practical and pragmatic patterns from the following links, it may help a better solution:

https://github.com/grpc-ecosystem/go-grpc-prometheus
https://github.com/grpc-ecosystem/go-grpc-middleware?tab=readme-ov-file#observability


Both en-sg and en-SG locales exist

Describe the bug
Both en-sg.js and en-SG.js locales exist on the latest release (2.25.0).
This is causing issues for our builds (using broccoli) as it can't resolve the conflicting capitalizations.

I'm having the same issue, and it's blocking me as several dependencies are pulling this version of moment.

Same, so I submitted a pr

Yes, we don't have anything in place to remove renamed locale files, as we never renamed locales before. Will release in a new patch version soon!

2.25.1 released


Missing OS reqs

I came to this page after learning the Defender for Endpoint only works on Server 2019. I can't see that limitation listed in any docs anywhere. This seems like the most sensible page to have a note on it

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

ID: 20c81719-d8d4-2eb3-a5a9-1cc35dd04bec
Version Independent ID: 8288bd95-d44e-8b1c-accb-0b221a71cd73
Content: Azure Security Center's features according to OS, machine type, and cloud
Content Source: articles/security-center/security-center-services.md
Service: security-center
GitHub Login: @memildin
Microsoft Alias: memildin

Other versions are also mentioned in this guide: https://docs.microsoft.com/microsoft-365/security/defender-endpoint/configure-server-endpoints?view=o365-worldwide

I will proceed to close this thread but if I'm missing something and you still think this should be added, feel free to tag me in the comments.


Add GeoServer 2.13.1 as default

Update the default GeoServer supported to the newest release 2.13.1.

Resolved in https://github.com/AppliedIS/dcos-geoserver/commit/5ed4292fa04cc83dfdacb26ff6827f10e4af4e8f


Horizontal bar chart

Please vote for this issue by adding a 👍 to help prioritize new features.

+1

This would really be helpful


Implement bulk deletion for services

Fix #1743

Codecov Report
Base: 61.30% // Head: 8.49% // Decreases project coverage by -52.81% :warning:

Coverage data is based on head (a9dd8c5) compared to base (f1c5981).
Patch coverage: 0.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #1790       +/-   ##
==========================================
- Coverage   61.30%   8.49%   -52.82%
==========================================
  Files         178      90       -88
  Lines       13649    9239     -4410
==========================================
- Hits         8368     785     -7583
- Misses       4223    8400     +4177
+ Partials     1058      54     -1004

Flag
Coverage Δ

acceptance-api
?

acceptance-apps
?

acceptance-cli
?

unittests
8.49% <0.00%> (-0.03%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

internal/api/v1/router.go
36.66% <ø> (-63.34%)
:arrow_down:

internal/cli/commons.go
0.00% <ø> (ø)

internal/cli/delete.go
0.00% <0.00%> (-50.00%)
:arrow_down:

internal/cli/services.go
8.59% <0.00%> (-56.17%)
:arrow_down:

internal/cli/usercmd/app.go
8.92% <0.00%> (-57.08%)
:arrow_down:

internal/cli/usercmd/client.go
61.90% <ø> (-23.81%)
:arrow_down:

internal/cli/usercmd/configuration.go
0.00% <0.00%> (-70.75%)
:arrow_down:

internal/cli/usercmd/service.go
0.00% <0.00%> (-67.28%)
:arrow_down:

pkg/api/core/v1/client/services.go
0.00% <0.00%> (-37.77%)
:arrow_down:

main.go
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 164 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

Checked working after release v1.4.0
➜  ~ epinio service delete svc-1 svc-2

🚢  Deleting Services...
Names: svc-1, svc-2
Namespace: workspace

✔  Services Removed.
Names: svc-1, svc-2
Namespace: workspace

➜  ~ epinio version
Epinio Version: v1.4.0
Go Version: go1.18.5


Enable —no-color on Windows

Fixes #326
Signed-off-by: David Gageot<EMAIL_ADDRESS>

Thanks @dgageot for another great contribution!


Use changed-log to report the commit messages for individual dependencies

Probably before reporting the install command, use bahmutov/changed-log to report what has changed for each working upgrade

Done, see video https://asciinema.org/a/dti7zzrqt5lm9u7qksimekxvs


Added 'long' profile in driver-mapping pom.xml.

'short' and 'long' tests in driver-mapping were not being executed in
jenkins because there was no 'long' profile defined in the pom.xml.
This will enable these tests to be run when using the 'long' profile.

Good catch.
You can commit this kind of change directly.


update status pages app version

The NR1-status pages app had a high priority feature added and we need to bump the app version
AC

Follow these instructions to bump this app for this PR .

I'm pretty sure the catalog manager app is broken... 😢 : I was going to bump the version for status-pages but I'm getting a ton of platform specific JS errors. is there anyway to bump an app WITHOUT using the Catalog manager?

updated the app in the US,  @LizBaker was able to use the cat manager locally. I requested to upgrade this app for the EU / Staging in the #help-NR1-Platform-Services channel

EU/Staging are updated.


Detect if on the filesystem and disable pushstate

CanJS pushstate routing doesn't make sense on the file system

Is this for the native apps? Same as https://github.com/donejs/donejs/issues/92 ?

Yes it is. Closing this one.


fail to get identity when reloading page

Description
Fail to get identity when reloading page
Steps to reproduce
Log in successful in https://eostoolkit.io/

Reload page

our web can not get identity
Versions

React-Boilerplate (see package.json):
Node/NPM:
Browser: Chrome

@RalfWeinand , Pls take a look at this: https://github.com/eostoolkit/eostoolkit/pull/130


feature/hyper-express <- stats api

Unlike master branch, this currently caches statistics in-memory, since storing to database is a tad too elaborate for me
Maybe in the future

@Pitu if there is nothing else that needs to be changed, this should be good to go

Instead of only listing mime types of non-images and non-videos, uploads' mime types map will contain all of them, sorted by count, then alphabetical
Future-proofing, if perhaps making them be clickable, to automate uploads searching gets implemented
Also added 1 "service" table for node/process-related information, instead of being categorized together under "system" table


[macOS] Fix PostgreSql pester test — take second split from the end

Description
The version output for postgresql has changed — we have (Homebrew) at the end of the output now so need to take not the last but the previous split part as a version in the test.

Related issue:
Check list

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

/azp run macos1015, macos11, macos12

/azp run macos1015, macos11, macos12


LimePlayer: how to

How to make and use LimePlayer? What are the required dependencies?

You need LimeSuite and SoapySDR.
https://wiki.myriadrf.org/Lime_Suite#Ubuntu_PPA
If you are interested in real-time GPS signal generation, try LimeGPS :)
https://github.com/osqzss/LimeGPS


Allows explicitly setting cache to 0, closes #158

This closes https://github.com/indexzero/http-server/issues/158
Should be ok to merge. I strongly feel this is the desired behavior - in normal circumstances you should be able to explicitly set max-age to zero.

It took me a while today to figure out why my changes are not visible in the browser. Thanks for that!
You've got my +1

+1


Can't edit NPC details

Creating an NPC in v0.3.2 I am not able to edit the NPC character sheet

NPCs are not implemented yet, but I plan on adding them soon. Short term you can use a character sheet for NPCs, since NPCs will be essentially a simplified character sheet

NPC sheets are implemented in v0.4.0!


megapixels: Drop OPNA2608 from maintainers

Description of changes
While I do still own & daily-drive my PinePhone, I never managed to get a proper Mobile NixOS installation running on it to really test this on intended hardware. And I've kinda given up on putting much effort into this thing, so that likely won't change anymore…
I haven't really given much input on the last few PRs for this package either, and pings that get ignored are helpful to no-one, so removing me from the maintainers of this package brings the information there closer to reality.
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

I own a PinePhone but don't use it at all and never got around to installing Mobile NixOS, so I guess there's no one using this package.

I'm using this package on my PinePhone. I'll add myself to the list next time I change something in that file.


[FEATURE] validate driver and server constraints on connect

Is your feature request related to a problem? Please describe.
One JDBC driver can be used to connect to different versions of the same database, or even different database kinds (e.g. Postgres wire protocol is used by different database venders). And different versions of JDBC might be used as well.
I my case, I want to validate the database product name and version in my Tableau connector.
Describe the solution you'd like
I would like to access and validate some attributes via JDBC DatabaseMetaData interface, e.g. getDatabaseProductName(), getDatabaseProductVersion(), getDriverName() and getDriverVersion(). If the validation fails, error it out and close connecting.
Regarding to the validation rules, I would like to be able to define a callback function somewhere so Tableau connector SDK can call it after connection is made.
Describe alternatives you've considered
No workaround exist for this problem.
About You
Name: Song Gao
Company: Starburst Data

Tracked with: W-14701240

Very useful feature.
Another use case:
Most of the database providers offer the same product as a self managed on-prem or fully managed SaaS.
Sometimes it makes sense to adapt Connector behaviour (ex. available authentication methods) based on the type of data source.


Update showcases.js

Name: RetroKydz
Description: An NFT collection of 2,000 3D pixelated generative unique little Kydz represented as NFTs on the Cardano blockchain. Each Kyd is randomly generated from over 225 hand-drawn rarities including: headwear, eyewear, pets, bags and more!
Link:   https://retrokydz.io
Source: null
Tags:

opensource
nft
tokens

Thank you for your contribution and for building on Cardano. This PR has been open for quite a while and I would like to give you a little feedback. Projects need 3 reviewer approvals and it seems the project didn't get enough "traction" among the reviewers.
Please note that showcase projects need to provide a unique value from existing showcase items - which is especially difficult with many tokens and NFT or NFT platforms. Feel free to join the discussion here: #243 (or in case you prefer to discuss this not in GitHub please join https://discord.gg/Exe6XmqKDx)
Suggestion: you can add your project to https://github.com/input-output-hk/essential-cardano


Hot-Update is not working when changes are saved

Description:
I have a composite application with dependencies to a synapse project.
I also have an EI server defined withing eclipse.  I'm able to add the composite application to the server from DevStudio and it gets deployed correctly, also I'm able to re-deploy correctly after some changes.
The problem is automatic hot-update seems not to be working at all, then when I make changes and save them, the CApp is not automatically re-deployed.
Suggested Labels:

Suggested Assignees:

Affected Product Version:
Devstudio-tooling-ei
OS, DB, other environment details and versions:
Oracle JDK 1.8.0_162
Eclipse Java EE IDE for Web Developers.  (bundle downloaded)
Version: Oxygen.3a Release (4.7.3a)
Build id: 20180405-1200
OS: Windows 10, v.10.0, x86_64 / win32
Steps to reproduce:
Create a synapse project
Add a proxy service
Add a property mediator to the in sequence
Add a Log mediator to log the value of the property
Create a Composite application project
Edit pom.xml and associate the recently created synapse project
Go to server's view in Java EE perspective, Add a new Server Enterprise Integrator
Ensure you select "enable hot update" check-box
Add the composite application to this server
Start the server from server's view in Eclipse
Call proxy and observe log in Console View
Modify property value in the proxy service
Save changes
Observe Console View...   No hot update happening
Related Issues:

@jairo-fernandez
This is the default behavior of the Integration Studio. Only the redeploy option of the capp will deploy the new changes. (Even server start/restart will not take the new capp changes into consideration. It just starts/restarts the server with existing capps)
When we make a change to a synapse artifact and save it, there is no way to know if this is the final version of the capp which needs to be deployed and tested. Hence, the capp redeploy option is given to the user to deploy the capp with new changes when needed.
We will be closing this issue, please feel free to reopen this for further clarification.


Type and provider refactor

Refactor swap_file to use type and provider
Deprecate usage of class to create swap_files (will remove in next major release)
Add defined type to use instead of class

If I was being pedantic (never) I'd say improve the type with better input validation and requisite tests. But other that the passing comments on a very brief run through looks OK.

@garethr  Good plan, I need to actually read the swapfile man pages and find out what some sensible validation is.
Thank's the review! :+1:

Ok, so the only thing I can validate is the parameter of file, which I've added in and added tests :+1:


Add cost of selected plan to statistics

I have implemented the aforementioned idea of an "experiment-runs" flag. See the following commit: https://github.com/LiUSemWeb/HeFQUIN/commit/b76aed8e1be894ad74c1c27280d5401aed4186da