Dependabot V2 behaviour on PR merge

Previously, using app.dependabot.com we had automatic rebasing disabled because otherwise we're swamped with dependabot builds on every PR merge.
However, when we merged a PR, dependabot would close any PRs that had been solved in the PR we just merged, whereas v2 dependabot only seems to look at HEAD (not the rebased HEAD) and so never closes any PRs.
This means we have to go to each PR and @dependabot rebase on each one which is really time consuming and slow.
Basically the behaviour before where Dependabot would do its checks vs the rebased HEAD (without actually rebasing HEAD on our PR) was much better behaviour.

Another side-effect of having the rebase policy set to disabled is it seems Dependabot doesn't monitor for changes anymore in versions and close and superceed versions like it used to

I believe this is the same request as https://github.com/dependabot/dependabot-core/issues/3880, yes?

@asciimike Thanks, sorry I couldn't find the issue when I searched, it does seem that's a similar/same issue!

Sounds good, closing as a dupe, and we're hoping to look into rebasing behavior and either document the current behavior better or tweak it (I believe we stopped rebasing everything by default since it was thrashing actions and burning folks' minutes, but if folks want that behavior, we may expose the option for folks to rebase everything if desired).

Hey @asciimike, you're conflating two different issues. Rebasing the actual PR is annoying as it burns minutes or in our cases using too many agents in Azure DevOps, however Dependabot used to rebase on its end and not update the PR.
Basically checks for dependencies are run against a local rebase branch and never pushed to the PR.


The html rendering of the  quote tag  in markdown file is strange

Examples can be seen in the post ： https://chenxofhit.com/posts/2021tumudayi/

hello @chenxofhit, seem the scss tags are not correct, take look on my patch
https://github.com/RobbiNespu/hugo-theme-cactus/commit/264db8a14c77c4fafc291c8d5ac9ea2cfaa02669.patch

Examples can be seen in the post ： https://chenxofhit.com/posts/2021tumudayi/

Hi, you can use this link and put it in : /static/custom/style.css
it works on me.


Fix #8507: Consider UnknownSymbols values for import/export purposes

Fixes #8507
This is important for JS files. regardless if we found the module or not, we should be always emitting the import/export to grantee valid transpilation. Also note that this is the same behavior for --isolatedModules, and it is currently an error case anyways, so there should not be any breaks from changing this behavior to working code.

//cc: @vladima

@vladima any more comments?


Added custom dependencies based on user config

looking good! need to add config validation such as the tickers which the exchange receives from there


Filter disable won't work on the same query

Hi, I've created filter to my entities:
this.SoftDeletableFilter = this.Filter<TestEntity>(x => x.Where(a => a.Deleted == false));
First I call db query with filter enabled, after that I call same query with filter disabled but it returns same results.
db.Entities.Select(e => e.Name).ToList();
db.SoftDeletableFilter.Disable();
 db.Entities.Select(e => e.Name).ToList();
EntityFramework somehow caches database query and disabling filter is not invalidating this cache. How can I turn off this EF query caching?
Project repository link: https://github.com/kf-damianurbaszewski/EFQueryCache where you can repeat this case.
Regards
Damian

Hello @kf-damianurbaszewski ,
Thank you for reporting this issue.
It was caused by a fix done this weekend. We made a small mistake by trying to solve another issue.
The v1.4.44 has been released and should now work correctly.
Best Regards,
Jonathan

Hi, after making some changes in my project, I occured the same error.
Now I use the same context for all actions.
Error always occured when i try call the same action twice (disable or enable) when i call actions
alternately error also occured after a few calls.
Repo link: https://github.com/kf-damianurbaszewski/EFQueryCache

Hello @kf-damianurbaszewski ,
Thank you,
Using your example, we have easily found & fixed the issue.
It's always the same kind of issue because Interceptor is called only the first time the query is compiled and we try to optimize thing by getting the compiled query for futur call.
I believe we have no other choice than making every query unique when a modification on QueryFilter is made. In short, the query is compiled and will be re-used as long as the filter doesn't change but as soon as a filter modification is made, the query will need to be re-compiled (calling the DbInterceptor).
We will think about it a little bit and release a fix probably this night or tomorrow if it gets accepted.
Best Regards,
Jonathan

Hello @kf-damianurbaszewski ,
The v1.4.46 has been released.
Let hope it's the last issue you find in this feature ;)
Best Regards,
Jonathan

Closing: Fix confirmed


Is it possible already to render on canvas?

Something like a layer on CoreGraphics to draw simple shapes, like circles, lines, etc.

@andrey-str As mark said it's not possible, and we sadly have a lot of subjects to address before this. Feel free to contribute a PR if you have an idea on how it could work on both iOS and Android.
Thanks.


⚠  Beta test extra fields  ⚠

I have just merged a significant update to master, which is the addition of Extra/Custom Fields!
I need people to help me test this before I push an update for it. It's possibly the biggest addition to Spoolman since the beginning, so I want some thorough testing done before I'm happy to release it.
How to use
Add new fields in the new Settings page in the UI. Then you should be able to go in to existing/new objects and set that new field's value.
Backup
Make sure to backup whatever database you have before you move along with this, and hold tight to that backup. Your data should be safe from corruption, but you never know.
How to beta test
If you use docker, simply change the docker tag from latest to edge and then update. Edge is an automatic build of the latest changes in master, and should be used carefully, since master can sometimes contain broken code.
If you use standalone, open the most recent job here: https://github.com/Donkie/Spoolman/actions/workflows/ci.yml?query=branch%3Amaster, scroll to the bottom and then download the "spoolman" artifact. Unzip to a new directory. Copy over your .env from your existing installation to the new installation. Stop your existing server sudo systemctl stop Spoolman. Run bash ./scripts/install_debian.sh.

Added a field with "option" as data type to spools and created 3 choices.
-> When creating a spool, everything works fine
-> When editing the spool, the field cannot be changed because the dropdown does not work

Added a field with "option" as data type to spools and created 3 choices.
-> When creating a spool, everything works fine 👍
-> When editing the spool, the field cannot be changed because the dropdown does not work ⚠

Hmm ok, can you open your browser console and see if any error appears when you open the spool edit page, and also when you try open the drop down

Sure. No errors when opening the edit page but two warnings when trying to open the dropdown:
index-zEphGS_-.js:3 [Violation]Added non-passive event listener to a scroll-blocking 'touchstart' event. Consider marking event handler as 'passive' to make the page more responsive.  Consider marking event handler as 'passive' to make the page more responsive. See https://www.chromestatus.com/feature/5745543795965952 (anonymous) @ index-zEphGS_-.js:3 (anonymous) @ index-wPEuVktr.js:140 pv @ index-wPEuVktr.js:40 Xx @ index-wPEuVktr.js:40 AT @ index-wPEuVktr.js:40 AT @ index-wPEuVktr.js:40 Wq @ index-wPEuVktr.js:40 Yq @ index-wPEuVktr.js:40 Yl @ index-wPEuVktr.js:40 Zx @ index-wPEuVktr.js:40 Ol @ index-wPEuVktr.js:38 (anonymous) @ index-wPEuVktr.js:40
Interestingly, the form on the edit page seems to get refreshed constantly as soon as I open an edit page that inherits custom fields. Nothing like that seen on pages without custom fields.

Sure. No errors when opening the edit page but two warnings when trying to open the dropdown:
index-zEphGS_-.js:3 [Violation]Added non-passive event listener to a scroll-blocking 'touchstart' event. Consider marking event handler as 'passive' to make the page more responsive.  Consider marking event handler as 'passive' to make the page more responsive. See https://www.chromestatus.com/feature/5745543795965952 (anonymous) @ index-zEphGS_-.js:3 (anonymous) @ index-wPEuVktr.js:140 pv @ index-wPEuVktr.js:40 Xx @ index-wPEuVktr.js:40 AT @ index-wPEuVktr.js:40 AT @ index-wPEuVktr.js:40 Wq @ index-wPEuVktr.js:40 Yq @ index-wPEuVktr.js:40 Yl @ index-wPEuVktr.js:40 Zx @ index-wPEuVktr.js:40 Ol @ index-wPEuVktr.js:38 (anonymous) @ index-wPEuVktr.js:40
Interestingly, the form on the edit page seems to get refreshed constantly as soon as I open an edit page that inherits custom fields. Nothing like that seen on pages without custom fields.

Thank you, I could reproduce the issue and I've pushed a fix now. New release should be available soon


Very slow to delete plans

Deleting plans from the UI seems very slow. On my local instance with about 6 small plans, deleting a plan takes about 40 seconds. On our team test venue (16 plans from a few different users) I am not able to delete any plans because the graphql queries never complete.

Possibly related to https://github.com/NASA-AMMOS/aerie/issues/631


Methods should not return constants#1

What is the code smell/issue?
Methods should not return constants
Why is this code smell relevant?
There’s no point in forcing the overhead of a method call for a method that always returns the same constant value. Even worse, the fact that a method call must be made will likely mislead developers who call the method thinking that something more is done.
This rule raises an issue if on methods that contain only one statement: the return of a constant value. This increases the complexity of the code and also can cause readability issues.
How can we resolve this issue?
declaring a constant instead of a method that returns a constant value can resolve this issue which in turn decreases the complexity of the code.

This one looks ok, even though I would have changed the variable name as the current one suggests this is still a function.


chore(main): release 1.7.0

:robot: I have created a release beep boop
1.7.0 (2023-05-25)
Features

