Support for ConvTransposeNd

Hello, thanks for this super cool project!
I'd really like to try this out, but my network architecture requires a transposed convolution.
Is it possible to support this as well?

Hi,
yes that indeed should be possible. Can you clarify what functionality of transpose convolution you need? Would be happy to offer advice how to implement it; maybe you can even submit a PR.
(I haven't had time to ship many additional operations, but they are all pretty similar. Feel free to check out Table B.2 in the paper for an extended overview)
Best,
Felix

My network has an initial step which extracts patches from the input (like a ViT) which I then need to reverse at the end of the network to get back the same original shape:
import torch

x = torch.randn(16, 3, 32, 32)

conv_in = torch.nn.Conv2d(3, 64, kernel_size=8, stride=8)
conv_out = torch.nn.ConvTranspose2d(64, 3, kernel_size=8, stride=8)

print(x.size())  # torch.Size([16, 3, 32, 32])

h = conv_in(x)

print(h.size())  # torch.Size([16, 64, 4, 4])
# generally there's a bunch more layers in the middle here e.g. a Transformer

y = conv_out(h)

print(y.size())  # torch.Size([16, 3, 32, 32])

The way I understand it, a ConvTranspose op is essentially just the transposed version of the lowered matrix form of the convolution. This makes me think that in an einsum the ConvTranspose would just amount to reversing the order of the input labels, although I'm not sure if my intuition maps that nicely to reality.
Is it clear enough what I'm looking to do?

Hi,
I think you're intuition should be correct in that you can simply exchange the role of spatial input and output dimensions in the corresponding einsum of the convolution.
May I ask why you cannot rely on the built-in transpose convolution?

I'd like my model to be Nd!
Right now I have to maintain 3 different copies of it for different data types, but by leveraging an Nd convolution I'd only have one.

Gotcha,
would you be interested in adding the equivalents of torch.nn.functional.conv_transpose{1,2,3}d and torch.nn.ConvTransposeNd to the library? I can provide the einsum expression.

Yes, I think if you could help with the einsum expression then I can probably figure out how to add in the necessary functions to the library.
Reduced performance is definitely a downside. I'll have to benchmark it and see how big the gap is in practice. I think a minor performance degradation would still be worth it for a fully dimensionality agnostic architecture.

If performance is crucial, I suggest using PyTorch's built in convolution and something like
N = conv_dim
conv_cls = {1: Conv1d, 2: Conv2d, 3: Conv3d}[N]
conv_transpose_cls = {1: ConvTranspose1d, 2: ConvTranspose2d, 3: ConvTranspose3d}[N]

in your code.
I'll keep you updated on the einsum expression for transpose convolution 👍

Quick update: I pulled together the einconv.expressions.conv_transposeNd_forward.einsum_expression on the conv-transpose-expression branch. For now it is tested for N=1 and I will add cases for N=2,3 next.
With that, it should be relatively straightforward to make the following contributions (1. and 2. can be separate, but for each the testing part of 3. would be mandatory 😉)

Add einconv.functionals.conv_transpose.py that contains an equivalent of PyTorch's nn.functional.conv_transpose{1,2,3}d. The code should be very similar to the convolution case in einconv.functionals.conv.py.
Add einconv.modules.conv_transpose.py that contains an equivalent of PyTorch's nn.ConvTranspose{1,2,3}d. The code should be very similar to the convolution case in einconv.modules.conv.py. Also, one needs to use torch.nn.modules.conv._ConvTransposeNd._output_padding to compute the output_padding parameter if the user specifies output_size in forward. See here for an example.
Add to API, add to documentation, add tests, then merge 🚀


Add Vitess API documentation and script for generating documentation. Associated changes to left nav, CSS, .proto file.

This change adds an introduction and client libraries section to the User Guide on vitess.io.

LGTM awesome!

Published.


Edit CITATION.cff for version 2.6

Description

Just preparing the PR that edits the CITATION.cff files to take into account the new upcoming release

Before you get started

Checklist
It is the responsibility of the author to make sure the pull request is ready to review. The icons indicate whether the item will be subject to the 🛠 Technical or 🧪 Scientific review.

[x] 🛠 This pull request has a descriptive title
[x] 🛠 Code is written according to the code quality guidelines
[x] 🛠 Documentation is available
[x] 🛠 Tests run successfully
[x] 🛠 The list of authors is up to date
[x] 🛠 Any changed dependencies have been added or removed correctly
[x] 🛠 All checks below this pull request were successful

To help with the number of pull requests:

🙏 We kindly ask you to review two other open pull requests in this repository

Actually I am not sure if today will be the day because the doc build is failing in https://github.com/ESMValGroup/ESMValTool/pull/2740


Add troubleshooting/help docs

Fixes https://github.com/Chatterino/wiki/issues/1
Fixes https://github.com/Chatterino/wiki/issues/5

Looks good to me 👍 Thank you for your contribution!

305d350 broke the link to the Visual C++ Redistributables by the way (line 9 in Help.md)

305d350 broke the link to the Visual C++ Redistributables by the way (line 9 in Help.md)

I see this, I might need to directly add it into the line.


add macOS ARM64 host on CI

This adds macOS ARM64 as a host here, it has a arch -arm64 wrapper to run it as arm64 instead of rosetta, so that's why I prevent the installation of a new esy as that would mean removing the wrapper. This will be needed until github actions runner support macOS ARM64.

This is ready, the others are failing for unrelated reasons


⚠ Bofrost.at has degraded performance

In 42dfa87, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1339 ms

Resolved: Bofrost.at performance has improved in a37cada after 6 minutes.


Set DATASET_URL default and enable to run out of the box

Developer's Certificate of Origin 1.1
   By making a contribution to this project, I certify that:

   (a) The contribution was created in whole or in part by me and I
       have the right to submit it under the Apache License 2.0; or

   (b) The contribution is based upon previous work that, to the best
       of my knowledge, is covered under an appropriate open source
       license and I have the right under that license to submit that
       work with modifications, whether created in whole or in part
       by me, under the same open source license (unless I am
       permitted to submit under a different license), as indicated
       in the file; or

   (c) The contribution was provided directly to me by some other
       person who certified (a), (b) or (c) and I have not modified
       it.

   (d) I understand and agree that this project and the contribution
       are public and that a record of the contribution (including all
       personal information I submit with it, including my sign-off) is
       maintained indefinitely and may be redistributed consistent with
       this project or the open source license(s) involved.

@lresende @ptitzler Should dataset link be https://dax-cdn.cdn.appdomain.cloud/dax-noaa-weather-data-jfk-airport/1.1.4/noaa-weather-data-jfk-airport.tar.gz ?

Would be curious to understand how hardwiring the data set URL helps, considering that we are positioning load_data as a generic download node for DAX data sets. It doesn't save the user any time or make things simpler

I just want to make the notebook runnable by itself, if the user is not running via a pipeline.


Make httpd optional

Description
Similar to https://github.com/rh-ecosystem-edge/ztp-pipeline-relocatable/pull/646, we'd like to skip the httpd step. The httpd pod does not honor the cluster proxy settings, so when the cluster is behind a proxy, it does not work.
When we omit spec/osImages from the AgentServiceConfig, it will pull in a set of default images from the Internet. The assisted service pod does honor the proxy settings, so it works through a proxy.
@flaper87
Type of change
Please select the appropriate options:

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update
[ ] This change is a documentation update

Testing
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[ ] Test A
[ ] Test B

Test Configuration:

Versions:
Hardware:

Checklist

[ ] I have performed a self-review of my own code
[ ] If a change is adding a feature, it should require a change to the README.md and the review should catch this.
[ ] If the change is a fix, it should have an issue. The review should make sure the comments state the issue (not just the number) and it should use the keywords that will close the issue on merge.
[ ] A change should not be merged unless it passes CI or there is a comment/update saying what testing was passed.
[ ] PRs should not be merged unless positively reviewed.

I want to reiterate that we don't support the hub cluster creation

Yes, I think this should be clarified in the docs. The docs have large sections devoted to how to run the hub pipeline[1]. That should all be removed if it's not supported as part of the solution.
There are some dependencies between the hub and edge pipeline though. For example, they both reference $CLUSTERIMAGESET for the name of the ClusterImageSet, therefore the edge pipeline is expecting a ClusterImageSet with a specific naming convention.
[1]https://rh-ecosystem-edge.github.io/ztp-pipeline-relocatable/1.0/ZTP-for-factories.html#running-the-hub-cluster-factory-install-pipeline_ztp-for-factory-prerequisites


Addressed safety issues up to 2024-06-24

No review needed.

Pull Request Test Coverage Report for Build test-28
Details

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

Totals

Change from base Build test-25:
0.0%

Covered Lines:
128

Relevant Lines:
568

💛  - Coveralls


New filters to include multi-currency support

Hi, currently, the plugin detects the currencies added by WooCommerce multilingual, but there are more multicurrency plugins for WooCommerce.
I added two filters to allow other developers to add more currencies to PayU settings.

@oscargare
Thanks for your suggestion. We made our own change (https://github.com/PayU-EMEA/plugin_woocommerce/commit/3ca485f2e7f448888dcb4b4a336becc200230c59) because we released a new version of the plugin and completed the plugin documentation.


Can't import palettes from .ase palette converter

I've been using your site: http://ase.andrewfiorillo.com/ to convert .ase palettes to sketch palettes to import into the plugin in. After the latest update, it is no longer working. I can use the site to convert the file, but when I import the file to sketch, nothing seems to happen - and there are no document colors added.
I know this site isn't directly tied to the plugin, but it is super useful!

Hello. Glad you're finding it useful! I wasn't sure if anyone was actually using that. Since it was a bit of a hack just to see if I could get it working, and isn't part of the plugin itself, I can't guarantee how soon I'll get to it. But, I'll take a look when I get a chance!

Thank you! Please reply here when you get a chance to take a look.
The bit of a hack has been much appreciated!


Device status information

There seems to be no way to list the currently stored keys on a device, like e.g. the number of stored residential keys.
If possible, this would be quite useful especially to find out of there is still space available  for additional keys.
No idea what happens when a user tries to store more then 50 residential keys on a Nitrokey Fido2 device for example and it would be good to have a way to find out how many can still be stored on a particular device.

In general you can manage RKs through Google Chrome / Chromium browsers.
I agree it would be nice to have it in the Nitrokey provided tool. We have a solution, that needs to be finished yet:

https://github.com/Nitrokey/pynitrokey/pull/216

cc @daringer

nitropy fido2 list-credentials now prints the number of residential keys on the device and the number of available slots.


update notify with fix for hight cpu usage

Should fix the ocasional 100% CPU hangs.
I've tried yes > test.rs which would cause it before, and now on my computer it stays below 10%, and stops as soon as I interrupt the command, unlike previously which would stay at 100% for a while.

bors r+


panic: interface conversion: net.Conn is *tls.Conn, not *net.TCPConn

Connection Hijacking/handover does not work with TLS.

Turns out you can't just copy a tls.Conn because of states and stuff...
A solution would be a connection proxy in a separate process.


Fix jvm lookup logic

When java-18-openjdk-amd64 is deployed ccm wrongly percieve it as
Java-8 and uses it

@fruch , @tchaikov , can you please take a look at it


Star Citizen 4.0 Isdsues

Stream deck Star Citizen  plugin doesnt appear to be working.
Streamdeck running as Admin
SC Launcher Ruinning as admin
Plugin is getting key binds pulled in
but when buttons are pressed on the stream deck the commands are not being recognized by the game

for 4.0_PREVIEW 4.0.0-live.9470730 or HOTFIX 4.0.0-live.9470730 the files are in a different directory.
You need to set this directory yourself in:
%appdata%\Elgato\StreamDeck\Plugins\com.mhwlng.starcitizen.sdPlugin\appsettings.config
and restart streamdeck.exe after changing this file.
See the documentation here on github.
On my pc, they are in these directories:
C:\Program Files\Roberts Space Industries\StarCitizen\LIVE\Data.p4k
C:\Program Files\Roberts Space Industries\StarCitizen\LIVE\user\client\0\Profiles\default
C:\Program Files\Roberts Space Industries\StarCitizen\HOTFIX\Data.p4k
C:\Program Files\Roberts Space Industries\StarCitizen\HOTFIX\user\client\0\Profiles\default
C:\Program Files\Roberts Space Industries\StarCitizen\4.0_PREVIEW\Data.p4k
C:\Program Files\Roberts Space Industries\StarCitizen\4.0_PREVIEW\user\client\0\Profiles\default
use a simple button to test first (e.g. f1=mobiglass)
Also note that functions that don't have a keyboard binding defined in the game configuration screen, don't work.
Also note that keys where there is a modifier other than ctrl/shift/alt don't work. (like F4+arrow for camera controls)
I tested the hotfix version, my pluginlog.log looks like this when I press the mobiglass button:
2024-12-21 11:18:27.6224|INFO|com.mhwlng.starcitizen|1|C:\Program Files\Roberts Space Industries\StarCitizen\HOTFIX\Data.p4k needs update : True
2024-12-21 11:18:27.6224|INFO|com.mhwlng.starcitizen|1|C:\Program Files\Roberts Space Industries\StarCitizen\HOTFIX\Data.p4k
2024-12-21 11:18:45.3438|INFO|com.mhwlng.starcitizen|1|C:\Program Files\Roberts Space Industries\StarCitizen\HOTFIX\USER\Client\0\Profiles\default\actionmaps.xml
2024-12-21 11:18:45.3662|INFO|com.mhwlng.starcitizen|1|Keyboard Detected, language : 409 keyboard : 409 culture : en-US
2024-12-21 11:18:45.4183|INFO|com.mhwlng.starcitizen|1|Keyboard Detected, language : 409 keyboard : 409 culture : en-US
2024-12-21 11:18:45.4183|INFO|com.mhwlng.starcitizen|1|Keyboard Detected, language : 409 keyboard : 409 culture : en-US
2024-12-21 11:18:45.4373|INFO|com.mhwlng.starcitizen|1|monitoring key binding file
2024-12-21 11:18:45.4373|INFO|com.mhwlng.starcitizen|1|Finished Init Star Citizen
2024-12-21 11:18:45.5783|INFO|com.mhwlng.starcitizen|1|Plugin [com.mhwlng.starcitizen] Loading - 3 Actions Found
2024-12-21 11:18:45.6892|DEBUG|com.mhwlng.starcitizen|1|Plugin Loaded: UUID: 77DED2B56424AD2B4D54C6E6AA3484C2 Device Info: Devices:
[Id: 49C5DB3C77BEDDEB4EE9DA88ED76CF59 Type: StreamDeckXL Size: Rows: 4 Columns: 8]
[Id: 8F9F54D3085E17917873D7BA481EA6B7 Type: StreamDeckPlus Size: Rows: 2 Columns: 4]
[Id: A2F05B845C24AD67556305C9EE9CA0DC Type: StreamDeckClassic Size: Rows: 3 Columns: 5]
[Id: 8403C4817A9F3533236625316AE80F1A Type: StreamDeckClassic Size: Rows: 3 Columns: 5]
ApplicationInfo: Language: en Platform: windows Version: <IP_ADDRESS>52
PluginInfo: Version: 1.1.2
2024-12-21 11:18:45.6892|INFO|com.mhwlng.starcitizen|1|Plugin version: 1.1.2
2024-12-21 11:18:45.6892|INFO|com.mhwlng.starcitizen|1|Connecting to Stream Deck...
2024-12-21 11:18:47.7193|INFO|com.mhwlng.starcitizen|1|Connected to Stream Deck
2024-12-21 11:18:47.7193|INFO|com.mhwlng.starcitizen|1|GlobalSettingsManager initialized
2024-12-21 11:19:13.6888|INFO|com.mhwlng.starcitizen|8|{DikF1}
2024-12-21 11:19:13.7499|INFO|com.mhwlng.starcitizen|8|{DikF1}
2024-12-21 11:19:14.8856|INFO|com.mhwlng.starcitizen|8|{DikF1}
2024-12-21 11:19:15.0120|INFO|com.mhwlng.starcitizen|8|{DikF1}

not sure what changed that this is required now, I've been running this plugin pretty  much since it came out,  As I am running as many as 3 different version of SC at any one time, live, ptu tech preview, etc, via streamdeck and its just worked across all versions.    Is there support in path in the xml file for wildcard usage of any type, or will I need go in and modify  these files before I load a different version of the game.   I tiried the standard * instead of   the build name but it didnt seem to have the desired effect.
i.e
c:\Roberts Space Industries\StarCitizen*\Data.p4k

Wildcard will not work you need exact path


Push replication failing with large data sets

Hey,
I'm developing an Android app in Kenya with a user base is that is often rural and doesn't have consistent access to the internet. We need to back up our users' data, and have just started to use couchbase lite for Android. Because of their poor connectivity they often go long periods of time without uploading any data to the server so push replications are frequently pretty huge. We need push to work with any reasonably large set of documents that take up a reasonably large amount of space. Luckily we don't need to sync any images are large files, just documents that are generally pretty small.
I haven't plugged in a real dataset to test yet, and am just testing with one type of document with randomly generated data to make sure I can roll out a scalable solution before we drop that app into real-world conditions. I'm currently testing on an Alcatel One Touch (which is a pretty sub-standard tablet, but it's the kind of thing are users are likely to use, so it's a good test device) with ~65MB worth of documents (25,000 docs at ~2.5kb each). I flush the server before running the tests, so it's purely a push replication.
By the time the ChangeEvent has registered about 24,000 changes (it hasn't started uploading at this point) it starts generating Out of Memory errors.
First of all, I'm not sure why the upload isn't incremental in this case? When I test with smaller data sets the upload starts before the CBLite has registered all of changed items. For example, when I run the push replication with 750 items, the upload starts after getChangeCount() returns 500 items, and that number is smaller (proportionally) with smaller data sets.
Second, is there anything I can do to change this behaviour? I have some hacky solutions in mind to break up the upload, but I don't want to implement them if I don't have to.
Here are the OOM errors I'm getting:
06-09 19:31:20.321 6978-7368/org.mc.mc E/Sync: RemoteRequest.run() exception: %s
java.lang.OutOfMemoryError
      at com.fasterxml.jackson.core.util.BufferRecycler.calloc(BufferRecycler.java:156)
      at com.fasterxml.jackson.core.util.BufferRecycler.allocCharBuffer(BufferRecycler.java:124)
      at com.fasterxml.jackson.core.util.BufferRecycler.allocCharBuffer(BufferRecycler.java:114)
      at com.fasterxml.jackson.core.io.IOContext.allocConcatBuffer(IOContext.java:186)
      at com.fasterxml.jackson.core.json.UTF8JsonGenerator.<init>(UTF8JsonGenerator.java:126)
      at com.fasterxml.jackson.core.JsonFactory._createUTF8Generator(JsonFactory.java:1284)
      at com.fasterxml.jackson.core.JsonFactory.createGenerator(JsonFactory.java:1016)
      at com.fasterxml.jackson.databind.ObjectMapper.writeValueAsBytes(ObjectMapper.java:2801)
      at com.couchbase.lite.support.RemoteRequest.setBody(RemoteRequest.java:326)
      at com.couchbase.lite.support.RemoteRequest.run(RemoteRequest.java:96)
      at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:422)
      at java.util.concurrent.FutureTask.run(FutureTask.java:237)
      at java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.access$201(ScheduledThreadPoolExecutor.java:152)
      at java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(ScheduledThreadPoolExecutor.java:265)
      at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1112)
      at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:587)
      at java.lang.Thread.run(Thread.java:841)
