pud.h uses C++ keyword "private"

It is currently not possible to use war2tools within a C++ project since the structure "Pud" in pud.h contains the following field:
Pud_Private *private; /**< Internals, private and opaque data. Don't touch ;-) */
Simple fix that works is:
Pud_Private *private_; /**< Internals, private and opaque data. Don't touch ;-) */

Good catch! Thanks.


Stuck In "Lurk Mode"

If for any reason I want to go back to the default twitter view I have to remove the add-on. This might be a hassle especially if I only want the default view for a couple seconds, I'd have to remove the extension and restart the browser to get back the default view, then re install the add-on and restart the twitter tab when I want to go back in "lurk mode". I suggest making you add a toggle feature on to the addon so users can easily toggle "lurk mode" on and off at will. It would be nice if you get even get that in shortcuts as well.

The extension can be disabled/enabled from the chrome settings panel, takes effect once the page is reloaded. See gif below


404 not found, in version 1.8, i was use below that version not have trouble.

My Route List :

Providers :

Alliases :

I have already follow any instruction in issue
100
83
309
283
but not resolve my problem,
I was follow the instruction like ussually i used below version in
installation
and if i publish the vendor, file can't make automatically to public, views, and config.
I Had make it self by looking file that i used before.
But still NOT WORKING. Pls resolve this problem.
Sorry my english not good.
Thank you.

I have installed new project and install all library first before i had copied recent project file, it works. I don't know whats wrong with my project before -_-. but me surely all step it same.

@hai-ru which version are you using? If you are using v1.8.2-alpha, in config/lfm.php there is a url_prefix, which defines the url to the filemanager. Check if you have such a config.

Ok thank you it work


tooltips: Group tooltips for smooth transition.

Fixes: #24825

This PR groups the tooltips for compose box formatting buttons so that the delay only applies to the first button you hover.
This allows two things:

Smooth transitions of the tippy between many different reference element targets
Elements with tooltips next to each other that have a delay can be "grouped" so they appear to share a timeout, which greatly improves UX.

Previously , the tooltips of formatting buttons were treated independently and each of them had their own delays which doesn't feel smooth while hovering.
Now, the shared timer across the grouped tooltips helps the transition feel smooth and clean.
Fixes:

