Add resource limits for upgrade and delete hook batch jobs

+1 to a global setting for these. Switching it to only be global though may break some existing users.
Maybe use the sprig mergeOverwrite function to support both? (https://masterminds.github.io/sprig/dicts.html)

@kfox1111 These global resource limits are only being set for all batch related containers. Which other case of resource limits do you think this will conflict with?

Oh. I guess I saw a commit wiz by (https://github.com/spiffe/helm-charts-hardened/pull/322/commits/d099e572be8de5548cf276b34e1a9e26925bb61c) and thought it was changing existing code. Never mind my previous comment. Sorry for the confusion.

We can't write to this repo, and we haven't heard back from the original author in a long time. Continuing development in https://github.com/spiffe/helm-charts-hardened/pull/366 instead since we can write to it there.


Support for fuzzy matching subcommand

Fix: #1689

CI failed on upgrade test, I'll reschedule once https://github.com/teemtee/upgrade/pull/9 is merged.

/packit test

Hi @lukaszachy, do we need to add a simple test for this PR?

Hi @lukaszachy, do we need to add a simple test for this PR?

Of course we do ;) We claim in the docs 'command abbreviations'  as a feature which (until this PR) was broken for some cases.
With test we mitigate regression risk.

Hi @lukaszachy, do we need to add a simple test for this PR?

Of course we do ;) We claim in the docs 'command abbreviations' as a feature which (until this PR) was broken for some cases. With test we mitigate regression risk.

A simple test is added. @lukaszachy, please help to review, thanks!

F-37 failed on downloading epel rpm. On other distros everything has passed so I don't see any reason why to waste resources and time thus merged.


doc: Remove indefinite article and ending period from example meta.description

so that meta.description examples shown in the documentation align with recommendations given in the "Meta attributes" section in pkgs/README.md.
The changes were made with the following commands:
nix run nixpkgs#silver-searcher -- -l0 'description\s*=\s*"([Aa]n?|[Tt]he)\s' doc \
  | xargs -0 nix run nixpkgs#gnused -- -i '' -Ee '/description/s/"([Aa]n?|[Tt]he)\s(.)/"\U\2/'
nix run nixpkgs#silver-searcher -- -l0 'description\s*=\s*".*\."' doc \
  | xargs -0 nix run nixpkgs#gnused -- -i '' -Ee '/description/s/\."/"/'

Description of changes

Things done

Built on platform(s)

[ ] 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

[ ] Fits CONTRIBUTING.md.

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

Thanks for reviewing and merging this, @mweinelt, much appreciated 👍


2023general_dateupdate_mail-in

Hey can you approve the request, its a date update over here to oct 10


Core logic rewrite and refactor.

The PR introduces some code reorganization to make future work simpler, as well as adds some new tests.
Changes introduced

Separates the gathering of facts from incoming unstructured types (i.e. configMap reads) from actually writing those facts. This was a single step before - but this is a change towards SRP.
Gets rid of "interface" type  - thereby supporting better static type checking across the entire system.
Deprecates support for 'sbom' and 'image' types. The build-deploy tool only actually sends 'sbom-gz' and 'image-gz' types since we introduced compression. Supporting both paths introduces needless complexity.
Deprecates support for the 'direct' fact type introduced https://github.com/uselagoon/insights-handler/pull/18 - this has been replaced by the 'direct.facts' and 'direct.problems' types
Because of previous deprecations above, we can, further, deprecate processing of non-binary payloads in general for non-typed/direct insights payloads.

This PR gives me strong healing/cleansing vibes 😆


Remove vendor directory

Hi!
Please ignore the vendor directory when generating the package.
Thanks!

PRs welcome

On 9. Aug 2020, at 16:07, Justin Dane Vallar<EMAIL_ADDRESS>wrote:

Hi!
Please ignore the vendor directory when generating the package.
Thanks!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

@aeneasr I can do a PR but I'm not really sure if this is the right repo to be submitting it since this is generated using openapi generator. Do you have a guide for contributors?


Simplify logic in calculating proposer score

The call to get_ancestor when checking if the proposer score should be calculated in get_latest_attesting_balance seems to be unnecessary so I removed it here.

While the simplifying changes in this PR were sound for the underlying logic in the spec, the underlying logic itself was incorrect (see above comment).
New changes in #2760 remove the simplification & fix the spec logic (as pointed out in #2757).
Thanks @realbigsean for both this PR and #2757!


Question about query results

The pyorient.otypes.OrientBinaryObject object contains after each query returns a dictionary, how to parse it.

I'm not quite clear to which dictionary you're referring, but for OrientBinaryObject, take a look at test_csv_decoding() in tests/test_serializations.py in the develop branch.


Create German Readme.md translations

WIP
I am  interested in this project, so I started translating German.

[x]  1
[ ]  1-1
[ ]  1-2
[x]  1-3
[ ]  2
[ ]  2-1
[ ] 2-2
[ ] 2-3
[ ] 2-4
[ ] 3
[ ] 3-1
[ ]  3-2
[ ] 3-3
[ ] 4
[ ]  5
[ ]  5-2
[ ] 5-3
[ ]  6
[ ]  6-1
[ ] 6-2
[ ] 6-3
[ ] 6-4
[ ] 6-5
[ ] 6-6
[ ] 7
[ ] 7-1
[ ] 7-2
[ ] 7-3
[ ] 7-4

As mentioned in this command, I will create/ edit the German Translation files in the next days and will add them to this PR.

thank you very much! cc @FloorD

hi, can I get a status update on these translations? thanks!

hi, can I get a status update on these translations? thanks!


Structure Refactoring

Restructure the base folder layout of instance scheduler to more clearly divide the various components. New core structure is
/source
  /app
    /tests
  /infrastructure
    /tests
  /cli
    /tests (currently empty/not used)

Changes also include the removal of build-open-source-dist from the deployment path (actual file to be removed later) and resetting .viperlightignore as most of the ignores are no longer needed.
Things tested

Full deployment pipeline runs successfully
SonarQube picks up test coverage
Solution successfully deploys to clean account
Lambda is still able to execute and disable EC2 instances

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

The commit history needs cleaned up.

done


perf(encoding): add estimate size for value encoding

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
add function to estimate the value encoding size of the datum
prepare for https://github.com/risingwavelabs/risingwave/issues/8683
Checklist For Contributors

[x] All checks passed in ./risedev check (or alias, ./risedev c)

Checklist For Reviewers

[ ] I have requested macro/micro-benchmarks as this PR can affect performance substantially, and the results are shown.

Documentation

[ ] My PR DOES NOT contain user-facing changes.

Click here for Documentation
Types of user-facing changes
Please keep the types that apply to your changes, and remove the others.

Installation and deployment
Connector (sources & sinks)
SQL commands, functions, and operators
RisingWave cluster configuration changes
Other (please specify in the release note below)

Release note

Why it’s estimated_size? IIUC it’s exact.
It’s better to have fuzz test here, but It may be a little complicated. At least, you should write UT for every type to ensure the format will not be broken silently.

You can write some datums, encode them, and compare with your pre-calculated result.


Feat/#tc 39 create page layout

This pull request closes:
✔ Closes: #208
🏁 Jira Issue: TC-39
Which service(s) does this issue affect:

[ ] 🛰 API
[ ] 🛒 webstore
[x] 🎓 admin-page

Types of changes

[ ] 🐛 Bug fix (a non-breaking change which fixes an issue)
[x] 🧱 New feature (a non-breaking change which adds functionality)
[ ] 💥 Breaking change (fix or feature that would cause existing functionality to change)
[ ] 🤖 CI/CD (a change to the CI/CD pipeline)
[ ] ⚗ Refactoring (an update to some already existing code)

Documentation Updates

[x] 📂 No need for updates
[ ] 📁 Requires an update ( not done within this issue )
[ ] 🗃 Has been updated ( done within this issue )

Testing checklist

[x] 💪 Manual tests
[x] 🔧 Automatic tests

Browser compatibility - [Frontend only]

[ ] Tested on Safari
[x] Tested on Chrome
[x] Tested on Mozilla Firefox

What changes have been done to this issue?
I created layout.component, which sets the structure of a page. It contains the navbar, a collapsible sidebar with buttons for all of the resources we plan on managing, and a space for displaying the content of the page.

I implemented a temporary routing solution to test how my component would be working.
How should this be manually tested?
By launching the admin page app (nx serve admin-page) and checking that

the navbar is rendered properly and its buttons work correctly.
the sidebar is rendered and is collapsible
the sidebar buttons work
if the sidebar's contents do not fit within the sidebar (due to the window being too small, for instance), a scrollbar will appear on the y axis.

Note: A button is considered to work properly if it correctly changes the URL to reflect the button's name and if the content properly displays a text with the button's name.

Screenshots or example usage

I have verified that it works on Safari ✅


Add client tests

Adding tests for:

[x] GET caching
[x] POST caching
[x] HEAD no caching
[x] force_cache extension
[x] cache_disabled extension

Any other methods should be tested?

Got rid of multiple function calls for date formatting in tests. Made it a single call and just used a var everywhere else.

You have merge access, right?

You have merge access, right?

I do. I wasn't sure if you want me to do it or not tho.

Feel free to merge the RP that you think is ready.

Feel free to merge the RP that you think is ready.

Ok! Tbh I thought if this is enough tests and maybe we should add tests for every method?

I think these tests are more than enough. Clients are not really important because all the caching logic is defined in the transports.

Ok then.


Ability to manually save state

Is your feature request related to a problem? Please describe.

If I know a particular action changes something significant enough to warrant saving the state to disk, I want to be able to request a save (similar to how you can use ctx.request_repaint() and related functions if you know a repaint will be needed).
Describe the solution you'd like

The idea that comes to mind is adding a ctx.request_save() or similar, which triggers the same behavior as if the autosave duration was reached.
Describe alternatives you've considered

I tried storing the creation context passed to the eframe::App, but that caused ownership problems.
I tried storing a reference to the eframe::Storage instead, and also ran into ownership problems.
I can't even figure out a messy way to do it with existing functionality.

I've found a workaround, basically by making auto_save_interval return 0 milliseconds if a save was requested, and 30 seconds. Setting self.needs_save = true means the next time maybe_autosave is called, it should save.
#[derive(Default)]
struct MainApp {
    // ...
    tasks: TaskList,
    needs_save: bool,
}

impl eframe::App for MainApp {
    // ...

    fn auto_save_interval(&self) -> std::time::Duration {
        if self.needs_save {
            std::time::Duration::from_secs(0)
        } else {
            std::time::Duration::from_secs(30)
        }
    }

    fn save(&mut self, storage: &mut dyn eframe::Storage) {
        eframe::set_value(storage, eframe::APP_KEY, &self.tasks);
        self.needs_save = false;
    }
}


