Summa-Camels Data Resource Bugs and Coordination

Here is an error that is not clear for users.

This data is needed for netcdf files before we are able to make the resource public.

We're aware of this issue when uploading many files.  The 504 is due to timing out when processing all of the files.  You can watch this issue to see updates of where we are at with handling this situation better. https://github.com/hydroshare/hydroshare/issues/3852

https://help.hydroshare.org/hydroshare-resources/content-types/multidimensional/
While any NetCDF file can be used in HydroShare, we encourage use of Climate and Forecast (CF) and ESIP Attribute Convention for Data Discovery 1-3 (ACDD) conventions. HydroShare matches metadata from the header file with CF and ACDD convention tags to identify and populate content level and resource level metadata (see terms). When a NetCDF file with author, keyword, spatial and temporal reference, and other matched metadata is imported to HydroShare, there is no need for the user to enter this metadata as it will be extracted automatically.

@gantian127 is this the paper with the netcdf details? https://www.sciencedirect.com/science/article/abs/pii/S1364815219311661

@ChristinaB That is not the paper for the netCDF details. Please check this one: https://doi.org/10.1016/j.envsoft.2020.104706. Thank you.

[ ] move over data -- https://www.hydroshare.org/resource/b4c7120f38e843c8bdfc137ea1609989/

[ ] add netcdf detailed info - https://doi.org/10.1016/j.envsoft.2020.104706   - do we NEED a geospatial reference on every netcdf.  does this have to be used.  NOAA & NASA netcdf - do they have it?

[ ] talk to @Castronova  about installing hstools on Cheyenne so I can move it to https://www.hydroshare.org/resource/b4c7120f38e843c8bdfc137ea1609989/

[ ] collaboration hygiene planning

[ ] irods instructions from @hyi

@gantian127 there may be an issue with requiring spatial resolution to make a resource public.  If the netcdf is only lat/long degrees, the spatial resolution may not be included.
Also, great paper! do you have any other resources with tutorial or how to materials? I need to edit this Notebook so that the netcdf files output are in the correct format for HS resources so we can make use of  your cool features.

@sblack-usu  @martinseul   This resource CAMELS SUMMA Dataset - Preliminary Cheyenne Migration still has a weird title editing error from the glitch related to the CZO title for resources or is it related to any resource created by CZO resource owners?

@hyi how do I get a new irods password?  the password I expected is not working when I try to log in to irods from this CAMELS resource, and we need to upload 2G of climate data. I would prefer have the Notebooks run on Cheyenne (or any HPC) to send data directly to iRods...is this an option? Are there other existing examples?

@ChristinaB Here is the instruction: https://help.hydroshare.org/creating-and-managing-resources/how-can-i-reset-or-change-my-irods-password/ I can reset your iRODS password for you if you let me know your username (we can touch base offline). Yes, sending data directly to iRODS from the Notebooks running on any HPC should be an option if icommands is installed on the Notebook server. I believe @Castronova has Jupyter notebook examples to show how to do that. If not, I can send you example commands to show you how to do that.

@ChristinaB A public resource requires spatial coverage (bounding box) and the spatial reference of the netCDF data. Here is an example NetCDF file that works well with HydroShare metadata functions https://www.hydroshare.org/resource/deb0250d446441fd886a2e288b5a8a90/. But there is no specific how to tutorial.
For spatial coverage, one way to provide this information is to add global attributes in the NetCDF file by following the ACDD convention, which includes geospatial_lat_max, geospatial_lat_min, geospatial_lon_max, geospatial_lon_min. For spatial reference, it needs to define a grid mapping variable with attributes by following the CF convention (In the example NetCDF file, the grid mapping variable is "polar_stereographic"). Details for the attributes and the variable can be found from the file metadata ("NetCDF Header Information") of the example NetCDF. Thank you!

@gantian127  this is a good almost tutorial
for building a template https://github.com/gantian127/ueb-analysis/blob/bd716ac9062c18e07a788d7301a0ad31fd3cf64e/xmrg_to_netcdf_3.py    but I'll update it for netcdf4 and need to add the WGS for Summa or any latlong degree coordinates
@Castronova @AmberSJones what do the current instructions look like?

@sblack-usu can you please verify with @alvacouch if this is still an issue that needs to be addressed?

The 504 timeout issue can still occur if a user uploads several files at once, especially if there is a lot of extra processing required from extracting metadata.  There are other issues discussed here that seem to be more about building the resource than the HydroShare application.
Large file upload could limit the potential for the 504 timeout, but we could also implement some lower hanging fruit to avoid this error.

We can limit the number of files that can be uploaded simultaneously
We can suggest the user uploads a zip of the files and then unzips the file on HydroShare

@sblack-usu thanks for the quick review. Should this be classified as a bug or an enhancement?

The netcdf coverage issue has been addressed (with issue # 4592 and pr #4619).  The other points in this thread are stale or difficult to act on so closing this.  If there are specific items that need attention, please open a new issue about them.


Merge golint into main GHA

Signed-off-by: Gábor Lipták<EMAIL_ADDRESS>

@guillaumerose @n1hility please approve workflow


Update fb_dropIfExists on SQL Server to correctly escape identifiers

Rationale
Stored procedure fn_dropIfExists fails to bracket-escape table, schema, and other object names on SQL Server. This results in SQL exceptions when attempting to operate on names with spaces, special characters, or reserved words. One concrete example: attempt to drop the ETLTest schema (which has a table named "Delete").
Changes

Rid the core bootstrap scripts of calls to fb_dropIfExists
Reorder and coalesce the core and comm bootstrap scripts
Bracket all object names when used in SQL commands

The old and new versions of fn_dropIfExists are in different files, so here's a diff between the two to make review easier:
17a22,27
>
>     Note for implementors: Names that are part of SQL commands executed below should be bracketed to handle names that
>     have spaces, special characters, or reserved words. Names that are used in comparisons, in this code and in WHERE
>     clauses, must not be bracketed. @fullname is bracketed, since it's only used in SQL commands. All other vars are not
>     bracketed since they're often used in comparisons. When referencing @objname, @objschema, @subobjname, etc. be sure
>     to add brackets where required.
23c33
<   SELECT @fullname = (@objschema + '.' + @objname)
---
>   SELECT @fullname = ('[' + @objschema + '].[' + @objname + ']') -- @fullname is always bracketed, to handle names that are reserved words, etc.
31c41
<       ELSE IF @objname LIKE '##%' AND OBJECT_ID('tempdb.dbo.' + @objname) IS NOT NULL
---
>       ELSE IF @objname LIKE '##%' AND OBJECT_ID('tempdb.dbo.[' + @objname + ']') IS NOT NULL
33c43
<           EXEC('DROP TABLE ' + @objname )
---
>           EXEC('DROP TABLE [' + @objname + ']')
39c49
<       IF OBJECTPROPERTY(OBJECT_ID(@fullname),'IsView') =1
---
>       IF OBJECTPROPERTY(OBJECT_ID(@fullname), 'IsView') =1
48c58
<       SELECT @fullername = @fullname + '.' + @subobjname
---
>       SELECT @fullername = @fullname + '.[' + @subobjname + ']' -- Unlikely to require brackets, but doesn't hurt
66c76
<           EXEC('ALTER TABLE ' + @fullname + ' DROP CONSTRAINT ' + @subobjname)
---
>           EXEC('ALTER TABLE ' + @fullname + ' DROP CONSTRAINT [' + @subobjname + ']')
83,84c93,94
<           EXEC('ALTER TABLE ' + @fullname + ' DROP CONSTRAINT ' + @DEFAULT)
<           if (@printCmds = 1) PRINT('ALTER TABLE ' + @fullname + ' DROP CONSTRAINT ' + @DEFAULT)
---
>           EXEC('ALTER TABLE ' + @fullname + ' DROP CONSTRAINT [' + @DEFAULT + ']')
>           if (@printCmds = 1) PRINT('ALTER TABLE ' + @fullname + ' DROP CONSTRAINT [' + @DEFAULT + ']')
117,119c127,129
<                   SELECT @fullname = @fkSchema + '.' +@fkTableName
<                   EXEC('ALTER TABLE ' + @fullname + ' DROP CONSTRAINT ' + @fkConstName)
<                   if (@printCmds = 1) PRINT('ALTER TABLE ' + @fullname + ' DROP CONSTRAINT ' + @fkConstName)
---
>                   SELECT @fullname = '[' + @fkSchema + '].[' +@fkTableName + ']'
>                   EXEC('ALTER TABLE ' + @fullname + ' DROP CONSTRAINT [' + @fkConstName + ']')
>                   if (@printCmds = 1) PRINT('ALTER TABLE ' + @fullname + ' DROP CONSTRAINT [' + @fkConstName + ']')
143c153
<                   SELECT @fullname = @objschema + '.' + @soName
---
>                   SELECT @fullname = '[' + @objschema + '].[' + @soName + ']'
201,202c211,212
<                   EXEC('DROP SCHEMA ' + @objschema)
<                   if (@printCmds = 1) PRINT('DROP SCHEMA ' + @objschema)
---
>                   EXEC('DROP SCHEMA [' + @objschema + ']')
>                   if (@printCmds = 1) PRINT('DROP SCHEMA [' + @objschema + ']')
215c225
<       IF OBJECTPROPERTY(OBJECT_ID(@fullname),'IsProcedure') =1
---
>       IF OBJECTPROPERTY(OBJECT_ID(@fullname), 'IsProcedure') =1
280,281c290,291
<               if (@printCmds = 1) PRINT('DROP INDEX ' + @index + ' ON ' + @fullname + '-- index drop')
<               EXEC('DROP INDEX ' + @index + ' ON ' + @fullname)
---
>               if (@printCmds = 1) PRINT('DROP INDEX [' + @index + '] ON ' + @fullname + '-- index drop')
>               EXEC('DROP INDEX [' + @index + '] ON ' + @fullname)
292,294c302,304
<               SELECT @fkFullName = @fkSchema + '.' +@fkTableName
<               if (@printCmds = 1) PRINT('ALTER TABLE ' + @fkFullName + ' DROP CONSTRAINT ' + @fkConstName + ' -- FK drop')
<               EXEC('ALTER TABLE ' + @fkFullname + ' DROP CONSTRAINT ' + @fkConstName)
---
>               SELECT @fkFullName = '[' + @fkSchema + '].[' +@fkTableName + ']'
>               if (@printCmds = 1) PRINT('ALTER TABLE ' + @fkFullName + ' DROP CONSTRAINT [' + @fkConstName + '] -- FK drop')
>               EXEC('ALTER TABLE ' + @fkFullname + ' DROP CONSTRAINT [' + @fkConstName + ']')
304,305c314,315
<               if (@printCmds = 1) PRINT ('ALTER TABLE ' + @fullname + ' DROP CONSTRAINT ' + @ConstraintName + ' -- default constraint drop')
<               EXEC('ALTER TABLE ' + @fullname + ' DROP CONSTRAINT ' + @ConstraintName)
---
>               if (@printCmds = 1) PRINT ('ALTER TABLE ' + @fullname + ' DROP CONSTRAINT [' + @ConstraintName + '] -- default constraint drop')
>               EXEC('ALTER TABLE ' + @fullname + ' DROP CONSTRAINT [' + @ConstraintName + ']')
316,317c326,327
<             if (@printCmds = 1) PRINT ('ALTER TABLE ' + @fullname + ' DROP CONSTRAINT ' + @ConstraintName + ' -- other constraint drop')
<             exec ('ALTER TABLE ' + @fullname + ' DROP CONSTRAINT ' + @ConstraintName)
---
>             if (@printCmds = 1) PRINT ('ALTER TABLE ' + @fullname + ' DROP CONSTRAINT [' + @ConstraintName + '] -- other constraint drop')
>             exec ('ALTER TABLE ' + @fullname + ' DROP CONSTRAINT [' + @ConstraintName + ']')
321,322c331,332
<           if (@printCmds = 1) PRINT ('ALTER TABLE ' + @fullname + ' DROP COLUMN ' + @subobjname )
<           EXEC('ALTER TABLE ' + @fullname + ' DROP COLUMN ' + @subobjname )
---
>           if (@printCmds = 1) PRINT ('ALTER TABLE ' + @fullname + ' DROP COLUMN [' + @subobjname + ']')
>           EXEC('ALTER TABLE ' + @fullname + ' DROP COLUMN [' + @subobjname + ']')
342c352
<     RAISERROR('Invalid object type - %s   Valid values are TABLE, VIEW, INDEX, CONSTRAINT, DEFAULT, SCHEMA, PROCEDURE, FUNCTION, AGGREGATE, SYNONYM, COLUMN', 16,1, @objtype )
---
>     RAISERROR('Invalid object type - %s   Valid values are TABLE, VIEW, INDEX, CONSTRAINT, DEFAULT, SCHEMA, PROCEDURE, FUNCTION, AGGREGATE, SYNONYM, COLUMN', 16, 1, @objtype )
346a357
> GO


KeyError: 'MatchID'

Region: RU
Server: EU

Should be fixed if you changed region in %appdata/valorant-rpc :)

