custom actions are lost events

Rasa version:0.14
Python version: 3.6.8
Operating system (windows 10):
Issue:
Encountered an exception while running action 'action_listen'. Bot will continue, but the events of the action are lost. Make sure to fix the exception in your custom code.
Content of action file (actions.py):
from __future__ import absolute_import
from __future__ import division
from __future__ import unicode_literals

#from rasa_core_sdk import Action
#from rasa_core_sdk.events import SlotSet
#from rasa_core.action import Action
#from rasa_core.events import SlotSet

from rasa_sdk import Action
from rasa_sdk.events import SlotSet

class ProductDetails(Action):
	def name(self):
		return ('action_pdetails')

	def run(self, dispatcher, tracker, domain):
			curl = ("http://localhost:5000/webhook/sms/demosendsms/")
			pro = tracker.get_slot("pdetail")
			current = curl.get(q=pro)
			details= current['pdetail']
			response = """The Product Details are {}""".format(details)
			dispatcher.utter_message(response)
			return [SlotSet('pdetail', pro)]

Content of domain file (domain.yml) (if used & relevant):
slots:
  pname:
   type: text

 plist:
   type: text

 pdetail:
   type: text

 order_number:
   type: text

 deny:
   type: text

 bestp:
   type: text

 sellp:
   type: text

entities:
 - order
 - pname
 - pdetails
 - plist
 - product
 - quantity
 - price
 - idproof
 - stock
 - outstock
 - confirm
 - time
 - order_id
 - bestp
 - sellp
 - toffer
 - doffer

intents:
  - greet
  - action_listen
  - action_restart
  - can_help
  - ask_product
  - product_name
  - ask_product_name
  - ask_product_nvalue
  - order
  - deny
  - affirm
  - need_quantity_value
  - product_details
  - product_list
  - product_price
  - order_confirm
  - today_offers
  - discount_offers
  - thanks
  - product_in_stock
  - order_details
  - order_time
  - outstock
  - idproof
  - out_of_scope
  - goodbye
  - suggest_best_product
  - most_selling_product
  - product_help
  - order_modify
  - order_cancel

actions:
 - utter_greet
 - utter_help
 - utter_can_help
 - utter_ask_product
 - utter_ask_price
 - utter_ask_quantity
 - utter_affirm
 - utter_goodbye
 - utter_order_confirm
 - utter_total_price
 - utter_order_id
 - utter_thanks
 - utter_id_proof
 - utter_product_price
 - utter_product_details
 - utter_product_list
 - utter_any_thing_else
 - utter_order_ready_time
 - utter_default
 - action_listen
 - action_restart
 - action_pdetails
 - utter_suggest_best_product
 - utter_today_offer_list
 - utter_discount_offers
 - utter_product_stock
 - utter_product_outstock
 - utter_product_help
 - utter_order_modify
 - utter_order_cancel

templates:
 utter_greet:
   - text: "Hey there! "
   - text: "Hi "
   - text: "Hey"
   - text: "Hello!"

 utter_can_help:
   - text: "I’ll be happy to help you with the Product Booking, Product list with their Details, Discounts, and your related queries"

 utter_product_help:
   - text: "I’ll be happy to help you with the Product Booking, Product list with their Details, Discounts, and your related queries"

@amn41 @sheldon @fungiboletus @phoet @ryanmcdonough @bderenzi @akelad

@rishi454 please don't list default rasa actions in your actions file. If you do, rasa expects a custom implementation. You need to remove these two from your actions list:
 - action_listen
 - action_restart


Removed list comprehensions in Gurobi QP interface

This should speedup the Gurobi interface by using numpy indexing instead of list comprehensions.
Related to #617

Coverage remained the same at 82.621% when pulling 4a24001351f9965e3021959efe8b763cee18182a on bstellato:master into e8c2a3358ab46d67ea427a352c732c07b7563876 on cvxgrp:master.

The build is failing for some flake8 issues not related to this PR. It seems that flake8 no longer likes some math symbols in the comments.

Hmm we'll have to look into this. Obviously not related to your change.


instrumentation: initial revision of libsplunk.so

Going to use this repo instead of https://github.com/signalfx/linux-autoinstrumentation/ which I will close.
This change has been partially reviewed here:
https://github.com/signalfx/linux-autoinstrumentation/pull/1
before I moved it into this repo.

It could be quite helpful moving forward for a build container so we have a shared dev/test environment. It would also be easier that way to intuitively tie into CI.


Shorten URLs to denofun modules

Fix https://github.com/galkowskit/denofun/issues/53
/cc @galkowskit

Yeah, that looks like a great idea from my side! :)


json: +-Inf and NaN

json encodes math.Inf and math.Nan (which is against json specification)
it fails to decode it
it isn't compatible with standard encoding/json which raises error
it isn't compatible with json standard (https://www.json.org http://www.ecma-international.org/publications/files/ECMA-ST/ECMA-404.pdf)
Quote:

Numeric values that cannot be represented as sequences of digits (such as Infinity and NaN) are not permitted

this piece of code does not work:
func main() {
	encVal := math.Inf(1)
	s, e := jsoniter.MarshalToString(encVal)
	log.Println(s, e)

	var dec float64
	err := jsoniter.UnmarshalFromString(s, &dec)
	log.Print(err)
}

Thank you


Readme - incorrect version number

Hi,
Thanks for an excellent simple plugin, I am using to teach Coder Dojo kids about animation in Flutter.
Readme version number appears to be incorrect.
dependencies:
animated_rotation: ^0.1.0 ...... fails
dependencies:
animated_rotation: ^0.0.2 ......works

Fixed and 1.0.0 released :)


pip install webui -  Failed to find MSCV 10

Hello,
At the first look at the description text for this module I was WOW just what I was looking for :)
I'm trying to install it on a Windows 7(64bit), Python 3.4.4(64bit). I have installed Microsoft Visual C++ 2010(x64 and x86).
I tried to install separately PySide but I got the same error even if PySide is installed.
This is what I get:
Collecting pyside==1.2.1 (from WebUI)
Using cached PySide-1.2.1.tar.gz
Installing collected packages: pyside, WebUI
Running setup.py install for pyside ... error
Complete output from command c:\python34\python.exe -u -c "import setuptools
, tokenize;file='C:\Users\ALINCM~1\AppData\Local\Temp\pip-build-zad1xx
li\pyside\setup.py';f=getattr(tokenize, 'open', open)(file);code=f.read().
replace('\r\n', '\n');f.close();exec(compile(code, file, 'exec'))" install -
-record C:\Users\ALINCM~1\AppData\Local\Temp\pip-xf08osxu-record\install-record.
txt --single-version-externally-managed --compile:
Removing C:\Users\ALINCM~1\AppData\Local\Temp\pip-build-zad1xxli\pyside\pysi
de_package
running install
running build
Python architecture is 64bit
nmake not found. Trying to initialize the MSVC env...
Searching MSVC compiler version 10.0
error: Failed to find the MSVC compiler version 10.0 on your system.
Is there a quick fix for this?
I'm not using a compatible Python version?
Thank you for your work

I get this same error. Try install with:
pip install git+https://github.com/Widdershin/flask-desktop.git

This commando install WebUi directly from git repository (use PyQt5). It seems, the pypi version is outdated, using PySide (that require extra dependencies).

Tried using "pip install git+https://github.com/Widdershin/flask-desktop.git", it didn't work, so I copied as zip the entire repository and run "python setup.py install" - didn't work either.
Most probably it's because I use Python 3.4 (compatibility reasons), I saw that PyQT5 it's for Python 3.5 and up.
When I'll move to a bigger version than Python 3.4, I think I will not have this problem anymore :)


Add buffer to progress

When looking at generated DOM, I don't quite get why the approach is complex/different for a relative simple component. For linear one, bar & track doing translate with total 200% width movement, buffer on the other hand doing width increment. Circular one could be more complex with left/right split.

I changed track to be static in linear, and value to grow in width rather than moving

NIT: Linear Progress has a min-width of 400px, not sure if there was a design decision behind this ? - but on smaller screen sizes this may have an impact. ( Doesn't really concern this pr so a change can follow up if needed )


Change back to old Error inheritance

Babel class extension of built-in types is causing a whole host of issues with older browsers (especially IE <= 10 which does not allow extending built-in types which basically broke feathers-client but only when an error ocurred) but also other environments (like React native).
I am suggesting to go back to old school JavaScript inheritance and avoid all those problems. From the test it seems to be fully backwards compatible.

:shipit:

Yup :shipit:
The only reason I had class stuff in there is I was trying to be fancy and future forward and use the constructor name to identify the type, but then Babel choked on it so I was adding the props manually anyway.
I think this is a good call. It might actually help clear up the stack trace a bit. I was actually thinking that I'd ideally like to not have to use Babel on this lib as well because currently most stacktraces look like this:
/app/node_modules/feathers-errors/lib/index.js:21:28ExtendableBuiltin
/app/node_modules/feathers-errors/lib/index.js:96:116FeathersError
/app/node_modules/feathers-errors/lib/index.js:149:117new NotAuthenticated
/app/node_modules/feathers-authentication/lib/hooks/authenticate.js:79:31none
/app/node_modules/async-listener/index.js:421:23propagateAslWrapper
/app/node_modules/async-listener/glue.js:188:31none
/app/node_modules/async-listener/index.js:458:70none

Which is just okay but not great.

Cool, thanks guys! I'll make a minor release. Greenkeeper should catch if it's breaking anything but I don't think so.


Fix bug when filtering by state in Edge.

This adds a polyfill for Edge to fix bug when filtering by state; due to lack of support for HTMLCollection.prototype.forEach. See below.

Hey @patricknelson! Want to confirm if this fix is necessary? According to caniuse, Edge does support the use of .forEach. Curious why you seem to have run into an issue with it. Was it an older version, or perhaps IE?
Separately, I'm not sure if adding polyfills for specific JS methods is the best approach. It could get messy fast (if there are more methods in the future we want to support, we'd have a long list of polyfills in our code!), and there are good, if not better, alternatives for .forEach that we could easily use instead (e.g., plain old for loop or for... of).
Let me know if this is still creating an issue for you thought, happy to dig into this further!

The problem was not .forEach missing on Array or NodeList, it was missing on HTMLCollection (screenshot in the first comment). This is Edge 17.
Again, I agree but this is a quick fix. It is messy and I wouldn't have used .foreEach() but there are also arrow functions and JS templates as well as modules. Right now we're optimized for speed and not entirely cleanliness. I'd suggest a refactor too;
In fact, there's a ticket for that; would you like to submit a PR for #166?

Yikes. This is indeed ugly, but if it patches Edge for now, let's do it and submit a follow-up to clean up.
The big question is how much we care about older browsers. If we do actually care, we should create a real deploy pipeline with babel and webpack or something.

Precisely my thought @awong-dev. That requires a more comprehensive pipeline and dev workflow which would be good to dig into eventually.


Reverse order of entity children when displaying variable tree

The issue is, given this entity tree

we end up with this list of entities

Instead, we want Household observation type to appear immediately below Household. Reversing the order of the children of an entity will address this.
This may not be a long term solution, but we will try it for now.

Not much context for this ticket, so glad I'm not a reviewer of the PR.

Sorry, I got distracted when I made this issue and never updated the description. Just updated it.


⚠ SolarOscp has degraded performance

In 7502299, SolarOscp ($SOLAROSCP_URL) experienced degraded performance:

