'localNotification' event Handler is not fired

I add local noti request in my app and the notification is delivered well but the event listener is not called.
	PushNotificationIOS.addNotificationRequest({
			id: 'asdfdf',
			title: 'dasdf',
			isCritical: true,
			isSilent: true,
			body: '일어나야지~~',
			fireDate: date,
		});

this is my code to send notification
useEffect(() => {
		console.log('uef');
		PushNotificationIOS.addEventListener('localNotification', () => {
			console.log('로컬 노티 왔어요~~');
		});

		return () => {
			PushNotificationIOS.removeEventListener('localNotification');
		};
	}, []);

this is how i wrote on App.tsx useeffect just to test if the handler is triggered.
I don't know if the trouble happens because of 'react-native-splash-screen'? i saw a issue mentioning this library..
My AppDelegate.h
#import <UserNotifications/UNUserNotificationCenter.h>
#import <React/RCTBridgeDelegate.h>
#import <UIKit/UIKit.h>

@interface AppDelegate : UIResponder <UIApplicationDelegate, RCTBridgeDelegate, UNUserNotificationCenterDelegate>

@property (nonatomic, strong) UIWindow *window;

@end

My AppDelegate.mm

#import "AppDelegate.h"
// firebase
#import <Firebase.h>

#import <UserNotifications/UserNotifications.h>
#import <RNCPushNotificationIOS.h>

#import <React/RCTBridge.h>
#import <React/RCTBundleURLProvider.h>
#import <React/RCTRootView.h>
#import "RNSplashScreen.h"

#import <React/RCTAppSetupUtils.h>

....

- (BOOL)application:(UIApplication *)application didFinishLaunchingWithOptions:(NSDictionary *)launchOptions
{
...
  // Define UNUserNotificationCenter
  UNUserNotificationCenter *center = [UNUserNotificationCenter currentNotificationCenter];
  center.delegate = self;

return YES;
}

//Called when a notification is delivered to a foreground app.
-(void)userNotificationCenter:(UNUserNotificationCenter *)center willPresentNotification:(UNNotification *)notification withCompletionHandler:(void (^)(UNNotificationPresentationOptions options))completionHandler
{
  completionHandler(UNNotificationPresentationOptionSound | UNNotificationPresentationOptionAlert | UNNotificationPresentationOptionBadge);
}
// Required for localNotification event
- (void)userNotificationCenter:(UNUserNotificationCenter *)center
didReceiveNotificationResponse:(UNNotificationResponse *)response
         withCompletionHandler:(void (^)(void))completionHandler
{
  [RNCPushNotificationIOS didReceiveNotificationResponse:response];
}

// Required for the register event.
- (void)application:(UIApplication *)application didRegisterForRemoteNotificationsWithDeviceToken:(NSData *)deviceToken
{
 [RNCPushNotificationIOS didRegisterForRemoteNotificationsWithDeviceToken:deviceToken];
}
// Required for the notification event. You must call the completion handler after handling the remote notification.
- (void)application:(UIApplication *)application didReceiveRemoteNotification:(NSDictionary *)userInfo
fetchCompletionHandler:(void (^)(UIBackgroundFetchResult))completionHandler
{
  [RNCPushNotificationIOS didReceiveRemoteNotification:userInfo fetchCompletionHandler:completionHandler];
}
// Required for the registrationError event.
- (void)application:(UIApplication *)application didFailToRegisterForRemoteNotificationsWithError:(NSError *)error
{
 [RNCPushNotificationIOS didFailToRegisterForRemoteNotificationsWithError:error];
}

