Initial structure

This PR implements the initial version for sqlalchemy-omnisci
Some information:

using sqlalchemy.testing.suite for tests
the current output for the test is: 66 passed, 207 skipped, 253 deselected
There are a lot of skipped and deselected  (tests that are breaking the tests or that was not possible yet to skip them).
Among the skipped tests are tests that uses some functionality not supported by OmniSci.
The tests for apache superset are still a work in progress, as there are still some configuration and implementation needed in the sqlalchemy-omnisci side, it was necessary to stop the superset tests for now.

All the missing points will be treated in follow-up PRs. the idea is to resolve the problems and decrease the number of deselected tests

thanks @alexbaden !


Verify hard-neg mining notebook works with large unlabeled sets.

Try notebook with negative (aka unlabeled) dataset of e.g. 20k images.

@loomlike has this been tested?

No it has not been tested using a large dataset of >10,000 images. I will do that when I implement this for the OD scenario.

Tested with 7,000 negative images and it's working wonderfully.


Add title and extension to license

The title is not strictly required, but it's useful metadata, and part of the recommended license template text (see http://choosealicense.com/licenses/isc/ and https://opensource.org/licenses/isc-license)
The extension helps with the display of the license on github (it activates text wrapping)

@LukyVj ping :)

Hey, thanks for opening this.
I'll look into it as soon as possible.
I have a full schedule so far!
Thanks again.


Thorium overwrites any Web Data file it did not generate

System Details

OS: EndeavourOS
Thorium version: 122.0.6261.132 (Official Build) stable, built on Ubuntu (64-bit)

Problem
I'm not sure if this should be a feature request or a bug, but Thorium immediatly overwrites the 'Web Default' file which contains various customized options (such as custom search engines).
I was able to flawlessly migrate this file between different chromium-based browsers (Chromium, ungoogled-chromium, Brave), but Thorium overwrites it at launch.

@awsms This is probably because of the extra search engines that I added to thorium. As part of that, an integer called "kProfileDataVersion" is set, and this number is always one version ahead of official Chromium, specifically so that people migrating from older Chromium/Thorium versions will get the new search engines, and get the new favicons with the search engines.

@awsms This is probably because of the extra search engines that I added to thorium. As part of that, an integer called "kProfileDataVersion" is set, and this number is always one version ahead of official Chromium, specifically so that people migrating from older Chromium/Thorium versions will get the new search engines, and get the new favicons with the search engines.

That's what I assumed from the README (with the search engine changes), but why not add them to the Web Data file, instead of replacing any not-thorium made file?
I wasn't able to find the kProfileDataVersion property, is that in a config file? Or the Web Data sqlite one? A bit lost here :(


Greyed out connectors

Sometimes, connector pins have a grey center. Does this mean anything or is it a bug?

I noticed that, too. I wasn't able to figure out what caused the issue and since it didn't affect functionality, I ignored it. I will put this at the end of my fixing queue.


Ask confirmation before quiting

Is your feature already implemented in the latest master?
Not implemented.
Is your feature request related to a problem? Please describe.
I tend to use q to close a window, but it immediately quits the application and stops playing. This is frustrating.
Describe the solution you'd like
Asking for confirmation before quitting could prevent this.

it might be better for you to re-bind q to ClosePopup and use another shortcut to quite the app

Thanks for the suggestion. This solves it.


Camera loading in ionic react-app

when I try to use the scanner in an ionic react app for my android phone, the app runs fine on my laptop webapp and the camera opens, but whenever I build in android studio and try to launch the app on my android device it just shows a big triangle play icon and the word loading

I'm not sure about Ionic, but you could try the new beta and let me know 2.0.0-beta.8.

I got it to work after restarting my pc and android device (idk why), but thank you for the quick reply. Also there is no QrScanner exported in the beta versions and I wanted to use the onDecode so I'm using the 1.2.10 version. For starting and stopping the scanner

The onDecode was removed for onScan. onScan return an array, since now is possible to scan multiple QR Codes at the same time. If you only one Qr Code, just get the first element of it. You can try the beta, it is faster and has a much smaller package size.

am I doing this wrong? because this is what i get when I treat it as an array

This is the type that is returned
export interface IDetectedBarcode {
    boundingBox: IBoundingBox;
    cornerPoints: IPoint[];
    format: string;
    rawValue: string;
}

You should use rawValue instead of text.


Add markdownlint as GitHub Action workflow check

Signed-off-by: David A. Wheeler<EMAIL_ADDRESS>

This sequence is just sadness. Let's use #93 instead :-).


GEN-25: Meeting Notes: Week#2 Meeting#3

To Do

Take notes during the meeting.
Create the wiki page.
Add links to Sidebar and Homepage.

Deadline
18.10.2022 @15.00
Additional Information
No response

Notes are added to both homepage and sidebar.


[10.x] Enhanced GeneratorCommand->qualifyClass() to capitalize the first letter of each word in the class name and remove spaces

We have good processing of migration names, why not do the same for make: command?
so if i make event 'user logged out event', it will create UserLoggedOutEvent.php

Thanks for your pull request to Laravel!
Unfortunately, I'm going to delay merging this code for now. To preserve our ability to adequately maintain the framework, we need to be very careful regarding the amount of code we include.
If possible, please consider releasing your code as a package so that the community can still take advantage of your contributions!
If you feel absolutely certain that this code corrects a bug in the framework, please "@" mention me in a follow-up comment with further explanation so that GitHub will send me a notification of your response.

Hi @taylorotwell , i'm not sure that's something worthy of a package. I just though that it could increase "user-friendliness" of the make: command, if you don't think so - feel free to dismiss this message! :)


No 'Default Date' Option in General Settings

In my test instance I have the following options in General settings:
Company Name:
Country:
Date Format:
However, in my live instance, I don't have the Date Format option, only Company Name and Country.
Why might this be?

hi @AsimPervez93,
Make sure your live instance has the most recent version of code.
Please ensure that you do following when updating code of live version:
https://github.com/timeoff-management/application#updating-existing-instance-with-new-code

Hi @vpp
I get stuck at the db update script with the following error:
Error: Cannot find module '/var/www/html/application/node_modules/.bin/sequelize'

Hi AsimPervez93,
Consider running
npm install
Sent from my iPhone

On 27 Feb 2017, at 14:40, AsimPervez93<EMAIL_ADDRESS>wrote:
Hi @vpp
I get stuck at the db update script with the following error:
Error: Cannot find module '/var/www/html/application/node_modules/.bin/sequelize'
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Hi @vpp
I tried that and still get the error at the db-update script:

Error: Cannot find module '/var/www/html/application/node_modules/.bin/sequelize'
at Function.Module._resolveFilename (module.js:469:15)
at Function.Module._load (module.js:417:25)
at Module.runMain (module.js:604:10)
at run (bootstrap_node.js:394:7)
at startup (bootstrap_node.js:149:9)
at bootstrap_node.js:509:3
npm ERR! Linux 4.4.0-64-generic
npm ERR! argv "/usr/bin/nodejs" "/usr/bin/npm" "run-script" "db-update"
npm ERR! node v6.9.5
npm ERR! npm  v3.10.10
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>db-update: node node_modules/.bin/sequelize db:migrate --config=config/db.json --models-path=lib/model/db/
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>db-update script 'node node_modules/.bin/sequelize db:migrate --config=config/db.json --models-path=lib/model/db/'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the jswebapp package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node node_modules/.bin/sequelize db:migrate --config=config/db.json --models-path=lib/model/db/
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs jswebapp
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls jswebapp
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     /var/www/html/application/npm-debug.log

hi @AsimPervez93,
Do you have "node_modules" in the checkout directory?
If so can you send what directories are there?
The idea of the failing command is to run the "sequelize" file. Can you have quick look around to check if maybe it is in some other place?

@AsimPervez93, one more point, for migrations the system uses the sequelize-cli tool:
http://docs.sequelizejs.com/en/v3/docs/migrations/#the-cli
Please check how they suggest to install it.
I've used local installation and it worked. Maybe you can consider global installation?

@vpp
I'm not sure what was causing this.  I ran apt-get update and apt-get upgrade, then restarted the machine.
Then ran npm install again and this seemed to have worked.  I successfully was able to update from git.  I know this has worked because the app.js has returned to default.

@AsimPervez93
Does it mean that the issue is sorted?

@vpp
I'm just testing it just now.  I'll comment again in 15 mins or so once I've tested the Date format.

Hi @vpp
I get this error after logging in:

Missing helper: "as_date_formatted"

Error: Missing helper: "as_date_formatted"
at Object. (/var/www/html/application/node_modules/handlebars/dist/cjs/handlebars/helpers/helper-missing.js:19:13)
at eval (eval at createFunctionContext (/var/www/html/application/node_modules/handlebars/dist/cjs/handlebars/compiler/javascript-compiler.js:254:23), :3:130)
at Object.prog [as inverse] (/var/www/html/application/node_modules/handlebars/dist/cjs/handlebars/runtime.js:219:12)
at Object. (/var/www/html/application/node_modules/handlebars/dist/cjs/handlebars/helpers/if.js:17:22)
at eval (eval at createFunctionContext (/var/www/html/application/node_modules/handlebars/dist/cjs/handlebars/compiler/javascript-compiler.js:254:23), :8:32)
at prog (/var/www/html/application/node_modules/handlebars/dist/cjs/handlebars/runtime.js:219:12)
at execIteration (/var/www/html/application/node_modules/handlebars/dist/cjs/handlebars/helpers/each.js:51:19)
at Object. (/var/www/html/application/node_modules/handlebars/dist/cjs/handlebars/helpers/each.js:61:13)
at Object.eval (eval at createFunctionContext (/var/www/html/application/node_modules/handlebars/dist/cjs/handlebars/compiler/javascript-compiler.js:254:23), :42:31)
at main (/var/www/html/application/node_modules/handlebars/dist/cjs/handlebars/runtime.js:173:32)
at ret (/var/www/html/application/node_modules/handlebars/dist/cjs/handlebars/runtime.js:176:12)
at ret (/var/www/html/application/node_modules/handlebars/dist/cjs/handlebars/compiler/compiler.js:525:21)
at ExpressHandlebars._renderTemplate (/var/www/html/application/node_modules/express-handlebars/lib/express-handlebars.js:247:12)
at ExpressHandlebars. (/var/www/html/application/node_modules/express-handlebars/lib/express-handlebars.js:173:21)