HTTP code: 200
Response time: 382 ms

Resolved: SolarOscp performance has improved in 9fd752b after 17 minutes.


Null properties do not play well with dynamic and null propagation/null coalescing operators

I have this code:
dynamic obj = JsonConvert.DeserializeObject("{ \"a\": null }");
CASE 1:
I'm trying to use null propagation operator:
var c = obj.a?.b;
The expected result is to have 'c' equal null. But, in reality the following exception is thrown:
Microsoft.CSharp.RuntimeBinder.RuntimeBinderException: 'Newtonsoft.Json.Linq.JValue' does not contain a definition for 'b'
CASE 2:
I'm trying to use null coalescing operator:
var c = obj.a ?? 666;
The expected result is to have value '666' in the variable 'c'. In reality, it contains a JValue with Type.Null.
Hovewer, this:
var c = obj.b ?? 666;
produces expected result.
CONCLUSION:
To me, both of thess cases seem to produce very unexpected, non-intuitive behavior and are highly confusing for the reader of the code. Also, it is pretty hard to spot these issues. What this means is that to avoid such pitfalls we  should stop using dynamic in conjunction with Newtonsoft.Json altogether. And that is not what many people would like to do :).
So, may be i'm missing something here and there are known ways to make this work as expected. Well, I know that I can traverse my JToken and remove all JValues with Type.Null, but that is a crazy thing to do in my opinion.

Second this, it would be nice to have it play well with null conditional operators


EMASTER & MASTER Output Error

Hi,
When I run the code on the EMASTER and MASTER files I get an error and no data output.

In metastockread (line 124)
Warning: The following error was caught while executing 'onCleanup' class destructor:
Invalid file identifier. Use fopen to generate a valid file identifier.

I do get a data output when I run on the XMASTER file.
I've attached the raw data to the following download link:
http://expirebox.com/download/a0d58ac48a5b16af739e099a3a2f70df.html
See the three images below, for the output from Matlab for the EMASTER, MASTER and XMASTER files.
Regards,
Joe

Hi thanks for the report, could you please provide the steps to reproduce the error?

Run metastockread.m
"Load metastock file" explorer window opens. Browse to "Joe_Data" Folder
Select "EMASTER", "MASTER" or "XMASTER"

@joeleen2016 Try now, it should not warn anymore.

Brilliant ! Thank you !

@joeleen2016 Thanks again for taking the time to report the issue, very appreciated.


RemoveCallsToSystemGc

Fixes #1664

For the proper review:

[ ] what would the considerations be on adding a reference for instance in at the bottom of rewrite-java/src/main/resources/META-INF/rewrite/java-best-practices.yml ?
I'm thinking users might be a bit surprised if this recipe is executed by default, but not referencing it anywhere might also be odd.
[ ] Should this recipe also cover Runtime.getRuntime().gc() as mentioned in RSPEC-1215 ?

Github's "re-request review" buttons behave strange to me in this issue.
Anyway - kindly requesting another review @pway99, @sambsnyd, @timtebeek or anyone.

Kindly requesting yet another review @pway99, @sambsnyd, @timtebeek or anyone.

 what would the considerations be on adding a reference for instance in at the bottom of `rewrite-java/src/main/resources/META-INF/rewrite/java-best-practices.yml` ?
  I'm thinking users might be a bit surprised if this recipe is executed by default, but not referencing it anywhere might also be odd.

This is a good question @timtebeek; I would agree that the recipe should be referenced and that the java-best-practices aka Use the Java standard library in a way that is most idiomatic. a good place for this one.
I also agree this one is good to go!  Thanks @mccartney


Cython compiling error while installing

Hi again :)
I would like to try out your One Class Random Forest Classifier (I read your paper as well).
However, in installing the code, I get some errors related to Cython files.
For example,

Do you know the problem or is there something I'm doing wrong?
Best,
Minori

@ngoix have you modified this file (fast_dict.pxd). If not it means that the error comes from scikit itself.
@minoriinoue could you tell us you numpy and cython version?

Thanks for the reply, @RomainBrault :)
Versions should be:
Numpy version: 1.13.3
cython version: 0.27.3

Ok it seems that the error comes from gcc-7 and higher (at the time it has been tested again gcc-6/gcc-5). Note that sklean 0.19 also fixed in the same file a bug related to clang.
@minoriinoue In the file sklearn/utils/fast_dict.pxd you should add right below the line
cimport numpy as np
a new line
import numpy as np
I proposed a pull request fixing this available in the branch cythonize-patch-1. Waiting for feedback on this fix before merge.

NB: sklearn 0.20 proposed to simply remove the DTYPE and ITYPE lines instead of the 'import numpy as np'.


bug_fix: updated docker-base image and fixed test

Signed-off-by: Gabriel<EMAIL_ADDRESS>Description
Updated the test to a fail state. Ran make docker-update-base and the test passed. Also did a quick manual verification on the dirty image.
This PR fixes https://github.com/datawire/ambassador/issues/1211

@kflynn This PR should fix the issue with set-cookie.


Gulp accessibility plugins for foundation-zurb-template?

I'm just starting to use them, but the following could be nice to have baked in given the focus on a11y in F6. I'll give some more feedback as I use them (and others can chime in). Sort of a tentative feature request pending validation.
https://github.com/Charca/gulp-arialinter
https://github.com/yargalot/gulp-accessibility

