Create internationalization (i18n) translations

Create i18n feature for future translations.

How about creating a folder with some files in it. When the user "load" the code on the index.html file, he passes a parameter called 'language' (default EN), and the code just load the corresponding file.

For example like the following structure :
In 'index.html' :
[...]
.loadData({
  language : 'EN'
})
[...]

And the code just check the file in src/language/EN.json and load it into a global constant object like 'LANGUAGE'.
To get a text, we just need to do a LANGUAGE.get(_id) to get the corresponding string.

Check my modification test.

Check my modification test.

Its ok for me!
Create a PR!!

Done! Check #48.

wiki: https://github.com/myliang/x-spreadsheet/wiki/Internationalization


Has this to do with my config or is there a known bug ?

Hm, weird. Does the same thing happen if you log manually? Eg. type :echo 'TEST'

No, it stays


Bump go version

The current code base is using go v1.18. We should bump this to at least 1.20.

the entry in the go.mod file will still work with golang 1.21.x

but yes, we should bump it.


feat: adds encoding for TIS-620

[related to #1843]
Adds encoding for TIS-620 (Thai Industrial Standard 620-2533)
Documentation: wikipedia
According to this paragraph, this encoding should be equivalent to ISO-8859-11, except for hex A0:
A nearly identical version of TIS-620 has been adopted as ISO/IEC 8859-11 in 2001,
the sole difference being that ISO/IEC 8859-11 defines hex A0 as a non-breaking space,
while TIS-620 leaves it undefined but reserved. (In practice, this small distinction is usually ignored.)

~Should this just point to the ISO-8859-11 table, or is it worth it to have a new table defined, as done in this PR?~
Thanks!

Worth having its own table, no problem.

Nice work!


Make mapping global variable, use builtin ceil

I made METHOD_MAPPING global so that it is only created one time during loading time.
Also math.ceil is used to replace the ceil function you wrote. :)
block_iter  is a generator now.

I didn't notice this but it's an interesting found with timeit
>>> timeit.timeit('int(3 // 2 + bool(3 % 2))', number=100000)
0.026131337999970583
>>> timeit.timeit('ceil(3 / 2)', number=100000, setup="from math import ceil")
0.009041516000024785
>>> timeit.timeit('3 // 2', number=100000)
0.0008856160000050295
>>> timeit.timeit('floor(3 / 2)', number=100000, setup="from math import floor")
0.008675524000011592

Ok, but note this:
import math

floor = lambda a, b: a // b

x = 10
y = 3.4

print(type(math.floor(x / y))) # <class 'int'>
print(type(floor(x, y))) # <class 'float'>


Test Regex Scraped from crates.io

As part of evaluating my masters thesis, I scraped crates.io for regex and ran the resulting regex though my compiler to see how many could be optimized. I don't think it would be too hard to clean up my scraping script a bit and then write a test which executes each of the regex on a quickcheck generated string with all (3? 5? 6? depends how you count them) backends. I'm not sure when I'll do this, but I wanted to leave a note here so that I don't forget.

Hmm. Good point. One thing I remember reading in a note Russ Cox made about the testing approach for RE2 is that he wrote some code to construct a random matching string from a regex, so it would might be worthwhile to give that a crack. The two sources of random input would probably do a pretty good job of testing both the positive and negative cases.

Yup, that's another good avenue to try!

After a first pass at this just using quickcheck to generate random input, I've turned up the following failing test cases.
extern crate regex;

#[test]
fn word_boundary_backtracking_default_mismatch() {
    use regex::internal::ExecBuilder;

    let backtrack_re = ExecBuilder::new(r"\b")
        .bounded_backtracking()
        .build()
        .map(|exec| exec.into_regex())
        .map_err(|err| format!("{}", err))
        .unwrap();

    let default_re = ExecBuilder::new(r"\b")
        .build()
        .map(|exec| exec.into_regex())
        .map_err(|err| format!("{}", err))
        .unwrap();

    let input = "䅅\\u{a0}";

    let fi1 = backtrack_re.find_iter(input);
    let fi2 = default_re.find_iter(input);
    for (m1, m2) in fi1.zip(fi2) {
        assert_eq!(m1, m2);
    }
}

#[test]
fn uppercut_s_backtracking_bytes_default_bytes_mismatch() {
    use regex::internal::ExecBuilder;

    let backtrack_bytes_re = ExecBuilder::new("^S")
        .bounded_backtracking()
        .only_utf8(false)
        .build()
        .map(|exec| exec.into_byte_regex())
        .map_err(|err| format!("{}", err))
        .unwrap();

    let default_bytes_re = ExecBuilder::new("^S")
        .only_utf8(false)
        .build()
        .map(|exec| exec.into_byte_regex())
        .map_err(|err| format!("{}", err))
        .unwrap();

    let input = vec![83, 83];

    let s1 = backtrack_bytes_re.split(&input);
    let s2 = default_bytes_re.split(&input);
    for (chunk1, chunk2) in s1.zip(s2) {
        assert_eq!(chunk1, chunk2);
    }
}

#[test]
fn unicode_lit_star_backtracking_utf8bytes_default_utf8bytes_mismatch() {
    use regex::internal::ExecBuilder;

    let backtrack_bytes_re = ExecBuilder::new(r"^(?u:\*)")
        .bounded_backtracking()
        .bytes(true)
        .build()
        .map(|exec| exec.into_regex())
        .map_err(|err| format!("{}", err))
        .unwrap();

    let default_bytes_re = ExecBuilder::new(r"^(?u:\*)")
        .bytes(true)
        .build()
        .map(|exec| exec.into_regex())
        .map_err(|err| format!("{}", err))
        .unwrap();

    let input = "**";

    let s1 = backtrack_bytes_re.split(input);
    let s2 = default_bytes_re.split(input);
    for (chunk1, chunk2) in s1.zip(s2) {
        assert_eq!(chunk1, chunk2);
    }
}

The last two look like the are probably dups.
My work on this currently lives here, and I think it is basically ready for a PR except for a few config issues.

Right now to run the checks I just added a new test binary and marked the entrypoint with #[test]. This is bad for a few different reasons.

The tests take a long time to run, and I'm not sure they should be in regular CI. If it is possible to have two different test profiles (one for regular CI and a more complete one for releases and major new features), that would be great.
There is a lot of work going on for one #[test], and we really should be seeing output to the screen to be able to monitor progress (--nocapture works for this when you are focusing on the test, but it would not fit into a bigger run of the test suite). Ideally when you do cargo test it would just invoke a binary without capturing the output and then report that the suite failed if the exit code is non-zero. I think I once saw that there was a way to do this on a stackoverflow post, but my google-fu has failed me.

The specific cases that I just pulled out should definitly be part of the regular test suite, but they are not dependant on the definition of the regex! or regex_set! test macro, so they should not be run for every different test config. I'm not sure where the best place to put them for that is. I've just stashed them in the test_crates_regex test that I made for now, but if we turn that off for CI, it is the wrong place to put them.

It may be worth looking at https://crates.io/crates/regex_generate when I get around to generating matching strings from a regex.

I'm going to say that this is closed by the work done a while back in #472. If there's something else we should, please feel free to file a new issue!


update palemoon to 26.3.2

Travis is saying undefined method chomp' for nil:NilClass (NoMethodError)`. What does that mean?

Test failure seems irrelevant, and I'll look into it. For now, this look alright.


Determine version from otelcol-sumo, pull artifact from workflows

WIP.
Signed-off-by: Justin Kolberg<EMAIL_ADDRESS>

This will be included with #123.


How to Change Theme Default Colors

Hello, What's up?
While using Airframe in a project I needed to modify one of the default theme colors, I need to replace Primary with a darker blue.
Could you help me on how to do such a thing?

@0wczar  I second this. We need a more detailed documentation  on how to utilize and use this product. If you are serious about your open source product, please support it and produce better documentation.
It does not have to be much, just a short readme on how to override color schemas, use custom fonts and where to put logos etc. I am sure you want this product to serve as your beacon of light in the marketing darkness on Internet. It is a good product, but it needs some extra love to really take off. No need to focus on new features, supporting more frameworks etc. Make some good documentation and this will come automatically.

@christerjohansson
Sorry for the late reply.
The situation got complicated.

I need to earn money to support my family.
Airframe doesn't earn, so I have a hard time keeping it up.
I am currently looking after my family, my wife works in a hospital (Covid)

If you have ideas for help with "Airframe" I will be grateful!

2. Airframe doesn't earn, so I have a hard time keeping it up.

Sorry to hear that! I have seen Airframe being on sale over at WrapBootstrap.com,  I will buy it to support it. I am merely looking for a quick and dirty text on how to change things. It does not have to be fancy, just a text file explaining roughly how to do it. If I manage to fix it, I'll help out making a documentation for it. I really like this one, you've done a great job with it.

@christerjohansson
Sorry for the late reply.
The situation got complicated.

I need to earn money to support my family.
Airframe doesn't earn, so I have a hard time keeping it up.
I am currently looking after my family, my wife works in a hospital (Covid)

If you have ideas for help with "Airframe" I will be grateful!

Sorry to hear that! I have seen Airframe being on sale over at WrapBootstrap.com, I will buy it to support it. I am merely looking for a quick and dirty text on how to change things. It does not have to be fancy, just a text file explaining roughly how to do it. If I manage to fix it, I'll help out making a documentation for it. I really like this one, you've done a great job with it.

@christerjohansson Did you come up with something decent? Currently trying to do the same thing

@christerjohansson Did you come up with something decent? Currently trying to do the same thing

No, I did not... Airframe is a "beta" product, not quite ready to use yet. You can change stuff in it, and it do have theming...but it will require more dev-time to reach its full potential. I really hope the author gets some time do finish it, and make a proper documentation for it. I ended up purchasing one of his other products instead.

Has anyone succeeded with changing $primary color?


Set up

In this milestone, the following changes were implemented;

Created a new Rails app called 'hello-rails-react'.
Use the command rails new -j webpack my_new_rails_app as described in the previous lesson's tutorial.
Verify that the jsbundling-rails gem is added to your Gemfile.
Initialized your project with Git.
Install needed node dependencies as described in the previous lesson's tutorial.
Created a babel.config.js in the root folder of your project as described in the previous lesson's tutorial.
Updated webpack.config.js to include the babel loader as described in the previous lesson's tutorial.
Made sure that your project has a Postgres database set up.
Created a table for storing your messages and create 5 different greetings.
Created an API endpoint that selects a random greeting from your table (you will need a controller with an action and Rails route).
Created a static view that will be the root of your app.
Created the App component with react-router and render it in your static view.
Created the Greeting component that will display a greeting. Set it up as a route in your App component.
Created a store, an action, and a reducer that will connect to your API endpoint to get the random greeting.
Displayed the random greeting in your Greeting component.

