Traditional-SPICE-Format Netlister

Hdl21 / vlsir.circuit's initial netlisting targets have been Verilog and Spectre.
Next on the priority-stack is traditional-format SPICE.
The issue: implement it.
This should all be derivable from the existing content of hdl21/netlist/__init__.py. Tasks would include:

Create a SpiceNetlister or similar, derived from the base Netlister
Add it to the enumerated NetlistFormats, and its netlister method
The primary difference (in addition to overall syntax) will be to Primitives, particularly the "ideal" flavor. Traditional SPICE formats identify their types via instance-name prefixes, e.g. "R1" is a resistor, "M1" is a MOSFET, etc. Existing Spectre-format netlisting has a small table of ideal-primitive module-names. From SpectreNetlister.resolve_reference:

        elif ref.WhichOneof("to") == "external":  # Defined outside package

            # First check the priviledged/ internally-defined domains
            if ref.external.domain == "hdl21.primitives":
                msg = f"Invalid direct-netlisting of physical `hdl21.Primitive` `{ref.external.name}`. "
                msg += "Either compile to a target technology, or replace with an `ExternalModule`. "
                raise RuntimeError(msg)
            if ref.external.domain == "hdl21.ideal":
                # Ideal elements
                name = ref.external.name

                # Sort out the spectre-format name
                if name == "IdealCapacitor":
                    module_name = "capacitor"
                elif name == "IdealResistor":
                    module_name = "resistor"
                elif name == "IdealInductor":
                    module_name = "inductor"
                elif name == "VoltageSource":
                    module_name = "vsource"
                else:
                    raise ValueError(f"Unsupported or Invalid Ideal Primitive {ref}")

Something like this will need to occur in instance-generation (write_instance) as well. A reasonable (if old) reference on the prefix-characters is available on Berkeley's own BWRC server.

PR #12 looks like a great start.
I added some more (in branch spice_netlister2 to avoid conflicting) which adds handling for the spice-primitives.
They way this works:

There is an enumeration of the spice primitive types, along with their prefixes: https://github.com/dan-fritchman/Hdl21/blob/cda2efaa8b57a116ffa74201cf25d98cb28f8627/hdl21/netlist/base.py#L24

class SpicePrimitive(Enum):
    """ Enumerated Spice Primitives and their Instance-Name Prefixes """

    RESISTOR = "r"
    CAPACITOR = "c"
    INDUCTOR = "l"
    MOS = "m"
    DIODE = "d"
    VSOURCE = "v"
    ISOURCE = "i"

There is also a ResolvedModule helper-class, which includes a few collected pieces of information on each Module-referent:
@dataclass
class ResolvedModule:
    """ Resolved reference to a `Module` or `ExternalModule`.
    Includes its netlist-sanitized name, and indication of
    whether spice-formatting should treat the device as a primitive. """

    module: ModuleLike
    module_name: str
    spice_primitive: Optional[SpicePrimitive]

The content of these things are sorted out by resolve_reference, around here.
And finally, SpiceNetlister adds a write_instance_name called by write_instance.
Lastly, that commit breaks the netlisters into different module-files, base.py, spice.py, etc. (Sorry if that's a headache.)

Other notes:

While Xyce and ngspice are great test-cases for the content this generates, on our research servers the (commercial) hspice will be easier to use for initial checks (primarily since it's already installed).
It also seems there will be plenty of overlap between this and SpectreNetlister. Probably worth sharing more of it explicitly.
Eventually this may become several more-specific sub-classes, as many Spice simulators have their own "dialect" choices. Grabbing a fun example from a previous parser project:

r1 1 0
+ fun_param='((0.5*(x-2*y))+z)/(2*(a-2*b))'
* A mid-stream line comment
+ funner_param=11e-21

The middle "mid-stream line comment" is in fact interpreted as a comment, at least by some simulators. And the final line with funner_param is interpreted as part of the instance-declaration for r1 on the first line. For this dialect, lines that are either (a) blank, or (b) include only comments, allow devices (and all kinda other multi-line things) to wrap around them.
Now - we should absolutely not write netlists with stuff like this. But we'll at least need to be aware enough to avoid it.

The "dialects" are now broken out in branch spice_netlister2's head-commit. Reading through its manual and firing up some trial runs, Xyce particularly has some healthier differences from what I'd think of as "generic spice".
@vighneshiyer - is there more you'd like to add to #12 before getting it merged/ upstreamed?

There's nothing that I need to add to #12, you can merge it. We can leave this issue open until I write enough tests (on a future PR) to cover most of the new code we added.
I did have some issues running the tests on Python 3.10. We should add a CI target for it. The pytest version needs to bumped and there are a few RecursionErrors that pop up from pydantic.


E2E example bot doesn't work as expected

Rasa Open Source version
Rasa Version      :         2.8.6
Minimum Compatible Version: 2.8.0
Rasa SDK Version  :         2.8.1
Rasa X Version    :         None
Python Version    :         3.8.0
Operating System  :         macOS-10.16-x86_64-i386-64bit
Python Path       :         /Users/johannes/miniconda3/envs/rasa-master/bin/python

What happened?
Rasa's example bot under examples/e2ebot doesn't respond according to the training stories when I talk to it on rasa shell. Specifically, we get TED predicted 'utter_search_restaurants' based on user text. far too often.
Relevant log output
Example of a wrong decision:
Your input ->  show me a place to eat
2021-09-09 11:06:55 DEBUG    rasa.core.lock_store  - Issuing ticket for conversation 'a209c017cf6f4e43b1cafc7fd089e612'.
2021-09-09 11:06:55 DEBUG    rasa.core.lock_store  - Acquiring lock for conversation 'a209c017cf6f4e43b1cafc7fd089e612'.
2021-09-09 11:06:55 DEBUG    rasa.core.lock_store  - Acquired lock for conversation 'a209c017cf6f4e43b1cafc7fd089e612'.
2021-09-09 11:06:55 DEBUG    rasa.core.tracker_store  - Recreating tracker for id 'a209c017cf6f4e43b1cafc7fd089e612'
2021-09-09 11:06:55 DEBUG    rasa.core.processor  - Received user message 'show me a place to eat' with intent '{'id': -9142254842228968108, 'name': 'search_restaurant', 'confidence': 0.9999998807907104}' and entities '[]'
2021-09-09 11:06:55 DEBUG    rasa.core.processor  - Logged UserUtterance - tracker now has 9 events.
2021-09-09 11:06:56 DEBUG    rasa.core.policies.ted_policy  - User intent lead to 'utter_suggest_cuisine'.
2021-09-09 11:06:56 DEBUG    rasa.core.policies.ted_policy  - User text lead to 'utter_search_restaurants'.
2021-09-09 11:06:56 DEBUG    rasa.core.policies.ted_policy  - TED predicted 'utter_search_restaurants' based on user text.
2021-09-09 11:06:56 DEBUG    rasa.core.policies.ensemble  - Made e2e prediction using user text.
2021-09-09 11:06:56 DEBUG    rasa.core.policies.ensemble  - Added `DefinePrevUserUtteredFeaturization(True)` event.
2021-09-09 11:06:56 DEBUG    rasa.core.policies.ensemble  - Predicted next action using policy_0_TEDPolicy.
2021-09-09 11:06:56 DEBUG    rasa.core.processor  - Predicted next action 'utter_search_restaurants' with confidence 1.00.
2021-09-09 11:06:56 DEBUG    rasa.core.processor  - Policy prediction ended with events '[<rasa.shared.core.events.DefinePrevUserUtteredFeaturization object at 0x7fbdabd734c0>]'.
2021-09-09 11:06:56 DEBUG    rasa.core.processor  - Action 'utter_search_restaurants' ended with events '[BotUttered('great! here's what I found ...', {"elements": null, "quick_replies": null, "buttons": null, "attachment": null, "image": null, "custom": null}, {"utter_action": "utter_search_restaurants"},<PHONE_NUMBER>.085574)]'.
2021-09-09 11:06:56 DEBUG    rasa.core.policies.ted_policy  - TED predicted 'action_listen' based on user intent.
2021-09-09 11:06:56 DEBUG    rasa.core.policies.ensemble  - Predicted next action using policy_0_TEDPolicy.
2021-09-09 11:06:56 DEBUG    rasa.core.processor  - Predicted next action 'action_listen' with confidence 0.99.
2021-09-09 11:06:56 DEBUG    rasa.core.processor  - Policy prediction ended with events '[]'.
2021-09-09 11:06:56 DEBUG    rasa.core.processor  - Action 'action_listen' ended with events '[]'.
2021-09-09 11:06:56 DEBUG    rasa.core.lock_store  - Deleted lock for conversation 'a209c017cf6f4e43b1cafc7fd089e612'.
great! here's what I found ...

But it should have run utter_suggest_cuisine, i.e. "how about greek food?" etc.
Failed test stories from rasa test:
version: "2.0"
stories:
- story: Happy path accepts suggestion (./tests/test_stories.yaml)
  steps:
  - intent: greet
  - action: utter_greet
  - intent: search_restaurant
  - action: utter_suggest_cuisine  # predicted: utter_search_restaurants
  - intent: affirm
  - action: utter_search_restaurants
- story: Happy path accepts 3rd suggestion (./tests/test_stories.yaml)
  steps:
  - intent: search_restaurant
  - action: utter_suggest_cuisine
  - intent: deny
  - action: utter_suggest_cuisine  # predicted: utter_search_restaurants
  - intent: deny
  - action: utter_suggest_cuisine
  - intent: affirm
  - action: utter_search_restaurants
- story: Happy path accepts suggestion indirectly (./tests/test_stories.yaml)
  steps:
  - intent: greet
  - action: utter_greet
  - intent: search_restaurant
  - action: utter_suggest_cuisine
  - intent: affirm  # predicted: search_restaurant: I can always go for sushi
  - action: utter_search_restaurants
- story: Happy path rejects suggestion, indirectly (./tests/test_stories.yaml)
  steps:
  - intent: search_restaurant
  - action: utter_suggest_cuisine  # predicted: utter_search_restaurants
  - intent: deny  # predicted: search_restaurant: I had that yesterday
  - action: utter_suggest_cuisine

Also, TED seems to interpret the user input "/search_restaurant" as text.

Ok, it looks like this works fine iff we train with --augmentation 0.

But /restart doesn't work. The bot always repeats its last output after restart and doesn't get any further actions right. This is because /restart is interpreted on text basis by TED and this overwrites the regex interpreter it seems.

Closing this as things are fine with --augmentation 0. Creating a new issue about the /restart and general / interpretation problem.


Default false for all notification channels

Hi, i think we should set false for all notification channels, example when configuring with msteam, I am not aware slack default true so need to check logs to debug issue

Thanks for your PR @bkvtin
Changing the default behavior in such a way is not backward compatible.
As such it may cause issues on the next upgrades, for users that use the default configuration.
We try not to break the existing behavior.


Issue 13 fix guides page

Retire standalone guide pages
Create tabs on guides pages
Adjust view and content (slightly) of each guide
Moved guides content into includes, so the guides page won't be so crowded

Fixes #13
Note: We need to improve the content of the guides. Since the solo crunching guide looks OK, the other two are definitely need some attention. While working on this issue I almost didn't touch the content itself as I can not consider myself as a tech writer (furthermore I ain't no native speaker) so I just removed non-related text from there. The guides on the original .us website for pool and  staking-only aren't detailed enough and a little bit rather confusing, so perhaps some work needs to be done there as well to my opinion.
Hence I suggest to not rely on the content of this PR but rather have additional issues to adjust content.

Why are you putting underscores at the front of guides? The html and markdown have no _ at the start

The only reason is to keep thing consistent in the _includes folder.

@barton2526 was it a question or suggestion to change? could you please be more specific?
as I can see all files within the _includes folder are prefixed with underscore.

I'm not familiar enough with the original rationale to say.  Perhaps @RoboticMind knows

Underscores tell Jeykll to not try to build that file or folder directly.

Every file or directory beginning with the following characters: ., _ , # or ~ in the source directory will not be included in the destination folder

https://jekyllrb.com/docs/structure/
Technically the files in the _include directory don't need to have and _ in front of them because the folder is already ignored. The files were like that before I started working with the gridcoin.us site and I've just kept it consistent with that. I would have no issue changing it to not have underscores in that folder as long as you change all the {%include _filename.extension %} to be {%include filename.extension %}, it would be fine.

@RoboticMind I would do this as a different task, as this is not related to the issue itself. And it would be easier to test. wdyt?

@RoboticMind I think it would be better to have separate issue for that as this is not related to this one. Plus it would be easier to test. wdyt?

any other issues with this PR?

@RoboticMind please have another look at this PR.

And I think it is worth to deploy it despite the fact that some parts are missing. Those issues could be addressed as another PR. I believe this improvements worth to have deployed, it should be better anyway.

I guess the proper way would be to Squash during the merge. I believe github should offer this choice. My attempt seems failed.

That option would result in one commit.  I think Robotic meant to reduce 22 to a more manageable number, but still keep them modular (ie >1)

I understand, so I tried before your message doing this in a regular way (rebase ~10, the mark as squash) but seems like because this branch is already on a remote I can not rewrite the history.
I may doing something wrong so I can use some help here.

You need to squash locally and then force push to gridcat:issue-13-fix-guides-page

@barton2526 thank you, --force did a trick!

Merged


View Data within a CausalFrame similar to a DataFrame

Is there any way to look at all the data at the same time similar to that of a DataFrame. If we try to see to see the value in a single replication we see the following error:
Python 3.7.5 (default, Nov  7 2019, 10:50:52)
[GCC 8.3.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> from justcause.data.sets import load_ihdp
>>> rep = load_ihdp(select_rep=[0])[0]
>>> rep
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/tanmay/model_env/lib/python3.7/site-packages/pandas/core/frame.py", line 687, in __repr__
    show_dimensions=show_dimensions,
  File "/home/tanmay/model_env/lib/python3.7/site-packages/pandas/core/frame.py", line 818, in to_string
    line_width=line_width,
  File "/home/tanmay/model_env/lib/python3.7/site-packages/pandas/io/formats/format.py", line 614, in __init__
    self._chk_truncate()
  File "/home/tanmay/model_env/lib/python3.7/site-packages/pandas/io/formats/format.py", line 700, in _chk_truncate
    frame = concat((frame.iloc[:row_num, :], frame.iloc[-row_num:, :]))
  File "/home/tanmay/model_env/lib/python3.7/site-packages/pandas/core/reshape/concat.py", line 284, in concat
    return op.get_result()
  File "/home/tanmay/model_env/lib/python3.7/site-packages/pandas/core/reshape/concat.py", line 503, in get_result
    return cons._from_axes(new_data, self.new_axes).__finalize__(
AttributeError: 'functools.partial' object has no attribute '_from_axes'

On creating my own CausalFrame

It shows all the data in the CausaFrame.  What is the expected behavior? Can the expected behavior be similar to that of a DataFrame?

Hi @Tanmay-Kulkarni101, a CausalFrame is just a DataFrame with the described additional functionality, so you can expect in that regard the same behavior.
Also for me it perfectly works:

I would assume it has something to do with your Pandas or maybe pyarrow version? I am using Pandas 0.25.2 and pyarrow 0.14.1 right now. As you can see from your stacktrace, the error happens deeply within the pandas code, so I really don't think it's related to justcause. Rather from the fact that the actual data came from a parquet file maybe and something was read in.
So you cannot work with any of the data sets we provide? Does this error happen for all of them?

I created two fresh environments to reproduce the errors. I tried to run it on both Python 3.6 and Python 3.7. However, I got the same result for IHDP.
For Python 3.6
(test_36) tanmay@T-TAKULK-AIO:~$ python3
Python 3.6.9 (default, Nov  7 2019, 10:44:02)
[GCC 8.3.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> from justcause.data.sets import load_ihdp
>>> rep = load_ihdp(select_rep=[0])[0]
>>> rep
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/tanmay/test_36/lib/python3.6/site-packages/pandas/core/frame.py", line 687, in __repr__
    show_dimensions=show_dimensions,
  File "/home/tanmay/test_36/lib/python3.6/site-packages/pandas/core/frame.py", line 818, in to_string
    line_width=line_width,
  File "/home/tanmay/test_36/lib/python3.6/site-packages/pandas/io/formats/format.py", line 614, in __init__
    self._chk_truncate()
  File "/home/tanmay/test_36/lib/python3.6/site-packages/pandas/io/formats/format.py", line 700, in _chk_truncate
    frame = concat((frame.iloc[:row_num, :], frame.iloc[-row_num:, :]))
  File "/home/tanmay/test_36/lib/python3.6/site-packages/pandas/core/reshape/concat.py", line 284, in concat
    return op.get_result()
  File "/home/tanmay/test_36/lib/python3.6/site-packages/pandas/core/reshape/concat.py", line 503, in get_result
    return cons._from_axes(new_data, self.new_axes).__finalize__(
AttributeError: 'functools.partial' object has no attribute '_from_axes'

For Python 3.7
(test_37) tanmay@T-TAKULK-AIO:~$ python3
Python 3.7.5 (default, Nov  7 2019, 10:50:52)
[GCC 8.3.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> from justcause.data.sets import load_ihdp
>>> rep = load_ihdp(select_rep=[0])[0]
>>> rep
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/tanmay/test_37/lib/python3.7/site-packages/pandas/core/frame.py", line 687, in __repr__
    show_dimensions=show_dimensions,
  File "/home/tanmay/test_37/lib/python3.7/site-packages/pandas/core/frame.py", line 818, in to_string
    line_width=line_width,
  File "/home/tanmay/test_37/lib/python3.7/site-packages/pandas/io/formats/format.py", line 614, in __init__
    self._chk_truncate()
  File "/home/tanmay/test_37/lib/python3.7/site-packages/pandas/io/formats/format.py", line 700, in _chk_truncate
    frame = concat((frame.iloc[:row_num, :], frame.iloc[-row_num:, :]))
  File "/home/tanmay/test_37/lib/python3.7/site-packages/pandas/core/reshape/concat.py", line 284, in concat
    return op.get_result()
  File "/home/tanmay/test_37/lib/python3.7/site-packages/pandas/core/reshape/concat.py", line 503, in get_result
    return cons._from_axes(new_data, self.new_axes).__finalize__(
AttributeError: 'functools.partial' object has no attribute '_from_axes'

Packages installed in Python 3.6
(test_36) tanmay@T-TAKULK-AIO:~$ pip list --format=columns
Package              Version
-------------------- ------------
absl-py              0.9.0
astor                0.8.1
attrs                19.3.0
causalml             0.7.0
certifi              2019.11.28
cffi                 1.14.0
chardet              3.0.4
cycler               0.10.0
Cython               0.29.15
dill                 <IP_ADDRESS>
future               0.18.2
gast                 0.3.3
google-pasta         0.1.8
grpcio               1.27.2
h5py                 2.10.0
idna                 2.9
importlib-metadata   1.5.0
Jinja2               2.11.1
joblib               0.14.1
JustCause            0.3.2
Keras                2.3.1
Keras-Applications   1.0.8
Keras-Preprocessing  1.1.0
kiwisolver           1.1.0
lightgbm             2.3.1
Markdown             3.2.1
MarkupSafe           1.1.1
matplotlib           3.2.0
more-itertools       8.2.0
numpy                1.18.1
packaging            20.3
pandas               1.0.1
patsy                0.5.1
pip                  20.0.2
pkg-resources        0.0.0
pluggy               0.13.1
progressbar2         3.50.0
protobuf             3.11.3
py                   1.8.1
pyarrow              0.16.0
pycparser            2.20
pydotplus            2.0.2
pygam                0.8.0
pyparsing            2.4.6
pytest               5.3.5
python-dateutil      2.8.1
python-utils         2.4.0
pytz                 2019.3
PyYAML               5.3
requests             2.23.0
rpy2                 3.2.6
scikit-learn         0.22.2.post1
scipy                1.4.1
seaborn              0.10.0
setuptools           46.0.0
shap                 0.35.0
simplegeneric        0.8.1
six                  1.14.0
statsmodels          0.11.1
tensorboard          1.14.0
tensorflow           1.14.0
tensorflow-estimator 1.14.0
termcolor            1.1.0
tqdm                 4.43.0
tzlocal              2.0.0
urllib3              1.25.8
wcwidth              0.1.8
Werkzeug             1.0.0
wheel                0.34.2
wrapt                1.12.1
xgboost              1.0.2
zipp                 3.1.0

Packages installed in Python 3.7
(test_37) tanmay@T-TAKULK-AIO:~$ pip list --format=columns
Package              Version
-------------------- ------------
absl-py              0.9.0
astor                0.8.1
attrs                19.3.0
causalml             0.7.0
certifi              2019.11.28
cffi                 1.14.0
chardet              3.0.4
cycler               0.10.0
Cython               0.29.15
dill                 <IP_ADDRESS>
future               0.18.2
gast                 0.3.3
google-pasta         0.1.8
grpcio               1.27.2
h5py                 2.10.0
idna                 2.9
importlib-metadata   1.5.0
Jinja2               2.11.1
joblib               0.14.1
JustCause            0.3.2
Keras                2.3.1
Keras-Applications   1.0.8
Keras-Preprocessing  1.1.0
kiwisolver           1.1.0
lightgbm             2.3.1
Markdown             3.2.1
MarkupSafe           1.1.1
matplotlib           3.2.0
more-itertools       8.2.0
numpy                1.18.1
packaging            20.3
pandas               1.0.1
patsy                0.5.1
pip                  20.0.2
pkg-resources        0.0.0
pluggy               0.13.1
progressbar2         3.50.0
protobuf             3.11.3
py                   1.8.1
pyarrow              0.16.0
pycparser            2.20
pydotplus            2.0.2
pygam                0.8.0
pyparsing            2.4.6
pytest               5.3.5
python-dateutil      2.8.1
python-utils         2.4.0
pytz                 2019.3
PyYAML               5.3
requests             2.23.0
rpy2                 3.2.6
scikit-learn         0.22.2.post1
scipy                1.4.1
seaborn              0.10.0
setuptools           46.0.0
shap                 0.35.0
simplegeneric        0.8.1
six                  1.14.0
statsmodels          0.11.1
tensorboard          1.14.0
tensorflow           1.14.0
tensorflow-estimator 1.14.0
termcolor            1.1.0
tqdm                 4.43.0
tzlocal              2.0.0
urllib3              1.25.8
wcwidth              0.1.8
Werkzeug             1.0.0
wheel                0.34.2
wrapt                1.12.1
xgboost              1.0.2
zipp                 3.1.0

Thanks for helping me out.

I also tried the same with the Twins dataset. I got the same error.
For Python 3.6
(test_36) tanmay@T-TAKULK-AIO:~$ python3
Python 3.6.9 (default, Nov  7 2019, 10:44:02)
[GCC 8.3.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> from justcause.data.sets import load_twins
>>> data = load_twins()
>>> data
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/tanmay/test_36/lib/python3.6/site-packages/pandas/core/frame.py", line 687, in __repr__
    show_dimensions=show_dimensions,
  File "/home/tanmay/test_36/lib/python3.6/site-packages/pandas/core/frame.py", line 818, in to_string
    line_width=line_width,
  File "/home/tanmay/test_36/lib/python3.6/site-packages/pandas/io/formats/format.py", line 614, in __init__
    self._chk_truncate()
  File "/home/tanmay/test_36/lib/python3.6/site-packages/pandas/io/formats/format.py", line 700, in _chk_truncate
    frame = concat((frame.iloc[:row_num, :], frame.iloc[-row_num:, :]))
  File "/home/tanmay/test_36/lib/python3.6/site-packages/pandas/core/reshape/concat.py", line 284, in concat
    return op.get_result()
  File "/home/tanmay/test_36/lib/python3.6/site-packages/pandas/core/reshape/concat.py", line 503, in get_result
    return cons._from_axes(new_data, self.new_axes).__finalize__(
AttributeError: 'functools.partial' object has no attribute '_from_axes'

For Python 3.7
(test_37) tanmay@T-TAKULK-AIO:~$ python3
Python 3.7.5 (default, Nov  7 2019, 10:50:52)
[GCC 8.3.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> from justcause.data.sets import load_twins
>>> data = load_twins()
>>> data
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/tanmay/test_37/lib/python3.7/site-packages/pandas/core/frame.py", line 687, in __repr__
    show_dimensions=show_dimensions,
  File "/home/tanmay/test_37/lib/python3.7/site-packages/pandas/core/frame.py", line 818, in to_string
    line_width=line_width,
  File "/home/tanmay/test_37/lib/python3.7/site-packages/pandas/io/formats/format.py", line 614, in __init__
    self._chk_truncate()
  File "/home/tanmay/test_37/lib/python3.7/site-packages/pandas/io/formats/format.py", line 700, in _chk_truncate
    frame = concat((frame.iloc[:row_num, :], frame.iloc[-row_num:, :]))
  File "/home/tanmay/test_37/lib/python3.7/site-packages/pandas/core/reshape/concat.py", line 284, in concat
    return op.get_result()
  File "/home/tanmay/test_37/lib/python3.7/site-packages/pandas/core/reshape/concat.py", line 503, in get_result
    return cons._from_axes(new_data, self.new_axes).__finalize__(
AttributeError: 'functools.partial' object has no attribute '_from_axes'

The issue seems to be the same in both the IHDP and twins.

Could you try the current version on master? I removed some functionality.
causalml is a just to fast-moving target to keep supporting it as well as everything that depends on Tensorflow. My idea would be to focus JustCause more on the evaluation and data provider part. This will also make the installation a lot easier. What do you think @MaximilianFranz?

Bold move, but makes total sense!

I build your package from source, it seems to be working now. I still am not sure what went wrong. Nice work guys.


Multiple score features

Could this be adapted to use multiple score features instead of a single score?

You can use multiple scores during the training to get a weighted score label.
For instance, two give an essay 4 and one give it 3 and then score label is 2/3 for 4 and 1/3 for 3.
But I didn't try it to train the model and don't know if it helps.

Thanks. What I mean is similar to multi label classification. So in this case it would be multiple score points per response for a variety of criteria. For example, 1-5 for expression and 1-8 for quality of ideas.


Latlng graticule

As discussed,
(You may want to change default initialization values.)

Hmm - If I check that out, so far all I get is
leaflet.latlng-graticule.js:10 Uncaught TypeError: Cannot read property 'extend' of undefined
    at leaflet.latlng-graticule.js:10
    at leaflet.latlng-graticule.js:530
(anonymous) @ leaflet.latlng-graticule.js:10
(anonymous) @ leaflet.latlng-graticule.js:530
(index):200 CONNECT TO /worldmap/socket
(index):260 NOT in an iframe
(index):271 Uncaught TypeError: L.latlngGraticule is not a function
    at (index):271

Are you sure you tested this ?

Hi,
Apologies, apparently ...not (I was attached to the wrong environment...testing different branch!).
please disregard this PR!

OK - will close it for now


Pre-trained models fail to infer with batch>1

Several Reshape nodes have hard-coded batch dimension value in the 2nd input. Could you please fix the models to preserve batch?
Reshape nodes with hard-coded output batch:

bert/embeddings/Reshape_2
bert/embeddings/Reshape
bert/embeddings/Reshape_3
bert/encoder/layer_0/attention/self/Reshape_1
bert/encoder/layer_0/attention/self/Reshape
bert/encoder/layer_0/attention/self/Reshape_2
bert/encoder/Reshape
bert/encoder/layer_1/attention/self/Reshape
bert/encoder/layer_1/attention/self/Reshape_2
bert/encoder/layer_1/attention/self/Reshape_1
bert/encoder/layer_2/attention/self/Reshape_1
bert/encoder/layer_2/attention/self/Reshape_2
bert/encoder/layer_2/attention/self/Reshape
bert/encoder/layer_3/attention/self/Reshape_2
bert/encoder/layer_3/attention/self/Reshape_1
bert/encoder/layer_3/attention/self/Reshape
bert/encoder/layer_4/attention/self/Reshape_2
bert/encoder/layer_4/attention/self/Reshape
bert/encoder/layer_4/attention/self/Reshape_1
bert/encoder/layer_5/attention/self/Reshape
bert/encoder/layer_5/attention/self/Reshape_1
bert/encoder/layer_5/attention/self/Reshape_2
bert/encoder/layer_6/attention/self/Reshape_1
bert/encoder/layer_6/attention/self/Reshape
bert/encoder/layer_6/attention/self/Reshape_2
bert/encoder/layer_7/attention/self/Reshape_1
bert/encoder/layer_7/attention/self/Reshape_2
bert/encoder/layer_7/attention/self/Reshape
bert/encoder/layer_8/attention/self/Reshape
bert/encoder/layer_8/attention/self/Reshape_1
bert/encoder/layer_8/attention/self/Reshape_2
bert/encoder/layer_9/attention/self/Reshape
bert/encoder/layer_9/attention/self/Reshape_1
bert/encoder/layer_9/attention/self/Reshape_2
bert/encoder/layer_10/attention/self/Reshape
bert/encoder/layer_10/attention/self/Reshape_1
bert/encoder/layer_10/attention/self/Reshape_2
bert/encoder/layer_11/attention/self/Reshape
bert/encoder/layer_11/attention/self/Reshape_2
bert/encoder/layer_11/attention/self/Reshape_1
bert/encoder/Reshape_13

The problem is still relevant for me.
I would like to use pre-strained models using my custom scripts without a need to take this whole repository with me.
To do this I had to manually load currently available models and patch modeling.py function get_shape_list to keep Shape calculating sub-graphs and to never hard-code Reshape second inputs and to let shape inference flow from Placeholders without constraints. I deleted these lines
if not non_static_indexes:
    return shape

Could you guys be so kind to upload pre-trained models in such form? With Shape sub-graphs.


Errors related to monitors

🔍 Before submitting the issue

[X] I have searched among the existing issues
[X] I am using a Python virtual environment

🐞 Description of the bug
A customer reported following errors while using pyfluent from Fluent nodes in optiSLang:

@seanpearsonuk @ajain-work
📝 Steps to reproduce
Unfortunately I donot know which command leads to this, since it was reproduced on the customer machine. If more information is required, Please let me know.
💻 Which operating system are you using?
Windows
📀 Which ANSYS version are you using?
2023R2
🐍 Which Python version are you using?
3.10
📦 Installed packages
ansys-fluent-core

Thank you for submitting this, @tejalprabhu01.
Yes, for the customer's sake, if they can provide the ful details, it will help in finding a solution.

@seanpearsonuk @tejalprabhu01
I have investigated it now.
The primary guess is that the user has either not read data file or not called iterate() before executing methods of solver_session.monitors_manager.
e.g
solver_session.monitors_manager.get_monitor_set_names()
solver_session.monitors_manager.get_monitor_set_data(monitor_set_name="residual")

Normal solver session
>>> import ansys.fluent.core as pyfluent
>>> from ansys.fluent.core import examples
>>> import_case = examples.download_file(file_name="exhaust_system.cas.h5", directory="pyfluent/exhaust_system")
Checking if specified file already exists...
File already exists. File path:
C:\Users\hpohekar\AppData\Local\Ansys\ansys_fluent_core\examples\exhaust_system.cas.h5
>>> import_data = examples.download_file(file_name="exhaust_system.dat.h5", directory="pyfluent/exhaust_system")
Checking if specified file already exists...
File already exists. File path:
C:\Users\hpohekar\AppData\Local\Ansys\ansys_fluent_core\examples\exhaust_system.dat.h5
>>> solver_session = pyfluent.launch_fluent(precision="double", processor_count=2, start_transcript=False, mode="solver", cleanup_on_exit=False)

Monitors list empty now.
>>> monitors_list = solver_session.monitors_manager.get_monitor_set_names()
>>> monitors_list
[]

Monitors list is still empty after reading case file
>>> solver_session.tui.file.read_case(import_case)
pyfluent.tui WARNING: Currently calling the TUI commands in a generic manner. Please run `python codegen/allapigen.py` from the top-level pyfluent directory to generate the local TUI commands classes.
The following solver settings object method could also be used to execute the above command:
<solver_session>.file.read_case(file_name = r"C:\Users\hpohekar\AppData\Local\Ansys\ansys_fluent_core\examples\exhaust_system.cas.h5")
>>> monitors_list = solver_session.monitors_manager.get_monitor_set_names()
>>> monitors_list
[]

We get monitors list only after reading a case file or after calling iterate() method.
>>> solver_session.tui.file.read_data(import_data)
The following solver settings object method could also be used to execute the above command:
<solver_session>.file.read_data(file_name = r"C:\Users\hpohekar\AppData\Local\Ansys\ansys_fluent_core\examples\exhaust_system.dat.h5")
>>> monitors_list = solver_session.monitors_manager.get_monitor_set_names()
>>> monitors_list
['residual', 'mass-bal-rplot', 'mass-tot-rplot', 'mass-in-rplot', 'point-vel-rplot']

calling iterate() method.
>>> solver_session.tui.solve.set.number_of_iterations(15)
The following solver settings object method could also be used to execute the above command:
<solver_session>.solution.run_calculation.iter_count = 15
>>> solver_session.tui.solve.iterate()
The following solver settings object method could also be used to execute the above command:
<solver_session>.solution.run_calculation.iterate(iter_count = 15)
>>> solver_session.monitors_manager.get_monitor_set_names()
['residual', 'mass-bal-rplot', 'mass-tot-rplot', 'mass-in-rplot', 'point-vel-rplot']

I also verified following source code behavior (shared by user in 1st screenshot). It's working fine.

>>> for monitor_set_name, monitor_set_info in solver_session.monitors_manager._monitors_info.items():
...     print(monitor_set_info["monitors"])
...
['continuity', 'x-velocity', 'y-velocity', 'z-velocity', 'energy', 'k', 'omega']
['mass-bal']
['mass-tot']
['mass-in']
['point-vel']
>>>

@seanpearsonuk @tejalprabhu01
@ajain-work and I tried to reproduce this bug and we are able to reproduce this.
We will create a PR soon.
Thank you.


Usage example doesn't work

I'm a first-time Mava user. I'm trying to figure out how to use it. In the docs homepage there's a "Usage" section with a code snippet. I'm trying to run this code snippet, but it fails because it's trying to import a local module called helpers, which isn't provided.

I've managed to run some example, located at https://github.com/instadeepai/Mava/tree/develop/examples/tf/debugging/simple_spread/feedforward/centralised.

@jdtotow : Thank you, can you share which of the 3 files worked for you?

I tested only run_mddpg.py and works successfully. However i'm having difficulty to retrieve the "executor" for interacting with the environment


Add missing unit test for Int.reached.

Fixed a bug previously in #170 but had no unit tests. While this test is not optimal it's better than nothing. Also the logic of this reached function always confuses me.

Nice test :) yeah it confuses me too ... the extractRuleToRulesetIdMap-function is even worse :D


Unit tests throw timeout error after upgrading to 0.10.0

Note: GitHub issues should be reserved only for bug reports. Please use the
Kong mailing list or
Gitter for user support, development
questions, feature requests, etc.
Summary
This is a combination of 2 issues.
After upgrading from kong version 0.9.9 to 0.10.0, the lua unit tests of my custom plugin started throwing timeout error. They were executing fine before the version upgrade. Apparently, the default timeout is set to 10s, so on doing this
before_each(function()     client = helpers.proxy_client(15000)   end)
the tests were executed successfully. Could this be specific to my local environment?
All the existing unit tests also throw timeout error.
Recently, I had to modify my source code to use resty.http module for http requests. After this change, again the unit tests started throwing timeout error even after setting it > 10s.
Additional Details & Logs

Kong version ($ kong version) : 0.10.0
Kong debug-level startup logs ($ kong start --vv)
Kong error logs (<KONG_PREFIX>/logs/error.log)
Kong configuration (registered APIs/Plugins & configuration file)
Operating System : Kong is running on Vagrant. Host OS is Windows 7.

This may be a duplicate of #2352.
Does setting KONG_NGINX_DAEMON=on make your tests work?

@shiprabehera any update based on the feedback above?

@patricksanders @p0pr0ck5
This does not work.  I am still getting timeouts. Sorry about the late update.

It's a bit hard to say what's happening with looking at the test files, but I don't think a GitHub issue is the appropriate place, as this isn't related to kong itself, but your custom plugin tests (from what I read). There have been other reports of odd/unexpected behavior with Vagrant + Windows setups, so I wouldn't be surprised if such an environment was at least partially responsible. It may also be that your use of resty.http is possibly conflicting with the busted helpers, but again it's hard to say for sure without debugging. One other suggestion I might have would be to use Travis CI to run the tests as well, to exclude your environment as a cause for timeouts.

Closing after no response. Feel free to re-open here if this seems like an issue with Kong, or continue the conversation on the mailing list. Thanks!

Leaving this here, as it's possibly the same issue and may save someone else a few hours...
I brought up the kong-vagrant box and tried running the /kong-plugin/spec tests.  These failed with a timeout.  Tried again increasing the timeout to 60 secs and got an error message:
failed to receive reply from UDP server <IP_ADDRESS>:53: timeout
Turns out Google Public DNS is blocked by the firewall in my organisation's network.  The workaround was to comment out a line in /kong/spec/kong_tests.conf:
# dns_resolver = <IP_ADDRESS>


pull request for e2e test at 01/21/2017 13:26:15

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Colour Scheme

I downloaded it and it installed properly but my default colour scheme is very different from the example. Fields are solid red and namespaces are solid green as an example.  I'm okay with it not installing properly, I have other plugins and other colour customization so I'm less worried about it installing with the right colours but it would be good if there was some schemes that you can download and apply. Like a dark and light theme for example.

Hey @nykwil, good idea. Sorry that the default colors are a bit awful. I don't know how to place defaults for different color schemes.
I thought everybody who bothers to install such a plugin is willing to configure it properly. And just needs some verification that the plugin indeed works as promoted.
I guess I was wrong.
It would be great if we had some valid schemes. I only have a dark scheme and no idea how to extract it into something redistributable.
Can someone provide a good light theme?
And a hint how these are distributed best?

Tools -> Import and Export Settings... starts a import/export wizard for creating partial settings files. There's even online databases for vssettings files (like https://studiostyl.es/). Just need people to submit schemes. I'm really bad at selecting colours, just knowing colours that don't work.

@nykwil thank you for your useful hints. I have exported my settings:
arBmind_InputMono_Viasfora_SemanticColorizer_Dark.zip
studiostyl.es only allows to create styles using their website, which does not support styles for plugins. So we cannot upload it there.
@all - Does these kind of exported style work for you?


Get server consumption since boot up

Problem
Hi,
I am working with scaphandre and prometheus and was wondering if there was a way to get a server power consumption since boot up with a query like  scaph_process_power_consumption_microwatts that would give me details about processes consumption.
If I'm correct, microwatts queries only return the server's power consumption at the instant of the query.
Thanks for the help !

Update
I found a metric that fits my needs, scaph_host_energy_microjoules gives a counter that represents a server's consumption in microjoules.


Added missing accelerator keys to the buttons on the 'Find Uses' form

But it seems that my IDE changed the size of the controls as well, maybe it has something to do with my monitor which is 1080P with a 135% system DPI...

Thank you edwinyzh


Remove double meaning of editor.worker.js

editor.worker.js has 2 meanings. It needs to be used in 2 different contexts, which is confusing.
One context is that it must be imported as a web worker. I.e. user worker-loader. For example as in this monaco-yaml example
The other context is that it must be imported as a regular module by monaco plugins. This can be seen for example in monaco-yaml and monaco-json.
I suggest to split this file into two files.
workerInitialize.js contains the following contents:
import { SimpleWorkerServer } from '../base/common/worker/simpleWorker.js';
import { EditorSimpleWorker } from './common/services/editorSimpleWorker.js';
let initialized = false;
export function initialize(foreignModule) {
    if (initialized) {
        return;
    }
    initialized = true;
    const simpleWorker = new SimpleWorkerServer((msg) => {
        self.postMessage(msg);
    }, (host) => new EditorSimpleWorker(host, foreignModule));
    self.onmessage = (e) => {
        simpleWorker.onmessage(e.data);
    };
}

editor.worker.ts contains the following contents:
import { initialize } from './workerInitialize';

self.onmessage = (e) => {
    initialize(null);
};

import { initialize } from 'monaco-editor-core/esm/vs/editor/workerInitialize';
import { JSONWorker } from './jsonWorker';

self.onmessage = () => {
    worker.initialize((ctx, createData) => {
        return new JSONWorker(ctx, createData);
    });
};

This way users could just specify that *.worker.js needs to be imported as a web worker using a pattern in their webpack configuration or other build tool instead of depending on the webpack loader syntax, but plugins can still use its shared logic.
Preferably workerInitialize.js should also have type definitions.

I recently updated to Webpack 5. Webpack 5 supports web workers natively, making the use of worker-loader obsolete, so I don’t think this is an issue anymore.


Showing target choices when there is only one target

If there is only one implementation for a target then we shouldn't ask.

:+1:  was holding back since more are coming ... though for ocaml doubt there will be (didn't even anticipate we'll have that one on launch)

Couldn't we add the logic of "If there's more than one, show up" instead of doing it on a per target basis?

yeah, that's what i'll be doing ...

There won't be another one for OCaml, you can just remove the All checkbox, but I think it's important to keep the "Cohttp" somewhere because it's important information.

I think a design update is needed, I wanna be able to show the "library" info somewhere, link to source ... @jitachi

Should still remove it for now because it's really weird. That OCaml uses a library needs to be apparent inside the code or embedded widget since people will be seeing it embedded in another website. A code comment at the top would be sufficient, just like we do with Unirest on mashape.

@montanaflynn yeah, I think we're saying the same thing, I mean for inside the widget, not the site, be nice to give credit to the library source & even the contributing author for each target converter
something along the lines of:
Unirest support by @therebelrobot

something along those lines:

maybe for libraries the text would be: COhttp support by ... since the language name is already obvious in the tab

I was mostly replying to @SGrondin to make sure the library is mentioned inside the embedded widget.
So in the top of his code he would put something like:
// Requires https://github.com/mirage/ocaml-cohttp
and we would do the same for Unirest and other non-native targets.

ah, code comments, yes those are good.

I'll add a comment.


Is there any progress in biblatex-japanese?

Hello, I am really looking forward to this package. Since you haven't updated these files for almost seven months, so please allow me to consult the progress of this project.
Good luck.

I'm sorry for the inconvenience, but I'm now very busy for my daily works. If you can contribute to this package, it is very welcome.
I began this project to create a framework for biblatex to support Japanese language. But you might be interested in CJK languages support, not only Japanese language. Since I don't have any knowledge on Chinese or Korean bibliographies, if you have any idea for CJK languages support, it is also very welcome.
Thank you.


Prevent the taker from acting on outdated state

Add a fingerprint to every projection that is sent out.
When the taker UI makes a request to the daemon we include the fingerprint the UI knows and compare it in the daemon against the fingerprint of the last projection sent out.
That way we can detect if we are acting on outdated state and can fail the request with a warning to the user.

Nice summary, don't really have anything to add so will unassign myself.

This project is unmaintained


[ Misc ] Improve Min Capability Checking in compressed-tensors

SUMMARY:

add min_capability to CompressedTensorsScheme
actually check this and raise an error if the scheme does not support the device

BEFORE SUBMITTING, PLEASE READ THE CHECKLIST BELOW AND FILL IN THE DESCRIPTION ABOVE

 PR Checklist (Click to Expand)
Thank you for your contribution to vLLM! Before submitting the pull request, please ensure the PR meets the following criteria. This helps vLLM maintain the code quality and improve the efficiency of the review process.
PR Title and Classification
Only specific types of PRs will be reviewed. The PR title is prefixed appropriately to indicate the type of change. Please use one of the following:

[Bugfix] for bug fixes.
[CI/Build] for build or continuous integration improvements.
[Doc] for documentation fixes and improvements.
[Model] for adding a new model or improving an existing model. Model name should appear in the title.
[Frontend] For changes on the vLLM frontend (e.g., OpenAI API server, LLM class, etc.)
[Kernel] for changes affecting CUDA kernels or other compute kernels.
[Core] for changes in the core vLLM logic (e.g., LLMEngine, AsyncLLMEngine, Scheduler, etc.)
[Hardware][Vendor] for hardware-specific changes. Vendor name should appear in the prefix (e.g., [Hardware][AMD]).
[Misc] for PRs that do not fit the above categories. Please use this sparingly.

Note: If the PR spans more than one category, please include all relevant prefixes.
Code Quality
The PR need to meet the following code quality standards:

We adhere to Google Python style guide and Google C++ style guide.
Pass all linter checks. Please use format.sh to format your code.
The code need to be well-documented to ensure future contributors can easily understand the code.
Include sufficient tests to ensure the project to stay correct and robust. This includes both unit tests and integration tests.
Please add documentation to docs/source/ if the PR modifies the user-facing behaviors of vLLM. It helps vLLM user understand and utilize the new features or changes.

Notes for Large Changes
Please keep the changes as concise as possible. For major architectural changes (>500 LOC excluding kernel/data/config/test), we would expect a GitHub issue (RFC) discussing the technical design and justification. Otherwise, we will tag it with rfc-required and might not go through the PR.
What to Expect for the Reviews
The goal of the vLLM team is to be a transparent reviewing machine. We would like to make the review process transparent and efficient and make sure no contributor feel confused or frustrated. However, the vLLM team is small, so we need to prioritize some PRs over others. Here is what you can expect from the review process:

 After the PR is submitted, the PR will be assigned to a reviewer. Every reviewer will pick up the PRs based on their expertise and availability.
 After the PR is assigned, the reviewer will provide status update every 2-3 days. If the PR is not reviewed within 7 days, please feel free to ping the reviewer or the vLLM team.
 After the review, the reviewer will put an  action-required label on the PR if there are changes required. The contributor should address the comments and ping the reviewer to re-review the PR.
 Please respond to all comments within a reasonable time frame. If a comment isn't clear or you disagree with a suggestion, feel free to ask for clarification or discuss the suggestion.

Thank You
 Finally, thank you for taking the time to read these guidelines and for your interest in contributing to vLLM. Your contributions make vLLM a great tool for everyone!

/ready

Can we do anything about the root get_min_capability() you get for "compressed_tensors"?
https://github.com/vllm-project/vllm/blob/36425d27a6a34c88ae9c7706e0f4555bbf8debfa/vllm/model_executor/layers/quantization/compressed_tensors/compressed_tensors.py#L37-L39
I'm thinking about how we use the quant_method name for this testing utility. https://github.com/vllm-project/vllm/blob/main/tests/quantization/utils.py#L7

unfortunately not, though this should be bumped to 70


404 issue trying to install NextLS on Archlinux

> Target is being overridden with BURRITO_TARGET linux_amd64
> Burrito is building target: linux_amd64
> Burrito will build for target:
        OS: linux
        CPU: x86_64
        Qualifiers: [libc: :gnu]
        Debug: false
----> PHASE: :fetch
--> Working directory: /tmp/burrito_build_7BBBA6A0B3A5BB1C
--> Resolving ERTS: {:precompiled, [version: "26.2.1"]}
--> Remote ERTS From Beam Machine: https://burrito-otp.b-cdn.net/OTP-26.2.1/linux/x86_64/gnu/otp_26.2.1_linux_gnu_x86_64_ssl_1.1.1s.tar.gz?please-respect-my-bandwidth-costs=thank-you
--> Downloading file: https://burrito-otp.b-cdn.net/OTP-26.2.1/linux/x86_64/gnu/otp_26.2.1_linux_gnu_x86_64_ssl_1.1.1s.tar.gz?please-respect-my-bandwidth-costs=thank-you
** (RuntimeError) Failed to fetch https://burrito-otp.b-cdn.net/OTP-26.2.1/linux/x86_64/gnu/otp_26.2.1_linux_gnu_x86_64_ssl_1.1.1s.tar.gz?please-respect-my-bandwidth-costs=thank-you! (Got 404) Perhaps we haven't built a pre-compiled Erlang for this release yet? If this was a 404, please file an issue! Thanks!

See also https://github.com/elixir-tools/next-ls/issues/359

This is trying to pull from a very very old version of Burrito's CDN-- it should be using the new "universal" CDN which was added in burrito 1.0.x on hex.pm -- that mix file needs to be updated

I think this was resolved, as the other linked issue was resolved by next-ls 😄


ci: configure codecov

Codecov Report

Merging #32 (79c98f2) into main (72c719d) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #32   +/-   ##
=======================================
  Coverage   91.57%   91.57%
=======================================
  Files          20       20
  Lines         273      273
  Branches       38       38
=======================================
  Hits          250      250
  Misses         23       23

Impacted Files
Coverage Δ

packages/library/test/fixtures/index.ts
100.00% <ø> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


env_config: reduce api_auto_update_secs.

We've reducing this in https://github.com/Homebrew/formulae.brew.sh/pull/752

Review period skipped due to critical label.


Add campaign slug to the signup link in the login page

We're interested on knowing how many signups come from the login page: adding a signup slug will allow us to track those.

Thanks!


Heading in overview

We updated Hugo, which fixed one issue, but I noticed that headings have started to show up on https://maplibre.org/news/

Sounds good and thank you!
What do you mean with the images don't work? For me the images load fine?

I mean the news entries with images

Oh I see, yes. The preview should just be the first paragraph I guess.


fix cannot read 'instance' of null when closing tooltip on weak devices

No related issue
Fix error "TypeError: Cannot read property 'instance' of null", that appears when two or more close events occur on tooltip simultaniously and device handles them with significant delay.
Changelog
Changed

fix error with tooltip close events

Interesting. It looks like the whole point of the existing if is to fix this problem. Could we just ditch the outer if then and only check it in setTimeout?

Done


[docs] Don't suggest putting a Switch inside a ListItemSecondaryAction

[x] I have followed (at least) the PR section of the contributing guide.

Avoid future cases like #23013

Looks good, I think that we could also explain for this test case the tradeoff around ListItemSecondaryAction.

@sirlantis It's a great first pull request on Material-UI 👌🏻. Thank you for working on it!

@oliviertassinari Thanks, but to be honest, you did the majority of it. So: double thanks. 👍


Ds namespace refactoring

I did not test the migrations. Do I do this by running an UPDATE install script?
This is a simple refactoring of namespaces.

Namespaces match file paths
Namespaces are now file scoped, except for autogenerated migration files, which retain their namespace structure.

@dstarr migrations also run on the deployment script (essentially, instead of our hardcoded sql script for deploying database, we ask Entity Framework to generate the script based on the migrations scripts that are also auto-generated)
The update script does almost the same thing as the deploy script except infrastructure (i.e. deploys code and database changes only).
So, if the deployment script works with the new namespace changes, it means everything including migrations work correctly.
Looking at the changes, I see no reason for it to fail.

There are 4 files under the "Migrations" folders that were changed.
Those files are auto-generated by Entity Framework Migrations tool so they will probably get changed again next time we make a DB change.

Closing this pull request until other PRs are accepted and code-freeze is implemented.


Answer cuts off

Hi, I've tried several prompts but the answer always cuts off. Any idea why that would be?
I don't think it has to do with the tokens limit.. This was the last response:
['The best approach to start a consulting agency to improve the processes a financial company has is to first conduct a thorough analysis of the current processes. This analysis should include identifying areas of opportunity, potential areas of improvement, and any potential risks associated with']

Hey 👋 When did you install? And yes perhaps it's the token limit

Around May 27-30


Exe Won't Run

I have downloaded the .exe file and it won't run i even tried the standalone still won't run idk what to do.

same.

Same here ...
Strange things, from fresh install Soft is trying to download and stop :

But if I run an adb server, it stop when detect trying to get the adb version :

I've tried to install all the .net SDK 6.0, but this not solved the issue ...

There's no need to install the SDK for dotnet 6, the runtime is included with the app.
This issue is a duplicate of (link), essentially the app crashes without triggering the unhandled exception handler, (which shouldn't be possible) so there's no logging to explain what went wrong. I have some evidence to suggest it relates to the avalonia update qp recently underwent.
It doesn't relate to the startup process or anything, as it occurs at a random point during startup, typically when the window is clicked, but not always. I can't reproduce it at the moment but I'm doing all I can.

There is some suggestion that running as administrator fixes the issue.

It does! Thank you!

closing due to being a duplicate


Last layer activation

High, thank you for making such a wonderful tutorial! It helped with my study greatly.
I see that, the linear layer is used to compute the score of each vocab.
as in  self.fc = nn.Linear(decoder_dim, vocab_size) .
In other paper, they would use softmax activation, since we are training with
nn.CrossEntropyLoss().to(device)
what are your thoughts on this? Will there be performance increase when linear activation is swapped with softmax?  (of course...softmax is computationally expensive with larger vocab size)

Hi @miroblog, thank you, glad it's helpful for you.
We're training with the Cross Entropy loss too, so the softmax activation has to be applied. However, this is done internally by torch.nn.CrossEntropyLoss(). It's mentioned in its documentation that -

This criterion combines nn.LogSoftmax() and nn.NLLLoss() in one single class.

The input is expected to contain scores for each class.

These are just raw, unactivated scores. Therefore, we do not need to apply the softmax (or log) operations ourselves.
On the other hand, if we use torch.nn.NLLLoss(), which is generally the same as the Cross Entropy Loss, we need to apply both the log and softmax operations ourselves.

The input given through a forward call is expected to contain log-probabilities of each class.

Oh thank you for your prompt response :)


New operator : validate

validate should allow users to pass a Validator to a Traversable type, that performs a sequence operation on the results of the validation.
Ideally the return type would be an
Ior<Traversable<? extends F>,Traversable<? extends S>>

where F is the failure type and S is the success type, and each type overrides the return type to
Ior<MYTYPE<? extends F>, MYTYPE<? extends S>>

But this isn't possible. So
Ior<ReactiveSeq<F>,Reactive<S>>

This would allow us to process the failure and success streams separately.

LazyFutureStream<Data> inputStream;
inputStream.validate(validationRules)
           .map (s->s.toFutureStream().peek(this::save).run())
           .swap()
           .map(s->s.forEach(this::logError);

Implemented in validate method for Traversable #134


Gaming

Love the project!
How can i full screen quake 3? Is it possible to add custom dos game to this image or maybe bind a volume to it to load from outside?

Thanks! I recently added maximize support to Quake 3, but in general I do not yet have support for most apps in full screen. It's something I will consider adding, like I have done for Photos. Thanks for the idea to add it.
Indeed you can mount external folders from the local system to load custom dos games. You can also drag/drop files in. And on my website I have my favorite 13 DOS Shareware games.
https://dustinbrett.com/?url=/Users/Public/Documents/Games/DOS Bundles

Is there an exisitng folde i can map to or if use a new naem will it create it for me in the os when it mounts?

Is there an exisitng folde i can map to or if use a new naem will it create it for me in the os when it mounts?

No there is nothing pre-existing and the mounting feature is just simply connecting a local folder into the site.


Compact Style Add line break on every line

What i can do?  maybe i do something wrong?

Hey! Sorry, but these empty lines are added by node-sass. I haven't found any option to disable that.

mmmm bad news...

I read lot on node-sass docs but no one had the same problem..

Well, I could add an option for removing empty lines. But then, every empty line is deleted?!

Like sass the  documentation say the correct logic would be:
"Nested rules are placed next to each other with no newline, while separate groups of rules have newlines between them"
http://sass-lang.com/documentation/file.SASS_REFERENCE.html#_15

But I'm not the author of node-sass ;) You should create an issue here

Ok many thanks for your amazing job!


Validate translation word in e2e test

What kind of PR is this?:

/kind feature
What this PR does / why we need it:
Currently we have 3 language support other than english. In this PR we are laying the foundation to test the translation words. To verify translation we have added e2e test check for all languages
to use this feature here is the steps to use

Assign test id to the component containing text to be tested in this way

<Component testID='text-translation-<SCREEN_NAME>=<TEXT>

for eg:
<ThemedSectionTitle
  testID='text-translation-screens/BalancesScreen=PORTFOLIO'
  text={translate('screens/BalancesScreen', 'PORTFOLIO')}
/>

In e2e test case use predefined cypress method as

cy.checkLnTextContent(<FUNCTION_TO_VISIT_SCREEN>, <SCREEN_NAME>)

for eg:
cy.checkLnTextContent((): void => {
  cy.getByTestID('bottom_tab_dex').click()
  cy.getByTestID('dex_guidelines_screen').should('exist')
}, 'screens/DexGuidelines')

Which issue(s) does this PR fixes?:

Fixes #855
Additional comments?:
Developer Checklist:

[x] Read your code changes at least once
[ ] Tested on iOS/Android device (e.g, No crashes, library supported etc.)
[ ] No console errors on web
[ ] Tested on Light mode and Dark mode*
[ ] Your UI implementation visually matched the rendered design*
[x] Unit tests*
[x] Added e2e tests*
[ ] Added translations*

Closing this for now, let's re-open another one once we're working on it again


Search API returns 2 items while TotalRows is greater

Target SharePoint environment
SharePoint Online
What SharePoint development model, framework, SDK or API is this about?
SharePoint REST API
Developer environment
None
What browser(s) / client(s) have you tested

[ ] 💥 Internet Explorer
[ ] 💥 Microsoft Edge
[ ] 💥 Google Chrome
[ ] 💥 FireFox
[ ] 💥 Safari
[ ] mobile (iOS/iPadOS)
[ ] mobile (Android)
[X] not applicable
[ ] other (enter in the "Additional environment details" area below)

Additional environment details

browser version
SPFx version
Node.js version
etc

Describe the bug / error
I'm performing a search using the following:
_api/search/query?querytext=%27benedict%27&querytemplate=%27{searchterms}%20AND%20PECVStatus:3%27
In this case, I know the querytext is part of a searchable managed property & PECVStatus is a queryable managed property. The query returns 2 results (rowCount), while stating there are a number of TotalRows of 6. If I search for other names that I know should also be found I have the same behaviour (2 results while more available).

It happens for some other querytexts even when not providing a querytemplate, but only some:
_api/search/query?querytext=%27Chris%27

However, if I add startrow=1 to the request, the rowCount returned is correct (except I'm missing the first item).
_api/search/query?querytext=%27benedict%27&querytemplate=%27{searchterms}%20AND%20PECVStatus:3%27&startrow=1

Steps to reproduce

Expected behavior
The number of results returned (RowCount) should be equal to TotalRows, at least in cases where RowLimit and RowsPerPage is not specified

This is just how SharePoint Search "works". Every project I've worked on that relies on SPO Search has resulted in delays and compromises in the final product due to issues like this. In some cases, crawling can take months. You may also notice that new users might see even fewer results initially (for the first few days, weeks or however busy your tenant and shared infrastructure is).
You can log a call with Microsoft but they can only tell you how many tasks are queued for Search but they can't tell you any details (such as, where the majority of index requests are coming from and so on due to data privacy).
The only advise I can give you is to refrain from manually reindexing lists or sites. Only do that if a Microsoft engineer advises you to do it as it may delay your support call being escalated. This is because they might say that due to too many reindexing requests, you have to wait x number of days or weeks and then assess the situation which means your call gets put on hold (this has happened on a project I once worked on) .


Suggestion: recipes book

Could you add a Book to describe all pipes and if you klick on it it displays the crafting recipe in a GUI? Would be awesome ifplayers wont need to browse the internet to learn how to use it :)

Yeah, I already added this feature to my todo list ;)

1.13 does this by itself, no need for a custom gui.

@tr7zw how do you mean? Does 1.13 display custom recipes in the recipes book? :O

@ScuroK Recipes added via the Bukkit api will show up in the clients recipe book. The only thing that may have to be done is unlocking the recipe when joining, not when its crafted for the first time.


Update convert_gt_xml.py

Fix the inconsistent type when writing the text file.
For this error:

Thanks! The thing is that I wanted to address this issue https://github.com/Cartucho/mAP/issues/38.
Apparently, some values from the XML format can be floats, therefore we need to convert them to int.
We can solve this by changing that line of code to:
      new_f.write(obj_name + " %d %d %d %d\n" % (float(left), float(top), float(right), float(bottom)))
This way we first convert the string of float/int to string of int.

Another possibility is to treat this as only strings:
new_f.write("%s %s %s %s %s\n" % (obj_name, left, top, right, bottom))
And then do the parsing to int in the main.py.
Which one do you think it is best?

Thanks, @Cartucho. For calculating the mAP, does it require everything to be integers? If we can just write the float as strings, then we do not need to parse them back and forth, thus the second one :) Let me know, and I will change my pr.

I agree with you, let's do it with the second one.

Thanks. Please take a look :)


where is "flame.pkl"?

I have downloaded the code. When I ran the command "python -m main.demo --config config/unitalker.yaml test_out_path ./test_results/demo.npz", an error happend, just like below:
with open(model_path, 'rb') as f:
FileNotFoundError: [Errno 2] No such file or directory: 'resources/binary_resources/flame.pkl'

I have downloaded the code. When I ran the command "python -m main.demo --config config/unitalker.yaml test_out_path ./test_results/demo.npz", an error happend, just like below: with open(model_path, 'rb') as f: FileNotFoundError: [Errno 2] No such file or directory: 'resources/binary_resources/flame.pkl'

Hi, anddyhzw
FLAME2020: Please download FLAME 2020 and move generic_model.pkl into resources/binary_resources/flame.pkl.
We have updated README to help future researchers.


ARROW-7022, ARROW-7023: [Python] fix handling of pandas Index and Period/Interval extension arrays in pa.array

Fixes https://issues.apache.org/jira/browse/ARROW-7022, and while doing this noticed another bug this is fixing (for which I opened https://issues.apache.org/jira/browse/ARROW-7023)

I've restarted AppVeyor as it seems the build had stopped abruptly.

Thanks!


Fix wording that says this is a web application

The guide says Download the resulting ZIP file, which is an archive of a web application that is configured with your choices. but this is not a web application.

Closed with PR #43


Integrate Pusher Beams into Chatkit 🚀

What?
Integrate Pusher Beams into Chatkit.

CC @hamchapman

          3 Warnings

:warning:
Any changes to library code should be reflected in the Changelog. Please consider adding a note there. \nYou can find it at CHANGELOG.md.

:warning:
PR is classed as Work in Progress

:warning:
The Cartfile or Cartfile.resolved was updated, but there were no changes in the podspec. Did you forget updating the podspec?

SwiftLint found issues
Warnings

File
Line
Reason

AppDelegate.swift
10
Line should be 120 characters or less: currently 121 characters

ViewController.swift
106
Line should be 120 characters or less: currently 143 characters

ViewController.swift
111
Line should be 120 characters or less: currently 123 characters

ViewController.swift
116
Line should be 120 characters or less: currently 123 characters

ViewController.swift
128
Line should be 120 characters or less: currently 131 characters

ViewController.swift
190
Line should be 120 characters or less: currently 131 characters

ViewController.swift
7
Delegates should be weak to avoid reference cycles.

ViewController.swift
8
Delegates should be weak to avoid reference cycles.

ChatManager.swift
109
TODOs should be avoided (This could be nicer).

ChatManager.swift
110
TODOs should be avoided (We don't need to wait for init...).

ChatManager.swift
111
TODOs should be avoided (Do we need to nil out subscrip...).

ChatManager.swift
124
TODOs should be avoided (This is madness).

ChatManager.swift
209
TODOs should be avoided (This being here at the end see...).

ChatManager.swift
213
TODOs should be avoided (Maybe we need some sort of Cha...).

ChatManager.swift
253
TODOs should be avoided (When can percent encoding fail...).

ChatManager.swift
92
Function body should span 40 lines or less excluding comments and whitespace: currently spans 72 lines

ChatManager.swift
257
MARK comment should be in valid format. e.g. '// MARK: ...' or '// MARK: - ...'

ChatManager.swift
113
Variable name should be between 3 and 40 characters long: 'cu'

ChatManager.swift
172
Line should be 120 characters or less: currently 130 characters

ChatManager.swift
191
Line should be 120 characters or less: currently 138 characters

ChatManager.swift
195
Line should be 120 characters or less: currently 129 characters

ChatManager.swift
199
Line should be 120 characters or less: currently 138 characters

PCCurrentUser.swift
24
TODOs should be avoided (This should probably be [PCUse...).

PCCurrentUser.swift
392
TODOs should be avoided (Use the soon-to-be-created new...).

PCCurrentUser.swift
482
TODOs should be avoided (Do we need to fetch users in t...).

PCCurrentUser.swift
685
TODOs should be avoided (Do we need a Last-Event-ID opt...).

PCCurrentUser.swift
792
TODOs should be avoided (What happens if you provide bo...).

PCCurrentUser.swift
814
TODOs should be avoided (This should probably be a room...).

PCCurrentUser.swift
1104
TODOs should be avoided (What to do with an error? Just...).

PCCurrentUser.swift
1054
Unused parameter "data" in a closure should be replaced with _.

PCCurrentUser.swift
712
Function body should span 40 lines or less excluding comments and whitespace: currently spans 42 lines

PCCurrentUser.swift
915
Function body should span 40 lines or less excluding comments and whitespace: currently spans 55 lines

PCCurrentUser.swift
208
MARK comment should be in valid format. e.g. '// MARK: ...' or '// MARK: - ...'

PCCurrentUser.swift
267
MARK comment should be in valid format. e.g. '// MARK: ...' or '// MARK: - ...'

PCCurrentUser.swift
1152
MARK comment should be in valid format. e.g. '// MARK: ...' or '// MARK: - ...'

PCCurrentUser.swift
632
Arguments can be omitted when matching enums with associated types if they are not used.

PCCurrentUser.swift
632
Arguments can be omitted when matching enums with associated types if they are not used.

PCCurrentUser.swift
6
Variable name should be between 3 and 40 characters long: 'id'

PCCurrentUser.swift
67
Variable name should be between 3 and 40 characters long: 'id'

PCCurrentUser.swift
178
Variable name should be between 3 and 40 characters long: 'id'

PCCurrentUser.swift
195
Variable name should be between 3 and 40 characters long: 'id'

PCCurrentUser.swift
229
Variable name should be between 3 and 40 characters long: 'id'

PCCurrentUser.swift
284
Variable name should be between 3 and 40 characters long: 'id'

PCCurrentUser.swift
344
Variable name should be between 3 and 40 characters long: 'id'

PCCurrentUser.swift
1131
Enum element name should be between 3 and 40 characters long: 'messageIDKeyMissingInMessageCreationResponse'

PCCurrentUser.swift
122
Collection literals should not have trailing commas.

PCCurrentUser.swift
656
Opening braces should be preceded by a single space and on the same line as the declaration.

PCCurrentUser.swift
199
Line should be 120 characters or less: currently 122 characters

PCCurrentUser.swift
217
Line should be 120 characters or less: currently 144 characters

PCCurrentUser.swift
229
Line should be 120 characters or less: currently 140 characters

PCCurrentUser.swift
233
Line should be 120 characters or less: currently 137 characters

PCCurrentUser.swift
513
Line should be 120 characters or less: currently 123 characters

PCCurrentUser.swift
541
Line should be 120 characters or less: currently 121 characters

PCCurrentUser.swift
566
Line should be 120 characters or less: currently 139 characters

PCCurrentUser.swift
570
Line should be 120 characters or less: currently 139 characters

PCCurrentUser.swift
592
Line should be 120 characters or less: currently 133 characters

PCCurrentUser.swift
603
Line should be 120 characters or less: currently 132 characters

PCCurrentUser.swift
697
Line should be 120 characters or less: currently 123 characters

PCCurrentUser.swift
953
Line should be 120 characters or less: currently 123 characters

PCCurrentUser.swift
1059
Line should be 120 characters or less: currently 136 characters

PCCurrentUser.swift
1161
Line should be 120 characters or less: currently 123 characters

Errors

File
Line
Reason

ViewController.swift
31
Force casts should be avoided.

ChatManager.swift
285
Line should be 120 characters or less: currently 366 characters

ChatManager.swift
294
Line should be 120 characters or less: currently 248 characters

PCCurrentUser.swift
5
Type body should span 200 lines or less excluding comments and whitespace: currently spans 685 lines

PCCurrentUser.swift
694
Variable name should be between 3 and 40 characters long: 'r'

PCCurrentUser.swift
1174
File should contain 400 lines or less: currently contains 1174

  Generated by :no_entry_sign: Danger

@hamchapman I've addressed all the comments and squashed all the commits.


Paper attached

Hi, is there any paper associated with the code?
Best regards.

https://arxiv.org/pdf/1703.02910.pdf


Mapzen logo jumpiness on mobile

Hovering over the Mapzen logo in the navbar in a mobile-width viewport makes the logo jumpy:

@rfriberg does this fit better in the website repo?

D'oh.  I probably introduced that with a change I pushed yesterday.  Nah. Leave it here. I'll fix.
Thanks!


feat(lab): link visit to labs

Add functionality to link visit to lab, and updated the schema to have a visitId in the Lab model.
fix #2184
Fixes #2184.
Changes proposed in this pull request:

Updated Lab db model to have an optional visitId
In the "New Lab Request" screen, added a required dropdown to associate the lab request with a visit id.

In #2243, @jackcmeyer mentioned "updating the pouchdb schema" to describe the relationship between appointments and labs. We have since switched to associating visits with labs. Does updating the schema still apply here? I do not see any inclusion of "visits" in the schema already.