Screenshots and screen captures:
Previously:
https://github.com/zulip/zulip/assets/97145463/acdd50f8-5c0b-4d80-bd47-e097a2eadfd2
Currently:
https://github.com/zulip/zulip/assets/97145463/bfaaf235-ee0e-486f-8339-e1c7c76e4d1a

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[x ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[x] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[x] Responsiveness and internationalization.
[x] Strings and tooltips.
[x] End-to-end functionality of buttons, interactions and flows.
 I haven't tested it.

@shashank-23002 Tested locally and have some feedback:

The formatting for multiline tooltips (global time and poll buttons) is broken - visible from your screencap too.
When disabled on hover, the poll button should have its tooltip appear instantly, which does not happen now. If you read the code deleted in your commit, you'll find the code setting the instant hover delay and comments detailing this as well. Please be careful when deleting code -- take care to not accidentally remove any functionality.

I haven't really reviewed the code, but I'd like you to notice that we use snake_case and not camelCase for naming variables and functions in JavaScript. Make sure to follow these conventions in any code you add too.

Thanks @N-Shar-ma for reviewing the code .
I have implemented the required changes to improve the formatting for the multi-line tooltip.
Also fixed the implementation of the deleted code.
PTAL

Hey @sayamsamal @N-Shar-ma , it's been quite a while.
Just a gentle reminder if you could please go for a quick round of review.

Hi @shashank-23002, my apologies for the delay! Encountered some unexpected issues. I've taken a quick look at your code and will share feedback shortly. Thanks for your patience!

Sure @sayamsamal , I will look for other modifications till then. Hoping for a quick review.

@shashank-23002 I have left you some comments where we could definitely use some rethinking the logic. But to summarize, (1) we should avoid using click handlers to add tippy constructors since that leads to duplicate tippy isntances and (2) avoid duplication of code and instead use the pre-exisitng methods whenever possible.

The most major issue as pointed over here, still remains unaddressed. Please take another look at tippyjs.ts and the tippyjs docs, to see how we handle trigger and targets in tippyjs. In short, we shouldn't be using our own event listeners here, and instead pass the triggers to tippyjs.
const btn_tippys = tippy(".compose_control_button_container, .compose_control_button[data-tooltip-template-id], .compose_control_button[data-tippy-content]");

const singleTon = createSingleton(btn_tippys, {...})

Here is a screencast for you, so that you can better visualize the problem with your approach.
https://github.com/zulip/zulip/assets/82862779/0de4430d-dff7-4900-b61e-4411d092ac2f

@sayamsamal it's been quite a while. Just a gentle reminder for you to have a look.

@sayamsamal I think I have addressed the feedback and resolved conflicts. I am attaching a screen recording so that you can have a quick review -
https://github.com/zulip/zulip/assets/97145463/2af554f4-f073-41fa-a2ec-d09308fdfb96

@karlstolley I have addressed the feedbacks with one question.
PTAL


Allow download of attached files/media using API Token

When we get a results list from an API call we do not have the link to the attached files. Also, the attached files/media of the responses can only be downloaded today if the user is logged in.
Our proposal is to allow download of media files using the API token (use the route media/objects#show).

Hi @viniciuscb, similar to #671, have you tried our new OData API for this feature?

If the new API doesn't meet your need please do reply here.


Create iteration for single file mode for timelines

Top Meta is timeline Meta while down file is additional metas for each entry
Can work with Dropbox.


Multi part queries

Changes Unknown when pulling d7e9e81b2419e50eb473d15c49123ddca629b20c on multi-part-queries into ** on master**.


Support Aliyun DNS

#9382

@yasker fixed
thanks a lot for your comment

@yasker done
thanks again

Hi @janeczku
Can you review it as well?

@janeczku  done
thanks a lot!

@janeczku Does it look good now?

Thanks for changes @loganhz! In case you were intending to create a Aliyun DNS catalog template this would be a good starting point: https://github.com/rancher/community-catalog/tree/master/infra-templates/digitalocean-dns/3.


4.4. User guides. Next button.

The next button on the User guides page is leading to the User guides page again instead of the next chapter.

thanks! fixed


Unable to change the group for the docker socket because the group does not exist

I'm running the docker dind image like this:
docker run --rm -it --privileged docker:17.05.0-ce-dind

The very first line of the log is:

could not change group /var/run/docker.sock to docker: group docker not found

And as such, the permissions of the docker socket are completely owned by root. Is this a bug or intentional?
I will be volume-ing the socket to other containers. Those containers run as non-root but are part of the docker group.

We have not had a docker group in the docker images.  Just give a group id to dind to use: docker run --rm -it --privileged docker:17.05.0-ce-dind --group 1000.  Then you'll guarantee that the id is whatever value you expect and can give the id to the other containers that use the socket.

Ah, I didn't know about that argument.
I did something similar: making my own image with dind as a base.
FROM docker:17.05.0-ce-dind
RUN addgroup -g 2999 docker

The command you give saves me from needing to make a whole new image. Thanks for that!


TypeScript with "moduleResolution": "bundler" can't import the npm package

Steps to reproduce

Create a TypeScript project with "moduleResolution": "bundler"
yarn install imaginesdk or npm install imaginesdk
Try to import import { client } from 'imaginesdk'

Expected behavior
imaginesdk module should be imported with all type definitions
Actual behavior
TypeScript build throws an error ERROR(TypeScript)  Cannot find module 'imaginesdk' or its corresponding type declarations.
Additional context
Node.js - v18.18.0
TypeScript - v5.2.2
Vite - v4.4.11

Hey @Iverson, we couldn't reproduce the error, can you please share the sample codebase?

Hey @Iverson, we couldn't reproduce the error, can you please share the sample codebase?

Hey @m-shariq , I prepared a repo:
https://github.com/Iverson/imagine-sdk-issue/tree/main - branch with the released version of imaginesdk
https://github.com/Iverson/imagine-sdk-issue/tree/fix - branch that uses a version with my fix of imaginesdk
Just run these commands to test the build
yarn
yarn dev


Regarding favourite/recently visited rooms #11 and #12

I think the best option would be a drop down menu on the same level as the search bar, but on the left hand side. It'll use up that gap.
Only thing I am not sure of with this though is if the SB window is set to it's minimum distance where there is a single column of room links. The drop down menu would have to go underneath the search bar then maybe.
Thoughts?

We have a plan here for a persistent top bar, with room list top left.  Closing for now since this is mentioned in other issues.  Cheers Daniel!


perfdash: parse vanilla results correctly

What type of PR is this?
/kind bug
What this PR does / why we need it: density_vanilla items were incorrectly parsed as density.
Which issue(s) this PR fixes:

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

NONE

Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.:

/cc @marseel who is sig/scalability oncall AFAICT

/lgtm
/approve


Adding partID and a partition recognition process

Sometimes partition tables are corrupted, harddrives are damaged or disfunctional and pertinent data needs to be retreived. In certain situations when you're dealing with deployment strange stuff happens. Like sometimes people will want to get the entirety of their systems from one harddrive to another. Dealing with Windows without their new imagex system has many a time been a pain in my tuchus. But Nix(OS) however should be able to "automagically" facilitate this process easily. Now before you go on about how this filesystem supports this kind of redundancy and that kind redundancy remember: we want a general tool that can be used with Nix, to not only install and manage systems, but IMHO to easily reinstall them without much hastle (just a couple of nix files, a copy of the partition, and you're set).
So what I would suggest is adding a new file to /etc/nix/ that contains a store of information required to identify partitions, that is created or added to under the install or rebuild process. To coin a function like this is hard though, so I'll just call recognise:
fileSystems."/" = {
  device = "/dev/sdb3";
  fsType = "ext4";
};
fileSystems."/nix/store" = {
  device = "/dev/sda2";
  fsType = "btrfs";
  autoFormat = true;
};
fileSystems."/home" = {
  partID = "21d70c40-83f8-400c-b6ac-510fc7449dab"; # Just a reference point
};

As you see this setup does three things: it mounts root, it formats /nix/store partition on install/rebuild and the /home partition just has a partID. The reason the partID is even included (and maybe need re-coining) is to use it as an index number, really. The process should then check something like /etc/nix/partid.bin, through a secure process (in any case only nix and root should have access to it), for recognizable values or patterns that may identify this partition. In any case if you have a damaged partition we can start discussing how to facilitate an interactive "attempt to fix" process, or through declarations for that matter.
In any case it would make moving from system to system a breeze, which is again more of usability than functionality. But I'd still like to see something like this...

No action items @jagajaga


Unable to manually install this

Hello, I am unable to use Package control to download this project since I am behind a company proxy. I tried manually installing this but your instructions are unclear.
In point 3 you have "Download [zip package][12]" do you mean download the repo and rename the whole project? If yes I did it and it didn't work. Also, your screenshots have a program with extension .ccl but CCL programs have different extension i.e. .prg, will .prg files work with autocomplete and other features?
This looks really useful and I am excited to see if I can use it. Please let me know what am I missing.
Thank you

Looks like last updated was 2 years ago..  I'm very interested as well...


The API files get undeployed, when the NFS server goes down.

Description:
When the /server directory is mounted with NFS, all the APIs are undeployed from the gateway once the NFS Server goes down. This is due to the hot undeployment feature where missing files will be undeployed by running a scheduler task. The two scenarios where a single file gets deleted and the whole directory goes down due NFS failures should be distinguished and we should stop hot undeployment when the whole directory dissappears.
Steps to reproduce:

Mount the /server directory with NFS

Create an API and invoke.

Stop the NFS server and try to invoke the API.

Affected Product Version:
APIM 2.1

Fix in 1.6.1.wso2v41 https://github.com/wso2-support/wso2-axis2/pull/194


not able to select api version

library always uses v2.6 although v2.10 is now available

yes exactly. 2.10 should be minimal

https://github.com/conbus/fbmq/pull/35

any update on this? could potentially update the invocation of new page object to include version number and leave the default to 2.6

@wessilfie see #44 (release 2.4.2)


chore: Fixed prettier

[BOT] Preview available: https://embed.typeform.com/f9d061719b362ddbecf9382dc24856c414508774/embed.js

[BOT] Preview available: https://embed.typeform.com/6a321f908b4d885ee2f4bc524584dba38b9adb59/embed.js

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[TextField] Box shadow style gets cut off with the "filled" variant when zooming out Firefox browser to 80%

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Steps to reproduce 🕹
Link to live example:
https://codesandbox.io/s/lucid-microservice-lnj95v?file=/Demo.tsx
(2nd text field in the screenshot)

Steps:

Use "filled" variant of TextField
Open in Firefox or Safari browser
Zoom out to 80%

Current behavior 😯
Box shadow seems to get cut off on the input
Expected behavior 🤔
Expected to see the entire box shadow just like at 100% default zoom
Context 🔦
Trying to solve a rendering issue reported by a client
Your environment 🌎

npx @mui/envinfo
  Don't forget to mention which browser you used.
  Output from `npx @mui/envinfo` goes here.

This is caused by how Firefox renders fractional values that are a result of scaling. The similar issue occurs with native elements as well. The native input from your demo shows the same behavior when given different dimensions:

I suggest you submit a bug report to the Firefox team.

@michaldudak @be185058 Was there any workaround/fix found for this? Still seeing this issue with latest version of firefox.

Nothing I'm aware of.


mutually_exclusive_ranges gives non-deterministic results in certain situations where zero-length ranges are allowed

Describe the bug

The window functions used in the test only order by the lower_bound_column. If zero_length_ranges are allowed, it is possible for this to result in failure when it shouldn't.
Steps to reproduce

RECORD    STATUS    LOWER_BOUND            UPPER_BOUND
1         good      2021-12-01 00:00:00    2022-01-01 00:00:00
2         ok        2022-01-01 00:00:00    2022-01-01 00:00:00
3         bad       2022-01-01 00:00:00    2022-02-01 00:00:00

Expected results

This data should pass a mutually_exclusive_ranges test with configuration
lower_bound_column: lower_bound
upper_bound_column: upper_bound
zero_length_range_allowed: true

and fail mutually_exclusive_ranges test with configuration
lower_bound_column: lower_bound
upper_bound_column: upper_bound
zero_length_range_allowed: false

Actual results

Both of the following are recordsets ordered by LOWER_BOUND. The database could produce either of the following orderings of the records given above.
Ordering A:
RECORD    STATUS    LOWER_BOUND            UPPER_BOUND
1         good      2021-12-01 00:00:00    2022-01-01 00:00:00
2         ok        2022-01-01 00:00:00    2022-01-01 00:00:00
3         bad       2022-01-01 00:00:00    2022-02-01 00:00:00

Ordering B:
RECORD    STATUS    LOWER_BOUND            UPPER_BOUND
1         good      2021-12-01 00:00:00    2022-01-01 00:00:00
3         bad       2022-01-01 00:00:00    2022-02-01 00:00:00
2         ok        2022-01-01 00:00:00    2022-01-01 00:00:00

If the first ordering is returned, the dataset will pass the test. If the 2nd ordering is returned, the dataset will fail the test. Which ordering is returned appears to be non-deterministic, at least in Snowflake.
Also, at least in Snowflake, there is no guarantee that the same ordering will be used in the different window functions. I.e., it's possible for next_lower_bound to use Ordering B, but is_last_record to use the reverse of Ordering A.
Screenshots and log output

System information
The contents of your packages.yml file:
packages:
  - package: dbt-labs/dbt_utils
    version: 0.8.4

Which database are you using dbt with?

[ ] postgres
[ ] redshift
[ ] bigquery
[x] snowflake
[ ] other (specify: ____________)

The output of dbt --version:
installed version: 1.0.0
   latest version: 1.0.0

Up to date!

Plugins:
  - snowflake: 1.0.0

Additional context

Are you interested in contributing the fix?

Yes, I am preparing a PR to include the upper_bound_column as a second field in the order by, which would still not quite provide a deterministic ordering, but should provide orderings that are identical in every way that matters for the test.

This makes a ton of sense, thank you for the well-fleshed out backstory and test case! Off to have a look at the PR now.


Tune the github issue template a bit

[x] User relevant changes are recorded in CHANGES.txt
[x] Touched code is covered by tests
[x] Documentation has been updated if necessary
[x] CLA is signed

@seut pushed a fixup


proof of concept for authUser api response

Includes Kyles code for that adds the middleware that supports sending the authUser in the response.
Currently works in the search api and because it was manually added and does not seem to break anything the workspace api (get). We will need to evaluate adding to the other apis.

Progress on #437 and related to #438
Still need to implement for the rest of the apis.


Implement Tale.Music.Sync() for syncing actions with the music

A music sync action should be added in order to make syncing other actions with the music easier (e.g. the scroll in the credits scene).
This action should block the queue (like the Wait action) until the current music audio source reaches a given timestamp. This can be manually done with Wait. However, this action is more reliable than Wait for this purpose because it doesn't wait for a fixed amount of time. The wait time is dynamically adjusted. Therefore, regardless of how unstable the time of multiple actions between Syncs is, the syncing should be always accurate (except, of course, if 2 syncs with minor time differences are separated by actions that take a long time).
Tale.Parallel(Tale.Music.Play("MyMusic"));

Tale.Music.Sync(1, 15); // Wait until the audio source reaches 1 minute and 15 seconds

// do  stuff

Tale.Music.Sync(2, 37); // Wait until the audio source reaches 2 minutes and 37 seconds

// do some more stuff

...instead of the unreliable:
Tale.Parallel(Tale.Music.Play("MyMusic"));

Tale.Wait(75); // Wait 75 seconds

// do  stuff

Tale.Wait(157); // Wait 157 seconds

// do some more stuff

Added.


Controlling lifetime of Lua references

I'm looking for a way to invalidate Lua references so I can match their lifetimes to the CPP object they point to.
It looks like manually setting the metatable to nil is one option:
{
	Test test;
	test.a = 54;

	lua["test"] = &test;
	sol::userdata testRef = lua["test"];

	lua.script(R"(
		print(test, test.a)
	)");

	testRef[sol::metatable_key] = sol::nil;
}

// script error instead of dereferencing a dangling pointer
lua.script(R"(
	print(test, test.a)
)");

Are there any problems with doing this? Is there a better way?

You may be interested in this issue as well https://github.com/ThePhD/sol2/issues/1367


drop valueerror as this was from when 4bit required gptq

I still get: ValueError: .tois not supported for4-bitor8-bit bitsandbytes models.
Error is triggered from axolotl/cli/merge_lora.py L48 to axolotl/cli/__init__.py L147  -> accelerate -> transformers/modeling_utils.py#L2861


XFCE desktop slow start with various window managers

Description
When using certain window managers like bspwm and fvwm, xfdesktop and xfce4-panel are slow to start on login.
Steps To Reproduce

Enable XFCE desktop
Enable bspwm window manager
Log in with xfce+bspwm session

Expected behavior
XFCE panel and desktop should start immediately.
Additional context
I have fixed this problem by disabling the enableDesktop option and manually installing xfce.xfdesktop and xfce.xfce4-panel. I then execute them with the --disable-wm-check flag and everything works normally.
Adding the above flag resolves this issue when using non-xfwm window managers. I just don't know how to modify the option code and make a PR to do it myself. If that flag could be added to the xfdesktop and panel start when a non-xfwm window manager session is launched, it would fix this bug.
Notify maintainers
@teams.xfce.members: @romildo @muscaln

I'm having the exact same issue.  What exactly did you do for your workaround?  If I don't enable the xfce desktop and instead just download the panel and desktop manager, then launch them from a bare none+bspwm session (for example starting them from the bspwmrc) then the panel is broken - unable to add items and changes to settings don't persist after reboot etc.

I'm having the exact same issue. What exactly did you do for your workaround? If I don't enable the xfce desktop and instead just download the panel and desktop manager, then launch them from a bare none+bspwm session (for example starting them from the bspwmrc) then the panel is broken - unable to add items and changes to settings don't persist after reboot etc.

I put these lines at the end of my bspwmrc :
xfce4-panel --disable-wm-check
xfdesktop --disable-wm-check


Fixing sql server task publish profile issue

looks fine

Approved

https://github.com/Microsoft/vso-agent-tasks/issues/1336


ResultSetBuilder::buildModel(): Argument #2 ($indexName) must be of type string

ResultSetBuilder::buildModel() should accept array of indexes, when searching between multiple indexes
use JoliCode\Elastically\Client;

class ContentRepository
{
    public function __construct(
        private readonly Client $client)
    {
    }

   function getSearch(Query $query): Search  {
      $search = $this->client->getIndex('content-cs')->createSearch();
      $search->addIndexByName('content-de');
      return $search->setQuery($query);
  }
}

(new ContentRepository())->getSearch($query)->search();

[ResultSetBuilder::buildModel] Argument #2 ($indexName) must be of type string
Found problem here
https://github.com/ruflin/Elastica/blob/6d51cdbff2fb37aceac8b70adf9042ccb9be0aed/src/Result.php#L126
getParam() returns empty array when key not exists and yes, it is missing
https://github.com/ruflin/Elastica/blob/6d51cdbff2fb37aceac8b70adf9042ccb9be0aed/src/Result.php#L68

Hi, thanks for the detailed report.
Did you use something like the Elasticsearch filter_path option to have the "_index" field missing? Because it's always there by default.
The thing is, buildModel will always require the index name because it use it to map the source to the appropriate Model class.

@damienalexandre whow!  you are life saver, I am using filter_path
problem solved! maybe there should be exception when empty array is returned
https://github.com/jolicode/elastically/blob/a116efb71eb81576bf21efab32ea6d0bbaf9b57d/src/ResultSetBuilder.php#L70
if (!$result->getIndex()) {
throw new \Exception('Returned index is empty. Check your filter_path')
}

Yes that would be a good idea!
Lets keep this issue open to implement this (feel free to open a PR if you have the time by the way!).


Support for NIO::Monitor#interests=

One line note, otherwise this looks like a good start. Can you try to squash your commits?
https://www.atlassian.com/git/tutorials/rewriting-history/git-rebase

@UpeksheJay it looks like everything is almost working, however this looks like memory corruption: https://travis-ci.org/celluloid/nio4r/jobs/71805466

It(https://travis-ci.org/celluloid/nio4r/jobs/71805466) came after the
change did in monitor.c . The problem is it does not occur always. For some
time I did not saw this error until it occurred in "Travis".
But it seems like the error is caused by the line
ev_io_set(&monitor->ev_io, &monitor->ev_io.fd, monitor->interests);
ev.io_fd might be the reason for this. Before I used the constants instead
of io_fd, with the constants this error never occurred.
I search about the error, still no possible solution found.
Upekshe Jayasekera
Undergraduate
Department of Computer Science and Engineering
University of Moratuwa
Sri Lanka
On 21 July 2015 at 11:17, Tony Arcieri<EMAIL_ADDRESS>wrote:

@UpeksheJay https://github.com/UpeksheJay it looks like everything is
almost working, however this looks like memory corruption:
https://travis-ci.org/celluloid/nio4r/jobs/71805466
—
Reply to this email directly or view it on GitHub
https://github.com/celluloid/nio4r/pull/73#issuecomment-123170107.

@Upekshe could you try storing the file descriptor to the monitor struct, and then just doing &monitor->fd instead?

Yes. K I'll try that too. :)
Upekshe Jayasekera
Undergraduate
Department of Computer Science and Engineering
University of Moratuwa
Sri Lanka
On 23 July 2015 at 04:34, Tony Arcieri<EMAIL_ADDRESS>wrote:

@Upekshe https://github.com/Upekshe could you try storing the file
descriptor to the monitor struct, and then just doing &monitor->fd instead?
—
Reply to this email directly or view it on GitHub
https://github.com/celluloid/nio4r/pull/73#issuecomment-123899897.

Hmm, lots of problems in the Java spec on the latest build:
https://travis-ci.org/celluloid/nio4r/jobs/76679465

@UpeksheJay finally merged this. Sorry it took so long! Thank you for your contribution!


console.log left in build

Which version are you using?
1.24.1
Is the latest version affected?

[x] Yes
[ ] No

Which library are you using?

[ ] Vanilla JS (tsparticles)
[x] React (react-tsparticles)
[ ] Vue.js 2.x (particles.vue)
[ ] Vue.js 3.x (particles.vue3)
[ ] Angular (ng-particles)
[ ] Svelte (svelte-particles)
[ ] Preact (preact-particles)
[ ] Inferno (inferno-particles)
[ ] jQuery (jquery-particles)

tsParticles Configuration
  N/A

Describe the bug
A clear and concise description of what the bug is.
console.log() production build
I dont know if it was to be expected that the console.log() was left in the production build. I only noticed it when testing my app.
To Reproduce
Steps to reproduce the behavior:

Go to https://www.adamalston.com/
Inspect Element
Console
See particles object

Expected behavior
Remove console.log() from production build
Screenshots

Desktop (please complete the following information):
N/A
Smartphone (please complete the following information):
N/A
Additional context
I do have an additional question. It seems like the particle speed has been increased significantly between 1.22.x and 1.24.x. What is the reasoning/explanation behind this?

Thanks for reporting that @adamalston, I didn't noticed that. The 1.24.2 is now live without the console.dir. The associated PR is building but the version is live. You can safely update to that 😉

I had a question at the end of the issue that I was hoping someone could answer:

It seems like the particle speed has been increased significantly between 1.22.x and 1.24.x. What is the reasoning/explanation behind this?

Ops I didn't noticed that.
I wrote an article about that: https://dev.to/matteobruni/tsparticles-1-23-0-released-4hn5
I'll explain it too:
The formula was wrong, it used random values that makes particles with a speed of 6 have a lower visual speed since it was calculated with a random number multiplied by the horizontal and vertical speed. This could lead to an horizontal speed of 4 and a vertical of 0.5 and the 6 of speed makes no sense.
Emitters with that formula also appears rectangular instead of circular and it was really ugly.
So starting from 1.23.0 the new formula uses length/angle Vector formulas and the length is kept in the randomization process, so only the angle changes. This creates emitters with circular shape and a more uniform speed effect.
Starting from 1.24.0 instead, you can set a { min: 0, max: 6 } value to speed (also size.value, opacity.value and rotate.value) to have a different length (speed) too, but this is still a more controlled way of randomizing speed since you can't have slow particles in a fast configuration (example a { min: 25, max: 50 }, before it could happen).
For those using top, left, right and left move directions nothing has changed since the length is still ±1.
I wanted to release it in the 2.0.0 version initially, but I found it unnecessary release a major version just for a speed algorithm. What is coming with the current 2.0.0  version is a massive architecture rework where you can only include what you need for your configuration and nothing more, but it's not ready for production yet.
I'm releasing faster because I'm detaching some ready features from that version that could be brought here easily.
In the end I thought that the new speed formula will bring more benefits than issues.

The new values are:

new x
new y
old x
old y

top
0
-1
0
-1

top right

0.5
-0.5

right
1
0
1
0

bottom right

0.5
0.5

bottom
0
1
0
1

bottom left

-0.5
0.5

left
-1
0
-1
0

top left

-0.5
-0.5

Thanks for the explanation, the math behind the change seems interesting. Also sounds like performance will see an improvement due to the circular emitter and speed being more linear - not that it was bad before.


Denoise-Bilateral Variants

Hey,
since its not mentioned in the latest commit. Whats the reasons for the different Denoise-Bilateral Variants?
Do you have already found a use case for the different methods? e.g one works well with dark scenes etc?

The bilateral mode filter can introduce severe aliasing if set too high. The bilateral mean introduces a lot of blur if set too high but has no aliasing issues. The bilateral median is a good compromise between the two but is much slower. (The median is the hardest central tendency to find of the three.)
If your scene is always dark you can try to set the INTENSITY_POWER_CURVE to 0. It is not 0 by default since lines in anime are mostly always black and it causes too much smoothing.

Here's a difference between Median and Mode with Upscale_CNN_M_x2_Deblur and Thin/Dark lines (all default settings except line intensity slightly reduced).
Although, Median is much heavier so it's probably fairer to compare against Mode + L deblur.
Both comparisons median is on left.
Here's what no shaders and MadVR looks like.
Source: Funimation(Tokyo Majin) - 480p so easier to compare but it's still 8.1MB/s...

Thank you both @bloc97 and @Jules-A  for the comparison!
So if speed isn't an issue (or hardware) Median is the way to go i assume.
I am trying to figure out the best settings for a variety of anime. Kind of like profiles of shaders for different kind of anime.

Thank you both @bloc97 and @Jules-A for the comparison!
So if speed isn't an issue (or hardware) Median is the way to go i assume.

If speed isn't an issue then I think you are better with the Anime4K_3.0_Upscale_CNN_L_x2_Denoise variants but it's too heavy (even the M) for my system so I haven't done any real detailed comparisons. It runs fine for 480p anime but struggles on anything higher.

Oh yeah sure. I was just curious about the Denoise Bi. Since i upscale from 1080 to 4K (with my python wrapper which i may publish soon). I always use the XL variants (ryzen 3600, gtx 1060) to encode the file (i dont do live watching). I think in encoding its not so much an issue since you dont have to watch it at the same time. The only issue when encoding is RAM and VRAM. I have run into cases where are Movie Remux upscale to 4K got killed by the OOM Killer of Linux before i ran out of 16GB RAM.


Missing content in recommendation 1.3

Is the recommendation 1.3 a copy-paste error from one of the storage services? As ACI hosts a compute layer, it can certainly be used to host web applications and as such, it would be meaningful to discuss options to protect it, in the spirit of the Baseline.
For example, I would presume this could include recommending the use of Azure Front Door WAF, Azure Firewall, or other options.
Here's my proposal, copied from the App Service ASB:

Secure an internet accessible app in an ACI by deploying a Web Application Firewall-enabled device in front of your app. Drive all application traffic outbound through an Azure Firewall device and monitor the logs.

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

ID: 653e1999-4419-0d8e-8fc1-2162774178f4
Version Independent ID: 6c828bb1-3227-ca9a-f2c3-627105d328ce
Content: Azure security baseline for Container Instances
Content Source: benchmark/azure/baselines/container-instances-security-baseline.md
Service: container-instances
GitHub Login: @msmbaldwin
Microsoft Alias: mbaldwin

@karlgots - Thanks for brining this to our attention. We will investigate it further and update you shortly.

@karlgots - Thanks again for bringing this to our attention. We have updated the Doc and it will be live once its merged.  We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


dolphinEmuMaster ui doesn't match the rest of my desktop

Describe the bug
A clear and concise description of what the bug is.
I have upgraded from 20.03 to 20.09-pre and the dolphinEmuMaster qt ui doesn't look right anymore. See Screenshots section for how the ui looks. See Expected behavior for how the ui should look.
To Reproduce
Steps to reproduce the behavior:

Upgrade to 20.09
Install dolphinEmuMaster
Run dolphin-emu-master or dolphin-emu and look at the ui

Expected behavior
A clear and concise description of what you expected to happen.
I would like the ui to look like the rest of my desktop. Like this:

Screenshots
If applicable, add screenshots to help explain your problem.

Additional context
Add any other context about the problem here.
I am guessing that the issue is in top-level/all-packages.nix dolphinEmuMaster is built with qt5.callPackage instead of libsForQt5.callPackage, because when I build with the latter the ui looks fine.
Notify maintainers

@MP2E @ashkitten or @ttuegel in case they are able to help :man_shrugging:
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.

system: "x86_64-linux"
host os: Linux 5.4.70, NixOS, 20.09pre-git (Nightingale)
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.3.7
channels(root): "nixosUnstable-20.09pre242714.a31736120c5"
nixpkgs: /nix/store/9pj63qfmvczn94wdas3vk3802hjm51m7-source

Maintainer information:
# a list of nixpkgs attributes affected by the problem
attribute: dolphinEmuMaster
# a list of nixos modules affected by the problem
module:

It looks fine to me on 007126eef72271480cb7670e19e501a1ad2c1ff2 - current nixos-unstable. https://github.com/NixOS/nixpkgs/pull/101369 doesn't affect it at all.

I completed more than 3 games on stable channel with unstable dolphin-emu. Never ran into this issue. Also, your latest Dolphin 5.0 - 11824 is more than 10 months ago. Is it still an issue today?

I completed more than 3 games on stable channel with unstable dolphin-emu. Never ran into this issue. Also, your latest Dolphin 5.0 - 11824 is more than 10 months ago. Is it still an issue today?

Sorry. Yes, this was fixed. Thanks!

Sorry. Yes, this was fixed. Thanks!


chore(deps): update debug to 4.3.4

This PR updates the remaining examples to use<EMAIL_ADDRESS>(the latest version, published on Mar 17, 2022) instead of the deprecated<EMAIL_ADDRESS>It follows on from https://github.com/cypress-io/github-action/pull/939.
Issue
<EMAIL_ADDRESS>which was published on May 29, 2020 is deprecated.
$ npm view<EMAIL_ADDRESS>deprecated
Debug versions >=3.2.0 <3.2.7 || >=4 <4.3.1 have a low-severity ReDos regression when used in a Node.js environment. It is recommended you upgrade to 3.2.7 or 4.3.1. (https://github.com/visionmedia/debug/issues/797)

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: MikeMcC399:x: cacieprinsYou have signed the CLA already but the status is still pending? Let us recheck it.


Made ui-router noImplicitAny compliant

This resolves issue https://github.com/angular-ui/ui-router/issues/2693.
I know the issue had been marked as WontFix but i've created this PR given that users of the latest router will have to be on TypeScript 2.0 (which hasn't yet shipped) and have specific a tsconfig option set in order to use the ui-router (usage of noImplicitAny is pretty commonplace). That's a real shame and a slight barrier to entry.
For the most part it was just a case of taking a trip through the code and adding to method signatures an any annotation per parameter where there is no annotation at all at present.  If you decided to upgrade the types to something more specific over time that would be completely open to you.
Whilst working on this I noticed there were some invalid references to the stateEvents.ts file in files.js and Gruntfile.js.  I've fixed these so they point to the legacy subfolder.

@johnnyreilly I haven't forgotten about this PR and I appreciate the interest and the work that you've done.
I'm not going to merge it, based on the following reasons:

Function parameter typings are now used to generate our API docs.
Lack of a type on a parameter implies that the documentation for that function isn't complete
Existence of a type on a parameter implies that the function has been documented.

Adding : any type to functions that can't actually take any will end up in the API documentation, and will be misleading to end users.

However I am open to making UI-Router noImplicitAny safe by adding explicit types to all parameters.  This would require each method to be analyzed and the correct types added.
This is something I will commit to tackling as a low priority effort, as I'm working through other issues.

Thanks for the detailed explanation @christopherthielen. I appreciate that.


Privacy button cut off in settings if there are additional panels

Issue Summary
Privacy button is cut off at the bottom if there are more panels after it. This is in part because of .field-row's negative margin bottom, but removing that still leaves it without white space between the button and the next panel.

Steps to Reproduce
Add another FieldPanel to the settings_panels after the privacy panel

I have confirmed that this issue can be reproduced as described on a fresh Wagtail project: No

Technical details

Python version: 3.6.4
Django version: 3.0.7
Wagtail version: 2.10.2
Browser version: Firefox 82, Chrome 85

Fixed in #6454


Board Report: Show meetings with the Report on Agenda

@reginafcompton Right now, the meetings an Item has been on the Agenda for only show up after the meeting, since this is based on Minutes/Histories. Find how to tie items to meetings before the meeting happens. Maybe a separate meetings section on the Board Report page which lists the Agendas it will be on.

Addressed in #348


How to use component in assets

How to use component listed in assets -> example folder

They are actually used for code viewer in pro version.
You can use the component code from assets folder.

They are actually used for code viewer in pro version.
You can use the component code from assets folder.

I am using free version, Can you tell me how to achieve  the same?

code viewer is just for showing code of a component, so that user can copy the code from demo.

Hi where is the table component and error component ?
demo its different from code
can you explain


fix few signed/unsigned warnings

Can one of the admins verify this patch?

@tensorflow-jenkins test this please

Jenkins, test this please.


Windows carriage returns causing failure at run

I've got a script in Jenkins that runs auditjs as the last build step. Recently it started failing saying
/usr/bin/env: node
: No such file or directory
Apparently this is the first line of the audit.js file. The support engineer hosting the devops services said this is because there is Windows carriage return after node (notice the split lines in the error output), and the system can't find program named node\r.
I haven't heard of such issue before. In our support engineer right or could there be some other problem.

Heavy. It is possible, as I occasionally do work on my windows machine. I will dos2unix the whole thing and push a clean version (also do some testing to see if I can duplicate the issue).

It seems that the issue is that some Linux systems use node as the executable names, while some use nodejs, and we need to prepare for both. On your system the script was not finding node, hence the error message -- the \r only served to corrupt the error message.
I happen to have testing systems which represent both. I pushed a new version to npm and it seems to work on both my systems now. I am closing the issue, but definitely reopen if you still have issues on your side.
Thanks for the report!

Hi
We upgraded to auditjs v2.1.4 and now suddenly I'm seeing an odd error.
/jenkins.plugins.nodejs.tools.NodeJSInstallation/Node.js_7.7.1/lib/node_modules/auditjs/audit.js:120
var output = ${programPackage.toString().split('.json').slice(0, -1)}.xml;
^
SyntaxError: Unexpected token ILLEGAL
at Module._compile (module.js:439:25)
at Object.Module._extensions..js (module.js:474:10)
at Module.load (module.js:356:32)
at Function.Module._load (module.js:312:12)
at Function.Module.runMain (module.js:497:10)
at startup (node.js:119:16)
at node.js:902:3
Totally clueless why this happens. Any ideas?

Whoa. That's odd. I'm checking into it now...

My best guess at this moment is your installed version of node is quite old. The new code that was added tried to detect the appropriate nodejs command, first checking nodejs, then checking node. It is done through this command (line 2 of the main script)
$(command -v nodejs || command -v node)

Try running the following commands and send the results:
nodejs --version
node --version

I expect one of the commands likely fail in your case (hence the original issue), but we should get a version number from the other.

nodejs --version
v0.10.25
node --version
v7.7.1

Alright. Out of curiosity, is this the same machine as the original problem, or a different one? On the machine I was able to reproduce the original problem the issue was that node was installed under the name "nodejs" instead of "node". My other machines have node installed under the name "node" instead of "nodejs".
It appears on your current machine node is installed under both names, but the nodejs version is quite out of date.
As the "official" node.js binary name is node I can check there first. This has two possible issues:

Someone else might have a version problem in the other order as you
In theory a conflicting package on debian might have taken the "node" name and this will break the script if I check that one first.

The second issue should be rather rare, since I believe that the conflicting package is a hamradio package -- its likely fairly rare that it should crop up.
As I do not know which situation is more likely with regards to node or nodejs being out of date, perhaps I will just swap them around and see if there are complaints.
A proper fix would be to run both executables and do a version check (and ensure they are node) and choose the best one. This will be much more time consuming.
I think as a quick fix I will swap the order of the check and hope for the best, but leave this issue open pending a more complete fix. I will let you know when a new version is available with the hack/fix.

Ha. Alright, turns out the original problem for you really was likely the ^M character alone, since my yet a different test machine was able to reproduce your issue. Very curious that some machines seems to care about ^M and some don't. I was not solely an issue of a missing node command (as one of my other machines was inferring).
So now I look for node first and use that if possible, then fall back to nodejs if required (as is needed on some Debian/Ubuntu machines). This will still fail if someone has installed the node package on some old Debian/Ubuntu installations (which is not the same as nodejs) since the executable is something else entirely. This will hopefully be a rather rare situation.
sigh
Version 2.1.6 has been pushed to npm. It appears to be working on several of my test machines which have different situations set up. It will fail if the non-nodejs node is installed or if the nodejs node executable is an old version, but there is not a heck of a lot I can do quickly about that. I will need to write a preamble to the script which can identify the installed node version(s) and choose the best one, and I am disinclined to do that unless someone runs into a problem, and even then I will be somewhat angsty ;)

To your earlier question: yes, I ran it on the same machine. It appears to have both node and nodejs installed, with very different versions (nodejs being very old, as you said). Node version can be changed from the jenkins configuration GUI, but nodejs can't. The admins could if they bothered.
Just re-ran the builds having auditjs on, and it works now! I hope it doesn't break in other people's environments.
SO, BIG THANKS TO YOU!

No one has complained. It looks like we have a reasonably acceptable fix, so I will close the issue.
Thanks for the help.


I2C not working on ESP32-S2 TFT

Describe the bug
I2C devices are not discoverable on ESP32-S2 TFT boards.
I have tried numerous different I2C sensors including several BME280 I2C sensors .
Manually trying to add a bme280 doesn't work either as it can't see the I2C device
Arduino board
ESP32-S2 TFT
To Reproduce
Connect I2c Sensor

Go to devices / I2C Scan
Click on Scan Again
No devices will show
Scan show's complete

Expected behavior
expecting to see the i2c device that is connected's address
Which components are connected to your device
none, unable to add any due to i2c not working
Screenshots

Desktop (please complete the following information):

OS: win10
Browser chrome
Version 105.0.5195.127 (Official Build) (64-bit)

Additional context
Tried with different ESP32-S2 TFT boards and all had same problem.

any update on this?

This issue will be looked at this week as part of the batch of issues tagged for beta 52
https://github.com/adafruit/Adafruit_Wippersnapper_Arduino/milestone/3

Testbed I2C scan works on my board

Default port I2C scan: 0x0, 0xB, 0x38, 0x77,

WipperSnapper I2C scan is same as issue

Hi @isomorphic85  - I've resolved this issue. You do not need to update the firmware! There was a mismatch between the I2C pins on the definition we use and the actual I2C pinout. This has since been fixed.
Please try deleting the ESP32-S2 TFT Feather from your Adafruit IO devices page.  Please let me know if it's working for you. I tested it as well with a similar setup:
I2C scan with BME280 and AHT20 attached to ESP32-S2 TFT's I2C Port:

BME280 attached to ESP32-S2 TFT

That worked!  Awesome.
Thank you

side note- any support for images or data on the screen in the works through wipper?

Like on a TFT? Yes, it's on the roadmap.


Fix typo

I was checking out your project readme and noticed a typo. Figured I'd submit a quick Pull Request through the GitHub GUI to fix it.

Hello @blakeNaccarato, thank you for the contribution!


Possible IndexOutOfBoundsException in StateRepresentation.tryFindLocalHandler

In my case when a guard condition is false the 'actual' array designed to have the one and only TriggerBehaviour is empty.
Perhaps this needs to be added to the code:
if (actual.isEmpty())
{
return null;
}

Apparently already fixed in the return statement of tryFindLocalHandler
return actual.isEmpty() ? null : actual.get(0);


"table of contents" header needs panel-heading styling

Currently, e.g. openssl has the red border just around the "table of contents" text, rather than the whole box. This appears to be because that panel uses details and summary tags rather than the div structure of the other panels.
Great upgrade!

Could you attach a screenshot of what you see please? This is what I see (and what was intended):
collapsed: 2017-01-18-230348_731x625_scrot
expanded: 2017-01-18-230356_728x933_scrot

Also, which browser version and operating system do you use?

Sorry, I forgot archive.org doesn't have screenshots, archive.is does indeed display as intended.
I'm using firefox-esr on Jessie, 45.6.0esr-1~deb8u1 and I created a fresh profile to double check my addons weren't affecting it:

It appears that the details expandable behaviour was only added in Firefox 47.
Comparing the styling with Chromium inspector, the following CSS is listed as "user agent stylesheet" and make it look like the other panels without affecting chromium:
screenshot from 2017-01-18 22:48:23
summary, details {
  display: block;
}

Thanks for the details. I can reproduce the issue with firefox-esr, and indeed adding the display: block seems to fix this issue. The fix will be live once we re-render the repository (within the next 24h).


Fix Java Nightly Snapshot Failures

A quick fix on the java nightly snapshot validations. The Leaderboard and Gamestats examples are no longer accept 'output' arguments.
R: @pabloem
cc: @alanmyrvold

Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

It will help us expedite review of your Pull Request if you tag someone (e.g. @username) to look at it.
 I'll merge after precommit returns .


Use f-strings in integrations starting with "A"

Breaking change

Proposed change

Replaces string.format() with f-strings enforcing the new style guidelines.
This PR only affects integrations that start with the letter A for easier and quicker reviews.
Type of change

[ ] Dependency upgrade
[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New integration (thank you!)
[ ] New feature (which adds functionality to an existing integration)
[ ] Breaking change (fix/feature causing existing functionality to break)
[x] Code quality improvements to existing code or addition of tests

Example entry for configuration.yaml:

# Example configuration.yaml
N/A

Additional information

This PR fixes or closes issue: fixes #
This PR is related to issue:
Link to documentation pull request:

Checklist

[ ] The code change is tested and works locally.
[x] Local tests pass. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.
[x] I have followed the development checklist
[x] The code has been formatted using Black (black --fast homeassistant tests)
[ ] Tests have been added to verify that the new code works.

If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated for www.home-assistant.io

If the code communicates with devices, web services, or third-party tools:

[ ] The manifest file has all fields filled out correctly.
Updated and included derived files by running: python3 -m script.hassfest.
[ ] New or updated dependencies have been added to requirements_all.txt.
Updated by running python3 -m script.gen_requirements_all.
[ ] Untested files have been added to .coveragerc.

The integration reached or maintains the following Integration Quality Scale:

[ ] No score or internal
[ ] 🥈 Silver
[ ] 🥇 Gold
[ ] 🏆 Platinum

@MartinHjelmare thanks for the suggestion, looks cleaner now 👍


UPSTREAM: : enable TCP_USER_TIMEOUT on client connections

it allows the client to detect broken connections to the api server more quickly than the default TCP timeout of 15 minutes.
requires https://github.com/openshift/library-go/pull/967

/assign @sttts

/lgtm
/approve


Deep linking issue on Android

I believe I am running into an issue with linking on Android. It seems to be the #state=... portion that is throwing react-navigation off from picking up the link. Testing deeping linking works via CLI: npx uri-scheme open io.act.auth://CreateCheckin --android.
Heres is my linking config:
prefixes: ["io.act.auth://"],
config: {
  screens: {
   CreateCheckin: {
     path: "CreateCheckin"
    }
  }
}

Here is the redirect URI that react-navigation is trying to parse:
io.act.auth://CreateCheckin#state=978f7de7-42a1-424d-bacd-53a746c030ab&session_state=0436a428-d734-490f-a3bc-812712458efd&code=6627fcff-abac-4139-99b7-54ed2d2a9fee.0436a428-d734-490f-a3bc-812712458efd.011ac3f2-b54f-419d-8108-6b1b67bd3f12

What would be the proper config to match the #state... portion?
Originally posted by @joe307bad in https://github.com/react-keycloak/react-native-keycloak/issues/27#issuecomment-857341555

Hi @joe307bad.
We set it up this way and it worked perfectly.
I hope it helps you
import 'react-native-gesture-handler';
import React, { useRef, useEffect } from 'react';
import { StatusBar } from 'react-native';
import { useLinking, NavigationContainer } from '@react-navigation/native';
import { RNKeycloak, ReactNativeKeycloakProvider } from '@react-keycloak/native';

import Routes from './routes';

const keycloak = new RNKeycloak({
  url: "my_url_keycloack",
  realm: "name_my_realm",
  clientId: 'my_client_id',
});

const App: React.FC = () => {

 const ref = useRef(null);

  const { getInitialState } = useLinking(ref, {
    prefixes: ['my_prefix://'],
    config: {
      screens: {
        signinkeycloak: 'signinkeycloak',
      },
    },
  });

  const [isReady, setIsReady] = useState(false);
  const [initialState, setInitialState] = useState<any>();

  useEffect(() => {
    getInitialState().then((state) => {
      if (state !== undefined) { setInitialState(state); }
      setIsReady(true);
    });
  }, [getInitialState]);

  if (!isReady) return null;

  return (
    <ReactNativeKeycloakProvider
      authClient={keycloak}
      initOptions={{
        redirectUri: 'my_prefix://signinkeycloak',
        inAppBrowserOptions: {
          modalPresentationStyle: 'fullScreen',
          enableBarCollapsing: false,
          modalEnabled: true,
          ephemeralWebSession: true,
          dismissButtonStyle: 'ok',
        },
      }}>
      <NavigationContainer initialState={initialState} ref={ref}>
            <StatusBar barStyle="default" backgroundColor="transparent" />
            <Routes />
      </NavigationContainer>
    </ReactNativeKeycloakProvider>
  );
};

export default App;

Thanks @ oneil-marcelo! What does your Routes component look like?

The Component was like this
import React from 'react';
import { createStackNavigator } from '@react-navigation/stack';

import SignInKeycloak from '../screens/SignInKeycloak';
import OthersPage from '../screens/OthersPage';

const Routes: React.FC = () => {
  const { Navigator, Screen } = createStackNavigator();

  return (
    <Navigator>
      <Screen name="signinkeycloak" component={SignInKeycloak} />
      <Screen name="othersPage" component={OthersPage} />
    </Navigator>
  );
};

export default Routes;

Thanks a ton @oneil-marcelo, I did get it working it seems.
I added this as my valid redirect URI in keycloak:
io.act.auth://io.act.host/CreateCheckin/*

Then I had this as my ReactNativeKeycloakProvider:
<ReactNativeKeycloakProvider
      authClient={keycloak}
      initOptions={{
        redirectUri: 'io.act.auth://io.act.host/CreateCheckin/'
      }}
    >
      <Root.make />
</ReactNativeKeycloakProvider>

And then finally my linking config:
linking: {
    prefixes: ["io.act.auth://io.act.host/"],
    config: { screens: { CreateCheckin: "CreateCheckin/:id" } }
}

Excellent, congratulations @joe307bad !


Add foreign key to ActiveRecordProviderSQLite::makeTable() INTEGER switch case

That method now creates foreign keys correctly, no additional fix required.


SAK-46301 Search: include content in indexing ops

https://sakaiproject.atlassian.net/browse/SAK-46301

@adrianfish cheers!

@axxter99 No problem! You were right about those bananas all along.

@ern I've updated this now. The issue was with the various es queries used in the post processing phase

See #10242 which solves this issue


Unknown logo missing for the Eco-Score (and possibly Forest Footprint)

What
Unknown logo missing for the Eco-Score (and possibly Forest Footprint)
Blocking

[ ] #500

Screenshots

It's a tricky one.
Most of the time, for an attribute, we've got a value: e.g. for nutriscore, the value can be ABCDE or I don't know (value="I don't know").
Here it's different: it's not that "we know that we don't know", it's "we don't know at all" (in code terms: null)
Already evoked with @stephanegigandet: should we display that info? I said yes, because I didn't want the end-user to say: "wait a minute, why don't they display the ecoscore for this product?".
But as we usually display what we get from the server, and in that case it's nothing at all for ecoscore, we currently display the default icon (red question mark) with the "we don't know at all" text ("No data").
I guess it would make sense to display at least the "we don't know" icon, while keeping the "No data" text.
@teolemon Do you agree with that?

I just opened: https://github.com/openfoodfacts/openfoodfacts-server/issues/4835
My belief: we can solve it in the app

[ ]  visually (with slightly different logos with ? vs grey for instance), and
[ ] clear text in the attribute as to why it's not computed.  "Nutri-Score is not applicable to alcohol" "We miss some nutrients to compute the Nutri-Score" "We need a category to compute the Nutri-Score".
[ ] Eventually, when clicked, they should open an edit modal to photograph/add the missing information. When the users hits save after adding the data, the logos transforms from grey to colored.

@monsieurtanuki : For the Eco-Score and the Forest Footprint, if the attributes are not returned at all (it happens only when the user is not in France), then we should not display anything. It's not "no data", it is that those attributes apply only for France at this time.

when clicked, they should open an edit modal to photograph/add the missing information.

@teolemon : yes, but this should happen in the product page, not on product cards. Clicking anywhere on the product card should open the product page. If something different happens when I click on a different part of the card, it would be very confusing.

@stephanegigandet I can code the "if null, don't display". I understand the uselessness of having the same red question mark for all products.
The problem is that we asked end-users about the ecoscore in the preferences - it's a source of confusion:

should we hide the ecoscore from the preferences when the country is not FR?
that would also mean that we would have to maintain a hard-coded list of displayed/hidden attributes for each country exception, if for instance we have not only this problem with ecoscore/FR but also "forest" for other countries

Regarding product change, I agree with @stephanegigandet: that could be a bit complicated to ask the end-user to fix ingredients on the fly while on a scanning bulk action.
I have a suggestion: the concept of "meta-lists" (or "auto-lists"?), computed automatically from all the products on the user's smartphone.
Obvious cases: "all the products with expiration date in the next 2 days"; "all the products that need to be fixed".

that would also mean that we would have to maintain a hard-coded list of displayed/hidden attributes for each country exception, if for instance we have not only this problem with ecoscore/FR but also "forest" for other countries

The best thing would be that we don't return those attributes when the app queries /api/v2/attribute_groups, unless the cc parameter is set to "fr".

Auto lists are an interesting idea. A bit like how Google Photos creates albums based on time and location. I was thinking about that for scans, we could have lists like "Scans from February 8th".
Of course the challenge is going to be on how to present those lists, so that users get only the lists they need.

Except that in the query we specify the language, not the country: https://world.openfoodfacts.org/api/v2/attribute_groups?lc=$languageCode
The obvious use-case for auto-lists is "soon expired" - as the white bear knows ;)

@teolemon I think you're right, https://github.com/openfoodfacts/openfoodfacts-server/issues/4835 could be the solution. Possibly in combination with @stephanegigandet's suggestion about attribute groups and country code.

Ok, assuming it's a rare exception, we can say: "if the current country on the smartphone is not FR, let's not display EcoScore and ForestFootprint anywhere in the app".
In addition to that display change, there's also an impact on the match score: the EcoScore being set to "important" in the default (reset) preferences, we should also say: "whatever importance we gave to EcoScore or ForestFootprint, if the country is not FR, consider they are not important".
I can do that.

@teolemon This is what I got today with "your" Koenigsbier: it looks like we could close this issue, couldn't we?

yes, we can


Updates

Description

e.g. Fixes #(issue)
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update

How has this been tested?

[ ] I have run bash ./tools/build.sh && bash ./tools/test.sh (at the root of the project) locally and passed
[ ] I have tested this feature in the browser

Test Configuration

Browerser type & version:
Operating system:
Bundler version:
Ruby version:
Jekyll version:

Checklist:

[ ] My code follows the Google style guidelines
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] Any dependent changes have been merged and published in downstream modules

