fix: cannot delete service registry by name

Fixes: https://github.com/redhat-developer/app-services-cli/issues/785

@craicoverflow this fixes broken documentation on master and also minor bug in registry delete.

I have tested that before and now:
[wtrocki@graphapi app-services-cli (main ✗)]$ rhoas service-registry create test
Creating Service Registry with name: test
Successfully created Service Registry
{
  "href": "",
  "id": "121",
  "name": "test",
  "owner": "Unauthenticated",
  "registryUrl": "",
  "status": "PROVISIONING"
}
[wtrocki@graphapi app-services-cli (main ✗)]$ rhoas service-registry delete test -y
Deleting "test" Service Registry instance.

Service Registry instance "test" was deleted.

Rebased. Ready for review

Not able to test it in either mock or staging, is it expected?

Pasting the debug log:

@rkpattnaik780 sadly enough backend folks broken entire CLI

Backend we have provisioned is broken but running it locally with date fix works. Merged as there are multiple fixed that I needed in follow up PRs


"+more" is not displaying in Custom Event when navigate from/to Week/Day/Agenda or Back/Next clicks from initally rendered month view

Do you want to request a feature or report a bug?
bug
What's the current behavior?
As per use case event is required to display in multi line, this is good in month view when initial load/rendered. But when I navigate from/to Week/Day/Agenda or Back/Next buttons clicks then "+more" is not displaying any more noticed events are fully occupied date cell.
Initial page load or when page refresh displays +more:

After navigation not displaying +more:

Project used below calendar version:
"react-big-calendar": "^0.22.1"
https://codesandbox.io/s/flamboyant-hill-zfrqr?from-embed
What's the expected behavior?
'+more' should display when navigate from/to 'Week/Day/Agenda' or 'Back/Next' clicks

https://codesandbox.io/s/flamboyant-hill-zfrqr?from-embed

have same issue


update README with development steps

Add local development section in README with steps:

to enable Chrome dev tool: console logs, network, source, etc.
enable hot reloading: to use local ui code without rebuilding

ref: https://www.docker.com/blog/build-your-first-docker-extension/

@ibreakthecloud ill do this

@ibreakthecloud ill do this

sure. Assigned!


Arrange into project folders.

To prepare for asset library usage by making the folders not clash with the base project when installed.

should we move the demos to the addons dir instead?

they are not necessary there so I don’t ynderstannd.

move to res://addons/ply/demo/demo.tscn instead of res://ply/demo.tscn, then there's no chance of conflict at all -- we can just kill the script on the demo scene as well, i just used it as a pseudo-repl for scratch testing. although whatever is standard for plugins is ok


Deleting EventTypes when changing sink to a non-Broker kind

Proposed Changes

Changing event type reconciler to delete EventTypes of Sources whenever they point to a non-Broker sink.
Added UTs.

Release Note

NONE

/assign @n3wscott
last one Scott, I promise :)
This is the eventing-sources hopefully last registry fix :)

This should be merged before the 0.6 cut.

/lgtm
/approve


unknown message

There seems to be an error with the ping iq messages.
2024-05-09 18:08:55.395 | DEBUG    | pyjabber.network.XMLProtocol:data_received:97 - Data received: <iq id=":hyBXT6OCXTkjcG8vWi6s" type="get"><ping xmlns="urn:xmpp:ping"/></iq>
2024-05-09 18:08:55.396 | ERROR    | pyjabber.network.XMLProtocol:data_received:117 - SAXParser Error parsing XML: <unknown>:1:1077: parsing finished

Implemented ping xep in f43e7f3885fbf20540d890e9082032bb8a2df151


(docs)Makefile: document the origin of PULL_BASE_REF

It's just a tiny thing, but hopefully it will save the next reader a few minutes of searching.

/approve
/lgtm


Publishing recommended feeds

I'm exploring the idea of publishing "suggested feeds" in a discoverable way. For example, a website owner may wish to promote similar sites, or sites to which they personally subscribe. This use case is a little different than what's described as the feeds belong to different sites and have a different author.
Is this use case something this RFC would support?
As a concrete examples, I'm trying to find a way to publish my feedland feed list (https://feedland.com/opml?screenname=robalexdev) to make it easier for people on my website to discover those feeds. Eventually, it would be nice if RSS user agents could show those suggestions in a social network sort of way.

Interesting! I can't see any reason why not. OPML's rich enough that it could easily be used to provide several groups of content, including those both on and off the site, and both "belonging" to and merely "recommended" by the site.

@scripting proposes an alternate scheme for publishing a blogroll, BTW, which is more-like what already exists and so more-likely to gain traction. Worth a look: https://opml.org/blogroll.opml

That's exactly what I was looking for. Thank you!

For completeness, I also found .well-known/recommendations.opml in use for this purpose:

https://www.manton.org/.well-known/recommendations.opml
https://www.google.com/search?hl=en&q=.well-known%2Frecommendations.opml


Update http4s-circe, http4s-dsl, ... to 0.23.15

Updates

org.http4s:http4s-circe
org.http4s:http4s-dsl
org.http4s:http4s-ember-server

from 0.23.14 to 0.23.15.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.http4s" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "@monthly" },
  dependency = { groupId = "org.http4s" }
}]

labels: library-update, early-semver-minor, semver-spec-patch, commit-count:1

Codecov Report

Merging #344 (3acb98f) into main (73a2b94) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #344   +/-   ##
=======================================
  Coverage   88.76%   88.76%
=======================================
  Files          91       91
  Lines        2812     2812
  Branches      157      157
=======================================
  Hits         2496     2496
  Misses        316      316

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Remove dead code for fixup sequence points

Whilst looking at a portable PDB issue I noticed that, fixupOverlappingSequencePoints  was only ever set to false.  So that allows me to remove some code, arguments, fields et.c.
Anyway here it is:

Good job


add SVSM_CAA ovmf section type

implementing the discussion in https://github.com/virtee/sev-snp-measure/issues/50#issuecomment-2130406004
The issue was closed with a corresponding pull request and I encountered it in my work as well

Thank you for your contribution @Garandor . However, the same patch has been suggested in #53 ; while submitted a few minutes after this PR, #53 contains tests cases and some docs, so I'm going to continue there. Feel free to comment and join the discussion in #53 and test it.
Hopefully next time your PR will be accepted!

👍 Just wanted to make sure it won't fall under the rug


fix(kcodeblock): not searching long content with short queries

Summary
Removes the limitation to only trigger a search when (1) a piece of code is shorter than 1000 characters or (2) the query is greater than 2 characters.
PR Checklist

[x] Conventional Commits all commits follow the conventional commit standards outlined in the main README.
[x] Tests coverage: test coverage was added for new features and bug fixes
[X] Docs: includes a technically accurate README

Comment: The Percy failure is in the date picker component so unrelated to this change.


Missing svg elements

https://developer.mozilla.org/en-US/docs/Web/SVG/Element

... missing in what way?

It does not minify svg elements.

Because you are doing opt-in instead of opt-out, aka parse a fixed element list?

Yes, I have like a whitelist of elements that should be minified. Svg elements are not yet included.

@inoas SVG files as well as Custom Elements are now minified. I no longer keep a "whitelist" of elements. Much smarter now I think and I even saved a 100 rows as bonus.


Edit metadata in MusicList.vue generating error

Clicking on "Edit Metadata" in the MusicList.vue component causes an error that prevents further use of the application. Requires refreshing the page to restore functionality.


Close and Open code pane breaks syntax highlighting

observe syntax highlighting
close code pane
open code pane
observe different/broken syntax highlighting

Not going to assign anyone, but CC @franknoirot and @jessfraz, (not sure if editor or pane problem)

Super weird because it’s all just codemirror now, maybe I fucked up the codemirror react thing when we got rid of the other plugin?

I 100% am convinced it’s the react component I made, the theme gets set thru there, I’m sorry I literally am not good enough at react to properly figure this out

Mostly because admittedly I didn’t know what I was doing in the first place

Maybe @lf94 can help 🙏

Moved to high priority since it’s a regression and we can get qa wolf to add a test

Roger


Error when installing own library dependent on another own library

Be sure to check the existing issues (both open and closed!), and make sure you are running the latest version of Pipenv.
Check the diagnose documentation for common issues before posting! We may close your issue if it is very similar to one of them. Please be considerate, or be on your way.
Make sure to mention your debugging experience if the documented solution failed.
Issue description
When trying to install its own library (with pipenv 2023.11.15), which in turn uses (among others) its own library, it fails when installing the latter.
This behavior does not occur with version 2023.4.29. I didn't try it with other versions.
Expected result
Install all libraries.
Actual result
No matching distribution found for mylibrary1==2.0.4
Steps to replicate
1.Create library (mylibrary1). Publish in private index.
2.Create the library (mylibrary2) whose 1 of its dependencies is mylibrary1. Publish to the same private index.
3.In a new project, install mylibrary22: pipenv install mylibrary2

What does your Pipfile look like -- is the first and default index your private index?  I suspect its pypi or something else and so everything that relates to a secondary index must be specified in the Pipfile.

If it were the pipfile, then it wouldn't work with version 2023.04.29 either or did you change the way it should be configured?
Pipfile of my project:
[pipenv]
install_search_all_sources = true
[[source]]
url = "https://nexus.cloud.com.ar/nexus/repository/pypi-proxy/simple"
verify_ssl = false
name = "pypi"
[[source]]
url = "https://nexus.cloud.com.ar/nexus/repository/infraestructura-pypi/simple"
verify_ssl = false
name = "infra"
[packages]
uvicorn = "==0.23.2"
mylibrary2 = {version = "==0.0.12", index = "infra"}
[dev-packages]
pytest = "==7.4.0"
httpx = "==0.24.1"
locust = "*"
[requires]
python_version = "3.8"

hello. Any news on this ?

Hello.
I'm not sure I understand the comment correctly.
Currently the mylibrary2 library uses many dependencies (oracledb, fastapi and mylibrary among others).
When I use mylibrary2 in a project, it correctly installs all the libraries (with pipenv==2023.4.29).
With the version of pipenv==2023.11.15, it installs all the libraries correctly, except mylibrary.
In the latest versions, do I have to specify to install mylibrary verbatim? I don't like this very much, since developers will have to know which libraries mylibrary2 uses internally, my idea is to simplify this for them.
Note: In my company, we have an index for all the libraries that exist in pypi (it is like a pypi proxy) since we do not have internet access.
Additionally, other libraries we created have their own internal index, such as mylibrary and mylibrary2. That's why in Pipfile I define the two index sources (pypi-proxy and infraestructura-pypi).

Currently the mylibrary2 library uses many dependencies (oracledb, fastapi and mylibrary among others).

It will default all non-specified indexes to the first index, so as long as they mirror pypi, you shouldn't have to re-specify those, but everything from the secondary indexes needs to be called out explicitly at this point in time.   There are notes in our docs about this regarding the secondary package indexes and the recommendation to mirror relevant pypi packages there so it mitigates the need for a secondary index.

https://pipenv.pypa.io/en/latest/indexes.html#index-restricted-packages


Update test_string.py

Location of errors have changed

The code changes look great. But string.py looks like to be updated from CPython 3.8.3 but test_string.py is not - it seems it is the 3.8.3 test file but it was already from the version but not updated by this commit. Could you reword your commit message for test_string.py to not to mention CPython 3.8.3?