Hi @vpp
Again, I'm not sure what caused this but I was able to resolve it by deleting the entire application folder and restoring a backup version, then doing the npm install again, then updating the git files.
Since updating the files, I can confirm that the Date format is now present in the General Settings.


Update the installation instructions.

Fixes #123.

Sure, I will test the installation docs tomorrow!
For uploading the docs to the INRIA server what do you mean exactly?

For uploading the docs to the INRIA server what do you mean exactly?

Docs from https://github.com/tlaplus/tlapm/tree/main/doc/web have to be published somehow at https://tla.msr-inria.inria.fr/tlaps/. That should be done in the CI (maybe on a release, not each commit). Such publishing is done for  tlaplus here: https://github.com/tlaplus/tlaplus/blob/03c7bf4f35ccf1217ac2c53e5bf60d1220c8b0d8/.github/workflows/main.yml#L180. Maybe TLAPM CI scripts should be updated similarly.

@kape1395 the docs website is now served using github pages from the main branch of this repo at http://proofs.tlapl.us/doc/web/content/Home.html

@ahelwer, thanks for the update! Maybe you have reviewed the changes to the installation instructions?
@lemmy, we should redirect somehow https://tla.msr-inria.inria.fr/tlaps/ to https://proofs.tlapl.us/, otherwise it will be confusing to have two sites for docs. Only the first one is suggested by google, for me at least.

@lemmy, we should redirect somehow https://tla.msr-inria.inria.fr/tlaps/ to https://proofs.tlapl.us/, otherwise it will be confusing to have two sites for docs. Only the first one is suggested by google, for me at least.

Done:
root@tla:/etc/apache2/sites-enabled# tail -7 default-ssl.conf

	Redirect 301 /tlaps https://proofs.tlapl.us/

	</VirtualHost>
</IfModule>

@lemmy, this works partially. Paths are different, redirects should consider this, IMO. For example Google suggests https://tla.msr-inria.inria.fr/tlaps/content/Download/Binaries.html
Which is redirected to https://proofs.tlapl.us//content/Download/Binaries.html and gets 404.
It should be redirected to https://proofs.tlapl.us/doc/web/content/Download/Binaries.html (consider doc/web in the path).

Do you know how to set this up in Apache 2?

Please try again and let me know:
root@tla:/etc/apache2/sites-enabled# tail -7 default-ssl.conf

	Redirect 301 /tlaps https://proofs.tlapl.us/doc/web

	</VirtualHost>
</IfModule>

Looks good now. Thanks!

@lemmy, @ahelwer, the MacOS version cannot be built anymore.
The scripts refer to macos-latest, and it points now to macos-14-arm64. It was macos-12 in the latest successful build.
The old Isabelle (2011) does not work with the ARM chips.
The new Isabelle (2024) is in progress. All the non-skipped proofs in the Examples repo have been built successfully, but some degradations still exist (it looks like something with integers/naturals).
The question:

