High number of rexi_DOWN messages

Description
I have multiple clusters with q=1 and n=3, with different data but same configuration. On one cluster when I query _node/_local/_stats I get a very high value in the parameter number of rexi_DOWN messages handled. The other clusters have 0 or a very low value for the same parameter.
On the surface, the affected cluster does not appear to have any functionality issues. However given the critical nature of the data it stores, I want to be sure that these errors are not a symptom of some deeper problem which might compromise the server and the data.
Extract from _node/_local/_stats:
{
  "rexi": {
    "buffered": {
      "value": 1,
      "type": "counter",
      "desc": "number of rexi messages buffered"
    },
    "down": {
      "value": 425565,
      "type": "counter",
      "desc": "number of rexi_DOWN messages handled"
    },
    "dropped": {
      "value": 0,
      "type": "counter",
      "desc": "number of rexi messages dropped from buffers"
    }
  }
}

In the logs I noticed some cluster unstable messages which may or may not be related to the issue.
[notice] 2019-08-21T23:19:10.858745Z<EMAIL_ADDRESS><0.275.0> -------- rexi_server : started servers
[notice] 2019-08-21T23:19:10.867696Z<EMAIL_ADDRESS><0.279.0> -------- rexi_buffer : started servers
[info] 2019-08-21T23:19:10.868037Z<EMAIL_ADDRESS><0.9.0> -------- Application rexi started on node<EMAIL_ADDRESS>[notice] 2019-08-21T23:19:10.988247Z<EMAIL_ADDRESS><0.276.0> -------- rexi_server_mon : cluster unstable
[notice] 2019-08-21T23:19:10.988294Z<EMAIL_ADDRESS><0.280.0> -------- rexi_server_mon : cluster unstable
[notice] 2019-08-21T23:19:10.988535Z<EMAIL_ADDRESS><0.275.0> -------- rexi_server : cluster unstable
[notice] 2019-08-21T23:19:10.988615Z<EMAIL_ADDRESS><0.279.0> -------- rexi_buffer : cluster unstable
[notice] 2019-08-21T23:19:11.035312Z<EMAIL_ADDRESS><0.276.0> -------- rexi_server_mon : cluster unstable
[notice] 2019-08-21T23:19:11.035349Z<EMAIL_ADDRESS><0.280.0> -------- rexi_server_mon : cluster unstable
[notice] 2019-08-21T23:19:11.035465Z<EMAIL_ADDRESS><0.275.0> -------- rexi_server : cluster unstable
[notice] 2019-08-21T23:19:11.035503Z<EMAIL_ADDRESS><0.279.0> -------- rexi_buffer : cluster unstable

Also, in the logs I have some errors related to rexi_server:
[error] 2019-08-26T10:12:36.704836Z<EMAIL_ADDRESS><0.27053.209> -------- rexi_server: from: couchdb@couchdb-eu-3.d4h.live(<14388.27013.1517>) mfa: fabric_rpc:all_docs/3 exit:timeout [{rexi,init_stream,1,[{file,"src/rexi.erl"},{line,265}]},{rexi,stream2,3,[{file,"src/rexi.erl"},{line,205}]},{fabric_rpc,view_cb,2,[{file,"src/fabric_rpc.erl"},{line,462}]},{couch_mrview,map_fold,3,[{file,"src/couch_mrview.erl"},{line,526}]},{couch_bt_engine,include_reductions,4,[{file,"src/couch_bt_engine.erl"},{line,1074}]},{couch_bt_engine,skip_deleted,4,[{file,"src/couch_bt_engine.erl"},{line,1069}]},{couch_btree,stream_kv_node2,8,[{file,"src/couch_btree.erl"},{line,848}]},{couch_btree,stream_kp_node,8,[{file,"src/couch_btree.erl"},{line,819}]}]
[error] 2019-08-26T10:12:36.704897Z<EMAIL_ADDRESS><0.3634.210> -------- rexi_server: from: couchdb@couchdb-eu-1.d4h.live(<14387.921.191>) mfa: fabric_rpc:all_docs/3 exit:timeout [{rexi,init_stream,1,[{file,"src/rexi.erl"},{line,265}]},{rexi,stream2,3,[{file,"src/rexi.erl"},{line,205}]},{fabric_rpc,view_cb,2,[{file,"src/fabric_rpc.erl"},{line,462}]},{couch_mrview,map_fold,3,[{file,"src/couch_mrview.erl"},{line,526}]},{couch_bt_engine,include_reductions,4,[{file,"src/couch_bt_engine.erl"},{line,1074}]},{couch_bt_engine,skip_deleted,4,[{file,"src/couch_bt_engine.erl"},{line,1069}]},{couch_btree,stream_kv_node2,8,[{file,"src/couch_btree.erl"},{line,848}]},{couch_btree,stream_kp_node,8,[{file,"src/couch_btree.erl"},{line,819}]}]

Your Environment

CouchDB Version used: 2.3.0
Operating System and version: Ubuntu 18.04

Hi @mariocasciaro, I don't know that this is an immediate cause for concern, but when you see these rexi_DOWN messages it means the node could not contact some peer in the cluster over a distributed Erlang link. That could be because you restarted CouchDB on the other node, it could be that the network had some temporary issue, etc.
You would expect to see spikes of that counter whenever one of those types of issues occurs, so it might be a good idea to feed that metric into grafana or a similar tool. You should definitely not see a steady drumbeat of those messages -- that would be an indication to possibly check the /_membership endpoint and see if you've somehow got an extra node listed in the cluster_nodes field that you didn't intend to list.
I'm going to go ahead and close this issue for now but feel free to reopen if needed.

@kocolosk Thanks for the hint. The number of rexi_DOWN messages is indeed growing at a steady rate, 1-3 every 10 seconds. This applies to all 3 nodes on the cluster. I double checked the connectivity of the nodes and it all seems configured and working properly. The nodes can talk to each other and _membership also configured properly. I tried to enable debug logging, but there's nothing relevant happening while those rexi_DOWN number goes up.
Is there any way I can dig further into the matter, like knowing what node is seen as problematic by the cluster?

i have same problem.
[error] 2019-10-17T11:08:25.163774Z couchdb@localhost <0.23546.3014> -------- rexi_server: from: couchdb@localhost(<0.22855.3023>) mfa: fabric_rpc:all_docs/3 exit:timeout [{rexi,init_stream,1,[{file,"src/rexi.erl"},{line,265}]},{rexi,stream2,3,[{file,"src/rexi.erl"},{line,205}]},{fabric_rpc,view_cb,2,[{file,"src/fabric_rpc.erl"},{line,462}]},{couch_mrview,finish_fold,2,[{file,"src/couch_mrview.erl"},{line,682}]},{rexi_server,init_p,3,[{file,"src/rexi_server.erl"},{line,140}]}]
Im runnning couchdb replication from another server. Errors are on target. Target is cluser with 1 node so nothing to communicate too.... Why is this failing?


model.schema() crashes on models with with type Type[...]

Checks

[x] I added a descriptive title to this issue
[x] I have searched (google, github) for similar issues and couldn't find anything
[x] I have read and followed the docs and still think this is a bug

Bug
Any model that includes a Type or Type[...] typed field, crashes on the schema() method.
This reproduction is based on the example in the docs, but with the added call to schema():
from typing import Type
from pydantic import BaseModel

class LenientSimpleModel(BaseModel):
    any_class_goes: Type

my_model = LenientSimpleModel(any_class_goes=int)
my_model.schema()

Traceback:
.../site-packages/pydantic/main.cpython-38-darwin.so in pydantic.main.BaseModel.schema()
.../site-packages/pydantic/schema.cpython-38-darwin.so in pydantic.schema.model_schema()
.../site-packages/pydantic/schema.cpython-38-darwin.so in pydantic.schema.model_process_schema()
.../site-packages/pydantic/schema.cpython-38-darwin.so in pydantic.schema.model_type_schema()
.../site-packages/pydantic/schema.cpython-38-darwin.so in pydantic.schema.field_schema()
.../site-packages/pydantic/schema.cpython-38-darwin.so in pydantic.schema.field_type_schema()
.../site-packages/pydantic/schema.cpython-38-darwin.so in pydantic.schema.field_singleton_schema()

/Library/Frameworks/Python.framework/Versions/3.8/lib/python3.8/abc.py in __subclasscheck__(cls, subclass)
    100         def __subclasscheck__(cls, subclass):
    101             """Override for issubclass(subclass, cls)."""
--> 102             return _abc_subclasscheck(cls, subclass)
    103
    104         def _dump_registry(cls, file=None):

TypeError: issubclass() arg 1 must be a class

I'm on python==3.8.7, pydantic==1.8.2, on MacOS 11.3.1
Am I missing something, or is this a real bug?

I'm having this same problem.  In my application I have a pydantic model that expects a subclass of AppModel which is very reproducible with any type reference:
from typing import Type
from pydantic import BaseModel
from myapp import AppModel

class ModelRef(BaseModel):
   cls: Type[AppModel]

ModelRef.schema()
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "pydantic/main.py", line 710, in pydantic.main.BaseModel.schema
  File "pydantic/schema.py", line 167, in pydantic.schema.model_schema
  File "pydantic/schema.py", line 548, in pydantic.schema.model_process_schema
  File "pydantic/schema.py", line 589, in pydantic.schema.model_type_schema
  File "pydantic/schema.py", line 241, in pydantic.schema.field_schema
  File "pydantic/schema.py", line 495, in pydantic.schema.field_type_schema
  File "pydantic/schema.py", line 839, in pydantic.schema.field_singleton_schema
  File "/usr/local/lib/python3.9/abc.py", line 102, in __subclasscheck__
    return _abc_subclasscheck(cls, subclass)
TypeError: issubclass() arg 1 must be a class

We’re no longer actively developing Pydantic V1 (although it will continue to receive security fixes for the next year or so), so we’re closing issues solely related to V1. If you really think this shouldn’t be closed, please comment or create a new issue 🚀.


Updated App Icon to match MacOS 11-12

What's this PR do?

[x] Big Sur Style app icon
[x] Updated Readme icon to reflect the same

Screenshots (if appropriate)

@imabhaysutar, thanks for your contribution. Do you check on both Dảk and light themes?

@viettrungphan I have. Does cheater-background.png need updating? or it is deprecated?

I have, tho the icon looks a bit rough on smaller sizes, will make a new commit for that in a bit.

@viettrungphan Tested with dark/light modes etc. I see no issues, attaching screenshots here.

When will this be released?

@viettrungphan When are you expecting the next release to go out?


added more mui classes for buttons

Address #588
Added size and variant for the button, along with testcase and example. Please let me know if this is on the right track and required any changes.

Hi @FredLL-Avaiga, the changes have been made. Please have a look based on your time.

We're discussing with @FabienLelaquais what should be done ...

@quest-bot loot #588


[nginx] Remove 'Forwarded' header