+Edited as requested

#3453 is just merged - which resolves new errors. Could you rebase this PR once again without new expectedFailure decorators?

@waroad thank you for contributing


Issue displaying a 3D Tiles, tileset with a gltf that have two sided material

Hi! I’m working on a project with Cesium for Unreal (tag 2.7.0), where I’m trying to load a tileset containing a glTF model with double-sided materials.
Since double-sided materials aren’t enabled by default in the glTF layer material (ML_CesiumGltf) within the plugin, I created a custom material. This material takes all output from the "CesiumGlTFFunction" material function, and just have two-sided rendering enabled. I then applied this custom material to my tileset.
However, now I’m seeing an issue where one side appears very dark, and I’m not sure what’s causing it.
If anyone has suggestions on how to resolve this, I’d greatly appreciate the help!
Here’s a screenshot of what it looks like in CesiumJS:

And here’s how it looks in Unreal:

I’ve also attached one of the tilesets exhibiting this issue:
testtileset.zip
Thanks in advance!

Okay I figured a fix, if I make my material  unlit, and use color as emissive it works, but I think there might be an issues on how the shadow of the gltf is computed, cause this is really weird and haven't got this issues with other models.

@HouleAdrien see this bit in the glTF spec:

When this value is true, back-face culling is disabled and double sided lighting is enabled. The back-face MUST have its normals reversed before the lighting equation is evaluated.

I think this is most likely what your custom material is missing.

Thanks this is indeed what I was missing in my case, so i'm closing this issue on that :)


rename to mini-Castro

it might be best to rename this to reflect it's heritage to Castro

this has been done


Update the writing style guide 'Style conventions' section for Nimbus

Update the different sections in the 'Style conventions' page to reflect what rules apply to user documentation and what rules apply to infrastructure documentation.

This issue relates to actions for the Nimbus project that are no longer needed.


[Unity Plugin] Pre-Export Scene takes into account disabled gameobject

When having a gameobject (wall) infront of an audio source and using Pre-Export Scene the occlusion works as expected. When disabling the same gameobject and hitting Pre-Export Scene again the occlusion is still active. Deleting the wall and hitting Pre-Export Scene disables the occlusion.
Is this intended?

Thanks, this is not intended behavior, and is actually a bug. (It's not consistent with how dynamic geometry works; when dynamic objects are disabled they stop interacting with sound.) We have a potential fix for this, which will be included in the next release of Steam Audio.

Excellent. Looking forward to it.


Update Hangman Game Template

Update Button/UI Structure
Utilize the Theme System for the Button/UI
Add You Won, or You Lost Screen for when a Game is over, that will take you back to the Lobby List to find a New Game

This will also Refresh the List
Create Settings Menu
Add Mute Button
Add Global Volume Slidebar
Add UI Volume Slidebar
Create Scrolling Credits Menu with Json (Name, Title, Url)
Add Blazium Logo/Branding to the Template
Basic UI Sfx
Basic Music (Main Menu, Lobby List, Hangman Game, Win/Lose Screen)

I would like to add:

Removing the password CheckButton. The text of password should be by default enabled, and if you type anything, it just gets a password to the lobby.
Max Players -> LineInput as well, instead of using Label. Ofc gotta filter for numbers but that's easy.


Just the kernels

This PR does away with the title of this repository, exposing the full IPython/Jupyter kernels API. This means you can just request a kernel type directly:
In [1]: import requests

In [2]: requests.post("http://docker:8000/krn/api/kernels", json={'name': 'python2'}).json()
Out[2]: {'id': '019bb6ed-3d7d-42d4-9baf-cc7e25c35938', 'name': 'python2'}

In [3]: requests.post("http://docker:8000/krn/api/kernels", json={'name': 'python3'}).json()
Out[3]: {'id': '23ce3d7a-b444-4d41-92a4-992dca518f44', 'name': 'python3'}

Should I just add the kernelspecs API too?

This makes it tempting to add this __main__ to IPython.html.services.kernels.handlers, as it's the only component of this code that isn't just in the Notebook.

I like this a lot -- it will give a lot more fine grained control to people making different frontends for the kernels.

Alright, I'm going to go ahead and merge this and rename this to "kernels-service".


Multiple Allowed Url Parameters problem

Hey there.
If I set "ano, mes" in "Allowed Url Parameters" I get the following error:
There was an error while executing the configured data source: There is an error in your SQL at line 1: Incorrect syntax near 'nvarchar'. Must declare the scalar variable "@url_mes".
Module bug?
Best regards,
João

No bug, please read the documentation how you can implements URL parameters to carry forward to your SQL statement


Hey ! are you still alive ?

If yes kindly accept the pull request for that null safety migration 👍

It's done :)


Images built with Maven and jib are marked as made 52 years ago

$ docker images
REPOSITORY        TAG                             IMAGE ID       CREATED        SIZE
tomcat            9-jdk17-openjdk-slim-bullseye   c4fa5f231b6d   2 weeks ago    428MB
<none>            <none>                          5d614f747910   52 years ago   428MB
<none>            <none>                          f42cc5884380   52 years ago   428MB
<none>            <none>                          facd872d02f2   52 years ago   428MB
<none>            <none>                          07be1761f399   52 years ago   428MB

hello @behrangsa, you can find the reason here.

Thanks for the the helpful link @celcius112!


Continuous crawl randomly misses items

We have continuous crawl enabled for our sharepoint content
We find it will randomly miss items. These items are not picked up unless they are edited again
Is this expected behaviour

@jay1050 thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.


[Android FHIR SDK] - P2P FHIR Engine search on DateTime property not working

https://github.com/google/android-fhir/issues/1363

Done on SDK


Fatal startup errors don't exit immediately

The following results in a un-readiness error after a few seconds, instead of pretty-printing this error and immediately exiting:
{"is_panic":false,"message":"\u001b[1;91merror\u001b[0m: \u001b[1mEnvironment variable not found: DATABASE_URL.\u001b[0m\n  \u001b[1;94m-->\u001b[0m  \u001b[4mschema.prisma:10\u001b[0m\n\u001b[1;94m   | \u001b[0m\n\u001b[1;94m 9 | \u001b[0m  provider = \"postgresql\"\n\u001b[1;94m10 | \u001b[0m  url      = \u001b[1;91menv(\"DATABASE_URL\")\u001b[0m\n\u001b[1;94m   | \u001b[0m\n\nValidation Error Count: 1","meta":{"full_error":"\u001b[1;91merror\u001b[0m: \u001b[1mEnvironment variable not found: DATABASE_URL.\u001b[0m\n  \u001b[1;94m-->\u001b[0m  \u001b[4mschema.prisma:10\u001b[0m\n\u001b[1;94m   | \u001b[0m\n\u001b[1;94m 9 | \u001b[0m  provider = \"postgresql\"\n\u001b[1;94m10 | \u001b[0m  url      = \u001b[1;91menv(\"DATABASE_URL\")\u001b[0m\n\u001b[1;94m   | \u001b[0m\n\nValidation Error Count: 1"},"error_code":"P1012"}

This issue needs to either have some check in the engine startup or probably some general log reading from the engine which is streamed via jsonrpc, parsing it and then printing the message property and the code or something like that – I'm not so sure myself.

I'll take a look later and will ask for clarifications whenever i have to. thanks!


FormFlow and LuisDialog

I would like to use both FormFlow and LuisDialog in a combination.
For example, if you have an Intent OrderSandwich that initiate the conversation. This should follows a FormFlow that will ask the question and user can give the response in Natural Language with the help of Luis.
Conversation
User: I want to buy a sandwich?
Bot: Off course, we have A, B, C types
User: What is A? (Optional)
Bot: Its made up of......
User: Okay, I would like to take B
Bot: Would you like to have a medium or large
User: What is the difference between medium and large (Optional)
Bot: Medium cost xx.xx while large cost xx.xx but.....
User: Okay I will take the large one
Bot: Do you have any other questions
User: No thanks, bye
Bot: Have a nice day
Even one can ask more details about the types of Sandwich, but such queries are optional.
Anyone has tried or is it possible with current version of Bot Framework?

I've been trying to achieve similar things in the past few days and my conclusion is that it's not possible with the current implementation.
There are a few built-in commands in the form flow dialog (help, ?, status, quit, back) and I was thinking of adding a new one: "define" to achieve exactly what you described. Didn't have time to write the code though...

There is no way to do this in the current framework, but we are working on a general way of composing together dialogs that should help.

Do you have an open issue you could link so we can keep track on that progress? Would be interested to get notifications on this.

I've tried with LuisDialog by making intents and for each message exchange but its really difficult to maintain the conversation flow. I'd be more than happy to see such a conversational logic in the coming versions.

@chrimc62 In addition to above, the control words such as help, status, back should also be interfaced with Luis. For example to change a previous entered form field PizzaSize, allow user to enter "I want to change size". Similarly help, status etc would also follow.

@ganeshnj @ankitbko @chrimc62 @alexdrenea
Hi,
I am trying to create a bot using LUIS intents and i need a conversation flow. How to implement a context flow from language understanding?
Actual Result- i have intent named as "moisturizer" and its utterance is "do you have moisturizer" but when i retrain the model and test "do you have abcd" it still hit the moisturizer intent, which i don't expect. I want this utterance to hit to "None" intent.
Expected - I have huge data in my model, after hitting to top score intent i am using dialog for further conversation (also adding to this do you have another solution except FormFlow and Dialog?). So i don't want my dialog to initiate when user enters the partial utterance as mentioned above. please provide some solution to this.


Add ItemWidth and ItemHeight properties to WrapPanel

What does the pull request do?
Adds the ItemWidth and ItemHeight properties found in WPF's WrapPanel.
Checklist

[x] Added unit tests (if possible)?
[x] Added XML documentation to any related classes?

Fixed issues
Closes #3003

The test is failing in the case of the second child. I'm not sure why given that WrapPanel is pretty much a line-for-line copy of WPF's at this point. The second child shows that its PreviousArrange is the correct size but the Bounds end up being something else somehow. In WPF the second child would be the size 20,15 as I try to set here with ItemWidth and ItemHeight. If Width and Height of the second child are "Auto"/double.Nan, then it works as intended. I think this is a bug somewhere else in Avalonia that someone else would have to check.

@Kermalis your unit tests appear to be failing.

Yeah, check my previous comment, I'm not sure why it's failing because the control is a copy of WPF's so the problem might be elsewhere

Since the test was failing due to some other issue, I will create an issue for it once this gets merged. The control works and should be merged


Feat-cli-and-wasm

The following was added:

rhai-cli crate with the rhai executable, this currently has only the lsp in it, but I plan to merge the existing repl implementations and a formatter in the future
rhai-common for common types shared between the lsp and the cli (such as the Rhai.toml config, logging configuration, etc.)
rhai-wasm a wasm wrapper currently only exposing the LSP to javascript
js/core and js/lsp, these are the js libraries that are supposed to be consumed, they are not published anywhere and are used locally by the vscode extension, I squatted the @rhaiscript org over npm in case we want to publish them (e.g. for usage of the lsp in the browser).
the vscode extension can be standalone now with the LSP bundled with it.

I pretty much copied the architecture I use for publishing stuff for taplo.

the vscode extension can be standalone now with the LSP bundled with it.

Do we need to make pre-built binaries?  Or everything in wasm?

Do we need to make pre-built binaries? Or everything in wasm?

No need for platform-specific stuff, all WASM.

For some reason the latest drop fails to start:
Error: Cannot find module '@rhaiscript/lsp'
Require stack:
- c:\Users\Stephen\.vscode\extensions\tamasfe.rhai-0.0.0\dist\server.js
    at Function.Module._resolveFilename (node:internal/modules/cjs/loader:987:15)
    at Module._load (node:internal/modules/cjs/loader:832:27)
    at Function.c._load (node:electron/js2c/asar_bundle:5:13343)
    at Module.require (node:internal/modules/cjs/loader:1059:19)
    at require (node:internal/modules/cjs/helpers:102:18)
    at Object.<anonymous> (c:\Users\Stephen\.vscode\extensions\tamasfe.rhai-0.0.0\dist\server.js:1:105)
    at Module._compile (node:internal/modules/cjs/loader:1163:14)
    at Object.Module._extensions..js (node:internal/modules/cjs/loader:1216:10)
    at Module.load (node:internal/modules/cjs/loader:1035:32)
    at Module._load (node:internal/modules/cjs/loader:876:12) {
  code: 'MODULE_NOT_FOUND',
  requireStack: [
    'c:\\Users\\Stephen\\.vscode\\extensions\\tamasfe.rhai-0.0.0\\dist\\server.js'
  ]
}

OK, I think something that has to do with the fact that Windows does not have symbolic links.
So the build on lsp failed because core couldn't be found.
Fixing...

OK, I got stuck on this:
$ vsce package --no-yarn
Executing prepublish script 'npm run vscode:prepublish'...

><EMAIL_ADDRESS>vscode:prepublish
> yarn build

yarn run v1.22.19
warning<EMAIL_ADDRESS>The engine "vscode" appears to be invalid.
$ rm -rf dist && yarn build:syntax && yarn build:node
warning<EMAIL_ADDRESS>The engine "vscode" appears to be invalid.
$ js-yaml syntax/rhai.tmLanguage.yaml > syntax/rhai.tmLanguage.json
warning<EMAIL_ADDRESS>The engine "vscode" appears to be invalid.
$ rollup -c rollup.config.js

src/server.ts, src/extension.ts → dist...
(!) Circular dependency
node_modules/semver/classes/comparator.js -> node_modules/semver/classes/range.js -> node_modules/semver/classes/comparator.js
created dist in 6.3s
Done in 7.53s.
This extension consists of 1950 files, out of which 564 are JavaScript files. For performance reasons, you should bundle your extension: https://aka.ms/vscode-bundle-extension . You should also exclude unnecessary files by adding them to your .vscodeignore: https://aka.ms/vscode-vscodeignore
 ERROR  invalid relative path<EMAIL_ADDRESS>

Great, js environments are always a pain, I can't even imagine it on windows. I've fought with relative local packages in yarn before even on linux.
Alternatively we can publish the packages on npm and just pull them from there in the extension. That's what I intend to do anyway I just didn't want to publish WIP stuff if it's not necessary.

I'm just trying to resolve it such that it builds...

I know, you can try yarn pack, or npm pack and then add the .tar.gz as a dependency, it should be pretty similar to pulling it from npm.
I'll boot up a vm and see if I get it working as well.

This occurs when I cleaned it up and changed file:../ to ../
Running yarn in editors/vscode
$ yarn
yarn install v1.22.19
info No lockfile found.
[1/5] Validating package.json...
warning<EMAIL_ADDRESS>The engine "vscode" appears to be invalid.
[2/5] Resolving packages...
error Package "" refers to a non-existing file '"C:\\Git\\rhai-lsp\\editors\\core"'.
info Visit https://yarnpkg.com/en/docs/cli/install for documentation about this command.

Well, I'm totally stuck.  I can't build the thing on Windows.
Something to do with yarn and not find the rollup command and broken relative paths etc.

Alright, I've went through the steps in an almost empty windows vm and updated the readme with them on master.

Ah, so it seems yarn install --force is the missing command that I omitted!
The extension builds cleanly except for a few warnings:
warning<EMAIL_ADDRESS>The engine "vscode" appears to be invalid.
$ yarn build:syntax && yarn build:node
warning<EMAIL_ADDRESS>The engine "vscode" appears to be invalid.
$ js-yaml syntax/rhai.tmLanguage.yaml > syntax/rhai.tmLanguage.json
warning<EMAIL_ADDRESS>The engine "vscode" appears to be invalid.
$ rollup -c rollup.config.js

I wonder if we can make a rebuild shell script to automate this, although I guess it shouldn't be difficult to manually rebuild.


Estimate contributions to Wikidata by PBB

Could retrieve all contributions for user ProteinBoxBot, iterating through a query like this:
https://www.wikidata.org/w/api.php?action=query&list=usercontribs&ucuser=ProteinBoxBot&uclimit=50&ucdir=newer&ucstart=2016-09-01T01:01:01Z
And then grabbing the instance of statements to determine their types
Estimate statements changed looking at diffs:
https://www.wikidata.org/w/api.php?action=query&prop=revisions&titles=Q23558991&rvprop=ids|timestamp|flags|comment|user|content&format=json

Would also be interesting how many statements from those edits still persist.


[Feature] Add on_after_backward in Callback. Enable ModelGradTrackerCallback

🚀 Feature

Motivation

The call_hook on_after_backward  is already implemented, but not added in Callback class.
It boils down to add it within Callback Class
Pitch

Adding this new hook within callback could be used to implement something likeModelGradTrackerCallback.
Alternatives

Additional context

#3657 already requested :)

pls, update the existing one if needed :]


