[Proposal] Cursor Handle drag should be customizable as per native platform (Android/IOS)

Steps to Reproduce

Execute flutter run on the code sample
See the difference

As you can see, when we drag the cursor on native text field, it goes through the whole letter but, on the Flutter version, it jumps through the letters. On native, the cursor can go through the whole availble space as well, but on Flutter, it can go only through the text.
Expected results:
https://user-images.githubusercontent.com/45696119/151722759-902c06e0-7b93-4d06-a10d-1ee4b585f21a.mp4
Actual results:
https://user-images.githubusercontent.com/45696119/151722769-eed1d050-5711-4c89-ab58-23988ecda77d.mp4

Code sample

import 'package:flutter/material.dart';

void main() => runApp(MyApp());

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      debugShowCheckedModeBanner: false,
      theme: ThemeData(
        primarySwatch: Colors.blue,
      ),
      home: Scaffold(
        body: Container(
          alignment: Alignment.center,
          margin: const EdgeInsets.all(20.0),
          child: const TextField(
            decoration: InputDecoration(
              hintText: 'TYPE YOUR TEXT',
            ),
          ),
        ),
      ),
    );
  }
}

Logs

[√] Flutter (Channel stable, 2.8.1, on Microsoft Windows [versÃ£o 10.0.22000.434], locale pt-BR)
    • Flutter version 2.8.1 at C:\Users\bruno\fvm\versions\2.8.1
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 77d935af4d (6 weeks ago), 2021-12-16 08:37:33 -0800
    • Engine revision 890a5fca2e
    • Dart version 2.15.1

[√] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at C:\Users\bruno\AppData\Local\Android\sdk
    • Platform android-31, build-tools 31.0.0
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)
    • All Android licenses accepted.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[√] Visual Studio - develop for Windows (Visual Studio Community 2019 16.11.7)
    • Visual Studio at C:\Program Files (x86)\Microsoft Visual Studio\2019\Community
    • Visual Studio Community 2019 version 16.11.31911.196
    • Windows 10 SDK version 10.0.19041.0

[√] Android Studio (version 2020.3)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)

[√] VS Code (version 1.63.2)
    • VS Code at C:\Users\bruno\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.32.0

[√] Connected device (3 available)
    • Windows (desktop) • windows • windows-x64    • Microsoft Windows [versÃ£o 10.0.22000.434]
    • Chrome (web)      • chrome  • web-javascript • Google Chrome 97.0.4692.99
    • Edge (web)        • edge    • web-javascript • Microsoft Edge 97.0.1072.76

• No issues found!

Tested on Samsung devices.

Hi @bdlukaa, Thanks for filing the issue. Although WhatsApp may not be the right app to compare the native behavior But I am seeing similar behavior in both native and flutter handle goes through each letter when dragged.
Skip to 18 secs in the flutter video to see handle drag

Native (Android)
Flutter

Ps: On a side note handle drag doesn't seem to work when there is space between letters

Although WhatsApp may not be the right app to compare the native behavior

The same can be tested from other apps as well:

telegram
instagram

discord
google translator

Android Version: 11
As you can see above, the cursor is a lot smoother than the Flutter version. It moves above the whole letter, while the Flutter version jumps through the letters.
Also, the native cursor can move through the whole text field width, while the flutter version can only go through text

Thanks for the detailed analysis @bdlukaa, I am quite not sure How are you able to drag the cursor when there is no text entered in the textfield, I have checked this on Instagram and telegram none of the app allows dragging a cursor on an empty textfield. Also the drag behavior looks similar to whatsApp
Instagram
https://user-images.githubusercontent.com/31410839/151808042-b63ca8e7-4e9b-41c0-ab1a-dc896eb2d2cb.mov
Also, may I know what device are you seeing this behavior?
cc: @justinmc

Also, may I know what device are you seeing this behavior?

Tested on Samsung A32. Android 11

Thanks for the info @bdlukaa, Although I think the behavior that you are seeing is device-specific, since I don't see that behavior on other devices Verified on Xiaomi k20 pro (Android 11), Poco M3 (Android 10), Pixel 4 emulator (API 30), and none of the devices show the behavior that you are seeing. Would it be possible for you to confirm this behavior on other devices (Non samsung)?

Sure! I tested on the devices I have home and it seems that it's only new Samsung devices that have this

Samsung A32
Samsung M22
Xiomi Redmi 7
Motorola Moto X4
Samsung J5

✅
✅
❌
❌
❌

Thanks for your analysis @bdlukaa That certainly makes me think that the flutter cursor handle is working as intended and matches the native android platform and what you are seeing is specific to new Samsung Devices,
Do you still think this issue is valid?

@maheshmnj Well, I think we should have different ways to manage cursors. Maybe add the developer the option to change the cursor behavior with something like the following:
TextField(
  cursorBehavior: const JumpCursorBehavior(), // for the default android behavior
);

Could have some like:

CupertinoCursorBehavior(), for the cursor behavior like ios'
GoThroughCursorBehavior(), for the cursor behavior like samsung's, as I showed above

It makes sense to me that Flutter should make this behavior possible one way or another, so let's keep the issue open.  I'm not sure how easy it would be to provide this though.

Thanks for the update @bdlukaa, That does look like a valid proposal.

1000 times yes to this. Flutter has come a long way in feeling more native, but TextField/Input behavior (especially cursor dragging) is one area where it still falls very short. This is particularly problematic in apps that are text heavy (chat, note-taking, etc.). Years of text messaging have set a very high bar in all users's minds. Thanks for any efforts towards improving this!


Claim abandon fails - 64: non-mandatory-script-verify-flag (Data push larger than necessary)