Encrypt or make it harder for users to mess with cookies

Currently, The cookies that hold most of the data are easily edited by any user.
I had someone with 1000 points this morning which I had to delete, We need to either encrypt it from the server side or instead of setting the cookies in the front end we can do it server-side and set it to httpONLY which will be designed to be inaccessible to JavaScript, so they cannot be modified or accessed by client-side code so only API calls can edit it which will require a lot of code changes to the backend.
Opened to ideas

@nazrul7711 would you have the knowledge to work on this?

i will try @Chris5613

@nazrul7711  Hey any updates? I know this is a pretty hard issue I've been having alot of trouble figuring it out myself.

@Chris5613 I haven't started working on it . I was busy with something else. I will start working on it today

@nazrul7711 Hey you no longer need to work on this issue. I solved it, I deleted the cookies and moved it on to the server side so users can't mess with it.


Create VSIX and upload it to a publicly accessible blob store after every build, including CI.

We would like an automatic way of sharing even PR builds with our customers as well as other stakeholders.
We already upload bits to Azure blob storage for every build going through our automation, but we don't make the VSIX available for each build yet.
Ideally, we could then create 1-off builds for stakeholders that have specific issues we want to get more information from, and have a lasting record of how we went about getting that information in our CI system. We also want to ensure that we aren't sending any builds created by our own machines out to anyone!

What's the benefit of creating build for each and storing it?

The scenario I have in mind is the one I've seen in past issues where we create a build (presumably via CI, but perhaps from our local workstation) that is to help discover a specific hard to reproduce issue that a user reports. If we go through CI we can just create a PR and send it in to have the user test things out.
Pedantic, perhaps, but I believe that in the ideal world, we never create anything locally to share with our users.

Oh, and we create the VSIX (essentially, if not practically) already in VSTS CI, and we actually are already uploading it with each build anyways...

Wee push a PR then have users test the dev release. I don't think we need the overhead of more builds, storage maintenance.

we actually are already uploading it with each build anyways...

Yes, but only in master. I.e. not in PRs.

Yes, but only in master.
Nope. All builds. As a necessity of the parallelism I've added.

I don't think we need the overhead of more builds,...

 However, this would not be an issue. 0 extra builds to manage, not sure what exactly you meant by that.

...[I don't think we need the overhead of] storage maintenance.

You are correct that storage maintenance would become a thing, and would  mean we would clear out older builds weekly/monthly using a very simple script (or we can just let them accumulate, disk is cheap, clear them out once a year manually or whatever).

You are correct that storage maintenance would become a thing, and would mean we would clear out

As we don't need this at this stage, my suggestion would be to leave this for now.
I personally don't see any harm in doing this if it doesn't add any overhead of maintenance. I'm not saying its a bad thing, just saying we don't need it now.

What's the storage overhead? Bits are cheap for us on Azure. As long as we only ever have one .vsix per PR I don't see how this is really a maintenance issue beyond the initial setup.
But I also don't view this as critical either, so definitely a low priority thing behind us writing more tests, getting signing going, taking over the language server builds, etc.

What's the storage overhead?

It is extremely minimal. We have one VSIX and one zip archive full of the built intermediate 'bits' stored in  a folder on blob storage in Azure.
The past months worth of builds amounted to $0.25 CAD on a commercial (not internal) Azure subscription.

@d3r3kk
Do we still need this? Can't remember the details, we already upload to Azure on CI.

If it's not done, then let's leave it open


Add system test for insert/fetch of rows w/ nested schema.

Uses #3118 as a base.
Derived from schema / data from @tswast.
Toward #3029.

@tseaver should be able to merge #3118 and rebase this off master.
Also, there's a lint error in this branch just FYI.

@dhermes I think I answered your questions.

@dhermes, @lukesneeringer, @daspecster Any remaining issues?

None from me. :-)


Update dependencies to Kubernetes v1.28

Updates dependencies to Kubernetes v1.28

:+1:

v0.2.15 is available to reference


Database proposal: GunDB

Since Blockstack is no more supported, a very nice idea would be adopting GunDB. It's a decentralized (off-chain) database that syncs data across all clients - and what they call a SuperPeer. There's also the possibility to create a user space with private data. @brandoncorbin It would be very cool if we could implement this 😍

Hey Sammosna thanks for the suggestion, I've played around with GunDB in the past but didn't get very far mainly due to the collective file sizes of the nomie dataset. However, I'm actively working on 6 and specifically the storage engine - so I'll take a look at it again and see what it would take to make it a viable nomie storage engine.


feat: Cloud option to download file-based artifacts

Feature scope
Other
Description
There's a few ideas here, so maybe warrants another issue. If a user runs target-csv, target-jsonl, or basically any file-based target, that data currently just disappears. We could store that for them and enable them to download it via the CLI and UI.
Similarly, as we work on #https://github.com/meltano/meltano/issues/7081 we should enable users the ability to download these artifacts from Cloud.
We'd need to set some sort of deletion policy, but this could be a useful feature.
Discussed with @visch :smile

Thought about this more after the call (and happened to see this), this makes a lot of sense for me when I"m running with a job runner (Gitlab runner in my case) but if I was running on pure meltano cloud setting up a push to S3 or something wouldn't be that big of a deal as we'd be in Meltano's infra which would presumably allow outbound connections. This just avoids me having to ask folks to open up external output ports (if they lock them down) or any other web filters / etc that get in the way.

@tayloramurphy can I pick this up?

@tayloramurphy @edgarrmondragon can I pick this ?

@XshubhamX Generally I wouldn't recommend picking up any features for Meltano Cloud. Also, with this feature in particular, it would not be possible to support without changes to the backend for Meltano Cloud, which is not open source.


WorkspaceManager: loading in-memory models (fromString) yields error message in some situations

