s3 url

Does the s3 backup option support a non amazon s3 endpoint?  We use internal s3 storage and I dont think that the s3://@/  will not work for that as there is no option to include the http(s) endpoint.

You can specify a non Amazon S3 endpoint by specifying the AWS_ENDPOINTS key in the Secret that you are using to provide the access keys. See this page for more information. In the example shown, we have the AWS_ENDPOINT set to http://minio-service.default:9000 for a locally running Minio service. Just substitute that with your endpoint, and then configure everything else accordingly.

I used my private minio as backup and got this error
AWS Error: RequestTimeTooSkewed The difference between the request time and the server's time is too large.

I guess the reason is my server time is different from the default us-east-1 region time.
I tried to change
s3://backupbucket@us-east-1/

to
 s3://backupbucket@cn-north-1/

but it didn't work.


BigInt_to_string missing (only BigInt_to_hex_string)

I'd prefer it be called BigInt_to_decimal or BigInt_to_decimal_string or BigInt_to_dec_string, explicitly mentioning decimal encoding.
I don't need this function for my use case but I can imagine some UI might want to show large numbers as decimals.


[W-10187060] Release Gating Card for Spring 22

Changes

Spring '22 card will now appear under Release Management in Education Cloud Settings.

Issues Closed
W-10187060

New Metadata

Labels

stgReleaseGateEDASpring22Label
stgReleaseGateEDASpring22Description
stgReleaseGateEDASpring22FeatureDescription
stgReleaseGateEDASpring22FeatureHelpLabel

Apex Class

EDASpring22ReleaseGate.cls
EDASpring22ReleaseGate_TEST.cls

Testing Notes

Go to Education Cloud Settings tab
Click on release management
Under "Education Data Architecture", it should have a card with the following details

Card header should say "Spring '22"
Description header should say, "There were no features gated for this release therefore all features were automatically released with Spring'22"
Bullet item 1 header should say, "Please see EDA's release notes for Spring'22 for full details.
Bullet item 2 header should say, "Activate Spring'22"
EDA release gating card should direct users to https://trailhead.salesforce.com/trailblazer-community/groups/0F94S000000kHOcSAM?tab=discussion&sort=LAST_MODIFIED_DATE_DESC

@aarsvoboda I ended up creating new labels for Spring 22 and just matching the naming convention from Winter '22. Looking at it more closely, I presume we could probably just rename the generic text under one label instead of separate ones for releases (FeatureHelpLabel = Follow EDA Releases in the EDA Release Announcements group)
For the redirect link, it currently just goes to the trailblazer group based on the WI although that won't necessarily take them easily to the release notes. I have an open question with Jason about whether only the latest release card should show or if both should show. If both show, I figure we should update the Winter '22 release notes redirect to the trailblazer post of the release notes?


Add minimal pyproject.toml

See #386.
Does not replace setup.py (unless an alternative build backend like
poetry or flit is used), but just lets installers like pip know what is
required and how to run setup.py.

Thanks for the PR, @clbarnes!
I'll defer to resident python experts @bfirsh and @andreasjansson to review this.

I tried poetry install locally and got an error, but it was a pretty quick fix!
Expand for details of the error messages
I tried the poetry compatibility locally and got this error:
❯ cd <...>/cog_repo/python
❯ poetry install

  PyProjectException

  [tool.poetry] section not found in <...>/cog_repo/python/pyproject.toml

  at ~/.poetry/lib/poetry/_vendor/py3.9/poetry/core/pyproject/toml.py:56 in poetry_config
      52│     def poetry_config(self):  # type: () -> Optional[TOMLDocument]
      53│         if self._poetry_config is None:
      54│             self._poetry_config = self.data.get("tool", {}).get("poetry")
      55│             if self._poetry_config is None:
    → 56│                 raise PyProjectException(
      57│                     "[tool.poetry] section not found in {}".format(self._file)
      58│                 )
      59│         return self._poetry_config

I added [tools.poetry] with just name but then got:
❯ poetry install

  RuntimeError

  The Poetry configuration is invalid:
    - 'version' is a required property
    - 'description' is a required property

  at ~/.poetry/lib/poetry/_vendor/py3.9/poetry/core/factory.py:43 in create_poetry
       39│             message = ""
       40│             for error in check_result["errors"]:
       41│                 message += "  - {}\n".format(error)
       42│
    →  43│             raise RuntimeError("The Poetry configuration is invalid:\n" + message)
       44│
       45│         # Load package
       46│         name = local_config["name"]
       47│         version = local_config["version"]

So I added version/description and made setup.py read the version from pyproject.toml. After that, poetry install is happy.
Then tested flit install and got:
❯ flit install
Neither [project] nor [tool.flit.metadata] found in pyproject.toml

and updated accordingly. flit install still throws an error but it's not due to pyproject.toml so I'm going to leave that alone. Glad poetry install is happy now.

Maybe I'm misunderstanding poetry and flit, but aren't they alternative build backends rather than frontends? The point of specifying the build-backend in pyproject.toml is to tell installers/ build frontends (i.e. pip) to use setuptools and setup.py to build it, not some other build backend. Both of those errors are the result of trying to use it like it's a poetry-built or flit-built project (which is why it's looking for metadata specific to those build tools), where it's not.
I think it's best to avoid setup.py reading pyproject.toml, as build frontends/ installation tools (i.e. pip; as opposed to build backends, i.e. flit/poetry) read pyproject.toml to learn that they should read setup.py in order to install it.

@clbarnes I agree that it's crufty, I was surprised that poetry (1.1.12) still requires [tools.poetry] even when version/description are specified in the generic part of pyproject.toml. So without that, poetry cannot be used as a development tool for managing a virtualenv for cog.

The point of specifying the build-backend in pyproject.toml is to tell installers/ build frontends (i.e. pip) to use setuptools and setup.py to build it, not some other build backend.

Correct, and in your branch the pyproject.toml is pointing to the setup.py, and in my branch it is as well.
Subtle point: adding the [tools.poetry] causes no issues, and it is ignored when someone's not using poetry. So it's crufty in the pyproject.toml, but cleaner behavior IMHO

@clbarnes Does my PR make a little more sense now?
Honestly no worries if don't want to merge it! I poked around because I was curious, there's nothing wrong with the minimal pyproject.toml for the sake of build-backend = "setuptools.build_meta"

Interesting. My understanding was that Poetry and Flit are not generic developer-side tools to be applied to any project you work on (the way that, say, pyenv or virtualenvwrapper are). They are opinionated tools which enforce a particular development workflow on particular projects configured to use them (as does setuptools, it's just a workflow which has been round a lot longer). Don't get me wrong, they're great tools! But I think any given project's compatibility being mutually exclusive between setuptools/poetry/flit is a feature rather than a bug; one probably shouldn't use setuptools to manage a poetry-based project and vice versa, and the fact that you can write a frankenstein pyproject.toml which happens to support several build backends for a particular subset of their configurations is more likely to lead to confusion than anything else.

@clbarnes I see where you're coming from. I get the contrast you're drawing between {setuptools, flit} vs. {virtualenv, pyenv}, but I think {poetry, Pipenv} (but especially poetry) can be used as a personal development tool. Sometimes I use poetry even when a project doesn't require it. (In those cases I leave pyproject.toml untracked if teammates don't want to switch to poetry).
My branch has the same result as yours, with just the additional option of using poetry install as a one-command way to install into a virtualenv. But it's not necessary!

We ended up replacing setup.py with pyproject.toml in #843.


the 3.0.1 version encode aes-128-cbc is much slower than 1.1.1m and slower than speed command

I installed openssl 1.1.1 on windows7 x64 from https://slproweb.com/products/Win32OpenSSL.html.
and copy 3.0.1 version binary files to D:\OpenSSL-Win64\bin
openssl speed commands shows their speeds are samilar.
but I useopenssl enc encode a big file(883,893,021 byte), the 3.0.1 version is much slower(8s vs 4s). and sometime it's very very slow (I canceled at 86s).
if I use speed test result to compute the elapsed time, it should be
842MB/574109.14k=1.5s
https://slproweb.com/download/Win64OpenSSL_Light-1_1_1m.exe
--1.1.1m

C:\Users\LD>openssl speed -evp aes-128-cbc
Doing aes-128-cbc for 3s on 16 size blocks: 77371491 aes-128-cbc's in 3.00s
Doing aes-128-cbc for 3s on 64 size blocks: 26191364 aes-128-cbc's in 3.01s
Doing aes-128-cbc for 3s on 256 size blocks: 7378834 aes-128-cbc's in 3.00s
Doing aes-128-cbc for 3s on 1024 size blocks: 1873128 aes-128-cbc's in 3.00s
Doing aes-128-cbc for 3s on 8192 size blocks: 220403 aes-128-cbc's in 3.01s
Doing aes-128-cbc for 3s on 16384 size blocks: 104955 aes-128-cbc's in 3.00s
OpenSSL 1.1.1m  14 Dec 2021
built on: Wed Dec 15 13:50:37 2021 UTC
options:bn(64,64) rc4(16x,int) des(long) aes(partial) idea(int) blowfish(ptr)
compiler: cl /Z7 /Fdossl_static.pdb /Gs0 /GF /Gy /MD /W3 /wd4090 /nologo /O2 -DL_ENDIAN -DOPENSSL_PIC -DOPENSSL_CPUID_OB
J -DOPENSSL_IA32_SSE2 -DOPENSSL_BN_ASM_MONT -DOPENSSL_BN_ASM_MONT5 -DOPENSSL_BN_ASM_GF2m -DSHA1_ASM -DSHA256_ASM -DSHA51
2_ASM -DKECCAK1600_ASM -DRC4_ASM -DMD5_ASM -DAESNI_ASM -DVPAES_ASM -DGHASH_ASM -DECP_NISTZ256_ASM -DX25519_ASM -DPOLY130
5_ASM -D_USING_V110_SDK71_ -D_WINSOCK_DEPRECATED_NO_WARNINGS -D_WIN32_WINNT=0x0502
The 'numbers' are in 1000s of bytes per second processed.
type             16 bytes     64 bytes    256 bytes   1024 bytes   8192 bytes  16384 bytes
aes-128-cbc     413306.60k   556741.25k   630665.53k   640381.54k   599684.40k   574109.14k

E:\>del e:\test.aes1

E:\>d:\timer64 openssl enc -aes-128-cbc -salt -k<PHONE_NUMBER>abcdef -in e:\test.gz -out e:\test.aes1
*** WARNING : deprecated key derivation used.
Using -iter or -pbkdf2 would be better.

Kernel  Time =     2.418 =   56%
User    Time =     1.653 =   38%
Process Time =     4.071 =   95%    Virtual  Memory =      4 MB
Global  Time =     4.274 =  100%    Physical Memory =      8 MB <-----------------------4s