Ignore this, woops :)


Generate p12 file for Elastic/OpenSearch

This pull requests will overwrite the script that currently exists for cassandra. Is it possible to change the name of the file to indicate it is for elastic/opensearch?


CCG Custom Plugin Returning Error (The remote procedure call failed.)

Describe the bug
Setting up custom ccg plugin on my local machine to run windows containers as non-domain joined failing with RPC error.
To Reproduce
Steps to reproduce the behavior:

Downloaded gmsa-ccg-plugin project
Build solution in release mode.
Register plugin using following script install-gmsaplugin.zip
Run docker command
docker run --security-opt "credentialspec=file://servereps_gmsaplugin.json" --hostname webapp01 -it mcr.microsoft.com/windows/server:ltsc2022 powershell
Output
 docker: Error response from daemon: hcs::CreateComputeSystem f8a7b59bca075e1afea0a808f725bf1468f8ad163d1889bc98c1dd3775b8302f: winapi error #3489792023.

Expected behavior

Windows container should authenticate with remote domain and start up.

Configuration:

Edition: Windows 11
Base Image being used: mcr.microsoft.com/windows/servercore:ltsc2022, mcr.microsoft.com/windows/server:ltsc2022
Container engine: docker
Container Engine version 27.2.0

Additional context

@srinivas-kujala Thanks for calling our attention to this error. The issue has been brought up to the team that owns CCG. We'll update this when we have something to share.