Langium version: 0.3.0
Package name: langium
Summary
(see https://github.com/langium/langium/discussions/391)
In Langium 0.3, in the WorkspaceManager: I wonder what is intended to be passed as URI to LangiumDocumentFactory.fromString when using an in-memory model (which has no corresponding file). Using a non-existing filename seems to work, but yields logs on the console in some situations:
The following happens when I move the cursor over a reference to the model built in memory (I used the URI URI.parse("built_in")).
[Error - 8:00:43 PM] Request textDocument/documentHighlight failed.
  Message: Request textDocument/documentHighlight failed with message: ENOENT: no such file or directory, open '/built_in'
  Code: -32603

Probably I misunderstood how I should call the API?
Steps To Reproduce

Create a custom WorkspaceManager extending the DefaultWorkspaceManager
There, in loadAdditionalDocuments load a built-in model from a string to be referenced from user models with the following snippet (note: the file "built_in" does not exist):

const doc : LangiumDocument = this.factory.fromString(model_text, URI.parse("built_in"));
_collector(doc);

Reference data from the built-in-model in your user model (references are resolved correctly, no error).
Move with the cursor over the reference text
Then you get the "effect" in the vscode log ("Request textDocument/documentHighlight failed", see above)

Link to code example:

https://github.com/goto40/item-lang/blob/4a852104ae30f32e9bd8ed20b993b19a2aaeaed5/src/language-server/item-language-workspace.ts#L32 (if this example is too complicated, I can try to reproduce with a modified hello world example).
In this example you need to build the language and open the example with the generated plugin: https://github.com/goto40/item-lang/blob/4a852104ae30f32e9bd8ed20b993b19a2aaeaed5/examples/model/array_demo.item#L7
move over built_in.uint32

The current behavior
You get a message Request textDocument/documentHighlight failed...
The expected behavior

The ability to pass "no URL" to fromString or to document what to pass for in-memory model text snippets.
No such message: Request textDocument/documentHighlight failed...

Hey @goto40,
I believe that the LangiumDocuments service tries to resolve the in-memory file from the disk when getting the request, since it hasn't been added to the cache. Can you add a call to LangiumDocuments#addDocument and add doc this way?
Also URI.parse("built_in") creates a uri which actually tries to point to a file. Since the URI schema is missing, VSCode probably mistakenly expects that the URI points to a file. Something like URI.parse("memory://built_in") should be better.

Thank you @msujew ! Your suggestion (calling this.langiumDocuments.addDocument(doc);) solved the issue. Very good! I will add a hint in the discussion section.

Thank you @msujew ! Your suggestion (calling this.langiumDocuments.addDocument(doc);) solved the issue. Very good! I will add a hint in the discussion section: #391.

Note: for langium 0.4.0 this is not necessary any more (if addDocument is called you get an error indicating the document is already loaded),


shape replace not working as expected with BPMN 2.0 shape library

Preflight Checklist

[ x] I agree to follow the Code of Conduct that this project adheres to.
[x ] I have searched the issue tracker for a feature request that matches the one I want to file, without success.

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

create a diagram with the BPMN 2.0 shape library
add a shape
drag another shape over it
See error

Expected behavior
the shape
Screenshots

draw.io version (In the Help->About menu of the draw.io editor):

draw.io desktop APP version [22.1.2]

Desktop (please complete the following information):

OS: MacOS
Browser: Safari
Version [22.1.7]

Additional context
tested in the desktop app and the safari app

I'm having the same problem with a custom library we use. We previously could drag/drop icon images from the custom library onto images on the canvas and they would replace that image. This no longer works with any of our custom library icons.
It also doesn't work with the shapes from the BPMN 2.0 library.
It DOES work with icons from the web icons file.
I've tested multiple browsers and multiple patterns of placing images on the canvas and then drag/drop replacement. The consistent problem is that BPMN library shapes (and our custom library images) will not update the image on the canvas.
OS: Linux
Browser: Chrome, Firefox, Edge
Drawio Version: 21.3.2

Fix in 21.3.4


WIP: feature/stacking_earn_boxes

Change note:

Add basic stacking / unstacking smart contract base on copying from pseudocode.
Still lack of issue boxes when user unstacking.
Still lack of "advance" security since I use a basic method (onlyOwner).
Still lack of testing.
Still lack of deployment code.
Need @chiro-hiro review and input your comments to improve my coding style.

Overall, LGTM


Updates for CircleCI

This is a couple updates for the Chem CircleCI config.

You don't want to checkout-if-exists develop if you are testing develop. Why? Well a branch named develop exists on a lot of repos and things go a bit nutty. This was discovered in GEOSgcm_GridComp
Update to use Baselibs 6.0.10. This is future-proofing for MAPL 2.1 (essentially, do it now so I don't forget later). It works just fine with MAPL 2.0 as well.

@mathomp4 Does to update from Baselibs 6.0.4  to  6.0.11   have any chance of non-zero-diff?

@mathomp4 Does to update from Baselibs 6.0.4 to 6.0.11 have any chance of non-zero-diff?

Nope. Not unless you are using any weird functions from one of Tom's libraries from outside of MAPL. Almost all the differences were in them. MAPL 2.1.1 needs to 6.0.11 (which the GEOSgcm now uses) and it's zero-diff to MAPL 2.0.

This PR passed all zero-diff checks when tested with PChem. The 1-day free run did not zero-diff when testing with GMI.


I think we should use html / js / css for the game, what about you?

(also we should probably come up with a game idea)

Couple of things

Move this to Discussions. If you can't find that, give me full access and I'll find it for you.
Yes, HTML5 and JS sounds like a good bet so that it's easy to run (no downloading, no compilation).
Yes, we should get a game idea.


fix(variance): Fix static css mutability for css [GM-235]

Overview

Some how I managed to forward theme on to all static CSS using system.css by using an unsafe assign.  This meant the theme would be added to the previous static object and returned as CSS even if the parser ignored it.
Normally this wouldn't be an issue if you called system.css with an object literal as the response would be cached from the first call - preventing the mutability issue:
system.css({
  m: 4,
  '&:hover': {
     bg: 'blue',
  }
})

However if you used the object between multiple css functions the select configuration would be mutated by the first call the create the cache and causing the second css function to attempt to include theme as styles:
const hoverStyles = {
  '&:hover': {
     bg: 'blue',
  }
}. as const;

// Works as expected and caches correctly
const buttonStyles = system.css({
  // stuff
  ...hoverStyles
});

// Whole bunch of pain
const linkStyles =  = system.css({
  // other stuff
  ...hoverStyles
});

This fixes the mutability issue by adding a spread where the unsafe assign was used.
And by adding theme to the filter list if it is somehow passed manually.

PR Checklist

[ ] Related to designs:
[x] Related to JIRA ticket: https://codecademy.atlassian.net/browse/GM-235
[x] I have run this code to verify it works
[ ] This PR includes unit tests for the code change

📬Published Alpha Packages:@codecademy/gamut-icons@4.14.5-alpha.03179b.0@codecademy/gamut-illustrations@0.17.1-alpha.03179b.0@codecademy/gamut-kit@0.4.5-alpha.03179b.0@codecademy/gamut-labs@18.3.5-alpha.03179b.0@codecademy/gamut-patterns@0.3.1-alpha.03179b.0@codecademy/gamut-styles@10.2.1-alpha.03179b.0@codecademy/gamut-tests@2.4.4-alpha.03179b.0@codecademy/gamut@33.4.3-alpha.03179b.0@codecademy/konamimojisplosion@0.2.35-alpha.03179b.0@codecademy/markdown-overrides@0.5.47-alpha.03179b.0@codecademy/styleguide@40.3.3-alpha.03179b.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://60b6f58e4baf50475157f018--gamut-preview.netlify.appDeploy Logs


Improve error page displayed during prod outages

During the Azure outage that happened on 9/7/22, we were serving users the following screen when trying to load the production app, as well as lower environments:

We could not deploy the maintenance banner because Azure was down. Find out where we're hosting this error page and work to make it more useful. Possibly a good idea to work with content and/or product to come up with the best message to convey what's happening.

Design/content ask here: a static HTML page we can provide as an alternative to the existing page.
Once we have that, we can ask CDC to change this in Akamai.


ValueError which I do not know how to solve

2021-08-11 22:09:32,774 ERROR trial_runner.py:482 -- Error processing event.
Traceback (most recent call last):
  File "/home/neousys/anaconda3/envs/flow/lib/python3.7/site-packages/ray/tune/trial_runner.py", line 426, in _process_trial
    result = self.trial_executor.fetch_result(trial)
  File "/home/neousys/anaconda3/envs/flow/lib/python3.7/site-packages/ray/tune/ray_trial_executor.py", line 378, in fetch_result
    result = ray.get(trial_future[0], DEFAULT_GET_TIMEOUT)
  File "/home/neousys/anaconda3/envs/flow/lib/python3.7/site-packages/ray/worker.py", line 1457, in get
    raise value.as_instanceof_cause()
ray.exceptions.RayTaskError(ValueError): ray::PPO.train() (pid=29651, ip=<IP_ADDRESS>)
  File "python/ray/_raylet.pyx", line 636, in ray._raylet.execute_task
  File "python/ray/_raylet.pyx", line 619, in ray._raylet.execute_task.function_executor
  File "/home/neousys/anaconda3/envs/flow/lib/python3.7/site-packages/ray/rllib/agents/trainer.py", line 444, in train
    raise e
  File "/home/neousys/anaconda3/envs/flow/lib/python3.7/site-packages/ray/rllib/agents/trainer.py", line 433, in train
    result = Trainable.train(self)
  File "/home/neousys/anaconda3/envs/flow/lib/python3.7/site-packages/ray/tune/trainable.py", line 176, in train
    result = self._train()
  File "/home/neousys/anaconda3/envs/flow/lib/python3.7/site-packages/ray/rllib/agents/trainer_template.py", line 129, in _train
    fetches = self.optimizer.step()
  File "/home/neousys/anaconda3/envs/flow/lib/python3.7/site-packages/ray/rllib/optimizers/multi_gpu_optimizer.py", line 140, in step
    self.num_envs_per_worker, self.train_batch_size)
  File "/home/neousys/anaconda3/envs/flow/lib/python3.7/site-packages/ray/rllib/optimizers/rollout.py", line 29, in collect_samples
    next_sample = ray_get_and_free(fut_sample)
  File "/home/neousys/anaconda3/envs/flow/lib/python3.7/site-packages/ray/rllib/utils/memory.py", line 33, in ray_get_and_free
    result = ray.get(object_ids)
ray.exceptions.RayTaskError(ValueError): ray::RolloutWorker.sample() (pid=30009, ip=<IP_ADDRESS>)
  File "/home/neousys/anaconda3/envs/flow/lib/python3.7/site-packages/ray/rllib/utils/tf_run_builder.py", line 94, in run_timeline
    fetches = sess.run(ops, feed_dict=feed_dict)
  File "/home/neousys/anaconda3/envs/flow/lib/python3.7/site-packages/tensorflow_core/python/client/session.py", line 956, in run
    run_metadata_ptr)
  File "/home/neousys/anaconda3/envs/flow/lib/python3.7/site-packages/tensorflow_core/python/client/session.py", line 1156, in _run
    (np_val.shape, subfeed_t.name, str(subfeed_t.get_shape())))
ValueError: Cannot feed value of shape (1, 132) for Tensor 'default_policy/observation:0', which has shape '(?, 129)'

During handling of the above exception, another exception occurred:

ray::RolloutWorker.sample() (pid=30009, ip=<IP_ADDRESS>)
  File "python/ray/_raylet.pyx", line 633, in ray._raylet.execute_task
  File "python/ray/_raylet.pyx", line 634, in ray._raylet.execute_task
  File "python/ray/_raylet.pyx", line 636, in ray._raylet.execute_task
  File "python/ray/_raylet.pyx", line 619, in ray._raylet.execute_task.function_executor
  File "/home/neousys/anaconda3/envs/flow/lib/python3.7/site-packages/ray/rllib/evaluation/rollout_worker.py", line 471, in sample
    batches = [self.input_reader.next()]
  File "/home/neousys/anaconda3/envs/flow/lib/python3.7/site-packages/ray/rllib/evaluation/sampler.py", line 56, in next
    batches = [self.get_data()]
  File "/home/neousys/anaconda3/envs/flow/lib/python3.7/site-packages/ray/rllib/evaluation/sampler.py", line 99, in get_data
    item = next(self.rollout_provider)
  File "/home/neousys/anaconda3/envs/flow/lib/python3.7/site-packages/ray/rllib/evaluation/sampler.py", line 327, in _env_runner
    active_episodes)
  File "/home/neousys/anaconda3/envs/flow/lib/python3.7/site-packages/ray/rllib/evaluation/sampler.py", line 551, in _do_policy_eval
    eval_results[k] = builder.get(v)
  File "/home/neousys/anaconda3/envs/flow/lib/python3.7/site-packages/ray/rllib/utils/tf_run_builder.py", line 53, in get
    self.fetches, self.feed_dict))
ValueError: Error fetching: [<tf.Tensor 'default_policy/add:0' shape=(?, 2) dtype=float32>, {'action_prob': <tf.Tensor 'default_policy/Exp_1:0' shape=(?,) dtype=float32>, 'action_logp': <tf.Tensor 'default_policy/sub_2:0' shape=(?,) dtype=float32>, 'vf_preds': <tf.Tensor 'default_policy/Reshape:0' shape=(?,) dtype=float32>, 'behaviour_logits': <tf.Tensor 'default_policy/model/fc_out/BiasAdd:0' shape=(?, 4) dtype=float32>}], feed_dict={<tf.Tensor 'default_policy/observation:0' shape=(?, 129) dtype=float32>: [array([3.62607287e-01, 3.62607287e-01, 3.62607287e-01, 3.62607287e-01,
       3.62607287e-01, 3.62607287e-01, 3.62607287e-01, 3.62607287e-01,
       3.62607340e-01, 3.62607340e-01, 3.62607340e-01, 3.62607340e-01,
       3.62607287e-01, 3.62607287e-01, 3.62607287e-01, 3.62607287e-01,
       3.62607287e-01, 3.62607273e-01, 3.62607287e-01, 3.62607287e-01,
       3.62607340e-01, 3.33613052e-01, 3.62607340e-01, 3.62607340e-01,
       3.62607287e-01, 3.62607287e-01, 3.62607287e-01, 3.62607287e-01,
       3.62607287e-01, 3.62607287e-01, 3.62607287e-01, 3.62607287e-01,
       3.62607287e-01, 3.62607287e-01, 3.62607287e-01, 3.62607287e-01,
       3.62607287e-01, 3.62607287e-01, 3.62607287e-01, 3.62607340e-01,
       3.62607340e-01, 3.62607340e-01, 3.62607340e-01, 3.33333333e-01,
       9.43940256e-05, 1.20095672e-04, 9.43940256e-05, 9.43940256e-05,
       9.30966234e-02, 9.31132363e-02, 9.31395871e-02, 9.31451387e-02,
       1.86117317e-01, 1.86149407e-01, 1.86156975e-01, 1.86150932e-01,
       2.79174202e-01, 2.79149417e-01, 2.79173169e-01, 2.79159042e-01,
       3.72200282e-01, 3.72188549e-01, 3.72195222e-01, 3.72175878e-01,
       4.65219661e-01, 4.65186232e-01, 4.65201644e-01, 4.65181475e-01,
       5.58237048e-01, 5.58225759e-01, 5.58259103e-01, 5.58234339e-01,
       6.51255705e-01, 6.51230188e-01, 6.51253134e-01, 6.51250062e-01,
       7.44284794e-01, 7.44298284e-01, 7.44270765e-01, 7.44280813e-01,
       8.37336632e-01, 8.37314828e-01, 8.37329671e-01, 8.37286100e-01,
       9.30318294e-01, 9.30349433e-01, 9.30293162e-01, 7.50051750e-01,
       0.00000000e+00, 2.50000000e-01, 5.00000000e-01, 7.50000000e-01,
       0.00000000e+00, 2.50000000e-01, 5.00000000e-01, 7.50000000e-01,
       0.00000000e+00, 2.50000000e-01, 5.00000000e-01, 7.50000000e-01,
       0.00000000e+00, 2.50000000e-01, 5.00000000e-01, 7.50000000e-01,
       0.00000000e+00, 2.50000000e-01, 5.00000000e-01, 7.50000000e-01,
       0.00000000e+00, 2.50000000e-01, 5.00000000e-01, 7.50000000e-01,
       0.00000000e+00, 2.50000000e-01, 5.00000000e-01, 7.50000000e-01,
       0.00000000e+00, 2.50000000e-01, 5.00000000e-01, 7.50000000e-01,
       0.00000000e+00, 2.50000000e-01, 5.00000000e-01, 7.50000000e-01,
       0.00000000e+00, 2.50000000e-01, 5.00000000e-01, 7.50000000e-01,
       0.00000000e+00, 2.50000000e-01, 5.00000000e-01, 7.50000000e-01])], <tf.Tensor 'default_policy/action:0' shape=(?, 2) dtype=float32>: [array([-0.8724061, -0.5400728], dtype=float32)], <tf.Tensor 'default_policy/prev_reward:0' shape=(?,) dtype=float32>: [1.0425041000281703], <tf.Tensor 'default_policy/PlaceholderWithDefault:0' shape=() dtype=bool>: False}

