Notebook testing

Description

Add support for testing the notebooks
Improve Dockerfile in terms of layers, and integration with SuperDuper's dependencies
Fix documentation links.

Related Issues

Checklist

[ ] Is this code covered by new or existing unit tests or integration tests?
[ ] Did you run make test successfully?
[ ] Do new classes, functions, methods and parameters all have docstrings?
[ ] Were existing docstrings updated, if necessary?
[ ] Was external documentation updated, if necessary?

Additional Notes or Comments

Codecov Report
Attention: 41 lines in your changes are missing coverage. Please review.

Comparison is base (cbe6b32) 78.22% compared to head (629c1f7) 81.04%.
Report is 52 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1083      +/-   ##
==========================================
+ Coverage   78.22%   81.04%   +2.81%
==========================================
  Files         101       98       -3
  Lines        6351     6404      +53
==========================================
+ Hits         4968     5190     +222
+ Misses       1383     1214     -169

Files
Coverage Δ

superduperdb/__init__.py
100.00% <100.00%> (ø)

superduperdb/__main__.py
71.42% <100.00%> (ø)

superduperdb/db/base/db.py
64.70% <100.00%> (-1.01%)
:arrow_down:

superduperdb/db/filesystem/artifacts.py
75.86% <100.00%> (ø)

superduperdb/container/vector_index.py
85.18% <80.00%> (-0.32%)
:arrow_down:

superduperdb/db/mongodb/data_backend.py
47.14% <33.33%> (-0.62%)
:arrow_down:

superduperdb/db/mongodb/query.py
80.47% <0.00%> (-0.92%)
:arrow_down:

superduperdb/misc/superduper.py
86.58% <64.28%> (-4.60%)
:arrow_down:

superduperdb/container/schema.py
53.48% <11.11%> (-10.41%)
:arrow_down:

superduperdb/container/model.py
82.82% <56.52%> (-1.35%)
:arrow_down:

... and 1 more

... and 11 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


numba fails to compile ray casting function

Reporting a bug