https://slproweb.com/download/Win64OpenSSL_Light-3_0_1.exe
--3.0.1
D:\OpenSSL-Win64\bin>openssl speed -evp aes-128-cbc
Doing AES-128-CBC for 3s on 16 size blocks: 75975502 AES-128-CBC's in 3.01s
Doing AES-128-CBC for 3s on 64 size blocks: 25391907 AES-128-CBC's in 3.00s
Doing AES-128-CBC for 3s on 256 size blocks: 7087778 AES-128-CBC's in 3.00s
Doing AES-128-CBC for 3s on 1024 size blocks: 1657780 AES-128-CBC's in 3.00s
Doing AES-128-CBC for 3s on 8192 size blocks: 225013 AES-128-CBC's in 3.01s
Doing AES-128-CBC for 3s on 16384 size blocks: 111088 AES-128-CBC's in 3.00s
version: 3.0.1
built on: Thu Feb 24 15:29:05 2022 UTC
options: bn(64,64)
compiler: cl /Z7 /Fdossl_static.pdb /Gs0 /GF /Gy /MD /W3 /wd4090 /nologo /O2 -DL_ENDIAN -DOPENSSL_PIC -D_USING_V110_SDK71_ -D_WINSOCK_DEPRECATED_NO_WA
RNINGS -D_WIN32_WINNT=0x0502
CPUINFO: OPENSSL_ia32cap=0x7ffef3ffffebffff:0x37ab
The 'numbers' are in 1000s of bytes per second processed.
type             16 bytes     64 bytes    256 bytes   1024 bytes   8192 bytes  16384 bytes
AES-128-CBC     403746.59k   542558.64k   605789.11k   566758.76k   612227.54k   607656.96k

D:\OpenSSL-Win64\bin>del e:\test.aes1

D:\OpenSSL-Win64\bin>d:\timer64 openssl enc -aes-128-cbc -salt -k<PHONE_NUMBER>abcdef -in e:\test.gz -out e:\test.aes1
*** WARNING : deprecated key derivation used.
Using -iter or -pbkdf2 would be better.

Exit code: -1073741510

Kernel  Time =     0.046 =    0%
User    Time =     0.046 =    0%
Process Time =     0.093 =    0%    Virtual  Memory =      4 MB
Global  Time =    86.783 =  100%    Physical Memory =      9 MB
^C
D:\OpenSSL-Win64\bin>del e:\test.aes1
D:\OpenSSL-Win64\bin>d:\timer64 openssl enc -aes-128-cbc -salt -k<PHONE_NUMBER>abcdef -in e:\test.gz -out e:\test.aes1
*** WARNING : deprecated key derivation used.
Using -iter or -pbkdf2 would be better.

Kernel  Time =     2.480 =   30%
User    Time =     2.230 =   27%
Process Time =     4.711 =   57%    Virtual  Memory =      4 MB
Global  Time =     8.159 =  100%    Physical Memory =      9 MB <-----------------------8s

Perhaps this is a windows-only thing? I cannot reproduce the performance problem on Linux.

How big is the test.gz file?

They note "a big file(883,893,021 byte)"

Ah I've overlooked that.

From the time measurements it seems that there is some slowdown in the user time - which is the time spent by the CPU in the user space. But it is not major. The case where the command stuck seems to be unrelated to OpenSSL as all the time was spent outside the process - perhaps there is some problem with the disk storage on the testing machine?

the test is done on same computer.
version 1.1.1m is installed with the installer on the computer.
version 3.0.1 is installed with the installer on other computer, then I copy the binary files to the test compuer.
Becuase I want keep two versions on the test computer.
I test a bigger file.  same problem occurs.
883,893,021 test.gz
bigger file
 2,085,582,828 lineorder.tbl.gz

E:\>del e:\test.aes1

E:\>d:\timer64 openssl enc -aes-128-cbc -salt -k<PHONE_NUMBER>abcdef -in e:\ssb\10\lineorder.tbl.gz -out e:\test.aes1
*** WARNING : deprecated key derivation used.
Using -iter or -pbkdf2 would be better.

Kernel  Time =     5.803 =   58%
User    Time =     3.868 =   39%
Process Time =     9.672 =   97%    Virtual  Memory =      4 MB
Global  Time =     9.915 =  100%    Physical Memory =      8 MB

vs

D:\OpenSSL-Win64\bin>del e:\test.aes1

D:\OpenSSL-Win64\bin>d:\timer64 openssl enc -aes-128-cbc -salt -k<PHONE_NUMBER>abcdef -in e:\ssb\10\lineorder.tbl.gz -out e:\test.aes1
*** WARNING : deprecated key derivation used.
Using -iter or -pbkdf2 would be better.

Kernel  Time =     7.768 =   29%
User    Time =     5.709 =   21%
Process Time =    13.478 =   51%    Virtual  Memory =      4 MB
Global  Time =    26.131 =  100%    Physical Memory =      9 MB

D:\OpenSSL-Win64\bin>del e:\test.aes1

D:\OpenSSL-Win64\bin>d:\timer64 openssl enc -aes-128-cbc -salt -k<PHONE_NUMBER>abcdef -in e:\ssb\10\lineorder.tbl.gz -out e:\test.aes1
*** WARNING : deprecated key derivation used.
Using -iter or -pbkdf2 would be better.

Kernel  Time =     7.815 =   28%
User    Time =     5.522 =   20%
Process Time =    13.338 =   48%    Virtual  Memory =      4 MB
Global  Time =    27.600 =  100%    Physical Memory =      9 MB

Unfortunately I really cannot reproduce this on Linux. There is practically no slowdown there. @faramir-dev m Could you please try to reproduce this on Windows?

I found the root of the problem. If  I do not use "C:\Program Files\OpenSSL-Win64\start.bat" to start the Win64 OpenSSL Command Prompt,
the version 1.1.1m is as slow as version 3.0.1.
the content of start.bat is follwing:
@echo off
@set PATH=%PATH%;%~dp0bin

title Win64 OpenSSL Command Prompt
echo Win64 OpenSSL Command Prompt
echo.
openssl version -a
echo.

%SystemDrive%
cd %UserProfile%

cmd.exe /K

C:\Users\LD>cd C:\Program Files\OpenSSL-Win64\bin

C:\Program Files\OpenSSL-Win64\bin>del e:\test.aes1

C:\Program Files\OpenSSL-Win64\bin>d:\timer64 openssl enc -aes-128-cbc -salt -k<PHONE_NUMBER>abcdef -in e:\test.gz -out e:\test.aes1
*** WARNING : deprecated key derivation used.
Using -iter or -pbkdf2 would be better.

Kernel  Time =     2.683 =   31%
User    Time =     2.355 =   27%
Process Time =     5.038 =   58%    Virtual  Memory =      4 MB
Global  Time =     8.563 =  100%    Physical Memory =      8 MB

and I use the version 3.0.1  directory to overwrite the 1.1.1m directory,  the version 3.01 runs as fast as 1.1.1m.
Win64 OpenSSL Command Prompt

OpenSSL 3.0.1 14 Dec 2021 (Library: OpenSSL 3.0.1 14 Dec 2021)
built on: Thu Feb 24 15:29:05 2022 UTC
platform: VC-WIN64A
options:  bn(64,64)
compiler: cl /Z7 /Fdossl_static.pdb /Gs0 /GF /Gy /MD /W3 /wd4090 /nologo /O2 -DL_ENDIAN -DOPENSSL_PIC -D_USING_V110_SDK7
1_ -D_WINSOCK_DEPRECATED_NO_WARNINGS -D_WIN32_WINNT=0x0502
OPENSSLDIR: "C:\Program Files\Common Files\SSL"
ENGINESDIR: "C:\Program Files\OpenSSL\lib\engines-3"
MODULESDIR: "C:\Program Files\OpenSSL\lib\ossl-modules"
Seeding source: os-specific
CPUINFO: OPENSSL_ia32cap=0x7ffef3ffffebffff:0x37ab

C:\Users\LD>del e:\test.aes1

C:\Users\LD>d:\timer64 openssl enc -aes-128-cbc -salt -k<PHONE_NUMBER>abcdef -in e:\test.gz -out e:\test.aes1
*** WARNING : deprecated key derivation used.
Using -iter or -pbkdf2 would be better.

Kernel  Time =     2.511 =   56%
User    Time =     1.700 =   38%
Process Time =     4.212 =   94%    Virtual  Memory =      4 MB
Global  Time =     4.467 =  100%    Physical Memory =      9 MB


Ruby Capstone project

Description

Add link to the project
We are three team members in our group

Team member 1 - @Stephen-Adom
Team member 2 - @veronica365
Team member 3 - @Amen-Musingarimi

Hi Student  ,
Status: Approved :heavy_check_mark: :heavy_check_mark: :heavy_check_mark:

To Highlight 💪🏻

You Kanban is Ready , Goodluck ✔

Good job implementing the requirements  :+1: Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉
Optional Suggestions  :

N/A

Every comment with the [OPTIONAL] prefix is not crucial enough to stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag  ichala in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


Error

it's showing me this error I don't know what to do...

Please sideload livecontainer with sidestore/altstore. If you did that, try this build: https://github.com/hugeBlack/LiveContainer/actions/runs/12083344132
Change the signer to zsign in app settings by long press the app and try to run it again.

I tried now its not opening the app.

Then I think you did not install your store correctly, do a fresh install and try again

Then I think you did not install your store correctly, do a fresh install and try again

https://github.com/user-attachments/assets/9e35c07a-4cd7-4592-be81-39cf73a203c3I I reinstalled everything, i did everything you told me to do. it just crashes when I try to open the app..

 I reinstalled everything it's still crashing

Kill LiveContainer from app switcher and try to open it again
I don't see that option where is it ?

By app switcher I mean this: https://support.apple.com/guide/iphone/iph1a1f981ad/ios
By the way, zsign is now quite sable in this version though, so you can switch back to altsign if you correctly installed your store

By app switcher I mean this: https://support.apple.com/guide/iphone/iph1a1f981ad/ios
By the way, zsign is now quite sable in this version though, so you can switch back to altsign if you correctly installed your store

how do you kill it then? just close it ?

yes, just swipe up

I got the same problem. It just appeared out of nowhere it worked just fine before

Oh i didn't realize you guys are using altstore. Maybe altstore updated and broke something. Can you try zsign? @noerauer

I now used SideStore to install it but with altstore it worked just fine and just randomly while using LiveContainer I could use it anymore and it just crashed and I had to restart the app and then I got the message with the OpenSSL. I used the newest alstore classic(installed via Mac) and use a free profile since altstore became free for everyone.

Now I get the same problem on my iPad with AltStore... I tried reinstalling but nothing works

Can you try 3.1.0 from https://github.com/hugeBlack/LiveContainer/actions and use ZSign? After you upgrade, select ZSign in LiveContainer settings, long press the app, open setting, select ZSign for your app and tap force re-sign


Add TCP listener for remote code pushes

Issue for tracking progress on remote code pushing integration into firmware.