When I add many vehicles , the Value Error produces ,especially when I use the network except RingNetwork. Sometimes , when the number of my vehicles are not too high ,and the length of the network is long ,the error won't produce. I do not know where the error produce. What code should I modify to avoid the error reproducing. Could anyone give me some advise?

Perhaps your road is not enough to accommodate vehicles?


Environment Editor - Frustrating to Use?

This is more of a personal issue, but compared to older versions of the editor, the environment editor is much more frustrating to use.
I'd like to see it partially return to the old system, where you can edit it from anywhere inside or outside of editor mode (when in SP only?), and for it to stop resetting things like the various shadow slice, etc. settings on startup.  Additionally, I'd also like it to support manually typing in values that go past the default limit (the ingame one currently supports up to 10k for shadowslices?  but ovk's envs use easily much more than that, fog values do this as well).
I'd also recommend making whatever panel it uses a very low opacity, or having a button to temporarily hide it while performing edits.  Past that, though, I do quite like the current rendition of it.

the new editor is based of the actual editor so no can do, the old one was extremely buggy and had many missing features.
I'll see if it reset the values and comment here again about it.
I can always increase any values if it's needed.
I already planned to add something to hide the editor screens

Currently, I've been messing around with the shadows, and it seems there's a lot of oddities with how each value behaves, and there should be different max/min limits..?  I need to test more to see what these could possibly be.
Also, ambient scale appears to be non-functional in editor.

Not really sure what you mean by that but I'd prefer you to open a new issue for that


[Bug]: Skip current sql statement does not take effects

ODC version
4.3.2
OB version
no related
What happened?

In the task details, the system does not respond, and the second batch of SQL statements is not executed
What did you expect to happen?
The skip function takes effect and the second batch of SQL statements is executed
How can we reproduce it (as minimally and precisely as possible)?
In the task details, the system does not respond, and the second batch of SQL statements is not executed
Anything else we need to know?
No response

Fixed In Commit fd586b4208b852f5ccd57358b87b22f29cb8d2c5
------
畅晚
Thu Sep 19 15:00:51 2024 +0800

pass


openssl-1.1.1: possible memory leak

when debugging "nginx + openssl + gost-engine" using valgrind, I've found some things likely not related to gost-engine, but related to openssl itself:
==12862== 8 bytes in 1 blocks are possibly lost in loss record 1 of 348
==12862==    at 0x4C29BC3: malloc (vg_replace_malloc.c:299)
==12862==    by 0x606D48D: CRYPTO_zalloc (mem.c:230)
==12862==    by 0x5FD3956: CTLOG_STORE_new (ct_log.c:94)
==12862==    by 0x5C99241: SSL_CTX_new (ssl_lib.c:2946)
==12862==    by 0x44EE99: ngx_ssl_create (ngx_event_openssl.c:253)
==12862==    by 0x4B6008: ngx_http_proxy_set_ssl (ngx_http_proxy_module.c:4265)
==12862==    by 0x4B6008: ngx_http_proxy_merge_loc_conf (ngx_http_proxy_module.c:3231)
==12862==    by 0x456FC2: ngx_http_merge_locations (ngx_http.c:648)
==12862==    by 0x457E57: ngx_http_merge_servers (ngx_http.c:605)
==12862==    by 0x457E57: ngx_http_block (ngx_http.c:268)
==12862==    by 0x432915: ngx_conf_handler (ngx_conf_file.c:463)
==12862==    by 0x432915: ngx_conf_parse (ngx_conf_file.c:319)
==12862==    by 0x42FD69: ngx_init_cycle (ngx_cycle.c:275)
==12862==    by 0x41C5B6: main (nginx.c:291)

another one:
==12862== 32 bytes in 1 blocks are still reachable in loss record 3 of 348
==12862==    at 0x4C2B955: calloc (vg_replace_malloc.c:711)
==12862==    by 0x4E3754F: _dlerror_run (dlerror.c:141)
==12862==    by 0x4E36F80: dlopen@@GLIBC_2.2.5 (dlopen.c:87)
==12862==    by 0x5FE76B4: dlfcn_load (dso_dlfcn.c:117)
==12862==    by 0x5FE7FD3: DSO_load (dso_lib.c:161)
==12862==    by 0x60417C1: int_load (eng_dyn.c:386)
==12862==    by 0x60417C1: dynamic_load (eng_dyn.c:413)
==12862==    by 0x60417C1: dynamic_ctrl (eng_dyn.c:336)
==12862==    by 0x6041322: ENGINE_ctrl_cmd_string (eng_ctrl.c:327)
==12862==    by 0x604323E: ENGINE_by_id (eng_list.c:328)
==12862==    by 0x6040A91: int_engine_configure (eng_cnf.c:100)
==12862==    by 0x6040A91: int_engine_module_init (eng_cnf.c:171)
==12862==    by 0x5FD21F4: module_init (conf_mod.c:312)
==12862==    by 0x5FD21F4: module_run (conf_mod.c:171)
==12862==    by 0x5FD21F4: CONF_modules_load (conf_mod.c:102)
==12862==    by 0x5FD27A6: CONF_modules_load_file (conf_mod.c:138)
==12862==    by 0x5FD2A1F: openssl_config_int (conf_sap.c:68)

I'm pretty sure that SSL_CTX_free() calls CTLOG_STORE_free(), so I
don't see how the leak can be in OpenSSL.

It's not because OpenSSL allocates something, that it's OpenSSL
that is leaking things.

Ilya Shipitsin wrote:

another one:
==12862== 32 bytes in 1 blocks are still reachable in loss record 3 of 348
==12862==    at 0x4C2B955: calloc (vg_replace_malloc.c:711)
==12862==    by 0x4E3754F: _dlerror_run (dlerror.c:141)
==12862==    by 0x4E36F80: dlopen@@GLIBC_2.2.5 (dlopen.c:87)
==12862==    by 0x5FE76B4: dlfcn_load (dso_dlfcn.c:117)
==12862==    by 0x5FE7FD3: DSO_load (dso_lib.c:161)
[SNIP]

Tracing an engine with valgrind was tricky. There is(was) limitation
in valgrind and the only way to get a proper report was to comment line
"dlclose(ptr);" in function dlfcn_unload. This produce report like above
but allows me to investigate other leaks in engine.

If this is not the case, I guess that engine or engine key is still
in use.
I don't know code of reported application. In PKIX-SSH I use code that
explicitly "shutdown" loaded engines:

static void
ssh_engine_reset(const char *engine) {
    ENGINE *e = NULL;
e = ENGINE_by_id(engine);
    if (e == NULL) return;
/* decrease functional&structural reference from load - ENGINE_init */
    ENGINE_finish(e);
/* decrease structural reference from above - ENGINE_by_id */
    ENGINE_free(e);
}
So I guess that reported application lack ENGINE_finish(e);
Remark: an engine could be loaded and initialized by openssl configuration.
Roumen

application is an nginx, it does not have an idea about engines. engine is loaded by openssl itself
# cat /etc/openssl-micro/openssl.cnf
openssl_conf = openssl_def

[openssl_def]
engines = engine_section

[engine_section]
gost = gost_section

[gost_section]
default_algorithms = ALL
engine_id = gost
CRYPT_PARAMS = id-Gost28147-89-CryptoPro-A-ParamSet

I'm not sure app should explicitely care about engines.  shouldn't it be released via SSL_CTX_free() ?

I would like to config leaks in DSO load.
Test command is "openssl genpkey -engine gost -algorithm gost2001 -pkeyopt paramset:A", where openssl is build from 1.1.1_stable branch installed, gost engine is build from master branch but not yet installed.

OPENSSL_ENGINES set to gost build, default configuration.        32 bytes in 1 blocks are still reachable in loss record 2 of 2
32 bytes in 1 blocks are still reachable in loss record 2 of 2
at 0x4C2EC50: calloc (vg_replace_malloc.c:623)
by 0x55BE626: dlerror_run (in /lib64/libdl-2.23.so)
by 0x55BDFA0: dlopen@@GLIBC_2.2.5 (in /lib64/libdl-2.23.so)
by 0x51CE465: dlfcn_load (dso_dlfcn.c:117)
by 0x51CEEC3: DSO_load (dso_lib.c:161)
by 0x51B556D: COMP_zlib (c_zlib.c:226)
by 0x4E69A4A: do_load_builtin_compressions (ssl_ciph.c:461)
by 0x4E69A4A: do_load_builtin_compressions_ossl (ssl_ciph.c:458)
by 0x57CFAA8: _pthread_once_slow (in /lib64/libpthread-2.23.so)
by 0x52CAE08: CRYPTO_THREAD_run_once (threads_pthread.c:113)
by 0x4E6C2C6: load_builtin_compressions (ssl_ciph.c:482)
by 0x4E6C2C6: SSL_COMP_get_compression_methods (ssl_ciph.c:1936)
by 0x4E6ED13: ossl_init_ssl_base (ssl_init.c:99)
by 0x4E6ED13: ossl_init_ssl_base_ossl (ssl_init.c:24)
by 0x57CFAA8: __pthread_once_slow (in /lib64/libpthread-2.23.so)

2.1) custom configuration (OPENSSL_CONF set) with engine_id = gost and  OPENSSL_ENGINES set.
32 bytes in 1 blocks are still reachable in loss record 2 of 2
at 0x4C2EC50: calloc (vg_replace_malloc.c:623)
by 0x55BE626: _dlerror_run (in /lib64/libdl-2.23.so)
by 0x55BDFA0: dlopen@@GLIBC_2.2.5 (in /lib64/libdl-2.23.so)
by 0x51CE465: dlfcn_load (dso_dlfcn.c:117)
by 0x51CEEC3: DSO_load (dso_lib.c:161)
by 0x5224FF0: int_load (eng_dyn.c:386)
by 0x5224FF0: dynamic_load (eng_dyn.c:413)
by 0x5224FF0: dynamic_ctrl (eng_dyn.c:336)
by 0x5224C81: ENGINE_ctrl_cmd_string (eng_ctrl.c:327)
by 0x5226C61: ENGINE_by_id (eng_list.c:328)
by 0x522435B: int_engine_configure (eng_cnf.c:100)
by 0x522435B: int_engine_module_init (eng_cnf.c:171)
by 0x51B845D: module_init (conf_mod.c:312)
by 0x51B845D: module_run (conf_mod.c:171)
by 0x51B845D: CONF_modules_load (conf_mod.c:102)
by 0x51B8BE6: CONF_modules_load_file (conf_mod.c:138)
by 0x51B8E4A: openssl_config_int (conf_sap.c:69)
2.2) custom configuration (OPENSSL_CONF set) with dynamic_path, i.e without OPENSSL_ENGINES.
32 bytes in 1 blocks are still reachable in loss record 2 of 2
at 0x4C2EC50: calloc (vg_replace_malloc.c:623)
by 0x55BE626: dlerror_run (in /lib64/libdl-2.23.so)
by 0x55BDFA0: dlopen@@GLIBC_2.2.5 (in /lib64/libdl-2.23.so)
by 0x51CE465: dlfcn_load (dso_dlfcn.c:117)
by 0x51CEEC3: DSO_load (dso_lib.c:161)
by 0x52251EB: int_load (eng_dyn.c:374)
by 0x52251EB: dynamic_load (eng_dyn.c:413)
by 0x52251EB: dynamic_ctrl (eng_dyn.c:336)
by 0x5224C81: ENGINE_ctrl_cmd_string (eng_ctrl.c:327)
by 0x52241FB: int_engine_configure (eng_cnf.c:90)
by 0x52241FB: int_engine_module_init (eng_cnf.c:171)
by 0x51B845D: module_init (conf_mod.c:312)
by 0x51B845D: module_run (conf_mod.c:171)
by 0x51B845D: CONF_modules_load (conf_mod.c:102)
by 0x51B8BE6: CONF_modules_load_file (conf_mod.c:138)
by 0x51B8E4A: openssl_config_int (conf_sap.c:69)
by 0x5251EEF: ossl_init_config (init.c:293)
by 0x5251EEF: ossl_init_config_ossl (init.c:291)
Summary:

