Error/bug

Any ideas why this is happening?

We've seen this occur on Arduinos with power or connection issues. The Arduino starts a connection, then immediately closes it. You can try connecting your Arduino to your computer to verify the issue, or try configuring with a different Arduino. Which type of Arduino are you using?

I have tried 2 different types of arduinos, both with external power
supplies. They both work fine on a computer.
On 24 Feb. 2018 9:43 am, "Gabriel Burnworth"<EMAIL_ADDRESS>wrote:

We've seen this occur on Arduinos with power or connection issues. The
Arduino starts a connection, then immediately closes it. You can try
connecting your Arduino to your computer to verify the issue, or try
configuring with a different Arduino. Which type of Arduino are you using?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/FarmBot/farmbot_os/issues/452#issuecomment-368133048,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AERrgf7PkAjYX3MQJvU7fxMapCQ3uANSks5tXyL1gaJpZM4SRcJC
.

Which two types have you tried?

rcube and a generic. I also cant upload via the Arduino IDE as there are multiple errors

Can you provide the information the Arduino IDE lists under Tools > Port for each of those boards? VID and PID from Get Board Info would also be helpful.

Generic -
BN: Unknown board
VID: 1A86
PID: 7523
Rcube -
BN: Unknown board
VID: 0403
PID: 6001

Thank you for that info. Do those both use the ATmega2560 (Mega 2560) processor?

Yes they do. Is it a problem that the Pi is seeing them as ttyUSB0

I managed to get it working. I updated the bootloader and it all seems to be ok now

I updated the bootlader on the mega and it worked until I did an update on the web-os. Now i get the error,

I am not sure what is wrong now. I reflashed the Pi back to the factory image and now it has gone back to its original issue.

Is someone able to help? I have tried other boards, extra power supplies and still it wont work. What am I doing wrong?

You may have a faulty pi, but since you said your PC had trouble communicating with the Arduino, it could be the cable, or some unknown factor. Then try configuring Farmbot with just the Arduino, (no ramps shield).
For more information it might be helpful to see a picture of your setup

Also, did any of the Arduinos you tried have a recognized board name? (instead of BN: Unknown board)

My computer is working fine talking to the Arduino. I found out one was a
1280 and that's why 1 didn't work but I have 2 more. One worked fine until
the pi did an update and now it won't work again. The pi is fine and can
see the Arduino. I plugged a video cable into the pi to see what was
happening
On 26 Feb. 2018 7:28 pm, "Connor Rigby"<EMAIL_ADDRESS>wrote:

You may have a faulty pi, but since you said your PC had trouble
communicating with the Arduino, it could be the cable, or some unknown
factor. Then try configuring Farmbot with just the Arduino, (no ramps
shield).
For more information it might be helpful to see a picture of your setup
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/FarmBot/farmbot_os/issues/452#issuecomment-368402993,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AERrgWrMOhMjL0ov346ItMWFDBsLrWwgks5tYk7wgaJpZM4SRcJC
.

They are all unknown as I don't have any genuine 2560 boards.
On 26 Feb. 2018 7:30 pm, "Gabriel Burnworth"<EMAIL_ADDRESS>wrote:

Also, did any of the Arduinos you tried have a recognized board name?
(instead of BN: Unknown board)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/FarmBot/farmbot_os/issues/452#issuecomment-368403437,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AERrgfXWbzxjvY6DcPrPPv7TfxfPhBx5ks5tYk-igaJpZM4SRcJC
.

The update isn't what caused the issue, the issue was the reboot. Something is causing the USB to Serial converter chip on the clone Arduino reset. Farmbot OS is detecting /dev/ttyUSB0 then it is resetting while the driver is initializing, and probably showing up as /dev/ttyUSB1.
A number of things can cause it, most commonly bad power.

Also, I don't think you will see any information on video. I think that all gets disabled.

I am seeing ttyUSB0 on the monitor but no mention of UBS1.  The power supply is ok as it can handle 10 Amps.

Can I get a picture of both your setup and the output on the monitor if at all possible?

I'm also trying to get it working on a MKS Base v1.5 but I am still getting the same issue. It is a ftdi USB interface

I just installed version 5.0.9 and for the first time it uploaded the firmware to the arduino, it has never done that before and it is also working fine now. How long can i tell it not to update before it stops working?

The same issue will happen upon reboot unfortunately regardless of updates.
We usually support old releases for a few months. I will try to figure out if it is a Linux driver issue in the meantime

That would be good if you could find out what's happening. It would be nice to be able to use the MKS board as its an all in one and not a shield.

Can we get a picture of the arduino board you are using? The log you captured above is displaying the usage of a CH341 chip, but the board you are talking about should come with a genuine USB Driver Chip FT232RL.

I have changed boards since that screenshot. And yes, the board comes with a FT232 chip. I am on a mobile device and not sure how to attach an image

So you are saying that the FT232 board does not work with ~> 6.x releases?
As for an image you can use external upload services such as imgur
Id like to see a screen capture of the monitor on 6.0.1 and a picture of the board that is not working if possible.

That is correct. the FT232 board is not working with v6.x.
I have attached images of the setup and what is on the monitor

Have you had any luck? It would be nice to get it working with the latest FW

Okay so this turned out to be partially a bug on the FBOS side. It will be fixed in the next release, but in the meantime here is what you can do:

Flash your Arduino manually with the latest firmware.

If you need help with this i can walk you through it, there are a number of ways you can accomplish this.

Flash your device with 6.0.1
Boot and configure Farmbot without your Arduino device plugged in.
Upgrade using the web app.
Wait until Farmbot is fully booted up, and maybe do a sync to make sure everything is up and running.
Plug your Arduino
Wait a few seconds, and you should be able to use it.

For viewers at home here is the line that is currently messed up.

Thank you. I will try that tonight when I get home and let you know how I go.

Depending on your timezone, we will probably have a proper release out by "tonight"

It worked to begin with, right up until I plugged the Arduino board back in. I took a screen shot of the log on my.farmbot and the web gui on the Rpi.

It appears to be working. I had issues but I forgot to change the board type in Boards.h.

Sorry for the delay. This should be fixed in the latest release
You will have to flash 6.0.1 first, without your Arduino plugged in, upgrade until there are no updates available, then you can plug your Arduino in and it should work properly and persist reboots. I will have more elaborate instructions in the FAQ if you are having trouble.

Here is the relevant FAQ entry.


Departments show up twice

Description
Filter drop-down for department contains two of each field.

@evelynhasama or @elizjo look into this whenever possible.

Closed by #33


Fix hide "Powered by Stoplight" for Professional Plan

User story.
As a StoplightProject user on a Professional plan, I can hide Powered by Stoplight, so that I can keep people looking at my docs instead of being distracted by Stoplight's awesome tooling.
Is your feature request related to a problem?
Hosted Docs will let Workspace Admins hide "Powered by Stoplight" as part of the Professional plan, but currently StoplightProject shows it for all projects all the time.
Describe the solution you'd like
Check the plan or check settings to see if they would like to hide "Powered by Stoplight"

API and frontend logic is there, the problem is that the api returns hidePoweredBy while we expect hide_powered_by


Error saving custom sklearn transformer and pipeline

reported by @ykasimov
pip install pandas xgboost scikit-learn mlem

from sklearn.base import BaseEstimator, TransformerMixin
from xgboost import XGBClassifier
from mlem.api import save
import pandas as pd

from sklearn.pipeline import Pipeline
import typing

class SubtractTransform(BaseEstimator, TransformerMixin):
    def __init__(
        self
    ) -> None:
        super().__init__()

    def fit(
        self, X: pd.DataFrame, y: typing.Optional[pd.DataFrame] = None
    ) :
        print("fit")
        return self

    def transform(
        self,
        X: pd.DataFrame,
    ):
        print("transform")
        X_ = X.copy()  # creating a copy to avoid changes to original dataset
        return X_

clf = Pipeline(
        steps=[
            (
                "SubtractTransform",
                SubtractTransform(
                ),
            ),
            (
                "xgboost",
                XGBClassifier(
                ),
            ),
        ]
)

x = [[0,0],[0,1], [1,0], [1,1]]
y = [0, 1, 1, 0]
clf.fit(x, y)
save(clf, "model.clf", sample_data=x)

$ python train.py
fit
transform
transform
transform
Traceback (most recent call last):
  File "/Users/aguschin/Git/iterative/mlem-docker-bug/.venv/lib/python3.10/site-packages/mlem/utils/module.py", line 456, in wrapper
    source = dill.source.getsource(obj)
  File "/Users/aguschin/Git/iterative/mlem-docker-bug/.venv/lib/python3.10/site-packages/dill/source.py", line 374, in getsource
    lines, lnum = getsourcelines(object, enclosing=enclosing)
  File "/Users/aguschin/Git/iterative/mlem-docker-bug/.venv/lib/python3.10/site-packages/dill/source.py", line 345, in getsourcelines
    code, n = getblocks(object, lstrip=lstrip, enclosing=enclosing, locate=True)
  File "/Users/aguschin/Git/iterative/mlem-docker-bug/.venv/lib/python3.10/site-packages/dill/source.py", line 271, in getblocks
    lines, lnum = findsource(object)
  File "/Users/aguschin/Git/iterative/mlem-docker-bug/.venv/lib/python3.10/site-packages/dill/source.py", line 215, in findsource
    line = lines[lnum]