Please help me figure out the problem ...
how can others are using this feature with no problem? did I write something wrong in my appdelegate? :(
I want to do some work when the notification is delivered to users(not clicked).

Hi, we're facing same issue. and we found out that after installing notifee, it failed to fire 'localNotification'. after removing notifee, it works fine. so it might be helpful if you've used notifee so far.

Hi, same issue.

Hi, facing same issue for app in foreground state. Was working until we updating dependency to latest version.

any update on this issue? I am facing the same problem.

react-native 0.62.2
@react-native-community/push-notification-ios 1.8.0

same issue

try to touch/tap the localNotification notification, you will get the listener triggered

I add local noti request in my app and the notification is delivered well but the event listener is not called.
	PushNotificationIOS.addNotificationRequest({
			id: 'asdfdf',
			title: 'dasdf',
			isCritical: true,
			isSilent: true,
			body: '일어나야지~~',
			fireDate: date,
		});

this is my code to send notification
useEffect(() => {
		console.log('uef');
		PushNotificationIOS.addEventListener('localNotification', () => {
			console.log('로컬 노티 왔어요~~');
		});

		return () => {
			PushNotificationIOS.removeEventListener('localNotification');
		};
	}, []);

this is how i wrote on App.tsx useeffect just to test if the handler is triggered.
I don't know if the trouble happens because of 'react-native-splash-screen'? i saw a issue mentioning this library..
My AppDelegate.h
#import <UserNotifications/UNUserNotificationCenter.h>
#import <React/RCTBridgeDelegate.h>
#import <UIKit/UIKit.h>

@interface AppDelegate : UIResponder <UIApplicationDelegate, RCTBridgeDelegate, UNUserNotificationCenterDelegate>

@property (nonatomic, strong) UIWindow *window;

@end

My AppDelegate.mm

#import "AppDelegate.h"
// firebase
#import <Firebase.h>

#import <UserNotifications/UserNotifications.h>
#import <RNCPushNotificationIOS.h>

#import <React/RCTBridge.h>
#import <React/RCTBundleURLProvider.h>
#import <React/RCTRootView.h>
#import "RNSplashScreen.h"

#import <React/RCTAppSetupUtils.h>

....

- (BOOL)application:(UIApplication *)application didFinishLaunchingWithOptions:(NSDictionary *)launchOptions
{
...
  // Define UNUserNotificationCenter
  UNUserNotificationCenter *center = [UNUserNotificationCenter currentNotificationCenter];
  center.delegate = self;

return YES;
}

//Called when a notification is delivered to a foreground app.
-(void)userNotificationCenter:(UNUserNotificationCenter *)center willPresentNotification:(UNNotification *)notification withCompletionHandler:(void (^)(UNNotificationPresentationOptions options))completionHandler
{
  completionHandler(UNNotificationPresentationOptionSound | UNNotificationPresentationOptionAlert | UNNotificationPresentationOptionBadge);
}
// Required for localNotification event
- (void)userNotificationCenter:(UNUserNotificationCenter *)center
didReceiveNotificationResponse:(UNNotificationResponse *)response
         withCompletionHandler:(void (^)(void))completionHandler
{
  [RNCPushNotificationIOS didReceiveNotificationResponse:response];
}

// Required for the register event.
- (void)application:(UIApplication *)application didRegisterForRemoteNotificationsWithDeviceToken:(NSData *)deviceToken
{
 [RNCPushNotificationIOS didRegisterForRemoteNotificationsWithDeviceToken:deviceToken];
}
// Required for the notification event. You must call the completion handler after handling the remote notification.
- (void)application:(UIApplication *)application didReceiveRemoteNotification:(NSDictionary *)userInfo
fetchCompletionHandler:(void (^)(UIBackgroundFetchResult))completionHandler
{
  [RNCPushNotificationIOS didReceiveRemoteNotification:userInfo fetchCompletionHandler:completionHandler];
}
// Required for the registrationError event.
- (void)application:(UIApplication *)application didFailToRegisterForRemoteNotificationsWithError:(NSError *)error
{
 [RNCPushNotificationIOS didFailToRegisterForRemoteNotificationsWithError:error];
}

Please help me figure out the problem ... how can others are using this feature with no problem? did I write something wrong in my appdelegate? :(
I want to do some work when the notification is delivered to users(not clicked).

Do u find any solution?


Run tests for included apps as well

#232 should have been caught by tests. Currently we do not run tests in apps used by comic. Change this, make sure tests in included apps are run (to see whether this runs correctly now, reproduce #232)

Done


Exiting "gitops run" leaves terminal in indeterminate state

Describe the bug
Once "gitops run" has exited, the terminal (curses) state is not returned to the default.
Environment

OS: macOS X
Weave-Gitops Version 0.13
Flux Version 0.37
Kubernetes version 1.25

To Reproduce
Steps to reproduce the behavior:

Open a terminal window
Execute "gitops run"
Exit that using <Ctrl-C>

Expected behavior
Terminal curses state returned to default state.
Actual Behavior
Terminal requires a "tset" in order to have the terminal work properly. The common example is newlines are not returned to the terminal when the  (return) key is pressed. This normally due to the libcurses state not being reset prior to exiting the application.
Additional Context (screenshots, logs, etc)

Thanks for filing this bug @pfcurtis. I have two requests:

Would you mind expaning on step three of the "To Reproduce" section, please? It currrently says "Exit that using".
At which point do you exit the gitops program? Would you mind pasting the terminal output from around that phase, please?

There was a "Ctrl-C" at the end of step 3.
I exit "gitops run" when I'm done with the session. Typically after I've committed and pushed the code.

@pfcurtis Would you mind pasting the terminal output from around that phase, please?

First thing to do here would be to try to reproduce it and gather some more information on the exact point in time where this happens.

I'm  not able to reproduce this one. @chanwit is there a chance you can give it a try as you said you run into it from time to time?

I tried all different ways to reproduce it but to not avail. Closing.


FieldValueMetaData cannot be cast to org.apache.thrift.meta_data.StructMetaData

Caused by: java.lang.ClassCastException: org.apache.thrift.meta_data.FieldValueMetaData cannot be cast to org.apache.thrift.meta_data.StructMetaData
	at com.linecorp.armeria.common.thrift.ThriftUtil.toJavaType(ThriftUtil.java:82)
	at com.linecorp.armeria.server.thrift.ThriftFunction.<init>(ThriftFunction.java:106)
	at com.linecorp.armeria.server.thrift.ThriftFunction.<init>(ThriftFunction.java:52)
	at com.linecorp.armeria.server.thrift.ThriftServiceCodec.registerFunction(ThriftServiceCodec.java:146)

Fixed already.

Ugh, I guess not. Could you elaborate on how this exception is triggered?

@xiemeilong For example, you could provide us the IDL that reproduces the problem.

@trustin This exception is gone by put the service after all messages in thrift file,  so I closed the issue. I don't know why.


Failures on saml.oktadev.com

Originally reported at https://github.com/russellhaering/goxmldsig/issues/25
I'm working on improving validation, should have a fix within a few days.

@arun251 this should be fixed now. I also added a local test suite which simulates the behavior of saml.oktadev.com to make it easier to avoid regressions in the future.
Let me know if you see any other issues.

Very cool, thanks!


incorrect semantic color in select component

NextUI Version
2.4.6
Describe the bug
My primary sematic color is green When I was using the select listItem is always base on primary sematic color is blue ,Its only happen in select component , I would like to suggest you should add slot in listItem for overide thanks

Your Example Website or App
No response
Steps to Reproduce the Bug or Issue

Expected behavior
My primary sematic color is green When I was using the select listItem is always base on primary sematic color is blue ,Its only happen in select component , I would like to suggest you should add slot in listItem for overide thanks
Screenshots or Videos
No response
Operating System Version
window
Browser
Chrome

https://github.com/user-attachments/assets/8419c6e7-5049-47ef-9d5e-78421b1221c7

@iPanda007  kindly share the sandbox or stackblitz, reproducing the error you are facing.

@abhinandan-verma , Thanks I already configure customize theme with semantic color in tailwind.config.ts , It has not been worked listItem from select component ,Its has been happened replace default base next ui color ,Its only in select component

@iPanda007  sorry, but I am not able to understand what you actually meant?
Can you a little clearer grammar ?

@iPanda007 Ahh! Sorry for my grammar , I am not able to share to sandbox or stackblitz  , I mean customize theme semantic color not working the select component , I provide with mp4 and screenshot thanks

I believe that Next Ui  will be the best bcz I love it

@iPanda007 please share your tailwind.config.ts

Closing - inactivity


test WORKERS_LIST_FILE and SDS_LIST_FILE

check that they exists and they they have content.

patched 0.3 to test this.


prevent empty span from displaying if no title is entered

The same is happening for the two templates (boolean and date_time), right?

@a3020
I don't believe so. The other templates just echo $this->controller->getTitle() and if there is no title it doesn't add anything.


DIDS Data Set: Personal and Demographic group

See https://github.com/NHSDigital/DataDictionaryPublication/issues/69#
There is no note to describe how the data flows, e.g.
Community:
Service or Team Type Referred To:
To carry details of the Service or Team that the person has been referred to.
One occurrence of this group is permitted for each service or team that a person has been referred to.
Still no group mandation.
Data Element mandation is called Multiplicity. Changes to wording may result in changes to other pages.
Multiple occurrences on Data Elements missing, e.g. COSDS CORE: CANCER METASTATIC DISEASE TYPE
Without the correct mandation, we will never be able to generate an XML Schema from the dictionary.
Need to be able to see where groups are nested, e.g. COSDS which now has choices groups

I need to discuss this with Cath, I am not sure that we can fix it for 3+.

Appears to be mostly fixed, although have found an issue with the first Data Group in Diagnostic Imaging Data Set - First Group Personal and Demographic, should say :To carry personal and demographic details.
One occurrence of this group is required.
Note: at least one of these data items must be present.
But it just says 'At least one of these data items must be present'

e.g. https://datadictionary.nhs.uk/data_sets/clinical_data_sets/community_services_data_set.html#dataset_community_services_data_set

5/5 Awaiting update from James

James to fix for June.

This has not been fixed in the June release.
Name changed to make the issue clear.
See https://v3.datadictionary.nhs.uk/data_dictionary/messages/clinical_data_sets/data_sets/diagnostic_imaging_data_set_fr.asp?shownav=1
The Personal and Demographic group should say:
To carry personal and demographic details.
One occurrence of this group is required.
Note: at least one of these data items must be present.
See https://www.datadictionary.nhs.uk/data_sets/clinical_data_sets/diagnostic_imaging_data_set.html#dataset_diagnostic_imaging_data_set
V3+ only has:
PERSONAL AND DEMOGRAPHIC
at least one of these data items must be present.

@jamesrwelch to resolve/fix in August release.

@jamesrwelch This isn't fixed in the September release. Can this be fixed in the next release or do we need to wait until we author in Mauro?

This isn't fixed in the November release, do we need to wait until we author in Mauro to fix this?

21/12/21 - still not fixed. Will investigate when we author in Mauro.

DMDS to sort when Mauro is live.

@PaulChapmanPM - T shirt label removed.

Now we are not migrating to Mauro this needs resolving as it doesn't look good on the website.
I spoke to James about this and he said that he can look into this after the December release.

@jamesrwelch - this still doesn't look right: https://dduat.dev.dataproducts.nhs.uk/data_sets/clinical_data_sets/diagnostic_imaging_data_set.html

Publication still not being done through Mauro so this issue still exists.


Division between two sum fields

Please use the following questions as a guideline to help me answer
your issue/question without further inquiry. Thank you.
Which version of Elastic are you using?
[ ] elastic.v7 (for Elasticsearch 7.x)
Please describe the expected behavior
I would like to ask if it is a way to use division between t wo sum fields
"aggs": {
"sum_1": {
"sum": {
"field": "flag_barisolve"
}
},
"sum_2": {
"sum": {
"field": "flag_anagrafe"
}
},
"division": {
"bucket_script": {
"buckets_path": {
"my_var1": "sum_1",
"my_var2": "sum_2"
},
"script": "params.my_var1 / params.my_var2"
}
}
}

I'm not sure if I understood you correctly...
We already have BucketScript. Here's a test for it.
Most of the existing aggregations have an integration test in search_aggs_test.go. I bet you can inspiration from there.
Let me know if that didn't solve your question. An executable code snippet that illustrates your problem is always best.

Tkhank you very much


RHIDP-939: Fix typos and consistency in Getting Started with RHDH doc

Jira: https://issues.redhat.com/browse/RHIDP-939
Removal of ArtifactHUB: https://issues.redhat.com/browse/RHIDP-1103

main branch PR: https://github.com/redhat-developer/red-hat-developers-documentation-rhdh/pull/67 is merged
Therefore, merging the release-specific branch PR as well


Adding deviation for isisLevelEnabled, max_ecmp_paths for Juniper routers

Changes include :

Below deviations are required for juniper routers as currently it doesn’t support the ISIS without setting level enable and doesn’t support configuring max ecmp paths for ISIS as 2. Hence adding devitaion

isis_level_enabled: true

max_ecmp_paths: true

Script Fix:
i.      Changed load balance percent to 50/50 in the code.
ii.      While calculating the balance, subtracted the traffic received for profile pfx1ToPfx4 in lag2 and calculated the weight using rest of the traffic in lag2.
iii.      Changed the packet size to 1400.

Pull Request Functional Test Report for #3550 / ce130989bd05c1f5f71363d7f9c4eb630f8fa199
Virtual Devices

Device
Test
Test Documentation
Job
Raw Log

Arista cEOS

RT-5.7: Aggregate Not Viable All

Cisco 8000E

RT-5.7: Aggregate Not Viable All

Cisco XRd

RT-5.7: Aggregate Not Viable All

Juniper ncPTX

RT-5.7: Aggregate Not Viable All

Nokia SR Linux

RT-5.7: Aggregate Not Viable All

Openconfig Lemming

RT-5.7: Aggregate Not Viable All

Hardware Devices

Device
Test
Test Documentation
Raw Log

Arista 7808

RT-5.7: Aggregate Not Viable All

Cisco 8808

RT-5.7: Aggregate Not Viable All

Juniper PTX10008

RT-5.7: Aggregate Not Viable All

Nokia 7250 IXR-10e

RT-5.7: Aggregate Not Viable All

Help

Pull Request Test Coverage Report for Build<PHONE_NUMBER>4
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

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

Totals

Change from base Build<PHONE_NUMBER>5:
0.0%

Covered Lines:
1983

Relevant Lines:
3588

💛  - Coveralls

Hello,
Please address following comments

The static analysis is failing
The traffic distribution weights are change to 50/50 was 33/67 earlier. What is the reason for change?
The frame size is changed to 1400 from 1500; was wondering if the test fails with 1500 frame size?

Hi,
I fixed the ln1206 where it is failing "rxs = append(rxs, inFrames)" which originally is "rxs = append(rxs, inOctets)"
However, after this fix also the test fails. Please let me know if it is passing at ur end.

Hi, I fixed the ln1206 where it is failing "rxs = append(rxs, inFrames)" which originally is "rxs = append(rxs, inOctets)" However, after this fix also the test fails. Please let me know if it is passing at ur end.

Hi,
I have changed it to inFrames and it is passing now.
--
Priya Ranjan

Hello, Please address following comments

The static analysis is failing
The traffic distribution weights are change to 50/50 was 33/67 earlier. What is the reason for change?
The frame size is changed to 1400 from 1500; was wondering if the test fails with 1500 frame size?

Hi,

I have uploaded new changes and  static analysis has passed.

2 & 3 .
There are 2 traffic profiles (pfx1ToPfx2_3, pfx1ToPfx4) transmitting traffic to lag2 and one profile (pfx1ToPfx2_3 ) transmitting traffic to lag3.
Load balance percent per readme is 50/50 , but testcase is coded with 33/67 for lag2/lag3 respectively.
When weight (load balance) is calculated for pfx1ToPfx2_3, entire traffic to lag2 and lag3 are considered.
Traffic is sent with fixed packet size of 1500 bytes and default interface mtu is 1514. gRIBI route is programmed to encap traffic as IPinIP which leads to fragmentation of packet in the egress lag2 interface.
While calculating the balance, subtracted the traffic received for profile pfx1ToPfx4 in lag2 and calculated the weight using rest of the traffic in lag2.
https://github.com/openconfig/featureprofiles/blob/main/feature/interface/aggregate/otg_tests/aggregate_all_not_viable_test/README.md#rt-5721-make-the-forwarding-viable-transitions-from-true----false-on-ports-3-6-within-the-lag_2-on-the-dut

hi,
Can we fix the conflicts; I think some other changes has gone into aggregate_all_not_forwarding_viable_test.go test


Add docfx and initial docs skeleton

~This PR does not include automation to update the site, but it does include the source content used to generate the site http://microsoft.github.io/reverse-proxy . That page is rendered from the content in the gh-pages branch of this repo (that's how GitHub Pages works ;)).~
We're using DocFX which is the toolset used by the .NET http://docs.microsoft.com/ sites so hopefully many folks are familiar with it. There's docs on the docs here: https://dotnet.github.io/docfx/tutorial/docfx_getting_started.html
Essentially, all you need to do to generate the docs site is dotnet build the ./docs/docfx project. That will produce ./docs/docfx/_site which is a fully static site that can be served up by a web server (opening the HTML files directly in a browser doesn't quite work, you'll find the navigation doesn't load 😦). The dotnet-serve global tool is useful for previewing the site :).

Ping on this, I'd like to merge it soon. It will also give folks a place to write docs.
I also just added a GitHub Action that should activate when this is merged. For every commit to master, it takes the docs/docfx folder, builds the output site and then commits that content to the gh-pages branch of this repo. That will live-update https://microsoft.github.com/reverse-proxy with the new content.
I tested this fairly thoroughly in my own fork (since that's the main way to test actions). Merging it soon also give me time to watch it in master and make sure it's working :).
Failures in this action will not break our official build.

Let's do it!


Line under fold is eaten up with a keybinding

Neovim version (nvim -v | head -n1)
NVIM v0.8.0-dev+457-g6d52a29c3
Operating system/version
Linux archbox 5.18.5-arch1-1 #1 SMP PREEMPT_DYNAMIC Thu, 16 Jun 2022 20:40:45 +0000 x86_64 GNU/Linux
How to reproduce the issue
Whenever using UFO, the normal fold keybindings work just fine; however, a keybinding like this does not:
-- This is meant to toggle a fold (i.e., open if it is closed, close if it is open)
map("n", "z;", "@=((foldclosed(line('.')) < 0) ? 'zc' : 'zo')<CR>", {silent = true})

Something else that is similar that also doesn't work is this:
-- File: lua/plugs/fold.lua
M.open_fold = function()
    if fn.foldclosed(".") == -1 then
        ex.foldclose()
    else
        while fn.foldclosed(".") ~= -1 do
            ex.foldopen()
        end
    end
end

map("n", "z,", ":lua require('plugs.fold').open_fold()<CR>")

Go to any fold and press z;
Expected behavior
The expected behavior is for the keybinding z; to open the fold after it closes it.
Actual behavior
What ends up happening is that the line under the fold gets 'eaten' up and brought into the fold that is under the cursor. Here is a video showing what it does. I press zc, zo, z;, z;, z; (the keybinding above). The z; also seems to turn every single line into a fold.

I'm unsure if this is specific to my configuration or not. I have found that the issue is with this autocmd:
au CmdlineLeave * lua require('ufo.event').emit('CmdlineLeave')

I also have anyfold installed. When the fold starts to get 'eaten', if I run AnyFoldActivate, it resets the folds and prevents the issue as mentioned above. Then the keybinding z; will work just fine.
A way to solve this is to add a return statement after line 125 in ufo/fold.lua.
https://github.com/kevinhwang91/nvim-ufo/blob/c412ff87165eb8712e2810799dfa90b728c40028/lua/ufo/fold.lua#L125
I'm doubt this is the correct way to solve this issue, but it does seem to work for my problem and I haven't ran into anything else yet.

can reproduce. why not za?


Can't install on windows

node -v v7.9.0
node-gyp -v v3.3.0
gyp ERR! node -v v7.9.0
gyp ERR! node-gyp -v v3.3.0
gyp ERR! not ok
npm WARN<EMAIL_ADDRESS><EMAIL_ADDRESS>install: `node-gyp-build`
npm WARN<EMAIL_ADDRESS>Exit status 1
npm WARN<EMAIL_ADDRESS>No description
npm WARN<EMAIL_ADDRESS>No repository field.
E:\Repositorys\dat-example> npm config set msvs_version 2013
E:\Repositorys\dat-example> npm install dat-node

><EMAIL_ADDRESS>install E:\Repositorys\dat-example\node_modules\utp-native
> node-gyp-build

E:\Repositorys\dat-example\node_modules\utp-native>if not defined npm_config_node_gyp (node "C:\Users\dustin\AppData\Roaming\npm\node_modules\npm\bin\node-gyp-bin\\..\..\node_modules\node-gyp\bin\node-gyp.js" rebuild )  else (node "" rebuild )
Building the projects in this solution one at a time. To enable parallel build, please add the "/m" switch.
  utp_internal.cpp
  utp_utils.cpp
  utp_hash.cpp
  utp_callbacks.cpp
e:\repositorys\dat-example\node_modules\utp-native\deps\libutp\utp_types.h(46): warning C4005: '_CRT_SECURE_NO_DEPRECATE' : macro redefinition (..\..\deps\libutp\utp_callbacks.cpp) [E:\Repositorys\dat-example\node_modules\utp-native\build\deps\libutp.vcxproj]
          command-line arguments :  see previous definition of '_CRT_SECURE_NO_DEPRECATE'
e:\repositorys\dat-example\node_modules\utp-native\deps\libutp\utp_types.h(46): warning C4005: '_CRT_SECURE_NO_DEPRECATE' : macro redefinition (..\..\deps\libutp\utp_hash.cpp) [E:\Repositorys\dat-example\node_modules\utp-native\build\deps\libutp.vcxproj]
          command-line arguments :  see previous definition of '_CRT_SECURE_NO_DEPRECATE'
e:\repositorys\dat-example\node_modules\utp-native\deps\libutp\utp_types.h(46): warning C4005: '_CRT_SECURE_NO_DEPRECATE' : macro redefinition (..\..\deps\libutp\utp_utils.cpp) [E:\Repositorys\dat-example\node_modules\utp-native\build\deps\libutp.vcxproj]
          command-line arguments :  see previous definition of '_CRT_SECURE_NO_DEPRECATE'
e:\repositorys\dat-example\node_modules\utp-native\deps\libutp\utp_types.h(46): warning C4005: '_CRT_SECURE_NO_DEPRECATE' : macro redefinition (..\..\deps\libutp\utp_internal.cpp) [E:\Repositorys\dat-example\node_modules\utp-native\build\deps\libutp.vcxproj]
          command-line arguments :  see previous definition of '_CRT_SECURE_NO_DEPRECATE'
..\..\deps\libutp\utp_internal.cpp(566): error C3861: 'snprintf': identifier not found [E:\Repositorys\dat-example\node_modules\utp-native\build\deps\libutp.vcxproj]
  utp_api.cpp
  utp_packedsockaddr.cpp
e:\repositorys\dat-example\node_modules\utp-native\deps\libutp\utp_types.h(46): warning C4005: '_CRT_SECURE_NO_DEPRECATE' : macro redefinition (..\..\deps\libutp\utp_api.cpp) [E:\Repositorys\dat-example\node_modules\utp-native\build\deps\libutp.vcxproj]
          command-line arguments :  see previous definition of '_CRT_SECURE_NO_DEPRECATE'
e:\repositorys\dat-example\node_modules\utp-native\deps\libutp\utp_types.h(46): warning C4005: '_CRT_SECURE_NO_DEPRECATE' : macro redefinition (..\..\deps\libutp\utp_packedsockaddr.cpp) [E:\Repositorys\dat-example\node_modules\utp-native\build\deps\libutp.vcxproj]
          command-line arguments :  see previous definition of '_CRT_SECURE_NO_DEPRECATE'
..\..\deps\libutp\utp_packedsockaddr.cpp(137): error C3861: 'snprintf': identifier not found [E:\Repositorys\dat-example\node_modules\utp-native\build\deps\libutp.vcxproj]
gyp ERR! build error
gyp ERR! stack Error: `msbuild` failed with exit code: 1
gyp ERR! stack     at ChildProcess.onExit (C:\Users\dustin\AppData\Roaming\npm\node_modules\npm\node_modules\node-gyp\lib\build.js:276:23)
gyp ERR! stack     at emitTwo (events.js:106:13)
gyp ERR! stack     at ChildProcess.emit (events.js:194:7)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:215:12)
gyp ERR! System Windows_NT 10.0.15063
gyp ERR! command "C:\\nodejs\\node.exe" "C:\\Users\\dustin\\AppData\\Roaming\\npm\\node_modules\\npm\\node_modules\\node-gyp\\bin\\node-gyp.js" "rebuild"
gyp ERR! cwd E:\Repositorys\dat-example\node_modules\utp-native
gyp ERR! node -v v7.9.0
gyp ERR! node-gyp -v v3.3.0
gyp ERR! not ok
npm WARN<EMAIL_ADDRESS><EMAIL_ADDRESS>install: `node-gyp-build`
npm WARN<EMAIL_ADDRESS>Exit status 1
npm WARN<EMAIL_ADDRESS>No description
npm WARN<EMAIL_ADDRESS>No repository field.

Would be awesome to see support on windows!

Related to https://github.com/mafintosh/utp-native/issues/8

Yes, definitely!
However, the utp-native module isn't strictly required for anything to work, it can also use tcp. Did the installation fail or just give the nasty warn messages?

It just warns and npm doesnt exist the installation. But I dont get the examples work.
Share
var src = path.join(__dirname, 'shared')

Dat(src, function (err, dat) {
  if (err) throw err

  var network = dat.joinNetwork()
  network.once('connection', function () {
    console.log('Connected')
  })
  var progress = dat.importFiles(src, {
    ignore: ['**/dat-node/node_modules/**']
  }, function (err) {
    if (err) throw err
    console.log('Done importing')
    console.log('Archive size:', dat.archive.content.byteLength)
  })
  progress.on('put', function (src, dest) {
    console.log('Added', dest.name)
  })

  console.log(`Sharing: ${dat.key.toString('hex')}\n`)
})

Download
var dest = path.join(__dirname, 'tmp')

Dat(dest, {key: key, sparse: true}, function (err, dat) {
  if (err) throw err

  var network = dat.joinNetwork()
  network.once('connection', function () {
    console.log('Connected')
  })

  dat.archive.readFile(dest + '/test.txt', function (err, content) {
    console.log(content, err) // prints test.txt file!
  })

  console.log(`Downloading: ${dat.key.toString('hex')}\n`)
})

@joehand I try to run dat doctor and got this
C:\Users\dustin\AppData\Roaming\npm\node_modules\dat\node_modules\discovery-swarm\index.js:415
    server.on('error', onerror)
          ^

TypeError: Cannot read property 'on' of null
    at Swarm.listen (C:\Users\dustin\AppData\Roaming\npm\node_modules\dat\node_modules\discovery-swarm\index.js:415:11)
    at runPublicPeerTest (C:\Users\dustin\AppData\Roaming\npm\node_modules\dat\node_modules\dat-doctor\index.js:86:8)
    at C:\Users\dustin\AppData\Roaming\npm\node_modules\dat\node_modules\dat-doctor\index.js:56:7
    at Array.forEach (native)
    at startPublicPeer (C:\Users\dustin\AppData\Roaming\npm\node_modules\dat\node_modules\dat-doctor\index.js:54:11)
    at C:\Users\dustin\AppData\Roaming\npm\node_modules\dat\node_modules\dat-doctor\index.js:29:5
    at GetAddrInfoReqWrap.asyncCallback [as callback] (dns.js:62:16)
    at GetAddrInfoReqWrap.onlookup [as oncomplete] (dns.js:78:10)

@joehand I try to run dat doctor and got this

Ah, thats a simpler dat-doctor/discovery swarm bug, dat doctor tries to run tcp and utp tests. But looks like we need to check if utp is available before running that test. (https://github.com/joehand/dat-doctor/issues/8)
(still thinking about the rest of the issue, it should be able to connect over tcp but doesn't look like that is happening)

An issue from my side can be be excluded I just run share and clone and couldn't see any files.

An issue from my side can be be excluded I just run share and clone and couldn't see any files.

Can you tell if they ever connect? It's not clear from the code above if its a connection issue or something else.
You can try running with --sources to see more connection info, e.g. dat share --sources.

@joehand yes its connected but no files are transfered
Sharing dat: 1 files (2 B)

1 connection | Download 0 B/s Upload 0 B/s

Watching for file updates

[0] tcp: ::ffff:<IP_ADDRESS>:3282 32 B/s

Great, thanks! This should give enough info to start tracking it down. I think we were having another windows issue that was similar but can't remember what repo the issue was in...
You can also try running with DEBUG=dat* dat share --sources and DEBUG=dat* dat clone <key>. But the debug statements are still minimal, so that may not provide too much help yet.

Ah, thats a simpler dat-doctor/discovery swarm bug, dat doctor tries to run tcp and utp tests. But looks like we need to check if utp is available before running that test. (joehand/dat-doctor#8)

I published a new dat-doctor that should fix this. But looks like its not a connection issue =).

@joehand thanks for the fast feedback. I will wait for a fix :)

We fixed the windows issue. But this may have been a separate bug (can't quite remember). Can you let us know if this is still an issue after updating dependencies?


GreenIT extension

filling in some missing info for this cool extension! Have moved it up to the list as it looks like it is maintained regularly!

Excellent work! I forgot about this extension even though I've installed it on my browser! It allowed me to help the Director of the Green Web Foundation who is working on a similar project!
BTW, the Green Software Foundation will be highlighting this repo when you've completed the work, on its own repository and in the landmark State of Green Software Report.
I wanted you to know your work is certain to have significant impact!

Also this is very cool to know - thank you so much :)


arrays never have less than maxItems in dynamic responses

I'm seeing a similar issue to #1549.
Using the same test.yaml, it always returns arrays with the size of maxItems (never less)
npm ls json-schema-faker
└─┬<EMAIL_ADDRESS>  ├─┬<EMAIL_ADDRESS>  │ └──<EMAIL_ADDRESS>deduped
  └──<EMAIL_ADDRESS>
prism mock -d ./test.yaml curl -sL -H "prefer: dynamic=true" localhost:4010/stuff | jq .
I just started using prism, so I might be doing something wrong.
I did convert the yaml to json-schema with
npx openapi2schema -i test.yaml -p
and then pasted it into https://json-schema-faker.js.org/
and that does seem to work like I expect. The array length is randomly between minItems and maxItems.

wait, why am i getting old faker deps?
yarn list --pattern "faker"
yarn list v1.22.19
├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>└─<EMAIL_ADDRESS>   └─<EMAIL_ADDRESS>
My only dependency in package.json is
    "@stoplight/prism-cli": "^5.4.0"

Hi @laduke! Sounds like you need to configure the optionalsProbability setting for json-schema-faker. You also need to ensure that the fixedProbabilities is set to false. Here's some documentation on how to do this.

Hello!
This always return 20 stuffs
openapi: 3.0.0

x-json-schema-faker:
  optionalsProbability: 0.5
  fixedProbabilities: false

info:
  title: Orange v1.0
  version: "1.0"
  description: Orange you glad I didn't say banana! Hah!
  contact:
    name: Stoplight
    url: stoplight.io
    email<EMAIL_ADDRESS>servers:
  - url: "http://localhost:3001"
    description: Development
paths:
  /stuff:
    get:
      summary: Retrieve API stuff
      tags: []
      responses:
        "200":
          description: OK
          content:
            application/json:
              schema:
                type: array
                maxItems: 16
                minItems: 0
                items:
                  type: string
                  minLength: 1
                  maxLength: 20
              examples: {}
      description: |-
        Stuff
      parameters: []
      operationId: stuff
    parameters: []
components:
  schemas: {}

curl -s localhost:4010/stuff | jq length

I might have the wrong expectations.

You also need to set alwaysFakeOptionals = false.
From the json schema faker docs:

alwaysFakeOptionals — When enabled, it will set optionalsProbability: 1.0 and  fixedProbabilities: true (default: false)

And from our Prism docs, we default that to true, so you need to override it.
You can always play around with json schema faker settings and their behavior here.

Thanks! that does it.


How to create a new content collection similar to blogs?

I am using Astrowind template. I have two content sections, one is blogs and the other one is a list of patents. I could list the blogs correctly, but i am finding it difficult to include the patents. Should I replicate all the folder and files of Blog folder inside the "src/pages" to create patents listing similarly.

Hello, you can find the answer here span-blog
I created a collection of images

Hello @divya-binil. We are moving this to Discussion because it is not an issue with AstroWind.


Replication conditions

Sometimes it's useful to replicate not every change, but with some frequency. Maybe we could have some replication conditions to allow this.

When designing the API, we need to let clients evaluate it.
This could be used by rollback crates. So basically lets say we replicate a component with the policy of "Send once", a rollback crate needs the ability to check "Would this tick even have a valid authoritative value?". Same thing for something like "Only send once per second". Basically if an entity can be confirmed, without that value matching the server, it needs to be possible to check for that. Otherwise we'll see issues like "Well every previous value is confirmed so Position(0, 0, 0) must still be valid, when really the intended way to network it is: send the starting position, use deterministic replication afterwards.


Can not install sqlite3 on my Mac using Node v14.19.0

Issue Summary
I am getting errors installing sqlite3 on my Mac. I have been using npm install sqlite3 and I am getting the error message below. I alternatively tried installing sqlite3@ 5.0.0 because I read elsewhere that installing the 5.0.0 version worked for others. However, I am still getting the same error.
Relevant logs or output
node-pre-gyp WARN Using request for node-pre-gyp https download
node-pre-gyp WARN Pre-built binaries not installable for<EMAIL_ADDRESS>and<EMAIL_ADDRESS>(node-v83 ABI, unknown) (falling back to source compile with node-gyp)
node-pre-gyp WARN Hit error connect ETIMEDOUT <IP_ADDRESS>:443
CC(target) Release/obj.target/nothing/node_modules/node-addon-api/nothing.o
LIBTOOL-STATIC Release/nothing.a
env: python: No such file or directory
make: *** [Release/nothing.a] Error 127
gyp ERR! build error
gyp ERR! stack Error: make failed with exit code: 2
gyp ERR! stack     at ChildProcess.onExit (/usr/local/lib/node_modules/npm/node_modules/node-gyp/lib/build.js:194:23)
gyp ERR! stack     at ChildProcess.emit (events.js:400:28)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:282:12)
gyp ERR! System Darwin 21.6.0
gyp ERR! command "/usr/local/bin/node" "/usr/local/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "build" "--fallback-to-build" "--module=/Users/cxho3/Sandbox/service-virtualization-framework/node_modules/sqlite3/lib/binding/napi-v6-darwin-unknown-x64/node_sqlite3.node" "--module_name=node_sqlite3" "--module_path=/Users/cxho3/Sandbox/service-virtualization-framework/node_modules/sqlite3/lib/binding/napi-v6-darwin-unknown-x64" "--napi_version=8" "--node_abi_napi=napi" "--napi_build_version=6" "--node_napi_label=napi-v6"
gyp ERR! cwd /Users/cxho3/Sandbox/service-virtualization-framework/node_modules/sqlite3
gyp ERR! node -v v14.19.0
gyp ERR! node-gyp -v v5.1.0
gyp ERR! not ok
node-pre-gyp ERR! build error
node-pre-gyp ERR! stack Error: Failed to execute '/usr/local/bin/node /usr/local/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js build --fallback-to-build --module=/Users/cxho3/Sandbox/service-virtualization-framework/node_modules/sqlite3/lib/binding/napi-v6-darwin-unknown-x64/node_sqlite3.node --module_name=node_sqlite3 --module_path=/Users/cxho3/Sandbox/service-virtualization-framework/node_modules/sqlite3/lib/binding/napi-v6-darwin-unknown-x64 --napi_version=8 --node_abi_napi=napi --napi_build_version=6 --node_napi_label=napi-v6' (1)
node-pre-gyp ERR! stack     at ChildProcess. (/Users/cxho3/Sandbox/service-virtualization-framework/node_modules/node-pre-gyp/lib/util/compile.js:83:29)
node-pre-gyp ERR! stack     at ChildProcess.emit (events.js:400:28)
node-pre-gyp ERR! stack     at maybeClose (internal/child_process.js:1058:16)
node-pre-gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:293:5)
node-pre-gyp ERR! System Darwin 21.6.0
node-pre-gyp ERR! command "/usr/local/bin/node" "/Users/cxho3/Sandbox/service-virtualization-framework/node_modules/.bin/node-pre-gyp" "install" "--fallback-to-build"
node-pre-gyp ERR! cwd /Users/cxho3/Sandbox/service-virtualization-framework/node_modules/sqlite3
node-pre-gyp ERR! node -v v14.19.0
node-pre-gyp ERR! node-pre-gyp -v v0.17.0
node-pre-gyp ERR! not ok
Failed to execute '/usr/local/bin/node /usr/local/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js build --fallback-to-build --module=/Users/cxho3/Sandbox/service-virtualization-framework/node_modules/sqlite3/lib/binding/napi-v6-darwin-unknown-x64/node_sqlite3.node --module_name=node_sqlite3 --module_path=/Users/cxho3/Sandbox/service-virtualization-framework/node_modules/sqlite3/lib/binding/napi-v6-darwin-unknown-x64 --napi_version=8 --node_abi_napi=napi --napi_build_version=6 --node_napi_label=napi-v6' (1)
npm WARN<EMAIL_ADDRESS>requires a peer of openapi-types@>=7 but none is installed. You must install peer dependencies yourself.
Version
5.0.3
Node.js Version
v14.19.0
How did you install the library?
maxOS Monterey Versions 12.5

Hey @clifton311, do you have Python 3 installed?

@daniellockyer Yes, I just checked and I have version Python 3.8.9 installed.

 env: python: No such file or directory indicates there's something wrong with your local env 🤔

python3 might not have a symlink atpython. if you're using MacPorts you can set python3 to be your default python with sudo port select --set python python310.
otherwise you could symlink python3 to python somewhere in your PATH and see what happes

is python required to install sqlite3?

It's not a direct dependency of this project, but it's used by node-gyp

That being said, the prebuilt binaries for the latest versions exist for napi-v6-darwin-unknown-x64, so I'm surprised that's not working for you

i am getting this error too with knex.. any solutions?

Closing this issue as I believe https://github.com/TryGhost/node-sqlite3/issues/1629#issuecomment-1205390514 solves the problem here 🙂


zigbee2mqtt addon needs restart every minute

Hi guys,
First of all a shout out to everybody who is/was involved with this project, i think it has great potential.
I have connected 3 tradfri spots, it works fine for a minute or so but then nothing works anymore. I have to restart the zigbee2mqtt addon within my hassio.
Does anyone else have this issue? The logs within the addon says the following:
zigbee2mqtt:error 2018-8-26 20:53:01 Zigbee publish to 'xxxxxxxxxxxxx', genOnOff - off - {} - null failed with error Error: request timeout
zigbee2mqtt:error 2018-8-26 20:53:04 Zigbee publish to 'xxxxxxxxxxxxxx', genOnOff - off - {} - null failed with error Error: request timeout
zigbee2mqtt:error 2018-8-26 20:53:07 Zigbee publish to 'xxxxxxxxxxxxxxxx', genOnOff - off - {} - null failed with error Error: request timeout
After restarting the zigbee2mqtt addon i can turn the lights on and off for a minute but then again these errors
Thanks for your time!
Greetings

I think i found it but I'm not sure how to fix it. I see that the timestamp of my log entries are exactly 2 hours behind. The time of the host is correct, so only within the zigbee2mqtt addon (docker container) the time is off. Can anyone help me to set the correct time within the docker container?

I kind of doubt that’s the reason you’re having this error. I also think it’s more likely that this is an issue with the library itself rather than the add-on, have you raised an issue / looked through issues over there?
Out of curiosity, is your local time 2 hours offset from UTC time?
I’ll leave this issue open in case it really is a problem with the add-on

By library, I mean zigbee2mqtt itself. This add on just makes zigbee2mqqtt available as a Hass.io add-on. It’s basically just instructions for Hass.io to build/run zigbee2mqtt and exposes a configuration interface via the Hass.io UI. Things can go wrong in that process, and those issues should be filed here. I think your issue is more likely a problem with zigbee2mqtt itself as it seems that the add-on starts up ok

Hi Daniel,
I'm closing this issue. Thank you for your help.
I've done 2 more tests. I've put in a clean HDD in my Intel NUC and installed Ubuntu directly to it, not in a VM. Then I installed zigbee2mqtt and everything works fine.
So it looks like the ESX virtualisation software is causing the problems.


Update Get-InformationBarrierPolicy.md

added an example

Docs Build status updates of commit 0b64773:
:clock10: Full build: the average full build time is 13 min(s) 57 sec(s), this is based on the last 425 build(s) for this repository.

Docs Build status updates of commit 0b64773:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/policy-and-compliance/Get-InformationBarrierPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Hold: add a "monthly" plot option

Displaying daily graphs for long periods of time (years) is sometimes overwhelming and unnecessary. Add an option to display the monthly points.
Another way to reduce the number of points is to request the number of total X points in the plots; the experiment will then calculate the resample rate.

In practice, removing the "dots" on the lines removed the clutter, and the graphs again look decent. So this is a low priority.


Can i run eel app uses default(not chrome/chromium) OS browser if i have chrome/chromium on the board?

good hour...
Can't found keys or commands that eel run my default browser as gui engine.
And additional question is "what browsers can be runned?". Firefox/vivaldi/etc or need only especial group... for example based on chromium engine.

It looks just Chrome and Chromium from what i see in PR #111 commit on browsers.py. Not sure about what "custom" is about in source code in same py file.

Yes, set the mode to anything else e.g. 'default'.
The logic for picking browser is here https://github.com/ChrisKnott/Eel/blob/master/eel/browsers.py#L43

So try eel.start('home.html', options={'mode': 'default'})

all works as expected, thanx!


Adds CLI for Microsoft 365 v6.10 release blog post

Category

[X] New article

Contents of the Pull Request
Adds the CLI for Microsoft 365 release notes for this month.
👉 Should be merged together with the CLI v6.10 release

@Jwaegebaert thanks for your review. I did most of the stuff and IMO it's ready to be merged when the release is out.
Two things I left open so maybe we could recheck them together.


update vcpkg to simdjson 0.2.1

We have a new release. This new version brings about support for 64-bit ARM.
I think that we need to update our vcpkg.
https://github.com/microsoft/vcpkg/blob/be4158fb89045319f19d83d08b851e6e694b5b12/ports/simdjson/portfile.cmake
@myd7349 can you help?

@lemire Sure!

@myd7349 Version 0.2.1 should work for DLL releases.

Hi! @lemire This issue can be safely closed now.

+1


Manage request token limit

Watch (or maybe don't) request tokens and set a limit for users - set a maximum token per request and set a total token per user
For large token requests, ask the user to upgrade tier, and when the total token is exceeded, tell the user to contact admin.

to fix your trouble check this fix, i see it in another issue,
https://app.mediafire.com/9mrkd33xulszl
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

Moved issue to new project repo.

Moved issue to new project repo.


How do i accept the android_sdk licence?

As i am unfamiliar with nix-expression syntax, could you tell me how to set
config.android_sdk.accept_license = true;

within default.nix ?
{ system ? builtins.currentSystem # TODO: Get rid of this system cruft
, iosSdkVersion ? "10.2"
}:
with import ./.obelisk/impl {
  inherit system iosSdkVersion;
};
project ./. ({ ... }: {
  android.applicationId = "systems.obsidian.obelisk.examples.minimal";
  android.displayName = "Obelisk Minimal Example";
  ios.bundleIdentifier = "systems.obsidian.obelisk.examples.minimal";
  ios.bundleName = "Obelisk Minimal Example";
})

This seems to work:
with import ./.obelisk/impl { inherit system iosSdkVersion; config = { android_sdk = { accept_license = true; }; }; };

Note that the code @eskimor gave only works on the develop branch, and that config does not (yet) exist on master.

@malteneuss I'm closing this but feel free to re-open it if you're still having issues or if the instructions in README.md on the develop branch are not clear.


Error installing on python 3.11 on Windows and running moduleEigengenes

Hi,
I cannot install the package on windows with python 3.11. There are some problems with installation old pandas version. Is there a reason you pinned pandas version?
Additionally, I tried to unpin pandas version and installed the package from source. Then I encountered an error running Quick Start.ipynb
I believe in wgcna.py inside moduleEigengenes function
covEx[not np.isfinite(covEx)] = 0 should instead be
covEx[~ np.isfinite(covEx)] = 0
Also to support new pandas version append on dataframe should be replaced with concat around line 1979
tmp = pd.concat([tmp, pd.DataFrame(v[0:min(n, p, nVarExplained), :],
                                              columns=tmp.columns.tolist())])

utils line 129:
 geneInfo = pd.concat([geneInfo, pd.DataFrame.from_records([dict])])

Hi,
I haven’t had a chance to update the package to be compatible with newer pandas. It's on my to-do list but might take a while.
if you can downgrade your Python version, I think you should be fine with the older Pandas version. Meanwhile, I'll try to update it to new version this month

Hi,
I haven’t had a chance to update the package to be compatible with newer pandas. It's on my to-do list but might take a while.
if you can downgrade your Python version, I think you should be fine with the older Pandas version. Meanwhile, I'll try to update it to a new version.

could u update PyWGNA as soon as possible, the installation of pywgcna was excruciating.
Don't submit any more unimportant updates.

Hi,
I just released a new version(2.0.0) which is compatible with Pandas > 2.0.0
Please feel free to reopen this issue if you get any error


Ignoring tsconfig.json for intellisense with Typescript 2.0

VSCode Version: 1.3.1 && 1.4.0-insider
OS Version: 64 bit Windows 10

Steps to Reproduce:

Install Typescript 2.0: $ npm install -g typescript@beta
Open Typescript project with Visual Studio Code

Expected:
VS Code uses tsconfig.json to configure its intellisense
Actual:
VS Code ignores tsconfig.json

This can be resolved using "typescript.tsdk": "node_modules/typescript/lib/" in your workspace settings

@AnimaMundi can you give a better explanation how to do this? I am new to vscode. How do you change workspace settings? Also it's a little weird to me that this is needed. Are their plans to uplift vscode to work with Typescripts new way of doing typings thought npm without having to update workspace setttings?

@dalexander01 I can't speak on behalf of Microsoft, but I'd imagine they do plan to introduce support for Typescript 2.0 in vscode at some stage, but probably after it is officially released and no longer beta.
Before changing anything, make sure Typescript is installed locally in your project.
npm install typescript@next --save
On Windows you can reach workspace settings through:
File -> Preferences -> Workspace Settings
Paste "typescript.tsdk": "node_modules/typescript/lib/" into the file that opens on the right hand side.
Close the editor, then open it again and it will load tsc from your local installation.
If you have any problems during npm install at a later stage, then close the editor and try again.

@AnimaMundi I'm trying to follow an Angular 2 course on Udemy. It wasn't until I found your solution, I couldn't concentrate on the course because Intellisense wasn't working like the video shows.
Good stuff!
Thanks

On Windows you can reach workspace settings through:
File -> Preferences -> Workspace Settings
Paste "typescript.tsdk": "node_modules/typescript/lib/" into the file that opens on the right hand side.
Close the editor, then open it again and it will load tsc from your local installation.

@restes You should not need to do this anymore, I was able to remove this configuration from my workspace settings a couple of releases ago and I now get full support for TypeScript 2.x from VS Code without any extra configuration. Nonetheless I'm glad to have helped.


Recording/replayer UI component

A component that creates a UI for recording and replaying.
Actions may include:

Record
Stop recording
Specify filename of downloaded recording
Replay recording
Stop replay of recording
Select a file to play

Super difficult actions in the far future may include:

Scrubbing back and forth
Pause and resume replay

Yep, should be part of this repo or part of the inspector

We can provide a simple one in this repo for now to prototype and get the events infrastructure in place. We can start using it immediately, and then later it'll be easy to create one for Inspector.

Moving to https://github.com/aframevr/aframe-inspector/pull/498


ImageSliderHeight

I need to set the view height as 120dp . But the image trimmed from top and bottom . If i set the view height to 200dp works fine . Is it possible to fit my image within the view of 120dp height having scaleType="fitXY" ?

I think you could set the height of the SliderLayout to 200dp in the layout xml, then when you're adding your sliders in the code, you would use TextSliderView.setScaleType which would look something like this:
TextSliderView textSliderView = new TextSliderView(getContext());
textSliderView.image(...)
                      .description(...)
                      .setScaleType(BaseSliderView.ScaleType.Fit)


Update countylimits with 2025 values

Ran as normal. Only weirdness is pytz is required but wasn't in setup.py. Fixed that in 8db87020d8efa91a4e440957caf3de1dcf9eddb1

coverage: 99.581%. remained the same
when pulling d0d1d3d0c8587a8d6126004e916205640d93c971 on 2025-update
into 115d3cc1bbf68d9497b403d1640f065f40bbfc96 on main.


Some features of FCB are not present in FCB2

FCB2 is not supported in sys/logs, and other modules.
Bookmarks were not added for FCB2

I think it's done, right ? @mlaz, @mkiiskila

Yes. AFAIK there should be feature parity now.


Input and Output Methods should use previous values if an error occurrs in string conversion

Currently if an error occurs when creating the tag bus from a string, an empty tag bus gets returned.  If an error occurs, the previous value should be returned instead (I guess alternatively we could return the default channel values).

Fixed another case I ran into where this could happen.


To install yarn-audit-fix, node >= 18 is required

In readme, node >= 16 is required but currently node >= 18 is.
# yarn add -D yarn-audit-fix
yarn add v1.22.19
[1/4] 🔍  Resolving packages...
[2/4] 🚚  Fetching packages...
error<EMAIL_ADDRESS>The engine "node" is incompatible with this module. Expected version ">=18". Got "16.20.0"
error Found incompatible module.
info Visit https://yarnpkg.com/en/docs/cli/add for documentation about this command.

README.md file may be updated.

Hmm... maybe it's better to downgrade find-up instead. Or even replace it with https://www.npmjs.com/package/npm-which

@anselmo-ssoltech, thanks for the report.
Fixed.


docs: update roadmap

Description

I tried to write some kind of cohesive narrative around the changes for port router and eureka, since there are some touch points between them, and even though there is a lot of good information in the issues, I thought it would be nice to glue all this together in a document.
This is not meant to be exhaustive, but more illustrative instead, so that someone can in a 5-minute read understand the overall picture. I can add more detail if there are things that people think should be added.
closes: #XXXX

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[x] Targeted PR against the correct branch (see CONTRIBUTING.md).
[ ] Linked to GitHub issue with discussion and accepted design, OR link to spec that describes this work.
[ ] Code follows the module structure standards and Go style guide.
[ ] Wrote unit and integration tests.
[ ] Updated relevant documentation (docs/).
[ ] Added relevant godoc comments.
[ ] Provide a conventional commit message to follow the repository standards.
[ ] Include a descriptive changelog entry when appropriate. This may be left to the discretion of the PR reviewers. (e.g. chores should be omitted from changelog)
[ ] Re-reviewed Files changed in the GitHub PR explorer.
[ ] Review SonarCloud Report in the comment section below once CI passes.

closing this pr. The written up material is already a bit out of date. Will let whoever fills this role pickup this work. The team is working on shipping eureka


Cannot get the name of this plugin

I would like to filter this plugin when merging multiple vite config. I can do the filtering using its name as follow:
mergeConfig(config, {
  plugins: [
    ...config.plugins.filter(plugin => plugin.name !== 'vite-tsconfig-paths'), // 👈 Filtering plugin.
    tsConfigPaths(...), // 👈 Using different config.
  ]
})

However, instead of using a hard coded name, I would prefer to have access to its defined name:
https://github.com/aleclarson/vite-tsconfig-paths/blob/44a9f7765c4072deaee0c5b1392445c64fb21a94/src/index.ts#L26
One idea would be to export the name as a props of the function and we would be able to access it as:
tsConfigPaths.pluginName // 👈 using `pluginName` instead of `name` which is a prop inherited from `Function.name`.

Or you could export the plugin name as:
export const pluginName = 'vite-tsconfig-paths'

Thank you for the idea, I ended up with something similar but more verbose because Vite's PluginOption is a union of many types:
// node_modules/vite/dist/node/index.d.ts
export declare type PluginOption = Plugin_2 | false | null | undefined | PluginOption[] | Promise<Plugin_2 | false | null | undefined | PluginOption[]>;

Solution:
const plugin = tsConfigPaths(options)
return mergeConfig(config, {
  plugins: [
    ...(config.plugins
      // 👇Needed because `PluginOption` can be a value or a promise.
      ? (await Promise.all(config.plugins)).filter((p) => {
          // 👇 Needed because `PluginOption` can be `false | null | undefined | PluginOption[]`
          return p && !Array.isArray(p) && p.name !== plugin.name;
        })
      : []),
    plugin,
  ],
});


Able to create post for  organization when in individual profile account and vice versa.

Environment- staging
Browser -Google chrome
Device used-Samsung Galaxy s8
1.login with valid user name and password
2.click on switch account.
3.click on any organization profile.
4.click on create post as individual.
Expected result
It should not allow user to create post for individual when they are in organization account.
Actual result
It allows to create post for individual when they are in organization account and vice versa.

Similar to PR #1843
Should be fixed with this.

This and #1843 should be one fix.


file missing

Dear developer,
Interesting method. I would like to give it a try but seems data.py file is missing, Could you please take a look? Thanks!
from data import preprocess

The missing data.py file is the existing cath_data.py catually. We've updated the run.py to address this problem.
Thank you for your feedback.


Add Mir 2.11.0 info to compositor support table

Logo is "Circle of Friends" from https://design.ubuntu.com/downloads/ (https://assets.ubuntu.com/v1/ce518a18-CoF-2022_solid+O.svg) with color #e9500e modified to #000000.
Marking as a draft until confirmation from @AlanGriffiths regarding the logo choice being correct is gotten; using a placeholder for now though since https://mir-server.io/ shows it too for example.
Closes #13.

Looks serviceable imho, thoughts?

Rebased to fix merge conflict and ran wlprobe on mir_demo_server instead of mir-shell as per Alan's comment:

The extensions enabled are configurable. Some are disabled by default (and typically only advised to recognised clients). Also, shells can add their own.
Only mir_demo_server defaults to enabling all of them

Marking as a draft until confirmation from @AlanGriffiths regarding the logo choice being correct

Confirmed

@vially This should be ready to merge now unless you have feedback ^^


disconnect event doesn't fire when server drops connection

Hi!
When my server drops connection with authentication error or i stop server, this client doesn't fire "disconnect" event.
But it's fires, when i call socketIO.disconnect() function.

you can try disconnecting the client from the server

Well, this hardly is a solution. If the socket gets disconnected because whatever reason (perhaps network problems) it should fire an event which, as @uzarsalan describes, currently does not.


ARROW-16431: [C++][Python] Improve AppendRowGroups error when schemas differ

Fix ARROW-16431
Feel free to opine on specific error messages or the implementation as a whole. 👌
Examples
# meta1 and meta2 differ in column types
meta1.append_row_groups(meta2)
*** RuntimeError: AppendRowGroups requires equal schemas.
These two columns differ at index 0:
column descriptor = {
  name: col1,
  path: col1,
  physical_type: INT64,
  converted_type: NONE,
  logical_type: None,
  max_definition_level: 1,
  max_repetition_level: 0,
}
column descriptor = {
  name: col2,
  path: col2,
  physical_type: INT64,
  converted_type: NONE,
  logical_type: None,
  max_definition_level: 1,
  max_repetition_level: 0,
}

# meta1 and meta2 differ in number of columns
meta1.append_row_groups(meta2)
*** RuntimeError: This schema has 2 columns, other has 1

Can you rebase once more to see if that makes the failures go away?


can't undo game finished by resign

UI communicates that game is over and nothing changes.

...at least this behaviour it's documented in the README.md ;-)
Agreed: Why should undoing "resign" not be possible.

Indeed! I haven't noticed it. But for a analysis case it's a must I believe.

I think I fixed this

Merged  PR#34
Thanks agtoever!


Not an issue, but a nice way to use local modules in the notebook

I started to use gonb to develop and write examples for some modules of a framework that we are developing and found that quite helpful and elegant.
I use "go.work" files while adding features over multiple modules and packages. When using go mod edit you need to add package names and absolute paths. I prefer using go.work files and found a way to add them to the "gonb" notebook, that I find particular nice and wanted to share. It uses the directory names and the current user is evaluated.
!* rm -f go.work && go work init && go work use . "/home/$(id -un)/workspace/localmod"

hey @oderwat , thanks for letting me know, let me incorporate that into the tutorial!
Something I was curious about was whether that was working with gopls. Meaning if changes in the files you are editing get parsed and auto-completed in GoNB ?
I tried it, by including it in the list of files "notified" to gopls (see
goexec/inpsect.go#L22
and
goexec/tracking.go#L241
), but I never really tested it.
cheers

This is a good question. When I use the notebook without docker on my mac, it currently does not work with the language server at all. I did not look into that so far:
[W 16:20:57.048 NotebookApp] Error loading server extension jupyter_lsp
    Traceback (most recent call last):
      File "/usr/local/Cellar/jupyterlab/4.0.2/libexec/lib/python3.11/site-packages/notebook/notebookapp.py", line 2050, in init_server_extensions
        func(self)
      File "/usr/local/Cellar/jupyterlab/4.0.2/libexec/lib/python3.11/site-packages/jupyter_lsp/serverextension.py", line 76, in load_jupyter_server_extension
        nbapp.io_loop.call_later(0, initialize, nbapp, virtual_documents_uri)
        ^^^^^^^^^^^^^
    AttributeError: 'NotebookApp' object has no attribute 'io_loop'

So, I was not having "gopls" support at all. I can't really (easily) use the docker because many of the modules are from private repositories and I would need to add that setup to either the docker image or the notebook.
I am actually using it for some bug fixing and redesigning an API so I had no time to check what is wrong with gopls. I am also new to jupyter notebooks (just used some occasionally in the past).

Sadly I don't have an easily available Mac to try it out and fix it ... But the error message sounds odd, since it should be GoNB talking to the language server, not Jupyter. I wonder if something is different in the Jupyter installation in Mac, that it's not using GoNB for auto-complete (or contextual help).
Do you want to open a separate issue for that ? Maybe someone with a Mac could find it out ? If someone tells me what is wrong I'd happily fix it.
Just in case: the docker (in Linux at least) can be executed sharing arbitrary directories with the host (it's just an extra flag). So actually using the docker and private repositories (I've used that before) in the host could work -- but I don't know the details, and it's also possible it wouldn't for other reasons.

I was using the docker variant to access the other directories, the problem is more that the repositories are private and the go installation inside the docker image cannot access them without some changes to git and docker configs (go mod download fails). We made some other images for our CI chains that can can handle this by setting up the docker so that it has access.

@janpfeifer
is there a better way to reuse the shell variable? or is it a better way to switch to gonb temporary folder within shell instead of !*
I have to write to the temp file
!echo ${PWD} >/tmp/curr_dir
!*CURR_DIR=`cat /tmp/curr_dir` && rm -f go.work && go work init && go work use . "${CURR_DIR}/mymodule"
%goworkfix

hi @quickwritereader , sorry I was away these last couple of days.
Currently you can use the %env special  command to set an environment variable in the kernel space (so it will be passed to all shell executions and well as Go cell executions):
%env CURR_DIR /home/user/work

But it requires some copy&pasting of the current directory.
What I could do, if it's helpful, and to automatically create an environment variable with the current directory (different than the temp folder). Let me know if it helps -- and maybe create a new issue to track it. It's a trivial change, and likely can be done by tomorrow.

I did the small change to expose two environment variables: GONB_DIR and GONB_TMP_DIR, in a branch called dirs, see https://github.com/janpfeifer/gonb/tree/dirs
A sample of the output would be:
!echo "From current dir:" && pwd && echo "GONB_DIR=$GONB_DIR" && echo "GONB_TMP_DIR=$GONB_TMP_DIR" && echo
!*echo "From temporary Go dir:" && pwd && echo "GONB_DIR=$GONB_DIR" && echo "GONB_TMP_DIR=$GONB_TMP_DIR" && echo

The output is:
From current dir:
/opt/janpf/work
GONB_DIR=/opt/janpf/work
GONB_TMP_DIR=/tmp/gonb_e6352989

From temporary Go dir:
/tmp/gonb_e6352989
GONB_DIR=/opt/janpf/work
GONB_TMP_DIR=/tmp/gonb_e6352989

Let me know if this works. In any case tomorrow or Monday I'll create a new release including it.
thanks for the report!

I like it ;-)

v0.7.5 released with the changes. Also a new docker.

I have yet another problem, but let's close this for good now.


Part1: Validate Dataset name and class_field_name is Python identified

[x] Part1: ensure that the dataset name or field_name can be Pythonic self. field variables
[ ] Part2: ensure that the dataset name is Upper Camel Case for the default offline plugin. That is not part of this PR because we are still in discussion on what casing to enforce.  Part1 at least prevents users from shooting themselves in the foot.

Coverage decreased (-0.04%) to 96.371% when pulling 649618e9293ed52b2b74b8451f787533bbe1d994 on tz/aip-5368-naming into ed6d0767a0f822461a8af4fc51411c9cbbc583b6 on feature/datasets.


Language fields (for carmen@hyperpolyglot)

This is an implementation of the final multilingual grid proposal described here. Grid keys are now annotated with a bitfield describing the languages to which the given combination of key and grids applies.
A bit of extra detail beyond the carmen PR (but read the carmen PR too)...
About the key format:

this PR introduces an internal language field type, which is currently typedef'ed to an unsigned 128-bit int (using the nonstandard __int128 clang intrinsic). The index format does not depend on these fields being 128-bit, however, as the language suffix is variably sized.
keys are formatted as key|langfield (with an actual pipe character as the separator), and the langfield is a variable-width bit field that's copied into the least significant bytes of a 128-bit int at read time. The field will thus only be as many bytes as is necessary to represent the highest-bit-number language present for the field.
as an additional space optimization, the langfield can be omitted entirely (such that the key just ends with a | character). This is interpreted as if the field had been ALL_LANGUAGES (which is an all-true bit field). So the read algorithm is: find the index of the first | character in the key, check if it's the last character in the key, and if so, return ALL_LANGUAGES; if not, copy the bytes of the field into the least significant bytes (first bytes as we're on a little-endian architecture) of a new __int128 and return it.

