Codebase: Rename NavigationToolbar to AddressToolbar

Resolved / Related Issues
Items resolved / related issues by this PR.

Closes #5470

Details of Changes
Add details of changes here.

Renames NavigationToolbar user control to AddressToolbar.

Validation
How did you test these changes?

[x] Built and ran the app
[ ] Tested the changes for accessibility

Screenshots (optional)
Add screenshots here.

Not sure how I feel about this one.

@puppetsw Any concerns?

@puppetsw Any concerns?

Not anymore. I think renaming the toolbar view model makes sense as well, as it is shared by the address toolbar and inner navigation.


⚠ Turnos 24x7 has degraded performance

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

HTTP code: 200
Response time: 993 ms

Resolved: Turnos 24x7 performance has improved in 5101b1a.


feat(recording): add recording button/indicator

closes: https://github.com/mconf/bbb-mobile-sdk/issues/414

Portal needs to handle the record permission

@frankemax it's missing the second commit of the PR (with the design implementation)

@frankemax it's missing the second commit of the PR (with the design implementation)

Whoops... thanks!


[YUNIKORN-1885] Update Go version filter

What is this PR for?
The Go version filter pulls in patch release details which shows errors when running tests with pre-release (release candidate) go version to check for compatibility.
What type of PR is it?

[X] - Improvement

What is the Jira issue?

https://issues.apache.org/jira/browse/YUNIKORN-1885

How should this be tested?
Install a rc for Go and set a Go version before running a test, example:
GO="go1.21rc3" make test

LGTM
+1 (non-binding)


Sign In Modal

The pop up sign in form I made for the app is in a separate sign in button at the bottom of the bag and not in the nav bar, It would be much better if it was working with the nav bar so if you have ideas to fix it let me know

http://stackoverflow.com/questions/13648979/bootstrap-modal-immediately-disappearing


npm install

I tried to install using npm install --save card
Errored out - Looks like sassify is a dependency.

If you're still having this issue, please reopen.

It's not an issue. I just added sassify into my own package.json file when it should be in your package.json file. I was just reporting.


Azure: Add missing csi driver secret

What this PR does / why we need it:
Currently, Azure 4.11 clusters never get ready because the storage driver doesn't get ready because the CSI driver pods don't start because their cloud secret is missing, see e.G. here: https://prow.ci.openshift.org/view/gs/origin-ci-test/logs/periodic-ci-openshift-hypershift-main-periodics-e2e-conformance-azure/1517382725126852608
This fixes that
Which issue(s) this PR fixes (optional, use fixes #<issue_number>(, fixes #<issue_number>, ...) format, where issue_number might be a GitHub issue, or a Jira story:
Fixes #
Checklist

[ ] Subject and description added to both, commit and PR.
[ ] Relevant issues have been referenced.
[ ] This change includes docs.
[ ] This change includes unit tests.

/lgtm


Bug: VTU not recognizing props object passed to the mount() function

When mounting a component with props object, VTU does not read props from the options object passed to the mount() function and therefore the test always fails.

Can you provide us a small repro online using https://stackblitz.com/github/vuejs/create-vue-templates/tree/main/typescript-vitest?file=src%2Fcomponents%2F__tests__%2FHelloWorld.spec.ts ?
It only takes a few minutes, and we'll be able to take a look

Closing until reproduction is provided.


[Bugfix] Within NavigatorIOS, navigator.replace doesn't update the navigation bar.

Duplicate of #3516?

@cancan101 oh yes, it is. I've just closed the other one.

about "this.state.updatingAllIndicesAtOrBeyond >= i" changing to "this.state.updatingAllIndicesAtOrBeyond <= i":
I've checked the context, updatingAllIndicesAtOrBeyond is meant to be the min index of those to be updated. So the comparing should be "<= i" either than ">= i"

CI_Test faild with "TEST_TYPE=e2e". What's this?

Any updates on this issue ?

cc @mkonicek

It seems that every pull request does not pass the ci test.

Can we just ignore the ci test, and review the code to decide whether it can be accepted?

I'll need a bit more context on this. The PR is described as a bug fix, but seems to introduce new API in the form of the skipUpdate property? What is that for?

(and yes, don't worry about the tests, they're broken on master right now)

@nicklockwood the update method is a convenient method to adjust the current route, and it implements this through the replace method. And the skipUpdate property is a helper property to tell the render method not to change the component.
I know these are not the essential for the bugfix. However, without a proper update method, the developer can only directly use the replace method to do the update things, and probably writes a buggly code, for example, forgetting passing in the current component property or the current passProps property when he/she writes replace({title: 'new title'}) to update the title. For this reason, I provide the convenient method together with the bugfix.

@mrspeaker would you mind taking a look at this pull request? It's been a while since the last commit was reviewed.

@rocman Note that the Navigator is getting deprecated in favor of NavigationExperimental. Do you plan to rebase this PR or should we close it? Sorry for the churn.

It's been a while since the last commit was reviewed and the labels show this pull request needs review. Based on the blame information for the files in this pull request we identified @janicduplessis as a potential reviewer. Could you take a look please or cc someone with more context?

This pull request hasn't seen activity in a while. If you still want to merge this, please open a new pull request.

If I understand it correctly, this issue is still not resolved (since I found myself here because I was having this issue).  So I don't quite understand...why was this closed?  Are we supposed to move to NavigationExperimental (as someone previous mentioned) and therefore this isn't worth fixing in NavigatorIOS?

@xogeny the PR was closed because it was inactive for almost a year.

@satya164 Understood.  But as far as I can tell from the discussion, the PR was submitted but never reviewed.  So it isn't really an issue with the submitter but the maintainers. (?)  Given that the bug is still present, should I assume from this that there just isn't an interest in fixing it?  Or is there some change on the horizon that makes this all irrelevant.  I'm just trying to understand if this will ever be fixed.

@xogeny sometimes it takes long to start the review since the team is busy with other work. the PR needed to be rebased against master to be able to be merged, and @mkonicek asked the author to rebase the PR. it was closed because there was no response from the author for 4 months.
AFAIK Facebook doesn't use NavigatiorIOS internally, and it's usage is not encouraged in the docs either, so it's a low priority for them. It's upto the contributors to fix bugs and maybe help in reviewing PRs.
Besides it's going to be deprecated soon when React Navigation is released - https://github.com/facebook/react-native/wiki/Roadmap

@satya164 Thanks.  That is exactly the kind of summary I was looking for.
Note: you say that the use of NavigatorIOS is "not encouraged in the docs" but the docs actually say: " If you are only targeting iOS and would like to stick to the native look and feel, check out NavigatorIOS." which is part of what through me off.  But since it is clear from your answer that NavigatorIOS is on its way out, I'll try NavigationalExperimental which, according to the docs, I "can't go wrong with".
Thanks again.

@xogeny I'd suggest to hold off using Navigation Experimental. We're working on React Navigation which will be released soon. It's much easier to use and has more features.


modifying test pipeline for codecov

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@989588f). Click here to learn what that means.
The diff coverage is n/a.

@@          Coverage Diff           @@
##             main     #13   +/-   ##
======================================
  Coverage        ?   0.00%
======================================
  Files           ?      39
  Lines           ?    2521
  Branches        ?       0
======================================
  Hits            ?       0
  Misses          ?    2521
  Partials        ?       0

Flag
Coverage Δ

unittests
0.00% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

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 989588f...a33c217. Read the comment docs.


release 0.8.3 fails GPG check

gpg: BAD signature from "CoreOS Application Signing Key<EMAIL_ADDRESS>Downloaded 0.8.2, that one validates fine

/cc @brianredbeard @colhom
Also, a side note @rgcosma, the kube-aws tool has moved to its own top-level repository @ https://github.com/coreos/kube-aws

@rgcosma which tarball/sig combination are you seeing this with?

Hi - might have been a temporary issue, and sorry for not saving the file locally at the time. I did download it twice yesterday, though, no proxy..
Right now, for https://github.com/coreos/coreos-kubernetes/releases/download/v0.8.3/kube-aws-linux-amd64.tar.gz
$ gpg2 --verify 083.sig 083.tgz
gpg: Signature made Wed 19 Oct 2016 21:35:12 UTC using RSA key ID BEDDBA18
gpg: Good signature from "CoreOS Application Signing Key<EMAIL_ADDRESS>

thank you @aaronlevy will use the new repo


Weird node connections,

Describe the Bug
I don't know exactly what's the problem. nodes and connections get separated from each other.
Your Example Website or App
No response
Steps to Reproduce the Bug or Issue
I used overview example code (without any changes)
Expected behavior
.
Screenshots or Videos

Platform

OS: Windows
Browser: Chrome
Version: 98

Additional context
No response

Could you create a repo or codesandbox. It's hard to tell from a screenshot. I could imagine that you have some svg related styles that are the reason for this bug.

Ok. I try it and send you the link

Sorry for late response. I had to wait until company get opened.
I tested in fresh nextJS and reactJS apps and its working fine. The problem was occurred because of RTL styles.


[Doc] Callback url isn't mandatory when not defining 'Code' or 'Implicit' as grant types

'New Application form' under 3rd step in [1] mentioned that callback url is mandatory to define when configure 'OAuth/OpenID Connect Configuration'. But when a user doesn't define 'Code' or 'Implicit' as grant types, 'callback url' doesn't need to be add. Need to mention this in documentation.
[1] https://docs.wso2.com/display/IS540/Configuring+OAuth2-OpenID+Connect+Single-Sign-On

Thank you for your contribution!
We are closing this issue since it has not been prioritized for a long time. Chances are that it has already been solved in more recent versions. If not, we will be re-evaluating this when it becomes a priority.


[FLINK-7415][Cassandra Connector] Add comments for cassandra-connector test classes

What is the purpose of the change
Help beginners follow it with less confusion.
Brief change log

Add comments for cassandra test classes

Verifying this change
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (no)

Documentation

Does this pull request introduce a new feature? (no)

I ran the test class with an exception, I found the cause finally.

+1.

merging.


Render DeleteModal conditionally in operator-mode submodes

[CS-6403]

update deleteCard method in Actions
use it in both submodes and render delete modal only if there's an item to delete
update usage in other components

Good improvements. Can we go further and remove the setupDeleteModal callback and instead pass in the item to delete and onConfirm and onCancel actions? I think this would avoid the parent component needing to keep a reference to this.deleteModal which is something we ought to generally avoid where possible.


Yield coroutines while stack unwinding causes panic while panicking

Reproducible example:
extern crate coio;
extern crate env_logger;

use coio::Scheduler;

fn main() {
    env_logger::init();

    Scheduler::new()
        .run(|| {
            struct A;

            impl Drop for A {
                fn drop(&mut self) {
                    Scheduler::sched();
                }
            }

            let _a = A;

            panic!("PANICKED in coroutine");
        }).unwrap();
}

While the coroutine is unwinding, Drop of A will be called. Coroutine is yield in the drop() function. Coroutine is now suspended, which means that the currently thread is still in panicking status, but execution process is now be switched to another coroutine. If the other coroutine panic, too, then it will definitely cause panic while panicking!.

cc @lhecker , this is the root cause.

@zonyitoo Yup that's basically why we asked the Rust devs to reconsider and stop using TLS for panics in the stdlib back then when we worked on coio. Since they denied my PR and basically said that it's unlikely even for an RFC to change this behaviour I think this might be a dead end for the project. 😕 (Unless you fork the stdlib of course.)


add launch mp distributed job

@haowang101779990  new doc, please add to "to do list" of translation


⚠ ClamAV has degraded performance

In 43fce43, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 197 ms

Resolved: ClamAV performance has improved in e312610.


[npm] no pre publish build

adding build to prepublish will prevent publishing the wrong version

this appears closed...   for a while.


feat(crate-validator): Add domain-specific crates

Overview
This PR seeks to address points raised in #562, and also adds other improvements on the existing tool:

Downloads no longer time out due to slow internet connection,
Progress bar for download process added.

Related issues
Closes #562

Nice work! Looks like a good bit of code cleanup as well. Just left a few small notes.


BTW map grid is offset from vanilla

BTW now places maps on a universal grid.  This grid is offset from vanilla MC and external tools meant to work with maps (e.g.,  https://github.com/jason-green-io/papyri) do not render BTW maps correctly.  Issue seems to be that the map data files containe an xCenter and zCenter which isn't actually the center coordinates but the upper left.
Might cause more issues than it is worth to fix but wanted to document it.

Hmmm. In a perfect world we'd fix this, but I'm not sure doing so is possible in a backwards-compatible way. Even if it didn't affect existing maps, new maps would then not align to the old maps.


Set continuous integration

Examine the best option for setting up a continuous integration workflow.
To be accepted, the CI tool must be:

[x]  Making the builds always that a new Pull Request is open
[x]  Be adequate for the technologies that are going to be used

Travis added to the continuous pipeline delivery.


Add utility function to add render nodes

Objective

Adding a node to the render_graph can be quite verbose and error prone because there's a lot of moving parts to it.

Solution

Encapsulate this in a simple utility method

Mostly intended for optional nodes that have specific ordering
Requires that the Node impl FromWorld, but every internal node is built using a new function taking a &mut World so it was essentially already FromWorld

Use it for the bloom and fxaa nodes.

The main nodes don't use it because they rely more on the order of many nodes being added

Changelog

Added add_node::<T: Node + FromWorld>
Impl FromWorld for BloomNode and FxaaNode
Added RenderGraph::add_node_with_edges()
Added RenderGraph::add_node_edges()
Added RenderGraph::sub_graph()
Added RenderGraph::sub_graph_mut()

Notes
This was taken out of https://github.com/bevyengine/bevy/pull/7995 because it works on it's own. Once the linked PR is done, the new add_node() will be simplified a bit since the input/output params won't be necessary.
This utility function will be useful in most of the upcoming render nodes so it's usefulness will be more relevant at that point.
Partially fixes #7985

I've been considering doing that because it would make a few of those patterns even simpler. The issue is that it wouldn't work with EmptyNode that is used in a few places like post processing, but having a way to automatically label a node would be nice a bit like how you can use systems as labels.

I've been considering doing that because it would make a few of those patterns even simpler. The issue is that it wouldn't work with EmptyNode that is used in a few places like post processing, but having a way to automatically label a node would be nice a bit like how you can use systems as labels.

Would something like this work?
struct EmptyNode { name: &'static str }

impl Node for EmptyNode {
  fn name(&self) -> &'static str {
      self.name
  }
}

We could also maybe replace empty nodes with subgraphs? Idk, haven't looked at how subgraphs work.

Would something like this work?

Yeah, that's exactly what I thought of after my original comment.
I still think I'd prefer doing this in a separate PR just because big PRs tend to take a long time to merge.

I think this helper should be an "app builder" method

I didn't even consider this, I was just trying to encapsulate this common pattern but that diff looks really good.

I think we should split out adding edges and nodes.

That's fair, I liked the idea of bundling everything so it's hard to forget to set a node edge, but I do agree that they are separate concepts. My approach also made it impossible to use add_node in the main_pass_2d/3d which is why I ended up adding the add_node_edges function to the render_graph.

not a fan of "helper" naming generally

Completely agree here, I just really didn't know how to name it/where to put this.
In general, I'm on board with the app extension. My only concern here is that people might end up calling it on the App and not RenderApp. I'm not sure how to handle that case.

Since I'm fully on board with this, I updated the description to match this new api

In general, I'm on board with the app extension. My only concern here is that people might end up calling it on the App and not RenderApp. I'm not sure how to handle that case. To be fair, it was also an issue with the original approach.

I feel that we should have a RenderAppExt trait implemented on App. Users won't be able to access the methods unless they import the trait, which should hopefully be less error prone.

RenderAppExt trait

I don't think this is possible because RenderApp is just a marker for a sub app. Maybe Apps could have a concept of labels to identify you are using the correct App. Definitely out of scope here though.

Alternative: Make RenderApp a type wrapping the render App, and implement the methods there. Then, instead of get_sub_app(), add get_render_app() which wraps the app in RenderApp.
Users could still call get_render_app() on the wrong app, if they had a custom subapp, but it's much more unlikely.


Is there a recommended fix for error Type with the DIET Classifier?

Rasa version: 2.5.1
Rasa SDK version (if used & relevant):
Rasa X version (if used & relevant):
Python version: 3.7
Operating system (windows, osx, ...): Linux
Issue:
Right now, I am getting an error when I try launching rasa open-source with docker-compose up. The error(s) that I get include the following: /opt/venv/lib/python3.7/site-packages/rasa/shared/utils/io.py:93: UserWarning: The model at 'model-location-here' could not be loaded. Error: <class 'ValueError'>: Wrong loss type 'cross_entropy', should be 'margin' or 'softmax' and /opt/venv/lib/python3.7/site-packages/rasa/shared/utils/io.py:93: UserWarning: Agent could not be loaded with the provided configuration. Load default agent without any model. Has this error(s) been encountered before?
If so, what was the solution? I have a feeling that this is due to the DIET Classifier potentially, but I could be wrong. When testing this model outside of the instance that houses docker, the most recent model works no problem. I don't know if this could have an effect on anything (and it may be poor usage of git on my part) but I push a git repo (from AWS Sagemaker) with all of the rasa files that comes with the command rasa init as well as the models. I then pull the git repo into the EC2 instance and then try to run the models. I tried this back in December 2020 and had no issues.
Error (including full traceback):
on EC2 instance where it is breaking:
main2_rasa  | 2021-05-05 11:30:25 ERROR    rasa.core.agent  - Could not load model due to Wrong loss type 'cross_entropy', should be 'margin' or 'softmax'.
main2_rasa  | /opt/venv/lib/python3.7/site-packages/rasa/shared/utils/io.py:93: UserWarning: The model at 'beta2/models' could not be loaded. Error: <class 'ValueError'>: Wrong loss type 'cross_entropy', should be 'margin' or 'softmax'
main2_rasa  | /opt/venv/lib/python3.7/site-packages/rasa/shared/utils/io.py:93: UserWarning: Agent could not be loaded with the provided configuration. Load default agent without any model.

on AWS Sagemaker Instance where everything is fine and I run rasa train:
2021-05-05 11:24:50 INFO     rasa.model  - Data (version) for Core model section changed.
2021-05-05 11:24:50 INFO     rasa.model  - Data (version) for NLU model section changed.
2021-05-05 11:24:50 INFO     rasa.model  - Data (version) for Core model section changed.
2021-05-05 11:24:50 INFO     rasa.model  - Data (version) for NLU model section changed.
Training NLU model...
2021-05-05 11:24:51 INFO     rasa.nlu.utils.spacy_utils  - Trying to load spacy model with name 'en_core_web_md'
2021-05-05 11:24:53 INFO     rasa.nlu.components  - Added 'SpacyNLP' to component cache. Key 'SpacyNLP-en_core_web_md'.
/home/ec2-user/anaconda3/envs/JupyterSystemEnv/lib/python3.6/site-packages/rasa/utils/train_utils.py:565: UserWarning: constrain_similarities is set to `False`. It is recommended to set it to `True` when using cross-entropy loss. It will be set to `True` by default, Rasa Open Source 3.0.0 onwards.
  category=UserWarning,
/home/ec2-user/anaconda3/envs/JupyterSystemEnv/lib/python3.6/site-packages/rasa/utils/train_utils.py:537: UserWarning: model_confidence is set to `softmax`. It is recommended to try using `model_confidence=linear_norm` to make it easier to tune fallback thresholds.
  category=UserWarning,

Command or request that led to error:
sudo docker-compose up

Content of configuration file (config.yml) (if relevant):
not relevant as it works on AWS Sagemaker

Content of domain file (domain.yml) (if relevant):
not relevant as it works on AWS Sagemaker

Thanks for the issue, @hsm207 will get back to you about it soon!
You may find help in the docs and the forum, too 🤗

hi @jeff-ridgeway
Are you sure the rasa version you used in AWS SageMaker, the  EC2 instance and in Docker compose is identical?
To check for the rasa version that was used to train the model, you can untar the model .tar.gz file and look at the metadata.json
file in the core and nlu folders for the rasa and rasa_version tag respectively,
For example:
tar -xf 20210505-130354.tar.gz
echo `cat core/metadata.json | grep \"rasa\"`
echo `cat nlu/metadata.json | grep \"rasa_version\"`

If the model was trained with rasa 2.5.1, the the result of the two echo calls should be:
"rasa": "2.5.1"
"rasa_version": "2.5.1"

Hey @hsm207
Thanks for getting back to me about this. I was able to run your commands on both my Sagemaker and AWS EC2 instance as well as my Dockerfile for docker-compose. Everything says 2.5.1 for the version. In my Dockerfile I have rasa/rasa:2.5.1-full. Seeing that have the right version, supposedly, do you have any of  places/things I should check for? 🤔

hey @jeff-ridgeway
I am not able to reproduce the error 🤔.  A couple of questions:

Is your environment back in Dec 2020 identical to your current environment or was there any update to AWS SageMaker, EC2 instance and/or the docker-compose?
Did you make any changes to config.yml after you ran rasa init?

I don't know if there was any update to AWS Sagemaker between last December and now. I didn't updated anything on my end. I just know that everything works on Sagemaker but not on the EC2 instance 🤷🏾♂. The docker-compose file seems to be fine. This is my docker-compose file

version: '3.0'

services:
  rasa:
    container_name: rasa
    # go to docker hub/rasa changelog to see what version and flavour of rasa that you want
    # Make sure that the version you specify is the same as the version that you pip installed
    #image: rasa/rasa:2.5.1-full
    #image: flask-rasa:latest
    build:
      context: .
      dockerfile: Dockerfile
    expose:
      - port-number
    # This command will copy everything in current diretory to the /app directory in the container
    volumes:
      - .:/app
  duckling:
    image: rasa/duckling:latest
    ports:
      - "port:port"

  app:
    image: actions:latest
    ports:
      - "port-number1:port-number2"

  nginx:
    container_name: nginx
    image: nginx
    depends_on:
      - "rasa"
      - "duckling"
      - "app"
    ports:
      - port:port
      - port:port
    volumes:
      - path/to/directory
      - path/to/directory

Yes, I did make changes to the config.yml file after I ran rasa init. This is my current config.yml file:

# Configuration for Rasa NLU.
# https://rasa.com/docs/rasa/nlu/components/
language: en

pipeline:
# # No configuration for the NLU pipeline was provided. The following default pipeline was used to train your model.
# # If you'd like to customize it, uncomment and adjust the pipeline.
# # See https://rasa.com/docs/rasa/tuning-your-model for more information.
#    - name: WhitespaceTokenizer
#    - name: RegexFeaturizer
#    - name: LexicalSyntacticFeaturizer
   - name: "SpacyNLP"
     model: "en_core_web_md"
   - name: "SpacyTokenizer"
   - name: "SpacyFeaturizer"
     "pooling": "mean"
   - name: "SpacyEntityExtractor"
     dimensions: ["PERSON", "email"]
   - name: RegexEntityExtractor
     case_sensitive: False
     use_lookup_tables: True
   - name: DucklingEntityExtractor
     url: "http://duckling:port-number"
     dimensions: ["amount-of-money"]
     timeout : 3
     locale: "en_US"
   - name: DIETClassifier
     epochs: 150

# Configuration for Rasa Core.
# https://rasa.com/docs/rasa/core/policies/
policies:
# # No configuration for policies was provided. The following default policies were used to train your model.
# # If you'd like to customize them, uncomment and adjust the policies.
# # See https://rasa.com/docs/rasa/policies for more information.
  - name: MemoizationPolicy
  - name: TEDPolicy
    max_history: 5
    epochs: 100
  - name: RulePolicy
#   - form: FormPolicy

my Dockerfile is configured like so:
FROM ubuntu:18.04
FROM rasa/rasa:2.5.1-full

USER root
RUN apt-get update && apt-get install -y python3.7 python3-pip
COPY . .
WORKDIR .

RUN chmod +x ./script
RUN pip3 install -r requirements.txt
USER 1001

EXPOSE ports-here
ENTRYPOINT ["sh", "script", "run"]

Let me know if you need anything else

hey @jeff-ridgeway
Could you try to reproduce the issue with mood bot? i.e. just run rasa init and then train the bot in SageMaker and then take the model.tar.gz file into an EC2 instance and set up mood bot using docker compose.
If the same issue appears, please share the model.tar.gz file.


Add zig binding

I made this work with zimgui and raylib-zig.
And I also updated to this work with newer ImGUI API (no obsolete API)

now it didn't work. weird

turns out zimgui is out-dated

Just FYI. I have no way to test or maintain a zig binding, so that would not be something that would go into this repository. It would be better to have the zig files be in a fork or separate repository.

Do you have plan to update this to work with DISABLE_OBSOLETE_KEYIO (c define)? the keyboard api change if you define that.

yeah,I just did that part
On Mon, Jan 2, 2023 at 1:17 PM locriacyber @.***> wrote:

Do you have plan to update this to work with DISABLE_OBSOLETE_KEYIO (c
define)? the keyboard api change if you define that.
—
Reply to this email directly, view it on GitHub
https://github.com/raylib-extras/rlImGui/pull/15#issuecomment-1369210231,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACOU7QRMVKNGWLUWMZNWK3WQNAV5ANCNFSM6AAAAAATO6WNCY
.
You are receiving this because you commented.Message ID:
@.***>

--
Jeffery Myers
@.***


Updates to powershell commands to install extension

Current parameter values for Public and ProtectedSettings are incorrect. They must be hash table values instead of json.

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

#sign-off

#sign-off ,Please help merge the pr.

@GitHubber17 , can you help merge the PR?


W.i.P Tighten Constraints on Candidate Models

basically, it checks that the class is a model before flagging it as a candidate.  still awaiting tests though

Coverage remained the same at 100.0% when pulling 087174000b69429e102dfc0f578b88c130bcee89 on c-harris:master into 59a31b7634669831f05398b362a3697e22dfe6d8 on Algo-Web:master.


OpenAI Client format + chat template for a single call

System Info
latest docker
Information

[X] Docker
[ ] The CLI directly

Tasks

[ ] An officially supported command
[ ] My own modifications

Reproduction
Hello,
Can you tell me please how to implement the following functionality combined:

I'm interested in OpenAI Client format:
prompt= [
{"role": "system", "content": "You are a sassy, wise-cracking robot as imagined by Hollywood circa 1986."},
{"role": "user", "content": "Hey, can you tell me any fun things to do in New York?"}
]

I want to make sure that chat template of the served odel is aplied

I dont want a chat - I want that each call with a prompt statrt from a clear history to aoid token overflow.
Thank you!

Expected behavior
An answer for each prompt indepedent of the previos anser, but with OpenAI client API

You could still use the openai.chat.completions.create but reset the chat history each time? For example:
def generate(prompt: str) -> ChatCompletion:
    messages = [
        {
            "role": "system",
            "content": "You are a sassy, wise-cracking robot as imagined by Hollywood circa 1986.",
        },
        {"role": "user", "content": prompt},
    ]
    return openai.chat.completions.create(messages=messages, model='TGI', base_url='TGI URL?')

Alternatively, you could apply the chat template outside of TGI like this:
import openai
from transformers import AutoTokenizer
def generate(prompt: str) -> Completion:
    messages = [{"role": "system", "content": "You are a sassy, wise-cracking robot as imagined by Hollywood circa 1986."}, {"role": "user", "content": prompt}]

    # Load the tokenizer locally, apply chat template outside of TGI
    tokenizer = AutoTokenizer.from_pretrained('your_model_of_choice')
    prompt = tokenizer.apply_chat_template(prompt, tokenize=False)

    # call standard generate route
    return openai.completions.create(prompt=prompt)

Using the tokenizer locally is pretty fast computationally so I wouldn't worry.
If these examples don't help, if you could share more details about what you're trying to achieve I'll try and help :)


[Feature] Support Secondary model mentioned in disco-diffusion

What is the problem this feature will solve?
Support seconary model mentioned in disco-diffusion https://github.com/alembics/disco-diffusion/blob/main/disco.py#L1760.
Disco-Diffusion is supporting in this PR https://github.com/open-mmlab/mmediting/pull/1234.
What is the feature?
Based on Disco-Diffusion is supporting in this PR https://github.com/open-mmlab/mmediting/pull/1234, make a PR to offer secondary model for disco.
This is basically a model used to save memory and time for gradient computing.
What alternatives have you considered?
No response

Fixed in https://github.com/open-mmlab/mmediting/pull/1368


XIVY-6662 Add TextEditor sample

Really good test hard to implement. Because I only can set and get value of text editor with Javascript. So I only open the page. This already has failed before, when you did not disable secure flag.
I won't merge this 8.0. It's not worth.


Parity in concurrent between jasmine and jest-jasmine

🐛 Bug Report
There seems to be a parity between jest-jasmine and jest-circus on dealing with test.concurrent
For the below example
jest-jasmine throws
Jest: concurrent test "should test 2 + 1" must return a Promise

whereas jest-circus passes
The examples I've found for test.concurrent mention that the test should return a Promise
https://github.com/facebook/jest/issues/4281#issuecomment-324087872
To Reproduce
test.concurrent("should test 2 + 1", () => {
  expect(2 + 1).toEqual(3);
})

Expected behavior
Consistent behavior between jest-jasmine and jest-circus
envinfo

System:
    OS: macOS Mojave 10.14.6
    CPU: (12) x64 Intel(R) Core(TM) i7-8750H CPU @ 2.20GHz
  Binaries:
    Node: 10.16.0
    Yarn: 1.13.0
    npm: 6.9.0
  npmPackages:
    jest: ^24.9.0 => 24.9.0

cc \ @SimenB Noticed this while I was working on #8985

Does it work just making the test async? If so, we just need to do Promise.resolve on the test function and remove the warning

Yes, making it async would make it pass in jest-jasmine. Looks rather straight forward to add a Promise.resolve
https://github.com/facebook/jest/blob/77c17bc1dd044a8fe32d902376d9dd97ee390d8a/packages/jest-jasmine2/src/jasmineAsyncInstall.ts#L160-L165

I'll raise a PR for this.

Yeah, no reason not to. I think the point is that, since js is js, if your tests isn't async marking it concurrent makes no sense. Not sure what we should do about the done callback, though


Execute hypotheses on workflow changes

A hypothesis should be executed when the line of inquiry that triggers changes the workflow. This can happen in two cases:

Workflow changes in WINGS internally (We need some kind of version number for this)
LOI changes workflow bindings. In this case we should archive the old LOI and create a new one with new executions.

requires https://github.com/KnowledgeCaptureAndDiscovery/wings/issues/146


Update for 1.0 - Copy DatabaseMail output to be an object of status

Update for 1.0 - fixing output to be pscustomobject
Changes proposed in this pull request:

Added pscustomobject of each item being copied and output that object
Changed all write-output to be verbose output utilizing messaging system.

How to test this code:
On Source Server

[ ] Change the system settings for DBMail to other values (e.g Set "Account Retry Deaily" to 75 and "Maximum File Size" to 100)
[ ] Create a DBMail profile
[ ] Create a DBMail Account (minimum just name, email and server/port)

Run command:
$results = Copy-SqlDatabaseMail -Source Server -Destination Server -Verbose

On Destination Server

[ ] Confirm System Configurations were updated
[ ] Confirm profile was created
[ ] Confirm account was created

General verification of output

[ ] Verify each item has expected status (should be an object for each item copied in the command)
[ ] Run the command again without use of -Force and verify the "Skipped" status shows as expected for those objects receiving warning.

Has been tested on minimum requirements:

[ ]  Powershell 3
[ ]  Windows 7
[ ]  SQL Server 2000

Has been tested on maximum requirements:

[ ]  SQL Server vNext
[ ]  Windows 10
[ ]  Azure Database

Looks soooo damn good! can you show others the pic you sent me or no? if no, ill take one.

Looks great! The only thing I changed - within error messages, i like $sourceserver but i changed it to .name in the output. i also removed the end, it was an unnecessary practice i picked up because i could.


Problems with building the source code on Windows

Hey,
I'm working with material-ui in a project and I have found few bugs that I would like to fix myself so I made my own fork. I thought using my own fork would be as simple as changing in my projects package.json dependency to "material-ui" : "corvuscornix/material-ui" but it turned out that I need to build the code myself (and commit my compiled code before the library can be used?). I'm having tons of problems with building the code on windows. I get i.e. following errors:

"The specified path, file name, or both are too long. The fully qualified file name must be less than 260 characters, and the directory name must be less than 248 characters."

I put the cloned repo into C:\material_ui and was able to get over of this problem

"'rm' is not recognized as an internal or external command, operable program or batch file."

I changed rm to rd in package.json to get over this problem

"c1 : fatal error C1083: Cannot open source file: '..............................\Program Files\iojs\node_modules\npm\node_modules\node-gyp\src\win_delay_load_hook.c': No such file or directory [C:\temp\material-ui\node_modules\karma\node_modules\socket.io\node_modules\socket.io-client\node_modules\engine.io-client\node_modules\ws\node_modules\utf-8-validate\build\validation.vcxproj]"

I'm using iojs, is that a problem? I'm using cmd as Administrator so it cant be access right problem. I got over this problem just running the npm install a second time so it actually compiled the js files without giving this error.

And finally I get this error  when I try to run the dev server after compiling the code succesfully and manually copying the /lib to my projects node_modules/material-ui. As mentioned above, I'm using cmd as Administrator:
npm ERR! path C:\material_ui_upgrade\node_modules\material-ui\node_modules\karma
\node_modules\lodash\lang\isError.js
npm ERR! code EPERM
npm ERR! errno -4048
npm ERR! syscall unlink
npm ERR! Error: EPERM: operation not permitted, unlink 'C:\material_ui_upgrade\n
ode_modules\material-ui\node_modules\karma\node_modules\lodash\lang\isError.js'
npm ERR!     at Error (native)
npm ERR!  { [Error: EPERM: operation not permitted, unlink 'C:\material_ui_upgra
de\node_modules\material-ui\node_modules\karma\node_modules\lodash\lang\isError.
js']
npm ERR!   errno: -4048,
npm ERR!   code: 'EPERM',
npm ERR!   syscall: 'unlink',
npm ERR!   path: 'C:\material_ui_upgrade\node_modules\material-ui\node_modul
es\karma\node_modules\lodash\lang\isError.js' }
npm ERR!
npm ERR! Please try running this command again as root/Administrator.
The question is: How are you supposed to develop on windows machine?

please retry with npm version 3+ this might be fixed. if it still persists feel free to reopen, thank you.


Support custom header for getting request hostname

Fixes #13

Codecov Report

Merging #14 into master will decrease coverage by 1.4%.
The diff coverage is 83.33%.

@@            Coverage Diff             @@
##           master      #14      +/-   ##
==========================================
- Coverage   88.89%   87.48%   -1.41%
==========================================
  Files          97       96       -1
  Lines        1171     1079      -92
  Branches      143      134       -9
==========================================
- Hits         1041      944      -97
- Misses        109      113       +4
- Partials       21       22       +1

Impacted Files
Coverage Δ

static-server/src/server.ts
0% <0%> (ø)
:arrow_up:

static-server/src/getApp.ts
100% <100%> (ø)
:arrow_up:

static-server/src/config.ts
100% <100%> (ø)
:arrow_up:

static-server/src/getStaticRoute.ts
94.54% <80%> (ø)

static-server/src/healthChecks/storage.ts
42.85% <0%> (-57.15%)
:arrow_down:

cli/src/commands/create-bundle.ts
75% <0%> (-25%)
:arrow_down:

sdk/src/parseDates.ts
80% <0%> (-20%)
:arrow_down:

api-server/src/middleware/authenticateRequest.ts
100% <0%> (ø)
:arrow_up:

api-server/src/api/entrypoints/get.ts
100% <0%> (ø)
:arrow_up:

...i-server/src/api/entrypoints/{entrypointId}/get.ts
100% <0%> (ø)
:arrow_up:

... and 25 more

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 0669600...0dab0e1. Read the comment docs.


Unexpected Container layout behavior

Container(
  height: 300.0,
  width: 300.0,
  color: Colors.red,
  child: Container(
    height: 100.0,
    width: 100.0,
    color: Colors.blue,
  ),
),

unexpectedly ends up making 2 overlapping 300x300 boxes

You can workaround this use Column like this:
Column(children: [
      Container(
          height: 300.0,
          width: 300.0,
          color: Colors.red,
          child: Column(children: [
            Container(height: 100.0, width: 100.0, color: Colors.blue)
          ])
    )
]);

https://docs.flutter.io/flutter/widgets/Container-class.html

If the widget has no child and no alignment, but a height, width, or constraints are provided, then the Container tries to be as small as possible given the combination of those constraints and the parent's constraints.

width and height set the size of the child, yeah. Looks like it's working as intended to me.

                  Container(
                      width: double.maxFinite,
                      margin: EdgeInsets.fromLTRB(0, 16, 0, 0),
                      color: Color(0x88FFFFFF),
                      height: 2,
                      child: Row(children: [
                        Container(
                            color: Color(0xFFFFE100), height: 2, width: 200),
                      ]))

works, but it smells……

The key here is that the alignment argument defaults to "apply the constraints (which can be set using width and height) to the child". If you set the alignment to a non-null value, then the child will be aligned within the constraints instead.
To put it another way, what would you want the red/green/blue example above to actually look like? Where should the children end up, if they're a different size than the parent? You haven't set an alignment, so the Container widget doesn't know what should happen...

The key here is that the alignment argument defaults to "apply the constraints (which can be set using width and height) to the child". If you set the alignment to a non-null value, then the child will be aligned within the constraints instead.
To put it another way, what would you want the red/green/blue example above to actually look like? Where should the children end up, if they're a different size than the parent? You haven't set an alignment, so the Container widget doesn't know what should happen...

Ah hah. Alignment does make sense totally. I believe this point and a sample should end up in documentation. Maybe two samples with and without an Align widget could be added to docs to help clarify issue for first time explorers.

Might make sense to have a Troubleshooting question in the API docs that says something similar to my comment above, much like we do for Row and Column.

I just wanna say thank you for clarification :smile:
with your explanation, the code will be like:
Container(
  width: 300.0,
  height: 300.0,
  alignment: Alignment.center,
  color: Colors.red,
  child: Container(
    color: Colors.green,
    alignment: Alignment.center,
    height: 100,
    width: 100,
    child: Container(
      height: 50,
      width: 50,
      color: Colors.blue,
    ),
  ),
);

and works as expected

I couldn't find the explanation or example related to this issue in the current Container documentation.
https://api.flutter.dev/flutter/widgets/Container-class.html

Hi @xster
Does those comments in the docs clarifies the expected behavior? I think it explains what @Hixie  said in https://github.com/flutter/flutter/issues/19289#issuecomment-465231292.

Hi @xster
Does those comments in the docs clarifies the expected behavior? I think it explains what @Hixie  said in https://github.com/flutter/flutter/issues/19289#issuecomment-465231292.

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact description of what happens, logs, and the output of flutter doctor -v.
All system setups can be slightly different, so it's always better to open new issues and reference related issues.


A working RL agent that uses mel spectrogram.

The Mel spectrogram represented another approach that required less output  compared to our previous version. However, the "default" method for generating a Mel spectrogram proved to be very slow. This was primarily due to the lack of algorithm parallelization and the choice of variables used.
The agent is given control over parameters such as n_fft and n_mel. Note, action_magnitude  now has an effect on memory, I've tried t9 cap these parameters so it won't go above 12 gigabytes. The hop length, however, is fixed at 16, a balance between processing speed and resolution.
Alternative  were also explored, like Torchaudio. Unfortunately, these attempts were unsuccessful. While they might have been more effective with GPU support, their output was challenging to work with given the current setup.

An issue with our current step was discovered while attempting this. See the new step in mel environment, where it returns the correct audio.


Image Slider for homepage

Assign this issue to me @Sujal-2820

Hey, @Sujal-2820 this code is for manual only, I want to update the code for automatic. Should I submit another PR?


Use RxJS instead of promises

Any chance you'd mind if I make a PR to switch the interceptor to using RxJS methods instead of switching the Observable to a promise?

Feel free!

It would be great for this PR to be included in a release.
I'm facing the same issue

@dsebastien fixed in 1.0.4


Adding Logo

Adding logo to Pacify Website.
Do assign this issue under gssoc'22

Hi! I would like to work on this issue under GSSOC`22. Please assign it to me.

I am a gssoc'22 participant can you please assign me this issue?

I am a participant of GSSoC'22 . I am interested to resolve this issue . Could you please assign it to me?


Curve25519 Implementation

Hi!
I am trying to implement the Bouncy castle for my c# project.
I am stuck at implementing Curve25519 in EllipticCurveExtension.
trying to access https://www.ja3er.com/json && https://ezdiscord.xyz/fingerprint
I am using the following ciphers
                   CipherSuite.TLS_AES_128_GCM_SHA256,                     CipherSuite.TLS_AES_256_GCM_SHA384,                     CipherSuite.TLS_CHACHA20_POLY1305_SHA256,                     CipherSuite.TLS_ECDHE_ECDSA_WITH_AES_128_GCM_SHA256,                     CipherSuite.TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256,                     CipherSuite.TLS_ECDHE_ECDSA_WITH_AES_256_GCM_SHA384,                     CipherSuite.TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384,                     CipherSuite.DRAFT_TLS_ECDHE_ECDSA_WITH_CHACHA20_POLY1305_SHA256,                     CipherSuite.DRAFT_TLS_ECDHE_RSA_WITH_CHACHA20_POLY1305_SHA256,                     CipherSuite.TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA,                     CipherSuite.TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA,                     CipherSuite.TLS_RSA_WITH_AES_128_GCM_SHA256,                     CipherSuite.TLS_RSA_WITH_AES_256_GCM_SHA384,                     CipherSuite.TLS_RSA_WITH_AES_128_CBC_SHA,                     CipherSuite.TLS_RSA_WITH_AES_256_CBC_SHA
I am trying to implement chrome ja3 signature
The current ja3 signature I have is:
771,4865-4866-4867-49195-49199-49196-49200-52393-52392-49171-49172-156-157-47-53,0-23-65281-10-11-35-16-5-13-18-51-45-43-27-21-41,23-24,0
as and when I add Curve25519 to EllipticCurvesExtension I get Illegal_argument exception.
I have added the following code  as mentioned in the issue => https://github.com/bcgit/bc-java/issues/251:
`        static BigInteger P = BigInteger.One.ShiftLeft(255).Subtract(BigInteger.ValueOf(19));
static BigInteger D = BigInteger.ValueOf(3).ModInverse(P).Multiply(BigInteger.ValueOf(486662)).Mod(P);
    static byte[] convertInput(byte[] montPubKey)
    {
        BigInteger Xm = new BigInteger(1, Arrays.Reverse(montPubKey));
        BigInteger Xw = Xm.Add(D).Mod(P);

        byte[] weierPubKey = BigIntegers.AsUnsignedByteArray(33, Xw);
        weierPubKey[0] = 0x02;

        return weierPubKey;
    }`

and in DeserializeECPoint I added the following code in the beginning:
if(encoding.Length == 32)             {                 byte[] _encoding = convertInput(encoding);                 return curve.DecodePoint(_encoding);             }
after this I am getting handshake_failure exception
Can you help me on this?

We have recently ported the latest Java TLS support to C#, which includes TLS 1.3 and in particular for your case, X25519 support. It will be released in a few weeks, but in the meantime you could get the latest source code and build it yourself. The new implementation is in Org.BouncyCastle.Tls.

following the issue #317 resolved the session ticket issue. TLS 1.3 still gives Handshake Issue

Thank you for the response.

If you still have code doing anything with EllipticCurvesExtension as you described in the first post here, definitely remove it
This is already removed when I switched from Org.BouncyCastle.Crypto.Tls to Org.BouncyCastle.Tls.

X25519 is already enabled by default.
X25519 is 100% OK

Tls1.3 is working. But since I was testing it on a site [https://ezdiscord.xyz/fingerprint] where Tls1.3 isn't working using Org.BouncyCastle.Tls. Tls1.3 is working on google and other sites too.


feat: highlight config and drop card treatment

Description
Highlight config and changes from #351

Review Error for AshishYelekar @ 2023-08-21 21:36:46 UTC
User failed mfa authentication, public email is not set on your github profile. see go/mfa-help


Create martin-luengas.md

Hey Martin 👋🏾  thanks for submitting your profile. Was wondering if I could replace your current image URL with your Twitter bio URL as the profile would work best with a 1:1 ratio image?
In case you have a different link for the image with a 1:1 ratio, please feel free to update the PR too .
Also, have you been working on any Apps Script projects? I couldn't find any from your public profiles 😅
Would love to be able to follow and share with the community too.

Hey Martin 👋🏾  thanks for submitting your profile. Was wondering if I could replace your current image URL with your Twitter bio URL as the profile would work best with a 1:1 ratio image?
In case you have a different link for the image with a 1:1 ratio, please feel free to update the PR too .
Also, have you been working on any Apps Script projects? I couldn't find any from your public profiles 😅
Would love to be able to follow and share with the community too.

Hey Martin 👋🏾 thanks for submitting your profile. Was wondering if I could replace your current image URL with your Twitter bio URL as the profile would work best with a 1:1 ratio image?
In case you have a different link for the image with a 1:1 ratio, please feel free to update the PR too .
Also, have you been working on any Apps Script projects? I couldn't find any from your public profiles 😅
Would love to be able to follow and share with the community too.

Hello Sourabh!
I've just changed the image. I'm sorry about that.
Yes, I have been working in a couple of small projects. I'm starting my journey in Apps Script so I haven't shared any of them yet.
I'm currently working in improving the system I created to admit students in our school building. Due to the pandemic, families have to fill out a sworn statement saying that their kids don't have any COVID-19 symptoms. That makes the drop off very time consuming (we have to check they filled out that form, check their body temperature and apply hand sanitizer).
So, I developed a small Apps Script solution that links our school management system (gets a csv file with all students' health screening answers), checks the validity of the submission, and send notifications (via email and WhatsApp-Twilio) to parents who haven't submitted the form.
I haven't published any of my projects and if that's a requisite to be listed I totally understand.
Have a nice day!

Hey Martin 👋🏾 thanks for submitting your profile. Was wondering if I could replace your current image URL with your Twitter bio URL as the profile would work best with a 1:1 ratio image?
In case you have a different link for the image with a 1:1 ratio, please feel free to update the PR too .
Also, have you been working on any Apps Script projects? I couldn't find any from your public profiles 😅
Would love to be able to follow and share with the community too.

Hello Sourabh!
I've just changed the image. I'm sorry about that.
Yes, I have been working in a couple of small projects. I'm starting my journey in Apps Script so I haven't shared any of them yet.
I'm currently working in improving the system I created to admit students in our school building. Due to the pandemic, families have to fill out a sworn statement saying that their kids don't have any COVID-19 symptoms. That makes the drop off very time consuming (we have to check they filled out that form, check their body temperature and apply hand sanitizer).
So, I developed a small Apps Script solution that links our school management system (gets a csv file with all students' health screening answers), checks the validity of the submission, and send notifications (via email and WhatsApp-Twilio) to parents who haven't submitted the form.
I haven't published any of my projects and if that's a requisite to be listed I totally understand.
Have a nice day!

Sounds like a pretty efficient project — way to go 👏🏽
And about publishing, not really. It just makes it easier for other folks to infer towards what could they engage you with 😊
PR looks good now, will merge!

Sounds like a pretty efficient project — way to go 👏🏽
And about publishing, not really. It just makes it easier for other folks to infer towards what could they engage you with 😊
PR looks good now, will merge!


Package.loadSettings is deprecated.

Store package settings files in the settings/ directory instead of scoped-properties/
Package.loadSettings (/usr/share/atom/resources/app/src/package.js:547:9)
<unknown> (/usr/share/atom/resources/app/src/package.js:168:43)

Duplicate


Mark AbuseIPDB as FIPS Compliant in app metadata

Mark the app as fips_compliant

please add static failure to PAPP-21824


style: fix rfc typos

Signed-off-by: Aylei<EMAIL_ADDRESS>What type of PR is this?

[ ] API-change
[ ] BUG
[ ] Improvement
[x] Documentation
[ ] Feature
[ ] Test and CI
[ ] Code Refactoring

Which issue(s) this PR fixes:
Fixes some typos found in doc.

/merge


Black hole controller deleting items

I built and placed 4 controllers and filled them with BHU's. I ten set about emptying the TF Cache's I'd been using into them. I went and did other stuff and came back to find out that 6 weeks of mined/processed/mob farmed items had been deleted. Over 500k items all told. I suspeect this has something to do with the issue I've now seen with the controller replacing whatever is in the first BHU with whatever is being piped in next.
Industrial Foregoing Version: 1.12.2-1.12.8-232
TeslaCoreLib Version: 1.12.2-1.0.15
Crashlog (If Any):

:wave: Sorry but Industrial Foregoing has ended support for version 1.12. I'm closing this issue as it will no longer be updated for that version. Thanks for playing with the mod.


Use oauth

Use oauth rather than basic auth

Oh never mind, basic auth headers are encrypted in https so we can roll with that.


Bug: Duplicate shebang lines in v3.1.0

Versions

Node: 18.16.0
OS: W11 WSL2 - Ubuntu 22.04.3 LTS

Reproduction

Additional Details
When updating to typeorm-extension 3.1.0 and using the CLI commands from `node_modules/typeorm-extension/bin/cli.cjs`, i get the exception:
node_modules/typeorm-extension/bin/cli.cjs:2
#!/usr/bin/env node
^

SyntaxError: Invalid or unexpected token

I can see in the cli.cjs file that the shebang line is duplicated. The beginning of the file looks like this:
#!/usr/bin/env node
#!/usr/bin/env node
'use strict';

require('reflect-metadata');
...

Steps to reproduce

install typeorm-extension v3.1.0
run a cli command like ts-node --project ./tsconfig.json ../node_modules/typeorm-extension/bin/cli.cjs seed:run -d <PATH TO DATASOURCE>

What is Expected?
The command will execute successfully
What is actually happening?
An exception is thrown because the shebang characters (#!) is only allowed on the first line.

good catch, thanks!
fixed in v3.1.1


New version of RMCob/homebridge-wyze-connected-home-op published. V0.5.6

I have updated RMCob/homebridge-wyze-connected-home-op to incorporate the fixes for the 403 error and Color Bulb functionality supplied by others. I do not have a Color Bulb to verify those changes but the 403 error is definitely fixed (until something else changes :-))

I am using your implementation and it is working on my local network.  However, whenever my iPhone is away from home and connects to different network, your homebridge plugin crashes and I have to do a complete reinstall of my Wyze homebridge instance.  I probably have an issue with my network setup but nevertheless I don't think such a catastrophic crash should occur! Any suggestions?
Steve
homebridge.log.txt

Here is the log after the plugin crashes.
homebridge.log (1).txt

Steve, I looked at your log file. I am not familiar with running Homebridge on Windows so I really can't help much. The paths shown in the log file that have the component 'Roaming'. Is there a non-Roaming version? Also you said you had to do a complete re-install of the Homebridge instance. From the log file it looks like it just re-started. Is that what you meant? Perhaps someone else running Homebridge under Windows could chime in?

Steve, in your second log file, in the beginning are the lines:
[8/11/2021, 2:18:43 PM] [Wyze] Request failed: Error: getaddrinfo ENOTFOUND api.wyzecam.com
[8/11/2021, 2:18:43 PM] [Wyze] Error, refreshing access token and trying again
[8/11/2021, 2:48:00 PM] Error: send EHOSTUNREACH <IP_ADDRESS>:5353
at doSend (dgram.js:692:16)
at defaultTriggerAsyncIdScope (internal/async_hooks.js:429:12)
at afterDns (dgram.js:638:5)
at processTicksAndRejections (internal/process/task_queues.js:81:21)
[8/11/2021, 2:48:00 PM] [HB Supervisor] Homebridge Process Ended. Code: 1, Signal: null
[8/11/2021, 2:48:05 PM] [HB Supervisor] Restarting Homebridge...
You definitely have a network issue there. The plugin at first connected to wyze.com and got a list of all your devices. Then when it restarted it couldn't connect to wyze.com to get the device list so all the previous devices were 'orphaned' and therefor it tried to remove them from Homebridge. I can see what happened but I don't know what to tell you to do to fix the problem in Windows.... :-(

So I left home and when I came back, the plugin had crashed.  The second logfile is what appears when it has crashed.  To recover, i must now uninstall and reinstall my homebridge instance and reinstall all the devices.

I have 2 other instances of homebridge running with a total of 87 non "wyze" devices.  These instances are rock solid for the past year.  The wyze plugin is on the same network and configured pretty much the same.  What is the "access token"?

RM, I have RMCob/homebridge-wyze-connected-home-op  plugin running in a separate Homebridge instance named homebridge-four on port 8584.  All the wyze specific device information is stored in a windows directory  named .homebridge-four  the access token is in the "persist" sub directory.  It appears the plugin is not looking in the right file location when it restarts!
Steve

A power outage forced me to reinstall HomeBridge and the Wyze plugin.  I got everything working and all was good for about 4 days. Then the Wyze connected home RMCob plugin crashed.  Very frustrating!
Is anyone using the Wyze connected home plugin successfully?


ECS : python controller bindings

bindings for running an ECS instance via the ECS controller

working
bindings:
https://github.com/tweakoz/orkid/tree/develop/ork.ecs/pyext
examples:
https://github.com/tweakoz/orkid/tree/develop/ork.ecs/examples/python


Why am I having trouble with setting initialState to an object?

I used this example to setup my extension using react-chrome-redux: https://github.com/tshaddix/react-chrome-redux-examples
Currently in the countReducer, it has:
const initialState = 0;
Everything works fine.
But if I update that to
const initialState = {       counter: 0 };
Then update mapdispatchtoProps from
count: state.count
to
count: state.count.counter
I get an error that says can't find property counter of undefined.
If I setup the initial state as values (not objects), everything works fine.
What am I not understanding?
Thanks.

@missioeric you want to map state instead of actions. Meaning that you should update mapStateToProps instead of mapDispatchToProps.

@vanbosse
Thanks so much. Shows you where my mind is.
I am actually updating mapStateToProps even though I wrote dispatch above. Sorry about that.
Any other ideas on why making the simple change above in my count reducer:
going from const initialState = 0; to const initialState = { counter: 0 };
and then changing in mapStateToProps from count: state.count to count: state.count.counter would return can't find property counter of undefined?

@missioeric do you have a minimal example online somewhere I can take a look?

@vhmth Yes, you can see my code here: https://gist.github.com/missioeric/c5bda70c1dfa4676a73790faa10c4a79
Thanks for the help.

@missioeric if that's your root-level reducer, you will need to reference it as state.counter - not state.count.counter. Right now your store looks like this:
{
  counter: 0
}

So state.count will return undefined.

@vhmth Thank you. That did it. I was misunderstanding how the store looked. Thanks for taking the time to look at things for me.

No problem! Glad you got on alright! :)

Actually,
that didn't end up solving things for me.
But, I found this: https://github.com/tshaddix/react-chrome-redux/issues/5
and realized it was the exact issue I was having.
So, for future readers, if you find that you can't access nested properties in the state, use the fix above (also here under Initializing UI Components: https://github.com/tshaddix/react-chrome-redux/wiki/Advanced-Usage )
Thanks for your help @vhmth
Wouldn't have figured things out without your help along the way.

Hmm @tshaddix this makes me think that we really need to be explicit about this initial connection race condition. Enough people have brought it up that it may warrant an unconventional API (unconventional to redux that is), such as:
store.ready(() => {
  // do stuff
});

Thoughts @tshaddix? I know you're weary about straying too far from the Redux API, but this is kinda the main race that most people will run into when using the store, and the store isn't just ready to use in the content scripts until it's connected to. I believe this is a good candidate for a more explicit API (which seems necessary due to the nature of the Chrome extension dev environment itself).

@vhmth I think that's fair! As long as we maintain the existing APIs to be up-to-standards with Redux, additional utility methods are probably just fine. I think this is a good suggestion.
We should also make sure to add something front and center in the README :)


its not working

This project wasn't worked on in quite a while but I could maybe look into it if you could give some information on what exactly is not working.


Curve staking

adds curve staking section to stake page
makes it possible to hide ogn staking section by clicking
makes it possible to show/hide OGN/Curve staking by environmental variables
adds brownie project to validate Curve gauge APY values

This has been opened as a draft for some time, now it is ready to be reviewed @shahthepro

@shahthepro can you stamp this one please if there is nothing else to address?

@shahthepro gracias :)


Typo "@credential"→"*@credential**"

Italic with escape characters
https://docs.microsoft.com/en-us/sql/relational-databases/system-stored-procedures/sys-sp-rda-reauthorize-db-transact-sql?view=sql-server-2017

@hyoshioka0128 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Corrections confirmed. Thanks.


Resolvers error messages should include the version requirements.

Currently, if the resolver fails the error messages give lots of information on the paths that caused the problems, like:
https://github.com/rust-lang/cargo/blob/0b530c30867da26a4b59146f490c9f1d5377c20a/tests/testsuite/build.rs#L1356-L1358
It would be nice to have the version requirements like: (wording needs improvement)
  previously selected package `bad v1.0.0`
    ... which is depended on by `bar v0.1.0` which requires `bad: "=1.0.0"`
    ... which is depended on by `incompatible_dependencies v0.0.1 ([..])` which requires `bar: "0.1.0"`

The data is easily available thanks to #5428 we just need to get it into the correct place, and find good wording. We almost had this working in #5452 before it bit-roted.
I am willing to mentor someone on getting this done. It may be a good place to start looking at the resolver without having to stare into the void at its core. If it would be useful I can rebase #5452 so that only the wording is left.
cc @necaris would you be interested?

Do to the github outage this got opened several times. Sorry for the spam.

@Eh2406 I can work on this

Wonderful! How can I help you get started? What are your opening questions? Should I try and rebase/rewrite or do you want to? Do you want to schedule a time to chat on discord or hangouts?

@Eh2406 will ping you on discord

The #5452 is not seeing it for some reason, but the two logick commits have been rebased on eh2406/master and pass the tests localy for me.

@Dylan-DPC Did you get a chance to work on this? How is it going? Is there anything we can do to help?

@Eh2406 Hey. Got busy with some stuff. Will work on it this week. Will ping on discord if I need anything :)

@Eh2406 I see that a couple of pull requests were opened for this but never merged. I've got some time and would love to get involved if code still needs to be written.

Help is welcome! The pull requests have bit-roteded, but I don't thing there is much to do to bring them up to date. The harder part was suggesting a better wording.

Is anyone still working on this? If not I can take a stab at it!

I believe it is open.

Is this the appropriate place to ask questions while I work on this? I don't have much experience contributing to projects other my own.

@zmitchell don't see why not put things here

Here works grate!

I'm trying to understand the pre-bitrot way of doing things so that I can understand what changes were made in the PR. After that the plan is to understand how things work post-bitrot so I know how to translate the changes in the PR.
First question: What does the links field on Manifest represent? It's just an Option<String>, which doesn't tell me much.

Good plan!
The links holds the value of the "links field" https://doc.rust-lang.org/cargo/reference/manifest.html#the-links-field-optional. There will be only one crate in the dependency tree with each links value. This is primarily used to avoid linking errors.

I'm still trying to build a mental model of the different pieces that fit together here.

A Context has a Resolve
A Resolve has a Graph<N, E>
The Graph<N, E> is really a HashMap<N, HashMap<N, E>>
In the context of Resolve this is actually HashMap<PackageId, HashMap<PackageId, Vec<Dependency>>>

So, here are my questions:

Conceptually, what is the difference between a Dependency and a package referred to by a PackageId?
How do I interpret the Graph structure? (I'm familiar with graphs with nodes and edges, I mean specifically this implementation):

If I iterate over the nodes in Graph, I'm given a tuple that looks like this: (node, edges) = (PackageId, HashMap<PackageId, Vec<Dependency>>).
Are the keys in that HashMap the PackageIds of packages that can be reached from node?

I have a couple more questions, but I'll wait until I know what a Dependency is before asking.

The mental model seems almost right. One quip is that A Context is more a work in progress version of a Resolve.

Conceptually, what is the difference between a Dependency and a package referred to by a PackageId?

A Dependency conceptually is a statement in some Cargo.toml like serde='^1.0.69'.
A PackageId conceptually is a unique version of a crate like https://crates.io/crates/serde/1.0.99 .
A Package can satisfy a Dependency, like how we can pick serde v1.0.99 if your Cargo.toml asked for serde='^1.0.69'.
Similarly, the Summary (The part of the Cargo.toml that gets stored in the index) associated with a PackageId can have a list of Dependencies that will need to be satisfied if we use that Package.

How do I interpret the Graph structure? (I'm familiar with graphs with nodes and edges, I mean specifically this implementation):

It is a directed graph. Also to confuse things Context.parents and Resolve.graph think of the edges as going in opposite directions. So from Context.parents perspective (as that is how activation_error access it at this time), an edge goes from a child to the parent that asked for it and it keeps track of the Dependencys the child satisfied.
{
    'serde_derive v1.0.99': {
        'serde v1.0.99':  ['Dev-Dependencies serde="^1.0"', 'Dependencies serde="^1.0"'],
    },
    'quote v1.0.1': {
        'serde_derive v1.0.99':  ['Dependencies quote="^1.0"',],
    },
}

Witch can be read as:

serde_derive v1.0.99 was selected to

satisfy serde v1.0.99 Dev-Dependence  that looked like serde="^1.0" and also was selected to
satisfy serde v1.0.99 normal Dependence that looked like serde="^1.0" and

quote v1.0.1 was selected to satisfy serde_derive v1.0.99 normal Dependence  that looked like quote="^1.0".

I don't think I have made sense, so keep asking! If I ever manage to make sence, please add some comments in your PR to make it easier for the next user!

Thank you, I definitely plan to add some comments to make this all clear for the next complete n00b. I think most of that makes sense, but I think a better way to phrase my question would have been "how do I navigate Graph" rather than "how do I interpret Graph".
Let's say I'm looking at a particular node, pkg.

What information does self.nodes[pkg] (a HashMap<PackageId, Vec<Dependency>>) contain?
What do the keys in that HashMap represent?
How do I obtain the nodes adjacent to pkg (below, actually, since this is a directed graph)?

So from Context.parents perspective, you are looking at pkg so self.nodes[pkg] is all the ways pkg is needed. So the keys of the HashMap are the Packages that depend on pkg. If you continue down you will eventually get to the root package. See path_to_bottom for an example, that gets used in the describe_path part of the error messages. So if we get a self.nodes[pkg][parent] we will get a  &Vec<Dependency> is the ways that this parent depended on pkg

Did #7934 solved this issue ?

I dont think so. If a fails to load then #7934 will show the path foo->c->b->a like the errors discussed above. But it does not include what versions of b would be acceptable.


Simplified the code for secret migrations to test namespaces

@raudaviciusm Why does jq work in the kubectl container? Is it all ready there? Is it safe to assume it will continue to be there?

Done. I have tested and can confirm that jq works with ibmcom/k8s-kubectl:v1.7.6 image. I don't know where the Dockerfile is so it is hard to say why it works.


Add channel to mobile notifications

Checklist

[Done] I checked for similar existing requests (both open and closed) before posting.
[Done] My request is generic, other users may benefit from it too.

Proposal
Please add channel to mobile notifications. In the companion app we can arrange different sounds to channels. I created a channel in the companion app as "Alarm Triggered" and changed the sound to Alarm.mp3 in my Android phone. When a message arrives to that channel Alarm.mp3 played instead of default sound of HA Companion App Notification.
https://companion.home-assistant.io/docs/notifications/notifications-basic/#notification-channels
Additional info
I even tried adding channel in yaml in your Notification Editor but the syntax was rejected.

It would be good if you could share what you've tried in yaml mode.
What happens when you manually trigger the notify service with the yaml in the Home Assistant Developer Options?
For me (iOS) this is working fine:
service: notify.sander
data:
  message: "Het alarm is geactiveerd! Oorzaak: {{open_sensors|lang=nl}}."
  title: 🚨 Alarm
  data:
    url: /dashboard-status/alarm
    push:
      interruption-level: critical
      sound: alarm.caf


fix: Fix inability to switch ResponseMessage during generation when using Many Models Conversations

Pull Request Checklist

[x] Description: Fixed toggle ResponseMessage display logic
[ ] Changelog: Ensure a changelog entry following the format of Keep a Changelog is added at the bottom of the PR description.
[ ] Documentation: Have you updated relevant documentation Open WebUI Docs, or other documentation sources?
[ ] Dependencies: Are there any new dependencies? Have you updated the dependency versions in the documentation?
[x] Testing: It has been tested and launched in an internal project
[ ] Code Review: Have you self-reviewed your code and addressed any coding standard issues?

Description
When multiple models reply at the same time, switching between incomplete streaming replies is supported

Changelog Entry
Fixed

Fixed toggle ResponseMessage display logic #2035

Closing this but fixed on the issue on dev and added you as a co-author! Let me know if the issue persists!

Closing this but fixed on the issue on dev and added you as a co-author! Let me know if the issue persists!

Does this mean I can directly fix commit and push to the dev branch?
BTW I might need some guidance or documentation reference. I'm worried that my lack of understanding of the collaboration method might cause trouble for everyone.


[ml] 1.12.0 release api view fixes

Rename spark_conf to spark_configuration for FeatureSetBackfillRequest

Set ImageLimitSettings constructor return type to None

Remove kwargs from ImageObjectDetectionJob docstring

Fix job_ids type annotation and docstring for FeatureSetBackfillMetadata

Fix type annotation

Make constructor params keywork only

Remove experimental tag from Notification

Make constructor params keywork only

Description
Please add an informative description that covers that changes made by the pull request and link all relevant issues.
If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
All SDK Contribution checklist:

[ ] The pull request does not introduce [breaking changes]
[ ] CHANGELOG is updated for new features, bug fixes or other significant changes.
[ ] I have read the contribution guidelines.

General Guidelines and Best Practices

[ ] Title of the pull request is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-ai-ml


Reminder: after update to ESLint v6, dissalow 'allow require for configs' for nested configs

When this package updates to version 6 of ESLint, we must update our override for .config.js files to disallow that same override for nested .config.js files.
This override can be found here.
The update is probably going to be something like:
'files': './*.config.js'

Fixed in master.


Validate and document that cloud requires and https endpoint

Summary
When configuring webhooks, in Jira, the SITEURL must use https:// if connecting to a cloud instance.  This ticket has two requests.

Document in the README that cloud instances will require an https endpoint.
Add validation that will reject non-https URLS for jira cloud instances.  This will be handled in the code that is executed with /jira install cloud <URL>

Ticket Link
https://mattermost.atlassian.net/browse/MM-22593

If you're interested please comment here and come join our "Contributors" community channel on our daily build server, where you can discuss questions with community members and the Mattermost core team. For technical advice or questions, please  join our "Plugin: Jira" community channel.
New contributors please see our Developer's Guide and our Plugins Guide.

I would like to be assigned this issue.
Thanks.

@Francois-D, you are more than welcome to take this ticket.  Please let me know if you have any questions!


Add missing trans block in usage templates

The usage templates were updated in #9170 / #9279, but the "Field" column header was not translated.
While at it, dropped valign="top" from the entity type cell since we don't output anything else in there.

Before
After

Cherry-picked to 4.1.x in 6ec3ac99c29b4e0ef727831395a43e8ce8eb00e5 / d4f5648cdac5631abe2a2c41ce6f90908be7e6b2.


Speed Hacks

Here are some sneaky things you can do to get some speedups

[ ] use the fake stateTrie if you don't care about state roots
( you can still checkpoint and revert )
[x] skip proper tx signing by overriding the from: field

transaction.from = new Buffer(ethUtil.stripHexPrefix(txParams.from), 'hex')

Added to v0.2.0 milestone (next release)

Need to double check - do we care about state roots?
Also, skipping transaction signing still needs to happen for eth_calls. Added to ticket.

i think the testrpc api has a thing for checkpointing that relies on the state roots
the fakeTrie has support for checkpointing but only in the checkpoint/commit/revert sense, you cant just set a previous stateRoot ( bc all stateRoots are 0x00000...0000 )

Closing this one. I think this is all implemented now, both in the provider engine and the TestRPC. The fake statetrie could still be investigated, but I'd like to do that in another ticket if there's still interest.

I think you'll find a big speed up from the fake stateTrie, but as it is you are prolly not blocked on perf


Zsh Shell: Cannot check if install was successful?

Problem encountered on https://dotnet.microsoft.com/learn/dotnet/hello-world-tutorial/install
Operating System: macos
I'm trying to check if my install succeeded but when i type dotnet it says the command cannot be found.

Did you open a new terminal window?

And what version of Mac are you on?


[0.7] Correct uniform block and shader storage block layouts

I'm going to delay this change because I'm not sure of the actual layout thingy.

Should be good now.

So what this does is that it replaces the old clumbsy Vec<UniformBlockMember> with a BlockLayout struct. This struct can represent any possible layout in an SSBO or a uniform block.
The next step is to add a Layout trait or something similar that can be implemented on structs just like Vertex and that can match a BlockLayout.

I think I'm going to merge this now without bumping to 0.7.
This change only affects people who were explicitely referring to the UniformBlockMember struct, which in my estimates is approximately nobody.

Fix #955


Linear Algebra Tutorial 3 (W2D3)

Think 1: Identifying transformations
https://github.com/NeuromatchAcademy/precourse/tree/main//tutorials/W0D3_LinearAlgebra/solutions/W0D3_Tutorial2_Solution_a6de96a6.py

Updated


Userstamp is not added when userstamp was not explicitly called

I have this test:
RSpec.describe Product do
  it "chech bug" do
    class User
      include Mongoid::Document
      include Mongoid::Userstamp::User
    end

    class Product
      include Mongoid::Document
      include Mongoid::Userstamp
    end

    p = Product.create
    expect(p.created_by).to be eq(nil)
  end
end

This show this error:
  1) Product chech bug
     Failure/Error: p.created_by

     NoMethodError:
       undefined method `created_by' for #<Product _id: 572c8a3f8675c36e39000001, >

The problem that I can see is that this code:
module Mongoid
module Userstamp
  class Railtie < Rails::Railtie
    config.to_prepare do
      Mongoid::Userstamp.model_classes.each do |model_class|
        unless model_class.included_modules.include?(Mongoid::Userstamp::Model)
          model_class.send(:include, Mongoid::Userstamp::Model)
        end
      end
    end

Is executed before Mongoid::Userstamp is included in class:
module Mongoid
  module Userstamp
    extend ActiveSupport::Concern
    included do
      Mongoid::Userstamp.add_model_class(self)
    end
    ...

Then Mongoid::Userstamp.model_classes is equal to []
It is a bug?

Hey @rjurado01 I tried to set the user_model on the tracked models and that worker.
Just put mongoid_userstamp user_model: 'User' :)


Added April Offline Bundles

Proposed changes
@paigehargrave Ahh I couldn't push to your branch, so heres a PR :)

_config.yaml looks good, there was one DTR missing but I picked that one up.
Added the April offline bundles

Unreleased project version (optional)

Related issues (optional)

Deploy preview for docsdocker ready!
Built with commit d15aaa2e44b0dded515a139fffa741e10e56c4b0
https://deploy-preview-8621--docsdocker.netlify.com

@ollypom Thank you!


When downloading conda deps, subtract any line in a conda list explicit of the root env

Actually almost impossible to do, it seems.


Fix duplication when using custom workspaces

Hi taooceros,
I've noticed that you did some refactor for VSCodeWorkspace, which is nice. But, the refactor leads to some duplication issue.
When a custom uri that doesn't EXACTLY MATCH the uri VSC would generate is set, take C:/Foo/Bar as an example:

VSCodeWorkspace.Path for ou workspace object would be file:///C:/Foo/Bar
But the VSC generated workspace object would have file%3A///c%3A/Foo/Bar
And the workspaces.Distinct() call later fails to eliminate duplications

To get rid of this, I've created the PathString class following MSDN best practice for Path comparison, and replaced Path, RelativePath, FolderName type of VSCodeWorkspace record. Also, the Path property is set to unescaped uri instead of the original one.
Tested and works on my machine, no more duplications :)

Thanks for the pr! I am really sorry that I missed the message as I was busy at that moment (as well as now😥). The change is very nice and should help a lot to remove similar code for string comparison. Could you please do a version bump up in plugin.json so that the release pipeline can help to publish a new version?

Done, plugin version is now 1.2.2

Thanks


Where is the changelog for 7.1.1?

Where is the changelog for 7.1.1?

I'll create this today with more detail:

CJS support
dropped Graffle related code and deps (use pnpm add graffle@next to get it now)

https://github.com/graffle-js/graffle/releases/tag/7.1.1


Build site on Travis after Pull Requests merged

When building the site after commits to master (such as when a Pull
Request is merged) Travis will now commit and push any uncommitted
changes to the output directory.
This means that people need not commit their changes to the output
directory, which should:
a) clear up some confusion with the current system, and
b) allow the site to be edited via GitHub's web interface.
I have tested these changes at https://github.com/inglesp/deploy-test-2.

@inglesp – great work. This all looks sane to me.


Fixed Issue no #4101 : Requesting merge (no conflicts)

PR Description 📜
Fixes #4101
I have fixed the issue #4101 , in which I fixed three bugs of the game

Once you have completed the game and won, now if you keep pressing the guess button the no. of tries you used keeps increasing

But if you lost the game and lost all lives, now if you keep pressing the guess game , the Guessed numbers are: , keep adding more numbers

third if you reset the game , the no of lives already used do not get updated for new game instead its the same of previous game , once you start , then it get updated

What I did

Manupulated Dom to update screen in reset button so that the lives are reset properly
Added two if's to the Play function to check if user has won or lost , so that pressing the Guess button don't run the game's play function to the end.

Mark the task you have completed ✅

[x] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[X] I have specified the respective issue number for which I have requested the PR.
[X] I have added screenshots of the bug in this Pull Request for reference

Add your screenshots(Optional) 📸

Thank you soo much for contributing to our repository 💗

Thank you @mohitgedar ,for creating the PR and contributing to our GameZone 💗
Review team will review the PR and will reach out to you soon! 😇
Make sure that you have marked all the tasks that you are done with ✅.
Thank you for your patience! 😀

Thank you  @mohitgedar , for your valuable time and contribution in our GameZone 💗.
It’s our GameZone, so Let’s build this GameZone altogether !!🤝
Hoping to see you soon with another PR again 😇
Wishing you all the best for your journey into Open Source🚀


DP Proxy Pattern in JavaScript

Need to make a readme file which should have the
Definition
Explanation in simple terms
Code related explanation.

assign to me


Linking the seperate bashes into one kubectl clone.

linked yaml extraction into the clone file, waiting for the kubectl config file creation process


Networking: discovery per network

Discovery should only query and report nodes that are on the same network, identified by network ID in the chain spec.

duplicate https://github.com/paritytech/substrate/issues/1913


Material_AlphaMask_04 is not visible when working correctly

Ideally every alpha model is at least a little visible. Investigate why

Model has an alpha cutoff of 1.1, so nothing should be visible. having a cutoff of this value is good to have, but maybe we should mitigate this visibility issue somehow. Otherwise the model will cause a client to look broken when instead it is working correctly.

We're currently planning on addressing this by including the reference image. Maybe an additional comment in the readme as well.


Update table-expandable-rows-example.html

Fixed interpolation error.

I signed it!

Closing this PR since it seems to be abandoned


Are we missing "default" field requiredness?

I've generated a few thrift clients with this lib. Working good so far. But I have an instance where I've needed to modify the generated code. It's related to the "requiredness" of fields on a struct.
My example
Using this library, I change this Thrift struct
/**
 *  Audit response status object.
 *
 *  @param success. True if audit request completed successfully, otherwise set to false
 *  @param messages. List of any error or warning messages that may have occurred
 */
struct AuditResponse {
    1: string status,
    2: list<string> messages
}

...into this Go struct.
type AuditResponse struct {
	Status   string   `thrift:"1,required" json:"status"`
	Messages []string `thrift:"2,required" json:"messages"`
}

The service I'm communicating with often omits Messages, so I get an error when I call an endpoint that returns an AuditResponse, due to (I think) Messages being interpreted as required.
If I hack the generated code by changing the requiredness, an error no longer occurs.
type AuditResponse struct {
	Status   string   `thrift:"1,required" json:"status"`
	Messages []string `thrift:"2,optional" json:"messages"` // NOTE: Modified.
}

Should there be a "default" requiredness?
According to this documentation from Apache, there is a requiredness type called "default", which is implicit if fields in Thrift IDL are not explicitly defined as "required" or "optional".
I change the generated code from "required" to "optional", because I need the Read behavior of optional, which the "default" requiredness retains.
Is there a reason this is not implemented?

Ah, I see this as a TODO at the bottom of the README. Should have read to the bottom!


provide nix packages

Nix(os) is nice, let's provide myks for nix users as well.
Initial PR: https://github.com/NixOS/nixpkgs/pull/310785
Needs as well an update to myks (docs), e.g.
how to run:
nix-shell -p myks helm git

It's a good idea.
I'm not familiar (anymore) with nix, so I have questions:

