The output conversion file name does not reflect the conversion format

Hi, I'm having trouble converting the image format.
I have a conversion that converts images to WebP format. Here is my config file:
    'image_conversions' => [
        '*' => [
            'thumbnail' => [
                'format' => Manipulations::FORMAT_WEBP,
                'width' => 150,
                'height' => 150,
                'fit' => Manipulations::FIT_CROP,
            ],
        ],
    ],

However, when I input an image named sample.png, the resulting conversion file is named sample.thumbnail.png instead of sample.thumbnail.webp in the conversions directory. Despite the file extension being PNG, I have confirmed that the binary content of the converted image is indeed in WebP format.
Then I looked into the source code and found that the extension for the image conversion is always taken from the original media's extension, resulting in the incorrect file extension.

Is this the expected behavior? If not, I think the correct way to get conversion file names should be something like the following:
    public function getConversionFileName(Media $media, $conversionName)
    {
        [$fileName, $extension] = FileHelpers::splitNameAndExtension($media->file_name);

        $conversion = (MediaHub::getConversionForMedia($media))[$conversionName];
        $extension = isset($conversion['format']) ? $conversion['format'] : $extension;

        $fileNamer = MediaHub::getFileNamer();
        return $fileNamer->formatConversionFileName($fileName, $extension, $conversionName);
    }

Let me know what you think and looking forward to hearing from you!
Versions:
PHP 8.3
Nova 4.32.9
Nova Media Hub: 1.7.1

Same problem here. Would like to use the conversion to include webp but the file does not get renamed.


GLoader3D::onChangeSpine uses wrong spine API with cocos2d-x 4.0

GLoader3D::onChangeSpine uses incompatible calls to the spine AnimationState. The spine API in cocos2d-x 4.0 is quite different so the errors lead to the compilation errors in FairyGUI SDK.

You need to upgrade spine-runtime to 3.8 manually.


display erc1155 transfers

because of the way erc1155 transfers work, they do not appear in the list of account transactions returned by the indexer. thus some significant massaging will be required in order to display them in the same list. there is also a performance hit when using the transfers endpoint due to the many joins it has to do.
ultimately it may make more sense to have a page dedicated to showing a detailed list of token transfers separate from the transactions page similar to other explorers
Consider

Define the interface and mock it until the backend is ready.
Create an issue to swap to the real API.

From telegram : Ez, [Jul 26, 2023 at 1:09:11 PM]:
also something to think about, do we want to do anything special for 1155 transfers on the wallet > transfers tab? like include a link to the NFT detail page, or show a thumbnail, anything like that?
also 1155 doesn't have a symbol like erc20 i believe (i.e. there is no cognate of "TLOS" for an 1155) so we need to think about how to display that for a transfer
we could show the NFT name there instead of symbol, and limit it to x number of characters (since the nft name field can be arbitrarily long, and definitely won't fit in many cases)


json() function in guidance library generates content only in English, can't output in other languages

I've been using the json() function from the guidance library, and while it generates content that adheres to the provided JSON schema, the output is always in English. I'm unable to generate JSON content in other languages, such as Chinese.
Steps to reproduce:
with assistant():
with guidance.silent():
lm += json(name='res2', schema=json_schema)
resall.append(lm['res2'])
Even after trying various prompts to instruct the model to generate content in Chinese, the output remains in English. It seems that the json() function does not currently provide an option to control the language of the generated content.
My main questions are:
Is there a way to make the json() function generate content in a language other than English (e.g., Chinese)?
If it's not currently supported, are there any plans to add support for multi-language content generation in the future?
I would appreciate any clarification or potential solutions regarding this issue. Thank you for developing such a powerful tool!

@

@Harsha-Nori @hudson-ai  Do we have any plan to add multilingual support ? This is something I have in mind since a while and I had a prototype implementation in transformers-cfg for unicodes.  I'd be happy to contribute to this, though I wouldn’t be able to start immediately due to our ongoing work.

Curious to know as well

I looked at the source code, and it seems to be related to regular expression matching. English characters are one byte, while Chinese characters are two bytes.

Thanks a bunch for bringing this to our attention.
Will definitely dig into this -- I suspect you're right that the regular expressions for JSON strings are a bit naive and are causing the problem.
Would you mind sending a short snippet of JSON that the model should be able to produce but is unable to? Appreciate it!

{
        "document_id": 404,
        "causality_list": [
            {
                "causality_type": "直接",
                "cause": {
                    "actor": "移动卫星公司",
                    "class": "科技发展",
                    "action": "获得",
                    "time": "",
                    "location": "",
                    "object": "美国专利和商标办公室颁发的第7181181号专利"
                },
                "effect": {
                    "actor": "该专利",
                    "class": "科技发展",
                    "action": "覆盖",
                    "time": "",
                    "location": "",
                    "object": "多频带/多模式卫星无线电话通信系统的关键技术"
                }
            },
            {
                "causality_type": "直接",
                "cause": {
                    "actor": "该专利",
                    "class": "科技发展",
                    "action": "覆盖",
                    "time": "",
                    "location": "",
                    "object": "多频带/多模式卫星无线电话通信系统的关键技术"
                },
                "effect": {
                    "actor": "公司",
                    "class": "科技发展",
                    "action": "决定使用",
                    "time": "",
                    "location": "",
                    "object": "此技术建造下一代先进卫星系统"
                }
            }
        ]
    }

The above is the data I want to generate. I used the following JSON schema but was unable to get the Chinese content, only the English version.
{
  "type": "object",
  "properties": {
    "causality_list": {
      "type": "array",
      "items": {
        "type": "object",
        "properties": {
          "causality_type": {
            "type": "string",
            "pattern": ".{1,10}"
          },
          "cause": {
            "type": "object",
            "properties": {
              "actor": {
                "type": "string",
                "pattern": ".{1,10}"
              },
              "class": {
                "type": "string",
                "pattern": ".{1,10}"
              },
              "action": {
                "type": "string",
                "pattern": ".{1,10}"
              },
              "time": {
                "type": "string",
                "pattern": ".{1,10}"
              },
              "location": {
                "type": "string",
                "pattern": ".{1,10}"
              },
              "object": {
                "type": "string",
                "pattern": ".{1,10}"
              }
            }
          },
          "effect": {
            "type": "object",
            "properties": {
              "actor": {
                "type": "string",
                "pattern": ".{1,10}"
              },
              "class": {
                "type": "string",
                "pattern": ".{1,10}"
              },
              "action": {
                "type": "string",
                "pattern": ".{1,10}"
              },
              "time": {
                "type": "string",
                "pattern": ".{1,10}"
              },
              "location": {
                "type": "string",
                "pattern": ".{1,10}"
              },
              "object": {
                "type": "string",
                "pattern": ".{1,10}"
              }
            }
          }
        }
      }
    }
  }
}

By the way, how should this issue be resolved? I hope to use Guidance in an upcoming data mining competition. It's a great tool.

Hey @chengmonk I did some testing on my side. Guidance has recently undergone some big changes to JSON handling, none of which are in the last official release (0.1.16), and only some of which are in the latest pre-release (0.2.0rc1). One such unreleased improvement is full (well, fingers crossed 😅) unicode support inside of JSON strings.
If you're daring, I'd suggest installing guidance directly from the main branch (pip install git+https://github.com/guidance-ai/guidance).
If you do so, I'd really appreciate any feedback. But I believe it should solve your problem, with the caveat that some of the strings in your example above are longer than 10 characters and are therefore invalid under your schema. There is another known issue on the main branch that has to do with how the LLM handles whitespace in JSON, but I am looking to resolve this in the next few days. Just FYI in case you run into anything weird.
Hoping for another official release soon. Please let us know if you need any more help -- I'm really happy you're enjoying guidance as a tool and want to use it in a competition! Exciting!

@Saibo-creator maybe you can try as well? :)

Works well on my side with the current main branch @917fe353bf
import guidance

lm = guidance.models.Transformers(model="microsoft/Phi-3.5-mini-instruct")

lm.echo = False

json_schema = {
    "type": "object",
    "properties": {
        "姓名": {
            "type": "string"
        },
        "年龄": {
            "type": "integer"
        }
    },
    "required": ["姓名", "年龄"],
    "additionalProperties": False
}

lm = lm + guidance.json(name="unicode_json", schema=json_schema)

print(lm["unicode_json"])

I got
{
    "姓名": "张三",
    "年龄": 25
}

Fantastic, thanks for checking!
Note to self: add explicit unicode tests to the JSON test suite.

That's so cool, it's working fine now. Truly a great tool.


Trusted key issue attempting to apply to host in Tailscale tailnet

hello :wave: thanks for building this tool :) I wanted to give it a go for managing some miscellaneous boxes I have attached to my Tailscale tailnet.  I am trying to use Colmena to push to one such box as a test, but am running into a problem, something to do w/ a lack of trusted keys.
My flake
{
  nixConfig = {
    extra-substituters = [
      "https://colmena.cachix.org"
    ];
    extra-trusted-public-keys = [
      "colmena.cachix.org-1:7BzpDnjjH8ki2CT3f6GdOk7QAzPOl+1t3LvTLXqYcSg="
    ];
  };
  inputs = {
    nixpkgs.url = "github:NixOS/nixpkgs/nixos-unstable";
  };
  outputs = {nixpkgs, ...}: {
    colmena = {
      meta = {
        nixpkgs = import nixpkgs {
          system = "x86_64-linux";
          overlays = [];
        };
      };

      cloudy = {
        deployment = {
          targetUser = "cloudy";
        };
        imports = [./configuration.nix];
      };
    };
  };
}

