Please add a note to use new entity

Please update this page to list the suggestion to use entity "SalesInvoiceJournalHeaderEntity" instead of "SalesInvoiceHeaderV2Entity" which may cause performance problems for some data

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

ID: 6ef365a4-a54f-8c78-b60c-dc085db2c2a3
Version Independent ID: b423fe0b-b666-ea32-29ac-471af0507961
Content: SalesInvoiceHeaderV2Entity in AccountsReceivable - Common Data Model - Common Data Model
Content Source: common-data-model/schema/core/operationsCommon/Entities/Finance/AccountsReceivable/SalesInvoiceHeaderV2Entity.md
Service: common-data-model
GitHub Login: @llawwaii
Microsoft Alias: weiluo

@Venkata1920 Do you know about these eneities?


A way to safely remove a arranged subView

The current implementation of removing arrangedSubView is not great because even after removing arranged subView the view is still present. Remove arrangedSubView should work as expected.
A bonus would be if we can remove all the arrangedSubViews together. Some thing like -
    @objc func removeAllArrangedSubViews()  {         for arrangedSubView in arrangedSubviews {             removeArrangedSubview(arrangedSubView)             arrangedSubView.removeFromSuperview()         }         updateConstraints()     }

The official UIStackView does not behave like that.
"This method removes the provided view from the stack’s arrangedSubviews array. The view’s position and size will no longer be managed by the stack view. However, this method does not remove the provided view from the stack’s subviews array; therefore, the view is still displayed as part of the view hierarchy."

Interesting, at least the animation is still preserved!
    [stack removeArrangedSubview:subview];
    [subview removeFromSuperview];
    // Still animates out nicely!


maestro: 1.21.2 -> 1.21.3

Description of changes
https://github.com/mobile-dev-inc/maestro/releases/tag/cli-1.21.3

Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 213584 run on aarch64-darwin 1

1 package built:

maestro


TV 2's new video URLs cause "Unsupported URL"

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I remove or skip any mandatory* field

Checklist

[X] I'm reporting a broken site
[X] I've verified that I'm running yt-dlp version 2022.09.01 (update instructions) or later (specify commit)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[X] I've read about sharing account credentials and I'm willing to share it if required

Region
No response
Provide a description that is worded well enough to be understood
TV 2 Norway has introduced a new URL format, and it is not picked up by the extractor. Eg.:
https://www.tv2.no/video/nyhetene/tv-2-hjelper-deg/saa-mye-kan-du-spare-paa-klesvasken/1787176/
But manually changing the URL back to the old format works:
https://www.tv2.no/v/1787176/
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
[debug] Command-line config: ['https://www.tv2.no/video/sport/carlsen-ga-seg-etter-to-trekk-fullstendig-uakseptabel-oppfoersel/1788052/', '-vU']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version 2022.09.01 [5d7c7d6] (pip)
[debug] Python 3.8.9 (CPython 64bit) - macOS-12.5.1-arm64-arm-64bit
[debug] Checking exe version: ffmpeg -bsfs
[debug] Checking exe version: ffprobe -bsfs
[debug] exe versions: ffmpeg 5.1 (setts)
[debug] Optional libraries: Cryptodome-3.15.0, brotli-1.0.9, certifi-2022.06.15, mutagen-1.45.1, sqlite3-2.6.0, websockets-10.3
[debug] Proxy map: {}
[debug] Loaded 1670 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Latest version: 2022.09.01, Current version: 2022.09.01
yt-dlp is up to date (2022.09.01)
[debug] [generic] Extracting URL: https://www.tv2.no/video/sport/carlsen-ga-seg-etter-to-trekk-fullstendig-uakseptabel-oppfoersel/1788052/
[generic] 1788052: Downloading webpage
WARNING: [generic] Falling back on generic information extractor
[generic] 1788052: Extracting information
[debug] Looking for Brightcove embeds
[debug] Looking for embeds
WARNING: [generic] 1788052: Failed to parse JSON: Extra data: line 18 column 7 (char 876)
[debug] Identified a twitter:player iframe
[debug] [generic] Extracting URL: https://www.tv2.no/video/sport/carlsen-ga-seg-etter-to-trekk-fullstendig-uakseptabel-oppfoersel/1788052/?wf=ext
[generic] ?wf=ext: Downloading webpage
WARNING: [generic] Falling back on generic information extractor
[generic] ?wf=ext: Extracting information
[debug] Looking for Brightcove embeds
[debug] Looking for embeds
WARNING: [generic] ?wf=ext: Failed to parse JSON: Extra data: line 18 column 7 (char 876)
ERROR: Unsupported URL: https://www.tv2.no/video/sport/carlsen-ga-seg-etter-to-trekk-fullstendig-uakseptabel-oppfoersel/1788052/?wf=ext
Traceback (most recent call last):
  File "/Users/krhah771/Library/Python/3.8/lib/python/site-packages/yt_dlp/YoutubeDL.py", line 1459, in wrapper
    return func(self, *args, **kwargs)
  File "/Users/krhah771/Library/Python/3.8/lib/python/site-packages/yt_dlp/YoutubeDL.py", line 1535, in __extract_info
    ie_result = ie.extract(url)
  File "/Users/krhah771/Library/Python/3.8/lib/python/site-packages/yt_dlp/extractor/common.py", line 670, in extract
    ie_result = self._real_extract(url)
  File "/Users/krhah771/Library/Python/3.8/lib/python/site-packages/yt_dlp/extractor/generic.py", line 3078, in _real_extract
    raise UnsupportedError(url)
yt_dlp.utils.UnsupportedError: Unsupported URL: https://www.tv2.no/video/sport/carlsen-ga-seg-etter-to-trekk-fullstendig-uakseptabel-oppfoersel/1788052/?wf=ext

Just changing the _VALID_URL pattern fixes this.
Is there any view as to whether the pattern /video/category/subcat/item/id (if my scarcely existent Norwegian is right) is always used? Do the new URLs have all these path components? Are there /audio/... URLs? And so on.
Also, the patterns for the TV2ArticleIE don't seem to be used now, but videos from a URL like https://www.tv2.no/mening_og_analyse/dette-er-ikke-laerernes-ansvar/15120661/ can be grabbed from the assetId inside a TV2.TV2Video({...}) JS fragment. The IE's pattern should probably be extended to cover tv2.no pages not handled by TV2IE.

The old format,tv2.no/v/[id] is still in use, for example on this overview page
It seems just /video/category/idis in use in some places, eg. https://www.tv2.no/video/nyhetene/venter-paa-groent-lys-saa-skjer-dette/1790240/.
And /video/absolutely/anything/here/without/any/limits/id works, so from what I can tell, just ignoring anything between videoand id could be the safest option.
/audio/... is not a thing.


Always get "ERROR ITMS-90046" while deliver to iTunesConnect

I am always running into an error in the "deliver" action:
[17:16:11]: [Transporter Error Output]: ERROR ITMS-90046: "Invalid Code Signing Entitlements. Your application bundle's signature contains code signing entitlements that are not supported on iOS. Specifically, value 'development' for key 'aps-environment' in 'Payload/EMQ.app/EMQ' is not supported."

I googled around, and tried sigh and resign actions, but that doesn't help. I double check the provisioning profile that the "sigh" action downloads, it looks fine, the "maps-environment" in the provisioning profile is "production", not "development". Is there anyone has similar issue like me?
And here is the Fastfile
lane :deliver_demo do
#   increment_build_number
    sigh(output_path: "./fastlane/profiles")
    ipa(
      workspace: 'EMQ.xcworkspace',
      configuration: 'Debug',
      scheme: 'EMQ-Demo',
      ipa: 'EMQ-Demo.ipa',
      embed: './fastlane/profiles/Distribution_com.emq.emqapp.mobileprovision',
      identity: 'iPhone Distribution: EMQ Limited (28HUT34N52)',
      clean: true,
    )
    resign(
      ipa: 'EMQ-Demo.ipa',
      signing_identity: 'iPhone Distribution: EMQ Limited (28HUT34N52)',
    )
    deliver(
      beta: true,
      skip_deploy: true,
    )
end

The best way to do code signing right is to set the provisioning profile in the project settings, since there is a bug in the xcodebuild command. Please update your project file and try again. You'll not need resigning or specifying the embed and identity any more.

@KrauseFx Thanks. The solution works!

Where should I put this information to make it easier for new users?

Maybe a FAQ document?


arcdps isn't updating

Please attach a copy of config.yaml or config.ini to the issue.
Describe the bug
The gw2addon_arcdps.dll file located in ~\Guild Wars 2\addons\arcdps hasn't been updating when I run the addon manager. I would continue to get the errors shown in the screenshot below.

