[javascript] Invariant Violation: TurboModuleRegistry.getEnforcing(...): 'SourceCode' could not be found. Verify that a module by this name is registered in the native binary.

Description
I do not know.
Just run the project and get that error. No changes in git HEAD and the project built successfully yesterday.
Version
0.66.0
Output of npx react-native info
System:
    OS: macOS 12.3
    CPU: (8) arm64 Apple M1
    Memory: 121.17 MB / 16.00 GB
    Shell: 5.8 - /bin/zsh
  Binaries:
    Node: 16.13.2 - ~/.nvm/versions/node/v16.13.2/bin/node
    Yarn: 1.22.18 - ~/.nvm/versions/node/v16.13.2/bin/yarn
    npm: 8.5.5 - ~/.nvm/versions/node/v16.13.2/bin/npm
    Watchman: 2<IP_ADDRESS> - /opt/homebrew/bin/watchman
  Managers:
    CocoaPods: 1.11.3 - /opt/homebrew/bin/pod
  SDKs:
    iOS SDK:
      Platforms: DriverKit 21.4, iOS 15.4, macOS 12.3, tvOS 15.4, watchOS 8.5
    Android SDK:
      API Levels: 27, 28, 29, 30, 31
      Build Tools: 28.0.3, 29.0.0, 29.0.2, 29.0.3, 30.0.2, 30.0.3, 31.0.0
      System Images: android-30 | Google APIs ARM 64 v8a, android-31 | Google APIs ARM 64 v8a, android-31 | Google Play ARM 64 v8a
      Android NDK: Not Found
  IDEs:
    Android Studio: 2021.1 AI-211.7628.21.2111.8193401
    Xcode: 13.3/13E113 - /usr/bin/xcodebuild
  Languages:
    Java: 11.0.11 - /usr/bin/javac
  npmPackages:
    @react-native-community/cli: ^6.1.0 => 6.3.1
    react: 17.0.2 => 17.0.2
    react-native: 0.66.0 => 0.66.0
    react-native-macos: Not Found
  npmGlobalPackages:
    *react-native*: Not Found

Steps to reproduce

open Xcode
press Run.

Snack, code example, screenshot, or link to a repository
No response

I remove and re-install node_modules, clean cache and this issue is still persisted.

I restart my computer and it fixed. 🙄


fix(serverless-helpers): use region to validate function names

Currently the test is invalid since the blocking item is never the function name but rather the role name

I created the following plugin https://github.com/adriencaccia/serverless-custom-iam-roles-per-function to solve the problem of lambda role length.
I think we should keep the test on functions names length, and add the plugin to swarmion 😉

Ok sounds fair enough

See https://github.com/swarmion/swarmion/pull/244


added dag_name option to is_stopped

I think something like this would be useful. What do you think>?

response to #13

this is incorrect for two reasons:

is_stopped is a property
is_stopped does not indicate if a DAG is finished

I'll await your response on the issue. Closing this for now. thanks!


Grunt support for docker

I played around with node/grunt on docker to see if we can get this supported in cookiecutter-django out of the box.
The cleanest solution would be to install all node dependencies in a container (like with python in /site-packages/). As far as I understand it, this is not possible because grunt looks for node_modules in the current working directory (wtf). To make this worse, npm install needs to chown the directory which leads to permission problems if the UID of the mounted container volume is not matching with the host volume.
From getting started with grunt:

Each time grunt is run, it looks for a locally installed Grunt using node's require() system. Because of this, you can run grunt from any subfolder in your project.

If a locally installed Grunt is found, the CLI loads the local installation of the Grunt library, applies the configuration from your Gruntfile, and executes any tasks you've requested for it to run. To really understand what is happening, read the code.```

Would it be possible to to provide an alternative with libsass-python or python-scss for SASS compiling?

When I take a look at https://github.com/pydanny/cookiecutter-django/issues/300 and https://github.com/pydanny/cookiecutter-django/issues/286 we should be able to drop the whole node/grunt thing.
That would make the whole docker thing a lot easier. Any thoughts? Ping @noisy

It's standard for a project's node dependencies to go into node_modules in the current working directory (repo root of generated project), which I believe is just how node works, not just grunt. See http://stackoverflow.com/questions/16738177/grunt-possible-to-relocate-node-modules-for-plugins
What about doing an npm install to create node_modules/ in the current directory as per http://stackoverflow.com/questions/32096424/dockerizing-npm-bower-install-using-the-digitallyseamless-nodejs-bower-grunt-d -- would that work?

The main problem here are filesystem permissions and the way how volumes are mounted.
It would work on the production configuration, because the /app directory is inside the container. In development, we are mounting the /app directory from the host filesystem so that we don't have to rebuild on every code change. The whole stack basically runs inside of docker, but it uses the code from our filesystem.
There are ways around that on linux machines that have a native docker installation by mapping the user id of the user inside the container to the user on the host filesystem. On OS X and Windows docker actually runs inside a virtual machine which uses vboxfs to mount the host filesystem to the virtual machine for docker to mount to the running container. This is where it gets really painful. Actually, it's already painful on linux but with a third layer involved on Win and OS X you have to use nfs or cifs mounts with special permissions to make it accessible.


[Bug]: Downloader not found after fresh installation

Contact Details
No response
URL with problems
Not URL specific
What happened?
Repeated "downloader not found" error message after fresh installation.
Version
<IP_ADDRESS> (Latest)
Relevant log output
No response
Code of Conduct

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

This will be fixed in <IP_ADDRESS> with a dummy package of the downloader to force an update.


Great idea :smile:

I changed the referentials API :
https://github.com/ucoin-io/cutecoin/commit/da58e28e46fc2556aa363633527f16a3d5075e7a
Should be more easy to implements such features when we need ;) .

Please check out my proposal and see if it corresponds to what we need.
We should add a parameter in the display section, to enable or not the scientific notation.

Hi, I think this is a good idea but it should be implemented in a less radical way...
For example in my transaction I see:
1.000 UD and 9.091 dUD
I would prefer to see 1.000 UD and 0.909 UD which I find less complex.
I think there needs to be a balance between the scientific notation and the international one.
For example for Litres we often use hL, L, cL and mL but very rarely use daL and dL ... There is no need to use all notations thanks to digit rounding.
Thanks

So which units should we display ?

Hmm I'm not sure, any idea?
At least not dUD

kUD, UD, mUD, μUD, nUD?

I suggest:
0,001 UD => 1 mUD
1 UD => 1 UD
the switch to dUDs is very confusing.
even better would be to display UD in UD per day (%) for example 100 UDD (UD per day).
in this case we would not need much units below 0.

dUD, cUD, mUD are not standard ways to scale numbers. A correct scale number (international normalisation) is 10⁻¹,10-²,10⁻³ etc.
So a correct way to propose a scale should be to put the power number in preferences and to put informations at the top of transactions, in the same line than the dates, and avoid to put "UD" unit in the tabs (as it will be noted on that top line).
Line : Date begin, Date end, Money Name, Referential (Q,UD,QZ,RZ...), Unit Power (10⁻¹,10-²,10⁻³...)

I'm not sure that 10⁻¹,10-²,10⁻³ would be really user friendly.  It's probably standard, but not common in daily life. I'd prefer to go for the proposal to remove centi and deci units.

In the current use of money only ¨cents¨ are common. Scientific notation is more for scientific use not for people who use money.
Therefore I change my suggestions to use:
0,01 UD => 1 cUD
If by default we display the UD in 100 UDUnits per day, then we would not have to use much numbers below 0.

I agree that we should stick to the language people already use for currency.
1 cUD might be relevant but then 0,01 UD is not hard to read/understand and people are used to 10^-2 because of price tags. The problem is that currency language is mixing scientific and non-scientific notation... (eg we say k€ for thousands but not for kilo, and million instead of mega for millions...): https://en.wikipedia.org/wiki/Metric_prefix#List_of_SI_prefixes
I prefer your suggestion of UD, mUD. Perhaps we could have:
kUD => 1,000
UD
mUD => 0,001
But then what if we need to go higher or lower? Higher we could have million (MUD), billion (BnUD) which are understandable in terms of currency language. But what about lower?

maybe we dont need at all kUDs mUD and so on if the default unit is one Unit = 1% of UD per day.
in this case for normal use we will not get much below 0 (we dont have the bitcoin problem :))
above 0 is easy to read for  humans: 1.000 or even 1.000.000 is still ok to display, humans love big numbers :) in average one participant will have 1003012*10 cUDD  = 360.000 cUDD thats still ok to display :)

Don't mix display form and units. With mUD, cUD the display change, not the unit/referential.
If your display needs a transform formula on the value before display, it is a new unit, so a new referential.
@Arcurus : I suggest you to create a new referential for UDD.
Please give us the following informations :
Name: Name of the referential in the selector.
Units: text displayed after values.
Formula: Formula to transform the internal value in the new referential.
Description: An explanation of what is this referential and his purpose.
With enough informations, a new referential can be created and explain to users.
So I can select the UD referential (which I prefer), or select UDD when I please.

Name: Mango (Universal dividend per day in percent / Mango is just a replacement until we have a nice name :) )
Units: Mango
Formula: 1 Mango  = Units * 100 / (UD of one day ) )
Description:   Universal Dividend per Day displayed in percent. The purpose is to have a default unit that is easy to use and understand. 100 Mangos equal the universal dividend received per member per day.

Thanks @arcurus ! I will implement your new referential asap, if @Insoleet is ok.

I think it's okay. But this won't have any sense probably with Meta_brouzouf since we are already generating UD every day...

much thx @vtexier
lol yea for meta_bounzouf it would be nearly the same, just multiplied 100   :)
but for the beta currency it would be different :)
also because it is multiplied by 100  we wont need much thinking about how to display small units properly ;)
in true with meta_bounzouf with simulation one day one year we would have to multiply it with 36500 :)

@Arcurus :
I've done it in the dev branch !
Sorry for the name Mango, but you mentioned "UDD" and "Dividend per day", which I found more explicit, so I used it.
I have received 461,17 mUD from a member in UD Referential today.
It appears as 46,12 UDD in Dividend per day Referential.
Monetary mass and average per member are as Relative Referential but multiplied by 100.
Is it what you expected ?

yea UDD is fine :)
if its correct i dont know, i must see the formula:
Formula: 1 UDD = Units * 100 / (UD of one day ) )
really correct it would be if the UDD amount of the  transactions is counted in the UD per Day of the time of the transaction. the account balance on the other hand is always counted with the current UD per day.

The code to compute the referential is the following :
        dividend = await self.community.dividend()
        params = await self.community.parameters()
        if dividend > 0:
            return (self.amount * 100) / (float(dividend) / (params['dt'] / 86400))
        else:
            return self.amount

It looks ok to me.

It is ok, but as @Arcurus pointed out :

really correct it would be if the UDD amount of the transactions is counted in the UD per Day of the time of the transaction.