This seems to build fine with colmena build.  However, on attempting to perform the apply command, I get the following
[ERROR] Failed to push system closure to cloudy - Last 20 lines of logs:
[ERROR]   stderr) copying path '/nix/store/rismgwk2zj5087248w179c2zskhg5rv3-unit-serial-getty-.service' to<EMAIL_ADDRESS>[ERROR]   stderr) copying path '/nix/store/33jgjqgc7rxq0pzsq1ymj06s2cxd521d-system-units' to<EMAIL_ADDRESS>[ERROR]   stderr) copying path '/nix/store/3bx9jah2snamfn9p4y9dyb347q3xygvd-udev-rules' to<EMAIL_ADDRESS>[ERROR]   stderr) copying path '/nix/store/4kfr14vil0ry3ys3nw86sx8qqdnc0dqc-system-shutdown' to<EMAIL_ADDRESS>[ERROR]   stderr) copying path '/nix/store/css7wvp4qq8yckkb797cwl7lvhc7i52r-locale.conf' to<EMAIL_ADDRESS>[ERROR]   stderr) copying path '/nix/store/hh8mvnm6z8mzg9k6n867mbzqbmh6va10-hwdb.bin' to<EMAIL_ADDRESS>[ERROR]   stderr) copying path '/nix/store/n8x9khlzvv3hbn6083sfckhsc7br7bxd-hosts' to<EMAIL_ADDRESS>[ERROR]   stderr) copying path '/nix/store/qb39bhgdzmr07rj3b8679kr69nvph5a7-etc-fstab' to<EMAIL_ADDRESS>[ERROR]   stderr) copying path '/nix/store/qdd4q4zvfylz1vdh01v4xqwpsbna84z6-etc-pam-environment' to<EMAIL_ADDRESS>[ERROR]   stderr) copying path '/nix/store/r7rb76f880ifd0lvsp2zj5ba0xyxp2ma-system-generators' to<EMAIL_ADDRESS>[ERROR]   stderr) copying path '/nix/store/vg7k92ndkzk8zbkgivnjskm9qx9p0x5x-etc-hostname' to<EMAIL_ADDRESS>[ERROR]   stderr) copying path '/nix/store/vn7x0gfkvppj2d50ndpb49manqwga525-user-units' to<EMAIL_ADDRESS>[ERROR]   stderr) copying path '/nix/store/y7pbprhqxhk01j2xhrz9hqmyqkfp6xsd-etc-profile' to<EMAIL_ADDRESS>[ERROR]   stderr) copying path '/nix/store/s7ahj52jjliyzfa5hf1awhqic3w35d7x-etc' to<EMAIL_ADDRESS>[ERROR]   stderr) copying path '/nix/store/5gg8ndkb1y88srwnmmb59iqm8gsrwm0z-initrd-fsinfo' to<EMAIL_ADDRESS>[ERROR]   stderr) copying path '/nix/store/hkp5yna00r2c0q3h6gliph44v9x0ia5n-stage-1-init.sh' to<EMAIL_ADDRESS>[ERROR]   stderr) copying path '/nix/store/wxhhrdns3wjj6kwd78293p5ha98nwjsq-initrd-linux-6.6.22' to<EMAIL_ADDRESS>[ERROR]   stderr) copying path '/nix/store/18vjlisy7c34j9cbj3hj6ffgsvw5gg7q-nixos-system-cloudy-24.05pre-git' to<EMAIL_ADDRESS>[ERROR]   stderr) error: cannot add path '/nix/store/03z5f3spgpfhxihyq3hdxh8a7l1sgp4i-linux-6.6.22-modules-shrunk' because it lacks a signature by a trusted key
[ERROR]  failure) Child process exited with error code: 1
[ERROR] Failed to complete requested operation - Last 1 lines of logs:
[ERROR]  failure) Child process exited with error code: 1
[ERROR] -----
[ERROR] Operation failed with error: Child process exited with error code: 1
Hint: Backtrace available - Use `RUST_BACKTRACE=1` environment variable to display a backtrace

I'm not clear what "key" is lacking here, my gut is telling me that it has something to do with Tailscale handling the authentication for the SSH connection, rather than a keypair available on the host, but, could be completely wrong :)
If anyone has any insight / ideas, would appreciate it!

It means the targetUser isn't trusted and the paths that you built on the deployer machine aren't signed. The simplest solution is to either deploy as root or mark the user as trusted by the nix-daemon (note that this is effectively root-equivalent).

nice :) that, with the addition of security.sudo.wheelNeedsPassword = false; seems to have gotten me to a working colmena deployment.
It still seems to bork out at the tail end of the deployment
cloudy ✅ 4s Evaluated cloudy
cloudy ✅ 0s Built "/nix/store/skqsx2wx8k4gh0lc67g492222yk5xl4k-nixos-system-cloudy-23.11pre-git"
cloudy ✅ 0s Pushed system closure
cloudy ❌ 61s Activation failed: Child process exited with error code: 4
[ERROR] Failed to complete requested operation - Last 1 lines of logs:
[ERROR]  failure) Child process exited with error code: 4
[ERROR] Failed to deploy to cloudy - Last 20 lines of logs:
[ERROR]   stderr) setting up tmpfiles
[ERROR]   stderr) warning: the following units failed: NetworkManager-wait-online.service
[ERROR]   stderr)
[ERROR]   stderr) × NetworkManager-wait-online.service - Network Manager Wait Online
[ERROR]   stderr)      Loaded: loaded (/etc/systemd/system/NetworkManager-wait-online.service; enabled; preset
: enabled)
[ERROR]   stderr)     Drop-In: /nix/store/21z081cir6bksm67mlcw4sp8yb1chlkh-system-units/NetworkManager-wait-on
line.service.d
[ERROR]   stderr)              └─overrides.conf
[ERROR]   stderr)      Active: failed (Result: exit-code) since Thu 2024-03-28 20:10:09 EDT; 58ms ago
[ERROR]   stderr)    Duration: 3min 56.052s
[ERROR]   stderr)        Docs: man:NetworkManager-wait-online.service(8)
[ERROR]   stderr)     Process: 10264 ExecStart=/nix/store/xkg9pr1lgcn5c2869pc4kxhgpc60xc4b-networkmanager-1.44
.2/bin/nm-online -s -q (code=exited, status=1/FAILURE)
[ERROR]   stderr)    Main PID: 10264 (code=exited, status=1/FAILURE)
[ERROR]   stderr)          IP: 0B in, 0B out
[ERROR]   stderr)         CPU: 16ms
[ERROR]   stderr)
[ERROR]   stderr) Mar 28 20:09:09 cloudy systemd[1]: Starting Network Manager Wait Online...
[ERROR]   stderr) Mar 28 20:10:09 cloudy systemd[1]: NetworkManager-wait-online.service: Main process exited,
code=exited, status=1/FAILURE
[ERROR]   stderr) Mar 28 20:10:09 cloudy systemd[1]: NetworkManager-wait-online.service: Failed with result 'e
xit-code'.
[ERROR]   stderr) Mar 28 20:10:09 cloudy systemd[1]: Failed to start Network Manager Wait Online.
[ERROR]  failure) Child process exited with error code: 4
[ERROR] -----
[ERROR] Operation failed with error: Child process exited with error code: 4
Hint: Backtrace available - Use `RUST_BACKTRACE=1` environment variable to display a backtrac
``

but it seems like the deploy does work, and Nix for some reason is just hanging at the service restart.  Likely just something wonky with my config.

Really appreciate the help!! gonna close this ticket


Pad percentage outputs

This is just a quick one - I love this wee plugin!
How would I go about padding the CPU and MEM % outputs?  I'd like them to always take up the same amount of space.

