Added script for creating ProductMetadata.plist

Hi mkuron,
I work at a print-management company and we have found your code very useful. I couldn't find any license terms, but I hope you are fine with us using it.
I ran into a few problems, when the /Library/Updates/ProductMetadata.plist was empty or non-existing. Therefore, I extended your script a bit to create a new cache. It's tested for printers and works so far without any hiccups. It will most likely also work for other updates.

Do feel free to use my code for your own purposes. I have just added a LICENSE.txt file to the repository to clarify the license.
It shouldn't be necessary to add a ProductMetadata.plist file. That file automatically gets created when SoftwareUpdate.framework scans for updates. I have not seen any situation where PredicateInstaller failed due to that file missing. If you can describe your situation in more detail, I'll see if I can reproduce it. On a related note, I do have code to populate /Library/Printers/InstalledPrinters.plist in the install_printers.py script. That always seemed to suffice.

Thanks for adding such a permissive license, it's much appreciated.
We encountered the problem on a Yosemite test machine with a completely new install. I found the cause to be, that the ProductMetadata.plist file contained just an empty array. Same was encountered once on my MacBook with El Capitan.
I can reproduce it, by moving (or deleting) ProductMetadata.plist, go into the App Store and click the update tab. Wait for it to say "No Updates Available" and confirm the content of the file to be an empty array:
<?xml version="1.0" encoding="UTF-8"?>
<!DOCTYPE plist PUBLIC "-//Apple//DTD PLIST 1.0//EN" "http://www.apple.com/DTDs/PropertyList-1.0.dtd">
<plist version="1.0">
<array/>
</plist>

Now running this as sudo:
/usr/libexec/PlistBuddy -c "Add InstalledPrinters: string MANUFACTURER:HP;MODEL:Officejet Color X555" /Library/Printers/InstalledPrinters.plist
/usr/libexec/PlistBuddy -c "Add Manufacturers: string HP" /Library/Printers/InstalledPrinters.plist
/usr/bin/python predicate_installer.py '"MANUFACTURER:HP;MODEL:Officejet Color X555" in tags'

Will give the output:
Traceback (most recent call last):
  File "predicate_installer.py", line 62, in <module>
    raise Exception("No products found for %s" % sys.argv[1])
Exception: No products found for "MANUFACTURER:HP;MODEL:Officejet Color X555" in tags

Hmm ok, I can reproduce that. However, when I run softwareupdate --list, ProductMetadata.plist ist rebuilt and everything works. softwareupdate --list is a lot simpler than what your pull request is doing. I haven't yet found a way to update the cache using the framework. I can start a scan, but that doesn't appear to suffice:
--- a/predicate_installer.py
+++ b/predicate_installer.py
@@ -43,7 +43,17 @@ if len(sys.argv) < 2:
 predicate = sys.argv[1]
 predicate = NSPredicate.predicateWithFormat_(predicate)

+if int(platform.release().split('.')[0]) >= 15: # OS X 10.11 and higher
+       from objc import YES
+       from AppKit import NSBundle
+       bundle = NSBundle.mainBundle()
+       info = bundle.localizedInfoDictionary() or bundle.infoDictionary()
+       info['NSAppTransportSecurity'] = {'NSAllowsArbitraryLoads': YES}
+
 if int(platform.release().split('.')[0]) >= 13: # OS X 10.9 and higher
+       # update cache
+       scan = SUScan.alloc().init()
+       scan.performSynchronousScan()

        class ControllerDelegate(NSObject):
                def __getattr__(self, name):

I'm pretty sure SUScan is responsible for eventually writing the cache file, so feel free to play around with that scan.performSynchronousScan() line and see if you can find a way to make it do that.

Ah, didn't know about that side effect. I'm also much more comfortable with softwareupdate --list than my Python script.
I changed my code to this:
--- a/predicate_installer.py
+++ b/predicate_installer.py
@@ -9,6 +9,9 @@ import platform
 from threading import Thread
 from PyObjCTools import AppHelper

+if (not os.path.isfile(metaFile) or os.path.getsize(metaFile) < 2**20): #1KB
+       subprocess.check_call(['/usr/sbin/softwareupdate', "--list"])
+
 finishedInstall = False

 def installlog():

I'm going to test it to see if everything works. If it does, then I think I'll go with that.

Oops, that's a typo. I meant to write 2**10. If I remember correctly, an empty one could be up to around 300 bytes.

I decided to squash your pull request (2b1700c) because none of the individual commits would have actually worked. It's merged now (b1209f3). Don't forget to
git fetch --all
git reset --hard b1209f3


[Event Request] codeunit 951 "Time Sheet Approval Management" - OnAfterApproveBeforeModify

Could we please have an event OnAfterApproveBeforeModify in Codeunit 951 "Time Sheet Approval Management" in function Approve?
Code Example:
procedure Approve(var TimeSheetLine: Record "Time Sheet Line")
begin
with TimeSheetLine do begin
if Status = Status::Approved then
exit;
TestField(Status, Status::Submitted);
CheckApproverPermissions(TimeSheetLine);
Status := Status::Approved;
"Approved By" := UserId;
"Approval Date" := Today;
// start kvspsa - Event Raiser
OnAfterApproveBeforeModify(TimeSheetLine);
// stop kvspsa
Modify(true);
case Type of
Type::Absence:
PostAbsence(TimeSheetLine);
Type::Service:
AfterApproveServiceOrderTmeSheetEntries(TimeSheetLine);
end;
OnAfterApprove(TimeSheetLine);
end;
end;
Event:
[IntegrationEvent(false, false)]
local procedure OnAfterApproveBeforeModify(var TimeSheetLine: Record "Time Sheet Line")
begin
end;
Thank you and kind regards

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


[Docs] Create/Update the Database page in spanish, that goes in the CONCEPT section in the spanish docs site

Current State
The spanish docs site doesn't have the Database page translation to spanish
English page to translate: https://docs.meshery.io/concepts/architecture/database
Desired State
The spanish docs site has the Database page translated to spanish in the CONCEPT section
Contributor Guides and Resources

🛠 Meshery Build & Release Strategy
📚 Instructions for contributing to documentation

Meshery documentation site and source

🎨 Wireframes and designs for Meshery UI in Figma
🙋🏾🙋🏼 Questions: Layer5 Discussion Forum and Layer5 Community Slack

Tomaré este issue.


Add 100Starlings

Please review our Contribution Guidelines to ensure your changes abides by it.
Changes should be made to the end of the file.
Review your changes and complete the checklist below.

[x] Company name
[x] Industry
[x] Company location
[x] Company description
[x] How Elixir is used
[x] GitHub / blog links
[x] ~Job listings~

Thank you @lackac!


Embed Angular component tag inside Markdown file

I have recently discovered the Angular2 Markdown project and I have found it extremely valuable for a project that I am currently building in Angular. It is such a great tool.
I would like to ask about a procedure that seems not to be possible, but that may be useful to discuss.
The question is whether it is supported to insert some Angular component tag inside a Markdown file which is to be read afterwards. Ideally it would be something like this:
some-template.html
------------------

...
<markdown path="a-markdown-file.md"></markdown>
...

and then
a-markdown-file.md
------------------

...
<app-some-component></app-some-component>
...

In this direct way, it seems not to work (which is sensible, because this is not proper Markdown syntax). Would there be some roundabout when compiling? I have tried using the renderer to convert some protected character into the tags, but the result does not work.
In case it is useful, let me explain the context in which this question arises. I am building an application with lots of posts, so the combination Angular + Markdown is perfect: the posts are written in independent .md files, and Angular takes care of arranging everything. But the posts themselves usually have some repetitive chunks of code (to insert graphics, interactive content and others) and it would be great to replace this repetitive content with the aid of an Angular component.
Thank you very much for your attention; any suggestion or idea is welcome

@jxm-math I'm sorry for the delay. I have tried different ways but this is not possible to implement right now. We are using marked library to convert markdown to HTML. It doesn't recognize custom element.
I'm closing this issue for now. Feel free to open it again. If you find any solution.

@dimpu Thank you very much for your attention and efforts; it is sincerely very kind of you

Actually i think you can do it
i have a project for a Wiki  and in that i have links in markdown that need to be router links for navigation.
to make that work i call the markdown compile function and then pass that html to a directive that calls the angular compiler to convert the markup to where angular will use it.
see this Question about dynamic elements
for some info and a link to a plunker that has the directive code.


Make Prod Look Like Dev™

Followup to #144.
We still haven't gotten around to this. I'm putting it on my radar for next quarter by creating a separately referencable bug for it.

I've been talking with @marumari about ways to move forward on this and improve things.


Super+v for clipboard history often stops since NixOS 24.11

https://github.com/kachick/dotfiles/blob/94abc320d081510290fa01509df546b2158610f3/home-manager/gnome.nix#L160-L165
70%+ broken since https://github.com/kachick/dotfiles/pull/943
... :thinking:

The extension actually registered with 'Shift+v' ... And cannot register Super.

I cannot notice keybind overriding by GNOME extensions...


new address

Many didn't update the new address, please update the address！！

For example?

