"Select the database you want to use. / Use SQLite if you don't have a database running yet." stays visible after I select MySQL or PostgreSQL in init

λ yarn prisma2 init foo
yarn run v1.21.1
$ C:\Users\Jan\Documents\prisma-cli-blackbox-tests\node_modules\.bin\prisma2 init foo

  Select the database you want to use.
  Use SQLite if you don't have a database running yet.

 SUCCESS  Introspection was successful.

  Select the Prisma Framework tools you want to use.
  Learn more: https://github.com/prisma/prisma2

┌─ Prisma Framework tools (toggle with [space]) ────────────────────────────────────────────────────────────────────────────────────────────────────┐
│                                                                                │
│ > [×] Photon  Type-safe database client                                                                                │
│   [×] Lift    Declarative data modeling & migrations                                                                                │
│                                                                                │
│   Note: You can still add a tool to your project later                                                                                │
│                                                                                │
└───────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────┘

> Confirm
< Back (Database credentials)

I am unable to reproduce this on a Mac. Maybe you can list the exact flow to reproduce this.

@janpio is this still reproducible?

Yes, this still happens from time to time on Windows.

I am still unable to reproduce this. I used hyper as the terminal emulator. Maybe it is more related to your terminal emulator. Which one are you using?

Cmder running cmd.exe

Still no luck :/


Metrcis 01

REQUIRED: Add a summary of your PR here, typically including why the change is needed and what was changed. Include any design alternatives for discussion purposes.

changed based on review comments

REQUIRED: Fill out the below checklists or remove if irrelevant

If you are opening a PR for Official Notebooks under the notebooks/official folder, follow this mandatory checklist:

[X] Use the notebook template as a starting point.
[X] Follow the style and grammar rules outlined in the above notebook template.
[X] Verify the notebook runs successfully in Colab since the automated tests cannot guarantee this even when it passes.
[X] Passes all the required automated checks. You can locally test for formatting and linting with these instructions.
[ ] You have consulted with a tech writer to see if tech writer review is necessary. If so, the notebook has been reviewed by a tech writer, and they have approved it.
[X] This notebook has been added to the CODEOWNERS file under the Official Notebooks section, pointing to the author or the author's team.
[X] The Jupyter notebook cleans up any artifacts it has created (datasets, ML models, endpoints, etc) so as not to eat up unnecessary resources.

5: --> 104 with open(file_path) as f:
Step #5:     105     return yaml.safe_load(f)
Step #5:
Step #5: FileNotFoundError: [Errno 2] No such file or directory: 'tabular_classification_pipeline.json'

New branch and PR created for this Notebbok.
Merged #847 into main.


Port to GitHub Actions

FYI I ported your code to GH Actions, got the sonarcloud version working here: https://github.com/dylan-smith/sonarcloud-buildbreaker
Gonna do a sonarqube server version next.

Awesome :) Is this support for sonarcloud/sonarqube build breaking in GitHub Actions?


New Crowdin updates

@karlomikus I've managed to 100% the Italian translation through crowdin. It should be included in these commits.

Nice,
Grazie!


A question about defineProperty of Proxy object

When calling Object.defineProperty as below, the internal method [[DefineOwnProperty]] will be  invoked.
According to ES10.0, when invoking the internal method [[DefineOwnProperty]] of the proxy object, the method defineProperty of proxy handler is called. So the overwritten property defineProperty  below is called. And a false will be returned after using ToBoolean dealing with its return value, because the method's default return value is undefined.
Finally, a TypeError will be thrown as other engines(like V8,spiderMonkey and quickjs) do.
But Chakra didn't throw any error as my expection, Is there any information I have missed?
version
1.11.19

command
chakra/cc_linux_x64_1_11_19/ChakraCoreFiles/bin/ch testcase.js

testcase
function main() {
var handler = {defineProperty:function(){
        print("overwrite definedProperty");
        }
     };
var target = {"a":3};
var descBefore = Object.getOwnPropertyDescriptor(target,"a");
for(var j in descBefore){
  print(j,descBefore[j]);
	}

var p = new Proxy(target,handler);
var desc = {value:200};
var v21 = Object.defineProperty(p,"a",desc);
print(v21);

var descAfter = Object.getOwnPropertyDescriptor(target,"a");
for(var i in descAfter){
print(i,descAfter[i]);
	}
}
main();

output
value 3
writable true
enumerable true
configurable true
overwrite definedProperty
[object Object]
value 3
writable true
enumerable true
configurable true

expected output
value 3
writable true
enumerable true
configurable true
overwrite definedProperty
TypeError: proxy defineProperty handler returned false for property '"a"'

contributor:@YuanWangC

ping @zenparsing, @chicoxyzzy

This is a bug, per https://tc39.es/ecma262/#sec-definepropertyorthrow if the DefineOwnProperty method of an object (in this case the proxy trap) returns false-ish then should throw.
Should be an easy-ish fix, need to look at the implementation of Object.defineProperty (probably called) EntryDefineProperty and see how it handles calling a false return value.


"Fix" ID by updating to state's new URL

Idaho moved its warn PDF from https://www.labor.idaho.gov/dnn/Portals/0/Publications/WARNNotice.pdf to https://www.labor.idaho.gov/wp-content/uploads/publications/WARNNotice.pdf. The scraper follows this transparently, so there's no breakage, but seems like a good policy to update the URL to reflect the current location.

One note here: the state's page linking to this file actually links to https://www.labor.idaho.gov/warnnotice/ which does a redirect to the PDF with a note that says "(link is updated as notices are received)." That reads to me like the file is updated continuously, but it could also mean they change the link on a semi-regular basis. So, we have a couple options:

retrieve the file at the current URL of the PDF
retain the current behavior and rely on the redirect from the file's old URL
rely on the /warnnotice/ redirect
scrape the HTML page to know which URL to check

I think it's probably a crap shoot, but the simplest thing to do to improve the situation might be #1.


Use use_diff_of_y and predict_ahead == [0] at the same time

When using use_diff_of_y you apparently can't set predict_ahead = [0] in TimeseriesMLP, there are multiple checks for this in the code, and removing the first error will lead to predicting a straight line. Using use_diff_of_y with any other predict_ahead works as expected.

I took some time to investigate this. The reason why you can't set predict_ahead = [0] with use_diff_of_y is because the predict_ahead (or lags in make_shifted_target where differencing is actually calculated) is used to determine the offset for the differencing. So for a predict_ahead of 3 the differencing is calculated using the third next timestamp. Therefore a predict_ahead of zero would result in the "differenced" y being all zero (subtracting the value from the 0th next timestamp; itself).
There is however still a valid reason to want to use differencing with a nowcast (when predict_ahead is zero), because the X features would then be aligned with the nowcast. For example if we have a timeseries with a frequency of 1 minute. Then predict_ahead = [5] would result in a prediction:

For the difference between timestamps 00:05 and 00:00
Indexed as 00:00
Using features from X at index 00:00

Whereas predict_ahead = [0] with a manually configured differencing offset of 5 would result in a prediction:

For the difference between timestamps 00:05 and 00:00
Indexed as 00:05
Using features from X at index 00:05

The last point is the important one: The predict_ahead = [0] case would use data from the last of the two timstamps that are differenced, whereas the predict_ahead = [5] case uses data from the first of the two timestamps.
To support such a "manually configured differencing offest" for the nowcast, since we can't just use the predict_ahead/lags value of zero, you would change the calculation in make_shifted_target[^1] from the current implementation:
[^1]: It is also necessary to shift y by the same offset in inverse_differenced_target when adding the differences back.
result = pd.concat([-1 * y.diff(-1 * lag) for lag in lags], axis=1)

To something like:
result = pd.concat([y.diff(nowcast_offset) if lag == 0 else -1 * y.diff(-1 * lag) for lag in lags], axis=1)

Which would at the very least introduce a nowcast_offset parameter in make_shifted_target and probably also in the __init__ of BaseTimeseriesRegressor.
So then, it might be worth it to implement this for some relatively niche application where the desired differencing offset is high and it has significant impact which timestamp is used from X, but I'm not convinced such a case will ever come up. So I think the current behavior of rejecting the combination of use_diff_of_y with predict_ahead = [0] is totally acceptable.


Needs update for Storybook 6.2+

This module needs an update for Storybook 6.2+. A few things have changed making this addon incompatible. I really like the idea of this addon though. I plan on managing a large library with loads of dependents. Would like to get an overview of components touched when changing something.
Is someone gonna pick this up?

Thanks - i would gladly accept PRs, but do you have a sample project that shows the error? 6.2 is a minor version upgrade and shouldn’t introduce incompatible changes afaik.

@atanasster You can use my example zip from #11
I get the following error after running npx sb@next upgrade & npm run storybook

thanks @puyt I will try to get some free time soon and will let you know.

Thanks guys, just released v 2.2.0, should be compatible with 6.2

Thanks guys, just released v 2.2.0, should be compatible with 6.2

Works for me, thanks a lot! :+1:


[BUG] App Migration ucs to AS3 takes forever

Describe the bug
App Migration usc to AS3 takes forever. cannot load configuration.
To Reproduce
Steps to reproduce the behavior:

Start Application Service Migration
upload ucs file
enter master key
the next dialog is telling "loading configuration..." and it takes forever

Expected behavior
app should load the existing configuration. After 2 hours waiting we gave up.
Screenshots

Context

Please specify, app version:
f5devcentral/f5-bigip-journeys-app:v3.2.4
And Platform version of the system you want to migrate.
<IP_ADDRESS>

UCS loading performance definitely is currently an open issue we plan to tackle eventually, but I haven't yet seen loading times this long (I think the highest I've encountered was 30 minutes or so?), so it might be some rare bug with an infinite loop somewhere. Would it be possible for you to create an SR ticket uploading the problematic UCS file for us to investigate? https://www.f5.com/services/support

Hi @jszczepanskif5 ,
Thanks I will open a case and I can send the UCS file then.

Hello @ApolloDS,
Could you retry with the latest version 3.3.1 and let us know if this resolves the issue?
You can find the update instructions in the readme.
Thanks
Roman

Hi Roman,
I just tested it with the config and now it will be loaded, great!
So it works now, thank you very much.
Do you know what kind of the object was the issue?
Thanks,
Peter

