Add insertmode indicator

This adds the insertmode indicator to all except the minimal theme because it already has it.

@Valinora give me a ping when you can clean up the commit history a bit and would like me/others to take a look. Thanks for the PR, looking forward to being able to merge this at some point! :+1:

This doesn't seem to be optional. Perhaps a check on an env variable (ZIM_PROMPT_INSERTMODE= 1 or 0) could be added to toggle the RPROMPT?

That's a consideration. I'm likely to re-implement what I did here.


—
Reply to this email directly, view it on GitHub
https://github.com/Leanplum/Leanplum-Android-SDK/pull/515#event-7558980867,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A2TTPEQNGEFWMQPYCVLJT7LWCT4ERANCNFSM6AAAAAARBCLPTE
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***
com>


Prevent conversion from UTC to local

I did search & checked the specs but wasn't able to get a consistent / clear answer on this.
Is it possible to preserve UTC and prevent clients from converting it to their local time? The use case we have is that all calendar events take place at the same physical location be we don't know their timezone so we send all events as UTC and ensure the times are correctly - we just don't want the events to localize to where they are.
Is there any way to prevent that from happening?

  Start = new CalDateTime(DateTime.ParseExact(event.Start.ToString(format: "MM/dd/yyyy HH:mm"),                         "MM/dd/yyyy HH:mm", CultureInfo.InvariantCulture));
Was able to sovle my issue by manually converting the date to prevent timezone conversions.


Ores are not spawning, they leave holes in the terrain gen

I made a new world in creative to test this since I tried it on survival and I have noticed that no ores were spawning, instead they left holes like the mod was removed, since I made 2 new worlds today the ore isnt spawning maybe a conflict between mods I do not know.
here is the list of mods I have installed:
Mods installed.txt
If you look at the image, at the top left you can see the mini map and it shows holes which could suggest that the mod tried to spawn the ore but for some reason goes away.

Well this is weird, last time i tested it was working... Now I tested it myself and I had the same issue. I'll look into it and send a fix as soon as possible, sorry about the inconvenience!

@GabrielOlvH thanks for a quick response, I know I might be extending a bit but are you willing to maybe add a command to so the ore gen can be redone I dont want to restart my world, of course if its too time consuming I understand!

I'm not even sure how to do that, sorry, but you'll still be able to find new ores in new chunks though

Fixed as of 1.2.0-BETA, which you can download here. Let me know if you still have any issues.

@GabrielOlvH Thanks will test it now!


ddollar/heroku-accounts auto-uninstalled, and didn't provide alternative

$ heroku accounts
Uninstalling deprecated ddollar/heroku-accounts plugin... done
* work
work-admin
$ heroku accounts
 !    `accounts` is not a heroku command.
 !    See `heroku help` for a list of available commands.

