dart run realm generate stopped working

What happened?
The following command has stopped working...
dart run realm generate
Repro steps
When running dart run realm generate the below is outputted in the terminal, no errors are reported in the terminal and no .g auto generated realm files are generated...
Building package executable... (1.5s)
Built realm:realm.

We have tried flutter pub clean cache and flutter clean and reinstalled flutter pub get, with no luck.
Version
realm 1.2.0, flutter 3.10.1
What Atlas Services are you using?
Both Atlas Device Sync and Atlas App Services
What type of application is this?
Flutter Application
Client OS and version
MacOS 13.3.1
Code snippets
No response
Stacktrace of the exception/crash you're getting
No response
Relevant log output
No response

Hi @dotjon0!
We will investigate that.
Meanwhile, you can try to upgrade to flutter 3.10.4 and to check if you will have the same issue.
You can also try flutter pub run build_runner build --delete-conflicting-outputs or dart run build_runner build --delete-conflicting-outputs

thank you @desistefanova. we have tried upgrading to flutter 3.10.4 and it made no difference.
when we run flutter pub run build_runner build --delete-conflicting-outputs it forces us to add the dependency build_runner: ^2.4.5, which we added. However when running flutter pub run build_runner build --delete-conflicting-outputs or dart run build_runner build --delete-conflicting-outputs it then gives the following error in the terminal:
Building package executable... (1.3s)
Failed to build build_runner:build_runner:
../../../../.pub-cache/hosted/pub.dev/build_runner_core-7.2.6/lib/src/logging/build_for_input_logger.dart:13:7: Error: The non-abstract class 'BuildForInputLogger' is missing implementations for these members:
 - Logger.onLevelChanged
Try to either
 - provide an implementation,
 - inherit an implementation from a superclass or mixin,
 - mark the class as abstract, or
 - provide a 'noSuchMethod' implementation.