This should fix the problem (more or less like #17, but applied manually after the installation):
SYSSTAT_SCRIPTS_DIR="$HOME"/.tmux/plugins/tmux-plugin-sysstat/scripts
cd "$SYSSTAT_SCRIPTS_DIR"
sed -i 's/%.[01]f%%/%2.0f%%/g' {cpu,mem,swap}.sh

SYSSTAT_SCRIPTS_DIR is where the files cpu.sh, mem.sh and swap.sh are located.
Cheers

Thanks for taking a look. I tried it out and it works. The only downside that I see for this fix is that if CPU usage is low (less than 10%) the script is still reserving a 2 character space even if you remove the floating point, so you end up with
Original:   "CPU: 5.5%"
Modified: "CPU:  5%"
(Ideally I think it should be "CPU: 5%", without the extra space)
But it's a really particular use case.

@tuv7041 Actually the padding was intentional, as requested in this issue: as the CPU usage percentage varies from X to XX, it causes the status widget to change the width a lot, what I think is very disturbing, probably why it was requested by OP.
Cheers

oh, sorry, my bad. I commented here a long time ago and forgot about the context (I was more focused on the decimal than the padding). Thanks again for fixing it.


How many system threads to expect?

I've configured gubernator to start 1 worker via env var.
I've confirmed that gubernator logs "starting 1 workers".
However, pstree output didn't seem to change, with 11 or 12 threads running in total.
I guess perhaps some of these are golang deps, like dns?
How can I validate that the worker setting is really in effect?

I think it's a red herring.
I don't have any gubernator traffic at the moment, and system threads as well as goroutine dump only show the golang innards, and not workers.
Workers limited to 1
root@hexanator-0:/# cat /proc/23/environ | tr \\0 \\n | grep GUBER
GUBER_WORKER_COUNT=1
root@hexanator-0:/# pstree
pebble-+-gubernator---11*[{gubernator}]
       `-12*[{pebble}]

Workers not limited, 8 are started per cpu detection
root@hexanator-0:/# cat /proc/38702/environ | tr \\0 \\n | grep GUBER
root@hexanator-0:/# pstree
pebble-+-gubernator---9*[{gubernator}]
       `-12*[{pebble}]

I've confirmed that GUBER_WORKER_COUNT is honoured by doing a SIQUIT goroutine dump after hammering the service with concurrent traffic


Optional prop by default for Option<T>

For components with many fields, often many of these are optional. It would be a big ergonomic win to forego these when calling the component. Props can be layed out like so:
#[derive(Prop)]
struct Props {
    name: String,
    age: i32,
    custom_style: Option<String>, // Optional
}

And since custom_style is an Option, if the user doesn't put this in the component call, it will be set to None, otherwise any value passed in will be wrapped in Some.
If we still want to allow users to use normal Options with normal functionality, perhaps this special functionality can be enabled with a flag:
#[derive(Prop)]
struct Props {
    name: String,
    age: i32,
    #[optional]
    custom_style: Option<String>, // Optional
}

Dioxus allows for optional props with their #[inline_props] macro, so look to them for reference.

You can do this:
#[derive(Prop)]
struct Props {
    name: String,
    age: i32,
    #[builder(default)]
    custom_style: Option<String>, // Optional
}

#[builder(default)] used the default if not given.

I agree, and if it's by default, then I would think any args passed  in would automatically be wrapped in a Some(), rather than each arg passed requiring the Some() wrapping


Twemproxy going OOM

Hi,
We're running twemproxy behind set of memcached servers. Below's the obfuscated config
some_pool:
  listen: <IP_ADDRESS>:11213
  hash: crc16
  distribution: ketama
  backlog: 4000
  timeout: 30
  preconnect: true
  server_connections: 200
  auto_eject_hosts: false
  server_retry_timeout: 2000
  servers:

   - some_host_port:1 k1
   - some_host_port:1 k2
   - some_host_port:1 k3
   - some_host_port:1 k4
   - some_host_port:1 k5
   - some_host_port:1 k6
   - some_host_port:1 k7
   - some_host_port:1 k8
   - some_host_port:1 k9
   - some_host_port:1 k10
   - some_host_port:1 k11
   - some_host_port:1 k12

Clients connected to Twemproxy issue a single GET command
The throughput is constant and is ~ 2.2K QPS
No: of clients connected to twemproxy is fairly low number < 100
No: of items in In Queue(and in_queue_bytes) is very less

At some point twemproxy started using very large amount of memory. Also saw the connection timeouts around the same time: close s: Connection timed out in debug logs
Memory consumption grows high until Kernel kills the nutcracker instance

@charsyam
Can you please have a look @ my question ^^^. Also, we've added more capacity and the performance has come back to normal. But yet, we're still observing high memory consumption and eventual death of twemproxy process(but, not so quite often as earlier)

@auror sorry to reply late.
Twemproxy will close connection when server send incorrect reply.
and drop the buffer for it.
and sometimes later, it will try to add server again to use.

Np. So, ho do we debug this? Do you need any additional info? Any clue/direction would help.


chore: remove some double spaces

There many occurrences of two spaces after a period, entire sections are written in this style. I'm not sure if we want to make that more consistent, \.  [A-Z] (with 2 spaces) has 523 matches, while \. [A-Z] (which 1 space) has 16529 matches, so clearly this is more common.

Is this awaiting-review?

Yes! awaiting review now; I added a few more.

bors r+
Thanks!


Digital Ocean Spaces support?

Hi there,
I'm trying to use the keystone-storage-adapter-s3 for my File field with my S3 Digital Ocean Spaces, but no matter what I try, I keep on getting Field errors when trying to save my model...
import keystone from 'keystone';
const Types = keystone.Field.Types;

const s3Storage = new keystone.Storage({
	adapter: require('keystone-storage-adapter-s3'),
	s3: {
		// endpoint    : 'ams3.digitaloceanspaces.com',
		key         : process.env.S3_KEY,
		secret      : process.env.S3_SECRET,
		bucket      : process.env.S3_BUCKET,
		region      : process.env.S3_REGION, // -> ams3
		path        : 'uploads',
		uploadParams: {
			ACL     : 'public-read',
		}
	},
	schema: {
		bucket: true,
		etag  : true,
		path  : true,
		url   : true
	}
});

const MyModel = new keystone.List('MyModel ', {
	map: {
		name: 'title'
	}
});

MyModel.add({
	fileUpload: {
		type        : Types.File,
		label       : 'Upload file',
		storage     : s3Storage,
		initial     : true,
		required    : true
	}
});

Is this adapter not compatible with other endpoints then AWS?
I found this post that I have to set the endpoint:
https://www.digitalocean.com/community/questions/how-to-use-digitalocean-spaces-with-the-aws-s3-sdks
But I don't find any reference to that In the adapter's code anywhere?
Thanks!

I ended up making my own fork which works with Digital Ocean Spaces: https://www.npmjs.com/package/@booreiland/keystone-storage-adapter-s3


Update dependencies and add missing attributes in package.json

This update also fixes 16 vulnerabilities (6 low and 10 high) in the old dependencies.
The version has also been bumped up to v1.0.1 (due to the changes made here as well as recent bug fixes), so a new release tag will be required once this PR is merged.

@setuparimi Can you please submit your review?


Compute EFMs via EFMtool crashes CNApy (on Windows)

Error produced with SmallExample.
efmtool4cobra isn't able to get reversibilities.
Traceback (most recent call last):
File "C:\Users\phili\cnapy\cnapy\cnapy\gui_elements\efmtool_dialog.py", line 45, in compute     self.constraints.checkState() == Qt.Checked)
File "C:\Users\phili\cnapy\cnapy\cnapy\core.py", line 16, in efm_computation
reversible, irrev_backwards_idx = efmtool4cobra.get_reversibility(
AttributeError: module 'efmtool_link.efmtool4cobra' has no attribute 'get_reversibility'

No thanks for opening the issues that makes it much easier for us.


Runtime crash in SimpleChromeCustomTabs

Caused by android.content.ActivityNotFoundException: No Activity found to handle Intent { act=android.intent.action.VIEW dat=Http: pkg=com.android.chrome (has extras) }
    at android.app.Instrumentation.checkStartActivityResult(Instrumentation.java:1904)
    at android.app.Instrumentation.execStartActivity(Instrumentation.java:1624)
    at android.app.Activity.startActivityForResult(Activity.java:3761)
    at android.app.Activity.startActivity(Activity.java:4028)
    at android.support.customtabs.CustomTabsIntent.launchUrl(CustomTabsIntent.java:163)
    at com.novoda.simplechromecustomtabs.navigation.SimpleChromeCustomTabsWebNavigator.navigateTo(SimpleChromeCustomTabsWebNavigator.java:37)

The line 37 executes when the the connected condition is true for the SimpleChromeCustomTabsWebNavigator class so it is contradictory to not find the activity.
Connection is established in the onResume() and disconnected in onPause() of the base activity.
Crash stats:
Devices affected -
Samsung 59%
HTC 19%
LG 11%
Others 11%
OS -
Lollipop 77%
Marshmallow 18%
Kitkat 4 5%
So pretty much affects all Android devices.

I think there is a problem in your code. Given the stacktrace you are placing:
No Activity found to handle Intent { act=android.intent.action.VIEW dat=Http: pkg=com.android.chrome (has extras) }

It seems that you are instructing SimpleChromeCustomTabs to navigate to Http:. Make sure you provide a correct Uri to navigate to.
As an additional note the following code will crash for you with exactly the same stacktrace:
Intent intent = new Intent(Intent.ACTION_VIEW).setData(url);
startActivity(intent);

Where url is your desired Uri, in your example Http:
Closing this issue.


Unable to list containers

Hi,
I really like what you did, thanks for open sourcing it.
I hope you'll be able to help me make it work.
OS X 10.10.5
docker-machine 0.4.0
docker 1.8.1, build d12ea79
Virtualbox

I'm using Chrome app and here's what I did:

docker-machine ssh default
sudo vi /var/lib/boot2docker/profile
added the following line at the bottom:

DOCKER_OPTS="--dns <IP_ADDRESS> --dns <IP_ADDRESS> -H tcp://<IP_ADDRESS>:2375 -H unix:///var/run/docker.sock -api-enable-cors"

docker-machine restart default

I added docker hosts without any problems (they are green), but when I click on Refresh in the container list tab I get:
Error: [$resource:badcfg] Error in resource configuration for action `query`. Expected response to contain an array but got an object
http://errors.angularjs.org/1.4.1/$resource/badcfg?p0=query&p1=array&p2=object
    at angular.js:68
    at $http.then.value.$resolved (angular-resource.js:587)
    at processQueue (angular.js:14551)
    at angular.js:14567
    at Scope.$eval (angular.js:15830)
    at Scope.$digest (angular.js:15641)
    at Scope.$apply (angular.js:15935)
    at done (angular.js:10356)
    at completeRequest (angular.js:10528)
    at XMLHttpRequest.requestLoaded (angular.js:10469)(anonymous function) @ angular.js:12314(anonymous function) @ angular.js:9101processQueue @ angular.js:14559(anonymous function) @ angular.js:14567Scope.$eval @ angular.js:15830Scope.$digest @ angular.js:15641Scope.$apply @ angular.js:15935done @ angular.js:10356completeRequest @ angular.js:10528requestLoaded @ angular.js:10469

And one more thing, this is how the Docker host information page looks like (I assume that should be filled in):

Hm. I have to check i. I've nevers seen that before :) Give me some time :)

Thanks for looking into it :+1:


Allow downloading and installing updates when starting to charge

When the "required battery level" toggle is activated, why not already start to download and install available updates when connecting the phone to the charger?

It does essentially work that way since it will run them once they're no longer low on battery which will only take a few minutes.


Multiline search using regex fails to return all results

VSCode Version: Code 1.8.1 (ee428b0eead68bf0fb99ab5fdc4439be227b6281, 2016-12-19T14:49:23.350Z)
OS Version: Windows_NT ia32 6.1.7601
Extensions:

Extension
Author
Version

Hi,
I'm trying to find all occurences of:
''LTS+/\r&
''IGNORED\r&
I use the following regex:
''LTS\+/\\r&[\r\n].*''IGNORED.*[\r\n]
which seems to work in sublime text, for instance.
On VSCode, it only returns one hit.
Steps to Reproduce:

Our text search in general only works line-by-line

Oh, well. At least I know why it isn't working :)
:+1:  for the feature request.

Atom does not have multi-line regex searching atm it seems: https://github.com/atom/find-and-replace/issues/303
But I wouldn't mind seeing it implemented. I personally don't care for line breaks in the middle of code but I could see it being useful for others.

Actually there is an existing feature request for this: https://github.com/Microsoft/vscode/issues/13155

@danwulff I don't know if it still isn't supposed to have multiline regex, but I have made it work.  Here is where I posted about it with explanation and screen shots:
https://github.com/atom/find-and-replace/issues/303#issuecomment-319508730


RuntimeError: CUDA error: CUBLAS_STATUS_NOT_INITIALIZED when calling cublasCreate(handle)

During the training, I am getting this error, but if it is inference then there is no problem.
File "F:\transformer\CSWinTT\lib\train../..\lib\models\cswintt\transformer_cs.py", line 256, in forward
_query, _key, _value = self.qkv_embedding(merge_feat).chunk(3, dim=-1)

跑stark和mixformer就没用出现这个错误

Hi, have you runned this code in train phase?


Truncate $GITHUB_REF at the right level when testing for pip install success

The workflow that checks that blackjax can be installed from PyPi currently fails because $GITHUB_REF is truncated before the leading version number (currently 0). This commit fixes this.

Yet another spurious codecov failure.


Updating French language.

/check-cla

Welcome @DimitriDR! It looks like this is your first PR to kubernetes/website 🎉. Please refer to our pull request process documentation to help your PR have a smooth ride to approval. You will be prompted by a bot to use commands during the review process. Do not be afraid to follow the prompts! It is okay to experiment. Here is the bot commands documentation. You can also check if kubernetes/website has its own contribution guidelines. You may want to refer to our testing guide if you run into trouble with your tests not passing. If you are having difficulty getting your pull request seen, please follow the recommended escalation practices. Also, for tips and tricks in the contribution process you may want to read the Kubernetes contributor cheat sheet. We want to make sure your contribution gets all the attention it needs! Thank you, and welcome to Kubernetes. 😃

/check_cla

/retitle Update some pages in French localization

/lgtm


Feature/pim typesupport [6639]

This PR makes TypeSupport a base class, still compatible with TopicDataType but by storing it as shared_ptr.

Build status:

Linux
Mac
Windows
Performance

Build status:

Linux
Mac
Windows
Performance


Error reported when executing 'mlc_chat_cli'

❓ General Questions

mlc_chat_cli --model mlc-chat-Llama-2-7b-chat-hf-q4f16_1

Hi @linjie830914, the error message suggests VK_ERROR_INCOMPATIBLE _DRIVER and may be because the Vulkan driver has not been installed or wrongly installed or the device is incompatible. Could you follow https://www.vulkan.org/tools#vulkan-gpu-resources and make sure the driver is correctly installed?


Cache IntWidths to avoid redudant object creations

CHANGE BASE TO MASTER
SHOULD BE MERGED AFTER #289
Addresses #272

@aswaterman looks good?

By the way, this speeds up ExpandWhens a lot:

With BOOM on a6: ~18 secs -> ~10 secs

I found the size of the performance improvement surprising so I looked into it a little.
it seems that in mux_width_and_types we needlessly create a new IntWidth and UInt/SIntType and in AND and NOT we do the same. I'm going to fix that issue in a separate pull request but the improvement from this one is encouraging!
I still think the measured performance improvement is a bit high but I'm not going to complain

One worry I have is this adds a lot of implementation code to IR.scala, which should be as lean as possible. For now, the benefits are worth merging this, but I'd like to keep this on the horizon to see if we can factor it out of IR.scala, especially when we clarify the distinction between WIR and IR.

I'm running Jenkins on this which checks it and the branches it depends on

Commit 144041a23329f258541e9d461b5bcc7f2284e686 should be dropped from this, no? It was squashed and merged into master as 7c38199ce7a5d9dd7e27ffbb9b2b2770b972ed94

K I rebased on master and will merge after Travis


ENG-988 improve GraphQL error handling

Summary
The GraphQL client can return errors with different formats (regular string or JSON object). The new method GetErrorResponse(err error, methodName string) unifies the client error message.
Old
$ plural shell purge

2022/11/15 13:28:30 {"networkErrors":null,"graphqlErrors":[{"message":"no shell available for this user","path":["deleteShell"],"locations":[{"line":2,"column":2}]}]}

New
$ plural shell purge

2022/11/15 16:21:20 GraphQL error: no shell available for this user: DeleteShell

Network error:
2022/11/15 16:25:44 Network error: Response body {"message":"invalid_token"}: ListArtifacts

Very often the GraphQL server returns an unknown error
TODO:
Improve error messages on the API side:
$ plural api list artifacts aa
2022/11/15 16:22:44 GraphQL error: unknown error: ListArtifacts

Labels

Test Plan

Checklist

[ ] If required, I have updated the Plural documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] I have added a meaningful title and summary to convey the impact of this PR to a user.
[ ] I have added relevant labels to this PR to help with categorization for release notes.

