Promo materials - updated files and create page

Collecting updated promo materials - place in new docs folder https://github.com/CDLUC3/dmp/tree/development/docs
Text for page from Gdrive Content folder: https://docs.google.com/document/d/1pffWZ-HN4EDl923S1m3j2r1LZIr_bSbbIKLf3hn7z7I/edit#

static content and files added. closing issue.


[Bug]: No location or stack trace for Firefox console messages

Minimal, reproducible example
const puppeteer = require("puppeteer");
const http = require("http");

let server = http.createServer((req, res) => res.end());
server.listen(8080, async () => {
	await test("chrome");
	await test("firefox");
	process.exit(0);
});

async function test(engine) {
	console.log(`Testing ${engine}`);
	let browser = await puppeteer.launch({
		browser: engine
	});

	let page = await browser.newPage();
	page.on("console", log => {
		console.log("Location:", log.location());
		console.log("Stack trace:", log.stackTrace());
	});
	await page.goto("http://localhost:8080", {
		waitUntil: "networkidle0"
	});
	await page.evaluate(() => {
		console.log("This is a log");
	});
}

Background
When running on Firefox, ConsoleMessages have no location() or stackTrace() information.
Expectation
Expected output:
Testing chrome
Location: {
  url: 'pptr:evaluate;test%20(%2Fworkspaces%2Ftest%2Fbug.js%3A25%3A13)',
  lineNumber: 1,
  columnNumber: 10
}
Stack trace: [
  {
    url: 'pptr:evaluate;test%20(%2Fworkspaces%2Ftest%2Fbug.js%3A25%3A13)',
    lineNumber: 1,
    columnNumber: 10
  }
]
Testing firefox
Location: {
  url: 'pptr:evaluate;test%20(%2Fworkspaces%2Ftest%2Fbug.js%3A25%3A13)',
  lineNumber: 1,
  columnNumber: 10
}
Stack trace: [
  {
    url: 'pptr:evaluate;test%20(%2Fworkspaces%2Ftest%2Fbug.js%3A25%3A13)',
    lineNumber: 1,
    columnNumber: 10
  }
]

Reality
Actual output:
Testing chrome
Location: {
  url: 'pptr:evaluate;test%20(%2Fworkspaces%2Fholoprint%2Fbug.js%3A25%3A13)',
  lineNumber: 1,
  columnNumber: 10
}
Stack trace: [
  {
    url: 'pptr:evaluate;test%20(%2Fworkspaces%2Fholoprint%2Fbug.js%3A25%3A13)',
    lineNumber: 1,
    columnNumber: 10
  }
]
Testing firefox
Location: {}
Stack trace: []

Puppeteer configuration file (if used)
No response
Puppeteer version
23.7.0
Node version
20.17.0
Package manager
npm
Package manager version
10.8.2
Operating system
Linux

So in WebDriver BiDi protocol which is used for Firefox by default, the stack trace is only captured for "assert", "error", "trace", and "warn" messages. Chrome here is not very compliant with the WebDriver BiDi spec but compliant with existing CDP behavior. @SuperLlama88888 what is your use case for using stacktraces on console.log? It would help us understand if the WebDriver BiDi standard needs to be extended to capture stacktraces here too.
cc @whimboo

I would reference https://github.com/whatwg/console/issues/203 here as well.

@whimboo do you recall if we had an issue for WebDriver BiDi for this? As far as I see, the logic for deciding about the stacktrace is defined in WebDriver-BiDi spec so we could change it there https://w3c.github.io/webdriver-bidi/#ref-for-console-steps?

@OrKoN I am trying to filter console messages from 3rd-party libraries by looking at the origin URL of messages.
Thanks for your help on this! Really appreciate it.

Thanks, I think for this use case and for the location method it actually might be an issue solvable within Puppeteer.

@whimboo do you recall if we had an issue for WebDriver BiDi for this? As far as I see, the logic for deciding about the stacktrace is defined in WebDriver-BiDi spec so we could change it there https://w3c.github.io/webdriver-bidi/#ref-for-console-steps?

No I don't think that we have an issue for WebDriver BiDi yet. If you think that we should relax on our side, please file an issue for tracking. Thanks.

@SuperLlama88888 please let me know if https://github.com/puppeteer/puppeteer/pull/13273 is helpful for your use case. It would only be helpful if messages are coming from different frames.
For the spec, I filed  https://github.com/w3c/webdriver-bidi/pull/807

Sorry for getting back to you so late. #13273 doesn't really help me unfortunately. It only captures the location, which is the current frame, and an empty stacktrace. There is still no way to know which script messages came from.
Chrome:
Text: WARNING: Multiple instances of Three.js being imported.
Location: {
  url<EMAIL_ADDRESS>  lineNumber: 11,
  columnNumber: 606730
}
Stack trace: [
  {
    url<EMAIL_ADDRESS>    lineNumber: 11,
    columnNumber: 606730
  }
]
Text: This is a log
Location: {
  url: 'pptr:evaluate;test%20(%2Fworkspaces%2Fhp-pr-test%2Ftest.js%3A26%3A13)',
  lineNumber: 2,
  columnNumber: 10
}
Stack trace: [
  {
    url: 'pptr:evaluate;test%20(%2Fworkspaces%2Fhp-pr-test%2Ftest.js%3A26%3A13)',
    lineNumber: 2,
    columnNumber: 10
  }
]

Firefox:
Text: WARNING: Multiple instances of Three.js being imported.
Location: { url: 'http://localhost:8080/' }
Stack trace: []
Text: This is a log
Location: { url: 'http://localhost:8080/' }
Stack trace: []


Remove entrypoints in setup for internal backends

This PR aims to avoid conflicts during the transition period between the old backend implementation and the new plugins.
During the transition period will coexist both external backend plugins and internal ones.
Currently, if two plugins with the same name are detected, we just pick one randomly. It would be better to be sure to use the external one.
Main changes:

Remove from setup.cfg

Store in the internal backend and stored in the dictionary in plugins.py. The dictionary is updated with the external plugins detected by pkg_resources.

Move the class BackendEntrypoints in common.py to resolve a circular import.

Add a test

[x] Related to https://github.com/pydata/xarray/issues/4309

[x] Tests added

[x] Passes isort . && black . && mypy . && flake8

I merge it as it involves only the new APIv2 code.


Couldn't collect tokens

Hi, i've wrote a code using cloudflare-scrape, and read "Integration" section, but i'm recieving an error 'forum.goodchoiceshow.ru' returned an error. Could not collect tokens. Where did i made a mistake?
request = urllib2.Request(url + param_joiner + buildblock(random.randint(3,10)) + '=' + buildblock(random.randint(3,10)))
	cookie_value, user_agent = cfscrape.get_cookie_string(host)
	request.add_header('User-Agent', user_agent)
	request.add_header('Cookie', cookie_value)
	#request.add_header('User-Agent', random.choice(headers_useragents))
	request.add_header('Cache-Control', 'no-cache')
	request.add_header('Accept-Charset', 'ISO-8859-1,utf-8;q=0.7,*;q=0.7')
	request.add_header('Referer', random.choice(headers_referers)+random.choice(keyword_top))
	request.add_header('Keep-Alive', random.randint(110,120))
	request.add_header('Connection', 'keep-alive')
	request.add_header('Host', host)
	index = random.randint(0,len(ips)-1)
	proxy = urllib2.ProxyHandler({'http':ips[index]})
	opener = urllib2.build_opener(proxy,urllib2.HTTPHandler)
	urllib2.install_opener(opener)
	try:
			urllib2.urlopen(request)
			if(flag==1): set_flag(0)
			if(code==500): code=0
	except urllib2.HTTPError, e:
			set_flag(1)
			code=500
	except urllib2.URLError, e:
			sys.exit()
	else:
			inc_counter()
			urllib2.urlopen(request)
	return(code)

Hmm looks like your trying to use cloudflare-scrape with HULK :P
Anyway, I don't think urllib2 works with cloudflare-scrape, but instead with Requests (https://github.com/kennethreitz/requests)

@JamesJGoodwin I successfully got the cookies for this site with this snippet of code:
import cfscrape
scraper = cfscrape.create_scraper()
result = scraper.get_cookie_string('http://domain.com')
print result # tuple of the cookie value and the user-agent


crawls unit does not check for failed spider units

Crawls service uses the Fleet API to schedule a spider unit for execution and then enters an exponential backoff loop waiting for the spider's results to become available in etcd. However, if the unit fails for whatever reason the crawls service will wait forever (almost) until it gives up. Crawls service should be checking for failed units while also checking for spider results in etcd and if a failed unit is detected an error should be returned to the spider requestor.

this will be resolved with issue #20 in that crawls will be modelled as jobs and job output will be spider output so polling for spider output will effectively check the job's/spider's status


library: implement pod-overrides and container-overrides annotations

Which area this feature is related to?

/area library
Which functionality do you think we should add?
the devfile/library should have an implementation for interpreting pod-overrides and container-overrides annotations as described in https://github.com/devfile/api/issues/920#issuecomment-1244059075
Why is this needed? Is your feature request related to a problem?
we are getting user inquiries about how they can set advanced parameters like

service accounts
security context
additional resource limits

implemented in https://github.com/devfile/library/pull/158
/close


feat: bar label position for horizontal bar charts

This PR is regarding issue #3520, the label placement for horizontal bar charts was always on top, this moves labels for horizontal bar charts to either left or right, depending on negative or positive value of the bar.
Included a recipe called Bar with Value Labels (horizontal), looks like this attached image.
Some work may need to be done somewhere around adjusting the size of the container graph if the labels go outside of the drawing area?

Thanks @aespitia, this looks great!

Some work may need to be done somewhere around adjusting the size of the container graph if the labels go outside of the drawing area?

myPlot.Axes.Margins(bottom: 0, top: .2); // before
myPlot.Axes.Margins(left: .2, right: .2); // after


Clinician portal for proms/sparc

WORK IN PROGRESS. Includes the locking feature.
To test with sparc:

build this cards branch with mvn clean install -Pdocker
build sparc (main or other branch) with mvn clean install
from the cards directory, start with PROJECT_VERSION=1.0.0-SNAPSHOT ./start_cards.sh --dev --project sparc --locking
create patient information and visit information forms
navigate to the patient subject/visit subject pages

As a trusted user I can only see clinician forms. Is this correct?

As a trusted user I can only see clinician forms. Is this correct?

Before the patient submits the survey, yes, but after you should see all of them.


// , Docs don't point out that you need to follow someone and send them a message to file share.

// , Keybase GUI Version: 1.0.23-20170519175207+d6c5e9e
Update https://keybase.io/getting-started accordingly.
Also, thanks for making it relatively easy to report bugs.

// , This isn't much of a problem for experienced users, but for newbies, it can waste an hour of time.


Split unix passwd, shadow, group artifacts.

@joachimmetz @pidydx
GRR has a parser that works on /etc/passwd, so I need these split out.

LGTM

