Failure during the cleanup job of EC2 snapshots

I've experienced a new failure during the cleanup job of EC2 snapshots:
neobolt.exceptions.ClientError: Unable to load NODE with id <redacted>.

Full log:

ERROR:cartography.graph.job:Unhandled error while executing statement in job 'cleanup EBS Snapshots': Unable to load NODE with id <redacted>.
ERROR:cartography.sync:Unhandled exception during sync stage 'aws'
Traceback (most recent call last):
  File "/usr/local/lib/python3.9/site-packages/cartography/sync.py", line 73, in run
    stage_func(neo4j_session, config)
  File "/usr/local/lib/python3.9/site-packages/cartography/util.py", line 66, in timed
    return method(*args, **kwargs)
  File "/usr/local/lib/python3.9/site-packages/cartography/intel/aws/__init__.py", line 202, in start_aws_ingestion
    _sync_multiple_accounts(
  File "/usr/local/lib/python3.9/site-packages/cartography/intel/aws/__init__.py", line 139, in _sync_multiple_accounts
    _sync_one_account(
  File "/usr/local/lib/python3.9/site-packages/cartography/intel/aws/__init__.py", line 57, in _sync_one_account
    RESOURCE_FUNCTIONS[func_name](**sync_args)
  File "/usr/local/lib/python3.9/site-packages/cartography/util.py", line 66, in timed
    return method(*args, **kwargs)
  File "/usr/local/lib/python3.9/site-packages/cartography/intel/aws/ec2/snapshots.py", line 117, in sync_ebs_snapshots
    cleanup_snapshots(neo4j_session, common_job_parameters)
  File "/usr/local/lib/python3.9/site-packages/cartography/util.py", line 66, in timed
    return method(*args, **kwargs)
  File "/usr/local/lib/python3.9/site-packages/cartography/intel/aws/ec2/snapshots.py", line 99, in cleanup_snapshots
    run_cleanup_job(
  File "/usr/local/lib/python3.9/site-packages/cartography/util.py", line 32, in run_cleanup_job
    GraphJob.run_from_json(
  File "/usr/local/lib/python3.9/site-packages/cartography/graph/job.py", line 95, in run_from_json
    job.run(neo4j_session)
  File "/usr/local/lib/python3.9/site-packages/cartography/graph/job.py", line 45, in run
    stm.run(neo4j_session)
  File "/usr/local/lib/python3.9/site-packages/cartography/graph/statement.py", line 48, in run
    tx: neo4j.Transaction = session.begin_transaction()
  File "/usr/local/lib/python3.9/site-packages/neo4j/__init__.py", line 614, in begin_transaction
    self._open_transaction(metadata=metadata, timeout=timeout)
  File "/usr/local/lib/python3.9/site-packages/neo4j/__init__.py", line 619, in _open_transaction
    self._connect(access_mode)
  File "/usr/local/lib/python3.9/site-packages/neo4j/__init__.py", line 381, in _connect
    self._connection.sync()
  File "/usr/local/lib/python3.9/site-packages/neobolt/direct.py", line 527, in sync
    detail_delta, summary_delta = self.fetch()
  File "/usr/local/lib/python3.9/site-packages/neobolt/direct.py", line 419, in fetch
    return self._fetch()
  File "/usr/local/lib/python3.9/site-packages/neobolt/direct.py", line 461, in _fetch
    response.on_failure(summary_metadata or {})
  File "/usr/local/lib/python3.9/site-packages/neobolt/direct.py", line 755, in on_failure
    raise CypherError.hydrate(**metadata)
neobolt.exceptions.ClientError: Unable to load NODE with id <redacted>.
Traceback (most recent call last):
  File "/usr/local/bin/cartography", line 8, in <module>
    sys.exit(main())
  File "/usr/local/lib/python3.9/site-packages/cartography/cli.py", line 441, in main
    return CLI(default_sync, prog='cartography').main(argv)
  File "/usr/local/lib/python3.9/site-packages/cartography/cli.py", line 421, in main
    return cartography.sync.run_with_config(self.sync, config)
  File "/usr/local/lib/python3.9/site-packages/cartography/sync.py", line 150, in run_with_config
    return sync.run(neo4j_driver, config)
  File "/usr/local/lib/python3.9/site-packages/cartography/sync.py", line 73, in run
    stage_func(neo4j_session, config)
  File "/usr/local/lib/python3.9/site-packages/cartography/util.py", line 66, in timed
    return method(*args, **kwargs)
  File "/usr/local/lib/python3.9/site-packages/cartography/intel/aws/__init__.py", line 202, in start_aws_ingestion
    _sync_multiple_accounts(
  File "/usr/local/lib/python3.9/site-packages/cartography/intel/aws/__init__.py", line 139, in _sync_multiple_accounts
    _sync_one_account(
  File "/usr/local/lib/python3.9/site-packages/cartography/intel/aws/__init__.py", line 57, in _sync_one_account
    RESOURCE_FUNCTIONS[func_name](**sync_args)
  File "/usr/local/lib/python3.9/site-packages/cartography/util.py", line 66, in timed
    return method(*args, **kwargs)
  File "/usr/local/lib/python3.9/site-packages/cartography/intel/aws/ec2/snapshots.py", line 117, in sync_ebs_snapshots
    cleanup_snapshots(neo4j_session, common_job_parameters)
  File "/usr/local/lib/python3.9/site-packages/cartography/util.py", line 66, in timed
    return method(*args, **kwargs)
  File "/usr/local/lib/python3.9/site-packages/cartography/intel/aws/ec2/snapshots.py", line 99, in cleanup_snapshots
    run_cleanup_job(
  File "/usr/local/lib/python3.9/site-packages/cartography/util.py", line 32, in run_cleanup_job
    GraphJob.run_from_json(
  File "/usr/local/lib/python3.9/site-packages/cartography/graph/job.py", line 95, in run_from_json
    job.run(neo4j_session)
  File "/usr/local/lib/python3.9/site-packages/cartography/graph/job.py", line 45, in run
    stm.run(neo4j_session)
  File "/usr/local/lib/python3.9/site-packages/cartography/graph/statement.py", line 48, in run
    tx: neo4j.Transaction = session.begin_transaction()
  File "/usr/local/lib/python3.9/site-packages/neo4j/__init__.py", line 614, in begin_transaction
    self._open_transaction(metadata=metadata, timeout=timeout)
  File "/usr/local/lib/python3.9/site-packages/neo4j/__init__.py", line 619, in _open_transaction
    self._connect(access_mode)
  File "/usr/local/lib/python3.9/site-packages/neo4j/__init__.py", line 381, in _connect
    self._connection.sync()
  File "/usr/local/lib/python3.9/site-packages/neobolt/direct.py", line 527, in sync
    detail_delta, summary_delta = self.fetch()
  File "/usr/local/lib/python3.9/site-packages/neobolt/direct.py", line 419, in fetch
    return self._fetch()
  File "/usr/local/lib/python3.9/site-packages/neobolt/direct.py", line 461, in _fetch
    response.on_failure(summary_metadata or {})
  File "/usr/local/lib/python3.9/site-packages/neobolt/direct.py", line 755, in on_failure
    raise CypherError.hydrate(**metadata)
neobolt.exceptions.ClientError: Unable to load NODE with id <redacted>.

CC: @achantavy

@marco-lancini: This feels like a DB provisioned with not enough resources - it might be worth asking Neo4j community forums if this comes up again. @mpurusottamc have you seen something like this before?

Otherwise if this is due to bad code, we could improve by employing the approach described here: https://github.com/lyft/cartography/issues/522#issuecomment-885402304
Specifically, we could do something like
def perform_db_action(session: Session, data_list: List[Dict[str, Optional[str]]], update_tag: int) -> None:
    session.write_transaction(_tx_function, data_list, update_tag)

@timeit
def _tx_function(tx: neo4j.Transaction, data_list: List[Dict], update_tag: int) -> None:
    query = """
        neo4j query goes here
    """
    tx.run(query, Data=data_list, UpdateTag=update_tag)

The write_transaction function gives us built-in retries on failures like this.

This feels like a DB provisioned with not enough resources

The reason why I don't think it is purely Neo4j related is that the error is consistent for the same resource every time I try to run it

Reopening while we test e2e to verify.


Always tries to download static resources that don't exist

log:
 Downloading failed: Server returned HTTP response code: 403 for URL: https://download2.gluonhq.com/substrate/javafxstaticsdk/openjfx-19-ea+8-linux-x86_64-static.zip
Check the log files under null

I manually accessed the download address, and the return result told me that the file no longer exists.
In addition, I hope to solve the problem of not recognizing the Graalvm version caused by the vm option automatically added by Idea on Linux.Just like this question：
https://github.com/qjebbs/vscode-plantuml/issues/384
info：
os ：Linux kali 6.1.0-kali9-amd64 #1 SMP PREEMPT_DYNAMIC Debian 6.1.27-1kali1 (2023-05-12) x86_64 GNU/Linux
jdk : openjdk version "21.0.2" 2024-01-16
OpenJDK Runtime Environment GraalVM CE 21.0.2+13.1 (build 21.0.2+13-jvmci-23.1-b30)
OpenJDK 64-Bit Server VM GraalVM CE 21.0.2+13.1 (build 21.0.2+13-jvmci-23.1-b30, mixed mode, sharing)
graalvm :  GraalVM <IP_ADDRESS> Java 17 CE (build by gluon)
Many thanks for any answer

What are you trying to do? What is the expected result? What is your configuration?

main：
supernaut seems to specify downloading outdated resources resulting in no build

description：
I apologize for the lack of detail in the issue.
I encountered a problem when trying to build the uppernaut-fx-sample-maven project using the mvn gluonfx:build command.

The main issue was the inability to download the static resources of version 19-ea+8.
 Attempts to resolve this by changing the JavaFX version in the pom file were unsuccessful.
I suspect that the version of the static resources may have been specified by Supernaut.
 The problem was eventually solved by copying and renaming a static resource from version 21-ea+11.1 to 19-ea+8.
Thank you for your assistance.

What's ridiculous is that when I replied to you,I tried the build again,
and the result was that the build was completed very smoothly without any errors,
not to mention the error of downloading the wrong version.
 (Maybe I haven’t cleaned up my computer lately and I did last night? 😂）
Sorry for wasting your time

log:
 Downloading failed: Server returned HTTP response code: 403 for URL: https://download2.gluonhq.com/substrate/javafxstaticsdk/openjfx-19-ea+8-linux-x86_64-static.zip
Check the log files under null

Access results：
This XML file does not appear to have any style information associated with it. The document tree is shown below.
<Error>
AccessDenied
<Message/>
<BucketName>gluon</BucketName>
<RequestId>tx0000064158f99f6b8ff6b-0065d15aa4-7a12a618-nyc3c</RequestId>
<HostId>7a12a618-nyc3c-nyc3-zg03</HostId>
</Error>


Update DG - Common Classes

To add elaboration and update with any diagrams if needed here

Hey @charmainehly, am suggesting to close this as I believe it's just a minor note. Checked some teams and past teams, no one changed it. If in agreement, can just close bah. Thanks! 😉


Improve debugging functionality.

Make use able to easily add more debugging options and also make a 2D array which 1 contains either it should debug or not other contains the option (like debugging whatsapp, SPIFFS etc)


Implement core::convert::From so by_name can be used in try!()

Getting the following:
Compiling gtfs v0.0.1 (file:///C:/users/nolan/Projects/Gtfs-rs)
:6:1: 6:32 error: the trait core::convert::From<zip::result::ZipErr or> is not implemented for the type std::io::error::Error [E0277]
:6 $ crate:: convert:: From:: from ( err ) ) } } )
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
:1:1: 6:48 note: in expansion of try!
src\feed.rs:19:16: 19:58 note: expansion site
Attempting the following:
let file = try!(self.archive.by_name("agencies.txt"));

New to Rust so perhaps I'm missing something.
Thanks.

One way to fix your problem is by returning a ZipResult in your function, that does have an impl the other way around: impl convert::From<io::Error> for ZipError.
Nevertheles I like your suggestion and will push a commit with this addition soon

Excellent, works. Thanks!


Missing docs for a parameter should be an lint warning.

E:0210: Missing docs for parameter: "action"

Well warning at least.

Actually, I think we changed the style guide a few months ago, so you no longer need to provide documentation (i.e. a textual description of what the param is) if it's obvious from the name. But you do still need to give the type.


Path with same name gets overwritten

Version
4.1.6
Reproduction link
gist.github.com
Steps to reproduce
You can find everything on the gists link provided.
Here is also the stackoverflow issue on what I've tried https://stackoverflow.com/questions/74282811/vue-router-4-path-with-same-name-gets-overwritten
What is expected?
I should be able to access localhost:8080/gallery/231.
What is actually happening?
I guess that the /admin/gallery child somehow overwrites the /gallery/:id route, because if you swap them around, so that /admin/gallery comes first, the /gallery:id route works but not the admin one.

I hope this is not my mistake, else I am very sorry. But the documentation didn't help and it looks like I did everything correctly.

Names are unique so this is expected.

Remember to use the forum or the Discord chat to ask questions!


[APERO] non_dealloc: deallocating None error

First reported by @larnoldgithub
In apero_precheck
Error:
Fatal Python error: none_dealloc: deallocating None
Python runtime state: initialized

Happens in:
apero/core/core/drs_database.py line 2226 update_entries
apero/core/core/drs_database.py line 1842 in add_entry

Doesn't seem to have come back. Closing for now.

Okay happened to me.
This seems to be a similar problem reported here: https://stackoverflow.com/questions/76193436/fatal-python-error-none-dealloc-deallocating-none-bug-likely-caused-by-a-refc
The solution seems to be to upgrade to python 3.12 or downgrade to mysql-connector-python==8.0.20

@larnoldgithub the downgrade of mysql-connectoer-python may be the solution for you until I support python 3.12
I'll put it into the requirements to use this older version.

Does the downgrade require a complete reinstall ? or is it possible to just update that package ?  I'll ask Chris to do that when your requirements are updated

Wow, good find! I was able to use that info to track down a corresponding bug report: https://bugs.mysql.com/bug.php?id=111111
According to that, it has also been fixed in version 8.1.0. That appears to be confirmed in the release notes here: https://dev.mysql.com/doc/relnotes/connector-python/en/news-8-1-0.html

After downgranding  to mysql-connector-python==8.0.20 the precheck on the full SPI data set (99000 files) took about 4h with no errors.

@cusher wow great job finding the bug report! I'll test out 8.1.0 for future releases.


Wildcards do not appear to work with GUI file managers

I'm having some issues getting files to open properly using GUI file managers. They do not open files with the correct program.
Tested GUI file managers:

pcmanfm-qt
thunar

Here's my barebones mimeapps.list
[Default Applications]
image/*=org.nomacs.ImageLounge.desktop;
text/*=nvim.desktop;

I have tried touching handlr.toml, however I was not able to get anything to work. I gave up messing with it and am just using defaults.
enable_selector = false
selector = "rofi -dmenu -i -p 'Open With: '"
term_exec_args = '-e'

I tried to add nomacs as my default image viewer on handlr.toml as follows, but it isn't recognized when running handlr get image/png. This is when unsetting the image/* assignment under mimeapps.list. So I'm not too sure if this is the problem, as I followed a similar formatting to the readme.
[[handlers]]
exec = "nomacs %f"
terminal = false
regexes = ['image/.+']

For reference, this is what the .desktop for nomacs is.
[Desktop Entry]
Name=nomacs
GenericName=Image Viewer
Comment=nomacs is a free, open source image viewer.
Exec=nomacs %f
Terminal=false
Icon=org.nomacs.ImageLounge
Type=Application
Categories=Graphics;RasterGraphics;Viewer;2DGraphics;
MimeType=image/avif;image/bmp;image/gif;image/heic;image/heif;image/jpeg;image/jxl;image/png;image/tiff;image/webp;image/x-eps;image/x-ico;image/x-portable-bitmap;image/x-portable-graymap;image/x-portable-pixmap;image/x-xbitmap;image/x-xpixmap;
GenericName[de]=Bildbetrachter
Comment[de]=Ressourcenschonender, plattformübergreifender Bildbetrachter mit Unterstützung für verschiedenste Formate
Name[pt]=nomacs
GenericName[pt]=Visualizador de imagens
Comment[pt]=Um visualizador de imagens desenvolvido em Qt

Running xdg-mime with debug image/png yields the following.
> env XDG_UTILS_DEBUG_LEVEL=2 xdg-mime query default image/png
Checking /home/timmy/.config/mimeapps.list
Checking /home/timmy/.local/share/applications/mimeapps.list
Checking /home/timmy/.local/share/applications/defaults.list and /home/timmy/.local/share/applications/mimeinfo.cache
Checking /home/timmy/.local/share/applications/defaults.list and /home/timmy/.local/share/applications/mimeinfo.cache
Checking /home/timmy/.local/share/flatpak/exports/share/applications/defaults.list and /home/timmy/.local/share/flatpak/exports/share/applications/mimeinfo.cache
Checking /home/timmy/.local/share/flatpak/exports/share/applications/defaults.list and /home/timmy/.local/share/flatpak/exports/share/applications/mimeinfo.cache
Checking /var/lib/flatpak/exports/share/applications/defaults.list and /var/lib/flatpak/exports/share/applications/mimeinfo.cache
Checking /var/lib/flatpak/exports/share/applications/defaults.list and /var/lib/flatpak/exports/share/applications/mimeinfo.cache
Checking /usr/local/share//applications/defaults.list and /usr/local/share//applications/mimeinfo.cache
Checking /usr/local/share//applications/defaults.list and /usr/local/share//applications/mimeinfo.cache
Checking /usr/share//applications/defaults.list and /usr/share//applications/mimeinfo.cache
chromium.desktop

Running handlr get image/png yields what I expect: org.nomacs.ImageLounge.desktop.
I have xdg-open shadowed as seen in the readme.
> xdg-open
error: the following required arguments were not provided:
  <PATHS>...

Usage: handlr open <PATHS>...

For more information, try '--help'.

Using a TUI file manager like yazi has the file open correctly. I have only tested yazi.
No, I do not have any other *-mimeapps.list files under ~/.config.
I have gotten this to work when I use image/png. It will work on my GUI file explorers, and when using built-in xdg-open. However, this does seem to defeat the purpose of handlr-regex and wildcard support.
I don't know what I'm doing wrong or if it just straight up isn't supported for GUI file handlers. Any help in troubleshooting this, etc., would be appreciated.

That's what I was speculating, darn. Because the only way it wouldn't work is if it straight up did not use xdg-open, or directly referenced the binary on the system rather than looking up $PATH.
Do you have any recommendations on what does work? Perhaps we could have a small documentation of known GUI file managers that utilize xdg-open? Or maybe I'm not looking at the right places on the arch wiki.

Do you have any recommendations on what does work?

Unfortunately, I do not. It has been years since I last regularly used a GUI file manager.
I know nnn, yazi, xplr, superfile, and possibly ranger use xdg-open at least as a fallback, but those are all TUI file managers.

Perhaps we could have a small documentation of known GUI file managers that utilize xdg-open?

That's a good idea. I'd be up for documenting that as soon I know of any examples. I'm sure there's got to be some program that fits that description.

Or maybe I'm not looking at the right places on the arch wiki.

I haven't been able to find much information there regarding GUI file managers that use xdg-open. Unfortunately, I haven't had much luck elsewhere either.

I see. I needed a way to manage pictures, hence my usage of thumbnails in GUI file managers. I wonder if there's a TUI that can display like a grid of images but still as TUI. That's something I guess I'd have to do research on my own.
But that said, as a workaround, earlier I mentioned that using image/gif works as expected. What are your thoughts on having image/* insert all known mime types of image/ into mimeapps.list? I could see it as potentially troublesome to manage, however.

I see. I needed a way to manage pictures, hence my usage of thumbnails in GUI file managers. I wonder if there's a TUI that can display like a grid of images but still as TUI. That's something I guess I'd have to do research on my own.

I know nnn has something like that with its imgview plugin.
You probably already know this, but other TUI file managers like yazi, xplr, and I think maybe ranger support image previews, but only for the currently selected file.

But that said, as a workaround, earlier I mentioned that using image/gif works as expected. What are your thoughts on having image/* insert all known mime types of image/ into mimeapps.list? I could see it as potentially troublesome to manage, however.

I might consider adding an option that enables that behaviour, but I can't promise I'll be able to get around to implementing it anytime soon.
You might want to check out mimeo. It works that way and it even has a feature that's similar to handlr-regex's regex handlers. In fact, it inspired this fork.

Thank you. I will take a look at mimeo, but I'll take a look around at handlr's code when I get the chance to see how feasible it would be to do something like that. I really like handlr's user logic and would like to continue using it.

I have added an option to expand wildcards for next release. See README.md for more information.


Support creating a card token

I make a new issue to avoid spamming the subscriber in https://github.com/arlyon/async-stripe/issues/32.
Could you please elaborate more ?
Based on my understanding, The path is:

fetching stripe open API spec from https://github.com/stripe/openapi/tree/master/openapi
feed it to async-stripe/openapi/
async-stripe/openapi/ will parse the openapi and write the generated Rust code into out directory.
copy the directory into ``async-stripe/src/resources/generated/`

At first I thought it was something like this:
modified   src/resources/generated/token.rs
@@ -99,6 +99,10 @@ pub struct CreateToken<'a> {
     /// The PII this token will represent.
     #[serde(skip_serializing_if = "Option::is_none")]
     pub pii: Option<CreateTokenPii>,
+
+    /// The Card this token will represent.
+    #[serde(skip_serializing_if = "Option::is_none")]
+    pub card: Option<CreateTokenCard>,
 }

 impl<'a> CreateToken<'a> {
@@ -110,6 +114,7 @@ impl<'a> CreateToken<'a> {
             expand: Default::default(),
             person: Default::default(),
             pii: Default::default(),
+            card: Default::default(),
         }
     }
 }
@@ -206,6 +211,14 @@ pub struct CreateTokenPii {
     pub id_number: Option<String>,
 }

+#[derive(Clone, Debug, Deserialize, Serialize)]
+pub struct CreateTokenCard {
+    #[serde(skip_serializing_if = "Option::is_none")]
+    pub number: Option<i64>,
+    pub exp_month: Option<i32>,
+    pub exp_year: Option<i32>,
+}
+

But I realized it will be overidden by code generation, so is it someting like this:
modified   openapi/src/mappings.rs
@@ -578,6 +578,11 @@ pub fn field_mappings() -> FieldMap {
             ("create_payment_method", "billing_details"),
             ("BillingDetails", "Option<BillingDetails>"),
         ),
+
+        (("create_token_card", "exp_month"), ("Card", "Option<Card>")),
+        (("create_token_card", "exp_year"), ("Card", "Option<Card>")),
+        (("create_token_card", "number"), ("Card", "Option<Card>")),
+
         (("transfer_schedule_params", "delay_days"), ("DelayDays", "Option<DelayDays>")),
         (("transfer_schedule_params", "weekly_anchor"), ("Weekday", "Option<Weekday>")),
         (("create_webhook_endpoint", "api_version"), ("ApiVersion", "Option<ApiVersion>")),

I tried to search for a PR that adds a new feature for me to follow. I want to know what parts need to be changed. Unfortunately, I can't find it. I also checked the git blame, but no still no clue.
after adding the line above (in mapping.rs) I run openapi-install, but I don't see card token addition in src/resources/generated/token.rs. So I must be missing something.

Sorry for the delayed response, I am in the process of moving atm.  Thanks for the interest in this project :)

From the linked PR in the parent thread,  it seems like this API is deprecated in favor of payment intents: https://github.com/arlyon/async-stripe/pull/44#issuecomment-866815471


Can't handle interaction via block_actions

Hello,
I have a app_home_opened that offers static select:
SlackRubyBotServer::Events.configure do |config|
  config.on :event, 'event_callback', 'app_home_opened' do |event|
    # […]
    slack_client.views_publish(view: app_home, user_id: user)
  end
  config.on :action, 'block_actions' do |action|
    # […]
  end
end

No problem with the app_home_opened, the block_kit is correctly published. But when I select a value, the block_actions never enter the config.on :action, 'block_actions' block.
I see that warning on grape level:
18:07:13 web.1  | W, [2023-03-28T18:07:13.802178 #87405]  WARN -- : Grape::Exceptions::ValidationErrors: token is missing, type is missing
18:07:13 web.1  |   /Users/thomas/.rbenv/versions/3.1.1/lib/ruby/gems/3.1.0/gems/grape-1.6.2/lib/grape/endpoint.rb:369:in `run_validators'
18:07:13 web.1  |   /Users/thomas/.rbenv/versions/3.1.1/lib/ruby/gems/3.1.0/gems/grape-1.6.2/lib/grape/endpoint.rb:263:in `block in run'
18:07:13 web.1  |   /Users/thomas/.rbenv/versions/3.1.1/lib/ruby/gems/3.1.0/gems/activesupport-7.0.3/lib/active_support/notifications.rb:208:in `instrument'
18:07:13 web.1  |   /Users/thomas/.rbenv/versions/3.1.1/lib/ruby/gems/3.1.0/gems/grape-1.6.2/lib/grape/endpoint.rb:241:in `run'
18:07:13 web.1  |   /Users/thomas/.rbenv/versions/3.1.1/lib/ruby/gems/3.1.0/gems/grape-1.6.2/lib/grape/endpoint.rb:320:in `block in build_stack'
18:07:13 web.1  |   /Users/thomas/.rbenv/versions/3.1.1/lib/ruby/gems/3.1.0/gems/newrelic_rpm-8.8.0/lib/new_relic/agent/instrumentation/middleware_tracing.rb:100:in `call'

Any idea on what I do wrong?
Thank you.

Ok, found the "issue".
The request_url for interactivity was set to /api/slack/event (same as EventSubscription) instead of /api/slack/action, messing the matchers.

Awesome!


Huge memory usage regression in dotnet 7 on linux

Description
A dotnet 7 application consumes much more memory on linux than a dotnet 6 or dotnet 5 application.
After upgrading our application to dotnet 7 we noticed a huge increase in memory consumption in linux, services that used 100 MB were now using twice that amount.
The base issue can be easily reproduced

create new console application
make sure it does not exit instantly, e.g. by using a while loop like:

while (true)
{
    Console.WriteLine("Hello, World!");
    Thread.Sleep(TimeSpan.FromSeconds(1));
}

Verify that in your .csproj file you are targeting dotnet 7:

<TargetFramework>net7.0</TargetFramework>

Switch to a Release configuration
Start it under linux, e.g. by select WSL in the dropdown to launch it.
Verify the used memory, e.g. by running ps aux in your wsl shell and taking a look at the RSS column:

USER       PID %CPU %MEM    VSZ   RSS TTY      STAT START   TIME COMMAND
root      5335  2.5  0.4 273549912 103944 pts/6 Sl+ 17:17   0:00 /usr/bin/dotnet /mnt/c/projects/sources/DotNet7Test/DotNet7Test/bin/Release/net7.0/DotNet7Test.dll

Realize in horror that the console process consumes 100+ MB
Stop the debugging
Target dotnet 6 by updating your csproj file to:

<TargetFramework>net6.0</TargetFramework>

Start it under linux, e.g. by select WSL in the dropdown to launch it.
Verify the used memory, e.g. by running ps aux in your wsl shell and taking a look at the RSS column:

USER       PID %CPU %MEM    VSZ   RSS TTY      STAT START   TIME COMMAND
root      5383  1.0  0.0 2961956 25504 pts/6   Sl+  17:21   0:00 /usr/bin/dotnet /mnt/c/projects/sources/DotNet7Test/DotNet7Test/bin/Release/net6.0/DotNet7Test.dll

Wonder why the same process consumes only 25MB using dotnet 6.

Configuration
Dotnet 7 on Linux x64, see above
Regression?
Yes, this is a major performance regression in dotnet 7 compared to dotnet 5 or 6.
Other information
Notes:
The memory consumption on windows on dotnet 7 is reasonable - it's just a little higher, but not by crazy amounts.
The difference between dotnet 5 and dotnet 6 on linux is negliable.
Take care,
Martin

CC @dotnet/linux-team


Add location

Do you mind fixing the CI issues first? Thanks.
![Uploading 222960147...]

Do you mind fixing the CI issues first? Thanks.

Done!

LGTM


Fix select component fixture having two selected values

The Select component's "with selected value" fixture/example had two values that were selected: one at the item level and one at the component level.
Based on the related test, only item 2 is expected to be selected, so I've changed the fixture to only select item 2.
This is based on the assumption that having two items selected wasn't a purposeful decision — i.e. to see if the two methods of selecting a value can be used in tandem, but given there isn't a test for this, it doesn't seem like that was the intention.
Raised in #3317.

@querkmachine @36degrees Not necessarily in scope, but should selected here be overridden by value?

@colinrotherham Having value take priority makes sense to me if we want to Highlander it ("there can only be one!")
value is a newer addition, which might be why it's more 'passive' than it could be, to avoid potentially breaking changes:?


CSS Class Namespacing

Our classes should probably be prefixed to ensure we don't conflict with commonly used class names.
Example:
All class names in crankshaft could be prefixed with.cs-
I think that feels long and would get tedious, but would be less likely to conflict. Anyone else got suggestions?

I say that they should follow a bem style configutation
** tabs module **
.tab-module {}
.tab-module__tab {}
.tab-module__tab--active {}
.tab-module__panel {}
.tab-module__panel--active {}

name space for specific module not the framework

I agree with BEM. you may be right about namespacing not being super important, bootstrap's approach is
We give you these base elements, you decide to use the same class names as us, that's your problem deal with it.
I do think namespacing the modules like you said is an absolute must.
What I want to do is get a guideline doc/wiki page or documentation site page going with best practices for building custom modules. Not all of them would necessarily be best practices for the framework itself some might simply be best practices for module development in general. Like namespacing the module classes and css. BEM usage.


HostedZoneNotEmpty Error When Nuking

  ERROR   [Failed] unable to nuke hosted zone: HostedZoneNotEmpty: The specified hosted zone contains non-required resource record sets and so cannot be deleted.
                status code: 400, request id: 266ce129-2b1b-497d-bbdb-e82a547c8d89
  ERROR   [Failed] /hostedzone/Z1WT74Z7EAS5SB: HostedZoneNotEmpty: The specified hosted zone contains non-required resource record sets and so cannot be deleted.
                status code: 400, request id: 266ce129-2b1b-497d-bbdb-e82a547c8d89

hey,
can you share the content of that route53 hosted zone after you get this error?
did you try to re-run nuke again? any custom config file was specified?
I'm interested if this is just a "timing issue" or some other dependency.

No longer seeing this issue actually.


[Android] Android 14 / api 34 Crashing issue on android 14 devices

Describe the bug
I've upgraded my app using :
compileSdkVersion = 34
targetSdkVersion = 34
and it gave this error:
FATAL EXCEPTION: main

java.lang.RuntimeException: Unable to start activity ComponentInfo{mypackagename/MainActivity}: java.lang.SecurityException: mypackagenamei: One of RECEIVER_EXPORTED or RECEIVER_NOT_EXPORTED should be specified when a receiver isn't being registered exclusively for system broadcasts
I see the files show this:
context.registerReceiver(stateReceiver, intentFilter)
does this need to be adjusted to accommodate android 14?
To Reproduce
Steps to reproduce the behavior:
Set the API to 34:
compileSdkVersion = 34
targetSdkVersion = 34
Screenshots

Plugin version:

@capacitor-community/bluetooth-le: 6.0.0 or 3.1.1 [ Tried on both]

Smartphone (please complete the following information):

Device: Samsung S21 / S24
OS: Android 14

Just copying over my notes from https://github.com/don/cordova-plugin-ble-central/issues/1020#issuecomment-2226819996
Relevant documentation from Android on this:

https://developer.android.com/develop/background-work/background-tasks/broadcasts#context-registered-receivers
https://developer.android.com/about/versions/14/behavior-changes-14

Although these are all system broadcasts, because they're being registered via an Activity, they're effectively context-registered receivers, and thus need a RECEIVER_EXPORTED flag, as the app needs intents coming from outside the application itself (i.e., from the system).

Same issue happen on cordova capacitor. Please check  https://github.com/cordova-sms/cordova-sms-plugin/pull/233/files
Now i dont have android 14 device to check
i am not sure in kotlin  if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.TIRAMISU)
{
   context.registerReceiver(bondStateReceiver, intentFilter,Context.RECEIVER_EXPORTED)
}
else
{
  context.registerReceiver(bondStateReceiver, intentFilter)
}

@Marcello17 or @lajith111 do either you have the capacity to see if https://github.com/capacitor-community/bluetooth-le/pull/667 fixes the problem for you?

Thank you @Marcello17 for reporting back.
I can confirm that with
compileSdkVersion = 34
targetSdkVersion = 34

my test app works fine on an Android 14 device.


suport time precision of nano-second

DateTime support Tick which is 0.1 macro-second. This is 10000 times better than milli-second.
Current minimal precision of "ms" is not enough. Please support precision "ns".

Will do. :) There was a PR for this, but it introduced unnecessary new classes (duplicates with 2 appended at the end) so I need to rework this and can push it to a new nuget then.

While work is being done with nanosecond-level precision, it might be beneficial to have a workaround to bridge the gap between tick and nano precision.
After some testing and unsuccessful attempts using ulong and DateTime I'm fairly certain the best way to tackle this gap is to add a secondary integer value bounded between 0-99 to account for the last two digits of nanosecond precision.
Every other attempt I made to have everything managed by a single property either ran into the MaxValue limit of ulong around year 583 or involved System.Numerics.BigInteger. The latter is doable but uncommon enough that it might not be intuitive for most users.

Would you consider taking a dependency on Noda Time? You could make use of the Instant class throughout this library. Might make things easier in the long run.

I think I prefer what @jschmidtWK proposed. Introducing NodaTime (although I love it personally) would introduce kinda big-ish breaking changes for many people and I actually want to keep the number of 3rd party dependencies to a minimum.


Address hostname issues

set hostname according to inventory_hostname
use ansible_nodename in /etc/hosts for localhost

I think this looks good. :+1: (but don't merge before #697 )

Testing was aborted, care to start it again once you rebase on novadocker?

Hrm, testing didn't run, have you ran testing locally or in tardis with this change?

Hrm, rabbit is failing to start up in the jenkins run.


chart.x() doesn't hold 'localtime: true' into account?

When I output the dates in my 'x' column with chart.x(), the dates are formatted as follows, whether I enable 'localtime: true' in the x-axis part of the generate object or not:
[Tue Mar 20 1990 00:00:00 GMT+0100 (CET), Sat Sep 22 1990 00:00:00 GMT+0200 (CEST)]
The input is an array with strings representing dates formatted like this: ['Date', '1990-03-20', '1990-09-22',]
Example of localtime configuration:
axis: {
	x: {
	   type: 'timeseries',
           localtime: true,

If try to create the dates locally, they are off by one or two hours (depending on DST) from what I get from c3.js:
console.log(new Date('1990-03-20')) // returns 'Tue Mar 20 1990 01:00:00 GMT+0100 (CET)'
console.log(new Date( '1990-09-22')) // returns 'Sat Sep 22 1990 02:00:00 GMT+0200 (CEST)'

Background information: I want this because I want to write a function that lets you input x-values to select points in a graph. The current chart.select needs an index, which is not really feasible when you have a lot of data points or want people to be able to select a date with a picker. It would be great if I could just select on the column/line and the raw x-value I want.

Looking at this further, this seems to be browser/system related, and not C3. On my locale, an hour gets added from (date at exactly 0:00 am) when I use new Date, and I don't know why. If someone could explain this to me (new to javascript obviously) what is going on here, that would be very much appreciated.
I can solve my problem using moment.js, which converts both c3's chart.x() output and my dumb string of dates into the same format with this function:
function convertToIsoStrings (inputArray) {
                var outputArray = [];
                for (var i = 0; i < inputArray.length; i++){
                    outputArray.push((moment(chartXDates[i]).format()))
                };
                return outputArray;}

Running this on the first 2 dates from c3's chart.x() object and my array of ['1990-03-20', '1990-09-22'] returns ["1990-03-20T00:00:00+01:00", "1990-09-22T00:00:00+02:00", "1991-03-20T00:00:00+01:00"] for both. So I can use indexOf() to lookup the index I need to select the points with the chart.select() function.
That's a great relief, but I still would like to understand why I can't hack this with vanilla javascript.

Closing because not related to c3js, maybe take a look at Date.UTC.
For further help you should use the Google Group at https://groups.google.com/forum/#!forum/c3js


Add ansible-later

Would you be interested in adding ansible-later to your list? It's a fork of ansible-review with some benefits and an alternative best practice checker to ansible-lint.
Regards
Robert

Yes, totally!, Feel free to open up an PR, or anyone for that matter :+1:


[BUG] The "Cancel" button is never available

Acknowledgement of preliminary instructions

[x] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.

What problem did you encounter?
Hi, when I want to cancel the Bloxstrap, well.. i can't.

It's only available when Roblox is installing/upgrading.


Feature request: export with multiple library names

Do you want to request a feature or report a bug?
feature

What is the current behavior?
Only single library name can be passed to output.library.
It would be useful if multiple library names could be passed.
e.g.
output: {
  // ...
  library: ["jquery", "jQuery", "$"]
  // ...
}

If the current behavior is a bug, please provide the steps to reproduce.
-

What is the expected behavior?
-
If this is a feature request, what is motivation or use case for changing the behavior?
It would be useful for me and probably for some other folks (see https://github.com/webpack/webpack/issues/2981)
Please mention other relevant information such as the browser version, Node.js version, webpack version and Operating System.
webpack: 3.3.0

Feel free to either do a PR to add support for this or just use a BannerPlugin to add custom content to the bundle.


Disable All Reformatting?

Hello all,
I am trying to use this fantastic program to simply edit some values in an an XML formatted file that is a configuration backup for an AV device. Once I edit it, I can then import it to replace the current configuration.
The trouble I am having is it seems like some of the basic reformatting that happens is causing the import to fail. No details are given. If I use WinMerge to compare, here are some differences:
This:
<?xml version="1.0" encoding="UTF-8"?>
Becomes:
<?xml version="1.0" encoding="utf-8"?>
This:
<item name="ungrouped" sequence="1"/ >
Becomes:
<item name="ungrouped" sequence="1"/>
(The space before the closing > is removed)
I don't know what is and isn't correctly formatted XML, but the point here is the device seems to be rejecting my edits based on things being autoformatted. All I want to do is open it, find text, replace text, and save.
Thanks!

This text is problematic:
<item name="ungrouped" sequence="1"/ >

The space between "/" and ">" is non-compliant according to the XML specification.  I would recommend fixing this using a text editor like "Notepad" removing that space before you load the file into XML Notepad.  XML Notepad should have failed to load this, unless you have a typo and perhaps the original file contains the space before the slash? This space after foo "" and no space "" is identical as far as the XML specification is concerned.  Any application that rejects one of these is non-compliant.  XML Notepad currently does not have a way to change this behavior.
Have you tried manually changing the lower case "utf-8" to upper case "UTF-8" and does this make your import work correctly?  The XML specification says "XML processors SHOULD match character encoding names in a case-insensitive way".

I'm still trying to get to the bottom of what the device accepts and doesn't accept, but I don't have any control over that device's firmware and I doubt the manufacturer will listen to me in terms of getting their XML files to be compliant. So, that being said, XML Notepad may not be best program to use to edit, or I'll have to do some tweaking after using it. It would be nice for a simple "retain existing formatting" setting where the program only modifies values inside of quotes.

Excellent debugging, and yes you can turn on the "No byte order mark on save" option under Formatting Options in the Options dialog:

Perfect! Thanks.
I would love to see the program just format to the same as the source file automatically.

So, I just tried that option, but I get the following error:

Doh, that's bad, working on a fix...

I pushed a fix to the repo, a new version <IP_ADDRESS> will be available shortly.

Awesome, glad to hear it, thanks for the bug replace, please close it if you are happy with this outcome.  Cheers.


Support boolean in Translation

Summary

New Feature for existing Widget: OptionsRadio

Examples
{
    "type": "object",
    "properties": {
        "field1": {
            "enum": [false, true],
            "widget": "OptionsRadio"
        }
    },
    "required": [
      "field1"
    ]
  }

It kinda already works, but not correctly rendered.

Fix is released with 0.3.0-alpha.5, now boolean enum values are available as yes or no for translation/labeling.


datetimes don't implement hash

Datetimes don't have a hash in Numba, but they do in NumPy. It should be simple to fix this: Implement the hash as hash(dt.view(np.int64)).
Reproducer:
import numpy as np
import numba as nb

def dt_test(x):
    a = set()
    a.add(x)

dt_test(np.datetime64(0, 'ms')) # Works

dt_test_nb = nb.njit(dt_test)
dt_test_nb(0) # Works
dt_test_nb(np.datetime64(0, 'ms')) # Doesn't

If someone is willing to give me some pointers I'd be happy to make a PR to fix this.

@hameerabbasi thanks very much for reporting this! I have tried this locally and can confirm that it does indeed seem to be a bug. Also, I can confirm, that it doesn't seem to be related to the lack of __hash__ as I got the following error when I tried with a typed dictionary.
Unknown attribute '__hash__' of type datetime64[ms]

@hameerabbasi as for pointers, @sklam or @stuartarchibald probably know more, but from a simple git grep, it looks like numba/targets/hashing might be a good place to start looking:
$ git --no-pager grep __hash__
docs/source/developer/hashing.rst::func:`__hash__` member function on the supplied argument. This makes it
docs/source/developer/hashing.rst:type's :func:`__hash__` method. For example::
docs/source/developer/hashing.rst:    @overload_method(myType, '__hash__')
numba/charseq.py:@overload_method(types.UnicodeCharSeq, '__hash__')
numba/charseq.py:@overload_method(types.CharSeq, '__hash__')
numba/controlflow.py:    def __hash__(self):
numba/cuda/cudadrv/driver.py:    def __hash__(self):
numba/datamodel/models.py:    def __hash__(self):
numba/ir.py:    def __hash__(self):
numba/ir.py:    def __hash__(self):
numba/ir.py:    def __hash__(self):
numba/roc/hsadrv/driver.py:    def __hash__(self):
numba/targets/hashing.py:# hash(obj) is implemented by calling obj.__hash__()
numba/targets/hashing.py:        return obj.__hash__()
numba/targets/hashing.py:@overload_method(types.Integer, '__hash__')
numba/targets/hashing.py:@overload_method(types.Boolean, '__hash__')
numba/targets/hashing.py:@overload_method(types.Float, '__hash__')
numba/targets/hashing.py:@overload_method(types.Complex, '__hash__')
numba/targets/hashing.py:@overload_method(types.BaseTuple, '__hash__')
numba/targets/hashing.py:@overload_method(types.UnicodeType, '__hash__')
numba/tests/test_dictobject.py:    def __hash__(self):
numba/tests/test_exceptions.py:    def __hash__(self):
numba/tests/test_exceptions.py:        return hash((super(UDEArgsToSuper).__hash__(), self.value0))
numba/tests/test_exceptions.py:    def __hash__(self):
numba/tests/test_exceptions.py:        return hash((super(UDENoArgSuper).__hash__(), self.deferarg,
numba/tests/test_looplifting.py:            h = b.__hash__()
numba/types/abstract.py:        A property used for __eq__, __ne__ and __hash__.  Can be overriden
numba/types/abstract.py:    def __hash__(self):
numba/types/functions.py:    def __hash__(self):
numba/types/functions.py:        return Type.__hash__(self)
numba/typing/templates.py:    def __hash__(self):
numba/utils.py:    def __hash__(self):

Also, I can confirm, that it doesn't seem to be related to the lack of hash as I got the following error when I tried with a typed dictionary.

I wonder how Python does it then. 🤔

It's failing because the types for representing np.datetime64 and np.timedelta64 are not inheriting from types.abstract.Hashable when they should be, that'll fix the issue with set rejecting the type. Once that is done an @overload can be written to implement __hash__ for the aforementioned NumPy time related types. Here's a starter patch if anyone is interested:
https://github.com/stuartarchibald/numba/commit/575817d3f32a3f3cfc6cd5255c2412f66d8e13db


Not detect, if asterisk-server is down

Monast not detect, if asterisk-server is down (not monast host). After change server or click button Reload, monast show "On Line".

The "On Line" message is related to monast.py service not asterisk...


select-notifications: misaligned

Description
It’s hard to tell? But I think it’s 1-2 pixel higher. This should be fixed by letting the browser align by baseline rather than tweaking via +1px/-1px

How to replicate the issue + URL
https://github.com/notifications
Extension version
24.8.20
Browser(s) used
Safari

I'd be happy to work on this

@fregante This isn't being duplicated on my Safari

This issue isn't about duplication


feat(VSCode): Define metadata API

Merging this as a first iteration of the needed API, will be revisited once the implementation is more mature.


ProjectAPI version filter doesnt work when versions like slbeta6.1 and slbeta10 are used

Description:
The filter needs to be updated to handle decimals and numbers larger than 9
Steps to reproduce:
Affected Versions:
OS, DB, other environment details and versions:
Related Issues (optional):

Suggested Labels (optional):

Suggested Assignees (optional):

This is a blocker for slbeta 6 patch

This will be addressed by the new version scheme. Since patch versions are not supported in package resolution and central for beta releases this patch should go as slbeta6. We did slbeta5.1 in a similar manner. The version will be only indicated in zip or installer.

Closing the issue as per the above comment.


-bash: zappa: command not found

Context
i solved my issue by uninstalling pip and python and re setting up the environment and installation ect

i have not solved the issue and continue to get errors and seems to be no end in site. help would be amazing, thanks.

ok so i found this in my activate file and im pretty confused as to why this is set up this way, is it possible someone who worked for me did something malicious here with the code to steal my deployments? my folder is not this folder that the file is pointing at, is this activate file suspicious? im basicly freaking out here cuz i dont know why this file is set up pointing virtual_env to a folder that is 2 years old and i deleted from my computer yesterday to clean up and since my ability to run deployments is messed up and so confused as to the fact this other folder i had forgot about is controlling my deployments?

I configured the virtual environment without pyenv, then I solved the issue.


Remove CDS_HELM_MULTIPLE_MANIFEST_SUPPORT_NG from docs

Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the CODEOWNERS.
What Type of PR is This?

[ ] Issue
[ ] Feature
[ ] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

Issue(s)

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[ ] Tested Locally
[ ] Optional Screen Shoot.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://656662946010330406d68658--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://656663e542862400977cfe54--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6566647cdf22d8038f2ff49c--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://656664c2022a8100a2dd4391--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://656664f93c0b7f00997c5352--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6566657211b7480323426dd2--harness-developer.netlify.app


Use path_helper in user shell in darwin

Fixes #21655

Managed to work around the build issues with NPM5 and get a copy of this branch running via ./scripts/code.sh. I rebased onto master before building with ./scripts/npm.sh install
The flowtype extension executes without complaint and without needing to launch vscode from inside a terminal whose CWD is the project directory. I manually instrumented the flowtype extension here to log the process.env.PATH and it appears to include everything. 🎉 🎉 🎉 ✨ 🍰
However, this is still an instance of vscode launched from inside a terminal. I wracked my brains trying to figure out how to produce a double-clickable build from the instructions to no avail. gulp vscode-darwin succeeds but the resulting binary at .build/electron/Code - OSS.app opens up a generic electron window like this:

Attempting to drop various promising-looking files on the DnD target didn't yield anything (various files in out, out-build, and out-vscode.)
Any pointers on how to test this effectively?

@idan If you'd like to run this build, just download it.

cc @idan @NinoFloris

Doing a process.env in this build shows me it actually did not run? still just path defaults

Same here
On Thu, Aug 17, 2017, 19:54 Nino Floris<EMAIL_ADDRESS>wrote:

Doing a process.env in this build shows me it actually did not run? still
just path defaults
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/vscode/pull/29864#issuecomment-323146871,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABYw460t95uuukwNPWVQ9IxCuisLr-Vks5sZH5KgaJpZM4OJaLh
.

That is odd @NinoFloris, as Code should be running path_helper as suggested in #21655. So, apparently, that doesn't fix it.

it should as running path_helper from within fish shell returns a lot of info.
Probably something with the build or the call to path_helper going wrong.

it should as running path_helper from within fish shell returns a lot of info.

I might be thick but how does getDarwinEnvironment actually return an environment from that func?

It doesn't launch /usr/libexec/path_helper from fish shell, it just spawns it and parses its result. Launching from within fish shell is a chicken-egg problem... if we could do that, the whole thing would be fixed.

OK, so I looked at how atom is handling this, and it looks like it's the same sort of setup.
After some experimentation, it seems that the wiring of stdout is busted. When I try running path_helper like so from inside code's developer console on my machine, I get the correct $PATH:
(() => {
    const cp = require('child_process')
    const buffers = []
    const child = cp.spawn(process.env.SHELL, ['-lc', 'command /usr/libexec/path_helper'], {encoding: 'utf8', detached: true, stdio: 'pipe'})
    child.on('error', (e) => {
        console.error(e)
    })
    child.stdout.on('data', (data) => {
        buffers.push(data)
    })
    child.on('close', (code, signal) => {
        console.log(Buffer.concat(buffers).toString('utf8'))
    })
})();

The above results in output (redacted) like:
PATH="/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin:~ALLMYOTHERPATHENTRIES~"; export PATH;

I know that originally this used a stdio setup like ['ignore', 'pipe', process.stderr], but this led to stdout being empty.
If invoked with -i, it produces the same output, but prefixed with a block like:
begin;  function __direnv_export_eval --on-event fish_prompt; 	eval (direnv export fish); end
 ;end <&3 3<&-
PATH="/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin:~ALLMYOTHERPATHENTRIES~"; export PATH;

Not sure how this affects other shells yet; but at least it's showing the correct value for me, which is progress. Maybe it's a good clue on how to proceed?

The use of command  before path_helper is interesting... I wonder what that does?

https://askubuntu.com/questions/512770/what-is-use-of-command-command
If you remove command  from that snippet does it still work?

Yes, using pipe makes your setup work, but pretty much everyone else fail. We tried it and ended up reverting it: 44fcdbf31c60eb7ee660df6b6d2dedc4c735a9bc

@idan People over at https://github.com/Microsoft/vscode/issues/21655 say this is no longer an issue. How about for you?

Looks good in 1.19! 😍


Update package reference to cadl-providerhub

#870

Closing since this has gotten stale. I'll fix up the test to use arbitrary package names in another PR.


BrillPOSTagger examples produce errors or no output

Two issues attempting to use the BrillPOSTagger system.
Occurs running the latest code (as of today) Version 0.4.1 cloned from the GitHub repository.
Node: 6.2.2
NPM: 3.8.6
Natural: 0.4.1
OS: Ubuntu 16.04
First example uses code from the README.md on the GitHub repository. Second example uses code from the egghead.io class.
Example 1:
https://github.com/NaturalNode/natural contains the following example:
var natural = require("./lib/natural");

var base_folder = "some_path/lib/natural/brill_pos_tagger";
var rulesFilename = base_folder + "/data/tr_from_posjs.txt";
var lexiconFilename = base_folder + "/data/lexicon_from_posjs.json";
var defaultCategory = 'N';

var lexicon = new natural.Lexicon(lexiconFilename, defaultCategory);
var rules = new natural.Ruleset(rulesFilename);
var tagger = new natural.BrillPOSTagger(lexicon, rules);

var sentence = ["I", "see", "the", "man", "with", "the", "telescope"];
console.log(JSON.stringify(tagger.tag(sentence)));

I have the following code, which is the above with the folders set:
const natural = require('natural')

const Tagger = natural.BrillPOSTagger

const baseFolder = './node_modules/natural/lib/natural/brill_pos_tagger/data/English'
const rulesFilename = baseFolder + '/tr_from_posjs.txt'
const lexiconFilename = baseFolder + '/lexicon_from_posjs.json'
const defaultCategory = "N"

var lexicon = new natural.Lexicon(lexiconFilename, defaultCategory);
var rules = new natural.Ruleset(rulesFilename);
var tagger = new natural.BrillPOSTagger(lexicon, rules);

var sentence = ["I", "see", "the", "man", "with", "the", "telescope"];
console.log(JSON.stringify(tagger.tag(sentence)));

Running the program produces the following error:
npm ERR! Please include the following file with any support request:
npm ERR!     /home/john/Documents/fenestra/development/gids-web/natural-language-processing/npm-debug.log
john@john-aspire-7741:~/Documents/fenestra/development/gids-web/natural-language-processing$ node brills-error.js

/home/john/Documents/fenestra/development/gids-web/natural-language-processing/brills-error.js:11
var rules = new natural.Ruleset(rulesFilename);
            ^

TypeError: natural.Ruleset is not a constructor
    at Object.<anonymous> (/home/john/Documents/fenestra/development/gids-web/natural-language-processing/brills-error.js:11:13)
    at Module._compile (module.js:541:32)
    at Object.Module._extensions..js (module.js:550:10)
    at Module.load (module.js:458:32)
    at tryModuleLoad (module.js:417:12)
    at Function.Module._load (module.js:409:3)
    at Module.runMain (module.js:575:10)
    at run (node.js:348:7)
    at startup (node.js:140:9)
    at node.js:463:3

See attached log
npm-debug.log.zip
Example 2
Attempted to execute code from egghead.io class: https://egghead.io/lessons/javascript-tag-parts-of-speech-using-natural

Executing the code:
const natural = require('natural')

const Tagger = natural.BrillPOSTagger

const myString = "Lys soldered the beautiful jewelry pieces".split(" ")

const baseFolder = './node_modules/natural/lib/natural/brill_pos_tagger'
const rules = baseFolder + '/data/English/tr_from_posjs.txt'
const lexicon = baseFolder + '/data/English/lexicon_from_posjs.json'
const defaultCategory = "N"

const tagger = new Tagger(lexicon, rules, defaultCategory, function(err) {
    console.log('starting')
    if (err) {
        console.log(err)
    } else {
        console.log(tagger.tag(myString))
    }
})

Produces the following empty output:
john@john-aspire-7741:~/Documents/fenestra/development/gids-web/natural-language-processing$ node index.js
john@john-aspire-7741:~/Documents/fenestra/development/gids-web/natural-language-processing$

RuleSet should be spelled with a capital S. When it is spelled with lower
case s it will not find the constructor. This is incorrect in the README
file of natural. I will correct this.
Regards!
Hugo
2017-01-13 21:02 GMT+01:00 John Elrick<EMAIL_ADDRESS>
Example 1:
https://github.com/NaturalNode/natural contains the following example:
var natural = require("./lib/natural");
var base_folder = "some_path/lib/natural/brill_pos_tagger";var rulesFilename = base_folder + "/data/tr_from_posjs.txt";var lexiconFilename = base_folder + "/data/lexicon_from_posjs.json";var defaultCategory = 'N';
var lexicon = new natural.Lexicon(lexiconFilename, defaultCategory);var rules = new natural.Ruleset(rulesFilename);var tagger = new natural.BrillPOSTagger(lexicon, rules);
var sentence = ["I", "see", "the", "man", "with", "the", "telescope"];console.log(JSON.stringify(tagger.tag(sentence)));
I have the following code, which is the above with the folders set:
const natural = require('natural')
const Tagger = natural.BrillPOSTagger
const baseFolder = './node_modules/natural/lib/natural/brill_pos_tagger/data/English'const rulesFilename = baseFolder + '/tr_from_posjs.txt'const lexiconFilename = baseFolder + '/lexicon_from_posjs.json'const defaultCategory = "N"
var lexicon = new natural.Lexicon(lexiconFilename, defaultCategory);var rules = new natural.Ruleset(rulesFilename);var tagger = new natural.BrillPOSTagger(lexicon, rules);
var sentence = ["I", "see", "the", "man", "with", "the", "telescope"];console.log(JSON.stringify(tagger.tag(sentence)));
Running the program produces the following error:
npm ERR! Please include the following file with any support request:
npm ERR!     /home/john/Documents/fenestra/development/gids-web/natural-language-processing/npm-debug.log
john@john-aspire-7741:~/Documents/fenestra/development/gids-web/natural-language-processing$ node brills-error.js
/home/john/Documents/fenestra/development/gids-web/natural-language-processing/brills-error.js:11
var rules = new natural.Ruleset(rulesFilename);
^
TypeError: natural.Ruleset is not a constructor
at Object. (/home/john/Documents/fenestra/development/gids-web/natural-language-processing/brills-error.js:11:13)
at Module._compile (module.js:541:32)
at Object.Module._extensions..js (module.js:550:10)
at Module.load (module.js:458:32)
at tryModuleLoad (module.js:417:12)
at Function.Module._load (module.js:409:3)
at Module.runMain (module.js:575:10)
at run (node.js:348:7)
at startup (node.js:140:9)
at node.js:463:3
See attached log
npm-debug.log.zip
https://github.com/NaturalNode/natural/files/705212/npm-debug.log.zip
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/NaturalNode/natural/issues/328, or mute the thread
https://github.com/notifications/unsubscribe-auth/AHd2W4mb4Ry1HI1kDYhxY5Fj8lk2YZ4Eks5rR9g8gaJpZM4LjQOw
.

--
Met vriendelijke groeten,
Hugo ter Doest
06-23 43 60 33

Your second example applies to an older version of the tagger with an asynchronous constructor of the tagger. This can no longer be used.
Hugo

For the POS Tagging part I get the following error
natural.Lexicon is not a function! I checked index.js and it does not export Lexicon...what is the alternative?

Please add your code so I can see what is going on.
Hugo

Its the same code as in the readme:
var natural = require('natural'),
var base_folder = "my_path/lib/natural/brill_pos_tagger";
var rulesFilename = base_folder + "/data/tr_from_posjs.txt";
var lexiconFilename = base_folder + "/data/lexicon_from_posjs.json";
var defaultCategory = 'N';

var lexicon = new natural.Lexicon(lexiconFilename, defaultCategory);
var rules = new natural.Ruleset(rulesFilename);
var tagger = new natural.BrillPOSTagger(lexicon, rules);

var sentence = ["I", "see", "the", "man", "with", "the", "telescope"];
console.log(JSON.stringify(tagger.tag(sentence)));

I tried to reproduce the error you reported. I installed the natural module from NPM and then tried to instantiate a tagger following the readme. This resulted in the same error.
So the problem is that NPM is not up-to-date with the latest version on Github. To get this working you should make a clone of the natural module from Github and then try again.
Hugo

I created a PR for the readme, I think there are a few easy errors from things that changed including there is now a language directory and Ruleset -> RuleSet. I also added an easy way to figure out the right directory without a user having to change anything so it should be copy/paste.

also just published a new version to npm so everything should be up to date now @julianfrank


Implement full instruction set virtual machine

Description
This is a full list of changes that need to be made to the virtual machine.
Task list
Each task will likely be made into an issue of its own right.

[ ] Call Stack

Gonna ditch the Stack structure and replace it with a CallStack struct.
Also need to add CallFrames and just get everything set up properly

[ ] Instruction Buffer

Ditching the Buffer struct for InstructionBuffer struct
This will be more specific while also having some better features (hopefully)

[ ] Registers

I want registers to be on the stack, so they're probably going to be a basic array

[ ] Variable Pools

These need to be set up properly
Variables should already be mangled by the assembler by the time we get them

[ ] Block Handler

Needs to find the .data, .meta, and .code blocks
Use .meta data for configuration
Use .data data to set up variables, constants, and structs

[ ] Error Handler

Basic error handling, use r8 and r9.

[ ] Implement all the methods needed for each instructions

Closed as stale


Add auto merge

@conda-forge-admin, please add bot automerge

@conda-forge-admin, please add bot automerge

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added bot automerge in conda-forge/pymatgen-lammps-feedstock#2.


Support for additional character roles

Add support for the <background/> and <additionalVoices/> elements in the code base.
Related ticket: https://github.com/cjcodeproj/vtmedia-schema/issues/39

Commit:
https://github.com/cjcodeproj/medialibrary/commit/a4a2f50867e6baec1f35dca347a5263bba7d4fa2
Notes:

Total unit tests: 91
First attempts to add to_element methods to already existing objects.
Always update the copyright year.


Make Cloud Storage client retry on backend error

We're operating at scale on GCS and are regularly experiencing transient HTTP 410 status codes when accessing Cloud storage. Those 410 status codes returned by Cloud storage are bogus though, as they are effectively just hiding an internal backend error on GCS, which is reflected in the error details:
Caused by: com.google.api.client.googleapis.json.GoogleJsonResponseException:
410 Gone { "code" : 503, "errors" : [ { "domain" : "global", "message" : "Backend Error", "reason" : "backendError" } ], "message" : "Backend Error" }

The google-cloud-storage client does not treat the 410 status code as retryable, understandibly so. It should be retrying on backend errors, though, which are typically exposed with status code 500 or 503. I'm suggesting to treat backend errors in the client in the same way as it treats internal errors, namely match on reason == backendError independently of HTTP status code.
Note that we're not the first ones to experience this, and the client should be resilient against these transient GCS errors.

https://stackoverflow.com/questions/33056415/uploading-files-into-google-cloud-storage-500-backend-error
https://stackoverflow.com/questions/35125891/google-cloud-dataflow-jobs-failing-inaccessible-jars-410-gone-errors
https://stackoverflow.com/questions/41215541/dataflow-jobs-fail-after-a-few-410-errors-while-writing-to-gcs

I've contacted the storage backend team and if they aren't against it I'll add the retry logic.

Based on the discussion with storage backend the 410 happens during a JSON API resumable upload session. The error likely indicates that the upload session has already been terminated and retrying the  individual HTTP request would not work (the entire upload session has to be restarted). An internal bug has been filed and storage team is actively working on it.

@hzyi-google and @JustinBeckwith
Is this bug still blocked by internal error?

For googlers: b/116709007 is the internal bug.
According to the bug it seems they decided that it's not possible to fix it in the client libraries.
Dataflow already effectively retries 410s by retrying every failed shard 4 times regardless of why it failed, so it won't be a problem for Dataflow users.
b/115694839 tracks the implementation of resumable uploads. People who are having these 410s directly in their projects might need to wait for this feature.

This issue is important and unfortunately not solvable by clients.  I'm going to close this issue, since there's nothing we can do here.

@hzyi-google  Could you please update the status of this issue? It's been almost a year...

@romange Sorry I do not work in this repo any more. cc/ @kolea2


Check for Laravel version and change cache from minutes to seconds

Starting in Laravel 5.8 the cache now takes time in seconds instead of minutes. This PR checks if the app is running Laravel >= 5.8 and transforms the TTL to seconds.

Thank you @rbcunhadesign 👍 ❤


ParsingResult not serializable if firstStage is present

FirstStageParsingResult holds a reference to the ANTLR parse tree root which is not serializable.
I think we should make the entire first stage transient, because it mostly replicates some of the information in the ParsingResult. Or, alternatively, we could mark only FirstStageParsingResult.root as transient.

If I understand correctly we have a workaround for this: setting the FirstStageParsingResult to null when looking to serialize a node. Is this correct?

Yes. In fact I've done this, only to bump into the problem with Issue.


added new spatialy-varing diffusivity files and rmu.[ab] , also updat…

…ed expt_preprocess.sh and blkdat.input

The expt_preprocess.sh  will read thkdf4&veldf4 from blk file and if negative it then will then copy the corresponding file from /relax/$E to SCRATCH to be used by the associated run


Add Orin NX installation instructions

This PR adds the official documentation to install and configure the Jetson Orin NX board

Here:

https://martinaxgloria.github.io/documentation/icub_operating_systems/icubos/jetpack/

the website to see the outcome

There was only a wrong indentation in the Booting the Nvidia Jetson board in Recovery mode section. Fixed https://github.com/icub-tech-iit/documentation/pull/355/commits/38e1665e1b520c4ec866bab1a457b86a55e96f99. Rebuilt the page:

https://martinaxgloria.github.io/documentation/icub_operating_systems/icubos/jetpack/


MissingPropertyException when setting effort of SpotBugsExtension

When optimizing the configuration time using the gradle-profiler and checking the flame graph output I noticed the SpotBugsExtension.setEffort() method throwing MissingPropertyExceptions. I'm using project.getExtensions().getByType(SpotBugsExtension.class).setEffort("max") to set the effort.
It's this line Effort effort = name == null ? null : Effort.valueOf(name.toUpperCase()) causing the exception, but I don't have enough Groovy knowledge to understand why it is throwing the MissingPropertyException.
Here you can see the time it takes to apply the Spotbugs plugin and configure basic stuff of the extension. One third of the time is spent with SpotbugsExtension.setEffort().

Unfortunately I cannot share the full stacktrace, but this is the final part of the stacktrace:
invokeStaticMissingProperty:1021, MetaClassImpl (groovy.lang)
getProperty:1866, MetaClassImpl (groovy.lang)
getProperty:1842, MetaClassImpl (groovy.lang)
invokeStaticMethod:1474, MetaClassImpl (groovy.lang)
callStatic:62, StaticMetaClassSite (org.codehaus.groovy.runtime.callsite)
callStatic:216, AbstractCallSite (org.codehaus.groovy.runtime.callsite)
valueOf:-1, Effort (com.github.spotbugs.snom)
call:-1, Effort$valueOf (com.github.spotbugs.snom)
setEffort:192, SpotBugsExtension (com.github.spotbugs.snom)

Environment:
Gradle 6.7
Java openjdk 11.0.6 2020-01-14
Spotbugs gradle plugin 4.5.0

I would think this would cause your build to fail since it is throwing an exception. Looking at the stack trace it seems it is somehow trying to set the effort to -1?

Some dynamic groovy calls are happening, the exception is silently caught in MetaClassImpl
try {
    prop = this.getProperty(this.theClass, this.theClass, methodName, false, false);
} catch (MissingPropertyException var13) {
}

Does this happen if instead of using project.getExtensions().getByType(SpotBugsExtension.class).setEffort("max") you use something like
spotbugs {
  effort = 'max'
}

?

Yes, it hits the exact same functionSpotBugsExtension.setEffort().

so  Effort effort = name == null ? null : Effort.valueOf(name.toUpperCase()) is just another way to write
Effort effort = null;
if(name != null){
     effort = Effort.valueOf(name.toUpperCase())
}

So the only way it is throwing an error there is if name.toUpperCase() doesn't equal any of the enum values in Effort.
With the intentional swallowing of the exception by Gradle, I feel like this is a bug on their end.

I understand that part, is just very strange that groovy is throwing the MissingPropertyException. This issue is to find out if a different way of writing that statement is possible with groovy which does not throw the exception.

I'm checking how Groovy executed the code.
Here I assume that the Gradle version is 2.5.12 used by the latest Gradle v6.7.1:
invokeStaticMissingProperty:1021, MetaClassImpl (groovy.lang)
getProperty:1866, MetaClassImpl (groovy.lang)
getProperty:1842, MetaClassImpl (groovy.lang)
invokeStaticMethod:1474, MetaClassImpl (groovy.lang)
callStatic:62, StaticMetaClassSite (org.codehaus.groovy.runtime.callsite)
callStatic:216, AbstractCallSite (org.codehaus.groovy.runtime.callsite)
The point is here: MetaClassImpl catches and ignores MissingPropertyException. I also confirmed that this behaviour is not improved even in the master branch.
So @daanschipper I suggest you to issue a report to Groovy language project. I tried the following solution, but I think it won't be solution because we still need to invoke a static method in Effort class.
diff --git a/src/main/groovy/com/github/spotbugs/snom/Effort.groovy b/src/main/groovy/com/github/spotbugs/snom/Effort.groovy
index 791ac9c..aff86a5 100644
--- a/src/main/groovy/com/github/spotbugs/snom/Effort.groovy
+++ b/src/main/groovy/com/github/spotbugs/snom/Effort.groovy
@@ -13,6 +13,8 @@
  */
 package com.github.spotbugs.snom;

+import edu.umd.cs.findbugs.annotations.NonNull
+
 /**
  * The {@code Effort} is configuration to adjust SpotBugs detectors. Use lower effort to reduce
  * computation cost.
@@ -50,5 +52,16 @@ enum Effort {
      * The effort level that maximize the computation cost. SpotBugs will run Interprocedural Analysis
      * of Referenced Classes.
      */
-    MAX
+    MAX;
+    static Effort fromString(@NonNull String string) {
+        Objects.requireNonNull(string)
+        switch (string.toUpperCase()) {
+            case "MIN": return MIN;
+            case "LESS": return LESS;
+            case "DEFAULT": return DEFAULT;
+            case "MORE": return MORE;
+            case "MAX": return MAX;
+            default: throw new IllegalArgumentException("'${string}' is not valid as Effort value")
+        }
+    }
 }
diff --git a/src/main/groovy/com/github/spotbugs/snom/SpotBugsExtension.groovy b/src/main/groovy/com/github/spotbugs/snom/SpotBugsExtension.groovy
index 20a4146..b0050ab 100644
--- a/src/main/groovy/com/github/spotbugs/snom/SpotBugsExtension.groovy
+++ b/src/main/groovy/com/github/spotbugs/snom/SpotBugsExtension.groovy
@@ -196,7 +196,7 @@ class SpotBugsExtension {
     }

     void setEffort(@Nullable String name) {
-        Effort effort = name == null ? null : Effort.valueOf(name.toUpperCase())
+        Effort effort = name == null ? null : Effort.fromString(name)
         getEffort().set(effort)
     }

getEffort().set(Effort.MAX) can be used to avoid this issue altogether.


Create an enclosure case for TPU prints

REF: https://github.com/sqfmi/beepberry-hardware/issues/3
The current enclosure is awesome, but it would be nice to have a flexible case to put around it to protect from drops. Something that's flexible like TPU, similar to a phone case.

Like this:
20230605_beeper_bumper.webm


Clarify "same route with different parameter names" error message

An attempt to improve the error message for this unusual case.
Also ensures the error appears in release builds.

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@a0801a3). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #111   +/-   ##
=======================================
  Coverage        ?   84.17%
=======================================
  Files           ?        4
  Lines           ?      278
  Branches        ?        0
=======================================
  Hits            ?      234
  Misses          ?       44
  Partials        ?        0

Flag
Coverage Δ

unittests
84.17% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

These changes are now available in 4.3.1


RemovedInDjango19Warning (2)

I think this is different from #71
When I try to use ThreadedComment model in my app, I get a single
threadedcomments/models.py:9: RemovedInDjango19Warning: Model class threadedcomments.models.ThreadedComment doesn't declare an explicit app_label and either isn't in an application in INSTALLED_APPS or else was imported before its application was loaded. This will no longer be supported in Django 1.9.
  class ThreadedComment(Comment)

After reading for example this question on StackOverflow and examining how similar projects were fixed (https://github.com/DanielJDufour/python-social-auth/commit/b51629fcc54ad755f1b17f27712aacc860f34129), I suggest to remove imports from init. ThreadedComments model is imported during package import, before Django app is initialized, thus the warning is printed.

Thanks! the pull request is merged!
If you see any other Django 1.9 issues, don't hesitate to make another PR!


Reimagine data storage architecture

We are iterating based on a new view in the database and will not be doing any re-imagining at this time. We will revisit when needed.


Update tests

I'm not sure it makes sense to change the less_version to 1.7.5 until the php code has actually been updated to be compatible with 1.7.5. Here are all the changes

Thanks


iOS - QR Codes with region codes should not checked for business rules

Same problem as issue https://github.com/eu-digital-green-certificates/dgca-verifier-app-ios/issues/106 in verifier app

@kerstin-oppermann-tsi please check on latest version of app


When two or more classes are in one header file, only the last class's code source with line number.

Describe the bug
When two or more classes are in one header file, only the last class's code source with line number.
If use the original doxygen, no problem (Every line with a line number).
Expected behavior
Every class in the header file should have a line number each line.
Screenshots

To Reproduce
Just a simple header file, with two or more classes declaration in it.
Version
Windows 11, 64 bit
Doxygen 1.9.8
doxygen-awesome-css 2.2.1

I face the same issue, line number are shown only for the last class in the file.

If I remove the doxygen-awesome-css/doxygen-awesome.css fie from HTML_EXTRA_STYLESHEET, the whole file is ok:

The file used in these images is detail.h, but the same happens to all files in my projects.
Any idea how to fix this?
macOS x64 Sonoma 14.1
Doxygen 1.9.8
doxygen-awesome-css 2.2.1

I confirm that the line numbers now show on my site.
Could you make a new release so I can pick the commits? (2.2.2 if since 2.2.1 there were only bug fixes, or 2.3.0 if there were any additions to the api)

Thx for the feedback!
I will need to do some more checks to make sure that the change is backwards compatible with older versions of Doxygen, then I will make a release! 👍

BTW, your cool theme was highly appreciated by those who reviewed my sites:

https://micro-os-plus.github.io/micro-test-plus-xpack/
https://micro-os-plus.github.io/utils-lists-xpack/

Thank you!


Upgrade sass-loader to ^11.1.1

Why

Fix https://github.com/facebook/create-react-app/issues/10944

I couldn't find any test to check if something is wrong.
I don't see that much usage of sass-loader, which may implies that this change doesn't break anything.
The problem here is that sass-loader v11 upgrades node-sass from v5 to v6, which removes support for Node v10, which is a problem for CRA because the documentation says that it supports => 10.16
Deeper explanation in https://github.com/facebook/create-react-app/issues/10944#issuecomment-869760083
Feel free to reject this PR if I'm missing something. Everything ran too smooth. It feels weird that works on the first try

Fixed in https://github.com/facebook/create-react-app/pull/11201

Fixed in #11201

Brand new install of create-react-app still has "sass-loader": "^10.0.5", in react-scripts dependencies. I am still having the issue with node-sass as a result. Any suggestions?
I did make sure to uninstall global package first.


ambiguous codelink in OCSPPolicy.swift:71

swift-unidoc found an ambiguous codelink in OCSPPolicy.swift:71. there are two possible link targets.
/swift-certificates/checkouts/swift-certificates/Sources/X509/OCSP/OCSPPolicy.swift:72:161: warning: codelink 'PolicyEvaluationResult.failsToMeetPolicy(reason:)' is ambiguous
The OCSP query is considered unsuccessful and will fail verification in both ``OCSPFailureMode/soft`` and ``OCSPFailureMode/hard`` failure mode.
The certificate is then considered to not meet the ``OCSPVerifierPolicy`` and ``OCSPVerifierPolicy/chainMeetsPolicyRequirements(chain:)`` will return ``PolicyEvaluationResult/failsToMeetPolicy(reason:)`` with the given ``reason``.
                                                                                ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^
- Parameter reason: the reason why the OCSP query failed

(unknown) note: did you mean 'PolicyEvaluationResult.failsToMeetPolicy(reason:) [3E21]'? (static X509.PolicyEvaluationResult.failsToMeetPolicy(reason: @autoclosure @Sendable () -> Swift.String) -> X509.PolicyEvaluationResult)

(unknown) note: did you mean 'PolicyEvaluationResult.failsToMeetPolicy(reason:) [3TP9A]'? (static X509.PolicyEvaluationResult.failsToMeetPolicy(reason: X509.PolicyFailureReason) -> X509.PolicyEvaluationResult)

Sorry for missing this when it was filed at the time. Do you want to fix it up? The correct target is the second one.


Does my static website need Content Security Policy?

Hi, thank you so much for the useful package.
I have a static blog website built using Next.js and Strapi headless CMS.
I recently came to know about the Content Security Policy.
But having really a hard time understanding it thoroughly.
I was reading articles about it online and some of them say that it is not required for a static website.
https://csp.withgoogle.com/docs/why-csp.html
https://reesmorris.co.uk/blog/implementing-proper-csp-nextjs-styled-components
My website is an SSG (automatically generated static HTML +JSON).
The only place where the server is involved is the sitemap.xml file which is handled by (pages/api/sitemap.js). I can't generate it statically because of dynamic URLs coming from Strapi.
I have a cookie consent banner on the website and that is the only cookie I think besides Google Analytics & TruConversion Heatmap cookies. (If this is relevant)
I am planning to have a "contact" page on my website in the future and it will contain the contact form. Probably, the only place where users may input data.
I'll be very thankful to you if you suggest to me that whether I actually need a CSP or not.

Hi @trezy,
I'm so sorry for the delayed response.
Yes, I completely agree with you that CSP is needed even if our website is static.
We have actually delayed this thing for now, but when I'll implement it on our website, I'll post my solution here.
So, anybody who comes here for help in the future will be able to get help from it.
Thank you so much for your time.

I find the term 'static website' when talking about this confusing, given the meaning it has today.
When we talk about "static pages" in the context of Next, we usually mean those pages that use getStaticProps. Those pages aren't more secure or insecure towards XSS than "dynamic pages" with getServerSideProps, "static pages" are build-time prerendered, "dynamic-pages" are per-request prerendered. But both hydrate on the client and become fully interactive React apps. This means that on both you can in principle do things that are an invitation to XSS. The decisive thing is whether you load dynamic user content or not.
Say you would dangerouslySetInnerHtml user comments fetched via API from your database without sanitizing them. If then someone sneaks in a script into your database that steals your users' session data by posting a comment, it doesn't matter what pre-rendering strategy was used before serving the page to them. That script will be loaded and executed on their client either way when they browse the comment section.
So I would say the need for a CSP depends on the degree to which your website loads dynamic user content. The more your site does that, the more it would benefit from a strong CSP.
If your website is this:
const MyWebsite = () =>
Hello World

export default MyWebsite

a CSP would probably be overkill 😄
I personally think of a strong and effective CSP as 2FA against XSS. Strict CSP3 with hash or nonce would reliably block the script from the scenario above. Yet you still would sanitize user input, just as you don't use "123" as a password even with 2FA enabled.


fix: if value of key undefined check keyCode against keyMapEntries

Allows mapping of keyCodes for custom keys

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Minor: add runtime asserts to RowGroup

Which issue does this PR close?
Follow on to https://github.com/apache/datafusion/pull/10607/
Rationale for this change
@advancedxy noted https://github.com/apache/datafusion/pull/10607/files#r1611810009 that there was an assumption that
was not programatically validated
What changes are included in this PR?
Add assert to ensure the assumption is correct
Are these changes tested?

Are there any user-facing changes?

Thanks @viirya  and @advancedxy


Many timeout exceptions

I'm using this library via the Home Assistant (2202.8.6) integration, to gather data from multiple sensors.
The Luftdaten API tends to be slow. My logs are spammed with multiple timeout exceptions that are not handled properly, and it also causes the Home Assistant sensors to go offline.

Would it be an idea to increase the timeout from 5 seconds (default of httpx) to something higher? Maybe 10 or 15 seconds?
Can timeout exceptions be ignored, such that they do not make the sensor go offline?

Here is an example exception, from the Home Assistant logs:
2022-08-23 22:25:48.074 ERROR (MainThread) [homeassistant.components.luftdaten] Unexpected error fetching luftdaten_70908 data:
Traceback (most recent call last):
  File "/opt/apps/homeassistant/env/lib/python3.10/site-packages/anyio/streams/tls.py", line 130, in _call_sslobject_method
    result = func(*args)
  File "/usr/lib/python3.10/ssl.py", line 916, in read
    v = self._sslobj.read(len)
ssl.SSLWantReadError: The operation did not complete (read) (_ssl.c:2548)

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/opt/apps/homeassistant/env/lib/python3.10/site-packages/httpcore/backends/asyncio.py", line 33, in read
    return await self._stream.receive(max_bytes=max_bytes)
  File "/opt/apps/homeassistant/env/lib/python3.10/site-packages/anyio/streams/tls.py", line 195, in receive
    data = await self._call_sslobject_method(self._ssl_object.read, max_bytes)
  File "/opt/apps/homeassistant/env/lib/python3.10/site-packages/anyio/streams/tls.py", line 137, in _call_sslobject_method
    data = await self.transport_stream.receive()
  File "/opt/apps/homeassistant/env/lib/python3.10/site-packages/anyio/_backends/_asyncio.py", line 1265, in receive
    await self._protocol.read_event.wait()
  File "/usr/lib/python3.10/asyncio/locks.py", line 214, in wait
    await fut
asyncio.exceptions.CancelledError

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/opt/apps/homeassistant/env/lib/python3.10/site-packages/httpcore/_exceptions.py", line 8, in map_exceptions
    yield
  File "/opt/apps/homeassistant/env/lib/python3.10/site-packages/httpcore/backends/asyncio.py", line 31, in read
    with anyio.fail_after(timeout):
  File "/opt/apps/homeassistant/env/lib/python3.10/site-packages/anyio/_core/_tasks.py", line 118, in __exit__
    raise TimeoutError
TimeoutError

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/opt/apps/homeassistant/env/lib/python3.10/site-packages/httpx/_transports/default.py", line 60, in map_httpcore_exceptions
    yield
  File "/opt/apps/homeassistant/env/lib/python3.10/site-packages/httpx/_transports/default.py", line 353, in handle_async_request
    resp = await self._pool.handle_async_request(req)
  File "/opt/apps/homeassistant/env/lib/python3.10/site-packages/httpcore/_async/connection_pool.py", line 253, in handle_async_request
    raise exc
  File "/opt/apps/homeassistant/env/lib/python3.10/site-packages/httpcore/_async/connection_pool.py", line 237, in handle_async_request
    response = await connection.handle_async_request(request)
  File "/opt/apps/homeassistant/env/lib/python3.10/site-packages/httpcore/_async/connection.py", line 90, in handle_async_request
    return await self._connection.handle_async_request(request)
  File "/opt/apps/homeassistant/env/lib/python3.10/site-packages/httpcore/_async/http11.py", line 105, in handle_async_request
    raise exc
  File "/opt/apps/homeassistant/env/lib/python3.10/site-packages/httpcore/_async/http11.py", line 84, in handle_async_request
    ) = await self._receive_response_headers(**kwargs)
  File "/opt/apps/homeassistant/env/lib/python3.10/site-packages/httpcore/_async/http11.py", line 148, in _receive_response_headers
    event = await self._receive_event(timeout=timeout)
  File "/opt/apps/homeassistant/env/lib/python3.10/site-packages/httpcore/_async/http11.py", line 177, in _receive_event
    data = await self._network_stream.read(
  File "/opt/apps/homeassistant/env/lib/python3.10/site-packages/httpcore/backends/asyncio.py", line 30, in read
    with map_exceptions(exc_map):
  File "/usr/lib/python3.10/contextlib.py", line 153, in __exit__
    self.gen.throw(typ, value, traceback)
  File "/opt/apps/homeassistant/env/lib/python3.10/site-packages/httpcore/_exceptions.py", line 12, in map_exceptions
    raise to_exc(exc)
httpcore.ReadTimeout

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/opt/apps/homeassistant/env/lib/python3.10/site-packages/homeassistant/helpers/update_coordinator.py", line 205, in _async_refresh
    self.data = await self._async_update_data()
  File "/opt/apps/homeassistant/env/lib/python3.10/site-packages/homeassistant/helpers/update_coordinator.py", line 164, in _async_update_data
    return await self.update_method()
  File "/opt/apps/homeassistant/env/lib/python3.10/site-packages/homeassistant/components/luftdaten/__init__.py", line 43, in async_update
    await sensor_community.get_data()
  File "/opt/apps/homeassistant/env/lib/python3.10/site-packages/luftdaten/__init__.py", line 29, in get_data
    response = await client.get(str(url))
  File "/opt/apps/homeassistant/env/lib/python3.10/site-packages/httpx/_client.py", line 1751, in get
    return await self.request(
  File "/opt/apps/homeassistant/env/lib/python3.10/site-packages/httpx/_client.py", line 1527, in request
    return await self.send(request, auth=auth, follow_redirects=follow_redirects)
  File "/opt/apps/homeassistant/env/lib/python3.10/site-packages/httpx/_client.py", line 1614, in send
    response = await self._send_handling_auth(
  File "/opt/apps/homeassistant/env/lib/python3.10/site-packages/httpx/_client.py", line 1642, in _send_handling_auth
    response = await self._send_handling_redirects(
  File "/opt/apps/homeassistant/env/lib/python3.10/site-packages/httpx/_client.py", line 1679, in _send_handling_redirects
    response = await self._send_single_request(request)
  File "/opt/apps/homeassistant/env/lib/python3.10/site-packages/httpx/_client.py", line 1716, in _send_single_request
    response = await transport.handle_async_request(request)
  File "/opt/apps/homeassistant/env/lib/python3.10/site-packages/httpx/_transports/default.py", line 352, in handle_async_request
    with map_httpcore_exceptions():
  File "/usr/lib/python3.10/contextlib.py", line 153, in __exit__
    self.gen.throw(typ, value, traceback)
  File "/opt/apps/homeassistant/env/lib/python3.10/site-packages/httpx/_transports/default.py", line 77, in map_httpcore_exceptions
    raise mapped_exc(message) from exc
httpx.ReadTimeout


Change bf16 to amp_bf16

Hi,
I tried replicating the pretraining bert script and when I ran it with the yaml script I got the following error: Value bf16 is not available in Precision. I traced the code and changed bf16 to amp_bf16 to get it to work. Not sure if I am using the outdated code but flagging it here.
https://github.com/mosaicml/examples/blob/main/examples/bert/yamls/main/hf-bert-base-uncased.yaml  - line 82

Thanks for catching this! You are correct in using the amp_bf16 flag

@jacobfulano was this fixed?

@jacobfulano : I submitted a PR. Just got bitten by this myself.


Last list item in stocks list shouldn't have a row under it

Either the last list item in stocks list shouldn't have a row under it, or the first one should have a row above it. Not sure which, but either way it's inconsistent right now and that makes it ugly. (Most obvious in the PORTFOLIO pane.)

Maybe the MaterialList should custom paint the lines?  Then it could draw them in between the items.  Currently we the items themselves paint the lines and they don't know if they're first or last.

This is moot these days.


Handle POST requests with no data (e.g., ping)

I'm not 100% sure why no one else is seeing this, but it happens 100% of the time locally. Perhaps because I'm running it against a Github enterprise server (v2.8)?


DBZ-177 Changes the way the PostgreSQL connector loads the JDBC driver, to use the connector's classloader

@rhauch the CI failure seems to be around a MySQL 5.7 integration test

@hchiorean that failure is intermittent, and I haven't had time to investigate. I've restarted the build.

I'm still going to merge since the failure is unrelated to this change.


[5.7] Avoid breaking change of package removal

Perhaps we should move these from require to suggest in 5.8?

@GrahamCampbell this seems to break the Travis build.


Nodejs Debug params inside file itself instead of launch.json [Feature Request]

It will make our life easy if you allow passing debug params from file itself instead of configuring in launch.json everytime.
e.g. in the comment section, we should have something like this.

/**
debug-params: --env prod --job process-record --thread 10 --iid b048dc30
**/

So, while debugging current file, if we have something like above in the file itself then debugger should take params from the same file otherwise from launch.json
It is necessary because sometimes we have so many scripts with different param combinations and its overhead to put everything in launch.json

Thanks for this feature request However hard to happens since the vscode frontend is not lagnuage aware. On top of that vscode frontent is not aware of waht file will be entry point of debug, only the debug adapter is interpreting the program attribute.
I can keep this open but in reality I know we will not implement this since it goes against our architecture. So I choose to close it, hope it makes sense. Thanks
Also putting everyting in a file instead of a alunchjson does not seem like a huge benefit to me


git-delete-merged-branches: Fix >=6.3.0 dependencies

Related change upstream:
https://github.com/hartwork/git-delete-merged-branches/commit/440513c21e7b332c6b285d9f9a35354b62c74f15#diff-60f61ab7a8d1910d86d9fda2261620314edcae5894d5aaa236b821c7256badd7L36
CC @SuperSandro2000

Motivation for this change
Things done

Built on platform(s)

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

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

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

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

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

[ ] Fits CONTRIBUTING.md.

Thank you! I don't think I would have noticed that one for a while.


fix: added source headers

Added source headers to messages sent to Datasinks via Adapters (Data-Source-Name and  Data-Source-Type)

Can you run this by Gary as well please to make sure he's using the same header values in the doc pipeline?

This also isn't a fix, it's a new feature so feat: would make more sense.


PR: Bugfixes & Button Clicking & Other New Functions

Added Code/Stuff

Added button interactions
Fixed some typos/bugs here and there
Added the buttons and dropdowns attribute to messages
Added a fetch_button function which gets a button with the specified label
P.S. this adds onto @ThePrivatePanda's PR

Fork the Development branch not stable.


Write unit tests for bot/interpreter.py

Write unit tests for bot/interpreter.py.
Implementation details
Please make sure to read the general information in the meta issue and the testing README. We are aiming for a 100% branch coverage for this file, but if you think that is not possible, please discuss that in this issue.
Additional information
If you want to work on this issue, please make sure that you get assigned to it by one of the core devs before starting to work on it. We would like to prevent the situation that multiple people are working on the same issue. To get assigned, leave a comment showing your interesting in tackling this issue.

Internal discussions deemed these tests to be unnecessary.