feat(MatMenu): option to open menu instantly

Feature request
The menu should (optionally) open without the animation (like stated in the Material Design Spec:
https://material.io/design/components/menus.html#behavior -> Motion Section)
What is the expected behavior?

Desktop menus have the option to open instantly

What is the current behavior?
Menu opens with (kinda slow) animation.
What is the use-case or motivation for changing an existing behavior?
Alignment with the Material Design Spec and faster user interaction on Desktop Browsers
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
6.4.7
Is there anything else we should know?

In 7.0 the menu will have a much quicker animation that is in-line with the latest Material Design spec.

Thats good to hear but there would be no way for skipping the animation altogether just for the MatMenu?

You should be able to use the @.disabled binding which lets you disable animations on a particular section of the DOM.

@DevVersion Can #12829/#12839 be expanded to include MatMenu in respecting @.disabled?

@josephperrott Actually @.disabled should work properly. I guess it's related to the fact that the menu overlay animates in the cdk-overlay-container that is most likely not a child of the @.disabled element.
We need to come with a better solution for such things. Unfortunately there is no easy way to detect the @.disabled state of an element because it's part of the custom animations renderer. But there might be possibilities. I will have another look.

Any news about it ? I need it too :)

I'm also looking for an option to disable animation for MatMenu only. It had terrible performance issue on MS Edge.

I have observed change detection triggers on mouse hover on matMenu or any menu in matMenu, could be one of the reason for slowness.

This looks related to https://github.com/angular/components/issues/18340. As stated in my initial comment, we need to come up with a better solution for these overlay based components.
@crisbeto sent a proposal for dialog to allow configuration of enter and exit duration. That might could work for the bottom-sheet and menu too. I'm hoping we can find a consistent solution that works for all overlay-based components.

We would also appreciate this option. Users are spending more time waiting for the menu to appear than making a selection.

we're considering using another menu library because the animation feels too toy-like and distracting, and doesn't suit a more mature, enterprise audience.

use the cdk connected overlay

use the cdk connected overlay

To disable the animation in the mat menu, add the following CSS.
.ng-animating, .ng-trigger-transformMenu{
    animation: none !important;
    transition: none !important;
    transform-origin: center center !important;
    animation-duration: 0s !important;
    transform:  none !important;
    opacity: 1 !important;
    pointer-events: all !important;
}

Please note: this will affect all components.


Please set up the launch configuration file for your application

VSCode Version:1.2.0
OS Version:Windows 10

Steps to Reproduce:

I updated VSCode to 1.2.0
When i debug a nodejs app and set config, but VSCode still tell me to set up config

I just updated today and am getting the same thing.

I install back to 1.1.1, everything is ok.

updated 1.2, same thing

Just open the launch.json or select a launch config from the drop down menu and problem disappears.

@weinand Which dropdown menu? having the launch.json file open in the editor or not doesn't seem to have any effect.

@ghurlman the launch config dropdown:

@weinand Thanks! I'd only ever launched via F5, so I totally missed that.

Re-choosing "Launch" for my NodeJS app (Launch was already specified in the dropdown) addressed this issue for me. It's like the default option for the dropdown isn't getting set in the debugger or something.

@JoBrad yes, your analysis is correct. Sorry for the inconvenience.

Thanks for the fix! Was really bugging me :)

@weinand Thanks

For the recovery vscode build we have decided on a simple solution - open the debug viewlet when the launch.json is generated for the first time. I understand this changes how the UI behaves, but making the vielwet more discoverable on first setting up debug is not that bad IMHO.
Later in June we will look to address this feature without changing the UI


Expand append(contentsOf:) benchmarks

In prep for further refactoring for FIXME ABI #13 (combining the Sequence and Collection versions of append)
Also bumped inner loop count to increase run times slightly to avoid jitter and make marginal changes clearer.

@swift-ci Please smoke test

@swift-ci please benchmark

Benchmarks look OK in build log, but will fail to post here due to a bug doing the old/new comparison when adding new benchmarks.

Build comment file:
Optimized (O)


Identify device type (or mode type)

added the int returned from getCurrentModeType to DeviceInfo for determining what kind of device is running Branch (Phone/TV/Watch/etc)

Test steps:
Create emulator devices for watch and TV from Android Studio. Run testbed from Android Studio with logging enabled and ensure that v1/open and or v1/install are including the correct "ui_mode"
Expected results when Testbed launches:
Mobile Device: UI_MODE_TYPE_NORMAL
TV: UI_MODE_TYPE_TELEVISION
Watch: UI_MODE_TYPE_WATCH
example payload of v1/install from a mobile device: https://gist.github.com/EvangelosG/1c5d73fd1e533b62b1c4c2ea336410c0


rename PGoType inferFrom to make clear which language is being parsed

This bug is irrelevant as of e2a09aec8ae706313ddc28607b6779f1d121f51e; the function no longer exists.


Package Builder

Package Builder Todo List:

[ ] phpstan compat by using (and replacing) placeholders
[ ] rename to pgkname-blade-component
[ ] pgkname-livewire-component
[ ] assets - js mit ausgabe
[ ] assets - css mit sichbarkeit
[ ] tests für komponenten + assets
[ ] https://github.com/usetall/tallui/tree/dev/_components/tallui-app-components/.github/ISSUE_TEMPLATE - in all packages as well
[ ] Finish READMEs - app-components readme, then the rest, add relevant things to builder
[ ] Test using TalluiPackageServiceProvider from Core (use one existing package)
[ ] Add Icons from https://github.com/blade-ui-kit/blade-icons-template to CorePackageProvider, Add Test icons
[ ] Bring in all features from https://github.com/spatie/laravel-package-tools
[ ] Extend with all missing features
[ ] Update Package Builder
[ ] Rebuild all existing packages
[ ] Add docs from https://github.com/spatie/laravel-package-tools and https://github.com/blade-ui-kit/blade-icons

Features
All

[x] ServiceProvider
[x] Configs in /config

Iconset

[ ] Icons

Database (Full, Data, CRUD)

[ ] Models
[ ] Migrations
[ ] Collections
[ ] Casts
[ ] Seedings
[ ] Translatables
[ ] Factory
[ ] Resources

Components

[ ] Blade
[ ] Livewire

Views

[ ] Blade
[ ] Livewire
[ ] ViewComposer

Themes

[ ] Website
[ ] Admin

Others

[ ] Controllers - Path /src/controller -
[ ] Facades
[ ] Commands in /src/commands
[ ] InstallCommand
[ ] Routes
[ ] Components
[ ] Assets
[ ] Translations
[ ] Traits
[ ] Docs
[ ] Middlewares
[ ] Broadcast
[ ] Notifications
[ ] Mails
[ ] Exceptions in /src/exceptions
[ ] Jobs
[ ] API Connections
[ ] Contracts
[ ] Sorting
[ ] Filter
[ ] Listing

Tests

[ ] Unit
[ ] Feature
[ ] Surface

Actions

[ ] Test
[ ] Analyze
[ ] Stylefix

Handled by Spatie Package Tools
         ->name('your-package-name')
            ->hasConfigFile()
            ->hasViews()
            ->hasViewComponent('spatie', Alert::class)
            ->hasViewComposer('*', MyViewComposer::class)
            ->sharesDataWithAllViews('downloads', 3)
            ->hasTranslations()
            ->hasAssets()
            ->publishesServiceProvider('MyProviderName')
            ->hasRoute('web')
            ->hasMigration('create_package_tables')
            ->hasCommand(YourCoolPackageCommand::class)
            ->hasInstallCommand(function(InstallCommand $command) {
                $command
                    ->publishConfigFile()
                    ->publishMigrations()
                    ->copyAndRegisterServiceProviderInApp()
                    ->askToStarRepoOnGitHub();
            });

These presets should be available:

[ ] 1. Full Package - all features
[ ] 2. Components - Blade and Livewire Components
[ ] 3. Iconset - Blade SVG Icons');
[ ] 4. Theme - TailwindCSS Config, Views
[ ] 5. Module - Admin Panel Module
[ ] 6. Plugin - Website Plugin
[ ] 7. Crud - Listing with CRUD-features
[ ] 8. Block - Editor Blocks
[ ] 9. Dataset - Static datasets

Has Components?

Blade Components
Livewire Components

Loads Assets?
Has Translations?

10 most popular
all 100

Has Migration?
Has Seeding?
Has Routes?
Has Middleware? OR Has Theme?
Has Facade?
All

Config

Components
Full

Config
Controller
Facade
Command
InstallCommand
Route
Component

Blade
Livewire

View

Blade
Livewire
ViewComposer

Asset
Theme

Website
Admin

Database

Model
Migration
Seeding

Demo
Static
Product

Translatable