can you make translate for all region?

I mean it is in readme tho

EU work, but RUS not

change it to EU and it should work

Obviously, there is no RUS server. There is only EU, KO, NA and AP.


Exclude nursery clippy (#18)

Better avoid experimental lints, unless there is a clear benefit.

Or just rebasing over main would work


ImportError: No module named importlib after install

Hi guys :)
After install django-rest-auth i got this message:

Traceback (most recent call last):
File "manage.py", line 12, in
execute_from_command_line(sys.argv)
File "lib/djangae/core/management/init.py", line 65, in execute_from_command_line
return _execute_from_command_line(namespace.sandbox, ['manage.py'] + namespace.args, **overrides)
File "lib/djangae/core/management/init.py", line 45, in _execute_from_command_line
return django_management.execute_from_command_line(argv)
File "lib/django/core/management/init.py", line 353, in execute_from_command_line
utility.execute()
File "lib/django/core/management/init.py", line 327, in execute
django.setup()
File "lib/django/init.py", line 18, in setup
apps.populate(settings.INSTALLED_APPS)
File "lib/django/apps/registry.py", line 115, in populate
app_config.ready()
File "lib/debug_toolbar/apps.py", line 15, in ready
dt_settings.patch_all()
File "lib/debug_toolbar/settings.py", line 228, in patch_all
patch_root_urlconf()
File "lib/debug_toolbar/settings.py", line 216, in patch_root_urlconf
reverse('djdt:render_panel')
File "lib/django/core/urlresolvers.py", line 568, in reverse
app_list = resolver.app_dict[ns]
File "lib/django/core/urlresolvers.py", line 360, in app_dict
self._populate()
File "lib/django/core/urlresolvers.py", line 293, in _populate
for pattern in reversed(self.url_patterns):
File "lib/django/utils/functional.py", line 33, in get
res = instance.dict[self.name] = self.func(instance)
File "lib/django/core/urlresolvers.py", line 417, in url_patterns
patterns = getattr(self.urlconf_module, "urlpatterns", self.urlconf_module)
File "lib/django/utils/functional.py", line 33, in get
res = instance.dict[self.name] = self.func(instance)
File "lib/django/core/urlresolvers.py", line 410, in urlconf_module
return import_module(self.urlconf_name)
File "/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/importlib/init.py", line 37, in import_module
import(name)
File "/Users/briva/Desktop/owls_global/owls_core/core/urls.py", line 26, in
url(r'^rest-auth/', include('rest_auth.urls')),
File "lib/django/conf/urls/init.py", line 52, in include
urlconf_module = import_module(urlconf_module)
File "/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/importlib/init.py", line 37, in import_module
import(name)
File "lib/rest_auth/urls.py", line 3, in
from rest_auth.views import (Login, Logout, UserDetails, PasswordChange,
File "lib/rest_auth/views.py", line 12, in
from .app_settings import (TokenSerializer, UserDetailsSerializer,
File "lib/rest_auth/app_settings.py", line 10, in
from .utils import import_callable
File "lib/rest_auth/utils.py", line 2, in
from django.utils.importlib import import_module
ImportError: No module named importlib

Someone did view this bug ?

@briva Seems like you're using an old version of DRA, upgrading to the most recent will resolve your problem.


OperatorHub and OLM version planned?

Hello
Is there anything on the roadmap regarding making available your operator on Operator Hub and having it managed via Operator Lifecycle Manager (OLM)? I am asking in an openshift 4.8 / 4.9 Context, Kubernetes 1.22.
From an ops perspective, I find it strange that e.g. the Google Cloud platform version of the operator is not maintained.
Are there any plans to involve Google and especially RedHat more to make flink on K8s / openshift 4.9 truly production-ready?
I am talking about financial services, medical services here.

I'm also curious whether it is alive as I had hard times getting connected with the author and there is only 1 maintainer. We are looking into moving out of Lyft's operator, and it seems like one from Google is dead.
There is a new one from Apple under the Apache roof as well, I'm starting to think it will have a better feature.

Hello Alex: Very nice indeed. I think both users and developers should focus on that one
https://flink.apache.org/news/2022/04/03/release-kubernetes-operator-0.1.0.html
https://github.com/apache/flink-kubernetes-operator

Hello, Thanks for bringing this up!

Is there anything on the roadmap regarding making available your operator on Operator Hub and having it managed via Operator Lifecycle Manager (OLM)? I am asking in an openshift 4.8 / 4.9 Context

There are no plans since we don't use openshift. However, if the community is interested and it's willing to help we can make it happen.

From an ops perspective, I find it strange that e.g. the Google Cloud platform version of the operator is not maintained.

The reason is unclear to us as well and several attempts were made through the community to clarify what was going on but not avail. That's this fork started! In a way I'm actually happy with it because it allows us to move faster and add the features we need.
We (Spotify) are using this for production use cases and even though there's still a lot to be done we will continue to work on this project. We do have some new upcoming work planned to better support batch and streaming use cases. On a side note, it might seem just one dev backing this up, but there's a team 😄
The apache project is still fairly new and it will take some time to catch up to our needs plus we will lack in some way the ability to move fast.
With that said, it will be up to the user to decide which flavor suits better their needs right now.
Thanks!

Yes, or go with a product like Ververica, if production-readiness and commercial support is needed.

Thank you for your feedback, it helped surveying the landscape and possibilities.


Squid Parameters Manager

the test writesReportsFromPrawnFile fails because of a call to new Task()... not sure yet what the issue is

do make sure to change the left side of your model declarations to "ParametersModel":
public void setReferenceMaterial(ParametersModel refMat);
public ParametersModel getReferenceMaterial();
public void setPhysicalConstantsModel(ParametersModel physConst);
public ParametersModel getPhysicalConstantsModel();
public void setCommonPbModel(ParametersModel model);
public ParametersModel getCommonPbModel();

be sure to rename PbBlankICModel to be commonPb

the problem is when you use public static PhysicalConstantsModel defaultPhysicalConstantsModel = getDefaultModel("EARTHTIME Physical Constants Model", "1.1");

make this call as part of the task constructor instead ... then you can test if it excepts or not

also I am getting many exceptions from XStream - I deleted my lab data and started the app and
the build fails

caveat to that last comment - it works when I restore your code ... the problem is with the design not supporting the test mode as discussed.

suggestion -  make "sigfigs" a spinner from 1 to 12; as it is you can enter negative numbers; a spinner is more elegant . (see task manager for example - use integer instead of double tho)

looking good overall

Common Pb needs additional ratios:
207Pb/206Pb
208Pb/206Pb

It would help readability if a light grid appeared in the tables

Great work!

if you open common lead first, the menu items under edit are not properly initialized relative to enable
let's make physical constants the third flavor in menus and tabs
change all left-side references to the class ParametersModel and then
add another attribute for Task:
protected ParametersModel concentrationReferenceMaterial;
this is not a new model type, rather the task will have two referenceModels
make class ParameterModels abstract
in ParameterModels, chain the constructors so you do not repeat assignments - see Task constructors for example)
the location calculations for the labdata window do not change when the window moves; also, if I have it open and then click on a menu item, it jumps to a new location
add a space between Squid and LabData in the menu

that's it for now...

When you "save and register" a current edit, it will be saved in the .ser file in squid3 user data. You can export as xml through the file menu... I can change how that works if you want.

Looking great!
A couple tweaks:

detect if the four models are null in a task (i.e. any existing task from before this lab data) and set them to the defaults.

The drop downs for Ref Mat (and probably the others) are not showing names in correct order.

I chose a model for each and saved the project and exited Squid.  When I re-opened, the drop down boxes did not display my choices until I clicked on them.  I realize we are going to have to add small labels to the left of each (Ref Mat, Conc Ref Mat, Phys Const, Common Pb) also.

Almost there!

Great work by @ryanbarrett1515 !


Atom always in setup mode

I've downloaded Atom recently and I can't find it ok the quick start. I have the atom setup.exe.
Everytime I open it it says installing, then I restart my Laptop and it still the setup version

Thanks for reaching out!
We require the template to be filled out on all new issues and pull requests. We do this so that we can be certain we have all the information we need to address your submission efficiently. This allows the maintainers to spend more time fixing bugs, implementing enhancements, and reviewing and merging pull requests.
It sounds like you might need to check where your OS has applications installed to for where to launch Atom? The installer isn't going to turn into a shortcut for the app.
Thanks for understanding and meeting us halfway :grinning:


duplicate babble_max_topic_size

Lines 22-24 were duplicating lines 11+12, but making this setting hidden. Since you've also added a UI text for this item, I guess this is an oversight... ;)

Ah, yeah, thanks.


Is the SDK actually required to install tools?

This page states:

The only complicating factor of using these tools in a container is that they are installed with the .NET Core SDK and many Docker containers run without the .NET Core SDK present. One easy solution to this problem is to install the tools in the initial Docker image. The tools don't need the .NET Core SDK to run, only to be installed.

Is this correct, or is that only true of the dotnet tool method of installation? I noticed the tools have stand-alone downloads available, for example:
https://docs.microsoft.com/en-us/dotnet/core/diagnostics/dotnet-dump#install
Do those installers still require the SDK? I'm trying to put together some documentation for production server support, and corporate policy prohibits SDK installation for obvious reasons.
I'm happy to PR a change, or if the stand-alone installers do require the SDKs, I'll PR changes to those pages instead (and then we have big trouble internally trying to figure out how to deal with that...)

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

ID: cb469ec9-2273-52d7-059d-8288eb6d8db5
Version Independent ID: d24b97d5-8931-408e-20e0-78c4787365c0
Content: Collect diagnostics in containers
Content Source: docs/core/diagnostics/diagnostics-in-containers.md
Product: dotnet-fundamentals
GitHub Login: @tommcdon
Microsoft Alias: tommcdon

@mjrousos Could you answer this question? Also, should you be marked as the author of this article?

The answer, per our internal Microsoft DSE:

Dotnet tools are nuget packages that get installed to a bin on the PATH. You need the DSK to use the tool installer. The actual utility is just an exe that you don't need the SDK to use. You can just copy it to where you need it from the download link on the docs page.

@hoyosjs @mikem8361

Is this correct, or is that only true of the dotnet tool method of installation? I noticed the tools have stand-alone downloads available

We also have direct download links documented here for framework: https://docs.microsoft.com/dotnet/core/diagnostics/dotnet-dump

Do those installers still require the SDK?

The dotnet-dump tool is framework dependent so it does require the .NET Runtime but it does not require the .NET SDK.


Watchtowers

@JSwambo wdyt ?

First we should probably state that the communication is secure (by an assumed correct set-up ceremony).
We should state that wallet <-> watchtower communication is only for associated wallet-WT pairs.
Wouldn't it be more efficient to have the sync server post spend requests to the WT rather than have the WT regularly poll the sync server?
Probably not the priority now but it would be good to get some WT spend-policy examples? Perhaps a set of templates that restrict amounts/ destinations/ subsets of traders/ time.
Wouldn't it be more efficient to have:
 SPENDER's WALLET                SYNC_SERVER
    ||   -- request_spend    ---->   ||  // I'd like to spend this vault.
    ||   <------- ACK ------------   ||  //  Acknowleged, getting oppinions.
    ||                      (...)                   ||  // Sync Server communicates with WTs
    ||  <--- spend_opinions  ---    ||  // Eventually all watchtowers responded.

sig_ack:
A reasonable balance for the WT to hold would be something which can historically (e.g. 99% of time) broadcast a tx which can be confirmed in 6 blocks. The total balanced doesn't need to be used, just available for ensuring security through availability of the revocation txs.
spend_requests:
I notice there is a small risk that if each WT receives a fully signed spend tx, but not all WTs agree based on their spend policy, then the spend tx becomes a tx that any WT can broadcast to trigger the non-accepting WT to broadcast the revocation txs. Of course we want the watchtower to be able to revoke when it disagrees, but perhaps restricting malicious spend tx attempts to the wallet owners (rather than their WTs) leaves less room for DoS by compromised WTs? If we decide this is a significant risk, the countermeasure might be to provide a proof-of-knowledge of the set of valid signatures over the spend tx, instead of the signatures themselves.
spend_opinion:
I think an extra signature is unnecessary with secure communications already set-up.

First we should probably state that the communication is secure (by an assumed correct set-up ceremony).

Yep, but even if it clarifies some of the choices made here I'm not sure it belongs to messages.md.

We should state that wallet <-> watchtower communication is only for associated wallet-WT pairs.

Absolutely, I'll amend if it it wasn't made explicit.

Wouldn't it be more efficient to have the sync server post spend requests to the WT rather than have the WT regularly poll the sync server?

It would. However having the watchtower initiate every connection to the sync server avoids us to have both the watchtower exposing a service (even a hidden one) to another entity than its paired wallet, and to have the (untrusted) sync server not knowing anything about how to connect the (so crucial) watchtowers.

Wouldn't it be more efficient to have:
--snipped--

You mean to have the sync server explicitly ACK, instead of having the wallet polling to see if it updated itself ?
I don't think it's more efficient. It's for sure less explicit but has the nice property that we would never end up in a weird state (in case of a sneaky bug or anything) where the sync server would have ACK but not updated. By calling the same method the watchtower will call for retrieving our demand, we can just make the wallet re-transmit the request if it looks like the sync server dropped it.

sig_ack:
A reasonable balance for the WT to hold would be something which can historically (e.g. 99% of time) broadcast a tx which can be confirmed in 6 blocks. The total balanced doesn't need to be used, just available for ensuring security through availability of the revocation txs.

Hmm in this case we hope for the feerate to never pass above the historical higher one ?
I think we should choose xsats such as x = [2/CONSERVATIVE feerate] * [sum of the size of all cancel tx of all vaults].

spend_requests:
I notice there is a small risk that if each WT receives a fully signed spend tx, but not all WTs agree based on their spend policy, then the spend tx becomes a tx that any WT can broadcast to trigger the non-accepting WT to broadcast the revocation txs.

No, a watchtower cannot trigger another as no WT has any unvault transaction signature :).
However you pointed out a real concern here: DoS by a compromised wallet.