feat: Create NixOS module

Overview
Adds a semi-opinionated NixOS module to ease deploying TeslaMate on NixOS.
Trade-offs

Does not use docker/podman for service. This was done as it is typical for NixOS services to run natively since Nix itself + systemd provides much the same isolation/reproducibility that docker does.
Uses built-in NixOS grafana/postgres modules. This makes development easier/more-consistent however it does mean we are polluting the "global" grafana/postgres instances. I felt this was worth the trade-off but am open to discussion.
Provides native nix bindings for most teslamate configuration but not all. I chose the minimal set of configurations option that would be required to run Teslamate+Grafana behind a reverse proxy. Additionally, any missing configuration can still be provided by adding to systemd.services.teslamate.environment
Does not setup MQTT server. I chose not to include this since setting up a MQTT server in NixOS is quite simple and TeslaMate is not opinionated on its configuration.
RELEASE_COOKIE must be passed in via the secretFile. Values used in the NixOS config are globally readable so secrets should be read from disk at runtime. We could use either a hard-coded RELEASE_COOKIE value or generate something based on hashes, but given it is technically a secret, I chose to just require the end user to provide it.

Testing
nix flake check --impure (--impure require due to devenv) will run a simple telsamate configuration in a VM and assert that a server starts up on port 4000

I love the work on the NixOS side. What I wonder is how to do backup and restore with this implementation? This is the only reason why I have not yet migrated my TeslaMate instance to my NixOS server.

Backup/restore is just:

Backup the database as usual.
Backup the nixos configuration for the system that lets you recreate the system. Personally I have a nix flake in a central git that lets me configure all my systems.

I have not used postgresql on nixos, so I am not really familiar with upgrading databases, etc. I think I saw a webpage with instructions somewhere.
The potential gotcha is configuring secrets. You don't want to include secrets in plain text format in the nix repo, or at least this is frowned upon. I personally use agenix for these.

At present I just install teslamate on my nixos system with a docker image, but I think using this flake will be simpler.

Backup the database as usual.

Sometimes the most obvious is the hardest to see :-)

Backup the nixos configuration for the system that lets you recreate the system. Personally I have a nix flake in a central git that lets me configure all my systems.

Yeah, that's what I do as well

The potential gotcha is configuring secrets. You don't want to include secrets in plain text format in the nix repo, or at least this is frowned upon. I personally use agenix for these.

Same for me, using agenix for secrets and git-crypt as well


Fix Open Graph tagging

We need to change up the open Graph tagging up to fit with the new community orientation of ODK.

@susanjrobertson let me know if you need any help writing or adapting copy here.

@jess here's the current data in the Open Graph tags:
<meta name="twitter:creator" content="@bocoup">
    <meta name="twitter:site" content="@bocoup">
    <meta name="twitter:card" content="summary_large_image">
    <meta name="twitter:description" content="Open Design Kit: methods for doing distributed design">
    <meta name="twitter:title" content="{{ site.title }}">
    <meta name="twitter:image" content="http://opendesignkit.org/img/odk-twittercard.jpg">

We currently only have twitter but could also do Facebook. Here's what I have on my own personal site:
<!-- Open Graph here we go -->
        <!-— facebook open graph tags -->
        <meta property="og:type" content="website" />
        <meta property="og:url" content="{{page.url}}" />
        <meta property="og:title" content="{{page.title}}" />
        <meta property="og:description" content="{% if page.tldr %}{{ page.tldr }}{% endif %}" />

        <!-— twitter card tags additive with the og: tags -->
        <meta name="twitter:card" content="summary">
        <meta name="twitter:domain" value="susanjeanrobertson.com" />
        <meta name="twitter:title" value="{{page.title}}" />
        <meta name="twitter:description" value="{{ page.tldr }}" />
        <meta name="twitter:url" value="{{page.url}}" />

What data do you want to have/use? If we iron this out then I can whip up a PR really quickly.

Hi @susanjrobertson I'm helping out Jess going forward, would you be able to push your suggestion through, or do you need any help with this?

I @dmichl, I don't have a fix yet because I'm not sure what we want to be in the various pieces of meta data, so I'd need that and then I can happily to a PR for a fix.

@susanjrobertson thanks for your quick reply! and it's lovely to meet you :) @mjchamplin and @iamjessklein do you have any information to add here to get this moving again?


Use a synchronized tenant map

When creating tenants there are multiple threads populating this map,
this may lead to an inconsistent state unless using a regular map.

Yes, I am thinking about createTenant() in the bootstrap phase


bosh package conflict - bosh helper

Hello,
im trying to colocate 2 bosh releases in my deployment

https://github.com/cf-platform-eng/docker-boshrelease
http://bosh.io/releases/github.com/cf-platform-eng/networking-boshrelease

The aim is to give transparent internet access to the docker vm, by adding a defaut gateway to our internal internet gateway.
I meet a bosh package conflict (see below).
Are the 2 packages identical ? How could i work around this issue ?
Thks for ur help
Pierre
Started preparing deployment > Binding templates. Failed: Package name collision detected in job docker': template docker/docker' depends on package docker/bosh-helpers', template networking/routes' depends on `networking/bosh-helpers'. BOSH cannot currently collocate two packages with identical names from separate releases. (00:00:00)
Error 80011: Package name collision detected in job docker': template docker/docker' depends on package docker/bosh-helpers', template networking/routes' depends on `networking/bosh-helpers'. BOSH cannot currently collocate two packages with identical names from separate releases.