@ntrappe-msft, thank you for acknowledging my request. Awaiting next steps.
Thanks

@ntrappe-msft , do we have any update on this issue from CCG team?

@srinivas-kujala Hi, I just pinged the thread again. I'll share an update when they respond. Thanks!

Would you mind opening a ticket with Microsoft support? The CCG team prefers to go through the official mechanisms.

Huh, that's weird? Can I ask what browser you're using? I just tested it on Safari + Firefox and it works for me. You might want to try and (1) clear your cache or (2) login to another Microsoft page, then retry the Services Hub one.

@ntrappe-msft , I've cleared cache tried chrome on windows laptop and safari on mobile. No luck

I am currently experiencing a similar issue with an AKS cluster configured with gMSA and utilizing the pre-installed aksasv plugin. Upon connecting to the Windows node via RDP and examining the logs, I observed the following output. It seems related to @srinivas-kujala issue.
I have confirmed that the GMSACredentialSpec (or credentialspec) configuration is accurate, with particular attention to the PluginInput field within the HostAccountConfig section.
Edition: Microsoft Windows Server 2019 Datacenter
Base Image being used: mcr.microsoft.com/windows/servercore/iis:windowsservercore-ltsc2019
Also @ntrappe-msft I also cant access this page to raise a ticket

@ntrappe-msft , looks like we are unable to raise the ticket. Is there anything else we can do to get some help. I see others are facing same issue. The error message doesn't give any clue for us to even troubleshoot from our end we are stuck.
hordurmarhafsteinsson
Raphael-kainos

I would normally tell you to use the Support troubleshooter but that isn't working either 🙃. Give me a few days to figure out how to raise this Issue & what's going wrong with our Support page.

I would normally tell you to use the Support troubleshooter but that isn't working either 🙃. Give me a few days to figure out how to raise this Issue & what's going wrong with our Support page.

Services Hub one Started working yesterday for me, I have raised a ticket.

Oh sweet! If you don't get a response in a week, let me know. I can try to put more pressure on that ticket.

@ntrappe-msft, should I be raising another ticket?

@ntrappe-msft , is there any update on this.

@srinivas-kujala You could raise another ticket pointing to this issue if you want to bring more attention to it. I haven't been contacted by any of our support teams so I don't know the status of the ICM.


[BUG] envvar should not show up for "nested" config

/// The configuration related to Umami analytics. More information
/// [here](https://umami.is/docs/tracker-configuration).
#[derive(Debug, Serialize, Deserialize, Clone, Config)]
#[config(rename_all = "snake_case", env_prefix = "FRONTEND_UMAMI_")]
pub struct FrontendUmamiConfig {
    /// For example: https://umami.is/script.js
    pub script_url: String,
    pub website_id: String,
    pub domains: String,
}

#[derive(Debug, Serialize, Deserialize, Clone, Config)]
#[config(rename_all = "snake_case", env_prefix = "FRONTEND_")]
pub struct FrontendConfig {
    /// Settings related to Umami analytics.
    #[setting(nested)]
    pub umami: FrontendUmamiConfig,
}

Renders:
# Settings related to frontend storage.
frontend:
  # Settings related to Umami analytics.
  # @envvar FRONTEND_UMAMI
  umami:
    # For example: https://umami.is/script.js.
    # @envvar FRONTEND_UMAMI_SCRIPT_URL
    script_url: ""

    website_id: ""

    domains: ""

I don't think the @envvar should show up above the unami property.

Fixed


Document for update if some fields are not provided then it overwrite with default

the update operation overwrites existing values that are not passed with default.
for example if command executed is
Say current rule has protocol tcp, the below update command only updates source-asgs, however, all other parameters are replace with default value. So the protocol now becomes any.
az network nsg rule update --name $name --nsg-name $nsg_name --resource-group $resource_group --source-asgs $source_asg --source-address-prefixes=

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

ID: afe4941c-7b46-b9fb-1c52-98e68f6354cc
Version Independent ID: d1bc72fb-ccf9-9645-7b1d-98c91bc75a55
Content
Content Source
Service: virtual-network

Thanks for the report @nirmalparekh1984. This is definitely a bug.

It also changes the direction of the Rule, by default it makes the direction as inbound

No update command in the CLI should have any default value.

az network nsg rule create -g $resource_group --nsg-name $nsg_name -n $rule_name --priority $priority --access $policy --source-address-prefixes "<IP_ADDRESS>,<IP_ADDRESS>" --destination-address-prefixes $destination --protocol $protocol --destination-port-ranges $destination_port --description $downloadUri
Is it possible to add in --source-address-prefixes 2 or more IP's as I describe in my command above?
For now, error occurred: "has invalid Address prefix. Value provided:<IP_ADDRESS>,<IP_ADDRESS>"


Sjs/cell viewport enter fixes

Fixes #1850
Fixes #1796

Fix uninstantiated codecell visuals
Normal mode:

Advanced mode:

looks good but has merge conflict

boom! so excited to have this in now :)


capturePictureWithCompletion will not do imageCompletion

if imageAlbumName not specified. this will happen. I'm version 5.0.0

@sheepshen132
can you please try with v5.1.0 and see if the problem persists?

it's ok


Upgrade ZwaveJS2Mqtt to v5.10.1

Proposed Changes
Upgrade ZwaveJS2Mqtt to v5.8.0
Related Issues
https://github.com/zwave-js/zwavejs2mqtt/releases/tag/v5.10.0
https://github.com/zwave-js/zwavejs2mqtt/releases/tag/v5.9.0

@asucrews Next time you are going to contribute to an open source project, please make sure you tests your changes.
You didn't in this case, if you did, you would have known this isn't enough.
Safe yourself and the maintainer (me in this case) some time by not contributing in that case.


Single threaded in production

We are currently used to using thin via rackup in production for single-threaded but evented (and thus concurrent) request handling with eventmachine.
We are curious about leaving eventmachine for ruby3. The documentation says that using falcon with rackup is not recommended, but is this just because it will be single-threaded?  Since our code will all be async, that seems like not such a big deal and probably we can expect performance similar to thin when used in this way?

How many processor cores do you have on the instance and do you want to use them all?

I'm aware that a single ruby process won't use multiple cores, but our webapps are generally I/O bound not CPU bound so that's never been a concern.


Register K8s components with existing models

Notes for Reviewers

This PR makes sure that the K8s components when registered look for an existing(more appropriate) model to be attached to which can hijack them.

One extra change here is in countour configuration to remove sticky session.

Signed commits

[ ] Yes, I signed my commits.

Codecov Report
Patch coverage has no change and project coverage change: -4.18 :warning:

Comparison is base (a905d18) 16.29% compared to head (0091b3d) 12.12%.

:exclamation: Current head 0091b3d differs from pull request most recent head dff6edc. Consider uploading reports for the commit dff6edc to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #7525      +/-   ##
==========================================
- Coverage   16.29%   12.12%   -4.18%
==========================================
  Files         119      119
  Lines       16455    16451       -4
==========================================
- Hits         2682     1994     -688
- Misses      13286    14135     +849
+ Partials      487      322     -165

Flag
Coverage Δ

gointegrationtests
?

unittests
12.12% <0.00%> (+<0.01%)
:arrow_up:

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

Impacted Files
Coverage Δ

server/handlers/k8sconfig_handler.go
2.09% <0.00%> (+0.02%)
:arrow_up:

... and 12 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

@Revolyssup merge conflict. Looks good for merge otherwise. ✅

Conflicts resolved.


[spark] Support merge into for append table

Purpose
Support merge into for append table
Tests

API and Format

Documentation

+1


IBM Cloud CLI won't let me type my password or One-time code

I have just started to use the IBM cloud CLI by following the ibm-cloud-professional-developer-616 course
So i am using
Release version v2.27.0 on Windows
After i installed IBM cloud CLI and installed the container-register plug-in i went to the next step which is to sign in
using the command
ibmcloud login

Username prompt works fine, but the password prompt won't let me type in a password.
But after i put in my email, the password input was not working as much as i tried to type anything, no matter how much nonsense i mashed into my keyboard nothing was being typed
Using a one-time passcode to log in with the CLI
Then i tried the the command
ibmcloud login --sso

But here after i had received my One-time password, when i go to type it out or paste it. Nothing works again.
I end up having to CTRL + C my way out.
Unless this is a "Feature" of some sorts i don't know what i am supposed to do.
I even typed out my password and the One-time password just assuming that the CLI had decided not to show it to me but was taking in whatever i was typing; It was not. It gave me an error saying my credentials were wrong(Meaning that nothing that i had typed out was being registered)

@Dhyaan1 we are not able to reproduce this failure. What version of Windows are you using? What type of shell are you using -- it looks like just a Command Prompt? If it is a Command Prompt, you may have to use "Edit" - "Paste" from the window menu in order to successfully paste the credentials.

Its Windows 11 and yes its the command Prompt( I tried this on powershell too, same problem)
The  "Edit" - "Paste" from the window menu that you mentioned is probably a windows 10 thing
But i did find its windows 11 equivalent
Here,

This does work for normal cases the same way CTRL + V works
But it still doesn't work for the password prompt.
https://github.com/user-attachments/assets/b1bca8e5-44cd-4de0-befa-b6e7553a0378
Same thing happens for the One-time code too as you can see in the original issue, i say yes, it opens a tab on my browser to show me a code, i copy the code, i cant do anything with the code.
The only issue i could find that was similar to this was Here, but they didn't solve the problem because they apparently never needed to log in(run ibmcloud login).
To help reproduce this failure
Context: I started to use this software today while following an IBM course, i have never used IBM Cloud CLI ever before, this issue pretty much arose in the set-up stage for me.
This is the documentation my course linked me to
https://cloud.ibm.com/docs/cli?topic=cli-getting-started
which pointed me to the releases on GitHub
I downloaded the IBM Cloud CLI from here (The windows 64 bit version for the latest release)
https://download.clis.cloud.ibm.com/ibm-cloud-cli/2.27.0/IBM_Cloud_CLI_2.27.0_amd64.exe
Possible point of failure 1:
I used the releases on GitHub to install IBM Cloud CLI instead of simply doing the command that the page suggested for windows( I was hoping I'd get to chose the location of installation if I'd use the GitHub releases)