Um... that seems overly-aggressive. Couldn't it at least say something like "install the official plugin using command" or... something?
Also, -r production is no longer working for me when attempting to run heroku commands. I get this:
$ cat .git/config
[remote "production"]
    url = git@heroku.work:my-cool-app.git
    fetch = +refs/heads/*:refs/remotes/production/*
$ heroku run rails c -r production
 ▸    could not find app name in production git remote

This is a heroku-accounts issue, because if I change the gitconfig remote to look at<EMAIL_ADDRESS>the -r option works just fine, but when i use heroku accounts:set work it changes it back to<EMAIL_ADDRESS>Why can't I just... keep using the old working version of the plugin?  :(

The old plugin won't work because we're replacing the entire toolbelt and with the new one we will no longer support the monkey-patching functionality that the other plugin depends on to function. Also, we've moved away from using ssh to connect for git push which that plugin depends on. It will no longer work with heroku run at all.
A deprecation warning has been up for months, but at this point enough of the commands no longer function we decided to fully deprecate the old plugin.
The issue with -r you're experiencing isn't from this version of the plugin, it's from the other one. If you use heroku git:remote it will fix your remotes to what they should be.

A deprecation warning has been up for months, but at this point enough of the commands no longer function we decided to fully deprecate the old plugin.

That deprecation warning started showing up for everyone in my team only last week...
Thanks for the note on heroku git:remote, with the official plugin & resetting my remote w/ that, it seems to be functioning as expected.

@dickeyxxx thanks for this explanation - where should we be keeping an eye out for these deprecation messages in the future? I follow heroku changelog on twitter and didn't see anything here. I only saw the deprecation start last week around when this issue cropped up: https://github.com/ddollar/heroku-accounts/issues/85

well there were 2 levels of deprecation, one only happened when you ran heroku login, that one has been in for a long time, but last week we made the warning much more apparent, but because it was also incompatible with more commands (most importantly, run) we decided to be a bit more forceful due to support tickets where people weren't able to get run to work with having the plugin installed


Update mongodb-cloud-tutorial.md

Added role mappings section and related content in the tutorial.

wrong user, sorry


Change cppguide.xml to proper XML serialized HTML

cppguide.xml does not contain valid XML. xmllint can not parse it and chrome can not display it.
# xmllint cppguide.xml
cppguide.xml:8: parser error : Opening and ending tag mismatch: meta line 6 and head
</head>
       ^
cppguide.xml:18: parser error : Opening and ending tag mismatch: meta line 5 and html
</html>
       ^
cppguide.xml:19: parser error : Premature end of data in tag meta line 4

^
cppguide.xml:19: parser error : Premature end of data in tag head line 3

^
cppguide.xml:19: parser error : Premature end of data in tag html line 2

^

chrome http://google.github.io/styleguide/cppguide.xml
XML Parsing Error: mismatched tag. Expected: </meta>.
Location: http://google.github.io/styleguide/cppguide.xml
Line Number 8, Column 3:

</head>
--^

Validated the changes with https://validator.w3.org/

thx for the contribution


Add nodeSelector support for helm chart

Summary

Ticket Link

Hello @FlipEnergy,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

Hello @FlipEnergy,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

@stylianosrigas @jwilander any idea if this is a new error in linting:
Adding stable repo with URL: https://kubernetes-charts.storage.googleapis.com
Error: error initializing: Looks like "https://kubernetes-charts.storage.googleapis.com" is not a valid chart repository or cannot be reached: Failed to fetch https://kubernetes-charts.storage.googleapis.com/index.yaml : 403 Forbidden
Error linting charts: Error initializing Helm: Error waiting for process: exit status 1

@stylianosrigas @jwilander any idea if this is a new error in linting:
Adding stable repo with URL: https://kubernetes-charts.storage.googleapis.com
Error: error initializing: Looks like "https://kubernetes-charts.storage.googleapis.com" is not a valid chart repository or cannot be reached: Failed to fetch https://kubernetes-charts.storage.googleapis.com/index.yaml : 403 Forbidden
Error linting charts: Error initializing Helm: Error waiting for process: exit status 1

@stylianosrigas @jwilander any idea if this is a new error in linting:
Adding stable repo with URL: https://kubernetes-charts.storage.googleapis.com
Error: error initializing: Looks like "https://kubernetes-charts.storage.googleapis.com" is not a valid chart repository or cannot be reached: Failed to fetch https://kubernetes-charts.storage.googleapis.com/index.yaml : 403 Forbidden
Error linting charts: Error initializing Helm: Error waiting for process: exit status 1

@jaydeland This stable repo is deprecated and replaced by https://charts.helm.sh/stable

@stylianosrigas @jwilander any idea if this is a new error in linting:
Adding stable repo with URL: https://kubernetes-charts.storage.googleapis.com
Error: error initializing: Looks like "https://kubernetes-charts.storage.googleapis.com" is not a valid chart repository or cannot be reached: Failed to fetch https://kubernetes-charts.storage.googleapis.com/index.yaml : 403 Forbidden
Error linting charts: Error initializing Helm: Error waiting for process: exit status 1

@jaydeland This stable repo is deprecated and replaced by https://charts.helm.sh/stable

oh - I see @FlipEnergy can you merge the latest code from master into your branch?

oh - I see @FlipEnergy can you merge the latest code from master into your branch?

oh - I see @FlipEnergy can you merge the latest code from master into your branch?

Hm, my branch appears to be only 1 commit on top of the latest master (commit 6906a2126a931a000edc15bc403203ee6cf15835) so it is up to date. I did double check by pulling and such.

oh - I see @FlipEnergy can you merge the latest code from master into your branch?

Hm, my branch appears to be only 1 commit on top of the latest master (commit 6906a2126a931a000edc15bc403203ee6cf15835) so it is up to date. I did double check by pulling and such.

oh - I see @FlipEnergy can you merge the latest code from master into your branch?

Hm, my branch appears to be only 1 commit on top of the latest master (commit 6906a21) so it is up to date. I did double check by pulling and such.

oh sorry - I can see that tests/ct.yml has the correct updated stable repo on master and assumed that was the problem.

oh - I see @FlipEnergy can you merge the latest code from master into your branch?

Hm, my branch appears to be only 1 commit on top of the latest master (commit 6906a21) so it is up to date. I did double check by pulling and such.

oh sorry - I can see that tests/ct.yml has the correct updated stable repo on master and assumed that was the problem.

Hm, my branch appears to be only 1 commit on top of the latest master (commit 6906a21) so it is up to date.

You are correct @FlipEnergy, but since then and now Helm 2.x, which is being used in quay.io/helmpack/chart-testing:v2.4.1 has been fully deprecated, you can bump the chart-testing version to v3.3.1 and give it another go.

Hm, my branch appears to be only 1 commit on top of the latest master (commit 6906a21) so it is up to date.

You are correct @FlipEnergy, but since then and now Helm 2.x, which is being used in quay.io/helmpack/chart-testing:v2.4.1 has been fully deprecated, you can bump the chart-testing version to v3.3.1 and give it another go.

The file .circleci/config.yml is in the blocklist for external contributors. Hence, these changes are not tested by the CI pipeline active until the build is re-triggered by a core committer or the PR is merged. Please be careful when reviewing it.
/cc @mattermost/core-security @mattermost/core-build-engineers

The file .circleci/config.yml is in the blocklist for external contributors. Hence, these changes are not tested by the CI pipeline active until the build is re-triggered by a core committer or the PR is merged. Please be careful when reviewing it.
/cc @mattermost/core-security @mattermost/core-build-engineers

Ok, new error, that's good!

Ok, new error, that's good!

Thanks all!

Thanks all!


gulp-glitch first pass

Hey Jeff,
I've edited the code to use await instead of on but wasn't sure how to move away from the usual pipe syntax w/in gulp. This currently works but would like your thoughts on how to revise. I'll also start recording some documentation on how to scale this to other glitch projects.
Phillip

Updated w/o gulp piping, um yeah you are totally correct, way easier and clearer.

Great! Glad that approached worked for you.

@jeffposnick this latest commit demonstrates for two demos what a top level express server might look like. Since we are launching an express server, we need package.json and updateServer.js at the top level. In addition, the express server has to manage all requests, so by doing this I am forced (unless I'm mistaken) to split updateServer.js into two parts: the top serves index.html, sw.js, and any static files. I then use a comment to indicate that the bottom half is unrelated to the demo. That's where we place the gulp hook listener. Let me know if this seems reasonable.


New Group inc ransom

http://incblog7vmuq7rktic73r4ha4j757m3ptym37tyvifzp2roedyyzzxid.onion

Thanks.
Done with Pull https://github.com/JMousqueton/ransomware.live/pull/19


Improve batch closing of all tabs

Is your feature request related to a problem? Please describe.
Given I have a number of tabs open and I'd like to close them in a batch, e.g. via Close All Tabs or when closing the application.
In these cases there is a lot of unnecessary tab loading / dirty checking going on.
Describe the solution you'd like
During the tab closing process...

[ ] Do not open tabs that have not previously been opened
[ ] Do not perform the dirty check if the next thing we're going to do is closing the tab
[ ] Do not open tabs that are not dirty

Describe alternatives you've considered
No real alternatives.

Related to SUPPORT-5401.

We have a big chance here to improve some crucial dirty check workflows here. Should we pull it in our current (or even next) milestone?

How much is broken right now? I'd suggest we create a test case that verifies the above first and assert how much is actuall broken.

Closed via https://github.com/camunda/camunda-modeler/pull/2709


Sync: Test Script

Task/Issue URL:
Description
Steps to test this PR
Feature 1

[ ]
[ ]

UI changes

Before
After

!(Upload before screenshot)
(Upload after screenshot)

Current dependencies on/for this PR:

develop

PR #3621

PR #3622   👈

This comment was auto-generated by Graphite.


ci(drone): remove privileged in build stage

ci(drone): remove privileged in build stage
Signed-off-by: Yang Chiu<EMAIL_ADDRESS>

@mergify backport v1.5.x v1.4.x

For any CI configs, we don't need to backport in the future.
cc @longhorn/dev


UI components - Form storage: Birth date no logo and initial selected date

Should be fixed in Framework7 V3, if not, please open new issue.


Improve logging across the package

We should add multiple debug logs to help the developers to debug users problems.
Some debug logs that could be added:

log requests and responses
log contract initialization
log ape actions
framework args serialization
input and output of osiris des/serialization

This should be done using the logging module and we should create a utility function that can easily be added to start the log level of the library like:
from giza.agents.utils import debug_logging

# This should make logging to show all the DEBUG and INFO logs
debug_logging()

can I work on this?

Fixed on #60


Add header X-Forwarded-Port to preset transparent

1. What does this change do, exactly?
My change adds the requested server port to the header X-Forwarded-Port for preset transparent.
2. Please link to the relevant issues.
#2412
3. Which documentation changes (if any) need to be made because of this PR?
On page https://caddyserver.com/docs/proxy you have to add "header_upstream X-Forwarded-Port {server_port}" within the code block for preset transparent.
4. Checklist

[X] I have written tests and verified that they fail without my change
[X] I have squashed any insignificant commits
[X] This change has comments for package types, values, functions, and non-obvious lines of code
[X] I am willing to help maintain this change if there are issues with it later

I updated the docs and they'll go out with the next site release.


Enhancements to ID cards

Hi Guilhem,
I'm not sure whether this is a bug or an enhancement request. Currently, when an ID card is updated any sources that are associated with the card are lost and have to be re-added. Would it be possible to have an option to maintain the existing associations?
Also, would it be possible to 'wildcard' the object name for the ID card associations? For example, we have hundreds of indexes that begin with 'awscloudtrail'. They would all use the same troubleshooting/remediation documentation and creating the associations would be a pain. I did think of setting the Global ID card to this but there is an equal number of 'awscloudfront' indexes that have a different troubleshooting/remediation document :(
I know that I've requested a few enhancements recently, please don't think that I'm in any way critical of your excellent app.
Cheers,
Mark.

Oops, I meant to also add ( sorry ) - markdown in the ID card text frame?

Thank you @cidermark  and don't worry, I am very happy getting your feedback and inputs.
The answer to the first question is no, when a card is updated that should be removing or changing any association, I need to verify then!
In 1.2.35, I introduced new REST API endpoints, which allows handling the cards:
https://trackme.readthedocs.io/en/latest/rest_api_reference.html#identity-cards-endpoints
Just FYI

Ah yes! I'll look in to creating the associations using the API. Thanks

@cidermark

The UI issues which was leading to loose current associations is fixed in 1.2.36
A new wildcard matching feature addresses your requirements, please review:

https://trackme.readthedocs.io/en/latest/userguide.html#data-identity-wildcard-matching
Fixed in 1.2.36 published in Splunk Base.


Cannot assign value of type '[String : Any]?' to type '[NSAttributedStringKey : Any]?'

I change in swift 4:
Go to pods projects in your workspace.
Select RSSelectionMenu target.
Go to Build Settings and set Swift Language Version to 3.2
But " Cannot assign value of type '[String : Any]?' to type '[NSAttributedStringKey : Any]?' "

Hi @songcamhoi, please update your pod to latest version RSSelectionMenu (3.1)
Swift 4.0 support is added now.


Setting graphics background color and clearing doesn't work right

http://www.intfiction.org/forum/viewtopic.php?f=7&t=19261
[ ReColorSub;
    if (~~gg_graphwin)
        print_ret "There is no graphics window.";
    glk_window_set_background_color(gg_graphwin, parsed_number);
    glk_window_clear(gg_graphwin);
    print_ret "Set default window color to ", (Hex6) parsed_number, ", then erased.";
];

Fixed.


Example with children

hi! I'm looking for an example using children for tabs, like a "tab panel" or something like that, I'm trying to match with children but I can't.
And I have an issue for view in mobile, when is less of 900px in screen, the tabs are small, instead of scroll with draggable.
thanks!

Hey
About the tab panels i was thinking to add them in the next version but for now maybe i will add any example abour how to add them manually and matching their tab if you could provide me with an example so i can start with it.
The tabs are focused more on navigation and this is the reason why i built them.
About on mobile can you reproduce a codesandbox? Or any thing so i can see the issue.

hi! I'm looking for an example using children for tabs, like a "tab panel" or something like that, I'm trying to match with children but I can't. And I have an issue for view in mobile, when is less of 900px in screen, the tabs are small, instead of scroll with draggable. thanks!

Hey, I updated the demos and added tabs screens, I dont know if this is what u want but I hope this helps u.
u can check it out from here, it's the second example.
https://react-tabs-scrollable.vercel.app/demos#withTabScreens


fix: fix missing pip bug and add 'python_executable' option

This pull request makes sure that pip will be installed when it is not found. It also adds the python_executable action argument, allowing users to specify the python executable they want to use. It was created to solve #35.

ping @rickstaa san

@tsuyoshicho Thanks a lot for your feedback!

Is it okay to change the execution environment for subsequent workflows by installing pip? (Assuming that flake8 has no problem).

That's an excellent point. On second thought, I don't think most people would appreciate this. Especially since I'm using the non recommended way of installing pip to support more distributions. Maybe, throw a warning when pip is not found? Alternatively, can we create an install_pip action argument and set the default to false?

If it is a local runner, isn't it the creator's responsibility to create an environment where actions can be executed? (Python symlink, install pip)
(Note: Python setup itself in a normal GHA environment is the responsibility of actions/setup-python)

Again I think you are right on this point. Maybe let's throw a warning when python is not found?
Let me know what you think so that I can change this pull request.

@rickstaa Thanks response.
I think fix these two point.

python  is configurable (to include python3 or specific version /bin/python3.10)
limitation: action-flake8 user to required either:

flake8 setup was already done
python with pip setup was already

re-ping @rickstaa san

@tsuyoshicho, due to my thesis, sadly haven't had the time to update this PR. You can, of course, also change the pull request yourself to the form you see fit and ask me to review it. Or we can just close it since, as you pointed out in https://github.com/reviewdog/action-flake8/pull/36#pullrequestreview-914991630 the problem encountered in https://github.com/reviewdog/action-flake8/issues/35 is quite an edge use case that will likely not be encountered by a lot of people. 🤔


Add IngestFiles example

Added this example as I think it is better here than added to the Tutorial as I suggested in this issue.
@Anton-4 if you concur I'll close that issue.

Related zulip discussion


Panic when reading from partitioned datasets with columns that have ' in them

Describe the bug
There is a bug when reading from partitioned tables that have commas in their names
Here is the test
https://github.com/apache/arrow-datafusion/blob/b2a04519da97c2ff81789ef41dd652870794a73a/datafusion/sqllogictest/test_files/copy.slt#L109
To Reproduce
Run this script
-- create a table with quotes in the column names
create table test ("'test'" varchar, "'test2'" varchar, "'test3'" varchar);
insert into test VALUES ('a', 'x', 'aa'), ('b','y', 'bb'), ('c', 'z', 'cc');
copy test to '/tmp/escape_quote' (format csv, partition_by '''test2'',''test3''');

-- read back from the table
CREATE EXTERNAL TABLE validate_partitioned_escape_quote STORED AS CSV
LOCATION '/tmp/escape_quote/' PARTITIONED BY ("'test2'", "'test3'");

-- This panics
select * from validate_partitioned_escape_quote;

Here is an example:
❯ -- create a table with quotes in the column names
create table test ("'test'" varchar, "'test2'" varchar, "'test3'" varchar);
insert into test VALUES ('a', 'x', 'aa'), ('b','y', 'bb'), ('c', 'z', 'cc');
copy test to '/tmp/escape_quote' (format csv, partition_by '''test2'',''test3''');

0 rows in set. Query took 0.008 seconds.

+-------+
| count |
+-------+
| 3     |
+-------+
1 row in set. Query took 0.009 seconds.

+-------+
| count |
+-------+
| 3     |
+-------+
1 row in set. Query took 0.029 seconds.

❯ -- read back from the table
CREATE EXTERNAL TABLE validate_partitioned_escape_quote STORED AS CSV
LOCATION '/tmp/escape_quote/' PARTITIONED BY ("'test2'", "'test3'");

0 rows in set. Query took 0.004 seconds.

❯ -- This panics
select * from validate_partitioned_escape_quote;

thread 'thread 'tokio-runtime-workertokio-runtime-worker' panicked at ' panicked at /Users/andrewlamb/Software/arrow-datafusion/datafusion/core/src/datasource/physical_plan/file_scan_config.rs/Users/andrewlamb/Software/arrow-datafusion/datafusion/core/src/datasource/physical_plan/file_scan_config.rs::248:thread '54248:
:tokio-runtime-workerindex out of bounds: the len is 0 but the index is 054' panicked at
/Users/andrewlamb/Software/arrow-datafusion/datafusion/core/src/datasource/physical_plan/file_scan_config.rs:248:
:index out of bounds: the len is 0 but the index is 054
:
index out of bounds: the len is 0 but the index is 0
stack backtrace:
   0: rust_begin_unwind
             at /rustc/07dca489ac2d933c78d3c5158e3f43beefeb02ce/library/std/src/panicking.rs:645:5
   1: core::panicking::panic_fmt
             at /rustc/07dca489ac2d933c78d3c5158e3f43beefeb02ce/library/core/src/panicking.rs:72:14
   2: core::panicking::panic_bounds_check
             at /rustc/07dca489ac2d933c78d3c5158e3f43beefeb02ce/library/core/src/panicking.rs:208:5
   3: datafusion::datasource::physical_plan::file_scan_config::PartitionColumnProjector::project
   4: <datafusion::datasource::physical_plan::file_stream::FileStream<F> as futures_core::stream::Stream>::poll_next
   5: datafusion_physical_plan::stream::RecordBatchReceiverStreamBuilder::run_input::{{closure}}
   6: tokio::runtime::task::core::Core<T,S>::poll
   7: tokio::runtime::task::harness::Harness<T,S>::poll
   8: tokio::runtime::scheduler::multi_thread::worker::Context::run_task
   9: tokio::runtime::scheduler::multi_thread::worker::Context::run
  10: tokio::runtime::context::runtime::enter_runtime
  11: tokio::runtime::scheduler::multi_thread::worker::run
  12: <tokio::runtime::blocking::task::BlockingTask<T> as core::future::future::Future>::poll
  13: tokio::runtime::task::core::Core<T,S>::poll
  14: tokio::runtime::task::harness::Harness<T,S>::poll
  15: tokio::runtime::blocking::pool::Inner::run
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.

Expected behavior
Note the data is written correctly
andrewlamb@Andrews-MacBook-Pro:~/Software/influxdb_iox$ find /tmp/escape_quote
/tmp/escape_quote
/tmp/escape_quote/'test2'=x
/tmp/escape_quote/'test2'=x/'test3'=aa
/tmp/escape_quote/'test2'=x/'test3'=aa/3zMw255TXFQxId14.csv
/tmp/escape_quote/'test2'=y
/tmp/escape_quote/'test2'=y/'test3'=bb
/tmp/escape_quote/'test2'=y/'test3'=bb/3zMw255TXFQxId14.csv
/tmp/escape_quote/'test2'=z
/tmp/escape_quote/'test2'=z/'test3'=cc
/tmp/escape_quote/'test2'=z/'test3'=cc/3zMw255TXFQxId14.csv

andrewlamb@Andrews-MacBook-Pro:~/Software/influxdb_iox$ cat /tmp/escape_quote/\'test2\'\=x/\'test3\'\=aa/3zMw255TXFQxId14.csv
'test'
a

Additional context
@devinjdangelo  found this in https://github.com/apache/arrow-datafusion/pull/9240

I tried searching documentation of various engines to see if ' is allowed in partition columns. I didn't find anything concrete.
However, I tried the equivalent example in DuckDB and it does work (see below). It would probably be best to tighten up our parsing of non standard column names. It may be easier to make this robust by extending sqlparser-rs upstream. The recent discussion on the mailing list is relevant<EMAIL_ADDRESS>devinjd@devinjd$ ./duckdb
v0.10.0 20b1486d11
Enter ".help" for usage hints.
Connected to a transient in-memory database.
Use ".open FILENAME" to reopen on a persistent database.
D create table test ("'test'" varchar, "'test2'" varchar, "'test3'" varchar);
D insert into test VALUES ('a', 'x', 'aa'), ('b','y', 'bb'), ('c', 'z', 'cc');
D select * from test;
┌─────────┬─────────┬─────────┐
│ 'test'  │ 'test2' │ 'test3' │
│ varchar │ varchar │ varchar │
├─────────┼─────────┼─────────┤
│ a       │ x       │ aa      │
│ b       │ y       │ bb      │
│ c       │ z       │ cc      │
└─────────┴─────────┴─────────┘
D copy test to '/tmp/escape_quote' (format csv, partition_by ('''test2''','''test3'''));
D select * from read_csv('/tmp/escape_quote/*/*/*.csv', hive_partitioning=1, header=true);
┌─────────┬─────────┬─────────┐
│ 'test'  │ 'test2' │ 'test3' │
│ varchar │ varchar │ varchar │
├─────────┼─────────┼─────────┤
│ a       │ x       │ aa      │
│ b       │ y       │ bb      │
│ c       │ z       │ cc      │
└─────────┴─────────┴─────────┘

Another case:
DataFusion CLI v37.0.0
❯ create external table test123(a string, ```a=b``` string) stored as parquet location '/tmp/test123/' partitioned by (`a=b`);
0 row(s) fetched.
Elapsed 0.002 seconds.

❯ insert into test123 select 'a', 'b';
+-------+
| count |
+-------+
| 1     |
+-------+
1 row(s) fetched.
Elapsed 0.007 seconds.

❯ select * from test123;
thread 'main' panicked at /home/jeffrey/Code/arrow-datafusion/datafusion/core/src/datasource/physical_plan/file_scan_config.rs:261:54:
index out of bounds: the len is 0 but the index is 0
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
datafusion-cli$

Files exist on disk:
datafusion-cli$ ll /tmp/test123/
total 0
drwxr-xr-x   2 jeffrey jeffrey   60 Apr  5 20:43 '%60a=b%60=b'/
drwxrwxrwt 119 root    root    2.6K Apr  5 20:43  ../
drwxr-xr-x   3 jeffrey jeffrey   60 Apr  5 20:43  ./
datafusion-cli$ ll /tmp/test123/\%60a=b\%60=b/
total 4.0K
-rw-r--r-- 1 jeffrey jeffrey 282 Apr  5 20:43 33T7kTcVyecaVk07.parquet
drwxr-xr-x 3 jeffrey jeffrey  60 Apr  5 20:43 ../
drwxr-xr-x 2 jeffrey jeffrey  60 Apr  5 20:43 ./
datafusion-cli$

Note I had to get cheeky with the column and partition names, as just this didn't work:
DataFusion CLI v37.0.0
❯ create external table test123(a string, `a=b` string) stored as parquet location '/tmp/test123/' partitioned by (`a=b`);
Arrow error: Schema error: Unable to get field named "`a=b`". Valid fields: ["a", "a=b"]


Handle case where backend returns nil in M-x straight-get-recipe

This seems to have been fixed at some point (perhaps when I introduced recipe lookup caching?), although I'm not sure when :P


More header_rewrite examples

This didn't cherry pick cleanly to 7.1.x branch. Miles, if we want to add this to that branch, can you make another PR against the branch please?


[BSO] add =monster for all custom bosses

🚀 Feature Proposal
there is a =monster mally & =monster treebeard, but none of the other custom bosses.

This has been fixed with show_info

Still doesnt apply to some custom bso monsters


slot migrate not support json type

Search before asking

[X] I had searched in the issues and found no similar issues.

Version
v2.9.0
Minimal reproduce step
1、start slot migrate with json type data
What did you expect to see?
slot migrate work well on json data type
What did you see instead?
not working
Anything Else?
No response
Are you willing to submit a PR?

[ ] I'm willing to submit a PR!

@Genuineh will support this at the next release.

Thanks, #2580 seems can be completed together and when is the next release version expected to be launched? @git-hulk

@PragmaTwice @git-hulk  Thanks, slot migration work well on raw-key-value mode. By the way, the company I am in has a strong intention to use a database with a kv structure based on rocksdb to replace databases such as mongodb to store a large amount of business data. However, formal businesses often have various analysis or other needs, and these needs are usually handled by some specialized databases. At present, kvrocks may only temporarily provide migration to redis, which is not very friendly for those with such business needs. The current structure has a not-low learning cost for expansion. If the community needs to do a large amount of adaptation for different databases, it is a huge project. If a standard scheme that users can quickly expand can be provided, these tasks don't need to be done and users' different needs can be met. For a simple scheme, for example, it can directly connect to raw-key-value. Users only need to rewrite a writer and a checker.


404 page when entering results url of non-existent poll

Entering URL such as http://<IP_ADDRESS>:8000/polls/9999/results/ shows a 404 page.

Thanks for the reviews I have fixed the 404 error page for the results page now.


Checkbox toggle

Tips

Use the non-standard appearance: none; to reset the native look & feel. Alternatively hide the element using display: none; or any other technique we've used before.
An <input type='checkbox'> is toggled when a containing <label> is pressed.
Use the :checked pseudo-class, optionally with :not(:checked) inversion.
Maybe look into + or ~ sibling selectors.

Source
Not a solution, just a UX mockup animation.
http://gibbon.co/c/d5a67253-2672-4a97-82c3-d69ebcf99b1c/life-at-khan-academy

http://codepen.io/cbas/pen/KVXjZy?editors=110

http://codepen.io/sevresbabylone/pen/RrLzJr

http://codepen.io/lanbau/pen/GoMbGq

Because I initially chose to style the label rather than the span within the label, I'm now facing difficulty changing the border colour when the input is checked! http://codepen.io/tongrhj/pen/KVXjmX

@lanbau Looking good. Couple of remarks:

Be consistent (OCD-level) with white space in the CSS/HTML.
LI can only exist inside OL or UL. Not randomly inside any elements; that's what SPAN/DIV are for (non-semantic markup).

@tongrhj

Target the parent using ! (CSS level 4 ... not sure if supported anywhere yet) https://www.w3.org/TR/selectors4/#subject
The Jade template generates duplicate IDs.
Outline on the checkbox is a bit weird. Should be on the entire target?
Use autoprefixer from the CSS settings for the -webkit-appearance: none; vendor prefix.
Use semantic names for the colours. Otherwise the purpose is defeated. $checkboxblue can just be $checkboxActive so you won't need to update the variable names everywhere when the designer changes their mind.

ok updated! thanks @cbas.

Used javascript to select the parent label element instead
Setting outline: 0 on the label element removed the strange blue outline around the checkbox

A combination of today's exercises... http://codepen.io/calvintan/pen/gPGVbW

http://codepen.io/yongjun21/pen/XXzGyP


Introduce 'lenient' mode for hostname validation.

This PR should address the long-standing #73 issue. It is now possible to enable a more permissive hostname validation using the lenientHostnameValidation option:
> var tldjs = require('tldjs');
> tldjs.isValid('foo.bar_baz.com')
false
> var customTld = tldjs.fromUserSettings({ lenientHostnameValidation: true });
> customTld.isValid('foo.bar_baz.com')
true

Good thinking and thank you! 🙂 👍

I'm pondering if the setting should be only global (like you implemented) or global +local (at a function level with an extra options parameter).
I'd be curious to hear more from @ikari-pl, @ZLightning, @caffeinewriter and @nebulade on this. They are users who expressed some interest on this topic in #73.

When it is lenient, does it means it is less compliant with the RFC and closer to the real world implementations?
I'm pondering if we should follow the de facto usage (lenient) or the RFC (not lenient).

Regarding the settings, I was thinking that since we already have a way to specify it via the factory, it could be a good starting point. No strong opinion on that though.
By lenient I meant more permissive. Which means that by default, underscores are not allowed in hostnames (strict mode, which is the current behavior). When lenientHostnameValidation is set to true, then underscores are allowed. This was based on the discussion at #73, but it could very well be that other constraints are relaxed in the future?

@oncletom Since this PR also introduces some options, shall we open an issue about the API and the way to specify options and have a more high-level discussion there?

I like the idea of lenient mode, but I think the implementation should include an option for only allowing the lenient mode on subdomains of the TLD/public suffix.  Another option could be 3 modes, strict, lenient, and sloppy where sloppy would allow even base domains to contain non-standard characters, lenient would test that the base domain is RFC compliant and skip validating only the subdomain portion, and strict would test the whole FQDN for RFC compliance.
For general purposes, validating the base domain would still be of some importance.  Registrars generally follow the RFCs on valid characters, though I am not sure all public suffixes would validate the subdomains users are allowed to create.
I believe the default should be what I describe as "lenient" mode, with "strict" and "sloppy" being options.

I do support the idea of the fix, of course, as the problem was pretty annoying when it hit me.
It feel weird to me, though, that I suddenly have to create an instance of tldjs where I didn't. It could be, as @oncletom suggested, a per-call option. It would feel cleaner to only add this in the code where I need a specific mode, and would look more verbose in the app logic.
Either way, happy to see the discussion here.

Thanks a lot for your feedback!
@ZLightning I understand that having more flexibility would be needed in some cases. On the other hand, we need to think about what goals tld.js has. We had a similar discussion regarding URL parsing, which is not a trivial task. In the end we decided to keep the good-enough implementation from Node.js but allow users of tld.js to plug their own hostname extraction functions to customize to their needs. Regarding domain validation, I wonder if a similar approach would make sense? The current function is simple but maybe good enough for most needs. Also, it is relatively straightforward to use, fast to run, and should work in most cases. When this is not enough, we could consider plugging another validation function to be used in tld.js instead (which could come from a validation library for example). In fact, to go a step further, it would make sense to allow overriding all functions from the public API which do not have to do with public suffix/tld extraction, but are provided for convenience (isValid, isIp, extractHostname).
@ikari-pl I agree with you, creating a new instance of tld.js to allow any form of customization is cumbersome. On the other hand, we need to be careful, as options applied to one function might impact the behavior of other functions of tld.js, in which case applying options globally by instantiating a new tld.js makes it clear that it might impact different parts of the library. In the end this is a trade-off, and maybe we need to allow both: give an extra argument to isValid, and also a global option which would change the behavior of isValid, as well as all other methods of the API depending on isValid.
What about this:

We keep the same API, but some function start accepting optional arguments to customize their behavior (e.g.: isValid accepts some option for lenient mode, as suggested by @ikari-pl)
fromUserSettings will also accept options to customize the behavior, and will customize methods the same way users can: by using their optional arguments.

Because these changes could introduce some breaking changes (not all of them of course), maybe this could be done in two times?

Introduce optional options for some methods such as isValid could be done transparently, without breaking compatibility. That would be a low hanging-fruit and could already allow people to benefit from more granular customization of individual methods of tld.js.
We could discuss deeper changes in the API, as initiated by @oncletom in #124 where breaking changes can be considered.

What do you think?

I'm all in favour of going step by step 👍
I don't know how many people assume it's strict or lenient at the moment and a semver bump is a good occasion to clarify the API intents and to set the good defaults.

I think it is essential to distinguish between domain part and the host part. There are possibly no underscore domains available even though RFC would allow but there are tons of real world _ hostnames out there. Especially from big providers. I personally think you should allow _ in host part without any setting by default. I think we all want to use this wonderful repo in real world. I am parsing billions of logs daily with it and had to make some workarounds to allow _ in host.

@taskinosman Thanks for your comment. As mentioned in a previous comment, if you need this change quickly you could give tldts a shot. By default it will allow _ in both host and domain parts, would that work for you? If not, we can work to make the behavior more accurate.


Feature/255 serve single header file

As a result of the discussion #255, it's been decided that fkYAML provides its single-header-file version.
The single-header-file version is generated by running make amalgamate which utilizes the amalgamation tool (a third-party Python script imported from here).
See README.md and CONTRIBUTING.md for detailed information.

Pull Request Checklist
Read the CONTRIBUTING.md file for detailed information.

[x] Changes are described in the pull request or in a referenced issue.
[x] The test suite compiles and runs without any error.
[x] The code coverage on your branch is 100%.
[x] The documentation is updated if you added/changed a feature.

Please don't

The C++11 support varies between different compilers and versions. Please note the list of supported compilers. Some compilers like GCC 4.7 (and earlier), Clang 3.3 (and earlier), or Microsoft Visual Studio 13.0 and earlier are known not to work due to missing or incomplete C++11 support. Please refrain from proposing changes that work around these compiler's limitations with #ifdefs or other means.
Please refrain from proposing changes that would break YAML specifications. If you propose a conformant extension of YAML to be supported by the library, please motivate this extension.
Please do not open pull requests that address multiple issues.

coverage: 100.0%. remained the same
when pulling 169c0ba0796c4730f8ff3f219714c61b4e54d7cf on feature/255_serve_single_header_file
into 477422f96f97d1dcee435cd3a7a3aeb3d6da8e7f on develop.


Fixes Favicons not being generated by FaviconsWebpackPlugin and Search

As pointed out by @codedmonkey on podcastindex.social, the favicon was not being specified correctly.
This commit sets up the FaviconsWebpackPlugin to generate all the images from the svg needed for favicon and pinned mobile shortcuts.
While doing this investigation, I found that the shortcut adds ?homescreen=1 to the URL. The current code for getting the search parameter accepted any parameter instead of only using ?q=. The dependency query-string was added to do the parsing of the URL parameters and cleanSearchQuery was updated to use only the q parameter.
The files in www were updated by running the yarn build command.

Hey @stevencrader! Can you pull and merge master? I gitignored the www folder with input from @codedmonkey so we can make sure to avoid large PR file changes with www.

I pushed a new commit based on the current master.


Feature/adding AWS secret manager

Hi guys,
I need to include SecretManager as part of terratest AWS module.
I share the gist file with test passed:
https://gist.github.com/ismferd/466cb3eb14331de0760181681c2f73da
Thanks for this awesome project!

Thanks for your contribution! However, given the relative simplicity of the functions, I am not sure if we will include this into our API. Could use a second opinion from @brikis98 on this.
To explain, as Terratest has grown (we're approaching 4,000 stars!), we've had to think more about what types of helper functions we want in the library to make sure it doesn't become so big as to be unusable. Please check out the contribution guidelines, especially the part about infrastructure and validation helpers for more details. These secretsmanager helpers meet requirements for "platforms," "popularity," and "creating infrastructure," but fall short on "complexity" (i.e., these are direct wrappers around the AWS SDK as opposed to something more complicated).

@yorinasub17 thanks for your answer, basically one of the thing that I want to achieve with that is the possibility of add the retry.DowithRetry in order to get my resource as I see in other parts of your project.

That's probably a special case but I could extend the SecretManager API in a close future.
So, I'm looking forward your answer.

add the retry.DowithRetry in order to get my resource as I see in other parts of your project.

Can you elaborate a bit further? E.g., would something like the following work for you:
sess, err := terraws.NewAuthenticatedSession(region)
require.NoError(t, err)
secretsMgr := secretsmanager.New(sess)
secret := retry.DoWithRetry(... omitted for brevity ..., func() (string, error) {
  resp, err := secretsMgr.GetSecretValue(&secretsmanager.GetSecretValueInput{SecretId: aws.String(secretID)})
  if err != nil {
    return "", err
  }
  return aws.StringValue(resp.SecretString), nil
})

I could extend the SecretManager API in a close future.

If you can describe some of your use cases here that may be difficult to do with the SDK directly or is not obvious, that can help in deciding if we should include this.

Currently, the case is basically that: get a secret previously added into secretmanager.
Trying to abstract me of AWS API (things such as sessions) and just making sense to my test code.
Thanks a lot @yorinasub17

A few thoughts:

As-written, this does seem like a very thin wrapper over the AWS SDK. Such thin wrappers require effort to document, find, learn, maintain, extend to handle other use cases, etc, so as Yori said, I'm not sure it's worth adding to Terratest.
AWS Secrets Manager is eventually consistent, so after you write a secret, you do have to use retries to wait until you can read that secret back. Adding that to this PR would make it slightly more compelling, but (a) we already have a retry helper, so as Yori showed, you can do that in your own code and (b) not all use cases would need retries.

So I think I'm in agreement with Yori that this probably doesn't quite meet the bar of being complicated enough to merit creating an abstraction for it in Terratest.

@brikis98 thanks for your answer,
I only kept in mind to continue with the abstraction of the aws module that you have.
If you don't see appropriate to have an abstraction of secret manager I'm going to use @yorinasub17 tip.
Thank both for your quick answer.


Remove stylus from direct dependencies

Since stylus-loader is not really depends on particular stylus
package version, it makes sense to remove it from the dependencies,
so a user could manually manage stylus version (upgrade, lock etc).

:+1: I agree with this.
If something in upstream stylus breaks you have to cd inside stylus-loader to change the stylus version.
The note in README reads odd (I'm assuming english isn't your first language).
I would revise to something like this:
> *Note*: `stylus-loader` does not include `stylus`, so it must be installed alongside `stylus-loader`.

Note: stylus-loader does not include stylus, so it must be installed alongside stylus-loader.

Simple and concise.

Currently as a dependency, if stylus changes, we are still using the version tagged that works. So you should never need to cd inside stylus-loader if something upstream breaks.
If we don't specify the dependency, what happens when stylus changes their API? Wouldn't that break this module? The only downside to us having stylus as a dependency, is you might need to wait a little longer for us to catch up when it updates, which is usually like a couple days.
That seems less bad then potentially breaking builds only to find out it's because some version of stylus doesn't work with some version of stylus-loader.

might need to wait a little longer for us to catch up when it updates, which is usually like a couple days.

@shama Not in practice though. This loader currently depends on ^0.52.4, released in September. 0.53.0 was released in December, and 0.54.0 was released in March. So we're currently 3 months and waiting for an already outdated version, and 3 weeks waiting for the current latest release.

Fair enough. We can try it this other way and see how it goes.

Sweet, thanks! Should stylus be a peer dep?

I don't think it should.  Neither of which will help the user understand that stylus-loader is now broken because they didn't install the correct version of stylus.


lemminx: enable

Does that work with custom schemas like we use for c3d2-web?

It probably is, at least it works with pom.xml.

That sounds promissing.


Function won't download to path.

When I run the program it only downloads the first time. When I run again images don't download to path.

fixed with sub folder path


New Providers

Statusphere currently has 657 companies in its database.
Of those, 298 are not indexed
curl 'https://statusphere.metoro.io/api/v1/statusPages' | jq '.statusPages[] | select(.isIndexed == false) | length' | wc -l
This is because the scrapers are unable to parse them, so we need more providers.
We should figure out which providers to prioritize based on the number of pages it would enable the scraping of.

From a random sampling of around 30 pages. It seems like the most beneficial would be

status.io
hund.io
Rss reader


Unable to connect to the event hub emulator using PySpark

Describe the bug
I am getting connection aborted/refused when trying to connect to the eventhub emulator through pyspark
To Reproduce
Steps to reproduce the behavior:
Pre-requisite is to install pyspark library version: 3.5.0 or 3.5.1 from pypi with pip install pyspark==<version_number>
A basic spark streaming application:
from pyspark.sql import SparkSession

spark = (SparkSession.builder.appName("spark-app")
         .config("spark.jars.packages",
                 "com.microsoft.azure:azure-eventhubs-spark_2.12:2.3.22")
         .getOrCreate())

eventhub_connection_string = "Endpoint=sb://localhost;SharedAccessKeyName=RootManageSharedAccessKey;SharedAccessKey=SAS_KEY_VALUE;UseDevelopmentEmulator=true;EntityPath=eh1"

eventhub_config = {
    "eventhubs.connectionString": spark.sparkContext._jvm.org.apache.spark.eventhubs.EventHubsUtils.encrypt(
        eventhub_connection_string
    )
}

df = spark.readStream.format("eventhubs").options(**eventhub_config).load()

df.writeStream.format("console").outputMode("append").start().awaitTermination()

Expected behavior
The pyspark application should start consuming data from the eventhub emulator streaming source continuously as it happens with a real Azure EventHub
Desktop:

OS: MacOS Sonoma 14.5 w/ M2 processor

Image Platform-Architecture Used:

Linux-arm64, docker image: latest

Docker Version:

Docker Version: 26.1.0-rd, build cca8e72

Arguments && Environment variables to start Emulator:

ACCEPT_EULA: "Y"
CONFIG_PATH: "eventhubConfig.json"

Emulator Launch Method:

Direct Docker compose, exactly as per the official documentation. I also tried mapping local ports 5671 and 5672 to container's 5671/5672 ports

Additional context:
I can send/consume events from the emulator with a python script using the standard python EventHubConsumerClient library, but this is not my use case as I do need to consume with pyspark, so I guess something may be wrong somehow with the connectivity or maybe even with the Azure Eventhub maven project that somehow may not be compatible with the event hub emulator
Log stack:
Setting default log level to "WARN".
To adjust logging level use sc.setLogLevel(newLevel). For SparkR, use setLogLevel(newLevel).
24/07/03 11:10:24 WARN ResolveWriteToStream: Temporary checkpoint location created which is deleted normally when the query didn't fail: /private/var/folders/xl/1f8_mm81143b8xqvtpfqzv7w0000gn/T/temporary-9dbef554-4b26-4836-863d-a07f4b7e8228. If it's required to delete it under any circumstances, please set spark.sql.streaming.forceDeleteTempCheckpointLocation to true. Important to know deleting temp checkpoint folder is best effort.
24/07/03 11:10:24 WARN ResolveWriteToStream: spark.sql.adaptive.enabled is not supported in streaming DataFrames/Datasets and will be disabled.
24/07/03 11:10:24 WARN ConnectionHandler: onTransportError hostname[localhost:5671], connectionId[MF_2b6422_1720001424588], error[org.apache.qpid.proton.engine.TransportException: connection aborted]
24/07/03 11:10:24 WARN MessagingFactory: onConnectionError messagingFactory[MF_2b6422_1720001424588], hostname[localhost], error[org.apache.qpid.proton.engine.TransportException: connection aborted]
24/07/03 11:10:24 WARN MessagingFactory: onConnectionError messagingFactory[MF_2b6422_1720001424588], hostname[localhost], open hasn't complete, stopping the reactor
24/07/03 11:10:24 WARN MessagingFactory: onConnectionError messagingFactory[MF_2b6422_1720001424588], hostname[localhost], error[org.apache.qpid.proton.engine.TransportException: connection aborted]
24/07/03 11:10:24 WARN MessagingFactory: onConnectionError messagingFactory[MF_2b6422_1720001424588], hostname[localhost], closing current connection
24/07/03 11:10:24 WARN MessagingFactory: messagingFactory[MF_2b6422_1720001424588], hostName[localhost], message[stopping the reactor because thread was interrupted or the reactor has no more events to process.]
24/07/03 11:10:24 WARN ConnectionHandler: onTransportError hostname[localhost:5671], connectionId[MF_39118f_1720001424652], error[org.apache.qpid.proton.engine.TransportException: connection aborted]
24/07/03 11:10:24 WARN MessagingFactory: onConnectionError messagingFactory[MF_39118f_1720001424652], hostname[localhost], error[org.apache.qpid.proton.engine.TransportException: connection aborted]
24/07/03 11:10:24 WARN MessagingFactory: onConnectionError messagingFactory[MF_39118f_1720001424652], hostname[localhost], open hasn't complete, stopping the reactor
24/07/03 11:10:24 WARN MessagingFactory: messagingFactory[MF_39118f_1720001424652], hostName[localhost], message[stopping the reactor because thread was interrupted or the reactor has no more events to process.]
24/07/03 11:10:24 WARN ConnectionHandler: onTransportError hostname[localhost:5671], connectionId[MF_c187ce_1720001424669], error[org.apache.qpid.proton.engine.TransportException: connection aborted]
24/07/03 11:10:24 WARN MessagingFactory: onConnectionError messagingFactory[MF_c187ce_1720001424669], hostname[localhost], error[org.apache.qpid.proton.engine.TransportException: connection aborted]
24/07/03 11:10:24 WARN MessagingFactory: onConnectionError messagingFactory[MF_c187ce_1720001424669], hostname[localhost], open hasn't complete, stopping the reactor
24/07/03 11:10:24 WARN MessagingFactory: messagingFactory[MF_c187ce_1720001424669], hostName[localhost], message[stopping the reactor because thread was interrupted or the reactor has no more events to process.]
24/07/03 11:10:24 WARN ConnectionHandler: onTransportError hostname[localhost:5671], connectionId[MF_c128fa_1720001424685], error[org.apache.qpid.proton.engine.TransportException: connection aborted]
24/07/03 11:10:24 WARN MessagingFactory: onConnectionError messagingFactory[MF_c128fa_1720001424685], hostname[localhost], error[org.apache.qpid.proton.engine.TransportException: connection aborted]
24/07/03 11:10:24 WARN MessagingFactory: onConnectionError messagingFactory[MF_c128fa_1720001424685], hostname[localhost], open hasn't complete, stopping the reactor
24/07/03 11:10:24 WARN MessagingFactory: messagingFactory[MF_c128fa_1720001424685], hostName[localhost], message[stopping the reactor because thread was interrupted or the reactor has no more events to process.]
24/07/03 11:10:24 WARN ConnectionHandler: onTransportError hostname[localhost:5671], connectionId[MF_4a2afc_1720001424701], error[org.apache.qpid.proton.engine.TransportException: connection aborted]
24/07/03 11:10:24 WARN MessagingFactory: onConnectionError messagingFactory[MF_4a2afc_1720001424701], hostname[localhost], error[org.apache.qpid.proton.engine.TransportException: connection aborted]
24/07/03 11:10:24 WARN MessagingFactory: onConnectionError messagingFactory[MF_4a2afc_1720001424701], hostname[localhost], open hasn't complete, stopping the reactor
24/07/03 11:10:24 WARN MessagingFactory: messagingFactory[MF_4a2afc_1720001424701], hostName[localhost], message[stopping the reactor because thread was interrupted or the reactor has no more events to process.]
24/07/03 11:10:24 WARN ConnectionHandler: onTransportError hostname[localhost:5671], connectionId[MF_9c3b23_1720001424716], error[org.apache.qpid.proton.engine.TransportException: connection aborted]
24/07/03 11:10:24 WARN MessagingFactory: onConnectionError messagingFactory[MF_9c3b23_1720001424716], hostname[localhost], error[org.apache.qpid.proton.engine.TransportException: connection aborted]
24/07/03 11:10:24 WARN MessagingFactory: onConnectionError messagingFactory[MF_9c3b23_1720001424716], hostname[localhost], open hasn't complete, stopping the reactor
24/07/03 11:10:24 WARN MessagingFactory: messagingFactory[MF_9c3b23_1720001424716], hostName[localhost], message[stopping the reactor because thread was interrupted or the reactor has no more events to process.]
24/07/03 11:10:24 WARN ConnectionHandler: onTransportError hostname[localhost:5671], connectionId[MF_777ada_1720001424733], error[org.apache.qpid.proton.engine.TransportException: connection aborted]
24/07/03 11:10:24 WARN MessagingFactory: onConnectionError messagingFactory[MF_777ada_1720001424733], hostname[localhost], error[org.apache.qpid.proton.engine.TransportException: connection aborted]
24/07/03 11:10:24 WARN MessagingFactory: onConnectionError messagingFactory[MF_777ada_1720001424733], hostname[localhost], open hasn't complete, stopping the reactor
24/07/03 11:10:24 WARN MessagingFactory: messagingFactory[MF_777ada_1720001424733], hostName[localhost], message[stopping the reactor because thread was interrupted or the reactor has no more events to process.]
24/07/03 11:10:24 WARN ConnectionHandler: onTransportError hostname[localhost:5671], connectionId[MF_0c9fe4_1720001424749], error[org.apache.qpid.proton.engine.TransportException: connection aborted]
24/07/03 11:10:24 WARN MessagingFactory: onConnectionError messagingFactory[MF_0c9fe4_1720001424749], hostname[localhost], error[org.apache.qpid.proton.engine.TransportException: connection aborted]
24/07/03 11:10:24 WARN MessagingFactory: onConnectionError messagingFactory[MF_0c9fe4_1720001424749], hostname[localhost], open hasn't complete, stopping the reactor
24/07/03 11:10:24 WARN MessagingFactory: messagingFactory[MF_0c9fe4_1720001424749], hostName[localhost], message[stopping the reactor because thread was interrupted or the reactor has no more events to process.]
24/07/03 11:10:24 WARN ConnectionHandler: onTransportError hostname[localhost:5671], connectionId[MF_593bf7_1720001424765], error[org.apache.qpid.proton.engine.TransportException: connection aborted]
24/07/03 11:10:24 WARN MessagingFactory: onConnectionError messagingFactory[MF_593bf7_1720001424765], hostname[localhost], error[org.apache.qpid.proton.engine.TransportException: connection aborted]
24/07/03 11:10:24 WARN MessagingFactory: onConnectionError messagingFactory[MF_593bf7_1720001424765], hostname[localhost], open hasn't complete, stopping the reactor
24/07/03 11:10:24 WARN MessagingFactory: messagingFactory[MF_593bf7_1720001424765], hostName[localhost], message[stopping the reactor because thread was interrupted or the reactor has no more events to process.]
24/07/03 11:10:24 WARN ConnectionHandler: onTransportError hostname[localhost:5671], connectionId[MF_76334d_1720001424780], error[org.apache.qpid.proton.engine.TransportException: connection aborted]
24/07/03 11:10:24 WARN MessagingFactory: onConnectionError messagingFactory[MF_76334d_1720001424780], hostname[localhost], error[org.apache.qpid.proton.engine.TransportException: connection aborted]
24/07/03 11:10:24 WARN MessagingFactory: onConnectionError messagingFactory[MF_76334d_1720001424780], hostname[localhost], open hasn't complete, stopping the reactor
24/07/03 11:10:24 WARN MessagingFactory: messagingFactory[MF_76334d_1720001424780], hostName[localhost], message[stopping the reactor because thread was interrupted or the reactor has no more events to process.]
24/07/03 11:10:24 WARN ConnectionHandler: onTransportError hostname[localhost:5671], connectionId[MF_54eab3_1720001424795], error[org.apache.qpid.proton.engine.TransportException: connection aborted]
24/07/03 11:10:24 WARN MessagingFactory: onConnectionError messagingFactory[MF_54eab3_1720001424795], hostname[localhost], error[org.apache.qpid.proton.engine.TransportException: connection aborted]
24/07/03 11:10:24 WARN MessagingFactory: onConnectionError messagingFactory[MF_54eab3_1720001424795], hostname[localhost], open hasn't complete, stopping the reactor
24/07/03 11:10:24 WARN MessagingFactory: onConnectionError messagingFactory[MF_54eab3_1720001424795], hostname[localhost], error[org.apache.qpid.proton.engine.TransportException: connection aborted]
24/07/03 11:10:24 WARN MessagingFactory: onConnectionError messagingFactory[MF_54eab3_1720001424795], hostname[localhost], closing current connection
24/07/03 11:10:24 WARN MessagingFactory: messagingFactory[MF_54eab3_1720001424795], hostName[localhost], message[stopping the reactor because thread was interrupted or the reactor has no more events to process.]
24/07/03 11:10:24 ERROR MicroBatchExecution: Query [id = 33c12279-1a58-465b-900b-0ba12751cab0, runId = a5385b7c-17b4-4aa6-8524-b4c0fd5eafd1] terminated with error
com.microsoft.azure.eventhubs.EventHubException: org.apache.qpid.proton.engine.TransportException: connection aborted
	at com.microsoft.azure.eventhubs.impl.ExceptionUtil.toException(ExceptionUtil.java:74)
	at com.microsoft.azure.eventhubs.impl.MessagingFactory.onConnectionError(MessagingFactory.java:525)
	at com.microsoft.azure.eventhubs.impl.ConnectionHandler.onTransportError(ConnectionHandler.java:220)
	at org.apache.qpid.proton.engine.BaseHandler.handle(BaseHandler.java:191)
	at org.apache.qpid.proton.engine.impl.EventImpl.dispatch(EventImpl.java:108)
	at org.apache.qpid.proton.reactor.impl.ReactorImpl.dispatch(ReactorImpl.java:324)
	at org.apache.qpid.proton.reactor.impl.ReactorImpl.process(ReactorImpl.java:291)
	at com.microsoft.azure.eventhubs.impl.MessagingFactory$RunReactor.run(MessagingFactory.java:784)
	at java.base/java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:572)
	at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:317)
	at java.base/java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(ScheduledThreadPoolExecutor.java:304)
	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1144)
	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:642)
	at java.base/java.lang.Thread.run(Thread.java:1583)
Traceback (most recent call last):
  File "/Users/victor/PycharmProjects/development/spark-docker.py", line 20, in <module>
    df.writeStream.format("console").outputMode("append").start().awaitTermination()
  File "/Users/victor/PycharmProjects/development/.venv-3.11/lib/python3.11/site-packages/pyspark/sql/streaming/query.py", line 221, in awaitTermination
    return self._jsq.awaitTermination()
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/victor/PycharmProjects/development/.venv-3.11/lib/python3.11/site-packages/py4j/java_gateway.py", line 1322, in __call__
    return_value = get_return_value(
                   ^^^^^^^^^^^^^^^^^
  File "/Users/victor/PycharmProjects/development/.venv-3.11/lib/python3.11/site-packages/pyspark/errors/exceptions/captured.py", line 185, in deco
    raise converted from None
pyspark.errors.exceptions.captured.StreamingQueryException: [STREAM_FAILED] Query [id = 33c12279-1a58-465b-900b-0ba12751cab0, runId = a5385b7c-17b4-4aa6-8524-b4c0fd5eafd1] terminated with exception: org.apache.qpid.proton.engine.TransportException: connection aborted

Hi @victorfuzaro , Event Hubs emulator only supports interaction with latest SDKs. The PySpark connector’s incompatibility with the emulator stems from its use of older SDKs.
To tackle such use cases, we’re actively working on Kafka support for our emulator however there is no clear ETA yet. Once Kafka support is available, we recommend using the more regularly updated native Kafka connectors. Kindly let us know if there are additional concerns/queries.

We just released Kafka support in Event Hubs emulator @victorfuzaro . You can find more details here


Bug: When is screen 682 hand sign is not visible

pls assign me this issue

@AnshuPathak-88825 you mean the hand tool on the toolbar or the hand tool as a cursor pointer? For the former, there is a hand tool icon located on the right side of your screenshot. However, for the latter, it seems you are in responsive mode, which displays touch interactions rather than tool icons.
If I'm missing something please explain in more detail.

Screencast from 10-10-23 08:34:50 PM IST.webm
you can see when I decrease it's browser size then some of the icons disappear

Its showing on the right side

I'm sorry everyone I didn't noticed that hand pointer moved to right side

it's working fine pls close this issue


Can't edit tests.

I can't edit any tests at all, regardless of whether they're public or private, mine or other people's.
It either gives the error Not all tests inserted or Please review required fields and save again., or sometimes an infinite loop of Captchas. There also isn't an add code-snippet button as described in #247.
Tried Chrome and Edge on Windows 10, and Chrome on Mac OS X.

I met the same issue.

#236


Update XmlParser.hs

Comment in the middle of an expression seems to break build

I stumbled against this issue as well. It is now fixed in master.

@wismill so I could close this PR?

@robstewart57 Yes it can be closed.

The RDF/XML parser has been replaced by an xmlbf based re-implementation.
https://github.com/robstewart57/rdf4h/commit/eca4daefccaf005e36f2e831b966ef7ba8512258
Closing this PR.


Update docker run hello-world on ubuntu1404-20160211-1

@karajas,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot


Pivot Table - Wrong Sort Order

When I create a pivot table with a dimension in "group by" section and a numeric metric, the rows are not sorted correctly.
Expected results
Rows should be sorted from the biggest number to the lowest one
Actual results
Row are sorted as if the metric is a strinc (90 > 9.0 > 80 > 8.1 > 70 etc.)
Screenshots

How to reproduce the bug

Go to 'Chart'
Click on 'Create' / Pivot Table
Select a dimension in "group by"
Create a metric (sum(x) for instance)
Sort the metric column by clicking on the top right logo if needed

Environment
(please complete the following information):

superset version: 1.0.1
python version: 3.7.9
node.js version: doesn't apply, I run on Kubernetes, using gunicorn as server
source : Athena

Checklist
Make sure to follow these steps before submitting your issue - thank you!

[x] I have checked the superset logs for python stacktraces and included it here as text if there are any.
[x] I have reproduced the issue with at least the latest released version of superset.
[x] I have checked the issue tracker for the same issue and I haven't found one similar.

Additional context
/

Hello,
Was this bug solved in new versions of Superset ?

Hello, Was this bug solved in new versions of Superset ?

It's not fixed in master.
I guess the issues was closed as Pivot Table became deprecated, but Pivot Table v2 is lack of a user friendly sorting UI.


Option To Send Commands Through Telegram

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
At the moment, to execute commands the user
needs to ssh into a server or physically access a computer.
Describe the solution you'd like
A clear and concise description of what you want to happen.
If the user could send commands through telegram that would solve this issue (also shouldnt be too hard to implement)
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Using ssh on phone but its janky
Additional context
Add any other context or screenshots about the feature request here.

I'm surprised that no-one has suggested this before. I've thought of this myself as well, it wouldn't be too hard but requires a lot of labour due to the high amount of different functions and options for them.
Perhaps someone could start from adding some basic functions first, just to have a starting point. From there, more people would be able to add more functions in since there would already be a base.

I'm surprised that no-one has suggested this before. I've thought of this myself as well, it wouldn't be too hard but requires a lot of labour due to the high amount of different functions and options for them.
Or alternatively, it could be possible to use the predetermined input system that exists already. But that would make the TG controlling quite hard to use, since it would require to remember the exact order of options with no insight. But still, it would be beneficial for advanced users who just want to throw in commands which they very much do remember from the memory.
Perhaps someone could start from adding some basic functions first, just to have a starting point. From there, more people would be able to add more functions in since there would already be a base.

I think if we wrote a wrapper around print statements than find and replaced all the print statements, that would solve most of the issue. i might make a pr for it during christmas time when i have time unless someone else picks it up before.

I'm surprised that no-one has suggested this before. I've thought of this myself as well, it wouldn't be too hard but requires a lot of labour due to the high amount of different functions and options for them.
Or alternatively, it could be possible to use the predetermined input system that exists already. But that would make the TG controlling quite hard to use, since it would require to remember the exact order of options with no insight. But still, it would be beneficial for advanced users who just want to throw in commands which they very much do remember from the memory.
Perhaps someone could start from adding some basic functions first, just to have a starting point. From there, more people would be able to add more functions in since there would already be a base.

I think if we wrote a wrapper around print statements than find and replaced all the print statements, that would solve most of the issue. i might make a pr for it during christmas time when i have time unless someone else picks it up before.

Yes making a wrapper around print is a good idea if you want to send everything from the terminal to telegram, but keep in mind ikabot is mostly designed to work on a certain aspect ratio (adequate for a terminal) and that the messages you receive on telegram will be hideous at best and unreadable at worst.
The other issue is sending commands from telegram to ikabot, and this is a bit harder to do. I suppose that simply running a command (which we can do with the ikabot's run function) with predetermined input would do the trick, but then we'd have to add a special command line argument that will tell that instance of ikabot to send replies to telegram instead of terminal (or both).
Another option is to modify the read function to constantly poll telegram for new commands, and this would give you a more terminal-like experience (print sends to telegram and terminal, read just waits for keyboard input and/or telegram reply (whichever one comes first)).
This would require some sort of background thread on the main menu process that just polls telegram for replies every second or so and dumps them into the keyboard input buffer (as if it came from keyboard).
Keep in mind that you wouldn't actually be able to start an ikabot instance this way because it would already have to be running to read from telegram. We would need a process that is always running in the background that is simply used for spawning another ikabot instance when you send a message like "start ikabot" or something like that. Alternatively, you could just keep one instance of the main menu process open at all times and never close it, that would solve the issue as well.
I think this feature is quite useless, at least in my opinion. I'm not going to work on this but PRs are welcome, just try not to break the entire project. There are many ways to solve this and I think it's possible but mostly stupid. You want to stop using ssh on your phone because it's ugly and instead use a chat app to send commands that will be infinitely more ugly?

I'm surprised that no-one has suggested this before. I've thought of this myself as well, it wouldn't be too hard but requires a lot of labour due to the high amount of different functions and options for them.
Or alternatively, it could be possible to use the predetermined input system that exists already. But that would make the TG controlling quite hard to use, since it would require to remember the exact order of options with no insight. But still, it would be beneficial for advanced users who just want to throw in commands which they very much do remember from the memory.
Perhaps someone could start from adding some basic functions first, just to have a starting point. From there, more people would be able to add more functions in since there would already be a base.

I think if we wrote a wrapper around print statements than find and replaced all the print statements, that would solve most of the issue. i might make a pr for it during christmas time when i have time unless someone else picks it up before.

Yes making a wrapper around print is a good idea if you want to send everything from the terminal to telegram, but keep in mind ikabot is mostly designed to work on a certain aspect ratio (adequate for a terminal) and that the messages you receive on telegram will be hideous at best and unreadable at worst.
The other issue is sending commands from telegram to ikabot, and this is a bit harder to do. I suppose that simply running a command (which we can do with the ikabot's run function) with predetermined input would do the trick, but then we'd have to add a special command line argument that will tell that instance of ikabot to send replies to telegram instead of terminal (or both).
Another option is to modify the read function to constantly poll telegram for new commands, and this would give you a more terminal-like experience (print sends to telegram and terminal, read just waits for keyboard input and/or telegram reply (whichever one comes first)).
This would require some sort of background thread on the main menu process that just polls telegram for replies every second or so and dumps them into the keyboard input buffer (as if it came from keyboard).
Keep in mind that you wouldn't actually be able to start an ikabot instance this way because it would already have to be running to read from telegram. We would need a process that is always running in the background that is simply used for spawning another ikabot instance when you send a message like "start ikabot" or something like that. Alternatively, you could just keep one instance of the main menu process open at all times and never close it, that would solve the issue as well.
I think this feature is quite useless, at least in my opinion. I'm not going to work on this but PRs are welcome, just try not to break the entire project. There are many ways to solve this and I think it's possible but mostly stupid. You want to stop using ssh on your phone because it's ugly and instead use a chat app to send commands that will be infinitely more ugly?

yeah this makes sense, i will probably not work on this as i also just found a better ssh client that doesn't look so hideous. i will probably stop playing this game anyway, just hopped in for nostalgia for a bit and the sentiment already wore off.

Consider leveraging the logger library for seamless integration of a log handler for Telegram. Various Python libraries, including https://pypi.org/project/python-telegram-logger/, are available for fulfilling this requirement. The implementation involves replacing the current logging system with Python's logging library. Subsequently, replace all print statements with functions like logger.info(...).
However, it's essential to note that a pull request aimed at replacing the logging system had been previously submitted, but it was rejected by @ikabot. Consequently, I am unsure whether it's feasible to proceed with this approach in the current context.


#3: Added cvl grammar (CQL Literals) to support expressing test resul…

…ts as literals
This PR adds a "cvl" grammar that allows for CQL literals to be expressed. It uses the JavaScript ANTLR generated visitor from this grammar to then provide a JavaScript representation of any CQL literal expression.
There are still quite a few TODOs, but this covers a good 90% of the test cases in a way that doesn't involve changing the test cases, but does address the issue with having the same engine parse the expected result and produce the actual result.
With this in place, the CQF Ruler CQL Evaluation Service is up to:
pass: 1162 skip: 38 fail: 325 error: 44

One thought with going this route is that I don't think this grammar helps us achieve multi-statement tests. One of the reason we leaned towards the JSON representation is that each statement result in multi-statement CQL could be all asserted on.

Agreed it doesn't support multi-statement tests in the $cql operation, but it should in the Library/$evaluate


Support AsciiDoc admonition blocks

Admonition blocks are a cool feature of AsciiDoc:

(see here)
Unfortunately, notepadium doesn't contain the appropriate CSS:

I checked the (S)CSS of the AsciiDoctor project; integrating what they do (which may be too much?) in a meaningful way into your style is beyond me. :/ Can you please help us AsciiDoc users out and provide styling for admonition blocks?

try Asciidoctor skins ?
https://github.com/darshandsoni/asciidoctor-skins.git
config.toml
[params.assets]
css = ["css/adoc-github.css"]

Good reference, thanks!
But doing that messes with Notepadium in many ways.
This is more like it:
grep .admonitionblock ../../themes/asciidoctor-skins/css/asciidoctor.css \
    | sed -e 's/^.*\(\.admonitionblock [^,{]*\)[^{]*{/\1 {/' \
    > admonitions.css

Result:
.admonitionblock td.content>.title {line-height:1.45;color:#7a2518;font-weight:400;margin-top:0;margin-bottom:.25em}
.admonitionblock td.content>.title {text-rendering:optimizeLegibility;text-align:left;font-family:"Noto Serif","DejaVu Serif",serif;font-size:1rem;font-style:italic}
.admonitionblock>table{border-collapse:separate;border:0;background:none;width:100%}
.admonitionblock>table td.icon{text-align:center;width:80px}
.admonitionblock>table td.icon img{max-width:none}
.admonitionblock>table td.icon .title{font-weight:bold;font-family:"Open Sans","DejaVu Sans",sans-serif;text-transform:uppercase}
.admonitionblock>table td.content{padding-left:1.125em;padding-right:1.25em;border-left:1px solid #ddddd8;color:rgba(0,0,0,.6)}
.admonitionblock>table td.content>:last-child>:last-child{margin-bottom:0}
.admonitionblock td.icon [class^="fa icon-"] {font-size:2.5em;text-shadow:1px 1px 2px rgba(0,0,0,.5);cursor:default}
.admonitionblock td.icon .icon-note:before {content:"\f05a";color:#19407c}
.admonitionblock td.icon .icon-tip:before {content:"\f0eb";text-shadow:1px 1px 2px rgba(155,155,0,.8);color:#111}
.admonitionblock td.icon .icon-warning:before {content:"\f071";color:#bf6900}
.admonitionblock td.icon .icon-caution:before {content:"\f06d";color:#bf3400}
.admonitionblock td.icon .icon-important:before {content:"\f06a";color:#bf0000}

Unfortunately, the result doesn't even come close. Compare their demo

with what I'm getting:

Some issues:

The border shouldn't be there. Can't tell why they are.
No icons.

Need to set the icons: font attribute. Okay.
No Font Awsome. Adding @import doesn't help, but I note that the paging icons also don't appear?

IMHO, using web fonts doesn't fit with Notepadium, anyway ("100% Javascript free").

Regarding the icon font: I suspect that it has something to with me looking at the hugo server output in my local browser.
But nevertheless, tangent:

Is this necessary? Can we include the font as asset? (This is beyond my frontend experience and thus a question, not a PR.)

I stored the font as static/iconfont.ttf and loaded it in iconfont.css by
src: url('../iconfont.ttf') format('truetype');

Resolves, but icons still don't show. :thinking:
Ah! Of course, the admonition icons are simply not included in your minimal font. Makes sense. Paging, though, still confuses me.

FWIW, I use this script put in themes of my Hugo project to create a CSS asset that adds admonition boxes to Notepadium:
#!/usr/bin/env bash

set -eu

skin="asciidoctor"

themes_path="$(realpath "${0}" | xargs dirname)"
css_assets_path="$(realpath "${themes_path}/../assets/css/")"
admonitions_css_path="${css_assets_path}/admonitions.css"

# Copy style from chosen skin
echo "/* From ${skin} by github.com/darshandsoni/asciidoctor-skins */" \
    > "${admonitions_css_path}"
