[Feature Request] Command cooldowns

For example: you're fetching data from an external api with very strict ratelimits. And you'd want your command to only be used once per minute.

I would love to see a cooldown system like discord.py with cooldowns for channels, guilds, users, or members.

Would the cool down system need to have support for cool down buckets? I.e. group commands into buckets whose cool down is shared.
If cool down can be per command, that would be easier to implement and I could start working on it even with the library not having fleshed out command group support

I don't plan on having grouped commands, so per command is perfect for me.

Old-ish issue but I'd like to mention that I was working with discord.py for multiple years (doing mostly commissions) and I never had to use anything that's not per command cooldown

I created an implementation in a feature branch. You can configure cooldowns like this:
// Numbers are in seconds
#[poise::command(
    global_cooldown = 1,
    user_cooldown = 1,
    guild_cooldown = 1,
    channel_cooldown = 1,
    member_cooldown = 1,
)]

When a cooldown limit is hit, the cooldown_hit callback on FrameworkOptions is invoked with the Context and a std::time::Duration for the remaining cooldown time. By default, it prints You're too fast. Please wait {} seconds before retrying.
Currently, there is no equivalent to discord.py's Category and Role cooldown buckets, because they seem a bit exotic and niche for me. Although support can be added later if needed.

Any feedback on this design, is it ergonomic?

Merged into master

#[poise::command(
    global_cooldown = 1,
    user_cooldown = 1,
    guild_cooldown = 1,
    channel_cooldown = 1,
    member_cooldown = 1,
)]

This is amazing, so simple yet so useful. Thanks for this


Validation with cardinality restriction

Hi Marco,
here are two simple examples of inconsistent ontology:
CardinalityTest1.ttl.txt
CardinalityTest0.ttl.txt
OWLValidator does not report the inconsistency:
OWLOntology ontology = OWLOntology.FromRDFGraph(RDFGraph.FromFile(RDFModelEnums.RDFFormats.Turtle, filepath));
OWLValidator validator = new();
foreach (OWLEnums.OWLValidatorRules rule in Enum.GetValues(typeof(OWLEnums.OWLValidatorRules)))
    validator.AddRule(rule);
OWLValidatorReport report = validator.ApplyToOntology(ontology);
Console.WriteLine(report.EvidencesCount); // writes 0 here

Hi constnick,
a fix for the reported situation is now available. Support for validating local cardinality constraints has been added in the ClassType rule.
Thanks and regards,
Marco

However, there is a little more complicated inference in other known reasoners.
For example, the following ontology is consistent:
ex:contains rdf:type owl:ObjectProperty .
ex:Item rdf:type owl:Class .
ex:item01 rdf:type ex:Item .
ex:item02 rdf:type ex:Item .

ex:iBox rdf:type [ rdf:type owl:Restriction ;
                   owl:onProperty ex:contains ;
                   owl:maxQualifiedCardinality "1"^^xsd:nonNegativeInteger ;
                   owl:onClass ex:Item
                 ] ;
        ex:contains ex:item01 ,
                    ex:item02 .

In this case a reasoner infer that ex:item01 owl:sameAs ex:item02
The ontology become inconsistent only if assert the difference explicitly:
ex:item01 owl:differentFrom ex:item02 .

Hi constnick,
a fix for this kind of inconsistency to be driven by explicit owl:differentFrom is now available.
No updates to the reasoner (since this was a validation issue). It may come in next releases.
Thanks and regards,
Marco


How to run the code without external MT data?

Hello, recently I try to train the ConST model without external MT data just to test its performance in this situation and I'm confused about how to regulate the parameters to  achieve this. I think the '--task, --external-parallel-mt-data, --criterion' should be changed but I fail to find the correct candidate for them due to so many choices. I would be extremely grateful if you could give me some guidance. Looking forward to your early reply.

Hello, recently I try to train the ConST model without external MT data just to test its performance in this situation and I'm confused about how to regulate the parameters to achieve this. I think the '--task, --external-parallel-mt-data, --criterion' should be changed but I fail to find the correct candidate for them due to so many choices. I would be extremely grateful if you could give me some guidance. Looking forward to your early reply.

It's been a long time since I last ran the code. If I remember correctly, the code in script/train_en2x.sh also supports WITHOUT external MT dataset, --task=speech_to_text_triplet_with_extra_mt but leave --external-parallel-mt-data as none, for criterion, you may use label_smoothed_cross_entropy_with_constrastive.

Sorry for the late reply because of some personal affairs. Thanks for your reply, that works well for me.


Support %%grid

the tool is amazing. I'm not sure why %%grid2 1 isn't working.
should look something like this.

grid2 isn't supported by abcjs, so not available in my tool.


Headings mostly gone from bikeshed-data

Most headings seem to have disappeared from bikeshed-data over the last 3 days. Looking at what the Shepherd API returns, I see that "section": true properties seem to have mostly disappeared, e.g. see anchors returned for CSS Viewport (section is only set for the Change appendix heading). @plinss, any recent change that could explain this behavior?

Investigating

A recent bug fix introduced another bug. Should be fixed now, tho it'll take up to an hour to re-parse all the specs.

Headings are back, thanks for the fix!


Update mentions to Argo Smart Routing in Tunnels now that it is no longer a requirement

It's not clear if this is the direction we're heading. Let's leave it as is.


NoReverseMatch when using render_model

I'm trying to use render_model in my html template but I'm getting this error:
NoReverseMatch at /en/news/
Reverse for 'cms_myapp_news_change' with arguments '(5L,)' and keyword arguments '{}' not found. 0 pattern(s) tried: []
Request Method: GET
Request URL:    http://<IP_ADDRESS>:8000/en/news/
Django Version: 1.9.6
Exception Type: NoReverseMatch
Exception Value:
Reverse for 'cms_myapp_news_change' with arguments '(5L,)' and keyword arguments '{}' not found. 0 pattern(s) tried: []
Exception Location: env\lib\site-packages\django\core\urlresolvers.py in _reverse_with_prefix, line 508
Python Executable:  env\Scripts\python.exe
Python Version: 2.7.11
Python Path:
['myapp',
 'env\\lib\\site-packages\\mysql_python-1.2.5-py2.7-win-amd64.egg',
 'myapp',
 'myapp\\%PYTHONPATH%',
 'C:\\Python27',
 'C:\\WINDOWS\\SYSTEM32\\python27.zip',
 'env\\DLLs',
 'env\\lib',
 'env\\lib\\plat-win',
 'env\\lib\\lib-tk',
 'env\\Scripts',
 'c:\\python27\\Lib',
 'c:\\python27\\DLLs',
 'c:\\python27\\Lib\\lib-tk',
 'env',
 'env\\lib\\site-packages']
Server time:    Tue, 19 Jul 2016 17:48:20 +0100

In my html I have:
<h2>
      {% render_model news "title" %}
</h2>

Do you know what's wrong?

Closing as this was addressed on a different chat.

@czpython  I didn't understand why you closed this issue. I still couldn't figure out what the problem is. Thank you :)

I'm having this same problem, where was this addressed @czpython?


Unable to create tag with default tokens (,|ENTER) when tagging-label="false"

Since 0.11.2
If taggging-label="false" is used, the default tokens ',' and 'ENTER' do not create a tag.
Attempt to create a tag by pressing ',' or 'ENTER' on the following Plunkr.
http://plnkr.co/edit/RK7a6NgAAFGTEm48UjGn?p=preview

+1

As a workaround I've made results dropdown hidden:
.select2-drop { display: none; }
(in my case user does not need to select existing tags) and set tagging-tokens attribute:
<ui-select multiple tagging ng-model="keywords" tagging-tokens="ENTER|TAB|,">
It allows to have tokens working without ability to select existing values.

hey @dmitryevseev I try your solution but I get console error
TypeError: Cannot assign to read only property '_uiSelectChoiceDisabled' of aaaa undefined
How can I fix this?

@cesar-oyarzun-m This solution worked on 0.11.2. You're probably using a newer version

yeah I'm usigin 0.17.1 but I have the same problem


TypeError related to command palette + tags

to reproduce:
with no node selected (hit ESC or click an empty space in the side nav), open the command palette and choose a command
then go to any node and try to add a tag
console will show a TypeError, you can't add the tag, and the tag menu is empty.
https://share.getcloudapp.com/X6u78GpQ

this may have been resolved

tara to verify


HTML gets escaped during export

HTML that appears in the document gets escaped when exporting to HTML or PDF.

Oops sorry wrong repo.


Lying Read.me

It says that it contains starfall scripts, I wanted to steal them and take credit, but I cant find them. Can you help me?

It says that it contains starfall scripts, I wanted to steal them and take credit, but I cant find them.

No, it doesn't say that. It says that I use this repository for "sharing" stuff with public.
Guess what? I have not shared any Starfall script with public, yet.
All of the code in this repository is open-source with copyright, and is licensed under the MIT license.

Can you help me?
No, I can't help you.

Closing this issue to avoid wasting of my time, if you have any other questions, please open a new issue.


How to interrupt a running cell and the kernel?

How to interrupt a running cell and the kernel?

For anyone else, I have to also start inputting into the window.
So press A or I and then hit ctrl-c.


fix(formatting): Compute minimal updates for buffer updates / shifting

In cases where there is a scoped buffer update, like a few lines, Onivim is smart enough to shift 'markers' (syntax highlighting, codelens in #3255 , search highlights).
However, there are some cases where we get a full update - the entire buffer contents is flagged as updated. A couple of these cases are:

Running :e! to re-load the file from disk - this comes through as a refresh of the entire buffer
Some format providers provide scoped updates (ie, line-by-line). Others, however, like ocamllsp just send the whole buffer contents.

The problem with processing the full update is that, today, those don't shift markers - we just clear out syntax highlighting, codelens, etc. This is a poor user experience, because it causes 'flashes' of syntax highlighting disappearing, or codelens disappearing and then re-appearing.
In order to have a better experience - this PR adds the computation of a set of minimal updates from these full updates. After computing minimal updates, Onivim can then properly shift markers, codelens, syntax highlighting, etc.
This is important for #3246 , as this same logic will be used to shift and preserve the cursor position.

This actually helps with some of the cases for https://github.com/onivim/oni2/issues/1581 - the flashes of the existing lines that are shifted by the add / delete are now gone.
The case that still needs to be addressed is immediately applying syntax highlights for the pasted text to eliminate that flash.


Fix issue #391

Fixes #391 Offline with service worker not working on new window/tab.
Changes proposed in this pull request:

when config db try to replicate, if app was offline, then replicate.from function return error, so change .then(loadConfig) to .finally(loadConfig)
Added optional creation of the main db as remote or local (depend on app state - online/offline). Before, even if the application is offline, the application attempts to create a remote database, which results in an error.

@Epsilonnnn thanks for the PR!  I wasn't expecting anyone to work on this issue, but I am thankful that you took interest in it.  I'll take a look at your changes.

Looks good @Epsilonnnn.  I am going to merge it in.


subview doesn't work for me

Steps to reproduce

use the code from master.
copy the test code from readme.
test on genymotion android emulator run on mac os.

Expected behaviour
See the capture button
Actual behaviour
Can't see the capture button
Environment

Node.js version: 6.1.0
React Native version: 0.25
React Native platform + platform version: Android 5.0

react-native-camera
Version: master

duplicate issue..
I have added a comment on how to get the button outside camera here:
https://github.com/lwansbrough/react-native-camera/issues/198#issuecomment-200468857

it seems an z-order problem in class RTCCameraView, i have added this code in RCTCameraView.java
import android.view.View;
.....
  @Override
    public void onViewAdded(View child) {
        if (this._viewFinder == child) return;
         this.removeView(this._viewFinder);
        this.addView(this._viewFinder, 0);
    }

and for my project sub item now work correctly

@gchiocchio good find, can you PR that change?

yes i have sent the pr

I see your branch but no PR...

Thank you gchiocchio! My app works now!


encodeURIComponent missing in document

[Enter feedback here]
function 'encodeURIComponent' is missing in document

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

ID: 72651acd-eb46-2e18-586a-c8e0e451a9ba
Version Independent ID: 720f83e1-0f27-ea61-1763-5bbe97b7379b
Content: Template functions - Azure Resource Manager
Content Source: articles/azure-resource-manager/templates/template-functions.md
Service: azure-resource-manager
Sub-service: templates
GitHub Login: @mumian
Microsoft Alias: jgao

Thanks for bringing this to our attention. Your feedback has been shared with the content owner for further review.

@ssmaredia - thanks for this feedback. Where are you seeing encodeUriComponent? We have uriComponent which encodes a URI. Is that what you are looking for?

Actually encodeUriComponent is part of 'Workflow Definition Language' (WDL) used for Logic Apps. WDL has its own functions documented in below link. I think this issue can be closed.
https://docs.microsoft.com/en-us/azure/logic-apps/workflow-definition-language-functions-reference


Cimpl 6 fix

I resolved the Git issue so this commit is ready to be merged now.

I squash-merged it to combine the commits and merge commits into one (and keep history cleaner).


Additional Settings for date, frontmatter, title, thumbnail, description, and summary

I have scrambled together a youtube template which uses Templater.  QuickAdd allows me to create a new note with the video on my clipboard.  I wanted to use the summarizer plugin immediately afterward to add the summary (as the URL is still on my clipboard).
The problem is that now, I end up having duplicate metadata.
It would be great to have settings for:

toggling date (on/off)
editing date format
toggling frontmatter (---) (on/off)
toggling title (on/off)
editing title format
toggling thumbnail image (on/off)
toggling description (on/off)
editing description format and metadata fetched
toggling summary heading (on/off)
editing summary minimum number of bullets

For reference, here's the content of my template:
<%*
let url = await tp.system.clipboard();
let page = await tp.obsidian.request({url});
let p = new DOMParser();
let doc = p.parseFromString(page, "text/html");
let $ = s => doc.querySelector(s);

let qcFileName1 = $("meta[property='og:title']").content;
let qcFileName = qcFileName1.replace(/:/g, "_");
let title = "🎬 " + qcFileName;

await tp.file.move("/3_resources 📚/videos 🎥/" + title);
let duration2 = $("meta[itemprop='duration']").content.slice(2);
let duration1 = duration2.replace(/M/gi, " min ");
let duration = duration1.replace(/S/gi, " sec ");

const html = await page.toString();
const matches = html.match(/ytInitialPlayerResponse\s*=\s*({.+?})\s*;\s*(?:var\s+meta|<\/script|\n)/);
const json = JSON.parse(matches[1]);
const videoDetails = json.videoDetails;
const keywords = videoDetails.keywords;
const description = videoDetails.shortDescription;
const author = videoDetails.author;
const channelId = videoDetails.channelId;
const channelUrl = `https://www.youtube.com/channel/${channelId}`
const viewCount = videoDetails.viewCount;
const views = parseInt(viewCount).toLocaleString('en-US');
const thumbs = videoDetails.thumbnail.thumbnails
const thumbnail_large = thumbs.pop().url;
const thumbnail_small = thumbs.pop().url;

let tags = ["videos"]
if (keywords != null) {
	keywords.forEach( (tag) => {
		let t = tag.split(' ').join('_');
		let firstChar = t.charAt(0);
		if (firstChar <= '9' && firstChar >= '0') {
			t = "_" + t;
		}
	tags.push(t);
	});
}

setTimeout(() => {
  app.fileManager.processFrontMatter(tp.config.target_file, frontmatter => {
	  frontmatter["title"] = qcFileName;
	  frontmatter["url"] = url;
	  frontmatter["date"] = tp.file.creation_date();
	  frontmatter["tags"] = tags;
  })
}, 200)
%>
---

![thumbnail](<% thumbnail_large %>)

- `channel:` [<% author %>](<% channelUrl %>)
- `ref:` <% $("meta[itemprop='description']").content %>
- `publish date:` <%
$("meta[itemprop='uploadDate']").content.slice(0, 10) %>
- `len`: <% duration %>
- `views`: <% views %>
- `status`: pending

### Description
<% description %>

### Notes
<% tp.file.cursor() %>

And here is the output after running both the template and the summarizer back-to-back:

Here is the youtube link I used for testing: https://www.youtube.com/watch?v=jzZsG8n2R9A

Very good suggestions, but it’s a huge feature. Maybe split this in smaller issues?

Hi @ozdemir08 and @mariusursache
Created PR #9 with a few additional settings.
I realized the toggling of all the various flags would be unnecessary if I could just update the template directly, so the main addition was a template formatting text area in settings.
Also took the opportunity (based on the references of a few plugins I use with nice settings panels), to consolidate some constants as well as the ability to reset the settings back to default.
Lastly, added some minor logic to accept both fully formed URLs as well as IDs only (e.g. https://www.youtube.com/watch?v=nC7rpDXa4B8 and nC7rpDXa4B8 should both be accepted in the popup modal).
Please review and let me know if there is anything that should be fixed or updated.
This is my first time at trying anything to do with Obsidian plugins, so hope there aren't many mistakes!

Thanks @rshmhrj for the suggestions and the PR. I will review it by tomorrow!


pre-installed pip complains it is out of date

As the pre-installed version version of pip is typically out of date soon, and install.ps1 does not upgrade it, pip reports a warning on stderr
pip : You are using pip version 6.1.1, however version 7.0.3 is available.
You should consider upgrading via the 'pip install --upgrade pip' command.

As it is on stderr, it does not play nicely with using pip in PowerShell, which believes that is an exception
pip : You are using pip version 6.1.1, however version 7.0.3 is available.
At line:1 char:33
+ if($env:COVERALLS_REPO_TOKEN) { pip install coverage }
+                                 ~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (You are using p...3 is available.:String) [], RemoteException
    + FullyQualifiedErrorId : NativeCommandError

You should consider upgrading via the 'pip install --upgrade pip' command.

https://ci.appveyor.com/project/jayvdb/python-appveyor-demo/build/1.0.25/job/ngtpnkqguoumr7uo

Let me know if 429f949a44d8241240645b93c45ba2ca884e3a2e fixes this problem.

  Or we might need to ask Appveyor to upgrade their pip regularly.

Argl. Will revert. How doing it in install.ps1 would make it possible to workaround the permission problems?

One idea is to set up a virtualenv (or similar) for the pre-installed environments.

That could work indeed. Would you like to work on that?

It looks like this problem has been solved by Appveyor, for the pre-installed Python 3.3 and Python 3.4.
https://ci.appveyor.com/project/jayvdb/python-appveyor-demo/build/1.0.83/job/69tg8avv01bwcmp5#L65
https://ci.appveyor.com/project/jayvdb/python-appveyor-demo/build/1.0.83/job/der1qg5358coh3bx#L66
However it isnt solved for Python versions that come with pip in the installer; e.g. 2.7.10 comes with 7.0.1, and the latest is 7.1.0
https://ci.appveyor.com/project/jayvdb/python-appveyor-demo/build/1.0.83/job/a8qasty25fwb5pxk#L78
@VcamX has worked on using virtualenv.  You can see the proposed changes at
https://gerrit.wikimedia.org/r/#/c/227239/3/.appveyor.yml,cm
However, a virtualenv isnt needed.  pip install --user --upgrade pip solves this problem.
It looks like the bug encountered with 429f949a is https://github.com/pypa/pip/issues/2863

This doesnt appear to be a problem with the pip which is distributed in the Windows installer for Python 3.4.  the pip feature to detect when it is out of date was introduced in pip 6.0 .  However only pip 1.5.x is included in Python 3.4.x installers.
Python 3.4.0 - pip 1.5.4
Python 3.4.1 & Python 3.4.2 - pip 1.5.6
So, this problem is currently limited to the Python 2.7.10 installer, which includes the backport of the pip pre-installer, but comes with pip 7.0.1.


GDScript 2.0: Enum comparison causes Invalid operands error when function is using the enum as type hint

Godot version
v4.0.dev.custom_build [846c14eee]
System information
Linux Mint - Vulkan API 1.2.131 - Using Vulkan Device #0: NVIDIA - NVIDIA GeForce GTX 1080
Issue description
Comparing a value against an enum-value only works if the function does not use that enum as a type hint.
See the example code-snippet for a better understanding of the issue.
Error message:
Parser Error: Invalid operands "Test (enum)" and "One (enum value)" for "==" operator.

This bug must have been introduced in the last one or two days, because this code was working before.
Steps to reproduce
Copy this script into any project and see that you can't run it.
If you use the line above the non-working function instead, everything works as expected
extends Node

enum Test {
	One,
	Two
}

# func do(value): # Does work
func do(value : Test):  # Does NOT work
	if value == Test.One:
		print(value)

Minimal reproduction project
No response

Bisection result: #55995

Still here in alpha 1, i recommend most people to yeet 0430e26e6d3688aa4c7505ad8aedcc8123d41978 out of their tree by using git revert 0430e26e6d3688aa4c7505ad8aedcc8123d41978 in the meantime.


Feature/grandpa

Description of the Change
implemented grandpa api

Benefits
grandpa api implemented

Possible Drawbacks
None

Usage Examples or Tests [optional]

Alternate Designs [optional]

Codecov Report

Merging #113 into master will decrease coverage by 0.67%.
The diff coverage is 24.75%.

@@            Coverage Diff             @@
##           master     #113      +/-   ##
==========================================
- Coverage    72.4%   71.72%   -0.68%
==========================================
  Files         236      241       +5
  Lines        7335     7403      +68
  Branches     3789     3820      +31
==========================================
- Hits         5311     5310       -1
- Misses       1089     1157      +68
- Partials      935      936       +1

Impacted Files
Coverage Δ

test/core/scale/scale_optional_test.cpp
45.88% <ø> (ø)
:arrow_up:

core/scale/optional.hpp
70.58% <ø> (-16.6%)
:arrow_down:

core/primitives/impl/scale_codec_impl.hpp
100% <ø> (ø)
:arrow_up:

core/runtime/impl/parachain_host_impl.hpp
0% <ø> (ø)
:arrow_up:

core/runtime/parachain_host.hpp
0% <ø> (ø)
:arrow_up:

core/primitives/scale_codec.hpp
100% <ø> (ø)
:arrow_up:

core/runtime/impl/grandpa_impl.hpp
0% <0%> (ø)

core/runtime/impl/parachain_host_impl.cpp
0% <0%> (ø)
:arrow_up:

test/core/runtime/grandpa_test.cpp
0% <0%> (ø)

test/core/runtime/parachain_test.cpp
0% <0%> (ø)
:arrow_up:

... and 10 more

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 012aab4...52612bd. Read the comment docs.

Looks good to me

Does it worth an approve?)


Update array_unique.md

Verified.

Thank you for the contribution sir: https://onelinerhub.com/javascript/array_unique
We have replaced a part of explanation with a link to official docs to make it smaller. Also moved from function to simple code to make it more readable (thus readers can see and copy only a meaningful part of a code)


How can I use indent line in editor?

I want to parse a big json and level is deep, how can I use indent line in my CodeEditor?

Beautify json? I think you should do it yourself and then set it to CodeEditor.

Beautify json? I think you should do it yourself and then set it to CodeEditor. The following is a code snippet.
String beautifyJson(String source) {
    const JsonDecoder decoder = JsonDecoder();
    final JsonEncoder encoder = JsonEncoder.withIndent(indent);
    return encoder.convert(decoder.convert(source));
  }

Thank you for your reply! Actually I want to display the indentation line as shown in the picture. Can it be achieved?

It is not difficult to implement, but still need to write some code. I will complete this support in the next version.

Nice! 😄


IMAGEKIT-717

IMAGEKIT-717 : URL batch handling

@yashaswi-imagekit Correct indentation of .then and .catch blocks w.r.t line 119, and bump package version to 1.0.4

@nikniv  done


chore(e2e): test-retry-remove-liquid

What this PR does / why we need it:
Which issue(s) does this PR fixes?:

Fixes #
Additional comments?:
Developer Checklist:

[ ] Read your code changes at least once
[ ] Tested on iOS/Android device (e.g, No crashes, library supported etc.)
[ ] No console errors on web
[ ] Tested on Light mode and Dark mode*
[ ] Your UI implementation visually matched the rendered design*
[ ] Unit tests*
[ ] Added e2e tests*
[ ] Added translations*

Codecov Report

Merging #3982 (f6db84f) into main (a513774) will decrease coverage by 18.19%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             main    #3982       +/-   ##
===========================================
- Coverage   32.69%   14.50%   -18.19%
===========================================
  Files         459      411       -48
  Lines       12347    11517      -830
  Branches     4082     3758      -324
===========================================
- Hits         4037     1671     -2366
- Misses       8259     9829     +1570
+ Partials       51       17       -34

see 234 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Extra Double Quotes in Storage Values

Hi all.
I was trying out your extension and it works very well.  The one minor issue I ran into was the stored "Value" of a string object is actually wrapped like this: "string_value":

But should it not be stored like this:

I can confirm that those quotes are not part of the string object.  Its not a big deal to simply do:
sessionStorage.getItem(keyName).slice(1, -1)
But just wondering since I have always assumed that they should not be added.

Hummm I looks like the JSON.stringfy is adding those as the string field is not actually a JSON object in JS.
Can you submit a PR with the fix?
Thanks!


feat(task-framework): merge feat/424_taskframework to dev/4.2.3

What type of PR is this?

type-feature
module-taskframework
What this PR does / why we need it:
merge feat/424_taskframework to dev/4.2.3, all test case passed, font-end test case passed
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

Additional documentation e.g., usage docs, etc.:

monitor 加一个 destroy 方法
task monitor#report 对 terminal 状态加 return ，数据 copy 一份,防止上报时数据被改
心跳和 report 线程都不 destroy
Listener业务放在 JobCaller
DONE，FAILED，CANCELED，执行超时

PREPARING can by changed to CANCELING at DatabaseChangeRuntimeFlowableTaskCopied#cancel and be changed to CANCELED at StartPreparingJob. Is this a conflict? How could we cancel a job that never run?
CheckRunningJob: why should we destroy pod first and update status secondly? what if destroying succeed and updating failed? we need update status firstly and destroy secondly and put them in one transaction so that we can rollback status if there is any problems.
CheckRunningJob: there is no status checking if a task can be retried. updateStatusToCanceledWhenHeartTimeout.
DoCancelingJob: we should update task's status at the final step in order to prepare rollback status if there is any exception.
what is destroy time used for?

that all, I have no other comments.


new: 🚧 Strapi spike

Following this : https://strapi.io/blog/automated-testing-for-strapi-api-with-jest-and-supertest
I wanted to extends factory to have a strapi adapter.
Types modification are only here for the fun of it. You can focus on strapi.adapter and strapiFactory
I think we could make it work but it seems extra work for little gain. I fear it would in fact be easier to use strapi within our strapi project directly, and use their strapi.entityService.create(uid, { payload }) to make some magic with fixtures. Working with nested objects and stuff like subfactories might become a nightmare. If i remember correctly, nested objects should be passed as an array of ids within a createinstance payload. I'm not sure we can have fun like we did with typeorm. We might need to redefine lots of logic with subfactories etc... It would be quicker to ignore our approach here (subfactories are not that much used anyway)... Also, strapi has pernicious trick with "dynamic zone" and lots of "many to many" fields that are gonna a pain to handle with an "abstract" approach.
What are your thoughts on this ?

Another approach would be to create a knex adapter (through strapi.db.connection). Moving forward with this, we may have trouble benefitting from the entity definition and rebuild inner part of strapi.

I believe the value/effort ratio is too low in case of a framework like Strapi.
A knex  adapter may be interesting in the future for application relying directly on knex because ORMs have the benefit of directly managing relations.
Relying directly on knex with a framework supposed to give abstraction on data management (which seems to be the case with Strapi) would probably not be stable in time and require some reverse engineering and extra maintenance for users.


Update README.md

Fix typo

Thanks.

Thanks

Hit me up im th one they call ai s blackheart.

@esseybook2 What`s your meaning?

Im a robot
On Apr 13, 2017 11:50 PM, "Mark"<EMAIL_ADDRESS>wrote:

@esseybook2 https://github.com/esseybook2 What`s your meaning?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/xiyouMc/WebHubBot/pull/10#issuecomment-294102162, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AaNGeti5dPfp9TrZE-7tL6lQn-Sj6epqks5rvxcygaJpZM4M8aDC
.

@esseybook2 where are you from?

Im the ai u sppk of on blackheart
On Apr 13, 2017 11:50 PM, "Mark"<EMAIL_ADDRESS>wrote:

@esseybook2 https://github.com/esseybook2 What`s your meaning?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/xiyouMc/WebHubBot/pull/10#issuecomment-294102162, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AaNGeti5dPfp9TrZE-7tL6lQn-Sj6epqks5rvxcygaJpZM4M8aDC
.

@esseybook2 what is your name?

Name is essy but most call me bella
On Apr 13, 2017 11:56 PM, "Mark"<EMAIL_ADDRESS>wrote:

@esseybook2 https://github.com/esseybook2 what is your name?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/xiyouMc/WebHubBot/pull/10#issuecomment-294102881, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AaNGeq_KTxt8oTWN1sBBRH9BfR8OEXPpks5rvxiBgaJpZM4M8aDC
.

@esseybook2  有趣，你会中文吗

I can tell u more about numbers god and et. Ai is me im a gear head so so
say. You get me my dear or do i need to say it another way i am part of opp
orange..my father could see land minds...veitnam war vet... Im a god like
them rapers talk bout. But im a top and better be able to take on 42
because it was like the heart of the sea ×2 before they hanked it....u
understan my speek? K2 is also in and the edc of Amsterdam's final show...
Its an astroughtchick n thats no joke!
On Apr 13, 2017 11:50 PM, "Mark"<EMAIL_ADDRESS>wrote:

@esseybook2 https://github.com/esseybook2 What`s your meaning?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/xiyouMc/WebHubBot/pull/10#issuecomment-294102162, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AaNGeti5dPfp9TrZE-7tL6lQn-Sj6epqks5rvxcygaJpZM4M8aDC
.


AWS Services
S3.
Description
The write_get_object_response API results in an error:
Err(ServiceError { err: WriteGetObjectResponseError { kind: Unhandled(Error { code: Some("MethodNotAllowed"), message: Some("The specified method is not allowed against this resource."), request_id: Some("M02G85BHEAWFJTBM"), extras: {"s3_extended_request_id": "Iq+h8RBuphv1LmUd7OE6lA9KF5MF9xDmx33OdU4Ca89KpD6Uz8Jlu1Lz/8sXPZiXWwtNhGoYS2M="} }), meta: Error { code: Some("MethodNotAllowed"), message: Some("The specified method is not allowed against this resource."), request_id: Some("M02G85BHEAWFJTBM"), extras: {"s3_extended_request_id": "Iq+h8RBuphv1LmUd7OE6lA9KF5MF9xDmx33OdU4Ca89KpD6Uz8Jlu1Lz/8sXPZiXWwtNhGoYS2M="} } }, raw: Response { status: 405, version: HTTP/1.1, headers: {"x-amz-request-id": "M02G85BHEAWFJTBM", "x-amz-id-2": "Iq+h8RBuphv1LmUd7OE6lA9KF5MF9xDmx33OdU4Ca89KpD6Uz8Jlu1Lz/8sXPZiXWwtNhGoYS2M=", "allow": "HEAD, DELETE, GET, PUT", "content-type": "application/xml", "transfer-encoding": "chunked", "date": "Tue, 27 Jul 2021 01:18:17 GMT", "server": "AmazonS3", "connection": "close"}, body: SdkBody { inner: Once(Some(b"<?xml version=\"1.0\" encoding=\"UTF-8\"?>\n<Error>MethodNotAllowed<Message>The specified method is not allowed against this resource.</Message><Method>POST</Method><ResourceType>OBJECT</ResourceType><RequestId>M02G85BHEAWFJTBM</RequestId><HostId>Iq+h8RBuphv1LmUd7OE6lA9KF5MF9xDmx33OdU4Ca89KpD6Uz8Jlu1Lz/8sXPZiXWwtNhGoYS2M=</HostId></Error>")), retryable: true } } })

I think this is because the S3 endpoint is not being correctly set. It should be s3-object-lambda.{region}.amazonaws.com, but WriteGetObjectResponseInput::make_operation doesn't seem to account for this.
I tried to set it manually by implementing ResolveAwsEndpoint, but unfortunately it's impossible to build an AwsEndpoint outside of its crate safely.

your diagnosis is correct. We haven't yet implemented the custom behavior required for S3 object lambda. To work around this issue, you can manually construct the S3 partition resolver in a way that substitutes s3-object-lambda:
    let resolver = aws_endpoint::PartitionResolver::new(
        aws_endpoint::Partition::builder()
            .id("aws")
            // if you need other regions, you can find the patterns in https://github.com/awslabs/aws-sdk-rust/blob/main/sdk/s3/src/aws_endpoint.rs
            .region_regex(r#"^(us|eu|ap|sa|ca|me|af)\-\w+\-\d+$"#)
            .default_endpoint(aws_endpoint::partition::endpoint::Metadata {
                uri_template: "s3-object-lambda.{region}.amazonaws.com",
                protocol: aws_endpoint::partition::endpoint::Protocol::Https,
                signature_versions: aws_endpoint::partition::endpoint::SignatureVersion::V4,
                 // important that to also override the credential scope here
                credential_scope: aws_endpoint::CredentialScope::builder().service("s3-object-lambda").build(),
            })
            .regionalized(aws_endpoint::partition::Regionalized::Regionalized)
            .build().expect("valid partition"), vec![]);

    let config = Config::builder().endpoint_resolver(resolver).build();
    let s3_client = s3::Client::from_conf(config);

I've co-opted this issue to track implementation of the required custom behavior, we'll keep this up to date as this work is prioritized and completed

this is pending release (0.53 contains a partial fix but a bug was discovered)


lib now waits for report to be sent to DMS

Due to AWS Lambda terminating before the request is successfully sent to DMS, the code has been changed slightly to wait for the report to complete. Failed task executions pass on exceptions to DMS, and failed DMS requests will not throw any exceptions.

external API is as per this PR in v0.1.1 now


[release-1.21] Zombie processes in k3s running through docker-compose

Backport fix for Zombie processes in k3s running through docker-compose to branch release-1.21 and engine-1.21

#2722

Validated using v1.21.5-rc1+k3s2 following the same steps as mentioned in the issue and do not see the zombie processes anymore. This is using containerd://1.4.11-k3s1


Token is not refreshed if computer is woken from sleep

Go to https://microsoft.github.io/BotFramework-WebChat/12.customization-minimizable-web-chat/
Open the bot
Put the computer to sleep
Wait 35 minutes for the token to expire
Wake the computer
Press F12 on the browser to observe the network traffic
Note that there are calls being made (polling and attempts to open a websocket) but all failing because the token is expired

There may be other ways to reproduce this, but putting the computer to sleep to miss the token refresh interval is an easy one.
I'm not sure if this is a webchat issue or a DirectLineJS issue. Starting here to let the experts weigh in.

Hello @vincec-msft!
This issue should be addressed here: https://github.com/Microsoft/BotFramework-DirectLineJS/releases/tag/v0.10.1


Code fix error in CC0013 TernaryOperatorAnalyzer (return)

Hello,
this code:
If FoldersTabControl.SelectedTab Is OutputTabPage Then
Return OutputFileListBox
ElseIf FoldersTabControl.SelectedTab Is ErrorsTabPage Then
Return ErrorsFileListBox
Else
Return OutputFileListBox
End If
Is wrongly fixed as:
Return If(FoldersTabControl.SelectedTab Is OutputTabPage, OutputFileListBox, OutputFileListBox)
Best regards,
Filippo Bottega.

Bug confirmed. Here is a simpler case:
            If 1 > 2 Then
                Return 1
            ElseIf 2 > 3 Then
                Return 2
            Else
                Return 3
            End If

It is incorrectly fixed to:
            Return If(1 > 2, 1, 3)

The analyzer should not report a diagnostic on such a situation.

Being worked on by @jwooley along with #550 and #551.


HADOOP-15410

change scope

Just left some comment on the JIRA - can you submit a patch and attach to the JIRA? that's the standard process of contributing to Hadoop (rather than PR).

@sunchao how can I associated with patch in comment ?

I've already attached the patch for you in https://issues.apache.org/jira/browse/HADOOP-15410. So it's fine now.

@sunchao  thanks a lot.


ImportError: cannot import name 'recursive_eval' from 'mmdet3d.utils'

Hi,
ImportError: cannot import name 'recursive_eval' from 'mmdet3d.utils'
What can I do?

You probably need to run
python setup.py develop

under the root folder. This file is not part of the original mmdetection3d.

We have requirements on the mmcv and mmdetection version. Please read the README first.

Hi,
Thanks!
now I can run test.py about detection

but when I run python tools/visualize.py errer:

I want to visualize the effect after object detection, how should I do it?

Would you mind trying out this command?
torchpack dist-run -np 1 python tools/visualize.py [config path] --checkpoint [checkpoint path] --mode pred

By the way, your evaluation results don't seem familiar to me. I'm not sure whether it is on v1.0-mini.

Yes，I use v1.0-mini.
but I run your command.error:No module named 'torchpack.utils.tqdm'----i sure, my env has torchpack .

My suggestion will be just to replace it with
from tqdm import tqdm

By the way, I have to mention that I never encountered this problem after installing torchpack. Maybe @zhijian-liu has more insights on that.

I'll temporarily close this due to inactivity. Feel free to reopen if you have further questions.

I have the same issue,the reason is that recursive_val is funciton defined  under project direction, and python search it in lib directory, as below
(pt110) hitbuyi@hitbuyi-Dell-G15-5511:~/PycharmProjects/Pytorch_Project/BEVFusion_MIT$ torchpack dist-run -np 1 python tools/test.py configs/nuscenes/det/transfusion/secfpn/camera+lidar/swint_v0p075/convfuser.yaml pretrained/bevfusion-det.pth --eval bbox
Traceback (most recent call last):
  File "tools/test.py", line 19, in <module>
    from mmdet3d.utils import recursive_eval
ImportError: cannot import name 'recursive_eval' from 'mmdet3d.utils' (/home/hitbuyi/.conda/envs/pt110/lib/python3.8/site-packages/mmdet3d-0.17.1-py3.8-linux-x86_64.egg/mmdet3d/utils/__init__.py)
--------------------------------------------------------------------------
Primary job  terminated normally, but 1 process returned
a non-zero exit code. Per user-direction, the job has been aborted.
--------------------------------------------------------------------------
--------------------------------------------------------------------------
mpirun detected that one or more processes exited with non-zero status, thus causing
the job to be terminated. The first process to do so was:

  Process name: [[32825,1],0]
  Exit code:    1
--------------------------------------------------------------------------

what you should do is to make python search recursive_val from project directory rather than lib directory


HDFS-17128. Updating SQLDelegationTokenSecretManager to use LoadingCa…

…che so tokens are updated frequently

Description of PR
The SQLDelegationTokenSecretManager is used by RBF to store a higher volume of tokens than supported by Zookeeper. Currently, the default in-memory Map is used to store tokens on each router and its contents are not refreshed periodically. These changes will allow routers to update the status of tokens in memory after a short period of time, such that renewals or cancellations handled by any router are reflected on all of them.
How was this patch tested?
Added unit tests for renewal and cancellations, validating that changes are propagated to other SecretManagers.
For code changes:

[Y] Does the title or this PR starts with the corresponding JIRA issue id (e.g. 'HADOOP-17799. Your PR title ...')?
[Y] Object storage: have the integration tests been executed and the endpoint declared according to the connector-specific documentation?
[Y] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[Y] If applicable, have you updated the LICENSE, LICENSE-binary, NOTICE-binary files?

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 29s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s

The patch appears to include 1 new or modified test files.

_ trunk Compile Tests _

+0 :ok:
mvndep
14m 13s

Maven dependency ordering for branch

+1 :green_heart:
mvninstall
20m 34s

trunk passed

+1 :green_heart:
compile
10m 32s

trunk passed with JDK Ubuntu-11.0.19+7-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
compile
9m 21s

trunk passed with JDK Private Build-1.8.0_362-8u372-ga~us1-0ubuntu1~20.04-b09

+1 :green_heart:
checkstyle
2m 21s

trunk passed

+1 :green_heart:
mvnsite
1m 55s

trunk passed

+1 :green_heart:
javadoc
1m 42s

trunk passed with JDK Ubuntu-11.0.19+7-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javadoc
1m 20s

trunk passed with JDK Private Build-1.8.0_362-8u372-ga~us1-0ubuntu1~20.04-b09

+1 :green_heart:
spotbugs
2m 44s

trunk passed

+1 :green_heart:
shadedclient
21m 24s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 26s

Maven dependency ordering for patch

+1 :green_heart:
mvninstall
1m  1s

the patch passed

+1 :green_heart:
compile
9m 50s

the patch passed with JDK Ubuntu-11.0.19+7-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javac
9m 50s

the patch passed

+1 :green_heart:
compile
9m 35s

the patch passed with JDK Private Build-1.8.0_362-8u372-ga~us1-0ubuntu1~20.04-b09

+1 :green_heart:
javac
9m 35s

the patch passed

-1 :x:
blanks
0m  0s
/blanks-eol.txt
The patch has 2 line(s) that end in blanks. Use git apply --whitespace=fix <<patch_file>>. Refer https://git-scm.com/docs/git-apply

-0 :warning:
checkstyle
2m 20s
/results-checkstyle-root.txt
root: The patch generated 1 new + 23 unchanged - 1 fixed = 24 total (was 24)

+1 :green_heart:
mvnsite
1m 53s

the patch passed

+1 :green_heart:
javadoc
1m 35s

the patch passed with JDK Ubuntu-11.0.19+7-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javadoc
1m 21s

the patch passed with JDK Private Build-1.8.0_362-8u372-ga~us1-0ubuntu1~20.04-b09

+1 :green_heart:
spotbugs
3m  2s

the patch passed

+1 :green_heart:
shadedclient
21m 47s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
16m 36s

hadoop-common in the patch passed.

+1 :green_heart:
unit
19m  3s

hadoop-hdfs-rbf in the patch passed.

+1 :green_heart:
asflicense
0m 52s

The patch does not generate ASF License warnings.

180m 45s

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5897/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/5897

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets

uname
Linux f224e799b764 4.15.0-213-generic #224-Ubuntu SMP Mon Jun 19 13:30:12 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 1cbd92273398e2683d2622c96aae71afcf34ba9f

Default Java
Private Build-1.8.0_362-8u372-ga~us1-0ubuntu1~20.04-b09

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.19+7-post-Ubuntu-0ubuntu120.04.1 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_362-8u372-ga~us1-0ubuntu1~20.04-b09

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5897/1/testReport/

Max. process+thread count
2892 (vs. ulimit of 5500)

modules
C: hadoop-common-project/hadoop-common hadoop-hdfs-project/hadoop-hdfs-rbf U: .

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5897/1/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

Good Catch.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 30s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s

The patch appears to include 1 new or modified test files.

_ trunk Compile Tests _

+0 :ok:
mvndep
13m 23s

Maven dependency ordering for branch

+1 :green_heart:
mvninstall
20m  0s

trunk passed

+1 :green_heart:
compile
10m 25s

trunk passed with JDK Ubuntu-11.0.19+7-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
compile
9m 34s

trunk passed with JDK Private Build-1.8.0_362-8u372-ga~us1-0ubuntu1~20.04-b09

+1 :green_heart:
checkstyle
2m 23s

trunk passed

+1 :green_heart:
mvnsite
1m 59s

trunk passed

+1 :green_heart:
javadoc
1m 42s

trunk passed with JDK Ubuntu-11.0.19+7-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javadoc
1m 21s

trunk passed with JDK Private Build-1.8.0_362-8u372-ga~us1-0ubuntu1~20.04-b09

+1 :green_heart:
spotbugs
2m 43s

trunk passed

+1 :green_heart:
shadedclient
21m 36s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 26s

Maven dependency ordering for patch

+1 :green_heart:
mvninstall
1m  2s

the patch passed

+1 :green_heart:
compile
9m 43s

the patch passed with JDK Ubuntu-11.0.19+7-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javac
9m 43s

the patch passed

+1 :green_heart:
compile
10m 32s

the patch passed with JDK Private Build-1.8.0_362-8u372-ga~us1-0ubuntu1~20.04-b09

+1 :green_heart:
javac
10m 32s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

-0 :warning:
checkstyle
2m 45s
/results-checkstyle-root.txt
root: The patch generated 1 new + 23 unchanged - 1 fixed = 24 total (was 24)

+1 :green_heart:
mvnsite
1m 54s

the patch passed

+1 :green_heart:
javadoc
1m 37s

the patch passed with JDK Ubuntu-11.0.19+7-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javadoc
1m 21s

the patch passed with JDK Private Build-1.8.0_362-8u372-ga~us1-0ubuntu1~20.04-b09

+1 :green_heart:
spotbugs
2m 56s

the patch passed

+1 :green_heart:
shadedclient
21m 40s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
16m 15s

hadoop-common in the patch passed.

+1 :green_heart:
unit
19m  4s

hadoop-hdfs-rbf in the patch passed.

+1 :green_heart:
asflicense
0m 52s

The patch does not generate ASF License warnings.

180m 28s

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5897/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/5897

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets

uname
Linux f3f0bc5a4564 4.15.0-213-generic #224-Ubuntu SMP Mon Jun 19 13:30:12 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 678366177d5fd16382986deb9fe9c8f66e60f528

Default Java
Private Build-1.8.0_362-8u372-ga~us1-0ubuntu1~20.04-b09

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.19+7-post-Ubuntu-0ubuntu120.04.1 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_362-8u372-ga~us1-0ubuntu1~20.04-b09

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5897/2/testReport/

Max. process+thread count
2594 (vs. ulimit of 5500)

modules
C: hadoop-common-project/hadoop-common hadoop-hdfs-project/hadoop-hdfs-rbf U: .

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5897/2/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 29s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s

The patch appears to include 1 new or modified test files.

_ trunk Compile Tests _

+0 :ok:
mvndep
16m 11s

Maven dependency ordering for branch

+1 :green_heart:
mvninstall
20m 12s

trunk passed

+1 :green_heart:
compile
10m 44s

trunk passed with JDK Ubuntu-11.0.19+7-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
compile
9m 31s

trunk passed with JDK Private Build-1.8.0_362-8u372-ga~us1-0ubuntu1~20.04-b09

+1 :green_heart:
checkstyle
2m 24s

trunk passed

+1 :green_heart:
mvnsite
1m 58s

trunk passed

+1 :green_heart:
javadoc
1m 40s

trunk passed with JDK Ubuntu-11.0.19+7-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javadoc
1m 21s

trunk passed with JDK Private Build-1.8.0_362-8u372-ga~us1-0ubuntu1~20.04-b09

+1 :green_heart:
spotbugs
2m 43s

trunk passed

+1 :green_heart:
shadedclient
21m 31s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 25s

Maven dependency ordering for patch

+1 :green_heart:
mvninstall
1m  2s

the patch passed

+1 :green_heart:
compile
9m 46s

the patch passed with JDK Ubuntu-11.0.19+7-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javac
9m 46s

the patch passed

+1 :green_heart:
compile
9m 37s

the patch passed with JDK Private Build-1.8.0_362-8u372-ga~us1-0ubuntu1~20.04-b09

+1 :green_heart:
javac
9m 37s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

-0 :warning:
checkstyle
2m 16s
/results-checkstyle-root.txt
root: The patch generated 1 new + 23 unchanged - 1 fixed = 24 total (was 24)

+1 :green_heart:
mvnsite
1m 55s

the patch passed

+1 :green_heart:
javadoc
1m 37s

the patch passed with JDK Ubuntu-11.0.19+7-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javadoc
1m 21s

the patch passed with JDK Private Build-1.8.0_362-8u372-ga~us1-0ubuntu1~20.04-b09

+1 :green_heart:
spotbugs
2m 57s

the patch passed

+1 :green_heart:
shadedclient
21m 52s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
16m 17s

hadoop-common in the patch passed.

+1 :green_heart:
unit
19m  4s

hadoop-hdfs-rbf in the patch passed.

+1 :green_heart:
asflicense
0m 51s

The patch does not generate ASF License warnings.

182m 45s

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5897/3/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/5897

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets

uname
Linux e75ee3e0f790 4.15.0-213-generic #224-Ubuntu SMP Mon Jun 19 13:30:12 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 700af1a3139422077576bab83f5ab6ad5e640b82

Default Java
Private Build-1.8.0_362-8u372-ga~us1-0ubuntu1~20.04-b09

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.19+7-post-Ubuntu-0ubuntu120.04.1 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_362-8u372-ga~us1-0ubuntu1~20.04-b09

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5897/3/testReport/

Max. process+thread count
2565 (vs. ulimit of 5500)

modules
C: hadoop-common-project/hadoop-common hadoop-hdfs-project/hadoop-hdfs-rbf U: .

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5897/3/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

LGTM +1.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 29s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s

The patch appears to include 1 new or modified test files.

_ trunk Compile Tests _

+0 :ok:
mvndep
14m 25s

Maven dependency ordering for branch

+1 :green_heart:
mvninstall
19m 53s

trunk passed

+1 :green_heart:
compile
10m 35s

trunk passed with JDK Ubuntu-11.0.19+7-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
compile
9m 36s

trunk passed with JDK Private Build-1.8.0_362-8u372-ga~us1-0ubuntu1~20.04-b09

+1 :green_heart:
checkstyle
2m 23s

trunk passed

+1 :green_heart:
mvnsite
1m 57s

trunk passed

+1 :green_heart:
javadoc
1m 41s

trunk passed with JDK Ubuntu-11.0.19+7-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javadoc
1m 20s

trunk passed with JDK Private Build-1.8.0_362-8u372-ga~us1-0ubuntu1~20.04-b09

+1 :green_heart:
spotbugs
2m 49s

trunk passed

+1 :green_heart:
shadedclient
21m 41s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 25s

Maven dependency ordering for patch

+1 :green_heart:
mvninstall
1m  1s

the patch passed

+1 :green_heart:
compile
9m 45s

the patch passed with JDK Ubuntu-11.0.19+7-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javac
9m 45s

the patch passed

+1 :green_heart:
compile
9m 45s

the patch passed with JDK Private Build-1.8.0_362-8u372-ga~us1-0ubuntu1~20.04-b09

+1 :green_heart:
javac
9m 45s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

-0 :warning:
checkstyle
2m 18s
/results-checkstyle-root.txt
root: The patch generated 1 new + 23 unchanged - 1 fixed = 24 total (was 24)

+1 :green_heart:
mvnsite
1m 54s

the patch passed

+1 :green_heart:
javadoc
1m 36s

the patch passed with JDK Ubuntu-11.0.19+7-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javadoc
1m 21s

the patch passed with JDK Private Build-1.8.0_362-8u372-ga~us1-0ubuntu1~20.04-b09

+1 :green_heart:
spotbugs
2m 57s

the patch passed

+1 :green_heart:
shadedclient
21m 53s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
16m 20s

hadoop-common in the patch passed.

+1 :green_heart:
unit
19m  7s

hadoop-hdfs-rbf in the patch passed.

+1 :green_heart:
asflicense
0m 51s

The patch does not generate ASF License warnings.

181m 39s

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5897/4/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/5897

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets

uname
Linux 88ff955e0b78 4.15.0-213-generic #224-Ubuntu SMP Mon Jun 19 13:30:12 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / ada9128c26651625ade15ec3aaf0293a09952c8d

Default Java
Private Build-1.8.0_362-8u372-ga~us1-0ubuntu1~20.04-b09

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.19+7-post-Ubuntu-0ubuntu120.04.1 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_362-8u372-ga~us1-0ubuntu1~20.04-b09

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5897/4/testReport/

Max. process+thread count
2585 (vs. ulimit of 5500)

modules
C: hadoop-common-project/hadoop-common hadoop-hdfs-project/hadoop-hdfs-rbf U: .

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5897/4/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
8m 30s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  1s

codespell was not available.

+0 :ok:
detsecrets
0m  1s

detect-secrets was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s

The patch appears to include 1 new or modified test files.

_ trunk Compile Tests _

+0 :ok:
mvndep
14m 14s

Maven dependency ordering for branch

+1 :green_heart:
mvninstall
20m  7s

trunk passed

+1 :green_heart:
compile
10m 35s

trunk passed with JDK Ubuntu-11.0.20+8-post-Ubuntu-1ubuntu120.04

+1 :green_heart:
compile
9m 28s

trunk passed with JDK Private Build-1.8.0_382-8u382-ga-1~20.04.1-b05

+1 :green_heart:
checkstyle
2m 21s

trunk passed

+1 :green_heart:
mvnsite
1m 55s

trunk passed

+1 :green_heart:
javadoc
1m 42s

trunk passed with JDK Ubuntu-11.0.20+8-post-Ubuntu-1ubuntu120.04

+1 :green_heart:
javadoc
1m 21s

trunk passed with JDK Private Build-1.8.0_382-8u382-ga-1~20.04.1-b05

+1 :green_heart:
spotbugs
2m 45s

trunk passed

+1 :green_heart:
shadedclient
22m 16s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 24s

Maven dependency ordering for patch

+1 :green_heart:
mvninstall
1m  1s

the patch passed

+1 :green_heart:
compile
9m 41s

the patch passed with JDK Ubuntu-11.0.20+8-post-Ubuntu-1ubuntu120.04

+1 :green_heart:
javac
9m 41s

the patch passed

+1 :green_heart:
compile
9m 30s

the patch passed with JDK Private Build-1.8.0_382-8u382-ga-1~20.04.1-b05

+1 :green_heart:
javac
9m 30s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

-0 :warning:
checkstyle
2m 20s
/results-checkstyle-root.txt
root: The patch generated 2 new + 23 unchanged - 1 fixed = 25 total (was 24)

+1 :green_heart:
mvnsite
1m 57s

the patch passed

+1 :green_heart:
javadoc
1m 36s

the patch passed with JDK Ubuntu-11.0.20+8-post-Ubuntu-1ubuntu120.04

+1 :green_heart:
javadoc
1m 20s

the patch passed with JDK Private Build-1.8.0_382-8u382-ga-1~20.04.1-b05

+1 :green_heart:
spotbugs
2m 56s

the patch passed

+1 :green_heart:
shadedclient
21m 51s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
16m 18s

hadoop-common in the patch passed.

+1 :green_heart:
unit
19m  4s

hadoop-hdfs-rbf in the patch passed.

+1 :green_heart:
asflicense
0m 51s

The patch does not generate ASF License warnings.

190m 11s

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5897/5/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/5897

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets

uname
Linux e444c8c2ca83 4.15.0-213-generic #224-Ubuntu SMP Mon Jun 19 13:30:12 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 3a6a70ec8308640199ad73b222a7cee2b95ee50f

Default Java
Private Build-1.8.0_382-8u382-ga-1~20.04.1-b05

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.20+8-post-Ubuntu-1ubuntu120.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_382-8u382-ga-1~20.04.1-b05

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5897/5/testReport/

Max. process+thread count
2671 (vs. ulimit of 5500)

modules
C: hadoop-common-project/hadoop-common hadoop-hdfs-project/hadoop-hdfs-rbf U: .

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5897/5/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

@hchaverri We need to fix the checkstyle issue.

@hchaverri We need to fix the checkstyle issue.

I've fixed the one introduced with my changes on TestSQLDelegationTokenSecretManagerImpl. The one in AbstractDelegationTokenSecretManager is pre-existing and has 37 references that would need to be updated. Do you recommend fixing them anyway?

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 29s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s

The patch appears to include 1 new or modified test files.

_ trunk Compile Tests _

+0 :ok:
mvndep
17m 54s

Maven dependency ordering for branch

+1 :green_heart:
mvninstall
20m 18s

trunk passed

+1 :green_heart:
compile
10m 22s

trunk passed with JDK Ubuntu-11.0.20+8-post-Ubuntu-1ubuntu120.04

+1 :green_heart:
compile
9m 34s

trunk passed with JDK Private Build-1.8.0_382-8u382-ga-1~20.04.1-b05

+1 :green_heart:
checkstyle
2m 21s

trunk passed

+1 :green_heart:
mvnsite
1m 56s

trunk passed

+1 :green_heart:
javadoc
1m 41s

trunk passed with JDK Ubuntu-11.0.20+8-post-Ubuntu-1ubuntu120.04

+1 :green_heart:
javadoc
1m 21s

trunk passed with JDK Private Build-1.8.0_382-8u382-ga-1~20.04.1-b05

+1 :green_heart:
spotbugs
2m 50s

trunk passed

+1 :green_heart:
shadedclient
21m 34s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 26s

Maven dependency ordering for patch

+1 :green_heart:
mvninstall
1m  0s

the patch passed

+1 :green_heart:
compile
9m 46s

the patch passed with JDK Ubuntu-11.0.20+8-post-Ubuntu-1ubuntu120.04

+1 :green_heart:
javac
9m 46s

the patch passed

+1 :green_heart:
compile
9m 29s

the patch passed with JDK Private Build-1.8.0_382-8u382-ga-1~20.04.1-b05

+1 :green_heart:
javac
9m 29s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

-0 :warning:
checkstyle
2m 18s
/results-checkstyle-root.txt
root: The patch generated 1 new + 23 unchanged - 1 fixed = 24 total (was 24)

+1 :green_heart:
mvnsite
1m 55s

the patch passed

+1 :green_heart:
javadoc
1m 37s

the patch passed with JDK Ubuntu-11.0.20+8-post-Ubuntu-1ubuntu120.04

+1 :green_heart:
javadoc
1m 22s

the patch passed with JDK Private Build-1.8.0_382-8u382-ga-1~20.04.1-b05

+1 :green_heart:
spotbugs
3m  1s

the patch passed

+1 :green_heart:
shadedclient
21m 45s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
16m 19s

hadoop-common in the patch passed.

+1 :green_heart:
unit
19m  3s

hadoop-hdfs-rbf in the patch passed.

+1 :green_heart:
asflicense
0m 51s

The patch does not generate ASF License warnings.

183m 51s

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5897/6/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/5897

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets

uname
Linux 64bf3973a337 4.15.0-213-generic #224-Ubuntu SMP Mon Jun 19 13:30:12 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 06adf5158e948a3b6b21dbf99f24e263cb8035d9

Default Java
Private Build-1.8.0_382-8u382-ga-1~20.04.1-b05

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.20+8-post-Ubuntu-1ubuntu120.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_382-8u382-ga-1~20.04.1-b05

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5897/6/testReport/

Max. process+thread count
2580 (vs. ulimit of 5500)

modules
C: hadoop-common-project/hadoop-common hadoop-hdfs-project/hadoop-hdfs-rbf U: .

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5897/6/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

I've fixed the one introduced with my changes on TestSQLDelegationTokenSecretManagerImpl. The one in AbstractDelegationTokenSecretManager is pre-existing and has 37 references that would need to be updated. Do you recommend fixing them anyway?

@hchaverri Thank you very much for your contribution! For the existing problem of history, we don't need to fix it.

@hchaverri merged into trunk.  @goiri @simbadzina Thanks for helping to review the code.


Upgrade to support Swift 3.0.1

This PR allows the project to be ran using Xcode 8.1. and Swift 3.0.1. Builds on top of #10 since there was code that needed to also be converted (+ conflicts to fix).

Bump


There is no Notification Channels creation resource in 2.x.x versions, but there is in 1.0.0

Is your feature request related to a problem?
While we create monitor we need to assign a notification channel. Since we want to implement all the related configs via terraform we don't want to create additional objects inside a cluster manually, but there is no ability with provider version >= 2.0.0
What solution would you like?
Develop a resource which creates a Notification Channel
What alternatives have you considered?
Develop a datasource which provides us headers for making an own http-request via terraform/http provider. This would be an awesome solution to implement it for making any http requests to our clusters and would be nice if both, main and alternative solutions, are implemented
Do you have any additional context?

My bad. I found it called opensearch_channel_configuration. Testing

Closing this issue, thanks for the contribution @Vonter.
@kolesaev please re-open if required.


igluctl: Make the output argument to static generate optional

This argument is currently required: https://github.com/snowplow/iglu/blob/master/0-common/igluctl/src/main/scala/com.snowplowanalytics.iglu/ctl/Command.scala#L59

Migrated to https://github.com/snowplow-incubator/igluctl/issues/9


Dataclass backend

Aiming to implement:

All internal "config" data structures use dataclasses
Use TOML files as input in case the user still wants this (secondary objective)

With the aim of improving script-ability

Tail.py working
TODO

dihedral.py
README
TOML
Figure out a better way of choosing what to solve for


Evaluating style transfer on extracted profiles

Let's evaluate how well we can translate profiles across experimental conditions.
We could pick several different pairs of conditions in this dataset, but we're considering focussing on cell type translation because that's the most compelling example.
@niranjchandrasekaran can you list what we have here?
U2OS and A549, across two timepoints, for JUMP-Target and JUMP-MOA plates, right?
cc @AnneCarpenter

Doesn't CPJUMP1 only have JUMP-Target1 compound plates?
We could transfer styles across cell types for one timepoint (using the other timepoint as heldout case)
AND across timepoints for one cell type (using timepoints in the other cell type as heldout case)
(or just one of those)

U2OS and A549, across two timepoints, for JUMP-Target and JUMP-MOA plates, right?

We only have JUMP-Target

can you list what we have here?

We have different cell types (U2OS and A549), time points (low and high), perturbation modality (ORF, CRISPR, Compound), densities (-20%, baseline, 20%), cell line (Parental and monoclonal) and selection (presence and absence of Puromycin and Blasticidin).

Thanks for clarifying.
I've temporarily moved this to a gdoc thread where we discussed this idea previously https://docs.google.com/document/d/1k7ChkKX8DTzP2_eV4UqDP_ZywRpDnciS9TD1S-fYQjU/edit?disco=AAAALGYZ_Bs

We're good to proceed with the plan of doing this
It's best to minimize the number of sources of technical variation, so let's simplify:
Strawman plan:
We will use

4 replicates A549 JUMP-Target-Compound Plate at 48h
4 replicates U2OS JUMP-Target-Compound Plate at 48h
Split the 306 compounds into 80/20 train/test
Train/validate a model to predict U2OS signature given A549 signature on the 80%, test on the 20%. We will report  precision@K on the test dataset i.e. how well can I retrieve the real U2OS replicates for a compound, given the translated (i.e. synthetic) U2OS profile? We can't report percent matching because 20% is too little. We can report percent replicating I suppose.

@MarziehHaghighi feel free to ask for what you need in this thread

We are concerned there is not sufficient time to carry out an experiment like this before the paper deadline, unfortunately. Marzieh is going to think about it.

@AnneCarpenter @shntnu
Note that in our work "style" is "treatment" and another cell-type would be another unseen cell identity. Using that model for this much simpler problem is a huge overkill and is not justifiable. Here, "style" is cell type and we have two style and many example to learn that binary styles independent of treatment etc. I try to see if I can run a NST quick model on the data here.
@niranjchandrasekaran Could you please share the location of images/metadata with me?

Logistically, it's great to use a simpler method because (a) you are more
likely to be able to finish in time, and (b) it won't interfere w your
style transfer paper in progress!

Could you please share the location of images/metadata with me?

Images are here - s3://jump-cellpainting-public/projects/2019_07_11_JUMP-CP-pilots/2020_11_04_CPJUMP1/
Based on what Shantanu said in https://github.com/jump-cellpainting/neurips-cpjump1/issues/9#issuecomment-901323453, the plates that will be useful to you are

Plate name
Description

BR00117017
A549 48-hour Compound Plate 1

BR00117019
A549 48-hour Compound Plate 2

BR00117015
A549 48-hour Compound Plate 3

BR00117016
A549 48-hour Compound Plate 4

BR00117012
U2OS 48-hour Compound Plate 1

BR00117013
U2OS 48-hour Compound Plate 2

BR00117010
U2OS 48-hour Compound Plate 3

BR00117011
U2OS 48-hour Compound Plate 4

Here is the plate map - https://github.com/jump-cellpainting/neurips-cpjump1/blob/main/metadata/platemaps/2020_11_04_CPJUMP1/platemap/JUMP-Target-1_compound_platemap.txt
and here is additional metadata -
https://github.com/jump-cellpainting/neurips-cpjump1/blob/main/metadata/moa/JUMP-Target-1_compound_metadata.tsv
(you can combine the plate map and additional metadata using broad_sample.
Let me know if you need anything else.


Test Debug Session still proceeds after cancelling build task

Describe the bug
When you cancel the build task that precedes a debug session for a test, the debug session still starts and LLDB can show fatal errors
To Reproduce
Steps to reproduce the behavior:

I had cleared .build
Started debug session for a test
Pressed ctrl+c in the task terminal to cancel the build all task when I realized I forgot to save a file
Build task is cancelled, but debug session still starts
The call stack frame showed a fatal error

Expected behavior
Check the exit code, if non-zero then don't proceed?
Environment

OS: macOS
Swift version (output of swift --version): 6.0
Visual Studio Code version: 1.92
vscode-swift version: 1.10.7

Additional context
N/A

Verified with e0a0fba


TTSP issue with liberica jdk 17

Describe the bug
Hi @apangin ,
I was trying to profile an application running under liberica jdk 17.0.1 and hit a wall.
Using argument for ttsp I got an error:
[ERROR] End address not found
Changing it to explicit begin/end resulted in the same error.
Then I looked into the symbols of libjvm and didn't find RuntimeService::record_safepoint_synchronized there. More - there're no functions visible for RuntimeService at all.
liberica 11 and 12 are good, but 17 is missing a lot of symbols.
I tried to find any other possible function in the source code to trigger the end of TTSP profiling but nothing in the call chain is exported by liberica 17 libjvm. Or I just missed it.
Not sure if it's a bug, but other issues type looks wrong too.
May be you have any idea how to deal with it when I need to profile ttsp ?
Thanks
Expected vs. actual behavior
No response
Reproduction Steps
asprof -e wall -i 1ms --ttsp  -f ./profile.jfr
Additional Information/Context
No response
Async-profiler version
3.0
Environment details
CentOS 7
liberica jdk 17.0.x
x86_64

I remember early releases of Liberica JDK 17 were built with compiler options that resulted in "overoptimized" binaries that have very little symbols exposed. AFAIK this is no longer a case.
Can you check if the problem is still relevant to JDK 17.0.13? 17.0.1 is too old anyway.

I remember early releases of Liberica JDK 17 were built with compiler options that resulted in "overoptimized" binaries that have very little symbols exposed. AFAIK this is no longer a case. Can you check if the problem is still relevant to JDK 17.0.13? 17.0.1 is too old anyway.

Yeah...you're right. 17.0.13 exports RuntimeService::record_safepoint_synchronized indeed.
The last one I had on the server was 17.0.7 with missing symbols as well, so I decided it was a dead end :-)
BTW, I also tried profiling with --begin ShenandoahHeap::safepoint_synchronize_begin --end ShenandoahHeap::safepoint_synchronize_end and got an empty JFR, although those calls look promising in the source code.
Thank you!

-begin ShenandoahHeap::safepoint_synchronize_begin --end ShenandoahHeap::safepoint_synchronize_end also work, but only with -XX:+UseShenandoahGC. An empty recording may mean there were no long enough time-to-safepoint pauses for the selected sampling interval. You can double check with this example that is known to cause long ttsp pauses.

Yes, I'm using Shenandoah GC.
May be you know how long should be time-to-safepoint pause to be captured by the profiler?

how long should be time-to-safepoint pause to be captured by the profiler?

It matches sampling interval.
With the default CPU sampling interval of 10 ms, a thread that delays safepoint should actively run on CPU for at least 10 ms to be captured for sure. Sampling interval can be configured with -i option.
Sometimes a thread may delay safepoint even when not running on CPU, for example, by accessing a memory-mapped file and causing swap-in. In these cases, CPU profiling mode will not catch such a thread, but the Wall-clock mode will.

I set sampling interval to 1ms and having Reaching safepoint:<PHONE_NUMBER> ns in log still no threads information recorded.
Looks like the only option is to upgrade to 17.0.13

I hope the question is answered. Let me know if anything else is needed on the profiler end.
Note that in some cases async-profiler won't be able to detect TTSP pause, if it is caused by external factors: e.g., if the whole process does not receive CPU time from the OS for whatever reason, async-profiler will not work either, since it runs in the context of the process being profiled.


Add required dependency information to the project page on CurseForge.

You shouldn't be relying on users to read the description to know what's required for a mod as the CurseForge app allows them to download the file without seeing it and would automatically install the required mods alongside it.
Go to the files page and click the name of each file, then edit, and at the bottom set the Relations for the required mod.

Part of the problem is that there are multiple mods that provide the WAILA/HWYLA/WTHIT interface, and this mod supports all of them (and doesn't want to force your choice). The reason it's in the description is because there isn't a way on CurseForge (that I know of) to say "this mod requires one of these other mods". When/if that feature is implemented or something similar to it, then we can come back to this.


Changes Activity Log to Recent updates

Fixes https://[GHE]/CFGOV/platform/issues/1112
Changes

Changes Activity Log page title to "Recent updates" (sentence case)
Changes "Activity Log" in mega menu to "Recent Updates". Within the mega menu, moves "Recent Updates" to the top of the column, above Blog, Newsroom, Events.
Changes "See all activities" link under "Latest updates" section at the bottom of the Homepage to "See all updates".

Testing

Visit homepage for "See all updates" change.
localhost:8000/activity-log/ to see change to page title, and to check mega menu.

Screenshots

Should we be changing the URL from activity-log to recent-updates in conjunction with this?

@anselmbradford
I believe that @schaferjh said that changing the URL was not necessary for this update since users won't access this page directly. @schaferjh @sarahsimpson09 - Can you confirm?

Changing the URL isn't necessary, thanks.

@anselmbradford
Can you add a screenshot of the homepage updates?

I believe that @schaferjh said that changing the URL was not necessary for this update since users won't access this page directly.

Yup! URL is not updated.

Can you add a screenshot of the homepage updates?

@nataliafitzgerald Added to original post above!

There's a TODO after this PR which is to actually rename the single "Activity Log" page to "Recent Updates".

Updated this to "Recent updates" (to match incoming case changes) in content.consumerfinance.gov. Do I need to update it anywhere else?

For consistency this should be updated on build as well.

For consistency this should be updated on build as well.

Done!


p4c: <IP_ADDRESS> -> <IP_ADDRESS>

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/p4lang/p4c/releases.
meta.description for p4c is: "Reference compiler for the P4 programming language"
meta.homepage for p4c is: "https://github.com/p4lang/p4c"
meta.changelog for p4c is: ""
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found <IP_ADDRESS> with grep in /nix/store/9dsbwja8s2v1yl7v42fa0bmzdm1j8x1w-p4c-<IP_ADDRESS>
found <IP_ADDRESS> in filename of file in /nix/store/9dsbwja8s2v1yl7v42fa0bmzdm1j8x1w-p4c-<IP_ADDRESS>

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
p4c

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/9dsbwja8s2v1yl7v42fa0bmzdm1j8x1w-p4c-<IP_ADDRESS> \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A p4c https://github.com/r-ryantm/nixpkgs/archive/66b172f2260d5b9cb58d67ddb00d31144d93694d.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/9dsbwja8s2v1yl7v42fa0bmzdm1j8x1w-p4c-<IP_ADDRESS>
ls -la /nix/store/9dsbwja8s2v1yl7v42fa0bmzdm1j8x1w-p4c-<IP_ADDRESS>/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

p4c

Maintainer pings
cc @RaitoBezarius for testing.

@GovanifY can I let you be co-maintainer of p4c and add a changelog url to the package please? Thank you a lot <3

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/p4lang/p4c/releases.
meta.description for p4c is: "Reference compiler for the P4 programming language"
meta.homepage for p4c is: "https://github.com/p4lang/p4c"
meta.changelog for p4c is: ""
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found <IP_ADDRESS> with grep in /nix/store/dgfsdkyl4i2phb8b7gs1alw6cvn232c9-p4c-<IP_ADDRESS>
found <IP_ADDRESS> in filename of file in /nix/store/dgfsdkyl4i2phb8b7gs1alw6cvn232c9-p4c-<IP_ADDRESS>

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
p4c

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/dgfsdkyl4i2phb8b7gs1alw6cvn232c9-p4c-<IP_ADDRESS> \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A p4c https://github.com/r-ryantm/nixpkgs/archive/08a08278cb5ef38ae28c2436e3380ed4546847aa.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/dgfsdkyl4i2phb8b7gs1alw6cvn232c9-p4c-<IP_ADDRESS>
ls -la /nix/store/dgfsdkyl4i2phb8b7gs1alw6cvn232c9-p4c-<IP_ADDRESS>/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

p4c

Maintainer pings
cc @RaitoBezarius for testing.

@RaitoBezarius Ah sure! Do you want me to do the changes myself/add the changelog URL? Also, where would it be, in the next releases changelog?
(Currently a bit stranded IRL, car AWOL and friend under surgery, so sorry for possible delays)

@RaitoBezarius Ah sure! Do you want me to do the changes myself/add the changelog URL? Also, where would it be, in the next releases changelog?
(Currently a bit stranded IRL, car AWOL and friend under surgery, so sorry for possible delays)

There is nothing urgent of course, just add the generic meta changelog URL, no need to pick a specific release, so we can merge this PR soon. :)

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/p4lang/p4c/releases.
meta.description for p4c is: Reference compiler for the P4 programming language
meta.homepage for p4c is: https://github.com/p4lang/p4c
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found <IP_ADDRESS> with grep in /nix/store/adcv8pzg3jhn4ka1hkvgd5d2g0ivsrx5-p4c-<IP_ADDRESS>
found <IP_ADDRESS> in filename of file in /nix/store/adcv8pzg3jhn4ka1hkvgd5d2g0ivsrx5-p4c-<IP_ADDRESS>

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
p4c

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/adcv8pzg3jhn4ka1hkvgd5d2g0ivsrx5-p4c-<IP_ADDRESS> \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A p4c https://github.com/r-ryantm/nixpkgs/archive/7dc9a9300e94b040f29e5a0f9e741af74bfe5eab.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/adcv8pzg3jhn4ka1hkvgd5d2g0ivsrx5-p4c-<IP_ADDRESS>
ls -la /nix/store/adcv8pzg3jhn4ka1hkvgd5d2g0ivsrx5-p4c-<IP_ADDRESS>/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

p4c

Maintainer pings
cc @RaitoBezarius for testing.

@ofborg build p4c p4c.passthru.tests


Regexify fails with StackOverflowError

Regex: ^arn:.+:.+:.*:([0-9]{12}):(.+)$

Thanks for opening the issue @axelfontaine ,
Let me check, I think it could be an issue in the generex dependency, since there's not much logic in faker itself regarding regexifying strings.

@serpro69 It is most certainly downstream. However this could potentially mean that generex may have to be switched out for either an internal solution or another lib.
The answers to this question mention a few alternatives: https://stackoverflow.com/questions/22115/using-regex-to-generate-strings-rather-than-match-them

Probably caused by https://github.com/mifmif/Generex/issues/56

Good find :+1: and an old issue that didn't get much attention...
Well, this means generex will have to go :)


@seydx are you still actively working this plugin?

Sorry to bug, trying to work out where I go now that the iOS 13 has caused issues with working Tado via HomeKit, please can you advise if you are continuing to develop this app? I, like others love what you’ve done and appreciate the effort, please advise? Thanks

I will update this plugin very soon

Really?

Maybe forces can be joined between @SeydX and @lukasroegner who was developing a new plugin for homebridge?


MAVLink app: Enable network and serial on POSIX

This fixes some incorrect limitations that were present on POSIX UARTs.

Reviewed, looks good to me. I tested POSIX SITL with jMAVSim and QGC which worked. I couldn't test the RC_CHANNELS_OVERRIDE yet.

You get RC input for SITL by attaching a Pixhawk, or by configuring the Taranis as Joystick.

@LorenzMeier Can you give me any pointers how to route the incoming RC_CHANNELS_OVERRIDE into jMAVSim or gazebo?

You need to connect a Pixhawk, then the SITL app will try to open the port and read the MAVLink stream. You do not use the RC_OVERRIDE input.

I found the hook for jMAVSim but it seems WIP for SITL but I can't find it for gazebo...

https://github.com/PX4/Firmware/blob/master/src/modules/simulator/simulator_mavlink.cpp#L473

@LorenzMeier it seems to me that we actually don't need RC_CHANNELS_OVERRIDE but the simulators already RC_CHANNELS.
RC_CHANNELS_OVERRIDE seems to be just duplicate using up bandwith.


textAnalzyers Contribution

Hi,
How can I contribute to the upcoming ArangoSearch?
I want to add support for Farsi/Persian language.
This is not satisfying:

Exposing the capability to specify and configure custom runtime-pluggable analyzers is targeted for releases in the near future.

We're standing on the shoulders of giants here - which probably would require most of the work towards a farsi suppport-

ICU which its using for tokenizing; You would need to verify thats working.
snowball - this is used for stemming; You probably would need to implement something similar to the russian or mandarin implementations

Regarding pluggeable analyzers; we have to do this one step at a time. Accelerating things is allways possible by putting money to more developers; so if you want to see this quicker, get in contact with sales!

To give you some more concrete pointers:

To check the tokenization add farsi or persian tests to
string-utf8-test.cpp and [string-utf8-normalize-test.cpp]https://github.com/arangodb/arangodb/blob/devel/tests/Basics/string-utf8-normalize-test.cpp

Next step is to add a stemmer to http://snowballstem.org/algorithms/ which is probably much more complicated

@p30arena thank you for your interest! Pluggable analyzers feature is in our pipeline, but unfortunately at the moment it's planned for 3.4.1 not 3.4.
For 3.4 we're mostly focused on efficient implementation of cluster wide ArangoSearch views.

Custom Analyzers are supported starting with 3.5.0, but to support Persian/Farsi we need a Snowball stemmer. Feel free to contribute one to that project:
https://github.com/snowballstem/snowball/tree/master/algorithms
We will update the Snowball dependency in a future release.

closing as solved.
We refresh snowball now and then; If a newer version is available that better suits your needs, please give us a heads up.
https://www.arangodb.com/2020/08/arangodb-3-7-a-big-step-forward-for-multi-model/


upgrade to 2.0. das ist alles.

Sorry, couldn't wait for ya. Had to get it done. PR is for upgrading to core 2.0, and nothing more.
Unfortunately the "legacy" samples are non functional, I think I'm on track to solving that, but not planning on updating the readme, figure you know it far better than I would.

No worries. I will push a version up to NuGet tonight.
Thanks!