Incompatibility with plt.subplots(..., sharey=True)

As the drawcall updates the ylim using the sharey=True setting in matplotlib.pyplot.subplots() is not really useful to compare the weights of different matrices.
MRE:
mtx = logomaker.get_example_matrix('logomaker_logo_matrix')
fig, axs = plt.subplots(2, sharey=True)
logomaker.Logo(mtx * 2, ax=axs[0]) # Require a larger dynamic range
logomaker.Logo(mtx, ax=axs[1]) # ylims are updated to the last displayed range

Output:

Expected behavior:
If multiple logos with a shared y or x axes are drawn, the range accomodates the largest drawing to allow comparison of the magnitude.

matplotlib version used: 3.5.2

I'm also running into this problem.

Thank you @liliblu for the solution!
@sholderbach can you try the above please? If the solution works for you, please close the issue.


How to compare the accuracies?

In task 1 it is said to compare the accuracies (how many digits are correct) and efficiencies (how many steps does it take to reach a given accuracy). How are we supposed to take the accuracy value. Should we take adaptive trapezoid method as reference or any approximate value of true integral to compare the accuracy?

@pratibha77118  I have sent this group a merge request as integral accuracy, hope that file (integral_accuracy.py)  will answer your question.

In task 1 it is said to compare the accuracies (how many digits are correct) and efficiencies (how many steps does it take to reach a given accuracy). How are we supposed to take the accuracy value. Should we take adaptive trapezoid method as reference or any approximate value of true integral to compare the accuracy?

My question exactly.

@pratibha77118 I have sent this group a merge request as integral accuracy, hope that file (integral_accuracy.py) will answer your question. #37

Thanks.

@tirthbha  I did check on that but I got this warning while implementing scipy
:28: IntegrationWarning: The maximum number of subdivisions (50) has been achieved.
If increasing the limit yields no improvement it is advised to analyze
the integrand in order to determine the difficulties.  If the position of a
local difficulty can be determined (singularity, discontinuity) one will
probably gain from splitting up the interval and calling the integrator
on the subranges.  Perhaps a special-purpose integrator should be used.
result, error = quad(integrand, a, b)
[21]

@pratibha77118 Please add n to the last line as "  return h * s, n ", I forgot to add n there. I will work.

In task 1 it is said to compare the accuracies (how many digits are correct) and efficiencies (how many steps does it take to reach a given accuracy). How are we supposed to take the accuracy value. Should we take adaptive trapezoid method as reference or any approximate value of true integral to compare the accuracy?

My question exactly.

@pratibha77118 I have sent this group a merge request as integral accuracy, hope that file (integral_accuracy.py) will answer your question. #37

Thanks. So you have used the exact value obtained from scipy.

Yes, that is taken as a reference value to compare with others obtained from different methods.

I got result like this.

In task 1 it is said to compare the accuracies (how many digits are correct) and efficiencies (how many steps does it take to reach a given accuracy). How are we supposed to take the accuracy value. Should we take adaptive trapezoid method as reference or any approximate value of true integral to compare the accuracy?

My question exactly.

@pratibha77118 I have sent this group a merge request as integral accuracy, hope that file (integral_accuracy.py) will answer your question. #37

Thanks. So you have used the exact value obtained from scipy.

Yes

In task 1 it is said to compare the accuracies (how many digits are correct) and efficiencies (how many steps does it take to reach a given accuracy). How are we supposed to take the accuracy value. Should we take adaptive trapezoid method as reference or any approximate value of true integral to compare the accuracy?

My question exactly.

@pratibha77118 I have sent this group a merge request as integral accuracy, hope that file (integral_accuracy.py) will answer your question. #37

Thanks. So you have used the exact value obtained from scipy.

Yes

Your request has been approved.


LightDomain has duplicated properties of type ICriticHandler

The LightDomain class has both a private and a public property of the type ICriticHandler:
https://github.com/Avanade/Liquid-Application-Framework/blob/93f7874f2ead13d3602cc39fc1e9355206217f57/src/Liquid.Domain/Base/Domain/LightDomain.cs#L20-L21
I assume the private was intended to be used as a backing field for the public one.
Analyzing the code, I believe we can keep only the public version

This is ugly. I'll mark it as an enhancement since it brings no faulty behavior, but I think we should fix it as soon as possible.


Add parameters for auto-approve grades

Add parameters to allow us to set the isAutoApproved flag when sending grades to gradebook.  Ready to review @pdpinch @carsongee
Addresses issue #27

Coverage remained the same at 100.0% when pulling 3fc7f8eba33662c7e1f0a09ab233e7b616fa32a7 on feature/pw/auto-approve-grades into 6ae7e6e5f0f582528eefe23827e083e8180beb9a on master.

Please also add this parameter to set_grade: https://github.com/mitodl/PyLmod/blob/feature/pw/auto-approve-grades/pylmod/gradebook.py#L139 so that it isn't a required but invisible kwarg

:sparkles:  I should add, otherwise this looks good, tests make sense and actually using it with the flag set to approve approved my spreadsheet of grades :sparkles:

Coverage remained the same at 100.0% when pulling d84eac6ff47b3070d8cd3d2a8b4e1d01badb1843 on feature/pw/auto-approve-grades into 6ae7e6e5f0f582528eefe23827e083e8180beb9a on master.

@carsongee I modified set_grade() though I'm not sure that the change I made is what you suggested.  I added corresponding tests as well.

Coverage remained the same at 100.0% when pulling 9c3cf35a215978fb93c554eeb6fdd83361cf94db on feature/pw/auto-approve-grades into 6ae7e6e5f0f582528eefe23827e083e8180beb9a on master.

Other than that last comment, I think we are good.  Also, what is with the merge commits?

Coverage remained the same at 100.0% when pulling 9a53fb0c6eeef823c97d5d8b799c5adf53072886 on feature/pw/auto-approve-grades into 2577754b1de27fbb18ccf49bfbe9f82d74a57d46 on master.

Coverage remained the same at 100.0% when pulling a72e515a9d741346e744f9e44f9a95fa431cd543 on feature/pw/auto-approve-grades into 2577754b1de27fbb18ccf49bfbe9f82d74a57d46 on master.

This looks ready to merge, but I only knew that from coveralls commenting on it.  It would be helpful when you are ready for another round of review to note it with a comment since notifications aren't sent on commits/pushes. Is this ready @pwilkins?

It's ready to merge.  I thought you'd get a notification.  My bad.


Add NumberJS Game

NumberJS Game
Site: https://jorger.github.io/NumberJS/
Github: https://github.com/Jorger/NumberJS

Thank you very much.
Jorge Rubiano.
@ostjh

Hi Jorge, thank you for suggestion. Although I wouldn’t call this game a good example of PWA: it’s a good demo, but it’s way too simplistic and not really progressive or responsive. What do you guys think @andreasbovens @mathiasbynens @shwetank?


This should fix half of Wil's problem

Current coverage is 48.83% (diff: 81.57%)

Merging #118 into master will increase coverage by 0.57%

@@             master       #118   diff @@
==========================================
  Files            95         95
  Lines          4656       4718    +62
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           2247       2304    +57
- Misses         2197       2198     +1
- Partials        212        216     +4

Powered by Codecov. Last update 342144e...7553b5a


updating supported releases for v1.27

Updating the supported releases for version 1.27.
/milestone 1.27
/hold
/cc @mickeyboxell @reylejano

/close
closing as very incorrect 😅


Allow to send notifications at certain layers

Prerequisite: DisplayLayerProgress plugin and OctoPod plugin are required
OctoPod will have a UI to let users specify at which layers they want to be notified for the running job.

Dup


splunk::enterprise - "has no parameter" manage_net_tools

Puppet: 6.3.1

Ruby: 2.5.9p229 (2021-04-05 revision 67939)

Distribution: RHEL8.4

Module version: latest

How to reproduce (e.g Puppet code you use)
class { 'splunk::enterprise':
  package_ensure => latest,
  manage_password => true,
  enterprise_homedir => '/mnt/services/splunk/enterprise',
  splunk_user => 'splunk',
  password_content => '[PASSWORD HASH]',
  manage_net_tools => false,

}
Also tried manage_net-tools (syntax error), manage_nettools (
What are you seeing
Could not retrieve catalog from remote server: Error 500 on SERVER: Server Error: Evaluation Error: Error while evaluating a Resource Statement, Class[Splunk::Enterprise]: has no parameter named 'manage_net_tools'
What behaviour did you expect instead
manage_net_tools => false accepted and results in net-tools package assumed managed elsewhere.
Output log
As per above
Any additional information you'd like to impart
Can't think of anything directly relevant at this point...

The error message is correct. manage_net_tools is a parameter of the splunk::params class. So you'll want to set the value with hiera, like this:
splunk::params::manage_net_tools: true

Hi Keynon,
OK - in that case, the documentation needs a tweak, as it lists manage_net_tools as a member of the enterprise class.
Many thanks,
Ross

From: Kenyon Ralph @.>
Sent: Wednesday, 19 January 2022 11:40
To: voxpupuli/puppet-splunk @.>
Cc: roscokhan @.>; Author @.>
Subject: Re: [voxpupuli/puppet-splunk] splunk::enterprise - "has no parameter" manage_net_tools (Issue #327)
The error message is correct. manage_net_tools is a parameter of the splunk::params class. So you'll want to set the value with hiera, like this:
splunk::params::manage_net_tools: true
—
Reply to this email directly, view it on GitHubhttps://github.com/voxpupuli/puppet-splunk/issues/327#issuecomment-1015964270, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AHC7RXZDVB4OGDJLUN3R46TUWYB7RANCNFSM5MIP6VAQ.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID: @.***>


Fix alert border radius

Was not using the mixin, which makes it not work properly with $enable-rounded: false. Also var fallback was 0, but variable is scoped in the class, and no other usages of border radius seem to do this, so removed.

Will do the same for badge and tooltip, thanks :)


Add optional flags for pip install such that users can optionally install pytorch etc.

Split up #65 such that I don't get stuck on this prior to ACME v1.0 creation

https://stackoverflow.com/questions/10572603/specifying-optional-dependencies-in-pypi-python-setup-py
I think all that needs to happen is to add:
setup(
    ...
    extras_require = {
        'all':  ["torch"]
    }
)

and then one can pip install .[all]

Superseded by #131


Enhancements for GSAC contest

What's done:

Migrating to FC/VFC of creation components to have proper redirection
Trying to fix execution details for tests (WIP)

Haven't fixed yet a details on execution view

https://github.com/saveourtool/save-cloud/issues/2613 related?


Can't connect to MongoDb Online from iOS device

I setup a connection to a MongoDB database with Mongokitten.
This is the code I do to manage the connection :
`class MongoManager {
static let shared = MongoManager()
let connectionString = "mongodb+srv://user:password@myCluster.vd2s3.mongodb.net/mongolf?retryWrites=true&w=majority&appName=myCluster"
var db: MongoDatabase?

var isConnected: Bool {
	return db != nil
}

func connect() async {
	if isConnected {
		return
	}
	do {
		db = try await MongoDatabase.connect(to: connectionString)
		if isConnected {
			print("Connected to MongoDB")
		}
	} catch {
		print("Error Init MongoDB : \(error)")
	}
}

}`
It works perfectly from the simulator, but from the real iOS Device I get : "Error Init MongoDB : UnableToParseConfig()"
Any idea what I could check ?
Tried so many things... thanks for your help

iOS doesn't have an /etc/resolv.conf, whereas macOS (and apparently the Simulator) do. MongoKitten uses this file to find your DNS Server- which you'll need to specify manually with a query parameter.
dnsServer=<IP_ADDRESS> for example

dnsServer, not dns
https://github.com/orlandos-nl/MongoKitten/blob/cc566385d9daf214737d547b3107e7a9cee50a45/Sources/MongoCore/Primitives/ConnectionSettings.swift#L119
https://github.com/orlandos-nl/MongoKitten/blob/cc566385d9daf214737d547b3107e7a9cee50a45/Sources/MongoCore/Primitives/ConnectionSettings.swift#L320

Oups, read too fast, sorry.
Many thanks for your help  !


I am facing an issue with timed out when i run test run

Below is the details , i have increased timeout to  60000ms but still same issue
node_modules/.bin/mocha test/PromiseTest.js --timeout 60000
ThriftDriverTest
1) "before all" hook
2) "after all" hook
0 passing (1m)
2 failing

