Conflict with Postgresql jsonb operators

Hi!
I have sql:
SELECT id, name, price, files->'photos'->0->>'name' as photo, annotation
FROM product
WHERE tags ?& array['17']
ORDER BY (tags->>'17')::int
LIMIT 10

If i run this sql direct in postgresql client it work fine
If i run this sql through node-postgres it work fine again:
var pg = require('pg');

var connPg = "postgres://user:password@localhost/mydbname";
var clientPg = new pg.Client(connPg);
var sqlProductsByTags = [
  "SELECT id, name, price, files->'photos'->0->>'name' as photo, annotation ",
  "FROM product ",
  "WHERE tags ?& array['17'] ",
  "ORDER BY (tags->>'17')::int ",
  "LIMIT 10 "
];

clientPg.connect(function(err) {
  if(err) {
    return console.error('could not connect to postgres', err);
  }
  //console.log('SQL = ' + sqlProductsByTags.join(''));

  clientPg.query(sqlProductsByTags.join(''), function(err, result) {
    if(err) {
      return console.error('error select products by tags', err);
    }

    for(var i = 0, max = result.rows.length; i < max; i++) {
      console.log('name: ' +  result.rows[i].name);
    }

    clientPg.end();
  });
});

If i run raw sql by knex, it fail with error (below listed):
var connPg = "postgres://user:password@localhost/mydbname";
var pg = require('knex')({
  client: 'pg',
  connection: connPg
});
var sqlProductsByTags = [
  "SELECT id, name, price, files->'photos'->0->>'name' as photo, annotation ",
  "FROM product ",
  "WHERE tags ?& array['17'] ",
  "ORDER BY (tags->>'17')::int ",
  "LIMIT 10 "
];

pg.raw(sqlProductsByTags.join(''))

This Error
{ [error: syntax error (at or near "$1")]
  name: 'error',
  length: 142,
  severity: 'ERROR',
  code: '42601',
  detail: undefined,
  hint: undefined,
  position: '98',
  internalPosition: undefined,
  internalQuery: undefined,
  where: undefined,
  file: 'scan.l',
  line: '1053',
  routine: 'scanner_yyerror' }

I think that it fail because of i used jsonb operator '?&', precise only one symbol '?'
Reference to PostgreSQL JSONB operators

One way of fixing this completely (and keep things future-proof for other dialects) would be to generate the SQL text as an array instead of a flat string, starting a new array element at the position of each binding.
For example, instead of:
{ method: 'select',
  options: undefined,
  bindings: [ 3857, 999999 ],
  sql: 'select "geom", ? as "srid" from "map" where ("id" = ?)' }

...we'd have:
{ method: 'select',
  options: undefined,
  bindings: [ 3857, 999999 ],
  sql: ['select "geom", ', ' as "srid" from "map" where ("id" = ', ')'] }

This means that each dialect would get the appropriate placeholders by just calling .join() with the placeholder (or, in the case where an incrementing number is needed in the placeholder, using .reduce()), and this wouldn't interfere with operators like ?.

Any updates on this issue?
will it reach the master ?
I'm also having several problems using postgresql jsob operators.
Thanks.

+1 Is there any workarounds how I could make queries with json ?,?|,?& operators already with latest stable knex version?

I'm also curious whether this will be fixed soon. I'll need this before I can switch my generic web-based query tool over to using knex, since this would prevent the user from being able to use the JSONB operators.
@tgriesser Your PR looks fine for now, since it would provide a workaround for the time being. Eventually, I think it might make more sense to switch to something like I suggested above, but for now I think it'd be useful to at least get some fix merged.

Hi @whitelynx. I believe @tgriesser is currently away but I'm happy to get a fix merged. If someone who knows the code base is able to review and update the PR as necessary, I'm happy to facilitate by merging it into master.

@tgriesser +1, I use JSONB all over the place :)

This is also a problem with hstore methods.
    ELSIF hstore(NEW) ? 'is_bycatch' THEN

This is also a problem when using knex.raw on a constant which includes a ?.  Below is an oversimplified example that is easy to work around, but in my real situation I can't avoid the knex.raw().
Code:
console.log(knex.table('blah').insert({col1: knex.raw("'http://www.google.com/?q=foobar'"), col2: {a:1}}).toString());