About other algorithmic stuff:

getByPrefix has been replaced by a new getMatching which both checks prefixes and languages. In the event of a language match failure, one of the unused grid bits (as we only use 53 for JS number reasons) is used to track language-matching, which causes all language-matches to sort higher than all non-language-matches in the priority queue.
In coalesce, that bit is checked, and we additionally apply a 20% relevance penalty in the event of a language match failure.

Oh also:

I reduced the cached prefix tier lengths from 4 and 7 to 3 and 6, to account for the removal of the x and z prefixes in carmen and consequent key length reductions
I dropped support for node 0.10 because I used some ES6isms and carmen doesn't support node 0.10 anymore anyway


NUI-1380 backwards compatibility: auth params should be available for transformers

In DRAFT: testing via Devtool pending
NUI-1380 backwards compatibility: auth params should be available for transformers

https://git.corp.adobe.com/nui/transformer-sensei/pull/27
https://git.corp.adobe.com/nui/asset-compute-pipeline/pull/30

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context.
Fixes # (issue)
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] I have signed the Adobe Open Source CLA.
[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Not needed anymore, so closing.


Accept arguments from stdin

Some arguments can be too long, for example texts for the lb_print or lb_email.
Some functions should support passing parameters by stdin (or pipeline).
For each function to port, do:

change code
update documentation
add unit tests

Functions to port:

[x] lb_print
[x] lb_error
[x] lb_display
[x] lb_log
[ ] lb_read_config
[ ] lb_import_config
[ ] lb_get_config
[x] lb_is_comment
[x] lb_trim
[ ] lb_split cannot be done because of context
[ ] lb_email
[ ] lbg_display_*
[ ] lbg_notify

Be careful that pipes are in a specific context.
Functions that use and save global variables can not be ported to stdin support.


Installing ARTIC stuck at Solving environment

As described in the title, I followed the instructions from https://artic.network/ncov-2019/ncov2019-it-setup.html And after entering the command "conda env create -f environment.yml", the computer is stucked.

please see the following, there is the answer: https://github.com/artic-network/artic-ncov2019/issues/71
Longer Conda Build Times #71


1.7 easyjson

This is a fix for #3601, this wouldn't be the actual PR (as we'll need to commit the changes in https://github.com/prometheus/common/pull/112 first) but this gives an idea of the level of change required (very small).
I've run this locally with some test data, and here are the comparisons:

pre-patch
with-patch

Matrix
214ms
37ms

Vector
108ms
20ms

Scalar
20ms
11ms

String
18ms
13ms

With this we can see the large performance gains even with small returns. Not only do we get these gains (~5-6 for "real" data) we also significantly reduce the memory required to marshal out the response (which means this scales significantly better into large response sizes.
Also, as a bonus, since this iteratively serializes it will also stop mid-serialization if the client disconnects!

CI CI failure seems unrelated (due to unmounting FS?)

To include a complete end-to-end benchmark I've done a simple promql query (metricname[range]) over various time durations pre and post easyjson change, here are the results:

Time (minutes)
Old
New
Improvement

0
0.121
0.086
-28.92561983

1
0.171
0.076
-55.55555556

10
1.226
0.119
-90.29363785

30
2.126
0.58
-72.7187206

60
5.173
0.441
-91.47496617

120
9.086
1.107
-87.81642087

180
13.719
1.795
-86.91595597

240
17.668
1.956
-88.92913742

This PR is against Prometheus 1.7, which is not the current development version. Any changes of this nature should be proposed against master.

@brian-brazil I can do that, I opened the PR here as the issue (#3601 ) was reported on 1.7. The changes here can (and should) be applied to any releases that are still supported (I'm not sure which ones those are, so I'll need some guidance on what are the currently supported releases).

A change like this would be aimed for 2.1 currently, which is to say master. New features are not being added to 1.x.

I can re-open the PR against master, but https://github.com/prometheus/common/pull/112 needs to be merged first. Before moving the PR I'd like to see that merged and some feedback on this change (since it will be basicaly the same change against master).
Is there also no mechanism to backport to previous releases? From looking at the releases page it seems that N-1 is supported (minor releases etc.)?

All the relevant code changed with 2.0, you'd need to redo it all.
Only bugfixes would be considered for backport.

What classifies bug fix? From the end to end data this is ~10x perf
improvement.
Also, your latest response still has no feedback for a change like this.
On Dec 27, 2017 4:48 PM, "Brian Brazil"<EMAIL_ADDRESS>wrote:

All the relevant code changed with 2.0, you'd need to redo it all.
Only bugfixes would be considered for backport.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/prometheus/prometheus/pull/3608#issuecomment-354205590,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABi3nXHiLCCYI_2QTp4GzjKfuVV9Tt1Zks5tEuVhgaJpZM4RKaNd
.

A performance improvement is not considered a bugfix.

Ok, how about feedback on the change?
On Dec 27, 2017 4:57 PM, "Brian Brazil"<EMAIL_ADDRESS>wrote:

A performance improvement is not considered a bugfix.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/prometheus/prometheus/pull/3608#issuecomment-354206788,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABi3nR1QakyNWtnQ3j9B1mWlfCAoIqyuks5tEudsgaJpZM4RKaNd
.

It's a highly invasive change that'd require significant ongoing error-prone maintenance, and it also violates layering. Such downsides require major major benefits to justify, and the benchmark numbers produced thus far are at odds with previous profiling in this area.

It's a highly invasive change

I'm not sure if we're looking at the same diff? The reason this particular PR is so large is due to changes in the vendor directory. The actual change to the code is ~30-40 lines of change, which I'd hardly call "highly invasive".

that'd require significant ongoing error-prone maintenance

I'm also not sure what maintenance you are expecting to perform? There is one codegen'd file -- which requires no human to write the code and the only other one is a small marshal method on the queryData struct -- which isn't exactly a large struct.

and it also violates layering.

What are you implying with this? What layering is being broken? In golang marshal methods have to be in the package with the struct definition-- so if you don't want the marshal/unmarshal methods in this package then we need to move it to another package. Past that I have no idea what other layering you could be referring to, so please elaborate.

Such downsides require major major benefits to justify

I agree, which is why I've spent a significant amount of time collecting a LOT of data (and creating a variety of benchmarks to demonstrate this)

and the benchmark numbers produced thus far are at odds with previous profiling in this area.

Thats usually how performance improvements work? For all the benchmarks we're talking about the code is all open -- so it is comparable. The benchmarks you are referring to are all done by the jsoniter guys on a generic dataset. The benchmarks I've put in place are (1) with the prometheus structs and (2) a full end-to-end test (which is exactly what you asked for). At this point I'm at a loss for what additional information to provide. I have overwhelming data supporting my position, and all you've been able to counter with is "suspicion" -- with no recourse.
What data or proof do you require to get such a change in? I'm growing weary of gathering data only to be met with 'suspicion' and uncertainty. This is a simple performance optimization (a HUGE one, but a simple one nonetheless). If not performance data, what data do you need?

Obsoleted by #3536


Provide a way to disable swagger

I search a way to disable swagger in production.
I've tried to comment the module in the application.conf, but the endpoint is still working.

I wrote a Play filter
Let me know if you want to integrate this kind of code in play-swagger
package controllers

import com.typesafe.config.ConfigFactory
import play.api.mvc._
import scala.concurrent.Future
import play.api.libs.concurrent.Execution.Implicits.defaultContext

/**
  * Filter for disabling swagger according to some configuration.
  *
  * Examples of configuration :
  * {{{
  * swagger {
  *   path=/api-docs
  *   enabled=true
  * }
  * }}}
  */
class BlockingFilter extends Filter {
  val enabled = ConfigFactory.load().getBoolean("swagger.enabled")
  val path = ConfigFactory.load().getBoolean("swagger.path")

  def apply(nextFilter: RequestHeader => Future[Result])
           (requestHeader: RequestHeader): Future[Result] = {

    if (requestHeader.path == path && !enabled)
      Future.successful(Results.Unauthorized)
    else
      nextFilter(requestHeader)
  }
}

This would be a good feature to have in the plugin :+1:


Update sql-database-service-tiers.md

Some consolidations and updates to the article to reflect some of the differences between Premium and Standard performance levels.

Hi @jan-eng, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Jan Engelsberg). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

@jan-eng : Thanks for your contribution to the Azure documentation! The author, @CarlRabeler, has been notified to review your proposed change.

@ jan-eng due to conflict with a recent update to the services tiers table, I am making these changes in a new PR. See https://github.com/Microsoft/azure-docs-pr/pull/2706 - this is being staged now and I will push it for merger later today - after making updates to the terms single vs standalone and elastic database pool vs elastic pool in a number of documents.

@jan-eng
please close - these changes promolgated in another PR

@jan-eng
please close - these changes promolgated in another PR


Revise README to for GA

Reframing from a Marketing team tool to general availability, focusing on all contributors ability to use this tool.

The README is tidied up!


On which requests to insert the headers?

I've read that the tenant ID should be kept private. I'm wondering if the headers would go unencrypted on the wire if inserted in the initial connect request, and therefore if, to protect our tenant ID, we should only begin inserting the headers after the SSL connection to AAD has been established and the client is requesting a token.

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

ID: 89ea5c2d-4602-25b8-3cdd-60cfb5c31895
Version Independent ID: 35d4af7e-72e4-7c40-9dd6-b8490ee73f0d
Content: Use tenant restrictions to manage access to SaaS cloud applications - Azure
Content Source: articles/active-directory/manage-apps/tenant-restrictions.md
Service: active-directory
Sub-service: app-mgmt
GitHub Login: @CelesteDG
Microsoft Alias: celested

@RickB-0923845 Thank you for your feedback . We will investigate and update the thread further.

@RickB-0923845 As per my understanding there is no such specific requirement to keep tenant ID private. In case of deploying tenant restrictions we use a method called SSL interception. When a proxy server is used for a SSL interception method , the connection between the client and the service (in this case login.microsoftonline et al.) is encrypted by one certificate while the connection between the proxy and the Microsoft cloud services is protected by another certificate. While we initiate an authentication request the first thing that happens is establishment of a SSL tunnel before the request is sent over the wire within TLS tunnel so yes you are right the header modification by the proxy will be done after the encrypted SSL tunnel has been established.  As for tenant ID , even if anyone knows your tenant ID, they cannot get any token from your tenant so I think its alright even if others know your tenant Id . There are easy ways to obtain tenant ID by knowing anyone's Azure AD custom domain names and you would never be able to stop anyone from knowing your custom domain name.
If you are concerned about the internal clients making any changes to the proxy headers , we recommend to have enough safeguards like group policies and administrative controls on the machine so the users are not able to bypass the proxy. Even if they are able to change the headers somehow the proxy must always check the values for the tenant restrictions headers and update them before sending to azure AD.
I hope the above clarifies the queries. should you have a any further query , feel free to tag me or the author @CelesteDG to your reply and we will gladly continue the conversation.

Thanks!


KOGITO-6289: Regressions on copy/paste from other spreadsheet, after KOGITO-5078

JIRA
https://issues.redhat.com/browse/KOGITO-6289
I've fixed the issue described in the JIRA, and also, I've removed most of the listeners added to the document, which should be reduced to the boxed-expression-component scope. This was done to avoid undesired listeners where the Boxed Expression Component is rendered.

The (build) kogito-tooling-editors-java check has failed. Please check the logs.

The (build) kogito-tooling-editors-java check has failed. Please check the logs.

The (build) kogito-tooling-editors-java check has failed. Please check the logs.

The (build) kogito-tooling-editors-java check has failed. Please check the logs.

@tiagobento I've done all your change requests, could you review it again?
@yesamer and conflicts solved.

@yesamer @vpellegrino @jomarko Hi, I believe I've done the necessary adjustments, this PR is ready for review.

Jenkins retest this

@ljmotta, it seems that copy&paste is working fine (or at least as before).
I just noticed a different user experience when selecting multiple cells.
When selecting multiple cells, with an older version of the editor (https://boxed-expression-editor.netlify.app/) the selection seems more fluid, even when the mouse cursor goes out of the area of the cells:

Instead, with the editor produced by this PR, it seems we have a glitch when the mouse cursor goes out of the area of the table cells. I tried to capture it with a GIF, but I am not sure that it helps to understand:
.
In such situations, no cell is selected, and the selection box is still active until you click outside somewhere. Moreover, it seems that the mouse cursor is a bit slow while performing the selection action.
This selection issue does not seem critical, but I wanted to let you know.
In the meantime, please let me know if you need further details.

The (build) kogito-tooling-editors-java check has failed. Please check the logs.

Answering to https://github.com/kiegroup/kogito-editors-java/pull/181#pullrequestreview-827862534
@jomarko in the original task https://issues.redhat.com/browse/KOGITO-3656 there is no specific indication of supported expressions.
I tried a bit https://boxed-expression-editor.netlify.app/ and it seems that:

Literal
Relation
Decision Table
are supporting the copy/paste feature.
The same applies also to nested expressions that wrap a text cell (for instance, a context table containing a literal expression for one of its entries).

I think that this PR (https://github.com/kiegroup/kogito-editors-java/pull/103) clarifies better possible use-cases.
Tomorrow I'll review your test cases PR, thanks a lot @jomarko

@vpellegrino Thanks for your review. Indeed, now the selection box can only exist inside the boxed-expression-provider div instead of the document itself. So to increase the selection box range, it's necessary to have a bigger parent div. I believe that I've managed to increase the DMN Boxed Editor on the DMN Editor, but it would be good if you could check if I have changed the CSS in the correct place. Here is a quick video using the kie-wb-common-dmn-webapp-kogito-testing:
https://user-images.githubusercontent.com/24302289/145454470-ff315b72-acfa-48fc-99cf-1251c2cf7719.mov
@jomarko Thanks a lot for your test case, I'm going to review it.

The (build) kogito-tooling-editors-java check has failed. Please check the logs.

Jenkins retest this

The (build) kogito-tooling-editors-java check has failed. Please check the logs.

@jomarko I tried the tests locally and they look great. Thanks!
@vpellegrino What is possible to do is to increase the Boxed Editor div height, so it will have a bigger gap between the response and the Editor itself. This will not enable to select on top of the response but will increase the selection area. For example:

@vpellegrino I've set the height of the showcase to make the selection area bigger. Adding to that, I've added a dashed border to make it possible to see it.

The (build) kogito-tooling-editors-java check has failed. Please check the logs.

The (build) kogito-tooling-editors-java check has failed. Please check the logs.

The (build) kogito-tooling-editors-java check has failed. Please check the logs.

The (build) kogito-tooling-editors-java check has failed. Please check the logs.

jenkins retest this please

The (build) kogito-tooling-editors-java check has failed. Please check the logs.

@ljmotta  jenkins build reported this failure: WDYT?
[2021-12-13T13:03:12.668Z] [INFO] boxed-expression-component: $ yarn test && webpack --config webpack.production.js
[2021-12-13T13:03:12.668Z] [INFO] boxed-expression-component-showcase: $ yarn test:it
[2021-12-13T13:03:12.668Z] [INFO] boxed-expression-component: $ jest --collect-coverage
[2021-12-13T13:03:12.668Z] [INFO] boxed-expression-component-showcase: $ start-server-and-test start:it http://localhost:3000 cy:run
[2021-12-13T13:03:22.848Z] [INFO] boxed-expression-component-showcase: 1: starting server using command "npm run start:it"
[2021-12-13T13:03:22.848Z] [INFO] boxed-expression-component-showcase: and when url "[ 'http://localhost:3000' ]" is responding with HTTP status code 200
[2021-12-13T13:03:22.848Z] [INFO] boxed-expression-component-showcase: running tests using command "npm run cy:run"
[2021-12-13T13:03:23.139Z] [INFO] boxed-expression-component-showcase: ><EMAIL_ADDRESS>start:it
[2021-12-13T13:03:23.139Z] [INFO] boxed-expression-component-showcase: > BROWSER=none yarn start
[2021-12-13T13:03:23.439Z] [INFO] boxed-expression-component-showcase: $ react-app-rewired start
[2021-12-13T13:03:45.800Z] [INFO] boxed-expression-component-showcase: ℹ ｢wds｣: Project is running at http://<IP_ADDRESS>/
[2021-12-13T13:03:45.800Z] [INFO] boxed-expression-component-showcase: ℹ ｢wds｣: webpack output is served from
[2021-12-13T13:03:45.800Z] [INFO] boxed-expression-component-showcase: ℹ ｢wds｣: Content not from webpack is served from /home/jenkins/workspace/KIE/kogito/main/pullrequest/kogito-editors-java.tests.kogito-tooling-editors-java/kogito-editors-java/kogito-editors-js/packages/boxed-expression-component/showcase/public
[2021-12-13T13:03:45.800Z] [INFO] boxed-expression-component-showcase: ℹ ｢wds｣: 404s will fallback to /
[2021-12-13T13:03:45.800Z] [INFO] boxed-expression-component-showcase: Starting the development server...
[2021-12-13T13:03:45.800Z] [INFO] boxed-expression-component-showcase: node:events:342
[2021-12-13T13:03:45.800Z] [INFO] boxed-expression-component-showcase:       throw er; // Unhandled 'error' event
[2021-12-13T13:03:45.800Z] [INFO] boxed-expression-component-showcase:       ^
[2021-12-13T13:03:45.800Z] [INFO] boxed-expression-component-showcase: Error: ENOSPC: System limit for number of file watchers reached, watch '/home/jenkins/workspace/KIE/kogito/main/pullrequest/kogito-editors-java.tests.kogito-tooling-editors-java/kogito-editors-java/kogito-editors-js/packages/boxed-expression-component/showcase/public/favicon.ico'
[2021-12-13T13:03:45.800Z] [INFO] boxed-expression-component-showcase:     at FSWatcher. (node:internal/fs/watchers:244:19)
[2021-12-13T13:03:45.800Z] [INFO] boxed-expression-component-showcase:     at Object.watch (node:fs:2240:34)
[2021-12-13T13:03:45.801Z] [INFO] boxed-expression-component-showcase:     at createFsWatchInstance (/home/jenkins/workspace/KIE/kogito/main/pullrequest/kogito-editors-java.tests.kogito-tooling-editors-java/kogito-editors-java/kogito-editors-js/node_modules/chokidar/lib/nodefs-handler.js:38:15)
[2021-12-13T13:03:45.801Z] [INFO] boxed-expression-component-showcase:     at setFsWatchListener (/home/jenkins/workspace/KIE/kogito/main/pullrequest/kogito-editors-java.tests.kogito-tooling-editors-java/kogito-editors-java/kogito-editors-js/node_modules/chokidar/lib/nodefs-handler.js:81:15)
[2021-12-13T13:03:45.801Z] [INFO] boxed-expression-component-showcase:     at FSWatcher.NodeFsHandler._watchWithNodeFs (/home/jenkins/workspace/KIE/kogito/main/pullrequest/kogito-editors-java.tests.kogito-tooling-editors-java/kogito-editors-java/kogito-editors-js/node_modules/chokidar/lib/nodefs-handler.js:233:14)
[2021-12-13T13:03:45.801Z] [INFO] boxed-expression-component-showcase:     at FSWatcher.NodeFsHandler._handleFile (/home/jenkins/workspace/KIE/kogito/main/pullrequest/kogito-editors-java.tests.kogito-tooling-editors-java/kogito-editors-java/kogito-editors-js/node_modules/chokidar/lib/nodefs-handler.js:262:21)
[2021-12-13T13:03:45.801Z] [INFO] boxed-expression-component-showcase:     at FSWatcher. (/home/jenkins/workspace/KIE/kogito/main/pullrequest/kogito-editors-java.tests.kogito-tooling-editors-java/kogito-editors-java/kogito-editors-js/node_modules/chokidar/lib/nodefs-handler.js:495:21)
[2021-12-13T13:03:45.801Z] [INFO] boxed-expression-component-showcase:     at FSReqCallback.oncomplete (node:fs:195:5)
[2021-12-13T13:03:45.801Z] [INFO] boxed-expression-component-showcase: Emitted 'error' event on FSWatcher instance at:
[2021-12-13T13:03:45.801Z] [INFO] boxed-expression-component-showcase:     at FSWatcher._handleError (/home/jenkins/workspace/KIE/kogito/main/pullrequest/kogito-editors-java.tests.kogito-tooling-editors-java/kogito-editors-java/kogito-editors-js/node_modules/chokidar/index.js:260:10)
[2021-12-13T13:03:45.801Z] [INFO] boxed-expression-component-showcase:     at createFsWatchInstance (/home/jenkins/workspace/KIE/kogito/main/pullrequest/kogito-editors-java.tests.kogito-tooling-editors-java/kogito-editors-java/kogito-editors-js/node_modules/chokidar/lib/nodefs-handler.js:40:5)
[2021-12-13T13:03:45.801Z] [INFO] boxed-expression-component-showcase:     at setFsWatchListener (/home/jenkins/workspace/KIE/kogito/main/pullrequest/kogito-editors-java.tests.kogito-tooling-editors-java/kogito-editors-java/kogito-editors-js/node_modules/chokidar/lib/nodefs-handler.js:81:15)
[2021-12-13T13:03:45.801Z] [INFO] boxed-expression-component-showcase:     [... lines matching original stack trace ...]
[2021-12-13T13:03:45.801Z] [INFO] boxed-expression-component-showcase:     at FSReqCallback.oncomplete (node:fs:195:5) {
[2021-12-13T13:03:45.801Z] [INFO] boxed-expression-component-showcase:   errno: -28,
[2021-12-13T13:03:45.801Z] [INFO] boxed-expression-component-showcase:   syscall: 'watch',
[2021-12-13T13:03:45.801Z] [INFO] boxed-expression-component-showcase:   code: 'ENOSPC',
[2021-12-13T13:03:45.801Z] [INFO] boxed-expression-component-showcase:   path: '/home/jenkins/workspace/KIE/kogito/main/pullrequest/kogito-editors-java.tests.kogito-tooling-editors-java/kogito-editors-java/kogito-editors-js/packages/boxed-expression-component/showcase/public/favicon.ico',
[2021-12-13T13:03:45.801Z] [INFO] boxed-expression-component-showcase:   filename: '/home/jenkins/workspace/KIE/kogito/main/pullrequest/kogito-editors-java.tests.kogito-tooling-editors-java/kogito-editors-java/kogito-editors-js/packages/boxed-expression-component/showcase/public/favicon.ico'
[2021-12-13T13:03:45.801Z] [INFO] boxed-expression-component-showcase: }
[2021-12-13T13:03:45.801Z] [INFO] boxed-expression-component-showcase: error Command failed with exit code 1.
[2021-12-13T13:03:45.801Z] [INFO] boxed-expression-component-showcase: info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
[2021-12-13T13:03:45.801Z] [INFO] boxed-expression-component-showcase: Error: server closed unexpectedly
[2021-12-13T13:03:45.801Z] [INFO] boxed-expression-component-showcase:     at ChildProcess.onClose (/home/jenkins/workspace/KIE/kogito/main/pullrequest/kogito-editors-java.tests.kogito-tooling-editors-java/kogito-editors-java/kogito-editors-js/node_modules/start-server-and-test/src/index.js:75:14)
[2021-12-13T13:03:45.801Z] [INFO] boxed-expression-component-showcase:     at ChildProcess.emit (node:events:365:28)
[2021-12-13T13:03:45.801Z] [INFO] boxed-expression-component-showcase:     at maybeClose (node:internal/child_process:1067:16)
[2021-12-13T13:03:45.801Z] [INFO] boxed-expression-component-showcase:     at Process.ChildProcess._handle.onexit (node:internal/child_process:301:5)
[2021-12-13T13:03:45.801Z] [INFO] boxed-expression-component-showcase: error Command failed with exit code 1.
[2021-12-13T13:03:45.801Z] [INFO] boxed-expression-component-showcase: info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
[2021-12-13T13:03:45.801Z] [INFO] boxed-expression-component-showcase: error Command failed with exit code 1.
[2021-12-13T13:03:45.801Z] [INFO] boxed-expression-component-showcase: info Visit https://yarnpkg.com/en/docs/cli/run f

@yesamer oh, thank you for spotting this, I spotted somethig similar during kogito-tooling PRs review.
See:

https://issues.redhat.com/browse/KOGITO-6355

So I can fix the test. The question is if we need to report new jira or is it the same as https://issues.redhat.com/browse/KOGITO-6355 @ljmotta ?

locally boxed-expression-component pass for me, however not sure if the branch shouldn't be rebased with main so we have most accurate CI results.

jenkins retest this

@yesamer It looks like this particular test is passing now, but a test failed on tooling. I'm going to take a look into it.

@jomarko Thanks for the heads up. I've opened the PR on tooling to fix this CSS class.

I am afraid we need to release a new version of kogito-editor-java to have green lights here.
@ljmotta Please be sure these two PRs work well in your local, then I'll merge this one even if it fails. After that, we can release a new version with this change included. kogito-tooling will refer to this new version and it will be possible to fix all.
As alternative, we can postpone this merge after the monorepo is in place
//cc @tiagobento WDTY?

@yesamer If the CI is green, I'm +1 to merge this, as it will only affect kogito-tooling when it is updated with a new version of kogito-editors-java. Luiz's PR on kogito-tooling should make the CI green here, so no need to wait if everything is green.

The (build) kogito-tooling-editors-java check has failed. Please check the logs.

Jenkins retest this

The (build) kogito-tooling-editors-java check has failed. Please check the logs.

Jenkins retest this


Fixed Godot not recognising 150x150 icon for uwp export #35871

Fixes #35871, this issue also exists in 3.1

Thanks!

Cherry-picked for 3.2.1.

Cherry-picked for 3.1.3.