gulp-arialinter wants NPM4.
node_modules/gulp-arialinter/node_modules/arialinter/node_modules/jsdom/lib/jsdom.js:3
() => `jsdom 7.x onward only works on Node.js 4 or newer: https://github.com/t
 ^
SyntaxError: Unexpected token )

gulp-accessibility is working with the current stack, but pretty verbose by default. Having a separate task might be good, I just tossed it in pages to kick the tires.

Very interesting! We'd like to keep Node 0.12 support for now, though, so gulp-accessibility would probably be the way to go. I'm going to roll this into our main feedback thread: #7760


[BUG][PSA] Arborist loses all data upon saving/changing scenes

Godot version
v4.2.2.stable.official [15073afe3]
Plugin version
1.3.2 (asset library)
Platform
Windows 10
Issue description
Okay this has happened a few times already, but I'm not that far into the project that catastrophe has happened. Some times when I save another scene and change to the scene with gardener, the plugin simply goes haywire and stops working, spewing endless stream of errors:

From this situation I've tried 2 different options:

Save and reload the project. After reloading I'm greeted with following errors:

res://addons/dreadpon.spatial_gardener/arborist/arborist.gd:89 - Invalid call. Nonexistent function 'duplicate_tree' in base 'Nil'.
INFO: gardener.gd (Gardener): Found existing Arborist
  res://addons/dreadpon.spatial_gardener/arborist/mmi_octree/mmi_octree_node.gd:316 - Invalid set index 'cast_shadow' (on base: 'Nil') with value of type 'int'.
  res://addons/dreadpon.spatial_gardener/arborist/mmi_octree/mmi_octree_node.gd:316 - Invalid set index 'cast_shadow' (on base: 'Nil') with value of type 'int'.
  res://addons/dreadpon.spatial_gardener/arborist/mmi_octree/mmi_octree_node.gd:316 - Invalid set index 'cast_shadow' (on base: 'Nil') with value of type 'int'.
  res://addons/dreadpon.spatial_gardener/arborist/mmi_octree/mmi_octree_node.gd:316 - Invalid set index 'cast_shadow' (on base: 'Nil') with value of type 'int'.
  res://addons/dreadpon.spatial_gardener/arborist/mmi_octree/mmi_octree_node.gd:526 - Cannot call method 'get_node_or_null' on a null value.
  res://addons/dreadpon.spatial_gardener/arborist/mmi_octree/mmi_octree_node.gd:567 - Cannot call method 'get_child_count' on a null value.
  res://addons/dreadpon.spatial_gardener/arborist/mmi_octree/mmi_octree_node.gd:526 - Cannot call method 'get_node_or_null' on a null value.
  res://addons/dreadpon.spatial_gardener/arborist/mmi_octree/mmi_octree_node.gd:567 - Cannot call method 'get_child_count' on a null value.
  res://addons/dreadpon.spatial_gardener/arborist/mmi_octree/mmi_octree_node.gd:526 - Cannot call method 'get_node_or_null' on a null value.
  res://addons/dreadpon.spatial_gardener/arborist/mmi_octree/mmi_octree_node.gd:567 - Cannot call method 'get_child_count' on a null value.
  res://addons/dreadpon.spatial_gardener/arborist/mmi_octree/mmi_octree_node.gd:526 - Cannot call method 'get_node_or_null' on a null value.
  res://addons/dreadpon.spatial_gardener/arborist/mmi_octree/mmi_octree_node.gd:567 - Cannot call method 'get_child_count' on a null value.
  res://addons/dreadpon.spatial_gardener/arborist/mmi_octree/mmi_octree_node.gd:526 - Cannot call method 'get_node_or_null' on a null value.
  res://addons/dreadpon.spatial_gardener/arborist/mmi_octree/mmi_octree_node.gd:567 - Cannot call method 'get_child_count' on a null value.
  res://addons/dreadpon.spatial_gardener/arborist/mmi_octree/mmi_octree_node.gd:526 - Cannot call method 'get_node_or_null' on a null value.
  res://addons/dreadpon.spatial_gardener/arborist/mmi_octree/mmi_octree_node.gd:567 - Cannot call method 'get_child_count' on a null value.
  res://addons/dreadpon.spatial_gardener/arborist/mmi_octree/mmi_octree_node.gd:526 - Cannot call method 'get_node_or_null' on a null value.
  res://addons/dreadpon.spatial_gardener/arborist/mmi_octree/mmi_octree_node.gd:567 - Cannot call method 'get_child_count' on a null value.
  res://addons/dreadpon.spatial_gardener/arborist/mmi_octree/mmi_octree_node.gd:526 - Cannot call method 'get_node_or_null' on a null value.
  res://addons/dreadpon.spatial_gardener/arborist/mmi_octree/mmi_octree_node.gd:567 - Cannot call method 'get_child_count' on a null value.
  res://addons/dreadpon.spatial_gardener/arborist/mmi_octree/mmi_octree_node.gd:316 - Invalid set index 'cast_shadow' (on base: 'Nil') with value of type 'int'.

All the props are in place, but the gardener doesn't recognize them. Inspecting diff of the scene file it seems that some data has been permanently lost.
OR

Reload saved scene. This seems to fix the issue, so as long as I know to do this I can prevent data loss (unless there was unsaved changes).

And this is where the PSA comes in:
If Gardener starts spamming errors, reload saved scene. DO NOT SAVE.
Steps to Reproduce

Make a scene with Gardener (let's say test.tscn), and second unrelated scene (character.tscn) instantiated in that scene.
Have both scenes opened in editor.
Save the character.tscn.
Change tabs to test.tscn.

Minimal reproduction project
GardererMRP.zip
Do steps 2-4 with this MRP.

Might be a problem with destruction code added in 1.3.2 which executes in _exit_tree, which apparently is called when switching scenes.
Will need to work this version and if true, look for an alternative which triggers on the actual node deletion (when scene is being closed), probably a NOTIFICATION_PREDELETE

If you have the time to test the MRP in 1.3.1 - this might help in locating the problem

I can confirm that this is indeed a regression in 1.3.2. I couldn't reproduce the bug with 1.3.1

Fixed with https://github.com/dreadpon/godot_spatial_gardener/commit/902fe9c1e4de209b2d9d867e297335c14cc40e91
But not 100% sure some echo of this problem will not return in the future. The fix was weird. Turns out we were already using NOTIFICATION_PREDELETE, but inside it we set certain references to null and it really messed with Godot's resource caching (?) system. These certain references weren't mandatory to be set to null, and it's probably better if they aren't, but the observed problem and the fix don't quite add up, especially considering the trigger requirements.
Will count this as closed for now, we'll return to this if further problems are revealed on your side @aXu-AP. No release yet, please use master. Found other things I need to investigate and fix before releasing 1.3.3.
Will keep this Issue open for PSA visibility.

The fix above seems to cause bugs in managing MultiMeshes (some of them get left behind, leading to weird behavior). Discovered by running autotest on 1.3.2 compared to 1.3.1.
Fixed in 729b413 by reverting some of the changes (the fix itself still applies).

@aXu-AP would appreciate it if you put the master version through some basic real-world work cases to see if it's stable enough to release a hotfix. Your MRP seems to work fine though, and my autotests didn't show any more problems.

I updated Gardener to master branch in my game, and so far it does no problems have risen 👍

Good to know, will make a release tomorrow.
Don't forget to use Git!)

Yeah I did do a commit before in case something goes wrong 😎

Closed by 902fe9c1e4de209b2d9d867e297335c14cc40e91


cannot use hook.Entries (type []logrus.Entry) as type []*logrus.Entry in return argument

after import  package "github.com/stellar/go/clients/horizon"
compiler says
go/src/github.com/stellar/go/support/log/entry.go:125:14: cannot use hook.Entries (type []logrus.Entry) as type []*logrus.Entry in return argument

Very likely that you did not use glide to install dependencies: https://github.com/stellar/go#dependencies

Sorry I read services/horizon instead of clients/horizon.

@bartekn I hotfix it in my local repo with change []*logrus.Entry to []logrus.Entry in 2 files

What dependency management tool are you using in your workflow?

Just run example build_transaction
https://github.com/stellar/go/tree/master/examples/build_transaction

I'm getting the same error here, using dep to manage dependencies. Any idea what is it about?

glide seems to ignore transitive dependencies. This is one of many reasons we switched to dep recently. You have two options:

Switch to dep too, and update Git revision of github.com/stellar/go to 435de795b0e1ae4ea4638344c5637bff14e31c99 or later - this is where we started using dep (recommended).
If you really want to stick to glide add github.com/sirupsen/logrus at 68cec9f21fbf3ea8d8f98c044bc6ce05f17b267a to your glide.yaml and/or glide.lock files.


The useController example code has an error and differs between JavaScript and TypeScript

Version Number
latest website
Codesandbox/Expo snack
none
Steps to reproduce

Go to the useController documentation.
Observe that line 7 is defining name as a const, even though name is already defined in this scope.
Click TS to see the Typescript version. Observe that the TypeScript version of the code is a completely different example, when typically it would be the same example but with typescript annotations added.

Expected behaviour

name would not be re-defined as a const in a scope where it is already defined.
The TS example would correspond directly to the JS example

What browsers are you seeing the problem on?
Firefox, Chrome, Safari, Edge
Relevant log output
As an aside: The bug report template does not accommodate documentation bugs well.

Code of Conduct

[X] I agree to follow this project's Code of Conduct

thanks for report, what's the issue in there, we don't keep codesandbox 100% the same, but the code should be repesetnt the usage.

Click TS to see the Typescript version. Observe that the TypeScript version of the code is a completely different example, when typically it would be the same example but with typescript annotations added.

can you past the code for for more detail?

JS:
`import React from "react";
import { TextField } from "@material-ui/core";
import { useController, useForm } from "react-hook-form";
function Input({ control, name }) {
const {
field: { onChange, onBlur, name, value, ref },
fieldState: { invalid, isTouched, isDirty },
formState: { touchedFields, dirtyFields }
} = useController({
name,
control,
rules: { required: true },
defaultValue: "",
});
return (
<TextField
onChange={onChange} // send value to hook form
onBlur={onBlur} // notify when input is touched/blur
value={value} // input value
name={name} // send down the input name
inputRef={ref} // send input ref, so we can focus on input when error appear
/>
);
}
function App() {
const { control } = useForm();
return ;
}`
TS:
`
import * as React from "react";
import { useForm, useController, UseControllerProps } from "react-hook-form";
type FormValues = {
FirstName: string;
};
function Input(props: UseControllerProps) {
const { field, fieldState } = useController(props);
return (

<input {...field} placeholder={props.name} />
{fieldState.isTouched && "Touched"}
{fieldState.isDirty && "Dirty"}
{fieldState.invalid ? "invalid" : "valid"}

);
}
export default function App() {
const { handleSubmit, control } = useForm({
defaultValues: {
FirstName: ""
},
mode: "onChange"
});
const onSubmit = (data: FormValues) => console.log(data);
return (

<Input control={control} name="FirstName" rules={{ required: true }} />

);
}
`
These are completely different examples. Typically when showing a TS example alongside a JS example you use the same example. Doing it that way makes it so you have fewer examples to maintain: You only have to create a TS example, and the JS example is auto-generated from by stripping the TS type annotations.


source_files must be set

Hi,
I am trying to setup coveralls with Travis and I have a source_files must be set  error
$ php coveralls.phar -v
Load coverage clover log:
  - /home/travis/build/Toolwatchapp/tw-backend/application/tests/build/logs/clover.xml
Collect git info
Read environment variables
source_files must be set

#0 phar:///home/travis/build/Toolwatchapp/tw-backend/application/tests/coveralls.phar/src/Satooshi/Bundle/CoverallsV1Bundle/Entity/JsonFile.php(154): Satooshi\Bundle\CoverallsV1Bundle\Entity\JsonFile->ensureJobs()
#1 phar:///home/travis/build/Toolwatchapp/tw-backend/application/tests/coveralls.phar/src/Satooshi/Bundle/CoverallsV1Bundle/Api/Jobs.php(98): Satooshi\Bundle\CoverallsV1Bundle\Entity\JsonFile->fillJobs(Array)
#2 phar:///home/travis/build/Toolwatchapp/tw-backend/application/tests/coveralls.phar/src/Satooshi/Bundle/CoverallsV1Bundle/Repository/JobsRepository.php(123): Satooshi\Bundle\CoverallsV1Bundle\Api\Jobs->collectEnvVars(Array)
#3 phar:///home/travis/build/Toolwatchapp/tw-backend/application/tests/coveralls.phar/src/Satooshi/Bundle/CoverallsV1Bundle/Repository/JobsRepository.php(66): Satooshi\Bundle\CoverallsV1Bundle\Repository\JobsRepository->collectEnvVars()
#4 phar:///home/travis/build/Toolwatchapp/tw-backend/application/tests/coveralls.phar/src/Satooshi/Bundle/CoverallsV1Bundle/Command/CoverallsV1JobsCommand.php(161): Satooshi\Bundle\CoverallsV1Bundle\Repository\JobsRepository->persist()
#5 phar:///home/travis/build/Toolwatchapp/tw-backend/application/tests/coveralls.phar/src/Satooshi/Bundle/CoverallsV1Bundle/Command/CoverallsV1JobsCommand.php(114): Satooshi\Bundle\CoverallsV1Bundle\Command\CoverallsV1JobsCommand->executeApi(Object(Satooshi\Bundle\CoverallsV1Bundle\Config\Configuration))
#6 phar:///home/travis/build/Toolwatchapp/tw-backend/application/tests/coveralls.phar/vendor/symfony/console/Command/Command.php(259): Satooshi\Bundle\CoverallsV1Bundle\Command\CoverallsV1JobsCommand->execute(Object(Symfony\Component\Console\Input\ArgvInput), Object(Symfony\Component\Console\Output\ConsoleOutput))
#7 phar:///home/travis/build/Toolwatchapp/tw-backend/application/tests/coveralls.phar/vendor/symfony/console/Application.php(840): Symfony\Component\Console\Command\Command->run(Object(Symfony\Component\Console\Input\ArgvInput), Object(Symfony\Component\Console\Output\ConsoleOutput))
#8 phar:///home/travis/build/Toolwatchapp/tw-backend/application/tests/coveralls.phar/vendor/symfony/console/Application.php(192): Symfony\Component\Console\Application->doRunCommand(Object(Satooshi\Bundle\CoverallsV1Bundle\Command\CoverallsV1JobsCommand), Object(Symfony\Component\Console\Input\ArgvInput), Object(Symfony\Component\Console\Output\ConsoleOutput))
#9 phar:///home/travis/build/Toolwatchapp/tw-backend/application/tests/coveralls.phar/vendor/symfony/console/Application.php(123): Symfony\Component\Console\Application->doRun(Object(Symfony\Component\Console\Input\ArgvInput), Object(Symfony\Component\Console\Output\ConsoleOutput))
#10 phar:///home/travis/build/Toolwatchapp/tw-backend/application/tests/coveralls.phar/bin/coveralls(21): Symfony\Component\Console\Application->run()
#11 /home/travis/build/Toolwatchapp/tw-backend/application/tests/coveralls.phar(12): require('phar:///home/tr...')
#12 {main}

The build is here https://travis-ci.org/Toolwatchapp/tw-backend/builds/174537515 and the commit built is here https://github.com/Toolwatchapp/tw-backend/tree/1220893cdce17553ab98ffc28efd028108ec770c
Thanks,
M.

The solution is to add to -r option with the root directory of travis (i.e. php coveralls.phar -v -r  /home/travis/build/Toolwatchapp/tw-backend/)


Support for ridge plots

As we previously discussed, support for ridge plots (see Seurat's RidgePlot function) in DBPlot would be nice.

Check out the new dittoPlot and associated dittoRidgePlot wrapper =)


Inconsistent promp when calling LLMGraphTransformer with ChatOllama using from langchain_ollama import ChatOllama

Checked other resources

[X] I added a very descriptive title to this issue.
[X] I searched the LangChain documentation with the integrated search.
[X] I used the GitHub search to find a similar question and didn't find it.
[X] I am sure that this is a bug in LangChain rather than my code.
[X] The bug is not resolved by updating to the latest stable version of LangChain (or the specific integration package).

Example Code
from langchain_experimental.graph_transformers import LLMGraphTransformer
from langchain_community.chat_models import ChatOllama
from langchain_core.documents import Document
from langchain_community.graphs.graph_document import GraphDocument

llm = ChatOllama(model=local model)
llm_transformer = LLMGraphTransformer(
    llm=llm,
)
doc= Document(page_content="The dog has a ball which is red")
graph=llm_transformer.convert_to_graph_documents([doc])
print(graph)

from langchain_ollama import ChatOllama

llm = ChatOllama(model=local model)
llm_transformer = LLMGraphTransformer(
    llm=llm,
)
doc= Document(page_content="The dog has a ball which is red")
graph=llm_transformer.convert_to_graph_documents([doc])
print(graph)

Error Message and Stack Trace (if applicable)
No response
Description
In this code example the outcome of of processing a simple text through LLMGraphTransformer changes if I replace the definition of ChatOllama from
from langchain_community.chat_models import ChatOllama
to
from langchain_ollama import ChatOllama
Output :
from langchain_community.chat_models import ChatOllama
page_content='The dog has a ball which is red'
[GraphDocument(nodes=[Node(id='dog', type='Animal', properties={}), Node(id='ball', type='Object', properties={})], relationships=[Relationship(source=Node(id='dog', type='Animal', properties={}), target=Node(id='ball', type='Object', properties={}), type='HAS_OBJECT', properties={})], source=Document(metadata={}, page_content='The dog has a ball which is red'))]
from langchain_ollama import ChatOllama
page_content='The dog has a ball which is red'
[GraphDocument(nodes=[Node(id='John Doe', properties={}), Node(id='Red Ball', properties={})], relationships=[], source=Document(metadata={}, page_content='The dog has a ball which is red'))]
System Info
langchain_core '0.3.1'
langchain_community '0.3.0'
langchain_experimental '0.3.0'
langchain_ollama  '0.2.0'

It seems to be related by looking at ollama Debug logs, to an incorrect prompt syntax
"[INST] You are a top-tier algorithm designed for extracting ..."
becomes
prompt="[AVAILABLE_TOOLS] [{"type":"function","function":{"name":"DynamicGraph","description":"Represents a graph document consisting of nodes and relationships.","parameters":{"type":"object","required":["nodes","relationships"],"properties":{"nodes":{"type":"","description":"List of nodes"},"relationships":{"type":"","description":"List of relationships"}}}}}][/AVAILABLE_TOOLS][INST] # Knowledge Graph Instructions for GPT-4...
There might have been changed in chatollama to accomodate for function calling ("With Structured Output") that are not working correctly and referecing instruction for GPT4


[Bug]: OpenAI's embedding model is limited to 8191 in length, but the truncated length in the code is 8196

Is there an existing issue for the same bug?

[X] I have checked the existing issues.

Branch name
main
Commit ID
c59c1b6
Other environment information
No response
Actual behavior
[ERROR]Embedding error:Error code: 400 - {'error': {'message': "This model's maximum context length is 8192 tokens, however you requested 8196 tokens (8196 in your prompt; 0 for the completion). Please reduce your prompt; or completion length.

Expected behavior
No response
Steps to reproduce
use graphrag to build knowledge base

Additional information
No response

hi @lixiangtiandashen  thanks for mention that, will fix it soon


Add Plex connection selector

When signing into Plex, ErsatzTV imports the wrong IP for my Plex Server, in this case <IP_ADDRESS>. The correct IP is <IP_ADDRESS>, the <IP_ADDRESS> IP is the IP of another (virtual) network adapter. Would it be possible to add an IP selector or manual input?

The correct address is the fourth entry in a list of 6 addresses, in the meantime, would manually moving the entry into the first place using a sql editor work for my case?

I would try shutting down the app and running the following queries before restarting it
update PlexConnection set IsActive = 0
update PlexConnection set IsActive = 1 where Uri = '[the correct uri]'


Workflow for ingesting Alerting

This PR provides a workflow for ingesting alert rules from consumer charms. This works as follows

Consumer charms must have a directory named prometheus_alert_rules at the top level in their source repository.
This directory can contain any number of files with extension .rule. Each such file can have one alert rule in YAML format for example

- alert: ScheduleableInstances
        expr: sum(hypervisor_schedulable_instances{nova_service_status="enabled",%%juju_topology%%}) < {{ min_schedulable_instances }}
        for: {{ wait_time }}
        labels:
          override_group_by: host
          severity: page
        annotations:
          description: |
            There is low capacity on the cloud, less than {{ min_schedulable_instances }} instances can be scheduled
          summary: There is low capacity on the cloud, less than {{ min_schedulable_instances }} instances can be scheduled

All occurrences of %%juju_topology%% in the alert rule expr are replaced by appropriate Juju topology labels of the charm submitting the alert rule.

@balbirthomas I think we need more docs and example about why using %%juju_topology%% for each metric is so important.

@balbirthomas I think we need more docs and example about why using %%juju_topology%% for each metric is so important.

Agreed. I am about to start on unit tests and then documentation. Hence this is still a draft PR.


Add i18n support for projects interface and Japanese message catalogue

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)

Proposed changes

This PR adds i18n support for Project interface and Japanese message catalogue for it.
Checklist

[x] I have read the contribution guidelines
[ ] For non-bugfix PRs, I have discussed this change on the mailing list/slack team.
[x] I have run grunt to verify the unit tests pass
[ ] I have added suitable unit tests to cover the new/changed functionality

Coverage remained the same at 74.894% when pulling 392ed706fdd4a33333ec35a5a7b7ae49b0130c98 on node-red-hitachi:0.19-editor-projects-i18n-jp into 0ff0f25aaff479246030f1aab5b77fe0aaf166c7 on node-red:0.19.


Install cuDNN

Download
tar zxvf cudnn-6.5-linux-x64-v2.tgz
cp -a libcudnn* $CUDA_HOME/lib64/
cp cudnn.h $CUDA_HOME/include/

Can it be fully scripted or does it require a NVidia developer account to download?

I could not find a direct link.
On Sat, Jul 11, 2015, 14:11 Olivier Grisel<EMAIL_ADDRESS>wrote:

Can it be fully scripted or does it require a NVidia developer account to
download?
—
Reply to this email directly or view it on GitHub
https://github.com/deeplearningparis/dl-machine/issues/12#issuecomment-120650173
.

@jdemouth may be able to confirm?

It requires an account as far as I know. I'll check with our marketing if there's a workaround.

thanks!
On Sat, Jul 11, 2015 at 8:46 PM, Julien Demouth<EMAIL_ADDRESS>wrote:

It requires an account as far as I know. I'll check with our marketing if
there's a workaround.
—
Reply to this email directly or view it on GitHub
https://github.com/deeplearningparis/dl-machine/issues/12#issuecomment-120652098
.

is there a workaround?

Not yet. Our marketing is still looking for a solution...

Link for v4: https://github.com/NVIDIA/nvidia-docker/blob/8c340c5c2907f8bb283135c98e468dbecaf8c1ad/ubuntu-14.04/cuda/7.5/devel/cudnn4/Dockerfile#L13
v3: https://github.com/NVIDIA/nvidia-docker/blob/8c340c5c2907f8bb283135c98e468dbecaf8c1ad/ubuntu-14.04/cuda/7.5/devel/cudnn3/Dockerfile#L13

Good news!


Adding a project with full github url

It would be nice if we could add a project by adding a link to an github project instead by typing it manually.
For instance: Pasting the link to you project (https://github.com/vfeskov/win-a-beer) doesn't find any projects to watch

definitely a must, thanks for the idea!


Geocoder input is left uncollapsed after selecting an item

When auto-collapsing is used, when the user clicks on a result, the box is left open. It should be auto-closed, as the user has made a definitive decision and the action is complete.
I have created a workaround, but this is unsatisfactory because it force-clears the input:
// Auto-close on select; see: https://maplibre.org/maplibre-gl-geocoder/classes/default.html#on
geocoder.on ('result', function () {
	document.querySelector ('.maplibregl-ctrl-geocoder--button').click ();    // Click to remove the search value
	document.querySelector ('.maplibregl-ctrl-geocoder--input').blur ();      // Move away from the search box
});

I tried using the _collapse method, which seems to be advertised as public despite the underscore, but this had no effect, as I see its internal implementation ignores the call if there is focus.

"advertise as public" is probably an incorrect documentation migration due to the migration to typescript, _ methods are private.
Can you link to a jsbin showcasing this issue?

Reproduce case:

Go to this Codepen example I found: https://codepen.io/tsamaya/pen/KKxGwLj
In the Javascript, add collapsed: true to the example given, in the new MaplibreGeocoder options parameter
The map will load below, with the control collapsed by default
Hover over the control
Click in the geocoder input box
Type a location, e.g. Paris, and press return
[The map will go to the first location, and place a marker, which is a very odd behaviour but that's a separate problem]
Click on the drop-down that appears, to select a desired item
The map moves to that location
The box is left open, even though the user has made a selection. The user has to erase the contents, and then blur away, in order to get rid of it, which is not intuitive.

The last step should not be necessary - selection of an item from the drop down should auto-close the control.


interface methods should be marked 'shared'

In Java, all interface methods are public by default. Therefore, the converter should mark them shared, even if they don't have an explicit public!

This should be fixed now.

Thanks mate!


GPS send null position

I was in version 2023-7-13 and everything in the tracker was working as expected
I updated to version 2023-8-24
with that version, (after the GPS acquire the position, led blinking) the tracker send out a packet with null position.2023-08-25 12:59:39 CEST: IW2EJH-5>APLRT1,WIDE1-1,qAR,iw2ejh-11:>https://github.com/richonguzman/LoRa_APRS_Tracker 2023.08.19
type: status
srccallsign: IW2EJH-5
dstcallsign: APLRT1
status: https://github.com/richonguzman/LoRa_APRS_Tracker 2023.08.19
2023-08-25 13:02:35 CEST: IW2EJH-5>APLRT1,WIDE1-1,qAR,iw2ejh-11:!null7?TWQ$XR[E8Q Bat=4.20V (87mA) [Invalid packet]
type: location
srccallsign: IW2EJH-5
dstcallsign: APLRT1
messaging: 0
while with the previous FW, the packet looks like:
2023-08-25 12:46:13 CEST: IW2EJH-5>APLRT1,WIDE1-1,qAR,iw2ejh-11:!/7?UVQ$Xo[EIQ Bat=4.12V (-90mA)
type: location
format: compressed
srccallsign: IW2EJH-5
dstcallsign: APLRT1
latitude: 45.813525461638 °
longitude: 10.070165858986 °
altitude:<PHONE_NUMBER>6278 m
symboltable: /
symbolcode: [
messaging: 0
posresolution: 0.291 m
comment: Bat=4.12V (-90mA)
I rolled back to that version and it work fine, I put again the new version and got the same behaviour.
additional info:  on the display the gps the lat and long are correctly displayed

Hi,
can you tell me which board are you using?
my last firmware available for the Tracker is 2023.08.19 so this is strange

ops sorry, yes version is 2013-8-19,  i named the directory with the date I download the fw.
The board is 'ttgo-t-beam-v1'  and fw is 2023-08-19

could it be you flashed another filesystem? or changed something when uploading the new firmware?
which version was the "before" version which has no flaws for you?

version 2023-7-13  was the version I then updated to 2023-8-19 and the formed packet did not show the gps position.  I then rolled back to version 2023-07-13 and the gps position in the packet was correct.
I then updated again to 2023-08-19 and got again the null? in the packet sent
you can see the trace in aprs.fi:  https://aprs.fi/?c=raw&call=IW2EJH-5
Only thing i can have touched is the config file, i cup/paste the first part, I will try to reload with the original  and run a new test.

Please update also the filesystem with the new data (not pasting the old) as new features has been added and this may be causing the problem

Maybe wait a little longer and tell me about it later.
Usually the time is the first to be received , then the date and finally the gps fix (and position)

did it got resolved?

The "null?" packet sent out to aprs in the packet string is solved.  The date  displayed on the Oled is still at 1999-12-31. This is not important at the moment, I will program a new board for a friend tomorow and check if it work. I will update when test done.


Preview layer does not scale well with Iphone x screen

Hi,
Setting the video gravity to resize does not scale well with iphone x screen.
videoGravity =.resize
Any idea how solve this issue ?

We have fixed the issue by below code, might be possible to adjust the code to below.
previewLayer = PreviewView(frame: view.frame, videoGravity: .resizeAspectFill)
previewLayer.center = view.center

Great fix.
@Awalz pls add this fix to the code base

Hi,
Setting the video gravity to resize does not scale well with iphone x screen.
videoGravity =.resize
Any idea how solve this issue ?

Hello when I upload video from iPhone X backend people are receiving but unable to play the video what is the issue in iPhone X ?? unable to play the video which I have recorded from iPhoneX Only

I still have the same problem in 2020, have someone managed to solve it? I tried all the upper "solutions" but they don't work correctly

We have fixed the issue by adding centering previewLayer to upper view as in the below code.
previewLayer = PreviewView(frame: view.frame, videoGravity: .resizeAspectFill) // or gravityVariable
previewLayer.center = view.center

It works for me
Thanks

We have fixed the issue by adding centering previewLayer to upper view as in the below code.
previewLayer = PreviewView(frame: view.frame, videoGravity: .resizeAspectFill) // or gravityVariable
previewLayer.center = view.center

It works for me
Thanks


Keyboard clicks can be too soft to hear without turning volume up too high

It seems like the virtual keyboard is using the system ringer volume. Anecdotally, I tend to want to make the keyboard clicks as loud as possible (especially when playing audio from the iMac speakers), but on an iPad Pro that means calls ring way too loud.
Perhaps use the user's volume setting instead of the ringer's volume?

We use AudioServicesPlaySystemSound(), for which the documentation includes:

Sounds play at the current system audio volume, with no programmatic volume control available

So this won't be possible, unless we change the way we play the audio altogether.


Vive Tracker not working with XR Plugin

Hi,
Exploring Unity XR SDK, I've came across the following problem: one of my Vive trackers (model_number: Vive Tracker MV, htcComposeTime: 2017-04-05 10:43:01.844000) does not appear in the InputDevices list even though it displays correctly in SteamVR Plugin. In addition, when exiting the application with the tracker connected, the following exception occurs:
[XRInputSubsystem] A device disconnection with the id 3 has been reported but no device with that id was connected.
UnityEngine.XR.Management.XRLoaderHelper:StopSubsystem()
Unity.XR.OpenVR.OpenVRLoader:Stop() (at Library/PackageCache/com.valve.openvr@dcd8923d94/Runtime/OpenVRLoader.cs:241)
UnityEngine.GUIUtility:ProcessEvent(Int32, IntPtr)

The newer version of the Vive tracker (model_number: VIVE Tracker Pro MV, htcComposeTime: 2019-02-14 23:57:48.952000) works fine and  and there are no such problems
Unity  2019.3.13f1
SteamVR 1.12.5

Can confirm. Using OG Vive trackers and Vive HMD:
If trackers are OFF before hitting "play" in Unity, and then turned ON while playing, then:
-Trackers are not detected; Unity throws error:
[XRInputSubsystem] A device disconnection with the id 3 has been reported but no device with that id was connected.
-With the tracker still ON, when stopping the player, Unity throws the error:
[XRInputSubsystem] A device disconnection with the id 3 has been reported but no device with that id was connected.
UnityEngine.XR.Management.XRLoaderHelper:StopSubsystem()
Unity.XR.OpenVR.OpenVRLoader:Stop() (at Library/PackageCache/com.valve.openvr@db9e35089d/Runtime/OpenVRLoader.cs:241)
UnityEngine.GUIUtility:ProcessEvent(Int32, IntPtr, Boolean&)
If trackers are ON before hitting "play" in Unityx, then:
-Trackers are not detected;
-With the tracker still ON, when stopping the player, Unity throws the error:
[XRInputSubsystem] A device disconnection with the id 3 has been reported but no device with that id was connected.
UnityEngine.XR.Management.XRLoaderHelper:StopSubsystem()
Unity.XR.OpenVR.OpenVRLoader:Stop() (at Library/PackageCache/com.valve.openvr@db9e35089d/Runtime/OpenVRLoader.cs:241)
UnityEngine.GUIUtility:ProcessEvent(Int32, IntPtr, Boolean&)
If the tracker is switched OFF during runtime, Unity throws error:
[XRInputSubsystem] A device disconnection with the id 3 has been reported but no device with that id was connected.


Proposal: Rename this repository to "h2"

The name has caused confusion due to the hyper project being deprecated. And to be fair I've definitely forgotten that this URL is special and doesn't match it's PyPI package name. Should we rename the project to simply h2?

I think this is sensible.

I agree - I considered it before v4 release, but didn't have a strong reason yet.

Since there's 3 in favor I'll leave this open a few days to see if any are opposed, if not I'll go ahead with the rename.

Thanks for picking up that conversation from the HTTPX repo. :)
I would personally also support a rename of the repo, especially given that the PyPI name and package name are already h2.

I've got time to work on OSS today and seems there's many +1s now, going forward with the rename.


JCenter is being sunsetted

Consider migrating to another package repository.
https://jfrog.com/blog/into-the-sunset-bintray-jcenter-gocenter-and-chartcenter/

Thanks - I am already looking into this. I will leave this issue open to put some updates into.

I've now got a repository created on oss.sonatype, so will be syncing to maven central once I publish the first enro component to that repository. I'm about to go away for a few days, so it is unlikely this will happen before next week.
See link here: https://issues.sonatype.org/browse/OSSRH-65111?page=com.atlassian.jira.plugin.system.issuetabpanels%3Aall-tabpanel

I have started getting publishing working with the sonatype repository. I will continue working on this over the weekend. My current plan is to release 1.3.0, and publish that version to maven central. The upgrade to 1.3.0 might be a little painful, as I am intending to migrate the package names to "dev.enro" instead of "nav.enro", as "enro.dev" is a domain that I control.
Other than the package name change, the other major change in 1.3.0 will be that "enro-result" will no longer exist, as I have decided that this functionality should be a core part of Enro, and 1.3.0 will pull everything from "enro-result" into "enro-core". ResultNavigationKey will also be renamed NavigationKey.Result.
If you really need to keep using 1.2.x, these artifacts will remain available through Github packages.
Please let me know in advance if you see any major issues with this.

I've published the first release of Enro 1.3.0 to the Sonatype OSS repository. I am currently awaiting the sync to Maven Central to be activated (you can check it out the progress on this here: https://issues.sonatype.org/browse/OSSRH-65111)

It appears the sync to Maven Central has been activated. Enro 1.3.0 and higher will now be available on Maven Central, so I'm closing this issue.
Please be aware that the correct dependency is now dev.enro:enro:1.3.0, not nav.enro.
Let me know if you have any trouble.
Thanks for raising the issue and showing interest in Enro!


[Add] Add logout functionality

Add logout functionality and reset all token and user data in the local database.

Related to #6


Pass target-cpu to C compilers

It's possible to set target compilation architecture via RUSTFLAGS="-C target-cpu=<arch>".
As far as I can tell, the C compiler is not made aware of this, so it won't be producing matching binaries. It affects performance and I suspect it may also affect correctness when it affects C calling convention (newer SSE registers are used).
So I think it'd be best if the cc crate detected Rust's compilation target and automatically configured the C compiler to match (abstracting away compiler-specific flags and possibly cpu arch naming mismatches, so that build scripts don't have to reinvent this).

I've cross-referenced https://gcc.gnu.org/onlinedocs/gcc/x86-Options.html and rustc --print target-cpus
Most of them match exactly. Below is the list of odd ones.
The extra options on GCC side can be ignored (they're mostly exotic/outdated and not fully supported anyway).
The extra options on LLVM side may need to be mapped to a closest equivalent.

march
Back-end
Description

skx
LLVM
skx processor

slm
LLVM
slm processor

atom
LLVM
atom processor

core-avx-i
LLVM
core-avx-i processor

core-avx2
LLVM
core-avx2 processor

corei7
LLVM
corei7 processor

corei7-avx
LLVM
corei7-avx processor

generic
LLVM
generic processor

penryn
LLVM
penryn processor

x86-64
LLVM
x86-64 processor

yonah
LLVM
yonah processor

eden-x2
GCC
VIA Eden X2 CPU with x86-64, MMX, SSE, SSE2 and SSE3 instruction set support. (No scheduling is implemented for this chip.)

eden-x4
GCC
VIA Eden X4 CPU with x86-64, MMX, SSE, SSE2, SSE3, SSSE3, SSE4.1, SSE4.2, AVX and AVX2 instruction set support. (No scheduling is implemented for this chip.)

esther
GCC
VIA Eden Esther CPU with MMX, SSE, SSE2 and SSE3 instruction set support. (No scheduling is implemented for this chip.)

knm
GCC
Intel Knights Mill CPU with 64-bit extensions, MOVBE, MMX, SSE, SSE2, SSE3, SSSE3, SSE4.1, SSE4.2, POPCNT, AVX, AVX2, AES, PCLMUL, FSGSBASE, RDRND, FMA, BMI, BMI2, F16C, RDSEED, ADCX, PREFETCHW, AVX512F, AVX512PF, AVX512ER, AVX512CD, AVX5124VNNIW, AVX5124FMAPS and AVX512VPOPCNTDQ instruction set support.

nehemiah
GCC
VIA Eden Nehemiah CPU with MMX and SSE instruction set support. (No scheduling is implemented for this chip.)

samuel-2
GCC
VIA Eden Samuel 2 CPU with MMX and 3DNow! instruction set support. (No scheduling is implemented for this chip.)

c7
GCC
VIA C7 (Esther) CPU with MMX, SSE, SSE2 and SSE3 instruction set support. (No scheduling is implemented for this chip.)

nano
GCC
Generic VIA Nano CPU with x86-64, MMX, SSE, SSE2, SSE3 and SSSE3 instruction set support. (No scheduling is implemented for this chip.)

nano-1000
GCC
VIA Nano 1xxx CPU with x86-64, MMX, SSE, SSE2, SSE3 and SSSE3 instruction set support. (No scheduling is implemented for this chip.)

nano-2000
GCC
VIA Nano 2xxx CPU with x86-64, MMX, SSE, SSE2, SSE3 and SSSE3 instruction set support. (No scheduling is implemented for this chip.)

nano-3000
GCC
VIA Nano 3xxx CPU with x86-64, MMX, SSE, SSE2, SSE3, SSSE3 and SSE4.1 instruction set support. (No scheduling is implemented for this chip.)

nano-x2
GCC
VIA Nano Dual Core CPU with x86-64, MMX, SSE, SSE2, SSE3, SSSE3 and SSE4.1 instruction set support. (No scheduling is implemented for this chip.)

nano-x4
GCC
VIA Nano Quad Core CPU with x86-64, MMX, SSE, SSE2, SSE3, SSSE3 and SSE4.1 instruction set support. (No scheduling is implemented for this chip.)

related: https://github.com/conan-io/conan/issues/847

I would suggest (since these have to be mapped and handled separately per ToolFamily), that you start with a minimal list and expand support as they are requested by other projects.
GCC 4.8 requires corei7-avx (rather than sandybridge) and core-avx2 (rather than haswell). I don't see gcc version detection, and I think those values are forward compatible.
Maybe we should start with a mapping table and add columns for MSVC, CLANG, and GCC. I would imagine that we'd have to target baseline versions and only enable advanced mappings if we know the compiler version.

For this crate, what are the minimum supported versions of clang, gcc, and msvc?

The same applies to target-features.

I need this for WASM (target-cpu=mvp) in order to address https://github.com/paritytech/polkadot-sdk/issues/3192 with LLVM 16+.


sheet custom detent: min(.ideal, ctx.maximumDetentValue * 0.5)?

how would I compose a detent for a sheet that's the minimum of half the height of the screen, or the ideal size?
I tried using a .custom detent, but I don't see how to get the ideal height, the ResolutionContext doesn't seem to have it, so i'm not sure what Float to return from the .custom closure

ideally, i would have the view being presented passed in as part of the ResolutionContext, so I can get it's ideal size

Yea I can add this!

Added in 1.5.1!


FEATURE REQUEST: [MongoDB] Enable memory volume

Is this a request for help?:
No
Is this a BUG REPORT or FEATURE REQUEST? (choose one):
FEATURE REQUEST
I would like to the ability to specify the medium for the mounted /data/db directory. We are currently using the medium Memory since we do not need persistent data with our mongodb deployment.

/remove-lifecycle rotten

/lifecycle frozen

Closing this old issue. This repo is deprecated; for any further questions on this topic, refer to https://github.com/bitnami/charts/tree/master/bitnami/mongodb.


[Bug]: tokio-curl tries to access dropped socket

What happens: It happens randomly when downloading many pages simultaneously. Sometimes tokio-curl drops a socket from the slab and then tries to reuse it.
RUST_BACKTRACE=1 RUST_LOG=trace output example: here
Way to reproduce: example
OS: Arch Linux x86_64 hybrid

I've made a fix but it needs some testing.

Got the same issues when I'm running tests that do network requests on travis.

Oops. I placed the check in the wrong place. I was quite sleepy back then. I'm sorry about that. Now it's corrected.

Thanks for the report! I'll publish a new version with the fix


Allow Output path for sfdx force:source:retrieve

Is your feature request related to a problem? Please describe.
It would be nice if you could specify a different root output path when running sfdx force:source:retrieve.
This would allow you to pull down source from multiple orgs and compare them
Describe the solution you'd like
Add a optional param (--retrievetargetdir  -r) which would allow you to specify the result path.
Describe alternatives you've considered
The only alternative is to clone the project before running the command..... Way less than ideal.
Additional context
Blocking https://github.com/ChuckJonas/vscode-salesforce-diff/issues/1

@clairebianchi, yes potentially.  This would really only be for secondary use-cases of this command.  EG: I want to fetch source from two different orgs and compare them using diff-tool.
However, I can see how this behavior could impact other features, so it's honestly not as critical as something like #97...

This exists since v7.180.  We didn't have your issue linked, though, and I just now noticed this was here.
https://github.com/forcedotcom/cli/blob/main/releasenotes/sfdx/README.md#71800-dec-8-2022


Fix warning in export log "Wide character in print"

https://curation.pombase.org/dumps/builds/pombase-build-2021-12-06/logs/log.2021-12-06-22-21-44.export_warnings

The fixed that warning for Thursday morning.


Add delay between remove and add resident log

What's the purpose of adding such a delay, could you explain furthermore?

This problem happens in the GIF above, apparently adding notifications quickly can happen eventually from UI not allocating correctly, I had quite a few problems with this in Unity, I remember solving it exactly the same way, putting a delay. This problem was no different, adding the delay solved it.


[admin] Merge to live!

Docs Build status updates of commit db7d8f0:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/exchange-web-services/how-to-find-autodiscover-endpoints-by-using-scp-lookup-in-exchange.md
:white_check_mark:Succeeded
View

docs/exchange-web-services/how-to-perform-paged-searches-by-using-ews-in-exchange.md
:white_check_mark:Succeeded
View

docs/legacy-protocols/how-to-authenticate-an-imap-pop-smtp-application-by-using-oauth.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Create thoth-tech-worklog-guideline.md

Description
Please include a summary of the change and which issue is fixed. Please also include relevant
motivation and context. List any dependencies that are required for this change.
Fixes # (issue)
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as
expected)
[✓ ] Documentation (update or new)

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can
reproduce. Please also list any relevant details for your test configuration
Testing Checklist

[ ] Tested in latest Chrome
[ ] Tested in latest Safari
[ ] Tested in latest Firefox

Checklist

[✓ ] My code follows the style guidelines of this project
[✓ ] I have performed a self-review of my own code
[ ] I have commented my code in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have requested a review from ... on the Pull Request

All checks passed. Good to go

Your last commit has lint error on how-to-publish-an-astro-website-to-netlify-guide.md file

Changes looks good. Please follow up with Amin to approve

@ivanhub7 as mentioned during the Standup. Create a netlify account and host the Documentation website on there to be able to generate the appropriate steps for the document.

 Create a Worklog to be adopted by the Company

 Create markdown files to showcase Splashkit's projects for the Documentation Website

 Migrate Splashkit's Tutorial folder and files to the Documentation Website.

 Review the folders in SplashKit determine which folder they fall under (Projects, Documentation, Issues and Resolution)

Always read the details of your failed checks

 Migrate documents from Company Operations Repo to the Documentation Website.


fix bugzilla issue 7141

http://d.puremagic.com/issues/show_bug.cgi?id=7141

Looks like I missed a lot, real life has it's downsides :(
After unfortunate illness intermixed with university affairs I've been off for a couple of months. As such I wasn't tuned on bugzila/groups etc. at the time. Fortunately I noticed the paths in the merge notification, issue 7141 isn't exactly descriptive.
The fix puzzles me a bit, it pinpoints the root of the problem but I'd rather do it in less intrusive way.Adding 2 extra states for parser is an overkill, as it was already a messy place.
The problem with this commit is that I think it sides-steps some range magic e.g. this test now fails:
unittest{
string pattern2 = [&-z];
bool m = !match("b",pattern2).empty;
assert(m);
}
It's late here,  I'm following on this one tomorrow.

I figured you'd be busy with studies, sorry to hear you've been sick too. Great to hear your'e better, and Happy New Year.
Feel free to implement a simpler fix in a future pull request. More importantly, it would be great to discuss starting on the EBNF parser we discussed a while ago. That's a very exciting and high-profile project!

Oh, and GSOC 2012 is upon us. We need to work on applications.


Feature Request : Bluetooth Proxy Support on BK7231x

As Beken ships have bluetooth like ESP32, Having a BT Proxy would be amazing https://esphome.io/components/bluetooth_proxy.html

This will help a lot, since tons of modules are using these chips right now !

Yes this would be amazing. For me it would mean that every lightbulb could be a potential Bluetooth proxy for less than $/€ 4.
Anyone that knows of implementation is ongoing and how far away this is?

@kuba2k2 Any idea if anyone is working on this?

Yes, nobody is working on this.

Yes, nobody is working on this.

I have no idea what to do (I am ready to learn) else I can help in anyway I can.

Is there a way to get the bluetooth library file and compile it?

Vâng, không ai đang làm việc này.

no news in this thread?
Today I flashed one of these modules and to my surprise I found out that it also contains bluetooth

Did you managed to install a BT proxy as well. If so, I will probably pee I'm my pants :-)

Did you managed to install a BT proxy as well. If so, I will most probably pee I'm my pants :-)

Nope. LibreTiny may not be a great platform. Bugs don't get addressed, no features get worked on. If OpenBeken finds a way to implement BTProxy. I am moving to that.

See my reply in https://github.com/libretiny-eu/libretiny/issues/278#issuecomment-2213807829

See my reply in https://github.com/libretiny-eu/libretiny/issues/278#issuecomment-2213807829

Does it mean that the code needs to be refactored before it's worth to add enough features to support BT proxy.
As you probably have understood, what we are aiming for is to turn every light bulb into a BT proxy

LibreTiny may not be a great platform. Bugs don't get addressed, no features get worked on. If OpenBeken finds a way to implement BTProxy. I am moving to that.

This is a pretty disrespectful attitude to spout in comments. Libretiny is not some corporate product that took your money and failed to deliver - it's a free and open source project that is run mostly by a single dev, and frankly it is amazing that such a thing exists!
 It's not a failing, it's just the reality of the situation at hand. I was (and remain) super excited to discover an esphome-based implementation for beken chips, and I had spent some time in the openbeken side of things - I am so glad that Libretiny exists and has achieved as much as it has, and especially that it's easily incorporated into the standard esphome setup (for one thing, I don't have to have multiple places to store my iot codebases, awesome!).
Frankly, it's possible that the beken chips will never be able to do bluetooth proxy - the device might not have the grunt and memory to manage it (proxy is much more demanding than just sending broadcasts or supporting a single connection). I'd love to see bt proxy support happen (I have a vested interest, my Bermuda project) but the reality is it might not even be possible, so it's a significant investment for someone to dive into this to spend the time to possibly just confirm it can't be done - it's deeply unreasonable to criticise anyone for not choosing to take that gamble with their time.
People like kuba2k2 who put in serious work and offer it up to all of us for free deserve to be treated with respect, if you don't like their work pass on by, if you think it could be better, consider sponsoring them or someone else to help out, but eroding a dev's passion for something you directly benefit from by complaining about how much of their life they are donating to you is not helpful, and I'd encourage anyone to think compassionately about how they can best engage with the open source community in ways that benefit everyone and acknowledges contributions, rather than putting them down.

LibreTiny may not be a great platform. Bugs don't get addressed, no features get worked on. If OpenBeken finds a way to implement BTProxy. I am moving to that.

This is a pretty disrespectful attitude to spout in comments. Libretiny is not some corporate product that took your money and failed to deliver - it's a free and open source project that is run mostly by a single dev, and frankly it is amazing that such a thing exists!
 It's not a failing, it's just the reality of the situation at hand. I was (and remain) super excited to discover an esphome-based implementation for beken chips, and I had spent some time in the openbeken side of things - I am so glad that Libretiny exists and has achieved as much as it has, and especially that it's easily incorporated into the standard esphome setup (for one thing, I don't have to have multiple places to store my iot codebases, awesome!).
Frankly, it's possible that the beken chips will never be able to do bluetooth proxy - the device might not have the grunt and memory to manage it (proxy is much more demanding than just sending broadcasts or supporting a single connection). I'd love to see bt proxy support happen (I have a vested interest, my Bermuda project) but the reality is it might not even be possible, so it's a significant investment for someone to dive into this to spend the time to possibly just confirm it can't be done - it's deeply unreasonable to criticise anyone for not choosing to take that gamble with their time.
People like kuba2k2 who put in serious work and offer it up to all of us for free deserve to be treated with respect, if you don't like their work pass on by, if you think it could be better, consider sponsoring them or someone else to help out, but eroding a dev's passion for something you directly benefit from by complaining about how much of their life they are donating to you is not helpful, and I'd encourage anyone to think compassionately about how they can best engage with the open source community in ways that benefit everyone and acknowledges contributions, rather than putting them down.

I'd agree I was maybe a tad bit over the line, but that's my passion and love for the project.
I do contribute in any way I can.
What I meant by bugs don't get fixed - here's an example https://github.com/libretiny-eu/libretiny/issues/280.
The frustration is not towards Kuba2k2 but more about even tho LT is added to ESPhome it's not getting the attention and love the main line gets. We did discuss a bit about it on the other bug linked above.

I agree with you that it would be nice to see a way for the esphome team to chip in here - it seems like a lot of the heavy lifting to get bk chips as a platform in esphome is done and benefits esphome already. I noticed your comment on the other bug and that you have reached out to esphome to see how they can contribute, which is awesome.
I know I came on a bit strong, in part I'm reacting to a pattern of OSS devs getting burnt-out and it's clear that you care about the project.

I'm not sure I'd say I'm burnt-out with this project. I definitely will work on LibreTiny much more in the future, just don't know when. Right now I'm working on another big project of mine, which I've had for nearly 6 years now, and which needs attention after more than a year of no maintenance whatsoever. When I finish that, I see no reason not to get back to LibreTiny (unless a different particular "commercial" project comes up, I can't tell yet).

Did you managed to install a BT proxy as well. If so, I will most probably pee I'm my pants :-)

So my pants will stay dry at least for some months/years... 😅

Bluetooth at its core is apparently working in LibreTiny on Beken (at a base proof-of-concept level).  The heavy lift here is in ESPHome, as the current implementation is very ESP32 dependent.  ESPHome's ble_client and bluetooth_proxy would need to be rewritten into agnostic standardized contracts/headers, and then it would be easier to implement platform-specific versions of it.  Prior to LibreTiny, ESP32 was the only platform which had bluetooth.  Whoever takes on such a project for any new platform would need to include that refactor, unless it is addressed by ESPHome in advance.  So far no one has volunteered, and there hasn't been enough user input to increase priority.

Bluetooth at its core is apparently working in LibreTiny on Beken (at a base proof-of-concept level).  The heavy lift here is in ESPHome, as the current implementation is very ESP32 dependent.  ESPHome's ble_client and bluetooth_proxy would need to be rewritten into agnostic standardized contracts/headers, and then it would be easier to implement platform-specific versions of it.  Prior to LibreTiny, ESP32 was the only platform which had bluetooth.  Whoever takes on such a project for any new platform would need to include that refactor, unless it is addressed by ESPHome in advance.  So far no one has volunteered, and there hasn't been enough user input to increase priority.

In that case if you know what has to be said and where, we can start putting pressure so things can move in the right direction...


Upgrade PyYAML to 5.x

https://trello.com/c/51brdJKx/1201-upgrade-pyyaml-to-5x
I've been cautious and made this a major bump, in case anyone was thinking of using this with an older version of Python.
We are careful to use yaml.safeload() instead of the vulnerable yaml.load(), but this version of PyYAML deprecates the latter completely so it's no longer even an option 😸

@risicle Yep - will pull it into a content-heavy app like Brief Responses FE and run the functional tests locally.


See-through portals feature

https://stabyourself.fandom.com/wiki/See-through_portals
Maurice wrote SE has See-through portals feature but it is broken. If you can fix, that will be come a new feature of CE (or a cheat)

I can see a number of issues with see-through portals that'd have to be addressed before even starting:

The way the camera currently follows the player means that it'd jitter around when the player goes through portals. In the examples on that page, the camera is always locked to the player, which is a different behavior from Super Mario Bros;
The easiest way to implement it would make it so that you can't see portal views through other portal views (such as portal loops and such). Just like in those examples, you'd only have one view window for orange, and one view window for blue;
It'd be exceptionally problematic in multiplayer. Unless we somehow limited the portal views to only show the one from the pertaining player, it'd be very difficult to have multiple pairs of portals showing their views without causing major visual issues;
One scenario that all those examples conveniently omit is when your portal view overlaps one of the portals (like, for instance, standing under a portal on the ceiling, and the second portal being on the floor above the first). The portal window would cover the portal, making it really bad for gameplay. Also, unless we disabled portal windows for portals facing away from the player, it could create a very weird portal window for the other portal;
Overlapping portal windows! How would we handle it when the two portals are so close together that both their views should occupy the same space?

I'm leaving this issue open if anyone wants to tackle all these issues, but the way I see it, it's just not feasible for Mari0 to have see-through portals, from a technical standpoint. Executing see-through portals shouldn't be too difficult, but it'd require addressing all of the problems described above.


Reverse a Singly Linked list.

Description
I can come up with a simple and easy to understand code to Reverse a Singly Linked List.
Please Assign (GSSOC'22)
Domain
Data Structure
Type of Contribution
Addition
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of LearnCPP.

Hello, Can you please assign this issue to me Under GSSoC so that I can start working on it

Um.. I actually created the issue so that I can do it 😬😬

Okay Sure .

Hy @svenkat19  I Want To Work On It Can You Assign Me Under GSSoC So That I Start Working On It

/assign

Hey, I have already submitted a pull request for this a week ago. Pls unassign yourself

/assign

@svenkat19 why this issue not showing any assign or something and still showing in issue section

/assign

Well, I created the issue(as a contributor). And here my the thread for my PR: https://github.com/Lakhankumawat/LearnCPP/pull/272#issuecomment-1063642871

/unassign

/assign


default_settings.yaml

Where is the file default_settings.yaml?

It will be created automatically when first run and is then in the user folder.


Add events and logs in key places during service processing

Add events and logs in key places during service processing, such as allocate and release EIP, GoBGP synchronization Nexthop, Layer2 mode selection Node

/assign

@zheng1 assign this issues to me please

@zheng1 assign this issues to me please

@felix0080 done

thank you

Hi @felix0080,
Any updates on this issue?


Question about  the paper

Hi, thanks for your great work!
I have a question about the paper. In 2.2 of the paper, you say

Note that concurrent methods [20, 16, 12] also use diffusion models for this task, but they only report the upper bound of performance, which is not available in real-world applications.

For example, I can't find any  ' upper bound of ' about the evaluation stage in [16] .
Hope for your reply, Thanks!

In fact, [16] doesn't mention whether they used P-Best or P-Agg.

But in there code, I see they use P-AGG for evaluation.

Thanks for pointing this out! Their code wasn't available when I submitted the paper, so I'm not sure which metrics they used. The description in our paper is inaccurate, and I've revised our paper (which is not yet submitted to arxiv). The difference between our method and theirs is that they use P-Best or P-Agg, whereas we use J-Best or J-Agg.

Thx, learned a lot from your paper!


Add support for querying by long timestamp for sql

When querying via SQL, our timestamps are stored as longs, but when we add WHERE timestampfield = LONG the query fails, but as soon as we modify it to be WHERE timestampfield = 'YYY-MM-DD HH:mm:ss.zzz it works
It'd be nice if it supported longs

Hi @bputt, I think the current way is the SQL standard. You can cast timestampfield as long if you want to filter by a long value.


Where is the list of device types?

Where is the list of device types?  Node defaults to type 272a?  But no clue what device that is?  Phone App does not list a node type?  So how do we find out what type of device 'type' value we have?

For those that might like the information... type 648a is RM 4 Mini.

Added in v2.1.5


Release Test to Prod

Links to Jira Tickets
n/a
Description of Changes
Release latest changes from Test to Prod.
Testing Notes
Standard QA.

Reminder: Make sure to "Create a merge commit" when merging this in:

Not the end of the world if its missed, but will make the next release more complicated as we'll get merge conflicts.


Editing webhook: cannot update Short Name

Repro:

https://registry.hub.docker.com/u/XXX/YYY/settings/webhooks/
Click Edit
Change short name
Click Save

Name doesn't change.

Hi,
Hub issues should be reported to https://support.docker.com/
This repository here is about the (deprecated) open-source registry, and is unrelated to your issue.


Bad Error message when trying to use props values as Css values

Describe the bug
i know i can't use dynamic values as CSS values when using vanilla-extract , but the error message displayed is not helpful.
Description:

When deploying a project to vercel i get the error message: "Error: The Edge Function "_qwik-city" is referencing unsupported modules: @qwik-city-plan.js: @vanilla-extract/css, @vanilla-extract/css/functionSerializer"

Cause:
A div component with class ={ style({ backgroundColor: props.value.toString() }) }
Suggested Error Message:
vanilla-extract-css -- dynamic values are not allowed in this context -- $file_name:$file_line
Reproduction
npm create qwik; npm run qwik add vanilla-extract-css; npm run qwik add vercel-adapter ; npm run build; npm run deploy
System Info
node --version: 20.3.0
openSuse tumbleweed linux x64
linux-header: 6.8.1-1-default

- package version: 0.5.4 (vanilla-extract)
- framework: qwik: 1.5.4

Used Package Manager
npm
Logs
"Error: The Edge Function "_qwik-city" is referencing unsupported modules:
    - @qwik-city-plan.js: @vanilla-extract/css, @vanilla-extract/css/functionSerializer"

Validations

[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] The provided reproduction is a minimal reproducible example of the bug.

I don't think the referencing unsupported modules error comes from Vanilla Extract. That seems like a vercel thing. If you call style within a component at runtime, you'd probably see this error message.

OK good point, but the error message is still wrong, the configuration is right, the preview works so there's no  at runtime (styles are already loaded to the component).
the problem relies with assingning variables to style(), witch the dev should not be doing.
I just think the error message is not helping the dev debugging the problem

I can't reproduce your error with the reproduction you've provided. npm run qwik add vanilla-extract-css and npm run qwik add vercel-adapter don't seem to be valid commands. I think the integrations have different names.
Even when I did add the VE and vercel integrations correctly, I managed to deploy the app just fine. If you could create a repo with a minimal example of the error, or at least provide steps on reproducing the error in a simple qwik app, that would be appreciated.

Of course, I'll work on that, and get back to you in few days, sorry for the delay if that's a problem

npm create qwik@latest basic ./project;  cd ./project ; npm install; npm run qwik add vercel-edge; (Press Enter); npm run qwik add styled-vanilla-extract; (Press Enter); npm install --legacy-peer-deps; mkdir src/components/buggy-component; touch src/components/buggy-component.tsx;  echo 'import { component$ } from<EMAIL_ADDRESS>import { style } from "styled-vanilla-extract/qwik";
export default component$(() => {
const value = "anything"
return (
<div class={style({ backgroundColor: value})}/>
)
})' >> ./src/components/buggy-component/buggy-component.tsx;
npm run build; npm run build.server ; npm run deploy (Remember to configure your vercel project to do npm install with the --legacy-peer-deps option)
I think now its reproducible

You can't use style in a runtime file, i.e. outside of a .css.ts file. Even though you're using styled-vanilla-extract/qwik, this is still the case.
I'm guessing that vercel is detecting some VE dependencies a runtime dependencies due to your use of style within buggy-component.tsx, resulting in that error.
Again, I don't think this is an issue that VE can help with, since your usage of the library is incorrect, and we can't easily influence how vercel chooses to surface this error.

fine man, thanks for you time


Add files via upload

Hi! Thanks for contributing! Please make sure that you have:

Added the correct info
Stared the repo :wink:
Looked over the Code of Conduct and this content is inline with it

Please delete the above lines and replace them with your own content

@subhamtandon Congrats on your first PR!


Broken Link on Persistent Storage Docs

https://rancher.com/docs/rancher/v2.x/en/cluster-admin/volumes-and-storage/examples/ebs/
Has a link to setting up the concept in rancher but that link points to
https://rancher.com/docs/rancher/v2.x/en/cluster-admin/volumes-and-storage/examples/attaching-existing-storage
Which 404's
I believe it should point to here:
https://rancher.com/docs/rancher/v2.x/en/cluster-admin/volumes-and-storage/attaching-existing-storage/

closing this issue because it is fixed in this commit https://github.com/rancher/docs/commit/a723b9bc997c4d0c1720f32364726011b70f8648


It's a perfect job, but I wonder if the system can be used for protein-ligand complex systems, which are not mentioned in the paper.

It's a perfect job, but I wonder if the system can be used for protein-ligand complex systems, which are not mentioned in the paper.

The current version of AI2BMD supports the proteins with a single chain. We'll update it to support more scenarios in the future study.


drivers: intc: add irq-manager driver and API

This driver rides on top of a typical interrupt controller and provide APIs to dynamically allocate interrupts line. The range of interrupts managed by this driver is configured in the devicetree.

what's the use case of this? I am not keen to add another zephyr-ism in DT without a good reason

This pseudo driver is helpful in a scenario when multiple services need access to IRQs only when they are running, it allows a pool of IRQs to be shared by these services.
For example, the perf tool can be extended to support SMP by installing IPI ISRs on RISC-V to trigger each of the cores when it is running, these interrupt lines would usually be dormant when perf isn't running. With irq-manager, these interrupt lines can be used for other purposes.

There is not enough description in this PR of what this actually is , I'm not clear even what aspect of IRQs this is supposedly managing, I'm not even sure what it means by allocating and freeing interrupt lines. And looking at what irq_base parameter is being used like, it looks like maybe this is the part that relates to hardware, but it seems architecture specific? I would ask you to improve the documentation but I am still skeptical of this whole PR, so instead can you just improve the PR description first ? And since it seems like each "irq manager" instance correlates directly to a single interrupt controller, why does there need to be a separate DT node to represent this driver ? that seems just not acceptable

I'm guessing the arch you are working with has dynamically routeable interrupts? as in an interrupt from one core can be connected to another interrupt generating device at runtime?

to be clear, whatever this is, I am not outright opposing it, it's just not clear to me what exactly this is, and it seems specific to some architecture but posing as a generic thing, this just needs more explanation.

Pull this back to draft for now, will update the doc & comments later


generated frame question

hello，thanks for your work，but I have a question here, how to make the generated frames start with the reference image, rather than the reference image that is the result of the middle of the generated frame，is there a parameter to control this?

Not yet. We may add this control in the future.

ok，looking forward. Thank you for your reply
| |
Wenwu Zhang
|
|
@.***
|
---- Replied Message ----
| From | @.> |
| Date | 04/01/2024 22:03 |
| To | Doubiiu/DynamiCrafter @.> |
| Cc | PMPBinZhang @.>,
Author @.> |
| Subject | Re: [Doubiiu/DynamiCrafter] generated frame question (Issue #54) |
Not yet. We may add this control in the future.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


add auth setup

📝 Changed routes:

/api/auth/[...thirdweb]

Commit 85e9934a9daada002ae1d8a3f0e883105eb1e3f0 (https://thirdweb-neyzuxu1p.thirdweb-preview.com).


Teacher's Day Filters #712

Related Issues or bug
Fixes: #712
Proposed Changes

Added a new folder "Festive-Filter/Teacher's Day Filter" inside the Snapchat filter folder containing all the required files

This feature is not required!


Not compatible with Version >3 of ChartJS

Not compatible with Version >3 of ChartJS
RR! Found<EMAIL_ADDRESS>npm ERR! node_modules/chart.js

npm ERR! Could not resolve dependency:

npm ERR! **peer chart.js@">= 2.7.0 < 3"** from<EMAIL_ADDRESS>npm ERR! node_modules/chartjs-plugin-piechart-outlabels

npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force, or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.

import PieChartOutLabels from 'chartjs-plugin-piechart-outlabels'; generates error:
TS7016: Could not find a declaration file for module 'chartjs-plugin-piechart-outlabels'. 'C:/project/node_modules/chartjs-plugin-piechart-outlabels/dist/chartjs-plugin-piechart-outlabels.js' implicitly has an 'any' type.
  Try `npm i --save-dev @types/chartjs-plugin-piechart-outlabels` if it exists or add a new declaration (.d.ts) file containing `declare module 'chartjs-plugin-piechart-outlabels';`

After created a typings folder, added chartjs-plugin-piechart-outlabels.d.ts with content:
declare module 'chartjs-plugin-piechart-outlabels';
Made sure tsconfig.app.json has "include": ["src/**/*.d.ts"].
Error: Uncaught (in promise): TypeError: Cannot read properties of undefined (reading 'merge')
TypeError: Cannot read properties of undefined (reading 'merge')
at chartjs-plugin-piechart-outlabels.js:22:25
at chartjs-plugin-piechart-outlabels.js:11:65
at 2818 (chartjs-plugin-piechart-outlabels.js:10:2)


Update README.md with organization name

using the old 'ansible/ansible-lint-action' syntax in GitHub Actions (SaaS) results in an error: 'Unable to resolve actions. Repository not found : ansible/ansible-lint-action.'. This occurs despite GH redirecting properly. The proposed change executes properly.

renamed, not removal and so far github was good at redirects for both branchs and org changes, that error is quite weird. Still, it would not hurt to correct that url.


true not adding any metrics

Hello,
Just wanted to reach out with a small issue I am observing.
We are using this appender, and configuring this property to true.
   <appender name="LOKI" class="com.github.loki4j.logback.Loki4jAppender">
        <metricsEnabled>true</metricsEnabled>
        <format>
            <label>
                <pattern>name=${name},host=${HOSTNAME},level=%level,pid=${PID},thread=%thread</pattern>
                <readMarkers>true</readMarkers>
            </label>
            <message>
                <pattern>

With this metrics enabled, we were hoping to see "some metrics somewhere".
Our application exposes micrometer metrics, as well as JVM metrics.
By turning this property to true, there is no any metrics.
Therefore, I would like to report an issue where despite having this property to true, it sems there is no metrics being exposed.
Thank you

Hi @patpatpat123, I've checked metrics and they work as expected.
Metrics are written to Metrics.globalRegistry. Make sure it's added to your registry.

Hello, may I ask what metrics are you seeing?
All I am seeing are these (please see screenshot)

Something like logback_events_total
But I think they are part of the spring logback by default, nothing loki related.
Could you please let me know what are the metrics you are seeing?
Thank you


API: Makes prediction functions configurable

Another doozy. This PR:

Adds an API for configuring prediction functions, and implements dot-product and cosine-distance prediction.
Removes old alignment API in favor of dot-product prediction functions.
Adds top level API methods to do dot product and cosine distance, regardless of prediction configuration.
Changes loss_graph API to accept objects, not classes.
Removes WMRBAlignmentLossGraph.
Various tests.

going to have to go through this one in stages

Building on this doozy: I've added EuclidianDistancePredictionGraph and a fun script to plot some movies from movielens

This has become a behemoth. Somebody stop me. Please.


error occurred when git commit

Used the example code to watch a git repo,
It works just fine until i do git commit in the watched repo.
> cargo run --verbose
       Fresh libc v0.2.2
       Fresh bitflags v0.3.3
       Fresh fsevent-sys v0.1.3
       Fresh fsevent v0.2.11
       Fresh dummy v0.1.0 (file:///Users/yangcheng/repos/dummy)
     Running `target/debug/dummy`
thread '<unnamed>' panicked at 'Unable to decode StreamFlags: 1115136', ../src/libcore/option.rs:333
thread '<main>' panicked at 'called `Result::unwrap()` on an `Err` value: RecvError', ../src/libcore/result.rs:736
Process didn't exit successfully: `target/debug/dummy` (exit code: 101)

Thanks for the bug report. Just released an update with the fix. Let me know if everything is ok for you now.

Works fine for me now, thanks~


class shouldn't "inherit" default constructor from its container's superclass's inner class

Similar to https://github.com/ceylon/ceylon/issues/5961, a class may currently "inherit" a default constructor from another class that happens to have the same name:
JavaA.java:
public interface JavaA {
    class Class {
        public Class(String s) {}
    }
}

JavaB.java:
public interface JavaB extends JavaA {
    class Class {}
}

run.ceylon
JavaA.Class x = JavaB.Class(""); // disallow

Note 1: This sort of problem can't exist in Ceylon since member classes are polymorphic and must have matching signatures.
Note 2: If JavaB.Class weren't defined, JavaB.Class("") would be valid, based on the idea that static members are inherited. But a) supporting inheritance for static inner classes and also disallowing the original example (in full generality) seems incompatible with the current getMember algorithm, and b) I'd lean towards not having static members being inherited.
Note 3: JavaB.Class isn't even a subtype of JavaA.Class.

I disagree that this is wrong. Nested classes of interfaces are implicitly static, so the rules are the same as for other static members. Ceylon, just like Java, allows static members to be "inherited" by subclasses. I actually did some research on this yesterday, and I noticed that disallowing this would break a surprising amount of code. Finally, JavaB.Class() vs JavaB.Class("") is simply regular overloading, which is allowed for Java classes.

Did you notice that the two Classs are completely unrelated? And even if they were related, it doesn't make sense for constructors to be inherited in Ceylon or Java. I very much doubt any valid, existing code looks like run.ceylon.

Finally, JavaB.Class() vs JavaB.Class("") is simply regular overloading, which is allowed for Java classes.

It's not.
Object o = new JavaB.Class(""); // error
// The constructor JavaB.Class(String) is undefined

I suspect this would be difficult to fix. I'm not pushing for it to be prioritized for 1.2.3.

@jvasileff In fact this would be rather trivial to fix. But it would mean a completely different behavior for classes compared to methods, and that doesn't seem right to me.
Now, just because Java is completely inconsistent here, and treats classes differently to methods, doesn't justify us doing the same.

I find it highly unusual that this can be an error due to the LHS type:
JavaB.Class x = JavaB.Class(...);

In Ceylon, I expect the capitalized identifier Class on the RHS to indicate a type. The actual JavaB.Class should not be irrelevant to the expression JavaB.Class("").
The same in Java. I expect the type of the expression new X(...) to be X, not something that may be disjoint from the type X.

because Java is completely inconsistent here

If you consider that a constructor's container is the class it constructs, there is no inconsistency. JavaB.Class doesn't inherit anything else from the unrelated JavaA.Class, so why should it inherit its default constructor?
JavaB.Class masks, not extends, JavaA.Class.

completely different behavior for classes compared to methods

Sure, if you don't read the code as JavaB.Class.implicitDefaultConstructor("").
But when choosing which inconsistencies or oddities to accept, I feel that having an error for value bc = JavaB.Class("") is the best compromise, because surely that is incorrect/misleading code.

I agree that it's a little strange that JavaB.Class refers to different types on each side of the =. I still don't see why I should feel motivated to do anything about this. I think its reasonable and consistent.

So we agree then. Well, except for highly unusual vs. a little strange :smile:
If you don't mind, let's leave this open so I have time to come up with a killer argument or example.

Fine.


About Databox IP address confirmation

Hello,
When building the databox with ./databox-start,
IP confirmation message occured and select the favourite IP.

The IP confirmation is just for private network IP?
Could be build by using public IP?

Hello,
When building the databox with ./databox-start,
IP confirmation message occured and select the favourite IP.

The IP confirmation is just for private network IP?
Could be build by using public IP?

Thay are the IPs viable to the scripts. If an external ip is not viable then you must be tuning in a vm or behind a router performing NAT. As suggested befor running in a vm with bridged networking is the best option.

Now I have built Databox in Azure Centos7 Virtual Machine.
In Azure VM, has public IP and  private IP.
But when I built it, there is no IP confirmation message and if built successfully, I could not access http://:8989

Now I did it ,
I added the port 8989 in Azure VM :) I could access the Databox.
When I accessed the Databox from android app,the loading time is little slow.