Then according to my course i first ran the command

which gave me this

Which i am assuming means that it works
My next task was to Install the Container Registry CLI Docs

Which also appears to have worked
ibmcloud plugin list

Now, for the task where it went wrong

As i have shown in my original Issue, any keys i click on the keyboard don't seems to do anything when the password or One-time code message shows up.

Possible point of failure:
Perhaps my IBM cloud account is not working as it should, the CLI has the same password prompt blocking issue when i use anything as an email. So, it maybe be blocking me off because it thinks my email is wrong?

Right before i was going to send this comment i noticed another behavior
Even though i am blocked from typing out my password, it does still detect when nothing was typed(Times when i didn't even try typing before clicking enter)

But when i do type out something or do a CTRL + V

As i mentioned in my original Issue, i did try and type out my actual password/One-time code several time even though i had no CLI response
I will keep trying this and update on this issue thread, i think i might be able to work around the "No CLI showing" issue

Update: Got it working!!
Even though the One-Time code here is invisible, it is apparently there when you CTRL + V without trying to type out anything before hand (even if you type like 3 letters and then backspace, it will work).

The Email and password method still doesn't work though.
Conclusion
I think its because i may have a federated ID

Which is why only the One-time code method works
Why the password and OTC prompts are invisible? i don't know.
The login works, so i am closing this Issue.

You are correct, these credential prompts are hidden to avoid inadvertent disclosure. Also glad you were able to resolve the method of login for your account.


[WIP] User experience check

Hi,
I'm trying this bundle and starting to encounter some issues.
So I will keep this as meta issue for questions & fixes.

FOSUser mapping use different folder for it's mappings #76
GenerateUserSysCommand compatibility with sf2.3 issue #78
Fix errors translation of security login page #80

Sorry, can't find time to do doc fixes.
Still thread somewhat documenting itself  :smile:
Thank you on your assistance with this one :)

No problem :)


How to convert voxel to stl?

Thanks for your code. Now, I have a question about how to convert voxel to stl and I have no idea about converting it.  Hope for you help.

@wanghan0501
Could it be explained in https://github.com/cpederkoff/stl-to-voxel/wiki?
Take a look at the wiki's explanation, and if you don't understand, ask a question


⚠ Fetchers has degraded performance

In 305a170, Fetchers ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 201 ms

Resolved: Fetchers performance has improved in b8c742b.

Resolved: Fetchers performance has improved in b8c742b.


Wrong PURL in CHEMINF

Hey! In our great OBO wide effort to fix our PURLs, I noticed the following wrong ones being used in CHEMINF:
http://purl.obolibrary.org/obo#is_enantiomer_of

same seems to hold for: http://purl.obolibrary.org/obo#is_tautomer_of

These are from ChEBI from some time ago so probably best to see what ChEBI is doing nowadays.

http://purl.obolibrary.org/obo/chebi#is_enantiomer_of
http://purl.obolibrary.org/obo/chebi#is_tautomer_of

@jmillanacosta, can you perhaps make a patch to update the URLs? See https://github.com/semanticchemistry/semanticchemistry/blob/0eb01cbe921c79088823c7c2b1e1e430fa26fc15/ontology/cheminf.owl#L262-L283


mouseup bug

On my end, v.1.1.0
mouseup triggers outside of specified object.
had to implement this workaround to work as wanted . looking for 'intersected' key value to be true.
object.addEventListener('mouseup', function (event) {

            if (event.intersected) {

                   console.log('hello');

            }

        })

Hi Anthony, that is actually intended behaviour. If you want the mouse up to fire only on the object you can also use the "click" event.


feat(FN-1089): fix next report period start

Introduction
The next report period start date was being calculated incorrectly. For example, if the next report due is April 2023, the next report period start date was being calculated as 1 April 2023 whilst it should be 1 May 2023.
Resolution
Update the utilisation report upload controller to correctly calculate the next due report period start date. Also updates component tests to reflect the expected behaviour.

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

:exclamation: No coverage uploaded for pull request base (main-stbr@5f7cc75). Click here to learn what that means.

Additional details and impacted files
@@             Coverage Diff              @@
##             main-stbr    #2339   +/-   ##
============================================
  Coverage             ?   23.84%
============================================
  Files                ?     1091
  Lines                ?    26355
  Branches             ?     4197
============================================
  Hits                 ?     6285
  Misses               ?    17514
  Partials             ?     2556

Flag
Coverage Δ

ui-test
64.18% <0.00%> (?)

unit-test
4.79% <0.00%> (?)

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.


Mac printer bug

Hi
Thanks for this great project. I updated our product with Skia enabled but we encountered a weird issue. The printer list on a mac will show only a pdf printer. People cannot find their printer any more. If I disable Skia, the pdf driver is gone and the list contains all the printers on the mac.
Pretty serious.
I included a test project.
A workaround is to copy the fmx.skia.canvas.pas file from the embarcadero source to the project folder and comment out the line which contains {$DEFINE SKIA_PRINTER}
I also reported this to embarcadero:  https://embt.atlassian.net/servicedesk/customer/portal/1/RSS-1401

Demo attached on embarcadero portal: Printerlist bug.zip

Just for future test, a new version of the demo with print capability: PrinterDemo.zip


New version: Brave.Brave.Beta version <IP_ADDRESS>

[X] Have you signed the Contributor License Agreement?
[X] Have you checked that there aren't other open pull requests for the same manifest update/change?

[ ] Have you tested your manifest locally with winget install --manifest <path>?
[X] Does your manifest conform to the 1.4 schema?

Microsoft Reviewers: codeflow:open?pullrequest=https://github.com/microsoft/winget-pkgs/pull/114207&drop=dogfoodAlpha

@wingetbot run

/azp run


Incorrect teacher last names

e.g. Wen Yu Ho becomes Yu instead of Ho.
The issue might be that the second word in the name string is being chosen instead of the last one.

Fixed with commit 87c68c2.


Definition update<EMAIL_ADDRESS>

Hi,
it is possible to update the definition of mongodb to verison 2.x?

That would be nice.

MongoDB is updated!


Enable to track admin who is in charge of the day

As for current version, every user is only assigned to the first admin.
Admin couldn't assigned who will be in charge of today's users.
Wish to add on:
i) Admin can assigned which admin will be in charge of the day
ii) Update at the updateMenu().

Hi, can we help to contribute to your project? We want to try out whether we can manage to solve all your issues posted.
bryansheng0251@gmail << email me if possible for a direct message for a detailed discussion.


problems running fetcher on current main branch