Hello,
Great, we had a similar problem with another customer having iRules causing the UCS loading to get into some sort of loop. This was addressed in ACC 1.19 which Journeys used in the backend for the translation of non-AS3 config to AS3. If you look at the Dockerfile, you can see the ACC version Journeys used (right now 1.19). The Journeys team also did some perf improvement in 3.3.0.
I will close the issue for now, feel free to open a new one if you encounter any other problems.
Thanks,
Roman


Application Hangs In Chrome Version - 90.0.4430.85  - Print feature

Attach (recommended) or Link to PDF file here:
Configuration:

Web browser and its version:  Google Chrome Version 90.0.4430.85 (Official Build) (64-bit)
Operating system and its version: Windows 10
PDF.js version: "pdfjs-dist": "^2.7.570",
Is a browser extension:

Steps to reproduce the problem:

From our application, we open a new window which is showing the PDF.
Click on "Print" Icon. Print Dialog box will Open
Close the new Window.
On closing the new window, our application hanged.

This above issue is not happening in

Google Chrome Version 89.0.4389.128 (Official Build) (64-bit)
Operating system and its version: Windows 10
PDF.js version: "pdfjs-dist": "^2.7.570",

What is the expected behavior? (add screenshot) : Our application should not hanged.
What went wrong? (add screenshot)
What I observed is pdf.js using Google print Cloud. This feature has been depreciated in 2021 which might cause the issue

What I observed is pdf.js using Google print Cloud. This feature has been depreciated in 2021 which might cause the issue

Unfortunately "Google print Cloud", whatever that may be, isn't something that the PDF.js library can provide support for. (Even more so, when by your own account, it's apparently not even supported by Google any more.)
Hence this issue unfortunately doesn't seem actionable/valid here, since we cannot support third-party services.
Furthermore, since we cannot even reproduce this without access to "Google print Cloud", please see https://github.com/mozilla/pdf.js/blob/master/.github/CONTRIBUTING.md (emphasis mine):

If you are developing a custom solution, first check the examples at https://github.com/mozilla/pdf.js#learning and search existing issues. If this does not help, please prepare a short well-documented example that demonstrates the problem and make it accessible online on your website, JS Bin, GitHub, etc. before opening a new issue or contacting us in the Matrix room -- keep in mind that just code snippets won't help us troubleshoot the problem.


Fixed Downscroll Held Notes

This took a while, but I finally managed to make a working fix for downscroll held notes.

Looks good


Question about seeding initial data for server side rendering

First off, thanks for the great library! This has been a great help in getting our .net MVC site working with React. Now on to my question..
We are working on bringing over our site into React. We have wired up server side rendering, which is working great. One thing we've come across now though, is looking at how to pass some "global"  data into the server side rendering context, without it being passed in through every single component. This data is in fact used by many components, through use of React's "mixin" functionality.
In other words, I can see how we can make data available to the server side render by passing in data through Html.React, or Html.ReactWithInit (thanks for this new method btw, we've enjoyed using it in a partial already), and we started going down the path of having a base model which is extended by the model used for any given view, and passing our data through there. This however will start to get ugly as we start to render multiple components to a page, duplicating the information we are storing in the base model.
After digging around in the source a little bit, it seems like one possible approach would be to find some means to make a call to ReactEnvironment.Execute(string code), and pass in a string from a Razor view that would let us pass in some JSON, etc, to seed our data into some kind of global variable.
Another alternative could be to create some sort of 'invisible' 'global-state-seeding' component which we could pass our data in through, however this would require our developers are all very aware of making sure this component is present in all pages, and only once.. which sounds less attractive than finding some means to have this data automatically wired up.
Do you have any suggestions on this? Are these feasible approaches, and if not, could you offer some insight on any alternatives?
Thanks!

Going to close this out, but feel free to post more comments if you have any further questions.


Update to the version is 0.8.2

The current version of sbt-dependency-graph is 0.8.2.
Could you please update the version of this plugin?

+1

I've just released 0.8.2

Thank you.


Relax dependency on spsdk to allow using 2.1.x?

Currently spsdk is pinned to<2.1 in the pyproject.toml file.
Could this be relaxed, to allow using versions 2.1.x? I had a quick glance and found lots of doc changes between 2.0.x and 2.1.x, but not many real code changes. If spsdk uses semantic versioning, there should not be major / breaking changes in those versions.
Thanks in advance!
Johanes

Unfortunately, spsdk does not follow semver, see https://github.com/nxp-mcuxpresso/spsdk/issues/34, and 2.1 is indeed a breaking change, see https://github.com/Nitrokey/pynitrokey/pull/524.  The next pynitrokey release will fix this, either by merging the linked PR to make it compatible with spsdk 2.1, or by vendoring the required components of spsdk directly in pynitrokey, see https://github.com/Nitrokey/pynitrokey/pull/519.

Thanks  a lot @robin-nitrokey that was quick! :-)


Drag and drop from terminal editor to panel instance sometimes doesn't work

Not sure on repro, happens and I drag back and forth and make splits. It seems to be because the terminal editor's onRequestAddInstanceToGroup event isn't being listened to by TerminalService in this case.

Repro:

Open a terminal editor
Split via the editor action button
Try to drag that new terminal into the panel

Terminal editor instances created via TerminalEditorInput.copy don't get their TerminalService listeners setup:
https://github.com/microsoft/vscode/blob/855d764df58f3f53c41e4a9dd9e3093e678a5574/src/vs/workbench/contrib/terminal/browser/terminalEditorInput.ts#L32-L36
Needs to call:
https://github.com/microsoft/vscode/blob/1d3a7b6bc0ad09c1320e929ef70bef600cbeb094/src/vs/workbench/contrib/terminal/browser/terminalService.ts#L703-L737

I think the fix will register the listeners multiple times for most terminals due to other calls to _initInstanceListeners

I did some logging and didn't see that

The fix was on https://github.com/microsoft/vscode/compare/merogge/profileIcon, and we'll need to remove TerminalService.createInstance so all instances go through the same creation flow


sssd: add meta.pkgConfigModules, passthru.tests.pkg-config

Description of changes

Add meta.pkgConfigModules and passthru.tests.pkg-config

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

@ofborg build sssd sssd.tests

All the tests pass locally. Let me try re-running ofborg.
@ofborg build sssd.tests

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/1585

Rebased against master


My custom dataset has a class that dose not exist in kitti

I'm trying to train Pointpillar model with my own dataset whhich consists of three classes: Car, Bus, Truck.
I found in CUSTOM_DATASET_TUTORIAL.md that I have to map class names of my dataset to kitti class names, but there is no class in kitti  matches "Bus" in my dataset. How can I deal with this?

just match "bus" with "car"  and it just has effects on  the evaluation.

Thanks.
But I also have “Car" in my dataset, " there is also the class 'Car.' Will mapping 'bus' to 'car' cause confusion between these two classes?
Besides i wonder what to do while training?

MAP_CLASS_TO_KITTI: {
    'Vehicle': 'Car',
    'Pedestrian': 'Pedestrian',
    'Cyclist': 'Cyclist',
}


Add pre-commit hooks.  Closes #78.

Sets up configuration for https://github.com/pre-commit/pre-commit and gets our repo to pass those checks, but does not install pre-commit (since there's no python virtualenv / gem list / whatever).
People with Python and pre-commit will get source code checking; those without Python or pre-commit shouldn't notice any difference.

@dlgallagher , @tomelm @asottile : How does this look for a multi-language-friendly pre-commit install?

fwiw according to buck python is installed more often than bash so "not having python" in general isn't a problem.
That said, at the least you can include them in CI to keep your stuff working well.  Those that install them during development'll get faster feedback.
pre-commit aims to be useful to any language, though hooks haven't necessarily been written for all of them yet :D


don't use issubdtype to check for integer dtypes in polyval

Why
While running the "make" command for building the documentation, I had the following error :
(I am on Windows 11 and using Anaconda Prompt as my CLI)
WARNING: ources... [ 98%] user-guide/computation
>>>-------------------------------------------------------------------------
Exception in C:\Users\91942\xarray\doc\user-guide\computation.rst at block ending on line 466
Specify :okexcept: as an option in the ipython:: block to suppress this message
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
Cell In[83], line 1
----> 1 xr.polyval(coord=x, coeffs=out.polyfit_coefficients)

File ~\anaconda3\envs\xarray-docs\lib\site-packages\xarray\core\computation.py:1958, in polyval(coord, coeffs, degree_dim)
   1954     raise ValueError(
   1955         f"Dimension `{degree_dim}` should be a coordinate variable with labels."
   1956     )
   1957 if not np.issubdtype(coeffs[degree_dim].dtype, int):
-> 1958     raise ValueError(
   1959         f"Dimension `{degree_dim}` should be of integer dtype. Received {coeffs[degree_dim].dtype} instead."
   1960     )
   1961 max_deg = coeffs[degree_dim].max().item()
   1962 coeffs = coeffs.reindex(
   1963     {degree_dim: np.arange(max_deg + 1)}, fill_value=0, copy=False
   1964 )

ValueError: Dimension `degree` should be of integer dtype. Received int64 instead.

In the file computation.py(present in xarray/core), the program raises an error if the function np.issubdtype() returns false. Here, it wanted coeffs[degree_dim] to be of int data type (windows default int data type is int32) , but it was of int64 datatype, and as the documentation of np.issubdtype() states here : "Similar types of different sizes are not subdtypes of each other" so the function returned false and the program raised an error.
To fix this error, I have changed the following :
What
I replaced the np.issubdtype call with obj.dtype.kind not in "i"  to check if coeffs[degree_dim] is a general signed integer data type.

this happens with any code that calls polyval on coefficients with int32 dtypes. The example from the docs:
In [19]: import xarray as xr
    ...:
    ...: x = xr.DataArray(np.arange(10), dims=["x"], name="x")
    ...: a = xr.DataArray(3 + 4 * x, dims=["x"], coords={"x": x})
    ...: out = a.polyfit(dim="x", deg=1, full=True)
    ...: out
Out[19]:
<xarray.Dataset>
Dimensions:               (degree: 2)
Coordinates:
  * degree                (degree) int64 1 0
Data variables:
    x_matrix_rank         int64 2
    x_singular_values     (degree) float64 1.358 0.3963
    polyfit_coefficients  (degree) float64 4.0 3.0
    polyfit_residuals     float64 4.522e-28