d3js,echarts,spritejs, etc.
Possible domestic no url, but should pay attention to。
| |
m17771449436
|
|
@.***
|
---- Replied Message ----
| From | @.> |
| Date | 02/24/2023 13:37 |
| To | @.> |
| Cc | @.> ,
@.> |
| Subject | Re: [docschina/docschina-issue-discussions-open] new address (Issue #50) |
For example?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


NaN or Inf found in input tensor.

I encounter this problem when i train.
NaN or Inf found in input tensor.
100%|█████████████████████████████████████████████████████████████████████████████████████████████████████████████████| 861/861 [06:36<00:00,  2.17it/s]
NaN or Inf found in input tensor.
NaN or Inf found in input tensor.
[epoch 8 PSNR: nan --- best_epoch 3 Best_PSNR 23.9811]
[epoch 8 SSIM: nan --- best_epoch 5 Best_SSIM 0.8553]

The above mentioned problem you faced, which is gradient explosion.
Because the value of loss is represented as the gradient, which is calculated by the differential.
This is standard phenomenon of gradient explosion.
Usually happen on the training data is insufficient or the architecture of training model is too complex,
the gradient vanish is also caused by the same reason but the loss value will be displayed as zero.
There are some common solutions to solve the gradient problem:

Decrease the initial learning rate to slow down the convergence time.
Decrease the parameters of model. (Like the initial feature channels from 96 to 64)
Increase the training data.

In your case, I noticed that your training data has only 861 (raindrop) images, it's not enough.
Please see Train part, subsection Details of Hyperparameters in README.md and preprocess in dataset README.md to increase your training data.


Either export for Node/CommonJS or RequireJS, not both

At the moment the following happens:
/* export Chess object if using node or any other CommonJS compatible
 * environment */
if (typeof exports !== 'undefined') exports.Chess = Chess;
/* export Chess object for any RequireJS compatible environment */
if (typeof define !== 'undefined') define( function () { return Chess;  });

When using only Node this works fine, because the second statement is not executed. When using Webpack, however, both statements are executed (because it supports both if I'm correct), which means that the first statement is overwritten (at least it seems like that) and, hence, is therefore not longer compatible with code that is written for Node. And the code works without Webpack.
Note that I don't have this problem when using chess.js as a direct dependency, only when chess.js is an indirect dependency.
Therefore, I would suggest to turn these two separate ifs into in an if-else.

Also, should we change:
if (typeof exports !== 'undefined') exports.Chess = Chess;
to:
if (typeof module !== 'undefined') module.exports.Chess = Chess;?

@ngocdaothanh module.exports is unique to Node, so I think that would break with other CommonJS loaders.

I need this enhancement. I’m using Webpack to bundle a React app, but my tests run in Node.
const { Chess } = require('chess.js');

works in my tests, but not in my app.
The real issue is that CommonJS loaders return an object with the Chess constructor as a property, while AMD loaders return the Chess constructor directly. I was going to open a pull request to simply add the else as @pheyvaer suggested, but I think the real fix is to return the same thing for both CommonJS and AMD, e.g.
/* export Chess object if using node or any other CommonJS compatible
 * environment */
if (typeof exports !== 'undefined') {
    exports.default = Chess;
}
/* export Chess object for any RequireJS compatible environment */
else if (typeof define !== 'undefined') {
    define( function () {
        return Chess;
    });
}

That would be a breaking change to the interface, though, requiring CommonJS users to use a default import instead of a named import.
Of course, if there is any code out there where the tests use RequireJS but the code is bundled with Webpack, or maybe code that uses RequireJS in Node, I think adding the else would break it.

My current (ugly) workaround:
const { isNode } = require('browser-or-node');

let Chess;

if (isNode) {
    const chess = require('chess.js');
    Chess = chess.Chess;
}
else {
    Chess = require('chess.js');
}

It turns out there's an easy way to fix this with Webpack. Simply disable the AMD parser for chess.js in your Webpack config:
module: {
    rules: [
    ...
        {
            test: require.resolve('chess.js'),
            parser: {
                amd: false
            }
        }
    ]
}

Now you can run
const { Chess } = require('chess.js');

in both your Node code and your Webpack-bundled code.

Fixed in 0.13.0


"About:" keyword "does something"

Hi all,
I was trying to set a section title with "something(space)about(possible space):", but something weird happened. The title was moved to the right, and I don't know why.
About word can be upper or lowercase, but when you put a space before and a colon after it "pops away".
Is it a feature that I don't know or a bug?

I think this comes from how classes are dynamically added to the element for syntax highlighting! When you add a title, it gets its own class so that it can be customized in the stylesheet and it seems that having a space isn't accounted for in the transform. This happens here: https://github.com/irrationalistic/atom-tasks/blob/master/lib/tasks.cson#L12
This uses downcase to set everything to lowercase, so i'm trying to see if there's a way to also override spaces so that we don't get this weird behavior.

Hi @irrationalistic ,
The weird thing is that this occurs only when you write About (upper or lowercase) after the space.
If you write
something(space)somethingelse:
everything works as expected

@massic80: it's mostly just a weird coincidence that there is a special style for anything with the "about" class! You can try it with test cursor: or test indent-guide: or any of the classes available in that context. Essentially, to help with custom styling for groupings or headings, the text of the heading is added to the resulting syntax as a class (which you can target in your custom stylesheet). This ends up having conflicts since a space is actually a delimiter in class names, so if you use a single word as header, it ends up something like class="syntax--source syntax--myheader, but if you put a space in the header, it'd end up like class="syntax--source syntax--my header, which is actually three classes!

So it looks like the only transforms supported are downcase and upcase in the first-mate library: https://github.com/atom/first-mate/blob/254d43b7a941a7fe63a4ad85a7766108561fec7c/src/pattern.coffee#L3
This means that fixing this particular issue could take a few different paths:

Do nothing and just try to avoid using "about" as the second word
Remove the custom classname functionality from headers
Submit PR to first-mate to add more support for textmate transforms (https://macromates.com/textmate/manual/references#format-string)

I'm inclined to go with 2 on this one, but i'm not sure of the usage of custom styles that might break. I think i just added the functionality when building this, so i don't think it was a requested feature and folks may not even know about it. What are your thoughts?

Hi! I also found out that "console" keyword pushed the ":" to a new line


[nfc] fix : CheckpointIndexFile is not defined

📌 Checklist before creating the PR

[ ] I have created an issue for this PR for traceability
[x] The title follows the standard format: [doc/gemini/tensor/...]: A concise description
[x] I have added relevant tags if possible for us to better distinguish different PRs

🚨 Issue number
Not involved
📝 What does this PR do?
fix "CheckpointIndexFile" is not defined"
modified:   colossalai/checkpoint_io/utils.py  line 371
Before adjustment

💥 Checklist before requesting a review

[ ] I have linked my PR to an issue (instruction)
[ ] My issue clearly describes the problem/feature/proposal, with diagrams/charts/table/code if possible
[x] I have performed a self-review of my code
[x] I have added thorough tests.
[x] I have added docstrings for all the functions/methods I implemented

⭐ Do you enjoy contributing to Colossal-AI?

[x] 🌝 Yes, I do.
[ ] 🌚 No, I don't.

Tell us more if you don't enjoy contributing to Colossal-AI.

@Fridge003  @FrankLeeeee
in #4142 Modified were deleted
In the latest version there will still be this prompt
""CheckpointIndexFile" is not defined"
am i wrong ?

There will be a circular importing error if adding from .index_file import CheckpointIndexFile.


Wait for notification

Needs review from RnD.
Targeted for 10-beta-2.

Hi @ishan1729, I've reviewed the language, style, and the screenshots of the PR. I also left one comment above. My changes are in this commit.
Please verify if there is anything that is not correct from a technical point of view. Thanks!

I've just approved the PR and I will make sure to merge the PR when 10.0.0 Beta 2 is released.


Better repo description

You might want to change the current repo description for better SEO and searchability. Something along the lines of: "A mobile interface for SCELE Fasilkom UI" perhaps?
Anyway, congrats for shipping! 🎉

Done. Thank you!


PreviewModelDialog Columns & Rows

If we change:
#define MAX_DIALOG_PREVIEW_ROWS 2
#define MAX_DIALOG_PREVIEW_COLUMNS 6

To
#define MAX_DIALOG_PREVIEW_ROWS 4
#define MAX_DIALOG_PREVIEW_COLUMNS 8

Yea, you are suppose to change them!

Yea, you are suppose to change them!

but we are not supposed to get the shown results

but we are not supposed to get the shown results


KeyVault: missing private key sample

Remark section says:
This property contains only the public key. If you must retrieve the key pair including the private key instead of performing cryptographic operations in Azure Key Vault, see the sample:
But, there is no sample...

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

ID: a2c7ead4-7dcc-3149-ca5e-7543eaca58ff
Version Independent ID: b5c97c78-2cbc-d103-4d63-b76e433c8625
Content: KeyVaultCertificate.Cer Property (Azure.Security.KeyVault.Certificates) - Azure for .NET Developers
Content Source: xml/Azure.Security.KeyVault.Certificates/KeyVaultCertificate.xml
Service: azure
GitHub Login: @CamSoper
Microsoft Alias: casoper

@danieljurek: This appears to be a tooling problem when transforming the doc comments.    The page renders without a samples link:

The source, however, links to the sample in a valid <see> element:
        /// <summary>
        /// Gets the CER formatted public X509 certificate.
        /// </summary>
        /// <remarks>
        /// This property contains only the public key.
        /// If you must retrieve the key pair including the private key instead of performing cryptographic operations in Azure Key Vault, see the sample:
        /// <see href="https://docs.microsoft.com/samples/azure/azure-sdk-for-net/get-certificate-private-key"/>
        /// </remarks>
        public byte[] Cer { get; internal set; }

Thank you for your feedback.  Tagging and routing to the team member best able to assist.

There were changes made to the render and this is fixed:


New-lines in options cause formatting issues with title attribute (if multiple options selected).

New-lines in options cause formatting issues.
Windows 10
IE 11 / Edge
VS 2017
Bootstrap 3.3.7
Bootstrap-select 1.12.4
https://plnkr.co/edit/nBiCB4xZAP8Be4wzpj9A?p=preview

You might say "don't put new lines in", but sometimes this happens during VS/Resharper auto formatting.

I can't seem to recreate this in your linked example in Chrome, Edge, or IE11 on Windows 10. It almost looks like CSS or some JS isn't being loaded (as it doesn't make sense that none of the other options would be visible; even the "Close" button is missing).

You need to select Apple and Banana.  This is on IE Edge although it's the same on any browser.

Ah - thanks for the clarification. Should be an easy fix.

Yes, I think something like this may do it:
  if (typeof $this.attr('title') !== 'undefined') {
    return $this.attr('title');
  } else if ($this.data('content') && that.options.showContent) {
    return $this.data('content').toString();
  } else {
    return icon + $this.html()**.trim()** + subtext;
  }
}

}).toArray();

Released in v1.13.0!

/move to bootstrap-select


feat: Dev mode with sections + reordered

Hi everyone,
The dev mode page has a lot of text and it isn't easy to find things.
That's why, I've reordered everything + added some sections:

Codecov Report
Attention: Patch coverage is 0% with 93 lines in your changes missing coverage. Please review.

Project coverage is 7.43%. Comparing base (4d9c7fc) to head (128f52e).
Report is 185 commits behind head on develop.

Files
Patch %
Lines

...b/pages/preferences/user_preferences_dev_mode.dart
0.00%
67 Missing :warning:

...ib/pages/preferences/user_preferences_widgets.dart
0.00%
26 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##           develop   #5372      +/-   ##
==========================================
- Coverage     9.54%   7.43%   -2.12%
==========================================
  Files          325     380      +55
  Lines        16411   19485    +3074
==========================================
- Hits          1567    1448     -119
- Misses       14844   18037    +3193

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Update review-database to 0.12.0-alpha.1

Codecov Report
Patch coverage: 75.00% and project coverage change: +0.04 :tada:

Comparison is base (439067f) 55.14% compared to head (501911a) 55.18%.

:exclamation: Current head 501911a differs from pull request most recent head 844920d. Consider uploading reports for the commit 844920d to get more accurate results

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #25      +/-   ##
==========================================
+ Coverage   55.14%   55.18%   +0.04%
==========================================
  Files          45       45
  Lines        8285     8277       -8
==========================================
- Hits         4569     4568       -1
+ Misses       3716     3709       -7

Impacted Files
Coverage Δ

src/graphql/data_source.rs
52.91% <75.00%> (+1.39%)
:arrow_up:

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Double sided material

The double sided material does not seem to work properly with the ray tracer. It might be due to the use of power() method.

Need to update reset!() so that it modifies both powers and not only one. We can create a method for it.


tag Mondo terms with Clingen Preferred, where applicable

see spreadsheet: https://docs.google.com/spreadsheets/d/13eIa2Ihe6ChwOjH0s-8hp0K9IGMtveIqpFTtVvuvWK4/edit#gid=1532441036
Nicole to do:

[x] think of other ways to query terms that should have the clingen preferred label and let Nico know

cc @matentzn

query for https://clinicalgenome.org/affiliation/ in source annotations (there may be some duplicates with xrefs)
could we cross-check tickets in GitHub that has clingen tag and make sure the term that was requested has a clingen preferred label (probably has to be done manually)

draft ROBOT template is here


Column Limit when Uploading a dataset?

Trying to upload a CSV with over 300 columns and I'm getting an unspecified error. When I trim the file down to 200 columns it uploads fine into the platform. Is there some sort of column limit? How do I increase this? This same file was being uploaded in the previous version of cartodb.

Last week I increased the Mb and rows for imports in https://github.com/sverhoeven/docker-cartodb/commit/af47cc8cce4c5ba2c32305e46b959747f68b7359
I did not see any limit on number of columns.
Can you have a look in job queue log file  /cartodb/resque.log to get a better error message?

I don't see anything in the resque.log. I just tried uploading the same file onto cartodb hosted version and I get the same error.

function() { if ( list ) { // First, we save the current length var start = list.length; (function add( args ) { jQuery.each( args, function( _, arg ) { var type = jQuery.type( arg ); if ( type === "function" ) { if ( !options.unique || !self.has( arg ) ) { list.push( arg ); } } else if ( arg && arg.length && type !== "string" ) { // Inspect recursively add( arg ); } }); })( arguments ); // Do we need to add the callbacks to the // current firing batch? if ( firing ) { firingLength = list.length; // With memory, if we're not firing then // we should call right away } else if ( memory ) { firingStart = start; fire( memory ); } } return this; }


validation: More updates to api/ui form validation

Signed-off-by: Ryan Northey<EMAIL_ADDRESS>

Codecov Report

Merging #60 (3736bb3) into main (3a7a15b) will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##             main      #60   +/-   ##
=======================================
  Coverage   23.64%   23.64%
=======================================
  Files          38       38
  Lines         956      956
=======================================
  Hits          226      226
  Misses        730      730

Impacted Files
Coverage Δ

ui/src/network/forms.js
10.84% <0.00%> (ø)

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 3a7a15b...3736bb3. Read the comment docs.


Fixed scratch buffer extern declaration for CEVA-DSP platforms

Changed extern declaration in header from pointer to array
BUG=cleanup

@yair-ehrenwald: note that you should be able to apply the ci: run label to run the CI.


Error if two rows of the same height are generated

Description
A error should be generated if two different sites of the same height are selected for row generation.  This will lead to errors during placement and is not supported.
Suggested Solution
Add a check to initialize_floorplan.
Additional Context
No response

@mina1460 please take care of this when you have a moment


Accurate ML model for price prediction

Improvise sample models. Refer to https://github.com/priyanshux/cryptopy/blob/main/CryptoPy/Sample Models.ipynb

@priyanshux I want to work on this issue.

Hi @Gaurav7888 , thanks for your interest. Please join our Slack workspace to discuss more

Hi I want to work on this issue please assign this issue to me

@yashvijain0710 sure, you may start working on this

note: multiple students can work on same issues

@priyanshux  how to colab our work then ?

I'd like to work on this issue


Linted unused imports on 2.12.2

-Xlint tries to be extra helpful in 2.12.2, noted in community build.

Thanks! 👍


Docker installation issues, composer

Followed the documentation and was met with the following error
root@48cc4e20f20e:/application# sh docker-first-run.sh
Installing dependencies from lock file (including require-dev)
Verifying lock file contents can be installed on current platform.
Your lock file does not contain a compatible set of packages. Please run composer update.

  Problem 1
    - php is present at version 7.4.33 and cannot be modified by Composer
    - symfony/orm-pack is locked to version v2.4.0 and an update of this package was not requested.
    - symfony/orm-pack v2.4.0 conflicts with php <8.1.

> php -r "include_once 'installer//AutoInstaller.php'; use Installer\AutoInstaller; AutoInstaller::runDocker();" < /dev/tty

Fatal error: Uncaught Exception: Composer autoload file was not found, did You called `composer install` earlier? in /application/installer/AutoInstaller.php:12
Stack trace:
#0 Command line code(1): include_once()
#1 {main}
  thrown in /application/installer/AutoInstaller.php on line 12
Script php -r "include_once 'installer//AutoInstaller.php'; use Installer\AutoInstaller; AutoInstaller::runDocker();" < /dev/tty handling the pms-installer-docker event returned with error code 255

Tried running composer update and got
root@48cc4e20f20e:/application# composer update
Loading composer repositories with package information
Updating dependencies
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - Root composer.json requires specshaper/encrypt-bundle dev-master -> satisfiable by specshaper/encrypt-bundle[dev-master].
    - doctrine/orm[2.11.0, ..., 2.20.0] require doctrine/common ^3.0.3 -> found doctrine/common[3.0.3, ..., 3.4.5] but it conflicts with your root composer.json require (^2).
    - specshaper/encrypt-bundle dev-master requires doctrine/orm ^2.11 -> satisfiable by doctrine/orm[2.11.0, ..., 2.20.0].

Use the option --with-all-dependencies (-W) to allow upgrades, downgrades and removals for packages currently locked to specific versions.

This should not happen, but You could edit the docker-first-run.sh and set the first line to:
composer install --ignore-platform-reqs

Which is not the real solution but might just work for You

Well it did happen and I had to fix a load of the dependencies as one one them was set as 'dev-master'. I got it working in the end.

I will close it then. V 2.0 has simplier self-hosting logic. So it's no issue in the wip branch.


Add Support for Photo posts

Preparing to import my Instagram posts and having a photo post type would be nice.
Currently I'm just embedding photos into an 'e-content' and calling it a day, however 'u-photo' should exist outside of 'e-content' in much the same way a checkin does.

I've been posting photos myself (with a delay) manually after posting them to Instagram: https://rusingh.com/category/post/photo/
I'd love to see this happen!

Here's a sample layout for one or more photos with a single caption for all photos:
https://github.com/MaybeThisIsRu/smix-eleventy-starter/blob/main/src/layouts/post/photo.liquid

And if you wanted to have a caption for each photo, the IndieWeb wiki recommends a collection post type.

Thanks for the links Ru! I'm thinking about how I want to add Photo (and maybe video?) posts. I think I like the simplicity of a single caption for all photos and it better reflects how I'd blog media.
i.e. If I wanted to share a bunch of photos with a different caption, I'd probably write an article and intersperse pictures as I saw fit, rather than thinking of it as "media" post.


gnomeExtensions.ddterm: unbreak

Description of changes
fixes https://hydra.nixos.org/build/241558776
ZHF https://github.com/NixOS/nixpkgs/issues/265948

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Priorities
Add a :+1: reaction to pull requests you find important.

Given that https://github.com/ddterm/gnome-shell-extension-ddterm/commit/0a263a15a6ac82274ebf3e4f6b98e7b226e8188a is the commit that broke it, are you sure this is the best way to fix this?
I am just tempted to simply apply a revert of that commit as a patch for now, and then maybe communicate with upstream for a better solution. CC @jtojnar
(Btw apologies for the breakage, I didn't properly test the last update before the merge, at least two more extensions need fixups too)

I don't know enough js nor gnome-shell to understand the issue. From what i gather i assume they want to run the script from js with the proper runtime instead of relying on its shebang?
Then a revert is better :+1:

Yeah, this will not work. The wrapper turns the JS file into an ELF program but ddterm now calls a JS interpreter with the supposed JS file as an argument. That will fail because JS interpreter will not be able to run an ELF program.
Revert and talking to upstream is probably best option. Example from another extension: https://github.com/GSConnect/gnome-shell-extension-gsconnect/commit/71c86dbb1e3434859331de56bbf0bb38560c8baa

For reference see:
https://github.com/ddterm/gnome-shell-extension-ddterm/issues/597
https://github.com/NixOS/nixpkgs/issues/259035

The revert would look something like this:

index e195ebc9b8e3..ec7148a6763f 100644
--- a/pkgs/desktops/gnome/extensions/extensionOverrides.nix
+++ b/pkgs/desktops/gnome/extensions/extensionOverrides.nix
@@ -1,6 +1,7 @@
 { lib
 , ddcutil
 , easyeffects
+, fetchpatch
 , gjs
 , glib
 , gnome
@@ -49,9 +50,20 @@ super: lib.trivial.pipe super [
   (patchExtension<EMAIL_ADDRESS>(old: {
     nativeBuildInputs = [ gobject-introspection wrapGAppsHook ];
     buildInputs = [ vte ];
+    patches = [
+      (fetchpatch {
+        name = "use-same-gjs-interpreter-when-launching-script.patch";
+        url = "https://github.com/ddterm/gnome-shell-extension-ddterm/commit/0a263a15a6ac82274ebf3e4f6b98e7b226e8188a.diff";
+        hash = "sha256-pU1PbAPMRUvalLn+FxzLEW6EuJBi8hQFeV8iBymjRZc=";
+        revert = true;
+      })
+    ];
     postFixup = ''
+      # fetchpatch strips file permission changes
+      # https://github.com/NixOS/nixpkgs/issues/266556
+      wrapGApp_() { chmod +x "$1" && wrapGApp "$1"; }
       wrapGApp<EMAIL_ADDRESS>-      wrapGApp<EMAIL_ADDRESS>+      wrapGApp_<EMAIL_ADDRESS>     '';
   }))

while a store-patch is much more succinct, but does not guarantee that the gappsWrapperArgs[@] makeWrapper arguments take effect.

--- a/pkgs/desktops/gnome/extensions/extensionOverrides.nix
+++ b/pkgs/desktops/gnome/extensions/extensionOverrides.nix
@@ -51,7 +51,10 @@ super: lib.trivial.pipe super [
     buildInputs = [ vte ];
     postFixup = ''
       wrapGApp<EMAIL_ADDRESS>-      wrapGApp<EMAIL_ADDRESS>+      substituteInPlace<EMAIL_ADDRESS>\
+        --replace \
+          'function get_interpreter() {' \
+          'function get_interpreter() { return "${lib.getExe gjs}";'
     '';
   }))

This line change may indicate that the wrapper is not needed?

Wrapping will be almost certainly needed if the script uses any typelibs or settings schemas not used by GNOME Shell. If only glib typelibs are used, those should be available in gjs even without any extra environment variables. If the spawned process does not inherit environment variables from GNOME Shell, wrapper will be needed for any settings schemas.
But since it looks like the upstream patch was introduced because of NixOS in the first place, we might want to revert it upstream.

According to upstream, simply removing the wrapGApp<EMAIL_ADDRESS>should be sufficient, as it is dead code anyways on NixOS given that all dependencies are properly installed.


users created with --system option cannot be logged in

Fix attempt in: https://github.com/raspiblitz/raspiblitz/pull/4172

#4172 merged


Localize to Virginia Beach

Change references from Norfolk to Virginia Beach and replace the WardFinder

Remove Ward Finder
Change Text reference from Norfolk to Virginia BEach

This has been fixed in staging: http://okcandidate-staging.code4hr.org/

Fixed in production.


[Spanish] localize 'Stateless Apps'

Localize the content/en/stateless-apps.md document to Spanish

Hey @electrocucaracha could you assign me this issue? I'd like to work on it when possible.

closed by https://github.com/cncf/glossary/pull/2015


Add save html

What kind of change dose this PR introduce?
Adding the '--save-html' flag.
What is this used for?
Users can now save ascii-art as a html file. To further copy text with colored data.
Why is it important?
Colored ascii-art can not be copied from the terminal including the colored data. Furthermore txt files can not save color. By introducing the html save useres can open the generated html file and copy the ascii-art including its text color.

This looks great. I'll test and review this in detail soon. I appreciate you actually making the change instead of opening an issue :)

@TheZoraiz Friendly bump here. This would be very useful

Apologies for the delay. Some personal stuff has been getting in the way lately. I'll get on this as soon as I can


config: Wrong default OAI Repository metadata formats

I think the default config CFG_OAI_METADATA_FORMATS in invenio/base/config.py
CFG_OAI_METADATA_FORMATS = {
    'oai_dc': ('XOAIMARC', 'http://www.openarchives.org/OAI/1.1/dc.xsd',
                           'http://purl.org/dc/elements/1.1/'),
    'marcxml': ('XOAIDC',
                'http://www.loc.gov/standards/marcxml/schema/MARC21slim.xsd',
                'http://www.loc.gov/MARC21/slim'),
}

Aren't the keys of the dictionary wrong?

Hi @jgarcial, Is there a recommended standard/convention for these names? If I check the OAI-PMH standard I can only find in the examples oai_dc, e.g. see:
http://www.openarchives.org/OAI/openarchivesprotocol.html#ListMetadataFormats
Additionally, the oai_marc mentioned in the example is different MARC serialization than our exposed marcxml: see http://www.openarchives.org/OAI/2.0/guidelines-oai_marc.htm

@kaplun It's simpler than that. The mapping is wrong.
'oai_dc'  -> 'XOAIMARC'
'marcxml'-> 'XOAIDC'

:tada: ahah. I haven't noticed that. It's something introduced when porting the moduele to the new master, as the configuration is correct in legacy.
Can you send a PR with the correction on top of maint-2.0?
Cheers!
Sam


onValueChange not called when slider tapped iOS

Description
The onValueChange callback is not called when the user taps the slider in iOS (i.e. they don't slide the slider)
The current workaround is to use onSlidingComplete

here i have the same problem any solutions?

Would also love to see any workarounds or fixes available.


Typespec python generator doesn’t consider namespaces

We would like to be able to generate python Api code that takes into account the namespaces. This is useful for parameters and function name simplification in a global API context.
We use TypeSpec compiler v0.53.1
Sample description:

We want to define an API that contains two different classes: classA1 and class A2 each class defined in their own namespace.
Each class defines two operations (nameOperation1 and nameOperation2) with the same name but with different route path (see apis.tsp files)
Each operation is a customOperationCollection with an operation request and response, having the same name (see api-models.tsp)

--> Typespec-ts manage to generate the client.
--> Typespec-python generates the following error:
File<EMAIL_ADDRESS>line 467, in
).process()
^^^^^^^^^
File<EMAIL_ADDRESS>line 349, in process
code_model = CodeModel(yaml_data=yaml_data, options=options)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File<EMAIL_ADDRESS>line 73, in init
self.sort_model_types()
File<EMAIL_ADDRESS>line 255, in sort_model_types
self._sort_model_types_helper(
File<EMAIL_ADDRESS>line 224, in _sort_model_types_helper
raise ValueError(
ValueError: We have already generated a schema with name Operation1Request
We would like to be able to have code generated for every operation – this can be done by creation a _models.py per namespace or by adding the namespace into the method name.

@cdrlseg we are able to generate operations in different namespaces, but the design of the python SDK is to have all of the models defined in the models subnamespace. What seems to be happening here is you have an anonymous model (a model defined inline in an operation) for both of your operations in each namesapce. I would recommend just defining one model in this case that both operations will use.
I believe this would also be better typespec definition code so you don't generate duplicate models when not needed

Hi @iscai-msft . I don't know if you opened the attached sample. In our case we would like to be able to define two models that have the same name but not the same signature, each model is in a separate namespace.
Here we have operation1Request // operation2Request as well as operation1Response//operation2Response with same name but in different namespaces (classA1 and classA2). Or the generator do not accept this behavior as the namespace is not taken into account. As the generated methods have the same name the function sort_model_types() returns an error.
Below the code extraction:
namespace classA1{
using TypeSpec.Http;
using TypeSpec.JsonSchema;
model operation1Request {
parameters1?: string;
parameters2?: string;
};
model operation2Request {
parameters1?: string;
parameters2?: string;
};
model operation1Response {
parameters1?: string;
parameters2?: string;
parameters3?: string;
};
model operation2Response {
parameters1?: string;
parameters2?: string;
parameters3?: string;
};
}
namespace classA2{
using TypeSpec.Http;
using TypeSpec.JsonSchema;
model operation1Request {
parameters1?: string;
parameters2?: string;
parameters3?: string;
parameters4?: string;
};
model operation2Request {
parameters1?: string;
parameters2?: string;
};
model operation1Response {
parameters1?: string;
parameters2?: string;
parameters3?: string;
};
model operation2Response {
parameters1?: string;
parameters2?: string;
parameters3?: string;
};
}

I see, so you had purposefully designed two models called Operation1Request in 2 separate namespaces. Python still doesn't support models in different namespaces, we only generate them in the .models namespace, so this does result in a name conflict error.
Let me forward this issue to our Python code architect though @johanste

Currently, our generators do not support generating things into separate modules/packages/namespaces. We have a feature on the backlog to override the namespace for a given model.
The main reason that it has not been prioritized is that we/azure services avoid using the same model name for any given service due to the possibilities of confusion that @iscai-msft called out. In other words, it has not popped up as an urgent need.
Can you elaborate a bit on the use case you have? Is there a specific scenario where having the same name is strongly preferred?

Hi @johanste , @iscai-msft
Thanks for your answer.
I will explain our use case.
We are developing a financial library that provides endpoints for calculation. The API is designed using Typespec and we would like to generate SDK libraries in Python and Typescript for calculation access.
The exposed API-s are very similar but only differ by the instrument calculation type.
As a sample we have calculation for instrument type FxSpot, FxForward, Curves, etc.
For a FxForward instrument we define interface and models as here:
(there are just an extract for explanation purpose)
interface FxForwardsService
extends CollectionService<FxForwardCollectionResponse, FxForwardResource, FxForwardResponse>
{
@doc("Price a Fx Forward Instrument (pre-trade) on the fly")
@post
@route("/instruments/fx-forwards/$price")
price is customOperationCollection<
FxForwardOnTheFlyPriceRequest,
FxForwardOnTheFlyPriceResponse

;
}

We are using models for the parameters like:
model FxForwardOnTheFlyPriceRequest {
...FxForwardAsTransient;
parameters?: PricingParameters;
marketData?: MarketDataInput;
};
model FxForwardOnTheFlyPriceResponse {
context?: FxForwardOnTheFlyPricingContext;
data?: FxForwardAnalyticsPricing;
marketData?: MarketDataOutput;
};
For fxSpot we have something quite similar:
interface FxSpotsService
extends CollectionService<FxSpotCollectionResponse, FxSpotResource, FxSpotResponse>
{
@doc("Price a Fx Spot Instrument (pre-trade) on the fly")
@post
@route("/instruments/fx-spots/$price")
price is customOperationCollection<
FxSpotOnTheFlyPriceRequest,
FxSpotOnTheFlyPriceResponse

;
}

where parameter models are:
model FxSpotOnTheFlyPriceRequest {
...FxSpotAsTransient;
parameters?: PricingParameters;
marketData?: MarketDataInput;
};
model FxSpotOnTheFlyPriceResponse {
context?: FxSpotOnTheFlyPricingContext;
data?: FxSpotAnalyticsPricing;
marketData?: MarketDataOutput;
};
The definitions are quite similar for FxForward or  FxSpot instrument. We are using in parameters names the instrument type that is already known.
What we would like to be able to do is to continue to have the same definition in a namespace, for simplification reasons.
That means the “price” operation defined as:
price is customOperationCollection<
FxSpotOnTheFlyPriceRequest,
FxSpotOnTheFlyPriceResponse

;
can then be simplified to:
price is customOperationCollection<
OnTheFlyPriceRequest,
OnTheFlyPriceResponse
;

Models as:
FxSpotOnTheFlyPriceRequest can then be OnTheFlyPriceRequest.
FxForwardOnTheFlyPriceRequest becomes OnTheFlyPriceRequest as well.
With the current generator we cannot define the same model’s name in two different namespaces.

I can send to you our typespec APi definition if you need.


Respond to https://hypothes.is/a/Nz7SfDTtEeq7JMMvPVjbFw

Respond to hypothes.is annotation Nz7SfDTtEeq7JMMvPVjbFw on https://jrnold.github.io/r4ds-exercise-solutions/transform.html (user<EMAIL_ADDRESS>created: 2020-01-12T03:40:02.635572+00:00, updated: 2020-01-12T03:40:02.635572+00:00)

Repeated word

Link: https://hyp.is/Nz7SfDTtEeq7JMMvPVjbFw/jrnold.github.io/r4ds-exercise-solutions/transform.html

Already fixed.


Getting correct LogGroupName by retrieving the restApiId first

Fix Issue #1

Todo: update also the test script

@ckho
After merging this I realized that getApiGatewayExecutionLogGroupName is unnecessarily listing the log groups now. Once we have to the API GW ID, we have the full log group name, we only need to test if it exists.


Migrating to travis-ci.com?

The travis-ci.org site will not run any new builds after June 15th, 2021.
Do we have any plans to migrate the repository to travis-ci.com? This is important for architectures like s390x which have CI support through Travis only.

I had plans to migrate to GHA completely, but unfortunately that's not am option for s390 and ppc64le
Thanks for reminder, will try to do migration.

@deniszh Any update on this?

Sorry, I'm offline currently, will check on the weekend

I'm still not sure what to do here TBH. Travis behaviour as a company not becoming better, and I would prefer not to use it. But I do not see alternatives for it. :(
If someone knows - please let me know.

I'm still not sure what to do here TBH. Travis behaviour as a company not becoming better, and I would prefer not to use it. But I do not see alternatives for it. :(
If someone knows - please let me know.

Can you please elaborate more about the Travis behaviour so that we can get issue points and check with Travis to get them resolved.

@V-mourya
I'm doubt that this can be solved. Just google "travis security breach", it's all over the news. E.g. https://arstechnica.com/information-technology/2021/09/travis-ci-flaw-exposed-secrets-for-thousands-of-open-source-projects/


gocyclo:ignore does not work

Hi,
I found that goreportcard somehow ignores gocyclo:ignore which should prevent reporting high scores. When I add comment //gocyclo:ignore to high complexity function the goreportcard still shows this function as high complexity even though manually run gocyclo correctly skip it. You may have a look at dbs2go repo where I have plenty of such examples. It would be nice if you'll fix the issue and skip reporting if function is properly declared with //gocyclo:ignore comment.

@vkuznet I've upgraded gocyclo to v0.3.1 so it should work now. Let me know if you still see the issue.


Fixes #713 by requiring MarkupSafe anterior to 2.1.0

I don't think this is an appropriate fix? The amaranth package shouldn't be introducing dependencies to fix issues with its dependent amaranth-boards package. I'll take a closer look at this later if you can't improve on this PR.

well, the jinja2 team clearly expressed that they won't do that, and that users ought to pin MarkupSafe themselves (see the links to comments in the issue)
Another solution if migrating to latest jinja2 is not possible, would be to fork the target jinja2 release, and fix the code to use the new name of safe_unicode. I may consider doing that too.
Anyway this morning I was just trying using github editor. There are two choices, either keep the current jinja version (this PR) and migrate to latest (PR #715), to see what is more painful.

Codecov Report

Merging #714 (3632bed) into main (9a5a614) will decrease coverage by 0.23%.
The diff coverage is 19.51%.

@@            Coverage Diff             @@
##             main     #714      +/-   ##
==========================================
- Coverage   81.22%   80.99%   -0.24%
==========================================
  Files          49       49
  Lines        6467     6483      +16
  Branches     1526     1747     +221
==========================================
- Hits         5253     5251       -2
- Misses       1021     1038      +17
- Partials      193      194       +1

Impacted Files
Coverage Δ

amaranth/_toolchain/cxx.py
95.12% <ø> (ø)

amaranth/build/plat.py
25.97% <0.00%> (-1.47%)
:arrow_down:

amaranth/tracer.py
90.24% <80.00%> (+1.05%)
:arrow_up:

amaranth/_toolchain/yosys.py
59.13% <100.00%> (-1.89%)
:arrow_down:

amaranth/__init__.py
66.66% <0.00%> (-22.23%)
:arrow_down:

amaranth/build/run.py
21.73% <0.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

See https://github.com/amaranth-lang/amaranth/issues/713#issuecomment-1256881764.


FreeRTOS Unix/Posix simulator

https://www.freertos.org/FreeRTOS-simulator-for-Linux.html

This will allow platforms that use freertos calls on linux to work on SBCs like Raspberry Pi and BeagleBoneBlack.

Resolved by #691


fix for duplicate isle module

fix #299

Pull Request Test Coverage Report for Build 689

5 of 7 (71.43%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.2%) to 72.234%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

isle-sofa-boot-starter/src/main/java/com/alipay/sofa/isle/deployment/impl/AbstractDeploymentDescriptor.java
5
7
71.43%

Totals

Change from base Build 664:
0.2%

Covered Lines:
1652

Relevant Lines:
2287

💛  - Coveralls

Pull Request Test Coverage Report for Build 689

5 of 7 (71.43%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.2%) to 72.234%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

isle-sofa-boot-starter/src/main/java/com/alipay/sofa/isle/deployment/impl/AbstractDeploymentDescriptor.java
5
7
71.43%

Totals

Change from base Build 664:
0.2%

Covered Lines:
1652

Relevant Lines:
2287

💛  - Coveralls


Remove unit enums from device.py

A number of units enums are defined in procedures.py.  They should live in utils/units.py instead.

closed by #440


Executable detection in Linux

After reading the web and github of elastix, I still can not figure out how to install the Elastix command line application. And an error occurred to me, i.e. [Errno 2] No such file or directory: '/opt/local/bin'
I tried to install SimpleElastix, but lots of error occurred, so I gave up and tried to use pyelastix.

I think its a matter of downloading a zip-file and putting it in a sensible place :)

Sorry, I don't know what "a sensible place" mean? could you give me an example, really thanks for your reply

Good point :)  If you're on Windows, that'd be c:\program files (https://github.com/almarklein/pyelastix/blob/master/pyelastix.py#L98)

Good point :) If you're on Windows, that'd be c:\program files (https://github.com/almarklein/pyelastix/blob/master/pyelastix.py#L98)

thanks, what if I am in linux system, does it mean that I should do some modifications like: ### sys.platform.startswith('linux'), and indicate a specific path like: ### "/my_account/bin/", and download elastix in this path. I will try it, really thanks for your reply

Good point :) If you're on Windows, that'd be c:\program files (https://github.com/almarklein/pyelastix/blob/master/pyelastix.py#L98)

Sorry, I occurred a new error, I solve the former problem by doing these:

I downloaded the elastix from http://elastix.isi.uu.nl/download_links.php, I clicked "### Download" and  downloaded binaries via GitHub (linux version), and unzip it to ### "/my_account/elastix"
I modified ### pyelastix.py#L102 as possible_locations.extend([ "/my_account/elastix/bin"]), since elastix and transform files are in ./bin/
a new error occurred:###  Could not find Elastix executable. Download Elastix from http://elastix.isi.uu.nl/. Pyelastix looks for the exe in a series of common locations. Set ELASTIX_PATH if necessary.

To solve it, I try this code: ### os.environ['ELASTIX_PATH'] = r'/my_account/elastix/bin/' before excuting pyelastix.register(), however, unsuccessful.
Then, I downloaded win version from GitHub, I found there were two files: elastix.exe and transform.exe, but the unzipped linux version files are: elastix and transform, without .exe, (I download elastix from https://github.com/SuperElastix/elastix/releases/tag/4.9.0), I don't know how to solve it.

Good point :) If you're on Windows, that'd be c:\program files (https://github.com/almarklein/pyelastix/blob/master/pyelastix.py#L98)

Sorry, I occurred a new error, I solve the former problem by doing these:
1. I downloaded the elastix from http://elastix.isi.uu.nl/download_links.php, I clicked " **Download**" and  downloaded binaries via GitHub (linux version), and unzip it to **"/my_account/elastix"**

2. I modified  **pyelastix.py#L102** as possible_locations.extend([ "/my_account/elastix/bin"]), since elastix and transform files are in ./bin/

3. a new error occurred: **Could not find Elastix executable. Download Elastix from http://elastix.isi.uu.nl/. Pyelastix looks for the exe in a series of common locations. Set ELASTIX_PATH if necessary.**

To solve it, I try this code: os.environ['ELASTIX_PATH'] = r'/my_account/elastix/bin/' before excuting pyelastix.register(), however, unsuccessful.
Then, I downloaded win version from GitHub, I found there were two files: elastix.exe and transform.exe, but the unzipped linux version files are: elastix and transform, without .exe, (I download elastix from https://github.com/SuperElastix/elastix/releases/tag/4.9.0), I don't know how to solve it.

To remove your last confusion.. windows uses .exe-files to denote executables, unix system won't use exentions like that. So that seems fine I guess

Good point :) If you're on Windows, that'd be c:\program files (https://github.com/almarklein/pyelastix/blob/master/pyelastix.py#L98)

Sorry, I occurred a new error, I solve the former problem by doing these:
1. I downloaded the elastix from http://elastix.isi.uu.nl/download_links.php, I clicked " **Download**" and  downloaded binaries via GitHub (linux version), and unzip it to **"/my_account/elastix"**

2. I modified  **pyelastix.py#L102** as possible_locations.extend([ "/my_account/elastix/bin"]), since elastix and transform files are in ./bin/

3. a new error occurred: **Could not find Elastix executable. Download Elastix from http://elastix.isi.uu.nl/. Pyelastix looks for the exe in a series of common locations. Set ELASTIX_PATH if necessary.**

To solve it, I try this code: os.environ['ELASTIX_PATH'] = r'/my_account/elastix/bin/' before excuting pyelastix.register(), however, unsuccessful.
Then, I downloaded win version from GitHub, I found there were two files: elastix.exe and transform.exe, but the unzipped linux version files are: elastix and transform, without .exe, (I download elastix from https://github.com/SuperElastix/elastix/releases/tag/4.9.0), I don't know how to solve it.

To remove your last confusion.. windows uses .exe-files to denote executables, unix system won't use exentions like that. So that seems fine I guess

Thanks, after I unzip elastix linux version, transform and elastix (2 files) are in ./bin, and I indicate the path but program error told me there is no executable Elastix, I am really confused.

My advice would be to add a print statement here (and decomment the one a few lines down), so you can see what files are attempted.

On Linux, one needs to set ELASTIX_PATH to full path to elastix binary (.../elastix-4.9.0-linux/bin/elastix).
Also, the distributed libraries must be available from the common locations (e.g. /usr/lib). Alternatively, set/add to LD_LIBRARY_PATH the path to lib directory of elastix (.../elastix-4.9.0-linux/lib/).

The issue with [Errno 2] No such file or directory: '/opt/local/bin' is a duplicate of #15 and will be closed with #20 .

How should this be set for MacOS? I have installed elastix using method 3.2.1 ("The really easy way") in the manual and added the PATH and DYLD_LIBRARY_PATH info to my .bash_profile file. I can run $ elastix --help in Terminal just fine. However, with Python, pyelastix.register() fails with the error

RuntimeError: Could not find Elastix executable. Download Elastix from http://elastix.isi.uu.nl/. Pyelastix looks for the exe in a series of common locations. Set ELASTIX_PATH if necessary.

I tried explicitly setting the ELASTIX_PATH environment parameter using
os.environ["ELASTIX_PATH"] = "/Applications/elastix/bin/elastix"
(which is where elastix lives according to $ which elastix in Terminal).
Mac OS Catalina 10.15.3
Python 3.7.4
elastix 5.0.0
pyelastix 1.2

@JepsonNomad same problem here. MacOS Catalina 10.15.6.
Did you find any solution for this?
RuntimeError: Could not find Elastix executable.
Download Elastix from http://elastix.isi.uu.nl/. Pyelastix looks for the exe in a series of common locations.
Set ELASTIX_PATH if necessary.

which elastix works fine, and setting the os.environ['ELASTIX_PATH'] manually also still runs into the same error message.

@horsto Sadly I gave up on pyelastix after juggling some other projects. Good luck, friend.

Thanks @JepsonNomad.
Any idea @soupault ?

On Linux, one needs to set ELASTIX_PATH to full path to elastix binary (.../elastix-4.9.0-linux/bin/elastix).
Also, the distributed libraries must be available from the common locations (e.g. /usr/lib). Alternatively, set/add to LD_LIBRARY_PATH the path to lib directory of elastix (.../elastix-4.9.0-linux/lib/).
The issue with [Errno 2] No such file or directory: '/opt/local/bin' is a duplicate of #15 and will be closed with #20 .

I also encounter this issue:
Could not find Elastix executable. Download Elastix from http://elastix.isi.uu.nl/. Pyelastix looks for the exe in a series of common locations. Set ELASTIX_PATH if necessary.
The problem is solved when i set paths as below. Hope it useful for someone.
os.environ['LD_LIBRARY_PATH'] = '/path/lib'
os.environ['ELASTIX_PATH']= "/path/bin/elastix"

在terminal：
cd /home/anaconda3/envs/lib/python3.11/site-packages
wget https://github.com/SuperElastix/elastix/releases/download/5.0.0/elastix-5.0.0-linux.tar.bz2 #下载失败；从官网下载上传至site-packages
注意此处的版本elastix-5.0.0-linux，因为系统版本gcc version 7.5.0 (Ubuntu 7.5.0-3ubuntu1~18.04)，elastix-5.0.0-linux的gcc 7.4.0
tar -xvjf elastix-5.0.1-linux.tar.bz2
export PATH=$PATH:/home/anaconda3/envs/lib/python3.11/site-packages/elastix-5.0.0-linux/bin
which elastix
在shell中：
echo 'export PATH=$PATH:/home/anaconda3/envs/lib/python3.11/site-packages/elastix-5.0.0-linux/bin' >> ~/.bashrc
source ~/.bashrc
echo 'export LD_LIBRARY_PATH=$LD_LIBRARY_PATH:/home/anaconda3/envs/lib/python3.11/site-packages/elastix-5.0.0-linux/lib' >> ~/.bashrc
source ~/.bashrc
os.environ['LD_LIBRARY_PATH'] = '/home/anaconda3/envs/lib/python3.11/site-packages/elastix-5.0.0-linux/lib'
os.environ['ELASTIX_PATH']= "/home/anaconda3/envs/lib/python3.11/site-packages/elastix-5.0.0-linux/bin/elastix"
配准参数设置
params = pyelastix.get_default_params(type='RIGID') #type='BSPLINE','RIGID', 'AFFINE', 'BSPLINE'
params.NumberOfResolutions = 4  # 多分辨率策略
params.MaximumNumberOfIterations = 500  # 每级最大迭代次数
print("mov→ref:", mov,ref)
执行配准
result,fields = pyelastix.register(mov,ref, params)
if isinstance(result, pyelastix.Image):
result_array = np.array(result)
result = sitk.GetImageFromArray(result_array)
result.CopyInformation(ref_image)
保存配准结果
output_filename = mov.replace('.nii.gz', '_r.nii.gz')
print("Saving result to:", output_filename)
sitk.WriteImage(result, output_filename)


Add correct extension store links when displaying the text "check out the watch on odysee extension"

Bug
When the text "check out the watch on odysee extension" is shown and you click on it, you are always redirected to the Chrome Extension Store. Add support for other browsers (Firefox, Edge, Opera). Best way would be probably to check the user agent and change the link accordingly.
To Reproduce
Refresh the homepage until the text appears.
Expected behavior
The text should redirect to the correct store.
Screenshots

System Configuration

Additional context

This is a good idea, thought the link won't show up for long. But might be cool to include the installation link in other areas, so still useful!


[WIP] Removing legacy modules. Disabled SSE/WS client tests for now

Preview branch removing all legacy modules. This will be merged in late 2024. From then on, the DGS/Spring GraphQL integration will be the only implementation.

@srinivasankavitha @kilink Preparing this branch for onboarding purposes. Not ready to merge yet, but it's a preview of things to come.
Note that I've disabled some client SSE/WS tests for now, need to replace some things in the setup with Spring GraphQL to get those to work.

Looks good to me. I've noticed quite a few projects that add graphql-dgs-spring-boot-starter as a dependency in their build.gradle files in addition to the starter. This is incorrect usage in any case. Calling it out explicitly so we are not surprised by breakages there.


Tests?

They are the most important todo in the todo list :(

@PirosB3 tests were written, as of e631553322c8c49450cebb57af84855498374aef
I think I can close this ;)


possible to add contributor without a name - even though it appears as compulsory field

@raycarrick @briri
I am testing on DMPonline-dev the latest 3.3 release. Not sure
The contributor name appears as compulsory field but when I left it empty it was alright to save:

See contributor without a name:

The logic in the code requires one or the other (name or email). Should we actually require both or is one more important?
Whoever addresses this ticket should also remove the phone number per #2941

Will look at https://github.com/DMPRoadmap/roadmap/issues/2941 next. Have a PR see above.

@johnpinto1 both still appears as compulsory fields on dev follow up with @dsisu once fixed so she can re-test and close

The view is displaying wrong info. The code saves either name or email, but view shows incorrectly that both name and email are required.

Trivial fix done and PR created https://github.com/DMPRoadmap/roadmap/pull/3120

Just catching up with things as part of trying to understand what the latest release covered: how is it obvious to a user in this new version that either name or email needs to be provided? Do we provide a meaningful error message if it isn't? Should we add a sentence saying that one of the two fields needs to be filled?

This is a good point. @johnpinto how easy would it be to add a message to that effect? Fill in at least one of these fields?

@raycarrick-ed & @pherterich It should be trivial. Famous last words. Its just adding an text.

"Please provide either Name or Email in addition to the fields indicated as mandatory using *"
Other suggestions on how to phrase this @dsisu @mariapraetzellis ?

If the logic makes this easy to implement, I would say make both name and email address fields mandatory (and marked as such).
From the users' perspective, as readers, it looks much better if all contributors are named. The email address can be very different from the name and give no obvious indication who the contributor is.
From the user's perspective as the form-filler (the author of the plan), it's not a lot of information to ask of them. They must have spoken to their contributors so they will probably have their name and email address close to hand.
If we don't make both fields mandatory, then we Patricia's suggested wording is fine.

@dsisu CDL requires either name and email address fields , so it would be better to not require both mandatory.


Admin password is written in plaintext to log on admin creation

When admin user does not exist, the system creates the user and outputs the username AND password in plaintext to the log, which is quite bad for security:
Admin user created. Username: ADMINUSER, Password: ADMINPASSWORD

Resolved.


results for LoRA

Thank you for your valuable contributions. I am currently attempting to replicate the outcomes presented in your research paper. However, I am encountering difficulties in obtaining the desired results when I attempt to re-run LoRA adapters.
copa: 76.00 (2.00), h-swag: 26.64 (0.36), storycloze: 84.87 (0.21), winogrande: 51.14 (2.13), wsc: 65.38 (2.88), wic: 51.57 (0.63), rte: 59.57 (0.36), cb: 51.79 (1.79), anli-r1: 34.80 (0.80), anli-r2: 34.00 (2.40), anli-r3: 32.92 (1.08)
Have you encountered situations where the training of "h-swag" and "rte" did not yield successful results?

Ignore it. I did not set the eval_epoch_interval for that before. Sorry.


PMM-7 fixed portal toast

https://github.com/percona/pmm-ui-tests/actions/runs/5966628864/job/16186661990


[WIP] use tweetnacl for bundled nacl instead of libsodium

avoids need to bundle an extra library
Bummer: build settings for tweetnacl have never worked (should be fixed in libzmq-4.1.5)
This works fine on my laptop (OS X), but I can't get the static linking working properly on Linux.
I'll squash all the silly debug commits once I figure out what's up.

Thanks for doing this Min.

It works!

Hoorah!


Time Inject - Enhanced - another bug i think

Using the Time Inject Enhanced node - runs perfectly for just sending message payload
If i add another option to inject - this time global.variable (full variable name and path added) the system accepts the input - however does not prompt me to redeploy when i save the changes - i have to reopen the node (and the global.variable is blank), refill the global variable and then change something else in the node, then save and then i am prompted to redeploy - i do a full deploy of the workspace not just changed nodes
I can then go back in and the global variable is again blank - when i left it overnight wanting to see what would happen - this morning when it went to run it show Internal Error in Red under the node in the editor

Expected behavior
I expect the system to remember the global variable name i am trying to set and to inject it at the designated time
System information (please complete the following information):

Node Version [e.g. 0.4.7]
Node-Red Version 3.02

Additional context
Add any other context about the problem here.

Very similar issue happening with a flow variable.
Same behaviour after adding a couple of flow variables to be set up with local timestamp information once the node is triggered:

the node is accepted and properly deployed;
at the first scheduled occurrence (once per day) it returns: 'internal error'
opening the node to check the initially set flow variables are not shown / retained

System information:

Node-Red version: 3.1.7
Node.js version: 20.11.1
node-red-contrib-sun-position version: 2.2.0-beta3


Invariant Violation: Native component for "SRSRadialGradient" does not exist

Hi! Thanks for this project!
I`m trying to get it set on a new React Native project on iOS, carefully going through your instructions.
But got this and have no idea how to fix it:
Invariant Violation: Native component for "SRSRadialGradient" does not exist
This error is located at:
in SRSRadialGradient (at index.js:28)
in RCTView (at View.js:71)
in View (at index.js:27)
in RadialGradient (at App.js:21)
in App (at renderApplication.js:35)
in RCTView (at View.js:71)
in View (at AppContainer.js:102)
in RCTView (at View.js:71)
in View (at AppContainer.js:122)
in AppContainer (at renderApplication.js:34)
getViewConfig
requireNativeComponent.js:112:6
get$1
ReactNativeRenderer-dev.js:11859:17
createInstance
ReactNativeRenderer-dev.js:12022:27
completeWork
ReactNativeRenderer-dev.js:8814:14
completeUnitOfWork
ReactNativeRenderer-dev.js:10418:8
performUnitOfWork
ReactNativeRenderer-dev.js:10523:32
workLoop
ReactNativeRenderer-dev.js:10584:43
_invokeGuardedCallback
ReactNativeRenderer-dev.js:134:15
invokeGuardedCallback
ReactNativeRenderer-dev.js:67:32
renderRoot
ReactNativeRenderer-dev.js:10677:30
performWorkOnRoot
ReactNativeRenderer-dev.js:11449:34
performWork
ReactNativeRenderer-dev.js:11364:8
requestWork
ReactNativeRenderer-dev.js:11258:18
scheduleWorkImpl
ReactNativeRenderer-dev.js:11099:22
scheduleRootUpdate
ReactNativeRenderer-dev.js:11690:17
render
ReactNativeRenderer-dev.js:12919:35
renderApplication
renderApplication.js:58:21
runApplication
AppRegistry.js:196:26
__callFunction
MessageQueue.js:353:47

MessageQueue.js:118:26
__guardSafe
MessageQueue.js:316:6
callFunctionReturnFlushedQueue
MessageQueue.js:117:17
I tried to fix this using https://stackoverflow.com/questions/47672941/invariant-violation-native-component-for-rnsvgsvgview-does-not-exist but this didn`t help.
Maybe you have some ideas?

Hi,
Please use the latest library v1.0.2. It should solve your IOS problem.

Hi,
I am having the same problem. Just install react-native-radial-gradient and my version is  "^1.0.3".
Any help?
Thanks!

I think you have created the react native project using create-react-native-app. This library renders a native view. So it will not work with projects that are created using create-react-native-app. You should create your project using react-native init or you have to convert your project to a native implementation. Follow this link for more info

Facing same issue, how can I fix it.
Version: 1.0.5
ReactNative version: 0.59.0
Not using expo, created with react-native init

I am using RN 0.60.4 and react-native-radial-gradient version 1.0.5
I am using Cocoapods, therefore I have included it in pod file
pod 'SRSRadialGradient', :path => '../node_modules/react-native-radial-gradient/ios'
and getting this error
Invariant Violation: requireNativeComponent: "SRSRadialGradient" was not found in the UIManager.

I am using RN 0.60.4 and react-native-radial-gradient version 1.0.5
I am using Cocoapods, therefore I have included it in pod file
pod 'SRSRadialGradient', :path => '../node_modules/react-native-radial-gradient/ios'
and getting this error
Invariant Violation: requireNativeComponent: "SRSRadialGradient" was not found in the UIManager.

I am using RN 0.61.2 and got the same issue

Same here, why was this issue closed? Did you guys find a solution?

same issue. using Expo and "react-native-radial-gradient": "^1.1.4"

Issue has been happening for 6 years. Anyone have a solution?


Feature/update hal drivers

In order for me to adopt amp-hal-st for one of my projects I require a custom startup script. (I need to  initialize ethe ccmram). This ccmram is not configured by the default startup script of the G4 series.
Would it be possible to have the startup scripts as optional dependencies?


[Documentation] Doc mapping field types

The documentation is unclear raw types vs composite types.
raw type also contains json type, which can be considered a composite type.
It's unclear when to pick json or object

Partially linked to #3134


linux: Remove 4.8

Motivation for this change
The 4.8.x kernel series has been EOL'd, we should get rid of it before we close in on the next release.

I wonder if 4.9 might become a longterm branch.  Using 4.4 as default for release-17.03 would seem rather old to me...

According to http://kroah.com/log/blog/2016/09/06/4-dot-9-equals-equals-next-lts-kernel/ it will, but let's wait regardless for the official announcement. I suspect it will be announced with 4.10 release

It is now officially a LTS: https://www.kernel.org/category/releases.html

Yes, we switched the default already: https://github.com/NixOS/nixpkgs/commit/0fdef7d2f170f


Improper processing of the key modifiers (Ctrl, Alt, Shift, Win)

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

VS Code Version:
Version: 1.93.0
Commit: 4849ca9bdf9666755eb463db297b69e5385090e3
Date: 2024-09-04T13:02:38.431Z
Electron: 30.4.0
ElectronBuildId: 10073054
Chromium: 124.0.6367.243
Node.js: 20.15.1
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.22631

OS Version:
OS Name: Microsoft Windows 11 Pro
Version: 10.022631 Build 22631

Issue description:
I have a keyboard switcher that performs some actions by pressing the RightCtrl and RightShif keys. This does not work in VSCode because pressing and releasing RightCtrl doesn't pass to the system. Maybe it is treated as a noncompleted VSCode shortcut, and VSCode aborts its processing.

Steps to Reproduce:

Press RightCtrl
Release RightCtrl

Result: Nothing is happening in VSCode, and nothing is happening in the system.
Expected result: nothing is happening in VSCode, Press and Release event passing next to the system so other apps can process pressing and releasing RightCtrl.

I suggest you use the information at https://github.com/microsoft/vscode/wiki/Keybinding-Issues to investigate.

Thank you for the useful link. I studied it but was not lucky enough to find a working fix. ChatGPT advised me this:
"keyboard.dispatch": "keyCode"
and this
code --disable-features=ElectronEnableGlobalShortcuts
but I was not sure whether I applied them correctly or if they may have worked.

Unbelievable! Running VSCode NOT as Administrator resolved this issue. How can this be connected?


SQL File change gets deployed even when author attribute is null

Search first

[X] I searched and no similar issues were found

Description
@jccampanero  has identified an issue when when a SQL file change is included in a changeset with author=null a validation error is displayed , but SQL file gets executed anyway.
Steps To Reproduce
Having a changelog like the one below:
databaseChangeLog:
  - changeSet:
      id: create-schema
      changes:
        - sqlFile:
            path: changelog/sql/storeProcedure.sql
            splitStatements: false

where author has not been provided, a validation error for missing author property is returned, but SQL file change gets deployed anyway.
Returned error:
[2023-09-04 16:59:39] SEVERE [liquibase.integration] ERROR: null value in column "author" of relation "databasechangelog" violates not-null constraint
  Detail: Failing row contains (create-schema, null, changelog/yaml/missing.author.changelog.yaml, 2023-09-04 16:59:39.031778, 1, EXECUTED, 9:f818070af488662a19fefd0af125ea34, sqlFile path=changelog/sql/storeProcedure.sql, , null, DEV, null, null,<PHONE_NUMBER>). [Failed SQL: (0) INSERT INTO public.databasechangelog (ID, AUTHOR, FILENAME, DATEEXECUTED, ORDEREXECUTED, MD5SUM, DESCRIPTION, COMMENTS, EXECTYPE, CONTEXTS, LABELS, LIQUIBASE, DEPLOYMENT_ID) VALUES ('create-schema', NULL, 'changelog/yaml/missing.author.changelog.yaml', NOW(), 1, '9:f818070af488662a19fefd0af125ea34', 'sqlFile path=changelog/sql/storeProcedure.sql', '', 'EXECUTED', NULL, NULL, 'DEV', '3857578976')]
liquibase.exception.CommandExecutionException: liquibase.exception.LiquibaseException: liquibase.exception.DatabaseException: liquibase.exception.DatabaseException: ERROR: null value in column "author" of relation "databasechangelog" violates not-null constraint
  Detail: Failing row contains (create-schema, null, changelog/yaml/missing.author.changelog.yaml, 2023-09-04 16:59:39.031778, 1, EXECUTED, 9:f818070af488662a19fefd0af125ea34, sqlFile path=changelog/sql/storeProcedure.sql, , null, DEV, null, null,<PHONE_NUMBER>). [Failed SQL: (0) INSERT INTO public.databasechangelog (ID, AUTHOR, FILENAME, DATEEXECUTED, ORDEREXECUTED, MD5SUM, DESCRIPTION, COMMENTS, EXECTYPE, CONTEXTS, LABELS, LIQUIBASE, DEPLOYMENT_ID) VALUES ('create-schema', NULL, 'changelog/yaml/missing.author.changelog.yaml', NOW(), 1, '9:f818070af488662a19fefd0af125ea34', 'sqlFile path=changelog/sql/storeProcedure.sql', '', 'EXECUTED', NULL, NULL, 'DEV', '3857578976')]
	at liquibase.command.CommandScope.execute(CommandScope.java:237)
	at liquibase.integration.commandline.CommandRunner.call(CommandRunner.java:55)
	at liquibase.integration.commandline.CommandRunner.call(CommandRunner.java:24)
	at picocli.CommandLine.executeUserObject(CommandLine.java:2041)
	at picocli.CommandLine.access$1500(CommandLine.java:148)
	at picocli.CommandLine$RunLast.executeUserObjectOfLastSubcommandWithSameParent(CommandLine.java:2461)
	at picocli.CommandLine$RunLast.handle(CommandLine.java:2453)
	at picocli.CommandLine$RunLast.handle(CommandLine.java:2415)
	at picocli.CommandLine$AbstractParseResultHandler.execute(CommandLine.java:2273)
	at picocli.CommandLine$RunLast.execute(CommandLine.java:2417)
	at picocli.CommandLine.execute(CommandLine.java:2170)
	at liquibase.integration.commandline.LiquibaseCommandLine.lambda$execute$2(LiquibaseCommandLine.java:383)
	at liquibase.Scope.child(Scope.java:192)
	at liquibase.Scope.child(Scope.java:168)
	at liquibase.integration.commandline.LiquibaseCommandLine.lambda$execute$3(LiquibaseCommandLine.java:358)
	at liquibase.Scope.child(Scope.java:192)
	at liquibase.Scope.child(Scope.java:168)
	at liquibase.integration.commandline.LiquibaseCommandLine.execute(LiquibaseCommandLine.java:356)
	at liquibase.integration.commandline.LiquibaseCommandLine.main(LiquibaseCommandLine.java:96)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.base/java.lang.reflect.Method.invoke(Method.java:566)
	at liquibase.integration.commandline.LiquibaseLauncher.main(LiquibaseLauncher.java:107)

As well as below log messages showing store procedure from the included sql file has been successfully deployed:
[2023-09-04 16:59:38] FINE [liquibase.database] Executing Statement: CREATE OR REPLACE PROCEDURE testHello()
LANGUAGE plpgsql
AS $$
BEGIN
    raise notice ' Hello From The Database! ';
END; $$
[2023-09-04 16:59:38] FINE [liquibase.executor] CREATE OR REPLACE PROCEDURE testHello()
LANGUAGE plpgsql
AS $$
BEGIN
    raise notice ' Hello From The Database! ';
END; $$
[2023-09-04 16:59:39] FINE [liquibase.executor] 0 row(s) affected
[2023-09-04 16:59:39] INFO [liquibase.changelog] SQL in file changelog/sql/storeProcedure.sql executed

Expected/Desired Behavior
author null validation error should be returned, and SQL file change should not be deployed if there is any error.
Liquibase Version
latest
Database Vendor & Version
PostgreSQL 14.4
Liquibase Integration
CLI
Liquibase Extensions
No response
OS and/or Infrastructure Type/Provider
No response
Additional Context
No response
Are you willing to submit a PR?

[ ] I'm willing to submit a PR (Thank you!)

Thank you @jccampanero for testing #4249 and #3200 for us and finding this issue while performing your test.
Regards,
Daniel.

Sorry for the late reply @MalloD12 Daniel.

Thank you very much for opening this new issue: I would be glad to work on it if you wish.

Sorry for the late reply @MalloD12 Daniel.

Thank you very much for opening this new issue: I would be glad to work on it if you wish.

That would be great, thank you @jccampanero!

Hi @jccampanero,
Is there anything I can help you with on this issue?
Thanks,
Daniel.

Sorry for the late reply @MalloD12 Daniel.
Probably Daniel 😄 , a lot of things...
Seriously: I am very sorry, mainly for personal reasons, I haven't had time to work on the issue, in any issue in fact.
I will try to do my best and work on it during this holidays.
By the way, Merry Xmas and Happy New Year my friend.

Sorry for the late reply @MalloD12 Daniel.
Probably Daniel 😄 , a lot of things...
Seriously: I am very sorry, mainly for personal reasons, I haven't had time to work on the issue, in any issue in fact.
I will try to do my best and work on it during this holidays.
By the way, Merry Xmas and Happy New Year my friend.

No worries, that sounds good to me. Merry Xmas and Happy New Year, Jose!!


We are IBM not Microsoft....

https://github.com/IBM/ibm.github.io/blob/85002f387daa2fadde5974b70629b12a26c8888c/index.html#L4
We should probably change that incase someone gets stuck.

Opps, looked at the wrong link... my mistake!


Laravel is not respecting time zone settings in the php.ini

Currently, the date.timezone setting in the php.ini is ignored.
Removing "timezone" from the app/config.php will always default to UTC no matter what but date_default_timezone_get() will default to UTC if the date.timezone setting is not set but use the timezone specified if it is set.
Developers can still set a default timezone in the php.ini, have it be respected, and override it in the application if necessary.
I have a follow-up change for laravel/laravel to comment the timezone config by default.

This is massive breaking change and something we should absolutely avoid. I think UTC as a default is the right choice and this has been the default for many years already.

This is massive breaking change and something we should absolutely avoid. I think UTC as a default is the right choice and this has been the default for many years already.

How is it a breaking change? If date.timezone is not set in the php.ini, date_default_timezone_get returns UTC which is what is currently hard coded. https://www.php.net/date_default_timezone_get

@haydenk if people have date.timezone set to something else than UTC in their php.ini they'll suddenly get a different timezone.

@driesvints That would mean that they explicitly set the timezone in PHP since it is not set by default. The laravel/laravel project could still keep setting the timezone to UTC by default as it could still override the setting.

Agree with Dries here. Choosing a timezone that is not UTC is almost always the wrong option, and even then, this is a huge breaking change.


Update attachment_adobe_image_lure.yml

FN coverage. Adding message screenshot

/update-test-rules

Testing looks great, this is gtg.


Update docs url on angular-cli repo

The docs on the angular-cli repo point to cli.angular.io but should point to angular.dev/cli as they currently redirect there.

Link has been updated.


Haxe build problem

Describe your problem here.
I don't know if this is a problem with the engine or a problem with haxe but, whenever I initiate the command lime build windows (keep in mind that it only happens with this command, all other commands do what they're supposed to without this happening), I have to reinstall haxe in order to initiate lime commands again.
Haxe version:
4.2.5
Psych engine version:
0.6.2
Are you modding a build from source or with Lua?
Source
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
No

Try lime test windows

Try lime test windows

Like I said, that command works fine without this happening, it only happens with lime build windows


Consistent use of your-tenant

In Create an Azure App, step 7, SignOn URL, your-tenant is used to refer to the base part of the tenant name. In Configure Azure AD as an Identity Provider, step 7, Metadata URL, it is not clear if your-tenant refers only to the base part, or to the full tenant name contoso.onmicrosoft.com. Can you please be consistent and provide an example for the metadataurl.

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

ID: bac6db2d-bff9-cbcf-d522-966ed17ea27f
Version Independent ID: 91e3eeb7-8602-9e26-21c3-1fa70d03a354
Content: Set up sign-in Azure Active Directory accounts a built-in policy in Azure Active Directory B2C
Content Source: articles/active-directory-b2c/active-directory-b2c-setup-oidc-azure-active-directory.md
Service: active-directory
GitHub Login: @davidmu1
Microsoft Alias: davidmu

@mlinehan17
Thanks for your feedback! We will investigate and update as appropriate.

I agree that it could be clearer. Based on other documentation your-tenant in this context is just the base part so it would look like: https://fabrikamb2c.b2clogin.com/fabrikamb2c.onmicrosoft.com/
I'm assigning this to the content author, @davidmu1 , to look into. I think it would be helpful to have a sample link like the one above.

Added examples of the appropriate URLs:
For the Sign-on URL, enter the following URL in all lowercase letters, where your-B2C-tenant-name is replaced with the name of your Azure AD B2C tenant. For example, https://fabrikam.b2clogin.com/fabrikam.b2clogin.com/oauth2/authresp:
For Metadata url, enter the following URL replacing your-AD-tenant-domain with the domain name of your Azure AD tenant. For example https://login.microsoftonline.com/contoso.onmicrosoft.com/.well-known/openid-configuration:
#please-close

@mlinehan17
We will now proceed to close this thread. If there are further questions regarding this matter, please respond here and @YutongTie-MSFT and we will gladly continue the discussion.

Thanks for updating the docs. Much clearer now 😊
Michael
Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows 10

From: David Murray<EMAIL_ADDRESS>Sent: Monday, October 22, 2018 9:22:37 PM
To: MicrosoftDocs/azure-docs
Cc: mlinehan17; Mention
Subject: Re: [MicrosoftDocs/azure-docs] Consistent use of your-tenant (#17230)
Added examples of the appropriate URLs:
For the Sign-on URL, enter the following URL in all lowercase letters, where your-B2C-tenant-name is replaced with the name of your Azure AD B2C tenant. For example, https://fabrikam.b2clogin.com/fabrikam.b2clogin.com/oauth2/authresphttps://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Ffabrikam.b2clogin.com%2Ffabrikam.b2clogin.com%2Foauth2%2Fauthresp&data=02|01||c0e164c9f15148b9041d08d6385c1c54|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636758365587240469&sdata=7yoDr%2FxBv32Mt2gialdcivwl0KCY2lzfwNuuv7Js0aU%3D&reserved=0:
For Metadata url, enter the following URL replacing your-AD-tenant-domain with the domain name of your Azure AD tenant. For example https://login.microsoftonline.com/contoso.onmicrosoft.com/.well-known/openid-configurationhttps://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Flogin.microsoftonline.com%2Fcontoso.onmicrosoft.com%2F.well-known%2Fopenid-configuration&data=02|01||c0e164c9f15148b9041d08d6385c1c54|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636758365587240469&sdata=8yDvGjx1jjrqjJLIykwsZ5piP%2BxIMwPyCBx8zZ0as%2BA%3D&reserved=0:
#please-close
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F17230%23issuecomment-431968896&data=02|01||c0e164c9f15148b9041d08d6385c1c54|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636758365587240469&sdata=dN6dSV6ILg0Whu5eUfzocHI3i7km48XQx4ck1YRwc%2Fc%3D&reserved=0, or mute the threadhttps://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FANS_KF3ziLuA7ZzGkgB3HE1ubSiw1GuPks5unikNgaJpZM4XwgNF&data=02|01||c0e164c9f15148b9041d08d6385c1c54|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636758365587240469&sdata=emhbvSvdIg09HceeasbSV7BaA0gm1WG9J3B57TxTFHc%3D&reserved=0.


Fix tutorials examples

This is just a reminder that we should update the examples in the tutorials repo to work before 1.5.0 (or fix HPX before 1.5.0 to make the examples work).

The non-CUDA examples now build (which means everything on the CircleCI builder): https://app.circleci.com/pipelines/github/STEllAR-GROUP/tutorials/. The CUDA examples have other problems, and need more work anyway. I'll close this now.


Print expected type instead of its string representation

Observed
Presently, if a user passes an object O other a list to a Schema ListOf object, the error message will print the string representation S of O. Unfortunately, the error message is obscured when the S is large (e.g. O is a set with hundreds of thousands of strings).
Expected
I think the error message would be just as effective if we simply printed the type of O.
Thanks!

Moved to securesystemslib


Fix path issue.

Running node server.js was expecting server.js to be in the root of the project, not the src directory. With this patch you can now run the app via node src/server.js.

I went ahead and merged this but it looks like it messes up the dynamic routing (RESTful apis won't load now - 404 errors). I'm going to leave it in place and tweak things to see if I can get it back to the prior state since I like the general idea of not worrying about where server.js is.


PDB file is not downloaded from Windows Symbol server

Version and Platform (required):

Binary Ninja Version: 4.0.4958 Personal, ddff9339
OS: windows
OS Version: 10
CPU Architecture: x86_64

Bug Description:
No symbols are shown, and no PDB downloaded from MS symbol server
Steps To Reproduce:

Run Binary Ninja
Load any Windows driver module .sys (that was not disassembled yet)

Expected Behavior:
Binary Ninja shows disassembly with symbols from MS symbol server
Screenshots:
Additional Information:
Tried with both default symbol directory and custom C:\Symbols
Also it works as expected if I manually download the PDB file (it has weird name .blob) manually via the link shown in the log and place it into the folder where Binary Ninja expects to find the PDB file.

[Default] Check file exists: C:\symbols/spaceport.pdb/7B52D30874904F02848B1676EF2FCB1B1/spaceport.pdb
[Default] Check file exists: C:\symbols/spaceport.pdb/7B52D30874904F02848B1676EF2FCB1B1/file.ptr
[Default] Check file exists: https://msdl.microsoft.com/download/symbols/spaceport.pdb/7B52D30874904F02848B1676EF2FCB1B1/spaceport.pdb
[Default] HEAD: https://msdl.microsoft.com/download/symbols/spaceport.pdb/7B52D30874904F02848B1676EF2FCB1B1/spaceport.pdb
[Default] Read file: https://msdl.microsoft.com/download/symbols/spaceport.pdb/7B52D30874904F02848B1676EF2FCB1B1/spaceport.pdb
[Default] GET: https://msdl.microsoft.com/download/symbols/spaceport.pdb/7B52D30874904F02848B1676EF2FCB1B1/spaceport.pdb
[Analysis] Analysis update took 13.168 seconds
[Default] Analysis starting...
[Analysis.LinearSweep] Adding section: .text <0x1c0001000 - 0x1c0032a1e> for analysis.
[Analysis.LinearSweep] Adding section: INIT <0x1c007a000 - 0x1c007a21a> for analysis.
[Analysis.LinearSweep] Adding section: PAGE <0x1c0046000 - 0x1c0079b9d> for analysis.
[Analysis.LinearSweep] Analysis Phase 2.0 created 0 functions. [x86_64:windows-kernel-x86_64]
[Analysis.LinearSweep] Analysis Phase 2.1 created 0 functions. [x86_64:windows-kernel-x86_64]
[Analysis.LinearSweep] Analysis Phase 3.0 created 8 functions [x86_64:windows-kernel-x86_64]
[Analysis.LinearSweep] Analysis Phase 3.1 created 0 functions [x86_64:windows-kernel-x86_64]
[Analysis.SignatureMatcher] 0 functions matched total, 0 name-only matches, 6 thunks resolved, 34 functions skipped because they were too small
[Analysis.PointerSweep] Initial code analysis complete, starting pointer sweep...
[Analysis.PointerSweep] Pointer sweep found 0 pointer(s).

@feoff3 Could you please attach a file that can reproduce this? It would make it easier for us to reproduce it

I tested with a few spaceport.sys that I can find on my end, and they do not show the behavior.

I tried with spaceport.sys available in Windows 2016, Windows 2019 and
Windows 2022, and other drivers.
Might be a problem on my end then like a networking issue. Is there a way
to diagnose the PDB downloading process?
On Fri, 3 May 2024, 14:30 xusheng, @.***> wrote:

I tested with a few spaceport.sys that I can find on my end, and they do
not show the behavior.
—
Reply to this email directly, view it on GitHub
https://github.com/Vector35/binaryninja-api/issues/5357#issuecomment-2092595112,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABV5W5W2PYJWRIVY77KGQ2LZANGZLAVCNFSM6AAAAABHE7KC6KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOJSGU4TKMJRGI
.
You are receiving this because you were mentioned.Message ID:
@.***>

If you download it manually and put it in the same folder it should just work. We also support a bunch of standard environment variables for PDB caches. https://docs.binary.ninja/guide/index.html#loading-pdbs

Yes, it works when I download it manually.
It is not very convenient to be honest.
The auto download worked fine for me just a couple of weeks ago - I was
worried that some last update had broken the functionality
On Fri, 3 May 2024, 16:16 Jordan, @.***> wrote:

If you download it manually and put it in the same folder it should just
work. We also support a bunch of standard environment variables for PDB
caches. https://docs.binary.ninja/guide/index.html#loading-pdbs
—
Reply to this email directly, view it on GitHub
https://github.com/Vector35/binaryninja-api/issues/5357#issuecomment-2092756560,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABV5W5QBSDAWPMNPNP4BORDZANTKHAVCNFSM6AAAAABHE7KC6KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOJSG42TMNJWGA
.
You are receiving this because you were mentioned.Message ID:
@.***>

I have tried uninstalling the latest version, and installed version 3.5.4526
the result is similar - no symbols downloaded

Example of spaceport binary for which I have an issue
spaceport.zip

I tested and can reproduce this bug with the last stable release. The bug has been fixed in the dev branch a while ago. Please update to the latest development branch following this guide: https://docs.binary.ninja/guide/index.html#development-branch


Support simple transitivity on join predicate pushdown

For #3452
Current implement is very simply and conservative, because our query planner is error-prone.
After we implement the new query planner, we could do this work by Predicate Equivalence Class and PredicatePushDown rule like presto.

@kangkaisen ，Mysql has implemented an optimizer with Multiple Equivalence Predicate (MEP). For example, a
=b and b= c and c = 123  ===>>  =(a,b,c,123), which implement transitivity of predicates. Does this algorithm solve your problem? Add my WeChat: xinghuayu520 to have a disscussion.


Dear author,When I used the google-patents -Scraper

Dear author,When I used the google-patents -Scraper, I typed the terms and then clicked search, it was the program that flitted back and reported the error. May I ask the reason?

我使用的ssr作代理

Setting proxy isn't necessary, it was a bug.
I just fixed it and updated the repo, thx for your notification!
I didn't do lots of error handling, welcome to fork the repo and create a pull request, thanks!
ps. I saw your comment on youtube demo before this lol, just paste the same reply I made there so others who read this can know the progress.


Restart kubelets in a gke test

Forked off from disucssion on https://github.com/GoogleCloudPlatform/kubernetes/pull/11339. That restart test needs to ssh into the master to restart all the master components, but we need something similar that restarts just the kubelet on the nodes on gke (since master ssh isn't allowed on gke).
@mbforbes

Thank you for opening!
Yes, this would be great to minimize creeping GCE/GKE divergence.
To be clear, we're talking about test/e2e/daemon_restart.go.


helm dep up --skip-refresh crash for repos that weren't added prior to dep update

Repoening https://github.com/helm/helm/issues/11001
helm dep up --skip-refresh still fails for missing repos even though https://github.com/helm/helm/issues/11369 was resolved.
Steps to reproduce:
$ helm create test
$ cd test
$ cat << EOF >> Chart.yaml
dependencies:
- name: mysql
  version: "^8.0.29"
  repository: "https://charts.bitnami.com/bitnami/"
EOF

$ helm dep up --skip-refresh .
Error: no cached repository for helm-manager-18e255b69e00fd26e162cb50e07897f40240e1ed2aca7c1c49fd669a4f5157b2 found. (try 'helm repo update'): open ~/Library/Caches/helm/repository/helm-manager-18e255b69e00fd26e162cb50e07897f40240e1ed2aca7c1c49fd669a4f5157b2-index.yaml: no such file or directory

Output of helm version:
version.BuildInfo{Version:"v3.14.3", GitCommit:"f03cc04caaa8f6d7c3e67cf918929150cf6f3f12", GitTreeState:"clean", GoVersion:"go1.22.1"}

Similar to #12920 although the error message is different. I believe the presence of Chart.lock changes the behavior.