Hi guys, is this going to be possible on Tessel 1, or a Tessel 2 feature?

Hi @programbo, Tessel 2 will have remote code deployments but Tessel 1 will not.


Test failures affected by global state in CWP 2.0

Context: CWP 2.0.0, SilverStripe 4.1.1, CWP starter theme, silverstripe/html5 module
Some test failures which are checking template markup content are failing:

SilverStripe\Forms\Tests\EmailFieldTest::testEmailFieldPopulation
SilverStripe\Forms\Tests\FormTest::testValidationExemptActions
SilverStripe\Forms\Tests\FormTest::testSessionValidationMessage
SilverStripe\Forms\Tests\GridField\GridFieldDetailFormTest::testValidator
SilverStripe\Forms\Tests\LookupFieldTest::testNullValueWithNumericArraySource
SilverStripe\Forms\Tests\LookupFieldTest::testUnknownStringValueWithNumericArraySource
SilverStripe\Forms\Tests\LookupFieldTest::testArrayValueWithAssociativeArraySource
SilverStripe\Forms\Tests\LookupFieldTest::testArrayValueWithNumericArraySource
SilverStripe\Forms\Tests\LookupFieldTest::testArrayValueWithSqlMapSource
SilverStripe\Forms\Tests\LookupFieldTest::testWithMultiDimensionalSource
SilverStripe\Forms\Tests\SelectionGroupTest::testSelectedFieldHolder
SilverStripe\Forms\Tests\TreeDropdownFieldTest::testReadonly

Some of these are looking for CSS selectors that depend on either no theme at all or the simple theme installed. In the case of the former we should enforce disabling themes in the tests that don't expect one to override templates, and in the case of the latter we should remove this coupling (I suspect it's the former though).
Also some tests are probably affected by silverstripe/html5 being installed, since the expectations fail due to two HTML tags being on separate lines rather than one single line.

Fixed with #8124


Update 0.5.0 report with results for Gemma 2 model family

Please add rows to the overall-eval CSVs as well


Issue with popup on multiple screens.

I was trying to set popup position to be wherever user pointer is. This is the code I am using (custom control inheriting Popup):
private void OnPointerMoved(object sender, Avalonia.Input.PointerEventArgs e)
{
      if (_isLeftMouseButton)
      {
                Point location = e.GetPosition(null);

                Host.ConfigurePosition(PlacementTarget, PlacementMode.Top, location);
      }
}

This works fine as long as pointer is on screen where placement target is. However popup will not follow pointer if it moves to another screen.  Is this behavior intended? We are trying to build custom docking solution and I was thinking of using popups to move content around but I need popup to follow cursor even on another screen. Alternative is to simply use window but using popup is convenient.

Popup isn't guaranteed to have its own surface. There are platforms that draw popups on the window's surface. So I think you have to use a window instead of a popup to achieve this. Just my thought. Maybe others know more.

we are trying to build custom docking solution and I was thinking of using popups to move content around but I need popup to follow cursor even on another screen

Note that it won't work with Wayland because of the lack to access to window coordinates in any way, shape or form


Road to Unity 2020.X

XRTK - Mixed Reality Toolkit
Analyzing which parts of the toolkit are currently relevant to Unity 2020.X support.
Here's the first batch of errors I get when opening an XRTK project in 2020.2:
Library\PackageCache\com.xrtk.core@0.2.0-preview.111\Runtime\Providers\CameraSystem\BaseCameraDataProvider.cs(90,60): error CS0619: 'XRDevice.isPresent' is obsolete: 'This is obsolete, and should no longer be used. Instead, find the active XRDisplaySubsystem and check that the running property is true (for details, see XRDevice.isPresent documentation).'
Lots of errors in XRTK-WMR due to missing WMR legacy package

Yeah I'm worried about what the future holds if we're forced to have to use the xr subsystems. I really don't want anything to do with them. At the very least we'll need a shim and make a XRTK have it's own XR Subsystems that just forward info.
I have a feeling Unity may get rid of the XR Subsystems entirely once they adopt open xr.

WebXR with unity's xr subsystems is impossible atm bc of the way the xr apis are implemented.

@StephenHodgson here's what I've been thinking.
What if there is a way for us to support both?
A.) Native all the way
B.) XR SDK Manager
B.) Opens up a way to support 2020.X in short term
Essentially the ONLY thing we need XR SDK Manager is to put the editor in "XR Enabled" mode. Nothing else. We can still use our own plugins/dlls etc. So essentially the path could be:

On XRTK import check whether we are above or below 2020.X
-> If we are at least in 2020.1 then check for XR SDK Management package, if not available offer to install it
-> If 2019.4 or below, nothing changes for us

Enable XR using the SDK package and keep everything else the way it is

Well I don't wanna do both, bc then it's more work, and duplicating features is never fun, nor rewarding.

Essentially the ONLY thing we need XR SDK Manager is to put the editor in "XR Enabled" mode. Nothing else. We can still use our own plugins/dlls etc.

This is essentially what I've been thinking but I have a feeling that people taking on the xrtk might want us to forward the data back to the xr subsystems, which I'm not really open to, just bc they're more likely to try to access the data from there instead of the services directly as intended.
If we do end up having to implement some basic boilerplate XR subsystems to get XR working it'd likely be the XR Subsystem Display and that is it.
I've already implemented XR Subsystems for other hardware venders in the past year so I know the basics of what needs to happen there.

Essentially the XRTK will be a plugin option in the XR SDK Manager preferences, which I don't like, but atm we have no other option.


Fix hostname env.

Ref https://github.com/containerd/cri/pull/898#issuecomment-419991978
Add HOSTNAME back. /cc @estesp
Signed-off-by: Lantao Liu<EMAIL_ADDRESS>

Nice; thanks @Random-Liu

We should cherrypick this into all supported branches.

/lgtm


Fixes Broken Lightmaps

Fixes an issue where old scenes with lightmaps would break with the new change.
To do this, any material with the 'lightmap' property will be saved or loaded
as 'indirectLightMap'.

Added directly to areo-r96 branch in https://github.com/SpinVR/three.js/commit/e00a3037472d883ff75cffcfea0a7fefef46f342.


How to start things up ?

Hi,
i'd like to use your work to create my first home cluster (zero experience on that).
I first tried k8s-at-home/template-cluster-k3s, i followed all the steps and it worked but i'd like to use yours so i'm "yelling" for a little help. Actually is a big please.
Hardware that i have:

Mikrotik router which seems able to work with CoreDNS
TrueNas for storage
XCP-ng Hypervisor for VMs

Things that confuses me:

i don't see any ansible so how can i deploy cluster (seems to be in .gitignore) ?
what is "siderio", it seems is another cluster that deploys "cluster-o" ?
is a must to have multiple nodes for this cluster to work ?

Could you lease help me with some hints to deploy the cluster, set networks, storages, lb, dns ? :)
Thank you

I've checked old commits where you removed /ansible from project so how you create/update/remove  cluster and how you generate all sop secrets from each manifest/app ?

Are you that busy so you can't help me with some hints ?

Are you that busy so you can't help me with some hints ?

Yes, I have been.

i don't see any ansible so how can i deploy cluster (seems to be in .gitignore) ?

There is no Ansible involved. See below.

what is "siderio", it seems is another cluster that deploys "cluster-o" ?

Yes, it is exactly that (link).

is a must to have multiple nodes for this cluster to work ?

Yes, because (amongst other reasons) my storage cluster requires 3 replica's.
In general, my repo can serve as an inspiration for anyone to get their own cluster set up, but trying to do so will require some basic understanding of how to set up a cluster. Unfortunately I do not have the time to provide any support services on those steps.

Thanks


Add text_array_from and text_from methods to Android and iOS Screen Base classes

Methods added:
text_from
text_array_from