grep .admonitionblock \
      "${themes_path}/asciidoctor-skins/css/${skin}.css" \
    | sed -e 's/^.*\(\.admonitionblock [^,{]*\)[^{]*{/\1 {/' \
    >> "${admonitions_css_path}"

# Integrate into Notepadium
cat >> "${admonitions_css_path}" <<STYLE

/* Use Notepadium's icon font */
.admonitionblock td.icon [class^="fa icon-"] {
    font-family: "iconfont" !important;
    -webkit-font-smoothing: antialiased;
    -moz-osx-font-smoothing: grayscale;
}

/* Override general table styling for admonition boxes */
.admonitionblock td.icon {
    border: none !important;
}
.admonitionblock td.content {
    border-right: none !important;
    border-bottom: none !important;
    border-top: none !important;
}
STYLE

I consider this a workaround, and I wouldn't want to just add that CSS to the code styles.
Maybe you could start off a separate CSS file specific to AsciiDoc which we could add to?

Further workaround: created static/admonition_icons.ttf using fontello.com with this config:
{
  "name": "",
  "css_prefix_text": "icon-",
  "css_use_suffix": false,
  "hinting": true,
  "units_per_em": 1000,
  "ascent": 850,
  "glyphs": [
    {
      "uid": "4b20e1deee87faf4c3fab735fbd4bc1a",
      "css": "lamp",
      "code": 59392,
      "src": "entypo"
    },
    {
      "uid": "e82cedfa1d5f15b00c5a81c9bd731ea2",
      "css": "info-circled",
      "code": 59393,
      "src": "fontawesome"
    },
    {
      "uid": "c76b7947c957c9b78b11741173c8349b",
      "css": "attention",
      "code": 59394,
      "src": "fontawesome"
    },
    {
      "uid": "b035c28eba2b35c6ffe92aee8b0df507",
      "css": "attention-circled",
      "code": 59395,
      "src": "fontawesome"
    },
    {
      "uid": "254077e7c0a9e6d303b32f9c35b96a40",
      "css": "fire",
      "code": 59396,
      "src": "fontawesome"
    }
  ]
}