IndexError: list index out of range

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/Users/aguschin/Git/iterative/mlem-docker-bug/train.py", line 50, in <module>
    save(clf, "model.clf", sample_data=x)
  File "/Users/aguschin/Git/iterative/mlem-docker-bug/.venv/lib/python3.10/site-packages/mlem/telemetry.py", line 50, in inner
    return f(*args, **kwargs)
  File "/Users/aguschin/Git/iterative/mlem-docker-bug/.venv/lib/python3.10/site-packages/mlem/core/metadata.py", line 122, in save
    meta = get_object_metadata(
  File "/Users/aguschin/Git/iterative/mlem-docker-bug/.venv/lib/python3.10/site-packages/mlem/core/metadata.py", line 53, in get_object_metadata
    return MlemModel.from_obj(
  File "/Users/aguschin/Git/iterative/mlem-docker-bug/.venv/lib/python3.10/site-packages/mlem/core/objects.py", line 786, in from_obj
    mlem_model.add_processor(MAIN_PROCESSOR_NAME, mt)
  File "/Users/aguschin/Git/iterative/mlem-docker-bug/.venv/lib/python3.10/site-packages/mlem/core/objects.py", line 810, in add_processor
    self.requirements += model_type.get_requirements().expanded
  File "/Users/aguschin/Git/iterative/mlem-docker-bug/.venv/lib/python3.10/site-packages/mlem/contrib/sklearn.py", line 86, in get_requirements
    super().get_requirements()
  File "/Users/aguschin/Git/iterative/mlem-docker-bug/.venv/lib/python3.10/site-packages/mlem/core/model.py", line 341, in get_requirements
    ) + get_object_requirements(self.model)
  File "/Users/aguschin/Git/iterative/mlem-docker-bug/.venv/lib/python3.10/site-packages/mlem/utils/module.py", line 643, in get_object_requirements
    a.dump(obj)
  File "/Users/aguschin/Git/iterative/mlem-docker-bug/.venv/lib/python3.10/site-packages/dill/_dill.py", line 394, in dump
    StockPickler.dump(self, obj)
  File "/Users/aguschin/.pyenv/versions/3.10.7/lib/python3.10/pickle.py", line 487, in dump
    self.save(obj)
  File "/Users/aguschin/Git/iterative/mlem-docker-bug/.venv/lib/python3.10/site-packages/mlem/utils/module.py", line 614, in save
    return super().save(obj, save_persistent_id)
  File "/Users/aguschin/Git/iterative/mlem-docker-bug/.venv/lib/python3.10/site-packages/dill/_dill.py", line 388, in save
    StockPickler.save(self, obj, save_persistent_id)
  File "/Users/aguschin/.pyenv/versions/3.10.7/lib/python3.10/pickle.py", line 603, in save
    self.save_reduce(obj=obj, *rv)
  File "/Users/aguschin/.pyenv/versions/3.10.7/lib/python3.10/pickle.py", line 717, in save_reduce
    save(state)
  File "/Users/aguschin/Git/iterative/mlem-docker-bug/.venv/lib/python3.10/site-packages/mlem/utils/module.py", line 614, in save
    return super().save(obj, save_persistent_id)
  File "/Users/aguschin/Git/iterative/mlem-docker-bug/.venv/lib/python3.10/site-packages/dill/_dill.py", line 388, in save
    StockPickler.save(self, obj, save_persistent_id)
  File "/Users/aguschin/.pyenv/versions/3.10.7/lib/python3.10/pickle.py", line 560, in save
    f(self, obj)  # Call unbound method with explicit self
  File "/Users/aguschin/Git/iterative/mlem-docker-bug/.venv/lib/python3.10/site-packages/dill/_dill.py", line 1186, in save_module_dict
    StockPickler.save_dict(pickler, obj)
  File "/Users/aguschin/.pyenv/versions/3.10.7/lib/python3.10/pickle.py", line 972, in save_dict
    self._batch_setitems(obj.items())
  File "/Users/aguschin/.pyenv/versions/3.10.7/lib/python3.10/pickle.py", line 998, in _batch_setitems
    save(v)
  File "/Users/aguschin/Git/iterative/mlem-docker-bug/.venv/lib/python3.10/site-packages/mlem/utils/module.py", line 614, in save
    return super().save(obj, save_persistent_id)
  File "/Users/aguschin/Git/iterative/mlem-docker-bug/.venv/lib/python3.10/site-packages/dill/_dill.py", line 388, in save
    StockPickler.save(self, obj, save_persistent_id)
  File "/Users/aguschin/.pyenv/versions/3.10.7/lib/python3.10/pickle.py", line 560, in save
    f(self, obj)  # Call unbound method with explicit self
  File "/Users/aguschin/.pyenv/versions/3.10.7/lib/python3.10/pickle.py", line 932, in save_list
    self._batch_appends(obj)
  File "/Users/aguschin/.pyenv/versions/3.10.7/lib/python3.10/pickle.py", line 956, in _batch_appends
    save(x)
  File "/Users/aguschin/Git/iterative/mlem-docker-bug/.venv/lib/python3.10/site-packages/mlem/utils/module.py", line 614, in save
    return super().save(obj, save_persistent_id)
  File "/Users/aguschin/Git/iterative/mlem-docker-bug/.venv/lib/python3.10/site-packages/dill/_dill.py", line 388, in save
    StockPickler.save(self, obj, save_persistent_id)
  File "/Users/aguschin/.pyenv/versions/3.10.7/lib/python3.10/pickle.py", line 560, in save
    f(self, obj)  # Call unbound method with explicit self
  File "/Users/aguschin/.pyenv/versions/3.10.7/lib/python3.10/pickle.py", line 887, in save_tuple
    save(element)
  File "/Users/aguschin/Git/iterative/mlem-docker-bug/.venv/lib/python3.10/site-packages/mlem/utils/module.py", line 614, in save
    return super().save(obj, save_persistent_id)
  File "/Users/aguschin/Git/iterative/mlem-docker-bug/.venv/lib/python3.10/site-packages/dill/_dill.py", line 388, in save
    StockPickler.save(self, obj, save_persistent_id)
  File "/Users/aguschin/.pyenv/versions/3.10.7/lib/python3.10/pickle.py", line 603, in save
    self.save_reduce(obj=obj, *rv)
  File "/Users/aguschin/.pyenv/versions/3.10.7/lib/python3.10/pickle.py", line 687, in save_reduce
    save(cls)
  File "/Users/aguschin/Git/iterative/mlem-docker-bug/.venv/lib/python3.10/site-packages/mlem/utils/module.py", line 614, in save
    return super().save(obj, save_persistent_id)
  File "/Users/aguschin/Git/iterative/mlem-docker-bug/.venv/lib/python3.10/site-packages/dill/_dill.py", line 388, in save
    StockPickler.save(self, obj, save_persistent_id)
  File "/Users/aguschin/.pyenv/versions/3.10.7/lib/python3.10/pickle.py", line 560, in save
    f(self, obj)  # Call unbound method with explicit self
  File "/Users/aguschin/Git/iterative/mlem-docker-bug/.venv/lib/python3.10/site-packages/mlem/utils/module.py", line 495, in save_type_with_classvars
    pickler.save(attr)
  File "/Users/aguschin/Git/iterative/mlem-docker-bug/.venv/lib/python3.10/site-packages/mlem/utils/module.py", line 614, in save
    return super().save(obj, save_persistent_id)
  File "/Users/aguschin/Git/iterative/mlem-docker-bug/.venv/lib/python3.10/site-packages/dill/_dill.py", line 388, in save
    StockPickler.save(self, obj, save_persistent_id)
  File "/Users/aguschin/.pyenv/versions/3.10.7/lib/python3.10/pickle.py", line 560, in save
    f(self, obj)  # Call unbound method with explicit self
  File "/Users/aguschin/Git/iterative/mlem-docker-bug/.venv/lib/python3.10/site-packages/mlem/utils/module.py", line 474, in wrapper
    raise Exception(
Exception: Cannot parse code for <function SubtractTransform.transform at 0x136e312d0> from /Users/aguschin/Git/iterative/mlem-docker-bug/.venv/lib/python3.10/site-packages/sklearn/utils/_set_output.py

Another error with this model: if you try to mlem.api.load this, you get:
AttributeError: Can't get attribute 'SubtractTransform' on <module 'main' from '/Users/yurykasimov/Documents/dvc_playground/custom_transformer/load.py'>

the workaround to that is to import SubtractTransform explicitly.
However it leads to another error:
mlem.core.errors.HookNotFound: No suitable ModelHook for object of type "Pipeline". Registered hooks: [<class 'mlem.contrib.callable.CallableModelType'>]

the workaround for that is to add import sklearn to the script.
Needs to be investigated why mlem fails to discover the hooks.

I can't reproduce original issue, copy-pasted and it just worked. Maybe share your env and I will try to re-create it.
As for loading, you need to do two things, if you are using something custom with mlem.

Always use if __name__ == '__main__' syntax. When python imports a module, it actually just runs in. Since we need the code to recreate the object you saved, it means that fit and save code will run every time you try to load it, unless you put into main.
Put everything custom in a separate module (not the one you are running). The running module in python is always called __main__ instead of the actual name. Because of that when we save the model with custom stuff, __main__ is treated as a requirement. When you load it, there is a collision with the __main__ module that you run your code from. This can work in some cases, but there is no guarantee.

Those are limitations of python and pickle/dill and there is nothing we can do about it, except put it somewhere in docs

After that, do pip install scikit-learn=1.1.1 and try to load the model:
(.venv) tmp %f $ python load.py                                                                                28.53s
Traceback (most recent call last):
  File "/Users/aguschin/Git/iterative/tmp/load.py", line 4, in <module>
    load("model.clf")
  File "/Users/aguschin/Git/iterative/tmp/.venv/lib/python3.9/site-packages/mlem/core/metadata.py", line 156, in load
    meta = load_meta(
  File "/Users/aguschin/Git/iterative/tmp/.venv/lib/python3.9/site-packages/mlem/telemetry.py", line 50, in inner
    return f(*args, **kwargs)
  File "/Users/aguschin/Git/iterative/tmp/.venv/lib/python3.9/site-packages/mlem/core/metadata.py", line 245, in load_meta
    meta.load_value()
  File "/Users/aguschin/Git/iterative/tmp/.venv/lib/python3.9/site-packages/mlem/core/objects.py", line 853, in load_value
    self.model_type.load(self.relative_artifacts)
  File "/Users/aguschin/Git/iterative/tmp/.venv/lib/python3.9/site-packages/mlem/core/model.py", line 277, in load
    self.model = self.io.load(artifacts)
  File "/Users/aguschin/Git/iterative/tmp/.venv/lib/python3.9/site-packages/mlem/core/model.py", line 98, in load
    return pickle.load(f)
AttributeError: Can't get attribute 'SubtractTransform' on <module '__main__' from '/Users/aguschin/Git/iterative/tmp/load.py'>

I reproduced and investigated original error.
Loading error should be fixed too (but you still need main for it to work properly)


feat: instrument service with OpenTelemetry

Marked as draft pending review and merge for https://github.com/goto/salt/pull/5

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

213 of 279 (76.34%)  changed or added relevant lines in 10 files are covered.
6 unchanged lines in 4 files lost coverage.
Overall coverage decreased (-0.2%) to 87.518%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

core/asset/errors.go
1
3
33.33%

core/asset/service.go
34
36
94.44%

internal/server/v1beta1/server.go
7
9
77.78%

pkg/worker/pgq/pgq_processor.go
10
14
71.43%

internal/store/postgres/postgres.go
13
19
68.42%

internal/store/elasticsearch/es.go
50
58
86.21%

internal/store/elasticsearch/discovery_repository.go
46
65
70.77%

internal/store/elasticsearch/discovery_search_repository.go
33
56
58.93%

Files with Coverage Reduction
New Missed Lines
%

internal/store/elasticsearch/discovery_repository.go
1
67.97%

internal/store/elasticsearch/es.go
1
74.83%

internal/store/postgres/postgres.go
1
62.79%

internal/store/elasticsearch/discovery_search_repository.go
3
81.23%

Totals

Change from base Build<PHONE_NUMBER>:
-0.2%

Covered Lines:
6107

Relevant Lines:
6978

💛  - Coveralls


[rom_ctrl] Enable secure FIFO counters

Signed-off-by: Michael Schaffner<EMAIL_ADDRESS>

i actually don't know who's working on rom_ctrl dv, but should we let him/her know? it looks like rom_ctrl is already at V2S, so I'm not sure if this needs to be added.

I just added it to be consistent accross the board. let me also loop in @weicaiyang for this.

i actually don't know who's working on rom_ctrl dv, but should we let him/her know? it looks like rom_ctrl is already at V2S, so I'm not sure if this needs to be added.

I just added it to be consistent accross the board. let me also loop in @weicaiyang for this.

Thanks for tagging me. Looks like this sec_cm test should be able to test this counter.
Loop in @prajwalaputtappa


habitat_accessibility_model distribution adjustments

Additional data to include in the distribution generated by https://github.com/smnorris/bcfishpass/blob/main/release/freshwater_fish_habitat_accessibility_model.sh, for reference:

PSCIS crossings not matched to streams
all potential natural barriers (not just the minimal barriers currently posted) ie:

gradient barriers (5% intervals)
subsurface flow
falls

In addition to above layers, include a method allowing users to select which watershed groups support the species of interest. Will need to consider if a column in the streams table is needed, or if simply including /data/wsg_species_presence.csv is sufficient.

Re species presence, a column in the streams table is probably the user friendly way to go.
Sticking with wsg_species_presence as the name, with a string of species codes like this CH; SK as the value.
What the column means will have to be clearly documented.

Another possible addition:
Unless bcfishobs data is available through some other channel, it might be good to include various data associated with observations (date, activity, life stage etc), making this an easy spot to access FWA linked observation data.

fixed:
$ ogrinfo /vsizip//vsicurl/https://bcfishpass.s3.us-west-2.amazonaws.com/freshwater_fish_habitat_accessibility_MODEL.gpkg.zip
INFO: Open of `/vsizip//vsicurl/https://bcfishpass.s3.us-west-2.amazonaws.com/freshwater_fish_habitat_accessibility_MODEL.gpkg.zip'
      using driver `GPKG' successful.
1: crossings (3D Measured Point)
2: gradient_barriers (3D Measured Point)
3: barriers_subsurfaceflow (3D Measured Point)
4: barriers_falls (3D Measured Point)
5: barriers_salmon (3D Measured Point)
6: barriers_steelhead (3D Measured Point)
7: model_access_salmon (3D Measured Line String)
8: model_access_steelhead (3D Measured Line String)
9: observations (3D Measured Point)

$ ogrinfo /vsizip//vsicurl/https://bcfishpass.s3.us-west-2.amazonaws.com/freshwater_fish_habitat_accessibility_MODEL.gpkg.zip observations -so
INFO: Open of `/vsizip//vsicurl/https://bcfishpass.s3.us-west-2.amazonaws.com/freshwater_fish_habitat_accessibility_MODEL.gpkg.zip'
      using driver `GPKG' successful.
Layer name: observations
Geometry: 3D Measured Point
Feature Count: 52879
...
Geometry Column = geom
fish_observation_point_id: Integer (0.0)
fish_obsrvtn_event_id: Integer64 (0.0)
linear_feature_id: Integer64 (0.0)
blue_line_key: Integer (0.0)
wscode_ltree: String (0.0)
localcode_ltree: String (0.0)
downstream_route_measure: Real (0.0)
watershed_group_code: String (4.0)
species_code: String (0.0)
observation_date: Date (0.0)
activity_code: String (100.0)
activity: String (300.0)
life_stage_code: String (100.0)
life_stage: String (300.0)
acat_report_url: String (254.0)


Phpunit8 compat

This is a major BC-break for several reasons, detailed in the first commit message.

Ah looks like this is blocked by the config lib, it has to be moved first.

Woops didn't see #109 , closing in favor of that.


Possible improvement to serialization documentation

I'm not completely sure this is a good idea, but... for me, a common desire is "package up, in one file, both the pytree leaves, and instructions for creating the pytree skeleton". Concretely, I might have a function
def make_model(key, **hyperparameters):
    return MyModel(depth=hyperparameters['depth'], key=key)

and then I would like to save in a single file both the dict hyperparameters and the weights of MyModel.
This is of course easy, but it's not immediately obvious from the documentation that it's possible at all (at least it wasn't to me). I think it's wroth adding to the "Tricks" page (or perhaps FAQ) a section explaining that it is possible (because tree_serialise_leaves is happy to dump to any BinaryIO), with an example like:
def save_model(filename, model, hyperparameters)
  leaves = io.BytesIO()
  eqx.tree_serialise_leaves(leaves, model)
  leaves.seek(0)
  with open(filename, 'wb') as f:
    pickle.dump(hyperparameters, f)

This makes sense to me! I'd be happy to take a pull request on this -- I could see a more fully-featured example being a good way to demonstrate how to make this pattern work in full. FWIW this is the approach I take in my own code. The only requirement is that the hyperparameters themselves must be serialisable, e.g. not a Python function.
For completeness I note that there is also the Orbax library for serialisation. They support asynchronous serialisation, and sharded (multi-device) JAX arrays, which are useful for bigger models. It looks like their orbax.checkpoint.PyTreeCheckpointer() operates on essentially the same premise as eqx.tree_{de,}serialise_leaves in that it expects a PyTree structure at deserialisation time, so probably the pattern used here would apply to them too.

Closing as done!


select-first-match appear doesn't work properly

I think that this option enable to add the first item from autocomplete list that is highlighted, but it doesn't work like.
If I have this list:
apple,
banana,
cherry
And I type in input "ana":
apple
banana
cherry
And a hit 'Enter', the 'banana' should be selected. Or not?

Are you running Angular 1.5? I've managed to reproduce the exact same bug after I upgraded to 1.5.
For me, it seems to be caused by a change in the registration order of the input-keydown event on both the editor and the autocomplete dropdown; the order has been reversed in Angular 1.5. This causes the editor to receive the ENTER key press before the autocomplete dropdown, closing the dropdown and simply creating a tag based on the partial string you've typed (just as it would if the autocomplete dropdown wasn't being used).

This issue was selected to be spring-cleaned by a script. More information can be found here.
If anyone thinks this was a mistake and this issue should be reopened, please leave a message below explaining why. Before doing so, please consider reading the CONTRIBUTING file.


cmd/compile: bad mpint length: 65 > 64

What version of Go are you using (go version)?

$ go version
go version go1.19 linux/amd64

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN="/home/jay/.local/bin"
GOCACHE="/home/jay/.cache/go-build"
GOENV="/home/jay/.config/go/env"
GOEXE=""
GOEXPERIMENT=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOINSECURE=""
GOMODCACHE="/home/jay/.local/share/go/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="linux"
GOPATH="/home/jay/.local/share/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/lib/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/lib/go/pkg/tool/linux_amd64"
GOVCS=""
GOVERSION="go1.19"
GCCGO="gccgo"
GOAMD64="v1"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD="/home/jay/src/go/playground/go.mod"
GOWORK=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -Wl,--no-gc-sections -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build1919621165=/tmp/go-build -gno-record-gcc-switches"

What did you do?
https://go.dev/play/p/_jct1xi435j
What did you expect to see?
Nothing, I was just exploring untyped constants
What did you see instead?
# playtime
./main.go:10:13: internal compiler error: bad mpint length: 65 > 64 (type untyped int, value<PHONE_NUMBER>01234567890123456789012345678901234567890912345678901234567890123456789012345678901234567890912345678901234567890123456789012345678901234567890912)

Please file a bug report including a short program that triggers the error.
https://go.dev/issue/new

This could be a false alarm, I'm just following what the toolchain asked me to do.

CC @golang/compiler
Optimistically assigning to @griesemer but giving it a low priority since this case doesn't seem like it would come up much in practice. Seems like it could just use a better error message. @griesemer Please feel free to unassign!

The crash is actually happening in iexport, but the issue is that the integer constant requires more than 64 bytes (512 bits) to represent.
I think the issue is that types2 rejects arithmetic that overflows 512 bits, but it doesn't check that constant literals themselves fit into 512 bits. If I change the declaration of BIG to include + 0, then types2 reports "constant addition overflow" even though it represents the same value.


mergify: automatically apply backend label

Automatically apply the backend label to any PR that touches a file
under the docs/backend directory.
Signed-off-by: Russell Bryant<EMAIL_ADDRESS>

Nit: misspelled "autoamtically".

Thanks -- fixed in the commit message and PR title


Part of GroupImage path is duplicated and confused

1. Shortcut set to open in .\TaskBarGroups\config\group

With all the groups I created, part of the path to the GroupImage is duplicated and confused:
...\TaskBarGroups\config\Chat\config\Browsers\GroupImage.png

Once I click OK, it then loops through the rest of the folders in the config folder and displays the popup with each of the next folders' names:
...\TaskBarGroups\config\Chat\config\Chat\GroupImage.png
...\TaskBarGroups\config\Chat\config\Connectivity\GroupImage.png
...\TaskBarGroups\config\Chat\config\Dev_Tools\GroupImage.png
Once it finishes with all the popups, then it opens TaskBarGroups, but with no groups that had been created before.
2. Shortcut set to open in .\TaskBarGroups

If I change the start folder of the shortcut lnk to "C:\Program Files (x86)\TaskBarGroups", Then the group opens for a few milliseconds, and then opens TaskBarGroups.exe and shows all previously created groups. There is never an opportunity to actually click on any of the items in the group.

Did you put any direct shortcut to the taskbar group application itself and/or did you change the shortcuts that were generated?
What seems like what is happening is that a shortcut messed up, and the application doesn't detect the correct input from the shortcut parameters making it think it is just launching the editing application instead of the correct group. Does this happen when you have the application in a folder on say the Desktop?


Support ondevicechange from mediaDevices

Add support for the method ondevicechange from mediaDevices. This way we can know when a device is added or removed so we can refresh our list of devices:
https://developer.mozilla.org/en-US/docs/Web/API/MediaDevices/devicechange_event
More details can be found on this linear ticket:
https://linear.app/dailyco/issue/ENG-4100/react-native-webrtc-support-ondevicechange-from-mediadevices
This PR, depends on this PR.
This is still a proposal.

https://github.com/daily-co/rn-daily-js-playground/pull/82/files#r851534184


UI: User can finish the create connector wizard with invalid config

It is possible for the user to finish the create connector wizard with an invalid config.  Consider the following scenario

Enter valid connection properties on step 2.
Click next twice - go to step 4
Enter an invalid property value on step 4 (for example enter 10 for 'Columns PK Mapping') - validate and get invalid message
Click back, go to the previous page. (on the previous page user can click Finish)
Click finish results in config with the invalid value from step 4
User should not be able to finish unless all pages are valid

closing - accidentally created duplicate


fix: JToken to value container

:tada: This PR is included in version 1.1.0 :tada:
The release is available on:

GitHub release
v1.1.0
v1.1.0

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.1.0-alpha.1 :tada:
The release is available on:

GitHub release
v1.1.0-alpha.1
v1.1.0-alpha.1

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 2.0.0-alpha.2 :tada:
The release is available on:

GitHub release
v2.0.0-alpha.2
v2.0.0-alpha.2

Your semantic-release bot :package::rocket:


Dougbu/add.performance.apps

I want to pull the project just for learning at the moment

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


Consider compat lib for seamless ScalaTest migration

Hi @olafurpg ,
I just found about Munit at the ScalaLove conf and decided to give it a go in one of my projects using  ScalaTest and WordSpec so far.
In due course, I have found a nice way to migrate to MUnit without rewriting a single line of my tests, except of course of changing class dependency to munit.FunSuite.  That would make it easier for many people to try MUnit first and then eventually switch.
The resulting compatibility trait and an example test is presented here https://gist.github.com/arturopala/4cfd20de97006031329a946f08c54f20
Do you think it would make sense to have such a compatibility tool in a separate project?
Cheers, Artur

Thank you for taking the time to implement and share your experiments with building a ScalaTest compatibility API! I think this is an important discussion and I suspect several people are interested in using such a compatibility API because it would significantly reduce the cost of migration, it has at least come up several times on Gitter.
While I agree it's desirable to reduce the migration cost for ScalaTest users, I am still on the fence about including something like this in MUnit because it would mean that users face the choice of deciding what syntax they want to use: munit.FunSuite or AnyWordSpecCompat. An important goal for me personally is to make it easy for a beginner Scala developer to use MUnit without having to evaluate multiple testing DSLs.

That would make it easier for many people to try MUnit first and then eventually switch.

Do you have suggestions for how we can ensure this API is only used for the initial evaluation and not permanently after the eventual switch?

@olafurpg I share your point of view and see possible win-win alternatives to shipping anything inside Munit:

separate, small scale library accompanying MUnit, replicating basic ScalaTest patterns with a limited set of popular Matchers

dedicated migration guide on the website, with live Scastie examples ready to use

The reasoning behind option 2 is that the effort and cost of coming with a comprehensive compat library might be substantial, while a simple library might not be of real use and disappointing in the long run. Having some simple seeds to copy, like AnyWordSpecCompat, might be a quicker and more scalable solution, allowing developers to gradually migrate and scale down to the minimal set of matchers, which might be a different target for different people.

dedicated migration guide on the website, with live Scastie examples ready to use

Sounds good. I'm OK with linking to example code that users can copy-paste. Do you have a preference between writing the code examples on Scastie or in the MUnit website? All of the code examples on the website are typechecked. I'm OK with adding a dependency on ScalaTest in the docs project.
I primarily want to avoid publishing artifacts with a ScalaTest since that would mean we take on the burden of maintaining a compatibility layer for ScalaTest, which as you say is hard to deliver 100%.

Closing this as "wontfix". The ScalaTest migration guide has been expanded with more details https://scalameta.org/munit/docs/scalatest.html thanks to the contribution in https://github.com/scalameta/munit/pull/166 and https://github.com/scalameta/munit/pull/164
Additionally, I'm open to merge a PR based on the work in https://github.com/scalameta/munit/pull/191 adding Scalafix migration rewrites from ScalaTest to MUnit.


fix: image add is_standard field, filter by is_standard=false|true

这个 PR 实现什么功能/修复什么问题:
修正：image增加is_standard字段，标示是否为标准镜像
是否需要 backport 到之前的 release 分支:

release/2.10.0
release/2.11.0

Codecov Report

Merging #1815 into master will decrease coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           master   #1815      +/-   ##
=========================================
- Coverage    7.62%   7.61%   -0.01%
=========================================
  Files         412     412
  Lines       63484   63501      +17
=========================================
  Hits         4838    4838
- Misses      58236   58253      +17
  Partials      410     410

Flag
Coverage Δ

#aFlag
7.61% <0%> (-0.01%)
:arrow_down:

Impacted Files
Coverage Δ

pkg/compute/models/storages.go
1.23% <0%> (-0.01%)
:arrow_down:

pkg/compute/models/wires.go
2.09% <0%> (-0.03%)
:arrow_down:

pkg/compute/models/vpcs.go
2.1% <0%> (-0.02%)
:arrow_down:

pkg/compute/models/hosts.go
0.4% <0%> (-0.01%)
:arrow_down:

pkg/compute/models/storagecaches.go
2.42% <0%> (-0.02%)
:arrow_down:

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 ca18d24...9aae98c. Read the comment docs.

/area glance
/area image

/lgtm
/approve


movielens recommendations transformers example broken

The notebook/code/colab is broken for the example "A Transformer-based recommendation system":
The ipynb fails to load:
https://github.com/keras-team/keras-io/blob/master/examples/structured_data/ipynb/movielens_recommendations_transformers.ipynb
Colab is also broken, with the following error:

There was an error loading this notebook. Ensure that the file is accessible and try again.
Unexpected token ] in JSON at position 25830
https://github.com/keras-team/keras-io/blob/master/examples/structured_data/ipynb/movielens_recommendations_transformers.ipynb
Unexpected token ] in JSON at position 25830
SyntaxError: Unexpected token ] in JSON at position 25830
at JSON.parse ()
at Ca.program_ (https://colab.research.google.com/v2/external/external_polymer_binary.js?vrz=colab-20210212-080411-RC00_357173882:1374:452)
at Ea (https://colab.research.google.com/v2/external/external_polymer_binary.js?vrz=colab-20210212-080411-RC00_357173882:19:336)
at Ca.next_ (https://colab.research.google.com/v2/external/external_polymer_binary.js?vrz=colab-20210212-080411-RC00_357173882:17:503)
at Ga.next (https://colab.research.google.com/v2/external/external_polymer_binary.js?vrz=colab-20210212-080411-RC00_357173882:20:206)
at f (https://colab.research.google.com/v2/external/external_polymer_binary.js?vrz=colab-20210212-080411-RC00_357173882:62:101)

Hi @ddofer ,
We are going through backlogs and this issue is not persistent now.
I have tested the tutorial and I got encoding related error at below code.
I have also raised a PR #1406  for this which is under review now.
If we change the encoding and executing the code then everything working fine. Please refer attached gist. The reported error by you no more exists now.
Please cross check and confirm whether is it good to close this issue now. Thanks!


⚠ api.piste.gouv.fr healthcheck has degraded performance

In 3d0d01b, api.piste.gouv.fr healthcheck (https://api.piste.gouv.fr/cassation/judilibre/v1.0/healthcheck) experienced degraded performance:

HTTP code: 200
Response time: 2594 ms

Resolved: api.piste.gouv.fr healthcheck performance has improved in 59ff445.


Test suite in second-transfer works in the stack environment...

…by faking OpenSSL 1.0.2 symbols when older versions of the library are used. This closes this issue.

:+1:

Hmm, the testsuite still failed in the Stackage nightly build with second-transfer-<IP_ADDRESS>.


Change select to use react-windowed-select

Hoist P/R Checklist
Review and check off the below. Items that do not apply can also be checked off to indicate they
have been considered. If unclear if a step is relevant, please leave unchecked and note in comments.

[x] Up to date with develop branch as of last change.
[ ] Ready for review (or open as a draft PR / add wip label).
[ ] Added CHANGELOG entry (or N/A)
[ ] Reviewed for breaking changes (add breaking-change label + CHANGELOG if so, or N/A)
[ ] Updated doc comments / prop-types (or N/A)
[ ] Reviewed and tested on Mobile (or N/A)
[ ] Created Toolbox branch / PR (link provided here, or N/A)

Pull request reviewers: when merging this P/R, please consider using a squash commit to
collapse multiple intermediate commits into a single commit representing the overall feature
change. This helps keep the commit log clean and easy to scan across releases. PRs containing a
single commit should be rebased when possible.

To improve performance of Select on large lists, per this issue:
https://github.com/xh/hoist-react/issues/1458

@amcclain -- any concerns here?  I would like to merge this and have us all start kicking the tires.  As @haynesjm42 can attest, this is a pretty serious bug

I would also ask that you please:

Add a Toolbox example with a large number of generated options (1000?  What kind of item count do we think will still be reasonable in terms of performance?)
Test the mobile implementation (uses the same library)
Add a note in the Select component header re. the use of WindowedSelect
Add a CHANGELOG entry

Those will let us tick off a few more of our PR checklist items.  Thanks 🙏

In the previous state, this was broken for everything except 'normal' selects- create and async did not work at all. Now they should work, I believe.

* Add a Toolbox example with a large number of generated options (1000?  What kind of item count do we think will still be reasonable in terms of performance?)

The branch https://github.com/xh/toolbox/tree/bigSelect has a new select with 5000 items on app/forms/inputs.

I am not sure menuIsOpen works correctly, i.e. the onKeyDown logic at Select:240-Select:249 does not seem to work properly.

I have found out in more detail what is wrong with menuIsOpen.
react-windowed-select does not export a creatable or async WindowedSelect. It exports only a 'basic' WindowedSelect. Therefore, I have to make Creatable and Async WindowedSelect myself.
I do this here:

const Select = WindowedSelect;
const AsyncSelect = (makeAsyncSelect(Select));
const AsyncCreatable = (makeCreatableSelect(AsyncSelect));
const Creatable = (makeCreatableSelect(Select));

makeAsyncSelect and makeCreatableSelect are functions exported from react-select which add creatable or async functionality to an existing select.
The problem is that there is an additional step in creating an async or creatable Select. Take a look at the very bottom of this file:
https://github.com/JedWatson/react-select/blob/master/packages/react-select/src/Creatable.js
const SelectCreatable = makeCreatableSelect<ElementConfig<typeof Select>>(
  Select
);

export default manageState<ElementConfig<typeof SelectCreatable>>(
  SelectCreatable
);

SelectCreatable is wrapped in a StateManager, which is what handles the controllable props. We do not have access to the manageState function, so we cannot wrap our Selects in a StateManager and cannot get its functionality.
This breaks at least the functionality of menuIsOpen. On Select.js, the following branch (lines 243-247) will never be taken on a creatable or async Select:
    const {menuIsOpen} = this.reactSelectRef.current ? this.reactSelectRef.current.state : {};
    if (menuIsOpen && (e.key == 'Escape' || e.key == 'Enter')) {
        console.log('menu is open');
        e.stopPropagation();
    }

It may also break the functionality of onChange, onInputChange, and the other things listed as controllable props. I believe this won't be a problem, since we already handle onChange, value, onInputChange, and inputValue ourselves.

Something else noteworthy: the library's handling of the StateManager is also buggy! Compare the code for Async and the code for Creatable:
https://github.com/JedWatson/react-select/blob/master/packages/react-select/src/Creatable.js
const SelectCreatable = makeCreatableSelect<ElementConfig<typeof Select>>(
  Select
);

export default manageState<ElementConfig<typeof SelectCreatable>>(
  SelectCreatable
);

It first makes the Select creatable, then wraps it in a StateManager.
https://github.com/JedWatson/react-select/blob/master/packages/react-select/src/Async.js
const SelectState = manageState<ElementConfig<typeof Select>>(Select);

export default makeAsyncSelect<ElementConfig<typeof SelectState>>(SelectState);

It first wraps the Select in a StateManager, then makes it async.
They do not do the same thing. And the implementation for Async is broken! Add a print statement inside the branch on line 243-247:
                const {menuIsOpen} = this.reactSelectRef.current ? this.reactSelectRef.current.state : {};
                if (menuIsOpen && (e.key == 'Escape' || e.key == 'Enter')) {
                    console.log('menu is open')
                    e.stopPropagation();
                }

The expected behavior is that hitting Escape while the Select is open will cause 'menu is open' to be printed to the console. This works as expected for a normal Select and a creatable Select, but not for an async Select.

Lee and I discussed this, and we believe it's appropriate to just swallow the keydown events even if the menu is closed. This means we no longer need to care about whether or not the menu is open, so my concerns are probably not a problem.

@amcclain

Test the mobile implementation (uses the same library)
I have tested the mobile implementation using the Selects available in the mobile version of Toolbox. However, I do not know how to test the Async or Creatable selects on mobile.

Peter, Colin, and I discussed.  There are issues with using this for the asyn/ createable/async creatable cases, and known bugs around monitoring the menu open state, that will effects how we handle key presses.
Shame to let this cause problems for the majority of simple selects by trying to force its use automatically.
Safer to expose this as an option --  useWindowedSelect that can be opt-in for apps.  We should document it as having known limitations, and primarily for non-async non-creatable cases.

I think this is ok for merging now.  Its a very safe change now, that should have no impact, unless the options is set.  Toolbox example has some issues we could go over on monday

After discussing with Anselm late on Friday, we decided to go with a safer, and more minimal change that only effects basic selects.  We can always revisit, and remove this restriction, if needed and if react-windowed-select fixes its bugs.
Also tested large option lists on mobile.  Weirdly, performance for large option lists seems to be fine, so not adding the prop to the mobile component at this time.


[Do Not Merge][Analysis] Feature/Evaluate Django-Querycount [PLAT-482]

Purpose
Evaluate adding django-querycount to make devs aware of expensive endpoints and make optimizations easier.
Changes
Add the django-querycount middleware.  Only works when DEBUG mode set to true.
In action
Make a local request to GET /v2/nodes/

Make a local request to POST /v2/nodes/

QA Notes
No QA testing needed.
Documentation

Side Effects

Ticket
https://openscience.atlassian.net/browse/PLAT-482

Closing as we won't be merging this for the time being.


4K movies take too long to start

Using play-glfw on a 4K mp4 movie takes tlRender about 4 seconds to start.  With tlplay creating the thumbnails it is much worse and playback is not smooth at least until the thumbnails are created.
In comparison, mrViewer opens the 4K movie in 1 second or less.

Hi, I think the slow down is from tlRender opening and closing the movie file multiple times. I've created a new git branch to implement a cache so that the opening and closing is minimized. If you have time could you test it out and see if there is an improvement on your test files? On my test files it seems about twice as fast. The branch is called "open" and "aad0b46cdef6a80bf65b57d1ea652c65279c35f6" is the latest commit. Thanks!

Thanks for testing out the "open" branch, I just merged it into main.


Apostrophe generation

Add apostrophe.js file to run the apostrophe:generation command

Nothing is changed on the multisite setup, I only did the following:

Moved the siteConfig to another file
Create apostrophe file to run cli commands like: apostrophe:generation based on the SAMPLE_DB database.
Removed unused code.
FIx apostrophe:generation errors in javascript.

Created PR to Amsterdam


ATTU 2.3.3 can't login with milvus-standalone 2.3.3

Describe the bug:
ATTU 2.3.3 can't login with milvus-standalone 2.3.3
Here is my docker compose file, it works fine for ATTU 2.3.2 and milvus-standalone 2.3.1. But when I upgrade ATTU to 2.3.3 and milvus-standalone to 2.3.3, it can't login as expect.
`
version: '3.5'
services:
etcd:
container_name: milvus-etcd
image: quay.io/coreos/etcd:v3.5.5
environment:
- ETCD_AUTO_COMPACTION_MODE=revision
- ETCD_AUTO_COMPACTION_RETENTION=1000
- ETCD_QUOTA_BACKEND_BYTES=4294967296
- ETCD_SNAPSHOT_COUNT=50000
volumes:
- ${DOCKER_VOLUME_DIRECTORY:-.}/volumes/etcd:/etcd
command: etcd -advertise-client-urls=http://<IP_ADDRESS>:2379 -listen-client-urls http://<IP_ADDRESS>:2379 --data-dir /etcd
healthcheck:
test: ["CMD", "etcdctl", "endpoint", "health"]
interval: 30s
timeout: 20s
retries: 3
minio:
container_name: milvus-minio
image: minio/minio:RELEASE.2023-03-20T20-16-18Z
environment:
MINIO_ACCESS_KEY: minioadmin
MINIO_SECRET_KEY: minioadmin
ports:
- "9001:9001"
- "9000:9000"
volumes:
- ${DOCKER_VOLUME_DIRECTORY:-.}/volumes/minio:/minio_data
command: minio server /minio_data --console-address ":9001"
healthcheck:
test: ["CMD", "curl", "-f", "http://localhost:9000/minio/health/live"]
interval: 30s
timeout: 20s
retries: 3
standalone:
container_name: milvus-standalone
image: milvusdb/milvus:v2.3.3
command: ["milvus", "run", "standalone"]
environment:
ETCD_ENDPOINTS: etcd:2379
MINIO_ADDRESS: minio:9000
volumes:
- ${DOCKER_VOLUME_DIRECTORY:-.}/volumes/milvus:/var/lib/milvus
healthcheck:
test: ["CMD", "curl", "-f", "http://localhost:9091/healthz"]
interval: 30s
start_period: 90s
timeout: 20s
retries: 3
ports:
- "19530:19530"
- "9091:9091"
depends_on:
- "etcd"
- "minio"
attu:
container_name: attu
image:  zilliz/attu:v2.3.3
environment:
MILVUS_URL: standalone:19530
ports:
- "8000:3000"
depends_on:
- "standalone"
networks:
default:
name: milvus
`
Browser console :
     index-esjUO3SY.js:107    POST http://<IP_ADDRESS>:3000/api/v1/milvus/connect net::ERR_CONNECTION_REFUSED

Steps to reproduce:

use above attu&milvus version in docker-compose
do login on http://localhost:8000/#/

Attu version:
2.3.3
Milvus version:
2.3.3

attu 2.3.2 + milvus-standalone 2.3.2 can login as expect.

@shanghaikid  thank you. attu can login after adjust the port.


TypeError: Gradient accumulation supports only int and dict types

TypeError: Gradient accumulation supports only int and dict types.
How to deal with this problem？

Are you using the pokemon fine tune example?
Fix the typo in the notebook. There's a reference to a variable called NUM_GPUS but it doesn't exist. The actual name is N_GPUS.


⚠ BroadcasTheNet has degraded performance

In a56a1f9, BroadcasTheNet ($BTN) experienced degraded performance:

HTTP code: 200
Response time: 2047 ms

Resolved: BroadcasTheNet performance has improved in 294eaf9.


chore: upgrade supabase-auth-helpers to v1.4.1

Do we still want this? 🙏

ah no, sorry, this is outdated.


Eliminate errors if SupportServiceProvider not loaded.

Leaves permissions migration to auth package, but there were still a few unrelated minor repairs that are worth keeping.


Remark --> document hooks

Right now we only document the HOC.

add docs for the hooks and render props component
note how this helps solve issues with static queries
document the globalJsonForm hook as well for remark

closing, duplicate of #112


Attachments are not being parsed

Hi there! First off: I am amazed by mailin, awesome piece of software. Saved me days in development. I have a slight issue with the attachments though:

I am not using the webhook, but rather the embedded mailin server. For that, I plugged myself into mailin.on('message', function(connection, data, content) {});. And this works fine, but somehow data.attachments is always non-existing. For debugging I printed both data and content:
Excerpt from content:
--Apple-Mail=_1CE57DD1-5830-4E39-BB06-17F285909745
Content-Disposition: attachment; filename=X
Content-Type: application/octet-stream;name="X"
Content-Transfer-Encoding: 7bit

TEST

But in data (shortened):
  { html: 'html',
    headers:
     { received: [ Object object ],
       from: 'Julian Krenge<EMAIL_ADDRESS>       to: 'Tracking Agent<EMAIL_ADDRESS>       subject: 'test',
       'thread-topic': 'test',
       'thread-index': 'AQHRJ1UYzntn5mE26ECrsi6sDecCjw==',
       date: 'Wed, 25 Nov 2015 07:44:20 +0000',
       'message-id'<EMAIL_ADDRESS>       'accept-language': 'de-DE, en-US',
       'content-language': 'en-US',
       'x-ms-has-attach': 'yes',
       'x-ms-tnef-correlator': '',
       'authentication-results': 'spf=none (sender IP is ) smtp.mailfrom=julian@parcellab.com;',
       'x-ms-exchange-messagesentrepresentingtype': '1',
       'x-originating-ip': '[<IP_ADDRESS>]',
       'x-microsoft-exchange-diagnostics': 'string,
       'x-microsoft-antispam': 'UriScan:;BCL:0;PCL:0;RULEID:;SRVR:AMXPR07MB184;',
       'x-ld-processed': 'c9ad2ae3-2857-429e-bddf-50e548a323c0,ExtAddr',
       'x-microsoft-antispam-prvs'<EMAIL_ADDRESS>       'x-exchange-antispam-report-test': 'UriScan:;',
       'x-exchange-antispam-report-cfa-test': 'BCL:0;PCL:0;RULEID:(601004)(2401047)(5005006)(8121501046)(520078)(10201501046)(3002001);SRVR:AMXPR07MB184;BCL:0;PCL:0;RULEID:;SRVR:AMXPR07MB184;',
       'x-forefront-prvs': '0771670921',
       'x-forefront-antispam-report': 'string',
       'received-spf': 'None (protection.outlook.com: parcellab.com does not designate permitted sender hosts)',
       spamdiagnosticoutput: '1:23',
       spamdiagnosticmetadata: 'NSPM',
       'content-type': 'multipart/signed; boundary="Apple-Mail=_F0CC39CA-E1ED-482A-B618-7EFEA7071FED"; protocol="application/pkcs7-signature"; micalg=sha1',
       'mime-version': '1.0',
       'x-originatororg': 'parcellab.com',
       'x-ms-exchange-crosstenant-originalarrivaltime': '25 Nov 2015 07:44:20.6059 (UTC)',
       'x-ms-exchange-crosstenant-fromentityheader': 'Hosted',
       'x-ms-exchange-crosstenant-id': 'c9ad2ae3-2857-429e-bddf-50e548a323c0',
       'x-ms-exchange-transport-crosstenantheadersstamped': 'AMXPR07MB184' },
    subject: 'test',
    messageId<EMAIL_ADDRESS>    priority: 'normal',
    from: [ Array ],
    to: [ Array ],
    date: Wed Nov 25 2015 07:44:20 GMT+0000 (UTC),
    receivedDate: Wed Nov 25 2015 07:44:21 GMT+0000 (UTC),
    dkim: 'failed',
    spf: 'failed',
    spamScore: 0,
    language: 'romanian',
    cc: [],
    connection: [ Object object ],
    envelopeFrom: { address<EMAIL_ADDRESS>args: false },
    envelopeTo: [ { address<EMAIL_ADDRESS>args: false } ] }

Then I tried to check the webhook: But enabling this even throws an exception and mailin crashes.
Any ideas? :-)

Any updates on this?

 :-)

What happened? I'm having a similar issue: the attachment array is populated correctly, but I have no way to access the actual content of the attachment. I tracked it down to https://github.com/andris9/mailparser#attachment-streaming, and it's pretty clear how to get the actual content from a MailParser instance, but I just don't know how to access that instance from within mailin. Any ideas?

For me, the whole attribute attachments was gone, so I think it was rather removed by different code before processing. But, I hacked it a little bit:
I basically store the whole raw email, as I want to archive the file anyways:
mailin.on('message', function(connection, data, content) {
  fs.writeFile(output, content, function(err) {
    if (err) console.log(err);
  });
});

Then I monitor the folder where the emails are stored and extract contents manually:
AttachmentMaster / index.js.
We will fix this, but for the meantime this also kind of works. :-)

Thanks! :+1:


⚠ Bofrost.fr has degraded performance

In dd07c6b, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 825 ms

Resolved: Bofrost.fr performance has improved in 2c14c1c after 8 minutes.


watchOS storyboards do not support target device type "iphone".

Trying to run specs with PCK watchKit, but when I add the storyboard in the specs target i get this error. Xcode 7 seems to add some sort of setting that makes it so that PCK cannot load the storyboard and instantiate the interface controller. We are trying to run specs as a separate specs target since running them with the test bundle seems to fail.

Xcode 7 no longer supports compiling WatchKit storyboards within targets that deploy on iOS. The technique I used in #156 is to create a separate WatchKit app target, add the storyboard to that app, and embed the WatchKit app inside the spec bundle.


Improve File Extension Logic

Add alternative LICENSE and README names
Add mjs, cjs, mts, cts
Undetect filenames without extension, e.g. py (except LICENSE and README (there should probably be more))
Lookup level2ExtensionNames first even for filenames with more than 2 dots, e.g. foo.bar.spec.js
Refactor to remove duplicated logic

Thanks for cleaning up the code!


Remove templating

This is the first few commits to #161 that just affect the C code (and not setuptools). It's been universally approved so far so I'll merge it tomorrow morning if there are no objections and we can hash out setuptools/testing later.
Separated out so #164 can be rebased onto it.

Appveyor fails as usual, but passes in #161 which includes these commits.

+1

+1


Fix osx mangling hack

Rust automatically adds one _ in the symbol. If we don't remove it, we will
endup getting three _.
r? @emilio

Thanks for the pull request, and welcome! The Servo team is excited to review your changes, and you should hear from @emilio (or someone else) soon.

Thanks a lot, looks great!
@bors-servo r+

:pushpin: Commit 26b9143 has been approved by emilio

:hourglass: Testing commit 26b9143884a818ed040afb6a9a21a2fe4cb3822b with merge 595b9a3861751a3364b0b98b5434527e5981f962...

:sunny: Test successful - status-travis
Approved by: emilio
Pushing 595b9a3861751a3364b0b98b5434527e5981f962 to master...


Fix all linting errors in the examples

Also run npm run lint on the examples in the CI steps.
I confirm I have read the contributing guidelines and signed the Contributor License Agreement.

Hey @erikdubbelboer,
It looks like you've been running in BEAST MODE over the holidays. Thanks so much for these PRs! The team will slow return from holidays over the next week or so and then we'll start to review and merge.
Thanks again!


show alignments

I can see just lines on stdout, how can I see the produced alignments and export to a file?
Thanks for the your efforts.

I'm not sure I understand your question - can you post the command you are running and the terminal output you get?

I got results form of like as you shown:
[0]:[0]:0.156006
[1]:[1]:0.160997
[2]:[2]:0.217155

But is there any chance to get like this:
Source sentence : target sentence : score
and so on...

The code doesn't produce that format directly, but the output gives the sentence indexes into each document, so you can easily convert it to the format you want.


Button to add event to my agenda

Available after you register to the event ?

Suggestion:

Use a Button with a simple (Menu component) with Google, Outlook, ICS
Links can be found here: https://github.com/jasonsalzman/react-add-to-calendar/blob/master/src/helpers/index.js#L32

We could also send confirmation emails after joining events with the ICS file included.
See https://stackoverflow.com/questions/24323452/how-to-send-a-meeting-request-correctly-with-nodemailer


Accept char iterator as input

Work-in-progress to handle #3.
Please let me know your thoughts on this style. I just worked on katakana_to_hiragana(). Once we figure out how the best way to do this, I will work on the other modules.
Add iterator input APIs to

is_hiragana

[ ] is_hiragana()

is_japanese

[ ] is_japanese()

is_kana

[ ] is_kana()

is_kanji

[ ] contains_kanji()
[ ] is_kanji()

is_katakana

[ ] is_katakana()

is_mixed

[ ] is_mixed()
[ ] is_mixed_pass_kanji()

is_romaji

[ ] is_romaji()

to_hiragana

[ ] to_hiragana()
[ ] to_hiragana_with_opt()

to_kana

[ ] to_kana()
[ ] to_kana_with_opt()

to_katakana

[ ] to_katakana()
[ ] to_katakana_with_opt()

to_romaji

[ ] to_romaji()
[ ] to_romaji_with_opt()

tokenize

[ ] tokenize()
[ ] tokenize_detailed()
[ ] tokenize_with_opt()

trim_okurigana

[ ] is_invalid_matcher()
[ ] is_leading_without_initial_kana()
[ ] is_trailing_without_final_kana()
[ ] trim_okurigana()
[ ] trim_okurigana_with_opt()

utils

[ ] hiragana_to_katakana()
[X] katakana_to_hiragana()
[ ] romaji_to_hiragana()

There is a place where the next char is peeked:
https://github.com/PSeitz/wana_kana_rust/blob/c45ebcdf3e8acc6421e967e792374ac390947df8/src/to_hiragana.rs#L29-L31
So it would require a peekable iterator I think
https://doc.rust-lang.org/std/iter/struct.Peekable.html
There may be other similar cases.

On second thought, we could always just use the minimal api instead of a uniform one, which would not require Peekable here. What do you think?

On second thought, we could always just use the minimal api instead of a uniform one, which would not require Peekable here. What do you think?

@PSeitz What do you mean by minimal vs. uniform API?
In some cases, we can get away with avoiding the "direct indexing" approach. For katakana_iter_to_hiragana_with_opt(), I added a previous_char variable to track the previous character (instead of indexing into the previous index).

@PSeitz What do you mean by minimal vs. uniform API?

I mean defining a single iterator api which is used by all methods, vs minimal api everywhere.


Compatibility with Laravel 5.1?

It seems that requires Laravel 5.2. It would be great if it has compatibility with Laravel 5.1 (because this is a LTS version).
Thanks!

Sure! Done: https://github.com/JeroenNoten/Laravel-AdminLTE/releases/tag/v1.6.3
Note that the php artisan make:adminlte command only works for Laravel 5.2 and higher, because of the dependency on php artisan make:auth. Other functionality should work in Laravel 5.1 now.


Color upgrades

Color hex and rgb
I divided configuration for color into two parts

HEX ->#FFFFFF or 0xFFFFFF
RGB -> 255, 255, 255

Safety removal of spaces
When one of the user tried out msetup, when they came to buttons region, where the split is " ", they made two or three, which caused a very big issue, and same for the new line

Color hex and rgb
I divided configuration for color into two parts

HEX ->#FFFFFF or 0xFFFFFF
RGB -> 255, 255, 255

Safety removal of spaces
When one of the user tried out msetup, when they came to buttons region, where the split is " ", they made two or three, which caused a very big issue, and same for the new line

Update to Color 'hex and 'rgb'`
Merged into one /config color command, but this time it supports 3 things

[x] Hex -> starting with 0x or #
[x] RGB -> seperated by comma
[x] Roles -> type <role> in to get alfred's role colour [Expecting it to change as the role colour changes]


download link for model.json needed for unanimity

The page https://github.com/PacificBiosciences/unanimity describes how to place a file named model.json in a given location
In such cases, download the latest version of the model parameters and place them in a subdirectory of ${SMRT_CHEMISTRY_BUNDLE_DIR}:
but I find nowhere a link to download that file and do not find it in my SMRTLink tree either.

Those instructions are mainly meant for people using SMRTtools from the tarball. Bioconda releases of consensuscore2 always include the latest chemistry, so you should in general never have to sideload the chemistry information, only keep your bioconda installation up-to-date.


API /api/dart-internal-subscription has been failing consistently due to Key not found

> ERROR 2023-08-07T16:00:54.409Z Found 0 entries for builder Windows engine_release_builder
> ERROR 2023-08-07T16:00:54.409Z Failed to find an existing task for the buildbucket build: HTTP 500: Expected to find 1 task for Windows engine_release_builder, but found 0
> ERROR 2023-08-07T16:00:54.409Z Creating Task from Buildbucket result
> ERROR 2023-08-07T16:00:54.409Z Creating task from buildbucket result: build(id:<PHONE_NUMBER>995263905, builderId: dart-internal/flutter/Windows engine_release_builder, number: 579, status: Status.success, tags: null)
> ERROR 2023-08-07T16:00:54.409Z Repository: engine
> ERROR 2023-08-07T16:00:54.409Z Branch: flutter-3.13-candidate.13
> ERROR 2023-08-07T16:00:54.409Z Hash: 590ff28649eb547b6f4dd4ac162a7c373d1042be
> ERROR 2023-08-07T16:00:54.409Z Slug: flutter/engine
> ERROR 2023-08-07T16:00:54.409Z Start/end time (ms):<PHONE_NUMBER>724,<PHONE_NUMBER>803
> ERROR 2023-08-07T16:00:54.417Z Key not found: Commit:flutter/engine/flutter-3.13-candidate.13/590ff28649eb547b6f4dd4ac162a7c373d1042be #0 DatastoreDB.lookupValue (package:gcloud/src/db/db.dart:382:9) <asynchronous suspension> #1 Task.fromBuildbucketBuild (package:cocoon_service/src/model/appengine/task.dart:161:27) <asynchronous suspension> #2 DartInternalSubscription.post (package:cocoon_service/src/request_handlers/dart_internal_subscription.dart:96:22) <asynchronous suspension> #3 RequestHandler.service.<anonymous closure> (package:cocoon_service/src/request_handling/request_handler.dart:53:24) <asynchronous suspension> #4 SubscriptionHandler.service (package:cocoon_service/src/request_handling/subscription_handler.dart:138:5) <asynchronous suspension> #5 main.<anonymous closure>.<anonymous closure> (file:///app/bin/server.dart:293:11) <asynchronous suspension>
> ERROR 2023-08-07T16:00:54.417Z Failed to process Instance of 'PushMessage'. (500) Key not found: Commit:flutter/engine/flutter-3.13-candidate.13/590ff28649eb547b6f4dd4ac162a7c373d1042be #0 DatastoreDB.lookupValue (package:gcloud/src/db/db.dart:382:9) <asynchronous suspension> #1 Task.fromBuildbucketBuild (package:cocoon_service/src/model/appengine/task.dart:161:27) <asynchronous suspension> #2 DartInternalSubscription.post (package:cocoon_service/src/request_handlers/dart_internal_subscription.dart:96:22) <asynchronous suspension> #3 RequestHandler.service.<anonymous closure> (package:cocoon_service/src/request_handling/request_handler.dart:53:24) <asynchronous suspension> #4 SubscriptionHandler.service (package:cocoon_service/src/request_handling/subscription_handler.dart:138:5) <asynchronous suspension> #5 main.<anonymous closure>.<anonymous closure> (file:///app/bin/server.dart:293:11) <asynchronous suspension>

Failed to process Instance of 'PushMessage'. (500) Key not found: Commit:flutter/engine/flutter-3.13-candidate.13/590ff28649eb547b6f4dd4ac162a7c373d1042be
#0      DatastoreDB.lookupValue (package:gcloud/src/db/db.dart:382:9)
<asynchronous suspension>
#1      Task.fromBuildbucketBuild (package:cocoon_service/src/model/appengine/task.dart:161:27)
<asynchronous suspension>
#2      DartInternalSubscription.post (package:cocoon_service/src/request_handlers/dart_internal_subscription.dart:96:22)
<asynchronous suspension>
#3      RequestHandler.service.<anonymous closure> (package:cocoon_service/src/request_handling/request_handler.dart:53:24)
<asynchronous suspension>
#4      SubscriptionHandler.service (package:cocoon_service/src/request_handling/subscription_handler.dart:138:5)
<asynchronous suspension>
#5      main.<anonymous closure>.<anonymous closure> (file:///app/bin/server.dart:293:11)
<asynchronous suspension>

/cc @drewroengoogle

I think the following is occurring:
A new candidate branch is cut, which cocoon doesn't schedule builds for. However, the LUCI scheduler still schedules builds, and when it tries to find a commit to link the task to, it doesn't find it, because it is relying on one of the cocoon tasks to create a commit.
This is probably a P1 given that it isn't affecting any releases, but is causing a lot of noise. I'll take a look at creating a fix today.

ready to close?

@ricardoamador confirmed that is good to close


perlPackages.TextWrapI18N: don't propagate glibc

Motivation for this change
Having this as a dependency could break your build because the include order would get messed up.
Relates #110346.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 113180 at 53abd00b run on x86_64-linux 1

7 packages built:

autofs5
modem-manager-gui
perl530Packages.Po4a
perl530Packages.TextWrapI18N
perl532Packages.Po4a
perl532Packages.TextWrapI18N
sssd

Result of nixpkgs-review pr 113180 at 53abd00b run on aarch64-linux 1

7 packages built:

autofs5
modem-manager-gui
perl530Packages.Po4a
perl530Packages.TextWrapI18N
perl532Packages.Po4a
perl532Packages.TextWrapI18N
sssd

@ofborg build pkgsMusl.perlPackages.TextWrapI18N

Thanks for the contribution


How to completely uninstall DBeaver on Mac

System information:

macOS Catalina 10.15.6 (19G73)
DBeaver version: 7.1.5

Describe your question:
I want to completely remove everything DBeaver related and start over fresh with a brand new install that will not remember any of my preferences, cache, history, settings, connections, drivers, etc.
How can I do this on Mac? I've tried a few routes and after I reinstall, I can tell on opening it again that not everything got removed (it'll remember connection info, theme, etc.).

If you need to cleanup all your local files/settings then you can simple delete directory ~/Library/DBeaverData.
Don't forget to close DBeaver before that.

On MacOS dark theme is default if you use dark theme in MacOS itself.
In any case you can change it in the Preferences->Appearance.

Thanks. Closing.


Infrastructure view for System HA stacks also includes non-system HA hosts

Rancher Version: 1.1.0-dev5
Setup Details: (single node rancher vs. HA rancher, internal DB vs. external DB)
Rancher HA with 3 hosts
Steps to Reproduce:

Deploy HA cluster with 3 servers

Add an environment with at least 1 host

Select Admin | High Availability.

Select the Infrastructure tab

Results:
The select environment is System HA and hosts are shown.
The list of hosts includes both the HA servers (3 instances) plus the hosts of the previously selected environment.
Expected:
Only the HA hosts (servers) should be shown.
I noted that requesting the URL for the System HA stack (https://rancher.mydomain.io/v1/projects//hosts) returns only the System HA hosts. So something else on the Infrastructure tab is requesting hosts from a different environment and they are being merged together.

Once on the Infrastructure tab, if I do a browser refresh, then only the System HA hosts are displayed.

Going into the HA page is changing the selected env/project to System HA but not clearing the browser store of all the other records from the previous env...

No longer applies in 1.2, HA is configured by CLI.


⚠ Turnos 24x7 has degraded performance

In b1041e9, Turnos 24x7 (https://reservas-dev.alternativasinteligentes.com/) experienced degraded performance:

HTTP code: 200
Response time: 879 ms

Resolved: Turnos 24x7 performance has improved in 2afe9cd.


feat: show waveform thumbnails for audio pieces, if available

This PR is being opened on behalf of myself.

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

Feature

What is the current behavior? (You can also link to an open issue here)

Audio pieces have limited inspectability.

What is the new behavior (if this is a feature change)?

Audio pieces now display a waveform image on hover, if available from Package Manager.

Other information:

This PR depends on https://github.com/nrkno/sofie-package-manager/pull/93
Status

[ ] Code documentation for the relevant parts in the code have been added/updated by the PR author
[x] The functionality has been tested by the PR author
[ ] The functionality has been tested by NRK

I think this is a very nice feature, but I'm hesitant if we should base this of what we currently have and increase code duplication. I think it would be nice if we could refactor the components so that we have a generic component that can display a thumbnail, and it would be able to do that for any Piece that has a corresponding thumbnail package (with a hoverscrub video proxy taking over, if available). The current architecture is a holdover from the MediaObject-based previews system and is very inflexible. Having a new architecture based on the flexibility of packages would allow for previews and thumbnails of any content, as long as Package Manger is able to provide a preview for it.

Closing this one due to inactivity.


Encoding issue with -

This should work:
scala> import records.Rec
import records.Rec

scala> Rec("crazy-name" -> "Hannah").`crazy-name` // works
res0: String = Hannah

scala> Rec(`crazy-name` = "Hannah").`crazy-name` // does not work
java.util.NoSuchElementException: key not found: crazy-name

It does not seem to be restricted to dashes -, rather a general problem with special characters:
Rec(`@_id` = "42").`@_id` // does not work either
Exception in thread "main" java.util.NoSuchElementException: key not found: @_id

Is this maybe just a bug, because the name is decoded?
scala> import scala.language.dynamics
scala> object Foo extends Dynamic{ def applyDynamicNamed(m: String)(args:(String, Any)*) = println(args) }
scala> Foo(`foo-bar`=5)
WrappedArray((foo$minusbar,5))

We should undo encoding when we use Rec(foo = 1) syntax. We can do this after fixing #118.

We cannot undo encoding. After encoding, there is no difference between $plus and + anymore (for example). But the intended field names are (obviously) different. If applyDynamicNamed gives encoded names, this is a blocker for #118.

I think this is a bug in Scala. Why doesn't it let the user chose if he wants to encode names or not? We could try to submit a PR and try to fix it in Scala.

We may have a misunderstanding about what is the encoded and what is the decoded version.
Is $minus encoded and - decoded?
@gzm0 aren't encode and decode inverse functions? The compiler uses them.

They are not unfortunately. Try your example with Foo($plus = 1).

@cvogt Nope. Encoding and decoding is not reversible.
@vjovanov Well... this can be considered a bug in Scala or the intended behavior of applyDynamicNamed. In any case, this will unlikely be changed before 2.13 (if even there) due to compatibility concerns. So we are pretty much stuck with it.

Well guys pick one:

Treat "$plus" as "+". Who would ever name stuff with "$plus" :) For the enthusiasts we can provide some support through extension methods.
Drop the whole feature and back to tuples (this is not our mojo from the very beginning).
Try to do voodoo with source positions (this is our mojo from the very beginning :).

I will think about this for a bit before I choose.

I gave it a thought; we can do 1 and provide an alternative constructor with strings Rec.forEncodedFieldNames(("$plus", 1)). We already have the code for this.

To my knowledge $ is not a valid character in a non-compiler generated identifier. I don't think we need to support that. Not sure the spec is actually explicit about that, but if you use $ multiple things can go wrong, i.e. conflicts with compiler generated methods or objects.

@gzm0 is encode decode reversible if we exclude $ from the valid input characters?

Hmmm... I would not sign that but can't think of a case where it would break..

Sure, however, somebody's database does not really care about Scala conventions.

@gzm0
scala> Foo(`$minus-`=5)
WrappedArray(($minus$minus,5))

indeed looks evil. I'd be ok with not supporting $, i.e. explicit compile error. And yes, applyDynamic should probably provide "$minus-"

Hmmm.. but some occurrences of $ will work (e.g. $foo)... If $ should be a compile error is discussable... But maybe it is better to be safe than sorry...

Restricting all $ is a bit radical--I am afraid users would have to use tuples too often. Also, I do not see how treating $minus- as -- in record construction can lead to fatal errors that are not detected at compile-time?

Well, it shouldn't. But currently it seems we are not consistent about the usage of decoded vs encoded names.
I agree that fully restricting $ is too radical. Users will get a compile time error in this case and can act upon it.

I think we should avoid surprising behaviors. "Works some of the time" or "read this list of cases first" is not an easy interface. We should at least make sure that if we don't support cases, that they fail at compile time.
@lrytz I remember from my early LAMP days that $ was not a valid character for identifiers. But maybe I am making this up. Do you have any insight here?

$ is valid by the spec, but should not be used. there are things in the compiler like (approximately): 'def isAnonymousClass = name.endsWinth("$anon")'. sure, this is brittle already, but there's plenty of assumptions about $

another point: having a $ in a name makes problems using it in an interpolated string

Sounds to me like just blocking any usage of $ in a field name would be a good idea. If representing a database table people seem to be better off having a name mapping scheme. We can always relax the restriction later, if new evidence for the need of $ arises. WDYT @gzm0 @vjovanov ?

It indeed seems so that this is the best option for now. It is true that it's better if we are restrictive now and relax later than v.v.

So back to the start: without $ can we get rid of the tuple syntax ;)?

Ahahahahahaha :)

(wrong ticket for discussion, moving over).

also, just hit a use case, where records need $... serializing a record into json to be used as a MongoDB query

Ahahahahahaha :) So I guess we're good :P