In [20]: xr.polyval(coord=x, coeffs=out.assign_coords(degree=out.degree.astype("int64")))
Out[20]:
<xarray.Dataset>
Dimensions:               (x: 10)
Dimensions without coordinates: x
Data variables:
    x_matrix_rank         (x) int64 2 4 6 8 10 12 14 16 18 20
    x_singular_values     (x) float64 0.3963 1.754 3.111 ... 9.899 11.26 12.61
    polyfit_coefficients  (x) float64 3.0 7.0 11.0 15.0 ... 27.0 31.0 35.0 39.0
    polyfit_residuals     (x) float64 4.522e-28 9.044e-28 ... 4.07e-27 4.522e-27

In [21]: xr.polyval(coord=x, coeffs=out.assign_coords(degree=out.degree.astype("int32")))
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
Cell In[21], line 1
----> 1 xr.polyval(coord=x, coeffs=out.assign_coords(degree=out.degree.astype("int32")))

File .../xarray/core/computation.py:1972, in polyval(coord, coeffs, degree_dim)
   1968     raise ValueError(
   1969         f"Dimension `{degree_dim}` should be a coordinate variable with labels."
   1970     )
   1971 if not np.issubdtype(coeffs[degree_dim].dtype, int):
-> 1972     raise ValueError(
   1973         f"Dimension `{degree_dim}` should be of integer dtype. Received {coeffs[degree_dim].dtype} instead."
   1974     )
   1975 max_deg = coeffs[degree_dim].max().item()
   1976 coeffs = coeffs.reindex(
   1977     {degree_dim: np.arange(max_deg + 1)}, fill_value=0, copy=False
   1978 )

ValueError: Dimension `degree` should be of integer dtype. Received int32 instead.

this looks good to me, but we'd need some tests to make sure we don't reintroduce this issue. I'd probably extend the "simple" case of xarray/tests/test_computation.py::test_polyval by creating two versions: one with int32 values for degree and one with int64 values

@Ravenin7 do you need some guidance on how to add a test for this? Happy to help if so. It would be great to get this fix merged soon because a number of other people have encountered the same bug!


Update bug_report.yml [Invite users to explicitly state what model they are using that resulted in the issue]

What
This modification updates the "create bug" interface and asks the user to explicitly mention the model used in their experiments.
Why
Frequently, many issues can be better understood by reviewing the model used. For example, small models have limitations in their ability to generate code in code blocks or follow complex instructions leading to errors.
Before/after

Why are these changes needed?

Related issue number

Checks

[ ] I've included any doc changes needed for https://microsoft.github.io/autogen/. See https://microsoft.github.io/autogen/docs/Contribute#documentation to build and test documentation locally.
[ ] I've added tests (if relevant) corresponding to the changes introduced in this PR.
[ ] I've made sure all auto checks have passed.

Do you think it would also be helpful if we asked them to specify which OS/environment they're working in as well? A lot of the time bugs are caused due to platform incompatibility as well.

Thanks for this!


can't exclude app paths with exclude_paths

If I try:
exclude_paths: ['app/assets/javascripts/i18n','vendor/assets/javascripts']

or just:
exclude_paths: ['app/assets/javascripts/i18n']

The i18n directory still gets checked. The vendor stuff is excluded, no problem. Do I need to change the files declaration also?

I am having a VERY similar issue. Tried all manner of combinations of things with a directory inside of my javascripts directory and it did not exclude it.

Sorry about this folks - I can see how this would be causing you some pain based on the example you provided.
I'll see what I can do about this to make it more flexible - rest assured I'll get something together to fix this :smile:

This has been addressed in this PR. Sorry for the wait :(


Custom error message instead of "xx is a required property"

# This is my schema
schema = {
    "type": "object",
    "properties": {
        "pid": {"type": "integer", "error": (4001, "pid error")},
        "email": {"type": "string", "error": (4002, "email error")},
        ...
    },
    "required": ["pid", "email",  ...]
}

# This is how I process the errors
my_data = {
    "pid": 1,
    "email"<EMAIL_ADDRESS>}
v = Draft7Validator(schema)
err_list = []
for error in v.iter_errors(my_data):
    err_list.append(error.schema["error"])
print(err_list)

My requirement is to get all validation errors, so I used iter_errors to get them.
But I only can retrieve the error field that I defined myself when the pid has data but the wrong type. I can not get the custom error field if I don't post pid in my_data.
Does anyone know the resolution? (I must add the custom error message for every field)

Hi. This sounds like a duplicate of #119. Please have a look there, and certainly a PR is welcome.


improve allOf merging for composite schemas

Using allOf left some holes in the diffing - this PR merges the rest of the schema together and adds a test which changes the type of a merged schema.

Thank you for your work @damianw !


Update Composer dependencies (2021-06-16-00-12)

Visual regression test passed!
View the visual regression test report


cannot delete object related to another object

IntegrityError: https://github.com/acdh-oeaw/apis-core-rdf/issues/809#issuecomment-2096059131
create entity A
delete entity A
=> works
create entity A
add relation to another entity B
delete entity A
=> doesn't work

We are able to reproduce the error and are looking for a solution


What is difference between tensorflow version and pytorch version?

What is difference between tensorflow version and pytorch version?
and why did you change? (split of training dataset and add dropout layer)

To Optmize probably , I'm not sure but i think that is the reason.


KeyError when Calling Answer Box

I've attempted to get the results from the answer box using the documentation here.
I noticed the Playground does not return these results either.
Is there any way to get this URL also?
Output Returned when Attempting to Run the Sample Provided:
from serpapi import GoogleSearch

params = {
  "q": "What's the definition of transparent?",
  "hl": "en",
  "gl": "us",
  "api_key": ""
}

search = GoogleSearch(params)
results = search.get_dict()
answer_box = results['answer_box']

Hi @ag-skyler. This was fixed some time ago.

(Sorry for the delay in the response. You can open bug reports on https://forum.serpapi.com, we're checking these boards on a daily basis.)


Remove SSM Resources from Manifest Pipeline CloudFormation Stack

The developers are wanting SSM resources (Parameter Store) and their values to be managed outside of the Manifest Pipeline CloudFormation.
Once this is done, we will also need to update the CloudFormation Role Policy to remove the Delete and Put options.

Resolved by #40


INIFile buffer is not enough

Bug will occur when enable all FATEs and restart DFAssist.
Double the buffer will solve it.

Thank you for the information :)
Solved it now.


Disable tests that don't compile in release, which compiling release

I like https://github.com/swift-aws/aws-sdk-swift-core/pull/134 more, since that keeps test-coverage the same. However, I'm open-minded!

Deleted in favour of #134


PhilPapers

Language: primarily but not exclusively English
Date ranges: 1600s-2020
Size: +260,000 works indexed and +52,000 available to download via PhilArchive
PhilPapers is an international, interactive academic database of journal articles for professionals and students in philosophy. It is maintained by the Centre for Digital Philosophy at the University of Western Ontario.
I think we could at least add the section of open access works, which should all be downloadable as PDFs from https://philarchive.org/ There may also be other works availabe for download on the main site. In addition, we could probably scrape abstracts for the rest.

On it! Should be done in a few days. Quick sampling of about 1K documents from https://philarchive.org/ give an estimate of about 3GB of total text. This is without language filtering, as there are some documents in Russian and Italian (at least). It looks like this may knock it down 10-20%.
While the website claims 52K, the API reports around 45K. Many articles are duplicated or have been deleted/retracted.

@thoppe It would be best if we don't do language filtering.

This is complete. https://github.com/thoppe/The-Pile-PhilPapers
 ✔ Saved to data/PhilArchive.jsonl
 ℹ Saved 33,990 articles
 ℹ Uncompressed filesize 2,610,566,629
 ℹ Compressed filesize     79,7708,027

The issue can be closed and marked done on the board.


Change to minutes does not affect (unit_of_measurement)

What happened?
If i change from seconds to minutes for the sensor (unit_of_measurement) then the value stays the same.
How can i let me show the irrigation time in minutes instead of seconds?
How to reproduce
.
Relevant log output
.

Which version are you running?
.
Diagnostics file

[X] I have attached a diagnostics file

Additional information
.

I do it with a helper template. I find keeping things in seconds form all the way till display is much easier to manage as no time formatting conversions need to take place till you want to display it.  I just use a helper template and use this to convert the seconds into "hours:minutes:seconds". Hope this helps.
{{ timedelta(seconds=states('sensor.your_sensor') | int(0)) }}

See comment above. Closing as won't fix.


fix(a14): adding a14 to peerDependencies

v13.1.1 has been released and contains a fix for the issue. Feel free to reopen the issue or to submit a new one if you meet any problems.


Check for is not None for primitive fields in as_dict()

Fix #103

Integration tests are passing:


feat(server): Add an ssl example

Adds an example using hyper + ssl for a server.
Should resolve the closed issue #1942

any reason to change 100644 → 100755 ?

Totally, not sure why/how the permissions go changed on these files! Updated accordingly.

https://github.com/ctz/hyper-rustls/blob/master/examples/server.rs

Note: I think this might currently be broken due to this change: https://github.com/hyperium/hyper/commit/71d088d3d0c062aa05459697f53bf15018cfd651, at least until tokio updates their dependencies to future 0.3.1 as well.
@Darkspirit that's awesome!! If you'd like to replace this with an example from that repo that'd be great. It just seems like for such a simple setup there should be an example to point people towards if they happen to want to have SSL on their server.

Thank you for trying to merge a tls server example at all! :) I would be very grateful if you could update this pull request to use (the well received) rustls instead of non-rusty native-tls. Rustls and its dependencies webpki and ring are for example used by time.cloudflare.com, Google Fuchsia and Baidu. (Ring is based on Boringssl, Google's improved fork of OpenSSL, but got lots of C and C++ replaced with Rust.) It's quite sad that native-tls is spreading a bit like cancer, especially because it doesn't use Rust at places where its safety guarantees are most useful.

Oh totally! Wasn't even aware of the differences/benefits!! Always happier
to use the rusty alternative anyway!
On Sat, Nov 16, 2019, 9:40 AM Darkspirit<EMAIL_ADDRESS>wrote:

Thank you for trying to merge a tls server example at all! :) I would be
very grateful if you could update this pull request to use (the well
received) rustls instead of non-rusty native-tls. Rustls and its
dependencies webpki and ring are for example used by time.cloudflare.com,
Google Fuchsia and Baidu. (Ring is based on Boringssl, Google's improved
fork of OpenSSL, but got lots of C and C++ replaced with Rust.) It's quite
sad that native-tls is spreading a bit like cancer, especially because it
doesn't use Rust at places where its safety guarantees are most useful.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/hyperium/hyper/pull/1990?email_source=notifications&email_token=AAJWOLP4HNXGC7AZC55GEELQUAA4RA5CNFSM4JDKM6IKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEEHS6YI#issuecomment-554643297,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJWOLNU3TEJ6EBS22UA373QUAA4RANCNFSM4JDKM6IA
.