Unfortunately this is a bosh constraint, no matter if the package contents are the same or not, you cannot collocate 2 jobs from different releases that use identical package names. I don't believe there's any quick work-around to fix this without modifying the bosh release to use different package names (that I don't want to do).
If your problem is about internet access, have you tried using the proxy properties?

I did use the proxy property on my previous deployment, worked like a charm.
But i have a gateway transparent proxy on my new cloudstack infra, and i didnt want to set up a dedicated squid ...
BTW, i find cf-networking is a really usefull bosh release to associate with other releases. Thus it should have minimal namespace impact to be widely used ... Just to precise, is it just a name conflict or are the 2 packages identical ?

It's the same package.

Thanks @frodenas ! I ll be able to combine networking with docker bosh release :-)


'<=' not supported between instances of 'tuple' and 'int'

0% 0/10 [00:00<?, ?it/s]
Traceback (most recent call last):
File "scripts/demo_image.py", line 144, in
focal_length=focal, height=image.shape[0], width=image.shape[1])
File "/content/HybrIK/hybrik/utils/render_pytorch3d.py", line 81, in render_mesh
imgs = renderer(mesh)
File "/usr/local/envs/hybrik/lib/python3.7/site-packages/torch/nn/modules/module.py", line 722, in _call_impl
result = self.forward(*input, **kwargs)
File "/usr/local/envs/hybrik/lib/python3.7/site-packages/pytorch3d/renderer/mesh/renderer.py", line 54, in forward
fragments = self.rasterizer(meshes_world, **kwargs)
File "/usr/local/envs/hybrik/lib/python3.7/site-packages/torch/nn/modules/module.py", line 722, in _call_impl
result = self.forward(*input, **kwargs)
File "/usr/local/envs/hybrik/lib/python3.7/site-packages/pytorch3d/renderer/mesh/rasterizer.py", line 151, in forward
cull_backfaces=raster_settings.cull_backfaces,
File "/usr/local/envs/hybrik/lib/python3.7/site-packages/pytorch3d/renderer/mesh/rasterize_meshes.py", line 115, in rasterize_meshes
if image_size <= 64:
TypeError: '<=' not supported between instances of 'tuple' and 'int'
我在运行colab时出现的，一开始是在scipy==1.1.0时出现No module named 'scipy.spatial.transform'，我把scipy更新后出现上述bug

I was also confused yesterday. now I solved this problem. It is because the version of PyTorch is too old. pip can install version 0.3.0 only. I use Conda to solve this problem like this:
!source activate hybrik && conda install -c pytorch3d pytorch3d -y


gvametaconvert does not create features for different model

The python script gallery_generator.py launches the following gst pipeline for a sample image:
gst-launch-1.0 filesrc location=<face_img.png> ! decodebin ! video/x-raw ! videoconvert !         gvadetect model=/root/intel_models/Transportation/object_detection/face/pruned_mobilenet_reduced_ssd_shared_weights/dldt/face-detection-adas-0001.xml !         gvaclassify model=/root/intel_models/Retail/object_reidentification/face/mobilenet_based/dldt/face-reidentification-retail-0095.xml !         gvametaconvert model=/root/intel_models/Retail/object_reidentification/face/mobilenet_based/dldt/face-reidentification-retail-0095.xml converter=tensors-to-file layer-name=658 tags=sometag location=./gallery/features !         fakesink sync=false
This works fine. The model face-reidentification-retail-0095.xml takes input of shape [1,3,128,128] and outputs [1,256,1,1] at layer name 658 (type Convolution).
For our model (Its insightfaces FR model). Its input is [1,3,112,112] and output is layer name pre_fc1 of type FullyConnected with shape [1, 512].
There are no features produced. The earlier gvaclassify plugin in the same pipeline does not seem to fail as the pipeline seems to take a long time when using this mode, but the gvametaconvert seems to fail when producing features for gallery generation. Below is the pipeline for our model:
gst-launch-1.0 filesrc location=<face_img.png> ! decodebin ! video/x-raw ! videoconvert !         gvadetect model=/root/intel_models/Transportation/object_detection/face/pruned_mobilenet_reduced_ssd_shared_weights/dldt/face-detection-adas-0001.xml !         gvaclassify model=/root/models/model-fr.xml  !         gvametaconvert model=/root/models/model-fr.xml converter=tensors-to-file layer-name=pre_fc1 tags=sometag location=./gallery/features !         fakesink sync=false
Also, please note that in your original case, the layer-name was not needed.

model file name should be same as name attribute of the root net element in the model xml file.

model file name should be same as name attribute of the root net element in the model xml file.


Adds docs for ECR

Signed-off-by: Alex Ellis (OpenFaaS Ltd<EMAIL_ADDRESS>Description
Adds docs for ECR
How Has This Been Tested?

Tested independently through manual steps, rather than e2e with ofc-bootstrap
Checklist:
I have:

[x] checked my changes follow the style of the existing code / OpenFaaS repos
[x] updated the documentation and/or roadmap in README.md
[x] read the CONTRIBUTION guide
[x] signed-off my commits with git commit -s

This has now been tested e2e on AWS using ECR and EKS 1.13. OAuth and TLS were disabled.


TruncatedInputfromMFB

sorry，my english is bad
class TruncatedInputfromMFB(object):
    """
    input size : (n_frames, dim=40)
    output size : (1, n_win=40, dim=40) => one context window is chosen randomly
    """
    def __init__(self, input_per_file=1):
        super(TruncatedInputfromMFB, self).__init__()
        self.input_per_file = input_per_file

    def __call__(self, frames_features):
        network_inputs = []
        num_frames = len(frames_features)

        win_size = c.NUM_WIN_SIZE
        half_win_size = int(win_size/2)
        #if num_frames - half_win_size < half_win_size:
        while num_frames - half_win_size <= half_win_size:
            frames_features = np.append(frames_features, frames_features[:num_frames,:], axis=0)
            num_frames =  len(frames_features)

        for i in range(self.input_per_file):
            j = random.randrange(half_win_size, num_frames - half_win_size)
            if not j:
                frames_slice = np.zeros(num_frames, c.FILTER_BANK, 'float64')
                frames_slice[0:(frames_features.shape)[0]] = frames_features.shape
            else:
                frames_slice = frames_features[j - half_win_size:j + half_win_size]
            network_inputs.append(frames_slice)
        return np.array(network_inputs)

frames_slice = np.zeros(num_frames, c.FILTER_BANK, 'float64')Is the code wrong？
is
frames_slice = np.zeros((num_frames, c.FILTER_BANK), 'float64')

Yes, you are right. It needs to be modified.
Thank you.


On a case-sensitive filesystem, React Native fails to build

Description
Some issues with building yoga were discovered while using expo: https://github.com/expo/expo/issues/16283 but the root cause is in the way the Yoga pod is installed.
The build failure has the error
Use of undeclared identifier 'YGValue`

On a case-sensitive filesystem, the symlinks Yoga-umbrella.h and Yoga.modulemap are created in ios/Pods/Headers/Public/yoga instead of ios/Pods/Headers/Public/Yoga (note the case of yoga), which causes the build failure.
If I change https://github.com/facebook/react-native/blob/main/ReactCommon/yoga/Yoga.podspec#L30 from 'yoga' to 'Yoga', everything builds fine.
Version
0.63.0
Output of npx react-native info
$ npx react-native info                                                                                info Fetching system and libraries information...
System:
    OS: macOS 12.3
    CPU: (12) x64 Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz
    Memory: 883.43 MB / 16.00 GB
    Shell: 5.8 - /bin/zsh
  Binaries:
    Node: 12.22.11 - ~/.volta/tools/image/node/12.22.11/bin/node
    Yarn: 1.22.18 - ~/.volta/tools/image/yarn/1.22.18/bin/yarn
    npm: 6.14.16 - ~/.volta/tools/image/node/12.22.11/bin/npm
    Watchman: 2<IP_ADDRESS> - /usr/local/bin/watchman
  Managers:
    CocoaPods: 1.10.1 - /Users/tngacker/.rbenv/shims/pod
  SDKs:
    iOS SDK:
      Platforms: DriverKit 21.4, iOS 15.4, macOS 12.3, tvOS 15.4, watchOS 8.5
    Android SDK:
      API Levels: 28, 29, 30
      Build Tools: 28.0.3, 29.0.2, 30.0.3
      System Images: android-30 | Google APIs Intel x86 Atom
      Android NDK: Not Found
  IDEs:
    Android Studio: 2021.1 AI-211.7628.21.2111.8193401
    Xcode: 13.3.1/13E500a - /usr/bin/xcodebuild
  Languages:
    Java: 11.0.14 - /usr/bin/javac
  npmPackages:
    @react-native-community/cli: Not Found
    react: 17.0.1 => 17.0.1
    react-native: 0.64.3 => 0.64.3
    react-native-macos: Not Found
  npmGlobalPackages:
    *react-native*: Not Found

Steps to reproduce
I believe this happens on multiple versions of react native, see https://github.com/expo/expo/issues/16283
I can follow up again if necessary.
Snack, code example, screenshot, or link to a repository
No response

Same issue for me, any update ?

On a case-sensitive filesystem

Out of curiosity, on which file system are you?

This is still issue.
@cortinico In my case I have encrypted case sensitive drive (Mac)

Same issue for us (pretty painful timesink). We working around it by simply having a step after yarn and before pod installs:
sed -i -e "s/spec.module_name = 'yoga'/spec.module_name = 'Yoga'/g" node_modules/react-native/ReactCommon/yoga/Yoga.podspec

If I change https://github.com/facebook/react-native/blob/main/ReactCommon/yoga/Yoga.podspec#L30 from 'yoga' to 'Yoga', everything builds fine.

I'm not sure why noone made a PR to do this fix before now, but I went ahead and made one: https://github.com/facebook/react-native/pull/34058 so that at least we can test that it's actually that and there are no other side effects


fmu.model.name and fmu.model.revision optional or not?

Align meta.FMUModel and globalconfiguration.Model
To be able to produce metadata we require fmu.model.name and fmu.model.revision to be set in the global config. But the scema version used to validate before sumo upload has these fields as optional.
https://github.com/equinor/fmu-dataio/blob/58a554f06663e1ced617dbc2ab86df79423ee573/src/fmu/dataio/datastructure/meta/meta.py#L157-L170
https://github.com/equinor/fmu-dataio/blob/58a554f06663e1ced617dbc2ab86df79423ee573/src/fmu/dataio/datastructure/configuration/global_configuration.py#L41-L48

@perolavsvendsen do you think we can start setting fmu.model.name and fmu.model.revision as required in the schema? It should not affect any users that produces their metadata through dataio.
Of course there might exists some old preprocessed data somewhere without these fields... they will then get an error saying its missing and that they have to re-create the preprocessed data.

Yes, definitely. I thought they were required already. There is a lot of dependencies on them. It would cause problems if (particularly case metadata) was produced without these two.
Regarding pre-processed data - is fmu.model strictly necessary to have in pre-processed (stub) metadata at all? Or can we live with this being appended when running FMU (like fmu.case etc)?

Nice! I will make the change 👍
And forget about my comment regarding preprocessed data .. you are correct they will not be in there anyway and no error will be raised! It is only added when the fmu block is added upon re-export into scratch 😄


install -g ethereumjs-testrpc: permission denied

trying to install with npm on OSX (Sierra 10.12.5):
$ sudo npm install -g ethereumjs-testrpc

I get the npm error:

npm ERR! code 1
npm ERR! Command failed: /usr/bin/git clone --depth=1 -q -b master  https://github.com/debris/bignumber.js.git /Users/xxxxx/.npm/_cacache/tmp/git-clone-b4c1ee5c
npm ERR! /Users/xxxxx/.npm/_cacache/tmp/git-clone-b4c1ee5c/.git: Permission denied

Which permission? Does anybody know why, please let me know.
Thanks,
Bernhard..

I found out how to set the right permissons:
$ sudo chown -R $(whoami) $(npm config get prefix)/{lib/node_modules,bin,share}

$ sudo chown -R $(whoami) "$HOME/.npm"

$ npm cache verify

Now, all npm installs should work
Helpfull links:

https://docs.npmjs.com/getting-started/fixing-npm-permissions
https://github.com/npm/npm/wiki/Troubleshooting
https://gist.github.com/DanHerbert/9520689

I also have this problem, but your fix doesn't work for me.

You should never install npm modules with sudo (not even global ones). I would recommend you to uninstall node completely (and all global dependencies) and reinstall it through homebrew by running brew install node. This will set permissions properly and you won't need to use sudo for any installation.
If you want to have a different version of node: npm i -g n and e.g. n 6.9.5. Use n ls to see all available versions.
After that, rerun npm i -g ethereum-testrpc.

@kevin-smets solution worked for me

NOENT: no such file or directory, access '/usr/local/lib/node_modules/ganache-cli'

Closing due to inactivity/age

@BLukassen or anyone finding this issue after the fact - your best bet to avoid permissions issues in node is to use NVM. Windows users should check out nvm-windows or nodist. I personally use nvm-windows on windows machines, and I haven't tried nodist.

cc @jbarros35. Also homebrew also works great on mac (per @brdtrpp), but my preference is still to use NVM over homebrew because it supports rapid switching of node versions with a different version per terminal session if desired.


[Dev] Basic auth improvements

Currently username and password flag is used to accept credentials for basic auth, instead of that basic-auth flag can be used to accept credentials in the following format.
-basic-auth user:password

Done with https://github.com/projectdiscovery/simplehttpserver/pull/11


reduce per request log flood

reduce the log flood / avoids
I0927 16:37:27.035598   29029 grpcServer.go:223] AccessLog Quota foo.svc.cluster.local 1/1

for every request
NONE

This change is

/lgtm

Access log is per request so this is removing access log.
How does access log work in envoy? @kyessenov?

It's on by default, but out of request data path and can be disabled.

no I'm not touching access log, this is just removing that stderr log about quota in each request path


Allow KServe to have its own local gateways for Serverless mode

What this PR does / why we need it:
These changes introduce the possibility to configure KServe with its own Istio local gateway, to partially decouple KServe from the Knative local gateway.
Typically, it is OK to re-use the already configured Knative local gateway for KServe uses (as long as configs do not conflict). However, there are cases where having a dedicated local gateway for KServe is beneficial. Just to give some examples:

To have the ability to use strict mTLS in Istio
To reduce some pressure on the Knative local gateway by having a dedicated gateway deployment (it still would hit Knative gateway, but only once, rather than twice)
To be able to configure TLS on cluster-local hostnames (Knative support is still experimental)

To have a dedicated Gateway in KServe, similar configurations to Knative are need to be done. At the very least, and if not having a dedicated gateway deployment, a v1/Service and an Istio Gateway resource need to be created for KServe. Such resources would need to be configured in localGateway and localGatewayService. KServe still needs to rely on Knative routing for the KSVCs it creates. Thus, after handling an incoming request and resolving its target, it needs to be forwarded to be handled by Knative. This is the reason for introducing a new knativeLocalGatewayService in the ConfigMap.
The removed ingressService seems to be unused. Apparently, it became unused when the v1alpha2 API of the InferenceServices was deprecated and removed.
Type of changes
Please delete options that are not relevant.

[x] New feature (non-breaking change which adds functionality)
[ ] This change requires a documentation update

Feature/Issue validation/testing:
TODO
Please describe the tests that you ran to verify your changes and relevant result summary. Provide instructions so it can be reproduced.
Please also list any relevant details for your test configuration.

[ ] Test A

[ ] Test B

Logs

Checklist:

[ ] Have you added unit/e2e tests that prove your fix is effective or that this feature works?
[ ] Has code been commented, particularly in hard-to-understand areas?
[ ] Have you made corresponding changes to the documentation?

Release note:

Allow KServe to have its own local gateways for Serverless mode

just something come up in my mind maybe can be a future enhancement if it is useful.
Do you think having a dedicated gateway CRD to manage the gateway creation and configuration here will be useful? so that users don't need to look into how to configure Istio and KNative, and maybe will be helpful for the raw deployment mode (to support different gateway implementations?)

I have a pull request in the website that may give better idea about why these changes are relevant: https://github.com/kserve/website/pull/374.

@yuzliu I have an approval. Would you like to review?

Sorry... in my previous comment I wanted to tag @yuzisun

/assign @yuzisun

@israel-hdez @terrytangyuan I am not convinced we need this additional configuration but I could miss something, maybe we can schedule a call to discuss.

@yuzisun We'll reach you on Slack either on Monday 8th or Tuesday 9th to schedule the call.

@yuzisun After some long vacations, I'm retaking this. I wonder if there is something else that needs to be clarified for this PR?

@yuzisun After some long vacations, I'm retaking this. I wonder if there is something else that needs to be clarified for this PR?

other than the configuration naming it looks good

/lgtm
/approve


Search button bug

[x] CSS styles.
[x] Fetching collection twice at the beginnin.

Frontend tests were OK :+1: (details)


Update Header Prefix for JWTs

As outlined in app-backend/#193, The Header label for authentication tokens needs to be changed. This is the relevant place for the change.

Seems to be done. Closing.


Does Moor support offline-online sync mode across multi devices, like mobync?

Like Mobync

Moor doesn't support offline-online synchronization out of the box. It's essentially a wrapper for sqlite3, so I don't think we should provide network synchronization and other complex features in this library. You can take a look at https://github.com/simolus3/moor/issues/136 for a further discussion, but in the end I think you'll have to implement synchronization logic yourself.

Do you have any good suggestion about libraries which can be used in Dart for this purpose(offline-online sync logic across multi devices)?

I'm not aware of a package that does everything for you, since it's a complicated topic.
A good abstraction for this purpose are conflict-free replicated datastructures. There are some good talks on them like this one. There's a crdt package in pub, but I've never used that. Maybe you could use that together with moor somehow.
If you have a simpler synchronization behavior, you could also store when a row has last been read from a server and just attempt a refresh if you have connectivity and some time has passed. For locally-created rows, you could store whether they have been written to a server and do if necessary.

@simolus3 Sorry to bother you again since this bug has has been closed.
As you can see Mobync client is used along with sqlite rather than moor.
My question is that if Moor is built on the top of sqlite like? If yes, is it possible to make mobync able to work with moor?
We are considering using mobync. Thanks.

It looks like you can implement a moor-based MobyncClient client and use that, yes. I've never tried that, but I don't see why it shouldn't be possible. If you have specific questions on how to write a moor-based implementation for that I can try to answer those as well, but I can't write all of the implementation for you ;)

@simolus3 Thanks, apart from mobync, we have other choice, once we decide to use it and have questions, I will contact you. Thank you in advance.

@jims57 what did you settle on?

Hi,
I've built a synchronization framework that runs on top of Moor library called netcoresync_moor. You can find the repository here and the pub.dev package here. This framework allows you to have local Moor databases on each user's device and synchronize them to a centralized database hosted on the server.
This framework actually was built for Xamarin technology a couple years ago, and recently ported to Flutter because our team has started moving to Flutter for building our products. And we've chosen to built it on top of Moor (after having insightful discussions with @simolus3, thanks!) because I think Moor is the perfect choice for proven database framework in Flutter right now. Give it a try and feel free to ask questions, thanks.

@aldycool Thank you for your framework.


populating matches to quickfix menus

Hello Y'all,
I am trying to see if this is a feature of LSPs or not: being able to add matches found in a buffer via the / key can be added to the quickfix menu?
I'm expecting the following workflow:

/{string} to search for match <CR>
:copen to open the quickfix list

I've done some research and I've seen vim-qf and vim-grepper.

Hi. Please file an issue against whichever LSP you want or against Neovim itself. This repo is ONLY for the kickstart.nvim project and we can't be a support hub for the entire Neovim ecosystem.
Thanks for the report. Closing for now.


IOS App Force closing

Hi
I am using rn v 0.29.2
My IOS app is force closing showing no error.
installed using
npm install -save react-native-video
linked
rnpm link react-native-video
My code below
import React, { Component } from 'react';
import {
  AppRegistry,
  StyleSheet,
  Text,
  View,
  AlertIOS
} from 'react-native';
import Video from 'react-native-video';

class test extends Component {
render() {
    return (
      <View style={styles.container}>
        <Video source={{uri: "splash"}}
           resizeMode="cover"
           repeat={false}
           style={styles.fullScreen} />
      </View>
    );
  }
}

const styles = StyleSheet.create({
  container: {
    flex: 1,
    justifyContent: 'center',
    alignItems: 'center',
    backgroundColor: 'transparent',
  },
  welcome: {
    fontSize: 20,
    textAlign: 'center',
    margin: 10,
  },
  instructions: {
    textAlign: 'center',
    color: '#333333',
    marginBottom: 5,
  },
  fullScreen: {
    position: 'absolute',
    top: 0,
    left: 0,
    bottom: 0,
    right: 0,
  },
});

AppRegistry.registerComponent('test', () => test);

same issue

@keung725
I think you have to add the video file(s) to xcode resources.
I am using, https://github.com/ldn0x7dc/react-native-media-kit

@edelworksgithub
Thanks!!!!!!
For 0.30, need to use https

Looks like this is "fixed". Closing for now but please feel free to re-open if this is still an issue.


feat(docs): Add model config spec

Signed-off-by: Ce Gao<EMAIL_ADDRESS>
What this PR does / why we need it:
Add model config v1alpha1 specification, compared with the internal manifest definition:

Add extensibility for dataset and training. Currently we only support git repo but keep the extensibility for other sources.
Remove resource limits in the manifest

Which issue(s) this PR is related to (optional, link to 3rd issue(s)):
Fixes #3
Reference to #

Special notes for your reviewer:
/cc @ddysher @codeflitting

Release note:

NONE

LGTM
/cc @simon-cj

LGTM

LGTM

/lgtm

/lgtm
/approve


can it have lables

can it have  labels  i want to create x and y axis for  bar chart

There's no official support for adding axes or labels (the charts are expected to be too small) but I imagine it could be achieved (though knowledge of undocumented Peity features and internals would likely be required 😬). It's not what you asked for but here's an example of adding tooltips to charts (taken from discussion in #71): https://jsbin.com/vufayan/edit?html,js,output

There's no official support for adding axes or labels (the charts are expected to be too small) but I imagine it could be achieved (though knowledge of undocumented Peity features and internals would likely be required 😬). It's not what you asked for but here's an example of adding tooltips to charts (taken from discussion in #71): https://jsbin.com/vufayan/edit?html,js,output


[ci] Better bitstream build cache filtering

We want to avoid rebuilding bitstreams if possible. To do this we look at the list of changed files and determine if a rebuild is warranted.
Currently the filtering is entirely hardcoded rules. This PR uses Bazel to get a list of dependencies required for the bitstream, and filter the changes based on that list.

Hi @nbdd0121! Is this still something that we should be doing? It looks sensible to me: maybe it's worth rebasing?

I think I still want to pursue this, but it's unlikely that this could be merged before the quiet period, so I'll defer this to a later date.

Apparently now building the bitstream does not require opentitanlib, so we can now ask Bazel to find dependencies and do extra filter on top without too much faff.


feat: Upgrade RHACM from 2.7 to 2.8

Is your feature request related to a problem? Please describe.
RHACM release 2.8 is out, so it is time to upgrade the version of RHACM in the repository.
Describe the solution you'd like
Upgrade the subscription channel in the repository, update eventual CRs, remove deprecated ones, etc
Describe alternatives you've considered
None, the requirement is pretty narrow.
Additional context
Add any other context or screenshots about the feature request here.

:tada: This issue has been resolved in version 0.26.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


route blinding: use all valid routes during path construction iteration instead of the currently capped set

Today, if we are constructing set of blinded payment paths to add to an invoice, we:

Select up to MaxNumPaths routes to our own node that are considered blinded path candidates.
This set is then passed to blindedpath.BuildBlindedPaymentPaths which attempts to use each path to build a valid blinded path
If blindedpath.BuildBlindedPaymentPaths finds that it cannot use one of the paths it was given, it skips it and moves on to the next one

This order of operations means that we may run out of paths in blindedpath.BuildBlindedPaymentPaths and we wont go back and try some of the paths that we initially found by ChannelRouter's findBlindedPaths .
What we instead want is for blindedpath.BuildBlindedPaymentPaths to be able to continue with the iteration through possibly all of the routes found in findBlindedPaths until it reaches the max specified by MaxNumPaths.

Good idea, so you mean we should immediately when having the blinded path check if we can build the whole route before exiting the route search, so we make sure the returned routes are also possible to use ?

@ziggie1984  -  yeah, or we make a way for the logic that is doing the iterating and building to query like NextCandidateRoute or something like that. Or actually, perhaps should return all possible routes and then do the MaxPath cap later on instead of as early as it is currently done

@MPins  - im saying it currently does the following:

Chooses MaxNumPaths candidate paths
Passes that to BuildBlindedPaths which might drop some of the paths returned cause they turn out not to be valid.

This is not ideal cause we then we can easily run out of paths to try.
Instead, the MaxNumPaths restriction should be applied later on instead. So when selecting candidates, we should not limit at that point.

Got it. If nobody had started working on it, I can give a try.

This is for my education...why do we have a limit capped by MaxNumPaths?

otherwise the invoice would become super huge, there might be a huge number of paths to a particular node.


Mocha debug

Hi there. This is a hack I've thrown together to let me easily debug my mocha specs. It works but it's ugly :)  If you think it's a feature worth pursuing, I'd be happy to make any necessary improvements to get it merged.
The use-case for this branch is to enable dropping into the debugger in node from vim. In rails, you can add a binding.pry in your test/code and you will drop into a repl when you run the test. In node, however, adding a debugger in your test/code will not drop you into a repl unless you run node in debug mode. Mocha provides a debug flag that will run node in debug and observe debugger break points.
To use this branch, you will need to modify your .vimrc and add a g:mocha_debug_command and pass a parameter to RunNearestSpec (0 === just run the test, 1 === run in debug mode).
For example:
" rspec and mocha setup {{{
let g:rspec_command = "!rspec {spec}"
let g:mocha_coffee_command = "!mocha {spec}"
let g:mocha_js_command = "!time NODE_PATH=./frontend mocha --require ./frontend/tests/mocha.js {spec}"
let g:mocha_debug_command = "!NODE_PATH=./frontend mocha debug --require ./frontend/tests/mocha.js {spec}"

" Rspec.vim mappings
map <Leader>t :call RunCurrentSpecFile()<CR>
map <Leader>s :call RunNearestSpec(0)<CR>
map <Leader>d :call RunNearestSpec(1)<CR>
map <Leader>l :call RunLastSpec()<CR>
map <Leader>a :call RunAllSpecs()<CR>
"}}}

Sorry for the (extremely) long delay 😞 Personally haven't used this in a while, but if this works for you, more than happy to merge it in. Code looks fine to me 👍


kernel: ensure stages that stopped early are not reported as blocking;

Can you maybe write a test which triggers the problem with the current behavior and then of course no longer triggers with your fix?


vmservice tests fail to import observatory package on Windows

The following seems to be caused by Windows not handling the dart/pkg/observatory symlink:
FAILED: none-vm-checked release_x64 standalone/vmservice/allocations_test
Expected: Pass
Actual: RuntimeError
CommandOutput[vm]:
stdout:
Uncaught Error: FileSystemException: Cannot open file, path = 'E:\b\build\slave\vm-win64-release-be\build\dart\build\ReleaseX64\packages\observatory\service_io.dart' (OS Error: The system cannot find the path specified.
, errno = 3)
Stack Trace:
#0      _File.open.<anonymous closure> (dart:io/file_impl.dart:349)
#1      _RootZone.runUnary (dart:async/zone.dart:1088)
#2      _Future._propagateToListeners.handleValueCallback (dart:async/future_impl.dart:488)
#3      _Future._propagateToListeners (dart:async/future_impl.dart:571)
#4      _Future._completeWithValue (dart:async/future_impl.dart:331)
#5      _Future._asyncComplete.<anonymous closure> (dart:async/future_impl.dart:393)
#6      _asyncRunCallbackLoop (dart:async/schedule_microtask.dart:23)
#7      _asyncRunCallback (dart:async/schedule_microtask.dart:32)
#8      _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:126)

Obsolete bug.


[#289] Added interop-rx3 for Rxjava3 support

Description
Closes #289
Check list

[x] I have updated CHANGELOG.md if required.
[x] I have updated documentation if required.

Do not forget https://github.com/bumble-tech/appyx/pull/297

@zsoltk / @KovalevAndrey thoughts? I don't believe this will cost us much to support, and it definitely helps library users (they don't need to add rx2 converters)

@zsoltk / @KovalevAndrey thoughts? I don't believe this will cost us much to support, and it definitely helps library users (they don't need to add rx2 converters)

I don't see reasons why not to


[pulsar-broker] Added support to force deleting tenant

Motivation
Some users expect an option to force deleting tenant for simplicity, it can be useful in some situations.
Modifications
Add a optional field to force the deletion of all stuffs related to tenant.

I've fixed the testIndividualAck cpp unit test failure in #8141 , please rebase and commit again, thx!

move to 2.8.0 first

move to 2.8.0 first

@codelipenghui I can add a test to cover your comments and we expect this feature as soon as possilbe. Can this be introduced in coming 2.7.0?

@codelipenghui @jiazhai I have added a test, please help to take a review when available, thanks.

@codelipenghui @jiazhai I have added a test, please help to take a review when available, thanks.

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

Close this and submit a new pull request using metadata-api.


Fix Jenkins.

This may be what's messing up our Repo.
Bukkit/Bukkit@1a68bf226b76a5208956992e40ba61cd6496e3c2
@lithium3141 could you take care of this (meaning investigate and possibly solve?)

Jenkins has been fixed.


'module' object has no attribute 'Consumer' on Ubuntu

Hi, i have a problem with oauth2 on Ubuntu, this is my code:
import oauth2 as oauth
CONSUMER_KEY = 'ZZZ'
CONSUMER_SECRET = 'KKK'
def oauth_req(url, key, secret, http_method="GET", post_body=None, http_headers=None):
consumer = oauth.Consumer(key=CONSUMER_KEY, secret=CONSUMER_SECRET)
token = oauth.Token(key=key, secret=secret)
client = oauth.Client(consumer, token)
resp, content = client.request( url, method=http_method, body=post_body, headers=http_headers, force_auth_header=True )
return resp, content
content, home_timeline = oauth_req( 'https://api.twitter.com/1.1/statuses/home_timeline.json', 'YYY', 'XXX' )
And this is the traceback
Traceback (most recent call last):
File "/home/facundo/Escritorio/oauth2.py", line 1, in
import oauth2 as oauth
File "/home/facundo/Escritorio/oauth2.py", line 13, in
content, home_timeline = oauth_req( 'https://api.twitter.com/1.1/statuses/home_timeline.json', '163064210-ZvmxvqAAJOPwUMNqhEcIM3ODDMmpso59uX38eMMJ', 'ijRSQyku55sTNtaNh1eRzBvM2OQ59CkryQQnic1NfcHX6' )
File "/home/facundo/Escritorio/oauth2.py", line 7, in oauth_req
consumer = oauth.Consumer(key=CONSUMER_KEY, secret=CONSUMER_SECRET)
AttributeError: 'module' object has no attribute 'Consumer'
i install oauth2 with this comand "pip install oauth2".
i forget to do something ?? What is worng ??
Tranks

Hi, not sure if you're aware but this repo is no longer maintained. If you could please close your issue that would be a big help. Thanks.

Hi @facuchaves can you please refactor your example using github markdown so we can better help you? I'll close this Issue if we haven't had a reply in 7 days. Thanks!

I just did a check, if you did pip install python-oauth2 then that would be your issue there. I've tried the above and I'm not getting your error on 2.7.10 sorry.

Annnnd you're issue is because Consumer is a class not a property of something... see #168.

Hi @jaitaiwan, I have the same issue on windows8, I was using python3.4 with anaconda and this issue was coming. So i installed python2.7 kernel as well and run the same code.
It was not working, the code is as follows;
def oauth_get_tweet(tid, http_method="GET", post_body='',  http_headers=None):
    url = 'http://api.twitter.com/1.1/statuses/show.json?id=' + tid
    consumer = oauth.Consumer(key=CONSUMER_KEY, secret=CONSUMER_SECRET)
    token = oauth.Token(key=ACCESS_KEY, secret=ACCESS_SECRET)
    client = oauth.Client(consumer, token)

    resp, content = client.request(
        url,
        method=http_method,
        body=post_body,
        headers=http_headers
    )
    return content

AttributeError                            Traceback (most recent call last)
<ipython-input-50-e54d8133b446> in <module>()
----> 1 download_tweets( fetch_list, user_params['rawDir'] )

<ipython-input-45-e2ec58e04190> in download_tweets(fetch_list, raw_dir)
    114
    115         # pull data
--> 116         data = oauth_get_tweet(item[2])
    117         with open(raw_dir + item[2] + '.json', 'wb') as outfile:
    118           json.dump(data, outfile)

<ipython-input-45-e2ec58e04190> in oauth_get_tweet(tid, http_method, post_body, http_headers)
     80         http_headers=None):
     81     url = 'http://api.twitter.com/1.1/statuses/show.json?id=' + tid
---> 82     consumer = oauth.Consumer(key=CONSUMER_KEY, secret=CONSUMER_SECRET)
     83     token = oauth.Token(key=ACCESS_KEY, secret=ACCESS_SECRET)
     84     client = oauth.Client(consumer, token)

AttributeError: 'module' object has no attribute 'Consumer'

Hey @emarshah, did you run pip install python-oauth2? Because the correct command to run is pip install oauth2. I think you'll find that's the issue. python-oauth2 installs as oauth2 so you'll have to run pip uninstall oauth2.

