Inline CSS Loading Fails with Vite 5

Describe the bug
When using Vite 5, the function {{ craft.vite.inline(craft.vite.entry('emails.css')) }} fails to return any output, indicating a significant issue with inline CSS loading. The core of the problem lies within the extractEntry function in manifestHelper.php, which is required to delve deeper due to changes in how CSS files are represented in manifest.json in Vite 5. The malfunction occurs because the styleKey is not correctly retrieved. This is due to the filenameWithoutHash function not functioning correctly, as it only searches for dots in filenames, ignoring dashes. This specific issue traces back to the transition from Vite 3 to Vite 4, where the separator in filenames changed from dots to dashes. However, the repercussions of this change only became apparent with the introduction of Vite 5, due to adjustments in the manifest file structure and the handling of CSS files. A workaround involves modifying line 300 in ManifestHelper.php from $filenameParts = explode('.', $filename); to $filenameParts = preg_split("/[.-]/", $filename);, which addresses the issue and allows the CSS from email.css to be correctly inlined.
To reproduce
To reproduce the behavior, follow these steps:

Upgrade to Vite 5.
Attempt to inline CSS in a Craft CMS template using {{ craft.vite.inline(craft.vite.entry('emails.css')) }}.
Observe that the expected inline CSS is not rendered, indicating the function returns nothing.

Expected behaviour
The expected behavior is for the CSS from emails.css to be correctly inlined when using the {{ craft.vite.inline(craft.vite.entry('emails.css')) }} function in Craft CMS with Vite 5.
Screenshots
N/A
Versions

Plugin version: Vite 4.0.8
Craft version: Craft 4.7.1

Appreciate the issue! I think:
$filenameParts = preg_split("/[.-]/", $filename);

Might be problematic if the file name has a - in it however.

Yeah, you're right. The quick and dirty fix was a stopgap for my scenario. Perhaps splitting only at the last dash and keeping splits on dots for older Vite versions could be a more stable solution.

Yeah, you're right. The quick and dirty fix was a stopgap for my scenario. Ideally, if possible, checking which version of Vite is being used and then deciding whether to split file names at dots, or just at the last dash (for Vite 5) based on that information, could be a more stable solution.

Addressed in the above commits.


improved docs with images

the doc branch is visible on read the docs: https://vanilla.robotools.dev/en/doc/


isBounded is not working?

I am trying to do isBounded={true} in react grid layout but it's not working.

It is working for me, are you on version 1.0.0?

yup, my all library, but no luck, it always out of the box
    "@testing-library/jest-dom": "^4.2.4",
    "@testing-library/react": "^9.3.2",
    "@testing-library/user-event": "^7.1.2",
    "lodash": "^4.17.19",
    "react": "^16.13.1",
    "react-dom": "^16.13.1",
    "react-grid-layout": "^1.0.0",
    "react-hotkeys-hook": "^2.2.2",
    "react-scripts": "3.4.1",
    "styled-components": "^5.1.1"

I am trying to do isBounded={true} in react grid layout but it's not working.

i'm too, my version is 1.0.0 but it not working. anybody helpp!

Can anyone help me with the usage of isBounded in 'react-grid-layout' ? What is this used for?

Can anyone help me with the usage of isBounded in 'react-grid-layout' ? What is this used for?

3 years late but https://react-grid-layout.github.io/react-grid-layout/examples/16-bounded.html. Basically it prevent the draggable box to go outside main space.


Using with sqlite3 with nw and angular

I tried to manually compile sqlite3 and add in in the project using nw-gyp as described in the README.
Many of the tutorial say that to use sqlite3, nw and angular together I have to approach like
Angular -> Express -> sqlite3
Where Angular handles the UI part, using express I create an API and the api endpoints performs
database read write operations.
I'm using the template from here to create a minimal angular nw app and then at the root of this project I create an API folder in which my express api resides.
However when I try to import manually built sqlite3 in to the code I get following error and my express server fails to start.

in my node_modules/sqlite3/lib/bin I have folder named node-webkit-v0.46.0-darwin-x64 as expected from the manual build.

I'm starting the express server using node-main argument from nw pointing towards my startup script.
The minimal reproducible example for the following just to import the sqlite3 in a script and then put that script in node-main to start it before the nw application loads.
What is the cause for this error and how to fix this?

Same problem...

Have you found any alternative?@panther7

@vatsalkgor https://github.com/JoshuaWise/better-sqlite3 ?

Thanks for the suggestion. Because of JoshuaWise/better-sqlite3#65, I didn't try it till today.
However, to my surprise, when I tried it today it worked like a charm. You can find the complete boilerplate with nwjs here.
Thanks again for the suggestion though. I would not have tried it if you haven't suggested it.