Translation
Trait
Icon
Docs
Middleware
(Notification)
(Exception)
(Job)

            ->name('your-package-name')
            ->hasConfigFile()
            ->hasViews()
            ->hasViewComponent('spatie', Alert::class)
            ->hasViewComposer('*', MyViewComposer::class)
            ->sharesDataWithAllViews('downloads', 3)
            ->hasTranslations()
            ->hasAssets()
            ->publishesServiceProvider('MyProviderName')
            ->hasRoute('web')
            ->hasMigration('create_package_tables')
            ->hasCommand(YourCoolPackageCommand::class)
            ->hasInstallCommand(function(InstallCommand $command) {

         ->name('your-package-name')
            ->hasConfigFile()
            ->hasViews()
            ->hasViewComponent('spatie', Alert::class)
            ->hasViewComposer('*', MyViewComposer::class)
            ->sharesDataWithAllViews('downloads', 3)
            ->hasTranslations()
            ->hasAssets()
            ->publishesServiceProvider('MyProviderName')
            ->hasRoute('web')
            ->hasMigration('create_package_tables')
            ->hasCommand(YourCoolPackageCommand::class)
            ->hasInstallCommand(function(InstallCommand $command) {
                $command
                    ->publishConfigFile()
                    ->publishMigrations()
                    ->copyAndRegisterServiceProviderInApp()
                    ->askToStarRepoOnGitHub();
            });

Files
Todos:

Test using TalluiPackageServiceProvider from Core (use one existing package)

Add Test Icons to CorePackageProvider

Finish and Test Icons

Bring in all features from https://github.com/spatie/laravel-package-tools

Extend with own features

Bring all features to Package Builder

Rebuild all existing packages

We skip this for now ... builder is ready, icons builder is near...


asd

The build for this pull request was not successful. Please review the logs for more details AND please review our contributing guidelines.


Selecting substructure and moving it for 3D Tiler

Should also include ability to copy and paste the substructure.

Move functionality can be generated by copying and pasting the current configuration while deleting the old.


Mystery Seeds

This didn't prevent my completion of the seed, but I thought I should bring up that I was able to get mystery seeds before I think I was intended to in logic from Frypolar's room in D8 from the respawnig grass.  doesn't really matter too much, but you might want to consider it in logic. (This is in seasons btw)

It's known but never required in normal difficulty. Hard difficulty sometimes requires it.


Hide some generated Razor page properties in the debugger

We're improving the ASP.NET Core debugging experience. One area that can improve is debugging Razor pages.
A generated Razor pages add some properties:
https://github.com/dotnet/razor/blob/dd8b2717ea1c0707cbeb9cbdb4807d8395229705/src/Compiler/Microsoft.AspNetCore.Mvc.Razor.Extensions/test/TestFiles/IntegrationTests/CodeGenerationIntegrationTest/ViewWithNamespace_Runtime.codegen.cs#L28-L47
These don't provide useful information in the debugger and clutter the debug view for the page. We want to limit the debug view to useful information such as the page model, properties added to the page by the developer, and request information.
Because this properties are added in generated pages, the Razor page generator needs to place [DebuggerBrowsable(DebuggerBrowsableState.Never)] on the properties.
These should be hidden in Razor pages and MVC views.

This is what a Razor page looks like in the debugger. You can see the properties I mentioned above add to the clutter:

Because this properties are added in generated pages, the Razor page generator needs to place [DebuggerBrowsable(DebuggerBrowsableState.Never)] on the properties.

Don't think this is the best approach. While not useful in most scenarios I'm sure that certain segments of customers do find them useful. This property effectively hides them in all cases.

Discussion about that here: https://github.com/dotnet/aspnetcore/pull/49353#discussion_r1261049746
Possible solution is to change auto properties to have private fields that can still be viewed in debugger.

@JamesNK did we come to a consensus here?
//cc @danroth27

I don't think there is a good way to do this. Closing for now.


Does onelogin.saml2.debug really do anything?

Maybe I'm missing something obvious, but I don't see where onelogin.saml2.debug actually does anything within the core code. I thought this was controlling the logging level, but that doesn't appear to be the case.
In tracing the code, it gets mapped to the private Saml2Settings.debug variable, which makes Saml2Settings.isDebugActive() return true or false based on the setting, but I don't see any of the code that actual does anything with this information.
The only place it used is in the acs.jsp to determine how to show errors.
Is that the only thing it's used for?

Is an internal setting of the toolkit.
And the way to use it is with the isDebugActive as you saw on acs.jsp as a way decide when to show (for debugging) the reason for SAML Messages invalidation.

Thanks for the information.
When I saw the setting in the INI file, it felt like it was controlling the logging output. It might be worth noting in the INI file the debug setting does not affect logging and is simple a boolean to affect your own implementation of the API.

Sorry, is there something that I need to do, in addition to this to get the debug output?
Enable debug mode (to print errors)
onelogin.saml2.debug =  true
I'm running the SAML Java examples in Apache Tomcat 9.0.98 and I receive the following error.  When I make the config change above, restart Tomcat, reproduce the issue, I see no additional debug in any logs?
com.onelogin.saml2.exception.ValidationError: SAML Response could not be processed
com.onelogin.saml2.authn.SamlResponse.loadXmlFromBase64(SamlResponse.java:160)
com.onelogin.saml2.authn.SamlResponse.(SamlResponse.java:118)
com.onelogin.saml2.authn.SamlResponse.(SamlResponse.java:139)
com.onelogin.saml2.factory.SamlMessageFactory.createSamlResponse(SamlMessageFactory.java:55)
com.onelogin.saml2.Auth.processResponse(Auth.java:1204)
com.onelogin.saml2.Auth.processResponse(Auth.java:1253)
org.apache.jsp.acs_jsp._jspService(acs_jsp.java:160)
org.apache.jasper.runtime.HttpJspBase.service(HttpJspBase.java:67)
javax.servlet.http.HttpServlet.service(HttpServlet.java:623)
org.apache.jasper.servlet.JspServletWrapper.service(JspServletWrapper.java:466)
org.apache.jasper.servlet.JspServlet.serviceJspFile(JspServlet.java:376)
org.apache.jasper.servlet.JspServlet.service(JspServlet.java:324)
javax.servlet.http.HttpServlet.service(HttpServlet.java:623)
org.apache.tomcat.websocket.server.WsFilter.doFilter(WsFilter.java:51)
Thanks!

@tylerrstevens I believe you want to configure the "com.onelogin" logger in whatever logging framework you use. For example, this is my configuration for logback:
<logger name="com.onelogin" level="trace" />


More empirical datasets

[ ] Ebola data from epidist?
[ ] Covid data from scoringutils?
[ ] Others?

outbreaks?

Hmm yeah that's probably the right answer here.


Peer2Peer_CodeReview 5th Nov 2021

Good job so far @williamrolando88! Nevertheless there a little things to have in mind.

Is really good that you have a good idea of how to structure a page using the adequate html tags for each part of the page.

Although is a good thing that the majority of your code in the css file is good structured as well your index file, It would be a great idea that you, position all the changes of default behaviours for native tags in the begging of the file or each section of media queries.
For example:
https://github.com/williamrolando88/personal-portfolio/blob/6b956d72be79189fccf53419ef4e6d25d2dffb85/style.css#L135-L150
In that part you may change that id behaviour at the last part of the changes to default ones.

Hi William,
Great work so far!  You have done a great job implementing previous advice given to you.
One thing you could look at. I understand why you have left the a tags open below, but is it necessary for the I tag to also be open?  Perhaps this is intentional and maybe it is not a problem :)
https://github.com/williamrolando88/personal-portfolio/blob/6b956d72be79189fccf53419ef4e6d25d2dffb85/index.html#L49-L55


apply env to array elements

This fixes #348.
In particular:
KAPACITOR_INFLUXDB_0_URLS_0=http://localhost:9000 kapacitord config
should now produce a config which does the expected thing.

@jonseymour Thanks for the PR, I know I don't have any tests around config parsing, but could you add a test that demonstrates the new behavior? Don't worry about testing everything else just the changes. We will get to the other tests in time.

Ok, I'll add tests. I'll have to abstract the call to os.GetEnv() to allow tests to mock this call, as required.

@nathanielc I've extended a test and done a more complete reorganisation to simplify the structure of the applyEnvOverrides function.

Actually, I missed an additional simplifcation that is now possible. I'll re-roll.

@jonseymour Just to make sure, you made that last simplification correct? Also can you add a changelog entry? Otherwise this looks good to go. Thanks!

Yes, simplifications complete. Changelog now updated.

Nice, this should remove sed magic from my Docker image, thanks!


implement xarray/netcdf following stac

See:

https://github.com/stac-extensions/xarray-assets
https://github.com/stac-extensions/file

cc: @francesconazzaro


Use the config from current branch instead of default branch

The current implementation uses the configuration file from master branch instead of the active branch. This is not really aligned with how github actions config works in general.
If a user wants to use this plugin for the first time, he needs

create a pull request and add the configuration file (.github/auto_request_review.yml) into master without testing it.
create a second pull request to add the workflow and test the config file from step 1.

If we use the current branch instead, the two steps above can be merged into a single step: both the config file and workflow can be added by one Pull Request.
P.S. @necojackarc the error in the build checks seem to be related to permission. Unfortunately there is nothing I can do about it. Please let me know if I should bump the version in package.json within this pull request. Many thanks for making this action!

@hex0cter Thanks for you pull request!

If we use the current branch instead, the two steps above can be merged into a single step

Yeah, I think that's a fair point. The reason why that's implemented in that way is that I'm a bit worried about the situation where you can be removed from reviewers without you knowing it if you use the current branch. If you wipe out all reviewers in a pull request, no one will be assigned as reviewers.
But I feel like I'm trying to address some unlikely event at the sacrifice of convenience and some sort of naturalness.
Do you have any thoughts on this point? I'll sleep on it tonight and make a decision but any thoughts are welcome!

the error in the build checks seem to be related to permission

Yeah, I'm not yet very familiar with what secrets.GITHUB_TOKEN is but it looks like the default token of the author of PR, so it doesn't have necessary write access to this repository. I think I should make a token with the bare minimum permissions to assign reviewers.

I added a ticket for the token permission issue - #35

@necojackarc Thanks for sharing the thoughts. You have a fair point as well. 😄
I think the only drawback is whenever someone updates the configuration file (.github/auto_request_review.yml), there is no way to test it before merging it.
If you decide to keep the current implementation, it would be good to mention the idea behind it in the documentation so people don't have to debug it in the future.  🥰

@hex0cter Thanks for your feedback. I'll be merging this PR as-is since such an event that I'm worried about is quite rare and probably, I should trust people a bit more 😄


Fuzzer failure for might_contain Spark function

Bug description
Spark fuzzer fails consistently: https://app.circleci.com/pipelines/github/facebookincubator/velox/23944/workflows/e8a5f992-094e-4eb2-abff-9819b181490e/jobs/152090
I20230427 10:24:21.481151 303514 ExpressionFuzzer.cpp:1138] ==============================> Started iteration 9512 (seed: 827068474)
I20230427 10:24:21.482075 303514 ExpressionVerifier.cpp:88] Executing expression: greaterthan(might_contain(RAJt=p8%,"c0"),between(abs(exp(round(power(multiply(unaryminus("c1"),"c2"),0.576674263458699),"c3"))),0.8262873732019216,multiply(0.8124061767011881,add(divide(null,"c4"),divide(multiply(round(0.6955040397588164,"c5"),"c6"),"c7")))))
I20230427 10:24:21.482115 303514 ExpressionVerifier.cpp:31] 8 vectors as input:
I20230427 10:24:21.482127 303514 ExpressionVerifier.cpp:33] 	[DICTIONARY BIGINT: 100 elements, 11 nulls], [DICTIONARY BIGINT: 100 elements, 10 nulls], [FLAT BIGINT: 100 elements, 13 nulls]
I20230427 10:24:21.482142 303514 ExpressionVerifier.cpp:33] 	[DICTIONARY DOUBLE: 100 elements, 10 nulls], [FLAT DOUBLE: 100 elements, 9 nulls]
I20230427 10:24:21.482153 303514 ExpressionVerifier.cpp:33] 	[FLAT DOUBLE: 100 elements, 7 nulls]
I20230427 10:24:21.482162 303514 ExpressionVerifier.cpp:33] 	[DICTIONARY INTEGER: 100 elements, 13 nulls], [FLAT INTEGER: 100 elements, 11 nulls]
I20230427 10:24:21.482173 303514 ExpressionVerifier.cpp:33] 	[DICTIONARY DOUBLE: 100 elements, 10 nulls], [FLAT DOUBLE: 100 elements, 12 nulls]
I20230427 10:24:21.482184 303514 ExpressionVerifier.cpp:33] 	[DICTIONARY INTEGER: 100 elements, 12 nulls], [FLAT INTEGER: 100 elements, 7 nulls]
I20230427 10:24:21.482195 303514 ExpressionVerifier.cpp:33] 	[FLAT DOUBLE: 100 elements, 14 nulls]
I20230427 10:24:21.482204 303514 ExpressionVerifier.cpp:33] 	[FLAT DOUBLE: 100 elements, 9 nulls]
E20230427 10:24:21.484517 303514 Exceptions.h:68] Line: ../.././velox/common/base/BloomFilter.h:67, Function:merge, Expression: kBloomFilterV1 == version (1 vs. 82), Source: USER, ErrorCode: INVALID_ARGUMENT
E20230427 10:24:21.486140 303514 Exceptions.h:68] Line: ../../velox/functions/sparksql/MightContain.cpp:40, Function:apply, Expression: serialized->isConstantMapping() , Source: USER, ErrorCode: INVALID_ARGUMENT
E20230427 10:24:21.486294 303514 FuzzerToolkit.cpp:93] Only simplified path threw exception:

System information
n/a
Relevant logs
No response

@jinchengchenghh Would you help look into this? This failure happens currently on all PRs and prevents us from getting a clean signal.

CC: @rui-mo