To Reproduce
Run Gw2-64.exe
Additional context
Issue was manually resolved by downloading arcdps, renaming from d3d9.dll to gw2addon_arcdps.dll, and placing in the ~\Guild Wars 2\addons\arcdpsdirectory effectively replacing the old.dll` file
don't forget to attach the config file to the issue
log and config.zip

I think this was resolved shortly after I posted.


openapi: Remove documentation for unstable timezones.json path

The location of files in /static is not part of our stable API.

Yeah, that's reasonable. Merged, thanks @andersk!
It's possible we should define a stable API for this if clients need that data set.


Comparisons with other text-processing systems

We want to compare TextDB with other related systems.  We will create an issue for each system, and include those issues here.
@xuxip

IBM Bluemix: See separate email
Lingpipe: http://alias-i.com/lingpipe/
Meta-toolkit: https://meta-toolkit.org/
https://en.wikipedia.org/wiki/Information_extraction#Free_or_open_source_software_and_services
General Architecture for Text Engineering "General Architecture for Text Engineering", which is bundled with a free Information Extraction system
OpenNLP Apache OpenNLP is a Java machine learning toolkit for natural language processing
OpenCalais Automated information extraction web service from Thomson Reuters (Free limited version)
Machine Learning for Language Toolkit (Mallet) is a Java-based package for a variety of natural language processing tasks, including information extraction.
DBpedia Spotlight is an open source tool in Java/Scala (and free web service) that can be used for named entity recognition and name resolution.
Natural Language Toolkit is a suite of libraries and programs for symbolic and statistical natural language processing (NLP) for the Python programming language

@xuxip @fukoyui Per our discussion last Friday, for your own task, create a separate issue to keep track of the progress.  The goal is to use the use case of Zika extraction to understand the behavior of those systems.

@shirleyqt If I understand correctly, a lot of the above systems have been studied but there are no references to their Wiki pages or the PRs. Could you please update this issue with that information?

@kishore-narendran Sure. Do you mean to update the links in this issue or on the wiki page of TextDB?

@shirleyqt Update the links here. Yes.

@kishore-narendran I have updated the ones that were evaluated by me and Hailey. For the other parts (unevaluated), do I delete them or update the links and leave them there?

@shirleyqt Just use the checkboxes in the first comment left by the professor. And we will know


PyTest "XPASS" is incorrectly treated as a failure regardless of strict setting.

When using the xfail (expected failure) mark on a test, teamcity-messages treats an XPASS result (when the test unexpectedly passes) as a failure state even though PyTest only considers an unexpected pass to be a failure of the test when strict is set to True in the xfail mark.

Unfortunately TeamCity has only three test states: passed/failed/ignored. How do you want to present XPASS result without strict mark?

Hi, when extract py.test results to junit xml it going to skipped category:
the test:
import pytest

@pytest.mark.xfail()
def test_a():
    assert True

the result:
<?xml version="1.0" encoding="utf-8"?>
<testsuite errors="0" failures="0" name="pytest" skips="1" tests="0" time="0.015">
  <testcase classname="test_a" name="test_a" time="0.000144958496094">
    <skipped message="xfail-marked test passes unexpectedly"/>
  </testcase>
</testsuite>

So i think ignored is good category.


[ERROR] IoTHub client connect fail: MQTT CONNECT FAIL status: 65541

Hi I am using the b_u585i_iot02a board for the Azure-IOT-Central application. I applied all of the instructions to execute the NetXDuo/Nx_Azure_IoT application but I am stuck at this point.
ATCModem:Tempo started (150000 ms)...
ATCModem:Tempo started (150000 ms)...
ATCore ERROR:AT_sendcmd error
ATCore ERROR:AT_sendcmd error: parse from rsp
LOG ERROR #3: channel=1 / errorId=11 / gravity=2
LOG ERROR #4: channel=1 / errorId=14 / gravity=2
[ERROR] IoTHub client connect fail: MQTT CONNECT FAIL status: 65541
Disconnected from IoTHub!: error code = 0x00010005
Reconnecting iothub, after backoff
ATCModem:Tempo started (150000 ms)...
ATCModem:Tempo started (150000 ms)...
Could someone help me to fix this?

Hi @Clyasenth Thank you for reaching out to us. Looks the issue is for cellular module (ATCore ERROR), have you connected the cellular module to u5? You may also ask for help from ST: https://community.st.com/t5/stm32-mcus-embedded-software/bd-p/mcu-embedded-software-forum

Hi @bo-ms thanks for the quick reply. Yeah, I connected the cellular module to u5 which uses bg96. This log appears after the cellular module initialization.
Init sensors on board:
    Init magnetometer IIS2MDC sensor

    Init gyroscope and accelerometer ISM330DHCX sensor

    Init temperature and humidity HTS221 sensor

    Init temperature and pressure LPS22HH sensor

    Enabled magnetometer IIS2MDC sensor

    Enabled accelerometer ISM330DHCX sensor

    Enabled gyroscope ISM330DHCX sensor

    Enabled temperature HTS221 sensor

    Enabled humidity HTS221 sensor

    Enabled pressure LPS22HH sensor

Init Full Scale 4G for accelerometer ISM330DLC
Init Full Scale 250DPS for gyroscope ISM330DLC
[INFO] Azure IoT Security Module has been enabled, status=0
IoTHub Host Name: *****;
Device ID: *****.
ATCModem:Tempo started (150000 ms)...
ATCModem:Tempo started (150000 ms)...
ATCore ERROR:AT_sendcmd error
ATCore ERROR:AT_sendcmd error: parse from rsp
LOG ERROR #1: channel=1 / errorId=11 / gravity=2
LOG ERROR #2: channel=1 / errorId=14 / gravity=2
[ERROR] IoTHub client connect fail: MQTT CONNECT FAIL status: 65541
Disconnected from IoTHub!: error code = 0x00010005
Reconnecting iothub, after backoff
ATCModem:Tempo started (150000 ms)...

Hi @Clyasenth I am not quite sure if the AT_sendcmd error impacts it. For IoT Central application, I guess you use certificate for connection, correct? could you check if the certificate is set correctly in your project? Where do you get this project? if ST provides this project, you may contact them. Thanks.

Closing due to no further response, feel free to reopen the issue


Got error when executing Hive on Snappy supported Hadoop

Hi, I just tried to run hive script with azkaban 3.0 with the hive plugin, and this error came up:
"Unable to get CompressorType for codec (org.apache.hadoop.io.compress.SnappyCodec). This is most likely due to missing native libraries for the codec."
My Hadoop supports SnappyCodec and every configuration in hadoop and azkaban plugin seem correct. I think the problem may come from missing native library configuration but I can't find any place to set. Any suggestion?
Thanks

I just knew how this came up. The problem is from my tez configuration. I set tez.runtime.compress to true to compress my intermediate data and use SnappyCodec as the compression method.  After I set the config to false, every thing succeeded. I still don't understand why this parameter cannot be used.


Bulleted lists use double spacing - takes too much vertical space

Bulleted lists, whether created using HTML

 tags or markdown asterisks, gobble up too much vertical space.
For example, the following table of contents takes up almost a full screen on my laptop:

I fixed this: https://github.com/flutter/website/pull/368


fetch-client does not correctly handle multipart/form-data

Description
When using the fetch-client and generating the services the attached schema snippet does not correctly send a multipart/form-data request. The body is set correctly however the Content-Type incorrectly gets set to "application/json".
For the fetch-api to correctly send a multipart/form-data request the body needs to be of type FormData and no Content-Type header should be set. When manually trying to pass FormData into the body parameter (cast to any) of the service method  it still sent a request with Content-Type "application/json" instead.
Reproducible example or configuration
config file:
import { defineConfig } from '@hey-api/openapi-ts'

export default defineConfig({
  input: './openapi/swagger.json',
  output: './generated/client',
  client: '@hey-api/client-fetch',
  services: {
    asClass: true
  }
})

usage:
  client.setConfig({
    baseUrl: 'http://localhost:3000'
  })
  const ret= await DefaultService.UploadFile({
    client,
    body: { name, file }
  })

OpenAPI specification (optional)
  "post": {
    "operationId": "UploadFile",
    "responses": {
      "200": {
        "description": "Ok",
        "content": {
          "application/json": {
            "schema": {
              "type": "number",
              "format": "double"
            }
          }
        }
      }
    },
    "parameters": [],
    "requestBody": {
      "required": true,
      "content": {
        "multipart/form-data": {
          "schema": {
            "type": "object",
            "properties": {
              "name": {
                "type": "string"
              },
              "file": {
                "type": "string",
                "format": "binary"
              }
            },
            "required": ["name", "file"]
          }
        }
      }
    }
  }

System information (optional)
No response

Thanks for reporting @sinomsinom, this should get fixed in v0.52.2 https://github.com/hey-api/openapi-ts/pull/854


Don't run CI for draft PRs

This changes our CI to not run on draft pull requests and marks CI as "skipped." When the PR is marked as ready for review CI should run.
To do so, we add an if conditional to every job that checks for draft status. In addition, if an override exists (currently only one job uses the override), we'll concatenate the conditionals. The brackets in the conditional have been removed as they're unnecessary (github actions automatically treat if blocks as being enclosed in brackets) which makes the concatenation much easier.

Is there a way to manually schedule CI on drafts, or nope?

Is there a way to manually schedule CI on drafts, or nope?

I don't think so. My typical workflow before would've been to open a draft, see CI pass, and then mark as ready for review. After talking to @benhillis, I think we decided that we'd still rather not run on drafts so you would need to mark it as ready before seeing the result of CI.


https://github.com/Ph0rk0z/text-generation-webui-testing/commit/367ec0aa5ed5c2bf42b782f75e3b01c4e4993d95

File "/UNICOMFS/hitsz_mzhang_1/.conda/envs/quantize/lib/python3.9/site-packages/transformers/models/gpt_neox/modeling_gpt_neox.py", line 155, in forward
qkv = self.query_key_value(hidden_states)
File "/UNICOMFS/hitsz_mzhang_1/.conda/envs/quantize/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1518, in _wrapped_call_impl
return self._call_impl(*args, **kwargs)
File "/UNICOMFS/hitsz_mzhang_1/.conda/envs/quantize/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1527, in _call_impl
return forward_call(*args, **kwargs)
File "/UNICOMFS/hitsz_mzhang_1/.conda/envs/quantize/lib/python3.9/site-packages/peft/tuners/lora/layer.py", line 310, in forward
result = self._linear(x)
File "/UNICOMFS/hitsz_mzhang_1/.conda/envs/quantize/lib/python3.9/site-packages/peft/tuners/lora/layer.py", line 298, in _linear
return F.linear(input, transpose(self.weight, self.fan_in_fan_out), bias=self.bias)
RuntimeError: self and mat2 must have the same dtype, but got Half and Int
this error occurred when I use the code, have any one see this before?

You trained a LORA for neox?


Improve uncalled function check related to parenthesized and binary expressions

fixes https://github.com/microsoft/TypeScript/issues/37598

@typescript-bot test this
@typescript-bot user test this inline
@typescript-bot perf test this faster
@typescript-bot test top100

@jakebailey you requested perf results for this some time ago. Any chance that you'd have some time to review this now when they are already here? :)

@jakebailey I was wondering if there is a chance to land this before entering the 5.0 beta period (3 days left 😅 )

I was hoping for someone else to weigh in (I don't entirely trust myself to merge large-ish checker PRs), but it still seems fine to me given it's an "experience enhancement" aka "obviously an improvement" and top100 passed (so, probably low-risk).


[Flow] Make CollapseDimensions iterative

Each operation must propagate it's collapse information to every other node in the dispatch to ensure that no collapse_shape ops are needed between ops in the dispatch.
If a directed edge is defined as going from
consumer -> producer, information is first propagated by going N steps following the direction of the edge, and then M steps opposite the edge. This is "1 iteration". It will take >1 iteration to propagate this information. The failing IR was similar to:
    Root
   /    \
Child1  Child2

First, each child is updated. Then the root is updated from each child. However, no information can be propageted between Child1 <-> Child2.

Fixes issue mentioned in https://github.com/iree-org/iree/issues/18005#issuecomment-2278885607 (failure mentioned in the comment; not the original error)


Create Mocked Tests for Volume Actions

Overview
Add more test coverage for Pydo, specifically for Volume Actions.
Instructions

Clone this repo.
If you haven't already, install Python (version: >= 3.7.2) through their official installer.
CD into the directory and run make install
View file: tests/mocked/test_block_storage.py/mocked
Add tests for volume_actions.get, volume_actions.list,  volume_actions. post, volume_actions. post_by_id
API documentation is here
Pydo Documentation is here

hey @danaelhe can I look into this?

hey @danaelhe can I look into this?

Absolutely 😄 Thanks!


Optimization variable selector for flash

The corresponding branch is variable_dispatch, I made it from dev.
The aim of the dispatch is a comfort way to switch between different optimization variables.
I see them in a manner of
"See [`PhysicalVariables`](@ref), [`FractionVariables`](@ref), [`IdealIdentityVariables`](@ref)."
abstract type VTFlashStateVariables end

"Moles and volume."
struct PhysicalVariables <: VTFlashStateVariables
end

"Mole of i-th component in 1st phase by total moles of i-th component, saturation."
struct FractionVariables <: VTFlashStateVariables
end

"Variables for which ideal part of hessian of compositions is identity matrix."
struct IdealIdentityVariables <: VTFlashStateVariables
end

vt_flash(PhysicalVariables(), mix, nmol, volume, RT)
vt_flash(FractionVariables(), mix, nmol, volume, RT)
vt_flash(IdealIdentityVariables(), mix, nmol, volume, RT)

# End-user API
vt_flash(mix, nmol, volume, RT; state_variables=PhysicalVariables())

Where the structs <: VTFlashStateVariables are singletons for dispatch.

162388bc55221166145111840d37f5cde9bac30e relates to calculation of gradient and hessian for NVT statement in "physical variables": moles [mol] and volume [m3]. This calculation is "tolerant" to the singletons (see issue message).
I guess, for other sets of variables corresponding hessians may be derived by scaling from those general functions. Perhaps, the scaling may be computationally inefficient in some cases.

I also thought about some wrapping of state minimization-vector, e.g.
struct PhysicalVariablesState{T}
     x::Vector{T}
end

PhysicalVariablesState(mix) = ... # allocates `ncomponents(mix) + 1` vector

state = PhysicalVariablesState(mix)

And dispatching on type of state.
But this cause .-boilerplate like nmol = @view state.x[1:end-1] instead of nmol = @view state[1:end-1] like in current code.
Also, when we work with VarsFoo and want to (reuse code)

Call a method corresponding to VarsBar
Post-process the output, transforming to VarsFoo

It seems to causing extra wrapping (possibly, allocations or "remind where the pointer to x is and where x is origin vector") or some manipulation with convert.

Draft of general interface
4aa9bc29bd5734a7447d90c447f6b9edb27e5a56 introduces draft of general vt_flash interface. For now, both versions can be ran.
The interface is on top of src/vt_flash/vt_flash.jl file.
PhysicalState -base calculation is tested on a two-phase point for (methane + n-pentane) using the draft.

Ratio variables
875e99b4bee1a2ed9b5650e35c0798818c6b7d1f and 4b5d2b0e9bce037a9aab841d3de87dab86bebd0f introduce RatioState variables which are N'1 / N1, ..., N'n / Nn, V' / V.
These variables are still used in vt_flash(mix, nmol, volume, RT) with another (old) implementation. So old implementation may be compared to vt_flash(mix, nmol, volume, RT, RatioState).
Test example
For a four-component mixture the implementations give
julia> mix
BrusilovskyEoSMixture{Float64}(nitrogen + methane + propane + n-decane)

julia> nmol
4-element Vector{Float64}:
 0.0004925999999999999
 0.0004416
 0.0004416
 0.0006242

julia> volume
1.0e-6

julia> RT
4157.23130907662

julia> ratiostate = vt_flash(mix, nmol, volume, RT, CubicEoS.RatioState; gtol=1e-3/RT);

julia> fracold = vt_flash(mix, nmol, volume, RT);

julia> dump(ratiostate)
CubicEoS.VTFlashResult{Float64, CubicEoS.RatioState{Vector{Float64}}}
  singlephase: Bool false
  RT: Float64 4157.23130907662
  nmolgas: Array{Float64}((4,)) [0.0004516319327931407, 0.0003890882771558721, 0.0003401765215206347, 0.00013229574796212154]
  volumegas: Float64 8.378179547180461e-7
  nmolliq: Array{Float64}((4,)) [4.096806720685924e-5, 5.251172284412791e-5, 0.00010142347847936527, 0.0004919042520378785]
  volumeliq: Float64 1.6218204528195384e-7
  state: CubicEoS.RatioState{Vector{Float64}}
    x: Array{Float64}((5,)) [0.08316700610405856, 0.1189124158607969, 0.22967273206378006, 0.7880555143189337, 0.16218204528195385]
  optim: CubicEoS.VTFlashOptimStats
    converged: Bool true
    iters: Int64 7
    calls: Int64 8

julia> dump(fracold)
CubicEoS.VTFlashResult{Float64, Vector{Float64}}
  singlephase: Bool false
  RT: Float64 4157.23130907662
  nmolgas: Array{Float64}((4,)) [0.00045163193279314093, 0.0003890882771558725, 0.0003401765215206362, 0.00013229574796212186]
  volumegas: Float64 8.378179547180467e-7
  nmolliq: Array{Float64}((4,)) [4.096806720685899e-5, 5.2511722844127456e-5, 0.00010142347847936376, 0.0004919042520378781]
  volumeliq: Float64 1.6218204528195325e-7
  state: Array{Float64}((5,)) [0.08316700610405806, 0.11891241586079587, 0.22967273206377664, 0.7880555143189332, 0.16218204528195326]
  optim: CubicEoS.VTFlashOptimStats
    converged: Bool true
    iters: Int64 7
    calls: Int64 8

julia> norm(ratiostate.state.x - fracold.state)
3.688777608244322e-15

Same results!
P.S. for now I'm considering

refactor for closure generation, because they seem very similar for RatioState and PhysicalState;
a destructive option for nmolvol(s::AbstractVTFlashState...) for memory saving;
non-allocating versions for gradient! and hessian!(hess, s::AbstractVTFlashState, ...).

Ideal Identity variables
1e7bd57936849834eb0cbc1d79451f5bce901ab9 and 310c0be0dc759f1282d7b193d8316cbd0da1b10d implement IdealIdentityState for which ideal part of hessian over moles is identity matrix.
    IdealIdentityState

Variables that make ideal part of hessian over moles to identity matrix.

state = [ 2 √Nᵢ arcsin(√N'ᵢ / √Nᵢ); √N V'/V ], where N = Σᵢ Nᵢ (total moles).
         |-----------------------| |-------|
                    α'ᵢ               α'V

4dfe4a81d0dc14efc4bf0fbb46eeb704eb51528f adds comparing test between all implemented AbstractVTFlashStates (physical, ratio and idealidentity). They are passed under rtol=1e-5 for moles and volumes of gas and liquid phases, when gtol=1e-5/RT is used in calculations.
Currently, the comparison assumes transitivity of ≈-operator (x ≈ y rtol=1e-5).
The test is passed!

Newton vtflash
0ab2a57ed7721b263eed211fa5f0c6f4daf11ea3 add support for vt_flash calculation based on Newton optimization.
The solver is (unit-)tested 351470feecbade830d6d3aa9d5a57fe3d1d31dc7 against bfgs.

TODO: check type-stability of nvt-functions and state-related functions like hessian, gradient etc.

Closed by #22.


prevent initial focus when container has button listen enter event

Details

Fixed Issues
$ https://github.com/Expensify/App/issues/48108
PROPOSAL: https://github.com/Expensify/App/issues/48108#issuecomment-2327146125
Tests

Go to Search
Click on Filters
Click on a specific filter
Enter your query, use return key on your keyboard to confirm
You’re taken back to the main filters page
Now enter the return key again on the filter summary/confirmation screen
Verify that the filter is applied

[x] Verify that no errors appear in the JS console

Offline tests

QA Steps

Go to Search
Click on Filters
Click on a specific filter
Enter your query, use return key on your keyboard to confirm
You’re taken back to the main filters page
Now enter the return key again on the filter summary/confirmation screen
Verify that the filter is applied

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

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

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native
https://github.com/user-attachments/assets/84ba454d-d528-4e39-a2c6-c1c19602132c

Android: mWeb Chrome
https://github.com/user-attachments/assets/0de3036a-6214-4801-b157-00514b0dac76

iOS: Native
https://github.com/user-attachments/assets/05ddfc58-a246-4ecf-8235-3e6bb1b2ba02

iOS: mWeb Safari
https://github.com/user-attachments/assets/169bd303-df17-43fa-8db9-c34fc2cba7bf

MacOS: Chrome / Safari
https://github.com/user-attachments/assets/5171f3ea-d57c-4ce6-8ee9-b3d70beee6c3

MacOS: Desktop
https://github.com/user-attachments/assets/7bdd2e05-0942-4571-9d96-0ecbb189a591

@alitoshmatov The PR is ready for review

Reviewer Checklist

[x] I have verified the author checklist is complete (all boxes are checked off).
[x] I verified the correct issue is linked in the ### Fixed Issues section above
[x] I verified testing steps are clear and they cover the changes made in this PR

[x] I verified the steps for local testing are in the Tests section
[x] I verified the steps for Staging and/or Production testing are in the QA steps section
[x] I verified the steps cover any possible failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[x] I checked that screenshots or videos are included for tests on all platforms
[x] I included screenshots or videos for tests on all platforms
[x] I verified tests pass on all platforms & I tested again on:

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

[x] If there are any errors in the console that are unrelated to this PR, I either fixed them (preferred) or linked to where I reported them in Slack
[x] I verified proper code patterns were followed (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick).
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method
[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I verified that this PR follows the guidelines as stated in the Review Guidelines
[x] I verified other components that can be impacted by these changes have been tested, and I retested again (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar have been tested & I retested again)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG)

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR reviewer checklist, including those that don't apply to this PR.

Screenshots/Videos

Android: Native
https://github.com/user-attachments/assets/e9469c91-1fcb-417a-b4d3-8b873f32966b

Android: mWeb Chrome

iOS: Native
https://github.com/user-attachments/assets/575336ff-61a9-4350-b6db-979f538a0270

iOS: mWeb Safari
https://github.com/user-attachments/assets/1583454a-e9a5-441f-8219-4b1cf343b88a

MacOS: Chrome / Safari
https://github.com/user-attachments/assets/102a2b9f-74fc-44ad-96b5-096fb65824e8

MacOS: Desktop
https://github.com/user-attachments/assets/ec14361a-6301-4291-a514-01d3fe3ad98c

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/luacmartins in version: 9.0.38-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/grgia in version: 9.0.38-4 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Discrepancy in information pulled in Cloud Vision website vs API - DominantColorAnnotation

hi there, the results I get from using the API vs what I get from the website are very different. I understand that the highest score reflects most accurately the dominant colour. But I've uploaded the picture to both the website and my backend server (which calls the API) and the results from the JSON file from the google website and my data are different.
for example this is what I get from my log (10 results)
 {     color: { red: 225, green: 226, blue: 223, alpha: null },     score: 0.28191062808036804,     pixelFraction: 0.41662895679473877   },   {     color: { red: 24, green: 27, blue: 35, alpha: null },     score: 0.11526796966791153,     pixelFraction: 0.09317941963672638   },   {     color: { red: 210, green: 210, blue: 206, alpha: null },     score: 0.070244699716568,     pixelFraction: 0.09265127778053284   },   {     color: { red: 41, green: 49, blue: 63, alpha: null },     score: 0.1388927698135376,     pixelFraction: 0.08767164498567581   },   {     color: { red: 44, green: 47, blue: 57, alpha: null },     score: 0.09161456674337387,     pixelFraction: 0.07393994182348251   },   {     color: { red: 24, green: 30, blue: 44, alpha: null },     score: 0.10027392953634262,     pixelFraction: 0.061641767621040344   },   {     color: { red: 72, green: 83, blue: 99, alpha: null },     score: 0.04788002744317055,     pixelFraction: 0.03123585321009159   },   {     color: { red: 76, green: 80, blue: 88, alpha: null },     score: 0.02934001013636589,     pixelFraction: 0.026859816163778305   },   {     color: { red: 105, green: 118, blue: 134, alpha: null },     score: 0.020987385883927345,     pixelFraction: 0.013882601633667946   },   {     color: { red: 73, green: 42, blue: 26, alpha: null },     score: 0.01754932850599289,     pixelFraction: 0.00920476857572794   } ]
vs this is what I get from the website

"dominantColors": {       "colors": [         {           "color": {             "blue": 63,             "green": 49,             "red": 41           },           "hex": "29313F",           "percent": 31.317973386555092,           "percentRounded": 31,           "pixelFraction": 0.087671645,           "rgb": "41, 49,\n 63",           "score": 0.29549706         },         {           "color": {             "blue": 86,             "green": 107,             "red": 150           },           "hex": "966B56",           "percent": 2.4598561685261897,           "percentRounded": 2,           "pixelFraction": 0.015919723,           "rgb": "150, 107,\n 86",           "score": 0.023209684         },

I will not paste the whole thing but the first result from the json is the most accurate colour. If you compare their scores its different. Any idea why? I've been playing around with this for nearly a week.)
Why is there a discrepancy?
Would appreciate all the guidance.
This is my code to get the logged results.
   const colors =         results[0]?.imagePropertiesAnnotation.dominantColors.colors;         colors?.forEach(color => console.log("all the colours",color));       const dominantColour = colors[0]?.color;

The demo on the website is intended to showcase the functionality, but is not a "source of truth" for testing.
If we are able to find out any more specifics about parameters, API version, etc. used in the demo, we can update this ticket.
We'll go ahead and close this issue for now, as there isn't a bug in the Node JS library, which this repo supports.


Get fills params issue

I tried to get list fills with request parameters:

startAt =<PHONE_NUMBER>000
endAt =<PHONE_NUMBER>000

The reponse content is a message "Get List Fills doesn't support get fills in a large timespan (> 24 hours)" with code "400100"

Looks like this is something built into the api, probably to limit the amount of data being transferred via a rest request. If you check out the telegram channel: https://t.me/KuCoin_API this looks to be happening to others. I would suggest dropping your timespan to less than 24hours as the error message suggests.


Document config files

We should add comments to the config files explaining how things link together there. We can use rdfs:comment for that similar to how it is used in the generated configs.

Also perhaps put some extra thought in directory organization, file naming, etc.

FYI, there's the comment shortcut that's available by default:  https://github.com/LinkedSoftwareDependencies/Components.js/blob/master/components/context.jsonld#L58-L60


Package via pip is Out of Date/Broken for Python 3

The python package available on via pip install django-shortuuidfield is an old version that is broken for python3. (Specifically all commits after and including the updated raised exception handling aren't reflected in the package.
   from shortuuidfield import ShortUUIDField
  File "/.../python3.4/site-packages/shortuuidfield/__init__.py", line 4
    except Exception, e:
                    ^
SyntaxError: invalid syntax

Steps to Repro:

pip install django-shortuuidfield (https://pypi.python.org/pypi/django-shortuuidfield/0.1.2)
Try to use the package on django + python3

Thanks for the heads up . I'll try to get the pip version updated right away.

@mriveralee sorry it took a few days to get around to this, if you can fork to make it py3 compatible and submit a pull request, I'll pull it in and get the pypi file updated.  I noticed there is an older pull request but it has conflicts so i can't cleanly merge it, so you can fork that one if you want or just start from master here. Sorry to be neglectful... just finished up my tour of duty on the international space station

@nebstrebor -- I submitted a pull-request that merges the fixes from the older pull request you provided.

merged, and pypi updated.  Thanks for your help.


[FEATURE] - Sidebar Options Design Enhancement

Description
Working on the styling of the side navbar options styling to improve their design.
Screenshots
Current Design

Expected Updated Design

Additional information
No
Record

[X] I agree to follow this project's Code of Conduct
[X] I have checked the existing issues
[X] I'm a GSSoC'23 contributor
[X] I want to work on this issue

Hi @narayan954
Kindly assign me this issue.

Hi @iamRabia-N pr #1193 is in progress for this section currently. Let's wait out before we work on this component.
Also I'm not sure if this color looks nice here. Closing this as not planned right now


penlight registering globals?

Hi,
I wrote
require "pl"
and it seems that its tables (for ex. path) are being registered as globals rather than being part of their own namespace. I went into path.lua and it's setting the local _G to the global _G. Any reason why this is so?
(This is in the Luajit repl btw)

require "pl" is used when you are in a hurry ;)  It does create globals like path,etc, although does not load these modules unless they're actually used.  (So at least it does not pull in ~ 12,000 lines of Lua.)  When writing actual modules that use Penlight, the recommended way is to do it explicitly:
local utils = require 'pl.utils'
local path = require 'pl.path'

If these damage the global environment (except for pl.strict) then I'd consider it a bug!

Ah, ok, thanks!

Ah cool, thanks!

Ah, cool, thanks!


Update build instructions (and build process)

I am building biber (on Linux as well as on Windows, mainly because my Linux system has only Perl 5.26 and I am not sure it finishes. edit: it does not, see below). I noticed the following things:

Docs

have a working Perl 5.24+ installation

It now requires Perl 5.30, according to Build.PL:
    !  perl (5.26.1) is installed, but we need version >= 5.30.0

Also later in build.sh, I get:
/home/berker/perl5/bin/pp: Cannot find file or directory /usr/local/perl/lib/5.30.0/Unicode/Collate/Locale for packing
/home/bers/perl5/bin/pp: Cannot find file or directory /usr/local/perl/lib/5.30.0/Unicode/Collate/CJK for packing
/home/bers/perl5/bin/pp: Cannot find file or directory /usr/local/perl/lib/5.30.0/Unicode/Collate/allkeys.txt for packing
/home/bers/perl5/bin/pp: Cannot find file or directory /usr/local/perl/lib/5.30.0/Unicode/Collate/keys.txt for packing
/home/bers/perl5/bin/pp: Cannot find file or directory /usr/local/perl/lib/site_perl/5.30.0/Mozilla/CA/cacert.pem for packing
/home/bers/perl5/bin/pp: Cannot find file or directory /usr/local/perl/lib/5.30.0/x86_64-linux-thread-multi/PerlIO for packing
/home/bers/perl5/bin/pp: Cannot find file or directory /usr/local/perl/lib/5.30.0/x86_64-linux-thread-multi/auto/PerlIO for packing
/home/bers/perl5/bin/pp: Cannot find file or directory /usr/local/perl/lib/site_perl/5.30.0/Business/ISBN/RangeMessage.xml for packing

Docs

source tree from SF

This is now GitHub, I guess.

Build.PL

> perl ./Build.PL

WARNING: the following files are missing in your kit:
        META.json
        META.yml
Please inform the author.

Docs

Run the test suite with Build test

This could/should be ./Build test (also a few more times elsewhere)

pp
I had to manually install PAR::Packer on both Windows and Linux.

Win32::Unicode::File
I had to force-install Win32::Unicode::File (due to https://github.com/xaicron/p5-win32-unicode/issues/7). This is an issue many Windows users may face, see also https://www.perlmonks.org/?node_id=1208723

build.bat
On Windows, I had to edit build.bat as my perl is installed in C:\Programs\Strawberry, and not in C:\Strawberry.

pp -u
I also removed from this file --unicode for now due to https://github.com/rschupp/PAR-Packer/issues/27 (although I guess this will not be required long-term, for PAR::Packer>1.050).

build.sh
On Linux, I also had to edit build.sh as my perl is installed in /home/bers, and I had to adapt several library locations (/usr/lib64, /usr/lib), library names (libz.so ), and versions (libssl.so.1.1).

Test
biber-MSWIN64.exe --validate-control --convert-control test works, with one warning:

INFO - This is Biber 2.16 (beta)
INFO - Logfile is 'test.blg'
INFO - 'test.bcf' validates against schema 'C:\Users\bers\AppData\Local\Temp\par-59616e6e69636b\cache-13c92ab7980b36f3fc123230ca0f3cd98a08e73c\inc\lib/Biber\bcf.rng'
INFO - Converted BibLaTeX control file 'test.bcf' to 'test.bcf.html'
INFO - Reading 'test.bcf'
INFO - Using all citekeys in bib section 0
INFO - Processing section 0
INFO - Looking for bibtex format file 'test.bib' for section 0
Use of uninitialized value in subroutine entry at C:\Users\bers\AppData\Local\Temp\par-59616e6e69636b\cache-13c92ab7980b36f3fc123230ca0f3cd98a08e73c\inc\lib/Biber/Utils.pm line 361.
Use of uninitialized value $filename in concatenation (.) or string at C:\Users\bers\AppData\Local\Temp\par-59616e6e69636b\cache-13c92ab7980b36f3fc123230ca0f3cd98a08e73c\inc\lib/Biber/Input/file/bibtex.pm line 121.
WARN - Data source '' is empty, ignoring
INFO - Overriding locale 'en-US' defaults 'variable = shifted' with 'variable = non-ignorable'
INFO - Overriding locale 'en-US' defaults 'normalization = NFD' with 'normalization = prenormalized'
INFO - Sorting list 'nty/global//global/global' of type 'entry' with template 'nty' and locale 'en-US'
INFO - No sort tailoring available for locale 'en-US'
INFO - Writing 'test.bbl' with encoding 'UTF-8'
INFO - Output to test.bbl
INFO - WARNINGS: 1

Docs will be updated for 2.16 accordingly.


Automatic wrapping of Gzip streams

At compile time mark assets as Gzipped if the header value is there, and automatically wrap in Crate.open() without needing to check at runtime. Also provide an openRaw() alternative for cases where the user may not want it to be wrapped.

https://github.com/oriley-me/crate/commit/ecc2ee1a196d44cfcf8d57d6e94c21b107bef51c


License

There doesn't seem to be a license specified for this project, was this intentional? Just wanted to know if I'm able to fork this and make a version with a UI. I haven't looked at the code quite yet so if not I can go ahead with a clean implementation.

Done, thank you!


Landmarker live page is no longer reacheable

www.landmarker.io now is some sort of advertising website, and the domain is on sale

@grigorisg9gr @jabooth @lirsacc  Hi ! I am facing a similar issue. Could someone please look into this ? Thank you so much !

The issue still holds on. Is the menpo project still maintained?


Revert "Fix fetching of ID tokens when using a service account (#487)"

I thought it would be more intuitive to just do a full revert (i.e. back to version 2.32.4, instead of 2.32.4 being the same as 2.32.6)

But 2.32.5 has already been released, so we'd need to yank that version?

But 2.32.5 has already been released, so we'd need to yank that version?

I'm not sure how our automatic analysis-runner "please update" messages would handle that.

I'm not sure how our automatic analysis-runner "please update" messages would handle that.

Not very well: https://github.com/populationgenomics/analysis-runner/blob/43cb4ce72c4a8601846a453a4b8c92821926fe32/analysis_runner/util.py#L147


Directory parameter

Can not use directory 'sources/css' in the root repository.
Output message: "find: sources/css: No such file or directory"
I am using "uses<EMAIL_ADDRESS>

I already solved it by adding checkout and commit actions.


minlength being ignored

Anyone know why this might be happening?
const h = require("hyperscript");

const el = h("input", { type: "text", minlength: 42, maxlength: 420 });
console.log(el.outerHTML);

<input type="text" maxlength="420">

Just want to confirm that this doesn't seem to be caused by the underlying HTML-Element library:
const document = require("html-element").document;

const el = document.createElement("input");
el.setAttribute("type", "text");
el.setAttribute("minlength", 42);
el.setAttribute("maxlength", 420);
console.log(el.outerHTML);

https://developer.mozilla.org/en-US/docs/Web/API/HTMLInputElement
The hyperscript library sets DOM properties, not HTML attributes, try minLength and maxLength

Wow, that's fantastic. I don't think that I realized the difference. I'm a bit surprised that maxlength worked, but I'll try this instead. Thanks a bunch.

Sorry, I'm getting the same result with minLength. Are you getting different behavior between minlength and minLength? It's the same for me.

Looks like a bug in the HTML-Element library -- I'll report upstream.
const document = require("html-element").document;

const el = document.createElement("input");

el.type = "text"
el.minlength = 42
el.minLength = 43
el.maxlength = 420
el.maxLength = 421

console.log(el.outerHTML);

<input type="text" maxlength="420" maxLength="421">

But running this in a browser yields:
<input type="text" minlength="43" maxlength="421">

That PR got merged and they've released a new version of the package so this issue is probably solved now?

A quick test and the issue seems to still be there for me even when using latest html-element, but someone should take a closer look


Remove dependence on regtest in our unit tests

Some of our unit tests rely on our API, regtest specifically. As regtest is being deprecated, we should change this to use testnet.
Reenable:

src/store/transactions/fungible-token-transfer.spec.ts
src/pages/transaction-signing/hooks/use-submit-stx-transaction.spec.ts

@kyranjamie mind t-shirting this with one of the new labels?

Which unit test(s) in particular are currently impacted by this dependence?

This should be done soon because there are 2 important tests not running at the moment to release with confidence

Neither of the files listed in the description exist any longer but the heystack mocks all use regtest so it does still need to be addressed.

Why do we keep heystack alive? our docs are not using it anymore.
@markmhx can we just archive it?

Yep go for it


Using key:object (rather than key:string) in translation file

Subject of the issue
Using key:object in translation file
Description
We are using Smartling for our translation service. They allow for us to provide instructions inside of the translation in the format of
"FOO": { 	"translation": "Translated Foo", 	"notes": "Limit to a total of 20 characters" }
however I am not finding a way to be able to handle this with angular translate. I could write a simple parser but would rather not do that. I was reading up on custom interpolation but since that just passes to $interpolate with no way of knowing if the translation is a string or object it is of little help. Is there some way to handle this?

Did you check on namespaces? (or do I get the question wrong?)
https://angular-translate.github.io/docs/#/guide/02_getting-started
You can access
FOO.translation
and
FOO.notes

Did you check on namespaces? (or do I get the question wrong?)

This would work but only if we are following that format for everything. Right now we only use that format if we need notes.

The dot syntax is essential in the project currently. Feel free to provide a concept on how to achieve your suggestion without breaking changes, but I personally currently don't know what the goal is currently. Maybe you could provide a conceptional example / explanation.

"FOO": { "translation": "Translated Foo", "notes": "Limit to a total of 20 characters" }, "BAR": "Translated Bar"
Is what I would like to accomplish where there is a way to define the library to always look at the "translation" key for the translation if it exists. Otherwise just use the value provided for the key. I do not see a way to configure this sort of thing. Hopefully I explained that clearly enough to make sense.

Sorry, I don't get it.

Hello,
in part of a "Spring Cleaning 2017", we are closing this issue as part of several others. The meta discussion is available here for full transparency.
 Said this, we must keep the focus and scope on the project.
We are grateful and appreciate to everyone for each made contribution (regardless being code, idea, documentation, and support).
We are deprecating/closing all issues which fulfills a specific set of criteria: mainly outdated, nice-to-have, or just out-of-scope.

Although the list has been curated to the best of our's knowledge, there will be some lack of understanding.
Okay, there a different stories behind all open issues, but we do not have the time for everything:

Support: Because we have only limited resources, we cannot handle support questions any more. Please use appropriate boards like Stackoverflow.
Feature requests or ehancements: Altough it is probalbly a great idea (otherwise it would be closed already), we can not giving more resources to things out of scope or which are useful for too specific scenarios only.
Bugs: Filed a long time ago, this bug either has a reasonable workaround, no effective solution without harming all others, or is lacking additional information/use-case.
Docs: All documentation efforts not matching English will be abonded.

Thank you.
angular-translate team

FAQ
"But this is really important!"
Well, we are all humans. Maybe we have overseen an important issue. If we missed something, we hope you are reaching as again filing a new issue with the reference to the old one. But please be aware: A bug without further details and a demo included will be rejected again.
"But this would a great feature!"
Reconsider: Are we really required to add something in order to make this feature happen? Or could you also create a project on your own utilizing this project?


[TypeScript] DragSource component return type expects unneeded props

Describe the bug
When a component is wrapped in DragSource, the return value is a component that expects props to be passed in from a parent, but still expects the props that DragSource is providing itself.
To Reproduce
Simple example:
import * as React from 'react';
import * as ReactDOM from 'react-dom';
import { ConnectDragSource, DragSource, DragSourceConnector } from 'react-dnd';

interface OwnProps { name: string; }
interface DragProps { connectDragSource: ConnectDragSource; }

type AllProps = OwnProps & DragProps;

class Foo extends React.Component<AllProps> {
  public render(): JSX.Element {
    return this.props.connectDragSource(
      <div>{this.props.name}</div>
    );
  }
}

function mapDragProps(connect: DragSourceConnector): DragProps {
  return { connectDragSource: connect.dragSource() };
}

const source = {
  beginDrag(props: AllProps): { name: string } {
    return { name: props.name };
  }
};

const DraggableFoo = DragSource('FOO', source, mapDragProps)(Foo);

ReactDOM.render(<DraggableFoo name='dylan' />, document.body);

Expected behavior
Code compiles
Actual behavior
Type '{ name: string; }' is not assignable to type '(IntrinsicAttributes & IntrinsicClassAttributes<Foo> & Readonly<{ children?: ReactNode; }> & Read...'.
  Type '{ name: string; }' is not assignable to type 'IntrinsicAttributes & IntrinsicClassAttributes<Component<AllProps, ComponentState, any>> & Readon...'.
    Type '{ name: string; }' is not assignable to type 'Readonly<AllProps>'.
      Property 'connectDragSource' is missing in type '{ name: string; }'.

I also tried changing the 2nd-to-last line to:
const DraggableFoo = DragSource('FOO', source, mapDragProps)(Foo) as React.ComponentClass<OwnProps>;

— which fixes the rendering props issue but fails at coercing:
Type 'typeof Foo & DndComponentClass<AllProps>' cannot be converted to type 'ComponentClass<OwnProps>'.
  Types of parameters 'props' and 'props' are incompatible.
    Type 'OwnProps' is not comparable to type 'Readonly<AllProps>'.
      Property 'connectDragSource' is missing in type 'OwnProps'.

Desktop:

OS: macOS
Version: react 16.4.1, react-dnd 5.0.0

I was having the same issue earlier. Turns out that I was typing the source wrong.
In your code you should type props as OwnProps instead of AllProps.
 const source = {   beginDrag(props: OwnProps): { name: string } {     return { name: props.name };   } };

@c16192 Nice, thanks!

@c16192 Whoops, spoke too soon. This doesn't solve it for me. When I change the beginDrag line to use OwnProps, I get the following error:
Argument of type 'typeof Foo' is not assignable to parameter of type 'ComponentClass<OwnProps>'.
  Types of parameters 'props' and 'props' are incompatible.
    Type 'OwnProps' is not assignable to type 'Readonly<AllProps>'.
      Property 'connectDragSource' is missing in type 'OwnProps'.

I might be missing something obvious here...

@schneidmaster Which version are you on? I didn't realize that was generic, so I think you're onto something here, but I'm getting Expected 5 type arguments, but got 1. on<EMAIL_ADDRESS>

Ah, I'm on<EMAIL_ADDRESS>Not sure offhand what the type signature looks like on 4.

Cool, yep, confirmed that that does work on 5.0.0, but has caused some other errors. Will try and get a reproducible example. Thank you!

I stumbled upon the same problem. Looking at DragSource.d.ts I actually don't understand why both the argument and return type use the same probs. The argument should consume the collected probs next to the external probs?
This is the current definition from https://github.com/react-dnd/react-dnd/blob/master/packages/react-dnd/src/DragSource.ts#L79:
return function decorateSource<
  TargetClass extends
    | React.ComponentClass<Props>
    | React.StatelessComponent<Props>
>(DecoratedComponent: TargetClass): TargetClass & DndComponentClass<Props>

I think it should read something like this:
return function decorateSource(
  DecoratedComponent: React.ComponentType<Props & CollectedProps>
): React.ComponentType<Props> & DndComponentClass<Props>

React.ComponentType is just  a shorthand for React.ComponentClass<Props> | React.StatelessComponent<Props>. The important bit is appending CollectedProps to the props of the argument.

@sebastian-lenz it totally right. Was just here to open a PR myself but checked the tickets before. Would you mind open a PR for this? Same for DropTarget of course.

The typings should be much better in 7.4


Nonzero op extension

support dynamic output shape...

This should probably be implemented with prefix_sum. Since we don't have a gpu implementation yet, we can at least insert synchronization in lowering and just use the ref implementation,

This should probably be implemented with prefix_sum

Maybe it would be better to have a filter operator that takes in a tensor of booleans to filter, but we would still need scan for the GPU.


Remove misplaced curly bracket

There is a misplaced closing curly bracket in TODO10 replacement JavaScript.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/develop/create-sso-office-add-ins-aspnet.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.


[HotFix] Git plugin

This is a hot fix for the last Git plugin update #2790.
STATUS: READY for MERGE

TODO:

[x] Fix the local call — fixes #3962
[x] Bring back the old gg* aliases — fixes #3972
[x] Recreate aliases for git add — see #3535 and #3682 and #3972, fixes #3980
[x] Change the gdc alias — fixes #3977
[x] Fix the error messages in the gwip alias — see this comment
[x] Investigate the current_branch function — see this comment
[x] Fix the gg* functions to accept additional arguments — see this comment
[x] Fix the ggpnp function — fixes #3991

Closing forgotten issues: fixes #3751, fixes #3756

I don't think there's a way to make that top-level git_cmd work as a local variable. It's not the plugin loading system's fault, it's a zsh limitation: Zsh only supports dynamic scoping, not lexical scoping, and doesn't have namespaces or packages. (Same for other shells.) You want that $git_cmd to be shared between independent functions and persist between calls (and, it looks like, make it visible to the Zsh completion system). That needs a regular global variable.
If the plugin loading system changed such that the top-level local really was local because the plugin was loaded inside a function, then it would break, because it would only be local to the initial run of the plugin definition file itself. Subsequent calls to the functions defined in the plugin would not see the "local" value.
A global $git_cmd isn't just the easy fix, it's the only fix AFAIK.
Maybe just rename it to $_omz_git_git_cmd to make it clear where it's coming from and avoid collision with variable names that the user or other scripts might use; $git_cmd is pretty generic and I could see other things picking that.
You could probably do something like declare one big global $_OMZ_VARS associative array for various OMZ-defined variables, and have all the functions that want to use it temporarily pop it in and out of truly local variables inside their functions. You'd still be using global vars, but you'd reduce the number of names it was taking. However, since they're still global names, you could still collide with those other variable names inside the OMZ function calls, e.g. if you're using callbacks, or calling other commands that use the collided variables. So the code would be a lot uglier, and still brittle in subtle ways. I'm not a fan of this hack.

Thanks a bunch @apjanke! I will rename the variable and remove the local keyword, until we figure out a variable handling system.

@itsNikolay @tejasbubane @Newman101 @MarcoSero @opsidao @tehmaspc @ChrisJefferson @blueyed @Stibbons
Do we agree on gaa for git add --all?
What alias would you suggest to replace gap for git add --patch?
What alias would you suggest to replace gdc for git diff --cached?

@ncanceill gaa for git add --all is good for me :+1:

Until a safe aliasing system is written, easy fix is to use something else than gap.

I would introduce the safe_alias method for this:
omz_safe_alias() {
    # With `safe_alias foo='bar baz'`, define the alias only if it is not a
    # command, function, or alias already.
    local alias lhs rhs
    alias=${(s:=:)1}
    lhs=$alias[1]
    rhs=$alias[2,-1]
    if (( $+commands[$lhs] || $+functions[$lhs] || $+aliases[$lhs] )); then
        return 1
    fi
    alias $lhs="$rhs"
}

Otherwise the next complains would be about this alias being removed / not added again.

@blueyed looks fine, can you make a PR? also we will need to echo a warning to advise the user that the alias was not created, and maybe a setting in .zshrc to change the behavior to instead erase the command with the alias and print an appropriate warning.

@ncanceill

looks fine, can you make a PR?

I'd say to include/use it in the git plugin first (with this PR), and then move it to a central place later (with the settings etc).
I've changed the return 1 to return 73 (according to Python's os.EX_CANTCREAT), to make it different than a failure from the alias itself (if you try alias foo=bar baz).

If we want to have this used throughout OMZ entirely, why would we first implement it in the Git plugin? I think this belongs either in oh-my-zsh.sh or in a lib file.
Nevertheless, I agree on testing it in the Git plugin first.

OK all problems should now be fixed.
Last thing to do is pick aliases for git add --patch and git diff --cached. Please send suggestions!
I want to get this PR out fast, so without any suggestion by tomorrow I will have to pick the aliases myself.

Nevertheless, I agree on testing it in the Git plugin first.

Then you could use it for gap and gdc, can't you?

+1 for gdca

OK that does it, time is up for hot fix suggestions.
@blueyed I think pushing a new, non-tested feature in a hot fix would be irresponsible, I will be working on it in a new PR.

New/changed aliases:

gaa = git add --all
gapa = git add --patch
gdca = git diff --cached

@robbyrussell please pull this hot fix in ASAP

Can't wait until this gets merged.

Thanks for jumping on this, folks!

+1 (late review)

Thanks @robbyrussell, and sorry to have caused this mess in the first place ;-)

@ncanceill no one knows future - so don't judge yourself :+1:

Speaking as the original complainer about the 'gap' alias, having 3 letter aliases is (a) useful for most people most of the time, but was (b) always going to clash with something, some day :) I'm glad things are now improved :)


Item position changes on upon page reload

Hi. Thanks for the great script. I've noticed the position of parallax items changes when reloading the page, differing to their position from initial load of the page. Why is this and how can it be resolved?
Thanks

I noticed something similiar to this when I resize the browser. When I refresh the page, the position resets to where it should be though.

First of all thanks for this small and beautiful plugin. It does the job just right.
Yes I see the same issue. Once we load the page and resize the browser. The element losses its original position. Usually user don't resize their browser but developers do. It would be great if this plugin takes care of it.

I had the same problem that after a reload the elements were shifted. I noticed that after some scrolling all the elements were back in place. So I simply created the following dirty hack. This function scrolls the page down 1 pixel after a page reload:
$(window).on('load', function () {
    $('html, body').animate({
        scrollTop: '+=1px'
    }, 100);
});


Triangle performance

Thank you for helping out with embedded-graphics development! Please:

[ ] Check that you've added passing tests and documentation
[ ] Add a CHANGELOG.md entry in the Unreleased section under the appropriate heading (Added, Fixed, etc) if your changes affect the public API
[ ] Run rustfmt on the project
[ ] Run just build (Linux/macOS only) and make sure it passes. If you use Windows, check that CI passes once you've opened the PR.

PR description
I've found an easy optimization for the triangle drawing performance. is_collapsed was unnecessarily calculated once per scanline. Calculating it only once in the constructor gives a significant performance improvement:

I'm appreciating these smaller PRs btw, thanks for splitting the changes up like this. I'll attempt to do the same in the future.

Thanks for the quick review!


Cannot launch vscode with command 'code'

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.74.2
OS Version: MacOS 13.2, intel chip

Steps to Reproduce:

install VS Code and install code command to path.
open terminal, and try to use code . to open the folder in vscode

I've tried to create that folder, but the system doesn't allow me to do so, even with root account.

I've checked other two folders, and both have write permission.

What's the result of running env in your terminal?

TERM_SESSION_ID=w0t0p0:E7F5AB85-2681-4E8D-AF11-938DD0816DEF
LC_TERMINAL_VERSION=3.4.18
COLORFGBG=15;0
ITERM_PROFILE=Default
XPC_FLAGS=0x0
LANG=zh_CN.UTF-8
PWD=/Users/pangxw
SHELL=/bin/zsh
__CFBundleIdentifier=com.googlecode.iterm2
TERM_PROGRAM_VERSION=3.4.18
TERM_PROGRAM=iTerm.app
PATH=/Users/pangxw/.local/share/cargo/bin:/Users/pangxw/.local/share/rustup/toolchains/stable-x86_64-apple-darwin/bin:/Users/pangxw/.local/share/pnpm:/usr/local/opt/ccache/libexec:/Users/pangxw/.ghcup/bin:/Users/pangxw/.local/bin:/Users/pangxw/.bin/flutter/bin:/Users/pangxw/.bin:/Users/pangxw/.local/share/opam/default/bin:/Users/pangxw/.config/nvm/versions/node/v18.12.1/bin:/Users/pangxw/.zi/polaris/bin:/usr/local/bin:/System/Cryptexes/App/usr/bin:/usr/bin:/bin:/usr/sbin:/sbin:/Library/TeX/texbin:/usr/local/MacGPG2/bin:/Library/Apple/usr/bin:/Users/pangxw/.cabal/bin:/Users/pangxw/.local/bin
LC_TERMINAL=iTerm2
COLORTERM=truecolor
COMMAND_MODE=unix2003
TERM=xterm-256color
HOME=/Users/pangxw
TMPDIR=/var/folders/j6/kxwnfz_x44s4mxdd676wg1kh0000gn/T/
USER=pangxw
XPC_SERVICE_NAME=0
LOGNAME=pangxw
ITERM_SESSION_ID=w0t0p0:E7F5AB85-2681-4E8D-AF11-938DD0816DEF
__CF_USER_TEXT_ENCODING=0x0:25:52
SHLVL=1
OLDPWD=/Users/pangxw/.zi/snippets/OMZL::git.zsh
ZPFX=/Users/pangxw/.zi/polaris
PMSPEC=0fuUpiPsX
ZCDR=/Users/pangxw/.config/zi
ZSH_CACHE_DIR=/Users/pangxw/.cache/zi
STARSHIP_SHELL=zsh
STARSHIP_SESSION_KEY=2005720050298513
XDG_CONFIG_HOME=/Users/pangxw/.config
XDG_CACHE_HOME=/Users/pangxw/.cache
XDG_DATA_HOME=/Users/pangxw/.local/share
XDG_STATE_HOME=/Users/pangxw/.local/state
XDG_RUNTIME_DIR=/run/user/501
DOCKER_CONFIG=/Users/pangxw/.config/docker
GRADLE_USER_HOME=/Users/pangxw/.local/share/gradle
IPFS_PATH=/Users/pangxw/.local/share/ipfs
MACHINE_STORAGE_PATH=/Users/pangxw/.local/share/docker-machine
FFMPEG_DATADIR=/Users/pangxw/.config/ffmpeg
GOPATH=/Users/pangxw/.local/share/go
JUPYTER_CONFIG_DIR=/Users/pangxw/.config/jupyter
KDEHOME=/Users/pangxw/.config/kde
PYENV_ROOT=/Users/pangxw/.local/share/pyenv
TEXMFHOME=/Users/pangxw/.local/share/texmf
TEXMFVAR=/Users/pangxw/.cache/texlive/texmf-var
TEXMFCONFIG=/Users/pangxw/.config/texlive/texmf-config
EDITOR=nvim
PUB_HOSTED_URL=https://mirror.sjtu.edu.cn/dart-pub
FLUTTER_STORAGE_BASE_URL=https://mirror.sjtu.edu.cn
GHCUP_USE_XDG_DIRS=true
NPM_CONFIG_USERCONFIG=/Users/pangxw/.config/npm/npmrc
NPM_CONFIG_TOKEN=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
NVM_DIR=/Users/pangxw/.config/nvm
NVM_CD_FLAGS=-q
NVM_BIN=/Users/pangxw/.config/nvm/versions/node/v18.12.1/bin
NVM_INC=/Users/pangxw/.config/nvm/versions/node/v18.12.1/include/node
OPAMROOT=/Users/pangxw/.local/share/opam
OPAM_SWITCH_PREFIX=/Users/pangxw/.local/share/opam/default
CAML_LD_LIBRARY_PATH=/Users/pangxw/.local/share/opam/default/lib/stublibs:/Users/pangxw/.local/share/opam/default/lib/ocaml/stublibs:/Users/pangxw/.local/share/opam/default/lib/ocaml
OCAML_TOPLEVEL_PATH=/Users/pangxw/.local/share/opam/default/lib/toplevel
PNPM_HOME=/Users/pangxw/.local/share/pnpm
PLTUSERHOME=/Users/pangxw/.local/share/racket
HOMEBREW_PREFIX=/usr/local
_=/usr/bin/env

@Pcrab there might be some GH secrets in your comment. We suggest you edit your comment and rotate your secrets.

Please unset XDG_RUNTIME_DIR from your maxOS shell.


Move  keep_pointer_in_view to pointer.js and add viewport to modules list

Move keep_pointer_in_view to pointer.js. viewport is already a module.

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

Merged.  I expanded the commit message for the "Add viewport module" commit to clarify that the viewport module already existed.
Thanks for doing this, @hackerkid!  I'm definitely appreciating these clean commits.


Bug Fix for Hygieia App version display

Spring boot tokenization works only for properties bundles inside the spring boot jar. To cater to this and preserve backwards compatibility.

Introduced a new file called version.properties
File gets updated by maven during build phase with current version.
The property value is injected in the class requiring this.

LGTM

LGTM


[RFR] Bullet-proof scut-image-replace

Reference: https://twitter.com/HugoGiraudel/status/565093151904645120.
Bug on Chromium: https://code.google.com/p/chromium/issues/detail?id=457146.
Cheers. :)

Also thanks.


Locale detection

It's a first version of a locale detection extension. The locale can be determined by cookie and request header (Accept-Language). The result is written to the Detected-Locale request header and can be accessed by the {locale} placeholder.
The tests are running and the locale seems to get parsed, but somehow the middleware does not seem to get called. I'm new to caddy, so maybe I'm missing something. A review would be welcome.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.

Well, that are a lot of suggestions. I'll see them through. Unfortunately, none seems to solve the functional issue, that the middleware isn't called at all.
localhost:3000

locale en de {
  detect cookie header
  cookie locale
}

rewrite / {
  to index.{locale}.html index.html
}

I've added locale to the directive list in caddy/directives.go and the setup code is called on startup. A configured middleware is returned, but is not called on a request. I'm sure it's something simple that I oversee.

The reason why it is not called is because locale is at the end of the directives list. Rewrite is ahead and would have executed and gotten empty value for {locale} before locale is called.

Great! Thanks for the quick answer. That worked!

It might feel hostile at first when receiving one's first code review(s). It means being questioned, having to justify, results in work whose purpose is not immediately visible. We all have been there. But, it's something inherently friendly: One purpose of a code review is to help you make the best contribution you can.
A PR is not merely a gift you make to others—who would point out flaws in a gift? It's, on top of that, asking someone else to maintain (eventually) and accept blame for it.
Absolutely everyone sometimes gets 'suggestions' and/or exposes identifiable flaws. And has to iterate on something. Do not feel offended, nobody thinks less of you. (And, we're certainly able to identify an impasse or equivalent ways of solving things.) Carry on with professional grace. :wink:

Thanks for the philosophical insight :-) But let's focus back on the code.
I did some refactoring and after @abiosoft 's hint, the code is working now. Next for me would be this scoping issue. Should not take too long.

Ok, the locale detection is now scoped. For me, the code works now.

Most comments have not been addressed.

Hey Philipp, thank you very much for your PR! I think it'll be important for Caddy to better support international audiences.
I think the tests here are very good. The _test package convention is, however, inconsistent with the rest of Caddy's middleware tests, since it forbids us from unit testing the unexported functions at a later time if we choose to do so without first changing the package name.
@wmark Why does the locale detection need to be scoped to a path? Wouldn't a site owner want this for the whole site?

It's about having no more than necessary in critical chains.
You need this if you have content C1 in a set of languages L1 only,  and a different set L2 for C2.
Paths meant for API access could not be using this at all.

@phifty It will be helpful if you can have a high level description of this middleware since there is no related issue. Basically, Caddyfile syntax and expected output.
Example. https://github.com/mholt/caddy/pull/259.
That way, anyone can get the idea before (or without) going through the code.

@mholt You're welcome! Thanks for the great work on caddy too. It really makes it much less painful to set up a webserver :-)
I usually take this external-package approach, because it makes the tests a good documentation of the package's usage. Since I write the test first, it also makes me think about the package's interface and behavior before I actually implement anything. If the implementation gets too deep at some point, I usually take as a hint that I can break up the package into smaller pieces.
Anyway - since consistency is an argument, I can move it into one package.
I find the path scoping quite unnecessary since it doesn't invoke any expensive operation and does not modify the request (apart from adding the header). But I don't have a strong opinion about this.
@abiosoft Sure, here is schema of the current configuration.
locale <availableLocales...> {
  detect <methods...>
  cookie <cookie name>
  available <availableLocales...>
  path <path scope>
}

A method can be currently cookie or header. If cookie is added, cookie name defines from which cookie the locale is read. The header method extracts the locales from the Accept-Language header. The first availableLocale is also the default locale, which is picked if none of the locales from the detection methods is in availableLocales.
The defaults are: methods = [header],  cookie name = locale, path scope = /.
My production configuration looks like this:
locale en de {
  detect cookie header
}

rewrite {
  ext \
  to index.{locale}.html index.html
}

header / Vary "Cookie, Accept-Language"

It's a single page app, which serves the index on all path without extensions.

This is looking really good so far, thanks @phifty -- I think all that is left is to decide whether this should be an add-on or built into core. What do other web servers do about locale handling?

@mholt I don't know. By what criteria is this decision made usually? I think apache and nginx have the locale detection in a separate module.

@phifty Ultimately it's my decision; I try to learn about the feature as much as I can, and have to weigh the cost and benefits of including it in core. The main question here is knowing whether this will be used by enough Caddy users to put into core. I tend to lean toward making new middlewares add-ons because we can always bring one into core later, it's harder to take it out.
At this point, it's a very well-written middleware and it will make it into Caddy one way or another. I'm just not sure how.
Anyone want to put in their vote (with reasoning, would be helpful)?

I'd vote for the add-on. Let's see how many users actually using it. I imagine that there will be some users that want locale detection by hostname (subdomain) or path (/en/...). Both is not implemented yet.

FWIW, the hostname is implied by the header of the server block where the address of the site is:
hostname {
   locale ...
}

Anyway, I'm OK with going forward to package this as an add-on. Here's the guide: https://github.com/mholt/caddy/wiki/Extending-Caddy - you've basically already done step 1 except that it should be moved to its own package.

Ok, the code has it's own repository now: https://github.com/simia-tech/caddy-locale
I'll close this PR now, since it seems obsolete.


As a community library manager, I want to record the merit score of a borrower

so that I can check if a borrower is eligible to borrow more books.

create merit score attribute in person
integrate into borrow

Closed by #62


Using as a typeahead?

First off, thanks for all of the hard work that went into this library! Sorry for not following the issue template, downshift/npm/node version are irrelevant as this is more of a question.
Problem description:
It seems like this module was built with flexibility in mind but right now, as far as I can tell, you cannot enter a custom value not present in the "options" array. E.g. if the options are [ 'Apples', 'Oranges', 'Bananas' ], I can't enter a value of Kiwi. Once you type into the input and lose focus, the input clears.
Consider the following example, and try typing "kiwi" and then tabbing / clicking outside of the input:
https://codesandbox.io/s/r9mzy8zo
Suggested solution:
Typing in a value and not selecting an option persists the inputValue prop.
If this doesn't align with the views of this package, I totally get it. But it seems like it has all of the foundation to make this an option :)

Hey, sorry I don't have much time (will come back later to help more when I have more time), but this is totally possible by controlling the selectedItem (and potentially inputValue) state. See the README for how to do that.

Type Ahead example here. This is based on Bare Bones autocomplete but enhanced to support type ahead.
Two changes required to Bare Bones to make this work convert to stateful component and use control props. Specifically these two arguments passed to Downshift:
onStateChange={({ inputValue }) => this.setState({ inputValue })}
selectedItem={this.state.inputValue}

@kentcdodds I added the downshift:example tag to this CodeSandbox, will that add it to the README? If not I will manually add.

Thanks @codiemullins! Do you think that you could do a little more in the example to show how that could be useful?
Then you'll need to make a PR to add it in the README. Thanks again!

@kentcdodds I'm happy to do so. If you or @nehero have any thoughts on how to make the example more useful let me know! I am struggling to think of ideas

Thanks for the solution @codiemullins! Sure, I'll come back to this in a bit with an example

I think that you could just render the selectedItem in another place on the page and that would be sufficient...

Sounds good! PR for readme coming. Updated https://codesandbox.io/s/82m2px40q9 to display the value on the page.
@nehero Note: I had to improve the onStateChange in the example because after the input lost focus I noticed if the typed in value wasn't in the items array then it would set it to undefined, checking for undefined resolves the issue.

Thanks for the example! I'll go ahead and close this now.
As an after-thought, it'd be really cool if the example could autofill the input with the first result with the part I haven't yet typed "selected" in the input field (so I can keep typing). That would actually be a very awesome example.


Write to indexeddb first

This is a safety check to make sure that we write to indexeddb first. This should be done so if writing to the in memory map fails or if the program stops in the middle, we make sure we right to indexeddb which is actually persistent. That way if writing to memory fails, then if the user just refreshes, the data is not lost.
Second commit changes it to use try_read and try_write, we need to use these because the others would block if waiting which could cause a panic for us. This instead will return an Error and can be handled correctly.

Second commit changes it to use try_read and try_write, we need to use these because the others would block if waiting which could cause a panic for us. This instead will return an Error and can be handled correctly.

How does this work then? What if two things are trying to write to memory at the same time? One fails?

How does this work then? What if two things are trying to write to memory at the same time? One fails?

Yes we can't have concurrent locks as it would cause us to block. Before it would block and cause a panic, now this will cause an error instead so it should be safer. We don't really have a way around this besides getting bdk and ldk to implement async storage.

How does this work then? What if two things are trying to write to memory at the same time? One fails?

Yes we can't have concurrent locks as it would cause us to block. Before it would block and cause a panic, now this will cause an error instead, which we can properly handle, so it should be safer. We don't really have a way around this besides getting bdk and ldk to implement async storage.

I still think we can do something spawn_local & channel related to make sure that writes are happening correctly, but maybe we just wait to see how it performs without that first?

I still think we can do something spawn_local & channel related to make sure that writes are happening correctly, but maybe we just wait to see how it performs without that first?

That might be a better long term solution but for now this PR is a strict upgrade.


Source code for decoder.worker

Is the source for the decoder.worker files available?

Sorry, I don't understand what you need. Do you need the source code of decoder.worker?
It comes from tinyh264

Thank you for this information about this being based on tinyh264, that is good to know I didn't see information about this in any of the documentation. But how are you compiling this project? Can you include the build scrips or commands that you are using?

I just compiled the wasm file to base64 for convenience. Actually you don't need to do this.
h264bsd and tinyh264 both provide examples for reference.

Thank you all of this has helped, I think we can close this issue.


Tone.Sampler()  envelope attack > 1 sec?

` let sampler = new Tone.Sampler(partials, {
baseUrl: "myDir/",
onload: this.onInstrumentReady,
attack:<PHONE_NUMBER>00, // I've tried all sorts of values here (1, 10, 100, 1000...)
release: 100000 // same here
});
sampler.connect(this.audioChain.mainGain);
sampler.triggerAttackRelease(frequency, 10, Tone.now(), velocity);`
I'm trying to have the Sampler a long attack (like 4-5 seconds), but can't seem to get anything past about 1 second.

I'm not able to recreate this issue here. I hear the envelope fade in.
The issue might be related to #363 if you're using the current release and passing in a velocity value. Try using the latest build tone@next and see if it still has the issue.

Hi @tambien thanks for the quick response!
Actually, using your same fiddle, I'm totally able to reproduce the issue. Try changing the release value from 2 to 20, 200 or even 2000 - one would expect an extremely long release, but it sounds about exactly the same as the original value of 2. I'm assuming "2" is 2 seconds?
let sampler = new Tone.Sampler({
	'C4' : 'https://tonejs.github.io/examples/audio/casio/C2.mp3'
}, {
  onload: () => {
    sampler.triggerAttackRelease('C4', 2000)
	},
  attack : 1
}).toMaster()

I'm sorry I'm really not trying to extend the length of a sample.
Please try your fiddle again with a much longer sample source. I'm happy to be proven wrong of course! But I'm trying this on 20 second+ samples and not seeing the longer release / decay values.

Sorry for my confusion. I've dug into this a little more and i'm pretty sure this has to do with the fact that the sampler uses a setTargetAtTime envelope for the attack and decay portions. That envelope type is not very conducive to long, gradual changes.
I've added a "curve" attribute to Sampler which can be either 'linear' or 'exponential'. Try a 'linear' envelope for your situation. hopefully that creates the effect that you're going for.

Thanks for continuing to look into this!
I've tried this snippet and I do feel like the ramp up is more gradual, but it doesn't appear to be an exact mapping to time (seconds). This may be due to the shorter sample length.
let sampler = new Tone.Sampler({ 	'C4' : 'https://tonejs.github.io/examples/audio/casio/C2.mp3' }, { 	curve: "linear",   onload: () => {     sampler.triggerAttackRelease('C4', 2000) 	},   attack : 5 // is this supposed to be 5 seconds? }).toMaster()

It looks like the curve is behaving normally, but is limited by the duration of the sample and the math of the exponential attack curve.
If you open this demo and set the attack curve to be much larger than the sample, you'll still get the audio of the sample because the curve is exponential, most of the amplitude is towards the beginning of the curve. so even if you set it to a large number (like 1000), you'll get at about 50% of the amplitude within that first second of audio.
If you set that curve to linear you'll see that it does only rise a very small amount before the end of the 1 second sample.
Going to close again. lmk if i'm missing something.


Turning this into a shadCn template

Hey Guys,
Given that Vercel is somehow pushing shadCn on their Ai editor and stuff,
is turning this into a shadCn template in the pipeline at all.
If not, is anyone interested in collaborating on that?
cheers,
Latif

Hey @latifs thanks for considering this. We don't have any concrete plans for this, but would love to hear what this would take and definitely would appreciate a contribution to make that happen 💚

hey @latifs let's chat about collaboration.


JavaScript autocompete not working when TypeScript extension disabled

VSCode Version: 1.21.1 (1.21.1)
OS Version: macOS High Sierra 10.13.3 (17D47)

Assuming I have two files in my project: one.js and two.js
Steps to Reproduce:

Disable the builtin TypeScript and JavaScript Language Features extension
When typing in one.js

With TypeScript extension enabled it works fine

Does this issue occur when all extensions are disabled?:  No

The TypeScript and JavaScript Language Features extension is what provides TypeScript and javascript intellisense

@mjbvz okay, but how can I disable TypeScript only? I use flow in my project and I don't need TypeScript. Having TypeScript suggestions along with Flow (when I don't use TypeScript) is annoying.

What you are seeing are the JavaScript suggestion from the TypeScript and JavaScript Language Features extension. They are not TypeScript specific.
The flow extension should provide all suggestions on its own. If you don't see that, please consider opening an issue against that extension