SQL:
insert into "blah" ("col1", "col2") values ('http://www.google.com/'{\"a\":1}'q=foobar', ?)

@zacronos said:

This is also a problem when using knex.raw on a constant which includes a ?. Below is an oversimplified example that is easy to work around, but in my real situation I can't avoid the knex.raw().

I believe the correct way to do that with the current version of knex is to provide it as a parameter (then it is automatically escaped correctly, as well as avoiding this issue):
console.log(knex.table('blah').insert({col1: knex.raw('?', [' http://www.google.com/?q=foobar']), col2: {a:1}}).toString());

@whitelynx, ah, that does seem to work.  Thanks!

This has been a big issue for us. Have recently started to use jsonb columns. Any update on the PR?

@eknkc There is an existing PR. Not sure what the hold up is.

Actually looks like it's passing, but I don't know the code base well enough to review it.

@rhys-vdw, the PR passes tests, but it has merge conflicts.
I took a look myself earlier to see what it would take, and unfortunately the code involved has changed quite a bit since the PR was created.  I believe at least one of the modified files no longer even exists.  It's possible it would be easier for someone to create a new PR from scratch rather than try to merge the existing one, but I don't know the codebase, so I can't say for sure.

The JDBC driver seems to have solved this using ?? as an escape for literal ?: pgjdbc/pgjdbc#227
One could write this in that case
knex.raw("SELECT * FROM items WHERE tags ?? ?", ["tagname"])

for
SELECT * FROM items WHERE tags ? 'tagname'

I would expect a more general purpose escaping scheme like \? or ?? would be better than just special-casing question mark operators.

+1 for ??

Except, looks  like ?? is already used for wrapping binding value to be used as an identifier e.g.
raw("??", ['TableName.colName'] => "TableName"."colName"
raw("?", ['TableName.colName'] => 'TableName.colName'

https://github.com/tgriesser/knex/blob/master/test/tape/raw.js#L12

Good catch, @elhigu.
Perhaps \? would be more appropriate then?

Looks like we need to do \? actually with \\? since javascript requires that we escape \ char in string. However I added pull request with implementation, test and documentation. Should I run also npm run build and add commit the generated code as well?

I know it's been a couple of months but I am still in this predicament on 0.9
can anyone let me know what is going on with this query?
qb.whereRaw("tags \\?& array['" + Array(tags.length + 1).join("?").split("").join("','") + "']", tags)

the error i get is
select "id" from "tablename" where tags \$1 array['foo','bar'] and "locale" = $2 order by "created_at" desc limit $3 offset $4 - syntax error at or near "\"

It seems the escape sequence doesn't get applied in any case. I would appreciate it if anyone can give light to this.

I'm getting same kind of results... looks like e.g. when one calls .toString() for query builder, it cannot handle //? escaping correctly.
knex.from('test_table_two').whereRaw("(json_data->'me')::jsonb \\?& array['keyOne', 'keyTwo']").where('id', '>', 1).toString()

select * from "test_table_two" where (json_data->\'me\')::jsonb \\\'1\'& array[\'keyOne\', \'keyTwo\'] and "id" > ?

Normal case when one just executes query it goes a bit different route to pg and ? replacement is never done by knex, but they are just converted to $1, $2 etc bindings of prepared sql statement for postgres.
I'll write test for this case so that //? escaping is handled for .toString() replacements too.

@jamoy I added pull request to fix your issue

thank you very much @elhigu wish I knew the codebase enough to contribute. hoping this would get merged soon :+1:

This still isn't working

@SeanCannon, in my codebase I saw this working just fine over a year ago after it was fixed.  Chances are very good that if you're seeing a similar issue, it is a new problem, not a continuation of this problem.
I suggest you create a new Issue, complete with the version of knex you're using, example code that demonstrates the problem, and output of the error and/or expected vs actual results.

@zacronos https://github.com/tgriesser/knex/issues/2225

Be sure you have jsonb field but not json !!


fundamental issue with pub/sub

Hi Justin
http://stackoverflow.com/questions/29906637/node-js-shutdown-hooks
is this a fundamental problem with pub sub? How can we ensure the last published message gets sent before an exit?

This is a fundamental problem with process.exit(). You should really try avoid using that. No asynchronous operation (and yes, you should consider sending over a ZMQ socket to be async) will survive a process.exit call.

By the way, as much as I love stackoverflow, but why post your issue there instead of here in the issue tracker in the first place? :)


cockroach: support building with Xcode 8.3

Binaries built with Xcode 8.3 immediately fail to execute with Killed: 9. See golang/go#19734 for details. The workaround is to pass -ldflags -s; this commit accomplishes that by building with LINKFLAGS=-s.
Fixes #7.

@benesch sure you're linking to the right issue on golang/go?

https://github.com/golang/go/issues/19734

Good catch, @tschottdorf, thanks!


new web app plugin version and authentication ways (#885)

Codecov Report

Merging #888 into develop will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             develop     #888   +/-   ##
==========================================
  Coverage      44.46%   44.46%
  Complexity       766      766
==========================================
  Files            135      135
  Lines           5184     5184
  Branches         748      748
==========================================
  Hits            2305     2305
  Misses          2705     2705
  Partials         174      174

Continue to review full report at Codecov.

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


Prevent Extensions from breaking without providing info in case of Vue incompatibilities

Description
The upgrade of Dashboard to Vue3 will have as a consequence to break Rancher Extensions. This will include several case scenarios which will have to be covered:

Update Dashboard version to Vue3 with old plugins with Vue2
Install new plugins with Vue3 to older Dashboard version with Vue2
Install old plugins with Vue2 in a new Dashboard with Vue3

Implementation
Current
We currently just let extensions break the UI and it can be fixed using ?safemode query parameter (docs).
Planned
TBD
Note: This is going to be part of the outcome for https://github.com/rancher/dashboard/issues/11049
Examples

https://github.com/rancher/dashboard/issues/10675

This has been completed with https://github.com/rancher/dashboard/pull/12071, closing.


[zh] Sync PR wranglers reorg

The PR wanglers section was reorganized in 22284. This PR syncs the changes to Chinese localization.

/lgtm
/approve


if one entity has multiple mentions,how to deal this situation?

Thanks for sharing you code! in the paper you use cosine cross entropy as loss function

and the code in the file models.py
`loss_fct = nn.CrossEntropyLoss()
# Calculate loss with hard negatives
if num_sent == 3:
    # Note that weights are actually logits of weights
    z3_weight = cls.model_args.hard_negative_weight
    weights = torch.tensor(
        [[0.0] * (cos_sim.size(-1) - z1_z3_cos.size(-1)) + [0.0] * i + [z3_weight] + [0.0] * (z1_z3_cos.size(-1) - i - 1) for i in range(z1_z3_cos.size(-1))]
    ).to(cls.device)
    cos_sim = cos_sim + weights

loss = loss_fct(cos_sim, labels)`

but if the input is <mention,entity> pair,and one entity may has multiple mentions,so  one entiy may have multiple mentions labeled 1,how to process this situation? looking forward you answer

So you mean there are multiple "positive pairs" for one instance? In that case, you can simply add all the positive instances to the nominator.


--default-action parameter not recognized

I am trying to set the default network access rule to Deny via Azure CLI as described in the article. When running the command, it says that the --default-action parameter is an unrecognized argument. How can I by default deny network access using the CLI?

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

ID: 46701fde-2216-84d6-467c-67e6ed85b847
Version Independent ID: 92beb36b-f717-b3b4-f1b8-9e0caf832621
Content: Configure Virtual Networks for Azure Cognitive Services - Azure Cognitive Services
Content Source: articles/cognitive-services/cognitive-services-virtual-networks.md
Service: cognitive-services
GitHub Login: @aahill
Microsoft Alias: aahi

Document is mistaken, we don’t support modifying this property via az cognitiveservices CLI.
You can use Az Resource CLI
az resource update --ids {resourceId} --set properties.networkAcls="{'defaultAction':'Allow'}"
or you can use Az PowerShell which has full contract support.

@yangyuan Thanks for the command above. Can you update the article and replace the wrong Azure CLI commands against the correct ones?

Yes, I'll update the documentation shortly.
#please-close


Support both dev and prod

/lgtm
/approve
We can add support for dev registry mirroring later

/cherrypick release-0.14


Session Handling and Token Invalidation

Hello,
Recently, Bluesky added access tokens (https://github.com/MarshalX/atproto/issues/151), and right now, the library allows exporting a session string (client.export_session_string()). This string includes the handle, DID, their access token, and a refresh token.
Previously, the refresh token had been implemented with a 2-month lifetime (and then developer should generate another refresh token with client.login(login=..., password=...)), but now, the CreateSession function returns another refresh token with another 2-month lifetime.
In my codebase, it's something like this:
def app_init():
    # On app init
    client = Client()
    client.login(BLUESKY_USER_HANDLE, BLUESKY_USER_APP_PASSWORD)
    with open(BLUESKY_SECRET_FILE, "w") as f:
        f.write(client.export_session_string())

def post_message(text):
    session_string = "..."  # content from file
    client = Client()
    client.login(session_string=session_string)

    if client._access_jwt and client._should_refresh_session():
        client._refresh_and_set_session()
        # Update file with new session
        with open(BLUESKY_SECRET_FILE, "w") as f:
            f.write(client.export_session_string())

    # client.send_post(...)

My point here is that protected methods are being used to resolve this problem with autorefresh. Is it better to make them public (if the token needs to be invalidated in this way)?
My initial expectation is that methods like client.send_post()/client.com.atproto.repo.create_record()/etc will take care of token invalidation (if needed), but I receive an error, and it looks like invalidation doesn't work for those methods.

Hi! But SDK cares about refreshing of access token using refresh token for a long time 🧐 was added here https://github.com/MarshalX/atproto/pull/27
The problem that I can understand that we must export session every 2 hours because of refreshed tokens 😢
Or do you want so say that auto refresh logic was broken is SDK?

I mean that all methods are checked if the session is expired and renew it

I mean that all methods are checking if the session is expired and renewing it if necessary
Aah, thanks! I'll recheck my implementation – looks like something wrong with tokens on my end.

The problem that I can understand that we must export session every 2 hours because of refreshed tokens

Yeah, you're correct! I guess I can do something like this one:
def post_message(text):
    session_string = "..."  # content from file
    client = Client()
    client.login(session_string=session_string)

    client.send_post(...)

    if client.export_session_string() != previous_session_string:
        # export it again

But probably some more elegant way is needed 🤔

We definitely should save the session to persistent storage at the end of the script. But we also must catch all errors first to not exit without saving the exported session.
This doesn't fit well with envs, CI/CD secrets, and so on 🥲 sad

We can use the exported session for two months before updating it in persistent storage. Isn't it?

There is also an idea to add an option to store the exported session as a file (this file could be always updated when a new token is generated, so you don't have to catch all errors) when Client() is initialized. Also, looks like some user/password fallback is needed as well.

We can use the exported session for two months before updating it in persistent storage. Isn't it?

You're correct in general, but It is something strange happens there with token with expiration date 2 days ago

and refresh token with expiration in December:

This token raises an error on client.login(session_string=...):
atproto.exceptions.BadRequestError: Response(success=False, status_code=400, content=XrpcError(error='ExpiredToken', message='Token has been revoked'), headers=Headers({'date': 'Tue, 19 Sep 2023 19:48:13 GMT', 'content-type': 'application/json; charset=utf-8', 'content-length': '59', 'connection': 'keep-alive', 'x-powered-by': 'Express', 'access-control-allow-origin': '*', 'ratelimit-limit': '3000', 'ratelimit-remaining': '2999', 'ratelimit-reset': '1695153193', 'ratelimit-policy': '3000;w=300', 'vary': 'Accept-Encoding'}))

🤯🤯🤯

I'd assume that something is broken on bsky side with this token – with newly generated tokens, everything works fine! I think we're good to use the same exported session for 2 months!

I can add the ability to register callback on token refresh. And the user can implement storing this token in db or file it whatever

I'd assume that something is broken on bsky side with this token – with newly generated tokens everything works fine! I think we're good to use the same exported session for 2 months!

Didn't you changed password or remove app password?

Didn't you changed password or remove app password?

Nope. Furthermore, for a token that was generated yesterday, I'm getting the same "Token has been revoked" error. Password/handle haven't been changed, and I am still able to generate new tokens with the same password.
Token generated at<PHONE_NUMBER>, exp=1695157990
Refresh token generated at<PHONE_NUMBER>, exp=1702926790 (Dec, 18)
>>> client = Client()
>>> client.login(session_string=sec)
atproto.exceptions.BadRequestError: Response(success=False, status_code=400, content=XrpcError(error='ExpiredToken', message='Token has been revoked'), headers=Headers({'date': 'Wed, 20 Sep 2023 09:07:36 GMT', 'content-type': 'application/json; charset=utf-8', 'content-length': '59', 'connection': 'keep-alive', 'x-powered-by': 'Express', 'access-control-allow-origin': '*', 'ratelimit-limit': '3000', 'ratelimit-remaining': '2998', 'ratelimit-reset': '1695201121', 'ratelimit-policy': '3000;w=300', 'vary': 'Accept-Encoding'}))

...

>>> client._import_session_string(sec)
SessionString(...)
>>> client._should_refresh_session()
True
>>> client._refresh_and_set_session()
atproto.exceptions.BadRequestError: Response(success=False, status_code=400, content=XrpcError(error='ExpiredToken', message='Token has been revoked'), ...)

so... does it mean that when you refresh the session and get a new refresh token, the old one is revoked?


[question] how to get additional empty variable inside state

Hi, thank you for your work and your good choice: mealy-machine!
I've a little question, I've create new Machine thats work inside Vue.js!
Now I want to get additional parameter parentAction = new  inside the state.
var stent = Machine.create('actions', {
  state: { name: 'start', parentAction: 'new' },
  transitions: {
    'start': {
      'take note': 'note',
      'create ordine': 'order',
      'take party': 'party'
    },
    'note': {
      'take another note': 'note',
      'create order': 'order',
      'take party': 'party'
    },
    'party': {
      'add participant': { name: 'participant', parentAction: 'new' }
    },
    'participant': {
      'create ordine'': 'order',
      'add participant': 'participant',
    },
...

The first state start has stent.state.parentAction = 'new', correct,
but when I change the state to note the stent.state.parentAction is always new.
There is a way to get undefined, null, empty variable for states that haven't stent.state.parentAction defined?

Hey, that's probably a bug. I'll investigate and get back to you asap.

To simplify, here's my issue simulated without vuejs: https://runkit.com/menteora/stent-issue14

Hey I think I misunderstood your use case. That's how it works by design. If you use the short notation for changing state only the name property is updated keeping the rest untouched. In order to remove/change parentAction you have to use an object, function or a generator. For example:
var { Machine } = require("stent");

var stent = Machine.create('actions', {
  state: { name: 'start', parentAction: 'new' },
  transitions: {
    'start': {
      'take note': { name: 'note' }
    },
    'note': {
      'take another note': { name: 'note', parentAction: null }
    }
  }
})

const appMachine = Machine.get('actions');
console.log(appMachine.state.name, appMachine.state.parentAction);
appMachine.takeNote();
console.log(appMachine.state.name, appMachine.state.parentAction);
appMachine.takeAnotherNote();
console.log(appMachine.state.name, appMachine.state.parentAction);

Great! I totally missed this particular, sorry.
Thank you!

No worries. Let me know if something else is not working as expected.


Update dependencies to include security fixes

Please see: https://github.com/opengeospatial/cite/wiki/Dependencies-of-test-suites

<dependency>
  <groupId>com.github.everit-org.json-schema</groupId>
  <artifactId>org.everit.json.schema</artifactId>
  <version>1.13.0</version>
</dependency>

Contains as a transitive dependency commons-validator:commons-validator:1.6. However, other test suites use commons-validator:commons-validator:1.7.

Created pull request to update dependency rest-assured: https://github.com/opengeospatial/ets-eo-geojson10/pull/2


optimal_transport days information missing for *** cells

Thanks for developing such nice analysis tool.When I run
wot optimal_transport --matrix  my_matrix_file_path  --cell_days my_cell_day_file_path--out my_tmaps
but I got the error "Days information missing for (the cell number) cells
No day pairs"
I got the two files form as follows
https://broadinstitute.github.io/wot/cli_documentation/#matrix_file
I do no know what is the problem,I got stuck in this for few days,can you help with this?
Thanks in advance!

can you post here a snippet of your two files?
I mean the matrix and the day information.
I have had a similar problem. What I find out was that the day column was not identified as integer values and therefore couldn't be processed. do you have only numbers in the second column?

"id"	"day"
"4hpf_AAAGATGCAGCTGTAT"	4
"4hpf_AAAGCAATCAACACCA"	4
"4hpf_ACTATCTCAGCCTATA"	4
Thanks for your answer,this is my day_file snippet ,I only have numbers in the second column.
I can not figure out what is the problem.

is it tab-delimited?
can you also remove the "?

OK,I try this.

I have solved the problem,Thank you very much！

what was the problem or the solution?

pls close the issue, if done.

I remove "" in the file


what encoding does cppsafe support ?

I have some .cpp file's encoding is GB2321. And seems that  cppsafe can't support it , because cppsafe outputs many error , which describe that  GB2321 Chinese words can not be recognized.
Does this have any solution?
thanks for helping.   ~v~

fix by:
-Wno-invalid-source-encoding


How implement query sections outside of query=>bool?

For example how I can get highlight section?
There is solution to get non eloquent fields?

The search method returns collection of models, there is no place for meta data like highlighting. You can use the searchRaw method to specify any additional parameters you want. But keep in mind, that it returns raw data from ES.


Add support for ticks_tooltip option

This PR adds support for seiyria/bootstrap-slider ticks_tooltip option.
Also updates library version in bower.json as such option was added in version 9.3.0

I kinda want to get this done and out of the way so i'll accept the PR and take the testing piece on this morning.

Thanks @ksnyde !

Thank you. 👍
BTW, you can find published on npm as version 0.5.0.


Cannot read property 'Constants' of undefined

Current Behavior
I have a CRNA ejected app. I've followed the video for setting up the project, also including the instructions in #144 otherwise there are more issues.

and it compiles with these errors

then I replace all these imports as following

and the build succeeds.
My bridging header files is
#import <React/RCTRootView.h>
#import <React/RCTView.h>
#import "React/RCTBridgeModule.h"
#import "RNVideoProcessing.h"

and when I import { ProcessingManager } from 'react-native-video-processing'; I get this error

Expected Behavior
no error

Your Environment

software
version

react-native-video-processing
1.13.0

react-native
0.50.3

node
v8.9.0

xcode
9.2

swift version
3.2

Have you added the GPUImage.framework under Embedded Binaries ? Just to make sure :) Though it's unlikely to solve the issue anyway.

i still face the issue cannot read property of constant undefined how to resolve it please help


View admins E2E

Coverage remained the same at 100.0% when pulling 2588e2d02999898a255515f037b9ad42518478ac on view_admins into 00921ac9a4bb5c05eab6d5708f0da6ff30384bc9 on master.


Add ST_LineMerge function from GEOS

While building a native OSM reader, every existing implementation I have found relies on the ST_LineMerge or linemerge function that is imported from GEOS. Since GEOS is already in the extension, maybe it won't be a huge problem to add it (even with mapping from native LINESTRING_2D to GEOS object first).
For now, I have utilized the python function registering, but this won't be as fast as the native function.
And on a similar topic, do you know how can I split the MultiLineString into the list of LineStrings? ST_NGeometries returns a value 5, but I don't know where to go from here without another python function and iterating over geoms. The same question probably applies to MultiPoints and MultiPolygons.
Edit: I have found the ST_GeometryN function in PostGIS that allows for the extraction of subgeoms. So maybe it's another function to add to duckdb?
from shapely import wkt as shp_wkt
from typing import List, cast
from shapely.ops import linemerge
import duckdb

connection = duckdb.connect(config=dict(allow_unsigned_extensions=True))
connection.install_extension('spatial') # loading from github action artifact
connection.load_extension('spatial')

def _linemerge_shapely(wkt_strings: List[str]) -> str:
    linestrings = [shp_wkt.loads(wkt_string) for wkt_string in wkt_strings]
    merged_linestring = linemerge(linestrings)
    merged_linestring_wkt = cast(str, merged_linestring.wkt)
    return merged_linestring_wkt

connection.create_function(
    "py_linemerge_shapely",
    _linemerge_shapely,
    [duckdb.list_type(type=duckdb.typing.VARCHAR)],
    duckdb.typing.VARCHAR,
)

duckdb_linemerge_macro_query = """
CREATE OR REPLACE MACRO ST_LineMerge(geometries) AS
ST_GeomFromText(py_linemerge_shapely([ST_AsText(g) for g in geometries]));
"""
connection.execute(duckdb_linemerge_macro_query)

connection.sql('''
SELECT ST_LineMerge([
    ST_GeomFromText('LINESTRING (0 0, 1 1)'),
    ST_GeomFromText('LINESTRING (0 0, 0 1)'),
    ST_GeomFromText('LINESTRING (0 1, 1 1)'),
    ST_GeomFromText('LINESTRING (1 1, 1 0)'),
    ST_GeomFromText('LINESTRING (1 0, 0 0)'),
    ST_GeomFromText('LINESTRING (5 5, 6 6)'),
    ST_GeomFromText('LINESTRING (1 1, 100 100)'),
]) as merged_lines
''')

┌────────────────────────────────────────────────────────────────────────────────────────────┐
│                                        merged_lines                                        │
│                                          geometry                                          │
├────────────────────────────────────────────────────────────────────────────────────────────┤
│ MULTILINESTRING ((1 1, 1 0, 0 0), (0 0, 1 1), (0 0, 0 1, 1 1), (1 1, 100 100), (5 5, 6 6)) │
└────────────────────────────────────────────────────────────────────────────────────────────┘

Hello! Thanks for raising this issue!
I agree that ST_LineMerge along with other constructive aggregates like ST_Polygonize would be nice to expose since those are some of the really valuable operations provided by GEOS.  I'll look into it.
To split the geometrycollection, it looks like we would need something like ST_Dump. Its shouldn't be too hard to implement, its just the path return-value that's slightly annoying, but we could return it as part of a STRUCT.


RTC not keeping track of time - VBAT Pin (IDFGH-4265)

Hello guys, how are u?
I'm having some troubles using the internal RTC of ESP32. I'm used to develop with microcontrollers that has internal RTC and also a pin called VBAT Pin, that i'm able to wire a coin cell battery to this pin and the system still keeps track of time, even if it has no power supply.  Is there any VBAT pin in ESP32? I read on some other topic to wire the coin cell on GPIO36, but it didn't work, time reseted after power went off.
I know i can wire a battery to VCC but due to PCB and budget requirements, this is not possible.This is the same reason why i can't use DS3231 or something similar either. Also, i tought since ESP32 has an internal RTC, why to use an external module? Unless, there is no way to keep track of time while powered off.
I may or may not have internet to get the time from SNTP when the system restart, that's why i need RTC to keep counting. And time is a vital parameter for my system to execute properly.
I really hope ESP32 have a way to keep track of time with a VBAT pin, it would be weird to me that such a robust microcontroller doesn't have this feature like other common devices on the market.

Hello @matheusmbgeng
I have not yet come across an ESP32 with a VBAT pin. And GPIO36 is simply an analog input pin which to my knowledge cannot be used to power the RTC or anything else for that matter.
For the RTC in the ESP32 to be running the ESP32 needs to be powered, but it can be in light or deep sleep to conserve power.
However please note that with the internal oscillator the RTC is not very precise. You can improve that by adding an external 32kHz crystal or oscillator.
Check out section 2.4.2 RTC (optional) in the ESP32 Hardware Design Guidelines about what clock sources are available for RTC.
Also if you haven't looked at the ESP32 datasheet document I recommend you do that.
BTW: Searching though it with term bat yields exactly one hit which talks about when powering the complete chip by battery.
Thanks
Felix

Hi @matheusmbgeng,
This issue tracker is for bug reports and feature requests for ESP-IDF project. For chip-related questions, such as the power supply scheme, please use either the esp32.com community forum or contact Espressif technical support at https://www.espressif.com/en/contact-us/technical-inquiries/hardware-issues.


[9.x] Update Docblock Methods for Cache.php

I added missing (public) methods from \Illuminate\Cache\CacheManager and \Illuminate\Cache\Repository in Cache.php
At the end i also sort all of the methods in alphabetical order

I don't understand why some methods were removed entirely.


Add golos.memo abi to store memo keys from Golos

memo is not used now, but it's useful to have it in genesis. create table to store it

PR #697


Possible bug when using datetime for default arguments

The following default arguments might be a problem: https://github.com/dimartinot/gee_sar_fetcher/blob/790201555e6c93a27e31e026be77505e490b1706/geesarfetcher/init.py#L30-L31
Default arguments are evaluated at import time. If importing is performed at 23:59 in an interactive session, then running the script at 00:01, a day will be missing from the results. Likewise in a long running script which might start at 23.40 and fetch() gets to run after 00:00.
Suggested solution
def foo(start_date = None):
    if start_date is None:
        start_date = datetime.today() - timedelta(days=365)

or make these mandatory arguments.
(Kindly suggested by https://github.com/pmav99)

This is a very good suggestion, I will work into it.
Thanks,


Add cues for when there are unsaved changes and prevent user from navigating away

Just added autosaving. Should still stop user from refresing or navigating away from the page.


Update stale K8s docs after directory restructure

@apc999 @calvinjia fyi
@jiacheliu3 we can can update the docs further post-merge

alluxio-bot, merge this please.


Bugfix/category hydration issues

Short description and why it's useful

Fixes hydration errors caused randomly by not fetching scripts before client hydration started.
This should increase TTI and get rid of this problem.
During tests, this error hasn't shown up even once. Haven't noticed any unpleasant side-effects.
QA - please test it thoroughly by refreshing category page multiple times.
As change is in the root of the application, please also check Other pages to be sure.
Edit: Extended explanation
The error appears mostly on Linux-based systems distributions (with docker, Linux, mac). When a client gets server image it starts hydrating too fast, even if didn't get yet route component or it's dependencies.
To avoid that kind of situations route chunks (and all dependencies) should be smaller. In some points, client browser can skip waiting for all dependencies and start hydrating too soon.
To switch off this new set of lazy hydrating route component we have the property:
themeConfig.lazyRouteComponentHydration
in a config file, which by default is set to true
Which environment this relates to
Check your case. In case of any doubts please read about Release Cycle

[ ] Test version (https://test.storefrontcloud.io) - this is a new feature or improvement for Vue Storefront. I've created branch from develop branch and want to merge it back to develop
[ ] RC version (https://next.storefrontcloud.io) - this is a stabilisation fix for Release Candidate of Vue Storefront. I've created branch from release branch and want to merge it back to release
[x] Stable version (https://demo.storefrontcloud.io) - this is an important fix for current stable version. I've created branch from hotfix or master branch and want to merge it back to hotfix

Upgrade Notes and Changelog

[x] No upgrade steps required (100% backward compatibility and no breaking changes)
[ ] I've updated the Upgrade notes and Changelog on how to port existing VS sites with this new feature

IMPORTANT NOTICE - Remember to update CHANGELOG.md with description of your change

Try if switching off lazy layout loading in App.vue is solving this issue - it would be probably a btter (simpler) solution:https://github.com/DivanteLtd/vue-storefront/blob/463036f87461e9142caeb54fd092be4ca8294228/src/themes/default/App.vue#L11
Switching to normal eager imports, commenting out not-used layouts

Cool; let's merge it in

cc @dawerx - pleas confirm that works for you guys :)

I think this change requires more testing. It could cause some problems on low end mobile devices or on slower network connections. Because you are lazy-hydrating the whole page, if user interacts with it before hydration (which has higher probability of happening when the page is loading for longer period of time - low end phones and slow network), Vue discards the page and re-renders it from scratch. This causes the page to disappear for a brief moment and resets the whole state (so for eaxmple selected filter won't be selected anymore).
It's possible that most of these issues will be automatically solved by caching from service worker. Not on initial load, though.
That's why I would suggest testing it on production mode, on real devices and real networks.

Hi.
I agree with Adam.
The fix seem to be working, I was not able to reproduce the SSR issue on vanila VSF, however we should dig deeper to identify the core problem.
The page seem to be slower though, menu content appears after 1 second.
I will test this with Dr. Max VSF instance as well.

Please note that the impact of this issue (and the solution) is pretty limited as it can occur only when hydration takes place which is always just the 1st page load (all the subsequent requests are processed 100% CSR)

@adamjedlicka , @Darwex guys, there is also another way to help you with your problem quickly. Not too pretty but should work if you would turn off lazy loading on the category page. This one is causing your problems so it would be the quickest way for fix.
Then you'll just change lazyRouteComponentHydration config to false. And in router:
const Category = () => import(/* webpackChunkName: "vsf-category" */ 'theme/pages/Category.vue')
// change to
import Category from 'theme/pages/Category.vue'

We tested our develop which has category module rewritten. Problem is not appearing there, could you check it by yourselves on develop branch if you're able to reproduce it there?
Then with the new version, you could return to lazy load category page and everything should be good.
Please let us know what do you think!

@patzick I was still able to reproduce it on the latest develop of VSF. But it took a long time (about 10 minutes). The reason why it's much more rare on the develop is because of this commit c2c6e8f9f69d76ed8dd487fd48aa4a1aa56cf73d . This one is on the right track of solving the issue properly but there is still some race condition left.

@adamjedlicka this PR hasn't been yet merged to develop. please make syoure you've tested rhe right version - this PR not 1.11.rc1 (develop)
git checkout -b patzick-bugfix/category-hydration-issues hotfix/v1.10.1
git pull https://github.com/patzick/vue-storefront.git bugfix/category-hydration-issues

@pkarw Are you sure it's not merged? I can see it here: https://github.com/DivanteLtd/vue-storefront/blob/develop/core/client-entry.ts#L28 and here: https://github.com/DivanteLtd/vue-storefront/blob/develop/core/modules/url/store/actions.ts#L32

@adamjedlicka I meant that this PR (#34030) hasn't been yet merged :-)

@adamjedlicka small misunderstanding here between you guys; we'll be checking to develop also to provide a better fix for this issue.
But for this version, there is nothing more we can do I think. Have you guys considered turning off lazy load category on the router or using lazy hydration from this PR?  Vue should abstract the difference away, if there is bug in Vue in lazy loading of components I'm sure someone else in different project would already noticed.
Also it does not happen only in category. It happens anywhere where dynamic URL dispatch is used (category, product, ...). That's why I think it's caused by the router.

OK, but to just sync; @adamjedlicka does this PR (3403) with a lazy-hydration for the whole page content solves the issue you?

@Darwex is testing it right now. I'm fairly certain that this will fix the SSR issue because it is basically the same as my fix, so we will probably just merge this change and be done with it. But let's wait for David's final opinion.

OK, cool thanks for confirmation - @patzick is double-checking one more another hypothesis regarding the client-entry.ts .mount() call - if there is a race condition between this call (which causes hydration) and between urlDispatcher. If it's a case we'd be able to propose another, simpler solution to this issue. However, it's basically the same solution as lazy-hydrate (to postpone the hydration)  just maybe more elegant ;)

@adamjedlicka @Darwex we've found a solution to this problem which is not related with lazy hydration. Please give ma ~1h to clean up and final tests of a solution and I will provide another PR.
Btw. that's really great discussion here, and I'm grateful guys that you provided your concerns and point of view.
Expect another PR really soon 🚀

@adamjedlicka , @Darwex let's move to #3412, I'm closing this one without merge :)


Add error when ikey/connection string not present in TelemetryClient

Previous code throwing error while creating Config was causing issues with agent, preventing better logging for diagnostics.


Utilities to read Exec runtimes from manifest.

Add runtimes manifest to ansible group vars.
Write out manifest to whisk.properties and make available through WhiskConfig.
PG3/340.

PG3/350 🔵

merged


fix: custom component memory leaks

This PR fixes custom component usage by:

using markRaw instead of shallowRef so that the component will never be converted to a proxy
emitting close event from the custom component, then the Toast component receives and calls the deleteToast function.

Fixes

https://github.com/xiaoluoboding/vue-sonner/issues/18
https://github.com/xiaoluoboding/vue-sonner/issues/15

Any custom component then can just do $emit('closeToast') and the toast should close. Here's an example:
import { defineComponent, h, markRaw } from 'vue'
import { toast } from 'vue-sonner'

const Custom = defineComponent({
  setup(_props, { emit }) {
    return () =>
      h('button', {
        onClick: () => emit('closeToast')
      }, 'Click me to close')
  }
})

toast(markRaw(Custom))

LGT<

Hi @wobsoriano , just out of curiosity, what is the major issue that causes memory leaking here?
Is it related to shallowRef  ?

@cjboy76 For rendering a custom toast, yes. Unlike markRaw, shallowRef still proxies the component.

What is the answer because im STILL getting this error and im running the latest version 1.1.2
toast.custom(markRaw(JobItemWrapper), {
        componentProps: {
          JOB_ID: job.job_id,
        },
        duration: Infinity,
      });

The toast is showing but when i close it using toast.dismiss(0) it returns
Uncaught (in promise) Maximum recursive updates exceeded in component <Toaster>. This means you have a reactive effect that is mutating its own dependencies and thus recursively triggering itself. Possible sources include component template, render function, updated hook or watcher source function.

Please help me.  @wobsoriano @xiaoluoboding @cjboy76

@xiaoluoboding any update? its been couple months..


Fix Inactive Color Issues and Spacing Issues at Tabs and Breadcrumbs

Description

[x] Fix breadcrumbs spacing issue.
[x] Optimize the style of breadcrumb separator to better match the Xcode.
[x] Add inactive colors to Tabs and Breadcrumbs.

Related Issue

#520
#523

Checklist

[x] I read and understood the contributing guide as well as the code of conduct
[x] My changes generate no new warnings
[x] My code builds and runs on my machine
[x] I documented my code
[ ] Review requested

Screenshots

@lilingxi01 : Can you take a screenshot of when there is no file open?

@lilingxi01 : Can you take a screenshot of when there is no file open?

When no file is open neither the tab bar nor the breadcrumbs bar is visible. So this is completely unrelated.

Comparing the two, Xcode seems to be using blue folder icons. There is even less space between the end of the label and the > symbol. Compare the s in cypress with the s in Breadcrumbs in your screenshot. The text label is also a bit lighter when inactive. Looking good so far!

Can you update the screenshot?

@austincondiff : I'm still trying to fix the folder icon color issue. I currently cannot find any NSColor that is similar to the Xcode's one.

They're probably using the same color as the macOS folder icon.

Try Rgb 97 182 223.

Please change the inactive native tab background to 45% black instead of 50% black in dark mode.

Also I noticed Apple uses shadows instead of a hard border as well...

Sorry to be so nit-picky. These tabs are going to be so good, thank you for all you're doing on this!

I'd like to get your opinion. In finder, a tab activates on press. In Xcode, a tab activates on release. Do you think we should activate a tab on press only if displaying native tabs?

@austincondiff : I think active on release makes more sense and feels better.

Current style of tab bar and breadcrumbs:

Current style of Inactive States:

Shadow Compare

And, tab divider now goes under the shadow (and top/bottom divider):

This PR has already finished updating. Ready for next review! 🎉


jellyseerr: 1.7.0 -> 1.8.1

Description of changes
closes https://github.com/NixOS/nixpkgs/issues/305362
https://github.com/Fallenbagel/jellyseerr/releases/tag/v1.8.1
https://github.com/Fallenbagel/jellyseerr/compare/v1.7.0...v1.8.1

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

Result of nixpkgs-review pr 305441 run on x86_64-linux 1

1 package built:

jellyseerr


[GOBBLIN-1494] Fixes flaky test on HiveSource

Dear Gobblin maintainers,
Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below!
JIRA

[ ] My PR addresses the following Gobblin JIRA issues and references them in the PR title. For example, "[GOBBLIN-XXX] My Gobblin PR"

https://issues.apache.org/jira/browse/GOBBLIN-1494

Description

[x] Here are some details about my PR, including screenshots (if applicable):

HiveSourceTest returns 2 watermarks for testGetWorkunitsAfterWatermark instead of 1.
This is due to System.getCurrentTime.millis() returning a different value when comparing against the filesystem last updated filetime. To guard against this, change the test to incorporate the workunit within the watermark by assigning it an earlier creation time, thus implying that the file was modified so it needs to be ingested again.
Tests

[x] My PR adds the following unit tests OR does not need testing for this extremely good reason:

Commits

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

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

Codecov Report

Merging #3338 (30bb958) into master (08db23e) will decrease coverage by 3.52%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #3338      +/-   ##
============================================
- Coverage     46.55%   43.03%   -3.53%
+ Complexity    10138     1942    -8196
============================================
  Files          2051      394    -1657
  Lines         79547    16876   -62671
  Branches       8880     2075    -6805
============================================
- Hits          37036     7262   -29774
+ Misses        39086     8813   -30273
+ Partials       3425      801    -2624

Impacted Files
Coverage Δ

...lin/util/filesystem/FileSystemInstrumentation.java
92.85% <0.00%> (-7.15%)
:arrow_down:

.../org/apache/gobblin/cluster/GobblinTaskRunner.java
63.46% <0.00%> (-0.33%)
:arrow_down:

...runtime/mapreduce/GobblinWorkUnitsInputFormat.java

...bblin/converter/string/StringToBytesConverter.java

...writer/TarArchiveInputStreamDataWriterBuilder.java

...source/extractor/filebased/GZIPFileDownloader.java

.../apache/gobblin/records/RecordStreamProcessor.java

...ive/provider/HiveMetastoreBasedUpdateProvider.java

.../java/org/apache/gobblin/runtime/TaskExecutor.java

...mented/fork/InstrumentedForkOperatorDecorator.java

... and 1649 more

Continue to review full report at Codecov.

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


RPM Binary missing

Greetings.  Thanks so much for your continued maintenance of Tomboy.
I'm trying to install a new instance of Tomboy-ng in Fedora, but the link on the Download Release wiki page gives a 404 page not found error.  The file it supposedly points to is:
https://github.com/tomboy-notes/tomboy-ng/releases/download/v0.36c/tomboy-ng-0.36c-2.x86_64.rpm
Before I go down the bash script for deb->rpm or build from source roads, I thought I'd just ask for an rpm.
Thanks!

Thats messy gpshumway, very sorry. Looks like I fixed a packaging issue a couple of weeks ago and forgot to update the Fedora part of that download page. Very sorry, all fixed now.
You might like to consider the current pre-release. You would want https://github.com/tomboy-notes/tomboy-ng/releases/download/v0.36g/tomboy-ng-0.36h-1.x86_64.rpm
Still has a few things to solve wrt packaging but probably a lot better product than 0.36c right now.
Davo

Thank you very much!
Fast and friendly service from my local open source developer. :-)
-Bernie Nelson (aka Gordon Shumway)
On Tuesday, December 12th, 2023 at 5:11 PM, Davo  <> wrote:

[IronVest]
You can reply directly and still remain anonymous.
To protect your privacy, do not forward this message, or add new recipients like CCs or BCCs.
Is this email spam or just don’t want to see it anymore? [Disable @.***/toggle)
Thats messy gpshumway, very sorry. Looks like I fixed a packaging issue a couple of weeks ago and forgot to update the Fedora part of that download page. Very sorry, all fixed now.
You might like to consider the current pre-release. You would want https://github.com/tomboy-notes/tomboy-ng/releases/download/v0.36g/tomboy-ng-0.36h-1.x86_64.rpm
Still has a few things to solve wrt packaging but probably a lot better product than 0.36c right now.
Davo
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

We aim to please.  :-)
I'll close this now, its all done, too easy !

Thanks again for the prompt service, Dave.
I'm afraid I've got another problem.
I stupidly did my backup for my Fedora 39 upgrade by exporting my notes to HTML in Tomboy. I don't know why I did that, in the past I've been smart enough to copy the directory.
I found SupraLance's issue on the Tomboy-ng Github and tried to follow the workflow using aconvert.com, but I'm having issues.
I don't see a way either with Tomboy-ng or with TomboyTools to import an RTF file. How was it that you resolved his issue? Any better way since 2018?
Thanks again, I really appreciate it.
-Bernie
On Tuesday, December 12th, 2023 at 5:11 PM, Davo  <> wrote:

[IronVest]
You can reply directly and still remain anonymous.
To protect your privacy, do not forward this message, or add new recipients like CCs or BCCs.
Is this email spam or just don’t want to see it anymore? [Disable @.***/toggle)
Thats messy gpshumway, very sorry. Looks like I fixed a packaging issue a couple of weeks ago and forgot to update the Fedora part of that download page. Very sorry, all fixed now.
You might like to consider the current pre-release. You would want https://github.com/tomboy-notes/tomboy-ng/releases/download/v0.36g/tomboy-ng-0.36h-1.x86_64.rpm
Still has a few things to solve wrt packaging but probably a lot better product than 0.36c right now.
Davo
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

A discussion rather than an answer I am afraid.
OK, we sure do have a problem here I am afraid.  TomboyTools does not import HTML, it could be made to do so but with a fair bit of work. You see, I could not make a generic HTML importer, just one to handle tomboy-ng's exported notes. So, not widely useful. And time would be a problem, almost certainly not before Christmas.
So, some questions -
You were not using Sync in any form before your upgrade ?
When installing F39, you allowed it to format the /home partition ?
No other backups ?
I ask the latter because, by default, tomboy-ng makes regular auto archives, one of those might appear in your backups ?
Next, using SupraLance's online converter ?  I'd hate to see my notes go through a public service like that. But maybe it does do a reasonable job of converting to RTF ?  I don't really know, RTF as a standard is pretty diverse, again, the converted notes that aconvert make may contain rtf that we cannot import. But perhaps worth a try. I can pretty easily make a little add on that will turn a directory full of rtf files into notes.
Another approach might be a lot quicker, private and sort of OK.  Did you export your notes using tomboy-ng or Tomboy ?   SupraLance used Tomboy and that produced some pretty complicated CSS code. The html  that tomboy-ng makes is reasonably clean html. I could make you a filter that strips out that simple HTML and converts your notes to plain text. tomboy-ng will happily import plain text.
Obviously, you will loose all the markup but hopefully no content. I think ....
Thoughts ?
Davo

Thanks so much for your reply, Dave. To answer your questions:

I was not using any form of sync (I've never had any luck making it work)
It was a clean install of Fedora 39, so the whole drive was wiped.
I have some older backups, but some of my notes are continuously updated, so it would be messy.
I was using Tomboy, not Tomboy-ng. I'm afraid I just took what was in the Fedora repositories. Thusly, my HTML files are likely the CSS heavy versions produced by Tomboy.

I know what you mean about sending my notes through a public converter, but it's the best option I've seen so far. The old issue thread seemed to indicate SupraLance was successful. My tests indicate aconverter does a better job of formatting HTML files than LibreOffice, which is what I've got locally.
Honestly, the final format I'm looking for is markdown. I think I'm moving to a Nextcloud based solution for notes and that's Nextcloud Notes' format. Any ideas on converting html to markdown? A test export from Tomboy-ng seemed to work just great.
Thanks again for all your help, I really appreciate it.
-Bernie
On Wednesday, December 13th, 2023 at 4:40 PM, Davo  <> wrote:

[IronVest]
You can reply directly and still remain anonymous.
To protect your privacy, do not forward this message, or add new recipients like CCs or BCCs.
Is this email spam or just don’t want to see it anymore? [Disable @.***/toggle)
A discussion rather than an answer I am afraid.
OK, we sure do have a problem here I am afraid. TomboyTools does not import HTML, it could be made to do so but with a fair bit of work. You see, I could not make a generic HTML importer, just one to handle tomboy-ng's exported notes. So, not widely useful. And time would be a problem, almost certainly not before Christmas.
So, some questions -
You were not using Sync in any form before your upgrade ?
When installing F39, you allowed it to format the /home partition ?
No other backups ?
I ask the latter because, by default, tomboy-ng makes regular auto archives, one of those might appear in your backups ?
Next, using SupraLance's online converter ? I'd hate to see my notes go through a public service like that. But maybe it does do a reasonable job of converting to RTF ? I don't really know, RTF as a standard is pretty diverse, again, the converted notes that aconvert make may contain rtf that we cannot import. But perhaps worth a try. I can pretty easily make a little add on that will turn a directory full of rtf files into notes.
Another approach might be a lot quicker, private and sort of OK. Did you export your notes using tomboy-ng or Tomboy ? SupraLance used Tomboy and that produced some pretty complicated CSS code. The html that tomboy-ng makes is reasonably clean html. I could make you a filter that strips out that simple HTML and converts your notes to plain text. tomboy-ng will happily import plain text.
Obviously, you will loose all the markup but hopefully no content. I think ....
Thoughts ?
Davo
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

Hmm, thats all bad news I am afraid. I vaguely remember looking at Tomboy exported html and shaking my head sadly.  It does sound like using aconvert, get one complicated one turned into  RTF and send it to me and I will try to pull it into a KMemo. That will tell us your chances of making tomboy notes from them again. If it looks interesting, then I can make you a simple tool to bring them in as that way, one at a time (?) how many notes do you have ?
Important, make sure that one you send me is publicly viewable !
Once in tomboy-ng, you can export as MarkDown. One at a time or in bulk with tomboytools.
I was at one stage looking at syncing through NextCloud but they did not quite understand the risks of bad timekeeping with sync and I quickly saw the sort of problems we used to have with Tomboy Sync, especially with Rainy. So, I backed away, Still have a lot of code I prepared for it but would not ship it my worst enemy !
Davo

Update:
I used Firefox to save the Outback Suspension HTML note as plain text, then changed the extension to ".md" and imported it into NextCloud. The result is formatted acceptably. I use colons frequently and Nextcloud seems to think they're emojis, but I can live with that.
Now if only there were a way to batch it...
I found this-
https://github.com/abiyani/automate-save-page-as
A bit old and doesn't seem to have a parameter to save as plain text..
Keeping on...
-Bernie
On Thursday, December 14th, 2023 at 5:01 AM, Davo  <> wrote:

[IronVest]
You can reply directly and still remain anonymous.
To protect your privacy, do not forward this message, or add new recipients like CCs or BCCs.
Is this email spam or just don’t want to see it anymore? [Disable @.***/toggle)
Hmm, thats all bad news I am afraid. I vaguely remember looking at Tomboy exported html and shaking my head sadly. It does sound like using aconvert, get one complicated one turned into RTF and send it to me and I will try to pull it into a KMemo. That will tell us your chances of making tomboy notes from them again. If it looks interesting, then I can make you a simple tool to bring them in as that way, one at a time (?) how many notes do you have ?
Important, make sure that one you send me is publicly viewable !
Once in tomboy-ng, you can export as MarkDown. One at a time or in bulk with tomboytools.
I was at one stage looking at syncing through NextCloud but they did not quite understand the risks of bad timekeeping with sync and I quickly saw the sort of problems we used to have with Tomboy Sync, especially with Rainy. So, I backed away, Still have a lot of code I prepared for it but would not ship it my worst enemy !
Davo
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


Picture frontpage

@margotvermeylen & @biekemaex
Like your channel pic, but, I would for sure take a picture of crayfish on the home page. From start on it makes it clear the project is about crayfish...

What do you think of the picture now?

And the marmerkreeft_invasief.jpg picture? We don't want them to seem to cute on our website...

I'll put it up to see how it looks. I don't think a lot of people will find them particularly cute though :)

Not sure I'm a fan of how this picture turned out in the banner...

@biekemaex I think this is a general problem. None of the chosen pictures on the banner has good resolution...

Here's are a couple of front page banner suggestions. I suggest we vote on which we like best (compositionally, message). We can then brighten the image to offset the banner.
Option 1 (current)

Option 2

Option 3

Option 4

Option 5

Option 6

I think I like 6 the best, but 2 and 4 also look good.

Here's an adapted version of 6, that aligns better with the green banners used elsewhere:

Looks much better than the greyish image it was before!

Updated, see https://craywatch.inbo.be/


[Feat] 您好，想咨询一下AF和NEF功能大约什么时候发布？

Hello,
If everything goes smoothly, the NEF will be released shortly.

Hello,
We have uploaded the roadmap on the official website.
You can see the schedule for our upcoming features there: https://free5gc.org/.
Thanks for your support to free5GC!


License file missing

Please specify what are the rules of using this software. Commercial/non commertial use. possibility of modification.

I would also like to know this.

To all ChatDev authors - awesome work on this project and initiative!
I'd recommend getting a formal LICENSE.txt file in place at the root of the project to make it very clear how this is being licensed.  The snippet in the README.md isn't straight forward, and might not be picked up on by code scanners for those concerned with licensing conflicts.
@adamskrodzki & @2good4hisowngood - seems the ChatDev project licensing has changed quite drastically under this recent commit entitled "add git function":
https://github.com/OpenBMB/ChatDev/commit/75fbb9a5906ca5c5905e384312fbc80f180f4c2b


[$250] Android - Mark down - App crashes when deleting text with mark down in a sentence with heading

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.17-0
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Issue reported by: Applause - Internal Team
Action Performed:
Precondition:

Device keyboard must be Samsung keyboard
Issue is not reproducible with Gboard

Launch New Expensify app
Go to chat
Type # hey hey
Double tap to select and highlight the second word with mark down _ hey _
Tap delete on Samsung keyboard

Expected Result:
App will not crash
Actual Result:
App crashes
This issue is only reproducible with Samsung keyboard and not Gboard
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/user-attachments/assets/b0bc1b8b-a385-4021-a81a-38a074fb0cb6
logs (2).txt
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01bc8d3436a587add7
Upwork Job ID:<PHONE_NUMBER>702134137
Last Price Increase: 2024-08-22

Issue OwnerCurrent Issue Owner: @alitoshmatov

We think that this bug might be related to #Live Markdown

@isabelastisser  FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

No proposals yet.

Still waiting for proposals.

Waiting for proposals

No proposals yet.

No proposals yet

Still waiting for proposals.

no proposals yet.

Proposal Updated

Add result video

@wildan-m Thank you for your proposal, I think your RCA is correct and we can go with you solution
C+ reviewed 🎀 👀 🎀

We have chosen a proposal and are waiting for @neil-marcellini's review.

@wildan-m Thank you for your proposal, I think your RCA is correct and we can go with you solution
C+ reviewed 🎀 👀 🎀

Wow great job! Hiring

@alitoshmatov The upstream PR is ready. https://github.com/Expensify/react-native-live-markdown/pull/469

@alitoshmatov @wildan-m, can you please provide an update? Thanks!

@isabelastisser discussing optimal approach to avoid duplicate calls in upstream. https://github.com/Expensify/react-native-live-markdown/pull/469#issuecomment-2335002872

https://github.com/Expensify/react-native-live-markdown/pull/469 has just been merged

Thank you, @tomekzaw!
@alitoshmatov @neil-marcellini Should we simply update the version in the E/App to apply it? I see that the live markdown version in E/App is not the latest. Are there specific steps to follow?

There's already a PR that bumps react-native-live-markdown here: https://github.com/Expensify/App/pull/48626

@tomekzaw can we bump the version to 0.1.140 there or we should temporarily create patch until that PR closed?

Let's avoid patches and bump directly to latest if possible

Should be fixed with https://github.com/Expensify/App/pull/48626

Waiting for react-native-live-markdown bump on https://github.com/Expensify/App/pull/48626

Issue not reproducible during KI retests. (First week)

Not overdue, The fix PR passed the test and is now live in production https://github.com/Expensify/App/pull/48626#issuecomment-2353604788.

Not overdue.
FYI: I was only involved on selecting proposal, I was not involved in reviewing corresponding PRs

What's the next step here? What are we waiting for?

We are in a regression period now. Solution went into production on 16th september

@neil-marcellini @alitoshmatov, it looks like this is ready for payment. Do we need a regression test?

The payments were processed in Upwork. All set!


Update 2.0 sample TFM+packages

Updates sample to target only netcoreapp2.0.

@guardrex,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot


PS Move Controllers not showing in Steam VR

Only my Pimax 4k show up on steam vr without any controllers or cameras. I test all my camera there all work fine and all my controllers show up in the config tool.
steamvr.vrsettings
{
"psmove": {
"circle": "a",
"cross": "button_8",
"meters_per_touchpad_units": 0.075000002980232239,
"move": "touchpad",
"ps": "system",
"rumble_suppressed": false,
"select": "button_9",
"square": "grip",
"start": "button_10",
"triangle": "application_menu",
"trigger": "trigger",
"use_spatial_offset_after_touchpad_press_as_touchpad_axis": true
},
"steamvr": {
"activateMultipleDrivers": true,
"mirrorViewGeometry": "0 0 1209 746"
}
}
is anyone else having this problem and know how to fix it?
also i'm getting this error on PSMoveService log

Got it to work.

Howd you get it working man, no ones replied to my issue and from what you sent me we are in the same boat. What version of psmove you using? Do you know how to get the steamvr bridge working? Please im so tired and frustrated.


remove the interaction_name parameter from end_interaction method

Per the change in iBot 3.10.

@steveims Good point. I'll ping Adam about marking that as optional (or whatever the equivalent of deprecated is).


feat(board): Add Waveshare ESP32-S3-Matrix support

Description:
Add support for the Waveshare ESP32-S3-Matrix board.

This can be same for ESP32-S3-Zero

@kdpuvvadi Looks like it should. It has extra pins that won't be picked I by the Arduino "Dx", "Ax" notation.

Can you point me to right direction or can you do a pr for the same

@kdpuvvadi
Looks like support for the ESP32-S3-Zero is already in the Espresssif arduino-esp32 repository 👍.
So all you need to do is create the board support file and raise a PR to add that to the pioarduino repository.
Use the boards/ws_esp32_s3_matrix.json as a template and modify it for the Zero. You'll need to open the boards.txt file from the Espressif arduino-esp32 repository to ensure the names in the new  .json file you create line up. Use the boards.txt entry for the ESP32-S3-Matrix as a reference to compare to the  ws_esp32_s3_matrix.json file.
To test your .json file, copy it into the boards folder on your local machine. It should appear in the pio Board Explorer.


Remove dependency on czmq

The changes to dep and dockerfiles look good to me, but I’ll leave this up to @rickmak to accept the changes.


ERAF-5: Impl. proxing to communicate with BE server in dev and prod modes (self-origin scripts)

Research on proxying in react.js
Add needed dependencies and setups

Parcel set-up


Make See all (transaction) more noticeable

Not relevant for iOS, only for android (separate task https://github.com/BeamMW/android-wallet/issues/342)


Add pyproject.toml for Custom Node Registry

We are building a global registry for custom nodes (similar to PyPI). The registry makes using custom nodes more reliable by reserving globally unique names for each custom node, and supporting semantic versioning.
Here’s some more information on the registry.
Action Required:

[ ] Go to the registry. Login and create a publisher id. Comment it here and we will update the PR (or just merge it and change it yourself).
[ ] Write a short the description.
[ ] Merge the separate Github Actions PR and run the workflow.
If you want to publish the node manually, install the cli and run comfy node publish

Please message me on Discord if you have any questions!

hey @MitoshiroPJ, wanted to send a small nudge and ask if you had any questions!

@MitoshiroPJ, wondering if there's any way we can support you on this PR!


Custom colors added by a plugin removed

Hi, I'm switched my Tailwind CSS project to windicss. But unfortunately my custom theming plugin didn't work as expected.
Reproduction
Repo: https://github.com/sibbng/sibbng.com
git clone https://github.com/sibbng/sibbng.com.git
cd sibbng.com
yarn
yarn dev

How my custom theming plugin work?
First of all, themes defined in https://github.com/sibbng/sibbng.com/blob/main/theme.yml
My theming plugin is placed on https://github.com/sibbng/sibbng.com/blob/main/themewind.js
This plugin reads yaml file and adds something like these as utility classes:
[data-theme="dark"], [data-theme="light"] {
  --active: #0099FF;
}

[data-theme="dark"] {
  --primary: #111111;
  --on-primary: #B4B4B4;
  --on-primary-active: #F2F2F2;
  --frame: #1E1E1E;
  --on-frame: #808080;
}
[data-theme="light"] {
  --primary: #FFFFFF;
  --on-primary: #888888;
  --on-primary-active: #333333;
  --frame: #EFEFEF;
  --on-frame: #808080;
}

Also extending default theme with these semantic colors:
{
  active: 'var(--active)',
  primary: 'var(--primary)',
  'on-primary': 'var(--on-primary)',
  'on-primary-active': 'var(--on-primary-active)',
  frame: 'var(--frame)',
  'on-frame': 'var(--on-frame)',
  'on-frame-active': 'var(--on-frame-active)'
}

In my template I'm using them like class="bg-primary text-on-primary". You can see production deployment of tailwind version at https://sibbng.netlify.app/
That plugin was working in Tailwind. But didn't work in windicss.

windicss/windicss@babe597 You can see the generated css is correct. So I will transfer this issue to vite-plugin.

If I am not wrong your tests adds that color utilities to directly to config's itself. Not passed to windicss through a plugin.

.bg-on-primary {
    --tw-bg-opacity: 1;
    background-color: rgba(var(--on-primary), var(--tw-bg-opacity));
  }

Also, this code is wrong. Because windicss doesn't transform my [data-theme="dark"] variables to rgb equivalent and shouldn't. Because Tailwind itself is doesn't. Look my website for example https://sibbng.netlify.app/


Update wordpress deployment example

This was missing configuration for the required database server, which meant that whoever wanted to use this as an example of how to get a "more complex" application running on Akash would be presented with database errors.


Enable a "next item" button

The user should be allowed to easily browse to a next random Item, otherwise the workflow gets stuck.
Related to #57

Closing, and leaving https://github.com/google/primarysources/issues/57 open instead.


Add LOINC long names to HIV devices in test and stg

After we sent the test HIV message to ReportStream's lower environment, they notified us that the message was missing the LOINC long names. The LOINC long names need to be manually updated for the two HIV devices in test and stg. We have already written the code to include them in the FHIR message if they are populated.
The two devices are:

OraQuick In-Home HIV Test
INSTI HIV-1/HIV-2 Rapid Antibody Test

Both test performed LOINC long name and test ordered LOINC long name need to be added manually, probably by running a mutation against the environments, since we don't support modifying these fields in our device page.
After updating the fields, submit an HIV result in test and notify RS, including the test event UUID in the message.

@DanielSass
OraQuick In-Home HIV Test

LOINC: 33866-5, https://loinc.org/33866-5
Long Common Name: HIV 1 Ab [Presence] in Capillary blood by Immunoassay
This device is not included in the HIV LIVD table, but is specifically called out on the LOINC website: "This is the POC test, such as OraQuick Rapid HIV-1 Antibody test - Source: Regenstrief LOINC"

INSTI HIV-1/HIV-2 Rapid Antibody Test

LOINC: 80387-4, https://loinc.org/80387-4
Long Common Name: HIV 1+2 Ab [Presence] in Serum, Plasma or Blood by Rapid immunoassay
This device is included in the HIV LIVD table

@bobbywells52
OraQuick In-Home HIV Test

Manufacturer: OraSure Technologies [OraSure]
Model type: "...rapid self-administered over-the-counter (OTC) test." [FDA]

INSTI HIV-1/HIV-2 Rapid Antibody Test

Manufacturer: bioLytical Laboratories Inc. [bioLytical]
Model type: "Single-use rapid assay" [FDA]

Long names have been updated in both staging and testing. For SR to review they can look at the two following tests conducted with the following IDs in the RS org in the test env:

752cc59e-89c5-4485-aab3-636bbb24a812
ddea8f9c-2838-49bd-8946-3e2f95e39639

Checked back in with ReportStream and they confirmed that the new messages do have the LOINC long name in the correct place so we are all good! Closing this ticket now, thanks @bobbywells52


pkg/kwok/controllers: should not add duplicate job

What type of PR is this?
/kind bug
What this PR does / why we need it:
When kube-apiserver restarts, the watch might return ADD event which we already received.
We should not add the cronjob for the same node. Otherwise, the number of lease updates will be increased 2-3 times.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?

Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.:

/ok-to-test

/retest

/approve
/lgtm
/label tide/merge-method-squash
/release-note-none


Base Url and Product Url showing error

Always base Url and product Url showing error when we are trying with this library
GPHOTOS

POSTMAN RESULT
When doing same request with postman giving correct output

the results seem to be there though, Xcode seems not to be able to show it. can you try to print out the baseString values?


requests.exceptions.MissingSchema Error

When scanning www.nytimes.org (spider level 2), after discovery of http://takingnote.blogs.nytimes.com/feed, the script errors out with a missing schema error. It appears the script is making a request for a url without including the schema (http/https).

Traceback (most recent call last):
File "./scan_feeds.py", line 7, in
for url in generate_feed_urls('http://www.nytimes.com',spider=2):
File "/usr/local/lib/python3.6/dist-packages/feed_seeker/feed_seeker.py", line 420, in generate_feed_urls
for feed in FeedSeeker(url, html).generate_feed_urls(spider=spider, max_links=max_links):
File "/usr/local/lib/python3.6/dist-packages/feed_seeker/feed_seeker.py", line 200, in generate_feed_urls
for url, _ in self._generate_feed_urls(spider=spider, max_links=max_links):
File "/usr/local/lib/python3.6/dist-packages/feed_seeker/feed_seeker.py", line 252, in _generate_feed_urls
for url, seen in spider_seeker._generate_feed_urls(**kwargs):
File "/usr/local/lib/python3.6/dist-packages/feed_seeker/feed_seeker.py", line 252, in _generate_feed_urls
for url, seen in spider_seeker._generate_feed_urls(**kwargs):
File "/usr/local/lib/python3.6/dist-packages/feed_seeker/feed_seeker.py", line 227, in _generate_feed_urls
if not self.html:
File "/usr/local/lib/python3.6/dist-packages/feed_seeker/feed_seeker.py", line 152, in html
self._html = self.fetcher(self.url)
File "/usr/local/lib/python3.6/dist-packages/feed_seeker/feed_seeker.py", line 104, in default_fetch_function
response = session.get(url)
File "/usr/local/lib/python3.6/dist-packages/requests/sessions.py", line 546, in get
return self.request('GET', url, **kwargs)
File "/usr/local/lib/python3.6/dist-packages/requests/sessions.py", line 519, in request
prep = self.prepare_request(req)
File "/usr/local/lib/python3.6/dist-packages/requests/sessions.py", line 462, in prepare_request
hooks=merge_hooks(request.hooks, self.hooks),
File "/usr/local/lib/python3.6/dist-packages/requests/models.py", line 313, in prepare
self.prepare_url(url, params)
File "/usr/local/lib/python3.6/dist-packages/requests/models.py", line 387, in prepare_url
raise MissingSchema(error)
requests.exceptions.MissingSchema: Invalid URL '//www.nytimes.com/2017/01/10/watching/faq.html': No schema supplied. Perhaps you meant http:////www.nytimes.com/2017/01/10/watching/faq.html

Fix: https://github.com/mitmedialab/feed_seeker/commit/989ae5a55ba8d7f196c24fecfa1b80833b110395

https://github.com/mitmedialab/feed_seeker/commit/989ae5a55ba8d7f196c24fecfa1b80833b110395


add  roles, roles_mapping ,indices on kibana can't synchronize to configuration file of opendistro_security

I found that these new configurations could not be synchronized to the relevant configuration files on  opendistro_security,When I add a roles,role_mapping,internal user to the kibana page.and when
i upload the changes via sgadmin  ,configuration files is gone on kibana page . I wonder why settings isn't synchronous.

install of rpm

Hi @HLLavender, if I'm understanding correctly, this is intentional. The configuration files only store initial configuration. Most configuration is stored in the .opendistro_security index in the cluster itself so that it can be shared between nodes. You can dump the index to a set of configuration files if you want to migrate them to a new cluster.


Removed export from template

Hello,
I removed the export from the template because it breaks Ubuntu cron files. cron will deactivate the file because its invalid. Also a export is not needed to set a environment variable in a cron file :)

Heh, export was added since restic didn't work on some platform. It was added in #31. I think this will need testing on non-debian platforms first to settle it once and for all :)

I need to remove the exports too for it to work on Ubuntu 16.04. I think cron may be executed with dash instead of bash there, and export X=y directly is a bashism. However, I didn't need the exports at all for the cron job to work. It's also noticeable in #31 that the original committer apparently reverted this export adding in one of their own branches. So, from my point of view, it doesn't seem necessary to have those exports at all but they break things.


add MCC implementation, with LSAP matching

sklearn-based implementation of Matthew's correlation coefficient (MCC)
Since this is a classification metric requiring matched labels, I followed the example of https://github.com/SpatialHackathon/SpaceHack2023/pull/107 and first calculated the best match using the sklearn linear_sum_assignment (https://docs.scipy.org/doc/scipy/reference/generated/scipy.optimize.linear_sum_assignment.html).
I also added a flag --matched_labels that could be passed to skip the matching if we want to simply precalculate the matching somewhere in the pipeline and pass it to all metric modules (this would probably be a good idea to be positive all methods are using the same matching).

We might need to discuss if and how we introduce this new command line arg

The arg is currently optional, and if not provided the default is to assume labels are not matched and perform the matching, so I don't see this getting in the way of integration.


Question 2 function expression

For the function expression in Q2
var myFunc1 = new function() {}
var myFunc2 = new function whatever() {} // whatever is not a variable here

I think it would be better to use something like the code below to distinguish the 'function' and the 'Function' object
var myFunc1 = function() {}
var myFunc2 = function whatever() {}
var myFunc3 = new Function ('a', 'b', 'return a+b')

Sorry, I don't understand why using new function(...) {...} rather than new Function()

the whole question will be replaced

done


Question: Cross-process connection pool

Hello :)
Per my understanding, the thread pool and semaphore are within the bound of one process, is that correct? This means that, for one service, as it could be deployed on multiple machines, there isn't a global connection pool that limits the concurrent connection it could make to another service, right? Hystrix is about client side throttling and resillience, what about the server side?
Thanks in advance.

Your intuition is correct.  Hystrix is client-side only and single-instance only.  Other tools should be used for server-side resilience.

Closing due to inactivity, please re-open if there's more to discuss


Dask-related test failures in pull request

After creating a pull request unrelated to Dask, two tests involving Dask failed. See this log for details: https://github.com/i4Ds/Karabo-Pipeline/actions/runs/9172253662/job/25218346513?pr=569
To avoid the test failures, I disabled Dask in the two affected tests, see comments with "TODO DASK_TEST_ISSUE" prefix in the code base.
This should be looked at in detail though to make sure Dask is working as expected.

Something seems to be off indeed with Dask.
In source_detection.ipynb, running the following code twice:
restored = RascilImageCleaner(     RascilImageCleanerConfig(         imaging_npixel=imaging_npixel,         imaging_cellsize=imaging_cellsize,         ingest_vis_nchan=16,         clean_nmajor=1,         clean_algorithm="mmclean",         clean_scales=[10, 30, 60],         clean_threshold=0.12e-3,         clean_nmoment=5,         clean_psf_support=640,         clean_restored_output="integrated",         use_dask=True,     ) ).create_cleaned_image(     ms_file_path=visibility_askap.ms_file_path, )
results in a "RuntimeError: IOLoop is closed" as well (first run is fine, second one crashes).


Refactoring and linting for error handling

[ ] Use errors.As and errors.Is working with errors
[x] Wrap errors with %w (fmt.Errorf("oops: %w", err))
[ ] Enable https://github.com/polyfloyd/go-errorlint
[ ] Enable https://github.com/tomarrell/wrapcheck (optional)

not relevant


Update README.md

added myname(khalid khan) to the name list
Related issue
What changes does this PR do?
it will add my name to the name list
Screenshots (If any)
Questions
is it a good practice to make changes in github(ofcourse by creating a new branch ) directly or we should make the changes locally and then push it to github

Hello, @khalid569 Thank you for the changes, but it appears that there is another person with the same name. It would be fantastic if you could make any changes to your name to make it less confusing.


Log if class receiver is null [phab:D1341]

Created by salguarnieri at 2017-02-15 17:23:28:
While indexing some code, we were getting assertions errors that QT was
null. Add logging to figure out why this happens. The documentation at
https://clang.llvm.org/doxygen/classclang_1_1ObjCMessageExpr.html#a6457400eb37b74bf08efc178d6474e07
says that null is returned if the expression is not a class message, but
it doesn't promise that null is never returned if the expression is a
class message.

zarko wrote at 2017-02-15 17:34:31:
I've found that anywhere a type can have a null value will sometimes have a null value in Clang, even if the documentation suggests something to the contrary. It's also important to note that the documentation up on llvm.org is possibly for a different version of Clang than the one we're using.

zarko accepted this revision at 2017-02-15 17:34:31

salguarnieri closed via commit 2db1c9b7498eba2c1660baabac99415fcb4f88ef at 2017-02-15 17:38:54


Emit compile error when no runtime feature specified

Inspired by an error I saw when using https://github.com/launchbadge/sqlx. Hopefully gives a bit quicker feedback about how to solve an otherwise giant compilation error when forgetting to specify a runtime feature, though could still be improved as mentioned in the comment.

Lovely, TY! I have a PR to fix a longstanding bug https://github.com/arlyon/async-stripe/issues/246 which I will request your feedback on if you'd like. I know I said we should sync real quick and talk about plans (since you've decided to stick around! :) ) so if you want to you can book a slot here
https://usemotion.com/meet/alexander-lyon/meeting
Merging!

Lovely, TY! I have a PR to fix a longstanding bug #246 which I will request your feedback on if you'd like. I know I said we should sync real quick and talk about plans (since you've decided to stick around! :) ) so if you want to you can book a slot here

Thanks, will book one! (Though please don't have too high hopes time I can spend on this tends to be inconsistent :))


[opencl] silx.opencl tests are not executed, and they fail

It looks like we forgot to run some of the tests. The  __init__.py in the main test package should probably include:
from ..opencl import test as test_ocl
test_suite.addTest(test_ocl.suite())

But the tests in silx.opencl.test seem to currently fail.
knobel@linsimak:~/git/silx$ python run_tests.py -i silx.opencl.test.suite
WARNING:run_tests:Building silx to /users/knobel/git/silx/build/lib.linux-x86_64-2.7
running build
running build_py
running build_ext
WARNING:run_tests:Patched sys.path, added: '/users/knobel/git/silx/build/lib.linux-x86_64-2.7'
WARNING:run_tests:Test silx 0.4.0-dev0 from /users/knobel/git/silx/build/lib.linux-x86_64-2.7/silx
beignet-opencl-icd: no supported GPU found, this is probably the wrong opencl-icd package for this hardware
(If you have multiple ICDs installed and OpenCL works, you can ignore this message)
Error clEnqueueNDRangeKernel failed: invalid work item size on WG=2048
.EMaximum valid workgroup size 1024 on device <pyopencl.Device 'Intel(R) Xeon(R) CPU E3-1245 v5 @ 3.50GHz' on 'AMD Accelerated Parallel Processing' at 0x1299410>

======================================================================
ERROR: test_measurement (silx.opencl.test.test_addition.TestAddition)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/users/knobel/git/silx/build/lib.linux-x86_64-2.7/silx/opencl/test/test_addition.py", line 116, in test_measurement
    for pid, platform in ocl.platforms:
TypeError: 'Platform' object is not iterable

----------------------------------------------------------------------
Ran 2 tests in 0.010s

FAILED (errors=1)
WARNING:run_tests:Test suite failed

The source code of test_measurement is not up to date.
It seems like the source code should looks like something close to :
    @unittest.skipUnless(ocl, "pyopencl is missing")     def test_measurement(self):         """         tests that all devices are working properly ...         """         for platform in ocl.platforms:             for did, device in enumerate(platform.devices):                 print(device)                 meas = measure_workgroup_size(device)                 self.assertEqual(meas, device.max_work_group_size,                                  "Workgroup size for %s/%s: %s == %s" % (platform, device, meas, device.max_work_group_size))
But then an error occur in sopencl.init._measure_workgroup_size because the given device is not a pyopencl.Device but a silx.pyopencl.Device maybe we can just cast it but I am not sure the test_measurement function is still usefull ?


Add wiki documentation about index.yaml

Having the correct indices in the index.yaml file is a prerequisite for deployment, but the update flow for this file is unlikely to be clear to new contributors. It's detailed here: https://cloud.google.com/appengine/docs/flexible/configuring-datastore-indexes-with-index-yaml?tab=python
This can result in contributors getting edits in their index.yaml file that they didn't put there, and not knowing what to do with them. We don't seem to mention this gotcha at all in the wiki. Perhaps we should add an index.yaml page in the developer reference in the sidebar so that if a contributor runs into this issue they have some idea of what to do. (See e.g. https://github.com/oppia/oppia/pull/17742 for an example of this happening in practice.)

Please assign this issue to me @seanlip @U8NWXD @jahd2602 .

@KunjMaheshwari  Per the guidance at https://github.com/oppia/oppia/wiki/Contributing-code-to-Oppia#choosing-a-good-first-issue, please provide an explanation of what your PR will do (with names of files you're changing, what you plan to change in each file, etc.). If it looks good, we can assign you to this issue.
Please also follow the other instructions on that wiki page if you have not yet done so. Thanks!

Hey @seanlip I want to work on this issue. So, I created a new index.yaml file in the codebase in workflows section however I am not sure where do we have to create this file? Can you plz tell. I did the following additions. Have a look. If these are fine then kindly assign me this issue. I will create a PR today itself

Hi @vaishnavi192, I think you might be misinterpreting the issue. The index.yaml file is a file that's used by Google App Engine to manage datastore indexes. The aim of this issue is to create a wiki page explaining to developers how to use it.
If you are not familiar with index.yaml in Google App Engine then I would suggest not taking up this issue. Thanks!

Index.yaml-Explanation.md
Here is my first attempt at documentation for index.yaml. Let me know if there is anything else I need to add. My plan for PR is to add this file to the oppia-web-developer-docs repository. Let me know if any further steps are needed.


[ ] I have updated the version appropriately in the ==UserStyle== header of the catppuccin.user.css file.

@GenShibe why draft it?

@GenShibe why did you draft it? Also version number should be increased.

not finished with the fixes that's why, bunch more fixes to be done

not finished with the fixes that's why, bunch more fixes to be done

@GenShibe alr


GradleDaemon deamon hangs unrecoverably if it receives console output too fast

Expected Behavior
Gradle completes build.
Current Behavior
GradleDaemon hangs while running a JUnit test. Gradle would consume lots of CPU, but then go idle after some time and become unresponsive. Attempts to connect with VisualVM, jmap, jstack or gradle --stop fail. Sometimes it would print "Expiring Daemon because JVM heap space is exhausted" before becoming unresponsive, but sometimes not.
Note: The culprit for the memory problem is ThrottlingOutputEventListener. It stores output bound only by time. For unfortunate combinations of log amount and available memory, it will exhaust the whole Java heap.
Context
We run GradleDaemon with restricted memory, to get a reproducible CI environment:
./gradlew -Dorg.gradle.jvmargs="-Xmx256m -Xms256m"

Usually the GradleDaemon is lean and doesn't consume too much memory. 256 MB is enough all of the times. It was when we added a test, which would print tons of log lines, when GradleDaemon started to exit with OutOfMemoryError.
I tried to make it reproducible, but my first attempt led to GradleDaemon hanging instead.
Steps to Reproduce
test-project.zip
Run ./test.sh in the attached project. It runs the only available test in the project, which print tons of lines.
Your Environment
Arch Linux
Java 14
Gradle 6.7.1

Still occurs with Gradle 7.4.2. I updated the test project.
test-project2.zip

The issue is in the backlog of the relevant team and is prioritized by them.


styling

[ ]  apply dark theme
[ ]  apply material styling according to the design https://www.figma.com/file/OsQOZmNc1VFNzj5iQ7uTqC/Translations?node-id=0%3A1&t=uUUFJPPRg30XlV5N-0

closed by #13


object 'ggplot.data.frame' not found

I don't know if this is an issue with ggcyto or R versions or updates.
ggcyto was working perfectly, then I updated R and now I get the same error for each attempt to plot.

version
_
platform       x86_64-w64-mingw32
arch           x86_64
os             mingw32
system         x86_64, mingw32
status
major          3
minor          5.0
year           2018
month          04
day            23
svn rev        74626
language       R
version.string R version 3.5.0 (2018-04-23)
nickname       Joy in Playing

library(ggcyto)
library(flowCore)
#load sample flowset
fs <- read.flowSet(path=system.file("extdata", "compdata", "data",

package="flowCore"), name.keyword="SAMPLE ID",
phenoData=list(name="SAMPLE ID", Filename="$FIL"))

autoplot(fs,"FSC-H")

Error in get(name, envir = asNamespace(pkg), inherits = FALSE) :
object 'ggplot.data.frame' not found
Googling hasn't produced anything comprehensible about what this might mean - is there a way to fix it?
Thank you

For additional information
ggplot(fs, aes("FSC-H"))+geom_density()
produces an appropriate looking curve

ggcyto(fs, aes(FSC-H))+geom_density()
Error in get(name, envir = asNamespace(pkg), inherits = FALSE) :
object 'ggplot.data.frame' not found

and ggplot(fs, aes("FSC-H", "SSC-H"))+geom_hex() produces a solid blue screen

I get the exact same issue as of today.

autoplot(gs_comp_trans[[6]])
Error in get(name, envir = asNamespace(pkg), inherits = FALSE) :
object 'ggplot.data.frame' not found

NB: autoplot was working perfectly fine until yesterday.

version
_
platform       x86_64-apple-darwin15.6.0
arch           x86_64
os             darwin15.6.0
system         x86_64, darwin15.6.0
status
major          3
minor          5.0
year           2018
month          04
day            23
svn rev        74626
language       R
version.string R version 3.5.0 (2018-04-23)
nickname       Joy in Playing

Thank you in advance for your help,
Damien

Last commit was 9 days ago. So this is likely because you updated ggplot2?

Thank you Greg.
First re-installed ggplot2 v2.2.1 which failed again but v2.2.0 works fine so far. :-)
I would like to take profit to also thank you for developing these great tools.
Damien

As @gfinak said, it was due to the api changes from the latest ggplot2. I've just pushed the fix and please pull the latest ggcyto from trunk branch and that should work with the latest ggplot2 3.0

Hi,
Sorry to bother you again but I am quite new in R and I am also experiencing this issue. Could you be more precise as to how to pull the latest ggcyto from trunk branch to fix this issue or redirect me to the relevant literature?
Thanks a lot for your help!
Rosa.

I was able to do it using DevTools, the important part is the ref=trunk to make sure that's what you get
#install.packages("devtools")
#devtools::install_github("RGLab/ggcyto", ref='trunk')

@efinlay, now works for me, thanks!

This thread just saved me 2 more hours of googling !
I was also running under the issue of
"Error in get(name, envir = asNamespace(pkg), inherits = FALSE) :
object 'ggplot.data.frame' not found" even though it had already worked great in the past.
@efinlay your solution fixed it so THANKS !

is this slated for "release" to BioC soon?  Or, already done?  Inquiring minds<EMAIL_ADDRESS>are seeking release stability....

It's in bioc release already.


Fixed null deprecation in Zend/Pdf/Resource/Font/Simple

I tried to fix the iconv() null deprecation on parameter #3 in the sources that use the PDF lib, but that proved to be very difficult to pin. Even numerous fixes in drawText() (maybe it's not from it) the errors keep popping up.

I think we can replace
        if ($string === null || $string === '') {             return 0;         }
with
        if ( empty($string) ) {             return 0;         }
..to make things more readable :)

I think we can replace

No. Empty is bad.
$string = '0';
var_dump(($string === null || $string === '') === (empty($string)));


Check if a job is locked

Great gem!
Is there any way to check (without enqueueing) if a job with some specific arguments is already locked?

Hey @dup2
Currently, the gem does not provide this functionality. But every job instance has the #lock_key method exposed, so you can just check if Redis has corresponding key stored.
Redis.current.exists(MyJob.new(locked_args).lock_key)
=> true

Redis.current.exists(MyJob.new(unlocked_args).lock_key)
=> false

Great, thanks for the quick reply!


Ensure Bazel gpg public key is updated

This change is

I'm about to be out for the day.  Maybe @svenevs tomorrow morning?

The nightly build is passing again without this change. I'm going to close this, we can open back up if needed.


Crash

Crashes when entering something like
google.com,
or
youtube.co,

Thanks for bringing this to my attention. It seems to happen whenever you use any invalid punctuation after 'x.y'. The current Regex pattern believes it is a valid URL when it shouldn't be.


Add portal/uploads to .gitignore

------ ENHANCEMENT -------
Current Version
<IP_ADDRESS>
What is the problem you want to solve?
portal/uploads, in current version, contains 1GB+ of csv files that are getting sync'd to the repo.
The storage limit of github (per file) is 100MB and these are all more than that each.
What is your take on the correct solution to the problem?
Add this directory to the .gitignore so it is not sync'd to the repo - will be much smaller and we won't have customer data on the web.

Added uploads directory to gitignore so those files would no longer attempt to sync with the github repository.  See 2457370f3af17faa950b5d722c7b1d71437b8a34 for diffs