linux-sgx: Improve the remote attestation

The current implementation of remote attestation does not take into account the integrity of the wasm module.
The SHA256 of the wasm module has been put into user_data to generate the quote, and more parameters are exposed for further verification.

@JamesMenetrey Does it meet your needs? Please help me test it ;)

Waouh many thanks @zeuson0, was fast! I'm going to review and test it. 😃

Hey, I'm going to review the PR on Thursday if this is ok for all of you.

Hey, I'm going to review the PR on Thursday if this is ok for all of you.

@JamesMenetrey No problem, it is good for me.


Update README.md

Hi Attumm! I have fixed some language errors but have left the content relatively untouched.

I'm very impressed, and a bit embarrassed.
Thanks allot roangsong :+1:


Create User Dialogs if errors occur

Maybe just log everything to a file and if there is an error, just show the file location in a popup.


PowerShell Core does not have -Encoding Byte

Prerequisites

[X] Verify that this is not a Windows issue;
[X] Refer to the system requirements;
[X] Refer to the How to use;
[X] I do not use a homebrew Windows image;
[X] I did not tweak Windows before that could cause system instability;
[ ] If your issue concerns the Wrapper, please mention @BenchTweakGaming in issue

Steps to reproduce
Sophia -Functions UnpinAllStartApps
Windows Version
Windows 11 Pro 22H2 22621.675
Sophia Script version
6.2.3

"-Encoding Byte" is replaced by "-AsByteStream"

Thanks a lot!


MapAssert's containsEntry does not consider custom comparators