06-09 19:31:23.645 6978-7371/org.mc.mc E/dalvikvm-heap: Out of memory on a 7817-byte allocation.
06-09 19:31:23.652 6978-7371/org.mc.mc E/Sync: RemoteRequest.run() exception: %s
java.lang.OutOfMemoryError
      at com.fasterxml.jackson.core.util.ByteArrayBuilder.toByteArray(ByteArrayBuilder.java:118)
      at com.fasterxml.jackson.databind.ObjectMapper.writeValueAsBytes(ObjectMapper.java:2807)
      at com.couchbase.lite.support.RemoteRequest.setBody(RemoteRequest.java:326)
      at com.couchbase.lite.support.RemoteRequest.run(RemoteRequest.java:96)
      at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:422)
      at java.util.concurrent.FutureTask.run(FutureTask.java:237)
      at java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.access$201(ScheduledThreadPoolExecutor.java:152)
      at java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(ScheduledThreadPoolExecutor.java:265)
      at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1112)
      at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:587)
      at java.lang.Thread.run(Thread.java:841)

and here's my sync function:
final Replication pull = database.createPullReplication(createSyncURL(false));
        final Replication push = database.createPushReplication(createSyncURL(false));

        pull.setContinuous(false);
        pull.addChangeListener((new Replication.ChangeListener() {
            @Override
            public void changed(Replication.ChangeEvent event) {
                processSyncChangeEvent(event, pull, push, notifyManager, builder, appContext
                        .getString(R.string.downloaded_items), appContext.getString(R.string
                        .prep_downloaded_items), appContext);
            }
        }));
        if (pullListener != null) {
            pull.addChangeListener(pullListener);
        }

        push.setContinuous(false);

        push.addChangeListener(new Replication.ChangeListener() {
            @Override
            public void changed(Replication.ChangeEvent event) {
                processSyncChangeEvent(event, push, pull, notifyManager, builder, appContext
                        .getString(R.string.uploaded_items), appContext.getString(R.string
                        .prep_uploaded_items), appContext);

            }
        });
        if (pushListener != null) {
            push.addChangeListener(pushListener);
        }
        pull.start();
        push.start();