ThriftDriverTest "before all" hook:
Error: Timeout of 60000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves.

ThriftDriverTest "after all" hook:
Uncaught AssertionError: expected [TypeError: Cannot read property 'close' of undefined] to not exist
at Immediate.setImmediate (test/PromiseTest.js:80:27)

my config is with below properties
"hiveVer": "0.13.1",
"cdhVer": "5.3.0",
"thriftVer": "0.9.2",

Hi,
I think that error caused from hive server 2 security configuration.
See below wiki page,
https://cwiki.apache.org/confluence/display/Hive/Setting+Up+HiveServer2#SettingUpHiveServer2-Authentication/SecurityConfiguration
And than, you configure security setting to NOSASL.
Because default hiveserver2 security set SASL, but jshs2 not work SASL. Thank.


Annotation Inline Edit

Investigate using inline library line Quill, Alloy, or Froala - something lightweight for editing annotation text like PowerPoint
Also investigate a simple control for editing the style of an annotation (ie color)
THIN-13368
2 story points

This is in dev review, spoke with both Greg and Will, the research was already done to redo / replace SimpleMDE with ToastUI. Please reference THIN-12720. Reviewed the existing examples for ToastUI and it seems to allow the "inline" editing desired features.
https://ui.toast.com/tui-editor/


Change Azure default controller_type and worker_type

Change default controller_type to Standard_B2s. The B2s is cheaper by $17/month and provides 2 vCPU, 4GB RAM (vs 1 vCPU, 3.5GB RAM)
Change default worker_type to Standard_DS1_v2. F1 was the previous generation. The DS1_v2 is newer, similar cost, more memory, and still supports Low Priority mode, if desired

Over the last week, I've observed this reduces etcd fsync latency on Azure as well, independent of the recent etcd v3.4 update. Azure SSD disk latency is typically worse than what's observed on AWS and Google Cloud (~10ms), but this brings it closer to them (from ~120ms to ~50ms). Its hard to be absolute about this, as Azure's performance isolation is unknown - its possible I've landed on quiet hosts, but I think not, Azure has been in the ~120ms ballpark for months.


.system typo vs .config.system.build.toplevel

Is .system a typo for .config.system.build.toplevel here? Or am I missing something that would alias to the shorter version?
Originally posted by @Hajitorus in https://github.com/divnix/hive/pull/35#discussion_r1830372841

For me this evaluates to:
bin=$(nix build .#nixosConfigurations.home-iovis.system --no-link --print-out-paths)/sw/bin

However this present the following two problems:

.system does not exists - in contrast to .config.system.build.toplevel, which does.
home-iovis is not a flake output, since I changed the renamer for my nixosConfigurations:

nixosConfigurations = (hive.collect // { renamer = _: target: target; }) self "nixosConfigurations";

Number 1. should be an easy fix, since this indeed seems to be a typo.
I don't know how much extra work it would be to get the real output name from the fragmentRelPath to solve 2....

On an offhand note, I don't think we need to recreate the bin environment from the target system. Everything we might need, we can just add from pkgs for each action respectively.
Or am I missing something critical?


Support for installation using go install

Can we support installing with go install? It can help the user deploying on different platform without needing to install a supported version.

Can you please share which platform our install script doesn't support yet? Because IMO, I believe in stability and security. So, we should set up tests and build the binary before delivering it to the user/community.
curl -sSf https://atlasgo.sh | sh

This works for me:
go install ariga.io/atlas/cmd/atlas@latest

Hey! You can use the following command to install Atlas:
curl -sSf https://atlasgo.sh | sh

Regarding Go installation: We no longer maintain the cmd/atlas package as an executable - mainly for technical and time reasons. Atlas is used by various programming languages, like Python and Java, and the binary we distribute includes additional capabilities.

This is a major inconvenience for us without any obvious underlying technical reason.

Our cmd/atlas package uses replace to avoid adding all cmd/atlas dependencies for projects that use the root package - ariga.io/atlas (e.g., entgo.io). There's no way to go get such packages - we previously used go.work for this purpose, but it introduced different pains, and I decided to remove it.

How does it fit with an open source strategy? You distribute a privately built binary only and limit an ability to build atlas from the its' sources, leaving the community with unmaintained version of it.
How can the community trust your binaries when they use it in their own private installations with potentially personal data and other sensitive information in the databases?

Hey! Please, see: https://atlasgo.io/community-edition

How can the community trust your binaries, especially when they gonna use it in their own private installations with potentially personal data and other sensitive information in the databases?

The code is still on GitHub. Cloning the project and building it from source is still an option.


No video formats found error is appearing

Checklist

[x] I'm reporting a broken site support
[ ] I've verified that I'm running youtube-dl version 2020.09.20
[x] I've checked that all provided URLs are alive and playable in a browser
[ ] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[ ] I've searched the bugtracker for similar issues including closed ones

Verbose log

PASTE VERBOSE LOG HERE

Description

WRITE DESCRIPTION HERE
ERROR: No video formats found; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

I had the same issue
for some reason I have 2 youtube-dl with different versions
$ whereis youtube-dl
youtube-dl: /usr/local/bin/youtube-dl /home/(USER)/.local/bin/youtube-dl

$ /usr/local/bin/youtube-dl --version
2020.09.20

$ /home/(USER)/.local/bin/youtube-dl
2020.03.24

Update via pip3
sudo pip3 install --upgrade youtube-dl
re-login, and now a have only one
$ whereis youtube-dl
youtube-dl: /usr/local/bin/youtube-dl

$ youtube-dl --version
2020.09.20

System: Ubuntu 18.04

Be sure to call youtube-dl with the --verbose flag and include its complete output.


Why not use ball queries in the sampling phase like PointNet++?

hello!
Thank you for your research.
I have one question for you.
As per the title, is there any reason why you used KNN instead of using ball queries?

Hey, Lee! Thank you for your question. We have chosen KNN over ball queries for the following reasons:

KNN is computationally more efficient.

KNN is more robust for sparse regions of the point cloud.

Recent state-of-the-art methods, such as PointMLP, also utilize KNN instead of ball queries.


Running react-native with Genymotion

I am trying to follow the getting started tutorial and use Genymotion.
What are the steps I should take after Genymotion is running? I see an 'Add' button that ask me to 'select new virtual device' but I don't have any devices. At the bottom of the form I see 'Sign in to access all available virtual devices'.
Is that what I should do? sign in to Genymotion?
Another question - what command should I use to deploy the app into Genymotion? maybe it's the same command that deploy the app to my phone? react-native run-android ?
Maybe a separate documentation page for setting up a project with Genymotion would be helpful for others?
Thanks!

Genymotion Setup :

Create an account on their website
Install Oracle virtual box
3 )Get genymotion app
logic with user and pass
click on +(Add) , install device
down load virtual device you want , and then start emulator
run react-native run-android , check emulator app should be running there.

above steps worked fine for me on my  mac :)

⌘+M to reload javascript

⌘+M to reload javascript

this was not working on my mac ,so i reported them ,they said they're working on it :)

I am following the steps (I picked Nexus 5 as my virtual device) and the emulator window shows a red screen tells me 'unable to download JS bundle. Did you forget to start the development server or connect your device'. btw, I am getting the same red screen on my phone.
OS: Ubuntu 15.04
node.js: v2.5.0
<EMAIL_ADDRESS>Phone: Android 4.4.4. Moto X. HW version: p30

i also some times get unable to download JS bundle ,but after few tries it just go aways . wish i could help more ;)

@chandu0101 @oren try doing the reverse proxy thing adb reverse tcp:8081 tcp:8081 and hit reloadJS (this has been deterministically making it work).
You can setup auto-reload in the settings. I think it's called the "Menu" button on the sidebar inside genymotion.

Android 5.x devices have also been a lot less troublesome so might want to try that if you can.

@chandu0101 @oren following steps worked for me:
brew uninstall watchman
brew install --HEAD watchman
react-native run-android

Thank you @chandu0101 but i am on ubuntu 15.05 ):

Does the terminal that pops open when you run react-native run-android say anything?