spend_opinion:
I think an extra signature is unnecessary with secure communications already set-up.

I think you are right: as the sync server bridges these ACKs the wallet does need a sig from the WTs or it trusts the server.

Added a commit (which I couldn't squash because I renamed the file in the commit in-between..) to be clearer about the wallet-watchtower pair, the communication channel, and to add a signature to the spend_opinion message.

Hmm in this case we hope for the feerate to never pass above the historical higher one ?
I think we should choose xsats such as x = [2/CONSERVATIVE feerate] * [sum of the size of all cancel tx of all vaults].

Why 2/ Conservative feerate? I would have thought: x = (high feerate)*(sum of the size of all cancel tx of all vaults)

spend_requests:
I notice there is a small risk that if each WT receives a fully signed spend tx, but not all WTs agree based on their spend policy, then the spend tx becomes a tx that any WT can broadcast to trigger the non-accepting WT to broadcast the revocation txs.

No, a watchtower cannot trigger another as no WT has any unvault transaction signature :).

I meant when the unvault is eventually broadcast. Then the WT can broadcast the rejected spend tx and trigger re-vault.

Why 2/ Conservative feerate? I would have thought: x = (high feerate)*(sum of the size of all cancel tx of all vaults)

Yeah. Means the same things but 2/CONSERVATIVE is the higher "reasonable" feerate bitcoind will give you for estimatesmartfee.

I meant when the unvault is eventually broadcast. Then the WT can broadcast the rejected spend tx and trigger re-vault.

Hmm. Yeah, ok this is a possibility. However there would be a race after the CSV expiration between the valid and invalid spend_tx, and the attack being overt if it succeeds the users won't play the game twice.

I'd like to merge this. If the force'em-to-revault DoS is the last concern about the changes, we can discuss it in an issue ?


Can't set up the integration: problem while starting the configuration flow

Hey there,
I got stuck while activating the integration in Home Assistant (2023.7.3 on a Raspberry Pi 4b). I was able to install the integration via HACS but run into a problem when I try to add it:

I click on "Add integration"
I select the "Bird Buddy" integration
Dialog tells me for about 20-30 seconds that the configuration flow is being started
I get the error message that the configuration flow couldn't be loaded, incl. the 404 error message of my reverse proxy (Synology NAS)

As I don't see any errors in the HA log at all, I'm kind of stuck here - especially as other integrations can be installed without a problem. Does this integration rely on the localhost's IP or something during the setup process? Or does someone has any other ideas?
Any help is highly appreciated. :)

The first and only page of the config flow is requesting your BB credentials. And there are no network requests performed by the integration before that, until after it has the credentials (to confirm the login and proceed to setting up the config entry).
So if it's hanging before that, then it's more likely failing to install the dependencies. The integration requires these python packages, which normally HA would install as part of installing the integration:

pybirdbuddy
python-graphql-client
langcodes

Normally if and of that fails to install though, you should see it in the logs. Do you have any trouble accessing pypi?

Thanks for pointing me in the right direction - I stopped looking into any network-related problems and removed the integration entirely (again). After that I restarted the Raspberry and installed the integration again, this time without any problems, so I could configure it right away. Problem solved. 👍

Funny: restarting HA was going to be my original suggestion, but I rephrased to focus in on PyPI.


Remaining problem in unisusie pipeline

@hsun3163 Please note that uni_susie_2 in https://github.com/cumc/xqtl-pipeline/blob/main/code/fine_mapping/SuSiE/SuSiE.ipynb cannot be successfully executed yet. There are two problems:

Some of the rds files will be blank, and cannot be handled by readRDS() function.  Adding improt os; skip_if(os.path.getsize(_input) == 0) may help.
Because now output rds is only one susie_object, there will be errors about the variable susie_list in uni_susie_2. We no longer can use susie_list[[i]]$pip to extract pip. Instead, we should directly use susie_list$pip.

Because I currently don't know the strategy for correcting this step -- whether I should make a vcf for per gene per tissue, or integrate the result per gene in all tissues.  So I leave this issue here as a reminder.

@SamuelSun2000 could you provide me a minimal working example for this? (remember the style of the RNA-seq protocol you have tried last year we have the MWE). I'm swamped today until late in the PM .. but this one is now on the top of my TODO to finalize so others can use.

I see. @SamuelSun2000 the MWE you provided me can they reproduce these issues?

Yes, both of them.

I was thinking, perhaps we should take away the uni_susie_2/3 altogether. And make the transform to vcf step another utility in the susie_post_proc notebook. Along with the code that produces the three summary tables.

yes i remember your proposal. let me take a look maybe this evening or tmr


Kafka Scaler: Improve error logging for GetBlock() method

Hi team,
This PR aims to make the error logging for GetBlock() more informative.
Checklist

[x] I have verified that my change is according to the deprecations & breaking changes policy
[x] Changelog has been updated and is aligned with our changelog requirements
[x] Commits are signed with Developer Certificate of Origin (DCO - learn more)

Fixes #4232

Thanks for the contribution! ❤


Jesse: Watch +91-9654078255

Jesse requests the watch of the watch_number +91-9654078255. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

Approved by cigien in Charcoal HQ


Adds artifact-bucket to source-controller build

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/lgtm
/approve


Fix typo in Quick Start command in README.md

I was going through the README.md of sd-jwt and noticed a small typo that I believe needs correction. Here are the details:
$ go run example.main.go
stat example.main.go: no such file or directory

$ go run example/main.go
Combined Issuance Format
eyJhbGciOiJFUzI1NiIsInR5cCI6Ikp...

thanks! @kg0r0


Filter a query byheight doesn't works listing pools

System information
Osmosis version: Osmosis version
v15.0.0
Linux
Expected behaviour
When I query the pools from command line filtering by height it works.
Actual behaviour
When I do the same querying the LCD it doesn't
Steps to reproduce the behaviour
https://rest.cosmos.directory/osmosis/osmosis/gamm/v1beta1/pools?height=8710740
It should return 949 pools in place of "all" 957 (at the moment of writing this)

Yikes, this was related to historic heights not getting passed into REST properly, this has since been fixed.


Can this be used as just a UITextView?