So we have a big problem here.
Sakia should have a "forget Mode" as in ucoin apps to display transaction values at the time of the transaction.
So it should be a configurable mode of display, not a separate referential, which it is now with "pastUD".
It should apply to all "Relative to UD" referentials.
Quite some work here...

¨It should apply to all "Relative to UD" referentials.¨
that would be great. that should be the default for relative units.
we ¨just¨ need a function who can get the amount of UD at time t. it should not be too difficult to extract that from the headers of the blockchain or?¨

@vtexier : Actually it's quite easy to implement and it's a good idea. We need a toggle in Display parameters of sakia to enable "forget Mode". Then, all relative referentials could check this parameter to choose what to display. It's easy since referentials have an access to the Application instance : https://github.com/ucoin-io/sakia/blob/dev/src/sakia/core/money/base_referential.py#L9

Done !


Use MenuTitle instead of Title

As the sitemap is used for navigation purposes it makes sense to me to
use the MenuTitle.

From an accessibility point of view having the link text and title attribute the same is recommended against though, hence using the two fields separately here. What's probably a better solution is parametrising the fields used for the titleAttr and linkText to allow them to be set from config...

Fair point and that makes sense too me.  If you wanted to expand exactly what you mean by parametrising the fields i'd gladly look at actioning it.

Looks good to me! Can this be merged?

Thank you for your contribution here. I just wanted to let you know that we’re looking to improve our management of issues and pull requests, with a goal to providing clear direction on how Symbiote contributes to the open source community. As part of this ongoing effort:

issues and pull requests that have been open prior to December 2017 with no recent activity are being closed

this is included

if you can confirm this remains applicable for SS3.6+ please do open it again

remaining issues and pull requests will be

categorised

on a clear path to resolution (whatever the next step may be).

This ultimately brings us one step closer to our ambition; being able to prioritise current issues and contributions in a timely manner. If you have any questions, please see https://www.symbiote.com.au/contributing for further information.


Forcing manpower on the review team

The issue of manpower on the review team has been discussed in the Discord forums with some community members.  The uphill battle is getting more work out of volunteers.   They did do over 2100 reviews from Jan 12th to  Feb 16th.  Can that be accomplished again?
Limiting the depth of the algorithm, limit the number of hotspots that can be added at once.  Yes this can be controlled.

I feel this HIP should go further and limit the carte blanche given to Nova Labs by previous HIP's to solve the problem of gaming by any means. They have shown themselves unable to do this with the algorithm in thoughtful a way that does not impact badly on the Innocent as well as the guilty.  I propose adding to this HIP the following:   In any given month no more than X hotspots can be added to the deny list by algorithmic means.  X = 200 % of the number of hotspots which can be reviewed in the man hours offered in advance by the volunteer reviewers. So, for example if on average 3 reviews can be done an hour and in a given month 100 man hours of reviewing is offered then the maximum number of hotspots algorythmicaly  added to the deny list is 600. this means barring disasters there is time allocated to review at least 50% of all deny listed hotspots even if every single one has a submission made which is unlikely. On current evidence less than 30% of hotspots in the last algorithmic sweep have had a review request

Opened in error


Build with opencv 3.1 in ios failed because of the enum name

There is a conflict in stitching/detail/blenders.hpp line 63:
enum { NO, FEATHER, MULTI_BAND };
The enum name "NO" is conflict with the build-in BOOL type "NO" in iOS.
So an error occurred :
/opencv/Pods/OpenCV/opencv2.framework/Headers/stitching/detail/blenders.hpp:63:12: Expected identifier
/Applications/Xcode.app/Contents/Developer/Platforms/iPhoneOS.platform/Developer/SDKs/iPhoneOS9.2.sdk/usr/include/objc/objc.h:74:13: Expanded from macro 'NO'

Related #6047
If you don't need this module then try to replace opencv.hpp include to list if required modules.

I had a similar issue with OS X, they said they would not fix that either.  It was a conflict with an Apple macro 'check' and it was conflicting with a bool called 'check'.  I imported the opencv files before the Foundation/Foundation.  Try importing the opencv files before UIKIT/UIKIT in your ViewController and see if that helps.  These guys love their 'wontfix' and said the same to me, so I figured it out myself.  Good luck.  Also if you are using Xcode, just download the iOS framework and add it to your project.  I have a couple of apps that use the opencv framework in iOS.

Some kind of workaround has been introduced in PR #6161 - it will give you compilation warning with the explanation.

It was still giving me the same error even when I imported opencv.hpp above UIKit. Importing it in .mm instead of .h works too. Or you could follow the one suggested by alalek.
Instead of importing opencv2/opencv.hpp, you can import the necessary modules separately.

need import this.like this png.

In 3.2.0, I also encounter this problem. I comment the corresponding modules in opencv.hpp and it works, at least for building. But I haven't test the exactness.

I have moved #import <opencv2/opencv.hpp> above all other imports as suggested and that did the trick.

@amayatsky I got one problem i  am converting javacv mat to opencv mat using
opencv_core.Mat grabbedImage = converter.convert(grabber.grab()) Mat x=new Mat(grabbedImage.address());
i need a clone of x but when i clone , my system hangs , any solution for it ?

if you're not using stitching.hpp just comment the line 86 of opencv.hpp
//#include "opencv2/stitching.hpp"

#ifdef __cplusplus
#undef NO
#undef YES
#import <opencv2/opencv.hpp>
#endif

On 3.4.4 issue still exists,
In the .mm file I moved #import <opencv2/opencv.hpp> above #import <Foundation/Foundation.h> and it worked

You can change opencv_modules.hpp like this.
#ifdef USE_OPENCV_STITCHING
#define HAVE_OPENCV_STITCHING
#endif
So, if you want to use stitching module, you can define USE_OPENCV_STITCHING, otherwise I can avoid such this compiling problem.
But, personally, I guess OpenCV teams have to change this issue.

None of the above solutions worked for me in the RN 62 with autolinking. I ended up writing postinstall script in my Podfile that disables stitching module.
def opencv_post_install (installer)   opencv_path = installer.config.project_pods_root + 'OpenCV/opencv2.framework/Headers/opencv.hpp'   if File.exist?(opencv_path)     puts "POSTINSTALL::patch OpenCV opencv.hpp header file"     text = File.read(opencv_path)     new_contents = text.gsub('#include "opencv2/stitching.hpp"', '//#include "opencv2/stitching.hpp"')     File.open(opencv_path, "w") {|file| file.puts new_contents }   else     puts "POSTINSTALL::OpenCV patch NOT possible. No file at:#{opencv_path}"   end end

@pmachowski you seem to have experience with OpenCV in RN, so maybe you know what's going on in my case (trying to port an "old" RN 0.59 project to RN 0.61.5).
I'm getting a build error for the line in m .pch file (line #29) where I import OpenCV the first time:

Any help would be highly appreciated!

#ifdef __cplusplus
#undef NO
#undef YES
#import <opencv2/opencv.hpp>
#endif

//other imports

thank you，my god


Does contacts.search not find already added contacts?

Hi! Is it ok behavior? I have empty list when user_obj.contacts.search('contact_from_my_skype')

As a reminder, SkPy is merely a wrapper around the APIs that Skype for Web consumes -- it applies no business logic of its own, and only a minimal amount of response parsing in order to produce well-structured results like SkypeUser.
Data problems like this are a consideration of the APIs, not of SkPy, and are out of scope here.  You should be able to produce the same result by using Skype for Web -- the issue template, which you keep clearing instead of filling in, requests that you do this before raising an issue.  Please take some time to familiarise yourself with Skype for Web and the features it makes available, and ensure your questions actually relate to this library rather than Skype or Python in general, which are out of scope here and better asked elsewere.


SSL verification flag

Adds support for disabling SSL Verification from within the task. This should be visible from within the task as an additional checkbox.
I am also bumping the minor version for this enhancement.
Refers to open issue #6

Coverage remained the same at 100.0% when pulling 894ea4e1f1c263c5d1412dc75817d63714b82191 on ssl-verification-flag into b912b6c7e6bc3bbb8443958cc46b40618978931a on master.

@calebcartwright I'm just going to merge this in. I'm pretty confident in our builds and tests.

Sorry, was traveling today just got to the hotel.
LGTM :)


Add the VAT

Description
Adds the Vat block!
Closes #143
Todo

[ ] Block functionality
[ ] Recipes
[ ] GUI

Checklist:

[ ] My code follows the style guidelines of this project (.editorconfig, most IDEs will use this for you).
[ ] I have performed a self-review of my own code.
[ ] I have commented my code in areas it may be challenging to understand.
[ ] I have made corresponding changes to the documentation.
[ ] My changes are ready for review from a contributor.

Will recreate this PR from scratch in Neo after reworks


CC @lmvasquezg @KapJI as Windows experts

It seems our test suite isn't open sourced, I can add it myself later. If you can please post some basic example which I can turn into test.

My sample test:
BUCK:
# A rule that includes a single .rc file and compiles it into an object file.
windows_resource(
    name = "resources",
    srcs = [
        "resources.rc",
    ],
)
# A rule that links against the above windows_resource rule.
cxx_binary(
    name = "app",
    srcs = [
        "main.cpp",
    ],
    deps = [
        ":resources"
    ],
    linker_flags = [
        "User32.lib",
    ],
)

main.cpp:
#include <iostream>
#include <Windows.h>

int main(int argc, const char* argv[])
{
	std::string message;
	message.resize(50);
	int returnValue = LoadStringA(NULL, 4, message.data(), message.size());
	if (returnValue == 0)
	{
		std::cout << "Failed to read resource";
		return 1;
	}

	message.resize(returnValue);
	std::cout << message;
	return 0;
}

message.h:
#define MESSAGE_TO_PRINT "Hello from header"

resources.rc:
#include <winresrc.h>

#include "message.h"

STRINGTABLE
BEGIN
4 MESSAGE_TO_PRINT
END


chipping gen bug

when u put sth in it with full health it will not get consumed as u said
but if u put sf things it will consume them
I put an unbreakable thing from foxy + soulband  and it get consume

d

Fixed with 4bd5aee


Speed compare with vllm?

Speed compare with vllm?

We do not plan to publish performance numbers that compare TensorRT-LLM with vLLM.
Our internal measurements show that TensorRT-LLM’s in-flight batching and paged KV cache features work well and TensorRT-LLM can deliver great performance. We’d be happy to provide you with performance numbers for relevant cases.
Is there a particular use case that you’d be interested in?

We do not plan to publish performance numbers that compare TensorRT-LLM with vLLM.
Our internal measurements show that TensorRT-LLM’s in-flight batching and paged KV cache features work well and TensorRT-LLM can deliver great performance. We’d be happy to provide you with performance numbers for relevant cases.
Is there a particular use case that you’d be interested in?

I wonder how is the performance compared with FT.
In my test, TRT-LLM is only 60% ~ 70% the performance of FT on A100. Is that the fact?

does TensorRT LLM also support contiguous batching like vllm does ?

@StephennFernandes
trt llm support in-flight batching same as contiguous batching

