Added the contributors page with sleek ui and dynamic functionality.

Added the contributors page with sleek ui and dynamic functionality.
closes :- #83

https://github.com/user-attachments/assets/a55dc3cd-1e61-4149-9a5b-f187423667a5
@notsoocool Please check this PR, if you find any issue or you want any changes please mention me i will do those changes otherwise if all is fine you can merge this and assign this to me and put relevant labels such as gssoc 2024 ext, hacktoberfest and level.
i think it can be level 3 because the page is very important in open source to showcase the efforts of the people contributing in building it and it is not static it is dynamic as a new contributor is added to your repo it will be automatically added there so this is the beauty of this. (Just my suggestion all is upto you how you see my work.)

@tarunkumar2005
Why is the contributors page only showing 5 contributors when there are actually 11? Could you check if there's a limit or issue with the display logic?

@notsoocool I can't say about this the api is giving only 5 objects in the response. I tried to check this it might happen because github is not able to track their PR and contribution, it is by github only i can't do much in it.

@notsoocool What can i do about this github is only sending 5 objects in response.

@notsoocool What can i do about this github is only sending 5 objects in response.

No worries! I will go through it as soon as I have some free time. In the meantime, I'll merge your PR into a different branch. Thanks for your understanding!

@notsoocool Thank you i will also look into it if i find anything to help then i will ping you here.

@notsoocool Just asking to make it level 3 as the ui design is very good and it is responsive for all screen sizes so no need to work on this more right so it can be level 3.

@tarunkumar2005 I was initially assigning level3 to PRs that involve API and model changes or modifications that affect both the APIs and models, including the page. However, since @sanjay-kv has already labeled your PR as level3, I won't change it for now. Just keep this information in mind for future contributions. Thanks!


import is needless for the dead code

I have the code below
require('./c');

if (true) {
  require('./a');
} else {
  require('./b');
}

function A () {
  console.log('hhhh');
}

module.exports = A;

and I use commond cjstoesm a.js esm to transform. I got the result below

would you please to support this feature?

Or is this the duty of the terser ?

This is already answered in the FAQ here. As far your question about dead code removal, then no, cjstoesm simply converts CJS-based files into equivalent ESM (if possible).


Store Items not Showing or Being accessible

None of the shop items show, mod is installed with everything that it needs to work.

Nevermind, It was a new mod i installed that broke it.

Nevermind, It was a new mod i installed that broke it.

Which is the mod? I'm facing the same problem but I don't know which mod is


⚠ Inventory API Endpoint has degraded performance