Hi @Daphineatim 😃,
As much as I appreciate your hard work on this project, I'm afraid that I can't perform a review on this pull request.
Invalid Submission

Please fix the Linter checks in the PR

Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready. 🙏
Cheers and Happy coding!👏👏👏
Feel free to mention @MahmoudBakr23 to leave any questions or comments in the PR thread if something is not 100% clear.

Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

Invalid Code Review Request does not count into the code reviews limit.


Date and Time Parsing

Currently, parse_date in the util class gets called every event in order to check if the date and time is in range of start_date and end_date found in the config file. Need a way to optimize this or get rid of it completely.

It means to generate current date and time in YYYY-MM-DD format and return it. Or take a date in any format and extract date and return into the required format? Can you clarify me, so that i can contribute under hacktoberfest 2023?


Hyprbars: Activating windows uses different colored bars

Hyprbars: Activating windows uses different colored bars
or hightlight?

Dup of https://github.com/hyprwm/hyprland-plugins/issues/72


Add an entry point generator plugin and documentation on how to use it

The plugin, when used with a target - will generate an entry point, registering all classes with @Godot macro attached.

Lovely!


PA2 Level 5

Summary
My maxDepth is calculated wrongly and I'm not sure how to fix this.
Description
My FuncStat has 2 methods, one to increase count and another to increase maxDepth, each by 1. But it seems like maxDepth is increased every time ack is called instead of reflecting the maximum depth of recursion. Does anyone know how to fix this?
Idea
Screenshots (if any):

depth should be incremented before a recursive call and decremented after a recursive call. maxDepth will be the max value that depth has at any point

ahh okay i will try that. thanks!

I got the same issue because of flatMap carrying over the FuncStat from the recursive call, making the depth adding together at the end. You might need to remove this effect by doing accept on recursive call.

for my solution, I didn't have any accepts in level5. I did have a lot of flatMaps though, and two rounds of get/put to modify depth for the cases with recursive calls

Hello did you find out whats your issue?

Thank you for the suggestions! I am still stuck at level 5 but now my maxDepth seems to be lesser than the correct maxDepth by 1. For the 2 recursive cases, i increase current depth by 1 before recursive call and update max depth, then after that i decrease current depth by 1. For the first case i don't increase max depth but i update it.

the first case can possibly increment maxDepth if depth == maxDepth since you have to take into account that function call too


pre-commit update

!verified

!verified

!cherry-pick v4.11

!cherry-pick v4.11

!cherry-pick v4.10


⚠ HawkeUno has degraded performance

In 53ebe28, HawkeUno ($UNO) experienced degraded performance:

HTTP code: 403
Response time: 1840 ms

Resolved: HawkeUno performance has improved in 1834324 after 10 minutes.


Build image for CRD

We need to create image building scripts for the kubebench CRD and add that to e2e so the image can be published.

@xyhuang Assigned to me.


Chain From Mod

Following PSQL documentation: https://www.postgresql.org/docs/current/queries-table-expressions.html#QUERIES-FROM
The FROM clause derives a table from one or more other tables given in a comma-separated table reference list.
FROM table_reference [, table_reference [, ...]]

There should be possibility to use more than one from mod.
But for now each next overwrite previous one.
As on the example:
func main() {
	qs, _ := psql.Select(
		sm.Columns("id", "name"),
		sm.From("users"),
		sm.From("clients"),
		sm.Where(psql.Quote("id").EQ(psql.Arg(100))),
	).MustBuild()

	fmt.Println(qs)
	// Out: SELECT id, name FROM clients WHERE ("id" = $1)
	// Expected: SELECT id, name FROM users, clients WHERE ("id" = $1)
}

Also there should be a possibility to chain from table and from query.
As on the example:
func main() {
	qs, _ := psql.Select(
		sm.Columns("id", "name", "type"),
		sm.From("users"),
		sm.From(psql.Select(
			sm.Columns("id", "type"),
			sm.From("clients"),
			sm.Where(psql.Quote("client_id").EQ(psql.Arg("123"))),
		)).As("clients"),
		sm.Where(psql.Quote("id").EQ(psql.Arg(100))),
	).MustBuild()

	fmt.Println(qs)
	// Out:
	// SELECT id, name, type
	// FROM (
	//   SELECT id, type
	//   FROM clients
	//   WHERE ("client_id" = $1)
	// ) AS "clients"
	// WHERE ("id" = $2)

	// Expected:
	// SELECT id, name, type
	// FROM users, (
	//   SELECT id, type
	//   FROM clients
	//   WHERE ("client_id" = $1)
	// ) AS "clients"
	// WHERE ("id" = $2)
}

The documentation says:

If more than one table reference is listed in the FROM clause, the tables are cross-joined

Since there is a different (arguably clearer) way to achieve the same result, I chose to keep things simple. You can do this instead:
	psql.Select(
		sm.Columns("id", "name", "type"),
		sm.From("users").As("u"),
		sm.CrossJoin(psql.Select(
			sm.Columns("id", "type"),
			sm.From("clients"),
			sm.Where(psql.Quote("client_id").EQ(psql.Arg("123"))),
		)),
		sm.Where(psql.Quote("id").EQ(psql.Arg(100))),
	)

Doing this, I realised that I left out the ability to add aliases to CROSS JOIN, and I'll add that soon.


Implement test(s) against snfit for SALT2 model fits

Test fit results and generated light curve results.
For testing generated light curves, the program snlc in the snfit suite can be used to generate integrated light curves. E.g.,
snlc lc2fit_f814w.dat lc2fit_f105w.dat -p result_salt2.dat

where presumably result_salt2.dat is a result file containing parameters from fitting.

Done to some extent in db097847673001db4483c161eab11aab2357ed89 .


Feature/chatbot

Pull request for the chatbot.py please and thank you.
Is someone please to test it and then approve my pull request thank you :)

Chatbot works for me, I have python 3.8.5 installed. Instructions were helpful for setting it up. Spellcheck worked decently too.


GL_PIXEL_UNPACK_BUFFER causes RenderDoc's overlay UI to be a black box

Description
If the application does not unbind GL_PIXEL_UNPACK_BUFFER, RenderDoc draws the UI (including API information and FPS counter) with a solid black box.
Recording a capture via Queue Capture works just fine, but recording a capture at runtime using the F12 or PrintScreen key results in corrupted output (which might be due to the application, but at least the UI should be drawn correctly).
Steps to reproduce
I attached a ZIP file including the RenderDoc trace as well as the Windows executable and GLSL shader:
PixelUnpackBuffer.zip
Environment

RenderDoc version: 1.5 (built from a94f238e)
Operating System: Windows 10
API: OpenGL