BentoML team recently did a benchmark comparing vLLM, TRT-LLM, TGI: https://www.bentoml.com/blog/benchmarking-llm-inference-backends
With the current releases running Llama3 8b and 70b quantized, TRT-LLM shows much better token generation rate overall, but vLLM has better time to first token.

BentoML team recently did a benchmark comparing vLLM, TRT-LLM, TGI: https://www.bentoml.com/blog/benchmarking-llm-inference-backends
With the current releases running Llama3 8b and 70b quantized, TRT-LLM shows much better token generation rate overall, but vLLM has better time to first token.

LMdeploy niubi！


Placement Rules: Support Index and Override properties for Group

Development Task
Sometimes, we want to control the overlay order of groups and sometimes we want specific group to override rules in other groups.

Is there any scenario that groups don't need to override each other?
IMHO, new rules should always override old ones in all scenarios.

@djshow832 Maybe in the process of some manual tests. Anyway, overriding is a small task compared to overlay order.
I think that he means TiDB can mark its rules with index 1 > the default rule with index 0. So once TiDB starts to create rules, the default rule fails automatically.

In original design, PD allows multiple sources of placement rules, including pd-ctl. But I don't think there's a need to distinguish them. It just complicates the system and confuses users.

There are different use scenarios. Normally, rules in different groups come from different sources.
Sometimes, the source of the rules needs to add some additional rules to the original rules. For example, TiFlash needs to add some extra learner peer on TiFlash nodes, without affecting original rules. In this case, override is false, which means that multiple groups take effect at the same time and do not interfere with each other.
In some other cases, the source of rules needs full control of data placement, then we will need to set override property for the group to disable other groups.

Task dismantling:

[ ] consider group properties when calculating rule list
[ ] ruleGroup load / save
[ ] ruleGroup API + pd-ctl support


fixed return type on disconnect method in pgsql connection

According to the inherited docs this should return an instance of ConnectionInterface. Fixed this return type, asking myself why the feedback of the pg_close (boolean) method wasn't returned.
Anyone knows why the feedback isn't returned?

I will try to provide a unittest later today so this won't happen again. I'll if I can implement some more tests on the PostgreSQL driver since there are hardly any.

Help needed on the unittest. How to mock the resource in the connection to allow the pg_close method to execute?

@YP28 Best bet for getting assistance for the tests is to join the #zftalk.dev IRC channel on Freenode; somebody there will likely be able to assist.


Update Documentation and a few fixes in HTTP

Updated the YAML Format documentation
Also fixed HTTP post requests
Added support for Windows Blocking
Fixed unicode headers
Added stop_magic

Fixes #57

YO TRAV HIT THAT BEAT

@zmallen you're a nerd.


Design:  Review short term token support for aws, gcp, azure for OADP

Review the three clouds together to ensure:

The setup and configuration is as consistent as possible
Any DPA related variables have as much overlap as makes sense
Order of operations is consistent as possible

Review and rewrite of documentation that exists in OADP for our control

Review edge cases like:

imagestream backup and restore
multiple buckets
default buckets
anything else?

/remove-lifecycle stale

/remove-lifecycle stale


[newton] RLM-219 Reduce memory usage in AIOs

This adds a few optimizations for Galera and reduces
the amount of threads and workers in AIOS to help
alleviate memory pressure and allow failing Tempest
tests to pass.
Issue: RLM-219

Since Newton support is going eol soon, so it's probably better if we managed these values in our local branch.  I've pulled out all of the values that I could find that don't apply to Newton.
For previous discussion, see:
https://github.com/rcbops/rpc-openstack/pull/2576

recheck_cit_xenial_minorupgrade_swift
recheck_cit_xenial_deploy_swift
recheck_cit_trusty_minorupgrade_swift
recheck_cit_artifact

Minor upgrade job not required.