When abandoning certain claims, they are rejected by the mempool.  Not sure if this is related to the dust issue we saw in the past (https://github.com/lbryio/lbrycrd/issues/186) as the value here is higher.

64: non-mandatory-script-verify-flag (Data push larger than necessary)
[0100000001a7bfdb31abc9ca5eed68fc7da260476cd1e1d5216186d142ff8c150c1014fd36000000006a47304402206df8764da78d06d2c721b788803e0114539408e171280655c783bc7e861b693a02204f563d71754ce7fe648f35a2baa951d2afb7e868880ac0d3b819b3dd10f27366012103d07fe8a0deb7423f552bd1c182251c8a926951f688801e3b2ba681148783a506ffffffff0174180f00000000001976a914322ab934ea45bde72d3f9c8bb0dc00eb0b031f0188ac00000000]

decoded tx:
{
  "txid": "f913c3b658ba4715000207fd2b925b8ea581e62c1ba093d74ba91f197d1d9c31",
  "size": 191,
  "version": 1,
  "locktime": 0,
  "vin": [
    {
      "txid": "36fd14100c158cff42d1866121d5e1d16c4760a27dfc68ed5ecac9ab31dbbfa7",
      "vout": 0,
      "scriptSig": {
        "asm": "304402206df8764da78d06d2c721b788803e0114539408e171280655c783bc7e861b693a02204f563d71754ce7fe648f35a2baa951d2afb7e868880ac0d3b819b3dd10f27366[ALL] 03d07fe8a0deb7423f552bd1c182251c8a926951f688801e3b2ba681148783a506",
        "hex": "47304402206df8764da78d06d2c721b788803e0114539408e171280655c783bc7e861b693a02204f563d71754ce7fe648f35a2baa951d2afb7e868880ac0d3b819b3dd10f27366012103d07fe8a0deb7423f552bd1c182251c8a926951f688801e3b2ba681148783a506"
      },
      "sequence":<PHONE_NUMBER>
    }
  ],
  "vout": [
    {
      "value": 0.00989300,
      "n": 0,
      "scriptPubKey": {
        "asm": "OP_DUP OP_HASH160 322ab934ea45bde72d3f9c8bb0dc00eb0b031f01 OP_EQUALVERIFY OP_CHECKSIG",
        "hex": "76a914322ab934ea45bde72d3f9c8bb0dc00eb0b031f0188ac",
        "reqSigs": 1,
        "type": "pubkeyhash",
        "addresses": [
          "bHJXfJPMdafmSgyH2vcWLhPsfLJhwcb8uy"
        ]
      }
    }
  ]
}```

I put @kaykurokawa 's suggested fix into master. It now disables the minimal push validation on claims.


Remove DataStreams support

Since the library itself is deprecated, we should remove support for it here.
Tables.jl should be an adequate replacement, and that is already implemented.
Doing this will allow an AbstractResult type which may be useful for #84

@iamed2 , any chance you can provide an example on how to pull data using the Tables.jl interface?
Currnetly Data.close!(Data.stream!(result, DataFrame)) does not work because DataFrames no longer supports DataStreams.

I was about to open a PR to document this, however I realized I'm still not quite sure how to do inserts with Tables.jl.

Yep, I have a branch where I'm working on this and I'm replacing DataStreams with Tables in all the docs. Insertion can be done in an ugly way with a loop or broadcast but I think I'll try to support inserting Tables directly.
It sucks, but this week I've been basically unavailable. This will absolutely be solved by mid-next-week.


[bug] sun and moon stopped revolving

at some point after about 30 minutes in one of my sessions the sun and moon stopped revolving around
i noticed this and flew to the two poles in a circle for a minute or two watching as neither celestial body moved
no idea what caused this error

Interesting, I'll have to investigate that... Thanks!

I haven't seen this happening in the new version so closing for now.


Allow HTTP::Client to use ipv6 addresses

Fixes #5514
The URI#host is fine to have the brackets since they are part of rfc3986 host description.
The HTTP::Client needs to extract them before creating the TCPSocket.

I moved the unwrap to URI, but since in http client @host is String, it needs to be URI.hostname(host : String). But I also added URI#hostname nevertheless.
Regarding benchmark, the original code wins:
require "benchmark"

host = "[test#{Random.new.next_int}]"
pp host
pp host[0] == '[' && host[-1] == ']' ? host[1..-2] : host

Benchmark.ips do |x|
  x.report("index") do
    host[0] == '[' && host[-1] == ']' ? host[1..-2] : host
  end

  x.report("chop") do
    host.lchop('[').rchop(']')
  end
end

$ crystal bench.cr --release
host # => "[test778306676]"
(host[0] == '[') && (host[-1] == ']') ? host[1..-2] : host # => "test778306676"
index  15.24M ( 65.62ns) (± 7.53%)       fastest
 chop  10.08M ( 99.24ns) (± 8.62%)  1.51× slower

@bcardiff Sorry for the mistake. @host.starts_with?('[') && @host.ends_with?(']') seems faster, at least it is on my side.

We can remove the ugly URI.hostname if we move the normalization of @host to the HTTP::Client constructor which takes URI.

@RX14 that won't work. There are initializers that use String and not URI.
But I am ok to mark URI.hostname as :nodoc:.

@bcardiff that's fine - if someone passes HTTP::Client.new("[...]") they can fail.

@RX14 I don't agree. clients initialized with HTTP::Client.new("[::1]") will work. And actually HTTP::Client.new("::1") won't, or at least it is not guaranteed. The host http header are supposed to have a [::1] and not ::1. So the @host needs to have the brackets as in URI.
Otherwise we need to add logic to wrap the host upon client. It can be done, but is a different issue.
Bottom line @hostin  HTTP::Client is expected to match URI#host.

Well I really hate URI.hostname, so let's just open-code the removal of the [] into the method, like this was previously.
Then add URI#hostname in this or another PR.

But that would lead to two implementation of the same thing. Let's leave URI.hostname as :nodoc: and already have URI#hostname.
I'm satisfied with the state of the PR.

A bit of copy and paste really isn't that bad, especially for a snippet of code so small and unlikely to change as this.

I agree. Copying once is okayish. Copying a second time should trigger a refactor, but once is acceptable.

Done, the PR should be squashed on merge. (I have the feeling that URI.hostname will come back sooner that later, but ok)


Mnemonic functions instead of structs

Neat project! Perusing the source and had some thoughts -- take 'em or leave 'em :)
The following is a bit of a mouthful:
    push_eagerly_encoded_assembly(
      &builder.code_block, source_range,
      &(Instruction_Assembly){call, {instance->storage}}
    );

and might be a little more readable as:
call(&builder.code_block, source_range, instance->storage);

It might be possible to do this by defining a bunch of functions in the mnemonic macro in instructions.c but I am not sure how to handle multiple arities from function arguments -> struct members.

Neat project!

Thanks!

call(&builder.code_block, source_range, instance->storage);

I've thought about something like this as well, but there is an issue - many instructions support variable number of operands. In C language that would require either doing varargs with a null terminator or do some fancy tricks with non-standard macros. With former it is not going to be nice to use and with latter it is a bit of a nightmare to maintain.
So for now I decided that sticking with a more verbose form is OK.
Thanks for sharing your thoughts though - I appreciate any and all interest in the project!


better GuardDisable

maybe we don't need this blacklist table just use something like this
local ok, data = pcall(vim.api.nvim_get_autocmds,{group ='Guard',event = 'FileType', pattern = 'lua' })
if ok then
    vim.api.nvim_del_autocmd(data.id)
end

Conflicts getting messy, closing for now.

use git rebase then resolve conflict 😄

Yeah, I will also try to make the code structure cleaner, some files are a bit cluttered now.

thanks :)


[Bug] Emerald ores don't spawn

Minecraft Version
1.16.5
Forge Version
Latest
Mod Version
v1.16.5 - 2.1.0
Describe the bug
Emerald ores don't spawn due to they having a to low veinSize.
How to reproduce
Use default world gen settings and create a world.
Link to crashlog [If Applicable]
None
Screenshots / Images
None

Fixed in OreTweaker-1.16.5-2.2.1.jar


⚠ Cioos National CKAN has degraded performance

In ac93eff, Cioos National CKAN (https://catalogue.cioos.ca/) experienced degraded performance:

HTTP code: 200
Response time: 6155 ms

Resolved: Cioos National CKAN performance has improved in 3838997 after 59 minutes.


Multiple instances of DateTime.MinValue.ToUniversal()

This code acts weird on machines that are not running under UTC.
See #2785 for more context. We should do a pass in all of the NuGet.Services.Metadata code for this problem.

I think this was done


Update to latest tooling & dependencies

@Krzysztof-Cieslak can you please review this PR?
I did my best to update dependencies and tooling (using ionide repo as reference), but I believe that it is still can be simplified.

@Krzysztof-Cieslak do you think it can be merged? Or you prefer to keep this repo to be synced with your blog post?

Thanks a lot for the update ❤

Just for the record:
Here is one more vscode extension build using this template + .net backend using Saturn + Fable.Remoting
https://github.com/sergey-tihon/vscode-openxml-explorer


Paths aren't clipping properly? (not sure what else to call this behavior)

Describe the bug
Paths aren't clipping into the outer box like they should.

To Reproduce
Steps to reproduce the behavior:
Input:
<svg xmlns="http://www.w3.org/2000/svg" enable-background="new 0 0 24 24" height="24px" viewBox="0 0 24 24" width="24px" fill="#000000">
  <g>
    <rect fill="none" height="24" width="24"/>
  </g>
  <g>
    <path d="M21,1H3C1.9,1,1,1.9,1,3v18c0,1.1,0.9,2,2,2h18c1.1,0,2-0.9,2-2V3C23,1.9,22.1,1,21,1z M19,19H6.83l1.58,1.58L7,22l-4-4 l4-4l1.41,1.42L6.83,17H17v-4h2V19z M17,10l-1.41-1.42L17.17,7H7v4H5V5h12.17l-1.58-1.58L17,2l4,4L17,10z"/>
  </g>
</svg>

Output
<svg xmlns="http://www.w3.org/2000/svg" width="24" height="24">
  <rect fill="none" height="24" width="24"/>
  <path d="M21 1H3c-1.1 0-2 .9-2 2v18c0 1.1.9 2 2 2h18c1.1 0 2-.9 2-2V3c0-1.1-.9-2-2-2zM7 7h10v3l4-4-4-4v3H5v6h2V7zm10 10H7v-3l-4 4 4 4v-3h12v-6h-2v4z"/>
</svg>

This is my current config:
module.exports = {
  multipass: true,
  plugins: [
    'cleanupEnableBackground',
    'collapseGroups',
    'convertPathData',
    'mergePaths',
    'removeTitle',
    'removeViewBox',
    'sortAttrs'
  ]
}

Expected behavior
Desired SVG:

What I get:

Desktop (please complete the following information):

SVGO Version ^2.3.0
NodeJs Version v15.14.0
OS: Debian 10

Ahhh, fill-rule. I completely missed that. I was stuck on the paths. Thanks for pointing that out!


Added Firefox Developer Edition version 91.0b9.

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.0 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

This time it didn't work.


How do I run deferred tests -  e.g. suite.run({async: true})

Seems to be impossible to pass this option.

I thought you can use all options during creation.
Probably it was changed in Benchmark 2.0.0
I've reworked from gulp-bench which sent options into constructor of Benchmark.
Here are links to how it was going to work:
https://github.com/VicNumber21/gulp-benchmark#object-with-function-and-options
https://github.com/VicNumber21/gulp-benchmark#array-of-objects
You need just add your options ("async: true" from your example) to object which describes test / suite.
Wasn't it work for you?


Rolled back to more stable powershell sdk version

Docs Build status updates of commit b972775:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

azure-pipelines/update-template.yml
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


OCU-1048 CDH 5.13.3 upgrade

For Omnia Cloudera upgrade. Bumps uniform master from v1.15.3 (CDH 5.3.0) to v2.1.1 (CDH 5.13.3)

All looks very familiar.


Add option for re-running failed tests

Additional option '--re-run' for testing infrastructure.
To run one or more failed tests during the last 'test-backend' run.
Failed test info stored at zulip/var/last_test_failure.json
Tested on Ubuntu 16.04, would like to hear feedbacks on additions
or issues.
@showell @timabbott I hope this meets the needs. :)

Automated message from Dropbox CLA bot
@isht3, it looks like you've already signed the Dropbox CLA. Thanks!

This change looks pretty solid.  I think the only cleanup is that we need to enforce the command line protocol more directly and earlier.

Check for mypy annotation error in test_runner.py

This looks great!  Merged, thanks @isht3!  I made a small tweak to the argument parsing to reflect the fact that the new argument implied --nonfatal-errors now.


➕ Add Rave Names resolution

Add Rave Names resolution into the useLookupAddress hook.
The hook will by default resolve to ENS, but if a second optional parameter is set to true, the hook will resolve to a Rave Name.

Updated: https://github.com/TrueFiEng/useDApp/pull/1053/commits/f59a0b1eef14d2f59054f7a0d6b82f7c356ab574


Add iree_vm_value_{none,i8,i16} functions.

The _none function leaves the data union uninitialized. If you want something else, please specify what. Do we want to zero the full 8 bytes?

Generally agreed for user-visible things - but in this case the storage is an implementation detail of the type. Fixing the https://github.com/google/iree/issues/5542 TODO will prevent users from misusing the internal storage is better than initialized to 0 as that also has heisenbug potential and is hard to differentiate valid usage from invalid usage ("is the zero I read out something intended or a default initializer? if initialized to i16 0xFF00 and read as i8 0x00, is that part of the real value or an default initialized one, etc). If you have a variant with a type and then don't check or incorrectly map the type then you have a bug in your code and you want to know about that via loud errors (asserts/etc).
Basically, if it's invalid behavior we should be asserting instead of relying on default initialization - the 0 initialization is a safety net for broken code, not something that should be considered intentional behavior.
As for !NDEBUG, ASAN should be able to catch the use of uninitialized stack memory otherwise. If the user-visible API asserts then the !NDEBUG init is most useful for people like me who use debuggers and don't want to see garbage when looking at structs.


Every new environment remove the previous global privileges

Hello,
I am trying to add the account privilege EXECUTE TASK to a specific role for each environment using this following code:
resource "snowflake_account_grant" "execute_task" {
  roles             = [snowflake_role.main["CREATOR"].name]
  privilege         = "EXECUTE TASK"
  with_grant_option = false
}

snowflake_role.main["CREATOR"].name being something like PROD_CREATOR. The problem with that is that it will remove the role when applying to a new workspace. For example, if I am applying to DEV, it will remove PROD_CREATOR and add DEV_CREATOR because it is a global resource between workspaces. How can I do it to not remove previous roles?
Thanks in advance

Can you elaborate when saying applying to DEV and your definition of workspaces?


Review docs and examples

There are many changes to objects, e.g. ldap message objects, that may require changes to the docs and examples.

Nothing stands out as needing to be changed outside of the complete overhaul already noted in #544.


Timeout waiting for port

Environment
get-port-please v3.0.1
Node.js v16.20.0
Reproduction
https://stackblitz.com/edit/stackblitz-starters-vdm76f?description=Starter project for Node.js, a JavaScript runtime built on Chrome's V8 JavaScript engine&file=index.mjs,package-lock.json&title=node.new Starter
Describe the bug
I get
Error: Timeout waiting for port 59728 after 4 retries with 500ms interval.
    at async ESMLoader.import (https://stackblitzstartersvdm76f-xrf4.w-credentialless.staticblitz.com/blitz.33975f1d.js:213:4862)
    at async i.loadESM (https://stackblitzstartersvdm76f-xrf4.w-credentialless.staticblitz.com/blitz.33975f1d.js:60:915)
    at async handleMainPromise (https://stackblitzstartersvdm76f-xrf4.w-credentialless.staticblitz.com/blitz.33975f1d.js:168:264)

when running await waitForPort().
It happens on StackBlitz and MacOS 13.5 (Node 16.20.0, 18.16.1 when running Nitro tests).
Additional context
No response
Logs
No response

Hmm, I'm running into this when attempting to run playwright tests using nuxt/test-utils in a Github Action.

Hey I am getting the same error with the nuxt/test-utils in the Github Action, but sometimes locally as well.
GetPortError: Timeout waiting for port 38561 after 20 retries with 500ms interval.
 ❯ Module.waitForPort node_modules/.pnpm/get-port-please@3.1.2/node_modules/get-port-please/dist/index.mjs:348:9


Two Korean classification datasets added

There are 82 million speakers of Korean (acc to wiki https://en.wikipedia.org/wiki/List_of_languages_by_total_number_of_speakers), yet there was only one dedicated classification task. This adds two classification:

sarcasm detection: https://huggingface.co/datasets/kor_sarcasm
tweet hate classification: https://huggingface.co/datasets/kor_hate

Checklist for adding MMTEB dataset

Reason for dataset addition:

[x] I have tested that the dataset runs with the mteb package.
[x] I have run the following models on the task (adding the results to the pr). These can be run using the mteb run -m {model_name} -t {task_name} command.

[x] sentence-transformers/paraphrase-multilingual-MiniLM-L12-v2
[x] intfloat/multilingual-e5-small

[x] I have checked that the performance is neither trivial (both models gain close to perfect scores) nor random (both models gain close to random scores).
[x] If the dataset is too big (e.g. >2048 examples), considering using self.stratified_subsampling() under dataset_transform()
[x] I have filled out the metadata object in the dataset file (find documentation on it here).
[x] Run tests locally to make sure nothing is broken using make test.
[x] Run the formatter to format the code using make lint.
[x] I have added points for my submission to the points folder using the PR number as the filename (e.g. 438.jsonl).

I fixed it, so it uses self.stratified_subsampling now, thanks for suggestions @KennethEnevoldsen

Thanks @dokato! I have added you as a contributor to MTEB as well


Fix Dockerfile; install dependencies with glide missing

I forgot to actually install the dependencies with glide ...

will rebase


KeyError: 'LOGGING' after recent git pull

Hi
After having the bot run for about a month or two I decided to do a git pull two days ago.
Since then I'm not able to start it anymore, it gives me the following lines:
user@computer:~/gateio-crypto-trading-bot-binance-announcements-new-coins$ python3.9 main.py  Traceback (most recent call last):   File "/home/user/gateio-crypto-trading-bot-binance-announcements-new-coins/main.py", line 1, in <module>     from trade_client import *   File "/home/user/gateio-crypto-trading-bot-binance-announcements-new-coins/trade_client.py", line 1, in <module>     from logger import logger   File "/home/user/gateio-crypto-trading-bot-binance-announcements-new-coins/logger.py", line 15, in <module>     log_level = config['LOGGING']['LOG_LEVEL'] KeyError: 'LOGGING'
I already tried to recheck for requirements, but everything seems to be in order.
I also deleted the folder and cloned the code again, but the same error appears.
Is there anything I'm missing?
Any help/pointer appreciated

You need to use the new config template

Thank you for the hint, that did the trick!


Update Symbol-Packages.md

Fixing markup in last paragraph

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/Create-Packages/Symbol-Packages.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Thank you!


EPoch number does not make sense

I am training a dreambooth LORA,
Í have 18 training image, 1 repeat, batch = , gradient accumulation = 6, and 200 epoch, gradient checkpointing true.
But the output cell when starting training indicates that the training will last 1200 epochs.
It should be 200, not 1200 right?

gradient accumulation steps is basically batch size but for memory efficiency.
  # epoch数を計算する
  num_update_steps_per_epoch = math.ceil(len(train_dataloader) / args.gradient_accumulation_steps)
  num_train_epochs = math.ceil(args.max_train_steps / num_update_steps_per_epoch)
  if (args.save_n_epoch_ratio is not None) and (args.save_n_epoch_ratio > 0):
    args.save_every_n_epochs = math.floor(num_train_epochs / args.save_n_epoch_ratio) or 1

so to get valid max train steps, you need to multiply max_train_steps with args.gradient_accumulation_steps,
200*6 = 1200 steps.

Ok thanks.
On Tue, 14 Feb 2023, 02:03 Linaqruf @.***> wrote:

gradient accumulation steps is basically batch size but for memory
efficiency.
epoch数を計算する
num_update_steps_per_epoch = math.ceil(len(train_dataloader) / args.gradient_accumulation_steps)
num_train_epochs = math.ceil(args.max_train_steps / num_update_steps_per_epoch)
if (args.save_n_epoch_ratio is not None) and (args.save_n_epoch_ratio > 0):
args.save_every_n_epochs = math.floor(num_train_epochs / args.save_n_epoch_ratio) or 1

so to get valid max train steps, you need to multiply max_train_steps with
args.gradient_accumulation_steps,
200*6 = 1200 steps.
—
Reply to this email directly, view it on GitHub
https://github.com/Linaqruf/kohya-trainer/issues/82#issuecomment-1428942575,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AODP7RFIFNVKODQKKAJLONDWXLKXVANCNFSM6AAAAAAU2ZQ3TE
.
You are receiving this because you authored the thread.Message ID:
@.***>


Remove VersusGame import

Remove import of VersusGame which causes an error on script start
ImportError: cannot import name 'VersusGame' from 'src'

Yeah had the same issue, I also removed it and it works.

yeah I was looking at some old code when I was trying to import something - thank you for catching that


data: sejosephmartin

Fixes Issue

Changes proposed

Check List (Check all the applicable boxes)

[ ] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[ ] All new and existing tests passed.
[ ] This PR does not contain plagiarized content.
[ ] The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers

Also, move your file to data folder

can I move the file like this  data/sejosephmartin.json.. I change already some of the unnecessary script. should i create another branch when i commit this?


Windows WSL2: /app/storage/runtime isn't writable by PHP

Description
On a fresh Windows 10, WSL2 and Nitro install, I'm having this error when I try to access the site.
I'm getting errors out of composer too. Cache writing problems as can be seen in LOGS.
Everytime I'm trying to fix permissions, it's bugging me about another permissions.
Example: fixing permissions on storage/runtime leads to lack of permissions on storage/logs, then web/cpresources, ...
Steps to reproduce

Followed the windows https://craftcms.com/docs/nitro/2.x/installation.html docs
Created a site with nitro create xavierdemoor.com command
Chmoded like said in https://craftcms.com/docs/nitro/2.x/windows.html#set-permissions-when-adding-a-new-site

Additional info
SPECS:
Windows:         10 21H1
Nitro CLI:       2.0.8
Nitro gRPC:      2.0.8
Docker API:      1.41 (1.12 min)
Docker CLI:      1.41

LOGS:
xavier@maman:~/code$ nitro create xavierdemoor.com
Downloading https://github.com/craftcms/craft/archive/HEAD.zip ...
  … setting up project ✓
New site downloaded 🤓
Enter the hostname [xavierdemoor.com]: xavierdemoor.nitro
  ✓ setting hostname to xavierdemoor.nitro
  ✓ adding site ~/code/xavierdemoor.com
Enter the web root for the site [web]:
  ✓ using web root web
Choose a PHP version:
  1. 8.0
  2. 7.4
  3. 7.3
  4. 7.2
  5. 7.1
  6. 7.0
Enter your selection: 2
  ✓ setting PHP version 7.4
Add a database for the site [Y/n]
Enter the new database name: xavierdemoor
  … creating database xavierdemoor ✓
Database added 💪
Should we update the env file? [Y/n]
.env updated!
Preparing composer...
Cannot create cache directory /.composer/cache/repo/https---repo.packagist.org/, or directory is not writable. Proceeding without cache
Cannot create cache directory /.composer/cache/files/, or directory is not writable. Proceeding without cache
Cannot create cache directory /.composer/cache/repo/https---repo.packagist.org/, or directory is not writable. Proceeding without cache
Cannot create cache directory /.composer/cache/files/, or directory is not writable. Proceeding without cache
Loading composer repositories with package information
Updating dependencies
...
Generating optimized autoload files
50 packages you are using are looking for funding.
Use the `composer fund` command to find out more!
> @php -r "file_exists('.env') || copy('.env.example', '.env');"
> @php -r "unlink('composer.json');"
> @php -r "rename('composer.json.default', 'composer.json');"
Cannot create cache directory /.composer/cache/repo/https---repo.packagist.org/, or directory is not writable. Proceeding without cache
Cannot create cache directory /.composer/cache/files/, or directory is not writable. Proceeding without cache
Generating optimized autoload files
Generated optimized autoload files containing 4259 classes
> @php craft setup/welcome
...
Welcome to Craft CMS!

Run the following command if you want to setup Craft from your terminal:

    php ~/app/craft setup

composer create-project completed 🤘
Checking network…
  ✓ network ready
Checking proxy…
  ✓ proxy ready
Checking databases…
  … checking mysql-5.7-3306.database.nitro ✓
Checking services…
  … checking dynamodb service ✓
  … checking mailhog service ✓
  … checking redis service ✓
Checking sites…
  … checking xavierdemoor.nitro ✓
Checking proxy…
  … updating proxy ✓
Updating hosts file (you might be prompted for your password)
[sudo] password for xavier:
Cleaning up...
For your hostnames to work, add the following to `C:\Windows\System32\Drivers\etc\hosts`:
---- COPY BELOW ----
# <nitro>
<IP_ADDRESS> mysql-5.7-3306.database.nitro redis.service.nitro xavierdemoor.nitro
# </nitro>
---- COPY ABOVE ----
Nitro is up and running 😃

WSL PERMISSIONS:
xavier@maman:~/code/xavierdemoor.com$ ls -la
total 272
drwxr-xr-x  8 xavier xavier   4096 Aug  7 12:09 .
drwxr-xr-x  3 xavier xavier   4096 Aug  7 12:08 ..
-rwxrwxrwx  1 xavier xavier    621 Aug  7 12:09 .env
-rw-r--r--  1 xavier xavier    505 Aug  7 12:08 .env.example
-rw-r--r--  1 xavier xavier     31 Aug  7 12:08 .gitignore
-rwxrwxrwx  1 xavier xavier    475 Aug  7 12:08 composer.json
-rwxrwxrwx  1 xavier xavier 221645 Aug  7 12:09 composer.lock
drwxrwxrwx  4 xavier xavier   4096 Aug  7 12:12 config
-rwxr-xr-x  1 xavier xavier    770 Aug  7 12:08 craft
drwxr-xr-x  2 xavier xavier   4096 Aug  7 12:08 modules
drwxrwxrwx  4 xavier xavier   4096 Aug  7 12:09 storage
drwxr-xr-x  2 xavier xavier   4096 Aug  7 12:08 templates
drwxrwxrwx 39 xavier xavier   4096 Aug  7 12:09 vendor
drwxrwxrwx  3 xavier xavier   4096 Aug  7 12:08 web

After investing a bit into the container, I can see php-fpm and nginx process are running under user www-data (ID 82) but files are owned by user 1000 (user id of my user in WSL).
There isn't a better solution than chmoding 777 a bunch of things ?

Hi @xavierdemoor, copying my reply to you on Discord for visibility here:

The permissions on WSL2 and linux are a known issue. On macOS Docker Desktop, the application maps these permissions but the Linux specific Docker installs do not handle permissions between .
We have this issue which means you have to treat Nitro as more of a "Linux server" and set the permissions based on our documentation here: https://craftcms.com/docs/3.x/installation.html#step-2-set-the-file-permissions.
Following those steps is the best approach for Nitro and Docker based environments (comment here: https://github.com/craftcms/nitro/issues/240#issuecomment-782326053). We also have this PR (https://github.com/craftcms/nitro/pull/358) which adds a nitro permissions command, which might make its way into Nitro 3.0 (on the roadmap and in an internal build the team is using)  https://craftcms.com/roadmap

We can close this one, since there is no other way than chmodding 777 problematic folders for the moment.


Add button size class

Not sure about the naming.
Shall we prefix the classes with their property: -size-?

we prob need the same for icon button, link, etc.
also - maybe button-sm instead of button-size-sm ?

we prob need the same for icon button, link, etc.

Been thinking about that as well, but thought we'll add more and make it consistent in v3 anyway.

also - maybe button-sm instead of button-size-sm ?

So that was exactly my dilemma. Won't we run into naming issues with other properties somewhere if we don't prefix it with the property name?


Add Shiawase (VIP) X Thick Of It by Dion Timmer X KindleRar

Song Name
Shiawase (VIP) X Thick Of It
Artist Name
Dion Timmer X KindleRar
Source
Youtube
Youtube Link
ScHCrJIvv0Y
Direct File Link
No response
Song ID
1266014
Start Offset [ms]
0
Extra
Created from Auto Nong.
Created for level: 94555142
Downloaded: No

Hey man its gonna be cool if you all accept it
Em sex., 22 de nov. de 2024, 22:35, github-actions[bot] <
@.***> escreveu:

This submission is stale because it has been open for 14 days with no
activity. If it is still relevant, please leave a comment and we will
reopen it.
—
Reply to this email directly, view it on GitHub
https://github.com/FlafyDev/auto-nong-indexes/issues/6136#issuecomment-2495232826,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BMY3FOC7RCLACAWCTDCOHM32B7SYRAVCNFSM6AAAAABROHLRUKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIOJVGIZTEOBSGY
.
You are receiving this because you authored the thread.Message ID:
@.***>

Yes, I am working on it.
Em sáb., 7 de dez. de 2024, 22:50, github-actions[bot] <
@.***> escreveu:

This submission is stale because it has been open for 14 days with no
activity. If it is still relevant, please leave a comment and we will
reopen it.
—
Reply to this email directly, view it on GitHub
https://github.com/FlafyDev/auto-nong-indexes/issues/6136#issuecomment-2525390764,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BMY3FOECFQMQU5YNUWPSKAT2EOXW7AVCNFSM6AAAAABROHLRUKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMRVGM4TANZWGQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

Hi , I would like to reopen it
Em seg., 23 de dez. de 2024, 22:32, github-actions[bot] <
@.***> escreveu:

This submission is stale because it has been open for 14 days with no
activity. If it is still relevant, please leave a comment and we will
reopen it.
—
Reply to this email directly, view it on GitHub
https://github.com/FlafyDev/auto-nong-indexes/issues/6136#issuecomment-2560554720,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BMY3FOE4HOXS5TKO6G6N5QT2HDBSXAVCNFSM6AAAAABROHLRUKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNRQGU2TINZSGA
.
You are receiving this because you authored the thread.Message ID:
@.***>


make sure no resource has the same id for capability type

• Does any part of it require special attention?
• Does it relate to or fix any issue? closes #635
For now this only really will ever catch persist since no other resource type has multiple kinds, but checking them anyways in case they ever do.
Standard checks

Unit tests: Any special considerations? added
Docs: Do we need to update any docs, internal or public? no
Backwards compatibility: Will this break existing apps? If so, what would be the extra work required to keep them working? it shouldnt

fyi, this works, i needed to rebase from main locally because the bug from a couple days ago was still affecting this finding the persist annotations to begin with


Fixed Typo and Grammer

coverage: 93.889%. remained the same when pulling b92b7a5da8da2cfc290774f378c67e1e4e1f382a on Sanyam-2026:main into 4f10d6b3a60c1a3415bfab93441c86b8b8991754 on medplum:main.


[e2e]: generic push branch1 default slsa3

Repo: https://github.com/slsa-framework/example-package/tree/branch1
Run: https://github.com/slsa-framework/example-package/actions/runs/2654063061
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.generic.push.branch1.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.generic.push.branch1.default.slsa3.yml
Trigger: push
Branch: branch1
Date: Tue Jul 12 04:20:14 UTC 2022

Repo: https://github.com/slsa-framework/example-package/tree/branch1
Run: https://github.com/slsa-framework/example-package/actions/runs/2660899013
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.generic.push.branch1.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.generic.push.branch1.default.slsa3.yml
Trigger: push
Branch: branch1
Date: Wed Jul 13 04:06:32 UTC 2022
Tests are passing now. Closing this issue.


Do not panic generating accessors for large bitfields

This input
struct HasBigBitfield {
    __int128 x : 128;
};

results in this panic while generating bitfield accessor methods:

thread 'main' panicked at 'physical field containing bitfields should be sized 8, 4, 2, or 1 bytes'

But we should just skip the bitfield accessor method for such large bitfields instead of panicking. See Bitfield::codegen_fields inside src/codegen/mod.rs.
Happy to mentor someone on fixing this :)

Hi! If you have any questions regarding this issue, feel free to make a comment here, or ask it in the #servo channel in IRC.
If you intend to work on this issue, then add @highfive: assign me to your comment, and I'll assign this to you. :smile:

@barrotsteindev, perhaps this is something you'd be interested in hacking on?

@highfive assign me

Hey @chansuke! Thanks for your interest in working on this issue. It's now assigned to you!

@chansuke thanks for picking up this issue :) How is it going? Any questions I can answer for you?

@fitzgen Oh, thank you:)I started reading the code and will start changing this part.
if field_size_in_bits != 0 {
    // Flush the last physical field and its bitfields.
    fields.push(flush_bitfields(ctx,
                                field_size_in_bits,
                                last_field_align,
                                &last_field_name,
                                bitfields.drain(..),
                                methods));
}

I will stop calling flush_bitfields for large bitfields.Is this the right approach?

We still want to generate the correct physical field, so the struct containing it still has the correct layout (or at least has correct size, since we can't generate 128 bit alignment in Rust yet).
Here is what I would do. I would replace this panic:
            _ => panic!("physical field containing bitfields should be sized \
                         8, 4, 2, or 1 bytes")
```]