This is from running deploy.sh on main branch (deploying dev stack pbudne-indexer) on ifill
@pgulley I see AttributeError: module 'indexer.workers.fetcher' has no attribute 'BlacklistRedirectMiddleware' -- could it be related to your recent merge?
ELASTICSEARCH_INDEX_NAME_PREFIX=mediacloud_search_text
APP_HOST=.
LOG_LEVEL=info
HOSTNAME=8d375f7cb87a
PYTHON_PIP_VERSION=23.0.1
APP_DOCKER=/app
HOME=/root
GPG_KEY=A035C8C19219BA821ECEA86B64E628F8D684696D
ARGS=--yesterday --sample-size=5000 --num-batches=10 --batch-index=1
PYTHON_GET_PIP_URL=https://github.com/pypa/get-pip/raw/4cfa4081d27285bda1220a62a5ebf5b4bd749cdb/public/get-pip.py
ELASTICSEARCH_HOSTS=http://elasticsearch1:9200
PATH=/usr/local/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
RABBITMQ_URL=amqp://rabbitmq:5672/?connection_attempts=10&retry_delay=5
LANG=C.UTF-8
STATSD_URL=statsd://stats.tarbell.mediacloud.org
PYTHON_VERSION=3.10.13
PYTHON_SETUPTOOLS_VERSION=65.5.1
SENTRY_ENVIRONMENT=disabled_by_empty_dsn
PWD=/app
PYTHON_GET_PIP_SHA256=9cc01665956d22b3bf057ae8287b035827bfd895da235bcea200ab3b811790b6
RUN=fetcher
STATSD_REALM=pbudne
STORY_FACTORY=BaseStory
TZ=GMT
log: -t run-fetcher.sh -p debug invoking indexer.workers.fetcher.fetch_worker --yesterday --sample-size=5000 --num-batches=10 --batch-index=1
2023-12-03 01:15:45,115 | INFO | __main__ | Namespace(log_level=None, list_loggers=False, logger_level=None, amqp_url='amqp://rabbitmq:5672/?connection_attempts=10&retry_delay=5', from_quarantine=False, yesterday=True, fetch_date=None, num_batches=10, batch_index=1, sample_size=5000)
2023-12-03 01:15:45,115 | INFO | __main__ | Fetching for yesterday
2023-12-03 01:15:45,115 | INFO | indexer.app | sending stats to statsd://stats.tarbell.mediacloud.org prefix mc.pbudne.story-indexer.fetcher
2023-12-03 01:15:45,115 | INFO | __main__ | Fetching rss batch 0 for 2023-12-01
2023-12-03 01:17:39,019 | INFO | __main__ | Batch 0:  5 stories, from 246 domains (~0.02032520325203252 stories per domain
2023-12-03 01:17:39,019 | INFO | __main__ | Batch 1:  5 stories, from 249 domains (~0.020080321285140562 stories per domain
2023-12-03 01:17:39,020 | INFO | __main__ | Batch 2:  5 stories, from 249 domains (~0.020080321285140562 stories per domain
2023-12-03 01:17:39,020 | INFO | __main__ | Batch 3:  5 stories, from 249 domains (~0.020080321285140562 stories per domain
2023-12-03 01:17:39,020 | INFO | __main__ | Batch 4:  5 stories, from 250 domains (~0.02 stories per domain
2023-12-03 01:17:39,020 | INFO | __main__ | Batch 5:  5 stories, from 251 domains (~0.0199203187250996 stories per domain
2023-12-03 01:17:39,020 | INFO | __main__ | Batch 6:  5 stories, from 252 domains (~0.01984126984126984 stories per domain
2023-12-03 01:17:39,020 | INFO | __main__ | Batch 7:  5 stories, from 252 domains (~0.01984126984126984 stories per domain
2023-12-03 01:17:39,020 | INFO | __main__ | Batch 8:  5 stories, from 252 domains (~0.01984126984126984 stories per domain
2023-12-03 01:17:39,020 | INFO | __main__ | Batch 9:  5 stories, from 252 domains (~0.01984126984126984 stories per domain
2023-12-03 01:17:39,020 | INFO | __main__ | Initializing stories for 0 on 2023-12-01
2023-12-03 01:17:39,091 | INFO | __main__ | Initialized 500 stories
2023-12-03 01:17:39,098 | INFO | scrapy.utils.log | Scrapy 2.10.1 started (bot: scrapybot)
2023-12-03 01:17:39,101 | INFO | scrapy.utils.log | Versions: lxml <IP_ADDRESS>, libxml2 2.10.3, cssselect 1.2.0, parsel 1.8.1, w3lib 2.1.2, Twisted 22.10.0, Python 3.10.13 (main, Nov 29 2023, 05:26:45) [GCC 12.2.0], pyOpenSSL 23.3.0 (OpenSSL 3.1.4 24 Oct 2023), cryptography 41.0.5, Platform Linux-5.15.0-84-generic-x86_64-with-glibc2.36
2023-12-03 01:17:39,101 | INFO | __main__ | Launching Batch Spider Process for Batch 0
2023-12-03 01:17:39,104 | INFO | scrapy.addons | Enabled addons:
[]
2023-12-03 01:17:39,105 | INFO | scrapy.crawler | Overridden settings:
 'AUTOTHROTTLE_ENABLED': True,
 'AUTOTHROTTLE_TARGET_CONCURRENCY': 10,
 'CONCURRENT_REQUESTS_PER_DOMAIN': 2,
 'COOKIES_ENABLED': False,
 'DOWNLOAD_TIMEOUT': 60,
 'RETRY_HTTP_CODES': [502, 503, 504, 522, 524, 408, 429],
 'USER_AGENT': 'mediacloud bot for open academic research '
               '(+https://mediacloud.org)'}
2023-12-03 01:17:39,106 | WARNING | py.warnings | /usr/local/lib/python3.10/site-packages/scrapy/utils/request.py:248: ScrapyDeprecationWarning: '2.6' is a deprecated value for the 'REQUEST_FINGERPRINTER_IMPLEMENTATION' setting.

It is also the default value. In other words, it is normal to get this warning if you have not defined a value for the 'REQUEST_FINGERPRINTER_IMPLEMENTATION' setting. This is so for backward compatibility reasons, but it will change in a future version of Scrapy.

See the documentation of the 'REQUEST_FINGERPRINTER_IMPLEMENTATION' setting for information on how to handle this deprecation.
  return cls(crawler)
2023-12-03 01:17:39,123 | INFO | scrapy.extensions.telnet | Telnet Password: 0a021a5b5cdd22ff
2023-12-03 01:17:39,273 | INFO | scrapy.middleware | Enabled extensions:
['scrapy.extensions.corestats.CoreStats',
 'scrapy.extensions.telnet.TelnetConsole',
 'scrapy.extensions.memusage.MemoryUsage',
 'scrapy.extensions.logstats.LogStats',
 'scrapy.extensions.throttle.AutoThrottle']
2023-12-03 01:17:39,303 | INFO | __main__ | Fetched 0 stories in batch 0
2023-12-03 01:17:39,303 | INFO | indexer.worker | waiting until queues configured....
2023-12-03 01:17:39,319 | INFO | indexer.worker | queues configured
2023-12-03 01:17:39,323 | INFO | pika.adapters.utils.connection_workflow | Pika version 1.3.2 connecting to ('<IP_ADDRESS>', 5672)
2023-12-03 01:17:39,324 | INFO | pika.adapters.utils.io_services_utils | Socket connected: <socket.socket fd=11, family=AddressFamily.AF_INET, type=SocketKind.SOCK_STREAM, proto=6, laddr=('<IP_ADDRESS>', 50618), raddr=('<IP_ADDRESS>', 5672)>
2023-12-03 01:17:39,327 | INFO | pika.adapters.utils.connection_workflow | Streaming transport linked up: (<pika.adapters.utils.io_services_utils._AsyncPlaintextTransport object at 0x7fa102ef1f60>, _StreamingProtocolShim: <SelectConnection PROTOCOL transport=<pika.adapters.utils.io_services_utils._AsyncPlaintextTransport object at 0x7fa102ef1f60> params=<URLParameters host=rabbitmq port=5672 virtual_host=/ ssl=False>>).
2023-12-03 01:17:39,333 | INFO | pika.adapters.utils.connection_workflow | AMQPConnector - reporting success: <SelectConnection OPEN transport=<pika.adapters.utils.io_services_utils._AsyncPlaintextTransport object at 0x7fa102ef1f60> params=<URLParameters host=rabbitmq port=5672 virtual_host=/ ssl=False>>
2023-12-03 01:17:39,334 | INFO | pika.adapters.utils.connection_workflow | AMQPConnectionWorkflow - reporting success: <SelectConnection OPEN transport=<pika.adapters.utils.io_services_utils._AsyncPlaintextTransport object at 0x7fa102ef1f60> params=<URLParameters host=rabbitmq port=5672 virtual_host=/ ssl=False>>
2023-12-03 01:17:39,334 | INFO | pika.adapters.blocking_connection | Connection workflow succeeded: <SelectConnection OPEN transport=<pika.adapters.utils.io_services_utils._AsyncPlaintextTransport object at 0x7fa102ef1f60> params=<URLParameters host=rabbitmq port=5672 virtual_host=/ ssl=False>>
2023-12-03 01:17:39,334 | INFO | indexer.worker | connected to amqp://rabbitmq:5672/?connection_attempts=10&retry_delay=5
2023-12-03 01:17:39,336 | INFO | pika.adapters.blocking_connection | Created channel=1
Unhandled error in Deferred:
2023-12-03 01:17:39,388 | CRITICAL | twisted | Unhandled error in Deferred:
2023-12-03 01:17:39 [twisted] CRITICAL: Unhandled error in Deferred:

Traceback (most recent call last):
  File "/usr/local/lib/python3.10/site-packages/scrapy/crawler.py", line 245, in crawl
    return self._crawl(crawler, *args, **kwargs)
  File "/usr/local/lib/python3.10/site-packages/scrapy/crawler.py", line 249, in _crawl
    d = crawler.crawl(*args, **kwargs)
  File "/usr/local/lib/python3.10/site-packages/twisted/internet/defer.py", line 1947, in unwindGenerator
    return _cancellableInlineCallbacks(gen)
  File "/usr/local/lib/python3.10/site-packages/twisted/internet/defer.py", line 1857, in _cancellableInlineCallbacks
    _inlineCallbacks(None, gen, status, _copy_context())
--- <exception caught here> ---
  File "/usr/local/lib/python3.10/site-packages/twisted/internet/defer.py", line 1697, in _inlineCallbacks
    result = context.run(gen.send, result)
  File "/usr/local/lib/python3.10/site-packages/scrapy/crawler.py", line 134, in crawl
    self.engine = self._create_engine()
  File "/usr/local/lib/python3.10/site-packages/scrapy/crawler.py", line 148, in _create_engine
    return ExecutionEngine(self, lambda _: self.stop())
  File "/usr/local/lib/python3.10/site-packages/scrapy/core/engine.py", line 98, in __init__
    self.downloader: Downloader = downloader_cls(crawler)
  File "/usr/local/lib/python3.10/site-packages/scrapy/core/downloader/__init__.py", line 97, in __init__
    DownloaderMiddlewareManager.from_crawler(crawler)
  File "/usr/local/lib/python3.10/site-packages/scrapy/middleware.py", line 67, in from_crawler
    return cls.from_settings(crawler.settings, crawler)
  File "/usr/local/lib/python3.10/site-packages/scrapy/middleware.py", line 43, in from_settings
    mwcls = load_object(clspath)
  File "/usr/local/lib/python3.10/site-packages/scrapy/utils/misc.py", line 84, in load_object
    raise NameError(f"Module '{module}' doesn't define any object named '{name}'")
builtins.NameError: Module 'indexer.workers.fetcher' doesn't define any object named 'BlacklistRedirectMiddleware'

2023-12-03 01:17:39,389 | CRITICAL | twisted |
Traceback (most recent call last):
  File "/usr/local/lib/python3.10/site-packages/scrapy/utils/misc.py", line 82, in load_object
    obj = getattr(mod, name)
AttributeError: module 'indexer.workers.fetcher' has no attribute 'BlacklistRedirectMiddleware'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/lib/python3.10/site-packages/twisted/internet/defer.py", line 1697, in _inlineCallbacks
    result = context.run(gen.send, result)
  File "/usr/local/lib/python3.10/site-packages/scrapy/crawler.py", line 134, in crawl
    self.engine = self._create_engine()
  File "/usr/local/lib/python3.10/site-packages/scrapy/crawler.py", line 148, in _create_engine
    return ExecutionEngine(self, lambda _: self.stop())
  File "/usr/local/lib/python3.10/site-packages/scrapy/core/engine.py", line 98, in __init__
    self.downloader: Downloader = downloader_cls(crawler)
  File "/usr/local/lib/python3.10/site-packages/scrapy/core/downloader/__init__.py", line 97, in __init__
    DownloaderMiddlewareManager.from_crawler(crawler)
  File "/usr/local/lib/python3.10/site-packages/scrapy/middleware.py", line 67, in from_crawler
    return cls.from_settings(crawler.settings, crawler)
  File "/usr/local/lib/python3.10/site-packages/scrapy/middleware.py", line 43, in from_settings
    mwcls = load_object(clspath)
  File "/usr/local/lib/python3.10/site-packages/scrapy/utils/misc.py", line 84, in load_object
    raise NameError(f"Module '{module}' doesn't define any object named '{name}'")
NameError: Module 'indexer.workers.fetcher' doesn't define any object named 'BlacklistRedirectMiddleware'
2023-12-03 01:17:39 [twisted] CRITICAL:
Traceback (most recent call last):
  File "/usr/local/lib/python3.10/site-packages/scrapy/utils/misc.py", line 82, in load_object
    obj = getattr(mod, name)
AttributeError: module 'indexer.workers.fetcher' has no attribute 'BlacklistRedirectMiddleware'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/lib/python3.10/site-packages/twisted/internet/defer.py", line 1697, in _inlineCallbacks
    result = context.run(gen.send, result)
  File "/usr/local/lib/python3.10/site-packages/scrapy/crawler.py", line 134, in crawl
    self.engine = self._create_engine()
  File "/usr/local/lib/python3.10/site-packages/scrapy/crawler.py", line 148, in _create_engine
    return ExecutionEngine(self, lambda _: self.stop())
  File "/usr/local/lib/python3.10/site-packages/scrapy/core/engine.py", line 98, in __init__
    self.downloader: Downloader = downloader_cls(crawler)
  File "/usr/local/lib/python3.10/site-packages/scrapy/core/downloader/__init__.py", line 97, in __init__
    DownloaderMiddlewareManager.from_crawler(crawler)
  File "/usr/local/lib/python3.10/site-packages/scrapy/middleware.py", line 67, in from_crawler
    return cls.from_settings(crawler.settings, crawler)
  File "/usr/local/lib/python3.10/site-packages/scrapy/middleware.py", line 43, in from_settings
    mwcls = load_object(clspath)
  File "/usr/local/lib/python3.10/site-packages/scrapy/utils/misc.py", line 84, in load_object
    raise NameError(f"Module '{module}' doesn't define any object named '{name}'")
NameError: Module 'indexer.workers.fetcher' doesn't define any object named 'BlacklistRedirectMiddleware'
log: -t run-fetcher.sh -p debug indexer.workers.fetcher.fetch_worker --yesterday --sample-size=5000 --num-batches=10 --batch-index=1 exit status 0
log: -t bin/run-fetcher.sh -p info bin/run-fetcher.sh --yesterday --sample-size=5000 --num-batches=10 --batch-index=1 exit status 0

Any progress?

#183 addresses this!