Checking back in on this -- I have a PR open to the tokio-tls repo to add rustls as a tls option. That being said, it looks like that might take a while to get in, which is unfortunate.
While experimenting though, I also noticed that the way the code is currently written, a long-lived http call will block request execution. Does anyone have any ideas around this? Basically because filter_map does not continue accepting sockets, we would need a stream op that always takes the next item and forms a new stream whose order is based on the completion time of an async closure run on each element of the original stream.

This looks awesome, it would be great to have an SSL example in the repo. I've had a search around an most of the examples I found of SSL/hyper are outdated because the async code has been moving so fast. Any updates on this?

I'll try to get back on this now that https://github.com/tokio-rs/tls is a
real thing!
On Thu, Mar 12, 2020 at 8:22 PM Odin Dutton<EMAIL_ADDRESS>wrote:

This looks awesome, it would be great to have an SSL example in the repo.
I've had a search around an most of the examples I found of SSL/hyper are
outdated because the async code has been moving so fast. Any updates on
this?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/hyperium/hyper/pull/1990#issuecomment-598488621, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJWOLKKGH6QTSOW3REA76TRHF4EHANCNFSM4JDKM6IA
.

Thanks for the PR! I'd rather not include examples in the hyper repo selecting specific TLS libraries. I think something like this could be part of hyper-tls, once it gains a server API.


Switch to release based on commit message

Right now releases are based on digest change. This has some issues that I'd like to avoid. Switch to using commitlint and a standard commit format and determine if a release should happen based off that.

Actually nevermind


centralise client setup and access of api key

Overview
I've refactored the OpenAI client initialisation into a single module to clean up the codebase and make it more DRY.
Following this (if people would like), I want to add a mock OpenAI client (which can be used if a specific env var is set). This addition will enable contributors to test the project more easily and without a valid API key. The centralised client setup change should make adding this mock client more straightforward.
PR Feedback
I've introduced pydantic-settings for environment variable management (suggsted by @gruckion - see commit: link). I'm open to any feedback or concerns regarding the introduction of this new library (requirements.txt soon?).
Pydantic-settings streamlines our setup and eases the addition of APIs like Pinecone in the future, a point @daveshap touched on in his recent video about enhancing agent communication.

looks good 👍 Can be merged

It looks like this conflicts with another PR. Please take a look and rectify. Let's get these fixes merged!

Merge conflicts resolved @daveshap, and I've also added in client = get_openai_client() in a few more places where I spotted we were setting up clients.


No way to access the current element?

Hi, first of all thank you for this cool lib, however I find it strange to not able to access the current input on the onChange function, how you can get the input that the color changed for?

Your condescending way of phrasing this aside, thank you for reporting this!
It wasn't implemented simply because I don't need it, and seeing how this is a tool I created for my own use, there is nothing strange about not adding something I don't need.
Anyway, I will add that soon as it does make sense.

0.20.0 has been published to NPM with this fix


Increase QUIC code coverage

Signed-off-by: Ryan Hamilton<EMAIL_ADDRESS>Fixes: #23128
Risk Level: Low
Testing: Lots more tests :)
Docs Changes: N/A
Release Notes: N/A
Platform Specific Features: N/A

/assign @alyssawilk

didn't this have an associated issue we can mark as being fixed?

didn't this have an associated issue we can mark as being fixed?

Indeed. Updated!


feat(docs): replace contentlayer with content-collections

This PR replaces contentlayer with content-collections. Idea for this came from https://dub.co/blog/content-collections and @shadcn's recent tweet. contentlayer hasn't received updates in a while, with unecesarry warnings in dependancies. content-collections seems like the natural evolution of it, providing a zod api for schema validation.
Few other changes that make it here
Remark + plugins -> update to latest versions
Rehype + plugins -> update to latest versions
unist + unist-tree-builder -> update to latest version + update types
shikijs@compat -> shiki latest version, had to change some of the css classes as well
mdast-util-toc -> update to latest version
Many thanks to @sdorra, the creator of content-collections for his help with this PR.
Please let me know if anything needs changed.

oh woah! thank you. running ci...

Fetched the upstream main and sorted out the issue where it couldn't find the content collections cli command


Update NYCSubwayIcons.css

Force a better circle shape, independent of font used

Hey, thanks for contributing this! The actual subway icons are written in the .sass file, so if you know how to use sass I would prefer if you updated that. If you don't I can absolutely port your fixes to the .sass file for you. I'll have a look at this pull sometime tomorrow. Thanks for committing!

I'm horrible at GitHub.  Sorry about the multiple commits in this pull request.    I'm new to sass, so excuse if the new rules are in the wrong place

No apologies needed! We all start somewhere =). However you are adding things which are specific to the text's location in the circle and your sass won't compile because you left the semicolons in. So I can't merge this. However it has made me aware of the fact that there should probably be more in the readme about actually implementing these icons. The code we have here is, by design, agnostic of sizes for the font and circle. This is because different people may want different sizes.
So the way to implement it is like so:
<element class="subway-icon subway-circle-1">
    <element class="subway-text">1</element>
</element>

This way you can set the circles size and font size to whatever you want in your code.
.subway-icon
    height: 20px
    width: 20px

.subway-text
    font-size: 14px

Does that make sense to you? What this library provides is the building blocks for creating a circle, with mta colors that has a perfectly centered helvetica font inside the circle across all devices which is white for all lines except the NQR where it is black. Sizing is left up to whomever implements it. If you want to contribute sass code, please install and run sass on the .sass file to create the .css file. You've made me aware that I should probably give some sort of contributing guide, so i'll add that as an issue to be added to the readme.
Let me know if you need any more help implementing the library. expect some updates shortly addressing some of your concerns.


Add google_compute_network resource

Q
A

🐛 Bug fix?
no

🚀 New feature?
yes

⚠ Deprecations?
no

❌ BC Break
no

🔗 Related issues
#999

❓ Documentation
yes

Description
This PR implements new resource google_compute_network.

Conflicts must be resolved


Multiplatform support

Is it on roadmap to add support to Kotlin multiplatform?

Hi Javier,
That's not something that we have on the roadmap at this stage. Maybe in the future?

Btw, we are always open to contributions from the community 🙂


Revert "Make it an error to break doc macros (and fix existing broken macros)"

Reverts flutter/flutter#62071
docs-linux on cirrus is failing consistently after this change.

A forward fix for this has been submitted already: https://github.com/flutter/flutter/pull/62136

Cirrus on master is now failing for other reasons (can't resolve dependencies): https://cirrus-ci.com/task/5397529384714240

Cirrus on master is now failing for other reasons (can't resolve dependencies): https://cirrus-ci.com/task/5397529384714240

Ah, okay, I will close this, but this is generally the reason to prefer reverts---makes it more clear that there is some other problem when the tree gets redder.


everything-toolbar-np: Add version 1.0.5

Relates to #88

[x] I have read the Contributing Guide.

/verify

/verify


Made the datastore tests less flaky

TODO

[x] Upgrade google/php-tools to 0.6.3

Personally, I'd be ok just hardcoding a number, or incrementing the default in https://github.com/GoogleCloudPlatform/php-tools

@bshaffer
I think the current default 3 in the tool is somewhat appropriate. I looked at the logs of the recent failures and realized that we should use separate namespace for each test setup, which will likely be sufficient for stabler test result.
Also, I'd like to have an easy way to experiment with the retry count once if the datastore tests become flaky.

We could still do better than this, i.e. have the Trait use a default static variable eventuallyConsistentRetryCount and then override it in the test classes. Passing it into every single call is not optimal.

@bshaffer Yeah I tried that path first and I agree. https://github.com/GoogleCloudPlatform/php-tools/pull/19

@bshaffer PTAL

Running the tests on jenkins

The tests on our jenkins are not green so far for several failure.

Datastore eventually consistency error
I hadn't set DATASTORE_EVENTUALLY_CONSISTENT_RETRY_COUNT when I saw this error, so I set it to 10.

Some 502 errors after flex deployment
Sigh, it is still happening. I raised GOOGLE_DEPLOYMENT_DELAY to 60.

The tests on jenkins are still somewhat flaky, but the last build was green.
Kicked off the travis build, and I'll merge once it's green too.


WidgetsBindingObserver does not work on Desktop

Flutter (Channel stable, 3.0.0, on macOS 12.3.1 21E258 darwin-arm, locale en-NO)
WidgetsBindingObserver is working fine on Mobile, testing with Android and iOS.
With Desktop it won't work on any of the desktops, MacOS, Windows or Linux. There doesn't seem to be a solution for refocusing desktop?
This is kinda crucial for many reasons.
Here's some sample code
class _InitAppState extends State<InitApp> with WidgetsBindingObserver {
  @override
  void initState() {
    WidgetsBinding.instance.addObserver(this);
    super.initState();
  }

  @override
  void didChangeAppLifecycleState(AppLifecycleState state) {
    if (state == AppLifecycleState.resumed) {
      refreshData(context); // Contains 11 logicals around refocusing the app. Flutter Desktop won't fire.
    }
  }

  @override
  void dispose() {
    WidgetsBinding.instance.removeObserver(this);
    super.dispose();
  }

}

Hi @ollydixon, thanks for filing the issue.
I see there's an open issue addressing the case you described as @maheshmnj provided above.
Closing this issue as a duplication, so please follow up on it for further updates.


layers: Fix vkCmdCopyImage intersection check for array layer/mip level

RegionIntersects would return true if the source and destination array layers and mip level did not overlap, resulting in a spurious validation error when trying to copy between different array layers or mip levels of the same image.

Edited commit message for length, tested, and pushed.  Thanks a bunch!


File size restriction updated to 5 mb

r? @ustulation
(maidsafe_highfive has picked a reviewer for you, use r? to override)

r? @shankar2105

Reviewed 4 of 4 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from the review on Reviewable.io


Add models

Models to add:

[ ] Mistral
[ ] Llama
[ ] Gemini
[ ] Claude V2

Other changes

[ ] Metaclass for automatic class registry
[ ] Really up the documentation for people can see how to add their own models
[ ] Get advice from @apostolosfilippas how we make config file gracefully handle (many) tokens

@apostolosfilippas Can we discuss some general way of adding more models w/o having to do lots of surgery on edsl.Config for each new model? Maybe a separate "key" secret file for each model?


Add a chocolatey badge

Pull request checklist:

[x] CHANGELOG.md was updated, if applicable

Description
This pr adds the chocolatey badge on the readme

Closes #1981

Package approved: https://chocolatey.org/packages/chatterino

Looks good to me :+1:


chore: use rust 1.66.0

This PR passed the format, lint, build and test, but it was failed in Ensure Wasm up to date in my repo and local computer(windows 11), I am not sure this PR is no problem, if has any problem, please tell me.

@linbingquan you have to build the wasm using the build task

@linbingquan you have to build the wasm using the build task

Thank you for your reply.
I rebase to the main branch with deno task build.


Is it ok that field's type option is not in attrs?

It confuses a little.

I agree. However the value of the type option can be also textarea and static, so currently it's more the input type rather than the type attribute.

Ah, ok, got it)


frame-generator example doesn't render materials in .gltf

Hi there,
As suggested in this question, I've been converting one of my .objs using obj2gltf as a conversion tool. I can see the material nodes in the .gltf are assigned (but I'm not an expert with this format).
Using the example frame-generator when I provide my .gltf as input I'm unable to visualize the actual material, which I think is overwritten by the .mat file compiled using the matc generator.
Could you please point out if there's a way around this problem?
Do I need (how?) to disable the .mat used by the sample? Do I need to hack the code somewhere to use the materials provided by the .gltf?

frame_generator is designed to generate multiple frames with varying values for the parameters of a supplied material. What are you expecting frame_generator to do with your glTF file and its built-in materials?


Extend and clarify documentation

Make it more obvious how navbar menus and links interact with each other.
The Helper is really great!
I just had a hard time figuring out that I need a menu for horizontal links and that I need
echo $N->beginMenu();
echo $N->beginMenu("Menu Title");
echo $N->endMenu();
echo $N->endMenu();

for a single dropdown menu.

Current coverage is 49.66% (diff: 100%)

Merging #94 into master will not change coverage

@@             master        #94   diff @@
==========================================
  Files             9          9
  Lines           904        904
  Methods          86         86
  Messages          0          0
  Branches          0          0
==========================================
  Hits            449        449
  Misses          455        455
  Partials          0          0

Powered by Codecov. Last update 522f2b4...cb97cfd

Thanks for the update. A menu is a <ul> actually, and since in bootstrap the horizontal links are in a <ul>, you need an outer beginMenu().
I need to find time to write a proper documentation for the helpers, the current one is not covering the new helpers...


[1Password] Work with multiple vaults at the same time

Extension
https://www.raycast.com/khasbilegt/1password
Description
I'd love to be able to connect to multiple vaults at the same time
Who will benefit from this feature?
anybody who has multiple vaults
Anything else?
No response

I'd definitely like this feature.

This would be awesome. I have a work account and a personal account. When installing the extension I chose my personal account, and as far as I can tell there is no way for me to switch to my work account.
I want to be able to see the passwords of both accounts in the same raycast window.

This definitely would be amazing, rather than having to use the 1Password Quick Access to search for a password which does allow to work with multiple accounts, I would prefer to use this with Raycast instead.

I saw more issues have been opened about this, adding my comment just to say this would be really nice

Same here, but would like the option to choose which vaults it should search. Right now I'm using 6 valuts, but when I'm at work I need it to search only my "work vaults".. It seems to me now that is searches all vaults so would be nice with an option to be able to disable/enable which vaults should be included in the search.

@Luk4h How do you tell Raycast which account you want to use? It seems to choose (what seems like) randomly for me.

Per the 1Password documentation, it auto sign-in on the lastest signed on account. If you reset the extension cache it should ask for the account.
On another note, I am working on an action that will allow to switch to another account, as well other vaults.
I am looking for input on the Vaults... Currently I think the approach should be a new command "My Vaults" where you are allowed to select the vault you want to search the passwords on.

I'd love that feature too. The Alfred App (https://alfred.app/workflows/alfredapp/1password/) searches all vaults per default, this is one of the features I miss most about my whole Raycast experience.

I just ran into this issue when adding my work account to my 1password. I am running v8.
Raycast kept throwing an error until I ran the op cli command myself and it was asking for which vault to default to. Then this extension started working again. However I don't see in this extension's preferences to select which account to default.
To fix this issue I think it would be handy to keep a list of the current accounts logged in to change what vaults to search. And on top of that which vault in said account to list as well.

With the lastest update, the extension is able to use multiple vaults, filter items per vault, and switch accounts in extension. Right now there are no support for retrieving the passwords for multiple accounts at once. I will be closing this issue, since it is named as "Work with multiple vaults at the same time". And as for now, it has been fixed.
@raycastbot close this issue

Thanks for sharing! When is this shipping?
--gonto
On Tue, May 21, 2024 at 10:03 AM, raycastbot < @.*** > wrote:

Closed #4744 ( https://github.com/raycast/extensions/issues/4744 ) as
completed.
—
Reply to this email directly, view it on GitHub (
https://github.com/raycast/extensions/issues/4744#event-12879898128 ) , or
unsubscribe (
https://github.com/notifications/unsubscribe-auth/AAFQWCZPV7HCULRQ62IWVW3ZDNAZDAVCNFSM6AAAAAAUZDUKUOVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJSHA3TSOBZHAYTEOA
).
You are receiving this because you authored the thread. Message ID: <raycast/extensions/issue/4744/issue_event/12879898128
@ github. com>

It has been shipped! You should be able to use it right now, make sure to run "Check for extension updates" or
raycast://extensions/raycast/raycast/check-for-extension-updates

I'm up to date but it's not working. When I go to My Passwords I get the following
Thanks!
--gonto
On Tue, May 21, 2024 at 1:30 PM, Luiz Gustavo Marcondes < @.*** > wrote:

It has been shipped! You should be able to use it right now, make sure to
run "Check for extension updates" or
raycast://extensions/raycast/raycast/check-for-extension-updates
—
Reply to this email directly, view it on GitHub (
https://github.com/raycast/extensions/issues/4744#issuecomment-2123009347
) , or unsubscribe (
https://github.com/notifications/unsubscribe-auth/AAFQWC5LOV2EIVWOQLNNSYLZDNZDZAVCNFSM6AAAAAAUZDUKUOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMRTGAYDSMZUG4
).
You are receiving this because you authored the thread. Message ID: <raycast/extensions/issues/4744/2123009347
@ github. com>

I belive this issue is related to #12492, waiting for the merge on that one yet. 😢


Unexpected behaviour in Python API: time should be in quotes in select

I found that if you are using simple select query "select * from measurement where t = X" without quotes, result becomes unpredictable. For it's unclear why I should use quotes for time, if time is integer (or long).
from time import sleep
from influxdb import client as influxdb

db = influxdb.InfluxDBClient("localhost", 8086, "", "", "queue")

measurement = "wtr"

def drop():
    db.query("drop measurement " + measurement)

def show_all():
    print db.query("select * from " + measurement)

def check_one_way(t):
    result = db.query('select * from {0} where time="{1}"'.format(measurement, t))
    return bool(result)

def check_second_way(t):
    result = db.query('select * from {0} where time={1}'.format(measurement, t))
    return bool(result)

def write_to_influx(exists):
    t =<PHONE_NUMBER>231000000
    points = [{'fields': {'value': 1}, 'time': t, 'measurement': measurement}]

    if exists(t):
        print "Exists!!!", t
        return

    db.write_points(points)

    sleep(1)

    if exists(t):
        print "OK"  # Expected behaviour
    else:
        print "WTF?!"  # Unexpected behaviour

print "With quotes"
write_to_influx(check_one_way)
show_all()
print "="*80

print "Without quotes"
drop()
write_to_influx(check_second_way)
show_all()
drop()

I see this behaviour in the InfluxDB web interface too, so I guess this is a general InfluxDB thing (as per your original ticket) rather than specific to the Python library?

The documentation says that you should omit the quotes but append a code for the units of the timestamp... Sadly it doesn't appear to have a code for nanoseconds, but if you alter your code to look like this then it seems to work for your example:
result = db.query('select * from {0} where time={1}u'.format(measurement, t/1000))

Further investigation suggests that InfluxDB actually ignores any timestamp that you put in quotes!  For example, editing your check_one_way call to double the timestamp still returns the same result:
result = db.query('select * from {0} where time="{1}"'.format(measurement, t*2))

Addressed in #678


Add RequiredBracesMethods param to BlockDelimiters

The motivating use case for this in our codebase is to switch Sorbet sig declarations from do...end to {...}:
Syntax highlight for do/end on type signatures are distracting:

Braces are less distracting:

Currently the first is enforced by line_count_based, which we want for all other methods.

Before submitting the PR make sure the following are checked:

[x] Wrote good commit messages.
[ ] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Run bundle exec rake default. It executes all tests and RuboCop for itself, and generates the documentation.

I'm OK with the suggested change, but I think we should documented in the cop as well.

Good idea, done: https://github.com/rubocop-hq/rubocop/pull/7660/files#diff-650564e544ce47e9588aed04b1881391R110

Thanks @maxh! LGTM! 🚀

Thanks!


Could not extract repo info

Could not parse the following repo url could not extract user and repo name from github url git+https://github.com/tj/commander.js.git

This is from https://github.com/bahmutov/changed-log/issues/16


Add support for solaris - fix #69

Signed-off-by: kuba--<EMAIL_ADDRESS>Fixes https://github.com/src-d/go-billy/issues/69

@mcuadros gentle ping. 😅Working on Solaris support for Cirrus CI and would love to see this merged since our agent uses go-git for cloning which depends on go-billy.

https://github.com/src-d/go-billy/releases/tag/v4.3.1

@mcuadros greatly appreciate it! 🙌

Hey @kuba--, does seem like this change fixed the issue of building for Solaris. Could you please take a look at #71 where I've also added tests for building binaries for supported platforms.


Automated cherry pick of #3173: hostman: system_service: add IsEnabled() support

Cherry pick of #3173 on release/2.10.0.
#3173: hostman: system_service: add IsEnabled() support

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.10.0@480b87a). Click here to learn what that means.
The diff coverage is 73.68%.