As we discussed here, https://github.com/facebookincubator/velox/pull/4029#discussion_r1162195855
I change the exception to user check exception, and then the fuzzer tests passed.
The log indicates the exception is user exception, but it fails however.
Is this exception not correctly captured?
E20230427 10:24:21.486294 303514 FuzzerToolkit.cpp:93] Only simplified path threw exception:
I20230427 10:24:21.486917 303514 ExpressionVerifier.cpp:316] Persisted input: --fuzzer_repro_path /tmp/spark_fuzzer_repro/velox_expressionVerifier_antA4o --input_path /tmp/spark_fuzzer_repro/velox_expressionVerifier_antA4o/input_vector --sql_path /tmp/spark_fuzzer_repro/velox_expressionVerifier_antA4o/sql
terminate called after throwing an instance of 'facebook::velox::VeloxUserError'
  what():  Exception: VeloxUserError
Error Source: USER
Error Code: INVALID_ARGUMENT
Retriable: False
Expression: serialized->isConstantMapping()
Function: apply
File: ../../velox/functions/sparksql/MightContain.cpp
Line: 40
Stack trace:
Stack trace has been disabled.Use --velox_exception_user_stacktrace=true to enable it.

*** Aborted at<PHONE_NUMBER> (Unix time, try 'date -d @1682591061') ***
*** Signal 6 (SIGABRT) (0x4a19a) received by PID 303514 (pthread TID 0x7f21f020a2c0) (linux TID 303514) (maybe from PID 303514, UID 0) (code: -6), stack trace: ***
(error retrieving stack trace)
/bin/bash: line 8: 303514 Aborted

Should I change current
    VELOX_USER_CHECK(serialized->isConstantMapping())
    BloomFilter output{StlAllocator<uint64_t>(&allocator)};
    try {
      auto sv = serialized->valueAt<StringView>(0);
      output.merge(sv.data());
    } catch (const std::exception& e) {
      context.setErrors(rows, std::current_exception());
      return;
    }

to following which moves constantMapping check to try block?
    BloomFilter output{StlAllocator<uint64_t>(&allocator)};
    try {
 VELOX_USER_CHECK(serialized->isConstantMapping())
      auto sv = serialized->valueAt<StringView>(0);
      output.merge(sv.data());
    } catch (const std::exception& e) {
      context.setErrors(rows, std::current_exception());
      return;
    }

@jinchengchengh
yes that solve the problem we should not throw in the function body.

Glad to hear it could be solved. Please feel free to contact us if any other comment. Hope the spark fuzzer can recover soon.

I am taking over the issue its taking too long.

ok the problem in that is the function signature says that it can receive only constant inputs, the simplified path however goes and flatten the inputs.
the proposed solution is that eval simplified should not args  of functions that have the property in the signature that it only accepts constants.

@pedroerp  @mbasmanova  what do you think that?

https://www.internalfb.com/diff/D45420178

I think if we just completely remove  VELOX_USER_CHECK(serialized->isConstantMapping())
and then only checked that the function is initialized when constructed already with constant input
then we can solve this issue @jinchengchenghh

We discussed a plan to fix the underlying issue with constant parameters today. @laithsakka , could you share more details about it?

is this solved?

After this optimization https://github.com/facebookincubator/velox/pull/4728,  this issue has been solved, but the input argument should be serialized bloom filter, fuzzer test always fails for this function, so I think we can keep it in skip list.

Ideally we should have an API in the fuzzer that you could extend and provide the "fuzzed" input you would like to test your function (sort of fuzzed bloom filter binaries). We discussed about this in the past, but I'm not sure if this was already created. @kagamiori would know.

for this issue we discussed that during function resolution we shall make the binder only works with functions that expects constant null if the input expression is literal for those args positions, in that case we wont even need a runtime check
this is not high priority and is not blocking the fuzzer will move this to back log


Improve design for endpoint groups

Issue
https://gravitee.atlassian.net/browse/APIM-2398
Description
Delete was already implemented. I just changed the page design a bit to make it clearer
BEFORE

AFTER

Additional context
I added the "Add group" button, but no associated implementation. This will be done in another US.

📚  View the storybook of this branch here

@Okhelifi I don't think so. This is not a major change, and it will go with the capability of adding new groups.

@jourdiw good catch. I updated the screenshot. New design has been validated by @audrey-tlm


s/auth.py/auth.sh

fix typo in auth instructions, the described auth script is a shell script not a python one

Thanks @chilicuil ❤


Requires restarting ember server

Just had a few confusing minutes as a result of having the server already running. Restarting (not reload, the server reloads when the files change without the css being generated) is the only thing required to fix, but without restarting the SASS / SCSS never gets compiled.
Maybe just noting this in the README is good enough? Not even sure how much of that is available

That's certainly not the usual behaviour. I think it's unlikely that's caused by this module, probably another issue I'd say.

I've just confirmed that it is indeed reloading the CSS for me.