@cloudnull @odyssey4me Put a review upstream as well for newton (https://review.openstack.org/#/c/511051/), if that works we can push it there too and then migrate out these settings out once we have them in our newton build.
This will at least allow us to keep moving forward.

The minorupgrade tests have been removed with https://github.com/rcbops/rpc-gating/commit/01f05822fdbd9910677c4123c5c3686b199f442a so they are no longer applicable.

recheck_cit_xenial_swift_deploy

recheck_cit_xenial_swift_deploy


Set limit to phone number to maximum 15 digits

Maximum phone number length imposed by The [International Telecommunication Union] is 15 digits.
https://en.wikipedia.org/wiki/Telephone_numbering_plan#:~:text=The%20International%20Telecommunication%20Union%20(ITU,15%20digits%20to%20telephone%20numbers.

Codecov Report

Merging #282 (a5a97f0) into master (a40e975) will not change coverage.
The diff coverage is 0.00%.

@@            Coverage Diff            @@
##             master     #282   +/-   ##
=========================================
  Coverage     68.93%   68.93%
  Complexity      811      811
=========================================
  Files           131      131
  Lines          2897     2897
  Branches        376      376
=========================================
  Hits           1997     1997
  Misses          772      772
  Partials        128      128

Impacted Files
Coverage Δ

...address/logic/commands/applicant/AddApplicant.java
66.66% <0.00%> (ø)

...rc/main/java/seedu/address/model/person/Phone.java
90.00% <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a40e975...a5a97f0. Read the comment docs.


adds missing formats in the registry

fixes #845
closes #1811

@handrews thanks for the help here. Another question/remark I had is why do we need the issue link in the table view? could we do away with it to replace the column by notes (like using the content encoding in 3.1 instead of base64url format)?

@baywet I don't know anything about that table so someone else will have to answer the thing about the issue link.  It might be good to have a notes column, or an OAS version translation column of some sort.

@baywet I don't know anything about that table so someone else will have to answer the thing about the issue link. It might be good to have a notes column, or an OAS version translation column of some sort.

Looks like this was originally added by @webron and @MikeRalphson in #1762. Any opposition to removing the Issue column to make room for a notes and version column?

@baywet I don't know anything about that table so someone else will have to answer the thing about the issue link. It might be good to have a notes column, or an OAS version translation column of some sort.

Looks like this was originally added by @webron and @MikeRalphson in #1762. Any opposition to removing the Issue column to make room for a notes, parent type and version columns?

I have no objection to removing the issue column from the index table, as long as the optional issue link is maintained in the detail pages.

@MikeRalphson thanks! I just pushed some changes, let me know what you think.

@MikeRalphson thanks! I just pushed some changes, let me know what you think.

@baywet, I went to https://baywet.github.io/OpenAPI-Specification/ hoping you'd set up your branch as a GitHub pages source so we could view the built changes, and the top-level gh-pages index page is there, but https://baywet.github.io/registry/index.html 404s?

@baywet, I went to https://baywet.github.io/OpenAPI-Specification/ hoping you'd set up your branch as a GitHub pages source so we could view the built changes, and the top-level gh-pages index page is there, but https://baywet.github.io/registry/index.html 404s?

@MikeRalphson the fork had automatically setup the website. But the root is already used by my blog and it seems the template doesn't handle being on a sub segment very well. Also the branch was setup to gh_pages but I'm making my changes on a feature branch.
Bottom line, it's now refreshed at https://baywet.github.io/OpenAPI-Specification/registry/format/ and if you click on any link, the OpenApi-Specification segment will be missing, so you need to add it back.

@baywet Just noticed that int16 is missing here as well as in https://spec.openapis.org/registry/format/: can we close this gap between int8 and int32 with this PR?

@ralfhandl this is something I noticed as well and meant to ask when opening the PR but forgot about it.

Note: I have made PR #3179 that, if merged, would allow forks to render the pages with working links.
There is a lot going on in this PR, and I think it would be easy to finish if it were split up a bit.  I keep losing track of things, and now that I managed to get a local rendering of the changes up I realize there were more things I didn't spot.  I'd like to suggest breaking this up to ensure that:

the easy things (data presentation and JSON Schema formats) get merged now
the more complex things (examples for complex cases that may  not all make sense) can be discussed in a focused way
the novel things are each clearly visible to those who might care about them (e.g. the audiences for base64url, char, and int16 are different, as are the motivations for adding them)

@baywet I don't have any veto authority here and wouldn't try to override a larger consensus anyway, so if @OAI/tsc approves this PR as is then that's fine with me.  But I would suggest making the following PRs, ideally rebased onto #3179 if @MikeRalphson finds out that it works and merges it:

Update how the current data is presented:

Drop the issue links
Add the "Type" column
Add the "Source" column with links like the following (foruri-template): JSON Schema, RFC 6570 instead of the current "Open" links
Add a version column, but call it "OAS Version" instead of "OpenAPI Version" (OAS it the correct name for the specification) and put it between Source and Remarks (because all of the remarks have to do with versioning)
Add the Remarks column
Include the new boilerplate needed for each existing individual .md page
Include the RFC link fix for date.md, date-time.md, and any other existing file changes I've missed
Include the changes except the  new examples on byte.md and binary.md

Add examples to existing formats binary and byte
Add the missing formats from OAS 3.0.3 (this would be password.md which I think you missed?  If you intentionally left it out, you don't need to make a PR for it, but if it was an accidental omission, the OAS 3.0.3 format(s) should get their own PR separate from the JSON Schema ones and the new ones)
Add the missing formats from JSON Schema (I think this is everything else except base64.md, char.md, and uint16.md, but please double-check)
Add the base64url format as its own PR, including the examples
Add the char.md format as its own PR, noting that it is being proposed due to common usage in the wild, including kiota
Add the int16 format, noting that it is filling a gap in the current set (I'm confused over the presence of uint8 but no other uints, so if you and/or @ralfhandl think all of those should be filled in, you could do all of them as part of the same PR as it's the same motivation)

As I said, I neither have now want veto over any consensus to merge this PR as-is now, but I'm completely overwhelmed by the current state of things and am not comfortable signing off on it.

You'd probably want the display change PR merged first before doing the others, but that should be the most straightforward to resolve, I think.

@baywet For the first suggested PR I was saying what to include, with a few modifications:

Yes, I am asking you to change the title of the links from "Open" to indicating where they go.
Yes, I am asking you to move the version column and change its name
The point of splitting the example-adding into its own PR is that there are multiple confusing conversations about them, which might change based on the 3.0.4 work, and its hard to follow when that's tangled up in everything else - there's no harm in doing this in two steps
Everything else I was saying to include as you already have it

I'm not sure what you mean by reducing overhead.
It is fine with me if you want to keep the JSON Schema formats and password in this PR.  Likewise if you want to include the examples.  I find it more confusing than not, but I'm not the person in control of merging anyway and @ralfhandl signed off on it so I don't think I need to.
My suggestions here come from experience doing complex PRs on JSON Schema - it always ended up easier to move multiple PRs through, even if there were interdependencies, than manage mega-conversations.

Agreed, atomicity reduces complexity of individual PRs and makes them easier and faster to process at an individual level. And this usually results to a faster overall processing time. The only exception I was mentioning in my previous reply is when one starts to mix concerns in PRs and/or having interdependencies to coordinate.

added the label for the source
added the password format
moved and renamed columns

After https://github.com/OAI/OpenAPI-Specification/pull/3179 gets merged, I'll rebase this one, and after that I'll move base64url, chat, int16 and uint8 to individual PRs as well as move binary/byte examples to separate PRs.
Things are now on hold unless additional feedback comes up.
How does that sound?

@handrews now that #3179 has been merged, I've just rebased.
Before moving anything anywhere, I know @darrelmiller had collected feedback he needed to provide here from the part of the meeting yesterday I wasn't able to attend.

@baywet   Feedback from the meeting:

we don't want to include examples the registry entry page.  We can link to external sources that provide more information but we don't want to have to maintain the registry as source of guidance.
We would like to minimize the text that appears in the notes column to keep the table as compact as possible.  The remarks should be moved in the detail page.
We don't want to suggest that format values are tied to OpenAPI versions.  It would not be good if formats had different meanings in different OpenAPI versions.  We did agree that there is value in indicating that a format is no longer recommended for use in certain OpenAPI versions.  The suggestion was to include a deprecated column.  This column could contain a deprecation qualifier to indicate the OpenAPI version that it was deprecated since.

Thanks for the recap here @darrelmiller . I just made some of the changes.
For the deprecated since column, do we want to keep the OAS version column as well? or remove it too?
Should we have an "available since" column? (thinking about a number of formats that were not available in 3.0)

Thank you @baywet  for all your work on this.
  Thanks.

@baywet I don't see where int16 is part of the OAS spec?  It was also not present as a file before this PR.

I just noticed html as well.  That, too, needs to be split out.
@darrelmiller would it make sense to put "OAS" in the "source" column for the formats defined in at least one version of OAS? (presumably without linking because we don't want to get version-specific?)

@baywet apologies for my int16 comments, I got confused - yes, you are correct to leave uint8 where it is!

Nice, love this! Thank you for these additions.
One thing I'd like to see as well is an iso8601 format, indicating essentially "date or date-time". Or would format: [date, date-time] work similarly to how type: [string, number] works?

@rattrayalex I don't think this is something you can do (format doesn't support arrays AFAIK), you'd have to use 2 schema entries in an oneOf IMHO

@handrews :

html was already here before this PR and the only changes are template changes. Considering that, do you still want me to move it to a separate PR and what's the rationale?
decimal. moving to a separate PR soon. (next commit)

@darrelmiller would it make sense to put "OAS" in the "source" column for the formats defined in at least one version of OAS? (presumably without linking because we don't want to get version-specific?)

Resurfacing this comment for @darrelmiller

@darrelmiller would it make sense to put "OAS" in the "source" column for the formats defined in at least one version of OAS? (presumably without linking because we don't want to get version-specific?)

Resurfacing this comment for @darrelmiller

I guess we could.  I don't feel strongly about this.

Thanks for the precision. Added.
@darrelmiller @handrews @MikeRalphson @ralfhandl We should be good for hopefully final review this time.

For those following along, a reminder that the rendered version of this PR is here
@baywet I notice there's a blank entry in the formats table - I think it may be due to a YAML indent issue in idn-email.md?

@MikeRalphson thanks for catching that. Your suggestion appears to have fixed it.

Wait, why is there still a deprecated_note with a version if @darrelmiller wanted to not list a version, and all the UI does is change it to a Yes/No in the display?

@handrews The version is not displayed anywhere at this time, it's used as a flag for the table and as a note for us in case we need to review why was this particular format marked as deprecated. The only place where people can see it is if they view the source MD. If we want to have it gone from the YAML header too (no mention of the version), I'd be happy to update it.

@handrews @baywet  Sorry, I should have been more clear.  I wanted to call it deprecation note to indicate that a version was not a required value to indicate that something was deprecated.  e.g. If a registry entry was created because we discovered that people had using the term "flag" to indicate boolean but there was no corresponding specification, we should be able to indicate that we want it to be deprecated.
I don't have strong opinion on whether we should or shouldn't show the OAS version in the deprecation note.
I thought I had caught all the RFCs with missing links.  I'll do one more sweep and if I miss any today, I'll submit a separate PR for them.

Thank you @baywet @handrews @ralfhandl for your herculean efforts!

Ditto! Thanks everyone for the help getting this merged!!! 🚀🚀🚀🚀


Map export / save to PDF / PNG

Working but still needs some tweaking + added legend. But what do we think so far?


Travis CI doesn't build due to composer version issue

Travis CI is failing to resolve composer requirements (but Vagrant, building from the same composer.json, is fine)
Loading composer repositories with package information
Updating dependencies (including require-dev)
Your requirements could not be resolved to an installable set of packages.
  Problem 1
    - Installation request for codeclimate/php-test-reporter 0.1.* -> satisfiable by codeclimate/php-test-reporter[v0.1.2].
    - codeclimate/php-test-reporter v0.1.2 requires satooshi/php-coveralls 0.6.* -> satisfiable by satooshi/php-coveralls[v0.6.0, v0.6.1].
    - satooshi/php-coveralls v0.6.0 requires psr/log 1.0.0 -> satisfiable by psr/log[1.0.0].
    - satooshi/php-coveralls v0.6.1 requires psr/log 1.0.0 -> satisfiable by psr/log[1.0.0].
    - Conclusion: don't install psr/log 1.0.0

Also Travis should be building database-refactor.

version problem resolved by updating most versions for dev dependencies to "*" and phpunit to "<6".
phpunit 6 requires namespaces for tests.
different other versions of phpunit are required for different php versions (5.3 and 5.6 are both no longer supported by the latest phpunit, but we still want to use them with travisci for now)


Improve lomax documentation.

Add required packages, deps and other things one might need to build/test lomax

I added some documentation for the terraform bits.

I added some documentation for the terraform bits.

There's now some documentation on lomax as well, generated automagically using godoc.org here's the link: https://godoc.org/github.com/opendns/lemming


Warnings for long commit messages does not work anymore

VSCode Version: Code 1.11.0 (9122ec4b1068bd391e41974f46a4b54c1b9c90f9, 2017-04-05T21:13:24.700Z)
OS Version: Linux x64 4.8.0-41-generic
Extensions:

Extension
Author
Version

vscode-eslint
dbaeumer
1.2.8

githistory
donjayamanne
0.2.0

tslint
eg2
0.11.0

prettify-json
mohsen1
0.0.3

final-newline
samverschueren
0.3.2

join-lines
wmaurer
0.2.2

local-history
xyz
1.1.1

The Git UI used to show a warning when the commit summary exceeded 50 or so characters. This warning is gone and I can't find a config option.
Steps to Reproduce:

Enter a commit message with more than 50 characters in the first line.

Probably duplicate of #23412, see #21144

Tracking with #21144


Is there any way from which we can fetch specific nfts from wallet?

Following command does return all nfts:
spl-token accounts --output json
Is there any way we can filter nfts?

Hi! We don't use Github for support.  Join us on https://solana.com/discord instead


[BUG]Preview is not showing images if in dlc folder.

Describe the bug
Images are not shown if they are in dlc folder.
To Reproduce
Steps to reproduce the behavior:

Go to 'dlc000_xxx'
Open on 'any_file.gfx'
Turn on preview
See error

Expected behavior
All images should be shown.
Screenshots

Desktop (please complete the following information):

OS: Windows 7
VSCode version: 1.70.3
Extension version: v0.5.2

Fixed in https://github.com/herbix/hoi4modutilities/commit/25b4a46437b251a17941ca29cd05764afbff4b8b .


Not working with Due

Any chance for the update to support Due?

Sorry I am unable to do something about it as I currently don't own one. You are welcome to contribute.


[Process Action] Optimize split to have at least N model calls even if it holds in one context

This one is quite tricky. Will punt it until we get signal that this may be an issue.


library throw loader error when import in react cra

hi,
I want to use this library with react-player library.
At this official example,this package works fine. You can see demo
In my application it throws loaders error.
./node_modules/screenfull/index.js 105:54
Module parse failed: Unexpected token (105:54)
File was processed with these loaders:
  ./node_modules/babel-loader/lib/index.js
You may need an additional loader to handle the result of these loaders.
|   element: {
|     enumerable: true,
     get: () => document[nativeAPI.fullscreenElement] ?? undefined
|   },
|   isEnabled: {

package.json
 "react": "^16.13.1",
 "react-dom": "^16.13.1",
 "react-player": "^2.9.0",
 "react-router-dom": "^5.1.2",
 "redux": "^4.0.5",
 "redux-thunk": "^2.3.0",
 "react-scripts": "^4.0.3"

 "browserslist": {
    "production": [
      ">0.2%",
      "not dead",
      "not op_mini all"
    ],
    "development": [
      "last 1 chrome version",
      "last 1 firefox version",
      "last 1 safari version"
    ]
  },

Thanks for the help

Duplicate of #195

As a quick workaround,
You can downgrade screenfull version to 5.2.0.
In version 6.0.0. engine is higher than node v.14.


Git Squash Command

Squash command added to JGit repository. Also added to the JGitFileSystemProvider in the setAttribute method.
GitCommand added to implement git commands based on command pattern.

just note, @adrielparedes took over @Salaboy work on squash and finished it (finally!). thank for @Salaboy for initial impl; and thank you for @adrielparedes to work on this

👍 Well done @adrielparedes

Some concerns:

Lack of test coverage (not related with @adrielparedes works but legacy code)
what happens if two users do the squash operation concurrently?
what happens if the FS is locked in the middle of the operation?

@ederign at the updateReferenceAndReleaseRevisionWalk method, he tries to update the reference to the old commits by the new squashed commit. If the force update does not work because a LOCK_FAILURE, it throws a ConcurrentRefUpdateException.
@porcelli @ederign I fixed all the types and javadoc, what about the tests?
@Salaboy if you have something to add, please join us!

@adrielparedes is a newcomer; he didn't touch this code (he just moved it around to make it more readable/maintainable/organized as I asked).
Scope this task to understand how it deeply works is way too much in my opinion. Our jgit use is not trivial, after all these years this piece of code still maintained by very few. Ask @adrielparedes to acquire such knowledge in his first task, while mostly people has workaround it isn't very productive I think. @adrielparedes will eventually gain such knowledge as a consequence of his upcoming tasks in the future months/years.
Note: I'm quite sure that @adrielparedes would love to do so, I'm just raising concerns about tasks, priorities and deliveries and timeline.

Guys, definitely +1 for adding some tests. @porcelli you are saying that "Our jgit use is not trivial" and I agree - that's actually a good reason to increase existing test coverage. This stuff can be tricky, especially in clustered environment - and in that case creating unit coverage is less expensive.

@rsynek @adrielparedes is adding tests for what he touched, as already mentioned to @ederign I think we can't afford to have him adding tests for the whole jgit backend, in the scope of his current task.
this will happen over time

for the record: jgit backend impl has 85% of test coverage, probably one of the higher coverage number in the tooling ecosystem

+1
@ederign ?

@porcelli I've updated the PR with the try/finally and the test

+1 (ready to merge)

Seems like we are almost ready to go. @adrielparedes or @porcelli I would like to have a walk through in this solution before merge it. Can we have a quick g+ ?

thanks @adrielparedes and thanks for the explanation @porcelli !
merged!


Lab B1 - First custom engine agent

https://github.com/microsoft/copilot-camp/blob/main/docs/pages/custom-engine/01-custom-engine-agent.md
Update Exercise 1 Step 2 to reflect current state of Azure AI Studio. System Prompt templates are only available in the old look which is not the default. Instead of selecting a system message template, let users enter the following system prompt:
You are a Shakespearean writing assistant who speaks in a Shakespearean style. You help people come up with creative ideas and content like stories, poems, and songs that use Shakespearean style of writing style, including words like "thou" and "hath”.
Here are some example of Shakespeare's style:

Romeo, Romeo! Wherefore art thou Romeo?
Love looks not with the eyes, but with the mind; and therefore is winged Cupid painted blind.
Shall I compare thee to a summer’s day? Thou art more lovely and more temperate.

Labs are adjusted with the new AI Foundry.


Tracy profiling fails on TG

Description
Seeing below error when tracy profiling is enabled on TG. This particular error appears on profiling any TG test. Clean build doesn't resolve the issue. The following error also doesn't appear on N150.
Error
           BuildKernels | INFO     | ncrisc link failure -- cmd: cd /proj_sw/user_dev/kpaigwar/tt-metal/built/4096/kernels/vc_packet_router/10795137441912823728/ncrisc/ && /proj_sw/user_dev
/kpaigwar/tt-metal/tt_metal/third_party/sfpi/compiler/bin/riscv32-unknown-elf-g++ -mwormhole -march=rv32imw -mtune=rvtt-b1 -mabi=ilp32 -std=c++17 -flto -ffast-math -fno-exceptions -Wl,-z,ma
x-page-size=16 -Wl,-z,common-page-size=16 -nostartfiles -Os -T/proj_sw/user_dev/kpaigwar/tt-metal/runtime/hw/toolchain/ncrisc.ld ncrisck.o /proj_sw/user_dev/kpaigwar/tt-metal/runtime/hw/lib
/tmu-crt0k.o /proj_sw/user_dev/kpaigwar/tt-metal/runtime/hw/lib/substitutes.o  -Xlinker "--just-symbols=/proj_sw/user_dev/kpaigwar/tt-metal/built/4096/firmware/ncrisc/ncrisc_weakened.elf" -
o /proj_sw/user_dev/kpaigwar/tt-metal/built/4096/kernels/vc_packet_router/10795137441912823728/ncrisc/ncrisc.elf
/proj_sw/user_dev/kpaigwar/tt-metal/tt_metal/third_party/sfpi/compiler/bin/../lib/gcc/riscv32-unknown-elf/10.2.0/../../../../riscv32-unknown-elf/bin/ld: address 0xffb00c18 of /proj_sw/user_
dev/kpaigwar/tt-metal/built/4096/kernels/vc_packet_router/10795137441912823728/ncrisc/ncrisc.elf section `.ldm_data' is not within region `NCRISC_LOCAL_DATA_MEM'
/proj_sw/user_dev/kpaigwar/tt-metal/tt_metal/third_party/sfpi/compiler/bin/../lib/gcc/riscv32-unknown-elf/10.2.0/../../../../riscv32-unknown-elf/bin/ld: address 0xffb00c18 of /proj_sw/user_
dev/kpaigwar/tt-metal/built/4096/kernels/vc_packet_router/10795137441912823728/ncrisc/ncrisc.elf section `.ldm_data' is not within region `NCRISC_LOCAL_DATA_MEM'
/proj_sw/user_dev/kpaigwar/tt-metal/tt_metal/third_party/sfpi/compiler/bin/../lib/gcc/riscv32-unknown-elf/10.2.0/../../../../riscv32-unknown-elf/bin/ld: section .stack VMA [00000000ffb00bf0
,00000000ffb00fff] overlaps section .ldm_data VMA [00000000ffb00740,00000000ffb00c17]
collect2: error: ld returned 1 exit status
                 Always | FATAL    | ncrisc build failed
                 Always | FATAL    | Failed to generate binaries for vc_packet_router TT_THROW @ ../tt_metal/jit_build/build.cpp:408: tt::exception

fyi @cglagovichTT

@mo-tenstorrent @ubcheema Any update on this? We are currently blocked with profiling any perf changes on TG because of this issue.

@uaydonat fyi

This PR is a fix for the memory overflow issue we are seeing here. Should be in once it goes through CI.
Please give the branch a try ad confirm perf report can be generated for the particular test you are looking at.

The above PR is merged on main now.

Fixed 🙌


Rename onDismissible to onDismiss for OuiCallOut

Description
Rename onDismissible prop to onDismiss.
Issues Resolved

Check List

[ ] New functionality includes testing.
[ ] New functionality has been documented.
[ ] All tests pass

[ ] yarn lint
[ ] yarn test-unit

[ ] Update CHANGELOG.md
[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Isn't this a breaking change? If so, we should retain the old prop and mark it deprecated, but also honor it until OUI 2.

No, not a breaking change - it's a new prop we're introducing in this release. Just a correction to https://github.com/opensearch-project/oui/pull/985


Delete doesn't work

VPNv2/ProfileName
Unique alpha numeric identifier for the profile. The profile name must not include a forward slash (/).
Supported operations include Get, Add, and Delete.

I can confirm 100% that the delete function does not work when you un-assign the profile from a device. Is that by design ?

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

ID: c8b1b81d-3c35-0315-d149-f10af3a409b2
Version Independent ID: c34e7acb-ece7-d76f-fb41-470e19119ee0
Content: VPNv2 CSP - Windows Client Management
Content Source: windows/client-management/mdm/vpnv2-csp.md
Product: w10
Technology: windows
GitHub Login: @Dansimp
Microsoft Alias: dansimp

Closing this as this seems a duplicate of https://github.com/MicrosoftDocs/windows-itpro-docs/issues/10754. We'll track this issue there, as it has more details.


TODO

[x] Use private constants
[x] Missing test for square id
[x] index.as(Int32), replace by .not_nil!
[x] Clean up selection
[x] Make grid color configurable
[x] Handle config file from /etc/firegrid (if .config not available)
[x] fix ui texts position
[x] calculate  min ui size for precision (move method to grid ?)
[ ] Missing test for Square#precise_for?
[ ] Missing test for Grid#resize_for?
[ ] adjust precision/resize rates
[ ] Multiple monitors support (screenshot, focused monitor, low resolution, etc...)
[ ] Dev env (docker?) + README

Stuck

[ ] Validate configuration (create a crystal shard to validates hash (similar to ActiveModel::Validations outside this project first

Difference between my monitors seems to be related the grid format (as one of my monitor is ultra wide). Precision/Resize rates should take grid horizontal / vertical count into account.


Automate the app.module.ts file entries

Ionic version:  (check one with "x")
[ ] 1.x
[x ] 2.x
I'm submitting a ...  (check one with "x")
[ ] bug report
[x ] feature request
[ ] support request => Please do not submit support requests here, use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Current behavior:
CLI doesn't create relevant entries on app.module.ts file when we create the Pages and Providers. We have to do it manually.
Expected behavior:
I would like to have a CLI feature where when we create a Page or Provider it should automatically create the relevant imports, declarations,entryComponents and providers on the app.module.ts file.
Steps to reproduce:
> ionic generate page BillDetail
> ionic generate provider BillData

Hello! Thanks for using Ionic. So we are actually ahead of the game on this and have been working on this recently.  Excellent.Thank you @jgw96


Make Envoy global variables "leaky"

Currently Envoy has any number of global variables, for example:
  static DateFormatter date_formatter_;

From source/common/http/date_provider_impl.h. On program exist, these global variables are destructed. If Envoy is still running, this can lead to a use after free and a crash. On mobile this is particularly problematic. Instead, if we made these variables "leaky" (not destructed on shutdown) then we could avoid this issue

cc: @alyssawilk


Running flutter doctor through VSCode plugin isn't working

Steps to Reproduce

Install flutter and flutter plugin through VSCode
Install Android SDK through cli tools and add the folders to the PATH and other required variables like ANDROID_SDK_ROOT through .zshrc
Run flutter doctor through the plugin and through the integrated terminal

Logs

Output of flutter doctor in the VSCode output panel when running it via the plugin:
[flutter] flutter doctor -v
[✓] Flutter (Channel stable, 1.20.4, on Linux, locale en_US.UTF-8)
    • Flutter version 1.20.4 at /opt/flutter
    • Framework revision fba99f6cf9 (2 days ago), 2020-09-14 15:32:52 -0700
    • Engine revision d1bc06f032
    • Dart version 2.9.2

[✗] Android toolchain - develop for Android devices
    ✗ Unable to locate Android SDK.
      Install Android Studio from: https://developer.android.com/studio/index.html
      On first launch it will assist you in installing the Android SDK components.
      (or visit https://flutter.dev/docs/get-started/install/linux#android-setup for detailed instructions).
      If the Android SDK has been installed to a custom location, set ANDROID_SDK_ROOT to that location.
      You may also want to add it to your PATH environment variable.

[!] Android Studio (not installed)
    • Android Studio not found; download from https://developer.android.com/studio/index.html
      (or visit https://flutter.dev/docs/get-started/install/linux#android-setup for detailed instructions).

[!] Connected device
    ! No devices available

! Doctor found issues in 3 categories.
exit code 0

Output of flutter doctor in the VSCode integrated terminal using the cli command:
[✓] Flutter (Channel stable, 1.20.4, on Linux, locale en_US.UTF-8)
    • Flutter version 1.20.4 at /opt/flutter
    • Framework revision fba99f6cf9 (2 days ago), 2020-09-14 15:32:52 -0700
    • Engine revision d1bc06f032
    • Dart version 2.9.2

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
    • Android SDK at /opt/AndroidSDK
    • Platform android-29, build-tools 29.0.3
    • ANDROID_HOME = /opt/AndroidSDK
    • ANDROID_SDK_ROOT = /opt/AndroidSDK
    • Java binary at: /usr/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.8+10)
    • All Android licenses accepted.

[!] Android Studio (not installed)
    • Android Studio not found; download from https://developer.android.com/studio/index.html
      (or visit https://flutter.dev/docs/get-started/install/linux#android-setup for detailed instructions).

[✓] Connected device (1 available)
    • ONEPLUS A6000 (mobile) • dcd7f7fc • android-arm64 • Android 10 (API 29)

! Doctor found issues in 1 category.

Hi @sudo-panda
Just tried through the plugin, no issues
[flutter] flutter doctor -v
[✓] Flutter (Channel master, 1.22.0-10.0.pre.204, on Mac OS X 10.15.6 19G2021, locale en-GB)
    • Flutter version 1.22.0-10.0.pre.204 at /Users/tahatesser/Code/flutter_master
    • Framework revision b2b5f2636a (17 hours ago), 2020-09-16 08:02:06 -0700
    • Engine revision 1ef10b240e
    • Dart version 2.10.0 (build 2.10.0-124.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at /Users/tahatesser/Code/sdk
    • Platform android-30, build-tools 30.0.2
    • ANDROID_HOME = /Users/tahatesser/Code/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 12.0)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.0, Build version 12A7209
    • CocoaPods version 1.9.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.0)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 49.0.2
    • Dart plugin version 193.7547
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)

[✓] VS Code (version 1.49.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.14.1

[✓] Connected device (4 available)
    • Taha’s iPhone (mobile) • 00008020-001059882212002E • ios            • iOS 14.0
    • macOS (desktop)        • macos                     • darwin-x64     • Mac OS X 10.15.6 19G2021
    • Web Server (web)       • web-server                • web-javascript • Flutter Tools
    • Chrome (web)           • chrome                    • web-javascript • Google Chrome 85.0.4183.102
    ! Error: Taha’s iPhone is busy: Copying cache files from device. Xcode will continue when Taha’s iPhone is finished. (code -10)

• No issues found!
exit code 0

Following the indication of VS Code Flutter Extension, the correct way to address the problem is to open an issue in the dedicated repository.
Closing, as this isn't an issue with Flutter itself. If you disagree, please write in the comments and I will reopen it.
Thank you


CORS security: reflecting any origin header value when configured to * is dangerous

When CORS policy is configured to *, current Yii2 CORS filter will actively convert it to reflect any Origin header value. This kind of behavior is dangerous and has caused many security problems in the past.
Some similar security issues:
https://github.com/cyu/rack-cors/issues/126
https://nodesecurity.io/advisories/148
Some related blog posts:
http://blog.portswigger.net/2016/10/exploiting-cors-misconfigurations-for.html
https://ejj.io/misconfigured-cors/

What do you propose?

Hi Alexander, thanks for your reply.
Current CORS standards(both W3C CORS and WHATWG fetch standard) have a clear definition for the wildcard *, which means any domain is allowed. But they also have another important security requirement: Origin: * and Credentials: true cannot be used at the same time, to avoid overly loose permissions. Currently all browsers follow this requirement to disallow this configuration combination.
If a framework actively converts * to reflect any origin header value,  it means  Origin: * and Credentials: true can be used at the same time. This behavior leads to CORS protocol's security design to be bypassed, causing many misconfiguration security problems.
Therefore, I suggest frameworks to follow the standard definition of *.  When a user configures Origin:*,  frameworks just directly returns Access-control-Allow-Access: *. When a user configures both Origin:* and Credentials: true , frameworks  should warn users that this is a misconfiguration because browsers will not accept this combination.

Agree. Do you want to make a pull request?

Thanks. Yes, I'd love to make more contributions for this project.
I made a pull request(#16198 ) just now, and hope it could be helpful for you.


defining any "daysSelected" highlights a day before

Seem to be the same problem as the other person (https://github.com/mauroreisvieira/hello-week/issues/19) had in the issue tracker, even with the most recent version. Setting any "daySelected" option unfortunately highlights those day(s) before. I've opened a new issueTracker because it may not be related.
I'm from Canada, at Mountain Daylight Time (MDT) timezone.
I'm using the newest version: 2.2.0 of hello-week
Perhaps the bug has to do something with the Daylight Saving Time (DST)?

Hi @psishock thanks for you report. This timezone kill me :)
I will check this issue.
Thanks again.

Hi @psishock @vbrodrigo in fact the bug is already solved but I forgot to generate a new distribution, so I generated a new version with the old code.
I apologize for the inconvenience, you can test version 2.3.0.

Ah, perfect!
Thank you so much mauroreisvieira for looking into this. At least this time, apparently it was an easy fix. :) Indeed it works as intended now.

Oh, testing it some more revealed that this bugfix actually breaks the ".getDays()" method.
It returns any predefined days by the "daysSelected" parameter a day before.
However, it returns any additionally selected days (with a mouse click for instance) at the right date. But the predefined parameter days will still be wrong.
Can't we ignore the timezone-parsing all together, and just work with plain inputted and outputted numbers instead?

Hi @psishock to make all validations I convert all dates to timestamp, to compare dates and other points. Also is more simple to format date if is in timestamp.
The problem to convert this date in timestamp its the differents timezons.

Hi @psishock can you try the new version:
https://github.com/mauroreisvieira/hello-week/releases/tag/2.4.0
I need start writing the unit test to check that kind of changes don't break other methods.

Hi @mauroreisvieira, we're getting closer but still now quite there.
So now when we call the ".getDays()" method, it will print all the preset values a day before (this haven't changed). Also any further selected/clicked days are now printed with that method a day before as well (this was good at 2.3.0, but got broken at 2.4.0).
On the upside, at least it's consistently a day off everywhere now. :) Now if we can shift all the printed dates from the ".getDays()" a day after, it shall be great.

I need start writing the unit test to check that kind of changes don't break other methods.

Indeed, there are times when I try to cut corners by skipping writing unit testing and just do the testing mannualy by myself.
Skipping it works usually for small projects. But as the code grows larger, more complex, and the project keeps adding more and more features over a longer period of time, then unit testing do help a lot, by not making you test every single small features again and again manually.
That being said, writing tests for different timezones and similar, browser based and date related issues indeed can be challenging to do so.
To cheer you up a bit, here is a short little anecdotal video about working with dates:
https://www.youtube.com/watch?v=-5wpm-gesOY

Hi @psishock thanks again for your feedback. I don't have much time to fix this issues, so basically I write code, test in my browser and lunch the new version. And sometimes broke other methods.

This issue is stale because it has been open for 30 days with no activity.


chore: fix flaky ImportExportForkApplication_spec

Description
The  anchor tag momentarily can have an empty string as the url. On cypress the click seemed to happen before the url was set causing a reload at the /applications route itself. To fix we are hiding the button until the url is set.
Fixes https://github.com/appsmithorg/appsmith/issues/24016
Type of change

Chore (housekeeping or task changes that don't impact user perception)

Testing

How Has This Been Tested?

[x] Manual
[ ] Jest
[x] Cypress

Checklist:
Dev activity

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[ ] PR is being merged under a feature flag

/ok-to-test

/ok-to-test

/ok-to-test

/ok-to-test

I think we also left a comment on the respective bug yesterday itself.
Note: we have the random reconnect modal appearing bug still open.


Some photos in feed are displayed sideways

photos taken in portrait mode are shown in landscape mode and stretched

Same happens to me

Same here, makes the app slightly useless. Other than that, it seems great.

hm do any of you have a photo you are willing to share to troubleshoot this? perhaps something in the exif of the file is not correct. i cannot replicate this :/

hm do any of you have a photo you are willing to share to troubleshoot this? perhaps something in the exif of the file is not correct. i cannot replicate this :/

Sure.
The File "Screenshot1.png" is how it is displayed in my App in the feed, the File "Screenshot2.png" is how the picture looks inside the app when i open it, the File "IMG_20240109_163452.jpg" is the original picture.

Yes, I have the same issue. But here's another clue. The same photos in the searched view appear correct.

Once you open the photo, it is correctly orientated.

hm. can either of you share the actual photo file here so i can add it to my dev build and debug it? Don't modify it in any way so that the exif metadata is unchanged

Here are two images that appear sideways in the feed.

The fix is release in version 0.0.162 @marcel89l @nagarrido @serdeltomas @redsunfex

Thank you!

confirmed fixed.

Any chance to re-open the issue? Everything looks fine for jpg files, but heic files have this problem


Math on arrays

It might be nice to have some element-wise array methods defined in standard lib, like that:
x = [1, 2, 3]
y = [4, 5, 6]
x * y  # => [4, 10, 18]

I'm not sure about possible conflicts, but it seems to me a good topic to discuss.
P. S. I suppose there's a way to come up with an appropriate syntax for that. The product sign in the example may be misleading, since + defined for Array is not element-wise.

I'm not sure, Array doesn't have any math-related operations, it's just a container. Plus * could mean "product" or "dot product". And there's the + method which complicates things.

As a side note, there's Enumerable#sum, which I'm not sure should belong there because it's also math-related. But then there's Enumerable#min and Enumerable#max, so it's hard to draw a line.

I think this issue is equivalent to
call a block with elements of some enums which have same index. In this case 1 and 4, 2 and 5, 3 and 6.
How about zip method with block arg ?
x = [1, 2, 3]
y = [4, 5, 6]
acc = []

x.zip(y) {|a, b| acc << a*b}
acc
# => [4, 10, 18]

this example is ruby code, I feel we can return "array of result of block" in crystal world :)

Note also that we got Matrix on which doing the arithmetic rather than the set operations makes more sense than on Array.

I'd like to close this with pointing towards Matrix.

Yes, array is an element container, not a math object. (which makes me wonder if Enumerable#sum belongs there, but it's so convenient)

I agree that Array#*, Array#+, etc. that already known for Ruby should work the same.
Maybe OP was looking not exactly for Matrix, but for feature, that is called Listable in Wolfram Mathematica. Here is example of it ported to Ruby: https://github.com/Nakilon/mll/blob/master/spec/_spec.rb#L154

Thanks everyone for the discussion!


Cannot find cudnn, but cudnn (5005) works with my Theano install

I just installed tensorflow from source with no errors but upon running the example from the "get started" page...
$ cd tensorflow/models/image/mnist
$ python convolutional.py
I receive the error that cudnn cannot be found. Here is the full output:
I tensorflow/stream_executor/dso_loader.cc:111] successfully opened CUDA library libcublas.so locally I tensorflow/stream_executor/dso_loader.cc:105] Couldn't open CUDA library libcudnn.so. LD_LIBRARY_PATH:  I tensorflow/stream_executor/cuda/cuda_dnn.cc:3304] Unable to load cuDNN DSO I tensorflow/stream_executor/dso_loader.cc:111] successfully opened CUDA library libcufft.so locally I tensorflow/stream_executor/dso_loader.cc:111] successfully opened CUDA library libcuda.so.1 locally I tensorflow/stream_executor/dso_loader.cc:111] successfully opened CUDA library libcurand.so locally Successfully downloaded train-images-idx3-ubyte.gz 9912422 bytes. Successfully downloaded train-labels-idx1-ubyte.gz 28881 bytes. Successfully downloaded t10k-images-idx3-ubyte.gz 1648877 bytes. Successfully downloaded t10k-labels-idx1-ubyte.gz 4542 bytes. Extracting data/train-images-idx3-ubyte.gz Extracting data/train-labels-idx1-ubyte.gz Extracting data/t10k-images-idx3-ubyte.gz Extracting data/t10k-labels-idx1-ubyte.gz I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:925] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero I tensorflow/core/common_runtime/gpu/gpu_device.cc:951] Found device 0 with properties:  name: GeForce GTX 1070 major: 6 minor: 1 memoryClockRate (GHz) 1.7465 pciBusID 0000:01:00.0 Total memory: 7.92GiB Free memory: 7.40GiB I tensorflow/core/common_runtime/gpu/gpu_device.cc:972] DMA: 0  I tensorflow/core/common_runtime/gpu/gpu_device.cc:982] 0:   Y  I tensorflow/core/common_runtime/gpu/gpu_device.cc:1041] Creating TensorFlow device (/gpu:0) -> (device: 0, name: GeForce GTX 1070, pci bus id: 0000:01:00.0) Initialized! F tensorflow/stream_executor/cuda/cuda_dnn.cc:208] could not find cudnnCreate in cudnn DSO; dlerror: /usr/local/lib/python2.7/dist-packages/tensorflow/python/_pywrap_tensorflow.so: undefined symbol: cudnnCreate Aborted (core dumped)
I am not sure what to try because cudnn works with Theano on this machine, it also states that it finds the GPU.

Reinstalled and specified the cudnn version when prompted. Seems to work now.


Can I preserve the original Ctrl-R behaviour of bash

Category

[x] fzf binary
[x] Key bindings

OS

[x] Linux

Shell

[x] bash

Thanks for your great tool !!
Would it be possible to redefine Ctrl-R or use another shortcut for fzf?
I'm using Ctrl-R in bash since years and I would like to use it in it's original form.
So I need another shortcut for fzf.
Thanks

You can edit /usr/share/fzf/key-bindings.bash or copy it somewhere else and replace your source line in .bashrc with new path. Also, similar question was already asked #253.

Elvyria, thanks for this useful hint.
I've seen that I could also disable keybindings when installing fzf at all.
But for me it's not clear how I could then use fzf?
BTW:
I installed fzf only in my home-directory.
Now I could change .fzf/shell/key-bindings.bash
Would it be possible to use Ctrl-Shift-R as a keybinding? I yes, how do I have to do this?
So I could use both ways to search through the history.

I'm not using bash, but shouldn't uppercase letter do the Shift trick? Also, you can pipe any strokes into fzf to search through them, like ls | fzf

If you don't want to edit that file, you can use the following trick:
# Bind CTRL-R from fzf to CTRL-X CTRL-R
bind "$(bind -s | grep '^"\\C-r"' | sed 's/"/"\\C-x/')"

# Then bind CTRL-R to the original reverse history search
bind '"\C-r": reverse-search-history'

Would it be possible to use Ctrl-Shift-R as a keybinding?

No, terminals can't tell the difference between CTRL-SHIFT-X and CTRL-X.


Add several sites

AbeBooks
BookBub
Cacoo
CryptPad
Delivery Code
Dreamwidth
Freeletics
PlayPosit
Yummly

I had  to fix trailing whitespace problems, but other than that, great contribution, thanks!


Bug in BatchNormalization

I found the implementation of the BatchNormalization is wrong. If the mode is 0, "featurewise normalization" is used and two attributes which named  running_mean and running_std are used to store the mean and STD of the training data. But both of these attributes are implemented as a Theano expression in the get_output function. In the test stage, they refer to mean and STD of the input data, not the mean and STD of the training data as we expected.
To fix it, the running_mean and running_std should be implemented as a Theano shared value as model's parameters, and should be updated when training. And I noted that the updating mechanism of them is different from the conventional parameters which are updated by their gradient. Therefore, we need to define individual update methods in some especial Layer, and the update list should be integrated into the conventional one in the training function of the Model.

I saw that too. And for Theano shared variables, you can't do assignment.
I belieive way to have a state that is updated is if we implement BatchNormalization with a scan loop.
Do you have other ideas?

I change the BatchNormalization.get_output into:
def get_output(self, train):
    X = self.get_input(train)

    if self.mode == 0:
        if train:
            m = X.mean(axis=0)
            # manual computation of std to prevent NaNs
            std = T.mean((X-m)**2 + self.epsilon, axis=0) ** 0.5
            X_normed = (X - m) / (std + self.epsilon)

            # I changed the following 5 lines
            self.running_mean = shared_zeros(self.input_shape)
            self.running_std = shared_ones((self.input_shape))

            mean_update = self.momentum * self.running_mean + (1-self.momentum) * m
            std_update = self.momentum * self.running_std + (1-self.momentum) * std

            self.updates = [(self.running_mean, mean_update), (self.running_std, std_update)]
        else:
            X_normed = (X - self.running_mean) / (self.running_std + self.epsilon)

    elif self.mode == 1:
        m = X.mean(axis=-1, keepdims=True)
        std = X.std(axis=-1, keepdims=True)
        X_normed = (X - m) / (std + self.epsilon)

    out = self.gamma * X_normed + self.beta
    return out

And append the updates into the Model's update list, for example in Graph model:
def compile(self, optimizer, loss, theano_mode=None):
    ......
    updates = self.optimizer.get_updates(self.params, self.constraints, train_loss)
    # add the especial update rules into the training process
    for m in self.nodes:
        if hasattr(self.nodes[m],'updates'):
            updates.extend(self.nodes[m].updates)
    ......

This is fixed; it required some significant framework-wide changes. All layers / containers now support custom "updates" that they define in a init_update method run after the layer has been connected.


[feature] Implement CI/CD ESLint and Prettier Integration for Improved Code Quality

Description:
This pull request fulfills the request outlined in issue #80. I have successfully set up a CI/CD pipeline that integrates ESLint and Prettier, ensuring consistent code quality and formatting across the repository.
Key Features:

Implemented ESLint to enforce coding standards and catch errors
Integrated Prettier to maintain consistent code formatting
Configured CI/CD pipeline to run automated linting and formatting checks

Screenshot:

@dillionverma
Kindly ensure that permission is given to the GitHub Action Workflow. See the screenshots below:
(Repository Setting)

(Permission Setting)

thank you so much for this 🙏
got some good ideas and improved upon it in the latest commit
incredibly helpful
lmk if you have any feedback on the current setup
will close for now


Bug fixed

Added an indent at line 177
There was a logical fault, the subtitle time line should have been continuous. But it skipped some parts because it set the "region_start" to "None" unexpectedly.
The line 177 should be in the range of "if"

Thanks!


Active not being set properly

I have a very simple action here and when a product is invalid and I render the new action, the url does not match properly
# POST /admin/products
def create
  product.assign_attributes(product_params.merge(account: current_account).permit!)

  if product.save
    flash[:success] = I18n.t("flash.message.create.success", resource: I18n.t("resource.product"))
    redirect_to admin_products_path
  else
    breadcrumb I18n.t("buttons.actions.new", resource: I18n.t("resource.product")), new_admin_product_path
    render :new
  end
end

Hi Eric,
Thanks for giving loaf a try.
Having looked at your example, one thing that I'm rather confused about is that you expect to have a breadcrumb on create action that gets triggered by POST request? I mean, there is no page that corresponds with create action. I would try actually defining breadcrumb in the new action itself.

Hi Eric,
Thanks for giving loaf a try.
Having looked at your example, one thing that I'm rather confused about is that you expect to have a breadcrumb on create action that gets triggered by POST request? I mean, there is no page that corresponds with create action. I would try actually defining breadcrumb in the new action itself.

Hey @piotrmurach Thank you for responding.
My issue is that if the parameters are invalid on the Create action, I then render the new action instead of redirect. Just like standard rails scaffold
By rendering the new action during the create action as you can see with the example above, the Breadcrumb is never set which is why I call breadcrumb.
Any suggestions on how to fix this?

Eric, your issue as reported is not very clear to me. The 'properly' is rather vague word and doesn't specify what you expect to happen vs what actually happens. The code sample doesn't answer these questions. Ideally, if you could either create an example of a Rails app that replicates the issue or even better submit a failing test. The integration tests use a dummy Rails app to confirm behaviour. You could, for example, add new products controller that demonstrates the problem. Here is an example of posts controller that handles the create action.


Patterns styles not generated

Description
When using a pattern function, like hstack, the css is not generated. Any other directives like css do work however, so I think the panda config file is working.
// panda.config.ts
import { defineConfig } from '@pandacss/dev'

const config: ReturnType<typeof defineConfig> = defineConfig({
  preflight: true,
  include: ['./src/**/*.{js,jsx,ts,tsx}'],
  exclude: [],
  theme: {
    extend: {},
  },
  outdir: 'styled-system',
})

export default config

/* src/global.css */
@layer reset, base, tokens, recipes, utilities;

// src/app/layout.tsx
import 'src/global.css'
...

// src/components/navigation.tsx
import { cx } from '@styles/css'
import { hstack } from '@styles/patterns'

export type NavigationProps = { className?: string }

export default function Navigation({ className }: NavigationProps) {
  ...
  return (
    <nav className={cx(hstack({ justify: 'space-between' }), className)}>
      ...
    </nav>
  )
}

Expected
The styles to be included
Reality
The classes are included but the styles are not (only for pattern functions)

Link to Reproduction
https://cannot-create-a-next-app-router-app-in-stackblitz.com
Steps to reproduce
No response
JS Framework
NextJS (TS)
Panda CSS Version
0.7.0
Browser
Google Chrome 115
Operating System

[X] macOS
[ ] Windows
[ ] Linux

Additional Information
I have a suspicion that the files are only being watched for the css function and nothing else.

Hey, it's already fixed but not yet merged & published on npm
it'll be available once https://github.com/chakra-ui/panda/pull/1055 gets merged
and this was the PR that fixed this issue

Hey, it's already fixed but not yet merged & published on npm it'll be available once #1055 gets merged and this was the PR that fixed this issue

Thanks for the info. I'll keep track of that PR and close this Issue. Do you have any ETA on the release?

pretty soon I guess ? 😅  can't be more precise


How to support misaligned entity annotation

Rasa NLU version:
0.12.3
Operating system (windows, osx, ...):
Ubuntu
Content of model configuration file:
language: "en"

pipeline:
- name: "nlp_spacy"
  model: "xx_ent_wiki_sm"
- name: "tokenizer_spacy"
- name: "intent_featurizer_count_vectors"
- name: "intent_classifier_tensorflow_embedding"
  intent_tokenization_flag: true
  intent_split_symbol: "_"

- name: "ner_crf"
- name: "ner_synonyms"

Issue:
I get the following warning when I try to train my model for NER.
WARNING  rasa_nlu.extractors.crf_entity_extractor - Misaligned entity annotation in sentence { }. Make sure the start and end values of the annotated training examples end at token boundaries (e.g. don't include trailing whitespaces).
The problem is, sometimes the entities can be a part of a token, therefore it won't start and end at the token's boundaries. Is there a template that supports this case?

Can you provide the example where the entity is not the whole token?

Related to #267


Stream queue full when selecting first GPU (intel) to be the execution device

ubuntu@ubuntu-OptiPlex-7050:~/saf$ cvsdk/apps/detector -c rtsp --device 0 --detector_type cvsdk-ssd -m person-detection-retail-0013 --detector_targets person --detector_confidence_threshold 0.1 -d
W0915 10:39:11.005597  6292 context.h:159] Using encoder x264enc instead of default from configuration
W0915 10:39:11.005789  6292 context.h:166] using decoder avdec_h264 instead of default from configuration
Run detection demo
W0915 10:39:11.006166  6292 camera_manager.cpp:107] Not built with PtGray FlyCapture SDK, camera: ptgray is not loaded
W0915 10:39:11.006191  6292 camera_manager.cpp:115] Not built with AlliedVision Vimba SDK, camera: vimba is not loaded
I0915 10:39:11.006381  6292 operator.cpp:92] Starting rtsp...
I0915 10:39:11.006412  6292 operator.cpp:92] Starting Throttler...
I0915 10:39:11.006430  6292 operator.cpp:92] Starting ObjectDetector...
 Dropping frame: 16
W0915 10:39:12.016669  6295 stream.cpp:151] Stream queue full. Dropping frame: 17
W0915 10:39:12.056174  6295 stream.cpp:151] Stream queue full. Dropping frame: 18
W0915 10:39:12.096135  6295 stream.cpp:151] Stream queue full. Dropping frame: 19
W0915 10:39:12.137199  6295 stream.cpp:151] Stream queue full. Dropping frame: 20
W0915 10:39:12.176890  6295 stream.cpp:151] Stream queue full. Dropping frame: 21
W0915 10:39:12.216143  6295 stream.cpp:151] Stream queue full. Dropping frame: 22
W0915 10:39:12.257129  6295 stream.cpp:151] Stream queue full. Dropping frame: 23
W0915 10:39:12.296762  6295 stream.cpp:151] Stream queue full. Dropping frame: 24
without --device 0 can run without any problem.