@@               Coverage Diff                @@
##             release/2.10.0   #3176   +/-   ##
================================================
  Coverage                  ?   7.46%
================================================
  Files                     ?     414
  Lines                     ?   66192
  Branches                  ?       0
================================================
  Hits                      ?    4944
  Misses                    ?   60824
  Partials                  ?     424

Flag
Coverage Δ

#aFlag
7.46% <73.68%> (?)

Impacted Files
Coverage Δ

pkg/hostman/system_service/system_service.go
12.5% <0%> (ø)

pkg/hostman/system_service/sysv.go
47.22% <100%> (ø)

pkg/hostman/system_service/systemd.go
58.97% <80%> (ø)

Continue to review full report at Codecov.

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

/lgtm

/approve


How to set up customized ip address?

I am using the docker version on my macbook at it worked:
$ docker run -it -p 5000:5000 --rm -v pwd:/data cumulusprod/cirrocumulus cirro launch /data/pbmc3k.h5ad --host <IP_ADDRESS>
Change IP to <IP_ADDRESS> will run but cannot open the link. Says "This page isn’t working":
$ docker run -it -p 5000:5000 --rm -v pwd:/data cumulusprod/cirrocumulus cirro launch /data/pbmc3k.h5ad --host <IP_ADDRESS>
And changing the ip address to my macbook's ip address with not run:
$ docker run -it -p 5000:5000 --rm -v pwd:/data cumulusprod/cirrocumulus cirro launch /data/pbmc3k.h5ad --host [ip adress of my macbook]
...
OSError: [Errno 99] Cannot assign requested address
Question:
How can i set up with my ip address, or hostname?
Thanks,

Another question:
I was able to run the app with miniconda, but it cannot find files.
$ cirro launch ./data/pbmc3k.h5ad --host [host ip addr] --port 8080

[01/Aug/2020 21:46:08] "GET /api/filters?id=pbmc3k.h5ad HTTP/1.1" 200 -
[2020-08-01 21:46:08,763] ERROR in app: Exception on /api/schema [GET]
Traceback (most recent call last):
File "/apps/python/Miniconda/lib/python3.7/site-packages/flask/app.py", line 2447, in wsgi_app
response = self.full_dispatch_request()
File "/apps/python/Miniconda/lib/python3.7/site-packages/flask/app.py", line 1952, in full_dispatch_request
rv = self.handle_user_exception(e)
File "/apps/python/Miniconda/lib/python3.7/site-packages/flask/app.py", line 1821, in handle_user_exception
reraise(exc_type, exc_value, tb)
File "/apps/python/Miniconda/lib/python3.7/site-packages/flask/_compat.py", line 39, in reraise
raise value
File "/apps/python/Miniconda/lib/python3.7/site-packages/flask/app.py", line 1950, in full_dispatch_request
rv = self.dispatch_request()
File "/apps/python/Miniconda/lib/python3.7/site-packages/flask/app.py", line 1936, in dispatch_request
return self.view_functionsrule.endpoint
File "/apps/python/Miniconda/lib/python3.7/site-packages/cirrocumulus/api.py", line 127, in handle_schema
return to_json(get_schema_and_dataset()[0])
File "/apps/python/Miniconda/lib/python3.7/site-packages/cirrocumulus/api.py", line 119, in get_schema_and_dataset
schema = dataset_api.schema(dataset)
File "/apps/python/Miniconda/lib/python3.7/site-packages/cirrocumulus/dataset_api.py", line 46, in schema
value = provider.schema(self.fs_adapter.get_fs(path), path)
File "/apps/python/Miniconda/lib/python3.7/site-packages/cirrocumulus/anndata_dataset.py", line 57, in schema
return SimpleData.schema(self.get_data(path))
File "/apps/python/Miniconda/lib/python3.7/site-packages/cirrocumulus/anndata_dataset.py", line 50, in get_data
adata = self.read_adata(path)
File "/apps/python/Miniconda/lib/python3.7/site-packages/cirrocumulus/anndata_dataset.py", line 33, in read_adata
return anndata.read(path, backed=self.backed)
File "/apps/python/Miniconda/lib/python3.7/site-packages/anndata/readwrite/read.py", line 447, in read_h5ad
constructor_args = _read_args_from_h5ad(filename=filename, chunk_size=chunk_size)
File "/apps/python/Miniconda/lib/python3.7/site-packages/anndata/readwrite/read.py", line 486, in _read_args_from_h5ad
_read_key_value_from_h5(f, d, key, chunk_size=chunk_size)
File "/apps/python/Miniconda/lib/python3.7/site-packages/anndata/readwrite/read.py", line 510, in _read_key_value_from_h5
_read_key_value_from_h5(f, d[key_write], key + '/' + k, k, chunk_size)
File "/apps/python/Miniconda/lib/python3.7/site-packages/anndata/readwrite/read.py", line 510, in _read_key_value_from_h5
_read_key_value_from_h5(f, d[key_write], key + '/' + k, k, chunk_size)
File "/apps/python/Miniconda/lib/python3.7/site-packages/anndata/readwrite/read.py", line 544, in _read_key_value_from_h5
key, value = postprocess_reading(key, value)
File "/apps/python/Miniconda/lib/python3.7/site-packages/anndata/readwrite/read.py", line 541, in postprocess_reading
value = value.astype(new_dtype)
ValueError: invalid shape in fixed-type tuple.
[ip adress] - - [01/Aug/2020 21:46:08] "GET /api/schema?id=pbmc3k.h5ad HTTP/1.1" 500 -

@iiiir Thanks a lot for your interest in Cirrocumulus. @joshua-gould will look into your issue tomorrow.
Best,
Bo

Can you please confirm that the path to your h5ad file is correct?

@joshua-gould , If you are referring to my second post yes i can conform the path is correct. I was running under the same folder -- also tried full path which gave same error. HOWEVER, the run was in a conda environment, which I am not sure if the path should be specified otherwise.
The reason i use conda is because some dependent libraries cannot be installed as I am not admin for the cluster. e.g. cython and pandas (I do not have permission to uninstall older versions of cython).
I was able to run the command successfully with regular python environment (without conda), on my personal macbook.

Sorry for the delay in my response. Were you able to resolve these issues?


Proposed Direction for Fork Development

I think the pub feature, being able to make generic inscriptions without them being tied to the ordinal system, would be great for the ecosystem overall.
There are many potential use cases for inscriptions that don't fit into the ord protocol design as is. This includes protocols that would benefit from additional metadata, but there's no accessible way to add that when using the ord protocol. As an example, the pub protocol might benefit from having a field to specify an external public key and signature, so it can be (optionally) connected with other systems.
In this direction, there are parts of the original codebase that should be removed, such as the server code that produces the website. That's primarily the subcommand/server.rs and subcommand/preview.rs files, along with the associated points of interaction, like the parts of subcommand.rs that use Server and Preview. Other areas to remove include subcommand/traits.rs, decimal.rs, degree.rs, epoch.rs, media.rs, rarity.rs, and templates.rs.
There's more stuff that isn't needed for the raw inscription functionality, but that's a large chunk to get started cutting out and should help provide more understanding of the codebase.
As an additional note, I think a protocol should have a different identifier when it's desirable to be able to parse that particular group of inscriptions out of the blockchain and act on them in some way specific to the protocol, as well as when it doesn't make sense for the inscriptions to be connected to an ordinal, have a concept of transferring the inscription, or for their inscribed data to automatically show up in default explorers.
I've been working on a (currently unpublished) fork to implement arbitrary protocols and some related features. I think the goals are very complementary, but I do plan on keeping optional support for indexing. @jotapea, if you're interested, I'm willing to help, and perhaps we can combine our efforts more directly as things progress.

I have been thinking about how to proceed, because if this non-indexing variation of the ord software still produces valid ordinals, then it could continue using the same protocol_id. But at the same time... maybe it shouldn't?
What is the protocol as in the specific rules to adhere? But then, what prevents anyone from doing whatever anyway?
Having said this, and considering what you wrote in https://github.com/casey/ord/issues/1548, I have been thinking that for this eternal publishing concept we are discussing, an additional field that makes a lot of sense to me is the content hash.
This can auto-validate the protocol, as in the publications that don't match can be ignored (or treated different). And this could have its own focused content hash to transaction index, allowing avoiding writing duplicate data (but can be flexible, in like only focusing on certain amount of blocks).
Additional fields could make sense, can you elaborate on your external public key and signature use cases? Because I am thinking in the most fundamentally simple web files protocol and maybe it makes sense to only include what can apply to ALL content... or it can be optional.

I have been thinking about how to proceed, because if this non-indexing variation of the ord software still produces valid ordinals, then it could continue using the same protocol_id. But at the same time... maybe it shouldn't?

Perhaps it could, but I think it'd require modification to enable that, so I'd prefer to generalize inscriptions to achieve the same result. In my view, inscriptions aren't being used to their full potential without arbitrary protocol support.
Ordinals is primarily a set of two great innovations, of course the ordinal system, but also the inscription system. I consider the ord protocol to be one of many protocols that can be implemented on top of inscriptions, with another being your pub protocol.

Expanding on what you wrote in casey#1548, I have been thinking that for this eternal publishing concept we are discussing, an additional field that makes a lot of sense to me is the content hash.

Yes, that's a great point. I think a content hash field is something that makes sense at the top layer, for all protocols to have, since it's general purpose. A hash would be part of the overall metadata, then individual protocols can specify any additional metadata specific to their use case as key-value pairs in a header object.
This would allow the system to be used to easily define arbitrary protocols, working more like a developer making an app than needing to get into the really low-level stuff needed to make an inscription protocol as it currently stands. I think that should enable quite a rich ecosystem to be built on top of Bitcoin. Sometimes it can make sense to use the ordinal system, but for other use cases it can be better to use these generic inscriptions.

This can auto-validate the protocol, as in the publications that don't match can be ignored (or treated different). And this could have its own focused content hash to transaction index, to avoid writing duplicate data (but can be flexible, in like only focusing on certain amount of blocks).

Yes, exactly, that's another great point. That functionality can be used to deduplicate data that's being written, and it can be a feature that a given protocol can use or not use (although perhaps on by default to be space, and fee, efficient). These are the types of features that can be implemented in a straightforward way when arbitrary protocols are supported.

so it can be (optionally) connected with other systems

Yes it can have multiple optional features.