LTGM, one nit, why are they not using the short hand style as described here
https://github.com/ForensicArtifacts/artifacts/blob/master/docs/style_guide.adoc#sources

You're right, I'll fix that now.


Skip mocking of keyfile_json field

Description
Makes the required field keyfile_json empty, so that dbt can parse it if the project.yaml file is being mocked.
Related Issue(s)

closes #586
Checklist

[ ] I have made corresponding changes to the documentation (if required)
[ ] I have added tests that prove my fix is effective or that my feature works

Hi @joppevos , thanks a lot for the fix! Please add a test to ensure we don't have any regressions moving forward

Thanks @tatiana , I added a simple test. We could also run a dbt ls test on the mocked profile, but then we are kinda testing dbt itself for not breaking over time.

Hi @joppevos , thanks a lot for the fix! Please add a test to ensure we don't have any regressions moving forward

Thanks @tatiana , I added a simple test. We could also run a dbt ls test on the mocked profile, but then we are kinda testing dbt itself for not breaking over time.
Would be great to get this solved, cause you can't get a DAG pass general Airflow Ci tests when it's failing on parsing.

Hi @joppevos , thanks a lot for the fix! Please add a test to ensure we don't have any regressions moving forward

Thanks @tatiana , I added a simple test. We could also run a dbt ls test on the mocked profile, but then we are kinda testing dbt itself for not breaking over time.
Would be great to get this solved quickly, cause you can't get a DAG pass general Airflow CI tests when it's failing on parsing.

Thanks, @joppevos , this is very helpful.
This will be released as part of the 1.2 release, this week.

This was released as part of 1.2:

https://pypi.org/project/astronomer-cosmos/1.2.0/
https://github.com/astronomer/astronomer-cosmos/releases/tag/astronomer-cosmos-v1.2.0


Ability modifier didnt work

used an ability modifier on a mon and the ability didn't change.
Tried one more time to reproduce the same result but wasnt able to.
modifier got refunded.

The poke was impidimp, the original ability was Frisk, and I was trying to change it to Prankster

Tested this issue, could not recreate. If this happens again let us know. Sounds like a one-off glitch though!


Provide a way to mark System workqueue thread as K_ESSENTIAL

Is your enhancement proposal related to a problem? Please describe.
When system is highly dependent on system workqueue, (for example multiple drivers defer interrupt work to be done later by system workqueue) if the system workqueue thread crashed then a reboot is needed to get the system back up and running and there is no point in continuing to operate in such state.
Describe the solution you'd like
The Best solution would be to make use of Zephyr K_ESSENTIAL flag. so when sysworkq thread crash the fault handler can act accordingly for all essential threads in the system without special if statements or trying to match thread name to do special handling.
Describe alternatives you've considered
use k_thread_foreach to loop over all threads to find the sysworkq thread by its name then act.
Additional context
Add any other context or graphics (drag-and-drop an image) about the feature request here.

@henrikbrixandersen Should I make it configurable via KCONFIG ?

@henrikbrixandersen Should I make it configurable via KCONFIG ?

I don't see why the system work queue thread would not be essential. I am actually surprised it is not marked as such already. I'd go for a solution without a Kconfig unless others can come up with a good reason for making it configurable.


SQLite.SQLiteException: file is not a database

I am trying to create an encrypted database using SQLiteAsyncConnection in a Xamarin project. I am using sqlite-net-sqlcipher package and updated it to 1.6.258 beta version in PCL project. I also updated the same for Android app project as well. This is the only package I am using for database encryption. Am I supposed to use any other packages along with this?
I am using following lines of code to encrypt the database.
           SQLitePCL.Batteries_V2.Init();             m_databaseName = DependencyService.Get<SqlHelper>().GetDBFileName();             m_database = new SQLiteAsyncConnection(m_databaseName);             string password = "$1234#";             m_database.QueryAsync<int>("PRAGMA key='?'", password);
When i run this the first time, a DB file is created but it is not encrypted. I am able to open the database and view its contents. Second time when I run the App i get "SQLite.SQLiteException: file is not a database" exception.
As a test case, when i used SQLiteConnection instead of SQLiteAsyncConnection, database is encrypted correctly. Please let me know what is going wrong and if I am following the steps correctly. Thanks.

@wangminsho How did you resolved this issue ?

@wangminsho How did you resolved this issue ?


[DNM?] Ensure riff service invoke always prints any HTTP error

This has only been tested on macOS, but it should be tested on Windows too, so please do not merge
unless you have tested on Windows. To provoke a 404, hack service.go thus:
hostHeader := fmt.Sprintf("Host: %s", hostName+"x")

(I can do this next week - I left my Windows VM at home!)
The curl flags -sS are supposed to suppress output, but print any errors. This
would be the ideal solution here. Unfortunately, this doesn't appear to work
(at least with curl 7.54.0 on macOS).
Fixes https://github.com/projectriff/riff/issues/982

Works both on Windows and Mac 🎉

I took the liberty of carrying out the changes suggested for service.go. Merged as c5e123d5144c474f9d30624d5e6c3f339594add3.


fix(v2): optimize markdown parser regex (fix #4726)

Motivation
Fix #4726 (origin: #4713)
Have you read the Contributing Guidelines on pull requests?
Yes.
Test Plan

Clone my prepared repo and install
Start. Everything should work well
Incremental uncomment import statements in docs/intro.md, the compilation time will increase exponentially from around 15 imported components.
Apply the changes in packages/docusaurus-utils/src/markdownParser.ts of this PR into the corresponding compiled file (at<EMAIL_ADDRESS>of my repo.
Do the step 2-3 again to see the compile time improvement.

Please feel free to suggest any concrete performat tests that need to do.
Related PRs
N/A.

Thanks! I tried to hack around a bit locally and also found a similar potential solution.
That seems good enough to merge for now!
Seems to work, and could see your test being very slow without the regex change.
Not sure why but it seems to only affect the 2nd regex (alternateTitleMatch), not the first one.
Hey, if you are a regex pro and want to help, any improvement to support such syntax would be welcome, as the current regex does not cover edge cases:
          import Component1 from '@site/src/components/Component1';
          import Component2, {NamedImport, SecondImport} from '@site/src/components/Component2';
          import {Component3, NamedImport as NamedImportRenamed} from '@site/src/components/Component3';
          import * as Component4 from '@site/src/components/Component4';

Thanks ;)


feat: add -R shorthand to --role

Fixes #299.
I hope that there are no other parts in the code that should be changed.
To test the change:
mods -R default "What's up?"

Run this on the main branch version. This should give the following error:Check out  mods -h  for help.

unknown shorthand flag: 'R' in -R%!(EXTRA string=  )

Run this on my branch. This should give no errors.

thanks for the pr! just one comment though :)


next key after inlined object literal should be outdented

The following coffeescript
f =
  a: p1: 1, p2: 2
  next: 1

is compiled into
f = {
  a: {
    p1: 1,
    p2: 2,
    next: 1
  }
};

The expected behaviour should be
f = {
  a: {
    p1: 1,
    p2: 2
  },
  next: 1
};

as there is no trailing comma after "p2: 2" and "next: 1" is on the same
indention level as "a"
Best regards
Robert

This is now fixed on master.


customIcon in planetDefs.xml is not working

built from source
Set the texture of some planet to marslike but it still shows as earthlike

did you make sure it's set in the saves//advRocketry/planetDefs.xml as well?

It's a new world, imo there shouldn't be problems...

Ah, alright.  Can you still look because it's possible the config file didn't copy right or something and I'd like to rule that out if it's not much extra effort for you

umm no, there is no customIcon
2017-05-11 19:08 GMT+02:00 zmaster587<EMAIL_ADDRESS>
Ah, alright. Can you still look because it's possible the config file
didn't copy right or something and I'd like to rule that out if it's not
much extra effort for you
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/zmaster587/AdvancedRocketry/issues/624#issuecomment-300854672,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AC2b0YMo5ai9zdufYhlMjKqhYFeJDkS9ks5r40CUgaJpZM4NXFkz
.

Confirmed that it's not copied over

It does appear to actually copy.  It's just overwritten later. huh


Localization: Add additional keys for round-tag abbreviations

Hey there!
While creating the German locale, I noticed these two lines and thought I'd open a quick issue on this:
https://github.com/ChrisFeline/ToNSaveManager/blob/4ab3d6cff2ecc2ccd6b1b99f2333fa50c866d2ff/Localization/Language/en-US.json#L77-L78
I'd like to suggest adding some additional keys to allow for locale-specific replacement of these tags. My assumption is that these tags are referring to the English words "Respawn"/"Respawned", "Win"/"Won", and "Death"/"Died". Other languages might use different abbreviations, which can make these harder to interpret.
Additionally, adding a line to the tooltip on Line 78 might be useful, to explain what the abbreviations refer to. While I'm no expert on other languages, I believe the Japanese translation could use a single Kanji character in place as well, which might make it even more intuitive.

I will consider it

The reason I haven't addressed this issue yet is because the tags are hardcoded into an enumerator. Really a bad decision on my side.
I have tried using emojis on windows forms with the current font I've been using and it looks like this:

Do you think this looks good enough? If so I will proceed to replace the tags and add a more descriptive tool tip for the control.
For example:
"SETTINGS.SHOWWINLOSE.TT": "Entries will show a tag based on the source that triggered the save.\n🔄 Respawned.\n🏆 Survived.\n🔌 Disconnected.",

That looks great on my end, good idea from nomlasvrc!
I already updated my fork, made minor adjustments to it, as well as adding the line you provided preemptively, can update that prior to sending a pull request should you wish to change it 😃

@sageyx2002
That commit will not change the tool tip, it will change the whole label for the toggle.
The example is still just an idea, I am working internally on how to implement this in a way that works for me and everyone.
Also the next update will take some time, as it will be released with the next version of ToN. So, there will be a lot of translation changes and additions.

Got it! Going to keep that in mind.

Alright, I implemented this request. But please wait before updating your strings.
There will be 3 new keys:
"SAVE.TAG_R": "🔄",
"SAVE.TAG_W": "🏆",
"SAVE.TAG_D": "🔌",

These might change on release, but not worries.

These will most likely not need a translation, as this is pretty much globally visible.
There's a bit of a downside, I can't add these to the information in the settings window as they show as question marks. So instead I've changed the text and the context for the toggle label that enables this feature:

The tooltip for this entry will also change:
"SETTINGS.SHOWWINLOSE.TT": "Entries will show a tag based on the source that triggered the save.\n$$SAVE.TAG_R$$ Respawned.\n$$SAVE.TAG_W$$ Survived.\n$$SAVE.TAG_D$$ Disconnected.",

Note that this tooltip has string templates that inherit content from other keys.
$$SAVE.TAG_R$$
Thank you for your suggestions, hope this will be enough.

Absolutely! This is great!
I was a bit fast on that commit, and I will wait before updating. Thank you for the quick update!