Hi @swilks,
Thank you very much for your report.

What version of Couchbase Lite Android are you using? v1.2.1? v1.2.0?...
What is server? Sync Gateway or CouchDB? and what is its version?
Can you provide us sample data?

Thanks!

Hi @sethrosetter,

I'm currently testing on an Alcatel One Touch (which is a pretty sub-standard tablet, but it's the kind of thing are users are likely to use, so it's a good test device) with ~65MB worth of documents (25,000 docs at ~2.5kb each)

Can you implement functional test for single push replication with 25,000 docs which average doc size is 2.5KB.

Hi @hideki,

using v1.2.1
sync gateway v. 1.2.1 on AWS ec2 t2 micro (which I know is too small for Couchbase server, but should work for testing right? It hasn't been overloaded yet). Couchbase server community edition 4.0.0
-I'm on my phone right now and don't have access to my computer, but the docs are 4

Sorry! Finger slipped.

docs are 2 string fields with length 10-20 chars, 2 with length 2-5chars, and one with length ~1500 chars (to inflate the size of each doc for testing). 2 ints, two doubles

@swilks Thank you for additional information. We will try to reproduce the issue.

Sorry for the slow reply (time difference issues).
The model number is 9005x
8gb storage, 1gb ram
ARMv7 Processor rev3 (v7l) - 1.3ghz, pretty sure it's dual core

It's the Alcatel Pixi 8, 8gb version

Hey, I'm wondering what the status of this is? On reviewable it looks like it was marked as fixed? Is there any way I can incorporate these changes, or do I have to wait for the next release?

This ticket is still in Incoming queue. It means this is not fixed yet.

We removed pause logic, which prevent to process many requests in case of many documents are created or updated, because the implementation was not thread-safe, and it caused dead-lock.
https://github.com/couchbase/couchbase-lite-java-core/commit/d990d7d4a4a769352d35117c15c8d8f6e6f148a4#diff-f7bd4a59e65ba5e7a9ab2b15276b81f5
We need to implement pause logic for push replicator with thread-safty (no dead-lock.)

Great, thanks for the update. Any idea what the timeline for the fix will be? I can attempt a hackey solution in the meantime, but if this is a high-priority bug is that likely going to be fixed soon I won't bother

I could reproduce this OOM issue.
https://gist.github.com/hideki/5fce3fcb0a1752494a267842efde592d

Hi @swilks,
We fixed this issue. If you have time, could you please verify the fix with using dev build for 1.3.0?
http://files.couchbase.com/maven2/com/couchbase/lite/couchbase-lite-android/1.3.0-4/
Instruction: how to setup dependency to Couchbase custom maven.
https://github.com/couchbase/couchbase-lite-android#using-latest-official-release
Thanks!

Hey @hideki, sure! I'm not sure when I'll get time as we're running up against some deadlines, but I can let you know by early next week at the latest

Hey @hideki sorry for the slow response. I've tested the change, but ran into server-side problems that are preventing me from verifying it 100%.
I loose connection to the server after about 15000 documents, but push behaviour seems correct. I was hoping I could ask your opinion on what the likely problem is. I'm using a t2.micro instance on AWS which is pretty small (~1gb ram)  and I'm guessing that my instance is just too small to support anything remotely intensive. Does that sound correct? (I posted the full issue here: https://forums.couchbase.com/t/couchbase-server-down-after-large-push-replication-from-cblite-android-on-ec2-t2-micro/9083)
Thanks!

server side minimum requirements included here: http://developer.couchbase.com/documentation/mobile/current/develop/guides/sync-gateway/deployment/in-production/index.html

Couchbase Server -Quad-core/16GB RAM
Sync Gateway - Quad-core/4GB RAM

To isolate if the issue you are encountering is hosting server issue or bug in sync gateway or couchbase server, I recommend to test your app with Couchbase recommendation environment at least once.

@hideki thanks! I upgraded my instance push replications are working fine!
Quick follow-up. After I push I'm resetting my local database to attempt to pull the same items back from the server and the pull replication is is going through the first 50 items and then "successfully" completing. Any ideas what could be causing this? My pull replication code is essentially the same, and is stopping with the status STOPPED after 50 items are pulled. Push replications are working as expected, and the only change I've made is to change my sync class from static to a singleton.
Could it be problem with sync gateway?

Hi @swilks,
I believe permanent error occurs during pull replication. Please file new ticket with log from logcat?
Thanks,
Hideki


Update with Bloch theory

@JuliaRegistrator register()

Registration pull request updated: JuliaRegistries/General/24065
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.1.3 -m "<description of version>" ced2bbbfae2060143c148ec27b2339b4c3888d59
git push origin v0.1.3


Error 600: You are not connected to the Internet.

Trying to formate dax measure. And here below the error I had
Error 600: You are not connected to the Internet.
[TimedOut] A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond. (www.daxformatter.com:443)
Trace Id: 00-3f3d42ed56c22042a22b3a4d512c4351-fd3b0560d6928f44-00
Version: 0.9.0 (0.9.8121.26944)

It seems to be a problem with the internet connection. Are you using Bravo from a corporate network behind a proxy server?
Is the daxformatter.com website accessible via a standard web browser?

Proxy server support will be available with the next release (0.9.5), which should solve this issue (see https://github.com/sql-bi/Bravo/issues/385).

This should be fixed in version 0.9.5, feel free to reopen if the issue still persists


Bug: Showing incorrect manifest on app details page if 2 same named k8s objects are deployed in 2 different namespaces

📜 Description
Showing incorrect manifest on app details page if 2 same-named k8s objects are deployed in 2 different namespaces.
👟 Reproduction steps

Create 2 service accounts with the same name in 2 different namespaces.
Deploy and go to the app details page.
Comparing the manifests for the 2 service accounts, it'll appear both are same, meaning both manifests will belong to the same namespace.

👍 Expected behavior
metadata.namespace should have been different for the 2 identical objects.
👎 Actual Behavior
metadata.namespace are same. Meaning both manifests are shown for same object.
☸ Kubernetes version
EKS 1.24
Cloud provider

AWS

🌍 Browser
Chrome
🧱 Your Environment
NA
✅ Proposed Solution
NA
👀 Have you spent some time to check if this issue has been raised before?

[X] I checked and didn't find any similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

AB#5160

Same as #3425


Upgrade to 2024.12.0 breaks custom cards

“Custom element does not exist. “
Version of the custom_component
Up to date.
Configuration
Add your logs here.

Describe the bug
No custom charge cards appear to work after the update. Not sure if it’s linked to the way the custom cards are written, or the HAOS change?
Debug log

Add your logs here.

Appears to be isolated to the iOS App only, noteworthy break but can be circumvented for no.

I suspect the app hasn't loaded or refreshed something. Force close and reopen / clear cache?


Feature: Add issue templates

Description
There are sections about how to write an issue for Reporting Bugs and Suggesting Enhancements in the [How Can I Contribute?](How Can I Contribute?) section on the Contributing Guidelines. And these contain clear instructions about how to write issues.
However, there would still be a chance where contributors miss to include important information that maintainers need.
Suggested solution
Create issue templates for reporting bugs and suggesting enhancements. It will give these benefits:

Prevent contributors to miss giving all information needed.
Make it easier for maintainers to triage and prioritize the issues.

Hey @petermsouzajr it would be an honor if I can contribute to this one, if you agree with the proposal! 😊

@adiati98  That sounds like a great improvement, thanks! Would you like to create a PR to remove the  "Reporting Bugs" and "Suggesting Enhancements" sections from the CONTRIBUTING doc?

@petermsouzajr absolutely!
Just to confirm, do you want me to only remove them from CONTRIBUTING.md or also create the templates?

@adiati98  Yes, please also create the templates. 💪

@petermsouzajr will do!
Can you please assign the issue to me to prevent others spend their times to work on the issue?
Thanks!


Deleting namespace stuck at "Terminating" state

Describe the bug
When user deploy Antrea in a specific namespace eg: antrea-system, then trying to delete all resource in this namespace via kubectl delete ns antrea-system, the namesapce will stuck at terminating state.
To Reproduce

Change the default namespace in the antrea.yml to 'antrea-system'
Run kubectl apply -f antrea.yml
Delete all resources via kubectl delete ns antrea-system

Expected
The namespace should be deleted successfully.
Actual behavior
The namesapce will stuck at in terminating state.
Versions:

Additional context

There is an known issue in Kubernetes related with this one https://github.com/kubernetes/kubernetes/issues/60807,
the root cause is not from Antrea, but we probably need to provide a doc somewhere to remind user about this issue, avoid to delete namespace directly, but run kubectl delete -f antrea.yaml, then delete the namespace. and provide the workaround when this issue happens.

The behavior is  expected according to issue https://github.com/kubernetes/kubernetes/issues/75788, the most common case is with some of its resources do not get removed and Kubernetes keeps waiting for that. So provide some options here to clean up the stuck namespace in case if any user encounter this issue and try to find a solution.

You should see some apiservices registered by Antrea controller is not available, delete all False apiservices, then Kubernetes will remove the namespace in a short time.

kubectl get apiservice | grep False | awk '{print $1}' | xargs kubectl delete apiservices

Follow below steps to remove the namespace's finalizers, then Kubernetes will remove the namespace in a short time.

kubectl get namespace <terminating-namespace> -o json >tmp.json
# Edit your tmp.json file. Remove the kubernetes value from the finalizers field and save the file.
kubectl proxy &
# Make an API call with your temporary proxy IP and port:
curl -k -H "Content-Type: application/json" -X PUT --data-binary @tmp.json http://<IP_ADDRESS>:8001/api/v1/namespaces/<terminating-namespace>/finalize


Recognize non-standard licenses

Uniquely identifying non-standard licenses would make it possible to whitelist them or detect future changes in the text of the licenses.
Currently, repositories such as github/linguist, which have many licenses in submodules to validate, must use some custom scripts or rely on manual validations.
Having licensee generate a unique identifier for each non-standard license would make it possible:

to track any future modification(s) in the license text;
and to whitelist a few common licenses, such as the TextMate bundles license.

The unique identifier could eventually be a hash value of the license text.

Interesting. We could create a SHA1/MD5 hash of the normalized license body and expose it as License#hash or similar. Would that satisfy your use case?
//cc @bkeepers who may also have a use for this.

Would that satisfy your use case?

Yes, it would :smiley:


Adapting BlackrockIO for new features of version 2.3

Blackrock file version 2.3 enables features like a reset of the global timestamp clock or pauses during recording. This creates multiple segments in nsX data whereas in the nev files there is usually nothing that shows the beginning of a new segment. Thus, nev segments need to be mapped from the nsX or, if this is not available, segments can be distinguished, when a reset occurred and the timestamps start from 0 again. If the recording was paused and thus the timestamps were not reset, this can only be done using the nsX file.
This pull request enables the following:

Mapping segments found in nsX to nev:

If the recording was paused, a new segment border is entered using the timestamp of the nsX segment
If a reset occurred, the nev segment can be detected by a negative difference of timestamps.
If this is not possible, the segments cannot be distinguished and a ValueError is raised

Splitting nev into segments even when no nsX is available. A warning is issued, that 'paused' segments cannot be detected.
Empty segments (0 or 1 sample) that are contained in the nsX file are discarded

I will now create test files for this and write tests.

Hello @bjoern1001001! Thanks for submitting the PR.

In the file neo/rawio/blackrockrawio.py, following are the PEP8 issues :

Line 286:100: E501 line too long (102 > 99 characters)
Line 294:25: E127 continuation line over-indented for visual indent
Line 347:30: E127 continuation line over-indented for visual indent
Line 518:100: E501 line too long (100 > 99 characters)
Line 520:100: E501 line too long (100 > 99 characters)
Line 910:25: E127 continuation line over-indented for visual indent
Line 1112:5: E303 too many blank lines (2)
Line 1127:100: E501 line too long (103 > 99 characters)
Line 1132:100: E501 line too long (102 > 99 characters)
Line 1192:33: E116 unexpected indentation (comment)
Line 1192:33: E265 block comment should start with '# '
Line 1733:28: E127 continuation line over-indented for visual indent
Line 1734:28: E127 continuation line over-indented for visual indent
Line 1763:100: E501 line too long (112 > 99 characters)

Coverage decreased (-0.2%) to 48.727% when pulling dcbafa5ed7e2f33c8e3bd7bff49a1f8e1a5496a6 on bjoern1001001:timing_discard_empty into db9357c37de1ea281cbe04c4b46ea4095571d4b3 on NeuralEnsemble:master.

Coverage decreased (-0.2%) to 48.727% when pulling dcbafa5ed7e2f33c8e3bd7bff49a1f8e1a5496a6 on bjoern1001001:timing_discard_empty into db9357c37de1ea281cbe04c4b46ea4095571d4b3 on NeuralEnsemble:master.

I just added tests that make sure segments are detected correctly and their start and stop times are correct. There are multiple tests with different files in order to check all situations:

Failing, because segments could not be separated in nev
Warning because of spikes outside of segment
Warning because reset was not explicitly documented in nev file
Correctly running through with no problems
I will make a pull request to upload those files to gin now.

In my opinion this PR is thus finished.

Sharp eye.
It was to benchmark internal performence. No longer needed.


Proposed feature: configuration file

:abc: User Story

As a developer
I can specify a local configuration file
so I do not have to specify my favorite command line options on every invocation of stevedore

:information_source: Notes

Since stevedore has several command line flags, it would be nice to support a configuration file, so preferred behavior could be based on a central configuration
The configuration file could control:

exclusion
inclusion
colors
color output
color inversion
verbosity
debug level
path to global ignore file
full path output
no full path output

The configuration file could be located in:

$HOME/.config/stevedore.json
or in a local repository as .stevedore.json
The local file could have precedence over the global one

:ok: Acceptance Criteria

[ ] I can specify a local configuration file as .stevedore.json
[ ] Command line options take precedence over configuration files

:arrow_right: Resources and References

N/A

Story updated to reflect implementation in release v0.9.0.
Global configuration file pushed to separate issue (#28), due to challenges with the two parameters of:

config
command line


Is it intentional that a ResourceBundle properties file is required now?

While upgrading from 1.4.1 to 2.1.0, I noticed that for any custom check, violations are not reporting unless there's a .properties file used for the ResourceBundle. Is this intentional?
I also might be doing something wrong of course :)

Just noticed this issue, sorry. Are you overriding the getResourceBundleName() method to return null? In 2.1.0, I added the following to the ViolationReporter interface to drive the localization support:

/**
 * Base interface for violation reporters.
 */
@ConsumerType
public interface ViolationReporter {

    /**
     * Get the resource bundle base name for loading the default resource bundle for this violation reporter. Returns
     * {@code getClass().getName()} by default. If this method is overridden to return null, the oakpal framework will
     * not attempt to load a parent resource bundle specific to this violation reporter when creating a resource bundle
     * during initialization.
     *
     * @return the resource bundle base name
     */
    @Nullable
    default String getResourceBundleBaseName() {
        return getClass().getName();
    }

I must have intended to provide a default empty resource bundle at some point. I'll add it to the next release.


remove outdated statements from examples readme

it'll always be a work in progress.

Yep, you're right! Thx a lot! ) Your involvement is very important to us!


Settings configuration

upgraded to the latest from 0.2.2, today
Settings have changed in this version,
Instead of the ez to use popup panel, now I get some obscure configuration which seems to comply with graphql-config
Question:

How to get the popup back?
How to configure this file? I tried the examples from graphql-config with no success

Thank you

We replaced the popup with a new settings file in JSON format. The setting options are specific to the playground and you can (currently) only change the theme and enable/disable the header reuse option.
It is not supposed to support graphql-config markup here. 😉

Thank you,
Can you maybe share how to point to the graphql server?
playground is useless if I can't point to a schema...

Closing this due to inactivity.


Stop using Lolex, since we're using Jest's "modern" fake timers.

Planned followup to #4754.

Thanks! Merging.

Thanks for the review!


Ban Appeal

MC name: AltyBalty (old Unklet1)
Hello,
Today I got banned of the reason: "Reward glitching and xray"
Reward glitching: This is a common issue for me, because me and my brother are always playing the same games together. So when he wanted to join the pixelmon server aswell, I invited him to the server so I could get extra rewards. From staff perspective I understand that it looks weird, but unfortunately I couldn't explain that I was playing with my brother. I didn't know this counted as reward glitching and I wouldn't mind to give the rewards back.
More info:
My brother has a really difficult study and only had this week to play with me. So thats why he gave me all his stuff because he wasn't gonna play for a long time. He only helped me with getting the stuff I needed to move further in my leveling.
Xray: (I edited this party as I want to answer truthfully)
I was banned today for using Xray, and I admit I was using the mod. I wish to apologize for it and ask for a second chance since it was my first offense. In the rest of my appeal I will go into detail my motives of using Xray, how long I have used Xray, how much money I made using Xray, apologize for using it, and ask for forgiveness.
My motives for using Xray were originally to test it out and see what the hype was about, and then after I knew that I would stop using it. But after testing it I discovered the greatness of this modification. I realized I could make money using the mod. I realized I could have all the iron, diamonds, redstone, gold and other ores I could ever need, then sell what was left for profit. What started out as a test turned into an income that I enjoyed. My original intent was not to make profit.
I have been using Xray since about 2 days ago total, I estimate my profit to be about $15000, but I have more ore that I got by using this mod in chests around my houses. I believe I have about 1 stack of diamonds less than half a stack of gold, 14 and 1-2 stacks of coal in chests, furnaces, or my inventory. I do not know anything for sure, nor can I check because I cannot access the server anymore.
I wish to apologize for using this mod. Right from the start I knew it was illegal, but I wanted to test it. After testing it I did not stop using it, which was wrong of me. I do not believe I profited as much as other offenders of my type. Even so, what I did was wrong and it harmed the economy to a small extent, but it was still wrong.
Please forgive my offense. I will never use this mod on your server again, never. Not to any extent. If I do use this mod again I will accept any punishment that is served for two reasons. I would be lying to you if I continued using this mod after you gave me a second chance and I wouldn’t deserve a third chance if I did this again.
As you are the administrators of Oblique, you have the decision to do what you want for my offense of using Xray. If you want to give me a second chance on the server, I thank you. If you decide to not give me a second chance, I will leave the site alone and it will be as if I were never there. Please note, I tried writing this letter professionally and I hope you take it as such.
Thank you for understanding.
If you have any questsions please let me know.

I believe you are genuine in your apology, whats troubling is how you knew it was wrong / against rules to use such mods and used them anyway. Whats also troubling is how you say 'If I do use this mod again'. Makes it sound like you do plan on using it again. I hope not. I will give you a second chance. Please delete or disable any mods / xray mods you have before you log in next. I have also confiscated most of the things you obtained through xray and the sum of money. If we find you are using any mods again, you will be banned immediately, and like you say, you won't get a third chance.


Allow setting up a DNS name for the minecloud server

Implements #68
A DNS name is setup automatically when the parameter DOMAIN_NAME is set to a name pointing to an existing Hosted Zone. Setting up a hosted zone on AWS involves purchasing a domain and paying a for it on a yearly basis. This makes it hard to automate and is therefore left out of the automation. Instead an existing
hosted zone is referenced by the value of the parameter DOMAIN_NAME in the MineCloud-Service-Info.ts file. When this parameter is set, an additional branch of the CDK script gets executed, which assigns some
additional policies to the EC2 role to allow the VM to lookup the hosted zone id to update the IP address of the ARecord, which has been created by CDK.
The DNS name is derived by appending the value of
DOMAIN_NAME to the hostname "minecloud".
The DOMAIN_NAME is attached as a tag to the EC2 instance to make it accessible to the start_service.sh script which updates the ARecord with the assigned IP address. The bot pushes the domain name to the Discord channel only when the DOMAIN_NAME parameter has been set.
The implementation also uses an AWS internal way to get the IP address without the need for an external DNS service like opendns.com.
The changeset also includes a change to avoid the file MineCloud-Service-Info.ts with all the private data being committed to Git accidentally. Therefore the file is renamed to MineCloud-Service-Info.ts.sample and the README is updated to make a copy to the actual name MineCloud-Service-Info.ts before setting all the parameters. The MineCloud-Service-Info.ts is set to the .gitignore list so that the actual file is ignored upon git commits.

Looks great. Thanks, Michael!


pip -> pip3

docker image is pointing to pip from python 2.7, even though python3.5 is installed. will figure out details of docker & report back!


Force push down for nested queries

The way nested query execution is implemented today, it executes the inner most query on the historical nodes with the outer queries being executed on the broker node. This can get problematic when the inner query groups using a high cardinality dimension, returning too many records for broker node to handle. One of the options that we have been internally testing and exploring is the capability to push down the complete nested query to the historical nodes. Each historical node then will execute the nested query primarily dealing with the segments it owns for that query. Because the number of records returned by each historical node would potentially be much smaller in this case, it would be less intensive for the broker to perform the final merge and aggregation. The broker though won't need to perform any more dimension or segment level filtering since it will be taken care of at the historical nodes itself. Note that this way of distributing the aggregation to the historical nodes doesn't always return the same results as the final aggregation getting done on the broker node. However, there is a good set of cases (for ex - aggregating on dimensions that are used for hashing during ingestion) where this kind of push down logic will return the right results. I can get into this into more detail but the general idea was to leave the onus on the user to figure out if their data layout allows for this kind of push down.
This implementation provides user a way of forcing nested query execution through a query context variable. The next cut will focus on doing this automatically under the following conditions (credit to @gianm ) for clearly articulating the following:
The groupBy query granularity is equal to, or finer than, segment granularity;
and either:
2a) A time chunk uses HashBasedNumberedShardSpec, partitionDimensions is nonempty, the grouping dimension set contains all of the shard partitionDimensions, and there are no "extension" partitions (partitions with partitionNum >= partitions, which are created by ingest tasks that append data)
or:
2b) A time chunk uses SingleDimensionShardSpec and the grouping dimension set contains the shard dimension.
If Druid detects this it should push down the query automatically for that time chunk. There will be situations where the query can be pushed down for some time chunks but not others (for example: consider a data pipeline that loads data unpartitioned in realtime, and later has a reindexing job to partition historical data by some useful dimension). In this case, ideally the broker should be capable of pushing down the query for the time chunks where it can be correctly pushed down, and not pushing it down for others.
TODO: support "subtotalspec" for push down nested queries.

@jihoonson - it looks like something got messed up after druid code base was migrated to include incubator-apache in the repository url.

Contributed and merged in https://github.com/apache/incubator-druid/pull/5471


[fix][ci] Fix compile loader error

refer the latest toolchain CI code, the commit ID in server has passed >100 times..
so it alert this error. use the recent commit ID is fine

another way to solve this problem is that we change the --depth to 300 or more to let it run in the old version.
however, we need handle the latest version in the last :)


Constructors referencing deprecated fields with initializing formals do not generate hints at invocations

This came up at https://github.com/Dart-Code/Dart-Code/issues/2966. The following code doesn't generate any warnings about the deprecated field:
final c = MyClass(deprecatedField: 'b');
//                ^^^^^^^^^^^^^^^ Expect warning here?

class MyClass {
  MyClass({this.deprecatedField});

  @Deprecated('this member is deprecated')
  final String deprecatedField;
}

It seems like a warning should be generated at the call site as for all intents and purposes, it's referencing a deprecated field.
I have a CL to fix this (https://dart-review.googlesource.com/c/sdk/+/174805), though it could generate a lot of new warnings that weren't there before, so this may warrant some discussion.
@bwilkerson

@leafpetersen @lrhn @davidmorgan In case you have thoughts about this.

No worries from my side--we never treat deprecation warnings as breaking. Thanks.

As I see it, the invocation does not refer to the field. The name written in the invocation is the name of a formal parameter of the constructor, and that parameter is all it refers to. The fact that that parameter is an initializing formal for a field of the same name, is an implementation detail.
It's possible for the class to remove the field and retain the constructor parameter if it wants to, so it is at least one step removed from the deprecated field. Perhaps they're not removing the field, merely making it private, but they still need to initialize it.
So, I'd say no warning here. If the class author is planning on removing the constructor parameter too, they should deprecate that as well. (We might want to figure out if there is any way we can tell them that: "You're deprecating a field, but not the associated initializing formal. What's up with that?" - maybe make that a warning, and the workaround is to split it into ({String name}) : name = name; which will have to become _name = name anyway when the field is made private).

Perhaps they're not removing the field, merely making it private, but they still need to initialize it.

Good point - accessing the field as - er, the field - had completely slipped my mind.
I went back and checked the original issue at https://github.com/Dart-Code/Dart-Code/issues/2966 and it looks like the code example given (which fails to generate a warning based on the above) isn't actually the same as the issue described (Flutter's showDialog - which is a function), so I'll abort this and try to get a repro of that (since in my testing, showDialog is correctly reporting its deprecated fields).

We might want to figure out if there is any way we can tell them that: "You're deprecating a field, but not the associated initializing formal. What's up with that?" - maybe make that a warning, and the workaround is to split it into ({String name}) : name = name;

If there's value in this (though I don't know how common it is, given the example I had was misleading), I can have a go at it (w/ a fix to do the change). Let me know if you think so and I'll file a separate issue for it to avoid confusion with this one. Thanks!


data-start attribute not longer being applied to the  element

Looking at the prism documentation here it states "Optional: You can specify the data-start (Number) attribute on the pre element. It will shift the line counter." Adding something like data-start={{@start}} to the pre element in the code-block.hbs template corrects it.

@Syn-Tylor are you saying you were able to add this before or this is a new feature request?

Hey, ya it was working previously (on v0.5.0 at least).
I believe (could be wrong on this) it is not working because the previous implementation used a pre tagName. The newer version (v0.9.1) still does this, but in its template the ...attributes (where the data-start is contained) is being added to the <code> element instead of the <pre> element.

@Syn-Tylor that seems logical. Perhaps we should move ...attributes?

Uhm.. not too sure.
I will play with it a bit and see what the effects are. Set myself a reminder monday night to test it out a bit.

Sounds good, keep me posted!

So played around w it a bit today. Basically 2 routes I played with
a) adding data-start={{@start}} to <pre> element