Does it drop frames indefinitely? OpenVINO on GPU or VPU may take seconds at the very beginning for runtime model conversion/optimization. If the issue remains, let me know your kernel version using uname -a and your CPU models using cat /proc/cpuinfo | grep "model name".

@syang100 you are right, the first 8 seconds (the frame rate of the camera is around 25fps) of the rtsp stream is struggling but after that it start to be able to perform the detection without queue full warning but on and off we still can see the frame is dropping.
W0915 11:54:26.437106  6921 stream.cpp:151] Stream queue full. Dropping frame: 189
W0915 11:54:26.476429  6921 stream.cpp:151] Stream queue full. Dropping frame: 190
W0915 11:54:26.529036  6921 stream.cpp:151] Stream queue full. Dropping frame: 191
W0915 11:54:26.562630  6921 stream.cpp:151] Stream queue full. Dropping frame: 192
W0915 11:54:26.597756  6921 stream.cpp:151] Stream queue full. Dropping frame: 193
W0915 11:54:26.649333  6921 stream.cpp:151] Stream queue full. Dropping frame: 194
I0915 11:54:26.677289  6921 throttler.cpp:60] Frame rate too high. Dropping frame: 195
W0915 11:54:26.716606  6921 stream.cpp:151] Stream queue full. Dropping frame: 196
W0915 11:54:26.770166  6921 stream.cpp:151] Stream queue full. Dropping frame: 197
I0915 11:54:26.797819  6921 throttler.cpp:60] Frame rate too high. Dropping frame: 198
W0915 11:54:26.844909  6921 stream.cpp:151] Stream queue full. Dropping frame: 199
W0915 11:54:26.879650  6921 stream.cpp:151] Stream queue full. Dropping frame: 200
W0915 11:54:26.929898  6921 stream.cpp:151] Stream queue full. Dropping frame: 201
I0915 11:54:26.957720  6921 throttler.cpp:60] Frame rate too high. Dropping frame: 202
W0915 11:54:26.996174  6921 stream.cpp:151] Stream queue full. Dropping frame: 203
W0915 11:54:27.049743  6921 stream.cpp:151] Stream queue full. Dropping frame: 204
I0915 11:54:27.078083  6921 throttler.cpp:60] Frame rate too high. Dropping frame: 205
W0915 11:54:27.116828  6921 stream.cpp:151] Stream queue full. Dropping frame: 206
W0915 11:54:27.170799  6921 stream.cpp:151] Stream queue full. Dropping frame: 207
I0915 11:54:27.180596  6922 cvsdk_ssd_detector.cpp:53] CVSDKSsdDetector initialized
I0915 11:54:27.203244  6921 throttler.cpp:60] Frame rate too high. Dropping frame: 208
I0915 11:54:27.234066  6922 cvsdk_ssd_detector.cpp:118] Only 85 proposals found
I0915 11:54:27.235921  6922 object_detector.cpp:225] Object detection took 55.275 ms
I0915 11:54:27.289513  6922 cvsdk_ssd_detector.cpp:118] Only 82 proposals found
I0915 11:54:27.290237  6922 object_detector.cpp:225] Object detection took 54.125 ms
I0915 11:54:27.326926  6921 throttler.cpp:60] Frame rate too high. Dropping frame: 211
I0915 11:54:27.346698  6922 cvsdk_ssd_detector.cpp:118] Only 83 proposals found
I0915 11:54:27.347435  6922 object_detector.cpp:225] Object detection took 57.115 ms
I0915 11:54:27.399181  6922 cvsdk_ssd_detector.cpp:118] Only 83 proposals found
I0915 11:54:27.399544  6922 object_detector.cpp:225] Object detection took 52.039 ms
I0915 11:54:27.443581  6921 throttler.cpp:60] Frame rate too high. Dropping frame: 214
I0915 11:54:27.453964  6922 cvsdk_ssd_detector.cpp:118] Only 84 proposals found
I0915 11:54:27.454560  6922 object_detector.cpp:225] Object detection took 54.965 ms
I0915 11:54:27.507591  6922 cvsdk_ssd_detector.cpp:118] Only 84 proposals found
I0915 11:54:27.508298  6922 object_detector.cpp:225] Object detection took 53.685 ms
I0915 11:54:27.558902  6922 cvsdk_ssd_detector.cpp:118] Only 81 proposals found
I0915 11:54:27.559618  6922 object_detector.cpp:225] Object detection took 51.267 ms
I0915 11:54:27.563295  6921 throttler.cpp:60] Frame rate too high. Dropping frame: 217
I0915 11:54:27.611959  6922 cvsdk_ssd_detector.cpp:118] Only 83 proposals found
I0915 11:54:27.612848  6922 object_detector.cpp:225] Object detection took 53.138 ms
I0915 11:54:27.662087  6922 cvsdk_ssd_detector.cpp:118] Only 82 proposals found
I0915 11:54:27.662457  6922 object_detector.cpp:225] Object detection took 49.546 ms
I0915 11:54:27.687623  6921 throttler.cpp:60] Frame rate too high. Dropping frame: 220
I0915 11:54:27.718940  6922 cvsdk_ssd_detector.cpp:118] Only 83 proposals found
I0915 11:54:27.719678  6922 object_detector.cpp:225] Object detection took 57.167 ms
I0915 11:54:27.769197  6922 cvsdk_ssd_detector.cpp:118] Only 81 proposals found
I0915 11:54:27.769562  6922 object_detector.cpp:225] Object detection took 49.814 ms
I0915 11:54:27.803628  6921 throttler.cpp:60] Frame rate too high. Dropping frame: 223
I0915 11:54:27.820646  6922 cvsdk_ssd_detector.cpp:118] Only 85 proposals found
I0915 11:54:27.821051  6922 object_detector.cpp:225] Object detection took 51.433 ms
I0915 11:54:27.870065  6922 cvsdk_ssd_detector.cpp:118] Only 77 proposals found
I0915 11:54:27.870370  6922 object_detector.cpp:225] Object detection took 49.242 ms
I0915 11:54:27.920454  6921 throttler.cpp:60] Frame rate too high. Dropping frame: 226
I0915 11:54:27.921638  6922 cvsdk_ssd_detector.cpp:118] Only 82 proposals found
I0915 11:54:27.921936  6922 object_detector.cpp:225] Object detection took 51.515 ms
I0915 11:54:27.971905  6922 cvsdk_ssd_detector.cpp:118] Only 86 proposals found
I0915 11:54:27.972249  6922 object_detector.cpp:225] Object detection took 50.28 ms
I0915 11:54:28.021682  6922 cvsdk_ssd_detector.cpp:118] Only 84 proposals found
I0915 11:54:28.022091  6922 object_detector.cpp:225] Object detection took 49.816 ms
I0915 11:54:28.045123  6921 throttler.cpp:60] Frame rate too high. Dropping frame: 229
I0915 11:54:28.078054  6922 cvsdk_ssd_detector.cpp:118] Only 82 proposals found
I0915 11:54:28.079907  6922 object_detector.cpp:225] Object detection took 57.759 ms
I0915 11:54:28.129251  6922 cvsdk_ssd_detector.cpp:118] Only 77 proposals found
I0915 11:54:28.130342  6922 object_detector.cpp:225] Object detection took 50.323 ms
I0915 11:54:28.164475  6921 throttler.cpp:60] Frame rate too high. Dropping frame: 232
ubuntu@ubuntu-OptiPlex-7050:~/saf$ uname -a
Linux ubuntu-OptiPlex-7050 4.14.20-041420-generic #201802162247 SMP Fri Feb 16 22:48:07 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux
ubuntu@ubuntu-OptiPlex-7050:~/saf$ cat /proc/cpuinfo
processor	: 0
vendor_id	: GenuineIntel
cpu family	: 6
model		: 94
model name	: Intel(R) Core(TM) i7-6700 CPU @ 3.40GHz
stepping	: 3
microcode	: 0xc2
cpu MHz		: 1600.071
cache size	: 8192 KB
physical id	: 0
siblings	: 8
core id		: 0
cpu cores	: 4
apicid		: 0
initial apicid	: 0
fpu		: yes
fpu_exception	: yes
cpuid level	: 22
wp		: yes
flags		: fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush dts acpi mmx fxsr sse sse2 ss ht tm pbe syscall nx pdpe1gb rdtscp lm constant_tsc art arch_perfmon pebs bts rep_good nopl xtopology nonstop_tsc cpuid aperfmperf tsc_known_freq pni pclmulqdq dtes64 monitor ds_cpl vmx smx est tm2 ssse3 sdbg fma cx16 xtpr pdcm pcid sse4_1 sse4_2 x2apic movbe popcnt tsc_deadline_timer aes xsave avx f16c rdrand lahf_lm abm 3dnowprefetch cpuid_fault epb invpcid_single pti tpr_shadow vnmi flexpriority ept vpid fsgsbase tsc_adjust bmi1 hle avx2 smep bmi2 erms invpcid rtm mpx rdseed adx smap clflushopt intel_pt xsaveopt xsavec xgetbv1 xsaves dtherm ida arat pln pts hwp hwp_notify hwp_act_window hwp_epp
bugs		: cpu_meltdown spectre_v1 spectre_v2
bogomips	: 6816.00
clflush size	: 64
cache_alignment	: 64
address sizes	: 39 bits physical, 48 bits virtual
power management: