Move Progress Field into Library Table

Upon further deliberation, progress would be best in the library table. While this means any potential duplicate copies may not be synced, it also means that game entries can be aggregated and shared, and that deleting an in progress library entry will not cause that game to continue to appear on the index screen. It also allows a user to easily hyperlink to the library entry in question.

This is also causing difficulties with having games filtered by progress status.


setPattern function raise an error

When the function setPattern is called the following error is raised:
 ReferenceError: Module is not defined  at Project.TimePatternFunctions.setPattern (C:\work\dropletlive\droplet-live\packages\backend\node_modules\epanet-js\dist\epanet-js.cjs.development.js:867:5)
The function setPatternValue works correctly.

Hi @tproduit ,
This issue was resolved in #28 however I didn't push a new release, I will need to recompile and bump up to 0.4.0.
I'll do this ASAP

Solved with 0.4.0. Thanks!


Number of genders

The number of genders offered is astonishingly low. This is likely to assume someone's gender incorrectly.
Assuming someone's gender can cause trauma and is violence.

The number of genders offered is astonishingly low. This is likely to assume someone's gender incorrectly.
Assuming someone's gender can cause trauma and is violence.

Then you should create your own package that returns any gender for any name. Problem solved :)


Submission of Final Project of Kellyann Hayes

