feat: add method to fetch schema by-passing the cache

Issue
https://gravitee.atlassian.net/browse/APIM-802

Gravitee.io Automatic Deployment
🚀 A prerelease version of this package has been published on Gravitee's private artifactory, you can:

use it directly by updating your project with version: 1.0.0-apim802-add-cache-SNAPSHOT
download it from Artifactory here

:tada: This PR is included in version 1.0.0-alpha.2 :tada:
The release is available on:

1.0.0-alpha.2
GitHub release

Your semantic-release bot :package::rocket:


Stop in_exec immediately at shutdown

in_exec plugin with run_interval option does not stop immediately when sending shutdown signal.
It waits at most run_interval seconds because the process is sleeping and not interrupted.
Thread#run interrupts sleep, so this fixes the problem.

@sonots @tagomoris okay? If so, I will merge this PR and release v0.12.18.


command to open an issue in the browser

Closes #23

LGTM though we should look into better ways of adding these commands (maybe use a plugin architecture like Secretless?) in the future.


Fix stboot regression

update stboot repo to 6e2173d
Signed-off-by: Marcello Sylvester Bauer<EMAIL_ADDRESS>

will be merged in #129


send single command

it is possible to send single command to deamon like cec-client ??
for example something like "libcec-daemon KEY_VOLUME_UP" and volume go UP ??

yeah i would be interessted in this features too?
may implement an socket wich can handle CEC Commands?


Cannot find module "script-loader!./test.js"

require("script-loader!./test.js");
Console output:

app.js:4 Uncaught Error: Cannot find module "script-loader!./test.js"
at webpackMissingModule (app.js:4)

I have test.js in the same dir with the original script. I'm using Webpack 2.

Please provide more info about your setup (the test file, webpack.config.js)


feat: add callback module for scheduled execution of wasm contracts

Closes: #XXX
What is the purpose of the change

Add a description of the overall background and high level changes that this PR introduces

(E.g.: This pull request improves documentation of area A by adding ....
Testing and Verifying
(Please pick one of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added unit test that validates ...
Added integration tests for end-to-end deployment with ...
Extended integration test for ...
Manually verified the change by ...

Documentation and Release Note

[ ] Does this pull request introduce a new feature or user-facing behavior changes?
[ ] Changelog entry added to Unreleased section of CHANGELOG.md?

Where is the change documented?

[ ] Specification (x/{module}/README.md)
[ ] Osmosis documentation site
[ ] Code comments?
[ ] N/A

Closing this as we need to wait for a gov prop and forum post to discuss, looking for to discussing more! :smile:


TokenMismatchException in VerifyCsrfToken.php line 68

Laravel Version: 5.3.26
PHP Version: 5.6.28
Database Driver & Version: MySQL

Description: Some times this exception shows at the time of.

Login or
Register or
Logout or
Form Submit

What is the main logic behind the scene of this Exception?
How to remove this Exception?
Thank in Advance

Please google before opening an issue :) : http://laravel.io/forum/01-30-2015-laravel5-tokenmismatchexception-in-verifycsrftoken
Read docs also : https://laravel.com/docs/5.3/csrf

@CurosMJ I always use {{ csrf_field() }} in my form.

Please read more about CSRF protection https://laravel.com/docs/5.3/csrf

@themsaid i use {{ csrf_field()}} and this exception gives some times only not every time

@CurosMJ I have experiencing this Exception in Only Chrome Browser. Please Open this issue.

Dear All, I had the same problem and I found out that it only happen with php 5.6
I dump the variable $sessionToken and $token in vendor/laravel/framework/src/Illuminate/Foundation/Http/Middleware/VerifyCsrfToken.php in the tokenMatch method. The result is if the application running in php7.0, $sessionToken is equal to $token. but if it is running in php5.6, they are not the same.
Should we consider it is a bug in Laravel 5.3?

Finally, I found out what was happening on my problem. This is a configuration problem related to our web server which will add the Secure in all Set-Cookie header. so that only the application with https can send back the cookie to server. I just remove this setting and everything work properly again.

In your storage/cache folder you will find a session file. Delete it.


factory method for minification format

Maybe you can create a format for minification :) I'm using the following:
$format = new OutputFormat();
$format->sSpaceAfterRuleName =
$format->sSpaceBetweenBlocks =
$format->sSpaceAfterSelectorSeparator =
$format->sSpaceBeforeOpeningBrace =
$format->bSemicolonAfterLastRule =
$format->sIndentation = null;

Isn’t that essentially what OutputFormat::createCompact() does?

Yeah, though it could be more compact. From looking at OutputFormatTest, setSemicolonAfterLastRule(false) works fine, but isn't used by compact.


Custom domain is deleted from settings

I just wondering, every time when I use push-dir, npm run deploy
"deploy": "push-dir --dir=dist --branch=master --cleanup",

I just get my custom domain removed from settings of repository.
Any tips why is this happening?

Hi @yangwao, I don't have time right now to verify, but my guess is that this is happening because you have your dist folder added to your .gitignore
https://github.com/yangwao/yangwao.github.io/blob/dev/.gitignore

yes, because Nuxt will generate static content to /dist folder, which will push-dir push to the master branch.
Is there any other way how to accomplish this or is there some problems using /dist for this scenario?
Thanks, push-dir is great module! :)

I think the only way to achieve this with the current approach is to remove dist from the .gitignore and then do something like this:
$ npm run build
$ git add . && git commit -m build
$ npm run deploy

Am having the same prpblem,
I have removed the dist folder from gitignore with no luck; you can see in the video below.
https://www.useloom.com/share/a96acdbaff6c4ca0aaea4a97534aa129

@mmoscosa yes, looks pretty same. I think Nuxt should not affect this. I'm using Nuxt too.
I already moved to netlify for now, but I'd like to use github pages too for some projects with push-dir.

Also having the same problem with push-dir and Nuxt as static generator. I also removed the dist directory from my gitignore file and that didn't fix it.

https://github.com/nuxt/docs/blob/master/en/faq/github-pages.md

You need to put the CNAME in static

Although no longer documented by GitHub, using a CNAME file still works. I was having the same problem, and adding a single line with my custom domain name in static/CNAME, as @yipcma suggested, then running push-dir worked.

^ this works !


Patch 1

hey @s7dhansh is it really required? I am running on 2.3.6 and it works fine out-of-the-box. Do you have any errors?

@jankapunkt may be not. Will close.


UserGuide: Add hyperlink to return to top of file #430

Part of #430. For your review @eugenepeh, I only did for the UserGuide first :> Also, I added the return to top at the start of each section instead of at the end because it seemed more intuitive to me (user clicks on "AddCommand", glances at it, clicks return to top before clicking on other things).

You should also add one Return to Top right at the end of the document. If the section happens to be very long, then the user will have to scroll back up to press Return to Top.

Also, I added the return to top at the start of each section instead of at the end

Looks good, however this only works due to the fact that most of our sections are small (fits within the screen). Also, Return to top also helps to serve as an indicator that user has read to the end of the section.
Additionally, we should take into consideration that this will be used by students of CS2103 and hence, it may be better to follow the common practice (which is to put the Return to top at the end of each section) . It is innovative to place Return to top at the start of each section, instead of at the end, but it is not widely used/accepted yet.

v1
@Zhiyuan-Amos submitted v1 for review.

[v1 1/1] Documentation: Add hyperlink to return to top of file

(:books: Archive)

Checkout this PR version locally
git fetch https://github.com/se-edu/addressbook-level4.git refs/pr/496/1/head:BRANCHNAME

where BRANCHNAME is the name of the local branch you wish to fetch this PR to.

@eugenepeh @yamgent noted, made the necessary changes!

This is too boring :P I'll let someone else do instead.


:gear:  RoR group project :gear:

RoR group project: set up your Kanban board
This project is handled by only one person i.e @SimonGideon
Here is the link of the Kanban board here :point_right:

Hi @SimonGideon,
While you made a great effort in this project, unfortunately, I cannot proceed to review your code.
Invalid Code Review Request
- You need to mention the number of people assigned to this project. Kindly do that,
- Your link appears to be broken.

Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

Invalid Code Review Request does not count into the code reviews limit.

Sorry the project was private. I have now made it public

🟢 Project Approved! 🏆
Hi @SimonGideon  👋
Your project is complete! ✔✔ , I loved your work, it looks great 🔥🔥🔥🔥 🏁
Your code was very professional 💯
There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉
Highlights 🌟

✔ Descriptive commmit messsages

✔ Descriptive PR description

✔ No linter Errors

The code is top-notch! 💯 and have a huge potential.

Cheers and Happy coding!👏👏👏
Important Notes 📝

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

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


[feature]: support insert data by sql

Is your feature request related to a problem? Please describe.
if schema is predefined, data can be inserted by inserting statement.
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

What about:
INSERT INTO
    tb_name
        [TAGS (tag1_value, ...)]
        [(field1_name, ...)]
        VALUES (field1_value, ...) [(field1_value2, ...) ...]

Insert one record with function now(), the timestamp is always in the first position.

INSERT INTO d1001 tags("sh", "<IP_ADDRESS>") VALUES (now(), 10.2, 219, 0.32);

Insert multiple records with time, I think adding comma is better.

INSERT INTO d1001 VALUES
('2021-07-13 14:06:32.272', 10.2, 219, 0.32),
(1626164208000, 10.15, 217, 0.33),
(now()-1s, 1, 2, 3);

Specify column names when inserting data.

-- ts or timestamp ?
INSERT INTO d1001 (ts, current, phase) VALUES (now(), 10.27, 0.31);

Allow table creation during insertion ？

ref tdengine

I think it is difficult to define the type of fields in insert statement. So not allow table creation during insertion.
Steps:

