chore: pylint rules enabled in models

SUMMARY
Files tags.py and helpers.py can have reenabled pylint rules:

unused_import,
unused-argument - changed unused arguments to private and removed disabled rule,
removed redundant rules

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TEST PLAN

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

Codecov Report

Merging #11037 into master will increase coverage by 1.36%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master   #11037      +/-   ##
==========================================
+ Coverage   59.95%   61.32%   +1.36%
==========================================
  Files         782      816      +34
  Lines       37252    38381    +1129
  Branches     3350     3606     +256
==========================================
+ Hits        22335    23536    +1201
+ Misses      14738    14659      -79
- Partials      179      186       +7

Flag
Coverage Δ

#cypress
?

#javascript
61.68% <ø> (?)

#python
61.11% <0.00%> (-0.29%)
:arrow_down:

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

Impacted Files
Coverage Δ

superset/models/helpers.py
87.27% <ø> (ø)

superset/models/tags.py
79.31% <0.00%> (ø)

superset-frontend/src/views/App.tsx
0.00% <0.00%> (-100.00%)
:arrow_down:

superset-frontend/src/SqlLab/App.jsx
0.00% <0.00%> (-100.00%)
:arrow_down:

superset-frontend/src/views/menu.tsx
0.00% <0.00%> (-100.00%)
:arrow_down:

superset-frontend/src/explore/App.jsx
0.00% <0.00%> (-100.00%)
:arrow_down:

superset-frontend/src/views/index.tsx
0.00% <0.00%> (-100.00%)
:arrow_down:

superset-frontend/src/SqlLab/index.tsx
0.00% <0.00%> (-100.00%)
:arrow_down:

superset-frontend/src/dashboard/App.jsx
0.00% <0.00%> (-100.00%)
:arrow_down:

superset-frontend/src/explore/index.jsx
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 338 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6181994...9267180. Read the comment docs.


Handle button logic depending on keyboard layout

When working with other keyboard layouts the chars for playing the sound are not always in a row which makes it a bit hard to play. E.g. the german/austrian layout called QWERTZ has swapped the y and z keys. See here for reference.
It could be handled automatically or at least with manually configuring the buttons used to play.
PS:
This game has some beautiful art, well picked sounds and the movement feels very smooth. Congrats!

Ohh of course I missed this, what do you think about this: at the time the tutorial presents the flute, it will only explain the pitches showing 3 blank keys in a row.
When the player test the flute it will map the keys to the action. This way before the game starts the player will automatically set his/her own bindings

This sounds interesting. It should be clear to the player that this will be some kind of configuration but all in all I like the concept. :+1:

There is a method to check the kb layout.
API docs: http://docs.godotengine.org/en/3.0/classes/class_os.html#class-os-get-latin-keyboard-variant
Note: The QWERTZ layout was not mentioned in the docs for some reason but it exists in the API.
https://github.com/godotengine/godot/blob/7a142780f05d56aa751690c8716bd8aab429b140/core/bind/core_bind.cpp#L438

Yeah it could be configured automatically but you'll have to write some char-button-assignment-logic depending on the kb layout.

Note: The QWERTZ layout was not mentioned in the docs for some reason but it exists in the API.
https://github.com/godotengine/godot/blob/7a142780f05d56aa751690c8716bd8aab429b140/core/bind/core_bind.cpp#L438

Thanks for pointing this out! Would you mind creating an issue in the docs repo?

Gotta get in touch with the guys from Godot Engine, I think the logic will be the same as they do to configure custom actions in the project.

Well, I implemented this new approach at #10 tell me what do you think.
The controls got even snappy by using _input instead of _process. Hope this gives a better experience

I tested it a bit with kind of fuzzing and I totally broke your game with:

Pressing SKIP and NEXT until the tutorial is over and the player is not able to play anything
Press the arrow keys and space bar until no other key inputs (like z, x, c) are accepted
Press some special keys like ä, ö, ü or STRG + something to get some fancy UI:

Still love it though :)

omg lol
hmm I think there's a way to filter modifiers, like ctrl or shift but...man I'm very inclined to ostrich this. But I'll surely give a try. Still laughing tho lol


fix(handler): convert string to bool by compare

Hello! I found bug which always turn on scan on push feature.
bool(...) convert to True every non-empty string.
bool(get_env_var(name="REPO_SCAN_ON_PUSH", optional=True))

Here is an example.
assert bool("false") == True

Also I change return value in get_env_var from None to "". None and "" have the same behavior in if-statement.

@anapsix can you check PR with fix bug?

Thank you for your contribution, @a1k0u.
In case you have TF docs run failing, the extra content key needs to be removed on line 16 of .terraform-docs.yml 😅

@a1k0u, would you be able to repush you commit as signed commit

I've seen that you have already fixed "TF docs run" in #4.
I pushed commit as signed. I suppose that this PR can be merged.
@anapsix

merged and tagged as v0.1.1


Add Slime Block