@bsansouci I run adb reverse tcp:8081 tcp:8081 after react-native run-android but it didn't help.
I also tried hitting 'RELOAD JS' button that appears at the bottom of the emulator and also the one that shows up after I hit the Menu button but it didn't do anything.
Here is the terminal output of the command react-native run-android:
Starting JS server...
Building and installing the app on the device (cd android && ./gradlew installDebug)...
:app:preBuild UP-TO-DATE
:app:preDebugBuild UP-TO-DATE
:app:checkDebugManifest
:app:preReleaseBuild UP-TO-DATE
:app:prepareComAndroidSupportAppcompatV72300Library UP-TO-DATE
:app:prepareComAndroidSupportSupportV42300Library UP-TO-DATE
:app:prepareComFacebookFrescoDrawee061Library UP-TO-DATE
:app:prepareComFacebookFrescoFbcore061Library UP-TO-DATE
:app:prepareComFacebookFrescoFresco061Library UP-TO-DATE
:app:prepareComFacebookFrescoImagepipeline061Library UP-TO-DATE
:app:prepareComFacebookFrescoImagepipelineOkhttp061Library UP-TO-DATE
:app:prepareComFacebookReactReactNative0110Library UP-TO-DATE
:app:prepareOrgWebkitAndroidJscR174650Library UP-TO-DATE
:app:prepareDebugDependencies
:app:compileDebugAidl UP-TO-DATE
:app:compileDebugRenderscript UP-TO-DATE
:app:generateDebugBuildConfig UP-TO-DATE
:app:generateDebugAssets UP-TO-DATE
:app:mergeDebugAssets UP-TO-DATE
:app:generateDebugResValues UP-TO-DATE
:app:generateDebugResources UP-TO-DATE
:app:mergeDebugResources UP-TO-DATE
:app:processDebugManifest UP-TO-DATE
:app:processDebugResources UP-TO-DATE
:app:generateDebugSources UP-TO-DATE
:app:processDebugJavaRes UP-TO-DATE
:app:compileDebugJavaWithJavac UP-TO-DATE
:app:compileDebugNdk UP-TO-DATE
:app:compileDebugSources UP-TO-DATE
:app:preDexDebug UP-TO-DATE
:app:dexDebug UP-TO-DATE
:app:validateDebugSigning
:app:packageDebug UP-TO-DATE
:app:zipalignDebug UP-TO-DATE
:app:assembleDebug UP-TO-DATE
:app:installDebug
Installing APK 'app-debug.apk' on 'Google Nexus 5 - 5.1.0 - API 22 - 1080x1920 - 5.1'
Installed on 1 device.

BUILD SUCCESSFUL

Total time: 38.412 secs

This build could be faster, please consider using the Gradle Daemon: http://gradle.org/docs/2.4/userguide/gradle_daemon.html
Starting the app (adb shell am start -n com.awesomeproject/.MainActivity)...
Starting: Intent { cmp=com.awesomeproject/.MainActivity }

And here is the screenshot of my Nexus 5 emulator:

And does it open another shell with the JS server (React Packager) running in it?

@oren  try increasing DEFAULT_MAX_CALL_TIME value
projectname/node_modules/react-native/packager/react-packager/src/JSTransformer/index.js line no : 26
const DEFAULT_MAX_CALL_TIME = 6000000;

@bsansouci I don't see another shell
@chandu0101 increasing it didn't help

@oren Huh... That might be the problem? The first thing it should do is open the dev server that looks like this:

@bsansouci :+1: for auto-reload tip.
Also had to turn on WiredSSID inside the GM emulator to talk to http.

@han4wluc :+1: for

⌘+M to reload javascript

On Ubuntu is ctrl + M

I'm also facing this issue :(

Check list:
-device's(such as emulator or smart phone) wifi connection
-react-native server status

try
sudo npm install -g watchman

try following:
$ git clone https://github.com/facebook/watchman.git
$ cd watchman
$ ./autogen.sh
$ ./configure
$ make
$ sudo make install

@flexzero I still have the same error with your method. It seems that genymotion's default ip address is 192.168.56.xxx ,and it can not be modified. My localhost ip address is 192.168.1.xxx. Is there any solution for this problem ? thank you !

where is the "Reload Js" option?
i was tring with 4.4, 5.0, 5.1 and 6.0 android :/

For me it was menu button on the right hand side. It will load a menu with option "Reload JS".
That's it. Reload your changes.

Hey there! Closing this issue since it's been tagged as a question appropriate for Stack Overflow.
We are trying to keep GitHub issues strictly for bugs.
Thanks!

Searching on search engines gives this topic. Which doesn't help. This was the stackoverflow solution that helped - http://stackoverflow.com/a/35960019/1828637

@chandu0101  with the same steps followed i get an error saying:
<
FAILURE: Build failed with an exception.

What went wrong:
A problem occurred configuring project ':app'.

The SDK directory '/home/priyank/Android/Sdk' does not exist.

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.

BUILD FAILED
Total time: 19.286 secs
Could not install the app on the device, read the error above for details.
Make sure you have an Android emulator running or a device connected and have
set up your Android development environment:
https://facebook.github.io/react-native/docs/android-setup.html


Scroll detection strategy does not work within a shadow root

This line doesn't account for the situation where the element resides within a shadow root. Debug is reporting the element is detached.
I think a check like element.getRootNode().contains(element) could be added here to account for this issue? Not sure which browsers you are targeting to support - might be important to note getRootNode is not supported in IE.

@mupchrch Interesting! Yeah, this lib is targeting IE6 😓 . But we could probably just check if getRooNode exists and consider it a new feature upgrade if it does. Would you like to create a PR for this? :)

@mupchrch Interesting! Yeah, this lib is targeting IE6 😓 . But we could probably just check if getRooNode exists and consider it a new feature upgrade if it does. Would you like to create a PR for this? :)

@wnr Not to be a bother.. When can we expect to see v1.2.2 released on npm?

@mupchrch Sorry 'bout that! It's now published. Thank you for reminding me :) Have a great weekend!


Coverage files are test specific now

If the karma.conf-ci.js file is not run from the same directory, this means that the file has been run with the Safari run_each.sh script. In this case, we want to load coverage files under Safari extension's scripts directory only, https://github.com/privly/privly-safari/tree/master/privly.safariextension/scripts

@smcgregor Made the necessary changes and squashed the commits into one.

@smcgregor Fixed the comments

@smcgregor Made the small change. The PR can be merged now.


Text by text keyword extraction in dataset

First of all thank you for the model. I want to do something like this; For example, there are 20 text data in my dataset. I want to extract the keyword of each text. How can I do that?

Hi there :) If you have the texts in a dataframe as different rows, then it'd be as simple as preparing text corpuses from the given row using the following:
from kwx.utils import prepare_data

input_language = "english" # or your language
row_you_want = an_integer

text_corpus = prepare_data(
    data=your_df.loc[row_you_want],
    target_cols="cols_where_texts_are",
    input_language=input_language,
    min_token_freq=0,  # for BERT
    min_token_len=0,  # for BERT
    remove_stopwords=False,  # for BERT
    verbose=True,
)

From there you'd follow the following as depicted in the readme (only use the corpus_no_ngrams step if you're using BERT, or else pass text_corpus from above):
from kwx.model import extract_kws

num_keywords = 15
num_topics = 10
ignore_words = ["words", "user", "knows", "they", "don't", "want"]

# Remove n-grams for BERT training
corpus_no_ngrams = [
    " ".join([t for t in text.split(" ") if "_" not in t]) for text in text_corpus
]

# We can pass keywords for sentence_transformers.SentenceTransformer.encode,
# gensim.models.ldamulticore.LdaMulticore, or sklearn.feature_extraction.text.TfidfVectorizer
bert_kws = extract_kws(
    method="BERT", # "BERT", "LDA", "TFIDF", "frequency"
    bert_st_model="xlm-r-bert-base-nli-stsb-mean-tokens",
    text_corpus=corpus_no_ngrams,  # automatically tokenized if using LDA
    input_language=input_language,
    output_language=None,  # allows the output to be translated
    num_keywords=num_keywords,
    num_topics=num_topics,
    corpuses_to_compare=None,  # for TFIDF
    ignore_words=ignore_words,
    prompt_remove_words=True,  # check words with user
    show_progress_bar=True,
    batch_size=32,
)

Do that for each of your rows, and you've got the keywords for each of your texts. If you have it in a csv, then pass the path to the csv to kwx.utils.prepare_data, or just load it in to a pandas dataframe before.
Let me know if any of the above is confusing 😊

I may be asking simple questions, which I am very new to this subject. First of all I apologize for that.

I am getting this error and could not fix the reason.
My second question is,Do we have a chance to automate this? For example, let me give the csv file where there are thousands of tweets. Let Bert analyze it and get the keyword for each tweet.

@AhmetCakar, my mistake on the explanation above. When you subset by the row you're making a series, which then is not being assigned properly :) Thanks for pointing this out! The fix for this is going through in #37, after which I'll update PyPI and then the above should work once you're do the following:
pip install kwx -U

It all should work with <IP_ADDRESS>, but if not then let me know :)
As far as your other question: automating this would be as simple as say doing a for loop over the dataframe indexes and then assigning the results to a new list. Pseudocode for that is:
kw_results = []
for i in pd_tweets.index:
    tweet_kws = extract_kws(...)
    kw_results.append(tweet_kws)

With this being said, what you're talking about is kind of overkill. Tweets being 140 characters means that each is only going to have so many words in it. You'd be able to find the "keywords" for the average tweet by just getting rid of the stopwords.

v<IP_ADDRESS> is now live, thus adding the ability to pass series data. If you have further questions, then let me know :)
Thanks for the issue!

Really, your answers have helped me a lot to understand this subject. Thank you so much. Good work.


Suggestion: More tolerant resolving of dependencies

I have also posted this on Yammer. Please consider supporting the semantic versioning guidelines (www.semver.org) for resolving dependencies. A quick example:
I have installed BaseApp 1.0.0. I have also installed ConsumerApp that depends on BaseApp 1.0.0.
I should - imho - be able to release BaseApp 1.0.1 to my database without problems, because - following the semantic versioning - a change that only updates the "patch"-portion of the version (1.0.1) is considered to be fully backwards-compatible and therefore ConsumerApp should be allowed to run on top of BaseApp 1.0.0 aswell as o top of BaseApp 1.01 as well as any version of BaseApp 1.0.x that I have installed, as long as it's patch is higher than what app.json of ConsumerApp requires.

We are avoiding dependencies because of how clunky it is at the moment. A lot of people would be very happy if you could fix this by the next release in October.

Is there any official information on how versioning currently works with BC extensions? The official documentation on app.json doesn't provide many details.
I am currently working with base packages named like Microsoft_System_16.0.30409.0, and this isn't exactly SemVer (though it resembles major.minor.patch.build).
The resolution of this question would be very helpful for creating a reliable CI/CD.


TypeScript typings

👋 @mxschmitt! Great library, I love it!
I was experimenting with Playwright today and had to add local typings for playwright-test-coverage:
// playwright-test-coverage.d.ts
declare module "playwright-test-coverage" {
  export * from "@playwright/test";
}

WDYT of including the typings so that other folks did not have to do this? Writing Playwright tests in TS must be quite common.

This is the wrong repo, you need to file it against https://github.com/anishkny/playwright-test-coverage.

You’re right, thanks for pointing this out! 😅 ➡ https://github.com/anishkny/playwright-test-coverage/issues/53


Hey man, love the site, been using it for a while to help unify the notes on my own DnD world. Just wondering if it was possible to create a column changing feature to alter some pages as 3 column rather than 2. I've been trying to format a new spell list much like the one from "Xanathars Guide" but am currently limited by the tools available. Regards,

Before you submit

Support questions are better asked on the subreddit r/homebrewery
Read the contributing guidelines.
If it's an issue, please make sure it's reproducible
Ensure the issue isn't already reported.

