Finrl Meta Stock_NeurIPS2018_SB3.ipynb only support DOW_30_TICKER

Describe the bug
using :
df = YahooDownloader(start_date = TRAIN_START_DATE,
end_date = TRADE_END_DATE,
ticker_list = config_tickers.DOW_30_TICKER).fetch_data()
df.to_csv('sample.csv')
df = pd.read_csv('sample.csv')
i am able to generate FeatureEngineer technical indicator
So i generate a csv file with my own stock of malaysia market in the exact format from the above sample.csv.
i got error when run this:
fe = FeatureEngineer(
use_technical_indicator=True,
tech_indicator_list = INDICATORS,
use_vix=False,
use_turbulence=False,
user_defined_feature = False)
KeyError: "None of [Index(['tic', 'date', 'boll_ub'], dtype='object')] are in the [columns]"
I attached my sample.csv for anyone who can take a look. The error persist even after disabling vix as it use yahoo downloader to download vis. How to adapt this notebook to work with my own stocks instead?
sample1try.csv

To address your issue:

Tickers: The tickers you're interested in are from the Malaysia stock market. From Yahoo Finance, some Malaysia tickers include CMSB.KL, ALCOM.KL, PPHB.KL, and PRLEXUS.KL. You can find more tickers from the Malaysia market on Yahoo Finance.
You would replace the ticker_list with, and you can add more tickers to the list:
MALAYSIA_TICKERS = ['CMSB.KL', 'ALCOM.KL', 'PPHB.KL', 'PRLEXUS.KL']

And change the line:
ticker_list = config_tickers.DOW_30_TICKER

to:
ticker_list = MALAYSIA_TICKERS

Have you actually tried MALAYSIA_TICKERS = ['CMSB.KL', 'ALCOM.KL', 'PPHB.KL', 'PRLEXUS.KL']. I have tried and it doesnt work. Anyway my problem is not the data. I have my own ticker data. When i run the sample, there is error. I fixed the problem by ensuring all ticker must have same number of rows which is silly because some ticker have missing trading days due to various reason. Eventhough i have successfully running the sample also no use. The result is the drl agent buy everyday non stop for 7months wihout selling in the out of sample results. Obviously the drl agent got something wrong eventhough i have sucessfully train the agent.

Upon investigating the issue, I found that the FeatureEngineer function expects the date format to be of type str (or object in pandas terms) rather than a pandas datetime64 type. This is likely because the function internally processes or compares the date values as strings.
To remedy this, you can convert the date column back to a string format before passing the DataFrame to FeatureEngineer. Here's how you can do it:
df['date'] = df['date'].dt.strftime('%Y-%m-%d')

This code will convert the datetime64 column back to a string format in the 'YYYY-MM-DD' style.
After this conversion, you can then use the FeatureEngineer function as intended:
FeatureEngineer(use_technical_indicator=True,
                tech_indicator_list=config.INDICATORS,
                use_vix=True,
                use_turbulence=True,
                user_defined_feature=False)

I understand that this might not be the most intuitive requirement, but it solves the problem.

Thank you so much. I will try it out and let you know the result. Thanks

Hi. I already tried your method using the above sample1try.csv and run it at FinRL-master/examples/Stock_NeurIPS2018_1_Data.ipynb
and i got this error:
KeyError: "None of [Index(['tic', 'date', 'macd'], dtype='object')] are in the [columns]"
You able to run the above excel without problem?

Hello. I encountered the following problem:
df = YahooDownloader(start_date=TRAIN_START_DATE, end_date=TEST_END_DATE, ticker_list=DOW_30_TICKER).fetch_data()
print(df)
An error appears:
[100%%*] 1 of 1 completed
Traceback (most recent call last):
File "C:\Users\LIKEP\PycharmProjects\trading analyzes\main.py", line 52, in
df = YahooDownloader(start_date=TRAIN_START_DATE, end_date=TEST_END_DATE, ticker_list=DOW_30_TICKER).fetch_data()
File "C:\Users\LIKEP\PycharmProjects\trading analys\venv\lib\site-packages\finrl\finrl_meta\preprocessor\yahoodownloader.py", line 51, in fetch_data
data_df = data_df.append(temp_df)
File "C:\Users\LIKEP\PycharmProjects\trading analys\venv\lib\site-packages\pandas\core\generic.py", line 6293, in getattr
return object.getattribute(self, name)
AttributeError: 'DataFrame' object has no attribute 'append'


[CT-3086] [Feature] Add "name" to Postgres index configuration

Is this your first time submitting a feature request?

[X] I have read the expectations for open source contributors
[X] I have searched the existing issues, and I could not find an existing issue for this feature
[X] I am requesting a straightforward extension of existing dbt functionality, rather than a Big Idea better suited to a discussion

Describe the feature
Can we add the "name" config to Postgres index configuration in DBT? I have been using it but when it creates a hashed index name, it fails when I try and combine this with hasura. I would like to override the default index config to supply my own unique name for an index.
Describe alternatives you've considered
Writing a script that runs after DBT runs to rename the index
Who will this benefit?
everyone using Postgres and graphql
Are you interested in contributing this feature?
I could try with some direction
Anything else?
No response

@bdewilde in general, we prefer to limit complexity wherever possible. But we're also trying to enable & empower where it's practical. So often a tricky set of trade-offs!
We'd be open to consider specifying the "nulls [not] distinct" and "where" options, etc. if you are willing to open up separate issue(s) for them (including describing any workarounds you've utilized instead).
That said, expanding configuration of indexes in dbt-postgres isn't very high on our list and we might not be able to assess those feature requests in a timely manner.


Site returns console info to client

When I try to load up the website, instead of showing a webpage, it shows info about the site, without actually showing what is meant to be shown.

That's the home route of the site. Its not a website, but rather an api. To learn more please check the documentation - https://docs.saavn.me/

Thanks. I assumed that, but that was the site shown on GitHub, so wanted to make sure it wasn't a bug. Thanks!


Removes templates deprecated by campaign webSignup

What's this PR do?
Removes deprecated templates, now that Conversations uses the webSignup template defined on a campaign config for signup confirmation messages (deployed to prod in https://github.com/DoSomething/gambit-conversations/releases/tag/3.8.0)
How should this be reviewed?
👀
Checklist

[x] Tested on staging.

Photo post: https://gambit-admin-staging.herokuapp.com/topics/275xvSusaYe88k0Aqiu606
Text post: https://gambit-admin-staging.herokuapp.com/topics/1eNLcffhU0oMUAqyYsiI2i

Got this locally:
//<PHONE_NUMBER>0551
// http://localhost:5000/v1/topics/275xvSusaYe88k0Aqiu606?apiKey=totallysecret

{
  "error": {
    "code": 500,
    "message": "Cannot read property 'broadcastable' of undefined"
  }
}

//<PHONE_NUMBER>0701
// http://localhost:5000/v1/topics/1eNLcffhU0oMUAqyYsiI2i?apiKey=totallysecret

{
  "error": {
    "code": 500,
    "message": "Cannot read property 'broadcastable' of undefined"
  }
}

There's something strange going on with a draft entry, investigating. This error is happening on master, so I'll open a branch to fix.

I've found what was wrong -- I'm working locally in a branch that's adding support to adding support to add one of the new autoReplyTransition, photoPostTransition, and textPostTransition types as a response reference on the defaultTopicTrigger entry. A GET /topics/:id request  will end up querying for defaultTopicTriggers to include the list of triggers that reference the topic... which is why the error occurs everywhere and why it's time to ditch all of this nested query stuff that exists for the sake of Gambit Admin (a la #1090)


Add IPNS to the datasets

Have a unique CID for each published dataset.

IPNS is not production ready jet for something like this. For now, something like DNSLink should do it!


Synonyms for exporting internal objects

If you add synonyms to the library for exporting internal objects, then everything should be normalized:
package rueidis

import "github.com/rueian/rueidis/internal/cmds"

// Exported synonyms for command types.
type (
    Completed = cmds.Completed
    Cacheable = cmds.Cacheable
    Builder   = cmds.Builder
)

The error with VSCode with the "substitution" of cmds.Completed will be removed.
It will be possible to create functions that return prepared Redis commands:type Object struct {
    ID        string
    Datetime time.Time
    Message string
}

func (o Object) SaveCmd(client rueidis.Client, id string) rueidis.Completed {
    return client.B().
        Hset().
        Key("out:interval:"+o.ID).
        FieldValue().
        FieldValue("datetime", fmt.Sprint(o.Datetime.Unix())).
        FieldValue("message", o.Message).
        Build()
}

Can be used in the interface descriptiontype Client interface {
    B() Builder
    Do(ctx context.Context, cmd Completed) (resp RedisResult)
    DoMulti(ctx context.Context, multi ...Completed) (resp []RedisResult)
    DoCache(ctx context.Context, cmd Cacheable, ttl time.Duration) (resp RedisResult)
    DoMultiCache(ctx context.Context, multi ...CacheableTTL) (resp []RedisResult)
    Receive(ctx context.Context, subscribe Completed, fn func(msg PubSubMessage)) error
    Dedicated(fn func(DedicatedClient) error) (err error)
    Dedicate() (client DedicatedClient, cancel func())
    Nodes() map[string]Client
    Close()
}

Hi @mdigger,
Thank you for your suggestion and the PR #122.
Moving the cmds.Completed out of github.com/rueian/rueidis/internal is what I am considering since my first commit to this library.
But even today, I still don't figure out other ways to make the following code fail to be compiled unless I make the cmds.Completed live in internal packge:
client.Do(ctx, cmds.Completed{})
// or
client.Do(ctx, cmds.Completed(c.B().Set().Key("I am not a completed cmd")))

I think letting the above code won't pass the compiler is quite important. This could prevent users to make some mistakes.

Thank you for the detailed answer!
It is possible that I am wrong, but it seems to me that using inaccessible types in the library gives more problems than solutions.

You are right. The inaccessible type indeed creates some problems.
In addition to the disability you just mentioned to have prepared command function, the inaccessible type in the Client interface also prevent users from hooking or mocking it. That is why there are github.com/rueian/rueidis/mock and github.com/rueian/rueidis/rueidishook provided in this library.


Move AdlsIngestionApp details (app ID and client secret) to "AzureDataLakeStorageV1" section of the config

Currently, this config is provided through environment variables (function app variables) and set through the release definition. However, moving this to main config, especially to the section related to AzureDataLakeStorageV1 keeps things that are relevant together, making it more difficult to forget updates on correlated stuff.

Closed at #42


can't install on codium

ext install kraih.javascript-tmpl-support
this plugin doesn't exist in codium

I don't know what that means. This is a VSCode plugin. https://marketplace.visualstudio.com/items?itemName=kraih.javascript-tmpl-support

Is that referring to this? https://github.com/VSCodium/vscodium#why-does-this-exist I've never seen it before but it is interesting regardless

Published.


fix: go.mod has post -v0 path.

Either we release a v2.x version or we remove this from the module path. Try as I might it will not budge.
references:

https://stackoverflow.com/questions/52050146/go-build-keeps-complaining-that-go-mod-has-post-v0-module-path.
https://go.dev/doc/modules/version-numbers
https://go.dev/doc/modules/release-workflow

Closing the PR since with the 2.0.0 release this is fixed:
 ~/Projects/work/flb/cloud/ [yaml-v2] go get github.com/calyptia/go-fluentbit-config/v2
go: downloading github.com/calyptia/go-fluentbit-config/v2 v2.0.0
go: upgraded github.com/calyptia/go-fluentbit-config/v2 v2.0.0-20230330110656-0f7b7a73cb42 => v2.0.0


Error: Couldn't resolve the package 'flutter_platform_dropdown'

Invalid depfile: /Users/fgret//StudioProjects/alimFreight_accountant_app/desktop/.dart_tool/flutter_build/55fe2ff0dafbb2547e78b9097d0b8fe8/kernel_snapshot.d
Invalid depfile: /Users/fgret/StudioProjects/alimFreight_accountant_app/desktop/.dart_tool/flutter_build/55fe2ff0dafbb2547e78b9097d0b8fe8/kernel_snapshot.d
Error: Couldn't resolve the package 'flutter_platform_dropdown' in 'package:flutter_platform_dropdown/flutter_platform_dropdown.dart'.
../core/lib/presentation/ui_kit/widgets/editable_text_field.dart:3:8: Error: Not found: 'package:flutter_platform_dropdown/flutter_platform_dropdown.dart'
import 'package:flutter_platform_dropdown/flutter_platform_dropdown.dart';

Hello @AlimFreight,
Currently this package is work only in android and iOS not desktop and other platforms,
i am working on it, and it will be update soon
Thank you for the feedback :)


S3 etcd backup endpoint should be validated on downstream cluster, not Rancher

What kind of request is this (question/bug/enhancement/feature request):
Bug
Steps to reproduce (least amount of steps as possible):

Create a custom downstream cluster. The cluster needs to be in a network with access to a private DNS entry for a valid S3 endpoint.
Edit the cluster, select S3 for etcd Snapshot Backup Target. Enter the relevant information for the bucket, folder, keys, as well as the private DNS entry which can be resolved by the downstream cluster, but not the Rancher control plane. For example:

Result:

In Rancher logs:
2020/09/03 20:31:10 [ERROR] Unknown error: Unable to validate S3 backup target configration: Head https://my-private-s3.mycompany.com/myetcdbackups/: dial tcp: lookup my-private-s3.mycompany.com on <IP_ADDRESS>:53: no such host

Other details that may be helpful:
Validation of the s3 endpoint should be done on the downstream cluster, not on the Rancher control plane. They may not both be in the same network infrastructure and it's possible the Rancher control plane may not have access to the S3 endpoint needed by the downstream cluster for backups.
See also https://github.com/rancher/rancher/issues/22578 which involved an issue with how email notifiers had a similar issue.
Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI):
v.2.4.7-rc2
Installation option (single install/HA):
HA