We noticed that rack-attack was blocking the Cloudflare IP that forwarded the bad request, rather than the originating client IP.
I figured out a fix for this: #4823. However, I have been wondering why that fix was even necessary.
I suspect that it might have been necessary because of the fact that Rack prioritizes the Forwarded header over X-Forwarded-For. I suspect that maybe the way that these headers are handled (maybe as they relate to cloudflare-rails?) is different, and/or maybe we aren't formatting the Forwarded header correctly. Maybe on Dokku (before our recent migration to raw Docker (Compose)) we weren't sending a Forwarded header at all, and that's why we didn't need #4823 when on Dokku.
Therefore, I am going to try reverting the #4823 change and also stop sending a Forwarded header. If, after this change, rack-attack continues to block the originating client IP rather than the Cloudflare IP, then my above theory is probably correct, and we'll leave things as they will at that point be. If we see a regression, then my theory is probably not correct, and we'll probably revert this PR. But I would like to try it and see what happens.
Unfortunately, it's sort of necessary to test this in production, because outside of a production environment it is difficult to simulate what the real request headers and environment etc will be when these requests are handled.
One nice thing about this change is that it allows us to delete a decent amount of NGINX code.

This fixes https://davidrunger.atlassian.net/browse/DEV-77 .


Fix system path

I followed the steps in the manual and didn't see the icons for many items. After modifying the path, the icons were loaded correctly :)

resolve https://github.com/Forien/foundryvtt-forien-armoury/issues


Inconsistency in the function execution