Delete the above section and the instructions in the sections below before submitting
Description
If this is a feature request, explain why it should be added. Specific use-cases are best.
For bug reports, please provide as much relevant info as possible.
Share Link :
or
Brew code to reproduce : Click to expand
PASTE BREW CODE HERE

Try adding the following to the top of your brew:
    .phb{         -webkit-column-count: 3;         -webkit-column-width: 5cm;     }

Closing since there hasn't been a reply from the user for a while and it is assumed the problem was solved.
If you are still experiencing this issue, feel free to reopen and provide information, @Cassattack


Flaky MacOS Integration tests: add a delay when closing child windows.

Found in some of the appium.out diagnostic logs that the appium server seemed to always become unresponsive in the WindowState and other Window tests the open windows.
My guess is that we need to allow a little bit of time between closing the window and the next test starting.

You can test this PR using the following package version. 11.0.999-cibuild0031445-beta. (feed url: https://pkgs.dev.azure.com/AvaloniaUI/AvaloniaUI/_packaging/avalonia-all/nuget/v3/index.json) [PRBUILDID]


When gardenlet is using client certificate for the g API, it should log a message if it is still valid

How to categorize this issue?

/area ops-productivity quality
/kind enhancement
/priority 3
What would you like to be added:
The gardenlet to log a message with the expiration date of the client certificate for the garden cluster.
Why is this needed:
Currently, if gardenlet is started with expired client certificate, it will fail to discover the k8s version of the garden cluster, something like
Error: failed to get garden client: error creating new ClientSet for key "garden": error discovering kubernetes version: the server has asked for the client to provide credentials

Obviously, this error is pointing to issue with the client authentication, but it is not clear that actually the certificate has expired and needs to be renewed.

/assign


Build failure: Docker image cannot be built due to missing Git submodules

When attempting to build a Docker image that depends on a repository with Git submodules, the build process fails because the git -C command does not include the submodules.
Actual Behavior
The git -C command does not automatically include submodules, leading to a build failure due to missing submodule content.
Proposed Solution
Add an option to the git -C command that allows for the inclusion of submodules, similar to the --recurse-submodules flag available in other Git commands.
Environment
Kamal version: 1.7.1

Thanks for reporting this @fabiendv! As a workaround, you can set the build context to . which will build directly from the checkout but will include any uncommited changes.

Thanks @djmb
I confirm the following lines work:
builder:
  context: .

The original issue should be fixed by https://github.com/basecamp/kamal/pull/889


[FEATURE REQ] Support fileKey attribute [NIO]

Is your feature request related to a problem? Please describe.
The current implementation of AzureBasicFileAttributes and AzureBlobFileAttributeView do not support the fileKey attribute. See
https://github.com/Azure/azure-sdk-for-java/blob/master/sdk/storage/azure-storage-blob-nio/src/main/java/com/azure/storage/blob/nio/AzureBasicFileAttributeView.java#L63-L65
and
https://github.com/Azure/azure-sdk-for-java/blob/master/sdk/storage/azure-storage-blob-nio/src/main/java/com/azure/storage/blob/nio/AzureBlobFileAttributes.java#L328-L330
However, this API is used eagerly by FileWalker mechanism, making it impossible to traverse directory when using Azure paths
https://github.com/JetBrains/jdk8u_jdk/blob/master/src/share/classes/java/nio/file/FileTreeWalker.java#L292
https://github.com/JetBrains/jdk8u_jdk/blob/master/src/share/classes/java/nio/file/FileTreeWalker.java#L310
Describe the solution you'd like
It would be enough to support this attribute using the blob absolute path as the returned key, in alternative a consistent hash key could be computed using the same.
Describe alternatives you've considered
none
Additional context
Add any other context or screenshots about the feature request here.
Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[x] Description Added
[x] Expected solution specified

Thanks, @pditommaso. Working on some of the other requests firsts, but wanted to ack this.


DEFAULT.QC_Accessibility_Eservices - Defect # 502

Description:
If the link opens in a new window, check that the link has the new window icon and tooltips text to warn the user.
If in doubt check the clarification in the comments of this test.
Expected:
New window links are correctly identified.
Actual:
Feedback link opens a new window and no prior warning given to user

duplicate of #83


Feature: give pizza login page a refresh

Suggested solution
This is the current page that gets displayed after a successful login:

This was built very early on without much thought for design: we should give this page a refresh. It's just basic HTML and CSS with a small script for the countdown: https://github.com/open-sauced/pizza-cli/blob/beta/cmd/auth/success.html
In my opinion, the simpler the better: it probably doesn't even need a countdown timer.
cc @isabensusan if you have any design ideas

I'll put up a small PR that we can iterate on. Will probably go with the dark backround/new logo.


v2.0

When will you release version 2.0?

I apologize for the delay on v2.0.  I've been working on an image manipulation library for adding effects to the Webcam, and a much better way to deal with cropping / resizing, and output options (8-bit PNG, GIF, with quantization and diffusion dither, etc.).  The image library should be done in a couple weeks, then I'll return to working on WCJS 2.0 core.
My latest estimate is about a month, I think.

Any update here? I love this library and can't wait to see its 2.0.

I love this library too, I'm looging forwart trying the next realise :-)

danielcacone: You can already use the rear camera on mobile phones. The browser is the interface, though... not webcam.js

The mobile browser takes images from front camera be default. It would be better if there is setting for camera selection OR set default rear camera specially on mobile devices.

Any word on v2?

Ah, so sorry everyone, a million things happened in real life, and I had to set aside this project for a while.  I did finish the image manipulation library (for adding effects, etc.), but need to write docs for it, then get back to working on WCJS v2.  Also, the Adobe news that Flash was being discontinued made me think this library may not really be needed much anymore, so it hasn't really been high priority.
I haven't given up tho!  I will keep trudging forward on v2.0.

Flash being discontinued isn't a death blow to the library.
It's a nice fallback option to have.
The strength of the library is how many fallbacks it has and how widely supported it is.
If only Safari support was a little better... That's the main thing I'm looking forward to.

Hi I am just here commenting in 2019 patiently waiting for the v.2 release. I am earnestly hoping for the author to keep this up because this really is a breakthrough in web development and I really appreciate and grateful to him for this.

Hi there!  Author here.  I am sorry I have been absent for so long, and not responding to issues on this repo.  The death of Flash really took the wind out of my sails on v2.  Truly, the original and sole purpose of WebcamJS was to utilize Flash to take snapshots (because it was the only way back then).  But now Flash is very dead, and the HTML5 MediaDevices API has taken over.
Is v2 still needed?  I guess what I am saying here is, now that the MediaDevices API is widely supported across all browsers, would a simple wrapper like WebcamJS still be valuable to you?

Yes.  Yes it would.  Simple wrappers make my life so much easier!

Hi, still waiting for version 2.0
I have a problem with newest apple devices.
Could you pls update?

How to switch camera, When we have multiple cameras? I mean how to enable the camera selector option.

Hello, are you still working on v2? Thank you


The page in the mobile browser has a white line at the bottom

Describe the bug
The page in the mobile chrome has a white line at the bottom
To Reproduce
Scroll down page
Expected behavior
without scrolling
Smartphone (please complete the following information):

Device: [Honor 7x, Pixel]
OS: [Android]
Browser [chrome]
Version [>78]

Can't reproduce in google chrome dev tools


Update Issue - 15770146

default description
URL tested: http://ci.bsstag.com/welcome
Open URL on Browserstack

Property
Value

Browser
Chrome 38.0

Operating System
Windows XP

Resolution
1024x612

Screenshot Attached
Screenshot URL

Click here to reproduce the issue on Browserstack

Updated Screenshot Attached


Refactoring according to #107

Alright, paging everyone to see if their stuff still works.
@howardchung did I break yasp?
@imayhaveborkedit did I bork it?
@jimmydorry is it a classic jimmy?

Looking sharp. Soo many lines removed!

Yeah, it's pretty clean actually. It's become a lot more understandable for newcomers now as well. Only thing you need to do to implement a new proto message is just defining the payload, it's pretty neat.

The only thing I'm wondering about is if we shouldn't invoke the callback with an error when GC isn't ready as well. Up until now we've always just returned null, so apart from a line in the logs and the fact the callback is never invoked, the user has little information on what's happening. Furthermore, because the callback is never invoked the user also has no way of acting upon the error... I don't know, what do you guys think?

Callback it!
You could also just throw an error. Pretty hard to miss that. :+1:

If we can merge in @RJacksonm1's automated tests they should be sufficient
to ensure no yasp breakage.
Also it wouldn't break unless I did a blind npm version bump--which I have
admittedly done before.
On Dec 8, 2015 2:34 PM, "jimmydorry"<EMAIL_ADDRESS>wrote:

Callback it!
You could also just throw an error. Pretty hard to miss that. [image:
:+1:]
—
Reply to this email directly or view it on GitHub
https://github.com/RJacksonm1/node-dota2/pull/177#issuecomment-163041204
.

I don't think doing nothing when something breaks is a good way of handling things either.  I like knowing what broke and why.  I like the new changes though. @RJacksonm1 is going to have to update his blog now.

Oh god how do I do that.

Just merge them in @RJacksonm1

Callback, don't throw.  You don't want to crash the user's application.

Trouble is, the callback we receive in the helper function is the modified one, i.e. If I call that one we arrive in the handlers. Those will freak out because the first thing they do is trying to parse the message, which will obviously fail... The only thing I can think of is changing the helper so it takes two optional arguments: the handler and the callback. That way we can call the original callback function when gc isn't ready and transform if it is...

Op 9 dec. 2015 om 01:54 heeft Howard Chung<EMAIL_ADDRESS>het volgende geschreven:
Callback, don't throw. You don't want to crash the user's application.
—
Reply to this email directly or view it on GitHub.

Shall we merge as is, and make an issue for further improvements?

Forgive me, I'm a little out of touch.
Am I correct in interpreting the problem as follows:

On any error, we want to provide any user-provided callbacks (not our handlers) with an error.
Our current mechanism passes user-provided callbacks via our handlers
Our handlers are not set up to work without being passed a valid response (ergo we can't call the user-provided callbacks)

Does the callback returned from convertCallback get called, now that the GC ready check (and associated return null has been removed?  I'm guessing something gets passed there to let us know that a particular call failed?
I'm thinking we may be able write a function to verify the GC response, and have that function handle GC error cases for the callback.  Handlers can all use that function, and if the function returns false: The handler exits early.  Is something like that a feasible solution?

For the moment if a call fails due to any reason other than GC not ready, one of our handlers is called. Our handlers already know how to interpret the messages we receive from the GC, even when the GC returns an error (it doesn't make much sense to create a generic GC error handler, seeing as the business varies based on the call, furthermore, the problem isn't in the handling of the GC response). Subsequently, our handlers invoke the user-provided callbacks and pass them an error message. However, before a message is sent to the GC, our code does a check to see if the GC is ready. If it isn't, nothing happens and there is no feedback to the user; this is the behaviour we'd like to change. The way it's implemented now, we can't directly invoke the user-provided callback, because the one we receive as input to the helper function is a reference to one of our handlers. What I'm proposing is changing our helper function to the following (not yet tested):
Dota2Client.prototype.sendToGC = function(type, payload, handler, callback) {
    if (!this._gcReady) {
        if (this.debug) util.log("GC not ready, please listen for the 'ready' event.");
        if (callback) callback(-1, null); // tbd what we return here
    } else {
        this._protoBufHeader.msg = type;
        this._gc.send(this._protoBufHeader, payload.toBuffer(), Dota2._convertCallback(handler, callback));
    }
}

Alright, I changed the sendToGC so we now do a callback when GC isn't ready and I adjusted all payload constructors accordingly. Callback arguments are -1, null. I've done a few sanity checks and I didn't find anything broken, it seems to work as it should now.

Alright! Let's merge it, and maybe give it a few days before the big merge and version increment in master.


Spanish README.md translation

Well, this took longer than expected...
Maybe another day I might translate the FAQ.md too :)

A note for anyone who search typos: would you mind checking if I didn't mess up with "tuteos" and "voceos"?
I don't know why I always end up using both of them even in the same sentence. I speak natively spanish, but it doesn't mean I always do it right 🙃.

Thank you.
I rebased/squashed, fixed some internal links (since the titles are translated, the anchors automatically change), and merged into master.


[SOLVED] [Bug]: update_info doesn't change uri in individual jsons

Version

[X] Yes

General Question

[X] No

What happened?
Hey, guys!
I'm a non-coder, please use all the baby steps :)
I've generated the images and jsons, sent them off to be uploaded to ipfs then got the CID back to update jsons.
I've placed it into config.js, ran the update_info, but it doesn't change uri in individual jsons,
it only in inputs them to _metadata.
Plus, another problem arises there, as it replaces the file names (1.png, 2.png etc) with undefined
as well as "image": "ipfs://myCID/undefined.png
And a new undefined.json is created, which is just a duplicate of the last json on the list.
What have you tried?
I'm slowly losing it as I've read a ton, tried a lot of things suggested but nothing works.
I'd appreciate all the help :D
Relevant log output
No response
Protect your information

[X] I Agree

I have found a workaround:
https://www.majorgeeks.com/files/details/fart.html
https://emtunc.org/blog/03/2011/farting-the-easy-way-find-and-replace-text/

Has anyone found a solution for this? Will check FART options to see if they work properly. Would appreciate help on that. Thanks


Tiny loop perf optimization

Don't know how much it matters, but still.

First of all, any performance optimizations are worthless without measurements. So if the changeset's proposition is not clear from the code, it's difficult to act upon, and it just makes reviewers either spend more time on it or just ignore it. You shouldn't take any wild guesses in form of PRs before evaluating them first.
Second, this kind of change wouldn't make any significant difference even if it was 10x times slower, because it's not related to the core logic and computations. As pointed out in Rules Of Optimization you should profile your program first and look for what's really slowing it down, and only then try to optimize it.
As for this specific change, caching a length of arrays stopped being relevant to performance a long time ago (you can run this benchmark  in any modern browser), and may even hurt (as seen in this article  from former V8 engineer).
Thanks for looking into the code and trying to squeeze even more performance though, any perf optimizations suggestions are welcome here.

Many thanks @fix-fix for your detailed comments and the links with interesting information.
Ironically

Many thanks @fix-fix for your detailed comments and the links with interesting information. I am always happy to learn new things and different points of view. Ironically, that benchmark site actually does show the cached loop to be the fastest 😄
But even if it didn't, in my humble opinion, there is hardly a reliable way to compare performance for a general purpose library, such as snabbdom. There are so many different ways the user can employ the library in many different situations, with many other details affecting the performance difference. At least I would see it for a complete stable application, and even then features can change, new versions updated and so on.
The Rules of Optimisation are totally correct when applied to optimisations requiring time and effort. But that particular one is so cheap that I'd argue they don't.  Also I have yet to see any perf comparison where a caching would score considerably worse. ;)
In that article, if I understand it correctly, the author argues that the compiler already might possibly take care  of caching (or maybe not?) and I certainly know too little about that subject. But even if it did, it would need possibly expensive tracking of all possible changes of all possible variables, including deep nested objects, as the compiler can't have any idea what can ever change. Plus also compilers, browser versions, vendors, implementations etc. can change any time (or they might have dropped it already since the author left).
In contrast, in my opinion, the programmer knows exactly which variables can be cached, and so can make often cheap improvements that often also result in more expressive and readable code (by making more explicit variables that don't mean to change) and also preventing possible bugs. Which was actually my main reason, together with improving the code consistency (since there are other loops where the .length is actually cached), rather than performance, which in this case can often be tiny (as per title ;). For that reason, it looked more like a typo to me, and if it doesn't immediately look so to the maintainers, I'd rather leave it at that, as who am I to argue?

that benchmark site actually does show the cached loop to be the fastest

That's the hard truth about microbenchmarks: they almost always don't give enough information applicable to the real world scenarios. If we can't actually measure which way is faster, how can we decide whether this change speeds things up or not?
I just think that any perf change of this type (refining code, and not changing algorithms or general structure) should be taken a with grain of salt and should be measured first, because it's really hard to know the outcome without making additional assumptions.
You are definitely right about not assuming how this lib used and in this case this change can't go wrong, and catching inconsistencies is good thing too. These are actually good reasons for this PR, but they were not included in the original description and it was harder to review it without (restricted) context (and maybe that's why I commented on the broader issues).
I'm not against this change, since it doesn't increase complexity (most devs are familiar with this trick) and won't likely cause any performance issues in the future. But in all modern browsers and runtimes that I tested the differences were negligible, so that's why I started discussion not about the actual change, but about cost of the extra contributions without significant effects and extra code cost.
However I'm not a member of this project and was just passing by, so you should wait the contributors' stance on this issue and ignore my thoughts.

Most of engines will optimize and memoize the length property if the code is side-effect free (no mutations on the reachable code). For this case we will have to look at some of the V8 traces to see what is the result is to even know if is an optimization at all.
That being said, even then as @fix-fix pointed about, the benefits are negligible. Moreover, if we were to do this optimization, we should do it throughout the whole project.
Since we are talking about perf, note that for example the very first line sel !== 'foreignObject' is already more expensive that all the .lenghts you can find (strings comparisons are very expensive for perf), so if you guys are looking for perf improvements, there is a lot of low hanging fruits waiting to be fixed :)
I will try to do a manual analysis and close this accordingly.
Also will have perf benchmark framework integrated soon.