Cluster information

Cluster type (Hosted/Infrastructure Provider/Custom/Imported):
Custom
Machine type (cloud/VM/metal) and specifications (CPU/memory):
t3a.medium (2vCPU, 4GB RAM)
Kubernetes version (use kubectl version):

> kubectl version
Client Version: version.Info{Major:"1", Minor:"17", GitVersion:"v1.17.6", GitCommit:"d32e40e20d167e103faf894261614c5b45c44198", GitTreeState:"clean", BuildDate:"2020-05-20T13:16:24Z", GoVersion:"go1.13.9", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"18", GitVersion:"v1.18.8+k3s1", GitCommit:"6b595318666804506c19cfc3d3d228423d38fab1", GitTreeState:"clean", BuildDate:"2020-08-14T20:42:29Z", GoVersion:"go1.13.15", Compiler:"gc", Platform:"linux/amd64"}

Docker version (use docker version):

# docker version
Client:
 Version:           18.06.3-ce
 API version:       1.38
 Go version:        go1.10.3
 Git commit:        d7080c1
 Built:             Wed Feb 20 02:28:10 2019
 OS/Arch:           linux/amd64
 Experimental:      false

Server:
 Engine:
  Version:          18.06.3-ce
  API version:      1.38 (minimum version 1.12)
  Go version:       go1.10.3
  Git commit:       d7080c1
  Built:            Wed Feb 20 02:26:34 2019
  OS/Arch:          linux/amd64
  Experimental:     false

gzrancher/rancher#12124

Verified on 2.4-head - commit id: f210f54bd, 2.5-head - commit id: b722847 and master-head commit id: e90c4018

