track consumer lag?

Is there way to track the consumer lag? Consumer responses tell you the offset high watermark, so you can subtract the that by the current offset you've seen to determine how far behind you are.

yes for partition subscribers, no for topic/group subscribers.
because hw-offset is at message-set level, but topic/group subscribers are implemented to handle messages (not sets).
worth mentioning that if your messages are very small-sized, topic/group subscribers may not work very we'll due to the per-message callback and erlang-message passing overheads
we are working on a handle_message_set (optional in 2.2) behaviour callback for topic/group subscribers.

@clofresh you can take a look at how we implemented this in an external application: https://github.com/klarna/kastlex/blob/master/lib/kastlex/cg_status_collector.ex

cg_status_collector in kastlex collects ALL consumer group lags.
The hw-offset in fetched message-set should be enough if you only care about the lagging of current consumer (which may not be a group subscriber)

ok, thanks for the info!


Update to Azure DevOps

Updated:

Logo
GitHub URLs to new repo name
References of VSTS / Team Services / etc. to Azure DevOps and Azure DevOps Services: I left comments as-is.
Replaced "account" with "organization"

@kaylangan Are you need help with this PR?

@mfilippov a review would be great, thanks.


Add Elm language support

It would be great to have Elm support!
Elm style guide :
https://gist.github.com/laszlopandy/c3bf56b6f87f71303c9f

https://github.com/Glavin001/atom-beautify/pull/700 has been merged.


Fixes #809: warnings on Foundry v12

Fix a typo that resulted in a warning. It sounds like the description probably never worked right.
Move from the gridDistance and gridUnits fields to grid.distance and grid.units. I can't tell when these were added, so this needs to be tested on Foundry v11 (I only have a v12 system at this point).

Don't merge this, verified via the Foundry VTT discord that grid.distance doesn't work for v11.

Don't merge this, verified via the Foundry VTT discord that grid.distance doesn't work for v11.


Monitoring the status of an event consumer

We believe that it would be very useful if we could montior an event consumer (with respect to Event Hubs and an implementation of IEventProcessor in .NET). We want to determine its ability to keep up with the rate of event ingestion.  We'd like to do this at the level of a partition and in the context of a specific consumer group.
This could be used for monitoring, alerts, and potentially autoscaling.
We would need to make use of some of the APIs in the .NET SDK as well as querying the blobs that the EventProcessHost uses for checkpointing.
In our first version we plan to:

create a .NET class library that can be used in various contexts
determine the events/second/parition ingress for the event hub
determine the  events/second/parition egress for the consumer group
determine how many unprocessed events there are per partition
have a configurable polling rate for querying the state of the event hub/consumer group
keep track of these values over a configurable period of time in order to establish trends

Since checkpoints are likely to incur much less frequently, we'll need to make the unit of time for egress something that we can adjust.

Let's locate this in /src/Monitoring/DotNet/.


documentation bug, create needs a template parameter.

On the Gitter channel -
Peter Majchrak @petoknm Jul 09 09:54
i was missing a template parameter for create
auto ints = rxcpp::observable<>::create<int>( [](rxcpp::subscriber<int> s){ s.on_next(1); s.on_next(2); s.on_completed(); });

please update http://reactive-extensions.github.io/RxCpp/ to reflect that. thanks

I think the template parameter can be inferred by the type of the lambda passed in... I see it being too verbose having a template parameter on create and on subscriber as well if they always match... I think that the way the documentation had it (without the template parameters) was cleaner and nicer

Yes, that is true when using the type-forgetting subscriber<T>. Perhaps a different create overload could be added for that, but then create would have a different usage depending on the type of the parameter declared in the lambda. That seems too subtle to communicate to users.
Thoughts?

What exactly do you mean by "type forgetting subscriber<T>"? and what exactly would the difference in usage be?

std::function<> is type-forgetting. It uses virtual functions and heap allocation to hide the type of the function that was passed to its constructor. This allows it to be used as a parameter or struct member that can hold a variety of function implementations; lambda, object, function ptr, etc..
subscriber<T, Implementation> is the full type of subscriber. This can be efficient, because no heap alloc or virtual functions are required. When subscriber<T> is used the Implementation uses virtual functions and heap alloc to hide the real Implementation type.
type-forgetting is expensive and should be used only when needed.
If a create overload was added to remove the template param when using type-forgetting it would look like this.
auto ints = rxcpp::observable<>::create( [](rxcpp::subscriber<int> s){ s.on_next(1); s.on_next(2); s.on_completed(); });

However, the existing overload would still be needed when the type is not forgotten.
auto ints = rxcpp::observable<>::create<int>( [](auto s){ s.on_next(1); s.on_next(2); s.on_completed(); });

There is no way that create could extract int from the auto param of the lambda.


Assign pipelinerun data store to reconsiler

fix https://github.com/kubesphere/ks-devops/pull/825
same as: https://github.com/kubesphere/ks-devops/pull/855
pr to master branch
Support to store PipelineRun data into ConfigMap

/lgtm
/approve


FancyZones considering vertical task bar size twice on multiple monitor setup

ℹ Computer information

PowerToys version: v0.23.2
PowerToy Utility: Fancy zones
Running PowerToys as Admin: True
Windows build number: 20H2 OS Build 19042.572

📝 Provide detailed reproduction steps (if any)

Setup desktop environment with a vertical toolbar on the left monitor, docked on the left side
Setup powertoys-fancyzones, vertical grid spaces among monitors
Open one or multiple Windows
Move the windows to the most left zone in left monitor

✔ Expected result
The docked window should be aligned with the most left defined zone
❌ Actual result
The windows docked at most left are shifted right, considering the vertical taskbar width twice. The actual window width defined by Fancy Zones is conserved.
📷 Screenshots

Fancy Zones setup among two monitors, one vertical taskbar on main monitor at left

Docking window at most left column (expected)

Docking window at most left column (actual)

Mentioned in https://github.com/microsoft/PowerToys/issues/2252#issuecomment-704065044

I run into a similar issue. My set up is laptop screen at the bottom with the taskbar, and a secondary monitor at the top with no taskbar. It has no problem on the laptop screen, on the external monitor:

It feels as if they assigned a negative width to the taskbar, which does not exist on the external monitor.

@enricogior Could you also mention that 'negative width of taskbar when external monitor with no taskbar is at the top' in your tracker issue so I don't need to open a new issue?

@lonelykid
done. Thanks.

I have the same problem when using dual monitors.
When I drop a window to a zone, the window is snapped and shifted  by the width of the vertical taskbar.

@isshiki
can you please provide more information, otherwise is nearly impossible to debug it.
What are the exact monitor configuration (add a screenshot of the monitors from the Window 10 Settings, that is the native resolution and the DPI scaling).
What type of layout are you using? A template? A custom layout derived from a template? A canvas layout?
Did you apply the layout before moving the taskbar or after the taskbar was already in that position?
Does it happen with all the apps on just with some apps?

This problem occurs only when if I check [Allo zones to span across monitors (all monitors must have the same DPI scaling)] checkbox on [FancyZones] tab.
I took a video capture of my operation;
https://youtu.be/Thgb79NgfxI
My monitor configuration:

My layout selection:

I applied the layout after the taskbar was already in that position. However, I had the same problem if I apply the layout before moving the taskbar. I recorded my both operations:
https://youtu.be/phEy6DlZ-SM
This happens with all the apps.
Please let me know if you need more information.

@isshiki
ok thanks, the All zones to span across monitors option is the key to this problem.

In 0.35 there will be a fix for several taskbar issues, we didn't manage to reproduce this particular one, so we are not 100% sure it will be fixed, but after releasing 0.35 we will close the issue as fixed and then reopen it if it will reproduce.
Thanks.

This has been resolved in the 0.35 release.  https://github.com/microsoft/PowerToys/releases/latest


Missing dependencies for gramps

Issue description
At launch, gramps complains that
Gdk, Gtk, Pango or PangoCairo typelib not installed.
Install Gnome Introspection, and pygobject version 3.12 or later.
Then install introspection data for Gdk, Gtk, Pango and PangoCairo

Gramps will terminate now.

…and terminates.
Steps to reproduce
nix-shell -p gramps --run gramps

or
nix run -f channel:nixpkgs-unstable gramps -c gramps

Technical details

system: "x86_64-linux"
host os: Linux 4.14.32, NixOS, 18.03.131792.becbe4dbe16 (Impala)
multi-user?: yes
sandbox: no
version: nix-env (Nix) 2.0
channels(root): "nixos-18.03.131792.becbe4dbe16"
channels(ivan): "nixpkgs-unstable"
nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos/nixpkgs

I think wrapGappsHook missing for gramps.

can you check if #38472 solves the issue ?


Sync translation guide links in translations

All translation references added to all other translation documents which have "non-speakers" section

@Saydemr thank you very much!


Add Request: Minimal UI

Request to add the Minimal UI module to the supported modules.

Whetstone support would be up to the author to implement.


Pinecone metadata filtering

I noticed this open issue around filtering Pinecone by metadata: https://github.com/hwchase17/langchainjs/issues/333
Since the Pinecone NodeJS lib supports a filter param: https://docs.pinecone.io/docs/metadata-filtering
I added an optional filter param to the VectorStore methods similaritySearch, similaritySearchVectorWithScore, and similaritySearchWithScore. Then implemented it in PineconeStore.
I updated the specs, docs, and examples to reflect this new functionality.
One thing I couldn't figure out, though, is how to allow metadata filtering within a chain. It'd be cool to be able to add it as a param in VectorDBQAChain.fromLLM. What do y'all think? Doable?