reports leak is dso load due to initialization of compression, no leak due to load of engine.
2.*) report leak in load of engine due to load from configuration.

Remark: initial report is for leak from SSL_CTX_new . This case is not tested yet.

@petrovr , is " due to initialization of compression" automatically tested ? I see a lot of auto tests in openssl travis-ci matrix
anyway, will it be fixed ?

I use the tls version of websocketpp, and have the same problem.


Frontend userauth

@codekansas can you look into this stuff:
api.interceptors.request.use(
      (config) => {
        config.headers.Authorization = `Bearer ${getLocalStorageApiKey()}`;
        return config;
      },
      (error) => {
        return Promise.reject(error);
      },
    );

whatever you did does not seem to work

Hm, there is an issue in general. Isaac's code turns out to have inconsistent models, in the sense that he cooked up his own types instead of using what was in Models. So the form is actually incomplete.

So we're more or less back where we started. The backend on its own works perfectly fine. The frontend is still buggy (albeit buggy for different reasons)...
I'm thinking we prioritize making everything solid before we make any more feature requests


Feat: Allow price per credit input to accept values with two decimal places - 996

This PR updates the price per credit input box to support values with two decimal places in the credit proposal form.
Closes #996

Look good 👍

Thanks, Alex, for rewiring my PR.


Generation of HTML report is failing, json is generated just right.

I'm trying with the following configurations :
licenseReport {
    generateHtmlReport = true
    // These options are ignored for Java projects
    copyHtmlReportToAssets = true
}

however I get the following error, with an empty HTMLReport file.
Execution failed for task ':app:licenseReleaseReport'.
> > Task :app:licenseReleaseReport
saimos-unit-api dependency does not have a license.
saimos-common dependency does not have a license.

> Task :app:licenseReleaseReport FAILED

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':app:licenseReleaseReport'.
> HtmlReport::class.java.g…File.separatorChar)
      ) must not be null

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

when I use the Json configuration, json is generated with no issues.

Had the same problem and using 0.8.7-SNAPSHOT seems to fix it. The build is successful and now I have both the HTML and JSON files.

I am struggling with the same issue. @vstanese I am unsure how yo use the 0.8.7-SNAPSHOT version?

Thanks @vstanese.
@Mostafayehya, 0.8.7 has been released.


Added a feature of visualization

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 94.50549% with 5 lines in your changes missing coverage. Please review.

Files with missing lines
Patch %
Lines

flask_dollarbot/tests/test_analytics_api.py
93.93%
2 Missing :warning:

flask_dollarbot/tests/test_display_api.py
93.93%
2 Missing :warning:

flask_dollarbot/endpoints/analytics.py
96.00%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Files with missing lines
Coverage Δ

flask_dollarbot/endpoints/analytics.py
96.00% <96.00%> (ø)

flask_dollarbot/tests/test_analytics_api.py
93.93% <93.93%> (ø)

flask_dollarbot/tests/test_display_api.py
93.93% <93.93%> (ø)

... and 1 file with indirect coverage changes

🚨 Try these New Features:

Flaky Tests Detection - Detect and resolve failed and flaky tests
JS Bundle Analysis - Avoid shipping oversized bundles


Update AttributedLib.podspec

Cocoapods-Rome requires SWIFT_VERSION in order to build the framework
otherwise it will raise error like this
[!] Unable to determine Swift version for the following pods:

- `AttributedLib` does not specify a Swift version and none of the targets (`App`) integrating it have the `SWIFT_VERSION` attribute set. Please contact the author or set the `SWIFT_VERSION` attribute in at least one of the targets that integrate this pod.

Oh, good work! LGTM!


Match Id Multi  for Autocomplete is not showing proper code

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Summary 💡
Match Id Multi  for Autocomplete is not showing proper code as it is not showing how to set the value for the selected items
Examples 🌈
No response

are you looking for an example? https://react-hook-form-material-ui.vercel.app/?path=/docs/autocomplete--docs#match-id-multi
I quite dont understand what you are looking for.

Dear Dominic,
The match-id option is a boolean only right?
What I wanted was , how can i preselect multiple options when the
autocomplete loads up.
On Tue, Jun 20, 2023 at 4:12 AM Dominic Garms @.***>
wrote:

are you looking for an example?
https://react-hook-form-material-ui.vercel.app/?path=/docs/autocomplete--docs#match-id-multi
I quite dont understand what you are looking for.
—
Reply to this email directly, view it on GitHub
https://github.com/dohomi/react-hook-form-mui/issues/170#issuecomment-1597935101,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABPKPSGZSYNEESIEJLXB263XMD2F7ANCNFSM6AAAAAAZKHKIBY
.
You are receiving this because you authored the thread.Message ID:
@.***>

you do this on the react-hook-form with defaultValues:
https://github.com/dohomi/react-hook-form-mui/blob/master/apps/storybook/stories/AutocompleteElement.stories.tsx#L24

Can we do this with react-hook-form-mui
On Thu, Jun 22, 2023, 4:18 AM Dominic Garms @.***>
wrote:

you do this on the react-hook-form with defaultValues:
https://github.com/dohomi/react-hook-form-mui/blob/master/apps/storybook/stories/AutocompleteElement.stories.tsx#L24
—
Reply to this email directly, view it on GitHub
https://github.com/dohomi/react-hook-form-mui/issues/170#issuecomment-1601885644,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABPKPSHSOYCBFQURNEIBR5TXMOMNZANCNFSM6AAAAAAZKHKIBY
.
You are receiving this because you authored the thread.Message ID:
@.***>

what is your question?


Plugins not updating

When adding a plugin to the plugins folder on root, plugins do not change even after rebuilding the image and restarting the server.

Note that plugins on local-runner should be unzipped.  Please also be sure to use the latest (2.2.2) version.


Error message "400 / Bad Request / Internal Engine Error" when using backup.py on some VMs

When using backup.py on some(!) VMs, we get a "Bad Request / Internal Engine Error" during the cloning step:
$ ./backup.py -c ./config_testvm.cfg -d
Apr 12 10:50:05: Start backup for: testvm
Apr 12 10:50:24: Snapshot creation started ...
Apr 12 10:50:32: Snapshot operation(creation) in progress ...
Apr 12 10:50:41: Snapshot operation(creation) in progress ...
Apr 12 10:50:51: Snapshot operation(creation) in progress ...
Apr 12 10:51:00: Snapshot created
Apr 12 10:51:04: Clone into VM started ...
Apr 12 10:51:07: !!! Got a RequestError:
status: 400
reason: Bad Request
detail: Internal Engine Error
Apr 12 10:51:07: All backups done
Apr 12 10:51:07: Backup failured for:
Apr 12 10:51:07:   testvm
Apr 12 10:51:07: Some errors occured during the backup, please check the
log file
$

The used config file looks as follows:
$ cat config_testvm.cfg
[config]
vm_names: ["testvm"]
vm_middle=_BACKUP
snapshot_description=Snapshot for backup script
server=https://our.ovirt.server/ovirt-engine/api
username=xxxx@xxxx
password=xxxx
export_domain=our_export_domain
timeout=5
cluster_name=Default
backup_keep_count=3
dry_run=False
vm_name_max_length=32
storage_domain=our_storage_domain
storage_space_threshold=0.1
$