@rafaelanastacioalves Can you, please, write a description for this PR?
I know that we were talking about these functions in another PR, but you closed it and opened this other one, so all that information got lost :(

@oscartanner ,
this was the only description I provided:

Is that ok?


Better "subserializer" support

When nested serializers are defined with { ModelClass => SerializerClass, .... } the code doing the lookup uses
def serializer_for(serializer_data, model_class)
      if serializer_data.is_a?(Hash)
        serializer_data[model_class]        <---- returns a nil if it can't match on the class.
      else
        serializer_data
      end
    end

if serializer_data[model_class] misses to match nil is returned. subsequent code will call new on the "not found" serializer. It happened quite often during the last weeks that we had annoying debug sessions where a good error message could save lots of time. Sth like "Could not resolve (nested) serializer. Object to be serialized is of class XYZ, only serializers for [A, B, C] are defined....". Something like this.
Another feature could also be to support "inheritance". If no perfectly fitting serializer is found (say i serialize a nested student but only have serializers for persons and cars) maybe a parent class serializer is available (serialize student with person serializer).
So the topics would be:

[ ] Better error messages when serializer resolving fails
[ ] Optional: Match instances to serializers for parent class objects

Yeah, I think it makes sense, that this lookup will raise an error, when we don't find the appropriate class.
For the other solution, lookup based on inheritance might be harder to implement as this would require to loop through the entire keyspace of the hash and that would eliminate the O(1) lookup cost inside the hash.
WDYT about an alternative, that we can specify a fallback:
collection :drinks, serializer: {
  Drink          => DrinkSerializer,
  HeatedDrink    => HeatedDrinkSerializer,
  UnbrandedDrink => UnbrandedDrinkSerializer,
  fallback: GenericDrinkSerializer
}

This way we could implement it as:
serializer_data.fetch(model_class) do |key|
  serializer_data[:fallback] or raise(ArgumentError, "There's no serializer defined for a model of type #{key}")
end

WDYT?


Haskell linter error message get truncated

When I run Haskell linter and move the cursor to the line where there is an error, the status bar at the bottom shows the error message. But since the status bar has only one line, the error message get truncated.

Try running :Errors to see the messages in full.

Thanks for pointing that out!

It's a standard vim command but maybe it should be documented in this project somewhere. Would you be interested in sending a pull request to update the readme?


HELP failed launch browser

solusi ?

@bagusdwip Just install chromium-browser properly also if a problem persists, make sure to reinstall puppeteer & puppeteer core


Support GiB and its families

Technically, units with binary prefixes are based on 2, for example, 1 GiB = 1024 MiB, and without the binary prefixes are based on 10, for example, 1 GB = 1000 MB.
TiKV only supports units without binary prefixes and treats them as if they are prefixed. To make it more precise, we need to support GiB and its families too. And to keep compatibility, GB and its families should still be treated as they are.

I remember that we have discussed this problem a long time before. I think to make us less trouble, I prefer treating GB and GiB the same...

seem CockroachDB met the same problem before https://github.com/cockroachdb/cockroach/issues/4735

Agree. The purpose of this issue is to add support for *iB in ReadableSize parser.

Got it

@BusyJay  maybe this is good to the version downgrade..
I had try master code then downgrade to 3.0.rc1...but meet this
thread 'main' panicked at 'invalid auto generated configuration file "/data/deploy/data/last_tikv.toml", err only B, KB, MB, GB, TB, PB are supported: "10MiB" for key `readpool.storage.stack-size`', src/config.rs:1122:17
stack backtrace:
   0: std::sys::unix::backtrace::tracing::imp::unwind_backtrace
             at libstd/sys/unix/backtrace/tracing/gcc_s.rs:49
   1: std::panicking::default_hook::{{closure}}
             at libstd/sys_common/backtrace.rs:71
   2: std::panicking::default_hook
             at libstd/panicking.rs:227
   3: std::panicking::rust_panic_with_hook
             at libstd/panicking.rs:475
   4: std::panicking::continue_panic_fmt
             at libstd/panicking.rs:390
   5: std::panicking::begin_panic_fmt
             at libstd/panicking.rs:345
   6: tikv_server::main
             at src/config.rs:1122
   7: std::rt::lang_start::{{closure}}
             at /checkout/src/libstd/rt.rs:74
   8: main
   9: __libc_start_main
  10: <unknown>

then I need manual to replace MiB back to MB for every nodes....
maybe some big cluster upgrade need take care this problem :D

Sorry to see that happen, but it's intended not to support version downgrade.


Add specs for Module#refinements and Refinement#refined_class

#1016
[Feature #12737]

Module#refinements has been added

Thank you for the specs!


[reflection]: python: reflection returns original_request

Closes #36943
This PR adds original_request to any _reflection_pb2.ServerReflectionResponse generated by grpc_reflection.v1alpha

cc. @gnossen @XuanWang-Amos (as assignees to the related issue)

@Drarig29  Thanks for the contribution . Please Add unit-tests for the suggested changes.

You're welcome! - added the unit tests

Hi @sreenithi! I updated the unit tests and they passed locally. FYI, the "Mergeable" check is failing because of labels that I cannot change.

Hi @sreenithi! Gentle bump on the PR, is anything blocking?

Hi, apologies for the delay. The code looks good to me. I will review and try to merge it this coming week.

Hi @sreenithi and @sourabhsinghs! Is it expected that this PR wasn't released yet?
Could it be due to the failing CI on the merge commit? We have a Python service that requires that change in order to work properly (like other gRPC servers).

Hi @drfloob!
I'm pinging you as you are a member of the grpc org and you published a release recently. Could you have a look?
Thanks a lot! 🙇

Hi @Drarig29 , this PR has been merged to the master branch last month (see it here), there were no issues because of the failing CI test, but required some internal changes, and hence there was a delay initially, however, it is already merged at the moment.
Let me check and revert on which release will include this change.

@sreenithi did you find out which release will include this change?

Hi, yes, it will be included in the upcoming release 1.67.0 which is due for release in the next 1-2 weeks

Hi, the 1.67.0 was released on PyPi today. You should now be able to use this fix


Reorganize directory structure

While it was kind of nice having everything at the top level, the number of files is now getting a bit unwieldy and hard to understand, so I took a stab at organizing the directories without introducing too much nesting.
Test Plan:

Ran npm run serve to ensure that local builds still work
Ran npm run prepack then open dist/release/index.html to ensure that release builds still work
Ran scripts/deploy.sh to ensure that the deployed version of the site will still work when I eventually redeploy
Ran npm run jest to ensure that tests still work correctly

Coverage remained the same at 38.067% when pulling 021587a3d3d3e75bfffe9e338b861c87ac72204e on jlfwong/file-reorg into 11a3a67dce2f3eba90a2a96bc98d8745425cf1de on master.


RSS/Atom feed for blog

Does your blog have an RSS/Atom feed?
My reader, https://bazqux.com/ (written in Haskell and Ur/Web, BTW) cannot find it.

Seems like this has been fixed finally.


Inventory components for multiple children

Depend on the conclusion of weekly meeting 1, some components should be capable of having multiple direct children (based on normal HTML scenarios).
For example: Headings usually have only 1 direct child, whereas List and HyperLink might have multiple ones.

According to the conclusion of weekly meeting 3, some basic components will be replaced with HTML native tags. For example: HyperLink. Hereby this issue is closed for no longer needed for this moment.


Service Bus Connector  : Peak Lock trigger not available in Logic app standard

I am not able to see Service Bus Connector with peak-lock trigger in logic app standard. Its there in the consumption model. Is this not supported yet? I  have a scenario where I need to first peek the message in a topic and then delete later in logic app standard.
Logic App Standard -

Logic App Consumption

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

ID: c2aeacc5-5b74-c4be-335e-c7e768854e4a
Version Independent ID: d8693b52-fdd6-cc68-bc12-26807454a80b
Content: Exchange messages with Azure Service Bus - Azure Logic Apps
Content Source: articles/connectors/connectors-create-api-servicebus.md
Service: logic-apps
GitHub Login: @ecfan
Microsoft Alias: estfan

@nareshnagpal06
Thanks for your feedback! We will investigate and update as appropriate.

@nareshnagpal06 Thanks for reaching out. I believe you have created stateful workflow on your standard logic app.
You are trying to leverage service bus Build-in connector which doesn't support the peek look actions. In case if you want to leverage the peek lock action then you should use service bus Azure connector which supports the peek lock actions.

Alternatively, if you want to leverage the service bus Build-in connector then you should create the stateless workflow.
Let me know if you have any queries or concerns.

@nareshnagpal06 Hope the above helps and clears things up. We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.

@MayankBargali-MSFT & @nareshnagpal06: I also added a note to our how-to doc that peek-lock operations are available only with the managed connector, not the built-in version.

On further investigation, peek-lock operations are available with the built-in connector, but only for stateless workflows, not stateful workflows. We'll update the docs to reflect this new info.


(No. 17 C) PAD IDs in report map and layer style toggling [ Medium ]

Feedback, pulled from #93 :

[ ] Include PAD IDs in the pop-up within the map and allow users to turn on/off barriers to treat vs. not to treat in case someone wants to only see the solution in a print out of the solution.

Can we break this up into:

[ ] Include PAD ID in the pop-up in the map, and
[ ] allow users to turn on/off barriers to treat vs. not to treat in case someone wants to only see the solution in a print out of the solution.

Seems like the former is a smaller task than the latter.

2nd: addressed in overhaul of report page?
Breaking up report by budget tics, easier and more performant, but lose single table and single map output.

These have not been addressed as asked, but there are workarounds for both:

PAD IDs  can be pulled from table and from search box when clicked.
Can't turn off treat/don't treat, but can sort table, and there is a filtered CSV option.


Adding total level message option

Simple toggle and text box to allow displaying total level as part of the leveling Discord message.
Really being able to parse $total was the main requirement, but adding the check box and text box allows some customization from the player. Some friends and I are trying to max, so it's nice to keep track of each others totals when we see a Discord notification :)
Thanks!

Thanks for the improvement! Looks good!


Missing Plutus script version compatiblity

In Babbage ledger spec, at Figure 3, it states about "allowedLanguages" that indicates what different versions of scripts are allowed in a transaction depending on some of features transaction uses. Specifically, the spec say V1 and V2 scripts are allowed when transaction doesn't have reference script nor inline datum; only V2 scripts are allowed when transaction has reference script or inline datum.
Similar notation for Conway spec is needed so that developers can get a clear view on how a V3 script should be written to interact with older version scripts that are already deployed on chain.

That's right, the big issue here is that we don't yet have support for multiple versions in the spec. There are a few more pressing issues and challenges associated with this, so it could be a while until we manage to fix this.
The general policy for Plutus versions is that we don't allow old Plutus versions to run in transactions that use new features.

Thanks!
Will specification for this land before Conway lunch? Or will Conway nodes launch without any set spec over this topics

Okay, thanks for clarifying


Add/Edit Socket Events

This adds an "addItem" event and replaces the existing "updateItem" event with "editItem". It turns out that we really need a separate event for updating item positions in the editor since that event is called so rapidly. When it was generalized into the "editItem" event, it ended up being pretty slow because of the way React updates its state. So I created a new "updateItemPosition" event that is directly listened for in the RoomCanvas component.
I also think that it makes sense to have distinct sending/response events. For example, a client would send an "editItem" event to the server, and then if its valid and successful, the server would send back an "itemEdited" event, which the client would then use to update the element. I guess they could be named the same, but I think it makes it clearer to phrase them the way I did.
I also ended up changing the array of items on the frontend back to an array instead of a map since its more performant when it needs to be frequently remade (which it does in order for React to detect state changes when items are updated). This probably isn't a necessary change since I originally did it to solve the performance issues with the generalized "updateItem" event, so it might be better to go back to a map in the future.
And, because of this I also changed the backend List model back to storing and array of ListItems instead of a map, since it simplifies serving it to the frontend. Although I'm not sure if this is best either and again it may be that a map is more performant. Not really sure though which is better in Rethinkdb.

This works well on my machine, and I am sure we can optimize it once it becomes a bigger problem.


Typo in 'nodename nor servname'

There is a typo.  Should be 'node name nor server name'.
nodename nor servname provided, or not known

Can you please provide more descriptive information.


SubstituteSymbolTable should not allocate an array

While most of our ion-tests file tests take <0.01s to process, ion-tests/iontestdata/good/subfieldVarUInt.ion currently takes over 40s on my laptop. This is because the test imports a non-existent symbol table with an enormous maxId.
Because that symbol table cannot be resolved via the Catalog, the Reader falls back to instantiating a SubstituteSymbolTable, a symbol table for which all symbols are undefined.
Despite having no symbol data to manage, the SubstituteSymbolTable still allocates an array of length maxId. When maxId is enormous, this can take tens of seconds to complete.
We should modify the implementation to simply return 0 for any symbol IDs it's asked to resolve that fall within in the range of offset to maxId. This will require some refactoring of the SharedSymbolTable contract, which currently allows for direct access to the underlying Array that is usually in use.

Closed by #424.


Different body for thing and action POST and PUT

The structure of the request body for adding a thing or action differs from the request body for updating a thing or action
POST:
{
  "action": {
    "@class": "string",
    "@context": "string",
    "schema": {}
  },
  "async": true
}

PUT:
{
  "@class": "string",
  "@context": "string",
  "schema": {},
  "creationTimeUnix": 0,
  "lastUpdateTimeUnix": 0
}

Though this is technically not a bug, from a UX perspective I would expect them to have the same structure - even if one has additional key-value pairs.

The same goes for the validating a thing or action, which seems to be maintain the same body structure as with updating a thing or action, but not with adding a thing or action.
Validate POST:
{
  "@class": "string",
  "@context": "string",
  "schema": {}
}

Also, I've seen that the docs describe different request bodies compared to Swagger:
PUT in docs
{
    "action": {
        "@class": "Animal",
        "@context": "http://myzoo.org",
        "schema": {
            "name": "Charles",
            "species": "monkey"
        }
    }
}

POST validate in docs
{
    "action": {
        "@class": "Animal",
        "@context": "http://myzoo.org",
        "schema": {
            "name": "Bert",
            "species": "elephant"
        }
    }
}

I've started addressing all of these in #826. I'll keep this issue open until #826 is ready to be merged.

Closing as #826 has been merged.


SLING-7140  Support redirects to URLs provided by the underlying datastore.

This pull request does not require any changes in Oak.
It introduces 2 interfaces to Sling API to allow a ResourceProvider to implement URIProvider that will allow Sing to redirect to a direct URL on get operations. If no URIProviders are implemented, no redirects are issued. The URIProvider implementation would normally be implemented inside the ResourceProvider eg in the case of a S3DataStore for Oak, in the bundle where that code is implemented.

Applied via SVN and into the new Git repo for resource resolver.


the train image size

I'm trying to reproduce the results of the paper, and this is result I have now.
Easy   Val AP: 0.9136305801084846
Medium Val AP: 0.9004443217585435
Hard   Val AP: 0.7486346101347016

i test the val dataset with 800*800，but can not reach the accuracy in papers.Do you follow the training details in the paper。
thanks.

Yes.

@chenjun2hao @ywlife  Can you pls tell me the input size of images for validation and test set being fed into the model and also the threshold used to obtain the results you got in the paper?
I tried  it on 1280*720 with thresh=0.1 and got the following results:
Easy Val AP : 0.803
Medium Val AP : 0.826
Hard Val AP :  0.728


Current-bench gives up testing commits on the master branch if new commits have been pushed in the meantime

To reproduce:

Merge a PR (this triggers current-bench on the merge commit)
While current-bench is still running, merge another PR

This will cancel the bench on the previous merge commit an leave the github status for said merge commit as "pending" for all eternity.

On PRs, cancelling the previous run may still be a good idea, but on master we still want to benchmark for those commits, I think.


updated article with link to limits

https://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness/issues/1608

@LolaJacobsen Hi Lola, changes look correct. Could you please check and merge? Thanks!


chore: update repo by service bot

This is a routine update from cc-service-bot based on the service.yml in your repo. This PR
should be approved and merged as soon as CI passes.

CHANGELOG
FILE NAME - CHANGED BY - COMMIT MESSAGE
Makefile - framework - chore: update repo Makefile cc-mk-include v0.1247.0
cc-mk-include/compare/v0.1243.0...v0.1247.0


Update Thor::Actions#inside to return the value yielded by the block

🌈
So that calling code can more conveniently inspect & use this value.
The value previously returned by this method doesn't seem useful so this seems like a safe change to make.
If this looks OK, should I also update the CHANGELOG?

@rafaelfranca any thoughts on this?


[Bug]: Email input allows invalid default value

Is there an existing issue for this?

[X] I have searched the existing issues

Description
email input config panel doesn't validate default text
Steps To Reproduce

add input widget
set data type to "email"
add invalid default text "hello world"

no error messages

deploy
type something in email input

validation triggered
error shown

Public Sample App
No response
Version
cloud

@zhirzh the default text can be invalid and we expect developers to handle that. The validation only runs for the values input by the end user


Add Credential Management API

Please add info about support of credential management (status - editors draft):
http://w3c.github.io/webappsec-credential-management/
https://developers.google.com/web/updates/2016/04/credential-management-api

+1

Now available at http://caniuse.com/#feat=credential-management


Add Blooks

Doesn't Work In New Update / New Added Pack

Working cheats here: https://github.com/Minesraft2/Blooket-Cheats


Ambiguous reference to member 'withClass', with three "candidate" copies of TyphoonDefinition

I am working on an iOS app, and include Typhoon using either CocoaPods or source-as-subproject. In either case, I get this strange error. Typhoon links & imports correctly, however the Swift compiler complains that my use of withClass is ambiguous, and refers to three different Typhoon.TyphoonDefinitions for the source. However, it doesn't provide any info about why there are 3, or how they are different.
See my StackOverflow question for images: http://stackoverflow.com/questions/36120533/typhoon-di-framework-produces-ambiguous-reference-error

Hmm.. Did you solve that problem? If not, can you share demo project (so anyone can easily face with that issue)?

hi there, thanks for responding. I solved the issue for myself and made note on SO, but forgot to comment here.
I first installed Typhoon as a dependency by Carthage, but decided to undo that bc of some compatibility issues I was having with other dependencies. So i reinstalled it as a submodule. My best guess is there were some artifacts from this that left behind, and Xcode found duplicate symbols there or something -- once I cleaned out build folder and all derived data folders, etc, then rebooted everything, it compiled OK and the issue is gone.

:+1: Great! Then I guess this issue can be closed


Issue template

Introduce an issue template.

Do you think we can categorize opened issues by their purpose like

bugfix
refine
add
delete
or something else?

Good point. Will take that up in the template.


pagedmembers issue

HI Team,
I am learner here . I want to create a flow where bot will interact with 3 people . 1st interaction will take bot and chat initiator . 2nd interaction will have a state where bot will open up a window with another user to get an approval .
I used your botbuilder-samples\samples\javascript_nodejs\57.teams-conversation-bot code and got below error . Please suggest what to do
unhandled error: Error: /v3/conversations/e9452730-0f91-11eb-a20b-096b7b0fce21%7Clivechat/pagedmembers does not
exist

Hi @souvik01123
57.teams-conversation-bot is intended to be used from the Teams channel.  Based on this url, /v3/conversations/e9452730-0f91-11eb-a20b-096b7b0fce21%7Clivechat/pagedmembers  it looks like the emulator was used to connect to the bot?  Unfortunately, the emulator does not work with the Teams samples.
Please try the bot from Teams.  Some instructions can be found in the readme.

Edit the manifest.json contained in the teamsAppManifest folder to replace your Microsoft App Id (that was created when you registered your bot earlier) everywhere you see the place holder string <> (depending on the scenario the Microsoft App Id may occur multiple times in the manifest.json)

Zip up the contents of the teamsAppManifest folder to create a manifest.zip

Upload the manifest.zip to Teams (in the Apps view click "Upload a custom app")


not working

{"data":{"trackUnion":{"__typename":"GenericError"}},"extensions":{"cacheControl":{"version":1,"hints":[]}}}

Can you please provide more information?

Which function?
What's the data you are passing through it?

Sorry for not being precise... Okay so I was using it with nodejs and tsup... When I am trying to call the function to search for album its giving this error....
My tsup config:
export default defineConfig({
  clean: !isDev,
  dts: !isDev,
  entry: ["src/index.ts"],
  format: ["esm"],
  minify: !isDev,
  metafile: !isDev,
  sourcemap: true,
  target: "esnext",
  outDir: "dist",
  onSuccess: isDev ? "node dist/index.mjs" : undefined,
});

Kinda confused if that was typescript error... but when i cloned the repo and used the build it works fine

I have set the type to module and after that it complains like this:
🚧  Running in development mode
CLI Building entry: src/index.ts
CLI Using tsconfig: tsconfig.json
CLI tsup v6.7.0
CLI Using tsup config: /home/jabed/Devs/spotify/tsup.config.ts
CLI Running in watch mode
CLI Target: esnext
ESM Build start
ESM dist/index.js     177.00 B
ESM dist/index.js.map 331.00 B
ESM ⚡ Build success in 5ms
CLI Watching for changes in "."
CLI Ignoring changes in "**/{.git,node_modules}/**" | "dist"
WebapiRegularError: An error occurred while communicating with Spotify's Web API.
Details: No token provided.
    at _toError (/home/jabed/Devs/spotify/node_modules/.pnpm/spotify-web-api-node@5.0.2/node_modules/spotify-web-api-node/src/http-manager.js:39:12)
    at /home/jabed/Devs/spotify/node_modules/.pnpm/spotify-web-api-node@5.0.2/node_modules/spotify-web-api-node/src/http-manager.js:71:25
    at Request.callback (/home/jabed/Devs/spotify/node_modules/.pnpm/superagent@6.1.0/node_modules/superagent/lib/node/index.js:905:3)
    at /home/jabed/Devs/spotify/node_modules/.pnpm/superagent@6.1.0/node_modules/superagent/lib/node/index.js:1127:20
    at IncomingMessage.<anonymous> (/home/jabed/Devs/spotify/node_modules/.pnpm/superagent@6.1.0/node_modules/superagent/lib/node/parsers/json.js:22:7)
    at Stream.emit (node:events:513:28)
    at Unzip.<anonymous> (/home/jabed/Devs/spotify/node_modules/.pnpm/superagent@6.1.0/node_modules/superagent/lib/node/unzip.js:53:12)
    at Unzip.emit (node:events:513:28)
    at endReadableNT (node:internal/streams/readable:1359:12)
    at process.processTicksAndRejections (node:internal/process/task_queues:82:21) {
  body: { error: { status: 401, message: 'No token provided' } },
  headers: {
    'www-authenticate': 'Bearer realm="spotify"',
    'access-control-allow-origin': '*',
    'access-control-allow-headers': 'Accept, App-Platform, Authorization, Content-Type, Origin, Retry-After, Spotify-App-Version, X-Cloud-Trace-Context, client-token, content-access-token',
    'access-control-allow-methods': 'GET, POST, OPTIONS, PUT, DELETE, PATCH',
    'access-control-allow-credentials': 'true',
    'access-control-max-age': '604800',
    'content-type': 'application/json',
    'content-encoding': 'gzip',
    'strict-transport-security': 'max-age=31536000',
    'x-content-type-options': 'nosniff',
    date: 'Thu, 11 May 2023 05:34:23 GMT',
    server: 'envoy',
    via: 'HTTP/2 edgeproxy, 1.1 google',
    'alt-svc': 'h3=":443"; ma=2592000,h3-29=":443"; ma=2592000',
    connection: 'close',
    'transfer-encoding': 'chunked'
  },
  statusCode: 401
}

Node v : 18
OS : arch linux

Tried with .mjs

I can't find spotifly in the error stack rather the problem is with your spotify-web-api-node which is a different package.
Also, getTrack only works with track ids. To search use searchTracks.
Please read the documentation carefully (https://github.com/tr1ckydev/spotifly#-documentation).
Let me know if I can help you with anything else.

FFF!!!!!!!!!!!! Srry buddy I was just being dumb from yesterday night... I apolozise...  And got my problem solved also...


firefly: init at 1.4.3

Black box fuzzer for web applications
https://github.com/Brum3ns/firefly
Related to https://github.com/NixOS/nixpkgs/issues/81418
Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 342478 run on x86_64-linux 1

1 package built:

firefly


Electron (Standalone) Build.

An electron build would make it possible to use octo offline without the need of hosting a server, or digging through the files. (Could also be used for mobile ports.)

If we make an electron build, there are some UI changes that are worth considering:

Should there be a file menu?
Can we add a checkbox for auto-saving changes to the file to disk?
Does vim mode need to be extended to support wq?

Some of these might belong in separate tickets.

I'm not interested in maintaining an Electron port of Octo.
It is true that significant changes to the UI would be necessary to reflect the availability of general-purpose local filesystem interaction. Most of the exporting functionality in the Binary Tools panel of the toolbox would need tweaks, as another example.
Octo is open-source: anyone interested in Electron is free to pursue this idea in a fork.

Not interested in having it as part of this repo, or not interested in being the main person maintaining it?
Either way, it might still be a good idea to separate the tools from the current execution settings in the UI.

Also, I tried getting Octo loaded as-is in electron. How the manual is handled would also need some consideration, as the current appearance is a little odd:

I am neither interested in maintaining an electron port nor do I think it belongs in this repo. Electron depends on the npm ecosystem and draws in an enormous transitive closure of brittle, endlessly shifting dependencies and build infrastructure that I really do not want to deal with.
Octo currently fetches several resources from github pages while in operation. In addition to the manual, it retrieves the contents of the "examples" menu using GitHub APIs, and the standalone HTML build functionality fetches JS files from a github pages URL- see here: https://github.com/JohnEarnest/Octo/blob/gh-pages/js/sharing.js#L162
Octo "sharing URLs" are constructed by hitting a storage backend I host, and legacy URLs may also reach out to Github "Gists"- these are probably fine as-is, but without a URL bar there will need to be an entirely different UI for accessing/entering them.

Ok, I  strongly agree with you that npm is full of brittleness. I don't object to closing this ticket, but I'd still like to discuss UI changes once I've thought about it some more. There are some changes that might be useful even if an electron build is never made, and I will probably be making a new ticket for that.

It has nothing to do with Electron, but I have a solution to offline mode and avoiding web browsers: c-octo!


horizontal only?

I don’t think it’s possible to have a drag or zoom behavior that works only horizontally and doesn’t “trap” the natural vertical scrolling gesture?

duplicate of https://github.com/d3/d3-zoom/issues/48 and https://github.com/d3/d3-zoom/pull/171 ?

Not the same issue, since the difficulty here is not about applying only t.x. It's more about reading horizontal scroll, but leaving the natural vertical scroll to the browser.
hammerjs does it this way:
https://github.com/hammerjs/hammer.js/blob/master/src/touchactionjs/touchaction-constructor.js#L109


Add a strong name to Pyrolite

For assembly identity benefits.
https://docs.microsoft.com/en-us/dotnet/framework/app-domains/strong-named-assemblies
Fixes https://github.com/irmen/Pyrolite/issues/62

From what I gathered so far, doesn't the .snk file has to be kept secret? (i.e. not included in the github repo)

From what I gathered so far, doesn't the .snk file have to be kept secret? (i.e. not included in the github repo)

Only if it's intended to be used for security purposes.  If it's just for identity purposes, being able to put an assembly into the GAC, etc., it's fine to be disclosed.  This is actually briefly called out in the docs, albeit buried:
"If you are an open-source developer and you want the identity benefits of a strong-named assembly, consider checking in the private key associated with an assembly into your source control system."


Thread based concurrency

Concurrently runs a number of 'Racecar::Runner' instances in a fixed size thread pool.
Each thread starts a single Racecar::Runner with Racecar::Consumer class instance. All threads run the same consumer class, have the same config and consume partitions from the same topic(s).
ThreadPoolRunnerProxy can be combined with ParallelRunner, to run forks and threads. ParallelRunner is not used or battle tested (at Zendesk) and so this is still not a recommended thing to do.
Racecar does not yet implement a health-check mechanism so an uncaught error in a single worker thread will cause the process to start a graceful shutdown of all threads before exiting with an error.
Other inclusions:

Signal handling has been moved up one level to the CLI
Runner-like object interface standardized to #run #stop #running?
Tests can be run locally without Docker, export LOCAL=1
Some test bugs have been fixed, connections now always close and orphaned processes raise an exception
Suggest that 'parallel' be deprecated in favor of fork/forking next to thread/threaded for ease of understanding

My pleasure!
That sounds a little more complex but totally doable. This is very much MVP concurrency, changing as little as existing code as possible.
Unless someone already has that work in progress, I can give it a try.
Is this enough an improvement to consider merging? If so, I think we could iterate towards your proposed design without breaking the API. It should also be marked experimental anyway.

I'm not sure it provides enough value as it stands – with copy-on-write, the forking approach seems better if we're using separate consumer instances anyway.
I think the shared-main-loop approach should be pretty doable; if you look into Runner, you can see how batches are being processed sequentially – we would change that to an async model, with a fixed partition number -> worker thread mapping probably.


Create only one tag if workspace is a single git repo

For example my workspace contains two crates: main lib and proc macro attribute for it, the lib depends on the attr.  When cargo release tries to publish this crates it first publishes the attr crate, creates a tag for it, then it tries to create tag for the lib and, of course, fails with tag x.y.z already exists
Example output:
$ cargo release --skip-publish patch
Release version proc-macro-error-attr 0.3.2? [y/N]
y
Update proc-macro-error-attr to version 0.3.2
[cargo_release 5caec6a] v0.3.2
 1 file changed, 1 insertion(+), 1 deletion(-)
Creating git tag v0.3.2
Pushing to git remote
Enumerating objects: 11, done.
Counting objects: 100% (11/11), done.
Delta compression using up to 2 threads
Compressing objects: 100% (7/7), done.
Writing objects: 100% (7/7), 733 bytes | 244.00 KiB/s, done.
Total 7 (delta 4), reused 0 (delta 0)
remote:
remote: To create a merge request for cargo_release, visit:
remote:   https://gitlab.com/CreepySkeleton/proc-macro-error/merge_requests/new?merge_request%5Bsource_branch%5D=cargo_release
remote:
To https://gitlab.com/CreepySkeleton/proc-macro-error.git
   98a7569..5caec6a  cargo_release -> cargo_release
Enumerating objects: 1, done.
Counting objects: 100% (1/1), done.
Writing objects: 100% (1/1), 164 bytes | 54.00 KiB/s, done.
Total 1 (delta 0), reused 0 (delta 0)
To https://gitlab.com/CreepySkeleton/proc-macro-error.git
 * [new tag]         v0.3.2 -> v0.3.2
Finished proc-macro-error-attr
Release version proc-macro-error 0.3.2? [y/N]
y
Update proc-macro-error to version 0.3.2
[cargo_release 54f64dc] v0.3.2
 1 file changed, 1 insertion(+), 1 deletion(-)
Creating git tag v0.3.2
fatal: tag 'v0.3.2' already exists

Sounds like you are overriding some tag settings which makes it so the tag is the same for all crates?
Skipping out on the tagging would fail for other workspaces (like mine) where not all crates are released every time and so rely on unique tags (which are the default).
I would think instead if you are wanting to control whether to tag or not, you'd configure the crates you don't want tagged to skip it.

Sounds like you are overriding some tag settings which makes it so the tag is the same for all crates?
Yes, exactly, every release gets its own unique tag. All the crates get published simultaneously.

Skipping out on the tagging would fail for other workspaces (like mine) where not all crates are released every time and so rely on unique tags (which are the default).
I see.

I would think instead if you are wanting to control whether to tag or not, you'd configure the crates you don't want tagged to skip it.
Actually, that's what I do. I just thought it might be good default behavior.

Actually, that's what I do. I just thought it might be good default behavior.

One way to get closer to it being a default is to disable tagging in WORKSPACE_ROOT/release.toml and then re-enable it in one CRATE_ROOT/release.toml.
At this point, I think I'm going to close this since there are use cases this default would block.  Feel free to let me know if anyone disagrees and we can re-open it and discuss it.


trouble finding "yournamed" .gvoice to set pw

I am setting up pygooglevoice for the first time, so this is a pretty newbie
question. Basically I know that I need to find "yournamed".gvoice so I can
personalize the settings so I can log in from the terminal and such. However
I'm not sure what "yournamed" stands for, and even after trying various names
and manually searching through my files (mostly directed by where the terminal
said it installed other pygooglevoice files) I have not been able to find it.

My OS is Snow Leopard, so you know the general file structure. Obviously I
can't expect you guys to find it for me, but some helpful tips to find it would
be appreciated.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 13 Sep 2011 at 4:14

Hi! Any answers to this? I'm having the same issue!


Required policy to access bucket?

I'm using this IAM policy for the user configured for accessing the bucket located in AWS:
{
    "Version": "2012-10-17",
    "Statement": [
        {
            "Action": "s3:ListBucket",
            "Effect": "Allow",
            "Resource": [
                "arn:aws:s3:::mybucket"
            ],
            "Sid": "VisualEditor0"
        },
        {
            "Action": [
                "s3:DeleteObject",
                "s3:GetObject",
                "s3:PutObject",
                "s3:PutObjectAcl",
                "s3:RestoreObject"
            ],
            "Effect": "Allow",
            "Resource": [
                "arn:aws:s3:::mybucket/*"
            ],
            "Sid": "VisualEditor1"
        }
    ]
}

That's following the recommendation.
However, it doesn't work:
ERRO[2023/02/17 10:43:07] error syncing 'default-location-recurring-backup': handler backups: failed to check if s3 bucket [mybucket] exists, error: 400 Bad Request, requeuing

What's missing in the policy? 🤔
It would be very helpful if the docs explained what are the required minimal permissions for this kind of access.
@moduon MT-2305

It turns out the real error I was seeing is #302.


fix a typo

Please provide a description for what this PR is for.
And to help us figure out who should review this PR, please
put an X in all the areas that this PR affects.
[ ] Configuration Infrastructure
[x] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

🤔 🐛 You appear to be fixing a bug in Go code, yet your PR doesn't include updates to any test files. Did you forget to add a test?
Courtesy of your friendly test nag.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: zhaoyuguang
To complete the pull request process, please assign nmittler
You can assign the PR to them by writing /assign @nmittler in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

istioctl/OWNERS
pilot/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Hi @zhaoyuguang. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@zhaoyuguang Thanks for contribution :) need to sign the cla first.

I signed it!

/ok-to-test

@zhaoyuguang: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/integ-new-installer-k8s-presubmit-tests.sh
fab91ff40952bbe0b9edafa5ef9609b0f0b6d3f7
link
/test integ-new-install-k8s-presubmit-tests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@zhaoyuguang: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/integ-new-installer-k8s-presubmit-tests.sh
fab91ff40952bbe0b9edafa5ef9609b0f0b6d3f7
link
/test integ-new-install-k8s-presubmit-tests-master

prow/integ-mixer-k8s-presubmit-tests.sh
fab91ff40952bbe0b9edafa5ef9609b0f0b6d3f7
link
/test integ-mixer-k8s-presubmit-tests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@zhaoyuguang: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/integ-new-installer-k8s-presubmit-tests.sh
fab91ff40952bbe0b9edafa5ef9609b0f0b6d3f7
link
/test integ-new-install-k8s-presubmit-tests-master

prow/integ-mixer-k8s-presubmit-tests.sh
fab91ff40952bbe0b9edafa5ef9609b0f0b6d3f7
link
/test integ-mixer-k8s-presubmit-tests-master

prow/racetest.sh
fab91ff40952bbe0b9edafa5ef9609b0f0b6d3f7
link
/test istio-racetest-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/lgtm
/ok-to-test

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: howardjohn, zhaoyuguang
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

istioctl/OWNERS [howardjohn]
pilot/OWNERS [howardjohn]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@zhaoyuguang: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/integ-mixer-k8s-presubmit-tests.sh
fab91ff40952bbe0b9edafa5ef9609b0f0b6d3f7
link
/test integ-mixer-k8s-presubmit-tests-master

prow/racetest.sh
fab91ff40952bbe0b9edafa5ef9609b0f0b6d3f7
link
/test istio-racetest-master

prow/integ-new-installer-k8s-presubmit-tests.sh
fab91ff40952bbe0b9edafa5ef9609b0f0b6d3f7
link
/test integ-new-install-k8s-presubmit-tests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@zhaoyuguang: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/integ-mixer-k8s-presubmit-tests.sh
fab91ff40952bbe0b9edafa5ef9609b0f0b6d3f7
link
/test integ-mixer-k8s-presubmit-tests-master

prow/racetest.sh
fab91ff40952bbe0b9edafa5ef9609b0f0b6d3f7
link
/test istio-racetest-master

prow/integ-new-installer-k8s-presubmit-tests.sh
fab91ff40952bbe0b9edafa5ef9609b0f0b6d3f7
link
/test integ-new-install-k8s-presubmit-tests-master

prow/integ-pilot-k8s-presubmit-tests.sh
fab91ff40952bbe0b9edafa5ef9609b0f0b6d3f7
link
/test integ-pilot-k8s-presubmit-tests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@zhaoyuguang: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/racetest.sh
fab91ff40952bbe0b9edafa5ef9609b0f0b6d3f7
link
/test istio-racetest-master

prow/integ-new-installer-k8s-presubmit-tests.sh
fab91ff40952bbe0b9edafa5ef9609b0f0b6d3f7
link
/test integ-new-install-k8s-presubmit-tests-master

prow/integ-pilot-k8s-presubmit-tests.sh
fab91ff40952bbe0b9edafa5ef9609b0f0b6d3f7
link
/test integ-pilot-k8s-presubmit-tests-master

prow/integ-mixer-k8s-presubmit-tests.sh
fab91ff40952bbe0b9edafa5ef9609b0f0b6d3f7
link
/test integ-mixer-k8s-presubmit-tests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@zhaoyuguang: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/integ-new-installer-k8s-presubmit-tests.sh
fab91ff40952bbe0b9edafa5ef9609b0f0b6d3f7
link
/test integ-new-install-k8s-presubmit-tests-master

prow/integ-pilot-k8s-presubmit-tests.sh
fab91ff40952bbe0b9edafa5ef9609b0f0b6d3f7
link
/test integ-pilot-k8s-presubmit-tests-master

prow/integ-mixer-k8s-presubmit-tests.sh
fab91ff40952bbe0b9edafa5ef9609b0f0b6d3f7
link
/test integ-mixer-k8s-presubmit-tests-master

prow/racetest.sh
fab91ff40952bbe0b9edafa5ef9609b0f0b6d3f7
link
/test istio-racetest-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test istio-racetest-master


Generate playback showing which circles were clicked

Generally useful for debugging and seeing what exactly it's doing

To make this more efficient we can store the circles and their timestamp of when they were clicked (and maybe the frame? dunno about that one) and then upon closing it turns it into a gif or just spits out the images


Remove temporary String.strip() implementation

What changes did you make and why?
Context: #5141 this PR fixed the problem about white space characters not being trimmed properly when using java.lang.String#trim() by implementing String#strip() internal implementation because it was introduced in JDK 11. Since, we're at 17 we can remove this one with the actual one.

Thanks :)


Support for URL path and query

Capture the URL's path and query parameters of each incoming request, and visualize them on the show hub page.

This PR should actually solve the issue https://github.com/kyledayton/requesthub/issues/4


using declaration example containing goto doesn't compile

I am integrating this proposal into the Ecma C# standard.
This proposal contains the following text, along with an example:

There are no restrictions around goto, or any other control flow construct in the face of a using declaration. Instead the code acts just as it would for the equivalent using statement: ..

The example shows goto jumping to label target. However, this results in the following error:

CS8649: A goto cannot jump to a location before a using declaration within the same block.

Does this mean that the paragraph cited above (and its example) should be removed?
If not, how should it (and the example) be changed?
What about the "other control flow constructs;" is there anything we can still say about them?

Rex

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

ID: bc900853-b666-5815-9d69-e408af3e3900
Version Independent ID: 77f3af83-d0ca-7d48-d06d-0f28f5f05a49
Content: Pattern based using and using declarations - C# 8.0 draft specifications
Content Source: proposals/csharp-8.0/using.md
Product: dotnet-csharp
Technology: csharp-spec
GitHub Login: @dotnet-bot
Microsoft Alias: wiwagn

ping @jcouv @333fred
I tried the sample in question:
{
    using var f1 = new FileStream("...");
  target:
    using var f2 = new FileStream("...");
    if (someCondition)
    {
        // Causes f2 to be disposed but has no effect on f1
        goto target;
    }
}

it does generate a compiler error.
The equivalent using statement doesn't generate an error:
{
    using (var f1 = new FileStream("..."))
    {
  target:
        using )var f2 = new FileStream("..."))
        {
            if (someCondition)
            {
                // Causes f2 to be disposed but has no effect on f1
                goto target;
            }
        }
    }
}

Who would have the correct details on how we should update the spec?

@chsienki should.

Yup, looks like the speclet is out-of-date. There are definitely restrictions around jumps.

Yes, it was decided not to allow jumping 'back out' of a logical scope either, even though you can with a using statement. https://github.com/dotnet/csharplang/blob/3c8559f186d4c5df5d1299b0eaa4e139ae130ab6/meetings/2018/LDM-2018-12-03.md#goto-and-using-declarations


adding License  notice

Hello
I  open this ticket to propose a Pull-request concerning the licenses.
We would like to use your project as part of an open-source project and we  must  comply with  license law.
There are two aspects  the license of fastapi_client code and  the license of the generated code.

License of the project
For the moment since  there is no license  the default copy  copyright law applies, no one may reproduce, distribute, or create derivative works.  source GitHub documentation

License of the generated code
Since the project doesn't have any license yet, the default copyrights apply to the template. By consequence  the generated code include some copyrighted code.  In my understanding  nobody can use eventually the generated code.

For these reasons, could you consider adding :

A license in the project
A notice about the generated code  licence.

I could  take charge of  the modification and  the Pull-request.
Will you accept a pull request to do same as openapi-generator did regarding the license ?
Openapi-generator is under apache 2.0 license and the project  contains the following notice :

3.4 - License information on Generated Code
The OpenAPI Generator project is intended as a benefit for users of the Open API Specification. The project itself has the License as specified. In addition, please understand the following points:
The templates included with this project are subject to the License.
Generated code is intentionally not subject to the parent project license
When code is generated from this project, it shall be considered AS IS and owned by the user of the software. There are no warranties--expressed or implied--for generated code. You can do what you wish with it, and once generated, the code is your responsibility and subject to the licensing terms that you deem appropriate.
source