Thank you for the clear repro, indeed there were some cases causing both problems that were not properly temporarily removing the pixel unpack buffer. Pixel pack/unpack is a really nasty gotcha in the API :(.

Totally agree, also the entire pixel store configuration cost me a lot of time when I started learning OpenGL a decade ago.


is this still even working? - SyntaxError: invalid syntax

Describe the bug
this is the error I get when I try to use DM Spam or any option for that matter...
Select a bot: 4
Traceback (most recent call last):
  File "bots/DM/discord_text_spam_dm.py", line 12, in <module>
    import discord
  File "/usr/local/lib/python3.7/site-packages/discord/__init__.py", line 20, in <module>
    from .client import Client, AppInfo, ChannelPermissions
  File "/usr/local/lib/python3.7/site-packages/discord/client.py", line 38, in <module>
    from .state import ConnectionState
  File "/usr/local/lib/python3.7/site-packages/discord/state.py", line 36, in <module>
    from . import utils, compat
  File "/usr/local/lib/python3.7/site-packages/discord/compat.py", line 32
    create_task = asyncio.async
                              ^
SyntaxError: invalid syntax
Traceback (most recent call last):
  File "bots/DM/discord_text_spam_dm.py", line 12, in <module>
    import discord
  File "/usr/local/lib/python3.7/site-packages/discord/__init__.py", line 20, in <module>
    from .client import Client, AppInfo, ChannelPermissions
  File "/usr/local/lib/python3.7/site-packages/discord/client.py", line 38, in <module>
    from .state import ConnectionState
  File "/usr/local/lib/python3.7/site-packages/discord/state.py", line 36, in <module>
    from . import utils, compat
  File "/usr/local/lib/python3.7/site-packages/discord/compat.py", line 32
    create_task = asyncio.async
                              ^
SyntaxError: invalid syntax

Screenshots

Desktop (please complete the following information):

OS: macOS 10.15.4
Python Version 3.7.7

Use 3.6


setVertexArrayObject: set vao to

Not an issue per say but I would like to know the meaning behind this message. I am getting a TON of these messages on my console which makes it extremely hard to debug because I have to scroll through all the 'setVertexArrayObject' messages. For context, I am using a package which uses a webview.

Duplicate of #48079
Take a look at this issue

Thanks @Dasunx. Good to know I'm not the only one having this issue, however; like the most recent commenter, I don't want to downgrade (currently using stable 1.20.4). Also no mention of what this error message means

@kaarens93
Please provide flutter doctor -v and a minimal code sample that has plugin code you are using.
Thanks.

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact description of what happens, logs, and the output of flutter doctor -v.
All system setups can be slightly different, so it's always better to open new issues and reference related issues.


website

please how i can host it as website , i try but it give me error 405

@lahcenad Do you want to host the /docs path?


Feature/backend 1879

Currently, when error occurs corgi format error without stackTrace since it can leak internal structure of system.
in order to debug error easily,

if you set CORGI_ERROR_PASSWORD
error response would have "metadata" which is encrypted stackTrace + error message
you can decrypt this with same key

HMAC 까지는 안하고 IV 키만 추가해서 끝냈습니다.


YTC developer build no longer works with FF 201520110 nightly.

Doesn't even kick in anymore here.
Anyone can reproduce ?
YTC Dev build 461
FF nightly 2015-01-10

EDIT : Please disregard.
It's the abandoned Greasymonkey that is no longer compatible.
No GM scripts still work.


WebSSH in unattended and possible locked iphone terminates session

Describe the bug
Before I could run WebSSH, connect SSH to server, start upgrading, go get a coffee, return to phone, unlock the phone, see WebSSH still having connection in place with server and possibly even see the upgrade finished. This is not so no more. When I get back to the locked phone I see the connection is terminated after unlocking.
Informations

Device : iPhone CS
OS : iOS 15.0.1
WebSSH Pro version : 16.0

Screenshots
To Reproduce
Steps to reproduce the behavior:

Go to WebSSH
Establish connection to a server of choice
Dep on server OS, start upgrade. Eg sudo apt dist-install for debian or sudo yum update for centos.
Fetch coffee and talk to other persons about irrelevant things for 5 minutes
Return to iPhone and unlock it
See a disappointing terminated session with a choice of closing or reestablishing it

Expected behavior
Steps 1-5 above
6. See a living connection and possibly even a finished upgrade of the server.
7. Continue with a satisfying and rewarding life. Now with my steaming hot coffee in my hand.

Hi. Tried your advice, but "no cigar". I think you are right on the iOS 15 battery saving implementation being the origin of the problem. Nothing to do. Thx for your time and effort.

Sorry for spamming you, but I started Spotify and made sure to lock iPhone as always. As music was playing still I realized at that moment that Spotify must circumvent this power saving enforcement somehow. Perhaps they pay extra to do that or there must be a way. Just a thought.


How to set full screen splash for android

Hi,
on android i can't set full screen splash. How to set full screen?
Thanks,

Not sure I understand? What do you mean by full-screen?
Do you mean making the StatusBar hidden? See #11

yes, i wan't to hidden StatusBar


Fix MQTT reconnecting

The MQTT reconnect logic was flawed and did not work. I played around with it a bit and this should fix it. Included some tests to verify it works.
Core issue was that the interactions were happening in the following order:

connect
subscribe
on_connect
on_subscribe

But since we resetted the progress in on_connect, we never recorded the successful subscription in on_subscribe.
Changes:

Move callbacks into the MQTT class.
on_connect callback no longer resets topics in progress
on_connect will use the HA MQTT object to re-subscribe, making sure topics get added to progress
on_disconnect will reset progress dict and remove pending topics from topics hash

Fixes #873.

Testing with Ponte and Mosquitto now. I'll report back in a couple of days.


Error Occurred in Sim Agents Challenge Submission

Hi, I recently uploaded a test set submission to the sim agents challenge, but I got an unclear error shown in the following image:

There is no feedback but only a "publish to leaderboard" button. The name of the web page is "Error - Submission". Could you please have a check on what the problem is?
The submission account is<EMAIL_ADDRESS>The submission link is https://waymo.com/open/challenges/sim-agents/results/51cc2220-7b19/1727014742699000/.
This is the second submission on 9/22/2024 2:32 PM (the first one failed due to invalid naming).
I would appreciate your help!

Resolved. Thanks!


🎨 Style: Introduce Animation Effects to Components on home page

Description:
Certain components, such as headings in the main section and the dynamic profile section on the homepage, currently lack animation effects, resulting in a static appearance. To enhance the user experience and visual appeal, I propose incorporating animation effects using the Framer-Motion library.
Problem:
The static appearance of these components may lead to a less engaging user experience, and the absence of animations could hinder the dynamism of the page.
Proposed Changes:
Integrate animation effects for specific components in the main section and the dynamic profile section on the homepage using the Framer Motion library. This will introduce a layer of dynamism, creating a more visually appealing and engaging interface.
To-Do:

[ ] Add the Framer Motion library to the project dependencies.
[ ] Identify components in the main section and dynamic profile section for animation integration.
[ ] Implement animation effects that enhance the visual appeal and engagement.
[ ] Test animations across devices and browsers to ensure a consistent experience.

Checklist

[X] I have starred the repository
[X] I have checked the existing issues
[X] I am willing to work on this issue
[X] I am a SWOC'24 contributor

Hey @harmeetsingh11, assigned!
You can first show some demo from docs directly and then make changes maybe


[Suggestion] Reset RC, Approval and Verification counts to 0 after new version upload

It is kind of misleading to have a submission still have requested changes on it after a new version has been uploaded. Resetting the counts would mean that the approval process starts fresh when a new version is uploaded.

it does start fresh, but requested changes persist until the testes rechecks it, that's intentional

Gotcha, closing in that case


#1802 sap.mTokenizer + sap.m.MultiInput performance when pasting big …

Fixing issue  https://github.com/SAP/openui5/issues/1802

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Petr Plenkov seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

I don't know how you do this - but this CLA check doesn't work for me.
I tried both from Gitkraken and VScode. I do push with my Github user but it says I'm not a Github user.

@ThePlenkov you could try to fix your email in the local git config and then run
git commit --amend --no-edit --reset-author
git push --force

to update the commit with the new mail and update this PR, so you don't need to create a new one.

Thank you, I did and it helped! This is a new PR: https://github.com/SAP/openui5/pull/1810


LB: zero-downtime service upgrade fails

Rancher Version:
v1.0.1
Docker Version:
v1.11.1
OS and where are the hosts located? (cloud, bare metal, etc):
Ubuntu 14.04, DigitalOcean (Docker droplet)
Setup Details: (single node rancher vs. HA rancher, internal DB vs. external DB)
Single node rancher, internal DB
Environment Type: (Cattle/Kubernetes/Swarm/Mesos)
Cattle
Steps to Reproduce:

Create 2 hosts on DigitalOcean
"Add Service" to Default stack: nginx docker image, running on all hosts, default options (don't expose ports)
"Add Load Balancer" to Default stack. source port = 80, default port = 80. target service = nginx
run ab -lr -c 10 -n 10000 http://[ip_of_loadbalancer]
upgrade nginx service

Results:
ab tests failures > 0
Expected:
ab tests failures = 0

@naartjie a couple of things to confirm:

when doing upgrade, have you checked "Start before stopping" check box? By default we stop the old container, and only then create a new one. In order for original scale not to drop, you might want to select this check box, so the old container doesn't go down until the new one comes up.
do you have health check configured for nginx service? Presence of health check would ensure that haproxy don't send a request to the service till its fully ready.

@alena1108 thanks for looking into this

Start before stopping
It is checked, sorry, I should have mentioned that

Health check
No health checks, could you let me know how to configure the nginx health check, and I'll re-try with it on

@naartjie in the UI, when add nginx service: go to "Health Check" tab, pick up "Http" option there and there you can configure health check.

@alena1108 I configured an HTTP health check: path = / and port = 80.
I am still getting some receive failures, and sometimes Non-2xx responses when running ab:

Are you not seeing the same on your side? Can you not reproduce this? I am just wondering what the best way might be for me to share this. I don't know how to automate this test (any hints?), but I could record a screencast of the steps I take.

@naartjie was the number of failed requests reduced after adding health check?
Unfortunately there still a chance for small amount of dropped requests (the reasons are covered here: https://github.com/rancher/rancher/issues/2777). Using health checks and "start before stopping" on upgrade should reduce it to the minimum.

was the number of failed requests reduced after adding health check?

@alena1108 yes, there are definitely less errors after adding the health checks.
I have subscribed to #2777 and am watching it for updates. The description makes sense. Thanks for the info.

Closing this in favor of #2777


refactored shredding of json payload

What this PR does:
Provides simpler and more efficient document shredding based on the recursion. Operate only against the JsonNode and returns results in a minimal concrete class representation of shredding. The idea is that we add later on JsonShreddedRow#bind(Query inserQuery) and easily bind values based on the object state.
Open questions:

What should be the leaf & path values in case of empty objects and empty arrays?
Do we support primitives and what should be the leaf & path values if we do?
Should we escape also the prepend path? (seems this is not done at the moment)
Is the ErrorCode.DOCS_API_GENERAL_INVALID_FIELD_NAME still valid and when it should be thrown?

Which issue(s) this PR fixes:
None.
Checklist

[x] Changes manually tested
[x] Automated Tests added/updated

Does this change the way that data is actually stored in the database? Or would this just improve how we process data?

Processing only, currently we had the processing in DocsShredder.

@EricBorczuk can you have another look and answer to the new questions.. I want to get this straight.. Thanks!


Reorganizing

put the core functions in separate files. This should be easier to maintain.
Also added test coverage badge to README, this fixes #7

Codecov Report

Merging #9 into master will not change coverage.
The diff coverage is 0.31%.

@@          Coverage Diff           @@
##           master      #9   +/-   ##
======================================
  Coverage    0.13%   0.13%
======================================
  Files           3       7    +4
  Lines        1535    1535
======================================
  Hits            2       2
  Misses       1533    1533

Impacted Files
Coverage Δ

R/PCdfm.R
0% <0%> (ø)

R/BDFM.R
0% <0%> (ø)

R/MLdfm.R
0% <0%> (ø)

R/dfm.R
0% <0%> (ø)

R/utils.R
5.26% <5.26%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e6f7536...f77550d. Read the comment docs.

Also added a pkgdown website draft. Not sure if it works out of the box. If so, we should see it at:
https://wonderful-banach-66adf3.netlify.com/
If not, you need to point your netlify 'Publish directory' to the docs directory, where the website is located. See here:

I think the website will work. I can already see the preview:
https://deploy-preview-9--wonderful-banach-66adf3.netlify.com/docs/

OK... going to merge this before I start on anything today to avoid conflicts!


⚠ Bofrost.be has degraded performance

In 91eb8d1, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1832 ms

Resolved: Bofrost.be performance has improved in 392c3bf.


Technical advisory for race condition between long running reads and …

…replica removal.

Reviewable status: :shipit: complete! 0 of 0 LGTMs obtained (waiting on @lunevalex, @nickelnickel, and @shermanCRL)

advisories/a64325.md, line 29 at r3 (raw file):

This is resolved in CockroachDB by PR [#64324], which fix the race condition between read requests and replica removal.

The fix has been applied to maintenance releases of CockroachDB v20.1, v20.2, and v21.1.

Have we considered mentioning the exact release? I feel like people would ask "what do I need to upgrade to"?

It's mentioned in the mitigation section


New-WinGetSource cmdlet doesn't exist

Brief description of your issue
The cmdlet described thoroughly here: https://github.com/microsoft/winget-cli-restsource/blob/main/Tools/PowershellModule/doc/PowerShell/New-WinGetSource.md, doesn't seem to exist.

The "New-WinGetSource" is part of a release artifact of this project. There were issues raised with the scripts functionality that we are working to resolve.


Entity Framework Migrations issue

A bit of a strange one, but it seems that when you call:
app.UseDotNetify(config =>
            {
                // Doesn't matter what you config in here
                ...
            });

inside the Startup.cs Configure() method, it runs fine when you dotnet run it, but as soon as you call dotnet ef migrations add InitialDb then EF naturally calls the static IWebHost BuildWebHost() method and errors out on the above call with the following stack trace:
Application startup exception: System.Exception: ERROR: Assembly 'ef' does not define any view model!
   at DotNetify.VMController.RegisterAssembly[T](Assembly vmAssembly)
   at DotNetify.AppBuilderExtensions.UseDotNetify(IApplicationBuilder appBuilder, Action`1 config)
   at Startup.Configure(IApplicationBuilder app, IHostingEnvironment env, IConfiguration configuration) in C:\Dev\Projects\server\Startup.cs:line 50
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at Microsoft.AspNetCore.Hosting.ConventionBasedStartup.Configure(IApplicationBuilder app)
   at Microsoft.AspNetCore.Hosting.Internal.WebHost.BuildApplication()
crit: Microsoft.AspNetCore.Hosting.Internal.WebHost[6]
      Application startup exception
System.Exception: ERROR: Assembly 'ef' does not define any view model!
   at DotNetify.VMController.RegisterAssembly[T](Assembly vmAssembly)
   at DotNetify.AppBuilderExtensions.UseDotNetify(IApplicationBuilder appBuilder, Action`1 config)
   at Startup.Configure(IApplicationBuilder app, IHostingEnvironment env, IConfiguration configuration) in C:\Dev\Projects\server\Startup.cs:line 50
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at Microsoft.AspNetCore.Hosting.ConventionBasedStartup.Configure(IApplicationBuilder app)
   at Microsoft.AspNetCore.Hosting.Internal.WebHost.BuildApplication()
An error occurred while calling method 'BuildWebHost' on class 'Program'. Continuing without the application service provider. Error: ERROR: Assembly 'ef' does not define any view model!

The only 2 ways to get around this are to

comment out the app.UseDotNetify(config => part and run your migrations command and then uncomment it each time you want to add or run a migration
Implement an instance of IDesignTimeDbContextFactory<TContext> which basically repeats the same code as you would have in ConfigureServices() for adding SQL Server config - and if you do this, you still see the stack trace in the migration above, but it just keeps going and creates the migration anyway.

I'm guessing that DotNetify is doing some sort of assembly scanning on startup and finding an assembly called 'ef' which for some reason it expects to have a view model?
This issue is not stopping progress, but though some others might like to know the workaround. It would probably be even better if it didn't throw an exception at this stage because of the inner workings of DotNetify, but maybe I just missed something on the config?

The app.useDotNetify automatically assumes to look for the view models in the entry assembly if none is explicitly registered.  Apparently for 'dotnet migration' commands, 'ef' is the entry assembly, so in this case, the view model assembly must be explicitly registered:
app.useDotNetify(config => {
config.RegisterAssembly(...)
});

Thanks for the info! I'm assuming that there will be quite a lot of folks using EF with this template and DotNetify for that matter, so maybe this is something worth documenting as a known issue rather than fixing it?
I only mention it because it took me quite a while before I started looking in the right place - at first I was looking for the issue within EF itself which is quite a deep rabbit hole :-D. It was only the fact that the error message said something about the view model that led me to believe it must have something to do with DotNetify autodiscovery.
So I guess anyone using this in conjunction with EF will need to use either of the two workarounds I put in my first comment and ignore the stack trace on running migrations, or they will need to explicitly register all assemblies that contain view models then?

This can be properly fixed by deferring the exception-throwing check until the first client-side request is received.  I will put this in the next version release.  Thanks!

i had a view model derived from BaseVM which is working fine (without adding any additional RegisterAssembly), Now when i added another viewmodel derived from BaseVM, am getting below error -
"ERROR: 'UserVm' is not a known view model! Its assembly must be registered through VMController.RegisterAssembly."
I tried with registering assembly using both by config and RegisterAssembly still same error.
NOTE - was using 2.3.0-pre version, even after upgrading to 3.0.0-pre (for both nuget and client library).
app.UseDotNetify(config =>
{
config.RegisterAssembly(GetType().Assembly);
// Middleware to do authenticate token in incoming request headers.
config.UseJwtBearerAuthentication(new TokenValidationParameters
{
IssuerSigningKey = new SymmetricSecurityKey(Encoding.ASCII.GetBytes(AuthServer.SecretKey)),
ValidateIssuerSigningKey = true,
ValidateAudience = false,
ValidateIssuer = false,
ValidateLifetime = true,
ClockSkew = TimeSpan.FromSeconds(0)
});
        // Filter to check whether user has permission to access view models with [Authorize] attribute.
        config.UseFilter<AuthorizeFilter>();
     });

    VMController.RegisterAssembly(GetType().Assembly);

Please clarify.


Feature Update

[x] 1 Sever UPS Display
Slightly adjust the display a bit lower so it don't block the clock
Additional game speed percentage display

[X] 2 Deconstruction Planner Chat Display
Show a message to notify other.

[ ] 3A Player Data Ratio
Add ratio in the display

[ ] 3B Player Data Lookup
Allow other player data lookup in-game

[X] 5A Report Jail
Now it looks up if an active moderator is in-game.
Otherwise, then uses logic to check player data once.

[X] 5B Protection Jail
Now it looks up if an active moderator is in-game.

[x] 7 Spawn Area
Optional edit to allow customized generation.
This slightly boost the starting patch so it is faster to start.

[ ] 8 Vlayer
Optional virtual layer module to allow sending solar panel and battery inside.
This could allow saving a lot of space.

[ ] 9 Personal Logistic Slot
Optional setting for personal logistic slot. only for member and above.

[ ] 10 Auto Miner Deconstruction
Optional, an attempt for automatic miner deconstruction

Does point 3B cover #226 ? I feel like it does so once this is merged that issue can be closed.
Also point 8 sounds really cool to have, a balance suggestion for it is to have the "area" limited but with the option of expanding it using landfill. Another is to launch it into space but that feels far too costly.

most stuff is done now, basically need some more testing from others

fixed some issue on vlayer and research

In-game Testing: Addons
There were no issues found in the addons other than those listed below, please make changes to resolve these issues and only these issues.
Advanced Start

Config property is named "Grid" when it does not insert any items into the armour grid. This should either be renamed to "Items" or have the feature adjusted to automatically insert the items into the armour grid.

Decon Log

When a message is printed to game chat it states that the player was not allowed to decon. This is false because it will print even when tree and rocks are the only things being removed (which guests are allowed to remove). This message will become unhelpful because of how generic the trigger is, further more new players will not be aware that they cant use the decon planner and you will get lots of false positives. I advise you add a new condition which only prints to chat if there are more than a certain number of machines in an area.
The log for shooting rockets/nukes will trigger when when the ammo is refilled or swapped to another type. However this is not related to this PR so will be raised as a separate issue and does not need a fix within this PR.

Report Jail

Please add a comment to say how long "108000" is, or rewrite it in another form such as 60*60*30 to show that it is 30 minutes.

Spawn Area

By setting "spawn" as a friend of "player" it allows people to use inserters to remove the ammo from the turrets, this opens up the ability to get infinite free uranium ammo. There is no way to prevent this other than having it on a different force which is why the "spawn" force was created. Please revert these changes so that the player force and spawn force are not friends and all chests on the neutral force to allow player interaction.
Can you make it so the default resource spawn areas are in the four diagonal directions rather than in a straight line on the left side. Uranium can be to the north and oil to the south.
The resource refill does not refill the created patches, instead it refills based on proximity to 0,0 which is not expected behaviour. Please update this to refill the generated patches.
You have the resource refill function inside the creation event rather than the on tick event, this means that it is never getting called an no resources are being refilled at all. Please move this to the on_nth_tick event similar to turret refill. Don't put it into the same event as the refill times should not necessarily be the same.

Station Auto Name

I placed a station at spawn close to iron and copper and got this name [L] %[img=it-21.5m.-21.5opp-21.5r-or-21.5] something is clearly not right. This appears to be a result of lua pattern matching using %x for all hexadecimal characters. Because of this we should account using % in the config and instead switch to __icon__ __item_name__ __pos_x__ etc
X and Y values should be based on the entity position not the bounding box, also they should be floored to avoid decimals.

In-game Testing: Commands
Firstly, the following issues are fixed: Advanced Start and Report Jail. The rest remain open as per my comments above.
Here are the issues found with the commands:
Bot Queue

The limit of 3 is quite low for our server, this can be increased to 20
When setting the values you have 1 and 3 reversed, success should be 3*amount and failure should be 1*amount
The first argument to a command is player please use this and not game.player

Enemy

Invalid permission to use /kill-biters and /remove-biters

Friendly Fire

As with the other commands, please allow the user to specify a force to change the friendly fire state of. With the default being the players force.

Lawnmower

This command deletes your character, this can not happen. Worse still it deletes all entities. Remove the invert from the search.

Pollution

Please allow the user to select a surface for pollution clear, with the default being their current surface.
Using /pol-off does not remove pollution from the surface. Please make this command remove pollution from all surfaces.
Please give the commands full names /pollution-clear and /pollution-off

Auto Research

event.by_script in research_notification should only effect game.print the rest of the logic should still be triggered to avoid unexpected behaviour.
The logic used for the inventory bonus makes no sense. At Level 4, mining_drill_productivity_bonus = 0.4 which equates the following:

(event.research.force.mining_drill_productivity_bonus * 100) <= (config.bonus_inventory.limit * 10 / config.bonus_inventory.rate) to 40 <= 20*10/5
math.floor(event.research.force.mining_drill_productivity_bonus * 100) * config.bonus_inventory.rate + 10 to 40 * 5 + 10 = 210

This more than triples the inventory size, and what does the / config.bonus_inventory.rate achieve in the condition?
Please explain what the indented logic is here so I can suggest a sensible implementation.

Speed

Invalid role permission for the command.

Waterfill

Unknown locale key vlayer.waterfill-cliff and all other locale keys for this command. This should not be vlayer.
The command does not remove any cliff explosives from your inventory when it should.
Testing if the player position is the same as the entity is useless because of floating point comparisons.
Only one chest gets removed, and it doesnt even place down any water. This is because you are attempting to access the entity (steel chest) position after you have destroyed it.
Why are you even using steel chests? It would be better to check if the selected area is empty of all entities, and if it is then replace all tiles with water.

In-game Testing: Control, Data & GUIs
This covers the remaining sections of the code in this PR.
Miner

This is an addon not a control module, please move it to the correct folder
The current code does not have a 100% success rate, some depleted miners do not get marked for deconstruction.
This is because of the use of return during the for loop over entities, the issue is fixed if the loop body is moved to its own function

Vlayer

The GUI raises an error for players who can not edit the vlayer interface. This is because you are trying to use custom names like button_1 rather than using button_power.name
The circuit input does nothing, why does it exist? Please remove it or justify it.
The UI could use some work, but I am happy for this to be done at a later date.
The UI buttons never get disabled when above the interface limit.
The UI button to remove an interface does not work at all.

Bonus

Mods should be command/bonus not command/bonus/2

Personal Logistics

Invalid role permissions, cant use command
Using 0 as an argument should clear the automatic settings, not set them all to 0.
I feel we need tiers to these requests, because why would you want all three kinds of belt and all three types of assembler?
If you dont want to implement tiers then you should configure this for late game, so you request different amounts of each tier with a preference to higher tier; also things like stone furnaces and assembling machine 1s should always be trashed.

Research

When /cheat all is used it breaks the Gui because res_i[event.research.name] is nil on line 154. This error is also caused by using /research-all
Completing any research that is not in the list will also raise this same error.

Server UPS

The percentage you added does not work, you need to times it by 100.
The label has been moved to overlap with the multiplayer statistics, previously I was unaware of the overlap with the clock, so i propose we drop the label SUPS = and just have the number under the ups number but right of the clock. (-362, 30)

Warps

Error message for creating a warp too close to other entities was not updated to match changed radius. Line 122 change 4.5 to 2.5.


[Bug] Fast tab switch forces scroll up

Bug, feature request, or proposal:
Bug.
What is the expected behavior?
The tab should switch without triggering any scroll.
What is the current behavior?
When you switch tabs after making any scroll, the pages try to scroll up to the beginning of the page
What are the steps to reproduce?
Take a look at the last gif about the tabs example, here's the URL:
https://material.angular.io/components/tabs/api
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
"@angular/core": "~4.1.0"
"@angular/material": "^2.0.0-beta.8"
Is there anything else we should know?
This behavior can be seen in the next 2 gifs:

In the last one, you can see it even happens in material tabs example.

What's the browser that you're seeing this issue?

@donroyco in chrome and safari

@donroyco I updated the browser versions as requested.


Add WASM Support

Still, testing, but would you be open to supporting WASM? It looks like all that is needed is an extra dep for supporting std::time::Instant on web.
It's behind a feature flag so it doesn't effect anybody unconcerned with WASM.

Tested on my project and it looks like it works.

LGTM!

OK, I removed instant completely and just made sure that Mutex doesn't depend on std::time when targeting WASM. I didn't technically run tests on WASM, but if you take out the same code that is excluded when running on WASM then the tests pass on native.
I also added cargo check for WASM to CI!

Thanks! I would like to run tests on CI if possible -- do you think it is possible to do something like https://github.com/smol-rs/fastrand/pull/7?

I'll take a look at it.

There you go! Tests are passing for WASM in CI.

No problem, glad I could help! I love the simple smol ecosystem. 😃

Published in v2.4.0.


Typo error in command doc

I think the tag declared by https://github.com/theofidry/console/blob/main/doc/command.md?plain=1#L114 is false.
Should it be fidry.console_command ?

Indeed, thanks for spotting it :)


Use POMs directly from uberfire to reduce maintenance efforts

ip-bom is used in uberfire, so when upgrading we only
need changes there and not in this repo
both uf and uf-ext are always released together, so it
is fine that they uf-ext depends on SNAPSHOTs of uf

We will probably need to wait till jboss-integration-platform-parent:6.0.0.CR31 is synced into central, otherwise the build fails.
I will re-trigger the Jenkins build once the POM is synced, please don't merge until then.

Ready for review.

+1

Thanks.
master: 03bdc3a14651a5222270bdbf37af4f2d778d7377


Inconsistent Translation in "Files" App - "Allow" instead of "Autorisé" in Advanced Permissions

Description
I have noticed an inconsistency in the translation within the "Files" app. When accessing the properties and advanced permissions, the term "Allow" is displayed instead of the expected translation "Autorisé." This discrepancy is noticeable, especially considering that it is correctly translated as "Autorisé" on Crowdin.
Steps To Reproduce

Open Files
Right-click on a folder
Open Properties
Go to Security
Click on Advanced Permissions

Requirements
Ensure that the translation for "Allow" is updated to "Autorisé" in the appropriate language files.
Files Version
Version : <IP_ADDRESS>
Windows Version
Windows 11 22H2 22621.1778
Log file

Thank you for reporting this issue.


[FEATURE]: New feature integration in api

Description
Hello serengil i hope you are doing fine you have done a pretty good job till now actually i was interested in finding functionality of  deepface which can be accessed by running api service so i made the modification in routes and services file as well for accessing the find functionality but i am facing the issue of 404 not found when i am hitting the endpoint
Additional Info
No response

Find function wasn't added to api intent-fully. Because api should be stateless and you should not store facial database in file system. We are even running it with docker.
In future releases, we may add add to db, update and delete methods to store items in external db(s) but find function will not definitely be added to api as is.

so i am not storing the database in the system, but as we are using laravel for our project we put are facial database over server from where we can access the images of known faces and put it and for which i was modifying the route and services in my local but its not working so it would be very helpfull if you can provide a guide or tips to modify the api a bit in our local
thanks for understandinig


Swim, water, bikini, paddles, water sports

Need water sports icons for swim app. Thanks!

Similar to / Duplicate of (https://github.com/Remix-Design/RemixIcon/issues/64)

Thank you for your love of RemixIcon.
We'll try our best to add sports icons in the next version.
But, the next version of icons may be released after about a month.
@itsezc Thank you so much for your so much advice and follow-up.


Class 'Imagick' not found

I run phpinfo (); Inside a web application and php -i at the command line. Output of both if you mention that Imagick is installed. But still the same error. I don't know what's wrong

Any solution? I have same problem.


Import Postman files to burp

Hi @brenocss
Could you describe the issue since I'm not sure how you're in trouble?

Just a feature request, would be great if we could import to burp target map a postman json file

Hi @brenocss
I don't think it is a necessary feature since you can capture http requests from Postman on Burp after importing the postman file into postman.
What do you think?

it's just that redoing the requests by hand in postman just to send/proxy them to burp is a little boring. It would be more interesting to import everything and start an active/passive scan, autorepeater...

Hi @brenocss
I understand what you want now.
However, this extension was developed so that I could avoid pretty boring task of typing request infos into postman by hand.
I'm afraid your request is out of scope of this extension.
But once again, I understand what you want, so I might develop such an extension separately from this Postman Integration.
I appreciate for your undestanding.

I'm going to close this issue...


About the process for datamined cards

I am trying to insert the missing cards.
To avoid any confusion, I needed to know what command to use in cd to change path
Thank you again for your time spent

This explains the steps in detail https://github.com/pixeltris/YgoMaster/issues/113#issuecomment-1276747348


Fix compilation issues in dictionaryObjectRepresentation

This method was added but has some schema combinations that actually have preventing it from compiling. The scenario I found that errored was a property of type Map<String,Array<String>>
We should also add a integration test case that is exhaustive of all schema types to prevent this from happening

I've recently hit an issue (on v 1.3) where a certain schema doesn't generate the - (NSDictionary *)dictionaryRepresentation; message on the generated model. Reading this issue it appears that this was disabled possibly? Thing is, it does generate for the other .json schemas I use.
Is this possibly related to this issue?
{
    "id": "user.json",
    "title": "user",
    "description" : "Schema definition of a User",
    "$schema": "http://json-schema.org/schema#",
    "type": "object",
    "properties": {
        "id" : { "type": "string" },
    	"name" : { "type": "string" },
    	"role" : { "type": "string" },
    	"phoneNumber" : { "type": "string" },
        "avatar" : { "type": "string", "format":"uri" },
	"createdAt" : { "type": "string", "format": "date-time" },
	"updatedAt" : { "type": "string", "format": "date-time" },
	"latitude" : { "type": "integer" },
	"longitude" : { "type": "integer" },
	"isSignificantOther" : { "type": "boolean" },
        "relatedUsers" : {
		"type": "array",
                        "items": {
                             "type": "string"
                         }
            }
    }
}

@DreamingInBinary Hey - what is your exact problem? We had to disable dictionary representation as it caused issue.
That said, we just enabled json dictionary representation again on master. Can you pull master and see if this fixes your issue.

I’ll get out a new release soon but you should be able to pull and build the latest master to fix

Resolved in https://github.com/pinterest/plank/pull/122. Will be in the 1.4 release

@maicki I can confirm updating brought dictionaryObjectRepresentation successfully 👍


Added python solution for Vertical Order Traversal

Assigned Issue Number ?
#1394
Language Opted ?
Python
Number of files added or updated ?
1
Name of these files ?
algo_ds_101 => Data_Structures => Tree => VerticalOrderTraversal.py
Checklist

[x] I've read the contribution guidelines.
[X] I've referred the correct issue number.
[X] I've fill up this entire template correctly.

I added code for the program to take user input


evaluate undefined to type "undefined"

What kind of change does this PR introduce?
feature
closes #12063

Did you add tests for your changes?
yes

Does this PR introduce a breaking change?
no

What needs to be documented once your changes are merged?
nothing

Thanks


Optimize DistributedLoadCommand

Premise: Each job has an overhead of at least 1~2 seconds because of it takes at least JobWorker heartbeats (once for the work to go from JobMaster to JobWorker and once more for the JobWorker work completion to be reported backed to JobMaster) so it is important that when there are a lot of jobs to be done, that many jobs get submitted at the same time.
What was done:

Remove creating threads per job (only 1 thread)
Increase number of default outgoing jobs from 10 -> 1000 (also configurable)
Client round robins all of the outgoing active jobs, handles retries, and when they succeed, remove from the list to be filled with new jobs.

Test Environment:

Local machine
12000 Files
Approximately 450 bytes each
Ran fs free followed by fs distributedLoad, checked that distributedLoad was actually loading files and timed the run

New Code: 59 seconds
Old Code: 43 minutes 33 seconds
This is about a 45x speedup.
Memory Usage: Both the old version and new version memory leaked (the new version memory leaked much faster because it executes much faster) but the addition of https://github.com/Alluxio/alluxio/pull/10606 fixed the issue.

@chcn: This is probably testable already as is. What this is currently lacking is tests and better error handling.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/7208/
Test FAILed.

@chcn: I will but preferably not for the testing stage.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/7210/
Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/7219/
Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/7222/
Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/7224/
Test FAILed.

Ready for review.

@calvinjia: Want to look at this?

This change renders https://github.com/Alluxio/alluxio/issues/10608 relevant.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/7237/
Test PASSed.

addressed feedback. Not doing pooling.

Note that this is to be cherry-picked into 2.1.1

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/7240/
Test FAILed.

alluxio-bot, test this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/7243/
Test PASSed.

@calvinjia

alluxio-bot, merge this please.

alluxio-bot, cherry-pick this to branch-2.1 please.

Urgh.. I'll do the manual cherry-pick in that case.

@bradyoo any reason to change the option name? we typically will not change the name unless it is confusing. Also we need to update the CLI description string.

@apc999: The option name had to change because any previously used values for the option is woefully inappropriate for the new option. (Off by at least 2 orders of magnitude)


itkConceptChecking.h build problem

The build from development sources fails with the following error:
[ 94%] Generating ../../itkCannyEdgeDetectionRecursiveGaussianImageFilter.xml
In file included from /home/melrobin/packages/ITK/build/Wrapping/itkCannyEdgeDetectionRecursiveGaussianImageFilter.cxx:2:
In file included from /home/melrobin/packages/ITK/Modules/Core/Common/include/itkOffset.h:22:
In file included from /home/melrobin/packages/ITK/Modules/Core/Common/include/itkMath.h:33:
/home/melrobin/packages/ITK/Modules/Core/Common/include/itkConceptChecking.h:933:17: error: no viable conversion from 'UniqueType_bool' to 'UniqueType_bool<Self::Integral aka true>'
IntegralT a = FalseT();
^   ~~~~~~~~
/home/melrobin/packages/ITK/Modules/Core/Common/include/itkConceptChecking.h:941:3: note: in instantiation of member function 'itk::Concept::IsFloatingPoint::Constraints::constraints' requested here
itkConceptConstraintsMacro();
^
/home/melrobin/packages/ITK/Modules/Core/Common/include/itkConceptChecking.h:63:58: note: expanded from macro 'itkConceptConstraintsMacro'
using EnforcerInstantiation = Enforcer<&Constraints::constraints>;
^
/home/melrobin/packages/ITK/Modules/Core/Common/include/itkConceptChecking.h:134:8: note: candidate constructor (the implicit copy constructor) not viable: no known conversion from 'itk::Concept::IsFloatingPoint::Constraints::FalseT' (aka 'UniqueType_bool') to 'const itk::Concept::Detail::UniqueType_bool &' for 1st argument
struct UniqueType_bool
^
The problem occurs with gcc 11.2.1.  There was a small const problem when I reported a similar problem previously, but I do not know how to fix it.  There are several more errors of the type mentioned above.

@melrobin I ran into CastXML build errors with GCC 11.2.0, which are addressed with:
https://github.com/InsightSoftwareConsortium/ITK/pull/3112
However, I did not run into this error. Are there any non-default CMake configuration options?

I blew the build directory away and the build works.  I am adding modules one by one to try to track down where the problem could be.

Narrowed it down.  The error occurs when adding the LesionSizingToolkit

LesionSizingToolkit is in need of maintenance. Whoever uses it, should devote some time to it. Please submit this issue there and possibly cross-reference it to this one.

Aaaah  OK.  Thanks for that tip.


Display project/release journal

Sub ticket of #2734
We should show the maintainer some history of their project/release.  The current UI looks like this:

Questions:

Should this be on a project level?
Should this be on a release level?
Should it be on both?
What milestone should it be in?

On the last question, it's probably not a heap of work for me to get it into the maintainer MVP - so long as the data is ready to go :)
ping @di @ewdurbin @brainwane @dstufft

@nlhkabu Branch for you here: https://github.com/pypa/warehouse/tree/project-history


Typo "@@ROWCOUNT"→"@@ROWCOUNT"

https://docs.microsoft.com/en-us/sql/t-sql/functions/rowcount-transact-sql?view=sql-server-ver15
#PingMSFTDocs

@hyoshioka0128 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Looks good; thanks @hyoshioka0128.


orchestrator validate step getting different code coverage from the same ci pool

Hello everyone,
I'm running into an issue where on pipeline, same code, same type of scratch org in the same pool, keeps getting different code coverage, and diff a lot, also usually if I run validate locally, I can get 80%~85% in the same pool.
This is a unlocked package
@dxatscale/sfpowerscripts 10.2.6 (10.2.6)
sfdmu 4.12.7
sfpowerkit 4.0.3
node v12.22.3

any solution for this one?
Thank you

looks like it's 79% on the third run

We have the same issue, even having specific unit tests for each class in a package, the code coverage is not calculated.
I also checked the code coverage result from salesforce, and it seems salesforce doesn't take into consideration or skip some test

We are looking into this, the possible hypothesis is compilation is not finished due to delay at salesforce end, so tests is skipping those classes.

@azlam-abdulsalam I agree, since yesterday when I was investigating I saw that some classes didn't get code coverage due to "compilation in progress" status

@ypan0 @dieffrei  Initial analysis is pointing to the fact this happens only during parallel testing, as code coverage is getting skipped for classes as it being recompiled in parallel

We have disabled parallel testing in some orgs and it seems to be working better. Monitoring this issue

This has been fixed in alpha and will be part of the next release


Remove unreleased section from changelog

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

/lgtm

/approve


Yii::$container->set() config merge problem

There is a kartik\depdrop widget for the example. It has no localization:
$loading = ArrayHelper::getValue($this->pluginOptions, 'loadingText', 'Loading ...');

To solve the problem I tried to use DI:
Yii::$container->set(\kartik\depdrop\DepDrop::class, [
	'pluginOptions' => [
		'loadingText' => 'Загрузка...',
	],
]);

However, to use the widget, you must specify pluginOptions like this:
$form->field($model, 'attribute_id')->widget(DepDrop::class, [
	'data' => $someData,
	'pluginOptions' => [
		'depends' => ['depend_id'],
		'url' => 'some_url_to_fetch'
	]
])

The configuration is merged with array_merge:
https://github.com/yiisoft/yii2/blob/d0bf3cfeb4813cefbaa2e3ac0b6cbf0cbfa931c0/framework/di/Container.php#L168
And as a result, the use of Yii::$container->set() for set loadingText inside pluginOptions is not effective.
To solve the problem, need to replace array_merge() with ArrayHelper::merge().
Or how to solve this problem?

Duplicates #14821


Keycloak call /auth/admin/realms/GEFA-Intranet/users?max=250 is taking over 2 minutes

Hi,
I am using the camunda-platform-7-keycloak 7.17.0 version in a java springboot project with camunda version 7.14.0 to launch the camunda cockpit/tasklist.
Everything was working fine, but at some point the loading of the tasklist processes for some filters took a long time (about 2 minutes). Its not for all filters, only for some and I am not sure which criteria causes it to take so much time. The filter that takes so much time to load has only 9 open tasks.
I activated the debugging logs to see what takes so much time. The result is, that the keycloak call to /auth/admin/realms/GEFA-Intranet/users?max=250 takes over 2 minutes.
However I am asking myself why we would need this call? Couldn´t the plugin make a much more specific call instead of just loading 250 users from keycloak? This call is not even loading the users that we need in the process, it´s just loading every user, even the ones that don´t have roles that are needed for this process.
In the code I found the interesting part in the class KeycloakUserService with the method createUserSearchFilter.
There the call for Keycloak is created. It seems that this method is adding some arguments to make a specific call to Keycloak, but somehow in our case this call isn´t specified anymore, so we are loading just all users.
Do you have any idea why the method is not creating a more specific call anymore for keycloak?
Tanks in advance for your help.

Hi @MarvinKern,
the Keycloak Identity Provider has no influence on how the IdentityService of Camunda is used from the outside. If someone - in this case the tasklist - says "give me all users" the plugin requests all users... The mentioned method creates the user search filter according to the arguments given by the IdentityService API, there is no magic behind that.
So this is not an issue with the plugin itself, but more about the implementation of the Camunda Tasklist. As you already mentioned this condition is filter specific. So you can either optimize your filter or check if this is Tasklist specific and in a newer version of Camunda the behaviour has been optimized (e.g. check the Release Notes).

@VonDerBeck Thanks for your really quick answer.
In one of my filters that is really slow I defined one group that has the rights to read the filter. I would expect the tasklistk / keycloak plugin to only call for users from keycloak that are in that group, but like I mentioned its calling for all users.
Whats also strange to me is, that we have the same application in 3 different environments, but only in one of the environment the filters are really slow. So I have the feeling its not about the definition of the task, its maybe more about the tasks themselves, but I am really not could be wrong with them.

I found out, that the calls that are made to Keycloak are dependant on the assigned persons to a tasks. There are 2 scenarios:

I have several tasks in my filter, but only one person (TestUser) is assigned to those tasks, the plugin is making following calls:
/auth/admin/realms/Test-Realm/users?email=TestUser
/auth/admin/realms/Test-Realm/groups?max=250

I have several tasks in my filter and different persons are assigned to the tasks, the plugin is making following call:
/auth/admin/realms/Test-Realm/users?max=250

@VonDerBeck One quick win for me would be to have the maximum size of the users, which is currently set to 250 by default, configurable. Do you think that is an option?

@MarvinKern: restricting the maximum number of results in a query is easy - just have a look at the configuration options. maxResultSize does the job. Just try it and let us know.
The other option is to check Keycloak configuration itself. Are there some synchronization options leading to higher load than required? ...?

Sorry for the late response. We solved this issue with the help of configuring the maxResultSize attrbute to 50 and setting the cachEnabled to true :)

👍


fixed ffmpeg4.0 obsolencies

I have fixed some issues with ffmpeg 4.0
Both ada and glslviewer now compile fine in my machine
For the decode_video2 function I refactored this solution: https://github.com/pesintta/vdr-plugin-vaapidevice/issues/31
I don't know if there's a better way to fix these errors, and I have not changed the cmake nor added ifdefs to control versions, but I hope it's helpful

Thanks so much!
I'm getting some troubles compiling it in MacOS and Ubuntu 20.04. Do you know if it's possible to get the libav version and only apply the fix it on newer versions?


Add bFrameAdjustment configuration option to address PTS/DTS bug

I know you guys have been working on gap jumping as a workaround for the PTS/DTS bug, which should go a long way for many use cases. We have our own gap jumping implementation, but it has some limitations. Specifically: transitions to new periods are not smooth, and content with a correctly set base_media_decode_time (DTS) will not start because it ends up with a negative presentation start time.
This approach fixes the underlying problem, but requires uniform and predictable frame rates and bframe counts.
I think it is a relatively unobtrusive solution that goes a long way towards achieving smooth multi-period playback for most use cases.
Take a look and let me know what you think. If it is something you'd consider I can clean it up and fix tests.

WIP: pending discussion, will add/fix tests
This change introduces a configuration option that adjusts the timestampOffset and appendWindowEnd to address the PTS/DTS bug in Chrome. It assumes that the number of bframes is known, and constant. It also assumes that the manifest provides an accurate framerate.
Full explanation at: https://github.com/baconz/shaka-player/blob/bframe-adjustment-for-chrome/externs/shaka/player.js#L474

@baconz, this is very interesting.  I'm glad that this is a workable approach for you, but I would like to defer consideration of this PR until we are done with #555.  At that point, if our solution does not meet your needs, we can reconsider this PR or something based on it.  In the mean time, I'm glad you are not stuck waiting on us!

Sounds good @joeyparrish. Let me know if/when you'd like to discuss this further.

@baconz, we recently landed gap jumping in #555.  Is this PR still applicable?  Or does gap jumping solve the same problem this PR is meant to solve?

Hey Joey, I'm out this week, but will take a look next Monday when I'm
back.
On Mon, Apr 24, 2017, 16:15 Joey Parrish<EMAIL_ADDRESS>wrote:

@baconz https://github.com/baconz, we recently landed gap jumping in
#555 https://github.com/google/shaka-player/issues/555. Is this PR
still applicable? Or does gap jumping solve the same problem this PR is
meant to solve?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/google/shaka-player/pull/731#issuecomment-296872206,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABElxlPt66zzIIjp6j8h1qD8z_hkHncRks5rzVdGgaJpZM4Mkobw
.

@joeyparrish unfortunately #555 doesn't solve the whole picture for us. There are two problems:
(1) Chrome sees negative start times because we use PTS as the the PTO, and Chrome uses the DTS as the timestamp of the first sample.
(2) Chrome sees gaps because Chrome sees periods as shorter than they should be.
#555 solves (2), but not (1).
With (1) Shaka will throw a 3016 and you'll get an error along these lines in media-internals:
video frame with PTS 0us has negative DTS -66734us after applying timestampOffset, handling any discontinuity, and filtering against append window

Here's a diagram that represents the problem for a single segment:
   PTO
   PTS
   6006
      |

 |
 0
DTS

Here the PTO would be set to 6006, and Chrome would see the first timestamp as -6006.
This PR allows you to mitigate this problem by adjusting the PTO based on the maximum number of concurrent b frames in the underlying content.

Okay, I see now.
I understand the need for this, but I am still hesitant because this feels like a lot of code to work around a Chrome bug.  I don't know when, but this will get fixed in Chrome at some point.
How would a user of this feature employ it only on Chrome, and only on affected versions of Chrome?

I agree that it is a lot of code to work around this bug, but given that the bug has been open 3 years, we're not holding out hope that it will be fixed. To us it seemed cleaner to push the workaround as close as possible to the client instead of branching off somewhere further up the video pipeline.
We make the decision on whether or not to enable the flag when we call player.configure() based on out-of-band browser information (I think we use bowser).
If we really wanted to be fancy, we could embed a very simple fragment with a single frame and a known PTS. When we initialize the player, we could pass it to a test SourceBuffer and read back the buffered range to find out if the browser has this bug.
If we wanted to be really fancy, I think we could detect the number of consecutive b-frames by using the num_ref_frames parameter in the SPS, but that would require parsing the init.mp4 and parsing the SPS, which seems like a lot of code.

@baconz, I've been informed by the Chrome team that they are actively working on this and hope to land something soon.
Since you have this workaround implemented in your fork, there's no urgency to merge it, correct?  If so, I propose we defer for a couple more weeks to see what Chrome team can accomplish in terms of a fix in that time.

@joeyparrish sounds good, it would be awesome if they push a fix!

@baconz, I know this has dragged on for quite some time, but would you be willing to test another round of Chrome fixes with regard to this issue?
In particular, you should use Chrome beta (63) with the MseBufferByPts feature turned on via the command-line:
google-chrome-beta --user-data-dir=/tmp/user-data --enable-features=MseBufferByPts

I only recently became aware of the fact that Chrome team is experimenting with PTS-based buffering behind this flag, to avoid accidentally breaking any media sites that may only work with the buggy behavior they are trying to fix.

Alternately, a test case or demo page that showcases the problem would be allow us to go directly to Chrome team with your specific use case, and allow them to iterate more quickly toward fixing the issues that are affecting you.


[Fix 15520] wrong feature description (rollover) in changelog

Fixes #15520.
If this is accepted, someone should update the release notes accordingly

LGTM
Thanks!

LGTM thanks!

ping @calavera for editing the release notes?

Is there an example anywhere of how to actually use these options? as far as I can see there is a param maxSize for the json file driver but I cannot find any docs how to use it.

@vanthome See https://docs.docker.com/reference/logging/overview/#the-json-file-options


service keba.stop does not work

Calling service keba.stop (no rfid specified) does nothing, the car keeps on charging.
The disable button works fine, and so does keba.start service after the disable button is pressed.
Tried with version 3.0.3

I’m very happy with this integration because I now I’m no longer dependent on the (slow) BMW webservice to see if the car is charging or not. But like above I have a problem with starting and stopping charging via the integration. I have no RFID in use and therefore could not specify anything in the integrations setup. Start and stop now resulting in a authentication error. Is there a way to circumvent the RFID sequence?

I changed my scripts/automations to use
service: button.press
data: {}
target:
  entity_id: button.keba_p30_enable

for starting, (and button.keba_p30_disable for stopping). Could this be an option for you as well?

I do the same. As long as the charging is not stopped by the car (becaus the condifured soc is reached) enable will restart charging directly after hitting enable.
I the other case, when the car stopped the charging, I have to send the "charging" command to the car via api aswell.

I changed my scripts/automations to use
service: button.press
data: {}
target:
  entity_id: button.keba_p30_enable

for starting, (and button.keba_p30_disable for stopping). Could this be an option for you as well?

That is helpful @matlimatli . I compared this to the call that the Call Service node is making in Node-Red:

So this is not resulting in the same call. I cannot get to a button.press service as far as I can see now. What I do see is that I can select a Keba domain in the node which can be used to start and stop. Which results in the following call.

I have to test if that is doing anything / the same as your automation call.

Although the call is looking different it isn't.
I see that:

Is picked up fine by the API:

The keba domain start/stop doesn't do anything from Node-Red.

Actually, I have the same problem as OP. When I use the disable button, the charger sometimes suddenly starts charging again. I haven't really figured out why and when, but it is quite problematic for my charging scripts. So I would really like to be able to use the keba.stop service, which at least previously worked well.

@mattsve sorry for this very late response, I am currently updating the underlying library. Which version of the charging station do you have? If you do not have RFID, start and stop will do nothing. This is by purpose and should probably be renamed authoratization.

Calling service keba.stop (no rfid specified) does nothing, the car keeps on charging. The disable button works fine, and so does keba.start service after the disable button is pressed.
Tried with version 3.0.3

I guess that is the intended behaviour (https://www.keba.com/download/x/4a925c4c61/kecontactp30udp_pgen.pdf)
The command stop can be used to deauthorize a charging session if the authorization function is activated. The effect is the same as holding an RFID card in front of the RFID reader during a running charging session. The deauthorization will only work correctly if the RFID tag used is the same as the one used to start the charging session.


Added a test folder

put the tests in the test folder and then updated the "Project Structure" accordingly on the README.md file.

You screwed all this up Red-Burd... you were supposed to pull the repo from Green-Burd since he was the original repo owner...I'll try & fix this with him. :(


Add the GKE provisioner

What does this PR do?
Add the GKE provisioner
Motivation
https://github.com/DataDog/test-infra-definitions/pull/1105
Describe how to test/QA your changes
Possible Drawbacks / Trade-offs
Additional Notes

Nice!!


admin documentation

thanks, I'll look at this in 2-3 days

@gjwalk I merged your work but there are still a few issues related to names and links

Folders should also be lower-case kebap - so HostAdministration should become host-administration
Images should always be in a subfolder assets on each folder, to not burden the main folder - ATM they are usually in the same folder, which is not how we want the convention to be
ca. 54 images currently don't show because they are linked in places where they are not like

https://docs.oqtane.org/admin/Modules/adding-modules.html
https://docs.oqtane.org/admin/Modules/working-with-modules.html
etc.

Pls pull the latest changes (I had rebuilt the docs and republished as some details were missing) and correct this.
Thx for your work!


Plugin navigation bar cannot show its complete content when the navigation bar is folded.

Describe the problem/challenge you have
Since my plugin is located at the bottom of octant navigation column, the dashboard can not show its complete content when the navigation bar is folded.

The complete content can only be shown when the navigation bar is not folded.

Describe the solution you'd like
I am not sure if octant can enhance octant dashboard or plugin API to show the complete content, maybe by showing the floating menu of the last navigation bar above instead of below?
Environment:

Octant version (use octant version): v0.13.1
Kubernetes version (use kubectl version):
OS (macOS 10.15, Windows 10, Ubuntu 19.10 etc):

Thank you for reporting this @mengdie-song . This issue will be fixed via #1353
I've linked to our latest community meeting (20 minutes in) where @mklanjsek does a preview of some of the ideas we are working on for the new navigation. https://youtu.be/Yos2hhjiooY?list=PL7bmigfV0EqQ55lLisHHy2n1MDuRZasrR&t=1236

Thanks @wwitzel3 !


Adds a kubecontext global flag

Fixes #207

@blakebarnett thanks for the PR! Would you please sign your commit, and also run gofmt on the files you changed? Thanks!

Ah I see, I need to actually update the docs to include, I'll take care of that in the next commit.

@blakebarnett looking pretty good! Just run make update to update everything (docs).

@blakebarnett FYI, we'll pick this up as soon as master is open for v0.8.0 later this week. Thanks for your patience!

@blakebarnett we'd like to get this in sometime soon. Could you please squash & sign your commit?

I signed it, I thought GH would auto-squash on merge?

You currently have 4 commits, and each one must have the DCO stanza in it. I don't see it in any of them?

sorry, never used GH's signing stuff

DCO is good, thanks!

Tested this, can confirm that it works as described.

Ugh, helm uses --kube-context, kubectl uses --context and now ark uses --kubecontext.  I'm probably going to get it wrong every single time.

We could probably support both --kubecontext and --kube-context. I think --context is potentially too generic for Ark, though.

Yeah, I did it this way because it already had --kubeconfig but I'd prefer if it matched helm also.


Error when using search

Steps to reproduce
-Open Neko
-Go to "Browse" category
-Tap "Search" button
Expected behavior
Search menu should open filter and preset options
Actual behavior
An error log immediately appears, the issue remains after relaunching the app or even the phone.
Crash logs
neko_crash_log-202412061917.txt
Neko version
2.18.2
Android version
Android 10
Device
Redmi Note 8 Pro
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open issue.
[X] I have written a short but informative title.
[X] I have updated the app to the newest version Latest.
[X] I have filled out all of the requested information in this form.

I had this issue as well and had to clear my saved filters. If you want to do that you'd go to Settings -> MangaDex Settings -> Delete saved filters. Hopefully someone else will have a less destructive method though.

the above is the solution


perf: optimising variant pattern matches in special cases

My conviction is that Motoko should support idiomatic use of variants efficiently:

one-armed variants are akin newtypes in Haskell and Rust; while we can't avoid the heap allocation and the wrapping, we shouldn't waste energy on checking the variant tag
two-armed variants are often used for Result-like error propagation, and so these should only perform a single comparison (followed by an if) before unpacking
many-legged switches should use br_table, and find a common divisor for all hashes that gives the right table indices (as remainders).

Note: The ultimate goal here is not to generate perfect code in the backend, but to make sure that there are no obstacles for ic-wasm doing a good job cleaning this up.

Further considerations

type-guided serialisation could also use such tricks (if not already)

I performed a few division tests, but can only find few consecutive remainder sequences. I am now pondering multiplying all hashes with a prime p and then looking for a divisor. This theorem gives me hope: https://twitter.com/AlgebraFact/status/1677419576177352704?s=20

Better optimisations for patten matching are good ideas. There are more sophisticated methods for building decision trees for nested patterns as well. Of course, with all these optimisations, it is not immediately clear how big a gain they are in practice and hence how much bang-for-the-bucks they provide.
Re the use of br_table, could it work to modulo/xor the hash with a small number first (some single-digit multiple of the number of cases) and br_table on the result of that, and then resolve cases with collisions in respective branches by secondary comparisons?
PS: Oh, and would it be hard to use more meaningful issue/PR titles? ;)

Better optimisations for pattern matching are good ideas. There are more sophisticated methods for building decision trees for nested patterns as well. Of course, with all these optimisations, it is not immediately clear how big a gain they are in practice and hence how much bang-for-the-bucks they provide.

We have a dao micro-benchmark in the making and I'll check that in before these hit master.

Re the use of br_table, could it work to modulo/xor the hash with a small number first (some single-digit multiple of the number of cases) and br_table on the result of that, and then resolve cases with collisions in respective branches by secondary comparisons?

Absolutely, I've been thinking of masking out a small part of the hash, that has sufficient diversity of bits for a subsequent division.

PS: Oh, and would it be hard to use more meaningful issue/PR titles? ;)

@ulan do you have ideas/references about such optimisations?

Might be worth looking at how ocaml deals with polymorphic variants. Perhaps they have a good trick.

Is there no compile time algorithm that takes n numbers and returns a small number of numeric operations that map an input to 0..n-1 if it is the corresponding number (and arbitrary otherwise)? But probably nothing easy enough, else other languages with hashing tags would use it already, right?

I can write such a function in general, but it would be a polynomial of degree n, do hardly more efficient than n comparisons :-)

Minor thought: About avoiding irrefutable tag checks: Instead of making the compile.ml code more complicated, we could introduce a pattern failed prim, and have an IR-to-IR pass that elaborates pattern matches to be complete (by inserting a _ match that fails where needed). Then the backend can trust that pass and omit tag checks in the last clause.

I can write such a function in general, but it would be a polynomial of degree n, do hardly more efficient than n comparisons :-)

Good old Lagrange would rotate in his grave 😂

I have discussed the the UncheckedTagP idea with Kento yesterday. I am shying away from it (for now) because it seems overkill for such a small PR. As this idea grows we could move the machinery to IR, indeed. There is already one such PR (#3955, but that got unfavourable comments, so it is on hold). There are many more transformations that can optimise case lists with variants.


bsubmit first commit

bsubmit util makes user submitting job as another user

Signed-off-by<EMAIL_ADDRESS>


Initial clang-format file

This is a first proposal for the clang-format file along with a set of rules written in Markdown.
[update psychocoderHPC]
Please read https://github.com/ComputationalRadiationPhysics/contributing/pull/39#issuecomment-694956183 before you start to comment on this PR.

Since code style based on own opinions it is hard to find a con-sens.
Therefor I will apply some rules to this PR else there will be the question who is selecting the final value for the style option.
We introduce a voting system I will describe in this post.
To be clear, after we have a common code style the style file will be pushed to all our projects, including PIConGPU, cupla and alpaka
Rules

Use direct comment instead of the grouped review feature.
Do not comment that you like or dis-like something if you do not like to change an option or vote. (This reduce the number of comments)
If you dis-like an option start a vote

Voting:

if you like to change an option open a vote with the following template:

**VOTE**
current | <option1> |
--------|-----------|
 0      |   1

<long description why you like to change it or why you voted how you voted>

Everyone can only vote once!
Everyone who votes is increasing the value in the column he likes.
If you changed you opinion you comment again and reduce the column value you set by 1 and increase an other column.
If you like to set an other value than in the voting options than add a column.
At the moment where someone starts a vote everyone has 5 days to vote until the final value will be fixed set
If there is a race in the votes because two users start voting at the same line I will fix the counting with my admin power
If two voting options will have the same count after 5 days I will ask everyone to rethink his opinion (hope this solves such situation)
@j-stephan do not push changes to this PR

Example

USER1 not like the 79 char line length he/she starts

**VOTE**
current | 120 |
--------|-----------|
 0      |   1

I think 120 is fitting current monitors better

USER2

**VOTE**
current | 120 |
--------|-----------|
 1      |   1

I think a three way diff requires short lines

USER3

**VOTE**
current | 120 | 140 |
--------|-----------|---|
 1      |   1       | 1

Come on we need longer lines

USER 1 is changing is opinion
USER3

**VOTE**
current | 120       | 140 |
--------|-----------|---|
 1      |   0       | 2

Ohh USER2 is right, I like the suggestion

5day's later

**FINALIZED**
option will be changed to 140

If you thing someone should attend this code style finding process please provide the link of this PR to him/here.
CC-ing: @ComputationalRadiationPhysics/picongpu-developers @ComputationalRadiationPhysics/alpaka-developers

I like the format proposed by @psychocoderHPC. In addition I suggest that anyone who starts a vote also needs to supply an accompanying change to guidelines.md as part of the vote.

Please also note the separate vote in #40.

Since the interval ended for several votes this weekend, I prepared a new .clang-format file and ran it on the current develop of LLAMA. Here is the result: https://github.com/ComputationalRadiationPhysics/llama/pull/103/files

There are no more open votes. If there won't be any new votes until Friday I will close this PR - all options not voted for will have their current values then.

I opened another vote, which apparently went unnoticed: about tabs.

There are no more open votes. If there won't be any new votes until Friday I will close this PR - all options not voted for will have their current values then.


No graph visualization from examples on codepen.io

Reproduction:
Visit https://codepen.io/Rumyra/pen/qyMzqN/
Actual:
No graph is visualized.
Expected:
A graph should be visualized.
(This is one of Web Audio demos on MDN)
My suspicion is that DT events are not being fired from an embedded frame.

The iframes on the page probably have a different domain so I think we likely need to expand the extensions permissions.
https://developer.chrome.com/docs/extensions/mv3/declare_permissions/

The extension manifest is created from a template file https://github.com/GoogleChrome/audion/blob/main/src/build/manifest.json.mustache

I don't think I thought of this earlier, if it isn't a permission we need to add there may be an additional debugger target for the iframe. The extension attaches to the tab's debugger target. If there is a debugger targets for the iframes too, we may need to additionally attach and enable web audio events for those.

That sounds like a bit of engineering work then. For now, I can add a short note that the extension doesn't support inspecting iframes.

Example code location:
https://github.com/GoogleChrome/audion/blob/main/src/devtools/WebAudioGraphIntegrator.ts#L579


Add Haikubox Linker app

I made an app for my dad to send daily data about the birds in our yard from his Haikubox device to the Tydbit 2.  I haven't made any apps before so I'm sorry if it sucks. Also I'm planning on sprucing up the aesthetics a lot!

I have read the CLA Document and I hereby sign the CLA

recheck

An email may not have been associated with the commit, but I just updated my git email configuration to<EMAIL_ADDRESS>Please let me know if there is anything else I need to complete.


Navbar padding changed

Related Issue
[Cite any related issue(s) this pull request addresses. If none, simply state "None”]
Closes #1023
Description
[Please include a brief description of the changes or features added]
Final Pr for making navbar changes and adding proper padding
Type of PR

[X] Bug fix
[ ] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Screenshots / videos (if applicable)
[Attach any relevant screenshots or videos demonstrating the changes]

Checklist:

[X] I have performed a self-review of my code
[X] I have read and followed the Contribution Guidelines.
[X] I have tested the changes thoroughly before submitting this pull request.
[X] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[X] I have commented my code, particularly in hard-to-understand areas.

Additional context:
[Include any additional information or context that might be helpful for reviewers.]

mention your issue number

mention your issue number

Issue #1023 I had made a PR previously which you had merged and then reverted. Therefore the related issue is closed , but my changes have not been merged

Navbar of buy products page is not proper.