Thank you for adopting my idea! I am relieved that the pictograms are displayed correctly.
I added the Japanese translation of that tooltip in a separate branch.

Added on latest TON update.


Add unit declarator to class declarations

As of Rakudo 2015.05, the unit declarator is required before using
module, class or grammar declarations (unless it uses a block).  Code
still using the old blockless semicolon form will throw a warning. This
commit stops the warning from appearing in the new Rakudo.

Thanks!


Gradle downloading mods through http

First of all, thank you for the writeup!
In the light of the deserialization vulnerability, is it dangerous for Gradle to download mods over http?
I'm talking about allowInsecureProtocol = true like here.
When discussing the issue with the GTNH team, the following response was given:

Https on the maven server would be nice, but it's not a security issue. All jars going there are uploaded from microsoft azure (who run github actions), and downloads are checked against checksums there. If an actor has access to mitming malware, they can as easily install custom https certs on your. Https prevents ISP sniffing at a small scale: at large scales, obtaining fraudulent https certificates is not that difficult and has been done before by state actors, on a tiny scale infecting someone's PC is more dangerous and then no protocol can save you.

What are your thoughts on the security of the given situation? Is it indeed secure, or did we miss a vector that appears due to http? I'm not a part of the GTNH team, but I'm concerned about the security, so I wanted to check.