create table(#1010 )

create table if not exist cpu
(
    host tag,
    ip tag,
    load last,
    temperature max
);

insert data

-- single
INSERT INTO cpu (timestamp, host, ip, load) VALUES (now(), "sh","<IP_ADDRESS>",4);
-- batch
INSERT INTO cpu (timestamp, host, ip, load)
VALUES
(now(), "sh","<IP_ADDRESS>",4),
(now(), "sh","<IP_ADDRESS>",5),
(now(), "sh","<IP_ADDRESS>",4),
;

also depend(#1015  )


ignore_classes support for modules

It would be great if the ignore_classes infrastructure considered modules included in the exception's class and singleton class e.g.
Bugsnag.configure do |config|
  # Ignore any exceptions that include the ExceptionReporting::IgnorableError module
  config.ignore_classes << ExceptionReporting::IgnorableError
end

...

begin
  do_something!
rescue MyException => e
  # We generally want to report MyExceptions to Bugsnag but not in this particular case.
  # However we still want to re-raise the exception so things like Delayed Job failure handling
  # will kick in.
  raise e.extend(ExceptionReporting::IgnorableError)
end

The workaround is to use the proc form of ignore_classes:
Bugsnag.configure do |config|
  config.ignore_classes << Proc.new do |exception|
    exception.is_a?(ExceptionReporter::IgnorableError)
  end
end

That's a creative solution to the problem 🤔
I think we've a lot of complexity when it comes to ignore_classes and customization which would best be served by adding a more generic mechanism to configuration to allow users to edit (and cancel) error reports. Its something we support on other platforms now, and are working towards for the next major version, which will be completed in the near future.


Global config location?

Hi, does the Taplo CLI support a global config location? From the docs, it appears it only searches the working directory and the root folder of a project. My suggestion is to use a configurable location with a flag or to search something like $HOME/.config/taplo. Thanks!

Hi,
The docs for the CLI are non-exhaustive, there should be a flag that sets the path to the config file, I believe it's -c or --config.

Ah okay, thank you!


Feature/#45 show classifiers and types

Coverage increased (+1.08%) to 83.904% when pulling 2845c721452220a16948560316465a951304b85e on feature/#45-show-classifiers-and-types into 8a23c4833532456e31aad7f17389153c2776a651 on master.


Update community section

What this PR does / why we need it:
This adds info on the correct community meeting time as well as info on the new public standup.
Release note:

NONE

/lgtm
/approve


Only output report if there is something to report.

Fixes #104

Hi @jonparrott, thanks for the pull request. Before we can merge it, we need you to sign our Contributor License Agreement. You can do so electronically here: http://opensource.box.com/cla
Once you have signed, just add a comment to this pull request saying, "CLA signed". Thanks!

@Jeff-Meadows @boxcla I'm waiting for our Open Source lawyers to approve me signing the CLA. I'll get that resolves as soon as I can. Thanks!

CLA signed

Verified that @jonparrott has just signed the CLA. Thanks, and we look forward to your contribution.

Thanks! When can I expect the next release?

I just pushed a new release. flaky 3.2.0 has your changes!
On Wed, Jul 20, 2016 at 4:41 PM, Jon Wayne Parrott<EMAIL_ADDRESS>
wrote:

Thanks! When can I expect the next release?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/box/flaky/pull/105#issuecomment-234116895, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ADqVa6M6ZLH07MRwhEtx6HC7kBWsFbFaks5qXrIagaJpZM4JKrXI
.

Thank you much!


Add requirements.txt with proper versions

Please add the corresponding version number for dependencies. For eg. nptyping of 1.4 is required but it downloads the latest which has a different API

Thanks for pointing this out. I just fixed it.


zephyr-dom0-xt: fix default build without builtin domu1

The default build is failed now because string Kconfig options
CONFIG_DOM_CFG_DOMU1_IMAGE_BIN_FILE
CONFIG_DOM_CFG_DOMU1_DTB_BIN_FILE
are defined as empty strings always which isn't handled properly with pre-processor and causes defined(CONFIG_DOM_CFG_DOMU1_DTB_BIN_FILE) to return true.
Fix it by adding additional Kconfig option:
CONFIG_DOM_CFG_DOMU1_ENABLE
and make DomU1 Kconfig option depend on it. This will guarantee that DomU1 Kconfig options are not defined, so defined() directive will work as expected and build system will not try to include non-existed DomU1 images.
This will allow to build for qemu or rcar platform.

Example:

Thanks! Reviewed-by: Oleksii Moisieiev<EMAIL_ADDRESS>


Use next_node blocks in calculate-your-child-maintenance

We've agreed to consistently use next_node {} to define our next node rules. Having a single way of defining the rules will hopefully make Smart Answers easier to develop and maintain.
This will ultimately allow us to remove the predicate code (define_predicate, on_condition, next_node_if etc).

I think it would at least be good to explain your choice of case or if/elsif constructions in the PR description or commit messages.

Other than my very minor comments, this LGTM :+1:

I've changed the next_node blocks for :gross_income_of_payee? and :how_many_nights_children_stay_with_payee? so that they use a case statement instead of if .. else .. end. This is more consistent with what I've done elsewhere, as highlighted by @floehopper.
I've rebased on master and force pushed in preparation for merging.


Problem with Table tbody font-weight

When I use this package, it displays everything correcly and the style of the tbody is ok, but when i try to use the plugin by itself, i have all the td styled with bold for an unknown reason, is there anything i can do ??

font-weight goes thru css. Better consult your web designer or download the standard assets to apply the default styling of dataTables. See the plugins documentation for more ref. http://datatables.net/examples/index


Fix typo in device-mapper-driver.md

Proposed changes
Fix a typo in the device-mapper-driver.md file.

Thank you!


How to use it in React-Native ?

Can we use it in React-Native ? If yes, can you please make an example.

https://github.com/skavinvarnan/Cross-Platform-AES/issues/33#issuecomment-643967274


Check scikit-robot version in teaching_mode.py

@nakane11
Please review this pull request if you have time.

I confirmed that the following error occurs when running with skrobot == 0.0.37 and it works fine with 0.0.44
File "/home/rock/ros/kxr/src/riberry/ros/riberry_startup/node_scripts/teaching_mode.py", line 198, in <module>
   raise Exception(f"skrobot version is not greater than {required_version}. (current version: {current_version})\npip install scikit-robot -U")
Exception: skrobot version is not greater than 0.0.44. (current version: 0.0.37)


Support for the tensor package

The tensor package in LaTeX is quite handy for writing tensors with mixed covariant and contravariant indices in an ergonomic fashion.
I think the package is quite small and would be fairly easy to add. I doubt I will get around to doing it myself, but there is a small change that it could happen. Would a PR for this package as a first-party extension be accepted should I ever get around to it?

It does look like it would not be too bad to implement, and might be a good first project for someone interested in contributing to MathJax.  We would certainly entertain a PR for such an implementation, thought that is not a guarantee of acceptance.

Hay is anyone still working on this? I really need this feature, but I don't know how to do this yet


[BUG] ISO Week wrong week number and starting on Sunday

As title says, there seem to be two problems with ISO weeks.
Current date: June 25, 2024
ISO Week 26
Periodic Note Weekly Note Format Setting: [W]WW-MMM-YYYY
What is the weirdest is that in the little preview, the "Your syntax looks like", it correctly displays W26-Jun-2024 but then when I click on the ribbon button, it wrongly (tries to) creates W25-Jun-2024
Further, it still considers Sunday the first day of the week, even when using WW in the format setting.
Even looking at my Windows settings, it is set to first day of the week being Monday.

This issue has existed for ages, but by a mysterious coincidence today it annoyed me enough to come here and log this issue, but I see you have beat me to it 😄

I'm bothered by this same issue as well, I honestly wouldn't mind except for that I'm guessing this throws off the opening current weeks note command. Assuming this is related also - the integration with calendar is just all messed up in general.

Duplicate of #128?


The certificates do not work without --insecure

Which makes lab 3 fail on my machine

Thanks for reporting. This was fixed by @evenh in #2.


Clarifying that Bicep samples replace the usual .json parameter file

Description

Clarifying the fact that the Bicep code samples "act as parameter file"
Clarifying that if the user wants to use the .json parameters file, then they need to declare the parameters at the top of the Bicep main file

@gbeaud a few changes please. I think the test is failing as the version in the metadata section of main.json has likely changed (which is annoying)
@shenglol is there any way in the test to expose what the diff/change is to help troubleshoot? Also it may be worth updating the test triggers to not trigger on doc updates? (I'm happy to put a PR in for that, if you agree)

@gbeaud a few changes please. I think the test is failing as the version in the metadata section of main.json has likely changed (which is annoying)
@shenglol is there any way in the test to expose what the diff/change is to help troubleshoot? Also it may be worth updating the test triggers to not trigger on doc updates? (I'm happy to put a PR in for that, if you agree)

@shenglol, I think we should consider adding a path filter that excludes something like *.md. There seems to be something funny going on here, but its unrelated to the changes being proposed.

@gbeaud a few changes please. I think the test is failing as the version in the metadata section of main.json has likely changed (which is annoying)
@shenglol is there any way in the test to expose what the diff/change is to help troubleshoot? Also it may be worth updating the test triggers to not trigger on doc updates? (I'm happy to put a PR in for that, if you agree)

@jtracey93, to reproduce what is going on, you can open up the branch, and run the "brm generate" command in the directory. It looks like the output of "brm generate" may have changed. Now, none of this has anything to do with the changes to the README, but it may be worth still looking into and understanding what's going on.

@jtracey93 - Is this still in progress 🤷


Can I alter query params on a query?

Hey,
Nice library :)
The official Trello boards API page shows all the query params that can be passed to control things like the amount of data that is returned: https://developers.trello.com/v1.0/reference#boardsboardid-1
In particular, the fields property. Can I achieve this with this lib?

@jpsear
Can you put your request sample here?

@jpsear
Thank you for the quick reply. Can you also post function that you are using from this library?

I guess I wouldn't know which to use. It would possibly be trello.board.search(boardId)

@jpsear
That is already supported, here is the example :
    var boardId = 'BOARD_ID';
    var fields = {
        actions: 'all'
    };
    Trello.board.search(boardId, fields).then(function (response) {
        console.log('response ', response);
    }).catch(function (error) {
        console.log('error', error);
    });

    var boardId = 'BOARD_ID';
    var field = 'shortUrl';
    Trello.board.searchField(boardId, field).then(function (response) {
        console.log('response ', response);
    }).catch(function (error) {
        console.log('error', error);
    });


Medtronic device events

Adds support for:

alarms
prime events (cannula/tubing)
reservoir changes (rewind events)
status events
time change events

For more detail, see https://trello.com/c/6b5oFTZl
Note: #328 should be merged first. This PR also includes the code to download the binary blob reviewed in #331.

 [uploader] Medtronic device events

 [uploader] Button to get raw binary data from pump in debug mode

Pinging @jebeck for review.


Improve code-coverage & add GH check action

Background
Improve or add code coverage support to be able measure code quality.
Feature
New ability to measure current code coverage as one of QA metrics.
Add GH action to check changed file coverage.

Release notes proposal:
#59 - Add code coverage support improve

Used Absa fork of JaCoCo solution - with scala method filtering.
Added support for code coverage GitHub action check as part of PR.


added tests for work issues

https://github.com/bcgov/eagle-public/issues/665

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

:exclamation: No coverage uploaded for pull request base (develop@0aead41). Click here to learn what that means.

Additional details and impacted files
@@            Coverage Diff             @@
##             develop    #1666   +/-   ##
==========================================
  Coverage           ?   62.93%
==========================================
  Files              ?      248
  Lines              ?     8201
  Branches           ?        0
==========================================
  Hits               ?     5161
  Misses             ?     3040
  Partials           ?        0

Flag
Coverage Δ

epictrack-api
62.93% <0.00%> (?)

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

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


Fix prevDeps is undefined error when <MagicTabSelect> is child of <MagicMotion>

✨ Overview

this error occurs when  is a child of <MagicMotion> in v1.1.2

Here is a codesandbox with the error:
https://codesandbox.io/p/sandbox/react-magic-motion-sidebar-test-56rjpd?file=%2Fsrc%2FApp.tsx%3A2%2C34

I added a section to the docs page to address how to fix this:
https://www.react-magic-motion.com/troubleshootings


Accept single string values for mixin

Currently mixin requires a list of strings.  For consistency with other stuff, we may want to accept a single string value too.

@quasicomputational I think this one would still be neat to have for 0.31.0.  I'll open a PR.

On Hackage


[ELE-844] edr send-report with Slack webhook

Is your feature request related to a problem? Please describe.
edr send-report only currently supports Slack Token. Since Slack Token needs more access as compared to the Slack Webhook, some company's security policies may not allow it.
Describe the solution you'd like
edr send-report to support Slack Webhook after report is already generated and uploaded in GCS or S3.
Describe alternatives you've considered
As an alternative, perhaps the report which will be generated from edr send-report will be uploaded first to GCS or S3, and then still send the report/summary on Slack that is triggered via Slack Webhook.
Additional context
Add any other context or screenshots about the feature request here.
Would you be willing to contribute this feature?
Let us know if you're willing to contribute so that we'll be able to provide guidance.
ELE-844

Hi @kiasomabes, thanks for opening this issue.
The reason send-report doesn't support a Slack webhook is because you cannot send a file over a webhook in Slack.
However, your suggestion of being able to use it to only send the summary makes sense.
Would you be willing to contribute this feature?
I'd be able to provide guidelines if necessary.

Hi @elongl sure. Please do provide the guidelines and instructions, I'm not familiar with the codebase myself.

Hey @kiasomabes! I'll try to help out as much as I can and let me know if you need further help :)
First of all you can see here we're checking only if there's a slack token and not for a webhook
Once we pass that get here data_monitoring_report.py where we test to see if we should send the attachment, and this is also where we send the summary
So maybe checking for a webhook in the first location, and setting a var to specify "summary only" and use it in the second location could be a good place to start :)

hi @noaKurman thanks for providing the overview! how can I test if my code actually works? i've followed up until the requirements setup using this guide.

Hey @kiasomabes!
To test your code you can just install you own elementary command -
Just like you usually install elementary CLI using pip install elementary you can now install you own code by running pip install -e path/to/elementary/repo - and then just run the commands and make sure everything works :)

Hi @noaKurman, thanks for the response - I'm encountering some error, which I'm not sure was happening.
I just tried to run with edr send-report for now, to see whether it would actually work as usual. I also modified other files such as the config - but I'm not sure which enforces the parameters to be required in the first place.
return f(get_current_context(), *args, **kwargs)
TypeError: send_report() missing 1 required positional argument: 'slack_webhook'

This is my commit. Would appreciate feedback, thanks!

hi @noaKurman, I'm running into an error upon running edr send-report --slack-webhook <webhook-url>. The error is
raise DbtCommandError(err, command_args, err_msg) elementary.exceptions.exceptions.DbtCommandError: Failed to run dbt command.
My changes are in this fork. Would appreciate feedback. Thank you!

Hey @kiasomabes
Can you send the full error? Doesn't look like you changed anything in the data so I'm not sure if this error is related to the changes you made or something is just wrong with the envirnonment you set up

Hey @kiasomabes!
I was looking at the fork you shared with us, and I have another idea on how to implement it there 🙂
I think that adding an --slack-webhook option to the cli send-report command is good, but instead of adding the summary_only option to the send report command, you should:

Instead of passing summary_only param to send_report method you should use the CLI argument --disable like edr send report --slack-webhook <webhook> --disable html_attachment.
Replace the NotImlementedError on the send_report method at the SlackWebhookClient with an indicative error that explains that "Slack is not support sending files using webhooks, please use slack token instead".

That way we keep the DataMonitoringReport class a bit simpler, and we get the same effect 🙂

Hey @kiasomabes Can you send the full error? Doesn't look like you changed anything in the data so I'm not sure if this error is related to the changes you made or something is just wrong with the environment you set up

I think it has something to do with the dbt setup - not sure if it's related to some other issue with dbt.
`2023-05-23 16:25:16 — INFO — Running with edr=0.7.9
2023-05-23 16:25:16 — INFO — Downloading edr internal dbt package
2023-05-23 16:25:22 — ERROR — Unable to get the latest invocation: Failed to run dbt command.
2023-05-23 16:25:22 — INFO — Downloading edr internal dbt package
2023-05-23 16:25:22 — INFO — Running dbt deps --project-dir /Users/kia.somabes/Desktop/elementary/elementary/monitor/dbt_project
Usage: dbt deps [OPTIONS]
Try 'dbt deps -h' for help.
Error: Invalid value for '--profiles-dir': Path '/Users/kia.somabes/.dbt' does not exist.
Traceback (most recent call last):
File "/Users/kia.somabes/Desktop/elementary/elementary/clients/dbt/dbt_runner.py", line 65, in _run_command
result = subprocess.run(
File<EMAIL_ADDRESS>line 526, in run
raise CalledProcessError(retcode, process.args,
subprocess.CalledProcessError: Command '['dbt', 'deps', '--project-dir', '/Users/kia.somabes/Desktop/elementary/elementary/monitor/dbt_project']' returned non-zero exit status 2.
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/Users/kia.somabes/Desktop/elementary/venv/bin/edr", line 33, in
sys.exit(load_entry_point('elementary-data', 'console_scripts', 'edr')())
File "/Users/kia.somabes/Desktop/elementary/venv/lib/python3.10/site-packages/click/core.py", line 1130, in call
return self.main(*args, **kwargs)
File "/Users/kia.somabes/Desktop/elementary/venv/lib/python3.10/site-packages/click/core.py", line 1055, in main
rv = self.invoke(ctx)
File "/Users/kia.somabes/Desktop/elementary/elementary/cli/cli.py", line 67, in invoke
return super().invoke(ctx)
File "/Users/kia.somabes/Desktop/elementary/venv/lib/python3.10/site-packages/click/core.py", line 1657, in invoke
return _process_result(sub_ctx.command.invoke(sub_ctx))
File "/Users/kia.somabes/Desktop/elementary/venv/lib/python3.10/site-packages/click/core.py", line 1404, in invoke
return ctx.invoke(self.callback, **ctx.params)
File "/Users/kia.somabes/Desktop/elementary/venv/lib/python3.10/site-packages/click/core.py", line 760, in invoke
return __callback(*args, **kwargs)
File "/Users/kia.somabes/Desktop/elementary/venv/lib/python3.10/site-packages/click/decorators.py", line 26, in new_func
return f(get_current_context(), *args, **kwargs)
File "/Users/kia.somabes/Desktop/elementary/elementary/monitor/cli.py", line 579, in send_report
data_monitoring = DataMonitoringReport(
File "/Users/kia.somabes/Desktop/elementary/elementary/monitor/data_monitoring/report/data_monitoring_report.py", line 37, in init
super().init(
File "/Users/kia.somabes/Desktop/elementary/elementary/monitor/data_monitoring/data_monitoring.py", line 49, in init
self._download_dbt_package_if_needed(force_update_dbt_package)
File "/Users/kia.somabes/Desktop/elementary/elementary/monitor/data_monitoring/data_monitoring.py", line 91, in _download_dbt_package_if_needed
package_downloaded = self.internal_dbt_runner.deps()
File "/Users/kia.somabes/Desktop/elementary/elementary/clients/dbt/dbt_runner.py", line 102, in deps
success, _ = self._run_command(command_args=["deps"], quiet=quiet)
File "/Users/kia.somabes/Desktop/elementary/elementary/clients/dbt/dbt_runner.py", line 87, in _run_command
raise DbtCommandError(err, command_args, err_msg)
elementary.exceptions.exceptions.DbtCommandError: Failed to run dbt command.`
When I tried pip list | grep dbt, I got the following:
dbt-core                 1.5.0 dbt-extractor            0.4.1

Hey @kiasomabes thanks for continue and working on this contribution!
When you run edr send-report elementary try and generate the report based on your dbt project (using the elementary profile that you provide).
The inner dbt project, is used to connect to the DWH (with elementary profile) and create the report from elementary artifacts (created with elementary dbt package).
So the error that you see means that elementary couldn't generate a report, and therefore you couldn't test the webhook.
Do you have a dbt project that you used to run with elementary? If so you could use it to test the feature, and if you don't have we could help you with the tests as well 🙂


Marking parts not found at any distributor

Instead of coloring cells green when all distributors show as blank (no part found) - it would be best to color all cells red instead.   This will help identify where no part was found.

I tried one propose of this in the last pull request #81, it is working fine in LibreOffice Calc but, have to be tested in some different versions of Microsoft Office.

Closed!

@xesscorp, could you close this issue? I was checking the issue list and there is at least 5 already solved but opened yet. This create some confusion in each implementation we have to focus.


✔ added checks for override.cfg setup

mod_loader_setup.gd
Prevents the setup script from running the setup again, if the mod loader was installed via the override.cfg.
mod_loader.gd
Prevents the error logging in _check_first_autoload(), if a override.cfg file is present.

I think this code will never run, if there is no autoloads there will be no mod_loader.gd right?
https://github.com/GodotModding/godot-mod-loader/blob/ae19b71bec47fd02135f47251f3d4a4060efba29/addons/mod_loader/mod_loader.gd#L198-L199

you are right, that will never run. the mod loader will always occupy one slot or not be around to tell anyone. this should be in the setup, if at all.


Chapter 19 does not compile

I have been following along in the book. When I added the code for Chapter 19, the DatabaseExtensions.kt file will not compile. To ensure that I didn't type something wrong, I got a zip of the Chapter 19 commit and got the same error:

Error:(8, 19) Object must be declared abstract or implement abstract member public abstract fun parseRow(columns: Map<String, Any?>): T defined in org.jetbrains.anko.db.MapRowParser
Error:(9, 13) 'parseRow' overrides nothing
Error:(13, 18) Object must be declared abstract or implement abstract member public abstract fun parseRow(columns: Map<String, Any?>): T defined in org.jetbrains.anko.db.MapRowParser
Error:(14, 13) 'parseRow' overrides nothing

I'm using buildVersonSdk 25.0.0, targetSdkVersion 25, ext.support_version 25.1.0, ext.kotlin_version 1.0.6, ext_anko_version 0.9 in Android Studio. Everything had worked up to the transition from earlier chapters to chapter 19. Since I'm new to Kotlin, I don't have a clue what is wrong or how to fix it.

I reverted to Anko version 0.8.3 in AndroidStudio and the demo app code is working now. I don't know if something in Anko changed and demo the broke or if Anko has a bug. I suspect the latter, but am not knowledgeable enough to make that call and possibly send a bug report to Anko developers.

OK, I figured it out. Lack of nullable values in anko-db #154 in Anko changed the Pair<String, Any> to nullable Pair<String, Any?> which requires similar changes to the function signatures for parseList() and parseOpt().

Yeah sorry, that's the thing. I'm updating the book these days to cover changes in Kotlin 1.0.6 and Anko 0.9. Will also update this repository accordingly.
Thanks!

Well, 1.0.X version of a language, these kinds of things are going to happen. The worst part of this issue is the not-very-helpful compiler errors. I would have expected something more like "type mismatch" or, better, "non-nullable type used with nullable parameter".
Thanks for the comment and looking forward to the update.

I finished updating the book, and the new code is on the repo. If you have the digital edition, you'll receive the update soon (may be you did already).
Closing this. Thanks!


Add example auth configuration in config.dev.yaml

Make auth config settings more transparent by providing an example.

I will grab this. ATM there are only two auth providers. The configuration is simple, but requires right now digging into code to know the format of configuration for them. It should be easy to update the config.dev.yaml to show it (commented).


Feature Request: Insert Line Breaks When Formatting

Suggestion
Given this code:
assert.deepStrictEqual(
  context.getAndClearViewStates(),

  [
    'foo[bar]',
    'foob[ar]',
    'foobi',
    'foobi[zz]']
);

If I format this code, I would expect the outcome to be
assert.deepStrictEqual(
  context.getAndClearViewStates(),

  [
    'foo[bar]',
    'foob[ar]',
    'foobi',
    'foobi[zz]'
  ]
);

However, currently the formatter does nothing on the input code.
It would be nice if this is supported.
🔍 Search Terms
Formatting, Formatter, Line Break, Prettier
More Example
This is also not formatted when invoking the formatter:
const range = word ? { startLineNumber: 1, startColumn: word.startColumn, endLineNumber: 1, endColumn: word.endColumn } :
  Range.fromPositions(pos);

Thanks!


experiment reproduction

Hi, there.
Thanks for sharing this great work.
I download the code and try to reproduce the results in table 4. However, the results is lower than reported in paper.
due to gpu limitation, I just run two corruption.
I just run:
python3 main.py --data_corruption ./data/ImageNet-C \
--exp_type bs1 \
--method sar \
--model resnet50_gn_timm \
--output ./output/
For Resnet50-GN results in table 4. My result is：
2023-03-01 18:39:05,186 INFO    : Result under shot_noise. The adaptation accuracy of SAR is top1: 22.83800 and top5: 40.58400
2023-03-01 18:39:05,198 INFO    : acc1s are [20.255998611450195, 22.83799934387207]
2023-03-01 18:39:05,198 INFO    : acc5s are [37.077999114990234, 40.58399963378906]
For VitBase-LN results in table 4. My result is：
2023-03-01 18:24:27,754 INFO    : Result under shot_noise. The adaptation accuracy of SAR is top1: 25.73800 and top5: 44.81800
2023-03-01 18:24:27,820 INFO    : acc1s are [32.34600067138672, 25.737998962402344]
2023-03-01 18:24:27,820 INFO    : acc5s are [53.21399688720703, 44.817996978759766]
Did I miss something? Looking forward to your reply.

Hi,
Thanks for your interest and for finding this bug.
The reason for this issue is that the hyper-parameter [learning rate] was incorrectly set in the previous code for BS=1 experiments. For further details, please refer to Appendix C.2 of the paper.
I have added lines 205-207 in main.py. Now you can check and verify the BS=1 results in Table 4 again.
Below is the result that I reproduced with ResNet-50-GN and BS=1 (same as the paper):
2023-03-01 21:09:46,089 INFO    : Result under gaussian_noise. The adaptation accuracy of SAR is top1: 23.79600 and top5: 42.32000
2023-03-01 21:09:46,090 INFO    : acc1s are [23.79599952697754]
2023-03-01 21:09:46,090 INFO    : acc5s are [42.31999969482422]

great! Thanks for your kind reply


Missing training model configs

Hi team, thank you for the great work!
https://github.com/mlfoundations/dclm/blob/c0b0f6cf49c4344fb6f7f6248bd569f072eb036b/training/configs/411m_1x.json#L2
It seems training/open_lm_configs/d=1024_l=24_h=8.json should be there, but missing currently.

Thanks for catching this issue. I believe this is now resolved with this commit https://github.com/mlfoundations/dclm/commit/ab4ef62815215531f2580ebf5e70c2f5e51c51d8


fix(format): remove leading whitespace from lists

Leading whitespace is not strictly enforced, but is incorrectly present when the formatting of a list is fixed. The behaviour can be made consistent by requesting no blanks at the start of a list.
-[ 1]
-[ for x in y {}]
+[1]
+ [for x in y {}]

Fixes: #1023

FYI this needs a rebase too, I assume merging your other cue/format fix caused minor conflicts.

I've moved this change to gerrit.
https://review.gerrithub.io/c/cue-lang/cue/+/1171302


add samp_taxon_id slot to MIxS schema

Add samp_taxon_id to MIxS schema mixs.yaml, imported by nmdc.yaml. In nmdc.yaml assert the newly added samp_taxon_id slot on Biosample class.

@turbomam good feedback. I've enhanced the biosample test.


[merge] revert arrows aren't aligned properly sometimes

Sometimes, the arrows are placed in weird places. See https://jsfiddle.net/Lyeykf0w/
In the left side, the 2nd arrow is for the bottom diff but is not aligned anywhere near the bottom diff.
In the right side, the 2nd arrow is for the bottom diff but is placed somewhat higher than one might expect.

I forgot to note that if you further click on the bottom left revert arrow, the arrow's new position also seems odd.

Could you see whether attached patches help?

As far as I can tell, it looks much better, thanks!


Migration engine should remove dead code, not comment it out.

Currently, if the migration engine discovers dead code, it comments it out by inserting /* and */ strings.  This is problematic because:

It leads to an ugly migration result that the user must clean up
It doesn't do the right thing if the dead code previously contained /* style comments.

What we should do instead is simply delete the dead code; the user can use their source control tool (or some other migration tool UX that we provide) to see what is being deleted, and un-do the deletion if desired.

We received pretty strong feedback in the UX studies that we should implement this.

This is mostly obselete due to #41231, which avoids trying to delete unreachable code altogether.  However, we are still trying to change @required to required by commenting out the @ symbol, so there's still some work to do.

As of 1c7fe7196dd7a542f7f91cd6ea2ac81e1a13146c, the null safety migration tool has been removed from active development and retired. No further work on the tool is planned.
If you still need help, or you believe this issue has been closed in error, please feel free to reopen.


[Tracking] Falco 0.36.2

Falco 0.36.2 Release
Will keep this issue updated with the current status and progress.
Date
Release date: 2023-10-27
Release Steps
The process is described in this document.
## Release Blocking PRs and issues
- [x] [Falco 0.36.2 milestone](https://github.com/falcosecurity/falco/milestone/36)
- [x] [Libs 0.13.4 milestone](https://github.com/falcosecurity/libs/milestone/31)

## External dependencies
- [x] falcoctl -> Not needed
- [x] libs https://github.com/falcosecurity/libs/releases/tag/0.13.4
- [x] drivers -> Not needed
- [x] rules -> Not needed
- [x] plugins -> Not needed
- [x] charts -> https://github.com/falcosecurity/charts/pull/570

Action Items

[x] Pre-Release

[x]  Milestone -> https://github.com/falcosecurity/falco/milestone/36
[x] Code freeze in falcosecurity/falco
[x] Open release branch in Falco and protect it
[x] See External dependencies task lists
[x] Prebuilt drivers publish
[x] Code thaw in falcosecurity/falco

[x] Release

[x] Github Release in the Falco repo

[x] Website

[x] Create new snapshot

[x] Protect snapshot branch

[x] Merge all necessary PRs

[x] Announcements

[x] mailing list
[x] blog post -> https://github.com/falcosecurity/falco-website/pull/1191
[x] slack channel

[x] Post-Release

[x] Update supported fields and syscall documentation
[x] Archive community call meeting notes -> https://github.com/falcosecurity/community/pull/201
[x] Changelog

Miscs
/milestone 0.36.2
/assign @FedeDP
/kind documentation

A new bug in libs was discovered and promptly fixed: https://github.com/falcosecurity/libs/pull/1449
A new tag 0.13.4 was made!

Falco 0.36.2 tag is live; pipeline did start: https://github.com/falcosecurity/falco/actions/runs/6665609097

Release is out, everything done.
/close

@FedeDP: Closing this issue.

In response to this:

Release is out, everything done.
/close

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.


Use barrel in civility effects

Codecov Report

Merging #5 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master       #5   +/-   ##
=======================================
  Coverage   99.65%   99.65%
=======================================
  Files         163      163
  Lines        1175     1175
  Branches       44       44
=======================================
  Hits         1171     1171
  Misses          4        4

Impacted Files
Coverage Δ

src/app/civility/state/effects/civility.effects.ts
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


feature: better error message when branch does not exist

Did you check the docs?

[X] I have read all the lazy docs

Is your feature request related to a problem? Please describe.
I just saw this error message for one of my plugins after running :Lazy check and had to look in the code to see what it means:

Describe the solution you'd like
A better error message, e.g. branch "preview" does not exist
Describe alternatives you've considered
None
Additional context
For some reason, when running :Lazy update and the branch does not exist, the default main branch is selected. That seems like a minor inconsistency between the behaviors of check and update to me.

Thank you!


Kademlia: use swarm.MaxBins

minor cleanup, as a result of the previous PR that introduces MaxBins to swarm package. Pending other merge

Sure, that's why I mentioned Pending other merge in the description :)


feat: dynamically set curriculum href

Closes: #101
Closes: https://github.com/freeCodeCamp/freeCodeCamp/issues/40372
Note: This directs Campers to https://freecodecamp.org/<locale> NOT https://freecodecamp.org/<locale>/learn.

Thanks for the pr.
Deutsch  should also map to https://www.freecodecamp.org/german/

@ahmaxed Thank you for catching that. I assumed we did not have German for /learn yet.

Not sure if we need chinese-traditional, but we could double check with the localization team and adjust the settings accordingly.
Thank you @erikasby and @ShaunSHamilton


Add some missing common functions

We have max. We don't have min. Or avg, sum, etc. We should add these. They should not use the sql_function! macro, as they need to accept more than one argument type. min should have the same bounds as max. Others might need new bounds. I'm fine with just naming the trait something like CanBePassedToAvg or something, if there's not a good generic name. If you ever need to find the types which can be passed to a function in postgres, just open up psql and run this query:
select typname from pg_type where oid in (select proargtypes::text::oid from pg_proc where proname = 'sum');

Just to be clear, even though there's a PR that shows up here that says "Add sum function", we still don't have a sum function because @mfpiccolo is a lazy jerk.


[BUG/Help] 怎样测评耗时

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
关于各种语言生成模型，该如何测评推理速度呢？有没有相关的参考？
Expected Behavior
No response
Steps To Reproduce
bash time_inference.sh
Environment
- OS:
- Python:
- Transformers:
- PyTorch:
- CUDA Support (`python -c "import torch; print(torch.cuda.is_available())"`) :

Anything else?
No response

Duplicate of #24


2.53

Hotspot Question
For each of the following statements, select Yes if the statement is true. Otherwise, select No.
NOTE: Each correct selection is worth one point.

Each Azure subscription can contain multiple account administrators.

Y
Y
N


not working (conflict with clang-formatter in cpptool package)

Hi,
I have vscode cpptools installed, and seems it uses clang-format by default, and I have no idea how to resolve the conflict and get the astyle working.
Thank you.

Hi pollow,
It seems that currently vscode has no configuration for choosing language code formatting provider, but I notice that the latest installed provider will be the default one.
You can uninstall and install astyle again to fix the problem for now. At the mean time, I'll looking for other solutions, e.g. provide different context menu option or keymap to trigger astyle formatter.
Thank you.

Thanks for your reply and your great work! I'll try to work around to solve
this problem.
On Mon, Oct 31, 2016, 10:23 PM Chieh Yu<EMAIL_ADDRESS>wrote:

Hi pollow,
It seems that currently vscode has no configuration for choosing language
code formatting provider, but I notice that the latest installed provider
will be the default one.
You can uninstall and install astyle again to fix the problem for now. At
the mean time, I'll looking for other solutions, e.g. provide different
context menu option or keymap to trigger astyle formatter.
Thank you.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/welkineins/vscode-astyle/issues/3#issuecomment-257474888,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABZHbgJaoLOflpSaDID_d5K_AjIvz-IWks5q5qKFgaJpZM4KkgU-
.

With the latest version of you can disable the CLangFormat using:
Added option to disable clang-format-based formatting provided by this extension via "C_Cpp.formatting" : "disabled"
I've not tried it.

Thanks to @BillDenton to provide a good solution for this issue. :)
I studied the VS code API, it's not support dominating formatter for one language. If you have multiple formatter with same match score, the later loaded one would be used.
Reference

https://code.visualstudio.com/Docs/extensionAPI/vscode-api

I don't think the disable works in C++/C extension. I've reported an issue https://github.com/Microsoft/vscode-cpptools/issues/469. So far I haven't managed to get Astyle formatting even after disabling the whole of the C++/C extension.

I was using the wrong way of disabling CLangFormat. It should be:
"C_Cpp.formatting": "disabled",
Now just got to get some Astyle options that I'm happy with!


Doc [Minor]

@lpgeiger
Do we want to say "un-parallelized" ? --test-threads 1 will run all tests on main thread, disables multi threaded testing.
..
# Tests can be parallelized. If you suspect side-effects. Run with
> cargo test <keyword> -- --test-threads 1
..

https://github.com/OLSF/libra/wiki/Testing-0L

And broken links:
1
Faster Sync: sync the state from a later waypoint [Link]

https://github.com/OLSF/libra/wiki/Syncing-the-State-to-Latest-State
2
https://github.com/OLSF/epoch-archive/tree/main/state_ver_41315058.6168

https://github.com/OLSF/libra/wiki/Syncing-from-an-advanced-Waypoint

Patched.
#1 is now https://github.com/OLSF/libra/wiki/Restore-DB:-Syncing-from-an-advanced-Waypoint
#2 is deprecated


makes is_win more readable and idiomatic

This is just a little more compact and easy to read. Jankily tested from the interpreter:
>>> from ccmlib import common
>>> import inspect
>>> inspect.getsourcelines(common.is_win)  # check that I'm using my new implementation
(['def is_win():\n', '    return sys.platform in ("cygwin", "win32")\n'], 215)
>>> # I totally am
...
>>> common.is_win()
False
>>> import sys ; sys.platform = 'cygwin'
>>> common.is_win()
True

+1


Fixed possible bug in ForwardingLayer.getActivations()

In the previous implementation, thisSize will always have a value of 1. As a result, only one neuron from the current layer is added to the Vector.
The new implementation now makes use of functionality that is already implemented in getNeuralInput().

Fix looks valid, please add a test.

@censlin Please provide a test for this. I'll merge it in as soon as your branch gets the commit of the test case :)


Command failed due to signal: Segmantation fault: 11

I encounted this error in Xcode 8 using Swift 3 compitable version of 2.0. I cleaned derived data or re-install pod from 1.1.0 cocoapods but it don't work. Any suggestion?
Thanks in advance

This is a bug in Swift compiler and the work around right now is to use Fast, Whole Module Optimization in the debug configuration. Please refer image below.

Thank you it solved.

Added to Readme 👍🏻


Added Decimal-Roman Calculator

Fixes Issue🛠
Closes #1757
Description👨💻
Added a decimal roman calculator which changes given decimal number to roman number and vice-versa.
Type of Change📄

[ ] Bug fix (non-breaking change which fixes a bug)
[x] New feature (non-breaking change which adds functionality)
[ ] Style (non-breaking change which improves website style or formatting)
[ ] Documentation (non-breaking change which updates or adds documentation)
[ ] Responsiveness (non-breaking change which improves UI/UX on different screen sizes)

Checklist✅

[x] I am an Open Source contributor
[x] I have performed a self-review of my code
[x] My code follows the style guidelines of this project
[x] I have commented on my code, particularly in hard-to-understand areas

Screenshots/GIF📷

@Rakesh9100  This PR is reviewed & can be merged.


[FP]: New WildFly application server version reports 180 vulnerabilities day one

Package URl
<EMAIL_ADDRESS>CPE
cpe:2.3:a:redhat:wildfly:31.0.1:::::::*
CVE
CVE-2022-0866
ODC Integration
{"label"=>"CLI"}
ODC Version
9.0.9
Description
Running the dependency check on the new version of the WildFly application server, the report shows a staggering total of 180 vulnerabilities found.
https://github.com/wildfly/wildfly/releases/download/31.0.1.Final/wildfly-31.0.1.Final.zip
Most of the archives mentioned in the report are in their latest versions, so logically it shouldn't be vuln.
What's more, the same CVE-2022-0866 vulnerability appears in 114 different archives. It is therefore impossible to know which archive is actually affected by the vulnerability.
dependency-check-report.zip

<EMAIL_ADDRESS>

Affects all versions of JBoss EAP from 7.1.0 and all versions of WildFly 11+ when Elytron is enabled.

The example you have given is not a false positive.
RedHat products get a lot of CVE reports, and quite often they fix them for their customers with configuration changes, instead of patching the open source components. I'm afraid you have to manually evaluate those 180 vulnerabilities to see if they affect you, and fix, suppress, or ignore them yourself.


Feature Request: Add Brazil support

Amazon Music in Brazil uses domain music.amazon.com.br

Hello @eduardoweiland. @a-m-s has added support for Brazil. I'm still struggling with tiliado/nuvola-flatpaks#1. I will publish new flatpaks once it is resolved.

Thanks @fenryxo and @a-m-s

New flatpaks have been published.


default attribute added

a very useful option to not be redundant in your json :D

Thank you for your contribution!
Do you have an example on where this feature is useful?
I think changing the behavior, when default is used as property, might be unexpected. (Not to mention, this will introduce a breaking change).
We could use something like __default instead of default to differentiate it a little more, but I don't know yet how useful this feature is.

__default sound good aswell. In my case I want use ant design, and bootstrap (or other like that) with the same variables, but some case exits, for example: --blue and --blue-secundary.
{
blue: #color1,
blue : {
primary: #color2
}
} // --blue , --blue-primary
with the default if
{
blue : {
__default: #color1
primary: #color2
}
}  // --blue , --blue-primary

other example
...
primary: {
light: lighten(${blue.primary} 40%),
opacity: rgba(${blue.primary}, 0.2),
default: blue.primary
},
secondary: {
light: lighten(#ac39d4, 44%),
opacity: rgba(#ac39d4, 0.5),
default: #ac39d4
},

I can see how this is useful, especially with the second example.
I added a few notes about this and a test case.
Will merge this shortly.
Note: You can surround your code examples with triple backticks (```) on GitHub to make it a little more readable :)

oh ty for your tips 👍
console.log("...learning how to use this")
> "...learning how to use this"


bad_alloc

Hello, I'm trying to use anise_basil to assemble some breakpoints around a duplication.  I generated a basil.vcf file and am trying to run the anise step where I get the following error after it assembles 8/123 of the sites:
00:11/00:02:49]terminate called after throwing an instance of 'std::bad_alloc'
what():  std::bad_alloc
When I run it with the -v option, I get this error description:
Building Consensus...
Building alignment graph
fragments: 0
seqs: 295
progressive alignment...
build AG..
AG
numVertices = 295
numEdges = 0
componentLength.size() = 295
numComponents = 295
terminate called after throwing an instance of 'std::bad_alloc'
what():  std::bad_alloc
Aborted (core dumped)
I'm running it on a small (6 Mb) section of chr2, and the basil step detects 124 potential breakpoints, all of which I use in the anise assembly.
The machine I'm running it on has 16G, and anise seems to be using only a little over 1G, so it doesn't seem to be running out of memory.
Please let me know if you would like me to send my input files.
Thanks!

I think this was caused by using the -M flag to bwa.
If the problem still persists, please reopen.


Option to move command mode

I'd like the option to move the command mode down like Vim's standard location, but left the original as I'm sure that's how you prefer it.

Thank you for the PR.
I'm now working on the migration to crossterm from termion, and that may affect some of this PR.
After I merge the migration, will check this PR.

I'll likewise try to find time to port it over to your crossterm branch. Should be fairly easy I would think.

crossterm branch merged to develop branch, so if you'd like, please create PR to develop.


Disabled queries (enabled: false) get affected by invalidateQueries (refetchType: "all") when they are inactive (not being rendered anymore)

Describe the bug
This is similar to an old bug: https://github.com/TanStack/query/issues/3642
But now this only happens, if the disabled query isn't actively rendered anymore and TanStack Query still has it in the cache.
Then the query will be executed even though it was never enabled.
Your minimal, reproducible example
https://codesandbox.io/p/sandbox/brave-khayyam-62yx75
Steps to reproduce
In the CodeSandbox you have a button to invaldate the Query and another button to toggle if the query is being actively rendered.
If it's not actively being rendered, you can see that the queries get fetched on invalidation even though it was always disabled.
Expected behavior
Disabled queries shouldn't be fetched on invalidateQueries when they never were enabled in the first place.
How often does this bug happen?
Every time
Screenshots or Videos
https://share.cleanshot.com/Msh4Bk4q
Platform

OS: macOS
Browser: Arc (Chrome)
Version: 1.61.2 (54148) (Chromium Engine Version 129.0.6668.71)

Tanstack Query adapter
react-query
TanStack Query version
v5.56.2
TypeScript version
No response
Additional context
No response

oh wow. problem is that enabled is an observer level property. As soon as there is no observer anymore, a query cannot be “enabled” or “disabled”. This isn’t a problem if you use the standard invalidation, but if you bypass it to refetch everything, this will hit too.
wild that this was never reported before. need to think about what we can do here (if anything) because a query cannot (per definition) be enabled/disabled - only an observer can.

ah I see! .. my initial thought was: why put a query into the cache when it wasn't enabled in the first place?
there are probably arguments against this, which I'm not thinking about atm, but wouldn't it be okay to not even keep disabled queries in the cache (because they weren't even executed) .. which would then also fix this issue?

While most cases will have a query either enabled or disabled, you can temporary enable / disable a query. For example, you can have a list query that you want to disable while a dialog is open so that it doesn’t refetch until the dialog is closed. So it’s not that black & white.
However, I think the answer is simply skipToken. When enabled is passed, we lose that information once the observer unmounts. But skipToken is passed in place of the queryFn. So I think we can consider a query do be disabled if it has a queryFn that is equal to skipToken. This check would need to go here:
https://github.com/TanStack/query/blob/50315acbb39d0c18dbb8343bfd2e13c1ac588b6f/packages/query-core/src/query.ts#L258-L260
would you like to maybe contribute that + a test case?

Would this get closer to a deprecation of enabled in favor of skipToken?

yeah maybe. I’ll think about it

wow, thank you very much for the super fast fix @TkDodo ❤


Adding a separate section for user to store their favourite icons.

Adding  a section of favorite icons would help the user to find their "used before" or "liked" icons easily. If you are okay with the idea, please assign me the issue I would like to work on this. gssoc'23

Although we don't want to handle any database for the "liked" icons, you are free to explore if you want to handle that using local storage.
During PR please mention the screenshot of what you have done.
Assigning you this issue.

Closed this due to inactivity.


Change to svelte env access #1367

closes #1367
For the Submitter
Tasks

[x] Make sure "closes (issue number)" is written close to the top
[x] Attach the enhancement label for update tasks
[ ] Make sure CI completes successfully
[ ] Check everything written in the PR
[ ] Fix the issues pointed out by AI
[ ] Add reviewers
[ ] On Slack, mention the PR reviewer, write your thoughts on the issue, and inform them that the PR has been created

For the Reviewer
Before Reviewing

[ ] Write which PR you are reviewing on Slack

Checks

[ ] Each task for the related issue are written and in checkboxes
[ ] The branch name is close to the issue name
[ ] CI completes successfully
[ ] Changes include only what is necessary for this PR
[ ] Necessary unit tests are implemented
[ ] Necessary E2E tests are implemented
[ ] Perform functionality checks
[ ] If there are any problems, request revisions from the PR submitter and mention them on Slack

After Reviewing

[ ] Leave an approval comment (e.g. LGTM)
[ ] On Slack, mention the PR submitter and a project admin, write which PR you have reviewed, and ask them to merge the PR

こちらは、対応せずに close いたします。


Segmentation fault.

(gdb) bt
#0  0x000000000043630c in el::LogMessage::level (this=<error reading variable: Cannot access memory at address 0x7fa0eebb43d8>,
this@entry=<error reading variable: Cannot access memory at address 0x7fa0eebb43e8>) at /home/combile/src/installer/build/ds-nsfocus/internal/debug_log/easylogging++.h:2460
Cannot access memory at address 0x7fa0eebb43e8

Hi! @lazy-wangxl I'm a beginner, I wanna try fixing this issue, Can u please guide me while fixing this issue, step by step if u can
Thank you!


buck2 Documentation

This is a general tracking issue patching holes in the buck2 documentation regarding background information, general use, rule writing, etc. Some high level goals:

[ ] API docs for the prelude

providers
rules

[ ] buck2 for bazel users
[ ] glossary
[x] quick start with and without the prelude
[ ] developer environment setup w/ extensions
[ ] toolchains and toolchain rules

I'd be good to have examples of projects and roles other than cpp's. I'm interested in Java in particular, but there could be others :D


WIP

Lifecycle Hooks
Changes
How I tested this
Notes
Checklist

[ ] PR has an informative and human-readable title (this will be pulled into the release notes)
[ ] Changes are limited to a single goal (no scope creep)
[ ] Code passed the pre-commit check & code is left cleaner/nicer than when first encountered.
[ ] Any change in functionality is tested
[ ] New functions are documented (with a description, list of inputs, and expected output)
[ ] Placeholder code is flagged / future TODOs are captured in comments
[ ] Project documentation has been updated if adding/changing functionality.

This is a WIP, not ready for feedback or comments

TODO prior to release:

[ ] Sort out commits -- smaller pieces
[ ] Ensure sphinx docs is good
[ ] Add a few pre-canned examples, first-class

[ ] PrintlnHook
[ ] PermissiveTypeCheckerHook
[ ] ???

I haven't gone over this with a fine toothed comb, but I have run the branch locally and it seems to not introduce any regressions.

I'm doing a detailed pass to form another review


Open Designer is not working for existing Logicapps project

Describe the issue
1 Open Folder.
2. Select the logicapps project folder from local
3. right-click on workflow.json
4. Designer gives error. "Error in determing project root"
Are you using new designer or old designer
New Designer (With action panels)
Screenshots or Videos

Browser
Chrome
Additional context
No response

Hi @sukhmeetb thanks for raising this issue. When users open the designer usually, we try to find the logic app project root based on the following 3 things

There is a local.settings.json file in the root folder
There is a host.json file in the root folder
The host.json file in the root folder has a extensionBundle property like this one:
"extensionBundle": {   "id": "Microsoft.Azure.Functions.ExtensionBundle.Workflows",   "version": "[1.*, 2.0.0)" }

If your logic app project is in a multi-root workspace usually a dropdown menu will be prompted to enter the right logic app project root.
Is there any chance you can verify if none of the prior 3 requisites are met?


Adding settings for account deactivation

Let me know how I can better clean this up and issues I might have not addressed -- I'm sure there are plenty. Regarding #15.
Also, I added a .nvmrc file so it automatically switches my node version. I can include it since Winds only currently runs with node v 7.

Any feedback on this?


Create Intel(R) Core(TM) i5-1038NG7 CPU @ 2.00GHz.md

Before submitting a Pull Request, please test your code.
If you created a new created a new feature, then create the unit test function, too.
# COMPILE THE BACKEND SERVER
npm run build

# RUN THE TEST AUTOMATION PROGRAM
npm run test

Learn more about the CONTRIBUTING

thanks :) it's from 2020 intel MacBook Pro 13


[docs] Installation for Windows docs improved

Description
Improve the documentation for the installation of TornadoVM on Windows 10/11
Problem description
n/ a.
Backend/s tested
Mark the backends affected by this PR.

[X] OpenCL
[X] PTX
[X] SPIRV

OS tested
Mark the OS where this PR is tested.

[ ] Linux
[ ] OSx
[X] Windows

Did you check on FPGAs?
If it is applicable, check your changes on FPGAs.

[ ] Yes
[X] No

How to test the new patch?
Link to the documentation.

All comments resolved.


General improvements on translation

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Closed and Reopened this PR to attempt to resolve a specific Travis build failure.

@lugDroid Thank you for  creating this PR.  However, since it was created, the underlying document has changed.  Can you please review and resolve the current merge conflicts?  It could be the changes to the underlying document are so significant that your changes are no longer relevant.  If the latter is the case, it might be easier to just close this PR and create a new PR with any changes you still feel are important to make.
If you need any assistance or have questions about resolving the merge conflicts, please reply back or contact someone in the Contributors Room on Gitter.
Thank you.

We are restructuring a lot of the code and learning resources referenced in your pull request, so for now I am going to go ahead and close this pull request. Thanks again for opening this pull request and for contributing to the community.


Missing Quartz recipes

Cant make Diorite or Granite crafting recipe with Rosetite crystal. Since Rosetite is used as subtitude of quartz this should possible. Also loving the mod :) thanks for your work

Hey, thanks for playing my mod! I originally intended for Roseite Crystal to replace Quartz only for redstone-related recipes. However looking at again, it definitely would make sense to make the other recipes compatible with Roseite as well. I'll be putting this in the next update, thanks for the suggestion!


v2: ctx.WindowSize() panics if used in Model.Init()

Presumably because the window size hasn't been queried for yet (or maybe there's a race condition).

Closing at this is no longer relevant.


Underground Now Requires HTTPS connection?

Hello,
I have just got a M5Stack Fire and love your app. However I can't get the Weather app to function, am I correct in thinking that Wunderground only allow https connections now? If I put the correct string in a browser address with https://...... with my api key etc I can get the data but don't get anything using your app. Please comment.
Regards
Phil

AFIK weatherunderground is toast:
https://apicommunity.wunderground.com/weatherapi/topics/end-of-service-for-the-weather-underground-api
@PhilG1300 @botofancalin


Distance update

Note that I also changed regularize_covariance and compute_shared_covariance to accept a cov_type argument.  They were previously using self.cov_type, which is the training cov_type, but when computing the covariance for prediction or OOD, we want to pass those types

@nukularrr Thank you for running the formatter Allan.  I wasn't sure how to do that


Redefining baseDN from domain name instead of KDC

Hey there, this small PR fixes the baseDN calculation for the ldap protocol. Currently, it's based on a split of the value provided for the KDC with the --kdcHost argument. Actually it should be based on the domain name either supplied with the -d argument or automatically fetched by CME.
This also fixes the bugs some could face when trying to use CME with LDAP with kdcHost set to its IP address instead of its hostname (usually done when facing a foreign domain or when the DNS configuration is not correctly pushed through DHCP).

Fixed by #642 argparse :)


Successful Registration page not loading

The successful registration page does not show up after a successful registration.

solved


Environment brightness, contrast, saturation restore.

Not working color correction using a 3D texture.

Co-authored-by: CptPotato
Co-authored-by: QbieShay
Restored brightness, contrast, saturation adjustments.
Color correction is not working. Not sure why.
Bugsquad edit: This closes https://github.com/godotengine/godot/issues/41257.

Test project for future iFire.
Env Adjustments Game Project.zip

Setting the Color correction new sets the screen grey and still doesn't work. Revised with Reduz's comments.

Will rebase.

There's another patch incoming.

Sunset lut!

I though the design was to use 3D luts in master and I wasn't sure of a design that allows both.

LUT only texture3D and does not support texture2D
lut already experiment the result well --> https://youtu.be/FacL-08WpP8
already tried the gimp editor handle colors well import godot texture3D --> https://youtu.be/4lYdYqshSmY

I don't think it's the responsibility of this pr to write this conversion. I imagined that there was a way to convert Texture2D into a Texture3D using default settings of 8 tiles width and 8 tiles height.
A restart is required to convert a Texture2D to a Texture3D. A Texture2D that is 2D can be converted to a Texture3D of 64x64x1 which is 1 Tile width and 1 Tile depth compared to a regular Texture3D that is a 2D image converted to 64x64x64. Then, bi-linear interpolation from the 3D Material should function. I don't have any examples of a 2D LUT to test.

The current workflow for users is that they can create a LUT using a single gradient texture. While it is clear that a 1D LUT is extremely limiting, its what every Godot game uses and it is very easy for users to get something working. They just add a gradient texture and can have results in seconds.
Switching to a 3D LUT breaks every game that is using the 1D LUT and it makes adding a LUT considerably more difficult as the LUT needs to be generated outside of the engine.
While I understand that the 3D LUT is much more powerful and expressive (and we need support for them), there has been no discussion about breaking the previous workflow, in particular, there has been no proposal weighing the pros and cons.
That being said, adding support for texture2Ds and texture3Ds would be easy and would avoid creating breaking changes, so that is what I prefer. Otherwise, we need to find out from users if they are okay losing the current LUT approach.

The workflow for conversion of a 2D LUT would be to turn a png into a 1 tile width and 1 tile height 3d texture, which is doable.
On a related note, I had a hard time finding anyone providing examples for the 2D LUT workflow in Godot Engine.
Did you mean 2D LUT? A 1D LUT is a line. I think you can interpolate a line into a cube.

The current approach in (3.2) accepts a Texture2D which represents a 1D LUT.
I am not suggesting a conversion tool (although a conversion tool is a viable idea). I am suggested allowing users to use the current workflow and the new 3D texture workflow.

1D LUT example below screenshot? right

RGB Table 16х1 size

He means this.

Example in 3.2

As you can see here an 1D LUT can be a 3D LUT when you set tile width and tile height to 1.

I'm think 1D LUT better use a gradient no textures needed :)

@fire I guess the important thing is just allowing users to continue using GradientTextures. So, you can convert it to a 3D Texture under the hood if you want, or you can adapt it so the shader accepts either.

Got the Gradient texture to dynamically change. Needs testing.

good no problem

For others searching for what happens if you have a 3D LUT that is kept as a texture2D., the result looks spectacular.
The original is a 2D texture and it's supposed to be sliced 8 weight and 8 height into a 3D texture.

We discussed this and we decided the result is just a misuse of the feature, but we can't distinguish between the 1D and 3D LUT encoded into the same png.

LUT 3D 16x1 will future replace the old 8x8 ?

Go ahead.
On Nov 22, 2020, at 12:52 AM, Clay John<EMAIL_ADDRESS>wrote:

@clayjohn commented on this pull request.
In servers/rendering/rasterizer_rd/shaders/tonemap.glsl:

@@ -23,7 +23,8 @@ layout(location = 0) in vec2 uv_interp;
layout(set = 0, binding = 0) uniform sampler2D source_color;
layout(set = 1, binding = 0) uniform sampler2D source_auto_exposure;
layout(set = 2, binding = 0) uniform sampler2D source_glow;
-layout(set = 3, binding = 0) uniform sampler3D color_correction;
+layout(set = 3, binding = 0) uniform sampler2D color_correction_1d;
+layout(set = 4, binding = 0) uniform sampler3D color_correction_3d;
Mobile devices can only have 4 uniform sets.

The easiest way for this to work is to use a new shader version as we discussed on IRC. A shader version is created by adding an ifdef statement.
If you want I can make a PR to your branch, just let me know.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

Used ClayJohn's revision.
void Environment::set_adjustment_color_correction(Ref<Texture> p_color_correction) {
	adjustment_color_correction = p_color_correction;
	Ref<GradientTexture> grad_tex = p_color_correction;
	if (grad_tex.is_valid()) {
		if (!grad_tex->is_connected(CoreStringNames::get_singleton()->changed, callable_mp(this, &Environment::_update_adjustment))) {
			grad_tex->connect(CoreStringNames::get_singleton()->changed, callable_mp(this, &Environment::_update_adjustment));
			if (grad_tex->get_gradient().is_valid() && !grad_tex->get_gradient()->is_connected(CoreStringNames::get_singleton()->changed, callable_mp(this, &Environment::_update_adjustment))) {
				grad_tex->get_gradient()->connect(CoreStringNames::get_singleton()->changed, callable_mp(this, &Environment::_update_adjustment));
			}
		}
	}
	Ref<Texture2D> adjustment_texture_2d = adjustment_color_correction;
	if (adjustment_texture_2d.is_valid()) {
		use_1d_color_correction = true;
	} else {
		use_1d_color_correction = false;
	}
	_update_adjustment();
}

Also restored hard-coded gradient texture updating support. Probably want to make a note that if there's no hard coding, an editor restart is needed.

Looks ok to me

Note that the "Co-authored-by:" fields for @CptPotato and @QbieShay don't seem to work, I think you need to include their email (check their recently used name + email in git log).

@akien-mga Done.

Thanks!


Error OpenSSL Shutdown while in init - cURL - MSSQL

Environment
Php version: 8.2.19
PDO_SQLSRV: pdo_sqlsrv-5.12.0
Microsoft ODBC Driver version: <IP_ADDRESS>-1
MS SQL Server version: 2022  lastest (docker image mcr.microsoft.com/mssql/server:2022-latest)
Ubuntu via Docker for Mac: Ubuntu 22.04.4 LTS (Jammy Jellyfish)
OpenSSL version: OpenSSL 1.1.1f  31 Mar 2020
Problem description
The following code:
<?php

$pdo = new \PDO('sqlsrv:server=tcp:' . getenv('SQL_HOST') . ',1433 ; Database = ' . getenv('SQL_DATABASE') . ';TrustServerCertificate=yes;', getenv('SQL_USERNAME'), getenv('SQL_PASSWORD'));

$pdo->beginTransaction();

$curlResource = curl_init();

$curlOptions = [
    CURLOPT_SSL_VERIFYPEER => true,
    CURLOPT_CAPATH => '/data/ca',
    CURLOPT_CAINFO => '/data/ca/serverCa.pem',
    CURLOPT_RETURNTRANSFER => true,
    CURLOPT_URL => getenv('API_URL'),
];

foreach ($curlOptions as $option => $value) {
    curl_setopt($curlResource, $option, $value);
}

$responseContent = curl_exec($curlResource);
$errorNumber = curl_errno($curlResource);
$errorMessage = curl_error($curlResource);

curl_close($curlResource);

if ($errorNumber > 0) {
    echo 'Curl error: ' . $errorNumber . ' - ' . $errorMessage. "\n" ;
    $pdo->rollBack();
}

echo $responseContent;

Resulted in this output:
Curl error: 60 - SSL certificate problem: unable to get local issuer certificate

PHP Fatal error:  Uncaught PDOException: SQLSTATE[08S01]: [Microsoft][ODBC Driver 18 for SQL Server]SSL Provider: [error:140E0197:SSL routines:SSL_shutdown:shutdown while in init] in /data/test.php:30
Stack trace:
0 /data/test.php(30): PDO->rollBack()
1 {main}
  thrown in /data/test.php on line 30

But I expected no SQL error, only the cURL error :
Curl error: 60 - SSL certificate problem: unable to get local issuer certificate // no SQL error

The error only happens when the curl CA verification reports a certificate not allowed.
The error does not happen when I remove TrustServerCertificate=yes in the dsn.
The error does not happen when CURLOPT_SSL_VERIFYPEER is set to false.
It's seems that the Mssql driver call openssl function SSL_shutdown() while SSL_in_init() send true.
The SSL_in_init() seems to return true only when the curl CA verification reports a certificate not allowed.
I don't know if php curl implementation does not close the connection correctly or if the MSSQL driver should check if SSL_in_init() returns true before calling SSL_shutdown(), but there is something wrong here.
I opened an issue on PHP, but they suggested that I open an issue here (https://github.com/php/php-src/issues/14230).

It seems likely that curl also loads OpenSSL, just like the ODBC Driver and there is probably some interference there. I'm having a bit of trouble getting php-curl to work to verify, but the fact that error goes away when CURLOPT_SSL_VERIFYPEER is set to false, strongly suggests that curl's use of OpenSSL is involved. I will investigate further.
By the way, do you know which version of OpenSSL is used there?

thanks ! Yes we used OpenSSL 3.0.2 15 Mar 2022 (Library: OpenSSL 3.0.2 15 Mar 2022)

Can you strace a successful run and the one with the error? Something like strace -o output.txt -f php file.php if it's run directly.
I can now run the repro code, but I don't get the same error, since I don't have the API URL setup that would return "certificate not allowed", if I understand the setup correctly.

The error is triggered before the API call. I got the error even when https://www.google.fr is set as the API URL. Are you sure you are using the same set up ?

straceErrorScenario.txt
straceNoVerifyPeer.txt

Great, thanks. I'll see what I can find in those

Hi, turns out this is a bug in libcurl. More details here. They have fixed it for curl>=8.3.0. After upgrade the driver no longer throw an error.


Skip AbstractProvider when generating provider list

In some cases, when filesystem iterator returned AbstractProvider.php after USA.php, provider list generator returned AbstractProvider as the provider for USA, as abstract provider has the ID constant set to US (I'm unsure why it has this constant at all). The mechanism is simple: loop checks USA.php, adds 'US' => 'USA' to the array, then checks AbstractProvider, and overwrites the USA provider by adding 'US' => 'AbstractProvider' to the output.
To overcome this issue, I've expanded the condition checking if the class is a proper provider by checking if it's a subclass of AbstractProvider. That's why AbstractProvider won't be taken into consideration at all.

Thank you for your PR. Can you maybe share your code or an example where this issue appears? I tried to replicate it myself with the following simple script, but couldn't see any issue:
// Require the composer auto loader
require 'vendor/autoload.php';

$providers = Yasumi::getProviders();
ksort($providers);
print_r($providers);

Output:
Array
(
    [AT] => Austria
    [AU] => Australia
    [AU-VIC] => Australia/Victoria
    [BA] => Bosnia
    [BE] => Belgium
    [BR] => Brazil
    [CH] => Switzerland
    [CH-AG] => Switzerland/Aargau
    [CH-AI] => Switzerland/AppenzellInnerrhoden
    [CH-AR] => Switzerland/AppenzellAusserrhoden
    [CH-BE] => Switzerland/Bern
    [CH-BL] => Switzerland/BaselLandschaft
    [CH-BS] => Switzerland/BaselStadt
    [CH-FR] => Switzerland/Fribourg
    [CH-GE] => Switzerland/Geneva
    [CH-GL] => Switzerland/Glarus
    [CH-GR] => Switzerland/Grisons
    [CH-JU] => Switzerland/Jura
    [CH-LU] => Switzerland/Lucerne
    [CH-NE] => Switzerland/Neuchatel
    [CH-NW] => Switzerland/Nidwalden
    [CH-OW] => Switzerland/Obwalden
    [CH-SG] => Switzerland/StGallen
    [CH-SH] => Switzerland/Schaffhausen
    [CH-SO] => Switzerland/Solothurn
    [CH-SZ] => Switzerland/Schwyz
    [CH-TG] => Switzerland/Thurgau
    [CH-TI] => Switzerland/Ticino
    [CH-UR] => Switzerland/Uri
    [CH-VD] => Switzerland/Vaud
    [CH-VS] => Switzerland/Valais
    [CH-ZG] => Switzerland/Zug
    [CH-ZH] => Switzerland/Zurich
    [CZ] => CzechRepublic
    [DE] => Germany
    [DE-BB] => Germany/Brandenburg
    [DE-BE] => Germany/Berlin
    [DE-BW] => Germany/BadenWurttemberg
    [DE-BY] => Germany/Bavaria
    [DE-HB] => Germany/Bremen
    [DE-HE] => Germany/Hesse
    [DE-HH] => Germany/Hamburg
    [DE-MV] => Germany/MecklenburgWesternPomerania
    [DE-NI] => Germany/LowerSaxony
    [DE-NW] => Germany/NorthRhineWestphalia
    [DE-RP] => Germany/RhinelandPalatinate
    [DE-SH] => Germany/SchleswigHolstein
    [DE-SL] => Germany/Saarland
    [DE-SN] => Germany/Saxony
    [DE-ST] => Germany/SaxonyAnhalt
    [DE-TH] => Germany/Thuringia
    [DK] => Denmark
    [EE] => Estonia
    [ES] => Spain
    [ES-AN] => Spain/Andalusia
    [ES-AR] => Spain/Aragon
    [ES-AS] => Spain/Asturias
    [ES-CB] => Spain/Cantabria
    [ES-CE] => Spain/Ceuta
    [ES-CL] => Spain/CastileAndLeon
    [ES-CM] => Spain/CastillaLaMancha
    [ES-CN] => Spain/CanaryIslands
    [ES-CT] => Spain/Catalonia
    [ES-EX] => Spain/Extremadura
    [ES-GA] => Spain/Galicia
    [ES-IB] => Spain/BalearicIslands
    [ES-MC] => Spain/RegionOfMurcia
    [ES-MD] => Spain/CommunityOfMadrid
    [ES-ML] => Spain/Melilla
    [ES-NC] => Spain/Navarre
    [ES-PV] => Spain/BasqueCountry
    [ES-RI] => Spain/LaRioja
    [ES-VC] => Spain/ValencianCommunity
    [FI] => Finland
    [FR] => France
    [FR-57] => France/Moselle
    [FR-67] => France/BasRhin
    [FR-68] => France/HautRhin
    [GB] => UnitedKingdom
    [GR] => Greece
    [HR] => Croatia
    [HU] => Hungary
    [IE] => Ireland
    [IT] => Italy
    [JP] => Japan
    [LT] => Lithuania
    [LV] => Latvia
    [NL] => Netherlands
    [NO] => Norway
    [NZ] => NewZealand
    [PL] => Poland
    [PT] => Portugal
    [RO] => Romania
    [RU] => Russia
    [SE] => Sweden
    [SK] => Slovakia
    [UA] => Ukraine
    [US] => USA
    [ZA] => SouthAfrica
)

I was thinking about making a dedicated test for it, but it'd require quite a lot of refactoring. To replicate this bug, you'd have to replace filesystem iterators with a method call, and inject some mock here, that would return predefined file list.
But for the sake of example, I'd hardcode it into Yasumi. Let's hardcode normal file order into getProviders method:
//        $filesIterator = new RecursiveIteratorIterator(new RecursiveDirectoryIterator(
//            __DIR__ . $ds . 'Provider',
//            FilesystemIterator::SKIP_DOTS
//        ), RecursiveIteratorIterator::SELF_FIRST);

        $filesIterator = [
            new \SplFileInfo('/Users/leafnode/var/devel/yasumi/src/Yasumi/Provider/AbstractProvider.php'),
            new \SplFileInfo('/Users/leafnode/var/devel/yasumi/src/Yasumi/Provider/Belgium.php'),
            new \SplFileInfo('/Users/leafnode/var/devel/yasumi/src/Yasumi/Provider/USA.php'),
        ];

Now let's see what's inside the array that was returned:
        $providers = Yasumi::getProviders();
        var_dump($providers);

array(2) {
  ["US"]=>
  string(3) "USA"
  ["BE"]=>
  string(7) "Belgium"
}

Everything seems fine. But now let's change the order in which filesystem returned file list:
        $filesIterator = [
            new \SplFileInfo('/Users/leafnode/var/devel/yasumi/src/Yasumi/Provider/Belgium.php'),
            new \SplFileInfo('/Users/leafnode/var/devel/yasumi/src/Yasumi/Provider/USA.php'),
            new \SplFileInfo('/Users/leafnode/var/devel/yasumi/src/Yasumi/Provider/AbstractProvider.php'),
        ];

And now the output is:
array(2) {
  ["BE"]=>
  string(7) "Belgium"
  ["US"]=>
  string(16) "AbstractProvider"
}

Why? Because the pair 'US' => 'AbstractProvider' has overwritten the pair 'US' => 'USA'.
We could fix this issue in many different ways. One of them is to sort the input iterator by filename, but that's kind of a naive approach that relies on believing that we get the input in a specific order. My idea was to make sure that we load only actual providers basing on one thing we know: it has to extend AbstractProvider class, as well as contain ID constant. I was thinking about removing the constant from the AbstractProvider, but I'm not that accustomed with Yasumi's internals, and maybe there's some requirement for that constant to be there.

Thanks for the example! Totally understand the issue now. So it looks like the FilesystemIterator may return an array of the files in an order that is not always guaranteed to be the same.
The ID constant acts merely as an identifier for the provider. I believe I used 'US' as a default for no particular reason :)
Let me have a look in a bit more detail tomorrow; however your PR looks like a good solution.

Added the changelog entry.

stelgenhof added this to the v2.0.0 milestone 8 hours ago

Why 2.0.0? It looks like something that should be fixed in earliest possible version. It does not break anything, I dont think anybody was relying on random behavior.

@norzechowicz v2.0.0 will be the next official release. This PR will be merged in the development branch, so anyone can use that branch if they like :)

merge into develop does not sounds like a solution, my project relies on that library (and probably many others as well), it's also having composer.json in pretty good shape (meaning we can update dependencies without risk of breaking things), using not stable branches is against our dependencies policy. I would really appreciate if you could at least release 1.8.1 with this fix.
I'm also not sure what changes you are planning in version 2.0.0 but I can't allow to jump from 1.x to 2.x just like that, our dependencies are set to ^1.8, I hope you understand my concerns.
If you would need any help with what I'm asking for, just let me know what I can do.

Sure, understand your concern. Let me see if I can create a 1.8 patched version. :)
Just FYI: v2.0.0 will be a new branch supporting PHP 7 only going forward. v1.8 will be the last version based on PHP 5 and will not include any new features. Not sure yet when I'm going to release v2.0.0 yet (kind of occupied with other projects).

hey @stelgenhof any chance to release 1.8.1 anytime soon?

@norzechowicz No. Do you experience any critical issues with the current release?

Yes, random behavior that can't be predicted fixed by this particular PR

@norzechowicz  Perhaps I am not seeing the entire picture here, but could you describe your situation in more detail? That may give me more context. Are there also other unpredictable results besides the one described in this PR (USA example)?
Thanks!


make standalone runnable

I refactored the code a bit to make this a standalone runnable.
now takes a parameter to turn the switch on or off (true/false) and stores the settings in a properties-file.
Thus its possible to simply copy the jar a few times to control multiple switches via cron/scripts/whatever.
Long-term goal would be to create a graal native-image out of it, but haven't looked into that yet.

Great to hear that @JohnAZoidberg ! @K4CZP3R can we merge this?

Yes! Sorry for the delay, when I get home I will look at it.

Merged!

Amazing work! Would it be possible to add a "read state" argument for the standalone cli? Would be amazing to see if some plugs in another room are on or off.


[youtube] cannot download specific youtube video, but the rest of the channel's videos work

Checklist

[X] I'm reporting a broken site support issue
[X] I've verified that I'm running youtube-dl version 2020.12.14
[X] I've checked that all provided URLs are alive and playable in a browser
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched the bugtracker for similar bug reports including closed ones
[x] I've read bugs section in FAQ

Verbose log

Y:\youtube-dl>youtube-dl -v https://www.youtube.com/watch?v=6rgx3OR-Xu0
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-v', 'https://www.youtube.com/watch?v=6rgx3OR-Xu0']
[debug] Encodings: locale cp1252, fs mbcs, out cp437, pref cp1252
[debug] youtube-dl version 2020.12.14
[debug] Python version 3.4.4 (CPython) - Windows-10-10.0.19041
[debug] exe versions: ffmpeg 4.3.1-2020-11-19-full_build-www.gyan.dev, ffprobe 4.3.1-2020-11-19-full_build-www.gyan.dev
[debug] Proxy map: {}
[youtube] 6rgx3OR-Xu0: Downloading webpage
[youtube] 6rgx3OR-Xu0: Downloading MPD manifest
[debug] Default format spec: bestvideo+bestaudio/best
WARNING: Requested formats are incompatible for merge and will be merged into mkv.
[debug] Invoking downloader on 'https://r5---sn-ab5l6n6l.googlevideo.com/videoplayback?expire=1607997518&ei=7sPXX_OWA438igTJ-IiwBA&ip=2601%3A246%3A100%3Aed%3A9582%3A65ae%3A8491%3A3c1e&id=o-ADRYxEAUNJv1G0TnSzRZ_PzUZX67Nz5Mlibn_RFZoJI2&itag=248&aitags=133%2C134%2C135%2C136%2C137%2C160%2C242%2C243%2C244%2C247%2C248%2C278&source=yt_otf&requiressl=yes&mh=eX&mm=31%2C29&mn=sn-ab5l6n6l%2Csn-vgqs7nes&ms=au%2Crdu&mv=m&mvi=5&pl=26&pcm2=yes&initcwndbps=2555000&vprv=1&mime=video%2Fwebm&ns=Y0nByxtSAY3jHPXsdRoan7AF&otf=1&otfp=1&dur=1144.009&lmt=1489605219773214&mt=1607975580&fvip=5&keepalive=yes&beids=9466587&c=WEB&n=qIbE8PJRl1a-mhNPL&sparams=expire%2Cei%2Cip%2Cid%2Caitags%2Csource%2Crequiressl%2Cpcm2%2Cvprv%2Cmime%2Cns%2Cotf%2Cotfp%2Cdur%2Clmt&sig=AOq0QJ8wRgIhAO25mWpXCDd47PQX8elzgAElOpGn84rOnUUI2q_9Uh4jAiEAuGvZpYsgfgFgN78OHFeX1svdrjohCVUO5VJ9fEQ14lA%3D&lsparams=mh%2Cmm%2Cmn%2Cms%2Cmv%2Cmvi%2Cpl%2Cinitcwndbps&lsig=AG3C_xAwRgIhAKzhLC1p-IrdTcqYefzjHKDcaPL2K2x6PgH5c82LRqlUAiEA2VDuH6ruL9vf9ChiEATD17_DiJwygXUWHFNyHpwuMKA%3D&ratebypass=yes'
ERROR: unable to download video data: HTTP Error 404: Not Found
Traceback (most recent call last):
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpwx5ikkp6\build\youtube_dl\YoutubeDL.py", line 1930, in process_info
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpwx5ikkp6\build\youtube_dl\YoutubeDL.py", line 1875, in dl
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpwx5ikkp6\build\youtube_dl\downloader\common.py", line 366, in download
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpwx5ikkp6\build\youtube_dl\downloader\http.py", line 351, in real_download
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpwx5ikkp6\build\youtube_dl\downloader\http.py", line 116, in establish_connection
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpwx5ikkp6\build\youtube_dl\downloader\http.py", line 110, in establish_connection
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpwx5ikkp6\build\youtube_dl\YoutubeDL.py", line 2248, in urlopen
  File "C:\Python\Python34\lib\urllib\request.py", line 470, in open
  File "C:\Python\Python34\lib\urllib\request.py", line 580, in http_response
  File "C:\Python\Python34\lib\urllib\request.py", line 508, in error
  File "C:\Python\Python34\lib\urllib\request.py", line 442, in _call_chain
  File "C:\Python\Python34\lib\urllib\request.py", line 588, in http_error_default
urllib.error.HTTPError: HTTP Error 404: Not Found

Description

Hello. I have updated youtube-dl to the latest version (youtube-dl is up-to-date (2020.12.14)), and have looked for existing bugs similar to this one.
This concerns one particular video from the Vapor Memory channel; the rest of the channel works.
youtube-dl refuses to download the video, stating that it received a 404, but I can clearly see that the video works in Chrome, and that it is not private or location-locked. I even verified that the video was viewable in an incognito window.
I am using this on Windows 10 x64.
Please let me know if any further information is needed.

has the specific video been deleted or is it a private video ??

the link appears to have been broken in the OP, https://www.youtube.com/watch?v=6rgx3OR-Xu0

try this -  youtube-dl https://www.youtube.com/watch?v=6rgx3OR-Xu0 -f 137+140
video will now download.

That worked. Thank you for your help.
Was the URL malformed?

Server side Youtube issue.


Clean up empty symbols in a symbol table

A compiler's bytecode file contains ~500 empty symbols.

Closed by #829.


Where is the GitHub repo for outlines-core

https://pypi.org/project/outlines-core/ points to https://github.com/outlines-dev/outlines-core/ being the github repo for outlines-core but it returns a 404. Did you forget to make it public?

Also it would be nice to pre-build the wheels for outlines-core as many systems don't have rust installed on them


ImplicitlyAnimatedWidgetState code cleanup

preparing for https://github.com/flutter/flutter/pull/154378

I believe ImplicitlyAnimatedWidget is obsolete and should be deprecated in the near future. Despite this fact, it'd be nice to clean up the implicitly animated widget logic, if only to make the upcoming PR easier to review.

test-exempt: code refactor with no semantic change