Thanks
Christophe

Sorry for the delay!

Thanks you David


Mouse wheel pasting text

There is a weird issue with mouse wheel.
Steps to reproduce

Open new terminal window
Copy text to clipboard (from anywhere)
Scroll mouse wheel up (nothing happens)
Paste text to terminal
Scroll mouse wheel up (now it pastes text on every  scroll event)
Copy other text to clipboard
Scroll mouse wheel up (old text getting pasted until you paste new one into terminal)

Middle click pastes from selection again after very recent commits now.
To scroll in the terminal, hold alt or shift. Otherwise you will be inputing termschars that correspond to pasting in your shell.


[BUG] An exception occurred while running Bloxstrap

Acknowledgement of preliminary instructions

[X] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.

What problem did you encounter?
System.OverflowException: Arithmetic operation resulted in an overflow.
Bloxstrap_20240424T023040Z.log

Known bug, though thank you for reporting.


Disagreement in logging statements when condor_vault_storer call fails

There's some issue with the logging when condor_vault_storer fails.  We're saying that there's an error, but then the higher level calling function says that we were able to store vault/bearer tokens.  Fix this so both error messages say the same thing, and if possible, expand messages like the last to add a credd field.
time="2022-11-16T21:58:07-06:00" level=error msg="Error running condor_vault_storer to store and obtain tokens; exit status 1" service=nova_production
time="2022-11-16T21:58:07-06:00" level=error msg="\nAuthentication needed for nova_production\nAttempting kerberos auth with https://htvaultprod.fnal.gov:8200 ... succeeded\nAttempting to get token from https://htvaultprod.fnal.gov:8200 .
..htgettoken: Failure getting token from https://htvaultprod.fnal.gov:8200\n failed\n\ncondor_vault_storer: htgettoken failed\n  More details might be available by running\n    condor_vault_storer -v \"nova_production\" \n" service=nova_production
time="2022-11-16T21:58:07-06:00" level=error msg="Could not obtain vault token: exit status 1" serviceName=nova_production
time="2022-11-16T21:58:07-06:00" level=debug msg="Stored vault and bearer tokens in vault and condor_credd/schedd" credd=jobsub01.fnal.gov serviceName=nova_production

Fixed by e579753.


Makefile: Support namespace name variable for dr-cluster w/o hub

Use existing OPERATOR_SUGGESTED_NAMESPACE variable to specify dr-cluster namespace name for hubless case

What about the deploy-hub? I don't see code modifying the namespace.

The OPERATOR_SUGGESTED_NAMESPACE variable currently only applies to the Ramen dr-cluster namespace name, not the Ramen hub namespace name.

What about the deploy-hub? I don't see code modifying the namespace.

The OPERATOR_SUGGESTED_NAMESPACE variable currently only applies to the Ramen dr-cluster namespace name, not the Ramen hub namespace name.

Is this intended?
In ramenctl config we use the --namespace for both hub and managed clusters.

What about the deploy-hub? I don't see code modifying the namespace.

The OPERATOR_SUGGESTED_NAMESPACE variable currently only applies to the Ramen dr-cluster namespace name, not the Ramen hub namespace name.

Is this intended?
In ramenctl config we use the --namespace for both hub and managed clusters.

I think the original intent of the variable was to support downstream builds.  I'm not sure how the Ramen hub component is deployed....
There is a use case where a cluster is both a hub and dr-cluster that we've used for testing, but the deployments and config maps have different names so they can coexist in the same namespace.  In other words, that use case does not require separate namespace names.

What about the deploy-hub? I don't see code modifying the namespace.