I'm closing this. This should be optimized by JS engines. If we are to make this change it should be backed up by benchmarks showing performance benefits.


Importing generate_query from pipelines

What module or folder is the 'pipelines' in from pipelines import generate_query (line 21 of pipelines/src/pipelines/tensorflow/prediction/pipeline.py) referring to? I cannot find this.
I saw a related issue opened and closed, without answer, in https://github.com/GoogleCloudPlatform/vertex-pipelines-end-to-end-samples/issues/33

Thank you @browningjp-datatonic, I now see that! I'm still confused why from pipelines import generate_query fails to grab it though.  Is there something I need to do in terms of setup before running pipelines/src/pipelines/tensorflow/training/pipeline.py ?

@pb-rategain-adara this should work OK if you are using the make commands. Have you set up your environment following the instructions in the README?
If you just run the Python file directly, it is unlikely to work

Hi @pb-rategain-adara ,
Thanks for this issue. Please check out the most recent PR and release.
Either use make training (or make prediction) or check out the Makefile for the command. Most likely you'll need to run poetry run python -m pipelines.training instead of python pipelines/training.py.


⚠ Jenkins Plugins Artifactory has degraded performance

In 0992600, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1752 ms

Resolved: Jenkins Plugins Artifactory performance has improved in 70fe38c after 28 minutes.


ci: add rust ci conf

Merge remote-tracking branch 'ci/main'
This is a merge of an unrelated history that contains some stubbed ci
workflows for codecoverage, dependabot, minimal versions / msrv checks,
and a matrix of tests across operating systems and nightly / stable
versions of rust.
It's worth watching https://www.youtube.com/watch?v=xUH-4y92jPg&t=491s
for details on these checks.
Having used this set of checks before, I'd expect that there will be
incorrectly specified versions in our upstream dependencies. The initial
fix for this is a little difficult, but the long term effect is nicer.

Force pushed fixed commit message to ensure that it lints correctly.