I needed the slime blocks so I implemented them, it's client side everything that is slowing down and jumping
Here is a video to show that it works client side: click here
(it's on df of course)

Closing in favor of #445


⚠ Bofrost.es has degraded performance

In b8cad35, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1604 ms

Resolved: Bofrost.es performance has improved in 75e6306 after 20 minutes.


fix tasklist structure

When calling fb tasks form c code it is possible to create loops, which cause the server to crash.
To avoid this, execution should be triggered by the tasklist association.
In places where this is not possible (due to limitations of the concept) it is imperative to check that no loops exist.
Timed function blocks in vdivde3696 library are broken because of this as they cannot get the cycle time from the UrTask.

Closed by merging #60


Add Netflix / Scumblr

Thanks for pointing it out! Merged with #58.

Closing the comment after having merged it in #58. Thanks again!


undefined local variable or method 'title' when running example

This project looks great, but when setting up the header example in the docs, I get
undefined local variable or method title' for #<#Class:0x00007f834d7eb620:0x00007f834d7c9e08>`
I saw a few other closed issues relating to this, but it seems patches were issued that fixed them. Could this relate to me trying to use this with Rails 5? Any help is appreciated. Again, this looks like a fantastic gem.
I realize you'll likely need more than what I've provided here to help. Let me know and I can help and post additional info.

Looks like I was mistyping the class name in MountainView::Presenter as Mountainview::Presenter. I was tired. Working great now.


[8.x] Refactoring code that responsible for call (define method)

In AuthAccessGateTest.php, the code that calls the (define) a method in Gate.php repeated four times in the file so I create a new private function called (getGateDefine) take one parameter (Gate Object), and replace the code with calling the new function getGateDefine.
it'll help in case of change the (define) method in the future like add new parameters, instead of going to every line call this function in the (AuthAccessGateTest.php) file just we can change the method in the new function that calls the define method in the Gate.php file.
Also, it'll be easier to understand.

This seems unnecessary, especially since this is test code.

This seems unnecessary, especially since this is test code.

This seems unnecessary, especially since this is test code.

The same code repeated four times in AuthAccessGateTest.php file, In my opinion, it's not a good way.
let's Imagine that if the "define" function will change In the future, we should go to every line of code in AuthAccessGateTest.php instead of doing that we can just go to the new function and change the code. I know that's test code but even if it test code should be more readable and do the best way of coding.

This seems unnecessary, especially since this is test code.

The same code repeated four times in AuthAccessGateTest.php file, In my opinion, it's not a good way.
let's Imagine that if the "define" function will change In the future, we should go to every line of code in AuthAccessGateTest.php instead of doing that we can just go to the new function and change the code. I know that's test code but even if it test code should be more readable and do the best way of coding.

I agree with @AbdlrahmanSaberAbdo it is not good practice to repeat the same lines over and over, it makes the code much more complex to read, but i also agree that it is not necessary as it is only a test code, but this is not a reason why it should look clean 😄
Happy coding for y'all


Test force-closing CFD

Resolves #564

@luckysori spotted that on recent master, the test still passes without faking the clock (it didn't beforehand). I'll investigate this closer before resubmitting an update to the review. converting to draft for now.

bors r+


Connection refused during graceful shutdown and startup

(springboot http service, registered with zookeeper) The upstream service gracefully shutdown, or when it is just started, there is a "Connection refused" in the soul-bootstrap.
Suggestion: When the upstream service gracefully shutdown, first go offline from the registration center, wait 1-2 seconds before rejecting new requests, and finally stop the service until the existing requests are processed or timeout; when starting, delay register until the service is started.

can you fix it?

I think that the code in persistInterface can be change to async and delay n seconds. But I don't know how to judge that the service has been startup completed, to judge when to execute. I haven't figured out how to change the graceful shutdown.

In soul-register-common, set the started flag through ApplicationRunner, async persistInterface judges whether it can be register according to the flag.

Registration has been decoupled by queue, no longer need to be async

The problem of connection refused during shutdown was solved by adding a shutdown hook to close zkClient and delaying other hooks for n seconds.


build: :arrow_up: Upgrade graphql

Using https://github.com/zino-hofmann/graphql-flutter/pull/1463 for now.

https://pub.dev/packages/graphql/versions/5.2.0-beta.10/ is available


[Type checker] Recurse into statements when cleaning type variables.

Resolves two crashes, one of which (28470) is affecting the buildbots
but not my local builds.

Abusing my powers to help get the CI cleaned up.


Fix: Add mittal-parth's Portfolio

Closes #227

@all-contributors please add @mittal-parth for code


[BUG] yurthub invalid arguments error when using yurtadm join

What happened:
Yurthub fails to start when using yurtadm join  to join a new node into cluster.
What you expected to happen:
yurtadm join node succesfully
How to reproduce it (as minimally and precisely as possible):
Use yurtadm join to join a new node, it stuck in the postcheck wait hub agent ready phase.
Anything else we need to know?:
The log of exited yurthub container:
Error: invalid argument "" for "--enable-dummy-if" flag: strconv.ParseBool: parsing "": invalid syntax.
After checking the generated /etc/kubernetes/manifests/yurthub.yaml, we will see this
command:
    - yurthub
    - --v=2
    - --server-addr=https://<IP_ADDRESS>:6443
    - --node-name=$(NODE_NAME)
    - --join-token=jfsnze.c05kn77m7y1ror9h
    - --working-mode=cloud
    - --enable-dummy-if=
    - --enable-node-pool=

It seems that --enable-dummy-if is assigned with an invalid null value.
Environment:

OpenYurt version: 0.7.0
Kubernetes version (use kubectl version): v1.22.12
OS (e.g: cat /etc/os-release): CentOS Linux release 7.9.2009
Kernel (e.g. uname -a): 3.10.0-1160.71.1.el7.x86_64
Install tools: kubeadm & yurtadm
Others:

others
/kind bug

The reason may be obvious. We should use --enable-dummy-if=true or --enable-dummy-if instead of --enable-dummy-if=, same for --enable-node-pool=.

It should be a bug that yurtadm join cannot correctly handle arguments of bool.

Agree +1.  This file was generated here. And the passed ctx misses these options.  Also the yurtadm that I used are downloaded from release v0.7.0. This seems to be fixed in the PR. Use the latest yurtadm will be fine.

The reason may be obvious. We should use --enable-dummy-if=true or --enable-dummy-if instead of --enable-dummy-if=, same for --enable-node-pool=.

@luc99hen Thank you for raising issue. would you be able to post a pull request to fix this problem?

@rambohe-ch It seems has been solved. Maybe we need backport #863 to v0.7.0 as @luc99hen mentioned.

@luc99hen @Congrool I have backported #863 to branch release-v0.7.


CTRL+C does not kill underlying nodejs server

I am having a simple app created with https://github.com/facebook/create-react-app that I am using with this plugin. To start it I am running gradle yarn_run_start which starts a nodejs dev server with hot module reload etc. That all works fine, but when I exit out of it via CTRL+C and then later start the server again it reproducibly complains that the port is still open, i.e. the server was not killed through CTRL+C - I suppose SIGINT is not propagated correctly? Is there a way to make sure this happens?

Any ideas? i can work on a solution but i dont know where to start looking.

May just be a generic gradle issue with no good solution (other than maybe using --no-daemon flag)
https://discuss.gradle.org/t/exec-task-and-killing-processes/6189

Indeed.. adding --no-daemon solves the issue. It seems that the node server thread is managed by the gradle daemon.. and stopping the task is not enough to kill it

I've got a similar issue in Java - shutdownhook isn't called
https://stackoverflow.com/questions/64195246/how-to-send-ctrlc-sigint-signal-to-gradle-javaexec-task-type-forked-jvm


Scoreboards enabled after release are not used by the library if local scoreboards exist

I think this check keeps the game locked into local scoreboards even once they eventually exist (for the catalog game):
https://github.com/GammaGames/pdScoreboards/blob/83401d38f65bc8b9070f9d58fb3fd992006dd5ec/pdScoreboards.lua#L172-L176

🤔 Thank you for reporting this, I'll take a look when I get the chance! I remember some discussion around tweaking how the official scoreboards worked a bit ago so it might be related to that.

The issue here seems to be specifically with that check, and not with the official APIs, in the sense that:
scoreboards is read from the datastore scoreboards_file so if at any point in time the initialize function failed the official getScoreboards call for whatever reason and _init_scoreboards is called, the scoreboards_file gets created so that component is ~= nil, and since _init_scoreboards always sets scoreboards.not_authorized as true and _saves it that second component, if the scoreboards_file exists, is also always true.
This means that if _init_scoreboards is ever called then on any following initialize it will always be called, and from that point local scoreboards are active regardless.

Good catch!! I'll try to get a fix in within the next few days

I opted to remove the attempt at caching authorization: https://github.com/GammaGames/pdScoreboards/pull/2

That's pretty much what I did in my copy to bypass the issue. 👍

I realize this is an old thread, but since it's still open despite the fix and my inquiry is related I'll revive it. If these are better suited as separate issues let me know.

I think a lot of the value in having this convenient wrapper would be specifically to enable seamless migration to global scoreboards if/when they become available. That would require the locally cached high scores to get communicated to Panic's scoreboard servers. Does this happen magically already because the format used by this lib matches the scoreboard APIs own offline caching format? (does it?) Or could/should something be added to the API to report at least the highest local score to the global boards when they successfully load the first time? Actually — does this library interfere with that offline caching of the official APIs?

For some games there could be value in exposing both local and global leaderboards. As far as I can tell, the official scoreboard APIs don't provide any way to fetch all of a user's own high scores — just their highest on any given board (unless they happen to hold multiple top ten scores). Would you consider extending this library so as to provide local high score functionality in addition to the global functionality (if available)? I suppose you'd need an optional flag to getScores (or a getLocalScores function), and probably a way to check whether global high scores are available or not.

Thanks!


Added in "How to Use Docs" page.

I added in a "How to Use Docs" page and moved a subsection that covers this in the Executive Summary to this page and then expanded on that in the new page.
🎉 Issues resolved:

closes #31

@IFTGFTC (Thomas) - I would appreciate your feedback on this page.


LogSpanHandler uses JUL logging

Describe the Bug
We have applications which currently are not sending traces to an endpoint (by design - right now we just want traceIds in our logs for correlating requests). When no reporters are configured, the LogSpanHandler is attached by default. This handler uses JUL logging at an INFO level to log the span. The default JRE logging configuration means that these messages are printed to stdout.
This is problematic - unless we explicitly now include a logging.properties file and ensure it's loaded in our code, we get logs from this handler for every request which do not honour the log format of the rest of our application logging.
Steps to Reproduce
Create a basic Micronaut application as per https://guides.micronaut.io/latest/micronaut-microservices-distributed-tracing-zipkin-gradle-kotlin.html, but remove the http configuration
Expected Behaviour
There are two possibilities here:

Use a logging facade so that log messages from the library honour my own logging configuration
Log at DEBUG level, so traces are not logged by default

FYI I realised that I can set the sampler probability to zero (`tracing.zipkin.sampler.probability' in Micronaut terms) to get the behaviour that I want, but the original point still stands I think.


Adds destination status for Jaeger

Draft PR

@blumamir I'll work on these comments and make a commit soon :)

@blumamir please have a look


DNM: test trigger ci on 7.4

What problem does this PR solve?

Issue Number: close #xxx
What is changed and how it works?
Check List
Tests

Unit test
Integration test
Manual test (add detailed scripts or steps below)
No code

Questions
Will it cause performance regression or break compatibility?
Do you need to update user documentation, design documentation or monitoring documentation?
Release note
Please refer to [Release Notes Language Style Guide](https://pingcap.github.io/tidb-dev-guide/contribute-to-tidb/release-notes-style-guide.html) to write a quality release note.

If you don't think this PR needs a release note then fill it with `None`.

Wait until the tikv & tiflash binary builds are completed.

/retest


Change iOS deployment target version

Followed the minimum supported iOS version of RxSwift 6.0.

Okay! This has been released as 4.3.0! Thanks again.

Okay! This has been released as 4.3.0! Thanks again.


Slovenian locale file name

Language code for Slovene is sl, not si (Sinhala).
Thanks! :-)

fixed, released 1.0.14

The si in the locale file needs to be changed to sl as well.
si:  # <===
  cookies_eu:
    ...

bah, thx

:+1:

Another rerelease? :)

Thanks :)

yes :+1:
1.0.15


Add an endpoint to calculate Acid-Test Ratio

Note

The acid-test ratio, commonly known as the quick ratio, uses a firm's balance sheet data as an indicator of whether it has sufficient short-term assets to cover its short-term liabilities.

Add an endpoint to calculate Free Cash Flow to Firm.
Create a function in ./helpers/functions.py file for the calculation.
Map it with an endpoint GET /acid-test-ratio in the main.py file with properly.

@nikhil25803 Please assign this to me

Looks good, add the endpoint and make a PR.


Waterline calling exec() callbacks synchronously leading to maximum stack size exceeded in async

Hi there!
Original case: Read 40k items from a file and save to db
Problem: RangeError: Maximum call stack size exceeded, Process finished with exit code 8
Test case: Read 100k items from db, individually (!, to mimic the original case)
test: function() {
  var huge = _.range(1, 100000);
  async.eachLimit( // parameters: itemArray, maxParallel, iteratorFunction(item, callback), callbackFinal
    huge,
    4,
    function(item, cb) {
      if (item % 100 == 0) console.log(item);
      // return setTimeout(cb, _.random(5), null, item); // this will not fail
      return Product.findOneById(item).exec(cb); // this fails
    },
    function(e, r) {
      return sails.log.info('done');
    }
  );
}

This function does not fail if return setTimeout(...) is used but will fail if return Product.findOneById(...) is used. I thought it was probably because Waterline calls cb in exec synchronously (read: recursively). However, replacing the line with
    return Product.findOneById(item).exec(function(e, r) {
      return setImmediate(cb, e, r);
    });

will get slower and slower and finally fail, too (after 19,200 iterations on my computer). I don't see how Waterline could get used by async to loop over that amount of items...

I got this and not sure why.
node_modules/sails-mysql/node_modules/mysql/lib/protocol/Parser.js:82
throw err;
^
RangeError: Maximum call stack size exceeded
ps: I have wrapped every async calls into setImmediate as refered from caolan/async#696
But still crash
and it only happened in my linux server not from windows.

more found : node-mysql version is different
node-mysql in windows sails-mysql version is 2.3.0, which is ok.
in linux sails-mysql version is 2.5.4, which crashed.

please help this - my stack
Trace: stack overflowed!!!!!
at Parser.write (/var/lib/jenkins/jobs/sage-master/workspace/restserver/node_modules/sails-mysql/node_modules/mysql/lib/protocol/Parser.js:82:17)
at Protocol.write (/var/lib/jenkins/jobs/sage-master/workspace/restserver/node_modules/sails-mysql/node_modules/mysql/lib/protocol/Protocol.js:39:16)
at Socket. (/var/lib/jenkins/jobs/sage-master/workspace/restserver/node_modules/sails-mysql/node_modules/mysql/lib/Connection.js:82:28)
at Socket.args.(anonymous function) (/var/lib/jenkins/jobs/sage-master/workspace/restserver/node_modules/nodetime/lib/core/proxy.js:131:20)
at Socket.EventEmitter.emit (events.js:95:17)
at Socket. (stream_readable.js:745:14)
at Socket.EventEmitter.emit (events.js:92:17)
at emitReadable (_stream_readable.js:407:10)
at emitReadable (_stream_readable.js:403:5)
at readableAddChunk (_stream_readable.js:165:9)
/var/lib/jenkins/jobs/sage-master/workspace/restserver/node_modules/sails-mysql/node_modules/mysql/lib/protocol/Parser.js:83
throw err;
^
RangeError: Maximum call stack size exceeded


Response code is 302. Body:  Something went wrong. Please report issue.

Response code is 302. Body:  Something went wrong. Please report issue.

if you are trying to access anonymous endpoints, this means that scraper is redirected to login page. try residential proxies, or cloud proxy fork https://github.com/restyler/instagram-php-scraper which handles proxies for you.


Initial version of stress execution engine using Decorators.

Sumamry:
This checkin implements Stress execution using decorators which converts an existing 'functional/integration' test into a Stress test.
Details:
This magic is achieved by writing a 'stressify' method decorator. Whenever a class method is adorned with this decorator it rewrites the body of the method with 'stressfied' version of the original method. This 'stressification' occurs only when environment variable SuiteType is set to 'Stress' otherwise original method body is left untouched. This replacement of method body happens at code load time so this does not introduce any additional runtime checks.
Getting this change early allows me to setup labruns and start monitoring the while we build out more functionality.

Still more unittests are being written.

Next steps:

Implement perfmon counters collection using decorators.
publish perfmon counters to perf lab for baseline setting and tracking.
end to end testing in the labs.

Team I have moved the code to a new branch feat\Stress2. I am going to merge in this code from that branch. I have address all the feedback from this thread in the new pull request I am about to create from that branch.

Superseded by: https://github.com/microsoft/azuredatastudio/pull/5499


docs: add more committers

Therefore invited

Chris Paraiso (@cparaiso)
Phillip Ball (@illiphilli)
Ryan Mathis (@rmathis)

to uportal-web-components-committers GitHub team.
(Chris Beach (@cbeach47) already had a pending invite.)

Thanks @apetro!


Added/fixed tests and exceptions

Hi @to-be-the-best,
I've got a Parsing working with .vcf files.
Do you have some time to integrate more Properties from the list: https://github.com/jeroendesloovere/vcard/blob/new-version/README.md#properties


Reward loss goes NaN

Reward Loss goes NaN. Input of RewardModel doesn't have NaN, so the cause is in RewardModel or target reward.

This is because that DeepMind Control Suite set reward in TimeStep object NaN when the TimeStep object is come from env.reset().
I resolved this problem by making OpenAI gym wrapper for DeepMind Control Suite in this commit
93866eda98229038236a0128d790e4b5b2bb673f


STDOUT error

@samzzi
https://forum.getkirby.com/t/kirby3-janitor-a-panel-button-to-run-jobs-defined-in-php/23573/14?u=bnomei
I also have the STDOUT problem with my first implementation of the new janitor.
Not to test I created a fully new and empty plainkit project and followed your readme for the example command and even there I get the STDOUT :thinking: Kirby CLI is installed btw…
As a work arround I tried working with the callbacks but as soon as I add a button to my page I always get an error when adding a new page.
Example button:
processRegistrations:
  type: janitor
  icon: cog
  progress: 'Processing...'
  command: 'janitor:job --key processRegistrations'
  label: Process registrations

So when I add that and I want to create a new page I receive this:
When I remove the button everything works as expected :thinking:

there are various things that can go wrong here. please zip and send me the plainkit.

For the STDOUT problem (Plainkit Kirby, example code from README implemented): https://gnx.cloud/mykirby-commands-stdout-bug-oS5pMi/mykirby-commands-stdout-bug.zip
Fot the "page already exists" I'm having a hard time simulating that in a plainkit 🤔 Which would make wonder it's not a janitor problem but when I comment the janitor button I can add pages. If I uncomment the button I can't add pages. Either way it seems I'll have to do more testing and research on that one.

the example is using an uninitallized $model where $page should be used. i will fix that.

TIP: just call kirby example (name of your job) to better debug jobs in an actual terminal.

I'm also getting this with my janitor call...

@jklue have you most recent CLI, kirby and janitor version?
there is an open issue which kirby will fix eventually but i need to verify an PR
https://github.com/bnomei/kirby3-janitor/pull/101
maybe its related. please tell me. a bit more about your setup

the STDOUT error will appear as well if ANY exception is triggered by your code. nothing i can do there. please try with an command that executes not code at all.

Thanks @bnomei. I'm on...
Kirby v3.9.1
CLI v1.0.0
Janitor v3.4.2
I have the blueprints/site.yml blueprint as this...
title: Site

sections:
  pages:
    type: pages
  mainfields:
    type: fields
    fields:
      call_my_command:
        type: janitor
        #command: 'example --data test'
        command: 'janitor:job --key example --data test'
        label: Call `Example` Command

and the config/config.php as this

return [
    'example' => function ($model, $data = null) {
        return [
            'status' => 200,
            'message' => $model->title() . ' ' . $data,
        ];
    },
    // ... other options
];

here's my test repo: https://github.com/visualdialogue/janitortest/

@jklue please upgrade the CLI. v1 did not have the --quiet param. then try again.


QuickUnlock only works once

I really love this plugin but sadly it doesn't work properly on my pc.
After using QuickUnlock once the next time I want to unlock my database it won't let me use QuickUnlock and the little window from KeePass2 forgets my keyfile. I have to reselect the keyfile manually and use my masterpasswort! Then it works only once again.
I tried different periodes than "Unlimited" in the options but it's always the same.
Do you have any advice for me?

Read the readme chapter "HOW TO USE"

Okay thanks, clearly my fault! ^^
Too bad that the "Part of" only works once...
There is still one problem:
First login of the day(startup) or when the QuickUnlock key is expired I have to reselect my keyfile(It's not in the dropdown I need to search it with the file explorer!)
Is this intended?

No, but isn't it the default if you restart KeePass? Does KeePass remember the last key file location? I thought that only works because KeePass remembers the path in memory (and no other key provider was used).
I don't know if I can force KeePass to use the last known key file. These parts are not visible to plugin developers and I don't know if I should make changes to KeePass internals because they could change with every version.

Just tried it without plugins, KeyPass2 remembers keyfile location at startup.
With QuickUnlock enabled it forgets the keyfile on first login if I used Quickunlock once or more often in the last session. Double-checked!
I coudn't reproduce it with the expired key though.

Ok, I will see if I can resolve that.

I have looked into the code and sadly I can't force KeePass to remember the key file path after you have used QuickUnlock. KeePass tests if your masterkey contains a keyfile but after unlocking with QuickUnlock, the masterkey only contains the QuickUnlock key so no path is saved.
This would only work if key provider plugins are able to return more than one key.

Hello,
I'm using your plugin everyday, and it works great !
I would just like to know if this issue is still in investigation (maybe in relation with Keepass Dev Team ?) or if it won't be fixed ?
Thanks !
Quentin

Currently it won't be fixed because the KeePass API is designed as it is.


Request for Llama3.1 8B, 70B and 405B

Hi!
Title is pretty much explanatory, vLLM now correctly supports the models since 0.5.3post1 and the tokenizer has been fixed
Could the new Llama3.1 models be added to the leaderboard? I could only provide the results for the 8B, I don't have the resources for 70B and 405B unquantized models.
There seems to be another PR pending for the llama3.1 models but it's related to the tool calling so I don't think it would affect the results ( https://huggingface.co/meta-llama/Meta-Llama-3.1-405B-Instruct/discussions/12 )
Thanks!

Thank you for the suggestion and the update on the support for vLLM models and tokenizer fixes. Regarding adding the new Llama3.1 models to the leaderboard, the 8B and 8B Instruct versions are already in progress. We are also planning to run the 70B and 70B Instruct models soon.

It's added now. Thanks!

Did you used the updated model with the fixed tokenizer?
I'm getting a bit better results:
Llama3.1-8B-Instruct

overall
biology
business
chemistry
computer science
economics
engineering
health
history
law
math
philosophy
physics
psychology
other

48.28
65.41
56.27
40.37
49.51
59.72
32.61
56.60
43.83
33.79
50.11
44.49
42.88
62.66
49.57

vs

If you are referring to the update to chat_template in tokenizer_config.json on July 30th, the answer is no. Thank you for the reminder; we will re-evaluate using the updated tokenizer today.

We just re-evaluated using the updated tokenizer, and the results showed very minimal fluctuations. Did you obtain your results using the code from our Git repository? If not, are there any settings that differ from ours?


[1.20.1] Crash with Physics mod

Removing this mod fixed the issue for me, seems the structure generates banner in an incompatible way
Here is the log / info
https://github.com/haubna/PhysicsMod/issues/672

It's probably incompatible because the data that the Physics mod isn't save into the structure because I don't use said mod and wouldn't use other people's mods when saving the structure. They should just add a "if (bannerPattern != null) check to avoid this issue as I can't really do anything it looks like to me to fix this.


Light's modification

Reworked internal lights in cockpit.

Added some missed LIT textures to potentiometr 4 (Panel Light)
Reworked ambient lights. Now it turn on/off depend of switch
Reworked screen ambiend light. Now it works with workingtitle g3000 mod. Depends of screen brightness.
Removed panel lights knob 100% off.
Removed Landing Gear Warning when Throttle at zero
Changed Pedestal lights remaped to Access switch
Changed Panel Light knob at day start 0% at night 50%

Nice work. I just wanted to keep the Gear Warning at idle, since I think it's present on the real plane.

Nice work. I just wanted to keep the Gear Warning at idle, since I think it's present on the real plane.

Realy? =) When throtle at iddle position it's alarm "Gear Warning"? But it's not logical. I keep worning for full flaps. But when you going from high altitude to low you switch throttle to idle (for speed management) and Gear Warning is annoyng. But anyway i can revert back

Reverted back Gear Warning

Nice work. I just wanted to keep the Gear Warning at idle, since I think it's present on the real plane.

Realy? =) When throtle at iddle position it's alarm "Gear Warning"? But it's not logical. I keep worning for full flaps. But when you going from high altitude to low you switch throttle to idle (for speed management) and Gear Warning is annoyng. But anyway i can revert back

You will most likely never put your throttle at idle on a turboprop, the drag generated by the propeller at idle is ENORMOUS, a lot bigger than in MSFS in fact. And if you think the warning is annoying, you can just place the throttle at around 1%.


[Feature Request] Driver for Pico W Network Interface (CYW43)

Is your feature request related to a problem? Please describe.
Pico W wifi networking is not available for FreeRTOS-Plus-TCP
Describe the solution you'd like
Integration of the CYP43 driver as included in the Pico SDK (https://github.com/raspberrypi/pico-sdk).
Describe alternatives you've considered
FreeRTOS w/ lwIP: https://mcuoneclipse.com/2023/03/19/ble-with-wifi-and-freertos-on-raspberry-pi-pico-w/
Additional context
none

Thanks for submitting this, Adam, we'll add this feature to our input queue.


Kconfig: clean up FPU and FPU_SHARING entries

CONFIG_FPU: The architecture dependency list is redundant.
Having CPU_HAS_FPU being selected by those archs as a dependency
is sufficient.
CONFIG_FPU_SHARING: This really ought to always default to y when FPU
and MULTITHREADING are set. Turning this off is unsafe and should be
a conscious choice from people who know what they're doing.
Signed-off-by: Nicolas Pitre<EMAIL_ADDRESS>

First, I am not sure the ARCH dependency is redundant: CPU_HAS_FPU
signifies SoC capabilities, but, theoretically, it does not imply that
the FP is supported in the respective architecture, build system,
and/or that toolchain support is present, hence the ARCH dependency
should be covering all these.

Please give me a real example where CPU_HAS_FPU would be set for an
architecture not listed in the current dependency. I can't come up with
any that would make any sense.

Second, setting the default to Sharing mode requires some agreement
among Kernel/arch maintainers; there's definitely a memory/performance
footprint increase with sharing mode.

A default is the value you get in the absence of any explicit value.
The default should be the safest choice. Nothing prevents you from
selecting another value in your board config.

I am not against the change, though. Should this be accepted, I think
we need to state this explicitly in the documentation (Kernel FP
Services chapter) and mention this in the 2.6 release notes.

Fair point.

CPU_HAS_FPU is not set for an architecture; it is set for a platform. The platform maintainer might set CPU_HAS_FPU, thinking that their SoC has an FP unit - they should be able to do that if they wish, or even accidentally, without having to care whether Zephyr supports FP services on the corresponding ARCH. Whether an ARCH in Zephyr supports Kernel FP services should be reflected in the Kernel Kconfig dependencies, so we do not accidentally get the option turned-on if some platform contributor set CPU_HAS_FPU for an board that has an arch that does not support it.
Historically we used to have such kind of dependencies to protect from accidental enabling of kernel options. For example, we used to have ARM_MPU depending on a given Cortex-M architecture variant; ARMv7-M. That's because for some time, ARMv8-M MPU variant was not yet supported in the kernel, while we did have basic v8-M support and could still contribute ARMv8-M boards, where someone would just enable CPU_HAS_MPU, because the HW included the MPU unit.

The default should be the safest choice.

Agreed, but as I mentioned above, this is not in accordance with the feedback I got here:
https://github.com/zephyrproject-rtos/zephyr/issues/29590, at least for Risc-V and x86.
So, I would recommend to change this default setting to "y" only for the ARCHes that are really affected by the relevant compiler behavior. But I would leave it to the respective arch maintainers, anyways.

CPU_HAS_FPU is not set for an architecture; it is set for a platform.
The platform maintainer might set CPU_HAS_FPU, thinking that their SoC
has an FP unit - they should be able to do that if they wish, or even
accidentally, without having to care whether Zephyr supports FP
services on the corresponding ARCH.

You are kidding me, right?

Whether an ARCH in Zephyr supports
Kernel FP services should be reflected in the Kernel Kconfig
dependencies, so we do not accidentally get the option turned-on if
some platform contributor set CPU_HAS_FPU for an board that has an
arch that does not support it.

This isn't kindergarten here.
CPU_HAS_FPU is how you reflect FPU availability in the config. If it
is not available, you don't set it, period. Beyond that is pure hair
splitting.

Historically we used to have such kind of dependencies to protect from
accidental enabling of kernel options. For example, we used to have
ARM_MPU depending on a given Cortex-M architecture variant; ARMv7-M.
That's because for some time, ARMv8-M MPU variant was not yet
supported in the kernel, while we did have basic v8-M support and
could still contribute ARMv8-M boards, where someone would just enable
CPU_HAS_MPU, because the HW included the MPU unit.

Would be much more useful to add something like this in the code:
#if !defined(CONFIG_ARMv7-M) || !defined(CONFIG_...)
#error "your ARM architecture version is not yet supported"
#endif

This way you are noisy about the lack of support rather than silently
disabling it and papering over "accidental enabling of kernel options".
And that noise can be emitted precisely from the area where development
needs can be highlighted.

I thought that the "depends on" bit on architecture is whether an architecture has FPU support (e.g. registers saving/restoring and enabling exceptions). This is more like ARCH_HAS_*. Maybe this is what we should do instead?

I thought that the "depends on" bit on architecture is whether an
architecture has FPU support (e.g. registers saving/restoring and
enabling exceptions). This is more like ARCH_HAS_*. Maybe this is
what we should do instead?

Whether it is ARCH_HAS_* or CPU_HAS_* doesn't make much difference
to me. I don't mind either ways. We could even sidestep the semantic
debate with simply HAS_FPU.
What matters is to avoid growing a list of parallel dependencies where
someone might select HAS_FPU just to make the selection possible.
If you have special conditions on your platform then it is cleaner to
do:
config MY_WEIRD_BOARD
	...
	select HAS_FPU if BAZ_WITH_FOO

This way platform specific idiosyncrasies won't clutter generic Kconfig
files.

I restored the FPU_SHARING to n except for Cortex-M but moved it out.
Now behavior is completely identical.


Fix for navmesh baking when parsing StaticBody colliders

Fix for navmesh baking.Error message: Condition “!shapes.has(p_owner)” is true.

PR should target Godot 4 master first as it is the same in both versions, then cherry-picked for Godot 3.5+.


Create uninove.txt

Institutional student's e-mails have their domain as @uninove.edu.br
Institutional website is: http://www.uninove.br/

@rodrigopedra Pull request merged. Thank you!


Reopening issue #1117

Opening the Issue #1117 as the solution is not working for trocr checkpoints.

Same issue with https://layoutlm.blob.core.windows.net/dit/dit-fts/publaynet_dit-b_cascade.pth not able to download the weights
getting 'AuthenticationFailed' after adding '?sv=2022-11-02&ss=b&srt=o&sp=r&se=2023-06-08T16:48:15Z&st=2023-06-08T08:48:15Z&spr=https&sig=IWdBT4g5TaS3Mkqib5g6IIAN2u8VSEl90SvRPfzpYNU%3D' suffix

Same issue with https://layoutlm.blob.core.windows.net/dit/dit-fts/publaynet_dit-b_cascade.pth not able to download the weights
getting 'AuthenticationFailed' after adding '?sv=2022-11-02&ss=b&srt=o&sp=r&se=2023-06-08T16:48:15Z&st=2023-06-08T08:48:15Z&spr=https&sig=IWdBT4g5TaS3Mkqib5g6IIAN2u8VSEl90SvRPfzpYNU%3D' suffix

I am able to download using https://layoutlm.blob.core.windows.net/dit/dit-fts/publaynet_dit-b_cascade.pth?sv=2022-11-02&ss=b&srt=o&sp=r&se=2023-06-08T16:48:15Z&st=2023-06-08T08:48:15Z&spr=https&sig=IWdBT4g5TaS3Mkqib5g6IIAN2u8VSEl90SvRPfzpYNU%3D.

I am able to download using the string '?sv=2022-11-02&ss=b&srt=o&sp=r&se=2033-06-08T16:48:15Z&st=2023-06-08T08:48:15Z&spr=https&sig=a9VXrihTzbWyVfaIDlIT1Z0FoR1073VB0RLQUMuudD4%3D'


pull request for e2e test at 01/19/2017 13:26:14

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Fetch rolling releases from GCS

Closes https://github.com/bazelbuild/bazelisk/issues/228

I need to figure out how the tests work to update them

Yay, done

Thanks!


Add support for delegates so that downstream plugins can be supported

Added support for 'delegates'. Delegates is just the same as what exists in Multus and can be used to execute downstream chain of plugins. In my particular use-case, I need to execute sbr.
An example of how to specify delegates:
kind: NetworkAttachmentDefinition
metadata:
  name: sriov-net1
  annotations:
    k8s.v1.cni.cncf.io/resourceName: intel.com/mlnx_sriov_rdma1
spec:
  config: '{
  "type": "sriov",
  "cniVersion": "0.3.1",
  "name": "sriov-network",
  "delegates": [
          {
                  "cniVersion": "0.3.1",
                  "type": "sbr",
                  "name": "sbr"
          }
  ]
}

With the above example, I am able to invoke a subsequent CNI plugin and configure the routing on the pod.
This is a PR to replace #90 and adds support for delegates only as suggested in this comment: https://github.com/intel/sriov-cni/pull/90#issuecomment-551056995
Other parts of the original PR will be put up in a separate request.
Thanks to @aleksandrnull for the help.
Attn: @ahalim-intel

can you elaborated why delegates are required? according to sbr docs [1] it is chain plugin.
"This plugin must be used as a chained plugin" (it is in [1]).
It seem that what you need can be already achieved by
{
"name": "sriov-network-sbr",
"cniVersion": "0.3.0",
"plugins":
[
{ "type": "sriov,,
},
{
"type": "sbr",
}
]
}
[1] -https://github.com/containernetworking/plugins/tree/master/plugins/meta/sbr

can you elaborated why delegates are required? according to sbr docs [1] it is chain plugin.
"This plugin must be used as a chained plugin" (it is in [1]).
It seem that what you need can be already achieved by
{
"name": "sriov-network-sbr",
"cniVersion": "0.3.0",
"plugins":
[
{ "type": "sriov,,
},
{
"type": "sbr",
}
]
}
[1] -https://github.com/containernetworking/plugins/tree/master/plugins/meta/sbr

Thanks @moshe010 for the suggestion. I wanted to use this through multus and the NetworkAttachmentDefinition CRD. Not sure if that CRD allows plugin chaining.

it seem that is does [1]. can you give it a try?
[1] - https://github.com/intel/multus-cni/issues/29

Not sure what I should write in the configuration file for sriov's delegates.
I have 4 sriov devices (from different physical interfaces) that I need plugged into the container. I have 4 NetworkAttachmentDefinition objects for that. I want the SBR plugin to be called for each sriov device that is moved to the container.

Not sure what I should write in the configuration file for sriov's delegates.
I have 4 sriov devices (from different physical interfaces) that I need plugged into the container. I have 4 NetworkAttachmentDefinition objects for that. I want the SBR plugin to be called for each sriov device that is moved to the container.

you probably need to add sbr as chained plugin in each NetworkAttachmentDefinition.

right, so this is what we are doing with rdma-cni (which is chained plugin ) see [1].
[1] - https://github.com/Mellanox/rdma-cni
apiVersion: "k8s.cni.cncf.io/v1"
kind: NetworkAttachmentDefinition
metadata:
name: sriov-rdma-net
annotations:
k8s.v1.cni.cncf.io/resourceName: mellanox.com/sriov_rdma
spec:
config: '{
"cniVersion": "0.3.1",
"name": "sriov-rdma-net",
"plugins": [{
"type": "sriov",
"ipam": {
"type": "host-local",
"subnet": "<IP_ADDRESS>/24",
"routes": [{
"dst": "<IP_ADDRESS>/0"
}],
"gateway": "<IP_ADDRESS>"
}
},
{
"type": "rdma"
}]
}'

I think adding delegation could be a nice feature to add. wdyt @moshe010 @zshi-redhat

@blackgold  can you explain for what use-case? and why chained plugin is not resolving it?

I think this issue is stale and something the maintainers are not willing to support currently. Please reopen if you want to continue discussion or come to the community meeting.


Update signature-update-1.png

made changes to the screenshot as the options have changed

@JohanFreelancer9 Please copy review

issue#https://github.com/MicrosoftDocs/microsoft-365-docs/issues/7067

@mjcaparas Please merge this PR. Thanks


Allow piping user agent

Minor change; allows you to pass a custom user agent header. Requested by Fabian Jakobs for cases where we want to track internal products that are built on the databricks connector.
https://databricks.atlassian.net/browse/PECO-553

Who wants this feature? Can you add JIRA ticket or other notes to the PR? With using this, we lose the usage tracking?

+1 to @yunbodeng-db. clientId is a customizable part of user-agent string, and it should be enough for tracking purposes. @fjakobs have you tried to use clientId option, and if it didn't work for you - can you tell us why? thank you!

@yunbodeng-db I was requesting this feature but it turns out that clientId is doing exactly what I want. @kravets-levko thanks for pointing me to it.


Version support

Hi all, great package! I just wondering, does this MudCalender support older version of MudBlazor and .Net? It would be awesome if it could!
Thanks in advanced!

You can use older versions of MudCalendar.  MudCalendar 1.x will work with .Net 6 and MudBlazor 6.x.


Array.prototype.at returns intersection instead of union if one set is a subset of the other

🔎 Search Terms
"Array.prototype.at", "Array.at", "item located at the specified index",  "intersection", "union", "union type", "subset"
🕗 Version & Regression Information

This is the behavior in every version I tried, and I reviewed the FAQ for entries about everything (it's very interesting) but I could not find anything related.

⏯ Playground Link
https://www.typescriptlang.org/play/?target=99&ts=5.7.0-dev.20241026#code/C4TwDgpgBAggTnAhiGUC8UDeVEC4oCMUAvgNoC6A3ALABQoksCyAQullAEb4BMJFNWnQAmEAMYAbRHGhiA9gDsAzsBwIC+eEhRQAPk20tBdectVj20uAVIAGKnQD0jqK6gA9APxOXAdTlwANZKJooqUMKW6gB0iMAAFLYAlILObh7etACWAGZQ8cJJWHSuAEScpVBZChFQnhHRnFD4paI5iACuEqoAbogSHRClgq5p6eMZJVBjE25NWUpQPHTEdD5QACJyEIsKcqoA7gGBdAzQWsgAouzYEPhEZA704OfMIABiN1B3hJRQObx+E8ROIpDIoKZwlYeJo3td9BcPsZaJDVABzKJwHh2J4zSYosKqAAWmJ4sQSyVSLjcXjouXyRKKmCmpRylWqUBJ9SJ0TyLTanW6UD6AyGI2m1NmrlptFGkqleQWUA6CkCewOChWdCAA
💻 Code
type ArrayA = { a: 1 }[];
type ArrayB = { b: 2 }[];

declare const arr1: ArrayA | ArrayB;

const c = arr1[0];
//    ^?
// Works
const d = arr1.at(0);
//    ^?
if (d) {
  "b" in d ? d.b : "default value";
  //           ^?
  //           b is 2
}

// Does not work
type ArrayE = { e: 1 }[];
type ArrayF = { e: 1; f: 2 }[];

declare const arr2: ArrayE | ArrayF;

const g = arr2[0];
//    ^?
// const g: { e: 1 } | { e: 1; f: 2 }
const h = arr2.at(0);
//    ^?
// const h: { e: 1 } | undefined
if (h) {
  "f" in h ? h.f : "default value";
  //           ^?
  //           f is unknown
}

🙁 Actual behavior
In the example, const h type is only an intersection of those two types. Therefore, f property is unknown.
🙂 Expected behavior
const h should a union type of those two types and f property should be 2.
Additional information about the issue
No response

Look up "subtype reduction".

It’s not an intersection, either. An intersection would indeed be incorrect. Maybe you mean the keys are an intersection but that’s quite different.


custom ca option for s3 storage

Thanos, Prometheus and Golang version used: v0.10.1

Object Storage Provider: NetApp StorageGrid
What happened: No documentation or ability to specify a custom CA for internally provisioned certificates.
What you expected to happen:  Documentation on how to specify a custom CA chain, or if it's not available now, request it's added.
How to reproduce it (as minimally and precisely as possible):  Use S3 storage with certificate signed by CA that is not already recognized by the node OS.
Full logs to relevant components:

Anything else we need to know:

Still an issue


Button is not clickable in titleView

Here is my situation.
I managed to display my button in the titleView. But somehow the button is not fired when I click on it.
The titleView is created by my custom view from an xib file. I even tried it with a callback method but no luck
How to fix this issue?

Hi!
When I try to set the position of a button, button is not clickable.

Hey,
can you share example with this bug?

Maybe you are talking about #129?

Feel free to reopen with example code if you still have this issue.

Hey I am facing same problem the custom button which is not clickable just because of the size of custom view is not changing. Even i tried to directly add the custom button in titleview but i couldn't make it. please help me. Here is my example code,
UIView *myView = [[UIView alloc] initWithFrame:CGRectMake(20.0, 200.0, 280.0, 45.0)];
myView.userInteractionEnabled = YES ;
myView.backgroundColor = [UIColor redColor];
UIButton *button = [UIButton buttonWithType:UIButtonTypeRoundedRect];
[button addTarget:self
           action:@selector(MyAction:)
 forControlEvents:UIControlEventTouchUpInside];
[button setTitle:@"CONTINUE" forState:UIControlStateNormal];
button.frame = CGRectMake(20.0, 580.0, 280.0, 45.0);
button.backgroundColor = [UIColor colorWithRed:10.0f/255.0f green:144.0f/255.0f blue:203.0f/255.0f alpha:1.0f];
 button.titleLabel.font = [UIFont fontWithName:@"Helvetica Neue" size:16];
[button setTitleColor:[UIColor whiteColor] forState:UIControlStateNormal];
button.userInteractionEnabled = YES;

//    button.layer.cornerRadius = 5;
[myView addSubview:button];
intro.swipeToExit = NO ;
intro.skipButton = nil ;
intro.titleView = myView;

# Button Action:
-(void)MyAction:(UIButton *)sender
{
[[NSUserDefaults standardUserDefaults] setBool:YES forKey:@"isIntro"];
[self performSegueWithIdentifier:@"SignUpVC" sender:self];
}

You have title view with height 45.0, but inserting button on Y = 580.0.
If you change button origin to (0,0) it will work perfectly:

Thanks but i want button downwards ! if i am setting frame of button it comes to the perfect place but button is not clicked (touch up inside method is not called). Please help me.

Don't move button to the bottom, move the whole title view.

Thank you so much it works perfectly but only with static values !

I will consider improving titleView layout for simpler customization.

Hey i want to set title at Top above the image but it doesn't work can you please help me?

@PurviJani you can use your custom view or just create your xib for the pages, then load as EAIntroPage
EAIntroPage *page = [EAIntroPage pageWithCustomView:yourCustomView];


VS Code won't launch

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: Yes
OS Version: Yes

Steps to Reproduce:

Open on iMac, the app boot for half a second and then shut down, it's impossible to open it, tried everything possible, uninstalled 3 times, restarted my pc multiple times, code --disable-extensions doesn't even work, nothing works

I am also facing the same issue after updating my vs code today. I tried follow threads on stackoverflow but vs code refuses to launch

Probably https://github.com/microsoft/vscode/issues/219893 which is currently pinned prominently at the top of the Issues list on this repo.

I m just going to switch to IntelliJs Community, Tired of VS Code letting me down with broken updates.This is not the first time ive spent hours trying to fix vs code when I should be productive. Thank anyways gjsjohnmurray

Ditto on Linux Mint 21.1:
System:
  Host: virtu Kernel: 5.15.0-105-generic x86_64 bits: 64 compiler: gcc
    v: 11.4.0 Desktop: Cinnamon 5.6.8 tk: GTK 3.24.33 wm: muffin
    dm: GDM3 42.0, LightDM 1.30.0 Distro: Linux Mint 21.1 Vera
    base: Ubuntu 22.04 jammy

Fresh install, consistent crashing (code 139). Downgrading to 1.90-2 immediately solved the problem.

Same issue here on MacOS. App's name appears in the Menu Bar for a few milliseconds, then quits without ever drawing a window. Was working fine yesterday.
Model Name: MacBook Pro
Model Identifier: MacBookPro11,1
Processor Name: Dual-Core Intel Core i7
System Version: macOS 11.7.10 (20G1427)
Kernel Version: Darwin 20.6.0

/duplicate

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


Incorrect google sitekey

Hi, when using anticaptcha i always get incorrect google sitekey issue even tho the sitekey is correct. i've looked in devtools network tab to confirm that it's correct and it cam out that it is. this is a url it requested to:
https://hcaptcha.com/checksiteconfig?v=1f7dc62&host=discord.com&sitekey=f5561ba9-8f1e-40ca-9b5b-a0b3f719ef34&sc=1&swa=1
As you can see, the sitekey is f5561ba9-8f1e-40ca-9b5b-a0b3f719ef34 which i use in my phone verifier. It worked fine for some time but now this issue happen and i don't know why -> key haven't changed.
Do you got any solution for that?

Hi
Anyway to provide the code that you are using to test it?
Also, that error is returned by the site's api (either ERROR_RECAPTCHA_INVALID_SITEKEY or ERROR_WRONG_GOOGLEKEY depending on the site). Meaning that it's more of an error on their end, assuming everything is correct within your code.
While I haven't done any testing, I'd suggest making sure both the sitekey and the URL the captcha is on is 100% correct.

Hi
Anyway to provide the code that you are using to test it?
Also, that error is returned by the site's api (either ERROR_RECAPTCHA_INVALID_SITEKEY or ERROR_WRONG_GOOGLEKEY depending on the site). Meaning that it's more of an error on their end, assuming everything is correct within your code.
While I haven't done any testing, I'd suggest making sure both the sitekey and the URL the captcha is on is 100% correct.

Hi, yes here is the code
import captchatools
import json
from plugins.configuration.load import config

class bypasscaptcha():
    def __init__(self):
        self.CAPTCHASERVICE, self.CAPTCHAKEY, self.SITEKEY, _, _, _, _, _, _, _ = config().loadconfig()

    def hcaptcha(self):
        solver = captchatools.captcha_harvesters(solving_site=self.CAPTCHASERVICE, api_key=self.CAPTCHAKEY, captcha_type="hcaptcha", sitekey=self.SITEKEY, captcha_url="https://discord.com/api/v9/users/@me/phone")
        self.CAPTCHATOKEN = solver.get_token()
        return self.CAPTCHATOKEN

The contents of the variables used in the code are following:
self.CAPTCHASERVICE = 2
self.CAPTCHAKEY = "my scret key"
self.SITEKEY = "f5561ba9-8f1e-40ca-9b5b-a0b3f719ef34"
Here a screenshot which show that the sitekey is indeed correct:

Interesting. Ran it and it reproduced the expected error.
Looking further into it I can confirm, I see that AntiCaptcha is the one that is reporting it's an invalid key: {"errorId":31,"errorCode":"ERROR_RECAPTCHA_INVALID_SITEKEY","errorDescription":"Recaptcha server reported that site key is invalid"}.
I just triple checked their docs and nothing has changed. I opened a support ticket with them to report the false negative in hopes they fix it 😄. Aside from that, don't think I can do anything unfortunately.
Will keep you posted with their response.

Interesting.
Using the https://discordapp.com/api/v9/users/@me/phone as the captcha_url along with the original sitekey throws no errors. Tried that?
This is the code that worked for me:
import captchatools

CAP_KEY = "..."
SITEKEY = "f5561ba9-8f1e-40ca-9b5b-a0b3f719ef34"
CAP_URL = "https://discordapp.com/api/v9/users/@me/phone"

solver = captchatools.captcha_harvesters(solving_site=2, api_key=CAP_KEY, captcha_type="hcaptcha", sitekey=SITEKEY, captcha_url=CAP_URL)
CAPTCHATOKEN = solver.get_token()
print(CAPTCHATOKEN)

Interesting.
Using the https://discordapp.com/api/v9/users/@me/phone as the captcha_url along with the original sitekey throws no errors. Tried that?
This is the code that worked for me:
import captchatools

CAP_KEY = "..."
SITEKEY = "f5561ba9-8f1e-40ca-9b5b-a0b3f719ef34"
CAP_URL = "https://discordapp.com/api/v9/users/@me/phone"

solver = captchatools.captcha_harvesters(solving_site=2, api_key=CAP_KEY, captcha_type="hcaptcha", sitekey=SITEKEY, captcha_url=CAP_URL)
CAPTCHATOKEN = solver.get_token()
print(CAPTCHATOKEN)

Can confirm, using this url instead solved the issue however it seems like the solutions which these servies provide you with (capmonster & anticaptcha) are not great so discord won't consider sending a sms. I heard users from my tool saying that 2captcha works fine but i can't confirm this either as i just added $5 to 2captcha and so far discord haven't sent a sms too.
I've basically wasted some money now cause on anticaptcha i still got $30 left which i can't even use....

Glad to hear using that url solved the issue. Going to close this issue. Will respond if and when I get an update from anticaptcha.
I'm also adding nocaptchaai since another user says they have good rates. Maybe it'll work better for your case.
PS The good thing with anticaptcha is that they allow for refunds.

Glad to hear using that url solved the issue. Going to close this issue. Will respond if and when I get an update from anticaptcha.
I'm also adding nocaptchaai since another user says they have good rates. Maybe it'll work better for your case.
PS The good thing with anticaptcha is that they allow for refunds.

Hi, it seems to me that no single captcha solver (exscept 2captcha sometimes) provide a good quality captchatoken cause discird refuse to send a sms on these services for hours (2captcha sometimes work but around 1 time from 10 tries so not good too)
A new captcha implementation would be great, i've tried to add different AI Solvers by myself (not the https://nocaptchaai.com one) with no result i'm okay with. Hopefully your https://nocaptchaai.com  implementation will work better.
Thank you a lot for this Project :)


vite build can't resolve monorepo SCSS module hoisted by yarn

Describe the bug
In the monorepo I am working on (https://github.com/nstuyvesant/carbon-charts), vite build cannot find an imported SCSS file for a package because its module was hoisted by yarn.
Within a file /package/core/src/styles/styles.scss, this line fails...
@use '@carbon/styles/scss/config' with (
  $css--font-face: false
);

This happens because the @carbon/styles package was hoisted to the root-level's node_modules directory.
If I add this to my .yarnrc.yml, it successfully resolves its...
nmHoistingLimits: workspaces

Reproduction
https://github.com/nstuyvesant/carbon-charts
Steps to reproduce

git clone https://github.com/nstuyvesant/carbon-charts.git
cd carbon-charts
Modify root-level .yarnrc.yml to remove the nmHoistingLimits: workspaces
yarn install
cd packages/core
yarn build:package

System Info
System:
    OS: macOS 13.4
    CPU: (10) arm64 Apple M1 Pro
    Memory: 384.47 MB / 32.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 18.16.0 - /opt/homebrew/bin/node
    Yarn: 3.5.1 - /opt/homebrew/bin/yarn
    npm: 9.6.7 - /opt/homebrew/bin/npm
  Browsers:
    Chrome: 114.0.5735.90
    Firefox: 113.0.2
    Safari: 16.5

vite 4.3.9

Used Package Manager
yarn
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

Can you provide a smaller repro, and are there logs when the vite build fails? Those would help us debug the issue easier 🙏

Sure - will try to work on this. It means creating a new monorepo with multiple packages though so it will take some time.

Just realized this is a sass and not a vite issue - my apologies.

Import resolving is also handled by Vite partially, in case you're debugging this: https://github.com/vitejs/vite/blob/1e299cc4b2e62a138e69d80973dfd1aed4ed6b30/packages/vite/src/node/plugins/css.ts#L1641-L1654


Blog Layout Changes

[x] Image reorganization
[x] Addition of "latest post" on /blog/
[x] Addition of "Read my latest post" on home as new
[x] Slugification of blog posts for permalinks
[x] Liquid tags for footer_buttons, blog_disclaimer, and commento

Adding "ZeroSSL Domain Verification"


docs(lib): spelling fix in README.md

Some spelling and grammar fixes.

There were the following issues with your Pull Request

Commit: 47b14806a58ce94e9309896f6e46188757600bc9
Commits must be in the following format: %{type}(%{scope}): %{description}

Guidelines are available at https://github.com/rustless/rustless/blob/master/CONTRIBUTING.md

This message was auto-generated by https://gitcop.com

@bt thanks


Use latest docker containers

This change moves all of the docker images to the latest tag as part of https://github.com/dotnet/arcade/issues/10377.

I switched it to centos stream 8 since that's our lowest-supported distro in source-build now. @MichaelSimons or @crummel let me know if that looks wrong.

Can someone with write access check this in for me? Thanks!


NTR: inviable elongated mononucleate cell during nitrogen starvation

descendant of
elongated cell during nitrogen starvation (FYPO:0000998)
(should also have some relationship to abnormal arrest at G2/M transition)
refs
PMID:12399381 https://www.ncbi.nlm.nih.gov/pmc/articles/PMC1462273/pdf/12399381.pdf

should also have some relationship to abnormal arrest at G2/M transition

How would this fit together with FYPO:0001001 ! abnormal cell cycle arrest in mitotic G2 phase during nitrogen starvation? That term has "in G2" rather than "at G2/M transition" because that's the way @Antonialock worded the request (5 years ago, in #159). Can the G2/M transition be distinguished from the rest of G2 in some experiments?
(also, this is the same paper as in #2806)

All G2 arrest is at the G2/M transition.
This area  is all a bit complicated and I don' think the biology is fully clear.
I have a feeling that the 'sizer' is set in G1 but the block manifests in G2 but that's another issue. I need to discuss this with @jackyVH  related to  #2806
But the block is at the G2/M transition becasuse for G2 that is the only arrest point.

All G2 arrest is at the G2/M transition.

Just to be extra sure: is this also true of the abnormal G2 arrest in N starvation that FYPO:0001001 represents (where the main abnormality is that arrest is G2 instead of G1)? Should I rename FYPO:0001001?

G2 arrest in N starvation that FYPO:0001001
is at the G2/M transition
All G2 arrest is at the G2/M transition
Renaming would make this  more consistent.

new term
inviable elongated mononucleate cell during nitrogen starvation FYPO:0007435
renamed G2 arrest terms to use 'G2/M transition'
merged FYPO:0003373 into FYPO:0000400
Note: I rejigged G2 vs. G2/M for a bunch of terms, which might have the odd knock-on effect on term ancestry. If you notice any missing or dodgy links, open a new ticket.


FQDN is not supported in UDR

This doc is for setting UDR, but about the SCC relay, the FQDNs are provided.
It makes our customers confused.
About the DBFS and metastore, there are the warning messages to run some periodic jobs.
but there arent any statements with SCC relay.
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: a44cbeaf-af2c-ef7e-b411-24556279d898
Version Independent ID: aaced3ee-7d7c-a9d6-a81b-b132674fa055
Content: User-defined route settings for Azure Databricks - Azure Databricks
Content Source: databricks/administration-guide/cloud-configurations/azure/udr.md
Service: azure-databricks
GitHub Login: @mssaperla
Microsoft Alias: saperla

@xujiongda  Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

Thanks for your dedication to our documentation. Unfortunately, we have been unable to address your issue and apologize for the delayed response. We are closing this issue, but if you feel that it's still a concern, please let us know directly at<EMAIL_ADDRESS>#please-close

#please-close


Something is wrong with Python.CreateEngine(); in dotnet core.

I'm trying t add this piece of code in my dotnet Core application via IronPyhon package.
var py = Python.CreateEngine();
But the fact is that It's complaining about Microsoft.Scripting namespace. I'm trying to run python scripts into my C# code.
Error: The type 'ScriptEngine' is defined in an assembly that is not referenced. You must add a reference to assembly 'Microsoft.Scripting, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7f709c5b713576e1'.

Did you add a reference to Microsoft.Scripting?

Thanks a million @slozier and @slide .
the problem was the missing DynamicLanguageRuntime package.

Maybe I should do a release with that fix so more people are bitten by the issue.

@slide Sure do it. Unless more people will get into trouble. The'll stop using your solution or they make silly issues like this one.

The release to Nuget was done

@Mohsens22  hi! I have the same issue, I installed DynamicLanguageRuntime package but still the same problem..
var engine = Python.CreateEngine(); here is not work..

Try:
var engine = IronPython.Hosting.Python.CreateEngine();


Mosaic

Changes

Adds back in the beloved mosaic gallery.
Minor selector fix on the quartet gallery (was targeting all li rather than only direct descendants).

The Mosaic Lives!!

:+1:


Date Formatting should be automated

Instead of assuming the date is formatted as dd/mm/yyyy and forcing the months and days to switch if "month">12. If would be better if you used the date format provided in the XML returned by filemaker in FMData date handling section.

While I agree with you on this topic, have you ever seen this formatting change in the output from FileMaker? We wrote some of this code back in 2002-2005, IIRC at that time it was always the same, regardless of local system settings.


Remove "TBD Intern" from project

User needs helps removing "TBD Intern" from the intern project identified here: https://github.com/SeedCompany/management/issues/41

@blakekrammes I think this could be a candidate based on today's call
cc @sethmcknight

The deed is done. @sethmcknight should we make some issues to be able to delete engagements in the UI? Could at least give that power to admins.

@blakekrammes thanks! And FYI it already exists ;) seedcompany/cord-field#480

Excellent, thanks. I did a quick search on the project board, but didn't find it.


flutter 打包报错

What went wrong:
Could not determine the dependencies of task ':amap_location:mergeReleaseResources'.

Could not resolve all task dependencies for configuration ':amap_location:releaseRuntimeClasspath'.
Could not resolve com.amap.api:location:latest.integration.
Required by:
project :amap_location
Failed to list versions for com.amap.api:location.
Unable to load Maven meta-data from http://repo1.maven.org/maven2/com/amap/api/location/maven-metadata.xml.
Could not HEAD 'http://repo1.maven.org/maven2/com/amap/api/location/maven-metadata.xml'. Received status code 501 from server: HTTPS Required

我这儿也报错，升级到flutter 1.17版本之后
FAILURE: Build failed with an exception.

What went wrong:
Could not determine the dependencies of task ':amap_location:compileDebugJavaWithJavac'.

Could not resolve all task dependencies for configuration ':amap_location:debugCompileClasspath'.
Could not resolve com.amap.api:location:latest.integration.
Required by:
project :amap_location
Failed to list versions for com.amap.api:location.
Unable to load Maven meta-data from https://storage.googleapis.com/download.flutter.io/com/amap/api/location/maven-metadata.xml.
Could not get resource 'https://storage.googleapis.com/download.flutter.io/com/amap/api/location/maven-metadata.xml'.
Could not GET 'https://storage.googleapis.com/download.flutter.io/com/amap/api/location/maven-metadata.xml'.
Connect to storage.googleapis.com:443 [storage.googleapis.com/<IP_ADDRESS>] failed: Read timed out

同上 升级到1.17.0后报相同的错


[ReuseIR][Documentation] Create and deploy documentation page for ReuseIR

We need a documentation page that hosts docs generated automatically by MLIR tablegen, which should at least cover the attributes, types, passes, and operations defined in ReuseIR. The documentation page may additionally contain extra pages rendered from markdown files in the source tree (for example, the documentation of the ReuseIR frontend and runtime should also be included).
Some examples from other MLIR dialects:

MLIR official: https://mlir.llvm.org/
CIRCT: https://circt.llvm.org
ClangIR: https://clangir.org (Note that ClangIR doc does not keep consistency in visual styles with other typical MLIR dialect docs, we'd better avoid this)

Most MLIR dialect docs, including the MLIR official docs, use Hugo together with this Hugo template.
Some immediate works required:

[ ] Scaffold the Hugo site under a new directory reuse-doc, setup necessary github workflows to deploy this site to github pages.
[ ] Add a new CMake target under reuse-mlir that generates markdown files from documentations embedded in the tablegen files, and incorporate these generated markdown files to the hugo site.

Close as completed.


Fix prettier command for Windows compatibility

What is missing from the application? Please describe.
The current configuration of the npm script for running Prettier doesn't work on Windows.
Describe the solution you'd like
Change the format of the prettier commands from using single quotes to escaped double quotes.
Example:
{
        "prettier": "prettier --write \"**/*.{css,html,js,json,md,ts,yaml,yml}\"",
        "prettier:verify": "prettier --list-different \"**/*.{css,html,js,json,md,ts,yaml,yml}\"",
}

Done in https://github.com/dreamhouseapp/dreamhouse-lwc/pull/103


Remote branch not deleted when finish release

When a release is started, published and finished, the local branch is deleted but the remote one not.

Hi, you mean the release branch? It should not be pushed to a remote. Strange.

Yes, I do mean the release branch.
It is not pushed to the remote with releaseStart. Just with releasePublish. That is fine. But I would expect that releaseFinish would delete both, remote and local, or at least that releasePublish after a releaseFinish could do the trick, deleting the remote one, but it does not.

I'm a bit busy with work and the other OS projects Swagger2Markup and Javaslang.
Do you think you can create a Push Request?


Update counter example in README.md

Update the example to show single-file component syntax instead of plain functions.

I'd suggest adding a GIF to the README under the example!

I didn't realize this earlier but the DomRenderer actually more generally runs on Pyodide instead of pyscript, since pyscript is based on Pyodide. It's really only the examples that are based on pyscript. In other words, we should be able to add a pyodide example without adjusting anything about the DomRenderer. Might be interesting to mention this as well!

Might be interesting to mention this as well!

I've added a link to pyodide, but I don't feel like creating an example at the moment. I'll leave that opportunity to someone else :)


dalli unexpectedly inserts extra characters

Mike,
I'm really grateful for dalli. Thanks so much! I believe I've encountered a bug where Dalli inserts extra characters into a value when storing on memcache. Here are the steps to reproduce.
$ service memcached restart
Stopping memcached:                                        [  OK  ]
Starting memcached:                                        [  OK  ]
$ cat dalli_bug.rb
#!/usr/bin/ruby
require 'rubygems'
require 'dalli'
data = Hash.new
data["0000ebaf8d83785add834f97f3a81934"] = "ctx=4_240_h;ctx=4_272_m;ctx=2_463_m;ctx=2_464_l;ctx=2_23_l"
puts data.inspect
dalli_client = Dalli::Client.new("localhost:11211")
data.each do|key,value|
dalli_client.set(key, value)
end
$ ./dalli_bug.rb
{"0000ebaf8d83785add834f97f3a81934"=>"ctx=4_240_h;ctx=4_272_m;ctx=2_463_m;ctx=2_464_l;ctx=2_23_l"}
$ telnet localhost 11211
Trying <IP_ADDRESS>...
Connected to localhost.
Escape character is '^]'.
get 0000ebaf8d83785add834f97f3a81934
VALUE 0000ebaf8d83785add834f97f3a81934 1 62
"?ctx=4_240_h;ctx=4_272_m;ctx=2_463_m;ctx=2_464_l;ctx=2_23_l
END
So, it seems we have a   "?    that is appended at the beginning of the value. Am I doing something wrong or have I discovered a bug?
  Use :raw => true to store raw strings.

I didn't realize there was a raw mode. Your suggestion fixed my issue. Thanks so much for the quick response.


[feature-request] Mandatory parameters on top of list in md files generated by New-MarkdownHelp

In Powershell documentation, Microsoft is grouping parameters in two groups: Required and Optional. See example for command New-Module here. This helps a lot to quickly identify parameters that matter the most.
Can something like this be implemented in New-MarkdownHelp command?
The command is already parsing all parameters, so I think it will be easy to implement. If grouping would be a problem, then at least sorting?
In standard PowerShell, you can identify mandatory parameters with
((gcm New-Module).ParameterSets.Parameters | ? IsMandatory).Name | Select -Unique

Yes, sorting makes sense. Should be easy to add a switch to New-MarkdownHelp and Update-MarkdownHelp. Adding a header ## Required Parameters would alternate the schema and this is a pretty big change with backward compatibility problems and such, so I'd prefer not going this route.

If you look at the source the .md files, they don't distinguish optional and required parameters it is done in a separate process, likely docfx. Docfx does the .md to html conversation and other post processing.
https://github.com/PowerShell/PowerShell-Docs/blob/staging/reference/6/Microsoft.PowerShell.Core/New-Module.md

@iricigor @vors We (Microsoft) are getting rid of the grouping of parameters (mandatory vs. optional). The distinction is not valid. What is mandatory in one parameter set may be optional or non-existent in another parameter set.
The syntax blocks are the only way to accurately and concisely document what is mandatory or optional.

Hi @sdwheeler, thanks for the feedback.
I would say that "mandatory in at least one parameter set" is valid distinction, but yes, it is somehow hard for understanding. The same problem with "mandatory parameter and multiple parameter sets" goes all the way back into very Get-Help command, which has the same issue.
Do you have any other ideas how to help admins to distinguish important from less important parameters?  Using just syntax blocks can be really hard and emphasizing mandatory parameters would really help. For example, a simple command Get-Job has 12 parameters and 6 different sets. How long it takes to identify mandatory parameters from these syntax blocks?
screenshot 1
Taken from https://docs.microsoft.com/en-us/powershell/module/Microsoft.PowerShell.Core/Get-Job?view=powershell-5.1

I have noticed also that docs.microsoft.com started using colouring for mandatory parameters in syntax blocks. That is nice and helpful, but unfrtunatelly not compatible with markdown or console output. Also, it is not fully persistent, some mandatory parameters are not emphasized.
screenshot 2
Taken from https://docs.microsoft.com/en-us/powershell/module/Microsoft.PowerShell.Core/Get-PSSession?view=powershell-6

@iricigor Interesting. I had not noticed that highlighting behavior before. The highlighting is there because the syntax block is identified as a PowerShell code block in the markdown for Get-PSSession. That is not the case for Get-Job.  Across the docset we have a mix of syntax blocks unmarked and marked as "PowerShell". In our style guide we have decided that syntax blocks should be unmarked because they illustrate the syntax but do not represent usable PowerShell code as shown. Ideally, it would be nice to have highlighting code that understood the syntax blocks and could highlight the mandatory elements.

@sdwheeler yes, thank you for including this guideline!


Error when running capa with Binary Ninja backend on shellcode

Description
When running capa against shellcode using the Binary Ninja backend, I encountered an error related to an unexpected file format. The analysis fails, preventing capa from assessing shellcode capabilities as expected.
Steps to Reproduce
Run capa on shellcode using Binary Ninja as a backend with the following command:
capa.exe -r C:\Tools\capa-rules -f sc64 sc.bin -b binja -d
Expected behavior:
capa should analyze the shellcode file and report any detected capabilities based on the specified rules.
Actual behavior:
The above command produces the following error:
File "C:\Users\<USER>\AppData\Local\Programs\Python\Python311\Lib\site-packages\capa\features\extractors\binja\file.py", line 174, in extract_file_format
    raise NotImplementedError(f"unexpected file format: {view_type}")
NotImplementedError: unexpected file format: Mapped

Versions
capa version: 7.3.0
Installed as: Python library
Backend: Binary Ninja 4.1.5902
Potential solution:
To work around this issue, I modified the file file.py in the Binary Ninja extractor by changing the line:
elif view_type == "Raw":
to
elif view_type == "Mapped":
within the extract_file_format function. After this change, capa ran successfully and produced the expected results.

Good find, there's no proper support for shellcode in Binary Ninja. @xusheng6, is using Mapped a proper way to handle/identify shellcode?

Yes! I think you can change the line at https://github.com/mandiant/capa/blob/2987eeb0acc03e187bdd90bb1644377774a2045a/capa/features/extractors/binja/file.py#L136 from
 elif view_type == "Raw": to elif view_type in ["Raw", "Mapped"]:

Great, thanks for the input. @as0ni do you want to make the PR?

@mr-tz will do!

@as0ni @mr-tz let me fix this one. I need to verify it actually works after the proposed change


nixos/pantheon: use Pantheon's greeter

Motivation for this change
This depends on #68492 to work.
Literally took months of fishing around for this to work properly on NixOS :rofl: (elementary/greeter#318)
elementary greeter 5.0 Release Notes
Users of Pantheon defaults in NixOS won't nudge into https://github.com/NixOS/nixpkgs/issues/60651 issue anymore.
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

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

Notify maintainers
cc @

Would it be possible to move these changes to release branch 19.09 as well? I'm affected by https://github.com/NixOS/nixpkgs/issues/60651 but for now when I manually enable the pantheon gtk's greater for lightdm (I'm using xfce as DE). For now, when I enable pantheon's greeter, I get a warning:
trace: warning: The Pantheon greeter is suboptimal in NixOS and can possibly put you in

a situation where you cannot start a session when switching desktopManagers.
Thanks!

That's a difficult situation because I had to change the global environment in a substantial way for this to work properly https://github.com/NixOS/nixpkgs/pull/68492.
That particular change isn't safe for a backport. I could adapt this change so we can have it in 19.09.
It's something I will consider as I don't want to sacrifice user experience.

Ok thanks, let me know if you integrate it at some points!


Failure Loading s3 zarr OME-NGFF on web app (possible windows issue with zarr)

I generated an ome.zarr file using the example image from ome-zarr-py:
https://gist.github.com/constantinpape/69e3cb8e0401621365d814b4d6fda0bc
However, in Windows 10, the file explorer views this as a folder named "my-first.ome.zarr".  The contents of the folder appear to me to be formatted properly, and loading the folder works in ome-zarr-py.
When uploading to s3, I can only upload "my-first.ome.zarr" as a folder, and copying the URL yields "https://....[excluded]...amazonaws.com/my-first.ome.zarr/".  This is a public URL, and I followed the instructions on http://vitessce.io/docs/data-hosting/ for setting up the S3 bucket properly.  I checked and was able to download contents inside the subfolders just fine.
Then, on vitessce.io, on the examples page, using Blin et al., PLoS Biol 2019, I simply replaced the image URL with mine.  When I try to load this image, in status, it claims "Failed to fetch"
I think this may be due to vitessce expecting a url ending in .zarr, and the url I provided ending in .zarr/.  I'm not sure how linux recognizes .zarr - a file or folder, but this might be the issue with windows here.  I tried uploading in the windows aws cli, and the web interface, and neither supported me uploading the .zarr folder as a file.

Can you share your view config?  A url to your data? Or any of the network logs showing exactly how the requests failed?

Sure - it's just the confocal image example on the page, but I changed the url to mine:
https://lstest1.s3.us-east-2.amazonaws.com/my-first.ome.zarr/
Trying the url without the "/" at the end of .zarr also fails.
Sorry, I don't know how to view the network logs in the web app, just that all the other panes don't load anything, but in the 'Status' pane, it says "Failed to fetch"
{
  "version": "1.0.6",
  "name": "Blin et al., PLoS Biol 2019",
  "description": "Mouse blastocysts imaged by confocal microscopy",
  "public": true,
  "datasets": [
    {
      "uid": "test1",
      "name": "test1",
      "files": [
        {
          "type": "raster",
          "fileType": "raster.ome-zarr",
          "url": "https://lstest1.s3.us-east-2.amazonaws.com/my-first.ome.zarr/"
        }
      ]
    }
  ],
  "initStrategy": "auto",
  "layout": [
    {
      "component": "spatial",
      "x": 0,
      "y": 0,
      "w": 8,
      "h": 12
    },
    {
      "component": "layerController",
      "x": 8,
      "y": 0,
      "w": 4,
      "h": 6
    },
    {
      "component": "description",
      "x": 8,
      "y": 6,
      "w": 4,
      "h": 3
    },
    {
      "component": "status",
      "x": 8,
      "y": 9,
      "w": 4,
      "h": 3
    }
  ]
}

There appears to be a CORS error so I think there is something up with the bucket:

Thanks - never really used those developer tools!
It looks like the example s3 CORS on http://vitessce.io/docs/data-hosting/ is missing an additional term in "ExposeHeaders".  It needs to have    "Access-Control-Allow-Origin" in addition to   "Content-Range".
After doing that, I no longer get the 'Access-Control-Allow-Origin" error that you linked above.  Instead I get "Path for image not valid".  Attached are the errors and also the contents of my-first.ome.zarr.  In the config, for the url, I again tried ending it in .zarr and .zarr/

Hi @DippedPretzels, it looks like this image does not have the omero field in the root .zattrs file.
https://lstest1.s3.us-east-2.amazonaws.com/my-first.ome.zarr/.zattrs vs https://minio-dev.openmicroscopy.org/idr/v0.3/idr0062-blin-nuclearsegmentation/6001240.zarr/.zattrs
Apologies that this is not well documented (and this error message is not very clear), but here in the loader if there is no omero field, the error will be thrown. The omero.channels and omero.rdefs fields provide metadata that vitessce uses for the initial rendering settings.

@keller-mark Thanks you're right - it didn't have an omero field.  The example .zattrs for a czyx is helpful.  It looks like the issue I have now is creating a proper czyx .zarr file.  Both examples on ome-py-zarr (here) and ngff-writer (here) do not show a case creating a czyx file.  ngff-writer, when only 2 .tff z stacks are input, would yield a .zarr with only two sub directories (/0 and /1, my .zarr here).  Trying to load this .zarr in the Vitessce actually works with one channel, but obviously fails to load the other.
I know support for czyx data was a fairly recent addition in OME-NGFF, but is there example code of creating one with any of the existing tools (ome-py-zarr, ngff-writer, bio-formats)?

Here are the error messages in Vitessce when the 2nd channel is trying to load when only 0/ and 1/ subfolders are present (for 2 .tiff Z-stacks).

Hi @DippedPretzels
I just realized that there is a new spec for OME-NGFF (v4.0) that changed part of the JSON schema. Until this new version is supported in viv (and then here in Vitessce) you may want to use a previous version of ome-zarr-py that writes v3.0 OME-NGFFs (ome-zarr==0.2.1). An example writing to a zarr store using the axes parameter:
from tifffile import imread
import zarr
from ome_zarr import writer

my_image = imread("my_image.tif")
z_root = zarr.open_group("my_image.zarr", mode = "w")

default_window = {
    "start": 0,
    "min": 0,
    "max": 65_535, # may need to change depending on the numpy dtype of the my_image array
    "end": 65_535 # may need to change depending on the numpy dtype of the my_image array
}

writer.write_image(
    image = my_image,
    group = z_root,
    axes = "czyx",
    omero = {
        "name": "My image",
        "version": "0.3",
        "rdefs": {},
        "channels": [
            {
                "label": f"Channel {i}",
                "color": "FF0000", # may want to change to a different color for each iteration or channel
                "window": default_window
            } for i in range(my_image.shape[0])
        ]
    }
)

@keller-mark Ah - thanks for all your help on this.  After trying your code above on ome-zarr==0.2.1, the result on Vitessce looked goofy.  I realized ImageJ spits out merged TIFF files (say 2 channel, 10 z steps, 1024x pixels, 1024y pixels) in ZCXY, so the .zarr file gets created this way, then Vitesse reads it as 10 channels and 2 Z steps.  Is there a way around this?

One option would be to transpose the axes after reading the image
import numpy as np

my_image = imread("my_image.tif")
my_image = np.transpose(my_image, axes=(1, 0, 3, 2)) # zcxy to czyx

Also you may need to use the chunks argument of write_image
writer.write_image(
    image = my_image,
    # ...,
    chunks=(1, 1, 256, 256)
)

That worked!  Thanks!   I did not use the chunks argument.
In the 3D view, the Z-axis looks spaced equally to x and y, so for zxy of 10 x 1024 x 1024, the 3D volume looks squished in the Z axis.  Adding new interpolated frames in z fixes this - but I'm trying to figure out how to scale the z so that's not required.
It looks like the 0.4 version of NGFF has some way to define this, bit I can't see how the 0.3 version NGFF scales an axis:
https://ngff.openmicroscopy.org/0.3/index.html
I know Vitessce doesn't yet support downscaling across the Z axis, but at least in the full, undownscaled dataset, I'm not sure how to space out across Z.


DOC: fix small typo/mistake (NaN value not dtype)

Fix small mistake in documentation.  It said NaN is not a valid dtype for integer dtypes, this surely should be NaN is not a valid value for integer dtypes.

Thanks!


Generated report contains json structure instead of image since version 2.3.0

I'm not sure if this is an issue or intended behaviour, if so, then i was not able to find how to change configuration to get it working again. See below screenshots and then related content from files: plugins\index.js: and support\index.ts:
Actual report:

Report generated with version 2.2.1:

plugins\index.js:
module.exports = (on, config) => { require('cypress-mochawesome-reporter/plugin')(on);};
support\index.ts:
import 'cypress-mochawesome-reporter/register';
import addContext from "mochawesome/addContext";
Cypress.Screenshot.defaults({screenshotOnRunFailure: false,});
function generateScreenshotName(test) {
const MAX_SPEC_NAME_LENGTH = 220;
return [test.fullTitle().slice(0, MAX_SPEC_NAME_LENGTH)," (failed) ",test.hookName,]
.filter(Boolean)
.join("")
.trim();
}
afterEach(function() {
if (this.currentTest.state === "failed") {
cy.screenshot(generateScreenshotName(this.currentTest), {
capture: "runner",
});
}
});
Cypress.on("test:after:run", (test, runnable) => {
if (test.fail == true) {
addContext({ test }, ${generateScreenshotName(runnable)}.png);
}
return;
});

There were some changes in 2.3.0 that might cause this bug, I tried to reproduce this issue but I can see the screenshots. Can you please add your OS, Node.js and Cypress versions?
Also from what I understand from the code that you have in support\index.ts, you take a screenshot when a test fails, Cypress already does that by default, and when using this reporter you will see the screenshot in the report without additional configuration. So you can probably remove all the code in this file except for:
import 'cypress-mochawesome-reporter/register';

Code was prepared there for usage of different naming logic generation(but not touched yet), anyway I reduced file support\index.ts as you suggested, but behavior is still the same.
I tested on 2 machines:

Machine: CENTOS 8 run in docker container

node.js: 13.14.0
docker: 19.03.12
cypress: 7.3.0

Machine: Windows 10 (Build 19043)

node.js: 16.9.1 ||  14.18.0
cypress: 7.3.0 || 8.3.0 || 8.4.0 || 8.4.1 || 8.5.0

I tested all combination of versions on the Windows machine, but the only thing which does matter was - cypress-mochawesome-reporter: 2.3.0 || - cypress-mochawesome-reporter: 2.2.1

For some reason, I can't reproduce the bug on my computer.
It will really help if you can add more info, please add to cypress.json this setting:
"reporterOptions": {
    "saveJson": true
}

Full cypress.json example:
{
  "reporter": "cypress-mochawesome-reporter",
  "video": false,
  "reporterOptions": {
    "saveJson": true
  }
}

After running the cypress run command it should create a file cypress/reports/html/index.json
Please post the file output here or send it to my mail<EMAIL_ADDRESS>Thanks.

I didnt get the log file to my mail, just the reply.

Hello, i see what i did wrong, now hopefully i send it to correct email address.
Best regards,
Pavel

Hey,
Thank you for sending the files.
From the log file I can see that the reporter only ran before running the tests, but not after.
I suspect that something is overriding the after:run hook, and because this reporter depends on running before and after a test run, it can cause the bug.
Please run cypress (cypress run) and check the logs in the console.
Do you see similar logs in the console?

If you see these logs, then I'm not sure what is the problem and we will need to continue to investigate.
But you if don't see these logs then I think we found the problem:
Please see step 4 in the setup, if you have something that overrides before:run or after:run you will need to manually call beforeRunHook & afterRunHook functions.
const mochawesomeReporter = require('cypress-mochawesome-reporter/lib');

module.exports = (on) => {
  on('before:run', async (details) => {
    // some code...
    // ....

    await mochawesomeReporter.beforeRunHook(details);
  });

  on('after:run', async () => {
    // some code...
    // ....

    await mochawesomeReporter.afterRunHook();
  });
};

If this doesn't work, please send me your "cypress/plugins/index.js" file and we will continue to investigate.
Thanks.

Hello, i find out, that even code was in there, it wasn't executed due to i had 2 times module.exports = (...  in the 'cypress/plugins/index.js'. So it was due to mine code. Now it's working.
I'm very sorry for waisting your time.

np, happy it's working 😊

@LironEr can you help me out here


C Engineering System Improvements

This is an old epic.  Closing.  Nothing to improve at this point.  It's perfect. :)


Update project-charter.md

Note for 8.2.1 we should consider implementing a bot to get signoff on the contributor agreement.

I propose to depreciate this pull request and split up different changes:

#80 Update Charter: update project structure and add working groups
#81 Change Charter: add quorum rule to repeat a meeting
#82 Update Charter: add link to current code of conduct
#83 Change Charter: add other OSI license options

Works for me. I'll close it.


Intermediate certs should be trusted by default when root CA is trusted

At this moment, Intermediate certificates are not trusted by default when root is trusted.
For example site vps.tomasek.cz has certificate issued by Let's Encrypt. I manually uploaded ISRG Root X1 root and marked it s trusted. Later i uploaded certificate of my site, CZERTAINLY marked it as Validation Status = Invalid. I noted that intermediate is not automatically loaded (see https://github.com/3KeyCompany/CZERTAINLY-Core/issues/783), so I added R11 Intermediate manualy. But I did not set Trusted CA = true, in fact I kept it untouched.
CZERTAINLY was still marking certificate of vps.tomasek.cz as untrusted. Untill I marked R11 as trusted.
I think that CZERTIANLY should behave as browsers, or OpenSSL. Browser has trusted root CA preinstalled and marked as trusted by browser vendor. Intermediate CAs are being provided via TLS handshake during establishing connection to HTTPS site. And without any action from user, the certificate of the site is trusted.
In case of Intermediate CA maybe the trusted CA switch needs to be used in different way:

not trusted
inherit from root (default)
trusted

I'm doubtful about trusted state - in case when root is not trusted, intermediate should be also not trusted. Probably first two values are enough.

Building the certificate chain and validation process for certificate is described here: https://docs.czertainly.com/docs/certificate-key/concept-design/core-components/certificate#validation.
CZERTAINLY is trying to build the complete chain from information that is available in the certificate inventory and from the extensions of certificate (like AIA). The validation check is performed only when the complete certificate chain is available.
For certificates, that does not include AIA, user should upload certificates of CA to the inventory to have complete chain.
Every CA certificate is initially untrusted, because we want to keep the control on the trusted chains. User should be responsible to define, what is trusted, and what is not, even when there are multiple subordinate CAs issued by one single root CA. Automatic trust for all issued certificates under the root CA may not be the desired behaviour, especially when you would like to split the trust, using similar techniques like certificate pinning.
From my perspective, what you proposed is effectively the same as the current behaviour, where you can mark the CA certificate, or self-signed certificate, as trusted.


Fix negative size if specified filesize over Integer.MAX_VALUE for stressWorkerBench

What changes are proposed in this pull request?
Change to use long type store file size.
Why are the changes needed?
Without this PR, we cannot specified a filesize lagger than Integer.MAX_VALUE.
Does this PR introduce any user facing changes?
No

alluxio-bot, merge this please


Proposition 1.10 has problems

It has two problems:
1 It does not include "true" as an atomic formula.
2 It uses the phrase "uniquely determined" which has not yet been defined.

Thanks for reporting this!
https://github.com/OpenLogicProject/OpenLogic/blob/master/first-order-logic/syntax-and-semantics/unique-readability.tex