this worked w/o any notable issue
b) moved ...attributes from <CodeInline> to <pre>
this had some unintended consequences regarding styles

@Syn-Tylor the first option sounds good for now. Would you like to open a PR?

Hey, sry its been a while. Looked back at this today and created a branch with the fix, documentation update, and test coverage. Unfortunately (and I can't figure away around this) Prism uses CSS counter-increment property to show the actual line number. Meaning you cannot assert on the contents of the element as it is not bound in the DOM (ref: https://stackoverflow.com/questions/532073/how-can-i-read-the-applied-css-counter-value) . I could just leave that test out though.
Note: sorta blocked on this though as I can't push the branch up, getting permission error on git push -u origin fix-241-data-start-attribute-not-working. Tried a few things like removing/regenerating SSH keys and publishing the branch via GitHub desktop, but got the same error.

opened PR https://github.com/shipshapecode/ember-prism/pull/342


Fix: bold typo in readme.md

Fixes #730
@mariobehling @iamareebjamal @CloudyPadmal  can you review this pr-

Change title to match semantic naming scheme

Can you plz add the codeheat label on issue ?

It's not codeheat level issue


Error when running Flask application

Describe the bug
Error when running Flask application
新建文本文档.txt
(myenv) D:\my projects\instagraph>python main.py
File "D:\my projects\instagraph\main.py", line 262
match graph.lower():
^
SyntaxError: invalid syntax
Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [winows]
Browser [chrome]
Version [10]

Additional context
Add any other context about the problem here.

match requires python 3.10 or higher.
Please try latest version.


https://github.com/CyberReboot/poseidon/issues/255

forgot to update endpoint data..

Codecov Report

Merging #350 into master will increase coverage by 0.76%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #350      +/-   ##
==========================================
+ Coverage   89.86%   90.62%   +0.76%
==========================================
  Files          24       24
  Lines        1578     1578
  Branches       94       94
==========================================
+ Hits         1418     1430      +12
+ Misses        124      112      -12
  Partials       36       36

Impacted Files
Coverage Δ

poseidon/poseidonMonitor/test_poseidonMonitor.py
88.4% <100%> (+0.4%)
:arrow_up:

poseidon/poseidonMonitor/poseidonMonitor.py
78.33% <0%> (+4.58%)
:arrow_up:

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 89f83e9...14031ff. Read the comment docs.


Auto resize mixin rebased

This is a copy of #81 rebased on current master so I can have a look at the proper diff.

LGTM. @ronichoudhury: up to you if you want AutoResize to include InitSize (I can't think of a case where you'd want AutoResize and not InitSize). You can merge this (pre-rebased) and close your other 3 PRs or merge your other 3 PRs and close this one.

Closing in favor of #78, #80, and #81.


Added Password strength indicator

Title and Issue number
Password Strength Indicator #2510
Close #2510
Checklist:

[X] I have mentioned the issue number in my Pull Request.
[X] I have commented my code, particularly in hard-to-understand areas
[] I have created a helpful and easy to understand README.md

https://github.com/pranjay-poddar/Dev-Geeks/assets/133862694/0938ba36-776f-474d-b8b4-72bdbc346b20

@Saurabh-kumar09 @MohitGupta121

@Saurabh-kumar09 Please merge this

Already we have


Detected as a Trojan via Windows Defender

Details:

I also encountered this warning too .

me too

me too
Trojan:Win32/Bitrep.B

Related issue:
https://github.com/Jigsaw-Code/outline-client/issues/12

@mxschmitt, @griffinbai, @lifeva, @suonli - many thanks for the reports. This is really useful for us. I'm going to mark this as a duplicate of https://github.com/Jigsaw-Code/outline-client/issues/12 and continue discussion there.


Thrift 3103

See https://issues.apache.org/jira/browse/THRIFT-3103

Actually this pull request is only about THRIFT-3103, but you see also the previous THRIFT-3090 commit (which has a separate pull request, https://github.com/apache/thrift/pull/434), because this commit, THRIFT-3103, depends on THRIFT-3090. Hope it is clear :-)

@marco-m this is committed, could you please close the PR

@bufferoverflow done, thanks for accepting the PR :-)


db

Thanks for filling this.
You can create the needed database table by running something close to
marks -create-db -postgres-url=$DATABASE_URL

as stated in the https://github.com/kiasaki/marks/blob/master/app.json
The SQL used is in this file https://github.com/kiasaki/marks/blob/master/create_database.go
I'll add details to the README for it :D

Great work.Thank you for the update.


Update 10_Versioning.md

Fix typo in class name.

Thanks @BeQ  just a note, if you prefix your pull request with "DOCS" it helps the core committers find docs related things faster and will likely get merged quicker ;)

Oh, did not know about that. Use it next time :)


fix: address chrome 121 scrollbar style behaviour change (#1787)

Chrome adopted offical support for scrollbar-color in chrome 121. However, if you have both scrollbar-color and ::webkit-scrollbar styling, they conflict and chrome will only honour the native styling. This is bad because the native chrome mac scrollbars become really small, and overlap too much with out golden-layout drag.
Our webkit styling forces scrollbars that match monaco and grid, and are preferred. By removing scrollbar-color native styling, our webkit styles work. We only had scrollbar-color to style firefox in dark mode (//applies to firefox only is no longer true), however the most recent versions of firefox appear to color scrollbars automatically based on the theme, so this is no longer necessary.
The crappy part here is since this is browser change, we will have to backport this for several active enterprise versions.
See:
https://stackoverflow.com/questions/77919700/scrollbars-chrome-update-121

https://issues.chromium.org/issues/41494676

e2e tests don't seem to be running correctly on this branch, force-merging.


Setting all *.js to 'Content' under 'public' folder upon import

I created a node project from existing files.
Existing project has a significant number of files and subfolders under /public which are reporting as failed upon build due to their type set to 'Compile' instead of 'Content'....
Is there any way to mark all *.js files under particular folder (in this case 'public') as 'Content automatically?

We are planning to make it easier, but for a different reason (#142)... what type of build errors are you getting?

Well the issue is that upon import, all *.js files in /public/javascripts/ folder are flagged as 'Compile' and the build fails with several hundred "Ambiguous catch identifier. Cross-browser behavior difference." errors.  These are comming from 3rd party libraries over which I have no control. They should not be inspected at all and so to eliminate errors I have to flag them as 'Content'..one...at...the....time.....
I should be able to right click on the folder and choose 'Treat as Content' or something....
And then of course there is
"Files in your project exceed MAX_PATH, please move your project to a location with a shorter path."
which I understand is issue between how NPM lays out module structure and Windows not supporting more than 255 chars in path....But even moving to C:\src does not help nor does 'dedupe' helps....
I should be able to "turn off' this as a build error....

Gotcha, thx for the info. Can you give me an example of one of the 3rd party libraries that you're using that is having the issue? Are they VS extensions? It's useful to understand what tools people are using that have potential to conflict with the way we do stuff.
In the meantime... Have you tried excluding the files from the project altogether? (right click, exclude) Like node_modules - it's not always necessary to include files in the project itself in order to work with them.

Fixed in #243

There is now a context menu command on folders to enable this, and it's available in the latest dev build

Nice. it works....
I am not too fond of the labeling though - putting (node) and (browser) next to folders.....To verbose...
Perhaps folder icons should be indicators if code is node vs browser....
Just my 2c
In any case this issue is resolved.


Add Czech version of Firefox to Cask

If there’s a checkbox you can’t complete for any reason, that's okay, just explain in detail why you weren’t able to do so.
After making all changes to the cask:

[ ] brew cask audit --download {{cask_file}} is error-free.
[ ] brew cask style --fix {{cask_file}} reports no offenses.
[ ] The commit message includes the cask’s name and version.

Additionally, if updating a cask:

[ ] If the sha256 changed but the version didn’t,
provide public confirmation by the developer: {{link}}

Additionally, if adding a new cask:

[ ] Named the cask according to the token reference.
[ ] brew cask install {{cask_file}} worked successfully.
[ ] brew cask uninstall {{cask_file}} worked successfully.
[ ] Checked there are no open pull requests for the same cask.
[ ] Checked the cask was not already refused in closed issues.
[ ] Checked the cask is submitted to the correct repo.

I am currently await from my Macbook for few days, so cannot test the change now, but it's just a new language version + its sha.
What about adding all language versions to the Cask? Both to release, beta and nightly in homebrew-versions?

@mahori ?

Thank you @MikkCZ.


List Repositories by Org does not support sorting

Sorting repositories is supported by the v3 API, can those options be made available in the RepositoryListByOrgOptions struct?
Sorry for the duplicate, but the older issue is almost 3 years old (#321), figured I would reopen for visibility!

Related GitHub docs: https://developer.github.com/v3/repos/#list-organization-repositories
#1258 has already been created to address this issue.

@gmlewis this issue can be closed

Thank you, @wesleimp.
Closing.


Addition of queuing clients

You need to add clients to the repository for the different queues
AC:

[x] RabbitMQ
[ ] Kafka + Avro

In this repo there will be no client for kafka and avro I will create a separate repo that will have this implementation


[VERTICA] "Smart autocomplete" doesnt work on Vertica when using ${variables}

System information:

Windows 10 64bit
7.0.4

Connection specification:

Vertica 9.1
JDBC 9.1

Describe the problem you're observing:
"Smart autocomplete" doesnt work on vertica when using ${variables} with values. I've tested this feature on Oracle and it perfectly works.
It might be related: same happens when you want to inspect a table on ${schema}.table using CTRL+click on "table"
Steps to reproduce, if exist:

Execute:
select *
from ${schema}.table
Try to use CTRL+SPACE to autocomple when writing your "table"

Thank you for the report.

Fixed for Vertica and for Oracle.

e.

Verified


AST: Parse #_hasSymbol

Introduce the compiler directive #_hasSymbol which will be used to detect directly whether weakly linked symbols are present at runtime. It  is intended for use in combination with @_weakLinked import or -weak-link-at-target.
if #_hasSymbol("foo") {
  foo()
}

Currently, #_hasSymbol only accepts the symbol name as a string literal but we may be able to update it to also accept expressions that result in references to declarations.
Parsing only; SILGen is coming in a later commit.
Resolves rdar://99342017

https://github.com/apple/swift-syntax/pull/661
@swift-ci please smoke test

https://github.com/apple/swift-syntax/pull/661
@swift-ci please smoke test

https://github.com/apple/swift-syntax/pull/661
@swift-ci please smoke test

After some research I've concluded that taking a string literal to identify the symbol will not be as simple as I'd imagined. The corresponding decl ref needs to be identified during type checking in order to SILGen the condition. I'm going to rework this to take an expression instead of a string literal.

https://github.com/apple/swift-syntax/pull/661
@swift-ci please smoke test

https://github.com/apple/swift/pull/61001
https://github.com/apple/swift-syntax/pull/661
@swift-ci please smoke test Windows

https://github.com/apple/swift/pull/61001
https://github.com/apple/swift-syntax/pull/661
@swift-ci please smoke test


Emmet Abbreviation

We have written the needed data into your clipboard bec
Issue Type: Bug
Hi,
Im having issues with Emmet Abbreviation not working on one of my HTML files.   When I type in a tag word like "button" or "!" it does not auto populate my brackets or the rest of the autofill tags or closing tags.  The Emmet window does not even show up, only "abc" window.
It only started happening a few enteries ago on the same file I am having issues on (where exactly, I am not sure?)  It works perfectly fine on new files or other files.
I am assuming I accidentally toggled a shortcut key that has somehow disabled it?
Any help you can provide would be greatly appreciated, thanks in advance.
-Kyle
VS Code version: Code 1.63.2 (899d46d82c4c95423fb7e10e68eba52050e30ba3, 2021-12-15T09:40:02.816Z)
OS version: Windows_NT x64 10.0.19042
Restricted Mode: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-7300U CPU @ 2.60GHz (4 x 2712)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledskia_renderer: enabled_onvideo_decode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
7.89GB (3.33GB free)

Process Argv
--crash-reporter-id 1f71a3ae-d4e8-496a-a7b4-e03e3fb40e0f

Screen Reader
no

VM
0%

Extensions (25)

Extension
Author (truncated)
Version

django-html
bib
1.3.0

EditorConfig
Edi
0.16.4

vsc-material-theme
Equ
33.2.2

vsc-material-theme-icons
equ
2.2.1

prettier-vscode
esb
9.1.0

auto-close-tag
for
0.5.13

code-python-isort
fre
0.0.3

todo-tree
Gru
0.0.215

vscode-power-mode
hoo
2.2.0

dotenv
mik
1.0.1

python
ms-
2021.12.1559732655

vscode-pylance
ms-
2022.1.3

jupyter
ms-
2021.11.1001550889

java
red
1.2.0

markdown-preview-enhanced
shd
0.6.1

code-spell-checker
str
2.1.4

vscode-djaneiro
the
1.4.2

rst-vscode
tht
3.0.1

vscodeintellicode
Vis
1.2.15

vscode-java-debug
vsc
0.37.0

vscode-java-dependency
vsc
0.18.9

vscode-java-pack
vsc
0.20.0

vscode-java-test
vsc
0.33.1

vscode-maven
vsc
0.34.2

grammarly
znc
0.14.0

(1 theme extensions excluded)

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
pythonvspyl392cf:30425750
pythontb:30283811
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
pythondataviewer:30285071
vscod805:30301674
pythonvspyt200:30340761
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
vsaa593:30376534
vsc1dst:30424896
pythonvs932:30410667
vscop804:30404766
vscop341:30404997
vsclayoutctrt:30425946
vsrem710:30416614
vsbas813:30426126
vsic-wlc-tsjs:30426436

ause it was too large to send. Please paste.
Please excuse my coding, Im new and learning, just using this file as a "reference page."

HTML 201

<link
    rel="shortcut icon"
    href="C:\Users\Allan\Desktop\HTML Folder\Images\favicon\123_favicon.png"
    type="image/png">

Welcome to HTML 201

GO TO BOTTOM

        Lorem ipsum, dolor sit amet consectetur adipisicing elit.

     Temporibus dignissimos eos aut eius modi libero maxime cum nemo provident, similique dolorum cumque praesentium laudantium incidunt, eum laborum quidem nostrum voluptas.
     Temporibus dignissimos eos aut eius modi libero maxime cum nemo provident, similique dolorum cumque praesentium laudantium incidunt, eum laborum quidem nostrum voluptas.

A
B
C

First Cell Here
Second Cell Here

First Row Here
Second Row Here

1

2
3

        4

Welcome to HTML 201

Navigation

Content

    Lorem ipsum dolor sit, amet consectetur adipisicing elit. Error nobis praesentium inventore vero quam! Quos maiores laudantium veniam suscipit amet vel corporis! Sed asperiores odit facilis doloribus, sunt repellendus quo?

    Lorem ipsum dolor sit amet consectetur adipisicing elit. Quaerat non odit id, perferendis, incidunt reprehenderit eos beatae repellendus assumenda pariatur dicta nostrum aut est vero nesciunt soluta consequuntur. Modi, suscipit.

    Lorem ipsum dolor sit amet consectetur, adipisicing elit. Tempore incidunt error laudantium provident fuga molestias quaerat et mollitia suscipit exercitationem laboriosam quos magnam placeat, harum aspernatur expedita, debitis corporis blanditiis?

    Lorem ipsum, dolor sit amet consectetur adipisicing elit. Ea consectetur nesciunt nulla neque, ipsum, qui aut assumenda impedit officiis deleniti velit! Incidunt, qui. Cupiditate, corrupti rerum alias animi suscipit maxime.

 Format code

            Some stuff here       <b>This is bold</b>
        Second line Here

  Lorem ipsum dolor sit amet consectetur adipisicing elit. Commodi delectus nulla animi laboriosam aperiam earum natus illum minima veritatis, nisi est atque impedit inventore amet tempora, aliquid accusamus nesciunt in!

Forms

    <form action="">
        <input type="text" value="Hello World"
        placeholder="Enter your name">

        <input type="password" value="password"
        placeholder="Enter Your password"

    </form>

    <form action="" method="GET">

Text in here
    White Space Matters here.

Button
Reset
Submit

This is Value 1
This is Value 2
Selected by default
This is disabled

    </form>

BACK TO TOP
   ©2022
   &copy 2022
   ©2022

\gifPlease

I don't see any pictures. If the problem is specific to that one file, I wonder if you can attach it, assuming it has no confidential information.
Also, does the issue reproduce on VS Code Insiders?

No its nothing secret, I am an HTML 201 course through Udemy so it is just
a reference sheet that I had made for my own personal use.
Here is the code I wrote up, also I will try and reattach pics.

HTML 201

<link
    rel="shortcut icon"
    href="C:\Users\Allan\Desktop\HTML

Folder\Images\favicon\123_favicon.png"
type="image/png">

Welcome to HTML 201

GO TO BOTTOM

        Lorem ipsum, dolor sit amet consectetur adipisicing elit.

     Temporibus dignissimos eos aut eius modi libero maxime cum nemo

provident, similique dolorum cumque praesentium laudantium incidunt, eum
laborum quidem nostrum voluptas.
Temporibus dignissimos eos aut eius modi libero maxime cum nemo
provident, similique dolorum cumque praesentium laudantium incidunt, eum
laborum quidem nostrum voluptas.

A
B
C

First Cell Here
Second Cell Here

First Row Here
Second Row Here

1

2
3

        4

Welcome to HTML 201

Navigation

Content

    Lorem ipsum dolor sit, amet consectetur adipisicing elit. Error nobis
praesentium inventore vero quam! Quos maiores laudantium veniam suscipit
amet vel corporis! Sed asperiores odit facilis doloribus, sunt repellendus
quo?

    Lorem ipsum dolor sit amet consectetur adipisicing elit. Quaerat non
odit id, perferendis, incidunt reprehenderit eos beatae repellendus
assumenda pariatur dicta nostrum aut est vero nesciunt soluta consequuntur.
Modi, suscipit.

    Lorem ipsum dolor sit amet consectetur, adipisicing elit. Tempore
incidunt error laudantium provident fuga molestias quaerat et mollitia
suscipit exercitationem laboriosam quos magnam placeat, harum aspernatur
expedita, debitis corporis blanditiis?

    Lorem ipsum, dolor sit amet consectetur adipisicing elit. Ea
consectetur nesciunt nulla neque, ipsum, qui aut assumenda impedit officiis
deleniti velit! Incidunt, qui. Cupiditate, corrupti rerum alias animi
suscipit maxime.

 Format code

            Some stuff here       <b>This is bold</b>
        Second line Here

  Lorem ipsum dolor sit amet consectetur adipisicing elit. Commodi delectus nulla animi laboriosam aperiam earum natus illum
minima veritatis, nisi est atque impedit inventore amet tempora, aliquid
accusamus nesciunt in!

Forms

    <form action="">
        <input type="text" value="Hello World"
        placeholder="Enter your name">

        <input type="password" value="password"
        placeholder="Enter Your password"

    </form>

    <form action="" method="GET">

Text in here
    White Space Matters here.

Button
Reset
Submit

This is Value 1
This is Value 2
Selected
by default
This is disabled

    </form>

BACK TO TOP
   ©2022
   &copy 2022
   ©2022

On Mon, Jan 24, 2022 at 9:57 AM Raymond Zhao @.***>
wrote:

I don't see any pictures. If the problem is specific to that one file, I
wonder if you can attach it, assuming it has no confidential information.
Also, does the issue reproduce on VS Code Insiders
https://code.visualstudio.com/insiders/?
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/141253#issuecomment-1020320925,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXIOCERJHMYBRFOVR4JZUVLUXWAIRANCNFSM5MUFW4AQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Kyle Laloge
@.***
780-219-3825

At line 213, you forgot a >. Same with line 69. There's also a tbody tag on line 70 with no matching closing tag.
Leaving out the > characters likely broke Emmet's parser, which is why Emmet wasn't able to suggest or expand abbreviations.

OMG!!!  The closing bracket on line 213 was the culprit!
Thank you very much, I really believed that I had an issue with something I
accidentally toggled, especially with all the code that worked after that
line???
I have a lot to learn still and I really appreciate you taking the time to
help me with this.
Thank you thank you thank you! :)
On Mon, Jan 24, 2022 at 10:43 AM Raymond Zhao @.***>
wrote:

At line 213, you forgot a >. Same with line 69. There's also a tbody tag
on line 70 with no matching closing tag.
Leaving out the > characters likely broke Emmet's parser, which is why
Emmet wasn't able to suggest or expand abbreviations.
[image: Screenshot of line 213]
https://user-images.githubusercontent.com/7199958/150835297-07894b25-141b-4c04-9a99-37aa6dad7291.PNG
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/141253#issuecomment-1020368661,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXIOCETY46VXVK4G2X2IK2TUXWFT5ANCNFSM5MUFW4AQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Kyle Laloge
@.***
780-219-3825


INSTALL: document 'no-ui-console' rather than 'no-ui'

The UI interface itself is never disabled, but the console backend may
be.  'no-ui' is a deprecated backward compatibility alias for
'no-ui-console'.
Fixes #11551

Should this be backported to 1.1.1?

1.1.1 backport in #11554

Merged
301ea192c7e0df8a58bf418a147ee8bdb7f1305e INSTALL: document 'no-ui-console' rather than 'no-ui'

What happens, such as to the commands, if the UI console is disabled?

What happens, such as to the commands, if the UI console is disabled?

The commands will simply not prompt, just tell the user that a password is required, and the user is forced to use the -pass type command options.

My attempted Socratic question was that this should have been in the docs.  not worth a new PR for that.


jorgal and giellatekno are the same api

https://github.com/mikahama/uralicNLP/blob/a89d03fe674bf53d615a58e69463f8f38159b414/uralicNLP/translate.py#L80
doesn't work – there's only one api instance used by both mt-testing and jorgal (http://gtweb.uit.no/apy/)

That used to work as I dug the API urls up myself. If they are nowadays in the same url, maybe its just time to remove the ApertiumJorgalTranslator class altogether...


restructuring repo for DRA drivers

renamed binaries to have a name corresponding to its driver
updated variable BINARIES in Makefile
renamed Dockerfile

Please, give recommendations on what can be improved further here to align with the final goal (getting a general repo for all DRA drivers)

@byako, @uniemimu, @eero-t, @vrantala, PTAL

Was this preparation for adding drivers for other than GPU HW to the same repo?
Some comments:

gpu-kubelet-plugin => kubelet-gpu-plugin
gpu-gas... is tautology as gas=gpuallocationstate
PR was against very obsolete sources (e.g. alert-webhook missing)


Raw beta and LDpred beta are reported for opposite alleles

Hi,
Thanks for all your work developing and maintaining ldpred! I've encountered some uncertainty in interpreting the txt files of LDpred betas that I think is a bug and I'm hoping you can clarify.
In the *__LDpred_p{PS}.txt and *_LDpred-inf.txt results files output by ldpred.py, the raw_beta and ldpred_beta appear to report effects for opposite alleles. Specifically, raw_beta reports the effect of allele nt1 from the input summary statistics, and ldpred_beta reports an estimated effect for allele nt2.
Some motivating observations in this direction:

Most of the time raw_beta and ldpred_beta have opposite signs
That sign discordance between the betas occurs most reliably for the most significant SNPs in the input GWAS (i.e. the cases where LDpred estimation would be least expected to end up flipping the sign due to conditioning on surrounding SNPs).
Comparison to the input GWAS shows the raw_beta consistently matching the input effect size for allele nt1, including correctly tracking cases where alleles had to be swapped to match the LD reference panel.
PRS scoring with the ldpred_beta and allele nt1 leads to scores negatively correlated with the expected phenotype (vs. trending in the expected direction when scoring nt2)

As far as I can tell, this is caused by how coord_genotypes.py sets up tracking both the raw GWAS effect size (stored in chrom_dict[chrom]['log_odds'], e.g. line 316) vs. the standardized effect size derived from the p-value (chrom_dict[chrom]['betas'], e.g. line 317-319). The key element is that the signed standardized effect is computed as:
beta = sp.sign(raw_beta) * stats.norm.ppf(pval / 2.0)

which appears to be intended to maintain the sign of the input effect while getting a Z score from the GWAS p-value. But stats.norm.ppf(pval / 2.0) will always be negative (given p/2 will always lead to a quantile in the lower tail of the normal distribution), so instead of matching sign(raw_beta) the standardized beta used for input to LDpred modeling will always be sign-flipped from the raw beta. This then carries through and shows up as being equivalent to having scores for nt2 from LDpred in the output.
I guess my question then is:

Is this a correct understanding of current LDpred output?
Is this intentional? (I imagine not, but possible I've missed some other motivation)
If it's a bug, is there a good route to fixing it? Correcting the stats.norm.ppf() calculation seems simple enough, but I'd worry about breaking people's existing workflows (thus why I'm not just submitting a pull request). Perhaps better to flip raw_beta to also be the effect of allele nt2? Or at worst just provide some clear documentation of how the two betas are oriented?
[edit to add:] 4) I haven't checked how the sign flip impacts the validate.py script downstream. Probably worth looking at, and may inform the best course of action for fixes, if any?

Cheers,
Raymond
(cc @ce-carey who helped identify this)

Assuming this was unintentional, @liangyy has caught this in their fork and has addressed it here. This seems like a sensible solution.
In fact, @liangyy has identified at least one other potential bug and added quite a few niceties which @bvilhjal might want to consider merging.

I apologise for my very slow response, but I left science (for now) and haven’t really maintained LDpred (nor responded to LDpred emails) for the last couple of years.
Thanks for reporting this issue!
The reason for the sign difference is that some people use logistic reg effect estimates and others linear reg effect estimates in their summary stats.  Depending on what you use, you end up with opposite signed risk scores.
Regarding stats.norm.ppf, as (pval/2.0)<=0.5 then stats.norm.ppf(pval/2.0)<=0.  Hence taking the absolute will only change the sign of the effect.  However, if that is what you want, then that’s good 😊
I'll close this issue for now, but if you have good ideas for how to improve the code, please do so, and consider submitting a pull request with the changes to update the repository.
Best,
Bjarni


Add RAYMATH_STATIC_INLINE for raymath.h usage

Fixes #171

Tested on the same setup as #171 , we removed our temporary fix from last time and used the new release and got no errors this time.
I guess it worked, thank you !


New release planning

Hello,
Could you please tell me if there is a new release planned? The last one 0.61 is like 3 years old. I would be interested in this fix: https://github.com/rbock/sqlpp11/commit/9412851408e7bc290732b56d7a5e76c6942d49d8
Thanks!

Thanks for poking :-)
Done.


Create backup.py

Fixing a bug related to form validation messages not displaying correctly.
Trac ticket number

ticket-XXXXX
Branch description
Provide a concise overview of the issue or rationale behind the proposed changes.
Checklist

[ ] This PR targets the main branch.
[ ] The commit message is written in past tense, mentions the ticket number, and ends with a period.
[ ] I have checked the "Has patch" ticket flag in the Trac system.
[ ] I have added or updated relevant tests.
[ ] I have added or updated relevant docs, including release notes if applicable.
[ ] I have attached screenshots in both light and dark modes for any UI changes.

Please fill out the PR template next time to help others understand your contribution.


wrong doc about INTERNET_STATUS_CONNECTING_TO_SERVER

document about the following two status of OnStatusCallback  is wrong
INTERNET_STATUS_CONNECTING_TO_SERVER
INTERNET_STATUS_CONNECTED_TO_SERVER
the doc indicates that lpvStatusInformation is a pointer of SOCKADDR struct, while it's not, I have tested it on vc6 and VS2015,  lpvStatusInformation  is an IP string, like "<IP_ADDRESS>"

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

ID: def91c87-caa5-c44d-c0c5-b74e401b8cd4
Version Independent ID: 7791d315-a5ea-365a-48f8-ee2517faed2b
Content: CInternetSession Class
Content Source: docs/mfc/reference/cinternetsession-class.md
Product: visual-cpp
GitHub Login: @mikeblome
Microsoft Alias: mblome

@tongzhipeng In C:\Program Files (x86)\Microsoft Visual Studio\2017\Community\VC\Tools\MSVC\14.16.27023\atlmfc\src\mfc\inet.cpp I see this code which tells me the document is correct. The call to TRACE shows only the sa_data member, which perhaps is IP string?
case INTERNET_STATUS_CONNECTING_TO_SERVER:
	{
	sockaddr* pSockAddr = (sockaddr*) lpvStatusInformation;
	TRACE(traceInternet, 1, _T("connecting to socket address '%Ts'\n"), pSockAddr->sa_data);
	}
	break;

I haven't test it on VS2017,  the sockaddr struct definition is:
typedef struct sockaddr {
u_short sa_family;
char    sa_data[14];
} SOCKADDR, *PSOCKADDR, *LPSOCKADDR;
which means the first two bytes should be sa_family,  my test result shows lpvStatusInformation just starts with ip, not the sa family value.

@tongzhipeng Sorry, I don't have expertise to know what's happening here, and the topic seems correct based on mfc source. Maybe something in Windows DLL is the root cause? Can you raise the issue on the C++ Developer Community forum

@mikeblome  could i just close the issue?

yes let's close here and please ask on the Developer Community. Someone there should be able to answer.


change mgmt_ipv4/6 to mgmt-ipv4/6

Besides changing dash to underscore for ipv{4,6}_subnet via #1365.
I think the mgmt_ipv4/6 config stanza is the last remaining underscore topology setting.
I think, it would make sense to bundle this and #1365 in a single release, to have such breaking change just once.
This will in then in my view be enough to close #749

Yes, these changes definitely must go together.
Besides that, I think we need to create a special error message notifying people to check release notes when unmarshalling error happens for unrecognized fields

That would be here then.
https://github.com/srl-labs/containerlab/blob/main/clab/file.go#L76
What do you propose as the message to show?

PS. Currently I already see that this message is emitted:
 dclab des -c -t private/linux.clab.yml
Error: failed to read topology file: yaml: unmarshal errors:
  line 5: field ipv4-subnet not found in type types.MgmtNet
  line 6: field ipv4-range not found in type types.MgmtNet


MSFT_SPContentDatabase: Checking existing database always throw exception

Details of the scenario you tried and the problem that is occurring
Repo step:

Create new content database passing only name and webapplication
Ensure database is created
Run same script again
Get exception:
Specified database server does not match the actual database server. This resource cannot move the database to a different SQL instance.

Verbose logs showing the problem
Set-ContentDatabase -Name $ContentDatabaseName -WebAppUrl $WebApplication -Ensure Present -Debug
VERBOSE: Executing as the local run as user user
VERBOSE: Leaving BeginProcessing Method of Get-SPDatabase.
VERBOSE: Leaving ProcessRecord Method of Get-SPDatabase.
VERBOSE: Leaving EndProcessing Method of Get-SPDatabase.
VERBOSE: Leaving BeginProcessing Method of Get-SPWebApplication.
VERBOSE: Leaving ProcessRecord Method of Get-SPWebApplication.
VERBOSE: Leaving EndProcessing Method of Get-SPWebApplication.
Specified database server does not match the actual database server. This resource cannot move the database to a different SQL instance.
At line:32 char:21
+ ...             throw ("Specified database server does not match the actu ...
+                 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : OperationStopped: (Specified datab...t SQL instance.:String) [], RuntimeException
    + FullyQualifiedErrorId : Specified database server does not match the actual database server. This resource cannot move the database to a different SQL instance.

Here Set-ContentDatabase is Set-TargetResource. I just call it explicitly from resource file.
Suggested solution to the issue
if ($cdb.Server -ne $params.DatabaseServer)
                {
                    throw ("Specified database server does not match the actual database " + `
                            "server. This resource cannot move the database to a different " + `
                            "SQL instance.")
                }

This validation is always throw exception if I do not pass DatabaseServer param to resource (and I really don't wanna pass it).
The DSC configuration that is used to reproduce the issue (as detailed as possible)
# insert configuration here

The operating system the target node is running

Version of SharePoint that is used (e.g. SharePoint 2016)
Version and build of PowerShell the target node is running

Version of the DSC module that was used

We should not make the database server parameter mandatory, because of the fact that if you don't specify a database parameter in New-SPContentDatabase, the database is automatically created on the default database server.
On the other side, we also should not check against the default database server, because it is possible that the default database server is changed after creating the database. Automatically checking against the default database server would mean the resource is no longer in compliance.
The only good option in my opinion is to ignore the database server parameter if it is not supplied. So something like:
if ($params.ContainsKey('DatabaseServer') -and $cdb.Server -ne $params.DatabaseServer)

@ruandersMSFT Since this this update isn't a breaking change, we do not have to target that to v4.0. Could you create a new PR based on the master branch, which adds this check?

Tested and verified with both creating on new SPContentDatabase and re-run against an existing SPContentDatabase.

@ruandersMSFT Did you submit a PR with this change already? I cannot find it anywhere

@ykuijs Not yet.  Code change done, was going to look to add a test or two.  When are you planning to release v3.8?

As soon as possible. I would like to merge all current PRs and implement one more update to the new CI system. Hopefully later this week/early next week.


Adding support for multi-modal file input for Vertex AI

Context and Motivation
The existing vertexai_message_parser function only supports messages with either string content or a list of string content. This limits the flexibility of the Vertex AI client, as it cannot handle messages containing both strings and gm.Part objects, which are necessary for including images, videos, documents, and audio files.
This PR introduces support for providing Part objects, enhancing the flexibility and functionality of the Vertex AI client.
Summary of Changes

Modified vertexai_message_parser to handle messages with mixed content types, including strings and gm.Part objects.
Added unit tests to ensure the correct parsing of messages with mixed content types.

Detailed Explanation
The updated vertexai_message_parser now checks for list content, and if found, iterates through the list items. For each item, it checks if it's a string or a gm.Part object. If it's a string, it converts it to a gm.Part object. Otherwise, it simply appends the existing gm.Part object to the list of parts. This ensures that the final gm.Content object contains a list of gm.Part objects, correctly representing the message content.
Testing
New unit tests have been added to tests/llm/test_vertexai/test_message_parser.py to cover the following scenarios:

Parsing a message with string content.
Parsing a message with a list of string content.
Parsing a message with a list containing both strings and gm.Part objects.
Handling invalid content types.
Handling invalid items in a list.

Relevant Links

Issue #946
Image understanding  |  Generative AI on Vertex AI

:rocket:
This description was created by Ellipsis for commit 8de32adcfab67844aa6c50cccdf9c28fcef2656b

Summary:
Enhanced vertexai_message_parser to support mixed content types, adding tests for validation.
Key points:

Updated vertexai_message_parser in instructor/client_vertexai.py to handle mixed content types: strings and gm.Part objects.
Added error handling for unsupported content types in vertexai_message_parser.
Modified _vertexai_message_list_parser to support the updated vertexai_message_parser.
Added unit tests in tests/llm/test_vertexai/test_message_parser.py for various content scenarios, including mixed content types.
Added integration tests in tests/llm/test_vertexai/test_modes.py to validate mixed content handling in real scenarios.

Generated with :heart: by ellipsis.dev

👍 Looks good to me! Reviewed everything up to 8de32ad in 10 seconds
More details

Looked at 142 lines of code in 3 files
Skipped 0 files when reviewing.
Skipped posting 4 drafted comments based on config settings.

instructor/client_vertexai.py:36
instructor/client_vertexai.py:57
instructor/client_vertexai.py:35
tests/llm/test_vertexai/test_message_parser.py:1
Workflow ID: wflow_mRRx9diPpEgJ56lq

You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.

Addressed (1) and (2) in https://github.com/jxnl/instructor/pull/947/commits/0c37e7e3cf83dfe71e9f7e79ee659c18d59bf5c8

can you just post a screenshot of the vertex tests passing? I dont have a tests right now that are running on CI and im AFK.
Cc: @ivanleomk can you own this merge?

@jxnl Sure thing, will allocate some time to this, just need to understand what the PART is useful for

@smwitkowski this is the current status of the tests for me
tests/llm/test_vertexai/test_message_parser.py ....                                                                             [ 15%]
tests/llm/test_vertexai/test_modes.py ......                                                                                [ 38%]
tests/llm/test_vertexai/test_retries.py ....                                                                                [ 53%]
tests/llm/test_vertexai/test_simple_types.py ..FF.F..                                                                           [ 84%]
tests/llm/test_vertexai/test_stream.py FF..                                                                                [100%]

Were you able to get all of the tests passing on your end?

@ivanleomk - after rebasing with the changes made in #955 (thanks!) all my tests are passing.

Allowing for Part types enables users to take advantage of Gemini's multi-modal capabilities. With this change, they can provide documents, images, audio, and video files to Gemini, in addition to plain text.  Google uses Part to facilitate the transport of multi-modal content.
https://cloud.google.com/vertex-ai/generative-ai/docs/reference/python/latest/vertexai.generative_models.Part

Awesome! I spent yesterday fixing some bugs and now it should work. I tested it with the following code snippets
import instructor
import vertexai.generative_models as gm  # type: ignore
from pydantic import BaseModel, Field
import requests

client = instructor.from_vertexai(gm.GenerativeModel("gemini-1.5-pro-001"))
content = [
    "Order Details:",
    gm.Part.from_text("Customer: Alice"),
    gm.Part.from_text("Items:"),
    "Name: Laptop, Price: 999.99",
    "Name: Mouse, Price: 29.99",
]

class Item(BaseModel):
    name: str
    price: float

class Order(BaseModel):
    items: list[Item] = Field(..., default_factory=list)
    customer: str

resp = client.create(
    response_model=Order,
    messages=[
        {
            "role": "user",
            "content": content,
        },
    ],
)

client = instructor.from_vertexai(
    gm.GenerativeModel("gemini-1.5-pro-001"), mode=instructor.Mode.VERTEXAI_JSON
)
content = [
    gm.Part.from_text("Count the number of objects in the image."),
    gm.Part.from_data(
        bytes(
            requests.get(
                "https://img.taste.com.au/Oq97xT-Q/taste/2016/11/blueberry-scones-75492-1.jpeg"
            ).content
        ),
        "image/jpeg",
    ),
]

class Description(BaseModel):
    description: str

resp = client.create(
    response_model=Description,
    messages=content,
)

print(resp)

These both seem to work nicely. @smwitkowski I think this looks good to merge. Could I trouble you to add one more test for image understanding ( just make sure to use the JSON mode, function calling doesn't support it for now ) and we can merge it in?