? good how? support $ and tuple syntax? seems like it :-. Continuing the chain of brokenness...

interestingly on the just mentioned query use case, the applyDynamicNamed may give a false sense of type-safety

I am fine with disallowing $ in non-tuple syntax to avoid decoding issues (I'd actually encourage it after all this discussion).

good idea, let's start with that


Make all migrations / ontology maintenance run automatically

This is likely a "bigger" ticket, but I wanted to add this as a conversation point.
Since June 2021, we have been doing a periodic manual ontology update in minerva. This procedure currently requires two people, a pre-coordinated and publically communicated time, and around 2hrs. To date, it has been done around 64 times.
Additionally, given the manual nature and the fact that this is a high-value production system, when things do inevitably go wrong, we need to be cautious, sometimes leading to longer outages, redos, and general stress; e.g.:

https://github.com/geneontology/noctua/issues/904#issuecomment-2253546509
https://github.com/geneontology/noctua/issues/900#issuecomment-2226496543
https://github.com/geneontology/noctua/issues/861#issuecomment-1924590779  / https://github.com/geneontology/project-management/issues/34#issuecomment-1935258352

Finally, given that this is a production system, we try to do outages when things are "quite", typically meaning that we are encroaching on hours that would typically be considered "off" for the people performing these tasks.
Given all this, I would propose that automation would reduce errors and time spent, at least in the long-run. There are two approaches:

have minerva do migrations and updates automatically and internally
using automation scripts, create a turnkey system where a newly migrated and updated system is brought up (and automatically tested)

Tagging @vanaukenk

Adding to the pile:
https://github.com/geneontology/noctua/issues/912


'The input is not a valid Base-64 string' ...

Hi,
Using this code:
            string p8Key = ReadResource("AuthKey_XXXXXXX.p8");

            var settings = new ApnSettings()
            {
                AppBundleIdentifier = "com.hankinsoft.appwage",
                ServerType = ApnServerType.Development,
                TeamId = "XXXXXXX",

                P8PrivateKeyId = "XXXXXXX",
                P8PrivateKey = p8Key
            };

            string deviceToken = "<base64 encoded device token>";

            using (HttpClient httpClient = new HttpClient())
            {
                var apn = new ApnSender(settings, httpClient);

                var notification = new AppleNotification()
                {
                    Aps = new AppleNotification.ApsPayload()
                    {
                        AlertBody = "Reports are now available",
                        ContentAvailable = true
                    }
                };

                apn.SendAsync(notification, deviceToken).Wait();
            } // End of httpClient

And notification being:
    public class AppleNotification
    {
        public class ApsPayload
        {
            [JsonProperty("alert")]
            public string AlertBody { get; set; }

            [JsonProperty("content-available")]
            public bool ContentAvailable { get; set; }
        }

        // Your custom properties as needed

        [JsonProperty("aps")]
        public ApsPayload Aps { get; set; }
    }

I receive the error:

The input is not a valid Base-64 string as it contains a non-base 64 character, more than two padding characters, or an illegal character among the padding characters.

I'm not exactly sure what is not a valid Base-64 string. Is there any way to get more information about this?

@hankinsoft make sure you remove all the ------------ CERTIFICATE BLAH BLAH -------- from the cert file

Yep, that did it. Thanks!

Awesome!


Entire Document TextObject

Is your feature request related to a problem? Please describe.
For gq, gw, = operators, I often have to gg then execute the operator with G motion, e.g. gggqG, gggwG, gg=G, ...etc.
I looked through the docs/issues and didn't find something relevant to this.
Describe the solution you'd like
I would like to refer to the entire document/buffer as a textobject, like this, e.g. gqae, gwae, =ae.
Describe alternatives you've considered
gggqG, gggwG, gg=G, ...etc.

Is tree-sitter the right tool for defining that? I think https://github.com/kana/vim-textobj-entire is probably more suited since the entire documented can't be expressed easily via queries (would need a query with the root node per language)

Treesitter is not a right tool for such action. Just use the repo mentioned, or I believe there is a neovim various textobject plugin that does this.


Accessibility: [Screen Reader]:  Selecting spacebar/enter when the screen reader focus is on container response options, always selects the first response

Actual
With JAWS ON and keyboard tab navigation, notice that the screen reader focus comes to “response container”. JAWS announces all the responses/answers one after the other and then says “press spacebar to select”. When user enters spacebar always the first option is selected.  (see screenshot below –Are you experiencing any of the following symptoms ?]
Expected
JAWS focus should not come to “response container” during keyboard tab navigation. The selection experience is completely unpredictable.
WCAG Violation
Name, Role Value: 4.1.2
https://www.w3.org/TR/UNDERSTANDING-WCAG20/ensure-compat-rsv.html
Info and Relationships 1.3.1
https://www.w3.org/WAI/WCAG21/Understanding/info-and-relationships.html

These are accessibility issues of v3 of Web Chat, which has been deprecated and we no longer support unless there are security risks. For accessible Web Chat, please migrate to v4.

This is a bug on Adaptive Cards.
The content author is creating those buttons using hero card, which is rendered through Adaptive Cards.
@corinagum can you find the AC bug and associate this with it?

I believe this is because the AC card container is focusable, when it shouldn't be.
In our offline chat, we refer to this phenomena as "rogue tabindex=0". Will file a bug to AC and link it here.


add app.json for heroku

Not actually needed for this branch, but Heroku requires it for Review Apps to work.


Speed up float-to-half conversions

These take a noticeable amount of time when doing interactive rendering with a frame using the 'half' pixel type (which is the default).
This code is probably a great starting point for this task:
https://gist.github.com/rygorous/2156668

For reference: https://github.com/OpenImageIO/oiio/pull/494

For reference: OpenImageIO/oiio#494

From the link:

Closing this as obsolete. SSE is a much better way to go.


npm install is giving error whie installing<EMAIL_ADDRESS>

D:\vue-chat-master\vue-chat-master>npm install
npm WARN Error: EPERM: operation not permitted, unlink 'D:\vue-chat-master\vue-chat-master\node_modules\fsevents\node_modules\aws-sign2\package.json'
npm WARN  { Error: EPERM: operation not permitted, unlink 'D:\vue-chat-master\vue-chat-master\node_modules\fsevents\node_modules\aws-sign2\package.json'
npm WARN   stack: 'Error: EPERM: operation not permitted, unlink 'D:\vue-chat-master\vue-chat-master\node_modules\fsevents\node_modules\aws-sign2\package.json'',
npm WARN   errno: -4048,
npm WARN   code: 'EPERM',
npm WARN   syscall: 'unlink',
npm WARN   path: 'D:\vue-chat-master\vue-chat-master\node_modules\fsevents\node_modules\aws-sign2\package.json' }
npm ERR! path D:\vue-chat-master\vue-chat-master\node_modules\miller-rabin\bin\miller-rabin
npm ERR! code ENOENT
npm ERR! errno -4058
npm ERR! syscall chmod
npm ERR! enoent ENOENT: no such file or directory, chmod 'D:\vue-chat-master\vue-chat-master\node_modules\miller-rabin\bin\miller-rabin'
npm ERR! enoent This is related to npm not being able to find a file.
npm ERR! enoent
npm ERR!      C:\Users\XXXX\AppData\Roaming\npm-cache_logs\2017-10-12T17_41_57_774Z-debug.log

Hi @vibhor4all, cool that you try to install this little project, unfortunately I do not have a windows machine to see what could be causing your issue. Perhaps the package aws-sign2 is not compatible with windows.

@Intellicode is there any work around for that error? I cant get it running.

@vibhor4all Maybe you can have a look at this topic: https://github.com/npm/npm/issues/17671
Unfortunately, I don't have any windows machine here to test your setup.

@Intellicode I tried reverting the the npm to 5.0.4 and still no luck. Anyways, it wont install on my windows machine still giving lots of errors like the ones attached in the files.
I just wanted to see you this app you created in working state. So this is just a request, maybe you can create a video or post screenshots here so that i can see it in action. I have your code for reference. I am learning vue myself and nodejs is new for me too. So any help would be much appreciated.
2017-10-29T12_31_36_100Z-debug.log
vue-chat-cmd-error.txt

@vibhor4all I've created a 'dist' folder in this branch(https://github.com/Intellicode/vue-chat/tree/with-dist), you can try it out by serving the dist folder and connecting to localhost with choosen port (this domain has to be localhost)


Add test command to cli runner

The test command should clean, generate, build and test the resulting image.

i think it doesn't have to clean, build and generate - imagine scenario when you are developing test for the image, than you should be able to run some test only.
We have proposal to our cli to be more like Makefile than maven. So we should be consistent with test target too.

Done via: https://github.com/jboss-container-images/concreate/pull/85

Test invocation was change.
Concreate now doesn't support any kind of clean command (there is no need to implement rm -rf target for now in python)
The other difference that concreate doesnt run build for test execution - we need to support use cases when we want to run test for different existing image.
Only implicit operation for test subcommand is  generate which is used to fetch test features/steps from modules.


Feat: Add Select chain #7823 #6483

Thank you for your contribution to the KodaDot - One Stop Shop for Polkadot NFTs.
👇 __ Let's make a quick check before the contribution.
PR Type

[ ] Bugfix
[x] Feature
[ ] Refactoring

Context

[x] Closes #7823
[x] Closes #6483
[x] ## Did your issue had any of the "$" label on it?
[x] My DOT address: Payout

@roiLeo recreated pr #7905 because of too many commits in previous one

on Firefox
choosing network from the dropdown causes the page to look like this:

on Firefox choosing network from the dropdown causes the page to look like this:

fixed recheck

@vikiival my pr is accordingly  ready

@vikiival @roiLeo waiting for review

@daiagi are we finally able to merge this one

when changing wallet it doesn't update the handle input
can't set/update different identity between chains it's always the same

clear form on change

when changing wallet it doesn't update the handle input
can't set/update different identity between chains it's always the same

you're right and it works in the similar way because this is saves time and effort suppose when a user has selected wrong chain and then he tries to switch and all of his input clears so with this thought in mind I tried to make it user friendly and not a trouble any more

Hello @AshutoshSingh00001
Due to the low code quality, we have decided to close this PR.
It is highly time consuming for anyone from @kodadot/internal-dev to do a review of non-working, low-priority issue.
We would be happy to work with your PR

When the code will be working
The PR is refferencing to a higher prio issue.

Thanks for the understanding. 🤗😇

Hello @AshutoshSingh00001 Due to the low code quality, we have decided to close this PR.
It is highly time consuming for anyone from @kodadot/internal-dev to do a review of non-working, low-priority issue.
We would be happy to work with your PR

When the code will be working
The PR is refferencing to a higher prio issue.

Thanks for the understanding. 🤗😇

thanks @vikiival I was also putting a lot of time in in particular issue and the issues are not ending so I'll try to resubmit this pr later once I'll fully test it


functions can only be declared at top level

The latest version fails on my CI with this error message:
In strict mode code, functions can only be declared at top level or immediately
within another function. [./node_modules/a11y-dialog/a11y-dialog.js:469,0][app.js:7304,4]

Hello. That is unfortunate. 😔 I'm going to need more information to be able to debug it though. Which CI are you using? How do you use the library at all? Do you have a more complete output? In the meantime you can go back to v5.4.*.

I think it should be fixed with 61069a44d5febfe35dde5f867ccde0bd4f3d9adb in https://github.com/edenspiekermann/a11y-dialog/releases/tag/5.5.2. :)

Awesome. Thanks!