I did install the way you are suggesting i.e., python pip install python-oauth2 but this had the same issue of not able to look up for Consumer class. So I did as @xamiax did here https://github.com/joestump/python-oauth2/issues/168 i.e., copying the init.py content and doing the needful.

The way I suggested was pip install oauth2... Not python-oauth2

Oh... I totally missed it,
It's giving me following error today;
Microsoft Windows [Version 6.3.9600]
(c) 2013 Microsoft Corporation. All rights reserved.

C:\Users\a>pip install oauth2
You are using pip version 7.0.3, however version 7.1.0 is available.
You should consider upgrading via the 'python -m pip install --upgrade pip' comm
and.
Collecting oauth2
  Downloading oauth2-1.5.211.tar.gz
    Complete output from command python setup.py egg_info:
    Traceback (most recent call last):
      File "<string>", line 20, in <module>
      File "C:\Users\a\AppData\Local\Temp\pip-build-x0s31_87\oauth2\setup.py
", line 18
        print "unable to find version in %s" % (VERSIONFILE,)
                                           ^
    SyntaxError: invalid syntax

    ----------------------------------------
Command "python setup.py egg_info" failed with error code 1 in C:\Users\ammar\Ap
pData\Local\Temp\pip-build-x0s31_87\oauth2

What Python version?
On Sun, 9 Aug 2015 00:43 Ammar Hassan<EMAIL_ADDRESS>wrote:

Oh... I totally missed it,
It's giving me following error today;
Microsoft Windows [Version 6.3.9600]
(c) 2013 Microsoft Corporation. All rights reserved.
C:\Users\a>pip install oauth2
You are using pip version 7.0.3, however version 7.1.0 is available.
You should consider upgrading via the 'python -m pip install --upgrade pip' comm
and.
Collecting oauth2
Downloading oauth2-1.5.211.tar.gz
Complete output from command python setup.py egg_info:
Traceback (most recent call last):
File "", line 20, in
File "C:\Users\a\AppData\Local\Temp\pip-build-x0s31_87\oauth2\setup.py
", line 18
print "unable to find version in %s" % (VERSIONFILE,)
^
SyntaxError: invalid syntax
----------------------------------------

Command "python setup.py egg_info" failed with error code 1 in C:\Users\ammar\Ap
pData\Local\Temp\pip-build-x0s31_87\oauth2
—
Reply to this email directly or view it on GitHub
https://github.com/joestump/python-oauth2/issues/171#issuecomment-128987768
.


Is wifi strength available in the API?

@binarydev So far this fork has been rock solid.  Thank you again for your time and work on this!
Question - The Mobile Link app includes the wifi strength.  Do you know if this is available in the API?  And if so, would it be difficult to add to HA?

It doesn't appear to be in the /v2/apparatus/details api, but it is in the /v2/apparatus/list oddly enough.

Glad it's been so solid :) the original code did have mentions of it and some commented out references, but I haven't tried to expose it. Let me take a look and see if I can get it added back in.