Funny how we were working on this at the exact same time. #426
My approach was to add filter the same way namespace was implemented. I want to get metadata filtering in ASAP, so I would love to talk about the best approach to do so.
This is the way I plan on implementing filter:
// metadata filtering
      const filter = {
        metadataId: { $in: metadataIds },
      };

      const index = pinecone.Index(PINECONE_INDEX_NAME);
      /* create vectorstore*/
      const vectorStore = await PineconeStore.fromExistingIndex(
        new OpenAIEmbeddings({ modelName: 'text-embedding-ada-002' }),
        {
          namespace: PINECONE_NAME_SPACE,
          pineconeIndex: index,
          textKey: 'text',
          filter: filter,
        }
      );

      const model = openai;
      // create the chain
      const chain = VectorDBQAChain.fromLLM(model, vectorStore);

      //Ask a question
      const response = await chain.call({
        query: sanitizedQuestion,
      });

@hwchase17 @ElektrikSpark -- I'm open to either approach. I can move the filter param into PineconeLibArgs if you'd prefer. Let me know what y'all think?

Approach
Pros
Cons

PineconeLibArgs
Pinecone-specific
Have to re-init VS to change filters for query

similaritySearch
Filters associated with query
Not all VS have meta filtering

i think it should definitley be an option to pass it in during init. the reason i think that is important is that its annoying to plumb the search kwargs through all the time when youre using it in a deep nested way in a chain
but im also happy to allow it in kwargs, and just do some validation that only one exists, etc

@conroywhitney i merged @ElektrikSpark s pr in. im gonna take a stab at fixing merge conflicts and get this one in as well


Simplify class methods to improve readability of the code

Associated issue number and/or pull request reference
Fixes #75

Adds #77


Updated client dependencies due to GitHub's security alert.

Coverage remained the same at 34.412% when pulling 7f9f9c4eeb3f2c990da58b639a92d90b6e00076e on deps_update into 704dd92b51da94c0b9d7860d1dce741ab8445d75 on master.


chore: Update docker-compose.dev.yaml for automatic container restart and added instructions for allowing access on the entire network

Cleanup of docker compose and added restart instructions.
Also, added instructions to enable access on the entire network.

Thanks for being the first contributor!
The instructions are great, but I want to keep the initial README as simple as possible. I created a custom setup instruction file here:
https://github.com/rashadphz/farfalle/blob/main/custom-setup-instructions.md
Could you move those instructions there?


fix: alternate dest advanced log path 0 added to conf

Closes: #1611

:tada: This PR is included in version 2.32.3 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Incompatible with canjs 3.0

Because of https://github.com/canjs/can-test-helpers/commit/a8cd96500a3b7cda8b07ab8ec74fea40d1bbcaeb<EMAIL_ADDRESS>is incompatible with canjs@3.

For this, we want to update all of the other can-* packages (in 3.0) to depend on "can-log": "^1.0.0".
<EMAIL_ADDRESS>removed the "INFO:", "LOG:", etc from log messages, so there will be a small number of tests (maybe just 1) that need to be updated. Otherwise this change should be transparent.
We should do all of these to make sure the main can package works locally before publishing anything.

All of the can-* dependencies for canjs@3 already have versions published with<EMAIL_ADDRESS>I'm going to move this to canjs/canjs since all we need to do is update dependencies.

Issue moved to canjs/canjs #4032 via ZenHub


Requirements

Is there a requirements file for creating a stable environment here? Could you provide one? Thank you in advance.

Hi, guy, do you get the requirements?

Did you encounter any problems in reproducing the results? Can you provide detailed information?
It's been too long since opensource, we used pytorch1.7 or 1.8


Add loose mode

By spec nullish values will error out but that leads to having to do
if (maybeNullish) {
  for (let v of maybeNullish) {}
}

It would be rad if there was a loose mode to have it do
length = array ? array.length : 0 so I could just do
for (let v of maybeNullish) {}

The plugin was sherlocked by babel. Are you still using it?

Yep !

Want to write the PR?

If you have bandwidth I'll defer to you, but if you don't I'll put it on my list to tackle :P

Fixed by https://github.com/jridgewell/babel-plugin-transform-for-of-as-array/commit/b7737b0230c5a3f292b9a52bbf40574f5f298d25, released as v1.1.0.

Thank you!


Add a tx_meta_data table

@vargasmontero This should be provide the Tx metadata you are requesting. In the Haskell side it is just a Text containing the meta data encoded as JSON, but as far as PostgreSQL is concerned it should be JSON.
There currently is no tx metadata on the mainnet chain so I don't really have a way of testing this.

@vargasmontero Thinking some more it may be better to merge this now and fix it if it is incorrect.

Merging this.


in https://www.freecodecamp.org/news/learn-solidity-handbook/#how-typing-works   some contens is not display

Describe the Issue
some contens is not display
And the bytes I got returned were represented in hexadecimal form as:  is not display.

Affected Page
https://www.freecodecamp.org/news/learn-solidity-handbook/#how-typing-works
Steps to Reproduce
nothing
Expected behavior
it should be display.
Screenshots

System

Device: [Laptop]
OS: [ Windows 10,]
Browser: [ Chrome]
Version: [ 109.0.5414.120]

Additional context
No response

I understand your issue. it should be readable right.

I found that the value  of  0x0000... is in work-break:normal, but it should be work-break:break-all.

I think this issue is not related to the code because it causes problems with the style of the news. So we can contact the author via email or other social media to change the style.
https://www.freecodecamp.org/news/author/zubin/

@scissorsneedfoodtoo The code snippet needs to go into a code block in our ghost editor to make the paragraph readable. The respective article is linked above in the issue description.

Thanks for reporting this @luojiyin1987, and for the ping @Nirajn2311.
I just went in and updated the article so that long hex string is in a code block.
The fix should go live within the next few hours.
Please let us know if you notice any other issues in this article.


profilepic not showing

Hi. I tried the CoverLetter template. Works great, but the profilepic doesn't work. I tried \profilepic{images/ dummy_pp.jpg} and also tried my own picture, but it never showed. Thanks for your help!

Hi, on what theme did you try this? Only bw and gray themes support showing profile pictures, I'll add a comment for that to the template.

Hi, none of the cover letter themes were envisioned to support profile pictures. For CVs only the bw and gray themes support showing profile pictures.


Formatting errors in Compiler Options Hardening Guide for C and C++

The "Compiler Options Hardening Guide for C and C++" still has a few formatting problems, even after #209. There aren't many, but they detract from the awesome content.

Just after table 2 and just before "Enable warnings for constructs often associated with defects" is this odd thing:

https://developers.redhat.com/blog/2020/02/11/toward-_fortify_source-parity-between-clang-and-gcc and Poyarekar, Siddhesh. D91677 Avoid simplification of library functions when callee has an implementation. (LLVM Phabricator) https://reviews.llvm.org/D91677

Just after this note there are some isolated URLs: "NOTE: Despite its name the -Wall options does NOT enable all possible warning diagnostics, but a pre-defined subset. For a complete list of specific warnings enabled by the-Wall and -Wextra compiler please consult the GCC12 and Clang13 documentation respectively."

Similarly, just before "Mark relocation table entries resolved at load-time as read-only" there are some isolated URLs.

Thanks @david-a-wheeler for spotting this.
Question to the crowd: I'd actually like to make use of inline links in the footnotes, rather then just adding the URL. An example is this change. Let me know if you'd like to see those changes or prefer to keep the current layout.

Thanks @david-a-wheeler for spotting this.
Question to the crowd: I'd actually like to make use of inline links in the footnotes, rather then just adding the URL. An example is this change. Let me know if you'd like to see those changes or prefer to keep the current layout.

I wouldn't have an issue with it but don't know how well it complies with other documents OpenSSF is generating.


Http.Progress not working when the content is gzipped

If the content is not gzipped, the progress bar seems to work fine on all the browsers I have tested so far. But if the server sends it as gzip, only Firefox works¹.
This appears to be a XHR spec issue, for what I understood, chrome and others browsers does not implement tracking progress when the content is gzipped because the specs are unclear whether the value of the content-length header should be after-decoding or before-decoding.
Tests can be done in this link: https://pablohirafuji.github.io/elm-http-progress-example/

Is there any proposal of concrete changes that should be made to how this library works as a result of this underlying disagreement?

With the new elm/http and elm/file packages, there is not. I really like the new http api and the elm/file and elm/bytes packages. Just using your packages makes me a better programmer, gives me a better discernment of good and bad mental models. Yours being the standard for good, of course. I do not objectively know why, but it makes me do  simpler, debuggable, coherent, non contraditory, reasonable and enjoyful codes.
Thank you for being who you are!


implement autoBuild and  deployByDefault

devfile 2.2.0 added two new fiels we need to make sure that odo has a proper support for it
https://github.com/devfile/api/pull/709

https://github.com/devfile/api/issues/852

Should require making changes into https://github.com/redhat-developer/odo/blob/main/pkg/devfile/components.go#L12 as pointed by @feloy.

Make research on this issue

I did some research on this issue to better understand the impact on odo, based on the final proposal at https://github.com/devfile/api/issues/852#issuecomment-1211928487:

odo dev
odo deploy
odo build-images: currently, we are building all Image Components, regardless of whether they are referenced or not. So autoBuild won't change that behavior. We can continue to build all Images, regardless of autoBuild.
Binding and odo deploy: currently, odo add binding adds a Kubernetes component to the Devfile with deployByDefault not set and which is not referenced by any apply command. Per https://github.com/devfile/api/issues/852#issuecomment-1211928487, it will need to be automatically created on startup. So odo dev will auto-create it (as before). But I assume odo deploy would also do the same, which changes the behavior of odo deploy regarding ServiceBinding resources. I understand that this clarifies things, but would that be ok to change this behavior, @kadel?

I have started implementing the required changes in https://github.com/redhat-developer/odo/pull/6654, using this Devfile for the test cases.
And the Devfile Parser seems to currently return the default value (false) for deployByDefault and autoBuild, making it quite impossible to handle cases where those fields are not set, and the component is not referenced by any apply command. I'll need to investigate further..

