nixos/miniflux: use systemd notify and watchdog

Description of changes
Set systemd service type to notify, as miniflux supports type=notify since March 2021 (2.0.29).
Enable the systemd watchdog, which miniflux supports since June 2021 (2.0.31).
I also removed the usage of with lib; in the miniflux module.

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

I set WATCHDOG=1 in the config to make sure the watchdog is explicitly enabled (even though that setting is the default).

@ofborg test miniflux


Update package.xml

could you add in the subject or title the gus WI surrounded with @? Example: @W-12345678@


Deleting API keys of other users (as a server owner)

Hi!
I am trying to disable access to a Quetz server for "old" users. We are using azuread as the only authentication provider so these users should not be able to authenticate without a valid company Azure/Office account.
However existing API keys linked to these accounts still work. Therefore I want to remove all API keys for these accounts.
So far I have done this with a DELETE /api/users/{username} which removes the Identity, Profile, and API keys:
https://github.com/mamba-org/quetz/blob/56ab2cfc58c89111637c62f9fc54bec8c97f522e/quetz/dao.py#L233-L241
The problem with this method is that quetz-frontend requires Profile data to display the details of the uploader of a package/version. The result is that the page will not load and display an error for package version uploaded by a user that was deleted using this method.
Is there an alternative (existing) way of doing this?
I was thinking that adding an optional user parameter to the GET /api/api-keys resource could work. This would give me a list of keys to then delete.

This is my current solution: https://github.com/sr-pvgils/quetz/commit/7f639e6ff21ea9c54114c9d771be5538b55a1869
If I have some more time I will figure out the unit tests and submit a PR


Fix androidTargetSdkVersion

Changes

Fix for newer versions of Android API levels that do not get correctly parsed.

Checklist

[x] Read the contribution guide and accept the code of conduct
[x] Readme (updated or not needed)
[x] Tests (added, updated or not needed)

Codecov Report

Merging #425 (3fd972e) into main (db2d8b6) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 3fd972e differs from pull request most recent head d9aef32. Consider uploading reports for the commit d9aef32 to get more accurate results

@@           Coverage Diff           @@
##             main     #425   +/-   ##
=======================================
  Coverage   52.52%   52.52%
=======================================
  Files          66       66
  Lines        2056     2056
  Branches      353      353
=======================================
  Hits         1080     1080
  Misses        972      972
  Partials        4        4


Uncaught TypeError: Cannot read property 'modelName' of undefined

Version info

e.g.
DEBUG: -------------------------------
DEBUG: Ember      : 3.7
DEBUG: Ember Data : 3.7.0
DEBUG: Firebase   : 5.9.0
DEBUG: EmberFire  : 2.0.10
DEBUG: jQuery     : x.x.x
DEBUG: -------------------------------

Test case
Not able to push records which has relashionships like belongsTo and hasMany.
Expected behavior
Records should be pushed to the firebase
Actual behavior
TypeError:  Cannot read property 'modelName' of undefined

Please use Ember v3 4
If you really wanna use Ember v3.7, you have to use the beta release of the latest emberfire.

I'm getting the same issue, any updates here? I'm on Ember 3.6, should I really downgrade to 3.4?

edit Firebase.js in ember folder or just change the version in package.json and run npm install


Get POST fields

Hi,
I'm trying to get the parameters sended in the POST from my own. I have seen that Submit petitions are sended to /forms/formname/ and that you could put a 'redirect URL'. Is there anyway to received the data in that URL? because what I have done is to redirect to an URL which have a view like this:
 def testForm(request):
     print request
     return HttpResponse("OK", content_type="text/plain")

But the request doesn't have that info.
Thanks in advance

Can't help you there, maybe try stackoverflow.


Direct conversion of NSError to JSONString?

Is there any way to do this? Currently I have a JSONValueTransformer+NSError category and a custom object holding my error.
In order to log the error in JSON I have to do this:
NSLog(@"%@", [[Model alloc] initwithError:error].toJSONString);
I would prefer to do this:
NSLog(@"%@", error.toJSONString);
anyway to do this or should I just make a macro and be done with it?

The way you're doing it seems probably the best way. NSErrors can be rather complex objects and do not have serialization methods.


Pronto fails on .eslintrc

Recently updated my app to include webpacker and webpacker-vue.
now running pronto returns:
.eslintrc: F: Parsing error: Unexpected token import
Is it possible to prevent linting on  the .eslintrc file?

I believe this is a red herring. I dug into the eslintrb code and found that they default to saying the filename is .eslintrc if the error is fatal. No idea why... but that's what the code does. I had the same issue. I ended up setting a breakpoint in the eslintrb code so I could look at the actual offences and figure out where they came from.

I think this runner is effectively obsolete as eslintrb hasn't been updated since 2016.

FYI, this is actually a bug in pronto-eslint. I opened up a WIP PR that fixes this issue here. However, I'm still working through trying to get a couple of tests to pass.


Add different sand and prod endpoint test

Purpose

$subject

Codecov Report
Patch coverage has no change and project coverage change: +54.63% :tada:

Comparison is base (9e7eb8f) 25.47% compared to head (6a971e3) 80.10%.
Report is 1 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #1734       +/-   ##
===========================================
+ Coverage   25.47%   80.10%   +54.63%
===========================================
  Files         131        9      -122
  Lines       15162      548    -14614
  Branches     2563      246     -2317
===========================================
- Hits         3863      439     -3424
+ Misses      11168      109    -11059
+ Partials      131        0      -131

Flag
Coverage Δ

adapter
?

common-controller
?

config-deployer-service
?

idp-domain-service
80.10% <ø> (ø)

router
?

Flags with carried forward coverage won't be shown. Click here to find out more.
see 122 files with indirect coverage changes

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


Can i use it with overleaf?

Good question. Currently we don't support using it with overleaf.
Installing latex locally is the well supported solution so far.
On Wed, Mar 10, 2021 at 4:47 PM Sey Eeet @.***> wrote:

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/xuehuichao/freemind-latex/issues/37, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAHBDDTNBWFGZQY2L2GDIFDTDAAKRANCNFSM4Y7GRAEA
.

Thanks for your reply, it will be legendary if you guys can make it usable with overleaf


cluster-capacity support other scheduler

such as gang-scheduler and kube-batch

such as gang-scheduler and kube-batch

We have not discussed about it, but that sounds like a good idea to me to have more scheduler options as part of cluster-capacity. Would you like to take it forward?

@aveshagarwal I think cluster-capacity framework is the best place to wrapper these differences.

/reopen

such as gang-scheduler and kube-batch

+1


Align to the golang package names standards

Move the code under /health (major release)

I don't think it requires a major release (still on 0.x), but I wonder why you would do that. Instead, I'd simply use the gosundheit name in the examples. I think in case of a library it's okay to have code in the repo root.

I don't think it requires a major release (still on 0.x), but I wonder why you would do that. Instead, I'd simply use the gosundheit name in the examples. I think in case of a library it's okay to have code in the repo root.

Was closed by #28

Was closed by #28


wlr-seat: keyboard redesign

fixes #287
fixes #242
I haven't tested it with multiple keyboards because I only have one right now, but that should work correctly.
Tests:

run a binding and see that the keyboard key event does not go to the window
unplug the keyboard and plug it back in and see that it works
plug in two keyboards and alternate typing in a window with them and notice that works
the above but each keyboard has a different keymap
press a modifier (shift) on one keyboard and then type on the other and notice the modifier is not considered pressed while you type on the other.