Exactly. I provided the example of tying into an external public key infrastructure just as an example. It may or may not make sense for pub specifically, but as you've expressed in your proposals, it can be essentially any feature once the right interface is exposed for protocol developers to use. Features that have potential to be used across multiple protocols make sense to define at the top layer, and then specific protocols can implement additional features as individually necessary.

Ordinals is primarily a set of two great innovations, of course the ordinal system, but also the inscription system.

Agree. Ordinals introduced this data envelope standard which can now serve as a foundation for everyone to build on.
Maybe this repo doesn't need to focus on a single protocol id. While it can make sense to have a basic file publishing protocol id set as default ('pub' or another name), this could also serve as a bootstrap for any protocol id that wants to be based on inscriptions. So it can continue supporting the 'ord' protocol id in some capacity, at least in read/hash/index capabilities.

I'm working on preparing an initial release with the improvements I've described, so I don't think it makes sense to duplicate efforts in terms of indexing or similar. However, I think your original idea for a minimal tool to build and parse inscriptions has a lot of value in terms of allowing other programs to easily integrate inscription functionality. That can support arbitrary protocols as well, perhaps named something like sip (Simple Inscription Program) to reflect its focus.

@jotapea @tyjvazum I would like to participate in this effort as well.
maybe more can be discussed in another channel?
@sull

I've set up a discussion room on Gitter. It's easy to use, with just a simple link with your GitHub account as the sign-in method. Here's the link to join: https://app.gitter.im/#/room/#arb-proto:gitter.im

Maybe at this point is still better to continue the conversations in GitHub. @tyjvazum when can I expect to learn more about your efforts? As you said we should avoid duplicate work. @stet @sull welcome and glad you discovered this. Read you comment in https://github.com/casey/ord/issues/1501#issuecomment-1436436306 and it seems you are well aligned with the vision / potential we see for inscription-first protocols.

Maybe at this point is still better to continue the conversations in GitHub. @tyjvazum when can I expect to learn more about your efforts? As you said we should avoid duplicate work. @stet @sull welcome and glad you discovered this. Read you comment in casey#1501 (comment) and it seems you are well aligned with the vision / potential we see for inscription-first protocols.

We're already having some discussion on Gitter, but sure, I'm happy to converse on GitHub as well. I'm working toward an initial release as quickly as I can. I think it'll be relatively soon.

i noticed that the gitter is encrypted so i can't see any previous messages. maybe that should be changed? i like the direction being discussed.

@0attack glad to have you here!
@tyjvazum @stet can we just have all communications in GitHub? Even if @tyjvazum decides to make his code in another repo, we will still be able to reference each other.
I tried joining gitter and it asks to login and permissions and I just feel these add unnecessary friction to anyone that wants to participate in this effort. And its not only the friction, is also the dilution of the communication / information shared. And even more if the gitter is going to be public anyway... I just don't see clear benefits but see these drawbacks.
Just my opinion.

yeah whatever is easiest 👍

@jotapea, I'm fine communicating wherever people choose to, GitHub, Gitter, or even somewhere else. Gitter is being used to provide a chat room for Arbitrary Protocols. It simply provides another channel, with different features and a real-time style of communication. A lot of projects use Discord, which has much more friction, requiring an email address and phone number, and is not open source like Gitter, so I think it's a nice option.

@tyjvazum Any particular topic of interest that was discussed in the original room? Assuming you will make a separate repo for your project, will it be forked from casey/ord or will it be completely new?

@tyjvazum Any particular topic of interest that was discussed in the original room? Assuming you will make a separate repo for your project, will it be forked from casey/ord or will it be completely new?

could post screenshots, not too much in there i don’t think.

@tyjvazum Any particular topic of interest that was discussed in the original room? Assuming you will make a separate repo for your project, will it be forked from casey/ord or will it be completely new?

I'll post a screenshot or text file of it, depending on what works out best.
It's based on casey/ord, with changes to add functionality, but the same overall structure.

Not much but this is the meat of a brif chat.

Thanks for the screenshot @stet, all valuable comments. We all seem to be aligned so I'm looking forward to @tyjvazum work for next steps...

i found this aftr reviewing issues. awesome concept. i had a similar idea for a fork, even similar name arb but no time to code it yet. also don't know much rust. could you explain anything more?

Sure, it's based on casey/ord, but extended to support arbitrary protocols on top of Bitcoin. Developers can create a simple JSON specification file to declaratively express their protocol, which the software can then load sort of like an extension. I'm implementing a Bitcoin Identifier/Username protocol, which is helping me to implement the generalized system, and I also have some more protocols planned for the future.

Developers can create a simple JSON specification file to declaratively express their protocol

🚀🚀🚀

Sure, it's based on casey/ord, but extended to support arbitrary protocols on top of Bitcoin. Developers can create a simple JSON specification file to declaratively express their protocol, which the software can then load sort of like an extension. I'm implementing a Bitcoin Identifier/Username protocol, which is helping me to implement the generalized system, and I also have some more protocols planned for the future.

are you proceeding and any other discussion being done with ord team?

Yes, I am.
I just released the code that generates the list of domains for the sunrise period of the username protocol. I'll be releasing some specifications in the middle of the week, and then the client code later in the week. After that, it'll be an ongoing development effort.
There's no discussion currently, although I'm quite open about everything. However, there was some more chat in the Gitter if you haven't seen that.

alright sounds good, guess will wait for your preliminary release before contributing more.

I'm always open to feedback if you'd like to discuss anything sooner.
At the moment I'm working on the generalized system that provides cryptographic keys for the username protocol, and any other protocols, to use. It uses the Bitcoin Core wallet as a basis to generate Ed25519 keys from, and can be extended more in the future. Once that's done, most of the major features should be in place for the initial release.

I've released an example of the JSON specification file for the username protocol here. It's taking me a bit longer than anticipated to get everything ready, but I'm making solid progress. I'll continue releasing pieces as it comes together, with a full release ideally this month.
I also provided some information about an identicon system, that's a part of the username protocol, here. I came up with the concept after @RagnarLifthrasir started discussion in the Gitter. I think it's an interesting way to integrate the username protocol (and potentially others) with Bitcoin NFTs, as well as something that provides utility independently.
As I said in earlier messages, the overall goal of the project is to provide a surface that enables application developers to be able to create and deploy arbitrary protocols on top of Bitcoin, and I think that's advancing very nicely.
Some aspects of the designs will be much easier to adjust earlier rather than later, so feedback is encouraged and always welcome.

@tyjvazum Very cool identicons sample!

About the whole usernames discussion, I am not sure if they will work as a "unique namespace everyone will adopt". In the web, each domain has the freedom of creating their own usernames. The newest Bitcoin sub-protocol Ordinals have their own names. The old Counterparty sub-protocol has their own names.

I agree. It's, of course, not intended to be something that everyone has to adopt, and I'm sure there will continue to be others that people choose to use. It's just a novel option.
However, while it's not a goal for the project, there certainly is demand for universal usernames, as evidenced by the proliferation of OAuth via Big Tech services.
With regard to Ordinal names, that's a very different thing that has some significant tradeoffs (at least for the current proposal), which I went over in this comment.

I could see some potential for a DNS system, but unfortunately, there are also plenty of projects trying it. Could this be the one that "wins" and gets adopted by everyone, maybe. But I still believe the main effort should be focused on the features provided by these arbitrary sub-protocols.

Yes, exactly. It's mostly meant to help with direction for the underlying implementation, while also being a solid, decentralized naming system (for which there arguably isn't a good existing example of). It's created with features that are provided for any arbitrary protocol to use. I plan on using it in the future as a basis for related applications that need usernames.

The ordinal's inscription envelope PROTOCOL_ID is already serving the purpose of an open (and simple) top-level-domain.

Yes, that's what I'm building out infrastructure to make use of, but that only defines a protocol. It's not suitable for an individual identifier like a username since they need to be united within a common specification. Enabling use cases like that in an easy, developer-friendly way is the primary objective of the project.

I might be wrong and please don't get discouraged, but I just wanted to provide my sincere feedback. And having said all this, I still see a high potential in @tyjvazum's design approach.

I'm truly happy to have the feedback. Honest discussion is an incredibly useful tool for refining ideas. Thank you for the ongoing participation and encouragement 👍

Referencing some interesting approaches to add metadata to inscriptions:

https://github.com/casey/ord/issues/1859#issue-1603896380
https://github.com/casey/ord/issues/1878#issuecomment-1461018635