add custom_elements_ls mapping (#232) (76aff7a)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/williamboman/mason-lspconfig.nvim/releases/tag/v1.7.0 :sunflower:


Migrating AVL tree to Swift 3

Claiming AVL tree to Swift 3 migration

Progress so far:

AVL tree has been migrated to Swift 3
AVL tree tests have been migrated to Swift 3
Travis CI build script updated to Xcode 8. Commented out projects that are not yet updated and can successfully run tests for AVL tree on Travis CI
Readme updated
Playground updated. Need to investigate why display strings have changed. Will investigate tomorrow.

Output in Swift 2
Output in Swift 3

Suggestions and feedback on the progress so far would be very welcome! 👍

Ready for review/merge. Would you be able to have a look @kelvinlauKL?

Gave it a look, looks good! Thanks for the update @chris-pilcher!


Improvements and refactoring

Hey @interaminense, do you have improvements in mind?
I liked this project, so i can help you with style structure and i'm still learning React but i can also help with it

You can suggest improvements and changes. I suggest creating the improvement issues so I could get some stuff too. What do you think?

That's great, i think that we can start with the workflow that we could use to,  like branch workflow (names, purpose etc). I believe that it will maintain our commits more stable

i was separating the Header content in another component yesterday, so i created i branch called "components" for this purpose, what do you think about?

I think great!


Add unit tests

We need unit tests.

[x] Add AVA to project
[ ] Add unit test guidelines
[ ] Add tests

Closing this as too vague. Also hard to do with the APIs changing


Update Sentry SDK from version 5.7.1 to version 6.3.5

Sentry says "We recommend you update your SDK from version 5.7.1 to version 6.3.5". They link to https://docs.sentry.io/platforms/javascript/.
5.7.1 was released in 2019, so it's pretty old.

do you think this for 1.2 or following? we're in qa already..

After v1.2 for sure.

lol now it's 6.11


Better Generic Support For ISCs

What type of PR is this?

feature

What this PR does / why we need it:

Allow generics to be set as their Set() string in the config
Allow converting internal ISC types to Generics when:

The data used to define the Generic can be expressed in JSON
The target Generic implements FromJson([]byte) error
The source ISC implements Json(name string) ([]byte, error)

Update existing ISC implementations to also implement Json(name string) ([]byte, error)

And of course, there are tests for the new features.
As to why, the internal ISCs don't actually support Generics, as they don't know about these types to be able to deserialize into them directly. Proper support requires the ability to coerce the intermediate types into a Generic type. This can be done one of two ways - by using a string equivalent as you would on the CLI, or by implementing methods that can handle a JSON intermediate version on the Generic itself.
Given:
	app.Flags = []cli.Flag{
		altsrc.NewGenericFlag(&cli.GenericFlag{
			Name:   "location",
			Usage:  "where to connect",
			Value:  HostPort{},
		}),
		// ...
	}

The following examples are equivalent for the sample HostPort type added in this PR to the altsrc tests:
location: localhost:33

location:
  host: localhost
  port: 33

Third-party code that implements its own InputSourceContext may already support storing Generics in its internal structure, in which case the new code paths won't be hit, rendering full BC.
Testing
Tests were updated in the code to cover the new scenarios.
Release Notes
Better Generic support when using an InputSourceContext

@danhunsaker v2 is only in maintenance mode. You do realise that v3/main branch has support for generics already ?

v3 isn't released/usable/documented yet. I imagine this will need to be forward-ported to v3 as well at some point, but until it's actually released, I can't use it in my own apps. Thus targeting v2 for the moment.

@danhunsaker Thank you again for your patience! I've got much the same feedback as in #1702. How would you like to handle next steps?

Hey, @danhunsaker. Thank you for this PR.
Some merge conflicts showed up, and as Dan explained above, we're very reluctant to accept new features into v2 (much smaller PRs were getting rejected because of this policy).
Now, I'm a new maintainer and new to this codebase, so I'm not yet sure if it makes sense to re-target this PR against https://github.com/urfave/cli-altsrc.
I'm just trying to see what's the state that this PR is in. I don't like if they live for long, because they rot.
Again, thanks for all the work.

Extending code surface to  +416 new lines for a maintenance project is also a security risk.

Wont merge into v2 as it is in maint mode. v3 has better support for multiple value sources and is amenable to be extended easily.


Path cannot be null on init

Using VS 2019, .net core 2.1, Windows 10 Surface Pro on an ASP.NET Core app, when I run electronize init (via command prompt) I receive the following error:
C:\source\repos\TranslationMessageManager>electronize init
Adding our config file to your project...
Search your .csproj to add the needed electron.manifest.json...
Found your .csproj:  - check for existing config or update it.

Unhandled Exception: System.AggregateException: One or more errors occurred. (Path cannot be null.
Parameter name: path) ---> System.ArgumentNullException: Path cannot be null.
Parameter name: path
   at System.IO.FileStream..ctor(String path, FileMode mode, FileAccess access, FileShare share, Int32 bufferSize, FileOptions options)
   at ElectronNET.CLI.Commands.InitCommand.EditCsProj(String projectFile) in D:\Code\GitHub\Electron.NET\ElectronNET.CLI\Commands\InitCommand.cs:line 126
   at ElectronNET.CLI.Commands.InitCommand.<ExecuteAsync>b__10_0() in D:\Code\GitHub\Electron.NET\ElectronNET.CLI\Commands\InitCommand.cs:line 74
   at System.Threading.Tasks.Task`1.InnerInvoke()
   at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state)
--- End of stack trace from previous location where exception was thrown ---
   at System.Threading.Tasks.Task.ExecuteWithThreadLocal(Task& currentTaskSlot)
   --- End of inner exception stack trace ---
   at System.Threading.Tasks.Task`1.GetResultCore(Boolean waitCompletionNotification)
   at ElectronNET.CLI.Program.Main(String[] args) in D:\Code\GitHub\Electron.NET\ElectronNET.CLI\Program.cs:line 54

There isn't anything in that message that makes it clear to me what might be going on here - just figured I would report it in here in case it makes sense to someone on the team :)
Thanks!

I am going to close this - I was able to get past it. I think it was, maybe, because I was using Pages and not MVC application

I am closing this ticket as I am pretty certain the issue was that I was using Pages template rather than MVC template, although it could have been numerous other things. I was combining out-of-date usage instructions with up-to-date instructions.

I'm experiencing same issue. For me this is happening every time.
I'm creating new .net server-side blazor project on VS 2019. Then I added added ElectronizeNET.CLI nuget version 9.31.2 and entries to startup.cs and program.cs. The I run electronize init and get the error.
I'm using .net core version 3.1.

I am also experiencing this same issue

Experiencing the same issue here on a WebAPI project

I have te same issue Net Core 3.1 and ElectronizeNET.API nuget version 9.31.2

I could resolve a problem. I m should be in the folder where are the Startup.cs and Program.cs when I execute this commands.
I expect that this resolve yours problems too.

I recommend you guys open a new issue and provide as much detail as possible. It has been a long time since I personally saw this issue, and I don't recall specifically what the issue was or how I solved it.

So here's the thing:
At the following step:
At the first time, you need an Electron.NET project initialization. Type the following command in your ASP.NET Core folder:
$ electronize init

Be sure you cd to the correct folder in advance:
So if your folder structure is:
├── project
│   └── Startup.cs
└── project.sln

Then in the Terminal do this:
$ cd project
$ electronize init
$ cd ..
$ electronize start

@goonzaah That solution worked for me! (I have an ASP.NET Web Application MVC).
Just to reiterate the answer:
cd into the directory containing Program.cs and Startup.cs and run electronize init in there.


⚠ Secret Website 3 has degraded performance

In aede53e, Secret Website 3 ($SECRET_WEBSITE_3) experienced degraded performance:

HTTP code: 200
Response time: 1924 ms

Resolved: Secret Website 3 performance has improved in 05df75a.


--missing argument assign_tax

Hi!
Thanks for developing this tool!
I am trying to assign taxonomy to my merged DB (NCBI downloaded sequences plus our study specific local DB), however the --missing argument is unrecognized by crabs assign_tax module. Is there an alternative way to append the taxonomic information of the sequences of our local DB using crabs?
Thanks in advance for your help!
Karina

Hello Karina,
Thank you very much for using our software!
Would it be possible to provide the first couple of lines from each file? This will allow me to see what is going on. From my understanding, you have some new barcodes from species that are not in the NCBI taxonomy and would like to add these sequences to your reference DB? Is this correct?
Best,
Gert-Jan

Dear Gert-Jan,
Many thanks for your quick reply!
Yes, we generated a local database since our samples come from a region of high endemicity.
Here are a few lines from the input files:
head merged_NCBI_Paramo_extract_pga.fasta
>AB096873
TCTCGGCTCTCGCATCGATGAAGAACGTAGCGAAATGCGATACTTGGTGTGAATTGCAGAATCCCGTGAACCATCGAGTCTTTGAACGCAAGTTGCGTCCGAGGCCTCCTGGTCGAGGGCACGTCTGCCTGGGTGTCACGCATCGTCGCCCCCACTCC
CCTCGGCTCAAGAGGGCGGGGGCGGATACTGGTCTCCCGCGCGCTCCCGCCCGTGGTTGGCCTAAAATCGAGTCCTCGGCGACGGTCGTCACGACAAGCGGTGGTTGAGAGACCCTCGGACACGGTCGTGCGCGTGTTTGTCGCCCCGGGACCTCCCG
GACCCCCGAGCATTGGCTTTCAAGGATGCTCTCGTTGCGACCCCAGGTCAGGCGGGACTACCCGCTGAG
>AB194537
TCTTGGCTCCTGCAACGATGAAGAACGCAGCGAAATGCGATACGTAGTGTGAATTGCAGAATTCCGCGAATCATCGAGTCTTTGAACGCAAGTTGCGCCCGAGGCTTGACCAAGGGCATTTCTGCCAGAGCGTCACCGCGACACCCACCGATCCACTC
TACCACTGAGGAGTGCTGGATCGAGTGGAACTGGCCATCCGAATCGGCAATGGTTTTTTATGCCATTGAACATTGGGTTGGCTGAAATGGAGGGAATTTGGCCAAGTTGCCGGGGGACACAGCCCGCGATTAGGTGATCCAATGCGCGTTGGTTGTGC
CCCCCGGCGGGCTCGAGGATTCCAAAGCAGGCGCCCCCTTGTACCTCCCCTCTAGAGGGAAGGGAGCAAAGGGGGATCAGTCACGCATCAGACCTCTGATCAGAAAAGACTACCCGCTGAA
>AB194538
TCTTGGCTCCTGCAACGATGAAGAACGCAGCGAAATGCGATACGTAGTGTGAATTGCAGAATTCCGCGAATCATCGAGTCTTTGAACGCAAGTTGCGCCCGAGGCTTGACCAAGGGCATTTCTGCCAGAGCGTCACCGCGACACCCACCGATCCACTC
TACCACTGAGGAGTGCTGGATCGAGTGGAACTGGCCATCCGAATCGGCAATGGTTTTTTATGCCATTGAACATTGGGTTGGCTGAAATGGAGGGAATTTGGCCAAGTTGCCGGGGGACACAGCCCGCGATTAGGTGATCCAATGCGCGTTGGTTGTGC
CCCCCGGCGGGCTCGAGGATTCCAAAGCAGGCGCCCCCTTGTACCTCCCCTCTAGAGGGAAGGGAGCAAAGGGGGATCAGTCACGCATCAGACCTCTGATCAGAAAAGACTACCCGCTGAA

head nucl_gb.accession2taxid
accession       accession.version       taxid   gi
A00001  A00001.1        10641   58418
A00002  A00002.1        9913    2
A00003  A00003.1        9913    3
A00004  A00004.1        32630   57971
A00005  A00005.1        32630   57972

 head DB_Paramo_taxonomic_lineages_CRABS.tsv
CRABS:>086-4-Acy        1       Viridiplantae   Streptophyta    Magnoliopsida   Rosales Rosaceae        Acaena  cylindristachya
CRABS:>087-4-Ael        1       Viridiplantae   Streptophyta    Magnoliopsida   Rosales Rosaceae        Acaena  elongata
CRABS:>087-3-Ael        1       Viridiplantae   Streptophyta    Magnoliopsida   Rosales Rosaceae        Acaena  elongata
CRABS:>037-1-Abu        1       Viridiplantae   Streptophyta    Magnoliopsida   Asterales       Asteraceae      Ageratina     bustamenta
CRABS:>037-3-Abu        1       Viridiplantae   Streptophyta    Magnoliopsida   Asterales       Asteraceae      Ageratina     bustamenta
CRABS:>040-1-Aku        1       Viridiplantae   Streptophyta    Magnoliopsida   Asterales       Asteraceae      Ageratina     kupperi

And the command I'm using:
crabs assign_tax --input merged_NCBI_Paramo_extract_pga.fasta --output merged_NCBI_Paramo_taxonomic_lineages.tsv --acc2tax nucl_gb.accession2taxid --taxid nodes.dmp --name names.dmp --missing DB_Paramo_taxonomic_lineages_CRABS.tsv

Thanks again for your help :)
Karina