Works with two keyboards (laptop's built-in and normal).

Is this good to go?

I think it should be tested with the new keyboard config which I can do tomorrow.

Tested this on the new keyboard config with two different keyboard layouts, worked as expected.


Mac installer does not update PATH

Search first

[X] I searched and no similar issues were found

Description
Downloaded the mac installer today. Checked "yes" to the add to path option. When I start a new terminal (zsh) liquibase is not in my path.
At the very least I'd like the installer to tell me that adding to the path failed and give me an easy to copy line to update my shell profile with.
Steps To Reproduce
Covered above.
Expected/Desired Behavior
liquibase executable should be in path
Liquibase Version
4.23.2 (2023.09.11)
Database Vendor & Version
na
Liquibase Integration
cli
Liquibase Extensions
na
OS and/or Infrastructure Type/Provider
Mac Ventura 13.5
Additional Context
No response
Are you willing to submit a PR?

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

Hi @jcollum-nutrien, sorry for the late response.
The macOS installer doesn't currently have an "add to path" button, and what should be configured is the LIQUIBASE_HOME variable.
This is working as expected, so I'll be closing this issue. Feel free to leave a comment if the problem persists.
Thank you,
Tatiana


[FB] default_units: Update merge-base to commit 1873ce14

Update default units feature branch to base commit 1873ce1 (current head of master)
(Duplicate of #3792, but merging with --no-ff instead of --squash in order to preserve the commits)

The default_units feature branch merges into upstream/master without conflict when merged via the commandline:
git checkout upstream/master -b  default_units_base1873ce14
git merge --no-ff upstream/default_units

However, when trying to merge by creating a PR via GitHub, GitHub indicates a conflict in the file:  docs/iris/example_code/Meteorology/lagged_ensemble.py
which no longer exists in upstream/master.
As such, the conflict can not be resolved simply because there aren't any conflicts!
Therefore, I have created a new feature branch, upstream/default_units2, branching at merge-base 1873ce14.  I then merged in the original feature branch (as outlined above).  I confirmed that the docs/iris/example_code/Meteorology/lagged_ensemble.py does not exist (as in master).
When merged in default_units2, this should replace default_units, which will then merge into upstream/master correctly.
This will preserve the original commit-ids that were actually reviewed (rather than rebasing that alters the commits) and GitHub will preserve the issue links and commit information.

This is identical to #3792.  Comparing this default_units2 branch with master confirms that the 5 commits made to the feature branch have been preserved whilst updating the branch to head of upstream/master


Data types are rearranged after Data Types node refresh

System information:

Operating system (distribution) and version win10
DBeaver version 21.0.5 EA

Connection specification:

Database name and version PostgreSQL

Describe the problem you're observing:
Data types are rearranged  after Data Types node refresh (option is disabled in settings)

Fixed in DBeaver 21.1.1

verified


Actual JAR of the versions > 7.1.0 are not yet on search.maven.org

Software version (AuthzForce Core)
11.0.0
Platform JRE
1.8.0_151
Platform OS
Windows
Error stack trace or logs (if any)
Failure to find org.ow2.authzforce:authzforce-ce-core:jar:11.0.0 in https://repo.maven.apache.org/maven2
If in doubt include the whole thing; often exceptions get wrapped in other exceptions and the exception right near the bottom explains the actual error, not the first few lines at the top.
Your code and/or AuthzForce-specific configuration file(s)
none

Ok, my fault. The artifact I should depend on is: authzforce-ce-core-pdp-engine


Extending Sitemap with Url QueryStrings

Maybe I overlooked it, but is there an easy way to extend or augment the Sitemap generation with custom querystring parameters that are predefined?

Should be released now (package uploaded to nuget.optimizely.com).


Make connection label more expressive

Fixes https://github.com/mtlynch/tinypilot/issues/488

Usually I would say the DOM should reflect what you see on the page and not have “tons” of hidden things mounted behind the scenes. And my take on CSS is to use it for styling, not for view logic. That being said, however, I’m not quite sure yet whether these opinions can be transfered 1:1 to the approach with custom HTML components. So, very good question anyways… 🤔 I guess I need some more “getting-used-to” time to eventually make up my mind on this 😸
For now I’ve changed it, mainly because it’s more inline with how we do it elsewhere.

LGTM!

Usually I would say the DOM should reflect what you see on the page and not have “tons” of hidden things mounted behind the scenes. And my take on CSS is to use it for styling, not for view logic. That being said, however, I’m not quite sure yet whether these opinions can be transfered 1:1 to the approach with custom HTML components.

Yeah, that's fair. If it were Vue, I'd opt for conditional rendering so only one would be in the DOM, but I don't know of as clean a solution with VanillaJS. But I think you're right that it's not great to use CSS for view logic.

For now I think using CSS seems quite pragmatic at least, because creating anything more complicated than innerText is not as straightforward in vanillaJS as it is with FE libraries. To be fair, though, I haven’t done a lot of research around best practices with custom HTML components yet.


Bug: The execution record on the right shows that the execution is completed, but the list is not yet displayed.

Chat2DB Version
1.0.7
Describe the bug

Attached table structure

try refresh


Feature Request: API to get a family of entities.

I want a way to be able to easily get a family of entities with certain components that's decoupled from the systems.
Example:
        val familyContainer = world.familyContainer(
            allOf = ...,
            anyOf = ...,
            noneOf = ...,
        )
        val entities: List<Entity> = family.getEntities()

The family container should be automatically updated on addition/removal of entities and addition/removal of their components.

Hi @Kietyo ,
I just released a version yesterday that contains this feature. Here is a snippet from the ReadMe:
fun main() {
    val world = World {}
    val e1 = w.entity {
        add<MoveComponent> { speed = 70f }
    }
    val e2 = w.entity {
        add<MoveComponent> { speed = 50f }
        add<DeadComponent>()
    }
    val e3 = w.entity {
        add<MoveComponent> { speed = 30f }
    }

    // get family for entities with a MoveComponent
    // and without a DeadComponent
    val family = world.family(
        allOf = arrayOf(MoveComponent::class),
        noneOf = arrayOf(DeadComponent::class),
    )

    // you can sort entities of a family
    val moves = world.mapper<MoveComponent>()
    family.sort(compareEntity { entity1, entity2 -> moves[entity1].speed.compareTo(moves[entity2].speed) })

    // And you can iterate over entities of a family.
    // In this example it will iterate in following order:
    // 1) e3
    // 2) e1
    family.forEach { entity ->
        // do something with the entity
    }
}

Does this cover your needs for this use case?

Thanks! This does, are you supporting the korge-fleks version BTW? Cause that's the version I'm using:
https://github.com/korlibs/korge-next/tree/main/korge-fleks

I was afraid that you say that 😅
The KMP Version was done by @jobe-m. There were a few updates since then in the JVM version and I have of course plans to integrate that into the KMP Version as well but this will take time, sorry.
Any help is appreciated! I myself try to update that version by end of June.
I need to make a small release to the JVM version again this week for some Bugfixes and I try to support your Family listener idea. This version will then be the base for the next KMP Version as well.
Thank you for your input and suggestions! And sorry for the delay with the KMP part. This will be easier in the future once those two versions get merged into one 😉

I see, I've started contributing to the korge-fleks version actually... fyi
https://github.com/korlibs/korge-next/pull/715

Thanks for notifing me here 👍
I will update the korge-fleks copy with @Quillraven's latest changes to get it in sync again.
After that I plan to discuss if and how we upsteam the changes from https://github.com/korlibs/korge-next/pull/715 to the KMP and main branch here.

I think in the next branch I made a proper KMP gradle setup and I was able to add it to KorGE as a normal dependency as mentioned in the ReadMe. I suggest that this is the way to go but up for discussion :)

I like the proposal, thanks!

The proposal looks good!
BTW yesterday I merged the changes from master branch into the next branch. After running the unit tests locally I will create a PR.

@Kietyo : Please have a look at #50 and try out the next branch if that suits your needs.


Fix serialization of UPath

Currently, pickled instances of UPath will be un-pickled as pathlib.PosixPath. This PR fixes that by implementing __reduce__ and __setstate__ methods.
Fixes #44

Thanks for the PR, @normanrz! I made a comment to get the CI/CD tests passing. If you make that change, I get this merged ASAP.

I fixed the tests. I also refactored the / (i.e. __truediv__) operator so that the following two expressions produce the same representation. Previously, there were differences in _parts and _url.path, which are cloaked in _format_parsed_parts.
path_a = UPath("gcs://bucket/folder")
path_b = UPath("gcs://bucket") / "folder"

Also, the mock: tests gave me a bit of trouble on Windows, because the file path that is used is Windows-style path, but UPath implements _posix_flavour, which expects a / at the beginning in some methods.

I also refactored the / (i.e. __truediv__) operator so that the following two expressions produce the same representation. Previously, there were differences in _parts and _url.path, which are cloaked in _format_parsed_parts.
path_a = UPath("gcs://bucket/folder")
path_b = UPath("gcs://bucket") / "folder"

Great! Thank you!

Also, the mock: tests gave me a bit of trouble on Windows, because the file path that is used is Windows-style path, but UPath implements _posix_flavour, which expects a / at the beginning in some methods.

Interesting. I'm surprised that hasn't shown up on the CI/CD windows tests. Thanks for adding the fix though. I'll go ahead and merge this.

Thanks for merging! Could you please also create a new pypi release?

Thanks for releasing!


Produce Aggregate Estimate NI

-Once #270, #288 & #306 are complete we will be able to produce an estimate for NI per local authority.

It also might be worth changing the read_ni_stops function to just outputting a pandas dataframe as another function we need to use converts it into a geopandas dataframe.

There is a current issue with the code - when I run it, it's not finding the stops inside the polygon for the local authority and it can't find the population total.

Summary of discussion with @nkshaw23 :

The coordinates system for Easting and Northing (that was currently being used for NI) is incorrect for both trains and buses NI data.
Latitude and Longitude will be used instead as the projections for this are correct. This will then be converted into the correct coordinates system. However, this is not present in the NI trains csv file, and only extracted from the geojson file. We have decided to produce a hard CSV copy and upload it onto sync.
Commas present in pop_count or other columns will need to be removed from all cells so we can change the dtype to int.
We are going to correctly populate easting and northing coordinates for both bus and train stops so we have 2 coordinate systems to use.
Going to tidy the stops_geo_df so it is a clean copy for potential future use.
Have decided not to read in NI bus stops data via a URL link because the URLs tend to update frequently (this has happened before), so it cannot read in the data correctly and will create a bug in our code. Instead, the bus stops will be stored on sync.

Note to self: I didn't pull the commits from master so I need to pull all commits from master then rebase the branch.


feat(Combobox): Add prop to send initialValue

Resolves #1698

Why not just set initialValue as initial value in useState for selectedOptions?
Don't think we need to filter out valid values for and introduce another useEffect?

selectedOptions is of type Options[], meaning we need to go over the string value and find the associated Options:
type Option = {
    value: string;
    label: string;
    displayValue?: string | undefined;
    description?: string | undefined;
}


directx opencl interop samples (with copy from opencl image2d to buffer)

a simplified version of directx-opencl interoperability samples (should replace existing samples if everithing is ok)

@alalek Alexander, could you please review the request. Thanks.

@alalek
Alexander, please note, if samples looks ok, they should replace existing one (I intentionally did not replace them before review)

@vladimir-dudnik
Vladimir, it would be good adding a short description for each file (the sample demonstrates DirectX-OpenCL interoperability ...).

@alalek @SergeySivolgin please check modified version

Looks much better, thanks!
BTW, multiline macros should be avoided because it is almost impossible to debug.
Implementation of static void help() in the header is enough.
For main this declaration should work:
template <typename App>
void d3dsample_main(int argc, char** argv, const std::string& title)
{
    ...
}

:+1:

@vladimir-dudnik I would still recommend to add comments in the beginning of each file as a short description what each sample does.

I agree with @SergeySivolgin - please, add some comments at the top of each file and in key parts, where you actually do the initialization, processing and rendering.


Add formula DSL support for omitting install name rewriting for @rpath/* install names

Verification

[X] This issue's title and/or description do not reference a single formula e.g. brew install wget. If they do, open an issue at https://github.com/Homebrew/homebrew-core/issues/new/choose instead.

Provide a detailed description of the proposed feature
Currently, we rewrite all library install names to something like opt_prefix/library_install_name.basename.
https://github.com/Homebrew/brew/blob/e62a839001faf69160965c352b66870006c2af65/Library/Homebrew/extend/os/mac/keg_relocate.rb#L24-L27
Typically, these will be because the install name refers to a Cellar path, e.g., prefix/library_install_name.basename. The install name rewriting allows us to avoid unnecessary rebuilds of dependents with version/revision bumps.
This install name rewriting isn't needed when the install name starts with @rpath, because these will typically not hardcode a Cellar path reference that will need to be rebuilt on version/revision bumps.
Formulae should support some sort of DSL that makes the code in keg_relocate.rb skip rewriting an install name if it starts with @rpath.
What is the motivation for the feature?
Our rewriting of @rpath-prefixed install names breaks macdeployqt. See Homebrew/discussions#2823.
This is arguably an upstream bug, but convincing upstream to fix it is unlikely to succeed without a reproduction of the issue outside of Homebrew, and I don't think anyone has the time or energy to do this. Even if one had such a reproducer, it would likely still be rather difficult to convince upstream to fix this short of providing a patch.
Such a DSL could also replace a very old workaround in Rust:
  def post_install
    Dir["#{lib}/rustlib/**/*.dylib"].each do |dylib|
      chmod 0664, dylib
      MachO::Tools.change_dylib_id(dylib, "@rpath/#{File.basename(dylib)}")
      MachO.codesign!(dylib) if Hardware::CPU.arm?
      chmod 0444, dylib
    end
  end

How will the feature be relevant to at least 90% of Homebrew users?
It probably won't be, but it would likely be relevant to our users who use Qt, which form a non-negligible fraction of our user base.
What alternatives to the feature have been considered?
See alternatives described in this comment.

Also, to explain why we want an additional DSL rather than just skipping the rewriting of @rpath/* install names entirely:
Typically, these will make linking with libraries harder. For example, if libfoo.dylib has an absolute path for its install name, linking with libfoo only requires
-L$HOMEBREW_PREFIX/lib -lfoo

However, if libfoo has @rpath/libfoo.dylib for its install name, one would need some variation of
-L$HOMEBREW_PREFIX/lib -lfoo -Wl,-rpath,$HOMEBREW_PREFIX/lib

which is likely to trip up many users and result in may issues/discussion posts created.

Thanks @carlocab. Agree with everything you've said here and the desire for this for both Qt and Rust.

@carlocab May I ask for the progress of this PR? If this fixes the "Cannot resolve rpath" errors described here, it would help us a lot to get LibrePCB migrated to Qt6. Currently this task is blocked by the broken deployment with macdeployqt.

What's the status of this?

@pawsaw We're waiting for someone to implement it.

Here's a case that doesn't involve @rpath: Swift ships dylibs that point to /usr/lib/swift/libswift* rather than the installed file. This is intentional due to the unique way that Swift ships with the OS but new features can still be back deployed to older OS.
My initial thought was to skip anything that didn't start with HOMEBREW_PREFIX or the build directory. @rpath however is hard and do acknowledge the DSL argument here.


Error while starting notion-app

When I start the notion-app, I get this error message:
Error: Error: ENOENT: no such file or directory, open '/opt/Notion/resources/app-update.yml'.
I am using Fedora 38 and I installed it from the yum repository.

Try creating a text file in the /opt/Notion/resources folder and naming it app-update.yml

After creating the file I get the following error message:
 Error: TypeError: Cannot read property 'provider' of undefined     at createClient (/opt/Notion/resources/app.asar/node_modules/electron-updater/out/providerFactory.js:69:25)     at /opt/Notion/resources/app.asar/node_modules/electron-updater/out/AppUpdater.js:479:99     at async AppImageUpdater.getUpdateInfoAndProvider (/opt/Notion/resources/app.asar/node_modules/electron-updater/out/AppUpdater.js:482:20)     at async AppImageUpdater.doCheckForUpdates (/opt/Notion/resources/app.asar/node_modules/electron-updater/out/AppUpdater.js:503:20)     at async pollForElectronUpdates (/opt/Notion/resources/app.asar/main/autoUpdater.js:163:17)

Right, sorry - just double checked, the auto updater only works on Windows, so that file was intentionally left non-existent. Delete the file and you should be able to ignore the ENOENT error safely. If you're having infinite loading issues, they're unrelated to this error message, please see https://github.com/notion-enhancer/notion-repackaged/issues/116


Named locations table

Example query results
Resolving issue - https://github.com/turbot/steampipe-plugin-azuread/issues/190

Results
select * from azuread_conditional_access_named_location

![table_named_location](https://github.com/user-attachments/assets/7ce5963a-1017-4de9-bf46-8c4e97c3a16c)

Hi @TheRealHouseMouse, thank you for raising the PR. Could you please address the linting issues before we proceed?

@TheRealHouseMouse could you also add the missing table docs as well?
Thanks!!

Fixed the linting errors and added table docs :+1:


Improving documentation

Hello,
I would like to propose some suggestions to improve the current documentation:

in file doc/installation_windows.md

This guide is about compiling instead that installing so why not change the title into "Compiling Instructions" instead that "Windows 8.1 & Windows 10 Installation"?
Compiling librealsense with Visual Studio require "Visual C++ ATL support" installed, also this can be added

in file doc/rs400/rs400_advanced_mode.md color correction is cited. In code there are 12 float parameters colorCorrectionN that seem coefficients of a 3x4 matrix, how are they used by ASIC? Are for RGB or IR?

add a doc for Realsense Viewer that explain or give references to all parameters available

What do you think?
Thanks

@TheMikeyR Thank you Mikey, I will make a pull request. Could you give me some info about color correction or a reference to the DS5 (Vision Processor D4) programming guide with explanation of these registers?

I have just found that:
Color Correction Matrix is used by Intel Realsense to remove IR pattern (only works with D415 at 1280x720 resolution)
Just set these values:
ColorCorrection = [ 0.520508, 1.99023, 1.50684, -2, -2, -0.0820312, 1.12305, 1.01367, 1.69824, -2, 0.575195, -0.411133 ]
Very interesting..
Source:
Best-Known-Methods for Tuning Intel® RealSense™ D400 Depth Cameras for Best Performance

[Realsense Customer Engineering Team Comment]
@electro-logic
Yes, the color correction is used remove the IR pattern, there will be more documents available in the future can be found in web page https://realsense.intel.com/intel-realsense-downloads/


WIP: switch to callAst method in x2cpg in javasrc

WIP because some dataflow tests are failing with the new structure where there is no longer an argument edge to the receiver of the call. I'm leaving this aside for now to avoid possible conflicts with work @fabsx00 is doing on the dataflow engine.

I'll close this for now and just include the changes in a larger javasrc cleanup PR later


DnsNameResolver hangs for 5 seconds in InetAddress.getAllByName0 on Mac OSX

Expected behavior
DnsNameResolver constructor should be fast, anything slow offloaded until actually used.  Or a warning given that environment in badly configured.
This is an environmental problem, but raising because its a regression from java DNS which doesn't hang like this.
Feel free to close if you don't want to worry about it in Netty.
Actual behavior
constructor hangs at program startup
Steps to reproduce
    System.out.println("started " + new Date());

    ThreadFactory threadFactory = new DefaultThreadFactory("netty", true);
    NioEventLoopGroup eventLoopGroup = new NioEventLoopGroup(1, threadFactory);

    DnsNameResolverBuilder builder = new DnsNameResolverBuilder(eventLoopGroup.next())
            .channelType(NioDatagramChannel.class)
            .optResourceEnabled(false)
            .maxQueriesPerResolve(3)
            .recursionDesired(true);

    DnsNameResolver re = builder.build();

    System.out.println("finished " + new Date());

started Sat Feb 25 13:17:16 GMT 2017
finished Sat Feb 25 13:17:21 GMT 2017
Fix
Fix is http://justthesam.com/2016/10/fixing-java-net-inet6addressimpl-lookupallhostaddr-slowdown/
Netty version
4.1.8.Final
JVM version (e.g. java -version)
$ java -version
java version "9-ea"
Java(TM) SE Runtime Environment (build 9-ea+157)
Java HotSpot(TM) 64-Bit Server VM (build 9-ea+157, mixed mode)
OS version (e.g. uname -a)
Darwin xxx 16.4.0 Darwin Kernel Version 16.4.0: Thu Dec 22 22:53:21 PST 2016; root:xnu-3789.41.3~3/RELEASE_X86_64 x86_64
"main" #1 prio=5 os_prio=31 tid=0x00007f8c60809800 nid=0x1d03 runnable [0x00007000027a3000]
   java.lang.Thread.State: RUNNABLE
	at java.net.Inet6AddressImpl.lookupAllHostAddr(java.base@9-ea/Native Method)
	at java.net.InetAddress$PlatformNameService.lookupAllHostAddr(java.base@9-ea/InetAddress.java:921)
	at java.net.InetAddress.getAddressesFromNameService(java.base@9-ea/InetAddress.java:1501)
	at java.net.InetAddress$NameServiceAddresses.get(java.base@9-ea/InetAddress.java:840)
	- locked <0x00000006cfce83e8> (a java.net.InetAddress$NameServiceAddresses)
	at java.net.InetAddress.getAllByName0(java.base@9-ea/InetAddress.java:1491)
	at java.net.InetAddress.getLocalHost(java.base@9-ea/InetAddress.java:1623)
	at sun.management.VMManagementImpl.getVmId(java.management@9-ea/VMManagementImpl.java:144)
	at sun.management.RuntimeImpl.getName(java.management@9-ea/RuntimeImpl.java:59)
	at jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(java.base@9-ea/Native Method)
	at jdk.internal.reflect.NativeMethodAccessorImpl.invoke(java.base@9-ea/NativeMethodAccessorImpl.java:62)
	at jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(java.base@9-ea/DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(java.base@9-ea/Method.java:547)
	at io.netty.channel.DefaultChannelId.defaultProcessId(DefaultChannelId.java:121)
	at io.netty.channel.DefaultChannelId.<clinit>(DefaultChannelId.java:78)
	at io.netty.channel.AbstractChannel.newId(AbstractChannel.java:107)
	at io.netty.channel.AbstractChannel.<init>(AbstractChannel.java:79)
	at io.netty.channel.nio.AbstractNioChannel.<init>(AbstractNioChannel.java:84)
	at io.netty.channel.nio.AbstractNioMessageChannel.<init>(AbstractNioMessageChannel.java:42)
	at io.netty.channel.socket.nio.NioDatagramChannel.<init>(NioDatagramChannel.java:147)
	at io.netty.channel.socket.nio.NioDatagramChannel.<init>(NioDatagramChannel.java:115)
	at jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance0(java.base@9-ea/Native Method)
	at jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance(java.base@9-ea/NativeConstructorAccessorImpl.java:62)
	at jdk.internal.reflect.DelegatingConstructorAccessorImpl.newInstance(java.base@9-ea/DelegatingConstructorAccessorImpl.java:45)
	at java.lang.reflect.Constructor.newInstance(java.base@9-ea/Constructor.java:473)
	at java.lang.Class.newInstance(java.base@9-ea/Class.java:558)
	at io.netty.channel.ReflectiveChannelFactory.newChannel(ReflectiveChannelFactory.java:38)
	at io.netty.bootstrap.AbstractBootstrap.initAndRegister(AbstractBootstrap.java:321)
	at io.netty.bootstrap.AbstractBootstrap.register(AbstractBootstrap.java:235)
	at io.netty.resolver.dns.DnsNameResolver.<init>(DnsNameResolver.java:249)
	at io.netty.resolver.dns.DnsNameResolverBuilder.build(DnsNameResolverBuilder.java:347)

The more I think about it, the more I realise it is
a) actually related to defaultProcessId
b) kind of funny that Netty DNS relies on built in java DNS

Working around with
-Dio.netty.processId=$$

WIll take a look

adding system property
-Djava.net.preferIPv4Stack=true
should help avoid the problem

The current method of pulling the process id is a bit of a hack since it relies on VMManagementImpl.getVmId however this now seems to be the common way of obtaining it for a lot of projects < JDK9..
    public String getVmId() {
        int var1 = this.getProcessId();
        String var2 = "localhost";

        try {
            var2 = InetAddress.getLocalHost().getHostName();
        } catch (UnknownHostException var4) {
            ;
        }

        return var1 + "@" + var2;
    }

    private native int getProcessId();

Java 9 provides an API to pull the process id.
long pid = ProcessHandle.current().getPid();

@normanmaurer not sure there is much that can be done apart from time how long it takes and issue a warn if it exceeds a threshold.. or implement pulling the pid via Java 9 API using reflection, that said this seems like a machine misconfiguration not an issue with Netty.. but just so you know @yschimke, Norman loves contributions ;)

I'm approved at work for my own projects, Finagle, OkHttp but didn't ask for Netty.  I don't have that much spare time so it's not worth the CLA effort just for this fix.

BTW I agree about the misconfiguration, I fixed this on my machine and also commented with a workaround.
Two things set off my spidey sense

its something that is affecting Netty performance for some # of Mac users/developers.  potentially this could impact some GUI client software run on end users Macs.
When you choose to use Netty DnsNameResolver and google public servers, having it block because of your local Java default dns configuration seems weird.

It looks like an Mac OSX issue (resolution of the local host must happens inside java.lang.management.RuntimeMXBean#getName() causing the delay). The StackOverflow solution remove the delay.

Hi, I'm experiencing similar issue with netty 4.1.12, on macOS sierra, when binding to the loopback using ServerBootstrap.  After a few debugging, this seems to be related to defaultProcessId() in DefaultChannelId, which takes longer than expected.

Let me close this issue for now as I think with PR https://github.com/netty/netty/pull/6592 we should no longer require JDK DNS resolution as part of codec-dns's initialization


Ignore corrupt/in use tiles.

Currently Martin stops in case of corrupt tile file or when tile file is in use (being created by tippecanoe).
It would be better to console log this as a warning, ignore the tile and continue running the tile server.

To make reproduction less time intensive:
What input files are you using and what is the error you get?

Update: I reproduced this via the example file from https://docs.mapbox.com/help/glossary/mbtiles/ (have submitted feedback about this, no need to report this to them)

@nyurik In case of an unrecoverable permanent error like this, failing obvious is better imo => what do you think?

Thx for tracking it down!  I agree that early failure is preferred, rather than having the service come up but unable to serve from one of its services.  Note that this is different from the sporadic issues, e.g. network connection going down to a database or network issue with pmtiles - i.e. if we have a chance to recover later.  I do think all sources should function during startup...  Unless we want to introduce --allow-unavailable-sources ?

Let's not add this complexity for corrupted mbtiles files.
For locked ones though, I think retrying like proposed in this PR would be reasonable

#1545

This is what is currently looks like:

My use case is that I am generating a bunch of tiles dynamically while the app is already running. Ideally Martin would watch the directory and load the source when it is available and not corrupt (watch mode).
Validating every source and stopping Martin from starting seems a bit too strict to me. For example a web server/Plex/Winamp etc. will still start even if you have a corrupt file (source) in the directory that is being served.
I understand that early failure might be a design decision for Martin and my use case is out of scope - in that case I can just figure out a workaround or fork the code. Thank you!


user settings: Display avatar source.

For issue  #8225:
Approach: To see if the Gravtar is custom or generated, the method is  to add the get parameter d=404 to the url gravatar. If it returns a 404 status instead of a generated image if the user does not have an image on gravatar set, and it is displaying the custom avatar.
I have done a quick implementation of this by writing this function and I am unsure about a few things mentioned below which I require thoughts on (and hence I haven't added the tests.)

I have implemented this by calling the function mentioned about (check_custom_gravatar) every time an avatar related event occurs (fetching initial state, upload), Hence a request is send everytime to the gravatar url to check if it returns 404. We have to do this every time while fetching the initial state to detect if the user hasn't updated their gravatar and the gravatar is not custom anymore. (Hence a request goes out everytime, which is not good practice.) The two solutions I am confused with are below:

solution 1 : Add custom_gravatar as a field in UserProfile model, and only check if the gravatar is custom when the events upload and delete occur. ( Hence a request won't go out everytime the initial state is fetched and only when the mentioned  events occur.)
solution 2: Avoid the entire thing by displaying Sourced from Gravatar everytime without checking if it is custom.
Also need thoughts about the UI:
Current implementation: (only for a uploaded gravatar, for a custom gravatar, there would be no Sourced from Gravatar text. (This also shows solution 1: Displaying text only when avatar is not custom)

Here, always displaying the text below, looks kinda ugly. So we can add it only on image hover, like below.  (This also shows solution 2: Always displaying the text)

@rishig FYI

Thanks @rheaparekh!
Showing on hover is a great idea. And in that case I think we can do the simple thing, of showing the text whether or not it is custom.

Updated version:

This looks great; thanks!

Otherwise lgtm, aside from maybe that we might want to change the text "Sourced from Gravatar" to be something more standard, e.g. "Avatar from Gravatar".

Yeah, let's do this "Gravatar for<EMAIL_ADDRESS>We can make the link be a direct link to their user's gravatar profile, perhaps?  Or maybe to an appropriate documentation page.

I have made the suggested changes. This is the current text displayed which hyperlinks to gravatar url for iago :

Looks good, except the line-wrapping isn't right for if the email is reasonably long (e.g<EMAIL_ADDRESS> It's easy to check by just editing the HTML (or you can change the user's email).
Also, I think maybe the link should just go to https://en.gravatar.com/site/check/ ?

@timabbott That review is quite helpful, teaches me how to handle border cases! What would we want to do for too long emails (Just checked, maximum length of an email address is 254 characters.) We could wrap to the next line, but I think Avatar from Gravatar (hyperlinked) would be better in this case. What is your opinion?

Can we just use text-overflow?

@rishig we can; it's sorta annoying to make things take up space upwards when doing that, but possible.    The problem is I'm not sure the result with any version of wrapping actually would actually look good, and the current sizing is such that many emails would wrap.

cool. Avatar with Gravatar also sounds good.

I have updated this PR! I have used Avatar from gravatar (lowercase g) as I got the capitalisation error.
Also I have used the link https://en.gravatar.com/.

Merged, after changing the spelling to capitalize Gravatar (since that's how they do it on their website).  Thanks @rheaparekh!

Thankyou  @timabbott and @rishig for the review!


[Improvement] Home Assistant integration

I've recently come across this project and I enjoy using it to apply a pseudo-random color palette to my room (I like the Hue scenes but of course those aren't available for non-Hue lights) but I'm missing a way to apply those scenes programatically/without leaving the HA UI (no offense but the custom panel is ugly and doesn't remember my choices). It'd be awesome if this project exposed a service to call with the entities/area which it should apply one of the scenes to.

HA integration like that is going to be a bit tricky, since this is entirely a front-end app.
However, I've been thinking about adding a button that'd let you save a palette as a scene you could then recall later with the usual HA stuff -- however, it wouldn't be randomized/varied, then. Would that work for you?

It is dense, though I feel like the space isn't being used wisely efficiently (I've highlighted some "empty space").

I wonder if the space limitation would be less problematic if the UI was more scroll-oriented and less "fit everything at once"-oriented. I can't say whether it's better to show all controls at once or to have a less cramped UI, I'm not a UX designer (though I work with one almost every day)


Add test for randomProfile feature

The test is rather simple but I'd like it to serve as an example how you can write DOM tests.
This test doesn't use the browser, so there's no way to check actual visible changes. It checks if the modified "source" of the page is correct (menu item added) and checks if clicking the menu item runs the correct code.
This test uses the Tolkien-1 profile. The profile page is saved as Tolkien-1.html in the repository and it is not automatically in sync with the live wikitree.com site. To update a new version of the Tolkien-1.html file, run npm run update-test-pages. Note that you need to log in with a WikiTree account to update this file because the page looks different for logged in users. In particular, not-logged-in users don't see the "Find" menu.

I'm sorry, I fixed the conflict now.


Show an information dialog if settings can not be accessed

From verifying #10317:

remove permissions to read ~/.config/Code - Insiders/storage.json
start VSCode
VSCode starts up without error message. No folder history, but all seems to work.

Starting code --verbose shows the issue
A warning dialog or message (also without verbose) would be good so the user can fix the permission issue.

We closed this issue because we don't plan to address it in the foreseeable future. If you disagree and feel that this issue is crucial: we are happy to listen and to reconsider.
If you wonder what we are up to, please see our roadmap and issue reporting guidelines.
Thanks for your understanding, and happy coding!


[request] Support generic S3 endpoints

Please describe problem to be solved
Storage-specialized cloud providers like blackblaze and wasabi offer cheaper storage and s3 endpoints. Skyplane should
Solution

Add programmable gateway support via #792 (in-progress)
Support one-sided transfer plans
Add generic S3 endpoint connector
Support authentication with other providers (e.g. Wasabi)

Use-case
One use-case a potential user described was being able to transfer 30TB between two Wasabi regions.

Another use case is transferring data to private OpenStack implementations in public HPC clusters like Compute Canada or Jetstream2. I am currently trying to export 20TB of 100 million small objects from AWS S3 to OpenStack's Object Storage using AWS DataSync, which is taking multiple days in the "preparing" phase even with 3 agents running in parallel.


OCI Cursors

What steps will reproduce the problem?
PLS-00382: expression is of wrong type
`
$c = oci_connect($host, $pass, $db) or die;
// set cursor
$refcur = oci_new_cursor($c);
$q = "BEGIN :RC := REPORT_CARDS.GET_DATA(:I_YEAR); END;";
$data = Yii::$app->db->createCommand($q)
                     ->bindParam(':I_YEAR', $s, PDO::PARAM_INT)
                     // will this line below work with Yii2?
                     ->bindParam(':RC', $refcur, NULL, -1, OCI_B_CURSOR)
                     ->execute();`

What is the expected result?
That data is returned, which it is using plain PHP.
`$stid = oci_parse($conn, "BEGIN :RC := REPORT_CARDS.GET_DATA(:I_YEAR); END;");
// get cursor
$refcur = oci_new_cursor($c);
$I_YEAR = 2014;

// Bind the input parameter
oci_bind_by_name($stid,':I_YEAR', $P_YEAR, 4);
oci_bind_by_name($stid, ':RC', $refcur, -1, OCI_B_CURSOR);

oci_execute($stid);

oci_execute($refcur);
oci_fetch_all($refcur, $res);
var_dump($res);`

What do you get instead?
PLS-00382: expression is of wrong type
A couple of things are:
1 - I'm unsure how to call oci_new_cursor() with my existing DB connection, I tried Yii::$app->db, but it failed? So I just hard coded it into a new oci_connect().
2 - How to bind the refcursor properly within Yii2? Is it possible or do I need to use plain PHP here?

https://www.google.ru/search?q=PLS-00382:+expression+is+of+wrong+type&ie=utf-8&oe=utf-8&gws_rd=cr&ei=mokrV9mCNMuwsQGSnqL4BQ

@lynicidn I have looked at it through Google. I still can't bind the $refur param using ->bindParam(':RC', $refcur, OCI_B_CURSOR, -1)

first link contains
Change the datatype of sqlCommand from NVARCHAR2 to varchar2. Execute immediate expects varchar, varchar2 or char. Code given below should work

@lynicidn So the $refcur needs to be a PDO::PARAM_STR? Why?

don't know, but it problem not in yii

It is if oci_bind_by_name() can't be used within createCommand() and needs a feature request.

http://php.net/manual/en/function.oci-bind-by-name.php
look here

ah yeah in context yii in can not work you not set cursor on execute

u use http://php.net/manual/ru/ref.pdo-oci.php ?

@klimov-paul Can I feature request this and a fuller OCI set of tools?


Adding placement="top" by default causes unnecessary attribute mutations

When the user has not provided any placement of the tabs intentionally, the component adds the placement="top" to the host element when connected to DOM. This causes an extra call to attributeChangedCallback lifecycle method which is unnecessary.
Moreover, altering the host element internally it might be confusing to the users that consume the component.

Fixed by PR #6


Support ARM computers

Push ARM versions of the docker images so that this workspace can run natively on ARM computers like the M1 Macbooks.
Resources

https://www.docker.com/blog/multi-arch-build-and-images-the-simple-way/

Not a priority until we have someone on the team that uses an ARM computer

@hhenry01 can we use docker containers on the main computer?

Yes, but there's not really necessary. Ideally we would compile the code on our own computers and copy the binaries over.

Resolve blockers:

[ ] Installing OMPL with Python bindings: see issue 958 in ompl repo

Resources for building multi-arch docker images

https://www.docker.com/blog/multi-arch-build-and-images-the-simple-way/
https://billglover.me/notes/build-multi-arch-docker-images/


Drake Companions too big to sneak from level 1

Describe the bug
Drake Companions have the skill penalties from size as if they are huge from level 1
To Reproduce
Steps to reproduce the behavior:

Make a Draconic Shaman, Draconic Druid, Silver Champion, or Claw of the False Wyrm
Note the size of the Drake unit. Appears to be Small or Tiny, roughly the right size.
Level them up to 1, while setting their skills, hover over Stealth.
Note a -8 Size penalty in place of the +8 size bonus to Stealth.

Expected behavior
Tiny creatures should have a +8 on Stealth checks.
Screenshots

Additional context
Appears to happen with all Drake Companion types, tested explicitly with Silver, Red, Green, and Umbral.

Correction, this appears to be fixed outside of the creation/level up screens.

Correction, this appears to be fixed outside of the creation/level up screens.


Refactoring.nvim not work with typescript

When trying to use the plugin with typescript, I have following error

Extract Function: give me the error: Current selected region is empty, have to provide a non empty region to perform a extract func operation. I already in Visual mode, and select line 2 and 3.
Extract variable: give the error : block_scope is nil! Something went wrong. I was in visual mode, and selected 2 + a
Extract Block : work well

Code test
function testFunc(): number {
  let a = 2;
  console.log(a);

  return 1 + 2 + a;
}

var x = testFunc();
var y = x + 3 + 4;

config (with LazyVim)
  {
    "ThePrimeagen/refactoring.nvim",
    opts = {},
    config = function(_, opts)
      local refactoring = require("refactoring")
      -- local map = require("user.util").map

      refactoring.setup(opts)

      vim.keymap.set({ "v", "n" }, "<leader>rr", refactoring.select_refactor, { noremap = true })
      -- vim.keymap.set("n", "s", require("substitute").operator, { noremap = true })
    end,
    dependencies = {
      "nvim-lua/plenary.nvim",
      "nvim-treesitter/nvim-treesitter",
    },
  },

Did I make a mistake?
The interesting part is "sometime" it works, but almost time, the error raised, especially when I have just start my nvim

Ditto here. Seems like if I run one of the Ex commands, the mapping starts to work, but otherwise I get these errors.
-- Using lazy.nvim
  {
    "ThePrimeagen/refactoring.nvim",
    keys = {
      {
        "<leader>rr",
        function()
          require("refactoring").select_refactor()
        end,
        mode = { "n", "x" },
        desc = "Refactor",
      },
    },
    config = true,
    dependencies = { "nvim-lua/plenary.nvim", "nvim-treesitter/nvim-treesitter" },
  }

This should be solved by this commit. Could you try it?
It looks like vim.ui.select exits visual mode without setting the visual mode marks (< and >), so we need to exit visual mode before calling it in order to set the marks.

This should be solved by this commit. Could you try it?
It looks like vim.ui.select exits visual mode without setting the visual mode marks (< and >), so we need to exit visual mode before calling it in order to set the marks.

Thanks for the fix.
Now "Extract function" work well
However, a new problem raised with "Extract variable" . I tried to extract "2+a" to a variable named "temp" and here is the result.
Sure, it's not I intended to do
function testFunc(): number {
  let b = 1;
  let a = 2 + b;

  console.log(a);

  const temp = 1+2+a;
  return temp;
}

Seem there some problem when determine the VISUAL region

That is kind of a limitation of treesitter (as far as I know). Currently, the Plugin asks Neovim for a node containing the visual selection and, because how nodes of binary operations work, the smallest node containing the visual selection may be the whole right side of the variable assignment.
I've been trying to use a different method for extract var but im currently stuck. It's posible to get the sexpression of each node in the selection (for example, a number, a + and an identifier`). But, we then would need to query treesitter for each permutation of the three nodes being either siblings or two of them being siblings and one of them being part sibling of a parent node (a binary operation, for example) and thus would be exponentially more difficult for each node on the visual selection.
I would choose this issue, could you open a new one copying this other error to keep track of it?

Sure. I will create new one.
Thanks


For some reason that I did not identify the SwipeToDeleteItem only works on the computer.

Description of the Bug
For some reason that I did not identify the SwipeToDeleteItem only works on the computer.
I click and hold the item, drag it to the left and it does not work on android, only on the computer.
Code and Logs
Code extracted from End full code
Before sending this model of the documentation I took care to package it for testing and the same problem occurs.
from kivy.lang import Builder
from kivy.properties import StringProperty

from kivymd.app import MDApp
from kivymd.uix.card import MDCardSwipe

KV = '''
<SwipeToDeleteItem>:
    size_hint_y: None
    height: content.height

    MDCardSwipeLayerBox:
        padding: "8dp"

        MDIconButton:
            icon: "trash-can"
            pos_hint: {"center_y": .5}
            on_release: app.remove_item(root)

    MDCardSwipeFrontBox:

        OneLineListItem:
            id: content
            text: root.text
            _no_ripple_effect: True

MDScreen:

    MDBoxLayout:
        orientation: "vertical"
        spacing: "10dp"

        MDToolbar:
            elevation: 10
            title: "MDCardSwipe"

        ScrollView:

            MDList:
                id: md_list
                padding: 0
'''

class SwipeToDeleteItem(MDCardSwipe):
    text = StringProperty()

class TestCard(MDApp):
    def __init__(self, **kwargs):
        super().__init__(**kwargs)
        self.screen = Builder.load_string(KV)

    def build(self):
        return self.screen

    def remove_item(self, instance):
        self.screen.ids.md_list.remove_widget(instance)

    def on_start(self):
        for i in range(20):
            self.screen.ids.md_list.add_widget(
                SwipeToDeleteItem(text=f"One-line item {i}")
            )

TestCard().run()

Versions
OS: Ubuntu 20.04.2 LTS
Python: 3.8.5
Kivy: 2.0.0
KivyMD: 0.104.2.dev0, git-8f83652, 2021-05-16

The main issue is with MDList padding is 0 which means on the phone screen it is the very edge of the phone nearly impossible to  touch. The other issue I found is the default swipe_distance is too small to comfortably touch and move the slide.
This is the solution
from kivy.lang import Builder
from kivy.properties import StringProperty
from kivymd.app import MDApp
from kivymd.uix.card import MDCardSwipe

KV = '''
<SwipeToDeleteItem>:
    size_hint_y: None
    height: content.height
    swipe_distance: "200dp"

    MDCardSwipeLayerBox:
        padding: "8dp"

        MDIconButton:
            icon: "trash-can"
            pos_hint: {"center_y": .5}
            on_release: app.remove_item(root)

    MDCardSwipeFrontBox:

        OneLineListItem:
            id: content
            text: root.text
            _no_ripple_effect: True

MDScreen:

    MDBoxLayout:
        orientation: "vertical"
        spacing: "10dp"

        MDToolbar:
            elevation: 10
            title: "MDCardSwipe"

        ScrollView:

            MDList:
                id: md_list
                padding: 2
'''

class SwipeToDeleteItem(MDCardSwipe):
    text = StringProperty()

class TestCard(MDApp):
    def __init__(self, **kwargs):
        super().__init__(**kwargs)
        self.screen = Builder.load_string(KV)

    def build(self):
        return self.screen

    def remove_item(self, instance):
        self.screen.ids.md_list.remove_widget(instance)

    def on_start(self):
        for i in range(20):
            self.screen.ids.md_list.add_widget(
                SwipeToDeleteItem(text=f"One-line item {i}")
            )

TestCard().run()

This should be mentioned in the document please.

I fully agree! After your tips worked here too!


:sparkles: Update to Kubernetes 1.30

Summary
This updates kcp to Kubernetes 1.30.0. The corresponding branch for our Kubernetes fork is kcp-1.30. Mainly this PR adds new APIs or removes feature gates where they have been removed by upstream.
The bigger changes in this PR are:

Updating to the new ValidatingAdmissionPolicy plugin infrastructure.
Rewrite the OpenAPI controller we recently added to accommodate for new upstream types.

The following bigger TODOs exist, but I would like to tackle them separately:

Look at enabling the flags I've disabled in this PR, most notably --authentication-config, which seems like a hugely useful feature for a multi-tenant API server like KCP.
Write an e2e test for the new GA ValidatingAdmissionPolicy feature. I've tried wiring it up correctly, but I think we should have a test case for it (maybe we already have and I missed it, though).

Related issue(s)
Fixes #3068
Release Notes

Update to Kubernetes 1.30.0

🥳 🔥 🎆

/retest

/lgtm
/approve
/hold
@sttts wanna read through or fix on master? :)

Modulo the e2e style comments, lgtm.

Looks like the failing tests aren't a flake:
maximalpermissionpolicy_authorizer_test.go:232:
        	Error Trace:	/home/prow/go/src/github.com/kcp-dev/kcp/test/e2e/apibinding/maximalpermissionpolicy_authorizer_test.go:404
        	            				/home/prow/go/src/github.com/kcp-dev/kcp/test/e2e/apibinding/maximalpermissionpolicy_authorizer_test.go:232
        	Error:      	Received unexpected error:
        	            	cowboys.wildwest.dev "cowboy-e2e-workspace-fhj5z" is forbidden: ValidatingAdmissionPolicy 'policy-4lr9q' with binding 'binding-4vxvf' denied request: expression 'object.spec.intent != 'bad'' resulted in error: no such key: intent
        	Test:       	TestMaximalPermissionPolicyAuthorizer
        	Messages:   	error creating cowboy in consumer workspace "root:e2e-workspace-jmr78:e2e-workspace-fhj5z"

This isn't always triggered, but it needs to be addressed.

/retest
flake.

/lgtm

/hold cancel

/kind feature


Replot the graph when the user clicks Graph View after manually updating the graph range

Temporary solution to issue #1187
Description of the changes:

Replots the graph when the user manually updates the range in Reset Grid

How changes were validated:

Manually, add a graph manually move the range (pan, zoom, graph settings), add an equation, click the graph view button. Tested variations of these steps.

/azp run


Include artifact building scripts

Description

./hack/publish_build_from_tag.sh --publish  script to build bundle and catalog images based on previous operand tag and operator tag.   @bparees Used for internal-preview images.
./hack/rebuild_bundle_catalog.sh developer script to rebuild the bundle folder and build bundle and catalog images from source.
Makefile to have update-bundle-catalog to update the artifacts for bundle and catalog builds

Type of change

[ ] Refactor
[ ] New feature
[ ] Bug fix
[ ] CVE fix
[ ] Optimization
[ ] Documentation Update
[ ] Configuration Update
[ ] Bump-up dependent library

Related Tickets & Documents

Related Issue #
Closes #

Checklist before requesting a review

[ ] I have performed a self-review of my code.
[ ] PR has passed all pre-merge test jobs.
[ ] If it is a core feature, I have added thorough tests.

Testing

Please provide detailed steps to perform tests related to this code change.
How were the fix/results from this change verified? Please provide relevant screenshots or results.

/lgtm

/retest

/lgtm
/approve


HAWQ-1400. An improvement in SQL Util destructor function for feature test

This is an external improvement from test tool side to make more stable CI testing.

@huor @paul-guo- @stanlyxiang Please review, thanks.

Thanks. I think at least we should add comment here at first.

comment is added to explain the sleep

+1 for the fix. The clarification looks better to have root cause


rabbit.shutdown() promise never settles

I'm building a program, and I would like to cleanly shutdown the listener before shutting everything down.
In order, I first shutdown rabbit, then my database connection and then finally exit.
For shutting down, I use the following code:
console.log("Closing message listener");
rabbit.shutdown()
  .then(() => {
    console.log("Message listener closed");
    // close database connection
  });

However, the second console.log statement is never executed, no matter how long I wait.
Am I doing something wrong or is this a bug?

@cascer1 - really odd. Under normal circumstances, the promise returned from shutdown does resolve. Have you established a successful connection in your program at the point you're calling this? I've tested this with a simple program using 1.0.8 and it works reliably.

I do have a connection established (I'm receiving messages), however it's definitely not resolving.

@cascer1 - can you share demo so I can repro? The promise resolves in the tests and it resolves in pub/sub demos I've created to try and figure out what you might be running into. Another thing you can also try is setting DEBUG=rabbot.* when starting the service that won't exit. This will result in a lot of output but you might be able to see what's not happening (yeah, that's weird). Shutdown is supposed to:

stop all subscriptions
resolve any remaining messages (ack batching)
close down queue channels once messages are resolved
close down exchange channels
close the connection

You can more or less see all of these things happening in the logs. I'd like to try and find the edge case you're hitting, but I'm not sure how to reproduce it yet.

Thanks for getting back to me. I'll be sure to turn on the debug logging to see what's going on. The project having this issue is this one, more specifically, this and this file.

Apologies for my late response. After some further debugging it seems I had the exit handler waiting on any of multiple promises to be resolved, instead of all promises. So the program exited as soon as the fastest promise resolved, which was not the rabbit promise.

I have the same issue @arobson here were some message is pending and not just with shutdown() also using closeAll() or close(name)
The only pattern I could check is that if have some message pending to delivery to some subscribe queue/handler it never ends to stop connection/s.


Apparently this can include an msbuild-sdk element

I know there is a See Also that points to the relevant documentation. But it might be clearer if the msbuild-sdks element was mentioned in the "global.json schema" section of this document.

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

ID: a94c335b-0b65-96cc-1c25-65777fbe4bb5
Version Independent ID: 6d80613f-42dc-35d7-aeff-3187a36dbbca
Content: global.json overview
Content Source: docs/core/tools/global-json.md
Product: dotnet-core
GitHub Login: @mairaw
Microsoft Alias: mairaw

Thanks for your feedback @martingbrown. I think it makes sense to add that.
@KathleenDollard is msbuild-sdks part of the official schema for global.json?

@nguerrera can you help with this one? Would this be similar to what you said in https://github.com/dotnet/docs/issues/14687#issuecomment-537626540?


Type checking is incorrect in array filtering

🔎 Search Terms

check type in null filtering in array
null-value is still detected by typescript although they was filtered in array

🕗 Version & Regression Information

This is a crash
This changed between versions ______ and _______
This changed in commit or PR _______
This is the behavior in every version I tried, and I reviewed the FAQ for entries about _________
I was unable to test this on prior versions because _______

⏯ Playground Link
No response
💻 Code
const filteredClaimIds = claimIds.filter((i) => typeof i === "string");
if (filteredClaimIds.length > 0) {
  await this.claimService.updateClaimBySubscriptionId(
    filteredClaimIds,
    {
      status: "cancelled",
    }
  );
}

🙁 Actual behavior
this.claimService.updateClaimBySubscriptionId requires string[] as first parameter and typescript check filteredClaimIds that still contains null value but I filtered in previous line.

🙂 Expected behavior
Should be passed without error
Additional information about the issue
Node version: 20.10.0
Typescript version: 5.4.5
Yarn version: 1.22.21

You forgot to fully fill out the issue template.
Duplicate of #38390.

Among other things, the template says

Please try the nightly version of TS to see if it's already been fixed. Install typescript@next or use the Playground.

Did you do that? I can't check because your code isn't a reproducible example, but it's quite likely to be already fixed due to TS5.5's upcoming support for inferred type predicates.
Please edit and fill out the issue template properly, and if this is indeed a duplicate of #38390 and fixed in nightly, then do the TS team the favor of closing the issue so they don't have to. Thanks!


chore(ci):added new node versions

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+1.2%) to 96.471%

Totals

Change from base Build<PHONE_NUMBER>:
1.2%

Covered Lines:
139

Relevant Lines:
142

💛  - Coveralls


Wow, is kivy-designer ever in sad shape...

First, I tried installing onto Windows (latest version from repo).  It "runs", but practically everything I try -- even dragging the Screen object around in the UI Creator -- either fails outright, or crashes the designer entirely.  I have yet to discover anything that actually fully works, frankly.  After saving a very primitive design, then it crashes upon subsequently reopening that design.  Definitely "Not Ready For Primetime" on that platform...
Next, I tried installing onto a RaspberryPi 3 (the platform I'm actually targeting), using an official RPi touchscreen display.  Here, it crashes at startup with a lengthy traceback, but since the crashed app has commandeered helm control, I can't click the Report Bug or Copy To Clipboard button.  I'm not going to manually retype the entire traceback, but the point-of-failure entry is (forgive any typos):
:
File "kivy/_clock.pyx", line 168, in kivy_clock.ClockEvent.tick (/tmp/pip-hLtZee-build/kivy/_clock.c:3530)
TypeError: create_window() takes exactly 1 argument (2 given)
Really?  Something as basic as creating a window specifies the wrong number of args and wedges the tool?
Finally, figuring that all the implementers/maintainers are Linux-centric or -only, and that all the testing is done there, I fired up my Ubuntu 14.04 VM and installed it.  There, at least things didn't crash right and left, and basic functionality "works" (kinda, sorta, I guess, see below), but even things like simply trying to open the Project Settings dialog from the main menu crashes the app:
pip.exceptions.InstallationError: Could not open requirements file ....
No, really.  There's the requrements.txt file, right there...  And this is with the latest "stable" build, installed following the documented instructions to the letter.
Even when things don't crash, they don't really "work".  Even something as simple as highlighting a widget then trying to Delete it, doesn't delete it.  Or perhaps I'm misusing this feature, in which case: a) it is incredibly non-intuitive, and b) there no actual documentation about how to do basic things like this in case there are subtleties I'm not hip to.
Um, I'd really like to buy in to the promise of what kivy-designer intends to be.  It's a great idea!  Most of my Python GUI work over the years has been using PySide/PyQt and PyGtk/PyGObject, and in those toolchains Qt-designer and glade are indispensable.  I'm surprised and disappointed that kivy-designer is not in a further state of evolution.  This should be a really cool, simple, intuitive, (even reasonably) stable app.  But... it's not.  If I had the time, and wasn't already working four projects, I'd volunteer to move this along up the food chain a little...
This is not a troll, this is an exhortation to Do Better Work.  As it is now, in the state it's in, it shouldn't be seeing the light of day.

Thanks for the clarification, @inclement.  I wouldn't have been so harsh in my critique if these working presumptions were made clear.  Given the focus on polishing the documentation and many app features that are "niceties" when even basic core functionality is fundamentally broken, I naturally came to the conclusion that this was much further along toward productization.

It's maybe worth noting that one reason for this neglect is that the core developers mostly don't find a major need for such a tool, especially considering the major effort that would be needed to make it anywhere near as flexible as kv language.

Having done GUIs for commercial products over the last 15 years, I can personally attest to the idea that, on the contrary, a designer tool is extremely valuable.  Yes, 'kv' is more flexible.  But flexibility is only one attribute of many in a highly usable toolchain such as GTK+ or Qt, which Kivy should rightly aspire to match and surpass.  If I have to write "code" -- albeit in a mostly descriptive language like kv -- to position and provision every widget in my conventional GUI app, my productivity plummets in comparison to a workflow that incorporates a pseudo-WYSIWYG tool like a designer.  (Hardware designers will confirm the same -- you don't see them rushing out to replace their schematic capture and board layout tools with others that would require them to represent their designs in "code".)
Besides, however powerful and flexible a language kv may be or become, it is no match for the power and flexibility of Python itself, nor should it try to be.  Separating the View from the Controller should be the whole point here, and confining kv to a view-like operation space is fine.  Anything that improves on CSS as an expressive vehicle in this regard is a step in the right direction.
Anyway, in the absence of an evolved Kivy Designer, and not having the time myself to devote to its progression, I am halfway inclined to create a stopgap tool that would take the XML output emitted from an existing tool like qt-designer or glade, and morph it into a kv description.  That would be a much simpler first step that could yield the benefits of having a more mature designer tool without the enormous effort required to polish the creation of another one, or wait on others to do so.  If I do so, I will be sure to share it.


error converted to print statement

while reviewing a PR, I saw this line:
https://github.com/ploomber/jupysql/blame/51782f1871c63f5c73d33bdbbba8a19ea599df68/src/sql/magic.py#L440
an exception is converted into a print statement, which shouldn't happen. instead, we should raise another exception with whatever message we want to display. because an exception will interrupt execution of the notebook (which we want), while a print statement will not

This was an existing flow.
if config.short_errors = True, then it should print the exception only.
if config.short_errors = False it would re-raise the original exception.
Also, if I replace the print(e) with raise many of the existing test cases were failing, since now those are raising exceptions. Some issues with the the tables created in conftest.py also. they don't always get dropped after the function exits.
@edublancas

thanks a lot for the context!
we need to change this since printing exceptions isn't ideal (doesn't interrupt the program workflow, which you'd expect when your cell fails). but since this is an existing workflow inherited from ipython-sql, it isn't urgent) so let's keep this issue open.

I'm clearing the issue assignment since it's not a priority


ResourceManager support get all resource instance by generic type

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]

Done


I found a serious bug about UIWebView+AFNetworking

demo about this bug

Current coverage is 73.78%

Merging #3286 into master will decrease coverage by -0.27% as of d2822b6

@@            master   #3286   diff @@
======================================
  Files            5       5
  Stmts         3002    3002
  Branches         0       0
  Methods
======================================
- Hit           2223    2215     -8
  Partial          0       0
- Missed         779     787     +8

Review entire Coverage Diff as of d2822b6

Powered by Codecov. Updated on successful CI builds.

Thanks so much! I've patched this in #3289
:beers:


line number color has changed recently, only for transparent background

Hi,
I see the color of line numbers has changed from gray to white, especially in transparent background.
It seems okay in non-transparent configuration.
Here are my screenshots and their mimimal init.lua file contents:

It was gray back in v0.0.2 like this:

Is this intentional, or is there anything I missed?

check 5528f3c


Dev Home does not appear in the All Apps list or the start menu, nor in the Windows Tools folder

Dev Home version
0.503.261.0
Windows build number
10.0.22621.2361
Other software
OS Build Version: 10.0.22621.2361.amd64fre.ni_release.220506-1250
.NET Version: .NET 6.0.21
Steps to reproduce the bug
Install Dev Home.
Expected result
Dev Home should be present either in the All Apps list of the start menu or in the Windows Tools folder of the same.
Actual result
Dev Home does not appear in the All Apps list or the start menu, nor in the Windows Tools folder. The only way to find it is through search.
This behavior appears to be inconsistent between machines, as it appears on one machine but not on two users of the other. Both are on 22621.2361.
Included System Information
Physical Memory: 15.73GB (4.92GB free)
Processor Architecture: x64
Included Extensions Information
Extensions:
Microsoft.Windows.DevHome_0.503.261.0_x64__8wekyb3d8bbwe
Microsoft.Windows.DevHomeGitHubExtension_0.501.261.0_x64__8wekyb3d8bbwe

When creating a new user on the faulty machine, the following behavior is exhibited:

The app first appears as "Dev Home" in the Windows Tools folder.
When the full app is installed and replaces the stub, the app disappears from Windows tools and does not appear in All Apps.

Number 1 seems to be correct behavior, but not number 2. It also appears to be a machine-wide issue and not per-user.
This issue is still present with 0.7.

I'd like to note that an in-place upgrade of Windows did not fix this or otherwise change the described behavior.

Same happens for me, running on Windows 11 26058. Really annoying.

Not repro'ed by @EricJohnson327 and @lauren-ciha with version 0.1701.

In the past months I have done two clean installs of Windows on two separate machines, and both have this issue. This has not been resolved!

@adrianghc What Windows version(s) are you seeing this on? During Windows setup, are you opting in as a "developer" or do you have "developer mode" turned on? Are you seeing it in neither Windows Tool or All Apps once the full app is installed?

@krschau I'm seeing this on both versions 23H2 and 24H2. If I recall right, on one machine, I opted in as a developer during OOBE while on the other I did it from Settings. I don't have developer mode turned on on either machine. The app doesn't appear in Windows Tools nor in All Apps.

Same here on fresh 24H2 install. Shows up in winget list and was updated once, but doesn't appear through search or in All Apps.


fix target_blank image link on teaser default template

The linked image wasn't opening in a new tab

@goldhand thanks for this fix, sorry it took so long to merge


Cambio de URL de Olympus Scanlation

Source information
Olympus Scanlation 1.4.10
Source language
Spanish
Source new URL
https://olympuscomic.com/
Other details
No response
Acknowledgements

[X] I have updated all installed extensions.
[X] I have opened WebView and checked that the source URL has not changed yet.
[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] I will fill out all of the requested information in this form.

No response

Esta pagina cambia de dirección a muy menudo debido al Copyright, pero no modifica nada de su interfaz, no seria mejor crear una extensión en la cual se cambie solo su URL en los ajustes de la extensión, varias apps de contenido multimedia o lectura de novelas que también funcionan con extensiones, tienen una función para clonar sitios web idénticos, solo digo mi humilde opinión de alguien que no sabe programación. 🙂

This page changes its web address very often due to copyright, but it doesn't change anything of its interface, wouldn't it be better to create an extension in which only its URL is changed in the extension settings, several apps for multimedia content or reading novels that also work with extensions, have a function to clone identical websites, I just say my humble opinion of someone who doesn't know programming 🙂

@Zahyra16 I think that something similar to what is done in the Ikai Mangas extension is possible to do for Olympus, I will analyze and implement this.


How to use it?

"Could you please tell me how to use this library? I don't see the construct_parallel_models() API."

You can relate to Galvatron Model Usage for more details. The construct_parallel_models() API is named construct_hybrid_parallel_model_api in each model. It can be found in galvatron/models/MODEL_TYPE/MODEL_TYPE_hybrid_parallel.py.


Unwanted empty auto generated file

As suggested by discussion in #45
: Quote :
Ok, just to confirm something
I use the inline approach for all of the generated code in my project.
The following file demands I have an output key.
while reading .yml 'some/path/.sourcery.yml'. 'Invalid output. 'output' key is missing or is not a string or object.'
The presence of an output key in the following file seems to generate an empty file called someName.generated.swift
sources:
  - some/path
templates:
  - some/path
output:
  ./

There is currently no option to stop this file from being generated, right?
It's not the end of the world as I can just exclude it from compile sources & git.
: EndQuote :
I have appended a reproducible example.
$ brew info sourcery
sourcery: stable 0.11.2 (bottled), HEAD
https://github.com/krzysztofzablocki/Sourcery
Poured from bottle on 2018-04-24 at 16:07:47
From: https://github.com/Homebrew/homebrew-core/blob/master/Formula/sourcery.rb

Test.zip

You should put // MARK comment inside inline block so that it is inlined with the rest of generated code, then AutoEquatable.generated.swift will be removed.


Fix editing of some properties in CodeHighlighter

Unforeseen regression from:

https://github.com/godotengine/godot/pull/88231

These clear operations serve little direct purpose and break editing, while there might be some way to handle the editing of the dictionary differently there's no reason for these clears, and it solves the underlying issue.

Fixes: https://github.com/godotengine/godot/issues/97411

Thanks!

Thank you!


Can I report issues with reference to a specific line of code?

As the title suggests, is there a way for us to mark/highlight the portion of codes I might be reporting issues of?

Yes, click the line number and the "copy permalink" as shown here:


Docs - cannot use env_vars in the vars block

Document an exception where env_vars is not rendered in the vars block of a dbt_project.yml file.
re: https://github.com/dbt-labs/dbt/issues/3105#issuecomment-779761596
Description & motivation
From @jtcohen6 's comment on https://github.com/dbt-labs/dbt/issues/3105#issuecomment-779761596 - jinja isn't rendered in the vars block in a dbt_project.yml file. The docs currently say you can use env_var anywhere in the dbt_project.yml
Pre-release docs
Is this change related to an unreleased version of dbt?

[ ] Yes: please

update the base branch to next
add Changelog components: <Changelog>[New/Changed] in v0.x.0</Changelog>
add links to the "New and changed documentation" section of the latest Migration Guide

[ x] No: please ensure the base branch is current
[ ] Unsure: we'll let you know!

Sorry! Misunderstood the original comment. Just seen this page on available functions:
https://docs.getdbt.com/reference/dbt-jinja-functions/dbt-project-yml-context/
I have a weird bug with env_var not rendering but it must be something else.


chore: bdd-spring-1572379695 to 0.0.1

chore: Promote bdd-spring-1572379695 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Move new unit tests to Cesium.Unit.X namespace

They were added previously, but not in this namespace.
CI executes the tests from this namespace. If your test isn't here, it won't run.

Thanks for the pull request @csciguy8!

:heavy_check_mark: Signed CLA found.

Reviewers, don't forget to make sure that:

[ ] Cesium for Unreal Samples works.


Add support for loading OSes

Payload will be placed at 0x80400000 if the path is mentioned in the just run command. It is tested with opensbi jump firmware and linux image, both built with opensbi artificat repo.
This PR aims to close #52.

 So I think this should be controlled from the configuration file, rather than through the just run command (because we might have multiple way to start the OS in the future, like OpenSBI do with fw_payload and fw_jump for instance).
We can discuss that on Monday :)

The PR is marked as a draft, so I'm not sure if you want a review. Remove the draft status when you want me to have a look :)


ManhwasNet

https://manhwas.net/es (Español)

hope you can add this source we need this spanish source cause is one of the most updates


[bug fix] assigning the reurn value of new by referece is now deprecated.

@MarvinKlemp Hi, Can you review and merge this PR if it is ok?
looking forward to your reply.

@mahone3297 looks good to me :)


Export-Excel strange finds

Some code looks strange.

Parameter InputObject has an alias TargetData. In line 262 the expression foreach ($TargetData in $InputObject) { uses a variable named TargetData. This does not produce any issue but it is better to rename internal variable.
In line 399 there is expression if ($PSBoundParameters.ContainsKey($TargetData)) {. What is $TargetData here?

Long time ago. IIRC, I had called it targetdata and then to support similar function in the PS world, moved to $InputObject and it was aliased.
I don't believe there are sufficient unit tests to safely change that without a bunch of work and testing.

I catch myself in situations where I’m too lazy to delete things that already work, but I understand that it’s backwards compatible

This module was beyond "organic." At the time I was and had been experimenting with many ways to capture/display/read data with PowerShell. I hit on being able to read Excel data that was reliable, not needing Excel or the COM approach (which was nice but brittle).
I thought it was cool and published it, that's why it is called ImportExcel.
Then I discovered I could write to Excel and expanded. Didn't feel like renaming it, didn't know if this would ever go anywhere.
Never spread the word about it and folks would criticize that I didn't supply help/docs.
They didn't like the Examples folder.
As I found cooler things, I added. Never had a plan, roadmap or vision.
The only reason it somewhat hangs together is because I've done software development for decades and have picked up a few good habits along the way.
This was/is a PoC and I consciously work to not over engineer.
I follow YAGNI - You ain't gonna need it. Why invest in building parts that may never be used, again ImportExcel as the name.
Was just having fun building. 🙂


How to get started with Data Analysis?

I want to know how can I start learning about Data Analysis. I've done a course on Coursera by IBM but what do I do after it? How to get more experience?

Great question. I have forwarded this to an expert. As soon as I get his time, I'll record and publish.


Introduce Landlock isolation support

Leaving this here for now, the most important design problem with this is how Landlock does not allow whitelisting files if they are not created. (So, we'd have to force the user to use a directory for that instead.)
We also need to avoid "parsing" the same --file-mapping inputs twice, as well as not use OnceLock for enforcing the whitelist when the kernel is actually being loaded. UhyveVm::new is called in a new thread because Landlock enforces the restrictions for the entire process and its children. We're not testing if the sandbox is applied correctly yet.
See: https://docs.kernel.org/userspace-api/landlock.html
Fixes https://github.com/hermit-os/uhyve/issues/766

We need to move the split_guest_and_host functionality that is used during the initialization of UhyveFileMap::new away from that function for two reasons:

We need to be able to "hard fail" if a file does not exist and/or its parent directory is not whitelisted and we use Landlock instead of running absolute. See https://github.com/hermit-os/uhyve/pull/819.
We are effectively parsing the --file-mapping argument twice.

uhyvefilemap_test works. The problem is that you can't run UhyveVm::run multiple times, as the Landlock restrictions have already been applied process-wide once. Additionally, the temporary directory gets dropped after the first instance of UhyveVm::run, so trying to re-enforce Landlock causes an error as the temporary directory has been dropped.
My approach to the problem of "not being able to whitelist files that don't exist yet" is just whitelisting the parent directory and letting UhyveFileMap contain file operations to that one specific file only (in the whitelisted directory). We do that by iterating over the file's parents and establishing whether they exist, once - but this could be made configurable, and it should be fine if we disclose to the user that whitelisting directories is safer.
However, whitelisting a directory is not always practical, because we can't map the entirety of /root to anything yet, as the /root component gets popped by Uhyve before a hypercall is executed (see fs-test.rs and create_file.rs for a workaround), see: https://github.com/hermit-os/kernel/blob/12d4854554f8ec48fa66a8d282ebcf46b7bf9499/src/fs/mod.rs#L160

The change incorporates some changes from https://github.com/hermit-os/uhyve/pull/844.


Add minimal pattern value length

Partial solution for #178
Changes:

add minimal len for pattern based values: 12
move minimal len to config
add min len argument to ValueLengthCheck
add examples that detected as creds in current master, but not in this branch

CredData benchmark:
Master:
result_cnt : 4102, lost_cnt : 65, true_cnt : 3701, false_cnt : 336
credsweeper -> TP : 3701, FP : 336, TN : 19454363, FN : 882, FPR : 0.0000172709, FNR : 0.1924503600, ACC : 0.9999374078, PRC : 0.9167698786, RCL : 0.8075496400, F1 : 0.8587006961

This branch:
result_cnt : 4094, lost_cnt : 60, true_cnt : 3701, false_cnt : 333
credsweeper -> TP : 3701, FP : 333, TN : 19454366, FN : 882, FPR : 0.0000171167, FNR : 0.1924503600, ACC : 0.9999375619, PRC : 0.9174516609, RCL : 0.8075496400, F1 : 0.8589996519

Delta:
FP : 336 -> 333
F1: 0.8587006961 -> 0.8589996519

Lines that detected by master, but removed in this branch:
"line": "  \"refresh_token\": \"eyJrefresh\",",
"line_num": 214,
"path": "CredData/data/7738e44d/other/c.md",
"value": "eyJrefresh",

"line": "\t\t\t\t\tSo(authorizationHeaderCall1, ShouldStartWith, \"Bearer eyJ0e\")",
"line_num": 176,
"path": "CredData/data/8cda00f3/test/bu.go",
"value": "eyJ0e",

@meanrin https://github.com/Samsung/CredSweeper/pull/185 - suppose, minimal number of credentials is enough to check

Regarding ActionTest, it seems it could be passed by changing .github/workflows/action.yml file like below.
    - name: Prepare commit status
      run: |
        ITEMS_CNT=$(jq '. | length' output.json)
        if [ 19 -eq ${ITEMS_CNT} ]; then


Support for Version 5.0.X Replays?

Ran the random agent on the simple 64x64 map, saved a replay, then when I run:
python -m pysc2.bin.play --replay RandomAgent/Simple64_2022-03-24-03-57-25.SC2Replay
... I get:
Traceback (most recent call last):
  File "C:\Users\chalu\AppData\Roaming\Python\Python37\lib\runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "C:\Users\chalu\AppData\Roaming\Python\Python37\lib\runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "C:\Users\chalu\AppData\Roaming\Python\Python37\site-packages\pysc2\bin\play.py", line 234, in <module>
    app.run(main)
  File "C:\Users\chalu\AppData\Roaming\Python\Python37\site-packages\absl\app.py", line 312, in run
    _run_main(main, args)
  File "C:\Users\chalu\AppData\Roaming\Python\Python37\site-packages\absl\app.py", line 258, in _run_main
    sys.exit(main(argv))
  File "C:\Users\chalu\AppData\Roaming\Python\Python37\site-packages\pysc2\bin\play.py", line 174, in main
    run_config = run_configs.get(version=version)  # Replace the run config.
  File "C:\Users\chalu\AppData\Roaming\Python\Python37\site-packages\pysc2\run_configs\__init__.py", line 40, in get
    return max(configs.values(), key=lambda c: c.priority())(version=version)
  File "C:\Users\chalu\AppData\Roaming\Python\Python37\site-packages\pysc2\run_configs\platforms.py", line 123, in __init__
    version=version, cwd="Support64")
  File "C:\Users\chalu\AppData\Roaming\Python\Python37\site-packages\pysc2\run_configs\platforms.py", line 63, in __init__
    data_dir=base_dir, tmp_dir=None, version=version, cwd=cwd, env=env)
  File "C:\Users\chalu\AppData\Roaming\Python\Python37\site-packages\pysc2\run_configs\lib.py", line 112, in __init__
    self.version = self._get_version(version)
  File "C:\Users\chalu\AppData\Roaming\Python\Python37\site-packages\pysc2\run_configs\lib.py", line 220, in _get_version
    versions = self.get_versions(containing=game_version)
  File "C:\Users\chalu\AppData\Roaming\Python\Python37\site-packages\pysc2\run_configs\platforms.py", line 111, in get_versions
    containing, sorted(ret.keys())))
ValueError: Unknown game version: 5.0.9. Known versions: ['3.16.1', '4.7.0', '4.7.1', 'latest'].

Any hope of getting the latest versions of the game supported? My current is 5.0.9. Really hope so as this would be huge for my project. Thanks!

This doesn't exist on linux, and likely never will. You should be able to do it on Windows or Mac now if you use the latest version (or from git). The data versions are known, but you need to have the right version installed.


Make ZipWriter.finish() consume ZipWriter

In terms of the API of this crate, one of the simplest use cases (writing a zip file) with (admittedly not well-formatted but easily imaginable) code like this has a footgun:
fn zip_files(paths: &[PathBuf]) -> Result<Vec<u8>, ZipError> {
    let mut io_cursor = Cursor::new(Vec::<u8>::new());
    let mut zipwriter = zip::ZipWriter::new(&mut io_cursor);
    let options = FileOptions::default().compression_method(zip::CompressionMethod::Stored);
    for path in paths {
        let mut buffer = vec![];
        zipwriter.start_file(path.to_str().unwrap(), options)?;
        let mut filehandle = std::fs::File::open(path)?;
        filehandle.read_to_end(&mut buffer)?;
        zipwriter.write_all(&buffer)?;
    }
    zipwriter.finish()?;
    // Code does not compile without this
    // drop(zipwriter);
    Ok(io_cursor.into_inner())
}

Without an active drop(), this code does not compile, as the zipwriter keeps the exclusive reference even though it's finished.
This seems unnecessary, since using the writer after completing the zip file is explicitly not recommended behaviour anyway.
This would not be a problem if zipwriter.finish() simply consumed self, which could also drop it after finish() is completed.
This could be solved with an own scope too, but that is obviously somewhat equivalent to the active drop() invocation.
Obviously this would be a breaking change, but I imagine this could be a worthwhile improvement to the API of ZipWriter

I'm not madly in love with the overall ZipWriter API. Having start_xxx() and finish_xxx() pairs smells like it should be a separate struct. Having start_xxx() consume self and return a io::Write impl, which in turn has a finish() method consuming self and returning the original ZipWriter would categorically prevent logic bugs where you interleave writes to different files.

In zip 1.x, ZipWriter::finish does return the inner writer. I'll look into creating objects for individual files.

Related: https://github.com/zip-rs/zip2/pull/98


docs docs

I realised our deployed docs had useless junk in them which would stop us actually sending people to our docs site. Did some very quick tidying, and give us a clearer place to dump our own test pages so we can see them on branch builds instead of them being indistinguishable from actual docs

Don’t merge this yet - pushing up versions to get the netlify deploy working so we get the example pages on not-master

Like all good changes, this need to be merged in to master to make sure it's correct, ie that the pages I don't want to see aren't seen on master. Cheers!


Fix the Edit on GitHub link

This button currently works but is not smooth.
Steps to reproduce bug

Go to homepage (or any page) https://docs.meli.sh/
Click on Edit on Github button. It's link is https://github.com/getmeli/meli-docs/tree/master/src/data/docs/index.md
When you click through, you'll be warned that the branch master does not exist and that it will use latest instead.

Solution
Change the button to use latest instead of master in the URL.
Therefore a change to .env.example is necessary.
Outside of the codebase, if there are any environment variables set on the GH repo secrets or remote server, then the env variable needs to be updated there to be the maintainer.

Additional details I found digging in the code:

I found this component which sets the href as the github URL, here
And Banner URL is set in Content here
That value gets set here in docs-page.js.
Which comes from the env value here: here

Additional details I found digging in the code:

I found this component which sets the href as the github URL, here
And Banner URL is set in Content here
That value gets set here in docs-page.js.
Which comes from the env value here: here

@MichaelCurrin thanks so much for digging into this. The link is controlled by the GATSBY_GITHUB_DOCS_URL environment variable which is incorrect in the current setup. I'll update this right away !

@MichaelCurrin thanks so much for digging into this. The link is controlled by the GATSBY_GITHUB_DOCS_URL environment variable which is incorrect in the current setup. I'll update this right away !