Bring up minio - using AWS instance with priv IP.
Link an internal LB to this instance. And add this minio details as S3 details to a custom cluster (AWS nodes, same VPC as the minio server 's instace) created in a rancher server (on a linode machine)
No error seen in the Rancher logs
User is able to take snapshots and see it get saved on Minio


Augment Semantic Tokens with ComponentTokens

We might only want to component tokens on our components in Figma and Coded Comps.
This will make our system  more concise, reliable and maintainable.
Semantic token should remain, but be used as aliases only.

As discussed today, we distanced us again from the approach described here. We might still want to do this in the future, but not now. Therefor this issue was closed.


Funnel all writing through one function and add logging.

Are we closing this one?

Yes, superseeded by #9.


Remove unused method: getCurrentPage()

This is method is intentionally exposed to enable users to get the carousel page state programatically from outside the library.
It should be added to the docs. (I will try to get around to do that.)


Change "Dərsiliyi Götür" to "Dərsliyə Başla"

Change Dərsiliyi Götür to Dərsliyə Başla

I disagree with this merely from "marketing" perspective. If you currently go to the preview, the button and the text do no clash in terms of words: https://deploy-preview-24--az-reactjs.netlify.com
However, if we change it to "başla," the two buttons will clash. Maybe "dərsliyi oxu?"

I disagree with this merely from "marketing" perspective. If you currently go to the preview, the button and the text do no clash in terms of words: https://deploy-preview-24--az-reactjs.netlify.com
However, if we change it to "başla," the two buttons will clash. Maybe "dərsliyi oxu"?

yes it is also possible as "dərsliyi oxu", but not "Dərsliyi Götür"

Then I say lets do Dersliyi Oxu because then we will have Bashlamag and Dersliyi Oxu instead of Bashlamag and Dersliyi Bashla. If you agree, please make the changes and push.

updated to "Dərsliyi Oxu"

Wasn’t there two places for that button? One at the top and another one at the bottom?


[EASI-2816] Search Changes in Model Plan Hierarchy by ID

EASI-2816
This query will fetch all changes related to a model plan by searching with a model plan ID.
It supports:

Date range
Returns results in reverse chronological order
Paging (via limit and offset search parameters)

Changes and Description

Added a new endpoint named SearchChangeTableByModelPlanIDAndDateRange

How to test this change
Run the endpoint SearchChangeTableByModelPlanIDAndDateRange. Verify that it functions and returns the proper data.

PR Author Review Checklist

[ ] Met the ticket's acceptance criteria, or will meet them in a subsequent PR.
[ ] Added or updated tests for backend resolvers or other functions as needed.
[ ] Added or updated client tests for new components, parent components, functions, or e2e tests as necessary.
[ ] Updated the Postman Collection if necessary.

PR Reviewer Guidelines

It's best to pull the branch locally and test it, rather than just looking at the code online!
Check that all code is adequately covered by tests - if it isn't feel free to suggest the addition of tests.
Always make comments, even if it's minor, or if you don't understand why something was done.

@OddTomBrooks, I think that gql gen just needs to be run again for linting to pass. I can approve again  if needed after it is updated


Spring cleaning

Codecov Report

Merging #239 (fef0e39) into dev (8a01903) will decrease coverage by 3.34%.
The diff coverage is 31.11%.

@@            Coverage Diff             @@
##              dev     #239      +/-   ##
==========================================
- Coverage   57.63%   54.29%   -3.35%
==========================================
  Files         116      130      +14
  Lines        2795     3144     +349
  Branches      373      423      +50
==========================================
+ Hits         1611     1707      +96
- Misses       1177     1430     +253
  Partials        7        7

Impacted Files
Coverage Δ

packages/server/src/app.module.ts
0.00% <0.00%> (ø)

packages/server/src/app.roles.ts
100.00% <ø> (ø)

...ges/server/src/attendance/attendance.controller.ts
0.00% <0.00%> (ø)

...ackages/server/src/attendance/attendance.module.ts
0.00% <0.00%> (ø)

...ckages/server/src/attendance/attendance.service.ts
0.00% <0.00%> (ø)

packages/server/src/invoice/invoice.controller.ts
71.42% <0.00%> (+8.92%)
:arrow_up:

...ages/server/src/membership/membership.constants.ts
0.00% <0.00%> (ø)

...ges/server/src/membership/membership.controller.ts
0.00% <0.00%> (ø)

...ackages/server/src/membership/membership.module.ts
0.00% <0.00%> (ø)

...ckages/server/src/membership/membership.service.ts
0.00% <0.00%> (ø)

... and 44 more

Continue to review full report at Codecov.

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


Finalize 3D Asset editor

Make the 3D asset creator form dynamic, in the WP Front end.

3D Asset editor has now all the basic components needed to function. It needs to be connected with the back-end.
Commit dfd3c220ceb6c3154c58da5358e0ca438318a58f closes this.


extract cluster-type crate

Problem
solana_sdk::genesis_config::ClusterType imposes a solana_sdk dep on solana_metrics. The genesis_config module is quite big when you include the other modules it pulls in, while solana_metrics only needs the ClusterType enum.
Summary of Changes

Move ClusterType into its own crate and re-export with deprecation notice

✅


[docopt] build failure

Host Environment

OS: Windows
Compiler: clang-cl

To Reproduce
Steps to reproduce the behavior:
./vcpkg install docopt
Failure logs
https://ci.appveyor.com/project/acgetchell/cdt-test#L4750
Additional context
Full build steps:
https://github.com/acgetchell/CDT-test/blob/master/.appveyor.yml
Builds successfully on Linux/gcc:
https://travis-ci.com/acgetchell/CDT-test/jobs/247251406
With:
https://github.com/acgetchell/CDT-test/blob/master/.travis.yml

@acgetchell In docopt's CMakeLists.txt line 37-54:
if(MSVC OR XCODE)
    # MSVC requires __declspec() attributes, which are achieved via the
    # DOCOPT_DLL and DOCOPT_EXPORTS macros below. Since those macros are only
    # defined when building a shared library, we must build the shared and
    # static libraries completely separately.
    # Xcode does not support libraries with only object files as sources.
    # See https://cmake.org/cmake/help/v3.0/command/add_library.html?highlight=add_library
    add_library(docopt SHARED ${docopt_SOURCES} ${docopt_HEADERS})
    add_library(docopt_s STATIC ${docopt_SOURCES} ${docopt_HEADERS})
else()
    # If not using MSVC or Xcode, we will create an intermediate object target
    # to avoid compiling the source code twice.
    add_library(docopt_o OBJECT ${docopt_SOURCES} ${docopt_HEADERS})
    set_target_properties(docopt_o PROPERTIES POSITION_INDEPENDENT_CODE TRUE)

    add_library(docopt SHARED $<TARGET_OBJECTS:docopt_o>)
    add_library(docopt_s STATIC $<TARGET_OBJECTS:docopt_o>)
endif()

line 68-71:
if(NOT MSVC)
	set_target_properties(docopt PROPERTIES OUTPUT_NAME docopt)
	set_target_properties(docopt_s PROPERTIES OUTPUT_NAME docopt)
endif()

So, I think it's by design.

Okay, thanks @JackBoosY ! I'll remember to look at package CMakeLists.txt next time!


Editor focus erratic and UI issues with "activation follows mouse" (x-mouse) on Windows

Brackets Version: 1.5.0-16538
OS: Windows 7 Ultimate x64 SP1, English
Extensions: I have confirmed the issue exists when loading with "Reload without extensions" (I'm using PHP-oriented extensions only)
Previous versions: This is my first version of the product, so I cannot tell if it was occuring on previous versions.
Errors: none when the issue occurs. The only error I have in the console is "PreferencesManager.convertPreferences() has been deprecated." -- Please look to the bottom of the report for full contents of this error.
ISSUE: interface issues (title bar flashing), losing focus with the editor.
NOTE: the issue is intermittent - sometimes it'll happen, sometimes it won't.

--------------------------------
STEPS TO REPRODUCE:
METHOD 1

Enable "activation follows mouse" (with 0ms delay) in system preferences (a registry edit may be required) and log out/in to enable. (this only needs to happen once per user account)
Launch Brackets
Open a file to edit
Alt+Tab away from the editor
Alt+Tab back to the editor
(optional) Start typing

What I expected to happen: I expected the editor to receive focus in its window, and I expected to be able to continue writing code from that point on. (I do this to write a bit, refresh page/reload code in test suite, and then continue)
What actually happens: the editor goes all "wonky", the title bar is flashing like crazy, and the keyboard focus is actually with the window that was just active (browser, terminal etc.). Sometimes the previous window will "flash through" the Brackets interface, alerting me to the error.
Sometimes I'll be able to type one letter in (that's why there's an optional step) before this happens, and sometimes it'll happen right away.

METHOD 2

Enable "activation follows mouse" (with 0ms delay) in system preferences (a registry edit may be required) and log out/in to enable. (this only needs to happen once per user account)
Launch Brackets
Open a file to edit
Type something in
Hit Ctrl+S to save
(optional) Start typing

What I expected to happen: I expected to be able to continue writing code from that point on.
What actually happens: the editor goes all "wonky", the title bar is flashing like crazy, and the keyboard focus is with the window directly below the Brackets interface (or with the desktop otherwise). Sometimes the window below will "flash through" the Brackets interface, alerting me to the error.
Sometimes I'll be able to type one letter in (that's why there's an optional step) before this happens, and sometimes it'll happen right away.

Steps to recover:
Method 1: Alt+Tab out of the editor and back into the editor.
Method 2: click within the editor window / editor title bar (not where the underlying window is "flashing through").

--------------------------------
Comments:
Note that I do realize that this issue, if related directly to my mouse/focus settings, probably affects only a small portion of users. (I have not tested if the issue also happens with the X-Mouse program)
It may be indicative of a deeper underlying issue within the window environment, so I felt it was better if I reported it.
Please note that this issue is preventing me from editing my code (enough to have me actually register an account on github and file a bug report!), and if it's not fixed, I will be forced to move on to other code editing software (I am currently test-driving a number of editors). The bug is absolutely devastating to my coding process.

Full error information (see top for explanation):
PreferencesManager.convertPreferences() has been deprecated. Please upgrade to the current Preferences system (https://github.com/adobe/brackets/wiki/Preferences-System#conversion-from-the-pre-36-preferences-system).
at Object.convertPreferences (/preferences/PreferencesManager.js:325:28)
at Object.eval (/utils/Resizer.js:548:24)

Note that I do realize that this issue, if related directly to my mouse/focus settings, probably affects only a small portion of users. (I have not tested if the issue also happens with the X-Mouse program)
It may be indicative of a deeper underlying issue within the window environment, so I felt it was better if I reported it.
Please note that this issue is preventing me from editing my code (enough to have me actually register an account on github and file a bug report!), and if it's not fixed, I will be forced to move on to other code editing software (I am currently test-driving a number of editors). The bug is absolutely devastating to my coding process.

Upon further investigation it seems that this issue -> https://github.com/adobe/brackets/issues/11131 (Brackets is losing keyboard focus) <-- could very well be related and indicate a bigger problem with window switching in general.

I would like to note that I am at this point actively using a different editor, but I am available to provide reproductions, videos, any required debug information etc. and to generally provide requested feedback in order to assist in debugging this issue.


Undefined constant 'INTL_IDNA_VARIANT_UTS46'

Describe the bug
When attempting to change a setting in the vmsAcars settings via the site admin panel........ an error is thrown
Undefined constant 'INTL_IDNA_VARIANT_UTS46'
Version
phpVMS version 7.0.0 -dev
To Reproduce
Steps to reproduce the behavior:

Go to 'admin' >>>> 'addons'>>>>'VMSacars'

Click on : Fare entry is required
Entries for the fares are required to be entered .......checkbox

Scroll down to 'Stacktrace error page line 141'

See error

Expected behavior
The setting change in the vmsAcars module is applied.
Screenshots
Attached Here

Hmm I will have to research this - this looks like a problem with the PHP install, maybe a missing extension

Thanks Nabeel
I have'nt yet checked if the setting change does get applied to the pireps( The checkbox ticked or unticked does happen though)

This looks like it could be related to an older version of the intl extension. Can you check with your host that the latest is installed? Can you provide your PHP version and a also the version of that ext?

This also looks like something with parsing the domain. Is the APP_URL set in your env.php and doesn't have any non-ASCII characters?

On Discord, if you can DM me that value, I can see if I can duplicate it tomorrow.

Sure..will get on it and try and get the details

Ok, that's an older INTL version. Trying to find a fix for that but can you see if you can get intl updated?

I'm on a shared host, not sure I can manage...going to try

Though I'm adding a polyfill now which should address this

Ah great

Yeah, that would a problem on shared hosts. Wait for the pull request to be merged into the latest dev and that should show up in the Discord as being ready, then try that one out.


How to set RAKE parameters?

How do we tune the Rake parameters, as in this example:
https://www.airpair.com/nlp/keyword-extraction-tutorial
So for example:
rake_object = rake.Rake("SmartStoplist.txt", 5, 3, 4)
Each word has at least 5 characters
Each phrase has at most 3 words
Each keyword appears in the text at least 4 times
These parameters change as a function of the corpus

This is how this package works.
from rake_nltk import Rake

r = Rake() # Uses stopwords for english from NLTK, and all puntuation characters.

r = Rake(<language>) # To use it in a specific language supported by nltk.

# If you want to provide your own set of stop words and punctuations to
# r = Rake(<list of stopwords>, <string of puntuations to ignore>)

r.extract_keywords_from_text(<text to process>)

r.get_ranked_phrases() # To get keyword phrases ranked highest to lowest.

Here we do not have tuning of parameters yet. You can raise a feature request and I will try to get to it.


Docker Image not updating public ip address as per interval configuration

Describe the bug
The docker image does not update the public ip(at least for cloudflare) as per the interval argument in the configuration file
The frequency of this issue

[x ] This issue is easy to reproduce.

Steps to Reproduce
 docker run -d --name godns --restart=always -v /home/<user>/godns/config.json:/config.json timothyye/godns:latest
the docker container runs correctly one time, managing to update the ip address
docker logs (on start)
time="2024-02-05T09:28:20Z" level=info msg="Creating DNS handler with provider: Cloudflare"
time="2024-02-05T09:28:20Z" level=info msg="GoDNS started, starting the DNS manager..."
time="2024-02-05T09:28:20Z" level=info msg="Checking IP for domain domain.com"
time="2024-02-05T09:28:22Z" level=info msg="Querying records with type: A"
time="2024-02-05T09:28:22Z" level=info msg="Record OK: domain.com - ip"
time="2024-02-05T09:28:22Z" level=info msg="Record OK: www.domain.com - ip"
time="2024-02-05T09:28:22Z" level=info msg="Checking IP for domain domain.com"
time="2024-02-05T09:28:23Z" level=info msg="Querying records with type: A"
time="2024-02-05T09:28:24Z" level=info msg="Record OK: domain.com -  ip"
time="2024-02-05T09:28:24Z" level=info msg="Checking IP for domain domain.com"
time="2024-02-05T09:28:25Z" level=info msg="Querying records with type: A"
time="2024-02-05T09:28:26Z" level=info msg="IP mismatch: Current( ip) vs Cloudflare(<IP_ADDRESS>)" --> manages to change correctly
time="2024-02-05T09:28:26Z" level=info msg="Record updated: blogdomain.com -  ip"
time="2024-02-05T09:28:26Z" level=info msg="Record OK: domain.com -  ip"

after 10mins(expected to be 5mins), there is no change in the docker logs output nor any change in the cloudflare ip addr(i set it back to a wrong ip)
Your GoDNS configuration
{
  "provider": "Cloudflare",
  "email": "mail.com",
  "password": "",
  "domains": [
    {
      "domain_name": "domain.com",
      "sub_domains": ["www", "@", "blog"]
    }
  ],
  "resolver": "<IP_ADDRESS>",
  "ip_urls": ["https://api.seeip.org/","https://ifconfig.me/ip"],
  "ip_type": "IPv4",
  "interval": 30,
  "socks5_proxy": "",
  "proxied": true
}```

**Expected behavior**
After 5 mins, a wrongly set ip record on blog.domain.com should be updated to reflect the correct ip and corresponding logs should be seen in the docker logs output

**Screenshots**
If applicable, add screenshots to help explain your problem.

**Environment (please complete the following information):**
 - DNS Provider: cloudflare
 - OS: ubuntu server
 - docker version: 24.0.5
 - image version: latest(as per writing)

**Additional context**
Add any other context about the problem here.

Update: the same issue occurs with systemctl as well. Am i misunderstanding something about the configuration? xd

Ah...I misunderstood the config

https://github.com/TimothyYe/godns?tab=readme-ov-file#display-debug-info
If you enable the debug_info feature, you will get more detailed debug information.


how to train a model that can generate midi file which can generate complex tone or harmony?

After I  used melody_rnn_generate commands to generate midi files.  I played the generated midi files, finding that the music sounds so machiney that without any harmony, complex tone, or etc.
The generated music is always monophonic. This always annoys me so much.
I want to ask how to adjust the model or any necessary step i need to take,if I'd like the model can generate melodious music!
Thanks for any help~~~

@cghawthorne @markmcd @astromme @notjosh @proppy

I'd suggest taking a look at Performance RNN. It can generate polyphonic music with expressive timing and dynamics. You might also be interested in our web demo of that model.

Thank you for your suggestion~ I will have a try~

Thank you for your suggestion~ I will have a try~ @cghawthorne


enabled flag

Problem Statement
I want to enable Sentry only in production builds
In other Sentry SDKs there is an enabled option
Solution Brainstorm
No response
Are you willing to submit a PR?
None

which other enable option do you refer to in other SDKs?

In JS SDK there's an enabled flag:

Gotcha, I will discuss this internally

What exactly are you trying to accomplish / expecting, just not send the event in debug builds?

What exactly are you trying to accomplish / expecting, just not send the event in debug builds?

Exactly! When developing, I will inevitably develop some runtime errors, and I don't want Sentry to be populated with these events, as they can become noise when debugging actual production issues.
For instance, we are notified in Slack for each new issue.

Only setting the DSN in production should do the trick?

either setting dsn in prod or using beforeSend, both work - whatever you prefer
if you want to minimize the overhead in general you could conditionally use SentryFlutter.init only on prod

We've decided not to go ahead with this, please use either beforeSend, empty dsn in debug or conditionally execute SentryFlutter.init only on prod


Introduce our new awesome logo

It's owl gonna change! Sophie has worked on a logo for SimpSpector, given @DavidBadura's idea of having an owl as a mascott.
Although losing a bit of our inspector gadget identity, we had decided using the gadget, since every project needs an animal mascott and the artwork is really great. So, let's use it, shall we?
Todos I see here are:

integrate in README.md / documentation (as soon as we have one)
integrate in layout: maybe the large version only on the dashboard and perhaps a small one in the menu bar?

+1 for the owl. And thanks for the great pun.
I think your suggestions are a good start. We'll feel , where to put the owl, when the time comes.

Close this issue, create more concrete ones?


fix: stuck on person screen

Fix an issue where the user can end up stuck on the Person Credential screen after the worklfow is completed. When this happens the user may not know what to do next.

You might have to install git hooks with npx<EMAIL_ADDRESS>install in the root of the project. Looks like the linter and tsc step weren't run

Ok, ready now @bryce-mcmath. I needed to make the change in two places.


Placeholder width cutting off text

We're having to patch select2 like so to get our placeholder text to appear properly.

ln 345: width: auto !important;

This happens when the new width option is either not provided or is set to 'element'. The placeholder gets an inline css width set, which mysteriously goes away and never comes back when the input is interacted with.

I am also seeing this issue even when the element is active and shown. It sets the inner select2-search__field to like 35px for me. My temp solution was a modified version of @jgeurts solution
.select2-container .select2-search__field {
width: 100% !important;
}
This works for version 4.0.0.

The above didn't work for me, I had to add the containing <li> as well:
    .select2-container .select2-search,
    .select2-container .select2-search__field {
        width: 100% !important;
    }

The problem is now the search field drops to the next line after you've added an element...

I take that back, the whole multi select is just a mess for me on closer inspection. Overflows, dropdown is partial covered. I am guessing some other styles are breaking it, but nothing jumps out. I only have generic reset styles applying. I will strip the CSS I have back to see what is playing with it.

Ran into the same issue by setting width of multiselect to 100%
This css  fixed the issue
.select2-container .select2-search__field:not([placeholder = '']) {
width: 100%!important;
}
The css will set input width to 100% only if placeholder attribute value is not empty.

I found the following way for my multiselect with hidden input:
.select2-container.select2-container-multi .select2-choices .select2-search-field:first-child {
     width: 100% !important;
}

it sets 100% only in case of there is no choises.

I fixed it by setting input field to width: auto;
copy pasted the selectors via G-inspector
.select2-container--krajee .select2-selection--multiple .select2-search--inline .select2-search__field {
    width: auto !important;
}

Seems to have no effect on other css stuff regarding select2
cool widget, lots of hassle

I'm using Select2 4.0.0 & here's my css fix to show the full placeholder and keep the height of the select wrapper. Please have a try:
.your-custom-context-selector .select2-container .select2-search__field:not([placeholder='']) {
  width: 100% !important;
  margin-right: 100000px;
}

I advise you to set a custom selector depending your needs.
Cheers

The jsfiddle
https://jsfiddle.net/shudhpandit/0h0qwtor/

I have a simple setup but i'm experencing the same error as in @ShudhPandit 's jsfiddle.

Still getting this in Ajax mode, it seems to manually set the placeholder to 100px. I am getting this on 4.0.2 but I have checked 4.0.3 commits and the fixes there shouldn't effect this so it seems this problem is still there

Had same problem with placeholder being cut off when form is shown with Ajax.  I'm on gem version 4.0.3.  @kankerlijer's solution worked for me.

I'm also getting the elements via Ajax, what I've done to solve this is delay calling the select2() by wrapping selector in a function and setTimeout with a 1ms delay.

@fro solution worked for me. Thanks!

@fro I couldn't add a custom class, unfortunately, however specifying it more did the trick without breaking other markups (so far ;-))
.select2-container .select2-selection.select2-selection--multiple .select2-search.select2-search--inline .select2-search__field:not([placeholder='']) {
  width: 100% !important;
  margin-right: 100000px;
}

/**
 * fix bug width input search
 */
$(".select2-selection__rendered").each(function (index, value) {
  var li = $(value).find('.select2-selection__choice');
  if($(li).length <= 0){
    var inputSearch = $(value).find('.select2-search__field');
    $(inputSearch).css('width', 'auto');
  }
});

This is closed even though the problem still exists?

This seems to fix the issue for me:
.select2-selection--multiple .select2-search--inline .select2-search__field {
    width: auto !important;
}

But I'd ask the maintainers to either reopen this issue, or point us all to look at a new issue.

Hi all !
Thanks a lot for your contribution !
It would be great if someone could open a new issue with an exhaustive presentation of the case.
Best,
James

This seems to fix the issue for me:
.select2-selection--multiple .select2-search--inline .select2-search__field {
width: auto !important;
}
It was resolved for me.

This seems to fix the issue for me:

Doesn't work if placeholder is too long.
So this is my fix:
Search.prototype.resizeSearch = function () {
  this.$search.css('width', '25px');

  var width = '';
  var widthParent = '';

  if (this.$search.attr('placeholder') !== '') {
    width = '100%';
    widthParent = '100%';
  } else {
    var minimumWidth = this.$search.val().length + 1;
    width = (minimumWidth * 0.75) + 'em';
    widthParent = 'auto';
  }

  this.$search.css('width', width);
  this.$search.closest('.select2-search--inline').css('width', widthParent);
};

@bews That fixed it for me. Thanks a ton!

Thanks for your feedback @eayin2, if this fixes the issue for more people it would be good to know.
Thanks for the patch @bews

@bews solution fixed it for me too.

Ok, I'll reopen this issue since it appears to still be a problem.  @bews have you submitted a PR to the repo?

@alexweissman I am still on v4.0.3. On this version, @bews solution fixed the problem. We can't upgrade our projects to v4.0.4 yet.

@alexweissman it's already here: https://github.com/select2/select2/pull/4898

To everyone in this thread:
There is a more recent version of this issue, #3817, which is a more focused discussion and has more up-to-date information on this bug.
So, I am closing this issue as per @jpic's suggestion.  Please subscribe to #3817 and we can continue the conversation there.


Fix #1583: Add JDK8 java.lang.Integer methods.

Review by @sjrd

That's all.

Updated.

That's all.

Updated.

LGTM after the space fix (which, in theory, won't pass scalastyle).

Refer to this link for build results (access rights to CI server needed):
https://scala-webapps.epfl.ch/jenkins/job/scalajs-pr/1912/
https://scala-webapps.epfl.ch/jenkins/job/scalajs-matrix-build/2466/
Test FAILed.

Refer to this link for build results (access rights to CI server needed):
https://scala-webapps.epfl.ch/jenkins/job/scalajs-pr/1915/
https://scala-webapps.epfl.ch/jenkins/job/scalajs-matrix-build/2469/
Test FAILed.

Refer to this link for build results (access rights to CI server needed):
https://scala-webapps.epfl.ch/jenkins/job/scalajs-pr/1911/
https://scala-webapps.epfl.ch/jenkins/job/scalajs-matrix-build/2465/
Test FAILed.

Refer to this link for build results (access rights to CI server needed):
https://scala-webapps.epfl.ch/jenkins/job/scalajs-pr/1917/
https://scala-webapps.epfl.ch/jenkins/job/scalajs-matrix-build/2471/
Test FAILed.

Refer to this link for build results (access rights to CI server needed):
https://scala-webapps.epfl.ch/jenkins/job/scalajs-pr/1918/
https://scala-webapps.epfl.ch/jenkins/job/scalajs-matrix-build/2472/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://scala-webapps.epfl.ch/jenkins/job/scalajs-pr/1919/
https://scala-webapps.epfl.ch/jenkins/job/scalajs-matrix-build/2473/
Test PASSed.

The commit is erroneously attributed to me.

Updated author of commit

LGTM

Refer to this link for build results (access rights to CI server needed):
https://scala-webapps.epfl.ch/jenkins/job/scalajs-pr/1921/
https://scala-webapps.epfl.ch/jenkins/job/scalajs-matrix-build/2476/
Test PASSed.


\ref{} is detected as end of a sentence

When I use \ref{}, the command was detected as end of a sentence:

It is expected to detect an error in sentences such as line 199, but errors outed in line 200 and 201.
\ref{} command is used for cross-reference frequently in academic writing.
In fact, such structure of sentences were contained a lot in my file, so the output were filled with these errors.

@atsuyaw
Thank you for reporting the issue. I confirmed that it is a bug.
Sorry for the delay in addressing the other reported issues 😥 .
We will do our best to fix it.

How to reproduce
Environment

Node.js v14.17.0
textlint-plugin-latex2e v1.1.3
textlint v12.0.0
textlint-rule-ja-no-mixed-period v2.1.1

Procedure
$ mkdir textlint-latex2e-issue168 && cd $_
$ npm init -y
$ npm install --save \
<EMAIL_ADDRESS>\
<EMAIL_ADDRESS>\
<EMAIL_ADDRESS>$ cat << EOF > sample.tex
本日は図\ref{fig:sunnyy}に示すように晴天なり。
EOF
$ cat << EOF > .textlintrc
{
    "plugins": ["latex2e"],
    "rules": {
        "ja-no-mixed-period": {
            "periodMark": "。"
        }
    }
}
EOF
$ textlint sample.tex

/Users/pudding/Factory/textlint-latex2e-issue168/sample.tex
  1:4  error  文末が"。"で終わっていません。  ja-no-mixed-period

✖ 1 problem (1 error, 0 warnings)

Cause of this issue
There seems to be an error in the way Paragraph is interpreted. There should be an inline node and a block node, but since \ref is interpreted as a block node, there are two separate Paragraphs, which should be one.
$ npx tex2tast -r sample.tex
{
  "type": "Document",
  "raw": "本日は図\\ref{fig:sunnyy}に示すように晴天なり。\n",
  "children": [
    {
      "raw": "本日は図",
      "type": "Paragraph",
      "children": [
        {
          "raw": "本日は図",
          "type": "Str",
          "value": "本日は図"
        }
      ]
    },
    {
      "raw": "\\ref{fig:sunnyy}",
      "value": "fig:sunnyy",
      "type": "Html"
    },
    {
      "raw": "に示すように晴天なり。",
      "type": "Paragraph",
      "children": [
        {
          "raw": "に示すように晴天なり。",
          "type": "Str",
          "value": "に示すように晴天なり。"
        }
      ]
    }
  ]
}

This problem can be avoided by interpreting it as an inline element, just like label and url.

hmm...
This problem is caused by #70 . HTML type nodes contain two types, inline and block, but there is no way to tell the difference, so they are always interpreted as block elements.
https://github.com/textlint/textlint-plugin-latex2e/blob/8bd09103dd6a3a4cd6d1588c5398093499e34537/src/paragraphize.ts#L42
@tani Do you know of any good solutions?

Oh, I found HtmlBlock in ASTNodeTypes definitions.
https://github.com/textlint/textlint/blob/master/packages/%40textlint/ast-node-types/src/index.ts#L26
We need to use both Html and HtmlBlock.

@atsuyaw
We published a new version of textlint-plugin-latex2e v1.1.4. It includes some fixes for #158 and this issue.
Please check and let me know if there are any bugs. Thank you!

@pddg
Thank you for your corresponding!
I confirmed the problems solved.


Project site not accessible

Tried checking out mollyjs.com, and it seems to be down. The site is referenced from the docs of the RethinkDB project, which is where I came across it from.
Is there anyway we could get this going again? Happy to help deploy it somewhere, doubt it would cost much.

For me the site ends up on a random argos.co.uk page.

This is what I get...


Allow sql.expression to be prepared which makes bind() a no-op (MLDB-1293)

Setting prepare to true massively reduces the latency of starting a query, which helps with ad-hoc query performance.

Cool. If a calls b and a is prepared, does it slow things down if b is not? I.e. does the whole tree need to be prepared or just the root?
Second question: if a calls b and a is prepared, and then b is deleted/recreated, what happens? Does a call the new b?

Code looks fine, but Im curious as to Nick's second question, so +1 when answered.

This is breaking for nested function calls as shown by the failing test.

In answer to Nic's questions:

Only the root of the tree needs to be prepared.  The rest will be bound at the same time.
If b is deleted after being pre-bound to A, then it there should still be a reference to everything needed and it should continue to function.  That hasn't been tested, though.

Note: tests all pass on my dev machine and Jenkins is slow.  I'll merge immediately once I get a +1

+1 but we should probably add a testcase that's using the new option

There is a testcase in MLDBpro... but you're right.  @nicolaskruchten would be worth adding another issue for that.


Show Model trigger events

Models on mutation fire events, that UI is listening. Is there a way I can monitor them? I am using baobab.

Jup, http://www.cerebraljs.com/utilities/events. Or you can point to controller.tree to access the actual Baobab tree and listen to events here :-)


.remove() should allow removing all instances of a substring without using a RegExp

Hi!
It's frustrating that '.remove()` only removes the first instance.
The current workaround is to use a RegExp:
"foo".remove('o'); // => "fo"
"foo".remove(/o/g); // => "f"

I suggest adding a flag allowing it to remove all instances:
"foo".remove('o', true); // => "f"

Suggested implementation (might be buggy and/or slow):
  function(substr, all) {
    if (all && Object.isString(substr)) {
      substr = new RegExp(substr, 'g');
    }
    return this.replace(substr, '');
  }

remove is really just a convenience method for replace, and if there's anywhere in the Sugar library whose value is questionable in the first place, this is probably it. I would be highly skeptical about the value of any deviation from acting like replace.
tl;dr Regexes are your friends, use them.

Isn't the whole point of Sugar to make developers' work easier and more enjoyable?
It's easier to write arr.flatten() than [].concat.apply([], arr) and that's why Sugar lets you use flatten.
It's easier to write str.remove('o') than str.remove(/o/g), and that's why Sugar... doesn't let you use the easier way?
And i'm not asking to deviate from a standard. remove is a custom convenience method, and improving it like that will not violate any standard and is even unlikely to break anyone's code.

Isn't the whole point of Sugar to make developers' work easier and more enjoyable?

Not at the expense of causing confusion or unpredictable behavior. This is crucial and it's part of the reason that things can be enjoyable in the first place. This isn't an extreme example of that or anything but it's a balance that always needs to be kept in mind.

It's easier to write str.remove('o') than str.remove(/o/g)

Is it? It's one more character.

And i'm not asking to deviate from a standard.

No, but I could see someone being just as frustrated that it does global replacements when they just wanted to remove the first match.

they just wanted to remove the first match

But why should .remove only remove first match by default? I believe that removing all matches is a much more common use case.

It's a balance between common use case and expected behavior. Replace acts like this when strings are passed and remove is intended to be parallel to that.
I'm also not convinced it's a "much more" common use case either.

Okay, i see your point, i'm sorry that i didn't see it earlier. I was thinking that remove has a standard counterpart and i didn't understand why a custom method should mimic an already existing one. But i finally realized that custom remove and standard replace do different things, i. e. remove supplements rather than replaces replace. And it would me much easier for everybody to use the same API on both of them.
But from the description of remove it is absolutely unclear that it will remove only the first occurrence, it's a big source of confusion for anyone who is not already familiar with replace. And if a person is already familiar with replace, he is likely to use replace rather than look up a convenience method on Sugar.
So my suggestions are:

Explicitly document that replace only removes the first occurence of a string, while with a regex it removes either first or all occurence(s) based on the g flag. Provide examples.
Create a removeAll method that removes all occurences of a given string. There are a number of concerns why this is important:

Having remove alongside removeAll will make it obvious that one of them does not remove all occurences of a string.
Not everyone is familiar with regexes and especially with their flags, and it will be much easier for those people to use .removeAll(str) than then figuring out regexes after stumbling over single replacement limitation.
You can't just put any string between slashes and have its regex counterpart.  That's tedious and absolutely unnecessary work.

Yeah I mean I saw your point from the start. Kinda wish replace didn't work that way, and if I were to ever think to use a string with it I'd probably have an "oh yeah that's right" moment.
I will definitely update the documentation.
As far as removeAll goes I started out typing this response with the intention of being against it, but I'm starting to think about it more. I think I'll have to think more, but let me just response to each of the points.

Having remove alongside removeAll will make it obvious...
Maybe... I guess. This one isn't so compelling to me tbh but you're not wrong.

Not everyone is familiar with regexes and especially with their flags
This I completely disagree with. Here is my reason. While Sugar is intended to make things easier for developers I have been very conscious about not "masking" the correct way to do things. The problem with JS is that in many cases the "correct" way is often sloppy (creating edge cases) or long-winded and this is where Sugar can be of help. This is not the cases with regexes. They are unquestionably the correct way to do things in many cases. The methods are readable (replace, match) and make intent clear once you know what they do. They take a bit of effort to learn but that's only because they are so powerful. There are some things developers should never have to care how to do (like, say, create an array from an arguments object), but using regexes is not one of them.

 While, again, I'm not interested in creating shortcuts so that users don't have to learn regexes, you're still correct that escaping can be a pain. You've also got me thinking that a replaceAll method might also be very useful here where global replaces can be replaced with the arguments passed. String#assign exists but it can be clunky as you need to have constructed the string yourself ahead of time... So I'm thinking of something like this:
'The shirt costs $50 and the shoes $70'.replaceAll(/\$\d+/, '$80', '$100');

That would actually be rather difficult to pull off normally... in fact I'm kind of surprised this method hasn't made it in already...
Shit I started out against adding one method, now I'm thinking it might make sense to add two.......

A replaceAll function wouldn't be too hard to add :wink:
// 'string'.replaceAll( /regex/, 'first replacement', 'second replacement', 'nth-replacement' );
String.prototype.replaceAll = function() {
	var indx = 0,
		replacements = Array.prototype.slice.call(arguments),
		regexp = replacements.shift();
	return this.replace( regexp, function(original) {
		// replace with original string if no replacement defined
		return replacements[indx++] || original;
	});
};

And using your example:
'The shirt costs $50 and the shoes $70'.replaceAll(/\$\d+/g, '$80', '$100');
// "The shirt costs $80 and the shoes $100"

Summarizing this issue:

$50 and $100'.remove('$') only removes the first instance of $.
@lolmaus claims that this is unexpected to those not familiar with regexes and also that doing it with regexes (remove(/\$/g)) is a pain to escape.
I disagree with the first point. Developers should familiarize themselves with regexes. As to the second point, this is not something that I see as being either all too common or such a large pain for smaller regexes, for example if it was just a dollar sign then it would be .remove(/\$/g), which is hardly a big problem. However if this becomes a larger string then I begin to see his point, as it would be .remove(new RegExp(RegExp.escape(largeStringWithTokens), 'g'). This kind of slight code reduction with improved expressiveness is very much in line with Sugar's domain, however the use case is rather obscure. Very much on the fence about this one.
However, having a replaceAll method would also allow this syntax:
'$50 and $100'.replaceAll('$', '¥', '£');

This could quickly and easily allow multiple substitutes for each token matched, which is quite powerful. If removeAll was added as well it would allow parallelism that would 1) address @lolmaus problem with ambiguity and 2) handle complex escaping situations, which would address both his issues.

Looking at it again I think that overall there is enough here to make adding these methods beneficial, so I will add them. Improving the documents will also make the functionality even more clear.

Ok, these two methods are now added! To see them all in a row:

String#replace - Native JS method. Strings passed will be case-sensitive, escape tokens, and only replace one occurrence. Regexes will respect global flag.

String#remove - Literally just calling replace with ''.

String#replaceAll - Replaces matches with arguments passed. Strings passed will be case-sensitive, escape tokens, and replace all occurrences. Regexes will have their global flag forced on.

String#removeAll - Identical to replaceAll and will replace with ''.

Documents updated to be more clear as well.

:+1:

I'm going to leave this open for now until this actually goes out.


[Playground] parse url_notebook example tag

resolves #24225
follow-up PR which uses new fields: #24425
example tag:
# beam-playground:
#   name: setting-pipeline
#   description: Setting pipeline example.
#   url_notebook: https://some-url/py-file.ipynb
#   multifile: false
#   context_line: 35

Resulting pg_examples entity has an old path field along with new ones: urlVCS(==path) and urlNotebook
Proposed update sequence:

infrastructure merged, examples CD step is run. Both backend versions, old and new consume different pg_examples fields
backend merged, deployed. New backend API provides all fields: link, url_vcs, and url_notebook to not break old frontend client
frontend merged, deployed. New frontend consumes only url_vcs and url_notebook, ignores link
Optional steps:
path is removed from pg_examples, infrastructure merged, examples CD step applied
link field removed from proto and marked reserved
backend, frontend deployed

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

tasks to address the comments:
#24654
#24645


Continued Work on #241 #2

This patch provides additional work to cleanup the intriniscs
logic to remove the #defines in favor of a cleaner, more organized
approach
Signed-off-by: “Rian<EMAIL_ADDRESS>

Coverage remained the same at 100.0% when pulling f4b641871b37ab9ac55da87847e5ec69ea053e0c on rianquinn:issue_241_cleanup_2 into 712d9f84ba8e6f437dd15e0438cbdfc6ab2b1615 on Bareflank:master.


Direction of polarity of C1 and C2

Hi,
When outputting a CPL file, are the polarity directions of C1 and C2 reversed?
https://jlcpcb.com/partdetail/KyoceraAvx-TAJA106K016RNJ/C7171

Hi, what about this issue?

Probably not, but it is important not to reverse polarity (otherwise those tantalums will burn out and short).
C1: GND is 1 (top), VCC is 2. This means the anode marker should point down.
C2: anode marker should point to the right

As far as checking C7171 in jlcpcb, it seems VCC is 1 and GND is 2.


Use dspace.name config variable instead of hard coded string "DSpace" in email templates

Describe the bug
The welcome email already uses variable interpolation for the config var dspace.name instead of just stating "DSpace" over at:
https://github.com/DSpace/DSpace/blob/main/dspace/config/emails/welcome
The other email templates should follow this same approach, to ensure that upon initial installation, the occurences of the word "DSpace" shouldn't be replaced manually by the institution's preferred repository name.
To Reproduce
Steps to reproduce the behavior:

Look at the email templates in https://github.com/DSpace/DSpace/blob/main/dspace/config/emails/
Look for the word "DSpace" and the variable interpolation ${config.get('dspace.name')}

Expected behavior
You should only see the variable interpolation ${config.get('dspace.name')}  and no occurrences of the hard coded string "DSpace".

Atmire would like to claim this issue

Thanks @alexandrevryghem .  Assigning to you for 7.6 then.


Document Podman limitations

What type of PR is this:
Document Podman limitations
Preview: https://deploy-preview-6653--odo-docusaurus-preview.netlify.app/docs/development/architecture/podman-support-limitations
/area documentation
What does this PR do / why we need it:
Which issue(s) this PR fixes:
Fixes #6592
PR acceptance criteria:

[ ] Unit test

[ ] Integration test

[ ] Documentation

How to test changes / Special notes to the reviewer:

• [FAILED] [0.776 seconds]
odo delete command tests [BeforeEach]
/go/odo_1/tests/integration/cmd_delete_test.go:25
  when a component is bootstrapped
  /go/odo_1/tests/integration/cmd_delete_test.go:160
    when the component is deployed in DEV mode and dev mode stopped
    /go/odo_1/tests/integration/cmd_delete_test.go:228
      when the component is deleted using its name (and namespace) from another directory (running-in="")
      /go/odo_1/tests/integration/cmd_delete_test.go:255
        when odo delete command is run again with nothing deployed on the cluster
        /go/odo_1/tests/integration/cmd_delete_test.go:292
          should output that there are no resources to be deleted
          /go/odo_1/tests/integration/cmd_delete_test.go:299

  Begin Captured GinkgoWriter Output >>
    Created dir: /tmp/3770631462
    Created dir: /tmp/3142201452
    Setting KUBECONFIG=/tmp/3142201452/config
    Creating a new project: cmd-delete-test299ueb
    Running kubectl with args [kubectl create namespace cmd-delete-test299ueb] and odo env: []
    [kubectl] Error from server (AlreadyExists): namespaces "cmd-delete-test299ueb" already exists
    Deleting project: cmd-delete-test670jna
    Running kubectl with args [kubectl delete namespaces cmd-delete-test670jna --wait=false] and odo env: []
    [kubectl] Error from server (NotFound): namespaces "cmd-delete-test670jna" not found
  << End Captured GinkgoWriter Output

  No future change is possible.  Bailing out early - after 0.443s.
  Running kubectl with args [kubectl create namespace cmd-delete-test299ueb] and odo env: []
  Expected
      <int>: 1
  to match exit code:
      <int>: 0
  In [BeforeEach] at: /go/odo_1/tests/helper/helper_cmd_wrapper.go:101

  There were additional failures detected after the initial failure.  Here's a summary - for full details run Ginkgo in verbose mode:
    [FAILED] in [AfterEach] at /go/odo_1/tests/helper/helper_cmd_wrapper.go:101

This is not the first time I stumbled upon this issue with existing namespaces/projects. Something to keep an eye on..
Overriding it for now since this is only a doc PR.
/override Kubernetes-Integration-Tests/Kubernetes-Integration-Tests

LGTM.
/hold
@kadel @valaparthvi Anything else to add to this PR?

I am fine with merging this PR. Thank you for documenting this, Philippe.

I am fine with merging this PR. Thank you for documenting this, Philippe.

/hold cancel


Add channel-specific TTLs

Same as #9, but for extra refinement if server owners wish for it. I need to give more thought on if this is actually beneficial, though.

Fwiw I forked this bot so I could use it on a singular channel rather than a server, so this would be useful for me

Maybe #12 would be better for you then, actually @thewilloftheshadow. I see huge potential for making the list of channels that the bot will/wont delete configurable via commands 👀

Hmm, actually, maybe #12 can be simply solved by this issue, if there is a special TTL (like -1 or a commented-out .env variable) that indicates no messages will be deleted until you set a TTL for something. Setting a server-wide TTL would apply it to every channel by default, and setting a channel TTL would only apply it to that channel.. I like that


Overview

Page
Include the link to the page you are reporting a defect
https://docs.alluxio.io/os/user/stable/en/Overview.html
Summary
What is the defect and suggested improvement
Suggest to update the sentence
""Questions can be directed to our User Mailing List (users who can not access the Google Group may use its mirror) or our Community Slack Channel.""
to be
""Questions can be directed to our Community Slack Channel or our User Mailing List (users who can not access the Google Group may use its mirror).""

/assign lzy-a

page
[Architecture - Alluxio v2.9.3 (stable) Documentation](https://docs.alluxio.io/os/user/stable/en/overview/Architecture.html)
Summary
In Section 6.4 Write to UFS Only (THROUGH), write completion and persistence are described in the wrong order. In the case of using UFS only, write should be completed after persistence to ensure that the written data will not be lost.
Suggest to update the sentence
”This write type ensures that data will be persisted after the write completes“
to
”This write type ensures that data will be persisted before the write completes“


Get the progress bars to work properly

This is quite tricky.
For data piped to standard input (at least for CSV and TSV data) it's only possible if we first buffer the data in memory.
For data from a file name we would have to read the file twice - once to count the rows and again to process them.
It's pretty important though. Waiting for the embedding API to run for a large number of documents is really tedious without a progress bar.

I think it's reasonable for the progress bar to only work when reading from files on disk.

After some experiments...
if input_path.raw.seekable():
    # We can find the file length like this:
    input_path.raw.seek(0, 2) # the 2 means from-the-end-of-the-file
    length = input_path.raw.tell()
    input_path.raw.seek(0)
    # Now we can use a progress bar

Here's how I solved this in sqlite-utils:
https://github.com/simonw/sqlite-utils/blob/fc221f9b62ed8624b1d2098e564f525c84497969/sqlite_utils/utils.py#L148-L180

Oh it's a bit more complicated than that - if --sql is used the progress bar needs to be driven by a count(*) query instead.

I have been wildly over-thinking this.
I'm going to offer the progress bar ONLY for the --sql option, where it can show progress against a known count of items.
If you want a progress bar, import your data into SQLite first and then run the command that way.


codestyle: Address issues related to E1101

This patch addresses the following issues detected by pylint:
************* Module keylime.tpm.tpm_abstract
keylime/tpm/tpm_abstract.py:303:20: E1101: Instance of 'AbstractTPM' has no '_get_tpm_rand_block' member (no-member)
************* Module keylime.tpm.tpm1
keylime/tpm/tpm1.py:541:20: E1101: Module 'M2Crypto.m2' has no 'bn_to_mpi' member (no-member)
keylime/tpm/tpm1.py:541:33: E1101: Module 'M2Crypto.m2' has no 'hex_to_bn' member (no-member)
keylime/tpm/tpm1.py:542:20: E1101: Module 'M2Crypto.m2' has no 'bn_to_mpi' member (no-member)
keylime/tpm/tpm1.py:542:33: E1101: Module 'M2Crypto.m2' has no 'hex_to_bn' member (no-member)
************* Module keylime.migrations.versions.eeb702f77d7d_allowlist_rename
keylime/migrations/versions/eeb702f77d7d_allowlist_rename.py:41:4: E1101: Module 'alembic.op' has no 'alter_column' member (no-member)
keylime/migrations/versions/eeb702f77d7d_allowlist_rename.py:47:4: E1101: Module 'alembic.op' has no 'alter_column' member (no-member)
************* Module keylime.migrations.versions.8a44a4364f5a_initial_database_migration
keylime/migrations/versions/8a44a4364f5a_initial_database_migration.py:34:4: E1101: Module 'alembic.op' has no 'create_table' member (no-member)
keylime/migrations/versions/8a44a4364f5a_initial_database_migration.py:50:4: E1101: Module 'alembic.op' has no 'drop_table' member (no-member)
keylime/migrations/versions/8a44a4364f5a_initial_database_migration.py:56:4: E1101: Module 'alembic.op' has no 'create_table' member (no-member)
keylime/migrations/versions/8a44a4364f5a_initial_database_migration.py:81:4: E1101: Module 'alembic.op' has no 'drop_table' member (no-member)
************* Module keylime.migrations.env
keylime/migrations/env.py:17:9: E1101: Module 'alembic.context' has no 'config' member (no-member)
keylime/migrations/env.py:24:11: E1101: Module 'alembic.context' has no 'get_x_argument' member (no-member)
keylime/migrations/env.py:74:12: E1101: Module 'alembic.context' has no 'configure' member (no-member)
keylime/migrations/env.py:82:17: E1101: Module 'alembic.context' has no 'begin_transaction' member (no-member)
keylime/migrations/env.py:83:16: E1101: Module 'alembic.context' has no 'run_migrations' member (no-member)
keylime/migrations/env.py:114:12: E1101: Module 'alembic.context' has no 'configure' member (no-member)
keylime/migrations/env.py:121:12: E1101: Module 'alembic.context' has no 'run_migrations' member (no-member)
keylime/migrations/env.py:138:3: E1101: Module 'alembic.context' has no 'is_offline_mode' member (no-member)
************* Module keylime.config
keylime/config.py:115:21: E1101: Module 'sys' has no '_MEIPASS' member (no-member)
************* Module keylime.revocation_notifier
keylime/revocation_notifier.py:30:34: E1101: Module 'zmq' has no 'SUB' member (no-member)
keylime/revocation_notifier.py:33:28: E1101: Module 'zmq' has no 'SUBSCRIBE' member (no-member)
keylime/revocation_notifier.py:36:33: E1101: Module 'zmq' has no 'PUB' member (no-member)
keylime/revocation_notifier.py:40:8: E1101: Module 'zmq' has no 'device' member (no-member)
keylime/revocation_notifier.py:40:19: E1101: Module 'zmq' has no 'FORWARDER' member (no-member)
keylime/revocation_notifier.py:56:32: E1101: Module 'zmq' has no 'PUB' member (no-member)
keylime/revocation_notifier.py:87:28: E1101: Module 'zmq' has no 'SUB' member (no-member)
keylime/revocation_notifier.py:88:22: E1101: Module 'zmq' has no 'SUBSCRIBE' member (no-member)
************* Module test.test_restful
test/test_restful.py:849:87: E1101: Instance of 'TestRestful' has no 'cloudagent_ip' member (no-member)
test/test_restful.py:849:107: E1101: Instance of 'TestRestful' has no 'cloudagent_port' member (no-member)
Signed-off-by: Stefan Berger<EMAIL_ADDRESS>

I had posted and removed this one as part of another PR before. Don't want to forget about it...


Introduce a tool for generating JVM bytecode & Groovy interceptors; Add more File related APIs to configuration input tracking

Example of an interceptor declaration for Groovy calls that also has a static extension signature:
@InterceptGroovyCalls
@GroovyProperty
@WithExtensionReferences(toClass = ResourceGroovyMethods.class)
public static byte[] intercept_bytes(
    @Receiver File self,
    @CallerClassName String consumer
) throws IOException {
    Instrumented.fileOpened(self, consumer);
    return ResourceGroovyMethods.getBytes(self);
}

The generated code for this would be:
/**
 * Intercepts the following declarations:<ul>
 *
 * <li> Groovy property getter {@link java.io.File#bytes}
 *      with {@link GroovyFileInterceptors#intercept_bytes(File, String)}
 *
 * <li> static method {@link org.codehaus.groovy.runtime.ResourceGroovyMethods#getBytes(File)}
 *      with {@link GroovyFileInterceptors#intercept_bytes(File, String)}
 *
 * </ul>
 */
private static class GetBytesCallInterceptor extends CallInterceptor {
    public GetBytesCallInterceptor() {
        super(InterceptScope.readsOfPropertiesNamed("bytes"), InterceptScope.methodsNamed("getBytes"));
    }

    @Override
    protected Object doIntercept(Invocation invocation, String consumer) throws Throwable {
        Object receiver = invocation.getReceiver();
        if (receiver instanceof File) {
            File receiverTyped = (File) receiver;
            if (invocation.getArgsCount() == 0) {
                return GroovyFileInterceptors.intercept_bytes(receiverTyped, consumer);
            }
        }
        if (receiver.equals(ResourceGroovyMethods.class)) {
            if (invocation.getArgsCount() > 0) {
                Object arg0 = invocation.getArgument(0);
                if (arg0 instanceof File) {
                    File arg0Typed = (File) arg0;
                    if (invocation.getArgsCount() == 1) {
                        return GroovyFileInterceptors.intercept_bytes(arg0Typed, consumer);
                    }
                }
            }
        }
        return invocation.callOriginal();
    }
}

and
if (owner.equals("org/codehaus/groovy/runtime/ResourceGroovyMethods")) {
    /**
     * Intercepting static method: {@link org.codehaus.groovy.runtime.ResourceGroovyMethods#getBytes(File)}
     * Intercepted by {@link GroovyFileInterceptors#intercept_bytes(File, String)}
    */
    if (name.equals("getBytes") && descriptor.equals("(Ljava/io/File;)[B") && opcode == Opcodes.INVOKESTATIC) {
        loadOwnerBinaryClassName(className);
        _INVOKESTATIC(GROOVY_FILE_INTERCEPTORS_TYPE, "intercept_bytes", "(Ljava/io/File;Ljava/lang/String;)[B");
        return true;
    }
}

@bot-gradle test RFN

I've triggered the following builds for you:

ReadyForNightly build

@bot-gradle test RFN

I've triggered the following builds for you:

ReadyForNightly build

@bot-gradle test RFN

I've triggered the following builds for you:

ReadyForNightly build

@bot-gradle test and merge

Your PR is queued. See the queue page for details.

Your PR is queued. See the queue page for details.

I've triggered a build for you.

@bot-gradle test ReadyForRelease

I've triggered the following builds for you:

ReadyForRelease build

@bot-gradle ACC

Sorry I don't understand what you said.
Currently, the following commands are supported:
@bot-gradle test <BuildTrigger1> <BuildTrigger2> ... <BuildTriggerN>

A trigger is a special build for this PR on TeamCity, common triggers are:

SanityCheck/CompileAll/QuickFeedbackLinux/QuickFeedback/PullRequestFeedback/ReadyForNightly/ReadyForRelease
Shortcuts: SC/CA/QFL/QF/PRF/RFN/RFR
Specific builds:

BD: BuildDistributions/BuildDocs so that you can preview the generated docs/distribution.
PT: PerformanceTest, all performance tests for Ready For Nightly stage.
APT: AllPerformanceTest, all performance tests, including slow performance tests.
AST: AllSmokeTestsPullRequestFeedback
AFT: AllFunctionalTestsPullRequestFeedback
ASB: AllSpecificBuildsPullRequestFeedback
ACC: AllConfigCacheTestsPullRequestFeedback
ACT: AllCrossVersionTestsReadyForNightly
AFTN: AllFunctionalTestsReadyForNightly
ACTR: AllCrossVersionTestsReadyForRelease
AFTR: AllFunctionalTestsReadyForRelease

@bot-gradle test and merge

queues this PR for testing and merges if all tests pass by:

Creating a merge commit from your PR branch HEAD and the target branch
Running a ReadyForNightly build against the merge commit
When it passes, fast-forward the target branch to this merge commit (i.e. merge the PR)

The merge commit is called a pre-tested commit, which means that it fully tests the integration
of your branch HEAD and latest master, instead of only testing your branch HEAD.
@bot-gradle cherrypick to <branch>

Cherrypicks the current PR to another branch.
A new PR will be created and a build will triggered automatically if there is no conflict.

@bot-gradle cancel

cancel a running pre-tested commit build or remove it from queue

@bot-gradle clean

clear the conversation history

@bot-gradle help

display this message

To run a command, simply submit a comment.  For detailed instructions see here.

@bot-gradle test ACC

I've triggered the following builds for you:

AllConfigCacheTestsPullRequestFeedback build

@bot-gradle test and merge

Your PR is queued. See the queue page for details.

I assume the test failures are unrelated, let's try to merge it once again.
@bot-gradle test and merge

@bot-gradle test and merge

I've triggered a build for you.

Pre-tested commit build failed.

@bot-gradle test and merge

Your PR is queued. See the queue page for details.

I've triggered a build for you.


Python Implementation.

Initial Implementation.
Minimal usability testing.
Full functionality reimplemented from original project.

Well done! Code looks good. Missing a few tests though ;)

Wow. Great work.
Just 1 suggestion: Please ignore *.pyc (binary) files in the commits.

This was added in a different commit - https://github.com/philwantsfish/shard/commit/be05d2c4de5d5ba7024328d8ccddb9558958ac49


Add sprite endpoints

no problem!

Just tagged this as v1.6.0 which also includes the generic-s3 endpoint.


"Make your own zoneStamp" timestamp seconds should always be 00

Steps to reproduce

Visit https://zonestamp.toolforge.org/
Pick a date and time (I was using 4th May 2023 17:00 UTC
Click "Generate stamp"

What happens?

URL contains a timestamp not ending in 00. There is an element of randomness, maybe from a rounding error, or from modifying the current time.

What should happen instead?

URL should contain a timestamp ending in 00. This is less confusing, and makes copy/pasting that timestamp into other things more accurate.

You mean the seconds offset ?  its indeed based on the current time. Note however that setting it to 0 will not make the url end in 00 but on a multiple of 60, as unixtimestamps are just an amount of seconds, and don't wrap on 60 seconds ;)

Yes good point about the multiple of 60, I misspoke. Will correct the ticket.


Vim sends last character of line twice when running in xterm.js

I'm working on a new feature for xterm.js that allows line reflow.
https://github.com/sourcelair/xterm.js/pull/609
I've found that when vim wraps a line that is too long, it sends the last character of the line twice.
Here is an example of the string data sent at 26 cols:
lorem ipsum dolor sit amet~                         ~

Then if I resize to 25 cols I get this:
lorem ipsum dolor sit ameet
~

As you can see the e in amet is duplicated and vim sends a line with a length of 26 characters, even though the terminal width is set to 25 cols.
If I reduce the size by 1 column again, I get this:
lorem ipsum dolor sit ammet
~

This time the m is duplicated, as it's the last character on that row.
I've found that this is happening when the terminal is xterm-color and vt100.
I'm not sure why this is happening and I was hoping the the Vim community could shed some light on this issue.

Lucian Buzzo wrote:

I'm working on a new feature for xterm.js that allows line reflow.
https://github.com/sourcelair/xterm.js/pull/609
I've found that when vim wraps a line that is too long, it sends the last character of the line twice.
Here is an example of the string data sent at 26 cols:
lorem ipsum dolor sit amet~                         ~

Then if I resize to 25 cols I get this:
lorem ipsum dolor sit ameet
~

As you can see the e in amet is duplicated and vim sends a line with a length of 26 characters, even though the terminal width is set to 25 cols.
If I reduce the size by 1 column again, I get this:
lorem ipsum dolor sit ammet
~

This time the m is duplicated, as it's the last character on that row.
I've found that this is happening when the terminal is xterm-color and vt100.
I'm not sure why this is happening and I was hoping the the Vim
community could shed some light on this issue.

This is intentional.  It's used to signal that the line continues,
instead of has a line break.  See src/screen.c, around line 5710.
xterm.js should replicate the behavior.
--
Yah, well, we had to carve our electrons out of driftwood we'd
find.  In the winter.  Uphill.  Both ways.
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

@brammool Thanks for the response!
What do you mean by "xterm.js should replicate the behavior" ? Perhaps I am being dense, but I don't understand what you mean 😄

Lucian Buzzo wrote:

@brammool Thanks for the response!
What do you mean by "xterm.js should replicate the behavior" ? Perhaps
I am being dense, but I don't understand what you mean 𐟘„

If xterm.js is to be compatible with xterm, it should handle this
sequence of characters in the same way.  Assuming you support
copy&paste.
--
Not too long ago, unzipping in public was illegal...
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

@brammool Thanks for your help, I think this issue can be closed.

This issue is still present in xterm.js


Add possibility to save structure as image

Ketcher API provides a method to save structure as SVG and PNG files by calling Indigo endpoints. In scope of this task is necessary to rework 'Save' dialog in order to  support possibility to save structure in different graphic formats (SVG and PNG).

Release/2.3 The issue is closed as partly fixed.
It's possible to save the structure as image in Standalone mode (as svg and png), in Remote mode (as svg). It will be possible to save the structure as png in Remote mode after the fix of the issue https://github.com/epam/ketcher/issues/371 will be done. Besides the sizes of the view boxes are different in Remote and Standalone modes now. It has been decided that the problems are not blocking, since the main functionality works correctly. Behavior will be rechecked after the fixes.


docs: add subrepos to README, switch to Apache-2.0

:construction_worker: Purpose
We have quite a lot of subrepos - they're (I think) currently all tagged with temporal, so a link to a github filter can bring up all related subrepos.
All other temporal components are under apache-2.0 now, so this PR also switches to the new licence
:rocket: Changes
link to subrepos
mit -> apache2.0
:warning: Breaking Changes
might break fossa

oh oops forgot to push version bump, will open another PR


Remove unneeded deps

What problem does this feature solve?
cross-env (deps)  and nodemon (devDeps) are not needed for every project. We should properly guard them as well.

Nodemon and cross-env is needed when i used express or another server js (expressjs , adonis , etc)

@MrJmpl3 That's what I was saying :relaxed:
If you use a Nuxt.js inside a server like express, koa, ... then you need it ofc.

Fixed in 2.9.1


Fix qt/paymentrequestplus.cpp for OpenSSL 1.1 API

https://github.com/bitcoin/bitcoin/pull/9326 Fix qt/paymentrequestplus.cpp for OpenSSL 1.1 API

utACK


Upgrade compile compatibility to Java 8

Details

Reformat mvrx/build.gradle file
Upgrade all other modules compatibility options to Java 8

Fixes

Upgrade compile compatibility to Java 8

@elihart @gpeal please, let me know if you have any doubts about the pull request :v:


Improve the design of negative-sampling

Fixes # (issue)
Goals :soccer:

Implementation Details :construction:

Testing Details :mag:

@marcromeyn can we make l2_normalization arg explicit for the user in the MatrixFactorizationModel()? thanks.

rerun tests

Marc might break this into smaller pieces: RMP tickets are being created.


Community SQL ingest docs

This branch contains newly written community ingest content for SQL databases.
What's needed

check the files in /docs/community/com-ingest directory contain accurate information
point out any errors of logic in the SQL-specific include files found in /_includes/com-ingest folder

Note

This branch has some broken links, the result of work in progress in the sql-statements branch.
/_includes/com-ingest folder is duplicated from the /com-ingest-csv branch which is in PR right now.

Hey @gthrone25 , @tgruben and @jaffee -- is this good to go or is it requiring more eyeballs into the stew?
shudder I can't believe I said that >.<

FYI have merged in changes from the CSV branch so this is now up-to-date.

@gthrone25  -- good to go with the merge? :D

@ljsinclair I did a quick scan and the only thing I noticed is the postgres example no longer exists. Add and merge or merge and add later unless it was left out for a reason 👍


Long audio needs much more time to transcribe

For example, a 30 minute audio takes 5 minutes to transcribe, but a 3 hour audio takes an hour to do so.
Why?
How can I make it faster? Should I split them up and transcribe them separately? If so, how can I pass the last window's prompt to the next split file?

Transcription time depends on the audio duration AND content, so comparing by duration alone isn't relevant, what is happening is that you are probably using a low quality audio with a lot of fallbacks, set temperature=[0] to avoid that on the expense of probably lower transcription quality, or use batched transcription


docs: add dlt-lancedb integration page

Thanks for the PR! Can we add this to the integrations page instead as discussed ? https://github.com/lancedb/lancedb/tree/main/docs/src/integrations
And you'll need to add this in the index for visibility in navigation https://github.com/lancedb/lancedb/blob/main/docs/mkdocs.yml#L126
https://github.com/lancedb/lancedb/blob/main/docs/mkdocs.yml#L220


First Tile of map is grey after hiding map

Hello, I've created a map on a product page along with a toggle to switch between the map and a standard image. On first load, all the tiles are show. When I toggle the preview image and then toggle the map again, the first tile of the map (which seems to be located at the top-middle) is not loaded but the others tiles are. When I then zoom out or zoom in, all the tiles are again visible. I have used 'invalidateSize()' on my map but it doesn't seem to work. What am I doing wrong here? This is all the code:
var basec="";
$(document).ready(function() {
        initializeMap();
     	$("#placeholder2").css("display","none");
    	openmap.invalidateSize(false);
     $("div.bhoechie-tab-menu>div.list-group>a").click(function(e) {
        e.preventDefault();
        $(this).siblings('a.active').removeClass("active");
        $(this).addClass("active");
        var index = $(this).index();
        $("div.bhoechie-tab>div.bhoechie-tab-content").removeClass("active");
        $("div.bhoechie-tab>div.bhoechie-tab-content").eq(index).addClass("active");
    });
    drawRegion();
});
function initializeMap(){
      	    basec =L.tileLayer('http://{s}.tile.openstreetmap.org/{z}/{x}/{y}.png', {
            attribution: ' © OpenStreetMap contributors'
		});
          openmap = L.map('map', {
    		center: [49.152969, 6.473007],
    		zoom: 9,
    		layers: [basec],
		});
}
var multipol = "";
function drawRegion(){
       var prhandle = "{{ product.handle }}"
      var jsonurl = "http://shopify-aquaterra.cloudapp.net/shopify/"+prhandle+".json";
      var xmlhttp = new XMLHttpRequest();
      xmlhttp.onreadystatechange = function() {
            if (xmlhttp.readyState == 4 && xmlhttp.status == 200) {
                  var myArr = JSON.parse(xmlhttp.responseText);
        	  getBounds(myArr);
    	    }
      }
      xmlhttp.open("GET", jsonurl, true);
      xmlhttp.send();
      function getBounds(arr) {
              var profeatures = arr.features;
              var polygons = [];
              for (i = 0;i<profeatures.length; i++){
                  	var coords = profeatures[i].geometry.coordinates[0];
                  	var polygonPoints = [];
                  	for (j = 0;j<(coords.length-1);j++)
                        {
                            var tpoi = transWebmercatorTOMercator(coords[j]);
                            var poi = L.latLng(tpoi[1], tpoi[0]);
                            polygonPoints.push(poi);
                       }
                  	//var polygon = new L.Polygon(polygonPoints);
                  	polygons.push(polygonPoints);
              }
              multipol = new L.multiPolygon(polygons,{
                 	color: '#6591c3',
    			fillColor: '#6591c3',
    			fillOpacity: 0.4,
                        weight:3,
                       opacity:1
              }).addTo(openmap);
              var bounds = multipol.getBounds();
	      openmap.fitBounds(bounds);
      }
}
function setOverview(){
        $("#map").css("visibility","visible");
        openmap.invalidateSize(false);
         $("#placeholder2").css("display","none");
       	 openmap.invalidateSize(false);
         basec.redraw();
}
function setPreview(){
        $("#map").css("visibility","hidden");
        openmap.invalidateSize(false);
        $("#placeholder2").css("display","block");
        openmap.invalidateSize();
        $("#placeholder2").css("margin-top","-400px");
        openmap.invalidateSize(false);
}

Can you post a proper JSFiddle test case?

https://jsfiddle.net/JulieVelghe/6mzbexpj/3/ this is the fiddle, it seems to work. However on the site I still have the problem: http://mapshop.aquaterra.be/collections/country/products/albania_08 (password is: breats)

So the first tile has an empty URL:

But that happens when the leaflet container is hidden. I guess it may be a side effect of too many invalidateSize().

Indeed that's the problem, when I zoom on the map, it seems the map gets the tiles back. I removed a few of the invalidateSize(), but the problem is still there. The fiddle doesn't have any problems at all

@YiuriV If you trace DOM modifications of the first tile, it turns out that MinimalTheme.switchImage() is being called with the first tile as an argument, and that is what's messing it up.
I think your jQuery selectors are wrong in a click handler.
So not a Leaflet problem after all.


Can't update other fields on translatable entity

Sorry for disturbing again, but I have been running from one problem to another with these extensions for some time now.
This is the latest and biggest one. I have been trying to come up with something for hours but with no luck.
Let say, you have this entity class: (letf out getters/setters)
namespace models;

use Doctrine\ORM\Mapping as ORM;
use Gedmo\Mapping\Annotation as Gedmo;

/**
 * models\Book
 *
 * @ORM\Table(name="book")
 * @ORM\Entity(repositoryClass="models\Repositories\BookRepository")
 * @Gedmo\TranslationEntity(class="models\Translation\Book")
 */
class Book extends Base\BaseEntity
{
    /**
    * @var integer $id
    *
    * @ORM\Column(name="id", type="integer")
    * @ORM\Id
    * @ORM\GeneratedValue(strategy="IDENTITY")
    */
    protected $id;

    /**
    * @var string $name
    *
    * @Gedmo\Translatable
    * @ORM\Column(name="name", type="string")
    */
    protected $name;

    /**
    * @var string $image
    *
    * @ORM\Column(name="image", type="string")
    */
    protected $image;

    /**
    * @Gedmo\Locale
    * Used locale to override Translation listener`s locale
    * this is not a mapped field of entity metadata, just a simple property
    */
    protected $locale;

    public function setTranslatableLocale($locale)
    {
        $this->locale = $locale;
    }
}

When I try to do something like:
$book = \models\Book::getRepository()->find(1);
$book->setImage('some image');
Doctrine::em()->persist($book);
Doctrine::em()->flush();

it doesn't update. It fills the values, after persist they are still there, but on flush it doesn't save them to the DB.
If I create a new entity and save it, it works. So insert works, update does not.
Also if I set the name field, the translation in the other table gets updated, but other entity fields in the entity table don't change.
When I remove the @Gedmo\Translatable annotation, and run the same code, it works just fine.
I'm using the #af3e430 changeset version. Tried it with Doctrine 2.2.0 and 2.2.2, no difference
Here is my configuration file http://pastebin.com/4yM6wYbW . It is a wrapper for Kohana framework
Also, I'm using custom change notification policy, like described here http://readthedocs.org/docs/doctrine-orm/en/latest/cookbook/implementing-the-notify-changetracking-policy.html. Maybe it has got something to do with this. I have a feeling that the change isn't registered on the managed entity, so flush thinks it has nothing to do

@zolakt Thank you very much for your comment. I had the same problem for several hours.
However, I have only commented the line :  $ea->clearObjectChangeSet($uow, $oid);
It works.
"doctrine/orm": "~2.2,>=2.2.3,<2.5",
"stof/doctrine-extensions-bundle": "^1.2"
Thanks !!


Add lru-memoize to benchmark

Codecov Report

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

@@          Coverage Diff          @@
##           master    #47   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           1      1
  Lines          32     32
  Branches        0      8    +8
=====================================
  Hits           32     32

Thanks for that


issue under send Debit note

[ ] after sending debit note , the status not changing to sent status
[ ]

issue fixed !


feat(tests): define reveal_type at runtime

Summary

Closes GH-596

Checklist

[x] If code changes were made, then they have been tested

[ ] I have updated the documentation to reflect the changes
[x] I have formatted the code properly by running task lint
[x] I have type-checked the code by running task pyright

[ ] This PR fixes an issue
[x] This PR adds something new (e.g. new method or parameters)
[ ] This PR is a breaking change (e.g. methods or parameters removed/renamed)
[ ] This PR is not a code change (e.g. documentation, README, ...)

This makes pyright completely ignore these reveal_types; it appears that it only considers those defined in typing/typing_extensions, which makes sense:

(screenshots since github doesn't particularly like rendering a 24000-line file)
I suppose one possible solution would be re-exporting typing_extensions.reveal_type in an if TYPE_CHECKING: block, but @onerandomusername might have something different in mind.

168fbd2


How to remove padding in custom view and disable title?

as this code :
boolean wrapInScrollView = true;
new MaterialDialog.Builder(this)
.customView(R.layout.custom_view, wrapInScrollView)
.positiveText(R.string.positive)
.show();
In my custom_view.xml, i have custom title using textedit, and also applied background color. The problem is, MaterialDialog creating padding on all side. So how do I remove the padding?

Pass false for wrapInScrollView and handle scrolling yourself.

tried that. But the title  padding is still there.

Oh, yeah you can't remove title padding. It used the Material Design
Guidelines specs.
On Wed, May 20, 2015, 10:16 PM azizimusa<EMAIL_ADDRESS>wrote:

tried that. But the title padding is still there.
—
Reply to this email directly or view it on GitHub
https://github.com/afollestad/material-dialogs/issues/533#issuecomment-104112032
.

@afollestad hey there, quick question, what about bottom padding?


⚠ ClamAV has degraded performance

In 4bb7c6a, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 290 ms

Resolved: ClamAV performance has improved in 321b5f9.


Rendering of & as & in TOC

Search terms
TOC, ampersand
Expected Behavior
Render ampersand in TOC as &.
Actual Behavior
Ampersand in TOC is rendered as &.
Steps to reproduce the bug

Environment

Typedoc version: 0.24.1
TypeScript version: 4.9.5
Node.js version: 16.19.1
OS: macOS 12.6.3

Shoot, I totally forgot that content would already be escaped... interesting dilemma here actually, since in-line elements probably should be stripped when generating the TOC. Ideally, I think getting the text from .textContent on the header should be shown, which is kind of annoying to get without DOM apis.

We see something possibly related over at https://github.com/jupyterlab/jupyterlab/pull/14367 (sorry, I was not sure if you prefer new issues or comment here) - the link in the title is rendered as HTML:

The title in question is # [JupyterLab](http://jupyterlab.github.io/jupyterlab/) in markdown (and actually ideally we skip it as this is just the title).

Yep, this is the same underlying issue

Wow, that was a lesson in insanity! HTML entities are special.

I confirm, with v0.24.2 the TOC is rendered correctly in my projects.
Thank you!


initiate msquic wrapper library development

Initiate and add msquic wrapper library development in stream module.
This issue is only to get the library development deps and base initiatives started as a base ground for its further step by step development.

vcpkg PR to add msquic v2.2 is in progress: https://github.com/microsoft/vcpkg/pull/29568
using fetchcontent with source code leads to really long and failing process of fetching its deep nested submodules. simply unfeasible.
@PooyaEimandar, how do recommend to proceed?
I'd suggest using pre-built binaries (downloaded via FetchContent from msquic's released packages in github) windows-only until v2.2 is added to vcpkg.

vcpkg PR to add msquic v2.2 is in progress: https://github.com/microsoft/vcpkg/pull/29568
using fetchcontent with source code leads to really long and failing process of fetching its deep nested submodules. simply unfeasible.
@PooyaEimandar, how do recommend to proceed?
I'd suggest using pre-built binaries (downloaded via FetchContent from msquic's released packages in github) windows-only until v2.2 is added to vcpkg.

Sounds good