Changed the script above:
#!/usr/bin/env bash

set -eu

skin="asciidoctor"

themes_path="$(realpath "${0}" | xargs dirname)"
css_assets_path="$(realpath "${themes_path}/../assets/css/")"
admonitions_css_path="${css_assets_path}/admonitions.css"

echo -e "/* Generated by ${0} */\n" \
    > "${admonitions_css_path}"

# Copy style from chosen skin
echo "/* From ${skin} by github.com/darshandsoni/asciidoctor-skins */" \
    >> "${admonitions_css_path}"
grep .admonitionblock \
      "${themes_path}/asciidoctor-skins/css/${skin}.css" \
    | sed -e 's/^.*\(\.admonitionblock [^,{]*\)[^{]*{/\1 {/' \
    >> "${admonitions_css_path}"

# Integrate into Notepadium
cat >> "${admonitions_css_path}" <<STYLE

/* Use custom icon font
   Note: Generated with fontello.com using Unicode encoding */
@font-face {
    font-family: "admonition_icons";
    src: url('../admonition_icons.ttf') format('truetype');
    font-weight: normal;
    font-style: normal;
}

.admonitionblock td.icon [class^="fa icon-"] {
    font-family: "admonition_icons" !important;
    -webkit-font-smoothing: antialiased;
    -moz-osx-font-smoothing: grayscale;
    font-style: normal;
}

.admonitionblock td.icon .icon-note:before {content:"\e801" !important} /* info circled */
.admonitionblock td.icon .icon-tip:before {content:"\e800" !important} /* lamp */
.admonitionblock td.icon .icon-warning:before {content:"\e802" !important} /* attention */
.admonitionblock td.icon .icon-caution:before {content:"\e804"!important} /* fire */
.admonitionblock td.icon .icon-important:before {content:"\e803" !important} /* attention circled */

/* Override general table styling for admonition boxes */
.admonitionblock td.icon {
    border: none !important;
}
.admonitionblock td.content {
    border-right: none !important;
    border-bottom: none !important;
    border-top: none !important;
}
STYLE

This is effective:

FWIW, I have now migrated to using SVGs for these. For reference, here is how that works.
Include this in config.toml:
[markup.asciidocext.attributes]
experimental = true
icons = "image"
icontype = "svg"
iconsdir = "assets/icons"

And put the necessary images into static/assets/icons, e.g. from https://github.com/nawroth/doctools/tree/master/src/main/resources/images/icons/admon

Still a pain to get this icons working. I could get it working today by adding a slash to iconsdir and I could also remove the experimental setting:
[markup.asciidocExt.attributes]
  icons = 'image'
  icontype = 'svg'
  iconsdir = '/assets/icons'