I've set up the repo for my fork arb here.
It doesn't include new features at the moment, but it has more information about the next steps I'll be taking, it gets the basic structure in place (which was a fair bit of work in itself), and it allows for feedback or questions in the issues.
I was planning on including Arbitrary Protocols, Content Compression, Hash-addressed Content, Inscription Metadata (JSON), and Off-chain Content (BitTorrent) today, but I want to do some more testing first and am currently waiting for reindexing to complete after the recent reorg (which I'd really like to implement a solution for). I'm aiming to release those features next week, maybe sooner if the final testing goes smoothly.
I look forward to any feedback and it'd be particularly helpful if someone can verify my signature on the v0.1.0 release to make sure everything was done correctly.

Thoughts on this proposed Bitcoin Ordinals DID Method Specification?

I've released an updated version of arb. It took more work and time than I'd initially hoped it would, as usual, and I'm sure there are bugs or oversights but I think it's a big step forward. It'd be great if anyone can help with testing, refactoring code, or writing documentation. I'll be continuing to work on the next batch of features.

Thoughts on this proposed Bitcoin Ordinals DID Method Specification?

@RagnarLifthrasir, I took my time thinking about it. It's somewhat compatible with what I'm working on, but I don't think treating an identity as connected to a satoshi is the best approach since I don't think identities are NFTs. I view the DID concept as primarily an interoperability mechanism and that's a useful thing, so I'll keep it in mind for my work.

@jotapea, I used your pub term as the protocol identifier for non-tracked inscriptions. I think it pairs well with the ord protocol naming. I'm still willing to help if you want to move forward with your idea for a minimal inscription reading/writing tool. If so, perhaps we should start a new issue for it since this one has grown a bit long.

I'll take it for a spin

@tyjvazum Thanks for your offer. I'm busy on other work at the moment, so I'm not sure when (or if) I'm coming back to this.
And it seems people are already using the ord envelope for arbitrary data as evidenced by the brc20 delusion lol. I'm pretty sure this people don't care about their assigned ordinal at all, is all about the inscribed data. Maybe they are already using something similiar to this repo but just keeping the ord protocol_id.
I glanced at your work and it looks very elaborate! What do you have in mind in terms of first proof of concept / prototype / use case to focus on? Do you have in mind (or are already working) on a product that will use this?

@tyjvazum Thanks for your offer. I'm busy on other work at the moment, so I'm not sure when (or if) I'm coming back to this.
And it seems people are already using the ord envelope for arbitrary data as evidenced by the brc20 delusion lol. I'm pretty sure these people don't care about their assigned ordinal, is all about the inscribed data. Maybe they are already using something similar to this repo but just keeping the ord protocol_id.
I glanced at your work and it looks very elaborate! What do you have in mind in terms of first proof of concept / prototype / use case to focus on? Are you working on a product that will use this?

There's been interest around BRC-20 support so that will likely end up being the first novel use case for arb, then my plan is to release the bid, bnw, and bfs protocols, probably in that order. All of this functionality can be implemented using a shared set of primitives, which is the arb design direction. Prototype code exists for these, but it needs to be polished into a cohesive whole as I put it out incrementally.
I have no plans for a commercial product, I just want to build useful things on top of Bitcoin, hopefully expanding and improving the overall ecosystem. I saw this potential sitting underutilized in ord and also saw it as an opportunity for me to learn Rust, which I've wanted to do but haven't had something useful to work on until starting this project.
Perhaps larger things will emerge going forward, or perhaps someone will want to use arb as the basis for a profit-seeking venture, but I'm having fun building and learning.
I'll close this issue since it seems to have served its purpose. If anyone wants to get involved just jump into the issues or discussions. As always, the more the merrier.


ORTC: Add RTCQuicTransport

Resolves #275

@Sean-Der I suggest we use the normal review process for merging into the ORTC branch.


Simplify the returned type in TypeSignature::get_possible_types

Is your feature request related to a problem or challenge?
get_possible_types returns Vec<Vec<DataType>> and convert to string for information schema.
Can we instead return the type define in each function as part of the documentation and return that instead?
We add return type and argument type in Documentation struct.
pub struct Documentation {
    // arg name, arg type, description
    pub arguments: Option<ArgumentDescription>,
    pub return_type: String,
}

pub struct ArgumentDescription {
   name: String,
   type: String,
   description: String,
}

For example, if one function signature and return type is func(string, int) -> int.
Instead of returning bunch of possible DataType combination like (Utf8, Int64), (Utf8View, Int64), (Utf8, Int32) ....
We can return (string, int) and int for simplicity
@goldmedal @Omega359 What do you think?
Describe the solution you'd like
No response
Describe alternatives you've considered
No response
Additional context
In postgres, they have data type for argument and return type too. And in more general term like numeric, text, int ...
The overall idea is I want the simple argument type and return type defined in documentation and take this information for information schema.

Odd that duckdb seems to be repeating the list there but I'd love to have that list for all functions in DF

Because we have the logic type now, maybe showing the logic type name would be a better choice. WDYT?

I prefer Logical Type

I love the idea of this, I'm not sure logical type will work for all functions, especially those that are user defined. It may be worth if to go through all the functions and build up a spreadsheet of what input/output types they have. I've been wanting to have a more detailed list of functions and their details anyways so if I get a chance I may do that

For user defined signature, they should provide their argument and return type, so we can just get the info from it.


Copied chisel code is out of date - Could possibly use API now?

We have improved the code in both CTM and RenderBlocksCTM to allow for more flexibility with blocks.
For instance the CTM now supports non-full block bounds, such as slabs. It will properly adjust the UVs to match these bounds.
On the whole the code is also much cleaner.
Both of these classes are now included in the API download of chisel, though I'm not sure what you want to do with that info :P

Here's the relevant stuffs https://github.com/TheCricket/Chisel-2/tree/dev/src/main/java/com/cricketcraft/chisel/api/rendering

Thanks so much! I'll take a look later tonight :)

@unpairedbracket, did you make any changes to the chisel code you adopted? I don't want to overwrite anything.

I had to make significant changes to the system they use for matching blocks that should CTM together, so that it's possible to add several lists of metadata values with different block instances to a CTM group. The salination blocks for example are split across 2 block instances, so I think they need 2 meta values from BasicBlock and 1 from BasicBlock2 to match, so the CTM works properly

Okay.  CTM is working fine for the most part right now, I think I'll leave things be.  If I ever run into any issues that are fixed in the new Chisel additions I'll see about doing a bit of refactoring and updating.

If it hasn't already been done by then, I'll look at updating the CTM code to be in line with recent chisel upstream when I finish the set of exams I'm working towards currently.


when [nzValue] use Object at nzSelect

Reproduction link
https://stackblitz.com/edit/ng-zorro-antd-start-sezsm6
Steps to reproduce
when [nzValue] use Object at nzSelect ; The first time you cancel the option, flash will appear.
What is expected?
no flash
What is actually happening?
has flash

Environment
Info

ng-zorro-antd
7.2.0

Browser
chrome

@web900621 I didn't quite get your point, I could not find any flash when canceling the option, could you give a screenshot or gif?

@vthinkxie
I am very sorry that there is no description.
To be precise, it should be when the select has a width.
Blinks when the option is cancelled for the first time.
I updated the connection: link
It should be more obvious now

@vthinkxie

@web900621 you can disable animation to skip this question
https://stackblitz.com/edit/ng-zorro-antd-start-rzxwbg?file=src/app/app.component.html
we will try to resolve it later, thanks for your reporting.

@vthinkxie
ok！ Thank you very much for your answer.

the animation works as design after checked, the flash including one option display and an option destroy
It looks strange but that is how angular animation works.


Setbackground doesnt work properly as expected.

setbackground(your_color) method isn't functioning properly unlike the other two like for example when I set the color to android.R.color.black the background becomes transparent, this has been tested in Lollipop 5.1.1, any other color seems to make it dark translucent black.

Oh my bad its using the Color class, but I think support for 'normal' hex code would also be nice. But awesome anyway.


SOLR-16839: Fix spread-domain bug in AffinityPlacementFactory

This commit fixes a bug where AffinityPlacementFactory would fail to compute placements for shards where one or more replicas are located in down nodes.
The feature hasn't been released yet, so this isn't affecting any official version

Thanks, @tflobbe. May I ask you whether it may affect https://github.com/apache/solr/pull/1690?

May I ask you whether it may affect https://github.com/apache/solr/pull/1690?

Just put a message there. You'll need to rebase before merging, but the conflicts should be easy to resolve (if auto-merge doesn't do it for you). That said, I'd take a look at https://github.com/apache/solr/pull/1650 too, since thee will be more conflicts with it


Is there a way to detect the language of a transliterated text and it's probability?a

Is there a way to detect the language of a transliterated text and it's probability?a
Example:
baagunnara is a transliterated Telugu sentence.  So, given this piece of text can we get the likelihood of the languages? in this case it's Telugu.

No IndicXlit doesn't support language identification given the piece of text. Although we are working on it as a part of another project.


Testing interactive menu broken in Juila1.8

I believe some changes were made to Pkg.jl, such stdin is no longer the global stdin, but instead an IOStream. Here's a link to that PR.
The testset that is breaking is SimulatedInputs
This makes it such that calling stdin.buffer results in:
Got exception outside of @test
  type IOStream has no field buffer

I tried just removing .buffer from stdin, but then ran into this error:
Default template: Error During Test at /Users/baileybox/Documents/Projects/JuliaProjects/1.8/dev/PkgTemplates/test/interactive.jl:84
  Test threw exception
  Expression: Template(; interactive = true) == Template(; user = USER)
  EOFError: read end of file

# Along with

Warning: TerminalMenus: Unable to enter raw mode:
│   exception =
│    MethodError: no method matching check_open(::IOStream)
│    Closest candidates are:
│      check_open(::Union{Base.LibuvServer, Base.LibuvStream}) at stream.jl:386
│      check_open(::Base.Filesystem.File) at filesystem.jl:114

The fix for this would be to setup our own type of stdin, or changing how we call interactive from with the tests.

As a short term work around maybe we can  restrict the interactive tests to only run on 1.7?


toString() error in Nextjs during development

i have been using apexcharts recently and i have very agressive usage of the charts in my app, during  development it keep popping .toString() error which is very annoying i need to doubble save my code to remove it, am i missing something here ??
TypeError: Cannot read properties of undefined (reading 'toString')

Did not follow bug template instructions

Added   width="100%" and it worked


Please invite me to the Recode-Hive GitHub Community Organization

Name
Varshini
Discord Username (if applicable)
No response
Additional Context
I want to contribute

Invitation sent to join the GitHub Organisation. Welcome to the community 🎉
Don't forget after accepting to make it public so it appears on your GitHub profile for everyone else to see. You can do this by finding your name in the GitHub organisation list and change the dropdown to public https://github.com/orgs/Recode-Hive/people


Unused DX9

Hi Rob,
A problem has been reported recently and it appears that Resolume reads an unused registry key value "DX9" after the Max Spout external sets it true. This does not affect the function of the external, but Resolume Spout sending fails. The DX9 flag is not required any more, so removing it would be best in the long run.
jit.gl.spoutsender lines 548,549,500. Remove the else that sets DX9 true. With format set to zero, the default DXGI_FORMAT_B8G8R8A8_UNORM is used.

thanks for filing, will check it out!

this change has been committed and the externals updated and pushed to the Package Manager

Thanks Rob. I have advised on the Spout discourse group.


Feat/light and glovy

DP-1016

review
remove rqt (too big: 1.5 GB), replaced by named tools (less big : 600MB)
add foxglove
enabling

test and document usage of foxglove

[x] Make sure you are opening from a topic/feature/bugfix branch
[x] Ensure that the PR title represents the desired changes
[x] Ensure that the PR description detail the desired changes
[x] Link to relevant issues in GitHub or Jira
[x] Link to relevant pull requests, esp. upstream and downstream changes
[x] Ensure you have provided tests - that demonstrates feature works or fixes the issue

This needs testing. I don't have any task assigned for this sprint to test the slimming of the images. @filipa-movai Do we have to prioritize this?

Only if it is blocking Alex

This needs testing. I don't have any task assigned for this sprint to test the slimming of the images. @filipa-movai Do we have to prioritize this?

Only if it is blocking Alex

This needs testing. I don't have any task assigned for this sprint to test the slimming of the images. @filipa-movai Do we have to prioritize this?

Only if it is blocking Alex

@AlexFernandes-MOVAI How urgent is this?