prevent exceptions printout in TempDirectory

Deletion of objects in python does not guarantee any particular order, so what is attempted here simply does not work reliably. In this particular case, the shutil module is already unloaded when TempDirectory.del is called. Avoid the exception printed to stderr. There is no real way to fix this in a clean way.
I checked and we use del overloading in a handful of places in WMCore. I would suggest to open an issue about this since none of them likely do what we think they do.

Oh, and btw, this only shows up for the Tier0 since the WMAgent components that use this all seem to have WMCORE_CACHE_DIR set and don't use this code path. The Tier0Feeder doesn't or at least some of the code it calls doesn't.

Ok, still investigating some issues here...

Ok, this looks better. Ready for review.

On possibility for this type of thing is to turn it into a context manager which has a defined exit run at a defined time. In fact, I thought we had one already

Yeah, looked at that too. But it wasn't obvious to me how this would apply to the current design where you create the temp directory in really low level code, pass a reference back that is then used in high level code.
If someone has an idea on how to do that...

Yes, I would have to completely rewrite the design of this to use a context manager. Not going to do that but if someone wants to put in an issue feel free. General issue to review our use of del overloading would likely also be a good idea. Please merge.

@ticoann please merge


Fix typo in workflow name

Coverage decreased (-0.5%) to 99.464% when pulling 260b17c95429d39423b13d6a09519529acc7af92 on fix-typo into b6e52ec75baf3c2d77eaf724ef64f43776b6c55d on master.


If module fedaration and single spa are used together modules can not be loaded!

I m using single-spa and module fedaration(wp5) together but when I tried to import my modules with System , I m getting error like
In my root-config
<template id="single-spa-layout">       <single-spa-router>         <main>           <route path="/">             <application name="productImage/ProductImage"></application>           </route>           <route path="login">             <application name="angularJs/AngularJs"></application>           </route>                 </main>       </single-spa-router>     </template>
productImage/ProductImage and  angularJs/AngularJs are my  modules which is exposed by module fedaration.

Is it possible to use this add-on to routing mechanism or should I find another way?

I'm not sure if this will address the problem for you (you might consider including your code so someone can help you more directly) but for anyone else that has run into a similar problem and also ends up here: I was able to get single-spa-layout working with Webpack Module Federation Plugin.
root config
const remotes = {
  someWidget: import("someWidget/SomeWidget"),
  anotherWidget: import("anotherWidget/AnotherWidget")
};

const routes = constructRoutes(document.getElementById("single-spa-layout"));

const applications = constructApplications({
  routes,
  loadApp: ({ name }) => {
    return remotes[name];
  }
});

constructLayoutEngine({ routes, applications });
applications.forEach(registerApplication);

start();