Should I merge this PR and have the docs updated (but the build on the main is broken).
Wait until the Isabelle branch is merged (#109, #124) and have the Installation instructions with broken links until then.

Can we split the macOS build into intel and arm64?

I think we will have to. Maybe for now, we can stick to macos-13 for the Intel version?
See https://github.com/actions/runner-images?tab=readme-ov-file#available-images

Is there a way to do cross-compilation instead of having to run the build on an intel mac?

I don't know; I never tried to do that.

Are we forced to compile on macos-latest? Why not an older runner: https://docs.github.com/en/actions/using-github-hosted-runners/about-github-hosted-runners/about-github-hosted-runners#standard-github-hosted-runners-for-public-repositories?

It works with macos-13, so I'll keep it like this for now.


Not able to unhide the wallet by using biometric.

Uploading Record_2023-11-10-15-22-36.mp4…

Not able to unhide the hidden wallet using biometric.
Initially if wallet has no balance and we hide it then unhiding is possible with biometric but this is happening only in ios.

Verified this issue on keeper dev app v 1.1.7(263)


Use p-d instead of the wiki everywhere

We missed a few yesterday.
I factorized a bit the code. It is possible to do better but well..

@jcristau The patch I was mentioning earlier
   before (last 100 chars): '>\r\n    from auto_nag.common import getCurrentVersion\r\nImportError: No module named auto_nag.common\r\n'

Coverage decreased (-0.004%) to 48.333% when pulling ebce93b978303d53866611f093eebf98bdd12b0d on sylvestre:master into dfcb1d29d7d8b20f797b7cac4c60b18b2064b36d on mozilla:master.


doesn't seem to separate and/or from field names

added potential fix here: https://github.com/goatquery/goatquery-go/pull/24


Is this project still maintained

Hi @avianto,
Just wondering if this project is still being maintained. There are a number of outstanding PR's which haven't been either merged or declined and also issues which are unanswered.
The Hugo Themes team is in the process of updating their guidelines for accepting and keeping themes in the showcase. If you're not wanting to continue maintaining this theme, I'm happy to submit my forked version, which has been updated to work with version 0.55.0 and later.

Hi @funkydan2,
I will be very happy if you want to keep maintaining this project with your fork.
As much as I want to keep on maintaining mine, seems like time is not on my side for unforeseen future. I might circling back someday, but I can't make any promise now.
Thanks for keeping this theme alive, appreciate it.


WIP

Fixes #2223

[ ] Title summarizes what is changing
[n/a] Includes a news entry file (remember to thank yourself!)
[x] Unit tests & code coverage are not adversely affected (within reason)
[x] Works on all actively maintained versions of Python (e.g. Python 2.7 & the latest Python 3 release)
[x] Works on Windows 10, macOS, and Linux (e.g. considered file system case-sensitivity)
[n/a] Dependencies are pinned (e.g. "1.2.3", not "^1.2.3")
[n/a] package-lock.json has been regenerated if dependencies have changed

A title called "WIP" does not warrant a summary of what is changing, hence I took back a ✔ 😉


React Group Project

Hi Reviewer 👋,
This is the link to the project: link
Partners:

@codedit334
@ShiroYaksha90
@shubhambhoyar077

Hi Team,
Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉
Highlights 💯

Descriptive info ✔
Tasks distributed equally ✔

Optional suggestions
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.

Hi @gemmen29,
Thanks for approval.


F2 doc fields

Added proc macro to create *_doc: Option<String> for most vehicle fields that can be skipped with #[doc_field(skip_doc)] attribute.
TODO:

[ ] publish fastim-proc-macros with new version
[ ] fix Cargo.toml path to not be local
[ ] improve documentation in fastsim-proc-macros/lib.rs

@kylecarow , this isn't ready to merge, but it's a decent idea for you to play with it.

After discussion today, it seems we are in agreement that a separate doc struct is the way to go for this


Add django celery beat and make celery beat the default scheduler

Resolves #55

Tests failed because I broke develop before I PRed off it. The merge should fix this.


$MaxQueue is 0 in End Block, PowerShell 2

MaxQueue is set to 0 in End block.  Reproducible in PowerShell 2 (Native and -Version switch):
1..2 | Invoke-Parallel {"What?"} -Verbose

Result:
VERBOSE: Throttle: '20' SleepTimer '200' runSpaceTimeout '0' maxQueue '0' logFile 'C:\temp\log.log'
VERBOSE: Creating runspace pool and session states
VERBOSE: Creating empty collection to hold runspace jobs
VERBOSE: Adding 1 to collection at 1/20/2015 7:52:26 PM
VERBOSE: "1/20/2015 7:52:26 PM";"Removing:'1'";"0 minutes";"Completed";
what
VERBOSE: 0 items running - exceeded 0 limit.

First item runs, then we get stuck thinking we've reached the MaxQueue ceiling.

Not sure exactly what happened.  Begin block sets $MaxQueue, verified with Verbose output.
In PowerShell 2 only, End block, Try block, we don't see this unless we reference the Script scope.


Rocketmq uses alibaba fastjson remote code execution vulnerability

Fastjson is less than 1.2.68 There is a remote command execution vulnerability. Alibaba's official website has also updated the security version for this purpose, and it is strongly recommended to fix. All versions of fastjson have remote command execution. Hackers can directly obtain server permissions. please update.
alibaba news:
https://github.com/alibaba/fastjson/wiki/security_update_20200601
https://github.com/alibaba/fastjson/releases

#2008 fixed it


Stream only updating once per second

I am only receiving an update once a second. This is not usable to create a real-time compas.

https://github.com/hemanthrajv/flutter_compass/pull/68
This fixed it

dependency_overrides:
  flutter_compass:
    git:
      url: git@github.com:hemanthrajv/flutter_compass.git
      ref: 48e155bc647593e588fdeb5b22ed066afc1275ef

@hemanthrajv is it possible to create a new release on pub.dev

I tried using this and got git access error. I solved it with using https. (Answer found here https://githubmemory.com/repo/trevorwang/retrofit.dart/issues/345, last comment))
dependency_overrides:
  flutter_compass:
    git:
      url: https://github.com/hemanthrajv/flutter_compass.git
      ref: 48e155bc647593e588fdeb5b22ed066afc1275ef

I tried using this and got git access error. I solved it with using https. (Answer found here https://githubmemory.com/repo/trevorwang/retrofit.dart/issues/345, last comment))
dependency_overrides:
  flutter_compass:
    git:
      url: https://github.com/hemanthrajv/flutter_compass.git
      ref: 48e155bc647593e588fdeb5b22ed066afc1275ef

That just means that your ssh keys are not setup correctly

That just means that your ssh keys are not setup correctly

Oh, good to know, thanks.

Fixed in #68

Create a new ticket #80 so we can get this updated to pub.dev


[CT-3233] [Bug] var() cannot see vars defined in dbt_project.yml in query-comment context

Is this a new bug in dbt-core?

[X] I believe this is a new bug in dbt-core
[X] I have searched the existing issues, and I could not find an existing issue for this bug

Current Behavior
According to the docs var() is available in the Jinja context for query-comment. However, it only partially works.
When calling var() in the value of query-comment, either as a templated literal or in a macro, vars passed on the DBT command line using --var work as expected. But vars defined in dbt_project.yml are not found.
Judging by the error message it seems as if vars specified in dbt_project.yml are unavailable when query-comment is being evaluated. I cannot find any documentation to that effect (in particular neither here nor here).
Expected Behavior
I expect that var() is able to resolve the values of vars defined in dbt_project.yml, in all cases, including in the context of query-comment.
Steps To Reproduce
dbt_project.yml:
name: 'dbt_query_comment_test'
version: '1.0.0'
config-version: 2
profile: 'default'
# all the standard paths...
models:
  dbt_query_comment_test:
    +materialized: table
vars:
  foo: "bar"
query-comment: "foo is {{ var('foo') }}"

result of dbt run:
01:22:37  Running with dbt=1.6.6
01:22:37  Registered adapter: postgres=1.6.6
01:22:37  Unable to do partial parsing because a project config has changed
01:22:38  Encountered an error:
Compilation Error
  Required var 'foo' not found in config:
  Vars supplied to <Configuration> = {}

Similarly, if we write a macro like what's in the docs:
{% macro get_query_comment(node) %}
    {% set comment_dict = {"some_key": "some_value"} %}

    {% set _foo = var("foo") %}
    {% do log("_foo=" ~ _foo, info=True) %}
    {% do comment_dict.update(foo=_foo) %}

    {% do return(tojson(comment_dict)) %}
{% endmacro %}

...and adjust dbt_project.yml:
query-comment: "{{ get_query_comment(node) }}"

...and do dbt run:
01:25:21  Running with dbt=1.6.6
01:25:21  Registered adapter: postgres=1.6.6
01:25:22  Encountered an error:
Compilation Error
  Required var 'foo' not found in config:
  Vars supplied to <Configuration> = {}

  > in macro get_query_comment (macros/get_query_comment.sql)
  > called by <Unknown>

Note that running dbt run --vars '{foo: bar}' does work, with both a simple templated query comment, and with the macro call.
(I normally use DBT with BigQuery but I put together this reproduction case with Postgres. I have no reason to believe this behavior is adapter-specific.)
Relevant log output
No response
Environment
- OS: MacOS 12.5.1 (Monterey)
- Python: 3.10.6
- dbt: 1.6.6

Which database adapter are you using with dbt?
postgres, bigquery
Additional Context
No response

Thanks for an excellent write-up @slotrans!
Documentation

Judging by the error message it seems as if vars specified in dbt_project.yml are unavailable when query-comment is being evaluated. I cannot find any documentation to that effect (in particular neither here nor here).

Indeed, within the dbt_project.yml context, only variables defined with --vars are available when using var. This is called out here, but we should consider adding it to one or both of the pages that you mentioned at well (and maybe here too).
Feature request
Regardless of the documentation, it still isn't doing what you hope or expect!
Namely it doesn't allow you to define vars within the dbt_project.yml context and then use them within your query-comments configuration 💥
👉 I think https://github.com/dbt-labs/dbt-core/issues/2955 would accomplish exactly what you are looking for. We would appreciate any comments or upvotes you want to add to that one. In the meantime, I added a quick comment here that notes your use-case.
I'm going to close this as covered by #2955. But I've opened https://github.com/dbt-labs/docs.getdbt.com/issues/4301 to improve the documentation about the current behavior.


Remove FetcherHistory service

Context
We replace the old FetcherHistory service with the new use for the data layer PriceDataRepository.
To-do list

[ ] Added tests
[ ] Tested on dev/sbx
[ ] Changelog was updated with current changes
[ ] Documentation or guides were updated (slab, readme)
[ ] Version was updated on package.json (releases/hotfixes)

Checklist before merge

[ ] there are no errors in logs in any workers
[ ] new blocks are being discovered and marked as finalized
[ ] foreign blocks are being dispatched
[ ] squash all commits before merging

you also need to add migration stript that will remove history collection from mongo.


Use UTF-8 charset in api-db connection

Checklist

[x] Affected Issues have been mentioned in the Closing issues section
[ ] Documentation has been written/updated
[x] PR title is ready for changelog and subsystem label(s) applied

The default character set used by MariaDB is latin1, a single-byte encoding scheme. Lagoon currently doesn't set the charset on the DB connection, so it uses the default latin1.
This means that if a multibyte UTF-8 character was encoded and written to the DB, then read back and decoded as latin1, it would cause mojibake.
Taking the string from #1681  and reversing the encoding/decoding shows that it has likely been mis-decoded as latin1:

This PR sets the charset of the database connection to UTF-8 to avoid this issue.
In this example I'm listing the project from #1681 using lagoon-cli:
Before:

After this PR:

Closing issues
Closes #1681

@smlx is there a need to correct any data that's already stored, or is it just a connection config issue?

hi @rocketeerbkw good question
project names seem to be protected from non-ascii chars by the API:
lagoon -l local add project -p foo… -g git://example.com/foo.git -S 2 -E prod
Error: graphql: Only lowercase characters, numbers and dashes allowed for name!

however environment names are not:
$ lagoon -l local list environments -p foo
ID	NAME	DEPLOYTYPE	ENVIRONMENT	ROUTE
15	foo…	branch    	development	none

The issue I saw in #1681 was in an environment name, but it was in an old lagoon installation that was migrated from pre-RBAC. So maybe the migration script connected to the DB using utf8 to write the environment names or something..
In my testing this will cause issues for fields that have non-ascii glyphs in them.
so you're right, this needs more consideration before merging. :thinking:

@smlx
there are entrypoints that are running when we update the DB:
https://github.com/amazeeio/lagoon/tree/master/services/api-db/docker-entrypoint-initdb.d
we could add a check in there to see if there is any wrong characters in the DB and throw an error that it should be handled manually by the admins?

I did a bit of reading on this, and I think there's a way to fix it cleanly going forward, without breaking existing lagoon installations/DBs.
Rather than the patch that I've got in this PR which sets the charset on the connection, the default server charset and collation should be set at the server level via:
SET character_set_server = 'utf8mb4';
SET collation_server = 'utf8mb4_unicode_ci';

This can be done in the DB update scripts that @Schnitzel linked above.
What this should do is only set the new encoding for new databases. Otherwise it will fallback to whatever encoding is used by the existing database. So effectively it will mean that new Lagoon installations will be utf8, while old ones will remain latin1 to avoid breaking anything.
When I get a chance I'll do some testing to make sure this works as expected.


Enhancements to Macro Examples: Unit Tests and Code Organization

This PR introduces a set of improvements to the macro examples, including the addition of unit tests, the categorization of macro interfaces, and the restructuring of the directory to make it more accessible and navigable.
Changes:

Add Unit Tests to Macro Examples.
Categorize Macro Interfaces and Standardize Playground: I've categorized macro interfaces and added MARK comments. This standardization not only cleans up the code but also makes it easier to find your way around.
Organize Macro Examples into Type-Specific Directories: To streamline the search process for newcomers, I've restructured macro examples into type-specific directories.

@swift-ci Please test

@swift-ci Please test

I believe something went wrong with Swift Test Windows

@swift-ci please test Windows platform


fix failing test

the content that the regex checked has changed.
probably want to make it more open so that it can capture this in the future.
also updated GHA checkout from v3->v4
and bumped version to 1.26.234
Where is this performed automatically?
I see commits from github bot but no workflow to support.

also, where does the webhook section of the release workflow report to? I've commented this out to get a release on my fork until you merge this.

Work on my fork was showing up here. :/
closed the PR but feel free to use the fixed test.


Breaking change in HikariDataSource constructor behaviour in 2.7.7

Hello,
It seems that the behaviour of the constructor for HikariDataSource(HikariConfig configuration) has changed between 2.7.6. and 2.7.7. The data source no longer contains the information passed to it via the given configuration and instead just the default values. Not sure what the underlying reasons for this was but it breaks code which assumed the information was there.
It used to contain configuration.copyStateTo(this) which got replaced by constructing and initialising a copy configuration.copyStateTo(copy).
Is it intended for the HikariDataSource to return just the default values on all of the methods that it exposes by inheriting from HikariConfig? Is there a list of any other potential breaking changes in that release?

2.7.8 is released, does that fix your problem?

@mkurz @MikeBlink v2.7.8 should fix this issue, but I will await confirmation before closing it.

@mkurz @brettwooldridge Works like a charm, thank you.


feat: support all sam local cli options

support all sam local cli options belows:

https://docs.aws.amazon.com/serverless-application-model/latest/developerguide/sam-cli-command-reference-sam-local-start-api.html
https://docs.aws.amazon.com/serverless-application-model/latest/developerguide/sam-cli-command-reference-sam-local-start-lambda.html

change Props of lib/generateAppModel.ts and lib/generateProxyModel.ts
I just only tested lib/generateAppModel.ts through that PR https://github.com/dealmore/terraform-aws-next-js-image-optimization/pull/13

Any idea to test the lib/generateProxyModel.ts ?

Wow, adding all options is a huge improvement, thanks!
Never thought too much about testing for this module since it is only used in test context itself.
lib/generateProxyModel.ts is used by terraform-aws-next-js, I will check the compatibility there before releasing a new version 👍

Now released as v1.5.0.


Dualshock 3 disconnected but not really

[x] I have searched open and closed issues for duplicates
[ ] I am submitting a bug report for existing functionality that does not work as intended
[x] I have read https://vigem.org/Community-Support/
[x] This isn't a feature request or a discussion topic

Bug description
This is ridiculously bizarre.
Basically, whenever I'm playing MGS3 HD on RPCS3 and get hit by The Fear boss, the Dualshock 3 on windows/rpcs3/etc loses all of its functionality apart from vibration. It's still recognized by windows and software, but the buttons/analogs stop working altogether. Nevertheless, you can still send vibration signals to it through applications. I can't even fathom the reasoning for this and how specific it seems to be.
Obviously, the hit from that boss triggers RPCS3 to send a signal to the controller, probably a vibration signal, and for some reason that signal is breaking every other functionality from the controller apart from vibration. This is my assumption. I am playing wired with device mode SXS, might I add.
Steps to reproduce
Play MGS3 HD, get hit by a dart from The Fear while in first-person.
Actual result: Controller loses all functionality apart from vibration.
Expected result: Controller shouldn't lose any functionality.
Screenshots
Machine info
CPU-Architecture: i5-10600K
Windows version: Windows 11 22000
Software/driver version(s):
DSHIDMINI 2.2.282.0
RPCS3 0.0.23-13988-052725fd Alpha | master
Any other helpful information

Wired or wireless?
Try playing around with these settings.
Cheers

I mentioned wired in the description.

Indeed, my bad.

I've also tried different values in the app. Enabled/disabled output report rate control, duplication, and different values for the rate control (from 50 to 300ms). Applied and restarted app after each change.

Hm, interesting case. Let me think about it.


[SRVKS-281] Explicit RBAC resources for non-cluster admins

As per title, this patch defines RBAC resources explicitly instead of "*".
This is a workaround until SRVKS-308 is addressed.

Since upstream has already moved the KS CR to operator.knative.dev why not just do the same downstream instead of this?

@jcrossley3 Yes, I agree. If we can solve SRVKS-308 soon, this PR is not necessary.
Unlike upstream, downstream(SRVKS-308) might need to prepare upgrade/migrate steps, so I wonder that it takes a little bit long time and filed this PR.

Unlike upstream, downstream(SRVKS-308) might need to prepare upgrade/migrate steps, so I wonder that it takes a little bit long time and filed this PR.

Good point. We can avoid that migration until we start consuming upstream's operator. We can stick with this for now. Sold! :)


visionmedia/class.js.git doesn't exist (anymore ?)

while updating submodules for vendor/express I get :
~/local/realie/vendor/express$ git submodule update
Initialized empty Git repository in /home/arthur/local/realie/vendor/express/lib/support/class/.git/
fatal: The remote end hung up unexpectedly
Clone of 'git://github.com/visionmedia/class.js.git' into submodule path 'lib/support/class' failed

That old version is gone too :(


Catalog is not displayed in nightly

Catalog view in nightly displays only the top bar without any APIs
Steps to reproduce

Log into https://nightly.apinf.io
Click Catalogue
(Optionally, try clicking any of the buttons in the top bar: My APIs, My Bookmarks or toggling the views)

Observed behavior
The view does not contain any APIs, i.e. there is no catalog
Expected behavior
The default API catalog view (card view) is displayed
Environment

Web browser: Chrome 51.0.2704.103
Operating system: Windows 10

@jykae @brylie Please inform @saralavanip when the fix is available so that she can start testing Catalog.

Verified resolved issue on nightly.apinf.io


gui-apps/waylock: Update BDEPEND limit of zig 0.10

waylock will builded fail at zig 0.11

Thanks for the PR!


WARNING 'werkzeug.local.LocalProxy' is not mapped

Bug description
I got the below error in logs:
WARNING:root:Class 'werkzeug.local.LocalProxy' is not mapped
WARNING:root:Class 'werkzeug.local.LocalProxy' is not mapped
WARNING:root:Class 'werkzeug.local.LocalProxy' is not mapped
WARNING:root:Class 'werkzeug.local.LocalProxy' is not mapped

How to reproduce the bug
Noting
Screenshots/recordings
No response
Superset version
4.0.1
Python version
3.11
Node version
I don't know
Browser
Chrome
Additional context
No response
Checklist

[X] I have searched Superset docs and Slack and didn't find a solution to my problem.
[X] I have searched the GitHub issue tracker and didn't find a similar bug report.
[ ] I have checked Superset's logs for errors and if I found a relevant Python stacktrace, I included it here as text in the "additional context" section.

I get this in my logs as well.  I'm running 4.0.2 so while Dosu says a relevant PR was merged in 4.0.0, I confirm it did not quiet this warning.  I believe this warning has existed as far back as 2.0.0, if not farther - I don't think it indicates a warning but it clutters the logs.

Yes, it’s the same. Do you have any suggestions for avoiding this clutter in the logs?

In a perfect world this warning would go away, but in case you want to simply tune out the log noise you can add something like this to your custom LoggingConfigurator class:
import warnings
warnings.filterwarnings("ignore", message="Class 'werkzeug.local.LocalProxy' is not mapped")

(I am assuming here that you pass in your own LOGGING_CONFIGURATOR via a config.py override.)

Thanks for the comment... I'll close this issue as a duplicate of https://github.com/apache/superset/issues/26020 in the meantime, and assume people find their way to this tip regardless of which one thread they follow :)


Sphinx VLS ESP32 naming contest

The Sphinx team is looking to name its new hardware signing device, and we are hoping you can help. The device allows you to run a hosted lightning node while keeping the keys private. The features we would like to reflect in the name are “fast, secure, and innovative.” Whoever comes up with the best name will receive 150,000 sats!
Linked is an article further explaining how the signing device operates.
We look forward to your submissions!
[A Lightning Nodes Problem with Hats](url)

SafeLN

Bolt Warden

SALSA - Sphinx Awesome Lightning Signing Apparatus

I've got a couple of ideas. In case you go for one entry, pick the first.
Hieroglyph
Sphinx Alley
The Great Piramide

Baal - ancient egyption god of lighting ( well thunder ... but you know... )

Hieroglyph

i kinda like that, or just “glyph”

Hieroglyph

i kinda like that, or just “glyph”

I kinda like it too keep your private keys written on the glyph also fits the Egyptian theme

Sphinx Flash Train (SFT)

Papyrus
Ostracon
Scarab (used for impression seals)

I like Scarab
how about "Lightning Scarab"

I have a few suggestions for the name:

Sphinx HSM
Medjay LS (Lightning Signer): https://www.toptenz.net/10-fascinating-facts-medjay-ancient-egypt.php
Medjay Key Manager (MKM)
SfinksCypher
SphinxCypher

I've got a couple of ideas. In case you go for one entry, pick the first.
Hieroglyph Sphinx Alley The Great Pyramid

Ok @NickGSzabo we are going with glyph… since it came from your suggestion, you win the 150k sats! I’ll try to find you on Sphinx


Handler additional artifacts and custom load methods

Explore how we could handle additional artifacts and support custom load methods.

This method should receive the model folder within MLServer as a parameter.


Webhooks

cc @larhauga
I think we could look at exposing the secret here, and SSH key in the other PR, as connection details


Pattern matching for Option and Result

I saw this here https://github.com/Threestup/monads
Looks like a neat approach for pseudo pattern matching.
let result = Some("Sam")

let message = result.match({
    Some: res => `Hello ${res}`,
    None : () => 'Hello World',
})

It would be great to have this. In the example above I made the None case be a function, because you might have an expensive computation there or side effects.

Hello,
Funny, It had it back then:  https://github.com/AlexGalays/option.ts/blob/master/option.ts#L50
I must have removed it because it wasn't very useful compared to fold and simulating pattern matching in a language that doesn't have them at all is a bit surprising.
I would be more inclined to wait for this proposal: https://github.com/tc39/proposal-pattern-matching and make spacelift nice to use with this new syntax.

Oh fold, I didn't even look at that. I was unfamiliar with this name for that. That works just fine.


Firefox "Apollo Client Devtools" broken link

The link to the Firefox DevTools addon is broken:
https://addons.mozilla.org/en-US/firefox/addon/apollo-developer-tools/

Hi! Please file issues related to Apollo DevTools in the appropriate repo: https://github.com/apollographql/apollo-client-devtools. Thanks!


pattern-finder.js - try/catch for receiveIframeMessage()

If there are some other messages triggering
window.addEventListener("message")
````
the JSON.parse(event.data) fails.
````
JSON Parse error: Unexpected identifier...
````
It's in 8 files /core/styleguide/js

@christianhaller -
Thanks for the heads up on this. It's been implemented as part of the Pattern Lab 2 release. It won't be fixed in PL1. I apologize for the delay.


Problem with elementum on kodi smart tv philips

Hello crack, first tell you that you are an artist, I have had a problem for 1 month, before that it has been working without problemsIt's a Philips smart TV, For a month or so I have been getting an elementum connection error:
There is no JSON-RPC connection available for Kodi. Enable options in Application Control in Settings > Service Settings > Control and restart Kodi.
But when I go to this configuration, it tells me startup error, I have tried a thousand things, reinstall kodi, tv, restart router..., it only happens to me on this tv , I also have a fire tv max, this doesn't give me any problems, if you have any idea why this happens to me, before it was going great but now I don't know what to do, thank you.

Expected Behavior

Current Behavior

Possible Solution

Steps to Reproduce (for bugs)

Context

Your Environment

Version used:

Environment name and version:

Operating System and version:

Link to Kodi debug log:

@peke198822 try to reinstall Kodi (remove and install again, so Kodi will have default configuration).
If it will not help - make debug log file , see http://elementum.surge.sh/contribute/log/

Thank you very much for the quick response, I reinstalled the entire Kodi,
I reinstalled the entire TV, it didn't work either, I reinstalled the
router, it didn't work either, I don't know what else to do. I've been
adding these days to upload movies in blue crystal via acestream, I don't
know if it has anything to do with it, I hope you have a solution for my
case.
El mié, 14 ago 2024, 7:43 p. m., Anton Soroko @.***>
escribió:

@peke198822 https://github.com/peke198822 try to reinstall Kodi (remove
and install again, so Kodi will have default configuration).
If it will not help - make debug log file , see
http://elementum.surge.sh/contribute/log/
—
Reply to this email directly, view it on GitHub
https://github.com/elgatito/plugin.video.elementum/issues/1056#issuecomment-2289444198,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AW5NZVO6JUUA3A376XIVLX3ZROJLZAVCNFSM6AAAAABMB6HYKGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEOBZGQ2DIMJZHA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hi again, with the last thing I meant that after starting movies via
acestream then elementum loaded, and I exited Kodi, I re-entered and it
worked without problems.
El mié, 14 ago 2024, 8:28 p. m., dani fernandez @.***>
escribió:

Thank you very much for the quick response, I reinstalled the entire Kodi,
I reinstalled the entire TV, it didn't work either, I reinstalled the
router, it didn't work either, I don't know what else to do. I've been
adding these days to upload movies in blue crystal via acestream, I don't
know if it has anything to do with it, I hope you have a solution for my
case.
El mié, 14 ago 2024, 7:43 p. m., Anton Soroko @.***>
escribió:

@peke198822 https://github.com/peke198822 try to reinstall Kodi
(remove and install again, so Kodi will have default configuration).
If it will not help - make debug log file , see
http://elementum.surge.sh/contribute/log/
—
Reply to this email directly, view it on GitHub
https://github.com/elgatito/plugin.video.elementum/issues/1056#issuecomment-2289444198,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AW5NZVO6JUUA3A376XIVLX3ZROJLZAVCNFSM6AAAAABMB6HYKGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEOBZGQ2DIMJZHA
.
You are receiving this because you were mentioned.Message ID:
@.***>

But when I turn the TV off and on again, it no longer works, nor does it
work following the steps above. I would love to know where the problem is.
I am sending you the two images that I get.
El mié, 14 ago 2024, 9:02 p. m., dani fernandez @.***>
escribió:

Hi again, with the last thing I meant that after starting movies via
acestream then elementum loaded, and I exited Kodi, I re-entered and it
worked without problems.
El mié, 14 ago 2024, 8:28 p. m., dani fernandez @.***>
escribió:

Thank you very much for the quick response, I reinstalled the entire
Kodi, I reinstalled the entire TV, it didn't work either, I reinstalled the
router, it didn't work either, I don't know what else to do. I've been
adding these days to upload movies in blue crystal via acestream, I don't
know if it has anything to do with it, I hope you have a solution for my
case.
El mié, 14 ago 2024, 7:43 p. m., Anton Soroko @.***>
escribió:

@peke198822 https://github.com/peke198822 try to reinstall Kodi
(remove and install again, so Kodi will have default configuration).
If it will not help - make debug log file , see
http://elementum.surge.sh/contribute/log/
—
Reply to this email directly, view it on GitHub
https://github.com/elgatito/plugin.video.elementum/issues/1056#issuecomment-2289444198,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AW5NZVO6JUUA3A376XIVLX3ZROJLZAVCNFSM6AAAAABMB6HYKGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEOBZGQ2DIMJZHA
.
You are receiving this because you were mentioned.Message ID:
@.***>

What Kodi tells me to do, it doesn't let me activate it, it tells me there
is no connection to the local server.
El mié, 14 ago 2024, 9:05 p. m., dani fernandez @.***>
escribió:

But when I turn the TV off and on again, it no longer works, nor does it
work following the steps above. I would love to know where the problem is.
I am sending you the two images that I get.
El mié, 14 ago 2024, 9:02 p. m., dani fernandez @.***>
escribió:

Hi again, with the last thing I meant that after starting movies via
acestream then elementum loaded, and I exited Kodi, I re-entered and it
worked without problems.
El mié, 14 ago 2024, 8:28 p. m., dani fernandez @.***>
escribió:

Thank you very much for the quick response, I reinstalled the entire
Kodi, I reinstalled the entire TV, it didn't work either, I reinstalled the
router, it didn't work either, I don't know what else to do. I've been
adding these days to upload movies in blue crystal via acestream, I don't
know if it has anything to do with it, I hope you have a solution for my
case.
El mié, 14 ago 2024, 7:43 p. m., Anton Soroko @.***>
escribió:

@peke198822 https://github.com/peke198822 try to reinstall Kodi
(remove and install again, so Kodi will have default configuration).
If it will not help - make debug log file , see
http://elementum.surge.sh/contribute/log/
—
Reply to this email directly, view it on GitHub
https://github.com/elgatito/plugin.video.elementum/issues/1056#issuecomment-2289444198,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AW5NZVO6JUUA3A376XIVLX3ZROJLZAVCNFSM6AAAAABMB6HYKGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEOBZGQ2DIMJZHA
.
You are receiving this because you were mentioned.Message ID:
@.***>

@peke198822 I do not see images, I think you need to attach them here in browser, not via email.
Also, create the debug log file, guide is above.
My guess is that  you installed some android application that uses the same port that kodi uses (port 9090) thus Kodi can't use that port anymore.
But without log file I can't be sure .

Thanks Antón, I'm attaching the .log. I don't know if you can open it,
there are some errors that have to do with json that I mentioned.
El mié, 14 ago 2024, 9:17 p. m., Anton Soroko @.***>
escribió:

@peke198822 https://github.com/peke198822 I do not see images, I think
you need to attach them here in browser, not via email.
Also, create the debug log file, guide is above.
My guess is that you installed some android application that uses the same
port that kodi uses (port 9090) thus Kodi can't use that port anymore.
But without log file I can't be sure .
—
Reply to this email directly, view it on GitHub
https://github.com/elgatito/plugin.video.elementum/issues/1056#issuecomment-2289658352,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AW5NZVKH7ULQPQTJYVEN32DZROUKZAVCNFSM6AAAAABMB6HYKGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEOBZGY2TQMZVGI
.
You are receiving this because you were mentioned.Message ID:
@.***>

I attach the image of the error that appears to me.

I will look for this setting in settings, services, it appears activated.

I attached the .log here.
kodi.log
kodi.old.log

@peke198822No veo las imágenes, creo que debes adjuntarlas aquí en el navegador, no por correo electrónico.
Además, cree el archivo de registro de depuración, la guía se encuentra más arriba.
Supongo que instalaste alguna aplicación de Android que usa el mismo puerto que usa Kodi (puerto 9090), por lo que Kodi ya no puede usar ese puerto. Pero sin el archivo de registro, no puedo estar seguro.

Ummm I'll check this out, but I've always had all the same apps and haven't installed anything new, I'll check this out as you say, thanks.

Hi Anton, regarding the ports coming from the TV, only the ones I attached, then port 9090 I don't see that any other applications use it.

Hi Anton, you are a genius, I found the apk that used this port, I really don't know what it's for but it lets me stop it, disable it no, but well knowing that it comes from here I can now fix it Attached is a photo of the apk, it is called UcService, Greetings and thanks again Antón.

@peke198822 awesome! Thanks for sharing this application name, so now other users can learn about this fix.

Quiero añadir que la apk SddpService tambien utiliza el puerto de kodi no dejando cargar bien la apk.

I want to add that the SddpService apk also uses the kodi port without allowing the apk to load properly.


CI Broken

Oops looks like I broke the deploy.sh part after migrating to Circle 2.0, gotta check this out later cc @vpeschenkov

Fixed, that was painful 😢


Alternative to active-win?

Why shouldn't I use https://www.npmjs.com/package/active-win?

You could use that package. It even has some advantages over this one.
Both libraries can:

get the window title
get the application name
get the application's path
get the application's PID

What active-win can do and @paymoapp/active-window can not:

get the window bounds (x, y, height, width)
on MacOS get the URL loaded in browsers
get the list of all open windows
uses standalone executables/system commands that are called with child_process
can get the window's memory usage

What @paymoapp/active-window can do and active-win can not:

uses native (C++, ObjectiveC++) addons to interact with system APIs so there're no child_process calls, the code can be signed and operating systems won't complain about the signature (Gatekeeper usually complains about electron applications including unsigned third party executables)
can fetch the application's icon as a high resolution PNG
on windows can determine if the application is an UWP application and return the UWP package ID if so
can operate in subscription mode (you don't have to poll for the active window, you will be notified as soon as the window changes)
lower memory footprint and resource usage and faster response times given that you don't have to spin up a new process on every request
has partial support for Wayland on linux


docs: fix resolve gif as -r bug was fixed

Describe your changes
At the time tcping_resolve.gif was generated, -r was bugged and generating one extra request, previous gif had this bug. Since bug was fixed, this is an update for the gif.
Type of change

[x] This change requires a documentation update

Hey @ravsii,
Thanks for remembering this and fixing it. Really appreciate it.


<EMAIL_ADDRESS>

本次提交修复的漏洞信息:
漏洞标题：Google Guava访问控制错误漏洞
<EMAIL_ADDRESS>漏洞编号：CVE-2020-8908
漏洞描述：Google Guava是美国谷歌（Google）公司的一款包括图形库、函数类型、I/O和字符串处理等的Java核心库。
Guava 30.0版本之前存在访问控制错误漏洞，该漏洞源于Guava存在一个临时目录创建漏洞，允许访问机器的攻击者可利用该漏洞潜在地访问由Guava com.google.common.io.Files.createTempDir()创建的临时目录中的数据。攻击者可以利用该漏洞访问特殊目录。
国家漏洞库信息：https://www.cnvd.org.cn/flaw/show/CNVD-2021-31252
影响范围：(∞, 30.0-android)
最小修复版本：30.0-android
缺陷组件引入路径：me.coley:recaf@2.21.12->com.github.Col-E:jphantom<EMAIL_ADDRESS>
另外我运行这个项目时，IDE的安全插件提示还有几个漏洞，我不确定升级是否会有兼容性问题。您有空的话可以查看报告修复下哈。感谢感谢。
相关漏洞详细报告：https://mofeisec.com/jr?p=p384ec

A temp directory creation vulnerability exists in all versions of Guava, allowing an attacker with access to the machine to potentially access data in a temporary directory created by the Guava API com.google.common.io.Files.createTempDir(). By default, on unix-like systems, the created directory is world-readable (readable by an attacker with access to the system). The method in question has been marked @Deprecated in versions 30.0 and later and should not be used. For Android developers, we recommend choosing a temporary directory API provided by Android, such as context.getCacheDir(). For other Java developers, we recommend migrating to the Java 7 API java.nio.file.Files.createTempDirectory() which explicitly configures permissions of 700, or configuring the Java runtime's java.io.tmpdir system property to point to a location whose permissions are appropriately configured.

https://nvd.nist.gov/vuln/detail/CVE-2020-8908

Recaf does not use the vulnerable method java.nio.file.Files.createTempDirectory().
If it did, Recaf already uses a version of Guava that this is patched in: 31. There is no reason to go backwards to 30.
You've replaced the JRE target artifact with the Android artifact. There is no real reason to target it over the JRE flavor.

我不确定升级是否会有兼容性问题。

You can easily make sure the project still compiled with mvn compile. And you can additionally make sure the test cases pass  with mvn test.


Translate local names to Greek

Additionally, add support for single and double quotes in Greek.
For #2683.

Thanks!


new rule: require jsdoc

Create a rule that requires each .ts file to contain at least some jsdoc formatted comments at the top level.
This rule exists in MSE and can be ported over.

fixed in 0.0.5


Updating net-sdk version for internal csproj

Issue describing the changes in this PR
Updating the reference to 4.1.3 in-proc sdk version as 4.1.0 version requires dotnet 6.
resolves #981
Pull request checklist

[ ] My changes do not require documentation changes

[ ] Otherwise: Documentation issue linked to PR

[ ] My changes should not be added to the release notes for the next release

[ ] Otherwise: I've added my notes to release_notes.md

[ ] My changes do not need to be backported to a previous version

[ ] Otherwise: Backport tracked by issue/PR #issue_or_pr

[ ] I have added all required tests (Unit tests, E2E tests)

Additional information
Additional PR information

/check-enforcer evaluate

/check-enforcer override


Create a Rock-Paper-Scissor game using Python.

Hi @dhruvpatel1706, Can I open pull request for this issue? I have implemented the Rock-Paper-Scissors game using Python.
Thanks.

Hi @dhruvpatel1706, Can I open pull request for this issue? I have implemented the Rock-Paper-Scissors game using Python.
Thanks.

Yeah, sure!

Yeah, sure!

I have opened a pull request. Please verify it.
Thanks.

Thanks, @kamal-kaur04 , for opening the pull request! 👏
Star ⭐ this project and tweet about your contributions and also you can join the discord channel of club gamma

Hey, I am new programmer looking to contribute to project. I can create a pyqt5 version of rock paper scissor


Fix/fix tests payout settings

This item belongs to payment request #F78085 on OnlyDust:

from oscarwroche to oscarwroche
35 items included
$1,000 for 2 days of work

This item belongs to payment request #D812A4 on OnlyDust:

from oscarwroche to oscarwroche
35 items included
$1,000 for 2 days of work


Issues with receiving with CoChannel (and possibly CoScope)

I noticed some weird behavior in my projects concerning CoChannels, and I wasn't sure if I was using it wrong or there was something buggy going on.
I have a scoped coroutine running and

The coroutine crashes when self is captured with unowned.
The receiving end slows down as new coroutines are launched using the same channel.

Not sure if I'm explaining the situation correctly, so I have a demo target here.
I also noticed that all the CoScopes are kepts around when the memory graph debugger is run like so:

The objects that own the CoScope instances are deallocated as expected.
The slowing down (or rather the skipping values) corresponds to the number of zombie CoScope instances.
So when there are four CoScope instances lying around, the log prints like so
9122080394873498997
showInt(_:)<PHONE_NUMBER>873498997
4655395568503796474
432049987236061310
1887288328159798314
1946205301628222178
showInt(_:)<PHONE_NUMBER>628222178

(I also noticed other weird behavior with my real project like objects captured by the SwiftCoroutine and not deallocating, but I'm not sure if it's in any way related to the issue demonstrated by the sample project. I can provide further information if you're interested.)
Sorry about the haphazard reporting, but it was hard to determine if they all pointed to the same cause or if they were all separate, so I decided to just lay it out.

@funct7 Hi, it's true that there is a leak in startCoroutine() when you pass CoScope. I'll fix it in next release.
To fix it in current release, please don't pass scope parameter in startCoroutine() or add scope.cancel() in Reactor.deinit().

@funct7 Hi, it's true that there is a leak in startCoroutine() when you pass CoScope. I'll fix it in next release.
To fix it in current release, please don't pass scope parameter in startCoroutine() or add scope.cancel() in Reactor.deinit().

Thank you. Once it's fixed, I'll see if other issues are also resolved.

Thank you. Once it's fixed, I'll see if other issues are also resolved.

@funct7 You can try with this fix - https://github.com/belozierov/SwiftCoroutine/releases/tag/2.1.10

Most of the issues have gone away--also with my real project. Thank you so much!
The coroutine still crashes when self is captured with unowned, but I'm guessing it's because the block of code is not guaranteed to be canceled as soon as self and CoScope assoiciated with it is deallocated.


Ebeam_

Missing opt_in labels for the alignment loopback GCs
The bottom grating coupler does not have the same x coordinate as the others


LEGO: Pull request from lego/hb_839_20170321084125398 to master

LEGO: Pull request from lego/hb_839_20170321084125398 to master with localized lcls

@csigs,
Thanks for your contribution.
To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla.microsoft.com.
It will cover your contributions to all Microsoft-managed open source projects.
Thanks,
Microsoft Pull Request Bot


Added storage class for object stores such as Minio

This PR adds a new SC for successful deployments of object stores such as Minio on top of our storage backend.

Just want to merge another PR first. It's going to be in by the end of the week!


Date Picker Does Not Display Correctly on Mobile Devices

Describe the bug
The date picker component does not render correctly on mobile devices. When selecting a date, the calendar appears out of alignment, leading to a poor user experience.
To Reproduce
Steps to reproduce the behavior:

Click on Patients from sidebar.
Now, click on any patient.
Click on the Date of onset date picker input field in small screens.
See responsiveness error of date picker overflowing.

Expected behavior
The date picker component should render correctly on mobile devices. When selecting a date, the calendar should appear aligned and fully visible within the viewport, providing a seamless user experience.

Before
After

Desktop (please complete the following information):

OS: Windows
Browser: Brave
Version: 1.70.126

I am working on this .

@shauryag2002 we are updating the date picker component in https://github.com/ohcnetwork/care_fe/pull/8504
This issue will be covered by that PR. Thanks for taking the time out to contribute!


Remove sudo: false as this is now Travis default

Tests failing as expected until profile is completed.


[Python] Claim Python3 support in Trove

What:
Add Python 3.4 and 3.5 to the Programming Languages section of the Trove classifiers (e.g. https://pypi.python.org/pypi/gax-google-logging-v2).
Why:
'Cause we now support them.

@tseaver FYI


Fix global settings function

Fixing the global settings function as it was not looking up the values from the db at all, as reported here: https://devdojo.com/question/admin-settings-bug-or-just-dumb-question

Thanks @bobbyiliev. This looks great!


Entry "rock\W?hard(?:\W?bull)?" removed from blacklist

Too many fps. Move to watchlist instead.

!!/approve


Not respect the color setting for label if default input is disabled

Describe the bug
The label overrides the original color if the corresponding input element is disabled, that seems fine. However when toggle back the input element into normal state, the label does not revert back to original color setting. This causing strange behaviour as the original state of the input element decides the label color.
It should either keeping the original color or change the label color according to the state of input element, which I think the original intention is the latter one according to the code.
I can help with a PR if necessary.
Corresponding code:
https://github.com/themesberg/flowbite-svelte/blob/079ca8795814e7c3b482cfb27ddfec5fd23fc683/src/lib/forms/Label.svelte#L20
Reproduction
https://stackblitz.com/edit/sveltejs-kit-template-default-5as3bp?file=src%2Flib%2FMyButton.svelte
Flowbite version and System Info
System:
    OS: macOS 13.4
    CPU: (4) x64 Intel(R) Core(TM) i7-7567U CPU @ 3.50GHz
    Memory: 1.85 GB / 16.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 19.7.0 - /usr/local/bin/node
    Yarn: 1.22.17 - /usr/local/bin/yarn
    npm: 9.5.0 - /usr/local/bin/npm
    Watchman: 2<IP_ADDRESS> - /usr/local/bin/watchman
  Browsers:
    Brave Browser: <IP_ADDRESS>
    Chrome: 114.0.5735.198
    Safari: 16.5
    Safari Technology Preview: 16.4
  npmPackages:
    flowbite-svelte: ^0.38.5 => 0.38.5
    svelte: ^3.55.1 => 3.58.0
    vite: ^4.1.0 => 4.2.1

You can use class:
Disabled

You can use class:
Disabled

Yes it can be modified by using explicit class to set the color, but the point is it created different result with the same code logic in different flow, which makes developer confused

This is what I can think of at the moment:
  import { Button, Input, Label } from 'flowbite-svelte'

  let normal = null
  let disabled = 'disabled'

  function changeState() {
    if (normal === null) {
      normal = 'disabled'
    } else {
      normal = null
    }

    if (disabled === null) {
      disabled = 'disabled'
    } else {
      disabled = null
    }
  }
</script>

Normal
<Input id="normal" name="normal" disabled={normal} />

Disabled
<Input id="default_disabled" name="default_disabled" {disabled} />

<Button color="green" class="my-2 p-3" on:click={changeState}>Change</Button>


[Suggestion] Iron Chest Support

I was wondering if it would be possible to add Iron Chest Support.

Any update on this

What about ae2 or Tom's Simple Storage

https://github.com/TartaricAcid/TouhouLittleMaid/commit/3dc6bded1687d9e61194b5c9a5aee6d3e41ce788


Feature request: add support for detecting generic relation type for PendingHasThroughRelationships

Larastan Version: 2.4.0
--level used: 6

Description
In https://github.com/laravel/framework/pull/45894/ a new PendingHasThroughRelationship was introduced to allow for easier through-relation building.
However, when using these relations in my application, I was faced with the following error:
 ------ -------------------------------------------------------------------------------------------------------
  Line   app\Models\BaseModel.php
 ------ -------------------------------------------------------------------------------------------------------
  x      Parameter #1 $callback of method Illuminate\Database\Eloquent\PendingHasThroughRelationship::has()
         expects (callable(Illuminate\Database\Eloquent\Model):
         Illuminate\Database\Eloquent\Relations\HasMany|Illuminate\Database\Eloquent\Relations\HasOne)|string,
         Closure(App\Models\ThroughModel):
         Illuminate\Database\Eloquent\Relations\HasMany<App\Models\RelatedModel> given.
 ------ -------------------------------------------------------------------------------------------------------

Laravel code where the issue was found
See also the PR mentioned above
class Project extends Model
{
    public function deployments()
    {
        return $this->through($this->environments())
            ->has(fn (Environment $env) => $env->deployments());
    }

    public function environments()
    {
        return $this->hasMany(Environment::class);
    }
}

class Environment extends Model
{
    public function deployments()
    {
        return $this->hasMany(Deployment::class);
    }
}

class Deployment extends Model
{
    //
}

Note that this has been implemented in https://github.com/laravel/framework/pull/51851

I can confirm that https://github.com/laravel/framework/pull/51851 and #1990 combined now yield correct typings for my usecase.
/close


There are no commands defined in the "ide-helper" namespace

php artisan ide-helper:generate
[Symfony\Component\Console\Exception\CommandNotFoundException]
There are no commands defined in the "ide-helper" namespace.

Getting same error.
[InvalidArgumentException]
There are no commands defined in the "ide-helper" namespace.
Using:
"post-update-cmd": [
"Illuminate\Foundation\ComposerScripts::postUpdate",
"php artisan clear-compiled",
"php artisan ide-helper:generate",
"php artisan ide-helper:meta",
"php artisan ide-helper:models --write",
"php artisan optimize"
],

Did you register the service provider?

Hi,
I have the same problem, but I registered service provider in config/app.php by adding Barryvdh\LaravelIdeHelper\IdeHelperServiceProvider::class, into section Providers.
root@marleen:/media/sf_Development/laravel# composer update
Do not run Composer as root/super user! See https://getcomposer.org/root for details
Loading composer repositories with package information
Updating dependencies (including require-dev)
Package operations: 0 installs, 2 updates, 0 removals

Updating laravel/framework (v5.4.25 => v5.4.27) Downloading: 100%
Updating laravel/passport (v2.0.10 => v2.0.11) Downloading: 100%
Writing lock file
Generating autoload files

Illuminate\Foundation\ComposerScripts::postUpdate
php artisan ide-helper:generate

[Symfony\Component\Console\Exception\CommandNotFoundException]
There are no commands defined in the "ide-helper" namespace.
Script php artisan ide-helper:generate handling the post-update-cmd event returned with error code 1
You have new mail in /var/mail/root
root@marleen:/media/sf_Development/laravel# composer update
Do not run Composer as root/super user! See https://getcomposer.org/root for details
Loading composer repositories with package information
Updating dependencies (including require-dev)
Nothing to install or update
Generating autoload files

Illuminate\Foundation\ComposerScripts::postUpdate
php artisan ide-helper:generate

[Symfony\Component\Console\Exception\CommandNotFoundException]
There are no commands defined in the "ide-helper" namespace.
Script php artisan ide-helper:generate handling the post-update-cmd event returned with error code 1
You have new mail in /var/mail/root
root@marleen:/media/sf_Development/laravel#

@barryvdh old support issue, I think we can close it :}

I removed this package, still I got this error on my log on every second and my log file is too big size within few hours. I removed it from composer, app.php appserviceprovider, where still this error coming from?? I could not find it.


[FLINK-16603] Fix quickstart Maven archetype

The quickstart Maven archetype was not working due to a few issues:

the archetype pom.xml not being included for filtering, causing the @project.version@ string not being replaced with the project version.
the archetype pom.xml did not define modelVersion

Verifying
An integration-test phase test has been added to verify that projects generated with the archetype can be compiled.

Thanks for reviewing Igal!
I've fixed the problem you mentioned - src/main/resources wasn't being filtered previously, now it should work.


[MINOR] Fix of flaky TestHoodieClientMultiWriter::testMultiWriterWithAsyncTableServicesWithConflict

Change Logs
Fixed flaky test.
Impact
No
Risk level (write none, low medium or high below)
No
Documentation Update
No need
Contributor's checklist

[x] Read through contributor's guide
[x] Change Logs and Impact were stated clearly
[x] Adequate tests were added if applicable
[ ] CI passed

@hudi-bot run azure


Implement mesh animations.

Implement ZMO which do vertex based animation.
e.g. flags around junon, or a bunch of effects use these too including the movement indicator.

105abde7e5087220bb32b22d3123a7d085b4d98a


Handle sites that have subdirectories

Coverage remained the same at 100.0% when pulling fa84de061599188fb3b1c4c2e1cbd2a0bb447f89 on get-sites into 72e4320dab728ce68bd1516b319a1d7e41afb8e0 on master.

Coverage remained the same at 100.0% when pulling e2514a8680e250c496c3d9b094cfb20cc4034639 on get-sites into 72e4320dab728ce68bd1516b319a1d7e41afb8e0 on master.


Delegated harvesting tests are missing

need to check whether delegated harvesting is implemented and write e2e tests for it

so the delegated harvesting is performed via account link which now has basic implementation and test ready. need to polish it up a bit but most of the work is done

PR created


What's wrong with this audio playlist

Based on a old Plyr audio playlist on codepen.
https://codepen.io/jmisavage/pen/WwVQoB
I just upgraded to the last Plyr CDN and the new version is not working.
https://codepen.io/onigetoc/pen/OxmROo

Hi @ducnv999
it do nothing

I'm closing this, since this is a support question for what's now an outdated version. The error you got looks like one that has been fixed. If you still need help, please upgrade to the latest version, simplify the code example to make it easier for us to understand it and go to our slack channel.


Update bootstrap package to install Salt 2015.5.5

At the moment salt-minion-2014.7.1-1.el7.noarch already causes problems on CentOS 5 for common.tmux state which would otherwise execute cleanly on salt-minion-2015.5.5-1.el7.noarch.
----------
          ID: deploy_tmux_resurrect_plugin
    Function: archive.extracted
        Name: /opt
      Result: False
     Comment:
     Started: 16:53:33.675349
    Duration: 9.463 ms
     Changes:
              ----------
              pid:
                  6441
              retcode:
                  2
              stderr:
                  tar: This does not look like a tar archive
                  tar: --: Not found in archive
                  tar: -s: Not found in archive
                  tar: -t: Not found in archive
                  tar: -r: Not found in archive
                  tar: -i: Not found in archive
                  tar: -p: Not found in archive
                  tar: --: Not found in archive
                  tar: -c: Not found in archive
                  tar: -o: Not found in archive
                  tar: -m: Not found in archive
                  tar: -p: Not found in archive
                  tar: -o: Not found in archive
                  tar: -n: Not found in archive
                  tar: -e: Not found in archive
                  tar: -n: Not found in archive
                  tar: -t: Not found in archive
                  tar: -s: Not found in archive
                  tar: -=: Not found in archive
                  tar: -1: Not found in archive
                  tar: -f: Not found in archive
                  tar: '/var/cache/salt/minion/files/base/_opt_tmux-resurrect.tar': Not found in archive
                  tar: Exiting with failure status due to previous errors
              stdout:

Commit e35511242b674c02b0d54656293ae0f679e60afd merges updates of Salt packages to version 2015.5.10 on both rhel5 and rhel7.


Should default branch be coq-8.18 or main?

The default branch is currently coq-8.16.  this can be changed by admins at https://github.com/metacoq/metacoq/settings

I just synced all branches, I think we can move the default branch to be coq-8.18 now. (Not main, because we don't want to give the impression that contributors are expected to compile Coq dev, and actually most contributors don't)

We now have a choice of either 8.18 or 8.19. I guess the later, which will be available everywhere soonish.

We'll switch to 8.19 now that it's released and a platform is comming for it.

I still see coq-8.17 as default


discrete index action is not converted to one-hot vector in AIRL

This is not a problem for AIRL, action is not input for rewards function.


(SP: 0.5) Rewrite spec for CreateCourse.tsx page

Rewrite spec CreateCourse.spec.jsx for page CreateCourse.tsx : CreateCourse.tsx

Described in #2323


Improve docs/gallery card component

Why are these changes needed?
The gallery features incredibly valuable community contributions, however it doesn't seem like the component in the current docs/gallery card is optimized:

So updated the component to make it look better and more engaging:

Let me know if any changes can be done to this to make it better.
I have continued the design approach by using the antd for typography.
Related issue number

Checks

[ ] I've included any doc changes needed for https://microsoft.github.io/autogen/. See https://microsoft.github.io/autogen/docs/Contribute#documentation to build and test documentation locally.
[ ] I've added tests (if relevant) corresponding to the changes introduced in this PR.
[x] I've made sure all auto checks have passed.

@namanbarkiya please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.
@microsoft-github-policy-service agree [company="{your company}"]

Options:

(default - no company specified) I have sole ownership of intellectual property rights to my Submissions and I am not making Submissions in the course of work for my employer.

@microsoft-github-policy-service agree

(when company given) I am making Submissions in the course of work for my employer (or my employer has intellectual property rights in my Submissions by contract or applicable law). I have permission from my employer to make Submissions and enter into this Agreement on behalf of my employer. By signing below, the defined term “You” includes me and my employer.

@microsoft-github-policy-service agree company="Microsoft"

Contributor License Agreement
Contribution License Agreement
This Contribution License Agreement (“Agreement”) is agreed to by the party signing below (“You”), and conveys certain license rights to Microsoft Corporation and its affiliates (“Microsoft”) for Your contributions to Microsoft open source projects. This Agreement is effective as of the latest signature date below.

Definitions.
“Code” means the computer software code, whether in human-readable or machine-executable form,
that is delivered by You to Microsoft under this Agreement.
“Project” means any of the projects owned or managed by Microsoft and offered under a license
approved by the Open Source Initiative (www.opensource.org).
“Submit” is the act of uploading, submitting, transmitting, or distributing code or other content to any
Project, including but not limited to communication on electronic mailing lists, source code control
systems, and issue tracking systems that are managed by, or on behalf of, the Project for the purpose of
discussing and improving that Project, but excluding communication that is conspicuously marked or
otherwise designated in writing by You as “Not a Submission.”
“Submission” means the Code and any other copyrightable material Submitted by You, including any
associated comments and documentation.
Your Submission. You must agree to the terms of this Agreement before making a Submission to any
Project. This Agreement covers any and all Submissions that You, now or in the future (except as
described in Section 4 below), Submit to any Project.
Originality of Work. You represent that each of Your Submissions is entirely Your original work.
Should You wish to Submit materials that are not Your original work, You may Submit them separately
to the Project if You (a) retain all copyright and license information that was in the materials as You
received them, (b) in the description accompanying Your Submission, include the phrase “Submission
containing materials of a third party:” followed by the names of the third party and any licenses or other
restrictions of which You are aware, and (c) follow any other instructions in the Project’s written
guidelines concerning Submissions.
Your Employer. References to “employer” in this Agreement include Your employer or anyone else
for whom You are acting in making Your Submission, e.g. as a contractor, vendor, or agent. If Your
Submission is made in the course of Your work for an employer or Your employer has intellectual
property rights in Your Submission by contract or applicable law, You must secure permission from Your
employer to make the Submission before signing this Agreement. In that case, the term “You” in this
Agreement will refer to You and the employer collectively. If You change employers in the future and
desire to Submit additional Submissions for the new employer, then You agree to sign a new Agreement
and secure permission from the new employer before Submitting those Submissions.
Licenses.

Copyright License. You grant Microsoft, and those who receive the Submission directly or
indirectly from Microsoft, a perpetual, worldwide, non-exclusive, royalty-free, irrevocable license in the
Submission to reproduce, prepare derivative works of, publicly display, publicly perform, and distribute
the Submission and such derivative works, and to sublicense any or all of the foregoing rights to third
parties.
Patent License. You grant Microsoft, and those who receive the Submission directly or
indirectly from Microsoft, a perpetual, worldwide, non-exclusive, royalty-free, irrevocable license under
Your patent claims that are necessarily infringed by the Submission or the combination of the
Submission with the Project to which it was Submitted to make, have made, use, offer to sell, sell and
import or otherwise dispose of the Submission alone or with the Project.
Other Rights Reserved. Each party reserves all rights not expressly granted in this Agreement.
No additional licenses or rights whatsoever (including, without limitation, any implied licenses) are
granted by implication, exhaustion, estoppel or otherwise.

Representations and Warranties. You represent that You are legally entitled to grant the above
licenses. You represent that each of Your Submissions is entirely Your original work (except as You may
have disclosed under Section 3). You represent that You have secured permission from Your employer to
make the Submission in cases where Your Submission is made in the course of Your work for Your
employer or Your employer has intellectual property rights in Your Submission by contract or applicable
law. If You are signing this Agreement on behalf of Your employer, You represent and warrant that You
have the necessary authority to bind the listed employer to the obligations contained in this Agreement.
You are not expected to provide support for Your Submission, unless You choose to do so. UNLESS
REQUIRED BY APPLICABLE LAW OR AGREED TO IN WRITING, AND EXCEPT FOR THE WARRANTIES
EXPRESSLY STATED IN SECTIONS 3, 4, AND 6, THE SUBMISSION PROVIDED UNDER THIS AGREEMENT IS
PROVIDED WITHOUT WARRANTY OF ANY KIND, INCLUDING, BUT NOT LIMITED TO, ANY WARRANTY OF
NONINFRINGEMENT, MERCHANTABILITY, OR FITNESS FOR A PARTICULAR PURPOSE.
Notice to Microsoft. You agree to notify Microsoft in writing of any facts or circumstances of which
You later become aware that would make Your representations in this Agreement inaccurate in any
respect.
Information about Submissions. You agree that contributions to Projects and information about
contributions may be maintained indefinitely and disclosed publicly, including Your name and other
information that You submit with Your Submission.
Governing Law/Jurisdiction. This Agreement is governed by the laws of the State of Washington, and
the parties consent to exclusive jurisdiction and venue in the federal courts sitting in King County,
Washington, unless no federal subject matter jurisdiction exists, in which case the parties consent to
exclusive jurisdiction and venue in the Superior Court of King County, Washington. The parties waive all
defenses of lack of personal jurisdiction and forum non-conveniens.
Entire Agreement/Assignment. This Agreement is the entire agreement between the parties, and
supersedes any and all prior agreements, understandings or communications, written or oral, between
the parties relating to the subject matter hereof. This Agreement may be assigned by Microsoft.

@microsoft-github-policy-service agree

PR looks good, but I think the card looks better with a border on light theme

Previously border was getting merged with the image, just adding the small padding to image and making the border true gave this output:

Previously border was getting merged with the image, just adding a small padding to image and making the border property true gave this output:

Should I push this one? @cheng-tan

yep, I think this is better for light theme

Done adding the changes @cheng-tan

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

Comparison is base (6cf5bb0) 33.19% compared to head (e7d5a0b) 33.05%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1445      +/-   ##
==========================================
- Coverage   33.19%   33.05%   -0.15%
==========================================
  Files          41       41
  Lines        4950     4950
  Branches     1135     1135
==========================================
- Hits         1643     1636       -7
- Misses       3177     3183       +6
- Partials      130      131       +1

Flag
Coverage Δ

unittests
33.05% <ø> (-0.11%)
:arrow_down:

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.

Thanks all!

Thanks for the merge!
@cheng-tan I am also thinking about adding a dark mode to cards, right now they are white in dark mode. Any suggestions?

@jackgerrits sorry to spam unassigned


Bump Golang version v1.23 and Ginkgo

Why we need this PR

Use newer Golang, Ginkgo, Medijk8s-common versions for new features, bug fixes, and regular improvements between versions.
Changes made

Bump Golang version (v1.21 -> v1.23) and Ginkgo (v2.15.0 -> v2.22.0)

Which issue(s) this PR fixes

Test plan

/test 4.18-openshift-e2e

/test 4.18-openshift-e2e

/lgtm


Overlay cannot be removed from dom after closing the dialog

with vaadin-dialog 2.0.0-beta3, when the dialog has lumo theme on, the vaadin-dialog-overlay will not be removed from the dom when close the dialog. and if i remove the theme, everything works.
related discussion on slack https://vaadin.slack.com/archives/C3TGRP4HY/p1522156886000459

I reproduced.
This issues is caused by CSS animation delay.
Without animation it works excellent.
Simple animation example to reproduce:
           :host([opening]) {                 animation:  overlay-enter 0.5s;             }             @keyframes overlay-enter {                 0% {                     opacity: 0;                 }             }


Store currency in DB

Add a currency column to the database and store it for each bet.

Cool, now just make sure the currency value is actually stored in the bet object for every site, because I'm relatively sure I didn't store them on most sites. Also show the currency in the bet history form.


Makefile: parallel build (make -j) fails on fresh checkout

The build fails if you run:
git clone https://github.com/SnabbCo/snabbswitch
cd snabbswitch
make -j

Looks like a missing dependency where it will try to build LuaJIT before downloading it by initializing the submodule.

Fixed by #493.


Alternate colors based on stride

Alternate line colours for memory views as default, but with the ability to instead specify a stride for colouring.
Hotkeys for going to next 'start of block' would be nice.

Dupe of https://github.com/emoon/ProDBG/issues/30


Respect JsonProperty#required when mapping inputs

Since Jackson meta-data is normally respected, JsonProperty#required should apply as well, and result in a non-null input field.

I disagree with this. Depending on what the the JsonProperty annotation is for, whoever uses this library may want to make sure that the JSON to deserialize contains that property, even if that property is null. Take this JSON for instance:
{
  "myValue": null
}

And this JsonCreator:
@JsonCreator
public MyValueClass(@JsonProperty(value = "myValue", required = true)Integer value){
  ...
}

It would not affect my code or anything, but I think just adding an extra annotation isn't too big a deal in this case.


v1.10.1

Fixes #3126
Somehow I managed to build v1.10.0 wrong for the npm release (I had locally linked in an unstable version of three.js). I've  now updated our release process doc to avoid this in the future. So, v1.10.1 is actually identical to v1.10.0 in terms of what you'd get if you pull  these release tags from github and built them. But the npm versions will be different, and v1.10.1 will be  correct.

Also, I suspect something is going wrong with BrowserStack. We had no trouble with  this test until they day before yesterday, at which point it became a high-probability flake. Then yesterday it was back to passing 100%, and today it's back to a high  rate of failure. No actual changes  to the  build have happened during this time and I can't reproduce it locally, so I think the fault is in our CI.


Add check to confirm release

Following a number of accidental releases that I have seen and executed in my short stay as an intern on the Reason team, I would like to propose a pre-release check mechanism. Idea: you are much less likely to publish to npm accidentally if you are required to type a pseudo-random string to confirm.

lol, I only use https://github.com/sindresorhus/np nowadays

Then why do we have make release?

I meant for my own repos. Try it!

I don't publish things to npm 🙃

Derp, just kidding. Apparently I have three packages.

@yunxing Added a check!

Unclear of the preferred order of these checks.

@tekknolagi Looks good then. Thanks! Let's merge this now since CI won't test this anyway.


Add ws.WsServer function for stopping a specific connection

Useful for testing.

Nice one, thought about adding this one before as well.
Could you maybe add a small test for this one?


Add new accordion component

TBD

This will not be included within WFP UI, as it's a pattern created specifically for WFP.org.


docs: Add limitation for rendering a RenderFragment

Follow-Up to #1282

Isnt this a bug, that is, the behavior described here? Surly triggering an event handler causes the a render?

It isn't at the moment - still it would be a limitation.
Even such a test does not pass:
[Fact]
public void ThatIsMyCoolTest()
{
	var output = string.Empty;
	var cut = Render(@<button @onclick='@(() => output = "Success")'>@output</button>);

	cut.Find("button").Click();

	cut.Find("button").TextContent.ShouldBe("Success");
}

But yeah - let's gather some examples and open a bug

Closes #1289


TUS is required

Recent changes in Vimeo's API caused the script to stop working. TUS is required.

i got the same problem, someone found a solution?

https://github.com/paulbaumann/vimeo-upload


Update design and wording

General refactoring and update on wording and design according to a meeting today from @Ly0n and myself.
Changes:

Updated links
Updated about section/wording
Updated How it works

Updated section looks like this now:

Since hackmd only support the diff view (https://hackmd.io/D-Uk_kDERtimwfXpqY_RZg) i post the the my proposal for the about sectin here. I tried to point out the positive effects and keep it in shorter easier to read junks.

About OpenClimate.fund
Open source technology powers thousands of solutions in the climate and sustainability space. From tracking forest health to designing eco-friendly products, these tools help build a more sustainable future. The best part about open source is that everyone can see how they work and contribute to improving them.
Many brilliant developers create these tools in their spare time. It's time to support their work and help these projects grow. That's why we're building a new way to fund open-source climate and sustainability projects.

OpenClimate.fund connects climate-focused open-source projects with funding. It’s simple: you help us discover great projects, and we help fund their development.

Join Our First Funding Round
We’re starting with $10,000 to support 100 projects with $100 each. Here’s how you can help:

Add an open-source sustainability project to OpenSustain.tech.
Pick another listed project to receive funding (as long as it’s not your own).

Once we review your contribution, we’ll send $100 to your chosen project through their preferred funding platform.
We’re creating a new model for funding open-source climate and sustainability projects, and we’d love for you to be part of it.
Want to support our next funding round? Reach out to Michael or Tobias on LinkedIn.
Read more on our blog post.