Hello Karina,
Thank you for the clarification and additional information!
The --missing parameter is misused I'm afraid. It should be used as an output file name where all sequences will be deposited by CRABS for which it cannot create a taxonomic lineage.
So, if you'd like to merge the downloaded sequences and your in house barcodes into a single file, I would suggest to copy the in house barcodes to the CRABS file after the assign_tax function. It's a bit of a hack, but the only way possible at the moment with CRABS.
Your idea of adding in functionality to provide your own taxonomic lineage info into CRABS for sequences you'd like to incorporate, but for which no taxonomy information is available on NCBI is a neat idea. Would you be okay if I incorporate such a parameter in the assign_tax function?
Best,
Gert-Jan

Hi Gert-Jan,
Ok, I was wondering if I misunderstood the usage of the --missing parameter. Thanks for your suggestion, despite being an  old hack it is not a big deal to copy the missing taxonomy manually to the file generated by crabs  --assign_tax
Sure, thanks for thinking about adding this feature to your code! :)
Best regards,
Karina


Re-implement chaining searchers

There is a thing I have to do now - re-implement chaining APIs to support grouped nodes.
From the year 2019 (beginning with this repository), I have chosen the algorithm and copied the basic implementation for Sudoku Explainer.
This algorithm has been lived for 5 years. I must say this algorithm is very good one that always makes me satisfied.
"But Xiaolu Sunnie", you might say, "Why you didn't use your own algorithm from the start?"
A rough problem is that, I'm not familiar with algorithm of chains. I just think that the chaining rule is too hard to implement, keeping a good performance.
I want to challenge this in not few weeks.

翻译：
有一个亟待解决的事情 - 我必须要开始重新实现链的 API 来支持区块节点了。
从 2019 年开始（梦开始的地方），我就选择了一个 Sudoku Explainer 的基础实现作为我的项目的链的基本处理算法。
这个算法在这个项目里存在了 5 年之久。我不得不说这个算法我一直都觉得不错。
不过你可能会问我小鹿：“为啥你不最开始就使用你自己的算法呢？”
有一个棘手的问题。我根本不熟悉链的算法。我就觉得这玩意儿有点难实现，而且要保证良好的性能。
我打算要不了几周之后开始挑战这个玩意儿。


[Release][Packaging] Use Debian bookworm for uploading binaries

Describe the enhancement requested
We're using bullseye:
https://github.com/apache/arrow/blob/61dde7133381897b5f779827d9c14f684814cb87/dev/release/binary/Dockerfile#L18
But bullseye will reach EOL on 2024-07:
https://wiki.debian.org/DebianReleases
Component(s)
Packaging, Release

Issue resolved by pull request 41241
https://github.com/apache/arrow/pull/41241


GUI Editor Line Gizmo Improvements

This PR introduces functionality to move, drag endpoints and rotate the Lines Gizmo on the GUI Editor.
Without the change:

With the change:

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

@RaananW thanks for the comments, updated the code based on them! :D

Snapshot stored with reference name:
refs/pull/13394/merge
Test environment:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/13394/merge/index.html
To test a playground add it to the URL, for example:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/13394/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/13394/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/13394/merge
https://gui.babylonjs.com/?snapshot=refs/pull/13394/merge
https://nme.babylonjs.com/?snapshot=refs/pull/13394/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/13394/merge#BCU1XR#0


Open In Visual Studio Code

Extension
https://github.com/raycast/extensions/tree/3746067740d57d122aa62bd2dc7f8345369b957d/extensions/open-in-visual-studio-code
Description
In this array:
https://github.com/raycast/extensions/issues/new?assignees=&labels=extension%2Cfeature+request&template=extension_feature_request.yml&title=[Extension+Name]+...
it would be helpful for those who use Visual Studio Code - Insiders edition to be able to launch that. Doing so only requires adding that app to the array on those lines.
Who will benefit from this feature?
Everyone who uses Visual Studio Code - Insiders
Anything else?
This is the change:
      "default": "Visual Studio Code",
      "data": [
        {
          "title": "Visual Studio Code",
          "value": "com.microsoft.VSCode"
        },
        {
          "title": "Visual Studio Code - Insiders Edition",
          "value": "com.microsoft.VSCodeInsiders"
        },
        {
          "title": "VSCodium",
          "value": "com.vscodium"
        }
      ]
    }
  ],

Can you make a PR?


ZET shouldn't try to munge root's files

ZET normally runs as non-root user "ziti" and so this will never succeed under normal circumstances:

sed creates a temporary file in /etc to facilitate an in-place substitution in /etc/resolv.conf
sed replaces /etc/resolv.conf

May 03 18:41:06 localhost.localdomain ziti-edge-tunnel[2343]: sed: couldn't open temporary file /etc/sedLZZv37: Permission denied
May 03 18:41:06 localhost.localdomain ziti-edge-tunnel[2343]: (2343)[        0.107]   ERROR ziti-edge-tunnel:utils.c:31 run_command_va() cmd{sed -z -i 's/nameserver/nameserver <IP_ADDRESS>\nnameserver/' /etc/resolv.conf} failed: 1024/2/No such file or directory

Sounds like it.


Can't get amp-access sample to run on localhost