webpack.config.js:
{
  ...other config...
  plugins: [
    new ModuleFederationPlugin({
      name: "root",
      remotes: {
        someWidget: "someWidget@http://localhost:8081/remoteEntry.js",
        anotherWidget: "anotherWidget@http://localhost:8082/remoteEntry.js"
      },
    },
    ...etc...
  ]
}

index.html:
<template id="single-spa-layout">
  <single-spa-router>
    <application name="someWidget"></application>
    <application name="anotherWidget"></application>
  </single-spa-router>
</template>

example of webpack.config.js in remote MFEs:
plugins: [
  new ModuleFederationPlugin({
    name: "someWidget",
    filename: "remoteEntry.js",
    exposes: {
      "./SomeWidget": "./src/entry.js"
    }
  })
]


Fix for ZeroDivisionError when verbose_eval equals to 0.

This pull request addresses an issue when verbose_eval parameter set to zero leads to ZeroDivisionError: integer division or modulo by zero.

Codecov Report

Merging #3115 into master will not change coverage.
The diff coverage is 100%.

@@            Coverage Diff            @@
##             master    #3115   +/-   ##
=========================================
  Coverage     43.79%   43.79%
  Complexity      228      228
=========================================
  Files           159      159
  Lines         12507    12507
  Branches        466      466
=========================================
  Hits           5478     5478
  Misses         6837     6837
  Partials        192      192

Impacted Files
Coverage Δ
Complexity Δ

python-package/xgboost/callback.py
87.8% <100%> (ø)
0 <0> (ø)
: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 375d753...d1072f6. Read the comment docs.


Husky, prepush lint

Prevents pushing code that outright fails linting. Good litmus before running CI tests.
Superset of #23.

Also runs tests locally now.


[Snyk] Upgrade eslint-plugin-promise from 6.2.0 to 6.5.1

Snyk has created this PR to upgrade eslint-plugin-promise from 6.2.0 to 6.5.1.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 4 versions ahead of your current version.

The recommended version was released on 22 days ago.

Release notes

Package name: eslint-plugin-promise

6.5.1 - 2024-07-206.5.1 (2024-07-20)
Bug Fixes

remove name from eslintrc config; fixes #489 (#490) (c011a1a)

6.5.0 - 2024-07-196.5.0 (2024-07-19)
Features

add name property to configs (for use with tooling) (#486) (ca9e9b4)

6.4.0 - 2024-07-016.4.0 (2024-07-01)
Features

add support for flat config (#479) (ecbce9f)

6.3.0 - 2024-07-016.3.0 (2024-07-01)
Features

add support for eslint v9 (#478) (4dfc8a7)

6.2.0 - 2024-05-276.2.0 (2024-05-27)
Bug Fixes

no-callback-in-promise: false positives when the exception is an argument (#446) (5e4546d)

Features

Resolve getAncestors and getScope calls in eslint v9 (#466) (c0c716b)

    from eslint-plugin-promise GitHub release notes

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:

🧐 View latest project report
📜 Customise PR templates
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

coverage: 98.802%. first build
when pulling 1d0d679eec520008942b1c90d812a653844d2ff4 on snyk-upgrade-242ba04f7e98d9956ec14a8f89d9e4bb
into 7e2d7d6e89384fb2c8709de2263598d8882f4711 on main.


Refact: install script formated messages

Refactoring the installation script formated messages.

Fix inconsistent quotes in typography
Replace echo command with printf
Replace ANSI  /x1B (Hexadecimal) with \033 (Octal) codes.
Improve title message format

Resolves #475
Closes #477
Closes #478
Resolves #480
If there's anything more to do that's related to the formated messages please let me known.

OK.


St. Petersburg, FL

To<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Dear St. Petersburg City Coucil,
My name is [NAME] and I am a citizen of St. Petersburg, FL. I am writing to you in regards to the national outcry over the murder of George Floyd, and concern over the conduct of the St. Petersburg Police Department.This past week, our nation has been gripped by protests calling for a rapid and meaningful reconsideration of the role of policing in communities as well as an end to racism and anti-Blackness in America. Our city has been at the forefront of much of this action. Accordingly, it has come to my attention that the budget for 2021 is being decided as these protests continue.
I demand that the City Council begin meaningfully defunding the St. Petersburg Police Department and re-allocate those funds to programs proven to more effectively promote a safe and equitable community: community-based mental health services, substance abuse treatment services, affordable housing programs, and more. I demand a budget that reflects the actual needs of St. Petersburg residents.
History has shown that police "reform" is not enough. We must take a hard look at the ways that the current system in place fails to serve--and in fact actively harms--our community, and come together to reimagine the role of police in our city. We can create a new city, safe and beautiful for all people of St. Petersburg who call this place their home. Through listening and learning from the communicated demands, needs, and desires of the Black community, we can build a sustainable and safe city for today and tomorrow.
Thank you for your time,
[YOUR NAME]
[YOUR ADDRESS]
[YOUR EMAIL]
[YOUR PHONE NUMBER]

Thanks @ritazwiee. I was just about to start putting this together myself -- you saved me time. I will submit a pull request so it can get added to the website.

@dionlarson Please review the teams before submitting PR

Changelog

Coucil -> Council
citizen -> resident
added missing space
sentence structure of last paragraph

Questions
Is there too much "we" at the end?
Current
To<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Dear St. Petersburg City Council,
My name is [NAME] and I am a resident of St. Petersburg, FL. I am writing to you in regards to the national outcry over the murder of George Floyd, and concern over the conduct of the St. Petersburg Police Department. This past week, our nation has been gripped by protests calling for a rapid and meaningful reconsideration of the role of policing in communities as well as an end to racism and anti-Blackness in America. Our city has been at the forefront of much of this action. Accordingly, it has come to my attention that the budget for 2021 is being decided as these protests continue.
I demand that the City Council begin meaningfully defunding the St. Petersburg Police Department and re-allocate those funds to programs proven to more effectively promote a safe and equitable community: community-based mental health services, substance abuse treatment services, affordable housing programs, and more. I demand a budget that reflects the actual needs of St. Petersburg residents.
History has shown that police "reform" is not enough. We must take a hard look at the ways that the current system in place fails to serve, and in fact actively harms, our community. We must come together to reimagine the role of police in our city. We can create a new city, safe and beautiful for all people of St. Petersburg who call this place their home. Through listening and learning from the communicated demands, needs, and desires of the Black community, we can build a sustainable and safe city for today and tomorrow.
Thank you for your time,
[YOUR NAME]
[YOUR ADDRESS]
[YOUR EMAIL]
[YOUR PHONE NUMBER]

Possible edit:
"History has shown that police "reform" is not enough. We must take a hard look at the ways that the current system in place fails to serve, and in fact actively harms, our community. We must come together to reimagine the role of police in our city. I believe that a new city, safe and beautiful, can be created for all people of St. Petersburg who call this place their home. Through listening and learning from the communicated demands, needs, and desires of the Black community, there can be a sustainable and safe city for today and tomorrow."

Thank you for your help @ritazwiee. I've added your suggestions to my above message.
It looks like the review process is on pause until after a meeting tonight at 9pm to better coordinate the full review process. I'll pick this up again once approved!

Proposed 2020-2021 budget that is submitted for approval on July 10th 2020. I have added some on the following in between paragraphs one and two.

$116,368,000 to police, 38.85% of $299,509,850 total budget
"In the FY20 Adopted Budget, we started a three-year plan to increase the department’s sworn personnel from 562 to 600 by including funding for 13 full-time Police Officers. For the second year of the plan, the city has applied for a COPS grant to receive partial funding for the remaining 25 full-time Police Officers needed to complete the plan. "

@dionlarson I think this is fine/ready for PR


PHOENIX-7229 Addendum to handle cases where a child view extends the

parent's PK but we do a point lookup using the parent's PK.

I will also need to handle aggregate queries.


Updated API documentation

Context
We recently had problems with large API presonses. We want to make sure that all API consumers are aware of our pagination capabilities
Changes proposed in this pull request
Update the API documentation with an explainer about how and why pagination works
Guidance to review

Check wording

Things to check

[ ] If the code removes any existing feature flags, a data migration has also been added to delete the entry from the database
[ ] This code does not rely on migrations in the same Pull Request
[ ] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[ ] If this code adds a column to the DB, decide whether it needs to be in analytics yml file or analytics blocklist
[ ] API release notes have been updated if necessary
[ ] If it adds a significant user-facing change, is it documented in the CHANGELOG?
[ ] Required environment variables have been updated added to the Azure KeyVault
[ ] Inform data insights team due to database changes
[ ] Make sure all information from the Trello card is in here
[ ] Rebased main
[ ] Cleaned commit history
[ ] Tested by running locally
[ ] Add PR link to Trello card

 Update API documentation to describe best practices for calling GET/Applications


Fix connectivity issue to NodePorts and router hostPort

The change to local gateway mode in ovnk needs the activation of ip forwarding for NodePorts and hostPorts to work.
It was only enabled conditionally when br-ex1 additional bridge was configured on the system.
Fixes-Issue: https://issues.redhat.com/browse/USHIFT-502 (cherry picked from commit 3f5bf8e9fab150e1c82702386663919c7e07d80a)

Which issue(s) this PR addresses:

Closes #

/lgtm


Can't access website

I can't access the website at configured url.
I already have a wireguard running well, installed docker, copied the docker-compose, configured values, started it via dockerp-compose up -d, logs stop at synchronizing ldap, which all seems good. But I can't access the web service.

have you tried http://HOST-IP:8123 ? by default, the web interface is running on port 8123, not port 80


Gimbal Rotation During Zoom In/Out Operation with SIYI ZT6 Optical Pod Camera

I am currently conducting R&D from this repository (Innopolis-UAV-Team / siyi-python-sdk)  , specifically with the SIYI ZT6 Optical Pod Camera.
Using this library as a base, I implemented zoom-in and zoom-out operations in Python. While we were able to successfully control the zoom levels in the camera feed, I observed an issue where the gimbal starts to rotate during these zoom operations.
I would like to avoid this unintended gimbal axis rotation. Is there a specific approach or configuration should be followed to address this issue?
Any guidance or suggestions would be greatly appreciated.

Hi! I can't say exactly what the problem is, since we haven't tested the code on this camera model. Could you send a video of it working and the code you're using?


validate-token-with-graph-api fails because the audience id of our web api is not the audience id auf Graph API

https://github.com/playerony/validate-azure-ad-token/blob/ec7dac593e9869089b3931ee2edda80ab8763662/src/index.ts#L87
This line produces "Access token validation failure. Invalid audience."
For me, this is clear. The audience id in our jwt tokens is the application id of our custom web api.
Graph API declines jwt tokens that are not issued for Graph API.
Why at all validate against graph api?

Also you don't check if "exp" claim is expired...

The reason why the validateTokenWithGraphApi function is validating the Azure AD token against the Microsoft Graph API is because the Graph API exposes endpoints that can be used to validate tokens issued by Azure AD. Specifically, you can use the /me endpoint to validate a token and retrieve information about the user associated with the token.

Man you are totally right, I wrote this package almost 2 years ago, so I do not remember specific reasons why I did it this way. But now you have your suggestion in version 2.1.0

Thank you very much! One last thing: As stated in the documentation of the parameter applicationId, only in v1.0 tokens the appid is present.
What if we want to validate v2 tokens?
It is not possible to leave applicationId null or empty.
https://github.com/playerony/validate-azure-ad-token/blob/main/src/index.ts#L64
https://github.com/playerony/validate-azure-ad-token/blob/main/src/core/validate-token-claims.ts#L22
The problem is that we want to secure our custom node.js Web API against several clients, e.g. SharePoint Online SPFx Solution and another custom Web App.
So we have at least two applications that are consuming our web API. Thus, we would have to decide which one should get access.
Either applicationId should be an applicationIds Array or implement v2 token.
That would be awesome.

@playerony You obviously used the Graph API to validate the signature of the token. Of course this was not the correct approach, but at least the signature was checked. Now the check is gone and one can easily create tokens manually that get accepted by this library.


Add support for external feed and scheduled triggers

Octopus version
2024.2.5350
Octoterra version
latest
What happened
The TF provider supports the other project trigger types now!
https://registry.terraform.io/providers/OctopusDeployLabs/octopusdeploy/latest/docs/resources/project_scheduled_trigger
Octoterra log output
N/a
Reproduction steps
N/a
More Information
No response

Schedule trigger support was added with https://github.com/OctopusSolutionsEngineering/OctopusTerraformExport/pull/21

Feed trigger supported added with https://github.com/OctopusSolutionsEngineering/OctopusTerraformExport/pull/22


Sum nodes and rels over all documents

https://github.com/neo4j-labs/llm-graph-builder/blob/main/backend/src/main.py#L42-L43
Also add relationship count to the UI

Done


Expose bearing accuracy

Following up on https://github.com/mapbox/mapbox-gl-native/issues/12137, we should expose bearing's accuracy data.

Does this refer to exposing the data from Location#getBearingAccuracyDegrees()?
Or does this refer to updating the render mode to include an option to show the bearing as depicted in the gmaps screenshot in mapbox/mapbox-gl-native#12137?

It would be really cool if both options are included in the mapbox sdk :)

And what I understood from the initial issue is that you were looking for the compass-bearing accuracy, is that correct @NataliaSamoilovich?

Yes. You are right. I was looking for compass bearing to implement behavior like in google maps.


[harpoon2] Request: Option to hide the 'saving { "example.txt" }' text being printed upon closing quick menu

Setup:

Harpoon2 on commit: 4ad05be

Issue:

After updating to the most recent version of harpoon 2, every time the quick menu is closed a message is printed to the command line saying 'saved { "example.txt" }'. It appears to print out the current contents of the harpoon table.

Cause:

Opening the harpoon quick menu - harpoon.ui:toggle_quick_menu(harpoon:list())

It appears to be coming from this newly added line in lua/harpoon/ui.lua, at line 196
print("saving", vim.inspect(list))

I like the idea of being able to enable this as an option (probably for debugging), but I don't think it should be the default behavior.

changed and accidentally let that in :)


Migrate Objective-C code to Swift

This PR takes all Objective-C code in the ios folder and replaces it with a Swift analogue. It also adds some slight tweaks that were keeping the jsbundle from compiling that were addressed in #7 but never merged.

I don't want to deviate from the output of the actual instructions listed in the README. If the React Native initialization script migrates to a Swift base project, we can baseline that output. Thanks for the PR though @aaronsky.


How to output Markdown content as raw HTML?

I have problems outputting the content of a collection's Markdown as pure HTML.
When I use {{ $collection->getContent() }} I get "
My content
" which literally outputs the HTML tags. When I use the blade raw directive: {!! $collection->getContent() !!}, I get the right result, but it this the best practice?
I use the content block in a partial and cannot use the @yield() command in that scenario I guess.

The typical setup would be to extend a parent template from each of your collection items, and render the content using @yield(); if you're doing something else that requires you to access $collection directly, then yes, {!! $collection->getContent() !!} would be the best way to output the processed Markdown content.
If you have a repo of your project I can take a look at, I can give you more specific advice...there may be a better way to structure your collection files.

Thanks for the feedback. We are using markdowns in a collection to add updates to youth camps that we run. This is the repo: https://github.com/CodeDesignInitiative/cdweb1801
Each camp is an own markdown from the camp collection which extends an layout. In this layout, we include an update.
The update pulls an entry from the update collection and compares the camp ids (e.g. ber1802) and shows the corresponding update.
I guess that in this setting, I cannot extend on an layout and need to to refer to {!! $collection->getContent() !!}.

Yes, if you're referencing an item directly, like you are with each update, you need to access its content using getContent.
But I've just added support for the ability to drop the ->getContent() part in a Blade view, in this PR. Once that's merged, you'll be able to simply call {!! $update !!} in your Blade template.


Feature/#12recommended users

Reccomended Users for current user Screen. Client side data fetching for now? maybe we change that to Server side fetching or with API route?

Deploy is failing. Because there are some TS errors.

I was trying to that one clientside rendering but somehow not manage to do that. no luck! 😿
Propose now a working client-side rendering and we can do and test a preloader component (missing anyway) or do it SSR later.


Fix parameter expansion in CI versioning

Overview
The CI yaml format uses ${{ ... }} for parameter expansion, which is apparently different from how you can specify things like $(Build.SourcesDirectory) on the line above.
This was causing

Formatted command: . 'f:\aipmragent_work\9\s\scripts\packaging\versionmetadata.ps1' -Directory: 'f:\aipmragent_work\9\s' -Version: '$(parameters.MRTKVersion)'

when CI runs.

/azp run


[release/2.2] Fix SDK version: include prerelease

Building this locally now, but it seems like a simple fix.

@dotnet-bot test OSX10.12 Release
Failed without detail during a smoke-test restore. Assuming flaky machine.

@dotnet-bot test CentOS7.1 Tarball Release
Heh, looks like since the portable leg's name contains the non-portable leg's name, the last comment's trigger retried both and that time the CLI build failed to find crossgen. I think it's safe to assume this is flakiness because the leg passed last time. Running it again to get a chance at green.


🐛 Bug Report: Error when widget binding is not initialized

👟 Reproduction steps

Create a flutter project
Create appwrite project and add flutter platform
Install appwrite package to flutter project
Initialize client as explained on example or readme

👍 Expected behavior
Connect to appwrite server
👎 Actual Behavior
Throwing exception
Null check operator used on a null value
🎲 Appwrite version
Different version (specify in environment)
💻 Operating system
Windows
🧱 Your Environment
Appwrite version: 3.0.1
👀 Have you spent some time to check if this issue has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

FYI, I've fixed the issue with this line before client initialization:
  WidgetsFlutterBinding.ensureInitialized();
If you want I can make a PR to update the docs

When you use async and await features in Flutter applications main function, you have to provide WidgetsFlutterbinding.ensureInitialized().


Update documentation to use virtualenv

README file now suggests using virtualenv to create isolated python environment to manage installed dependencies separately from system-level packages.

Thanks for the instructions.  I have some questions

Running virtualenv . mixes virtualenv tools with opensfm tools. Isn't it better to use a different folder?
Should we skip the line installing scipy and add it to requirements.txt instead?

@paulinus Sure, that makes sense.

Updated README to use env as an environment name, added env directory to .gitignore file.

I was wondering why scipy isn't there already, that's why I added it as a separate installation step. But if it was not done intentionally, it's obviously better to keep all dependencies in a single file. There's another question here: all dependencies are mentioned w/o version specification, which leads to a huge potential problem with reproducibility of the build. I attached requirements.txt file with concrete versions to avoid any problems in future.

@paulinus Removed version specification for numpy and scipy, due to problems with Linux builds. It works perfectly fine with numpy==1.12.1 but fails on TravisCI.

@kachayev I don't know what is the best way to handle versions in requirements.txt.
Using specific versions has the advantage of reproducibility, but it is a bit misleading. OpenSfM does not actually require those specific versions.
For example, I'm using different versions than the ones you added. And you seem to have a different version of numpy than the ones available in the Travis build.
I don't know what is the best practice for this problem. Do you have a reference?

https://packaging.python.org/requirements/#requirements-files
Minimal requirements go to setup.py while version pinned go to a requirements file.

Exactly what I was looking for. Thanks @zwn !
I'll then merge this PR and we can add pinning for numpy later.


Consider time-dependent processes

Considering both tsm and nsm, look through variables and processes that are time-dependent and consider how the (user-defined) time dim will be used to make sure the timestep is correctly handled.
As part of this issue, we should rename the timestep variable that's used in processes to dt to avoid confusion with the timestep incrementor.

Example processes that depend on time:
For TSM - these seem to assume a timestep of 1 second: https://github.com/EcohydrologyTeam/ClearWater-modules/blob/main/src/clearwater_modules/tsm/processes.py

dTdt_sediment_c
dTdt_water_c

For NSM, time dependent processes have a timestep variable, which is, by default, 1.
https://github.com/EcohydrologyTeam/ClearWater-modules/blob/main/src/clearwater_modules/nsm1/processes.py
Examples include:

Ap
Ab

@imscw95 @kewalak @aufdenkampe, I'm going to work on resolving this dt issue. @imscw95, I know you dug into this a little bit in the HEC-RAS "module tester," so it'd be great to get your insight.
NSM: Looking at the state variables and time-dependent processes (e.g., Ap) in NSM, it looks like it's currently set with units equal to 1 day and passing all tests.
TSM:

q_sediment converts units for thermal diffusivity from m2/day to m2/second.
dTdt_water_c -- working out these units I end up with Kelvin/second (?), so that further confirms the timestep is in units of seconds.

As an aside, should cp_water be in J/kg/C instead of J/kg/K? I'm a little confused on the units here

Seems like we'll want to align dt across modules to default to the same thing (either 1 day or 1 second). Any preferences? Should we skip aligning the modules for now and just focus on getting dt as an input for TSM and not worry about aligning modules?
Thanks!

Thanks! Sounds good, I think I will work on addressing this in TSM. Since TSM is passing all tests against calculated values -- can you confirm that the comparison was done at 1 second output? We can keep the values the same even with this change, and just modify the dt value. But I want to make sure that I understand what we're comparing it against before making that change!

Yes, confirmed that the TSM tests were for dt = 1 second.


Update sbt-scalafix to 0.9.27

Updates ch.epfl.scala:sbt-scalafix from 0.9.21 to 0.9.27.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "ch.epfl.scala", artifactId = "sbt-scalafix" } ]

labels: sbt-plugin-update, semver-patch

Superseded by #480.


Tentative fix for Vulkan missing offsets when binding buffers

This fixes the problem where buffers are not properly bound when using Vulkan. To be clear, I'm not suggesting this is a proper fix, but hopefully it will give @bkaradzic or somebody else enough information to create a better one.
I'm assuming adding a new field to Stream should be not necessary, but somewhere in the chain of commands an offset is being applied to dynamic & transient buffers that doesn't include the relative offset one can apply when doing bgfx::setVertexBuffer(<>, <>, OFFSET, <>), hence the need for fixing.

Can you rather provide repro case for issue you have? For example modify existing example with minimum number of changes to reproduce this issue.


Moving questions: show spinner

When moving questions within a questionnaire, it is not obvious the changes have been saved. Introduce a spinner on the question moving grip to show that the question reordering is being saved.
Currently the question moving graphql request utilises optimistic mutations (https://www.apollographql.com/docs/react/features/optimistic-ui.html), this stops the question jumping around as the question is saved to the backend and the questionnaire updating. This, however, may introduce complexity with getting a spinner setup.

 General refinement

This is harder than it seems. Will have to come back to it in a later release.


Fix for 'Inappropriate ioctl for device'

Fix for 'Inappropriate ioctl for device' error on posix systems.

Looks good!


Fill border can't be changed / styled

After the change 695cb78 fills are constanty showing a black border around them. Even if the LineStyle is changed, nothing changes.
It seems like the LineStyle should be forwareded to Fills internal Polygon instead of tjust keeping it and not using it.
What do you think? Currently I see no way to change the border around the fill since the corresponding polygon is private.

Thanks for reporting this! It seems related to #3720 and will be fixed shortly 👍

resolved in #3726

Thank you @swharden , love to see that. Works like a charm.


chore: add ability to emit metafiles

Metafiles can be used to understand our generated bundles
https://esbuild.github.io/analyze/

@millsp CI is failing with
/home/runner/work/prisma/prisma/packages/debug/dist/index.js:1
import {
^^^^^^

SyntaxError: Cannot use import statement outside a module
    at Object.compileFunction (node:vm:360:18)
    at wrapSafe (node:internal/modules/cjs/loader:1126:15)
    at Module._compile (node:internal/modules/cjs/loader:1162:27)
    at Module._compile (/home/runner/work/prisma/prisma/node_modules/.pnpm/pirates@4.0.5/node_modules/pirates/lib/index.js:136:24)
    at Module._extensions..js (node:internal/modules/cjs/loader:1252:10)
    at Object.newLoader [as .js] (/home/runner/work/prisma/prisma/node_modules/.pnpm/pirates@4.0.5/node_modules/pirates/lib/index.js:141:7)
    at Module.load (node:internal/modules/cjs/loader:1076:32)
    at Function.Module._load (node:internal/modules/cjs/loader:911:12)
    at Module.require (node:internal/modules/cjs/loader:1100:19)
    at require (node:internal/modules/cjs/helpers:119:18)
 ELIFECYCLE  Command failed with exit code 1.

Thanks for the ping. This is super weird and unexpected...


feat(ingest/airflow): update docs about JobTypeJobFacet error

Checklist

[ ] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[ ] Links to related issues (if applicable)
[ ] Tests for the changes have been added/updated (if applicable)
[ ] Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
[ ] For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

Closing in favor of https://github.com/datahub-project/datahub/pull/10106


[incubator/druid] fails with containerd 1.12.3 -- mkdir: can't create directory 'var/[tmp,druid]': Permission denied

Describe the bug
When using "containerd" as container runtime, Zookeeper and Postgresql are running fine, however... all the druid pods Error.
Version of Helm and Kubernetes:
helm version: 3.2.3
Kubernetes version: 1.18.6
minikube version: v1.12.1
Which chart:
incubator/druid
What happened:
$ kubectl get pods -n druid
NAME                                                 READY    STATUS                RESTARTS   AGE
druid-broker-6574f9468b-f4bl8                  0/1     CrashLoopBackOff     10         30m
druid-coordinator-7b6cb89658-lmq2q       0/1     CrashLoopBackOff   10         30m
druid-historical-0                                        0/1     CrashLoopBackOff   10         30m
druid-middle-manager-0                            0/1     CrashLoopBackOff   10         30m
druid-postgresql-0                                     1/1     Running            0          30m
druid-router-b7b54874-kr8tt                      0/1     CrashLoopBackOff   10         30m
druid-zookeeper-0                                     1/1     Running            1          30m
druid-zookeeper-1                                     1/1     Running            0          29m
druid-zookeeper-2                                     1/1     Running            0          28m
All druid pods Error then enter in a CrashLoopBackOff status with the following log messages:
$ kubectl logs -n druid druid-historical-0
[...]
mkdir: can't create directory 'var/tmp': Permission denied
mkdir: can't create directory 'var/druid/': Permission denied
[...]
This seems to be related to the fact that the securityContext is not set in the druid chart, as it is set in the postgresql, mysql and zookeeper ones.
What you expected to happen:
I am expecting to have the druid pods with status "Running"
How to reproduce it:
Minikube is installed on Fedora 32 with kernel 5.7.10-201.fc32.x86_64 from the  package:
rpm -i docker-machine-driver-kvm2-1.12.1-0.x86_64.rpm minikube-latest.x86_64.rpm
minikube start --cpus=6 --memory=24g --container-runtime='containerd' --kubernetes-version='1.18.6'
kubectl create ns  druid
helm install druid incubator/druid -n druid
Anything else we need to know:
using "docker" or "crio" instead of "containerd" leads to Druid running with no error.
Thank you!

It seem the securityContext issue. the fix PR https://github.com/helm/charts/pull/23069 blocked.


Automated Package Upgrade

:tada: This PR is included in version 2.2.408 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Kafka offset lags

Hi
I know that you mentioned that, but since i'm not using this project with Prometheus backend but InfluxDB & Telegraf for metrics collector, i'm not able to run the function that calculate the lag.
Cab you pls try and calculate the lag according the function you mentioned in the README file during the http query?
Thanks
D.

@panda87 working on this. Should be included in the next release.

Great @braedon! Thanks

@panda87 this has been released in 0.3.0

That's amazing! thanks!


Kyuubi 2715 hikaricp v2 recovery

Why are the changes needed?

How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[ ] Run test locally before make a pull request

Codecov Report

Merging #2785 (7b83dbc) into master (8578bcd) will increase coverage by 0.49%.
The diff coverage is 82.24%.

@@             Coverage Diff              @@
##             master    #2785      +/-   ##
============================================
+ Coverage     62.15%   62.65%   +0.49%
+ Complexity      424      201     -223
============================================
  Files           464      472       +8
  Lines         20827    21392     +565
  Branches       2814     2845      +31
============================================
+ Hits          12945    13403     +458
- Misses         6619     6706      +87
- Partials       1263     1283      +20

Impacted Files
Coverage Δ

...cala/org/apache/kyuubi/session/KyuubiSession.scala
100.00% <ø> (ø)

...g/apache/kyuubi/session/KyuubiSessionManager.scala
61.19% <30.00%> (-33.87%)
:arrow_down:

...i/server/statestore/jdbc/JdbcDatabaseDialect.scala
40.00% <40.00%> (ø)

.../kyuubi/server/statestore/StateStoreProvider.scala
50.00% <50.00%> (ø)

...main/scala/org/apache/kyuubi/util/ClassUtils.scala
71.42% <66.66%> (-16.08%)
:arrow_down:

...g/apache/kyuubi/operation/BatchJobSubmission.scala
81.30% <72.97%> (-8.44%)
:arrow_down:

...e/kyuubi/server/statestore/SessionStateStore.scala
83.95% <83.95%> (ø)

...kyuubi/server/statestore/jdbc/JDBCStateStore.scala
88.53% <88.53%> (ø)

.../apache/kyuubi/server/api/v1/BatchesResource.scala
86.95% <90.90%> (ø)

...apache/kyuubi/session/KyuubiBatchSessionImpl.scala
92.00% <94.11%> (+1.09%)
:arrow_up:

... and 26 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 8578bcd...7b83dbc. Read the comment docs.


spires parser should special-case journals

Originally on 2011-08-30
journal shouldn't display the behavior that most other things do, where they say that:
find t foo bar baz
means:
title:foo and title:bar and title:baz
it's not obvious, because of search_unit parsing of comma splits and so forth, but I think what we want is more like:
find j nucl phys b
becomes:
journal:"nucl phys b"

Please re-open in Invenio Query Parser if it is still relevant.


Extend solver interface

solveAll() could use an argument where the user can pass those variables that need to differ in each solution, e.g., to only search for models that differ on the user variables.
Add a propagate() method that finds the intersection of all solutions (again, on some given set of variables). Exact can then specialize this method to get a stateful propagate() routine.

solvers can have more functions, or with additional arguments, then the generic SolverInterface. So I'm happy for you to add this to Exact directly.
I don't think we'll want to cover this in all solvers...


Create style guide basics

Goal
The goal of this PR is to create a package we can release to setup the basics of our style guide.
Closes #1
Discussion
Currently the components are loading into /src in the package, so you have to reference that path when importing a component. Is there a way to avoid that?
Checklist

[x] Created Buttons
[x] Created Typography
[x] Created tailwind.config
[x] Created tailwind classes and initial styles

How To Test the Changes

Create package
Install package
Use package

Great work!
Am I right in assuming that a consumer of the package would not be interested in the Storybook UI (that's more for exploring the styles in this repo)? And that the consumer is really mostly interested in the base component/color/font definitions? If that's true, it might be best to separate that out into separate packages, so that the published package only contains the core components and not the Storybook UI. Take a look at the SIWF repository; that's also a Vite/Svelte project with separate packages; some of which are published & some that aren't.
If that's the route we think we want to go, I'd be okay with merging this & make that the subject of a subsequent issue/PR.

Great work!
Am I right in assuming that a consumer of the package would not be interested in the Storybook UI (that's more for exploring the styles in this repo)? And that the consumer is really mostly interested in the base component/color/font definitions? If that's true, it might be best to separate that out into separate packages, so that the published package only contains the core components and not the Storybook UI. Take a look at the SIWF repository; that's also a Vite/Svelte project with separate packages; some of which are published & some that aren't.
If that's the route we think we want to go, I'd be okay with merging this & make that the subject of a subsequent issue/PR.

Linked to #13


bump webassemblyjs

What kind of change does this PR introduce?

Did you add tests for your changes?

Does this PR introduce a breaking change?

What needs to be documented once your changes are merged?

Relevant changes: new utf8 encoder/decoder and couple of fixes.

Thanks


--timestamp does not work, Can't write signature, not enough space

Using open-pdf-sign with Let's Encrypt certificates works fine, but I like to timestamp those (as the LE-Certs are only valid for 90 days, so timestamping would be needed to keep the documents valid signed for a longer period).
So I tried this with several TSA-Servers like
http://timestamp.digicert.com/
http://timestamp.comodoca.com/
http://tsa.swisssign.net/
but without luck:
Exception in thread "main" eu.europa.esig.dss.model.DSSException: Unable to save a document. Reason : Can't write signature, not enough space; adjust it with SignatureOptions.setPreferredSignatureSize
>java -jar open-pdf-sign.jar --input test.pdf --output test-signed.pdf --certificate fullchain41.pem --key privkey41.pem --page -1 --timestamp --tsa http://timestamp.digicert.com --timezone Europe/Vienna
Exception in thread "main" eu.europa.esig.dss.model.DSSException: Unable to save a document. Reason : Can't write signature, not enough space; adjust it with SignatureOptions.setPreferredSignatureSize
        at eu.europa.esig.dss.pdf.pdfbox.PdfBoxSignatureService.saveDocumentIncrementally(PdfBoxSignatureService.java:422)
        at eu.europa.esig.dss.pdf.pdfbox.PdfBoxSignatureService.checkEncryptedAndSaveIncrementally(PdfBoxSignatureService.java:406)
        at eu.europa.esig.dss.pdf.pdfbox.PdfBoxSignatureService.signDocumentAndReturnDigest(PdfBoxSignatureService.java:230)
        at eu.europa.esig.dss.pdf.pdfbox.PdfBoxSignatureService.signDocument(PdfBoxSignatureService.java:166)
        at eu.europa.esig.dss.pdf.AbstractPDFSignatureService.sign(AbstractPDFSignatureService.java:294)
        at eu.europa.esig.dss.pades.signature.PAdESService.signDocument(PAdESService.java:218)
        at org.openpdfsign.Signer.signPdf(Signer.java:163)
        at org.openpdfsign.CLIApplication.main(CLIApplication.java:126)
Caused by: java.io.IOException: Can't write signature, not enough space; adjust it with SignatureOptions.setPreferredSignatureSize
        at org.apache.pdfbox.pdfwriter.COSWriter.writeExternalSignature(COSWriter.java:845)
        at org.apache.pdfbox.pdfwriter.COSWriter.doWriteSignature(COSWriter.java:793)
        at org.apache.pdfbox.pdfwriter.COSWriter.visitFromDocument(COSWriter.java:1199)
        at org.apache.pdfbox.cos.COSDocument.accept(COSDocument.java:452)
        at org.apache.pdfbox.pdfwriter.COSWriter.write(COSWriter.java:1435)
        at org.apache.pdfbox.pdmodel.PDDocument.saveIncremental(PDDocument.java:1414)
        at eu.europa.esig.dss.pdf.pdfbox.PdfBoxSignatureService.saveDocumentIncrementally(PdfBoxSignatureService.java:420)
        ... 7 more

I used wireshark to check if the retrieval of the Timestamp from the TSA-Server works: YES, request + response looks valid in wireshark.

Hm, that's strange! Thanks for reporting!
The same certificate without the timestamp works? Would it work by using the "cert41.pem" instead of the "fullchain41" as the certificate?

Thanks for the help!
I tried to increase the signature size in case of a timestamp being part of the signature.
Could you please try if this now works for you in case of timestamp + fullchain?
You can use the latest build from the dev branch: https://github.com/open-pdf-sign/open-pdf-sign/suites/10440287526/artifacts/516470579

Thanks, great! Can confirm this works now:
java -jar openpdfsign-0.1.2-dev.1+be37109c-jar-with-dependencies.jar --input test.pdf --output test-signed.pdf --certificate fullchain41.pem --key privkey41.pem --page -1 --timestamp --tsa http://timestamp.digicert.com --timezone Europe/Vienna


#5799 Python plots won't work with pandas Series

It also covers issue #5822
https://github.com/twosigma/beakerx/issues/5822

great thx please add an example of plotting a Series to pythonChartingAPI.ipynb

Example Added

thanks but it looks like the changes from my commit were lost: https://github.com/twosigma/beakerx/pull/5885/commits/d888e4dc37112e32b5ba5f137b080c4db85909bf
please check your merge...


fix: Update form_helper.php

Description
This PR enhances the form_open function in the following ways:

Function Existence Check:

Added a check to ensure the function form_open is defined only once to avoid redeclaration errors.

Documentation:

Included a detailed docblock to describe the function, its parameters, and their types for better understanding and maintenance.

Attribute Consistency:

Updated the method attribute to uppercase "POST" and adjusted the check for the 'GET' method to uppercase "GET" to maintain consistency.

CSRF and Charset Handling:

Ensured that CSRF and accept-charset attributes are handled consistently.

The updates to the form_open function make it work better under different conditions and make the code easier to read.

Description
Explain what you have changed, and why.
Checklist:

[x] Securely signed commits
[ ] Component(s) with PHPDoc blocks, only if necessary or adds value
[ ] Unit testing, with >80% coverage
[ ] User guide updated
[ ] Conforms to style guide

Thank you for sending this PR!
We expect the following in all Pull Requests (PRs).

PRs must be sent to the appropriate branch
All git commits must be GPG-signed
Must follow our style guide
Be commented in the PHP source file
Be documented in the user guide
Be unit tested
Pass all checks in GitHub Actions

[!IMPORTANT]
We expect all code changes or bug-fixes to be accompanied by one or more tests added to our test suite to prove the code works.

If pull requests do not comply with the above, they will likely be closed. Since we are a team of volunteers, we don't have any more time to work on the framework than you do. Please make it as painless for your contributions to be included as possible.
See https://github.com/codeigniter4/CodeIgniter4/blob/develop/contributing/pull_request.md

@virginus01 Please commit one thing in a git commit.

According to https://html.spec.whatwg.org/multipage/form-control-infrastructure.html#attr-fs-method
the method value is get or post, not uppercase.
So I think lowercase is better.
So, unfortunately, this PR is not acceptable.
Also, all PRs must pass all checks by GitHub Action.
In the future, please submit PRs that pass the checks.
Thank you.


composer support?

Bootstrap, CK Editor, Fontawesome, Jquery... They all have a composer.json file. It would be awesome if we could manage Angular in php apps through composer as well. Even our task runner (Robo) is a php tool and does all the asset building.
I don't mind creating a PR with a composer.json file, just let me know if it's desired or not.

Your task runner is in PHP, you poor soul.
Just kidding. I've used PHP too, it's not that bad.
I don't know about composer support. What's needed for that? Maybe this could also be a community project?

Composer support probably would require a build task modification modifying a composer.json file in all of the distribution repositories - here is an example composer.json file. Most of that can probably be chucked, and specifying the version is straightforward.

This never really got traction, and at this point in the AngularJS lifecycle it's probably not worth the effort to include this in the build.


Virtual script not found, may missing

Not quite sure what is going on here, but I am seeing the following error in VSCode as of this morning:
Request textDocument/inlayHint failed.

And I am also seeing the following:
Virtual script not found, may missing <script lang="ts"> / "allowJs": true / jsconfig.json.volar

Around every <script setup lang="ts"></script> in my project ...
I'm using version 0.40.4 of Volar

Related to #1786


[WIP] Provide influxdb as a new storage plugin

Please answer these questions before submitting pull request

Why submit this pull request?
[ ] Bug fix
[X] New feature provided
[ ] Improve performance

New feature or improvement

Describe the details and related test reports.

This PR provides a new storage plugin for OAP which uses two-level storage.
The first level is a relationship database stored something likes inventory, which has transactions and updates. Stored in MySQL.(Now, I make H2 as relationship database. Because it is easy for test.)
The second level is a time-series database, stored metrics, logs, and traces. Those data have strong time-lined and no-transaction, no-updated, no-deleted. Stored in InfluxDB.
InfluxDB is a very powerful time-series database. It is good at manipulating time-series data. So, it is suitable for our trace records that have strong time-lined, no-updated, and no-deleted.
Now, it is an experimental feature. We need to continuously improve it with you. And it is a meaningful experience for others NoSQL/TSDB likes Apache Druid, OpenTSDB.

TODO list

[x] feature
[ ] e2e
[ ] document
[ ] change MySQL as the level storage

How can I confirm that the submodule is the newest? If it is not the newest, what should I do?
I found my e2e always failed in my devel branch.
@wu-sheng , @kezhenxu94

How can I confirm that the submodule is the newest? If it is not the newest, what should I do?
I found my e2e always failed in my devel branch.
@wu-sheng , @kezhenxu94

I don't think it's a matter of the sub-modules, the interface IMetricsQueryDAO is recently modified and you did not modify your MetricsQuery corespondingly

I don't mean github action.  I am trying on my local branch which I have implemented.

How can I confirm that the submodule is the newest? If it is not the newest, what should I do?

Dependency changed, you need to update the LICENSE/NOTICE and dependency check files.

@dmsolr Besides the license issue, is this PR ready to be reviewed?

Please be noticed

This branch is out-of-date with the base branch

Do you merge the master branch with this branch correctly? Please fix them before we can do the review.

Is this PR ready to review?

Is this PR ready to review?

Yes.
Besides, I will adjust e2e-MySQL compatible with other storage-plugin(InfluxDB) in a new PR.

I will adjust e2e-MySQL compatible

What is the meaning of this?

Recommend to add e2e for this new storage in this PR. At least it could recheck this new feature is basically ready.

The corresponding yml in docker-entrypoint.sh should be also updated

Thanks for your review and reminder, @wu-sheng and @kezhenxu94
I will fix them later.

The corresponding yml in docker-entrypoint.sh should be also updated

Sorry, I don't understand. We need to update which file? @kezhenxu94

docker-entrypoint.sh files

@kezhenxu94 I don't want the e2e use submodule actually. Setting static proto files is good for compatible check and easy to check. I think we talked about this before, right?
For test reactor, it is not a block. It is test optimization.

I don't want this big PR blocked too long, it is last a long time, we could create an issue to track your test change proposal.

All proto files under test/e2e should be removed, instead, add the protocol repository as submodule, just like what e2e-ttl-es does, otherwise, they may be outdated
And I hope you could refactor the e2e-ttl-es to something like e2e-ttl that can be reused for multiple storage types, like e2e-profile, we can use the same codes to test multiple storage types

I am planning to do it, but it is a low priority in my schedule.

Profile e2e is failing, please recheck.

#4335 is going to be merged today. There would be the last two new storage methods introduced.

The strict format rule has been merged, please update this PR.

The ElasticSearch fails twice, I am rerunning it.

@dmsolr Please recheck. The ElasticSearch test fails agian. I don't find any change related to you. Please take a look, what is the issue.

@dmsolr Please recheck. The ElasticSearch test fails agian. I don't find any change related to you. Please take a look, what is the issue.

I had checked it yesterday. The error message is "No space leave on device". And this exception threw on other PR.

e2e fail because the profile task DAO is still accessing the H2/MySQL. It seems you miss a change. Please fix it.

We are waiting for nearly 24 hours, we will merge soon. Anyone has suggestions feels free to submit a separated PR


converted VSTeamQueue from format/type to class

PR Summary
converted VSTeamQueue to class and added it

to the project root in the provider
to Build Definition

PR Checklist

[X] [Write Unit Test]

Tests executed successfully
.\unit\test\buildDefinitions.Tests.ps1;
.\unit\test\queues.Tests.ps1;
.\unit\test\pools.Tests.ps1;

Tests executed
.\integration\test\010_projects.Tests.ps1;

I encountered a problem with my VSTS test account - but that should have no impact
  Context Feed exercise
    [-] Add-VSTeamFeed should add a feed 2.95s
      WebException: The remote server returned an error: (402) Payment Required.
...

Man, you are getting work done!  Thank you so much for the support. While I was reviewing this pr I found that one of the unit tests is failing.
Describing TeamsPSDrive

    Context VSTeamAccount & Projects
      [+] Should create VSTeamAccount 99ms
      [+] Should return projects 72ms
      [-] Should return builds, releases, repositories and teams

Looks like you added queues to the projects Get-ChildItems but did not update the test.

@DarqueWarrior is it possible that you grant me at least R/O rights to the CI - so I can check errors

I want to! I can't!. We are working through public projects and permissions. But now you can't change the build number which the build does. Sorry for the issue. I hope we get this fixed.

I have disabled the task that is causing the issues with builds failing queued from PRs.  They should pass going forward.


Firebase gen 2 ignoring httpsOptions

Environment
Nuxt 3.7.0 with Nitro 2.6.1
Node 18
firebase cloud functions V2
Reproduction
httpsOptions: {   region: 'asia-south1', }
Describe the bug
when selecting a region with Gen 2  function, the region is ignored and is deploying with default region (us-central1).
Additional context
When looked into the build output:
const server = onRequest(   {     // Must be set to public to allow all public requests by default     invoker: "public",     ...firebaseConfig.httpOptions   },   toNodeListener(nitroApp.h3App) );
firebaseConfig.httpOptions is undefined
the option object is firebaseConfig.httpsOptions
I tried replacing the code with replace option in nitro but it didn't work
Logs
No response

This will be fixed with https://github.com/unjs/nitro/pull/1663


Leff side menu backgroud color

Hi, I'm creating an angular2 version of AdminLTE.
when the content of the page is larger than the left side menu, the background color is not repeated, how you can see in the picture.
What could I have forgotten?

Please check your console and verify that you have loaded all the needed resources. You most likely didn't include app.js after jquery.js and bootstrap.js.
Thanks!


Broken link on BY-NC license page

Email from user:

The website
https://creativecommons.org/licenses/by-nc/4.0/
has a semi-broken FAQ link when clicking on "commercial use" and then
"more info".
The wrong link only leads to the FAQ overview because the link anchor
(#...) is wrong:
https://creativecommons.org/faq/#Does_my_use_violate_the_NonCommercial_clause_of_the_licenses.3F
Corrected version that directly points to the right FAQ question:
https://creativecommons.org/faq/#does-my-use-violate-the-noncommercial-clause-of-the-licenses

@TimidRobot I have created a PR.


Social Media Icon Buttons are not Working

Describe the bug
Social Media Icon Buttons are not Working properly in profile page. while clicking on them they are not redirecting to the particular path.
To Reproduce
Steps to reproduce the behavior:

Set-up the project in your localhost.
Now run and go to the profile page.
You find icons of facebook, github,linkedin etc.
click on them they are not redirecting before clicking make sure that you have provided the user profile name in the user settings path for that social media.

https://github.com/c2siorg/Codelabz/assets/106808387/f16d204b-1bff-45e6-8cc7-7ae278fa6f24
Expected behavior
while clicking on them they should redirect to the particular path. I mean user profile.

@Hemu21 Nice work, but this one is already solved brother i.e. storing the data at profile settings and then populating the social icons with it and redirecting.
But if you did it in some different way like something more cool or wild make sure to mention it, here in the threads or in the PR explicitly.

@Hemu21 You can create multiple PR for the same issue if you feel like more than one solution would be expected for that one in particular. You would know by looking at the issue, cause they want the best solution.
But for this one it's a no no

ok I am closing.


Still can't access to StreamDuration.

Still can't access to StreamDuration. it doesn't ignore the normal duration. Tried everything.

You have to add and import the StreamDuration library, from here too.
When you have added that to your pubspec and done a flutter pub get you can use it like this to for example pause your timer:
     final streamDuration = StreamDuration(
      Duration(seconds: seconds),
    );
    if (!isRunning) {
      streamDuration.pause();
    }
    return SlideCountdown(
      duration: const Duration(seconds: 0),
      streamDuration: streamDuration,
    );

You still have to add the duration argument, but it is ignored (see #36).


show results in ranking resource

Yeah showing the domain would be good for sure.

Please update to fix merge conflict

Yes.