Just published v0.2.1 which adds a new sensor to expose the wifi signal strength. It's presented in the API as a % string, so I just passed that through to the new sensor entity. The existing code from the original repo was trying to expose it as dbm (true signal strength whereas % is signal quality), but that can be done via a helper instead if you really wanted to.
The new version should show up in your HACS interface as soon as it checks if an update is available (if you're using HACS that is, otherwise just copy-paste the latest code into your custom_components directory). You can also force it to recognize that an update is avilable by going to http:///hacs/repository/733708339 -> 3 dots on top right -> Update information. Then go to http:///hacs/entry to see the update available for install. After install you'll see the new sensor for the device listed:

For anyone curious of how to convert % (quality) to strength (dbm): https://stackoverflow.com/questions/15797920/how-to-convert-wifi-signal-strength-from-quality-percent-to-rssi-dbm

This is great!  Thank you @binarydev!


reactivate entityactivation

Ported most of the invalid entityactivation code.
But still has some problems:

[x] LevelChunkMixin active chunk check
[x] WorldInfoBrige#bridge$isValid()
[x] fake Player check in EntityActivationRange#initializeEntityActivationState and checkIfActive
[x] Villager isMating check in EntityActivationRange#checkEntityImmunities
[x] global config usage in EntityActivationRange#addEntityToConfig
[x] usage of final fields in addEntityToConfig e.g. local autopopulate variable is false even if the actual value in the config is true.

https://github.com/SpongePowered/Sponge/commit/0e33d585b6a4e991ae56e31f4673ebd43d060acb


ROX-17157: Fix reconciliation for offline mode for Kubernetes Listener

Description
This PR relies on a few past PRs to make kubernetes reconciliation work when flipping between offline and online mode.

#6958: Add context to every messages in the pipeline. This prevents messages to linger in the queue and mess up the reconciliation.
#6692: Adds interface for cleaning up the in-memory stores. This is called when going into offline mode. This is required because during the reconciliation, we might pick up old deployments that were deleted while sensor wasn't listening to events.
#6376: Adds offline mode for Kubernetes listeners. This triggers the reconciliation once the listeners are re-created.

Checklist

[x] Investigated and inspected CI test results
[x] Unit test and regression tests added
[ ] Evaluated and added CHANGELOG entry if required
[ ] Determined and documented upgrade steps
[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

Testing Performed

[x] E2E Tests
[x] Implement new integration test for k8s reconciliation

I am wondering why ocp-4-10-sensor-integration-tests were run without prompting them... They should react only to an explicit /test ocp... command.

/test ocp-4-10-sensor-integration-tests


When using aliases unable to release an .apk (babel-plugin-module-resolver)

Is this a bug report?
Yes probably
Have you read the Contributing Guidelines?
Yup.
Environment
OS: Linux 4.13
Node: 9.5.0
Yarn: 1.3.2
npm: 5.6.0
Watchman: Not Found
Xcode: N/A
Android Studio: Not Found (but it's there, the newest version)
Created with create-react-native-app and ejected
Also this is working great in develoment mode.
Packages: (wanted => installed)
react: 16.0.0 => 16.0.0
react-native: 0.50.3 => 0.50.3
Target: Android (signed .apk)
Steps to Reproduce

Get  https://github.com/tleunen/babel-plugin-module-resolver
Update your alises in .babelrc file like so:

["module-resolver", {
          "cwd": "babelrc",
          "extensions": [
            ".js",
            ".ios.js",
            ".android.js"
          ],
          "alias": {
            "@assets": "./src/assets",
            "@features": "./src/features",
            "@common": "./src/common",
            "@config": "./src/config",
            "@services": "./src/services"
          }
        }],

Set up keystores etc.
Run cd android && ./gradlew assembleRelease

Expected Behavior
My .babelrc aliases are used and recognized and apk file is successfully built.
Actual Behavior
An error occurs on the first use of alias
Unable to resolve module `@common` from `/home/ppozniak/Projects/project-path/src/Root.js`: Module does not exist in the module map

More info
May be connected to: https://github.com/facebook/react-native/issues/8723
Probably due to react-native bundle own babel settings, ignoring the one in root of project.
I've tried adding
project.ext.react = [
    entryFile: "index.android.js", /* Your entry file */
    extraPackagerArgs: ["--projectRoots ../../native-config, ../../"]
]

however it seems that --projectRoots option is gone?
Stackoverflow: https://stackoverflow.com/questions/48986918/unable-to-resolve-module-when-building-react-native-apk-babel-plugin-module-re

Okay it was just me being silly a little bit.
Looking at my .babelrc you can
"env": {
    "development": {

And that was issue. After this some other bugs popped out with Android but eventually I've managed to fix all of them.
So lesson for future: watch for your envs.
My babelrc now:
{
  "presets": [
    "react-native",
    "babel-preset-react-native-stage-0/decorator-support",
  ],
  "plugins": [
    "transform-react-jsx-source", ["module-resolver", {
      "extensions": [
        ".js",
        ".ios.js",
        ".android.js"
      ],
      "alias": {
        "@assets": "./src/assets",
        "@features": "./src/features",
        "@common": "./src/common",
        "@config": "./src/config",
        "@services": "./src/services"
      }
    }]
  ]
}

@ppozniak Exact same issue here. Thanks for the solve.

@ppozniak Hey, that's saved my life! Thanks, bro!


.ToString should be called automatically

In C# we can do:
string text = "abc" + 123; which results in a single string "abc123".
However the following does not work:
	public class Unit
	{
		public string Name = "Test";
		public int Age = 123;
	}

DynamicExpresso.Interpreter interpreter = new DynamicExpresso.Interpreter(
			DynamicExpresso.InterpreterOptions.CommonTypes |
			DynamicExpresso.InterpreterOptions.SystemKeywords |
			DynamicExpresso.InterpreterOptions.PrimitiveTypes);

	interpreter.Reference(typeof(Func<>));

	string code = "u.Name + u.Age";

	var function = interpreter.ParseAsDelegate<Func<Unit, string>>(code, "u");

	var text = function.Invoke(new Unit());

If code is changed to u.Name + u.Age.ToString() everything works perfectly, so I guess it has to do with casting u.Age to an object before passing it to string.Concat?

Sorry for the late response (I have probably missed the notification). You are right, this can be an useful improvement. I will evaluate it for the future. Of course any help is appreciated :wink:


Allow for local dns resolution with a custom dialer

If the custom dialer updates the transport to proxy the mssql connection, and only the proxy can reach the network that mssql is running in, then any local DNS resolution will fail.
To address this, this PR ensures the custom dialer is used, if defined, to resolve the DNS.
We've been using this logic on Grafana's forked repo for the past few months.

Codecov Report

Merging #121 (fd2dcf7) into main (dd3e503) will decrease coverage by 0.04%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #121      +/-   ##
==========================================
- Coverage   73.95%   73.91%   -0.04%
==========================================
  Files          29       29
  Lines        5613     5620       +7
==========================================
+ Hits         4151     4154       +3
- Misses       1212     1215       +3
- Partials      250      251       +1

Impacted Files
Coverage Δ

protocol.go
93.06% <100.00%> (-0.55%)
:arrow_down:

... and 2 files with indirect coverage changes

thank you for submitting a PR!

@microsoft-github-policy-service agree company="Grafana"

@shueybubbles updated the README & Changelog! Thanks for the review!
@apoorvdeshmukh I updated the unit test, but I'm unable to kick off running the tests. Would you be able to kick it off for me?

Hey! Just wanted to check if there is anything I need to do here to get this merged. Thanks!

@apoorvdeshmukh it's waiting for you to approve then we can merge


TMLE additions

This is a longer term project. As I am reading through Targeted Learning, I will add to this list regarding features I would like to add. Also important notes that I have gleamed from the book.

[x] Add support for continuous outcomes (Targeted Learning Ch7 (pg 125, 126)

For a continuous Y, it must be bounded between 0-1 before starting the process. Transform using the following Y* = (Y-a) / (b-a) where a = min(Y), b = max(Y)

Psi = (b-a) Psi* to convert from the bounded causal effect back to the original

Use some alpha to keep logit(Y*) being undefined. alpha = 0.0005 maybe?

[x] Add support for F(A=1) and F(A=0)

Need to look up formulation in targeted learning book and IC

NOT IMPLEMENTING. James Robins showed in 1988 (Confidence intervals for causal parameters) that the corresponding confidence intervals may only be valid assuming deterministic potential outcomes. As a result, I have decided to not implement this feature (since I don't want to make that assumption

[x] Natively handle missing data

R tmle will be a good reference for this

Add an option to specify a missing data model. This should be optional to include

[ ] Mediation analysis (direct, indirect, total) (Targeted Learning Ch 8)

Can add this, but I am not largely convinced of current mediation analysis. I know that people do like to use it though...

pg 139 has some good notes

[ ] Collaborative TMLE (CTMLE)

g-model should be based on TMLE of Q, not the fit to g. CTMLE is an approach to formalized this. Might try to add as an additional class object (depending on utility and substantive differences)

Missing data process based on R tmle:

Estimate missing data model (something like missing_model() optional argument)

Multiply g1W and g0W by corresponding p(missing=0)

Missing needs to factor into influence curve calculation. ONLY the indicator though, not the weight itself

Don't plan on adding C-TMLE anytime soon. The problem it is used to solve can be fixed via cross-fitting (to my current understanding. if that changes, will consider adding). C-TMLE purports to give the correct answer when both g- and Q-models are incorrect. I think this is an artifact of the Kang-Schafer data
As for now, these remaining additions are on the back-burner. If anything changes or users request, I will re-open


PR: Fix fileswitcher search and add/improve tests

Description of Changes

[x] Added unit test(s) covering the changes (if testable)

Fixes the file switcher search being broken as described in the linked issue. Also, adds some tests for it, improves the existing test and makes it work on all platforms (not just Linux as before).
Issue(s) Resolved

Fixes #9332
Affirmation
By submitting this Pull Request or typing my (user)name below,
I affirm the Developer Certificate of Origin
with respect to all commits and content included in this PR,
and understand I am releasing the same under Spyder's MIT (Expat) license.

I certify the above statement is true and correct: CAM-Gerlach

Hello @CAM-Gerlach! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file spyder/widgets/fileswitcher.py:

Line 766:21: E129 visually indented line with same indent as next logical line


Adding Windows Getting Started doc

This addresses issue #11

LGTM
nit: I would recommend setting make build to be cargo build --release by default since the released binary is much much faster than debug mode.

@Mossaka To do this make build TARGET=release should do the trick.
Likewise sudo make install TARGET=release


Rebates XSD and Schematron Data Validation

Adding the xsd and schematrons for validating the HPXML files accepted by the API. These probably need a couple of rounds of iteration and probably some sort of testing.

Comments on Data and Tools Guide Requirements
Specifically Table 9 in this document.
12 - 14. Modeled electricity, gas, other fuel uses

Don't need these for measured since they won't be modeling.

15. Monthly utility usage data

Should be required for measured pathway. Maybe they're not submitting it in HPXML? Are they submitting it through some other API?

19 - 21. Attic and Roof Area, Roof Type

Attic area isn't in the required lists but roof area is.
The area fields come from Home Energy Score and are dependent on the roof type. Roof type isn't required, so it doesn't make sense to collect the area.
The following would work for the required fields:

Collect attic or roof area (depending on roof type as indicated) and the roof type.
Collect only the roof type and no areas
Collect neither the roof type nor the areas

27, 29-30. Foundation Type, Floor above crawlspace/basement R-value, Basement wall or slab edge R-value

Foundation type is easy to observe and should be collected for measured.
Since the R-values of foundation adjacent surfaces are required for that use case, it is necessary to know the kind of foundation we're talking about.

68 - 79. BPI-2400 Calculated Values

These values are metrics calculated as part of a BPI-2400 calibration.
They should be required for modeled since they will need to be calculated for a modeled pathway rebate so they might as well report them.
They should not be required for measured since they are not relevant metrics for that pathway.


People can create rose jobs that they don't own

resolution_fixed   |    by<EMAIL_ADDRESS>
Wenming imported some jobs from the Met Office which had pre-exisiting suite-info files. When added to subversion Rose will get the suite ownership from these files - since the users don't exist on our system this is an issue.
This is fixable - disable the pre-commit hook and change the suite-info file, but it would be nice if this couldn't happen.

Issue migrated from trac:44 at 2024-01-31 17:27:11 +1100

ibc599 commented

Some suggested super users ibc599, saw562, mrd599, rmb548

ibc599 commented

Version 2014-01+ has a new configuration item which specifies super users who have access to all suites regardless of what is specified in the rose-suite.info file.

ibc599 commented

Current options;

Have users to edit rose-suite.info file before commit
Failing option 1) disable the pre-commit hook and change the suite-info file, and then reenable the pre-commit hook.

Future option?
Modify the pre-commit hook to check users against access group

<EMAIL_ADDRESS>changed component from accessdev server to Rose

ibc599 changed reviews which not transferred by tractive

<EMAIL_ADDRESS>changed priority from major to minor

<EMAIL_ADDRESS>changed owner from `` to ibc599

ibc599 changed status from new to closed

<EMAIL_ADDRESS>commented

Milestone Future deleted

<EMAIL_ADDRESS>removed milestone (was Future)

ibc599 changed resolution from `` to fixed

ibc599 commented

Added super users.


Shorter URL / Filename

What and where about in the PHP script would I change if I'd like to shorten the generated filename? After digging around a bit, it looks like it's using the MD5 of the image file to generate the name? I'd like to be able to shorten it to the length of imgur's (e.g. imgur.com/s9K5x.png). Thanks for your help!

You can try make this changes (I have not test it, but it should work):
https://github.com/ziggi/zimg-host/blob/master/upload.class.php#L71
$new_name = hash('crc32b', microtime() . $urls_array[$i] . rand(0, 9999)) . '.' . $this->_allowed_types[ $array_result[$i]['type'] ]['file_format'];

https://github.com/ziggi/zimg-host/blob/master/upload.class.php#L142
$new_name = hash('crc32b', microtime() . $files[$i]['name'] . $files[$i]['tmp_name'] . rand(0, 9999))  . '.' . $this->_allowed_types[$type]['file_format'];

I will make those changes later.

Tested and working. Thanks!


Image Partition Explainer does not work with PyTorch

Following the example in https://shap.readthedocs.io/en/latest/example_notebooks/image_examples/image_classification/Image Multi Class.html but using a PyTorch ResNet model instead of Tensorflow ResNet results to an error.
Tensorflow's preprocess_input outputs numpy objects, which is supported by both the Tensorflow ResNet model and shap's masker. However, PyTorch's ResNet expects Tensor type, which causes the error below in shap's masker.
---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
/tmp/ipykernel_9878/30609154.py in <module>
----> 1 shap_values = explainer(to_explain, max_evals=100, batch_size=10, outputs=shap.Explanation.argsort.flip[:4])

/opt/conda/lib/python3.7/site-packages/shap/explainers/_partition.py in __call__(self, max_evals, fixed_context, main_effects, error_bounds, batch_size, outputs, silent, *args)
    135         return super().__call__(
    136             *args, max_evals=max_evals, fixed_context=fixed_context, main_effects=main_effects, error_bounds=error_bounds, batch_size=batch_size,
--> 137             outputs=outputs, silent=silent
    138         )
    139

/opt/conda/lib/python3.7/site-packages/shap/explainers/_explainer.py in __call__(self, max_evals, main_effects, error_bounds, batch_size, outputs, silent, *args, **kwargs)
    258             row_result = self.explain_row(
    259                 *row_args, max_evals=max_evals, main_effects=main_effects, error_bounds=error_bounds,
--> 260                 batch_size=batch_size, outputs=outputs, silent=silent, **kwargs
    261             )
    262             values.append(row_result.get("values", None))

/opt/conda/lib/python3.7/site-packages/shap/explainers/_partition.py in explain_row(self, max_evals, main_effects, error_bounds, batch_size, outputs, silent, fixed_context, *row_args)
    158         # if not fixed background or no base value assigned then compute base value for a row
    159         if self._curr_base_value is None or not getattr(self.masker, "fixed_background", False):
--> 160             self._curr_base_value = fm(m00.reshape(1, -1), zero_index=0)[0] # the zero index param tells the masked model what the baseline is
    161         f11 = fm(~m00.reshape(1, -1))[0]
    162

/opt/conda/lib/python3.7/site-packages/shap/utils/_masked_model.py in __call__(self, masks, zero_index, batch_size)
     65
     66         else:
---> 67             return self._full_masking_call(masks, batch_size=batch_size)
     68
     69     def _full_masking_call(self, masks, zero_index=None, batch_size=None):

/opt/conda/lib/python3.7/site-packages/shap/utils/_masked_model.py in _full_masking_call(self, masks, zero_index, batch_size)
     91                     masked_inputs = self.masker(delta_ind, *self.args).copy()
     92                 else:
---> 93                     masked_inputs = self.masker(mask, *self.args)
     94
     95                 # wrap the masked inputs if they are not already in a tuple

/opt/conda/lib/python3.7/site-packages/shap/maskers/_image.py in __call__(self, mask, x)
     88
     89             elif self.mask_value == "inpaint_telea":
---> 90                 out = self.inpaint(x, ~mask, "INPAINT_TELEA")
     91             elif self.mask_value == "inpaint_ns":
     92                 out = self.inpaint(x, ~mask, "INPAINT_NS")

/opt/conda/lib/python3.7/site-packages/shap/maskers/_image.py in inpaint(self, x, mask, method)
    102         reshaped_mask = mask.reshape(self.input_shape).astype(np.uint8).max(2)
    103         if reshaped_mask.sum() == np.prod(self.input_shape[:-1]):
--> 104             out = x.reshape(self.input_shape).copy()
    105             out[:] = out.mean((0, 1))
    106             return out.flatten()

AttributeError: 'Tensor' object has no attribute 'copy'

Hi, @igygi, I faced a similar issue. In PR #2400 I added torch models support and a new notebook example.


Add support for InstallScripts

Reason
InstallScripts are useful for setting up environment variables, running processes during installation, and cleaning up files upon uninstallation, thereby enhancing the overall user experience.
Changes

Adds support for InstallScripts

InstallScripts can now be optionally configured by setting the depot[id]InstallScriptPath variable.
Example
jobs:
  deployToSteam:
    runs-on: ubuntu-latest
    steps:
      - uses: game-ci/steam-deploy@v3
        with:
          username: ${{ secrets.STEAM_USERNAME }}
          configVdf: ${{ secrets.STEAM_CONFIG_VDF}}
          appId: 1234560
          buildDescription: v1.2.3
          rootPath: build
          depot1Path: StandaloneWindows64
          depot1InstallScriptPath: StandaloneWindows64/install_script.vdf
          depot2Path: StandaloneLinux64
          releaseBranch: prerelease

Checklist

[x] Read the contribution guide and accept the code of conduct
[x] Readme (updated or not needed)
[ ] Tests (added, updated or not needed)

Documentation PR available at https://github.com/game-ci/documentation/pull/467


[READY] Peg mustache version

Problem
Now that mustache has reached a 1.0 major version they've specified that it needs to be run with at least ruby version 2.0. This is a problem with jruby as the with the 1.7x current release the underlying version reported is 1.9.3p392.
You can run jruby with one of the following to run it in 2.0 mode:

jruby --2.0
export JRUBY_OPTS=--2.0
echo "compat.version=2.0" > ~/.jrubyrc

but this isn't fully supported and for production apps isn't really an option.
Solution
Peg the version so that all releases up to the major 1 release can be used.

@darnould @Integralist @revett This make sense guys, I need it for an upstream gem.

:cake: