cmd/cgo/internal/testshared: gccgo tests fail on some linux/ppc64le gccgo compilers

What version of Go are you using (go version)?
I have observed this using the fedora 36 system gcc and gccgo. It also happens using IBM's advance-toolchain 15 and newer.

$ go version
go version devel go1.21-3aea422e2c Wed Jun 14 18:53:13 2023 +0000 linux/ppc64le
$ gccgo -dumpversion
12
$ gcc -v
Using built-in specs.
COLLECT_GCC=gcc
COLLECT_LTO_WRAPPER=/usr/libexec/gcc/ppc64le-redhat-linux/12/lto-wrapper
OFFLOAD_TARGET_NAMES=nvptx-none
OFFLOAD_TARGET_DEFAULT=1
Target: ppc64le-redhat-linux
Configured with: ../configure --enable-bootstrap --enable-languages=c,c++,fortran,objc,obj-c++,ada,go,lto --prefix=/usr --mandir=/usr/share/man --infodir=/usr/share/info --with-bugurl=http://bugzilla.redhat.com/bugzilla --enable-shared --enable-threads=posix --enable-checking=release --enable-targets=powerpcle-linux --disable-multilib --with-system-zlib --enable-__cxa_atexit --disable-libunwind-exceptions --enable-gnu-unique-object --enable-linker-build-id --with-gcc-major-version-only --enable-libstdcxx-backtrace --with-linker-hash-style=gnu --enable-plugin --enable-initfini-array --with-isl=/builddir/build/BUILD/gcc-12.2.1-20221121/obj-ppc64le-redhat-linux/isl-install --enable-offload-targets=nvptx-none --without-cuda-driver --enable-offload-defaulted --enable-gnu-indirect-function --enable-secureplt --with-long-double-128 --with-long-double-format=ieee --with-cpu-32=power8 --with-tune-32=power8 --with-cpu-64=power8 --with-tune-64=power8 --build=ppc64le-redhat-linux --with-build-config=bootstrap-lto --enable-link-serialization=1
Thread model: posix
Supported LTO compression algorithms: zlib zstd
gcc version 12.2.1 20221121 (Red Hat 12.2.1-4) (GCC)

What did you do?

$ cd $GOROOT/src/cmd/cgo/internal/testshared
$ go test -v -test.run=TestGoPathShlibGccgo
--- FAIL: TestGoPathShlibGccgo (0.22s)
    shared_test.go:783: executing /home/murp/git/go/bin/go install -compiler=gccgo -linkshared ./exe failed exit status 2:
        panic: runtime error: invalid memory address or nil pointer dereference
        [signal SIGSEGV: segmentation violation code=0x1 addr=0x10 pc=0x4b074c]

        goroutine 1 [running]:
        debug/elf.(*Section).Open(0x0?)
        	/home/murp/git/go/src/debug/elf/file.go:138 +0x2c
        debug/elf.(*Section).Data(0x0)
        	/home/murp/git/go/src/debug/elf/file.go:119 +0x30
        cmd/go/internal/work.readpkglist({0xc00033c180?, 0x13d464?})
        	/home/murp/git/go/src/cmd/go/internal/work/action.go:387 +0x320
        cmd/go/internal/work.(*Builder).linkSharedAction.func1()
        	/home/murp/git/go/src/cmd/go/internal/work/action.go:786 +0x6c
        cmd/go/internal/work.(*Builder).cacheAction(0xc00034a000, {0xc00032e2a0, 0x24}, 0x0, 0xc000307558)
        	/home/murp/git/go/src/cmd/go/internal/work/action.go:418 +0xa8
        cmd/go/internal/work.(*Builder).linkSharedAction(0xc00034a000, 0x2, 0x2, {0xc00033c180, 0x5e}, 0x0)
        	/home/murp/git/go/src/cmd/go/internal/work/action.go:782 +0x128
        cmd/go/internal/work.(*Builder).addTransitiveLinkDeps(0xc0003078e8?, 0xc0003122c0, 0xc000312420, {0x0, 0x0})
        	/home/murp/git/go/src/cmd/go/internal/work/action.go:731 +0x2e0
        cmd/go/internal/work.(*Builder).LinkAction.func1()
        	/home/murp/git/go/src/cmd/go/internal/work/action.go:604 +0x390
        cmd/go/internal/work.(*Builder).cacheAction(0xc00034a000, {0x66d69f, 0x4}, 0xc00039c000, 0xc0003079a8)
        	/home/murp/git/go/src/cmd/go/internal/work/action.go:418 +0xa8
        cmd/go/internal/work.(*Builder).LinkAction(0xc00034a000, 0x1, 0x662500?, 0xc00031a001?)
        	/home/murp/git/go/src/cmd/go/internal/work/action.go:571 +0x84
        cmd/go/internal/work.(*Builder).AutoAction(0x0?, 0x0?, 0xc000324090?, 0x15?)
        	/home/murp/git/go/src/cmd/go/internal/work/action.go:427 +0x68
        cmd/go/internal/work.InstallPackages({0x781038, 0xb12760}, {0xc0000200e0, 0x1, 0x1}, {0xc00031a048, 0x1, 0x1})
        	/home/murp/git/go/src/cmd/go/internal/work/build.go:794 +0x5b8
        cmd/go/internal/work.runInstall({0x781038, 0xb12760}, 0xc000024480?, {0xc0000200e0?, 0x1, 0x1})
        	/home/murp/git/go/src/cmd/go/internal/work/build.go:722 +0x2e0
        main.invoke(0xad1b20, {0xc0000200b0, 0x4, 0x4})
        	/home/murp/git/go/src/cmd/go/main.go:264 +0x57c
        main.main()
        	/home/murp/git/go/src/cmd/go/main.go:182 +0x808
FAIL
exit status 1
FAIL	cmd/cgo/internal/testshared	1.270s

cmd/go crashes because it assumes .go_export section is present within the shared library built by gccgo, but it is missing.

I suppose that cmd/go shouldn't crash, but why is the .go_export section missing?  It should always be generated by gccgo.  Is something removing it?

It seems to be getting lost when linking the shared library. It exists in the input archive depBase.a, but is not in the resulting shared library, when the following runs:
/opt/at15.0/bin/gccgo -o $WORK/b004/libtestshared-depBase.so "-Wl,-(" -Wl,--whole-archive /tmp/shared_test3084291572/gopath/pkg/gccgo_linux_ppc64le_fPIC/testshared/depBase.a -Wl,--no-whole-archive "-Wl,-)" -Wl,--build-id=0x38416f4b645851384a4f5a346e564948596539682f38416f4b645851384a4f5a346e56494859653968 -zdefs -shared -nostdlib -lgo -lgcc_s -lgcc -lc -fPIC

What linker are you using?

In the above case GNU ld (GNU Binutils) 2.37.20220122.

Hrm, I suspect the issue is the .go_export section has the exclude flag set.
Looking at the output of gccgo, I see:
.section        .go_export,"ae",@progbits

Poking around the gccgo code, I wonder if the macro TARGET_AIX is not evaluating as expected.

I do see, in gcc/config/rs6000/linux64.h:
#undef	TARGET_AIX
#define	TARGET_AIX TARGET_64BIT

That dates back to 2023 and looks like a problem here.  There is similar code in freebsd64.h.
Perhaps we can change the code in go-backend.cc to test OBJECT_FORMAT_COFF rather than TARGET_AIX.  Want to give that a try?

I posted a couple of patches to gcc to update the usage of TARGET_AIX to TARGET_AIX_OS. This seems to preserve the original intent, while fixing this bug. I also updated the usage in a couple other places.


Find existing sources when creating/updating/upserting case and fill them in properly

I'm curious what you all think we should do here. This is going to cost us a bit as we do a round trip to fetch any existing source by URL before touching a case but this way we get consistent sub documents which was not the case before.
https://mongoosejs.com/docs/subdocs.html
I've tried to use mongoose pre save hooks (https://mongoosejs.com/docs/middleware.html#order) as it felt cleaner at first but there are a few issues with that approach:

There is a circular dependency between the schema and its model when before saving a source we're trying to fetch existing sources.
Subdocuments save hooks are called before top level documents save hooks so when a case is saved the sub sources are already saved so there's no easy way to know if the sub sources were just created or already existed before.

 Design/implement retrieval and storage of raw sources for manual entries


Update jQuery version

Context: Recently, Dependabot issued an alert for docs/xanadu_theme/static/jquery.js and recommended an upgrade to a minimum jQuery version of 3.5.0. This PR addresses the security vulnerability and updates the instructions for building the Sphinx documentation.
Description of the Change:

The static copies of jQuery and Bootstrap in the CSS theme have been removed (these are pulled from a CDN instead).

Benefits: Fixes dependabot security alert
Possible Drawbacks: None
Related GitHub Issues: None. [ch7715]

Flyby comment purely to say that these are my favourite types of PRs 🙌

We should really backport this fix to any other repos that have the same legacy theme

We should really backport this fix to any other repos that have the same legacy theme

Definitely time to move the theme to a central repository :(


Ordering of columns

The example below is a stylised example of the case when one is getting csv files from different source with similar data that needs to be extracted to the same schema - the various sources though have (1) different header names, and (2) different column order.
I have found that Sylvan deals with (1) very easily, but I have not found a way to do (2) easily - am I missing something ?
thanks
PS: I guess I could workaround it by defining a Record class and calling GetRecord() - but
using Sylvan.Data;
using Sylvan.Data.Csv;
using System.Data;

//Note: the end schemas here are the same, the source ones are not
CsvSchema schema1 = new CsvSchema(Schema.Parse("Name,Beauty:int,Charm:double"));
CsvSchema schema2 = new CsvSchema(Schema.Parse("Nome>Name,Bellezza>Beauty:int,Fascino>Charm:double"));
CsvSchema schema3 = new CsvSchema(Schema.Parse("blame>Name,duty>Beauty:int,chasm>Charm:double"));

string data1 =
@"Name,Beauty,Charm
MickeyMouse,2,2
DonaldDuck,1,3.5
Batman,-1,1.1";
string data2 =
@"Fascino,Bellezza,Nome
3.1,1,Topolino
2.3,2,Paperino
5,1,Paperinik";
string data3 =
@"duty,blame,chasm
1,Topo,0.4
2,Pape,0.5
1,nik,0.7";

var reader1 = CsvDataReader.Create(new StringReader(data1), new CsvDataReaderOptions { Schema = schema1});
var reader2 = CsvDataReader.Create(new StringReader(data2), new CsvDataReaderOptions { Schema = schema2});
var reader3 = CsvDataReader.Create(new StringReader(data3), new CsvDataReaderOptions { Schema = schema3});

DataTable dataTable1 = new DataTable();
dataTable1.Load(reader1);

DataTable dataTable2 = new DataTable();
dataTable2.Load(reader2);

DataTable dataTable3 = new DataTable();
dataTable3.Load(reader3);

//breakpoint here and view the 3 tables: the columnNames are the same (as expected),
//but the order is not:
//each table has the columns ordered as in the source "files",
// not as defined in the schema
;

Maybe this should be in Discussions and not Issues: please let me know if so, I'll move it

What is your scenario that requires specific column ordinals?
GUI - but I think it's probably not going to be v important.
Thanks for the other suggestion - but the issue here is that a priori we do not know what weird&wondeful variations of the csv's will come up in the future. So when a new flavour comes up, all we need to receive is the string that describes it (to pass to Schema.Parse()) , without having to write new code.


Hard to find method from list

As a user, when I want to add a method to a submission, I find it hard to parse the existing methods' list from the modal window. See this screenshot for reference:

I propose two possible solutions:

autocomplete, as for tags
alphabetical list

or a mix of the two.

For the moment, I think an alphabetically sorted drop-down is better. Soon, this might change, but it works for now.


Delete button each image

How can I add a delete button on each image? see the screenshot below
http://prntscr.com/oadydo
http://prntscr.com/oadzc7
Thank you

How did you fixed this?


Error using S3 with Digital Ocean space

I'm using the repo https://github.com/rafas/test-attachments with a digital ocean space and the 1.3.1 release, and when i try to upload any image, i got this error:
[1658595602786] ERROR (test-attachments/10228 on Chuya-Sekira): E_CANNOT_READ_FILE: Cannot read file from location
err: {
      "type": "CannotReadFileException",
      "message": "E_CANNOT_READ_FILE: Cannot read file from location \"image_upload_tmp\\cl5y5bn9y0001bcxi3h1y3jqi.jpg\"",
      "stack":
          CannotReadFileException: E_CANNOT_READ_FILE: Cannot read file from location "image_upload_tmp\cl5y5bn9y0001bcxi3h1y3jqi.jpg"
              at Function.invoke (D:\Sandbox\test-attachments\node_modules\@adonisjs\drive\build\src\Exceptions\index.js:30:23)
              [...]
              at UploadsController.store (D:\Sandbox\test-attachments\app\Controllers\Http\UploadsController.ts:22:5)
      "location": "image_upload_tmp\\cl5y5bn9y0001bcxi3h1y3jqi.jpg",
      "original": {
        "type": "NoSuchKey",
        "message": "UnknownError",
        "stack":
            NoSuchKey: UnknownError
                at deserializeAws_restXmlNoSuchKeyResponse (D:\Sandbox\test-attachments\node_modules\@aws-sdk\client-s3\dist-cjs\protocols\Aws_restXml.js:8595:23)
                [...]
                at S3Driver.get (D:\Sandbox\test-attachments\node_modules\@adonisjs\drive-s3\build\src\Drivers\S3.js:114:44)
        "name": "NoSuchKey",
        "$fault": "client",
        "$metadata": {
          "httpStatusCode": 404,
          "attempts": 1,
          "totalRetryDelay": 0
        },
        "Code": "NoSuchKey",
        "BucketName": "altab",
        "RequestId": "tx000000000000058b7198e-0062dc2b51-5242291f-fra1b",
        "HostId": "5242291f-fra1b-fra1-zg02"
      }
    }

It's different from the previous error, the location was
      "location": "image_upload_tmp/image_upload_tmp/cl3p71qps0001dgy8ezl0esv4.png",
and now
      "location": "image_upload_tmp\\cl5y4zxm10001w4xi1wrr6674.jpg",
The file is properly uploaded to the space on DO as you can see here:
<Contents>
<Key>image_upload_tmp/cl5y5bn9y0001bcxi3h1y3jqi.jpg</Key>
<LastModified>2022-07-23T17:09:37.080Z</LastModified>
<ETag>"adabedb0c7ce028fe5915f088c2d3ac7"</ETag>
<Size>579852</Size>
<StorageClass>STANDARD</StorageClass>
<Owner>
<ID>11535310</ID>
<DisplayName>11535310</DisplayName>
</Owner>
<Type>Normal</Type>
</Contents>

The key is different from the location, i don't know if it's normal and if not how can i fix it.
After some research, i modified the join fonction used to create the path on src/index.js
        this.fileName = attributes.fileName ?? '';
        this.path = attributes.path ?? '';
        this.isLocal = !!this.path || !!this.buffer;
        if (attributes.fileName) {
            // this.relativePath = (0, path_1.join)(exports.tempUploadFolder, attributes.fileName);
            this.relativePath = exports.tempUploadFolder + attributes.fileName;
        }
        else if (attributes.name) {
            this.relativePath = (0, path_1.join)(exports.tempUploadFolder, attributes.name);
        }

And it's working properly

Thank you for mentioning this.
There seem to be a double slash in the location: "location": "image_upload_tmp\\cl5y4zxm10001w4xi1wrr6674.jpg"
I think if it is removed, it might fix the problem.

I removed it using another 'join' function
if (attributes.fileName) {
            // this.relativePath = (0, path_1.join)(exports.tempUploadFolder, attributes.fileName);
            this.relativePath = exports.tempUploadFolder + attributes.fileName;
        }

Do you want make a PR with it ?

Do you want make a PR with it ?

Yes, please.

Hey again @ndianabasi ,
It look like the normalize method do not normalize:
Here was the original line
        const originalFileBuffer = this.buffer ?? (await this.getDisk().get(this.relativePath));

And with the recent PR, we use the normalize function from path, like that
        const originalFileBuffer = this.buffer ?? (await this.getDisk().get((0, path_1.normalize)(this.relativePath)));

Digital Ocean spaces use path with / and not \, while the normalize function replace thoses / by a \.
console.log(this.relativePath)
// log: image_upload_tmp\cl6gn9m6t00059gxi8xp2cisq.png
console.log((0, path_1.normalize)(this.relativePath))
// log: image_upload_tmp\cl6gn9m6t00059gxi8xp2cisq.png
console.log(this.relativePath.replace(/\\/g, '/'))
// log: image_upload_tmp/cl6gn9m6t00059gxi8xp2cisq.png

When we use normalize function, the Disk.get do not work
{
"message": "E_CANNOT_READ_FILE: Cannot read file from location \"image_upload_tmp\\cl6gnhjx40003o4xi9un1fhyt.png\""
}

It work properly when i'm normalizing my own way. so it look like we must normalize the path on our own, with .replace(/\\/g, '/')
What do you think about it ?

I will need to review the code and completely eliminate the cause of the double slashes in the first place. It was introduced in one of the PRs I merged recently.

Handling normalisation manually isn't the best at times. I think a code review is the best way forward at this time.


Imagery index

This switches the source of our imagery from ELI to imagery_index.
As followup I'd like to allow iD projects to work with alternate imagery indexes (and preset indexes).  But at least now the bulk of the work is done, to switch from "bundled" imagery (which only changes when iD releases happen) to "fetched" imagery (which can be released on different schedule from iD).
closes #7425

Pretty excited about this part

As per direction of the OSMF board, I've reverted the changes from this PR for now. I'm also locking conversation in this and #7425 until the best medium for discussing a path forward is determined.


not enough RAM

Hello,
I wanted to use your app on Bluemix with my account, but I noticed it failed on deployment.
By changing the amount of RAM provided, I was able to have the app working.
Is it possible to just chnage the value of 256 in the manifest.yml to 512, so that I won't have problems at startup ?
Thanks in advance.

Thanks for bringing this up, it's been fixed now.


Fresh Ubuntu 22.04 installation of Autoware

Clearance has been given to use the 500Gb HDD on The Beast, which currently has windows installed on it.
While doing this, ensure you don't wipe the 2TB NAS!
Purpose of this project is to see if a fresh installation of Ubuntu 22.04 will make installing Autoware easier in terms of resolving dependencies.

Attempted to wipe 500GB drive using terminal commands. Was careful not to wipe 2TB drive.
All partitions have been removed from sdb (the 500GB drive)
But I'm mostly sure that it hasn't been properly wiped, errors appear when using shred command
Linux *Ubuntu 22.04) is still needed to be installed