Can this be used as just a UITextView component dropped into an existing view controller and not a whole UIViewController of itself?

No. ZSSRichTextEditor is a UIWebview which allows for the formatting of content, viewing of HTML, export of HTML, etc.

what i mean is can I use this component outside of the view controller? say if i wanted a UITextfield above the rich text editor?

No, it currently only works as a viewController subclass.


Remove Admin theme for July release

Description
This PR is removing Admin theme for July release.
Reference
List of relevant links to GitHub issues, PRs, and other documentation.
Testing
Brief description of the testing done for this change showing why you are confident it works as expected and does not introduce regressions. Provide sample output data where appropriate.
Ran ./test.sh and docusaurus and all looked good.
Checklist
Checklist of tasks commonly-associated with schema pull requests. Please review the relevant checklists and ensure you do all the tasks that are required for the change you made.

[ ] Add relevant examples.
[ ] Add relevant counterexamples.
[ ] Update any counterexamples that became obsolete. For example, if a counterexample uses property A but is not intended to test property A's validity, and you made a schema change that invalidates property A in that counterexample, fix the counterexample to align it with your schema change.
[ ] Update in-schema documentation using plain English written in complete sentences, if an update is required.
[ ] Update Docusaurus documentation, if an update is required.
[ ] Review change with Overture technical writer to ensure any advanced documentation needs will be taken care of, unless the change is trivial and would not affect the documentation.

Documentation Website
Update the hyperlink below to put the pull request number in.
Docs preview for this PR.

I'm working on a more formal deprecation policy with @bglazer-meta. We intend to get it in front of the map data wg tomorrow. I recommend not merging this until that policy is in place, as it will have implications for how we remove admins. We will get this policy written and approved as soon as possible


valueChange event always tiggered when value is true

Bug description:
Hello, why onload time the valueChange event always tiggered when value is true?

Hi @realtica. Please provide an example of the issue as described in the Issue Template. I haven't received any other reports of this behavior, so an example will help narrow down the issue.


[DO NOT MERGE ] verbose package build

Just for testing

https://github.com/apple/sourcekit-lsp/pull/1221/files
@swift-ci Please Build Toolchain macOS Platform

https://github.com/apple/swift-installer-scripts/pull/295
@swift-ci please build toolchain Windows platform

https://github.com/apple/swift-installer-scripts/pull/295
@swift-ci please build toolchain Windows platform


UI: Forms footer buttons alignment

🐛 Bug Report
Footer buttons in forms are not right aligned with the container immediately above them.
To Reproduce
Access any new/view/edit form e.g. Patient, Lab, Medication, Incident, Imaging, you will notice the misalignment.
View/Edit Patient

New Appointment

Request Lab

This issue must be fixed for the following forms :

Patients (new, view/edit)
Appointment (new, view/edit)
Medications (new, view/edit)
Lab (new, view/edit)
Imaging (new, view/edit)
Incident (new, view/edit)

Expected behavior
For a great UX the buttons right margin must be aligned with the righ margin of the container immediately above them.
Your Environment

node version: 6,8,10
fastify version: >=1.0.0
os: Mac, Windows, Linux
any other relevant information

Hi @blestab, I would like to take this.

Hi @blestab, I would like to take this.

Thanks @Adarsh710, all yours.

Is this issue fixed ?

@TD-17 not yet

can I work on this ? @tehKapa

@TD-17 It only needed to be merge


[ZEPPELIN-1787] Add an example of Flink Notebook

What is this PR for?
This PR will add an example of batch processing with Flink to Zeppelin tutorial notebooks. There are no any Flink notebooks in the tutorial at the moment.
What type of PR is it?
Improvement
What is the Jira issue?
ZEPPELIN-1787
How should this be tested?
You should open Using Flink for batch processing notebook from the Zeppelin Tutorial folder and run all paragraphs one by one
Questions:

Does the licenses files need update? - no
Is there breaking changes for older versions? - no
Does this needs documentation? - no

Link for the note on ZeppelinHub
For convenience :)
@AlexanderShoshin Is this link right?

Yes, but ZeppelinHub does not show all the paragraphs. I will try to find out the reason.

@AlexanderShoshin Thanks for your contribution!
While I was just quickly looking through this example note in my Zeppelin, couple of things were noticed.

AFAIK, OSX doesn't have wget by default so the OSX users might need to install by themselves. So I would suggest you to use curl instead of wget to download datasets. (curl is built-in command I guess)

I saw you set the location of dataset to tmp/. It can occurs sth link "permission denied" error for that dir in Zeppelin like below.

Caused by: org.apache.flink.runtime.JobException:
Creating the input splits caused an error:
File /tmp/flights98.csv does not exist or the user running Flink ('ahyoungryu') has insufficient permissions to access it.

How about refer this note in Flink.md ?

@AhyoungRyu, thank you for your notes!

I've changed wget to curl as you suggested.
As for the error message with "permission denied" I think it is because %sh paragraphs with data download instructions did not finish correct. Each paragraph downloads for about 70 MB of data and unpack it then. They might have finished by timeout. In this case you don't have the /tmp/flights98.csv file at all. A added a recomendation to the notebook to increase shell.command.timeout.millisecs setting. It helped in my case.
We can't store data sets in "home" folder because %sh and %flink may be run by different users. So they may have different home folders. The /tmp/ folder is a common folder which normally can be accessed by each user. For the case of limited access I've added chmod 666 /tmp/flights<YY>.csv command for each csv set. Maybe there is a better solution for this issue. I will be glad to receive any suggestions :)

@tae-jun, I found that ZeppelinHub can't display new notebooks correctly. It is because the note.json structure was changed after ZEPPELIN-212 was merged (two weeks ago). Now notebook has a results attribute (instead of result) to store paragraph results and it seems that ZeppelinHub can't see it.

@zjffdu, should we do this in current PR or make a new issue for the documentation improving?

Thank you for the such precise explanation. Tested again and it works well!
BTW some markdown paragraphs are not shown like below, so i didn't noticed that there are some description in here(I need to click "show editor" to check what it contains).

It would be better to show the other description paragraph's result by default like first paragraph does :)

Oh, it should not looks like this. It might be another problem of new notebook json structure. I will correct this.

I've converted notebook to 0.6.2 format. So it should be displayed correctly now.
ZeppelinHub vew also works.

Thanks for your quick update!
Since #1780 is trying to update the all tutorial notes' format to the latest one, having older format of note can't be best solution I think.  Not sure, but as you said, I assume there is another problem \w new note json format itself.(If so, we need to fix this. But in another PR :D ) #1780 also has same problem as I left a comment in there.

@AlexanderShoshin Actually it was my bad. I checked again and this commit is perfectly working. Please ignore my last comment and sorry for the confusion.

Is there any other feedback, or was everything addressed and we shall merge it now?
Saw @zjffdu 's

How about refer this note in flink.md ?

@AlexanderShoshin do you think it's worth to update the interpreter documentation as well under ./docs, to mention this example?
Other than that, looks great to me!

I am confused a bit :)
@AhyoungRyu, do I need to drop my last commit or not?

@AlexanderShoshin Right that would be better. So sorry about that ㅠ_ㅠ

@bzz, I am not sure. We already have a Word Count example there to describe Flink usage. Where should we place the link to this example?

@AlexanderShoshin I think what @zjffdu and @bzz meant was that you could possibly call out this notebook in some where like docs/interpreter/flink.md

@AlexanderShoshin Seems the below two things need to be done before merge

revert the last commit 9013620

You can simply do it in your local by
please make sure you are on branch ZEPPELIN-1787 first
$ git checkout ZEPPELIN-1787

and check the last commit is "convert notebook to 0.6.2 format" or not
$ git log

if so,
$ git reset HEAD^ --hard

check again it's converted well, it should be pointing this commit: "add download instruction, change "wget" to "curl""
$ git log

push with forced updated!
$ git push origin -f ZEPPELIN-1787

add Flink docs link to the tutorial as @rawkintrevo said.

Please feel free to ping me if you need any help!

Sorry, I was not able to work on this issue during last weeks.
@AhyoungRyu, I will make the corrections soon.
Thanks

I've added a new commit to convert the notebook to 0.7.0 format because I found that 9013620  commit has several conflicts in json-file.
I've also merged master to avoid conflicts in flink.md.

@AlexanderShoshin Checked again and it looks nice. Except the minor suggestion, LGTM. Thanks for your effort and the awesome tutorial note!

@AhyoungRyu, thanks for the note. I've removed an url to the notebook.

@AlexanderShoshin Thanks! Will Merge if there are no more comments on this.


⚠ gorzdrav.org has degraded performance