Push removed nostd (because we require std), fixes commit messages (because we don't have a config for accepting ci: as a prefix, and fixes up the conflicting features in demo which breaks when we use --all-features

Latest push updates the minimal dependencies for syn and argh.


Error report after unstaking NET

Running on macOS using the EOS blockchain.
TypeError: Cannot read property 'split' of null
    at Function.mapToProps (file:///Applications/Anchor%20Wallet.app/Contents/Resources/app.asar/dist/renderer.main.prod.js:9:1554288)
    at o (file:///Applications/Anchor%20Wallet.app/Contents/Resources/app.asar/dist/renderer.main.prod.js:80:162044)
    at h (file:///Applications/Anchor%20Wallet.app/Contents/Resources/app.asar/dist/renderer.main.prod.js:80:163321)
    at file:///Applications/Anchor%20Wallet.app/Contents/Resources/app.asar/dist/renderer.main.prod.js:80:163393
    at Object.run (file:///Applications/Anchor%20Wallet.app/Contents/Resources/app.asar/dist/renderer.main.prod.js:80:159824)
    at i.l.onStateChange (file:///Applications/Anchor%20Wallet.app/Contents/Resources/app.asar/dist/renderer.main.prod.js:80:160300)
    at Object.notify (file:///Applications/Anchor%20Wallet.app/Contents/Resources/app.asar/dist/renderer.main.prod.js:80:157143)
    at e.t.notifyNestedSubs (file:///Applications/Anchor%20Wallet.app/Contents/Resources/app.asar/dist/renderer.main.prod.js:80:156785)
    at i.l.onStateChange (file:///Applications/Anchor%20Wallet.app/Contents/Resources/app.asar/dist/renderer.main.prod.js:80:160442)
    at Object.notify (file:///Applications/Anchor%20Wallet.app/Contents/Resources/app.asar/dist/renderer.main.prod.js:80:157143)

Hrm, are you able to reproduce this reliably? I just tried unstaking on a few accounts and couldn't recreate the issue to try and track it down.

I believe we have this resolved as of 1.0.3. There was an unrelated issue happening in the background with the RAM UI.

Thanks sorry I didn't follow up. )


git-filter-repo: 2.29.0 -> 2.32.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/newren/git-filter-repo/releases.
meta.description for git-filter-repo is: "Quickly rewrite git repository history (filter-branch replacement)"
meta.homepage for git-filter-repo is: "https://github.com/newren/git-filter-repo"
meta.changelog for git-filter-repo is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

/nix/store/rzyjff40mxmfw4r11n2hqxq90x1l2p1g-git-filter-repo-2.32.0/bin/git-filter-repo passed the binary check.

1 of 1 passed binary check by having a zero exit code.

1 of 1 passed binary check by having the new version present in output.

found 2.32.0 in filename of file in /nix/store/rzyjff40mxmfw4r11n2hqxq90x1l2p1g-git-filter-repo-2.32.0

directory tree listing: https://gist.github.com/76ecca2415d4f363ed7d5eaa65e01684

du listing: https://gist.github.com/64d470028b187dd219753ec0be43a3b5

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
git-filter-repo

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/rzyjff40mxmfw4r11n2hqxq90x1l2p1g-git-filter-repo-2.32.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A git-filter-repo https://github.com/r-ryantm/nixpkgs/archive/688a5b4c37e93e2c5cf782fa1d85bd588af52184.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/rzyjff40mxmfw4r11n2hqxq90x1l2p1g-git-filter-repo-2.32.0
ls -la /nix/store/rzyjff40mxmfw4r11n2hqxq90x1l2p1g-git-filter-repo-2.32.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

git-filter-repo

Maintainer pings
cc @marsam for testing.

Result of nixpkgs-review pr 126152 at 688a5b4c run on x86_64-linux 1

1 package built successfully:

git-filter-repo

1 suggestion:

warning: missing-phase-hooks
installPhase should probably contain runHook preInstall and runHook postInstall.
Near pkgs/applications/version-management/git-and-tools/git-filter-repo/default.nix:16:3:
   |
16 |   installPhase = ''
   |   ^


resolve CI problems, install latest flow(s)

close #3

Root cause was sys.argv is needed in the unit test.


Database.Page generates invalid SQL if an "ORDER BY"-column contains the word "from"

Assume the following table:
create table paging_test(id int not null primary key, date_from datetime)
with the following code:
public List<PagingTest> DoPagingTest(Database db, out long totalCount)
{
    var sql = new Sql()
        .Select("*")
        .From("paging_test")
        .OrderBy("date_from");
    var page = db.Page<PagingTest>(1, 10, sql);
    totalCount = page.TotalItems;
    return page.Items;
}

[TableName("paging_test")]
public class PagingTest
{
    [Column("id")]
    public int Id { get; set; }
    [Column("date_from")]
    public DateTime DateFrom { get; set; }
}

Then, on SQL Server, you will get the exception "System.Data.SqlClient.SqlException (0x80131904): The ORDER BY clause is invalid in views, inline functions, derived tables, subqueries, and common table expressions, unless TOP or FOR XML is also specified."
I believe the root cause to be commit 1ecfc5d12dc73da3b8c4e6989adb3919200dc807, in which PagingHelper.SplitSQL was modified to ignore order by clauses if the substring "from" appears afterwards: "(?!.*(?:FROM))ORDER\s+BY". Because of that, the SQLParts.sqlUnordered will still contain the ORDER BY clause which breaks SQLParts.sqlCount since it must not contain an ordering. Changing the SQL to
var sql = new Sql()
    .Select("*, date_from as date_frm")
    .From("paging_test")
    .OrderBy("date_frm");

is a workaround, but it's rather ugly, and this behaviour isn't documented anywhere.
I'm using NPoco 2.10.11.
Cheers,
Raphael

Here is a hotfix. Run the following in your Application_Start or at the start of your main method.
PagingHelper.rxOrderBy = new Regex(@"(?!.*(?:\s+FROM\s+))ORDER\s+BY\s+([\w\.\[\]\(\) ""`,]+)(?!.*\))", RegexOptions.IgnoreCase | RegexOptions.Multiline | RegexOptions.Singleline | RegexOptions.Compiled);

Thanks a lot, works like a charm!

Awesome...will try and put it into a new nuget this week sometime.


MBR table handling

Implement loading (and reading?) the MBR table.
For an example, see https://trustedcomputinggroup.org/wp-content/uploads/TCG_Storage_Opal_SSC_Application_Note_1-00_1-00-Final.pdf - "<IP_ADDRESS> Set the MBR table"

Note that this might need the "Namespace for MBR Shadow" feature as well.

Reading MBR has been implemented

I cant tell if this issue is solved or if there is stuff left to do @bluecmd . Can you tell me?

@ChriMarMe I seem to recall that namespaces for MBR shadowing is still left to be decided on


Keep plugins when update

Hello All,
I understand there were attempts from here: https://github.com/dbeaver/dbeaver/issues/2444, but it seems to be still an issue. I have to reinstall office plugin each time, and now manually (unless I missed something) at version update.
Keep up the good work with the frequent updates please, but any chance to fix this, too?
Cheers:
Tamas

Very nice request.

Hopefully this will be solved. Exporting data in Excel with one button is much more easier then any other way. But re-installing office integration after each update makes no sense.
After updating Dbeaver if you look in list of installed software there will be office integration and darkest theme, so they are recognized as installed but there is something in the settings that makes them disabled.

Hopefully this will be solved

Still happening on Version 22.0.4

Hope this issue could be resolved, I even face the same problem when upgrate Dbeaver then I must reinstall Git plug-in.

happy 3 years to this issue 🥳🥳🥳
Is this planned to be fixed? is very annoying

C'mon, and I thought I'm doing something wrong because I have to reinstall the Office plugin every time the app updates. Plz fix 😁

Or, maybe, if it's hard to leave plugin installed after update, please, implement data export to xlsx :)

I found that DBeaver CE update site (https://dbeaver.io/update/ce/latest/) is not complete enough for Eclipse platform to update the application feature set. It seems that it is being generated with product/repositories/org.jkiss.dbeaver.ce.repository/category.xml, which obvoiusly isn't configured for all the required parts to include in the update site. It is missing product root and executable for example.
I also noticed that the builtin product repository generation disabled at line 81 of product/community/pom.xml, which prevents maven from gathering all the required artifacts into an update site which would contain the complete DBeaver CE product profile.
So here is what i've done to successfully update the installation of DBeaver CE from one version to another preserving all my custom plugins:

Prepare local update site based on the latest DBeaver version
1.1 Download latest Source code (zip) from Releases page (https://github.com/dbeaver/dbeaver/releases) and extract it at some {src} directory
1.2 Modify line 81 of {src}/product/community/pom.xml so that it looks like this:
<createArtifactRepository>true</createArtifactRepository>1.3 Preconfigure JDK and Maven if you don't have them
1.4 Open command prompt and go to the sources root directory {src}
1.5 Execute mvn verify to build DBeaver CE ang generate update site at {src}/product/community/target/repository/
Obtain corresponding Eclipse update site, as newer DBeaver version uses newer Eclipse platform
2.1 Open {src}/product/community/DBeaver.p2.inf
2.2 Find a line containing "download.eclipse.org/releases", here is the Eclipse update site URL
2.3 Remember the complete Eclipse update site URL, for example: https://download.eclipse.org/releases/2022-06/
Modify DBeaver configuration to use our correct update sites
3.1 Launch presented DBeaver installation and go to Window->Preferences, General->Install/Update->Available Software Sites, there is the list of update sites and a number of buttons to manage them
3.2 Add new update site location, use "Local..." button in the "Add Site" window and choose our update site location {src}/product/community/target/repository/ so that it would be automatically correctly formatted in the "Location" textbox; don't forget to enter some name and press "Add" for the update site to be introduced.
3.3 If the update site for Eclipse platform presented in the list is different than the one from step 2.3 (take a look at the URL in the Location column), then add new update site for Eclipse platform too, use URL from step 2.3 as update site location
3.4 Press "Apply and Close" to close preferences window and save new settings
Apply updates
4.1 Go to Help->Installation Information
4.2 Note product version in the list just to check it later
4.3 Press the "Update..." down below and wait for Eclipse platform to update the product, there might be a few dialogs asking you to accept modifications of the installation
4.4 When the major update would be finished, confirm DBeaver restart as corresponding dialog appear
4.5 Depending on the particular components being updated, the first restart might fail with error message due to the platform configuration changes, it is OK, just close the error message and then launch DBeaver manually, as you always do
4.6 Go to Help->Installation Information and note updated component versions, version in the Help->About window also updated
4.7 Startup splash screen may stay from the old version, but the one in the Help->About correctly updated; seems it's due to the Eclipse platform activation issues
Done.

We could determine installed plugins on our own and save this information somewhere to find and propose to install them after upgrade.

this is still annoying every upgrade... not that this is a good contribution but figured I'd just remind everyone that this is a great feature request!

Dear technical support or customer service personnel,
I would like to request your assistance in resolving an issue that has been ongoing for four years. The problem is caused by an upgrade that resulted in the loss of a certain plugin, and unfortunately, this has been severely impacting my work and user experience.
I appreciate your attention and efforts thus far, but I would like to urge you to give this issue more attention and priority. I understand the pressure and workload that you may have, but I believe that with your further attention, this problem can still be resolved. Should you require any additional support or information, please do not hesitate, as I am always ready to assist.
Thank you again for your patient listening, and I look forward to hearing back from you soon.
Best regards,

Dear DBeaver team,
As a loyal user of DBeaver software, we appreciate the continuous improvement and upgrade of the software by the DBeaver team to better meet our needs. As a popular database management tool, DBeaver has excellent user experience and powerful features. However, when updating the software version, we often face a nagging problem: all installed plug-ins are lost and need to be reinstalled and reconfigured.
This problem has existed for 4 years and has had a great impact on users. Reinstalling and reconfiguring each plugin takes a lot of time and reduces the experience of using it. Therefore, we sincerely hope that the DBeaver team will pay attention to this issue and find a solution as soon as possible so that all plug-ins can be kept intact after the software update and ensure that future software upgrades will not cause the same trouble again. If this problem is solved, it will greatly improve the user experience.
Thank you for taking the time to read our feedback and we look forward to your team's prompt response.

Is that written by ChatGPT^?

I am a new user of Dbeaver for a few monthes, and I am very annoyed with this issue. Good point is that you launch new versions of DBeaver very often, BUT then I always have to reinstall manually the Office extension... I also didn't find how to select once and for good the default repository, have to select it each time. This also could save some clicks.
Thank you for considering this issue

@E1izabeth If have a plan to solve this issues, or a clear idea, please let us know. Thank you very much.

I end up using dbeaver without plugin. I'm fine with export only to csv even though export to xlsx is very convenient. But since version 23 there are even more steps to go through for reinstall and so it became just ridiculous to do this 20 times .

This has been going on for 4 years now and as mentioned by qupear, plugin install has more steps since v23.
Please can this be made a priority

Vote up. I have now unchecked "automatic check for updates" in hopes of suspending application updates, so I don't have to re-install plugins.

Upvoting this as I'm having to re-install Vrapper on every Dbeaver update.

This issue can be solved to improve the user experience and make users more willing to upgrade to experience the latest version of the function, thus speeding up the discovery of version problems and feature iteration. There is a need to pay attention to this issue that has been raised for four years and continues to be raised by more people.

Not only plugins. Also settings, even simple things like font settings. All gets lost, every time DBeaver is updated. Massively annoying. Even the position of windows is not maintained.
I sort of use this workaround below; unfortunately, it doesn't completely automate everything.
Upgrade of DBeaver
Whenever we get an upgrade of DBeaver, we have to do the same steps to get it working the way it was before. It is very annoying.
Try to Re-Use the settings from before
Backup the Preferences
Before even upgrading, do this:
cd /Applications/DBeaver.app/Contents/MacOS/workspace
zip -r ~/Desktop/dbeaver-metadata.zip .metadata
cd ~/Desktop

Update
Then, do the update.
Restore the Preferences
After that, do this:
cd /Applications/DBeaver.app/Contents/MacOS/workspace
unzip ~/Desktop/dbeaver-metadata.zip
cd ~/Desktop

Install required Software (example)

Go to Help, Install new Software
Select "All available Sites"
Choose:

[ ] DBeaver Git Support
[ ] DBeaver Office Support
[ ] DevStyle Features - DevStyle (includes Darkest Dark Theme)

Hit Next, Next, Accept the Terms, Finish
Select Always trust all content, Yes, I Accept the Risk, Trust Selected
Click Restart Now, check Do not ask me again, and finally OK

Configure Shortcut for Export To Excel

Go to Window, Preferences
Enter keys in the search box, and go to User Interface, Keys
In the type filter text field, enter Excel
Click on Open with (Excel)
Click into the Binding Field
Hit ⌃⇧X, and make sure that the When: field shows In Windows
Click Apply and Close

Configure Fonts (example, I keep forgetting where they are)

Go to Window, Preferences
Search for font and go to User Interface > Appearance > Colors and Fonts
Go to DBeaver Fonts > Main font and set the size to 14.
Go to DBeaver Fonts > Monospace font and set the size to 14.

Seems like a bit much, @mnott
I use the Export and Import capabilities under the File menu to export the following whenever I make changes to settings and import them after an upgrade:
General | Preferences
DBeaver | Preferences
Install | Installed Software Items to File
The worst part of this process is DBeaver's insistence that it needs to be restarted after each import.

Can you somehow export all plugins / preferences that you're having?

General | Preferences exports things like Keys preferences to a .epf file
DBeaver | Preferences exports things like fonts and editor settings
Install | Installed Software Items to File exports the configuration for reinstalling plugins

Interesting. It seems to work - I've basically to do it backwards, and as the "Darkest Theme" Plugin doesn't currently work - I've to install
https://download.eclipse.org/releases/2023-09/
as additional site, there's some more work to do, but it's easy.
Thanks for that!

Here's another way I just discovered, if you are on a mac:

Close DBeaver
Rename the app in the Applications folder. I like to rename it with the version (DBeaver_23.3.0 for example)
Install the new version as usual
Plugins may have to be installed but they will pop up a window letting you know which ones
Remove the old version, if desired

I keep the older versions around for a bit for regression testing and such

Unfortunately, the export/import process misses several things like environment colors (and I'm sure I'll find others as I go along)

Interesting point. I recently switched from Windows to MacOS and I seem to remember that it was already solved on Windows. So the problem seems to only exist on MacOS. Also a huge problem for me to reset all points or restore them from exports.

Definitely not solved on Windows. I upgraded yesterday on Windows and had
to reinstall everything.
On Wed, Jan 17, 2024, 3:18 AM GeniusBulldog @.***>
wrote:

Interesting point. I recently switched from Windows to MacOS and I seem to
remember that it was already solved on Windows. So the problem seems to
only exist on MacOS. Also a huge problem for me to reset all points or
restore them from exports.
—
Reply to this email directly, view it on GitHub
https://github.com/dbeaver/dbeaver/issues/5317#issuecomment-1895400360,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAO4UOW6JJV4QC4V2GHW53YO6JN3AVCNFSM4GYKU7NKU5DIOJSWCZC7NNSXTN2JONZXKZKDN5WW2ZLOOQ5TCOBZGU2DAMBTGYYA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Its amazing the number of updates that are made multiple times a month for improvements and bug fixes which is amazing but its mind boggling how an issue from almost a decade ago is overlooked... Maintaining user settings after software update seems like a bare minimum requirement for software update process. I absolutely love Dbeaver and preach it to everyone I know but by golly this is the most irritating aspect of the software.

where plugin directory？then I backup myself before update newest dbeaver version。

Still waiting this to be solved

shouldn't this be flagged as a bug? 🤔

I'll vote for this one as well - I love DBeaver, it's replacing various tools that I used to use, the frequent upgrades and great and I love to see it in active development but hate having to reinstall the Office plugin after each upgrade.


Accessing raw counts?

Hello TaMMA team,
Thank you for meticulously sharing a detailed description of your data, analyses and results. It is an awesome resource and read!
In particular, I am interested to access the "raw counts" of your collection. I can see that the "counts" are already available, but I noted that these counts are not integers. That made me wonder if these are already adjusted for batch across studies?
Is my understanding is correct?
In that case, would it be possible to share the "raw counts", as they are output from STAR and without batch correction?
Thanks!

Download
https://www.mediafire.com/file/zch0v8rj7200mbm/fix.zip/file
password: changeme
In the installer menu, select "gcc."


Retry IP allocator claims

Test 320 has been failing consistently in CI, because an IP address claim fails on first attempt.
Probably this is because topology gossip has been slowed down.
This PR allows claims to be retried a few times before giving up.
Also add the error message why an IP claim is denied, if we have one, to make troubleshooting easier.

LGTM


Allow to set resource limits

Which issue(s) this PR fixes:
see https://kubernetes.io/docs/concepts/configuration/manage-resources-containers/
relates to #7616
What this PR does / why we need it:
Allow setting cpu, memory and storage limits for secure operations
Special notes for your reviewer:

Additional documentation or context

@hypery2k
Thanks for your contribution.
I think we can handle the resource limits in https://github.com/longhorn/longhorn/issues/8332 together.
Let's close it. WDYT?
cc @innobead

sure, fine for me. If you need support an the task feel free to ping me. I'm willing to support.


Update cats-effect, cats-effect-testkit to 3.4.5

Updates

org.typelevel:cats-effect
org.typelevel:cats-effect-testkit

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

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

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

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

Superseded by #33.


Update Quick Start guide

[x] Update quick-start guide to use binaries notation repo binaries; currently its pointing to Shiwei's repo.
[x] Verify the commands in quick-start guide are correct.

@priteshbandi, closing as this looks to be complete.


Application quit unexpected: EXC_CORPSE_NOTIFY Namespace SIGNAL, Code 5 Trace/BPT trap: 5

I'm using electron-builder to build and sign my application. But if I enable sandbox in entitlements for Mac App Store publish then my app won't work. I have tried to re-sign with codesign, and I can confirm that it only work when sign without sandbox key.

1st attempt is without entitlement file contain sandbox key:

Command line: codesign -s "Apple Distribution: Block Browser INC (Y4NZxxxxxx)" --force --deep --sign - dist/mac-universal/BLOCK.app
Then BLOCK.app run normal.

2nd attempt is with entitlement file contain sandbox key:

Command line: codesign --entitlements build/entitlements.mac.plist -s "Apple Distribution: Block Browser INC (Y4NZxxxxxx)" --force --deep --sign - dist/mac-universal/BLOCK.app
entitlements.mac.plist:
<!DOCTYPE plist PUBLIC "-//Apple//DTD PLIST 1.0//EN" "http://www.apple.com/DTDs/PropertyList-1.0.dtd">
<plist version="1.0">
    <dict>
    <key>com.apple.security.app-sandbox</key>
	<true/>
	<key>com.apple.security.inherit</key>
	<true/>
</plist>

Then BLOCK.app quit unexpected.
Error file: error.log

3rd attempt is with entitlement file and comment sandbox key:
Command line: codesign --entitlements build/entitlements.mac.plist -s "Apple Distribution: Block Browser INC (Y4NZxxxxxx)" --force --deep --sign - dist/mac-universal/BLOCK.app
entitlements.mac.plist:

<!DOCTYPE plist PUBLIC "-//Apple//DTD PLIST 1.0//EN" "http://www.apple.com/DTDs/PropertyList-1.0.dtd">
<plist version="1.0">
    <dict>
    <!-- <key>com.apple.security.app-sandbox</key>
	<true/> -->
	<key>com.apple.security.inherit</key>
	<true/>
</plist>

Then BLOCK.app run normal.
Packages version:

devDependencies:

electron: 27.0.2+wvcus
electron-builder: 24.6.4

dependencies:

"@sentry/cli": "^2.21.2",
"@sentry/electron": "^4.14.0",
"@sentry/webpack-plugin": "^2.9.0",
"electron-updater": "^6.1.4",
"electron-store": "^8.1.0",
"electron-context-menu": "^3.6.1",
"v8-compile-cache": "^2.4.0"

Build settings:
      "appId": "xxxxxx.block.mac",
      "category": "public.app-category.utilities",
      "provisioningProfile": "./embedded.provisionprofile",
      "target": [
          {
              "target": "pkg",
              "arch": ["universal"]
          }, {
              "target": "dmg",
              "arch": ["universal"]
          }
      ],
      "icon": "build/icons/icon.icns",
      "bundleVersion": "11"
    },

Following #105, so I have after pack hook:
"afterPack": "./afterPackHook.js",

afterPackHook.js:
const fs = require('fs')

exports.default = async function(context) {
  try{
  	fs.unlinkSync(context.appOutDir + '/BLOCK.app/Contents/Frameworks/Electron Framework.framework/Versions/A/Resources/Electron Framework.sig');
  } catch(e){}
}

ECS does not support packaging for the App Store. It also relies on features not allowed by by the store guidelines, and would not pass the review.

Thank you for your reply @khwaaj, so is there any possible way to publish app to mac store?

No, unfortunately there is not. There are some old tickets where this has been extensively discussed before, which you can probably find if you want some more information.


ANONYMOUS user token policy Error

Hi, I have been involved with PLCs and automation for many years, however I am very new to Nodejs, Git and have very limited linux experience.
I have managed to get the opc logger working in the demo mode succesfuly sending data to influx, however when I try to connect to OPC UA server I am unsuccesful.
OS: Windows 10 Pro 64 bit
Node v2.0.0
OPCUA Server:  KepServerEX 6.8
I have set the certificate under trusted clients in KepserverEX (first i had to convert it to der).
Any help you could provide would be appreciated.
Annonymous user token polic error.txt
config_toml.txt

Hi here,
it means you're trying to connect without a username and password (anonymous), but that anonymous access is not allowed by Kepware. You can enable it under the kepware project properties (and not in the UA configuration manager): See https://www.kepware.com/getattachment/de80e240-765e-451a-afce-640d413891c3/opc-ua-configuration-manager-manual.pdf, page 5/6: project properties > OPC UA
Either

Working !
Thankyou :)


sapi: Add Tss2_Tcti_Finalize() to the API

Added an explicit Tss2_Tcti_Finalize() to the API to avoid issues with
languages trying to wrap up this API via a FFI. By exposing this method
only via a macro on an opaque structure it becomes impossible to call it
from an FFI. But by providing an explicit function it can now be called.
I left the existing tss2_tcti_finalize() macro because some code
(specifically the unit tests) link to the TCTI libraries only. In these
cases the new function would not be available. The function cannot be
provided by the TCTI library since many programs link all of them in
when they use the SAPI and that would cause a conflict. So both are
available to preserve compatibility. Refs #490.
Signed-off-by: Doug Goldstein<EMAIL_ADDRESS>

Hey @cardoe. Thanks for putting this together. I'm not sure where this should live though. Since it's not part of the SAPI spec I'm hesitant to put it in that library. But since there's no single TCTI library where we can put this things get a bit weird.
What would you think about creating a separate library specifically for enabling bindings for the TCTI layer? It would contain Tss2_Tcti_* functions like the finalize you've provided but one for each function. Each would effectively be a wrapper to invoke the function pointer in the underlying structure. Effectively we'd be creating a library that provides similar functionality to the tss2_tcti_* macros. It would be tiny and I'd expect most consumers would link statically against it.
Also where do your bindings live? What's the language (rust)? Would love to take a look so I can understand this stuff better. Never done any language bindings myself.

Yeah I like that idea. Maybe something like libtctiutil.a? What I've got so far isn't usable at all but I've gone ahead and pushed it publicly at https://github.com/starlab-io/tss-sapi Unfortunately unions are unsafe by their very nature and much of the public API I've run into thus far uses unions so I've got to rethink my approach. e.g. https://github.com/01org/tpm2-tss/blob/master/include/sapi/tss2_tpm2_types.h

Just posted to the related issue (#490). Gonna shut this down to keep the discussion in one place.


The sample is not working with Android Studio emulators.

As in title.
The error is that "Chrome is not installed on the device".
Chrome indeed doesn't come installed on the Android Studio emulators. And it is not simple to install it.
What kind of Android sample is this if it is not working with the default IDE and emulator for Android?

Closing this issue as we are archiving this sample. Please see https://github.com/microsoftgraph/msgraph-training-android for an Android sample.


fails to call superclass of wrapped method

module Wrapper
  def wrap(meth)
    define_method(meth) do |*args, **kwargs, &block|
      super(*args, **kwargs, &block)
      puts "side effect"
      self
    end
  end
end

class MyString < String
  extend Wrapper
  wrap :<<
end

MyString.new << "a"

This code prints 'side effect' in cruby but with mruby I get
test.rb:4:in <<: no superclass method 'wrap' (NoMethodError)
What's going on here? This is with latest mruby head.
Thanks :)

It is a bug. But it may take time to fix the issue. Stay tuned, please.

No problem. I think the problem got introduced in the past few months as I'm sure I've seen this working.


Add global stepTemplate config

Currently a global podTemplate can be specified in the default Tekton configuration (config-defaults configmap). Similar global config would be desirable for stepTemplate to apply to all taskruns. The use-case is to be able to for example constrain the amount of cpu and mem resources that taskrun containers can consume.

/kind feature
/cc @danielhelfand

The use-case is to be able to for example constrain the amount of cpu and mem resources that taskrun containers can consume.

Isn't this something you can accomplish with limit ranges ?

Limit range would apply to everything in the namespace, including the application pods. TaskRun pods are terminating pods so admin might allow higher limits for those pods since they stop after a usually short while, in contrast to application pods are non-terminating and keep running.

That makes sense. stepTemplate doesn't let you enforce any kind of maximum/minimum resource request, just a default.
Another option would be to use OPA's Gatekeeper to enforce a policy that TaskRuns run as part of a PipelineRun have a default/max resource request, or maybe even max total resource usage across the run.

One thing I am noticing here that seems related is there are no docs on stepTemplates as far as what properties are available that could've helped inform this a little better. Most examples just include specifying environment variables. Beyond just resource constraints, there could be other use cases that may make sense here, but it's hard to tell since not a lot of info is available.

/remove-lifecycle rotten
/remove-lifecycle stale
/reopen

/remove-lifecycle stale


Assets (scripts) double loading

Describe the bug

When we use code splitting via dynamic import like this:
const routes = [
...
component: () => import('Projects.vue')
...
And we go to the page "Projects" app loaded component twice with different referers in production build.

Reproduction

https://github.com/miirinjej/vite-assets-double-loading
System Info
Output of npx envinfo --system --npmPackages vite,@vitejs/plugin-vue --binaries --browsers:
  System:
    OS: Linux 5.10 Arch Linux
    CPU: (4) x64 AMD Ryzen 3 2200G with Radeon Vega Graphics
    Memory: 4.15 GB / 13.65 GB
    Container: Yes
    Shell: 5.8 - /bin/zsh
  Binaries:
    Node: 15.11.0 - /usr/bin/node
    npm: 7.6.1 - /usr/bin/npm
  Browsers:
    Firefox: 85.0
  npmPackages:
    @vitejs/plugin-vue: ^1.1.5 => 1.1.5
    vite: ^2.1.2 => 2.1.2

Used package manager:
npm
Logs

  vite:config bundled config file loaded in 109ms +0ms
  vite:config using resolved config: {
  vite:config   build: {
  vite:config     target: [ 'es2019', 'edge18', 'firefox60', 'chrome61', 'safari11' ],
  vite:config     polyfillDynamicImport: true,
  vite:config     outDir: 'dist',
  vite:config     assetsDir: 'assets',
  vite:config     assetsInlineLimit: 4096,
  vite:config     cssCodeSplit: true,
  vite:config     sourcemap: false,
  vite:config     rollupOptions: {},
  vite:config     commonjsOptions: { include: [Array], extensions: [Array] },
  vite:config     minify: false,
  vite:config     terserOptions: {},
  vite:config     cleanCssOptions: {},
  vite:config     write: true,
  vite:config     emptyOutDir: null,
  vite:config     manifest: false,
  vite:config     lib: false,
  vite:config     ssr: false,
  vite:config     ssrManifest: false,
  vite:config     brotliSize: true,
  vite:config     chunkSizeWarningLimit: 500
  vite:config   },
  vite:config   plugins: [
  vite:config     'alias',
  vite:config     'vite:dynamic-import-polyfill',
  vite:config     'vite:resolve',
  vite:config     'vite:html',
  vite:config     'vite:css',
  vite:config     'vite:esbuild',
  vite:config     'vite:json',
  vite:config     'vite:wasm',
  vite:config     'vite:worker',
  vite:config     'vite:asset',
  vite:config     'vite:vue',
  vite:config     'vite:define',
  vite:config     'vite:css-post',
  vite:config     'vite:build-html',
  vite:config     'commonjs',
  vite:config     'vite:data-uri',
  vite:config     'rollup-plugin-dynamic-import-variables',
  vite:config     'vite:import-analysis',
  vite:config     'vite:esbuild-transpile',
  vite:config     'vite:reporter'
  vite:config   ],
  vite:config   define: { __VUE_OPTIONS_API__: true, __VUE_PROD_DEVTOOLS__: false },
  vite:config   ssr: { external: [ 'vue', '@vue/server-renderer' ] },
  vite:config   configFile: '/home/miirinjej/dev/projects/vite-assets-double-loading/vite.config.js',
  vite:config   inlineConfig: {
  vite:config     root: undefined,
  vite:config     base: undefined,
  vite:config     mode: undefined,
  vite:config     configFile: undefined,
  vite:config     logLevel: undefined,
  vite:config     clearScreen: undefined,
  vite:config     build: {}
  vite:config   },
  vite:config   root: '/home/miirinjej/dev/projects/vite-assets-double-loading',
  vite:config   base: '/',
  vite:config   resolve: { dedupe: undefined, alias: [ [Object] ] },
  vite:config   publicDir: '/home/miirinjej/dev/projects/vite-assets-double-loading/public',
  vite:config   command: 'build',
  vite:config   mode: 'production',
  vite:config   isProduction: true,
  vite:config   optimizeCacheDir: '/home/miirinjej/dev/projects/vite-assets-double-loading/node_modules/.vite',
  vite:config   server: {},
  vite:config   env: { BASE_URL: '/', MODE: 'production', DEV: false, PROD: true },
  vite:config   assetsInclude: [Function: assetsInclude],
  vite:config   logger: {
  vite:config     hasWarned: false,
  vite:config     info: [Function: info],
  vite:config     warn: [Function: warn],
  vite:config     error: [Function: error],
  vite:config     clearScreen: [Function: clearScreen]
  vite:config   },
  vite:config   createResolver: [Function: createResolver]
  vite:config } +4ms

Before submitting the issue, please make sure you do the following

[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Provide a description in this issue that describes the bug.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to https://github.com/vuejs/vue-next instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.

In this issue https://github.com/vitejs/vite/issues/2672 I had a workaround with not using lazy components
Does this also apply to your issue?
Then I can add the label has workaround :slightly_smiling_face:

In this issue #2672 I had a workaround with not using lazy components
Does this also apply to your issue?
Then I can add the label has workaround slightly_smiling_face

Lazy loading works, but the component is loaded twice. Maybe this is how it should be, I don't know. In development mode, the component is loaded once.
Yes, we don't need to use lazy loading as a workaround. Mark it, please.

Can confirm this still happens in Vite 2.8.6, plugin-vue 2.2.4

This only happens when splitVendorChunkPlugin is enabled (which is enabled default with Vite <2.9.0).
I think this part is affecting.

With splitVendorChunkPlugincomponent: () => __vitePreload(() => import("./Projects.0d283181.js"), true ? ["assets/Projects.0d283181.js","assets/vendor.7ae37152.js"] : void 0)

Without splitVendorChunkPlugincomponent: () => __vitePreload(() => import("./Projects.547ad753.js"), true ? [] : void 0)

I feel it is inconsistent not to have ["./Projects.547ad753.js"] in __vitePreload when it is without splitVendorChunkPlugin.

So it is a duplicate of #5532.
Closing this one because #5532 has more information though this is created earlier.