The OPERATOR_SUGGESTED_NAMESPACE variable currently only applies to the Ramen dr-cluster namespace name, not the Ramen hub namespace name.

Is this intended?
In ramenctl config we use the --namespace for both hub and managed clusters.

I think the original intent of the variable was to support downstream builds. I'm not sure how the Ramen hub component is deployed....

We have deploy-hub target, not using the OPERATOR_SUGGESTED_NAMESPACE and bundle-hub using it. Looks like a bug to me.
If you fix deploy-dr-cluster (uses dr-cluster-config), it makes sense to fix also
deploy-hub.

We have deploy-hub target, not using the OPERATOR_SUGGESTED_NAMESPACE and bundle-hub using it. Looks like a bug to me.

bundle-hub sets it in the Ramen config map as the namespace to deploy the dr-cluster operator to.

We have deploy-hub target, not using the OPERATOR_SUGGESTED_NAMESPACE and bundle-hub using it. Looks like a bug to me.

bundle-hub sets it in the Ramen config map as the name of the namespace to deploy the dr-cluster operator to.

deploy-hub does not use bundle-hub and this is the way we deploy in the testing environment (see ramenctl/ramenctl/deploy.py).

Labeled low priority because impact is limited to developer productivity.

Closing as stale, part of general cleanup. Please reopen and update if still required.


how does this work

how does this work

I have no idea

Did either of you follow the docs? https://morecreative.fdd-docs.com/
If yes, what don't you understand, then we can help according to that


Pinned tabs are lost if you close a window

I am constantly losing my pinned tabs because I often end up closing a whole window.
The only way to close a window is to, first close all tabs except the pinned tabs, then focus another window.
This is not natural and the end result is that I always end up inadvertently closing my pinned tabs.

Close, because this is not achievable.
In addition, even if the browser allows this, I will not implement it because it takes time.
If you can achieve it, you are welcome to create a PR.

Why close the issue ? I thought this was an open source project ? So it's not only you working on this.
But by closing the issue, you make it invisible as something to fix for anyone else browsing this repository.

You don't understand open source, in fact I am the only one doing this, and your feature request is impossible.


Adds simple regex for unnatural_in_context_learning

Due to use of ! as required formatting default regex is not good for these task. Added regex captures everything until a new-line.

LGTM, thanks!


DNS challenge with route53 fails with MalformedInput in 0.4.1

I tried to run lego 0.4.1 on an EC2 instance using route53. It failed with message MalformedInput: ChangeBatch is not valid, expected ChangeResourceRecordSetsRequest. With lego 0.4.0 the same command worked and a certificate was issued.
Maybe this is related to https://github.com/aws/aws-sdk-go/issues/1550 ?
Complete log output (I replaced my domain with www.example.com and removed ids.):
./bin/lego<EMAIL_ADDRESS>--domains="www.example.com" --dns=route53 run
2017/10/01 08:26:25 [INFO][www.example.com] acme: Obtaining bundled SAN certificate
2017/10/01 08:26:25 [INFO][www.example.com] AuthURL: https://acme-v01.api.letsencrypt.org/acme/authz/<id>
2017/10/01 08:26:25 [INFO][www.example.com] acme: Could not find solver for: http-01
2017/10/01 08:26:25 [INFO][www.example.com] acme: Trying to solve DNS-01
2017/10/01 08:26:26 [www.example.com] Could not obtain certificates
        Error presenting token: Failed to change Route 53 record set: MalformedInput: ChangeBatch is not valid, expected ChangeResourceRecordSetsRequest
        status code: 400, request id: <request-id>

I did not need to configure AWS credentials as lego automatically uses the instance role. This is very useful, thanks for this feature!

Hello! Thank you for reporting this!
From what I can see from the bug you linked this may in fact be it. Would it be possible for you to try to compile lego yourself and see if it works now? If it does I will recompile the 0.4.1 release and update the binaries.

Hi @xenolf,
Sorry, but I can't compile lego myself at the moment as I don't have a linux machine available and I don't want to install the compiler toolchain on my server. In a few days I will have access to my linux box again and can try it.

I run into this with the docker image https://hub.docker.com/r/xenolf/lego/

@hostmaster are you able to try the latest master? It should be fixed there. If the fix is verified I will publish new binaries.

Sorry it's my bad. The Docker image works for me.

Able to reproduce
⟫ docker run "xenolf/lego" --version
lego version 0.4.1
Also I'm a bit confused, master and v0.4.1 are both 14 days old, but this issue was opened only 10 days ago. Is go updated in alpine 3.6?

Just wanted to confirm that building from the latest master solves the issue for me.

@xenolf Build from master branch resolved the problem. Thanks!

I take it new binaries have not been released? Still seeing this issue, and I am unable to build from master in the meantime due to #422.

I am seeing this issue too on lego version v0.4.1-0-g67c86d860a79. I'll build from master and see if that fixes it. In the meantime, perhaps a new image could be released?

This fixed my issue.
I rebuilt lego on Debian 9.1 from commit aa94fb4 using go version go1.9.1 linux/amd64.

The go compiler was installed from tarball into /usr/local/go.
The PATH was set to include /usr/local/go/bin.
Basically, it was a bog-standard setup exactly as instructed.

mkdir -p ~/go/src && cd ~/go/src
go get -u github.com/xenolf/lego
go build github.com/xenolf/lego
sudo cp ./lego /usr/local/bin/

Latest container is working for me now. Thanks!

Closed because the problem is solved


require at least one non-numeric character in cashtags

see this post: https://bitclout.com/posts/163c5aca37641a3505821be4cbb31ae5fe0a37c6e8a5d98d1332b46552afe35f

Great work!


View Only links sanitization inconsistent between initial naming and editing

Steps
Two weird versions:
Version 1:
Action: Create a new view only link with the name: Version 2:
Action: Create a new view only link with the name: <!blarg
Output: name: Empty (in red)
Action: Edit a name to be: <!blarg
Output: name: <!blarg
Expected
Editing the name and the original naming procedure should return the same values, however, the editing seems to sanitize less possibly. I have yet to find any exploits due to this, but it is slightly concerning that they don't return the same outputs when given the same inputs.

Can you update the bug report to be sure that the characters you typed display correctly?
Our sanitizer (bleach) HTML-escapes ASCII entities (and unicode-converts unicode entities), so what is stored in the database is sometimes different than what you typed in. This may be part of that same family of bugs, but seeing the actual snippets typed in would be helpful.

Updated to fix github rendering the coded version properly...


Refinement

:palm_tree:

Looks really good. Thanks for your help. We aren't Java developers at the end of the day. Please let us know if there is anything else you'd like to add or see in this.

No problem! Check out my latest commit and tell me what you think:
https://github.com/T145/ATLChecker/commit/cc335b3f77ad1203f6a783bbb0729db7ffaf7e5f

:+1:
You should drop by #nodecraft on Esper and chat with us :)


Add deprecation warning

Closes #114
Let's leave this open for ~1 month until setup-micromamba is tested a bit.

Adding a migration guide would also be helpful


Speed up MultiShotSampler

investigate the tf.gather speed in the multishotsampler. See if we can speed it up using something else, np.take or for loops?

Address #155 as well

Now in 0.14 from commit #160
Replaced tf.gather with a loop appending the items to a list and then casting to np.array.


Model Descriptions File

Regarding this issue

Adding a succinct model description file.
Store file in ./notebooks/.
Name file model_description.md.

This issue also covers adding the model descriptions to the README.


Upgrade Hibernate ORM to 6.3

Fix #1718
Fix #1703
We need to wait for the next Hibernate ORM release containing these changes.

Superseded by https://github.com/hibernate/hibernate-reactive/pull/1734


Add data-gtm-trigger for born digital items

For #11012
What does this change?
Adding a data-gtm-trigger attribute to links in the DownloadTable component that houses the born digital items that are available to download.
There will be an accompanying piece of work to complete within GTM to use thes data-attributes as triggers (in keeping with how we've done long-lived GTM tracking before) to fire tags.
How to test
I don't think it's possible to test locally because the triggers are behind a toggle which doesn't work in conjunction with GTM preview.
How can we measure success?
Events are being sent through to GA4
Have we considered potential risks?
can't think of any

I'm struggling to wrap my head around why the event doesn't come up in with Tag Assistant (and data-gtm-trigger="tree_chevron" does), why do we think it doesn't work?

Is it not just that it's. not been implemented in GTM yet? - See "There will be an accompanying piece of work to complete within GTM to use these data-attributes as triggers" above

But I see it in Tags, should it be added somewhere else as well?

I think the reason we don't see it in preview is that the relevant code is behind a toggle which the attached preview doesn't know about (I have a feeling we encountered this issue previously)

I also haven't been able to test this @gestchild. If we can't while the code is behind the toggle I think it would be low risk to test when live.

@LaurenFBaily David confirmed that something else was blocking the event (here) and was working on a fix, is this urgent or can it wait for his return?


REFACTOR-#6362: __setitem__ on multiple columns should be evaluated lazily

What do these changes do?

[x] first commit message and PR title follow format outlined here

NOTE:  If you edit the PR title to match this format, you need to add another commit (even if it's empty) or amend your last commit for the CI job that checks the PR title to pick up the new PR title.

[ ] passes flake8 modin/ asv_bench/benchmarks scripts/doc_checker.py
[ ] passes black --check modin/ asv_bench/benchmarks scripts/doc_checker.py
[ ] signed commit with git commit -s
[ ] Resolves #6362
[ ] tests added and passing
[ ] module layout described at docs/development/architecture.rst is up-to-date

@dchigarev so we can close it, right?

Looks like something similar already done in #6618. Feel free to reopen if needed.


Attempt to add cause info

This is a work in progress to link Java exception causes with Python ones.  Unfortunately, the cause slot is already used when throwing from Java to Python, so it is not clear if this can actually be made into a function PR.
Potentially fixes #1047

@Thrameos Since it came up lately (line no = -1 vs line no = 0), is this PR obsolete?

Likely though I don't remember what this was for.   I think I was trying at one point to unify the exception outputs more closely.   But was lost along the why fighting the lack of Python support for exceptions.


Fixed importbundle error caused by checkbox addition

Confirmed that spleen segmentation transforms runs successfully. Also fixed error when pressing cancel loading/saving transforms

thanks, verified this works from my desktop


Add nil check for getting read/write queries

If application does not call tx.Query() or tx.Exec(), *sql.Tx.tx instance is nil .
In this case, invalid memory address occur  at WriteQueries or ReadQueries.
So I add nil check for that.

Codecov Report

Merging #13 into master will increase coverage by 0.04%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #13      +/-   ##
==========================================
+ Coverage   75.67%   75.72%   +0.04%
==========================================
  Files          16       16
  Lines        2064     2068       +4
==========================================
+ Hits         1562     1566       +4
  Misses        319      319
  Partials      183      183

LGTM 👍

Thanks for review !