The same script with only the VM name changed in the configuration file works fine on other VMs.  When running the process for the failing VM manually through oVirt's admin webgui, then no error appears.
There is no obvious configuration difference between working and failing VMs.  The backup script is run directly on the oVirt server (our.ovirt.server).
System and oVirt version information:
$ cat /etc/os-release
NAME="CentOS Linux"
VERSION="7 (Core)"
ID="centos"
ID_LIKE="rhel fedora"
VERSION_ID="7"
PRETTY_NAME="CentOS Linux 7 (Core)"
ANSI_COLOR="0;31"
CPE_NAME="cpe:/o:centos:centos:7"
HOME_URL="https://www.centos.org/"
BUG_REPORT_URL="https://bugs.centos.org/"

CENTOS_MANTISBT_PROJECT="CentOS-7"
CENTOS_MANTISBT_PROJECT_VERSION="7"
REDHAT_SUPPORT_PRODUCT="centos"
REDHAT_SUPPORT_PRODUCT_VERSION="7"

$ rpm -qa | grep ovirt
ovirt-engine-restapi-<IP_ADDRESS>-1.el7.centos.noarch
ovirt-hosted-engine-ha-1.2.8-1.el7.centos.noarch
ovirt-release35-006-1.noarch
ovirt-engine-setup-plugin-websocket-proxy-<IP_ADDRESS>-1.el7.centos.noarch
ovirt-engine-extensions-api-impl-<IP_ADDRESS>-1.el7.centos.noarch
ovirt-engine-sdk-python-<IP_ADDRESS>-1.el7.noarch
ovirt-image-uploader-3.5.1-1.el7.centos.noarch
ovirt-engine-backend-<IP_ADDRESS>-1.el7.centos.noarch
ovirt-engine-userportal-<IP_ADDRESS>-1.el7.centos.noarch
ovirt-engine-setup-base-<IP_ADDRESS>-1.el7.centos.noarch
ovirt-engine-jboss-as-7.1.1-1.el7.x86_64
ovirt-engine-setup-<IP_ADDRESS>-1.el7.centos.noarch
ovirt-engine-dbscripts-<IP_ADDRESS>-1.el7.centos.noarch
ovirt-engine-cli-<IP_ADDRESS>-1.el7.centos.noarch
ovirt-engine-<IP_ADDRESS>-1.el7.centos.noarch
ovirt-engine-setup-plugin-ovirt-engine-common-<IP_ADDRESS>-1.el7.centos.noarch
ovirt-host-deploy-1.3.2-1.el7.centos.noarch
ovirt-engine-setup-plugin-ovirt-engine-<IP_ADDRESS>-1.el7.centos.noarch
ovirt-engine-tools-<IP_ADDRESS>-1.el7.centos.noarch
ovirt-iso-uploader-3.5.2-1.el7.centos.noarch
ovirt-host-deploy-java-1.3.2-1.el7.centos.noarch
ovirt-engine-webadmin-portal-<IP_ADDRESS>-1.el7.centos.noarch
libgovirt-0.3.3-1.el7.x86_64
ovirt-hosted-engine-setup-<IP_ADDRESS>-1.el7.centos.noarch
ovirt-engine-lib-<IP_ADDRESS>-1.el7.centos.noarch
ovirt-engine-websocket-proxy-<IP_ADDRESS>-1.el7.centos.noarch
$

Is there a way to further debug this kind of problem?  Like a -vvv option?  Looking at the ovirt-engine log on the server doesn't really help, as the relevant messages are mixed with hundreds of other messages being printed during normal operation of the oVirt engine.  In the concrete case it would e.g. be helpful to track down the concrete request (with data), which is failing.  We are currently using commit 394a8743fc59a9bb7b4a131b133aa992fb3a1f30 of oVirtBackup
$ git rev-parse HEAD
394a8743fc59a9bb7b4a131b133aa992fb3a1f30
$

Thanks in advance
Frank Thommen

Sorry, I forgot to document system type and oVirt versions:
$ cat /etc/os-release
NAME="CentOS Linux"
VERSION="7 (Core)"
ID="centos"
ID_LIKE="rhel fedora"
VERSION_ID="7"
PRETTY_NAME="CentOS Linux 7 (Core)"
ANSI_COLOR="0;31"
CPE_NAME="cpe:/o:centos:centos:7"
HOME_URL="https://www.centos.org/"
BUG_REPORT_URL="https://bugs.centos.org/"

CENTOS_MANTISBT_PROJECT="CentOS-7"
CENTOS_MANTISBT_PROJECT_VERSION="7"
REDHAT_SUPPORT_PRODUCT="centos"
REDHAT_SUPPORT_PRODUCT_VERSION="7"

$ rpm -qa | grep ovirt
ovirt-engine-restapi-<IP_ADDRESS>-1.el7.centos.noarch
ovirt-hosted-engine-ha-1.2.8-1.el7.centos.noarch
ovirt-release35-006-1.noarch
ovirt-engine-setup-plugin-websocket-proxy-<IP_ADDRESS>-1.el7.centos.noarch
ovirt-engine-extensions-api-impl-<IP_ADDRESS>-1.el7.centos.noarch
ovirt-engine-sdk-python-<IP_ADDRESS>-1.el7.noarch
ovirt-image-uploader-3.5.1-1.el7.centos.noarch
ovirt-engine-backend-<IP_ADDRESS>-1.el7.centos.noarch
ovirt-engine-userportal-<IP_ADDRESS>-1.el7.centos.noarch
ovirt-engine-setup-base-<IP_ADDRESS>-1.el7.centos.noarch
ovirt-engine-jboss-as-7.1.1-1.el7.x86_64
ovirt-engine-setup-<IP_ADDRESS>-1.el7.centos.noarch
ovirt-engine-dbscripts-<IP_ADDRESS>-1.el7.centos.noarch
ovirt-engine-cli-<IP_ADDRESS>-1.el7.centos.noarch
ovirt-engine-<IP_ADDRESS>-1.el7.centos.noarch
ovirt-engine-setup-plugin-ovirt-engine-common-<IP_ADDRESS>-1.el7.centos.noarch
ovirt-host-deploy-1.3.2-1.el7.centos.noarch
ovirt-engine-setup-plugin-ovirt-engine-<IP_ADDRESS>-1.el7.centos.noarch
ovirt-engine-tools-<IP_ADDRESS>-1.el7.centos.noarch
ovirt-iso-uploader-3.5.2-1.el7.centos.noarch
ovirt-host-deploy-java-1.3.2-1.el7.centos.noarch
ovirt-engine-webadmin-portal-<IP_ADDRESS>-1.el7.centos.noarch
libgovirt-0.3.3-1.el7.x86_64
ovirt-hosted-engine-setup-<IP_ADDRESS>-1.el7.centos.noarch
ovirt-engine-lib-<IP_ADDRESS>-1.el7.centos.noarch
ovirt-engine-websocket-proxy-<IP_ADDRESS>-1.el7.centos.noarch
$

ft.

More debugging is not easy with the SDK/API, the exception which is given from the SDK is currently displayed, much more is not possible I think.
But I made a branch with some changes and added a traceback. Can you checkout the branch "issue_16" and post the output here. Check first if it doesn't contain sensitive information's !!! ;-)
If this helps, I will add a more debug output as you suggested. Sound's like a good idea.

With this version I get
$ ./backup.py -c ./config_testvm.cfg -d
Apr 14 16:04:12: Start backup for: testvm
Apr 14 16:04:30: Found snapshots(1):
Apr 14 16:04:30: Snapshots description: Snapshot for backup script, Created on: 2016-04-12 13:15:34.983000+02:00
Apr 14 16:04:32: Snapshot deletion started ...
Apr 14 16:04:36: Snapshot operation(deletion) in progress ...
Apr 14 16:04:45: Snapshot operation(deletion) in progress ...
Apr 14 16:04:55: Snapshot operation(deletion) in progress ...
Apr 14 16:05:04: Snapshot operation(deletion) in progress ...
Apr 14 16:05:13: Snapshot operation(deletion) in progress ...
Apr 14 16:05:22: Snapshot operation(deletion) in progress ...
Apr 14 16:05:31: Snapshot operation(deletion) in progress ...
Apr 14 16:05:39: Snapshot operation(deletion) in progress ...
Apr 14 16:05:48: Snapshot operation(deletion) in progress ...
Apr 14 16:05:57: Snapshot operation(deletion) in progress ...
Apr 14 16:06:06: Snapshot operation(deletion) in progress ...
Apr 14 16:06:15: Snapshot operation(deletion) in progress ...
Apr 14 16:06:24: Snapshot operation(deletion) in progress ...
Apr 14 16:06:32: Snapshot operation(deletion) in progress ...
Apr 14 16:06:40: Snapshots deleted
Apr 14 16:06:41: Snapshot creation started ...
Apr 14 16:06:49: Snapshot operation(creation) in progress ...
Apr 14 16:06:58: Snapshot operation(creation) in progress ...
Apr 14 16:07:06: Snapshot created
Apr 14 16:07:20: Clone into VM started ...
Apr 14 16:07:23: !!! Got a RequestError:
status: 400
reason: Bad Request
detail: Internal Engine Error
Traceback (most recent call last):
  File "./backup.py", line 127, in main
    api.vms.add(params.VM(name=vm_clone_name, memory=vm.get_memory(), cluster=api.clusters.get(config.get_cluster_name()), snapshots=snapshots_param))
  File "/usr/lib/python2.7/site-packages/ovirtsdk/infrastructure/brokers.py", line 35196, in add
    headers={"Correlation-Id":correlation_id, "Expect":expect}
  File "/usr/lib/python2.7/site-packages/ovirtsdk/infrastructure/proxy.py", line 75, in add
    return self.request('POST', url, body, headers)
  File "/usr/lib/python2.7/site-packages/ovirtsdk/infrastructure/proxy.py", line 115, in request
    persistent_auth=self.__persistent_auth
  File "/usr/lib/python2.7/site-packages/ovirtsdk/infrastructure/connectionspool.py", line 79, in do_request
    persistent_auth)
  File "/usr/lib/python2.7/site-packages/ovirtsdk/infrastructure/connectionspool.py", line 156, in __do_request
    raise errors.RequestError(response_code, response_reason, response_body)
RequestError:
status: 400
reason: Bad Request
detail: Internal Engine Error
Apr 14 16:07:23: All backups done
Apr 14 16:07:23: Backup failured for:
Apr 14 16:07:23:   testvm
Apr 14 16:07:23: Some errors occured during the backup, please check the log file
$

The configuration is the same as before with an added use_short_suffix = false.  I have problems interpreting this traceback

Hi, its not really clear right now. So I wrote some check's to see if the config is correct. Can you checkout the latest changes on the branch "issue_16" and see if it prints some errors about missing config settings for export_domain, cluster_name and storage_domain. (I will refactor this later)
commit: 3e244711183976cec4faef4857f9474aaa00fcda

