haskellPackages.hs-speedscope: unbreak

Motivation for this change
hs-speedscope is currently broken because of a too tight upper bound on ghc-events (< 0.16) but builds okay with the newer version of ghc-events currently present in nixpkgs (<IP_ADDRESS>). In fact, current upstream of hs-speedscope has already bumped the bound, but this has not been released on Hackage (and Stackage). I opened an issue about releasing.
nixpkgs-review runs out of disk space on my machine :-(
Things done

Built on platform(s)

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

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

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

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

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

[ ] Fits CONTRIBUTING.md.

Thanks!


Passport Multi-Auth

Looks like this package doesn't support multi-auth using Passport. I'm using Passport for the User API authentication but I'm unable to use it for Admin API authentication at the same time created using this package.

I will test the package with Passport soon. Hope next week i will provide working solution.

I tried  working with the passport package but it seems passport wasn't built with the intention of having multi auth support.

Lets continue in this topic: https://github.com/Hesto/multi-auth/issues/9


linuxsampler: svn-2340 -> 2.1.0

libgig: svn-2334 -> 4.1.0
libscp: svn-2319 -> 0.5.8
Motivation for this change
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg eval

Merged in #32484.


embassy_embedded_hal::SetConfig impl needs to be marked unstable

Fixed in https://github.com/esp-rs/esp-hal/pull/2668


Remove support for non ERB outcome templates

NOTE. This is based on #1894 so I'll want to make sure that's merged to master before this.
This removes the ability to render Outcomes using data stored in the YAML locale files.
This looks like a lot of commits but they're all really quite small; particularly the 35 commits that remove use_outcome_templates from each of the Smart Answer flows. I committed those separately so that I could commit the removal of that line and the change to the related checksum YAML file.

Looks good to me :shipit:

Awesome. Thanks chaps.


A search and sorting mechanism for resources (and goals and topics)

This is especially important for resources as the number will grow the fastest. There will be also be redundant resources created inevitably because it is hard to find an existing one.
The "your resources" section of /a/resource should be to:

Search for resources (either fully client side or fetch / async calls, no page reloading)
Categorize resources and open / close categories ( see: #34 )

The selection of resources for the topics should also use the same mechanisms for the available resources
Additional related issues should be opened to evaluation and likely do the same for topics and goals.

This issue is being updated to reflect the the changes brought by the cc / agora merger. And reassigned to that project.
This is going to be evaluated as part of the dashboard revamp (#75) and is also being considered for the topic selection part of #92 . (Will likely matter for resource selection for topics as well).

Closing as #235 is evaluated in working system


Highlight without filter

👋🏼,
I want to use this plugin as a way for me to highlight certain patterns for all buffers. This seems to be a bit tricky as filter is   required. I've tried using:
{
  filter = function()
    return true
  end,
  pattern = 'NOTE',
  hl =<EMAIL_ADDRESS>},

and:
{
  filter = {},
  pattern = 'NOTE',
  hl =<EMAIL_ADDRESS>},

Sadly neither of these work.
What's the best way to apply an highlight for all buffers? 🙏

Should be fixed now. Either config option should work as expected.


Update Discord Bot Instances

ngl i dont use python and this is untested, but copilot said it looks good so here ya go

lgtm!


Cannot find module 'vuesax/src/components/vsButton/vsButton'

I'm using with nuxt.js(^1.0.0) and it is not working.

Okay fine, now it is working.


Filter can not detect upper or lower case

Hi,
I am using Turkish alphabet. My dual-list have some texts like MİHRİCAN. I am writing "mi" to filter input. But it doesn't find anything.
Any solution for this problem?

Is your browser locale set to 'tr_TR' or something else? This is a problem with the dual-list using toLowerCase() for the filtering instead of toLocaleLowerCase().
If it is Turkey, then I change do a new release to use toLocaleLowerCase() and I believe it'll take care of it. If you're using a different locale, then the escape codes will need to be removed as well.
You could create a custom-dual-listbox and override the onFilter function to do something like this:
    private standardizeString(text:string) : string {
        return text.toLocaleLowerCase().normalize('NFD').replace(/[\u0300-\u036f]/g, '');
    }

    onFilter(source:BasicList) {
        if (source.picker.length > 0) {
            const filtered = source.list.filter( (item:any) => {
                if (Object.prototype.toString.call(item) === '[object Object]') {
                    if (item._name !== undefined) {
                        return this.standardizeString(item._name).indexOf(this.standardizeString(source.picker)) !== -1;
                    } else {
                        return this.standardizeString(JSON.stringify(item)).indexOf(this.standardizeString(source.picker)) !== -1;
                    }
                } else {
                    return this.standardizeString(item).indexOf(this.standardizeString(source.picker)) !== -1;
                }
            });
            source.sift = filtered;
            this.unpick(source);
        } else {
            source.sift = source.list;
        }
    }

But let me know if just switching to toLocaleLowerCase() solves your problem and if it does I'll get a new build out.
You can test it in the console:
console.log('MİHRİCAN'.toLocaleLowerCase().indexOf('mih'.toLocaleLowerCase()) !== -1);

My locale is 'en' so, it comes out 'false', but if I explicitly set it to 'tr', then the result is true:
console.log('MİHRİCAN'.toLocaleLowerCase('tr').indexOf('mih'.toLocaleLowerCase('tr')) !== -1);

console.log('MİHRİCAN'.toLocaleLowerCase().indexOf('mih'.toLocaleLowerCase()) !== -1);
When I try thisi it comes out false.
console.log('MİHRİCAN'.toLocaleLowerCase('tr').indexOf('mih'.toLocaleLowerCase('tr')) !== -1);
But when I try this, it is returning true.
I guess toLocaleLowerCase('tr') solves my problem.
Now what will I do? Where will I use this code :)

@emany : I'll get a new package with toLocaleLowerCase in place in the next 24 hrs.

@czeckd ok, thanks. So how can i update dual-listbox package?

Release 4.6.3 is now on npmjs.com, so just:
npm i --save angular-dual-listbox@latest

@czeckd I installed lastest version. But my problem is not resolved.
It is still not finding "mi" in "MİHRİCAN".

Well that's disappointing. That leaves you the option of making a custom dual-list component with an override of onFilter as I described earlier above that strips out the escape codes, or not using filtering, or not using the dual-list component.

I've run up to a TypeScript bug  trying to explicitly set the locale as toLocaleLowerCase('tr'). The function definition for toLocaleLowerCase is incomplete and thus won't transpile with arguments without modifying the lib.es5.d.ts definition for it. If I can work around it, then I could provide a way to programatically set the locale without needing to override the onFilter function in the dual-list component.

@czeckd Ok, Do I wait for you or use a custom-dual-list?

Depends how soon you need it working. If you need it today or soon, then go custom. If you can wait a few days, then I might have something.

@czeckd I can wait a few days. So I am waiting you. Thank you for your interest.

@emany Please try this again. I've update the<EMAIL_ADDRESS>with support for explicitly setting the locale.
npm i --save angular-dual-listbox@latest

Add to your dual-list a format object with the locale set to be 'tr_TR' or 'tr'.
Example:
<dual-list [source]="source" [(destination)]="confirmed" [format]="{ locale: 'tr' }" >
</dual-list>

With any luck, the filter on dual-list will support lowercase İ correctly.

@czeckd My problem solved. Thank you for everything.


Maintain video aspect ratio on resizing window or adding/removing new camera (camview)

Hi, thanks for good job and many nice changes  in this plugin. I would like to tell you that I'm using camera-ffmpeg plugin and camera-ui in same homebridge. Everything is ok and any problems for me. Of course cameras have different names. I would like to suggest some changes for you. Dimmensions of videos are not same all the time and changing in web browser and mobile phone . When I'm changing dimensions of my browser in Mac or in iPhone also right aspect ratio ( 16:9 or 4:3 ) are changing. It should be always the same and stable. Resizing the video image should not change the video proportionality, smaller or larger browser's window, more or less cameras must not influent to video aspect ratio…
Originally posted by @Terminator0007 in https://github.com/SeydX/homebridge-camera-ui/issues/121#issuecomment-833002995

This is one feature which we must have, the correct aspect ratio is distorted also if switched between iPhone and iPad.

🎉 A new version of camera.ui
A new version of camera.ui is currently under active development. An initial alpha/beta release and previews are coming soon. Stay tuned for exciting updates: https://github.com/seydx/camera.ui/issues/448 .
This version will no longer be developed / fixed. The new version contains many novelties and fixes (most important is for HKSV recording).


If "@" is added to this regexp, it resolves the issue which is also seen in replace-requires

https://github.com/bendrucker/has-require/blob/master/index.js#L19
If "@" is added to this regexp, it resolves the issue which is also seen in replace-requires
https://github.com/bendrucker/replace-requires/issues/3
var anyRegExp =<EMAIL_ADDRESS>

Good catch, should be fixed


Small error message changes

Small changes to permission related error messages

If we always use You cannot access these permissions. everywhere, translators will have to translate just 1 string.

Yeah, I agree, we should switch to the these everywhere, including the files in this pull request.


[read-fonts] Add 'get' method to coverage tables

I wanted this for testing the table splitting in Mark2Base, and it's something we will want anyway.
This also fixes a little mixup in the signature of CoverageTable::iter; the elided lifetime here meant the compiler tied the iterator to the lifetime of the table object, whereas we really want to tie it to the lifetime of the underlying data.
JMM

ooh, looks like we have a new rustc/clippy.
Also I'm wondering if this shouldn't return usize, since we expect coverage indices to be used mostly to index into collections?


boards: add support for integer board revisions

Add support for simple integer board revisions (1, 2, 3, ...).
Signed-off-by: Henrik Brix Andersen<EMAIL_ADDRESS>

I have seen that feature, but it does not produce the same result. @2 is just a shorthand for @2.0.0 which will be what BOARD_REVISION is set to. With the NUMBER format in this PR, BOARD_REVISION will be set to 2.

Good point.


i can't get the images to zoom

when i try to zoom my images they start zooming only to bounce back to their original size and double tap won't work i installed the example and tried it the first two images zooms perfectly  but the last one have the same issue as mine
here is my code
func openGallery(_ btn: UIButton){
        let gallery = SwiftPhotoGallery(delegate: self, dataSource: self)

        gallery.backgroundColor = UIColor.black
        gallery.pageIndicatorTintColor = UIColor.gray.withAlphaComponent(0.5)
        gallery.currentPageIndicatorTintColor = UIColor.white
        present(gallery, animated: false) {
            gallery.currentPage = btn.tag;
        }

    }

extension CarInfoVC: SwiftPhotoGalleryDataSource, SwiftPhotoGalleryDelegate {
    func numberOfImagesInGallery(gallery: SwiftPhotoGallery) -> Int {
        return imgArr.count
    }

    func imageInGallery(gallery: SwiftPhotoGallery, forIndex: Int) -> UIImage? {

        return UIImage(named: imgArr[forIndex])
    }

    // MARK: SwiftPhotoGalleryDelegate Methods

    func galleryDidTapToClose(gallery: SwiftPhotoGallery) {
        dismiss(animated: true, completion: nil)
    }
}

Hello. You need larger images. I left the 3rd image in the example small on purpose to demonstrate this behavior.

is there a way to solve this without using larger images as the images comes from a server and i can't control its size

Yes, but in doing so you'd be zooming an image past its native size where it would appear blurry. Or you could just use an older device with lower resolution.

Maybe we could expose a maximumZoomScale property that just passes it through to the scrollView? Or just make the scrollView public instead of private, so people could just set this value themselves. That would certainly allow consumers to manipulate the behavior without needing to fork and modify SwiftPhotoGallery themselves.

thanks a lot for your answer


Timx 288 marc field method refactor

Purpose and background context
Field method refactor for transform class MARC (Part 1).

Added private methods for key elements in MARC-formatted XML records (<leader>, <controlfield tag="008">(
Added field methods and corresponding unit tests for alternate_titles and call_numbers.

Note(s):

For the time-being, I've added calls to the private methods at the beginning of get_optional_fields(), which is required by field derivations (that will eventually be moved into their own field methods).
The name get_control_field_general_info was chosen over get_fixed_length_data as there are technically multiple fixed length fields in MARC. The fixed length field that MARC depends on is for code 008, which describes "general information" about the record.

How can a reviewer manually see the effects of these changes?

Run make test and verify all unit tests are passing.
Run CLI commandpipenv run transform -i tests/fixtures/marc/marc_record_all_fields.xml -o output/marc-transformed-records.json -s alma

Output:2024-07-22 13:23:04,806 INFO transmogrifier.cli.main(): Logger 'root' configured with level=INFO
2024-07-22 13:23:04,806 INFO transmogrifier.cli.main(): No Sentry DSN found, exceptions will not be sent to Sentry
2024-07-22 13:23:04,806 INFO transmogrifier.cli.main(): Running transform for source alma
2024-07-22 13:23:05,330 INFO transmogrifier.cli.main(): Completed transform, total records processed: 1, transformed records: 1, skipped records: 0, deleted records: 0
2024-07-22 13:23:05,330 INFO transmogrifier.cli.main(): Total time to complete transform: 0:00:00.524199

Includes new or updated dependencies?
NO
Changes expectations for external applications?
NO
What are the relevant tickets?
https://mitlibraries.atlassian.net/browse/TIMX-288
Developer

[ ] All new ENV is documented in README
[ ] All new ENV has been added to staging and production environments
[X] All related Jira tickets are linked in commit message(s)
[X] Stakeholder approval has been confirmed (or is not needed)

Code Reviewer(s)

[ ] The commit message is clear and follows our guidelines (not just this PR message)
[ ] There are appropriate tests covering any new functionality
[ ] The provided documentation is sufficient for understanding any new functionality introduced
[ ] Any manual tests have been performed and verified
[ ] New dependencies are appropriate or there were no changes

One question for both @jonavellecuerdo and @ehanson8: I can't comment directly in the PR, but wondering about moving these static file mappings to methods/properties on the MarcTransformer class?
https://github.com/MITLibraries/transmogrifier/pull/200/files#diff-58573f6e57860b560075cbe825c45aace3f2830bb51d47f8d8dbf4b1b0b5d5ebR14-R30
Seems like we'd want to avoid them reading the file each time they are used, but that could be solved with cached instances, or even just read and save on class init.

One question for both @jonavellecuerdo and @ehanson8: I can't comment directly in the PR, but wondering about moving these static file mappings to methods/properties on the MarcTransformer class?
https://github.com/MITLibraries/transmogrifier/pull/200/files#diff-58573f6e57860b560075cbe825c45aace3f2830bb51d47f8d8dbf4b1b0b5d5ebR14-R30
Seems like we'd want to avoid them reading the file each time they are used, but that could be solved with cached instances, or even just read and save on class init.

As @ghukill and I discussed, we opted to only make the following change: assign 'crosswalk' variables as attributes of Marc transform class for the following reasons:

Setting them as class attributes has the same effect as having them as "loose" global variables in the module in that they are instantiated only once when the Marc module is imported; whereas calling a class / property method for each one would be called for every transformed record.
The idea of having these variables set on the instance via a __post_attrs_init__ was also floated, but transmogrifier does not currently use attrs, this would be a heavier lift. We also spoke about having a custom __init__ method but realized that the base Transformer.__init__ method is "final" and can't be overridden.

Setting them as class attributes has the same effect as having them as "loose" global variables in the module in that they are instantiated only once when the Marc module is imported; whereas calling a class / property method for each one would be called for every transformed record.

Good summary and I agree that seems like the best path!

One question for both @jonavellecuerdo and @ehanson8: I can't comment directly in the PR, but wondering about moving these static file mappings to methods/properties on the MarcTransformer class?
https://github.com/MITLibraries/transmogrifier/pull/200/files#diff-58573f6e57860b560075cbe825c45aace3f2830bb51d47f8d8dbf4b1b0b5d5ebR14-R30
Seems like we'd want to avoid them reading the file each time they are used, but that could be solved with cached instances, or even just read and save on class init.

As @ghukill and I discussed, we opted to only make the following change: assign 'crosswalk' variables as attributes of Marc transform class for the following reasons:

Setting them as class attributes has the same effect as having them as "loose" global variables in the module in that they are instantiated only once when the Marc module is imported; whereas calling a class / property method for each one would be called for every transformed record.
The idea of having these variables set on the instance via a __post_attrs_init__ was also floated, but transmogrifier does not currently use attrs, this would be a heavier lift. We also spoke about having a custom __init__ method but realized that the base Transformer.__init__ method is "final" and can't be overridden.

Thanks for making this change.  Kind of minor, but a bit of nice code organization I think.  Also, while I was definitely using "loose" in our conversations, it occurred to me those kind of variables are probably most accurately described as "module level" variables.  Maybe that's a good term we can use going forward to have a collective understanding of what we're talking about.

Thanks for making this change. Kind of minor, but a bit of nice code organization I think. Also, while I was definitely using "loose" in our conversations, it occurred to me those kind of variables are probably most accurately described as "module level" variables. Maybe that's a good term we can use going forward to have a collective understanding of what we're talking about.

I like "module level" and am always in favor of standardizing the vocabulary for our coding!


GPR plugin fails to get any repos, but after disabling/enabling it works again - "No git repositories found."

Extension version: 0.11.0
VSCode Version: 1.38.0 (stable)
OS: MacOS 10.14.6

Steps to Reproduce:

Open Github Pull Requests Plugin
Plugin says "No git repositories found."
Open terminal, set output to "github pull request"
Shows [Debug 19752s 997ms] Looking for git repository
Disable Github Pull Requests plugin
Reload
Enable
Go back to plugin
It now works

There's nothing useful in the logs apart from  [Debug 19752s 997ms] Looking for git repository
Clicking the refresh icon does nothing, nor does restarting vscode.

Thanks for catching this! Pushed a fix and published a new version of the extension.

speedy response!
thank you


BUG: Not all packages (v.0.24.0) were published properly on npmjs registry

I'm submitting a bug report
Library Version:
latest
Operating System:
macOS 10.15.5 Catalina
Node Version:
14.15.0
NPM Version:
6.14.7
Webpack
Latest
Browser:
Chrome Latest
Language:
TypeScript 3.9.5
Seems likes as not all of the packages are published properly on npmjs.com:

@aurelia-ux/components
@aurelia-ux/positioning
@aurelia-ux/textarea
@aurelia-ux/switch
@aurelia-ux/radio
@aurelia-ux/select
@aurelia-ux/sidenav
@aurelia-ux/progress
@aurelia-ux/slider

It's despite the fact that versions of all those packages were updated in corresponding package.json files.

Republished all packages at 0.25.0.


Using Gluestack-style along with Gluestack-UI?

Discussed in https://github.com/gluestack/gluestack-style/discussions/383

Originally posted by Seanmclem September  2, 2023
So I was using Gluestack style/ui about a month ago. After a recent update it is totally different now, so I refactored my project over to just using the newer Gluestack-ui.
If I want to be able to use custom styes or new custom components made with Gluestack-style, along side usage of gluestack-UI, what would that look like? Which StyledProvider vs GluestackUIProvider would I use? One or both? what about the theme-config? Would there be 2, now?
For the ui-docs and the style-docs for installation. Neither seem to mention each other. What would that even look like?

Hey @Seanmclem, You don't need to use StyledProvider as you are already using GluestackUIProvider.  GluestackUIProvider uses StyledProvider internally.  So you can straightaway create your own components which will use the same config as gluestack-ui.

Thanks, I'll try that and come back if I have any questions or close it.

Hey @Seanmclem, Closing this issue. Feel free to open a discussion or create a issue if you find one.


broken code in master "{timeout,  {gen_server,call, [cqerl,  {get_client,..."

how to reproduce

clone repo
run make
try to use it in erlang shell

$ erl -pa ebin deps/*/ebin
Erlang/OTP 18 [erts-7.1] [source] [64-bit] [smp:8:8] [async-threads:10] [hipe] [kernel-poll:false]

Eshell V7.1  (abort with ^G)
1> application:start(crypto).
ok
2> application:start(asn1).
ok
3> application:start(public_key).
ok
4> application:start(ssl).
ok
5> application:start(pooler).
ok
6> application:start(cqerl).
{error,{not_started,re2}}
7> application:start(re2).
ok
8> application:start(cqerl).
== Starting CQErl frontend. ==
ok
9>
9>
9> cqerl:new_client({ "<IP_ADDRESS>", 9042}, [{auth, {cqerl_auth_plain_handler, [{"developer", "paaasss"} ]}}]).
** exception exit: {timeout,
                       {gen_server,call,
                           [cqerl,
                            {get_client,
                                {{10,1,40,112},9042},
                                [{auth,
                                     {cqerl_auth_plain_handler,[{"developer","paaasss"}]}}]}]}}
     in function  gen_server:call/2 (gen_server.erl, line 204)
10>

OS: debian 8.2
erlang version: 18.1 (from erlang-solution repo)

How come you provide "<IP_ADDRESS>" in the function call, and the trace says it's using {10, 1, 40, 112}?

sorry for my "obfuscation" ))). of course the same IP address should be here.

No worries :smile: So, is that a problem that just appeared with a new version of cqerl or is it the first time you try cqerl out?

this version
commit 3fa42be3fcf10e3791c5d02a5c58511670091758
Merge: 3e016dd da71f80
Author: Mathieu D'Amours<EMAIL_ADDRESS>Date:   Thu Oct 29 16:46:11 2015 -0400

works well

What version of cassandra are you using?

we have 2 nodes with 2.1.7 and 2.1.12

Ahh that's it, so if you want things to continue working for now you can use the v0.8.1 tag, since currently, the version in master only support v2.2.x and v3.x of Cassandra. We plan on making these changes backward compatible with v2.1 of Cassandra soon though

Tracking this in #63


Add a wrapper to properly test CTRL-C (SIGINT) on Windows

Coverage remained the same at 100.0% when pulling b794b9c2443971206413138077d8d18038d72bb7 on paescuj:sigint-test-windows into 2a1197b9ce6b56ed82643fc64cd4c7f8b16628e3 on open-cli-tools:main.


PostgreSQL schema filtering excluding all schemas

System information:

Windows 10 21H1 19043.1288
DBeaver <IP_ADDRESS>111011413

Connection specification:

PostgreSQL 10.18 on x86_64-pc-linux-gnu, compiled by gcc (GCC) 4.8.5 20150623 (Red Hat 4.8.5-44), 64-bit
org.postgresql.Driver
Direct connection

Problem:
Schema filtering excluding all schemas
Steps to reproduce:
1- Create a new connection.
2- Edit connection. Open "General" -> "Filters" -> "Schemas/Users"
3- Add "pg_temp_%", "%temp%" or "temp" to "Exclude" list. Like in https://dbeaver.com/docs/wiki/Configure-Filters/
4- Save and reconnect.
5- Explore in navigation panel.
Screenshots
With filter

Without filter

Logs
dbeaver-debug.log

Thank you for the report.

Fixed in DBeaver 21.3.0

verified


v3.1.0

Codecov Report

Merging #72 into master will decrease coverage by 0.59%.
The diff coverage is 97.61%.

@@            Coverage Diff            @@
##           master      #72     +/-   ##
=========================================
- Coverage   96.99%   96.39%   -0.6%
=========================================
  Files           4        4
  Lines         432      416     -16
  Branches       60       59      -1
=========================================
- Hits          419      401     -18
- Misses          3        4      +1
- Partials       10       11      +1

Impacted Files
Coverage Δ

src/lineage/resources.py
100% <100%> (ø)
:arrow_up:

src/lineage/visualization.py
97.5% <100%> (+0.06%)
:arrow_up:

src/lineage/__init__.py
94.46% <83.33%> (-0.82%)
:arrow_down:

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 295a3ae...d418200. Read the comment docs.


✨ Minimap

Overview
This view represents your code in a smaller form. The VSCode documentation says in best...

A Minimap (code outline) gives you a high-level overview of your source code, which is useful for quick navigation and code understanding. A file's minimap is shown on the right side of the editor. You can click or drag the shaded area to quickly jump to different sections of your file.

Tip: You can move the minimap to the left hand side or disable it completely by respectively setting "editor.minimap.side": "left" or "editor.minimap.enabled": false in your user or workspace settings.

Resources

CodeEditorView minimap source

Actually I'd prefer the minimap like Xcode rather than VScode. The minimap of Xcode contains more useful features and looks cleaner and much more beautiful. It will definitely take more work to make one like that, but I believe you'll love it too.

@Yellow-GGG I agree the Xcode minimap is far more useful; however at the same time I think it may be wiser to ship a minimap with minimal functionality and then add complexity in future releases.

Hey @austincondiff, could you assign this to me? I already got some of the CodeEditorView code working:

This is the current state of the minimap. Still a lot of bugs to fix, but I am getting there slowly...
https://user-images.githubusercontent.com/82230675/231145498-c3f3ad40-2a18-4034-9e3c-c883e0843a42.mov
@thecoolwinter I need to set the bounds of the visibleRange in STTextview+VisibleRange to the visibleBounds of the MinimapView. How should I rewrite the code, so that I can pass props to visibleTextRange variable?

Looks great! Where are we with this?

Unfortunately, I have had little time recently, since I am currently working on my degree.
I have decided that I am going re-implement the minimap in textkit2, instead of textkit1, since the resizing was very laggy. I have also encountered multiple issues like #184, which need to be resolved first, so that the minimap can work properly.


IDE: Provide autocomplete suggestions for local variables

Problem
# typed: true
def main
  an_var = 10
  another_var = 11
  # Typing `a` or `an` does not provide any autocomplete suggestions
end

Proposed solution
Include local variables in autocomplete suggestions.

Implemented in #2106


rename:jsonrpc

Rename 'jsonrpc' -> 'rpc'
Implement and test rpc methods: syncing, class, classAt, classHashAt, Events
GW client 'TransactionTrace' impl, stub for RPC client
'type ContractClass' -> 'program' changed to 'interface{}' so it can be used in multiple contexts

Agreed on the list. I think we may spin up something in github.com/starknet-edu or github.com/starkware-community-libs for some sort of integration checklist and sync for all the major libs/SDKs

Generally looks good to me! Imo I would prefer to have an explicit type for program but happy to roll with this for now


feature: add dashboard

feature: add dashboard
What does this PR do?
Adds the following:

A dashboard that lists all the current builds
View current running build container logs
Viewing the completed build log
Deleting one build
Deleting multiple builds / bulk deletion
Be able to click and open the build output directory

Screenshot / video of UI

What issues does this PR fix or reference?

Closes https://github.com/containers/podman-desktop-extension-bootc/issues/196
How to test this PR?

Start a build
View it in the dashboard
Delete the build
Try building another build..
View the logs (click on the log button)
Click on the folder button (should open in default file manager on
your system)

Signed-off-by: Charlie Drage<EMAIL_ADDRESS>

Requires https://github.com/containers/podman-desktop-extension-bootc/pull/192 to be merged (based upon that PR)

Ready for review 👍 @deboer-tim

Ready for another review!
https://github.com/containers/podman-desktop-extension-bootc/assets/6422176/ff791b8a-b464-4c6b-9013-066698f39269
Some notes:

Suggested changes implemented
Progress column now removed
Scope reduced (no more leftover api calls such as "navigate to container")
Keeping it nice and super simple


Mark event handlers as passive where possible

Event listeners that don't call preventDefault can be marked as passive to improve performance.
So, unless there is a good reason to call event.preventDefault, event listeners should be marked passive, particularly in performance critical sections of the code, such as scrolling.
Steps which explain the enhancement

Current calls to event.preventDefault in performance critical code should be evaluated to see if they are necessary
If they are not necessary, they should be removed
the event listener should be marked as passive by passing an object with a property of passive set to true, i.e. {passive: true}, as the third parameter to addEventListener

Current and suggested behavior
Currently searching for "passive" in the atom repo doesn't show any event listeners marked as passive, and searching issues for passive doesn't show this being suggested before.
I've been using this code in my init.js file for a little while with no obvious ill effects.
(function () {
    //remove event.preventDefault() calls in TextEditorConponent's onMouseWheel listener
    atom.workspace.observeTextEditors(function (editor) {
        var component = editor.editorElement.component

        component.domNode.removeEventListener('mousewheel', component.onMouseWheel)

        component.domNode.addEventListener('mousewheel', (function(_this) {
            return function(event) {
                var previousScrollLeft, previousScrollTop, updatedScrollLeft, updatedScrollTop, wheelDeltaX, wheelDeltaY;
                wheelDeltaX = event.wheelDeltaX, wheelDeltaY = event.wheelDeltaY;
                if (Math.abs(wheelDeltaX) > Math.abs(wheelDeltaY)) {
                    previousScrollLeft = _this.presenter.getScrollLeft();
                    updatedScrollLeft = previousScrollLeft - Math.round(wheelDeltaX * _this.editor.getScrollSensitivity() / 100);

                    return _this.presenter.setScrollLeft(updatedScrollLeft);
                } else {
                    _this.presenter.setMouseWheelScreenRow(_this.screenRowForNode(event.target));
                    previousScrollTop = _this.presenter.getScrollTop();
                    updatedScrollTop = previousScrollTop - Math.round(wheelDeltaY * _this.editor.getScrollSensitivity() / 100);

                    return _this.presenter.setScrollTop(updatedScrollTop);
                }
            };
        })(component), {passive: true})
    })

}).call(this);

It removes the mousewheel listener from the TextEditorComponent and adds another version which is almost the same except the conditional calls to event.preventDefault have been taken out, (and it has been converted from coffeescript to javascript), and marks it as passive. Since the calls to event.preventDefault don't seem to be doing anything important (at least on my system?) I think they should be removed and the event listener should be marked as passive.
Why would the enhancement be useful to most users
Giving the runtime more information with which to optimize could have performance improvements in many areas of the codebase. Focusing on scrolling performance alone, here are some previously opened issues related to scrolling: #10188 #10987 #13407
It's also easy to find people complaining about atom's scrolling speed elsewhere.
Screenshots and GIFs
One day scrolling in Atom lagged for a moment. So I opened up dev tools to see if I could collect enough info for a bug report, and the following errors were in the console.

The top two are familiar, and appear to be being worked on. The third one is what prompted me to research passive event listeners and to break on mousewheel listeners and investigate what happens when scrolling in the text editor component enough to write the code above. Also note you can make the third type of error message happen by setting a breakpoint an a non-passive mousewheel listener and trying to scroll the editor and then resuming execution.
Atom Version:
Atom    : 1.12.7
Electron: 1.3.13
Chrome  : 52.0.2743.82
Node    : 6.5.0
OS and Version:
Ubuntu 16.04 LTS

Another performance related issue when scrolling: https://github.com/atom/atom/issues/12949

Yes please.  This does make perceived performance increase a lot since the UI still responds when scrolling.

/cc: @nathansobo

I think that in Atom's case, this might not be helpful.
We don't actually use the browser's implementation of scrolling at all for the editor, because we only render the lines that are actually visible on screen to minimize layout costs when editing. Since the browser's scrolling is threaded and asynchronous, using the default browser scrolling implementation leads to situations where a region of the content div is scrolled into view before our scripts are notified via a scroll event. This can cause a visible delay between scrolling to an area and rendering its content, which people don't like.
To avoid this flicker, scrolling is implemented completely in JS. Each time you use the mouse wheel or drag a scrollbar or move the cursor, it is translated into an explicit update to our model that moves the scroll position. Passive event listeners are designed to avoid blocking the browser's native scrolling implementation with JavaScript, but we literally implement scrolling with JavaScript. That JavaScript just needs to run as fast as possible.
I'm open to further discussion if there's something I'm missing, though this issue has been around for a while so it may be too late.


adding PSD2 Support and Tests

Coverage increased (+0.01%) to 97.382% when pulling dc3c0cab8f9f3c9c8f9fb07f46c6f4e454a873ab on add-psd2-support into 727b01afe8ea42dcf3db2c08c6072b022090c5f7 on master.


Assembly version in Xaml/Baml resource is removed

I've been using ILRepack for combining some assemblies that have baml resources, and I have a need to be able to load two different versions of the same assembly. However I'm noticing that after running ILRepack on an assembly with baml resources the assembly versions are removed from the resource. I believe this causes the .NET runtime to load an incorrect assembly version in side-by-side scenarios throwing various exceptions (invalid cast or type not found).
Here is the decompiled baml in a non-ILRepacked assembly (via ILSpy)
<UserControl x:Class="MyLib.UserControl1" xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation" xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml" xmlns:uc="clr-namespace:MyLib;assembly=MyLib,Version=<IP_ADDRESS>,Culture=neutral,PublicKeyToken=e103492ae46a5022">
  <Grid>
    <uc:UserControl2 x:Name="ctrl" />
  </Grid>
</UserControl>

Here is the decompiled baml in an ILRepacked assembly:
<UserControl x:Class="MyLib.UserControl1" xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation" xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml" xmlns:uc="clr-namespace:MyLib;assembly=MyLib">
  <Grid>
    <uc:UserControl2 x:Name="ctrl" />
  </Grid>
</UserControl>

Note how the xml namespace uc has been modified. These types are all in the main assembly.

After examining this issue for a while, I have an idea on how to resolve this. From what I can tell, the modification is made in BamlResourcePatcher.ProcessRecord(AssemblyInfoRecord record):
        private void ProcessRecord(AssemblyInfoRecord record)
        {
            var assemblyName = new System.Reflection.AssemblyName(record.AssemblyFullName);

            var isMergedAssembly = _otherAssemblies.FirstOrDefault(
                asm => asm.Name.Name == assemblyName.Name || asm.Name.FullName == record.AssemblyFullName) != null;

            // we are interested in the main assembly in order to fix the signing information, when
            // we sign the repacked assembly
            var isMainAssembly = assemblyName.Name == _mainAssembly.Name.Name;

            if (isMergedAssembly || isMainAssembly)
            {
                record.AssemblyFullName = _mainAssembly.Name.Name;
            }
        }

Specifically, I think I can change record.AssemblyFullName = _mainAssembly.Name.Name to record.AssemblyFullName = _mainAssembly.FullName. I am hesitant to make this change without understanding why it is the way that it is right now. Was there a specific reason to remove the assembly version here?
I tried making this change and get one failing test, which I haven't had a chance to dig into yet:

Test Name:	GivenSampleApplicationWithMahAppsAndSystemWindowsInteractivityWPF_MergedWPFApplicationRunsSuccessfully
Test FullName:	ILRepack.IntegrationTests.Scenarios.GivenSampleApplicationWithMahAppsAndSystemWindowsInteractivityWPF_MergedWPFApplicationRunsSuccessfully
Test Source:	C:\source\il-repack\ILRepack.IntegrationTests\Scenarios.cs : line 41
Test Outcome:	Failed
Test Duration:	0:00:00.149
Result StackTrace:
at ILRepack.IntegrationTests.Scenarios.RunScenario(String scenarioName) in C:\source\il-repack\ILRepack.IntegrationTests\Scenarios.cs:line 69
at ILRepack.IntegrationTests.Scenarios.GivenSampleApplicationWithMahAppsAndSystemWindowsInteractivityWPF_MergedWPFApplicationRunsSuccessfully() in C:\source\il-repack\ILRepack.IntegrationTests\Scenarios.cs:line 41
Result Message:
Process exited with error
Expected: 0
But was:  1
Result StandardOutput:
Scenario 'WPFSampleApplication' STDOUT: System.Windows.Markup.XamlParseException: Set property 'System.Windows.ResourceDictionary.DeferrableContent' threw an exception. ---> System.IO.FileLoadException: Could not load file or assembly 'WPFSampleApplication, PublicKeyToken=null' or one of its dependencies. The located assembly's manifest definition does not match the assembly reference. (Exception from HRESULT: 0x80131040)
at System.Reflection.RuntimeAssembly._nLoad(AssemblyName fileName, String codeBase, Evidence assemblySecurity, RuntimeAssembly locationHint, StackCrawlMark& stackMark, IntPtr pPrivHostBinder, Boolean throwOnFileNotFound, Boolean forIntrospection, Boolean suppressSecurityChecks)
at System.Reflection.RuntimeAssembly.InternalLoadAssemblyName(AssemblyName assemblyRef, Evidence assemblySecurity, RuntimeAssembly reqAssembly, StackCrawlMark& stackMark, IntPtr pPrivHostBinder, Boolean throwOnFileNotFound, Boolean forIntrospection, Boolean suppressSecurityChecks)
at System.Reflection.Assembly.Load(AssemblyName assemblyRef)
at System.Windows.Baml2006.Baml2006SchemaContext.ResolveAssembly(BamlAssembly bamlAssembly)
at System.Windows.Baml2006.Baml2006SchemaContext.ResolveBamlTypeToType(BamlType bamlType)
at System.Windows.Baml2006.Baml2006SchemaContext.ResolveBamlType(BamlType bamlType, Int16 typeId)
at System.Windows.Baml2006.Baml2006SchemaContext.GetXamlType(Int16 typeId)
at System.Windows.Baml2006.Baml2006Reader.Process_OptimizedStaticResource()
at System.Windows.Baml2006.Baml2006Reader.ReadKeys()
at System.Windows.ResourceDictionary.SetDeferrableContent(DeferrableContent deferrableContent)
at System.Windows.Baml2006.WpfSharedBamlSchemaContext.<>c.<Create_BamlProperty_ResourceDictionary_DeferrableContent>b__297_0(Object target, Object value)
at MS.Internal.Xaml.Runtime.ClrObjectRuntime.SetValue(Object inst, XamlMember property, Object value)
--- End of inner exception stack trace ---
at System.Windows.Markup.XamlReader.RewrapException(Exception e, IXamlLineInfo lineInfo, Uri baseUri)
at System.Windows.Markup.WpfXamlLoader.Load(XamlReader xamlReader, IXamlObjectWriterFactory writerFactory, Boolean skipJournaledProperties, Object rootObject, XamlObjectWriterSettings settings, Uri baseUri)
at System.Windows.Markup.WpfXamlLoader.LoadBaml(XamlReader xamlReader, Boolean skipJournaledProperties, Object rootObject, XamlAccessLevel accessLevel, Uri baseUri)
at System.Windows.Markup.XamlReader.LoadBaml(Stream stream, ParserContext parserContext, Object parent, Boolean closeStream)
at WPFSampleApplication.Program.Main() in C:\source\il-repack\ILRepack.IntegrationTests\Scenarios\WPFSampleApplication\Program.cs:line 13

I understand this only affects a narrow set of cases like side-by-side loading that I am trying to do, but if it's useful I can create a pull request for the change.

@mzboray How are you loading multiple versions of the same assembly? AFAIK this scenario is not supported in .NET (i.e., it will use latest version)
But regarding the fix itself, I think I used the simple name without a specific reason. If the assembly is not signed, you won't have the assembly version in the namespace. I just tried making ClassLibrary signed (the one from the integration tests suite in the project), but it still won't generate a reference with version, which is weird.
I think we can use the full version, but since it will be merged, in the end we'll need to represent the own binary in which we merge it.

@timotei Loading multiple versions of the same assembly is definitely supported in .NET in general. It requires that you strong name all assemblies which we are doing. In WPF/XAML, in particular, you need to include assembly version information the generated code which is not done normally, but can be forced using the AssemblyVersion property in the csproj file described here.
Ok it's good to know that there was not a specific known reason to only use the simple name. After some experimentation, I found that said test does not fail if I use just the assembly name and version. I'm not sure why the full name causes a failure.
My initial concern was that this seemed like a bug only because resources that do not need to be modified are in fact modified. In my case, I am only merging in some helper classes from another dll. No Baml resources should need modification.

@mzboray I tried reproducing this on the scenarios integration test: https://github.com/gluck/il-repack/pull/209
However, it doesn't seem to work as expected, and I have done this:

sign ClassLibrary
sign LibraryClassUsageInXaml
use the classlibrary xaml
I get this:

  <Window.Resources>
    <dc:DummyConverter x:Key="DummyConverter" />
  </Window.Resources>
  <TextBlock Text="{Binding Converter={StaticResource DummyConverter}}" />
</Window>

@timotei I believe this case is slightly different. In my case I have 3 assemblies:

Host application. (I don't control this, and if you can inspect the raw xaml you don't actually need this to demonstrate the issue.)
A library that has some xaml. All the xaml actually is within this assembly itself. We have set the AssemblyVersion property in the csproj file (see here). This is strong name signed.
A class library that contains no xaml. It is signed but will be merged completely into assembly 2.

My issue comes from the fact that after merging assembly 2 and 3. The version number information in assembly 2's xaml resources referring to itself are no longer there.
If you still can't repro, perhaps tomorrow I can create a repo with a demostration project I used for basic testing before deploying it on our application.

@mzboray I'd like to get a repro project, thanks!

@timotei See my repo with the demo code: https://github.com/mzboray/DemoWpfSideBySide

Is this now resolved or still a problem? I am having issues with missing resources


chore: fixed helm install errors

As title

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


Device rotation causes invalid size issue

This question is similar to #1095, however the answer doesn't fix my problem. When I rotate my device from portrait to landscape, the width of the cell inside my collection view stays the same width as it was in portrait, although the height changes correctly.
For example, suppose I have some text that looks like this when my device is in portrait mode:
Lorem ipsum dolor sit amet, consectetur adipiscing elit. Nam
accumsan neque sit amet velit efficitur, venenatis

When I turn the phone into landscape mode the whole text should fit on one line but instead it looks like this:
Lorem ipsum dolor sit amet, consectetur adipiscing elit. Nam

It cuts off the rest of the text that should be on that line. To fix this I tried calling performUpdates within viewWillTransition(from:to:) in my view controller like this:
override func viewWillTransition(to size: CGSize, with coordinator: UIViewControllerTransitionCoordinator) {
    super.viewWillTransition(to: size, with: coordinator)
    adapter.performUpdates(animated: true)
}

.. but it doesn't work and gives the error:
UICollectionViewFlowLayout instance is <MyApp.UICollectionViewFlowLayout: 0x10bdcf0e0>, and it is
attached to <UICollectionView: 0x106906600; frame = (0 401; 265 200); transform = [1, 0, 0, -1, 0, 0];
clipsToBounds = YES; autoresize = RM+BM; gestureRecognizers = <NSArray: 0x10bdce880>;
animations = { bounds.origin=<CABasicAnimation: 0x10c9647c0>; bounds.size=<CABasicAnimation:
0x10c9648b0>; position=<CABasicAnimation: 0x10c5b82b0>; }; layer = <CALayer: 0x10d7205e0>;
contentOffset: {0, 0}; contentSize: {557, 49}; adjustedContentInset: {0, 0, 0, 0}> collection view
layout: <MyApp.UICollectionViewFlowLayout: 0x10bdcf0e0>.
2018-06-22 15:02:31.625103+0100 QJAM[15682:3401723] Make a symbolic breakpoint at
UICollectionViewFlowLayoutBreakForInvalidSizes to catch this in the debugger.

Any subsequent cells that get added to the collection view when in landscape mode do size correctly though. How can I fix this?

I think you need to invalidate the layout in viewWillTransition instead of performing items update
E.g. ```
invalidateLayout(with: invalidationContext(forBoundsChange: .zero))

@weyert is right. This isn’t anything to do with IGListKit either, just a collection view thing.
Sent with GitHawk


makefile / k3s

https://blog.nootch.net/post/kubernetes-at-home-with-k3s/

https://github.com/Logitech/slimserver-platforms/tree/public/8.3/Docker

RPi Zero not supported by k3s!

here document https://www.pietrorea.com/2022/01/28/reclaiming-the-lost-art-of-linux-server-administration

infrastructure managed by docker-compse in open-smart-home


update API calls to use Authorization headers

Fixes to make slackin work in 2023 :)

Slack has deprecated passing tokens as parameters and now requires them to be passed in Authorization headers.
Pin dockerfile to node 10 which is the last version that works with this version of babel
Set default user poll interval to 60 secs, otherwise we're getting rate limited

@apoland you are awesome. Thank you!


Update run_perf_builder.py to run 'go test' commands sequentially

This changes effectively:

go test ./go/types/perf ./samples/csv/csv-import

to:

go test ./go/types/perf
go test ./samples/csv/csv-import

The perf results started going crazy when I started running a second
test package (./go/types/perf), this is one reason why it may have
happened. It's hard to say for sure what go test is doing.

PTAL. I increased test timeout as well, because, I can.

jenkins: perf this please

lgtm. interested to see the results from the tests though.

we'll be waiting a while. but no rush.

Current coverage is 87.76% (diff: 100%)

No coverage report found for master at 96cc9ff.

Powered by Codecov. Last update 96cc9ff...8e8c201


2023.03.05 & brightness

Hi,
After updating to 2023.03.05, brightness service and this code to show and adjust brightness are no longer working: no matter what interger value you user with "brightness" service, or how much you move the brightness slider, return brightness value is always 0.
But real brightness is not 0 as figures and icons are visible in the screen.
Please advise.
`number:

platform: template
name: "$devicename Brightness"
id: brightness_value
min_value: 0
max_value: 255
step: 1
lambda: |-
return id(rgb8x32)->get_brightness();
set_action:
lambda: |-
id(rgb8x32)->set_brightness(x);`

There was an error in my code. Its fixed in the 8x32 branch, wich will be released soon.


mark up of sequence

When showing sequences of genes the data could be enriched and show different features like 5UTR/3UTR, Stop Codon, etc.

I though this was odd, becasue we do? At least on the gene pages?

Yep, that one is a mystery.

No idea what this feedback referred to....


support target.json files

Fixes https://github.com/RalfJung/rustc-build-sysroot/issues/4

Yay I'm witnessing this in real-time :)


Update auto-merge-readme.yml

@sanjay-kv I have made some changes so as to prevent workflow from failing, please check and merge this PR

@sanjay-kv Actually the labels were assigned for wrong PR (#48 ) so removed them from there and added those over here....


version `CXXABI_1.3.9' not found

Hi,
Python=2.7
Ubuntu 14.04
I was following the instructions in read me file and got this errors
coesip@coesip-HP-Z230-SFF-Workstation:~$ cd Desktop/DF-Net-master/
coesip@coesip-HP-Z230-SFF-Workstation:~/Desktop/DF-Net-master$ python train_df.py --dataset_dir=/home/coesip/Desktop/DF-Net-master/dataset --checkpoint_dir=/home/coesip/Desktop/DF-Net-master
Traceback (most recent call last):
File "train_df.py", line 6, in
from core import DFLearner
File "/home/coesip/Desktop/DF-Net-master/core/init.py", line 2, in
from .DFLearner import DFLearner
File "/home/coesip/Desktop/DF-Net-master/core/DFLearner.py", line 9, in
from .BaseLearner import BaseLearner
File "/home/coesip/Desktop/DF-Net-master/core/BaseLearner.py", line 10, in
from .nets import *
File "/home/coesip/Desktop/DF-Net-master/core/nets.py", line 7, in
from .utils import flow_inverse_warp
File "/home/coesip/Desktop/DF-Net-master/core/utils.py", line 2, in
import matplotlib.pyplot as plt
File "/home/coesip/anaconda2/lib/python2.7/site-packages/matplotlib/pyplot.py", line 31, in
import matplotlib.colorbar
File "/home/coesip/anaconda2/lib/python2.7/site-packages/matplotlib/colorbar.py", line 32, in
import matplotlib.artist as martist
File "/home/coesip/anaconda2/lib/python2.7/site-packages/matplotlib/artist.py", line 16, in
from .path import Path
File "/home/coesip/anaconda2/lib/python2.7/site-packages/matplotlib/path.py", line 21, in
from . import _path, rcParams
ImportError: /usr/lib/x86_64-linux-gnu/libstdc++.so.6: version `CXXABI_1.3.9' not found (required by /home/coesip/anaconda2/lib/python2.7/site-packages/matplotlib/_path.so)
How to solve this?

Seems to be something related to matplotlib, but not this project.


Markdown tables don't render after using this package

For some reason, tables do not get build when I use the extension set given in the document

@xushengs  any updates on this?

@JAZ-CO  add ...md.ExtensionSet.gitHubFlavored.blockSyntaxes in ExtensionSet
extensionSet: md.ExtensionSet(
[
LatexBlockSyntax(),
...md.ExtensionSet.gitHubFlavored.blockSyntaxes,
],
[
LatexInlineSyntax(),
],
),


Layer order control issue with captureZIndex and arcGIS layers

I have an issue where the measurement controller always goes behind the arcGIS layers.
I have added the Measure tool like this:
var measureControl = new L.Control.Measure({position: 'topleft', captureZIndex: 10000 });
measureControl.addTo(map);

Then when I add esri arcGIS layers w/:
node.data.layer = L.esri.tiledMapLayer({
  url: node.data.url,
  pane: 'custom',
  useCors: false,
  zIndex:1
});

map.addLayer(node.data.layer);

my issue is that the measurement tool shows behind the arcGIS layers instead of being on top even though the zIndex is higher
Check it out here https://www.nwfsc.noaa.gov/data/
I thought perhaps this was a esri leaflet issue but it seems like it might be related to the "pane" used by this plugin. Any thoughts?

I have the same problem using QGIS. Got a solution?
https://github.com/ljagis/leaflet-measure/issues/141


Add a way to force refreshing the archive index

This will allow users to go to https://unbox.ifarchive.org/?refresh_index=1 to refresh the archive index, which, I believe, is the reason why we're getting some spurious 404 errors on unbox today.

I think merging this would be useful in order to address https://github.com/iftechfoundation/ifarchive-admintool/issues/37

The impact of having this option would be low - it would still do HEADER requests so it's not going to be forced to download a big file every time.
But I agree with Zarf to let things sit for a while and see how the manual process works.


Added PrePermission for camera

Closing this after this pull request was essentially the inspiration for this: https://github.com/clusterinc/ClusterPrePermissions/pull/13


github release plugin

addresses #439. Have done some manual testing over here: https://github.com/rgarcia/test/blob/master/.drone.yml e.g. https://github.com/rgarcia/test/releases/tag/0.3.7

@rgarcia just a note - this works well for duplicates if the release is not a draft. However, if the draft flag is set it will error out when a dup is found. Just fyi.


Login with OTP support added (#71) - fixed the registration edge case

Description
Supporting previous PR !71 by handling the edge case when the registration doesn't exist - creates registration for existing user

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 13 (0.0%)  changed or added relevant lines in 2 files are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.2%) to 24.618%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/api/fusionauth/fusionauth.service.ts
0
5
0.0%

src/api/api.service.ts
0
8
0.0%

Files with Coverage Reduction
New Missed Lines
%

src/api/api.service.ts
2
5.48%

Totals

Change from base Build<PHONE_NUMBER>:
-0.2%

Covered Lines:
569

Relevant Lines:
2104

💛  - Coveralls


Coda Daemon crashed

First, I entered command number 1, then number 2, and after the command number 3, an error occurred.

coda_crash_report_2020-11-19_10-05-12.129069.tar.gz

Hey @Claudi6020 , this error is occurring because you're already running mina somewhere in the background. Having said that we can better (1) update the docs and (2) make the error message clearer

Closing in favor of #6770


Ccp 631 fav list add to cart portals

Pull Request Test Coverage Report for Build 1321

1 of 2 (50.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.008%) to 35.319%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pages/Favorites/components/FavoritesList/components/Item/components/CTAButtons/index.jsx
1
2
50.0%

Totals

Change from base Build 1320:
-0.008%

Covered Lines:
1234

Relevant Lines:
3409

💛  - Coveralls


Rework testing to use common infrastructure

This PR implements several Pytest fixtures that make it easier to write and manage tests. Once CI passes, I'll start writing real documentation here.
The gist is that you should now add any of the following arguments to your tests, as needed:

golden

If you want to compare some string output, this will automagically be the string associated with the current test.
Pass --update-golden to Pytest to replace all the golden outputs.

compiler

If you need to compile and either load a DLL or run a test executable, this will handle it.

sde64

If you want to run something the compiler returned in the SDE emulator, this is how you do it.

You can now also mark a test as requiring a specific ISA to function. Use @pytest.mark.isa('AVX512f') to indicate that the test requires AVX512. The full list of features (along with their detection status) can be printed like so:
python -c 'import cpufeature, pprint; pprint.pprint(cpufeature.CPUFeature)'

This PR uncovered and fixed these bugs:

The UAST pretty printer wasn't using its ChainMap environments correctly. We had a similar bug in the LoopIR pretty printer a while ago, but it wasn't fixed in UAST.
The extract_method scheduling directive produced new methods with undefined argument ordering. It now sorts the arguments by name.
Coverage data was being collected about @proc, @instr, and @config decorated functions and classes.

This PR also enables AMX testing via SDE on GitHub Actions. We should be able to re-enable macOS testing in a follow-up PR.
Still have to rework the gemmini tests, since I don't have chipyard set up anymore and they were never run on CI.

I am a bit confused. Most of the test changes seem to be just formatting change, and they don't seem to be using the new golden or compiler infrastructure?

I am a bit confused. Most of the test changes seem to be just formatting change, and they don't seem to be using the new golden or compiler infrastructure?

I don't know what to say, but to look again? Every single test outside of tests/gemmini was converted.

It might be useful to walk through the commits and look at the ones that explicitly add golden outputs for tests. I let my editor run its formatting before making functional changes to files so they could be evaluated separately.

Most of the test changes seem to be just formatting change

Also, this is another major reason to deal with #59 since diffs between two files that have been autoformatted with the same tool and settings include only functional changes.

Could you please not run your editor's autoformatting, especially when it contains a functional change to many files? I recall @gilbo was not in favor of autoformatting the code (from the discussion in November), and IIRC, that discussion of whether we should use any autoformatting was still unfinished. I'm personally totally fine with running the autoformatting to all the files in the separate PR, but the issue of code formatting is not related to the main change in this PR, I think.

It's way too late for that, unfortunately. It would cost me several hours of work to undo that at this point.

>               raise CalledProcessError(retcode, process.args,
                                         output=stdout, stderr=stderr)
E               subprocess.CalledProcessError: Command '['ctest', '-C', 'Release', '--build-and-test', '/tmp/pytest-of-yuka/pytest-2/test_winograd0', '/tmp/pytest-of-yuka/pytest-2/test_winograd0/build', '--build-generator', 'Ninja', '--build-options']' returned non-zero exit status 1.

/usr/lib/python3.9/subprocess.py:528: CalledProcessError

^ This is the error I got from my environment

@yamaguchi1024 - can you run:
$ cmake --version
$ ctest --version

If it's below 3.21, you will need to upgrade.

Yeah it is 3.16, will upgrade later

@alexreinking upgraded them and both cmake and ctest versions are 3.22.2, but they still give the same error. Do you know what could be an other issue?

Can you try manually running the failing command and posting the full output? e.g.
ctest -C Release --build-and-test /tmp/pytest-of-yuka/pytest-2/test_winograd0 /tmp/pytest-of-yuka/pytest-2/test_winograd0/build --build-generator Ninja --build-options

Ok I just had to install ninja. Does README or requirements.txt document these requirements?

You can technically override it by running export CMAKE_GENERATOR="Unix Makefiles" but you're right that this should all be documented.

@yamaguchi1024 - I have added documentation to the README. Since all it took was installing the new dependencies and because it only affects the test suite, let's get this merged soon. I would like to start writing new tests (e.g. for the lift_if bug I found) using these helpers ASAP.

@alexreinking I'm good with these changes, but haven't we talked on Friday that we should ping users and check that it works on their machine before merging?

Also pinging @gilbo , you said that you wanted to go through the changes but is that done?

I appreciate the reasoning for ringing the alarm bells, but I don't actually think it will be very useful to delay the PR. That's because there are a number of mitigating factors...

This does not introduce any breaking changes to the main SYSTL library, so no one's code needs to be updated.
This will also not break any additional tests that users have added. They will see failures in the other tests, but it will not block their (private) progress.

They shouldn't be developing inside of tests/ anyway. They should be building and installing our package into a virtual environment and writing code like what's in apps/.

The steps to get to a sane build environment on CI are completely standard for any mixed Python and C project. CI also shows things are working at least as well as they have been on both Linux and macOS.
This is a big change and we don't want it to go stale.


Send message is pending forever

Since the last update of Zulip, I have problems sending messages. The messages are pending forever. Logout and Login help only for a short amount of time. Firefox and Chrome are affected. No other user has that problem.
In the development tools are an error and a warning.

I have not found any errors in zulips logs. I use the docker image from Dockerhub. https://hub.docker.com/r/zulip/docker-zulip
Current Zulip version: 2.1.4
Current Docker image version: 2.1.4-0

I'm guessing the root problem is something unrelated to the upgrade.  It looks like you are receiving presence data about a User ID 1 who no longer exists (and I would expect to be a system bot like Notification Bot).  Did someone hand-delete users from the database on your system?  Someone having done that with Notification Bot would explain this failure mode.

Thanks for the help. I am not aware that any users have been manually deleted from the database. I found a user with the ID 1 in the database. It is the feedback bot. The feedback button is still activated, but is no longer displayed in the UI. I try to disable the feedback function and report back to you.

Interesting!  I had not been aware that feedback-bot had been used anywhere but zulipchat.com since it was never documented as a feature.
You can likely address this for now using `
CROSS_REALM_BOT_EMAILS = [
<EMAIL_ADDRESS>
<EMAIL_ADDRESS>
<EMAIL_ADDRESS>
<EMAIL_ADDRESS>]

in /etc/zulip/settings.py (You may need to change the emails if your user ID 1 had a different fake email address).
I expect that https://github.com/zulip/zulip/pull/13791 will end up fixing this issue for you in the 2.2 release.

I just merged #13791, so upgrade-zulip-from-git master is also a possible solution to your problem if you don't want to do the hack I suggested above or wait for the next major release.
Thanks for the report @nicok1!


allow tagging

Interesting.
I largely coded this library blindly from the api, so am not quite familar with "tags".
Could you give me some more insight into how you use them?
In particular, I think its weird in the api that the tag is on the whole post, rather than for an individual string.
In your patch you've done it as an optional argument;
input_strings(["some string"], "Some tag")

I wonder if its better doing as part of an options hash;
 input_strings(["some string"], :tag => "Some tag")

But, by all means, lets add this option.
I'd also suggest not hardcoding the "Default" default.
As I see the API leaves it as optional, so lets keep it optional in our library too.
(therefore we leave it up to the server to decide what the default tag should be)

@matthewrudy we have rather peculiar requirements for handling our translations, and we started with gettext quite a while ago. So we aren't using i18n-one_sky, but I wrote a small rake task to combine the gettext string extraction and one_sky upload (actually pretty easy) combined with tagging. Hope this makes any sense?

Interesting,
I know there was the one_sky gettext gem, but it's been abandoned for the moment.

I'm going to merge your commit,
but use the "tag" as a key in the options.
I think that's more versatile, and future proof.
Especially as i'm still not sure the use of tags.
Hope that's ok.

Thanks, that's great as well!


add fltk-form example

quick fl2rust + fltk-form example
The only thing I am unsure about is the "dependencies" line I added in the Cargo.toml  is that the right way to add them?

Thanks for the PR. The dependency to fltk-form and fltk-form-derive should be added in the [dev-dependencies] since, like fltk, they're only used in the examples.

Update the README.md more when you have the link to the source code, and add any information you want.  I put the fltk-form section at the end of the README, feel free to change it!

Looks good 👍

Thanks!  I might create more to show a more detailed emit(), or how to access ui elements in more detail


make orderId optional in getAvailableItems to fix E_BILLING_RESPONSE_JSON_PARSE_ERROR

.getAvailablePurchases() throws E_BILLING_RESPONSE_JSON_PARSE_ERROR if user purchases an item with promo code. After looking into the details, it appears that items redeemed using promo code are associated with any orderId. Therefore, there's no orderId field in the json response.
Making orderId optional solves the problem.

Thanks for the PR. I'll merge this and will publish to a newer version since you faced the problem.


Campfire adapter hanging

I have created a new Campfire account for hubot, logged in with,I updated the 3 env variables (ubuntu box, no heroku no redis, I am testing)
use the API key of that account, room id and my account sub-domain.
Also, I changed the PORT to 9999 as I was getting the ERADDINUSE
When I issue the following command
bin/hubot -a campfire
I have a blinking command line and nothing is displayed. I switch to my room in campfire I can see that hubot user has joined the room
Can you you please let me know what I need to do configure more in order to make it work, 2nd, is there any manual on how to interact from hubot commandline with campfire
Thanks

Hi @eliassal, does it work from the command line if you start it up without the -a campfire option? What other scripts or npm modules are you loading?

Yes, when running this
:~/myhubot$ bin/hubot
I get (which is the name I chose)
salamnet>
Some commands respond correctly others not, for example, when I run
salamnet time
I get
Server time is: Fri Jun 05 2015 12:58:54 GMT+0200 (CEST)
but when I run
salamnet convert me 56MB KB
nothing happens

@eliassal glad to hear that it works from the command line!

is there any manual on how to interact from hubot commandline with campfire

When you start it up with the campfire adapter, it will look like it's just hanging, but hubot is expecting to interact with you in campfire. So I don't think you can use both the campfire adapter and the shell adapter at the same time.
When you start it up with -a campfire and it joins the campfire room, what happens if you type salamnet time in campfire? Does it respond?

but when I run
salamnet convert me 56MB KB
nothing happens

I'm guessing you saw the convert me command in docs on https://hubot.github.com/docs/? It looks like those are out of date and include plugins that are no longer bundled with hubot. I'll open an issue to get it updated.
You can see what your own hubot is capable of by running salamnet help:
hubot> hubot help
hubot adapter - Reply with the adapter
hubot animate me <query> - The same thing as `image me`, except adds a few parameters to try to return an animated GIF instead.
hubot echo <text> - Reply back with <text>
hubot help - Displays all of the help commands that hubot knows about.
hubot help <query> - Displays all help commands that match <query>.
hubot image me <query> - The Original. Queries Google Images for <query> and returns a random top result.
hubot map me <query> - Returns a map view of the area returned by `query`.
hubot mustache me <query> - Searches Google Images for the specified query and mustaches it.
hubot mustache me <url> - Adds a mustache to the specified URL.
hubot ping - Reply with pong
hubot pronounce <phrase> in <language> - Provides pronounciation of <phrase> (<language> is optional)
hubot pug bomb N - get N pugs
hubot pug me - Receive a pug
hubot the rules - Make sure hubot still knows the rules.
hubot time - Reply with current time
hubot translate me <phrase> - Searches for a translation for the <phrase> and then prints that bad boy out.
hubot translate me from <source> into <target> <phrase> - Translates <phrase> from <source> into <target>. Both <source> and <target> are optional
hubot youtube me <query> - Searches YouTube for the query and returns the video embed link.
ship it - Display a motivation squirrel

Here's an updated calculator script that you can install to get the convert command.

I started with  -a campfire  , issued the following commands but nothing is returned back, I get only a blank line in campfire
salamnet ping
salamnet pug me
except for salamnet time, i get some feedback, 11:10 PM.......etc
I have setup the updated calculator, running in the command line (without -a campfire)
salamnet convert me 56MB KB
gives an error as follows
salamnet convert me 56MB KB
but
salamnet convert 34 L in gal
it works fine.
I really appreciate your help, but still not being able to understand the big picture behind HUBOT, is there any article I can read to understand what business function it satisfys


Cannot add multiple Files + Folders at once? (setRunInThread issue)

There's my file tree that I want to compress:
+-- file1.txt
+-- folder1
|   +-- file2.txt
|   +-- file3.txt

So I'm adding file1.txt and folder1 in list, and using ZipFile.addFiles method to create an archive. But it creates an empty folders even if there were some files inside:
+-- file1.txt
+-- folder1 (empty)

If I add files and folders manually (using ZipFile.addFile and ZipFile.addFolder) it works, but since I use setRunInThread I'm stuck with ProgressMonitor.State.BUSY when adding next file.
(Possible duplicate of #84 ???)

Yes, this is a duplicate of one of #84. I will implement this feature in the next days. But I will close this as duplicate, and will update #84.


implement missing filters, basic sorting and pagination

Implemented missing pieces for Elastic Search integration based on MongoDb structure
Filter operations:

Range operations (gt, ngt, gte, ngte lt, nlt, lte, nlte)
String operations (startsWith, endsWith, contains)
In, NotIn (depends on index mapping)

Very basic sort capabilities (depends on index mapping)
Cursor and offset pagination
Closes #5033

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.9 out of 11 committers have signed the CLA.:white_check_mark: michaelstaib:white_check_mark: ferencleicht:white_check_mark: tobias-tengler:white_check_mark: oising:white_check_mark: jabellard:white_check_mark: arif-hanif:white_check_mark: sarvasana:white_check_mark: rstaib:white_check_mark: artola:x: dmg-hamann:x: A360JMaxxgamerYou have signed the CLA already but the status is still pending? Let us recheck it.

I guess that the basics are ready to review now. There are a few limitations I ran into.
The following information might not be perfectly correct. It is just how I understand the Elastic docs.

Sorting: Sorting on relies on analyzed fields. Generally ES generates a .keyword field which is analyzed. I hardcoded that .keyword is appended to the fieldname for the sort string. I was not able to find a better solution yet.
In and not in is quiet the same as sorting. It depends on the Index mapping. As far as I got the docs, you should use a term search for this case. But term searches do not work for normal text fields. Term searches are meant to be used on e.g. ids.

I was thinking about doing the OpenSearch lib too. It basically is copying the HotChocolate.Data.ElasticSearch project replacing the Nest nuget with the open search client and fixing the namespaces. Does that make sense? Or is there maybe a better way?

So first of all, Thansk a lot for the work you put into this integration :)

Sorting: I guess in the end it's up to the enduser to only expose the fields that are indexed. You have the same issue of course when you just use the EF core integration. If the field is not index, it should not be exposed anyway. We could enforce it in this case by disabling inference? if it is just broken when you do [UseSorting] this might be the best option
Same for in and not in. We could disable inference, but allow to include it?

OpenSearch: Yes, the drivers are essentially the same at the moment

I guess for sorting we would need to annotate fields to be either ignored for sorting or that the user can overwrite the field name in the index.  Because the type of the field must be keyword. As I mentioned, ES does create a keyword typed field for text typed fields by default(I guess  😆  ).  If the user could overwrite the ES field we sort on it would work.
I found another solution. Instead of creating a TermQuery we can just do a combined match query. That might not be the best solution performance wise. But at least it works for the simple cases like strings and numbers. I guess it would be different for e,g. dates. I already updated the code and was able to add unit tests for in and notin

Due to OpenSearch. I guess after ES got its way into main I might do another pr where I replace the client for OpenSearch.

I solved the merge conflict. If you want to I can switch the target branch to main aswell.
@PascalSenn  If there is anything I can help with to get this on the road let me know.

@PascalSenn hi any chance this get merged or are there any plans? or shall I create a custom external nuget?

Yes ... lets get it into 13.1 as experimental ... do you think we can get it in by the end of the week :) we will create a first RC on Saturday or Sunday. If not we can put it into 13.2

Thanks for the feedback. I will start to work on the comments right away. As I'm having a week of this week I guess i can get it done till the weekend.

So I guess i worked on all comments now.
There are basically two things to do which I explained in detail on the corresponding comment.

Field name abstraction
Missing configuration overwrites

I dont know what is left to do for a first experimental version. Please feel free to review again and tell me the missing pieces.
If there is more to do in the future based e.g. on user feedback I am happy to help you. I am interested to learn more about Elastic and I am also  very interested in this project. :)

@A360JMaxxgamer  checkout:
https://github.com/ChilliCream/graphql-platform/discussions/4883#discussioncomment-2420652


Add docs for enum's .exclude & .extract

I was looking for these in the docs but couldn't find them, type hints made me aware of their existence though

@JacobWeisenburger Can I take this one?


Image Object: Missing parameter from DB causes exception

Upgraded system with all images deleted. Started new instance requiring image download.
Jan  2 20:24:10 sf-3 INFO sf-queues-1609579445.7002475-000[22037] Unable to retrieve MD5SUMS for cirros image; url=http://download.cirros-cloud.net/0.5.1/cirros-0.5.1-x86_64-disk.img
Jan  2 20:24:10 sf-3 ERROR sf-queues-1609579445.7002475-000[22037] [Exception] Ignored error in sf-queues: 'sequence'
Traceback (most recent call last):
  File "/usr/local/lib/python3.6/dist-packages/shakenfist/daemons/queues.py", line 71, in handle
    image_fetch(task.url(), instance_uuid)
  File "/usr/local/lib/python3.6/dist-packages/shakenfist/daemons/queues.py", line 134, in image_fetch
    img.get([lock], instance)
  File "/usr/local/lib/python3.6/dist-packages/shakenfist/images.py", line 215, in get
    return self._get(locks, related_object)
  File "/usr/local/lib/python3.6/dist-packages/shakenfist/images.py", line 237, in _get
    actual_image = self.version_image_path()
  File "/usr/local/lib/python3.6/dist-packages/shakenfist/images.py", line 233, in version_image_path
    self.latest_download_version['sequence'] + inc)
KeyError: 'sequence'

Released.


Docs: type 'unknown' missing from 'Everydad Types'

Can someone please add the definition of the 'unknown' type and clearly describe the purpose of it, and how is differs from the type 'any'.
Sample coding show proper use of 'unknown' as well as the context to use it over 'any' would also be benificial.

This explanation/distinction is given in the "More on Functions" chapter of the handbook: https://www.typescriptlang.org/docs/handbook/2/functions.html#unknown


Implement an item class

The item class should have the following fields:

Title
Description
Affected stats
Value
Range
Type (ranged/melee/shield/other)
Rarity

https://github.com/lrudolph1/GenericRPG/commit/ee722ff50fced4af0d2eca942313130938ed75f2


[WIN M3] WIN (ZIP/MSI) Validation

Tasks(ZIP)
Tasks(MSI)
Estimate
Status(ZIP)
Status(MSI)
Notes

Browser download from website to test the WIN artifact availability in artifacts.opensearch.org
Same
1
Not Started
Not Started

Use curl to test the package availability through download links on website
Same
1
Not Started
Not Started

Install WIN-ZIP by extracting on the host
Install WIN-MSI by double-click the executable file and install automatically
1
Not Started
Not Started

Use corresponding utility to verify the signature of the WIN artifact
Same
1
Not Started
Not Started

User use batch scripts or custom scripts to start, and check whether the log output match the process manager status messages
User Windows service to start, and check whether the log output match the process manager status messages
1
Not Started
Not Started

Test the start time of the services through API and setup a threshold
Same
1
Not Started
Not Started

Allow users to easily upgrade/downgrade WIN artifact
Same
1
Not Started
Not Started

Implement automation to replicate these practice for testing purposes
Same
1
Not Started
Not Started

Create a matrix showing which version to which version is upgradable/downgradable for shards compatibility reasons
Same
1
Not Started
Not Started

Documentation on how to test windows distirbutions for plugin owners.
Same
1
Not Started
Not Started

Documentation on how to test / run windows distributions for users.
Same
1
Not Started
Not Started

#2761

We are also waiting for the #2725 from @Divyaasm as well.


Some dependencies contain CVEs

Hi, I have noticed that some library CVEs may be related to your projects. To prevent potential risk it may cause, I suggest a library update. Please note the following details.

Vulnerable Library Version: org.postgresql : postgresql : 42.1.4
CVE ID: CVE-2018-10936
Import Path: pom.xml
Suggested Safe Versions: 42.2.10, 42.2.10.jre6, 42.2.10.jre7, 42.2.5, 42.2.5.jre6, 42.2.5.jre7, 42.2.6, 42.2.6.jre6, 42.2.6.jre7, 42.2.7, 42.2.7.jre6, 42.2.7.jre7, 42.2.8, 42.2.8.jre6, 42.2.8.jre7, 42.2.9, 42.2.9.jre6, 42.2.9.jre7
Vulnerable Library Version: com.h2database : h2 : 1.4.196
CVE ID: CVE-2018-10054, CVE-2018-14335
Import Path: pom.xml
Suggested Safe Versions: 1.4.198, 1.4.199, 1.4.200

@Morty-se Noted.  However, these libraries are only used in the automated test suite and are not deployment-time dependencies for end-users.


Remove assert dev dependency from Key Vault

Solves https://github.com/Azure/azure-sdk-for-js/issues/17110#issuecomment-985076005.
Removing assert dev dependency since it is no longer needed when using chai and it is introducing a circular dependency issue when upgrading it to the latest version.

@JonathanCrd just noticed something in app-config that we should fix.
If you search through source code for
import * as assert from "assert";

You'll find that we still use it in various places. That will continue to work in node because it's a native module in node. The assert package brings the same set of APIs to the browser.
So, removing the package is not enough as you can see from the test failure. What you'd want to do is find any instances of the above string and use import { assert } from "chai" instead.
Please let me know if this doesn't make sense or if you have any questions.
Totally not a big deal, but I do believe that app-config nightlies are failing because of this if you can take a look at packages for which this has already been merged? cc @HarshaNalluru

/azp run js - keyvault-keys - tests

@maorleger Please merge at your convenience


Build with hdfs failed

* (HEAD detached at v2.9.3)
  main

Refer to https://docs.alluxio.io/os/user/stable/en/contributor/Building-Alluxio-From-Source.html#build-different-hdfs-under-storage
mvn clean install -pl underfs/hdfs/   -Dmaven.javadoc.skip=true -DskipTests -Dlicense.skip=true   -Dcheckstyle.skip=true -Dfindbugs.skip=true   -Pufs-hadoop-3 -Dufs.hadoop.version=3.0.0-cdh6.3.2

[INFO]
[INFO] ------------------< org.alluxio:alluxio-underfs-hdfs >------------------
[INFO] Building Alluxio Under File System - HDFS 2.9.3
[INFO] --------------------------------[ jar ]---------------------------------
Downloading from HDPReleases: https://repo.hortonworks.com/content/repositories/releases/org/alluxio/alluxio-shaded-hadoop/3.0.0-cdh6.3.2/alluxio-shaded-hadoop-3.0.0-cdh6.3.2.pom
[WARNING] The POM for org.alluxio:alluxio-shaded-hadoop:jar:3.0.0-cdh6.3.2 is missing, no dependency information available
Downloading from HDPReleases: https://repo.hortonworks.com/content/repositories/releases/org/alluxio/alluxio-shaded-hadoop/3.0.0-cdh6.3.2/alluxio-shaded-hadoop-3.0.0-cdh6.3.2.jar
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 3.809 s
[INFO] Finished at: 2024-05-08T11:20:26Z
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal on project alluxio-underfs-hdfs: Could not resolve dependencies for project org.alluxio:alluxio-underfs-hdfs:jar:2.9.3: Failure to find org.alluxio:alluxio-shaded-hadoop:jar:3.0.0-cdh6.3.2 in https://repo1.maven.org/maven2 was cached in the local repository, resolution will not be reattempted until the update interval of central has elapsed or updates are forced -> [Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/DependencyResolutionException

Also tried
mvn clean install -pl underfs/hdfs/   -Dmaven.javadoc.skip=true -DskipTests -Dlicense.skip=true   -Dcheckstyle.skip=true -Dfindbugs.skip=true   -Pufs-hadoop-3 -Dufs.hadoop.version=3.0.0

and

got  similar error

Actually , I can't find alluxio-shaded-hadoop and only alluxio-shaded-hadoop-client instead , how you guys make it work?

First problem:
[ERROR] Failed to execute goal on project alluxio-underfs-hdfs: Could not resolve dependencies for project org.alluxio:alluxio-underfs-hdfs:jar:2.9.3: Failure to find org.alluxio:alluxio-shaded-hadoop:jar:3.0.0-cdh6.3.2 in https://repo1.maven.org/maven2 was cached in the local repository, resolution will not be reattempted until the update interval of central has elapsed or updates are forced -> [Help 1]

you can look here.https://stackoverflow.com/questions/4856307/when-maven-says-resolution-will-not-be-reattempted-until-the-update-interval-of

First problem:
[ERROR] Failed to execute goal on project alluxio-underfs-hdfs: Could not resolve dependencies for project org.alluxio:alluxio-underfs-hdfs:jar:2.9.3: Failure to find org.alluxio:alluxio-shaded-hadoop:jar:3.0.0-cdh6.3.2 in https://repo1.maven.org/maven2 was cached in the local repository, resolution will not be reattempted until the update interval of central has elapsed or updates are forced -> [Help 1]

you can look here.https://stackoverflow.com/questions/4856307/when-maven-says-resolution-will-not-be-reattempted-until-the-update-interval-of

Thanks for your help. I  tried and still not working ......

Seems that the maven cannot download the jar, due to network problem? I'm not sure.

Close it due to no response.


Compare last modified with .py vs .pyc when packaging project

An optimization currently exists that uses the .pyc instead of the .py if it's available. The problem is that if the .py is newer but not compiled, the old .pyc will continue to be used. Should a .pyc present itself, ensure that the update time of the .pyc exceeds that of the .py file.

One line needed here: https://github.com/Miserlou/Zappa/blob/master/zappa/zappa.py#L341

I have code in place for this. How do I test this? I went into my env's site-packages and created a symlink to the dev Zappa/zappa dir, however I was getting an unusual error,
An error occurred (InvalidLocationConstraint) when calling the CreateBucket operation: The specified location-constraint is not valid
In master this didn't appear. Any idea how I can test my changes?

You should have a look at tests/tests.py for writing tests! This one shouldn't be too hard to test, the ones with the @placebo decorator are bit trickier, you just have to read the placebo documentation.

"The specified location-constraint is not valid" is a weird error that makes me think you've put an invalid aws_region in your settings? I don't think that's related to the packaging code at all.

If we can get this patched today then I'll can publish the next version

Solved with a301aa67


Elin/merge main v0.13.0 into main

This change is

Codecov Report
Attention: 6 lines in your changes are missing coverage. Please review.

Comparison is base (44ed16e) 71.01% compared to head (af85962) 70.97%.

Files
Patch %
Lines

crates/blockifier/src/state/cached_state.rs
55.55%
3 Missing and 1 partial :warning:

crates/native_blockifier/src/py_block_executor.rs
0.00%
2 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1311      +/-   ##
==========================================
- Coverage   71.01%   70.97%   -0.05%
==========================================
  Files          59       59
  Lines        7335     7341       +6
  Branches     7335     7341       +6
==========================================
+ Hits         5209     5210       +1
- Misses       1712     1717       +5
  Partials      414      414

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