FWIW, I have now migrated to using SVGs for these. For reference, here is how that works. Include this in config.toml:
[markup.asciidocext.attributes]
experimental = true
icons = "image"
icontype = "svg"
iconsdir = "assets/icons"

And put the necessary images into static/assets/icons, e.g. from nawroth/doctools@master/src/main/resources/images/icons/admon


Fix onChangeText not firing when clearing the value of TextInput with multiline=true on iOS

Summary:

This fix fixes the TextInput issue in #37784 with onChangeText not working on iOS only.
Changelog:

[IOS] [FIXED] - Fix onChangeText not firing when clearing the value of TextInput with multiline=true on iOS
Test Plan:

Run the repro code given in the issue (#37784).
Verified that onChangeText works after pressing the submit button.
Run the repro code from the issue (#36494) that caused this issue.
Verified that issue (#36494) is not reoccurring.

Looks like it fixes #37991 too

This PR reverts parts of this PR
This is what it fixes, link to my comment from that PR: https://github.com/facebook/react-native/pull/36930#issuecomment-1593028467
thanks @kkoudev

looking forward to getting this change :)

also would love this change!

This PR reverts parts of this PR
This is what it fixes, link to my comment from that PR: #36930 (comment)
thanks @kkoudev

Cc: @dmytrorykun

Excited to see this merged!

Leaving this here since there are multiple open PRs for this
https://github.com/facebook/react-native/pull/38166

thank you! to get this to work, apply the patch running npx patch-package react-native and then create a new build (I was missing this last step)


faiss-gpu: assert hasattr(self, 'getDevice'), 'GPU tensor on CPU index not allowed'

Due to the Faiss package, I cannot run apartment dataset completely.

Hi, it might be a hardware-related issue. Can you try the solution at https://github.com/GradientSpaces/LoopSplat/issues/5?


Saving PID of running node process into .txt file

This pull request holds necessary code for saving node process ID of the running process into a text file.

npm start works with placing the PID of the node process into temp/pid.txt file. npm stop picks the saved PID and sends a SIGINT to the running process. On receive that the running process exits and deletes the pid.txt file.


Addresses bugs with the section times

This PR Addresses a couple of bugs. Namely:
Stats only show on page refresh and not on initial load.
This bug existed because of the way Teachable use React Router to update the route from the /courses page to /courses/enrolled/<course_id> page rather than reload the page. Because I had configured the manifest match to only embed the content script on the enrolled pages, it needed a page refresh to embed correctly.
To address the fix I updated the manifest to embed the content script on the course page too and then used location.pathname to ensure it only updated the dom on the correct pages. I did have to use a Mutation Observer and this might not be the most performant way to achieve this. The alternative was to use a background.js script which seemed to introduce more bugs than it solved.
Sidebar Stats incorrectly showing on the Instructor Bio Tab
This feature essentially scans the page looking for the time stamps in the curriculum section. When you switch to the bio page, the extension no longer has access to the curriculum details and then displayed incorrect times. For now I disabled it showing on the Instructor Bio page.
Another solution would have been to store the data to local storage. I opted not to do this due to the added complexities and the potential edge case that would break the feature.
Stats showing in places that it shouldn't.
There were a couple of locations the stats showed up when you navigated around. I fixed these by ensuring two things:

It only updates the DOM on pages with /courses/enrolled/ in the url
It will only update the DOM if the new elements are not already on the page.

Thanks @MattCSmith
Merged it.


C# wrapper return value for pull_chunk is flat wrong

Unless there are further axes to grind on this, I think we should merge https://github.com/labstreaminglayer/liblsl-Csharp/pull/12 right away, because I found a bug.
This: https://github.com/labstreaminglayer/liblsl-Csharp/blob/master/LSL.cs#L652-L676 is wrong. I believe that  the return value for each overload (except for string maybe, I haven't thought about that yet) should be:
return data_buffer.GetLength(0);
Here's a picture of my debugger that proves it:

A quick explanation of what is happening here: chunk' is a class that holds some fields including the 2d array called InternalDatawhich is the shape thatpull_chunk expects. I was naively using the incorrect return value (samples) to count up my samples to process, but as you can see from the watch console, this variable equals 1. It should be equal to 64 because resshould be 512 and the number of channels (the return ofdata_buffer.GetLength(1)) is 8, but it might revert to 1 due to a marshalling conflict---I'm not sure, but we are trying to shove an unsigned longfrom C into auintin C# and divide that by a C#int`.
But that is of no matter. The wrapper functions for pull_chunk ought to return:
return data_buffer.GetLength(0);

Unless there are further axes to grind on this, I think we should merge #12 right away, because I found a bug.

Not the motivation I expected to merge it, but I won't complain ;-)
First I thought it'd be a classical row-major vs col-major confusion, but now I think the code is correct.
My understanding is that pull_chunk_* returns the number of elements, i.e. num_sample * num_chan it put into the num_sample x num_chan data buffer. data_buffer.GetLength(1) returns the number of channels, so res/data_buffer.GetLength(1) is the number of actual samples written to the buffer, which is less than or equal to data_buffer.GetLength(0), i.e. the capacity of the data buffer. Those will be identical when pull_chunk filled the buffer, but with the timeout argument set it might be less.
If I understood it correctly, only chunk[0,0] to chunk[0,7] in your example will have values pulled from the inlet and chunk[1,0] to chunk[512,7] will be left as they were before the call to pull_chunk. NuGet is currently broken on Linux because Microsoft forgot to renew a certificate, so I can't check it.
Could you either check that in your code or add a timeout to the ReceiveDataInChunks example and run it alongside the SendDataInChunks example? With a timeout of one second, it should receive and print ~10 samples at once.

After a walk,  I realized that I was completely wrong abiut this. I thought my buffer was being resized but it isn't. It is fixed. I am closing this. Sorry.

Also the 'axe to grind' comment was meant as a self-depricating joke, in case that wasn't clear. The lesson here is: don't post issues at 5pm...

After a walk, I realized that I was completely wrong about this. I thought my buffer was being resized but it isn't.

Generally, liblsl can't do anything to buffers provided by a runtime (e.g. the C# runtime) other than putting data in their and hope it's big enough. This behavior of pull_chunk_* is quite efficient, but also unintuitive and it should be explained in the docs some day.

Also the 'axe to grind' comment was meant as a self-depricating joke, in case that wasn't clear.

That's how I understood it, after all the hatchet's been buried. Once I've gotten around to fixing the commit attribution and examples, it could be merged.

The lesson here is: don't post issues at 5pm...

8am for me, so I had an unfair advantage.


Bump react-router-dom from 4.4.0 to 5.0.0

Fixes #235.
Version 4.4.0 was unpublished and published as 5.0.0 some time ago. Currently, running yarn install will fail because of this. Version 5.0.0 introduces no breaking changes, so this should be a safe upgrade.
See this post for details: https://reacttraining.com/blog/react-router-v5/

Amazing -- thank you so much!


[skia,crashpad,zeroc-ice,vcpkg-tool-gn] New vcpkg-tool-gn and update of several ports

Describe the pull request

What does your PR fix?
Updates Skia to newer version

Which triplets are supported/not supported? Have you updated the CI baseline?
unchanged

Does your PR follow the maintainer guide?

Yes

If you are still working on the PR, open it as a Draft: https://github.blog/2019-02-14-introducing-draft-pull-requests/

@bold84 , If this pr is ready for review please let me know, thanks!

This depends on https://github.com/microsoft/vcpkg/pull/24142

This is weird... I don't have these errors on two Windows machines with Windows 11 and VS 2022.
What's even weirder is that even the zeroc-ice port (x64-windows-static) suddenly fails on your windows servers, although nothing was changed.

What's even weirder is that even the zeroc-ice port (x64-windows-static) suddenly fails on your windows servers, although nothing was changed.

This is a baseline regression, also failing in the daily full builds: https://dev.azure.com/vcpkg/public/_build/results?buildId=70644&view=results

That you for letting me know. I figured that out by now 😊.
I was able to reproduce the issues and make everything build successfully, despite the regression.
I think that this was a good opportunity to fix a potential problem with the zeroc-ice port.
I will push the changes soon (still in bed).

@bold84 There is already other work on zero-ice: #24169

x86-windows has been disabled for skia.

v8 has been disabled or expected to fail on linux

Skia fails to build on linux. It's looking for fontconfig/fontconfig.h which is present but isn't found. I will solve this problem later.

zeroc-ice port has been changed so that the crt linkage and library linkage are now written directly to the project files before msbuild is called. @strega-nil please confirm that this solves the static crt issue you had.

Because I assume that this PR triggers a "world build" with every commit, I will continue working on Skia here:
https://github.com/bold84/vcpkg/tree/update_skia_local
Once this PR is merged, I will open a new PR with the changes therein.

What distribution are these linux builds running on?

What distribution are these linux builds running on?

It shouldn't matter.
/usr/bin/ld: cannot find -lmcpp
collect2: error: ld returned 1 exit status
make: *** [Makefile:28: bin/slice2cpp] Error 1

Do you know that you can download the CI failure logs yourself, via the publised artifacts in Azure Pipelines?
And I wouldn't run builds as root user on Linux on my local system.

What distribution are these linux builds running on?

It shouldn't matter.
/usr/bin/ld: cannot find -lmcpp
collect2: error: ld returned 1 exit status
make: *** [Makefile:28: bin/slice2cpp] Error 1

Do you know that you can download the CI failure logs yourself, via the publised artifacts in Azure Pipelines? And I wouldn't run builds as root user on Linux on my local system.

Thank you. Yep, I know bout the failure logs. Unfortunately they are only available after all builds are done.
Thank you for the hint about running the build as root. It's a Docker container :-P

Well, the linux build was fine before. But something has changed and caused it to fail. There is another PR, that fixes the mcpp library name issue. I guess I have to wait until it's merged.

@bold84 , If this pr is ready for review please let me know, thanks!

It’s ready for review.

@JonLiu1993 Okay... I see now how much I f...ed up the versioning / git-tree.
I think I have corrected everything to the values they should have. Please have a look.

https://github.com/microsoft/vcpkg/actions/runs/2195232664
What's wrong?

Error: Unhandled error: HttpError: Unknown error: {"count":11,"value":"Uri expired"}

@BillyONeal  Do you know why?

/azp run

Will rerun the test after other tests finished.

Depends on https://github.com/microsoft/vcpkg/pull/24267.

I have learned a lot from this PR 🤓. Thank you for your review and recommendations.
Besides other things, next time I'll plan changes to core files a bit better.

/azp run

Hmm… There are several things I could improve. But I’ll create new PRs for them.

rebased and fixed spaces; gonna wait for green one last time :)

rebased and fixed spaces; gonna wait for green one last time :)

I didn't review; I just merged green build based on this :)


Replace Discord with Discourse

Remove Discord links since the Discourse Community Forum there is becoming our focus.

Looks like I don't have write access so let me get that so I can merge this.