In 6a06f6f, gorzdrav.org (https://gorzdrav.org) experienced degraded performance:

HTTP code: 200
Response time: 1312 ms

Resolved: gorzdrav.org performance has improved in 480f8c3.


danfo.read with authorization bearer

danfo.read() should support a config parameter for request headers in order to secure the accesses (passing jwt tokens for ex.)
{ headers: {"Authorization" : Bearer ${Token}}

Added in latest version v0.3.0


AEM-3 rule is not working with the sonarQube 7.1 with sonar scanner 3.1.0 and aem rules-0.10.jar

Hi Team,
Thank you for these rules. We are using AEM cognifide rules but looks like AEM-3 rule is not picked by the sonar scanner. I purposefully broke AEM-1 rule and it's working as expected.
Can you please share if you have any idea on why it's not working on the files ?
AEM-3 Non-thread safe object used as a field of Servlet / Filter etc.
Thanks,
Surender

Thank you michal. Issue is with the sonar.java.libraries. After adding this property in my sonar-project.properties, i can see that issue got resolved and AEM-3 rule was reported.

That's great news. :-)
Please subscribe to https://github.com/Cognifide/AEM-Rules-for-SonarQube/releases.atom for new releases. We're aiming to add HTL support pretty soon.


Update create-comment.ts

MAX_COMMENT_JSON_LENGTH -> It's better to validate the length before stringify than stringify and then checking length.

Does removeUndefinedProps handle null?

Hi @firstuserhere, thanks for contributing!
This change doesn't appear to do anything different than before.
removeUndefinedProps requires a non-null object argument.


All melee ablilites have no tooltip enhancements at all

Something must be short circuiting the logic for melee abilities.
It seems to affect all warrior/rogue abilities and the paladin seals.
Hunter melee abilities as well, but most shots seem to work. (though with different bugs #2 )

Fixed by version 01.12.2022


Can not start project

Describe the bug
I install dependencies with vite ^5.3.1 (latest minor updates available) and fail with error
$ vite
file:///home/project/node_modules/vite/dist/node/chunks/dep-BWSbWtLw.js:61873
            url = injectQuery(url, "direct");
                                            ^

SyntaxError: Unexpected token ';'
    at compileSourceTextModule (node:internal/modules/esm/utils:340:16)
    at ModuleLoader.moduleStrategy (node:internal/modules/esm/translators:102:18)
    at #translate (node:internal/modules/esm/loader:433:12)
    at ModuleLoader.loadAndTranslate (node:internal/modules/esm/loader:480:27)

Node.js v22.11.0
error: script "dev" exited with code 1

Reproduction
https://google.com
Steps to reproduce
bun i and bun dev
equal results with
rm -R node_modules and npm i and npm run dev
System Info
System:
    OS: Linux 6.8 Ubuntu 22.04.5 LTS 22.04.5 LTS (Jammy Jellyfish)
    CPU: (12) x64 AMD Ryzen 5 5600G with Radeon Graphics
    Memory: 12.82 GB / 27.29 GB
    Container: Yes
    Shell: 5.1.16 - /bin/bash
  Binaries:
    Node: 22.11.0 - /usr/local/bin/node
    npm: 10.9.0 - /usr/local/bin/npm
    bun: 1.1.34 - ~/.bun/bin/bun
  npmPackages:
    @vitejs/plugin-react: ^4.3.1 => 4.3.3
    vite: ~5.3.1 => 5.3.6

Used Package Manager
bun
Logs
debug
file:///home/project/node_modules/vite/dist/node/chunks/dep-BWSbWtLw.js:61873
            url = injectQuery(url, "direct");
                                            ^

SyntaxError: Unexpected token ';'
    at compileSourceTextModule (node:internal/modules/esm/utils:340:16)
    at ModuleLoader.moduleStrategy (node:internal/modules/esm/translators:102:18)
    at #translate (node:internal/modules/esm/loader:433:12)
    at ModuleLoader.loadAndTranslate (node:internal/modules/esm/loader:480:27)

Node.js v22.11.0

Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[x] The provided reproduction is a minimal reproducible example of the bug.

Perhaps you can try node 20? The error doesn't make sense to me.

Closing due to lack of response


Changing CSV download to GET

Fix for #351

[Current coverage][cc-pull] is 90.56%

Merging [#370][cc-pull] into [master][cc-base-branch] will increase coverage by 0.07%

@@             master       #370   diff @@
==========================================
  Files            29         29
  Lines          1975       1981     +6
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           1787       1794     +7
+ Misses          188        187     -1
  Partials          0          0

Powered by Codecov. Last updated by [e1ff1aa...371ea4d][cc-compare]
[cc-base-branch]: https://codecov.io/gh/D-PLACE/dplace/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/D-PLACE/dplace/compare/e1ff1aa9ea2c722859b8df233cb8aba319c5e6dc...371ea4d3ad2ca363983a2ae70b26e0ec9439f8e7
[cc-pull]: https://codecov.io/gh/D-PLACE/dplace/pull/370?src=pr


can't authorized through remote machines on cli.

so i was trying this sweet cli tool for moving local-data to the drive but it seems it's not passing the parameters of usual "  --noauth_local_webserver" any way following is the output.
/home/ubuntu/.local/lib/python3.8/site-packages/oauth2client/_helpers.py:255: UserWarning: Cannot access /home/ubuntu/gist/drive-cli/drive_cli/token.json: No such file or directory
warnings.warn(_MISSING_FILE_MESSAGE.format(filename))
Your browser has been opened to visit:
https://accounts.google.com/o/oauth2/auth?client_id=xxxxxxxxxxxxxxgoogleusercontent.com&redirect_uri=http%3A%2F%2Flocalhost%3A8080%2F&scope=https%3A%2F%2Fwww.googleapis.com%2Fauth%2Fdrive&access_type=offline&response_type=code

If your browser is on a different machine then exit and re-run this
application with the command-line parameter
--noauth_local_webserver
^C

 --noauth_local_webserver this parameter just does not work

use the --remote flag

@grgcnnr thanks I was able to do that
$ drive --remote login


I need to re-publish emsg events when the playhead has reached emsg.startTime. What should I compare against?

When Shaka encounters emsg events in a stream, it parses the boxes and immediately publishes those events to the application. Our ads provider (YoSpace) has asked us to enqueue these events and republish them to their SDK when the playhead arrives at the emsg.startTime.
I am considering the following approach:

Listen for emsg events from Shaka
Listen for timeupdate events from videoElement
When an emsg event is received, push the event into a queue
When a timeupdate event is received, inspect the queue. For each emsg event which has startTime <= playheadTime, republish this event to the ad SDK

I'm hoping you can advise me about how to determine playheadTime in step 4. The emsg.startTime, I understand, is an absolute presentation timestamp. I need a value in the same frame of reference. Should I use Player#getPlayheadTimeAsDate() / 1000? Or should I just use videoElement.currentTime? Is there a difference?
Let me know if I can provide more information, and thank you in advance for any advice.

getPlayheadTimeAsDate returns a Date object, so you probably shouldn't use division on it (though it seems to work).  But it also only works on live streams, not VOD.  In a live stream, getPlayheadTimeAsDate returns the wall-clock time of the playhead position.  So if it is 10:30 now and you are playing 10 minutes behind the live edge, it will return 10:20.
video.currentTime will return the current playhead time in presentation time.  This is likely the time you want since that will be the time of emsg.startTime and emsg.endTime.

Thanks @TheModMaker, I'm grateful for the help.


State transition fix

Use the auth proof that is generated matching the last state on the chain.

The refactor broke the challenge respond code. looking at it


Async exception after adding TailwindCss

I’m trying to add tailwindcss after adding tailwind to the project I have gotten that exception Use process(css).then(cb) to work with async plugins after performing npm run dev the exception belongs to css-modules-require-hook and by googling that exception I found that css-modules-require-hook don’t support async functions. Unfortunately, am not a Webpack expert that's why am seeking for help and I hope I can find it here.
Example of what I have done so far in that repo: https://github.com/hasusuf/react-cool-starter-tailwind
Error stack:
$ better-npm-run dev
running better-npm-run in ~/playground/js/react/react-cool-starter-tailwind
Executing script: dev

to be executed: nodemon ./index.js
[nodemon] 1.18.3
[nodemon] to restart at any time, enter `rs`
[nodemon] watching: src/server.js src/utils/renderHtml.js
[nodemon] starting `node ./index.js`
~/playground/js/react/react-cool-starter-tailwind/node_modules/css-modules-require-hook/node_modules/postcss/lib/lazy-result.js:266
                throw new Error('Use process(css).then(cb) to work with async plugins');
                ^

Error: Use process(css).then(cb) to work with async plugins
    at LazyResult.sync (~/playground/js/react/react-cool-starter-tailwind/node_modules/css-modules-require-hook/node_modules/postcss/lib/lazy-result.es6:319:23)
    at LazyResult.warnings (~/playground/js/react/react-cool-starter-tailwind/node_modules/css-modules-require-hook/node_modules/postcss/lib/lazy-result.es6:157:21)
    at fetch (~/playground/js/react/react-cool-starter-tailwind/node_modules/css-modules-require-hook/lib/index.js:102:16)
    at hook (~/playground/js/react/react-cool-starter-tailwind/node_modules/css-modules-require-hook/lib/index.js:125:18)
    at Object.cssModulesHook [as .css] (~/playground/js/react/react-cool-starter-tailwind/node_modules/css-modules-require-hook/lib/attachHook.js:16:20)
    at Module.load (module.js:561:32)
    at tryModuleLoad (module.js:501:12)
    at Function.Module._load (module.js:493:3)
    at Module.require (module.js:593:17)
    at require (internal/module.js:11:18)
[nodemon] app crashed - waiting for file changes before starting...

@hasusuf If you don't use css modules you can turn it off from here and comment the css modules hook part out from here.

Thanks, @wellyshen for your reply!
The problem is that if I commented out the css-modules-require-hook definition, I will lose all postcss-config and Tailwind implementation as well here
My problem is related to this issue and am wondering if I can refactor the css-modules-require-hook part with an alternative hook/plugin that supports async functions.

@hasusuf I will spend time to check your repo. Give me a simple reproduced environment as you can.

@hasusuf
I create a branch based on yours. You can see how to Remove css modules hook through it. The differences are:

Remove all css modules from components.
Import scss styles in client-side only (styles can't work in SSR).

That's why I like this project the most! @wellyshen you rock 👍
Regarding the SSR, I will work on that and I will keep this issue updated.

@hasusuf Cool keep going.

Recently came across this starter and we're planning to use it for our application but I am running into the same issue trying to add Tailwind. I don't need css modules and tried turning them off but the import for tailwind styles fails. Any pointers/suggestions? @wellyshen @hasusuf

@hasusuf Can you help with it ☝with your "TailwindCss" experience?

@wellyshen Do you have any pointers for this? I tried disabling CSS modules and even the hook in hooks.js and then loading tailwind styles only in client.js but then they don't apply at all.
Any ideas on how I can get a basic version up and running with Tailwind?

@hasusuf I will spend time to see how to integrate it with this starter-kit. But not today, I need to work tomorrow...

@hasusuf It's not hard to integrate tailwind with this starter, you can see this example. I only did two things:

Follow the doc to compile it
Import it here

With this approach, we'd manually have to build Tailwind every time we make a change to the config, right?
I followed an approach similar to yours but I tried adding it in the postcss config and that's when I kept getting the error that this issue is about.

@FarazPatankar It's the easy way that I found to integrate with this starter. You don't need to build it manually. You can integrate it with yarn dev or so on. For example: "dev": yarn tailwind && better-npm-run dev.


1.1.26: "go generate" fails: can't find import: "github.com/miekg/dns"

src/github.com/miekg/dns/duplicate.go
2019/12/30 01:23:37 can't find import: "github.com/miekg/dns"
exit status 1
src/github.com/miekg/dns/duplicate.go:3: running "go": exit status 1

The following patch appears to fix the problem - please review:
--- a/duplicate_generate.go
+++ b/duplicate_generate.go
@@ -40,9 +40,9 @@
 }

 func main() {
        // Import and type-check the package
-       pkg, err := importer.Default().Import("github.com/miekg/dns")
+       pkg, err := importer.For("source", nil).Import("github.com/miekg/dns")
        fatalIfErr(err)
        scope := pkg.Scope()

        // Collect actual types (*X)
--- a/msg_generate.go
+++ b/msg_generate.go
@@ -45,9 +45,9 @@
 }

 func main() {
        // Import and type-check the package
-       pkg, err := importer.Default().Import("github.com/miekg/dns")
+       pkg, err := importer.For("source", nil).Import("github.com/miekg/dns")
        fatalIfErr(err)
        scope := pkg.Scope()

        // Collect actual types (*X)
--- a/types_generate.go
+++ b/types_generate.go
@@ -82,9 +82,9 @@
 }

 func main() {
        // Import and type-check the package
-       pkg, err := importer.Default().Import("github.com/miekg/dns")
+       pkg, err := importer.For("source", nil).Import("github.com/miekg/dns")
        fatalIfErr(err)
        scope := pkg.Scope()

        // Collect constants like TypeX

See also https://github.com/golang/go/issues/25723.

This seems like it’s superseded by the pending #1050. While waiting for that to merge, there’s a simple workaround from the linked go issue: go install.
There’s very little reason to run go generate in this library unless you’re planing on opening a pull request.
I’m going to close this for now, but feel free to reopen if #1050 doesn’t get merged.

"Simple workaround" is unavailable to downstream package maintainers where library is built in pristine controlled environment. Installation recursion is best to avoid (e.g. previous release of the library may be already installed so the test could be misleading as it is not testing what it should test).
Maybe for this library go generate is not too important but generally speaking it is a good practice for build integrity and reproducibiity.
I'm looking forward for #1050 to get merged, if you think it is a better solution. Thanks.

@onlyjob I’m confused at where you’ve heard that, but running go generate as a consumer is definitely not good practice and isn’t even possible in module mode. It’s very much a tool for package maintainers. Either way, yes #1050 is the solution here and will likely be merged relatively soon.

I am the package maintainer. ;)

I meant package author, as in miekg, or contributors (PR authors). I’m not sure what package maintainer means in this context.
Anyway #1050 has been merged so this issue is resolved regardless.

Thanks!


download link is broken

http://download.gna.org/wkhtmltopdf/0.12/<IP_ADDRESS>/wkhtmltox-<IP_ADDRESS>_linux-trusty-amd64.deb
is no longer active, can you pls update your buildpack with a new link?

@davideghz have you found a fix for this yet ?

yes, here https://github.com/dscout/wkhtmltopdf-buildpack/pull/19


Ordinal explorer: "Coinbase height" is poorly understood add explainer

Is your feature request related to a problem? Please describe.
Many users do not know that "coinbase" means "block reward". They may wonder what coinbase.com (the centralized exchange) has to do with the inscription 😄.
Describe the solution you'd like
In this case I think "Coinbase block height" would be more descriptive . Additionally it could be helpful to have a little "i" or "?" to explain that it is the the block reward that created the inscribed sat.
Describe alternatives you've considered
You could just show "block" but then there could be confusion about "what block". The block the sat was inscribed or the sats birthblock? In this case it is about its birth.
Additional context

This is how it is displayed on an other explorer.

i think ord.io explained this pretty well

https://twitter.com/ord_io/status/1644412548840661030?s=20

sat block number linked to mempool.space to the block

cc/ @janniks


Maintain sourcemapping for injected modules.

I am not 100% which side is the best to resolve this issue since there are multiple plugins involve. However, I do want to report this so the maintainer and other users are aware of this problem.
First, our setup is:

webpack
karma + mocha
Istanbul
inject-loader
istanbul-instrumenter-loader

The istanbul-instrumenter-loader was used to hook each module to Istanbul.
Now, when test package requires both a module, and an injected version of the same module. I'm getting exceptions from Istanbul. On the surface, the issue is caused by a mismatch between statements and statementMap, two internal variables of Istanbul to record statements and the location of them in the file. The length of the two didn’t agree. One recorded more statements than the other for "the same file".
I did some debugging and I believe this is what's happening:

inject-loader will add statements to the module for the injection
istanbul-instrumenter-loader registers both the original file and the injected version to Istanbul understand the same file name

The two functions, under the same name, have different statement count, led to the issue.
I also provided this info to Istanbul under gotwarlost/istanbul#192.

This is a problem of istanbul-instrumenter-loader
https://github.com/deepsweet/istanbul-instrumenter-loader/pull/12

Thanks for reporting. This is an issue I'm aware of. I think to work up a fix I will need to reprocess the source maps if they are supplied.
I'm on vacation for the next week or so, i plan to do a bunch of work on this stuff when I get back to work. :v:

Just ran into exactly this problem. Anything I can do to help?

Not to include same file with inject loader and without in the same build.
This is usually an indication, some of your unit tests include too much.
Though I made a dirty fix for istanbul-loader to address this (not accepted)

Did anyone find a workaround for this? I've got inject-loader working just fine, the coverage report still just reports tested lines as untested still.

To be honest I’m pretty out of my depth here — I know very little about the implementation details of sourcemaps.
Anyone have any useful pointers on how to resolve this?

Hi guys,
Did you guys ever looked into this. I am not using instanbul, and i am getting the same issue, as soon as a test injects uses this framework breakpoints are broken.
Did you guys ever created a solution for this?

@ricardolpd Looked into it, yes. But no solution—still an open issue.

I've added sourcemaps support in #36. Could someone give it a try?

Fixed by #36

@vladimir-tikhonov thanks for adding support for sourcemaps.
I just done a couple tests with my existent config:
It seems that breakpoints tests in my test folder are working now, however in my production code folder they only seem to work (not reliably though), if i add the breakpoint in the webpack output file and in same place in the original file.
I am using webpack 2.x + mocha, in vs code with node v7.5.0.
the sourcemaps i am using in webpack is devtool: source-map.
I would love to have this working so i am happy to keep debuging as much as possible, to see if this is a config issue on my side or something else, that could probably help down the line.

@ricardolpd are you sure that this issues are caused by inject-loader? Webpack itself has a lot of problems with sourcemaps (especially when combined with UglifyJSPlugin), check this link for details - https://github.com/webpack/webpack/issues/2145

@vladimir-tikhonov, yes i agree with you, sourcemaps and webpack are not amazing:(. But yeah i think i am pretty sure its caused by inject-loader.
I have it all working in classes where inject-loader is not in (Because of this i have issue i have separated the tests to tests that require mocking and those that i get away with no mocking).
Also i am not using uglifyJSPlugin.
I really would like to see this working but my knowledge with sourcemaps and webpack is not amazing. So if there is anything u need to help i am happy to oblige.
Also i am using node debugger with vscode, the only sourcemaps that work with vscode and webstorm (tools used at work, is the 'source-map', nothing else).


Disable UAS for USB-SATA Adapter cable PA023U3

Fix yet another UAS issue with a USB-SATA adapter 😀
Ref.: https://devicehunt.com/view/type/usb/vendor/7825/device/A2A4

@bolaum You can now close this, @AaronDewes added these changes to an existing PR for quirks: https://github.com/getumbrel/umbrel-os/pull/168.

Cool!


Refactor logger to be a part of context

putting the logger directly into the context object, so it's available everywhere and we don't have to pass it explicitly anymore.
In the future, we can also add additional properties like a tracing id via the same way.
Signed-off-by: Nico Schieder<EMAIL_ADDRESS>

/lgtm


some follow ups for GetAllModels adoption

Sync changes from this PR in autorest.csharp: https://github.com/Azure/autorest.csharp/pull/4682

API change check
API changes are not detected in this pull request.


a couple of small tweaks

ok, this one only has the two relevant commits.

All good - the reason we actually had the issue and was tricky to track down was the numBonds value was used to set the bond index and did not reflect the value returned by getNumBonds().
The index values could therefore exceed the bond count - causing a seg fault (or assertion failure in debug mode).
RWMol mol;
// load molecule with something
mol.clear();
// load molecule with something containing a ring
// sanitize (FindRings breaks due to assertion in boost about bit set size)

Essentially in the ring finding code a bitset was created using (mol.getNumBonds()) but then the bits were accessed using bond.getIdx();


View won't registered on youtube

Script version
5.6
Describe the bug
Youtube 90% of the time cannot recognize that I watched a video using the script, so it will not be saved in the history and the videos I have already watched continue to be recommended on my home page, this also harms the general daily recommendations
Expected behavior
I dont know 😂
Desktop:

OS: Windows 10
Browser Opera GX
Version 111.0.5168.90

history works with older verstion 5.3 but there the ads are not totaly blocked but "fast forward"


Error: refresh token must be passed in or set as part of setAccessToken

He utilizado el ejemplo de quickstart.php para Google Calendar y funciona todo correcto hasta que pasan algo más de dos horas y me devuelve un error al intentar refrescar el token al expirar.
He mirado en muchos foros y no funciona nada de lo que se indica.
Por favor, me puede ayudar.
Sample:
ERROR
`PHP Fatal error:  Uncaught LogicException: refresh token must be passed in or set as part of setAccessToken in C:\www\medel\pru\vendor\google\apiclient\src\Google\Client.php:258
Stack trace:
#0 C:\www\medel\pru\quickstart.php(51): Google_Client->fetchAccessTokenWithRefreshToken(NULL)
#1 C:\www\medel\pru\quickstart.php(59): getClient()
#2 {main}
thrown in C:\www\medel\pru\vendor\google\apiclient\src\Google\Client.php on line 258
Fatal error: Uncaught LogicException: refresh token must be passed in or set as part of setAccessToken in C:\www\medel\pru\vendor\google\apiclient\src\Google\Client.php:258
Stack trace:
#0 C:\www\medel\pru\quickstart.php(51): Google_Client->fetchAccessTokenWithRefreshToken(NULL)
#1 C:\www\medel\pru\quickstart.php(59): getClient()
#2 {main}
thrown in C:\www\medel\pru\vendor\google\apiclient\src\Google\Client.php on line 258`
Actual Behavior
Steps to Reproduce the Problem
1.He seguido los pasos que se indican en la guía de Google (https://developers.google.com/calendar/quickstart/php)
Specifications

PHP version (php -v): PHP Vr. 7.2.6
OS: Windows 10

This error is being thrown because the saved tokens in token.json doesn't contain a refresh_token, which is needed to refresh the access token after it's expired. At the moment the code isn't checking that a refresh token exists before it attempts to refresh the access token, nor handling the case when one doesn't exist. I'll put together a PR to fix this.

The  code from the developer contain a bug to handle the refreshtoken as today 9 September 2018.
i already fix the solution for this... you must save 2 location for token

handlerefeshtoken.json
2)token.json

hope this help...

Almost 2 day and 2 night to figured it out :( the example on the quickstart is not good example.. because as google developer, the dev should test before put at the website as example.. now many newbie cannot start with quickstart hehe...

😂😂😂
I'm looking forward about that. Because I'm a newbie so impatient to do
something that is my first small project...😁😁
Đăng Nam
Vào 16:31, T.3, 11 Th9, 2018 Syed Muhammad Bin Syed Hussin <
<EMAIL_ADDRESS>đã viết:

Almost 2 day and 2 night to figured it out :( the example on the
quickstart is not good example.. because as google developer, the dev
should test before put at the website as example.. now many newbie cannot
start with quickstart hehe...
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/gsuitedevs/php-samples/issues/16#issuecomment-420209877,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AkkcEF1zHvu_bgLyrR-WEO949Hdh1aMTks5uZ4L1gaJpZM4VtuOU
.

I pushed out some new code that handles this case better. Please try that new code and let me know if you still have a problem.

is it auto refresh-token for my apps or I have to copy the authenication from the warn on the CMD?

if I wanna auto the refresh-token, do I need some file to get that?

Hyperaktiv, do u already test the code? Suppose the code from eric now it will auto refresh the new token when reach after 3600 second..

ah, my fault, I've not read carefully.. Sorry
Thank you very much. Hope that I can keep in touch with you

ah, my fault, I've not read carefully.. Sorry
Thank you very much. Hope that I can keep in touch with you

No problem.. hope u can test the eric way, if any problem u can roger here, eric will fix the bug...


implement cluster flavors (csi,ccm)

What this PR does / why we need it:
this PR implements two cluster template flavors:

csi: deploy Nutanix CSI driver (snapshot v6.0.1 , storage v2.5.4)
ccm: prepare cluster for Nutanix Cloud Controller Manager by setting cloud-provider to external

How Has This Been Tested?:
deploy cluster with respective flavors
clusterctl generate cluster test-csi -f csi
clusterctl generate cluster test-ccm -f ccm
Release note:
- implement cluster template flavor to deploy Nutanix CSI
- implement cluster template flavor to prepare cluster for external Nutanix CCM

we will need to update documentation on the usage of these flavors. also any pointers to ccm docs would help too

correct , for CSI  let me add it
for CCM for the moment this is only dependency to pursue dev , not intended for common use , doc will be released with the full support in same time the CCM release

could you please also create a bash script in tools so that we can update these csi and ccm yamls for next version when needed without hand copying them to avoid any errors

I will add it


Unable to use external editor?

Tried to use the snap with Vim as external editor. The editor selection dialog only offers editors installed as snaps, and setting the explicit gvim binary in the Joplin settings ("/usr/bin/gvim" and variations thereof) silently fails.

Unfortunately, strict snaps (the majority of snaps in the store, ones with sandboxing enabled) and flatpaks can't just access arbitrary filesystem locations, external editors are facilitated using the xdg-flatpak portals integration. This should show more options than just other snaps, but from what I'd guess in order to get Vim to work, you'd need to create a .desktop file for it so it shows up in the start menu, make sure it's marked as an editor for markdown files,  and then it should appear in the external apps picker and otherwise work as expected. I certainly recall none snap apps being shown when I've used it.
I might remove the setting for an explicit binary since as you point out it's very misleading, and getting it to work in a way a user expects is a very involved effort and way harder than the option of just not using the snap.
Overall between this and your other posted issue, I think snaps are probably a bad fit for you, at least as much as Joplin goes. Joplin does provide an AppImage and a convienient install/update script for here https://github.com/laurent22/joplin/blob/dev/Joplin_install_and_update.sh, and my post in the other issue hopefully might help you get your data back if you need to recover it.

I've installed the Appimage already, and yes, while I like snaps I feel that for this particular app, the security and isolation features of snaps become a drawback more than a benefit. Still, thank you for your work on this and on snaps in general!

Much appreciated!
I think most people don't have an issue with the sandboxing in Joplin in particular, but there's certainly exceptions. I think you're the second person to have a problem with the external editor functionality that I've seen, the sentiments about snaps deleting their user data is fairly common among all snaps and the second I've seen again for this one in particular, and there's a few more complaints about removable-media / accessing files in sandboxed areas, printing, and the worst of them all, broken font glyphs.
While essentially most of these can be fixed by various technical improvements, some of which are already ongoing, I think I need to upgrade the README to have a FAQ so that people can tell in advance if Snap Joplin is a good fit and how to handle the edge cases that do often show up, it's looking pretty bare bones at the moment. It's a very unusual concept to users that synchronising data to various web hosts is easy but opening a text editor is complicated but at least they can told why.
Thanks for your feedback :)!

FWIW, there's a Flatpak version of Joplin now.
https://flathub.org/apps/details/net.cozic.joplin_desktop
Although you still have to accomodate for the Flatpak sandbox by writing the path using flatpak-spawn, it has most the advantages of a Snap build but a much more direct fix for this specific issue. (And it also uses the normal database location too, so solves https://github.com/MrCarroll/joplin-snap/issues/10 too)

FWIW, there's a Flatpak version of Joplin now.
https://flathub.org/apps/details/net.cozic.joplin_desktop
Although you still have to accomodate for the Flatpak sandbox by writing the path using flatpak-spawn, it has most the advantages of a Snap build but a much more direct fix for this specific issue. (And it also uses the normal database location too, so solves https://github.com/MrCarroll/joplin-snap/issues/10 too)


Could Not Load Manifest?

I followed the tutorial exactly, but I'm getting an error when I try to load the extension that says "Error loading extension: Manifest file is missing or unreadable." Help?

Hi @21SarahR.
That is a very interesting error. The two possible issues I can think of that could be causing it are:

Are all of the necessary files in the directory you downloaded Check PCR to? There should be a manifest.json file in there.
Did you select the correct folder when loading the extension? You'll need the select the folder which has that manifest.json file directly in it.

If one of these does turn out to be the problem, please tell me so I can update the instructions to be a little more clear.
Thank you!


Confidence interval question

I did a block maxima with a block period of one year and I fitted a gumbel distribution (this is snow data FYI). I used a strong confidence interval of 0.999.
block_size="365.2425D"
confidence_interval = 0.999
model = EVA(series)
model.get_extremes(method="BM", block_size=block_size, errors='ignore')
model.fit_model(model='MLE', distribution='gumbel_r')
model.plot_diagnostic(alpha=confidence_interval)

I was excpecting the confidence interval to be more like the one in the quickstart (picture below) https://georgebv.github.io/pyextremes/quickstart/

Do you know why there is a "linear" confidence interval ?
I have scipy 1.10.1 and pyextremes 2.2.7.

The reason is in the choice of distribution - you use Gumbel distribution which is exponential and appears linear when plotted at logarithmic scale. Also, if it's so tight at 99.9% confidence, this means that your choice of model was correct and that you have good data.

Ok thanks for your answer. I think I am misunderstanding the confidence interval. In the book  "An introduction to statistical modeling of extreme values" of S.Coles they give examples where confidence interval is computed based on a normal distribution. I saw in #36 that you are using a monte carlo method. If you have any details of this computation it would be amazing. I tried to dig in your source code but I admit that I did not understand everything.

Monte Carlo idea is really simple - you sample from your data (this way you have a "different" dataset), fit model to it, then repeat this process many times. In the end you have a large number of curves - one for each sample. To get 95% confidence interval you then take those curves and for each return period calculate return values and remove top and bottom 2.5% and then draw curves through those points.


Remove MetaMask custom token prompt

I don't think we need this prompt anymore since OUSD has been added as a supported token. However, we may need to reuse this for OGV or veOGV if we're unable to get those integrated quickly. For some reason, I didn't see this until after swapping my OUSD back to DAI after having swapped DAI for OUSD in the same session.

So for right now we dont need this prompt but in future we might need this right?

So for right now we dont need this prompt but in future we might need this right?

Correct, we should not be showing this prompt for OUSD. I'm seeing it after swapping DAI for OUSD despite the OUSD being shown in my MetaMask.

done via: https://github.com/OriginProtocol/origin-dollar/pull/979


Add release builds to CI

Adds CI builds from binary dockers. Depends on https://github.com/ros-planning/moveit2/pull/698

Just a note we should add the recent dockerfile changes to main (for galactic and rolling) tests to pass.

Re-invoking this since the sync is out @henningkayser @JafarAbdi @tylerjw


Nested {} blocks broke

I have a moderately complicated wildcard setup that includes blocks like {0-1$${in|at|near} a {0-1$$__location-adjective__} __location-noun__}. Until recently this worked fine, but currently it appears to treat the text in between the two $$'s as a separator even though it has {} in it. Was this a deliberate change?
Test : Expected : Result
{0$${in} a {0$$location-adjective} location-noun} : : location-noun
{0$${in} a {1$$location-adjective} location-noun} : : location-noun
{1$${in} a {0$$location-adjective} location-noun} : in a location-noun : location-adjective location-noun
{1$${in} a {1$$location-adjective} location-noun} : in a location-adjective location-noun : location-adjective location-noun

I'm using the automatic1111 webui, Dynamic Prompts 2.5.6.

Seems to be a bug alright – one would expect that to be parsed as a sequence but it's being parsed as one big variant in 10d1bd6f7bd6024cad753fdef8b95709ddcfa327.
VariantCommand(
    variants=[
        VariantOption(
            value=SequenceCommand(
                tokens=[
                    VariantCommand(
                        variants=[
                            VariantOption(value=LiteralCommand(literal="in"), weight=1.0),
                            VariantOption(value=LiteralCommand(literal="at"), weight=1.0),
                            VariantOption(value=LiteralCommand(literal="near"), weight=1.0),
                        ],
                        min_bound=1,
                        max_bound=1,
                        separator=",",
                    ),
                    LiteralCommand(literal=" a "),
                    VariantCommand(
                        variants=[VariantOption(value=WildcardCommand(wildcard="location-adjective"), weight=1.0)],
                        min_bound=0,
                        max_bound=1,
                        separator=",",
                    ),
                    LiteralCommand(literal=" "),
                    WildcardCommand(wildcard="location-noun"),
                ],
                separator="",
            ),
            weight=1.0,
        )
    ],
    min_bound=0,
    max_bound=1,
    separator=",",
)

This has been fixed in the feature/explicit-sampling-syntax branch which is currently in draft. It should be ready for testing soon

@adieyal I think that branch has been merged and shipped? Should this be closed?

Looks like there's still a parsing error somewhere
this works:
{0-2$$
    A|B|{2$$ and $$XY|Z}
}

but this doesn't
{0-2$$A|B|{2$$ and $$X|Y|Z}}


"[Keycloak Guard] Cannot handle token prior to" message

Hi
I got the error message of
KeycloakGuard\Exceptions\TokenException
[Keycloak Guard] Cannot handle token prior to 2020-12-02T01:54:16+0000

when try to verify the access_token.
My route is
Route::group(['middleware' => 'auth:api'], function () {
    Route::get('/protected', function () {
        return Auth::token();
    });
});

The route did returns full decoded JWT token from authenticated user only after several seconds after the token was generated.
I am not sure whether I need to clean cache or do something to make it work.
The error displayed on vendor/robsontenorio/laravel-keycloak-guard/src/KeycloakGuard.php: Line 41
    try {

      $this->decodedToken = Token::decode($this->request->bearerToken(), $this->config['realm_public_key']);

    } catch (\Exception $e) {

      throw new TokenException($e->getMessage());

    }

I am using Laravel 8

I need to wait several seconds, only then it will return the full decoded JWT token.

Yeah, i have issue with that. How to fix?

Yeah, I have same issue with that. How to fix?

Finally someone have the same issue. :D

I am on the latest L8 version and can't reproduce.
This problem occours to any account? Or specifics one?
Please, see https://github.com/auth0/auth0-PHP/issues/56

Maybe something related to  server time sync

I am on the latest L8 version and can't reproduce.
This problem occours to any account? Or specifics one?
Please, see https://github.com/auth0/auth0-PHP/issues/56

Maybe something related to  server time sync

Thank you very much sir. I follow the link provided, and turns out it related with leeway in the php-jwt. It works now!

Thank you very much sir. I follow the link provided, and turns out it related with leeway in the php-jwt. It works now!

@haizad Can you show me the code how to fix this?


display CordovaWebView controls on ThemeableBrowser?

I use the Ionic framework for development with ThemeableBrowser to display external links. If a button on the browser is to trigger an Ionic control (e.g. action sheet), the control would not be visible upon button pressed until the browser is closed. Is there a way to display these CordovaWebView controls on top of the browser, or I have to resort to native controls (with cordova plugins like this)?

The short answer is, you are right. ThemeableBrowser as well as InAppBrowser act more like a native browser on its own, except inside your app instance. So they create their own native layer of WebView/UIWebView (not CordovaWebView) that floats on top of CordovaWebView. While Ionic's directives and services are merely web elements that's inside CordovaWebView and styled to look native. So while ThemeableBrowser is being shown, nothing from the CordovaWebView underneath will become visible, since they are completely separate instances of WebView. (May even be completely different implementations if you factor in Crosswalk). However the good news is that despite being covered up, CordovaWebView is still active and can execute JavaScript on its own. So you have to utilize other Cordova plugins that gives you access to native widgets, so that they can get on top of ThemeableBrowser.
I can't say what your use case requires. I'd say the easiest course of action is to dismiss ThemeableBrowser first in response to any custom actions, so that the rest of the work flow can return to your app.

I need a few controls that can be triggered in both CordovaWebView and ThemeableBrowser, so it would simplify the structure as well as closely align the look and feel if I can use the same code. Thank you for your explanation and suggestion, to get things to market first I'm more inclined to dismiss the ThemeableBrowser to show the Ionic widgets at this moment, and come back to consider native ones later.


Numerical test test_chees_adaptation fails on aarch64-linux

Describe the issue as clearly as possible:
Using jax/jaxlib 0.4.28 and jaxopt 0.8.3, the following test fails on aarch64-linux:
FAILED tests/adaptation/test_adaptation.py::test_chees_adaptation - AssertionError:

Steps/code to reproduce the bug:
pytest

Expected result:
Tests pass.

I can't reproduce the failing test on my machine, so it's hard to debug. Making atol=1e-2 and rtol=0 would obviously fix it. Since the harmonic mean is random, increasing chains/steps/learning rates might not necessarily avoid the failing test.
Just for the sake of passing the test in aarch64-linux I would change rtol to atol, @junpenglao thoughts?

Yeah sure.

I can already confirm that the test is fixed thanks to @albcab's patch.
Maybe we can wait until the next release to mark this issue as closed.

Thanks for the feedback, it is a bit difficult for us to replicate, could you paste the full trace for all the test fail?
There is not much change on blackjax side re Chees, i suspect it is something upstream in JAX is causing this.

Sure, here it is: https://paste.glepage.com/upload/spider-bee-bison


Serving static files using Blazor routing

Is it possible to serve static files that are stored in the root folder(.PDF,.DOCX,.TXT, etc)?

Or do you mean the razor pages root folder?

Thanks for answering, @floreseken. And yes, wwwroot is the conventional place to put static content that you just want to serve as-is.


[v1.7.x] Initial parameter loading via SiK radios in QGroundcontrol stuck

I've found a strange issue when initial parameter loading sometimes stuck when using SiK radio.
If I use another channel (for example UART for companion computer) it loads successful. When this happens it persists until controller reboot. Qgroundcontrol reboot doesn't help. I've don't seen this on PX4 v1.6.
Sik radio: tried both mavlink v1 and v2 (from https://github.com/LorenzMeier/SiK).
nsh> mavlink status

instance #0:
GCS heartbeat: 355442 us ago
mavlink chan: #0
type: 3DR RADIO
rssi: 208
remote rssi: 200
txbuf: 99
noise: 64
remote noise: 54
rx errors: 0
fixed: 0
flow control: OFF
rates:
tx: 0.713 kB/s
txerr: 0.000 kB/s
rx: 0.051 kB/s
rate mult: 0.411
accepting commands: YES, FTP enabled: NO
MAVLink version: 1
transport protocol: serial (/dev/ttyS1 @57600)

instance #1:
mavlink chan: #1
no telem status.
flow control: OFF
rates:
tx: 7.217 kB/s
txerr: 0.000 kB/s
rx: 0.000 kB/s
rate mult: 0.511
accepting commands: YES, FTP enabled: YES
MAVLink version: 1
transport protocol: serial (/dev/ttyS2 @1500000)

https://logs.px4.io/plot_app?log=fc30cd9d-dbd4-45bb-9b78-aed21795e62f
screencast with qgroundcontrol:
https://drive.google.com/open?id=15QIV_hUYvJ9b_k1FzkWRM6zyYOaoDb4S
qgroundcontrol verbose logs:
qgc.log.zip
another screencast (with log capture process):
https://drive.google.com/open?id=19XxdTa-akUOsAeQBiqocP5shjNRPeGZd

➤ Daniel Agar commented:
I'll try to reproduce this locally with additional instrumentation/logging. Part of it looks like QGC/PX4 param sync getting into a bad state.

➤ Daniel Agar commented:
@DonLakeFlyer FYI

➤ Don Gagne commented:
@svpcom Make sure you are testing with a QGC daily build. There have been a large number of parameter loading fixing which have gone in since last stable.

➤ Vasily Evseenko commented:
I've tried with RTS/CTS flow control on SiK radio - stuck still eventually happens

➤ Vasily Evseenko commented:
@DonLakeFlyer Use 8bf441afa01b2acfa544012947a40635e8e8a6b0. Issue still happens
qgc.log.zip

➤ Don Gagne commented:
The key log entry is here: ParameterManagerLog: "V:1 C:1" Giving up on (paramIndex: 165 retryCount: 6 ) QGC asked for that param 5 times. It never got it so it gave up. Nothing QGC can do about that from its end. All other params came across ok.

➤ Daniel Agar commented:
I'll try to reproduce this locally with additional instrumentation/logging. Part of it looks like QGC/PX4 param sync getting into a bad state.

➤ Don Gagne commented:
@svpcom Make sure you are testing with a QGC daily build. There have been a large number of parameter loading fixing which have gone in since last stable.

➤ Daniel Agar commented:
@DonLakeFlyer FYI

➤ Don Gagne commented:
The key log entry is here: ParameterManagerLog: "V:1 C:1" Giving up on (paramIndex: 165 retryCount: 6 ) QGC asked for that param 5 times. It never got it so it gave up. Nothing QGC can do about that from its end. All other params came across ok.

➤ Vasily Evseenko commented:
I've tried with RTS/CTS flow control on SiK radio - stuck still eventually happens

➤ Vasily Evseenko commented:
@DonLakeFlyer Use 8bf441afa01b2acfa544012947a40635e8e8a6b0. Issue still happens
qgc.log.zip


Add golden mode in flatbuffer through python TTIR lowering infra

Add respective runtime components to fallback to host and do golden comparison

cannot append additional struct to this because function declaration only defines Operation and raw_ostream support
see: https://mlir.llvm.org/doxygen/namespacemlir.html#a86f6096a57734eed0290d2ab34ad5e79
TranslateFromMLIRFunction

@tapspatel any updates on this one ?
This one is marked as P0, and we need to report weekly updates on P0 bugs...

I've finished getting flatbuffer support and storing golden data into the flatbuffer itself. I'm working on runtime changes to do the golden comparison now.


[Bug]: Client creation hangs indefinitely on Azure, before raising an OperationalError

What happened?
When creating a persistent client on Azure, either inside a Notebook or on a compute the clients hang-on for 16min after creation:
client = chromadb.PersistentClient()
After more than 16 min getting this error message:
OperationalError: database is locked
Specifying a path doesn't help either, same results.
Versions
Chroma 0.4.9, Python 3.10, Ubuntu 20.04.6 LTS
Relevant log output
---------------------------------------------------------------------------
OperationalError                          Traceback (most recent call last)
Cell In[13], line 1
----> 1 persist_client = chromadb.PersistentClient()

File /anaconda/envs/azureml_py310_sdkv2/lib/python3.10/site-packages/chromadb/__init__.py:106, in PersistentClient(path, settings)
    103 settings.persist_directory = path
    104 settings.is_persistent = True
--> 106 return Client(settings)

File /anaconda/envs/azureml_py310_sdkv2/lib/python3.10/site-packages/chromadb/__init__.py:145, in Client(settings)
    142 telemetry_client = system.instance(Telemetry)
    143 api = system.instance(API)
--> 145 system.start()
    147 # Submit event for client start
    148 telemetry_client.capture(ClientStartEvent())

File /anaconda/envs/azureml_py310_sdkv2/lib/python3.10/site-packages/chromadb/config.py:268, in System.start(self)
    266 super().start()
    267 for component in self.components():
--> 268     component.start()

File /anaconda/envs/azureml_py310_sdkv2/lib/python3.10/site-packages/chromadb/db/impl/sqlite.py:93, in SqliteDB.start(self)
     91     cur.execute("PRAGMA foreign_keys = ON")
     92     cur.execute("PRAGMA case_sensitive_like = ON")
---> 93 self.initialize_migrations()

File /anaconda/envs/azureml_py310_sdkv2/lib/python3.10/site-packages/chromadb/db/migrations.py:128, in MigratableDB.initialize_migrations(self)
    125     self.validate_migrations()
    127 if migrate == "apply":
--> 128     self.apply_migrations()

File /anaconda/envs/azureml_py310_sdkv2/lib/python3.10/site-packages/chromadb/db/migrations.py:147, in MigratableDB.apply_migrations(self)
    145 def apply_migrations(self) -> None:
    146     """Validate existing migrations, and apply all new ones."""
--> 147     self.setup_migrations()
    148     for dir in self.migration_dirs():
    149         db_migrations = self.db_migrations(dir)

File /anaconda/envs/azureml_py310_sdkv2/lib/python3.10/site-packages/chromadb/db/impl/sqlite.py:149, in SqliteDB.setup_migrations(self)
    147 @override
    148 def setup_migrations(self) -> None:
--> 149     with self.tx() as cur:
    150         cur.execute(
    151             """
    152              CREATE TABLE IF NOT EXISTS migrations (
   (...)
    160              """
    161         )

File /anaconda/envs/azureml_py310_sdkv2/lib/python3.10/site-packages/chromadb/db/impl/sqlite.py:47, in TxWrapper.__exit__(self, exc_type, exc_value, traceback)
     45 if len(self._tx_stack.stack) == 0:
     46     if exc_type is None:
---> 47         self._conn.commit()
     48     else:
     49         self._conn.rollback()

File /anaconda/envs/azureml_py310_sdkv2/lib/python3.10/site-packages/chromadb/db/impl/sqlite_pool.py:31, in Connection.commit(self)
     30 def commit(self) -> None:
---> 31     self._conn.commit()

Hi, sounds like you are using Azure notebooks -  I believe this is due to the FS that azure notebooks is using.
https://stackoverflow.com/questions/53226642/sqlite3-database-is-locked-in-azure

Thanks!  What would be a solution to be able to use the PeristentClient?

@HammadB  Thanks for providing guidance.
Creating a temp dir seems to work, after which the contents can be copied manually to a more permanent destination:
`python
import tempfile
from pathlib import Path
chroma_path = Path(tempfile.gettempdir()) / 'chroma_db'
client = chromadb.PersistentClient(path=str(chroma_path))
`
This works, however having a clearer error message, and especially not waiting for 16 mins to get it would be very helpful?

I am experiencing a similar issue on my Mac in Python 3.10.  My script creates a PersistentClient, does a simple list_collections(), and finishes.  However the Python script does not exit immediately.  It hangs on for a minute or two before releasing.  This is a concern because when the object hangs on like this, it affects scalability.  And there is no explicit close() method for the object. Is this a bug or expected behavior?

+1 I am facing the same issue. Any fix?

same problem here, chromadb==0.5.1