I have started implementing the required changes in #6654, using this Devfile for the test cases. And the Devfile Parser seems to currently return the default value (false) for deployByDefault and autoBuild if not set, making it quite hard to handle cases where those fields are not set, and the component is not referenced by any apply command. I'll need to investigate further..

Ok, I've narrowed down the issue to a boolean passed to the Devfile parser arguments: FlattenedDevfile. If this is set to true (default value), the Parser sets all unset fields to their default values, making it hard to determine when they are not set:
https://github.com/devfile/library/blob/bd4a12f272573de1ab254f3e138a9b55904af620/pkg/devfile/parser/parse.go#L147-L153
Setting it to false (055f71a (#6654)) works fine in the scope of this issue, but creates other issues when parsing Devfiles with Parents..

Blocked by https://github.com/devfile/api/issues/1067


Zooming causes the cells selected to be displaced

When you zoom in or out while selecting cells and moving them the cells are displaced unexpectedly when not moving them. Zooming causes the cells to be moved. This also makes the cursor be at a different position than the selected cells.

It's not clear to me how to reproduce the bug: How can you zoom, select and moving cells at the same time?

To reproduce the bug do:

Select a group of cells
Move the cells
Use the mouse wheel while holding the left mouse button
The more you move the cells before zooming in or out the more it becomes displaced.

Hm, in my case it worked as expected, see
https://github.com/chrxh/alien/assets/73127001/61997450-04e5-4d14-a6df-432c5364a037

Oh it doesn't happen when you have it paused.

I was able to reproduce the bug.
It's fixed in https://github.com/chrxh/alien/commit/cb1a98f0b3dab39b60c7b03ceb968f35fb56f5ab (see https://github.com/chrxh/alien/assets/73127001/a2bbed96-b859-4970-b246-3939656c8743)
The fix will be contained in the next update.


Delete not working on german YT

Bug
Clicking trash icon in the menu opens context menu.

Possible Fix
Check on Trash Icon Path /html/body/ytd-app/ytd-popup-container/tp-yt-iron-dropdown/div/ytd-menu-popup-renderer/tp-yt-paper-listbox/ytd-menu-service-item-renderer[4]/tp-yt-paper-item/yt-icon/svg/g/path

Thanks for the report and hint! Sorry for this major oversight :disappointed:

Closing: This is now fixed in the 1.0.1 release. If you still have any problems, feel free to re-open the issue.


update content in zh assign-cpu-resource docs

update content in zh assign-cpu-resource docs
some kubectl commands wrong in docs

nice!
/lgtm
/approve


Manually link JWTDecode and SimpleKeychain

possible fix for us not using Carthage

[ ] All new/changed/fixed functionality is covered by tests (or N/A)
[ ] I have added documentation for all new/changed functionality (or N/A)

📋 Changes

📎 References

🎯 Testing

should we push this to https://github.com/auth0/Auth0.swift instead of forking?

Is this related to the thing they are actively working on here https://github.com/auth0/Auth0.swift/issues/870

@Kevin-J-Harrison doesn't look like the same thing. As far as I could tell our issue was that the dependency of Auth0 on JWTDecode only implicit in our build order, not explicit, so it was a race condition whether or not JWTDecode would be done building before Auth0 started

@1419krusty We could try, it seems to go against their preferred Carthage/Swift way of dependency management so I doubt they'd be too excited, and we'd probably have to make sure the iOS, tvOS, visionOS, watchOS targets worked with the same change.

I've made a feature request and a PR in the Auth0.swift repos:
https://github.com/auth0/Auth0.swift/issues/872
https://github.com/auth0/Auth0.swift/pull/873

Nice!
Just for my curiosity, is there a reason we aren't using the swift package?

yes, AppStore build configurations break swift packages because SPM expects debug or release only


chore(en): Added Linear-search docs in English

What kind of change does this PR introduce? (check at least one)

[ ] Bugfix
[ ] New algorithm
[ ] Optimization in previous algorithms
[ ] Code style update
[ ] Refactor
[x] Documentation
[ ] Other, please describe:

Added Linear Search Algorithm documentation in English according to the template given.

Please check now, Resolved the following:
Bundled the procedure as code,
corrected the error in Algorithm Description


can I use with any parameter???

hi. how can I use any parameters something like:
example.com/list/nature

Naturally, you can change it to your liking. For more advanced configurations, see our upMVC system.


[Bookmarks section] Fixed Major Bugs

Description
With this PR merged, the Bookmarks section will now display when there are any Bookmarks associated with a specific category.
Bug Fix Report :
Hey @JasonDsouza212, first of all sorry for being late with the PR, actually what happened was that while working onto the enhancement, I've come across another major bug that was going around with the Bookmarks section. The following were the bugs :

If there're no Bookmarks we can easily display that there're No Bookmarks from this section
Now initially NULL was set as a default case if there are no bookmarks,

https://github.com/JasonDsouza212/free-hit/blob/4993549a9bdaff0f091488c21557e898657eaee4/src/components/bookmarks.jsx#LL39C16-L39C16

as a result nothing was rendered into the page.
However, with this PR merged, the above issue will be resolved.

☑  Good News, I've worked around with all the bugs that were there in the Bookmarks section
Checklist

[x] I am making a proper pull request, not spam.
[x] I've checked the issue list before deciding what to submit.
[x] I ensured that the page remains responsive after the addition of the changes
[x] I've fixed all possible bugs that are in the Bookmarks section

Related Issues or Pull Requests
This PR closes #306
Add relevant screenshot or video (if any)
screen-capture (3).webm

Really, that's an amazing contribution, @pooranjoyb. Keep up the good work!

Really, that's an amazing contribution, @pooranjoyb. Keep up the good work!

Most Welcome @JasonDsouza212 Will be up for more productive Issues. Thanks! ❤


Improve hive_generator's dartx version to 0.3.0

I am trying to use hive on my project and getting this error:
Because hive_generator >=0.6.0 depends on dartx ^0.2.0 and mydenox_flutter depends on dartx ^0.3.0, hive_generator >=0.6.0 is forbidden.
So, because mydenox_flutter depends on hive_generator ^0.7.0+2, version solving failed.
pub get failed (1; So, because mydenox_flutter depends on hive_generator ^0.7.0+2, version solving failed.)

This is caused because hive_generator uses an old version of dartx

@shinayser I have a similar problem — hive_generator does not create the .g part at all, although there are no semantic/syntax errors.
This is using the sample project from the Hive docs, but it happens with other projects as well — latest Flutter and hive (running w/ latest Visual Studio Code on macOS).
Should I open a separate issue?
Thank you.

@adario definetly a separated issue. My issue happens even before compiling the project.

@shinayser you're right, of course — for some reason, a flutter clean was needed... not sure why, though.

Hi guys any news about this?
I have the same compatibility problem
Because no versions of hive_generator match >0.7.0+2 <0.8.0 and hive_generator 0.7.0+2 depends on dartx ^0.2.0, hive_generator ^0.7.0+2 requires dartx ^0.2.0. And because no versions of dartx match >0.2.0 <0.3.0, hive_generator ^0.7.0+2 requires dartx 0.2.0. And because dartx 0.2.0 depends on characters ^0.3.0 and lottie >=0.3.0+1 depends on characters ^0.5.0, hive_generator ^0.7.0+2 is incompatible with lottie >=0.3.0+1.

I think that I have the same problem here.
Running "flutter pub get" in app...
Because hive_generator >=0.6.0 depends on dartx ^0.2.0 which depends on quiver >=2.0.3 <2.1.0, hive_generator >=0.6.0 requires quiver >=2.0.3 <2.1.0.

And because every version of flutter_test from sdk depends on quiver 2.1.3, hive_generator >=0.6.0 is incompatible with flutter_test from sdk.

So, because app depends on both flutter_test any from sdk and hive_generator ^0.6.0, version solving failed.
pub get failed (1; So, because app depends on both flutter_test any from sdk and hive_generator ^0.6.0, version solving failed.)

As a temporary workaround, I'm using hive_generator: ^0.5.2

Assuming dartx adheres to semantic versioning and doesn't contain breaking changes (like, removing features) up to its next major version 1.0.0, hive_generator could depend on dartx: >= 0.2.0 < 1.0.0, making itself independent from the minor releases of dartx.

Is there any update on this? I'd love to use this package on my app but I cannot with this current issue

To anyone joining this issue thread just now: Three PRs have already been opened (#277, #318, #320) trying to fix this — please don't write yet another one. 😅
@ElZombieIsra It seems like package maintainer @leisim has shift priorities for the moment, but will be back soon. Then, a new version can be published.
While using dependency_overrides in production code is dangerous (because you're forcing possibly incompatible packages together), note that you can also always depend on one of the forks using git, which is safe as far as package-resolution goes.

After adding:
dependency_overrides:
  dartx: ^0.3.0

Because dartx >=0.2.0 <0.4.0 depends on characters ^0.3.0 and every version of flutter_test from sdk depends on characters 1.0.0, dartx >=0.2.0 <0.4.0 is incompatible with flutter_test from sdk.
So, because my_app depends on both flutter_test any from sdk and dartx ^0.3.0, version solving failed.

Any assistance would be greatly appreciated!

I think that I have the same problem here.
Running "flutter pub get" in app...
Because hive_generator >=0.6.0 depends on dartx ^0.2.0 which depends on quiver >=2.0.3 <2.1.0, hive_generator >=0.6.0 requires quiver >=2.0.3 <2.1.0.

And because every version of flutter_test from sdk depends on quiver 2.1.3, hive_generator >=0.6.0 is incompatible with flutter_test from sdk.

So, because app depends on both flutter_test any from sdk and hive_generator ^0.6.0, version solving failed.
pub get failed (1; So, because app depends on both flutter_test any from sdk and hive_generator ^0.6.0, version solving failed.)

As a temporary workaround, I'm using:
dependency_overrides:
  dartx: ^0.3.0

As you can find here: #203 (comment)

its work for me, thanks!

After adding:
dependency_overrides:
  dartx: ^0.3.0

I'm getting the following error:
Because dartx >=0.2.0 <0.4.0 depends on characters ^0.3.0 and every version of flutter_test from sdk depends on characters 1.0.0, dartx >=0.2.0 <0.4.0 is incompatible with flutter_test from sdk.
So, because my_app depends on both flutter_test any from sdk and dartx ^0.3.0, version solving failed.

Any assistance would be greatly appreciated!

dependency_overrides:
  dartx: ^0.3.0
  collection: ^1.14.13
  characters: ^0.3.0

This is working for me


Tunnel is disconnected almost immediately

Hi Folks,
First, let me start with big kudos for the solution!
I'd like to make it as self-hosting solution, but unfortunately even though I followed the readme I was not able to succeed.
The problem I've encountered is that server is running and accepting incoming connections but dropping them almost immediately.
On server side I have a linux box with ubuntu server.
I've tried both dockerized version and "standard" one.
My run command is node -r esm ./bin/server --port 3000 --domain tun.sub.domain.com --secure true
// exact name has been changed to tun.sub.domain.com //
Added some log.debug's and on the server side I got:
localtun_1  | [debug]   server listening on port: 3000
localtun_1  | [debug]   undefined
localtun_1  | [debug]   tcp server listening on port: 42405
localtun_1  | [debug]   removing client: mighty-skunk-48
localtun_1  | [debug]   closed tcp socket
localtun_1  | [debug]   closed tcp socket
localtun_1  | [debug]   server listening on port: 3000
localtun_1  | [debug]   undefined
localtun_1  | [debug]   tcp server listening on port: 46035
localtun_1  | [debug]   removing client: helpless-cougar-46
localtun_1  | [debug]   closed tcp socket
localtun_1  | [debug]   closed tcp socket

While on the client side
//
I have it behind nginx listening with SSL on port 3333 (config from your repo localtunnel-nginx) that's why you see port 3333 on client side
 ✘  npx localtunnel --port 8000 --host https://tun.sub.domain.com:3333
your url is: https://mighty-skunk-48.tun.sub.domain.com:3333

but no traffic and /api/status reply with {tunnels:0...}
 ✘  npx localtunnel --port 8000 --host https://tun.sub.domain.com:3333
your url is: https://helpless-cougar-46.tun.sub.domain.com:3333

but no traffic and /api/status reply with {tunnels:0...}
Actually I've discovered that if I hit /api/status fast enough I got reply with {tunnels:1...} for a friction of a second.
Any ideas what's wrong here?
Thanks a lot!

I had a similar issue. Turns out I had to open all the ports for my server in the firewall as Client tries to connect to assigned port on the go.
I hope this helps!

I had a similar issue. Turns out I had to open all the ports for my server in the network firewall as Client tries to connect to assigned port on the go.
I hope this helps!

I have stuck on that issue whole day!
It works, thanks a lot! 😄

I had a similar issue. Turns out I had to open all the ports for my server in the network firewall as Client tries to connect to assigned port on the go.
I hope this helps!

may you instruct how to to that?

@rsurgiewicz I'm also experiencing this odd behaviour.
But when I use the IP address of the VPS on the lt command it works fine:

Replicating the issue (my lt server is running on port 80):

> lt --host https://my.tunnel.cloud --port 3000 -s dumberror

However this works perfectly fine:

> lt --host <IP_ADDRESS> --port 3000 -s dumberror

Same server. Same instance of the localtunnel server. Both return https://dumberror.my.tunnel.cloud but only the second instance survives long enough to receive the request.
I am inclined to think this is some sort of bug with the client actually.
If you see how lt client works. It tries to connect to the IP and port specified by the lt server. The server responds as expected.
I logged the return of _getInfo in the client repo. In the first non-working case this is the info:
 {
  name: 'dumberror',
  url: 'http://dumberror.my.tunnel.cloud',
  cached_url: undefined,
  max_conn: 10,
  remote_host: 'my.tunnel.cloud',
  remote_ip: undefined,
  remote_port: 46199,
  local_port: 3000,
  local_host: undefined,
  local_https: undefined,
  local_cert: undefined,
  local_key: undefined,
  local_ca: undefined,
  allow_invalid_cert: undefined
}

The second case where it does work:
 {
  name: 'dumberror',
  url: 'http://dumberror.my.tunnel.cloud',
  cached_url: undefined,
  max_conn: 10,
  remote_host: '<IP_ADDRESS>',
  remote_ip: undefined,
  remote_port: 46771,
  local_port: 3000,
  local_host: undefined,
  local_https: undefined,
  local_cert: undefined,
  local_key: undefined,
  local_ca: undefined,
  allow_invalid_cert: undefined
}

Ok so when I give the host IP explicitly it works but not with the domain. This must mean that there's an issue with my DNS setup.
I set up my domain on cloudflare. DNS is an A record. I want HTTPS built in. So I use the cloudlfare DNS with the proxy setting on.
This results in a dig my.tunnel.cloud returning some other IP address.
This is what breaks the connection between the client and the server
The client requests a connection on the server. It responds with the port that it should connect to. However, because the host is the my.tunnel.cloud and CF is proxying it - the client does not have access to that port on CF's proxy servers.
So what is the solution? Turn off proxy on CF? No because I want https automatically. Is it to hardcode the IP on my lt client request? No that's ugly.
I think the compromise is for the lt server to add it's own IP address in the tunnel creation request response json and for the info field to infer the IP of the lt server from that response.

Unsurprisingly, others have also figured out the above.
In the client code, it can already consume the ip property from the json response to tell the client explicitly where to connect to.
All you have to do is add your explicit IP address in server.js:
        const url = schema + '://' + info.id + '.' + opt.domain || ctx.request.host;
        info.url = url;
        info.ip = '<IP_ADDRESS>'; // <<<<< add this line with your server IP address
        ctx.body = info;
        return;

https://github.com/localtunnel/server/blob/78c06f8fc43ce0eacea2aeb88f2d1d63afaf4e19/server.js#L110
then the client will override the inferred host with the explicit IP address provided by the server. IMO it should be a cli flag.
Hope that helps

Ok wow it turns out someone did think of this and somehow implemented it half way.
Step 1.
Change ./lib/server.js
const server = CreateServer({
    max_tcp_sockets: argv['max-sockets'],
    secure: argv.secure,
    domain: argv.domain,
    address: argv.address, // <<add this line
});

Step 2.
Change ./server.js in the same location as above
        const url = schema + '://' + info.id + '.' + opt.domain || ctx.request.host;
        info.url = url;
        info.ip = opt.address; // <<<<< add this line with your server IP address
        ctx.body = info;
        return;

Then make sure to add your explicit ip address with the --address flag:
> lt --host https://my.tunnel.cloud --port 3000 -s dumberror --address <IP_ADDRESS>

That's it. Now it will work as expected. smh.

May I know where you are hosting the localtunnel server? I am using https://render.com/ but that is not working.


Simple web service DAOs: No syncing or versioning

This removes all syncing/versioning logic from web service DAOs, replacing it with a JSON file-backed scheme. The new scheme will be safe to run in production under the following constraints:

We add custom health checks to ensure that JSON data have been loaded before instances receive traffic (to be in a follow-up PR);
The release process includes pushing new service code whenever we push new data; (then, once the new instances are ready, manually redirect traffic to the new instances with new data).

Codecov Report

Merging #224 into new-api-extractor will increase coverage by 0.86%.
The diff coverage is 22.44%.

@@                  Coverage Diff                  @@
##           new-api-extractor     #224      +/-   ##
=====================================================
+ Coverage              63.59%   64.45%   +0.86%
=====================================================
  Files                     39       34       -5
  Lines                   2461     2144     -317
  Branches                 171      146      -25
=====================================================
- Hits                    1565     1382     -183
+ Misses                   896      762     -134

Impacted Files
Coverage Δ

lib/confluence/browser_metric_data.es6.js
62.5% <ø> (-7.5%)
:arrow_down:

lib/web_apis/release.es6.js
35% <ø> (-5.91%)
:arrow_down:

lib/web_catalog/object_graph_importer.es6.js
34.61% <ø> (ø)
:arrow_up:

lib/web_apis/web_interface.es6.js
100% <ø> (ø)
:arrow_up:

lib/confluence/api_velocity_data.es6.js
100% <ø> (ø)
:arrow_up:

lib/web_apis/release_interface_relationship.es6.js
100% <ø> (ø)
:arrow_up:

lib/dao_container.es6.js
100% <ø> (+95.45%)
:arrow_up:

lib/local_json_dao.es6.js
17.14% <17.14%> (ø)

lib/json_dao_container.es6.js
31.57% <35.71%> (ø)

lib/confluence/set_ops.es6.js
87.17% <0%> (-4.49%)
:arrow_down:

... and 9 more

Continue to review full report at Codecov.

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

@arobins, to get serve.js working correctly again, I applied some bug fixes. PTAL and LMK if you want any of these changes punted to a separate PR. They're not very big.


Resubmitting - Version 1 progress report B

Use Jinja Template for Displaying Projects
-Use Jinja template to dynamically display projects on the home page.

Update Project Folder Structure -Update project folder structure: added 'project1' folder with description.txt, image, PDF, and link files.

Implement Clickable Image for Prototype -Make project image clickable; redirects to prototype link in Figma when clicked.

made a small change as image is going out of the container. I will close this and create a new pull request.


Admin password not rate limited

It appears that the check for the username against the password of the admin is not rate limited.

Fixed with 48daf30c93


Update BLS test format: output null for invalid case

In #2982, we updated our test vector YAML representer to output null when the value is Python None.
Before that PR, it was:
output:

After that PR, it becomes:
output: null

IMHO it's better to use only one rule for both cases. This PR changes the BLS test format to align with sync test format. null seems clearer than empty.
p.s. null is used in engine APIs and that was why I changed our YAML representer in #2982
/cc @asanso for updating bls12-381-tests test suite.

lgtm


fix(@angular-devkit/build-webpack): remove RxJS path mapping

<EMAIL_ADDRESS>added back the module and es2015 keys in the package.json and the path-mapping.js is retained for backward compatibility only.

Heya @sarunint, I think this change makes sense but ngtools/webpack still needs to support RxJs 5.6. So I don't think we can remove it right now.
@hansl can you confirm?

Angular 5 has a minimum rxjs version of 5.5.  Until Angular 5 support is dropped this cannot be removed.


[BUG]Unable to install Android 12 L Beta 2 framework-res.apk

Information

Apktool Version (apktool -version) - 2.4.1 or 2.6.0, I guess it doesn't matter...
Operating System (Mac, Linux, Windows) - Windows (using Power Shell)
APK From? (Playstore, ROM, Other) - redfin Android 12 L Beta 2 official firmware

Stacktrace/Logcat
Exception in thread "main" brut.androlib.AndrolibException: Invalid chunk type: expected=0x00000200, got=0x00000206
at brut.androlib.res.decoder.ARSCDecoder.checkChunkType(ARSCDecoder.java:542)
at brut.androlib.res.decoder.ARSCDecoder.readTablePackage(ARSCDecoder.java:88)
at brut.androlib.res.decoder.ARSCDecoder.readTableHeader(ARSCDecoder.java:82)
at brut.androlib.res.decoder.ARSCDecoder.decode(ARSCDecoder.java:48)
at brut.androlib.res.decoder.ARSCDecoder.decode(ARSCDecoder.java:40)
at brut.androlib.res.AndrolibResources.installFramework(AndrolibResources.java:872)
at brut.androlib.res.AndrolibResources.installFramework(AndrolibResources.java:854)
at brut.androlib.Androlib.installFramework(Androlib.java:707)
at brut.apktool.Main.cmdInstallFramework(Main.java:269)
at brut.apktool.Main.main(Main.java:82)
Steps to Reproduce

'java -jar .\apktool_2.6.0.jar if .\framework-res.apk'

Questions to ask before submission

Have you tried apktool d, apktool b without changing anything?
N/A
If you are trying to install a modified apk, did you resign it?
N/A
Are you using the latest apktool version?
Yes.

If we try and install framework-res.apk from Android 12, we can't decompile Android 12 L Beta 2 as well.
We get the same "Invalid chunk type: expected=0x00000200, got=0x00000206"

Its possible this is related to https://github.com/iBotPeaches/Apktool/issues/2714, but I would like a sample application or link to where I can pull Android 12 L, I'm not seeing that nomenclature when I try and pull platforms from SDK Manager.

Nevermind - not needed. I updated to latest API 31 and I now replicate issue. Put up a broken PR in meantime - https://github.com/iBotPeaches/Apktool/pull/2743

The same issue is now occurring on March Android 12 build. Just thought on letting you know.

Thanks - I started looking at it this weekend as these applications are increasing in quantity.

Need to add support for <staging-public-group-final> which will really only be in AOSP apps as they build against future SDKs, but can easily reassign resources once API stabilizes.

https://github.com/aosp-mirror/platform_frameworks_base/commit/2fedba9a32d9e92344eaf6e9faf5b43e1bc2ae70

          <public type="attr" name="finalized_res" id="0x01010001"/>
         <public type="attr" name="staged_s_res" id="0x01010002"/>
         <public type="attr" name="staged_s2_res" id="0x01010003"/>
         <public type="string" name="staged_s_string" id="0x01020000"/>
         <!-- S staged attributes (support staged resources in the same type id) -->
         <staging-public-group-final type="attr" first-id="0x01010050">
           <public name="staged_s_res" />
         </staging-public-group-final>
         <staging-public-group-final type="string" first-id="0x01fd0080">
           <public name="staged_s_string" />
         </staging-public-group-final>
         <!-- SV2 staged attributes (support staged resources in a separate type id) -->
         <staging-public-group-final type="attr" first-id="0x01ff0049">
           <public name="staged_s2_res" />
         </staging-public-group-final>
         <!-- T staged attributes (support staged resources in multiple separate type ids) -->
         <staging-public-group type="attr" first-id="0x01fe0063">
           <public name="staged_t_res" />
         </staging-public-group>

We also have 2 new types for overlay and policy.

https://github.com/aosp-mirror/platform_frameworks_base/commit/75e20dda2b1ef3d2593b02f6a366a02656c1c89d

<overlayable>
   <!-- Any overlay on the product_services, vendor, or product partition can overlay the value of
  @string/overlayable3 -->
   <policy type="product_services|vendor|product">
       <item type="string" name="overlayable3" />
   </policy>
</overlayable>

So working on issue-2740 branch and just going to skip all data to get disassembling working again, then slowly add support for the types.

Taking this AndroidL / API32 a bit differently than other upgrades. We have many things to do

Support the new ASRC changes (roughly done)
Adopt those ASRC changes into viable format to re-use/re-parse during compilation
Update aapt2 sources for new support

If we look at that list, the first one is done, but in pure skip mode. You can tell from this output
➜  2774 apktool d framework-res.apk -f
Picked up _JAVA_OPTIONS: -Dawt.useSystemAAFontSettings=on -Dswing.aatext=true
I: Using Apktool 2.6.2-df3b9f-SNAPSHOT on framework-res.apk
I: Loading resource table...
I: Skipping staged alias stagedId (1fe0000) finalId: 1020055
I: Skipping staged alias stagedId (1fe0001) finalId: 1020056
I: Skipping staged alias stagedId (1fe0002) finalId: 1020057
I: Skipping staged alias stagedId (1ff0000) finalId: 101064c
W: Skipping package group: android
I: Decoding AndroidManifest.xml with resources...
I: Regular manifest package...
I: Decoding file-resources...
I: Decoding values */* XMLs...
I: Copying assets and libs...
I: Copying unknown files...
I: Copying original files...
➜  2774

That we skip the data that was leading to the crash. The problem being is that information is used in aapt2 so it knows how to properly map assets during API development (ie releasing apps before API32 is finalized)
So I know that information has to be somewhere and right now we are just skipping it, effectively stopping the crash but sure as hell not making it work.
So I'm going to merge a PR shortly that refs this PR, but does not close it, then focus the aapt2 upgrade then loop back and find affected applications to see how we can boost test suite to properly account for staged aliases and overlays (if we find any)

I've merged updates to aapt2/aapt for these new properties. Still have:

Adopt those ASRC changes into viable format to re-use/re-parse during compilation

Remaining

During the large refactors of 2.8.0/2.8.1 and some of the upcoming 2.8.2 changes - this has been fully resolved.
Closing.


OPTION: Compress only a specified subset of img formats

Is there a way to compress only certain images? Like, only ["jpg", "webp"]

Sorry, made a mistake on my part.


Modify metadata in the human rendering wrapper

@pseudo-rnd-thoughts Does this solve #11? (as a proof-of-concept for now - I'll need to check other wrappers, and in some way handle duplicated render modes that might occur) I'm slightly confused because you were talking about overwriting the dict stuff on call, but I don't see any issues with this implementation (which you also seem to have proposed in the issue), so maybe I'm not seeing something.

Yes, Im not sure why I was thinking it would need to be more complex.
We also need to update the RenderCollection wrapper, could you do that as well
I can't think of anymore wrappers that need to be updated


hide ip row in container provider summary screen when value is nil.

@simon3z @abonas

@moolitayer @abonas @simon3z For PRs with UI changes, please provide before and after screenshots to help the reviewers.

actually this is wrong. turns out 'hostname' row always contains hostname/ip so I should just remove the ip row.

before:

after:

I'm wondering whether this should be a single PR, or a cross provider aligned thing (containers/infra/cloud)
Because iirc, the elimination of usage of ipaddress in favor of hostname was a cross all providers change.

@abonas I could test, however I do not have any non container providers currently.
I can add ovirt as a cloud provider I assume?
I could check that tomorrow.

@abonas I could test, however I do not have any non container providers currently.
I can add ovirt as a cloud provider I assume?
I could check that tomorrow.

yes, oVirt is considered an Infra provider

@abonas it would be nice to remove ip address from all other providers as well but it requires some research and it can be done later.
On the other hand @moolitayer has too many patches open at the moment and this one is done.
:+1: for merging this early (and remove ip address from other providers later) @Fryguy @blomquisg @chessbyte

ps @abonas @simon3z after first inspection i'm not sure this is relevant to other providers.
See:

I could not however add a provider using hostname, or edit it due to permissions.
(only the discovery worked).
See:

@abonas it would be nice to remove ip address from all other providers as well but it requires some research and it can be done later.
On the other hand @moolitayer has too many patches open at the moment and this one is done.
for merging this early (and remove ip address from other providers later) @Fryguy @blomquisg @chessbyte

np to merge early from my side.
as for other provider types, I remember that @jrafanie did some changes in this area, maybe he can shed some light on the topic.

@moolitayer can you rebase/push this?

@chessbyte :+1: ready!

@miq-bot assign @moolitayer


Added Cytron Maker Feather AIoT S3.

Added support for new board.

@hathach May I know how often a new release will be created?

we made release when needed (or requested for new boards) or there is major code changes. I will made a release soon enough.

we made release when needed (or requested for new boards) or there is major code changes. I will made a release soon enough.

Ok noted. Thanks.


ScalaFX  v.21.0.0-R32

ScalaFX  v.21.0.0-R32 changed the EventTarget handling,  adapting to the changes in JavaFX 21.
https://github.com/scalafx/scalafx/releases/tag/v.21.0.0-R32
DBTarzan code does not compile with this library and has to be changed.

DBTarzan upgraded to JavaFX 21 and ScalaFX v.21.0.0-R32.
Had to change the snapcraft YAML file to use pure GTK3


adding mce generation

https://issues.redhat.com/browse/ACM-8475
I have moved the main bundle generation functionality to this repo, where it will be used by a wrapper script within the MCH and MCE repos

/lgm

/lgtm


Store equilibrium quantities in output file

This branch implements functionality to store equilibrium quantities (primarily flux surface contours so far) in the output file. This can be used to generate more accurate visualizations of e.g. pellet shard trajectories.

This needs to be tested with the ANALYTIC and CYLINDRICAL radial grids before merging!

This needs to be tested with the ANALYTIC and CYLINDRICAL radial grids before merging!

This has now been checked and confirmed to work. I also improved the plotting by ensuring that the flux surfaces are closed.


SCM viewlet in no folder worksapce

No folder workspace
Open scm viewlet
notice how the commit box is present, upon adding a commit message - you get a warning
I would expect an 'open folder' button

Or even better we could improve this whole experience - intern project ^_^

#18615
( •_•)
( •_•)>⌐■-■
(⌐■_■)


Reduce the size of distribution tar

Before:
-rw-r--r--   1 kezhenxu94  staff   948M Mar 20 16:00 apache-dolphinscheduler-2.0.4-SNAPSHOT-bin.tar.gz

After:
-rw-r--r--   1 kezhenxu94  staff   792M Mar 20 15:39 apache-dolphinscheduler-2.0.4-SNAPSHOT-bin.tar.gz

Codecov Report

Merging #9019 (5335362) into dev (48dc0a0) will decrease coverage by 0.00%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                dev    #9019      +/-   ##
============================================
- Coverage     39.76%   39.76%   -0.01%
+ Complexity     4264     4263       -1
============================================
  Files           808      808
  Lines         32551    32551
  Branches       3649     3649
============================================
- Hits          12945    12944       -1
- Misses        18379    18380       +1
  Partials       1227     1227

Impacted Files
Coverage Δ

...lphinscheduler/rpc/common/AbstractRpcCallBack.java
0.00% <0.00%> (-100.00%)
:arrow_down:

Continue to review full report at Codecov.

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

@lenboo  Can you help link the corresponding milestones?


[En Garde] Add die modifiers of battle outcomes

Submission Checklist
Required

[x] The pull request title clearly contains the name of the sheet I am editing.
[x] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[x] The pull request makes changes to files in only one sub-folder.
[x] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

Changes / Description

Add die modifiers of battle outcomes
-1 dice roll modifier to  Death rolls for Battalion/Squadron Commanders
+1  dice roll modifier Mentions and Promotion rolls for Battle result better than superiors

Character Sheet Info Roll20 Internal Use only.


feat(multisig): query to get current key epoch or key ID by epoch.

Description
Allows Axelarons to query the latest 16 keys included in the commitment on the EVM gateway so that we can ensure all batches signed by those key IDs have been executed.
Todos

[ ] Unit tests
[ ] Manual tests
[ ] Documentation
[ ] Connect epics/issues
[ ] Tag type of change
[ ] Upgrade handler

Steps to Test
Expected Behaviour
Other Notes

@jack0son I am not entirely sure about exposing the concept of epoch externally. What about just a query such as GetActiveKeyIDs which returns all the key IDs that are currently considered active for the given chain?

am not entirely sure about exposing the concept of epoch externally. What about just a query such as GetActiveKeyIDs which returns all the key IDs that are currently considered active for the given chain?

Sure that would be fine too - is this a query we can add without doing a network upgrade? My understanding was that the multisig module has its own separate notion of ActiveKeys which is separate from the 16 that are active on the EVM gateway.


Delete fork

I transferred ownership of this repo so the repo settings and issues would be transferred, so the fork at https://github.com/roboblazers7617/Code_Standards can be deleted.

Just to confirm, you want me to delete the Code_Standards repository (with capital letters), the one that hasn't been updated in 4 days.

@fodfodfod Yes. The repo to delete (roboblazers7617/Code_Standards) should show as being forked from this one.

The repository has been deleted


Add an ersatz filtering mechanism to XCTestScaffold.

Because SwiftPM is not (yet) aware of swift-testing, swift test --filter does not have any effect on swift-testing tests. This PR adds temporary support for an environment variable, "SWT_SELECTED_TEST_IDS", to stand in for --filter until such time as SwiftPM can be modified to support swift-testing.
Note that the supported value of this environment variable is (intentionally) constrained to just test IDs right now. Matching display names or other metadata about tests is certainly possible, but it's beyond the scope of today's PR.

@swift-ci please test

@swift-ci please test


unix,win: add uv_os_{get,set}priority()

LGTM. Wasn't there an issue somewhere where this was discussed?

https://github.com/nodejs/node/pull/21675

Updated to try to test on zOS.
CI: https://ci.nodejs.org/view/libuv/job/libuv-test-commit/993/
The only failures on win2012r2-vs2017 and alpine-last-latest-x64 are unrelated.

Force pushed with lease to replace REPLACEME with 1.23.0. New commit is e57e07172eba9df2f8fa0cec051cac0c5cac9124.


Update android 13 permissions

Summary
Added permissions introduced in 13.
https://github.com/facebook/react-native/pull/33471

Test Plan
Run the example app and play with added permissions

What's required for testing (prerequisites)?
What are the steps to test it (after prerequisites)?
Compatibility

OS
Implemented

iOS
❌

Android
✅

Checklist

[x] I have tested this on a device and a simulator
[x] I added the documentation in README.md
[x] I added a sample use of the API in the example project (example/App.tsx)

LGTM! I will make a release this afternoon

@bang9 It's released: https://github.com/zoontek/react-native-permissions/releases/tag/3.6.0

@bang9 It's released: https://github.com/zoontek/react-native-permissions/releases/tag/3.6.0

Super fast! Thanks


EPIC: E2E Test Plan

This issue aims to describe the e2e test plan for the Block Node, each sub-plan divided by service (eg. Storage, State)
### gRPC
- [ ] https://github.com/hashgraph/hedera-block-node/issues/184
- [ ] https://github.com/hashgraph/hedera-block-node/issues/186

### Storage
- [ ] https://github.com/hashgraph/hedera-block-node/issues/187

### State

Disclaimer: Test Plan is incomplete and subject to changes and extentions.

Carve out a separate Epic for Ph-2 for the remaining work. @georgi-l95


build moar-blead compile error

Cloning the repo and then running rakudobrew build moar-blead produces this error after a few minutes of compiling:
from src/Perl6/Compiler.nqp:38  (blib/Perl6/Compiler.moarvm:command_eval)
 from gen/moar/stage2/NQPHLL.nqp:1838  (/home/hythm/.rakudobrew/versions/moar-blead-2019.03.1/install/share/nqp/lib/NQPHLL.moarvm:command_line)
 from gen/moar/main.nqp:48  (perl6.moarvm:MAIN)
 from gen/moar/main.nqp:1  (perl6.moarvm:<mainline>)
 from <unknown>:1  (perl6.moarvm:<main>)
 from <unknown>:1  (perl6.moarvm:<entry>)
make: *** [Makefile:513: CORE.setting.moarvm] Error 1
Command failed (status 512): make
Failed running /usr/bin/perl5.28.1 Configure.pl --backends=moar --gen-moar=master --gen-nqp=master --git-reference="/home/hythm/.rakudobrew/bin/.

@hythm7 This might be related to a change in rakudo that has been fixed a few hours ago. Can you try again?

@patzim That doesn't look like steming  from a build problem. Though more diagnostics would be useful.

I'm getting this same issue. At the very least its obvious the path it is building into is not correct:
/Users/nicklogan/.rakudobrew/versions/moar-blead-2019.03.1/

which should just be
/Users/nicklogan/.rakudobrew/versions/moar-blead/

Also rakudobrew build moar-blead is trying to build moarvm/nqp for 2019.03, not HEAD


Failed to move Surveillance Station

Moving process was failed with error (didn't saved it as a text - was something about application state), and Surveillance Station seems was moved to volume1 (as far I checked paths), but was displayed  as it is still installed on the volume2. Latter I found other apps like cloud sync was dissappeared from the app menu, and wrong old location. I tried reinstall (without uninstall) SS app from the spk, and this time moving was done. But I lost my camera setings and storage of its recordings was disassocitated from the camera. I hoped it will be reconnected after adding camera to SS with the same name, but it was not.
Restoring drives from the backup now (forgot to create a snapshot before moving).
p.s. during the first moving SS there was error like ' already exist folder/path /volume1/../lib/ffmpeg' on the destination volume (possible some old installation garbage). In the new SS this path is a symlink to /var/...

Pity you didn't save a screenshot of the script's output.

Yep, and restoring from my backup didn't help me - my SS settings lost. (i didn't restored from backup a volume where is surveillance share mounted - bcs it not included to backup). So after first login DSM asked me to Repair all my apps.
P.S. my dsm is virtual machine

I still have some data on the volume2 (original instalation volume), and will try to copy/backup/swap from /volume2/@appstore/SurveillanceStation to /volume1/@appstore/SurveillanceStation

I think he got this error
`You selected Surveillance Station in /volume1
Destination volume is /volume2
Checking size of Surveillance Station
Size of selected app(s) is 461 MB
Ready to Move Surveillance Station to /volume2? [y/n]
y
Copying /volume1/@appstore/SurveillanceStation to /volume2
Removing /volume1/@appstore/SurveillanceStation
Moving /volume1/@apptemp/SurveillanceStation to /volume2
cp: '/volume2/@appconf/SurveillanceStation' and '/volume2/@appconf/SurveillanceStation' are the same file
Line 583: ERROR Copying /volume2/@appconf/SurveillanceStation to /volume2 failed!
progstatus called from move_pkg_do line 847
`

@slimline33 Which version of the script are you using?

@007revad latest. <IP_ADDRESS>

@007revad but I see you changed something 4 days ago. Sorry, I can not try the newest version. Because I reinstalled it manually.

Line 583: ERROR Copying /volume2/@appconf/SurveillanceStation to /volume2 failed!
progstatus called from move_pkg_do line 847

Those line numbers don't match v3.1.60
If you still have the syno_app_mover.sh file that you used can you run the script with the -v option which will just show the version.

Line 583: ERROR Copying /volume2/@appconf/SurveillanceStation to /volume2 failed!
progstatus called from move_pkg_do line 847

@slimline33  Those line numbers don't match v3.1.60
If you still have the syno_app_mover.sh file that you used can you run the script with the -v option which will just show the version.


add support for configuring /etc/hosts

Some support seems to exist. It would be great if there was the option to override more. I haven't coded go, but I imagine you would allow a format like
manage_etc_hosts:
      - "<IP_ADDRESS> localhost"
      - "<IP_ADDRESS> someotherdomain"

Write now, this done via the 'write_files' directive. If you drop a custom /etc/hosts in, make sure you remove the "manage_etc_hosts" line in your yml file though or else it will overwrite your custom file with a blank "<IP_ADDRESS> localhost" one that it does by default.
Supporting this component of cloud-config seems long overdue to me, but because the fix is quite easy, I can see why it's been overlooked. But that overwriting part of it was a bit annoying to figure out for sure.

@brianclements Can you please list the steps in order to make the required change you mentioned above.
How would i write the write_files ?

write_files:
   - path: /etc/hosts
     content: |
        <IP_ADDRESS>     localhost
        x.x.x.y          somehost
        ....


[-wATCH-]— Mujib Zalmay Video Original Video Link Mujib Zalmay Video Viral On Social Media X Trending Now

20 seconds ago
L𝚎aked Video Mujib Zalmay Original Video Viral Video L𝚎aked on X Twitter Telegram
[-wATCH-]— Mujib Zalmay Video Original Video Link Mujib Zalmay Video Viral On Social Media X Trending Now
[-wATCH-]— Mujib Zalmay ʟᴇᴀᴋᴇᴅ Video ᴠɪʀᴀʟ On Social Media ˣ ᵀʷⁱᵗᵗᵉʳ
[-wATCH-]— Mujib Zalmay ʟᴇᴀᴋᴇᴅ Video ᴠɪʀᴀʟ On Social Media ˣ ᵀʷⁱᵗᵗᵉʳ
[-wATCH-]— Mujib Zalmay Video Original Video Link Mujib Zalmay Video Viral On Social Media X Trending Now
Mujib Zalmay Original Video video took the internet by storm and amazed viewers on various social media platforms. Mujib Zalmay, a young and talented digital creator, recently became famous thanks to this interesting video.
L𝚎aked Video Mujib Zalmay Original Video Viral Video L𝚎aked on X Twitter
Mujib Zalmay Original Video video oficial twitter
L𝚎aked Video Mujib Zalmay Original Video Viral Video L𝚎aked on X Twitter..

fghftght

cvc

cvbcvb


[MM-35761] Remove references to AWS provisioning and setting AWS server variables

Summary
This PR removes the deployment sidebar link and adds the developer mode setting requiremenet.

Ticket Link
https://mattermost.atlassian.net/browse/MM-35761

@aaronrothschild, @justinegeffen would one of you please review this?


IFolder.CheckExistsAsync - different behavior between Desktop and UWP

In UWP, the following code would return ExistenceCheckResult.NotFound.
In Desktop, LocalStorage is essentially ignored in the code below, and the result returned is ExistenceCheckResult.FileExists ...
var result = await FileSystem.Current.LocalStorage.CheckExistsAsync("e:\\dir1\\dir2\\file.xml");

The reason is due to the (documented) behavior of Path.Combine, which is used in the Desktop implementation of IFolder.CheckExistsAsync:
public static string Combine(string path1, string path2)

If path2 contains an absolute path, this method returns path2.

Can someone comment on whether this is intentional behavior or a bug in IFolder.CheckExistsAsync?

Same question here.

+1


Is this theme ready for 0.8.0? (Breaking changes)

See: https://dev.ghost.org/ghost-0-8-0/

Nope, not even for 0.7.0 I guess.


Improve error handling

Improve error handling and log meaningful messages to the Typewriter console output

Resolved in Typewriter 1.2


"Requested transaction contradicts existing jobs" messages on shutdown

Since uwsm 0.16.0 (and also with 0.16.1), I've noticed some odd trace in the systemd journal on shutdown. It happens with reboot and systemctl soft-reboot, but not with loginctl terminate-user ''.
Sun 2024-06-30 11:18:25 BST uwsm_sway[1434]: 00:01:46.471 [ERROR] [wlr] [libseat] [libseat/backend/logind.c:199] Could not close device: Remote peer disconnected
Sun 2024-06-30 11:18:25 BST uwsm_sway[1434]: 00:01:46.472 [ERROR] [wlr] [backend/session/session.c:356] Failed to close device 36: Resource temporarily unavailable
Sun 2024-06-30 11:18:25 BST systemd[1]: Requested transaction contradicts existing jobs: Transaction for systemd-logind.service/start is destructive (user.slice has 'stop' job queued, but 'start' is included in transaction).
Sun 2024-06-30 11:18:25 BST systemd[1]: Requested transaction contradicts existing jobs: Transaction for systemd-logind.service/start is destructive (user.slice has 'stop' job queued, but 'start' is included in transaction).
Sun 2024-06-30 11:18:25 BST uwsm_sway[1434]: 00:01:46.499 [ERROR] [wlr] [libseat] [libseat/backend/logind.c:199] Could not close device: Could not activate remote peer 'org.freedesktop.login1': activation request failed: a concurrent deactivation request is already in progress
Sun 2024-06-30 11:18:25 BST uwsm_sway[1434]: 00:01:46.499 [ERROR] [wlr] [backend/session/session.c:356] Failed to close device 37: Resource temporarily unavailable
Sun 2024-06-30 11:18:25 BST systemd[1]: Requested transaction contradicts existing jobs: Transaction for systemd-logind.service/start is destructive (veritysetup.target has 'stop' job queued, but 'start' is included in transaction).
[...several repetitions of similar trace...]
Sun 2024-06-30 11:18:25 BST uwsm_sway[1434]: 00:01:47.298 [ERROR] [wlr] [backend/session/session.c:356] Failed to close device 12: Resource temporarily unavailable
Sun 2024-06-30 11:18:25 BST systemd[1]: Requested transaction contradicts existing jobs: Transaction for systemd-logind.service/start is destructive (boot.mount has 'stop' job queued, but 'start' is included in transaction).
Sun 2024-06-30 11:18:25 BST uwsm_sway[1434]: 00:01:47.298 [ERROR] [wlr] [libseat] [libseat/backend/logind.c:365] Could not release control of session: Could not activate remote peer 'org.freedesktop.login1': activation request failed: a concurrent deactivation request is already in progress
Sun 2024-06-30 11:18:26 BST systemd[1350]: Stopped Main service for sway.
Sun 2024-06-30 11:18:26 BST systemd[1350]<EMAIL_ADDRESS>Triggering OnSuccess= dependencies.
Sun 2024-06-30 11:18:26 BST systemd[1350]: Stopped target Session services which should run early before the graphical session is brought up.
Sun 2024-06-30 11:18:26 BST systemd[1350]: Stopped target Preparation for session of sway Wayland compositor.
Sun 2024-06-30 11:18:26 BST systemd[1350]: Reached target Shutdown graphical session units.
Sun 2024-06-30 11:18:26 BST systemd[1350]: Stopping Environment preloader for sway...
Sun 2024-06-30 11:18:26 BST systemd[1350]: Stopped target Shutdown graphical session units.

It seems to be harmless, so maybe it's a non-issue, but I wanted to mention it. I'm using systemd 256.1, sway 1.9, Arch Linux.
It also feels like shutdown is a tiny bit slower than with uwsm 0.15.8. I'll experiment a bit more to see what is going on.

I have confirmed that this definitely doesn't happen with uwsm 0.15.8.

This is strange. User-level units should not affect system-level units via dependencies in any way.
I've found systemd-logind.service/start is destructive messages on shutdown all the way back to October, when neither wayland-session-shutdown.target nor<EMAIL_ADDRESS>did exist. This seems to be happening on my machine for a long time.
Found one such message from a user manager though, during a failed wayland session startup attempt. I will add replace-irreversibly to transactions invoking wayland-session-shutdown.target analogous to what systemd does to shutdown.target on system level. But this should just remove some errors from user manager in corner cases and would not affect system level.
If you have any ideas on why it might depend on uwsm versions on your system, but not on mine, it would be very helpful.

Thanks for looking into it!
I've also looked a bit more into it, first by increasing the systemd log verbosity, then by using busctl monitor to log all D-Bus messages during that time (via a little systemd service), but neither of these have really given me any more clues; I couldn't figure out why systemd is getting a job to start systemd-logind at that time.
Looking a bit more closely at the trace though, it seems that what must be going on is that sway wants to tell logind to close the session, and since logind is currently shutting down, this triggers some kind of socket activation where systemd wants to start up logind, hence it then complains that it is in the process of shutting down.
So, maybe one of the uwsm-managed services (e.g<EMAIL_ADDRESS>needs to have an After= on either systemd-logind.service, or maybe on the unit that contains the login service (which for me is currently session-43.scope)...?

Indeed, focusing on this part of the trace, which I see repeated numerously:
Sun 2024-06-30 11:18:25 BST uwsm_sway[1434]: 00:01:46.499 [ERROR] [wlr] [libseat] [libseat/backend/logind.c:199] Could not close device: Could not activate remote peer 'org.freedesktop.login1': activation request failed: a concurrent deactivation request is already in progress

The corresponding code is here in libseat, which gets called by wlroots here, and ultimately will be part of sway shutdown.

sway wants to tell logind to close the session

This is actually funny.

So, maybe one of the uwsm-managed services (e.g<EMAIL_ADDRESS>needs to have an After= on either systemd-logind.service, or maybe on the unit that contains the login service (which for me is currently session-43.scope)...?

User-level and system-level units can not depend on each other. The closest system unit is<EMAIL_ADDRESS>which is the template user's instances of systemd, which contain user-level units.
I've added After=systemd-logind.service to<EMAIL_ADDRESS>destructive transactions on soft-reboot seem to be gone.  But I'm not sure how good of a fix this is.

It seems to be a race condition. Since there is no ordering between<EMAIL_ADDRESS>and systemd-logind.service, slower shutdown of the former causes this situation. It is slower because uwsm it makes the compositor part of the ordered structure of graphical-session-pre, graphical-session, xdg-autostart. So compositor's shutdown is delayed until graphical autosart apps are brought down, then other graphical apps are brought down. Meanwhile, systemd-logind is stopped.
At least that is my hypothesis about your system. When I tried to test this on my system by launching a simple service
[Service]
Type=exec
ExecStart=sway -c /dev/null

with nothing else around and doing a soft-reboot, logind still left the picture before sway.

Good news, this is being resolved upstream.

Yeah, fair enough. Not something for uwsm to solve (and also a very minor issue to begin with). Feel free to close this again, I had just wanted to give an update.
Thanks for your help again, I learnt some good things here :)

and changing the final systemctl --user start --wait wayland-session-bindpid to run with SIGTERM being SIG_IGNored.

There is something wrong here, bindpid unit is not started with --wait, did you mean<EMAIL_ADDRESS>Stupid question: how do you run a process shielded from a signal?

and changing the final systemctl --user start --wait wayland-session-bindpid to run with SIGTERM being SIG_IGNored.

There is something wrong here, bindpid unit is not started with --wait, did you mean<EMAIL_ADDRESS>
Yes, you're right. I also did that experiment a bit hastily, not sure how much sense it made any more, I'll take a deeper look later when I have more time.

By doing signal.signal(signal.SIGTERM, signal.SIG_IGN) before the execlp().

Shielding a waiting systemctl is counterproductive. It needs to be TERMable in order for bindpid unit to work.
To hold session open, there should be another process that either ignores TERM and HUP or uses them as a trigger to check on units state, then exit only after they are deactivated.

I got it to work now by doing approximately what uwsm 0.15.8 did:
diff --git a/uwsm/main.py b/uwsm/main.py
index 8be8975..83024b9 100644
--- a/uwsm/main.py
+++ b/uwsm/main.py
@@ -3859,6 +3859,20 @@ def stop_wm():
     return True

+exiting = False
+
+def on_exit_signal(signum, frame):
+    global exiting
+    if exiting:
+        print_normal(f"Got signal {signum}: already exiting so ignoring")
+        return
+    exiting = True
+    print_normal(f"Got signal {signum}: stopping compositor then exiting")
+    stop_wm()
+    print_normal("Stopped compositor, exiting")
+    sys.exit()
+
+
 def main():
     "UWSM main entrypoint"

@@ -3982,17 +3996,23 @@ def main():
             )
             print_debug(sprc)

-            # replace ourselves with systemctl
+            signal.signal(signal.SIGTERM, on_exit_signal)
+            signal.signal(signal.SIGINT, on_exit_signal)
+            signal.signal(signal.SIGHUP, signal.SIG_IGN)
+
             # this will start the main compositor unit
             # and wait until compositor is stopped
-            os.execlp(
-                "systemctl",
-                "systemctl",
-                "--user",
-                "start",
-                "--wait",
-<EMAIL_ADDRESS>+            sprc = subprocess.run(
+                [
+                    "systemctl",
+                    "--user",
+                    "start",
+                    "--wait",
+<EMAIL_ADDRESS>+                ],
+                check=True,
             )
+            print_debug(sprc)

         except Exception as caught_exception:
             print_error(caught_exception)

This gets rid of the offending trace when sway exits. I guess now I could also remove<EMAIL_ADDRESS>entirely again, effectively reverting 6d6bdaf, but I haven't tried that. I also suspect you had a good reason for that commit – curious what it was?
Btw, for some reason my change introduces a shutdown delay of about 4 seconds in some random place:
Mon 2024-07-01 21:00:10 BST systemd[3480]: Stopping Display colour temperature adjustment...
Mon 2024-07-01 21:00:10 BST thermald[3441]: Terminating ...
Mon 2024-07-01 21:00:10 BST systemd[3480]: Stopping Idle manager for Wayland...
Mon 2024-07-01 21:00:10 BST systemd[1]: Stopping Thermal Daemon Service...
Mon 2024-07-01 21:00:10 BST systemd[1]: Stopping Daemon for power management...
[…]
Mon 2024-07-01 21:00:10 BST systemd[1]: Stopped Daemon for power management.
Mon 2024-07-01 21:00:11 BST thermald[3441]: terminating on user request ..
Mon 2024-07-01 21:00:12 BST systemd[1]: thermald.service: Deactivated successfully.
Mon 2024-07-01 21:00:12 BST systemd[1]: Stopped Thermal Daemon Service.
Mon 2024-07-01 21:00:14 BST systemd[3480]: Stopped Display colour temperature adjustment.

Though that is almost certainly a separate issue and maybe beside the point here.

This gets rid of the offending trace when sway exits. I guess now I could also remove<EMAIL_ADDRESS>entirely again, effectively reverting https://github.com/Vladimir-csp/uwsm/commit/6d6bdafe8b6da418226f048df62449269ec560c4, but I haven't tried that. I also suspect you had a good reason for that commit – curious what it was?

I did some experimenting with a couple of display managers, some of them strip session of resources rather abruptly. uwsm process crashed because it could not even print anything. Maybe it is their bug, but since uwsm relies heavily on systemd mechanisms anyway, I decided to get rid of running uwsm process entirely. It now sets the stage, pushes things into motion and steps aside. All the logic and killing is done by systemd (and waitpid).

I can confirm that 0.16.3 fixes this for me – thanks a lot!


python312Packages.wsme: drop nose dependency

Description of changes

#326513

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

This has no maintainer, isn’t used by anything in tree, has been disabled on Python ≥ 3.9 for years, and has been kept alive by mass bumps and Zero Hydra Failures for years…
Upstream doesn’t seem to be totally dead but I’m wondering if it’s worth continuing to devote more collective effort to this package that I get the impression nobody is using. It doesn’t have many reverse dependencies on PyPI.
Are you interested in adopting this package? If so, we ought to do a bit more clean‐up (e.g. it doesn’t depend on six any more at a glance, and there might be other outdated dependencies). Otherwise, maybe we should just drop this and if anyone else wants to use it they can easily bring it back and take up maintenance.

I can maintain it i think


Fix deprecation for  django.db.backends.util

Renamed django.db.backends.util to utils, ref:
sql.py:15: RemovedInDjango19Warning: The django.db.backends.util module has been renamed. Use django.db.backends.utils instead.

@dcramer please merge this change as this is preventing upgrade to Django 1.9

I dont use Django 1.9 or know anything about it. Additionally it's not clear to me that this wont break other versions of Django as it blindly changes an import statement.
Quite honestly, I'm tempted to deprecate this repo as it's no longer useful with the state of the world. Our primary reason for continuing its usage was for the threaded runserver, which is no longer only supported by devserver. If someone wants to take over maintainership they can take care of this patch.

This is not the right patch as it will break old Django versions.
@dcramer you should move this repository to the jazzband organization.

Fixed in 4c5864a


Update 'Key Differences' section of the About page

I've made some changes to the last section of this page and suggested updating it to be more general as this is where we talk about some of the advantages that make Frontity different from other React frameworks, not just Gatsby. Please feel free to review it.
I also removed a feature from the previous section that was no longer in the "Frontity Features" page (the link was not working).
I guess it will also be necessary to update the "How Frontity works" section when the new Architecture page is published.

Great work @rmartinezduque - I've merged your changes.


Extension schema for JSON Lines format support

How does this differ from #183? What is special about JSON Lines with respect to extensions?

JSON Lines does not contain information about extensions. A way to pass this information to the system is needed.

Aha, of course :)

I think --schema already handles this case.


Documentation: async, promise returning nature of migration and seed APIs

The async, promise returning nature of the migration and seed methods (e.g. knex.migrate.rollback(), knex.seed.run()) should be clearly documented. It seems to only be hinted at by a code example.

Looks like you are right. Pull requests, improving documentation are always welcome 👍


test-branch/execute-job/comment-from-wrong-user-on-pr/2b7f33eb-782f-4392-993e-802d44039d57

PR for testing 2b68ec5846a503890190549cea852e9be0ac439d

/canonical/self-hosted-runners/run-workflows 2b68ec5846a503890190549cea852e9be0ac439d


[CSV-148] Add testcases about withIgnoreSurroundingSpaces() and withTrim()

https://issues.apache.org/jira/projects/CSV/issues/CSV-148
Add test cases for CSV-148 tickets on Jira, distinguish between withIgnoreSurroundingSapces() and withTrim()

Coverage remained the same at 98.507% when pulling de441b83676172e194ea5856762062763897d17a on dota17:JiraCsv148 into 57c77341d42f54b71018931dd4f389d507a30a3c on apache:master.

https://issues.apache.org/jira/projects/CSV/issues/CSV-148
Add test cases for CSV-148 tickets on Jira, distinguish between withIgnoreSurroundingSapces() and withTrim()