I can't see a big difference:
$ ./backup.py -c ./config_testvm.cfg -d
Apr 15 11:09:06: Start backup for: testvm
Apr 15 11:09:24: Found snapshots(1):
Apr 15 11:09:24: Snapshots description: Snapshot for backup script, Created on: 2016-04-14 16:06:43.088000+02:00
Apr 15 11:09:26: Snapshot deletion started ...
Apr 15 11:09:30: Snapshot operation(deletion) in progress ...
Apr 15 11:09:39: Snapshot operation(deletion) in progress ...
Apr 15 11:09:48: Snapshot operation(deletion) in progress ...
Apr 15 11:09:58: Snapshot operation(deletion) in progress ...
Apr 15 11:10:07: Snapshot operation(deletion) in progress ...
Apr 15 11:10:16: Snapshot operation(deletion) in progress ...
Apr 15 11:10:26: Snapshot operation(deletion) in progress ...
Apr 15 11:10:35: Snapshot operation(deletion) in progress ...
Apr 15 11:10:44: Snapshot operation(deletion) in progress ...
Apr 15 11:10:52: Snapshot operation(deletion) in progress ...
Apr 15 11:11:01: Snapshot operation(deletion) in progress ...
Apr 15 11:11:09: Snapshot operation(deletion) in progress ...
Apr 15 11:11:18: Snapshot operation(deletion) in progress ...
Apr 15 11:11:27: Snapshot operation(deletion) in progress ...
Apr 15 11:11:35: Snapshots deleted
Apr 15 11:11:36: Snapshot creation started ...
Apr 15 11:11:43: Snapshot operation(creation) in progress ...
Apr 15 11:11:52: Snapshot operation(creation) in progress ...
Apr 15 11:12:01: Snapshot created
Apr 15 11:12:15: Clone into VM started ...
Apr 15 11:12:17: !!! Got a RequestError:
status: 400
reason: Bad Request
detail: Internal Engine Error
Traceback (most recent call last):
  File "./backup.py", line 145, in main
    api.vms.add(params.VM(name=vm_clone_name, memory=vm.get_memory(), cluster=api.clusters.get(config.get_cluster_name()), snapshots=snapshots_param))
  File "/usr/lib/python2.7/site-packages/ovirtsdk/infrastructure/brokers.py", line 35196, in add
    headers={"Correlation-Id":correlation_id, "Expect":expect}
  File "/usr/lib/python2.7/site-packages/ovirtsdk/infrastructure/proxy.py", line 75, in add
    return self.request('POST', url, body, headers)
  File "/usr/lib/python2.7/site-packages/ovirtsdk/infrastructure/proxy.py", line 115, in request
    persistent_auth=self.__persistent_auth
  File "/usr/lib/python2.7/site-packages/ovirtsdk/infrastructure/connectionspool.py", line 79, in do_request
    persistent_auth)
  File "/usr/lib/python2.7/site-packages/ovirtsdk/infrastructure/connectionspool.py", line 156, in __do_request
    raise errors.RequestError(response_code, response_reason, response_body)
RequestError:
status: 400
reason: Bad Request
detail: Internal Engine Error
Apr 15 11:12:17: All backups done
Apr 15 11:12:17: Backup failured for:
Apr 15 11:12:17:   testvm
Apr 15 11:12:17: Some errors occured during the backup, please check the log file
$

Can you say from this traceback, which step/function fails?

I see a difference :-) It means that the the values are correct.
Can you please attach the engine.log which is generated during the backup.

It is impossible for me to see, which entries in the 40k-line logfile came from the backup job :-).  I'll rerun backup.py to generate new entries and filter them out as they are being generated.  However I'd prefer not to post them publicly.  Is it ok if I send them to you personally?  Do you want me to also send you the engine.log entries for a VM where backup.py runs fine?

Yes you can send it to me, I will handle it with care and don't publish sensitive Information's and wipe it after we find the problem.
Can you also create a empty third test VM with a disk of 1GB without a Operating System and run the backup on it. Maybe the VM has a strange problem.
I had this before on a test network with a VM where I hit some official Bug's (again) which where already reported to oVirt. There was the problem that a snapshot was generate but the snapshot disk was, for unknown reasons, corrupt and the backup-script did the job but the export VM has no disks stored. During the debugging I saw that all the SDK methods report no errors but there where thousand entries in the engine.log which are not forwarded through the SDK.
In short, currently I'm sceptical since I switched from VMWare ESXi to oVirt, before I had never problems. But this is another story :-(

Logs and output are on the way to your mailbox.  As for the minimal test-VM, I'll only be able to do this early next week.  I'll then send you it's backup output as soon as it is available.

Seems to have been a problem with some VMs themselves which could also not exported and imported properly through the oVirt WebUI.  After rebuilding the VMs from scratch, the backup problems have gone.  Thanks a lot for all your help.


Added GamePad Management Classes, Will Need Iterations

Wanted to Use a GamePad with my game, so I added a manager class for each Game Pad to use and an overall manager's class to manage Setting up the GamePads. It's kinda rough, but ready for feedback.

First PR, thanks! Just looking in the code it seems GamePadMan doesn't need to extends Manager, because it is already updated by GamePads. But I will test the code here.