with something like this:

```rust
_ => return field,

so we just generate the physical field of the appropriate size and do not generate any bitfield getters or setters.
I would also move the match that this is a part of outside the loop since it doesn't need to be recalculated on every iteration.
Does that make sense?

@chansuke Have you made any progress?

Unassigning due to radio silence.
@chansuke, if you're still planning on working on this issue, leave a comment and we can re-assign it to you :)

@fitzgen Can I have a go at this?

@Tiwalun sure thing! Take a look at the backlog discussion, and if you have any questions, just ping me here or on IRC :)


Added propagator.jl and documentation.

Added Euler Iterator types, documentation using Docile.
WIP : Need to add propagate iterator types for CN method.

WIP : The links to the source are hard-linked. The docs need to be generated so that the links are not hard-linked.

@acroy a review would be helpful.

@acroy  is this ready to go or do we integrate ODE.jl and close this ? I guess a separate PR for ODE.jl would be nice.

You should remove the "old" functions solver_euler, solver_CN and initialize. ODE integration I would do separately.

@acroy done!

@acroy a review would be helpful.

@acroy I guess everything is set hopefully.

Apart from my minor nitpick this looks good. I would squash the commits and then merge.


feat: node interfaces + migration

PR inludes now:
Node objects:

Added field interfaces
Removed field uptime


Return 404 for empty list

or make startCursor/endCursor optional
https://github.com/interledger/open-payments/blob/f1b51c3c051c9030cea1f3d2542cfb492c4330d3/openapi/resource-server.yaml#L1242-L1270
Open Payments list endpoints currently return
local-peer-backend-1     | {
local-peer-backend-1     |   status: 200,
local-peer-backend-1     |   message: 'OK',
local-peer-backend-1     |   header: [Object: null prototype] {
local-peer-backend-1     |     'content-type': 'application/json; charset=utf-8'
local-peer-backend-1     |   },
local-peer-backend-1     |   body: {
local-peer-backend-1     |     pagination: { hasPreviousPage: false, hasNextPage: false },
local-peer-backend-1     |     result: []
local-peer-backend-1     |   }
local-peer-backend-1     | }

which causes
local-peer-backend-1     |   InternalServerError: response.pagination must have required property 'startCursor'
local-peer-backend-1     |       at Object.throw (/workspace/node_modules/.pnpm/koa@2.14.1/node_modules/koa/lib/context.js:97:11)
local-peer-backend-1     |       at /workspace/packages/openapi/dist/middleware.js:32:26
local-peer-backend-1     |       at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
local-peer-backend-1     |       at async /workspace/packages/backend/dist/open_payments/payment_pointer/middleware.js:16:9
local-peer-backend-1     |       at async bodyParser (/workspace/node_modules/.pnpm/koa-bodyparser@4.3.0/node_modules/koa-bodyparser/index.js:95:5)

I think we should make the startCursor/endCursor optional


add_hyperparameter question

Could it be a list of model names for problem.add_hyperparameter ?
problem.add_hyperparameter(('llama3-8', 'llama3-12'), "model_id')

Yes, but it should be a list with squared brackets [...].

Thanks.


feat: add pdo_sqlsrv 5.12.0, sqlsrv 5.12.0 releases

update pdo_sqlsrv with release 5.12.0
update sqlsrv with release 5.12.0

:tada: This PR is included in version 1.283.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


45 feature wfntp implementation of the activity index

Description
Please provide a detailed description of what was done in this PR.
Precise the issue that you are resolving.
Changes include

[ ] Bugfix (non-breaking change that solves an issue)
[ ] New feature (non-breaking change that adds functionality)
[ ] Breaking change (change that is not backwards-compatible and/or changes current functionality)

Breaking changes
Please complete this section if any breaking changes have been made, otherwise delete it.
Checklist

[ ] I have assigned this PR to myself
[ ] I have added at least 1 reviewer
[ ] I have added the needed labels
[ ] I have linked this PR to an issue
[ ] I have linked this PR to a milestone
[ ] I have linked this PR to a project
[ ] I have tested this code
[ ] I have added / updated tests (unit / functionals / end-to-end / ...)
[ ] I have updated the README and other relevant documents (guides...)
[ ] I have added sufficient documentation both in code, as well as in the READMEs

Additional comments
Please post additional comments in this section if you have them, otherwise delete it.

C'est plus ou moins le restatnt des commits de la branche apres avoire retirer la doc de roberto, les commits en lien avec la rapport et le commit en lien avec l'audit des images.

Il devrait y avoir principalement que le code concernant l'implémentation de la feature activity index, @COTHSC tu pourras confirmer ?


undefined is not an object (evaluating '_this.refs.pressRipple.run')

Hello. Thanks for your package.
I receive ""undefined is not an object (evaluating '_this.refs.pressRipple.run')"" error when tab pressing takes me to another page of the app. To note, this page renders properly when I go to it from side menu, so the component is not the problem here.
Please help to solve the problem

Is the BottomNavigation visible on this page/screen? I guess it could happen that, if you navigate to a screen which isn't using the BottomNavigation, there could be a potential of a crash.

no, the page to be navigated to does not have that TabBar. I solved the problem just with the help of setTimeout... Namely postponing the navigation. But don't sure this is the right way of operation. How the navigation from the same TabBar to different pages (both not using the TabBar) can be different: one succeed, and the another one-not

The issue is that the BottomNavigation has this circular animation while the screen changes. If you press on the Tab, it switches to the screen and tries to run the animation – but since the BottomNavigation is gone on this screen, it throws an error.
If you navigate without using the BottomNavigation, it doesn't trigger the animation, hence there's no error.
That's definitely a bug and something I didn't think of – that the BottomNavigation could be gone on a screen. I try to publish a hotfix in the next days.

ok thanks

I fixed this issue in v0.5.2 and published it just now 🎉
Please reopen, if this issue still exists. (But I don't think so, I tested it and the error is gone). Sorry for taking so long for this small fix.


video flip

The video recorded with a front-facing camera is flipped horizontally

This is a demo project which is showing how you can implement the CameraX, so you have to handle this kind of issues in your own project. :)


The extracted feature of the COCO dataset for caption

In CLIP-ViL/CLIP-ViL-Direct/caption/clip.py,         "x = x + self.positional_embedding[0, :, None, :].to(x.dtype)  # (HW+1)NC"
should be  " x = x + self.positional_embedding[:, None, :].to(x.dtype)  # (HW+1)NC"
Could you explain why you change the following code of the official clip?
In official clip,  out_proj_weight=self.c_proj.weight, out_proj_bias=self.c_proj.bias.
In your code,

@liujiaheng I have the same problem when I modified the original code to "x = x + self.positional_embedding[:, None, :]", the output of the multihead self-attention turned to be same values. have experienced the "same values" problem？

Hi @liujiaheng, thanks for pointing this out. That's some inconsistent coding in our preliminary exploration. We just fix that to be consistent with the official repo.

Hi,
Thanks for your modify. But when I apply for new modified codes, the model.encode(image) just has one output that shape is (1,2048), I suppose that this feature just tmp_fc. Could you check it, please? And there is another problem for me , when I run the eval.py, the shape of model is (512, 2560), but the feature should be (1, 2048), I think it is strange. Could you help me, please?
Many thanks and best wishes,
Peng
At 2022-01-13 14:32:26, "Sheng Shen" @.***> wrote:
Hi @liujiaheng, thanks for pointing this out. That's some inconsistent coding in our preliminary exploration. We just fix that to be consistent with the official repo.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you commented.Message ID: @.***>

Hi @Timon0327, that is due to CLIP official repo's encode(image) will only give you one vector for the whole image, u can refer to the code here about how to get the tmp_att https://github.com/clip-vil/CLIP-ViL/blob/cb8b011b9ef26a2e29a9f6ef35599ea725b6b9f0/CLIP-ViL-Direct/caption/scripts/clip_prepro_feats.py#L118-L132.


Using collapse component inside of a loop

Hello,
I am wondering what the correct way is to map over data and wrap each element inside of a collapse component.
For example, I am iterating over a list of blog posts and when they are clicked on I want to display additional data about the post and I figure using collapse is the best way to hide the data until called to be displayed.
But when I try this I click on one of the blog posts and all collapse components reveal themselves. I understand this has to do with the toggle function. I am using the same function to toggle all the components. So my question is there a good way to either bind the toggle event to the individual component or is there a way to uniquely identify each component and bind that identifier to the toggle function?
This is my code so far. Thank you in advance for any advice or direction.
    return Object.keys(cat).map(index=>{
      let date = cat[index].createdAt.toString();
      date = date.substring(0, date.indexOf('T'));
      return(
        <ListGroupItem key={index}>

{cat[index].title} - {date}

          <Collapse isOpen={this.state.collapse}>{cat[index].title}</Collapse>
        </ListGroupItem>
      )
    });

The best way to do something like this IMO would be to create a component which will manage the state for an individual ListGroupItem and it's collapse. Assuming you allow for multiple Collapses to be showing at the same time, you can do something like this:
ListGroupCollapse:
import React from 'react';
import { ListGroupItem, Collapse } from 'reactstrap';

class ListGroupCollapse extends React.Component {
  constructor(props) {
    super(props);

    this.toggle = this.toggle.bind(this);
    this.state = {collapse: false};
  }

  toggle() {
    this.setState({ collapse: !this.state.collapse });
  }

  render() {
    const cat = this.props.cat;
    let date = cat.createdAt.toISOString();
    date = date.substring(0, date.indexOf('T'));
    return (
      <ListGroupItem>

            {cat.title} - {date}

          <Collapse isOpen={this.state.collapse}>{cat.title}</Collapse>

      </ListGroupItem>
    );
  }
}

export default ListGroupCollapse

And use it in your map like so:
        Object.keys(cat).map((key, index) =>
          <ListGroupCollapse key={index} cat={cat[key]} />
        )

Demo: https://www.webpackbin.com/bins/-Ksfqpl_i2sF2Lmjcv_Z
If you need it where opening one closes the others, you can do something similar with a callback (see this demo: https://www.webpackbin.com/bins/-Ksftn1S-gZ1WdzKkVCA)
Basically, you want to avoid creating/bind function in the render. Also creating the small ListGroupCollapse component which is only concerned about a single Collapse is easier to understand, plus it doesn't re-render when the props do no change.

Wow! Thank you this worked liked a charm.

I have the same problem, could you pls advice?
import React from 'react'
import './index.css'
import {Collapse} from 'reactstrap'
class SubmenuGroup extends React.Component {
getDefaultCSSClasses = () => {
const classes = [this.props.classes || 'submenu_group']
return classes

}
renderSubmenu = (submenuitems) => {
return submenuitems.map ((submenuitem, i) => {
return (

{submenuitem}

)
})
}
render () {
const {submenuitems, isSubmenuOpen} = this.props
return (

      {this.renderSubmenu(submenuitems)}

)

}
}
export default class MenuGroup extends React.Component {
constructor (props) {
super (props)
this.state = {
isSubmenuOpen: false
}
}
getDefaultCSSClasses = () => {
const classes = [this.props.classes || 'menu_group']
return classes

}
getDefaultCSSLogoClasses = () => {
const { isSubmenuOpen } = this.state
var classes = ''
if (isSubmenuOpen) {
  classes = 'icon-chevron-down'
} else {
  classes = 'icon-chevron-right'
}

return classes

}
handleLogoClick = (menu_row, i) => {
console.log(menu_row, i)
this.setState ({
  isSubmenuOpen: !this.state.isSubmenuOpen
})

}
renderMenu = (menu_data) => {
return menu_data.map ((menu_row, i) => {
return (

{menu_row.menuitem_name}

 this.handleLogoClick(menu_row, i)}
/>

)
})
}
render () {
const {menu_data} = this.props
return (

    {this.renderMenu(menu_data)}

)

}
}

import React from 'react'
import './index.css'

import {Collapse} from 'reactstrap'

class SubmenuGroup extends React.Component {
  getDefaultCSSClasses = () => {
    const classes = [this.props.classes || 'submenu_group']

    return classes
  }

  renderSubmenu = (submenuitems) => {
    return submenuitems.map ((submenuitem, i) => {
      return (

          {submenuitem}

      )
    })
  }

  render () {
    const {submenuitems, isSubmenuOpen} = this.props

    return (

          {this.renderSubmenu(submenuitems)}

    )
  }
}

export default class MenuGroup extends React.Component {
  constructor (props) {
    super (props)
    this.state = {
      isSubmenuOpen: false
    }
  }

  getDefaultCSSClasses = () => {
    const classes = [this.props.classes || 'menu_group']

    return classes
  }

  getDefaultCSSLogoClasses = () => {
    const { isSubmenuOpen } = this.state
    var classes = ''

    if (isSubmenuOpen) {
      classes = 'icon-chevron-down'
    } else {
      classes = 'icon-chevron-right'
    }

    return classes
  }

  handleLogoClick = (menu_row, i) => {
    console.log(menu_row, i)

    this.setState ({
      isSubmenuOpen: !this.state.isSubmenuOpen
    })
  }

  renderMenu = (menu_data) => {
    return menu_data.map ((menu_row, i) => {
      return (

{menu_row.menuitem_name}

 this.handleLogoClick(menu_row, i)}
            />

          <Collapse isOpen={this.state.isSubmenuOpen}>
            <SubmenuGroup
              submenuitems={menu_row.submenuitems}
            />
          </Collapse>

      )
    })
  }

  render () {
    const {menu_data} = this.props

    return (

        {this.renderMenu(menu_data)}

    )
  }
}

I have the same problem, when I click the logo, all the submenus collapse. How to make each one collapse?
import React from 'react'
import './index.css'

import {Collapse} from 'reactstrap'

class SubmenuGroup extends React.Component {
  getDefaultCSSClasses = () => {
    const classes = [this.props.classes || 'submenu_group']

    return classes
  }

  renderSubmenu = (submenuitems) => {
    return submenuitems.map ((submenuitem, i) => {
      return (

          {submenuitem}

      )
    })
  }

  render () {
    const {submenuitems, isSubmenuOpen} = this.props

    return (

          {this.renderSubmenu(submenuitems)}

    )
  }
}

export default class MenuGroup extends React.Component {
  constructor (props) {
    super (props)
    this.state = {
      isSubmenuOpen: false
    }
  }

  getDefaultCSSClasses = () => {
    const classes = [this.props.classes || 'menu_group']

    return classes
  }

  getDefaultCSSLogoClasses = () => {
    const { isSubmenuOpen } = this.state
    var classes = ''

    if (isSubmenuOpen) {
      classes = 'icon-chevron-down'
    } else {
      classes = 'icon-chevron-right'
    }

    return classes
  }

  handleLogoClick = (menu_row, i) => {
    console.log(menu_row, i)

    this.setState ({
      isSubmenuOpen: !this.state.isSubmenuOpen
    })
  }

  renderMenu = (menu_data) => {
    return menu_data.map ((menu_row, i) => {
      return (

{menu_row.menuitem_name}

 this.handleLogoClick(menu_row, i)}
            />

          <Collapse isOpen={this.state.isSubmenuOpen}>
            <SubmenuGroup
              submenuitems={menu_row.submenuitems}
            />
          </Collapse>

      )
    })
  }

  render () {
    const {menu_data} = this.props

    return (

        {this.renderMenu(menu_data)}

    )
  }
}

Could you pls advice on the above?

Data comes via props:
    const menu_data = [
      {id: 0, menuitem_name: 'Home', submenuitems: ['Home1']},
      {id: 1, menuitem_name: 'Software', submenuitems: ['Software1', 'Software2']}
    ]

@TheSharpieOne Thank you for code. The link that you have provided for DEMO no longer exists can you upload it again somewhere CodePen or other.

I updated the links in the post above, also here they are:
https://stackblitz.com/edit/reactstrap-v5-4rezrr?file=Example.js
https://stackblitz.com/edit/reactstrap-v5-8f5phu?file=Example.js

thanks a lot.

How to implement it to li tag?

Hi all 👋 I was facing the same problem a few hours ago. I would like to share the solution.
Code that shows all the elements 👎
import React, { useState  } from 'react';
import { Collapse } from 'x.x.x';
const [isCollapse, setCollapse] = useState(false)

const toggle = () => setCollapse(!isCollapse);

const users = [ { name: 'Leo' }, { name: 'Mario'}, { name: 'Luc' }  ]

const listFriend = () => {
  return (

        {users.map(item => {
          return (
            <button onClick={toggle}>Show more<button>
            <Collapse isOpen={isCollapse}>

Hi !

           </Collapse>
          )
       })}

   )
}

Why does this happen 🤔
The answer is quite simple, we are not filtering item to be displayed. In the method toggle we are not filtering absolutely nothing. If we want to show an item individually it will be necessary to filter it. With the following code the problem should be solved
Code that will show an item individually 🎉

const users = [ { id: 1, name: 'Leo' }, { id: 2, name: 'Mario'}, { id: 3, name: 'Luc' }  ];

const toggle = index => {
    if (index) {
       // to show the rest of the elements
       setCollapse(!isCollapse);
    }
    if (index === 0) {
        //  to show first item
        setCollapse(!isCollapse);
      }
}

        {users.map(item => {
          return (
            <button onClick={toggle(item.id)}>Show more<button>
            <Collapse isOpen={isCollapse}>

Hi !

           </Collapse>
          )
       })}

NOTE: This solution can be applied in any library that has the Collapse component : )

Hi all wave I was facing the same problem a few hours ago. I would like to share the solution.
Code that shows all the elements -1
import React, { useState  } from 'react';
import { Collapse } from 'x.x.x';
const [isCollapse, setCollapse] = useState(false)

const toggle = () => setCollapse(!isCollapse);

const users = [ { name: 'Leo' }, { name: 'Mario'}, { name: 'Luc' }  ]

const listFriend = () => {
  return (

        {users.map(item => {
          return (
            <button onClick={toggle}>Show more<button>
            <Collapse isOpen={isCollapse}>

Hi !

           </Collapse>
          )
       })}

   )
}

Why does this happen thinking
The answer is quite simple, we are not filtering item to be displayed. In the method toggle we are not filtering absolutely nothing. If we want to show an item individually it will be necessary to filter it. With the following code the problem should be solved
Code that will show an item individually tada
const users = [ { id: 1, name: 'Leo' }, { id: 2, name: 'Mario'}, { id: 3, name: 'Luc' }  ];

const toggle = index => {
    if (index) {
       // to show the rest of the elements
       setCollapse(!isCollapse);
    }
    if (index === 0) {
        //  to show first item
        setCollapse(!isCollapse);
      }
}

        {users.map(item => {
          return (
            <button onClick={toggle(item.id)}>Show more<button>
            <Collapse isOpen={isCollapse}>

Hi !

           </Collapse>
          )
       })}

NOTE: This solution can be applied in any library that has the Collapse component : )

it works without a library?

Is there a way to use both collapse and uncontrolled collapse?  I'm mapping over some data and using collapse and it works fine until you get down to mobile size. Then I have that whole element disappear for the mobile element. The mobile is using uncontrolled collapse because in the mobile state I need multiple collapses.
I have multiple collapses to use until the browser hits a certain breakpoint. So I'm using uncontrolled collapse for that so I can use Ids. Well from commenting various things out, the collapse even though it's hidden and the elements that use it are hidden, it's messing with my styling. The uncontrolled collapses still work but none of my styling works at all. It's even affecting styling on other pages which I'm not even sure how that's possible. Is there some work around I've been at it for hours and I'm out of ideas....


Update kiali and add workloads config

Related to https://github.com/kiali/kiali/issues/1903

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

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

just FYI: we already moved up to Kiali v1.9 in both helm and istioctl installer for the Istio 1.4 release. And note this PR is only for the (soon to be deprecated) helm charts. Similar changes would need to be made in the istio/installer kiali charts.

@yacut thank you for your PR. We are in the process of standardizing install on the Kiali operator rather than our own home-grown installation technique. Also this directory is deprecated and replaced by https://github.com/istio/installer. Please see PR: #https://github.com/istio/operator/pull/622 for more details.
Cheers
-steve

Closing per above comment


fixed drop specialization assert for unreachable switch branches on u…

…nions
cause was
result2 =
    (popCtx1, n1) = Context.popStack? ctx
    (popCtx2, n2) = Context.popStack? popCtx1
    (popCtx3, n3) = Context.popStack? popCtx2
    Ok (Context.pushStack (Context.pushStack (Context.pushStack popCtx3 n2) n1) n3)

when result2 is
    Ok a ->
        Ok a

    # Being explicit with error type is required to stop the need to propogate the error parameters to Context.popStack
    Err EmptyStack ->
        Err EmptyStack

where drop specialization tries to optimize the drop of result2 in the Err EmptyStack branch. But it already knows result2 is Ok since we defined it above. This causes the assert to fail since the expected tag of the union fields does not match that of the branch (which is unreachable afaik).
Code is updated to only consider children with a matching tag. Effectively ignoring any known children.

I think something is up with the macos builder. This keeps failing in different spot in the macos builder. This time it failed building the zig builtins (which definitely is not caused by the changes in this PR). I highly doubt any of the other failures were caused by this PR either.
CC: @Anton-4 any ideas?

Going to take a look now


(BKR-745) added el support for s390x dev installs

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/qe_beaker_btc-intn/2740/

believe the failure is due to gem issues on the Jenkins. Should be re-kicked when those are resolved to confirm.

@puppetlabs-jenkins retest this please

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/qe_beaker_btc-intn/2752/

job was aborted due to maintenance window. Will re-kick after lunch

@puppetlabs-jenkins retest this please

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/qe_beaker_btc-intn/2757/

@tvpartytonight it's green!

:+1:  confirmed that this works to deploy puppet-agent to redhat-6 on s390x.
@kevpl you have made me VERY HAPPY!!!


Add new cache backends to Readme list

Closes #43

LGTM.
Some minor nitpicks is some inconsistency about having a period at the end. Also, the line above the newly added ones is a broken link. But we can fix that in followup change.


[DownloadBuildArtifactsV0] Normalize download path

Task name: DownloadBuildArtifactsV0
Description: Added new variable with normalized unzipLocation in the unzipContainer method.
Documentation changes required: N
Added unit tests: N
Attached related issue: https://github.com/microsoft/build-task-team/issues/1143
Checklist:

[X] Task version was bumped - please check instruction how to do it
[X] Checked that applied changes work as expected

Tested these changes manually, found no issues


Getting videos of channel not working [part.2]

HI,
I'm using the latest version 2.5.0. It worked a few days ago, but now it doesn't work anymore, it has the same error it had last year:
for video in videos:

File "/usr/local/lib/python3.10/dist-packages/scrapetube/scrapetube.py", line 75, in get_channel
for video in videos:
File "/usr/local/lib/python3.10/dist-packages/scrapetube/scrapetube.py", line 199, in get_videos
client = json.loads(
File "/usr/lib/python3.10/json/init.py", line 346, in loads
return _default_decoder.decode(s)
File "/usr/lib/python3.10/json/decoder.py", line 337, in decode
obj, end = self.raw_decode(s, idx=_w(s, 0).end())
File "/usr/lib/python3.10/json/decoder.py", line 355, in raw_decode
raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

Issue has already been raised and being investigated -  https://github.com/dermasmid/scrapetube/issues/36

fixed in latest release

Sorry to necro but I'm getting this issue too when last night I wasn't, has there been a chance to something on YouTube's side?


cmd/compile:  internal compiler error: 'CloneString': value .autotmp_9 (v95) incorrectly live at entry

What version of Go are you using (go version)?

$ go version
tip

Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env

What did you do?
https://go-review.googlesource.com/c/go/+/528799
What did you expect to see?
test passed.
What did you see instead?
https://storage.googleapis.com/go-build-log/95688dfe/windows-amd64-2016_e582fd3f.log
internal compiler error: 'CloneString': value .autotmp_9 (v95) incorrectly live at entry

Just a guess, but can you try to use b := make([]byte, len(s)) instead of MakeNoZero to see if the failure goes away?

Just a guess, but can you try to use b := make([]byte, len(s)) instead of MakeNoZero to see if the failure goes away?

I will try again later. I am currently trying git bisect and I have noticed that race is required for ICE to appear.

Just a guess, but can you try to use b := make([]byte, len(s)) instead of MakeNoZero to see if the failure goes away?

After trying your guess, still ICE.
internal\bytealg\bytealg.go:177:6: internal compiler error: 'CloneString': value .autotmp_11 (v97) incorrectly live at entry

Due to the presence of internal/cpu and<ABIInternal>, I use the following code for git bisect.
cat bytealg.go
// Copyright 2018 The Go Authors. All rights reserved.
// Use of this source code is governed by a BSD-style
// license that can be found in the LICENSE file.

package bytealg

import (
        "unsafe"
)

// MakeNoZero makes a slice of length and capacity n without zeroing the bytes.
// It is the caller's responsibility to ensure uninitialized bytes
// do not leak to the end user.
func MakeNoZero(n int) []byte {
        return nil
}

// Clone returns a copy of b[:len(b)].
// The result may have additional unused capacity.
// Clone(nil) returns nil.
func Clone(s []byte) []byte {
        if s == nil {
                return nil
        }
        return append([]byte{}, s...)
}

// CloneString returns a fresh copy of s.
// It guarantees to make a copy of s into a new allocation,
// which can be important when retaining only a small substring
// of a much larger string. Using Clone can help such programs
// use less memory. Of course, since using Clone makes a copy,
// overuse of Clone can make programs use more memory.
// Clone should typically be used only rarely, and only when
// profiling indicates that it is needed.
// For strings of length zero the string "" will be returned
// and no allocation is made.
func CloneString(s string) string {
        if len(s) == 0 {
                return ""
        }
        b := MakeNoZero(len(s))
        copy(b, s)
        return unsafe.String(&b[0], len(b))
}

Failed to re ICE.

I believe there is something racy with copy, if you replace the copy with an append it works even with -race:
diff --git a/src/internal/bytealg/bytealg.go b/src/internal/bytealg/bytealg.go
index 7d70b0b562..99faab7e9a 100644
--- a/src/internal/bytealg/bytealg.go
+++ b/src/internal/bytealg/bytealg.go
@@ -178,7 +178,6 @@ func CloneString(s string) string {
 	if len(s) == 0 {
 		return ""
 	}
-	b := MakeNoZero(len(s))
-	copy(b, s)
+	b := append([]byte{}, s...)
 	return unsafe.String(&b[0], len(b))
 }

I was unable to put together a reproducible case outside of the one from the CL

cc @golang/compiler

Strictly speaking, a bug in your CL (even if it is caused by existing weirdness in the compiler/library you are patching) is not a bug in the current compiler.   I'd like to close this to avoid cluttering up the dashboard.

Strictly speaking, a bug in your CL (even if it is caused by existing weirdness in the compiler/library you are patching) is not a bug in the current compiler. I'd like to close this to avoid cluttering up the dashboard.

This ICE caused CL to be blocked. I have encountered this situation for the first time. How can I solve it?

@qiulaidongfeng best bet would be to try and isolate the ICE to get a reproducible example that does not depend on your CL so it can be addressed as a compiler bug.

Note that internal/bytealg is treated as a runtime package by the compiler, so you may need to do special handling if you do un-usual thing.


[documentation] Clarify the role of override.ssh.private_key_path vs provider.ssh_key_name

Hi,
I am using this plugin, to provision a machine on DO.
My public ssh key was already part of my DO account.
if I do not set the provider.ssh_key_name in the Vagrantfile I get the following error:
$ vagrant up --provider=digital_ocean
Bringing machine 'default' up with 'digital_ocean' provider...
==> default: Creating new SSH key: Vagrant...
There was an issue with the request made to the DigitalOcean
API at:

Path: /v2/account/keys
URI Params: {:name=>"Vagrant", :public_key=>"[snip]"}

The response status from the API was:

Status: 422
Response: {"id"=>"unprocessable_entity", "message"=>"SSH Key is already in use on your account"}

If I set the provider.ssh_key_name to the key names then everything works. The key name can be obtained using the command  tugboatkeys.
Please note that I have chosen as  provider.ssh_key_namethe name of the same key that I have indicated in  override.ssh.private_key_path (i.e. '~/.ssh/id_rsa' is the private key corresponding to the $DIGITALOCEAN_SSH_KEY_NAME public key in my DO accout).
Here is the example Vagrantfile with some slight modification to fix this problem, I am using environment variables to store my private data:
Vagrant.configure('2') do |config|
  config.vm.hostname = 'dropletname.example.com'
  # Alternatively, use provider.name below to set the Droplet name. config.vm.hostname takes precedence.
  config.vm.provider :digital_ocean do |provider, override|

    # This is akways needed but if the key is already used in the DO account
    # you *need* to set the provider.ssh_key_name parameter.
    override.ssh.private_key_path = '~/.ssh/id_rsa'

    override.vm.box = 'digital_ocean'
    override.vm.box_url = "https://github.com/smdahlen/vagrant-digitalocean/raw/master/box/digital_ocean.box"

    provider.token = ENV['DIGITALOCEAN_TOKEN']
    provider.ssh_key_name = ENV['DIGITALOCEAN_SSH_KEY_NAME']
    provider.image = 'ubuntu-14-04-x64'
    provider.region = 'nyc2'
    provider.size = '512mb'
  end
end

I think that the documentation is unclear as of now, and this error could cause confusion.
Thank you for this great plugin.

Thank you for your feedback - by default everything works as expected. We feel this would cause more confusion as we currently note that by default, the key name is Vagrant.
provider.ssh_key_name
A string representing the name to use when creating a DigitalOcean SSH key for droplet authentication. It defaults to Vagrant.


Missing privacy manifest

App store requires privacy manifest file, this file exists in original Lottie framework, but missing in LottieProxy

ITMS-91061: Missing privacy manifest - Your app includes “Frameworks/LottieProxy.framework/LottieProxy”, which includes Lottie, an SDK that was identified in the documentation as a privacy-impacting third-party SDK. Starting February 12, 2025, if a new app includes a privacy-impacting SDK, or an app update adds a new privacy-impacting SDK, the SDK must include a privacy manifest file. Please contact the provider of the SDK that includes this file to get an updated SDK version with a privacy manifest. For more details about this policy, including a list of SDKs that are required to include signatures and manifests, visit: https://developer.apple.com/support/third-party-SDK-requirements..

Thank you for letting me know

fixed in 1.0.3


[Feature Request] Written reviews

I have been looking for something like this project for a long time as I'm a letterboxd fan simply as a place to keep my movie reviews. Is there any plan in the future to incorporate written user reviews (and maybe a star or numbered rating system besides just good or bad?)

I'm thinking for some time about precise rating instead of good/bad. I don't know if current flox users are for or against this idea. Plus, I should migrate existing rating to this new behavior.
With my fork, you're already able to write a user review by item.
Thanks a lot for showing your interest in this project. I'm really looking for feedback.

I didn't know about the written reviews. I haven't actually installed it yet but will do so now that I know that is a feature. I would like to contribute to this project as well, maybe a Dockerfile if there is a demand for that.

Any help is welcome and a Dockerfile could be interesting for a lot of people (first one spotted here).
You can check at my ideas for the future in this list.


Fix genesis sampling for non-debug_assertions scenarios

In addition, the existing 2 checks for genesis ratifications are merged into 1 and the related error message is fixed.
If there's a better ratification and finalization to use, I'm happy to adjust the PR.

Drafting, as it's tricky to set it up correctly across all the crates.


Tracking PyTorch 1.8.0 additions

New functions (most of them to improve numpy compatibility):
torch.nan_to_num (#44592)
torch.tensor_split (#45168)
torch.nanmedian (#45847)
torch.ravel (#46098)
torch.igamma (#46183)
torch.igammac (#48171)
torch.column_stack (#46313)
torch.row_stack} (#46313)
torch.kron (#45358)
torch.copysign (#46396)
Tensor.new_empty_strided (#47225)
torch.{swapdims,swapaxes} (#46041)
torch.tile (#47974)
torch.float_power (#44937)
torch.moveaxis (#48581)
torch.inner (#46716)
torch.msort (#48440)
torch.sinc (#48740)
torch.broadcast_to (#48997)
torch.xlogy (#48777)
torch.f{max,min} (#49312)
torch.diff (#50569)
torch.ldexp (#45370)
torch.broadcast_shapes (#43935)
linalg.tensorsolve (#46142)
linalg.cholesky (#46083)
linalg.tensorinv (#45969)
linalg.{eigh,eigvalsh} (#45526)
linalg.matrix_rank (#48206)
linalg.solve (#48456)
linalg.qr (#47764, #50046)
linalg.svd (#45562)
linalg.inv (#48261)
linalg.pinv (#48399)
linalg.slogdet (#49194)
linalg.cond (#45832)
New modules:
nn.PixelUnshuffle (#49334)
nn.GaussianNLLLoss (#50886)
Automatic shape inference in torch.nn:
nn.LazyLinear (#44538)
nn.LazyConv{1,2,3}d
nn.LazyConvTranspose{1,2,3}d (#47350)

New functions (most of them to improve numpy compatibility):

torch.nan_to_num (#44592)
torch.tensor_split (#45168)
torch.nanmedian (#45847)
torch.ravel (#46098)
torch.igamma (#46183)
torch.igammac (#48171)
torch.column_stack (#46313)
torch.row_stack} (#46313)
torch.kron (#45358)
torch.copysign (#46396)
Tensor.new_empty_strided (#47225)
torch.{swapdims,swapaxes} (#46041)
torch.tile (#47974)
torch.float_power (#44937)
torch.moveaxis (#48581)
 torch.inner (#46716)
torch.msort (#48440)
torch.sinc (#48740)
torch.broadcast_to (#48997)
torch.xlogy (#48777)
torch.f{max,min} (#49312)
torch.diff (#50569)
torch.ldexp (#45370)
torch.broadcast_shapes (#43935)

linalg.tensorsolve (#46142)
linalg.cholesky (#46083)
linalg.tensorinv (#45969)
linalg.{eigh,eigvalsh} (#45526)
linalg.matrix_rank (#48206)
linalg.solve (#48456)
linalg.qr (#47764, #50046)
linalg.svd (#45562)
linalg.inv (#48261)
linalg.pinv (#48399)
linalg.slogdet (#49194)
linalg.cond (#45832)

New modules:
nn.PixelUnshuffle (#49334)
nn.GaussianNLLLoss (#50886)

Automatic shape inference in torch.nn:
nn.LazyLinear (#44538)
nn.LazyConv{1,2,3}d
nn.LazyConvTranspose{1,2,3}d (#47350)

Implemented in next PR:
torch.nan_to_num (#44592)
torch.nanmedian (#45847)
torch.igamma (#46183)
torch.igammac (#48171)
torch.ravel (#46098)
torch.kron (#45358)
torch.column_stack (#46313)
torch.row_stack} (#46313)
torch.copysign (#46396)
torch.{swapdims,swapaxes} (#46041)

Really nice work @NiklasGustafsson . Do you also plan to port JIT in the future ?

The JIT is, for sure, an important feature to add. It will require a lot more thought than the recent porting effort. Tracing should be relatively straight-forward, but script-based JIT will be more complicated.

Thank you for this push on getting these functions. This will go along way in helping C#  getting adapted for scientific applications.


Add a test for readable and writable properties

Verify that the "readable" and "writable" attributes are correctly available and
configured on TextEncoder and TextDecoder objects.

@tyoshino PTAL.

495ace8bd9c742a2143c12807693c6ebdc43cb33 still lgtm


Add compound on Front End needs a link to ChemBL

Since the user must enter the CHEMBLID, it would be convenient to have a link to CHEMBL (opens in a new tab) to look up the ID.  What would be slick is if you could find the compound in CHEMBL and simply go back to the MPS tab and retrieve the info (i.e. the software would find the CHEMBLID in the web page on the other tab).  It would be sufficient though if the user found the CHEMBLID and then copy-paste to the MPS tab.
There should be hints of text formats on the Add Compound fields.  Perhaps Acetaminophen could be the example for the hints.

What hints do you think would be necessary/helpful?

As of https://github.com/UPDDI/mps-database-server/commit/c0b0766434844118496df1f06e6ec77f278889ce the Compound Add page now has a link to ChEMBL that will try to open in a new tab (depending on the user's browser settings, it may appear in a new window).


java.lang.ClassNotFoundException: io.netty.internal.tcnative.SSLContext during start of v2.18

What is the bug?
We have just picked up the v2.18 release and noticed a new warning and exception stack in the logs while OpenSearch is starting:
[2024-11-07T10:53:37,649][WARN ][o.o.s.s.SslSettingsManager] [analytics-storage-0] OpenSSL not available (this is not an error, we simply fallback to built-in JDK SSL) because of {}
java.lang.ClassNotFoundException: io.netty.internal.tcnative.SSLContext
        at java.base/java.net.URLClassLoader.findClass(URLClassLoader.java:445) ~[?:?]
        at java.base/java.lang.ClassLoader.loadClass(ClassLoader.java:593) ~[?:?]
        at java.base/java.net.FactoryURLClassLoader.loadClass(URLClassLoader.java:872) ~[?:?]
        at java.base/java.lang.ClassLoader.loadClass(ClassLoader.java:526) ~[?:?]
        at java.base/java.lang.Class.forName0(Native Method) ~[?:?]
        at java.base/java.lang.Class.forName(Class.java:534) ~[?:?]
        at java.base/java.lang.Class.forName(Class.java:513) ~[?:?]
        at io.netty.handler.ssl.OpenSsl.<clinit>(OpenSsl.java:95) ~[netty-handler-4.1.114.Final.jar:4.1.114.Final]
        at org.opensearch.security.ssl.OpenSearchSecuritySSLPlugin$4.run(OpenSearchSecuritySSLPlugin.java:218) ~[opensearch-security-<IP_ADDRESS>.jar:<IP_ADDRESS>]
        at java.base/java.security.AccessController.doPrivileged(AccessController.java:319) ~[?:?]
        at org.opensearch.security.ssl.OpenSearchSecuritySSLPlugin.<init>(OpenSearchSecuritySSLPlugin.java:213) [opensearch-security-<IP_ADDRESS>.jar:<IP_ADDRESS>]
        at org.opensearch.security.OpenSearchSecurityPlugin.<init>(OpenSearchSecurityPlugin.java:318) [opensearch-security-<IP_ADDRESS>.jar:<IP_ADDRESS>]
        at java.base/jdk.internal.reflect.DirectConstructorHandleAccessor.newInstance(DirectConstructorHandleAccessor.java:62) ~[?:?]
        at java.base/java.lang.reflect.Constructor.newInstanceWithCaller(Constructor.java:502) ~[?:?]
        at java.base/java.lang.reflect.Constructor.newInstance(Constructor.java:486) ~[?:?]
        at org.opensearch.plugins.PluginsService.loadPlugin(PluginsService.java:796) [opensearch-2.18.0.jar:2.18.0]
        at org.opensearch.plugins.PluginsService.loadBundle(PluginsService.java:744) [opensearch-2.18.0.jar:2.18.0]
        at org.opensearch.plugins.PluginsService.loadBundles(PluginsService.java:545) [opensearch-2.18.0.jar:2.18.0]
        at org.opensearch.plugins.PluginsService.<init>(PluginsService.java:197) [opensearch-2.18.0.jar:2.18.0]
        at org.opensearch.node.Node.<init>(Node.java:523) [opensearch-2.18.0.jar:2.18.0]
        at org.opensearch.node.Node.<init>(Node.java:450) [opensearch-2.18.0.jar:2.18.0]
        at org.opensearch.bootstrap.Bootstrap$5.<init>(Bootstrap.java:242) [opensearch-2.18.0.jar:2.18.0]
        at org.opensearch.bootstrap.Bootstrap.setup(Bootstrap.java:242) [opensearch-2.18.0.jar:2.18.0]
        at org.opensearch.bootstrap.Bootstrap.init(Bootstrap.java:404) [opensearch-2.18.0.jar:2.18.0]
        at org.opensearch.bootstrap.OpenSearch.init(OpenSearch.java:181) [opensearch-2.18.0.jar:2.18.0]
        at org.opensearch.bootstrap.OpenSearch.execute(OpenSearch.java:172) [opensearch-2.18.0.jar:2.18.0]
        at org.opensearch.cli.EnvironmentAwareCommand.execute(EnvironmentAwareCommand.java:104) [opensearch-2.18.0.jar:2.18.0]
        at org.opensearch.cli.Command.mainWithoutErrorHandling(Command.java:138) [opensearch-cli-2.18.0.jar:2.18.0]
        at org.opensearch.cli.Command.main(Command.java:101) [opensearch-cli-2.18.0.jar:2.18.0]
        at org.opensearch.bootstrap.OpenSearch.main(OpenSearch.java:138) [opensearch-2.18.0.jar:2.18.0]
        at org.opensearch.bootstrap.OpenSearch.main(OpenSearch.java:104) [opensearch-2.18.0.jar:2.18.0]

It looks like the version of netty was bumped in v2.18 and it's netty that is failing to find an available OpenSSL when initialising. For what it's worth, our environment does have OpenSSL v3.0.7 present.
How can one reproduce the bug?
Steps to reproduce the behavior:

Download version 2.18 that includes the security plugin and start it on a Linux platform with OpenSSL v3.0.7 present.
Monitor the log output and observe the message above.

What is the expected behavior?
I expect the native OpenSSL library to be found correctly and no warning or exception to appear during startup. This works correctly using v2.17.1 of OpenSearch.
What is your host/environment?

OS: Red Hat Enterprise Linux (Universal Base Image)
Version: 9.4 (Plow)

Do you have any additional context?
Using the bundled JDK.

[Triage] Thank you for filing this issue @mark-VIII. The exception is a warning message so the cluster starts up as expected, but it does pollute the logs. Marking this issue as triaged.


Fix bug where requestRegistrationCode not being called

Fixes #132

Changes proposed in this Pull Request:

Fixes issue in create-account workflow where email validation code wasn't being sent while progressing through the workflow.

1. First console when clicks on `Resend Verification Email`
2. Second console when clicks on `Next`.
As per issue 132 description changes working fine 👍 .


(more) Syn 2.0 updates

Description
NOTE: This PR is based on #3727, so only last 3 commits are actually belong to this PR
This PR refactors iroha_primitives_derive crate and VariantCount macro in iroha_derive. The gist of the changes: use manyhow to report errors and darling to parse attributes. Try to improve UX where possible.
I've kept API the same for all the macros, but it may be useful to change some of them:

confusingly, attributes for VariantCount are placed on fields instead of enum variants. I suggest we move them to variants, as it's really a property of the variant
the naming of #[skip_try_from] is confusing in the context of #[skip_from]: the different in names is in the try, but they correspond to different directions of the generated conversions: from is for Variant ->  Enum and try_from is for Enum -> Variant. I suggest changing the name to #[skip_into] or #[skip_try_into] to properly represent the change of conversion direction, even if the actual generated trait is From
Maybe change #[skip_container] to #[skip_from_container]?

I would like some input on this from other team members though ^^
Checklist

[ ] #3727 merged
[ ] All macros documented properly
[ ] API changes figured out

The changes are now part of #3727


The type 'Newtonsoft.Json.JsonConvert' exists in both 'Glimpse.Core.dll' and 'Newtonsoft.Json.dll'

Issue with Ilmerged Newtonsoft libraries.
Can you use /Internalize switch when combining the libraries so they're not visible outside Glimpse?
I guess this would fix issues  like :The type 'Newtonsoft.Json.JsonConvert' exists in both 'Glimpse.Core.dll' and 'Newtonsoft.Json.dll'

Any news on this one ??

This is not acceptable.

Only thing I can put forward atm is that you could do a manual build of Glimpse from source and not do the ILmerge step?

i had to uninstall glimpse because of this. why do this? isn't this what nuget is for?

Welcome to the club @Spongman
I really like Glimpse and even did a presentation about it at a company I was working before.
In the end, we removed it from all of our projects because of this issue.


Excluding images with fragments

Hello Jonas, thanks for this nice plugin!
Material Mkdocs use fragments on URL to select images using a theme to choose image, like:
![figure](assets/figure_black.png#only-light){: .center .shadow }
![figure](assets/figure_white.png#only-dark){: .center .shadow }

When a use these example:
INFO    -  found 2 unused assets
INFO    -  deleted 2 unused assets from final output

In debug mode mkdcos build -v:
INFO    -  found 2 unused assets
DEBUG   -  {'/tmp/mkdocs_d3510031/assets/figure_black.png', '/tmp/mkdocs_d3510031/assets/assets/figure_white.png'}

reference: https://squidfunk.github.io/mkdocs-material/reference/images/?h=image#light-and-dark-mode

Hey @dunossauro thanks for reporting!
I will release a new version today or tomorrow fixing this behavior and allowing to trim specific suffixes when looking for unused files.
"#only-light" and "#only-dark" will be added to the default-trimmed suffixes. It's probably too dangerous to trim all content after # for all files, since "#" is a valid filename-character. So I think a trim-list is a good alternative.

Closed in v1.3.0
Thanks for reporting :)


Simplify examples

Mostly this allows us to drop some "expensive" (as in, slow to build) dependencies.
Along with https://github.com/cloudflare/quiche/pull/352 this cuts the time required to run cargo test on my laptop in half.
Before:
 % time cargo test
cargo test  400.72s user 29.41s system 584% cpu 1:13.54 total

After:
 % time cargo test
cargo test  177.36s user 18.62s system 516% cpu 37.933 total

Seems like win:win of simplifying our examples back to the purpose of showing quiche API, and improving build times.
People that relied on the examples more complex features can use the new quiche-client and quiche-server but there there might be some pain in the transition period. It would be good if we could signpost this somehow, we can document stuff somewhere but I wonder would it be possible to detect a CLI error and print a message saying "you want to use tools/apps now"?

I think cargo test doesn't build examples/, so changes on examples/ won't improve test time?
Also, simplifying too much will make those examples less helpful for people who want to test. For example we usually ask people to show the output of RUST_LOG=trace ./http3-client ... to get a debugging logs for troubleshooting. If we remove all options and debugging logs from examples and suggest to use quiche-{client,server} for such purposes, examples/* will be forgotten soon because there is no reason to build/use it.
I'm ok with changes on src/cc/reno.rs (I didn't realize it's still there)

I wonder would it be possible to detect a CLI error and print a message saying "you want to use tools/apps now"?

I can add something based on the number of arguments maybe.

I think cargo test doesn't build examples/, so changes on examples/ won't improve test time?

It does. That's why there is a single dev-dependencies list which gets built when cargo test or cargo build --examples is run.

suggest to use quiche-{client,server} for such purposes, examples/* will be forgotten soon because there is no reason to build/use it.

Well, the same applies to the documentation, it's up to us to keep them up to date. But also, examples are built on CI as per the previous point, so that helps.

I can add something based on the number of arguments maybe.

Also, I'm planning to add some examples on how to use the apps to the README itself.


Call correct listener() and add test

As discussed here https://github.com/spotify/flo/pull/149/files#r212948632

Did some simplification, added a test to FloRunnerTest that verifies that the listener works at all and added missing propagation in ChainedListener.

👍


Problem with Loading Weights

Hi, thank you for your work.
I want to further instruction-tune the mBLIP mt0  model using my data. I have set the blip_pretrained_checkpoint argument to the pytorch_model-00001-of-00002.bin file from the mBLIP repository, and the lm_pretrained argument to the bin files in the mBLIP repository, which correspond to the encoder and decoder, along with the config.json and pytorch_model.bin.index.json files, in the bigscience/mt0-xl repository. However, when I instantiate the mBLIP class and load the weights, I encounter an error in line (https://github.com/gregorge/mBLIP/blob/f804c1f8bb84b13795b71aaaa6fe3f44851c908b/src/modules/modeling/mblip.py#L185C97-L185C97). The process proceeds without issues when I set load_in_8bit=False. But when I call model.generate(**inputs), it produces nonsensical output. Could you please advise on how to train the model based on your provided checkpoint?
Thank you very much.

Hi,
thanks for your interest in the project. I understand that the process of further training my checkpoints is tricky so I understand your problem. It is unfortunate that HuggingFace does not yet support loading only parts of a model in 8bit which makes the whole split into the LLM and Vision+Q-Former part necessary.
I was trying to write some brief instructions on how to do what you want but I realized that it is actually quite hard so I updated the code.
First of all, your process is nearly correct. The problem is that loading of the mt0 fails because 1) the checkpoint keys do not match what mt0 expect ("language_model." as prefix) and because 2) the index.json in mt0 is different from the index in my repository. Fixing those two things is possible but annoying. This would explain why it generates gibberish.
With the commit I pushed just now, you can set 'lm_pretrained=Gregor/mblip-mt0-xl' and it should work; blip_pretrained_checkpoint stays as pytorch_model-00001-of-00002.bin. Note that I only tested it briefly if the model loaded this way still produces something correct with generate so let me know if you have other problems.
Also, next time you have some issues, please include the full log and stack trace of the error.

Hello, thank you very much for your response. Actually, I encountered another error after saving checkpoints which raises an AttributeError: "'GroupedOptimizerTridentModule' object has no attribute 'lightning_module'". I receive this error when I run the instruction-tune experiment. The warm up is okay. Thank you very much.
run_jan.log

If scroll up your stack trace, you will see that your actual error is:
Traceback (most recent call last):
  File "/home/mBLIP/src/tasks/vllm/checkpoint.py", line 50, in _save_checkpoint
    model = trainer.model.model.model
  File "/home/benenv/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1614, in __getattr__
    raise AttributeError("'{}' object has no attribute '{}'".format(
AttributeError: 'DistributedDataParallel' object has no attribute 'model'

Unfortunately, I cannot replicate this problem. When I start a training run with ddp, in line 52 trainer.model.module.lightning_module.model.model  yields the mBLIP object as it should.
I suggest stopping there with a debugger (you can set trainer.limit_train_batches: 10,  trainer.limit_val_batches: 10 to get there quicker) and check what yields the expected object.
The lightning version might be a reason, I use 2.0.1.
Alternatively, you can remove ddp (if you use only one GPU) and see if that works.


[Navigation Material] BottomSheetState progress not updated as it should be

Description
After updating accompanist from 0.28.0 to 0.30.1 I couldn't use progress.fraction anymore from the SwipeProgress object in the bottom sheet state so I now use progress directly from the new bottom sheet state.
I noticed there is an issue with the update of this value and I wonder if this is intended behavior.
When I scroll in a BottomSheetScaffold, on the first pixels (around 50px) the value of progress is not updated right away. The same issue happens when the sheet is fully expanded and I swipe to collapse, progress value is not updated in the first pixels. You can check the videos down below for more context.

Before (0.28.0)
After (0.30.1)

https://user-images.githubusercontent.com/9885223/233347676-8c4e606e-e670-4f5d-8417-1079b8e10608.mp4
https://user-images.githubusercontent.com/9885223/233347757-b472a6d9-2b88-463b-a366-881b5c33f9ce.mp4

Steps to reproduce
@Composable
fun BottomSheet() {
    val scaffoldState = rememberBottomSheetScaffoldState(
        bottomSheetState = rememberBottomSheetState(
            initialValue = BottomSheetValue.Collapsed
        )
    )

    BottomSheetScaffold(
        sheetContent = {
            Content(scaffoldState.bottomSheetState)
        },
        scaffoldState = scaffoldState,
    ) { padding ->
        Box(
            modifier = Modifier
                .background(Color.Yellow)
                .fillMaxSize()
        ) {
            Text(
                modifier = Modifier
                    .align(Alignment.Center)
                    .padding(padding),
                color = Color.Blue,
                text = scaffoldState.bottomSheetState.progress.toString()
            )
        }
    }
}

@Composable
private fun Content(
    bottomSheetState: BottomSheetState,
) {
    println("updateProgress ${bottomSheetState.progress}")

    Column(
        modifier = Modifier
            .background(Color.Red)
            .fillMaxSize()
    ) {}
}

Expected behavior
The progress value should be updated as soon as the the bottom sheet is moving.

Any updates on this?


[Pk2] Modal not working (inactivates all content)

in both, Firefox and Safari on macOS 10.13 (Beta) the modal causes the page to "inactivate" all content. The user isn't able to close the modal and go back to the main content anymore.
See screenshot.
The only resolution is to reload the page.

Thank you for pointing out. It seems the problem is coming from having a position: relative and z-index on the .section. To fix this you have 2 options:

Move the modal-body outside of the div.section
Add this CSS at the end of paper-kit.css .section { position: unset !important; }

We will do some more tests and see which is the best option for the next update.
Best,
Alex