class BuildForInputLogger implements Logger {
      ^^^^^^^^^^^^^^^^^^^
../../../../.pub-cache/hosted/pub.dev/logging-1.2.0/lib/src/logger.dart:162:22: Context: 'Logger.onLevelChanged' is defined here.

This should force the generating.

dart run realm generator --clean gives me this error in the terminal:
Building package executable...
Built realm:realm.
Could not find an option named "clean".

Usage: dart run realm|realm_dart <command> [arguments]

Global options:
-h, --help    Print this usage information.

Available commands:
  generate   Generate Realm objects from data model classes
  install    Download & install Realm native binaries into a Flutter or Dart project
  metrics    Report anonymized builder metrics to Realm

Run "dart run realm|realm_dart help <command>" for more information about a command.

Built realm:realm.
Could not find a command named "generator".
Did you mean one of these?
generate
Usage: dart run realm|realm_dart  [arguments]
Global options:
-h, --help    Print this usage information.
Available commands:
generate   Generate Realm objects from data model classes
install    Download & install Realm native binaries into a Flutter or Dart project
metrics    Report anonymized builder metrics to Realm
Run "dart run realm|realm_dart help " for more information about a command.
Perhaps 'generator' was meant to be 'generate', in which case I get the following and the .g auto generated files still fail to appear:
dart run realm generate --clean:
Building package executable...  Built realm:realm.
dart run realm generator:
Building package executable...  Built realm:realm.

No we did not have build_runner before, so I removed build_runner and I get the same (as below) with no .g files auto generated...dart run
realm generate --clean:
Building package executable...  Built realm:realm.

dart run realm generate:
Building package executable...  Built realm:realm.

Ahh, yes.
It happens to me, but only on the Mac OS.
It works on Windows and Linux.
I will investigate it.
Thanks for reporting.

Ah brilliant you can replicate. Thanks for confirming

This error is appears because of the logging package that is implemented inside build_runner core. The issue is fixed in build_runner core. In order to get the fix, just run:

dart pub upgrade
and then
dart run realm generate

Ah yes I thought it was strange the logging package was throwing an error in the above thread. So I tried what you suggested and it did not work, so I then re-added the build_runner: ^2.4.5 as a dev dependency (even though we do not use build_runner) and retried what you suggested (dart pub upgrade and dart run realm generate) and it worked. Thank you so much for your help on this.

Great!
build_runner: ^2.4.5 shouldn't be needed in the dependencies. Probably, if you try to clean and get the packages again it will be fixed.
Yes, definitely, we will check why the error is not reported to the console.

Thank you so much again @desistefanova

Hello,
'dart run realm generate'  command works, but build_runner still fails.
We are using stable channel and following is the output of flutter --version
$ flutter --version
Flutter 3.10.5 • channel stable • https://github.com/flutter/flutter.git
Framework • revision 796c8ef792 (4 weeks ago) • 2023-06-13 15:51:02 -0700
Engine • revision 45f6e00911
Tools • Dart 3.0.5 • DevTools 2.23.1
We tried with build_runner versions below and both failed with same output as below.
build_runner: ^2.3.3
build_runner: ^2.4.6
dart run build_runner build --delete-conflicting-outputs    -- produces the output below.
Building package executable... (2.0s)
Failed to build build_runner:build_runner:
../../.pub-cache/hosted/pub.dev/build_runner_core-7.2.8/lib/src/logging/build_for_input_logger.dart:13:7: Error: The non-abstract class 'BuildForInputLogger' is missing implementations for these members:

Logger.onLevelChanged
Try to either
provide an implementation,
inherit an implementation from a superclass or mixin,
mark the class as abstract, or
provide a 'noSuchMethod' implementation.

class BuildForInputLogger implements Logger {
^^^^^^^^^^^^^^^^^^^
../../.pub-cache/hosted/pub.dev/logging-1.2.0/lib/src/logger.dart:162:22: Context: 'Logger.onLevelChanged' is defined here.
Stream<Level?> get onLevelChanged {
^^^^^^^^^^^^^^
Failed to build build_runner:build_runner:
../../.pub-cache/hosted/pub.dev/build_runner_core-7.2.8/lib/src/logging/build_for_input_logger.dart:13:7: Error: The non-abstract class 'BuildForInputLogger' is missing implementations for these members:

Logger.onLevelChanged
Try to either
provide an implementation,
inherit an implementation from a superclass or mixin,
mark the class as abstract, or
provide a 'noSuchMethod' implementation.

class BuildForInputLogger implements Logger {
^^^^^^^^^^^^^^^^^^^
../../.pub-cache/hosted/pub.dev/logging-1.2.0/lib/src/logger.dart:162:22: Context: 'Logger.onLevelChanged' is defined here.
Stream<Level?> get onLevelChanged {
^^^^^^^^^^^^^^

@desistefanova thank you for the steps to upgrade!

on mac machine it didn't work. getting below response and nothing changes. ->
Building package executable...
Built realm:realm.

on mac machine it didn't work. getting below response and nothing changes. ->
Building package executable... Built realm:realm.

@VikasAwate5 if you see the resolution steps above which resolved this (I had exactly the same issue, thus this issue)


Issue: 400 Unsupported parameter: 'max_tokens' using model 'o1-preview'

Issue you'd like to raise.
Changed the model in Playground to use o1-preview (I have Tier 5 support) but received this error instead
400 Unsupported parameter: 'max_tokens' is not supported with this model. Use 'max_completion_tokens' instead.

Error: 400 Unsupported parameter: 'max_tokens' is not supported with this model. Use 'max_completion_tokens' instead.
    at APIError.generate (file:///node_modules/openai/error.mjs:41:20)
    at OpenAI.makeStatusError (file:///node_modules/openai/core.mjs:268:25)
    at OpenAI.makeRequest (file:///node_modules/openai/core.mjs:311:30)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async file:///node_modules/@langchain/openai/dist/chat_models.js:881:29
    at async RetryOperation._fn (/node_modules/p-retry/index.js:50:12)

Suggestion:
Support for new model from OpenAI

For me, I'm just not getting any traces from o1-preview

@dennisofficial do you have code to share for that?

Any updates here? Thanks!

any new updates? i am also seeing the same error


Different error response for non-active users

During the login process, if the user is not set to active, return a different error response so we can let the user know. Right now it just says that the username and password are incorrect.

Do you want the message to be something like "You are not activated yet"?

Yea that would be perfect!


Create parser for description fields

Use a package such as natural tp parse description fields and extract field info.
Generate output files like usajobs.parsed.data.json to illustrate results.

Here is my current progress:
Code | Data
For anyone who wishes to continue development, I added comments on my code to help.
Some additional comments:
/// will not match "Software Engineer" or "software engineering" in general
classifier.addDocument(['software engineering'], 'software engineer');

vs.

// matches, but also classify anything with "engineer" or "software" as software engineering ("eg. Security Engineer")
classifier.addDocument('software engineering', 'software engineer');

The latter ends up tagging the majority of the internships as "software engineering".
It might have something to do with how the words are tokenized when it's in an array vs. just the string:
// 'software engineer'
 doc: { label: 'software engineer', text: [ 'softwar', 'engin' ] }

vs.

// ['software engineer]
  doc: { label: 'software engineer', text: [ 'software engineering' ] }

It seems like there have been other issues reported: Link #1 | Link #2
As suggested in Link #1, we may find better results writing our own tokenizers & stemmers: Explanation. I'm not entirely satisfied with the current results but it does the basics for now.

My current progress, worked with Jenny's parser code from her previous comment:
Code | Data
Didn't feel comfortable writing a tokenizer/stemmer, so I experimented with using includes() to parse skills from the position titles. Appeared to mainly help with parsing Data Science positions, didn't seem to have a significant difference for other positions.
Also added a soft skills section. This works similarly to Jenny's parser for the regular skills, but uses a different classifier trained with soft skill keywords instead. These are added to the softSkills field for each job object.

@miraabela Some steps to move forward:

Please split the definitions of "categories" into its own file, and then import it into the file that does the parsing. (Maybe export a single object that's a data structure where keys are categories and values are arrays of strings?)

Here's a potential interface to the parser for use by any scraper:

const parser = new Parser(descriptionString);

const skills = parser.getSkills();  // returns an array of skills,  or null if nothing found.
const due = parser.getDue(); // returns the due date for applications, or null if nothing found.
const compensation = parser.getCompensation(); // returns the compensation, or null if not found.

etc.

The idea is that a scraper can try to get as many fields in the canonical schema as possible, then ask the parser to try to provide any remaining ones that it can.


Update deps.

@dvc94ch Any particular reason to re-add minor and patch versions? I asked to omit these in two recent PRs because we don't depend on a specific minor/patch version and semver allows bumping everything but the most-significant version field anyway.
Assuming you used a tool like "crates" in VSCode to bump these all at once, that will now unnecessarily say the specified version is out-of-date really quick for crates that see fast releases on their patch version.
Shall I send a PR to simplify these again?

@dvc94ch Any particular reason to re-add minor and patch versions? I asked to omit these in two recent PRs because we don't depend on a specific minor/patch version and semver allows bumping everything but the most-significant version field anyway.
Assuming you used a tool like "crates" in VSCode to bump these all at once, that will now unnecessarily say the specified version is out-of-date really quick for crates that see fast releases on their patch version.
Shall I send a PR to simplify these again?

I just run cargo upgrade --workspace, I don't see much benefit in simplifying the version numbers. Unless you specify =a.b.c cargo will still use the latest patch version and if vscode reports differently, then vscode is broken. But if you prefer...

I just run cargo upgrade --workspace, I don't see much benefit in simplifying the version numbers. Unless you specify =a.b.c cargo will still use the latest patch version and if vscode reports differently, then vscode is broken. But if you prefer...

I think you missed this option then:
--skip-compatible     Only update a dependency if the new version is semver incompatible

That is - IMO - the only reason for making updates in Cargo.toml: if it brings us a new version that would not be allowed otherwise.
It's not VSCode itself btw, just an extension :)

I think you missed this option then:
--skip-compatible     Only update a dependency if the new version is semver incompatible

That is - IMO - the only reason for making updates in Cargo.toml: if it brings us a new version that would not be allowed otherwise.
It's not VSCode itself btw, just an extension :)


Enhance availability of refineGEMs

To enhance the availability of the whole refineGEMs package I would suggest the following options:

[ ] Task 1: Rearrange & rewrite scripts if necessary to obtain a refineGEMs main script that can be run from the command line with refinegems config.yaml or just refinegems
[ ] Task 2: Provide refineGEMs also via conda

For task 1, another idea would be to split the tool into different entry points. One for model investigation, one for model refinement and one for model curation.
How to achieve?

Generate four 'main' modules/ command line names: data, investigate, refine and curate
Extract code from io and main for the relevant modules per new 'main' module (see table)
Add entry points to the setup.py

'Main' module/Command line name
Relevant modules
Description

data
databases
Handles database input/output & initialisation

investigate
comparison, growth, investigate, gapfill.gap_analysis
Handles model comparison, growth simulation & other types of model investigation

refine
biomass, charges, modelseed, pathways, polish, sboann
Handles all possible refinement techniques implemented in refineGEMs

curate
curate, gapfill.gapfill_model
Handles the curation implementations of refineGEMs

The setup.py should be replaced by a setup.cfg as the setup.py file is legacy and might get deprecated. A possible setup.cfg already including the setup for the possible entry points was generated and uploaded to the branch structure_update.

All commits from the last comment to this comment are the result of my attempt to rebase these commits from dev to another branch. However, this did not work and, thus, I will cherry-pick the according commits to the other branch. This will likely add the same commits again in this issue after this comment.

Currently, refineGEMs works for python 3.10 and 3.9.
Release 2.0.0 should work under 3.10 and 3.11

[ ] 3.10
[ ] 3.11


ap_hook_check_authn undefined symbol mod_auth_openidc.so

I am getting error while starting apache... with OpenIDConnect configure on OEL 7.6
Error Message : Cannot load /usr/lib64/httpd/modules/mod_auth_openidc.so into server: /usr/lib64/httpd/modules/mod_auth_openidc.so: undefined symbol: ap_hook_check_authn
LoadModule auth_openidc_module "/usr/lib64/httpd/modules/mod_auth_openidc.so"
OIDCProviderMetadataURL https://########/fss/.well-known/openid-configuration
OIDCClientID @@@@@@@@@
OIDCClientSecret !!!!!!!!!!!
OIDCProviderTokenEndpointAuth client_secret_basic
OIDCSSLValidateServer Off
OIDCRedirectURI http://*******:^^^^^/oidc/
OIDCCryptoPassphrase BCfnG8ZGgVrL7d@ItiHDJfsqwLYlhQfABNJ6bX
OIDCIDTokenIatSlack 3600
OIDCOAuthVerifySharedKeys plain
OIDCAuthNHeader REMOTE_USER
OIDCOAuthSSLValidateServer Off
OIDCPassClaimsAs headers
OIDCRemoteUserClaim sub
Server version: Oracle-HTTP-Server/2.2.22 (Unix)
Server built:   Feb  8 2013 20:58:44
Server label:   APACHE_<IP_ADDRESS>.0_LINUX.X64_130208.1901
Appreciate, Why this request made invalid

You can get commercial support for Oracle-HTTP-Server/2.2.22 via<EMAIL_ADDRESS>

Hi,
We are also getting same issue , Please let us the resolution as we are performing POC for this product.
Regards
falah

Hi,
We are also getting same issue , Please let us the resolution as we are performing POC for this product.
Regards
falah

Hi Falah,
It was not working on apache 2.2 . I have never encountered this error on apache 2.4 though.

I understand this is closed but see lots of people facing the issue. Did anybody find a resolution?

No solution till now, do let me know if you find any solution.
On Thu, 12 Mar 2020, 01:23 dileep55<EMAIL_ADDRESS>wrote:

I understand this is closed but see lots of people facing the issue. Did
anybody find a resolution?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/zmartzone/mod_auth_openidc/issues/438#issuecomment-597884976,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOWKDUUJPFLLV2WTJZ2VTYDRG76LTANCNFSM4HHQE3WA
.

I was getting the same error but it was for version apache 2.2. This error
does not come in apache 2.4.
What is the version of your webserver ?
Regards,
Puneet
On Thu, Mar 12, 2020 at 11:48 AM gmfalah<EMAIL_ADDRESS>wrote:

No solution till now, do let me know if you find any solution.
On Thu, 12 Mar 2020, 01:23 dileep55<EMAIL_ADDRESS>wrote:

I understand this is closed but see lots of people facing the issue. Did
anybody find a resolution?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<
https://github.com/zmartzone/mod_auth_openidc/issues/438#issuecomment-597884976
,
or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AOWKDUUJPFLLV2WTJZ2VTYDRG76LTANCNFSM4HHQE3WA
.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/zmartzone/mod_auth_openidc/issues/438#issuecomment-598025409,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOMYOYD6DM6RAGUYGHQEYW3RHB5DPANCNFSM4HHQE3WA
.

It is indeed 2.2 and I cannot upgrade because of vendor dependencies. Sent from my T-Mobile 4G LTE Device
-------- Original message --------From: puneetsharma3277<EMAIL_ADDRESS>Date: 3/12/20  3:51 AM  (GMT-06:00) To: zmartzone/mod_auth_openidc<EMAIL_ADDRESS>Cc: dileep55<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [zmartzone/mod_auth_openidc] ap_hook_check_authn undefined symbol
  mod_auth_openidc.so (#438)
I was getting the same error but it was for version apache 2.2. This error
does not come in apache 2.4.
What is the version of your webserver ?
Regards,
Puneet
On Thu, Mar 12, 2020 at 11:48 AM gmfalah<EMAIL_ADDRESS>wrote:

No solution till now, do let me know if you find any solution.

On Thu, 12 Mar 2020, 01:23 dileep55<EMAIL_ADDRESS>wrote:

I understand this is closed but see lots of people facing the issue. Did

anybody find a resolution?

—

You are receiving this because you commented.

Reply to this email directly, view it on GitHub

<

https://github.com/zmartzone/mod_auth_openidc/issues/438#issuecomment-597884976

,

or unsubscribe

<

https://github.com/notifications/unsubscribe-auth/AOWKDUUJPFLLV2WTJZ2VTYDRG76LTANCNFSM4HHQE3WA

.

—

You are receiving this because you commented.

Reply to this email directly, view it on GitHub

https://github.com/zmartzone/mod_auth_openidc/issues/438#issuecomment-598025409,

or unsubscribe

https://github.com/notifications/unsubscribe-auth/AOMYOYD6DM6RAGUYGHQEYW3RHB5DPANCNFSM4HHQE3WA

.

—You are receiving this because you commented.Reply to this email directly, view it on GitHub, or unsubscribe.
[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/zmartzone/mod_auth_openidc/issues/438#issuecomment-598073500",
"url": "https://github.com/zmartzone/mod_auth_openidc/issues/438#issuecomment-598073500",
"name": "View Issue"
},
"description": "View this Issue on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]

I think 2.4 apache is a pre-requisite . Not sure how will 2.2 work with
this oidc module. Please do inform here if it works .
Regards,
Puneet Sharma
Mobile +91<PHONE_NUMBER>
On Thu, Mar 12, 2020 at 6:19 PM dileep55<EMAIL_ADDRESS>wrote:

It is indeed 2.2 and I cannot upgrade because of vendor dependencies. Sent
from my T-Mobile 4G LTE Device
-------- Original message --------From: puneetsharma3277 <
<EMAIL_ADDRESS>Date: 3/12/20 3:51 AM (GMT-06:00) To:
zmartzone/mod_auth_openidc<EMAIL_ADDRESS>Cc:
dileep55<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [zmartzone/mod_auth_openidc] ap_hook_check_authn undefined
symbol
mod_auth_openidc.so (#438)
I was getting the same error but it was for version apache 2.2. This error
does not come in apache 2.4.
What is the version of your webserver ?
Regards,
Puneet
On Thu, Mar 12, 2020 at 11:48 AM gmfalah<EMAIL_ADDRESS>wrote:

No solution till now, do let me know if you find any solution.

On Thu, 12 Mar 2020, 01:23 dileep55<EMAIL_ADDRESS>wrote:

I understand this is closed but see lots of people facing the issue.
Did

anybody find a resolution?

—

You are receiving this because you commented.

Reply to this email directly, view it on GitHub

<

https://github.com/zmartzone/mod_auth_openidc/issues/438#issuecomment-597884976

,

or unsubscribe

<

https://github.com/notifications/unsubscribe-auth/AOWKDUUJPFLLV2WTJZ2VTYDRG76LTANCNFSM4HHQE3WA

.

—

You are receiving this because you commented.

Reply to this email directly, view it on GitHub

<
https://github.com/zmartzone/mod_auth_openidc/issues/438#issuecomment-598025409
,

or unsubscribe

<
https://github.com/notifications/unsubscribe-auth/AOMYOYD6DM6RAGUYGHQEYW3RHB5DPANCNFSM4HHQE3WA

.

—You are receiving this because you commented.Reply to this email
directly, view it on GitHub, or unsubscribe.
[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "
https://github.com/zmartzone/mod_auth_openidc/issues/438#issuecomment-598073500
",
"url": "
https://github.com/zmartzone/mod_auth_openidc/issues/438#issuecomment-598073500
",
"name": "View Issue"
},
"description": "View this Issue on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/zmartzone/mod_auth_openidc/issues/438#issuecomment-598167037,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOMYOYCEXMD5UVQCRDS2XADRHDK4RANCNFSM4HHQE3WA
.

There's no publicly available package for Oracle-HTTP-Server  but you may be able to build it yourself. As mentioned there are commercial support options (for all versions) that include prebuilt packages.


Handle requests with ActivityPub media type in Accept header

According to ActivityPub specification, servers should return ActivityStreams object representation in response to requests having application/ld+json; profile="https://www.w3.org/ns/activitystreams" in Accept header, but Guppe only does that for application/activity+json media type.

Servers ... MUST present the ActivityStreams object representation in response to application/ld+json; profile="https://www.w3.org/ns/activitystreams", and SHOULD also present the ActivityStreams representation in response to application/activity+json as well.

-- https://www.w3.org/TR/activitypub/#retrieving-objects
Example:
curl -H 'Accept: application/ld+json; profile="https://www.w3.org/ns/activitystreams"' https://a.gup.pe/u/3goodthings

This command returns 404 error page instead of the actor object.

Thanks, must have been a change in type-is since I setup validators.jsonld (at the time it errored if profile was included but ignored it otherwise, so I had to truncate the comparison type to just application/ld+json to make it match), and I can see that my units tests are inadequate to cover this case

Fix is deployed to Guppe now. Was this in relation to compatibility with a new fediverse app?

The fix is working for actors. However, when I send Follow() activity to group inbox, it returns 404. This could be a different problem, but the server response looks similar. My request has Content-Type header with value application/ld+json; profile="https://www.w3.org/ns/activitystreams".

Was this in relation to compatibility with a new fediverse app?

Yes, I noticed that my server doesn't federate properly with Guppe. Media type incompatibility is a common issue.

@silverpill please give it another try - updated again

It's working. Thanks a lot!


add an explicit latched property for republishers

Adds basic latched topic remapping functionality.
When multiple input latched topics are mapped to the same output topic, only one message will be latched for subscriptors connecting to this output topic.
Even if we try using a difference instance of rospy.Publisher for each input topic we're mapping, under the covers rospy uses a global singleton and thus re-uses the same publisher for all topics, leading to a single latched topic being stored and sent to subscribers.
In this MR, we force the use of a small wrapper LatchPublisher class for each input topic, so that the last message from each latched input topic is stored and forwarded to subscribers of the output topic.
Since having a different instance of LatchPublisher per mapping is not efficient, input topics that are latched and should follow this behavior must be explicitly marked with the latched property in its mapping.

thanks @FlorGrosso and @mechatheo for the contribution


config/universal: mark default ns create-only

This might have caused scheduling annotations and labels to be reset during e2e 🤦♂
So innocent:

/lgtm


Adding option to create primary objects with policy based authorization

This commit should enable creating primary objects with authorization based on policies with an option to enforce policy.

@williamcroberts I will add some tests for them soon, cool with the PR pending until then. These changes should be in the main branch since these are simply defining object authorization with policy defined with tpm2_createpolicy and does not have session dependencies (that can be satisfied using tpm_shell)

Added tests for this PR


Bump Plutus to latest version.

Drop the (defunct) fee input field.
Also fix some Haddock errors.

this would close #2291, right @EncodePanda ?

@JaredCorduan I think it does, yes. Let me close that ticket once I double-check this (I will upgrade dependencies on our end and see if everything works)

Oh, wait. I thought it was merged already. I will wait till it's merged and then try to update consensus with bumped ledger and plutus.

test/Test/Cardano/Ledger/Alonzo/Examples/Utxow.hs:81:1: error:
Error:     Could not load module ‘PlutusCore.Evaluation.Machine.ExBudgetingDefaults’
    It is a member of the hidden package ‘plutus-core-<IP_ADDRESS>’.
(....)
   |
81 | import qualified PlutusCore.Evaluation.Machine.ExBudgetingDefaults as P
   | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^...

Darn, I had hoped that error only came on my machine. It's perplexing and I don't understand it. Will stare at it some more.

🤞

thank you for figuring out what changed with the cost model @nc6 !


PORTALS-2715 - confirmation dialog on selection change

show confirmation dialog if attempting to change filter criteria when making a selection. this does not affect changes the current page or sort
refactor to remove InfiniteQueryWrapper, now can use QueryWrapper with isInfinite prop
refactor to move row selection into QueryContext from QueryVisualizationContext
add stories for EnumFacetFilter and RangeFacetFilter

known issue: adding a new filter and cancelling the change will not reset the form to the previous query state. To fix this I think we may need to track the "next"/"uncommitted" query in state (which is what the facet filter forms show), and revert this object when the change is cancelled. That itself is a fairly meaty refactor, so suggest following up in a separate ticket and PR.
https://github.com/Sage-Bionetworks/synapse-web-monorepo/assets/17580037/848d5111-a835-4abe-bf06-d0002b909d00

known issue: adding a new filter and cancelling the change will not reset the form to the previous query state.

What happens if you cancel the confirmation dialog today?  Will it update the query and clear selection (and if so, is there any point in showing this dialog until this issue has been fixed)?

known issue: adding a new filter and cancelling the change will not reset the form to the previous query state.

What happens if you cancel the confirmation dialog today? Will it update the query and clear selection (and if so, is there any point in showing this dialog until this issue has been fixed)?

The query (and data) will not change, but the form will still indicate that the query was changed. This is only an issue with the facet filter controls, since they maintain their own state that synchronizes (and triggers the confirmation) after a delay (for the EnumFacetFilter) or after clicking apply (for the RangFacetFilter.


Color background in XSHD

There are foreground and background properties on color tags in XSHD, but the background isn't used for anything. I have seen examples of background highlighting by implementing an IBackgroundRenderer, but that usually finds the highlighting by some custom searching or matching. Is there any way to just blanket render all the backgrounds defined in your syntax highlighting? I was toying with the idea of getting all of the matches and making an IBackgroundRenderer that then colors them, but that's complicated by the fact that syntax highlighting only keeps track of part of the document at a time.
Any advice on this would be appreciated. I know I could always probably recreate the matching conditions I define in the syntax highligting, but that would mean a lot more file parsing (and we have some massive files) and redundant parsing, when what I want is exactly there already.

I just tested this. Setting the background property on Color or Keywords elements is working just fine. This was implemented in fc54a4e6eed67224536a966bc588829c18a16e40 over 7 years ago.


Improved code to Make white space normal the default for labels (#1zhk016)

This node version issue is taken care by Disha.

Currently don't have a great way to do this. will try again later.


Payment module problem

when member buys one of my memberships, it won't update his membership status.After payment is made i get an error "token expired" any idea?
Thanks for any help

What payment gateway did you use? Is the sandbox enabled.
Are the API key of the payment gateway you enabled is correctly configured at Admin -> Mod -> Billing -> Gateways Configuration?
Can you provide a screenshot of the "token expired" page, so I can see better from where it is.

Hi,
I'm using built in PayPal with sandbox disabled. Where do I put the API?
Thanks,
Walter

pH7Builder (pH7CMS) doesn't use PayPal Express, but just "PayPal Standard", so it's just your PayPal email address. That's all.

Concerning the screenshot, like I said, could you provide a screenshot of the "token expired" page, so I can see better from where it is. Thanks for the details! :)

I'm using only my email address for Paypal.This is the msg I get after payment is made to upgrade my membership.BTW, membership never gets updated even after payment.Its always "Free" see attached

screen shot

This issue has been fixed in pH7Builder, v15.0.0
Related ticket https://github.com/pH7Software/pH7-Social-Dating-CMS/issues/507#issuecomment-482691122


Clears timeout on calling done()

Aims to prevent this situation from occurring when running within a Jest test, I assume as a result of the timeout executing after the done() callback is called:
Jest has detected the following 1 open handle potentially keeping Jest from exiting:
    > 76 |     mockContext = AWSMockLambdaContext({
         |                                       ^
      77 |       timeout: 3
      78 |     });
      79 |

      at Object.<anonymous>.module.exports.options [as default] (node_modules/aws-lambda-mock-context/index.js:66:2)

Also having this issue. Any update on this pull request?

Hello, any news on this ? Thanks.


Feature/auto wiring

(Work in progress)
Overview
Added a feature for auto-assigning channels for a collection of Quantum elements given a specific QM instrument setup.
graph TD
    A(Instruments) --> D["allocate_wiring()"]
    C(Connectivity) --> D["allocate_wiring()"]
    subgraph Wirer
        D["allocate_wiring()"]
    end

Features
Supports the following:

assignment of channels to any combination of MW-FEM/LF-FEM/Octave/OPX+.
any mapping of N resonator lines to M qubits
any mapping of N fems to OPX1000 chassis slots
overflowing of assignment to multiple slots, chassis, modules, octaves, etc.
any combination of resonator, flux line, coupler line for each qubit.
hardcoding of con/slot/port for particular lines
basic visualization of a single OPX1000

Safety
It has some simple validation:

can't mix OPX+ and OPX1000
can't define two FEMs on the same slot on the same chassis

Examples
    instruments = Instruments()
    instruments.add_lf_fem(con=1, slots=1)
    instruments.add_mw_fem(con=1, slots=2)

    qubits = [1, 2, 3, 4, 5, 6]
    connectivity = Connectivity()
    connectivity.add_resonator_line(qubits=qubits)
    connectivity.add_qubit_drive_lines(qubits=qubits)
    connectivity.add_qubit_flux_lines(qubits=qubits)

    allocate_wiring(connectivity, instruments)

    if visualize:
        visualize_chassis(connectivity.elements, instruments.available_channels)

@TheoLaudatQM this is ready to for a final review and merge!
(the black formatting is failing, but I can't reproduce it locally).

Do poetry install to install the same version of black.

To me it is ready to merge, but I am not sure how to fix the linting issues raised by the test... (black is okay)

To me it is ready to merge, but I am not sure how to fix the linting issues raised by the test... (black is okay)

@TheoLaudatQM @deanpoulos - If you're stuck, schedule 30 minutes with me.

Essentially flake8 doesn't want the init.py files so I could just delete them but maybe they are here for a reason so let's wait for @deanpoulos

[celebrate]     Theo Laudat reacted to your message:

From: Dean Poulos @.>
Sent: Tuesday, October 22, 2024 2:05:47 PM
To: qua-platform/py-qua-tools @.>
Cc: Theo Laudat @.>; Mention @.>
Subject: Re: [qua-platform/py-qua-tools] Feature/auto wiring (PR #229)
Merged #229https://github.com/qua-platform/py-qua-tools/pull/229 into main.
—
Reply to this email directly, view it on GitHubhttps://github.com/qua-platform/py-qua-tools/pull/229#event-14786215006, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AXR3HVSE5EB7IW3NSSFELFTZ4ZLTXAVCNFSM6AAAAABNEBGCBKVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJUG44DMMRRGUYDANQ.
You are receiving this because you were mentioned.Message ID: @.***>


Improve project_gdf performance

Is your feature proposal related to a problem?
No.
Describe the solution you'd like to propose
For UTM projection in project_gdf function, unary_union is being used which is a computationally expensive operation. Since for finding UTM there's no need for exact location of the centroid, I propose to use representative_point function of GeoPandas to speed up the process. This is the function description:

Returns a GeoSeries of (cheaply computed) points that are guaranteed to be within each geometry.

Additional context
This is the change that I am proposing:
Replace:
avg_lng = gdf["geometry"].unary_union.centroid.x

with:
avg_lng = gdf["geometry"].representative_point().x.mean()

An example for comparing their performance:
import osmnx as ox

street = ox.graph_from_bbox(west=-96.0255, south=29.421, east=-94.782, north=30.512)
street = ox.graph_to_gdfs(
    street,
    nodes=False,
    node_geometry=False,
    fill_edge_geometry=True,
)

We can clearly see the performance gain from this change.

This is a clever suggestion. Yes, I think representative_point is good enough for auto-determining a UTM zone. Would you like to open a simple PR to contribute this performance enhancement?

Sure, would be happy to.

Also, I found this for computing zone number, do you want to follow that logic or prefer to keep it as is?

do you want to follow that logic or prefer to keep it as is?

Let's keep it simple and consistent for now. This might be worth revisiting in the future.

Closed by #743.


 This is because ChunkedEncoding's new reader function works via BufRead.
A downside of this PR is that read_to_end is only available for TryBufRead types, which we only define for HttpConnection in the library, and the tokio adapter in the tests.
The PR can most likely be simplified, I was just aiming to get it working and it took long enough.

Do we want to return BufferTooSmall errors in the "outer" read_to_end?


Automatically install missing packages

This is a branch off of https://github.com/neuropoly/slicer-manual-annotation/pull/41
TODO:

[ ] Remove installation issue from README troubleshooting
[ ] List required packages in requirements.txt (best practice)
[ ] Check if package installation failed

Tested on fresh Slicer install, and issue related to the missing qt import:
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/Applications/Slicer.app/Contents/lib/Python/lib/python3.9/imp.py", line 169, in load_source
    module = _exec(spec, sys.modules[name])
  File "<frozen importlib._bootstrap>", line 613, in _exec
  File "<frozen importlib._bootstrap_external>", line 850, in exec_module
  File "<frozen importlib._bootstrap>", line 228, in _call_with_frames_removed
  File "/Users/julien/code/slicer-manual-annotation/SlicerCART/src/SlicerCART.py", line 67, in <module>
    check_and_install_python_packages()
  File "/Users/julien/code/slicer-manual-annotation/SlicerCART/src/SlicerCART.py", line 52, in check_and_install_python_packages
    response = qt.QMessageBox.question(slicer.util.mainWindow(), 'Install Extensions', msg,
NameError: name 'qt' is not defined
[Qt] loadSourceAsModule - Failed to load file "/Users/julien/code/slicer-manual-annotation/SlicerCART/src/SlicerCART.py"  as module "SlicerCART" !
[Qt] Fail to instantiate module  "SlicerCART"
[Qt] The following modules failed to be instantiated:
[Qt]    SlicerCART
[Qt] Populating font family aliases took 106 ms. Replace uses of missing font family ".AppleSystemUIFont" with one that exists to avoid this cost.

Example of qt box on Slicer fresh install:


Using O2 and stack_arrays for ICON package

This change is already contained in #1011

Perfect. I didn't know. Thanks! Just for my understanding it is included there, as it is defined now in the main repo?

Cool, thanks :)


resolve asff parser doing

Why is Kubernetes Deployment failing?

That PostgresHA test has been super flakey of late - not sure why as well.

@devGregA could you please merge this?


[Travis Debugging] [Feature] [Migration] Add Institution Logout Option [#OSF-6590]

Purpose
In addition to being logged out of OSF, users who log in through institutions will be logged out of their institution as well. This is an option.
Changes
Add logout_url to institution model. The rest is handled by CAS.
Side effects

Migration required: python -m scripts.populate_institutions test;
Merged together with CAS-PR.

Ticket
https://openscience.atlassian.net/browse/OSF-6590
CAS-PR
https://github.com/CenterForOpenScience/cas-overlay/pull/20

Not sure about this error. All local tests (osf, api, addons and admin) passed.

@cslzchen Re: 502825b8860fe6c6c35d7cf6f2a87fba8b5ea72f, I believe you could also use the only feature (as mentioned in Flowdock), to instruct Mocha to only run one test.


Company not found - TEKINSPORTS

ENVIRONMENT: production
URL: https://solr.peviitor.ro
Browser: Microsoft Edge
Device: Laptop
OS: Windows 10
STEPS TO REPRODUCE:

Open URL in the browser
Go to "TEKINSPORTS" Company
Open TEKINSPORTS website

EXPECTED RESULTS:
The TEKINSPORTS website opens and active jobs can be checked
ACTUAL RESULTS:
The TEKINSPORTS website cannot be found and active jobs cannot be checked

Removed the expired job. The company should automatically delete.

This company is no longer on the peviitor website


[10.x] Adds ModelsServiceProvider

In laravel 10.x version models may have many things configured while booting.
The method call can get scattered in generic service providers and be eventually hard to find.
So we can have a brand new ModelsServiceProvider.php in the app/providers folder like this:

use Illuminate\Foundation\Support\Providers\ModelsServiceProvider as CoreModelsServiceProvider;

class ModelsServiceProvider extends CoreModelsServiceProvider
{
    /**
     * @var array<string, string>
     */
    protected $morphMap = [
         'comment' => 'App\Models\Comment',
         ...
    ];

    protected function preventLazyLoading()
    {
        return ! $this->app->isProduction();
    }
}

and you get the idea.

Thanks for your pull request to Laravel!
Unfortunately, I'm going to delay merging this code for now. To preserve our ability to adequately maintain the framework, we need to be very careful regarding the amount of code we include.
If possible, please consider releasing your code as a package so that the community can still take advantage of your contributions!
If you feel absolutely certain that this code corrects a bug in the framework, please "@" mention me in a follow-up comment with further explanation so that GitHub will send me a notification of your response.


Update request stub to use generics

This is already used in the default laravel stubs
https://github.com/laravel/framework/blob/9.x/src/Illuminate/Foundation/Console/stubs/request.stub#L22

Thanks!


[IoTCentral]CLI documentation improvements

Description
[IoTCentral]CLI documentation improvements
Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR title and description has followed the guideline in Submitting Pull Requests.

[x] I adhere to the Command Guidelines.

@zhoxing-ms @MyronFanQiu @Juliehzl , Please take a look at this PR changing some help text. And for the build test error:
test_network_security_partner_provider (azure.cli.command_modules.network.tests.latest.test_network_commands.NetworkSecurityPartnerProviderScenarioTest)
test_network_virtual_appliance (azure.cli.command_modules.network.tests.latest.test_network_commands.NetworkVirtualApplianceScenarioTest)
they seems are not introduced by my change but introduced by network command modules. Please help on that part. Thanks

IoTCentral

@kawayiboy Could you please sync the code from the remote dev branch?
Then please rerun the test of test_network_virtual_appliance in live mode, and submit the yaml file which is re-recorded.
( Run azdev test test_network_virtual_appliance --live locally)

@zhoxing-ms ,  I fixed the build

@zhoxing-ms @MyronFanQiu Could you help merge this PR? thanks


mgmt/mcumgr/lib: Add OS group command for obtaining mcumgr parameters

The PR adds two commits that:

add documentation for new OS management command that allows to obtain MCUMGR parameters;
add implementation for that commit together with Kconfig options.

Due to lack of better suggestions to https://github.com/zephyrproject-rtos/zephyr/issues/42325 this is the best thing we can have now.

Fixed, moved to zcbor, tested.


Update README.md added v1.4.1 release notes link

Category

[x] Content fix
[ ] New article

What's in this Pull Request?
Added link to SPFx v1.4.1 release notes

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

README.md
:white_check_mark:Succeeded

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

thx Vardhaman, appreciated.


increase test cov memstore

Codecov Report

Merging #180 into master will increase coverage by 0.42%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master    #180      +/-   ##
=========================================
+ Coverage   73.17%   73.6%   +0.42%
=========================================
  Files         111     111
  Lines       17181   17065     -116
=========================================
- Hits        12572   12560      -12
+ Misses       3646    3550      -96
+ Partials      963     955       -8

Impacted Files
Coverage Δ

memstore/common/upsert_batch_header.go
61.11% <ø> (+9.22%)
:arrow_up:

memstore/upsert_batch.go
71.98% <100%> (+2.35%)
:arrow_up:

memstore/kafka_redolog_manager.go
68.59% <0%> (-2.48%)
:arrow_down:

memstore/cuckoo_index.go
90.2% <0%> (-1.02%)
:arrow_down:

memstore/common/data_value.go
82.15% <0%> (+21.18%)
:arrow_up:

Continue to review full report at Codecov.

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


SSL Wildcard certificate update not picking up new URL addition

We recently updated our wildcard to include another site (from 3 to 4 urls).
In Rancher Infrastructure - Certificates, we added a new certificate. The new wildcard entry actually picks up the new url ("www.mainsite.com + 4 others"). We then updated the load balancer ensuring that the new wildcard is selected. The certificate does not seem to work on the new site, even after waiting for 1-2 days to propagate. Have we missed a step? Does Rancher cache the old certificate and need a manual refresh?
We notice that in AWS, there's wildcard load balancer with SSL Certificate setting in its "Listeners" section. But won't Rancher manage the setting for certificate?
We also want to update the EV certificate on top of the wildcard certificate. Will the step be the same?
Many thanks.

Useful
Info

Versions
Rancher v0.63.0 Cattle: v0.155.0 UI: v0.94.0

Access
github admin

Route
authenticated.project.help

Rancher is generally cloud-agnostic and does not know of or manage resources in AWS for you (with a few exceptions*).  The load balancer in Rancher is made up of containers running HAProxy and does not interact with ELB.  Changing a cert should "immediately" (seconds) update the balancer containers.
But it sounds like you have ELB in front of Rancher and it's doing SSL termination, so that's where the cert needs to be.  ELB could be configured several ways:

Terminate the SSL connection and send plaintext traffic to the backend:  Rancher doesn't need any cert.  Since you seem to be a financial institution, that sounds like a bad idea.
Terminate the SSL connection and re-encrypt it with a (hopefully) different cert: Rancher balancer needs to be configured to listen using the (hopefully) different cert.  Maybe this is what you're talking about with EV vs wildcard?  Because nobody will issue you a cert that is both EV+wildcard.
TCP balancing, do not terminate the connection: Rancher balancer needs the original mainsite.com cert because it is terminating the SSL connection.

*: Creating new EC2 instances via Add Host calls docker-machine to do things, and the Route53 DNS template in the catalog manages DNS entries.

Thanks for the explanation @vincent99.
We figured out the issue by checking up the logs in the Rancher elb. inconsistencies between private key and certificate loaded from PEM file turns out that the private key used was incorrect. The devops guy haven't documented which key is which. It actually is simple to setup wildcard and EV in Rancher. :)


Use native platform for stat by default

instead of using Java NIO APIs with a File.exists()
on Linux and Windows.
This is a preparation for #12966.

@bot-gradle test this

OK, I've already triggered ReadyForMerge build for you.


Filter scope by client/user

This is a proposal to address the question I raised a couple days ago. It allows filtering scopes by user permissions by attaching "valid" and "default" scopes to the client and "allowed" scopes to the user object. These can be checked in validateScopes

Hi there! Pardon the late reply, I've been quite preoccupied these last few days.
I think what you're saying makes sense. Thus far, I'd been considering the user as being the owner (implying total control) of one or multiple resources, access to which will be delegated to clients. But it seems reasonable that different types of users might have different levels of access to resource instances, as well. If you have any use-case in particular I'd be interested in hearing it, just to wrap my mind around it all properly.
I'll get on this as soon as I can. Meanwhile, I'll bump @thomseddon and @nunofgs -- would appreciate your input on this matter!

No worries. The simplest use case I can think of is to use scopes to manage roles, e.g. "user", "superuser", and "admin". Admittedly the app could do this internally but if the OAuth server could manage it that's less code the app needs to maintain.
More realistically, let's say my company has services it offers through various platforms. The services are "Maintenance History", "Parts and Labor", and "Diagnostics" and the platforms are "Shop Management" and "Repair Information". The default service offered through "Repair Information" is "Parts and Labor" and the defaults for "Shop Management" are the others, "Vehicle History" and "Diagnostics". Purchasing each product adds its default services to the user's profile as scopes.
Normally, if a user logs into "Repair Information", he's granted its default scope, "Parts and Labor", but if this user has purchased both products, then all three scopes are part of his user profile and the "Repair Information" product should expose the "Diagnostics" service as well.

Just passed the scope to the saveRefreshToken call as well. I'm not clear why this was omitted but it can be removed or moved to a separate PR if desired.

The only comment I've got so far is that useClientCredentialsGrant needs a slight modification for this to work -- I'd like to propose self.scope = self.req.body.scope || self.client.defaultScope; (grant.js, line 308) in order to allow the client to solicit a particular scope and fall back on the defaultScope settings introduced in this PR. Does that seem reasonable?

Yep.

Thank you for the PR @ccamarat. :)
@thomseddon, any outstanding issues before this can be merged into 3.0?


Issues/introduce ipaddress

Introduced IpAddress package
Created Readme.md for IpAddress package
Added reference in Nox.Demo

Just one comment, about docs
I saw removed exception docs, but better to have them and describe situation, on which this exception can happen
Also, let's not have just SomeType, but better some human-readable description, like IPAddress detection result object with error or country code, just as en example

Just one comment, about docs I saw removed exception docs, but better to have them and describe situation, on which this exception can happen Also, let's not have just SomeType, but better some human-readable description, like IPAddress detection result object with error or country code, just as en example

Logic was changed. It doesn't throw exception at all. It should get Search Result (Success,InvalidInput, NotFound)

Just one comment, about docs I saw removed exception docs, but better to have them and describe situation, on which this exception can happen Also, let's not have just SomeType, but better some human-readable description, like IPAddress detection result object with error or country code, just as en example

I guess these are straight forward results. What else can be explained if IpAddress not found, or InvalidInput. Surely, it's only single property with Message, but it's extra property.


Axios is not showing all headers of response

My backend app is sending JWT after login in Authorization header.
I do not see this header with Axios even if I see that in dev tools.
It is definitely not as it should be.

Please take a look at this question and answer: http://stackoverflow.com/questions/37897523/axios-get-access-to-response-header-fields

Also good reading
https://developer.mozilla.org/en-US/docs/Web/HTTP/Access_control_CORS#Access-Control-Expose-Headers

The thing is that it got the Access-Control-Request-Headers: Authorization in the preflight OPTIONS request, and it still can't access that  in the response.headers of the response of the following POST request.

Got the same problem as @greenlaw110. Any solutions?

Got the same problem as @greenlaw110. Any solutions?

@annso92 and @vinnyfs89
Unfortunately I still don't have a solution to this one. I've posted a SO here: https://stackoverflow.com/questions/61164197/axios-response-headers-missing-data-when-running-in-vuejs-app
In case you guys have a solution, please help to anwser my SO above. Thanks a lot!

At your app under cors ou need to enable Access-Control-Expose-Headers
example
nestjs backend app
app.enableCors({
    exposedHeaders: 'session-id'
  });

Now your client app will be able to access that response header


backend/wayland: flush remote display

It turns out wl_event_source_check is not enough to guarantee that the
remote wl_display will be flushed after we queue requests. We need to
explicitly flush, just like we do in our X11 code.
References: https://gitlab.freedesktop.org/wayland/wayland/-/issues/187

It turns out this is not a workaround, this is what needs to be done for correct wl_event_loop usage. See upstream issue for details.

It turns out this is not a workaround, this is what needs to be done for correct wl_event_loop usage. See upstream issue for details.


Main screen split view

This is not really a problem, but an enhancement.
mitmproxy's main screen lists the requests along with the response's status code. When you're pentesting, you need to take a look at each response, but you need to enter each one and switch to the response tab.
Now think about sifting through that when you have more than a thousand requests to check?
Would it be possible for the team to create a split view of the main screen where we can see the response of the highlighted request? I've created a (very ugly) mockup of my idea below.
Sorry, couldn't upload it here, github might have some aesthetic bots to block it...

Hi! We already support this. You can cycle through layouts using - and then pressing P in the second view. 😃


Iso surfaces broken

The surface is missed. This is not fixed by increasing the sampling rate and it exists in both regular and unstructured grids.

Hey @StasJ , is there a scaling factor being applied to the volume here through the instance transform (or grid spacing?). Could you try disabling AO and see if that resolves the artifact? (setting aoSamples 0 on the renderer).

Hi @Twinklebear, there is no scaling factor however disabling AO does resolve the artifact.

Oh cool, that's good to hear then since this is likely also an issue we've got fixed in the coming release. The latest release of OpenVKL added support for returning an epsilon value (i.e., for shadow rays) from the hit iterator which we now use in the renderer to adjust the AO ray interval and avoid these artifacts. This should be fixed if you use the latest OpenVKL release and OSPRay devel, and when the next OSPRay release is out

Sounds good, thanks!

I'll close since the fix is in v2.2.0; please reopen if there are still artifacts.


updating the eigen commit pointer - do not merge

issuing PR only for the CI run...please do not merge

Jenkins:  retest this please


com.domenicseccareccia.jpegautorotate.JpegAutorotateException: Unable to read JPEG image EXIF Image Width and/or Image Height metadata tags.

Hi,
With some jpegs, I am seeing this, and not sure of the best way to check for the details.
com.domenicseccareccia.jpegautorotate.JpegAutorotateException: Unable to read JPEG image EXIF Image Width and/or Image Height metadata tags.
thanks

Hi,
First, you can first check if the image you are rotate has the appropriate EXIF Image Width/Height metadata tags. You can use http://metapicz.com/#landing to check if such attributes are missing from your image.
Secondly, from a debugging perspective you can look at updateAllMetadataDimensions() method in the JpegImageMetadata class.
Rotation will not be possible unless the image file has these two properties.

Thanks, I appreciate the response.   I should have been more specific, as I'm trying to get the syntax right to check for the appropriate EXIF Image Width/Height metadata tags in an if statement.

This library does not provide the ability to extract the EXIF Image Width/Height metadata tags to check appropriateness. It simply just expects an image as input and returns either the rotated image or an error. Extraction and checking is handled for you when determining whether or not it is appropriate to rotate the image.


Is this real time/real time alternatives

I know this isn't an issue, but didn't know where else to ask so:
Is this real time depth estimation given a monocular lens, if not does anyone know of any that don't require extensive GPU that are quite accurate?
Thanks

Hi, currently the method is offline and is not realtime. To adapt to these models, it won't be too hard to implement an interface similar to the MiDaS one and enable MobileNetV3 as the backend.

This is probably the answer you are looking for


FATAL ERROR: Reached heap limit Allocation failed - JavaScript heap out of memory

I'm trying to migrate an existing project that uses rules_nodejs (5.6.0) to rules_js (1.4.0) and rules_ts (1.0.0-rc3) and after fixing various issues and trying to build the whole repo I can't seem to ever complete a full build.
I've seen a few different log outputs from when it fails, but I'm guessing they're all symptoms of the same problem, all heap memory being allocated. I'm trying to understand if this is from bazels JVM or if this is node itself (most likely) and how to mitigate it. Any advice would be grateful.
$ bazel build //...
INFO: Analyzed 1243 targets (0 packages loaded, 0 targets configured).
INFO: Found 1243 targets...
ERROR: /home/fredrik/code/monolith/modules/user/BUILD.bazel:4:12: Compiling TypeScript project //modules/user:user [tsc -p tsconfig.bazel.json] failed: Worker process did not return a WorkResponse:

---8<---8<--- Start of log snippet, file at /home/fredrik/.cache/bazel/_bazel_fredrik/b93f0bbfc3437d73f39541651170604b/bazel-workers/worker-14-TsProject.log ---8<---8<---
[... truncated ...]
9.3 (4125.2) -> 4034.8 (4126.7) MB, 86.5 / 0.0 ms  (average mu = 0.626, current mu = 0.669) allocation failure
[21327:0x5fc7fc0]  2089077 ms: Scavenge 4041.0 (4126.7) -> 4037.0 (4128.7) MB, 69.3 / 0.1 ms  (average mu = 0.626, current mu = 0.669) allocation failure
[21327:0x5fc7fc0]  2093077 ms: Scavenge 4043.4 (4129.5) -> 4039.4 (4147.0) MB, 3729.2 / 0.0 ms  (average mu = 0.626, current mu = 0.669) allocation failure

<--- JS stacktrace --->

FATAL ERROR: Reached heap limit Allocation failed - JavaScript heap out of memory
 1: 0xb09c10 node::Abort() [/home/fredrik/.cache/bazel/_bazel_fredrik/b93f0bbfc3437d73f39541651170604b/execroot/monolith/bazel-out/k8-opt-exec-2B5CBBC6-ST-625e526ca8a8/bin/external/npm_typescript/tsc_worker.sh.runfiles/monolith/../nodejs_linux_amd64/bin/nodejs/bin/node]
 2: 0xa1c193 node::FatalError(char const*, char const*) [/home/fredrik/.cache/bazel/_bazel_fredrik/b93f0bbfc3437d73f39541651170604b/execroot/monolith/bazel-out/k8-opt-exec-2B5CBBC6-ST-625e526ca8a8/bin/external/npm_typescript/tsc_worker.sh.runfiles/monolith/../nodejs_linux_amd64/bin/nodejs/bin/node]
 3: 0xcf8dbe v8::Utils::ReportOOMFailure(v8::internal::Isolate*, char const*, bool) [/home/fredrik/.cache/bazel/_bazel_fredrik/b93f0bbfc3437d73f39541651170604b/execroot/monolith/bazel-out/k8-opt-exec-2B5CBBC6-ST-625e526ca8a8/bin/external/npm_typescript/tsc_worker.sh.runfiles/monolith/../nodejs_linux_amd64/bin/nodejs/bin/node]
 4: 0xcf9137 v8::internal::V8::FatalProcessOutOfMemory(v8::internal::Isolate*, char const*, bool) [/home/fredrik/.cache/bazel/_bazel_fredrik/b93f0bbfc3437d73f39541651170604b/execroot/monolith/bazel-out/k8-opt-exec-2B5CBBC6-ST-625e526ca8a8/bin/external/npm_typescript/tsc_worker.sh.runfiles/monolith/../nodejs_linux_amd64/bin/nodejs/bin/node]
 5: 0xeb09d5  [/home/fredrik/.cache/bazel/_bazel_fredrik/b93f0bbfc3437d73f39541651170604b/execroot/monolith/bazel-out/k8-opt-exec-2B5CBBC6-ST-625e526ca8a8/bin/external/npm_typescript/tsc_worker.sh.runfiles/monolith/../nodejs_linux_amd64/bin/nodejs/bin/node]
 6: 0xec069d v8::internal::Heap::CollectGarbage(v8::internal::AllocationSpace, v8::internal::GarbageCollectionReason, v8::GCCallbackFlags) [/home/fredrik/.cache/bazel/_bazel_fredrik/b93f0bbfc3437d73f39541651170604b/execroot/monolith/bazel-out/k8-opt-exec-2B5CBBC6-ST-625e526ca8a8/bin/external/npm_typescript/tsc_worker.sh.runfiles/monolith/../nodejs_linux_amd64/bin/nodejs/bin/node]
 7: 0xec339e v8::internal::Heap::AllocateRawWithRetryOrFailSlowPath(int, v8::internal::AllocationType, v8::internal::AllocationOrigin, v8::internal::AllocationAlignment) [/home/fredrik/.cache/bazel/_bazel_fredrik/b93f0bbfc3437d73f39541651170604b/execroot/monolith/bazel-out/k8-opt-exec-2B5CBBC6-ST-625e526ca8a8/bin/external/npm_typescript/tsc_worker.sh.runfiles/monolith/../nodejs_linux_amd64/bin/nodejs/bin/node]
 8: 0xe848da v8::internal::Factory::NewFillerObject(int, bool, v8::internal::AllocationType, v8::internal::AllocationOrigin) [/home/fredrik/.cache/bazel/_bazel_fredrik/b93f0bbfc3437d73f39541651170604b/execroot/monolith/bazel-out/k8-opt-exec-2B5CBBC6-ST-625e526ca8a8/bin/external/npm_typescript/tsc_worker.sh.runfiles/monolith/../nodejs_linux_amd64/bin/nodejs/bin/node]
 9: 0x11fd626 v8::internal::Runtime_AllocateInYoungGeneration(int, unsigned long*, v8::internal::Isolate*) [/home/fredrik/.cache/bazel/_bazel_fredrik/b93f0bbfc3437d73f39541651170604b/execroot/monolith/bazel-out/k8-opt-exec-2B5CBBC6-ST-625e526ca8a8/bin/external/npm_typescript/tsc_worker.sh.runfiles/monolith/../nodejs_linux_amd64/bin/nodejs/bin/node]
10: 0x15f2099  [/home/fredrik/.cache/bazel/_bazel_fredrik/b93f0bbfc3437d73f39541651170604b/execroot/monolith/bazel-out/k8-opt-exec-2B5CBBC6-ST-625e526ca8a8/bin/external/npm_typescript/tsc_worker.sh.runfiles/monolith/../nodejs_linux_amd64/bin/nodejs/bin/node]
/home/fredrik/.cache/bazel/_bazel_fredrik/b93f0bbfc3437d73f39541651170604b/execroot/monolith/bazel-out/k8-opt-exec-2B5CBBC6-ST-625e526ca8a8/bin/external/npm_typescript/tsc_worker.sh: line 440: 21327 Aborted                 (core dumped) "$JS_BINARY__NODE_WRAPPER" ${JS_BINARY__NODE_OPTIONS[@]+"${JS_BINARY__NODE_OPTIONS[@]}"} -- "$entry_point" ${ARGS[@]+"${ARGS[@]}"} 0<&0
---8<---8<--- End of log snippet, 104 chars omitted ---8<---8<---
ERROR: /home/fredrik/code/monolith/services/backend/BUILD.bazel:148:13 ImageLayer services/backend/docker-layer.tar failed: Worker process did not return a WorkResponse:

---8<---8<--- Start of log snippet, file at /home/fredrik/.cache/bazel/_bazel_fredrik/b93f0bbfc3437d73f39541651170604b/bazel-workers/worker-14-TsProject.log ---8<---8<---
[... truncated ...]
9.3 (4125.2) -> 4034.8 (4126.7) MB, 86.5 / 0.0 ms  (average mu = 0.626, current mu = 0.669) allocation failure
[21327:0x5fc7fc0]  2089077 ms: Scavenge 4041.0 (4126.7) -> 4037.0 (4128.7) MB, 69.3 / 0.1 ms  (average mu = 0.626, current mu = 0.669) allocation failure
[21327:0x5fc7fc0]  2093077 ms: Scavenge 4043.4 (4129.5) -> 4039.4 (4147.0) MB, 3729.2 / 0.0 ms  (average mu = 0.626, current mu = 0.669) allocation failure

<--- JS stacktrace --->

FATAL ERROR: Reached heap limit Allocation failed - JavaScript heap out of memory
 1: 0xb09c10 node::Abort() [/home/fredrik/.cache/bazel/_bazel_fredrik/b93f0bbfc3437d73f39541651170604b/execroot/monolith/bazel-out/k8-opt-exec-2B5CBBC6-ST-625e526ca8a8/bin/external/npm_typescript/tsc_worker.sh.runfiles/monolith/../nodejs_linux_amd64/bin/nodejs/bin/node]
 2: 0xa1c193 node::FatalError(char const*, char const*) [/home/fredrik/.cache/bazel/_bazel_fredrik/b93f0bbfc3437d73f39541651170604b/execroot/monolith/bazel-out/k8-opt-exec-2B5CBBC6-ST-625e526ca8a8/bin/external/npm_typescript/tsc_worker.sh.runfiles/monolith/../nodejs_linux_amd64/bin/nodejs/bin/node]
 3: 0xcf8dbe v8::Utils::ReportOOMFailure(v8::internal::Isolate*, char const*, bool) [/home/fredrik/.cache/bazel/_bazel_fredrik/b93f0bbfc3437d73f39541651170604b/execroot/monolith/bazel-out/k8-opt-exec-2B5CBBC6-ST-625e526ca8a8/bin/external/npm_typescript/tsc_worker.sh.runfiles/monolith/../nodejs_linux_amd64/bin/nodejs/bin/node]
 4: 0xcf9137 v8::internal::V8::FatalProcessOutOfMemory(v8::internal::Isolate*, char const*, bool) [/home/fredrik/.cache/bazel/_bazel_fredrik/b93f0bbfc3437d73f39541651170604b/execroot/monolith/bazel-out/k8-opt-exec-2B5CBBC6-ST-625e526ca8a8/bin/external/npm_typescript/tsc_worker.sh.runfiles/monolith/../nodejs_linux_amd64/bin/nodejs/bin/node]
 5: 0xeb09d5  [/home/fredrik/.cache/bazel/_bazel_fredrik/b93f0bbfc3437d73f39541651170604b/execroot/monolith/bazel-out/k8-opt-exec-2B5CBBC6-ST-625e526ca8a8/bin/external/npm_typescript/tsc_worker.sh.runfiles/monolith/../nodejs_linux_amd64/bin/nodejs/bin/node]
 6: 0xec069d v8::internal::Heap::CollectGarbage(v8::internal::AllocationSpace, v8::internal::GarbageCollectionReason, v8::GCCallbackFlags) [/home/fredrik/.cache/bazel/_bazel_fredrik/b93f0bbfc3437d73f39541651170604b/execroot/monolith/bazel-out/k8-opt-exec-2B5CBBC6-ST-625e526ca8a8/bin/external/npm_typescript/tsc_worker.sh.runfiles/monolith/../nodejs_linux_amd64/bin/nodejs/bin/node]
 7: 0xec339e v8::internal::Heap::AllocateRawWithRetryOrFailSlowPath(int, v8::internal::AllocationType, v8::internal::AllocationOrigin, v8::internal::AllocationAlignment) [/home/fredrik/.cache/bazel/_bazel_fredrik/b93f0bbfc3437d73f39541651170604b/execroot/monolith/bazel-out/k8-opt-exec-2B5CBBC6-ST-625e526ca8a8/bin/external/npm_typescript/tsc_worker.sh.runfiles/monolith/../nodejs_linux_amd64/bin/nodejs/bin/node]
 8: 0xe848da v8::internal::Factory::NewFillerObject(int, bool, v8::internal::AllocationType, v8::internal::AllocationOrigin) [/home/fredrik/.cache/bazel/_bazel_fredrik/b93f0bbfc3437d73f39541651170604b/execroot/monolith/bazel-out/k8-opt-exec-2B5CBBC6-ST-625e526ca8a8/bin/external/npm_typescript/tsc_worker.sh.runfiles/monolith/../nodejs_linux_amd64/bin/nodejs/bin/node]
 9: 0x11fd626 v8::internal::Runtime_AllocateInYoungGeneration(int, unsigned long*, v8::internal::Isolate*) [/home/fredrik/.cache/bazel/_bazel_fredrik/b93f0bbfc3437d73f39541651170604b/execroot/monolith/bazel-out/k8-opt-exec-2B5CBBC6-ST-625e526ca8a8/bin/external/npm_typescript/tsc_worker.sh.runfiles/monolith/../nodejs_linux_amd64/bin/nodejs/bin/node]
10: 0x15f2099  [/home/fredrik/.cache/bazel/_bazel_fredrik/b93f0bbfc3437d73f39541651170604b/execroot/monolith/bazel-out/k8-opt-exec-2B5CBBC6-ST-625e526ca8a8/bin/external/npm_typescript/tsc_worker.sh.runfiles/monolith/../nodejs_linux_amd64/bin/nodejs/bin/node]
/home/fredrik/.cache/bazel/_bazel_fredrik/b93f0bbfc3437d73f39541651170604b/execroot/monolith/bazel-out/k8-opt-exec-2B5CBBC6-ST-625e526ca8a8/bin/external/npm_typescript/tsc_worker.sh: line 440: 21327 Aborted                 (core dumped) "$JS_BINARY__NODE_WRAPPER" ${JS_BINARY__NODE_OPTIONS[@]+"${JS_BINARY__NODE_OPTIONS[@]}"} -- "$entry_point" ${ARGS[@]+"${ARGS[@]}"} 0<&0
---8<---8<--- End of log snippet, 104 chars omitted ---8<---8<---
INFO: Elapsed time: 516.078s, Critical Path: 220.24s
INFO: 1326 processes: 373 internal, 134 linux-sandbox, 703 local, 116 worker.
FAILED: Build did NOT complete successfully

Running with NodeJS version: 16.15.0
Not sure what info you want/need, I'll gladly provide what I can. :slightly_smiling_face:

I'm trying to figure out how to pass options to nodejs, as this could probably be solved with --max-old-space-size.

If you disable workers (ts_project(supports_workers = False)) does it work? It seems like the OOM is happening within the worker process...


Details of total batch size

Hi, I find the following experimental setting details in your paper. And the max-token is 3584 in each gpu. How many GPUs you use to train En-De and En-Fr? I want to reproduce the results with the same total batch size over all gpus.

For FR, #warmup steps is 8000, max #epochs is 50, and peak learning rate is 0.0007. For DE, #warmup steps is 4000, max #epochs is 50, and learning rate is 0.001. Max #tokens in each batch is set to 3584 following

Thanks for asking. The total batch size is the product of Max token # and gpu number. For example, we use four gpus on En-De, and 8192 tokens (maximum) on each, which leads to 32768 tokens. The actual tokens per batch is about 3000 tokens. Training recipes are available at https://github.com/LiyuanLucasLiu/Transformer-Clinic/tree/master/nmt-experiments.


Error while using build

Error:
 electron-builder  version=25.1.8 os=10.0.19045
  • loaded configuration  file=X:\Projects\Github\iptvnator\electron-builder.json
  • writing effective config  file=release\builder-effective-config.yaml
  • executing @electron/rebuild  electronVersion=33.2.0 arch=ia32 buildFromSource=false appDir=./
  • installing native dependencies  arch=ia32
  • completed installing native dependencies
  • packaging       platform=win32 arch=ia32 electron=33.2.0 appOutDir=release\win-ia32-unpacked
  • updating asar integrity executable resource  executablePath=release\win-ia32-unpacked\IPTVnator.exe
  ⨯ Application entry file "electron\main.js" in the "X:\Projects\Github\iptvnator\release\win-ia32-unpacked\resources\app.asar" does not exist. Seems like a wrong configuration.  failedTask=build stackTrace=Error: Application entry file "electron\main.js" in the "X:\Projects\Github\iptvnator\release\win-ia32-unpacked\resources\app.asar" does not exist. Seems like a wrong configuration.

To Reproduce
Steps to reproduce the behavior:

add in to package.json line: "electron:build:windows": "npm run build:prod && electron-builder build -w"`
run script
get error

Expected behavior
Expected to see release folder with .exe launcher
Desktop (please complete the following information):

Electron application
OS: Windows 11
Browser Chrome

Additional context
Project documentation not finished. Impossible to build application

I updated the README. Since the project is moving to Tauri, you need to install Rust for building.


DEC-552: Flash messages are improperly cached

Why: Pages that contained flash messages were constantly flashing the message on refresh due to caching.
How: Refactored to use the cacheable_flash gem. This gem stores the flash message in a cookie, and when read, consumes the cookie so that it doesn't get displayed on subsequent reloads.

:+1:

:+1:


Query labels each line to a numerical value

Description of the new feature / enhancement
When a query/search is made for a package name, it generates a list of packages...
Rather than having to type in the whole package name or copy/paste. A number should be assigned to each line.
For example currently winget search:
PS C:\Users\Chay> winget search "windows app"
`
Name                           Id                                 Version Source
Windows App                    9N1F85V9T8BN                       Unknown msstore
Windows App Studio UWP Samples 9NBLGGH4R0W1                       Unknown msstore
LEARN WINDOWS APP EASY         9WZDNCRDC7C7                       Unknown msstore
Windows App Studio Installer   9NBLGGH4QTFX                       Unknown msstore
GoReefers Windows App          9N5ZD6TS2CP7                       Unknown msstore
Windows App Runtime            Microsoft.WindowsAppRuntime.1.4    1.4.3   winget
Windows App Runtime            Microsoft.WindowsAppRuntime.1.3    1.3.3   winget
Windows App SDK                Microsoft.WindowsAppRuntime.1.2    1.2.2   winget
Windows App SDK                Microsoft.WindowsAppRuntime.1.1    1.1.5   winget
Windows App SDK                Microsoft.WindowsAppRuntime.1.0    1.0.4   winget
Windows Application Driver     Microsoft.WindowsApplicationDriver <IP_ADDRESS> winget
Windows App Runtime            Microsoft.WindowsAppRuntime.1.5    1.5.0   winget
PS C:\Users\Chay> winget install microsoft.windowsappruntime.1.5
`
To install windows app runtime....
No	Name                              	Id                               	Version	Source
1	Windows App                       	9N1F85V9T8BN                    	 Unknown	msstore
2	Windows App Studio UWP Samples 9NBLGGH4R0W1                   Unknown	msstore
3	LEARN WINDOWS APP EASY             9WZDNCRDC7C7                 Unknown	msstore
4	Windows App Studio Installer       	9NBLGGH4QTFX                    	Unknown	msstore
5	GoReefers Windows App              	9N5ZD6TS2CP7                    	Unknown	msstore
6	Windows App Runtime                	Microsoft.WindowsAppRuntime.1.4  	1.4.3	winget
7	Windows App Runtime                	Microsoft.WindowsAppRuntime.1.3  	1.3.3	winget
8	Windows App SDK                    	Microsoft.WindowsAppRuntime.1.2  	1.2.2	winget
9	Windows App SDK                    	Microsoft.WindowsAppRuntime.1.1  	1.1.5	winget
10	Windows App SDK                   	Microsoft.WindowsAppRuntime.1.0  	1.0.4	winget
11	Windows Application Driver         	Microsoft.WindowsApplicationDriver	<IP_ADDRESS>	winget
12	Windows App Runtime                	Microsoft.WindowsAppRuntime.1.5  	1.5.0	winget
PS C:\Users\Chay> winget install No12
Proposed technical implementation details
No response

Duplicate of https://github.com/microsoft/winget-cli/issues/419
Related to https://github.com/microsoft/winget-cli/issues/1620
Related to https://github.com/microsoft/winget-cli/issues/301


ARROW-12661: [C++] Add ReaderOptions::skip_rows_after_names

Add a new csv reader option which allows the reader to skip rows after reading the column names from the csv.

Thanks for submitting these PRs!  I'm still brushing up on my parser knowledge but I'll take a look at the other one too.
This one looks pretty straightforward to me.  It's a bit unfortunate that we have two skip_rows but it does seem that there are cases for skipping before and after.  I do prefer this over the before/after boolean.  Another option would be to take in a list of row indices.  This is what pandas does.  So [0:1] skips two rows before the header (if any) and [1:2] would skip two rows after the header or [0, 2] would skip a row before and after.
@pitrou should probably take a look at this as he's got the most experience with the CSV reader.  He's out for the rest of the week so it might be next week.

Thanks for submitting these PRs! I'm still brushing up on my parser knowledge but I'll take a look at the other one too.
This one looks pretty straightforward to me. It's a bit unfortunate that we have two skip_rows but it does seem that there are cases for skipping before and after. I do prefer this over the before/after boolean. Another option would be to take in a list of row indices. This is what pandas does. So [0:1] skips two rows before the header (if any) and [1:2] would skip two rows after the header or [0, 2] would skip a row before and after.
@pitrou should probably take a look at this as he's got the most experience with the CSV reader. He's out for the rest of the week so it might be next week.

I like the idea of replacing the current skip_rows with a list of ranges to skip. This would be a more complex but a more robust solution. I assume that skip_rows would have to be kept but deprecated. The one difficulty would be if ParseOptions::newlines_in_values were true then the reader would have to be aware of the rows that are parsed to have a correct row index. I can start working on this solution ignoring the newlines_in_values issue now if that is the consensus.
I might make it so any range starting with 0 is handled by the reader, as skip_row does now, and any range not starting with 0 is handled by the parser so that potentially quoted new lines are handled correctly.
There might be a way to also tackle https://issues.apache.org/jira/browse/ARROW-8527 with this change.

Ah, I hadn't realized we had newlines_in_values.  Yes, sounds like there is a bit of complexity but you have a good grip on it.

I assume that skip_rows would have to be kept but deprecated.

I don't think so but I'll let @pitrou weigh in.  As long as there is a good reason for it the project seems to be in a place where breaking changes are ok.  Especially since this one would be pretty trivial to understand and adapt to.

There might be a way to also tackle https://issues.apache.org/jira/browse/ARROW-8527 with this change.

That would be good.  Eventually the dataset scanner will probably be getting a skip operation of some kind as well so that'll increase the pressure on ARROW-8527.  ARROW-12598 is also (admittedly tangentially) related since you seem to be on a roll :smile:

That would be good. Eventually the dataset scanner will probably be getting a skip operation of some kind as well so that'll increase the pressure on ARROW-8527. ARROW-12598 is also (admittedly tangentially) related since you seem to be on a roll smile

The only tricky part about a count(*) implementation with this is that skip_rows documented that it was skipping header rows which shouldn't be counted as part of a data row count. I fee like the row count operation would have to be a little different and maybe give an indicator for on which line the actual data rows start so that the header rows before that point could be skipped.
Maybe a simpler solution would a set of two indexes: column names and first data row . While this doesn't allow arbitrary row skipping in the middle this would allow for the most common use cases, including skipping over valid rows to first desired row. With another option or operation could be used to count the number of data rows starting at first data row. The defaults would be 0, 1 for when column names are part of the csv or -1, 0 when they are not.

Valid points.  In that case I wouldn't worry much about ARROW-8527 in this PR at all.  I think 8527 is really looking for "skip data rows" (even though the title says otherwise).  I'll add a comment over there.

Sorry meant to delete my other branch but accidentally deleted this one.

Thanks for the update @n3world ! I will merge this once CI is green.


Sync changes in /en to /zh. Oct 19

Will this pr be merged? I would love to get the ZH translation of this app updated in time. If there is something wrong in this pr, please let me know.


RYA-260 Fluo SPARQL Processing Aggregation Support

Description

What Changed?
Added Aggregation support to the Fluo PCJ application. It is a known problem that the Accumulo PCJ Index does not properly handle aggregation output exporting. That should be covered by a new ticket. Also fixed a bunch of resource leaks and changed up how the Fluo integration tests were implemented so that they use the Fluo recipe for Accumulo exporting integration tests.

Links
Jira
Checklist

[ ] Code Review
[ ] Squash Commits

People To Reivew
@meiercaleb

I've addressed everything.


Release 0.14.0

Close with tag: #557

Codecov Report
All modified lines are covered by tests :white_check_mark:
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Files
Coverage Δ

src/ingest.rs
65.90% <ø> (ø)

src/ingest/tests.rs
91.66% <ø> (ø)

src/peer.rs
47.94% <ø> (ø)

src/publish.rs
52.94% <ø> (ø)

src/publish/tests.rs
98.33% <ø> (ø)

src/storage/migration.rs
43.90% <ø> (ø)

:loudspeaker: Thoughts on this report? Let us know!.


feat(compatible): support for browser environment

Feature

plugin can run inside browser

CHANGES

replace @babel/core with @babel/traverse, @babel/types

Codecov Report

Merging #2 (9d22de1) into master (8a3dbf3) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master       #2   +/-   ##
=======================================
  Coverage   90.24%   90.24%
=======================================
  Files           3        3
  Lines          82       82
  Branches       30       30
=======================================
  Hits           74       74
  Misses          8        8

Impacted Files
Coverage Δ

src/index.ts
90.00% <ø> (ø)

src/transform/useCallback.ts
92.30% <ø> (ø)

src/transform/useState.ts
87.87% <ø> (ø)

Continue to review full report at Codecov.

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


fix: rewind resource for replication

Fixes #44

@tinect  I've tried the first fix with the rewind($resource) and it work's. I know that the  the second fix is necessary because of rewind and seekable, but  you are returning the resource and don't use it or or have I missed something?


The packet module should have the ability like here in trex

checkout this:
https://github.com/cisco-system-traffic-generator/trex-core/tree/master/src/common/Network/Packet
we need some thing like this. now the packet go module is loose and messy. the packet handle ability should separate with different network layer's header.
which will make users like me easy to understand and use it.

First of all, this is Go and not C++. All definitions are there in the structs, like IPv4Hdr with corresponding member functions - it was done historically to keep all the core IP stack processing together. There are some discrepancies for sure, but pure design elegance was sacrificed to the ease of use and that is why you are here. Feel free to submit PR that resolves "loose and messy" design.


Regression after reverting PR that fixed borders offsets

The recent commit 983c1237 reverted the changes that were made to fix a bug in xborders.
Please see this thread for context: https://github.com/deter0/xborder/pull/42#issuecomment-1428785735
Tagging interested party: @ZappaBoy

Must be an oversight, will fix. Apologies.

Nevermind, in the latest source files we are using get client geometry.


🚀 Feature: Upgrade to Astro 3

Bug Report Checklist

[X] I have tried restarting my IDE and the issue persists.
[X] I have pulled the latest main branch of the repository.
[X] I have searched for related issues and found none that matched my issue.

Overview
This is on Astro 2! But Astro 3 is a thing now! https://astro.build/blog/astro-3/
Additional Info
No response

@all-contributors please add @joshuakgoldberg for ideas.

🤖 Beep boop! This comment was added automatically by all-contributors-auto-action.
Not all contributions can be detected from Git & GitHub alone. Please comment any missing contribution types this bot missed.
...and of course, thank you for contributing! 💙


fix: Show cross filter option only when cross filter is enabled

SUMMARY
Fixes https://github.com/apache/superset/issues/16341
@junlincc @jinghua-qa @rusackas
AFTER SCREENSHOTS OR ANIMATED GIF
https://user-images.githubusercontent.com/70410625/130330118-f7ea77c8-01e8-4edf-8715-7ce6002df0a7.mov
TESTING INSTRUCTIONS
Check the original issue for instructions.
ADDITIONAL INFORMATION

[x] Has associated issue:
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

/testenv up


fix: fish format variable

Fix how we set variables in fish script

A little insta test for these could be good

Good point, added it and tested the result in a fish shell locally

Did you somehow forget to push the test?


What is the minimum Node version supported by GOV.UK Frontend v5?

The package.json for govuk-frontend suggests it is v4.2.0:
https://github.com/alphagov/govuk-frontend/blob/72c543a0447c15ea61040cf2fbf901005fe1a0c2/packages/govuk-frontend/package.json#L27
This seems unlikely at the best of times given the wondrous world of dependencies in Node, but my suspicion that this is incorrect was further heightened when I spotted that rollup v4.1.4 is listed as a dependency, and that only supports Node versions greater than 18.
Does the engines.node value in package.json need to be updated, and is v18 (moving to maintenance in the coming days) the correct minimum supported version?

Morning @paulrobertlloyd
This came up recently in https://github.com/alphagov/govuk-frontend/pull/3498/commits/60174e1c98ee62c45903e9b8bfa64aac9d7a610f as we're aware that some shipped features need:

Node.js v12.16.0 for conditional package exports
Node.js v12.17.0 for ES modules as *.mjs
Node.js v12.17.0 for subpath package exports
Node.js v12.20.0 for wildcard package exports

But during code review we discussed it further and decided a Node.js restriction wasn't necessary (yet)
Recommendations
That said, we've shipped package entry points since GOV.UK Frontend v4.1.0 and fixed "subpath export" snags for both Node.js and tools following Node.js package resolution rules—but these are optional features
Node.js v4.2.0 users could still use our Rollup UMD bundles (CommonJS compatible) so we currently recommend:

If you have not already installed version 4.2.0 or later of Node.js, install the latest Long Term Support (LTS) version.

But will soon recommend:

GOV.UK Frontend requires Node.js version 12.17.0 or later to support ECMAScript modules. Where possible, we recommend you install the latest Long Term Support (LTS) version.

Published code
For context, we don't npm publish any package.json dependencies so we're not tied to minimum versions:

Node.js >=18.0.0 for<EMAIL_ADDRESS>Node.js >=6.9.0 for<EMAIL_ADDRESS>Node.js >=14.0.0 for sass-embedded^1.69.2
Node.js ^14 for<EMAIL_ADDRESS>
i.e. Our published code is already compiled/transpiled/optimised so doesn't need these tools anymore

Ah, of course, all things are not what they seem. Thanks so much for the detailed answer, @colinrotherham. Will close this issue.


albert: 0.17.3 -> 0.17.5

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for albert is: "A fast and flexible keyboard launcher"
meta.homepage for albert is: "https://albertlauncher.github.io"
meta.changelog for albert is: "https://github.com/albertlauncher/albert/blob/v0.17.4/CHANGELOG.md"
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.17.4 with grep in /nix/store/7zkzzd89fw834yp836dk0mbbb80ly9y3-albert-0.17.4
found 0.17.4 in filename of file in /nix/store/7zkzzd89fw834yp836dk0mbbb80ly9y3-albert-0.17.4

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
albert

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/7zkzzd89fw834yp836dk0mbbb80ly9y3-albert-0.17.4 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A albert https://github.com/r-ryantm/nixpkgs/archive/c5f6b5a546718d78a1220efedbbbea25afbf58a5.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/7zkzzd89fw834yp836dk0mbbb80ly9y3-albert-0.17.4
ls -la /nix/store/7zkzzd89fw834yp836dk0mbbb80ly9y3-albert-0.17.4/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

albert

Maintainer pings
cc @ericsagnes @Synthetica9 for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for albert is: "A fast and flexible keyboard launcher"
meta.homepage for albert is: "https://albertlauncher.github.io"
meta.changelog for albert is: "https://github.com/albertlauncher/albert/blob/v0.17.5/CHANGELOG.md"
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.17.5 with grep in /nix/store/mkdgja40fyaw2yisfcwnkccsy8g3b4fq-albert-0.17.5
found 0.17.5 in filename of file in /nix/store/mkdgja40fyaw2yisfcwnkccsy8g3b4fq-albert-0.17.5

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
albert

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/mkdgja40fyaw2yisfcwnkccsy8g3b4fq-albert-0.17.5 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A albert https://github.com/r-ryantm/nixpkgs/archive/ef0bff10f3015d03340522ea00ca5975e3a74662.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/mkdgja40fyaw2yisfcwnkccsy8g3b4fq-albert-0.17.5
ls -la /nix/store/mkdgja40fyaw2yisfcwnkccsy8g3b4fq-albert-0.17.5/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

albert

Maintainer pings
cc @ericsagnes @Synthetica9 for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for albert is: "A fast and flexible keyboard launcher"
meta.homepage for albert is: "https://albertlauncher.github.io"
meta.changelog for albert is: "https://github.com/albertlauncher/albert/blob/v0.17.6/CHANGELOG.md"
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.17.6 with grep in /nix/store/4ck6qf0s6012nx4xk6fcmcgdshar7a2r-albert-0.17.6
found 0.17.6 in filename of file in /nix/store/4ck6qf0s6012nx4xk6fcmcgdshar7a2r-albert-0.17.6

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
albert

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/4ck6qf0s6012nx4xk6fcmcgdshar7a2r-albert-0.17.6 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A albert https://github.com/r-ryantm/nixpkgs/archive/2710b3f6d87d25661a503e8fc78a739f456b1b13.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/4ck6qf0s6012nx4xk6fcmcgdshar7a2r-albert-0.17.6
ls -la /nix/store/4ck6qf0s6012nx4xk6fcmcgdshar7a2r-albert-0.17.6/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

albert

Maintainer pings
cc @ericsagnes @Synthetica9 for testing.


Skeleton with child shows "Warning: Each child in a list should have a unique "key" prop."

** Before reporting this bug, please...**

Check the latest documentation: https://docs.nativebase.io/
Check for existing open/closed issues for a possible duplicate before creating a new issue: https://github.com/GeekyAnts/NativeBase/issues
Use the latest NativeBase release: https://github.com/GeekyAnts/NativeBase/releases
Check examples from NativeBase KitchenSink https://github.com/GeekyAnts/NativeBase-KitchenSink
For discussion purpose make use of NativeBase discord: https://discord.gg/TSgCw2UPmb

Describe the bug
Rendering <Skeleton.Text> or just <Skeleton> with a child shows the following warning:
ExceptionsManager.js:180 Warning: Each child in a list should have a unique "key" prop.

Check the render method of `ForwardRef(SkeletonText)`. See https://reactjs.org/link/warning-keys for more information.
    at Skeleton (http://<IP_ADDRESS>:19000/index.bundle?platform=ios&dev=true&hot=false&minify=false:181966:61)
    in SkeletonText
    in RCTView (created by View)
    in View
    in Unknown

To Reproduce
Steps to reproduce the behaviour:
        <Skeleton.Text
          isLoaded={!!audioPost}
          key={`${audioPost?.id}_title`}
          mt='1'
        >
          <Text numberOfLines={2} key={`${audioPost?.id}_title`} mt='1'>
            {audioPost?.title}
          </Text>
        </Skeleton.Text>

Expected behaviour
No warning, keys are set
CodeSandBox/Snack link
It would be easier for us to debug if you can reproduce the error in a codesandbox link or a snack link. You can use the codesandbox template/snack template.
Screenshots
If applicable, add screenshots to help explain your problem.
Platform

NativeBase version [e.g. 3.0.1] 3.3.0-alpha.12
Device: [e.g. iPhone6] iPhone 13 Pro Max
OS: [e.g. iOS8.1] iOS 13
Browser [e.g. stock browser, safari]
Browser version [e.g. 22]
React Native version [e.g. 0.64] 0.64.3
Expo sdk version [e.g. 41] ~43.0.2
node [e.g. 14.16.1] v12.22.7
yarn or npm [e.g. 6.1] yarn v1.22.5

Additional information

Add any other context about the problem here.
Is the bug present in both iOS and Android or in any one of them?
If you want your issue to be looked at quicker, please attach a snack reproducible with your issue. Makes it easier for us!

@garrett-gottlieb Thanks for reporting! This issue has been resolved in version 3.3.0 or later.


Cannot open folder in a OneDrive for Business path

VSCode Version: 1.13.0
OS Version: Windows 10 16215.1000

Steps to Reproduce:

Open VS Code and choose Open Folder
Select a path that is hosted in a One Drive for Business account
Click open folder

Expect:
Folder to open
Actual:
VS Code reports "The path '[One Drive for Business Path]' does not seem to exist anymore on disk."

This is being tracked in #28398
Please add your input to https://github.com/Microsoft/vscode/issues/28398#issuecomment-308151757

@DarrenDaGH are you doing anything different than this:

I was on OneDrive for Business – I did not check to see if regular OneDrive was working…
Darren
From: Benjamin Pasero<EMAIL_ADDRESS>Sent: Tuesday, June 13, 2017 9:25 PM
To: Microsoft/vscode<EMAIL_ADDRESS>Cc: DarrenDaGH<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [Microsoft/vscode] Cannot open folder in a OneDrive for Business path (#28640)
@DarrenDaGHhttps://github.com/darrendagh are you doing anything different than this: [panel-red] https://user-images.githubusercontent.com/900690/27115306-164dbbc8-50ca-11e7-830c-3fa74f542029.gif
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/Microsoft/vscode/issues/28640#issuecomment-308316071, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AcC8wdWuBv7bzR09wXCWmJdklnXRVYDMks5sD2CagaJpZM4N4v3W.


Improve caching mechanism

Closes #176
Testing done
interactive tests + unit tests

Submitter checklist

[X] Make sure you are opening from a topic/feature/bugfix branch (right side) and not your main branch!
[X] Ensure that the pull request title represents the desired changelog entry
[X] Please describe what you did
[X] Link to relevant issues in GitHub or Jira
[ ] Link to relevant pull requests, esp. upstream and downstream changes
[ ] Ensure you have provided tests - that demonstrates feature works or fixes the issue

We can merge it


Knight's Tour Problem

Description
Description
It is one of the popular and hard level problem in Problem solving.
I have learned some amazing techniques to solve this problem and I want to share and contribute in this repository.
Domain
Algorithm
Type of Contribution
Addition
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of LearnCPP.

/assign


Responsive Profile Card View

Description
It is a responsive profile card view with profile pic, name, no. of followers, no. of posts, etc which can be used in the Personal Portfolios and Websites.
Fixes #45
Type of change
Please delete options that are not relevant.

[x] This change requires a documentation update

Checklist:

[x] My code follows the style guidelines(Clean Code) of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests/screenshots(if any) that prove my fix is effective or that my feature works

@kiruba-r11 resolve conflict

@Ayush7614 Resolved the conflicts