In the process-results function in api/controller/activities.
The variables for the array activities2 and the flag isValid are defined as global variables.
Since JS is asynchronous in nature and so is Node, this can lead to inconsistency in the array and flags in the case the function is executed at the same time by two or more users (If the flag is raised for one user, it will also be raised for the second user. It's also possible that the array will end up returning the same value for parallel executions)
This bug has a slight possibility of being detected in case of a few users, but for the long term, it's better if it's resolved.
https://github.com/llaske/sugarizer-server/blob/2cf2d524055323ded375a310ba4259aadde5ee9f/api/controller/activities.js#L230-L237
https://github.com/llaske/sugarizer-server/blob/2cf2d524055323ded375a310ba4259aadde5ee9f/api/controller/activities.js#L252-L256
For now, it seems like a minor issue. Should I fix this?

In the process-results function in api/controller/activities.
The variables for the array activities2 and the flag isValid are defined as global variables.
Since JS is asynchronous in nature and so is Node, this can lead to inconsistency in the array and flags in the case the function is executed at the same time by two or more users (If the flag is raised for one user, it will also be raised for the second user. It's also possible that the array will end up returning the same value for parallel executions)
This bug has a slight possibility of being detected in case of a few users, but for the long term, it's better if it's resolved.
sugarizer-server/api/controller/activities.js
Lines 230 to 237 in 2cf2d52
function process_results(req, activities) {
//duplicate activities
activities2 = [];
//add options first for filtering
var opt = {};
opt = addOptions('name', req.query, opt);
sugarizer-server/api/controller/activities.js
Lines 252 to 256 in 2cf2d52
activities.forEach(function(activity, key) {
//flag
isValid = true;
For now, it seems like a minor issue. Should I fix this?

@NikhilM98  yes, this should be fixed. I also think it would lead to some bugs in a race condition(multiple users). Global variables also reflect an inconsistent codebase.
Personal opinion

@Anshuman71 thanks for the quick feedback. I've created a PR handling the global variable affecting the race case scenario.

Fixed in #140


Result in JSON array instead of JSON even using sendSingle in actions.mqtt

My rule file:
  "sql": "SELECT avg(temperature) AS t_av, max(temperature) AS t_max, min(temperature) AS t_min, COUNT(*) As t_count, split_value(mqtt(topic), \"/\", 1) AS device_id FROM demo GROUP BY device_id, TUMBLINGWINDOW(ss, 10)",
  "actions": [
    {
      "mqtt": {
        "server": "ws://localhost:8083/mqtt",
        "topic": "devices/demo_001/messages/events/",
        "clientId": "demo_001",
        "sendSingle": true
      }
    }
  ]
}

Expected result:
payload:'{"device_id":"demo_001","t_av":6.667538453021003,"t_count":1,"t_max":6.667538453021003,"t_min":6.667538453021003}'
Actual result:
payload:'[{"device_id":"demo_001","t_av":6.667538453021003,"t_count":1,"t_max":6.667538453021003,"t_min":6.667538453021003}]'

Please try to add dataTemplate property as in below.
actions": [
    {
      "mqtt": {
        "server": "ws://localhost:8083/mqtt",
        "topic": "devices/demo_001/messages/events/",
        "clientId": "demo_001",
        "sendSingle": true,
        "dataTemplate": "{{json .}}"
      }
    }

Thanks, it works!

@KShih A more detailed doc is wrote for how to use data template in Kuiper, fyi.

Cool! good to know. As I referred to this doc, there is no such detail about using sendSingle with {{json .}}


Time interval format in Excel

I'm trying to write a time in format: "05:45:34" to a xlsx output file. Normally in Excel such cell is interpreted as time and Excel is able to perform operations on it (e.g. SUM).  Example of such file in attachment.
Trying few hours now I'm not able to create such cell with SheetJS. I've tried .t = 'n', .z = 'gg:mm:ss' but nothing helped. I've even tried to read a xlsx file and write it again without changes, but it doesn't work. You can see the example below on JSFiddle, after uploading the test file there and downloading it values formatting is lost.
I'm not sure if that's an error, or lack of some function, or my lack of SheetJS knowledge. Is there any way to create cell with such value that works with Excel?
Demo: https://jsfiddle.net/gwa61kj0/
Test file: test.xlsx

Your locale will display gg:mm:ss but it's actually stored in the file using the en-US h:mm:ss or [h]:mm:ss (those are actually built-in table indices 21 and 46).


Add min_by, max_by, and minmax_by

These functions behave like min, max, and minmax, respectively, but take a function which maps stream values prior to comparing them.

Looks good to me! I've decided that I'm happy with the given naming convention (as opposed to arg_min, arg_max).

Great! Thanks! You want me to add some documentation?

If you wouldn't mind, that would be great, otherwise I will add it. Working on a revamp to the docs system, so it would be a little while.

I don't mind at all. Anything special I need to do to avoid stepping on your toes for the revamp you're working on?

Nope, you should be good to go.


Lucario doesn't register as a steel type.

Unlike other steel types, lucario is hit by poison moves and doesn't "respond" to steel type breeding blocks. I tried testing it in Singleplayer and it worked fine so seems to be an issue with oblique.

Which pokemon were oyu using to test poison moves?

I can confirm, lucario does not take super effecctive damage from fire punch. I believe this to be a pixelmon bug

I used sludge wave from a gengar.

Just tested this in a creative singleplayer world and it seems to be a server side issue since pyro ball was super effective on my test

Unsure what the problem is. Lucario is defined as a steel type in our stat files.

I can confirm that this is an existing bug, I saw a Scizor get poisoned by a Chansey's Toxic.

Fixed as of Wednesday.


unix only ?

i have this error on window at compilation time : error[E0433]: failed to resolve: could not find unix in os

IIRC unix is only used in test code.
Do you have the output and the command you use?

FYI openssh-sftp-client compiles on windows, in our CI
If you enable feature openssh though, it would pull in openssh crate, which is unix-only.


Extend required settings documentation for configuring Celery to use Redis Sentinel

Proposed Changes
We need to add a section to the Required Settings > Redis Settings section of the documentation to instruct the reader how to configure Celery to use Redis Sentinel.
It appears that you can easily use it per the Celery docs, e.g. adding this to `nautobot_config.py:
CELERY_BROKER_URL = 'sentinel://localhost:26379;sentinel://localhost:26380;sentinel://localhost:26381'
CELERY_BROKER_TRANSPORT_OPTIONS = { 'master_name': "cluster1" }

We'll need to test this setup in a sandbox to assert the recommened patterns.

Justification
There is already a section on configuring RQ to use Redis Sentinel, so we just need to give the same treatment to Celery.
Acceptance Criteria

[x] Task Queuing under "Required Settings > Redis Settings" needs to be updated to mention the configurations needed for Celery
[ ] Caching docs updated to link to required celery settings (caching doesn't require celery but requires it to be configured)

Closed with #1532


Use parquet_2gb_sorted for gpuCI query tests

https://github.com/rapidsai/ops/issues/2672

Codecov Report

Merging #1154 (6f8d05a) into main (f8b7d05) will decrease coverage by 0.44%.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##             main    #1154      +/-   ##
==========================================
- Coverage   81.43%   81.00%   -0.44%
==========================================
  Files          78       78
  Lines        4395     4395
  Branches      797      797
==========================================
- Hits         3579     3560      -19
- Misses        640      650      +10
- Partials      176      185       +9

see 6 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Looks like https://github.com/dask-contrib/dask-sql/pull/1154#pullrequestreview-1440617650 seems to be coming from an updated Numba version to 0.57 with window functions.
Need to investigate further but should be good to xfail until then

Opened https://github.com/rapidsai/cudf/issues/13426 to track

Looks like the cuml failures should be fixed via https://github.com/rapidsai/cuml/pull/5434. In the interest of unblocking CI, we should be good to ignore those failures until cuml import errors are resolved.


How to use multiple documents in one loader?

I would like to load multiple document in docx format to index. Is it possible to load and save the resulting index to not run again?

If you want to load multiple .docx files, you have 2 options:

Feed each one into a DocxReader and concatentae the resulting documents together.
Put them all into a directory and load them with SimpleDirectoryReader.

Once you have your final list of documents, create an index with index = GPTSimpleVectorIndex(documents). Then, if you want to save/load it from disk,
# save to disk
index.save_to_disk('index.json')
# load from disk
index = GPTSimpleVectorIndex.load_from_disk('index.json')


HikariPool-1 - Connection is not available, request timed out after 250ms.

Environment
HikariCP version: 3.4.2
JDK version     : 1.8.0_111
Database        : MySQL

Log on New Relic
caused by java.sql.SQLTransientConnectionException: HikariPool-1 - Connection is not available, request timed out after 250ms.
…m.zaxxer.hikari.pool.HikariPool.createTimeoutException (HikariPool.java:689)
com.zaxxer.hikari.pool.HikariPool.getConnection (HikariPool.java:196)
com.zaxxer.hikari.pool.HikariPool.getConnection (HikariPool.java:161)
com.zaxxer.hikari.HikariDataSource.getConnection (HikariDataSource.java:100)
….commons.dbutils.AbstractQueryRunner.prepareConnection (AbstractQueryRunner.java:319)
org.apache.commons.dbutils.QueryRunner.query (QueryRunner.java:327)
…h.clean.repositories.AbstractDbRepository.executeQuery (AbstractDbRepository.java:37)
The application has 14K rpm at the time of the error. It works by traffic gusts, can have drastic traffic changes.
Configuration of the pool (a singleton datasource)
hikariConfig.setMaximumPoolSize(20);
hikariConfig.setMinimumIdle(3);
hikariConfig.setIdleTimeout(120000);
hikariConfig.setConnectionTimeout(250);
hikariConfig.addDataSourceProperty("cachePrepStmts", "true");
hikariConfig.addDataSourceProperty("prepStmtCacheSize", "250");
hikariConfig.addDataSourceProperty("prepStmtCacheSqlLimit", "2048");
hikariConfig.addDataSourceProperty("useServerPrepStmts", "true");
hikariConfig.addDataSourceProperty("useLocalSessionState", "true");
hikariConfig.addDataSourceProperty("rewriteBatchedStatements", "true");
hikariConfig.addDataSourceProperty("cacheResultSetMetadata", "true");
hikariConfig.addDataSourceProperty("cacheServerConfiguration", "true");
hikariConfig.addDataSourceProperty("elideSetAutoCommits", "true");
hikariConfig.addDataSourceProperty("maintainTimeStats", "false");
I cannot find the correct configuration for this application

250ms seems to be a quite short timeout if your database is under load, try to increase this timeout to some seconds... try this :
hikariConfig.setConnectionTimeout(3000); // 3 seconds of connection timout

@jbaezml The comment above regarding database load by @arisjramos is correct.  However, even if your database answers all queries in single-digit millisecond response times, at 14k requests you are likely generating a TON of garbage.  My guess is that you are encountering GC stop-the-world (STW) pauses greater than 250ms.
On Java 8, for that kind of load, you need to be using the CMS garbage collector, and it needs to be tuned properly.
https://wikitech.wikimedia.org/wiki/JVM_Tuning
https://dzone.com/articles/how-to-reduce-long-gc-pause
And you probably want to set MaxGCPauseMillis to some reasonable value.  But setting this to something too low like 50ms is likely to jack-up your application.
You can also try, the G1 collector with MaxGCPauseMillis defined, but it should probably only be used if your heap is >-= 4Gb.


building package failed

Howdy pardners,
I failed to build and install the package. The main problem was that you had line EXTRA in .Rbuildignore: according to the R extension writing manual, matching will be case insensitive, and this will drop R/extrapolation.R from the package tarball. An alternative to this change is to rename R/extrapolation.R. If EXTRA is supposed to be the whole word, adding terminator (EXTRA$) will also suffice.
The error occurs when trying to use the canonical way of installing the package:
> R CMD build BNPvegan
# * building ‘BNPvegan_0.1.0.tar.gz’
> R CMD INSTALL BNPvegan_0.1.0.tar.gz
# ** testing if installed package can be loaded from temporary location
#Warning: S3 methods ‘extrapolation.DP’, ‘extrapolation.PY’, ‘extrapolation.sdm’ were declared in NAMESPACE but not found
#Error: package or namespace load failed for ‘BNPvegan’ in namespaceExport(ns, exports):
# undefined exports: extrapolation
#Error: loading failed
#Execution halted
#ERROR: loading failed

The same error also appears when using devtools::install_github("alessandrozito/BNPvegan") in R.
After this there were only 3 WARNINGS and 4 NOTES which are all easy janitor jobs.
BTW, I like your coding style! Really elegant!

.Rbuildignore is case-insensitive (see
https://cran.r-project.org/doc/manuals/r-release/R-exts.html#Building-package-tarballs)
and having EXTRA there will drop R/extrapolation.R from the package

Hello Jari,
Thanks a lot for your help. I merged your branch with the main package. There are indeed several WARNINGS and NOTES, but at the moment we ignored them as we are still building the documentation and adding important functionalities.


The error message of the translation command

Chore Request
Describe the work
When u type qq translate the error message doesn't seem that need. It should be worked on.

the error message should be replaced with "That error message, should be replaced by Error wasn't able to translate, language not available"
Additional information

This issue point to errors returned by the API:

Can I work on this issue? Could you tell me how to?

At first, I thought this might be simple to resolve.
Now, it seems like something is wrong with the library used for this.
I think we could make this library work for this purpose-
https://textblob.readthedocs.io/en/dev/api_reference.html?highlight=translate#textblob.blob.Sentence.translate


YOUTUBE ICON SIZE NOW INCREASING ON HOVER

Description
The size of Youtube Icon is now increasing on hovering over it in the front page.

Thanks for your contribution @Shreyans2004


Implement more Reformulation Methods

https://www.sciencedirect.com/science/article/pii/S0098135415000587

Expanding on the original issue, some reformulations that would be nice to implement include:

[ ] Improved big-M (https://www.sciencedirect.com/science/article/pii/S0098135415000587)
[ ] P-Split (https://arxiv.org/abs/2202.05198)
[ ] Cutting-plane (https://doi.org/10.1016/S1570-7946(03)80444-3)

Pyomo.GDP implements quite a few more: https://github.com/Pyomo/pyomo/tree/main/pyomo/gdp/plugins


JAMES-3700 Add dead letter policy for Pulsar MailQueue

see https://issues.apache.org/jira/browse/JAMES-3700
@chibenwa can you check if I'm going the right way or not?

Apparently, scala compilation fails...
[ERROR] Failed to execute goal net.alchim31.maven:scala-maven-plugin:4.5.6:compile (scala-compile-first) on project james-server-queue-pulsar: scala compilation failed -> [Help 1]
org.apache.maven.lifecycle.LifecycleExecu

Can you check?

Hello @haanhvu
Any follow up on this work?

Hello @haanhvu
Any follow up on this work?

I'm at the end of my school term and having some stuff to complete. Will come back to this PR this weekend.

Is there any plan to carry on this work?

Is there any plan to carry on this work?

No sorry. Let me close this now.


fix: set compile scope for specific transitive dependencies to fix examples

:tada: This PR is included in version 1.0.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


How to install graphviz?

I have install the graphviz but still got module not found error.
Can anyone share your way?

solved


Fix(support for toml config) for newer version of Alacritty.

The future releases starting from 0.13.0 of the Alacritty terminal are shifting to TOML-based config, this PR adds support for the same. --Thanks

Thank you!!


Updates for Veza-Entra SSO Integration

Since this documentation was created, Veza has updated their SAML configuration page and migrated SAML providers. A few items needed updates:

Refresh screenshots to show the latest UI
Add detail on Single Log Out and SP-initiated login
Add instructions for retrieving SP metadata from the Veza UI, instead of contacting support
Update example URLs to use the new domain from "cookie" > " "veza"

@microsoft-github-policy-service agree company="Veza"

@jeevansd
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@cookiedocs, we created a separate PR in our private repo for the requested updates and the document will get live within 2-3 working days.
#please-close


Brain Score Logs

Hello, I recently submitted a model, however I don't see any data regarding the submitted model. Would it show up after 24 hours?

Hi @amirozhan, thanks for reaching out! Normally you will get an email with a link containing your build log, where you can then check the status of your model. On a successful scoring run, your model score will then show up on the website leaderboard within 24 hours.
However, right now do to our team still working out some kinks in the 2.0 refactor, this might not happen yet.
So, I went ahead and  manually looked up your submission to see what happened to your model. I see three submissions, all from January 3 2024. From these logs, it looks like your model might have errored due to some issues with the 1.x version Tensorflow. Please see the log from your last build here, as well as Issue #526 that addresses some 1.x Tensorflow stuff. Please let me know if you have any other questions, we are happy to help.


Trying to do face detection but get Error that module 'mediapipe.tasks.python.vision' has no attribute 'FaceDetector'

Have I written custom code (as opposed to using a stock example script provided in MediaPipe)
No
OS Platform and Distribution
MacOs Ventura 13 -- Python 3.7.4
MediaPipe Tasks SDK version
No response
Task name (e.g. Image classification, Gesture recognition etc.)
Face detection and facial landmark detection
Programming Language and version (e.g. C++, Python, Java)
Python
Describe the actual behavior
AttributeError
Describe the expected behaviour
Object should be created
Standalone code/steps you may have used to try to get what you need
Following the code example (https://github.com/googlesamples/mediapipe/blob/main/examples/face_detector/python/face_detector.ipynb), but getting an error when calling `vision.FaceDetectorOptions` saying module 'mediapipe.tasks.python.vision' has no attribute 'FaceDetector'. I noticed that face_detector.py and face_lamdmarker.py are in Github, not in site_packages after running pip install mediapipe.

Other info / Complete Logs
No response

Hi @YashPotdar,
It appears there is an issue with the primary Python version. I was able to successfully execute the same example using Python 3.11.6 and MP version 0.10.8 on macOS (Sonoma 4.1.1 - build 23B81) without encountering any errors.
Firstly, We recommend upgrading your Python version from 3.8 to 3.11 and attempting the process again. Secondary, If the issue persists, we advise upgrading to the newer macOS version or updating to macOS (Sonoma). Also, Please inform us if this works for you.
Thank you


CDS annotations: @Capabilities support?

I want to restrict the creation of entities in my CDS service. Only users that are assigned to the 'create' scope should be able to insert a new record. I want the '+' Button in the ListView disabled or invisible.
I annotated my entity as follows (according to this documentation https://github.com/oasis-tcs/odata-vocabularies/blob/master/vocabularies/Org.OData.Capabilities.V1.md#InsertRestrictionsType):
entity BusinessPartnerGroup @(
    Capabilities: {
        InsertRestrictions: {Insertable: true, Permissions: [Scopes: [{Scope: 'create'}]]},
        UpdateRestrictions: {Updatable: true},
        DeleteRestrictions: {Deletable: false}
    },
    restrict: [
        { grant: ['READ'],   to: 'read' },
        { grant: ['CREATE'], to: 'create' },
        { grant: ['WRITE'],  to: 'update' },
    ]
) as projection on ds.BusinessPartnerGroup {

When compiling I get the following errors (line 17 is the line with InsertRestriction):
[ERROR] srv/BusinessPartner-service.cds:17:86-87: Extraneous ',', expecting ';', '@', ABSTRACT, ACTION, ANNOTATE, ANNOTATION, ASPECT, CONST, CONTEXT, DEFINE, ENTITY, EVENT, EXTEND, FUNCTION, SERVICE, TABLE, TEMPORARY, TYPE, USING, VIEW, <EOF>
[ERROR] srv/BusinessPartner-service.cds:17:83-84: Extraneous ']', expecting ';', '@', ABSTRACT, ACTION, ACTIONS, ANNOTATE, ANNOTATION, ASPECT, CONFIGURATION, CONST, CONTEXT, DEFINE, ENTITY, EVENT, EXTEND, FUNCTION, HANA, SERVICE, TABLE, TECHNICAL, TEMPORARY, TYPE, VIEW
[ERROR] srv/BusinessPartner-service.cds:18:41-42: Mismatched ',', expecting <EOF>
[ERROR] srv/BusinessPartner-service.cds:17:63-64: Mismatched ':', expecting ',', '.', ']'
[ERROR] srv/BusinessPartner-service.cds:17:74-82: Mismatched String, expecting Identifier, '{', ARRAY, ASSOCIATION, COMPOSITION, LOCALIZED, MANY, TYPE

Cause I don't see a syntax error on my side I guess it is an error in CDS compiler. Can you please check this and fix it if my assumption is correct?
Thanks in advance
Helmut

There was a syntax error that I didn't see:
InsertRestrictions: {Insertable: true, Permissions: [Scopes: [{Scope: 'create'}]]},
has to be
InsertRestrictions: {Insertable: true, Permissions: [{Scopes: [{Scope: 'create'}]}]},
Curly brackets around the Scopes array.


Jsmn: value to key

is this possible , if i get key by value. I have string on bith sides, "Key" : "value" . i want to search in both direction.

Yes, it is possible. While traversing objects, instead of looking for specific key, you just check the value instead.

can you give me some idea ? what i have to comepare , i am trying but not get it

When you have token for your object (it will be the first one for root object) you will want to first check how many children it has (size property). Now you go through the n next tokens which will be keys of the object. Now, as you are on the key you should check if the size property is 1, that means next token will be the value for the key. Otherwise if it's 0 that means the key is without a value and next token will be next key. But if the next token is a value then you just check if it's actually a string by checking type property. of so then you just have to compare the strings. Examples on how to access the string can be found in the repo. Also, be careful in case you have values that are objects/arrays. You will have to skip all tokens which are part of it by basically checking the amount of children for all the children and skipping those tokens.
I hope this lengthy explanation is understandable.

Some steps can be skipped if you know how exactly the data you are processing looks like and you trust it. For example if it's just an object of key and string values and all keys have values then you could forget about checking sizes of keys and sizes of values. You can skip by two tokens of value doesn't match (skipping key and landing no next value token)


No support for TxnData fields?

There doesn't seem to be any support for the TxnData1, TxnData2, and TxnData3 fields in PxPayAuthorizeRequest::getData when building a request.
Unless I've missed something.

OK, I see the master branch has support for the TxnData fields in PxPay requests but it's missing from PxPost requests.
I've created PR #25 with support for the fields in PxPost requests.


Getting column to collapse at <600px

I have a 3 column format where I want the rightmost column to become a row at <= 600px wide. Can't figure it out. Am trying stuff like this to no avail:
@media (max-width: 600px) {
	.container {
		display: flex;
		flex-flow: column;
	}
}

The offending file is at GitHub Source, Preview
Bonus points for anyone who can tell me why bluerobot.svg doesn't display.

 Worked perfectly. Thank you.

@tomcam You're welcome.


Multi GPU

Hey, will the Multi GPU feature be out soon?

Hey! Probably not, if there's strong enough interest I can take a look but rn seems like it would take a bit of time and budget.


organization for network tower 3.0

SUMMARY
organization for the tower exercises
ISSUE TYPE

Docs Pull Request

COMPONENT NAME
documentation for ansible_network
ADDITIONAL INFORMATION

@IPvSean Don't see any issues! Good to merge.


E518: Unknown option: ,0=:: when using with haskell-vim

After typing where keyword, neovim throws an error E518: Unknown option: ,0=::. After that indentation, syntax highlighting and completion stop working.
Minimal config
set nocompatible

call plug#begin()
Plug 'hrsh7th/nvim-cmp'
Plug 'neovimhaskell/haskell-vim'
call plug#end()

lua <<EOF
  require'cmp'.setup({})
EOF

haskell-vim defines following indentkeys
setlocal indentkeys=!^F,o,O,0{,0},0(,0),0[,0],0,,0=where,0=let,0=deriving,0=in\ ,0=::\ ,0=\-\>\ ,0=\=\>\ ,0=\|\ ,=\=\

I'm getting this even without haskell-vim installed

Fixed.


Option to update the PR comment

New option to edit the existing comment in PR instead of adding a new comment on every commit.
This makes it clearer in the PR page.

Any updates on this @oacik? we would love to have it!

FYI, I've made a lot of merges today and would like to release PR #20 first then maybe merge #42 with this. Thanks!

Hey @zgosalvez, I was out of the loop for a while but at first glance #42 seems to fix the problems of this PR already, I am ok with closing this PR and merging the #42

Moving to PR #42. Thank you. 🙏🏼


Add the Sqitch app for database management

@lethalman I have incorporated your comments in commit a3a98d7. Thank you for your helpful input. Please review.

I'm totally confused. There's already an AppSqitch in perl-packages.nix in git master, why this PR adds another one?

@lethalman perhaps you are looking at the diff between this branch and 'master' at the time that this PR was opened, instead of this branch and current 'master'. This branch does not add an AppSqitch module. That was done in an already-merged PR. Try examining with 'git diff -w master..a3a98d7'.

I'm looking at the "Files changed" tab of this PR. Perhaps cleanup this PR by rebasing your changes against the new master.


Can't scroll on component mount

I have the following function to scroll to the bottom of the container
  const scrollChatToBottom = useCallback(() => {
    animateScroll.scrollToBottom({
      containerId: "event-chat-messages",
    });
  }, []);

And I want to execute that function on the component mount
  useEffect(() => {
    scrollChatToBottom();
  }, [
    scrollChatToBottom,
  ]);

But it's not working, is there a reason to that happen?

Should most likely work.
Could you provide a Plunkr/Codepen demonstrating the issue?

@fisshy Sure! I created a sandbox in order to demonstrate the issue

I'm not really sure why it doesn't work on initial mount. but providing a ref solves it.
`
import React, { useRef, useEffect } from "react";
import { animateScroll } from "react-scroll";
import "./styles.css";
export default function App() {
const { ref } = useRef();
useEffect(() => {
animateScroll.scrollToBottom({
container: ref,
duration: 2000,
smooth: true
});
}, [ref]);
return (

Hello CodeSandbox
Start editing to see some magic happen!

);
}
`

Maybe the element isn't declared in the tree yet.

Yepp, could be.

This issue is not solved, @fisshy.
Should be resolved internally within the lib - otherwise everywhere we use this we'll have to add a ref or setTimeout - which is not documented not practical.
If you want we could submit a PR fixing this issue.

Yes, I was a bit fast to close it, sorry.
You both are more then welcome to make a PR. Would be cool to make something like “useScroll” which resolves after the three is updated.

@fisshy
How about this?
I solved the problem with this.
import { useEffect, useRef } from "react";
import { scroller } from "react-scroll";

const containerRef = useRef(null);

useEffect(() => {
if (containerRef && containerRef.current) {
      scroller.scrollTo("event-chat-messages", {});
}, [containerRef.current]);

@ru88s It's a good workaround, I think I actually have a PR solving this, but I've forgotten to merge it.
https://github.com/fisshy/react-scroll/pull/435
I should probably merge this.

@fisshy
I understand now. Thank you very much.
I'm having trouble with the process after the load is complete, so please merge.


UnicodeDecodeError: 'utf-8' codec can't decode byte 0xe9 in position 84: invalid continuation byte

how can i solve this.
UnicodeDecodeError: 'utf-8' codec can't decode byte 0xe9 in position 84: invalid continuation byte

@gurupetach To help us resolve this issue please tell us which code file were you running? Also, add the error stack trace.

managed to solve
On Fri, Feb 19, 2021, 13:40 Abhi Ojha<EMAIL_ADDRESS>wrote:

@gurupetach https://github.com/gurupetach To help us resolve this issue
please tell us which code file were you running? Also, add the error stack
trace.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/udacity/deep-learning/issues/287#issuecomment-781992329,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJNDRDYCHG5F7ISTWO4XSH3S7Y52ZANCNFSM4KXBBAVA
.


Akka Streams Kafka 0.11

Please review @adamw @kciesielski
cc @myfear @drewhk @2m @johanandren @ktoso

Looks good :+1:

LGTM! Would love to also see a Java example if possible?

@myfear We can link to the documentation, which contains similar examples in java http://doc.akka.io/docs/akka-stream-kafka/current/home.html

I didn't see the graphs before in the doc where we edited this - I don't think they're that good "marketing wise" (well, it is a release announcement thus it is a marketing thing).
But anyway, let's ship it - it's much better than it used to be after all!
LGTM

@kciesielski I added you as author also
Publishing it now

Thanks @patriknw

it's live http://blog.akka.io/integrations/2016/09/10/akka-stream-kafka

Just noticed a typo in Softwaremill, should be SoftwareMill :)


Use of exec.Command("ifconfig") fails on new distributions

The ifconfig command is not included by default on installations of Ubuntu 18.04 and other recent Linux distributions (Debian, Centos, etc). The interface should probably be brought up using the ip command or a platform-native call.
The code in question is at https://github.com/meshbird/meshbird/blob/master/src/meshbird/iface/iface.go#L42

An option for platform-native would be to use/copy the underlying implementation of net.Interfaces(), which is net.interfaceTable(). It has all the needed information.

https://golang.org/src/net/interface_linux.go#L16
https://golang.org/src/net/interface_windows.go#L47
(No Mac to check where that code lives)


Collect 3D Models

Beer
Waffles
Chairs
Tables
Screen
Podium
Door

https://www.turbosquid.com/Search/3D-Models/lecture+hall?max_price=100&min_price=1
https://www.turbosquid.com/Search/3D-Models/free/beer


How were the train pairs for Robust made?

I couldn't find some training pairs in qrels.
Also is eval data for robust used as training data?

Training pairs and qrels are found here: https://github.com/Georgetown-IR-Lab/cedr/tree/master/data
We used the standard Robust04 5-folds from Huston & Croft (2014).
I recommend using OpenNIR instead of this repository.

Can you please confirm that the training is done on eval qrels?
Thanks

We used 5-fold cross validation over Robust04. So in other words, 5 models were trained, each using 3 folds for training, one for tuning (e.g., tuning the training epoch), and the final one held out for evaluation.


Concrete implementation tutorial

Even the references contains example, a more complete tutorial would be more helpful. At least a one for implementation with the model field. Another one for the widget, and maybe field could be nice too.

Started 'Tutorials' document in  in 0.3.0-pre.5


duplicate ids in different genes

from mock submissions
One transcriptome submitted with duplicated transcripts IDs (same transcript ID in different genes IDs) → SQANTI3 failed

I tried to run SQANTI3 for Cindy's PacBio transcriptome (without expression because of the previously reported issues with the IDs), but I couldn't because 2 transcript_ids were used several times for different isoforms (all of them mapping to the chrEBV). Its path in the Drive folder is Long-read RGASP/Submissions/Cindy_submitter_id/pacbio_mod_isoforms/5-pacbio-collapse.isoforms.gtf .
The transcript_ids that gave me problems were: "m54284U_191110" and "m54284U_191111". The rest of the transcript_ids were not duplicated and they were larger e.g. "m54284U_191111_172039/99944726/ccs".

GTF validations detect this problem.  thanks!!


The script doesn't launch "requirements have not been satisfied properly."

Checklist for submitting an issue to KickThemOut:

[x] I have carefully read the README file and haven't managed to resolve my issue.
[x] I have searched the issues of this repo and believe that this is not a duplicate.
[x] I am running the latest version of KickThemOut.

OS name & version: macOS Sierra 10.12.16
Python version: 2.7
Scapy version: 2.7
Nmap version: ????
Link of Gist: https://gist.github.com/falbinosantos/8c40f44459674a9ef29dcd68767d89c0

Description: I'm with a trouble... The script doesn't launch as you can see here:

ButI already installed all the requirements.

Hi there. Have you followed the MacOS installation process located in the README file? @falbinosantos

Yes I do. I included even the installation process on my gist https://gist.github.com/falbinosantos/8c40f44459674a9ef29dcd68767d89c0

I am having the same problem

Try the updated macOS installation guide. Let me know if it works. Thanks. @falbinosantos @Clean6

@k4m4 , same problem here...
Felipes-MacBook-Pro:~ Felipe$ cd kickthemout/
Felipes-MacBook-Pro:kickthemout Felipe$ sudo -H pip install -r requirements.txt
Password:
Requirement already satisfied: scapy in /Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages (from -r requirements.txt (line 1))
Requirement already satisfied: python-nmap in /Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages (from -r requirements.txt (line 2))
Felipes-MacBook-Pro:kickthemout Felipe$ sudo python kickthemout.py

ERROR: Requirements have not been satisfied properly. Please look at the README file for configuration instructions.

If you still cannot resolve this error, please submit an issue here:
	https://github.com/k4m4/kickthemout/issues

Felipes-MacBook-Pro:kickthemout Felipe$

In truth I didn't realize what was update in the macOS installation guide. Thank you.


Custom OpenAl Service Token calculation and model choice

Describe the need of your request
Hi! Please add token calculation for "Custom OpenAl Service".
It would be also great to choose models like in "OpenAl Service"
Thanks for your efforts and work.
Proposed solution

Like here

Additional context
No response

Thank you!


models

Is it possible to find other camera models online? Other than the nikon one you provide?

Our pipeline is an augmented version of this reversible imaging pipeline. The page that used to host additional models as well as the code needed to learn new models is here but unfortunately the site appears to be down. This is possibly because the PI on the project Michael S. Brown has moved to York University. We've emailed him in the past with questions so I'm sure he would respond if you were to ask him for additional models or the training code to make new models.

Thank you very much, I'll do that.


Merging 4.0 back into main

Release branch merges should not be squashed. Just regular merge commit.

I'll make sure we do that for next time.

I will see if I can fix it. Shouldn't have any impact.

All done, you might have to pull the repos again since I force pushed 4.0 and 5.0 tags/branches

Thank you!


fix: undefined errors are caused by faulty polygon nodes

Fixed by https://github.com/TamtamHero/polygon-faucet/pull/30


(MAINT) Change status logger to debug level

I noticed this problem in puppetserver, and I'm going around updating the other projects that use tk-status's periodic status logging

This commit changes the status logger's logging level to debug. This logback logger picks up log messages from the tk-status periodic logging feature, if it is enabled.
The periodic status logging feature in tk-status logs at the debug level, so
unless the logback root logger is set to debug, for the status logger to inherit from,
no periodic status log messages will appear.
This change has no affect on any of the the other logging that the app does

Test FAILed


Allow modifying the VM image used for the podvm instance

Currently the VM image used for the pod vm instance is configured during the start of CAA.
However, there is a need to use a different VM image for the pod vm instance.
One option is to use pod annotation like what we use for specifying the instance type.

Hi @bpradipt, any new info on this particular issue? We are interested in this use case and wanted to understand a little more about the work involved in making this happen. If we're going down the annotation path, would enabling the appropriate hypervisor annotation in the runtime and then leveraging that on the provider side be enough?
Do you see any extra effort required to upstream it (e.g. any considerations with regard to attestation)?

Raised an issue in kata-containers for enabling the new annotation fields: https://github.com/kata-containers/kata-containers/issues/10240


Website Addons

A developer building a website under ibm.com/design/* should be able to use a shared library of components.

[x] Header - css
[x] SideNav - css
[x] Website Footer
[x] 404 page
[x] App switcher
[x] BacktoTop
[ ] PageHeader
[ ] PageTabs
[ ] CodeSnippet
[ ] NextPrevious
[ ] ClickableTile
[x] Anchor Links
[ ] WebsiteTabs
[ ] DoDontExample
[ ] Row
[ ] Column
[ ] IconLibrary
[ ] ImageComponent
[ ] markdown
[ ] Video
[ ] TypeScale
[ ] TypeSpec
[ ] TypeWeight
[ ] Breakpoint Changer (currently in IDL, not Carbon)

Yes to all of that. Not sure on the best way to share the gatsby config? (for the /design site I coped the carbon site and stripped out all of the things I didn't need)

@alisonjoseph I wonder if we could have the add-on have the gatsby dependencies and exports a common gatsby-config.js file that people can import and do:
'use strict';

const defaultConfig = require('@carbon/addons-website/config';

module.exports = {
  ...defaultConfig,
  myCustomOption: 'foo',
  plugins: [
    ...defaultConfig.plugins,
    'my-custom-plugin',
  ],
};

I imagine that would work, just wondering how that might complicate things if someone wants to change certain config settings. (for instance there is a maxWidth setting for images, that is different between the Carbon site and /design site, google analytics plugin has the unique tracking ID.
Wonder if it might make more sense/be easier for someone, to create a "website-starter" with the basics in there pulling from the addons. (just thinking out loud).

@alisonjoseph definitely understand. This is partially why I thought the toolkit project could be interesting, one could do:
npx toolkit create my-project --preset @carbon/cli-preset-design-website

Would basically create a whole new project for them, but instead of being static files it'd basically have this as a dependency. Something like:
{
  "name": "my-project",
  "dependencies": {
    "@carbon/cli-preset-carbon-website": "x.y.z"
  },
  "toolkit": {
    "presets": ["@carbon/cli-preset-carbon-website"]
  }
}

So as the package is updated, one could do:
npx toolkit upgrade @carbon/cli-preset-carbon-website

You could also extend it by having the some gatsby plugin for the CLI load the local gatsby-config.js in addition to its own.

For the other option, I would assume it would happen like:
const defaultConfig = require('@carbon/addons-website');

module.exports = {
  plugins: [
    ...filterConfig(defaultConfig.plugins, ['my-plugin-key']),
    {
      name: 'my-plugin-key',
      options: {
        ...findInConfig(defaultConfig.plugins, ['my-plugin-key']),
        myCustomOption: 'value',
      },
    }
  ],
};

@vpicone fyi, these are the list of components we need to get moved to add-ons so we can start re-building out the IDL site


Hidden field not working in grid

When specifying a hidden field and minWidth in other fields in a responsive grid the hidden field will appear periodically.  In this example the id column still shows.  If I remove minWidth it hides but then responsive doesn't work.
cdnjs.cloudflare.com/ajax/libs/gijgo/1.9.10/combined/css/gijgo.min.css
cdnjs.cloudflare.com/ajax/libs/gijgo/1.9.10/combined/js/gijgo.min.js

grid = $('#grid').grid({
primaryKey: 'id',
dataSource: '/list/' + _userId,
uiLibrary: 'bootstrap4',
iconsLibrary: 'fontawesome',
responsive: true,
columns: [
{ field: 'id', hidden: true},
{ field: 'user_id', hidden: true},
{ field: 'rank', title: 'Rank', renderer: rankRenderer, minWidth: 150, priority: 3 },
{ field: 'item', title: 'Item'},
{ field: 'price', title: 'Price'},
{ field: 'location', title: 'Location'},
{ field: 'picture_url', title: 'Picture', renderer: imageRenderer, minWidth: 100, priority: 3 },
{ field: 'update_date', title: 'Last Updated', minWidth: 150, priority: 2},
{ field: 'Edit', title: '', width: 42, type: 'icon', icon: 'fa fa-pencil', tooltip: 'Edit', events: { 'click': Edit } },
{ field: 'Delete', title: '', width: 42, type: 'icon', icon: 'fa fa-remove', tooltip: 'Delete', events: { 'click': Delete } }
]
});

Can you remove these columns from the definition of the column. If you want to hide them you not need to be defined. You can use the values in these columns in your code, because if you return them from the server they stay as part of the record in the front end.

I need these columns to sort the grid by these hidden columns. Ex: There is a column called "code" and other two columns named "number" and "year". The column that I need to show is "code" = "number" + "year" but the sort must be "year" and "number". Did you understand the issue?


Form Input Sizing

Here I have explained how the size of the input in form could be changed

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Hi
As there has been no response from your end for a while, we are closing this PR.
Feel free to request this PR to be reopened, if you still wish to make the changes you originally added. Also, remember to update your fork, so you have the latest version of master in case it has changed significantly.
Happy Coding! ✈


[Proposal] Add transaction management helper

Suggestion is to add helper method similar to the one described at
https://bun.uptrace.dev/guide/transactions.html#runintx

My first instinct is that this is not useful to Bob. While someone may decide to use a transaction wrapper, there is no need to bake this into Bob.
The reason is that every Bob method take a bob.Executor, which is similar to Bun's IDB interface.

If you have a top level bob.DB, then you can start a transaction with tx, err := db.BeginTx(ctx, nil).
If you have a top-level *sql.DB:var db *sql.DB
tx, err := db.BeginTx(ctx, nil) // tx is *sql.Tx
bobTxExec := bob.NewTx(tx)

You can then use this tx as your Bob executor, and then commit/rollback as you please (my preferred way is to create a new context and cancel with defer)
In your changes, I can rewrite your testOperationsWithDatabase to like this:
func testOperationsWithDatabase(ctx context.Context, db *sql.DB, value int32) error {
	ctx, cancel := context.WithCancel(ctx)
	defer cancel() // rollback transaction if not committed

	sqlTx, err := db.BeginTx(ctx, nil)
	if err != nil {
		return err
	}

	tx := bob.NewTx(sqlTx)

	q := psql.Update(
		um.Table("table"),
		um.Set(psql.Quote("field").EQ(psql.Arg(value))),
	)

	_, err = q.Exec(ctx, tx)
	if err != nil {
		return err
	}

	q2 := psql.Insert(
		im.Into("table2"),
		im.Values(psql.Quote("value").EQ(psql.Arg(value))),
	)

	_, err = q2.Exec(ctx, tx)
	if err != nil {
		return err
	}

	return nil
}

@stephenafamo thank you for the quick response. I understand your opinion. I would like to have such a helper as part of bob because it is currently impossible to write a universal helper. The reason for this is that BeginTx returns a type specific to bob rather than sql.Tx. I understand why this was done, but it forces us to have a helper repository specifically for projects that will be written using bob.  As a former user of bun, I am used to having such functionality included in the package itself.
Anyway, I will accept an decision.
Regarding the provided example, I am not sure that I understand how transaction will committed this way. Maybe I am not aware about some auto-commit mechanics which bob has?

Regarding the provided example, I am not sure that I understand how transaction will committed this way. Maybe I am not aware about some auto-commit mechanics which bob has?

My bad 🤦🏾 . I missed calling tx.Commit() in my example. Changed the last line to return tx.Commit()

I would like to have such a helper as part of bob because it is currently impossible to write a universal helper. The reason for this is that BeginTx returns a type specific to bob rather than sql.Tx. I understand why this was done, but it forces us to have a helper repository specifically for projects that will be written using bob.

🤔 Alright, I see how this could be useful.
However, instead of placing this in a separate package, how about a method on bob.DB.
func (db bob.DB) InTx(ctx context.Context, fn TxFunc, opts ...TxOption) error {
    // ...
}

Would this work well for your usecase?

I see the only benefit of a dedicated package is the ability to register additional callbacks like OnRollback and OnCommit, as provided in the example. However, in most cases, db.RunInTx will be sufficient.
I can prepare changes to add RunInTx as a DB method.

@stephenafamo I updated PR, but I was unable to test these changes within my code, since the newest version has some breaking changes. After I regenerated the code I see many errors related to missing method Apply.
Additionally, conditions like the ones described here - https://bob.stephenafamo.com/docs/models/table#insert - aren't working anymore and now return a query instead of the inserted entity. Were these changes expected?

Thank you very much for the example. However, this has come as an unpleasant surprise. It will necessitate changes in several services on our side if we want to upgrade the version.

I apologise for the breaking changes, it was not made lightly.
One of the main driving factors was making hooks robust. Unfortunately, it was very easy to write/use queries that bypassed hooks and I really want Bob to have very few footguns. If something can be done, it should work as expected.

Hello @dmakushin 👋🏾
Any progress with this?


1.18 Goaway changes

10K single cluster density test is currently having "etcdserver: too many requests" issue. Pick up community https://github.com/kubernetes/kubernetes/pull/88567 just in case it helps.
Other cherry pick changes are due to dependencies.
Plus Arktos copy right setting.
1/25 density test data looks better than 1/15 & 1/21. apiserver_current_inflight_request is much smooth and closer to 1.18. This is a good cherry-pick
Arktos 1/25/2021

1.18

/approve

/approve

/lgtm

/lgtm


TLS Server can't get client certificate

until read first byte from connection.
for example:
server: https://gist.github.com/rekby/201588f226de4d00a35a
client: https://gist.github.com/rekby/348b0e2faac09ccb6477

Call Handshake explicitly.


Comments after alpha audit of path forwarding

From internal alpha audit of ICS20 v2 path forwarding:

[x] Add a comment in unwindHops where we loop over the trace elements except the first one. [@gjermundgaraba] #6702
[x] Move all forwarding validation together to the same helper function if possible (including the check for empty source port and channel if unwind is true). [@bznein] #6706
[x] Run packet data validation after it is constructed in createPacketDataBytesFromVersion both for V1 and V2. Change signature of function to also return error . @chatton  https://github.com/cosmos/ibc-go/pull/6704
[ ] Pass only hops to sendTransfer. Change EmitTransferEvent to accept only hops as well. [@gjermundgaraba] #6703
[ ] getReceiverFromPacketData switch around the if statements and add comment about why we do that: If the forwarding is set, then do not process the data.Receiver field and instead just shortcircuit to create receiver from forwardingInfo [@gjermundgaraba] #6709
[ ] Switch to if-else blocks in OnRecvPacket instead of using continue [@crodriguezvega] https://github.com/cosmos/ibc-go/pull/6700
[ ] Maybe replace SenderChainIsSource and ReceiverChainIsSource with function IsTracePrefixed (like the one used in the ICS20 spec). Open a PR and see which one is easier to reason about. [@crodriguezvega]
[ ] Add hops to the ack error while propogating back to source so we know which chain the packet failed or timed out on [could also do counter]. Include also the deterministic error string from the error acknowledgement written by the chain where the forwarding failed. @chatton
[x] Authz:

Allow explicit list of allowed hops. Do not allow unwind boolean
Use repeated AllowedForwarding allowed_forwarding with AllowedForwarding with repeated Hop hops . We choose to go with this option. Granter will have to specify the list of hops to unwind. @chatton https://github.com/cosmos/ibc-go/pull/6701

[x] Replace deep equal in authz with something else. Check if slices.Contains works to get rid of the loop [bznein] https://github.com/cosmos/ibc-go/pull/6697

Thank you @chatton for leading the walkthrough and the rest of the team for the feedback.

Add hops to the ack error while propogating back to source so we know which chain the packet failed or timed out on [could also do counter]. Include also the deterministic error string from the error acknowledgement written by the chain where the forwarding failed

Spent some time looking at this with @chatton today! This is not so straight forward to do, we must also consider that the final destination hop may or may not be on ics20-1. In any case, it seems better to add the path forwarding info here, accepting the ack as an additional argument from the caller, here.
Using something like the following (where packet is the forwardPacket, and not the reverse (prevPacket):
// NewForwardErrorAcknowledgement returns a new error acknowledgement with path forwarding information.
func NewForwardErrorAcknowledgement(packet channeltypes.Packet, ack channeltypes.Acknowledgement) channeltypes.Acknowledgement {
	ackErr := fmt.Sprintf("%s/%s/%s", packet.GetSourcePort(), packet.GetSourceChannel(), ack.GetError())
	return channeltypes.Acknowledgement{
		Response: &channeltypes.Acknowledgement_Error{
			Error: ackErr,
		},
	}
}

Consider a path of len(4), where the final destination is ics20-1. We fail with error acknolwedgement for {code}.
Acks would be propagated like so:

RecvPacket failure (hop3): ABCI code: {code}: error handling packet: see events for details
Handle ack(hop2), append path info and writeAck: transfer/channel-2/ABCI code: {code} ...
Handle ack(hop1), append path info and writeAck: transfer/channel-1/transfer/channel-2/ABCI code: {code}...
Handle ack(hop0) - original sender. Note, we cannot modify the acknowledgement on the chain that it was not written on and therefore the first outbound port/channel ID from A->B would not be included in the emitted event.

Ultimately, we cannot control if the final hop has upgrade yet or not. However, we can add packet.DestPort()/packet.DestChannel() to ALL error acknowledgements using ibc-go/v9.
For example:
-func NewErrorAcknowledgement(err error) Acknowledgement {
+func NewErrorAcknowledgement(portID, channelID string, err error) Acknowledgement {
        // the ABCI code is included in the abcitypes.ResponseDeliverTx hash
        // constructed in Tendermint and is therefore deterministic
        _, code, _ := errorsmod.ABCIInfo(err, false) // discard non-deterministic codespace and log values

        return Acknowledgement{
                Response: &Acknowledgement_Error{
-                       Error: fmt.Sprintf("ABCI code: %d: %s", code, ackErrorString),
+                       Error: fmt.Sprintf("%s/%s ABCI code: %d: %s", portID, channelID, code, ackErrorString),
                },
        }
 }

I think this would give us a bit more info, but no guarantees that the final destination has been upgraded.
Personally, I think other solutions sound pretty hacky. You could add some index to the error string in the acknowledgement, and parse that string on every async ack write, and increment. And then use the index to determine the failed hop when you have been propagated back to the original src and have all forwarding path info available. I would vote against this!
cc. @AdityaSripal


backtrace fails with minimum supported Rust version (1.16)

The “Minimum Rust (Linux)” and “Minimum Rust (Windows)” jobs on Azure keep failing with:
Compiling backtrace v0.3.33
error: expected identifier, found keyword `crate`
  --> /usr/local/cargo/registry/src/github.com-1ecc6299db9ec823/backtrace-0.3.33/src/lib.rs:76:9
   |
76 | pub use crate::backtrace::{trace_unsynchronized, Frame};
   |         ^^^^^

error: aborting due to previous error

error: Could not compile `backtrace`.

Is there any way to fix this (e.g. only on Azure or in the Cargo.toml) or to work around it?

Ah it's fine to bump it to 1.32.0 in that case, feel free to send a PR!

I'm gonna close this since I don't think this is related to this crate, but PRs are always welcome to the backtrace crate!


support for async on_transition() and on_dispatch()

We just tried to make all the methods of our state machine async. For on transition we hit:
  --> bazel-out/k8-fastbuild/bin/otiv3/pipeline_manager_process/state_machines/state_machine.rs:87:21
   |
87 |     on_transition = "Self::on_transition",
   |                     ^^^^^^^^^^^^^^^^^^^^^ expected fn pointer, found fn item
   |
   = note: expected fn pointer `for<'a, 'b, 'c> fn(&'a mut StateMachineStruct<_>, &'b State, &'c State) -> ()`
                 found fn item `for<'a, 'b, 'c> fn(&'a mut StateMachineStruct<_>, &'b State, &'c State) -> impl Future<Output = ()> {StateMachineStruct::<Handler>::on_transition}`

We get a similar error for on_dispatch. These methods aren't used in your async example. Are they just not supported yet?
Thanks!

ON_DISPATCH and ON_TRANSITION in the trait IntoStateMachine are non-async.
Thus, the issue isn't just in the macro.

Draft PR: https://github.com/mdeloof/statig/pull/29
Draft because at the moment we haven't run the test suite yet.


[WIP] ci: Add pipeline change detection

Signed-off-by: Ryan Northey<EMAIL_ADDRESS>
Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

hah I was about to ask if it was easier for you to pick this up than suggest all the fixes.  Thanks so much for tackling it!  LMK when this is ready as I'd love to get something in this week

i have not added any actual exclusions to this PR - just the framework for detecting changes - as when we add any exclusions we need to think carefully about how it will run on main or other release branches and on tagged commits/releases
im inclined to not exclude anything when a PR lands - this does risk PRs passing and then failing when they land - but i would rather that than commits passing and then releases not passing and i think we need to ensure every change is tested somewhere

testing PR is #24204

this is probably gtg altho we may need to iterate - testing is hard because the PR introduces change itself

I'd be inclined to land with at least one CI we think is affected (i.e. https://github.com/envoyproxy/envoy/pull/24204/files) as I'm not sure landing this as-is is going to tell us anything. WDYT?

WDYT?

adding a change here is probably not going to help as the PR contains change itself
i think land this and then run the test PR a couple of times, and then clarify what should/not be excluded

adding a change here is probably not going to help as the PR contains change itself

hmm, actually thinking about it - yep i see - we need at least one thing to be excluded to test - ill add ...

yeah I'll also say now we have this (or almost) I am so excited to say not run all the Envoy tests for docs only changes.  This is going to be pretty awesome :-D

k - ive added a condition which hopefully will only run coverage iff either:

commit is postsubmit (on any branch)
the PR is neither docs-only nor mobile-only

should work


Moving Location and Market model, migration, factory.

Moved Market model.
Moved Location model.
Added model class and service references.
Added drewroberts/media package.
Moved Market + Location migrations and factories.

I need to tag a new version on the media package before releasing this package.


Roll Flutter Engine from 820645dbcc69 to 76a873d5fee3 (3 revisions)

https://github.com/flutter/engine/compare/820645dbcc69...76a873d5fee3
2024-01-09<EMAIL_ADDRESS>Roll Skia from 92b84715ba73 to 360944e3175f (2 revisions) (flutter/engine#49622)
2024-01-09<EMAIL_ADDRESS>Roll Skia from 06404ed554e7 to 92b84715ba73 (1 revision) (flutter/engine#49620)
2024-01-09<EMAIL_ADDRESS>Roll Dart SDK from e8aa66f435ae to 721fccffc708 (5 revisions) (flutter/engine#49616)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC bdero@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


product name typo: "Custom Vision Service" is correct

@overbrimming : Thanks for your contribution to the Azure documentation! The author, @azdocprs, has been notified to review your proposed change.

@jasongroce - Please review the proposed changes.  Thanks

@jmgsworld

@overbrimming You are right! Not sure why your pr did not get on our radar sooner. Thank you!!

#sign-off


it shows this error when file name is chiness，but when file name is english ,is ok

@prscX

大兄弟
遇到这个问题了吗
Could not find com.nbsp:library:1.8.


Covealls

Do we want Code coveralls?  https://coveralls.io/ ?

We dont need this. Closing this issue


UserWarning: [W036] The component 'matcher' does not have any patterns defined.   matches = matcher(doc)

Thanks a lot for your fabulous package; it is really helpful. However, when I tried to reproduce your results, I run into this error:
"UserWarning: [W036] The component 'matcher' does not have any patterns defined. matches = matcher(doc)"
Specifically, I used this code snippet:
import spacy
from spacy.pipeline import EntityRuler
from spaczz.pipeline import SpaczzRuler
nlp = spacy.load("en_core_web_sm")

entity_ruler = nlp.add_pipe("entity_ruler", before="ner")
entity_ruler.add_patterns(
    [{"label": "GPE", "pattern": "Nashville"},
     {"label": "GPE", "pattern": "TN"}]
)

spaczz_ruler = nlp.add_pipe("spaczz_ruler", before="ner") #spaCy v3 syntax
spaczz_ruler.add_patterns(
    [
        {
            "label": "NAME",
            "pattern": "Grant Andersen",
            "type": "fuzzy",
            "kwargs": {"fuzzy_func": "token_sort"},
        },
    ]
)

When I add patterns to the spaCy's EntityRuler, it works OK, when I add patters to SpaczzRuler, it throws the error I specified in above.
I am using Python 3.9, spaCy 3.2.2 on Ubuntu 16.04.
Any help will be highly appreciated.

Hi @kormilitzin, thanks for your interest in spaczz! The warning you are seeing normally happens because of the way I have implemented the TokenMatcher (which is part of the SpaczzRuler) which will sometimes result in an empty spaCy Matcher being called on text. This is an inefficiency and something I need to fix but it shouldn't actually affect the results of the EntityRuler or the TokenMatcher.
In the example above, I'm not entirely sure at the moment why the warning is coming up while adding patterns but the same principle applies. All the patterns you added should be there when you look at spaczz_ruler.patterns.
If however, you are seeing incorrect results from running the SpaczzRuler or in SpaczzRuler.patterns please update me here.

@gandersen101 Thanks a lot for your details response; that totally makes sense to me. This warning was a bit puzzling as your implementation did work as a charm (i.e. it identified fuzzy matches), but I was surprised to see this warning. I will keep trying and report if I find anything not right. Thanks again for your amazing package!

I believe this should be fixed by #81.


Removed untested methods that shouldn't be needed with new viz.json

I don't fully understand what those two methods are doing. They are not tested and examples work fine without them so :fire:. Also, we're re-doing the viz.json so things are going to change anyhow.
cc: @javisantana

Basically you can fire them but I would not remove them without adding the functionality at the right place


Container crashes when trying to connect with WinSCP

Hi,
I've set this up and am trying to use it for the first time. I am finding an issue where after connecting with WinSCP to begin using the server, it crashes and burns (and the container stops running)
After authentication,

Logs don't reveal much either. This is all that appears in Docker console before it stops running/crashes.

I've attempted to break into the container to grab more information, but I've hit a roadblock.
docker exec a89d0dd630d2 bash
OCI runtime exec failed: exec failed: container_linux.go:380: starting container process caused: exec: "bash": executable file not found in $PATH: unknown

Appreciate your help!

I think the issue is this line in the config:
ftp_data_port=21
it's trying to bind the data port on the same port used for control. Commenting out this line should fix the issue.


Arabic (.ar) translation doesn't work

We have used vernacular for other language translations successfully, but if we try to use arabic, we just get the untranslated terms back. I've tried to use another language's strings as well for the arabic version just to test whether it is the arabic text itself that's messing with us, but even if I use any latin language for the .ar version, it still doesn't work.
EDIT:
Interestingly enough, if I copied the Arabic text itself for another language (let's say Spanish resources), I will see the Arabic translation in the app. Somehow it seems like the Arabic flag is ignored in the project when it comes to recognising available languages?
Both for iOS (ar.lproj) and Android (values-ar).

Problem comes from the default value for pluralCount (= 1) in ResourceCatalog.GetResource.
It's used to generate the resource_id which is actually using a 0 value if no plural is specified.
By changing the default value of the method to 0, it works perfectly ... not sure it's the right fix though.


Terminal tab 'Run Active File' option only creates NotePad display of python 3.7 script.

VSCode Version:  1.30.1
OS Version:  Windows 10 version 1809

Steps to Reproduce:

Click Terminal tab
Select 'Run Active file
If first time for terminal prompt "Cannot read property 'indexOf' of undefiined error occurs.
Repeat a second time using the same terminal and the NotePad display of the .py script file appears.
If 'Run Python File from Terminal' selected from the Right Click menu:  The script produces the results expected.

Does this issue occur when all extensions are disabled?: Yes
The Right Click Menu now has only Change ..., Cut, Copy, Paste and Command Palette
I now hope I can recover the 3 Disabled extensions.  With the Python extensions disabled, I can no longer execute my scripts in any form.

Have you seen Tyriar's answer here: https://github.com/Microsoft/vscode/issues/65696?
From what is answered, it seems it will be fixed in December/January, as I understand.

https://github.com/Microsoft/vscode/issues/65831


SAMBA submodule for develop branch

Now this should be working---I am adding SAMBA as a submodule to the develop branch of bandframework so that it can be easily accessed.

Going to be a bit close to the rules while we start out.  You can keep updating your branch/this conversation as we go, you will not need to submit a new pull request.
Some comments:

Please indicate in the pull request text who tested your package as described in the README and tag them as reviewers.  They should write their approval here.
For your notebook instructions for testing, please include some expected output in addition to looking for "no errors".  This will help verify that the software is working correctly.
For your warnings, go ahead and include some common warnings (maybe just the ones seen by the testers).

1). Testers for the SAMBA package are Daniel Odell and Ozge Surer. They have now been tagged as reviewers of this PR. Both will have tested SAMBA by next week, so this can be merged once they have approved it.
2). I have now included some expected array output and estimated values for the methods in the notebooks, so they can be checked against something when tested. I will also add a README there to discuss the process a bit.
3). I have now included the warnings that pytest has raised when testing using the Tests folder files in the README of that folder.

@odell and @ozgesurer : once you have tested and found success, please post here or let me know and we can merge.


Infinite scroll view isn't working but pull to refresh is working with collection view?

Infinite scroll view isn't working but pull to refresh is working with collection view? Why so?
Target : iOS 7.0 and above

@hemangshah Same issue


Post exit update attempt abends if another user has VSCode open on the same machine.

Type: Bug
Under Windows 10, if user A opens vscode, switches to user B, opens vscode and closes vscode while an update to vscode is pending, the update process for user B reports obscure errors rather than simply identifying that another user (user A in this case) has vscode locked, and as such an update isn't possible at this time (or alternatively, just silently quiting).
VS Code version: Code 1.89.0 (b58957e67ee1e712cebf466b995adf4c5307b2bd, 2024-05-01T02:09:22.859Z)
OS version: Windows_NT x64 10.0.19045
Modes:

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.89.1. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!


search filter icons on mobile

on mobile (chrome) there is no way to tell which filter is selected. on the pc version there is filter icon and filter name displayed, and on the mobile there is only filter icon which is the same for all filters.

Will be addressed in the next major release (Iris)


Remove Node.js requirement with Scala.js

Previously, it was not possible to use MUnit with browser-based
JavaScript environments because MUnit imported the Node.js fs module.
This commit refactors the code so that MUnit should gracefully recover
when the Node.js fs module is not available.
Fixes #247

Thanks, checked it out locally. It does not throw as before, but now MUnit does not print any information about the assertion failures except the name of the test in which they appeared.
Munit on a JVM project prints this kind of useful error on failure:
==> X com.raquo.mapk.v1.V1MapKSpec.xxx  0.016s munit.ComparisonFailException: <projectRoot>/src/test/scala/com/raquo/mapk/v1/V1MapKSpec.scala:32
31:  test("xxx") {
32:    assertEquals(true, false)
33:  }
values are not the same
=> Obtained
true
=> Diff (- obtained, + expected)
-true
+false

Whereas MUnit in this PR prints only the following on a similar test in a JS project:
==> X com.raquo.domtestutils.MUnitJsDomSpec.xxx 0.01s munit.ComparisonFailException:

Also, the console is now printing a warning (not an error) before the tests:
[info] Writing and bundling the test loader
[warn] ./domtestutils-test-fastopt.js 4434:33-48
[warn] Critical dependency: the request of a dependency is an expression
[warn]     at CommonJsRequireContextDependency.getWarnings (<projectRoot>/target/scala-2.13/scalajs-bundler/test/node_modules/webpack/lib/dependencies/ContextDependency.js:40:18)
[warn]     at Compilation.reportDependencyErrorsAndWarnings (<projectRoot>/target/scala-2.13/scalajs-bundler/test/node_modules/webpack/lib/Compilation.js:1360:24)
[warn]     at <projectRoot>/target/scala-2.13/scalajs-bundler/test/node_modules/webpack/lib/Compilation.js:1168:10
[warn]     at AsyncSeriesHook.eval [as callAsync] (eval at create (<projectRoot>/target/scala-2.13/scalajs-bundler/test/node_modules/tapable/lib/HookCodeFactory.js:33:10), <anonymous>:13:1)
[warn]     at AsyncSeriesHook.lazyCompileHook (<projectRoot>/target/scala-2.13/scalajs-bundler/test/node_modules/tapable/lib/Hook.js:154:20)
[warn]     at Compilation.finish (<projectRoot>/target/scala-2.13/scalajs-bundler/test/node_modules/webpack/lib/Compilation.js:1163:28)
[warn]     at <projectRoot>/target/scala-2.13/scalajs-bundler/test/node_modules/webpack/lib/Compiler.js:622:17
[warn]     at eval (eval at create (<projectRoot>/target/scala-2.13/scalajs-bundler/test/node_modules/tapable/lib/HookCodeFactory.js:33:10), <anonymous>:9:1)
[warn]     at <projectRoot>/target/scala-2.13/scalajs-bundler/test/node_modules/webpack/lib/Compilation.js:1095:12
[warn]     at <projectRoot>/target/scala-2.13/scalajs-bundler/test/node_modules/webpack/lib/Compilation.js:1007:9
[warn]     at processTicksAndRejections (internal/process/task_queues.js:79:11)
[warn]  @ ./domtestutils-test-fastopt-loader.js

You mentioned in previous comments that you need the fs module to read the source files to display information about the failure. So... does this PR replace that logic with a stub if the fs module can't be loaded?

@raquo thank you for trying it out! What Scala.js linker settings are you using to reproduce that? It might require a small tweak to get the exception message

@olafurpg Here is all of the linker config and a few more relevant bits:
scalaVersion := "2.13.5" // Scala.js version is 1.5.0
(Test / scalaJSLinkerConfig ~= { _.withModuleKind(ModuleKind.CommonJSModule) })
(Test / requireJsDomEnv) := true
(installJsdom / version) := "16.4.0"
useYarn := true
(Test / parallelExecution) := false

No other scalaJSLinkerConfig settings (I also had source maps off, but commented that out, and still the same). I only added the line with CommonJSModule because MUnit docs told me to, but commenting it out does not change anything.
One weirdness on my side that I can think of is that my project where I'm trying to use this is a testing library, so it requires the MUnit dependency without appending % "test". Whereas my scalaJSLinkerConfig has a Test / prefix as you see above. So maybe something is not catching on? I tried removing that prefix or replacing it with Compile /, but that did not produce any effect.
For the record, the branch that I'm trying to build is here. The simple test that I made to fail is MUnitJsDomSpec. MUnit version is defined in Versions.sbt.

@olafurpg I can confirm this PR works over at https://github.com/http4s/http4s/pull/4938.
Thanks for making this change, I'm badly in need of this. Is there anything I can do to get this merged and (snapshot) released ASAP? I'm happy to polish up this PR as @gabro and @tgodzik see fit.


Error in bezier.Curve due to verify argument

Hi,
Thanks for publishing this useful code!
The issue
When I run this code, I get the following error:
Traceback (most recent call last):
  File "bezier-curved-edges-networkx/run.py", line 14, in <module>
    curves = curved_edges(G, positions)
  File "bezier-curved-edges-networkx/curved_edges.py", line 58, in curved_edges
    curveplots.append(bezier.Curve(nodes, degree=2).evaluate_multi(np.linspace(0,1,bezier_precision)).T)
  File "/lib/python3.9/site-packages/bezier/curve.py", line 101, in __init__
    self._verify_degree(verify)
  File "/lib/python3.9/site-packages/bezier/curve.py", line 161, in _verify_degree
    raise ValueError(msg)
ValueError: A degree 2 curve should have 3 nodes, not 4.

The potential fix
It looks like this is due to a release of the bezier package adding the verify argument to the Curve constructor (this release: https://bezier.readthedocs.io/en/stable/releases/0.11.0.html).
So when I set verify=False in line 58:
curveplots.append(bezier.Curve(nodes, degree=2, verify=False).evaluate_multi(np.linspace(0,1,bezier_precision)).T)
it works!
Or instead, I can change the degree:
curveplots.append(bezier.Curve(nodes, degree=3).evaluate_multi(np.linspace(0,1,bezier_precision)).T)
and this works too.

@Izzy-Newsham Apart from your solutions, we can now use from_nodes to not bother with the degrees. The code can thus be changed like this:
curveplots.append(bezier.Curve.from_nodes(nodes).evaluate_multi(np.linspace(0,1,bezier_precision)).T)


ClusterLoader - extracting prometheus query execution to util

Extracting prometheus query execution from api_responsiveness_proemtheus.go to pkg/measurement/util.

/assign @mm4tt
/cc @wojtek-t


FIx DIagnostic flaky tests

Packages impacted by this PR
@azure/cosmos
Issues associated with this PR
Describe the problem that is addressed by this PR
A number of diagnostic-related tests in the SDK are failing for the compute endpoint due to variations in metadata call behavior across different gateways. As a solution, this pull request eliminates these tests.
What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?
Are there test cases added in this PR? (If not, why?)
yes
Provide a list of related PRs (if any)
Command used to generate this PR:**(Applicable only to SDK release request PRs)
Checklists

[ ] Added impacted package name to the issue description
[ ] Does this PR needs any fixes in the SDK Generator?** (If so, create an Issue in the Autorest/typescript repository and link it here)
[ ] Added a changelog (if necessary)

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


Fix docs on older browsers

We are currently not using a build step to compile our es6 code for the docs. This works fine in the most recent browsers, but in some older browsers only a blank page shows up.
This issue is what is preventing us from adding algolia's docsearch, because their crawler that indexes the site uses an older version of firefox.
We might need to introduce a build step for the docs, although it is very convenient to not have one.

Fixed in ade738bbc3169f0ea4a0884e28240dd21f726761


feat(jenkins): support check license with go-licenser

Highlights

Enable the go-licenser stage in the pipeline
Enable the go-licenser goal in the makefile.

Tasks

[ ] Agree whether this is the approach to go
[ ] It requires https://github.com/elastic/go-licenser/issues/24 to support the python headers.
[ ] Makefile and Pipeline could potentially use the same scripts somehow.
[ ] Pre-commit should be adapted to the go-licenser

@v1v can you give a short explanation on the benefits of using go-licenser instead of the relatively simple bash script?
One benefit of the bash script is that we can use it very easily in a pre-commit hook: https://github.com/elastic/apm-agent-python/blob/a2442b364fa2c8ddeddc465f7ce3964ff2841204/.pre-commit-config.yaml#L21-L29
If we switch to go-licenser, it would be great if that could also be applied to the pre-commit hook somehow to ensure parity between the two checks.

That's a fair question.
The major benefit is to provide the same toolchain for all the agents if possible, I don't say the go-licenser is the one to go, but as it's already a tool provided and used, we could potentially extend it for the rest of agents and being able to have the same common toolchains.
At the moment the current implementation doesn't provide all the requirements for the python agent though. That's one of the reasons I raised this PR as a draft. Do you think I should move this conversation to an issue firstly rather than in this PR?
Are there any concerns in your end?
Thanks

I agree that harmonizing the license check would be great. But if it hugely complicates go-licenser just to be able to handle the Python agent (AFAIK, most other Elastic projects share the same Apache2 license), it might not be worth it. And as I said, having the check work in a pre-commit hook is pretty much a requirement for me.

I'll close this PR for the time being, and once we go the go-licenser in place we can revisit it.

zube didn't agree with having this closed. Let's try again


Add link to crystal implementation

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Thanks for your contribution!


Flyway schema creation issue on Oracle 12.2 DB with password policy enabled

Which version and edition of Flyway are you using?
5.0.7, Community Edition
If this is not the latest version, can you reproduce the issue with the latest one as well?
(Many bugs are fixed in newer releases and upgrading will often resolve the issue)
Which client are you using? (Command-line, Java API, Maven plugin, Gradle plugin)
Command-line
Which database are you using (type & version)?
Oracle database <IP_ADDRESS>.0
Which operating system are you using?
Oracle Linux 7.3 (database host)
Windows 10 EE (Flyway command line host)
What did you do?
(Please include the content causing the issue, any relevant configuration settings, the SQL statement that failed (if relevant) and the command you ran.)
Running "flyway migrate" command on an existing Oracle database instance with the password policy enabled (mininum characters 8), and where target schema does not yet exist.
My Flyway configuration file:
Sample Flyway configuration file
flyway.url=
flyway.user=FLYWAY
flyway.table=FLYWAY_SCHEMA_HISTORY
flyway.locations=filesystem:scripts
flyway.schemas=FOODEV
flyway.cleanDisabled=false
What did you expect to see?
Flyway creating the user/schema successfully.
What did you see instead?
Unable to create schema "FOODEV"
SQL State  : 99999
Error Code : 28003
Message    : ORA-28003: password verification for the specified password failed
ORA-20001: Password length less than 8

Why exactly do you feel this is a Flyway issue?

Thanks Axel.
Two reasons:

Oracle database 12.2 version provides a set of functions that you can use to manage the complexity of passwords.
https://docs.oracle.com/en/database/oracle/oracle-database/12.2/dbseg/configuring-authentication.html#GUID-F09749BF-2881-4EE5-B59D-041E4796BAC4

On enterprise level, it is quite common to use these, or customize them as per the organization security requirements.

Not sure if I'm looking at the correct source code file, but I feel it is hardcoded to "flyway". In my opinion, it should be at least configurable parameter.
https://github.com/flyway/flyway/blob/master/flyway-core/src/main/java/org/flywaydb/core/internal/database/oracle/OracleSchema.java

Thanks for the link. The first step we could do, could be moving from flyway to FFllyywwaayy00!! which should be compliant with all predefined policies. This doesn't fully handle the case of custom policies, but it should be sufficient for the vast vast majority of the cases. What do you think?

Thanks Axel for the reply. Yes, more complex default password (like your proposed one) would be good as a quick fix for the issue.


Fixed ApiKeyRequired usage

Fixes #208

Btw. it seems that examples/awsm.json is using old syntax (i.e. apiKeyRequired vs. ApiKeyRequired), I hadn't time to carefully check which parameters should be uppercased.


Add integration tests

Creating virtualenv is a big performance killer in testing.
I am considering to design a new (un)installation interface so that we can get a full mocked (un)installation and get rid of venv.

Whoa, after mocking installation, the time cost is reduced by 60%!

I am not apt to huge PRs, so if the review is done we can move on to merging stuff. No need to have a full test suite.


Prefixed array

Prefixed array and template

Codecov Report

Merging #11 (6b4cde8) into master (d11f4ac) will decrease coverage by 0.16%.
The diff coverage is 80.00%.

@@            Coverage Diff             @@
##           master      #11      +/-   ##
==========================================
- Coverage   96.55%   96.39%   -0.17%
==========================================
  Files          19       19
  Lines         523      527       +4
==========================================
+ Hits          505      508       +3
- Misses         18       19       +1

Impacted Files
Coverage Δ

src/Array.jl
98.24% <80.00%> (-1.76%)
:arrow_down:

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


Add API method to clear and reset config

This PR adds two methods, clear config and reset config.
This is used to change the config at runtime.

All great comments, Iv’e fixed the PR accordingly


Bottom nav issues

Related to #13.
The bottom nav covers the bottom part of the screen, and when scrolling on mobile two scrollbars are visible.

Oops, I might've introduced this via #32 because I made the navbar position: fixed;, but I believe I also added padding to the bottom of the content for mobile screens. I'm not able to reproduce this on Android, so I think this is an iOS Safari bug

Ah, the horizontal scroll bar is because the Zoom link on 2021-08-19 has a long string of text with no spaces, so it can't wrap, and it extends past the edge of the screen. The navbar doesn't cover up the bottom edge of the screen for me though

Yeah, on chrome the padding works properly, but on my phone it just doesn't. How strange!

I will also say that sometimes upon opening inspect the layout gets screwed.

I'll convert this into a separate issue.

Seems like this was a flex problem and was fixed in the PR.

Oh nice, two birds with one stone!


Vulnerability roundup 105: graylog-3.3.9: 2 advisories [9.8]

search, files

[ ] CVE-2021-37759 CVSSv3=9.8 (nixos-21.05)
[ ] CVE-2021-37760 CVSSv3=9.8 (nixos-21.05)

CVE details
CVE-2021-37759
A Session ID leak in the DEBUG log file in Graylog before 4.1.2 allows attackers to escalate privileges (to the access level of the leaked session ID).
CVE-2021-37760
A Session ID leak in the audit log in Graylog before 4.1.2 allows attackers to escalate privileges (to the access level of the leaked session ID).

Scanned versions: nixos-21.05: ad6e733d633.
Cc @fadenb

NixOS 21.05 is EOL, issue for unstable and 21.11 is #133959.