I was debating that myself, I liked the idea of the GamePadMan being able to be used independantly, but I don't know if that is actually useful. I think if manager were an interface it'd make more sense. As a class though if you were to add functionality to Manager I could see unintended consequences.
I'll rip the extension out for the time being since it doesn't net anything anyway outside of the active bool (which isn't in use anyway).

I couldn't test yet, but the notification code for the gamepad events is inside GamePads.hx, so GamePadMan will not work alone without the GamePads class. And even if the notification code were inside GamePadMan, the user would have to add one object in the engine per gamepad used, which is unnecessary. We can add a method "get(index:Int):GamePadMan" in GamePads.hx to have a "nice" way to get the gamepads.
I will change Manager.hx to a interface, it doesn't need to be a class, because its just a way to the input methods to be updated by the engine and all the implementation is made in the extended classes.


Several packages fail to build on arm-windows

I tried to build some software for arm-windows and I found that some of the required libraries cannot be compiled. At the moment, these are the ones that I have found, but probably there are also others:
curl
fluidsynth
gettext
glib
libffi
mpir
openal-soft
openssl (see also #2971)
pcre
sdl2
sdl2-net

libffi still fails to build with arm-windows.  Any updates?

curl fluidsynth libffi sdl2 sdl2-net
at least those packages build fine on arm64-windows now, no idea about arm-windows
Others likely build too already.

This issue hasn’t been updated more than 3 months, if it is still an issue, please reopen this issue.


Add example pre/post content sections to json file

As discussed in this issue: https://github.com/gigsboat/cli/issues/14
This will show folks using the template repository as the basis for their own repository that it is possible to add these sections.

Yes, totally needed!


fix(jsx/dom): fix cleanup for deps

There was a problem that cleanup was not called when changing "deps", so this has been fixed.
Author should do the followings, if applicable

[x] Add tests
[x] Run tests
[x] yarn denoify to generate files for Deno

@yusukebe, Would you please review?
Looking at the unit tests, you will understand what has been fixed.


divide by zero using influx CLI on pod

In my attempts to set a retention policy for the influxdb, I kept getting a divide by zero error using the CLI on the pod. I traced it down to not having a column width set.
stty cols 80
fixes the issue.

Closing this as I am not sure if this is a problem with how we configure the pod to start but how kubectl exec works.


Changing the C (clear) button functioning, it is not clearing the all the fields

In the calculator design we have to click on clear button to clear the input field.
And click on backspace button to clear the output field.
Which is creating problem for first time using user.
Would like to make changes to that?
Please assign this issue to me.
Thank You!!!

I am GSSoC'21 participant. Please assign this to me.

Yeah! I was thinking off the same problem, you catch it right.
Can you make backspace button to work as clear button also, i.e if pressed long for 2 seconds then it must clear fields, otherwise serve a backspace functionality.

Yup I can, thank you!
Can you add GSSoC'21 label to it?

Feel free to ask your doubts, if any
Deadline 15 March,2021 (11:59 PM IST)

Feel free to ask your doubts, if any
Deadline 15 March,2021 (11:59 PM IST)

Thank You!!!

I'm a GSSOC'21 participant kindly assign me this issue. I would love to work in this.

Hey @sairish2001 , I am done with this but I think it will be efficient if it clears when pressed for 1 sec.
What do you think?

yes! fine

Okay!!


Allow Disabling Pipenv Check

When I start VSCode with the vscode-python extension I get the warning:
Workspace contains pipfile but attempt to run 'pipenv --venv' failed

I don't need pipenv support and my venv is inaccessible within a virtual machine anyway. It would be nice if there were a setting to prevent vscode-python from trying to run pipenv even if there is a Pipfile in the workspace.

Do you want to take this on, @ericsnowcurrently , since I know you have talked about this?

Shouldn't we removing the needs decision label? Or do we need to decide on the UX?

This only seems to be an issue with Python 2 projects. VSCode doesn't recocgnise the virtual environment setting even if you specify them in your setting eg.
{
  "python.venvPath": "/Users/username/.local/share/virtualenvs/simple-enctyption-service-zyo_b2Lm/bin/python"
}

In Python 3 project this doesn;t seem to be an issue. I think this should be raised as a bug.

@sliedig please open a separate issue as what you're reporting is different than what this issue is about.

Not sure if this should be a separate issue, but this check failing seems to result in the extension giving up on running pylint over my code, even though I have python.pythonPath set to a suitably configured environment.
The reason I'm not sure if it should be a separate issue is because I think "Disable pipenv for the affected workspace" would be a perfectly acceptable answer (it's only broken because I haven't migrated the working directory into WSL yet, so editing the code from the Windows side works, and pylint mostly works, but the full environment won't come up as it includes Linux-specific bits)

@ncoghlan yep, it's a separate issue. 😄 Please file one so we can flag it as a bug and look into it.


fix: update exercise tracker test endpoints and test text

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the main branch of freeCodeCamp.
[x] All the files I changed are in the same world language, for example: only English changes, or only Chinese changes, etc.

Partially addresses https://github.com/freeCodeCamp/freeCodeCamp/issues/41529

This PR changes the test strings and test text for the URL Shortener project.
This needs to be tested and merged with #XXXXX in the Demo Projects repo.
Here is a link to a live version of the project with the proposed changes in #XXXXX: https://exercise-tracker-update.scissorsneedfoo.repl.co

Just to mention: Whilst testing, I noticed the example Exercise Tracker app does not pass the 3rd test (current tests, with current app):

A quick note that the third test times out on the new project, though a manual check of the functionality suggests it is working.
I think we just have too much data in the database and it takes too long to query it all 😅

Thanks for your thorough review of this @ShaunSHamilton and @nhcarrigan. I noticed that during testing too. It probably doesn't help that the db is filled with test users from all the tests I ran :slightly_smiling_face:

It probably wouldn't hurt to dump that collection, honestly. I'll take a peek at the data in there tomorrow and see what everyone thinks about cleaning it.

I'll take a peek at the data in there tomorrow and see what everyone thinks about cleaning it.

We should purge and re-seed all the data on the collections used for demo-projects. This should ideally be done on a daily basis with a CRON.

We should purge and re-seed all the data on the collections used for demo-projects. This should ideally be done on a daily basis with a CRON.

With the exception of the URL shortener - IIRC Quincy wanted to preserve that data in case folks are actually using it.


tsc does not resolve non-hoisted types by type reference directive when compilerOptions.paths is set

Bug Report
In the projects having non-hoisted types we got tsc resolver for type reference directive working differently with or without compilerOptions.paths causing failure in resolution in case we have it set.
The example project is attached.
Found in tsc 4.9.4, checked also in 4.9.5, beta (5.0.0-beta)
Minimal test-case
To have types out of node_modules/@types/ I am using use --public-hoist-pattern "" for pnpm, this is important on trying to reproduce.
Test case:

Use the test-case from this repository
Install dependencies with the exact command:
pnpm i --frozen-lockfile --public-hoist-pattern ""
Run tsc, it will fail to resolve type-referene directive 'express-serve-static-core'
To get tsc complete successfully remove paths from tsconfig.json and run tsc again.

Detailed description
// Example project tree:
├── node_modules
│   ├── express -><EMAIL_ADDRESS>│   └── @types
│       ├── express -> ../../.pnpm-v-store/@types/express@4.17.17/node_modules/@types/express
│       └── node -> ../../.pnpm-v-store/@types/node@18.15.3/node_modules/@types/node
├── package.json
├── pnpm-lock.yaml
├── src
│   ├── a.ts
└── tsconfig.json

// tsconfig.json:
{
    "compilerOptions": {
        "allowJs": true,
        "target": "es6",
        "strict": true,
        "sourceMap": true,
        "skipLibCheck": true,
        "esModuleInterop": true,
        "moduleResolution": "node",
	"baseUrl": "node_modules",
        "paths": { "*": [ "*", "./@types/*" ] }
    },
    "include": [ "src/**/*" ]
}

Resolve logs are a bit different in general as with paths tsc starts to resolve symlinks, but this doesn't look as a root cause, because it is still able to see the directories in both cases, but the main issue is it just skips search for {module}/package.json in case we have paths:
// Resolve log for express-serve-static-core with paths set in tsconfig.json:
======== Resolving type reference directive 'express-serve-static-core', containing file<EMAIL_ADDRESS>root directory '/tmp/tsc_test1/projA/node_modules/@types'. ========
Resolving with primary search path '/tmp/tsc_test1/projA/node_modules/@types'.
Looking up in 'node_modules' folder, initial location '/tmp/tsc_test1/projA/node_modules/@types/express'.
Directory '/tmp/tsc_test1/projA/node_modules/@types/express/node_modules' does not exist, skipping all lookups in it.
Directory '/tmp/tsc_test1/projA/node_modules/@types/node_modules' does not exist, skipping all lookups in it.
File '/tmp/tsc_test1/projA/node_modules/express-serve-static-core.d.ts' does not exist.
File<EMAIL_ADDRESS>does not exist.
Directory '/tmp/tsc_test1/node_modules' does not exist, skipping all lookups in it.
Directory '/tmp/node_modules' does not exist, skipping all lookups in it.
Directory '/node_modules' does not exist, skipping all lookups in it.
======== Type reference directive 'express-serve-static-core' was not resolved. ========

// Resolve log for express-serve-static-core without paths set in tsconfig.json:
======== Resolving type reference directive 'express-serve-static-core', containing file '/store_path_replaced/.pnpm-v-store/@types/express<EMAIL_ADDRESS>root directory '/tmp/tsc_test1/projA/node_modules/@types'. ========
Resolving with primary search path '/tmp/tsc_test1/projA/node_modules/@types'.
Looking up in 'node_modules' folder, initial location '/store_path_replaced/.pnpm-v-store/@types/express@4.17.17/node_modules/@types/express'.
Directory '/store_path_replaced/.pnpm-v-store/@types/express@4.17.17/node_modules/@types/express/node_modules' does not exist, skipping all lookups in it.
Directory '/store_path_replaced/.pnpm-v-store/@types/express@4.17.17/node_modules/@types/node_modules' does not exist, skipping all lookups in it.
File '/store_path_replaced/.pnpm-v-store/<EMAIL_ADDRESS>does not exist.
Found 'package.json' at '/store_path_replaced/.pnpm-v-store/@types/express<EMAIL_ADDRESS>'package.json' does not have a 'typesVersions' field.
File '/store_path_replaced/.pnpm-v-store/@types/express<EMAIL_ADDRESS>does not exist.
'package.json' does not have a 'typings' field.
'package.json' has 'types' field 'index.d.ts' that references '/store_path_replaced/.pnpm-v-store/@types/express<EMAIL_ADDRESS>File '/store_path_replaced/.pnpm-v-store/@types/express<EMAIL_ADDRESS>exist - use it as a name resolution result.
Resolving real path for '/store_path_replaced/.pnpm-v-store/@types/express<EMAIL_ADDRESS>result '/store_path_replaced/.pnpm-v-store/@types/express-serve-static-core<EMAIL_ADDRESS>======== Type reference directive 'express-serve-static-core' was successfully resolved to '/store_path_replaced/.pnpm-v-store/@types/express-serve-static-core<EMAIL_ADDRESS>with Package ID '<EMAIL_ADDRESS>primary: false. ========

// Main diff:
/// with paths
- File '/tmp/tsc_test1/projA/node_modules/express-serve-static-core.d.ts' does not exist.
- File<EMAIL_ADDRESS>does not exist.
- Directory '/tmp/tsc_test1/node_modules' does not exist, skipping all lookups in it.
// then it just goes up until root

/// without paths
+ File '/store_path_replaced/.pnpm-v-store/@types/express<EMAIL_ADDRESS>does not exist.
+ Found 'package.json' at '/store_path_replaced/.pnpm-v-store/@types/express<EMAIL_ADDRESS>+ 'package.json' does not have a 'typesVersions' field.
+ File '/store_path_replaced/.pnpm-v-store/@types/express<EMAIL_ADDRESS>does not exist.
+ 'package.json' does not have a 'typings' field.
+ 'package.json' has 'types' field 'index.d.ts' that references '/store_path_replaced/.pnpm-v-store/@types/express<EMAIL_ADDRESS>// the main difference that it reads `package.json` and takes `types` ref from there

🔎 Search Terms
tsc types with paths, type resolution paths, non-hoisted types and paths
🕗 Version & Regression Information

This is the behavior in every version I tried, and I reviewed the FAQ for entries about it.

⏯ Playground Link

The case need both tsconfig.json and specific dependencies, so I can't see a good way to implement it in the playground.

💻 Code
Test case:

Use the test case from this repository
Install dependencies with the exact command: pnpm i --frozen-lockfile --public-hoist-pattern ""
Run tsc, it will fail to resolve type-referene directive 'express-serve-static-core'
To get tsc complete successfully remove paths from tsconfig.json and run tsc again.

🙁 Actual behavior
When compilerOptions.paths is set we got no check for package.json in the referenced module so tsc couldn't resolve the type, in the same time with no paths set it could resolve it as package.json is checked and types field is taken into accound from there.
🙂 Expected behavior
In both scenarios tsc should resolve types in the same way keeping it consistent and reliable.

Just FYI, in your example repository you don't have a dependency to TypeScript, so it's not clear what version you're using. Hasn't been mentioned in the issue either, as far as I can see.

... version you're using ...
Thanks a lot, after checking around to make sure about all possible versions for the test-case I found that typescript@rc (5.0.1-rc) works just fine, so the issue was fixed there.


Implemented hot backups of the LevelDB database

Thank you for the pull request! I will review and get back to you.

Thank you very much for your contribution! I have merged your PR to the develop branch.
Please note that I only merged your first commit https://github.com/leveldb-erlang/h2leveldb/commit/b7e84bb3ea0e9e9c3bd25c2424c8b360e655d370, because I do not want to redistribute dependencies. Instead, I updated our local forks of HyperLevelDB and Snappy, and fixed some problems in build_deps.sh.

I had bundled the dependencies as I'm building using Basho's node_package code, and the build pipeline breaks.
Thanks for the merge!

I had bundled the dependencies as I'm building using Basho's node_package code, and the build pipeline breaks.

Oh, I see. node_package seems an interesting tool, and it supports all OS/distros that I am targeting to (RHEL, FreeBSD and SmartOS). When I have more time, I will try if I can fix the build pipeline without bundling the dependencies.
Thanks,


Feature/implement embed doe pie chart

This PR enables embed link for pie chart component in agriculture emissions page:
PIVOTAL

@tsubik Good point but I think it should be changed in CW components, to something like:
innerRadius: PropTypes.oneOfType([   PropTypes.string,   PropTypes.number ]),
so I'll merge this PR now, I'll take care of this warning later


feat: created home sidebar

PR Features:

Created a sidebar as per the new design format
The sidebar is responsive for all types of devices

Screenshot
https://github.com/user-attachments/assets/738dcce6-5b0e-4777-ae31-44e699a0cca8
Desktop Mode [Expanded]

Desktop Mode [Collapsed]

Collapsed [Hover will show a tooltip]

Responsiveness check

Resolves #888
Checklist before requesting a review

[x] I have performed a self-review of my code
[x] I assure there is no similar/duplicate pull request regarding same issue

@abhinandanmishra1 what happens when on a video? there are two side bars?

@siinghd This is how it looks for now. In the upcoming designs, we'll need to add a show/hide list feature at the place of the second sidebar. In this PR, I have only worked on one feature. I can implement the other features on top of this code if the main sidebar looks good.


Add gatsby mdx plugin testing

the purpose of this pull request is for visibility because we have multiple people contributing to this feature

😩 sorry about this mess, I only wanted to commit 2 lines of code. I'll try and fix it. Closing the PR doesn't seem to be an option

Oh, we probably need to merge master into add-gatsby-mdx-plugin?


Updated polgyonMesh.ts

Changed the scene to optional and updated the parameter doc
Sorry you guys beat me to merging, looks like one last tiny change here

::smh:: git can be seriously annoying sometimes

scene || Engine.LastCreatedScene
engine.LastCreatedScene ends up being Nullable same as | undefined

hrmm not letting me revert back to before it updated all those files guess ill just update the current state to the previous state that should solve :/

Ok to merge?

Appears to be