BlobRequestsOptions - some example involving Blobs would be good - I believe (and hope I'm wrong) that BlobRequestOptions is not actually available - or at least I can't find how to use this

[Enter feedback here]

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

ID: 8194899c-dee2-70ac-422a-edc06e779f7c
Version Independent ID: cafc7418-404c-5908-3037-dba7f452087b
Content: Azure service retry guidance - Best practices for cloud applications
Content Source: docs/best-practices/retry-service-specific.md
Service: architecture-center
Sub-service: best-practice
GitHub Login: @dragon119
Microsoft Alias: pnp

#please-close


Attempt to add template argument to cds::gc::hp::smr

HP in Libcds has list of thread_record items, allocated by custom allocator, defined by set_memory_allocator function.
in attach_thread libcds just tries to get 'empty' thread_record structure(from list, or allocate and add to list), ans assign it to some thread local pointer.
in detach_thread libcds just mark some structure as 'empty', and assign nullptr to some thread local pointer
in tls() libcds just get data from thread local pointer.
So in TLSManager you should specify how to change and get this local pointer.


Thursday

Workshops partial changed to allow for multiple presenters
Reveals Thursday's workshops and talks
Changes levels on workshops to match this document -- previously I was going by their form responses /shrug

Oh - that document was a guess - I'll update it, go from their form responses


Speed up check on website links

Please add a meaningful description for your change here

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

r @damccorm - this should also let us skip files that are very large to speed up the checks


Serve compressed files if already exist

If file is already compressed (ends with .gz or .br), no need to add those file extensions.
Sometime files are already compressed and needs to serve with the correct header.
In some cases engines like Unity output some of the files as compressed and others not.
This change allows to serve already compressed files even if Serve GZIP Files or Serve Brotli Files are false.
I'm not sure how to write a test, or if this change should be documented.
It seems to me like an obvious outcome when serving a .gz or .br file. So I think it's a bug fix rather than a feature or a change in a feature.
Relevant issues
Resolves #844
Resolves #780
Contributor checklist

[x] Provide tests for the changes (unless documentation-only)
[ ] Documented any new features, CLI switches, etc. (if applicable)

[ ] Server --help output
[ ] README.md
[ ] doc/http-server.1 (use the same format as other entries)

[x] The pull request is being made against the master branch

Maintainer checklist

[ ] Assign a version triage tag
[ ] Approve tests if applicable

Apparently there were already tests for gzip and brotli, and my first commit broke the Content-Type tests.
I moved the defaultType to the start of the process right after the options are parsed.
Also moved the MIME lookup to right after we get the file, which made redundant the special lookup for compressed files.
Some unrelated CLI tests don't work for me both on master and on this PR.
@thornjad is there something else I should do for this PR to be approved?
Thanks


Incorrect statment

File: engine/userguide/networking/work-with-networks.md, CC @mstanleyjones
Wrong Statement: Assigned IP addresses are preserved when using non-user-defined networks, because there is no guarantee that a container’s subnet will not change when the Docker daemon restarts unless you use user-defined networks.
In my understanding
Correct Statement: Assigned IP addresses are not preserved when using non-user-defined networks...

Can I take this up @mstanleyjones?

Sure, probably best to just remove the incorrect statement.

As I mentioned, you just need to fix the statement. It's useful statement, removing this is not a good idea.
Correction: Assigned IP addresses are not preserved when using non-user-defined networks, because there is no guarantee that a container’s subnet will not change when the Docker daemon restarts unless you use user-defined networks.


Bugs in random

randomInt32() gets 8 bytes of random for a 32 bit int. https://github.com/1Password/spg/blob/5a36bf7b8df16cd5642f55405bae33a71b5649d8/util.go#L44
int31n() gets a random uint32 [0, 2**32). When n is not a power of 2, max is at most 2**31-2. This has a >25% to <50% vs a >50% to <100% chance of success each loop. To fix this mask the random v := randomInt32() & math.MaxInt32.
https://github.com/1Password/spg/blob/5a36bf7b8df16cd5642f55405bae33a71b5649d8/util.go#L82-L95 if int31n() is called with n > 2**31 it is not uniform. This bug was caused by switching n from int32 to uint32. Switch n back, do if int32(n) <= 0 { panic..., or rename to uint32n() and fix max (see https://github.com/Sc00bz/ModRandom/blob/a2cd9247a0dcb0183ec6305574b5696f51186540/csprng-cpp/csprng.cpp#L259-L284).

(Note these are just performance bugs and a bug that isn't triggered in your code... well I'm pretty sure int31n() isn't called with an n that's more than 2**31)

Nice finds, @Sc00bz! I'm looking at this code for the first time as well. It looks like we intended to switch to uint32 but didn't quite get there as you noticed. So I was going to go with option 3: rename and fix max.
But then I found out why this was originally an int32 and not a uint32. In order to fix max, I need something like max := (1<<32 - 1) - (1<<32)%n, but 1<<32 overflows my uint32 and means I'd need to cast to uint64.
I looked at your implementation to see how you solved that, and I don't think you did. You're doing MaxUint32 - (MaxUint32 % n), which is wrong and means you have a modulo bias (unless I'm missing something obvious).

Best thing I could come up with is max := uint32(math.MaxUint32 - (1<<32)%uint64(n)). Could do some fancy modular arithmetic to keep values inside uint32, but at that point we're using more memory and doing more computations, so not really worth it.

 Yours is perfectly fine when n is not a power of 2, which was caught before we get here.


Error while importing in React

My code :
I am making a function to shorten the number of messages so that the tokens doesnt exceed max tokens.
import { GPTTokens } from "gpt-tokens";

let reduceTokens = (history, model = "gpt-3.5-turbo") => {
  // let usageInfo = new GPTTokens({model : model ,messages : history});

  // console.log(usageInfo.usedTokens);
  return history;
};

export default reduceTokens;

While importing i am getting this error

Please help...

OK, I fix it in v1.0.6

Originally this package was only developed for the node server  > ,0
Relevant hints have also been added to the README.md

If use vite, you need do this in vite.config.js,
import wasm             from 'vite-plugin-wasm'
import topLevelAwait    from 'vite-plugin-top-level-await'
import { defineConfig } from 'vite'

export default defineConfig({
    plugins: [wasm(), topLevelAwait()],
})

If use webpack, you can view this
https://www.npmjs.com/package/@dqbd/tiktoken#compatibility


added v1 README.md to the experiment folder

@jschepers also is a flowchart good for the experiment pipeline in README.md?

@jschepers also is a flowchart good for the experiment pipeline in README.md?

Yes, I think that's a good idea, but it's not super urgent. We can also add it later. Maybe we can add it to the experiment to-do list (after moving it to an issue) and add it as a not urgent to do (just that we don't forget).

@jschepers the discussed changes are added to the PR!


42 display eurosign

added the ability to add a font with eurosign. Strangely all unknown characters default to the eurosign

Since it is currently a problem that all special characters default to the Eurosign I wouldn't want to install it per default.
I also looked for a possibility to add the font when installing the widget, there probably is a possibiliy to provide a script for installation but I see that more as a concern of the initial setup/settings ticket.
If that feature is still desired, I have to go ask the teaching chair tomorrow.

This is not a requirement anymore therefor this will be closed


bug: There seems to be a problem when using StaticCredentials for the local YDB

Test that demonstrates the problem https://gist.github.com/Zork33/ba680a5e169f2d4e993c7e030f024ddf

log.txt
The test log shows that attempts to connect continue after the driver has already been removed.  This is obviously incorrect behavior.  To fix it's needed to add debugging information


Delegation flows

Purpose
Adds flows for configuring baker account, adhering to the new protocol version, and configuring delegation. Old baker flows are seamlessly replaced when new protocol version goes live on network.
Changes

Add flows for configuring account as baker
Add flows for configuring account to delegate
Deactivate old baker flows for protocol version 4 and onwards
Add slider component (used to control commission rates)
Add multi-step from component, to compose flows on multiple pages, along with specific versions of this for both single-/multi-sig account transactions.
Added chain data redux slice, to allow showing stale data from chain while getting an update.
Transaction cost estimates of configure baker/delegation
Transaction serialization for configure baker/delegation

Checklist

[x] My code follows the style of this project.
[x] The code compiles without warnings.
[x] I have performed a self-review of the changes.
[x] I have documented my code, in particular the intent of the
hard-to-understand areas.
[x] (If necessary) I have updated the CHANGELOG.

Some of the configure baker transactions no longer work, please fix.

Self reminder: bump the minimum ledger version to 2.2.0


[Bug] 1.1.0-pre1 Popup binding does not work on iOS

Description
In XCT 1.1.0-pre1, binding in a popup does not work on iOS (but it does work on Android).
Steps to Reproduce

Open sample app on iOS
Go to the XamlBindingPopup demo
Open the popup

Expected Behavior
The popup should contain a title and a message.
Actual Behavior
On iOS, the popup is empty. The bindings do not work.
Basic Information

Version with issue: 1.1.0-pre1
IDE: VS 2019 16.8.6
iOS:  14.4

Reproduction imagery

@tranb3r can you please provide a repro?

Can confirm. Until a proper fix is available you can use this as workaround:
Remove BindingContext from popup, add to inner root view element.
e.g. XamlBindingPopup.xaml
<StackLayout Style="{StaticResource PopupLayout}">
    <StackLayout.BindingContext>
        <vm:XamlBindingPopupViewModel />
    </StackLayout.BindingContext>
    ...
</StackLayout>

cc: @ahoefling

Fixed in #1003


Create a dnf/yum repo for ublue hosted RPM specs, etc

Background
There are several packages which we'd like to include in ublue, specifically kmods, which do not have an upstream akmod build, or do not have an RPMs built at all, even if the project provides a spec file.
Some examples:

openrazer ( see #5 )
xpad-noone ( see #26 )
snapraid (for ucore) ( see https://github.com/ublue-os/ucore/issues/25 )

Beyond these which have no current solution, there are things we install which are in random COPR repos or in negativo17. Negativo17 seems reliable, but it's still an additional third party beyond rpmfusion, and random COPRs, well, it's like ubuntu-PPA land. If we don't know the maintainer, it's a gamble.
Goal
The goal is to provide a team ublue managed repo that:

at LEAST contains packages we can't get anywhere else (see examples above)
possibly contains packages which we currently get from random COPRs and other 3rd party repos
will NOT replace rpmfusion as it is a hard dependency at this point

Suggested Implementations
1. Host our own SPEC files
For any RPMs we need to build, we should host the spec files in our own github repo. This ensures control of what we are building and minimizes risk of a third party causing our RPMs to include any surprises.
I believe we could have a single repo rpm-specs (open to suggestions on name) with the various maintained spec files in it.
If we went all in on some of my suggestions for packages above it could include:

openrazer.spec
snapraid.spec
xone.spec
xpad-noone.spec
etc

There may be subdirs for supporting files for each spec, or perhaps each spec goes into a distinct sub-dir by default.
2a. Github pages ublue-os rpm repo
One proposal, though a potentially janky approach, would involve having a dedicated github repo which is setup for github pages. This is distinct from rpm-specs, maybe call it rpm-repo.
Github actions in this repo would on whatever trigger, build the RPMs from rpm-specs, run a create-repo script that creates the yum repo metadata, and then commits resulting metadata and RPM binaries to the rpm-repo git repo such that the contents of the repo are available to use.
This is kinda janky, but it could possibly work.
The major pro is we don't need any extra user management, its just github.
The major con is we'll be reinventing some wheels and doing some janky stuff.
2b. ublue-os team COPR
Another proposal is to use a single COPR repo/project (insert proper terminology here). The spec files would still be hosted in a github rpm-specs repo as described above, but all building and repo hosting would be done in COPR.
There are webhooks and apis in COPR which should at least allow triggers of builds when changes are made to specs, or from other actions. Possibly the api could be used to add a new spec to the COPR.
The major pro here is COPR is specifically made to build and host RPMs.
The major con is user access/permissions management. We think it's single user. A possible workaround could be to manage limited access to a shared credential similar to how we already manage private key escrow for signing OCI images. Also, if we only need access to "setup" a build/project since webhooks could trigger rebuilds, this may be less of an issue than originally thought.
Feedback
I'd like some team feedback on this. A few of us have been thinking about this for months, but not really made progress. It becomes more of a blocker to progress as the days go on. Whichever path is taken will require some effort. No need to rush it, but it's good to be on the same page with goals and approach.

I believe we should go with a combination of 1 and 2b. Having a centralized place for spec files that we're using would be ideal. We can pull those into copr (since the infrastructure for that there is already excellent)
I understand there's a concern over shared access, but ideally, the package in copr just needs to be created and everything regarding maintenance after that would be done in a shared GitHub repository with an array of our spec files
I'm not saying 2a should be off the table. It would be cool and interesting to see something new there, but generally we already have the infrastructure we need without the jank


HTML storybook doesn't build since 4.1.11

Describe the bug
I'm using @storybook/html in a server-side project which needs to generate some HTML templates. It works fine on 4.1.9 and before but on upgrading to 4.1.11 start-storybook stalls before showing the pink box and "Storybook started" message. When loading it in the browser it shows a generic Cannot GET / message. build-storybook also fails but in a different way with the message
ERR! => Failed to build the manager
ERR! Module not found: Error: Can't resolve 'react-modal' in '/Users/tamlyn/projects/reason/symphony-rfq/node_modules/@storybook/html/node_modules/@storybook/ui/dist/modules/ui'
(node:82208) UnhandledPromiseRejectionWarning: ModuleNotFoundError: Module not found: Error: Can't resolve 'react-modal' in '/Users/tamlyn/projects/reason/symphony-rfq/node_modules/@storybook/html/node_modules/@storybook/ui/dist/modules/ui'

To Reproduce
Steps to reproduce the behavior:

Run start-storybook -c .storybook
Storybook does not successfully compile

(Sorry, these are not very useful repro steps, I can try and make a separate repo to demo the issue if necessary.)
Expected behavior
Storybook compiles
System:

OS: MacOS
Node: 8.15.0
Device: n/a
Browser: n/a
Framework: n/a
Addons: n/a
Version: 4.1.11

Additional context
The package is in a monorepo where some packages are using Babel (7) but this package is plain, untranspiled ES2018.

From the error message it looks like you don't have react-modal instated, try:
npm install --save react-modal

react-modal is installed at node_modules/@storybook/ui/node_modules/react-modal. If I forcibly install it into the root of the workspace and try a build again, I get a similar error about react-fuzzy.
However that did lead me to discover that I had @storybook/ui in two places in my node_modules. Deleting node_modules and reinstalling seems to have fixed it.
Thanks for encouraging me down the right path 🙂

No problem!


Add a Gitter chat badge to README.md

bpkg/clip now has a Chat Room on Gitter
@yorkie has just created a chat room. You can visit it here: https://gitter.im/bpkg/clip.
This pull-request adds this badge to your README.md:

If my aim is a little off, please let me know.
Happy chatting.
PS: Click here if you would prefer not to receive automatic pull-requests from Gitter in future.

Sorry, I did just re-authorize my gitter account now :-(


bug: TC profile field missing from pydantic model

Overview
When uploading a JSON v6 PD protocol with a TC profile step, I get a pydantic error. Upon further investigation, it seems as though this is because the pydanic model is missing a profilefield.
Steps to reproduce
Upload the protocol attached in the app (or any JSON v6 protocol with a TC profile step), you'll get a pydantic error like this:
line 135, in translate translated_obj = _translate_simple_command(command) File /Users/shlokamin/Desktop/workspace/opentrons/api/src/opentrons/protocol_runner/json_command_translator.py", line 103, in _translate_simple_command dict_command, File "pydantic/tools.py", line 35, in pydantic.tools.parse_obj_as File "pydantic/main.py", line 406, in pydantic.main.BaseModel.__init__ pydantic.error_wrappers.ValidationError: 86 validation errors for ParsingModel[Union[opentrons.protocol_engine.commands.aspirate.AspirateCreate, opentrons.protocol_engine.commands.dispense.DispenseCreate, opentrons.protocol_engine.commands.blow_out.BlowOutCreate, opentrons.protocol_engine.commands.drop_tip.DropTipCreate, opentrons.protocol_engine.commands.home.HomeCreate,

woow.json.zip

Moving this ticket to Jira: RSS-26.


MemoryLeakFix

Autorelease pool handles memory management for the pcm: Data object
created in the realtime audio input callback.
Fixes Issue #571

Codecov Report

:exclamation: No coverage uploaded for pull request head (memoryLeakFix@754347e).

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 c886cd3...754347e. Read the comment docs.

Looks good to me! Thanks for making the updates per our suggestions, I think this'll be clearer for people who look over it in the future.


[site] Added pprof doc

What type of PR is this?
/kind documentation

What this PR does / why we need it:
[site] Added pprof doc
Which issue(s) this PR fixes:

Fixes #983
Special notes for your reviewer:
Does this PR introduce a user-facing change?

NONE

/assign @trasc

@moficodes the bot is not adding a preview for this PR. What do we need to do?

/ok-to-test

@moficodes it has for quite some time afaik

@mrbobbytables
The bot did not add a deploy link to this PR. :(
I thought it fell off the netlify build.

Not sure how, but for some reason it reverted back to only doing previews for PRs targeting the main branch. Will probably need to push an empty commit to retrigger it.


Update all the tests to include groups

Update valid and invalid groups in the various test cases to ensure accurate and meaningful testing.

Parent issue: #73


Feature request: remove component

Hi Adam,
a feature that could be easily integrated and would greatly enhance Unicorn is a client side "remove component" action.
In many cases unicorn components are list elements or table rows that include action buttons like "Delete". You have examples for that using parent components that remove that row and rerender the whole list. This unnecessarily over-complicates the situation, as in IMHO many cases no parent component is needed - just a
 or
 element.
What e.g. the Tetra framework does, is to provide a self.client._remove_component() method, that just removes the whole element from the DOM at the client side.
So a simple component delete action would just be:
class MyComponentView(UnicornView)
    def delete():
        self.client._remove_component()

and the template:

  ...

      <button ... u:click="delete()">Delete</button>

Adding client side Js functions that can be called from the backend would be generally a huge benefit. (already there)
But the delete() action is the most important.

Maybe just the client side must be implemented here, as self.call() in the backend is already there.

0.58.0 of Unicorn adds a Unicorn.deleteComponent method to remove a component from internal component store. I could extend this to remove the actual DOM element as well if that would be useful.

I think this is what would be expected then, yes.


Fix specs for radio element decorator classes

Description
Updated specs for radio element to not check for has-success or has-error attributes as these are provided by respective decorators now.
Checklist

[x] I have read and understand the CONTRIBUTIONS.md file
[x] I have searched for and linked related issues
[x] I have created test cases to ensure quick resolution of the PR is easier
[x] I am NOT targeting main branch
[x] I did NOT include the dist folder in my PR

@json-schema-form/angular-schema-form-lead

@jozefizso I plan to remove those whole tests from this repo, I am looking at applying them in the decorator, the webpack/babel branch is the latest dev branch, I plan to merge it into development in the not-too-distant future.

@jozefizso I do appreciate you taking the time to submit the PR, but for me to accept this I'm just as happy for them to be removed completely by your PR rather than just bypassed :)

@Anthropic So..should this be closed or what?

@jozefizso all tests are fixed and running again now :)


RTD warns of build failure, log offers few clews

Details

Read the Docs project URL: https://readthedocs.org/projects/kevlar
Build URL (if applicable): https://readthedocs.org/projects/kevlar/builds/7085704/
Read the Docs username (if applicable):

Expected Result
Documentation builds
Actual Result
Build terminates with a failure message. Hard to see what the problem is, if any, from the build logs.

Maybe related to #3999

Perhaps. It looks like I'm seeing order of magnitude fluctuations in build time: +/- 70 seconds at best, and 500+ seconds at worst.

My latest build passed, so I'll chalk this up to transient behavior that is being actively monitored and addressed. Thanks!

Thanks @standage!


Add support me tab

Is your feature request related to a problem? Please describe.
A "Support Me" tab in the sidebar would be a great option.
Describe the solution you'd like
Support me can consist of options like: "Buy Me A Coffee", "Paypal", "Patreon" etc.
Describe alternatives you've considered
NA
Additional context
An example

*Other options can be added.

@rishavanand , I would go for this if possible, I see this as a great option for those who give great opensource projects for the community out of their time.

Yes @rohanprasadofficial, good point. I'll look into "support me" options along with GitHub sponsors. Thanks.

@rohanprasadofficial have you created buymeacoffee account? if you so please share the link?

@smkamranqadri  hey , i am taking something like this , the repo has , there is a "Buy me a coffee" link
https://github.com/rohanprasadofficial/quotes

In the repo or in the app it self?
also need your username anyhow.

Oh! I missed, that's your repo.

How does it look like?

@smkamranqadri we are planning to add it as tab like "Intro" or "My Skill Set" so the people can add Support me options on their readme.
https://github.com/rohanprasadofficial/quotes : This is the link one of my repo.

Okay, well, little bit more info will help understand the task.

Taking this, reviewing the previous code and will do accordingly.

Does it look good?

Preview

Thank you @smkamranqadri


[TRACEFOSS-436] Add license header

License template taken from https://github.com/catenax-ng/foss-example/blob/main/catenax-ng-repositories/CopyrightAndLicenseHeader_1.txt
I've also committed the script which add header. In order to use Node.js 16 should be installed and it can be run by:
node ./scripts/add-license-header.js --src=./src

Does anybody have a config for IDEA, so it adds the headers automatically?


How do you remove a binding event of a single element?

How do you remove a binding event of a single element?

try vm.$off


HTTP health check call timeout hardcoded to one second

This is a follow up of https://github.com/cloudfoundry/healthcheck/issues/5
Issue
It is not possible to configure the timeout for HTTP health calls; it is hard coded to one second.
When the application takes longer than 1 second to respond to http calls, it is immediately marked as unhealthy and restarted by CF.
Context
In many applications the HTTP health endpoint does more than just return 200, e.g. the spring actuator health endpoint tries to open a database connection (if one is configured). Under load, this easily takes longer than one second to succeed, and thus the application instance is restarted, even though it is perfectly healthy.
Steps to Reproduce
Create an application which takes longer than one second to respond for the http health endpoint. It will never succeed to start up.
Expected result
It should be possible to configure the HTTP health check connection timeout via the manifest and/or a push parameter.

Possible dupe #970

Implemented in this story.  Available in capi-release 1.54.0
Docs here

How to specify this within the deployment manifest.yml file?


Date splitting incorrect

An entry created for the 30th will be displayed from the 1st.

After restarting the app the display was correct again

However, as you scroll down, the top elements are removed from (display) memory.

And if you just don't remove it from memory? Would that create that much load?


Add --client, --dump, --restore and --readonly

Similar to the postgres PR, with some MySQL-specific notes:

The MySQL client doesn't support specifying a connection string URL, but it should be supported here (1) for consistency across database images and (2) because it's a nice, compact way to specify the connection details. I cribbed a script from Stack Overflow to parse the URL into its pieces, but it's pretty ugly. That's utilities.sh.

The MySQL client has a honeypot option --password which throws up
Warning: Using a password on the command line interface can be insecure.

That actually gets printed (to stderr) during mysqldump, creating an invalid dump file if one naively redirects output > dump.sql. However, the client doesn't complain if you specify an environment variable $MYSQL_PWD instead. Much more secure.

For super users, MySQL's read only mode has no effect at all.  I made that switch, but any databases already initialized from this image will not really be read only. Should the user be a super user?

utilities.sh is rough. Any thoughts on alternate approaches, especially for sharing that sort of functionality across the various "standardized" database repositories?

@fancyremarker @aaw

This looks great. Regarding the downgrade from superuser, i was looking at this page and I think the only privilege that a user with . privileges might have that a user with db.* privileges might need on Aptible is CREATE USER. We definitely have customers that see a need for creating users with diminished privileges. Maybe we could provision two accounts in --initialize, one superuser and one aptible user, both with the same password, and give everyone the aptible user in the handle? That way, we'd have a way of allowing people to log in a superuser and do admin operations like creating users but they would still by default run their apps with the less privileged account.

This is excellent. For utilities.sh, what do you think about isolating its functionality to a script, say parse_url, which given a URL, sets the environment variables SCHEME, HOST, PORT, USERNAME, PASSWORDandPATH`.
This parse_url function could be included in each of the base images (Alpine, Debian, Ubuntu).

Good ideas, thanks for the feedback.
I've added a password-accessible root user entry and kept the aptible user privileges restricted to the created database.
Re utilties.sh: as much as I dislike polluting global state, that seems to be the thing to do in shell script (and apparently the script was doing it anyway). Anyway, I took your suggestion @fancyremarker and the resulting code is cleaner and simpler, and can be shared without modification across these database images.
As for putting parse_url into the base images: seems like a good idea, but it presents some difficulties. First, I think we can't just drop it in /usr/bin and execute it: the environment variables it defines will disappear. We'd have to source it in the script, kind of like what I'm doing now. Second, it'll save some time now for me to just continue on to the other images and bring this technical debt file with me to each one that needs it.
Finally, I still like the lowercased variables returned by parse_url because they're less likely to collide with other things. For instance, I could see PATH causing problems :bowtie:. If you feel strongly about this, I'm happy to change them.

@jbergknoff : Duh, re: PATH. :flushed:
I think sourcing /usr/bin/utilities.sh is fine. Alternatively, you might be able to put it in /etc/profile.d/utilities.sh. For login shells, this would automatically source the functions. Not sure how that works with Docker init shells.

Merging. Again, thanks — this is great.


[Feature] JXL support

Is your feature request related to a problem? Please describe.
Loading JXL images
Describe the solution you'd like
Adding JXL support
Describe alternatives you've considered
None
Additional context
Glide Plugin:
https://github.com/awxkee/jxl-coder-glide
Coil Plugin:
https://github.com/awxkee/jxl-coder-coil

Will have to look at it a little deeper. My only concern would be that photok does custom image loading for the decryption.
If there plugins don't collide, I don't see any reason not to support this 👍

But I would prefer to finish #318 before

I see, coil plugin should work if you use coil to get rid of glide. it should be as simple as adding the jxl factory to image loader. best of luck


Categories leading to page 404

It seems to be leading to a dead end due to the url being https://thinkty.net/ko/categories/thoughts/ instead of https://thinkty.net/ko/categories/사설
Check the masthead source code for what is exactly happening

Currently disabled Breadcrumbs for other reasons and will be dismissed for now


Load balancers not always created, restarting controller-manager required

Rancher versions:
rancher/server or rancher/rancher: v1.6.17
rancher/agent or rancher/rancher-agent: v1.2.10
Infrastructure Stack versions:
healthcheck: v0.3.6
ipsec: v0.13.11
network-services: v0.7.20
scheduler: v1.10.0-rancher1-2
kubernetes (if applicable): v1.10.0-rancher1-2
Docker version: (docker version,docker info preferred)
Docker version 1.13.1, build 092cba3
Operating system and kernel: (cat /etc/os-release, uname -r preferred)
NAME="Ubuntu"
VERSION="16.04.4 LTS (Xenial Xerus)"
ID=ubuntu
ID_LIKE=debian
PRETTY_NAME="Ubuntu 16.04.4 LTS"
VERSION_ID="16.04"
HOME_URL="http://www.ubuntu.com/"
SUPPORT_URL="http://help.ubuntu.com/"
BUG_REPORT_URL="http://bugs.launchpad.net/ubuntu/"
VERSION_CODENAME=xenial
UBUNTU_CODENAME=xenial
Kernel:
4.4.0-1052-aws
Type/provider of hosts: (VirtualBox/Bare-metal/AWS/GCE/DO)
AWS
Setup details: (single node rancher vs. HA rancher, internal DB vs. external DB)
Single Node, External DB
Environment Template: (Cattle/Kubernetes/Swarm/Mesos)
Kubernetes
Steps to Reproduce:
Deploy Kubernetes service with AWS internal load balancer annotations
apiVersion: v1
kind: Service
metadata:
name: application-server-service
labels:
app: application-server-service
annotations:
service.beta.kubernetes.io/aws-load-balancer-internal: '<IP_ADDRESS>/0'
service.beta.kubernetes.io/aws-load-balancer-ssl-cert: "arn:aws:iam::<account_id>:server-certificate/<cert_name>"
service.beta.kubernetes.io/aws-load-balancer-connection-idle-timeout: "600"
service.beta.kubernetes.io/aws-load-balancer-backend-protocol: http
service.beta.kubernetes.io/aws-load-balancer-extra-security-groups: "sg-"
service.beta.kubernetes.io/aws-load-balancer-cross-zone-load-balancing-enabled: "true"
spec:
ports:
- name: https
port: 443
targetPort: 8000
selector:
app: application-server
type: LoadBalancer
Results:
If controller-manager container has not been restarted recently it sometimes does not create the load balancer in AWS and leaves service in pending state.  If I restart the controller-manager container it immediately sees the new ELB service config and finishes the setup.

With the release of Rancher 2.0, development on v1.6 is only limited to critical bug fixes and security patches.


Add FAQ entry re vim extension requiring Chrome

See https://github.com/cdr/code-server/issues/708

Should be fixed upstream now!


bump Zephyr to 3.5.0-rc3 and NCS to tip of main (2.5.0-rc1+)

Bump Zephyr to latest release candidate to test it before final 3.5.0
release.
Bump NCS to tip of main (slightly newer than 2.5.0-rc1 tag), to find
potential nRF91* platform regressions.
Replace rand32.h with random.h to suppress header deprecation warning. Add
a workaround for older Zephyr versions (like the one used by NCS fork) by
providing random.h just for internal Golioth SDK library visibility.
Tested on:

[x] qemu_x86
[x] nrf52840dk_nrf52840
[x] mimxrt1060_evkb
[x] esp32_devkitc_wroom
[x] nrf9160dk_nrf9160_ns

Looks like the build test failed, because the runner ran out of disk space. I hit the same error in the CP SDK, and you should be able to fix it by parallelizing the build similar to https://github.com/golioth/golioth-firmware-sdk/commit/5eb86b75ba9e9561d6fe013ef69b793eccf5301d

Looks like the build test failed, because the runner ran out of disk space. I hit the same error in the CP SDK, and you should be able to fix it by parallelizing the build similar to golioth/golioth-firmware-sdk@5eb86b7

Proposed solution (https://github.com/golioth/golioth-zephyr-sdk/pull/439) is slightly different, much smaller in boilerplate code size, but seems like a perfect fit for what we have now. This will give some input for comparison with the solution you proposed and implemented for CP SDK. Both are fine, maybe they can be combined together. My implementation was inspired by Zephyr CI matrix (but they use arbitrary number of matrix instances, which is then passed to twister, which decides by its own how to divide all test cases to be executed).

@sam-golioth Do you have some idea about why build job is "Expected - Waiting for status to be reported"? It looks like a job that is the parent for 3 matrix jobs. Those 3 jobs are successful, but still, build is somehow pending. In case that is something new, I'll just investigate what the issue might be (I guess something in workflow specification which is not exactly what GitHub expects, or I just need close/reopen to workaround some corner case in GitHub CI).

@sam-golioth Do you have some idea about why build job is "Expected - Waiting for status to be reported"?

I had marked the build job as a required check after it failed on this PR earlier. The reason is because when a runner runs out of disk space, the job is marked as "skipped" not "failed" and the PR UI treats skipped jobs like successful jobs (it still said "All checks have passed"), so it wasn't obvious that there was an issue. Marking it as required makes sure we don't miss when a job is "skipped". It looks like there isn't a way to mark the parent job as required, so I've just removed the requirement. We can mark the child jobs as required later, after this gets merged.
Also, the job being marked as "skipped" and not "failed" might be due to something in our workflow rather than an unsolvable GitHub actions issue, but I didn't want to put in the time to investigate that right now. Just something to keep in mind for the future.


compatible versions

thanks for this plugin.
but i found something wrong in my project:
CocoaPods could not find compatible versions for pod "native_filters":
i saw your flutter sdk version range(sdk: ">=2.18.2 <3.0.0"), but i can't find it at document: https://docs.flutter.dev/development/tools/sdk/releases

@binvb We developed new solutions for image/video filtering, you can try them.
https://pub.dev/packages/flutter_core_image_filters
https://pub.dev/packages/flutter_image_filters
https://pub.dev/packages/flutter_gpu_video_filters

thank you


🐞[Bug]: Removing all the unwanted codes

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
No bug
wanted to remove all the unwanted codes which is making the , file lenghty and also placing the css code in css file and html js in same way.Removing some unwanted file , which are not used atall.
What browsers are you seeing the problem on?
Chrome
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I have starred the repository

@varshith257 please assign me this issue.

please assign me this issue

@varshith257 please assign me this issue.


add topics

I suggest adding topics such as f2py, ctypes, interoperability, fortran, python in the About section at https://github.com/HugoMVale/fortran-in-python

thanks, just did so!


NPC Indicator Fill Opacity

With the new highlights for interactions added, I started looking into configuring my NPC indicators tile thickness and etc, and realized I wanted a way to make the fill have less opacity. Small config and appropriate access added.

A Discussion point, is before all Opacity was defaulted to 20, so this still links all those together (sw tile, hull, outline). In theory, if someone did a 255 settings for a solid color block, it would also completly fill the hull, which may not be exactly the user wants.
Should we (or I in this case) possibly break the render out into more specified  renders to avoid this? In that case I would end up changing this to "Tile Fill Opacity" and only apply it to SW and Tile renders and not hull and outline, which would stay at the default 20 (or I can make that a setting as well).
 (Nothing this in the wiki docs)

The color picker has opacity support and we typically use that for this type of thing. Any reason you are not doing that?

If you are referring to the opacity for the lines in the current color option, that didn't seem to apply to the tile fills. Just the outline itself.
And making the option a color picker over just a numerical value seemed overkill for just an alpha change.
Default Fill

0 Fill

255 Fill

If you think there is value in making the fill color different from the Outline color, I can look into doing that I don't think it would be a large change from what I already have, as you said would be more consistent with other feature sets, just didn't feel right to me in this particular implementation, but I was also blinded by my specific goal during implementation and testing.

I think that we can change this back to alpha/12 like how it was done in ab4cf1afe913879f9d0c1b031d77798b0b6c865e - this was reverted in fa43ca94dbe97235feb54d4fdce499a6be3f555e due to it being no longer impossible to set the color fully transparent in order to get a border less highlight, which some people want, however now the border width is configurable so that should no longer be an issue.

Hmm, my only problem with with the alpha/12 approach is that what if one wants 0 fill, but a bright tile highlight, in which case would not be achievable. I think the best solution would just be a color picker for fill, with a default to the current  Cyan at 20 alpha.

A 2nd color picker for the fill is fine too I guess.

Cool, I'll get that patched in and tested. Thanks for the feedback and discussion :D

added in 33ff3a2749372a1d98329d2c4aeddbeadf19434e


Fix: localhostnet -> localnet

I researched through other NEAR projects and found out nobody uses localhostnet except us but localnet.
This is the rename.

localhostnet was there for WAMP local development where you would not need to name the topics as mainnet/testnet, instead just spawn the backend and frontend using a common name localhostnet. It is too convoluted to run Explorer for localnet, which is why it has never been an option. You may just drop it now since with the migration away from WAMP, we don't need it.


Add configure registration tips for flask admin

Updated configuration docs and Flask-Admin example.

Pull Request Test Coverage Report for Build 63

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 78.448%

Totals

Change from base Build 60:
0.0%

Covered Lines:
91

Relevant Lines:
116

💛  - Coveralls


configure production workspaces

As product owner we would like the following workspaces to be configured in the production environment of jupyter:
workspaces:
    10_ws_advanced:
      display_name: Advanced Generic Workspace
      description: Advanced environment for testing with Python R and Julia.
      start_date: 2022-01-01
      end_date: 2030-01-01
      kubespawner_override: *jupyter_advanced
    vc-sd:
      display_name: VC-SD Workspace
      description: Shared workspace for VC and SD
      start_date: 2022-05-01
      end_date: 2025-12-31
      kubespawner_override: *jupyter_advanced
      storage:
        volumes:
          - name: landerhub-vc-sd
            persistentVolumeClaim:
              claimName: pvc-landerhub-vc-sd
        volume_mounts:
          - name: landerhub-vc-sd
            mountPath: /home/jovyan/landerhub-vc-sd
            readOnly: False

    lth-dst:
      display_name: LTH Data Science Team Workspace
      description: Shared workspace for LTH DST
      start_date: 2022-06-22
      end_date: 2025-12-31
      kubespawner_override: *jupyter_advanced
      storage:
        volumes:
          - name: landerhub-lth-dst
            persistentVolumeClaim:
              claimName: pvc-landerhub-lth-dst
        volume_mounts:
          - name: landerhub-lth-dst
            mountPath: /home/jovyan/landerhub-lth-dst
            readOnly: False

    mmu_dfu_cpu:
      display_name: Diabetic Foot Ulcer Research
      description: |
        Python environment with JupyterLab, Code and XCFE.
        Upto 8 vCPU and 32GB RAM. (No GPU)

        All work must be stored in /home/jovyan/dfu.
        Any work saved outside of this folder will be lost between sessions.
      start_date: 2023-07-06
      end_date: 2024-01-03
      kubespawner_override:
        image: crlander.azurecr.io/vvcb/cpu-dfu:2023-07-26
        defaultUrl: "/lab/tree/dfu"
        mem_guarantee: 4G
        mem_limit: 32G
        cpu_guarantee: 0.1
        cpu_limit: 8
      storage:
        volumes:
          - name: dfu
            persistentVolumeClaim:
              claimName: pvc-landerhub-dfu
        volume_mounts:
          - name: dfu
            mountPath: /home/jovyan/dfu
            readOnly: False

    avoidable_admissions:
      display_name: Avoidable Admissions
      description: Collaboration with Lancaster University on Avoidable Admissions
      start_date: 2023-09-01
      end_date: 2026-07-01
      kubespawner_override: *jupyter_advanced
      storage:
        volumes:
          - name: landerhub-avoidable-admissions
            persistentVolumeClaim:
              claimName: pvc-landerhub-avoidable-admissions
        volume_mounts:
          - name: landerhub-avoidable-admissions
            mountPath: /home/jovyan/landerhub-avoidable-admissions
            readOnly: False

This has been configured, awaiting confirmation from @vvcb that the current configuration is working as expected


The extension works fine in Desktop but not on the server.  I can see the tabs but the tab object does not display.  If I click export it actually exports data from the object.

which version of Qlik Sense Sever are you using?

Qlik Sense 3.0 SR2
From: David<EMAIL_ADDRESS>Sent: Monday, July 3, 2017 2:13 AM
To: dfarache/Tabbed-Container-Extension
Cc: Herms, Chris C UTCHQ; Author
Subject: [External] Re: [dfarache/Tabbed-Container-Extension] The extension works fine in Desktop but not on the server. I can see the tabs but the tab object does not display. If I click export it actually exports data from the object. (#23)
which version of Qlik Sense Sever are you using?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_dfarache_Tabbed-2DContainer-2DExtension_issues_23-23issuecomment-2D312557685&d=DwMCaQ&c=ilBQI1lupc9Y65XwNblLtw&r=6z3kuEi7abHyzBGRXIyyeI01c8kI6APVFP2As4MphWU&m=YJ4d-Y74wJq-GZoqC5DWwDWqQJQ_qHipMY703xvuKyM&s=kW_EQo39Ow2MPElz6h6VH6-7p8bRCEo0m_sqaqgvEeE&e=, or mute the threadhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AcWu2vhToDMlxJf9N-2DxSXDJZPRqqwNsRks5sKIZRgaJpZM4OKkJ4&d=DwMCaQ&c=ilBQI1lupc9Y65XwNblLtw&r=6z3kuEi7abHyzBGRXIyyeI01c8kI6APVFP2As4MphWU&m=YJ4d-Y74wJq-GZoqC5DWwDWqQJQ_qHipMY703xvuKyM&s=YML1YNSi7jvDAC1Mcwvpz9d9cj0VyNo0JJxOuQAEH1M&e=.

could you please check if the latest version fixes your problem?

Unfortunately it will take some time to get IT to update our Enterprise environment.
Are there know issues with 3.0 SP2 and your extension?
From: David<EMAIL_ADDRESS>Sent: Monday, July 3, 2017 11:12 AM
To: dfarache/Tabbed-Container-Extension
Cc: Herms, Chris C UTCHQ; Author
Subject: [External] Re: [dfarache/Tabbed-Container-Extension] The extension works fine in Desktop but not on the server. I can see the tabs but the tab object does not display. If I click export it actually exports data from the object. (#23)
could you please check if the latest version fixes your problem?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_dfarache_Tabbed-2DContainer-2DExtension_issues_23-23issuecomment-2D312671838&d=DwMCaQ&c=ilBQI1lupc9Y65XwNblLtw&r=6z3kuEi7abHyzBGRXIyyeI01c8kI6APVFP2As4MphWU&m=RwDzTvSFZ53MlSQmDeEAASAXXsB04wVGTOFBkkThanc&s=CIRff9YOIQxndLalbT-G6wF1WeIGyJkZ-gDSzTXpfns&e=, or mute the threadhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AcWu2lscHiTFlT6PsBO8CNd-2DaPnoz8ggks5sKQSkgaJpZM4OKkJ4&d=DwMCaQ&c=ilBQI1lupc9Y65XwNblLtw&r=6z3kuEi7abHyzBGRXIyyeI01c8kI6APVFP2As4MphWU&m=RwDzTvSFZ53MlSQmDeEAASAXXsB04wVGTOFBkkThanc&s=CmGeXFEjmCSlP7ho-J9p1Us68DqYSGMzlZEopSR1PGs&e=.

I meant the latest version of the extension. I pushed a small change yesterday that I hope will fix your problem

We reinstalled latest version but still not working.   It looks like I get a brief message in the tab window when I first open the sheet but it disappears immediately so I can’t see what it says.
From: David<EMAIL_ADDRESS>Sent: Tuesday, July 4, 2017 3:02 AM
To: dfarache/Tabbed-Container-Extension
Cc: Herms, Chris C UTCHQ; Author
Subject: [External] Re: [dfarache/Tabbed-Container-Extension] The extension works fine in Desktop but not on the server. I can see the tabs but the tab object does not display. If I click export it actually exports data from the object. (#23)
I meant the latest version of the extension. I pushed a small change yesterday that I hope will fix your problem
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_dfarache_Tabbed-2DContainer-2DExtension_issues_23-23issuecomment-2D312797079&d=DwMCaQ&c=ilBQI1lupc9Y65XwNblLtw&r=6z3kuEi7abHyzBGRXIyyeI01c8kI6APVFP2As4MphWU&m=oW-e6I67pN6cxUNNlKiwyF52pmjBUYzz2xjDepAua3g&s=I-DaMsQljDZSw7Py7vSfw-Eh1VZioVFjKX-CmGYH7pY&e=, or mute the threadhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AcWu2k6oTqj7D2oC4yxNeNaD8FYvC7Emks5sKeNkgaJpZM4OKkJ4&d=DwMCaQ&c=ilBQI1lupc9Y65XwNblLtw&r=6z3kuEi7abHyzBGRXIyyeI01c8kI6APVFP2As4MphWU&m=oW-e6I67pN6cxUNNlKiwyF52pmjBUYzz2xjDepAua3g&s=C7qWzvAusk7mOvMaCzBxoOZeHdsqvmwbBI06I4hYKMc&e=.

Which browser are you using?
Does the browser console display any errors?

Tried on IE11 and Chrome.  A few lines of text flash for a brief second in the tab object but I could not see what it says.
From: David<EMAIL_ADDRESS>Sent: Friday, July 7, 2017 3:46 AM
To: dfarache/Tabbed-Container-Extension
Cc: Herms, Chris C UTCHQ; Author
Subject: [External] Re: [dfarache/Tabbed-Container-Extension] The extension works fine in Desktop but not on the server. I can see the tabs but the tab object does not display. If I click export it actually exports data from the object. (#23)
Which browser are you using?
Does the browser console display any errors?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_dfarache_Tabbed-2DContainer-2DExtension_issues_23-23issuecomment-2D313611997&d=DwMCaQ&c=ilBQI1lupc9Y65XwNblLtw&r=6z3kuEi7abHyzBGRXIyyeI01c8kI6APVFP2As4MphWU&m=6oF9KojOuNYZkTx2FsbprerJlqco-1IEL4WnMPtpfq4&s=J087b2ZJII-aPuCI1lgyDPQ5fJzmfPRSQTZRxpgeTQI&e=, or mute the threadhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AcWu2vddF5C-5Fsx0PRw3rTs6A1qGz6EMXks5sLeItgaJpZM4OKkJ4&d=DwMCaQ&c=ilBQI1lupc9Y65XwNblLtw&r=6z3kuEi7abHyzBGRXIyyeI01c8kI6APVFP2As4MphWU&m=6oF9KojOuNYZkTx2FsbprerJlqco-1IEL4WnMPtpfq4&s=HM-ftHW2R29sIIaToz3uxgX37Y4h1Xf1W--c6cwXqDM&e=.

And do you see any errors in the Developer Tools Console? (to open it press F12 in Google Chrome)

I confirm the same bug, QS Server 3.1. No messages in console. It worked with previous extension version (but unstable - sometimes the chart got lost until reload).
Also sometimes we get the following  error in console when switching tabs. I think it is a different one as no chart display even if no errors in console.
require.js?1472561073404:44 Error: [ngRepeat:dupes] Duplicates in a repeater are not allowed. Use 'track by' expression to specify unique keys. Repeater: measure in details.Measures, Duplicate key: string:Delta, Duplicate value: Delta
http://errors.angularjs.org/1.5.0/ngRepeat/dupes?p0=measure in details.…0�ие&p2=Изменение
at require.js?1472561073404:43
at require.js?1472561073404:47
at d (require.js?1472561073404:45)
at n.$digest (require.js?1472561073404:45)
at n.$apply (require.js?1472561073404:45)
at require.js?1472561073404:45
at e (require.js?1472561073404:43)
at require.js?1472561073404:43

Thanks for the feedback. Unfortunately, I have not been able to reproduce this issue either on Sense 3.1 SR4 -which is the one we are using- nor on the latest releases.

I have just pushed a couple of changes that fix some issues that the extension had. Could you please check whether this has solved your issues?

@changeling76  @ccherms could you update the extension to the latest version and check if you still see this behavior?

The extension works fine in Desktop but not on the server. I can see the tabs but the tab object does not display.
The version of my Qlik Sense Server is 10.3.5


No way to disable autoformatting

The formatting engine seems to render weird code using indent size 2, for example:
namespace Foo
{
    class FooOne
    {
        public void DoIt()
        {
          if (false) {
                Console.Log("Hi");
            }
        }
    }
}

Perhaps this is an issue with using indent size '2' instead of '4' or something to do with atom's soft tabs; I'm not really sure.
However, given that the formatting doesn't appear to be 100% spot on, I feel there should be an option to disable the auto formatting entirely; It's a complete deal breaking for using Omnisharp when it screws up the code everytime you hit ';'.

Have a look at issue #515, which describes a way you can disable it (remap keys).

Irritatingly, because of https://github.com/atom/settings-view/issues/652, this is actually rather difficult to unbind, even so.

Any progress on this? I'd like to disable auto-complete. Cause it consistently guesses incorrectly.

I made some changes with how the formatting is bound (not using atom commands anymore).  On my short list is to get configuration options loaded so we can define configuration at the editor level (and adopt things like .editorconfig) as well as the solution level and make sure that omnisharp.json is honored.


Sample weights for validation data causing val_loss to go to nan

Using keras and theano, The val_loss metric is nan when I pass sample weights for validation data. The same issue arises when passing validation data parameter to the "fit" function as well as using the model.evaluate. Train loss is not nan irrespective of sample weights parameter but only val_loss goes to nan upon passing sample weights. Don't know if the same issue exists with tensorflow backend because I can't run my model with TF backend due to low gpu memory.

I have the same issue. I think its a purely computational issue since with sample_weights = np.ones(nbItems) it works. I work on tensorflow.

Any luck with this? I'm having the same issue.

I had the same problem and @daoudChami 's tip seems to have solved it. My sample_weight was a Pandas DataFrame before. Converting it to a Numpy array fixed the issue.

@daoudChami 's solution seems to have worked for me too.
Weird problem though

confirmed issue exists with TF backend as well

This happens to me with a numpy array.


Don't run if process already running

Something like this:
    if pidof -x gphotos-cdp >/dev/null; then
        echo "already running"
        exit
    fi

This should already be handled by flock:
https://github.com/JakeWharton/docker-gphotos-sync/blob/a2b56f39307db9ad0e9e3720aec950d8e2709a5f/root/etc/cont-init.d/20-cron.sh#L35
Are you sure you're seeing concurrent runs? With what container version?

I don't believe I saw it in the container but I didn't see any safeguards for it. If flock handles it then this can be closed.


Track cold/warm times to execute notebook cells

For #10176

Codecov Report

Merging #10180 into master will decrease coverage by 0.03%.
The diff coverage is 8.33%.

@@            Coverage Diff             @@
##           master   #10180      +/-   ##
==========================================
- Coverage   61.33%    61.3%   -0.04%
==========================================
  Files         567      567
  Lines       30759    30770      +11
  Branches     4408     4410       +2
==========================================
- Hits        18867    18863       -4
- Misses      10908    10921      +13
- Partials      984      986       +2

Impacted Files
Coverage Δ

src/client/telemetry/index.ts
87.38% <ø> (ø)
:arrow_up:

...ient/datascience/interactive-ipynb/nativeEditor.ts
55.26% <8.33%> (-1.06%)
:arrow_down:

src/client/common/utils/platform.ts
64.7% <0%> (-11.77%)
:arrow_down:

src/client/linters/pydocstyle.ts
86.66% <0%> (-2.23%)
:arrow_down:

src/client/datascience/debugLocationTracker.ts
76.56% <0%> (-1.57%)
:arrow_down:

src/client/common/process/proc.ts
14.49% <0%> (-0.73%)
: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 de7db28...284dd86. Read the comment docs.


Honor CSS transformations of the element the popover is attached to.

I just submitted a PR for the needs of my feature : popover inside a container.
I merged your change in #161 . Could you please check that my PR also solve your issue?

@anlumo If you want to test my PR, I made an unofficial release for this. See : https://github.com/jeremyVignelles/webui-popover/tree/7d89959051117dca95e3fd2c4a813c1f607779bb
If you are using bower, you can just link to that revision and see if it works.

Will do, thank you!

Sorry for the very very late response. The issue has been fixed for me in the latest release.


Nx workspace: Could not find project "undefined" in 'angular.json'

🐞 Bug report
Description
When creating a NX workspace and adding scully, starting it initially won't work and leads to the error message Could not find project "undefined" in 'angular.json' (see full error message below).
🔬 Minimal Reproduction
1.) create a nx workspace with an angular app
npx create-nx-workspace scully-plugins
npx: installed 48 in 4.391s
✔ What to create in the new workspace · angular
✔ Application name                    · scully-plugin-demo
✔ Default stylesheet format           · css
✔ Use Nx Cloud? (It's free and doesn't require registration.) · No

2.) add a node library (this is maybe not necessary)
chore: add @nrwl/node
npx nx g @nrwl/node:lib scully-plugin-foo

3.) add scully as described in the scully docs
npm install @scullyio/init
nx g @scullyio/init:install -- --project=scully-plugin-demo

The file scully.scully-plugin-demo.config.ts has been created correctly

💻Your Environment
Angular Version:
nx version:

Angular CLI: 13.0.4
Node: 16.13.1
Package Manager: npm 8.1.2
OS: darwin x64

Angular: 13.0.3
... animations, common, compiler, compiler-cli, core, forms
... language-service, platform-browser, platform-browser-dynamic
... router

Package                         Version
---------------------------------------------------------
@angular-devkit/architect       0.1300.4
@angular-devkit/build-angular   13.0.4
@angular-devkit/core            13.0.4
@angular-devkit/schematics      13.0.4
@angular/cli                    13.0.4
@schematics/angular             13.0.4
rxjs                            7.4.0
typescript                      4.4.4

Scully Version:

"@scullyio/init": "^2.0.5",
"@scullyio/ng-lib": "^2.0.0",
"@scullyio/scully": "^2.0.0",
"@scullyio/scully-plugin-puppeteer": "^2.0.0",

🔥 Exception or Error

npm run scully

><EMAIL_ADDRESS>scully
> npx scully --

  ⚠
      =====================================================================================================
      Config file "./scully.undefined.config.ts" not found, only rendering routes without parameters
      The config file should have a name that is formated as:
          scully..config.ts
      where  is the name of the project as defined in the 'angular.json' file
      If you meant to build a different project as undefined you can use:
          --project differentProjectName as a cmd line option

      When you are in a mixed mono-repo you might need to use the --pjFirst flag.
       which will look for package.json instead of angular.json to find the 'root' of the project.
    =====================================================================================================

  x Could not find project "undefined" in 'angular.json'.

Running npx scully --pjFirst will lead to the same result.
Handing over the config file explicitely will work:
npx scully --cf scully.scully-plugin-demo.config.ts

Am running into this too.  However,
npx scully --cf scully.myapp.config.ts
does not solve anything.

the same issue here

i tried to run npx scully --project other using flag  target: 'targets' in my config file. But does not solve.

https://stackoverflow.com/questions/73442069/no-scully-configuration-file-found-scully-undefined-config-ts-path-scully-unde/73442528#73442528

Could not find project "undefined" in 'angular.json'.
The right solutions is here!!!
in angular.json add in the root of the object:
"defaultProject": "my-blog",
{ "$schema"<EMAIL_ADDRESS>"version": 1, "newProjectRoot": "projects", "defaultProject": "my-blog", "projects": { "my-blog": { "projectType": "application", "schematics": { "@schematics/angular:component": { "style": "scss" } },
And name the scully file as scully.my-blog.config.ts

Yeah, defaultProject is deprecated in Angular 14 and removed when you do ng update commands
I had to add it again to make scully work as @TatyanaMolchanova said.

Same here. Thanks @TatyanaMolchanova .

Currently im using NX v14.1.9 with Angular v13.3.12. Here my config file
scully.web-showroom.config.ts

import {ScullyConfig, setPluginConfig} from '@scullyio/scully';
import '@scullyio/scully-plugin-puppeteer';
/** this loads the default render plugin, remove when switching to something else. */
import { baseHrefRewrite } from '@scullyio/scully-plugin-base-href-rewrite';
import { getHttp404Plugin } from '@gammastream/scully-plugin-http404';

const Http404Plugin = getHttp404Plugin();
setPluginConfig(baseHrefRewrite, { href: '/abc' });

export const config: ScullyConfig = {
  projectRoot: "./apps/web-showroom/src",
  projectName: "web-showroom",
  distFolder: './dist/apps/web-showroom',
  spsModulePath: 'YOUR OWN MODULE PATH HERE',
  outDir: './dist/static',
  defaultPostRenderers: [Http404Plugin,'seoHrefOptimise', baseHrefRewrite],
  target: 'targets',
  handle404: 'index',
  routes: {
  }
};

npx scully --scanRoutes --cf=scully.web-showroom.config.ts
its able to generate the static file. Hope can help


Update OpenTelemetry Collector

Add fixes for OPA update to the latest version
Signed-off-by: anastasia.malysheva<EMAIL_ADDRESS>

@denis-tingaikin , I think local nsm configs needed for this example because we have to be sure that tracing is enabled for forwarder and manager, even though it is enabled by default now. Plus this setup is not installing nsm as in other examples in the folder, but it applies it as any other resources.

@anastasia-malysheva Could you please resolve conflict?

@denis-tingaikin conflicts resolved


Fix import warning on PromptTemplate

This should silence the warning added in LangChain 0.0.302 😁

This is what I get from my local test

Ah, you know what... I've made an oversight. I will fix it today 👌

@Jped Give it a try now. It just slipped my mind there was another import used there 😅

@dmyoung9 looks good -- mind changing the library version?

Done!

Thanks for the merge! Glad I could help ☺


cli: error handling when the daemon exists unexpectedly. also fixes the case where you're talking to an old daemon

Hello @pmt, @landism,
Please review the following commits I made in branch nicks/errors2:
f42b533e363e069fc7831e9b9b858b5c89022f0f (2018-08-22 13:13:45 -0400)
cli: error handling when the daemon exists unexpectedly. also fixes the case where you're talking to an old daemon

updated this a bit. the main way i'm testing this is to have the daemon panic, and make sure the parent process captures the panic() output

hmmm...for some reason github didn't show me maia's message until just now. i don't care that much either way.

as discussed off-list, this doesn't kill any new processes, or change anything about how/why processes are killed. it only changes what the processes print when they exit.

(i'm fine with moving the daemon in-process, but gonna merge this for now because it unblocks the bugs i'm trying to diagnose)


oneOf and patternProperties validates incorrectly

I would expect the following test to pass:
  def test_one_of_pattern_properties
    schema = {
      "$schema" => "http://json-schema.org/draft-04/schema#",
      "oneOf" => [
        {
          "patternProperties" => {
            "^([a-z_0-9])+$" => {
              "type": "object",
              "properties" => [
                "foo" => { "type" => "string", "enum" => ["bar", "hello"] }
              ]
            }
          }
        }
      ]
    }

    assert_valid schema, { "test_thing" => { "foo" => "bar" }}
    refute_valid schema, { "test_thing" => { "foo" => "no" }}
  end

It fails with:
# Running:

F

Finished in 0.004238s, 235.9452 runs/s, 471.8904 assertions/s.

  1) Failure:
OneOfTest#test_one_of_pattern_properties [test/test_one_of.rb:103]:
{"test_thing"=>{"foo"=>"no"}} should be invalid for schema:
{"$schema"=>"http://json-schema.org/draft-04/schema#", "oneOf"=>[{"patternProperties"=>{"^([a-z_0-9])+$"=>{:type=>"object", "properties"=>[{"foo"=>{"type"=>"string", "enum"=>["bar", "hello"]}}]}}}]}.
Expected [] to not be equal to [].

1 runs, 2 assertions, 1 failures, 0 errors, 0 skips

Is this a bug? or am I missing something with my schema?

Fairly sure I was mistaken here.


How to Rewrite HTTP_AUTHORIZATION header with claims from token?

Hi.
I have legacy app (not editable) served by apache, this app require Basic authorization header.
I am developing service for integration with this app, service uses OAuth2 authorization.
I can put in token claims like app_user and app_password or just app_basic_auth_header claim (not sure apache can calculate base64).
How can I rewrite HTTP_AUTHORIZATION header with value from claim?
Found on stackoverflow this:
RewriteEngine On
RewriteCond %{HTTP:Authorization} ^(.)
RewriteRule . - [e=HTTP_AUTHORIZATION:%1]
So does this will work?
AuthType oauth2
OAuth2TokenVerify jwks_uri https://pingfed:9031/ext/one jwks_uri.ssl_verify=false
OAuth2TargetPass headers=on
RewriteEngine On
RewriteCond %{HTTP:OAUTH2_CLAIM_app_basic_auth_header} ^(.)
RewriteRule . - [e=HTTP_AUTHORIZATION:%1]

you can use mod_headers and add a RequestHeader directive with a value pulled from the environment variables OAUTH2_CLAIM_* that have been populated with claims

you can use mod_headers and add a RequestHeader directive with a value pulled from the environment variables OAUTH2_CLAIM_* that have been populated with claims


FR: adjust printed source

Currently, we print https://www.fr.ch/sites/default/files/2020-10/201013_statistiques_situation_COVID_site_V2.xlsx as the source for the FR data despite there only being case number data for the districts (no hospitalisations etc.).
We should change the printed source to https://www.fr.ch/de/gesundheit/covid-19/coronavirus-statistik-ueber-die-entwicklung-im-kanton.

I think the source is correct. FR publishes one excel file for the districts and the canton as a whole. It contains all data that we have of FR (including hospitalisation, deceased etc.)


Enable ollama multimodal

Enable ollama multimodal (only images in reality)
Open to a better solution to find out if a model is supplied by ollama or not

Definitely want to enable this.
One consideration is that on the frontend we restrict image input to specific models.
Is there a way to tell if an Ollama model allows image input via the local Ollama API?

in part yes, seehttps://github.com/jmorganca/ollama/issues/1772


Fix issues storing mp_id when using MariaDB

Since IntegerField has a limit of 2^31 on MariaDB, that doesn't work for saving the external ids for Payments. Use BigInteger instead.
Fixes #8

New release?

Just pushed v5.1.1, it should appear in a few minutes.


Support for Readeck

Hello,
Not sure if this is within scope, but I've recently started playing with Readeck.  Similar to Linkding, but with a few additional features that are pretty compelling (at least for me).
It has a well documented and fairly straight forward API.  Is there any interest in adding support?
Thanks,
Adam.

Hey Adam, some thoughts (tl;dr at the bottom):
Readeck looks like a nice manager with interesting features. Personally, I'm satisfied with the linkding feature set but I might check it out sometime.
My main concerns are two-fold: feature creep and - in relation to that - maintenance effort. As it is now, I have no automated test framework, which makes deploying new versions quite inefficient as I have to manually test various settings in two browsers and (by now) for a bunch of search engine websites. It's on the top of my list of things to do for the injector, but as I have no experience in writing automated tests for browser extensions, I unfortunately haven't found the time or motivation to dig into the topic.
Getting automated tests would definitely be a requisite for even considering implementing other bookmark services or other major new functionality.
Another aspect is UI/UX. The settings page would need an overhaul to allow choosing the bookmark service for injection and deal with any differences that the services might have.
I'm also not quite sure how high the demand for this would actually be. Gauging from the chrome extension user numbers, Readeck has about 1/10th of the linkding userbase. And the injector extension is used by roughly 1/10th of the linkding users.
That doesn't leave a lot of potential users. While this isn't an important factor for me, I am hesitant to put significant development time into features that I personally don't need and won't be used by most users.

tl;dr: I will keep this open as I'm not completely against this but I will also be honest in that you shouldn't expect developments on this any time soon. As always, I'm happy to accept pull requests. If someone wants to use the injector code as a basis for a Readeck fork that's perfectly fine as well. This happened for miniflux as well as shiori

Totally understand.
I've put in a feature request for them to add functionality similar to linkding-injector to their new browser extension.
If they aren't interested, and I end up moving to Readeck, I can always fork linkding-injector. :-)


Change the version number to allow packages to use the same PPA

This changes the format of the versions from xxxx-0ubuntu1~ppa1 to xxxx-0ubuntu1~24.10~ppa1 so that the noble and oracular versions do not collide in the PPA.
The version is simply overwritten because the new version will technically be older than the old version ("p" > "2"), so we should start over at the beginning. Since this is only in our testing PPA so far, we don't need to worry about breaking anyone's updates.

@hector-cao @phckopper same as the other last PR, just for oracular. Also, I don't seem to have merge rights even after an approval, so maybe we should add that (or just get in the habit of merging after approving)

@hector-cao @phckopper same as the other last PR, just for oracular. Also, I don't seem to have merge rights even after an approval, so maybe we should add that (or just get in the habit of merging after approving)

Gave you the admin rights, i m fine with manual merge but not against automatic merging even if i think that manual merging is safer


Request: Disable Nagle directly through patcher

It would be greatly appreciated if kanan is able to disable nagle directly. It'll be a great help to those who are on windows 10 or use proxy.

I would love this as well, as a fellow eurofag I have to use a proxy to play Mabinogi to actually have fun.
There is a very noticeable latency difference between Kanan and Abyss, it could be that proxies (mudfish, wtfast) have their own internet entries and that the TCPNoDelay values don't affect it, I don't know, it's just a guess.
Whatever the case, is it even possible to disable Nagle Algorithm with just memory editing?

Uhh, I can intercept socket creation and set TCP_NODELAY to true (which disables nagle). I might need to intercept setsockopt as well to make sure the game doesn't unset TCP_NODELAY. That should do it.

I've added what I think should be the correct way of doing it but since I have nagle disabled globally on my computer I can't easily test it. Let me know if it works please!

I got confirmation that I fixed it.


Add user field to LoginMethodResponse

We want:
# Type returned when the user logs in
type LoginMethodResponse {
  # Id of the user logged in user
  id: String!
  # Token of the connection
  token: String!
  # Expiration date for the token
  tokenExpires: Float!
  # Logged in user
  user: User!
}

But we would need to define the type User... Maybe pass the type name as an option?

Maybe the way to go is to define a 'User type' schema in resolvers ?
Thanks for spending time on this feature, I think this will help a lot.

@nicolaslopezj We could pass the user as an option, but how would you pass the custom user query to the client method?
I thought about something different. Let's create a onLogin event and pass a method which queries the logged in user and stores him in the local storage.
Another helpful function to do so would be storeByQuery. The onLogin method is called by loginWith* and createUser.
Here's an example:
import { onLogin, storeByQuery } from 'meteor-apollo-accounts'

onLogin(() => {

  storeByQuery({
    currentUser: `
      currentUser {
        _id
        profile	{
          firstname
          lastname
          name
        }
        emails {
          address
          verified
        }
      }
    `
  }
  )

})

The currentUser query resolver would look like this:
currentUser (root, args, context) {
  var { userId } = context ? context : { userId: null }
  if (userId) {
    return Meteor.users.findOne(userId)
  }
}

And is shipped with the Meteor package.
Making this work would require another option on the SchemaTypes function:
${SchemaTypes({
  CreateUserProfileInput: `
    firstname: String!
    lastname: String!
    name: String!
  `
  CustomTypes: `
    type User {
      _id: ID
      emails: [Email]
      profile: UserProfile
    }
    type Email {
      address: String
      verified: Boolean
    }
    type UserProfile {
      firstname: String
      lastname: String
      name: String
    }
  `
})}

Of course there would be a default fallback.
This ways also allows you to extend the user type with other attributes such as Role and still have it in the local storage of the client.
So basically in the end it's possible to do this:
import { user } from 'meteor-apollo-accounts'

user().role
user().profile.firstname

Is the "StoreByQuery" necessary?
Can't we define an User schema on server that developer can edit if required; e.g. : if User needs to be extended with something like Groups/Roles. I just don't know how is generated this LoginMethodResponse.

Maybe we could assume the app has a User type. @dbrrt wanna make a PR with the changes?

@nicolaslopezj  Yep, I think that the App should have a User type defined for authentication by the developer who can extend it depending his/her needs, and one User type should be enough for this case, and will avoid many confusion.

Can we imagine something like that :
import userType` from '/lib/user.js';

export default gql`
# Type returned when the user logs in
type LoginMethodResponse {
  # Id of the user logged in user
  id: String!
  # Token of the connection
  token: String!
  # Expiration date for the token
  tokenExpires: Float!
  # User Schema type defined in /lib/user.js
  user: User!
}

${userType}

It does the job, but requires a schema defined for user, the best scenario would be to define a default one, eg :
type User {
  _id: String
  profile: UserProfile
}

# Custom Userprofile
type UserProfile {
  firstname: String!
  lastname: String!
  groups: Groups!
}

# Groups Type
type Groups {
  groups: [Group]
}

# Group Type
type Group {
  group: String!
  role:  String!
}

The schema above should be overridable by the developer by defining one in /lib/user.js (for instance)

The user schema should be passed as parameter as it's already done with CreateUserProfileInput. Overriding the default schema by defining a file is not a good idea for many reasons (flexibility, file and folder structure, schema centralization, ...).

@janikvonrotz  I don't like it neither, your approach seems way better than defining the User schema in a file.
Have you an idea on how to implement storeByQuery or maybe already tested it?

@dbrrt No I'm not entirely sure how to do this, I'm also a bit busy with school right now. For now I would focus on parameterizing the SchemaTypes method with the additional user types, create a onLogin event method and use the apollo.query method to query currentUser  and save it to the local storage manually. Once done, decide wether storeByQuery would be a useful enhancement.

so if injecting a storage can work, than can be ok for React-Native too, which is great.
I'll try to implement that after solving storage issues.

It's simpler than it sounds, we have 2 alternatives to have this response:
type LoginMethodResponse {
  # Id of the user logged in user
  id: String!
  # Token of the connection
  token: String!
  # Expiration date for the token
  tokenExpires: Float!
  # User Schema type defined in /lib/user.js
  user: User!
}

On every method that returns a LoginMethodResponse add the user (user: Meteor.users.findOne()) to the response object. This will do the fetch for every login, even if the client doesn't requests the user.

The other way is to create the resolver for LoginMethodResponse:

{
  LoginMethodResponse: {
    user ({id}) {
      return Meteor.users.findOne(id)
    }
  }
}

But we don't have a option to pass resolvers to the apollo app, only for the mutations. We would need to make a breaking change to the api.

The problem now is how to store the Custom User schema and its associated sub schemas through GraphQL.

We would just assume that the user has a User type in their apps

I'd define one by default in the library resolver, that could be used if the user forgot/doesn't override it in his schema.
For now I'm defining it in my schema but the app and it works.

But if we define it in our library the user will have to change the user schema from our settings, not directly in their schema

I think we should just leave this up to the user to define their own User schema and resolvers and queries. A User type would be something thats used very often in their own queries, and not everyone would have the same shape as the default user object in meteor. For example, in my app, my user schema is:
type User {
    _id: ID!
    username: String
    email: String
    name: String
    picture: String
    bio: String
    social: UserSocial
    usertype: String
    title: String
    looks(skip: Int = 0, limit: Int = 10): [Look]
    shows(skip: Int = 0, limit: Int = 10): [Show]
    savedItems(skip: Int = 0, limit: Int = 20): [SavedItem]
    viewedItems(skip: Int = 0, limit: Int = 20): [SavedItem]
    likedLooks(skip: Int = 0, limit: Int = 20): [LikedLook]
    following: Boolean
}

Where the fields are resolved as follows:
{_id: user._id, username: user.username, email: user.emails[0].address, usertype: user.usertype, social: user.profile.social, name: user.profile.name, bio:user.profile.bio, picture: user.profile.picture}

Having the users schema change without them explicitly defining it as such is not really something we should do.

@hammadj I totally agree with that, in the app, we should be able to override a custom type User defined in the library, @nicolaslopezj  I didn't test it yet (lack of time).


Since python3.8 - module 'platform' has no attribute 'linux_distribution'

It seems, module platform attribute linux_distribution has been deprecated since Python 3.8 because of this OOM script is failing as below. I am running python 3.12 on Ubuntu 24.04.
Traceback (most recent call last):
File "", line 745, in
File "", line 737, in main
File "", line 632, in get_log_file
File "", line 75, in os_check
AttributeError: module 'platform' has no attribute 'linux_distribution'

Hey @sumitgarg44 thanks for raising the issue. As discussed directly, this is a problem with the old version of the script, but the newer version appears to be in working order. I'll close this issue.


[Snyk] Upgrade @unleash/proxy-client-react from 4.1.2 to 4.3.1

Snyk has created this PR to upgrade @unleash/proxy-client-react from 4.1.2 to 4.3.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 6 versions ahead of your current version.

The recommended version was released on 2 months ago.

Release notes

Package name: @unleash/proxy-client-react

4.3.1 - 2024-07-22What's Changed

chore: bump JS SDK dependency to ^3.5.2 by @ thomasheartman in #172

Full Changelog: v4.3.0...v4.3.1

4.3.0 - 2024-06-28What's Changed
Better support for FlagsProvider in microfrontends: you can pass initialized client, and set stop={false} to use Unleash client external to React SDK.

fix: update build pipeline by @ Tymek in #171
fix: provider initialization by @ Tymek in #169

Full Changelog: v4.2.4...v4.3.0

4.2.4 - 2024-05-22What's Changed

chore: update peer dependency version by @ kwasniew in #167

Full Changelog: v4.2.3...v4.2.4

4.2.3 - 2024-05-21What's Changed

chore(deps-dev): bump vite from 4.5.2 to 4.5.3 by @ dependabot in #164
chore: update js client version by @ kwasniew in #166

Full Changelog: v4.2.2...v4.2.3

4.2.2 - 2024-03-01What's Changed

chore(deps-dev): bump vite from 4.5.1 to 4.5.2 by @ dependabot in #159
fix: allow start client to be used for config by @ FredrikOseberg in #162

Full Changelog: v4.1.2...v4.2.2

4.2.2-beta.0 - 2024-02-27What's Changed

chore(deps-dev): bump vite from 4.5.1 to 4.5.2 by @ dependabot in #159
fix: allow start client to be used for config by @ FredrikOseberg in #162

Full Changelog: v4.1.2...v4.2.2-beta.0

4.1.2 - 2024-01-22What's Changed

fix: clear stale errors by @ FredrikOseberg in #153
docs: rename parameters related to feature name by @ Tymek in #156
chore(deps-dev): bump vite from 4.5.0 to 4.5.1 by @ dependabot in #155
chore(deps): bump @ adobe/css-tools from 4.3.1 to 4.3.2 by @ dependabot in #154
chore: bump unleash proxy client to 3.3.1 by @ sjaanus in #158
chore(deps): bump @ babel/traverse from 7.22.5 to 7.23.2 by @ dependabot in #144

New Contributors

@ sjaanus made their first contribution in #158

Full Changelog: v4.1.1...v4.1.2

    from @unleash/proxy-client-react 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

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 8.27%. Comparing base (7a0a36f) to head (fe668d9).

Additional details and impacted files
@@          Coverage Diff           @@
##           master    #801   +/-   ##
======================================
  Coverage    8.27%   8.27%
======================================
  Files          68      68
  Lines        3952    3952
  Branches      864     864
======================================
  Hits          327     327
  Misses       3625    3625

Flag
Coverage Δ

?

Flags with carried forward coverage won't be shown. Click here to find out more.

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

covered in https://github.com/RedHatInsights/ocp-advisor-frontend/pull/828


Settings constant import can cause circular import issues that armi.configure() hides

This was added to reactors.py recently:
https://github.com/terrapower/armi/blob/867ebecaa4f508239da4c3b4137c1b89beba8fbd/armi/reactor/reactors.py#L43
And it ended up causing a circular import when armi wasn't configured.
In [1]: from armi.mpiActions import MpiAction
---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
<ipython-input-1-a16b230e7ec8> in <module>
----> 1 from armi.mpiActions import MpiAction

c:\users\aopotowsky\codes\nala\framework\armi\mpiActions.py in <module>
     68 from armi import settings
     69 from armi import utils
---> 70 from armi.reactor import reactors
     71 from armi.reactor import assemblies
     72 from armi.reactor.parameters import parameterDefinitions

c:\users\aopotowsky\codes\nala\framework\armi\reactor\reactors.py in <module>
     41 from armi import runLog
     42 from armi.nuclearDataIO import xsLibraries
---> 43 from armi.physics.fuelCycle.settings import CONF_CIRCULAR_RING_MODE
     44 from armi.physics.fuelCycle.settings import CONF_JUMP_RING_NUM
     45 from armi.reactor import assemblies

c:\users\aopotowsky\codes\nala\framework\armi\physics\fuelCycle\__init__.py in <module>
     34
     35 from armi import interfaces
---> 36 from armi import operators
     37 from armi import plugins
     38 from armi import runLog

c:\users\aopotowsky\codes\nala\framework\armi\operators\__init__.py in <module>
     44 from armi import runLog
     45 from armi.operators.runTypes import RunTypes
---> 46 from armi.operators.operator import Operator
     47 from armi.operators.operatorMPI import OperatorMPI
     48 from armi.operators.snapshots import OperatorSnapshots

c:\users\aopotowsky\codes\nala\framework\armi\operators\operator.py in <module>
     37 from armi import runLog
     38 from armi import settings
---> 39 from armi.bookkeeping import memoryProfiler
     40 from armi.bookkeeping.report import reportingUtils
     41 from armi.operators import settingsValidation

c:\users\aopotowsky\codes\nala\framework\armi\bookkeeping\memoryProfiler.py in <module>
    325
    326
--> 327 class ProfileMemoryUsageAction(mpiActions.MpiAction):
    328     def __init__(self, timeDescription):
    329         mpiActions.MpiAction.__init__(self)

AttributeError: partially initialized module 'armi.mpiActions' has no attribute 'MpiAction' (most likely due to a circular import)

Here's it working:

It's odd to me that armi.configure() resolves the circular import....and I'm worried about other things that are masked by this. There are a NUMBER of unit tests with
from armi import configure, isConfigured
if not isConfigured():
    configure()

@opotowsky Can this ticket be closed?

This sort of thing is in a lot of places and only rears it head occasionally (when someone calls code that is USUALLY preceded by armi.configure but in their case is not).
I don't have time to research this though :-/
So you can close it and handle import issues as they arise or you can research it -- your choice!


path character issue

having issue with a single course that has multiple brackets in the output_path and file name
it's the only course I'm having this issue with
I have seen this issue with another script before but not positive its the issue.
however the full audio and video files are downloading to the temp folder
when the process_mpd.py util script gets to line 86, ffmpeg is used to merge audio and video but the output file produces is a 1KB file in the course folder.
if pause before the ffmpeg merge can see the files are there and are complete.
I have attached screen shots and textfile with url

course_url
courseurl.txt
ss1

ss2

ss3

The course you are trying to download has DRM Protection. You need to add --key with the actual decryption key to be able to decrypt the video lectures.
Without Decryption Key:

With Decryption Key:

will check again. maybe i have the wrong key.

confirmed was the issue.


fix: jira worklog collector fetches only the 1st page

Summary
fix: jira worklog collector fetches only the 1st page
Does this close any open issues?
Closes #7010
Screenshots

e2e failed


Added forceNormalizerByBassBoost option to ffmpeg filter options

Changes

Introduced the forceNormalizerByBassBoost option to control the enforcement of normalization when bass boost is applied.
By default this is set to true to keep current behavior.

Dependencies

No changes

lgtm!
Thanks for contributing :)


[PublishBuildArtifactsV1] Migrate to Node 10

Task name: PublishBuildArtifacts
Description:

Updated dependencies; removed unused @types/q dependency
Changed typescript version to 4.0.2
Changed deprecated MochaDone to Mocha.Done in L0.ts
Removed definitions reference from L0.ts
Bumped task version

Documentation changes required: No
Added unit tests: No
Attached related issue: https://github.com/microsoft/build-task-team/issues/708 (internal)
Checklist:

[X] Task version was bumped - please check instruction how to do it
[X] Checked that applied changes work as expected

Successful pipeline
Node 6 hotfixes branch

I've tried to run the following definition to test the task:
- bash: |
    echo 'test' > test.txt

- task: PublishBuildArtifactsm@1
  displayName: 'Ordinal publish to a container'
  inputs:
    PathtoPublish: 'test.txt'
    ArtifactName: 'drop'
    publishLocation: 'Container'

- bash: |
    mkdir TestDir

- task: PublishBuildArtifactsm@1
  displayName: 'Copy to local path'
  inputs:
    PathtoPublish: 'test.txt'
    ArtifactName: 'art'
    publishLocation: 'FilePath'
    TargetPath: './TestDir'

And it got stuck in a loop. It looks like a bug. @DaniilShmelev Could you please take a look?
Here's the failed build.

This issue also happens with the old version of the task. I've created a separate ticket (internal).


Update to zig master

I have updated your package to 0.12.0-dev.2105+60094cc3f. Pull it, please.

Apologies for the long wait. I have been busy with work. Check for conflicts and your PR will be merged. Thank you!


OSP Partner Invite | extId property pattern update needed

Detected in version: 1.8.0 RC1
*likely the issue exists since 1.7.0
Summary
Connected to the FE ticket: https://jira.catena-x.net/browse/CPLP-3763
In the current implementation an extID "123456" runs on an error
ExtID must be 6-36 digits long
although the submitted ID was 6 characters long
=> please recheck the pattern. Everything between 6 and 36 characters should be allowed (incl. uuid characters, digits, letters, etc)

@jjeroch I just checked the backend implementation, 123456 is possible as external id


Update JS selective resolutions

Cleaning up some selective resolutions.

Closed for simpler https://github.com/studentinsights/studentinsights/pull/2735


Custom genesis trigger, and deposit contract cleanups

Address items 2, 3, 4 in #1129.
Substantive changes:

Remove Eth2genesis log in favour of a custom is_genesis_trigger function
Reorder block_hash field within Eth1Data
Remove merkle_tree_index from Deposit log

I'm not seeing any changes wrt any of my comments. Did you not push something? For example "burnt" still in there

@djrtwo Sorry my git push failed and I walked away from keyboard.

I think all issues are addressed. (I also submitted a Vyper issue here—it seems cleaner to emit the Deposit log after validating the deposit data but tests fails when I do that.)


[LMS Admin] #45 don't show some events like...

don't show some events like login/logout (this should be configurable somehow)
Source url: https://admin-stage.escolalms.com/#/reports
Reported by<EMAIL_ADDRESS>Reported at: 19 Jan at 15:08 UTC
Console: 1× Error
Location: PL, Pomerania, Gdansk
Browser: Chrome 97.0.4692.71
OS: macOS 10.15.7
Screen: 1920x1080
Viewport: 1920x944
Screenshot:

For more details please visit report page on Ybug:
https://ybug.io/dashboard/reports/detail/29c320hcc78sp20bhmn4

for user endpoint /api/notifications filter events, skip login/logout

Done Close


Login flow update

Resolves #117
Resolves #84
Resolves #85

Just for clarity, I have added these three screens for onboarding and used private storage to know if the user is first time login or not

Question: currently I show “Following” and disabled button if the user is already following an account, should we allow unfollow which is the normal behaviour.?

This looks great!
Some feedback:

[ ] What is the criteria for the modal to come up and how can I make it not appear on load? (we should implement this before merge)
[ ] When it loads "Connect with others", I'm already following most of these people. It would be nice if it filtered these people out and only offered new people to follow. "Some people you may want to follow:"
[ ] Possible to add a back button? Will ask in designs

hey @elliotBraem, added all the feedback points.


Feature Request - Enhancements to Log Viewer in the UI

What kind of request is this (question/bug/enhancement/feature request):
Feature Request
Overview
Users would like to be able to easily select & export logs from the Rancher built in log viewer for pods.
Select CTRL-A or CMD-A selects more than just the logs in the log viewer.
Choosing CTRL-A or CMD-A selects components of the entire page, not just the log contents. There is no way to export the logs for download.
Other details that may be helpful:
Either a button that "selects all" or allowing keyboard shortcuts would suffice.
A button to "export" logs to a file for download would also suffice.
Environment information
rancher/rancher - UI specific
v2.1
v2.x
Cluster information
Any - UI related.
To Open Log Viewer:

Log Viewer View (no select all or export function):

CMD-A to select all selects more than just log contents:

@emm-dee We are going to investigate into adding the "Copy to Clipboard" button.

I'll be adding a Download Logs button that will grab the plain text content, minus the timestamp we prepend, to the container log modal.

Version - 2.2 master 11/14
Verified fixed