Description
When using MapAssert I want to define custom comparators that will be used for checks concerning entries. In my case (see below) (Java's) AtomicInteger does not have an equals method which lets the test fail, although I expected it to be successful.
When I dived into the assertion code I saw that there is a java.util.Objects#deepEquals comparison (see org.assertj.core.internal.Maps#containsEntry) instead of considering custom registered comparators.
Test case reproducing the bug
@Test
void comparator_test() {
	var map = Map.of("str1", new AtomicInteger(1));

	assertThat(map)
			.usingComparatorForType(Comparator.comparing(AtomicInteger::get), AtomicInteger.class)
			.containsEntry("str1", new AtomicInteger(1));
}

Environment

assertj core version: 3.22.0
java version: 11
test framework version: JUnit 5.8.2

The reason is that usingComparatorForType is only used for assertions comparing fields by fields.
To be discussed with the team but we could add support for usingValueComparator to register how to compare map values.

We might consider something like #2724, if possible at all.


Do you have python binding for the NVIDIA Jetson Nano?

Are you going to release a python binding which can run on arm64(aarch), for example, the NVIDIA Jetson Nano?
PS: Your latest python module works very well on amd64, it's amazing.

We currently don't have a plan to make pypi poackages for arm64.  But, you can follow the instruction (https://google.github.io/mediapipe/getting_started/building_examples#python) to build the python package on Jetson.

I've made it run on Jetson nano after few days of trial and error, take a look on the README:
https://github.com/yockgen/mediapipe_jetson_nano

I've made it run on Jetson nano after few days of trial and error, take a look on the README:
https://github.com/yockgen/mediapipe_jetson_nano

JFYI, in https://google.github.io/mediapipe/getting_started/python.html#building-mediapipe-python-package,  there is a section about building mediapipe python on aarch64 linux, which links to https://github.com/jiuqiant/mediapipe-python-aarch64 that provide the prebuilt python wheel files and detailed instructions.


No Longer Displays Images in Viewer

Describe the bug
When you click on the icon to allow the view to show on screen, the image that shows does not show up at all on any sites from my testing. This just started today and happens on Pixiv, Twitter(X), and other supported sites.
Browser And Version
Firefox: 133.0.3
Viewer version: 4.10.1
Log
12/22/2024, 10:59:53 PM EHVP:Idle Loader start at:80 Comic Looms.user.js:1403:15
Layout was forced before the page was fully loaded. If stylesheets are not yet loaded this may cause a flash of unstyled content. 125364113
Cookie warnings 5
Loading failed for the

Cannot reproduce this issue.
Try change the Read Mode to other.
Or reset style in CONF > Style.
Or reinstall this script from here: https://github.com/MapoMagpie/eh-view-enhance/raw/master/eh-view-enhance.user.js

Changing Reading Mode to a different one then back worked. Thank you!


Support numpy array tables in sklearn estimators

Besides table files and pandas' dataframes we should support numpy array as it is the standard of scikit-learn

Partially addressed in issue #156 in MR !86 because now sklearn estimators return numpy arrays.

Done in #103


Problem compiling on Mac OS Sierra

I am using Mac OS Sierra 10.12.6
I followed the tutorial building on Mac OS
But I encountered
fatal error: 'openssl/bio.h' file not found  when compiling xrdp.
I solved this problem by setting environment variable
CPPFLAGS=-I/usr/local/opt/openssl/include
LDFLAGS=-L/usr/local/opt/openssl/lib
and run ./configure rather than ./configure PKG_CONFIG_PATH=/usr/local/opt/openssl/lib/pkgconfig .
Maybe the tutorial should be updated?
This is my brew info of openssl
penssl: stable 1.0.2m (bottled) [keg-only]
SSL/TLS cryptography library
https://openssl.org/
/usr/local/Cellar/openssl/1.0.2l (1,709 files, 12.2MB)
  Poured from bottle on 2017-08-14 at 11:47:48
From: https://mirrors.tuna.tsinghua.edu.cn/git/homebrew/homebrew-core.git/Formula/openssl.rb
==> Dependencies
Build: makedepend ✘
==> Options
--without-test
	Skip build-time tests (not recommended)
==> Caveats
A CA file has been bootstrapped using certificates from the SystemRoots
keychain. To add additional certificates (e.g. the certificates added in
the System keychain), place .pem files in
  /usr/local/etc/openssl/certs

and run
  /usr/local/opt/openssl/bin/c_rehash

This formula is keg-only, which means it was not symlinked into /usr/local,
because Apple has deprecated use of OpenSSL in favor of its own TLS and crypto libraries.

No feedback from the reporter, macOS is out of scope, closing.

@cosmo0920 just for the sake of clarity, which release version were you using when you got it to compile?

@cosmo0920 I'm interested in your answer as well.


Rewiring the SS/IKS addition UI and logic for the addition of Signers

Task Description
When adding SS/IKS now since we are migrating the recreation flow, while adding the SS/IKS we will need two options

Configure a new one
Recover an old one

Mechs are addressed by https://github.com/bithyve/bitcoin-keeper/pull/3556

https://xd.adobe.com/view/f1543c84-c25a-4ad6-81ca-1e0c540e956e-a407/screen/0844dd1e-c2cf-4363-b669-4a43fcc72688/
@Deveshshankar content needs to be updated accordingly

verified this issue on keeper app v 1.1.12(300)


[ci] ci is consistently failing

Ci has been consistently failing with the error: java.io.IOException: Remote call on JNLP4-connect connection from ec2-34-210-115-84.us-west-2.compute.amazonaws.com/<IP_ADDRESS>:46651 failed
@tqchen

I have restarted the worker, we will see if that helps resolving the problem


Remove unused trace operator implementation.

Summary
This commit removes unused operator class about tracing facilities.
Background, Problem or Goal of the patch
In #27, we removed the tracing facilities but DefaultTraceOperator.java is left.
It has been already re-implemented in asakusafw project.
Design of the fix, or a new feature
N/A.
Related Issue, Pull Request or Code

#27

Wanted reviewer
N/A.

BUILD SUCCESS. <jenkins-prb-asakusafw-dag-31>


List of companies using gpm

The rest of the 13Floor crew and me are building a beautiful site for gpm, it'll have documentation, lists of plugins and much more, it'd be fantastic to have a list of companies who are using it.
Do you use gpm and/or know of a company or project that does? Please add them to this issue! If there's enough people interested we should be able to get their logos into the site. ^_^

drinkin.com does!

gyrosco.pe does.


Optimize w2dynamics

@ladinesa here is the merge request. At the end I managed to solve the issue (some dumb problem with the path to the Wannier90 files that I wanted to parse).

I moved all heavy code-specific data to be parsed as a HDF5Reference.
I moved out the call to the Wannier90Parser in W2Dynamics to avoid potential issues and to make it more maintainable.
I added docstrings in w2dynamics, what do you think? I can also add more in Wannier90 and later keep adding as long as I touch parsers.

I checked that with these and your fix in QE, the parsing of the data that the user shared is working.
Let me know what you think.

all looks good to me, please do not forget to run pylint, pycodestyle and pytest locally.


No way to get to PDF content attached to a message on Slack

I'm using the following code to send back a PDF file as an attachment to a message. On Slack I see the name of the file, but there is no way to download it. Note that on Skype I do see an option to download the file.
Code

Message on Slack

Message on Skype

Any ideas or workarounds?

Try setting the TitleLink property to the same link as ContentUrl.


Fix for Graph edge hover labels

Fixes bug where it would not find hover info because edge start/end points in datasource is always start/end while Graph.kdims can be any names.

Looks good. Merging.


Refactor app structure

Closes #41
Overview
These changes move Nebula Web from an SPA architecture to one that can use SSR. Additionally, the project is restructured once more into modules:

modules/ contains most code for non-UI functionality, like authentication and data transformations.
components/ contains code for Nebula Web-specific components.
routes/ corresponds to routes that are rendered by the browser.

Routes are now as follows:

/ - Landing page
/app - Interactive app home
/app/auth - Authentication catch-all route
/app/resources - Academic and support resources for the user
/app/plans - a list of all the user's plans
/app/plans/[planId] - UI specific to planning mode
/app/onboarding - New user flow
/app/profile - The profile for the current user
/terms - Service-wide terms of service
/privacy - Service-wide privacy policy

Final authentication is still a work in progress.

📝Changed routes:

/app/auth/[task]
/app

(view full size)
(view full size)

/app/onboarding
/app/plans/[...planId]

(view full size)
(view full size)

/app/plans
/app/profile

(view full size)
(view full size)

And 5 other routes:

/app/resources
/
/privacy
/support
/terms

3 deleted routes:

/Plans
/landing/LandingPage
/landing/LandingToolbar

Commit bbfc8a54846e9e68773be7e4843dfeb187314dd8 (https://nebula-web-bv4miab59-acmutd.vercel.app).

I'm just going to merge this into develop so others can start working on top of this.


Request: Kill the chrome process on crconsole's exit

Upon exiting the crconsole, the chrome process is left running. For example:
$ crconsole -s
Waiting for chrome to start:
chrome://newtab/
> SW registered
newtab> .exit
^C

Does not close the browser.

@1db8k it's a bit hard to figure out which process to kill - chrome forks after initial start and there are multiple processes to track


Right to Left checkbox in translation tester UI

Hello Tommi,
Would you consider using richtext instead of text and add a button to open files and extract their content directly in the source text.
What about translating tables and charts? and make the target text exactly seems as the original regarding the formatting

Why do not you improve the engine by learning on the spot as an interactive learning. When I translated a sentence in trados or memoq, immediately the tool finetunes the model with the new translation. For example, instead of file tuning the entire file (1000 segments), the tool should learn interactively with every sentence, so we do not need to make fine tuning at the end but interactively.

Re: the Rich text and other improvements in the Translation window:
I don't have the resources to develop the translation UI in the engine currently, since my focus is on the CAT tool integration currently (I have a bit of funding for that purpose). I might have funding for developing standalone features for the translation UI later, but that's uncertain.
Re: interactive learning
That something on my to-do list, but the list is very long :).


Supporting Apple "mps" device.

Hello,
When I set device="mps" on Apple M1, I still see the error:
TypeError: Cannot convert a MPS Tensor to float64 dtype as the MPS framework doesn't support float64. Please use float32 instead.
I see several have raised this issue, but I could not find a clear answer. Can someone help me? Does this feature requires specific version of some of the libs?
Thanks

I also use MacBookPro M1. However "mps" as device does not work for any pytorch application. Instead you can use
DEVICE = torch.device('cuda:0' if torch.cuda.is_available() else 'cpu')
Thanks

I have successfully used mps as a device for a PyTorch on several occasions on my MacBook Pro M2, but SAM is actually giving me a segmentation fault when I attempt it. The cpu implementation works just fine, however I would like the performance gains I find in other applications.
I don't even get the TypeError, it just dumps out.

reduce ops with int64 support in MPS are supported in nightly builds of torch (pip3 install --pre torch --force-reinstall --index-url https://download.pytorch.org/whl/nightly/cpu)
Can confirm, the nightly build fixes it.

ok
Thank you.
Best Regards -
Dhiraj Patra
Software Architect & AI Engineer
*Phone / whatsapp / signal / telegram: +91<PHONE_NUMBER>  *
skype: dhirajpatra  Profile https://github.com/dhirajpatra
P Please consider the environment before printing this e-mail
On Tue, Aug 29, 2023 at 9:24 AM CuiGaofeng @.***> wrote:

reduce ops with int64 support in MPS are supported in nightly builds of
torch (pip3 install --pre torch --force-reinstall --index-url
https://download.pytorch.org/whl/nightly/cpu)
Can confirm, the nightly build fixes it.
—
Reply to this email directly, view it on GitHub
https://github.com/facebookresearch/segment-anything/issues/453#issuecomment-1696727468,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA5UI5IMXIJQ72RU6UVVKY3XXVRXHANCNFSM6AAAAAAZS46U7I
.
You are receiving this because you commented.Message ID:
@.***>


Setting up an "App" vs "Website" in GA.

Tracking seems to work just fine when a property is setup as a "website." However, when the property is setup as a "mobile app" in Google Analytics, tracking does not seem to function.

Too bad to see all Google analytics projects are defunct at the moment.


Kanji not being romanized and wrong romanization

I noticed a few times, a resilient kanji does not get romanized; for example in the last sentence of this song, but also this one at 02:25.
But sometimes it's due to other reasons:
In this song at 01:09 "つれてって" is romanized 'tsure te tsu te', because つ is tsu but っ indicates repetition so it isn't 'tsure te tsu te' but "tsutre te tte"
Sometimes it's the particles for example "は" which is written with the 'ha' hiragana but should be pronounced "wa" and this isn't something a hiragana-latin converter can guess, same for o e and other particles.
Sometimes there's the issue of kanjis having multiple meanings therefore multiple ways of being pronounced and this means they can't be romanized without understanding which word the kanji corresponds to.
I know how difficult it is to romanize Japanese, and the extension does it pretty well 99% of the time, but to have a perfect romanization the only way would be to use a tool that knows Japanese, for example, Google Translate knows Japanese so when it displays romanization it's accurate (but sending the full lyrics to Google Translate just to get back the romanization would take much more longer than the time taken by this extension). If you visit this link and then do CTRL+F to search "tsuretette" you will see Google was able to understand it's not a hiragana used to add a new syllable but to change the following "te".
I know it would be incredibly hard to implement anything like that to a spotify extension, but I ought to let you know about this minor issue. Hope you have a nice day!

Thank you so much for being understandable regarding the difficulty involved with parsing the Japanese Language. I am definitely aware of some issues regarding the accurate romanization of certain words. I should probably add a comment about this in the README. That a small っ is being changed to tsu seems a bit concerning to me though...
Besides that there is, as you already mentioned, not much I can realistically do about the many Edge Cases in which romanization using a dictionary lookup fails.
Thank you for taking the time to leave this detailed feedback and I hope you have a very fine day yourself!

Thank you so much for being understandable regarding the difficulty involved with parsing the Japanese Language. I am definitely aware of some issues regarding the accurate romanization of certain words. I should probably add a comment about this in the README. That a small っ is being changed to tsu seems a bit concerning to me though... Besides that there is, as you already mentioned, not much I can realistically do about the many Edge Cases in which romanization using a dictionary lookup fails. Thank you for taking the time to leave this detailed feedback and I hope you have a very fine day yourself!

Sorry to barge on the issue here like that.but it also occured to me,but i thought it's a normal problem for a external translation.
But anyway you can fix it?

I noticed a few times, a resilient kanji does not get romanized; for example in the last sentence of this song, but also this one at 02:25.
But sometimes it's due to other reasons: In this song at 01:09 "つれてって" is romanized 'tsure te tsu te', because つ is tsu but っ indicates repetition so it isn't 'tsure te tsu te' but "tsutre te tte" Sometimes it's the particles for example "は" which is written with the 'ha' hiragana but should be pronounced "wa" and this isn't something a hiragana-latin converter can guess, same for o e and other particles.
Sometimes there's the issue of kanjis having multiple meanings therefore multiple ways of being pronounced and this means they can't be romanized without understanding which word the kanji corresponds to.
I know how difficult it is to romanize Japanese, and the extension does it pretty well 99% of the time, but to have a perfect romanization the only way would be to use a tool that knows Japanese, for example, Google Translate knows Japanese so when it displays romanization it's accurate (but sending the full lyrics to Google Translate just to get back the romanization would take much more longer than the time taken by this extension). If you visit this link and then do CTRL+F to search "tsuretette" you will see Google was able to understand it's not a hiragana used to add a new syllable but to change the following "te".
I know it would be incredibly hard to implement anything like that to a spotify extension, but I ought to let you know about this minor issue. Hope you have a nice day!

Very serious issue indeed.
Some songs were like fully mistranslated but af i understood, it was no problem.but u guys know anyway to manually correct it?


Remote storage adapter for nodejs backend

I am currently trying to write a storage adapter for a nodejs backend.
I have setup a http server and I am trying to read the request body.
I am trying to read the body by using following body parser, however I am getting an error message
app.use(bodyParser.raw({type: 'application/x-protobuf'}));

UnsupportedMediaTypeError: unsupported content encoding "snappy"

All remote exporters are written in golang, I would like to use nodejs for several reasons. Is there any documentation on how to parse the incoming request or any examples of storage exporters written for a nodejs backend?

Hello,
Thanks for your interest in Prometheus.
In addition to protobuf, the requests are also compressed with Snappy. There seem to be snappy libraries for nodejs:
https://www.npmjs.com/package/snappy
It is better to ask questions like this on the prometheus-developers mailing list rather than in a GitHub issue. On the mailing list, more people are available to potentially respond to your question, and the whole community can benefit from the answers provided.

@roidelapluie Thank you for the answer. I had a look over that before posting the issue but I didn't really try to uncompress the request as it looked too cumbersome to start with.
I gave it now a try and it seems to work. I am working on an example right now. When I'm finished, if you want, I can post an example for nodejs, maybe other users find it useful as well since I didn't find any useful material while I researched.
If you don't think it is needed, please feel free to close the issue

Hello,
We are not able to keep examples up to date, but you are very welcome to host your example code in a public github repository. If your final product is public, we can also link it from our documentation.
Thanks.


Design QA - Responsive view (1024 to 1200)

This breakpoint is not implemented as design:

looks good


Create docs page on managed tables vs. external tables

The docs should explain the difference between managed and external tables and why Unity Catalog supports both.

@MrPowers - happy to help out here if that'd be helpful!


Add python password generator

Issue resolved #65

@rajdip-b once i will test it out this project in my local machine then i will approve it and get merged this PR.

issue close #65


Bookmarking a Rule isn't working

When trying to bookmark a rule, I found that the bookmarking feature wasn't working. When I click the button it switches to bookmarked then back to not bookmarked.

Figure: Bookmarking a Rules Page

Figure: Network Request when trying to Bookmark Rule
AB#59878

Hi @KahaMason,
Done - Bookmarks are fixed

Figure: Successful bookmark
-Brady


↔ [Converter] Add support for aten::Int.Tensor in Torch-TensorRT

aten::Int.Tensor

Function Schema: aten::Int.Tensor(Tensor a) -> int

Original PyTorch API: torch.item() [documentation]

This converter/evaluator extracts the integer value from single-element tensors.

Is this for FX or TS?

This is for the TorchScript path and relates to a model for which this is the only missing operator.

The related issue closed more than 1 yr ago.
https://github.com/pytorch/TensorRT/issues/513


Screenlock Widget

Hi,
This is a screenlock widget I've created for the Bangle 2. This is the first time I've used github, so hopefully I've got everything right.
All the best,
Carl Read.

Thanks. I'll look into those.


tripleee: Blacklist star-writers.com

tripleee requests the blacklist of the following website(s):

star-writers.com - MS search

star-writers.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

With only 1 tp and no extenuating circumstances that I can see, I don't think we need this yet.


Add "RHEL6-STIG" to tar filename.

The download was "0.9.3.tar.gz". Recommend a more descriptive filename like "RHEL6-STIG-0.9.3.tar.gz".

This is a limitation of Github currently. The tarball filename is determined by the version tag associated with a release.


[3.2.4 RC2+3] Using webcam crashes on macOS

Godot version:
3.2.4 RC2 and RC3
(Works fine in RC1 and previous versions)
OS/device including version:
MacBook pro-mid-2012, running Mojave 10.14.6 with intel iris graphics
Happens both with gles3 and gles2.
Issue description:
Using the camera server to access the built-in camera causes the game window to crash. Haven't yet been able to figure out how to attach a debugger on macOS, since I only borrowed this device to confirm that this bug exists on macOS too.
Steps to reproduce:
Create a simple project and try to render the camera to an environment.
Minimal reproduction project:
https://github.com/BastiaanOlij/godot3_test_projects/tree/master/TestCameraServer

Can you build from source to bisect the regression? The 3.2.4rc2 commit is 9918fd722e3e555cb174f9806cdb38b6e8b0c2b7 whereas the 3.2.4rc3 commit is 4f891b706027dc800f6949bec413f448defdd20d.

Not really. I don't have any build tools on the device and can't really install them either. I tried to get my hands on the github actions artifacts, but couldn't figure out how to download them. If you can tell me how I can get them I could borrow the laptop again and quickly test them.
Btw, both rc3 and rc2 are broken. RC1 is the last version that worked properly.

Some more additional information: I found this bug while debugging my windows webcam PR, where I managed to track things down to the VisualServer. So it's possible that this is a rendering bug and not platform-related.

Crashes on BigSur only if is .app is signed. Unsigned app does not give access to camera at all.
Running <name>.app/Contents/MacOS runs without an issue and also have access to camera through Terminal.app permissions.
Some logs:
Thread 0:: Dispatch queue: com.apple.main-thread
0   libsystem_kernel.dylib        	0x00000001986a3988 __psynch_mutexwait + 8
1   libsystem_pthread.dylib       	0x00000001986d70cc _pthread_mutex_firstfit_lock_wait + 92
2   libsystem_pthread.dylib       	0x00000001986d4a04 _pthread_mutex_firstfit_lock_slow + 216
3   com.apple.cmio.DAL.AppleH13CameraInterface	0x0000000113edb844 0x113ecc000 + 63556
4   com.apple.cmio.DAL.AppleH13CameraInterface	0x0000000113ed8898 0x113ecc000 + 51352
5   com.apple.CoreMediaIO         	0x00000001a473f15c CMIO::DAL::PlugIn::ObjectHasProperty(CMIO::DAL::Object const&, CMIOObjectPropertyAddress const&) const + 32
6   com.apple.CoreMediaIO         	0x00000001a476a994 CMIOObjectIsPropertySettable + 156
7   com.apple.AVFCapture          	0x00000001aeb91f4c -[AVCaptureDALDevice becomeDeviceMaster:] + 100
8   com.apple.AVFCapture          	0x00000001aec365bc -[AVCaptureDevice_Tundra lockForConfiguration:] + 248
9   com.apple.AVFCapture          	0x00000001aebe17cc MediaIODeviceConfigureToBestMatchPixelBufferAttributesAndMinimumFrameDuration + 2684
10  com.apple.AVFCapture          	0x00000001aeb95f50 -[AVCaptureDeviceInput_Tundra sessionWillUseOutputDecompressionOptions:forPort:] + 208
11  com.apple.AVFCapture          	0x00000001aeb6d3a4 -[AVCaptureSession_Tundra _buildSupportUnitsForVideoInputPort:error:] + 1540
12  com.apple.AVFCapture          	0x00000001aeb6c910 -[AVCaptureSession_Tundra _buildGraphUnitsForInputPort:error:] + 904
13  com.apple.AVFCapture          	0x00000001aeb6ad30 -[AVCaptureSession_Tundra _buildAndRunGraph] + 572
14  com.apple.AVFCapture          	0x00000001aeb6c1d4 -[AVCaptureSession_Tundra _setRunning:] + 196
15  com.apple.AVFCapture          	0x00000001aeb6bdd8 -[AVCaptureSession_Tundra startRunning] + 200
16  org.godotengine.macgame       	0x0000000104790258 -[MyCaptureSession initForFeed:andDevice:] + 540
17  org.godotengine.macgame       	0x0000000104790e14 CameraFeedOSX::activate_feed() + 88
18  org.godotengine.macgame       	0x00000001059bfb68 CameraFeed::set_active(bool) + 52

Signing with camera entitlements fixed this issue. So I guess it's mostly signing/permissions problem.

@Ansraer self-signing Godot.app with
<?xml version="1.0" encoding="UTF-8"?>
<!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.device.camera</key>
	<true/>
</dict>
</plist>

entitlements or just running Godot binary from Godot.app/Contents/MacOS should work as temporary workaround.

Crashes on BigSur only if is .app is signed.

Crashes only if it's signed with hardened runtime (--options=runtime). I guess in addition to using entitlements for editor we should add a built-in entitlements editor for export (currently there's only an option to select an external entitlements file).


better default auth

currently uses no_really_i_dont_want_any_auth

@cppforlife I'm a big fan of this staying for the Virtual Box option. I understand it's maybe not great for deploying to GCP but it's definitely what I'm usually wanting if I'm deploying locally or on a private network.

@cppforlife: PR for ops file to support that?

ops files are now present and the user must specify one of them:

https://github.com/concourse/concourse-deployment/blob/master/cluster/operations/no-auth.yml
https://github.com/concourse/concourse-deployment/blob/master/cluster/operations/basic-auth.yml
https://github.com/concourse/concourse-deployment/blob/master/cluster/operations/github-auth.yml


[WIP] zerver/messages.py: Add has:reactions filter to messages search

https://github.com/zulip/zulip/issues/14578
Testing Plan:
Run backend tests for zerver
Test speed on query; optimize by adding has_reaction to messages table
Adds support for has:reaction search filter
GIFs or Screenshots:

Thanks for working on this @spareelectronics!  It looks like a great start.

We should also document the new feature in templates/zerver/help/.

@spareelectronics it looks like you pushed a new version of this PR, but it's still failing tests.  Is this ready for another review?
(If you don't have time to finish this, let @thedeveloperr know and he can pick it up).

@spareelectronics it looks like you pushed a new version of this PR, but it's still failing tests. Is this ready for another review?
(If you don't have time to finish this, let @thedeveloperr know and he can pick it up).

I probably can't get to this in time, so @thedeveloperr  feel free to take over.
While, the implementation does work, the tests that do fail is due to the test I wrote having an unused variable.
They query that's being run is this:
SELECT message_id, flags
FROM zerver_usermessage JOIN zerver_message ON zerver_usermessage.message_id = zerver_message.id
WHERE user_profile_id = :param_1 AND message_id IN (SELECT message_id
FROM zerver_reaction)

which seems like it would have a unreasonable query plan due to the nesting.
The other has_ operations use a boolean value which are stored in the zerver_message table.
Since reactions are handled differently, and have their own table that means one possible optimal solution would require adding another boolean column in the messages table for reactions, then adding a migrations file, and logic when reactions are created to populate the boolean value in the messages table indicating whether the current message has a reaction.

Closing in favor of the more active current PR for this: https://github.com/zulip/zulip/pull/15160


Tabs: Wrapped Tab.Trigger used inside the same export as the Controlled Tabs.Root breaks focus

Bug report
Should be active like this

Tab 1 loses focus after click/navigating

https://codesandbox.io/p/sandbox/hungry-sara-4drlmw?file=%2FApp.jsx%3A1%2C1-55%2C1
Current Behavior
When used like this Tab1 will break focus and cause focus to move out of the Tabs.Root

Click or use arrow keys to navigate to Tab1.
Will no longer be able to navigate to other tabs, focus has moved outside.

const TabsFocusIssue = () => {
  const [value, setValue] = React.useState('tab2')

  // Breaks focus
  const Tab1 = () => {
    return <Tabs.Trigger className="TabsTrigger" value="tab1">1
  </Tabs.Trigger>
  }

  // Works fine.
  const tab3 = <Tabs.Trigger className="TabsTrigger" value="tab3">3</Tabs.Trigger>

  return (
  <Tabs.Root className="TabsRoot" value={value} onValueChange={(value)=>setValue(value)}>
    <Tabs.List className="TabsList" aria-label="Manage your account">
      {/* In a controlled tabs component, when clicking or navigating to this initially,
      it will move the focus out of the tabs and break keyboard navigation.
      Have to click it again in this case to get focus back in tabs.  */}
      <Tab1/>
      {/* If I use one from out side this function it works fine... */}
      <Tab2/>
      {/* If I use one declared like tab3=<...> works fine... */}
      {tab3}
      <Tabs.Trigger className="TabsTrigger" value="tab4">
        4
      </Tabs.Trigger>
    </Tabs.List>
)
};

Expected behavior
Expect focus to remain on the activated tab.
Reproducible example
Focus Issue Demo
Suggested solution
...
Additional context
In the code sample I document two other ways that work.
Your environment

Software
Name(s)
Version

Radix Package(s)
Tabs
1.0.4

React
n/a
17.0.1

Browser
Chrome
115.0.5790.170

Assistive tech

Node
n/a
16.20.0

npm/yarn
npm
8.19.4

Operating System
OSX
12.6.7

That's because you are not forwarding props or and ref.
You would need to update your example to something like:
const Tab1 = React.forwardRef((props, ref) => {
    return <Tabs.Trigger className="TabsTrigger" value="tab1" ref={ref} {...props}>1
  </Tabs.Trigger>
  });

Hey thanks for the reply!
Updating it to use forwardRef does not appear to fix it—code here
It does work for Tab2 that doesn't forward it's ref BUT is defined outside of the function. Seems like there's something breaking when it's defined within the other component—but that's probably a bad idea and would get re-created on each render—probably needs to be refactored out of the component.


Fix tab and hyperlink tooltips to wrap long text

PR Checklist

[x] Closes https://github.com/microsoft/terminal/issues/9869
[x] CLA signed.
[ ] Tests added/passed
[ ] Documentation updated.
[ ] Schema updated.
[ ] I've discussed this with core contributors already.

@msftbot merge this in 1 minute


Map variable allow modifing readonly fields in spread field

Description:
Consider the below sample
function init() {
    string fieldB = "Some string";
    var map1 = {readonly fieldB};
    map1.fieldB = "some other string"; // gives compilation error as expected

    var map2 = {...map1};
    map2.fieldB = "some other string"; // no compilation error
}

Please find the spec issue here https://github.com/ballerina-platform/ballerina-spec/issues/956


feat: Add NetInfoConfig.

Overview
Add the ability to configure constant in NetInfo. Will fix issue like #156.
Test Plan
This doesn't change core behavior of NetInfo. Adding a config file and make sure the old test pass is my current plan.

Thanks @matt-oakes for the review. Will work on those next week.

Hey @matt-oakes
I tried to implement your suggestions here hopefully I haven't missed anything🤞 but I'm not sure how this feature would work when using useNetInfo() hook?
Should we make it so we supply the newConfig to the hook like useNetInfo(configObj);?
Also, should the setNetInfoConfig() throw an error if newConfig only contains reachabilityUrl but no reachabilityTest?

Closing this in favour of #230. There is an RC version published there to test. Check out the PR for details.
@mustafawm @hoangpham95 Thanks for your help. The changes you made looked good! However, I've decided to work this configuration method in as part of a larger refactor and release. Please let me know if the changes on the other PR work for you.


FORMS-253 # Dont mutate the definition object

A new object is constructed
Removed Cyclonejs library
Added tests to verify that the original defintion has not been touched

https://github.com/blinkmobile/forms/pull/62

We probably need to bump the devDependencies and double-check the ESLint rules in order to get this to pass Travis CI.

From the other thread, the question of semver version came up...

is it a major or a minute bump?

If we updated the consumer (Forms) to this new version without making any other changes in Forms, would anything break? If so, then this is definitely a MAJOR update.
Even if our 1st-party consumer doesn't break, I wonder if any hypothetical consumer might break?
I think it's hard to characterise this as a simple bug-fix, so it isn't a PATCH update.
So at a minimum we bump the MINOR. Is that similar to your thinking?

no, nothing breaks. and technically things are still the same. The only change that is worth noting is that any of a forms _element objects that contain an array (eg a select elements options) would be shared across form instances, but the way we handle options means this shouldnt be a worry, and I remember a conversation where we concluded if anyone was doing that then they shouldnt be a problem

yeah MINOR at least, I dont think this change is worth a MAJOR

LGTM :+1:
Did you want me to give you npm publish permissions? Or did you want me to handle releasing this?

I have not done an npm publish before but i'm happy to learn how. setup me up!


How to pass only one Logger to write to a file/db?

Type (choose one):

Question

NLog version: (e.g. 4.2.3)
Platform: . .Net 4.5
If I have the following code in my controller:
namespace MyReport.Controllers
{
    public class ReportController : Controller
    {
        private static Logger customMsg = LogManager.GetCurrentClassLogger();
        private static Logger excepMsg = LogManager.GetCurrentClassLogger();
        public ActionResult Index()
        {
         try
         { --- statement --- }
         catch (Exception e)
            {
                customMsg.Error("Custom message");
                excepMsg.Error(e);
            }
            return View();
        }
}

What should my rules look like in order to write only customMsg to a file/db, or only excepMsg, but not both?

Is the full name (including namespace) of the class.
If you log ${logger} to your logs, you can see it :)

So is it only upto the controller name? MyReport.Controllers.ReportController in my case? How do I specifically call the customMsg logger from my NLog.config file?

You need then LogManager.GetLogger("name1")

Keep in mind your can't "pass"  loggers. You write to loggers, and with rules the messages get passed to targets.
Loggers has names, and the currentClassLogger is a logger with a name from the current class.

By name1 in LogManager.GetLogger("name1") do you mean the full name including namespace? I

I was thing of
        private static Logger customMsg = LogManager.GetLogger("custom");
        private static Logger excepMsg = LogManager.GetLogger("excep");

and in your config:
  <rules>
<!-- control here what get logged to the file target, use enabled attribute for example, or filters -->
    <logger name="custom" minlevel="Trace" writeTo="file" enabled="true" />
    <logger name="excep" minlevel="Trace" writeTo="file" enabled="true"/>
  </rules>

Do you mean, you want to have multiple files but still use the one logger.
On Mar 7, 2016 19:25, "Julian Verdurmen"<EMAIL_ADDRESS>wrote:

I was thing of
    private static Logger customMsg = LogManager.GetLogger("custom");
    private static Logger excepMsg = LogManager.GetLogger("excep");

and in your config:

—
Reply to this email directly or view it on GitHub
https://github.com/NLog/NLog/issues/1288#issuecomment-193256175.

Ah that could also be the case.

I need a custom message as well as the exception of catch statement written to a database table, but I can only send one at a time as the parameter. Therefore I'm thinking of using two loggers, one to send exception message and the other to send a string of custom message. Then, both loggers would write to the same table simultaneously in different columns (I will configure the config file accordingly in <target> and <rules> for that. @304NotModified I will try your method and let you know how it goes. @Page-Not-Found It's exactly the opposite case, one database table but multiple loggers.

@Page-Not-Found If I could, I would do it with only one logger, that doesn't matter. I'm using multiple loggers only because I can't seem to get the work done with one, i.e. passing a string of custom message as well as the exception (from catch statement) both together for logging into the database table.

@304NotModified Working perfectly as I wanted. One problem though, it is writing in two separate rows, I want to combine them into single one.
<target name="DbCustom" xsi:type="Database"
           commandText="insert into dbo.tracelog (custom_message)
            values
            (@custom_message);" >
      <parameter name="@custom_message"         layout="${message}" />
    </target>

<target name="DbExcep" xsi:type="Database"
           commandText="insert into dbo.tracelog (excep_message)
            values
            (@excep_message);" >
      <parameter name="@excep_message"         layout="${exception}" />
    </target>

<rules>
<logger name="excep" minlevel="Debug" writeTo="DbExcep" />
<logger name="custom" minlevel="Debug" writeTo="DbCustom" />
</rules>

Any suggestions?

Yes, every entry will be a log record.
Can you elaborate on your case? As I don't fully understand what we're trying to achieve.

My database table has custom_msg and excep_msg columns. I want to pass a custom string for custom_msg column, and the catch statement's exception for excep_msg column, both in a single row of the table for each error/exception that occurs. The above method is logging custom and excep messages into two different rows, rendering the other one NULL in each case. So, in the table, instead of
excep_msg     custom_msg

I get
excep_msg     NULL
NULL          custom_msg

Well that's a common thing :)
You can:

Use 1 logger and 1 write statement: logger.Error(e,"Custom message"); //e = exception. In the logger you use [${exception}](https://github.com/NLog/NLog/wiki/Exception-Layout-Renderer) and ${message}

You use logevent-properties. E.g.

LogEventInfo theEvent = new LogEventInfo(LogLevel.Debug, "", "Pass my custom value");
theEvent.Properties["MyValue"] = "My custom string";
log.Log(theEvent);

You can then use ${event-properties}..

and/or you can use the fluent interface: http://nlog-project.org/2015/01/20/have-you-seen-our-new-fluent-interface.html

I am 100% sure I had tried passing it that way logger.Error(e, "Custom message") and it had given me some weird error saying I can only pass one parameter. Maybe coz I'm using GetLogger('name') instead of GetCurrentClassLogger() that's why I'm not getting error. Anyways, my code is working perfectly, can't thank you enough. :100:  :+1:   :)

GetLogger and GetCurrentClassLogger retuning the same type.  The only difference is the name.
I'm glad that it all works now :)

I have no idea why I got the error then, and not now! Anyways, I'm glad too. :) Thanks mucho!


(PUP-1985) Fix correct scope for parameter default evaluation

This commit ensures that a correct scope is used when evaluating
parameter default expressions. It impacts the parameter declarations
of functions, EPP templates, classes, and resources.
A new ephemeral scope ParameterScope was added to keep track of
which parameters that exists and in which order they are evaluated
and accessed. This scope is only in effect during the evaluation
of the parameter defaults. Since the parameters are evaluated left
to right, and each parameter is added to the scope, a default
expression of a parameter can use the parameters declared to the
left.
Each parameter in turn is evaluated using a private MatchScope to
ensure that the evaluation of one parameter cannot interfere with
another. No LocalScope is pushed though, since it's forbidden
to assign new variables in this context.
The commit also contain three new test suites (for function, epp-
template, and class+resource parameters). They all test default
parameter evaluation much in the same way but for different targets.

This PR should have been broken up into several commits. There is a lot going on here with refactoring of names (using relative names instead of absolute), cleanups, added validation etc. which makes it a lot slower to review.

Pushed a new commit where I think all issues have been addressed. I removed things related to refactoring/cleanup. Not really related to this PR anyway.

Pushed a new rebased commit where new tests mentioned here were added. I also refactored the Function and EPP parameter default tests and merged into a 'parameter_defaults_spec'. My hope is that their purpose are now clearer with more concise context info.

Just realized that my last push made the comments about ruby function doing bad things to the parameter scope outdated. The last commit ensures that the ParameterScope is read-only during all evaluation.

Can you explain. (I added a comment to the thread on this topic above). What makes it "read only", the fact that it is not seen by "evil" code i.e the ´$a=evil()` case, or something else?

A new boolean @read_only variable was introduced. Assignments are not permitted if it's been set.
The methods #evaluate and #evalutate3x (called for all parameter default evaluation) now do:
return as_read_only { evaluator.evaluate(expression, scope) }

and
return as_read_only { expression.safeevaluate(scope) }

The method as_read_only is defined on ParameterScope as:
    def as_read_only
      read_only = @read_only
      @read_only = true
      begin
        yield
      ensure
        @read_only = read_only
      end
    end

All good now. Waiting on Travis to go green before merging.

Ruby 2.0.0 timed out on Travis. All other succeed. Not related to my last commit.

Paint finally dried....


Adding a switch with nested routes

We have developed an extended switch component that interprets nested routes.
Ex:
<BrowserRouter>
     <InceptionSwitch>
          <DashboardTemplate exact path="/dashboard">
               <Route exact path="/profile" component={Dashboard} />
               <Route exact path="/settings" component={Settings} />
               <Route component={NoMatch}/>
          </DashboardTemplate>
          <Route exact path="/" component={Homepage} />
     </InceptionSwitch>
</BrowserRouter>
So with this code our application response to:
Route: /
Route: / dashboard / profile
Route: / dashboard / settings

@busterkikas Thanks for the PR. As you have obviously discovered, it is quite easy to write your own version of <Switch> since it is basically just a for loop.  However, I think that this would probably be best released as its own package. We want to keep the React Router API as simple as possible, which means sticking to a small set of core components instead of including lots of specialty components.


Solving the problem of popUntil

Solve popUntil shutdown page, no callback dispose method and GlobalKey state will not be destroyed.

I signed it!


fixing #10541

Hi,
Fixing #10541, but maybe isn't the right fix, depend if the error was expected...

That's not the right fix indeed, when we trigger an error message it's because this condition is a bug. So !tex should never be true.
The fix would be to prevent having a PanoramaSky without a texture.

Closing as this is not the right fix. See #10541 for further discussion about what should be done.


Allow basic Loading View customization. Change loading label text and…

… color, change progress bar color.

I am planning to release a new paging with recycler view. I am not accepting any more pull request in this project, sorry!


Clean up storage APIs

In implementing the new file storage APIs we noticed there is a lot of room for improvement in the other APIs.  E.g. in blob.go GetBlobRange() we use an "extraHeaders" parameter to specify well-defined options (which then requires that the consumer goes to the docs and reads through the request headers section; every consumer has to do this over again).  We should use well-defined params (e.g. struct) with defined types to make the APIs easier to use.

For further conversation on this topic, please see the issue linked above.


Rerun recipe using dummy changes

Trying to get #3 unstuck by resubmitting with a dummy commit (I think that having a way to resubmit without changes is underway, but not functional yet).

did you have luck with this @jbusecke ? I dont see a recipe run here corresponding to this commit -- they all seem older.

No not really. @andersy005 did you have any luck in implementing a way to retrigger runs without needing a commit?


Integration issues for News team

As originally reported by @sarafs in #11.

features/02_validation.feature View
Scenario: Submit to W3C validation Service
See http://validator.w3.org
When I submit the page to the W3C Markup Validation Service
Then there should be no errors
Note: Any site invariably always fails for the W3C validator

Heading hierarchy tests are failing -
Then(/^each heading must be followed by content or a heading of one level deeper (h2-h6)$/) do
puts "Heading hierarchy:"
puts page.heading_hierarchy.to_s
page.must_have_correct_heading_hierarchy

Note: from this spec heading_hierarchy_spec.rb the HeadingHierarchy module returns an invalid Xpath error.

def must_have_title_that_contains_h1_text
expect(browser.title).to include(browser.find('h1').text)
Note: Fails due to a miss-match in Page Title and h1 text

Scenario: View the page with JavaScript and CSS disabled
When I view the page with JavaScript and CSS disabled
Then all core content is available and functional
Note: Manual test

@sarafs are any of these issues still relevant after today's session?

@mattwynne these are the failures I see -
URL: http://www.bbc.co.uk/mundo & http://m.live.bbc.co.uk/news

Scenario: Check headings ✗

Unable to find xpath "/html/head/meta/meta/meta/meta/meta/meta/link/meta/link/link/link/link/link/meta/meta/meta/meta/meta/meta/meta/meta/link/link/link/link/body/div/div[3]/div/div/section/div/div/h2"
/Users/sarafs02/.gem/ruby/2.0.0/gems/capybara-2.4.4/lib/capybara/node/finders.rb:41:in block in find' /Users/sarafs02/.gem/ruby/2.0.0/gems/capybara-2.4.4/lib/capybara/node/base.rb:84:in synchronize'
/Users/sarafs02/.gem/ruby/2.0.0/gems/capybara-2.4.4/lib/capybara/node/finders.rb:30:in find' /Users/sarafs02/.gem/ruby/2.0.0/gems/capybara-2.4.4/lib/capybara/session.rb:676:in block (2 levels) in class:Session'
/Library/Ruby/Gems/2.0.0/gems/bbc-a11y-0.0.4/lib/bbc/a11y/cucumber_support/heading_hierarchy.rb:53:in block in heading_elements' /Library/Ruby/Gems/2.0.0/gems/bbc-a11y-0.0.4/lib/bbc/a11y/cucumber_support/heading_hierarchy.rb:52:in map'
/Library/Ruby/Gems/2.0.0/gems/bbc-a11y-0.0.4/lib/bbc/a11y/cucumber_support/heading_hierarchy.rb:52:in heading_elements' /Library/Ruby/Gems/2.0.0/gems/bbc-a11y-0.0.4/lib/bbc/a11y/cucumber_support/heading_hierarchy.rb:45:in headings'
/Library/Ruby/Gems/2.0.0/gems/bbc-a11y-0.0.4/lib/bbc/a11y/cucumber_support/heading_hierarchy.rb:30:in to_s' /Library/Ruby/Gems/2.0.0/gems/bbc-a11y-0.0.4/features/step_definitions/page_steps.rb:32:in /^each heading must be followed by content or a heading of one level deeper (h2-h6)$/'

URL: http://m.live.bbc.co.uk/news ( I have used 'm.live' instead of  'm.' as it forces the mobile site to load for the tests. 'm.' redirects to 'www' on desktop browser widths)
This looks more like a genuine accessibility issue on the page. @IanPouncey could you please confirm.
(On the page in 2nd column the Most - Read tab - 'most-popular__header__tabs--read open' has tabindex="0").

Correctly use tabindex attributes

Scenario: Check all tabindex values ✗

expected [#<Capybara::Element tag="a">].empty? to return true, got false
/Library/Ruby/Gems/2.0.0/gems/rspec-expectations-3.2.0/lib/rspec/expectations/fail_with.rb:29:in fail_with' /Library/Ruby/Gems/2.0.0/gems/rspec-expectations-3.2.0/lib/rspec/expectations/handler.rb:40:in handle_failure'
/Library/Ruby/Gems/2.0.0/gems/rspec-expectations-3.2.0/lib/rspec/expectations/handler.rb:50:in block in handle_matcher' /Library/Ruby/Gems/2.0.0/gems/rspec-expectations-3.2.0/lib/rspec/expectations/handler.rb:27:in with_matcher'
/Library/Ruby/Gems/2.0.0/gems/rspec-expectations-3.2.0/lib/rspec/expectations/handler.rb:48:in handle_matcher' /Library/Ruby/Gems/2.0.0/gems/rspec-expectations-3.2.0/lib/rspec/expectations/expectation_target.rb:54:in to'
/Library/Ruby/Gems/2.0.0/gems/bbc-a11y-0.0.4/lib/bbc/a11y/cucumber_support/page.rb:62:in must_not_have_any_positive_tabindex_values' /Library/Ruby/Gems/2.0.0/gems/bbc-a11y-0.0.4/features/step_definitions/page_steps.rb:45:in /^there should be no elements with a tabindex attribte of 0 or greater$/'

Closed in favour of #20 and #21


add SwissTable from abseil

Summary: Added absl::flat_hash_set and absl::node_hash_set to the
benchmark driver.  This diff also includes benchmark results from a
2-socket E5-2680 v4.

Thanks for doing this!  I was planning on adding this as soon as I got to work but you beat me to it :)


[PE-D][Tester A] Edited person details does not reflect in meeting list

When a person is edited, the edited details (for example, name) does not get updated in the meeting list.
Steps to reproduce error:

meetingAdd 3 md/Meeting with Charlotte ms/30-03-2020 12:30 me/30-03-2020 13:30

meetingAdd 3 md/Meeting with Charlotte ms/30-03-2020 13:31 me/30-03-2020 13:31

edit 3 n/Alex no Yeoh

Here are the screenshots from edit onwards

As seen, the 1. and 2. still has meetings with Alex's old name

Labels: type.FunctionalityBug severity.Medium
original: ARPspoofing/ped#15

fixed by pr #166


Throw error

I can't show errors on my signUp page please help me
const onSubmit: SubmitHandler = async data => {
const { email, password, name } = data;
try {
  const user = await signUp({
    email,
    password,
    name,
  });
  console.log(user);

  const res = await signIn('sanity-login', {
    redirect: false,
    email,
    password,
  });

  if (res?.error) {
    console.log(res);
    toast.error(res.error);
    return;
  }

  router.push('/');
} catch (error) {
  console.log(error);

  toast.error('Something went wrong. Please try again later.');
}

};

Hi @axelverglas! now in the latest version the lib throw errors, this should be fixed


Install additional plugins

I want to be able to install additional plugins without having to ssh in to every node.  For my specific use case I want the delete-by-query, kopf and azure-cloud plugins to always be installed.
An implementation is available in pull request #65

Closed via #65


Feature Request: allow regenerate_token

@curbsaleem Why not just set the token to nil and re-save the document?

Thnx!


WARNING  | TapSwap | [!] Error in auth: list index out of range

Hi. That's the kind of problem that appeared today, what could it be related to?

yeah same for me

++

Wait pls almost done

Ok

Fixed!

After the update, there is now such an error. At the same time, the window closes almost immediately.

@SannoV use python 3.10

js2py has issues on Python 3.12.

Thx u, working 👍

Everything is working now. Thank you very much!

Hi. This problem appeared again (Did the admins do something again?).


How to update safely?

Hi,
Sorry if this is the wrong place. I'm using this script with an old version and i think it's time i update it. But i'm not sure how i should go about it, i'm afraid of breaking something. Do i just remove the old files and replace them with the new? Or are there extra steps like rebooting the phone before to remove the daemon and reapply the one from the new files afterwards?
Thanks a lot for your help

If you have any farming set-ups in the old regular folder that you wish to save, copy them into a notepad. It's usually simpler to remove all the old files and then drop the new ones in. Rebooting and daemon depends on the android version as newer ones just need to enable permissions for ankulua.

I did it how you said and everything went really well, thank you very much!


Several UI fixes

Description links in Mechanisms of Actions section going to disease page instead of drug page
Fix formatting of action type in known drugs table using "sentence case"
Tweak column widths in Indications table in Drug page
In the Phenotype section of the disease page, don't show tooltips if there's no data (null)

Disease page:

Phenotypes: handle null/NA values with tooltips (phenotypes and aspect)
Synonyms: show new "relation" information in tooltip and remove duplicate synonyms; issue #1358
Nice to have: fix columns width in phenotypes (requires truncating long text with ellipsis)

Done here https://github.com/opentargets/platform-app/pull/293


fix(utils): remove unnecessary logic in get

Overview

ref #163
AS-IS
The current get function has travel logic with use two RegExp as below.
export const get = <T = any>(obj: Record<string, any>, path: string, defaultValue?: T): T => {
  // ...
  const result = travel(/[,[\]]+?/) || travel(/[,[\].]+?/);
  // ...
};

However, the first RegExp is sufficient to replace the second RegExp, so the first is unnecessary.
TO-BE
removed first RegExp as below.
export const get = <T = any>(obj: Record<string, any>, path: string, defaultValue?: T): T => {
  // ...
  const result = travel(/[,[\].]+?/);
  // ...
};

PR Checklist

[X] I read and included theses actions below

I have read the Contributing Guide
I have written documents and tests, if needed.

Last night I thought about this issue and actually found out why there are two regular expressions.
Consider the following object:
{
  "foo.bar": 2,
  "foo": {
    "bar": 1
  },
}

In the current implementation, get(obj, "foo.bar") would return 2, not 1. If we change the implementation, it would return 1.
I don't know whether this is an intended behavior...

Oh, I see.
Umm.. If that above case could be return 2 as you told me, I think it should be return 1 at the follow case.

But.. I don't know that is right case.. 🤔

Yes, that's right, and I think we might use three RegExps. However I think this is quite a minor case and not sure about whether or not we should support this behavior...

I think we don't have to support keys including separator like "a.b.c" because there are too many available separators(., [], ,). If there were only one separator we would support it.

Thanks, @leokang94 !


configurable namespaces

I'm looking to test out if kbatch meets my needs for launching some jobs from a JupyterHub instance, and one thing I want to do is apply resource quotas on a group basis. As I understand it, that means that I'd like to be able to route multiple users to the same namespace, and ideally not grant kbatch-proxy permission to create new namespaces. Does that make sense? I know how I'd hardcode an implementation in the proxy itself, but I'm not sure how it should look as configuration.

It's been a while since I looked (and FYI, I don't have a ton of time to work on k-batch these days, but maybe @yuvipanda can chime in).
As for the configuration, is your desired target namespace static, and doesn't depend on anything in the HTTP request?Maybe the kbatch-proxy deployment would get a user_namespace: <string | None> config value that would be None by default, which implies using the user's namespace. An admin could set that to a static value like jobs and it would us that if present?

Initially, it would be a static value because I've only got one group of users right now, but for the system I'm supporting, I think I'll eventually need at least two, and should be derived from the user.
It would be nice if I could drop-in my own function for namespace_for_username (and even nicer if it could be namespace_for_user with the whole User object returned from the auth).
Right now, there's an issue with collision possibilities in the current namespace_for_username derivation (depending on JupyterHub's source of usernames), which is another reason it would be nice to be able to override the namespace derivation.

I may have a bit of time to contribute to this over the next few months, if that would be helpful. I think I also need #34 (not from the user, but a constant set in the proxy). I will try to sync with @yuvipanda when he's back.

If a user-provided callable is not suitable for kube-proxy design goals, I think I can also make it work with dict config that is a mapping of group: namespace and/or username: namespace.

I’m fine either. I personally haven’t implemented anything like that before, but you and Yuvi are the experts here :)

On Sep 5, 2024, at 10:52 AM, Min RK @.***> wrote:
If a user-provided callable is not suitable for kube-proxy design goals, I think I can also make it work with dict config that is a mapping of group: namespace and/or username: namespace.
—
Reply to this email directly, view it on GitHub https://github.com/kbatch-dev/kbatch/issues/66#issuecomment-2332094171 or unsubscribe https://github.com/notifications/unsubscribe-auth/AAKAOIQYFTV33RA5SSUNV6LZVB43DBFKMF2HI4TJMJ2XIZLTSOBKK5TBNR2WLJDUOJ2WLJDOMFWWLO3UNBZGKYLEL5YGC4TUNFRWS4DBNZ2F6YLDORUXM2LUPGBKK5TBNR2WLJDUOJ2WLJDOMFWWLLTXMF2GG2C7MFRXI2LWNF2HTAVFOZQWY5LFUVUXG43VMWSG4YLNMWVXI2DSMVQWIX3UPFYGLLDTOVRGUZLDORPXI6LQMWWES43TOVSUG33NNVSW45FGORXXA2LDOOJIFJDUPFYGLKTSMVYG643JORXXE6NFOZQWY5LFVE2DCNZVGU3DOMZTQKSHI6LQMWSWS43TOVS2K5TBNR2WLKRSGUYDEMZQGEZTCNVHORZGSZ3HMVZKMY3SMVQXIZI.
You are receiving this email because you commented on the thread.
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Excited to see @minrk make additional changes here :)
@TomAugspurger can you invite me to the PyPI project too, so I can make releases?

I sent you PyPI invites sent for both kbatch and batch-proxy @yuvipanda.


Create bumail.txt

Bangkok University student email

Same as http://www.bu.ac.th/
but bu.ac.th is domain for teacher
and bumail.net is domain for student

@veeravat Pull request merged. Thank you!


Latest update causes the window to stop responding at random

VSCode says the window has stopped responding for little to no reason on startup and at  random

VSCode Version:

code --version
1.40.2
f359dd69833dd8800b54d458f6d37ab7c78df520
x64

OS Version:

Steps to Reproduce:
Unknown, My specs are a Core i3 3rd gen, Intel integrated graphics 4k series,
i even tried reinstalling and it didnt fix the issue, and no recent updates of any other component of my system have been upgraded. at all.

Does this issue occur when all extensions are disabled?: Yes,

Also noticed that when these issues occure, Memory usage drops from around 300-330mb, to <50mb

@Merith-TK Did this only start happening with the most recent VSCode update? Does this happen only for particular projects, or anything you have open?

It started with the most recent update yes. It occurs at random. i use a few extensions and develop golang
my workspace is a mounted Samba share. i can give you logs in a few hours once im out of school.

2019-12-11_19-30-28.zip
Logs from me opening today
the things that happened in order

The new update readme tab opened, nothing rendered, (BLANK TAB)
I opened the file explorer and navigated to open Z:\ which is my workspae
Window is not responding popup, clicked reload
Window reloaded. Window is not responding popup occured again, i clicked close

Can you try to reproduce with our nightly insider builds? You can give our preview releases a try from: https://code.visualstudio.com/insiders/

Will try tomarrow.
I am guessing that insider builds are more verbose with their logs?
On Wed, Dec 11, 2019, 9:58 PM Benjamin Pasero<EMAIL_ADDRESS>wrote:

Can you try to reproduce with our nightly insider builds? You can give our
preview releases a try from: https://code.visualstudio.com/insiders/
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/86633?email_source=notifications&email_token=ACPQOXVWSEVHBUP7VFTUACDQYHHJ5A5CNFSM4JYX5DW2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEGVR2RA#issuecomment-564862276,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACPQOXVQYUCTINXLDHXYAALQYHHJ5ANCNFSM4JYX5DWQ
.

2019-12-12_07-36-09.zip
Insiders doesnt crash, stops responding at first but doesn't crash (logs are insiders first run.)

@Merith-TK to clarify, "stops responding" means you cannot do anything in the UI? does eventually a dialog popup informing about that fact?

This time around, it is just the standard windows "application has stopped
responding" bar at the top for a a few seconds, and then resumes.
On Thu, Dec 12, 2019, 8:27 AM Benjamin Pasero<EMAIL_ADDRESS>wrote:

@Merith-TK https://github.com/Merith-TK to clarify, "stops responding"
means you cannot do anything in the UI? does eventually a dialog popup
informing about that fact?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/86633?email_source=notifications&email_token=ACPQOXT6ZNU7NE7FRMHAXTTQYJQ6JA5CNFSM4JYX5DW2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEGXG7UA#issuecomment-565080016,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACPQOXWQBWVFQPFFNO5BIL3QYJQ6JANCNFSM4JYX5DWQ
.

@Merith-TK does it reproduce with any folder you open or only specific?

It does it on startup,
Sorry for being unhelpful if i am, the weird thing is when im working over RDP it doesn't give issues,
SPECULATION:
which is leading me to believe it may be a graphics issue and electron, as i noticed RDP connections use a different graphics driver mode.
even though that makes no sense as chrome does not fail at all,

Can you see if it still reproduces when disabling the GPU support in VSCode? You can do that by running VSCode from the command line like this: code --disable-gpu
If that helps, you can permanently configure this option via:

Command Palette search for "Configure Runtime Arguments"
Set disable-hardware-acceleration: true
Restart

I'm seeing this also on a Mac with version 1.40.1, specifically anytime I try to search on anything. I have reverted to 1.39.2 and am not seeing any problems. It actually will crash my machine and I have had to do a hard reboot several times as the machine freezes up.
Environment

macOs Mojave - 10.14.6
Processor - 3.1Ghz i7
Memory - 16GB
Graphics - Radeon Pro 560

I also have two LED 27 monitors attached to my Mac running 2560x1440 resolution using USB-C to DisplayPort.
I need to get back to work, but will try reproducing with hardware-accelaration disabled later.

Please do.

Update for me, Hardcrashing (window not responding, windows saying that, not vscode) has stopped
But vscode's "not responding" still pops up occasionally, but i can just tell it to keep waiting
but the problem is that it feels like i am trying to program over vnc over cellular service, it is way too sluggish

@Merith-TK can you check what I asked in https://github.com/microsoft/vscode/issues/86633#issuecomment-565322500

Sure, testing now

Yeah that helps. alot,

Ok, closing given the workaround.


SASS?

Any chance for SASS support?

I have no plan at the moment.
Welcome new provider.
Please refer to the atom-source-preview-babel.

+1.

I just released source-preview-sass. Although I can't quite seem to get the source mapping to work… @aki77 Any ideas?

Never mind… I just released patch 0.1.1 which fixes source maps. This can be closed now @surfjedi 👍

Cool thanks - but whats to short cut? using source-preview from the palette doesn't do any thing on a .scss file?

but whats to short cut?

Not quite sure what you're asking there… I'm sure you've done apm install source-preview-sass?
I suspect you might be getting a 'Missing binding' error as I suddenly am after updating my homebrew installed version of Node to 6, which is causing a problem with node-sass (the only dependency of my package). It seems this will be fixed in their next release, but in the meantime (if I'm right, and you're on a mac) then you can download the correct darwin-x64-47 binding, rename the downloaded file from darwin-x64-47_binding.node to binding.node and place it inside a darwin-x64-47 folder. Then move this folder to $ATOM_HOME/packages/source-preview-sass/node_modules/node-sass/vendor/darwin-x64-47/binding.node which fixes the problem until the upstream is fixed, at which point I'll update the dependency and the Atom package.
Hope that helps.

@surfjedi It turned out that node-sass is currently not compatible with Electron (Atom), so the update didn't fix the problem. It might be in future… at which point I'll probably switch dependencies back to node-sass, but for now I implemented a workaround. So package should work smoothly now, although it takes a while to install.
@aki77 This can be closed.

@danielbayley Thanks!


Updating skipper image to correct GO version

Downgrading skipper image to go 1.22

/assign @ybettan

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 72.50%. Comparing base (fa23a9b) to head (ea197fc).
Report is 121 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #902      +/-   ##
==========================================
- Coverage   79.09%   72.50%   -6.60%
==========================================
  Files          51       65      +14
  Lines        5109     4462     -647
==========================================
- Hits         4041     3235     -806
- Misses        882     1043     +161
+ Partials      186      184       -2

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

/lgtm


Optimiaze npm packages

What this PR does / why we need it:
use the files array to specify only the files to be included
Which issue(s) this PR fixes:

Fixes #28
Special notes for your reviewer:
Using the following commands, you can verify that this is possible.
npm pack
tar -tf your-package-name-x.y.z.tgz

I initially mentioned that it's still in progress and intended to use the files array, but after reviewing the NPM documentation, I confirmed that CHANGELOG.md is not automatically excluded unless explicitly specified in .npmignore. If necessary, it should be excluded using .npmignore

https://docs.npmjs.com/cli/v7/using-npm/developers#keeping-files-out-of-your-package
Does this PR introduce a user-facing change?:

NONE

Additional documentation:

[npm doc for files](https://docs.npmjs.com/cli/v7/configuring-npm/package-json#files)

Checklist:

[x] Added relevant tests or not required
[x] Didn't break anything

Summary by CodeRabbit

New Features

Improved package configuration by including the dist/ directory in the package distribution, ensuring essential assets are bundled correctly.

Documentation

Enhanced clarity regarding the contents of the package by explicitly defining the files to be included during publication.

Thank you for the PR! The addition of the dist directory to the files array is a good improvement to ensure that the necessary build assets are included in the package distribution.
However, it’s important to note that NPM automatically includes certain files in the package distribution even if they are not specified in the files array. These files include package.json, README.md, LICENSE, and CHANGELOG.md. This means that these files will be included in the published package without needing to be explicitly added to the files array.
Given this, while your change is valid and improves clarity by explicitly defining the dist directory, there’s no need to worry about the README or other standard files being excluded. They are automatically included by NPM.
Again, thank you for your contribution! This makes our package distribution more intentional and ensures that only the essential files are bundled.


Pass thru CF-NetCDF flag attributes.

This PR allows the flag_masks, flag_meanings and flag_values CF-NetCDF attributes to be passed through from the file-format into the associated cube.attributes and/or coordinate.attributes dictionaries.
The contents of the attributes dictionaries are already sourced by the NetCDF saver, thus this metadata can now be maintained losslessly through Iris.

Certainly well tested! :wink:
:+1:

Yeah ... it got personal! I refused to back down and be mocked by mock, can you tell :smile:

@pelson Awesome, thanks!


Fix issue with defining empty closed records

Purpose

This is to fix the issue we had with not being able to define an empty closed record as record {||}. This also makes {| and |} delimiters single tokens (i.e., white space between the brace and the pipe is not allowed).

Fixes #14748
Approach

Defined closed record delimiters as tokens in the lexer.

Samples
type Foo record {||};

type Bar1 record { ||}; // compile error

type Bar2 record {|| }; // compile error

Check List

[x] Read the Contributing Guide
[ ] Required Balo version update
[ ] Updated Change Log
[x] Added necessary tests

[x] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

@pubudu91 Shall we fix the new grammar changes as well included in #15129?

@pubudu91 Since @NipunaMarcus looking in this, shall we mark this as WIP ?

@hasithaa +1 for making this WIP ...

@pubudu91 .. so when we define a sealed record for a specific type following is how we did earlier,
type Person record {
    string name;
    int id;
   string...;
}

Now with the new changes, this has to be defined as below?
type Person record {|
    string name;
    int id;
    string...;
|}

Is this correct?

Yes, that’s correct

@pubudu538 Can you check the build failure?

Yeah looking in to it.. It was in examples test. Ran it through Maven and nothing failed. Retriggered Travis and running a local Gradle build as well to see

Codecov Report

Merging #14872 into master will decrease coverage by 0.11%.
The diff coverage is 55.55%.

@@             Coverage Diff              @@
##             master   #14872      +/-   ##
============================================
- Coverage      23.3%   23.19%   -0.12%
+ Complexity     4970     4968       -2
============================================
  Files          1303     1302       -1
  Lines         65334    65618     +284
  Branches       8863     8869       +6
============================================
- Hits          15229    15221       -8
- Misses        48424    48718     +294
+ Partials       1681     1679       -2

Impacted Files
Coverage Δ
Complexity Δ

...rg/ballerinalang/langserver/formatting/Tokens.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

...erinalang/compiler/parser/BLangParserListener.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...erinalang/compiler/parser/BLangPackageBuilder.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...lang/langserver/formatting/FormattingNodeTree.java
72.67% <100%> (-0.01%)
665 <0> (-2)

...rg/wso2/ballerinalang/compiler/bir/BIREmitter.java
0% <0%> (ø)
0% <0%> (ø)
:arrow_down:

...alang/compiler/semantics/analyzer/TypeChecker.java
0% <0%> (ø)
0% <0%> (ø)
:arrow_down:

...ng/compiler/semantics/analyzer/SymbolResolver.java
0% <0%> (ø)
0% <0%> (ø)
:arrow_down:

...rinalang/compiler/semantics/model/SymbolTable.java
0% <0%> (ø)
0% <0%> (ø)
:arrow_down:

...2/ballerinalang/compiler/bir/model/BIRVisitor.java
0% <0%> (ø)
0% <0%> (ø)
:arrow_down:

...n/java/org/ballerinalang/model/types/TypeKind.java
0% <0%> (ø)
0% <0%> (ø)
:arrow_down:

... and 7 more

Continue to review full report at Codecov.

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


Changing the Implementation of IEventListener.Signal

I would like to WriteDebug HttpRequests and Responses when the -Debug switch is specified on command execution.
I was considering changing the implementation of IEventListener.Signal in each cmdlet, is this the right way to do it?
What extensibility points exist to hook into the pipeline and write out the desired information?

You will need to implement the EventListener In the generated/Modules.cs.


question about action after submit a form

Hi,
I am starting using ZeroStep in my project but I notice that when I use the sentence (only with Playwright):
await page.getByPlaceholder('Search patient').click();
it takes a moment until the page is loaded to after that click in the "Search patient"; but when I use ZeroStep, I put the sentence
await ai(Click on the "Search patient");
The script fails because immediately tries to click on the object before the page is loaded, and the error says:
Error: elementHandle.click: Element is not attached to the DOM
Do you know how can I fix it? Exist a way to apply the click action when the selector is becoming visible/stable/editable?
Here I share part of the code that I am using for only Playwright
    await page.goto('https://demo-company.com/');
    await page.getByPlaceholder('Enter your email').fill('testuser@company.com');
    await page.getByPlaceholder('Enter your Password').fill('admin123');
    await page.getByRole('button', { name: 'Sign in' }).click();
    await page.getByPlaceholder('Search patient').click();
    await page.getByPlaceholder('Search patient').fill('patient_test_Name');

And here, how I am using Playwright with ZeroStep
  await page.goto('https://demo-company.com/');
  await ai(`Enter the email ${email}`);
  await ai(`Enter the password ${password}`);
  await ai(`Click the Submit button`);
  await ai(`Click on the "Search patient"`);
  await ai(`Enter the Search patient as patient_test_Name`);

Maybe stating/using a wait for visible should fix your issue:
// ....
await ai(`Wait for "Search patient" to be visible`)
await ai(`Click on the "Search patient"`);

Something like that...
By default await page.getByPlaceholder('Search patient').click(); will wait.

Hi @dimitriharding thanks for your suggestion but after I applied it, the behavior ignores to wait a moment until the element can be interactable

Even if I change to other combinations, the behavior is the same:
await ai(`Wait for "Search patient" to be visible, enabled and stable`)
await ai(`Wait for "Search patient" to be attached`)

Could be something about configuration on ZeroStep?

ZeroStep AI steps will not auto-retry like regular Playwright steps. You could write this auto-retry logic in your wrapper if you want, but what I would recommend is use an explicit "waitFor" command like page.waitForURL or page.waitForSelector. You can see some usages of these in the examples that we provide.


Add description for "Running Tasks"

中：
执行中任务：正在执行的任务列表，可以在此进行停止任务。运行完成的任务不在此展示。
EN：
Running Tasks: A list of tasks currently being executed, where tasks can be stopped. Tasks that have been completed are not displayed here.

Review this issue both in the two versions of web interface:

The new browser is yet under construction. Check this.
The legacy web. Add the description to the legacy web.


Blazegold Repair exploit

Blazegold tools can be repaired by throwing it into flowing lava. The flowing lava is transformed into the obsidian but the source block remains. This means that you can repair tools passively by having a dropper just drop them into the flowing lava, have a breaker remove the obsidian and allow the source to flow out again.

Should be fixed in the next version, thank you!


feat: make plugin options optional

Resolves https://github.com/vercel/webpack-asset-relocator-loader/issues/144.

Looks like CI is failing:
    ENOENT: no such file or directory, open '/home/runner/work/webpack-asset-relocator-loader/webpack-asset-relocator-loader/test/project-chunking/dist/modules/chunks/541.js'


docs: add macos instructions

Simple instructions for configuring and running on macOS on Apple silicon. Draft just to share the instructions until they are validated and more information can be added.
Reliant on https://github.com/stiiifff/trappist-extra/pull/3

Looks great, thanks !

After "sdkmanager ..." I got the error:
The operation couldn’t be completed. Unable to locate a Java Runtime.
Please visit http://www.java.com for information on installing Java.
This fixed it (ref):
sudo ln -sfn /opt/homebrew/opt/openjdk/libexec/openjdk.jdk /Library/Java/JavaVirtualMachines/openjdk.jdk

Yeah, the '# install required packages, following additional instructions in resulting caveats for each package' bit should've indicated to follow the additional brew instructions for each package, but maybe they got lost when batch installing...


chore: Update Windows VHD packer job to use July 2023 patches

Reason for Change:

Updating Windows VHD packer job to use July 2023 patches
Issue Fixed:

Credit Where Due:
Does this change contain code from or inspired by another project?

[x] No
[ ] Yes

If "Yes," did you notify that project's maintainers and provide attribution?

[ ] No
[ ] Yes

Requirements:

[x] uses conventional commit messages

[ ] includes documentation
[ ] adds unit tests
[ ] tested upgrade from previous version

Notes:

/azp run


Fixes a typo in target a specfic child of an element waypoint

closes #5334
The current text now reads:

Here's how you would give the third element in each well the bounce class

I have tested locally.

Please change the title to be descriptive of the changes you have made.
Also state which issues, if any, that this PR closes in the description each on its own line. For example: closes #1337
see Pull-Request-Contribute

@BerkeleyTrue : Sorry messed up the first PR! Will take care next time. Do you want me to close this and do a new one? Github will not close the issue right?

No it is fine now. Thanks for making those changes.

Thanks! :+1:


Support for MacOS 10.1x

Any reason why this CocoaPod cannot support MacOS 10.12 and above ?

None whatsoever :) I updated the podspec to include MacOS, and published version 1.0.1.
Let me know if you run into any troubles.

Superb. Thanks for your responsiveness. Will report problems if any.
On Mon, Oct 29, 2018 at 11:23 markspanbroek<EMAIL_ADDRESS>wrote:

Closed #1 https://github.com/Charterhouse/base64url-swift/issues/1.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Charterhouse/base64url-swift/issues/1#event-1932550484,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AbmB8T6oJTQDIRxuWc6G80qxwHZwpM9sks5upx1ZgaJpZM4X6lZK
.


Issue with Update Object Dependencies

Facing problem while transferring object dependency

Development is Completed and Released in Version v1.2.0


Remove lodash sortby

Hi, removing lodash.sortby and making it by hand should significantly reduce the size of the package. If you check<EMAIL_ADDRESS>you will see that nearly half of the library is lodash sortby. I checked the source and the function is used only in one place, so it should be quite ok to rewrite that portion with custom sort by code.

@idarbuashvili That's a valuable comment. I'll surely look into it.

Made the #39 pull request, this should do the job. :)


proc macro && canadian cross toolchain && --target

Problem
Not able to set sysroot for host (proc macro) when using --target on a Canadian Cross toolchain.
Using .cargo/config
 [target.x86_64-unknown-linux-gnu]
 rustflags = ["--sysroot", "/opt/poky/3.1.9/sysroots/x86_64-pokysdk-linux"]

I don't see my sysroot show up in -vv output.
Notes
Output of cargo version: 1.51.0

For the curious SDKs available here:  https://drive.google.com/drive/folders/1OfaYuU6tg97cd4WJfcUsuwrxrgHFoCwG?usp=sharing
corei7(x86_64) and rpi4(aarch64) includes rust-cross-canadian 1.51.0 + clang 10.0.1.
Basic setup is:
./poky-glibc-x86_64-core-image-minimal-corei7-64-intel-corei7-64-toolchain-3.1.9.sh
. /opt/poky/3.1.9/environment-setup-corei7-64-poky-linux
sudo chown $USER:$USER -R /opt/poky/3.1.9/sysroots/corei7-64-poky-linux/home/cargo
cargo build --lib --target x86_64-poky-linux

I need a way to pass a sysroot to HOST for proc macro.  Nothing I have tried appears to work.  Any ideas?

Similar to #9712, I don't understand what is being reported here.  Can you show a reproduction of what you are trying to do?

Without more information, I'm going to close.  But if you can provide the information above, feel free to reopen.


[Plot] Profile, allow to choose sum or average

Allow an icon toggling (like with aspect ratio) associated to the width (for instance a "w" and a "w" with a horizontal bar on top) in order to allow the user to choose if the average profile or the sum is desired.

closed by #1929


Database metrics - adding conventions for database server metrics

Hi, We have following proposal for database server metrics, would like to hear some feedback from community.
We referenced to General Guidelines, Attribute Naming, Requirement Levels and current database metrics for connection pool, system metrics, process metric to draft this.
We attempted to extract metrics that are as generic as possible, but this is not an easy task due to the diversity of the databases, we are still working on it, and would like to get some guidance and feedback based on currently what we have.
Resource attributes
Database
Description: A database.

Attribute Key
Type
Description
Example
Requirement Level

db.system
string
An identifier for the database management system (DBMS) product being used.
db2, damengdb
Required

db.version
string
The version of the database
V11.5, V8
Required

server.address
string
Name of the database host.
db.testdb.com
Recommended

server.port
int
database listen port
50000, 5236
Recommended

db.name
string
This attribute is used to report the name of the database being accessed. For commands that switch the database, this should be set to the target database
BLUDB, DAMENG
Conditionally Required: If applicable.

Instance
Description: A database instance. Conditionally Required.

Attribute Key
Type
Description
Example
Requirement Level

db.instance.name
string
The db instance name
db2inst1, DMSERVER
Conditionally Required: If applicable.

db.instance.number
int
The db instance number
1
Conditionally Required

db.instance.host
string
The hostname of the db instance
db.testdb.com
Conditionally Required

Common metric instruments
All metrics in db. instruments should be attached to a Database resource and therefore inherit its attributes, like db.system.
All metrics in db.instance instruments should be attached to a Instance resource and therefore inherit its attributes, like db.instance.name.
Availability

Name
Description
Units
Instrument Type
Value Type
Attribute Key(s)
Attribute Values

db.status
Recommended. The status of the database. 1 (Active), 0 (Inactive)
{status}
Gauge
Int
state (Required)
Active, Inactive

db.instance.status
Recommended. The status of the db instance. 1 (Active), 0 (Inactive)
{status}
Gauge
Int
state (Required)
Active, Inactive

db.instances.count
Recommended. The total number of db instances.
{instance}
UpDownCounter
Int

db.instances.active.count
Recommended. The total number of active db instances.
{instance}
UpDownCounter
Int

Throughput

Name
Description
Unit
Instrument Type
Value Type
Attribute Key(s)
Attribute Values

db.sessions.count
Recommended. The number of database sessions.
{session}
UpDownCounter
Int

db.sessions.active.count
Recommended. The number of active database sessions.
{session}
UpDownCounter
Int

db.transactions.count
Recommended. The number of completed transactions.
{transaction}
UpDownCounter
Int

db.transactions.rate
Recommended. The number of transactions per second.
{transaction}
Gauge
Double

db.transactions.latency
Recommended. The average transaction latency.
s
Gauge
Double

db.sql.count
Recommended. The number of SQLs
{sql}
UpDownCounter
Int64

db.sql.rate
Recommended. The number of SQL per second.
{sql}
Gauge
Double

db.sql.latency
Recommended. The average SQL latency.
s
Gauge
Double

db.io.read.rate
Recommended. The physical read per second.
By
Gauge
Double

db.io.write.rate
Recommended. The physical write per second.
By
Gauge
Double

Performance

Name
Description
Unit
Instrument Type
Value Type
Attribute Key(s)
Attribute values

db.cache.hit
The cache hit ratio/percentage
1
UpDownCount
Double
type
(cache type)

Discussed on Sept 11 meeting, @jsuereth suggested creating a project proposal for a semconv working group on databases.

@niuxiaojuan-github Hi Xiaojuan, Sorry for let you wait. I just checked the list above and I think this list is very great and most index are common to all database products.
But just talk about DB2, here we have a product called DMC(data management console). It is a is an integrated database management tools platform that you can use to manage DB2 LUW. The metrix data collected by DMC is very useful for DB2 user to monitor the running of DB2. Perhaps the metrix of DMC can help something. Please refer to below link about DMC collected DB2 related metrix.
https://www.ibm.com/docs/en/db2-data-mgr-console/3.1.x?topic=metrics-summary


ethereumproject/tests VMTests status

This issue records all the current test status for VMTests in the ethereumproject/tests repository. It will be closed when all tests pass.
Individual files test status
vmArithmeticTest.json
🆗  All passed.
vmBitwiseLogicOperationTest.json
🆗  All passed.
vmBlockInfoTest.json
🆗  All passed.
vmEnvironmentalInfoTest.json
🆗  All passed.
vmInputLimits.json
Not yet.
vmIOandFlowOperationsTest.json
🆗  All passed.
vmLogTest.json
🆗  All passed.
vmPerformanceTest.json
🆗  All passed.
vmPushDupSwapTest.json
🆗  All passed.
vmSha3Test.json
🆗  All passed.
vmSystemOperationsTest.json
Not yet.
vmtests.json
Not yet.

All passed. Closing.


New message types for ipython-kernel

Background: I've been working on something that's essentially a new Jupyter frontend in Haskell, and I needed several more IPython message types to be supported in ipython-kernel. This is kind of an ongoing process, but I thought I'd get a PR out with the current changes.

This looks good, but

Why is ghc-parser in .gitignore? It's a package we want in our repository.
There should not be more than one stack.yaml. There's already one for ihaskell, we don't need one for ipython-kernel, just use the same one.

the ghc-parser was getting randomly updated by the build process (I was finding strings with my home directory in it). Removed from .gitignore.

I thought I needed it to use ipython-kernel as a local stack package. Removed now.

Looks good, just a few very minor comments remaining. Thanks for doing this, this is really helpful, I've actually needed this myself recently but didn't get around to it.
What are you using this for (what sort of front-end are you making)?

Oh sorry, I didn't mean for you to review that commit yet. Emacs haskell-mode was being a little overzealous in commenting unused imports. I'll get a finalized commit out later today.
I'm making a frontend that lets you embed code in a Markdown file -- I'll post it publicly pretty soon

Sorry @gibiansky , this fell off my radar for a while. I just updated it and addressed your comments, could we get this merged?


Support for Merge

Feature request here - could support for merge statements be added? I'm currently involving a stored procedure to do this, but i'd like to make use of the nitty gritty orm aspects of microlite?

I'm not overly familiar with MERGE, is it supported by all the database servers MicroLite targets?


Action space changes

Motivation and Context
Changing action space names to say what they actually do, adding the look up and look down actions
How Has This Been Tested
Tested before ICCV experiment push, not positive that Look Up and Look Down still work. You will need a new build of habitat-sim to support the look action changes.
Types of changes

Docs change / refactoring / dependency upgrade
Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have completed my CLA (see CONTRIBUTING)
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

@danielgordon10 -- same problem here.

moved to https://github.com/facebookresearch/habitat-api/pull/53


More intuitive printing of (symbolic) gates

When working with circuits, it would be nice to print the circuit with output like the following:
sCNOT(1,2)
sCNOT(3,4)
sHadamard(4)
The current style of output is very cumbersome and hard to work with (at least when working on the circuit level). For example:
Symbolic two-qubit gate on qubit 3 and 13
X₁ ⟼ + XX
X₂ ⟼ + X
Z₁ ⟼ + Z
Z₂ ⟼ + ZZ
Symbolic two-qubit gate on qubit 5 and 13
X₁ ⟼ + XX
X₂ ⟼ + X
Z₁ ⟼ + Z
Z₂ ⟼ + ZZ
Symbolic two-qubit gate on qubit 7 and 13
X₁ ⟼ + XX
X₂ ⟼ + X
Z₁ ⟼ + Z
Z₂ ⟼ + ZZ
Having the option to reduce this output to 3 lines would be a very nice feature. Perhaps we can add a "print circuit" function?

@amicciche , should be fixed now. You can see the linked pull request for the changes (using :typeinfo to detect whether we are being printed inside of a print call for an array)


Add support for disk persistence with CQEngine

Fixed.

There is a bug with disk persistence, reopening.

Update operation replaced to support disk persistence.


install vue3-ace-editor run dev failed Dep optimization failed with error:

[vite] Dep optimization failed with error:
[vite]: Rollup failed to resolve import "file-loader?esModule=false!./src-noconflict/ext-beautify.js" from " file-loader?esModule=false!./src-noconflict/ext-beautify.js?commonjs-external".
This is most likely unintended because it can break your application at runtime.
If you do want to externalize this module explicitly add it to
rollupInputOptions.external
Error: [vite]: Rollup failed to resolve import "file-loader?esModule=false!./src-noconflict/ext-beautify.js" from " file-loader?esModule=false!./src-noconflict/ext-beautify.js?commonjs-external".
This is most likely unintended because it can break your application at runtime.
If you do want to externalize this module explicitly add it to
rollupInputOptions.external
at Object.onwarn (D:\WorkSpace\big-screen-design\node_modules\vite\dist\node\build\index.js:70:19)
at Object.onwarn (D:\WorkSpace\big-screen-design\node_modules\rollup\dist\shared\rollup.js:19346:20)
at ModuleLoader.handleResolveId (D:\WorkSpace\big-screen-design\node_modules\rollup\dist\shared\rollup.js:18221:26)
at D:\WorkSpace\big-screen-design\node_modules\rollup\dist\shared\rollup.js:18211:22
at async Promise.all (index 0)
at async ModuleLoader.fetchStaticDependencies (D:\WorkSpace\big-screen-design\node_modules\rollup\dist\shared\rollup.js:18209:34)
at async Promise.all (index 0)
at async ModuleLoader.fetchModule (D:\WorkSpace\big-screen-design\node_modules\rollup\dist\shared\rollup.js:18186:9)
at async Promise.all (index 414)
at async ModuleLoader.fetchStaticDependencies (D:\WorkSpace\big-screen-design\node_modules\rollup\dist\shared\rollup.js:18209:34) {
watchFiles: [

Look like you installed file-loader, it is related to webpack. I'm not sure what happened with you because here not has a reproduction, please follow issue template before open issues.


Migrate Tools update broke our CI

Two weeks ago the Migrate Tools dev branch removed support for Drupal 8. Our testing matrix still includes testing on Drupal 8 which now fails. See @ajstanley 's recent PR build failure.
So, we can either drop Drupal 8 support in our testing matrix OR lock down Migrate Tools in our composer to 5.0 rather than ^5.
Do we have a preference between these two, @Islandora/8-x-committers?

If we are not dropping support for Drupal 8, then I vote for locking it for the time being.

There are several committers on the Sprint Call right now that vote for dropping D8 from the testing matrix since it is EOL anyway.

I agree that we should drop D8 since it's EOL.
I don't know how much it might be used in the Islandora community so do we need to lock it for now and phase it out with an announcement? I think it could be a fairly short phase out period but if folks don't think that's needed then yes we should just get rid of it.

I'm for we drop D8.

So, if we drop D8, is that a major version change, even though our API didn't change? Or something else?

I vote for a version change - is this just for the isladora module?

Bringing over observations made in a Slack thread:

Although, is there a reason why we have to include migrate-tools? Would this better be placed in an install profile?
Yes, we use the Migrate API to pull [islandora model terms] in, but that is part of an installation process, not how the code works. Migrate-tools is simply a useful tool for using the Migrate API.

So, we could just remove the dependency from islandora's composer and info.yml, relying instead on deployment methods to include it.

@bseeger: I'm for dropping 8 and for a major version change - is this just for the islandora module (for starters)?

Yes, only islandora would need to be 3.0. However, modules that include islandora as a dependency would also have to update their composer to be islandora/islandora:^2 || ^3 or drop D8 themselves and shift their major versions as well.

Building on my earlier comment about the third option, removing migrate_tools as an islandora dependency, I made PR 859 (linked above). It appears that this fixes the CI for the islandora module.
However, this puts the onus on deployment methods (e.g. ansible and isle) to install and enable migrate_tools themselves if they use a Drush version less than 10.4.
I know the playbook uses Geerling Guy's drush role which is configured to use Drush 9, so it would either need to override the Geerling Guy config or add migrate_tools to the drupal_composer_dependencies config and drupal_enable_modules config.
It looks like, for isle, we would just need to update the composer.json for the islandora_install_profile_demo which is currently set to use Drush 9 as well. Someone correct me if I'm wrong on that.
Oh, side-note: it appears that simply dropping D8 from the matrix won't help us without locking down composer in the CI to Drush < 10.4 because, even with D9, migrate_tools ^5 will still conflict with Drush 10.6, which the CI is grabbing before checking any other composer requirements and currently isn't us letting us downgrade Drush.

The Drush GeerlingGuy role may default to version 9 but it is overridden in the Playbook or the Drupal install itself is hosting Drush in its own structure, which the Drush launcher is happily running.
vagrant@islandora8:/var/www/html/drupal/web/sites/default$ drush --version
Drush Launcher Version: 0.6.0
Drush Commandline Tool 10.6.1
vagrant

@alxp, is that with the current dev branch or the new install-profile one?

The new one, there was some conflict that prevented an update to Drupal 9 if Drush 9 was pegged.

Looks like the dev branch is also asking for 10.3+.

Fixed with https://github.com/Islandora/islandora/commit/90d67951727452896b2a281337b8fab6485ff237.


Missing constant in Reason enum

The client throws the following error when fetching a transaction.
No enum constant com.braintreegateway.Dispute.Reason.UNRECOGNIZED
There is no "UNRECOGNIZED" constant defined in Reason enum.
Kind Regards

You are correct, Dispute.Reason does not have an UNRECOGNIZED enum, but [Dispute.Status and Dispute.Kind do] (https://github.com/braintree/braintree_java/blob/master/src/main/java/com/braintreegateway/Dispute.java#L10-L36). Is it possible you have selected the wrong enum type? I can't provide further help without more details about your integration, and GitHub isn't the best forum for debugging integrations. I'm closing this issue and encourage you to reach out to Braintree support to provide specific information about your account and integration.