@QMSS-G5072-2022/teaching_team
Hi teaching team, please note that I am also going to issue a commit for my test data and output - it contains parole information of some random folks that I used to show a test case (mostly John Does and John Smiths) but while the information is publicly avaialble I still felt weird about having it up on github, so I created a private repository for it. (this repository had to be public in order to build on Read the Docs.
Sorry for the inconvenience, and let me know if there's something else you would like me to do with it!

Sorry, forgot the SHA. Here it is:
9dc249c9e4af6e1c2b258161ffa75f132fccd4ab


120 and 3000  minute plans are not listed

120 and 3000  minute plans are not listed

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

ID: 6b07cfcf-8f31-91ed-b1de-c7826a1ab91d
Version Independent ID: dfbc4212-8f47-326a-f872-8186a915ae1e
Content: Country and region availability for Audio Conferencing and Calling Plans
Content Source: Teams/country-and-region-availability-for-audio-conferencing-and-calling-plans/country-and-region-availability-for-audio-conferencing-and-calling-plans.md
Service: msteams
GitHub Login: @CarolynRowe
Microsoft Alias: crowe

@TheBluLion Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

@officedocsbot assign @scanum

@TheBluLion Thank you for reaching out and helping improve MS Docs.  I put US at 'select your country' and the plans appeared in the new article.  Please can you let me know if you're still having this issue.

@TheBluLion We appreciate any feedback that improves the content in the Microsoft docs but we haven't had any response from you so we are going to close this issue.

Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.


Add error to Validate if password is common

Inspired by how bad pattern locks are on phones, I added some code to count continuous (or maybe contiguous) pattern lengths as 0.
If you swipe your finger across your keyboard, in any direction, along the red lines in this image, it will be length 0.
Please let me know which changes you'd like to see and if these additions are appropriate.  A couple of things:

I don't want to entropy rules of this package to get too complex, that said I think this is a good idea. In order to "keep it simple" I would rather just keep a hard-coded list of "common" patterns that we can punish. Things like "qwerty" and "asdf".
When we penalize a sequence we should still give it a length of 1 not 0
I don't think this API makes sense. Looks like there is a new internal package that depends on some global variable being set? I would rather just enforce the "common" patterns I mentioned above with no added configurability

The take-aways here are:

KISS
This package will enforce the "best" entropy values, and won't expose configuration options

I considered removing common-password substrings, but that would penalize battery and horse from correctbatteryhorsestaple, which is considered a decent password!
Instead, I made it so Validate sees if the password matches 1:1 to a common one, and will tack on some text to the error, telling the user they should use an uncommon password.
Some common patterns are in the password list, such as variations of qwerty and asdf.

Need to embed most-common-passwords.txt into a go file instead of ioutil.Reading it, and change length to 1 if it matches a password before this is ready

I applaud your efforts, but I feel we may be searching for a problem that doesn't exist. I don't love the idea of loading 10,000 strings into memory just by importing this package. Thanks for your efforts and I would love to see further contributions in the future. Maybe open an issue first so we can decide if it should be addressed before spending time coding!


Automatic installer should be provided via HTTPS

Hi,
In vim.spf13.com the easy install says to run:
curl http://j.mp/spf13-vim3 -L -o - | sh
Please replace it with
curl https://j.mp/spf13-vim3 -L -o - | sh
Although it makes a redirect to an HTTPS source, the redirect itself is vulnerable to MiTM.
Thanks

Not an issue anymore


Incorrect documentation about ruleset vs editorconfig precedence

Issue moved from dotnet/docs#26234

Please respond to @mavasani.

From @mavasani on Thursday, September 23, 2021 1:15:32 PM
@mavasani Sorry to come back to this again:
Though the documentation referred in my last post may be correct, but the documentation referred in my initial post

According to https://docs.microsoft.com/en-us/visualstudio/code-quality/use-roslyn-analyzers?view=vs-2019:

Setting a rule's severity in an EditorConfig file takes precedence over any severity that's set in a rule set or in Solution Explorer.

seems not.
Edit: Anyway, thanks for the helping support so far!
Originally posted by @julianbartel in https://github.com/dotnet/roslyn/issues/55920#issuecomment-925735644

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

ID: 36455e6f-5542-5223-b242-9cddf6df267c
Version Independent ID: 946abcdb-7e41-b5b5-e062-9cd8c914fbd3
Content: Code analysis using Roslyn analyzers - Visual Studio (Windows)
Content Source: docs/code-quality/roslyn-analyzers-overview.md
Product: visual-studio-windows
Technology: vs-ide-code-analysis
GitHub Login: @mikadumont
Microsoft Alias: midumont

@mavasani -- Manish, thank you for your followup.
@mikadumont -- Mika, please look into this issue.

https://docs.microsoft.com/en-us/dotnet/fundamentals/code-analysis/configuration-files#severity-options has the correct precedence rules

You should migrate to using only editorconfig and globalconfig files, precedence rules between those files are here: https://docs.microsoft.com/en-us/dotnet/fundamentals/code-analysis/configuration-files#precedence
Ruleset files are deprecated for managed projects

Thank you for your post. We believe this answer is now available. We will now proceed to close this issue. If you feel this should be reopened, let us know.


Load Average for KVM

Description
This adds the load average for the kvm hypervisor. The average Load already seems to be available for Xen.
In the listHosts the averageload is already present. In listHostMetrics it appears as "loadAverage".
It adds the 1-min load average from /proc/loadavg to the
listHosts
listHostsMetrics
API calls.

Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[ ] Bug fix (non-breaking change which fixes an issue)
[x] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

How Has This Been Tested?
Built and tested in kvm cluster.
Tested manually via API calls

@blueoragutan package

@GabrielBrascher a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✖centos6 ✔centos7 ✔debian. JID-420

@blueorangutan package

@DennisKonrad
I have tested it , the value it looks fine.
The only problem is, I spent some time to look for the value in the api responses.  it would be better to change them to same name
(local) 🐵 > list hosts filter=averageload
(local) 🐵 > list hostsmetrics filter=loadAverage

@weizhouapache I changed both of the keys to cpuloadaverage:

@blueorangutan package

@GabrielBrascher a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✖centos6 ✔centos7 ✔debian. JID-423

I would start a regression suite, but...
Do we need test automation for this?
/cc @weizhouapache @GabrielBrascher @DennisKonrad

@DaanHoogland I don't think that automation tests are strictly necessary in this case. It is a simple change, and we already have code reviews and one manual tests by @weizhouapache.
I am assuming that the centos6 failing to package is not related to this PR.

@DennisKonrad Why do the Host Metrics on your screenshots show a String and not a Double?
We should return this in the API as a Double and not a String.

Good point @wido. It is set as double or Double on HostStats.java, HostResponse.java, GetHostStatsAnswer.java, and HostStatsEntry.java. Additionally, at CPUStat.java the method getCpuLoadAverage() returns Double.
Therefore, it looks OK on the PR. However, list hostmetrics returns it as String; list hosts looks fine though. I will take a look around the list hostmetrics.
On a 5.0 we can normalize API values and fix issues such as % values returned as Strings instead of double.

@wido Good catch. Will change that to output as double

@GabrielBrascher centos6 is broken on a lot (most if not all) PRs. not to worry about that one.
so a package after @DennisKonrad 's next addition will be enough to carry on.

@wido I think it's better now.

Looking good!

Looking good to merge.
Merging it considering that all checks have passed, we have 4 LGTMs, a couple of manual tests, and the packaging results are ok for Ubuntu and CentOS.


couchup does not copy database permissions

couchup does not seem to copy database permissions from 1.6.1 to 2.2.0
Expected Behavior

I would expect the system to copy not only the data but also any permissions assigned to the database.
Current Behavior

Possible Solution

Steps to Reproduce (for bugs)

upgrade couchdb from 1.6.1 to 2.2.0
startup couchdb
run coucup replicate -a
4.when viewing the couchdb instance on a web browser no permissions as replicated

Context

Your Environment

Version used:  2.2.0
Browser Name and version: google chrome
Operating System and version (desktop or mobile):  RHEL7.4
Link to your project:  n/a

Confirmed, this is a concern. We need to manually grab the _security object and replicate it over.
The problem is that if your _users database doesn't have the same content, the security settings may prevent appropriate access to the database for non-server-admins.
Would you be OK if this was optional functionality enabled with a command line flag?

@wohali I’d make it opt-out for folks who run into that. I think the default should be “make exactly the same as source”

I've decided to just have moving _security over be the very last operation, after replication is done. A server admin can remove or alter _security if they need to after the fact.


Update Documentation -detailed (#2217)

Type of Change

[x] Bug fix (non-breaking change, fixes #2217 )
[ ] New feature (non-breaking change, adds functionality)
[ ] Breaking change (effects multiple commands or functionality)
[ ] Ran manual Pester test and has passed (`.\tests\manual.pester.ps1)
[ ] Adding code coverage to existing functionality
[ ] Pester test is included
[ ] Nunit test is included
[x] Documentation
[ ] Build system

Purpose

To Update the Documentation in Cmdlets that no longer use -Detailed Parameter (Specifically in Test-DbaDatabaseCollation, Test-DbaDiskAllocation, and Test-DbaMaxDop)
To Update Examples using the old Syntax

Approach

Replace the Description with language provided by Shawn Melton
Include Test-DbaDeprecation if it was not included already

Commands to test

Test-DbaDatabaseCollation -SqlInstance sqlserver2014a -Database db1, db2
Test-DbaDatabaseCollation -SqlInstance sqlserver2014a -Database db1, db2 -Detailed (Should Return Warning)
Test-DbaDiskAllocation -ComputerName sqlserver2014 | Select-Output *
Test-DbaDiskAllocation -ComputerName sqlserver2014 -Detailed (Should Return Warning)
Test-DbaMaxDop -SqlInstance sqlserver2016 | Select-Object *
Test-DbaMaxDop -SqlInstance sqlserver2016 -Detailed (Should Return Warning)

TEARIN IT UP!! 🔥 🔥🔥 🚒

LOL just trying to earn my stripes...

Thanks much! Just one thing, I changed the word from depreciated to deprecated. Merged 💃

Wish I could say that was the first time I mixed those up... :unamused: Thanks for the fix.


Update unit tests for datastore-api

Add unit tests for MetadataApi
Add unit tests for IndexApi

Resolves: OLPEDGE-1485, OLPEDGE-1483
Signed-off-by: ashevchu<EMAIL_ADDRESS>

Codecov Report

Merging #161 into master will increase coverage by 4.9%.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           master    #161     +/-   ##
========================================
+ Coverage    58.9%   63.8%   +4.9%
========================================
  Files          51      51
  Lines        1344    1344
  Branches      156     156
========================================
+ Hits          792     858     +66
+ Misses        523     452     -71
- Partials       29      34      +5

Impacted Files
Coverage Δ

@here/olp-sdk-dataservice-api/lib/index-api.ts
37% <0%> (+22.2%)
:arrow_up:

...here/olp-sdk-dataservice-api/lib/RequestBuilder.ts
51.8% <0%> (+44.4%)
:arrow_up:

@here/olp-sdk-dataservice-api/lib/metadata-api.ts
96.4% <0%> (+84.2%)
:arrow_up:

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 efd73ed...38fad57. Read the comment docs.


GLB file fails to import with cycling dependency error

Godot version
4.0.beta12.official
System information
Windows 11, Forward+, NVidia Geforce 3070 TI
Issue description
I exported a character as GLB from unreal engine 5.1 and tried to import it into Godot. The import fails with the following error:

Can't add child 'Mesh2' to 'Skeleton3D' as it would result in a cyclic dependency since 'Mesh2' is already a parent of 'Skeleton3D'.
scene/main/node.cpp:1641 - Condition "!owner_valid" is true.

I can load the GLB file file in Microsoft 3D Viewer.
Steps to reproduce
Download the below GLB and drop it in a blank projects root directory. As Godot attempts to import you'll see the error.
Minimal reproduction project
Download this GLB and drop it in a blank projects root directory. As Godot attempts to import you'll see the error.
SK_Character_Female_Druid.zip

@lyuma Nudge, there's two issues that seem to be in the same area. https://github.com/godotengine/godot/issues/68994

No longer an issue in Beta 17!


The question about label information passed to GraphEncoder

On line 369 of the file graph.py, why is the label tensor used to multiply the output of the graph after applying the gumbel_softmax function? This may potentially leak label information to the contrast_mask tensor, which could aid in downstream classification tasks?

The leak is designed on purpose. The true label information is used to build high-quality positive samples for contrastive learning. After training, positive samples as well as true labels are not needed.
Please refer to our paper for detail explanation.


Add naive approach to backfilling

Based on the last discussion, we'll be first implementing a naive approach where we compare emitTime against lastmodifiedtime. If emitTime < lastmodifiedtime, we'll discard the backfill event.
One scenario pointed out by Edwina where this approach might not be ideal:
1. MCE-1 for entity A goes into DLQ (emit time T1)
2. MCE-2 for entity A goes into DLQ (emit time T2)
3. backfill batches MCE-1 and MCE-2, and consolidates into MCE-2.
4. MCE-3 for entity A goes into DLQ (emit time T3)
5. backfill runs, and applies MCE-2 (modified time T4)
6. backfill runs for MCE-3 and is dropped, since T3 < T4.

We'll implement this approach and will keep the logic for the old schema.
Future tasks:

Add emitTime column in the new schema and use that as comparison to whether we should do backfill. This should not be too much on operational side since this info will be stored in the json column
Have the backfill tool to allow override emitTime so that the old schema could work

Checklist

[x] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[ ] Links to related issues (if applicable)
[x] Tests for the changes have been added/updated (if applicable)
[ ] Docs related to the changes have been added/updated (if applicable)

Merge on behalf of Derek who is having permission issues merging.


Unable to obtain connection from database (...) for user 'null': Access denied for user ''@'' (using password: YES)

Which version and edition of Flyway are you using?
5.2.4 Community Edition
Which client are you using? (Command-line, Java API, Maven plugin, Gradle plugin)
Gradle plugin
Which database are you using (type & version)?
MySQL 8.0.14 in Docker container
Which operating system are you using?
Antergos
What did you do?
(Please include the content causing the issue, any relevant configuration settings, the SQL statement that failed (if relevant) and the command you ran.)
CONSOLE OUTPUT
> Task :flywayMigrate FAILED
:flywayMigrate (Thread[Execution worker for ':' Thread 2,5,main]) completed. Took 30.373 secs.

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':flywayMigrate'.
> Error occurred while executing flywayMigrate

  Unable to obtain connection from database (jdbc:mysql://<REDACTED IP>:<PORT>/refund?verifyServerCertificate=false&useSSL=true) for user 'null': Access denied for user ''@'<REDACTED IP>' (using password: YES)
  --------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
  SQL State  : 28000
  Error Code : 1045
  Message    : Access denied for user ''@'<REDACTED IP>' (using password: YES)

  Access denied for user ''@'<REDACTED IP>' (using password: YES)

GRADLE FILE EXCERPT (NOTE both the username and true are printed, the values are right)
def is_ci = props.getProperty("ssh.user") == "username"
println is_ci
println ((is_ci) ? System.getenv("MYSQL_USER") : mysql_user)

flyway {
    user = ((is_ci) ? System.getenv("MYSQL_USER") : mysql_user)
    url = ((is_ci) ? System.getenv("MYSQL_URL") : mysql_url)
    password = ((is_ci) ? System.getenv("MYSQL_PASS") : mysql_pass)
}

What did you expect to see?
I wouldve thought that the user wouldnt be null / '' and that I'd be able to access the db
What did you see instead?
Above console output

Please share a small repository with build.gradle and docker-compose.yml files that reproduce this issue.

Issue has been resolved, issue was my lack of groovy knowledge.
I had a previously defined variable called user that mustve been messing with the flyway user variable. Just had to change the other user to something else and it was fine


Add completion even to android plugin

This was implemented in iOS, but not on Android, yet.

@jadennn happy to merge this?


Seems like startup scripts aren't being run

They're supposed to spit out some [OK] responses on boot, but it seems like they don't run, Furthermore, the custom startup scrupts do run (because they're triggered vy /etc/rc.d/startup) 🤔. I think I'll try running them from /etc/rc.d/startup, seems like right way.

Potentially fixed in semcos-packages/etc@d59bea751d7ba2821539300412576baa607f09a0.

It's worse than I thought - the CLFS bootscripts are completely incorrect. Found the right ones though!

I believe when i run my tests that i saw them [OK] Outputs, has this issue been resolved?

I had indeed solved this, but it remains open because of #21.

Closed thanks to #7


Take Calls Button

Describe the problem
We need a way to communicate to the AWS Connect API that this user is online and ready to take cals
Describe the solution
make the "Take Calls" button talk to AWS Connect and change the operator state to "Available to take calls"
Additional context and instructions
The result of pushing this button should

[ ] talk to AWS Connect and change the operator State
[ ] Change the Identification Card from this:

To this

with the button now converted to a "Status Button" in the active state of "Available for taking calls"

superseded by better plan.  done.


Course planning

What do we want to do:

[x] describe learning objectives
[x] describe target learner and pre-requisites
[x] identify learning outcomes and scope (#2)
[x] decompose into subsections, e.g. re-runnability, data provenance, etc. (#2)
[ ] develop exercises, at least one for each learning outcome, but actually very many
[ ] collect readings https://www.diigo.com/user/rutgeraldo/?query=%23opensci
[ ] prepare lectures
[ ] plan documentation

Hi @Pfern,
on the issue of exercises, we are planning to work through a single case where a number of analysis steps are performed. For this it will be most instructive if this is done with one of the platforms that best supports literate programming, so that the analysis can be mixed with the prose of the example publication. As far as I know, there are two plausible platforms for this: jupyter or rmarkdown. Here are some places that aggregate example cases:

jupyter: https://github.com/nborwankar/LearnDataScience (alert, examples 3 years old)
jupyter: https://github.com/jakevdp/sklearn_tutorial (<1 yr old)
rmarkdown: https://ropensci.org/usecases/

All the examples are relatively stats-heavy heavy but light on data volumes, for relatively obvious reasons, them being on github. This also means they are more geared towards ecology (e.g. niche modeling) than bioinformatics per se. Should we nonetheless pick one from among these?

We could throw-in some simple problems that start from sequence data.
I will find some...
Also from time series like this one
https://github.com/simplystats/simplystats.github.io/blob/master/_Rmd_files/2016-05-05-timeseries-biomedical.Rmd
Also see, for the advocacy of our cause
https://www.nature.com/sdata/policies/repositories
http://blogs.plos.org/plos/2017/04/open-data-projects-win-wellcome-trust-nih-and-hhmi-open-science-prize/
https://phys.org/news/2017-02-science-prize-software-tool-tracking.html

The ScikitLearn is impressive!


Update mocha integration tests

Update chromedriver/selenium-webdriver to current version
Make sure the tests are required and are testing the app properly

Convert tests that check text values to test for the existence of the element, or at least one element if it is in a list. Text values in facet lists, etc are dynamic and can't be tested for a literal value

Update incorrect xpaths


Include the required tx signers and the already signers into the "transaction view" output

Internal/External
Internal   if an IOHK staff member.
Area
Other   Any other topic (Delegation, Ranking, ...).
Describe the feature you'd like
When using incremental transaction signing, from an end-user perspective, it would be best to be able to see (maybe in the transaction view output):

what keys (or at least how many) need to sign the transaction
what keys already signed the transaction

Describe alternatives you've considered
Additional context / screenshots
related to https://github.com/input-output-hk/cardano-node/issues/3545

@dorin100 I'd suggest reducing scope of this to just what keys are already witnessing the transaction. The former would require knowing state, which I don't think view should require (and would be a lot more complex).

whatever is possible is already a plus; ideally would be both but we definitely rely on the developer feedback too.

Closing this. If this is still relevant please reopen.

this is still relevant


DiscordMfaToken

The Header of "Authorization" doesn't exist anymore and if you refresh on the page it automatically signs you off. So album artwork isn't possible anymore.

I just checked and it is still working. I am also signed off in the browser, however, the token keeps working.
I believe that instead of refreshing, I now just clicked something, e.g., the application I created inside Discord's developer portal. Does that work for you?

Hmmm Interesting yeah it does. If DevTools is open it seems to sign you out. Strangly though I am still not getting album artwork

I am going to update the README, thank you for reporting this. Regarding artwork, it takes ~15 min for it to become available after uploading.


Purechat fork

The purechat fork for PiggyFactions works good but instead of the "^" for leader it just puts leaderfactionname so like leaderFrance

Please use the latest build found at https://poggit.pmmp.io/r/88188/PureChat_dev-1.phar


Exceptions throw 404 rather than 500

node.js version: v12.4.0
npm/yarn and version: npm 6.9.0
koa-router version: 3.2.0
koa version: 2.7.0
Code sample:
module.exports.mycall = async (ctx, next) => {
  const object = await ObjectFinder.find(ctx.request.body.id); // this object doesn't exits and an exception is thrown.
  object.date = new Date();
  await object.save()
  ctx.status = 204
};

Expected Behavior:
As there was an exception the response code should be 500
Actual Behavior:
Got a 404, which makes no sense.
Additional steps, HTTP request details, or  to reproduce the behavior or a test case:

I have similar problem with koa-router (v. 7.4), koa(2.7) i am working with nodemailer, and catching 404 error before like mail sended for user,  but want send 200 status after sending mail
`const feedback = async (ctx) => {
const {
company, username, phone, email,
} = ctx.request.body;
// here get 404, but if written ctx.status = 200; it is ok, but not sense
const mailer = async () => {
const transporter = await nodemailer.createTransport({
host: 'smtp.gmail.com',
port: 587,
secure: false,
auth: {
user: mailuser,
pass: mailpass,
},
});
const output = `

company: ${company}

username: ${username}

phone: ${phone}

E-mail: ${email}

`;
const info = await transporter.sendMail({
  from: username,
  to: email,
  subject: email,
  html: output,
});

console.log(info.messageId);
return new Promise((res, rej) => {
if (info.messageId.length > 0) {
res();
return info;
}
rej();
return 0;
});
};
mailer().then(() => {
console.log('where is my OK status ??');
ctx.body = {
status: 200,
msg: 'ok',
};
});
};`
Log in my console
POST /sendfeed - 38ms - 404 - hostOrigin -> undefined userAgent -> PostmanRuntime/7.15.2
cookies: no cookies
9330e33a-8c8e-e962-1bfc-fd4dfd93ce8f@my-pc
where is my OK status ??


fix(backend): devops backend test for adminresolvers confirm two creations one after is flaky

🍰 Pullrequest

Issues

None

Todo

i had several ideas to solve this problem:

[X] introduce milliseconds in datetime columns to ensure correct order even when writing in the same second
[X] enable the database to lock single row instead of whole table during write access by defining indexes;
described here: https://stackoverflow.com/questions/2332768/how-to-avoid-mysql-deadlock-found-when-trying-to-get-lock-try-restarting-trans
[X] surround the write access with a retry logic to solve deadlocks of concurrent invocations of the same method;
described here: https://blog.tekenlight.com/2019/02/21/database-deadlock-mysql.html

All three ideas are implemented, but not till the third one with the retry logic solves the problem

there is a better solution in #2458
this PR will be closed


Header high

Can you implement changing of Header high?
First of all I mean the high of space between text and expanded object.
I think its extemelly big now;

Also it could be great if I can Fill child control (now its trouble)


nixops: unbreak

Motivation for this change
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@adisbladis Why did you get rid of the commits disabling some packages on Python 2? I had removed all commits that cause mass rebuilds.

@FRidh @dotlambda In the interest of getting this merged quickly I took some liberties and followed @FRidh's advice and extracted the commits that drops python27 into a separate PR targeting staging in https://github.com/NixOS/nixpkgs/pull/115903 and force pushed this one.

@adisbladis Why did you get rid of the commits disabling some packages on Python 2? I had removed all commits that cause mass rebuilds.

Sorry about that, I must have misread the Github labels. I thought the mass rebuild labels were still valid.


dead link: Plan your Phone System in Office 365 (Cloud PBX) solution:

this link doesn't work: "Plan your Phone System in Office 365 (Cloud PBX) solution: "

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

ID: 33bc456f-7b95-8779-3970-e3f476a5a4fb
Version Independent ID: f0549691-87ab-e7d8-d00d-a5169a8cddb7
Content: Here's what you get with Phone System in Office 365
Content Source: Teams/here-s-what-you-get-with-phone-system.md
Service: msteams
GitHub Login: @CarolynRowe
Microsoft Alias: crowe

What Mike said. Dead link

Thank you for your feedback.  This has been fixed and will go live today.


Add SearchScreenTest for Category Filter

Please add a test for the Category Filter.
It should be something like this: "when we filter tests with App Architecture - it should display App Architecture timetable items"
https://github.com/DroidKaigi/conference-app-2024/blob/3c08e519ed4eb673e902031398112facc30bb0dc/feature/sessions/src/androidUnitTest/kotlin/io/github/droidkaigi/confsched/sessions/SearchScreenTest.kt#L46
We can run the test by clicking the "Run" button on the left side of Android Studio.

I would like to work on this task. 🙋♂


Installation Instructions

How can I fully populate DB with all available data?

What class do you try to run, there are many entry points?

You need first to:

Create a DB login (https://github.com/mcekovic/tennis-crystal-ball/blob/master/crystal-ball/src/main/db/create-login.sql)
Create Postgres SQL extensions (https://github.com/mcekovic/tennis-crystal-ball/blob/master/crystal-ball/src/main/db/create-extensions.sql)
And finally create schema (files are located at https://github.com/mcekovic/tennis-crystal-ball/tree/master/crystal-ball/src/main/db)
You can run https://github.com/mcekovic/tennis-crystal-ball/blob/master/data-load/src/main/groovy/org/strangeforest/tcb/dataload/Main.groovy with application arguments '-cd' to run all schema creation scripts in order.
It will try to connect to PostgreSQL DB at jdbc:postgresql://localhost:5432/postgres?prepareThreshold=0 with credentials tcb/tcb by default, if not system properties as per https://github.com/mcekovic/tennis-crystal-ball/blob/master/data-load/src/main/groovy/org/strangeforest/tcb/dataload/SqlPool.groovy should be set.

After that you need to run DB population, and only after DB population, you can run the app.

Thanks for the details. I will try to follow these steps and will let u
know.
Thanks,
Neha
On Tue, Feb 27, 2018 at 6:58 AM, Mileta Cekovic<EMAIL_ADDRESS>wrote:

You need first to:

Create a DB login (https://github.com/mcekovic/
tennis-crystal-ball/blob/master/crystal-ball/src/main/
db/create-login.sql
https://github.com/mcekovic/tennis-crystal-ball/blob/master/crystal-ball/src/main/db/create-login.sql
)
Create Postgres SQL extensions (https://github.com/mcekovic/
tennis-crystal-ball/blob/master/crystal-ball/src/main/
db/create-extensions.sql
https://github.com/mcekovic/tennis-crystal-ball/blob/master/crystal-ball/src/main/db/create-extensions.sql
)
And finally create schema (files are located at
https://github.com/mcekovic/tennis-crystal-ball/tree/
master/crystal-ball/src/main/db
https://github.com/mcekovic/tennis-crystal-ball/tree/master/crystal-ball/src/main/db
)
You can run https://github.com/mcekovic/tennis-crystal-ball/blob/
master/data-load/src/main/groovy/org/strangeforest/tcb/
dataload/Main.groovy
https://github.com/mcekovic/tennis-crystal-ball/blob/master/data-load/src/main/groovy/org/strangeforest/tcb/dataload/Main.groovy
with application arguments '-cd' to run all schema creation scripts in
order.
It will try to connect to PostgreSQL DB at jdbc:postgresql://localhost:
5432/postgres?prepareThreshold=0 with credentials tcb/tcb by default,
if not system properties as per https://github.com/mcekovic/
tennis-crystal-ball/blob/master/data-load/src/main/
groovy/org/strangeforest/tcb/dataload/SqlPool.groovy
https://github.com/mcekovic/tennis-crystal-ball/blob/master/data-load/src/main/groovy/org/strangeforest/tcb/dataload/SqlPool.groovy
should be set.

After that you need to run DB population, and only after DB population,
you can run the app.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mcekovic/tennis-crystal-ball/issues/232#issuecomment-368850746,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGUz1_BXWbCRXR2kOioN1IE8pjMiMDqjks5tY-3MgaJpZM4SUECp
.

Hi, @mcekovic ,
How can I fully populate DB with all available data? Is it done by calling
tennis-crystal-ball/data-load/src/main/groovy/org/strangeforest/tcb/dataload/Main.groovy
with with all available application arguments?
Thanks,
Amir

Ensure Java JDK 1.8 installed on the machine.

Ensure  PostgreSQL 9.5+ (10.x preferable) is installed.

As PostgreSQL admin user, create PostgreSQL login tcb with password tcb
(https://github.com/mcekovic/tennis-crystal-ball/blob/master/crystal-ball/src/main/db/create-login.sql)

As PostgreSQL admin user, create PostgreSQL extension:
CREATE EXTENSION pg_trgm
(https://github.com/mcekovic/tennis-crystal-ball/blob/master/crystal-ball/src/main/db/create-extensions.sql)

Clone UTS Git repo https://github.com/mcekovic/tennis-crystal-ball.git:
git clone https://github.com/mcekovic/tennis-crystal-ball.git

Clone Jeff Sackmann ATP data git repo https://github.com/JeffSackmann/tennis_atp.git to some local directory also:
git clone https://github.com/JeffSackmann/tennis_atp.git

Build the UTS by going to UTS local source directory and running:
gradlew.bat assemble

Unzip content of data-load\build\distributions\data-load-1.0-SNAPSHOT.zip that is created by step 7.

Run database schema creation:
data-load-1.0-SNAPSHOT\bin\data-load.bat -cd
If you created PostgreSQL login with different username/pwd, this is configured with following Java system parameters added to above command line:
-Dtcb.db.username=tcb -Dtcb.db.password=tcb

Run data load (you need Internet connection during the data load to fetch additional data from ATP web site):
data-load-1.0-SNAPSHOT\bin\data-load.bat -lt -Dtcb.db.connections=15 -Dtcb.data.base-dir=%full path of tennis_atp data directory by Jeff Sackmann%
Again, if you created PostgreSQL login with different username/pwd, this is configured with following Java system parameters added to above command line:
-Dtcb.db.username=tcb -Dtcb.db.password=tcb

Wait 30-45 minutes

Exeption in data-load script

Also, data-load script don't take -db parameter and take default path /tcb/tennis_atp...
Centos 7

And this:

Data loader has been updated to reflect recent changes in Jeff's atp_tennis repo as well as recent changes in official ATP web site data. Data load now should complete (may need several retries if ATP web site hangs because of temporary data availability issues). However, the quality of data is not yet 100% guaranteed, as there are ranking data issues introduced in recent atp_tennis repo commits. Some ranking data is fixed by reloading data from ATP website, but some (pre 1978) are still present (see https://github.com/JeffSackmann/tennis_atp/issues/108)

hi ! thanks for this. Im having issues in the data-load.

Process 3251 waits for ShareLock on transaction 60947; blocked by process 3250.
Hint: See server log for query details.
Where: while inserting index tuple (15361,70) in relation "player_ranking"
SQL statement "INSERT INTO player_ranking
(rank_date, player_id, rank, rank_points)
VALUES
(p_rank_date, l_player_id, p_rank, p_rank_points)
ON CONFLICT (rank_date, player_id)
DO UPDATE SET rank = p_rank, rank_points = p_rank_points"
PL/pgSQL function load_ranking(date,integer,integer,integer) line 9 at SQL statement  Call getNextException to see other errors in the batch.
and then loads the file
Rows: 916296 in 2.404 min (6363 row/s)
Loading file '/Users/tomey/tennis_atp/atp_rankings_current.csv'
..................
Rows: 17401 in 2.783 s (8700 row/s)
Total rows: 2857080 in 7.631 min (6251 row/s)
and then 503 error appears
Fixing rank points...
Loading pre-ATP rankings...
Loading missing rankings...
Fetching rankings URL 'https://www.atptour.com/en/rankings/singles?rankDate=1977-08-23&rankRange=1-500'
Exception occurred: org.jsoup.HttpStatusException: HTTP error fetching URL. Status=503, URL=https://www.atptour.com/en/rankings/singles?rankDate=1977-08-23&rankRange=1-500 [retry 1 follows]
it´s broken?
thanks!!!

Re 1-st issue, which PostgreSQL version are you using?
Re 2-nd issue, unfortunately ATP has introduced anti-scraping protection (Cloudflare), so it does no work anymore.

Im using PostgreSQL 12.1 . I should use a low version?
Anyway, I have successfully set up a EC2 instance with the docker image. I
have all the  data in that container? or i need also Jef Sackman data? I
need data from 2019 only.
thanks for the good work!
On Mon, Oct 5, 2020 at 8:15 PM Mileta Cekovic<EMAIL_ADDRESS>wrote:

Re 1-st issue, which PostgreSQL version are you using?
Re 2-nd issue, unfortunately ATP has introduced anti-scraping protection
(Cloudflare), so it does no work anymore.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/mcekovic/tennis-crystal-ball/issues/232#issuecomment-703938218,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA2DB3M5RANLFFXWBJ2OER3SJJHP5ANCNFSM4ESQICUQ
.

In the docker image you have whole 2019 season (up to), but no 2020, so for 2019 you do not need anything more.
Re deadlock, it is actually a warning, not error. When multi-threaded and batched rankings load is working, it is possible to get into deadlocks when two parallel batches edit two players in reverse order, but in that case batches are further divided and retried, so no data is lost.


Bracket]0, pi]

]0, pi].
I have seen this type of bracket in task 1. I think this is open interval. The reason why open interval notation is not written as (0, pi) is because this notation would be ambiguous. The parentheses could be interpreted as either open interval notation or as a function call. To avoid this ambiguity, the notation ]0, pi] is used to explicitly indicate that the interval is open.
I am not sure but correct me guys if I am wrong

I believe this was discussed in the class, and I hope it's resolved now, if not, you can reopen it.


add the gpt-related repos on MeltanoLabs to the Hub

[ ]  https://github.com/MeltanoLabs/map-gpt-embeddings
[x]  https://github.com/MeltanoLabs/target-chromadb
[ ]  https://github.com/aaronsteers/tap-beautifulsoup

We already added https://hub.meltano.com/loaders/target-chromadb !


update_tags.sh: make UPDATED_SOURCE relative to tags file

Fixes https://github.com/ludovicchabant/vim-gutentags/issues/70.
TODO:

[ ] port it for plat/win32/update_tags.cmd

Hi, thanks but I don't want a dependency on Python so we'll have to find a different implementation...
When the cache dir is not set (i.e. when we store the tags file at the project root) we already run a chdir in Vim before running the script so I think making the file path relative in that same if block would solve your problem no?

I am using the script itself also outside of Vim, e.g. from a Git hook - so it should be done in the script.
It could be done in shell code, too - but having it in POSIX (/bin/sh shebang) certainly does not make it easier.  But I do not want to switch to using bash for the script just because of this, too.

I did what I said (make the file path relative in Vimscript) with commit 09fbdfce2e0eb8177e61ab93cf3e72392f461c2b. If you run the script from a git hook you shouldn't have problems passing that path to the script in a relative way, no? (since the hook itself runs in the git repo AFAIK).


Profiling

Sort of a note to self-- feel free to assign to me.
https://github.com/go-echarts/statsview
Should let us get some better insights into what's going on while syncing.
Can be a special build in the makefile so that it doesn't burden the default build.
Probably a starting point, not the destination.

https://github.com/arl/statsviz

This would be awesome to integrate, so we can understand where sync time and live execution time is going!
The thing I was thinking of before was making a separate build that ran with creating a cpu.out that could be passed into pprof. I think using the tools you linked is a better approach!

I assume that we're open to helping to make this a PhD project for HNUST CS students?

Sorry, missed the last message!

Well on some levels I agree with you. However, I think that what we should do is once the chain begins to sync What should show up in the log is blocks per second, updated once per second. It should also tell you how many peers you are connected to.

I think this is a great idea!

IAVL
Various mutexes
Stuff that can be optimized.

Totally agreed. FWIW, as I've been benchmarking various things for our epoch time, most of the computation time problem at epoch time is coming from many random SDK efficiencies, that cause way too many allocations. (But I imagine for most blocks, and for space usage, its all signatures & IAVL & mutex locks)

By the way, after we spoke, I started to look to hire or otherwise engage with someone who has more experience than I do with performance profiling in go. I think that this task is going to get quite in-depth.

This would be amazing, tysm!

I assume that we're open to helping to make this a PhD project for HNUST CS students?

I'm probably down! What exactly does it entail to make something a PhD project?

Closing for now in favor of #1014


Module requires "archive" class but metadata.json doesn't have archive as a requirement.

Metatada.json needs to require 'archive' module for proper functionality of Dynatrace OneAgent module.
Reference here
See #56

This is an easy pull request that solves a problem.

Is there anybody here?   It's been more than a year!

@luisr-escobar is there anyone managing this project?


Improve Buzzer Performance

Ideally, would be able to change the frequency every 1ms

Being able to change frequency rapidly would allow for percussive sounds (kick drums and white noise)

LEDC max frequency is 40MHz with fixed duty cycle

if we aim for 1MHz max frequency, we can get 6-bit duty cycle resolution

44100KHz would be 10-bit

(the equation is duty_cycle_resolution = floor(log2(80MHz / max_freq)) )

Using the ULP coprocessor, it may be possible to handle all the audio PWM instead of using ledc which may take longer to update

not testable as-is since GPIO0-21 are available and buzzer is on GPIO40, but SAO is available to use on GPIO17 and 18

This will be the ticket to spin up the second buzzer


Removed Pandas as a dependency

Replaced the small piece of code that uses Pandas, removed it as a dependency.

Thank you for your contribution, @naruvimama!
We plan to implement your changes in an upcoming release once we have a bit more time to test them alongside some of the other changes that are in flight.


About replication details

Hi.
I am also researching on DR topic. I focus the Google work also. I notice
that AUC for Messidor dataset reached around 0.59 in your work. I have trained all images from Kaggle( total 88702) without excluding any images and make a binary classification. And test my model on Messidor dataset. It reaches about 0.94 AUC score.
I use Resnet50 as the backbone of the network. Because some lesions like red small dots are extremely small, I set the size of the input image as 448*448. I refer some preprocessing procedures from first prize team from Kaggle. Many papers do not use EyePacs for testing. I suppose you may try using the data for training and validate only.
But my model achieved quite a poor performance on another dataset DIARETDB1. Would you mind testing your model on this dataset? Besides,  have you ever tried detecting lesions in fundus images?

The main goal of this work was to replicate Google's JAMA paper, which we refer to in the readme of this repo. We didn't necessarily focus on getting the best performance. Instead, we wanted to know if we could get similar results by following their methodology as accurately as possible.
I just skimmed through the DIARETDB1 data set. However as far as I can see it does not include any direct referable DR grades, it includes ground truths of lesions locations instead. If there aren't direct rDR grades I will need some time to "translate" these lesions ground truths into rDR labels.
I am also working on my "own" approach that doesn't follow Google's approach, but it is more leaning towards approaches from other Kaggle competitors, with several data augmentation techniques and for example larger image sizes. That work is soon to be released in another repo. So stay tuned :-)

Our research team is working on a second edition of the article with some new results after trying to improve the model, apart from replication. When it's done we will update the readme of this repo with a link to the article. Stay tuned! I'll close this issue for now.

@socoi
Hi.
I am working on DR topic recently too, but my results is not good. Can you mind to share your project about this?
Thinks

@WENWENYU
I am still testing my model. If you like to know some details, we can have a free talk in Chinese through
e-mail.

@socoi OK，thinks!


Nuxt 3 / SSR support

Hello, great project !!
I'm trying this out in a Nuxt 3 project and it seems that some components only work on the client-side (Dialog for example). The error is ReferenceError: Element is not defined
I'm not sure if this is preventable or not.

Hey @lewebsimple , thanks for reporting this issue 🙏
It should now be fixed in the latest rc : yarn add @ark-ui/vue@rc.
Let us know if you notice another issue, thanks !

Stupendous, it works like a charm !
Any plans for including a nuxt 3 module ?


Add include guards to header files

Some header files have no include guards. This should be fixed before the next release.

Bertin:
"i prefer #pragma once over #ifndef BLAH_H #define BLAH_H #endif to avoid a cluttered namespace but that is a design choice and not all compilers support the pragma once"


IP filtering of the stoa should be checked.

Could not access Stoa's endpoint '/block_externalized' and '/preimage_received' in the Docker Container.

Duplicate of #111 ?

I will close this PR because the issue has already been resolved by #85.

I will close this PR because the issue has already been resolved by #85.


Querying string literals with diacritics or angle brackets yields no result

I have indexed a dump of the authority data by the German National Library (cf. http://d-nb.info/standards/elementset/gnd). When I now query for objects including diacritical marks or angle brackets the relevant triples can't be found.
For example the GND holds information on "Frühe Neuzeit" (Early Modern Period):
curl  "https://api.ub.rub.de/gnd/sparql" --data-urlencode "q=PREFIX gndo:<http://d-nb.info/standards/elementset/gnd#>; SELECT * WHERE {<http://d-nb.info/gnd/7503866-3> ?p ?o }" -H "Accept: application/sparql-results+json"

But when I query for "Frühe Neuzeit" I get an empty result:
curl "https://api.ub.rub.de/gnd/sparql" --data-urlencode "q= SELECT * WHERE { ?s <http://d-nb.info/standards/elementset/gnd#preferredNameForTheSubjectHeading> 'Frühe Neuzeit' } LIMIT 1" -H "Accept: application/sparql-results+json"

Whereas querying for something without diacritics works as expected:
curl "https://api.ub.rub.de/gnd/sparql" --data-urlencode "q= SELECT * WHERE { ?s <http://d-nb.info/standards/elementset/gnd#preferredNameForTheSubjectHeading> 'Extrapyramidales Syndrom' } LIMIT 1" -H "Accept: application/sparql-results+json"

Another critical case is querying for something with angle brackets:
curl "https://api.ub.rub.de/gnd/sparql" --data-urlencode "q= SELECT * WHERE { ?s <http://d-nb.info/standards/elementset/gnd#preferredNameForTheSubjectHeading> 'Konfiguration <Informatik>' } LIMIT 1" -H "Accept: application/sparql-results+json"

Hope you can shed some light on why this is going wrong...

Hi @ahagenbruch many thanks for entering this and sorry for delay in response...I'm receing too many emails in these days :)
I will keep you with some news as soon as possible

Sorry for the SPARQL endpoint being unavailable at the moment, but we had a massive power outage this afternoon and I can't restart the machine before tomorrow morning...

No problem at all, because at the moment I'm fighting with a production issue in another project, so tomorrow sounds good

Hi @ahagenbruch, the issue has been fixed in master branch, please get the latest version and  retry.
I reproduced your scenarios and fix a couple of hidden things that happened at query time, many thanks.
Please leave the issue open as I'm working on integration tests on a parallel branch and I added some verification tests for this scenario. Unfortunately they are not yet visible in the master because I didn't merge (I had to finish a couple of things before). After integrating in the master if we are all happy I'll close the issue.

Hi @ahagenbruch, I'm closing this issue as I integrated an integration test case that verifies and validates the behaviour described in this issue.
In case you get some other issue related with this matter, please reopen the issue.

Hi @agazzarini, works like a charm now, thanks a lot...


validation rule: min: fix error message

Coverage remained the same at 100.0% when pulling 00426ce85a5698093bf0e60ae84be10ffdd35f72 on omeid:patch-1 into d1008ad1fd04ceb5faedaf34881df0c504382706 on go-ozzo:master.

Coverage remained the same at 100.0% when pulling 0611a220c8be14a967c039ae96f7c6488e36c684 on omeid:patch-1 into d1008ad1fd04ceb5faedaf34881df0c504382706 on go-ozzo:master.

Could you fix the test error as well? Thanks!

No worries, I just looked and the Max rule needs more consideration. I will create a pull request soon.

Fixed in https://github.com/go-ozzo/ozzo-validation/commit/86433d65440f530b10e94b1b1920895f08b22ec5.

Thanks for the fix! Sorry, I have been rather busy to wrap this up.


Run CI also with newer node versions

Also run CI with Node 20.x and 22.x as latest lts and latest versions in addition to node 16.x and 18.x.
Also upgrade to the latest actions versions of checkout and setup-node.

Removed 16/18 versions. Trying to keep some of my minutes limit 😬


Update Helidon guide for OpenAPITools gen with Helidon SE client generation information

Environment Details

Helidon Version: 2.x, 3.x
Helidon SE or Helidon MP SE
JDK version:
OS:
Docker version (if applicable):

Problem Description
See also #4022 (adding Helidon SE client gen to the tool) and #4018 (creating Helidon guide about using OpenAPITools generator)
This issue covers the work for updating the Helidon guide to using OpenAPITools with new info about Helidon SE client generation.

Covered by new Epic for Helidon Specific generator. Closing.


How can I manually trigger the .lazyload?

I have a webpage which filters list items.
 I notice that images only load on the scroll event.
If i resize the window and scroll of course the images will load. But I'd rather trigger a manual load if a filter is pressed. How to do this?

Trigger the event you have configured the plugin to use. It defaults to scroll.
$("img.lazyload").triggetr("scroll");

Doesn't work.
for example
$('.lazyload').lazyload({
    event: "loadImage"
});

will just load them on page load. Even without triggering the event.
Even happens on your example-webpage which is supposed to display them after 5 seconds of DOM-loading.

Just tested and the images below the fold are loaded only after five seconds in the demo page.
http://www.appelsiini.net/projects/lazyload/enabled_timeout.html

Ahh so the images above the fold still get loaded? Alright, sorry then, should maybe include that detail in the description of the custom event though.

This felt like a bug to me too, my application requires triggering all the images, not just the ones above the fold.

For loading all images one could use
$("img.lazyload").trigger("appear");

This worked for me >lazyLoadInstance.update()

For loading all images one could use
$("img.lazyload").trigger("appear");

Ty


Dock Icon stays visible when mouse leaves Atom window

Prerequisites

[X] Put an X between the brackets on this line if you have done all of the following:

Reproduced the problem in Safe Mode: http://flight-manual.atom.io/hacking-atom/sections/debugging/#using-safe-mode
Followed all applicable steps in the debugging guide: http://flight-manual.atom.io/hacking-atom/sections/debugging/
Checked the FAQs on the message board for common solutions: https://discuss.atom.io/c/faq
Checked that your issue isn't already filed: https://github.com/issues?utf8=✓&q=is%3Aissue+user%3Aatom
Checked that there is not already an Atom package that provides the described functionality: https://atom.io/packages

Description
If your Atom loses focus and the dock icon is visible it will stay there until you go back into Atom and click on a different pane.
Steps to Reproduce

Expected behavior:
If the editor loses focus I'd expect the icon to disappear.
Versions
1.17.0

Thanks for the report!  For this part:

If your Atom loses focus and the dock icon is visible it will stay there until you go back into Atom and click on a different pane.

Can you confirm that you actually need to click for the chevron to disappear?  I just need to hover outside of the tree-view when I cmd-tab back to Atom and the chevron disappears which is the same behavior when I'm just in Atom (i.e. haven't switched to another program and then switched back to Atom).

Note: This happens only if you "lose focus from right to left".
If you hover outside from left to right everything's fine.
It looks like an "onmouseout" event isn't thrown

@rsese No I do not need to click anywhere. It will just disappear as expected when I cmd-tab back into Atom.
Like @pistou said I think it is some sort of event that is missing when atom loses focus

I can reproduce this on Windows 10.  Also, the problem isn't when Atom loses focus (because Atom still has focus), it's when the mouse leaves the Atom window.

Reproduced on macOS 10.11.6 and 10.12.5, with Atom 1.17.2 and Atom 1.18. The chevrons should disappear once the mouse leaves Atom. Can the dock icons be disabled altogether?

@alexchandel The One themes have a builtin config option to disable them.  Otherwise: https://discuss.atom.io/t/hide-the-new-tree-view-dock-toggle-button/42793/2?u=wliu


Creating mocks not working as expected in KMP

When attempting to create a mock at the top of a test class like
private val mockedObject = mock<AClass>()
Where the AClass is open via openAll plugin and creating a class to test via
@BeforeTest
    fun setUp() {
        sut =
            ClassToTest(
                param = mockedObject ,
            )
    }

When attempting to modify an answer for the mocked object via
every { mockedObject .foo() } returns bar
I get the following error:
dev.mokkery.internal.NotSingleCallInEveryBlockException: Each 'every' block requires single mock call!
When Mokking the call in the block however it works as expected:
mock<AClass>() {
    every { mockedObject .foo() } returns bar
}

Running Kotlin Multiplatform v2.0.20, with the test in shared/commonTest, using Mokkery 2.3.0

Hi! Could you provide me a reproducer?

Hi, when trying to recreate this in a separate project everything worked as expected. Managed to get it working in my original project as well.
Seems to have been a Gradle issue, after upgrading Android Studio to Koala 2024.1.2, upgrading AGP from 8.3 -> 8.4.2 I saw no affect until replacing the deprecated
kotlinOptions {
                jvmTarget = "1.8"
            }

with
compileTaskProvider.configure {
                compilerOptions {
                    jvmTarget.set(JvmTarget.JVM_1_8)
                }
            }

Not exactly sure why this fixed the issue mentioned above, but if anyone else is having a similar issue it might be worth trying this fix :)

Apologies after trying multiple things I can't seem to reproduce in a separate project. Also tried updating everything on my end and nothing seems to have fixed it. Error seems sporadic too, where sometimes the test passes but will then fail later with the above error message.
If possible, could I add you to my private repo with the error so you can look?

Yes, I can take a look.

@dthomson117
I figured it out! The error message that you get in you project is actually correct. During every { ... } there is more than one call to the mock of StartupRepository:

In every { ... } - this is expected
ActiveSplitManager (initialized before each test)  starts a coroutine in the constructor, which calls method of the mock - this one must not be called

Mokkery mocks must not be called from other threads while configuring or verifying.. The problem does not occur always as multithreading is not deterministic and sometimes the calls were not executed simultaneously.

Ah that's brilliant thank you for finding that!

Just to add to this, adding an every { . . . } to the @BeforeTest function that is called during init seems to have fixed the issue as well without having to modify the original class


investigate refactoring document structure in semmeddb API

Right now, we have a distinct record for every row in the PREDICATIONS file.  For example, we currently have 73 documents for "imatinib TREATS leukemia", each differing by the pmid and predication_id: https://biothings.transltr.io/semmeddb/query?q=subject.umls:C0935989 AND object.umls:C0023418 AND predicate:TREATS
{
  "hits": [
    {
      "_id": "72444465",
      "_score": 19.359053,
      "object": {
        "name": "leukemia",
        "umls": "C0023418"
      },
      "pmid": 24979952,
      "predicate": "TREATS",
      "predication_id": 72444465,
      "subject": {
        "name": "imatinib",
        "umls": "C0935989"
      }
    },
    {
      "_id": "77117922",
      "_score": 19.359053,
      "object": {
        "name": "leukemia",
        "umls": "C0023418"
      },
      "pmid": 23934954,
      "predicate": "TREATS",
      "predication_id": 77117922,
      "subject": {
        "name": "imatinib",
        "umls": "C0935989"
      }
    },
    ...
  ]
}

We could also consider a design in which rows are combined based on unique subjects, predicates, and objects, collapsing the evidence into an array.  For the example above:
{
  "took": 5,
  "total": 73,
  "max_score": 19.359053,
  "hits": [
    {
      "_id": "xxxxxxxx",
      "_score": 19.359053,
      "object": {
        "name": "leukemia",
        "umls": "C0023418"
      },
      "predicate": "TREATS",
      "evidence": [
        {
          "pmid": "24979952",
          "predication_id: "72444465"
        },
        {
          "pmid": "23934954",
          "predication_id: "77117922"
        },
        ...
      ],
      "subject": {
        "name": "imatinib",
        "umls": "C0935989"
      }
    }
  ]
}

That would result in fewer documents, but some documents that are potentially very big (especially if we add sentences as is being explored in https://github.com/biothings/BioThings_Explorer_TRAPI/issues/563).  BTE does this collapsing eventually anyway, and making that change at the API level would give us the ability to add a pmid_count field (which we could use and dynamically adjust in queries).  Creating this issue to track other pros/cons, feasibility analysis, etc.

It's feasible with a customized RootKeyMergerStorage.
Note that {"on_duplicates" : "merge"} config in the manifest.json (as shown below) is NOT capable because it only finds duplicates on _id.
{
    "uploaders" : [
        {
            "parser" : "<path.to.module>:<function_name_1>",
            "on_duplicates" : "ignore|error|merge"  # optional, default to "error"
        }
    ]
}

At the 2023-03-01 team meeting, we agreed that the proposed structure would be better.  We may need to put in some caps on the number of items in the evidence array, since I seem to remember that the count can go 10,000+.
So, let's move forward with this ticket and #563 in parallel.  Assigning to @erikyao...

Discussion on Mar.16 with Andrew

Q: Does it mean we shall group all the documents by the triple of (object.umls, predicate, subject.umls)? Since object.name and subject.name are not standardized across the dataset.
A: I think the answer is yes that we want to "group all the documents by the triple of (object.umls, predicate, subject.umls)".  Beyond that, I think we want to do what's easiest during this exploratory period. So I think it'd be great to add all unique values in an array for object.name and subject.name, but choosing one arbitrarily is fine for the moment too.

Q: Would you prefer taking the semtypes into account? I.e. grouping by (object.umls, object.semtpye, predicate, subject.umls, subject.semtype)? Since one UMLS CUI may be mapped to multiple semtypes.
A: My gut says that we can also just combine unique values of semtypes in an array as well, but I'd like @colleenXu to chime in on whether that's an issue for the SmartAPI annotation. If that is a problem, then perhaps we need to aggregate based on unique records for (object.umls, object.semtype, predicate, subject.umls, subject.semtype).

Sample documents
7 Predications with 7 PMIDs
    {
        '_id': 'C0033011-PART_OF-C0032105',
        'predication_id': [
            27232377,
            31238107,
            40075886,
            72239153,
            72300825,
            72303926,
            72715980
        ],
        'pmid': [27753038, 1268148, 4276847, 23690098, 26287784, 26287791, 24659861],
        'predicate': 'PART_OF',
        'subject': {
            'umls': 'C0033011',
            'name': 'Plasma',
            'semantic_type_abbreviation': 'bdsu',
            'semantic_type_name': 'Body Substance',
            'novelty': 1
        },
        'object': {
            'umls': 'C0032105',
            'name': 'Pregnant Women',
            'semantic_type_abbreviation': 'humn',
            'semantic_type_name': 'Human',
            'novelty': 1
        }
    }

2 Predications with 1 PMID
    {
        '_id': 'C0012524-PART_OF-C0012854',
        'predication_id': [72324048, 72324366],
        'pmid': 16626110,
        'predicate': 'PART_OF',
        'subject': {
            'umls': 'C0012524',
            'name': 'DNA',
            'semantic_type_abbreviation': 'bacs',
            'semantic_type_name': 'Biologically Active Substance',
            'novelty': 1
        },
        'object': {
            'umls': 'C0012854',
            'name': 'Catechol Oxidase',
            'semantic_type_abbreviation': 'aapp',
            'semantic_type_name': 'Amino Acid, Peptide, or Protein',
            'novelty': 1
        }
    },

So I'm guessing that most records will be like the first example (7 Predications with 7 PMIDs), or at least most people will assume that is the more common pattern.  And I get that it can get even more confusing than the second example -- when we have 3 predications and 2 PMIDs, then we lose the mappings between predications and PMIDs.  Having said all that, I think this structure is fine.  For now, the pmid field will be interpreted as "supporting articles", and just showing the set of unique values from which the predications are drawn will be just fine.

For the second example, we can also make it
{
    "predicate": "PART_OF"
    "predication": [
        {
            "predication_id": 72324048,
            "pmid": "16626110",
            # "sentence_id": "omitted"
            # "sentence": "omitted"
        },
        {
            "predication_id": 72324366,
            "pmid": "16626110",
            # "sentence_id": "omitted"
            # "sentence": "omitted"
        },
    ],
    "subject": { ... }
    "object": { ... }
}

Code is forked to my own repo: https://github.com/erikyao/semmeddb2. An initial fix is committed already. Will deploy as a new API semmeddb2 tonight.
Considering ITRB might not sync the code change over the weekend. I will redirect access to semmeddb2 to the old biothings.ncats.io server (some NGINX config change needed).

I've made a separate SmartAPI yaml registration for this API: https://smart-api.info/registry?q=5634bdb9a1315f85f595979eaf90a504
It has the same operations/MetaEdges as the current SEMMEDDB API.

How to query this API using this registration
POST using the api-specific endpoint of dev/CI instances of BTE (or a local instance that is using main branches).
For example, one can use the CI-instance of BTE by sending POST requests to this url: https://bte.ci.transltr.io/v1/smartapi/5634bdb9a1315f85f595979eaf90a504/query

A complicated query I used for testing
uses reverse operations + multiple ID inputs + qualifier support
{
    "message": {
        "query_graph": {
            "nodes": {
                "n0": {
                    "ids": ["UMLS:C0164786", "UMLS:C1367307"],
                    "categories": ["biolink:Gene"],
                    "name": "Proto-Oncogene Proteins c-akt, STAT3 gene"
                },
                "n1": {
                    "categories": ["biolink:Polypeptide"]
                }
            },
            "edges": {
                "e01": {
                    "subject": "n0",
                    "object": "n1",
                    "predicates": ["biolink:affected_by"],
                    "qualifier_constraints": [
                        {
                            "qualifier_set": [
                                {
                                    "qualifier_type_id": "biolink:subject_direction_qualifier",
                                    "qualifier_value": "increased"
                                }
                            ]
                        }
                    ]
                }
            }
        }
    }
}

Response has the expected info:
Record with UMLS:C0164786 Proto-Oncogene Proteins c-akt
How it looks in the response:

The main differences are:

operation requestBody values will generate queries with a pmid_count:>5 filter (which requires an empty-scopes format)
I used infores:biothings-semmeddb-2 (which isn't a true infores) in the yaml's info.x-translator.infores field. This was meant to avoid confusion with the current semmeddb
operation parameters.fields values and response-mapping are modified so the following fields would be ingested by BTE

predication.pmid (replaces pmid field)
pmid_count
predication_count

operation requestBody values don't include subject.novelty / object.novelty filtering because according to Yao it's unnecessary (his parser already removes records where novelty != 1).

closing this one as complete

Updated the notebook / generated yaml / SmartAPI yaml https://github.com/NCATS-Tangerine/translator-api-registry/commit/794ffc25f5f6cfbb3c27b5fd87dcb485d19ac610 + registration so BTE will ingest the supporting sentences and so BTE will use the filter >1 (agreed upon by Translator (Slack link)).
Next step is moving BTE from using the older "semmeddb" to using "semmeddb2"...


[ENG-3654] OSFI: Update Shared SSO and Add MagLab/FSU to OSF Test

Purpose

Update OSFI Shared SSO to support both exact-match and substring-match
Add MagLab to the test server that shares SSO from SSU

DevOps Notes
!!! Let's update settings for both prod and test to save us another trip when nationalmaglab goes production.
Shibboleth
Before deployment, add the following to attribute-map.xml:
<!-- Shared SSO Filter Attribute for FSU and NationalMagLab -->
<Attribute name="urn:fsu:names:SAML:attribute:fsuEduAppRoles" id="userRoles"/>

CAS
Before deployment, add the following to existing FSU configuration in institutions-auth.xsl:
<userRoles><xsl:value-of select="//attribute[@name='userroles']/@value"/></userRoles>

!!! FSU prod and test probably have different configurations. Don't copy and paste the whole block.
<!--Florida State University (FSU) -->
<xsl:when test="$idp='https://idp.fsu.edu'">
    ...
</xsl:when>

OSF
!!! Test server only
After deployment., run python3 -m scripts.populate_institutions -e test -i nationalmaglab
TODO

Ready PR for CR

[x] Define criteria with enum
[x] Fix unit tests
[x] Add new unit tests if necessary

Ready PR for merge/deployment

[x] Update attribute name and value after IdP (test server) is configured
[x] Add MagLab to OSF test server after required information are provided
[x] Deployment notes for DevOps
[x] Postman testing (local) after CAS and Shibboleth work is done

Related Work

[ ] Update OSFI developer guide
[x] CAS PR
[x] CAS and Shibboleth configuration

Changes
TBD
QA Notes
TBD
Documentation
TBD
Side Effects
TBD
Ticket
https://openscience.atlassian.net/browse/ENG-3654

Squashed into<PHONE_NUMBER>eac690e8fc76ae84f4eb71bd81c1fc, consider this merged :octocat:


About a warning from valgrind when running through valgrind

Hi streamvbyte team,
I am getting an "invalid read" warning when running the attached example for streamvbyte through valgrind.
Also, it seems like when I changed to the allocation in the attached example to uint8_t *compressedbuffer = malloc(((COMPRESSED_SIZE*sizeof(uint8_t))+(32-1))/32*32);, the valgrind warning disappears.  Is this because of address alignment requirements for vector instructions?
example.c.txt
==2132== Memcheck, a memory error detector
==2132== Copyright (C) 2002-2015, and GNU GPL'd, by Julian Seward et al.
==2132== Using Valgrind-3.11.0 and LibVEX; rerun with -h for copyright info
==2132== Command: ./a.out
==2132==
==2132== error calling PR_SET_PTRACER, vgdb might block
==2132== Invalid read of size 16
==2132==    at 0x4010E8: _mm_loadu_si128 (emmintrin.h:698)
==2132==    by 0x4010E8: _decode_avx (streamvbyte_x64_decode.c:7)
==2132==    by 0x4010E8: svb_decode_avx_simple (streamvbyte_x64_decode.c:97)
==2132==    by 0x401248: streamvbyte_decode (streamvbyte_decode.c:72)
==2132==    by 0x4005F6: main (example.c:22)
==2132==  Address 0x522969f is 36,479 bytes inside a block of size 36,494 alloc'd
==2132==    at 0x4C2DB8F: malloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==2132==    by 0x4005C0: main (example.c:16)
==2132==
==2132== Invalid read of size 16
==2132==    at 0x401106: _mm_loadu_si128 (emmintrin.h:698)
==2132==    by 0x401106: _decode_avx (streamvbyte_x64_decode.c:7)
==2132==    by 0x401106: svb_decode_avx_simple (streamvbyte_x64_decode.c:99)
==2132==    by 0x401248: streamvbyte_decode (streamvbyte_decode.c:72)
==2132==    by 0x4005F6: main (example.c:22)
==2132==  Address 0x52296a3 is 36,483 bytes inside a block of size 36,494 alloc'd
==2132==    at 0x4C2DB8F: malloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==2132==    by 0x4005C0: main (example.c:16)
==2132==
==2132==
==2132== HEAP SUMMARY:
==2132==     in use at exit: 0 bytes in 0 blocks
==2132==   total heap usage: 3 allocs, 3 frees, 153,642 bytes allocated
==2132==
==2132== All heap blocks were freed -- no leaks are possible
==2132==
==2132== For counts of detected and suppressed errors, rerun with: -v
==2132== ERROR SUMMARY: 2 errors from 2 contexts (suppressed: 0 from 0)

I will review the issue, thank you. I would expect the code to be valgrind-clean.

So yes, this is a documentation bug: we may read up to 16 extra bytes beyond the compressed data, so you should ensure that these bytes are part of an allocated region.


Stackdriver log destination should not be dependent on environment.

When running inside GKE/GCE, Stackdriver logs go to GCE VM instance logs. This is actually pretty inconvenient when compared to running outside GKE.
I set the log name to be the name of the service that is running. In dev, this means I can go to Google Project -> {projectId} and then choose the log of the correct service name to get just the logs from that service.
In production, I have to go to GCE VM Instance -> all logs, because the logs are grouped by instance. If I want to see logs from a given service, my query has to be more complicated than it otherwise should be, to isolate it from all the other services and other things that report into those logs (which is a bunch of kubernetes stuff, fluentd etc).
My expectation is that when I do:
logClient, err := logging.NewClient(ctx, fmt.Sprintf("projects/%s", projectID))
logger := logClient.Logger(serviceName)

the logs go where I specify, and to the same place, regardless of the environment I'm running under. From my perspective, the behaviour currently happening outside of GKE is the more desirable one.
If you don't want to break backwards compatibility for this, could we have a  client option to ignore detectedResource(), so it falls back to monitoredResource()?

Realised I wanted a LoggerOption, not a ClientOption, and then found CommonResource(). For others trying to do this:
import mrpb "google.golang.org/genproto/googleapis/api/monitoredres"

logClient, err := logging.NewClient(ctx, fmt.Sprintf("projects/%s", projectID))
resourceOpt := logging.CommonResource(&mrpb.MonitoredResource{
  Type:   "project",
  Labels: map[string]string{"project_id": projectID},
})
logger := logClient.Logger(serviceName, resourceOpt)

will get you what you want.


Question: basic function declaration

Hi, say I wanted to declare a function like this one:
i32 func() {
    return 200;
}

Would doing something like this suffice?
auto func = tb_function_create(m, 4, "func", somelinkage, etc);
auto constant = tb_inst_sint(func, TB_TYPE_I64, 200);
tb_inst_ret(func, 1, &constant );

Thanks for any help, I've tried looking through the manuals but I was unable to find anything, probably because it wouldn't make much sense to develop a full doc when the IR format is still under development which may change how things work.

irrelevant


update changelog

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

:exclamation: No coverage uploaded for pull request base (main@18ac288). Click here to learn what that means.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #21   +/-   ##
=======================================
  Coverage        ?   90.88%
=======================================
  Files           ?       16
  Lines           ?      472
  Branches        ?        0
=======================================
  Hits            ?      429
  Misses          ?       43
  Partials        ?        0

Flag
Coverage Δ

pytest
90.88% <0.00%> (?)

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

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

I'm using auto-changelog, to track all changes, if you like it you an merge this, or I'll revert

Hmm, turns out to have been a bad idea: it's an RST file, not a markdown one....

sadly no

let me fix the filetype in anohter PR


Fix scalar leak in RankFixer

Fixes #10473.  Updates the RankFixer.close() method to close the BatchedRunningWindowBinaryFixer instance which is caching a scalar.

build

build

@mythrocks it just needs an RapidsPluginImplicits import

build


[move-function-value] When searching for uses to emit notes about in a sub-loop from the move, skip destroy_value.

Once we have detected that we have a use after a move, we do a second pass over
the function looking for uses that are later than the move but are in
sub-loop. In such a case, the use will not be a boundary use but will be LiveOut
due to the loop circularity. In such a case, we need to match what we do
normally with pruned liveness, namely ignore destroy_value uses.
This fixes a false positive note where we emitted an additional remark on a
destroy_value.

@swift-ci test

Int being 32 vs 64 problem.

@swift-ci test


JavaScript functions to raise a phishing

Describe the solution you'd like
A link is passed through multiple JS functions that examine its characters and outputs whether it is a phishing link or not. This prediction is done by an ML model.
I have programmed some functions of my own, would like to add some more and merge them with your repo @avinashkranjan !

Go Ahead @steffigeorge

Thanks @avinashkranjan !

phish_test.txt
some functions that I've coded

Create an PR as draft will be able to review better there.

@avinashkranjan pull request done!


Initial questions at the beginning of the conversation

Description
The goal of this task is to update the initialization controller for the Kyma Companion to streamline the setup process when a request is received. The steps involve initializing conversation history, retrieving information from the Kubernetes cluster, and generating initial questions based on the retrieved context. The function to retrieve information from the Kubernetes cluster must be reusable and manage access control effectively. Context is gotten from the resources we look at in busola-ui
Subtasks
1. Update Initialization Controller:

Modify the initialization controller to handle incoming requests for initializing the Kyma Companion interaction.

2. Initialize Conversation History:

Develop functionality to initialize the conversation history by inserting context related to the resource being discussed.

3. Develop Information Retrieval Function:

Create a function to retrieve information from the Kubernetes and Kyma cluster based on access information provided by the busload backend.
Ensure this function is reusable and manages access control properly, preventing unauthorized access.
Use these resources to form a context for the next steps.

4. Generate Initial Questions:

Develop functionality to generate up to 5 initial questions based on the retrieved context.
Ensure a balance between speed and functionality when selecting the appropriate model for question generation.

5. Return Initial Questions:

Ensure the generated questions are returned as the response to the initialization request.

6. Testing and Performance Metrics:

Write down a test in pytest to asses whether question generation exceeds on average 2.5 seconds or not.

Acceptance Criteria

[x]  The initialization controller handles incoming requests and initializes the conversation history with relevant context.
[ ]  The function to retrieve information from the Kubernetes cluster is reusable and includes effective access control.
[x]  The necessary Kubernetes or Kyma resources are retrieved and used to form context for question generation.
[x]  Up to 5 initial questions are generated based on the retrieved context.
[x]  The generated questions are returned as part of the initialization response.
[ ]  The question generation process is tested, test is written, and the average time taken for generating questions does not exceed 2.5 seconds.

Before the planning have discussion on how api should look like

Create a spike or an issue straight away to choose the necessary tool for prompt testing and integrate it into the companion

Phase 1:

Introduce a new API /conversations.
Inside the API, extract the response body and headers for the k8s cluster. Follow the API structure described in the doc: ai-force/docs/Readme.md
Query the Kubernetes cluster using k8s python client to get the relevant data.
Create the promptTemplate for initial questions.
Inject the kubernetes information as context in the prompt.
Query the LLM using LangChain or naked Gemini for initial questions.
Return the response as described in doc: ai-force/docs/Readme.md

Phase 2:

Add tests for correctness and performance.

Phase 3:

Check if Mansur has initialized the LangGraph.
Integrate the LangGraph instead of directly using LangChain or naked Gemini for initial questions

Phase 4:

Use LangGraph Checkpointing to save the conversation history.

Sample Request:

URI: POST /api/conversations
Body:{
  "resource_kind": "Deployment",
    "resource_api_version": "apps/v1",
    "resource_name": "nginx",
  "namespace": "nginx-wrong-image"
}

Headers:x-k8s-authorization: abc
x-cluster-url: https://api.xyz.shoot.canary.k8s-hana.ondemand.com
x-cluster-certificate-authority-data: abc

Sample Response:
Body:

Headers:

Response time summary output in Blackbox tests:

[ ] Save the context as well


[FEA] Add support for Databricks 12.2 ML LTS

Is your feature request related to a problem? Please describe.
Add support for Databricks 12.2 LTS https://docs.databricks.com/release-notes/runtime/12.2ml.html
Describe the solution you'd like
Add a shim layer for the new Databricks long term support version
Describe alternatives you've considered
Additional context

Is this really done?  Seems to me we need to address most of the followup issues noted in #8282 before we can consider this done.

I interpreted this as done as the linked pull request was merged which enabled the original shim to be available.  If this issue is meant to track overall effort, then we can reopen it.  If so, we should have all the follow-on issues linked to this issue.

Re-opened, with description updated to link to follow-on issues.

Closing this since these items are complete, and the other items have been moved to a follow-on epic https://github.com/NVIDIA/spark-rapids/issues/8860.


Card mechanics explanations

In addition to visual cues (see #74) I really like the hover-over effect explanations in HS. For instance: if a card has Taunt and you hover over it, a more detailed description of the mechanic is shown.
This is really useful for someone learning the game without having to go through and remember extensive tutorials or read a reference guide.

Indeed, I especially thing the Ranged feature could use some in-game explanation, as in addition to preventing return damage, it also only allows the creature to attack each 2nd turn.
In addition, the mana upkeep feature in Mythos could use some in-game explanation.
IMO, having in-game explanations is better than expecting users to read "game rules" documents.

I don't mind "floating" hints like that, probably pretty easy to do the CSS for it, if you guys can get the Angular part working I'll volunteer for the HTML/CSS.

This may be fairly easy to implement to CardModel in Vue.js client using Bootstrap popovers/tooltips.


W3CTTestRunner: Added a console log at the bottom to avoid annoying m…

…essage boxes popups and to have a log to copy with a contextual menu to easily report errors.

Is there any problem with my pr?

this seems to have conflicts with the current codebase, could you sort them out and update the PR?

you can close this


[BUG] Abnormal exit code 255 in Maestral v1.2.0 on macOS

Describe the bug
When attempting to launch the latest version of Maestral.app, nothing happens (GUI-wise) but the system.log records this:
Sep 16 12:54:15 com.apple.xpc.launchd[1] (com.samschott.maestral.87356[55539]): Service exited with abnormal code: 255

To Reproduce
Unknown.
Expected behaviour
The application should launch, represented by the menu bar icon.
System

Maestral version: v1.2.0
OS: macOS Mojave v10.14.6
Qt version: Unknown

Additional context
I was previously using Maestral v1.1.0.

Hrm, I have never seen this before. The bundle was build on macOS Big Sur but with a deployment target of macOS 10.13. It's running just fine on Big Sur... It looks like I'll need to rebuild on a different system.

Thanks. If there's anything I can do on my end to help diagnose the issue.
I can install PyPI version if ever you suspect the issue is not related to the app bundle.

Its definitely a problem with building on Big Sur. I rebuilt the bundle on Mojave now and uploaded the new version. Could you test if this works for you?
On my end, the new build runs fine on both Mojave and Big Sur.

It works on my end, on both of my computers (which both run Mojave). Thanks!
I've now noticed an error that occurs on launch on both computers with Maestral v1.2.0 (not just the first launch):
Sep 16 15:31:39 com.apple.xpc.launchd[1] (com.apple.xpc.launchd.domain.pid.Maestral.56800): Could not resolve origin of domain. XPC services in this domain's bundle will not be bootstrapped: error = 107: Malformed bundle, taint = missing executable

I can't find in my logs if this occurred in previous versions. Despite the error, Maestral seems to be working fine. I tried with a new file to make sure it syncs.

It looks like the executable path is incorrectly specified in the bundle plist. It reads MacOS/Maestral but should just be Maestral. This seems to be this issue https://github.com/pyinstaller/pyinstaller/issues/4413 with Pyinstaller.

And it's updated yet again, with a corrected executable path. Packaging Python for distribution can be mess.

I confirm that the console error is gone on my end. Thanks again!

Excellent. Thanks for reporting.


Add support for configuration file per profile

The user can have an application.properties with general configuration and one file for each profile, like application-dev.profile for dev profile.

https://github.com/quarkusio/quarkus/pull/11145

Properties in the profile aware file have priority over profile aware properties defined in the main file.

https://github.com/quarkusio/quarkus/issues/18292

I've worked on eclipse/lsp4mp#149, which should make it easier to implement this feature. The PR also lets us implement the support for per profile config files in quarkus-ls, so that the base lsp4mp project doesn't need to support it.


Notebook keybindings don't work as expected when focus is in the webview

VS Code 1.61.0-insider + Jupyter extension v2021.9.1001261101
on a windows machine:

open an ipynb file
delete a cell
press "ctrl-z"

expected: cell deletion is undone
actual: nothing happens. Menu Edit->Undo works though. I understand the Jupyter shortcut for undo is "z" but since menu Edit->Undo (which maps to Ctrl-z) works, as a user, I have expected ctrl-z to work the same.

Just talked with @roblourens. This repros when no cell in the notebook is selected (i.e. click anywhere in the notebook to deselect the cell), because in this case ctrl-z is applying to the cell output, not the entire editor. Since the cell is deleted (therefore no need to undo anything in the output), redirecting ctrl-z to apply to the editor would be reasonable.

We can see whether it's possible to know that the webview did not use undo and redirect the command to the list. Not sure whether that is possible though. This should only happen after clicking into the webview

@rebornix @roblourens Are you still able to replicate this issue?
I cannot replicate this anymore,


Builder cannot find modules imported with "@node/x"

I am using electron with SystemJS for the client side of my application. Electron modules are imported via import * from '@node/electron'; as suggested in https://github.com/systemjs/systemjs/issues/1261. Everything works perfectly fine without any additional System configuration.
However, when bundling the application with the builder I get an error: Cannot find module 'electron'. Even if I exclude it from the build it seems that the builder still tries to resolve it:
meta: {
  '@node/electron': {
    "build": false
  }
},

The only thing that worked was mapping the module to @empty, but this now breaks the application since SystemJS is no longer able to resolve @node/electron.
I am not entirely sure of the magic happening behind using@node/x but with the prebuilt version of electron the electron module is nowhere to be found in the node_modules folder. So for the modules provided by Electron my understanding is that they come packaged in the prebuilt version, and the provided require function that we use in both the main and the renderer processes knows how to find them. So even if we wanted we would not be able to include these modules in a systemjs bundle (and I suspect most people would not want to).

Sure this makes sense... I've included a fix in https://github.com/systemjs/systemjs/commit/85bca84c0f7830d6489e7ebfeb3a4fff4af7c9d0.

I can confirm the fix is working and properly excluding "@node/x" modules from the build.

Great, this will come through in the next release, not the current patch, but it shouldn't take more than a week or so I expect.

This fix caused some other breaks and had to be replaced with https://github.com/systemjs/systemjs/commit/9f886bb3bd0633616834bbe2f0e326ce4ac05c6f, but this should still work for your use case.

Released in 0.15.31.


make clear -no-cache is boolean

Updates the docs so that it's clear for users that -no-cache is a boolean that can be set to true or false

Thanks @TheDen!


Label

Note can be set a label
label could be any word (10 symbols)
labels couldn't repeat

update Entity - need to create Migration

#10


Operator: cleanup old kube resources

I've heard the Operator, like kfctl, does not clean up no-longer-wanted resources when upgrading a kubeflow (like a component which has disappeared in the latest version will still exist in the Kubernetes cluster after kubeflow upgrade to latest version).
Could you confirm?
If so, is it planned to add such support in the Operator or will still run using kustomize as its core which does not allow it?

@desaintmartin yes, we are looking at how to support this as part of the upgrade. We will track this issue under #193.


Rewrite UI components

The goal of this PR is to:

Have much better comments and detailed explanations of what that specific part of code does

Better plugins integration for futur API docs

Fasten some renderings by improving code parts

Progress:

[ ] Rewrite

[x] App
[x] TitleBar
[x] Terminals
[x] Terminal
[x] Shell
[ ] WindowControls

[ ] MacOS

[ ] Close
[ ] Maximize
[ ] Minimize

[ ] Windows

[ ] Close
[ ] Maximize
[ ] Minimize

[x] Tabs
[x] Tab
[x] CreateTab

[x] Better plugin support

Not going to rewrite WindowControls for now since they are way to heavy and repetitive and i'm planning on generating them with a script.
It's mergeable now!:tada:


feat/IPROD-93 Disabled ConnectionWizard page

feat/IPROD-93 Disabled ConnectionWizard page

merged in #26


Problem with the progress bar

The progress bars aren't showing the
correct progress area in the bar. @kroy96

I tried searching for errors in JS file but nothing helped.

[UPDATE]: Tried editing the HTML file, there were some wrong entries init.

@AayushVyasKIIT Bravo :) Good Work


Neato exceptions

Making our requests exceptions more user friendly providing some feedback around what is failing and adding more try/catch blocks against all requests to neato cloud.  Please feel free to tweak as needed :)
If possible please publish a new package to pypi so we can include this in home assistant :)

I think we should define only error types that really add some value, because they should be handled differently.

@stianaske  what is your take on this implementation?  Do you think we need to get more specific with error messaging? Like should we detect when a user changes their password, or if the neato api is having issues?

Maybe er should add TimeoutError and handle it same as ConnectionError. https://stackabuse.com/the-python-requests-module/

Do you think we need to get more specific with error messaging? Like should we detect when a user changes their password, or if the neato api is having issues?

I think that we need this here:
https://github.com/Santobert/home-assistant/blob/98f7aaebc0f39ff2295497085cb295557fb56df7/homeassistant/components/neato/__init__.py#L159

@dshokouhi There are two files exceptions.py now. One in the root directory and one in pybotvac. I think we only need the second one. Do you agree?

Looks good. Great work!


Update Composer dependencies (2022-02-21-00-11)

Visual regression test passed!
View the visual regression test report


Improve the CSS

We're mostly backend people, but I'm pretty sure we can do better...

I chanced upon rack-mini-profiler [0] while listening to a podcast [1], and checked out a video [2] about how it works. I thought it looked really simple and clean, and would like to suggest updates to the style for debug toolbar in the direction of that project.
Would the maintainers like to take a quick look? Will be happy to work on this if the core team is interested, thanks!
[0] https://github.com/MiniProfiler/rack-mini-profiler
[1] http://bikeshed.fm/17
[2] http://railscasts.com/episodes/368-miniprofiler

What did you have in mind for style changes? The toolbar currently expects that panels remain exclusive of each other. The rack-mini-profiler shows a general overview that contains data that exists across a few of the toolbars panels.

Suggestion:

This button is the "go back" button.  But it's on the right side, pointing right.  In almost every context, that's the convention to go forward, not back.  I hope I'm not the only one that ends up clicking "back" on my browser.
The suggestion is that nav could be an ounce more intuitive and conventional.

Regarding namespacing, we try to prefix everything with djdt-.

I think that the toolbar CSS might get overridden by Bootstrap too. I'm trying it with Bootstrap 4, which contains this:
[hidden],
template {
  display: none;
}

...which causes the Debug Toolbar to disappear, because it contains: <div id="djDebug" hidden="hidden"...

I think the of improving the UI/UX through better CSS is very welcome, but the goals of this issue are a bit nebulous with no clear end in sight. So closing.
To any reading or participating, please feel free to open PRs improving the UI. Also please feel free to open new issues that identify specific UI issues that can be acted on.

which causes the Debug Toolbar to disappear, because it contains: <div id="djDebug" hidden="hidden"...

This was fixed in 4111d930327f10a47c8bcc6b46f21d5aa62d4460.


[Enterprise InfluxDB] Add anti-entropy documentation

Fixed by https://github.com/influxdata/docs.influxdata.com/pull/1185.

Done.


Fix bug that $set and $unset categories same time.

Fix #1150

Thanks! Btw, please make PRs to devel if you can :)

Ok. I'll.


fix: better snapshot of highway test

Makes the "highway test" more realistic by providing actual typing values.
Changes

Test

Oh, this also fixes https://github.com/ftrackhq/ftrack-ts-schema-generator/issues/29.

And if you'd like to see the original change separated from the naming change, they are in individual commits.
af084ca
8251cb9


refactor(config): config generation and add unit testing

use flag.func instead of manual parsing
add unit tests
add snapshot tests
replace global config with parameter
improve Makefile

Can you break out the coverage and makefile stuff to its own PR? I think we can merge that pretty much straight away, the other things will need some time to look over.

Closing due to inactivity


v0.9.6 release

This release will be using the new org name, so some preparation is required.

Applied for the new org name on Sonatype OSS: https://issues.sonatype.org/browse/OSSRH-16665

Released this morning on bintray and Maven Central!