@adar0024 According to your first screenshot. It seems that there isn't any partition table for /dev/sdb. My guess is that someone probably wiped it off before.
From what I know, Input/Output error usually happens because of hardware or the use of NTFS(cause drivers are unstable according to this.
But I do not think that this is the case right now. I believe this is because there isn't any partition table inside the drive that means that it doesn't have the map of where things are. So, I think that's why it raised that error because you can't delete something that doesn't exist.
I am not 100% sure of how shred works. Just used it a couple of times but not sure why the first one works. The errors raised make sense because shred essentially replacing previous allocated data to scramble it. Because when you delete something you just delete the inode(something like a pointer) and the data is left untouched but the use of shred helps to make sure that the block is actually deleted by setting values to some value and repeating that process make sure that the data is unrecoverable. But in the case where it doesn't have partition table, there's no partition to access. I do not think shred is necessary if you just want to add some extra storage for the beast.
If you want to add more storage to the beast. You can use the gparted or fdisk whichever you prefer. Create a partition table add a partition and just format it with ext4 filesystem(I recommend btrfs but again it requires a little bit more work & its considered a 'new' filesystem because its the youngest out of all. But I like it and I wish I use it more often cuz it has snapshots like Copy-on-write thing :D). Then, just mount that to a directory.
You can make sure that its automatically mounted by modifying the /etc/fstab file and specify it there but be careful if you incorrectly specify things inside fstab or if you someday took out the drive, it (in most cases) failed to boot up but you can fix it easily by going to recovery mode and remove the line but yeah just be careful.
Hope this helps!

Hey Hubert, thanks for your insightful explanation!
I am going with Option 2 as Jason asked me to do in the AWSIM meeting yesterday. I was just wondering which settings in rufus I need to choose below before I burn the Ubuntu into a flash drive

Leaving the options on default should be fine.

Thanks, this is what my USB drive looks like now, should be all good now to plug the drive into the Beast and install today

Went into the same input/output error when trying to erase the disk and then install Ubuntu. It's also stuck on this error message and I can't get out of it

When erasing disk and installing Ubuntu, it gave me an Input/output error. I tried clicking the buttons but it was unresponsive so I'm stuck on this error message and can't get out of it
Maybe we could partition the 2TB NAS drive instead

@hubdwoo has installed Ubuntu 22.04 on the Beast last evening. Can be treated as a new fresh installation. Task is complete

What is the name of this partition?

@dylan-gonzalez the device name?


Retry Policy is not propagated on continue as new

Describe the bug
When starting a workflow with a RetryPolicy, if/when that workflow does a continue as new, the new workflow that starts does NOT carry across the RetryPolicy from its previous run.
To Reproduce
Yes
Steps to reproduce the behavior:

Start a workflow with a RetryPolicy
Have that workflow ContinueAsNew
Observe that the new workflow does not have a RetryPolicy

Expected behavior
I would expect the new workflow to inherit its previous workflow's RetryPolicy
Screenshots
N/A
Additional context
N/A

I think the bug is at client side, the RetryPolicy field is not set when creating the continueAsNew decision. https://github.com/uber-go/cadence-client/blob/master/internal/internal_task_handlers.go#L1491


feat: SDA-1062: warn users before closing running app

Description

On macOS, currently, when trying to install new version of SDA, we simply kill the app before asking user if they'd like to quit and proceed.
This fixes the issue and provides a better user experience.
SDA-1062

Solution Approach

Add bundleIdentifier as part of packages to check if Symphony is running.
Remove the killall Symphony command from preinstall script.

Related PRs
N/A

@VishwasShashidhar Does installer automatically prompt user of running instance?

@VishwasShashidhar Does installer automatically prompt user of running instance?

Yes @KiranNiranjan, correct. Based on the bundle identifier.


add wishlist sidebar

see in designs: http://sphereio.github.io/sphere-sunrise-design/pop.html

Related to https://github.com/commercetools/commercetools-sunrise-java/issues/334


Add other platforms

Currently only youtube is supported, users should be able to transcribe from the most popular podcasting platforms.
Current Scope:

Spotify (https://github.com/spotDL/spotify-downloader)
ApplePodcast (not sure if possible yet)
Soundcloud

by using librespot https://github.com/kokarare1212/librespot-python/
we can cache spotify episodes and other audio tracks as .wav files from the raw byte stream.
This should be enough to transcribe episodes with whisper.
Will investigate further tomorrow

Update: The functionality above has been implemented in
https://github.com/benearnthof/podcasty/commit/76ad9382f1d45ece70daacc538db464b8779a8dc
podcasty can now download tracks and podcast episodes in bulk from spotify when given a list of episode urls.
We will add functionality for further bulk processing (downloading all episodes of a podcast, or downloading from playlists) in the future, currently the focus is on extracting text data.

For future Reference:
Soundcloud : https://github.com/Suyash458/soundcloud-dl/blob/dev/downloader/downloader.py
Apple Podcast will not be supported for now.


Get rid of webp requirement in info file

WebP image format is supported by core without contrib extension
extra patches that could be applied to core

https://www.drupal.org/project/drupal/issues/3320689 (configure quality)
https://www.drupal.org/project/drupal/issues/3406267 (default conversions)
https://www.drupal.org/project/drupal/issues/3202016 (avif)

Here is PR https://github.com/skilld-labs/druxxy/pull/53

Avif is next


[MINOR] Clean up several build warnings, mostly due to internal use of old accumulators

What changes were proposed in this pull request?
Another PR to clean up recent build warnings. This particularly cleans up several instances of the old accumulator API usage in tests that are straightforward to update. I think this qualifies as "minor".
How was this patch tested?
Jenkins

Test build #60404 has started for PR 13642 at commit c8850d1.

Test build #60404 has finished for PR 13642 at commit c8850d1.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
   class SetAccumulator[T] extends AccumulatorV2[T, HashSet[T]]
   case class ColumnMetrics()

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/60404/
Test PASSed.

maybe we should add a new test for old accumulators, to keep backward compatibility?

Test build #60480 has started for PR 13642 at commit 20c2a6b.

Test build #60480 has finished for PR 13642 at commit 20c2a6b.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/60480/
Test PASSed.

thanks, merging to master!
@srowen , this PR conflicts with branch 2.0, can you send a new PR against 2.0? thanks!

I just cherry-picked this commit to 2.0 and resolved conflicts.


Deploy the website

@lgierth ipfs.io is not on DNSSimple but on Digital Ocean, this means I can't deploy it myself and that the shiny new make publish-to-domain won't work too.
Can you move it to DNSSimple?

Oh yeah good you remind me, that move is largely prepared already!

Site is deployed now -- ipfs.io DNS migration will take a few more days while I sort out issues with support.


in current HEAD  of 0.10-SHAPSHOT  WSS doesn't seem to work.

On the server side I'm getting the following in the logs:
ERROR [2017-01-26 22:54:49,842] io.moquette.server.netty.NettyMQTTHandler: An unexpected exception was caught while processing MQTT message. Closing Netty channel. MqttClientId = null, cause = null, errorMessage = io.netty.handler.codec.http.HttpObjectAggregator$AggregatedFullHttpRequest cannot be cast to io.moquette.parser.proto.messages.AbstractMessage.
I've dug through the handler pipeline.
I'm seeing  the BytesMetricsHandler   run for that port, but I'm not seeing the MQTTDecoder  run.
IN looking more closely at the pipeline setup vs the WS  pipeline setup, I see that the WSS  has:
                pipeline.addLast("httpEncoder", new HttpResponseEncoder());
                pipeline.addLast("httpDecoder", new HttpRequestDecoder());

At the front while the WS does not.
What is not clear to me is why the BytesMetricsHandler would run (specifically channelRead()  ), but then the following element in the pipeline would not.
I will endeavor to keep debugging.

silly me, didn't notice the BytesMetricsHandler is added first, so of course it will be called.

OK!  The problem is that my request to initiate a WSS connection did not have the hardcoded path /mqtt   as part of the URI.
Previous version of Moquette didn't have this requirement apparently.
It would be a good feature to make this an optional configuration element. Right now it's hardcoded:
new WebSocketServerProtocolHandler("/mqtt", MQTT_SUBPROTOCOL_CSV_LIST));
this problem would also exist within WS  protocol.


Wizard: include default option to profile in docker container

Tested using a clean Lubuntu OS and only necessary software installed (docker, python-tk, since tkinter was missing in the shipped python installation).


Plugin not working for me @Spigot 1.8.8

I was happy to find this plugin, and decided to put it on my Server, a work-in-progress Bungeecord-Network with Cloudsystem. It didn't seem to work though, so I checked the logs.
[04:20:49] [Server thread/INFO]: [eZProtector] Enabling eZProtector v4.1.6
[04:20:49] [Server thread/WARN]: [eZProtector] Your server version is not compatible with eZProtector.
[04:20:49] [Server thread/INFO]: [eZProtector] Disabling eZProtector v4.1.6
I tried downloading a fresh copy, and running it on a spigot 1.8.8 testserver on my pc, same result.
How can this be? I thought it is compatible with a "Spigot server running on any 1.7, 1.8, 1.9, 1.10, or 1.11 versions."

This issue has been completely removed in the newest versions of eZProtector, you should update.


Apply casting to embedded objects when .seting

    BlogPost.create({
        title: 'Woot'
      , comments: [
            { _creator: user1, content: 'Woot woot' }
          , { _creator: user2, content: 'Wha wha' }
        ]
    }, function (err, post) {})

user1 is not being casted to an _id because we're not giving special treatment to DocumentArrays in Document#set.

Similarly I am doing this and startTimes aren't getting cast as Dates.
Track.update({_id: trackId), {$set:{
   startTimes: {
          EST: '2011/10/18 06:00',
          EDT: '2011/10/18 06:00',
          CST: '2011/10/18 06:00',
          CDT: '2011/10/18 06:00',
          AKST: '2011/10/18 09:00',
          AKDT: '2011/10/18 09:00',
          HAST: '2011/10/18 11:00',
          HADT: '2011/10/18 11:00',
          PST: '2011/10/18 09:00',
          PDT: '2011/10/18 09:00',
          MDT: '2011/10/18 07:00',
          MST: '2011/10/13 07:00'
   },
   title: 'Dancing With the Stars',
})

This sounds like an easy problem to solve maybe. Any suggestions on where to start?


Security update: upgrade RubyGems to ensure >= 2.6.13

@krallin : The newer (patched) version of RubyGems hasn't yet been included in the official Ruby releases, so I'm just gem update --system and checking for the required version in Bats tests. Please let me know if you want me to make any changes!

Looks great, thanks @fancyremarker !


Blueprints

Implement blueprints so the project is more organized.

Won't do that in the future due to the apps simplicity.


UTF-8 with BOM file case a crash

Component
Forge
Have you ensured that all of these are up to date?

[x] Foundry
[X] Foundryup

What version of Foundry are you on?
forge 0.2.0 (a5171a4 2023-12-24T00:17:25.574824499Z)
What command(s) is the bug in?
forge build
Operating System
Windows
Describe the bug
When try to compile file that have the encode `UTF-8 with BOM' will crash (btw i use WSL2)
forge build
[⠒] Compiling...
[⠢] Compiling 1 files with 0.8.23
[⠆] Solc 0.8.23 finished in 5.89ms
The application panicked (crashed).
Message:  byte index 5 is not a char boundary; it is inside '\u{feff}' (bytes 4..7) of `1 | // SPDX-License-Identifier: GPL-3.0`
Location: /home/runner/.cargo/registry/src/index.crates.io-6f17d22bba15001f/foundry-compilers-0.1.1/src/artifacts/mod.rs:2085

The example file that case the problem: (json ext so i can upload it here)
Test.json

why do you want to use the BOM for your solidity files?

why do you want to use the BOM for your solidity files?

For some reason vs code was forcing me to use BOM. i want use BOM for sure, but its a bug!
maybe you guys can add a WARN or supporting BOM.

This should be fixed in the latest few releases @CorrM can you please confirm

This should be fixed in the latest few releases @CorrM can you please confirm

Version: forge 0.2.0 (05d6062 2024-01-10T00:16:31.110671064Z)
forge build
[⠢] Compiling...
[⠰] Compiling 25 files with 0.8.23
[⠔] Solc 0.8.23 finished in 56.67ms
Error:
Compiler run failed:
Error (7858): Expected pragma, import directive or contract/interface/library/struct/enum/constant/function/error definition.
 --> src/Test.sol:1:1:
  |
1 | // SPDX-License-Identifier: GPL-3.0
  | ^

its not panic, but throw error. Showing error that describe the problem are about the file encoding is better.

We pass files as is to solc, so we can't do much about it. You might want to open an issue in the solc repository.


Misspelling Message, work "occured" should be "occurred"

In the file Serenity.Web\Mvc\EndpointExtensions.cs on line 51 the following line of code has a misspelling:
error.Message = "An error occured while processing your request.";
It should be:
error.Message = "An error occurred while processing your request.";

Thanks, a6a025019fdebb36565db6e8ff80e185ccb60f2c


Can't open url with bootstrap_k8s_gateway_addr

When i set bootstrap_k8s_gateway_addr on router, I can't open internel server
# /etc/dnsmasq.d/99-k8s-gateway-forward.conf
server=/${bootstrap_cloudflare_domain}/${bootstrap_k8s_gateway_addr}

But when i use bootstrap_internal_ingress_addr, it's works

Thanks for reporting but I think you should share your experience in https://github.com/onedr0p/flux-cluster-template/discussions/719
DNS is wildly different for everyone :/


SAK-40645 - gradebook, accessibility: Set Score for Empty Cells inval…

…id score error message prevents other actions

@adrianfish It looks like there were a few issues from your first PR on #6414.

The javascript call had the parenthesis at the wrong location so "true" was actually never passed in
The form inputs were disabled by the submit but something needed to re-enable them, I think this was more of the issue than "dontDisable". I removed the code from the performRealSubmit since I don't think we can know at that point whether or not to disable.
IF there was a NumberFormatException this wasn't called either so I added to that catch

Re-opened as #8319 SAK-43847


More smoke test, c++ launcher profiler and more

add profiler to tf-lite launcher
add fps lock functionality to c++ launchers
extend smoke test coverage

@asesorov


[Feature Request] Support for a flag on darkness sources that ignores Devil's Sight

Some users may use darkness sources to represent more than just magical darkness (for example, Hunger of Hadar or Fog Cloud, both of which heavily obscure an area but do not care whether you have Devil's Sight). It would be handy if there could be a flag set on a darkness source like ignoreDevilsSight which would cause Vision 5e to ignore Devil's Sight for the purposes of being able to see through it.
My request is mainly just for the checking & consideration of the flag, though adding a checkbox to the light's configuration window would also be useful. (Though my personal use-case wouldn't require that, as I'd be creating the darkness sources programmatically).
Thanks for your consideration.

We still have light/darkness source priority as potential V13 feature, I think. I won't consider this feature request before I know this definitely won't be a V13 feature.


FancyZones: When decreasing the number of zones windows in the last zone get placed in the first zone.

Environment
Windows build number: 19041.172
PowerToys version: Master/v0.16.0
PowerToy module for which you are reporting the bug (if applicable): FancyZones

Steps to reproduce
Add a window to zone n, decrease the number of zones to n-1.
Expected behavior
The window should keep it's size and position.
(Ideally the window would return to it's size and position from before it was added to a zone but this is an active feature request).
Actual behavior
The window is sized and placed so as to be in zone 1. Note, when increasing the number of zones again the window is readded to the zone it was last in (if the appropriate setting is enabled).
I believe this error is caused by the function ZoneSet::MoveWindowIntoZoneByIndex in ZoneSet.cpp on line 229:
ZoneSet::MoveWindowIntoZoneByIndex(HWND window, HWND windowZone, int index) noexcept
{
    if (m_zones.empty())
    {
        return;
    }

    if (index >= int(m_zones.size()))
    {
        index = 0; // <-- Here
    }

    while (auto zoneDrop = ZoneFromWindow(window))
    {
        zoneDrop->RemoveWindowFromZone(window, !IsZoomed(window));
    }

    if (auto zone = m_zones.at(index))
    {
        zone->AddWindowToZone(window, windowZone, false);
    }
}

index should not be set to 0 in the second if block, instead the method should just return.

Hi @JordanAnthonyKing
thank your for reporting and debugging this.
I agree that returning instead of setting the index to zero is the right fix.
If you are willing to open a PR, it would be great.
Thank you.

I am happy to submit a PR.
What is the project's standard process for adding TODO comments? Ideally we would not simply be returning but calling whatever code comes out of https://github.com/microsoft/PowerToys/issues/166 (thought it seems unlikely that this will be completed any time soon).

The chances that https://github.com/microsoft/PowerToys/issues/166 will be implemented are very low if not zero, because it cannot be implemented for moving a window out of a zone using the mouse (the way Windows Snap does it, is at system level, but PowerToys is just an app and it cannot do the same thing).
So I would suggest to not add any TODO for this fix.

Fixed in v.0.17 https://github.com/microsoft/PowerToys/releases


Update installation page

What type of PR is this:

/kind documentation
What does this PR do / why we need it:
Updates the installation page with the following:

Reorganizes some wording with regards to where changes of the release
are, architectures, etc.
Moves the IDE portion into a separate section so it's better to
navigate via the sidebar
Moves the "building from source" to the CLI Binary part as it relates
to CLI rather than IDE

EDIT:
More updates!

Changes to the installation page to favour tabbed installation pages instead of the script that we previously used
Steps to verify the downloaded binary with the SHA-256 sum
Removed section from Windows with regards to setting PATH as it is forever changing. Recent Windows 10 update has changed the location. Windows 11 is different. Windows 7/8 is different too. At this point it is implied that the user already knows how / can google it.

There are now different suggested ways to install binaries from k8s projects:

For Linux it is recommended to use sudo install and suggest a way to use the binary on non-root-access machines.
For MacOS the normal reccomendation of copying to /usr/bin/ is still reccomended.
For Windows I have suggested using powershell to install

Which issue(s) this PR fixes:

N/A
PR acceptance criteria:

[X] Unit test

[X] Integration test

[X] Documentation

How to test changes / Special notes to the reviewer:
N/A
Signed-off-by: Charlie Drage<EMAIL_ADDRESS>

More updates!

Changes to the installation page to favour tabbed installation pages instead of the script that we previously used
Steps to verify the downloaded binary with the SHA-256 sum
Removed section from Windows with regards to setting PATH as it is forever changing. Recent Windows 10 update has changed the location. Windows 11 is different. Windows 7/8 is different too. At this point it is implied that the user already knows how / can google it.

There are now different suggested ways to install binaries from k8s projects:

For Linux it is recommended to use sudo install and suggest a way to use the binary on non-root-access machines.
For MacOS the normal reccomendation of copying to /usr/bin/ is still reccomended.
For Windows I have suggested using powershell to install

Ready for another review @dharmit @kadel @feloy

/approec


Preprocess data

This should be mostly good to go. All names snake_case and lower cased. Pulled in region data from previous years to back fill years where it was missing.
Columns:

Gross Domestic Product (GDP) = 'gdp_per_capita'
Life Expectancy = 'health_life_expectancy'
Generosity = 'generosity'
Social Support = 'family'
Freedom = 'freedom'
Corruption Levels = 'perceptions_of_corruption'

happiness_rank and region are already in! We should be all set.

happiness_rank and region are already in! We should be all set.


A client-side exception for Autocomplete in Next.Js production

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Current behavior 😯
When I use the following simple code in my Next.Js app it works fine in localhost. but in production version it crashes the page.
              <Autocomplete                 disablePortal                 id="combo-box-demo"                 options={top100Films}                 sx={{ width: 300 }}                 renderInput={(params) => (                   <TextField {...params} label="Movie" />                 )}               />
I am getting the following error
Error: Minified React error #31; visit https://reactjs.org/docs/error-decoder.html?invariant=31&args[]=object%20with%20keys%20%7Bstart%2C%20end%7D for the full message or use the non-minified dev environment for full errors and additional helpful warnings
If I remove this Autocomplete then the page works fine both in localhost and production.
I am using the following packages
"@emotion/cache": "11.4.0",
"@emotion/react": "11.4.1",
"@emotion/server": "11.4.0",
"@emotion/styled": "11.3.0",
"@mui/icons-material": "5.0.3",
"@mui/material": "5.0.3",
"@mui/styles": "5.0.1",
Expected behavior 🤔
It shouldn't crash page saying 'Application error: a client-side exception has occurred (see the browser console for more information)' in my Next.Js application production.
Steps to reproduce 🕹
https://trackem.vercel.app/
Context 🔦
Autocomplete is causing Next.Js page to crash in production mode (build version).
Your environment 🌎

System
System:
    OS: macOS 12.0.1
  Binaries:
    Node: 17.0.1 - /opt/homebrew/bin/node
    Yarn: 1.22.17 - /opt/homebrew/bin/yarn
    npm: 8.1.0 - /opt/homebrew/bin/npm
  Browsers:
    Chrome: 96.0.4664.93
    Edge: Not Found
    Firefox: Not Found
    Safari: 15.1
  npmPackages:
    @emotion/react: 11.4.1 => 11.4.1
    @emotion/styled: 11.3.0 => 11.3.0
    @mui/core:  5.0.0-alpha.50
    @mui/icons-material: 5.0.3 => 5.0.3
    @mui/material: 5.0.3 => 5.0.3
    @mui/private-theming:  5.0.1
    @mui/styled-engine:  5.0.1
    @mui/styles: 5.0.1 => 5.0.1
    @mui/system:  5.0.3
    @mui/types:  7.0.0
    @mui/utils:  5.0.1
    @types/react: 17.0.27 => 17.0.27
    react: 17.0.2 => 17.0.2
    react-dom: 17.0.2 => 17.0.2
    typescript: 4.4.3 => 4.4.3

Upgrading Next.Js package to 12.0.7 solved the issue.


Issue when trying to parse json

Hi, parser gives me the following error:
std::ifstream jsonStream("adaptivo_ac_weights.json", std::ifstream::binary);
auto model = RTNeural::json_parser::parseJson(jsonStream); <----- breaks here
JUCE v7.0.5
libc++abi: terminating with uncaught exception of type nlohmann::json_v3_11_1::detail::parse_error: [json.exception.parse_error.101] parse error at line 1, column 1: syntax error while parsing value - unexpected end of input; expected '[', '{', or a literal
terminating with uncaught exception of type nlohmann::json_v3_11_1::detail::parse_error: [json.exception.parse_error.101] parse error at line 1, column 1: syntax error while parsing value - unexpected end of input; expected '[', '{', or a literal
(lldb)

My model json was created with your script, it's a common dense / relu / softmax model created with tensorflow and keras.
It says error on line 1, I will past first lines:
{
    "in_shape": [
        null,
        40
    ],
    "layers": [
        {
            "type": "dense",
            "activation": "",
            "shape": [
                null,
                100
            ],
            "weights": [
                [
                    [
                        -0.04106162488460541,
                        -0.008081752806901932,
                        0.05740608274936676,
                        0.0283
 ---- continue ----

Solved, ifstream gives no warning when file is not found, I erroneusly mispelled a character.
Using if(myFile.fail) I was able to properly catch that.
Sorry and thanks for your marvellous work.

Ah yes, that particular exception usually means that the JSON parser was passed an empty stream.
In practice, if you need to be able to dynamically load models at run-time, it's possible to wrap the model loading process in a try/catch block, to avoid a crash in case an invalid or non-existent file is passed in.

Thanks for the comment! Yes you are right. I'm experimenting other problems but I opened a separate issue, thanks for your help


Sorting by empty type

I'm using empty tag components to differentiate two groups of entities with the same components. I'm attempting to sort entities of one type using the tag component, where I will use their other components to decide their sorting order, like so:
struct Empty {};

registry.sort<Empty>([](const entt::entity a, const entt::entity b) {
  ...compare other components...
});

This generates the following error:
cannot convert 'cpool->entt::basic_registry<entt::entity>::pool_handler<Empty>::<anonymous>.entt::storage<entt::entity, Empty>::<anonymous>.entt::basic_storage<entt::entity, Empty, void>::begin()' (type 'entt::basic_storage<entt::entity, Empty, void>::iterator_type' {aka 'entt::basic_storage<entt::entity, Empty, void>::iterator'}) to type 'entt::sparse_set<entt::entity>::iterator_type' {aka 'entt::sparse_set<entt::entity>::iterator'}

Having read through the documentation on ETO, I'm confused as to why this happens. As I understand the docs, this shouldn't be a problem. From the docs:

When an empty type is detected, it's not instantiated in any case. Therefore, only the entities to which it's assigned are made available. All the iterators as well as the get member functions of the registry, the views and the groups will return temporary objects. Similarly, some functions such as try_get or the raw access to the list of components aren't available for this kind of types.

Since I'm only reading the entities and no attempt is made to access the Empty component, why is this code not valid?

If I get the error and the entt::storage::sort() code correctly, then the function does not do what you expect it to do:
    template<typename Compare, typename Sort = std_sort, typename... Args>
    void sort(iterator_type first, iterator_type last, Compare compare, Sort algo = Sort{}, Args &&... args) {
        ENTT_ASSERT(!(last < first));
        ENTT_ASSERT(!(last > end()));

        const auto from = underlying_type::begin() + std::distance(begin(), first);
        const auto to = from + std::distance(first, last);

        const auto apply = [this](const auto lhs, const auto rhs) {
            std::swap(instances[underlying_type::index(lhs)], instances[underlying_type::index(rhs)]);
        };

        if constexpr(std::is_invocable_v<Compare, const object_type &, const object_type &>) {
            underlying_type::arrange(from, to, std::move(apply), [this, compare = std::move(compare)](const auto lhs, const auto rhs) {
                return compare(std::as_const(instances[underlying_type::index(lhs)]), std::as_const(instances[underlying_type::index(rhs)]));
            }, std::move(algo), std::forward<Args>(args)...);
        } else {
            underlying_type::arrange(from, to, std::move(apply), std::move(compare), std::move(algo), std::forward<Args>(args)...);
        }
    }

If I'm not too wrong then the comparison function if given instances of the component type and not the entities themselves.

@skypjack Will using a group solve the problem? The Doxygen documentation mentions a comparison function that accepts entities:
     * @code{.cpp}
     * bool(std::tuple<Component &...>, std::tuple<Component &...>);
     * bool(const Component &..., const Component &...);
     * bool(const Entity, const Entity);
     * @endcode

auto group = registry.group<EmptyType>();

group.sort ([](const entt::entity a, const entt::entity b) {
  ...compare other components...
});

auto group = registry.group<EmptyType>();

group.sort ([](const entt::entity a, const entt::entity b) {
  ...compare other components...
});

Unfortunately this does not compile either. The following assertion within the registry's group method is triggered:
static_assert(sizeof...(Owned) + sizeof...(Get) + sizeof...(Exclude) > 1);

Calling sort on this group also generates a similar compile-time error to the one I mentioned originally.

Well, I didn't test the code - it was more of the idea I had.

@KB9 How do you plan to sort the entities? If you are going to use other components for comparison then you can add them to the group with registry.group<EmptyComponent>(entt::get<Comp...>);. It will prevent static assertion.

I'm from mobile and I've to test it but I'm pretty sure you've found a bug from what I see in the log.
One of the rare corner cases not covered by a test yet. Thanks for pointing this out.
I'll look into it tomorrow probably, in the worst case on Wednesday (I'm a bit busy during these days). If that's what I think, it should be straightforward to solve.
What version of EnTT are you using? master or a tag?
This probably deserves a minor release.
@Innokentiy-Alaytsev maybe you're looking at the wrong sort function? Be aware that storage has a specialization for empty types.

@skypjack Thanks for the reply and clearing this up. I'm using the latest release (v3.2.0).

Bug confirmed. Fixed upstream on branch wip. I've also added a test for this case.
May I ask you to test the patch and confirm that it works as expected?

@skypjack Many thanks for the quick turnaround! Those changes have fixed things on my end.
Any idea when you'll release the latest version? I'm using the latest Conan package so it'd be easiest for me to simply update to a release where this is supported.

@skypjack I've been able to find only one version of the sort function for
storage (
https://github.com/skypjack/entt/blob/831f955b79a77455dadbf27de03d9bade92529c7/src/entt/entity/storage.hpp#L446).
Is it specialised somewhere else? Or is it overriden?
вт, 26 нояб. 2019 г., 3:35 Kavan Bickerstaff<EMAIL_ADDRESS>
@skypjack https://github.com/skypjack Many thanks for the quick
turnaround! Those changes have fixed things on my end.
Any idea when you'll release the latest version? I'm using the latest
Conan package so it'd be easiest for me to simply update to a release where
this is supported.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/skypjack/entt/issues/365?email_source=notifications&email_token=AAGWFK3ASA3CJAL4MNK3PCTQVROLLA5CNFSM4JRL3KGKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEFEFU7A#issuecomment-558389884,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAGWFKYREDIORGY6GHC76IDQVROLLANCNFSM4JRL3KGA
.

@KB9
I'll try to make a minor release this week, even though I'm a bit busy. Patches have a fast path on releases.
@Innokentiy-Alaytsev
That was the bug. The class specialization for empty types didn't have a sort function. Probably I thought that sorting a pool for empty types didn't make sense but it may help actually, especially if used in combination with groups or such.

Created version v3.2.1 - conan package should be released automatically, let me know if you've problems.


Remove Peer upper limit of peers on CL

Removed CC_PEERS_COUNT env and its usage.
Should use defaults of clients instead of forcing limit (often smaller than default).
Was breaking Teku as upper limit was smaller than lower limit.
Types of changes
What types of changes does your code introduce?
Put an x in the boxes that apply

[X] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Update
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Other (please describe):

Testing
Requires testing

[X] Yes
[ ] No

In case you checked yes, did you write tests?

[ ] Yes
[X] No

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

Project coverage is 87.26%. Comparing base (01fcb11) to head (dfd0191).

Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #404   +/-   ##
========================================
  Coverage    87.26%   87.26%
========================================
  Files           91       91
  Lines         4412     4412
========================================
  Hits          3850     3850
  Misses         467      467
  Partials        95       95

Flag
Coverage Δ

unittests
87.26% <ø> (ø)

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

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


[Feature]: Fix animation overflow hidden in achievement cards

Is your feature request related to a problem? Please describe.
In the "Achievements" page, the cards have hover effect in which the card rotates and the achievement are displayed, but during the rotation the corners which are overflowing are not visible.

Describe the solution you'd like
No response
Describe alternatives you've considered
No response
Additional context
No response

can  I work on this @SahilTyagii ?

Yes @pritam0721 I have assigned it to you!

Hello @pritam0721
I saw your solution, great work!
But i noticed that the card is shrinking on hover during rotation (which is fine), but when you take your cursor away during rotation, the card gets back to its original size and its corners are hidden again, will shrinking the card permanently to the size it is while hovering fix the issue?

Also you reduced the animation speed and added a red border, I am assuming that you did this just to demonstrate the changes you made more clearly and you will revert the animation speed and border to how it was previously, right?

I added the red border also the animation speed is slow because  I am testing, and debugging the issue, and the demonstration purpose.
In the CSS class, the card-container takes the full height of the parent element there was no separation between them.
It seems good and one thing (Ps: it's just a suggestion ) I want to add is the animation is too fast I think.

yeah @pritam0721  buddy go ahead

@pradeeptosarkar  can you check it ?

closing as resolved. thanks for contributing @pritam0721


items being assigned wrong names

our test cases for verifying items are receiving names that are consistent with OG Loot contract are currently failing, suggesting they are being assigned names that are not consistent with Loot.
run protostar test /workspaces/realms-contracts/tests/protostar/loot/loot/test_loot.cairo to reproduce failures

Fixed by @starknetdev via: https://github.com/BibliothecaDAO/realms-contracts/commit/be812ed1ddb3bf5f819bae0ff7511356333d8a84


Vitepress translation UI component doesn't appear anywhere using doc-before slot

Hi,
I tried to use the translation helper on the French translation but I encounter several issues by trying it:

the locale on the UI component was inferred from localeIndex from site info, but in my case the value is always root.
by adding it in the vitepress config and into the slot doc-before, nothing appears, when I tried aside-bottom it appears, is doc-before the right slot to use?

Aside those issues, this tool could be something I was so much wanted for helping translation diff, where ryu-cho have limits when there is conflict.
Is there a guide for automatizing the update of translation status?

This page might help:
https://github.com/vuejs-translations/guidelines/blob/main/README_ECOSYSTEM.md
also feel free to checkout Pinia and Vue Router which have practiced it already
Thanks.

Close since the discussion is not active anymore.


Use python3 to run the mnist example in "Get Started"

Since the dependency is python3, the example of mnist needs to run with:
python3 ~/nni/examples/trials/mnist-annotation/mnist.py

fixed


Please add a license

I would like to use your extension in one of my projects, but without a explicit license that is not possible. Please consider using the PostgreSQL License for this.

Thx!


Add .editorconfig

Same as I mentioned in https://github.com/timkurvers/wowser/issues/39, more consistency in coding style for all contributors if they use IDE that supports it.

Covered by 26655eb4b2bd2031f1d745d94c9f2011ac4582c3.

Thanks, am closing this :)


useTransitionStatus - open duration not applied

When using useTransitionStatus with isMounted, the open duration property does not work as expected and is not applied.
close works as intended.
This behaviour ALSO happens if you simply use duration: 300, close works but open does not.
The initial, open statuses ARE being correctly applied so it just seems to be an issue with duration.
To Reproduce
Steps to reproduce the behavior:

Go to https://codesandbox.io/p/sandbox/3xzd63
Click on 'Reference Element'
See it opens immediately, even with a 5000ms 'open' duration set
Click 'Reference Element' button again, see 300ms 'close' duration is correctly applied

Expected behavior
Floating element would take 5000ms before opening.
Screenshots If applicable, add screenshots to help explain your problem.
https://github.com/user-attachments/assets/16afb8c6-96dc-48a6-bf1e-cb44c481a777
Context:

OS: Mac
Browser Chrome
Version<EMAIL_ADDRESS>
Additional context Add any other context about the problem here.

I don't see anything wrong here?  The duration for the open state, unlike close, doesn't act like a delay. The close duration appears to act like a delay because it has to wait for the animation to finish before unmounting (no animation is present in the CodeSandbox). On the other hand, the open one doesn't need to wait for anything and can play the animation immediately.
Also: I'm not sure the open option is even necessary to begin with in the API and can probably be ignored.

I don't see anything wrong here? The duration for the open state, unlike close, doesn't act like a delay. The close duration appears to act like a delay because it has to wait for the animation to finish before unmounting (no animation is present in the CodeSandbox). On the other hand, the open one doesn't need to wait for anything and can play the animation immediately.
Also: I'm not sure the open option is even necessary to begin with in the API and can probably be ignored.

Ahhh ok, yes I definitely misunderstood the reason for the open/close. I have it doing what I want in my real use case. Thanks. Small suggestion I guess then would be to just remove the open option in the API. Thanks for the response.


Github switch

Codecov Report

Merging #4 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master       #4   +/-   ##
=======================================
  Coverage   79.79%   79.79%
=======================================
  Files          78       78
  Lines        9560     9560
  Branches     1218     1218
=======================================
  Hits         7628     7628
  Misses       1876     1876
  Partials       56       56

Continue to review full report at Codecov.

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

All fixed, ready for another review :smile:

All fixed :smiley:

Please rebase and then you can merge.


Please extend the documentation where should I see the page

I followed the steps provided, but unable to find the api page I'm looking for.
I placed my file under: docs/openapi/api.yaml
I tried /docs/api, /nuxt-openapi-docs-module/docs/api, /docs/api/api.yaml, /api-docs/ and many more.
I added the debug flag, I did not see any processing message or similar which would point me to the correct location.

Did you manage to work out what the url is? I'm currently at the same situation. I'ver tried loads of different routes and adde the path param in the config but can't see anything.

I ended up printing out the router and found a link to /docs/{File-Name}/info

Hi,
My solution was to drop this into the bin, and reimplemented it within an
hour by adding swagger dist into scripts and a simple page loading the
swagger file in assets.
Cheers
On 2023. Aug 7., Mon at 22:20 stephen-s1ds @.***> wrote:

I ended up printing out the router and found a link to
/docs/{File-Name}/info
—
Reply to this email directly, view it on GitHub
https://github.com/on-org/nuxt-openapi-docs-module/issues/16#issuecomment-1668518455,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AATYKVCGMS4RBPUK22YZVMTXUFE2LANCNFSM6AAAAAA25DQOXY
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Cserveny, Tamas

You can use Vue Devtools "Routes" section to see new routes.


Not working inside ion-modal-view

that white line is the morph-carousel
when I use the same code but inside ion-view everything works fine

(sorry for bad english)

Well, yeah, you're right and the problem is that I can't get carousels size, case modal is in "virtual state" before opening. I will look into it.

Ok, I think we have soluton here.
In case you're using carousel in modal you will need to update it after opening modal. Use 'identifier' attribute in order to access relevant carousel.
<morph-carousel data-items="items" on-selected="selectedItem" identifier="modal-carousel" show-value="number"></morph-carousel>

You can update it by using helper factory $morphCarousel. Just inject it into controller and use update() method.
$scope.openModal = function() {
  $scope.modal.show();
  $morphCarousel.update('modal-carousel');
};

Thanks, now its working on localhost web browser.
But i tested on iOS 7.1.2 and this happens

Can you please send me code of your application ("www" folder) to my email<EMAIL_ADDRESS>?
I want to take a look.

It seems like the morph-carousel is not rendered properly in the ion-modal-view when you go back and forward to the same page, followed by launching the ion-modal-view.
Possible to help?


Revert "HOSTEDCP-710: Make ImageContentSource immutable"

Reverts openshift/hypershift#2815
This is necessary for the ability to convert from public endpoints (public registries) to private endpoints (private registries) in IBM Cloud

/retest

@bryan-cox please let's put together a list of all the fields that need be immutable but we don't trigger a rolling update on Nodes yet. Then let's documente that limitation. Then let's double check with IBM for each of them and let's make the one which are not disrupting for them immutable until we plan to support each triggering rolling upgrade
/approve

/test e2e-aws

/test e2e-aws


Resolving table/column for UNION queries

@brianv0 I was leveraging the logic you provided in the examples/gather_columns.py script, and agree with your logic for a query of the form,
SELECT foo, b.bar FROM a JOIN b ON a.id = b.id

which results in a table/column resolution of:
SingleColumn(expression=QualifiedNameReference(name=QualifiedName(foo))): [Table(name=QualifiedName(a)), Table(name=QualifiedName(b))]
SingleColumn(expression=QualifiedNameReference(name=QualifiedName(b.bar))): [Table(name=QualifiedName(b))]

i.e., from the query one cannot decipher whether the column foo is from a or b, however for a UNION type query
SELECT foo AS bar FROM a UNION SELECT bar FROM b

the table/column resolution is:
SingleColumn(alias='bar', expression=QualifiedNameReference(name=QualifiedName(foo))): [Table(name=QualifiedName(b)), Table(name=QualifiedName(a))]
SingleColumn(expression=QualifiedNameReference(name=QualifiedName(bar))): [Table(name=QualifiedName(b)), Table(name=QualifiedName(a))]

although it's apparent from the query that the foo column is from table a and the bar column is from table b. Do you know of any resolution to this issue?

The visitor needs to be made more complicated so that only the tables for the query in context are checked. I've tested the following snippet out a bit. Some of the logic from print_column_resolution_order could be put into the visitor, I did some of that myself.
Another note: This doesn't currently handle subqueries, though it could (and probably should). I maybe should work on adding a few other examples and clarifying the scope of each use case.
def check_extracted_columns(query, with_resolution=False):

    class ColumnGroup(object):
        def __init__(self, columns, tables):
            self.columns = columns
            self.tables = tables

        def __repr__(self):
            return repr(self.columns)

        def __str__(self):
            return str(self.columns)

    class TableAndColumnExtractor(DefaultTraversalVisitor):

        def __init__(self):
            self.column_groups = []

        def visit_query_specification(self, node, context):
            columns = []
            maybe_tables = []
            tables = OrderedDict()

            if node.from_:
                if isinstance(node.from_, Join):
                    relation = node.from_
                    maybe_tables.append(relation.right)
                    while isinstance(relation.left, Join):
                        relation = relation.left
                        maybe_tables.append(relation.right)
                    maybe_tables.append(relation.left)
                else:
                    maybe_tables.append(node.from_)
                maybe_tables.reverse()

            # Make it easy to refer to tables
            for table in maybe_tables:
                handle, table = self._handle_and_table(table)
                if handle:
                    tables[handle] = table

            for item in node.select.select_items:
                if isinstance(item, SingleColumn):
                    columns.append(item)

            self.column_groups.append(ColumnGroup(columns, tables))

        def _handle_and_table(self, relation):
            handle = None
            table = None
            if isinstance(relation, AliasedRelation):
                if isinstance(relation.relation, Table):
                    handle = relation.alias
                    table = relation.relation
                else:
                    print("WARNING: Aliased Relation is not a table and "
                          "is omitted")
            else:
                handle = ".".join(relation.name.parts)
                table = relation
            return handle, table

    def print_column_resolution_order(column_group):
        print("\nTable Column Resolution for column group:")
        tables = column_group.tables
        for i in range(len(column_group.columns)):
            column = column_group.columns[i]
            if not isinstance(column.expression, QualifiedNameReference):
                print("Warning: Skipping column at ordinal %d" % i)
                continue
            names = column.expression.name.parts
            resolution = []
            if len(names) > 1:
                qualified_table_name = ".".join(names[:-1])
                if qualified_table_name in tables:
                    resolution.append(tables[qualified_table_name])
            else:
                resolution = [v for v in tables.values()]
            print(repr(column) + ": " + str(resolution))

    print("\n\nChecking query:\n" + query)
    visitor = TableAndColumnExtractor()
    visitor.process(parser.parse(query), None)
    print(visitor.column_groups)
    if with_resolution:
        for column_group in visitor.column_groups:
            print_column_resolution_order(column_group)

Thanks @brianv0 this was super helpful.

I'm going to close this for now. I've created a new issue as a placeholder to help determine if we need an simple API that can perform a canned set of query analysis jobs (like extracting columns) and how that might be implemented.


added logZ to results

Simply adding the marginal likelihood to the results.

Codecov Report

Merging #247 into master will not change coverage.
The diff coverage is 33.33%.

@@           Coverage Diff           @@
##           master     #247   +/-   ##
=======================================
  Coverage   76.05%   76.05%
=======================================
  Files         136      136
  Lines       14649    14649
=======================================
  Hits        11141    11141
  Misses       3508     3508


Feat: Making the logo Resposnive

Describe the feature
When we press the logo it is not redirecting to the web page i will Make it Responsive
Add ScreenShots
N/A
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

@Suchitra-Sahoo please Assign me the issue


Support for multiple non-root TypeScript folders in one project

For example, my recent projects are like this
project/
  pages/
  posts/
  assets/
  web/               // Created by Vue CLI
    vue.config.json  // Read .. for static assets
    package.json
  server/            // My personally created TypeScript project
    src/
      declaration.d.ts
    package.json
  lib/               // Additional libraries I have to create, that may share between web and server, and probably with other projects
    lib1/
      package.json
    lib2/
      package.json
  .gitignore
  config.json        // Or YAML, for text-based config
  package.json       // For easy running of shell scripts, and for forcing Heroku to install Node buildpack
  Procfile
  README.md

In this case, I have problems with TypeScript failing to pickup declaration.d.ts, and Vetur failing to pick up vuetify.
Do I need to create multiple workspaces?

Try placing a tsconfig.json in the 'web' and one in the 'server' folder. You can configure these to pull in the 'lib' folders. https://www.typescriptlang.org/docs/handbook/tsconfig-json.html

@chrmarti My issue is tsconfig.json fails to recognize /project/web/src/declaration.d.ts. I am forced to use /project/web/src/declaration.ts and import "./declaration".

https://stackoverflow.com/questions/48704150/how-to-configure-tsconfig-json-for-typings-not-in-types ?

It works, but VSCode detection is sometimes buggy. I have to close and open again.


Update telling-git-about-your-signing-key.md

Why:

Changing from $(tty) to $TTY as suggested in here by the maintainer of Powerlevel10k fixed the issue for the user who posted the question, which was the same as mine.
Before I made the change I was able to generate my GPG key, but I was getting the following error:
$ git commit -S -m "First signed commit"
error: gpg failed to sign the data
fatal: failed to write commit object

And this was another sign of the problem:
$ echo $GPG_TTY
not a tty

Adding export GPG_TTY=$TTY to my .zshrc fixed the issue. And you can tell by the Verified label.
What's being changed:

From:
$ test -r ~/.bash_profile && echo 'export GPG_TTY=$(tty)' >> ~/.bash_profile
$ echo 'export GPG_TTY=$(tty)' >> ~/.profile

To:
$ test -r ~/.bash_profile && echo 'export GPG_TTY=$TTY' >> ~/.bash_profile
$ echo 'export GPG_TTY=$TTY' >> ~/.profile

Check off the following:

[ ] I have reviewed my changes in staging. (look for the deploy-to-heroku link in your pull request, then click View deployment)
[x] For content changes, I have reviewed the localization checklist
[x] For content changes, I have reviewed the Content style guide for GitHub Docs.

@thecesrom It looks like our new action inadvertently closed this 🙃 I'm so sorry about that! We're merging a fix right now 💛

After it was closed I deleted my fork, maybe I'll try again at another moment.
Thanks for following up on this, @janiceilene.


Extract region function not working

Hi,
I want to draw region when the audio loads in browser. I found one function "extractRegions" for that. It works for the sample audio used in example but not with my audio files. I have to change the minValue for that. But it is different for different audio files.
Let me know if any other changes needed to work extract region function with all audio files.
Thanks

Try tweaking both minValue and minSeconds: https://github.com/katspaugh/wavesurfer.js/blob/51e2b3db0cd9b3ed1ab070c4890ad986929c570f/example/annotation/app.js#L140-L141
minSeconds is the minimum length of the region.

Thanks for the reply.
I tried changing both the values. But as I said it doesn't work for all the audio files. The default value works for the sample audio in sample example but not with my audio. If I change it then it worked for my audio but not for sample audio.
I think there should be some calculation so that we can make it work for all the audio.
Thanks again for the response.


license wizard improvements

license wizard not promoted, recognized as an option.

the questions of the license wizard are not necessarily recognized as an interactable filter.

after navigating the questions the user might want to see a summary of the choices that are acting as filter for the selectable licenses.


vacuum-go: 0.12.1 -> 0.14.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/daveshanley/vacuum/releases.
meta.description for vacuum-go is: The world's fastest OpenAPI & Swagger linter
meta.homepage for vacuum-go is: https://quobix.com/vacuum
meta.changelog for vacuum-go is: https://github.com/daveshanley/vacuum/releases/tag/v0.14.1
Updates performed

Golang update

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.14.1 with grep in /nix/store/qyqiivjxjjvmri4qxlv1nxlawaiwvfzd-vacuum-go-0.14.1
found 0.14.1 in filename of file in /nix/store/qyqiivjxjjvmri4qxlv1nxlawaiwvfzd-vacuum-go-0.14.1

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

vacuum-go

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/qyqiivjxjjvmri4qxlv1nxlawaiwvfzd-vacuum-go-0.14.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

Or:
nix build github:r-ryantm/nixpkgs/74273461f7eb3dbf5cc26feeeef82e01e0eab7cc#vacuum-go

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/qyqiivjxjjvmri4qxlv1nxlawaiwvfzd-vacuum-go-0.14.1
ls -la /nix/store/qyqiivjxjjvmri4qxlv1nxlawaiwvfzd-vacuum-go-0.14.1/bin

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

x86_64-linux

:white_check_mark: 1 package built:

vacuum-go

Maintainer pings
cc @konradmalik for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

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

@NixOS/nixpkgs-merge-bot merge


"Waiting for card transaction" message does not appear

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number:
Reproducible in staging?: needs reproduction
Reproducible in production?: needs reproduction
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @joaniew
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1705941921936509
Action Performed:
Precondition: Have an expensify card transaction

Open the ND app
Open the expense details

Expected Result:
Detail should display card transaction message similar to hybrid app
Actual Result:
Error message does not appear if logged in to ND app directly, it does when opening through hybrid app
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[x] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01ccb7dd396464613c
Upwork Job ID:<PHONE_NUMBER>079178240
Last Price Increase: 2024-01-23

@cead22 is this primarily waiting on you?

This issue is very incomplete and it's unclear which screenshot is for which app. It also seems like the violation is not showing in the money request preview which is in the works at the moment. Given that, I'm going to close this


Cannot see services

Hi, I am trying to setup Balena Sound on my Raspberry Pi 3 but I cannot see any services in the dashboard.
I selected "production" when setting it up and also my wifi network password is correct, the device shows as "online" and I can see logs and SSH into it but I cannot see any services.
Thanks

nevermind, i did it wrong, sorry


Must add  -TypeHandlerVersion paramater for Set-AzVMExtension

Must add the -TypeHandlerVersion paramater for this script to work:
Set-AzVMExtension -Name AMAWindows -ExtensionType AzureMonitorWindowsAgent -Publisher Microsoft.Azure.Monitor -ResourceGroupName  -VMName  -Location CanadaCentral -TypeHandlerVersion 1.0

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

ID: 8201f331-0fa7-a801-9f4d-75d65cd77625
Version Independent ID: 86a66791-11bb-4f20-f77d-42fe58f5dcf4
Content: Install the Azure Monitor agent - Azure Monitor
Content Source: articles/azure-monitor/agents/azure-monitor-agent-install.md
Service: azure-monitor
Sub-service: agents
GitHub Login: @bwren
Microsoft Alias: bwren

@yvesgermain Thanks for your comment. We will review the issue and get back to you shortly.

@yvesgermain Thanks for your feedback. I have made necessary changes to the document and they will be live soon.
We will close the issue for now. If there are further questions , please revert and we will be glad to assist you.


[BUG] console_scripts entry_points are not extracted

Describe the bug
A clear and concise description of what the bug is.
console_scripts entry_points cannot be extracted when not formatted as a dictionary.
The format which fails:
entry_points="""
    [console_scripts]
    awslimitchecker = awslimitchecker.runner:console_entry_point
    """

To Reproduce
Steps to reproduce the behavior:

Create a recipe for a formula that has entry_points formatted as above.
 entry_points is not extracted

Expected behavior
A clear and concise description of what you expected to happen.
The build section should have an entry_points section:
build:
  entry_points:
  - awslimitchecker = awslimitchecker.runner:console_entry_point

Outputs
If applicable, add the output to help explain your problem.
Current output:
{% set name = "awslimitchecker" %}
{% set version = "10.0.0" %}

package:
  name: {{ name|lower }}
  version: {{ version }}

source:
  url: https://pypi.io/packages/source/{{ name[0] }}/{{ name }}/awslimitchecker-{{ version }}.tar.gz
  sha256: 7610ac0fbe0817754e238099053a4ff4d3ea5260676094af86331891d6934a09

build:
  number: 0
  noarch: python
  script: {{ PYTHON }} -m pip install . -vv

requirements:
  host:
    - pip
    - python
  run:
    - boto3 >=1.9.175
    - botocore
    - python
    - python-dateutil
    - pytz
    - termcolor >=1.1.0
    - urllib3
    - versionfinder >=0.1.1

test:
  imports:
    - awslimitchecker
    - awslimitchecker.alerts
  commands:
    - pip check
  requires:
    - pip

about:
  home: https://github.com/jantman/awslimitchecker
  summary: A script and python module to check your AWS service limits and usage, and warn when usage approaches limits.
  license: AGPL-3.0
  license_file: LICENSE

extra:
  recipe-maintainers:
    - ADD_YOUR_GITHUB_ID_HERE

Environment:
Not dependent on environment
Additional context
Add any other context about the problem here.
This has been verified with two packages: https://github.com/conda-forge/staged-recipes/pull/13849 (Original entry_points) and https://github.com/conda-forge/staged-recipes/pull/13848 (Original entry_points)
Can also be reproduced by entering awslimitchecker into https://www.marcelotrevisani.com/grayskull
cc @ocefpaf and @lsetiawan

Just noticed, that this is a duplicate of #186 Should I close this and add the info in a comment there @marcelotrevisani?

Yes please, just copy your description to the other issue. So everything related to this can be tracked on that one

Of course, will do!

Thank you very much! :)

Happy to help :)


How to "share a vehicle configuration with Tesla"?

Great project!
The README is pretty self-explanatory, but step 6 is a bit vague:
https://github.com/teslamotors/fleet-telemetry/blob/2cb04c30d25fcf643ed34dc96f4627dcee588c7a/README.md?plain=1#L114-L127
How can this config be shared? Is there some site where interested service providers (and hopefully individuals?) can sign up/upload the config?

Great question @Niek. More info to come on this soon.

@aaronpkahn Creator of Tessie here - this looks incredible and is super exciting. Thank you for supporting this and all of the work you and your team do. ❤

The doors this will open up are truly endless. Hoping other manufacturers follow suit. Can you talk through the limitations (if any) when it comes to use-cases for this data access?
Or is the intention behind the open concept that if the user (owner) deems the 3rd party has value and decides to share there's no limitations?

@aaronpkahn can i put a vote in for an API endpoint using current auth used by app to set this up?

This is a really great development!
I'm also very interested in this, specifically how to become authorised to receive someone's vehicle telemetry. We're offering a smart charging service to allow people to plan their charging to occur especially when curtailment is happening on the grid ( Stekker.app ). We want to scope down the information we have access to, to a very small subset of what is being offered here, (e.g. only charging related data).
(on a nitpicking side-note: the file name SECURTIY.md has a typo)

@Niek any luck with this?
No, we're waiting on @aaronpkahn to provide details. I guess Tesla will launch this soon - maybe in tanden with the Q2 earnings call today?

I also wanted clarification on the "ca" field in the client config. It would seem that if Tesla CA is issuing certificates to the device and the server, the CA cert would not need to be provided in a production scenario.

Whoever provides the CA (you or Tesla) would also have to create the client and server certs. With the scenario of the vehicle sending data to your prod env I would prefer to manage those certs.

@tonychill thanks for reply. Yes, the same CA would need to issue both the certs -- and better it be in-house. Thanks for clarifying that. I was mistaken regarding the workflow earlier.

I'm curious if this will also provide some of the remote access to things like locks and air conditioning? Fleet managers would be able to provide their drivers with some of the remote access that native tesla owners have like turning on the AC before starting a trip.

@aaronpkahn any updates on this?

Not yet @beaylott.  We're currently in an active development and trial phase, and will communicate more when this phase completes.

@aaronpkahn any way to sign up for the trial, we have quite a few customers who are interested in this?

@tlbdk not quite yet. We need to first complete some infrastructure development to reduce the internal cost of onboarding fleets.

Any updates on this?

https://www.tesla.com/en_gb/fleet#contact
I assume this is the way to register. As far as I can see, this is mostly useful for larger companies.

Registration is live now along with documentation https://developer.tesla.com/docs/fleet-api

We have setup the new fleet-api, we are getting vehicle data via REST but still not sure how can we share this file with Tesla.
There is a section about registering partner account, is this how we should share this info with Tesla? There is no mention of telemetry on that api doc and I'm not sure how to move forward with streaming data

+1 release the kraken to stream data to our telemetry servers :-)

We are also confused about this. Any updates?

Please add WIP or update your API. Atm, we just lost time here.

@aaronpkahn 👋 We started a small driving journal just a few weeks before the announcement that version 2023.38+ would no longer return any location data. We quickly implemented this new fleet-telemetry to be able to get location data again and are now stuck with angry users and now here to put our client/vehicle config 😓 Any type of update or information regarding this issue?

@aaronpkahn 👋 We started a small driving journal just a few weeks before the announcement that version 2023.38+ would no longer return any location data. We quickly implemented this new fleet-telemetry to be able to get location data again and are now stuck with angry users and now here to put our client/vehicle config 😓 Any type of update or information regarding this issue?

Are you using fleet-telemetry or the REST API?
If you are using REST, there has been an update https://developer.tesla.com/docs/fleet-api#vehicle_data

Makes a live call to the vehicle. This may return cached data if the vehicle is offline. For vehicles running firmware versions 2023.38+, location_data is required to fetch vehicle location. This will result in a location sharing icon to show on the vehicle UI.

If you are using fleet-telemetry I'm eager to learn how did you share the config with Tesla and the car

@aaronpkahn Regarding Fleet Telemetry, do you have any update on the last step "Create and share a vehicle configuration with Tesla"? How to share (or upload, or POST) the file client_config.json?
Also curious if you got this to work @slashmili?
p.s. really appreciate the direction the Fleet API is taking, this is awesome, huge thanks to the team!

@mittsh nope! Still waiting for more information to put fleet-telemetry in use

FYI there has been an update in Tesla's Developer Doc regarding sharing the config:
https://developer.tesla.com/docs/fleet-api#fleet-telemetry

We have updated instructions and documentation regarding this here

We have updated instructions and documentation regarding this here

Thats great to hear @agbpatro, are Fleet API Support still responding to CSRs over the holiday period? The new API is working in so far as It does not like my self-signed certificate 😄

We have updated instructions and documentation regarding this here

Thats great to hear @agbpatro, are Fleet API Support still responding to CSRs over the holiday period? The new API is working in so far as It does not like my self-signed certificate 😄

Yes we are incrementally ramping up the process for responding to CSR requests


Support basic auth

Any plans to have the SDK support Basic Authentication? I'm interacting with a splunk server setup that for some reason has more reliable results when using that over splunk auth token mechanism and would like to try setting the SDK using basic auth to verify/benchmark etc.
https://github.com/splunk/splunk-sdk-ruby/blob/549526e62c9bb97b215fe3fbd1971cf58ab08646/lib/splunk-sdk-ruby/context.rb#L424
https://github.com/splunk/splunk-sdk-ruby/blob/549526e62c9bb97b215fe3fbd1971cf58ab08646/lib/splunk-sdk-ruby/entity/index.rb#L65

http://docs.splunk.com/Documentation/Splunk/6.4.3/RESTUM/RESTusing
Basic HTTP authentication

Splunkd also supports basic authentication as defined by RFC 1945. This is the mechanism used when you access resources using a Web browser

Wrong search - looks like it is supported. Sorry for the noise!
https://github.com/splunk/splunk-sdk-ruby/blob/master/lib/splunk-sdk-ruby/context.rb#L426-L429


Django 3.0 Support

According to this, elastic-apm supports up to Django 2.2.
Has anyone tested this with Django 3.0?
Is there an ETA on when Django 3.0 will be officially supported (e.g. on the next release)?

Hi @Palisand
We will release version 5.4 soon, which adds official support for Django 3.0. But as far as I remember, we didn't have to implement and Django 3.0 specific fixes. It should just work with the current version.

Thanks @beniwohli. I'll leave this issue open until official support is released.

@Palisand a new version of the agent has now been released.
Note that we didn't investigate ASGI yet, so there might be issues if you run your Django app in ASGI mode.

@beniwohli Currently still running in WSGI. Thank you!

Hi!
I'm running a Django app in ASGI mode. I'd want to integrate it with APM and I found this issue. Are there any plans to officially support Django ASGI apps? Is there any other way to currently integrate them with APM?
Any help would be appreciated. Thanks in advance 😊

@daniel-ruiz We definitely plan to officially support it. It may largely work now, we just haven't investigated yet.


Use try_load_entries_* more systematically

After the previous merge, I realize that there are more places where we can use try_load_entries and reduce the asynchronicity burden.

Based on the failed tests, some of the guards seem to be taken for longer than there were before. (Thankfully, we are failing instead of deadlocking!)

Based on the failed tests, some of the guards seem to be taken for longer than there were before. (Thankfully, we are failing instead of deadlocking!)

Yes, there was a loop over the views that could fail in the middle, that I removed.
After that, there was a wasmer errors, and I rerun it and now it passes.


TypstWatch command has no effect

OS: Windows 10 22H2
Nvim version: v0.9.5
Terminal emulator: Alacritty & Powershell
When using the TypstWatch command, the following line is displayed, but nothing else seems to happen.
Starting: typst  watch --diagnostic-format short 'test.typ' --open

The PDF file does not get updated nor generated if missing, and my PDF viewer does not open.
Running typst watch 'test.typ' --open in the terminal seems to work fine. It's like the command in the editor just isn't getting run for some reason.
I'm not really sure how to troubleshoot this to figure out what my issue is, so any advice would be appreciated.

Digging into this a little bit more: typing typst watch 'test.typ' into a command prompt gives the error as before but typing it into Powershell works. It seems like the command prompt just doesn't like the single quotation marks around the file name.
To confirm this, I removed two of the single quotation marks from this line to the following:
\ . " " . expand('%') . ""

With the above change, the TypstWatch command in the plugin seems to function normally.

Sorry for late response! Check if branch fix-84 fixes the problem.

Don't worry about it. Thanks for looking into a fix.
I can confirm that the fix-84 works on one of my machines. I won't be able to test on the other one until Monday.

Hi @muroc, did you manage to confirm it on windows?

Yes. I have tried it on both the Windows machines that I have access to, and the fix seems to work without issue.

@kaarmu,
Sorry to bug you about this again, but I would like to ask about moving forward. Is there any hope that this fix could be introduced into the main branch, or will this have to remain as a separate branch?
Thanks again for your help on this issue!


Click on element doesn't work properly

You can find details (e.g. test example) in the following private ticket: https://supportcenter.devexpress.com/internal/ticket/details/T930974
Your Environment details:

testcafe version:                   1.9.3
node.js version:                    v12.18.0
command-line arguments:  testcafe chrome test.js --skip-js-errors
browser name and version: Chrome 86.0.4240.75
platform and version:          macOS 10.15.6

the same issue in HH https://github.com/DevExpress/testcafe-hammerhead/issues/2313

fixed in https://github.com/DevExpress/testcafe-hammerhead/issues/2313


Minor documentation improvements.

Typo fix.
Join lines to avoid unwanted vertical whitespace in the output.
Mark up gvcf like the other options.

Christian -- thanks for all these fixes and improvements. Much appreciated.


Add canDrop API

Fixes #52

Any ideas when will this be available as a release? So it can be used by an NPM installation

@danjocha I just published it with 0.1.13. Let me know if you have any feedback!

nextParent, nextPath and nextTreeIndex are always returning same value whatever the case may be.
nextParent: null,
nextPath: [0],
nextTreeIndex: 0


Fix state bucket name in dev scripts.

What
These references to the old state bucket name were missed when the
bucket was renamed in ead368e. Without this, ssh to the concourse
instance fails because it can't find the key in the bucket etc.
How to review
Attempt to ssh to a concourse deployed from this repo. It should now succeed
Who can review
Anyone but myself.

Failed before, succeeds after. yay :)


Support permanent versioned URLs for "current" version

🚀 Feature
Docusaurus should support version-based permalinks for the current version of the documentation.
The current version does not have a versioned link until it becomes a previous version, meaning that it is not possible to provide a permalink to the current version.
E.g. docs/doc1.html points to version 2.0.0 (the current), and docs/1.0.1/doc1.html points to the previous version, but there is no docs/2.0.0/doc1.html until a new version comes around.
Have you read the Contributing Guidelines on issues?
Yes
Motivation
Not being able to provide a permalink to a particular version of the documentation from the start is problematic. The unversioned URLs (e.g. docs/doc1.html) could be pointing to version 2.0.0 now, but in a few months' time could be pointing to version 2.0.1 or 3.0.
We have worked around this with the current version of Docusaurus by introducing a "hidden" version at the top of versions.json, but using defaultVersionShown to pick a later version.
This means that the "current version" has versioned URLs, but it also means that there are no default "unversioned" URLs, which is a bit of a pain. The reason we had to do this is because of related communications that need to refer to the current version (1.0 in our case) specifically, not "any" version.
See https://fdc3.finos.org for the Docusaurus-based website where we used this workaround.
Pitch
The pitch is that for the current version there should be two valid ways to address it: with unversioned URLs, which are the default when browsing to the website, and which change version over time.
But there should also be versioned URLs that work correctly for the current version. Effectively the "unversioned URLs" becomes aliases for the versioned URLs of the current version - they are a "pointer" that moves on to the next version, while the versioned URL always stays fixed.

I was going to raise this request soon. Glad to see someone else opened it already.

@endiliey If I wanted to add a PR for this, would it be better to wait for v2?

cc @yangshun
i dont recommend having new feature like this because we plan to rework on our versioned docs for v2 and it will make it harder to migrate later on

@endiliey shouldn't we keep this open though, or recreate the same issue for v2? Quite a few people have expressed interest in the same feature. Otherwise we may lose track.

File it here https://v2.docusaurus.io/feedback

@rikoe - would you be able to share some details on how you've worked around it in V1?

To follow up, this feature request has been filed with Docusaurus v2.

The feature request says this is "Complete", but without providing any additional comment.
Does someone know how exactly has this been implemented? Is the intended solution to use lastVersion: 'current'?
But if I understand it correctly (and as described in https://github.com/facebookresearch/hydra/pull/2321), that means there won't be a /next/ version URL anymore and instead that version becomes the one shown by default, which might not be desired either if that represents the documentation for the not yet released upcoming version.

@Marcono1234, this issue is for Docusaurus v1. In v2+, the easiest solution is to add a redirect. As mentioned above: https://github.com/facebook/docusaurus/issues/1312#issuecomment-616144462, there's also a feature request for v2 already.

As mentioned above: #1312 (comment), there's also a feature request for v2 already

@Josh-Cena, yes I was referring to that feature request but the state there just says "Complete" without providing any additional information about how this was implemented.
Regarding configuring a redirect, that would have to be done manually though (and updated whenever you release a new version), right? But there is no feature to automatically add a permalink and redirect for the latest version?

@Marcono1234 I don't remember why I marked that feature request as completed. Maybe I thought it was just about adding a permalink metadata 🤷♂
Anyway, the issue to track for v2 is: https://github.com/facebook/docusaurus/issues/9049

But there is no feature to automatically add a permalink and redirect for the current version?

No, but you can easily automate this with a pre or post build script. Generating redirects with code remains achievable.