Overall looks good. Just some small section where an if err != nil isn't being done. Not sure if it is necessary but just an inconsistency I noticed. Otherwise it looks good.

to handle the err == nil cases, if we just passthrough nil in the GraphQlError wrapper, it should be sufficient and prevent inelegant branching in the code

Overall looks good. Just some small section where an if err != nil isn't being done. Not sure if it is necessary but just an inconsistency I noticed. Otherwise it looks good.

generally, the method 'GetErrorResponse' digests the error and checks if it's nil. If not then converts to the new format


[FEATURE]: CSM support for Openshift 4.12

CSM support for Openshift 4.12
As part of this feature, we need to add support for:

Enabling Openshift 4.12 support on all CSM components
Qualify all the Openshift 4.12 related scenarios supported by CSM components through HELM charts, CSI Operator and CSM Operator

@rajendraindukuri As part of seamless customer experience on OCP console for Raven, we will need operator support for CSI/CSM with Openshift 4.12     - Malathy Ramani

How are things going with this? This is a real blocker for me to upgrade to OpenShift 4.12 and I would really like to continue using the Dell certified operators.

Things are on track and official support for 4.12 is planned for end of June.

We do try and stay on track and align with the latest OCP releases each quarter. Q1 2023 was a bit different and for various reasons we could not qualify OCP 4.12. That being said, unfortunately at this time, we will not have official support for 4.12 in our operators (CSI/ CSM) by April 17th and it is planned for end of June as part of our next CSI/CSM release.


python311Packages.bimmer-connected: 0.14.0 -> 0.14.1

Description of changes

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

1 package failed to build:

home-assistant-component-tests.bmw_connected_drive

8 packages built:

python310Packages.bimmer-connected
python310Packages.bimmer-connected.dist
python310Packages.canals
python310Packages.canals.dist
python311Packages.bimmer-connected
python311Packages.bimmer-connected.dist
python311Packages.canals
python311Packages.canals.dist


Add support for Rails 3.2.

This patch will probably not apply to the current codebase, but Rails 3.2 support would be great! :)

I think it justifies at least a <IP_ADDRESS> or 1.0.1 version.

+1

+1

Can you apply this patch, please?

+1

Great, thanks for your work on this!

Preliminary tests show no issues with Rails 3.2.1.  However, I need https://github.com/blowmage/minitest-rails/issues/22 to be resolved for the tests to pass.  I've bumped the minitest-rails requirements locally and everything looks good.  Hopefully some action comes on that front.

Switching to mini_specunit resolves this issue and cleans up test_helper (http://git.io/Oynp-Q).
If you feel like living on the edge, add the following to your Gemfile:
gem 'redis-rails', :git => 'git@github.com:jodosha/redis-store.git'

And for the less daring, we now have RCs with Rails 3.2.1 support.  Add the following to your Gemfile:
gem "redis-rails", "~> 3.2.1.rc"


terminal not work after set "terminal.integrated.inheritEnv" false

Issue Type: Bug
terminal not work after set "terminal.integrated.inheritEnv" false

Since I just installed Anaconda, vs code python extension(maybe) informed me to keep "terminal.integrated.inheritEnv" false to prevent some issues related to integeated terminal and Anaconda. But after I set this, the integrated terminal does not work. Once I set it back to true, things will be ok.
I use fedora 32(Chinese language) and disable all extensions no help, change to other shell no help.
VS Code version: Code 1.48.0 (db40434f562994116e5b21c24015a2e40b2504e6, 2020-08-13T07:40:55.768Z)
OS version: Linux x64 5.7.14-200.fc32.x86_64

/duplicate https://github.com/microsoft/vscode/issues/76542


[DOCS]: Create a PyPi tutorial.

Describe alternatives you've considered
Would you please share your clear and concise description of any alternative/similar documentation you've considered.
Additional context
Would you please share any other context or screenshots about the documentation request here.

The PyPi Tutorial is now in the document framework.
Content needs creating.


Shortcut URL preview

Description of the new feature / enhancement
If I have a URL stored as a file (rather than as a bookmark in my browser) it would be nice to have a preview of what that URL points to. There is a very basic version of this built into Windows Explorer, if you leave the cursor on the shortcut then it will sometimes (it's not consistent) pop up a little display of the contents of the shortcut.
What I'm proposing is that when I select a shortcut, I would like to see the contents of the shortcut - e.g. www.github.com - appear in the preview pane. If you're feeling extra adventurous then a preview of the website it's pointing to would be nice but the first option is a wonderful start.
Thankyou for considering this.
Scenario when this would be used?
Much the same as the preview of .docx etc, it's nice to be able to see what a file or shortcut contains.
Supporting information
Thankyou.

If I have a URL stored as a file

So, .url files and possibly .lnk files with an internet url too?

@jeffbeechg what are you envisioning for the start of this?  Entry point is critical.  Like, a URL shortcut on your desktop and the preview is on the preview pane?
/needinfo

Okay the actual preview was a long shot and your argument sounds fair. But what about the simple url/location? That's only a bit of text from the file...

what the URL For the link is?  You can get that via a tooltip.

Hi Clint
Ahh, that’s the word. Tooltip, thanks. Yes I mentioned that when I wrote
about it being inconsistent. But it’s better than nothing and I’ll have to
live with it if you’re not going to implement my suggestion.
Thanks for considering it.
Best regards
Jeff
On Mon, Jul 18, 2022 at 6:32 PM Clint Rutkas @.***>
wrote:

what the URL For the link is? You can get that via a tooltip.
[image: image]
https://user-images.githubusercontent.com/1462282/179633857-4d4140ee-6be3-4157-9a7f-65c0a1c0ff14.png
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/PowerToys/issues/19459#issuecomment-1188436191,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXHCUD2S7SDDQN52AEGWIULVUXSRLANCNFSM53VLH52A
.
You are receiving this because you were mentioned.Message ID:
@.***>

@jeffbeechg when doesn't it work?

Good morning Jay,
Yes, the browser-ish preview was indeed a longshot. The simple URI would be
nice. I wouldn't be asking for this if the tooltip worked consistently, but
it doesn't (as I've been chatting with Clint about)
Thankyou again for considering this. I've been impressed with PowerToys.
Best Regards,
Jeff
On Mon, Jul 18, 2022 at 5:54 PM Jay @.***> wrote:

Okay the actual preview was a long shot and your argument sounds fair. But
what about the simple url/location? That's only a bit of text from the
file...
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/PowerToys/issues/19459#issuecomment-1188408340,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXHCUD543YMWGNK57S44JODVUXOA3ANCNFSM53VLH52A
.
You are receiving this because you were mentioned.Message ID:
@.***>


WIP: Fix Arlo Camera blocking IO

Description:
WIP PR to fix the issues introduced in 0.55. Will definitely want testing of this prior release, given that previous changes affected the whole user base.
Related issue (if applicable): (hopefully) fixes #9737
Pull request in home-assistant.github.io with documentation (if applicable): N/A
Example entry for configuration.yaml (if applicable):
ffmpeg:
  ffmpeg_bin: /usr/local/bin/ffmpeg
camera:
  - platform: arlo

Checklist:
If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

[ ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] New dependencies have been added to the REQUIREMENTS variable (example).
[ ] New dependencies are only imported inside functions that use them (example).
[ ] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[ ] New files were added to .coveragerc.

If the code does not interact with devices:

[ ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] Tests have been added to verify that the new code works.

Awesome! Thanks, all!


AIRGoogleMapLocalTile does not exist.

Is this a bug report?
yes
Have you read the Installation Instructions?
yes
Environment
react-native:  0.50.0
react-native-maps: 0.20.1

target is iOS and using GoogleMaps
Steps to Reproduce

using iOS
using Google Maps SDK
using LocalTile

Expected Behavior
Load local generated tiles and place them on the map.
Actual Behavior

+1 please introduce this in a next version.

@floorswinkels @hanselsen try the git version please and let us know

@rborn with the git version it's still not working.
When I switch to Apple's mapkit LocalTile is working. ( But in our case we need Google Map SDK )

@floorswinkels can you please set a small test/demo repo? because we have this https://github.com/react-community/react-native-maps/pull/1876 which should work.
So maybe it's a mistake somewhere in the PR (unless I'm totally getting wrong 😹)

Same issue here

Failing on this line https://github.com/react-community/react-native-maps/blob/f553317fcaa2e3a4889f38c4bb2e215693f3f20c/lib/components/MapLocalTile.js#L46