In fd39c2e, Inventory API Endpoint (https://inventory.roblox.com/v1/users/82738847/assets/collectibles?limit=10&sortOrder=Asc) experienced degraded performance:

HTTP code: 200
Response time: 4004 ms

Resolved: Inventory API Endpoint performance has improved in 2e636a3.


SvelteKit + webui problem

There may be a conflict between how a SvelteKit SPA boots itself and how zig-webui app loads its content. To see this, do the following.
Create a SvelteKit project with
npm create svelte@latest webui-sveltekit

Choose:

"Skeleton" app option
"No" do not use TypeScript, nor JSDoc
Add "ESLint"

Follow instructions to test it, ending with npm run dev.
Next, set it up to generate an SPA:
npm install @sveltejs/adapter-static

Edit svelte.config.js to look like this:
// import adapter from '@sveltejs/adapter-auto';
import adapterStatic from '@sveltejs/adapter-static';

/** @type {import('@sveltejs/kit').Config} */

const config = {
    kit: {
        // adapter-auto only supports some environments, see https://kit.svelte.dev/docs/adapter-auto for a list.
        // If your environment is not supported, or you settled on a specific environment, switch out the adapter.
        // See https://kit.svelte.dev/docs/adapters for more information about adapters.
        // adapter: adapter()

        adapter: adapterStatic({
            pages: 'build',
            assets: 'build',
            fallback: 'app.html', // required to generate app.html in build folder
            precompress: false,
            strict: false
        })
    }
};

export default config;

In the /routes folder, add a file, +layout.js and add this:
export const ssr = false;
export const prerender = false;

Run npm run dev to test.
The above project is available at https://github.com/sensiblearts/SvelteKit-WebUI-SPA-test.git
To build the SPA:
npm run build
Copy the build folder (I renamed it spa_build) and all of its contents (app.html, compiled JS, etc) into a new zig-webui test project, at the same level as its src folder.
Edit main.zig to read as follows:
pub fn main() !void {
    var nwin = webui.newWindow();

	const path = "./spa_build";
    const filename = "app.html";

    _ = webui.setDefaultRootFolder(path);

    _ = nwin.show(filename);

    webui.wait();
}

Run the program with
zig build run and you will see this:

Then, right-click on the screen to inspect the JS console:

In the above screenshot, the line app.html:28.11 is pointing to this:
kit.start(app, element);

Thus, it's the entry point to a SvelteKit app that is causing the problem. If you drill down on the first module listed in the Not found error, you see this:

Which, to me, looks like the first thing a SvelteKit app does is replace the client-side state -- I think they call this "hydration" -- and to do this it seems to be reloading itself -- looking for app.html for a second load. But it cannot find it...?
Note: The above happens without including webui.js in the app. If  we add  to app.html, we get the same 404, but the app crashes and reloads repeatedly, with this cycling in the console:

I think that someone who knows the internals of WebUI should talk to someone who knows the internals of SvelteKit.

To showcase usage of webui with sveltekit I published a mini-app this week is. https://github.com/ttytm/gsysmo. It doesn't have a lot of features but the concept of using SK also applies with zig-webui. Maybe the reference can be of tiny help.

I created a simple app for the purpose of having an example. It uses Go-WebUI and uses with Bun and SvelteKit (Svelte 5). Feature- and design-wise, it is an infant. Though the focus is on providing a simple example that is hopefully easy to understand.
[...]
I'm planning to keep one branch simple, without complicating things, just adding some concise notes that might help to point out some of the concepts on how this stack works together for the app. The goal is to show what is needed for a fully bundled executable in a simple way. Another branch is planned to add some real features to the application, which will be beyond an example.
Ref.: https://github.com/webui-dev/webui/issues/124#issuecomment-2143666423

Unfortunately, atm. I cannot make the time to give free individualized support. I will try to look into it soon if you can't resolve it or get better help than this.

Thank you @ttytm . I'm gone for a couple of days. When I get back I'll figure out what's different with your Go/SvelteKit5 app vs my Zig/SvelteKit4 app.
The only difference I see from my phone is that you set root folder to "build" whereas mine is set to "./spa_build" (dot slash).

@sensiblearts skimmed over the repo. It seems there is no  added anywhere (Note: this is a virtual file, you just need this line of code, not an actual file).
Putting it in app.html would be one way.
<! -- app.html -->
...
	<meta name="viewport" content="width=device-width, initial-scale=1" />

	%sveltekit.head%
...

Though I add it in ÷layout.svelte:
<! -- +layout.svelte -->
<svelte:head>

</svelte:head>

Having it there allows for some flexibility. E.g., you can make the import conditional and work  on parts of the frontend like on a regular website in a regular browser (plain npm run dev, or bun run dev and opening the localhost instance).
<! -- +layout.svelte -->
<svelte:head>
	{#if process.env.NODE_ENV === 'production'}

	{/if}
</svelte:head>

Depending on what you work on this might allow for a faster way to develop the frontend.
To also have an example for this exact case, there is something in +page.svelte. It loads mockdata instead getting the systemdata from the backend.
<! -- +page.svelte -->
...
onMount(async () => {
	if (process.env.NODE_ENV !== 'production') {
		// Alternatively, create a try condition: `try { webui; } catch { ... }`
		// to check whether `webui` is available.
		console.log('Using dummy data in development environment');
		const mockdata = await import('$lib/dev/mockdata');
		data = mockdata.data;
		setInterval(async () => {
			data.processes = mockdata.randomizeProcs();
			data.cpu_used_percent = Math.random() * 30;
			data.memory.used_percent = Math.random() * 30;
		}, 1000);
		return;
	}

	// Build environment, where webui is available.
	data = JSON.parse(await webui.getStats());
	setInterval(async () => {
		data = JSON.parse(await webui.getStats());
		data.processes.sort((a: Process, b: Process) => b.cpu_used_percent - a.cpu_used_percent);
	}, 1000);
});

Last things to mention that might be interesting to you are in the main.go file:
Example of opening the WebUI Window when you already run the website-like development frontend (like described above).
// Example that connects to a running localhost instance during development.
w.Show("")
w.Navigate("http://localhost:5173")

For the regular app I'm eventually building the frontend into a static site and use:
// Opening the statically build site
ui.SetDefaultRootFolder("ui/build")
w.ShowBrowser("index.html", ui.ChromiumBased)

So there are some more small differences that make the big impact :D.

@hassandraga thank you. I'm not sure how to proceed with a PR, or what you would like me to do next.

Duh, sorry. I just realized it's a committed PR. Thanks.

@sensiblearts sorry about the miss on your note about the
        \\ 		<link rel="modulepreload" href="/_app/immutable/entry/start.BTHfqu-a.js">
        \\ 		<link rel="modulepreload" href="/_app/immutable/chunks/entry.DfSPpY9w.js">
        \\ 		<link rel="modulepreload" href="/_app/immutable/chunks/scheduler.-OmmOMvn.js">
        \\ 		<link rel="modulepreload" href="/_app/immutable/chunks/index.Dr1V4utH.js">
        \\ 		<link rel="modulepreload" href="/_app/immutable/chunks/control.CYgJF_JY.js">
        \\ 		<link rel="modulepreload" href="/_app/immutable/entry/app.BEvzDeO2.js">
        \\ 		<link rel="modulepreload" href="/_app/immutable/chunks/preload-helper.BQ24v_F8.js">
        \\ 		<link rel="modulepreload" href="/_app/immutable/chunks/index.CA6owI4p.js">
        \\
        \\
        \\

        \\
        \\

        \\
        \\
    ;

    var nwin = webui.newWindow();

    _ = nwin.bind("", events);

    _ = webui.setDefaultRootFolder(path);

    _ = nwin.show(file_content);

    // _ = nwin.showBrowser(file_content, webui.Browsers.Chromium);
    // only seems to work with content as filename, not content as html string.

    webui.wait();
}

fn events(_: webui.Event) void {
    {}
}

Why I think it works:
The error in the last screenshot shows that the SvelteKit app is already loaded (js chunk "app...") but then tries to reload / refresh index.html. Why is it doing that? My guess is that webui is telling the browser to load it -- but it's already been loaded.
I guessed that webui would have no need to send a reload message to the browser if it was initialized with an html string. Maybe the guess was correct.
FYI, the adapter settings in svelte.config.js:
kit: {
			adapter: process.env.ADAPTER === 'static'
			? adapterStatic({
				pages: 'build',
				assets: 'build',
				fallback: 'index.html', // needed to generate index.html, because prerender == false
				precompress: false,
				strict: false,
			})
			: adapterNode(),

	}

@hassandraga I can close this if you're ready.

My pleasure. I've decided to go with webui instead of Tauri.
Thank you.


[Bug] Flickering and overlappying labels on line chart

Version
5.4.2
Link to Minimal Reproduction
https://jsfiddle.net/6n2dtwjh/
Steps to Reproduce

Create line chart with points close together
Turn labels on
Hover over the labels

Current Behavior
The value label appears on hover for every point you move over, but these sometimes overlap with the existing labels on the page.

Expected Behavior
Is there a way to turn off labels on emphasis without having them disappear on hover?
What I'd like is for the labels on the chart to stay as is when I hover over them. The tooltip is doing the work of showing the specific point's value, so I don't want to duplicate it.
I tried setting label: {show: false} in emphasis, but then it makes the label disappear when I hover over it:

Environment
N/A

Any additional comments?
No response

Try to set axisPointer.triggerEmphasis to false.

@plainheart that worked - thank you!!


Request: Support for new logging drivers in Docker 1.8

Hi,
Looks like the new logging drivers in 1.8 aren't supported. For example when I try to enable GELF as a logging driver I get the following in my agent log:
2015-10-12T03:42:15.446170281Z 2015-10-12 03:42:15,445 ERROR agent [139987274000816] [event.py:111] b2187b11-130c-4983-a8b7-ea3e54763901 : Unknown error
2015-10-12T03:42:15.446204421Z Traceback (most recent call last):
2015-10-12T03:42:15.446215345Z   File "/var/lib/cattle/pyagent/cattle/agent/event.py", line 94, in _worker_main
2015-10-12T03:42:15.446223213Z     resp = agent.execute(req)
2015-10-12T03:42:15.446229015Z   File "/var/lib/cattle/pyagent/cattle/agent/__init__.py", line 15, in execute
2015-10-12T03:42:15.446235466Z     return self._router.route(req)
2015-10-12T03:42:15.446250867Z   File "/var/lib/cattle/pyagent/cattle/plugins/core/event_router.py", line 13, in route
2015-10-12T03:42:15.446257189Z     resp = handler.execute(req)
2015-10-12T03:42:15.446262519Z   File "/var/lib/cattle/pyagent/cattle/agent/handler.py", line 34, in execute
2015-10-12T03:42:15.446268155Z     return method(req=req, **req.data.__dict__)
2015-10-12T03:42:15.446273440Z   File "/var/lib/cattle/pyagent/cattle/plugins/docker/compute.py", line 508, in instance_activate
2015-10-12T03:42:15.446279098Z     self._do_instance_activate(instance, host, progress)
2015-10-12T03:42:15.446284368Z   File "/var/lib/cattle/pyagent/cattle/plugins/docker/compute.py", line 566, in _do_instance_activate
2015-10-12T03:42:15.446290121Z     create_config['host_config'] = create_host_config(**start_config)
2015-10-12T03:42:15.446295570Z   File "/var/lib/cattle/pyagent/dist/docker/utils/utils.py", line 480, in create_host_config
2015-10-12T03:42:15.446301137Z     log_config = LogConfig(**log_config)
2015-10-12T03:42:15.446306483Z   File "/var/lib/cattle/pyagent/dist/docker/utils/types.py", line 27, in __init__
2015-10-12T03:42:15.446312060Z     ', '.join(self.types._values)
2015-10-12T03:42:15.446317340Z ValueError: LogConfig.type must be one of (json-file, syslog, none)

My environment is:
Rancher 	v0.40.0
Cattle 	v0.97.0
User Interface 	v0.54.0
Rancher Compose 	beta/latest
Docker: 1.8.2

The way I am trying to enable the log driver is via docker-compose.yml:
  external_links:
  - infra/graylog:graylog
  log_driver: "gelf"
  log_opt:
    address: "udp://graylog:12201"

Could you share your full docker-compose.yml? I'm not familiar with logging drivers, but would like to see it.

Hi,
Here is the docker-compose:
web:
  restart: always
  tty: true
  image: reg.local/test/hello-world:1.0.0
  stdin_open: true
  links:
  - graylog
  log_driver: "gelf"
  log_opt:
    address: "udp://graylog:12201"

graylog:
  restart: always
  tty: true
  image: graylog2/allinone
  environment:
    GRAYLOG_TIMEZONE: GMT
  stdin_open: true
  expose:
  - 9000
  - 12201
  - 12201/udp

This works on docker 1.8 via the command line. For example: docker run --log-driver=gelf --log-opt gelf-address=udp://graylog_ip:12201 as shown at https://docs.docker.com/reference/logging/overview/#specify-gelf-options. Support was also added to the docker-compose as shown at https://docs.docker.com/compose/yml/#log-driver

@willowandcreek We do support gelf and other logging drivers.  I tried your compose file and ran into a couple issues.

The log-opt is gelf-address and not address.  So basically the following

log_opt:
    gelf-address: "udp://graylog:12201"

You can't reference the name "graylog".  Even though you have linked the container graylog, the Docker daemon is the one that resolves the DNS name "graylog" and not the container.  So the name "graylog" is not going to be available on the host.  You need to point to some other address.

@ibuildthecloud Thanks for the feedback. Could I check if this support is there in v0.41.0? I just tried the following

Move graylog onto a standalone server
Upgraded my rancher from v0.40.0 to v0.41.0
Ran my image via docker cli and was able to see the log messages on the graylog server
Ran a plain ubuntu image using Infrastructure/Containers and was able to communicate with the graylog server (tried this just to rule out network comm issues)
Ran my image via rancher-compose and had the same error.

My updated docker-compose:
web:
  restart: always
  tty: true
  image: reg.local/test/hello-world:1.0.0
  stdin_open: true
  log_driver: "gelf"
  log_opt:
    gelf-address: "udp://<IP_ADDRESS>:12201"

where, <IP_ADDRESS> is a regular ubuntu server where my graylog server is running. Launching my stack resulted in the same stack trace:
2015-10-16T03:46:13.856180842Z Traceback (most recent call last):
2015-10-16T03:46:13.856191010Z   File "/var/lib/cattle/pyagent/cattle/agent/event.py", line 94, in _worker_main
2015-10-16T03:46:13.856198917Z     resp = agent.execute(req)
2015-10-16T03:46:13.884067034Z   File "/var/lib/cattle/pyagent/cattle/agent/__init__.py", line 15, in execute
2015-10-16T03:46:13.884101704Z     return self._router.route(req)
2015-10-16T03:46:13.884124600Z   File "/var/lib/cattle/pyagent/cattle/plugins/core/event_router.py", line 13, in route
2015-10-16T03:46:13.884133550Z     resp = handler.execute(req)
2015-10-16T03:46:13.884139580Z   File "/var/lib/cattle/pyagent/cattle/agent/handler.py", line 34, in execute
2015-10-16T03:46:13.884145887Z     return method(req=req, **req.data.__dict__)
2015-10-16T03:46:13.884151922Z   File "/var/lib/cattle/pyagent/cattle/plugins/docker/compute.py", line 508, in instance_activate
2015-10-16T03:46:13.884158079Z     self._do_instance_activate(instance, host, progress)
2015-10-16T03:46:13.884163927Z   File "/var/lib/cattle/pyagent/cattle/plugins/docker/compute.py", line 566, in _do_instance_activate
2015-10-16T03:46:13.884170067Z     create_config['host_config'] = create_host_config(**start_config)
2015-10-16T03:46:13.884175977Z   File "/var/lib/cattle/pyagent/dist/docker/utils/utils.py", line 480, in create_host_config
2015-10-16T03:46:13.884182091Z     log_config = LogConfig(**log_config)
2015-10-16T03:46:13.884187791Z   File "/var/lib/cattle/pyagent/dist/docker/utils/types.py", line 27, in __init__
2015-10-16T03:46:13.884193744Z     ', '.join(self.types._values)
2015-10-16T03:46:13.884199358Z ValueError: LogConfig.type must be one of (json-file, syslog, none)

Can you do a rancher-compose -v to let me know the exact version of Rancher compose that you are running? With v0.41.0, it should be used with v0.4.1.

Yep its v0.4.1. I always download the rancher-compose when I update my rancher server version.
~ $ rancher-compose -v
rancher-compose version v0.4.1

Looks like this was an issue with docker-py and was fixed in version 1.5.0 (http://stackoverflow.com/questions/32479530/using-docker-compose-with-gelf-log-driver).


Module1:Mobile overlay

Mobile overlay

In this milestone, we updated the navigation bar by adding a mobile overlay so we

Updated the HTML file
Editted the CSS file
Added a js file to make the page dynamic

Hi @derrick1451
Your project is complete  👍 💯   There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉
Highlights 👏🏻
✔ No linter errors.
✔ You used correct GitHub-Flow.
✔ Professional README
✔ Good Commit Messages
✔ Good job for Implemented Portfolio: mobile menu.
Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


RFC 7636 support

Will this support RFC 7636 for the public use ( distributing the applications source code to the public ) as mentioned here https://api.aweber.com/#tag/OAuth-2.0-Examples
Any information is really appreciated.
Thanks.

Closing; keeping #837 to track this. Thanks!


Update dd

Codecov Report

Merging #314 (d1398cb) into main (a121e22) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #314      +/-   ##
==========================================
- Coverage   80.04%   80.02%   -0.02%
==========================================
  Files          38       38
  Lines        2891     2889       -2
==========================================
- Hits         2314     2312       -2
  Misses        577      577

Impacted Files
Coverage Δ

src/methods/rasterize.jl
82.84% <ø> (ø)

src/show.jl
73.46% <100.00%> (-1.05%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Ruby 3.3

https://github.com/cloudfoundry/cloud_controller_ng/issues/3559

[ ] I have reviewed the contributing guide

[ ] I have viewed, signed, and submitted the Contributor License Agreement

[ ] I have made this pull request to the main branch

[ ] I have run all the unit tests using bundle exec rake

[ ] I have run CF Acceptance Tests

I wonder if we should pre-emptively run https://github.com/cloudfoundry/cf-performance-tests against this change to see if theres any noticeable positive/negative effect.

Wouldn't hurt, I suppose.  If we are starting to care about pre-release data, maybe we should have cf-performance-tests that run on prereleases, not just after capi is bumped in cf-deployment?
That said, I think most of our issues are due to DB queries, so I would be surprised if this makes a significant impact.  If Ruby code turns out to have a big impact maybe we should push the BOSH team get Rust in the bosh ruby package so we can turn on YJIT


Missing some localisation for MyAccount (maybe)

Environment:
SPA with yenv (develop branch)
Steps:
Go to electronics storefront
Login with a registered user
Change the language to Japanese
Go to MyAccount
Go to Address Book, and click on the dropdown menu for Title.
Actual Result

Sign out from "MyAccount" is not localised
The Mr, Mrs etc in the Title drop down menu aren't localised for Japanese (normal?)
Also "web"cam from the navigation bar.

we're revamping translation so i'm closing this one and we'll do a new review later


New GitHub oauth

boilers in place,
made some changes to by pass user model
-needs user model
code does not break app

*currently working on the cb url for oauth, since some oauth needs a registered url there is a work around for a test environment

good


Deploy Reference Documentation (ALDoc) action missing symbols and xml documentation not showing

I added ALDoc to my settings.json file (with beas supported country):

The CI/CD action deployed the generated documentation:

But the static site does not have all information I expected:

MissingTypeSymbol

The symbols for the parameter of a function isn't found

This is my procedure in AL:
    procedure SetStatementNoWithYearPrefix(var CODAStatement: Record "CODA Statement")

And this is the output on the generated Docs:

AL XML Documentation does not show

This is the doccumentation in AL:
    /// <summary>
    /// Set Statement No. With Year Prefix.
    /// Banks reset Statement No. every year. To prevent duplicate Statement No. in BC, we need to add year prefix to Statement No.
    /// </summary>
    /// <param name="CODAStatement">VAR Record "CODA Statement".</param>

And this is the output on the generated Docs:

And the "CODA Statement" record - is that in the same app? or in a dependent app?
Is in in a app from the same repository or a different repo?

Hi Freddy,
this is in the be localisation

got it - sounds like a bug - will investigate

I can definitely repro this, this is a bug. We will have to populate the a cache folder with referenced apps and hand that over to aldoc.

This reproes for symbols in any base apop
https://microsoft.github.io/bcsamples-bingmaps.pte/reference/bingmaps.pte/Codeunit/BingMaps-Geocode.html

If we add the parameter -c (packagecache) where the value is the path to the alpackages, the MissingTypeSymbols-Issue is gone.
Current call in AL-Go:

How we get it working in powershell:

I don't think we can currently control this -c parameter with setup in our project.


Is the learning rate 0.0001 (default) is good or the (0.00001) is good?

HI,
I was trying to tweak the learning_rate and dropout parameters for the handwriting_line_recognition.py model.
Since there is no much change in the loss for changing the dropout parameters (20%, 35%, 50%) i'm just fixing the default one.
But for the learning rate change from 0.0001 to 0.00001 there is a huge increase in the stability of the model as plotted below. (training loss is equivalent to the test loss)
plotted graph image: https://prnt.sc/rv6lzm
graph_label notations:
lr-e5 => learning_rate = 0.00001
lr-e4 => learning_rate = 0.0001
-> Bottom two lines are the train and test loss calculation for the 0.0001 learning_rate parameters and all above lines are plotted for 0.00001. We could see the bottom two lines are not stable where as the other lines are very stable (training loss is equivalent to the test loss)
Since the lr 0.00001 is better than 0.0001, can we fix 0.00001 as default or do we face any other problem if we use this new lr rate?
Please advice.
Thanks,
Anand.

From the plot it seems like the loss is smaller for the default LR compared to LR you suggested.

Yeah that's true my assumption is that if I would increase the Epocs it may converge as equal to the lr=0.0001, by this we can achieve a stability and reduced loss.


Boon or LoganSquare

Is Boon or LoganSquare faster?

Pretty sure this isn't an "issue".

My guess is that LoganSquare is faster, but considering Boon's documents proudly say

The goal of Boon was never to be a small jar file that could get used on Android.
(I actually hate Android and am a big iOS/OSX guy so... if it happens it will not be because I need it personally. It will be because someone else wants it and makes it happen. I had 5 Android devices. NOT. A. FAN. AT. ALL. )

I don't think it's worth bothering to compare them.


Improve build performance

Overview
Improves build performance on Circle CI. Reduces build time by 1-2 min for dev builds and 4-5 min for full master builds (includes sample apps). Prevents OOM errors on Circle CI container.
Proposed Changes

Updates Gradle options to limit memory use and dump heap on OOM error.
Runs unit tests against debug build only and reduces console output.
Upgrades Gradle to version 3.3 and gradle-android-maven-plugin to version 1.5.
Adds --configure-on-demand and -PdisablePreDex options to circle.yml.
Runs tests for each module in parallel using 3 Circle CI containers instead of 1.
Archives sample app APKs on Circle CI and deploys to AWS on release build only.
Publishes documentation to Mapzen website on release build only.
Installs core module to local .m2 repo before resolving dependencies.
Disables Javadoc tasks for all modules. May need to be re-enabled for production AARs.

Fixes #297

Starting reviewing this now.


Mount home folder of calling user to /root

Home folder is what is in $HOME or whats registered in /etc/passwd
Some programs ignore $HOME, but /root is usually not writable inside the container since it points the the host's /root

Closing, there is a standart, that HOME should be considered first. So using directly /root would be a bug in the program being run inside plash


Does this algorithm work on audio data?Can any body give me an example to use this algorithm for audio data classification?

@SibtainRazaJamali yes, as @LAV42 mentioned, you can use it similar to the image example. Let us know if you have any other questions.

Yes @LAV42 and @ledw Thanks for kind response , i can implement this but can you suggest me ,if there is any specific algorithm to extract important features from audio signal. Since spectogram of audio signal contains different features as compared to normal image.

@SibtainRazaJamali Apologize for the delay in replying. I am not familiar with algorithms to extract features from audio signal.


updated svHandle to use $scope.$ctrl rather than $ctrl

$ctrl is undefined within svHandle, but $scope.$ctrl is defined and is watched by svElement. Handles didn't work before (for me) but they do now.

but why?

$ctrl is injected via directive's require mechanism. If it's undefined you're doing something wrong. It should work and works in my examples.

svHandles should be working now in v0.0.13


Optional character

Add a new function to handle optional character in the mask like:
(99) 9999?9-9999

I tried using (99) 9999? 9-9999 ..
Runtime Error read property 'match' of undefined

Not yet.
Will be in one week.

Now?

And now?


Resource-based policies and implicit denies in other policy types (same account)

In the table provided here (under the section resource-based policies):  https://docs.aws.amazon.com/IAM/latest/UserGuide/reference_policies_evaluation-logic.html
The "IAM Role Session" is listed twice as the principal making the request in that table. In the first instance, it mentions "Allows Role ARN" and in the second, it mentions "Allows Role session ARN".
What is not clear to me is: How can we allow a Role ARN for a Role Session ARN? Is not the fact that it is a Role Session, it would always be equal to the following format: arn:aws:sts::111122223333:assumed-role/examplerole/examplerolesessionname
In other words, in which scenario (or example) would the "Allows Role ARN" work with a "IAM role session"? Could you please provide an example or clarify this point please?

It's meant to help you understand the impact of policy evaluation specific to resource-based policies, depending on which ARN you specified in the resource-based policy. If the role session ARN is what's specified in the resource-based policy, then other policy types are not evaluated.

Yes, I understood the logic behind it.
In the example of "IAM Role session" just below the table, the following format is explicitly given: arn:aws:sts::111122223333:assumed-role/examplerole/examplerolesessionname
However, there are 2 entries of "IAM Role Session" in that table, so I would like to know how can a "IAM role session" accept a traditional "Role ARN" as a value? In other words, can we inject an regular role ARN under a "Role Session ARN"?

The table is about who is trying to take the action. When you assume a role and then attempt to take an action, the principal trying to take the action is the ARN of the role session, not the ARN of the role itself. It's not about an IAM role session accepting a role ARN as a value.

Yes, this is more clear, however:
I did a test with an IAM user, where I put a deny S3 on all actions in his identity policy as below:
{
"Version": "2012-10-17",
"Statement": [
{
"Sid": "VisualEditor0",
"Effect": "Deny",
"Action": "s3:",
"Resource": ""
}
]
}
and under the S3 bucket resource policy itself, I allowed that user all actions on S3 as below:
{
"Version": "2012-10-17",
"Statement": [
{
"Sid": "statement1",
"Effect": "Allow",
"Principal": {
"AWS": "arn:aws:iam::123123123123:user/MyUser123"
},
"Action": "s3:",
"Resource": [
"arn:aws:s3:::mybucket/",
"arn:aws:s3:::mybucket"
]
}
]
}
according to the table, the request should be Allowed, if the request is coming from the IAM user itself (which is the case), and the resource-based policy allows it (which is also the case), even if the identity policy denies access (also the case). However, the request is actually denied. Am I misunderstanding something?

The table covers the impacts of implicit denies. An explicit deny (your first policy) trumps all.

Indeed, I missed that part.
One last thing. Under the IAM Role row, it says "A role itself cannot make a request. Requests are made with the role session after a role is assumed."
However, under "IAM role session", the Resource-based policy says "Allows Role Arn", would not that not apply since the IAM Role itself cannot make requests as mentioned in the row before it?
Thanks for your help and answers.

Keep in mind this is about the impact of resource-based policies on policy evaluation. The principal making the request is still the role session. The table illustrates what the combination of policies will result in.

I understand, I just feel there are contradictions with the way the table is presented, in some cases we mention it is "N/A", and in others we treat it as a potentially valid entry in the resource policy.
Anyhow, I also consider an allow in a resource policy as an "explicit allow" in the evaluation logic, so in my opinion, it would be "normal" that the final result is allowed, even with the implicit denies.


Expected barcode count does not include samples with lower than average number of reads

A barcode issue was address with large samples sizes (N>10) in issue #140. When troubleshooting a new project in issue #144 it was noted that samples with lower than average reads will also fail this QC check. An additional config feature must be added.
Feature:

add "barcode_qc_flag" to the snakemake_config.yaml file. This will turn off the barcode QC checking and create an arbitrary 'passed qc' file
this should only be done after the user reviews the data and determines this is appropriate. If a user does this on first pass a sample may be processed with little to no reads, which may not be expected.

complete with commit 9acf3eaf33c6708b930308fbad4d0941dcee79b5


List ResourceDescriptors

Include support for the following:

[x] All MonitoredResourceDescriptors
[x] Pages

This issue replaces #494, which was attempted but could not be completed because the service API does not expose a way to list MonitoredResources.

Closed via #528.


StackedBubbles don't draw properly when bubble has multiple lines (plus quick fix)

When you translated to swift 3 you wrote M_PI_2 instead of M_PI when defining a path causing the StackedBubbles to be drawn in a weird manner.
Just go to StackedBubbles.swift and in the sizeToBounds func  change the last addArc func call:
    path.addArc(center: CGPoint(x: rect.minX, y: rect.minY), radius: radius2, startAngle: CGFloat(M_PI_2), endAngle: CGFloat(-M_PI_2), clockwise: false)

to
    path.addArc(center: CGPoint(x: rect.minX, y: rect.minY), radius: radius2, startAngle: CGFloat(M_PI), endAngle: CGFloat(-M_PI_2), clockwise: false)

Hi @protecuta. If you want to submit a PR for it, I'd be happy to merge it in. Otherwise, I can fix it.

That was what I originally wanted to do, but I'm already fighting a deadline and every minute counts. I'm sorry that I can't do a pull request right now


Connection Issue

Hello Mickey,
I'm a student from Germany and me and a small team of colleagues is working on a simple Dashboard App for iOS. It's just a small project for university and we are all not that deep into programming. We've tried to add your library with Cocoapods and used your example project to test the connection with an OBDLink MX+ Adapter. Unfortunately, we can't get the app to connect to our Adapter or get any sign of connectivity. The build succeeds without issues and we tried several things like adding a Bluetooth Usage Key in Info.plist but nothing is working out for us.
Can you or someone help us to find the problem? Would be really appreciated.
Best Regards

Use LTSupportAutomative Framework in your app instead of cocoapods it will work fine like sample code, there seems to be a bug in Pod file.

Use LTSupportAutomative Framework in your app instead of cocoapods it will work fine like sample code, there seems to be a bug in Pod file.


Dynamic Table & Connection Relationship

Laravel Version: 5.7.dev
PHP Version: 7.2.4
Database Driver & Version: mysql & 5.7.24

Description:
I have 2 models

Book
BookCategory

Book use connection mysql
BookCategory can use same connection mysql or other connection
I have dynamic categories tables. Like book_categories_1, book_categories_2, book_categories_3 and so on
Now I have a relation in Book Model
class Book extends Model
{
    protected $fillable = ['id', 'name','account_id' ];
    protected $table = 'books';

    public function categories()
    {
        $instance = new BookCategory($this->account_id);
        $foreignKey = 'book_id';
        $localKey = 'id';
        return $this->hasMany( $instance, $foreignKey, $localKey);
    }
}

BookCategory Model
class BookCategory extends Model
{
    protected $fillable = ['book_id', 'cat_id', 'col_a','col_b' ];
    protected $table = 'book_categories'; //this name will be dynamic

    public function __construct($account_id = null, array $attributes = [])
    {
        parent::__construct($attributes);

        if ($account_id != null ) {
            $this->setTable('book_categories_'.$account_id);
        }
        /*
         * use specific connection is available
         */
        $this->connection = getClientSpecificConnection($account_id);

    }
}

Database Tables

CREATE TABLE `book_categories_7` (
  `id` int(11) unsigned NOT NULL AUTO_INCREMENT,
  `book_id` int(11) DEFAULT NULL,
  `cat_id` int(11) DEFAULT NULL,
  `col_a` varchar(64) DEFAULT NULL,
  `col_b` varchar(64) DEFAULT NULL,
  PRIMARY KEY (`id`)
) ENGINE=InnoDB DEFAULT CHARSET=utf8;

CREATE TABLE `books` (
  `id` int(11) unsigned NOT NULL AUTO_INCREMENT,
  `name` varchar(11) DEFAULT NULL,
  `account_id` int(11) DEFAULT NULL,
  PRIMARY KEY (`id`)
) ENGINE=InnoDB DEFAULT CHARSET=latin1;

As you can see I am setting table name dynamically and also connection dynamically
Now when I call this code
$book = Book::first();
$ele  = $book->categories()->first();
Getting this error
"message": "SQLSTATE[42S02]: Base table or view not found: 1146 Table 'test.book_categories' doesn't exist (SQL: select * from `book_categories` where `book_categories`.`book_id` = 1 and `book_categories`.`book_id` is not null limit 1)",

I debug the code and found out that hasMany relationship is calling newRelatedInstance() method.
public function hasMany($related, $foreignKey = null, $localKey = null)
{
    $instance = $this->newRelatedInstance($related);

    $foreignKey = $foreignKey ?: $this->getForeignKey();

    $localKey = $localKey ?: $this->getKeyName();

    return $this->newHasMany(
        $instance->newQuery(), $this, $instance->getTable().'.'.$foreignKey, $localKey
    );
}

And in newRelatedInstance() function a (new $class) is being created.
protected function newRelatedInstance($class)
{
    return tap(new $class, function ($instance) {
        if (! $instance->getConnectionName()) {
            $instance->setConnection($this->connection);
        }
    });
}

Shouldn't it use the same object if we are passing an object in the hasMany relationship?
I mean like this

return tap( is_object( $class ) ? $class : new $class, function ($instance) {
    if (! $instance->getConnectionName()) {
        $instance->setConnection($this->connection);
    }
});

Changing this code fix the issue

Is this still a problem for you?

I have implemented it like this
`class Book extends Model
{
protected $fillable = ['id', 'name','account_id' ];
protected $table = 'books';
public function categories()
{
    $instance = new BookCategory($this->account_id);
    $instance->setTable( 'book_categories_'.$this->account_id );
    $foreignKey = 'book_id';
    $localKey = 'id';

    return $this->newHasMany(
        $instance->newQuery(), $this, $instance->getTable().'.'.$foreignKey, $localKey
    );
}

}`
Now I have to watch the newHasMany in every release. If that somehow change then I have to update my code as well. Not sure if this is the best way but that is serving my purpose right now.

I wouldn't consider this a bug in the framework. The use case breaks database norms as well as Active Record definition. The most sensible approach for the problem seem to be partitioning the table instead.
As for multiple connections, you can define the model to point to a dynamic connection and establish the connection on the fly. It allows relationships to work fine. It just doesn't work with joins or sub queries.

Closing this as this is currently not supported and also rejected in a follow-up pr: https://github.com/laravel/framework/pull/28728

@mubbashar , thanks man, It worked for me as well!
I was facing the same problem in belongsTo relationship where I have to set table in user model to guest_user. And I used newBelongsTo in relation.
`    public function guestUser()
{
$user = new User();
$user->setTable('guest_users');
    return $this->newBelongsTo($user->newQuery(), $this, 'userId', 'id', 'guestUser');
}`


Update User Guide for Edit Command

The user guide lacks clarity on how the edit command works for the recent changes of edit command.
Updating the guide improves user experience by providing more detailed instructions and practical examples for the edit command.
Let's revise the existing documentation and add relevant examples.
This makes the guide more user-friendly and reduces confusion when using the edit command.

LGTM.


typo: Double word "single"

@nschonni : Thanks for your contribution! The author, @"billgib", has been notified to review your proposed change.

@billgib I'm guessing the bot just choked on the quotes in the frontmatter and it was supposed to notify you

Merge typo fix


Warn for gulp changes to check for side effects

Checks if any gulp files have changed, if they have, show a warning and recommend testing with --testAll or --testMaster to prevent side effects

:+1:

Have you considered adding the testAll stuff as a requirement to the Travis run of this if a gulp file changes?

Yep - that's what line 372 does, if the file is in the gulp folder, or if it's the actual gulpfile itself, it must run all tests.


updating load balancer error about security group

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
One of our clusters has been having this alarm event in kong namespace. we have reviewed the information and tried to reproduce it in the development cluster, I have created multiple security groups in ec2 instances but it does not reproduce. I am not sure if this is a bug in k8s or kic . Thanks for any suggestions
apiVersion: v1
count: 84
eventTime: null
firstTimestamp: "2022-11-08T08:39:43Z"
involvedObject:
  apiVersion: v1
  kind: Service
  name: kong-proxy
  namespace: kong
  resourceVersion: "22431"
  uid: 9d1e7f4a-6bc6-42f5-bf94-89fa68a4691b
kind: Event
lastTimestamp: "2022-11-10T09:53:04Z"
message: 'Error updating load balancer with new hosts map[ip-172-1-16-186.ap-northeast-1.compute.internal:{}
  ip-172-1-16-193.ap-northeast-1.compute.internal:{} ip-172-1-16-45.ap-northeast-1.compute.internal:{}
  ip-172-1-17-23.ap-northeast-1.compute.internal:{} ip-172-1-17-31.ap-northeast-1.compute.internal:{}
  ip-172-1-17-45.ap-northeast-1.compute.internal:{} ip-172-1-18-18.ap-northeast-1.compute.internal:{}
  ip-172-1-18-217.ap-northeast-1.compute.internal:{} ip-172-1-18-244.ap-northeast-1.compute.internal:{}
  ip-172-1-18-89.ap-northeast-1.compute.internal:{} ip-172-1-19-147.ap-northeast-1.compute.internal:{}
  ip-172-1-19-174.ap-northeast-1.compute.internal:{} ip-172-1-19-182.ap-northeast-1.compute.internal:{}
  ip-172-1-19-192.ap-northeast-1.compute.internal:{}]: Multiple tagged security groups
  found for instance i-04a013b4b4b53abca; ensure only the k8s security group is tagged;
  the tagged groups were sg-03a941bffd2126566(eksctl-web3-resource-nodegroup-web3-a-resource-SG-1X5LQMUGF4QQP)
  sg-0c7cc302b41fd7cc1(eksctl-web3-resource-nodegroup-web3-d-resource-SG-WZ7UJO822IJ7) '
metadata:
  creationTimestamp: "2022-11-10T09:53:04Z"
  name: kong-proxy.17258f498c75ac77
  namespace: kong
  resourceVersion: "122010140"
  uid: 01177a1b-5acb-4d8e-a211-76dc4421e84d
reason: UpdateLoadBalancerFailed
reportingComponent: ""
reportingInstance: ""
source:
  component: service-controller
type: Warning

Expected Behavior
No warning event
Steps To Reproduce
No response
Kong Ingress Controller version
v2.7 installed by helm chart

Kubernetes version
v1.21.14

Anything else?
No response

Hi @J1a-wei I am facing the same issue. Can you please point out how did you solve this?


file_not_found： technical report version

Hello, in footnote 4 of your paper, you mentioned using the Sinkhorn distance, Sliced-Wasserstein distance, and random sampling to approximate the Wasserstein distance, and noted that further details would be provided in the technical report version. However, after clicking the link, the corresponding document is not available. Could you please provide it?
link：https://anonymous.4open.science/r/SynMeter/SynMeter_report.pdf

Hi @hmsklv0,
I am sorry our paper is under review and has some changes compared to the Arxiv version. We will update it soon.
As regards the computation of Wasserstein distance, the short answer is that we find direct random sampling is the most simple and efficient way to approximate the true Wasserstein distance between two marginals, which is also the current implementation in SynMeter.
Thank you,


Failed to build Pods

Dear team,
I am trying to build bowser in order to debug the webview. The Pods fail to download a dependency from http://static.verkstad.net, see below.
How can I build bowser differently?
Thanks and regards,
Fabian
cepheus:lib fubeh$ git clone https://github.com/EricssonResearch/bowser.git
Cloning into 'bowser'...
remote: Counting objects: 427, done.
remote: Total 427 (delta 0), reused 0 (delta 0), pack-reused 427
Receiving objects: 100% (427/427), 369.95 KiB | 415.00 KiB/s, done.
Resolving deltas: 100% (233/233), done.
Checking connectivity... done.
cepheus:lib fubeh$ cd bowser/bowser-ios/
cepheus:bowser-ios fubeh$ pod install
Updating local specs repositories

CocoaPods 1.0.0.beta.4 is available.
To update use: `gem install cocoapods --pre`
[!] This is a test version we'd love you to try.

For more information see http://blog.cocoapods.org
and the CHANGELOG for this version http://git.io/BaH8pQ.

Analyzing dependencies
Pre-downloading: `OpenWebRTC-SDK` from `https://github.com/EricssonResearch/openwebrtc-ios-sdk.git`, commit `d00dd8657df6dcb0212f1ff3f76cd8e6c780949c`
Downloading dependencies
Installing OpenWebRTC (0.3.91)

[!] Error installing OpenWebRTC
[!] /opt/local/bin/curl -f -L -o /var/folders/b0/q472nb855xs8zjdprg3l37w80000gn/T/d20160302-38094-hf0lws/file.zip http://static.verkstad.net/OpenWebRTC_0.3.91.zip --create-dirs

  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0curl: (7) Failed to connect to static.verkstad.net port 80: Connection refused

You are trying to download an old version of the pod (0.3.91) which is no longer available on that server. Please use the latest version 0.3.95.
To get the latest, run rm -rf Pods/ Podfile.lock and then pod install again.

Thanks, Stefan, this worked.
Now I receive a ld: library not found for -lOpenWebRTC-SDKerror even though OpenWebRTC-SDK is in the Pods directory.
Thanks for the support!

Are you opening the .xcworkspace file?

Ouch ...
Thanks for your support, all set now!


Feature/5665 export page

Pull Request

@Qb10 What about the Page title? Don't we need to change that?


Inconsistent naming of permissions

Describe the bug
I stumbled upon that the naming of permissions in the Terraform Provider differs from the official documentation.
Example:
See https://www.jfrog.com/confluence/display/JFROG/Permissions
Repository Permissions > Deploy / Cache is equal to the write const here?
Requirements for and issue

[x] A fully functioning terraform snippet that can be copy&pasted (no outside files or ENV vars unless that's part of the issue)
[x] Your version of artifactory v7.24.3
[x] Your version of terraform

Terraform v1.0.7
on darwin_amd64
+ provider registry.terraform.io/jfrog/artifactory v2.6.7

Expected behavior
Permissions should have a consistent naming pattern either in the docs or in the provider. Another option would be to provide documentation for the mapping of permissions names between the docs and the provider.
Additional context

So, what are you actually asking? That I change them all and break backward compatibility? Or something else?

I disagree with @cbrgm.
This is an abstracted layer over the API. I would assert that the Provider itself should be self-consistent, and it would be useful to update the docs for the Provider to explain if any re-mapping is occurring, but by disconnecting the Provider interface from the low-level API, the Provider interface can be more stable.
And I favor stability over consistency-for-the-sake-of-consistency. By at least being self-consistent within this software, we can apply the UX principle of context-over-consistency.

Thank you for your feedback @chb0github @skyzyx ! No, I'm not asking for this to be a breaking change at all. From a UX perspective, this issue is just an experience report on my part that it is a bit unclear which permissions from the official documentation now match those of the provider here. I think mentioning this in the provider documentation would be very helpful for many users.

I`d be happy to contribute a PR if you can tell me where this documentation would fit in!

I think documentation would be a good choice here. There are 2 places:

Update the MD: artifactory_permission_target.md
Add a description field to the permissions key capturing, effectively, The documentation from no. 1. The file is here:

Unfortunately, changing these is just a non-starter, but some clearer documentation wouldn't hurt.


On Linux app doesn't check if FirstRun dir exists

🔍 Is there already an issue for your problem?

[X] I have checked older issues, open and closed

📝 Description
On Linux, I had to create $HOME/.config/gitify/FirstRun/ by hand to make the app happy.
🪜 Steps To Reproduce

make sure $HOME/.config/gitify/FirstRun doesn't exists.
start gitify
Enjoy an error message

ℹ System Info
OS: Arch Linux
Project Version: 4.4.0.r3.0fc4e8f (`main` branch checkout)

📸 Screenshots
No response

Will someone look into this? The app is unusable on Linux (!!?)

https://github.com/gitify-app/gitify/blob/1396b6650be639951160ed773c487cb5b49ba412/first-run.js#L37-L51 should be creating this directory if not present.


Observe errors from task that are abandoned to stop UnobservedExceptions being handled by the Finalizer

Background

Results

Before
After
How to review this PR

Quality :heavy_check_mark:

Pre-requisites

[ ] I have read How we use GitHub Issues for help deciding when and where it's appropriate to make an issue.
[ ] I have considered informing or consulting the right people, according to the ownership map.
[ ] I have considered appropriate testing for my change.

[sc-53482]

LGTM I wonder if we could add something to the integration tests to look for any unobserved exceptions and cause some sort of failure if any of them look to be within the client.

We could probably add something but the Finalizer is never guaranteed to run so it would not be guaranteed to catch these issues. It would be better than nothing though. We could potentially try and force GC and wait a short time for the Finalizer to hopefully run at the end of the Test Run to see if any unobserved errors occur and try and find a way to fail something. That might help catch some stuff!! I'll card it!


Adding CryptoTrust Token [CTE]

Added images/CTE.png
Updated contract-map.json

@rekmarks, @whymarrh can anyone do the review, please? Thank you

Hi @CryptoTrustExchange - I would like to merge this request but there is a new requirement that the logo has to be in SVG format. Can I ask you to add it as a replacement for the PNG one? Also will be great if you can solve conflicts.  Thanks


Allow users to configure security definitions

Is your feature request related to a problem? Please describe.
If I want to build proTES without providing (or providing at a later time) an OpenAPI file, I won't be able to do it because it is currently required in app.py, since add_security_definitions=True.
Describe the solution you'd like
A very easy solution to this would be to get that flag from the app_config.yaml file, where the authorization_required key is already available, thus changing the above-mentioned line in app.py to add_security_definitions=get_conf(config, 'security', 'authorization_required').
This is consistent with what is done for other arguments of register_openapi.
Describe alternatives you've considered
At the moment I'm solving this by simply replacing the original app.py file with my own version that has that simple code change already in place, but it is quite annoying.
Additional context
N/A

Thanks for the suggestion and detailed feature request. It's a good idea!
We are working hard for preparing for a COVID-19 virtual hackathon that's gonna take place in the second week of April (perhaps you are interested?). Not sure if I will be able to do it before after that is concluded. Possibly someone else will, but no guarantees. If you'd rather have it sooner, you are very welcome to file a PR and assign me as reviewer. I'll surely merge this.

Sure, I'm happy to open a PR for that, it's an easy fix. Thanks for your availability, and for sharing the hackathon, will surely have a look at that!


Get-PodeLogger ParameterBindingValidationException

Describe the Bug
Updated to 2.11.0 and started noticed this show up in my debug logs occasionally. I haven't noticed any ill effect yet, but thought I'd bring it up in case it's a bug or something I need to fix in code. There's a chance this has always been happening but I know there were some logging changes in this release.
Steps To Reproduce
Date: 2024-11-13 12:44:56
Level: Error
ThreadId: 0
Server: SERVER01
Category: InvalidData: (:) [Get-PodeLogger], ParameterBindingValidationException
Message: Cannot bind argument to parameter 'Name' because it is an empty string.
StackTrace: at <ScriptBlock>, <No file>: line 20

Expected Behavior
No exception.
Platform

OS: Windows Server 2022
Browser: NA
Versions:

Pode: 2.11.0
PowerShell: PS7.4.6

Could you give me more information?
Do you have a custom log?

Could you give me more information? Do you have a custom log?

We have these setup, if that's what you're asking
New-PodeLoggingMethod -File -Name 'api_requests' -Path $cfgLogPath -MaxSize 25MB -Batch 50 -BatchTimeout 15 | Enable-PodeRequestLogging

New-PodeLoggingMethod -File -Name 'api_logs_debug' -Path $cfgLogPath -MaxSize 30MB | Enable-PodeErrorLogging -Levels Informational, Error, Warning, Debug, Verbose


Notifications required

I cannot get notifications on email regarding these VM related events like unplanned reboot. I can see them in web interface, yet I cannot set up any alerts for them. Is the alerting feature missing for this or is there any other problem I am experiencing that keeps me unable to set it up?

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

ID: ad38fb43-e416-063b-86b1-8d170302cef9
Version Independent ID: ab706e1e-4009-dff1-0db2-491447518756
Content: Manage the availability of Linux VMs in Azure
Content Source: articles/virtual-machines/linux/manage-availability.md
Service: virtual-machines-linux
GitHub Login: @cynthn
Microsoft Alias: cynthn

Thanks for the feedback! We are currently investigating and will update you shortly.

You are able to get email alerts for these type of events in Azure.
I would suggest you start with this doc to get a better idea:
https://docs.microsoft.com/en-us/azure/azure-monitor/platform/alerts-overview
You can setup alerts for any action performance against the VM
https://docs.microsoft.com/en-us/azure/azure-monitor/platform/activity-log-alerts
Or in your case, if there is a service impacting event
https://docs.microsoft.com/en-us/azure/service-health/alerts-activity-log-service-notifications


Clarify best practice for load balancer(s) in multi-region deployments

Do we advocate for the use of global load balancers for the DB, or deploying app instances in each region and load balancing across these?

@mgoddard I think it's the latter?  According to a scan of https://www.cockroachlabs.com/docs/v21.1/movr-flask-deployment#global-application-deployment
however I am not super familiar with this area - @ericharmeling do you agree with the above?
If yes, and this is what we recommend, we probably need more docs that mention this pattern, and link to that page

Here are some of my thoughts (probably a good idea to loop in engineering though)...
For multi-region application deployments, we should advocate for a global load balancer in front of an application deployed in multiple regions, since it's likely the case that users/clients in different regions will be accessing a single entry point (e.g., a domain name). Each of the regional app instances can then directly connect to the gateway node in the regional database deployment closest to them.
Region-specific load balancers are preferred for multi-region database deployments. I don't think we want to advocate for a single, global load balancer in front of a multi-region database deployment, unless there is a single regional application deployment (not ideal with a multi-region database deployment on the backend), or unless end users/clients in different regions are accessing the database directly through a single entry point.
For the regional database load balancer(s), it's helpful to distinguish the two possible types of CRDB clusters that can do multi-region, as our recommendations for the two are a little different:

CockroachDB Dedicated (formerly known as CockroachCloud)
For our multi-region application deployment tutorial, we deploy app instances in each region, with a single global, load balancer in front of the app deployments. The app uses a multi-region CockroachDB Dedicated cluster on the backend, with no load balancers manually set up. According to our docs on CockroachDB Dedicated architecture, each region in a managed, dedicated cluster gets a single load balancer:

Users connect to a Dedicated cluster by using a load balancer in front of each region which leads to one connection string per region

So for Dedicated clusters, users only need to worry about setting up the application load balancer.

CockroachDB Self-hosted (i.e., manually-deployed CockroachDB core)
The guidance here is a little different, since CRL isn't managing the database deployment. Here is an example deployment pattern for each region, from our docs:

1 region with 3 AZs
3+ VMs evenly distributed across AZs; add more VMs to increase throughput
App and load balancer in same region as VMs for CockroachDB
The load balancer redirects to CockroachDB nodes in the region

I'm pretty sure this guidance is found in other places in our Deployment docs as well (@taroface can confirm?). So if you deploy the database manually in multiple regions, you should set up a load balancer in each region.

Re: what @rmloveland said

we probably need more docs that mention this pattern, and link to that page

I totally agree in the case of the global app load balancer. It's really only demonstrated in that multi-region app tutorial. This docs issue is a great place to track progress on these docs.
@awoods187 @ajstorm
Does that sound right?

Sounds about right to me.  One thing I'd add is that customers may want a global load balancer on top of the regional load balancers.  This would handle cases where the entire region goes down, but the apps are still alive.  In this case, you'd want them to automatically re-route to one of the surviving regions.  With only region-specific load balancers, this type of advance failover isn't possible.


Todoist login

Let's change Todoist login method from "user api key" to regular authentication. It's more user friendly.
There should be button in settings which connects or disconnects Todoist with the app.
There's a need to redesign settings too:

Add close if clicked away from modal event
No configuration but component composition should be used. It's much easier to customize
Changes should be saved automatically without need of a button. That means that changes should appear immediately.

Add option to specify project. Only its items will be shown

Add option to specify project. Only its items will be shown

Solved in #59


fix(connector): [Worldpay] use 4 digit expiry year

Type of Change

[x] Bugfix
[ ] New feature
[ ] Enhancement
[ ] Refactoring
[ ] Dependency updates
[ ] Documentation
[ ] CI/CD

Description
Described in #6316 point # 7
Additional Changes

[ ] This PR modifies the API contract
[ ] This PR modifies the database schema
[ ] This PR modifies application configuration/environment variables

Motivation and Context
Helps in transforming expiry date format into a 4 digit value for card payments for worldpay connector!
How did you test it?

1. Create a payment using 2 digit expiry year
cURL
curl --location 'http://localhost:8080/payments' \
    --header 'Content-Type: application/json' \
    --header 'Accept: application/json' \
    --header 'api-key: dev_vK8vdO0cS4wec9MP0W3K8eR0GfgrVoY0oxuhJrx4rCBJm7wxoH16bOdx9fLKvYSc' \
    --data-raw '{"amount":6500,"currency":"USD","confirm":true,"capture_method":"automatic","capture_on":"2022-09-10T10:11:12Z","customer_id":"cus_L8pJ8g0mnsdHmS0FPqrH_4","email":"guest@example.com","name":"John Doe","phone":"999999999","profile_id":"pro_1frLOUlLMVZWSYwOz3En","phone_country_code":"+65","description":"Its my first payment request","authentication_type":"no_three_ds","return_url":"https://hyperswitch.io","setup_future_usage":"on_session","customer_acceptance":{"acceptance_type":"offline","accepted_at":"1963-05-03T04:07:52.723Z","online":{"ip_address":"<IP_ADDRESS>","user_agent":"amet irure esse"}},"connector":["worldpay"],"payment_method":"card","payment_method_type":"debit","payment_method_data":{"card":{"card_number":"5555555555554444","card_exp_month":"08","card_exp_year":"25","card_holder_name":"John Doe","card_cvc":"123"}},"billing":{"address":{"line1":"1467","line2":"Harrison Street","line3":"Harrison Street","city":"San Fransico","state":"California","zip":"94122","country":"US","first_name":"John"}},"statement_descriptor_name":"joseph","statement_descriptor_suffix":"JS","metadata":{"udf1":"value1","new_customer":"true","login_date":"2019-09-10T10:11:12Z"},"browser_info":{"ip_address":"<IP_ADDRESS>","user_agent":"Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36","accept_header":"text/html,application/xhtml+xml,application/xml;q=0.9,image/webp,image/apng,*/*;q=0.8","language":"en-US","color_depth":32,"screen_height":1117,"screen_width":1728,"time_zone":-330,"java_enabled":true,"java_script_enabled":true}}'

Response
{"payment_id":"pay_YaS6joNtZ9HBVpj9MeBc","merchant_id":"merchant_1731407657","status":"processing","amount":6500,"net_amount":6500,"shipping_cost":null,"amount_capturable":0,"amount_received":null,"connector":"worldpay","client_secret":"pay_YaS6joNtZ9HBVpj9MeBc_secret_hKl5jcvcPvtajzBOoKFt","created":"2024-11-12T10:53:30.637Z","currency":"USD","customer_id":"cus_L8pJ8g0mnsdHmS0FPqrH_4","customer":{"id":"cus_L8pJ8g0mnsdHmS0FPqrH_4","name":"John Doe","email":"guest@example.com","phone":"999999999","phone_country_code":"+65"},"description":"Its my first payment request","refunds":null,"disputes":null,"mandate_id":null,"mandate_data":null,"setup_future_usage":"on_session","off_session":null,"capture_on":null,"capture_method":"automatic","payment_method":"card","payment_method_data":{"card":{"last4":"4444","card_type":"CREDIT","card_network":"Mastercard","card_issuer":"MASTERCARD INTERNATIONAL","card_issuing_country":"BRAZIL","card_isin":"555555","card_extended_bin":null,"card_exp_month":"08","card_exp_year":"25","card_holder_name":null,"payment_checks":null,"authentication_data":null},"billing":null},"payment_token":null,"shipping":null,"billing":{"address":{"city":"San Fransico","country":"US","line1":"1467","line2":"Harrison Street","line3":"Harrison Street","zip":"94122","state":"California","first_name":"John","last_name":null},"phone":null,"email":null},"order_details":null,"email":"guest@example.com","name":"John Doe","phone":"999999999","return_url":"https://hyperswitch.io/","authentication_type":"no_three_ds","statement_descriptor_name":"joseph","statement_descriptor_suffix":"JS","next_action":null,"cancellation_reason":null,"error_code":null,"error_message":null,"unified_code":null,"unified_message":null,"payment_experience":null,"payment_method_type":"debit","connector_label":null,"business_country":null,"business_label":"default","business_sub_label":null,"allowed_payment_method_types":null,"ephemeral_key":{"customer_id":"cus_L8pJ8g0mnsdHmS0FPqrH_4","created_at":1731408810,"expires":1731412410,"secret":"epk_e83ffd4da9e542ef90d16ce72cf6ac9c"},"manual_retry_allowed":false,"connector_transaction_id":"eyJrIjoiazNhYjYzMiIsImxpbmtWZXJzaW9uIjoiNS4wLjAifQ==.sN:g8wd64bwkbrp0md+bPxcanBnk2zLdsIqSa1pR99GGg8fCNQpPLoWNslSzWNPFBM5Tpa8tW7EFI5onKINsgChMHeJVoeH2lrBWCRyjZYT6h+lbqfJa+1BSoKFSY8HLge3KVdfHweTSyJ5K44EVLaxYshRzFR5L:3DblLntpx6mfgF9Uk2Www3mSVoKRxeIXIe9pic4l22Rveu3MW0qp:7+4GYJYOg3e9WdLnxDKf8zDOVuNV9Smp3TrKAPuNF+mGHv+EpbUsv07dKPa3jQZMZkSLOAGYpq4tIxEOnFazFMnNixp9xnyekZxhsndH6QYG9PvEiRYH4jvUN3Rj+v30knvWcoLtqA6gq4131NoZaq1HEMk0Or2+rYD5MFugS490qxbGU5BlL+g0uitRgZrQ==","frm_message":null,"metadata":{"udf1":"value1","login_date":"2019-09-10T10:11:12Z","new_customer":"true"},"connector_metadata":null,"feature_metadata":null,"reference_id":"3a722672-4625-45bd-ab45-0393d9be067b","payment_link":null,"profile_id":"pro_1frLOUlLMVZWSYwOz3En","surcharge_details":null,"attempt_count":1,"merchant_decision":null,"merchant_connector_id":"mca_4VLjE6cyJN8rQeR7yhYV","incremental_authorization_allowed":null,"authorization_count":null,"incremental_authorizations":null,"external_authentication_details":null,"external_3ds_authentication_attempted":false,"expires_on":"2024-11-12T11:08:30.637Z","fingerprint":null,"browser_info":{"language":"en-US","time_zone":-330,"ip_address":"<IP_ADDRESS>","user_agent":"Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36","color_depth":32,"java_enabled":true,"screen_width":1728,"accept_header":"text/html,application/xhtml+xml,application/xml;q=0.9,image/webp,image/apng,*/*;q=0.8","screen_height":1117,"java_script_enabled":true},"payment_method_id":null,"payment_method_status":null,"updated":"2024-11-12T10:53:32.161Z","charges":null,"frm_metadata":null,"merchant_order_reference_id":null,"order_tax_amount":null,"connector_mandate_id":null}

Checklist

[x] I formatted the code cargo +nightly fmt --all
[x] I addressed lints thrown by just clippy && just clippy_v2
[x] I reviewed the submitted code
[ ] I added unit tests for my changes where possible

@kashif-m Can you verify same in cypress test by passing 2 digit for worldpay


Options?

I may be blind but I cannot for the life of me find the options window. The size of some things are big and I'd like to resize or disable some things or even move them..

There is currently no configuration window but Goldpaw is working on one with some basic highly requested features.

Ah ok. Didn't know this was a brand new addon when I downloaded it lol. Didn't say anything in the addon description. But at least now I know 👍

You can always read the Wiki for some info.
https://github.com/cogwerkz/DiabolicUI/wiki


Multi-tenanted application needs to defer the secretOrKey until after the token is retrieved

My application needs to be able to defer the use of the secretOrKey until after the jwt is unpacked as it's a multi-tenanted application where each issuer has been issued its own unique key.
In this case the secretOrKey option needs to be a function whose callback returns the correct secret or key, rather than it being a value.
This requires a small enhancement to verify_jwt.js which I'd be happy to do.

Do you have examples of other strategies that work this way?  I was looking for examples and was unable to find any.

Closing due due to no response from author.

Sorry I didn't realise you were waiting on a response from me. I am not sure of other impls but this PR solved our immediate problem.
I'm not on that project any more so do what you like. :-)
Dave
Sent from my iPhone

On 25 Feb 2016, at 1:21 PM, Mike Nicholson<EMAIL_ADDRESS>wrote:
Closing due due to no response from author.
—
Reply to this email directly or view it on GitHub.

Ok, maybe I have misunderstood how JWT token is supposed to work. But lets say that I have a REST API that I want to authenticate using JWT. I would expect that every user of the API has his own secret instead of sharing the same key to all the users... how do you solve this scenario when the secretOrToken is a fixed value?

@manast the secret in this case is something your service supplies to client services and apps, not to specific users.


Inconsistent joint-transformation of images and masks with ImageDataGenerator

I am using ImageDataGenerator to transform images and masks together for semantic segmentation in the style presented in the Keras docs where two generators get zipped together and then get fed to a flow method. However, when I save the augmented image-mask pairs to a directory, the file naming indicates that the seeding for the augmentation isn't always the same for the image and mask. This is despite feeding the generators the same seed and also playing around with the setting of seeds at the top of the file. Here are the files produced in my 'aug' folder with the code below:
image_19_8801850.png
image_214_4543356.png
image_268_864897.png
image_388_2338108.png
image_394_7960136.png
image_516_6392335.png
mask_19_8801850.png
mask_214_4543356.png
mask_268_864897.png
mask_388_2338108.png
mask_394_8946097.png
mask_516_7424466.png
You can see that the seeds used for pairs 394 and 516 were different. The number of pairs generated that have different seeds seems to equal the batch_size*max_queue_size.
Any help would be appreciated, as I want to make sure all image-mask pairs are augmented in the same fashion.
Keras:  2.2.4
Keras-Applications:   1.0.6
keras-contrib :   2.0.8
Keras-Preprocessing:  1.0.5
tensorflow-gpu:  1.12.0
Ubuntu 16.04.5 LTS (GNU/Linux 4.4.0-1074-aws x86_64v)
import random
random.seed(1)
import numpy as np
np.random.seed(10)
import tensorflow as tf
tf.reset_default_graph()
graph_level_seed = 1
tf.set_random_seed(graph_level_seed)
operation_level_seed = 1

import os
from find_onh_model import unet
from keras.callbacks import EarlyStopping, TensorBoard, ModelCheckpoint
import gc

import imageio
from sklearn.utils import class_weight
from keras.preprocessing.image import ImageDataGenerator

gc.collect()

def adjust_data(img,mask,flag_multi_class,num_class):
    if(flag_multi_class):
    elif(np.max(img) > 1):
        img = img / 255
        mask = mask /255
        mask[mask > 0.5] = 1
        mask[mask <= 0.5] = 0
    return (img,mask)

def TrainGenerator(batch_size,train_path,image_folder,mask_folder,aug_dict,image_color_mode = "grayscale",
                    mask_color_mode = "grayscale",image_save_prefix  = "image",mask_save_prefix  = "mask",
                    flag_multi_class = False,num_class = 2,save_to_dir = None,target_size = (256,256),seed = 1,
                    subset=None):

    image_datagen = ImageDataGenerator(**aug_dict)
    mask_datagen = ImageDataGenerator(**aug_dict)
    image_generator = image_datagen.flow_from_directory(
        train_path,
        classes = [image_folder],
        class_mode = None,
        color_mode = image_color_mode,
        target_size = target_size,
        batch_size = batch_size,
        save_to_dir = save_to_dir,
        save_prefix  = image_save_prefix,
        seed = seed,
        subset = subset)
    mask_generator = mask_datagen.flow_from_directory(
        train_path,
        classes = [mask_folder],
        class_mode = None,
        color_mode = mask_color_mode,
        target_size = target_size,
        batch_size = batch_size,
        save_to_dir = save_to_dir,
        save_prefix  = mask_save_prefix,
        seed = seed,
       subset = subset)
    train_generator = zip(image_generator, mask_generator)
    for (img,mask) in train_generator:
        img,mask = adjust_data(img,mask,flag_multi_class,num_class)
        yield (img,mask)

# Load masks to get number of images available, image dimensions, and appropriate class weighting.
def load_images_from_folder(folder):
    images = []
    for image_name in os.listdir(folder):
        path = os.path.join(folder, image_name)
        im_arr = imageio.imread(path, pilmode='L')
        images.append(im_arr)
    return np.array(images)

usr_dir = os.path.expanduser('~')
train_data_dir = usr_dir

saved_models_dir = os.path.join(train_data_dir, 'saved_models')
os.makedirs(saved_models_dir, exist_ok=True)
mask_path = os.path.join(train_data_dir, 'label', 'image')
aug_data_dir = os.path.join(train_data_dir, 'aug')

masks = load_images_from_folder(mask_path)
n_images, height, width = masks.shape
print('Data dimensions: {}'.format(masks.shape))

height_new = 128 # height
width_new = 128 # width

# config
image_folder = 'data'
mask_folder = 'label'
VALID_SPLIT = 0.2
n_valid_images = np.floor(n_images*VALID_SPLIT)
n_train_images = np.ceil(n_images*(1 - VALID_SPLIT))
batch_size = 2
STEPS_PER_EPOCH = 1
N_EPOCHS = 1
VALIDATION_STEPS = np.floor(n_valid_images/batch_size)
PATIENCE = 10

train_aug_dict = dict(
                    rotation_range=90, #0.2,
                    width_shift_range=0.1,
                    height_shift_range=0.1,
                    shear_range=0.05,
                    zoom_range=0.05,
                    horizontal_flip=True,
                    vertical_flip=False,
                    fill_mode='constant',
                    cval=0,
                    validation_split=VALID_SPLIT,
                    )
train_generator = TrainGenerator(batch_size,train_data_dir,image_folder,mask_folder,train_aug_dict,image_color_mode = "grayscale",
                    mask_color_mode = "grayscale",image_save_prefix  = "image",mask_save_prefix  = "mask",
                    flag_multi_class = False,num_class = 2,
                    save_to_dir = aug_data_dir,
                    target_size = (height_new, width_new),
                    seed = operation_level_seed,
                    subset='training')

val_aug_dict = dict(
    validation_split=VALID_SPLIT,
     )

val_generator = TrainGenerator(batch_size,train_data_dir,image_folder,mask_folder,val_aug_dict,image_color_mode = "grayscale",
                    mask_color_mode = "grayscale",
                    flag_multi_class = False,num_class = 2,
                    target_size = (height_new, width_new),
                    seed = operation_level_seed,
                    subset='validation')

model = <mymodel>

save_best_only=True, mode='auto')
callbacks = [EarlyStopping(monitor='val_loss', patience=PATIENCE)]

print('Training')
history = model.fit_generator(train_generator, max_queue_size=1, steps_per_epoch=STEPS_PER_EPOCH,epochs=N_EPOCHS,callbacks=callback\
s, validation_data=val_generator, validation_steps=VALIDATION_STEPS, class_weight=class_weighting, shuffle=False)

I've noticed that changing the max_queue_size argument doesn't change training performance, so perhaps this issue is merely confined to image file generation?

Could you post your issue there: https://github.com/keras-team/keras-preprocessing/issues Thanks!


Current stable release version?

Based on https://github.com/jmxtrans/jmxtrans/releases/latest it's not clear to me whether 246 or 266 is the current stable release version. Could you clarify which should be in Homebrew now? Thanks!

I completely forgot about Github releases! Latest releases are always pushed to Maven Central. So 266 is the current latest. Releases are reputed stable unless proven otherwise. Intermediate development is published as snapshots following the usual Maven practices.
Not sure what to do about the Github releases. Should we keep them as archived? The main jmxtrans README does document where releases are, but that might not be enough. Feel free to submit a PR to that README file to clarify...

You just need to delete the explicit releases on GitHub, which should not disrupt the tags themselves.
https://github.com/jmxtrans/jmxtrans/releases/tag/v246
https://github.com/jmxtrans/jmxtrans/releases/tag/v245
https://github.com/jmxtrans/jmxtrans/releases/tag/v244
https://github.com/jmxtrans/jmxtrans/releases/tag/v243

Ok, releases are deleted on Github. This should be enough to cose this issue, feel free to reopen if needed.

@gehel yup, looks good! https://github.com/jmxtrans/jmxtrans/releases/latest now resolves to https://github.com/jmxtrans/jmxtrans/releases as expected. Thanks!

Note that I'm entirely unsure how MacOS support is... I don't own a Mac myself. If you find anything that would make sense to be corrected, please send a PR!

@gehel the analytics are
Josephs-MacBook-Pro:Formula joe$ brew formula-analytics --days-ago=100 jmxtrans
install events in the last 100 days for jmxtrans
=====================================================================================================
1 | jmxtrans                                                                           | 85 | 100.00%
=====================================================================================================
Total                                                                                | 85 |    100%
=====================================================================================================
Josephs-MacBook-Pro:Formula joe$ brew formula-analytics --days-ago=100 --install-on-request jmxtrans
install_on_request events in the last 100 days for jmxtrans
=====================================================================================================
1 | jmxtrans                                                                           | 43 | 100.00%
=====================================================================================================
Total                                                                                | 43 |    100%
=====================================================================================================

and, no complaints, so I think it's fine :)


shifted from opton api to composition api

Changes Made

Vue 3 composition API implementation and Layout handled through app.js
migration file refactor
Model scope  refractor
route file refactor
Vite config fixes

Hi there @anilkumarthakur60. We'd like to thank you for this but want to make you aware of https://github.com/inertiajs/pingcrm/issues/174. This is most likely something we'll handle ourselves over time as it's too much for us to review in one single PR.

Thanks for the heads up. I'll keep that in mind for next time


Bouncer

What does this PR do?

Pending Description

@yaahc to avoid flooding with notifications and so, I will say "will do" for all those comments that I didn't answer in particular. In this sense only one notification. Thank you very much for the comments.

@yaahc to avoid flooding with notifications and so, I will say "will do" for all those comments that I didn't answer in particular. In this sense only one notification. Thank you very much for the comments.

Sounds good, fwiw I generally use emojis when I want to reply without a notification.

looks like you got everything except the deserialization changes I mentioned, lmk if you need help on that part.

Oh! Forgot, let me work on that, thanks!!

looks like you got everything except the deserialization changes I mentioned, lmk if you need help on that part.

Check out the deserialization! hope looks better now


Add invoke tasks for consistent validation

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

2 of 6 (33.33%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.006%) to 5.917%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

swimprotocol/config.py
0
1
0.0%

swimprotocol/demo/screen.py
0
1
0.0%

swimprotocol/udp/pack.py
0
2
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.006%

Covered Lines:
60

Relevant Lines:
1014

💛  - Coveralls


Emissive map no longer works in ign-rendering6 (ogre2)

Environment

OS Version: Ubuntu 18.04
Source or binary build?
source build: ign-rendering6 branch

Description

Expected behavior: objects are rendered with emissive map
Actual behavior: emissive map has no effect

I think the feature is broken when we ported from ogre 2.1 to 2.2.
Steps to reproduce

Test with ignition gazebo's fuel_textured_mesh.sdf world

ign gazebo -v 4 fuel_textured_mesh.sdf

The tubes in the Tube Light model do not appear to be lit when emissive map is used.

Here is a comparison of the Tube Light model in Ignition Fortress (top) and Dome (bottom)

Things I've tried
The call to set ogre2 emissive map is done here:
https://github.com/ignitionrobotics/ign-rendering/blob/ign-rendering6/ogre2/src/Ogre2Material.cc#L473

I verified that this->ogreDatablock->getUseEmissiveAsLightmap() returns false and this->ogreDatablock->_hasEmissive() returns true
setting emissive values (this->ogreDatablock->setEmissive) works
I tried debugging the code in 800.PixelShader_piece_ps.any for applying emissive map and it looks like the emissive_map property is not set hence it does not affect the final color.

partial fix in #501. It's now rendering red color for grayscale emissive maps. We may need to look for a way to tell ogre that the emissive map is grayscale or convert the grayscale emissive texture to RGB.

fix for loading grayscale emissive map in #503


Option to Observe/Return entire Response

The generator should have an option from each api function to pass on an option whether to return the entire response for one use case  and/or to return the expected response data.
Currently the response body is what we're getting/observing.
@vmasek @gelidus
IMO the default value returned should be simplistic as to return the response body alone for most of the use cases.
Expected code snapshot when observing the entire response in

private sendRequest
function

   private sendRequest<T>(method: string, uri: string, headers: HttpHeaders, params: HttpParams, body: any): Observable<HttpResponse<T>> {     if (method === 'get') {       return this.http.get<T>(this.domain + uri, { headers: headers.set('Accept', 'application/json'), params: params, observe: 'response' });     }    }

Hi, this issue should be resolved by adding support for passing request options as requested in #4 .
Also, there will be default headers and params that can be set to affect each request.
Why do you need the Response itself in your use-cases? Currently, we don't need such behavior.

@vmasek I don't see having default headers and params to each request feature needed here, as DI will allow you to replace the service with extended service in case you need such behavior.

@gelidus yes you are right, my answer for headers and params was covering only part of the problem which is setting an Accept header

There is a new version out, please test if your request was fulfilled.

@resistancecanyon if you are referring to this request method https://github.com/angular/angular/blob/master/packages/common/http/src/client.ts#L192 , then yes, you will also be able to override the observe options on any request when this is integrated.

I'll test it out & let you guys know.
Thanks,


"X" button behavior limited in functionality.

The "x" button behavior could benefit from improvement/bug-fixing. The keyword argument "cancel_choice"  should allow you to define a string, which will be returned if the user presses the "x" button.
The "x" button appears to be limited to returning either None or False.
Returning “False” is problematic for the following case and similar cases, where pressing “x” would be the same as clicking “No” which should not be the expected result of clicking “x”
Human = easygui.ynbox(“Are you a Human?”)
If Human == True:
    easygui.msgbox(“You are a human! Welcome to the human club!”)
elif Human == False:
    easygui.msgbox(“You are not a human! Welcome to the non-human club!”)
else:
    sys.exit()

In this case and similar cases, the 'else' case will never be caught. Ideally, boolboxes, ynboxes and the like should have the option to return some value other than True or False when the 'x' button is pressed.
AT THE VERY LEAST: pressing "x" should either ALWAYS return None for any kind of box, or you should be able to explicitly define what is returned when the x button is pressed.

. I was able to successfully modify the derived_boxes.py in the EasyGUI module to get the desired result.
Where the boolbox function is defined I changed
if reply == choices[0]:
    return True
else:
    return False

TO:
if reply == choices[0]:
    return True
elif reply == choices[1]:
    return False
else:
    return None

Now, I am able to distinguish when the "x" button is pressed, versus when "No" being selected in a ynbox (and any other box that wraps a boolbox)

confirmed.  Good idea.

confirmed.  Good idea.

I finally got around to making this change.


feat(sca): #PLC-763 sca scan all command

This PR implements SCA scan all command logic.

input: a directory
filters sca related files with a call to SCA compute files API
(NOT IMPLEMENTED) launches the SCA scan with a call to SCA public api (not merged in backend for now)
returns the raw_results (output handling will be done later)

Codecov Report

Merging #578 (35f029b) into main-sca (4922c25) will decrease coverage by 0.79%.
The diff coverage is 51.56%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@             Coverage Diff              @@
##           main-sca     #578      +/-   ##
============================================
- Coverage     93.81%   93.03%   -0.79%
============================================
  Files            87       89       +2
  Lines          3928     3990      +62
============================================
+ Hits           3685     3712      +27
- Misses          243      278      +35

Flag
Coverage Δ

unittests
93.03% <51.56%> (-0.79%)
:arrow_down:

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

Impacted Files
Coverage Δ

ggshield/cmd/sca/scan.py
47.91% <27.27%> (-40.32%)
:arrow_down:

ggshield/sca/client.py
76.47% <50.00%> (-18.53%)
:arrow_down:

ggshield/sca/filter.py
100.00% <100.00%> (ø)

ggshield/sca/models.py
100.00% <100.00%> (ø)

ggshield/scan/scan_mode.py
100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes

@gg-mmill , @amascia-gg still a draft, but can you take a quick look on this when you have a moment ?

Thanks, let's wait on the route to be available !

@gg-mmill @tchaumeny Ready for review !

Thanks, all good - except I have added file_selection.py as well :angel:

@agateau-gg Indeed, it's just temporary waiting for the backend side to go in production. The cassettes were created with staging.


Upgrade MC

Hi @RasmusOrsoe,
I have added an I3FeatureExtractorUpgrade to read Upgrade data from I3-files. I have also made the DataConverter classes completely generic wrt. which data are extracted which means that we can now read multiple pulsemaps from the same I3-files, and write to the same SQLite database, at the same time. To do this, I have re-written some of SQLiteDataConverter to be more generic and remove hard-coded references to e.g. "truth", "RetroReco", etc.
Let me know what you think! (The next step will then be ingesting Upgrade data into the model, but one thing at a time.)
Closes #46.

Absolutely:
All instances of I3Extractor now has a name attribute [1]. This is what is used to determine the name of the table corresponding to the extraction in question [2,3]. For instance, in I3FeatureExtractor the name is the same as the pulsemap [4], so that is completely unambiguous. For the truth and retro extractors, the default names are "truth" and "retro" [5,6], respectively, but these can be overridden using the first argument to the constructors, e.g. I3RetroExtractor("RetroReco").
Wrt. the EXPLAIN QUERY PLAN test, I have followed your helpful suggestions (thanks!) and run the following code snippet for existing database files as well as ones generated with the proposed code changes:
import sqlite3
import pandas as pd

db = "/groups/icecube/asogaard/temp/sqlite_test_upgrade/data_test/data/data_test.db"
#db = "/groups/icecube/leonbozi/datafromrasmus/GNNReco/data/databases/dev_level7_noise_muon_nu_classification_pass2_fixedRetro_v3/data/dev_level7_noise_muon_nu_classification_pass2_fixedRetro_v3.db"
pulsemap = "I3RecoPulseSeriesMapRFCleaned_mDOM"
#pulsemap = "SRTTWOfflinePulsesDC"
event_number = 1
with sqlite3.connect(db) as conn:
    query1 = f"EXPLAIN QUERY PLAN select * from {pulsemap} where event_no = {event_number}"
    query2 = f"EXPLAIN QUERY PLAN select energy from truth where event_no = {event_number}"
    res1 = pd.read_sql(query1, conn)
    res2 = pd.read_sql(query2, conn)

(Commented-out lines are the reference.)
The results of the first query, for the two files, are:
SEARCH I3RecoPulseSeriesMapRFCleaned_mDOM USING INDEX event_no_I3RecoPulseSeriesMapRFCleaned_mDOM (event_no=?)  # New
SEARCH SRTTWOfflinePulsesDC USING INDEX event_no_SRTTWOfflinePulsesDC (event_no=?)  # Original

which seems to be the same apart from a pulse-series specific name for the index column (event_no_<pulsemap>).
The results of the second query are the exact same for the two files:
SEARCH truth USING INTEGER PRIMARY KEY (rowid=?)

Let me know whether you are happy with there results or would like to see additional checks.

[1] https://github.com/asogaard/gnn-reco/blob/b8527fd5907fa1f325a9a4176a36de4bbdda1a7f/src/gnn_reco/data/i3extractor.py#L13-L45
[2] https://github.com/asogaard/gnn-reco/blob/b8527fd5907fa1f325a9a4176a36de4bbdda1a7f/src/gnn_reco/data/sqlite_dataconverter.py#L64
[3] https://github.com/asogaard/gnn-reco/blob/b8527fd5907fa1f325a9a4176a36de4bbdda1a7f/src/gnn_reco/data/sqlite_dataconverter.py#L188-L192
[4] https://github.com/asogaard/gnn-reco/blob/b8527fd5907fa1f325a9a4176a36de4bbdda1a7f/src/gnn_reco/data/i3extractor.py#L66-L69
[5] https://github.com/asogaard/gnn-reco/blob/b8527fd5907fa1f325a9a4176a36de4bbdda1a7f/src/gnn_reco/data/i3extractor.py#L197
[6] https://github.com/asogaard/gnn-reco/blob/b8527fd5907fa1f325a9a4176a36de4bbdda1a7f/src/gnn_reco/data/i3extractor.py#L242


feature support cast to unsigned / signed

from mysql doc: https://dev.mysql.com/doc/refman/8.4/en/cast-functions.html#function_cast

run buildall

run buildall


ACCESS DENIED

Hi guys.
I have a problem using axios in INTERNET EXPLORER 11, I have error "access denied", but same request in google chrome the success
My code with axios.
        axios.post(_urlRequest,params, { timeout: pTimeout})           .then(response => {             console.log('@response@', response)             pRetorno(JSON.parse(response.data))           })           .catch(error => {             console.log('@error@', error)             pRetorno({ correcto: false, mensaje: error, listaResultado: [] })           })

Sorry but this doesn't look like an issue in Axios. Please review your code and reopen if you find more information that points to it.


[BUG/ISSUE] (Add the ability to change the folder for installing programs)

Hello, add the ability to change the folder for installing the program, that is, the autoruns program, which is not installed and it just opens after the jump, then it installs in an unknown folder for me to look for it, but I never found it and deleted it

Hi @mimimass16,
I don't understand you. Could you please provide more details?

If right-clicking -> packatge info, and then you copy the shown command and run it on a cmd window, does it do the same thing?

Yes?

If right-clicking -> packatge info, and then you copy the shown command and run it on a cmd window, does it do the same thing?

no, since I can specify a bunch and even if I don’t specify the path, it will install where the folder path is by default

what do you mean when saying I can specify a bunch?

Ah, I see what you mean.
However, this is winget's fault, not wingetui.
Winget is (one of) the tools that wingetui uses to install software. And it is winget who is in charge to install software and generate icons/shortcuts.
I'm sorry, but this can't be fixed here.


Rebuild for gcc-7

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


Vignette

Package currently has no vignette. The package manual is currently the main resource for training. Addition of a package vignette with clear guidance on how to use the package would be good

Vignette now added https://cran.r-project.org/web/packages/GGIR/vignettes/GGIR.html
Textual improvement needed


Editor windows will not add new line

We have written the needed data into your clipboard because it was too large to send. Please paste.
Issue Type: Bug

Launch VS Code.
Open a script or code file.
Try to create a new line.
==> New line can not be created.
open a new file from the file menu.
Select a language from the drop down menu.
==> I selected Shell script
==> not only can I not create a new line; I can not type #!/bin/bash

Help!!!
VS Code version: Code 1.56.2 (Universal) (054a9295330880ed74ceaedda236253b4f39a335, 2021-05-12T17:44:30.902Z)
OS version: Darwin x64 20.4.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-4870HQ CPU @ 2.50GHz (8 x 2500)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
2, 2, 2

Memory (System)
16.00GB (4.31GB free)

Process Argv
--crash-reporter-id dd9896a9-49e2-4a58-b174-1bb0739bccfa

Screen Reader
no

VM
0%

Extensions (20)

Extension
Author (truncated)
Version

npm-intellisense
chr
1.3.1

cocos-debug
Chu
0.0.1

zsh-completion-script
dua
1.0.0

vscode-npm-script
eg2
0.3.21

vscode-firefox-debug
fir
2.9.4

code-runner
for
0.11.4

cloudcode
goo
1.11.1

cocotools
jas
0.0.7

file-downloader
min
1.0.11

vscode-docker
ms-
1.12.1

vscode-kubernetes-tools
ms-
1.3.3

remote-containers
ms-
0.177.2

cpptools
ms-
1.3.1

debugger-for-chrome
msj
4.12.12

vscode-janus-debug
otr
1.3.1

vscode-commons
red
0.0.6

vscode-yaml
red
0.19.2

zsh-debug
rog
0.1.3

vim
vsc
1.20.3

quokka-vscode
Wal
1.0.368

(1 theme extensions excluded)

A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383:30185418
pythonvspyt602:30300191
vspor879:30202332
vspor708:30202333
vspor363:30204092
pythonvspyt639:30300192
pythontb:30283811
pythonvspyt551:30291414
vspre833:30267464
pythonptprofiler:30281270
vshan820:30294714
pythondataviewer:30285071
vscus158:30286553
vscgsv2:30301612
vscorehov:30302759
bridgeflightcf:30302070
vscod805:30301674

Please tell me exactly what you're doing when you try to "create a new line"

Hello,
It is just like I said.
I open an existing file or create a new file.  Then I try to type something or just hit return on my Mac.  I get no new line or no text.
The only new thing I added was to cut off the Kubernets extension because I was getting errors.  I never used it anyway.  Also I added Oh My zsh as my default shell in iTerm.
VS Code is  is unusable.
I am not very experienced..  I’m using Code Runner to write scripts.
Ed

On May 24, 2021, at 12:40 PM, Rob Lourens @.***> wrote:
Please tell me exactly what you're doing when you try to "create a new line"

Hello,
I finally found out what was going on.
I installed Vin, a emulation editor on VS Code.  It was waiting for me to turn on insert mode.  I once tried to use the vi editor back in the 90’s.  It gave me problems then too.
Code editor is not editing.
Thanks,
Ed Hackett
PS  How do I add this to bug log?

On May 22, 2021, at 8:58 PM, vscodebot[bot] @.***> wrote:
(Experimental duplicate detection)
Thanks for submitting this issue. Please also check if it is already covered by an existing one, like:
Can't create new line in file. (#123758) https://www.github.com/microsoft/vscode/issues/123758
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/microsoft/vscode/issues/124442#issuecomment-846497625, or unsubscribe https://github.com/notifications/unsubscribe-auth/AS7JUZHUDTJGEJ4ZC5IACZ3TPB4NJANCNFSM45LI54UQ.

Ok, glad you figured it out, I don't understand what you are asking for now

Code editor is not editing.
PS  How do I add this to bug log?

Sorry,
This problem killed VS Code for me.  The problem was so bad I had to revert to another code editor.  I just thought this knowledge should be understood by others.  I was asking how to put the solution in with the bug report.  That way others would not fall into the same pit fall!
I am new to all of this Git stuff and I probably could use a tutorial.  For example, if a bug is closed, canons someone still add a solution?
Sincerely,
Ed

On Jun 4, 2021, at 11:05 AM, Rob Lourens @.***> wrote:
Ok, glad you figured it out, I don't understand what you are asking for now
Code editor is not editing.
PS How do I add this to bug log?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/microsoft/vscode/issues/124442#issuecomment-854910006, or unsubscribe https://github.com/notifications/unsubscribe-auth/AS7JUZDUNYEUMT4Y7LTUC3TTREIVLANCNFSM45LI54UQ.

This problem was solved by uninstalling the Vim extension!  Please close.

Solved by uninstalling Vim!