We're living in 2023 and getting an HTTPS certificate is easier than ever, so I think HTTPS isn't just something "nice to have". Especially in the context of possible supply chain attacks that may arise from compromised build dependencies.
Download checksums also don't provide any security if they're served from the same HTTP-only webserver.
If you're only using a trusted network from a trusted provider then the risk of MITM attacks needed for this attack is pretty low. But since the file you linked is from an example gradle project, I don't know if this situation should be assumed in every case. It just creates an unnecessary attack vector that can be prevented easily.
So I would say it's not a super critical vulnerability, but it can definitely be exploited (depending on the situation).
If you want to read more about this topic, here are some links that may be useful:
https://blog.sonatype.com/central-repository-moving-to-https
https://infosecwriteups.com/want-to-take-over-the-java-ecosystem-all-you-need-is-a-mitm-1fc329d898fb
https://github.com/mveytsman/dilettante
(btw. I'm closing this issue as it has nothing to do with this repo but feel free to comment here if there are any more questions regarding this)


Bump check

Create a test to check if the project.clj version was changed if a *.clj file is modified/added/deleted.

Solved https://github.com/clj-holmes/clj-holmes/pull/33


Increase limits for loading DLS files

This PR modifies the example program to allow loading a DLS file (to help demonstrate the problem in issue #9) and increases limits to allow loading the tested DLS files, which resolves the issue.

I made the requested changes in the example file.


Handled host creating committee message

Closes EM-47.

Until #174, #184, and #185, please only review bbdd77470fca5a7e7104c89743cd3e00e2881d71.


Update Composer dependencies (2024-10-31-00-12)

Visual regression test passed!
View the visual regression test report


Hotfix animal emblem ranks

animal emblems shown
animal title shown

Thank you for the feedback.
As of now only the variable containing the absolute path should be edited to the relative path (RankGuide.tsx -> variable: pathToImgAndShowRankEmblem on line #493.
Other then that, all of your requests have been honored and executed.

Seems good 👍


When routing from content pages shall be available?

Its long since content pages don’t have option to post the comments. Any updates?

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

ID: 0193ed76-3eec-18ac-512f-45f8118dde34
Version Independent ID: 8e693833-3e45-2bb8-ddc2-d0c54936c3c1
Content: A New Feedback System Is Coming to docs.microsoft.com
Content Source: teamblog/a-new-feedback-system-is-coming-to-docs.md
Product: team-blog
GitHub Login: @EisenbergEffect
Microsoft Alias: EisenbergEffect

@jdanyow It would be cool if my name didn't appear on these and I didn't keep getting email (since I don't work for Microsoft).

@EisenbergEffect I'll take your login/alias out of this article's metadata.

Merging this change to live, @jdanyow @EisenbergEffect

Much thanks @dend and @jdanyow ! Keep up the great work. I'm never to far away :)

@dend
Thanks for your feedback, but please if i need to confirm about following team
https://docs.microsoft.com/en-us/sql/t-sql/functions/floor-transact-sql?view=sql-server-2017
?
They are still noticing to "Soon"; and i am unable to add feedback there....!!
Your help shall be appreciated.
Thanks


Refactor reminders specs to use python arrays

NOTE: this PR is on top of PR#291.  Only the last 3 commits belong.


Screenshot not rendering in README

The second screenshot in README.md isn't being rendered.

+1

Fixed. For some reason, I cannot close this.


added fitz and unstructuredio pdf loader py evadb script

Hi, @Stru17 how should the PDFReader be used by the EvaDB?

The idea is to replace add an additional PDF Reader backed by Unstructured IO -- not replace the default PDF reader.

@Stru17 The output and structure of this UnstructuredIOPDFReader class should match that of the original PDFReader class. Currently, it is more of a Python script.

I have provided an updated script, please take a look and provide some feedback on if I should further change/update it.


How to design a multithreaded application

I have an application that is split into multiple threads:
A) UI rendering
B) User input
C) Sending requests / receiving responses from TDLib
I have a class that presents an interface to interact with TDLib via methods like get_authorisation_state, get_chat_list, get_message_list etc and each one sends a request and receives a response. If you try to get the response in more than one thread, an error will occur. How can I implement these methods so that they can be called from different threads (for example, from thread C and thread A)?

I have an application that is split into multiple threads

This splitting is expected from a GUI application.
You can use td_send from any thread, but the way you dispatch response on the same thread depends on the programming language and framework you use. In some programming languages the best way is to use coroutines, in others you can use callbacks and framework-provided way to dispatch events between threads. In any case, you need an asynchronous scheduler provided by the programming language implementation or framework to simplify communication between threads.

My application will have a TUI interface and it is written in C++. I have a separate class for TDLib, in its constructor the TDLib update loop starts. This loop calls the get_authorization_state method, but I need to be able to call it in another thread too.

You must process responses asynchronously, so you need the ability to process response on another thread. Instead of full processing, you can just dispatch response to the original thread, but in any case you need the way to save request context and send data to other threads. For this you need some kind of multithread queues/multithread event dispatcher/multithread scheduler. There are a lot of decent already existing implementations of those, but I can't recommend any particular one.


Different training behavior between Sequential- and subclassed model (with JAX backend)

I wanted to streamline my models by using the Sequential class instead of creating a custom model via subclassing. In my code, I am using a Residual-Wrapper to realize more complex data flows within the Sequential class. Here, I observed exploding gradients when using the Sequential model vs. using the subclassed model with the same model topology, same initialization, and the same data.
I implemented a MWE to look into the issue, and it seems to be a problem with the Keras backend. Here, I create the same model using the Sequential API and the model subclassing. I train both models on the same data and compare the loss. I do this with jax and tensorflow backend. I don't use pytorch so I can not look into it.
Expected behavior: The models are identical, the training is deterministic due to global random seed. The loss values should be identical.
Observed behavior: With jax backend, there is a difference in the loss values between both models. With tensorflow backend, both models exhibit exactly the same loss values.
import os
os.environ['KERAS_BACKEND'] = 'jax'

from keras import Sequential, Model, layers, initializers
from keras.src.backend.config import backend
from keras import utils
import numpy as np

utils.set_random_seed(1)

conv_kwargs = dict(filters=1, kernel_size=3, padding='same', kernel_initializer=initializers.HeNormal())

class MyModel(Model):
    def __init__(self):
        super().__init__()
        self.layer = layers.Conv1D(**conv_kwargs)

    def build(self, input_shape):
        self.layer.build(input_shape)

    def call(self, inputs):
        return self.layer(inputs)

subclassed_model = MyModel()
sequential_model = Sequential([layers.Conv1D(**conv_kwargs)])

# build models
x = np.ones((1, 32, 1))
subclassed_model.build(x.shape)
sequential_model.build(x.shape)

# train both models on the same data
data = np.random.random((512, 32, 1))  # just some random data to map to itself during training
h = []
for m in [subclassed_model, sequential_model]:
    m.compile(loss='MSE', optimizer='Adam')
    history = m.fit(x=data, y=data, epochs=100, batch_size=32)
    h.append(history)

# get difference between loss values
diff = list(map(lambda item: abs(item[0] - item[1]), zip(h[0].history['loss'], h[1].history['loss'])))

assert not any(diff), f"[{backend()} backend] Deviation between Sequential() and subclassed model"

Try this one:
import os

os.environ["KERAS_BACKEND"] = "jax"

import numpy as np

from keras import Model
from keras import Sequential
from keras import backend
from keras import initializers
from keras import layers
from keras import ops

conv_kwargs = dict(
    filters=1,
    kernel_size=3,
    padding="same",
    kernel_initializer=initializers.HeNormal(seed=42),
)

class MyModel(Model):
    def __init__(self):
        super().__init__()
        self.layer = layers.Conv1D(**conv_kwargs)

    def build(self, input_shape):
        self.layer.build(input_shape)

    def call(self, inputs):
        return self.layer(inputs)

subclassed_model = MyModel()
sequential_model = Sequential([layers.Conv1D(**conv_kwargs)])

# build models
subclassed_model.build((1, 32, 1))
sequential_model.build((1, 32, 1))
for a, b in zip(subclassed_model.variables, sequential_model.variables):
    np.testing.assert_allclose(ops.convert_to_numpy(a), ops.convert_to_numpy(b))

# train both models on the same data
data = np.random.random((512, 32, 1))
h = []
for m in [subclassed_model, sequential_model]:
    # JAX backend internally uses `np.random.permutation` to shuffle the dataset
    np.random.seed(42)
    m.compile(loss="MSE", optimizer="Adam")
    history = m.fit(x=data, y=data, epochs=100, batch_size=32)
    h.append(history)

# get difference between loss values
diff = list(
    map(
        lambda item: abs(item[0] - item[1]),
        zip(h[0].history["loss"], h[1].history["loss"]),
    )
)

assert not any(diff), (
    f"[{backend.backend()} backend] Deviation between Sequential() and "
    "subclassed model"
)

print(
    f"[{backend.backend()} backend] No deviation between Sequential() and "
    "subclassed model"
)

I got:
[jax backend] No deviation between Sequential() and subclassed model

The root cause is that initializer should be seeded and np.random.seed should be called before every fit since JAX backend internally uses np.random.permutation to shuffle the dataset

Hi @james77777778, thanks a lot for looking into my code sample. That's an interesting observation. I used utils.set_random_seed(1) in my code which I expected to set the seed globally for all keras related stuff. This example provided by the keras team explicitly states that calling utils.set_random_seed() sets

numpy random seed
backend (in my case jax)
python random seed

The weight initialization in my code is deterministic without explicitly setting the seed for the initializer. The shuffling of the dataset obviously not. Is this a bug? I tested on keras 3.7.0 btw.

I don't think it's a bug. It's expected that multiple calls to np.random.* with a single np.random.seed will produce different outputs.
A demo here:
import numpy as np

np.random.seed(42)
print(np.random.uniform(size=()))  # 0.3745401188473625
print(np.random.uniform(size=()))  # 0.9507143064099162

np.random.seed(42)
print(np.random.uniform(size=()))  # 0.3745401188473625

So, theoretically, your code trains the model without deviation if you train one model at a time. Just make sure not to train both models within the same script.

Ah i see, that makes total sense. Thanks for the clarification!


CustomMetadata from the features is ignored in 1.19.0

In our use case we execute tests on separated embedded devices, adding custom Metadata at the json file level.
The json files are then stashed in our pipeline to be gathered for report generation.
Up until version 1.18.3 this worked flawlessly with the feature overview being populated with the custom metadata. On version 1.19.0 the feature overview does not include any of the custom metadata we insert in the json files.
Documentation-wise for metadata it is mentioned that the data from the features takes precedence over the passed metadata. I think that should be the same for customMetadata. However, this commit introduced in version 1.19.0 does not execute any kind of check over the feature.metadata. Instead it assigns options.metadata directly.
Environment (please complete the following information):

multiple-cucumber-html-reporter: 1.19.0
Node.js version: 12.22.7
Yarl version: 1.22.15
Platform name and version: Linux (amx64/armhf)
Cucumber version: 6.0.5

Expected behavior
The metadata from the feature should take precedence over the metadata passed as parameter. As mentioned the metadata value can vary a lot in some cases like ours and it doesn't make sense to set an overall custom metadata at all but instead rely on the metadata already available in the files.

I've released a new version, thanks for your contribution


ci: move from prettier to forge fmt

forge now supports formatting. we should replace prettier with this to reduce our dependency on JS.

Hello! Can I work on this issue?

all yours!

Update: it seems that the fmt tool documentation doesn't contain some flags/features that were recently developed.
One of them is needed to ensure that the function parameters are kept multiline instead of being clubbed together to fit a single line.
I am following up on that with them here!

if the function still respects line length, i think its fine to merge for now and make that update later when forge supports it

Hello! I just created a pull request for this issue here!
Requesting feedback!


Change default language

Is possible to change the default language (from English to Portuguese for example)? I wanted to make English as the second translation option, not the main one.

@guigaoliveira At this point, English is the default shown language. But you are correct that we should have an option to set what the default language is.
We actually have this as a feature request in https://github.com/facebook/Docusaurus/issues/586 - @endiliey is this something we should still target before v2?
Of course, the if you don't have translations on, you can just write your docs in Portuguese -- but that isn't the best user experience, obviously if you want multiple languages.

I am going to close this as a duplicate of #586  - but feel free to comment there as well.

@JoelMarcey I'm targeting that on v2 to not hardcode english as default language.
One thing to note is that in current version, all pages are located in pages/en/*.js, it should be pages/**.js in v2


Zones panel

Farm designer panel and map functionality for creating/viewing/editing zones.

Empty state graphic and nav icon: Zones Graphics.zip

have you added any feature to define safe zones or danger zones in farmbot? It is almost a year since you said it is "Ideas for the future"

@dr1khateeb We don't have plans to start work on the safe zone feature yet. There are still higher priority items we must work on and FarmBot only has two software developers (including myself). The "Ideas for the Future" column is not necessarily a list of priorities or deadlines, and I typically don't provide time estimates for things that are more than a month into the future. We definitely see the usefulness of the feature, but we need to balance that with demand for other features and a very small team size.
In 2020 our priorities are:

Reduce the number of bugs and stability issues.
Release v2 of the weed detector.
Increase usability for non-technical users.


Expose some sensors only

Hi @ebaauw,
Is it possible to expose some of the sensors and not others?
I'd like to expose the motion sensors, only for being able to disable them with homekit, without having all my accessories exposed twice (natively and through your plugin).
Thanks.

Yes, specify excludeSensorTypes in config.json (see https://github.com/ebaauw/homebridge-hue/wiki/Configuration) and/or create a resroucelink on the bridge as blacklist, see #170.

Thank you @ebaauw. I found the right type in the https://github.com/ebaauw/homebridge-hue/wiki/Services section of the wiki.


Implement dynamically updating gold standard list

@xyg123 please add details.
Related to old issue - https://github.com/opentargets/issues/issues/3526

Based on a conversation with @addramir :

We have agreed that the process of defining GS will be done manually when we have the final feature matrix (FM) and effector gene list (EGL). So we will assign credible sets to EGL and define GSP and GSN. This process should be part of ETL, yes, but only because we have limited time - we have to do it manually for the release. After that we have time to automate it and make it part of ETL.

25/10_manual_egl_yakov
First iteration of a dynamic gold standard list.
Positives were generated manually by @addramir. Positives are made with a list of effector genes derived from the gold standard annotation in production. 232 positives were expanded with negatives following the same approach we have now in Gentropy (no cherrypicking).
W&B Run:
https://wandb.ai/open-targets/gentropy-locus-to-gene/runs/yyfh2qok?nw=nwuseropentargets
The confusion matrix shows less bias towards the negative set, but the precision is still not good, probably due to the training set size.
Predictions are in gs://ot-team/irene/dynamic_l2g/25-10_manual_egl_yakov

Code to reproduce

import pyspark.sql.functions as f
from sklearn.ensemble import GradientBoostingClassifier
from wandb import login as wandb_login

from gentropy.common.session import Session
from gentropy.common.utils import access_gcp_secret
from gentropy.dataset.l2g_feature_matrix import L2GFeatureMatrix
from gentropy.dataset.l2g_gold_standard import L2GGoldStandard
from gentropy.dataset.l2g_prediction import L2GPrediction
from gentropy.dataset.study_locus import StudyLocus
from gentropy.dataset.variant_index import VariantIndex
from gentropy.datasource.open_targets.l2g_gold_standard import (
    OpenTargetsL2GGoldStandard,
)
from gentropy.method.l2g.model import LocusToGeneModel
from gentropy.method.l2g.trainer import LocusToGeneTrainer

session = Session("yarn")

## Build gold standard list

variant_index = VariantIndex.from_parquet(
    session, "gs://ot_orchestration/releases/24.10_freeze4/variant_index"
)
credible_set = StudyLocus.from_parquet(
    session,
    "gs://ot_orchestration/releases/24.10_freeze4/credible_set",
    recursiveFileLookup=True,
)
features_list = [
    # max CLPP for each (study, locus, gene) aggregating over a specific qtl type
    "eQtlColocClppMaximum",
    "pQtlColocClppMaximum",
    "sQtlColocClppMaximum",
    # max H4 for each (study, locus, gene) aggregating over a specific qtl type
    "eQtlColocH4Maximum",
    "pQtlColocH4Maximum",
    "sQtlColocH4Maximum",
    # max CLPP for each (study, locus, gene) aggregating over a specific qtl type and in relation with the mean in the vicinity
    "eQtlColocClppMaximumNeighbourhood",
    "pQtlColocClppMaximumNeighbourhood",
    "sQtlColocClppMaximumNeighbourhood",
    # max H4 for each (study, locus, gene) aggregating over a specific qtl type and in relation with the mean in the vicinity
    "eQtlColocH4MaximumNeighbourhood",
    "pQtlColocH4MaximumNeighbourhood",
    "sQtlColocH4MaximumNeighbourhood",
    # distance to gene footprint
    "distanceSentinelFootprint",
    "distanceSentinelFootprintNeighbourhood",
    "distanceFootprintMean",
    "distanceFootprintMeanNeighbourhood",
    # distance to gene tss
    "distanceTssMean",
    "distanceTssMeanNeighbourhood",
    "distanceSentinelTss",
    "distanceSentinelTssNeighbourhood",
    # vep
    "vepMaximum",
    "vepMaximumNeighbourhood",
    "vepMean",
    "vepMeanNeighbourhood",
]
full_fm = L2GFeatureMatrix(
    _df=session.load_data(
        "gs://ot_orchestration/releases/24.10_freeze4/locus_to_gene_feature_matrix"
    ),
    features_list=features_list,
)

positives_df = (
    session.load_data(
        "gs://genetics-portal-dev-analysis/yt4/20241024_EGL_playground/GSP_cs_selected.parquet"
    )
    .join(
        credible_set.df.select("studyLocusId", "variantId"),
        "studyLocusId",
    )
    .select(
        "studyId",
        "studyLocusId",
        "variantId",
        "geneId",
        f.lit("positive").alias("goldStandardSet"),
    )
    .distinct()
)

positives_negatives_df = OpenTargetsL2GGoldStandard.expand_gold_standard_with_negatives(
    positives_df, variant_index
)

gs = L2GGoldStandard(_df=positives_negatives_df, _schema=L2GGoldStandard.get_schema())

fm = gs.build_feature_matrix(full_fm, credible_set).fill_na()

# Train
wandb_key = access_gcp_secret("wandb-key", "open-targets-genetics-dev")

l2g_model = LocusToGeneModel(
    model=GradientBoostingClassifier(random_state=42),
    hyperparameters={
        "n_estimators": 100,
        "max_depth": 5,
        "loss": "log_loss",
    },
)
wandb_login(key=wandb_key)
trained_model = LocusToGeneTrainer(
    model=l2g_model,
    feature_matrix=fm,
).train("25/10_manual_egl_yakov")

predictions = L2GPrediction.from_credible_set(
    session,
    credible_set,
    full_fm,
    features_list,
    model_path=None,
    hf_token=access_gcp_secret("hfhub-key", "open-targets-genetics-dev"),
    download_from_hub=True,
)

Based on a discussion with @d0choa and @addramir, we want to fine tune the list of gene/EFO pairs that we extract from ChEMBL to build the effector gene list to include pairs that have been clinically validated.
👉 gs://ot-team/irene/l2g/validation/chembl_w_flags
For that I have prepared a dataset based on the ChEMBL target/disease evidence where I have added 2 columns:

isApproved: True when the target/disease pair is validated by a drug approved for that condition specifically.  We want to filter associations that haven't been clinically validated.

+----------+------+
|isApproved| count|
+----------+------+
|      true|108766|
|     false|102419|
+----------+------+

isComplex: True when the target is part of a protein complex. Because we don't have protein complexes in Open Targets, for a drug whose mechanism of action is to target a protein complex, we link all. Potentially we want to filter these associations out when calculating enrichment of approved targets.

Here's some examples of the list of target types I have flagged.

TYPE: protein family
+----------------------------+---------------+--------------------------------------------------------------------+
|targetName                  |chemblIds      |targets                                                             |
+----------------------------+---------------+--------------------------------------------------------------------+
|Alpha glucosidase           |[CHEMBL2110737]|[ENSG00000257335, ENSG00000171298, ENSG00000090402, ENSG00000257743]|
|Serotonin 2 (5-HT2) receptor|[CHEMBL304902] |[ENSG00000135914, ENSG00000102468, ENSG00000147246]                 |
+----------------------------+---------------+--------------------------------------------------------------------+
only showing top 2 rows

TYPE: protein complex group
+------------------------------+-----------------------+--------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------+
|targetName                    |chemblIds              |targets                                                                                |
+------------------------------+-----------------------+--------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------+
|GABA-A receptor; anion channel|[CHEMBL1256]           |[ENSG00000022355, ENSG00000145864, ENSG00000113327, ENSG00000166206, ENSG00000109158, ENSG00000182256, ENSG00000268089, ENSG00000011677, ENSG00000094755, ENSG00000187730, ENSG00000163285, ENSG00000163288, ENSG00000102287, ENSG00000151834, ENSG00000186297, ENSG00000145863]|
|Glutamate [NMDA] receptor     |[CHEMBL1714, CHEMBL742]|[ENSG00000176884, ENSG00000183454, ENSG00000273079, ENSG00000105464, ENSG00000116032, ENSG00000161509, ENSG00000198785]                                                                                |
+------------------------------+-----------------------+--------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------+
only showing top 2 rows

TYPE: protein complex
+-------------------------------+---------------------------+----------------------------------+
|targetName                     |chemblIds                  |targets                           |
+-------------------------------+---------------------------+----------------------------------+
|Sulfonylurea receptor 1, Kir6.2|[CHEMBL1589]               |[ENSG00000187486, ENSG00000006071]|
|GABA-B receptor                |[CHEMBL4594395, CHEMBL1342]|[ENSG00000204681, ENSG00000136928]|
+-------------------------------+---------------------------+----------------------------------+

+---------+------+
|isComplex| count|
+---------+------+
|     true|145758|
|    false| 65427|
+---------+------+

cc @xyg123

Hm...
But what should we do with complexes?
If we filter it by f.col("isApproved")=="true") & (f.col("isComplex")=="false") it will give very little number of ego-gene pairs. After distinct by EFO-gene - only 2769.

Covered by https://github.com/opentargets/issues/issues/3525


request help:  failed to record status change for ApisixRoute

Issue description
When creating a new ApisixRoute resource, the error is reported as follows:
ingress/status.go:91	failed to record status change for ApisixRoute	{"error": "Operation cannot be fulfilled on apisixroutes.apisix.apache.org \"api-product-v1\": the object has been modified; please apply your changes to the latest version and try again", "name": "api-product-v1", "namespace": "app"}
Use the latest version of CRD, but with some modifications.  diff file:

Environment

your apisix-ingress-controller: v1.1.0
your Kubernetes cluster version: v1.16.9

@Donghui0 Is this issue related to https://github.com/apache/apisix-ingress-controller/pull/609?

@Donghui0 Is this issue related to #609?

yep~

I found a scenario that can reproduce your problem.
When two controllers in the cluster change the status of the ApisixRoute resource at the same time, this error will be reported.
Please check if there are more than two controllers in your test environment.

I found a scenario that can reproduce your problem.
When two controllers in the cluster change the status of the ApisixRoute resource at the same time, this error will be reported.
Please check if there are more than two controllers in your test environment.

Yes. There are indeed multiple controllers in the same cluster. It seems that there are still some problems with running multiple controllers in the same cluster.

I found a scenario that can reproduce your problem.
When two controllers in the cluster change the status of the ApisixRoute resource at the same time, this error will be reported.
Please check if there are more than two controllers in your test environment.

Yes. There are indeed multiple controllers in the same cluster. It seems that there are still some problems with running multiple controllers in the same cluster.

The Ingress controller uses the election mechanism to determine which controller is effective. When configuring, ensure that only one controller handles one resource.

is my problem.
The other controller is fully watched.

I've read this commit. I don't understand why this issue is related to that commit.

@Donghui0 Is this issue related to #609?

yep~

Hi, I've read this commit. But I don't understand why this issue is related to that commit.


Update IsDependency to show direct/indirect and toolchain to utilize that

In light of the discussion in https://github.com/google/deps.dev/issues/12, it looks like we are able to parse the entire structure of the transitive depedencies from one deps.dev call.
These are the set of things that need to be worked on

[GraphQL] Add direct/indirect/unknown enum for IsDependency
[Ingestion/Parsers] Update parsers for IsDependency to include direct/indirect/unknown info
[Deps.dev collector] Update parsing for deps.dev to include structure of the tree
[guac-visualizer] Update to read version ranged properly (for  badcertify demo)
[guacone queryvuln] Update guacone to read version ranged properly (for query_vuln demo)
[Deps.dev collector] (optional) enable Collectsub identifier strings

Completed Items via #778:

Added direct/indirect/unknown enum for IsDependency
[Deps.dev Parser] Updated parser for IsDependency to include direct/indirect/unknown info
[Deps.dev collector] Updated parsing for deps.dev to include structure of the tree
[Deps.dev collector] enabled Collectsub identifier strings

Remaining Items:

Update the SPDX and CDX parsers to properly use direct/indirect/unknown info in IsDependency
[guac-visualizer] Update to read version ranged properly (for badcertify demo)
[guacone queryvuln] Update guacone to read version ranged properly (for query_vuln demo)

Here are the various outputs that can be received for the version range from deps.dev for the various "systems":
curl -s https://api.deps.dev/v3alpha/systems/npm/packages/react/versions/17.0.0:dependencies | jq
curl -s https://api.deps.dev/v3alpha/systems/cargo/packages/bincode/versions/1.3.3:dependencies | jq
curl -s https://api.deps.dev/v3alpha/systems/maven/packages/com.cloudhopper:ch-mq/versions/3.0.3:dependencies | jq
curl -s https://api.deps.dev/v3alpha/systems/pypi/packages/pytest/versions/7.3.1:dependencies | jq
curl -s https://api.deps.dev/v3alpha/systems/go/packages/cloud.google.com%2Fgo/versions/v0.102.1:dependencies | jq

We have revamped the UI to not handle this logic, so this issue should be complete


[Feature] waitForEvent("dialog")

Hi, In Playwright-TS, we have a function - page.waitForEvent("Dialog");
it will wait for the dialog (JS Alert), unfortunately, it is missing in Playwright-JAVA.
Why it is required?
To achieve explicit wait for an alert.
Sample code:
package org.example;

import com.microsoft.playwright.*;
import java.nio.file.Paths;

public class WebKitScreenshot {
  public static void main(String[] args) {
    try (Playwright playwright = Playwright.create()) {
      Browser browser = playwright.chromium().launch(new BrowserType.LaunchOptions().setHeadless(true));
      Page page = browser.newPage();
      page.navigate("https://letcode.in/waits");
      page.onDialog(dialog -> {
          System.out.println(dialog.message());
          dialog.accept();
      });
      page.locator("id=accept").click();
      page.waitForTimeout(6_000);
    }
  }
}

Expected: I want to achieve the same but without waitForTimeout.
Created this request as per the conversation with Max.
https://playwright.slack.com/archives/CSUHZPVLM/p1671698876739829

The dialogs are modal and must be handled synchronously within onDialog listener, otherwise subsequent calls to playwright API may hang because the page is blocked on the modal dialog and cannot handle any events. E.g. if click triggers navigation and a modal dialog then the following call would hang:
Dialog dialog = page.waitForDialog(() -> page.getByText('submit').click());

Ideally, we'd remove waitForEvent('dialog') in node.js too but that ship has sailed.
Re: your question about waiting for a dialog, you can do something like this:
    boolean[] didShowDialog = {false};
    page.onDialog(dialog -> {
      System.out.println(dialog.message());
      dialog.accept();
      didShowDialog[0] = true;
    });
    page.locator("id=accept").click();
    while (!didShowDialog[0]) {
      page.waitForTimeout(100);
    }


Ipy v2

Yet another copy of the componentizer script, this one is almost identical with the new CPython one but compiles components as for the new IronPython interpreter of Rhino8.
Necessary changes are visible in https://github.com/compas-dev/compas-actions.ghpython_components/commit/42a8ef452828eb591b7bb4e8ed1f8c13e8959155
@9and3 couldn't add you as reviewer, but would be great if you could have a look.
One thing I noticed is Rhino8 seems to be taking the input/output argument names from the metadata and replaces the RunScript args with them. This means that if argument names are different in the script itself (e.g. Centerline vs. centerline) the script will be broken..
It would be nice not to have to change components to match, but I currently don't see any other way..

Lookin good @chenkasirer thanks for tag me! 🌠 just a tiny missing datatype for RV8. As mentioned by @gonzalocasas the action (and readme) should be updated to have an additional interpreter otpion where in the workflow there would be:
- uses: compas-dev/compas-actions.ghpython_components@v5
        with:
          source: components
          target: build
          interpreter: ipyV8  # or something like this

And the action.yml would have just another case:
https://github.com/compas-dev/compas-actions.ghpython_components/blob/42a8ef452828eb591b7bb4e8ed1f8c13e8959155/action.yml#L25-L41

@9and3 thanks so much for taking a look!
I missed a bunch of guids that changes etc, but I think I got it right now.
What do we think about ipy_v2 instead of ipyV8? I tend to like ipy_v2 better..

@gonzalocasas @9and3 btw in the course of making this I made a small script which unpacks a compiled component and writes it as XML + code as python. it was really helpful to find out what I was doing wrong.
Not sure where to put it so made a gist: https://gist.github.com/chenkasirer/986edeb23f8e9ce6cf34fb0d4eb7ff9e

@gonzalocasas @9and3 btw in the course of making this I made a small script which unpacks a compiled component and writes it as XML + code as python. it was really helpful to find out what I was doing wrong.
Not sure where to put it so made a gist: https://gist.github.com/chenkasirer/986edeb23f8e9ce6cf34fb0d4eb7ff9e

That's great!!!

Lookin good @chenkasirer thanks for tag me! 🌠 just a tiny missing datatype for RV8. As mentioned by @gonzalocasas the action (and readme) should be updated to have an additional interpreter otpion where in the workflow there would be:
- uses: compas-dev/compas-actions.ghpython_components@v5
        with:
          source: components
          target: build
          interpreter: ipyV8  # or something like this

And the action.yml would have just another case:
https://github.com/compas-dev/compas-actions.ghpython_components/blob/42a8ef452828eb591b7bb4e8ed1f8c13e8959155/action.yml#L25-L41

@9and3 thanks so much for taking a look!
I missed a bunch of guids that changes etc, but I think I got it right now.
What do we think about ipy_v2 instead of ipyV8? I tend to like ipy_v2 better..

Shouldn't we use the legacy keyword to identify the old stuff like rhino does? ie cpython and ironpython for the new editor and ironpython_legacy for the old one?

Shouldn't we use the legacy keyword to identify the old stuff like rhino does? ie cpython and ironpython for the new editor and ironpython_legacy for the old one?

indeed!

Shouldn't we use the legacy keyword to identify the old stuff like rhino does? ie cpython and ironpython for the new editor and ironpython_legacy for the old one?

I like this suggestion.
What I'm worried about it that it would make IronPython2 components the default that are packaged with releases. These aren't backwards compatible. We need to come up with some mechanism that lets us package all versions with release and lets typical users a way to choose which ones to install (as Rhino7 and 8 share the same component directory, you can really have only one or another installed).

@gonzalocasas @9and3 so what do we say? I think we should leave the default behavior to be the legacy one to not break Rhino7 compatibility (this action is used directly from main..)
Once we have a good strategy of dealing with the different environments we can do this rework.


Translate feature-trans-app.md already merged to add/japanese

about
Translation of Transport and application level
ref
#20

about
Translation of Transport and application level
ref
bagder#20

間違えました


fix: Refactor logic to download form schema from the ConfirmFormDeleteDialog

Summary | Résumé
Closes

https://github.com/cds-snc/platform-forms-client/issues/3160
https://github.com/cds-snc/platform-forms-client/issues/3199

The ConfirmFormDeleteDialog was using the DownloadFileButton, which relies on the Template Store context.

Presently, this context is injected only in the Form-Builder layout, not in the My Forms or Admin Layout configuration. And, the ConfirmFormDeleteDialog is exclusively used in the last two groups (tbc).
Solution: Altered the logic in the dialog by incorporating an inline button to download the form schema from the backend instead of relying on the store.
Test instructions | Instructions pour tester la modification
Find a form to delete, the dialog and download a copy should work.

Noting - this fixes that "Missing Template Store Provider" error, but there seems to be another (unrelated?) bug- after the dialog closes, the /forms screen does not refresh so it appears as if the form wasn't deleted, have to do a hard refresh to clear it.

@dsamojlenko ill merge this PR and open a new issue


Option to have 2 layers instead of 3

Title says it all.
Example: Mobile version of https://www.apple.com

The hamburger concept "say" that it should be 3.

+1, 2 layers style looks much better in some designs, also it's good to have more options than less.

Also still waiting on this.

.hamburger-inner::before {
top: 0;
}
and then adjust padding

May consider this for a v2


Only One Shortest Path?

my environment running ArangoDB
I'm using the latest ArangoDB of the respective release series:

[ ] 2.8
[ ] 3.0
[ ] 3.1
[x] 3.2
[ ] self-compiled devel branch

Mode:

[ ] Cluster
[x] Single-Server

Storage-Engine:

[ ] mmfiles
[x] rocksdb

On this operating system:

[ ] DCOS on

[ ] AWS
[ ] Azure
[ ] own infrastructure

[ ] Linux

[ ] Debian .deb
[ ] Ubuntu .deb
[ ] SUSE   .rpm
[x] RedHat .rpm
[ ] Fedora .rpm
[ ] Gentoo
[ ] docker - official docker library
[ ] other:

[ ] Windows, version:
[ ] MacOS, version:

I want to get the shortest path between two vertexs, so I use the SHORTEST_PATH. But it seems to return only one shortest path? If there are many shortest path, how can I get them all?

Hello, currently SHORTEST_PATH provides only one shortest path. It is possible to get all shortest paths with two queries though. At first you use SHORTEST_PATH to calculate the length of your shortest path by:
LET length = SUM(FOR v IN SHORTEST_PATH @start TO @target @@edges RETURN 1)

Next do a graph traversal with the calculated length filtering by your targetVertex. Length has to be a known value, not a variable:
FOR v,e,p IN @length OUTBOUND @start @@edges FILTER v._id == @target RETURN p

@maxkernbach It's a good idea, thank you so much.

@maxkernbach please add this to the documentation with the query patterns at https://github.com/arangodb/arangodb/tree/devel/Documentation/Books/AQL/Examples

@maxkernbach
Using SUM() seems unnecessary. Instead, you could count the number of array elements LENGTH(...).
The RETURN 1 hack shouldn't be necessary, I've even seen queries that performed slightly better if they used RETURN v.
@mchacki says v._id would be a bit slower than p.vertices[-1]._id, although I couldn't find a measurable difference so far.

Same question https://stackoverflow.com/questions/46979305/multiple-path-search-in-arangodb with a slightly modified answer by me. Should definately make its way into the docs.

Multiple path search is now documented in the section AQL -> Usual Query Patterns.


Add optional Internal Zone setup to VPC

Requires that a new release of troposphere be released for this checkin:
https://github.com/cloudtools/troposphere/commit/cdd1b271030d26312c51fdc53ab04bc3aa6fc8b7

:+1:


Port mapping missing in docker-compose for local

While testing the latest Geo tax image using the helm repo, I found that the port configuration is missing in the YAML file:

version: "3.9"
services:
precisely-geotax-svc:
.
.
.
ports:
- "${_SERVICE_PORT:-8080}:8080"

This is to confirm the issue has been resolved!!


chore: Remove en-us from azure.microsoft.com links

These could also be re-written as docs.microsoft.com/azure

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

Verified removal of locale specifiers.
Resolved merge conflict with preference for version of content in master.


Where is my art in the gallery?

I don't see my art in the gallery!

Will update the gallery with new art pieces this week. Thanks for pointing out it was missing.


compare host using lowercase

use lowercase comparison for host lookup #542

@leprechau my Go skills are pretty basic and I had suggested this change as it worked for me.
(I have tested the binary for my use case)
I am perfectly fine with any other changes as well.
Should I resubmit with the other changes?

I think we can make all lookups and adds lowercase by making two simple changes in route/table.go like this ...
diff --git a/route/table.go b/route/table.go
index bb689dc..1646e74 100644
--- a/route/table.go
+++ b/route/table.go
@@ -136,6 +136,7 @@ func NewTable(s string) (t Table, err error) {
 // addRoute adds a new route prefix -> target for the given service.
 func (t Table) addRoute(d *RouteDef) error {
        host, path := hostpath(d.Src)
+       host = strings.ToLower(host) // maintain compatibility with parseURLPrefixTag

        if d.Src == "" {
                return errInvalidPrefix
@@ -383,6 +384,7 @@ func (t Table) LookupHost(host string, pick picker) *Target {
 }

 func (t Table) lookup(host, path, trace string, pick picker, match matcher) *Target {
+       host = strings.ToLower(host) // routes are always added lowercase
        for _, r := range t[host] {
                if match(path, r) {
                        n := len(r.Targets)

Running a make test those two changes to table.go don't appear to break existing tests.

@shantanugadgil Thank you!

hehe @leprechau thank you !!! 👍


Doesn't run on 13'' MacBook Pro 2016

I am using macOS 10.12.6 and I tried a couple of versions of Karabiner Elements including the latest and some old versions but every time when I launch the app, it shows only "Karabiner-Elements quit unexpectedly". But for my iMac, which is running the same version of macOS, works smoothly.

Finally, I found the answer. If you have the same issue, try to change the "Sharing & Permissions" settings in your User info to "Read & Write" instead of "Read Only". Save the change by hitting "Apply to enclosed items". Then execute the following command in your terminal:
diskutil resetUserPermissions / id -u
And you are done!
Hope this could help.

Where should I change the sharing & permissions settings? On the Applications folder in Finder?
And I am getting an error message about the incorrect number of parameters when I to run diskutil

The "Sharing & Permissions" is at the bottom of the info panel of your home folder. You can find it in the sidebar of Finder.

Home folder. Okay. I see it on every folder, so I didn't know if it had to be a specific one. Thanks. But I'm already Read & Write.
And then when I try to run the diskutil command I get:
Scotts-MacBook-Pro:~ scott$ diskutil resetUserPermissions / id -u
Incorrect number of parameters
Usage:  diskutil resetUserPermissions MountPoint|DiskIdentifier|DeviceNode UID
Reset the permissions of a user home directory.
Ownership of the affected disk is required.

May be you should check the original solution to see if you miss any steps:
https://support.apple.com/en-us/HT203538

Thanks! That link did it.
The command might be better formatted as diskutil resetUserPermissions / id -u``

My bad. Glad you fixed it!


Handle with \0 at the end.

Hi there,
I recently include this lib into my project. I facing an issue when a gearman worker crashed during the execution of a non-async job.
Logs :
  [Zikarsky\React\Gearman\Command\Exception]
  Unexpected WORK_FAIL(14)[job_handle=H:iMac-de-loconox.local:89\0]. Task unknown

(I rewrote the \0 at the end because it doesn't print correctly into my terminal)
The problem is that the gearman server responds with an handle name ending with \0 whereas tasks in array in Zikarsky\React\Gearman\Client are stored without this \0.

I think I have found the issue. See https://github.com/loconox/react-gearman/commit/dc374688293c67cba277f7521dfc53ccfd68d1e0

Strange. The spec defines WORK_FAIL with only a single argument 'job_handle'. The \0 is the standard delimiter for arguments and shouldn't cause trouble. Can you maybe try to define a testcase for the failure, so we can fix against a unittest?

Okay, I had a deeper look into this:
The problem is in an incorrect usage of the $limit-parameter of PHP's explode() in ..\ReadBuffer::handleBody.
This breaks the parsing of every command's last parameter if it is not DATA (maybe even data, it did not verify this yet). I demonstrated the problem in https://3v4l.org/EK67j.
The explode-call is limited to the number of arguments defined in the command-definition (which is 1 for WORK_FAIL). This allows binary DATA to contain \0-bytes, but doesn't remove the trailing \0 (used as an argument-delimiter) on other argument-types.
My proposed fix for this problem is:

Verify: Does the \0-byte also get added to DATA-arguments?
If yes: Just keep the existing explode-limit-logic, but use substr($buffer, 0, .-1) to exclude the trailing \0
If no: Use the yes variant, but find an elegant way to handle the DATA-exception

This also means we need a test for those edge-cases in ReadBufferTest.
I'll close your PR unmerged for now. Let me know if you want to work on the fix for this deeper issue or if you want me to allocate some time next week :smile:.

Well, I re-read the the doc and concluded that this could be a bug in the server implementation or in the doc. http://gearman.org/protocol/
Since \0 is the argument delimiter, their should not has a \0 at the end of a WORK_FAIL respond. For example, in the case of a JOB_CREATED response, where the last argument is a job_handle, there isn't \0 at the end.
First case, JOB_CREATED followed by WORK_COMPLETE :
\x00RES\x00\x00\x00\x08\x00\x00\x00\x19H:iMac-de-loconox.local:5
\x00RES\x00\x00\x00\r\x00\x00\x00\x1EH:iMac-de-loconox.local:5\x00toto

Second case, JOB_CREATED followed by WORK_FAIL :
\x00RES\x00\x00\x00\x08\x00\x00\x00\x19H:iMac-de-loconox.local:7
\x00RES\x00\x00\x00\x0E\x00\x00\x00\x1AH:iMac-de-loconox.local:7\x00

I agree. That's pretty much the same conclusion I had. What do you propose?
Maybe just adding DATA to the command-definition of WORK_FAIL is the right fix. In that case we should mention in a comment that some gearmand implementations behave this way, even though it's not in the protocol definition.

I've opened an issue in the server repo to know their opinion. https://github.com/gearman/gearmand/issues/108

Yep, as I fixed this issue in my repo https://github.com/loconox/react-gearman/tree/fix-workfail I suggest to add a DATA argument to the WORK_FAIL response and add a commentary to explain why. I'm updating my branch and make a PR.

Great, thanks!


Wireframe to code must be upgraded to newer gpt model (gpt-4-vision-preview depricated)

I get the following issue when trying to generate code from a wireframe:

Not sure how hard it would be to migrate to gpt-4o but this issue makes the "Wireframe to code"-feature unusable.

From what I quickly read on the openAI Vision API website the api seems compatible with what is currently implemented:
https://github.com/excalidraw/excalidraw/blob/4dc4590f247a0a0d9c3f5d39fe09c00c5cef87bf/packages/excalidraw/data/magic.ts#L50-L104
Seems like all that is needed is to change the model to "gpt-4o"?

I got it working by only changing the model to "gpt-4o".
However, I found another issue in the process. Wireframes inside a frame will just generate a generic website:

A totally unrelated issue but also kind of awkward... :/

We're actually in the process of moving away from bring-your-own-token workflow as it was too complicated and introduced it early only due to extremely low API limits on OpenAI's part. Further, it's limiting us to only use a specific service if we wanted to do something more dynamic.
We'll be updating this soon.

I'm using the wireframe to code feature in Excalidraw, but I'm getting this error message:
Error!
The model gpt-4-vision-preview has been deprecated, learn more here: https://platform.openai.com/docs/deprecati
I also tried changing the default vision model to gpt-4o, but I'm still getting the same error. Can help me with this issue?"

As @dwelle said, they will be updating this soon.
If you need it right away and have a ChatGPT 4o token (you need to pay $5 on platform.openai.com I think) then you could try this version (the one generated from my pull request):
https://excalidraw-git-fork-maekoos-fix-8147-excalidraw.vercel.app/


Only 1 user can be connected

Deleted

type this "nano /etc/openvpn/server_tcp.conf"
add "duplicate-cn"
then save and reboot your server

Thank you so much

Not working very well? Any another method?

nope that mothod should work


Bar collapsing

Description
Under Pycharm Community Edition 2017.1.2 the progress bar becomes smaller as DynamicMessages are appended, even with plenty of space on the right (see Figures below, where in the second one a new DynamicMessages is appended). Is it a bug or is it something intended?

Code
The Widgets setting for the second example is the following:
widgets = [

    'Batch: ', progressbar.Counter(),
        '/', progressbar.FormatCustomText('%(total)s',{"total":batch_number}),
        ' ', progressbar.Bar(marker="-", left='[', right=']'),
        ' ', progressbar.ETA(),
        ' ',
         progressbar.DynamicMessage('loss'),
        ' ',
        progressbar.DynamicMessage("accuracy"),' ',
    progressbar.DynamicMessage("epoch")
    ]

Versions

Python version: 2.7.12
Python distribution/environment: VirtualEnv
Operating System: Ubuntu 16.04
Package version: 3.34.0

Sorry for the slow response, I didn't see your question earlier.
The problem here is that it's impossible (as far as I know) to detect the width of the PyCharm console. The fallback result is that the progressbar defaults to the "standard" console width of 79 characters.
You can work around it by manually specifying the terminal width:
bar = progressbar.ProgressBar(
    widgets=widgets,
    term_width=120,
)

That works like a charm! thank you!


Always calculate the resources with the sidecar containers

https://github.com/kubernetes/kubernetes/pull/116429#discussion_r1190156615

@tzneal
I found more things to remove.

@tzneal
I found more things to remove. I'll squash this PR with b1f32a7 and make it as small as possible.

Look at line 557 in plugin/pkg/admission/limitranger/admission.go , can't comment on the line as it wasn't part of the review.

Look at line 557 in plugin/pkg/admission/limitranger/admission.go , can't comment on the line as it wasn't part of the review.

done.

squashed these commits with the existing resource calculation commit as they are basically about reverting.


autoscroll not enabled after sending message

In fixing the general "autoscroll doesn't" problem, I broke the nice feature where sending a message turns it back on. Now this feature is harder to implement, and so I'm punting it until the next milestone.

I simply have no idea how to implement this.


docs: Update max_marginal_relevance_search api reference url in multi_vector.ipynb

Description: Update VectorStore max_marginal_relevance_search api reference url in multi_vector.ipynb

next time feel free to do these as a single PR


[Feature][Api] Refactor&Add api.controller.QueueV2Controller

Purpose of the pull request

Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

(or)
If your pull request contain incompatible change, you should also add it to docs/docs/en/guide/upgrede/incompatible.md

Codecov Report

Merging #11389 (64cc681) into dev (03582f2) will increase coverage by 0.04%.
The diff coverage is 73.17%.

@@             Coverage Diff              @@
##                dev   #11389      +/-   ##
============================================
+ Coverage     39.93%   39.97%   +0.04%
- Complexity     4599     4606       +7
============================================
  Files           967      973       +6
  Lines         36643    36665      +22
  Branches       4091     4083       -8
============================================
+ Hits          14633    14658      +25
+ Misses        20500    20496       -4
- Partials       1510     1511       +1

Impacted Files
Coverage Δ

...heduler/api/dto/queue/QueueListPagingResponse.java
0.00% <0.00%> (ø)

...hinscheduler/api/controller/QueueV2Controller.java
62.50% <62.50%> (ø)

...inscheduler/api/dto/queue/QueueCreateResponse.java
100.00% <100.00%> (ø)

...phinscheduler/api/dto/queue/QueueListResponse.java
100.00% <100.00%> (ø)

...inscheduler/api/dto/queue/QueueUpdateResponse.java
100.00% <100.00%> (ø)

...inscheduler/api/dto/queue/QueueVerifyResponse.java
100.00% <100.00%> (ø)

...er/master/dispatch/host/assign/RandomSelector.java
77.77% <0.00%> (-5.56%)
:arrow_down:

...dolphinscheduler/remote/future/ResponseFuture.java
81.96% <0.00%> (-1.64%)
:arrow_down:

...rver/master/runner/task/BlockingTaskProcessor.java
75.29% <0.00%> (-0.57%)
:arrow_down:

...eduler/api/service/impl/DataSourceServiceImpl.java
28.47% <0.00%> (ø)

... and 12 more

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


save_image.py dosen't work

Traceback (most recent call last):
File "C:\Users\33Lab\Documents\Dev\Graduation_project\edsdk-python\examples\save_image.py", line 5, in
import edsdk
File "C:\Users\33Lab\Documents\Dev\Graduation_project\edsdk-python\edsdk_init_.py", line 1, in
from edsdk.api import *
ModuleNotFoundError: No module named 'edsdk.api'
Process finished with exit code 1

Hi,
Same here, I think I did setup as required in the Readme.
@GiChiken, did you find a solution since you got the error ?

Same here any solution to this...

直接打开examples 避免导入源代码里面的sdk模块 因为已经安装过edsdk-python包 重复了


Marksy/components is removing new lines from children prop

If I have markdown that looks like this:
<SomeComponent>
text1, val1
text2, val2

</SomeComponent>

and a compiler that looks like this:
const compile = marksy({
  createElement: createElement,
  components: {
    SomeComponent(props) {
      console.log('props: ', props); // this returns { children: ["text1, val1 text2, val2"] }
    }
  },
})

The children prop doesn't have the new lines I typed as the component's children.
I get back an object that looks like this:
{
  children: ['text1, va1 text2, val2']
}

My new lines are gone.
Is there a way I can override this behavior so I can have the new lines in the children?

@Stanley-Jovel Hi there!
Not sure what you mean by "new lines"? In HTML you can not add new lines by adding a linebreak. Like:

  hello
   there

These will be compiled by the browser to:
hello there
. It is same with React. It does not make sense to have linebreaks because those linebreaks will not appear in the UI :)
But maybe you can explain a bit more what you are trying to achieve and maybe there is an other way?

You are right. What I'm trying to achieve is not supported by react.
This is what I want to do:
So I'm using marksy/components to integrate with my react app.
I wan't my users to be able to put CSV data as a children of a component like this:
Markdown:
<CSVSource>
field 1, value
field 2, value
</CSVSource>

CSVSource component reads this.props.children and splits it by new line \n
The problem with this approach is that when we use babel to transpile this markdown into react
https://github.com/cerebral/marksy/blob/9fd762d584097a38f5cf3f35a56d1ca0111f42a1/src/components.js#L25
babel removes my newlines from the children, and they become
React.createElement(CSVSource, {}, "field 1, value field2, value"
(the new lines were removed)
So the "problem" is not a problem and this is the expected behavior for JSX compilation.
However I can still achieve the behavior I want if I use props instead. Like so:
Markdown:
<CSVSource data='
field 1, value
field 2, value
'>
</CSVSource>

the problem with this is that if I declare the data prop using single quotes data='' then I can't nest single quotes inside my csv data, and I can't use double quotes inside double quotes either.

Well, if you need multiline string literals, you have to use backticks anyway:
<CSVSource data={`
field 1, value
field 2, value
`}/>

This should also work with children BTW:
<CSVSource>
{`
field 1, value
field 2, value
`}
</CSVSource>


[Deprecate]: icinga/icinga -> puppet/icinga

Deprecated module
https://forge.puppet.com/modules/icinga/icinga/
Replacement module
https://forge.puppet.com/modules/puppet/icinga
What authority do you have for making this request?
I am a Vox Pupuli PMC member
More information
Hi,
the repo got migrated from the previous owner and released under the vox pupuli namespace.

icinga-icinga has been marked as deprecated in favor of puppet-icinga


Fix PR workflow.

Closing - per @stack72, a fix is in works for the upstream repo that generates these workflows and we shouldn't fix on a one-off basis.


Suggestions

-Peepers fall off when the player is teleported
-More Customization, like which moons they spawn on

Moon-specific spawn rates is something I'll probably have by the 1.0 release. As for teleporting, I don't plan on adding that because they'd be a bit too easy to deal with that way.

I like the teleportation idea, with one of the following caveats:

The player could take some damage when the peepers are forcibly removed by teleportation. (10 damage?)
Teleportation removal is optional config, off by default.

When is the 1.0 release?

It may be after the next vanilla update unless I get a sudden burst of free time and inspiration.

Wondering if you will be updating to v50 beta?

I'll be updating it for when v50 leaves beta because they're likely incompatible.

is it updated for v50 with Moon spesific configs?

It's updated for v50, but still not 1.0.0 yet.


[SPARK-7234][SQL] Fix DateType mismatch when codegen on.

Can one of the admins verify this patch?

Jenkins, test this please.

Merged build triggered.

Merged build started.

Test build #31347 has started for   PR 5778 at commit 1ad4cff.

Test build #31347 has finished for   PR 5778 at commit 1ad4cff.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.
This patch does not change any dependencies.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/31347/
Test PASSed.

Merged build finished. Test PASSed.

Thanks. Merging in master & branch-1.3.


DSd-708: Fixing nested footer elements in TemplateAppContainer

Fixes JIRA ticket DSD-708
This PR does the following:

This fixes an issue when a custom component meant to render a <footer> element is passed to the TemplateAppContainer or TemplateFooter components. For the new renderFooterElement prop, when it is set to false, a default footer element won't be rendered. This fixes the nested HTML <footer> elements.

How has this been tested?
Storybook and tests.
Accessibility concerns or updates

Not anymore. It's not a syntax issue but a semantic issue to have nested footer elements.

Checklist:

[ ] I have updated the Storybook documentation accordingly.
[ ] I have added relevant accessibility documentation for this pull request.
[ ] All new and existing tests passed.

Front End Review:

[ ] View the example in Storybook

Tugboat has finished building the preview for this pull request!
Link:

https://pr841-88nea9geux34jyun5gy92lyaezq9qnlo.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/61ddf2885f46ed13d82415ec

Tugboat has finished building the preview for this pull request!
Link:

https://pr841-88nea9geux34jyun5gy92lyaezq9qnlo.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/61ddf2885f46ed13d82415ec

Tugboat has finished building the preview for this pull request!
Link:

https://pr841-88nea9geux34jyun5gy92lyaezq9qnlo.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/61ddf2885f46ed13d82415ec

Tugboat has finished building the preview for this pull request!
Link:

https://pr841-88nea9geux34jyun5gy92lyaezq9qnlo.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/61ddf2885f46ed13d82415ec


Example tweaks

I've done some work on the CARLA example to clean it up and explain what's going on, but more is needed. It has a bunch of configuration options whose behavior is difficult to understand, so those should either be explained or removed. Generally, it's totally OK to remove some options from the code if they're not interesting from the demo perspective. We should endeavor to make this example simple and informative, as it will be the main starting point for people to start doing their own integrations.

Since this has stalled for now, can we merge the existing changes? I think it's an improvement over what's currently there so there's no reason to hold off.


kotlin: 1.0.0-beta-3594 -> 1.0.0-beta-3595

Release notes: http://blog.jetbrains.com/kotlin/2015/12/kotlin-1-0-beta-3-is-out/
Build with nix-build -A kotlin, installed with nix-env -f . -iA kotlin.
Installed binaries work just fine.

Oh oh, I will try this again, I rebased horribly...


feat: Add integration to track sessions as app in foreground

Closes #712
This PR

Adds the BrowserWindowSession integration which tracks sessions as when the app has focus
There is a timout to ensure that sessions are not ended immediatly if a user briefly switches away from the app

The default timeout is 30 seconds but this is configurable via the backgroundTimeoutSeconds constructor option.

The session on exit handling was moved to src/main/sessions.ts since this is now use by both session integrations
Some changes to the default integration logic were required to ensure only one session integration is used
Errors or crashes while the app is in the background will not be attributed to any session

const { init, Integrations } = require('@sentry/electron');

init({
  dsn: '__DSN__',
  integrations: [new Integrations.BrowserWindowSession({ backgroundTimeoutSeconds: 10 })],
});

I've limited this new feature to Electron >= v12 since older versions of Electron seem to unreliably report focus events which make this quite unreliable for sessions.

Errors or crashes while the app is in the background will not be attributed to any session

The behaviour we kept here is good to go!


Fix canonical distribution names

Fix the canonical distribution names section in the docs and update the canonicalize command.

I forget why it was important to replace - with _, so let me dig a bit to find it before we merge this change.

I think it had to do with the fact that the sdist filename took the value from the packaging settings without modifying it at all, so it could come out different at different times, and be different from the wheel itself which does force canonicalization. That means it's impossible to ensure we get the same name all the time unless we make the extra transformation. Since fromager relies so much on looking up settings and plugins by using the package name, we need some way to force it into a  consistent format.

https://packaging.python.org/en/latest/specifications/name-normalization/ defines the normalization rules.

The name should be lowercased with all runs of the characters ., -, or _ replaced with a single - character. This can be implemented in Python with the re module:

https://packaging.python.org/en/latest/specifications/name-normalization/ defines the normalization rules.

The name should be lowercased with all runs of the characters ., -, or _ replaced with a single - character.

OK. That's not what I was seeing, I think. I'll revisit this and see if we can use - instead of _.

PyPI isn't consistent. There is standard on one hand. And there is legacy support on the other hand.
PyPI are a ton of legacy packages that have a . in the package name and sdist name. For example most of the Zope 3 packages like https://pypi.org/project/zope.interface use legacy names. https://pypi.org/project/zope-interface (dash) redirects to  https://pypi.org/project/zope.interface (dot).

PyPI isn't consistent. There is standard on one hand. And there is legacy support on the other hand.
For example PyPI are a ton of legacy packages that have a . in the package name and sdist name. For example most of the Zope 3 packages like https://pypi.org/project/zope.interface use legacy names. https://pypi.org/project/zope-interface (dash) redirects to https://pypi.org/project/zope.interface (dot).

Right. I had to pick something. It looks like I picked the wrong thing. I probably picked that based on a bad example, but I don't remember any more.
In any case, any change would be backwards-incompatible, so I want to take good care to understand how much will break in the downstream builds before we make it.

PyPI isn't consistent. There is standard on one hand. And there is legacy support on the other hand.
For example PyPI are a ton of legacy packages that have a . in the package name and sdist name. For example most of the Zope 3 packages like https://pypi.org/project/zope.interface use legacy names. https://pypi.org/project/zope-interface (dash) redirects to https://pypi.org/project/zope.interface (dot).

Right. I had to pick something. It looks like I picked the wrong thing. I probably picked that based on a bad example, but I don't remember any more.
In any case, any change would be backwards-incompatible, so I want to take good care to understand how much will break in the downstream builds before we make it.

A conversation earlier today made me realize that the reason for replacing - with _ has to do with needing to be able to use those names for python package names for the override plugin entry point registration, and wanting to use the same naming everywhere so users don't have to know when to use different types of canonicalization.

I don't think we will want this, unfortunately. It's going to be hard to make it backwards compatible.


Vanced doesn't start when Google Account Linked

When being logged in with my Google Account in MicroG, YouTubeVanced doesn't load any content. When removing the account everything seems fine.
Tested this behavior on 2 different devices.
Edit: Doesn't load means that not even the feed is being loaded - neither do the navigation links at the bottom work.

Install Vanced 15.40.37 or lower via the manager
Login
Update to latest


#228 - skip invalid packages on incremental update

For #228 made incremental update skip invalid rpm packages.

Codecov Report

Merging #229 into master will increase coverage by 8.05%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master     #229      +/-   ##
============================================
+ Coverage     78.64%   86.70%   +8.05%
- Complexity      207      247      +40
============================================
  Files            36       36
  Lines          1185     1188       +3
  Branches         51       51
============================================
+ Hits            932     1030      +98
+ Misses          245      150      -95
  Partials          8        8

Impacted Files
Coverage Δ
Complexity Δ

...ain/java/com/artipie/rpm/ModifiableRepository.java
90.47% <100.00%> (+90.47%)
7.00 <0.00> (+7.00)

...main/java/com/artipie/rpm/pkg/FilelistsOutput.java
86.66% <0.00%> (+3.33%)
7.00% <0.00%> (+1.00%)

...rc/main/java/com/artipie/rpm/pkg/OthersOutput.java
85.18% <0.00%> (+3.70%)
7.00% <0.00%> (+1.00%)

src/main/java/com/artipie/rpm/Rpm.java
84.56% <0.00%> (+47.53%)
53.00% <0.00%> (+31.00%)

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 448f0c0...d7775b2. Read the comment docs.

This pull request #229 is assigned to @paulodamaso/z, here is why; the budget is 15 minutes, see §4; please, read §27 and when you decide to accept the changes, inform @g4s8/z (the architect) right in this ticket; if you decide that this PR should not be accepted ever, also inform the architect; this blog post will help you understand what is expected from a code reviewer; there will be a monetary reward for this job

@paulodamaso thank you for the review, I've created #241 as we need an instrument to validate metadata, assertions will be fixed here after #241 is implemented.

@paulodamaso/z this job was assigned to you 5days ago. It will be taken away from you soon, unless you close it, see §8. Read this and this, please.

Code review was too long (10 days), architects (@g4s8, @olenagerasimova/z) were penalized, see §55

@sereshqua/z please review this job completed by @paulodamaso/z, as in §30; the job will be fully closed and all payments will be made when the quality review is completed

@paulodamaso please make sure you will find at least 3 issues during next CR, thanks

@paulodamaso ping

@sereshqua I'll try to find at least 3 issues in my next revisions, thank you

@0crat quality acceptable


[🚀] REST service: Enable request throttling by remote IP.

Describe the solution you'd like
Plugin controllers, endpoints and system administrators should be allowed to request limits for throttling purposes. This should be understood as a basic way of DoS prevention since frequent repeated requests may be able to at least flood the underlying thread pool of Spark - especially when the response is calculated always.
Describe why you would like to see this implemented
Note: Check whether or not Spark already features such a thing!
An implementation could use simple thresholds per IP and endpoint to determine whether or not requests should be throttled (rejected with HTTP/X.X 429 TOO_MANY_REQUESTS).
A first draft implementation could utilize a simple global threshold to enforce over all requests so basic protection is available. We could use a ticking thread to reduce points for all remote addresses each tick while new requests would increase the number of points. The reduction of points per tick should be configurable while the tick rate could be made a system property.
Since this is about performance under load, we should start with the simple global threshold and implement it as simple and up-front as possible.

@finnWellers what do you think about this?

Also: Of course, we should respect reverse-proxy scenarios and their respective forwarding headers.


GPU memory is large when the initialization？

I want to adding the module to the baskbone in the conv2.In operation, I found a phenomenon batchsize >4,the Memory overflow。

Is it possible to specify your experimental details, such as the input image size, network type, GPU type you are using, and so on? Without this information, it is hard to diagnose.


first submission from KMobin

Hey @KMobin555 Thanks for submitting to the shared task!
Your predictions.json file is named prediction.json can you rename it to predictions.json please? Thanks!


Remove Moq

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

Comparison is base (af0d5fa) 93.99% compared to head (af34b9c) 93.97%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #168      +/-   ##
==========================================
- Coverage   93.99%   93.97%   -0.02%
==========================================
  Files          82       82
  Lines        5080     5080
  Branches      479      479
==========================================
- Hits         4775     4774       -1
- Misses        177      178       +1
  Partials      128      128

see 1 file with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Fixes supporting Fedora-base streaming to include references for fileset and binary name. #1

https://github.com/ucsdlib/encrypted-stream-name/issues/1

@mcritchlow I've moved the test out of main method as an JUnit test class, which can be run through ant task ant junit, and the argument length is line up with the noted parameters in the comment now.

Looks good. I'll merge and then setup Circle CI to run against any future changes. Thanks @lsitu !