@stickwithjosh I experience a similar issue. The css gets build once after ember serve. Changes to a SASS file are recognised by ember-cli, and a rebuild is triggered. But actually the new css is not generated...The only thing that helps is to restart the server :(

Can I ask whether you're putting the .scss files into styles/ or into a pod folder?


Menu::renderItems() throws exception if active item's class is defined as an array.

What steps will reproduce the problem?
Create a menu with an active item having class defined as an array. Exception is also thrown if item is first or last.
echo \yii\widgets\Menu::widget([
    'activeCssClass' => 'item--active',
    'items' => [
        [
            'label' => 'Foo',
            'url' => '/foo',
            'active' => true,
            'options' => [
                'class' => [
                    'item',
                    'item--foo'
                ]
            ]
        ]
    ]
]);

What is the expected result?

Foo

What do you get instead?
PHP Notice – yii\base\ErrorException
Array to string conversion
Additional info

Q
A

Yii version
<IP_ADDRESS>

PHP version
7.1.0

Operating system
macOS Sierra 10.12.3

@Pikseli are you sure it's supposed to work as you expected? Could you please provide a reference to the docs?

It's currently no supported. Probably worth implementing.

@Pikseli seems like it's not supposed to work as you expect it. Should not throw exception though.

options: array, optional, the HTML attributes for the menu container tag.

Yes. That would be consistent with Html::tag() with class option. Should be applied consistently through framework.


Unifying the Reporting API: without RaceBuffer for now

Closes #179
Based on #185 , but replaces the RaceBuffer with a basic ring buffer to get us by until the RaceBuffer work settles.

:+1:


Are you alive?

I hope so.
We have some important changes waiting in pull requests. Indeed, are 8 pull requests waiting for a long time. We are making a lot of improvements in this code but we simply can't contribute in this repository because it is abandoned.
That way, we have our fork miles away from the original repo. We don't like it. Fragmenting this piece of code is not the best way of keep it moving, in my view.
I hope you can add permissions to us (maybe some others contributors too) in this repo, that way, we can review, merge and continue to improve this framework. Otherwise, we have to mantain a fork of the same framework in two very different states.
Best regards.

:+1:


Create Chinese Sentence Creator.md

📝 Description
Add a brief description.
💣 Is this a breaking change (Yes/No):

[ ] Yes
[x] No

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

sorry, i didn't realize i commited to the main branch instead of the hacktoberfest one


Convert ETH / Cash

Should be a way to convert back/forth between eth/cash.  Can't atm in the UI

Added in 4c6697aa54227967ad0924a211077aa4930b8473.


Add vscodium to the editor list

This PR adds support for opening vscodium on error views
tested on macOS 10.12.6 + VSCodium 1.32.3

bump?

Thanks!


Suggestion for clarification w.r.t using a discard to suppress an exception

Hi, I'm Rex Jaeschke, editor of the C# Standard, under contract to MS. I was confused by what this meant, so I asked Mads Torgersen of the C# Team @ MS to help clarify. Here's what he sent to me:
"No, discards have no exception-suppressing capabilities! Here’s what happens in the example:

We Task.Run a lambda that ends by throwing an exception
The result of the Task is therefore the exception
If you await the exception, then that throws the exception in your code
To deliberately avoid that exception, you might want to simply drop the Task on the floor and not await it
But if you do, the compiler helpfully warns you saying hey, didn’t you forget to deal with this task?
To show the compiler that you’re dealing with the Task, all you have to do is use it somehow, e.g. by assigning it to a variable.
But now you have this dummy variable containing a Task that you specifically don’t want to touch again.
Better to use a discard!

You might want to borrow some of his words to help improve this page.
Rex
<EMAIL_ADDRESS>
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 8047b417-55af-c6df-9333-f1e1c428e594
Version Independent ID: 8bcc16bb-c2d8-0b6d-af20-6958964efe9e
Content: Discards - C# Guide
Content Source: docs/csharp/discards.md
Product: dotnet-csharp
Technology: csharp-fundamentals
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

Thanks @RexJaeschke
I'll add this to our list to update.

When this is fixed, explain that the behavior exhibited is different when a debugger is attached. (With the debugger, the debugger forces the app to break when the exception is thrown and not caught. It will be discarded without a debugger attached).
/cc @kendaleiv

When this is fixed, explain that the behavior exhibited is different when a debugger is attached. (With the debugger, the debugger forces the app to break when the exception is thrown and not caught. It will be discarded without a debugger attached).
/cc @kendaleiv


Console blows up with errors when hovering over a gravatar

When I hover my mouse over a gravatar and move it around it causes console errors to repeatedly get thrown.

Debugging into this exception I see the following:

As you can see here the has function has been overwritten with the value true. I believe it is getting overwritten in the has-gravatar component https://github.com/johnotander/ember-cli-gravatar/blob/605f5d61298882a513911dc01a85d925d4acdb89/addon/components/has-gravatar.js#L21.
I plan to rename the has property to something different and see if it fixes the issue I'm seeing, but I haven't completely ruled out the possibility that it's not some other addon or something, just seems to all make sense that it would be what I'm describing here.
I will keep you updated, just wanted to raise awareness and also get a reading of if it is important that the property be named has and not hasGravatar or something else. Maybe @AO16 has some insight to that?
Thanks!

@jimmay5469 Yea, I think you are on the right track. If you change it from has to hasGravatar I think that should do the trick.

Ok, I just cloned, npm linked, and commented the referenced line and the has: false definition and am no longer getting the console errors. So the problem is indeed with this addon.
I will try to see if I can rename the property internally but continue yielding it as has so that it doesn't break the current API.

Alright, looks like this fixed it and I was able to keep the existing has yield in order to keep the same API. See the #52 PR.


Is it possible to use YOLOv7 ?

非常棒的工作！
我正在拼命学习你们的研究并且尝试将其应用于实际领域。
我想问的是，

output文件夹下的'input_2D'文件夹内是否已经是3D 人体姿势估计的结果（也就是你们论文中所描述的主要工作生成的结果）？因为我想将其用在优化虚拟人物动作中。
目前使用的是略显“古老”的YOLOv3和体积很大的HRNet来工作，是否有可能将其换成新的YOLOv7和Lite-HRNet，以加快识别姿态的速度，并最终实现实时识别和多目标同时识别？

如果可行的话，能否给我讲一下修改方法？例如如何生成“ YOLOv7.weights”文件，以及能否直接在github仓库中下载Lite-HRNet来替换使用？
请原谅我作为一个cv领域的初学者可能提问过于小白，感谢您的耐心
Great Great Job!
I'm trying my best to study your research and try to apply it in real world.
What I want to ask is,

Is the 'input_2D' folder that in the ‘output’ folder the result of 3D human pose estimation(that is, the main work in your paper)? Because I want to use it in optimizing avatar motivation.
Currently working with a slightly "old" YOLOv3 and a large HRNet, is it possible to replace it with the new YOLOv7 and Lite-HRNet to speed up the recognition of poses and finally achieve real-time recognition and multi-tasking? target at the same time?

If possible, can you tell me how to modify it? For example, how to generate the "YOLOv7.weights" file, and can I download Lite-HRNet directly in the github repository to replace it?
Please forgive me as a beginner in cv area for asking such primary questions, thank you for your patience.

工作量好大

'input_2D' is the 2D poses detected by HRNet.
The demo code is following "https://github.com/fabro66/GAST-Net-3DPoseEstimation", which uses YOLOv3 and HRNet.


[BUILD] Issues rpath-ing the OpenImageIO.cpython-38-darwin.so when running make release (Apple Silicon)

Hi!
I am trying to build OpenImageIO on a macOS with an M1 chip (running on macOS 12.3).
The actual build process goes fine, no issues at all.
When I try to make install though, it spits out this error:
[ 93%] Built target testtex
[100%] Built target PyOpenImageIO
Install the project...
-- Install configuration: "Release"
error: /Library/Developer/CommandLineTools/usr/bin/install_name_tool: for: /opt/opensource-builds/oiio/dist/lib/python3.8/site-packages/OpenImageIO/OpenImageIO.cpython-38-darwin.so (for architecture arm64) option "-add_rpath /opt/opensource-builds/oiio/dist/lib" would duplicate path, file already has LC_RPATH for: /opt/opensource-builds/oiio/dist/lib
error: /Library/Developer/CommandLineTools/usr/bin/install_name_tool: no LC_RPATH load command with path: /opt/opensource-builds/oiio/build/lib found in: /opt/opensource-builds/oiio/dist/lib/python3.8/site-packages/OpenImageIO/OpenImageIO.cpython-38-darwin.so (for architecture arm64), required for specified option "-delete_rpath /opt/opensource-builds/oiio/build/lib"

The fact that

option "-add_rpath /opt/opensource-builds/oiio/dist/lib" would duplicate path, file already has LC_RPATH

seems to be true, in fact since I can see that:
$ otool -l  /opt/opensource-builds/oiio/dist/lib/python3.8/site-packages/OpenImageIO/OpenImageIO.cpython-38-darwin.so | grep -B 1 -A 3 RPATH
Load command 16
          cmd LC_RPATH
      cmdsize 32
         path /opt/homebrew/lib (offset 12)
Load command 17
--
Load command 20
          cmd LC_RPATH
      cmdsize 56
         path /opt/opensource-builds/oiio/dist/lib (offset 12)

Not sure if it's linked to it, but maybe this PR offers some context: https://github.com/OpenImageIO/oiio/pull/1671 ?
After looking at that PR, I tried commenting the set (MACOSX_RPATH ON) line, and make install went fine (no errors).
I'm not an expert of RPATHs on macOS, but it seems that the RPATH is already being set at compile time, so trying to re-set it at install time fails.
My diff is currently the following:
$ git diff
diff --git a/src/cmake/compiler.cmake b/src/cmake/compiler.cmake
index 7ba971a1..bba0e18a 100644
--- a/src/cmake/compiler.cmake
+++ b/src/cmake/compiler.cmake
@@ -608,7 +608,7 @@ endif ()
 ###########################################################################
 # Rpath handling at the install step
 #
-set (MACOSX_RPATH ON)
+#set (MACOSX_RPATH ON)
 if (CMAKE_SKIP_RPATH)
     # We need to disallow the user from truly setting CMAKE_SKIP_RPATH, since
     # we want to run the generated executables from the build tree in order to

Thanks!
Valerio

So basically you are saying that you think we should alter this one line in compiler.cmake to NOT enable the MACOS_RPATH variable?
I'm very much not an expert on its effect.
This line was originally added in #1671 (this section of code was in "install.cmake" at the time, much later transplanted to compile.cmake). This dates from 2017, and although I seem to be the author, I don't remember anything about why I did it.
We can certainly try removing it, and as long as it builds for me and passes CI, if that helps you, I don't see why we shouldn't do it.

Hi @lgritz !
More or less - from my tests it seemed redundant. Although I'm not really an expert either - I just examined the compiler warnings and checked what was going on with/without this line. If you have a CI for both ARM and x86 macOS builds, it might be worth trying to remove it and see if things still build fine for both architectures (and checking with which cmake version). Sadly I can't provide much more rational reasoning for the why.
Googling around, I found some more context here:

https://cmake.org/cmake/help/latest/prop_tgt/MACOSX_RPATH.html
https://cmake.org/cmake/help/latest/policy/CMP0042.html#policy:CMP0042

Specifically, CMP0042 says that

MACOSX_RPATH is enabled by default

so maybe that's what's happening here..?

How about this: #3523
It passes CI and also works for my on my own Mac laptop.
But neither of those are ARM-based Macs.

That's great. I'm on a ARM machine (M1 MAX) so I can confirm that commenting works there too!
Another wild guess for the "why" could be that the rpath is already being set via the INSTALL_NAME_DIR variable instead? Quoting from https://cmake.org/cmake/help/latest/prop_tgt/MACOSX_RPATH.html :

When this property is set to TRUE, the directory portion of the install_name field of this shared library will be @rpath unless overridden by INSTALL_NAME_DIR.

Looking around, it seems that you're already setting this var?
$ grep -B 5 -e CMAKE_INSTALL_NAME_DIR --color site/spi/Makefile-bits
## Generic OSX machines (including LG's laptop)
else ifeq (${platform}, macosx)
    USE_SIMD ?= sse4.2
    MY_CMAKE_FLAGS += \
        -DCMAKE_BUILD_WITH_INSTALL_RPATH=1 \
        -DCMAKE_INSTALL_NAME_DIR="${working_dir}/dist/lib"

Anyway, if that change works for you guys, it definitely works for me too!
PS: I also just realized that Github's CI is not currently currently supporting building on Apple Silicon, but it's on their roadmap: https://github.com/github/roadmap/issues/528
(now, the other part of the macOS ARM-equation would be to have the conda-recipe building fine for ARM too, but that might be a lot of work, given how the conda recipes are actually written.. )


docs: Added Limitation for UploadFiles

Added section about limitations for UploadFiles.
Closes #1521

I do not know why, but aren’t the pending ones our old Actions from month ago?
Can you merge that pr?

Not sure. Can you cancel them and restart the one in this PR?

Not sure. Can you cancel them and restart the one in this PR?

Well there isn't anything to cancel. Those runs don't exist in the first place.
👌 Verify bUnit was the name before you refactored them to ci with its steps.
So I am not sure what I can cancel. I could force-push and retrigger the pipeline in the hope of not seeing those "ghost" actions.

Yeah, that's weird. It's just force complete.


is there any way to specify fabric distribution properties with this plugin?

I'm uploading a build to Crashlytics with the following command:
./gradlew assembleRelease crashlyticsUploadDistributionArmv7Release

As it's described here it's possible to specify distribution group and release notes:
ext.betaDistributionReleaseNotes=”Release Notes for this build.”
<EMAIL_ADDRESS><EMAIL_ADDRESS>ext.betaDistributionGroupAliases=”my-best-testers”

The only way to do that now is to create build-extras.gradle
If you need to customize build.gradle, rather than edit directly, you should create a sibling file named build-extras.gradle. This file will be included by the main build.gradle when present.

I upload my APKs to Crashlytics using the crashlytics-devtools.jar at the command line which supports groups and release notes.
You can get a list of all of the command line switches by using the --help switch:
$ java -jar resources/android/crashlytics-devtools.jar --help

Thanks. Is it Crashlytics Ant plugin? https://docs.fabric.io/android/crashlytics/ndk.html#crashlytics-additions-to-ant-projects

Yes, you can get the jar by downloading the Ant package from https://fabric.io/downloads/ant.
You don't actually have to use the Ant build scripts to use the jar though. I use the jar directly solely for uploading builds from the CLI.

I'm trying to use the crashlytics dev tools jar to upload a beta apk file to Fabric, but get an illegal argument exception that I'm missing a required parameter. What parameters are you using on the command line?


Footer does not stick to the bottom

To make the footer stick, I propose making the app to have a minimum height equal to the viewport height and the footer sticky with a top of the viewport height.
Using tailwind this can be achieved by adding the min-h-dvh class to app and  sticky with top-[100vh] to the footer.

Could you please implement this and I'll take a look?~ Everything works as intended on my side 🤔

Sure. Let me get on it.


PG::ConnectionBad: could not connect to server: No such file or directory

Hey,
while fetching upstream for the latest code and running the usual bin/update from the MiQ's core, I get stuck on the command bin/rails db:migrate; more specifically, this is the error I'm thrown when trying to migrate the database and appending additional --trace flag at the end of the command for DB migration:

As said, core is synced with upstream branch.
The suggestion says to check if the server is running locally on Unix domain socket /tmp/.s.PGSQL.5432. I don't have any server running on that address:

I'm running local ManageIQ system on OSX. Any suggestions how to get around this error?
Any help would be greatly appreciated,
Tjaz

Do you have PostgreSQL installed and running?  One common approach on MacOS is to use Homebrew (see https://daily-dev-tips.com/posts/installing-postgresql-on-a-mac-with-homebrew/)

If you have installed postgresql through homebrew, be sure to double check brew services list and see that postgresql is running there.

Closing waiting for a response.  If you have any more problems, please reopne.


Add feature to check the values in id_feature are unique

This app accpets several fields: https://github.com/GeoscienceAustralia/dea-conflux/blob/main/dea_conflux/main.py#L72-L82 as id_field.
If the values in the guess_id_field are not unique, the final output will be totally incorrect. E.g. a shapefile with all 0 saved in ORIG_FID.
We have to add one more step in guess_id_field. Once we guess one field, check the values are unique or not. If not, move to next guess. If they are unique, return the current field as id_field.

This method: https://github.com/GeoscienceAustralia/dea-conflux/commit/1c715b9a1233ab31bd3717a7ec01ec33d353dc42#diff-9ff241560a9a672755e77d3ab48607e36e895413b4ee92a5c6ae8d95a93c565dR55-R70 will check the values in guess_id_field are unique or not.
I also add one unit-test method: https://github.com/GeoscienceAustralia/dea-conflux/commit/1c715b9a1233ab31bd3717a7ec01ec33d353dc42#diff-41d180f6f7233d175c2dfe36c45c4ea80eed4754fa7ed4eed46ccde1b2a778c7R74-R77 and new test shapefile (the id_field column same_id has the same value) to test it.


Figure out how we can measure "understandability" of code

Hey,
After the last docs WG meeting we figured out we need a better way to measure how understandable the docs are and what newcomers find confusing.
I'd like to toss around ideas for how we can measure how understandable the docs are.

It might work well to partner with nodeschool on this to collect feedback from nodeschool students.

This inspired me to suggest a Target Audience index for Guides and Topics to help steer users to docs appropriate for their skill levels in the roadmap. https://github.com/nodejs/docs/issues/59#issuecomment-174051423
Any thoughts?

+1 on nodeschool hook. Maybe also discuss nodeschool's roadmap on this.

Hi, I contacted Dror Feitelson from the Hebrew University who researches the understandability of code. I'm posting here his remarks on this issue:
great question.
I'd follow Gilb on this.  Gilb's law states that "Anything you need to quantify can be measured in some way that is superior to not measuring it at all". specifically he suggests a methodology of components, scales and metrics:

identify components of the thing you want to quantify
for each component find the scale with which it can be measured
and then find mechanisms to actually measure it in this scale

in your case of software documentation understandability, possible components that come to mind are:

find which function to use
how to use a certain function
etc.

 This is parameterized by developer type being a novice, a casual contributor, a core member, etc., and tasks being correcting a usage bug, using the function in new code, etc.
the mechanism can be to actually test this at least 5 times with different users and actually measure the time it takes them.

/cc'ing @ashleygwilliams and @feross on this — I suspect the Foundation may be able to help out here. We'd likely want to run polls and possibly run recorded UX interviews (e.g., an interviewer asking a developer "Show me how you'd find out how to do XYZ with Node")

@chrisdickinson Nice. Thoughts on how to conduct these polls and UX interviews? If there's a concrete proposal from the WG then I think we can take it to the board. Also, if possible, we should try to seek funding/volunteers from the community first, since more community ownership is always better IMO :)

Closing as this repository is dormant and likely to be archived soon. If this is still an issue, feel free to open it as an issue on the main node repository.


use_icu_alternatives_on_android feature porting.

Porting from Lite-15 into Lite-17. Controlled by flag: use_icu_alternatives_on_android XWalkCoreShell.apk(ARM), reduced size=22.2 -15.7 = 6.5M

@PeterWangIntel  icu porting done.


Clean up metadata for features

Requesting that the feature names be standardized. Here's what we have now:
ECF CXF Client Distribution Provider Feature
ECF CXF JaxRC Distribution Provier
ECF Remote Services Jersey Server Provider
Jersey Remote Service Client Provider Feature
After looking at the features in the main Eclipse ECF repo, I would suggest something like this:
ECF Remote Services JAX-RS CXF Client Provider
ECF Remote Services JAX-RS CXF Server Provider
ECF Remote Services JAX-RS Jersey Client Provider
ECF Remote Services JAX-RS Jersey Server Provider

Fixed and released as version 1.13.2.  See releases

Closing.


[8.x] Set the factory namespace based on the name argument

I don't understand why assigning the namespace of a factory based on the model namespace when it should be assigned based on the name argument. For example,
I have a model in App\Models\Cat.
I want to create a factory for that model in the namespace Database\Factories\Cats\CatFactory
But if i run php artisan make:factory Cats/CatFactory the file was created correctly, but the factory namespace is still Database\Factories.
I think I am unaware of something about how this command works, however, this PR fix that.

I have no plans to modify this logic again. Make your model namespaces and factory namespaces match or define the property yourself. Your model is right within the Models directory but for some reason you have chosen to nest your factory in a Cats directory which I don't understand.

@taylorotwell I understand your point of view, but I think it would not make sense to do a make:factory Subfolder/Factory and still have the namespace Database\Factory


Using kubeaudit as a Go package isn't possible since ClientOptions is in an internal package

ISSUE TYPE

[X] Bug Report
[ ] Feature Idea

BUG REPORT
SUMMARY
The kubeaudit auditors AuditLocal and AuditCluster both require a parameter of ClientOptions that is defined in
"github.com/Shopify/kubeaudit/internal/k8s" which can't be used outside of kubeaudit itself.
Steps To Reproduce

create a new project NOT a test under kubeaudit project.
copy the example code for a Local or Cluster audit from https://pkg.go.dev/github.com/Shopify/kubeaudit
try to run it and you will get this error "use of internal package github.com/Shopify/kubeaudit/internal/k8s not allowed"

ENVIRONMENT

Kubeaudit version: main / latest
Kubeaudit install method: DIY-BUILD

STEPS TO REPRODUCE

Create a new project, not a test in the kubeaudit package
Copy the code examples for AuditCluster or AuditLocal from https://pkg.go.dev/github.com/Shopify/kubeaudit
Try to compile your project.  You will get the following error "use of internal package github.com/Shopify/kubeaudit/internal/k8s not allowed"

EXPECTED RESULTS
Should be able to use kubeaudit as a package
ClientOptions needs to be moved out of the internal package in order for kubeaudit to be used as a Go package.

Closing this as a duplicate of https://github.com/Shopify/kubeaudit/issues/298. Feel free to reopen if you disagree!

Should be fixed by https://github.com/Shopify/kubeaudit/pull/348


Tensorflow 2.3 or up Installation on Mac OS Big Sur 11.3

Hi,
I am trying to install tensorflow 2.3 or up on Mac OS Big Sur 11.3.  I'm trying to do it in a conda environment with python 3.7.  This is a cpu install.
When I try pip install tensorflow, and go into a python terminal and import, it says that the module does not exist.
When I conda install tensorflow in the conda environment and go into a python terminal and import, I can only get tensorflow 2.0.0.
When I try conda install -c conda-forge tensorflow and go into a python terminal and import tensorflow, I get a slew of warnings of signed and unsigned types:
/Users/blah/anaconda3/envs/funkmuffin2/lib/python3.7/site-packages/tensorboard/compat/tensorflow_stub/dtypes.py:550:
FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.
_np_qint8 = np.dtype([("qint8", np.int8, 1)])
/Users/blah/anaconda3/envs/funkmuffin2/lib/python3.7/site-packages/tensorboard/compat/tensorflow_stub/dtypes.py:550:
FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.
_np_quint8 = np.dtype([("quint8", np.uint8, 1)])
.....
/Users/blah/anaconda3/envs/funkmuffin2/lib/python3.7/site-packages/tensorboard/compat/tensorflow_stub/dtypes.py:550: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.
np_resource = np.dtype([("resource", np.ubyte, 1)])
.... and when I conda update -f -c conda-forge tensorflow, I get the same warnings and I still only get tensorflow 1.14.
Would you know why I cannot simply get tensorflow 2.3 or higher installed?  and/or would anybody know how to do so?  Thank you.

Saw something similar. Are you on an M1 Mac?

Thanks for the speedy reply.  No.  It's Intel Core i7.

Saw something similar. Are you on an M1 Mac?

@o0stsou0o
What is the numpy version on your system, there seems to be compatibility issues with your tensorflow and numpy.
Can you try: pip install   tensorflow 2.4.1 with numpy  1.19.x and let us know. [ypu may refer to this comment, link,#30427,#31249]


provider.getBalance gives error for contract address parameter

Description
Ideally provider.getBalance function should work for both account addresses and contract addresses. At present it only supports account addresses and not contract addresses.
hethers.getDefaultProvider().getBalance errors out on passing contract address.
Error
Error: bad result from backend (method="AccountBalanceQuery", params={"address":"0x00000000000000000000000000000000021456fa"}, error={"name":"StatusError","status":{"_code":15},"transactionId":{"accountId":{"shard":{"low":0,"high":0,"unsigned":false},"realm":{"low":0,"high":0,"unsigned":false},"num":{"low":0,"high":0,"unsigned":false},"aliasKey":null,"_checksum":null},"validStart":{"seconds":{"low":1653292466,"high":0,"unsigned":false},"nanos":{"low":770677894,"high":0,"unsigned":false}},"scheduled":false,"nonce":null}}, code=SERVER_ERROR, version=providers/1.1.1)
Reproduction code
await hethers.getDefaultProvider().getBalance("0x00000000000000000000000000000000021456fa");
Bug Location
@hethers/providers/src.ts/base-provider.ts
function name - getBalance
 const balance = await new AccountBalanceQuery().setAccountId(AccountId.fromString(account)).execute(this.hederaClient);
Fix
Need to differentiate between account address and contract address and set the balance variable as below
 const balance = await new AccountBalanceQuery().setContractId(ContractId.fromString(account)).execute(this.hederaClient);
Search Terms
getBalance
contract address
contract balance

Fixed by https://github.com/hashgraph/hethers.js/pull/248


Plugin not initializing on Android

The following line inside push-plugin.android.js:
com.telerik.pushplugin.PushLifecycleCallbacks.registerCallbacks(app.android.nativeApp);
is causing the plugin to throw an error by the underlying telerik component:
The application is null, it's not passed correctly!
It appears that app.android.nativeApp is undefined/null at the time of the plugin being initialized.
Is anyone else experiencing this issue?
@yyosifov, would you be so kind to take a look? Thanks!

I experience the same issue as well. I'm using TNS 2.3 and TypeScript (but no Angular2) for my project.

hey @svzi, the issue is that the plugin is loading before the nativeApp has been initialized. You can resolve the issue by importing the push plugin after the app has initialized.
I simply moved this line:
var pushPlugin = require('nativescript-push-notifications') to inside the method that needed to use it which in my case is called after init so no longer has the issue.
Also take a look at this for another approach using application.launchEvent:
https://github.com/NativeScript/push-plugin/issues/25

hey @timricker! Thank you very much for your support, you really saved my day. 👍
Simply moving the line didn't resolve it in my scenario, but the approach using application.launchEvent did it.

For whatever reason it didn't work for me. I've stumbled upon other issues hereafter, so removed the plugin from my project and re-added it.
It's the same line of code that you mentioned 7 days ago, but I found out that for me it's not the app.android.nativeApp is null, but instead com.telerik.pushplugin is undefined.
The only line of code I added to my project is var pushPlugin = require("nativescript-push-notifications"); and I call it during the application.launchEvent. Why is com.telerik.pushplugin undefined at that stage? Why it's undefined at all?
Any ideas?

@svzi Can you please check how are you adding the dependency (e.g. the NPM package of the plugin), the problem you are encountering most probably signiifes that the dependency is not added correctly.

Im having the same problem, as @svzi, the com.telerik.pushplugin is undefined and for me the error is:

cannot read property 'PushLifecycleCallbacks' of undefined

on this line
com.telerik.pushplugin.PushLifecycleCallbacks.registerCallbacks(app.android.nativeApp);

@AntonDobrev Sorry, I was on a vacation last week. So my response is somewhat delayed.
I'm not really sure how I can check what you want me to check. Could you please advise? What do you need exactly? The dependency version from my package.json?
Btw I found a workaround for my issue. I just need to call the require-statement async with setTimeout like this:
setTimeout(function () {       var pushPlugin = require("nativescript-push-notifications");       ...

@svzi @yairvillarp This is indeed peculiar, but the only case where I was able to reproduce it was when the dependency was added manually and contained only the NS module w/o the native implementation. That is why my suggestion would be to:

Remove the push plugin
Remove android platform
Add Android platform
Add the push plugin from npm

If the problem persists, let me know of the exact steps to reproduce and the working environment or provide a sample project that illustrates it. Thanks in advance.

Closed due to inactivity. Let me know if there is an update on the situation.


Reconcile ballot entry figure

There are different numbers showing in the headline stats on The Awakening page (https://data.leeds2023.co.uk/metrics/events/awakening/) and microsite home page (https://data.leeds2023.co.uk/)
2374 vs 2421

I think this is individual vs individual + group.

Made tweaks to language on each of the pages. Sufficient for now.


Removed parameter "certificates" in create ASC request parameters and removed multiple KV certificate references in API response

ARM API Information (Control Plane)
Changelog
Add a changelog entry for this PR by answering the following questions:

What's the purpose of the update?

[ ] new service onboarding
[ ] new API version
[ ] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[x ] Other, please clarify

When are you targeting to deploy the new service/feature to public regions? Please provide the date or, if the date is not yet available, the month.
May 2023
When do you expect to publish the swagger? Please provide date or, the the date is not yet available, the month.
May 2023
By default, Azure SDKs of all languages (.NET/Python/Java/JavaScript for both management-plane SDK and data-plane SDK, Go for management-plane SDK only ) MUST be refreshed with/after swagger of new version is published. If you prefer NOT to refresh any specific SDK language upon swagger updates in the current PR, please leave details with justification here.

Contribution checklist (MS Employees Only):

[ x] I commit to follow the Breaking Change Policy of "no breaking changes"
[ x] I have reviewed the documentation for the workflow.
[ x] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
Breaking Change Review Checklist
If you have any breaking changes as defined in the Breaking Change Policy, request approval from the Breaking Change Review Board.
Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Additional details on the process and office hours are on the Breaking Change Wiki.
NOTE: To update API(s) in public preview for over 1 year (refer to Retirement of Previews)
Please follow the link to find more details on PR review process.

/azp run

this pr will be merged after @naveedaz review and approve it


build error

Building with scons:
g++ -o test-arac src/cpp/tests/test_datasets.o src/cpp/tests/test_main.o src/cpp/tests/test_optimzation.o src/cpp/tests/test_structure.o src/cpp/tests/test_utilities.o -L/usr/lib -L. -L/usr/local/lib -L/sw/lib -larac -lgtest

I get the error
./libarac.so: undefined reference to 'cblas_ddot'
./libarac.so: undefined reference to 'cblas_dger'
./libarac.so: undefined reference to 'cblas_dgemv'

Any advice as to how to workaround/fix this?

I have the same issue.

you need to install the cblas library.  I'm not sure where to get it.

You can download BLAS from here http://www.netlib.org/blas/


Allow -Z minimal-versions to fail

The actual build fails already (hence it was disabled by a scripted if).
With these changes, the build is ran but allowed it to fail. In its
current state, minimal versions can at most give an indication on the
health of our dependencies but is not required (as indicated by the
disabled build). Since this is also prone to the most failures from
nightly, it has only little bearing on the decision of merging a PR and
releases. The standard nightly target is still enabled regardless.

Clarification: If minimal-versions becomes available on stable then it should be enabled again.


Problem with IP POOLS

Good morning, I'm having a problem deleting an ip pool.
When I go to the section "IP POOLS" on the up right side of the screen, and I want to delete an ip pool previously configurated, the next message appears; "IP pool can not be removed because it is still assigned to server/rules", but I have already removed all assigned rules from the servers.
Previouly, doing tests, I removed all servers and organizations, but the error message was still appearing.
For now, I have to remove it from the database to resolve the problem.
Is this a bug, or am I doing something wrong ?
thanks for your attention.

I think the error message here is a little misleading. According to the code, there are two things that will prevent an IP pool being deleted:
  has_many :ip_addresses, :dependent => :restrict_with_exception
  has_many :servers, :dependent => :restrict_with_exception

From your description, I suspect the problem was likely that the pool still contained IP addresses. The error is not clear that IP addresses need to be removed before the pool can be removed, so I will fix that at least.


Update test for SetIrrigationPracticeTSPumpingMaxUsingWellRights

The test for SetIrrigationPracticeTSPumpingMaxUsingWellRights is failing due to changes in code that add more checks for complete data.  The warnings can be turned off via a parameter for basic tests.

I updated the test.  Close this issue.


Installation Issue - Composer conflict

Installing on existing Laravel v11.15.0 project.
Run:
composer require northwestern-sysdev/dynamic-forms
Error:
Problem 1
    - northwestern-sysdev/dynamic-forms[v1.0.0, ..., v1.0.1] require northwestern-sysdev/lodash-php ^1.0.0 -> satisfiable by northwestern-sysdev/lodash-php[v1.0.0, v1.1.0, v1.1.1].
    - northwestern-sysdev/lodash-php v1.0.0 requires sebastian/comparator ^1.2 | ^2.0 | ^2.1 | ^3.0 | ^4.0 -> found sebastian/comparator[1.2.0, ..., 1.2.4, 2.0.0, ..., 2.1.3, 3.0.0, ..., 3.0.5, 4.0.0, ..., 4.0.8] but the package is fixed to 5.0.1 (lock file version) by a partial update and that version does not match. Make sure you list it as an argument for the update command.
    - northwestern-sysdev/lodash-php[v1.1.0, ..., v1.1.1] require symfony/property-access ^2.7 | ^3.0 | ^4.0 | ^5.0 -> found symfony/property-access[v2.7.0, ..., v2.8.52, v3.0.0, ..., v3.4.47, v4.0.0, ..., v4.4.44, v5.0.0, ..., v5.4.40] but the package is fixed to v7.1.1 (lock file version) by a partial update and that version does not match. Make sure you list it as an argument for the update command.
    - Root composer.json requires northwestern-sysdev/dynamic-forms ^1.0 -> satisfiable by northwestern-sysdev/dynamic-forms[v1.0.0, v1.0.1].

Use the option --with-all-dependencies (-W) to allow upgrades, downgrades and removals for packages currently locked to specific versions.
You can also try re-running composer require with an explicit version constraint, e.g. "composer require northwestern-sysdev/dynamic-forms:*" to figure out if any version is installable, or "composer require northwestern-sysdev/dynamic-forms:^2.1" if you know which you need.

Running the following:
composer why sebastian/comparator
phpunit/phpunit 10.5.27 requires sebastian/comparator (^5.0.1)

Just closing this as it was resolved after I removed the composer.lock file and run composer install.


Move package version to details, and rename VersionString() to ToString().

Now that PackageVersion is no longer public, https://github.com/Azure/azure-sdk-for-cpp/issues/2053 is not breaking, and hence we don't require it for this release (it can be done after).
Good call on https://github.com/Azure/azure-sdk-for-cpp/issues/2052. I do think if we want to do it, we would need to do it this release since renaming/moving header files is a breaking change. Can you please take on that issue, @antkmsft?


adding conversational ai

https://elevenlabs.io/docs/conversational-ai-sdks/conversational-ai-guide

I released the guide on our docs, I will put a hyperlink in guide to this repo for people to pull and play with it directly


Accessing ClusterIP services doesn't work via a Tailscale pod

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
I am running Tailscale (a wireguard-based VPN) inside a pod with NET_ADMIN capabilities.
I have tested two scenarios:

Accessing a pod via its IP
Accessing a ClusterIP service

Accessing the pod works, but accessing the Cluster IP doesn't.
The traffic directed to the Cluster IP is also not visible in Hubble when I tried observing for it. Instead, the packets are forwarded to the default gateway (and lost there).
Tailscale installs the following iptables rules:
*nat
:PREROUTING ACCEPT [18:1296]
:INPUT ACCEPT [18:1296]
:OUTPUT ACCEPT [196:15731]
:POSTROUTING ACCEPT [196:15731]
:ts-postrouting - [0:0]
-A PREROUTING -d <IP_ADDRESS>/32 -j DNAT --to-destination <IP_ADDRESS>
-A POSTROUTING -j ts-postrouting
-A ts-postrouting -m mark --mark 0x40000/0xff0000 -j MASQUERADE
COMMIT
# Completed on Sat Jul 15 12:30:20 2023
# Generated by iptables-save v1.8.8 on Sat Jul 15 12:30:20 2023
*filter
:INPUT ACCEPT [799:96468]
:FORWARD ACCEPT [0:0]
:OUTPUT ACCEPT [814:92651]
:ts-forward - [0:0]
:ts-input - [0:0]
-A INPUT -j ts-input
-A FORWARD -j ts-forward
-A ts-forward -i tailscale0 -j MARK --set-xmark 0x40000/0xff0000
-A ts-forward -m mark --mark 0x40000/0xff0000 -j ACCEPT
-A ts-forward -s <IP_ADDRESS>/10 -o tailscale0 -j DROP
-A ts-forward -o tailscale0 -j ACCEPT
-A ts-input -s <IP_ADDRESS>/32 -i lo -j ACCEPT
-A ts-input -s <IP_ADDRESS>/23 ! -i tailscale0 -j RETURN
-A ts-input -s <IP_ADDRESS>/10 ! -i tailscale0 -j DROP
COMMIT
# Completed on Sat Jul 15 12:30:20 2023

On the host I can see the traffic in tcpdump as going from the tailscale pod's IP to the service IP. Unfortunately, it's routed to the default gateway (a router) that doesn't know what to do with it and drops it.
I have found a related thread on Tailscale's forum and one of the users suggested that enabling lbExternalClusterIP solved the issue for them. However it didn't do anything in my case. I am confused because accessing the ClusterIP from the physical node works as well (as you are probably well aware, I wasn't).
I don't understand why cilium doesn't seem to "pick up" this traffic.
Cilium Version
Client: 1.12.0 9447cd1 2022-07-19T12:22:00+02:00 go version go1.18.4 linux/amd64
Daemon: 1.12.0 9447cd1 2022-07-19T12:22:00+02:00 go version go1.18.4 linux/amd64
Kernel Version
Linux minio5 5.15.117-flatcar #1 SMP Tue Jul 4 14:43:38 -00 2023 x86_64 Intel(R) Xeon(R) CPU E5-2440 0 @ 2.40GHz GenuineIntel GNU/Linux
Kubernetes Version
v1.23.8+k3s2
Sysdump
https://drive.google.com/file/d/1GYxkeaGzXHIOuOM9DYFDLHFgUUXUypA3/view?usp=sharing
Relevant log output
No response
Anything else?
This exact setup works on another cluster but that "cluster" is just a single node k3s deployment. On that cluster I can access Cluster IPs via tailscale (same cilium version, same config).
https://forum.tailscale.com/t/tailscale-proxy-in-k8s-with-cilium-works-with-pod-not-with-svc/1910/8
seemingly related:
https://github.com/cilium/cilium/issues/26584
Code of Conduct

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

Thanks for the issue. Could you draw a diagram with all pods and show who is accessing what?

@wokalski take a look at this comment: https://github.com/cilium/cilium/issues/27758#issuecomment-1718282433

Fix https://github.com/cilium/cilium/pull/33421.

@brb thanks for fixing this. I have completely missed your comment from one year ago, ha!


vite-node incompatible with oclif

Describe the bug
I am writing a CLI for a project of mine and I want to be able to include files from that project when running the CLI for certain tasks.  However, since I rune vite (for the UI) and vite-node (for the API) for that project, that means I must be able to run vite-node for the CLI as well.  I have chosen oclif for the CLI, as it is a fairly popular CLI framework used by major companies to write their CLIs, however it does not run using vite-node.  I get the following error when it attempts to read the commands folder and load the individual command files:
(node:51441) [ERR_UNKNOWN_FILE_EXTENSION] TypeError Plugin: oclif-vite-node-repro [ERR_UNKNOWN_FILE_EXTENSION]: Unknown file extension ".ts" for /Users/danhulton/dev/oclif-vite-node-repro/src/commands/hello/index.ts
module<EMAIL_ADDRESS>task: toCached
plugin: oclif-vite-node-repro
root: /Users/danhulton/dev/oclif-vite-node-repro
See more details with DEBUG=*
(Use `node --trace-warnings ...` to show where the warning was created)
TypeError Plugin: oclif-vite-node-repro [ERR_UNKNOWN_FILE_EXTENSION]: Unknown file extension ".ts" for /Users/danhulton/dev/oclif-vite-node-repro/src/commands/hello/index.ts
    at new NodeError (node:internal/errors:393:5)
    at Object.getFileProtocolModuleFormat [as file:] (node:internal/modules/esm/get_format:79:11)
    at defaultGetFormat (node:internal/modules/esm/get_format:121:38)
    at defaultLoad (node:internal/modules/esm/load:81:20)
    at nextLoad (node:internal/modules/esm/loader:163:28)
    at ESMLoader.load (node:internal/modules/esm/loader:605:26)
    at ESMLoader.moduleProvider (node:internal/modules/esm/loader:457:22)
    at new ModuleJob (node:internal/modules/esm/module_job:63:26)
    at ESMLoader.#createModuleJob (node:internal/modules/esm/loader:480:17)
    at ESMLoader.getModuleJob (node:internal/modules/esm/loader:434:34)
module<EMAIL_ADDRESS>task: toCached
plugin: oclif-vite-node-repro
root: /Users/danhulton/dev/oclif-vite-node-repro
See more details with DEBUG=*

I do not know which package is responsible for the error, so I am opening an issue in each project and am cross-linking them here:
"vite-node incompatible with oclif" at oclif/oclif
Reproduction
A full reproduction can be found at DanHulton/oclif-vite-node-repro.
This is a minimal install of both oclif and vite-node, as well as the bare minimum amount of changes required to integrate the two (which boil down to changing the shebang to use vite-node and converting oclif to use ESM).
System Info
System:
    OS: macOS 13.4
    CPU: (8) x64 Apple M1 Pro
    Memory: 76.40 MB / 32.00 GB
    Shell: 3.6.1 - /usr/local/bin/fish
  Binaries:
    Node: 18.12.1 - ~/.nvm/versions/node/v18.12.1/bin/node
    Yarn: 3.3.1 - ~/.nvm/versions/node/v18.12.1/bin/yarn
    npm: 9.7.1 - ~/.nvm/versions/node/v18.12.1/bin/npm
  Browsers:
    Chrome: 114.0.5735.133
    Firefox: 113.0.2
    Safari: 16.5

Used Package Manager
npm
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

Getting the same error after upgrading aws-amplify:
FAIL  src/components/foobar.test.tsx [ src/components/foobar.test.tsx ]
TypeError: Unknown file extension ".ts" for /home/runner/work/project-name/node_modules/.pnpm/@aws-amplify+data-schema-types@0.7.17_@aws-sdk+types@3.387.0_aws-cdk-lib@2.139.0_constructs@10.3.0__constructs<EMAIL_ADDRESS>⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯
Serialized Error: { code: 'ERR_UNKNOWN_FILE_EXTENSION' }

This is because of an intrinsic dependency<EMAIL_ADDRESS>Their package.json is quite odd:
"main": "./lib-esm/index.d.ts",

We should probably mock the module or something like that, so I wonder if someone has already done something like this before.

@kachkaev Can you share more context of the error? Though their "main" field is indeed odd, the error itself is probably coming from NodeJS trying to import/require it, so as long as you put them into test.server.deps.inline (which makes Vite to process the dependency instead), it might work fine.
Also, OP was about the direct vite-node usage, so it might work slightly differently when using it from Vitest. It would be easier for us to help if you can raise an new issue with a proper reproduction.


Introduce values schema for the collector chart

Introduce values.schema.json for values.yaml input validation in opentelemetry-collector helm chart. All the objects in values.yaml with predefined set of properties should not allow unknown properties.
This is a preliminary step for any big change to the helm chart configuration, e.g. https://github.com/open-telemetry/opentelemetry-helm-charts/issues/91
The following schema can be used as a reference: https://github.com/signalfx/splunk-otel-collector-chart/blob/main/helm-charts/splunk-otel-collector/values.schema.json

I can look at this.

There is a lot of schema to add.  Will probably submit PRs in  small chunks to make review easier since each bit of validation added is valid in itself.

There is a lot of schema to add. Will probably submit PRs in small chunks to make review easier since each bit of validation added is valid in itself.

Sounds good. Thanks!


Support SPNEGO/Kerberos auth to Elasticsearch

Work is wrapping up on adding a Kerberos based authentication realm to Elasticsearch. The realm will secure the rest endpoints by means of SPNEGO. Since Kerberos is used heavily in the Big Data space, we should offer an integration that allows for ES-Hadoop to use the currently logged in user and their Kerberos Ticket to authenticate to Elasticsearch over SPNEGO. This should include token authentication and appropriate refreshing mechanisms to avoid the same pitfalls that Hadoop runs into with KDC spamming and token lifetime.

@jbaiera
This is great to know. We can now integrate ES also into a centralised Kerberos cluster. Any estimate of when this is going to be released?

There is a substantial amount of work that is already completed for this feature, but there is quite a bit more to go, including the extensive testing that we will need to eventually run to ensure correctness.
Additionally, we are working out the best course of action for handling token authentication in worker tasks. The current token functionality in ES is based on OAuth username and password grants, which does not provide tokens that are useable in Hadoop/Spark/Storm (the tokens change between refreshes, last only an hour, can only be refreshed once, etc).
Hopefully soon I will be able to pull together a preliminary PR for the work, but until then, I have the work periodically checked into a feature branch located at https://github.com/elastic/elasticsearch-hadoop/compare/master...jbaiera:feature-kerberos

This is currently waiting on #1183

Closed in error.

This is currently waiting on https://github.com/elastic/elasticsearch/issues/34383 for further development

#1244 is merged now.


Where is "run_synthet.sh"？

Hi, I think this is very interesting work.
I found that SIMNPO's README says "Please see 'run_synthetic.sh' for how to run the code."
But I can't seem to find the file "run_synthetic.sh" where?

Thank you for your recognition of and interest in our work! We have uploaded the run_synthetic.sh script as requested.
If you have any further questions, please don't hesitate to reach out to us!

The issue has been closed. If you have any further questions, feel free to contact us at any time.


about temp file

when upload one file, in the dirct of data genenrate two tempfile。 one file is lock file ， end of upload deleted，but other pid file not delete

I'm sorry but I have no idea what you actually mean. You are correct that a .lock file is there as long as an client is uploading but there should be no "pid file". Please be more descriptive.

   please check the picture，I give the  tag on the picture.

Ok, thanks for clarifying things. This file is indeed a temporary one used for creating the .lock file. What concerns you about it?

why don't delete it after upload file?
I findthe temp file in the folder , i think the file must be deleted .

You are correct, usually it will be removed however, it may have issues on Windows. I can look into this once I have some time left, as this has looks like a low priority.

This should have been solved with #117. Feel free to comment if this issue is still arising for you.


Upgrade Cadence

We are far behind the latest Cadence version (0.7.1 from last August vs 0.11.0).
The last release contains a bugfix related to task lists and MySQL, which affects us as well.

Moved to another ticekt.


Pagination with Authorization Rules

Before opening, please confirm:

[X] I have installed the latest version of the Amplify CLI (see above), and confirmed that the issue still persists.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.

How did you install the Amplify CLI?
npm
If applicable, what version of Node.js are you using?
v14.17.0
Amplify CLI Version
4.52.0
What operating system are you using?
Mac
Amplify Categories
auth, function, api, hosting
Amplify Commands
push
Describe the bug
The pagination of the connection models seems to be happening before the Authorization rules are evaluated.
Such that, if a limit is given, the items returned only if they fall within the limit or the pages. Otherwise the nextToken is also null when the items are null.
Expected behavior
The paginated items are properly returned.
Reproduction steps
** Cognito User Pools are enabled, and contain 6 different groups**

A Customer Model with global access with Roles linked as  (connection (auth:{allow:private}))
Orders model which is one-to-many for the Customer
Orders model restrict access based on Authorization rules on itself. Such that if a group can only access certain models.
18 Order items are created, each 3 orders require a specific user pool group to access.(total of 6 groups)
A specific cognito user is granted to 3 groups and access the Customers page.
i. 	With the nested query user will only see the items which they have access to, in the first 10 items. If they dont have any, they receive null.
A specific cognito user is granted to 6 groups and access the Customers page.
i. 	With the nested query user will see all the nested items

GraphQL schema(s)

# Put schemas below this line

Customer
@auth([{allow:private}])
{
	id: ID!
	customer_id: String!
	orders: [Orders] @connection(fields:["customer_id"]) @auth([{allow:private,operations:[read]}])

}

Order
@key(fields:["customer_id","order_id"])
@auth([{allow:groups,groupsField:"access_group"}]) {
	id: ID!
	order_no: String
	details: String
	access_group: String
}

Log output

# Put your logs below this line

Additional information
This scenario also happens in the base model if they have specific user group accesses

The issue I am facing is actually of two levels nesting, such if a OrderItems object would exist.
If that specific object has an access rule, above nesting mechanism disallows seeing.
I will review the issue myself before reporting again if it seems an issue with Amplify's resolver generation.


Generate pure enums

The enums generated by the generator are not pure. This creates a problem if a value of the enum is named the same as the name of the enum, except for casing. The simplest example is
syntax = "proto3";

package example;

enum Foo {
  FOO = 0;
  NOT_FOO = 1;
}

If you run the generator on this, you get the following nim file
# Generated by protoc_gen_nim. Do not edit!

import intsets

import protobuf/gen
import protobuf/stream
import protobuf/types

const
    FooDesc = EnumDesc(
        name: "Foo",
        values: @[
            EnumValueDesc(name: "FOO", number: 0),
            EnumValueDesc(name: "NOT_FOO", number: 1),
        ]
    )
generateEnumType(FooDesc)
generateEnumProcs(FooDesc)

This file will not compile because the name of the enum (Foo) conflicts with one of the values of the enum itself (FOO). It would be nice to have the possibility to generate pure enums, so that the value FOO would need to be scoped as Foo.FOO and no conflict would arise

Yeah I was thinking that it might be good idea to generate pure enums from the start because the language supports that. I was lazy so didn't do it yet but I'll definitely get it done at some point.
Thanks for reminding! :-)


Shiny Python app deployment on shinyapps.io error: Bundle does not contain manifest file (package)

Hi,
I have a Windows and when i try to deploy my app on shinyapp.io an error occurs. In particolar the error is:
"Error: Application deployment failed with error: Unhandled Exception: Child Task<PHONE_NUMBER> failed: Error parsing manifest: Bundle does not contain manifest file: demo\Lib\site-packages\Jinja2-3.1.2.dist-info\INSTALLER"
in the folder "demo\Lib\site-packages\Jinja2-3.1.2.dist-info" there is the file INSTALLER, so i don't understand the issue.
Thanks,

Hi, this is probably the same issue as: https://github.com/rstudio/rsconnect-python/issues/373

I believe this was fixed by https://github.com/rstudio/rsconnect-python/issues/373.


Can't customize focus colors for select and other inputs

I have buttons that work fine using the four focus colors listed here: https://daisyui.com/docs/colors/
But my selects, inputs, and other elements when focused have a super super light outline that is barely visible. I can't see any way, in daisyui, tailwind, or vanilla css to solve this. I can only override it and it still appears.

Additionally, when applying :focus in the inspector, this style doesn't appear, even though all other focus styles appear. Very bizarre

	button:focus,
	select:focus,
	input:focus,
	label:focus {
		@apply outline-base-700;
	}
	```
is my temp solution but I have to copy paste this across everywhere it's relevant because it is not possible from the config. Only buttons use the focus variables defined in the theme file.

Some new spec comes out, and the browser changed the pseudo-class name to :focus-visible
So we need to do this instead:
:focus-visible {
   @apply <your style name>
}

In order to change border/outline color when focus I used these classes
focus-visible:border-0
focus-visible:ring-0
focus-visible:ring-offset-2
focus-visible:ring-offset-orange-300


Miscellaneous

[ ] Review the PyOpenSci package reviewer guides and checklists to make sure your package meets the criteria on which it will be reviewed on next week in our peer review:

PyOpenSci reviewer guide and checklist

[ ] Make sure all GitHub Actions workflows run without error and are advertised as such using badges in the README.md of your repository

Do this if you have time


adds helper function for coco_parser

closes #188

Codecov Report

Merging #189 into master will decrease coverage by 0.33%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #189      +/-   ##
==========================================
- Coverage   54.89%   54.55%   -0.34%
==========================================
  Files         103      102       -1
  Lines        2583     2555      -28
==========================================
- Hits         1418     1394      -24
+ Misses       1165     1161       -4

Flag
Coverage Δ

#unittests
54.55% <100.00%> (-0.34%)
:arrow_down:

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

Impacted Files
Coverage Δ

mantisshrimp/datasets/coco/parsers.py
100.00% <100.00%> (ø)

mantisshrimp/test_utils/samples.py
81.81% <100.00%> (ø)

mantisshrimp/core/bbox.py
64.70% <0.00%> (-4.79%)
:arrow_down:

mantisshrimp/models/device_module_mixin.py
80.00% <0.00%> (-3.34%)
:arrow_down:

mantisshrimp/models/mantis_rcnn/mantis_rcnn.py
48.78% <0.00%> (-2.39%)
:arrow_down:

...antisshrimp/metrics/coco_metric/reference_utils.py
19.28% <0.00%> (-2.00%)
:arrow_down:

mantisshrimp/metrics/metric.py
81.81% <0.00%> (-1.52%)
:arrow_down:

...tisshrimp/models/mantis_rcnn/mantis_faster_rcnn.py
33.33% <0.00%> (-1.29%)
:arrow_down:

...antisshrimp/models/mantis_rcnn/mantis_mask_rcnn.py
39.58% <0.00%> (-1.24%)
:arrow_down:

mantisshrimp/core/category.py
85.00% <0.00%> (-0.72%)
:arrow_down:

... and 5 more

Continue to review full report at Codecov.

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


Update Component Namespace to rad- (from core-)

Summary
All component names, their CSS classes, and documentation/example references should have core- swapped out for rad-.
Description + Repro Steps
We should update the namespace on these components from core- to rad- to be more on brand with the name of the addon, and also to make it more specific to our code.
core- is a holdover from the uicore port and it would be wise to differentiate between the two to avoid confusion internally.
Example
Currently, we have components like core-button and core-modal; these would be renamed to rad-button and rad-modal.

This is a rad idea


retro-contest does not exit with an appropriate status

I've been scripting my job submission process, and in that script I test the exit status of commands like retro-contest submit and retro-contest login. I know in particular that these two commands don't exit with non-zero statuses when errors have occurred, and I believe this issue extends to other commands as well. I know it is only convention, but it is a fairly common convention.

Good call. I'll add this soon.


hideControls not working

I am making the Verticle_alternating but the hideControls are not working, though I have hide it with transparent background property but the buttons are still visible. The main issue still persists

Same issue here

new prop disableToolbar added to hide the toolbar.

that one is also not working, I don't know if the improvement has been broken
here is my code though
const TimeLine = () => {
  return (
    <section id="roadmap">
    <Chrono
      items={items}
      mode="VERTICAL_ALTERNATING"
      disableToolbar
      theme={{
        primary: "darkblue", // Primary color for the timeline
        secondary: "gray", // Secondary color for the timeline
        cardBgColor: "rgba(200, 216, 230, 0.2)",
        cardForeColor: "white", // Font color for the cards
        titleColor: "white", // Color of the titles on the timeline cards
        titleColorActive: "orange", // Color of the active card title
        cardSubtitleColor: "grey", // Color for subtitles on the cards
        cardTitleColor: "orange", // Color for card titles
        cardDetailsColor: "lightblue", // Details inside card
        cardOutline: "darkblue", // Outline color for the cards
        timelineStrokeColor: "orange", // Line color of the timeline
        markerBorderColor: "orange", // Border color for the markers
        markerFillColor: "darkblue", // Fill color for the markers
        toolbarBgColor	: "transparent",
        toolbarTextColor: "transparent",
        toolbarBtnBgColor: "transparent",
        iconBackgroundColor: "transparent",
      }}
disableToolbar

    />
      </section>
  );
};

export default TimeLine;

@ikhana looks like there was some issue with the npm publish operation. the correction version of the lib should be 2.6.0. Give it a try again by installing the latest version and let me know how it goes

@prabhuignoto thank you for mentioning the correct version, worked.


uInverter: Need to document port number on analog connector

This issue is a request to add an explanation of which uInverter connector port is uInv Conn 1 and which is uInv Conn 2 in this table.
The uInverter connections document has this great table of how the current and voltage sensing signals map to the AMDC ports:

However, as far as I can tell, we don't give any information on which of the two connectors is uInv Conn 1 and which is uInv Conn 2. Is the top connector uInv Conn 1 and the bottom connector uInv Conn 2? Or vice versa?
I suggest we do this in a similar fashion to how the connector numbers are indicated for the AMDC:

That is, via a graphic with port numbers labeled.

[ ] Add a rendering to uInverter connections page and number the connectors
[ ] Add link to AMDC analog connectors page


Adding a ganache-local on brownie

When adding ganache-local to the network at 5:38:46, the host

brownie network add Ethereum ganache-local host=http://<IP_ADDRESS>:8545 chainid=1337

Whereas brownie is listening on <IP_ADDRESS>:8545. This lead to the error of get_account() having list index out of range error because it has an empty list.
Note: I also had to change ganache's server from <IP_ADDRESS> to <IP_ADDRESS>
I've spent a long time trying to figure out the problem and almost gave up, would mean a lot if you could add it to the list of issues to help other people.

Hello @WyzzR Please share the following:

the command you are using to add the network to brownie
a screenshot of your ganache ui.

At 5:39:01 the video said to type in host=http://<IP_ADDRESS>:8545

However 2 minutes before that, at 5:37:07, just where the cursor is it says local RPC client listening '<IP_ADDRESS>:8545'

This is weird because the ganache RPC server is at http://<IP_ADDRESS>:8545 as you can see at 5:38:50 (after the previous picture)

So my problem was that brownie was listening on <IP_ADDRESS>:8545 but my ganache and the network settings on brownie was set to <IP_ADDRESS>:8545. The weird part is that in the video it works even though the RPC servers are different.
I have been looking for ways to solve it but haven't found any so it seams that I am the only one to have this problem.
Hope this helps

Depends on the operating system: <IP_ADDRESS> and <IP_ADDRESS> points to the same direction which is local host, but this is not always recognized by all OS. So you must just put what ever ganache throws you, not necessarily the exact same as Patrick.
So on your ganache, which ip is giving to you?

So here my ganache and the network I added to brownie is <IP_ADDRESS>

The problem is because the accounts is an empty list, I think. From the ss below, the object accounts works but when it try to call accounts[0] there is an out of index error.

However if I keep the same code and change both brownie's and ganache host to <IP_ADDRESS> then it works

Also, a different problem, do you know why there is INFO: Could not find files for the given pattern(s). after every time I run a brownie command? It hasn't caused any problems but Patrick did not have it.

Sorry for the late reply, and thank you very much.
I'm just thinking that it caused me a lot of trouble and I could not find an answer anywhere, other people can also have the same problem. So maybe adding it to the list of issues would help newcomers. I guess that they can read this but I was new to github and didn't even know this existed.
Thank you again, you can probably close this.


Private key not working

Defining private key in the following way does not work:
ssh_key: "xxx" or "{{ lookup('file', '/path/to/id_rsa') }}"
This is due to the template containing:
# {{ ansible_managed }}
Doing this makes it an invalid private key. Please remove it from the template or just remove the template as a whole as it is not needed.

I removed the comment for now.
Just out of interest, what's your environment, since there hasn't been an issue with this yet?

Thanks! I'm a minimal debian stretch.


How do I run this?

Awesome project!! How can I use it? I already downloaded a zip file!!

It's not currently ready for proper usage yet. It'll be a few weeks still. However, if you really want to try it in it's very early stage, it would depend on what OS you are using.
If Windows, download and run the exe file from the releases page.
If Max, download and run the dmg file from the releases page.
If Linux, download and run the AppImage file from the releases page.

okay cool. I can wait!


Signing of nuget package

Hi, I have noticed that someone previously asked to get the nuget package signed and you had responded saying that it had been done. However, the package you are distributing on nuget.org is still not signed - when I download and try to build it into my signed solution, Visual Studio complains. Please could you look into this?

I believe this is fixed now. Please have a look and let me know.

Hi @micampbell    I also encountered the same problem. How can I solve it？


No nodes available on the network

I cannot find any nodes for my notEvilDime wallet software to connect to. Do you have any nodes that I can add so that I may sync with the network? If you do, please add my node @ <IP_ADDRESS>:17933.

If there are no nodes, is there an archive containing the blockchain data that I can download and import into my wallet?


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/51169

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/53673


generated override for abstract members in sourceClazz

Codecov Report

Merging #16 into master will decrease coverage by 0.28%.
The diff coverage is 73.68%.

@@             Coverage Diff              @@
##             master      #16      +/-   ##
============================================
- Coverage     82.03%   81.74%   -0.29%
- Complexity      145      152       +7
============================================
  Files            24       24
  Lines           373      389      +16
  Branches         56       63       +7
============================================
+ Hits            306      318      +12
- Misses           25       26       +1
- Partials         42       45       +3

Impacted Files
Coverage Δ
Complexity Δ

...java/com/kaufland/model/field/CblConstantHolder.kt
88.88% <ø> (ø)
5 <0> (ø)
:arrow_down:

...ava/com/kaufland/model/field/CblBaseFieldHolder.kt
50% <ø> (ø)
7 <0> (ø)
:arrow_down:

.../java/com/kaufland/generation/WrapperGeneration.kt
96.49% <100%> (ø)
7 <0> (ø)
:arrow_down:

...n/java/com/kaufland/generation/EntityGeneration.kt
98% <100%> (ø)
14 <0> (ø)
:arrow_down:

...in/java/com/kaufland/model/field/CblFieldHolder.kt
84.61% <100%> (+1.28%)
17 <0> (+1)
:arrow_up:

...java/com/kaufland/model/entity/BaseEntityHolder.kt
84.61% <100%> (+1.28%)
10 <1> (+1)
:arrow_up:

.../src/main/java/com/kaufland/model/EntityFactory.kt
62.5% <54.54%> (-6.74%)
10 <7> (+4)

.../main/java/com/kaufland/validation/PreValidator.kt
50% <0%> (+4.54%)
9% <0%> (+1%)
:arrow_up:

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 4c76825...d79789a. Read the comment docs.


Upload new music with audio file (.mp3 only)

It involves

Design for "upload button" which will required music name and file (mp3 only). Can have a restriction e.g. no duplicate songs in the list.
Code

May I work on this issue 😃?

Hi Just want to make sure if you are working on it or not ?