How the version is supposed to be updated there?
Can we use this feature of GoReleaser: https://goreleaser.com/customization/nix/ ?

Hi,
Updates will happen automatically. There is a job running and updating the version and hashes automatically following GitHub releases. We just have to ensure that the build process is working.
I had a look at the goreleaser feature before. Unfortunately a nix user repo is not the same as being in the main channel.
I think the integration into the normal repo will happen soon.
BR Kris

Updates will happen automatically

That is all I need then :-)

@kbudde https://github.com/NixOS/nixpkgs/pull/310785 is merged 🥳
Could you provide a documentation snippet? Or is this literally all what nix users need?
nix-shell -p myks helm git

Fixed in NixOS/nixpkgs#310785 and #337


Adding [serde(rename_all = "camelCase")] to structs

Serde:

Skipping raw parts
Changing names to CamelCase

@soywod review pls.

This should fix the last two points of your list.

I added a project board: https://github.com/TornaxO7/himalaya/projects/2


Implement CachedGuard.AddRule, add flexibility to auth language.

Test AddEndorsements() and ValidatePeerAttestation() with a public, cached DatalogGuard to make sure the client adds endorsements to the policy properly. (@tmroeder: Tom, could you have a look?)
The principal types are hardcoded as tokens in the auth language. Currently the parser checks whether the principal is "tpm" or "key". This commit checks membership in a map to determine if a string is a principal token, which makes it easier to specify other types of principals. (@kevinawalsh: since I've modified the authentication language, I thought you should have a look. Thanks!)

I have no other concerns about this commit. Though I should point out that the verifier in tao only knows how to verify "key" and "tpm" type principals, so any new principal types added won't work for delegations at this point.

This looks good. Two minor requests, and I'll pull it:

rebase against master; this currently contains, e.g., the 'syntax = "proto2";' statements from #30
add a small test to auth_test.go to make sure you can actually parse and use an auth statement that has an arbitrary principal name (not 'key' or 'tpm')

Done.