[x] I am using the latest released version of Numba (most recent is visible in
the change log (https://github.com/numba/numba/blob/master/CHANGE_LOG).
[x] I have included below a minimal working reproducer (if you are unsure how
to write one see http://matthewrocklin.com/blog/work/2018/02/28/minimal-bug-reports).

I rely on a numba JIT-compiled function to perform a ray casting operation. The function performs reliably in ~99% of cases, but I have found a case where it fails.
The reproducible example is:
from numba import vectorize, bool_, float64
import numpy as np

# Define points to clip
x_vec = [468559.33, 468559.99]
y_vec = [5016610.22, 5016609.12]

bad_poly = np.array([[468560.495, 5016609.98611111],
                [468560.495, 5016607.625],
                [468558.245, 5016609.875],
                [468557.97700443, 5016610.41099114],
                [468558.13454149, 5016610.51065745],
                [468560.495, 5016609.98611111]])

good_poly = np.array([[468560.495, 5016609.98611111],
                     [468560.495, 5016607.625],
                     [468558.245, 5016609.875],
                     [468558.13454149, 5016610.51065745],
                     [468560.495, 5016609.98611111]])

poly = bad_poly

@vectorize([bool_(float64, float64)])
def ray(x, y):
    n = len(poly)
    inside = False
    p2x = 0.0
    p2y = 0.0
    xints = 0.0
    p1x, p1y = poly[0]
    for i in range(n + 1):
        p2x, p2y = poly[i % n]
        if y > min(p1y, p2y):
            if y <= max(p1y, p2y):
                if x <= max(p1x, p2x):
                    if p1y != p2y:
                        xints = (y - p1y) * (p2x - p1x) / (p2y - p1y) + p1x
                    if p1x == p2x or x <= xints:
                        inside = not inside
        p1x, p1y = p2x, p2y
    return inside

If poly=bad_poly then the function does not compile and the following error is produced
python: /tmp/build/80754af9/llvmdev_1546540969021/work/include/llvm/ADT/APInt.h:1124: bool llvm::APInt::operator==(const llvm::APInt&) const: Assertion `BitWidth == RHS.BitWidth && "Comparison requires equal bit widths"' failed.

If poly=good_poly, then the function compiles. It seems that deleting one or some of the coordinates allows the function to compile.

To make sure we understand your environment completely, can you run numba -s and pasted the output here?

Thanks!  We'll try to reproduce this.

Thank you!
I attempted to update to 0.44 and I receive the same error. This environment is much smaller and achieves the same error so it may help speed up the debugging.
I was able to compile the function on a different computer with no error yesterday, I will try to track down the environment for that as well.

The script passes under this
environment (I downgraded to 0.4.1). I am checking it on a large number of polys in a different script.

Thanks for the report. This is an LLVM issue:
NUMBA_CPU_NAME="ivybridge" NUMBA_CPU_FEATURES="" python issue4135.py

reproduces under Numba 0.44 with LLVM 8 backing. Nehalem chips onward seem to trigger the problem, however, switching on AVX512 seems to make it go away, e.g.:
NUMBA_CPU_NAME="skylake-avx512" NUMBA_CPU_FEATURES="" python issue4135.py

This does not reproduce on mainline for Numba 0.55 (with LLVM 11). Closing assumed fixed.


Adds support for alert presets

Adds support for alert presets to the logdna terraform provider.
Semver: Patch
Ref: LOG-6971

@dm36, are you waiting for the alerts getting deployed into the prod in order to publish this draft PR?

@smusali have tested preset alerts. also have added a file of tests for preset alerts


unban rak

please
hes good at maths, literally the only reason im in this server anymore is to shitpost in #maths
let him shitpost with me

Wow, what a convincing argument

Wow, what a convincing argument


fix #1 #2

fix: #1

feat: Add terminalCodeRunner.executorMapByGlob setting (like execMap, but uses glob as keys) #2

feat: Support vscode variables subtitions in execMap and executorMapByGlob settings

feat: Change default of execMap setting map to use tsx for typescript and node for javascript
BREAKING Default of execMap was changed! Shortcut for js language was removed.

@zardoy can't find a good way to debug, maybe you need manully check the functionality work fine

@zardoy can't find a good way to debug, maybe you need manully check the functionality work fine

@tjx666 Ah, this is super easy, sorry don't have much time to copy contributing.md for instructions over all my extenisons :)
But the launch config to debug extension is the same across all of my extensions, so I'll copy it here.

@zardoy I will add more commit, don't merge it too quick

What formatter are you using? I normaly use prettier, are you using vscode builtin? I want to add some vscode settings to this project

Is vscode support set default value to null?

Is vscode support set default value to null?

Yes, but I think you need to change type following then:
"anyOf": [
    {
        "type": "null"
    },
    {
        "type": "string"
    }
],

Also I added prettier to ci, can you run git rm -f src/generated.ts and then add it to gitignore, it was commited accidentally 😅

ok, all thing I want to add is done, if no any more question, would like to be merged.

Maybe we can let ai generate a extension icon

Maybe we can let ai generate a extension icon

Why not. Lets try this

I want to change the run icon position to https://code.visualstudio.com/api/references/contribution-points#contributes.menus:~:text=1_run - Commands related to running and debugging the editor.
but icon disappear after:
"menus": {
            "editor/title": [
                {
                    "when": "terminal-code-runner.runButton",
                    "group": "1_run",
                    "command": "runFile"
                }
            ]
        }

I don't think so, I'm going to make a few minor improvements & fix ci and will merge


How to edit source code and re-build the application

Say in case i want to change the logo, or the theme of the admin UI

I suggest you to follow this documentation https://strapi.io/documentation/3.x.x/advanced/customize-admin.html#customization
And then to package your app in a container.


vue-tsc CLI rest params runtime error in version 2.0.19 and beyond

Vue - Official extension or vue-tsc version
<EMAIL_ADDRESS>VSCode version
1.91.1
Vue version
3.4.31
TypeScript version
5.4.5
System Info
System:
  OS: macOS 14.5
  CPU: (12) arm64 Apple M3 Pro
  Memory: 83.59 MB / 18.00 GB
  Shell: 5.9 - /bin/zsh
Binaries:
  Node: 20.12.0 - ~/.asdf/installs/nodejs/20.12.0/bin/node
  Yarn: 3.6.3 - ~/.asdf/installs/nodejs/20.12.0/bin/yarn
  npm: 10.5.0 - ~/.asdf/plugins/nodejs/shims/npm
Browsers:
  Safari: 17.5

Steps to reproduce
Unfortunately, I'm not able to track down exactly what in our code is triggering the error, but I can provide some additional context/learnings I've come across. It seems like it might be related to rest parameters:

In<EMAIL_ADDRESS>this type change was made to event handlers.
For some reason, that change started triggering an error from the tsc runtime, which has an open issue and potential fix, but hasn't been resolved yet.
However, vue-tsc was not surfacing runtime errors, so after upgrading to 2.0.19, the type checker was encountering the runtime error above and silently erroring out itself without reporting any type errors. This gave the appearance that it finished successfully, so we didn't catch it right away.
In<EMAIL_ADDRESS>this change was made to surface runtime errors, and so now our type-checking errors out as it should.

What is expected?
vue-tsc successfully runs to completion, reporting errors along the way (if any).
What is actually happening?
vue-tsc encounters a runtime error from tsc and errors out. The stack trace logged to the console is:
TypeError: Cannot read properties of undefined (reading 'flags')
    at getCheckFlags (/Users/patrickhooper/src/mudflap-fleet-web/node_modules/typescript/lib/tsc.js:15948:17)
    at getTypeOfSymbol (/Users/patrickhooper/src/mudflap-fleet-web/node_modules/typescript/lib/tsc.js:53297:24)
    at getParameterCount (/Users/patrickhooper/src/mudflap-fleet-web/node_modules/typescript/lib/tsc.js:73708:24)
    at getRestTypeAtPosition (/Users/patrickhooper/src/mudflap-fleet-web/node_modules/typescript/lib/tsc.js:73679:28)
    at assignContextualParameterTypes (/Users/patrickhooper/src/mudflap-fleet-web/node_modules/typescript/lib/tsc.js:73835:41)
    at contextuallyCheckFunctionExpressionOrObjectLiteralMethod (/Users/patrickhooper/src/mudflap-fleet-web/node_modules/typescript/lib/tsc.js:74560:13)
    at checkFunctionExpressionOrObjectLiteralMethod (/Users/patrickhooper/src/mudflap-fleet-web/node_modules/typescript/lib/tsc.js:74535:5)
    at checkExpressionWorker (/Users/patrickhooper/src/mudflap-fleet-web/node_modules/typescript/lib/tsc.js:76319:16)
    at checkExpression (/Users/patrickhooper/src/mudflap-fleet-web/node_modules/typescript/lib/tsc.js:76221:32)
    at checkExpressionForMutableLocation (/Users/patrickhooper/src/mudflap-fleet-web/node_modules/typescript/lib/tsc.js:75979:18)

Link to minimal reproduction
No response
Any additional comments?
Based on the details provided in the steps to reproduce section, I can't tell if there's actually anything that can be done from a vue-tsc perspective, or if this needs to be solved on the tsc side.
Thank you!!

Thank you for your report and collection of relevant information! https://github.com/vuejs/language-tools/commit/86e38db1a531b1cce24814da4a5c42aef75d987b#diff-93ac10dff6acb9af8511a96c6ef3bdabf647f4ed0a85a9fe1566a0cf320cf44aR157 is to fix the TS type of parameterless event. I have not yet come up with an alternative way of writing virtual code. Unfortunately, we may have to wait for https://github.com/microsoft/TypeScript/pull/58440 to be merged.


Python: FunctionResultContent hash fix to handle lists/sets

Motivation and Context
Currently, the hash function in FunctionResultContent does not properly handle types of list or set. Adding the proper handling to turn these into a tuple, and then the hash function works properly.

Description
This PR:

Makes sure the FunctionResultContent __hash__ method can properly handle mutable types like lists or sets
Adds tests to ensure the new behavior works as expected
Ensures the create message content method can handle lists or sets when generating the content and the model can only accept a string.
Fixes #9977

Contribution Checklist

[X] The code builds clean without any errors or warnings
[X] The PR follows the SK Contribution Guidelines and the pre-submission formatting script raises no violations
[X] All unit tests pass, and I have added new tests where possible
[X] I didn't break anyone :smile:

Python Test Coverage Report • FileStmtsMissCoverMissingsemantic_kernel/agents/open_ai   assistant_content_generation.py1531292%93–96, 114–115, 348–356, 398, 400semantic_kernel/contents   function_result_content.py85298%163–165semantic_kernel/functions   kernel_function_from_prompt.py154795%167–168, 182, 203, 221, 241, 324TOTAL16743186089%
Python Unit Test Overview

Tests
Skipped
Failures
Errors
Time

2963
4 :zzz:
0 :x:
0 :fire:
1m 41s :stopwatch:


Improve documentation and update analysis.json

Fixes #42
Fixes #43
Improve documentation to include hints and user-specific cases about labels, value, placeholder and slots.

This change is

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Review status: 0 of 2 files reviewed at latest revision, 3 unresolved discussions.

vaadin-dropdown-menu.html, line 180 at r1 (raw file):

       *
       * Hint: You are able to change the visual representation of selected value in the input part
       * of the dropdown-menu. To do so please set the `label`s to the vaadin-dropdown-menu options.

It's difficult to read and understand in a first view. I would not use options but vaadin-item instead, options in this sounds like parameters for the dropdown-menu.
I would rephrase as:
By setting the label property of inner vaadin-items  you will be able to change the visual representation of the selected value in the input part.

vaadin-dropdown-menu.html, line 237 at r1 (raw file):

             * Hint: not to select item by default set the value the way it won't match any
             * dropdown-menu option's values or set all values of the options so they won't
             * match the value of dropdown-menu itself.

Also hard to understand, rephrase to something like :
If you do not want to select any item by default, you can either set all the values of inner vaadin-items, or set the vaadin-dropdown-menu value to an inexistent value in the items list.

vaadin-dropdown-menu.html, line 274 at r1 (raw file):

            /**
             * A hint to the user of what can be entered in the control.
             * Placeholder will be displayed only in case no items will be selected by default.

The placeholder will be displayed in the case that there is no item selected, or the selected item has an empty string label, or the selected item has no label and it's DOM content is empty.

Comments from Reviewable

Review status: 0 of 2 files reviewed at latest revision, 3 unresolved discussions.

vaadin-dropdown-menu.html, line 180 at r1 (raw file):
Previously, manolo (Manuel Carrasco Moñino) wrote…
It's difficult to read and understand in a first view. I would not use options but vaadin-item instead, options in this sounds like parameters for the dropdown-menu.
I would rephrase as:
By setting the label property of inner vaadin-items  you will be able to change the visual representation of the selected value in the input part.

Thank for advices! Changed.

vaadin-dropdown-menu.html, line 237 at r1 (raw file):
Previously, manolo (Manuel Carrasco Moñino) wrote…
Also hard to understand, rephrase to something like :
If you do not want to select any item by default, you can either set all the values of inner vaadin-items, or set the vaadin-dropdown-menu value to an inexistent value in the items list.

Thank for advices! Changed.

vaadin-dropdown-menu.html, line 274 at r1 (raw file):
Previously, manolo (Manuel Carrasco Moñino) wrote…
The placeholder will be displayed in the case that there is no item selected, or the selected item has an empty string label, or the selected item has no label and it's DOM content is empty.

Thank for advices! Changed.

Comments from Reviewable

Reviewed 2 of 2 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


#209 - replace webpack with vite

It should close #209

Is this going to work inside container?

It works in container. Just remember to add variables to .env file and restart containers.


Add support for custom shapes

Closes #11
Main API change is a switch of the Shape type from an enum to an interface. With an interface, consumers are free to create whatever shape they want, so long as they can draw it using canvas operations. I also added rectangles and drawables as built-in shape implementations.

Hi @mattprecious, thanks for the PR. I've been holding back adding custom shapes to Konfetti in order to keep it performant. First of all I have to say that I'm really happy with the code that you delivered including the improvements. At first glance it looks great!
Give me some more time to review the code and play with some custom shapes. Main goal is to make sure everything stays performant so I also want to run some tests but it already looks very promising.

Hi @mattprecious, thanks for the PR! I've been holding back adding custom shapes to Konfetti in order to keep it performant. First of all I have to say that I'm really happy with the code that you delivered including the improvements. At first glance it looks great!
Give me some more time to review the code and play with some custom shapes. Main goal is to make sure everything stays performant so I also want to run some tests but looking at the implementation I think it'll be fine! Really cool

Yep, keeping the performance is definitely important. I tried to keep it as lightweight as possible by sticking with the canvas APIs, but let me know if you find anything that needs addressing.
It's definitely possible to build a poorly-performing shape implementation, but I think that's fine as long as the built-in ones work well.

One thing I noticed with this change is that since you can now use shapes that aren't symmetrical both vertically and horizontally, it's very clear that they only flip along the horizontal axis.

One thing I noticed with this change is that since you can now use shapes that aren't symmetrical both vertically and horizontally, it's very clear that they only flip along the horizontal axis.

Just checked that indeed and you're right. I think that's fine for now. Letting people know the rotation effect looks best with symmetrical shapes is good enough for now. Adding this is already a major improvement and requested by a lot of people.

@mattprecious Can I or do you want to rebase the branch on master?

Go for it!


New forums

@carltongibson Did you have something similar in mind?

Sorry @carltongibson, do you think you would be able to have a look at this PR?


Convert to Tailwind CSS

Since I want to try how little the bundle can be.

No, we won't do that.


Memory Leak on FIRApp Config. SDK version - 6.34.0

[REQUIRED] Step 1: Describe your environment

Xcode version: 13.1___
Firebase SDK version: 6.34.0___
Installation method: CocoaPods | Carthage | Zip file | Swift Package Manager (select one) - CocoaPods
Firebase Component: _____ (Auth, Core, Database, Firestore, Messaging, Storage, etc)
Target platform(s): All | iOS | Catalyst | macOS | tvOS | watchOS - iOS

[REQUIRED] Step 2: Describe the problem
Steps to reproduce:
What happened? How can we make the problem occur?
Getting memory leaks on FIRApp Configuration.
If you have a downloadable sample project that reproduces the bug you're reporting, you will
likely receive a faster response on your issue.
Relevant Code:
// TODO(you): code here to reproduce the problem

Hi @pushparajakln, is the issue also reproducible when upgrading the SDK version (as this contains bug fixes)?

Hi @pushparajakln, is the issue also reproducible when upgrading the SDK version (as this contains bug fixes)?

Yes @rizafran. I am using latest version

The latest issue is 8.10.0.  The issue is specifying 6.34.0

Sorry. I thought it is a latest version. We cannot update to latest. Could you please check this issue in 6.34.0

Sorry, we're not likely to prioritize a fix to 6.x.


elastic remote logging

do we have support for elastic search remote logging?
if so is it useful in case of kubernetsPodOperator or do i still need to use persistence volumes?

@ArshiAAkhavan Airflow natively supports Elasicsearch, see apache-airflow-providers-elasticsearch.
I think many users will want to know this, so I raised the issue https://github.com/airflow-helm/charts/issues/368, to add some docs about this in the "How to persist airflow logs?" section.

Lets move discussion to: https://github.com/airflow-helm/charts/issues/368


Backup integration merge main

What will we do with LIskov Substitution Principle?
Maybe delete methods from Entity class and catch NotImplementedError/ AttributeError in all places?


In sqlite-jdbc, update failed when string is too long, without any exception. But in command line and navicat, it's OK

replace into articles(id,grade,no,title,content,newchars,update_time)
values(1394823098212,'1','8','title','LongUtf8String',1614001996557)
If the length of LongUtf8String exceeds 2689 bytes, the operation will fail, without any exception.
In sqlite command line and navicat, it can work.
Table definition:
create table if not exists articles (
id       long not null primary key,
grade    tinyint not null,
no       smallint not null,
title    varchar(255) not null,
content  text not null,
newchars text not null
);
Are there any ways to overcome the limit?
Thanks!!!

It is my fault :(
I limited the length when check parameters, and discard it without prompt.


New environment for oceanography image on SciServer

Updating the environment file for the new oceanography image.
Will keep committing changes to this while we work on updating the package list.

@asiddi24 I haven't tested this environment with the notebooks on Sciserver, did you?
Also, the same tests are failing,  which setting cartopy<0.20 should have fixed. The PR #218  passes all tests. I would combine the additional changes there to ensure the tests pass.

Yes ! I'm almost finished testing it. The tests are passing and all the notebooks are running, except the Kogur one which is taking a bit of time. Once it finishes, I think we're all set then to send the environment to Mitya.

Okay folks, @Mikejmnez @malmans2 , the tests are passing now that we've pinned cartopy<0.20 and I've tested the new environment on Sciserver and ran all the notebooks as well. @Mikejmnez I don't have your LLC4320 notebook, but apart from that all looks great !

You can just test if it is going to work by just creating the od with llc4320:
od = ospy.open_dataset.from_catalog("LLC4320")

if it works, then that is it!

I ran my ECCO notebook. It works, but shucks ! It gave me an error with the LLC4320 dataset.
---------------------------------------------------------------------------
OSError                                   Traceback (most recent call last)
File <timed exec>:1, in <module>

File ~/workspace/Storage/asiddi24/persistent/oceanspy/oceanspy/open_oceandataset.py:141, in from_catalog(name, catalog_url)
    138     mtdt = cat[entry].metadata
    140     # Create ds
--> 141     ds = cat[entry].to_dask()
    142 else:
    143     # Pop args and metadata
    144     args = cat[entry].pop("args")

File ~/miniconda3/envs/Oceanography_test/lib/python3.8/site-packages/intake_xarray/base.py:69, in DataSourceMixin.to_dask(self)
     67 def to_dask(self):
     68     """Return xarray object where variables are dask arrays"""
---> 69     return self.read_chunked()

File ~/miniconda3/envs/Oceanography_test/lib/python3.8/site-packages/intake_xarray/base.py:44, in DataSourceMixin.read_chunked(self)
     42 def read_chunked(self):
     43     """Return xarray object (which will have chunks)"""
---> 44     self._load_metadata()
     45     return self._ds

File ~/miniconda3/envs/Oceanography_test/lib/python3.8/site-packages/intake/source/base.py:236, in DataSourceBase._load_metadata(self)
    234 """load metadata only if needed"""
    235 if self._schema is None:
--> 236     self._schema = self._get_schema()
    237     self.dtype = self._schema.dtype
    238     self.shape = self._schema.shape

File ~/miniconda3/envs/Oceanography_test/lib/python3.8/site-packages/intake_xarray/base.py:18, in DataSourceMixin._get_schema(self)
     15 self.urlpath = self._get_cache(self.urlpath)[0]
     17 if self._ds is None:
---> 18     self._open_dataset()
     20     metadata = {
     21         'dims': dict(self._ds.dims),
     22         'data_vars': {k: list(self._ds[k].coords)
     23                       for k in self._ds.data_vars.keys()},
     24         'coords': tuple(self._ds.coords.keys()),
     25     }
     26     if getattr(self, 'on_server', False):

File ~/miniconda3/envs/Oceanography_test/lib/python3.8/site-packages/intake_xarray/xzarr.py:44, in ZarrSource._open_dataset(self)
     42     kw.setdefault("backend_kwargs", {})["storage_options"] = self.storage_options
     43 if isinstance(self.urlpath, list) or "*" in self.urlpath:
---> 44     self._ds = xr.open_mfdataset(self.urlpath, **kw)
     45 else:
     46     self._ds = xr.open_dataset(self.urlpath, **kw)

File ~/miniconda3/envs/Oceanography_test/lib/python3.8/site-packages/xarray/backends/api.py:873, in open_mfdataset(paths, chunks, concat_dim, compat, preprocess, engine, data_vars, coords, combine, parallel, join, attrs_file, combine_attrs, **kwargs)
    870     paths = [os.fspath(p) if isinstance(p, os.PathLike) else p for p in paths]
    872 if not paths:
--> 873     raise OSError("no files to open")
    875 if combine == "nested":
    876     if isinstance(concat_dim, (str, DataArray)) or concat_dim is None:

OSError: no files to open

The version of intake-xarray is 0.5.0+19.g77bd753

@asiddi24 it looks like it didn't find any files. Are you running it from the Poseidon container?

Aha ! of course. I was running it all in Ocean Circulation. Works like a charm now ! Yay ! lol, I'm having so much fun with this. Time to sleep now.
Okay, we're all set.

@asiddi24 Almost good to go. I think you forgot to use the stable release of xmitgcm

awesome! Lets wait until Ali (successfully) runs the notebooks on Sciserver before merging. @asiddi24 just let us know!


cleanup TSRMLS usage

This macro are only used in PHP 5
Still present in PHP 7 (for BC)
Dropped in PHP 8
So seems useful to cleanup to prepare to PHP 8

Notice: build with PHP 8.0.0-dev is ok, but not really the goal of this PR, and not tested

Coverage increased (+0.03%) to 74.401% when pulling 9d0fb467a9962a4f210e9f4b56097f04cd742ad7 on remicollet:issue-tsrmls into 5f76f838e5d44ca951a44bec6be1af05fae4bd93 on microsoft:master.

Resolved conflicts and merged. See #1130


runs-table should make numerous paginated queries instead of one large query

The home page of the IPS portal will try to load all of the events at once. In the current production environment, we have to load all 40,000+ events before the page will react to user inputs. We should instead paginate our queries:

Provide a "page" argument (number, 1-indexed based)
Provide a "pageSize" argument (number, positive)
Provide a "sort" column (we may be able to provide multiple sort parameters, will have to check to see if the DataTable API supports this.)

Completed in https://github.com/HPC-SimTools/IPS-portal/commit/f3a7b0297f0c84d30fe87c0dddf00ee99d9d80e8 - added new API endpoint which follows the specification laid out at https://datatables.net/manual/server-side


refactor: class to edit message_context

Description
AssignContextReader is a class that inherits from ArchitectureReader, but it implements almost none of ArchitectureReader's functions, making the code difficult to understand.
Delete the AssignContextReader class and change the implementation to be more in line with the actual situation.
Related links
https://tier4.atlassian.net/browse/RT2-1845
Notes for reviewers

Pre-review checklist for the PR author

[x] I've confirmed the contribution guidelines.
[x] The PR is ready for review.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR has been properly tested.
[ ] The PR has been reviewed.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] (Optional) The PR has been properly tested with CARET_report verification.
[ ] There are no open discussions or they are tracked via tickets.
[ ] The PR is ready for merge.

After all checkboxes are checked, anyone who has write access can merge the PR.

@xygyo77
Could you create a test for the ContextUpdater class?

@xygyo77 Could you create a test for the ContextUpdater class?

Since it is an alternative to the AssignContextReader class, I assume it is covered in caret_analyze/src/test/architecture/test_architecture.py.

test_update_message_context()
test_remove_variable_passing()

@xygyo77 Could you create a test for the ContextUpdater class?

Since it is an alternative to the AssignContextReader class, I assume it is covered in caret_analyze/src/test/architecture/test_architecture.py.

test_update_message_context()
test_remove_variable_passing()

The above test case was able to test for this change. It seems that we don't need to add any additional tests. Thanks.


Replication lag metric seems to be broken in 1.19.1

After updating from 1.19.0 to 1.19.1 I have constant alarms for cnpg_pg_replication_lag metric for all my db clusters. Restarting cluster resets metric back to zero and then it starts to grow again:

and unless I'm misreading this - status shows cluster in a perfectly synchronized state:
kubectl cnpg status account-postgres
Cluster Summary
Name:               account-postgres
Namespace:          qa
System ID:<PHONE_NUMBER>327348754
PostgreSQL Image:   ghcr.io/cloudnative-pg/postgresql:14.3
Primary instance:   account-postgres-1
Status:             Cluster in healthy state
Instances:          2
Ready instances:    2
Current Write LSN:  2/1A000000 (Timeline: 9 - WAL File:<PHONE_NUMBER>00000200000019)

Certificates Status
Certificate Name              Expiration Date                Days Left Until Expiration
----------------              ---------------                --------------------------
account-postgres-ca           2023-06-02 21:10:31 +0000 UTC  68.25
account-postgres-replication  2023-06-02 21:10:31 +0000 UTC  68.25
account-postgres-server       2023-06-02 21:10:31 +0000 UTC  68.25

Continuous Backup status
First Point of Recoverability:  2023-02-24T00:00:09Z
Working WAL archiving:          OK
WALs waiting to be archived:    0
Last Archived WAL:<PHONE_NUMBER>00000200000019   @   2023-03-26T00:05:11.121184Z
Last Failed WAL:                -

Streaming Replication status
Name                Sent LSN    Write LSN   Flush LSN   Replay LSN  Write Lag  Flush Lag  Replay Lag  State      Sync State  Sync Priority
----                --------    ---------   ---------   ----------  ---------  ---------  ----------  -----      ----------  -------------
account-postgres-2  2/1A000000  2/1A000000  2/1A000000  2/1A000000  00:00:00   00:00:00   00:00:00    streaming  async       0

Unmanaged Replication Slot Status
No unmanaged replication slots found

Instances status
Name                Database Size  Current LSN  Replication role  Status  QoS         Manager Version  Node
----                -------------  -----------  ----------------  ------  ---         ---------------  ----
account-postgres-1  34 MB          2/1A000000   Primary           OK      BestEffort  1.19.1           ip-10-20-26-37.us-west-1.compute.internal
account-postgres-2  33 MB          2/1A000000   Standby (async)   OK      BestEffort  1.19.1           ip-10-20-26-37.us-west-1.compute.internal

I'm happy to provide any extra information if needed.

The default lag metric is based on the difference from the last visible commit to the current time. It looks like your cluster is idle, so the metric is unreliable.

@mnencia Well yes, it's dev/qa clusters with no activity when nothing is being tested. Though similar situation can happen in prod as well, if service is using db seldom and not every second, like storing some data which is retrieved once a while.
I guess if is an expected behavior I'll just need to remove that specific alert rule.
But I don't understand why alarms started raising only after 1.19.1 update: was is broken before and fixed in this release or vice versa? Nothing in changelog seems to be related.

That's a good question. The only change I remember in the monitoring code is https://github.com/cloudnative-pg/cloudnative-pg/pull/1666, but I do not see how it could lead to the behavior you see.

Do you use helm chart @tbondarchuk? I seed diff when going to 1.17.0 (1.19.0) to 1.17.1 (1.19.1)

I'm seeing the exact same behavior, but to be fair there is not much load on my cluster either.

@mnencia I was able to track this to following chart update - replication lag query used to run on primary.
Not sure what was the reason to remove primary flag, but judging by this and this it seems to me that replication lag query has to run on primary? I'm not familiar with postgres replication internals but I believe if alarm is on for idle cluster which looks perfectly replicated otherwise - it must be wrong.
for now I'll simply add primary: true back to monitoring configmap but could it be reverted back in chart values? I'd rather avoid adding whole monitoring block to values override in my gitops repo.

@mnencia I think this it's fixed right? have you tried the latest dev version @tbondarchuk  ?

I'm using cloudnative-pg 1.20.2 released 3 weeks ago according to github, and seeing the very same behaviour.
Looking at https://github.com/cloudnative-pg/artifacts/blob/main/manifests/operator-manifest.yaml or the helm chart, I don't see the primary: true back, is that the correct fix ?

Does "primary: true" mean that the alarm is only used on the primary?
If so, thats imho not the solution as replication lag on the primary should always be 0, or not?

Does "primary: true" mean that the alarm is only used on the primary? If so, thats imho not the solution as replication lag on the primary should always be 0, or not?

Doesnt the stat track replication lag to replicas, so yes this is the solution?
Really hoping so because until finding this today ive spent the last few months staring at databases that all appear to be clones of eachother, getting told theyre not replicating, and questioning reality :D

Yep that was it, deployed and resolved now.

Created a pull request for this that includes the solution

The query is supposed to work both on the primary and the standby. We should not use primary: true. The SQL statement contains a CASE that returns 0 if the instance is not in recovery (ergo, it is a primary, not a standby).
I tried running the query on both a standby and a replica, and it works:
$ kubectl exec -ti pg16-1 -c postgres -- psql
psql (16beta3 (Debian 16~beta3-1.pgdg110+2))
Type "help" for help.

postgres=# SELECT CASE WHEN NOT pg_catalog.pg_is_in_recovery()
              THEN 0
              ELSE GREATEST (0,
                EXTRACT(EPOCH FROM (now() - pg_catalog.pg_last_xact_replay_timestamp())))
              END AS lag,
              pg_catalog.pg_is_in_recovery() AS in_recovery,
              EXISTS (TABLE pg_stat_wal_receiver) AS is_wal_receiver_up,
              (SELECT count(*) FROM pg_stat_replication) AS streaming_replicas;
 lag | in_recovery | is_wal_receiver_up | streaming_replicas
-----+-------------+--------------------+--------------------
   0 | f           | f                  |                  2
(1 row)

$ kubectl exec -ti pg16-2 -c postgres -- psql
psql (16beta3 (Debian 16~beta3-1.pgdg110+2))
Type "help" for help.

postgres=# SELECT CASE WHEN NOT pg_catalog.pg_is_in_recovery()
              THEN 0
              ELSE GREATEST (0,
                EXTRACT(EPOCH FROM (now() - pg_catalog.pg_last_xact_replay_timestamp())))
              END AS lag,
              pg_catalog.pg_is_in_recovery() AS in_recovery,
              EXISTS (TABLE pg_stat_wal_receiver) AS is_wal_receiver_up,
              (SELECT count(*) FROM pg_stat_replication) AS streaming_replicas;
     lag      | in_recovery | is_wal_receiver_up | streaming_replicas
--------------+-------------+--------------------+--------------------
 16715.884655 | t           | t                  |                  0
(1 row)

In my opinion, we should close this issue, without a patch.

But then this raise an alarm if we use https://github.com/cloudnative-pg/cloudnative-pg/blob/e133fd93d884f9a1fd2ec048a676988274c8b59a/docs/src/samples/monitoring/prometheusrule.yaml#L41C9-L41C32;
Should we create another issue for addressing that ?

I am actually about to propose a patch.

I think the query should look like this:
SELECT CASE WHEN (
                NOT pg_is_in_recovery()
                OR pg_last_wal_receive_lsn() = pg_last_wal_replay_lsn())
              THEN 0
              ELSE GREATEST (0,
                EXTRACT(EPOCH FROM (now() - pg_last_xact_replay_timestamp())))
              END AS lag,
              pg_is_in_recovery() AS in_recovery,
              EXISTS (TABLE pg_stat_wal_receiver) AS is_wal_receiver_up,
              (SELECT count(*) FROM pg_stat_replication) AS streaming_replicas

Tested on my cluster, it works ;)

I updated only the documentation stuff. I need to update also the configmap.


Race condition setting up/tearing down test couch databases in unit tests

Impact of the bug
WMCore (unit tests)
Describe the bug
As reported in this PR: https://github.com/dmwm/WMCore/pull/11177 , these [1] 3 unit tests seem to be eventually failing in our CI setup.
Looking at the Jenkins report for one of those tests, this [2] is the error I see. Apparently a race condition between creating our couch test databases and deleting them at the end of each unit test. The setUp method is executed before each unit test, while the tearDown is executed at the end.
How to reproduce it
Not sure we can reproduce it in the docker image. But we might try running all the unit tests for that module in one go.
Expected behavior
Unit tests should not be affected by setup/cleanup. In practice, we might want to create test databases based on the unit test name, thus avoiding any possible clash.
Additional context and error message
[1]
    WMCore_t.WMSpec_t.StdSpecs_t.Resubmission_t.ResubmissionTests:testCustomSimpleTaskChainACDC changed from success to error
    WMCore_t.WorkQueue_t.WorkQueue_t.WorkQueueTest:testProductionMultiQueue changed from success to failure
    WMCore_t.WorkQueue_t.WorkQueue_t.WorkQueueTest:testThrottling changed from success to failure

[2]
WMCore_t.WMSpec_t.StdSpecs_t.Resubmission_t.ResubmissionTests.testStandardTaskChainACDC (from nosetests)

  File "/build/cmsbld/jenkins/workspace/DMWM-WMCorePy3-PR-unittests/SLICE/1/label/cms-dmwm-cc7/deploy/2.0.4/sw/slc7_amd64_gcc630/external/python3/3.8.2-comp/lib/python3.8/unittest/case.py", line 60, in testPartExecutor
    yield
  File "/build/cmsbld/jenkins/workspace/DMWM-WMCorePy3-PR-unittests/SLICE/1/label/cms-dmwm-cc7/deploy/2.0.4/sw/slc7_amd64_gcc630/external/python3/3.8.2-comp/lib/python3.8/unittest/case.py", line 672, in run
    self._callSetUp()
  File "/build/cmsbld/jenkins/workspace/DMWM-WMCorePy3-PR-unittests/SLICE/1/label/cms-dmwm-cc7/deploy/2.0.4/sw/slc7_amd64_gcc630/external/python3/3.8.2-comp/lib/python3.8/unittest/case.py", line 630, in _callSetUp
    self.setUp()
  File "/build/cmsbld/jenkins/workspace/DMWM-WMCorePy3-PR-unittests/SLICE/1/label/cms-dmwm-cc7/code/test/python/WMCore_t/WMSpec_t/StdSpecs_t/Resubmission_t.py", line 57, in setUp
    self.testInit.setupCouch("resubmission_t", "ConfigCache")
  File "/build/cmsbld/jenkins/workspace/DMWM-WMCorePy3-PR-unittests/SLICE/1/label/cms-dmwm-cc7/code/src/python/WMQuality/TestInitCouchApp.py", line 104, in setupCouch
    self.couch.create(dropExistingDb=self.dropExistingDb)
  File "/build/cmsbld/jenkins/workspace/DMWM-WMCorePy3-PR-unittests/SLICE/1/label/cms-dmwm-cc7/code/src/python/WMQuality/TestInitCouchApp.py", line 55, in create
    self.couchServer.createDatabase(self.dbName)
  File "/build/cmsbld/jenkins/workspace/DMWM-WMCorePy3-PR-unittests/SLICE/1/label/cms-dmwm-cc7/code/src/python/WMCore/Database/CMSCouch.py", line 1009, in createDatabase
    self.put("/%s" % urllib.parse.quote_plus(dbname))
  File "/build/cmsbld/jenkins/workspace/DMWM-WMCorePy3-PR-unittests/SLICE/1/label/cms-dmwm-cc7/code/src/python/WMCore/Services/Requests.py", line 150, in put
    return self.makeRequest(uri, data, 'PUT', incoming_headers,
  File "/build/cmsbld/jenkins/workspace/DMWM-WMCorePy3-PR-unittests/SLICE/1/label/cms-dmwm-cc7/code/src/python/WMCore/Database/CMSCouch.py", line 135, in makeRequest
    self.checkForCouchError(getattr(e, "status", None),
  File "/build/cmsbld/jenkins/workspace/DMWM-WMCorePy3-PR-unittests/SLICE/1/label/cms-dmwm-cc7/code/src/python/WMCore/Database/CMSCouch.py", line 165, in checkForCouchError
    raise CouchPreconditionFailedError(reason, data, result)
CouchPreconditionFailedError - reason: Precondition Failed, data: {} result: b'{"error":"file_exists","reason":"The database could not be created, the file already exists."}\n'

We have continuous unstable failures with the unit tests caused by this issue and I think it's time to get it resolved. So moving it under Q2, High prio and Operations.


Can't run multiple queries in psql

When doing SQHExecute or SQHExecuteFile with multiple select statements, psql only returns results for the first query. This is a limitation in how psql itself handles the "-c" argument on the command line and the solution is to pass in commands with an echo and a pipe, a la echo "select * from user; select * from person;" | psql ... . I am currently working on a solution and should have this fixed shortly

Great stuff, I don't use postgres so I'm grateful for any help I get from actual users :D

Should be fixed in #41

https://github.com/joereynolds/SQHell.vim/commit/82827da8d85c0c1ede2ff4f0e5819f8357a634df


[incubator][VC] access to tenant own master

add flag to provide root ca for tenant certicate signing
Signed-off-by: zhuangqh<EMAIL_ADDRESS>releated to #250
test pass

should mount an API token into pods

not a required feature. closing....


Consider on-fail clause for while statement in all the node visitors

We haven't seem to consider on-fail clause for while statement in the visitors. Need to add that. #34857 is an issue due to this.

Fixed with #35306


add KT Engine

SteamDB app page links to a few games using this

Bet on Soldier: Blood Sport
FIA European Truck Racing Championship
FlatOut 4: Total Insanity
RiMS Racing
Test Drive Unlimited Solar Crown
The Cursed Crusade
TT Isle of Man
TT Isle of Man: Ride on the Edge 2
TT Isle of Man: Ride on the Edge 3
V-Rally 4
WRC 10: FIA World Rally Championship
WRC 5: FIA World Rally Championship
WRC 6: FIA World Rally Championship
WRC 7: FIA World Rally Championship
WRC 8: FIA World Rally Championship
WRC 9: FIA World Rally Championship
WRC Generations

Brief explanation of the change

After running changed rules in this pull request (this bot only tests rules.ini changes):
Changed technologies: Engine.KT_Engine
These apps now match: (24)<PHONE_NUMBER>70<PHONE_NUMBER>20<PHONE_NUMBER>60<PHONE_NUMBER>20<PHONE_NUMBER>30<PHONE_NUMBER>00<PHONE_NUMBER>750 1082180 1249970 1267540 1346010 1462810 1635280 1642150 1924170 1953520 2911710
Please confirm that all these changed apps are correct.


Remove python3Packages.twentemilieu package

Project description
I'm the author of the source of the package, considering the way other requests have been handled by this project. I don't want to be associated with the project in any way.
Therefore requesting removal of packages associated with me, and upstream written by me.
Please remove: python3Packages.twentemilieu

Locked for now. Please see https://github.com/NixOS/nixpkgs/issues/126677#issuecomment-860071285.

I'm closing this request, as we seem to have reached an understanding.


AttributeError: 'NamedElementOnion' object has no attribute 'middleware'

Hello! Whenever I try using get_market_module, get_nft_module and get_token_module I get AttributeError: 'NamedElementOnion' object has no attribute 'middleware'. Would someone kindly help solve this issue? Thanks!

Which python package manager are you using? I fixed this by specifying web3==5.28.0 in my pip requirements.txt.

Thanks I will look into this and check it out!


Model and Inference code for DFFD dataset

Hi! Can you please give me the code and model for the DFFD dataset? Do you have a model trained separately on Group B dataset? Or are they combined? If you have one trained only on Group B, that would be more helpful, but if you have trained on the whole DFFD dataset, that would work too. My email<EMAIL_ADDRESS>Thank you.

The pre-trained model on the whole DFFD dataset has been send to you.


Set up private Tangle for functional testing

Many of PyOTA's unit test rely on heavy use of mocks, which reduces the value of the test coverage significantly.
A more robust approach would be to set up automated functional tests, on a private Tangle.  Per conversation with Paul H on Slack, we should be able to set this up without too much difficulty:

Clone the repo.
Check out the latest public-testnet-* branch.
Modify the snapshot file.
Disable snapshot signature validation.
mvn clean compile package
Run the node with --testnet flag.
???
Profit!!!

Set up a proof of concept with a private Tangle and a script that transfers IOTAs between two addresses successfully.  We will iterate on it from there and create a comprehensive functional test suite 😺

The min weight magnitude is set here:
https://github.com/iotaledger/iri/blob/testnet/src/main/java/com/iota/iri/TransactionValidator.java#L19
It would be nice if we could ratchet this down for functional tests, but it may be tricky to modify this cleanly.

This can be done via Docker containers (e.g. https://docs.travis-ci.com/user/docker/).
Would that be a feasible approach?

Aye, I think that would be a good way to go about it.  I was talking with the folks at Travis CI; here's the outcome of the conversation:

(optimisation) Use build stages so that the unit tests run first, and then the functional tests run only if the unit tests passed.
Add services: docker to .travis.yml (https://docs.travis-ci.com/user/docker/).
Add a before_install step that downloads+modifies the IRI source and builds the Docker image.
Add to script:

Use docker command to launch 3 Docker containers using the image built in the previous  step.
Run functional tests.

So does the research advanced to some point ?

Hey Huntal.  Yeah, this guy's been pending for a really long time ):
I'm actually working on something similar at my job right now ("Instant Testnet:  Just add Docker!"), so I might be able to make some real progress on this once I hammer out all of the kinks.
That said, if anyone wants to take this on, just let me know; I'd be happy to collaborate!

I'll be happy to work with you on this project but i may think i'm less strong than you in coding master PyIOTA ;)

Thanks Hystenal!
There's a few facets to this issue that we need help with at the moment:

Dockerfile for launching an IRI node (mainnet or testnet, doesn't matter; we just need a template to work off of).
Proof-of-concept for starting up a testnet IRI node with a custom snapshot.txt file (see OP).  Bonus points if the process is scriptable.
This issue has been open through many IRI releases, so it would also be super helpful if someone could ping the IRI devs (e.g., on Discord) and double-check what's required to run an IRI node in testnet mode with a custom snapshot.txt.

Hey todofixthis,
I'll be happy to contribute to the Docker part. I've been already working on it and I have set it on a private GitHub repo.
You can contact me on both IOTA discord under the name Hystenal.

Here my repo for running an iota full node IRI with static neighbors to set.

@huntal put together an awesome proof of concept here:
https://github.com/huntal/testnetiota_dockerized

Any update about that? Is there any help I can provide to set up a private tangle?

Hi @jeamick this issue is still open; we're looking for someone to take the lead and submit a PR.


powerpc_ps added

Could be beefed up more, will be adding more towards ppc_ps lifting in the future. Also, seems to be auto selecting ppc_ps instead of ppc for the target architecture; may wanna swap that default.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

verified on 4.0

Thank you for the PR. I made a few changes and the result is on a new branch: https://github.com/Vector35/arch-ppc/tree/add_paired_singles.
It includes your commits up to (and including) Feb 5th, but not afterwards. I'd merge it into master now, but we're on code freeze with the next release imminent.
In the future, please create a branch and PR just that branch.  👍

Awesome, thank you Andrew! There are some edits to it that may be necessary, I may PR your branch for those, for instance there were some compat stuff added for windows that I wanted to cite the sources for.
Also, I did add the stuff afterwards for the other PPC extensions. Would be cool to see those in, but not too important. I'll see if I can put that into that new PR.


browsers-devices.md: escape what we read.

Because it introduces an JS error locally, and TBH I find this change useless myself.

Because it introduces a JS error locally, and TBH I find this change useless myself.

I don't see an error on my local dev for master. And if you find it useless, open an issue or PR to discuss that again—that change is unrelated to this PR.

It does introduce a JS error. Try clicking on the themes navbar link for example.

ReferenceError: ga is not defined[Learn More]

Split the commits myself—see cross-referenced PRs at #28488 and #28489.


Question: React Native upgrade 0.61.5: Flipper exception

Hello,
I upgraded React Native to 0.61.5, the iOS and Android parts compile successfully. The iOS app launch also successfully but not the Android app.
The Android app launch and stuck on splash screen with no error so I activated logs adb logcat and I found an error about Flipper:
System.err: java.lang.ClassNotFoundException: com.facebook.flipper.ReactNativeFlipper
After 2 day of try I have no idea how to resolve this issue. I have follow all migration procedure for React Native 0.6+ and all packages. This is where I am:
package.json
    "dependencies": {
        "@fortawesome/fontawesome-svg-core": "^1.2.25",
        "@fortawesome/pro-light-svg-icons": "^5.11.2",
        "@fortawesome/pro-solid-svg-icons": "^5.11.2",
        "@fortawesome/react-native-fontawesome": "^0.1.0",
        "@react-native-community/async-storage": "^1.6.3",
        "@react-native-community/netinfo": "^4.6.1",
        "@react-native-firebase/analytics": "^6.1.0",
        "@react-native-firebase/app": "^6.1.0",
        "bugsnag-react-native": "^2.23.2",
        "react": "16.9.0",
        "react-native": "0.61.5",
        "react-native-config": "^0.11.7",
        "react-native-dark-mode": "^0.2.1",
        "react-native-datepicker": "^1.7.2",
        "react-native-device-info": "^5.3.1",
        "react-native-gesture-handler": "^1.5.2",
        "react-native-iphone-x-helper": "^1.2.1",
        "react-native-keychain": "^4.0.1",
        "react-native-picker-select": "^6.3.3",
        "react-native-reanimated": "^1.4.0",
        "react-native-screens": "^2.0.0-alpha.17",
        "react-native-svg": "^9.13.3",
        "react-navigation": "^4.0.10",
        "react-navigation-drawer": "^2.3.3",
        "react-navigation-stack": "^1.10.3",
        "react-redux": "^7.1.1",
        "redux": "^4.0.1",
        "redux-logger": "^3.0.6",
        "redux-thunk": "^2.3.0"
    },
    "devDependencies": {
        "@babel/core": "^7.6.2",
        "@babel/preset-env": "^7.2.0",
        "@babel/runtime": "^7.6.2",
        "@types/enzyme": "^3.1.15",
        "@types/enzyme-adapter-react-16": "^1.0.3",
        "@types/jest": "^24.0.12",
        "@types/node": "^12.11.5",
        "@types/react": "^16.8.16",
        "@types/react-native": "^0.60.23",
        "@types/react-native-datepicker": "^1.7.0",
        "@types/react-native-vector-icons": "^6.4.4",
        "@types/react-navigation": "^3.4.0",
        "@types/react-redux": "^7.0.8",
        "@types/react-test-renderer": "^16.8.1",
        "@types/redux-logger": "^3.0.6",
        "@types/sinon": "^7.0.2",
        "babel-jest": "^24.8.0",
        "babel-plugin-transform-remove-console": "^6.9.4",
        "enzyme": "^3.7.0",
        "enzyme-adapter-react-16": "^1.7.0",
        "enzyme-to-json": "^3.3.5",
        "jest": "^24.8.0",
        "jest-fetch-mock": "^2.0.1",
        "metro-react-native-babel-preset": "^0.57.0",
        "pre-commit": "^1.2.2",
        "react-dom": "^16.6.3",
        "react-test-renderer": "^16.12.0",
        "sinon": "^7.2.2",
        "snapshot-diff": "^0.4.1",
        "ts-jest": "^24.0.2",
        "tslint": "^5.17.0",
        "typescript": "^3.7.3"
    },

android/app/build.gradle
dependencies {
    implementation fileTree(dir: "libs", include: ["*.jar"])
    implementation "com.facebook.react:react-native:+"  // From node_modules
    implementation 'androidx.appcompat:appcompat:1.1.0-rc01'
    implementation 'androidx.swiperefreshlayout:swiperefreshlayout:1.1.0-alpha02'

    if (enableHermes) {
        def hermesPath = "../../node_modules/hermes-engine/android/";
        debugImplementation files(hermesPath + "hermes-debug.aar")
        releaseImplementation files(hermesPath + "hermes-release.aar")
    } else {
        implementation jscFlavor
    }
}

configurations.all {
     resolutionStrategy {
       force "com.facebook.soloader:soloader:0.8.0"
     }
}

// Run this once to be able to run the application with BUCK
// puts all compile dependencies into folder libs for BUCK to use
task copyDownloadableDepsToLibs(type: Copy) {
    from configurations.compile
    into "libs"
}

apply from: file("../../node_modules/@react-native-community/cli-platform-android/native_modules.gradle"); applyNativeModulesAppBuildGradle(project)

apply plugin: "com.google.gms.google-services"

MainActivity.java
package com.dedge.centralinventory;

import android.os.Bundle;
import com.facebook.react.ReactFragmentActivity;
import com.facebook.react.ReactActivityDelegate;
import com.facebook.react.ReactRootView;
import com.swmansion.gesturehandler.react.RNGestureHandlerEnabledRootView;

public class MainActivity extends ReactFragmentActivity {

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(null);
    }

    @Override
    protected ReactActivityDelegate createReactActivityDelegate() {
        return new ReactActivityDelegate(this, getMainComponentName()) {
            @Override
            protected ReactRootView createRootView() {
                return new RNGestureHandlerEnabledRootView(MainActivity.this);
            }
        };
    }
}

MainApplication.java
package com.dedge.centralinventory;

import android.app.Application;
import android.content.Context;
import com.facebook.react.PackageList;
import com.facebook.react.ReactApplication;
import com.facebook.react.ReactNativeHost;
import com.facebook.react.ReactPackage;
import com.facebook.soloader.SoLoader;

import java.lang.reflect.InvocationTargetException;
import java.util.List;

public class MainApplication extends Application implements ReactApplication {

    private final ReactNativeHost mReactNativeHost = new ReactNativeHost(this) {
        @Override
        public boolean getUseDeveloperSupport() {
            return BuildConfig.DEBUG;
        }

        @Override
        protected List<ReactPackage> getPackages() {
            @SuppressWarnings("UnnecessaryLocalVariable")
            List<ReactPackage> packages = new PackageList(this).getPackages();
            // Packages that cannot be autolinked yet can be added manually here, for example:
            // packages.add();
            return packages;
        }

        @Override
        protected String getJSMainModuleName() {
            return "index";
        }
    };

    @Override
    public ReactNativeHost getReactNativeHost() {
        return mReactNativeHost;
    }

    @Override
    public void onCreate() {
        super.onCreate();
        SoLoader.init(this, /* native exopackage */ false);
        initializeFlipper(this); // Remove this line if you don't want Flipper enabled
    }

    /**
     * Loads Flipper in React Native templates.
     *
     * @param context
     */
    private static void initializeFlipper(Context context) {
        if (BuildConfig.DEBUG) {
            try {
                /*
                 * We use reflection here to pick up the class that initializes Flipper, since
                 * Flipper library is not available in release mode
                 */
                Class<?> aClass = Class.forName("com.facebook.flipper.ReactNativeFlipper");
                aClass.getMethod("initializeFlipper", Context.class).invoke(null, context);
            } catch (ClassNotFoundException e) {
                e.printStackTrace();
            } catch (NoSuchMethodException e) {
                e.printStackTrace();
            } catch (IllegalAccessException e) {
                e.printStackTrace();
            } catch (InvocationTargetException e) {
                e.printStackTrace();
            }
        }
    }
}

I tried to remove this line initializeFlipper(this); but it did not change anything.
If anyone of you tell me what could be the origin of the problem I will be grateful 🙏🏻

I'm having the same issue. Did you find any solution?

Hello @luiza-avelino
I was two issues that stuck my app on splash screen.

First issue was because of Flipper exception, I removed initializeFlipper(this); in MainApplication.java (don't worry it's an optional feature of RN).
Second issue was because when I removed react-native-splash-screen from my app I miss to remove the override of onCreate() method in MainApplication.java on Android side that stuck my app on splash screen.

The advice I can give you is 1/ to remove the usage of Flipper initializeFlipper(this); 2/ to empty all the cache of React Native 3/ to verify well what you modified in your app (if the error appeared during the update of React Native as was the case for me).
Good luck!


Better module doc within Vim

When searching for a module, generate list of introduced classes and properties to add to the user module documentation.
So from within vim, when entering :Nit opts, it shows:
# opts

Management of options on the command line

## Introduced classes
* Option: Super class of all option's class
* OptionArray: An option with an array as parameter
* OptionBool: A boolean option, `true` when present, `false` if not
* OptionContext: Context where the options process
* OptionCount: A count option. Count the number of time this option is present
* OptionEnum: An option to choose from an enumeration
* OptionFloat: An option with a Float as parameter
* OptionInt: An option with an Int as parameter
* OptionParameter: Option with one parameter (mandatory by default)
* OptionString: An option with a `String` as parameter
* OptionText: Not really an option. Just add a line of text when displaying the usage

## Introduced properties
+ VALUE  # Type of the value of the option
+ add_aliases(names: String...)  # Add new aliases for this option
+ default_value: VALUE  # Default value of this option
+ default_value=(default_value: VALUE)  # Default value of this option
+ errors: Array[String]  # Gathering errors during parsing
~ errors=(errors: Array[String])  # Gathering errors during parsing
+ helptext: String  # Human readable description of the option
~ helptext=(helptext: String)  # Human readable description of the option
+ hidden: Bool  # Is this option hidden from `usage`?
+ hidden=(hidden: Bool)  # Is this option hidden from `usage`?
+ init(help: String, default: VALUE, names: nullable Array[String])  # Create a new option
[...]

The list of properties can be long, but it is useful to search by keywords.
As usual, use :pc to close the preview window showing the doc.

+1

:shipit:


Packer ARM Succeeded - Now What?

So the packer script succeeded after assigning the application I created with the Owner role to the Subcription object. I see the blob in my storage account but now I'm stuck at figuring out how we actually hydrate that image into a running VM? There doesn't seem to be any functionality in the Azure portal to do that. Could you please advise on the recommended steps?

The latest Azure portal does not currently provide too much if you just want to deploy and go..  :frowning:  All of the legos are exposed, and you need (get?) to build them the way you want.  We are working on making improving the experience.
At the end of a packer build you will have a template (.json), and a .vhd.
 Click + New > See all > Search for "Network Interface" > Select "Network interface > Create.  Locate the NIC you just created, and copy the Resource ID (located under properties).
Locate the template file, and copy the contents.
Click + New > Template Deployment > Edit Template, and paste the contents of the template over all existing data.  Click save.

Edit Parameters - provide appropriate values for your template, you will be prompted for all of values that are required.  The hardest value to supply in my case is networkInterfaceId, which is a fully qualified resource string of the form "/subscription/{subscriptionId}/resourceGroups/{resourceGroup}/providers/Microsoft.Network/networkInterfaces/{name}.  If you do not have one, you will need to create one in the portal or via the CLI.

Azure CLI:
Assuming these are the names of our resources.

Resource Group: MyResourceGroup
Location: westus
Virtual Network: MyVirtualNetwork
Subnet: MySubnet
Deployment Name: MyDeployment
Template URI: https://... (copy the URI from the portal, or download it and reference it directly.)
NIC: MyNic

Create a NIC.
azure network nic create MyResourceGroup MyNic westus --subnet-vnet-name MyVirtualNetwork --subnet-name MySubnet

The above command will out an Id for the NIC just created.  Copy it, and use it in the command below.
Deploy a VM based on our image.
azure group deployment create -g MyResourceGroup -n MyDeployment --template-uri https://...

The above command will automatically prompt you for all of the values in the template that need to be set, such as vmName, adminUserName, adminPassword, and networkInterfaceId.
HTH.

We should at least create a post-processor for this...

@paulmey What would a post-processor look like for this?  Doesn't the process of creating a NIC, and then a VM, have to occur every time the custom VHD is used?

@Ancillas The builder deploys the initial VM with a template specifying all these resources. A post-processor could do the same thing. IIRC, there should be a json with a template left next to the vhd. See also #220, where we were discussing what the builder artifact should contain.

Here's the CLI guide for folks who are still finding their way around Packer and AzureRM:
Create the NIC
Assuming you're using the same ARM_RESOURCE_GROUP and LOCATION from packer build:
az network vnet create \
  -g $ARM_RESOURCE_GROUP -l $LOCATION \
  --name PackerVirtualNetwork --address-prefix <IP_ADDRESS>/16

az network vnet subnet create \
  -g $ARM_RESOURCE_GROUP \
  --vnet-name PackerVirtualNetwork \
  -n PackerSubnet --address-prefix <IP_ADDRESS>/24

az network nic create \
  -g $ARM_RESOURCE_GROUP \
  --vnet-name PackerVirtualNetwork \
  --subnet PackerSubnet \
  -n PackerDemoNic

This last command will spit out a lot JSON, you'll want the line that looks like this:
"id": "/subscriptions/12345678-abcd-1234-5678-1234567890abcd/resourceGroups/my_dev_packer/providers/Microsoft.Network/networkInterfaces/PackerDemoNic/ipConfigurations/ipconfig1",

The newer azure-cli doesn't prompt for template params, so I created x.parameters.json file like this:
{
  "$schema": "http://schema.management.azure.com/schemas/2015-01-01/deploymentParameters.json#",
  "contentVersion": "<IP_ADDRESS>",
  "parameters": {
    "vmName": {
      "value": "PackerDemoVm"
    },
    "adminUserName": {
      "value": "PackerDemoVm"
    },
    "adminPassword": {
      "value": "PackerD3moVm%%"
    },
    "networkInterfaceId": {
      "value": "/subscriptions/12345678-abcd-1234-5678-1234567890abcd/resourceGroups/tsa_dev_packer/providers/Microsoft.Network/networkInterfaces/PackerDemoNic"
    }
  }
}

Then I could run az group create:
az group deployment create -g $ARM_RESOURCE_GROUP \
  -n PackerDeployment \
  --parameters @centos.parameters.json \
  --template-uri \
  'https://....

HTH, Peter

Hey Everyone,
Wondering if packer can be used to generate a custom rhel/centos image for azure. (Azure uses a set of steps for Hyper-V - https://docs.microsoft.com/en-us/azure/virtual-machines/linux/create-upload-generic), The image I am referring to is not from any publishers from market place or aws, instead the image contains the OS plus a set of customizations that include separate filesystems / /home, etc to name a few, so when I build my VM through packer, the VM already contains all these I am looking for.
Or the right way to go with using packer to provision Azure VMs is to first have this custom .vhd file ready as depicted in the link here https://docs.microsoft.com/en-us/azure/virtual-machines/linux/create-upload-generic and use that VHD to build VMs on Azure? Now with this same VHD, can packer be used to provision VMs on other platforms like aws, vmware, etc?
Much appreciated.
Thanks.

Yes, you can start from a custom image.  Please review https://www.packer.io/docs/builders/azure.html.  Using the same image on Azure, AWS, and VMware will be trickier.  My default opinion, having never done this, is to instead create a common recipe for building the image, and then use the builder from each cloud provider with your common recipe.  I am sure you could make one image work, but you would have to deal with all of the various platform issues (kernel drivers, services, etc), which may not be worth it.
There are examples in the official Packer repository to help get you started.

https://github.com/hashicorp/packer/blob/master/examples/azure/linux_custom_image.json
https://github.com/hashicorp/packer/blob/master/examples/azure/linux_custom_managed_image.json


Moderation 2: SocialDB schema and Indexers to support moderation

Acceptance Criteria

Determine SocialDB entry formats to indicate:

user hide specific post
user hide all posts from one specific user
user report one post
user report to ban one specific user

Implement an indexer to collect reports into a similar table like this moderation_reporting table. The indexer code can also be fork from dataplatform.near/moderation indexer with minor changes.

This indexer should also calculate moderation decisions from indexing moderators' approve transactions and also who should be considered as moderators are determined by this indexer.

This indexer should put the result in the moderation_decisions table.

This indexer should also create a view, similar to the moderated_posts, which joins devhub indexer's posts_with_latest_snapshot and filter out those moderated posts.

Hi @ailisp, most of it is clear except for this point:

This indexer should also calculate moderation decisions from indexing moderators' approve transactions and also who should be considered as moderators are determined by this indexer.

How does this indexer calculate moderation decisions based on indexing moderators' approve transactions, and what criteria does it use to determine who should be considered as moderators?

Update: Last week researched on indexers and how to implement this. It took me longer than expected but will make a PR today or tomorrow

I will continue with moderation 3 because it will make more sense to me after forking this menu.
@ailisp Looks like we just need to add in the devhub moderators to get this indexer working. We also need to change the group variable from near.org to neardevhub.org.
That way we will be able to query the reports and decisions from the same table based on the group.
I don't have enough permissions to deploy an indexer with my account, since I get:
ExecutionError":"Smart contract panicked: Account thomasguntenaar.near does not have any roles"

I wrote down the schema near.social uses to report/hide posts on tldraw:


change when the tests are triggered

🤔 I don't understand what happend here, I have already fixed it here https://github.com/huggingface/lighteval/pull/304, but not it's commented out.
The problem with secrets is following:
The secrets are provided if one of the repo maintainer pushes a PR, however they are not provided when non-maintainer pushes the PR. That's why I added the skipif, so that the tests only runs when the secret is provided.

Your fix was not working so I commented it out

Interesting do you have an example?
I guess it's possible that the token is set with empty value. This the skipcheck should not check for None but simply for bool(os.environ["HF_TEST_TOKEN"])

It was the case on one of the PRs by chuandudx (closed now)

I will add a skip to this test then, to make sure users PRs can be merged

Lets just keep it skipped then


asHTMLDocument fails on many GT Book pages

The following snippet returns 284 failures.
failures := OrderedCollection new.
LeStorageExamples new gtBook pages
	do: [ :p | [ p asHTMLDocument ] onErrorDo: [ :err | failures add: p -> err ] ].
failures.

It looks like a visitor is not completely implemented for many kinds of snippets (eg examples snippets).

asHTMLDocument is  no longer in the image.


io-threads-do-reads

Describe the bug
I just upgraded my keyDB to the latest version (v6.0.13) and found an issues with io-threads-do-reads setting in the keydb.conf file.
Log Files
*** FATAL CONFIG FILE ERROR (Redis 6.0.13) ***
Reading the configuration file, at line 1007
>>> 'io-threads-do-reads yes'
Bad directive or wrong number of arguments

To Reproduce

update # io-threads-do-reads no to io-threads-do-reads yes in the keydb.conf file.

There was an earlier issue where this was in our default config file.  That was fixed a while ago, but we also need a dummy load to retain compatibility with redis config files even if this config is meaningless to us.
Fix change: 02b533fec


Graph Creation Request Fails When Specifying List of document_ids

Describe the bug
Call to create_graph() with list of document_ids via Python SDK yields error:
To Reproduce
Steps to reproduce the behavior:

Ingest documents
Enumerate document ids by calling get_all_documents()
Attempt to initiate graph generation by calling create_graph() with list of document_ids from step 3
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Hey, thanks for raising this issue. Could you share your snippet and the server logs? (you'll find them inside the r2r docker container)

Sorry fat fingered this one, please see #1116, closing this case.


were is the app in the files after downloading

<!--
i extracted them go through files no app to run just files needed help

Did you read the installation documents?
What OS are you on? There's no app except if you use this software:
https://github.com/PokemonGoF/PokemonGo-Bot-Desktop

We haven't heard from you for a while. Please reopen if you need further help.


error and interface not showing nothing

Can't access:

cameras
recordings
configuration

[2023-01-16 11:57:17] [ERROR   ] [viseron.components.ffmpeg.recorder.camera_3] - Could not get duration for: /segments/camera_3/20230116115716.mkv. Error: [matroska,webm @ 0x5572a23b6fc0] EBML header parsing failed
/segments/camera_3/20230116115716.mkv: Invalid data found when processing input
[2023-01-16 11:57:17] [ERROR   ] [viseron.components.ffmpeg.recorder.camera_3] - No segments were found
[2023-01-16 11:57:20] [ERROR   ] [viseron.components.ffmpeg.recorder.camera_2] - No segments were found
/segments/camera_2/20230116115717.mkv: Invalid data found when processing input
[2023-01-16 11:57:20] [ERROR   ] [viseron.components.ffmpeg.recorder.camera_2] - Could not get duration for: /segments/camera_2/20230116115717.mkv. Error: [matroska,webm @ 0x563e7238ffc0] EBML header parsing failed
[2023-01-16 11:57:20] [INFO    ] [viseron.components.nvr.nvr.camera_2] - Stopping recording in: 0
[2023-01-16 11:57:19] [INFO    ] [viseron.components.ffmpeg.recorder.camera_1] - Starting recorder
[2023-01-16 11:57:18] [INFO    ] [viseron.components.ffmpeg.recorder.camera_2] - Starting recorder
[2023-01-16 11:57:20] [INFO    ] [viseron.components.ffmpeg.recorder.camera_2] - Stopping recorder
[2023-01-16 11:57:20] [INFO    ] [viseron.components.nvr.nvr.camera_1] - Stopping recording in: 0
[2023-01-16 11:57:20] [INFO    ] [viseron.components.ffmpeg.recorder.camera_1] - Stopping recorder
[2023-01-16 11:57:20] [ERROR   ] [viseron.components.ffmpeg.recorder.camera_1] - Could not get duration for: /segments/camera_1/20230116115716.mkv. Error: [matroska,webm @ 0x55890ee8dfc0] EBML header parsing failed
/segments/camera_1/20230116115716.mkv: Invalid data found when processing input
[2023-01-16 11:57:20] [ERROR   ] [viseron.components.ffmpeg.recorder.camera_1] - No segments were found

IMHO, recordings UI must be completely decoupled from camera input availability.
And on cameras page, configured, but unavailable cameras must be marked, not missed.

IMHO, recordings UI must be completely decoupled from camera input availability. And on cameras page, configured, but unavailable cameras must be marked, not missed.

well ok, but how can i solve now? nothing is usable

I can't access cameras and recordings pages, without valid camera input, but configuration and entities works.

well ok, but how can i solve now? nothing is usable
I am not a viseron expert, but i think, main problem: ffmpeg can't decode input stream from cameras, for some reason.

Did you change something in the camera settings, or viseron's config ffmpeg -> cameras section before this happened? Or did it happen after viseron update?
I think @roflcoopter will need this info too, to help you.

solved.

solved this was not related to viseron itself.


[light-client-state] remove or prune unjustified_header field until we have proper finality check

Currently, we keep track of finalized heads, which have not been proven to be final with a justification in the unjustified_headers. Once we receive a block including a justification, we check it and mark all unjustified_headers as finalized, which finalizes them in our state.
However, this logic is partially flawed, as we there is only a justification in a block when the authority set changes. This in turn implies that, depending on the chain configuration, we will never get a justification and the unjustified_header field will grow indefinitely, which probably leads to #1374.
Considering this, I suggest for now removing the unjustified_header field until:

We either use a proper light-client that can infer finality, like https://github.com/smol-dot/smoldot.
We can use the beefy protocol to have finality proofs.

I can confirm that in OLIs case we also have a very slow parentchain block import. Probably because of deserializing many unjustified headers.


fix game issue + add explosion animation and sounds

list of changes

fixed conversion type bug in collide.py
explosion animation implemented using Explosion class in explosion.py
explosion sounds added - laser_hit.wav and explode.wav

@rusty-electron Will take a look at it, by the way the images you have added are of real explosion, can you send me an image of any one of them when it comes into action?
As according to me a pixelated explosion or arcady like explosion would've looked better

It does look pixelated in-game so see if you like the aesthetic or else I will try to find some other assets.

@rusty-electron
Okay, so couple of things,

Great job with the explosion graphics, but it seems a little out of place as the explosion is real and the game is a pixelated game in which an explosion with a more animated kind of look would have been much better, the implementation though is bang on.

Another thing that hit me instantly was when the ship was getting hit, it sounded like a sound of hitting a tin box, what I would've preferred is the sound that many games have when the health is deducted whenever it is hit. Not the sound of how would it sound when the ship would have been hit, the sound of health deduction, the sound is in my head lol I can't seem to explain myself that good about the sound, let me know if you get it or I will try to explain a little more or try to find a similar kind of sound.

Like there's for health up, something like this for health up I guess health boost
You might get the idea that what I am expecting for damage sound

The explosion sound is okish I think, you might wanna try Pixel chipmunk explosion, or something similar but better

ok, I'll look for pixelated assets.
I'll send you some sound samples on discord to get some idea about the sound.
I'll look into it. Do you know sites that offer free sounds other than freesound and itch.io?

ok, I'll look for pixelated assets.
I'll send you some sound samples on discord to get some idea about the sound.
I'll look into it. Do you know sites that offer free sounds other than freesound and itch.io?

I am not aware of that, though there is a youtube channel gfx sounds I think, they might provide with the links in the description maybe

@Mayank0255 I think it is merge-ready now.


uncomment non-required values in chart

Some chart values are given empty strings, but that should only happen for required values.
Non-required values should always be left out or uncommented if they are valuable to mention.
We should go over the values and check if they follow these requirements. The following are suspect so far:

cluster.apiServer

done, only minimal values remain


Add lineal spring notebook

There seems to be a bug if we choose
n = 10
mesh  = UnitSquareMesh(n, n)

which isn't apparent for n=8: the angle solver generates nan solutions. This needs fixing before the PR can be merged.

Fixed the bug: np.arccos was being passed something like 1.000004.


Stage 0: GeneratorExpression

So I don't think we need to support Array/generator comprehension
//ComprehensionBlock
//ComprehensionExpression
It won't be in ES7/ES2016 (maybe not even at all) and may be removed in babel 6.0 too.

Ok going to remove in elementTree


Support CMake system

This patch enables users to install clipp with CMake:
cd clipp/
mkdir build
cd build/
cmake -DCMAKE_INSTALL_PREFIX=${HOME}/local ..
make install

and use it from other CMake projects:
# CMakeLists.txt in awesome_project
project(awesome_project CXX)
# ...
find_package(clipp)
# ...
target_link_libraries(${PROJECT_NAME} PRIVATE clipp)

CMake is also useful for building and executing the tests:
cmake -DBUILD_TESTING=1 -DCMAKE_CXX_COMPILER=g++-8 ..
cmake --build . -- -j4
ctest -V

(I have not checked if it works equivalently as run_tests.py, though)
I am concerned that this PR might conflict with your policy. Please feel free to reject this, and I would be fine to keep it in my branch.

I can see that CMake would be useful for building the tests. But what is the purpose of the other CMakeLists.txt file? I mean clipp is a header only library. Don't you just include it in your project and only add the translation units and binary libs to your CMakeLists.txt and CMake will be able to figure out the dependencies on headers itself?
Anyway, for now I would like to keep anything that isn't neccessary for the library to work out of the repo. It's such a small project and I don't want to clutter it with files for all the different build systems. But I'm of course willing to change my mind if more users really want it.

just include it in your project

Yes, that's the quickest way to add single header libraries to a project. But in the long run, it makes it difficult to manage dependencies and follow their updates. Therefore, developers often want to keep external libraries out of their project repositories. For example, I like nlohmann/json, an awesome single-header library, and install it in /usr/local/, ~/local/, or some other locations to use it from several projects. Of course it can be achieved by just git + some scripts, but CMake provides more portable and configurable methods.

Anyway, for now I would like to keep anything that isn't neccessary for the library to work out of the repo. It's such a small project and I don't want to clutter it with files for all the different build systems. But I'm of course willing to change my mind if more users really want it.

I respect your opinion. Let's wait and see coming +1s.

Yeah Niels Lohmann's JSON lib is awesome; I've also used it a couple of times.
I agree that if you want to keep (even single-header) libs external the approach you proposed is better.
Personally I like to bundle projects with all their dependent libs so they don't interfere with other projects that use other versions of those libs (it also makes deployment easier). I just had to many bad experiences of projects not compiling/working anymore because some library was updated and the project needed the older version for various reasons. Of course this approach has limits when monsters like Qt are involved.

+1 for CMake support for easier dependency management and finding the (interface) library via find_package(clipp).
But you should not set CMAKE_CXX_STANDARD but go with modern CMake, using target_compile_features(clipp INTERFACE cxx_std_11).

Thank you @m-waka for the suggestion. Updated CMakeLists.txt accordingly.

Now that at least one other person has looked over it and the CMakeLists looks a lot cleaner I think it is ready to be merged.


End column should be exclusive, not inclusive

The v2 spec says:

3.22.8 endColumn property
When a region object represents a text region, it MAY contain a property named endColumn whose value is an integer whose value is one greater than the column number of the last character in the region.

Currently, the extension treats the endColumn as the column number of the last character in the region, so the highlighted range is one greater than it should be.
Note: endColumn was inconsistently defined in v1 and earlier versions of the v2 spec, which is probably why this confusion occurred.

Thanks for the bug @lukecartey I'll investigate, should be a relatively easy fix.

Thanks for tackling this, Rusty. Put me on the PR and I'll see if I can start ramping up a bit more on this code base.

Thanks for addressing this. My apologies for failing to notice this was scheduled for the 2.3 milestone.

Fix has been merged into dev branch, it'll be in update 2.3.0, releasing this week.


select() does not select all columns by default

Bug report

[x] I confirm this is a bug with Supabase, not with my own application.
[x] I confirm I have searched the Docs, GitHub Discussions, and Discord.

Describe the bug
The docs say the columns parameter of select() "defaults to *." It does not.
To Reproduce

Run supabase.table("t").select("*"), and get everything in t.
Run supabase.table("t").select(""), and get everything in t.
Run supabase.table("t").select(), and get no data.

Expected behavior
supabase.table("t").select() should get everything in t. I understand "defaults to *" to mean * is the default argument.

Hello,
As you've correctly pointed out, there is a discrepancy between the documentation and the actual behavior of the select() method. The documentation states that if no columns are provided, the select() function "defaults to *", which implies that all columns should be selected.
However, in the current implementation, if no arguments are passed to select(), it results in a HEAD request, which doesn't return any data.
I think HEAD request can be useful for users that only check if table exist or not without fetching any data which can slow down the request.
So perhaps an update for documentation is preferable in this case


use GPy if available

the modified block is wrapped in a try-catch block so it should be safe to merge


Firebase iOS 8.3.1 is missing Firebase.zip

[REQUIRED] Step 1: Describe your environment

Firebase SDK version: 8.3.1
Installation method: Zip file

[REQUIRED] Step 2: Describe the problem
Steps to reproduce:
GitHub release Firebase iOS 8.3.1 is missing Firebase.zip. See https://github.com/firebase/firebase-ios-sdk/releases/tag/8.3.1

8.3.1 was a Swift Package Manager only release since the only change was SPM specific tagging.
See https://firebase.google.com/support/release-notes/ios


[RESOLVED] Unavailable by ICMP ping

Problem started at 23:45:35 on 2023.06.03
Problem name: Unavailable by ICMP ping
Host: ESXi-VC
Severity: High
Operational data: Down (0)
Original problem ID: 23347
Event details in Zabbix: http://<IP_ADDRESS>/tr_events.php?triggerid=23568&eventid=23347

Problem has been resolved in 1d 5h 33m 0s at 05:18:35 on 2023.06.05
Problem name: Unavailable by ICMP ping
Host: ESXi-VC
Severity: High
Original problem ID: 23347


Apply Parser.filepath to normalise path retrieved from Editor

This fixes the "Load" command on Windows - and presumably all the other commands relying on this too.
I can't tell if this breaks anything else though - hopefully not!

Looks good to me!


Rings get unequipped somehow

Seems to be only the first ring equipped? This has happened multiple times but I haven't looked into the details yet.

blaino does it


menu entry swapper: Add ash scatter swap

Similar to the bones entry, this swap will prevent accidental scattering
of demonic ashes for players who wish not to gain prayer experience, or
wish to save them to sell or offer at a later time.

8944eb9be6d62ec8384e29484dab14c4c2858d9d was added to master, which sufficiently covers this use case.


Typo(s) in 1.14 announcement (EN)

Where to find the issue
https://www.coronawarn.app/en/blog/2021-03-09-corona-warn-app-version-1-14/
Describe the issue
The sentence:
"They can also specify whether they’ve met another person for more than 15 minutes or less, whether they wore a mask during the meeting, and whether they were outside our indoors."
is not right.
"our" is wrong
"outside" does not correspond to screenshot
Also in
"Under both, locations and people, users can also add their own notes."
It is confusing to place a comma after "both" in the above.
Suggested change
Change to "outdoors or indoors".
Remove the comma after "both".
PS Let me know if you want me to submit a PR to correct these two issues.

Internal Tracking ID: EXPOSUREAPP-5070

@HannaHei
Are you planning to correct the typos in the 1.14 announcement (English version) soon?
The nonsensical sentence:
"They can also specify whether they’ve met another person for more than 15 minutes or less, whether they wore a mask during the meeting, and whether they were outside our indoors."
still appears on https://www.coronawarn.app/en/blog/2021-03-09-corona-warn-app-version-1-14/.

@MikeMcC399 I forward it to the ticket.

The "our" has been fixed in #982

@dsarkar / @HannaHei
Both issues I described are fixed.
Thank you!


feat: Adds new config option: connectionString

Summary
The docusaurus-plugin-application-insights plugin's current options include only the instrumentationKey option, which is slated for deprecation in 2025. This PR adds support for Connection strings, which is the new way to define where to send telemetry data.
New options type:
export type ApplicationInsightsOptions = {
    instrumentationKey: string;
    connectionString: string;
};

These options are mutually exclusive.

@scalvert do you want me to merge?


Automobile Sales Data Analysis and Prediction

ML-Crate Repository (Proposing new issue)
:red_circle: Project Title : Automobile Sales Data Analysis and Prediction
:red_circle: Aim : The aim of this project is to create a machine learning model to predict the sales of the automobiles and prepare a data analysis of the same.
:red_circle: Dataset : https://www.kaggle.com/datasets/ddosad/auto-sales-data
:red_circle: Approach : Try to use 3-4 algorithms to implement the models and compare all the algorithms to find out the best fitted algorithm for the model by checking the accuracy scores. Also do not forget to do a exploratory data analysis before creating any model.

📍 Follow the Guidelines to Contribute in the Project :

You need to create a separate folder named as the Project Title.
Inside that folder, there will be four main components.

Images - To store the required images.
Dataset - To store the dataset or, information/source about the dataset.
Model - To store the machine learning model you've created using the dataset.
requirements.txt - This file will contain the required packages/libraries to run the project in other machines.

Inside the Model folder, the README.md file must be filled up properly, with proper visualizations and conclusions.

:red_circle::yellow_circle: Points to Note :

The issues will be assigned on a first come first serve basis, 1 Issue == 1 PR.
"Issue Title" and "PR Title should be the same. Include issue number along with it.
Follow Contributing Guidelines & Code of Conduct before start Contributing.

:white_check_mark: To be Mentioned while taking the issue :

Full name :
GitHub Profile Link :
Participant ID (If not, then put NA) :
Approach for this Project :
What is your participant role? (Mention the Open Source Program name. Eg. HRSoC, GSSoC, GSOC etc.)

Happy Contributing 🚀
All the best. Enjoy your open source journey ahead. 😎

Full name : Ankana Pari
GitHub Profile Link : https://github.com/ankana2113
Participant ID (If not, then put NA) : couldn't find
Approach for this Project : Would plot some graphs to determine the trend of the data and then try to approach which ml model fits best for e.g, regression or random regression etc.
What is your participant role?KWoC

I am facing a problem on how to convert datetime obj into a string because
standardizing using StandardScaler requires a string. Can you please help
me out?
Ankana
Issue #440
On Fri, Dec 8, 2023 at 7:57 PM Abhishek Sharma @.***>
wrote:

Assigned #440 https://github.com/abhisheks008/ML-Crate/issues/440 to
@ankana2113 https://github.com/ankana2113.
—
Reply to this email directly, view it on GitHub
https://github.com/abhisheks008/ML-Crate/issues/440#event-11194138289,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BCJWMCYOGMAX5E7XHZFLMTDYIMPTNAVCNFSM6AAAAAA76ZZH2WVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJRGE4TIMJTHAZDQOI
.
You are receiving this because you were assigned.Message ID:
@.***>

Can you share the problem in detail. You can connect with me in Discord.

Issue assigned to you @Mudassir-A

Aditi Kala
Github:- https://github.com/why-aditi
Participation ID:- NA
Approach: Clean and preprocess the data, handling missing values and ensuring all variables are in suitable formats. Conduct exploratory data analysis to understand relationships and patterns in the data. Select relevant features that influence sales, possibly using feature engineering techniques. Choose appropriate regression algorithms like Random Forest or Gradient Boosting, train the model on a portion of the data, and evaluate its performance using metrics such as MSE. Interpret the model's results to extract insights into key drivers of sales, such as the impact of car features or economic conditions. Visualize actual versus predicted sales values to assess model accuracy.
Participation Role:- SSOC Season 3

Aditi Kala Github:- https://github.com/why-aditi Participation ID:- NA Approach: Clean and preprocess the data, handling missing values and ensuring all variables are in suitable formats. Conduct exploratory data analysis to understand relationships and patterns in the data. Select relevant features that influence sales, possibly using feature engineering techniques. Choose appropriate regression algorithms like Random Forest or Gradient Boosting, train the model on a portion of the data, and evaluate its performance using metrics such as MSE. Interpret the model's results to extract insights into key drivers of sales, such as the impact of car features or economic conditions. Visualize actual versus predicted sales values to assess model accuracy. Participation Role:- SSOC Season 3

Implement 5-6 models for this project.
Assigned @why-aditi


lists:nth vs array:get

array module is more faster than lists about random access case
Try to check it.
Ref:
https://github.com/gar1t/erlang-bench/blob/master/index_lookup.escript

Close this issue. This may be not meaningful.


Move to theme.toml

New Sphinx 7.3 now prefers theme.toml over theme.conf: https://github.com/sphinx-doc/sphinx/pull/12265
Just a note to maybe consider changing.
(I noticed while debugging https://github.com/sphinx-doc/sphinx/issues/12299)

Do note, theme.toml was only introduced in 7.3 -- the current Sphinx minimum version in PDT is 5.0.
A

Ah good point.

reopening as a TODO for the future


RAC-A: Login messaging for AC-8

In order to comply with NIST 800-53 AC-8 all users must acknowledge monitoring is in effect and see clear guidance on the usage of the system whenever they login at https://login.fr.cloud.gov.
Acceptance Criteria

[ ] Users are forced to acknowledge the points in AC-8 (a) (1-4) before logging in (which meets AC-8 (b))
[ ] Users can find detailed information about the intended usage and conditions of their access (meeting AC-8 (c))
[ ] Users can find detailed information about the terms of service of cloud.gov (meeting PS-??)
[ ] The specific content of our messaging has been reviewed by the OGC

Implementation Sketch
Here's what we came up with:

Notes from discussion:

Careful not to lose the shading boundary separating the SAML and local login forms that's already in in production
Content of short text should be Your use of this US government service is monitored and recorded. Any misuse is subject to criminal and civil penalties.
Content of long text to be determined by consulting with OGC/GSA IT.

Can user acknowledgement be combined into the same page as the login screen, rather than forcing the user to explicitly accept prior to being sent to the login screen in a second step?
I won't point to this as an example of good design (it's just the first thing I thought of), but other government sites like https://goes-app.cbp.dhs.gov/main/goes include the message on the same screen as the login form & don't require two steps.
I'd recommend getting rid of that Accept & Continue button if possible, and put that disclaimer above the login form, saving the user a step.

Yeah, I don't know if it's acceptable/desired simply to have a notice that says something like:

Signing in to cloud.gov acknowledges that your use of this US government service is monitored and recorded. Any misuse is subject to criminal and civil penalties. Read more details.

AC-8 (b) says:

b.	Retains the notification message or banner on the screen until users acknowledge the usage conditions and take explicit actions to log on to or further access the information system;

I don't know if the clause indicates that the explicit action to log on itself can be considered to fulfill acknowledgement of the usage conditions. But if we just state that it does...
@mogul, do you have any additional guidance?

@marco, I started roughing this out on https://github.com/18F/cg-uaa/tree/dw-ac8
Now, assuming the functionality we outlined in the issue, we need:

All login modules to be hidden until the user confirms the notice.
The complete notice (currently set up in a modal) triggered when the user clicks on the link in the notice. (And be dismissable.)
Login modules to appear — and notice to disappear — when the user confirms the notice.

Does this page need javascript enabled to work? Should it work even without javascript?

I don't know if the clause indicates that the explicit action to log on itself can be considered to fulfill acknowledgement of the usage conditions. But if we just state that it does...
@mogul, do you have any additional guidance?

Having seen no precedent system using it, I feel less confident that we'd pass muster that way, and we're a bit under the gun to have something passable in place by this Friday. So I'd say we can keep that in mind as UX debt to be revisited in future when we have better guidance available and more time to clear it.

Does this page need javascript enabled to work? Should it work even without javascript?

Of course it would be great for it to work without Javascript, but so long as the non-Javascript state doesn't just let them waltz through (eg controls are hidden/disabled until Javscript enables them), I think it's fine to require Javascript.

I'd agree with Bret here, it doesn't need to work without javascript for now.

@marco, I'm gonna pull your js commit on my branch, do a quick check, and get a PR going for first thing tomorrow morning.

That worked great @msecret. Thank you.
I made a first pass at style polish on the modal. Here's how the whole flow works:

User sees notice with link to read more or agree and continue:

Read more details reveals the details in a dismissable modal.

Selecting 'Agree and Continue' reveals the standard login options

This first pass is addressed in a pull request: https://github.com/18F/cg-uaa/pull/18

I started a doc in our cloud.gov Google Drive folder for working on the longer text: https://docs.google.com/document/d/1Rr0P52p-0tRtnD17K3yEkjAGmGt0VSU55vUYAUsYB3k/edit#

It looks good in the staging environment, and will appear on https://login.cloud.gov and https://login.fr.cloud.gov shortly.

Deployed and live. This should be sufficient for compliance purposes.
One bug report: In a small window, the pop-over text is cut-off and doesn't scroll:

Resizing the window after that happens doesn't fix it... You have to reload with a larger window for it to work.
If this is a trivial fix, let's tackle it. Otherwise, feel free to note this on a separate card and throw it in the Icebox for a while.