As is, the <amp-access> sample will fail because the authorization, login and pingback endpoints (https://rocky-sierra-1919.herokuapp.com/) don't allow access from 'http://localhost:8080'.
I've tried replacing that endpoint with a src/json/authorization.json file the returns access: true but that causes another error in the browser console after I change the endpoints to <%host%>/json/authorization.json?...:

Response must contain the AMP-Access-Control-Allow-Source-Origin header

I've added w.Header().Set("AMP-Access-Control-Allow-Source-Origin", "*") to handleAuthorization and handlePingback but that didn't help.

Yeah - this will be fixed when we migrate the amp-access backend #303.
Until then, try adding:
		w.Header().Set("AMP-Access-Control-Allow-Source-Origin", "*")

after this line.

Thanks for the pointer. I might be doing something wrong, because that leads to this error:

If I change that line to
 w.Header().Set("AMP-Access-Control-Allow-Source-Origin", "http://localhost:8080")

the AMP-Access-Control-Allow-Source-Origin header header isn't emitted at all:

I've made sure that the line above is the only line changed in .go files, and the auth endpoint is the one I mentioned above. In amp-access.html:
 <script id="amp-access" type="application/json">
  {
    "authorization": "<%host%>/json/authorization.json?rid=READER_ID&url=CANONICAL_URL&ref=DOCUMENT_REFERRER&_=RANDOM",
    "noPingback": true,
    "login": "<%host%>/json/authorization.json?rid=READER_ID&url=CANONICAL_URL",
    "authorizationFallbackResponse": {
        "error": true,
        "access": false
      }
  }
  </script>

Seems like you need to explicitly define the source origin, try:
 w.Header().Set("AMP-Access-Control-Allow-Source-Origin", buildSourceOrigin(r.Host))

Can you please create a PR for the change. Thanks!

Fixed it as part of #451

This doesn't appear to be fixed... from a fresh git pull, gulp build && gulp backend:serve, I see these errors in the console when accessing http://localhost:8080/components/amp-access/ :

2016-11-04 20:30:36.431 http://localhost:8080/favicons/favicon-230x230.png Failed to load resource: the server responded with a status of 404 (Not Found)
2016-11-04 20:30:36.431 http://localhost:8080/favicons/favicon.ico Failed to load resource: the server responded with a status of 404 (Not Found)
2016-11-04 20:30:36.436 http://localhost:8080/favicons/favicon-96x96.png Failed to load resource: the server responded with a status of 404 (Not Found)
2016-11-04 20:30:36.437 http://localhost:8080/favicons/android-chrome-36x36.png Failed to load resource: the server responded with a status of 404 (Not Found)
2016-11-04 20:30:36.442 http://localhost:8080/favicons/android-chrome-72x72.png Failed to load resource: the server responded with a status of 404 (Not Found)
2016-11-04 20:30:36.500 sw.js:1 Uncaught (in promise) TypeError: Request failed
2016-11-04 20:30:36.515 xhr-impl.js:99 GET https://rocky-sierra-1919.herokuapp.com/amp-access/api/amp-authorization.js…ef=&_=0.6655503369876958&__amp_source_origin=http%3A%2F%2Flocalhost%3A8080 f.le @ xhr-impl.js:99ii @ xhr-impl.js:126f.fetchJson @ xhr-impl.js:166(anonymous function) @ amp-access-client.js:127
2016-11-04 20:30:36.515 xhr-impl.js:99 GET https://rocky-sierra-1919.herokuapp.com/amp-access/api/amp-authorization.js…ef=&_=0.6655503369876958&__amp_source_origin=http%3A%2F%2Flocalhost%3A8080 f.le @ xhr-impl.js:99ii @ xhr-impl.js:126f.fetchJson @ xhr-impl.js:166(anonymous function) @ amp-access-client.js:127
2016-11-04 20:30:36.515 localhost/:1 Fetch API cannot load https://rocky-sierra-1919.herokuapp.com/amp-access/api/amp-authorization.js…ef=&_=0.6655503369876958&__amp_source_origin=http%3A%2F%2Flocalhost%3A8080. No 'Access-Control-Allow-Origin' header is present on the requested resource. Origin 'http://localhost:8080' is therefore not allowed access. The response had HTTP status code 401.  No 'Access-Control-Allow-Origin' header is present on the requested resource. Origin 'http://localhost:8080' is therefore not allowed access. The response had HTTP status code 401. If an opaque response serves your needs, set the request's mode to 'no-cors' to fetch the resource with CORS disabled.
2016-11-04 20:30:38.193 log.js:243 Uncaught (in promise) Error: Pingback failed: : Fetch failed https://rocky-sierra-1919.herokuapp.com/amp-access/api/amp-pingback?rid=amp…mponents%2Famp-access%2F&__amp_source_origin=http%3A%2F%2Flocalhost%3A8080: Failed to fetch(…)f.assert @ log.js:243(anonymous function) @ xhr-impl.js:145

This is a different problem we can't fix. The problem is that we're using a different auth endpoint on heroku which obviously hasn't CORS configured for localhost. This will be fixed once we have migrated the amp-access auth endpoints to our own backend #303.


Create documentation

Documentation should be created using docfx or imilar (GitHub pages? Wiki?) containing:

API reference
Quickstart
Creating a parser
at minimum.

This is potentially not needed while the project is small as the readme covers a lot of it, however if it gets more complex it would be very useful to have.


New hcaptcha support?

When I try to solve hcaptchas I get this:

Is it already possible to bypass them or is it simply not supported yet?

same captcha here, extension does not work even tho i have enough credits

Same problem here, any predictions on when it will be supported?

This challenge has been solved for some time now. You may want to upgrade to the latest version for full compatibility.
Let me know if any of you are still having issues.


Coverage

This is the promised coverage support. I am not sure that coverage is always computed as we want it to (e.g. the parts where we use a library that is hooked into the build scripts), and analysis of programs that are aborted as part of their execution (e.g. via assert) seems to require gcov trickery. Finally, it relies on the fact that we do only perform validation via assertions, as it will obviously result in incorrect summaries if execution continues after performing a technically invalid memory op.
We also still need to decide how we want to do the analysis of the results (e.g. I encountered an invalid example that causes an invalid klee_assume to be done, which we can find in the resulting data).
This pull request utilizes gcovr to deal with the precision problem we discussed in the last meeting, which can be installed from the AUR. It also required our klee's runtime/Runtest/intrinsics.c to be modified to ignore model_version.

@danielschemmel Could you make the minor fixes that I requested? We can then merge what is here and iterate on it if need be on master.

@danielschemmel Thanks. I've not reviewed the python coverage script but I'll take a look at that later on in the week.


Should error events appear as exceptions?

I'm trying out the v2 API and would like to register exceptions without using one of the existing clients. From what I read in the documentation I thought I could use the /api/v{version}/projects/{projectId}/events endpoint and simply send an event like in the example:
{ "type": "error", "source": "Test", "date": "2013-10-16T18:59:09.579Z","@error": { "message": "Simple Exception","type": "System.Exception","stack_trace": " at Client.Tests.ExceptionlessClientTests.CanSubmitSimpleException() in c:\Code\Exceptionless\Source\Clients\Tests\ExceptionlessClientTests.cs:line 77"}}

I tried @error and @simple_error and the event appears in the event log, but not in the exceptions dashboard. What do I have to do in order to register exceptions?

Yes, that will work.
{
   "type":"error",
   "source":"Test",
   "@simple_error":{
      "message":"Simple Exception",
      "type":"System.Exception",
      "stack_trace":" at Client.Tests.ExceptionlessClientTests.CanSubmitSimpleException() in c:\\Code\\Exceptionless\\Source\\Clients\\Tests\\ExceptionlessClientTests.cs:line 77"
   }
}

First there are two things wrong with that example and I've updated our docs.

The date cannot be outside of your retention period, otherwise it's discarded. I'd recommend not even setting it unless you set it to the current time that the error was created.
The path wasn't escaped right in json and as such couldn't be deserialized.

If you are not breaking down an exception's stack trace into our Stack Trace model then you must use simple_error as it's handled differently. I'd recommend using one of our clients and submitting an exception and then using the api docs to retrieve the error event to take a look.
If you have any questions, please let us know. We are here to help.

Great, this works! Thank you


conform: suport for complex types #13

conform: suport for complex types #13

This looks awesome, thanks for the effort and the PR!

@bitwalker maybe it looks not very optimal for now, but I had not very much time for this. If i'll have it I'll return to it.
Thank you.

@0xAX No worries, it looks functional enough for now, we can always revisit it with improvements when it comes up. I do have one request, but if you don't have time, I understand, but could you add a bit of documentation to the README about this feature? I imagine it would be good for people to have an idea of how to set this up, and it would be unfortunate if they had to read the test code to figure that out. I can go through and whip something up too, but I figure it may be faster for you since you still have your head in it. Just let me know if you can't get to it.
Thanks again!

Just FYI, I published this as part of a new release, 0.14.0.

@bitwalker Yes, will add it now


Feature numeric comparison

This is an implementation of the idea from Vatsalan and Christen (Privacy-preserving matching of similar patients, Journal of Biomedical Informatics, 2015).
It creates tokens from numbers, such that numbers that a closer together have more tokens in common.
It works with both, integer and floats.
My first implementation was using the decimal module, which was quite nice. However, the test ran for too long, as the numbers could get quite big and then the hypothesis tests would time out.
Now I am using floats. This introduces some floating point problems. For instance, there is that arbitrary jump from 1.something_e308 to infinity.
That's why we don't support the full range of floats any more.
closes #299

I also forgot to mention, but the documentation needs to be updated too (example schema.rst), and maybe some tutorials...


[csl] fix the csl channel logic on transmitter

Currently whenever the CSL Transmitter receives a Child Update
Request from an SSED, it will update the CSL Channel. If there is a
CSL Channel TLV, it will set the CSL Channel to that value. Otherwise
it will set the channel to 'unspecified'. Considering Child Update
Request is also used by SSED to keep synchronization (by some
implemenations). And those SSEDs don't expect to update the CSL
Channel by not appending a CSL Channel TLV in these Child Update
Requests. So we should keep the CSL Channel unchanged if there is
no CSL Channel TLV.

Has been fixed in #8005


[CMake] Match lowercase "aarch64"

Seems like it is also possible for "aarch64" to be spelled lowercase, this fixes building on Ubuntu on arm at least.

Gah, GitHub cannot do rebases. @neonichu can you please rebase this on top on release/1.0 instead of main?

Rebased on release/1.0.
In terms of testing, @shahmishal should be able to answer this. I am assuming we have some way of doing CI on aarch64, but not sure what it is.

@swift-ci test

We should be able to cross pull request test this by creating a fake PR on Swift repo.

We should be able to cross pull request test this by creating a fake PR on Swift repo.

IIUC, toolchain builds will only test tagged releases, by design. But that's still much better than nothing! 👍

Looks like the toolchain build succeeded and I also tested locally in an aarch64 Docker container.

@lorentey lmk when we have a tag and I will pull this in

1.0.1 is out now -- https://github.com/apple/swift-collections/releases/tag/1.0.1


[BUG] catch-id-in-updating-records

This PR cataches id keys in ingested data structures within the from_dict method.
It expands testing in /home/ben/code/argilla-python/tests/integration/test_update_records.py but we should expand testing for the from_dict method.

Thanks for reviewing this, @burtenshaw.
I have a question here. Should we map to the external_id instead of the id since the id will be generated by the Argilla server? Let me show you the following example:
dataset.records.add(
    [
        {"text": "Hello, World!", "name": "John Doe", "custom_id": "1"},
        {"text": "Hello, World!", "name": "Jane Doe", "custom_id": "2"},
    ],
    mapping={"custom_id": "id"},
)

Here, the custom_id will be used as the record id, but won't be used to create the records (id will be generated by the server).
Maybe we should map to external_id by default instead of the id, and let the record id unfilled. So, based on the code below:
records = dataset.records.add(
    [
        {"text": "Hello, World!", "name": "John Doe", "custom_id": "1"},
        {"text": "Hello, World!", "name": "Jane Doe", "custom_id": "2"},
    ],
    mapping={"custom_id": "external_id"},
)

records[0].external_id == "1"  # True
records[1].external_id == "2" # True

or (as we have already)
dataset.records.add(
    [
        {"text": "Hello, World!", "name": "John Doe", "external_id": "1"},
        {"text": "Hello, World!", "name": "Jane Doe", "external_id": "2"},
    ],
)

Thanks for reviewing this, @burtenshaw.
I have a question here. Should we map to the external_id instead of the id since the id will be generated by the Argilla server? Let me show you the following example:
dataset.records.add(
    [
        {"text": "Hello, World!", "name": "John Doe", "custom_id": "1"},
        {"text": "Hello, World!", "name": "Jane Doe", "custom_id": "2"},
    ],
    mapping={"custom_id": "id"},
)

Here, the custom_id will be used as the record id, but won't be used to create the records (id will be generated by the server).
Maybe we should map to external_id by default instead of the id, and let the record id unfilled. So, based on the code below:
records = dataset.records.add(
    [
        {"text": "Hello, World!", "name": "John Doe", "custom_id": "1"},
        {"text": "Hello, World!", "name": "Jane Doe", "custom_id": "2"},
    ],
    mapping={"custom_id": "external_id"},
)

records[0].external_id == "1"  # True
records[1].external_id == "2" # True

or (as we have already)
dataset.records.add(
    [
        {"text": "Hello, World!", "name": "John Doe", "external_id": "1"},
        {"text": "Hello, World!", "name": "Jane Doe", "external_id": "2"},
    ],
)

This functionality makes sense but we want to hide external_id from the user and show just id instead. We previously discussed that the name external makes it confusing.

Yes. I'm conscientious about that. In the end, what we want is to expose the external_id (a custom id that users can use to identify a record in a dataset) as the record id.
Does it make sense if the record id maps internally to the external id? and the current id as the argilla_id or internal_id or whatever naming to make it clear is an autogenerated/read-only value?

Does it make sense if the record id maps internally to the external id?
Yes. There is probably better naming here, but for now this is functionally ok.

and the current id as the argilla_id or internal_id or whatever naming to make it clear is an autogenerated/read-only value?
That makes sense to me the internal id could also be _server_id ?

That makes sense to me the internal id could also be _server_id ?

Yes, but let's move this to a separate PR after reviewing the docs and how-to guides.

@frascuchon I responded to comments.


Cirrus: Support testing of VM cache-image changes

Previously, it was quite difficult to affect changes to VM cache images
without lots of manual work.  This commit adds a new optional testing
task which mirrors the official-image build task which only runs on
master.  In contrast, the new task may be run at any time in a PR, but
including a magic phrase in the PR description:
CIRRUS: TEST IMAGES
Update documentation to describe the new task and inform on it's usage.
Signed-off-by: Chris Evich<EMAIL_ADDRESS>

/hold

/cc @baude

Okay, so the task output from Cirrus was:
--> ubuntu-18: Exported artifacts in: [gs://packer-import/ubuntu-18-libpod-9d05a268.tar.gz]
--> ubuntu-18:
--> rhel-7: Exported artifacts in: [gs://packer-import/rhel-7-libpod-9d05a268.tar.gz]
--> rhel-7:
--> fedora-29: Exported artifacts in: [gs://packer-import/fedora-29-libpod-9d05a268.tar.gz]
--> fedora-29:
--> fedora-28: Exported artifacts in: [gs://packer-import/fedora-28-libpod-9d05a268.tar.gz]
--> fedora-28:

Those can get convered into images for use in .cirrus.yml with the gcloud command I documented.  Otherwise they expire automatically from the packer-import bucket.

/hold cancel

Reviewer Assistance doc: https://url.corp.redhat.com/38b48a8
(can't find how to make that publicly available...I can paste-bin it if needed)

Important output from the last 'test_build_cache_images':

==> Builds finished. The artifacts of successful builds are:
--> rhel-7: A disk image was created: rhel-7-libpod-5699523102900224
--> rhel-7:
--> ubuntu-18: A disk image was created: ubuntu-18-libpod-5699523102900224
--> ubuntu-18:
--> fedora-29: A disk image was created: fedora-29-libpod-5699523102900224
--> fedora-29:
--> fedora-28: A disk image was created: fedora-28-libpod-5699523102900224
--> fedora-28:

Those names can be dropped in to .cirrus.yml in this, or in a new PR, and stand a good chance of passing all tests.  I'm open to suggestions on simplifying/streamlining this workflow, but would prefer to do that in a follow-up PR (since this one was REALLY hard to get into a functional state).

Rebased and force-pushed with updated docs detailing manual steps further.

another rebase/force-push.

notes to me:
[ ]: Kill all CentOS & RHEL references

@edsantiago thanks so much for the helpful feedback.  I've moved them all into a little todo list (above) that captures them, for  while I apply the changes.

@edsantiago  I had to revert some of the req_env_var changes since they were giving a strange error message I'm not smart enough to debug.  Leaving that for a future improvement.  Otherwise I've --force pushed the changes...expecting it may not go well, let's see...

@cevich got a pointer to those error messages?

@edsantiago all green :clap: I even updated hack/get_ci_vm.sh to support passing SPECIALMODE so VMs can be spun-up to mimic CI through the setup_environment.sh, ready for debugging and issue reproduction.
@baude @mheon PTAL, this is a big-one, though it does come with docs, and many months of rebasing, care and feeding.  With this PR, everybody can manage the VM-Cache images and validate the changes from the safety of a PR.  Nothing goes "live" until produced image names are manually added into .cirrus.yml.

@cevich it looks like you missed my question yesterday: can you point me to logs of the errors you saw with req_env_var? I'm asking because I believe it's pretty important to implement it the way I suggested. Have you actually tested the function? In particular, have you tested that it actually works as expected? I believe it will not, because you are doing while xxx; do ... if yyy; then exit, and that's a classic pitfall. Could you try my suggestion again? Perhaps the failures you saw indicate a real problem.

Question - does this work for creating new cached images, not just existing ones? If I wanted to, say, tweak the F28 job into an F30 job, would I be able to accomplish it with these scripts, or is that a more manual thing?

@mheon With this we're closer, but not quite there.  This PR doesn't affect the requirement for a base-image to build off of.  Rather it automates the process of translating base-images into cache-images.
Most importantly, the new cache-images will also have the libpod integration tests run against them (in the PR).  They can also be used with the hack/get_ci_vm.sh script for even more in-depth playing/debugging/testing.
(I will add a card for the work required to bring in a new F30 base-image.)

it looks like you missed my question yesterday: can you point me to logs of the errors you saw with req_env_var?

@edsantiago yes I did miss that.  I produced the errors when running hack/get_ci_vm.sh to test lib.sh changes.  I'm positive it was something related to other problems (now fixed) and my lack of bash-knowledge.  What we have in here now appears to work, but is not ideal.  It's probably something best left to a dedicated PR, where improvements can be validated independently.

@cevich sorry, it looks like I wasn't clear enough. What I'm trying to say is that the req_env_var code, as it is, does not work. It is broken in two ways, and you can confirm that easily, just save this as foo.sh and execute it:
#!/bin/bash

source contrib/cirrus/lib.sh
req_env_var "fixme" "THIS_IS_PROBABLY_NOT_SET $THIS_IS_PROBABLY_NOT_SET"

echo uh-oh looks like we did not exit after all
exit 0

The first broken, obvious, is that the code is echoing $2 instead of $1. IOW: the entire req_env_var is a NOP. It is not testing anything.
The second broken is more subtle. You are doing an exit inside a subshell run as a pipe:
echo hi | while read line; do
    exit 2
done

This is one of the classic gotchas of bash programming.  It is not fixable.
You can fix the entire problem, and get rid of evil duplication as a bonus, by doing:
req_env_var() {
    ...
    for i; do
        if [ -z "${!i}" ]; then
            echo "FATAL: $caller requires \$$i to be non-empty" >&2
            exit 9
        fi
    done
}

(Took the liberty of adding >&2 which perhaps might not be appropriate for CI environment; adjust as necessary)
(Also took the liberty of removing unnecessary 'env. var.' because there isn't a UNIX user alive who wouldn't find it obvious from the dollar sign).
I hope this is clearer. If not, please ping me again because this is important.

(Given the scope of the req_env_var change I think it might make sense to submit that as a new fast-track PR, then, once that's merged, rebase this one. It would greatly simplify life for reviewers).

@edsantiago the magic word is:
/hold

okay, back to working on this...

...and interrupted again.  Yeah, Ed if you have a mind to set this straight, I'll help lean on that PR to get it merged quick.  TBH I'm really sick of looking at this code, but no prob rebasing again, been doin' it for months :smile:

@cevich see #3053

Okay, got this rebased with @edsantiago changes.  Working on fixingup and testing changes needed to hack/get_ci_vm.sh.  While I'm at it...also seeing if it's "just a quick thang" to build a F30 base-image...

There we go, this should be it @edsantiago  The only additions since you last reviewed are: Some twiddling of hack/get_ci_vm.sh and as the HEAD commit says...a stub for F30 work to follow.

(--force push to fixup missed 'FAH' reference that should have been removed)

The '****' addition is nice but it breaks unit tests.
Changes LGTM on first pass, but diffs are more complex than I expected so I'll need to make another pass; hopefully tonight.

It ain't pretty, but the simplest fix I can think of for unit tests is to fix check_result():
+    local actual=$(echo "$1" | tr -d '*>\012'|sed -e 's/^ \+//')
+    local expect=$2

...and replace $1/$2 with $actual/$expect in the next few lines.

@edsantiago and of course we get all the way through and fail in success.sh anyway :confused:   Simple typo, but yeah, there are a few places where being able to see that 'die' output clearly, is important (since more output follows it).  We're also not running the unit-tests in this image-test "mode" by design.  So I should probably remove the magic string to let the normal gating/lint-checking also take a stab.

This almost passes (still get two rc test failures).
diff --git a/contrib/cirrus/lib.sh.t b/contrib/cirrus/lib.sh.t
index ce51f8ad..28f28839 100755
--- a/contrib/cirrus/lib.sh.t
+++ b/contrib/cirrus/lib.sh.t
@@ -4,6 +4,8 @@
 #
 source $(dirname $0)/lib.sh

+set -o pipefail
+
 # Iterator and return code; updated in test functions
 testnum=0
 rc=0
@@ -20,6 +22,11 @@ function check_result {
     fi
 }

+function decoration_filter {
+    cat - | grep -v '\*' | sed -r -e 's/^>+ //'
+    return $?
+}
+
 ###############################################################################
 # tests for die()

@@ -30,7 +37,7 @@ function test_die() {
     local expected_msg=$4

     local msg
-    msg=$(die $input_status "$input_msg")
+    msg=$(die $input_status "$input_msg" | decoration_filter )
     local status=$?

     check_result "$msg" "$expected_msg" "die $input_status $input_msg"
@@ -50,7 +57,7 @@ function test_rev() {

     # bash gotcha: doing 'local msg=...' on one line loses exit status
     local msg
-    msg=$(req_env_var $input_args)
+    msg=$(req_env_var $input_args | decoration_filter)
     local status=$?

     check_result "$msg"    "$expected_msg"    "req_env_var $input_args"

(``--forcepushing that for now to confirm mysuccess.sh` changes are fine)

/hold
need to run a round of the regular test mode on this PR

OIC...the bug is with the grep in the filter.  It exits 1 when there is no match.

/hold cancel

/hold

/me stamps his feet
Well there's the problem...
df -lhTx tmpfsyy
Filesystem     Type      Size  Used Avail Use% Mounted on
devtmpfs       devtmpfs  2.0G     0  2.0G   0% /dev
tmpfs          tmpfs     2.0G   84K  2.0G   1% /dev/shm
tmpfs          tmpfs     2.0G  416K  2.0G   1% /run
tmpfs          tmpfs     2.0G     0  2.0G   0% /sys/fs/cgroup
/dev/sda1      ext4      4.9G  3.8G  856M  82% /

Typo fix, rebase, and --force push.  Thx @edsantiago for spotting that.

/hold cancel

(Note: A run of the latest 'TEST BUILD IMAGES' mode is happening in parallel on another PR)

Gosh...I'm up to four different task re-runs here, all of them passing on the second try.  The inconsistent instability makes me nervous :face_with_head_bandage:

...okay, looks like there's some new flakes introduced on master, and so picked up here on rebase.

...Also seeing something possibly wrong when building images, where F30 is using F29 base image, and F29 is using F28 base image.  That's really really bad, must confirm...

Ahhh...got it...found a rebasing problem needs fixing.

okay, rebased, and running with fresh new cache-images.  There's a high-probability flake on master affecting podman-in-podman and ubuntu.  Expecting the podman rm and some networking tests to fail on those two.

That 'testing ubuntu-18...' task failed with the flakes we have on master.  I expect 'SPECIALMODE: in_podman' will also fail.

/approve

@baude @mheon @edsantiago :balloon: passing now with images produced by and (additionally tested) in-line using the documented magic-string image-rolling scheme :balloon:

Note: You can see the in-line magic-string image-rolling happening (and also passing) over here:  https://github.com/containers/libpod/pull/3078

It's green. LGTM.
Would like final approval from @edsantiago though

@mheon yes I agree we should wait for Ed.  It's much easier for me to rebase a few more times if need be, than deal with this breaking CI for everybody (esp. after the last few weeks).

/hold for rebase/force push

/hold

LGTM; but most of what I did was diff against 55361d74.
/lgtm

LGTM; but most of what I did was diff against

That's fine, thanks for doing a final pass.  Okay, I'll do (hopefully) the last rebase and push with the updated comments re: the disk resize workaround...

note: Image-build testing is happening over on PR #3078 (rebased against this one).

/hold cancel
Okay here and the image-build workflow is functional as well

/hold cancel

@edsantiago may I trouble for one more /lgtm ?
(only addition was a fix for image-building bd0ac63)

/lgtm
(ignoring typo in comment; please let's just move past this)

Thanks again @edsantiago  -  and sorry for the PR size :confused:


Update logback-classic, logback-core to 1.4.6

About this PR
📦 Updates

ch.qos.logback:logback-classic
ch.qos.logback:logback-core

from 1.2.11 to 1.4.6
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "ch.qos.logback" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "ch.qos.logback" }
}]

labels: library-update, early-semver-minor, semver-spec-minor, commit-count:1

Superseded by #341.


Commit user is not identified as AzureDevops user

Hello,
I have this issue on contributions graph. Work item are reported under one user and commit under another one.

@luigisaggese The activity filters for work items (Created, Reactivated, Closed), are not enabled based on your screen capture.
True I see the missing commit from 6/19 for Kimyonok. @ostreifel would this be related to the identity lookup (since there are 0 contributions)?

@ostreifel @stedelen Yes also in my opinion looks something related to identity lookup.


Add GestureDetector.onDoubleTapDown()

Description
This change adds GestureDetector.onDoubleTapDown and GestureDetector.onDoubleTapCancel
callbacks (and corresponding callbacks to the double tap gesture recognizer as well).
The onDoubleTapDown callback will pass tap down details so that callers can capture the details
for when the onDoubleTap callback fires.
The onDoubleTapCancel callback will fire after onDoubleTapDown if the gesture loses the arena.
Related Issues
https://github.com/flutter/flutter/issues/20000
Tests
double_tap_test.dart already contained test cases for various interesting scenarios, so rather
than writing new tests, this change updates all existing tests to add extra expectations around
whether and when the new callbacks fire in the test cases.
Note that I did restructure the tests in that file to use setUp() and tearDown().  I did so for
the following reasons:

It saves a significant amount of boilerplate in the test file
Previously, an exception in one test would cause other tests to fail, because it would cause the gesture recognizer to fail to get properly disposed of

Checklist

[x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
[x] I signed the [CLA].
[x] I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement].
[x] I read the [Tree Hygiene] wiki page, which explains my responsibilities.
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change

[x] No, no existing tests failed, so this is not a breaking change.
[ ] Yes, this is a breaking change. If not, delete the remainder of this section.

You are amazing man! Thanks a lot for the work. Hope the added it soon 🥇


Min width of main window too large

I guess this is actually a roam issue, but I think it would fit nicely into the features of masonry. The flex-box width of the main roam view seems to be fixed to a minimum of 40% (on my monitor at least, don't know how those 40% are calculated).

This result in a lot of white space around the actual blocks in roam main. Maybe you can add resizing of the main window similarly to sidebar cards. Or somehow bypass the scaling restriction.

At the moment I just set this manually in full-screen with
.roam-main {
	flex: 0 0 25% !important;
}
#right-sidebar {
	flex: 0 0 75% !important;
}

This is wasted space right here. Could make the 2 sidebar pages bigger, if there was a solution for this padding issue. The thing is I love the size of the main window, how small it is. When I play with the padding it messes up with the size of the main window.

Get's quite bad on an ultra-widescreen:

Are you using the js scripts of the masonry?

Yes. This screenshot is from a test db where I don't use it. Issue is the same.

Alright. Just wanted to make sure.


Question

Hi,
Please tell me if this tool scans only the main index or there is a option to crawl and test the entire site? I'm sorry to ask, but I haven't seen anything about this.
Thank you
scan4log4shell remote url http://testphp.vulnweb.com -a
[i] Log4Shell Remote Vulnerability Scan
[i] Listening on c832bfasahsi8gnc8r20cednmxoyyyyyr.interact.sh
[i] Start scanning CIDR http://testphp.vulnweb.com
---------
[i] All request to http://testphp.vulnweb.com have been sent
[i] Completed scanning
[i] Waiting for incoming callbacks!
[i] Use ctrl+c to stop the program.

also, is it possible to scan a text file with multiple URL addresses? i.e scan4log4shell remote urls.txt
Cheers

Hi,
That’s great, we can copy and paste on the target.txt file the URLs from Burp. Thank you very much.
Cheers

it seems it can't stand these commands
cat targets.txt | scan4log4shell remote url
requires at least 1 arg(s), only received 0

cat targets.txt | scan4log4shell remote

Are you using the correct version v1.2.0?

Oh, I'm really sorry, it seems I was using the old version. my bad!
Thanks

Works like a charm now!
btw, last question, if the list of url addresses I scan is large, when the scanner finds a vulnerability does it stop or will it report them at the end when it finishes scanning?

or there is the option instead of interactsh to use Burp collaborator and then we see in real time when it finds an interaction?

The scanner does not stop. All targets are always scanned.
Various catchers (TCP, DNS, LDAP) are included. Burp is not yet supported. But you can easily extend it:
https://github.com/hupe1980/scan4log4shell/blob/78167796e8b7ffdd23132ec26fdb35365172b1ec/internal/catcher.go#L18-L23
Your welcome to create a pull request


Issue #58 Naming of repeat methods; additional tests; fix hard-coded …

As discussed in Issue #58:

Fix hard-coded GBP currency for repeat transactions.
Rename repeatPayment to repeatPurchase.
Introduce repeatAuthorize to complement repeatPurchase.

The repeatPayment has been deprecated, but left in for now as an alias of repeatPurchase, as it is in use in production.

Nice work @judgej - thanks for sorting out some of those issues.

I haven't tested it on my test Sage Pay account yet, so I'll do that before taking it any further.
The overriding card details may need a small rethink, as the card object contains both the CVV and the shipping address as a single object. You may wish to supply a CVV but not change the shipping address on a repeat, and that isn't possible here. Maybe the way to do it is to check all the shipping address fields, and so long as ALL are null, then assume no shipping address has been supplied. I suspect that is what the remote gateway does anyway, so if you supply even ONE shipping field, it will discard the whole saved shipping address and use that single field.

Trying this out on my test instance, and I'm getting a few odd results. Any attempt to do a repeatAuthorize (REPEATDEFERRED) gives me this error:

4006 : The TxType requested is not supported on this account.

I also get that if I attempt a repeatPayment on an authorize before that authotization is captured. My account settings is set to allow REPEATDEFERRED, so I'm sure there is some subtlety that I'm overlooking. I've double checked the documentation, and can't see anything wrong there. I'll raise it with Sage Pay support.

As an aside, the repeat message has introduced this inconsistency:

transactionReference - the related transaction when capturing an authorization.
relatedTransactionReference - the related transaction when performing a repeat payment.

I think the first (transactionReference) should probably be used in both cases, to be more consistent with OmniPay in general, but I have not checked other gateway drivers to see if that is the general consensus.

The REPEATDEFERRED is documented in the server/direct shared protocol docs, the latest of which are dated 2014-08-01. REPEATDEFERRED is not mentioned in the server or direct specific protocol docs (both of which indicate you should read the shared docs for REPEAT payments).
I am wondering if REPEATDEFERRED is something that was removed in protocol 3.00 years (years!) ago, but got left behind in the documentation? The official PHP SDK does not seem to cover REPEAT or REPEATDEFERRED.

All works for me now. Turns out Sage Pay did need to enable this feature on my account - REPEAT and REPEATDEFERRED need to be activated on the account separately.

I'm happy this fills in the gaps of PR #66 for issue #58
I'll check whether transactionReference needs to be an alias for relatedTransactionReference for consistency, then accept this PR #68 to finish off issue #58

I'll merge this before close today. Please shout if there are any problems with that. There should be no BC breaks in this.

I think the first (transactionReference) should probably be used in both cases, to be more consistent with OmniPay in general, but I have not checked other gateway drivers to see if that is the general consensus.

Agree with this.  I'm not aware of many other gateways that do this but I believe that Allied Wallet does and it uses transactionReference.

I'll put it in as an alias method for now, and deprecate the old one.


fix walkthrough failed in CI

Fixes #952

The grep logic for checking status of clusters is not precise enough.
It is possible that cluster2 is always in the first position of the federateddeployment status when query happens. cluster1 checking failed because it asks ^cluster1. This is actually not what I want. It is code error but works sometimes.
I guess it is the root cause. I tested several times, CI passed. Let's merge and monitoring if it fixes the issue.

/approve
/lgtm


Higher dimensions

Currently, only up to three dimensions are supported. More was requested as part of #3 , and would probably be a good thing to have.
It shouldn't be particularly difficult to add higher dimensions, but it'll need some testing to make sure it looks reasonable, doesn't regress in the future, etc. I'll look at adding this in the near future.

I've added support for up to six dimensions on the current main branch now. The addition of the extra dimensions does add a bit of compile time, so I've put it behind a compile time flag named "hd" for now - it may take a minute or two to compile. The following should work:
cargo install --features hd --git https://github.com/malthesr/winsfs

I've run some sanity checks in four dimensions and compared against realSFS, where everything seems reasonable, but please open an issue if something seems wrong. Also, bear in mind that runtime does scale roughly exponentially in the number of populations, so things may take a while.


use dep to do package management

name: Feature Request
about: Suggest your excellent idea for this project
Why you need it?
dep makes it easier to add/update dependencies.
How it could be?
As far as I can tell, dep is so far the most up-to-date supported tool to manage dependencies in go world(go mod might be the other choice, but I did not see much adoption yet).
Other related information
As the development going, we might have to introduce other dependencies, and this process has to be simple.

Is dep the official way of go dependent package management tool? @zhouhaibing089
Actually govendor is doing well currently in this project.
I would like to hear more reasonable advantages of dep.


annotations.md: label schema compatibility table

Provided a conversion table to describe the equivalent values between
label-schema.org and OCI image annotations. Some values are clarified.
The human-readable name and description have also been added.
Signed-off-by: Stephen J Day<EMAIL_ADDRESS>

cool cool. and the "name" is presumably not going to have any validation recommendations? or limited to only descriptors?
Otherwise,
👍

cool cool. and the "name" is presumably not going to have any validation recommendations? or limited to only descriptors?

The name is something like "Ubuntu 17.04" - human readable and no constraints.
Most of these labels are intended for landing on the config itself, except for the ref field.

LGTM


Adding/Modifying custom point (Individual) for the Population (NSGA2 or any other MOO)

Hello,
I would like to add my own point or replace an existing point with my own, to an existing population.
I have implemented the current code. Is my implementation correct? Or is there a better way to do this? Would this cause any problem?
If I specify eliminate_duplicates=True for the NSGA2 algorithm, and the custom point is already evaluated in a previous generation, would it cause any Issues? (In the actual code, most of my variables are discrete variables, and I'm performing a Mixed variable problem).
Thank you.
Modified from source: https://pymoo.org/algorithms/usage.html#Problem-Depdendent
import pprint as pp

import numpy as np
import datetime

from pymoo.algorithms.moo.nsga2 import NSGA2
from pymoo.factory import get_problem
from pymoo.optimize import minimize
from pymoo.core.evaluator import Evaluator
problem = get_problem("zdt1")

algorithm = NSGA2(pop_size=100)

# prepare the algorithm to solve the specific problem (same arguments as for the minimize function)
algorithm.setup(problem, ('n_gen', 10), seed=1, verbose=False)

# until the algorithm has no terminated
while algorithm.has_next():

    # ask the algorithm for the next solution to be evaluated
    pop = algorithm.ask()

    # an example of a custom function
    custom_point = np.random.rand(30)
    # pp.pprint(custom_point)
    for indx, indiv in enumerate(pop):
        if indx == 0:
            # pp.pprint(indiv.__dict__)
            indiv.X = custom_point
            # pp.pprint(indiv.__dict__)
            exit()

    # pp.pprint(pop)

    # evaluate the individuals using the algorithm's evaluator (necessary to count evaluations for termination)
    algorithm.evaluator.eval(problem, pop)

    # returned the evaluated individuals which have been evaluated or even modified
    algorithm.tell(infills=pop)

    # do same more things, printing, logging, storing or even modifying the algorithm object
    print(algorithm.n_gen, algorithm.evaluator.n_eval)

# obtain the result objective from the algorithm
res = algorithm.result()

# calculate a hash to show that all executions end with the same result
print("hash", res.F.sum())

Yes, your code injects a random point in each iteration. The exit() is not necessary, though.
If the solution is already in the population, it will be added twice, but depending on the algorithm, be eliminated in the advance function. But it all depends on your ultimate goal or, in other words, what you are trying to achieve with external solution injections.

Thank you.


extension search by publisher yields misleading results

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

VS Code Version: 1.77.0
OS Version: Darwin arm64 22.1.0

Steps to Reproduce:

Search for extensions by by a publishers name e.g. github
Open one of the extensions you know to be by that publisher e.g. GitHub Actions
Click on that publishers name to open the publisher search
The extension search will be populate with something like publisher:"GitHub"
Results not from that publish are included in the search results
This could be exploited to distribute malicious extensions

https://user-images.githubusercontent.com/1331872/230339742-502d754d-138a-44e0-ad40-b08095794548.mp4

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.77.1. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


On sign up form change default language to German

Let's set it to German by default and skip the check for browser's locale logic.

Hi @diegoaad after discussing with @dottorer, I did the following:

make "German" the first option in all select form elements
leave the locale check unchanged; it's already quite smart. The browser's locale is only used as last resort, before it checks if there's a circle and uses that language; see https://goo.gl/szMCQO

Fixed in recent deployment. Nice!


Improved code style and legibility and added limit option

I hope that can be interesting for you.

Some code improvements (nested ifs and preload regular expressions).
Improved styles removing some strong colors.
Added limit option.

Thanks! I'm at a conference, but I'll try to review soon.

Hi! remember me :)

SO SORRY!
Totally my bad here. I've bookmarked this and will get to it in 2 weeks... you won't believe it, but I'm about to head out on vacation again. My first one since back then. ;)

No problem!!  I want to thank you for working so hard on this code. I really hope that it has served you well. That said, you've submitted an enormous amount of changes for such a small library. In addition to the limit option there are numerous changes that aren't just "style" or "legibility" but large reorganizations of the code.
At this point, I don't think I can just merge this PR in, there's just too much that I can't verify easily. Of course, you can always keep your fork active, and I would be happy to merge in the limit functionality by itself if you wanted to submit that on its own. Additionally, if we were to create unit tests I'd be much more comfortable looking at these changes.
Essentially this just comes down to there being way too much for a single PR. Hope this doesn't deter you from helping this or other projects!!

Ok, no problem! Anyway thanks for chek my code. Regards!


Profile reset

I don't know what's going on, but I adjusted the configuration in the game, and after I quit the game, the configuration was reset again, and the modification file was the same, and it was also marked as read-only?

Looks like I'm having the same exact problem


Update crate version in docs example snippet.

There is an old crate version in Cargo.toml snippet in lib.rs

Ah thanks! Looks like #478 arrived shortly before this one. :-)


Fix etcd / gRPC errors on go get

What this PR does / why we need it:
This PR fixes inability to go get this project.
$ go get -v<EMAIL_ADDRESS>go.etcd.io/etcd/clientv3/balancer/resolver/endpoint
go.etcd.io/etcd/clientv3/balancer/picker
# go.etcd.io/etcd/clientv3/balancer/resolver/endpoint
pkg/mod/go.etcd.io/etcd@v0.0.0-20191023171146-3cf2f69b5738/clientv3/balancer/resolver/endpoint/endpoint.go:114:78: undefined: resolver.BuildOption
pkg/mod/go.etcd.io/etcd@v0.0.0-20191023171146-3cf2f69b5738/clientv3/balancer/resolver/endpoint/endpoint.go:182:31: undefined: resolver.ResolveNowOption
# go.etcd.io/etcd/clientv3/balancer/picker
pkg/mod/go.etcd.io/etcd@v0.0.0-20191023171146-3cf2f69b5738/clientv3/balancer/picker/err.go:37:44: undefined: balancer.PickOptions
pkg/mod/go.etcd.io/etcd@v0.0.0-20191023171146-3cf2f69b5738/clientv3/balancer/picker/roundrobin_balanced.go:55:54: undefined: balancer.PickOptions

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

Fix etcd / gRPC errors on go get

/cherry-pick release/v1.0


How can i use font-awesome icons in pages-menu?

Issue type
I'm submitting a ...  (check one with "x")

[ ] bug report
[ ] feature request
[x] question about the decisions made in the repository

I need to know how to use font-awesome icons in pages-menu, the demo describe this as possible, but i don't know how to use that icons in pages-menu, i have tried so many things, like add icon: 'fas fa-iconname', or icon: {icon: 'fas fa-iconname', pack: 'fontawesome'}, i don't know what can i do for this works.
Current behavior:
When is defined property "icon:" with some icon from font-awesome, icon is not showed.
Expected behavior:
Is expected that icon be showed.
Steps to reproduce:
Change an item from pages-menu, with property icon with font-awesome related string, like 'fas fa-check'.
Related code:
          {
            title: 'Declarações',
            icon: 'fas fa-undo',
            link: '/layout/stepper',
          },

Other information:
npm, node, OS, Browser
NPM last version, Node.JS last version, windows 10, Chrome/Firefox

Angular, Nebular
Angular 8, Nebular 4.4.0

it is very easy to register and use your own custom icon in sidebar menu by following simple steps
Step 1 (for registring the custom icon pack for your moduel )
go to this path  "src\app\pages\pages.module.ts"
Step 2
Mention this code
export class PagesModule {   constructor(iconsLibrary: NbIconLibraries) {     iconsLibrary.registerFontPack('fa', { packClass: 'fa', iconClassPrefix: 'fa' });     iconsLibrary.registerFontPack('far', { packClass: 'far', iconClassPrefix: 'fa' });     // iconsLibrary.registerFontPack('ion', { iconClassPrefix: 'ion' });     iconsLibrary.registerSvgPack('social-networks', {       'python': '',     });   } }
Step 3
go to this path  "src\app\pages\pages.menu.ts"
add your menu with icon like this
 {     title: 'Dashboard',     icon: { icon: 'facebook', pack: 'social-networks'},     link: '/pages/dashboard',     home: true,   },
see the result here

it is very easy to register and use your own custom icon in sidebar menu by following simple steps
Step 1 (for registring the custom icon pack for your module )
go to this path "src\app\pages\pages.module.ts"
Step 2
Mention this code
export class PagesModule { constructor(iconsLibrary: NbIconLibraries) { iconsLibrary.registerSvgPack('myicon', { 'python': '', }); } }
Step 3
go to this path "src\app\pages\pages.menu.ts"
add your menu with icon like this
 { title: 'Dashboard', icon: { icon: 'python', pack: 'myicon'}, link: '/pages/dashboard', home: true, },
see the result here

I appreciate that, but, i need to use FontAwesome icons as a pack, is impractical to register all icons manually on module constructor, and you explained how to use an SVG icon, not an FontAwesome Icon.

it is very easy to register and use your own custom icon in sidebar menu by following simple steps
Step 1 (for registring the custom icon pack for your module )
go to this path "src\app\pages\pages.module.ts"
Step 2
Mention this code
export class PagesModule { constructor(iconsLibrary: NbIconLibraries) { iconsLibrary.registerSvgPack('myicon', { 'python': '', }); } }
Step 3
go to this path "src\app\pages\pages.menu.ts"
add your menu with icon like this
 { title: 'Dashboard', icon: { icon: 'python', pack: 'myicon'}, link: '/pages/dashboard', home: true, },
see the result here

I appreciate that, but, i need to use FontAwesome icons as a pack, is impractical to register all icons manually on module constructor, and you explained how to use an SVG icon, not an FontAwesome Icon.

you can use this code to use font-awesome as icon pack
in-app module ("src\app\pages\pages.module.ts")
export class PagesModule {   constructor(iconsLibrary: NbIconLibraries) {     iconsLibrary.registerFontPack('fa', { packClass: 'fa', iconClassPrefix: 'fa' });     iconsLibrary.registerFontPack('far', { packClass: 'far', iconClassPrefix: 'fa' });       // ...     });   } }
and in page-menu ("src\app\pages\pages.menu.ts")
 {     title: 'Dashboard',     icon: { icon: 'beer', pack: 'fa'},     link: '/pages/dashboard',     home: true,   },
see result here

I see, i have tested now and it works, but the icon is too large, i think this is something about the font size.

I've just changed something on _overrides.css and the size fits.

It doesn't work for me. Does not show the icon. In the angular.json file it searches for the lib called fort-awersome / font-awesome, is that correct?

It doesn't work for me. Does not show the icon. In the angular.json file it searches for the lib called fort-awersome / font-awesome, is that correct?

Could you tell me how you did it?

I've just changed something on _overrides.css and the size fits.

Could you tell me how you did it?

I've just changed something on _overrides.css and the size fits.

Could you tell me how you did it?

I've just changed my _overrides.scss like this:
`@import './themes';
@mixin nb-overrides() {
nb-select.size-medium button {
padding: 0.4375rem 2.2rem 0.4375rem 1.125rem !important;
nb-icon {
  right: 0.41rem !important;
}

}
nb-layout-column {
padding: 1rem 1rem 0.5rem !important;
}
// USE THIS CODE BELOW
nb-icon {
font-size: 1rem !important;
}
}`

@codesiddhant but your solution does not work for me


Offline report

Is your feature request related to a problem? Please describe.
Able to generate local version of jscpd html report without the need of internet access. Currently the html report scripts' src is pointing to outside CDN which on-premise CICD may not have connection to outside world.
Describe the solution you'd like
A way for the html report scripts' src point to local environment with additional cli option like --offline

thank you for the feature request

Added new html reporter, all resources saved locally
<EMAIL_ADDRESS>

Added new html reporter, all resources saved locally
<EMAIL_ADDRESS>


Calculating volume of a sphere, and now got a broken autograder.

[WARN] /tmp/home/Sphere.java:17:17: '4.0' is a magic number. [MagicNumber]
[WARN] /tmp/home/Sphere.java:17:23: '3.0' is a magic number. [MagicNumber]
[WARN] /tmp/home/Sphere.java:17:57: '3' is a magic number. [MagicNumber]

Even though I have to calculate the volume of a sphere!

Circumvented the issue by putting them as static attributes.


Refactor: add combineLLM & refineLLM to loadQAChain&loadSummarizationChain

Refactor the code to achieve two goals:

Utilize a dedicated Language Model (LLM) as the final summary LLM to cater to specific LLM requirements at different stages of the chain.
Streamline the final results only, excluding the intermediate step's output from the streaming process.

Thanks, looks great!


Throw error if specific constructors are called as plain functions

The below constructors will now throw an error if called as plain functions as per their specifications:

CompressionStream
Headers
Request
Response
TextDecoder
TextEncoder
TransformStream
URL
URLSearchParams

These constructors can now only be called via new or Reflect.construct
I wasn't sure what we want to do with the Fastly specific constructors such as CacheOverride so I have left them as-is

I thought this would increase the amount of passed tests from web-platform-tests but it looks like this part of the WebIDL specification wasn't being fully tested within web-platform-tests.
I've made a pull-request to web-platform-tests which tests this part, if/when that gets merged I can update our version of web-platform-tests and then see how many more tests we pass :+1:

This is great, thank you!
I have two notes that should be fairly simple to address before landing this:
1. I think we should apply this to Fastly-specific constructors as well

2. how would you feel about introducing a `CTOR_HEADER` macro, parallel to the `METHOD_HEADER` one, and using that instead?

The obvious difference is that we can't infer the name to use from __func__, but that seems ok to me.

Oh that is a good suggestion! :relaxed:
I've made those changes and pushed it up now :+1:

Oh, turns out URLSearchParams doesn't require any arguments, so we have a bunch of test failures :)

Oh, turns out URLSearchParams doesn't require any arguments, so we have a bunch of test failures :)

Hooray for tests! :dancers:


ReactRedux: How to map to more than one state object and action creator?

Take the example of the FetchData.tsx component which at the end of the file contains the following code that generates the type for the props and connects to the redux store.
// Build the WeatherForecastProps type, which allows the component to be strongly typed
const provider = provide(
    (state: ApplicationState) => state.weatherForecasts, // Select which part of global state maps to this component
    WeatherForecastsState.actionCreators                 // Select which action creators should be exposed to this component
).withExternalProps<{ params: RouteParams }>();          // Also include a 'params' property on WeatherForecastProps
type WeatherForecastProps = typeof provider.allProps;
export default provider.connect(FetchData);

If I wanted to use another part of the state (without losing the weatherForecasts part of course)... how would I map that?
Same question for the actionCreators.
Is there an easy way to "merge" them? Or is having a 1-to-1 mapping between containers and store modules the only option?

I've been able to map to more than one store module by returning an object with one property for each module as so:
const provider = provide(
    (state: ApplicationState) => ({ weather: state.weatherForecasts, anotherModule: state.someModule }), // Select which part of global state maps to this component
    WeatherForecastsState.actionCreators                 // Select which action creators should be exposed to this component
).withExternalProps<{ params: RouteParams }>();          // Also include a 'params' property on WeatherForecastProps
type WeatherForecastProps = typeof provider.allProps;
export default provider.connect(FetchData);

But the same approach doesn't work with the action creators. Probably because by adding a wrapping object it adds a new level in the hierarchy and the provide function can't find the action creator functions where they are supposed to be, right?

Try this, it works fine:
// Build the WeatherForecastProps type, which allows the component to be strongly typed
const provider = provide(
    // Select which part of global state maps to this component
    (state: ApplicationState) => ({ weather: state.weatherForecasts, counter: state.counter }),
    // Select which action creators should be exposed to this component
    {
        requestWeatherForecasts: WeatherForecastsState.actionCreators.requestWeatherForecasts,
        increment: CounterState.actionCreators.increment
    }
).withExternalProps<{ params: RouteParams }>(); // Also include a 'params' property on WeatherForecastProps

The syntax will get cleaner still when TypeScript supports rest/spread properties, as you'll be able to write this:
{ ...WeatherForecastsState.actionCreators, ...CounterState.actionCreators }

where is provide, i add provide and it not found ?


ERROR: Could not find a version that satisfies the requirement dm-control

Hi,
I encounter an error when creating the Conda environment. It seems that the version of dm-control (from D4RL) is unavailable.
Is there any recommended version for the D4RL package? The following is the full error message.
Pip subprocess error:
  Running command git clone -q https://github.com/JannerM/d4rl.git /tmp/pip-req-build-enrmm6ao
  Running command git rev-parse -q --verify 'sha^d5719e2c6ef6ab3b1c678a846c02621abb8074a4'
  Running command git fetch -q https://github.com/JannerM/d4rl.git d5719e2c6ef6ab3b1c678a846c02621abb8074a4
  Running command git checkout -q d5719e2c6ef6ab3b1c678a846c02621abb8074a4
  WARNING: Missing build requirements in pyproject.toml for mujoco-py==<IP_ADDRESS> from https://files.pythonhosted.org/packages/2f/48/b108057c1a23c8da9f4cdc7a7c46ab7cec49c3563c0706d50f2527de6ba0/mujoco-py-<IP_ADDRESS>.tar.gz#sha256=d6ae66276b565af9063597fda70683a89c7356290f5ac3961b794ee90ec50eea (from -r /708HDD/hungyh/trajectory-transformer/condaenv.l4fl7x8g.requirements.txt (line 4)).
  WARNING: The project does not specify a build backend, and pip cannot fall back to setuptools without 'wheel'.
  Running command git clone -q git://github.com/deepmind/dm_control /tmp/pip-install-ehe12w5d/dm-control_d0b0cee6667746188485b8f85955e996
  fatal: unable to connect to github.com:
  github.com[0: <IP_ADDRESS>]: errno=Connection timed out

WARNING: Discarding git+git://github.com/deepmind/dm_control@90f00e4e80af56abb9f905070d0c152845db5602#egg=dm_control. Command errored out with exit status 128: git clone -q git://github.com/deepmind/dm_control /tmp/pip-install-ehe12w5d/dm-control_d0b0cee6667746188485b8f85955e996 Check the logs for full command output.
  Running command git clone -q git://github.com/aravindr93/mjrl /tmp/pip-install-ehe12w5d/mjrl_82435f9d000845a69223ca68a1c237e4
  fatal: unable to connect to github.com:
  github.com[0: <IP_ADDRESS>]: errno=Connection timed out

WARNING: Discarding git+git://github.com/aravindr93/mjrl@master#egg=mjrl. Command errored out with exit status 128: git clone -q git://github.com/aravindr93/mjrl /tmp/pip-install-ehe12w5d/mjrl_82435f9d000845a69223ca68a1c237e4 Check the logs for full command output.
ERROR: Could not find a version that satisfies the requirement dm-control (unavailable) (from d4rl) (from versions: 0.0.286587932, 0.0.286955599, 0.0.288398964, 0.0.288483845, 0.0.295778102, 0.0.300771433, 0.0.312466143, 0.0.318037100, 0.0.318066097, 0.0.319497192, 0.0.322773188, 0.0.355168290, 0.0.364896371)
ERROR: No matching distribution found for dm-control (unavailable)

failed

CondaEnvException: Pip failed

Thanks!

Thanks for catching this; it looks like it was caused by D4RL changing ownership. I have updated the dependencies in this commit.

Thanks for addressing the issue!


Thumbnail of photos from OneDrive is not loaded

Description

Steps To Reproduce
No response
Expected behavior
I'm expecting it to show the thumbnails properly like the way it does in the File Explorer app.

Files Version
<IP_ADDRESS>
Windows Version
10.0.22471.1000
Relevant Assets
No response

Restarting the app resolves the issue


Globalization should work with independent generation

Put your description here. Make a use of below checklist
Issue report checklist:
Version of the project: 0.0.6
Browser/version (+device/version): Not UI issue
Any other tested browsers/devices(OK/FAIL): Not UI issue
URL (minimal example if possible, other repository?): Not UI issue
Steps to reproduce the problem:

Trigger independent generation with globalization task
Call npm run preparePushResult -- -t {{topic}}
Documentation is only regenerated on root

https://github.com/YaaS/chewie/pull/62


Error when building a docker project that has the ng-svg-icon-sprite package

Hello. We use ng-svg-icon-sprite in our project. There was a problem that the docker container stopped building. When I run the docker build command, I get an "Cant import the named export from non Ecmascript module (only default export is available)" error. We have version 11 of Angular. There are suspicions that the problem is with dependencies. Could you suggest what can be done? Maybe we need to update Angular?
Update: We have been using the package for 2 years now, but the error appeared a couple of days ago
"ng-svg-icon-sprite": "^1.10.0",

Hello @loozmax, thanks for asking this question. The reason is that using ^ in the dependency version will pull the latest 1.x version (what is currently 1.2 for Angular 12). What you want, is just to upgrade the 1.1.x version of ng-svg-icon-sprite. This you can achieve by writing:
"ng-svg-icon-sprite": "~1.10.0"
See https://stackoverflow.com/questions/22343224/whats-the-difference-between-tilde-and-caret-in-package-json
Tell me if that worked for you

Working perfectly! Thanks so much


fix: typo in documentation

closes https://github.com/nvim-telescope/telescope-file-browser.nvim/issues/152

Thanks :) but I'm afraid this would have to be changed in the picker.lua file from which the docs are generated.


Expects "identified by" field for Transaction and Event resource types

As of composer v0.9.0 transactions and event resources should not define an identifier.  This plugin does not reflect this change.
e.g. The following code:

transaction IssueTokens {
  o Long amount
  --> Business recipient
}

Generates the following error:
file: 'file:////models/fitchain.cto'
severity: 'Error'
message: 'IllegalModelException: line 267 column 1, to line 272 column 2. Class IssueTokens is not declared as abstract. It must define an identifying field.'
at: '267,1'
source: 'Composer'

Hi @theophoric what version of the plugin were you using when you saw the above error? The latest 0.9.1 plugin version has been updated with the Composer v0.9.0 parser and so you need to use this version to use the updated transactions and events syntax that you show above.

So I have just tested this example transaction with the 0.9.1 plugin version and it does not give an error:
transaction CloseBidding {
--> VehicleListing listing
}
Therefore I'm closing this as fixed in the current release. If you still see a problem in your setup after updating to the latest version please re-open or raise a new issue. Thanks, MGK