I have the same issue :(

@merryejik did you try with the master version ?

@rborn the same :(

Any updates on this?

@simon4321 are you using master or on an expo project?

@ahmetabdi I'm using latest npm version without expo.

any update on this one?

I found this as temporary solution... We can use UrlTile as LocalTile, we just need to add file:// prefix to the tile's path.
<MapView {...props}>
    <UrlTile urlTemplate={this.offlineTilesPath} zIndex={-1} />
</MapView>

Closing as there is #2733 with better explanations.


[R-package] Add print() and summary() methods for Booster

Currently, when examining a lightgbm object, it shows information which is typically not relevant to the user, while missing important pieces of information that one would want to see, such as the objective function or the model dimensions. This PR adds a print and a summary method with shorter outputs and more relevant information. The old print can still be accessed through str.

That message from this project's linter exactly mirrors direct feedback we received from CRAN.
In one submission of {lightgbm} 3.0.0 to CRAN in September 2020, CRAN's response included the following (documented in cran-comments.md)

Please replace cat() by message() or warning() in your functions (except for print() and summary() functions). Messages and warnings can be suppressed if needed.

Since this PR is adding print() and summary() methods, I think uses of cat() in those specific cases will be ok with CRAN.
According to https://github.com/jimhester/lintr#project-configuration, it's possible to tell {lintr} to exclude specific lines by adding the comment # nolint. Please try adding such comments to the lines in this PR that use cat().

@jameslamb Can we merge this now?

yep! I'll merge it. Thanks again for the help @david-cortes !


db,sorter/leveldb(ticdc): remove snapshot

What problem does this PR solve?
Remove useless snapshot API.
Check List
Tests

Unit test
Integration test

Release note
None

Codecov Report

Merging #4058 (f9d052e) into master (3873d39) will decrease coverage by 1.9330%.
The diff coverage is 70.9349%.

Flag
Coverage Δ

cdc
58.6315% <70.9349%> (+0.3949%)
:arrow_up:

dm
52.1998% <ø> (-3.8348%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.
@@               Coverage Diff                @@
##             master      #4058        +/-   ##
================================================
- Coverage   57.0741%   55.1410%   -1.9331%
================================================
  Files           478        485         +7
  Lines         56551      59890      +3339
================================================
+ Hits          32276      33024       +748
- Misses        20978      23526      +2548
- Partials       3297       3340        +43

/merge

/merge

/merge

/merge

/run-verify
/run-dm-integration-test

/run-verify
/run-kafka-integration-test
/run-integration-test

/run-verify
/run-dm-integration-test

/run-dm-integration-test


Need tip with converting ldap provider

Im trying to convert ldap provider(https://registry.terraform.io/providers/Ouest-France/ldap/latest/docs)  there:
https://github.com/wNvSp1r1T/terraform-jet-ldap/commit/8dfc0a9c1c41559ce91e0bacf25d4340c9b7b85c
And i'm always get error: "summary":"LDAP Result Code 34 \"Invalid DN Syntax\"
1.664437610235876e+09	DEBUG	provider-jet-ldap	Cannot observe external resource	{"controller": "managed/ldapgroup.ldap.jet.crossplane.io/v1alpha1, kind=group", "request": "/hello-crossplane", "uid": "574384b6-1ec8-4bd9-9f1c-4b08821b7adb", "version": "208755", "external-name": "hello-crossplane", "error": "cannot run refresh: readObjectStart: expect { or n, but found 2, error found in #1 byte of ...
my examples/providerconfig/providerconfig.yaml is default one
my examples/providerconfig/secret.yaml is
    {
      "host": "ldap.ru",
      "port": "389",
      "bind_user": "bind_user",
      "bind_password": "test-ldap"
    }

my examples/ldapgroup/ldapgroup.yaml is:
kind: Group
metadata:
  name: hello-crossplane
spec:
  forProvider:
    description: "Managed with Crossplane ldap Provider (generated with Terrajet)"
    ou: "OU=test,OU=sre,OU=Resources,OU=Main,DC=X,DC=eu
  providerConfigRef:
    name: default

I will be grateful for your help

It doesn't seem like you have created anything in your repo.
Have you read this guide? https://github.com/crossplane/terrajet/blob/main/docs/generating-a-provider.md

It doesn't seem like you have created anything in your repo. Have you read this guide? https://github.com/crossplane/terrajet/blob/main/docs/generating-a-provider.md

i'm sorry, but i'm forget to use git add :D
now repo is up to date.
https://github.com/wNvSp1r1T/terraform-jet-ldap/commit/40462d2347371938936cd0e7841f982d61047d46

Have you tried creating the resource with Terraform using the exact same values as in your Crossplane provider ?

Have you tried creating the resource with Terraform using the exact same values as in your Crossplane provider ?

yep
it's work fine with terraform or with terraform-provider (crossplane)

Have you tried setting r.ExternalName = tjconfig.IdentifierFromProvider in the Configure function ?
Like so: https://github.com/ok-amba/provider-jet-opsgenie/blob/main/config/escalation/config.go#L26

Have you tried setting r.ExternalName = tjconfig.IdentifierFromProvider in the Configure function ? Like so: https://github.com/ok-amba/provider-jet-opsgenie/blob/main/config/escalation/config.go#L26

Thanks for your reply.
But it's didn't work :(
DEBUG	provider-jet-ldap	Cannot observe external resource	{"controller": "managed/ldapgroup.ldap.jet.crossplane.io/v1alpha1, kind=group", "request": "/hello-crossplane", "uid": "574384b6-1ec8-4bd9-9f1c-4b08821b7adb", "version": "210023", "external-name": "hello-crossplane", "error": "cannot run refresh: readObjectStart: expect { or n, but found 2, error found in #1 byte of ...|2022-10-03T|..., bigger context ...|2022-10-03T09:54:47.536+0300 [INFO]  Terraform vers|...: refresh failed", "errorVerbose": "refresh failed\nreadObjectStart: expect { or n, but found 2
i get the same error
error: LDAP Result Code 34 "Invalid DN Syntax": 0000208F: NameErr: DSID-03100229, problem 2006 (BAD_NAME), data 8350, best match of:\n\t'hello-crossplane'\n\u0000\n{"@level":"error","@message":"Error: LDAP Result Code 34 \"Invalid DN Syntax\\
it's looks like DN name formed incorrectly, but i didn't get why, it's works with terraform as is

Have to tried using the newest Terraform version ?
https://github.com/wNvSp1r1T/terraform-jet-ldap/blob/40462d2347371938936cd0e7841f982d61047d46/Makefile#L9
Also, for some reason, you are exporting TERRAFORM_VERSION twice in your Makefile

Have to tried using the newest Terraform version ? https://github.com/wNvSp1r1T/terraform-jet-ldap/blob/40462d2347371938936cd0e7841f982d61047d46/Makefile#L9
Also, for some reason, you are exporting TERRAFORM_VERSION twice in your Makefile

looks like it's works right now
thanks for your help!

Also, is it possibly to disable option prevent_destroy?
Cause this provider can-not update created element correctly

Do you mean deletionPolicy ?
https://crossplane.io/docs/v1.9/concepts/managed-resources.html

Do you mean deletionPolicy ? https://crossplane.io/docs/v1.9/concepts/managed-resources.html
yup, looks like it.
requests to create an object are made too often
because of it duplicate objects appear.
Is it possible to set the period between repeated requests?

No clue


Agent suggestion for messages with no agent mention

Context and design here: https://dust4ai.slack.com/archives/C050A0S2Z7F/p1695131873350469

@philipperolet thanks for picking up!
I think we want to do the following:
rely on editUserMessage
in editUserMessage trigger an agent run if there was no AgentMessage after that UserMessage.
WDYT?

We could do that, I would be fine with it ; it's the simplest & fastest
Another option is to have editMessage handle mentions => re-run all mentions (expected behaviour if the user changes its message with a mention), as a new version for existing children messages from the mentions, and as fully new messages for new mentions
Intermediate solution: only run new mentions (as initially suggested by the function editMessage's comments) => as a follow-up feature, provide a retry icon on agentmessages whose parent has been edited
After thought, I'd go for that: editMessage creates new agentMessages for new mentions (and ignores old ones)
Back to you WDYT ? :)

Other topic: no mention always implies agent suggestion buttons?
I'm starting on this assumption since for a first iteration it feels ok, but might need discussion

On behavior of edit, agreed with your analysis. Let's start simple and complexify as need arise?
On suggestion:
Yes let's start with no mention implies suggestions. We'll see if this is too much or not.
We do want to have any bot in the conversation as first proposal.


Adding new secret santa members later on?

Possibility for larger secret santa pools to add additional (minimum 5) new people to the secret santa pool.
https://twitter.com/pappie/status/679644218323812352

related issue #31


added level_order_traversal_binary_tree.py for issue #192 in Code/Python

closes #192
@plazzy99 I've made requested changes

@plazzy99  I've updated the readme.
Please review my PR and add gssoc & level labels. Thankyou

Thankyou so much :)
Can you please add gssoc and level labels? @plazzy99


Issue in autocommenting on merged pr #198

closes #198
I updated the comment POST code.
I have also tested this on my local repository and it worked.

@subhadipbhowmik Sir, please review it and merge it


Development

Changes Unknown when pulling 68b011336935524a5a6bad3b3627688c6d797f78 on development into ** on master**.

Changes Unknown when pulling 8755a11a58bd5319c077d3ccd0b0587c53187dd7 on development into ** on master**.

Changes Unknown when pulling 7a9eed662b5049f1db136d3b6af5d1e9756cd294 on development into ** on master**.


Windows build and contbuild stuff

TODO: right now cuda is still not supported, just sending this so we can get Ruben starting to test on Windows more easily.

@Yangqing, do you have built Caffe2_CPU.lib in windows successfully? I failed to build it in VS2015. Is it necessary to add or change something in source code of your windows branch?

seems that import is failing, I am submitting internally with @bwasti . Closing.

@ezineo I actually don't - this is actually only WIP and there are a few
Linux-specific things that we need to sort out before complete windows
support is ready.
Yangqing
On Mon, Feb 13, 2017 at 8:41 PM, ezineo<EMAIL_ADDRESS>wrote:

@Yangqing https://github.com/Yangqing, do you have built Caffe2_CPU.lib
in windows successfully? I failed to build it in VS2015. Is it necessary to
add or change something in source code of your windows branch?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/caffe2/caffe2/pull/146#issuecomment-279606616, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAho73sYIv1YoQO2KmpM5Y8snoVol7rIks5rcTCSgaJpZM4L_sJx
.


Fix NSJSONSerialization crash in iOS 13

This is to address https://github.com/mixpanel/mixpanel-iphone/issues/909 (Crashes in encodeArrayForAPI: and convertFoundationTypesToJSON: under iOS 13)
In iOS 13 there is a bug for encodeArrayAsJSONData: if the input array in encodeArrayAsJSONData:(NSArray *)array contains corrupted characters(not valid UTF-8), it will lead to a crash. see radar https://openradar.appspot.com/radar?id=4964054569320448  (NSJSONSerialization crashes on iOS 13) The fix is to check whether the string is a valid UTF-8 before sending to Mixpanel. Mixpanel backend can only handle UTF-8 string.

Hey @zihejia,
We saw a couple of crashes around archiving, is this related to this by any chance?

It's on 3.6.2.

Opened #912. Thank you!


24-Hour Forecast directly in the app

It should be nice to have the 24-Hour Forecast directly in the application, without opening a browser

It's me or in 5 months they added the  tag in the xml file? It should be easier to add it?
Unfortunately, I have no experience in developing mobile app, but maybe in another 5 months I will try to submit a PR if isn't done :)


runtime: TestTimePprof failure with "profiler refers to ExternalCode"

2020-12-09T19:12:27-f1980ef/solaris-amd64-oraclerel
--- FAIL: TestTimePprof (0.34s)
    crash_test.go:699: Type: cpu
        Time: Dec 9, 2020 at 9:56pm (CET)
        Duration: 215.70ms, Total samples = 20ms ( 9.27%)
        Showing nodes accounting for 10ms, 50.00% of 20ms total
        Showing top 1 nodes out of 11
              flat  flat%   sum%        cum   cum%
              10ms 50.00% 50.00%       10ms 50.00%  runtime._ExternalCode
    crash_test.go:703: profiler refers to ExternalCode
FAIL
FAIL	runtime	80.082s

See previously #41120.
CC @ianlancetaylor @aclements @tklauser (for TestTimePprof), @hyangah @dmitshur @zikaeroh (in case this is related to CL 275513)

2021-02-25T09:20:03-76c0003/solaris-amd64-oraclerel

Looks like there was an uptick in this failure rate in early November.
greplogs --dashboard -md -l -e '(?ms)FAIL: TestTimePprof.*profiler refers to ExternalCode' --since=2021-02-26
2021-11-18T20:11:45-24898d6/solaris-amd64-oraclerel
2021-11-17T18:14:23-9bdbed1/solaris-amd64-oraclerel
2021-11-12T22:20:50-9150c16/solaris-amd64-oraclerel
2021-11-11T19:41:06-f1935c5/solaris-amd64-oraclerel
2021-11-06T00:29:44-b74f2ef/solaris-amd64-oraclerel
2021-11-04T21:33:23-71fc881/solaris-amd64-oraclerel
2021-09-27T20:58:06-e54843f/solaris-amd64-oraclerel

Marking as release-blocker to either fix the test, or add a skip if the cause is believed to be platform-specific and the fix is non-trivial.
(CC @jeremyfaller)

CCing @cherrymui as the owner of pprof.
(CCing @prattmic as well, because it's likely he would be interested as well.)

Seems this only fails on Solaris? We use libc for syscalls, so the "ExternalCode" may refer to the libc function.

@cherrymui I agree.  I will send a CL.


NUnit Test Task Isn't Running

I'm trying to use the NUnit task to run my unit tests as part of the build process but they don't seem to be running.  I've tried to boil this down to the simplest possible scenario by hardcoding everything:
Task("Tests").Does(() =>
{
	NUnit(@"C:\_git\DTR\TestsTest\bin\debug\TestTests.dll");
});

I never see any test results in my output though:
========================================
Tests
========================================

Task                          Duration
--------------------------------------------------
Tests                         00:00:00.0260109
--------------------------------------------------
Total:                        00:00:00.0260109

It doesn't seem to matter if I have passing tests or failing tests, the output is the same.  I've tried running nunit-console with the exact same path and that gives meaningful output.
I know this is probably user error and not really a code issue but I couldn't find a better forum for getting help.  Thanks.

Just a few other notes...

Obviously that is just a snippet of my build file; I have other targets (like a compile target) that are running normally.
The NUnit call doesn't seem to fail no matter what crap I pass to it, even if I pass a made up file name it seems to behave the same.
I am using the 0.1.28 version released yesterday
I have added Info logging lines and other things to make sure my changes are actually taking effect and the right target is getting called.

I tried using the GetFiles method on my pattern just to see how many files are found.  Either there is something wrong wtih my pattern or a hard codes path doesn't count as a valid pattern because there were no files coming back.  I changed the pattern so it started finding the right files but now I'm getting the following error:
========================================
Tests
========================================
Found 1 files
An error occured when executing task.
Error: NUnit: Could not locate executable.

I'm not sure what that error means.  I've got the NUnit location in my path and can call it from the command line.

@bjmarte Hmm. Seems very strange since your code should work. Has this been working before and suddenly stopped working?

@bjmarte I found the error. The globber seem to fail for actual file paths, which is kind of bad, but I've filed an issue (#159) and will try to fix it tomorrow at lunch. In the meanwhile to get your tests running, you can do like this:
Task("Tests").Does(() =>
{
    NUnit(new FilePath[] { @"C:\_git\DTR\TestsTest\bin\debug\TestTests.dll" });
});

You will also need to tell Cake how to locate the NUnit runner.
There are two ways of doing this; either by putting it somewhere under the tools folder in your project root. Cake will look here by convention.

TheProjectRoot

tools

...

nunit.console.exe

build.cake

or by explicitly telling the Cake alias method for NUnit where it is by providing the path to it in the (normally optional) settings:
Task("Tests").Does(() =>
{
    NUnit(new FilePath[] { @"C:\_git\DTR\TestsTest\bin\debug\TestTests.dll" },
        new NUnitSettings {
            ToolPath = "C:/The/Path/To/NUnit/Runners/nunit-console.exe"
        });
});

I'm sorry for this, but I'm very happy that you reported this. Thank you!

You are welcome.  I fixed my issue with cake not being able to find the nunit executable after digging into the cake code.  I find the tools folder convention a little strange and didn't seem to work even after I copied NUnit into the folder. It was also not easy to determine how to tell cake to look for nunit in a different location.  I'm all good now though.

Hmm, sound strange. What did your folder structure to nunit-console.exe look like (from the root where you have your build script)?
The reason for the convention is that we need to have one to find things easy, and we settled on the same conventions as FAKE use for compatibility.
Sorry for the inconvenience. We're working on better documentation at the moment (better than we have at http://cake.readthedocs.org), but we're not quite there yet. When the new documentation is in place, it will hopefully be easier to find information like this.

@patriksvensson are you planning to still use readthedocs, or are you switching to something else?

@gep13 In a perfect world, I would like to continue to use readthedocs since it's awesome. The problem is that we will need something more complex to be able to generate proper API documentation for all public types and members of Cake.Core and Cake.Common as well as DSL documentation (for all the alias methods). This should be able to build automatically on AppVeyor when pushing to the master branch.
I've done some experimenting in the last couple of days, and I'm counting of having something in place at http://cakebuild.net in about two weeks.

@patriksvensson said
I've done some experimenting in the last couple of days, and I'm counting of having something in place at http://cakebuild.net in about two weeks.

Sounds good, look forward to checking it out!

Just a couple notes for documentation sake:
Information about how to include one script file from another would be helpful. I assumed that the syntax would be #load "", the same as scriptcs (according to their documentation, not experience).  I couldn't get that to work but then by trial and error figured out that #l "" does work.  I want to have separate files for general build, packaging, and deployments and this will be useful.  I imagine that if I have some library type functions and want to include them from multiple files that I will need to do some checking to prevent the same file from being imported twice, a documented pattern for how to accomplish that would be helpful too.

@bjmarte You're absolutely right about the documentation. It's still a quite young project, and documentation is something that needs constant attention to be up to date. If you want to change something or just notify someone about something that's missing, the documentation can be found at (https://github.com/cake-build/documentation)[https://github.com/cake-build/documentation].
The #l directive will only import a script once, so you should be able to divide your scripts in separate files without problem.

Added information about extensibility here: http://cake.readthedocs.org/en/latest/extensibility.html. Will add more information about addin support later.
Thanks for the feedback!

Fixed and released in version 0.1.29.


social media icons opening in new tab

Description
social media icons opening in new tab
Fixes #75
Type of Change:

Bugs

Checklist:

[x] My PR follows the style guidelines of this project
[x] I have performed a self-review of my own code or materials
[x] My changes generate no new warnings

@abhijeet141 thank you for your contribution 🥇

@abhijeet141 thank you for your contribution 🥇

Welcome 🙂


Improve packing

Follow up for the feedback left in https://github.com/dotnet/maintenance-packages/pull/65

Need to evaluate IsSourceProjectPackable in Directory.Build.props, not in targets, as it was too late.
The official yml only needs one leg to build and publish. The build linux one and the linux packing ones are unnecessary.

Looks good. You can revert all the project file changes and remove the new stuff in Directory.Build.props. What remains is to be able to only build the sources without the test for that packaging leg. Right now tests are built and packed:

Successfully created package '/__w/1/s/artifacts/packages/Release/NonShipping/System.Reflection.DispatchProxy.Tests.1.0.0-ci.24168.1.nupkg'.

With the latest commits, which pass the global property /p:Test=false in the packing leg for PRs, we're no longer creating nupkgs for the test projects: https://dev.azure.com/dnceng-public/public/_build/results?buildId=607838&view=logs&j=60dbaa24-68a7-5962-8c66-270b428b4658&t=0f846b6f-8d8e-592b-9ef5-850ae44e3537
The official run should still be able to generate the package if we set <IsPackable>true</IsPackable> in a csproj (tested locally).

which pass the global property /p:Test=false in the packing leg for PRs

Note that the Test property has a special meaning in Arcade's build wrapper project which could lead to unexpected behavior: https://github.com/dotnet/arcade/blob/777bc46bd883555cf89b8a68e3e2023fd4f1ee50/src/Microsoft.DotNet.Arcade.Sdk/tools/Build.proj#L106
You might want to condition on BuildTests instead and pass that in.


[py2py3] Evaluate the python version in a single and unified location

Impact of the new feature
WMCore in general
Is your feature request related to a problem? Please describe.
Not yet a problem, but in order to avoid using sys.version_info and so on in multiple places, I suggest we create such expression in a single module (likely under Utils) and then simply import it when needed.
Describe the solution you'd like
My suggestion is to create a new module named PythonVersion, or something more meaningful, and define the following 2 variables there:
PY3 = sys.version_info[0] == 3
PY2 = sys.version_info[0] == 2

In addition to that, I would suggest to create yet a second commit replacing all usage of sys.version_info by the import of this/these variables. This will unify the way we evaluate the major python version.
Describe alternatives you've considered
The Utilities module would be another candidate, but that one already imports quite some stuff and I'd rather have the python version evaluation as clean as possible.
Additional context
none

Sorry, this is a duplicate of https://github.com/dmwm/WMCore/issues/10509
Closing this one.


create fallback for no color settings library installed

Fail gracefully if the library is not present and use input text fields innate function instead.

after talking with Arddistan, might be to just bundle the color Picker library instead, which would be better AND easier. Will need to test.

Verified library support as a backup, closing.


Changing uppercase in some components

Hi,
I am new to Now UI and I am having trouble editing some titles. For example, I would like to change the capitalization of the NavarBrand component I have in my Navegation bar. I have commented the text-transform field in the css but the text is still in upper case, how can I keep the format in which I put the string in the brand component?
Thanks in advance.

Hello there, @ajaen4 ,
We actually do not use css, we use scss, so you need to change inside our scss the styles, instead of the css.
Take a look at the following links:

https://github.com/creativetimofficial/now-ui-kit-react/blob/master/src/index.js
https://demos.creative-tim.com/now-ui-kit-react/#/documentation/introduction

My recommendation is to not directly change our scss/css, since when we'll make an update to the product, we might change our styles, and that will make it hard for you to update your project.
Also, please read the following for an easier development with our products:

The easiest way to make updates is as follows:

You create two repositories on a versioning control system (for example GitHub, GitLab, Bitbucket etc.)  - if it's a PRO paid product, we suggest that the repositories to be private
On the first repository, you add our product unaltered
On the second repository, you add our product and/or develop your project
Each time we release a new version of the product at hand, you download again the product, and put it on top of the first repository, the versioning control system (for example GitHub, GitLab, Bitbucket etc.) will tell you all the differences between the versions
By using a repository, you can even make new branches for each of our new version, that way, you will always have all the versions

Hope this information is of help. If you have further questions, let us know.
Best,
Manu


Type for-in initializer as string

I think that for-in should have the same soundness guarantees as Object.keys. Since Object.keys always return string[], the same should be true for the for-in initializer - it has to be a string and can't be of a more narrow type.

Require for-in initializer to be assignable to string

the same should be true for the for-in initializer - it has to be a string and can't be of a more narrow type.

These two statements contradict.

Good catch :p

The statement that the for-in initializer must be a string and cannot be of a stricter type is contradictory. If the initializer is restricted to a string it would limit its usefulness as objects can have keys of different types. Therefore, it is appropriate that the for-in initializer accepts broader types such as string and any to cover all possible object keys.

@Mateusnasciment "Initializer" is a misleading term here IMO - this is specifically about the type of the range variable of for...in, which is always string.  The only other type that an object key can possibly be at runtime is symbol, and for...in ignores symbols.
According to MDN:

The for...in statement iterates over all enumerable string properties of an object (ignoring properties keyed by symbols), including inherited enumerable properties.

for (x in foo) { ... } - x can never be anything other than string.  In particular, for arrays, it's (maybe counterintuitively) not number.

@Andarist IIRC @RyanCavanaugh has said in the past that this was an intentional soundness tradeoff so people could do this...
const obj = { foo: 777, bar: 1206 };
let key: keyof typeof obj;
for (key in obj) {
    /* key :: "foo" | "bar" */
    console.log(obj[key]);  // normally an error, now ok
}

...otherwise you have to type-assert the key every time.

Ye, I get what is the consequence of that. However, I really can’t find a reason why this would not be consistent with Object.keys. If this is an intended soundness hole - that’s fine but the Object.keys then should have the same hole and it doesn't today

@RyanCavanaugh If this is true, thank you- it is a massive help! From a discussion in TS Discord:

ssalbdivad:  I almost always have to use let key: keyof MyObj...for(key in myObj) ... I assume it was added as a workaround. I think of it as a way to cast the key type as opposed to the assignment being safe... It seemed plausible there would be an escape hatch of some sort in that situation because what is the alternative? Creating another const variable in the loop just to cast the key type? Feels awful

If I understand correctly that this change would break that workaround, I'm strongly in favor of preserving the existing behavior. As far as I'm aware, without this workaround, you're essentially forced to declare an extra narrowed variable in the loop body just to narrow the key type. If you're adding runtime logic just to make your types happy, something has gone wrong- a cast should always be sufficient in these situations.

Aside: TIL for..in includes prototype properties 🙀

Enumerable ones but ye - it has been a popular combo in the past to have for-in+hasOwnProperty

The reason for-in is allowed is that it is (or at least was) idiomatic to do this with arrays:
const arr = [1, 2, 3, 4];
for (const i in arr) {
  console.log(arr[i]);
}

Rather than give i the completely-wrong type number, it's string with a carve-out to allow arr[i].
Should we just give up and type Object.keys as keyof T[] ? I am so tired of arguing about this.

The reason for-in is allowed is that it is (or at least was) idiomatic to do this with arrays:

I probably wouldn't cal this idiomatic in today's landscape so perhaps if that was the only-ish reason for this behavior then it could be revisited.

Rather than give i the completely-wrong type number, it's string with a carve-out to allow arr[i].

It could be given a type of ${number} today. That would allow us to remove the carve-out to allow arr[i] since it would already be legal. It wouldn't exactly cover perfectly cases like number[] & { prop: 1 } but those kind of things are not always handled by the compiler perfectly today so I think it should be fine here to type i as just string in cases like this.

Should we just give up and type Object.keys as keyof T[] ? I am so tired of arguing about this.

I totally get how you might be tired of this. I'm glad that Object.keys work like it does today - it gives me more confidence in my code and when I'm sure that I can only get keyof T then I cast it and it's fine. My only argument behind this PR is that the behavior of Object.keys and for-in should be identical because they are essentially the same(ish) operation. The for-in already works like Object.keys in the majority of cases and the weird carveout that I'm removing here just feels weird. It's like a hidden undiscoverable feature that feels out of place to me.

Should we just give up and type Object.keys as keyof T[] ? I am so tired of arguing about this.

The problem with that is you would get the wrong type as a result of subclassing:
interface Point { x: number, y: number }
interface Point3D extends Point { z: number };

const p2d: Point = { x: 0, y: 1 };
const p3d: Point3D = { ...p2d, z: 2 };

let p: Point;

p = p2d;
Object.keys(p); // ("x" | "y")[]

p = p3d;
Object.keys(p); // ("x" | "y")[], but it also contains "z"

The wider type is safer.

@RyanCavanaugh I agree with @Andarist and @rbuckton (and you) that the widened type makes sense.
It feels like the only good solution to this is an exact constraint on object types that allows narrowed key iteration and disallows assignment from non-exact objects.


Starfield

Add stub for secondaryDataDirectories to basicGames

This PR should not be needed if we default the function in uibase (and in plugin_python).


How to render videos for 360° scenes?

Hi, thanks for your great job! I want to know how could I render videos for 360° scenes? Looking forward to your reply!

Hi, since IBRNet uses only nearby images to render the novel view, and there is not a global coordinate frame as in NeRF, ideally IBRNet should work when rendering 360 scenes, as long as the input views are dense enough and cover the whole scene in 360°. But you might need to implement your own view selection algorithms to find the most relevant source views.


Session resumption semantics is not obvious when early data is allowed by the server

When s_server is started with -early_data and anti-replay is in effect, it uses stateful resumption, using the ticket as a key in the cache. On the other hand, if the client didn't close the connection gracefully with close_notify, the cache entry would be removed as a side effect of sending a fatal decode_error alert. As a consequence, the client would not be able to resume in the second connection.
I guess the behavior makes sense, though it was not obvious to me. Perhaps this could be either:

documented as a limitation, or
do not clear the cache entry in this specific case (TLS 1.3, early data with anti-replay enabled)

Here are the steps to reproduce (with openssl-3.0.0-beta1-211-g5303aa51c0):
$ apps/openssl s_server -accept 5556 -keyform pem -certform pem -key test/certs/serverkey.pem -cert test/certs/servercert.pem -early_data -num_tickets 1
$ apps/openssl s_client -host localhost -port 5556 -sess_out sess.out -CAfile test/certs/rootcert.pem
^C
$ apps/openssl s_client -host localhost -port 5556 -sess_in sess.out -CAfile test/certs/rootcert.pem
...
New, TLSv1.3, Cipher is TLS_AES_256_GCM_SHA384

related to #11378 ?

@ueno Could you please open a PR to document the limitation?


add  function that hot reloading config

Is your feature request related to a problem? Please describe.
Viper can help go-zero hot reload config, also can help us do not restart app to reload new config.
Describe the solution you'd like
change the conf.MustLoad from function main, using viper.
Describe alternatives you've considered
add viper into go-zero.
Additional context
when I use go-zero, suddenly thought about it.

#3035


Fill in Readme for resources repo

Readme should tell us how to use this repo.
Some thoughts...

For vsixes use the 64x64 icon, for nuget use 256x256 (or?)
For vsixes, copy the icon to the project, as it has to be referenced in the file system by the manifest
For nuget, use a link to rawgit.com (development) or cdn.rawgit.com (production)

For nuget, use a link to rawgit.com (development) or cdn.rawgit.com (production)

How did you see this working, @CharliePoole? Did you envisage the cake script passing in a different URL, based on if your building a dev or release build? What was the thinking - to cover cases of wanting the update the icon, and a delay in the CDN update?
I'll grab this issue if you don't mind, would like to get all this tidied up and in place before the next releases. 😄

@CharliePoole - we're cross posting! From your other post:

Regarding where to link... I'll write something for the resources/readme.md after I have a chance to actually try the different alternatives and see them work.

I'll leave this issue to you in that case! Still interested in your thoughts on the cdn, above!

I thought that we would just update the nuspec. But I only meant to use rawgit.com for initial development and cdn.rawgit.com thereafter. That's what is recommended on rawgit.
I'm not aware of a delay, but if there is one, I assume we just wait for it to be over. :-) Do you mean the delay for the caching?

I wasn't sure if there was a delay in updating the CDN - it was only a thought as I was trying to work out why you were suggesting the split development/production URLs. Sounds like we agree that the nuspecs stored in the repos should reference a cdn.rawgit link - I'll go with that!

They want you to use rawgit.com for developing the icon because it's only cached temporarily. OTOH cdn.rawgit.com caches the icon permanently and you would have to use different names each time you made a change.
Now that I think of it, does this say anything about our naming? Should we use nunit3_nn rather than nunit_nn?

To clarify, we should use rawgit.com when we are developing the icon, not the software. Once both the appearance and the naming of the icon are fixed, we should use cdn.rawgit.com. After that point, we can't change the icon any more without changing the name.
If we want the ability to use indirection (change the icon itself under a specific name) we shouldn't use the permanent caching feature but should either use rawgit.com or github directly.

Ah - I misunderstood!
On naming, I think the current is fine - we can make these the 'default', and add 'nunit2_xx' if required?

That makes sense... so now give me a 90x90 icon. 😄

Done. :-)


Creates a PR for ofbiz-vm3

This is only temporary IP change
to test before ofbiz-vm2 is gone
There are also changes related to
INFRA-17523 "Add error page 503 for ofbiz demo sites"
Because I pulled after
I also wonder if we should not rather create an ofbiz-vm3.apache.org.yaml file ? At least at some point...

I currently can't revert the change of the IP. But I prefer to create a new ofbiz-vm3.apache.org.yaml and keep ofbiz-vm2.apache.org.yaml as it was before this PR

I did not know about https://github.com/apache/infrastructure-p6/blob/production/data/nodes/ofbiz-vm3.apache.org.yaml.
Closing this PR


chore: LEAP-516,LEAP-517: upgrade follow-redirects package to address CVE-2023-26159

Upgrades follow-redirects in web/, docs/ and web/libs/editor/tests/e2e/. Methodology used was from here: https://github.com/yarnpkg/yarn/issues/4986#issuecomment-1500862435

@hlomzik @Gondragos - Clicking around on the Netlify docs previews above, links seem to be working okay. Is there a different way I should be testing the docs build?

@jombooth To be honest I can't say I'm an expert in this docs part. But I would say that docs were not generated this time. It looks like it needs some changes sources to do that or something like that.
As I understand here is an example of how it should look like: https://github.com/HumanSignal/label-studio/pull/4443
There is the Deploy Preview row in a table which I think might be an indication of an existing of a new build.

@Gondragos I think it does successfully build, then get updated to say "skipped" for some reason. See for example: https://app.netlify.com/sites/heartex-docs/deploys/65b194384af5560008ac732c

@jombooth Actually, I think you're right. From what I see there, it makes sense.
I just want @hlomzik to confirm that as I believe he could have some more context.


 name "foo" is already used by ID ""

Looks like the name store may accidentally get stuck in an improper state
$ sudo nerdctl run -it --rm --name=foo alpine
FATA[0000] name "foo" is already used by ID ""

$ sudo nerdctl rm -f foo
ERRO[0000] 1 errors:
no such container: foo

$ sudo nerdctl run -it --rm --name=foo alpine
FATA[0000] name "foo" is already used by ID ""

Probably a recent regression in the main branch.
Workaround: sudo rm /var/lib/nerdctl/1935db59/names/default/foo

Likely #2992
This is definitely (still) happening when restarting containers is involved.
It might be time to consider pulling the plug on the namestore...

I have #3357 for a specific case of this - just not sure this is the same.
lmk if you have a repro for yours here.


Cannot find ASAP_embedding_CLRadt.rds file

Hi, I'm trying to reproduce your experiment on the DOGMA-seq dataset. I tried to download the specified files from your provided link https://github.com/caleblareau/asap_reproducibility, but I cannot find the last file ASAP_embedding_CLRadt.rds. Could you provide the download link?

Hi, thanks for the question. Apperantly, the authors of that repo modified the files after we downloaded the data.

I can share with you this rds file through email privately. Can you provide me with your email, or can you send a email to<EMAIL_ADDRESS>so that I can sent it to you?

@.*** This is my email. Thanks :-)
严旭画
@.***
Original Email
Sender:"Du Jinhong"< @.*** >;
Sent Time:2023/7/31 20:45
To:"jaydu1/scVAEIT"< @.*** >;
Cc recipient:"XiHuYan"< @.*** >;"Author"< @.*** >;
Subject:Re: [jaydu1/scVAEIT] Cannot find ASAP_embedding_CLRadt.rds file(Issue #2)
Hi, thanks for the question. Apperantly, the authors of that repo modified the files after we downloaded the data.
I can share with you this rds file through email privately. Can you provide me with your email, or can you send a email to @.*** so that I can sent it to you?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

Sorry I cannot see the email address from your reply. Not sure if this is because of seurity reason when you reply through email. Can you send it again on the web?


Question about the MIM algorithm in OPEN-STL

rain loss: 0.2278 | data time: 0.1920: 100%|██████████| 57/57 [01:02<00:00,  1.10s/it]
[>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>> ] 32/33, 1.3 task/s, elapsed: 24s, ETA:     1sTraceback (most recent call last):
File "H:/OpenSTL-master/tools/train.py", line 42, in
exp.train()
File "H:\OpenSTL-master\openstl\api\train.py", line 332, in train
vali_loss = self.vali()
File "H:\OpenSTL-master\openstl\api\train.py", line 354, in vali
results, eval_log = self.method.vali_one_epoch(self, self.vali_loader)
File "H:\OpenSTL-master\openstl\methods\base_method.py", line 205, in vali_one_epoch
results = self._nondist_forward_collect(vali_loader, len(vali_loader.dataset), gather_data=False)
File "H:\OpenSTL-master\openstl\methods\base_method.py", line 166, in _nondist_forward_collect
pred_y = self._predict(batch_x, batch_y)
File "H:\OpenSTL-master\openstl\methods\predrnn.py", line 56, in _predict
img_gen, _ = self.model(test_dat, real_input_flag, return_loss=False)
File "E:\Anaconda3\envs\pytorch-gpu-openSTL\lib\site-packages\torch\nn\modules\module.py", line 1102, in _call_impl
return forward_call(*input, **kwargs)
File "H:\OpenSTL-master\openstl\models\mim_model.py", line 105, in forward
h_t[i], c_t[i], st_memory = self.stlstm_layer[i](
File "E:\Anaconda3\envs\pytorch-gpu-openSTL\lib\site-packages\torch\nn\modules\module.py", line 1102, in _call_impl
return forward_call(*input, **kwargs)
File "H:\OpenSTL-master\openstl\modules\mim_modules.py", line 133, in forward
cell = torch.cat((new_c, new_m), 1)
RuntimeError: Sizes of tensors must match except in dimension 1. Expected size 2 but got size 1 for tensor number 1 in the list.
Applying the MIM algorithm causes this problem at the end of training, resulting in a non-functioning？

This algorithm requires args.batch_size == args.val_batch_size, did you change the val_batch_size?

I will close this issue if there is no more question. Please feel free to open a new issue when you have new questions.


Galleries do not render

Error when trying to use a page bundle gallery. Does it matter that this is a project?
projects
|__/gallery
|__index.md
Inside index.md:
{{< gallery album="gallery" >}}
Error: Error building site: "/home/christian/website.d/website/content/projects/construction-products/index.md:23:1": failed to render shortcode "gallery": failed to process shortcode: "/home/christian/website.d/website/themes/academic/layouts/shortcodes/gallery.html:22:17": execute of template failed: template: shortcodes/gallery.html:22:17: executing "shortcodes/gallery.html" at <.Resize>: can't evaluate field Resize in type resource.Resource

Yes, a gallery can be created in a project. Just use the {{< gallery >}} shortcode in the Markdown body of your project and upload your photos to a gallery folder within your project's folder, as per the docs.

I have tried this and still get the same error as before. Even with the gallery named “gallery.”

Looks like there was an issue with capitalization in the image extension. All works now.


feat: Expose Setting for GRPC Channel-Level Compression at Client Side

All Submissions:

[x] Contributions should target the dev branch. Did you create your branch from dev?
[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

New Feature Submissions:

[x] Does your submission pass tests?
[x] Have you installed pre-commit with pip3 install pre-commit and set up hooks with pre-commit install?

Changes to Core Features:

[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[x] Have you written new tests for your core changes, as applicable?
[x] Have you successfully ran tests with your changes locally?

Issue
This PR refers to this issue #87 . It enables client to set GZip, grpc compression algorithm. By default no algorithm is applied.
Following code was modified

1st commit: functions in qdrant_client/connection.py

get_channel
get_async_channel

2nd commit: remote classes

class QdrantRemote in qdrant_client/qdrant_remote.py
class AsyncQdrantRemote in qdrant_client/async_qdrant_remote.py

3rd commit: client classes and grpc compression enum

class QdrantClient in qdrant_client/qdrant_client.py
class AsyncQdrantClient in qdrant_client/async_qdrant_client.py
qdrant_client/grpc/__init__.py

Usage
Gzip compression algorithm
from qdrant_client import QdrantClient
from qdrant_client.grpc import Compression

client = QdrantClient(
	prefer_grpc=True,
	grpc_compression=Compression.Gzip
)

No compression algorithm
from qdrant_client import QdrantClient
from qdrant_client.grpc import Compression

# client = QdrantClient(prefer_grpc=True)
# client = QdrantClient(prefer_grpc=True, grpc_compression=None)
client = QdrantClient(prefer_grpc=True, grpc_compression=Compression.NoCompression)

Hi @geetu040
Thanks for the contribution!
Do you think it would be possible to add some tests?

I have added the test cases

This PR Closes #87

Hi @joein
If you think the changes are all good to go, can you merge them?

Hey
Thanks for the contribution and sorry for the late response
Actually we would not like to extend client's signature with this parameter, since the signature is already large.
We haven't seen a big demand for this option either.
Thus, we would like to keep it in **kwargs

Hey @joein
I have moved grpc_compression parameter from client's signature to **kwargs

eh, found a discrepancy between pre-commit setup and async-client-generator, should be fixed by now

@geetu040 thanks for the contribution!


Add a method to print the DataFrame schema in code format

https://github.com/MrPowers/spark-daria/blob/master/src/main/scala/com/github/mrpowers/spark/daria/sql/DataFrameExt.scala

I can do it.

@SemyonSinchenko - awesome, assigned it to you!


Fix build issues

Fix a couple of issues affecting the build. These are only causing intermittent failures and I'm not exactly sure why, but this should hopefully bring things back to green.

Warnings

:warning:
It looks like this PR contains functional changes without a corresponding test.

  Generated by :no_entry_sign: dangerJS against bb6d4e70a67a2b7af1bc7db0dbb3cd65bde8fa0c

I was going to ask this in my next PR but I guess it fits here as well. How should I handle changes that will affect the functional tests (such as the removal of the weights and maxIssues). As far as I understood the functional tests use the latest version of detekt. I did change the tests in the PR but then the build failed.

As far as I understood the functional tests use the latest version of detekt

They should, but they're not in all cases for some reason. I have a POC fix in #6415 which is ugly but works and might be the way forward. We clearly need a reliable solution.

@detekt/maintainers please review. CI should be green going forward, and this should also catch missed updates to Gradle plugin functional test updates in future PRs.

I'll go ahead and merge to unblock other PRs.


Amend link from NHS start page pattern to GOV pattern

https://design-system.service.gov.uk/patterns/start-pages/ has moved to https://design-system.service.gov.uk/patterns/start-using-a-service/

Cancelled this task. We need to review our whole start page pattern in the light of the new GOV pattern: https://design-system.service.gov.uk/patterns/start-using-a-service/.


Remove unused code

https://github.com/smallfoot47/SupplyChainSecurity/blob/2f8a99b837b8f0d2069435f82e34a6f0d781a21d/util.py#L36
It reduces readability

Removed


printf rewrite

The current printf implementation is -- with all due respect to the original author -- a mess. So I'm rewriting it completely to be smaller and more readable.
I'm starting with just the printf function that matches the libc function. See https://github.com/uutils/coreutils/issues/5126. That means that I'm intentionally regressing the util printf. Once the function is done, we can add the functionality of the util back in, or use the internal functions of this implementation to build it.
The main changes are:

Parsing the format string and formatting are separated instead of weirdly combined.
Naming is (in my opinion) more logical (e.g. no more Memo and Sub).
Parsing no longer uses the confusing technique using itertools::PutBackN.
The entire implementation is reduced from 19 files to just 2.
The printf and sprintf functions no longer take slices of strings, but an iterator of an FormatArgument enum, which can specify integers and floats directly.

This is very much a WIP and does not compile the utils that use it. It needs some more work and tests, but I figured I'd commit this snapshot and share it. I've got most of the functionality and only one fourth of the original code size at the moment.

lot of conflicts, sorry!

I think most are easy to solve because this is a full rewrite of the functionality. I'll look into it soon.

So changing printf required me to change seq quite drastically... This is taking longer than expected.


The workspaceContains activation is not working correctly for Workspaces

VSCode Version: 1.18-insiders
OS Version: Windows10

Steps to Reproduce:
1.Pick an extension that has workspaceContains in its activationEvents
2.Open a folder that does not contains the file that activates the previous extension
3.The extension do not activates ...(which is correct)
4.Create a workspace with that folder
5.Add another folder that contains the file to activate the extension
6.The extension still does not activates ... (which is NOT correct)
7.Now.. another test
8.Close the Workspace
9.Reopen the Workspace
10.Now, the extension activates (which I guess, is correct)
Reproduces without extensions: No

@alefragnani I tried to reproduce and cannot, here is what I did:

set "activationEvents": ["workspaceContains:package.json"]
open a folder that has no top level package.json
extension does not activate
add another folder with a top level package.json
=> extension activates

What are you doing differently?

@bpasero, sorry for taking so long... it was my mistake. The extensions are activating correctly, in all scenarios.
Thanks for your help 👍


Data present under Modelica/Utilities

Reported by otter on 16 Oct 2010 22:36 UTC
For an example model, data is present under:
Modelica/Utilities/data

This should be moved to Modelica/Resources

Migrated-From: https://trac.modelica.org/Modelica/ticket/438

Comment by otter on 16 Oct 2010 22:38 UTC
Fixed in 9549f506f7ea5eb60781f983c69eb615ffee9be1.

Modified by dietmarw on 19 Oct 2010 06:08 UTC


Add mustReferenceOutput constraint tests

Pre-submit checklist:

Branch

[ ] Tests are provided (if possible)
[ ] Commit sequence broadly makes sense
[ ] Key commits have useful messages
[ ] Formatting, PNG optimization, etc. are updated

PR

[ ] Self-reviewed the diff
[ ] Useful pull request description
[ ] Reference the ADR in the PR and reference the PR in the ADR (if revelant)
[ ] Reviewer requested

You might want to use Versioned to reduce the complexity introduced by the V1 vs V2 tests. There's an example on the branch of my current ticket. At the moment, I think it's the easiest way to swap from between versions, but as it requires a bit of rewriting, I'd understand that you prefer to stick to this version.


#969: Inject credentials to ISettings to pass them into PushRunner

PR Summary
Inject credentials passed to Publish-PSResource into ISettings object in order to pass them along to the PushRunner that pushes the .nupkg to the feed.
Should solve #969
PR Context
This allows publishing PS resources to an authenticated feed.
PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[x] Summarized changes
[x] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[x] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: or [ WIP ] to the beginning of the title (the WIP bot will keep its status check at Pending while the prefix is present) and remove the prefix when the PR is ready.

Breaking changes

[x] None
OR
[ ] Documentation needed

[ ] Issue filed:

User-facing changes

[x] Not Applicable
OR
[ ] Documentation needed

[ ] Issue filed:

Testing - New and feature

[x] N/A or can only be tested interactively
OR
[ ] Make sure you've added a new test if existing tests do not effectively test the code changed

Tooling

[x] I have considered the user experience from a tooling perspective and don't believe tooling will be impacted.
OR
[ ] I have considered the user experience from a tooling perspective and enumerated concerns in the summary.

Hi all,
I'd like to test this change but based on the CHANGELOG this didn't make it into 3.0.21-beta21? #1091

I forgot to include this in the changelog, but it should be included in the beta21 code changes. We're aiming to release later today so the changes would be included when it releases.