Add workflow system info in Workflow Execution context

This PR adds workflow engine system information to WorkflowExecution context.
Ref https://github.com/StackStorm/st2/discussions/5373
{
    "engine_info": {
        "hostname": "st2",
        "pid": 1085262
    },
    "parent": {
        "user": "stanley",
        "pack": "examples"
    },
    "st2": {
        "action_execution_id": "6174ce479195da5bbddf5443",
        "api_url": "http://<IP_ADDRESS>:9101/v1",
        "runner": "orquesta",
        "workflow_execution_id": "6174ce48f3b5fca8fa795f97",
        "user": "stanley",
        "action": "examples.orquesta-sequential",
        "pack": "examples"
    }
}

This information will be used to identify orphaned workflow execution.

I won't have time to perform a detailed review, but just two quick comments / questions:

Performance implications

It appears that depending on the object passed to the function, we will either perform 1 or 2 additional GET one and 1 (2? I don't know why we need that delete at the end in finally) additional UPDATE query each time handle_workflow_execution and handle_action_execution is called.
A while back I spent a substantial amount of time of profiling and optimizing performance of that code path (which is considered a hot code path since it's executed often and has major performance implication on how long running / scheduling executions and workflows takes).
And per @m4dcoder coder, that code can be executed multiple times (each time execution it's picked up by workflow engine) so this makes it even more problematic.
Even though we made a lot of performance improvements which should make those GET and UPDATE queries much faster now, we should still think about it some more and see if there is any way to reduce number of queries and make that code more efficient.

Code formatting / style

I don't think we should use a decorator.
It makes code harder to read and understand.
It should be a regular function / method (or two of them so we can get rid of *args and **kwargs stuff which also makes code hard to read and understand) which gets called in places where it's needed.

@khushboobhatia01 what are your plans with this PR? Do you know when you'll be able to pick it up again and address some of the feedback here?


Telegraf memory leak

Telegraf memory usage seems to grow steadily over time in my cluster:

At first I let it grow expecting that the GC would kick in, but it never did. When consumption reached 1G, I just deleted the pods (which made the DaemonSet to create new ones).
When I removed ENABLE_KUBERNETES (setting it to false did not have any effect) from the DaemonSet manifest, the memory consumption stayed at ~13 MB. Of course then no metrics appear in Grafana.

Alright I will take a look at this today/monday

I see the same issue, I setup a new cluster on kube-aws / k8s 1.4.3 / workflow 2.7.0 on Firday and the controller became unresponsive today because telegraf is eating ~700 MByte/s RAM and the machine only has 2GB which is usally fine for a small cluster controller.
The logs of the telegraf pod on the controller show lots of entries like these, not sure if they are normal of part of the issue:
ERROR: input [nsq] took longer to collect than collection interval (1s)

@ged15 Which kubernetes version are you running?

My K8s version:
Client Version: version.Info{Major:"1", Minor:"4", GitVersion:"v1.4.1", GitCommit:"33cf7b9acbb2cb7c9c72a10d6636321fb180b159", GitTreeState:"clean", BuildDate:"2016-10-10T18:19:49Z", GoVersion:"go1.7.1", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"4", GitVersion:"v1.4.3", GitCommit:"4957b090e9a4f6a68b4a40375408fdc74a212260", GitTreeState:"clean", BuildDate:"2016-10-16T06:20:04Z", GoVersion:"go1.6.3", Compiler:"gc", Platform:"linux/amd64"}

The last entry in the logs of the Telegraf pod (no log entries afterwards):
2016/10/21 07:29:48 INF 1 [metrics/consumer] (<IP_ADDRESS>:4150) connecting to nsqd

Not sure if this is useful, but I do not get ERROR: input [nsq] took longer to collect than collection interval (1s) appear in my logs.

The timeout in my case might be related to the controller running out of memory. I had to add a swapfile to be able to talk to the k8s api again.

I did some debugging with @jchauncey and it appears the telegraf pod is leaking tcp connection to cadvisor (port 10255). You could check on a node by running netstat -tan | grep 10255 | wc -l, in my case there were more than 24k open connections.

@jchauncey There's currently no milestone assigned. Will the fix be included in Workflow v2.8.0?

yeah we are holding the release until we get this fixed

I think he's getting that error because nsq is slow to respond due to the
memory pressure
On Oct 24, 2016 9:16 AM, "Gediminas Šedbaras"<EMAIL_ADDRESS>wrote:

The last entry in the logs of the Telegraf pod (no log entries afterwards):
2016/10/21 07:29:48 INF 1 [metrics/consumer] (<IP_ADDRESS>:4150) connecting to nsqd
Not sure if this is useful, but I do not get ERROR: input [nsq] took
longer to collect than collection interval (1s) appear in my logs.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/deis/monitor/issues/153#issuecomment-255737357, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAaRGJGXp7FbykbcgJBSK7M0BwBdPLTzks5q3K_CgaJpZM4KdC8u
.

Just updated to 2.8 and the problem seems gone. Thanks people!
Although now the graph looks weird as it only displays data for the last 3 minutes:

Is this expected?

Yeah the graph resolution is my fault. You can change it in the top right corner. Ill fix it for next release.

Thanks @jchauncey and @felixbuenemann!


No Frontmatter to Edit for Selected Event

Whenever I click on a item in anyt view, I receive this error message. My user intuition tells me it is intended to open up a new note. Unsure if it's a bug or user error!

I get this as well.  I am clocking on items for an online calendar.  Certainly an issue with handling remote calendars that didn't exist previously

This isn't a bug. Remote calendars do not support being edited currently. Follow #25 to see progress on this. I have a PR #96 that takes a step towards two-way sync. In that PR, when you click on the remote calendar event it will pop up the edit event modal dialog, but doesn't allow editing.
@davish I'd recommend closing this out as it will be addressed with my PR.

This wasn't my use case, as I just wanted to display the details, not edit.  In the case where there is no editing, at least the event details should be shown, which is what I wanted.  I don't think 2 way sync is needed for view event without editing.

@pcause then when my PR gets merged, it should address your issue of being able to view the event details. I guess for @jeremytoce, it won't open a new note until there is two-way sync as in #25.

@dojoteef , thanks.  The calendar I was using is not a Caldav calendar.  Assuming your PR will handle this as well.

@dojoteef , thanks. The calendar I was using is not a Caldav calendar. Assuming your PR will handle this as well.

You previously stated:

Certainly an issue with handling remote calendars that didn't exist previously

Are you using a remote .ics calendar then?

I can confirm that both CalDAV and ics calendars are both read-only currently. I'm renaming this issue and recategorizing it as an enhancement to reflect that viewing info for readonly events is a new feature.
@dojoteef we can close out this issue when your next PR gets in to main :)


Add store items, regular items (in progress)

Codecov Report
Attention: Patch coverage is 36.66667% with 57 lines in your changes missing coverage. Please review.

Project coverage is 77.28%. Comparing base (42b949c) to head (6bc9a20).

Files with missing lines
Patch %
Lines

services/stores-service.js
10.00%
22 Missing and 5 partials :warning:

routes/stores.js
50.00%
18 Missing and 2 partials :warning:

services/items-service.js
33.33%
5 Missing and 1 partial :warning:

routes/items.js
55.55%
4 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #38      +/-   ##
==========================================
- Coverage   84.10%   77.28%   -6.82%
==========================================
  Files          35       37       +2
  Lines         629      700      +71
  Branches       69       77       +8
==========================================
+ Hits          529      541      +12
- Misses         89      140      +51
- Partials       11       19       +8

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


Update unary negation of unsigned integers

Hi,
Here it is.
Regards,
Ivan

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @huonw (or someone else) soon.
If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. The way Github handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.
Please see CONTRIBUTING.md for more information.

I've published this change as 0.2.7


v5.0.1: chore: update dependencies

What does this PR do?

Updates all the project dependencies to their latest versions.
Implements my own ESLint plugin.

How should it be tested manually?
yarn test
# or
npm test

Pull Request Test Coverage Report for Build 96

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 93:
0.0%

Covered Lines:
46

Relevant Lines:
46

💛  - Coveralls

Pull Request Test Coverage Report for Build 96

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 93:
0.0%

Covered Lines:
46

Relevant Lines:
46

💛  - Coveralls


Populate K8s annotations for outerloop = s2i build - component operator

Feature request
Populate K8s annotations used by the Component Operator prod = outerloop mode according to what is discovered within the current maven project and git repo
For the moment, we hard code the info needed to perform a s2i build using ap4k k8s annotation
https://github.com/snowdrop/component-operator-demo/blob/master/fruit-backend-sb/src/main/java/com/example/demo/CrudApplication.java#L30-L39
@KubernetesApplication(
        annotations = {
                @Annotation(key = "app.openshift.io/artifact-copy-args", value = "*.jar"),
                @Annotation(key = "app.openshift.io/component-name", value = "fruit-backend-sb"),
                @Annotation(key = "app.openshift.io/git-dir", value = "fruit-backend-sb"),
                @Annotation(key = "app.openshift.io/git-ref", value = "master"),
                @Annotation(key = "app.openshift.io/git-uri", value = "https://github.com/snowdrop/component-operator-demo.git"),
                @Annotation(key = "app.openshift.io/java-app-jar", value = "fruit-backend-sb-0.0.1-SNAPSHOT.jar"),
                @Annotation(key = "app.openshift.io/runtime-image", value = "fruit-backend-sb")
        })

but most of the properties could be calculated dynamically by ap4k and added to the yaml/json file
apiVersion: "v1"
kind: "List"
items:
- apiVersion: "component.k8s.io/v1alpha1"
  kind: "Component"
  metadata:
    annotations:
      app.openshift.io/java-app-jar: "fruit-backend-sb-0.0.1-SNAPSHOT.jar"
      app.openshift.io/artifact-copy-args: "*.jar"
      app.openshift.io/git-dir: "fruit-backend-sb"
      app.openshift.io/runtime-image: "fruit-backend-sb"
      app.openshift.io/git-ref: "master"
      app.openshift.io/component-name: "fruit-backend-sb"
      app.openshift.io/git-uri: "https://github.com/snowdrop/component-operator-demo.git"
    labels:
      app: "fruit-backend-sb"
      version: "0.0.1-SNAPSHOT"
      group: "dabou"
    name: "fruit-backend-sb"
  spec:
    deploymentMode: "innerloop"
    runtime: "spring-boot"
    version: "1.5.15.RELEASE"
...

@iocanel By looking at this briefly, my idea was to enhance ComponentHandler to include BuildInfo  and use that data in the createComponent method.
Does that sound like the proper way to proceed?

Could the BuildInfo class been used for standard s2i build or build managed by our Component Operator ?

BuildInfo is just a collection of data that is pulled from the build system (Maven, Gradle etc). From my limited point of view, it could be used in any context that needs such info.

any context that needs such info

+1

I can implement this, but since there are multiple ways of doing it, I would like @iocanel to confirm or deny whether the idea I had above is good or if there is a more elegant way

WDYT @iocanel ?

As @metacosm pointed out in our last call, it's not just about BuildInfo but about GitInfo too, which is something that currently we doesn't exist.
I think that we should add it all our processors have access to that kind of information.
Now, the ComponentHanlder could be one such place. What needs some thought though is the label names, as I have quite a few questions around them:
Why do we add Opesnhift prefix to our labels? Is Component CRD openshift only?

@cmoulliard ^

Why do we add Opesnhift prefix to our labels?

They have been prefixed as such according to DevTools Bu convention [1]. I'm not opposite to review/change them
[1] https://github.com/gorkem/app-labels/blob/master/labels-annotation-for-openshift.adoc#labels

Is Component CRD openshift only?

No. It supports k8s and openshift ;-)

I'll leave this one to you folks :). If you want me to implement something, just give me a heads up :)


Embeddings.create() got an unexpected keyword argument 'engine', when using ChromaDB

Context ℹ 💁🏼♂
I have a VectorDB with Chroma using OpenAI's Embeddings and created a retriever_model in dspy:
from dspy.retrieve.chromadb_rm import ChromadbRM
retriever_model = ChromadbRM(collection_name=COLLECTION_NAME, persist_directory=MY_PATH)

The following are my dependencies in the requirements.txt file. I am using Python 3.10.12.
chromadb==0.4.22
dspy-ai==2.1.9
openai==1.12.0
tiktoken==0.5.1

Problem 💩
When running
dspy.settings.rm("Who is Chuck Norris?")

I get the following error:
File [~/Library/CloudStorage/OneDrive-Personal/Code/AI/LLM/dspy/venv/lib/python3.10/site-packages/dspy/retrieve/chromadb_rm.py:139](139), in ChromadbRM._get_embeddings(self, queries)
    [125] """Return query vector after creating embedding using OpenAI
    [126]
    [127] Args:
   (...)
    [131]     List[List[float]]: List of embeddings corresponding to each query.
    [132] """
    [134] model_arg = {"engine": self.model_name,
    [135]     "deployment_id": self.model_name,
    [136]     "api_version": self.api_version,
    [137]     "api_base": self.api_base,
    [138] }
--> [139] embedding = self.openai_ef._client.create(
    [140]     input=queries, model=self._openai_embed_model, **model_arg, api_provider=self.openai_api_type
    [141] )
    [143] return [embedding.embedding for embedding in embedding.data]

TypeError: Embeddings.create() got an unexpected keyword argument 'engine'

Possible solution 👍🏼 ✅
I was able to solve it by using only input and model arguments in self.openai_ef._client.create() method, as shown below:
embedding = self.openai_ef._client.create(input=queries, model=self._openai_embed_model)

What do you guys think about this? Thanks in advance for your help!

Ok solved(?) by merging #400. Let us know if all is good now. Thank you @lukaskuhn-lku  and @animtel

Will be pushed to pypi on the next release


How to compile ?

When I ran ./run  happen this error "qemu-system-x86_64: -hda GRAMADO.VHD: Could not open 'GRAMADO.VHD': No such file or directory"
How can I create this GRAMADO.VHD ?

The system has two repositories at the moment. One called gramado/ and another one called hot/ .  You can use 'make' to compile everything in the given repository. If hot/ is present at the same parent directory so, gramado/ is gonna import some files from hot/  .... (build/gramado   ... build/hot)


Create ElevatorPitchGPT.md

📝 Description
Idea #2329
Our agent helps you create create a template to craft a concise and compelling elevator pitch to engage potential customers.
💣 Is this a breaking change (Yes/No):

[ ] Yes
[x] No

🚧 How to migrate?
Don't need.
📝 Additional Information
lol

@wenzhuo2022 please review


Request: Option to use system theme

There should be an option to make the app's light/dark theme toggle follow the system light/dark theme. I would probably make it opt-out, but whatever.
This would make it very useful for those of us who have Android be light in the day and dark at night. This could join the other apps that follow Android's preference.

Implemented in #89. Thank you, @mminer237 !


Choco-Snake

Your checklist for this pull request
Author name
Author:
ishan211
About your game
About me (a chocolate obsessed hacker) as a snake trying to hunt down as much chocolate as possible
How do you play your game?
Press WASD to move, I to restart. Collect chocolate bars, don't hit the walls or your own tail.
Code
Check off the items that are true.

[x] The game was made using the Sprig editor.
[x] The game is placed in the in the /games directory.
[x] The code is significantly different from all other games in the Sprig gallery (except for games labeled "demo").
[x] The game runs without errors.
[x] The name of the file/game contains only alphanumeric characters, -s, or _s.
[x] The game name is not the same as the others from gallery

Image (If an image is used)

[ ] The image is in the /games/img directory.
[ ] The name of the image matches the name of your file.

Thanks for your PR!

Looked at this game today. Still awaiting author to make changes.

Looked at this today, still awaiting changes from the author.

Closing this PR due to inactivity.  Please feel free to re-open this PR if you feel like you want to make further efforts in getting it merged and receiving a Sprig device grant.


Include CLI extension version info in user agent header in Azure Quantum requests

With this change, we're adding the quantum extension version information to the user agent header as it was done in track 1.
This checklist is used to make sure that common guidelines for a pull request are followed.

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your PR is merged into master branch, a new PR will be created to update src/index.json automatically.
The precondition is to put your code inside this repo and upgrade the version in the PR but do not modify src/index.json.

Quantum


Integrate Stellar language interpreter

We would like a general purpose language for creating UDFs and UDAFs.  Stellar (functional language from Metron project) is available and already oriented on the sorts of stream processing we need.  Order of integration tasks:

ability to create UDFs using Stellar
integration with GUI
ability to create UDAFs using Stellar
integration with GUI
integration with MPack

Please see #815 :
We have done a prototype integration of Stellar using the CustomProcessor framework. The StellarCustomProcessor code is available in the “stellar” branch of this repo: https://github.com/mattf-horton/sam-custom-extensions/tree/stellar
This work showed what interface and GUI changes would be needed for a more complete integration. They have been discussed with Harsha. A proposal is here: https://docs.google.com/document/d/19e2PxEJe9ybfL9-WYW4V9LNqtaaAI87b480afjyHRQI/edit#
If logging into Google Drive is not convenient, let me know.
I'm happy to demo the prototype, or discuss any of this proposal. Thanks.


Master

Add evolutions on catch and transfer.

use correct events

use names in filter

Update init.py

use correct lowercase & add ()

lowercase and add 'all'

use pokemon obj in after_transfer event

remove '[#]'

Closing as I will just rebase the branch.


Add pageExcludeRegex

This PR adds a new config option pageExcludeRegex; added to the README.md:
  # Regular Expression string to exclude pages from the tree
  # - Any page that is all-caps: "^[A-Z]+$"
  # - A specific set of pages: "^(?:SETTINGS|PLUGS|index|Library)$"
  # - Any path containing Hidden: "Hidden" e.g. test/Hidden/page1
  pageExcludeRegex: "^(?:SETTINGS|PLUGS|index|Library)$"

Closes #12.

I think somehow excluding notes with the #meta tag would be better

I think somehow excluding notes with the #meta tag would be better

@mufeedali great idea. I just pushed an update that makes this possible: instead of just a single pageExcludeRegex option there is now an exclusions option that can take multiple exclusion rules. And in addition to a page name regex you can also exclude based on tag name or a custom space-script function.
treeview:
  exclusions:
  - type: "tags"
     tags: ["meta"]
  - type: "regex"
     rule: "^(?:SETTINGS|PLUGS|index|Library)$"

pageExcludeRegex still works, but shows a deprecation message in the console. The type: "regex" exclusion does the same exact thing now.


apt-packages v.s. apt-get-packages

Hi, @danielbraun89.
You have recently released apt-get-packages in ccd4c1f71272d86be790f14d2562dc54e9af25ab, but there is already a functionality equivalent to this that has been released.
https://github.com/rocker-org/devcontainer-features/blob/main/src/apt-packages
Were you aware of this?
Wouldn't it be better to manage one jointly than two similar ones?
I have run into this same thing in the past.
devcontainers/spec#155

@eitsupi
I was not at all aware about the apt-packages feature😅 (my apt-get-packages was part of a larger attempt to have more generic features as per #172  (for example I have also added npm-package , pipx-package  and asdf-package)
I agree it would be better to be maintained jointly
As I see the feature has not yet been "grabbed" by github ill just delete the feature package, If we are lucky it will be enough to prevent from the headache of deprecating it later 😄
On another note - considering that a minor feature like k9s has already 1 2 and 3 implementations -and that this devcontainer feature framework is really still in its infancy, it is inevitable that someone somewhen will reimplement this yet again. but one step at a time!

@eitsupi
I was not at all aware about the apt-packages feature😅 (my apt-get-packages was part of a larger attempt to have more generic features as per #172  (for example I have also added npm-package , pipx-package  and asdf-package)
I fully agree it would be better to be maintained jointly!
As I see that my feature has not yet been "grabbed" by github ill just delete the feature package, If we are lucky it will be enough to prevent from the headache of deprecating it later
On another note - considering that a minor feature like k9s has already 1 2 and 3 implementations -and that this devcontainer feature framework is really still in its infancy, it is inevitable that someone somewhen will reimplement this yet again. but one step at a time!

Thanks for your quick response!
This is a really basic feature that I had hoped would be implemented in devcontainers/features (devcontainers/features#67) but no progress was made so I published it from rocker-org (since it is useful in combination with other rocker-org' Features).
Since common-utils version2 now supports not only apt but also dnf and apk, I expect that perhaps in the future more generic Feature will be implemented in devcontainers/features.

As I see that my feature has not yet been "grabbed" by github ill just delete the feature package, If we are lucky it will be enough to prevent from the headache of deprecating it later

Since which Features are published is recorded as metadata as https://github.com/devcontainers-contrib/features/pkgs/container/features, deleting only the Feature is insufficient.
We must remove the Feature from in the src/ directory and then run the publish action again to update the metadata.
(See devcontainers/spec#155)

On another note - considering that a minor feature like k9s has already 1 2 and 3 implementations -and that this devcontainer feature framework is really still in its infancy, it is inevitable that someone somewhen will reimplement this yet again.

No doubt.

done in 006e18efa8d69a0e0097ada514e11449bfcd0ce6

Thanks!


Update readme to reference brew recipe

Also cleaned up a little bit of the readme formatting.

This is fantastic @edwardloveall ! I took a break from eject for a few and totally forgot about this. How do I update homebrew when I make a new release?

Pretty simple, just follow the steps here: https://github.com/Homebrew/homebrew-core/blob/26825c7b2dc7aafa9f0ed85af968441ca465e918/CONTRIBUTING.md#submit-a-123-version-upgrade-for-the-foo-formula
Copied here for posterity:
Submit a 1.2.3 version upgrade for the foo formula

check if the same upgrade has been already submitted by searching the open pull requests for foo.
brew edit foo
edit url and sha256/tag, leave the bottle as-is
brew install --build-from-source foo
run brew audit --strict foo and fix any issues
git commit with commit subject foo 1.2.3
open a pull request and fix any failing tests

Just made a PR with an update. Much more painful than Cocoapods to update, but magic enough.

Yeah they do some weird things. If you don't like their approval process, there's always private taps that you can manage yourself. You could have a raiz tap with all your projects for example.


Swift/4.2

SwiftLint found issues
Warnings

File
Line
Reason

AgrumeCell.swift
52
Delegates should be weak to avoid reference cycles.

  Generated by :no_entry_sign: Danger


when a string is not found in a line, we do not always get a float back

I found that I got errors about the status file not being a status file even when it was (eh?). This should fix that. Also see #69

Oh, yeah, this looks better. I was wondering about the 1.0 thing.


[Feature Request] ColorPicker Component

Please try the example here : https://github.com/adhbh/react-color-selector
This is being developed for an organisation. But can it be added as a new component in material-ui?
If not this, the material-ui surely needs a ColorPicker component similar to this.
Suggestions and critiques most welcome.
Here are some other designs: https://dribbble.com/search?q=material+colorpicker

Very cool! Also worth a mention is that you developed it, and it's built on Material-UI. :+1:
As for making it part of Material-UI itself, I think the community may be better served if it remains a standalone project, and we link to it in the 'Related projects' section of the docs, but that's just one opinion. @callemall/material-ui ?

Although helpful, I think since a color picker is neither part of the material design guidelines, nor something that's required by or found in most applications, it shouldn't be a part of material-ui. It seems like more of a component that can be implemented by a third party library.

it shouldn't be a part of material-ui

I agree. However, this component could be really useful for the Theme section of the documentation.
We could let user pick a primary and a secondary color. Those colors would be reflected on the all doc site, as switching from the dark / light them is doing it today.

Nice idea! It would be good exposure @adhbh's component too.

So are you planning to make color picker component?..

@pavel06081991 From what I understand, this component has already been done by @adhbh here: https://github.com/adhbh/react-color-selector.

Any progress on this ? ;)

@LoicMahieu Project seems dates and there are good alternatives https://github.com/casesandberg/react-color

@LoicMahieu If by "progress" you mean adding a color picker to Material-UI, we pretty much agreed that was better served by standalone components, so this issue should have been closed sooner. (It is now 😄 )
@Rosenblad It's an open source component - all it needs is its dependencies bumped and it's good to go. There are, as you say, plenty of alternatives react-color being a comprehensive if somewhat heavyweight one. It depends on your requirements and priorities.

@mbrookes Sure it is better as standalone component : https://github.com/LoicMahieu/material-ui-color-picker ;)


Add --config flag

Adding onto Snowpack’s configuration file (#184), this lets the user manually specify a config file location with:
snowpack --config ./path/to/custom.config.js

Why?
This enables users to use multiple configurations per-project (for example, using --config dev.js and --config prod.js for different targets).
It also empowers projects like sveltet to use Snowpack behind-the-scenes with custom configuration files (even ones in node_modules).

cc @jakedeichert I was able to use third-party svelte components in svelvet with the following code on my local machine:
const proc = spawn('node', [snowpackLocation, '--include', includeFiles, '--dest', 'dist/web_modules', '--config', path.resolve(__dirname, 'snowpack.config.js'), maybeOptimize, maybeStats]

and the following snowpack.config.js:
module.exports = {
  rollup: {
    plugins: [require('rollup-plugin-svelte')()],
  },
};

@dangodev amazing!! 😄 That will allow us to close https://github.com/jakedeichert/svelvet/issues/49 🙌

Awesome! @dangodev thanks for remembering to close the loop on this one!

@jakedeichert this was released in 1.6.0! 🎉

Awesome! Thanks again for tackling this :)
I'll use your solution posted above to solve jakedeichert/svelvet#49 this week hopefully 👍


Ask Kyle how much of final demo he's willing to do

As a developer I want to make sure my work is adequately displayed without putting too much responsibility on my client.

[x] Talk to Kyle about our final demo and ask how much he would like to demo in class for our project
[x] Make demo videos to fill gaps

Make backup videos for everything.
Need curtain demo.
Kyle can set up a lock.
Kyle can hold up dampers.


Missing convenience entity data methods

While looking through Data 2.0, I noticed a few entity classes were missing data methods (convenience methods like getHealthData) which they should contain. The list is below.

Ageable

getBreedableData

Arrow

getKnockbackData

ExperienceOrb

getExperienceHolderData

Horse

getSaddleData

ItemFrame

getRotationalData

Pig

getSaddleData

Player

getFoodData (this may belong in Human; I'm not entirely sure)

Sheep

getDyeableData

ThrownPotion

getPotionEffectData

Wolf

DyeableData

ZombiePigman

getAngerableData

I'd be eager to create a pull request for these changes, but I'd like to first assert that they're valid concerns and allow for a discussion before starting work.

A lot of these aren't guaranteed to exist from the get go. If an entity has the likely capability of having these data, that isn't 100% true all the time, in which case, they are not included in the interfaces themselves. Besides, the whole point of Data API is that you don't have to rely on the various entity interfaces to deal with the data. You just query with entity.get(DyeableData.class) and if the optional is present, well, you can manipulate it.

ItemFrame
getRotationalData

Player
getFoodData (this may belong in Human; I'm not entirely sure)

ThrownPotion
getPotionEffectData

Sheep
getDyeableData

These I can see being added, but the rest, they're still optionally available.

Could you please clarify in regard to ExperienceOrb/ExperienceHolderData and ZombiePigman/AngerableData? To my understanding, experience orbs should always contain a value for experience (even if 0), and zombie pigmen should always have an anger level.

ExperienceHolderData doesn't apply because it doesn't have the notion of "level"s. It only contains a value of the total experience it holds, so quite possibly, I'll be adding another DataManipulator and/or Key for an ExperienceStoredData or something. All of these changes are being bundled in feature/data branch currently. With regards to ZombiePigmen, I meant to include them.

Fair enough. Am I clear to start a PR?

As I said before, these sort of changes are going to be bundled in feature/data.

Actually, @caseif, ItemFrames, when they have no ItemStack, they have no RotationalData to associate themselves with. So, that's not going to be an "accessor" in this case.
As with ThrownPotion, the thing is that the Projectile doesn't actually store the potion effects, but rather the ItemStack that is in the RepresentedItemData manages the potion effects being applied and rendered.
Sheeps already have getDyeData(), so no changes needed.
I've added the following:
Human#getFoodData(),
ZombiePigman#getAngerableData()
Arrow#getKnockbackData(),
ExperienceOrb#getExpOrbData()
The rest are not applicable, or not guaranteed to exist at all times.

RotationalData to associate themselves with. So, that's not going to be an "accessor" in this case.

Testing in 1.8.8 that doesn't appear to be the case. punching and replacing an item in an itemframe keeps it's orientation.

Testing in 1.8.8 that doesn't appear to be the case. punching and replacing an item in an itemframe keeps it's orientation. But there's no guarantee that this will always be true.

It's storing a cached value, but the point is that regardless whether an ItemStack is retained, the value shouldn't be considered to just remain "cached" and then magically break when it is no longer cached.


Expose half tile config in LLK (WH)

fyi: @davorchap

We will use half tile for shading weights in DRAM. The reason is that they're 12 banks and it's sometimes not divisible for the mammal weights. For example, for a weight tensor of 8192x1280. we shard 1280 by 12 banks. if use half tile, each bank can have 112 as width, together we have 112x12=1344. This requires much less padding than the full tile one, which needs to pad to 1536.

@yugaoTT can this issue be closed since you merged 16x32 and 32x16?

@rtawfik01 yeah I will close this one


Blank screen when app is launched a second time

There is a blank screen that appears to be like the app is launched but hasn't any UI to display. Pressing back dismisses the app and one can navigate normally with device otherwise everything appears like the device has frozen, which is not the case but that there's an app running with no interface and appears transparent to the device screen. This happens when the app is launched the subsequent times from the very first time it is installed and launched. Please we need this issue fixed asap.

Just to be clear - does everything work on the first launch? So on the first launch, if the welcome screen shows, does the app run normally after the welcome screen closes?
Does this happen on all Android versions?

Everything works fine on first launch. and yes it happens on most of the
versions of android I've tested with. (4.2, 4.4, 5.0, 5.1)
cheers
On Mon, Feb 13, 2017, 22:32 Stephen Tuso<EMAIL_ADDRESS>wrote:

Just to be clear - does everything work on the first launch? So on the
first launch, if the welcome screen shows, does the app run normally after
the welcome screen closes?
Does this happen on all Android versions?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/stephentuso/welcome-android/issues/32#issuecomment-279529246,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGMQfa9AmEIvXigc0R5os2iHqQN5P9ACks5rcMv1gaJpZM4L_u8_
.

Okay that's odd, is everything set up exactly like in the readme? Or is there anything else in your launcher activity that could possibly interfere? I've never run into that, so it's hard to know what the problem is. Is anything showing up in the logcat?

I'll have to look at the log cat to see
On Thu, Feb 16, 2017, 20:17 Stephen Tuso<EMAIL_ADDRESS>wrote:

Okay that's odd, is everything set up exactly like in the readme? Or is
there anything else in your launcher activity that could possibly
interfere? I've never run into that, so it's hard to know what the problem
is. Is anything showing up in the logcat?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/stephentuso/welcome-android/issues/32#issuecomment-280429947,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGMQfXnwgGpGeA13quhbyV8bKvpiPdUyks5rdKDIgaJpZM4L_u8_
.

same issue. No UI.

@felixeduardo15 Are any warnings or errors showing up in the logcat? Any chance you could post the onCreate method of your main Activity? What device/Android version are you using?

Closing due to inactivity


Warning with pass value CSS properties width and height

Warning: a div tag (owner: ReactPlayer) was passed a numeric string value for CSS property width (value: 260) which will be treated as a unitless number in a future version of React.
<ReactPlayer
   url={video.link}
   height="180"
   width="260"
/>

@AtNovember use 180px instead of 180, etc.

You should pass in numeric values like width={260} height={180} or, as @Redmega says, add units to the strings like width="260px" height="180px"


How can I disable gateway transit on peered networks?

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Versions

terraform:  v1.1.6
azure provider: 2.99.0
module: 1.1.4
Description
We are deploying an environment using this module that has an NVA in place, this is why we want gateway transit to be disabled on the peered VNET's from the hub. How can I disable this?
We are already using custom_settings_by_resource_type to alter the VPN Gateway settings but I can't seem to figure out what the syntax is for the Peerings.

Hey @cn-dylan,
Thanks for raising this.
I assume this is a requirement from a security perspective, as this setting isn't preventing anything from technically working, right?
Also, just to call out this setting prevents the spokes from being advertised via BGP from the Gateways to on-prem.
As it is today, it is not possible to disable/change the VNET peering setting we set on the Hub VNET side only in this module via a variable or advanced block as seen here:
https://github.com/Azure/terraform-azurerm-caf-enterprise-scale/blob/0b343c6ba9ce4595a0e494f97d2a439845a1cb2a/modules/connectivity/locals.tf#L1227-L1230
Therefore, our suggestion today would be to handle the peering's outside of the module to handle this scenario.
However, there is value to specify the spoke VNETs still in the module as below:
https://github.com/Azure/terraform-azurerm-caf-enterprise-scale/blob/0b343c6ba9ce4595a0e494f97d2a439845a1cb2a/variables.tf#L243
If you are planning to use the following setting for DNS as below
https://github.com/Azure/terraform-azurerm-caf-enterprise-scale/blob/0b343c6ba9ce4595a0e494f97d2a439845a1cb2a/variables.tf#L359
Looking ahead
We are currently thinking about our story for custom NVA users/customers and what that means. However, this is in very early phases.

Looking ahead
We are currently thinking about our story for custom NVA users/customers and what that means. However, this is in very early phases.

Just to add, it would be great to have your feedback on this point to help us shape this requirement. Thank you!


Fix broken link

Found this broken link when I was testing the doc build.

Merging without review to fix the broken build.


Breaking changes to VSTS templates?

Select the .NET Desktop template.

Maybe I'm a dummy but I can't find this template when I search for it. Not sure if it has been renamed or removed.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: a56478c9-7dfc-43e9-c348-357dc6862d73
Version Independent ID: dd3cd0a1-dc03-15d3-1298-b76d770e4760
Content: Build your SQL server database
Content Source: docs/pipelines/apps/aspnet/build-aspnet-dacpac.md
Product: devops
GitHub Login: @andyjlewis
Microsoft Alias: alewis

found it

@dbrtly I'm glad you found it, do we need to update our documentation to make anything more clear? Or was it just a case of that template being hard to find for some reason?


Add Documentation for Profiling Code in FrankenPHP to Enhance Developer Experience

Describe you feature request
Hello,
As a new user of FrankenPHP, I’ve encountered some performance discrepancies compared to my previous classic Nginx/PHP stack. The source of these differences isn’t immediately apparent, so I attempted to profile the execution to pinpoint potential bottlenecks.
In the past, I’ve used php-spx, which is straightforward to set up and provides a fantastic web UI for navigating flamegraphs.

However, I wasn’t able to make the SPX web UI available in my FrankenPHP setup, even though profiling through CI works correctly. This limitation seems to stem from a specific requirement in php-spx (@NoiseByNorthwest friendly ping FYI) :

N.B.: http://localhost/ must be served by a PHP script through standard web server features like directory index or URL rewriting. The PHP script will, however, not be executed—SPX intercepts and disables its execution to serve its content instead.

Here’s my current FrankenPHP's Docker configuration modifications for reproducing the issue:
# Development FrankenPHP image
FROM frankenphp_base AS frankenphp_dev

ENV APP_ENV=dev XDEBUG_MODE=off

RUN mv "$PHP_INI_DIR/php.ini-development" "$PHP_INI_DIR/php.ini"

RUN set -eux; \
    install-php-extensions \
        xdebug \
    ;

COPY --link .docker/conf.d/20-app.dev.ini $PHP_INI_DIR/app.conf.d/

RUN apt-get update && apt-get install -y zlib1g-dev \
    && git clone --depth=1 https://github.com/NoiseByNorthwest/php-spx.git /usr/lib/php-spx \
    && cd /usr/lib/php-spx \
    && phpize \
    && ./configure \
    && make \
    && make install \
    && rm -rf /var/lib/apt/lists/*

CMD [ "frankenphp", "run", "--config", "/etc/caddy/Caddyfile", "--watch" ]

For example, running SPX_ENABLED=1 SPX_FP_LIVE=1 php bin/console inside the container produces this output:

Given these challenges, I wanted to ask the community:

What’s the recommended approach for profiling PHP applications with FrankenPHP?
Are there any known workarounds or alternative tools for integrating SPX’s web UI in this context?

If a preferred method exists, I’d suggest documenting it in the official guides to help new users troubleshoot migration and performance optimization issues more effectively. A tool that supports both PHP and underlying Go/PHP source traces would be ideal.
While paid tools like Blackfire or Tideways are excellent, I believe showcasing an open-source solution in the documentation would greatly benefit the community.
Thanks for this great project and your insigths on this topic !

I just use xdebug's profiling tools (which works fine), I've never heard of spx before.
I have no idea how spx works, but it looks like an extension. Perhaps it needs to be modified to hook into the frankenphp sapi, or the frankenphp sapi is missing a call that is present in other sapis, and thus it is missing a hook.

@withinboredom Thanks for your input! I’ll explore this hook idea.
About your use of Xdebug’s profiling, do you typically use XDEBUG_MODE=profile and open the output with KCachegrind? Have you also managed to generate flamegraphs from it? Just curious.
Once I’ve dug deeper, I’ll document my findings and share the necessary Docker setup as a contribution.

@tigitz see https://github.com/NoiseByNorthwest/php-spx/issues/258
The only known issue is a crash while serving web UI via FrankenPHP, I could investigate it if someone provides an easy to use (docker-based) reproducer.
It is also possible, as a workaround, to use a web UI served by a standard PHP setup (fpm or even the builtin server) as long as it is configured to use the same data directory than your FrankenPHP setup.

@NoiseByNorthwest Thank you! I'll follow up on the issue you linked regarding the FrankenPHP/php-spx integration.
I'll keep this issue open to document profiling in FrankenPHP.


Navigo UI - Test Update 3

Changed the timeout of the 2 failing tests

Codecov Report

Merging #184 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master    #184   +/-   ##
======================================
  Coverage    71.3%   71.3%
======================================
  Files         245     245
  Lines       15558   15558
  Branches     1663    1663
======================================
  Hits        11093   11093
  Misses       4465    4465

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 794b6bc...3b07062. Read the comment docs.


endchan colors (again)

Yeah he added more colors
https://endchan.xyz/operate/res/6358.html#q6492
https://endchan.xyz/.static/posting.html
Can you make  [meme][/meme] each letter a different color and [autism] the same just big letters (doesn't have to have the rainbow effect) or is that not possible?

New colors added.
Yes, technically it's possible to make a rainbow but I won't do it.


[all] OkHttpClient revision. Also checking okHttp's shutdown

Want to check if we do something like this on Android's exit, as well as Desktop's shutdown.
            okHttpClient.dispatcher().executorService().shutdown();
            okHttpClient.connectionPool().evictAll();
            try {
                okHttpClient.cache().close();
            } catch (IOException e) {
                e.printStackTrace();
            }

Other interesting findings. YT's extractor uses its own custom Browser object during search.
This browser's disconnect() method doesn't seem to be invoked.
Browser uses a HTTPConnectionImpl to do its networking. It's implemented using a regular java.net.Socket, which I only see being closed whenever there's an IOException or in the event of a redirect. Never when we're done we close these sockets.
This might be a big performance issue causing network errors and out of memory errors.

Regarding your comments/questions:

I take care of the connection pool eviction in android in EngineService#stopOkHttp (and a little more), but not in desktop, since I didn't detect any problems there. The thread pools are "daemon" which not prevent the proper shutdown in desktop or android.

We can improve a little the object creation in OKHTTPClient, but that's mostly inconsequential, since they are short lived. The main reason they provide an API to derive from an already created object (we can't use the same exact instance since we pass different settings, like user-agent, timeout, etc.) is to make sure you use the same thread pool for the Dispatcher and the same ConnectionPool for improved HTTP performance. We do that in OKHTTPClient#newOkHttpClient already, with the same desired effect.

In Browser if you get to the method Browser#getPage and inside Browser#loadConnection, you will see the try-finally block with the connection disconnect().

doing something like this, so next time the app restarts HttpClientFactory#getInstance will recreate them.
// in HttpClientFactory
public static void destroyThreadPools() {
   if (okHttpClientPools == null) {
    return; //already destroyed or not created yet
   }
   okHttpClientPools.get(HttpContext.SEARCH).shutdown();
   okHttpClientPools.get(HttpContext.DOWNLOAD).shutdown();
   okHttpClientPools.get(HttpContext.MISC).shutdown();
   okHttpClientPools = null;
}

No, because shutdown only prevents new task to be submitted, not actual stop of running threads. The only way a "normal" JVM ignore a running thread is if it's daemon, which they are.

what about shutdownNow()

/**
     * Attempts to stop all actively executing tasks, halts the
     * processing of waiting tasks, and returns a list of the tasks
     * that were awaiting execution. These tasks are drained (removed)
     * from the task queue upon return from this method.
     *
     *
This method does not wait for actively executing tasks to
     * terminate.  Use {@link #awaitTermination awaitTermination} to
     * do that.
     *
     *
There are no guarantees beyond best-effort attempts to stop
     * processing actively executing tasks.  This implementation
     * interrupts tasks via {@link Thread#interrupt}; any task that
     * fails to respond to interrupts may never terminate.
     */
    // android-note: Removed @throws SecurityException
    public List<Runnable> shutdownNow() {
        List<Runnable> tasks;
        final ReentrantLock mainLock = this.mainLock;
        mainLock.lock();
        try {
            checkShutdownAccess();
            advanceRunState(STOP);
            interruptWorkers();
            tasks = drainQueue();
        } finally {
            mainLock.unlock();
        }
        tryTerminate();
        return tasks;
    }

There are no guarantees beyond best-effort attempts to stop processing actively executing tasks.
This does nothing if the actual running thread is not prepared to be interrupted. I doubt this is actually a problem, imagine if we need to to shutdown all the running pools. It's better to design the threads to properly finish, in my opinion.

gotcha, pretty hard to prepare them to do clean shutdown on InterruptedException.
I've still yet to take a snapshot of the JVM's state to see if we have a bunch of clients or not, that's really my only worry now, that the application's performance might be taking a hit with each search it makes.


Cannot publish to Cocoapods

~/dev/oss/defaults main
❯ pod trunk push --allow-warnings

[!] Found podspec `Defaults.podspec`
Updating spec repo `trunk`

CocoaPods 1.10.1 is available.
To update use: `sudo gem install cocoapods`

For more information, see https://blog.cocoapods.org and the CHANGELOG for this version at https://github.com/CocoaPods/CocoaPods/releases/tag/1.10.1

Validating podspec
 -> Defaults (4.2.0)
    - WARN  | url: The URL (https://twitter.com/sindresorhus) is not reachable.
    - NOTE  | xcodebuild:  note: Using new build system
    - NOTE  | xcodebuild:  note: Building targets in parallel
    - NOTE  | xcodebuild:  note: Using codesigning identity override:
    - NOTE  | xcodebuild:  note: Planning build
    - NOTE  | xcodebuild:  note: Constructing build description
    - ERROR | xcodebuild: Returned an unsuccessful exit code. You can use `--verbose` for more information.
    - NOTE  | xcodebuild:  note: Using codesigning identity override: -
    - NOTE  | xcodebuild:  warning: Skipping code signing because the target does not have an Info.plist file and one is not being generated automatically. (in target 'App' from project 'App')
    - ERROR | [iOS] xcodebuild:  Defaults/Sources/Defaults/Observation.swift:451:20: error: 'NSSecureCodingKeyChange' is only available in iOS 11.0 or newer
    - NOTE  | xcodebuild:  Defaults/Sources/Defaults/Observation.swift:451:1: note: add @available attribute to enclosing extension
    - ERROR | [iOS] xcodebuild:  Defaults/Sources/Defaults/Observation.swift:454:20: error: 'NSSecureCodingOptionalKeyChange' is only available in iOS 11.0 or newer
    - NOTE  | xcodebuild:  Defaults/Sources/Defaults/Observation.swift:454:1: note: add @available attribute to enclosing extension
    - ERROR | xcodebuild:  Defaults/Sources/Defaults/Observation.swift:451:20: error: 'NSSecureCodingKeyChange' is only available in tvOS 11.0 or newer
    - ERROR | xcodebuild:  Defaults/Sources/Defaults/Observation.swift:454:20: error: 'NSSecureCodingOptionalKeyChange' is only available in tvOS 11.0 or newer

[!] The spec did not pass validation, due to 5 errors.

PR welcome to fix this. I don't intend to waste time on Cocoapods.

This has nothing to do with Cocoapods, it is actually caused by https://github.com/sindresorhus/Defaults/issues/63 .
As it says, 'NSSecureCodingOptionalKeyChange' is only available in iOS 11.0 or newer
Same when integrated with SPM.


Client ID case sensitive

Hey, thanks for this plugin, it was pretty much exactly what I needed. However, I found a little typo in the readme, it says config.imgurClientID but that should be config.imgurClientId (lower case D), as that's what the code refers to.

I was just about to write this.  Not only is this a problem in the README, the plugin code alert message (clientIdMissing) specifies it in the wrong case too!   Really nasty when you get imgur bundled with your ckeditor.

@yfxie

@djvs @bastiaan89 fixed the problem, thanks you guys. the latest version is 1.0.2


Feature/official frontend auth0

Description
Once user logs in or signups created the ability to grab the token from auth0 and set it into local storage. we need to set it into storage because in the axioswithauth we are able to grab the token from storage and send it into header. Okta had similar functionality.  refactored the logout in the dashboard that you logout with auth0 and also remove the token from local storage. Added a not permanent logout button because we have to finish with the private route.
contributors
https://github.com/aaronamendez
Video Link
https://www.loom.com/share/2bcd60dc810e4ce195e7fd5344fa54f5
Trello Link
Auth0: add auth0 login and signup on the frontend
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] This change requires a documentation update

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have removed unnecessary comments/console logs from my code
[x] My changes generate no new warnings
[x] I have checked my code and corrected any misspellings
[x] No duplicate code left within changed files
[x] Size of pull request kept to a minimum
[x] Pull request description clearly describes changes made & motivations for said changes

Well done. Glad we are transitioning to Auth0. While honestly not a fan of the style guides, our component approach makes it feasible -- and you did a great job keeping it clean considering the restrictions)

Closing this PR and accounting for contributions on it as this work has been condensed into a new branch until breaking changes have been addressed.


UMT first block after hard fork is not stored to GCS and may be missed if archive is not started first

Preliminary Checks

[X] This issue is not a duplicate. Before opening a new issue, please search existing issues: https://github.com/MinaProtocol/mina/issues
[X] This issue is not a question, feature request, RFC, or anything other than a bug report. Please post those things in GitHub Discussions: https://github.com/MinaProtocol/mina/discussions

Description
After the hard fork, I started the node, waited for it to sync, and then turned up the archive.
Unfortunately, this meant I missed the first block (1277-3NK6S443vSpoSQ9va6ZCX4CewAUqmmig7d2eTZwRYe3oqPnWVMEx). Usually, that's not a big deal - a download missing blocks script will pick it up from GCS; however, this block is unique, and the precomputed block is not stored in GCS.
https://discord.com/channels/484437221055922177/1212960015795888188
Steps to Reproduce

At hard fork, start mina daemon and wait to sync before starting mina-archive

...
Expected Result
Missed "genesis"-ish block would be present in GCS.
Actual Result
Block was not put to archive db (because archive was not running at the moment) and was also not put to GCS, so there was no (normal) way to recover.
Stopping node, removing .mina-config, and starting up again, I was able to get the block inserted into the archive db.
Daemon version
Commit eb0a6d00675756972f4f22ecaa10532bae601c59 on branch hardfork-automation-umt
How frequently do you see this issue?
Rarely
What is the impact of this issue on your ability to run a node?
Low
Status
Mina daemon status
-----------------------------------

Global number of accounts:                     201806
Block height:                                  1300
Max observed block height:                     1300
Max observed unvalidated block height:         1300
Local uptime:                                  36m29s
Ledger Merkle root:                            jxLLJAweSiqXNXncqR41fYSQekzJ7SWCG8mVtQHbCYELVqxLhj2
Protocol state hash:                           3NKSQVVNN1zVAjupqktkyxf72zjKQQYkaVfx9rfhW4ZDHUMNW1SR
Chain id:                                      ded2784524f624295e63cede1a4ab9348fbcde7160876c4d55f1481b9d3ef542
Git SHA-1:                                     eb0a6d00675756972f4f22ecaa10532bae601c59
Configuration directory:                       /home/minar/.mina-config
Peers:                                         45
User_commands sent:                            0
SNARK worker:                                  None
SNARK work fee:                                100000000
Sync status:                                   Synced
Catchup status:
        To build breadcrumb:           0
        To initial validate:           0
        Finished:                      24
        To download:                   0
        Waiting for parent to finish:  0
        To verify:                     0

Block producers running:                       0
Coinbase receiver:                             Block producer
Best tip consensus time:                       epoch=0, slot=156
Best tip global slot (across all hard-forks):  3256
Consensus time now:                            epoch=0, slot=158
Consensus mechanism:                           proof_of_stake
Consensus configuration:
        Delta:                     0
        k:                         290
        Slots per epoch:           7140
        Slot duration:             3m
        Epoch duration:            14d21h
        Chain start timestamp:     2024-02-29 21:00:00.000000Z
        Acceptable network delay:  3m

Addresses and ports:
        External IP:
        Bind IP:        <IP_ADDRESS>
        Libp2p PeerID:
        Libp2p port:    8304
        Client port:    8301

Metrics:
        block_production_delay:             7 (0 0 0 0 0 0 0)
        transaction_pool_diff_received:     7
        transaction_pool_diff_broadcasted:  0
        transactions_added_to_pool:         33
        transaction_pool_size:              7
        snark_pool_diff_received:           111
        snark_pool_diff_broadcasted:        0
        pending_snark_work:                 0
        snark_pool_size:                    133

Additional information
Error from daemon (because archive service hadn't started yet.)
Could not send breadcrumb to archive:
...
["Unix.Unix_error","Connection refused","connect","<IP_ADDRESS>:3086"]
...

Is this going to be addressed and will the block be preserved somewhere/somehow?
See also https://github.com/MinaProtocol/mina/issues/15262#issuecomment-1984191237


Change social absolute links to relative links

Fixes issue found in #783
Describe the changes you have made in this pr -
Facebook and twitter links are now relative (/facebook instead of circuitverse.org/facebook)

Coverage remained the same at 83.278% when pulling def9a52cc1055e44701878abc60fa190dd6f61f6 on JoshHeng:fix-social-media-icon-links into a9f751a98add91d932acbeab8c8c792ae5b583a5 on CircuitVerse:master.

Coveralls doesn't matter in this case

It hasn't been merged but links from navbar have been removed for now, you can close it

@tachyons This can be closed.


Create main.yml

Description

Adding a workflow file to help jest work with sonarcloud

Well that was fast


Lorem ipsum text shown in Twitter card for playlist.wiki homepage

https://twitter.com/GregBoustead/status/697095257633267712

@ragesoss I think Twitter might have cached an old version of the site meta, I just ran playlist.wiki through their validator and the meta seems to be correct: https://cards-dev.twitter.com/validator

Cool beans.


StartX and StartY not assigned after Dashboard is created

Issue

Adding new colour widget with startX and startY with the help of + button. It's adding the widget but always adding at the top not at the place of startX and startY (Making startX and startY to 0).

How you can reproduce this issue?

You can add 2 more fields (startX and starY) in the addDialogBox. And assign startX and startY to the ColoredDashboardItem class while adding them to the itemController.

Thanks to your issue.
I added a mountToTop parameter to the add and addAll methods. If you set it to false it will try to insert it starting from the startX and startY you provided. It will never try to place before the starting points you give.
mountToTop default is true.

@Mehmetyaz Unfortunately, mountToTop isn't available if you use a storage delegate and want to save and restore startX and startY so the state is restored as it appeared originally. Returning these values from a storage delegate doesn't have an effect - it just lays the widgets out left-to-right, top-to-bottom. I have slideToTop set to false.


Not loading

Copied the folder into custom components, selected the integration in home assistant and put in latitude and longitude, occasionally get to second screen and have tried both options (station and fuel type) but no further configuration is possible? Am I missing something? Is there meant to be another screen? No sensor, no state, no attributes.

Copied the folder into custom components, selected the integration in home assistant and put in latitude and longitude, occasionally get to second screen and have tried both options (station and fuel type) but no further configuration is possible? Am I missing something? Is there meant to be another screen? No sensor, no state, no attributes.

did you ever get this to work? having the same problem. i entered the distance and lat/long... but not seeing any entities discovered. i notied in the python script that it was set to poll hourly - but would have thought it would poll immediately upon configuration... changing this value didn't help.

Copied the folder into custom components, selected the integration in home assistant and put in latitude and longitude, occasionally get to second screen and have tried both options (station and fuel type) but no further configuration is possible? Am I missing something? Is there meant to be another screen? No sensor, no state, no attributes.

did you ever get this to work? having the same problem. i entered the distance and lat/long... but not seeing any entities discovered. i notied in the python script that it was set to poll hourly - but would have thought it would poll immediately upon configuration... changing this value didn't help.

I did not, ended up using a plugin available on HACS which just tells me the cheapest in Australia for each fuel type and basing my judgement on that, quite a shame

Hi Guys - the aus_fuel integration is broken until i can find another free datasource for the prices.

Hey there's multiple API services.
There's an Australia wide one: https://fuelprice.io/api/
Theres also official government ones for QLD, NSW, and TAS, maybe a few more. But I think the Australia wide one looks the most promising as it gives all the information straight on the website.


eval caching implications of not using flakes under the hood?

I wonder what the implications of using
https://github.com/fluidattacks/makes/blob/939a40963d820ad35a296d00cabe43cdce16eae7/src/cli/main/__main__.py#L128-L144
are for eval caching?
See also elco's blog post about flakes, especially the last few paragraphs:
https://www.tweag.io/blog/2020-06-25-eval-cache/

I decided not to implement flakes simply because they are an unstable feature of Nix.
This means that when flakes get to be stable (which may be soon https://discourse.nixos.org/t/nix-release-schedule-and-roadmap/14204) we'll support it and use it instead of this nix-build that you mention
Now, as per the evaluation caching, I can not answer from a theoretical point of view, but I can tell you that we've not experienced performance degradations for not using flakes so far in our real life project (https://gitlab.com/fluidattacks/product), so I'm pretty sure real life users won't experience them either
From what I see, Makes is leveraging the current stable state of the art that Nix offers, as Nix offers it, so we won't change that for the moment.
However, I'm excited to see Flakes become stable and then upstream those benefits into Makes!


Fix capabilities response for multiple client versions

@ishank011 is not so simple.
We need the following:
2.7+ will use Chunking V1 if "capabilities > files > bigfilechunking" is "true" AND "capabilities > dav > chunking" is not there
2.7+ will use Chunking NG if "capabilities > files > bigfilechunking" is "true" AND "capabilities > dav > chunking" = 1.0
2.7+ will use TUS if "capabilities > files > bigfilechunking" is "false" AND "capabilities > dav > chunking" = "" AND "capabilities > files > tus_support" has proper entries.


✨ documentation colors

The background should be dark blue, this may make the pink easier to see.

the back was blue and the text pink both from Dracula, have adjusted the pink slightly ... possibly to improve readability.


⚠ Catalog API Endpoint has degraded performance

In 53b6750, Catalog API Endpoint (https://catalog.roblox.com/v1/bundles/details?bundleIds=192) experienced degraded performance:

HTTP code: 200
Response time: 4392 ms

Resolved: Catalog API Endpoint performance has improved in abd2494.


Update performance benchmarks with memory usage

Include "deploy" step on github actions (should update README)

This seems useful:

https://github.com/marketplace/actions/generate-update-markdown-content
https://docs.github.com/en/free-pro-team@latest/actions/guides/storing-workflow-data-as-artifacts


readme updates for br command name

minor, but solves the problem i hit.  see the readme formatting at https://github.com/brooklyncentral/brooklyn-cli/blob/readme-updates/README.md
expect this will change again when we reference this from brooklyn

BTW this command allowed me to build it in-place writing output to ./bin/: go build -o bin/br ./br
might be useful for when we need to build this integrated with other brooklyn build.  however it's far from enough, it is depending on my GOPATH being set and dependencies downloaded and i suspect it probably isn't in looking at changes made locally e.g. in api/*.

That does build from local source, so any changes will be picked up ok (but yes everything in Go relies on the GOPATH).


ksp-audit-mysql-stig-v-235155.yaml

@yasin-cs-ko-ak  Based on the conversation and the reference links provided fixing this using the current version of KubeArmor or CIlium is not possible. Closing the PR
cc: @syeddhadi @salman-accuknox @raviknox


Adorner

I've been trying to get Adorners to work with Granular.
What's currently breaking:
I'm missing the virtuals for the following overrides from the adorner class:
protected override int VisualChildrenCount
protected override Visual GetVisualChild(int index)
Although the adorner is ' fully implemented '
Is this a .Net version thing?
Thanks,
Henk

Hi,
Adorners are implemented, and by "implemented" I mean that there is AdornerLayer that host adorners and placed above other controls, and adorners are clipped and transformed as you would expect.
However their interface is not fully compatible with WPF's adorners

There is no OnRender method to override in UIElement - That's going to change in the coming version (sorry for the slow updates)
The Visual interface is also not fully compatible with WPF - There are no VisualChildrenCount and GetVisualChild members to override, visual children are added a bit differently, and in the Adorner case, there is a Child property that should be set.

This, like any other incompatibility can be worked around by separating the implementation into different files, in this case by using partial classes, and keeping the incompatible code in files that are not shared between projects, something like:

WpfApplication1\Adorner1.cs - Contains shared implementation
WpfApplication1\Adorner1.WPF.cs - Overrides WPF's Visual members to add a child (an "*.WPF.cs" exclusion rule may be added to ProjectLink.target)
WpfApplication1.Web\Adorner1.Web.cs - Sets the Child property

I know that it's far from ideal, but there are still a few places that I haven't got to fix yet, sorry about that.


Stacked VLANS (Q-in-Q) but not 802.1ad

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[X] I have read the contributing guide lines at https://github.com/opnsense/core/blob/master/CONTRIBUTING.md
[X] I am convinced that my issue is new after having checked both open and closed issues at https://github.com/opnsense/core/issues?q=is%3Aissue

Is your feature request related to a problem? Please describe.
My ISP provides a service to me with VLANS stacked inside a VLAN. Specifically these are 802.1q inside 802.1q. Both packets are tagged with 0x8100.
Traditionally the standard for this would be to use Q-in-Q with an outer 802.1ad VLAN and 802.1q VLANS inside. I can see that changes were made in OPNsense to meet this "standard" behaviour and to become more compliant last year.
However, my carriage service provider will drop packets tagged with 0x88a8 (802.1ad), so my ISP uses 802.1q inside 802.1q to work around this issue.
Currently I have to manually edit two files (src/opnsense/scripts/interfaces/reconfigure_vlans.php and src/etc/inc/interfaces.inc) and change the logic which sets the parent to 802.1ad if it has children. I simply change the conditional to hard set $vlan['proto'] to '802.1q'. This obviously won't survive an OPNsense patch update.
Describe the solution you like
I would like a override for this, located perhaps in Interfaces > Settings >Network Interfaces, perhaps with a name like "Q-in-Q 802.1q Override" or "Stacked VLAN Override" with a tick box for "Force Q-in-Q Parent to use 802.1q". The full help could advise something such as "Checking this option will force all parent VLANs which contain child VLANS (Q-in-Q) to be tagged as 802.1q (0x8100) instead of the default 802.1ad (0x88A8)".
Describe alternatives you considered
I discussed this in the forums and someone suggested that this should be a per-VLAN setting rather than global, and that sounds ambitious but potentially over-engineered for this specific use case. I'd be happy with a global setting to start with, and if someone wants to further enhance this afterwards that would be fine.
Additional context
Original forum thread:
https://forum.opnsense.org/index.php?topic=34617.msg167760
I'm keen to get this implemented so I won't have to manually patch files each time an update comes out. At this point in time I also can't update anything remotely as I would lose connectivity and would not be able to fix remotely.

I'm not against making this configurable, but it should indeed be set per vlan to avoid future issues.
Dumping the relevant spots (also in the forum) here for reference:
https://github.com/opnsense/core/blob/c3db32722d48335f7a9f396d729df2d2847669b1/src/etc/inc/interfaces.inc#L164
https://github.com/opnsense/core/blob/24dbe30cadd83fa5a1fcea8ed6b38357794a4d6a/src/opnsense/scripts/interfaces/reconfigure_vlans.php#L77

@wizdude this https://github.com/opnsense/core/commit/6487a9b62e31c253392f5fe2985da11b8225f99e should do the trick, it adds an advanced option for manual proto selection in the dialog.
To install, I expect the following should to the trick (revert with opnsense-revert  opnsense)
opnsense-patch 6487a9b

this is awesome. thanks. i've tested it on my lab box and it works well. i'll put it into production as soon as I can organise a suitable maintenance window.
does this mean that this new feature will automatically be part of future updates?

I think this will be in 23.7.


Web3 connect not happening for dapp

Hi
Love the web3 in browser
I am the cto of hipr.one a puzzle game on ethereum
.
For some reason we have it working in trust and metamask but we cant get it moving on opera. Https://github.com/hercone/restful-hipr is the code base.
We're also working on the up and an up and coming project CryptoFootball.
cryptofootballteaserstadium_test0001

Any help is appreciated , would be really cool if we could be on the home page of the dapps browser..
May I suggest make it A LOT easier to find the wallet.
The on boarding is awesome and I look forward to working with your team moving forward :)
Cheers!
LrG

Hi @lrgeoemtry,
Please contact the crypto-wallet team directly via<EMAIL_ADDRESS>The difference between us and Metamask is that we don't inject web3.js automatically which breaks some dapps. That is done intentionally as the dapp should be self-contained (as any other web-page) and shouldn't rely on JavaScript injection. See also Metamask's FAQ:

You can use the injected Web3.js directly, but the best practice is to explicitly bundle the version of web3.js you used during development.

Sent. Please close this issue @serso


Improve ComGS classes to deal with multiple antennas, frequencies, and modulation types

Overview
Improve ComGS classes to deal with multiple antennas, frequencies, and modulation types.
Details

To deal with multiple antennas

Need to have position and direction information in vector type like other components.
Need to select adequate Rx antenna responds to spacecraft attitude

To deal with multiple frequencies(S/X, up/down)

Implementation for up/down frequencies is easy.

Change ANT member variable frequency_ to tx_frequency_ and rx_frequency_ and add some programs.

Which components should have information about frequencies?

Making the TRP class and leaving the info for this class is good?
But how to have consistency with the current XTRP class?

How to check frequency matching with TX and RX antennas?

To deal with multiple modulation types

Modulation type, BER, coding type are now constants by implicit assumptions but should be variables.
It is required if users want to use this S/W to simulate both S and X bands.

Conditions for close
When the new ComGS class is implemented.
Supplement
N/A
Note

Write a conclusion when closing the issue.

このissueもう終わっている気がするので、一旦整理

Need to have position and direction information in vector type like other components.

搭載方向情報は実装済み。
位置情報はGNSSアンテナには重要だが、通信用アンテナでは現時点では重要でない気がする。将来GNSSアンテナと通信アンテナが統合されることがある時に考えれば良さそう？

Need to select adequate Rx antenna responds to spacecraft attitude

選択するのは、搭載アルゴリズムなどがやることなのでS2Eの担当範囲ではないので、実装しないで良さそう

Change ANT member variable frequency_ to tx_frequency_ and rx_frequency_ and add some programs.

これは既に実装済み

Which components should have information about frequencies?

現時点ではAntennaクラスが周波数情報を持っている

Making the TRP class and leaving the info for this class is good?

TRPクラスは作ったほうが良さそう

How to check frequency matching with TX and RX antennas?

これはいまいち意図が分からない

Modulation type, BER, coding type are now constants by implicit assumptions but should be variables.

これは途中で使う変調方式などを変える場合に必要。別のissueで設けるので良さそう。

It is required if users want to use this S/W to simulate both S and X bands.

一応今もS/Xを併用できはする。

ということで、一旦これは閉じて変調方式を制御できるようにするissueをたてる


docs: add documentation for replicated entities

Resolves #146.
Add docs for replicated entities, with replicated counter map shopping cart samples for JavaScript and TypeScript. These docs don't yet go as far as the Java SDK docs for replicated entities, which have an example for every replicated data type.
This only documents replicated entities as they are now. There's no codegen for replicated entity, so that's not covered. And typing for replicated data could be improved (making replicated entity and replicated data types generic) but currently just documenting as is.

Needs changes from #189 applied to the new samples.


mod_pagespeed behind https terminating proxy ignores X-Forwarded-Proto header

I'm running HAProxy, forwarding to Apache running mod_pagespeed. HAProxy terminates the https connection, so I make a regular http connection to Apache, passing a Http-X-Forwarded-Proto: https and X-Forwarded-Proto: https headers with the proxy request.
However, mod_pagespeed rewrites my css where I use a relative URL:
background-image: url(/test.png);
becomes
background-image: url(http://test.com/test.png);
Is it possible for mod_pagespeed to respect the X-Forwarded-Proto header and use https instead, or can I manage this in some other way?

Many thanks - I even read that page (I think) and somehow missed that!


.html() fails on writing two attributes when passed an argument

Writing the following code:
x$("#miniCartHeader").html('top', "X");

Prints the following:

X

I added a test in f730dab but cannot reproduce.
Which version of xui and also which browser/device do you see the issue with?


Do not re-use transceiver when negotiation is pending.

When a track is unpublished and published again, it is
possible that track events are not fired on client side
due to renegotiation debounce and transceiver re-use.
Although SSRC should change, looks like a re-use of
transceiver is not firing events.
So, when a renegotiation is pending, do not re-use
transceiver. This does affect efficiency of SDP when
there are too many track subscribe/unsubsribe is bunched
in a debounce interval. But, hopefully it is not a big
issue in real life scenarios. One option is to reduce
the debounce interval (from the current 150 ms) a little
bit to trade off potentially more re-negotiations for SDP
efficiency.

change looks good!


Add support for tofu custom releases URL

Thank you for your help @rogerxaic I will check this in the following days 👍

Thank you for your contribution :+1:

Thanks @alfespa17!
Sorry I missed the .zip part, MacOS logic apparently doesn't use the release's architecture value, and I hadn't properly tested it.
Is there any way you could trigger a release of this library, please?

Sure I think a can release some beta so you can test in more detail

It should be ready in some minutes.
https://github.com/AzBuilder/terraform-spring-boot/releases/tag/0.14.0-beta.1


build: update dependencies

Feature or Problem

Update dependencies.
This will require wasmCloud 0.80
Related Issues

Release Information

Consumer Impact

Testing

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin
[ ] x86_64-windows

Tested on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin
[ ] x86_64-windows

Unit Test(s)

Acceptance or Integration

Manual Verification

Isn't wash all over in wasmcloud/wasmcloud now or are we continuing work here?

@thomastaylor312 I partially migrated the history, but we can't archive this repo until we set up CI in wasmCloud/wasmCloud. So we'll need to do another subtree merge once we're ready to do that

please merge?


Replace Bootstrap 5 carousel with BlazorStrap?

Should the Bootstrap 5 carousel on the home page be replaced with something like the BlazorStrap carousel?

It looks as if BlazorStrap still uses Bootstrap behind the scenes (and Bootstrap 4 too), so not sure there's anything to be gained by doing this.
I'll keep looking for alternative ways to do the carousel though, because Bootstrap 5 inteferes with Tailwind at times, and ideally I would like to remove it entirely.


Fixes 52: Add EditContent Modal

This PR adds the ability to Edit repositories within a list.
Bulk edit functionality is also complete within the modal should we need it.
We would just need the multi-select boxes/Bulk Edit button/BulkEditAPI, to see that unrequested "feature" complete. 😜
Additional Changes:

Introduced new "single item" UI for both AddContent and EditContent (more streamlined).
Fixed a bug associated with the textfield on the "name/url" field on filters not being cleared properly
Fixed a bug associated with the versions not displaying/being stored correctly.
Fixed a bug associated with AddContent Modal validity after removing items.

https://issues.redhat.com/browse/HMSCONTENT-52

If I snap my window to the side of my screen, the modal closes. Is that working as intended?

Unfortunately, the entire page gets remounted when you do that. This is due to the parent switching from a mobile/desktop wrapper. There's a specific breakpoint (1200px) that this happens at, I'm not sure if there's a workaround.


add environment variables

https://keboola.atlassian.net/browse/KCOFAC-1764
@davidesner

Add new version in setup.py and merge to prod


Landing page footer broken on Safari

I can confirm this bug, can you fix it @ygorcosta?

Yes!


Mapping custom webparts with the Use of Webpartmapping xml

Hi Guys,
This is more inline to a query than an Issue. Need your guidance.
I am currently trying to map a content editor from Source to a react community script editor taken from Here.
The content editor contains a content under Edit Source - HTML button to simply trigger an alert.
The page transformation script is used as per the example and The Script editor SPFx app is compiled and is added as an app in the target site.
PageTransformationInformation pti = new PageTransformationInformation(page) {     // If target page exists, then overwrite it     Overwrite = true, }; pti.MappingProperties["UseCommunityScriptEditor"] = "true"; pageTransformator.Transform(pti);
Post transformation, I am able to see a text webpart saying - Content editor contains script and could not be loaded.
I'm assuming that I must be missing to change something in the webpart mapping XML. As of now it is in its vanilla form.
<Mapping Name="NonASPXUseCommunityScriptEditor" Default="false">             <ClientSideWebPart Type="Custom" ControlId="3a328f0a-99c4-4b28-95ab-fe0847f657a3" Order="10" JsonControlData="{"serverProcessedContent":{"htmlStrings":{},"searchablePlainTexts":{},"imageSources":{},"links":{}},"dataVersion":"1.0","properties":{"script":"{FileContentsEncoded}","title":"{Title}","removePadding":false,"spPageContextInfo":false}}"/>           </Mapping>           <Mapping Name="ContentUseCommunityScriptEditor" Default="false">             <ClientSideWebPart Type="Custom" ControlId="3a328f0a-99c4-4b28-95ab-fe0847f657a3" Order="10" JsonControlData="{"serverProcessedContent":{"htmlStrings":{},"searchablePlainTexts":{},"imageSources":{},"links":{}},"dataVersion":"1.0","properties":{"script":"{Script}","title":"{Title}","removePadding":false,"spPageContextInfo":false}}"/>           </Mapping>
Kindly help me in sorting this out.
Thanks

Breakthrough, I was able to read from the Content Editor and pass the content onto a Script Editor.
Currently I am trying to read a file in the ContentLink of the Content editor and this seems to be a Real issue. I am getting the below errors every time there is a file added in the contentlink a simple html or javascript.
`Built-in Function  |  An error occurred in LoadContentFromFile function    at Microsoft.SharePoint.Client.ClientRequest.ProcessResponseStream(Stream responseStream)
at Microsoft.SharePoint.Client.ClientRequest.ProcessResponse()
at Microsoft.SharePoint.Client.ClientRequest.d__6.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Microsoft.SharePoint.Client.ClientRequest.d__0.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Microsoft.SharePoint.Client.ClientRuntimeContext.d__0.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Microsoft.SharePoint.Client.ClientContext.d__4.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Microsoft.SharePoint.Client.ClientContextExtensions.d__7.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Microsoft.SharePoint.Client.ClientContextExtensions.ExecuteQueryRetry(ClientRuntimeContext clientContext, Int32 retryCount, Int32 delay, String userAgent)
at SharePointPnP.Modernization.Framework.Functions.BuiltIn.LoadContentFromFile(String contentLink)`

Hi @AllanSamuelT ,
I assume you've seen the docs around using the script editor web part as transformation target: https://docs.microsoft.com/en-us/sharepoint/dev/transform/modernize-userinterface-site-pages-webparts#enabling-the-use-of-the-community-script-editor-web-part-to-be-used-during-transformation
Can you show the content link you're using? It must be a server relative url (so starting from /sites/...). The error you get is thrown in below code block:
try
                {
                    // Content editor does allow a web part on a sub web to point to a file in the rootweb...Pointing to files outside of the current site collection is not allowed
                    Uri hostUri = new Uri(this.clientContext.Web.GetUrl());

                    // Find the web url hosting the content file
                    var webUrlResult = Web.GetWebUrlFromPageUrl(this.clientContext, $"{hostUri.Scheme}://{hostUri.DnsSafeHost}{contentLink}");
                    this.clientContext.ExecuteQueryRetry();

                    using (var cc = this.clientContext.Clone(webUrlResult.Value))
                    {
                        return cc.Web.GetFileAsString(contentLink);
                    }
                }
                catch (Exception ex)
                {
                    LogError(LogStrings.Error_LoadContentFromFile, LogStrings.Heading_BuiltInFunctions, ex);
                    return "";
                }

Hello Jansen @jansenbe,
Yup, I've read the documentation.
I am attempting to transfer the content (contains only HTML, no JavaScript) onto a Text webpart on the Canvas(Preferably) or a community script editor. I can ignore the source html file once the content is transferred onto the page canvas.
The URL is the ContentLink goes like "/Sites///Documents/htmlfile.htm".
How should I proceed with this ?

I ran the code block in my environment and found that the line return cc.Web.GetFileAsString(contentLink); - is returning File Not Found.
The url being passed is a server relative url. "/Sites/Subsite/Subsite/Documents/paintext.txt" - passed a text file.

If the content contains no html it will end up as a text part, we'll only use the script editor if the content contains JS and if you've enabled using the script editor. Seems like in your case something goes wrong with reading the file the content editor points to...Would you be able to debug this by pulling down the source code? Source code lives here: https://github.com/pnp/sp-dev-modernization/tree/master/Tools/SharePoint.Modernization. Function that fails is https://github.com/pnp/sp-dev-modernization/blob/master/Tools/SharePoint.Modernization/SharePointPnP.Modernization.Framework/Functions/BuiltIn.cs#L1006.
I also assume you did not update the web part mapping files and are using the built in one?

Hi @AllanSamuelT ,
Did you manage to debug this? If you're OK then we can close this issue?

Closing this issue as there was no feedback for the last 3 weeks. If this is still an issue please open a new issue and cross reference this one. Thanks.


Update howto-restrict-your-app-to-a-set-of-users.md

Changed "user" to "users" in the second sentence.

@ms-johnalex - nice catch. thanks!


Replicating the query explanations from Section 5.5/Figure 3

Hey thanks for releasing the code - is it possible to replicate your results from section 5.5 on explaining the answers to the queries?

Hi! Sorry for the super late reply -- @dfdazac has some code for that I think!
More recently we reimplemented CQD in @hyren et al.'s https://github.com/snap-stanford/KGReasoning/: https://github.com/snap-stanford/KGReasoning/pull/9 -- for analysing the intermediate variable assignments it should be enough to check their values in discrete.py: https://github.com/pminervini/KGReasoning/blob/main/cqd/discrete.py#L36

Hi @pminervini - no problem, thanks so much for getting back to me. Thanks for the pointers to the KGReasoning repo, i'll check it out.
I actually have two more questions if that is ok.

Do you have any plans to demonstrate how the approach could be run of a different datasets, including data preprocessing etc.
In trying to understand the beam search approach you discuss in the paper, am I right in thinking that there are no trainable parameters used in this?

Hi @sbonner0, @dfdazac is looking into merging the code for explanations in this repo!
About your questions:

At the moment we are using the datasets available in https://github.com/snap-stanford/KGReasoning/ -- we haven't looked into generating our own evaluation data yet, but that's definitely on the TODO list (e.g. it could be interesting to analyse the behaviour of the model on way more complex queries).

Yes! It uses an off-the-shelf ComplEx-N3 model trained on 1p queries, and there are no additional trainable parameters. It could be fun to explore models with a few additional trainable parameters, e.g. for learning parametric t-norms -- let us know if you plan to look into this :)

@pminervini  @sbonner0  I am currently working on the query generation for more complex queries and other datasets. It is working for our specific setting, but I want to generalize it a bit and separate it from the rest of the codebase.

Thanks @pminervini @miselico I look forward to hopefully being able to run your approach on other benchmark datasets such as WN18RR as well as getting some explanations.
@pminervini just to dig a little more into the second point - so given the ComplEx embeddings are pre-trained, there is no real concept of training data for the beam search based approach - or am I missing something entirely?

@pminervini just to dig a little more into the second point - so given the ComplEx embeddings are pre-trained, there is no real concept of training data for the beam search based approach

That's correct!

@sbonner0 thanks for opening this issue initially. I have merged some code in main and an explanation in the README regarding how to generate the explanations. The idea essentially consists of mapping the IDs of the top-k selected entities at each step to their actual name string, and then formatting everything nicely.


Feature Request: Option to disable warnings when projects aren't found.

I love Brackets, and use it constantly. I also use Transmit FTP to mount my sites as local disks. This works great, but when I switch between projects (and unmount those mounted discs), Brackets always pops up an error dialog telling me that it can't find the project any more. That's probably fine for people using local files only, but it's annoying for users like myself who work on remote files. If we could just disable those warnings, or even had them more subtle, and not have to close them every time, that would be great!

Alerting the user that the project path doesn't exist on disk/ cannot be located looks like a valid use case to me. If we suppress these messages, then unsuspecting users will not be able to get what went wrong.
If you like to alter the behavor, goto src/widgets/Dialog.js in the brackets source and modify the showModalDialog part or do a more targeted change.
Closing as expected behavior/ edge case.


Add flip expect animations and example activity

I've added Flip Animations and CameraDistance Animations to the system. I also thought about adding a possibility to cull the faces depending on the current rotation, but then I'd have to consider some solutions for this first.
If you like the idea of the implementaion of this flip animation I'd appreciate it if you'd review it @florent37 :)

yes I like it, just one think the flip is always a 360° flip ? if no, it should change the view width

this should extend RotationExpectation I think

Okay I will have a further look into that! So as I understood you'd also let FlipExpectation extend RotationExpectation, are there special reasons for that? :)
Currently I've set the flip to a 180 degree flip. I think this might be useful when having two-sided views. But I think I probably should implement some kind of backface culling? What do you think?

It should extend RotationExpectation because I use this expectation to know the final rotation of the view. And depending of this rotation, I will know the futur dimensions of the view.
So if the flip is fully, the dimension stay the same,
if not, it will change

Hmm okay makes sense. So I guess it depends on you, since I could not think of a scenario where the user actually wants to rotate the view on its x- and y-axis by a non 180 degree angle. That's why I did not take a change of dimensions into account.


[Dev] Block sync up still cost very long time

Description
Block sync up with user transactions still got very long time, sometimes maybe longer than one block generate time cost. And currently each block with transactions less than 1000 txs.
Few block sync up time cost log in below:
Current Round : 70. Current height: 226
[2018-12-12 06:50:47.829 - Info - BlockExecutor][26]: Executed block ac10930d26dd6b64e594a52006e7b0890a7536d44bdab44564ff5fc9fa07c92f with result Success, 556 txns, duration 6534 ms.
Current Round : 70. Current height: 227
[2018-12-12 06:50:53.501 - Info - BlockExecutor][36]: Executed block 3646decc1be421a17f492b80fc48daccbb4ba42e2d9ab54d1a115c2a3a809ad5 with result Success, 457 txns, duration 5630 ms.
Current Round : 70. Current height: 228
[2018-12-12 06:51:01.659 - Info - BlockExecutor][20]: Executed block 20af904607b40434833c04f027fbeabe785b54167976893c936a2ff52f40896c with result Success, 670 txns, duration 8133 ms.
Current Round : 71. Current height: 229
Current Round : 71. Current height: 230
[2018-12-12 06:51:05.068 - Info - BlockExecutor][26]: Executed block 711b2876f729dd2ee2de96238593cdca0e4cc63e30895bee65ee16e68edaf2ab with result Success, 272 txns, duration 3338 ms.
Current Round : 71. Current height: 231
[2018-12-12 06:51:10.098 - Info - BlockExecutor][19]: Executed block cd599c32210c561075c84fb378fbe56af67dd67570756c32bcf5fbcb862dd3f5 with result Success, 409 txns, duration 5009 ms.
Current Round : 72. Current height: 232
[2018-12-12 06:51:11.950 - Info - BlockExecutor][27]: Executed block bec0582cc4f868193df535241e0423bcaa467b20612d3539f26dbc36e86ecfe3 with result Success, 178 txns, duration 1825 ms.
Current Round : 72. Current height: 233
Current Round : 72. Current height: 234
[2018-12-12 06:51:14.146 - Info - BlockExecutor][19]: Executed block 998459d795161d79d70887b4b90ec3661f780de1b4ed0dcb5bbef5643a941bae with result Success, 207 txns, duration 2095 ms.

Sync up node.log.zip

Reference #775


Will This work with a ESPHome Bluetooth Proxy, or does it need to connect to the Bluetooth radio on my device running home assistant?

Will This work with a ESPHome Bluetooth Proxy, or does it need to connect to the Bluetooth radio on my device running home assistant?

Probably several months late, but in case anyone else looking, have got this working with ESPHome proxy (M5Stack Atom Lite using stock firmware from the website) with my tuya ble garden irrigation controller 😊 [markusg1234 fork with a small code adjustment to add support for single irrigation controller]

@richm90 do you mind publishing what the code change was, which computer did you add?


Add a command line option to merge the srt file and video

Description of proposed feature

If use addcaption function there will be a srt file. Which I think should give a command line option to auto merge the srt file to video.
How can the new feature be used?

Something like manim -pql --merge xxx  or manim -pqlx xxx
Additional comments

In some sense, this is against the spirit of subcaption files. If you want to render text in your video, why not simply use Text or Tex?
I'm happy to hear further arguments, and in case there is a striking use case for this feature we can reopen it -- but for now, I'll close this.


Course Resources Link Broken

When course resources link clicked getting "This site can’t be reached"

Hmm I've just tried it again in a few different browsers and it works fine. I wonder if you have a firewall or VPN or something that would stop Notion or Dub.so from working. Could you give me some info about your setup/location so that I can figure out if this link doesn't work in certain regions or setups?
Here's where it forwards to - https://prismic.notion.site/Fizzi-3D-Website-Course-Resources-9f23c46433f04e749b92aafaeeeb8630

I don't have any firewall or VPN, most likely this is a network-related issue from our region or ISP. I'm from Manila, Philippines. Thank you for the link resource!


adds REMOVE support to UpdateExpressions

Adds a new method to the DyanmoField struct for generating REMOVE update expressions.
DynamoDB Docs:
http://docs.aws.amazon.com/amazondynamodb/latest/developerguide/Expressions.UpdateExpressions.html#Expressions.UpdateExpressions.REMOVE

Build failure? Lgtm otherwise

Failure might be due to the localstack version bump, Karl discovered.

pinned localstack to 0.4.2