Modify search logic for get_obt_data, get_pending_fio_requests, get_sent_fio_requests

This story implements FIP-1 functionality:
https://github.com/fioprotocol/fips/blob/master/fip-1.md
and covers:

Updating get_obt_data, get_pending_fio_requests, and get_sent_fio_requests as described here: https://github.com/fioprotocol/fips/blob/master/fip-1.md#modification-to-existing-queries

PRs have been submitted and testing has begun.

A new table structure was proposed inside the fio.request.obt contract:
struct ledgerItems {
     vector<uint64_t> fio_request_ids;
     vector<uint64_t> recordobt_ids;
}

Index Table:
struct [[eosio::action]] reqledger {
      uint64_t account;
      ledgerItems transactions;

      uint64_t primary_key() const { return account; }
 }

Example:
Account: d98asauia
Transactions:

fio_request_ids[6,7,1, 55]
recordobt_ids[13,2,544,23232]

The above solution still does not fix the problem of iterating through the status table linearly. Example: A user requesting pending data needs to check a status record for each entry to see if it is still in the pending stage of the FIO workflow. This causes problems when the user has multiple transactions that have been completed before and after the record requested. To achieve an accurate more and offset value, you would have to search every record O((n+1)!). This type of structure does not support the type of FIO transactional record retrieval required. To achieve this the following proposal aims at fixing the overall architect and design of how requests and obt actions are stored on the blockchain.
Removal of old status & recordobt_info tables and combine them into a single table will be the best approach going forward:
Removed Table Structure:
struct [[eosio::action]] fioreqstss {
   uint64_t id;
   uint64_t fio_request_id;
   uint64_t status; //oversized variable for an enum
   string metadata;   //this contains a json string with metadata regarding the response to a request only.

   uint64_t time_stamp; // not used or returned on any API endpoint
};

Every request has one of these entries inside a table. This will begin to become a tedious task of interaction as the chain grows.
Add the following to the new fiotxts table:

uint64_t id;
string metadata;
uint64_t update_time;
uint64_t fio_data_type;

We would be renaming the following data memebers:

time_stamp to init_time.
-- The clear distinction between both timestamps allows for easier development.

We would still be using the following enum to classify statuses however a new status code has been added for obt and other types of blockchain actions:
enum class trxstatus {
   requested = 0,
   rejected = 1,
   sent_to_blockchain = 2,
   cancelled = 3,
   obt_action = 4,
   other = 5
};

id will be the new primary key for ALL transactions while still storing fio_request_id. This will allow for optimized searching and allow for a clear distinction between obt records. This will also ease development since the ordering of obt and request data migrations won't be required to be linearly transferred. New fio_request_ids will still be based on available_primary_key and will match all new records that are sent on the FIO blockchain.
#########################################
New request table:
struct [[eosio::action]] fiotxt {
   uint 64_t id;         // main lookup key
   uint64_t fio_request_id = 0;        // Keeping this in to look up past requests.
   uint128_t payer_fio_addr_hex;       // 'Allows for future searches on payer fio address. Needed?
   uint128_t payee_fio_addr_hex;     // 'Allows for future searches on payee fio address. Needed?
   uint8_t fio_data_type;  //Request (0) , OBT (4) , Pub address request (5) referenced from trxstatus enum
   uint64_t init_time;       // 'Rename of time_stamp
   string payer_fio_addr;    // 'Needed for return info
   string payee_fio_addr;     // 'Needed for return info
   string payer_key = nullptr;     // 'Needed for return info
   string payee_key = nullptr;     // 'Needed for return info
   uint128_t payer_key_hex;     // 'Allows for future searches on payee fio pub key. Needed?
   uint128_t payee_key_hex;

   string content;    // 'Holds metadata
   uint64_t update_time; // This will be set to the same time as init_time to allow for simple conditional checks if an update has been made to the transaction.

   //Future use
   string futureuse_string1 = nullptr;
   string futureuse_string2 = nullptr;
   uint64_t futureuse_uint64 = 0;
   uint128_t futureuse_uint128 = 0;
};

Update to the proposed ledger structure used in the transactions vector ( shown above in V1 ).
struct ledgerItems {

   std::vector<uint64_t> pending_action_ids;    // 'These are the incoming Pending Requests. Do we need a “rejected” status here, or just delete? If requestee sends funds, delete from pending and add to obt_actions_id.
   std::vector<uint64_t> sent_action_ids;    // 'These are the Sent Requests: requestor: [44]
   std::vector<uint64_t> cancelled_action_ids;    // 'Why are we keeping around cancelled requests? Do we need a separate API call?
   std::vector<uint64_t> obt_action_ids;      // 'All direct sends, and sends in response to a request. [44,78]
};

Use cases:
FIO Request:

Record is created and index is stored inside sent_actions_ids of the payee and pending_action_ids for the payer.

Record is:

approved - status is updated and index is removed from payers pending_action_ids and placed inside the obt_action_ids for both payee and payer. ( double entry for payee )
canceled - status is updated and index is removed from payers pending_action_ids but only the payee has the index stored inside cancelled_action_ids ( double entry for payee )
rejected - status is updated and index is removed from payers pending_action_ids. This record will no longer be on the payers transactional history when calling an API endpoint

Pros:

Obt records regardless of payee or payer are stored in order based on time.
Fast and efficient record retrieval.
offset and more are easily calculated based on single table queries.
Request obt request ids are easily comparable. ( if id != 0 && fio_request_id == 0 )
Remove the need for 3 tables!

Cons:

Require updates to all actions inside the fio.request contract that modify status.
Double integer entry for each record.
Add migration of another table to migration process.

This feature was delivered. Closing.
Release information
Transfer FIO Domain released in fio v2.0.0, fio.contracts v2.1.0
Transfer FIO Address released in fio v3.0.0, fio.contracts v2.3.0. For v2.3.0 it is only possible to transfer a FIO Address that has no associated FIO Request or OBT Record data. The ability to transfer any FIO Address will be released in a future contract release.


⚠ ClamAV has degraded performance

In add5b91, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 537 ms

Resolved: ClamAV performance has improved in e5cda39.


Incorrect escaping of singlequotes in view comments

Provider Version 0.35.0
Terraform Version 1.1.2
Describe the bug
When specifying a comment in a snowflake_view resource which includes a single quote character ('), this doesn't get escaped properly, resulting in a semi-broken SQL, which is getting repeatedly replaced on each terraform run.
Expected behavior
Escape the single quote, or at least detect it as an illegal character and fail.
Code samples and commands
resource snowflake_view view {
  database = "db"
  schema   = "schema"
  name     = "view"

  comment = "comment with 'quotes' is broken"

  statement  = <<-SQL
    select * from foo;
SQL
  or_replace = false
  is_secure  = false
}

Additional context
This is a bug of the annoying type, as even with the incorrect SQL structure, the view is working, just tf produces additional output logs and recreates the resource repeatedly.

This is still happening in version 0.83.0 and with different types of resources. We are unable to create a role with single quotes within the comment attribute, e.g:
resource "snowflake_role" "schema_usage" {
  name = "MKT_PROD"
  comment = "Role that grants access on the 'MKT' schema in environment 'PROD'"
}

This fails with the error:
│ Error: 001003 (42000): SQL compilation error:
│ syntax error line 1 at position 125 unexpected '' schema in environment ''.

Hey @lkorpalski-pgs
It should be the same case as in https://github.com/Snowflake-Labs/terraform-provider-snowflake/issues/2411. Lucky there's an open pr (https://github.com/Snowflake-Labs/terraform-provider-snowflake/pull/2448) that uses new SDK in view and materialized view, so after we merge and release it, the error shouldn't be there.

Hey @lkorpalski-pgs  👋
Yesterday, we released a new provider version (v0.95.0) that introduces a lot of changes (also the previously mentioned pr should resolve this issue). Please upgrade using the migration guide. Closing this one, please create another issue if the problem still persists in the newer versions.

Unfortunately I can't confirm the fix myself, as I got moved to another project since then. Thank you for the notice though.


MD-WIDGETS: Support CHROME target

Description
Currently seeing a few issues with flutter run -d chrome. Otherwise it is working great. Thanks.
Why
So that a web-based app functions fully with the package.
Details
VIDEO DOES NOT LOAD

SUBMIT FAILS
Survey 3, tap on SUBMIT results in

This issue has been fixed.


Exception#backtrace? raises an IndexError on macOS

I have not yet been able to successfully reduce the code. The relevant code is:
      begin
        message = @ex.inspect_with_backtrace
      rescue ex : IndexError
        # ERROR: Any of these will cause another IndexError (at least on macOS)
        # ex.inspect_with_backtrace # or @ex.inspect_with_backtrace
        # ex.backtrace? # or @ex.backtrace?
        message = @ex.message
      end

To reproduce, using macOS, you can follow these steps:

git clone git@github.com:amberframework/amber.git
git fetch && git checkout ms/fix-internal_server_error-error
Uncomment the line at src/amber/controller/error.cr:35 (or line 36): https://github.com/amberframework/amber/pull/625/files#diff-fefe72c37ea251ccaec8417bffb49e8cR35
Run crystal spec spec/amber/router/pipe/error_spec.cr:15

With the line commented out, the tests pass:
$ crystal spec spec/amber/router/pipe/error_spec.cr:15
.....

Finished in 128.61 milliseconds
5 examples, 0 failures, 0 errors, 0 pending

With the line uncommented, we get the Index out of bounds error.
$ crystal spec spec/amber/router/pipe/error_spec.cr:15
....E

Failures:

  1) Amber::Pipe::Error returns status code 500 for all other exceptions

       Index out of bounds

$ crystal -v
Crystal 0.24.1 (2018-01-27)
LLVM: 5.0.1
Default target: x86_64-apple-macosx

$ sw_vers
ProductName:	Mac OS X
ProductVersion:	10.13.3
BuildVersion:	17D47

From what I've looked at in the crystal code base, it may have to do with CallStack. There is a difference in darwin vs linux in that file:
https://github.com/crystal-lang/crystal/blob/351596855d635141deb204f7c572ed615f415ce5/src/callstack.cr#L8
What led me there is:
https://github.com/crystal-lang/crystal/blob/v0.24.1/src/exception.cr#L35
And then here:
https://github.com/crystal-lang/crystal/blob/351596855d635141deb204f7c572ed615f415ce5/src/callstack.cr#L157

Does it still happen with crystal master? I fixed a very similar issue recently.

I'm checking now. Is this the relevant commit: https://github.com/crystal-lang/crystal/commit/048f77e7a389cb85875b88fd0839183229866f66

Probably duplicate of #5224

I'm running into an unrelated issue when trying to test against master, may just need to update some dependencies (pg).
It is a JSON::Type error - that was updated to JSON::Any, correct?

This is fixed in master.
Sorry it took me a while to get setup - there are a handful of breaking changes that affect crystal-pg and amber that I had to work through.


Updating documentation for districtmDMX for version 1+

@stevealliance  - these changes look fine, but have to ask about whether removing the original docs is what you intended. Many pubs are still on PBJS 0.x, so perhaps you still want to have your adapter docs describe the 0.x params for an interim period?

@bretg good question we can still keep the old config  for our previous adapter you want update my pr?

Yes - you can update this PR. Either make another table for 0.x params, or add a column to the existing table to note which parameter goes with which version.

updating documentation for DMX for prebid version 1+

@bretg how do we get added to the download page... We do have a new adapter in Prebid.js project, but I don't see our script listed and we are in Prebid since version 1.5.
Thanks

Hi @matthewlane how can we get added to the download page


Error in npm install -g generator- angular

i got this error pls any one help me
npm WARN excluding symbolic link test/symlinks/dir1/dangling-symlink -> does-not-exist
/usr/bin/generate -> /usr/lib/node_modules/generator/bin/index.js
/usr/bin/generator -> /usr/lib/node_modules/generator/bin/index.js
npm WARN unmet dependency /usr/lib/node_modules/generator-angular/node_modules/yeoman-generator/node_modules/file-utils requires<EMAIL_ADDRESS>but will load
npm WARN unmet dependency /usr/lib/node_modules/generator-angular/node_modules/yeoman-generator/node_modules/lodash,
npm WARN unmet dependency which is version 2.4.2
npm WARN unmet dependency /usr/lib/node_modules/generator-angular/node_modules/yeoman-generator/node_modules/file-utils requires<EMAIL_ADDRESS>but will load
npm WARN unmet dependency /usr/lib/node_modules/generator-angular/node_modules/yeoman-generator/node_modules/isbinaryfile,
npm WARN unmet dependency which is version 2.0.4
<EMAIL_ADDRESS>/usr/lib/node_modules/generator
├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>└──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

Those are warnings and seems your installation succeed, didn't it?
Is that a Crouton related issue?

thank you iam new to yeoman
how to check is it installed or not

Not crouton related...see #262

@vijayaragavan123 it did not return any error and node showed you the structure of modules.
Please navigate to the nodejs community for more specific questions.


Audit logs view for privacy vault

What does this PR do?

Adds audit log view under Privacy Vault tab. This will let users see the security logs related to terminus project.
Type of change

[ ] Updated dependencies
[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How should this be tested?

[ ] Existing unit tests

Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code and corrected any misspellings
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

@niwsa https://github.com/retracedhq/logs-viewer/releases/tag/v2.8.0 is ready


slam pipeline enable_gui

when I tried to set 'enable_gui' to true,  subprocess was something wrong

I give up docker, and use source code of ORB_SLAM3


Log an ERROR when a provided argument does not exist in the config

When an argument does not exist in the template config, nothing is logged.
e.g.:
template.md
my_config [[#arg1]]

Using {{#template template.md arg1=foo arg2=bar}} does not log an error and the user is not aware of his mistake.
I propose to log an error in this case. It can be useful to identify broken templates and build CI jobs.

Hey @AlianBenabdallah, thanks for bringing my attention to this!
That sounds like a pretty helpful error to have. Unfortunately, I won't have the time to work on this. I'd happily accept a PR if you are willing to go ahead and implement this! Of course, I'd happily try to answer any questions you might have.

Thank you for your answer !
I don't have the access rights to open a new PR. If possible, could you make the function replace_args and every struct / trait in utils.rs public ?

I'm not sure what you mean, sorry. If you forked the repository, you should just be able to raise a PR?
Can you explain the reasoning behind making the functions/traits public? Thanks!

Thank you, I realized I could open the PR.
We are working on integrating mdbook-template to our website. We have a bunch of templates representing CLI commands. We want to implement a CI job to :

Identify every {{#template ...}} in the guide.
Replace it by the actual content with replace_template()
Check that the CLI command can be parsed.

We faced many problem because of what I previously mentionned in this issue. We also want to perform additional checks. In order to do that, we need a FileReader and replace_template().

Ah no worries, I didn't realise people would be using the functionality itself. Will review the PR now 👍

Merged #6 for your convenience! @AlianBenabdallah

Awesome, thank you so much !


add class name to  for easy customization

Would be great to have class name to easy write styles for glossary words.
Now I temporary use CSS Attribute Selectors:
.uk-article a[data-uk-tooltip=""] {
border-bottom: 1px #9ad8cf dotted;
}

Added to 1.0.4


Extra 2 ads on homepage

Extra 2 ads on homepage.

Closing this one for now. Will reopen later if needed


Hamburger Menu UI

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Hamburger menu is a very popular UI feature of mobile phones, and I want to replicate that feel on desktop as well.
Describe the solution you'd like
A clear and concise description of what you want to happen.
The Side menu is great, but if it could behave like "slide in and out of screen", that would be enough.{ or maybe few things extra{e.g glass design } to make it more befitting of title}
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
I Tried forcefully make one, I might succeed as well, but this got me thinking if such window could be directly become a part of Handy Control Project.
Additional context
Add any other context or screenshots about the feature request here.
Here are three images for the same

Same as issue #182

duplicate if #182


GMSH writer physical groups missing

It seems the writer does not write properly the physicals
I just write this small test
import meshio

infile ="./test.msh"
outfile="./test2.msh"
mesh = meshio.read(infile)
meshio.write(outfile,mesh,"gmsh4-ascii")

Input mesh generated with GMSH 4.4.1:
https://gist.github.com/gdolle/7a7e78ef1412f3304db5dd70f40e2ea3#file-test-msh-L61-L62
The highlighted section in the link is missing in the exported mesh.
I think it's related to "gmsh:physicals" in cell_data ? But the readme does not explain it.
This issue might be related to issue #536

@nschloe sorry I didn't see this open issue.

I'm looking at this issue now, but I can't quite figure out how to reproduce it. Could you create a minimal example that shows the behavior?

@nschloe, it seems this old bug is fixed with the current version ( gmsh4_ascii writer format does not exist anymore in favor of file_format="gmsh", binary=False).

Thanks for checking!


update id for token withdrawals

@wminshew what do you think about this change?
I need to test it out, but I believe this should allow us to query for just token withdrawals from users, or just token withdrawals from splits. Which should let us easily fetch top 10, and paginate (only by eth withdrawals still though).

yeah ig this makes sense to me -- i can't think of any functionality we need or might need that it would break
so is the idea that on query you can filter by id prefix? feels like a pretty good 80/20 sol'n

yea that's the idea. a little annoying it's needed, but until they support the child entity filters i think this is really the only way to do it.


Cannot Call createCustomMessage from actionProvider

Hi Fredrik,
I've been using this package with great success, but have run into a blocker. I have created a custtom message in config and it fires fine from the initial messages, but when I try to call it from my actionProvider, I get the createCustomMessage is not a function. Not quite sure where I lost the plot.
`import React from 'react';
const ActionProvider = ({ createChatBotMessage, setState, children, createCustomMessage }) => {
const handleDiabetes = () => {
const botMessage = createChatBotMessage('Diabetes Benefits are...');
    setState((prev) => ({
      ...prev,
      messages: [...prev.messages, botMessage],
    }));
};
const handleDental = () => {
    const botMessage = createChatBotMessage('Dental
Benefits are...');

    setState((prev) => ({
      ...prev,
      messages: [...prev.messages, botMessage],
    }));
};
const handleCustom = () => {
    const botMessage = createCustomMessage('Test', 'chatResponse');

    setState((prev) => ({
      ...prev,
      messages: [...prev.messages, botMessage],
    }));
};
return (

  {React.Children.map(children, (child) => {
    return React.cloneElement(child, {
      actions: {handleDiabetes, handleDental, handleCustom},
    });
  })}

);
};
export default ActionProvider;`

i am facing this error Class constructor ActionProvider cannot be invoked without 'new' with create-react-app

I was able to get the custom message working in my ActionProvider. The way I did it was to firstly write a new component to represent the custom message and save this in a new file
const QuotaMessage = () => {
  return (

      This chat is getting too long.

  );
};

export default QuotaMessage;

I imported this class into the config component and then added  the following to the config
  customMessages: {
    quota: (props) => <QuotaMessage {...props} />,
  },

Then in my actionprovider class, I imported  the createCustomMessage class from the react-chatbot-kit
import { createCustomMessage } from "react-chatbot-kit";

I was then able to amend one of the methods in the action provider class to use this:
    let newMessages = [];

      const customMessage = createCustomMessage(
        "some message.",
        "quota"
      );
      newMessages.push(customMessage);
    }

    setState((prev) => ({
      ...prev,
      messages: [...prev.messages, ...newMessages],
    }));

Thanks @greendinosaur, this worked for me for creating custom messages from the actionprovider!

Is there a way I can display a custom dynamic message?
My message is determined by a API call that I'm doing in ActionProvider. So, I want to display the message dynamically in CustomMessage.
Can this be achieved?


Volume Control doesn‘t work

Hey @ebaauw thank you for your perfect work and this great plugin. Just updated to latest version early this morning and added TV Mode. It works great. But I‘ve got one Problem with the remote widget. Only Play/Pause works but no Volume control whether I set up my config to Speaker, Fan, Switch or Bulb.
Hope you can help to fix.
Best regards Marv

The remote widget uses the hardware volume buttons on your iPhone or iPad for volume control. When pressing these, you should see log messages like:
[7/24/2019, 2:55:22 PM] [Sonos] Living Room TV Speaker: Volume Selector: Down

If they're not registering, you might need to select a different "television" and re-select the one you want to control.  The widget might be a bit flaky at times.

Tested it and works great Thanks —> closed


Dockerfile

Could you please show Dockerfile used to build docker image?

https://github.com/rhasspy/wyoming-addons/blob/master/openwakeword/Dockerfile


[INF] GitHub Auto-Release Pointer

PR Description

With a few simple lines added to .github/workflows/auto-release.yml, we can correctly reference (i.e. "point") to the latest release in the landing page of the GitHub repo. In addition, this will include a "prettier" interface when navigating the releases.
This has been tested on solid-octo-guac (thanks @ericmjl for keeping this up 😅 ) here
See https://github.com/ncipollo/release-action

This PR resolves #818
PR Checklist
Please ensure that you have done the following:

[X] PR in from a fork off your branch. Do not PR from <your_username>:dev, but rather from <your_username>:<feature-branch_name>.
[X] If you're not on the contributors list, add yourself to AUTHORS.rst.
[X] Add a line to CHANGELOG.md under the latest version header (i.e. the one that is "on deck") describing the contribution.

Do use some discretion here; if there are multiple PRs that are related, keep them in a single line.

Automatic checks
There will be automatic checks run on the PR. These include:

Building a preview of the docs on Netlify
Automatically linting the code
Making sure the code is documented
Making sure that all tests are passed
Making sure that code coverage doesn't go down.

Relevant Reviewers
@samukweku and @jk3587 this will add a single step to the auto-release script.
Please tag maintainers to review.

@ericmjl

Wow, impressive, @loganthomas! Thanks for handling this. Love that you've been testing it on that dummy repo 😄!

@loganthomas please do the honors!


Create a section for HADITH Book

Create a section call hadith, similar to this UI

Hello @devvsakib I want to Work on this issue

@devvsakib  Thanks For  Assigning me.
I have One Doubt , Did you Want one section below this similar to attached Screenshot.


Spring boot + Graphql + Oauth2 how can i secure my single endpoint?

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
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.

try keycloak.org

Closing due to insufficient information in question.


hashes: Check if we can remove trait imports

One can use Uniform Function Call Syntax (UFCS) and not import traits, we should check hashes over for other instances of this improvement.
Pointed out to me here: https://github.com/rust-bitcoin/rust-bitcoin/pull/3128#discussion_r1743138964

Within macros it's pretty-much always a good idea.
Outside of macros, if we find we're importing traits to call one function, sometimes it's a good idea.
A related, but more universal, win is to use use Trait as _ whenever we import traits just to get access to their methods. This pollutes our namespace less and clearly signals that we're just using the trait for methods, and when people are trying to find the origin of methods, the as _ is a good hint that maybe they're coming from traits.

Trait as _ still triggers lint warnings when unused so I rekon there will be less maintenance burden if we use UFCS in all macros. (I opened this issue not because I checked but because I've change the imports loads of times over the last couple years without thinking of this.)

In macros, definitely, in internal macros we don't have to use UFCS we can also import the trait but for a single function it's not relevant.
Side note: globally importing everything at the top of the file raises risk of pointless merge conflicts. Thankfully they are solvable with my merge driver but that still has some issues.

in internal macros we don't have to use UFCS

Isn't it still better to use UFCS in internal macros so they don't leak trait imports?

Imports like https://github.com/liamaharon/rust-bitcoin/blob/25d906d936eed34cf398e1a8b1abe8176c1b7c02/hashes/src/sha512.rs#L9 are fine right because they're not necessary for the macro usage?

@Kixunil I'm also curious what you meant by this. I agree with your factual claim but "globally importing everything at the top of the file" is standard Rust practice, and alphabetizing them/reformatting them standard rustfmt behavior, so I think this is just something we have to live with?

Basically, if you have Foo that's only used in fn bar it's less conflict-prone to write:
fn bar() {
    use foo::Foo;

   // ...
}

than:
use foo::Foo;

fn bar() {
   // ...
}

standard Rust practice

While there are many standard practices that are good for their own reasons I don't see how it would apply here. Though arguably my driver should just be improved to handle everything and we shouldn't worry about it in the first place.

Basically, if you have Foo that's only used in fn bar it's less conflict-prone to write:

Right. I think it's a bit more subtle than "only used in fn bar". It's more like "only used in fn bar and unlikely to be needed elsewhere in the future".
I try to do this in my own code, and don't mind encouraging people to do it, but I think it's too murky (and uncommon) for us to try to make it a hard rule. In particular I don't think @liamaharon should be holding up his PR over this.

and unlikely to be needed elsewhere in the future".

It's not uncommon that I duplicate an import (or avoid the entire use statement). Generally I tend to add global use when an item is used in function signatures repeatedly.
But yes, this is quite strict and makes the code a bit more annoying to write, possibly also to read. I think we would be better off reconciling our formatting rules with my merge driver so that it can do its job out-of-the-box.


0-length arrays in Buddy ranges

Buddy ranges can be instantiated with MAX_SIZE_BITS == MIN_SIZE_BITS (or, in principle, <=).  If this happens, the resulting class will contain 0-length arrays, and code that appears to attempt to access an element therein trips gcc's -Warray-bounds.
This PR contains two commits: one that adds a static_assert to ensure that Buddy's arrays are always of positive size, and one possible way to fix the case I'm tripping over.  The latter works because the Buddy's MIN_SIZE_BITS is instantiated at MIN_CHUNK_BITS, and so the change herein then ensures that its MAX_SIZE_BITS is strictly greater than MIN_CHUNK_BITS.
A different, and possibly better, solution would be to drop the LargeBuddyRange from CentralMetaRange when max_page_chunk_size_bits is computed to be equal to MIN_CHUNK_BITS.
Please advise.

You OK to merge even though macos CI is still pending?

You OK to merge even though macos CI is still pending?

Yeah. I think Mac os 11 support has been removed. I think we need to remove and possibly add the next version


Add sentry integration in prod

Add sentry integration in production.
Note that docker-compose up results in a production build, so any errors in a local docker compose stack will get pushed to sentry.  I don't really care that much, but it might be a problem later.
Also, if anyone tries to run this stack on their own, they'll ship their errors to our sentry collection.  That may or may not actually be preferable.  Punt for now.
I'd like @AlmightyYakob to review this because I think he has used sentry a lot more than me.

https://stackoverflow.com/questions/45802988/typescript-use-correct-version-of-settimeout-node-vs-window
/shrug

Note that docker-compose up results in a production build, so any errors in a local docker compose stack will get pushed to sentry. I don't really care that much, but it might be a problem later.

This could get pretty annoying if we actually care about addressing, although I think you have the option of ignoring errors raised from localhost. The way I've usually addressed this is by not keeping the DSN in source control, and only specifying it in the .env file in your particular deploy (i.e. in prod). That way, if you run it locally and don't specify the value (default), it won't initialize a sentry value.

The way I've usually addressed this is by not keeping the DSN in source control, and only specifying it in the .env file in your particular deploy (i.e. in prod)

Eh, then we'd have to put the file on the regular host and the AWS host, and I also think I prefer to get errors reported from customer's deployments (there may be a consent issue here).
Let's just take everything for now and we can refine it later if it's a problem?


Restrict New-Object in NoLanguage mode under lock down #14139

PR Summary

Restrict New-Object in NoLanguage mode under lock down #14139
PR Context

PR Checklist

[ ] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[ ] Summarized changes
[ ] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[ ] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: or [ WIP ] to the beginning of the title (the WIP bot will keep its status check at Pending while the prefix is present) and remove the prefix when the PR is ready.

Breaking changes

[ ] None
OR
[ ] Experimental feature(s) needed

[ ] Experimental feature name(s):

User-facing changes

[ ] Not Applicable
OR
[ ] Documentation needed

[ ] Issue filed:

Testing - New and feature

[ ] N/A or can only be tested interactively
OR
[ ] Make sure you've added a new test if existing tests do not effectively test the code changed

Tooling

[ ] I have considered the user experience from a tooling perspective and don't believe tooling will be impacted.
OR
[ ] I have considered the user experience from a tooling perspective and enumerated concerns in the summary. This may include:

Impact on PowerShell Editor Services which is used in the PowerShell extension for VSCode (which runs in a different PS Host).
Impact on Completions (both in the console and in editors) - one of PowerShell's most powerful features.
Impact on PSScriptAnalyzer (which provides linting & formatting in the editor extensions).
Impact on EditorSyntax (which provides syntax highlighting with in VSCode, GitHub, and many other editors).

The test case passed on my local machine where as it failed in the CI.


OS support for .NET core 2.0 on RHEL

The release notes for the supported OS's state that RHEL 6 is supported on .NET 2.0, but I believe that is incorrect. The MS article on supported OS's only lists RHEL 7:
https://docs.microsoft.com/en-us/dotnet/core/linux-prerequisites?tabs=netcore2x
I'm referencing this page:
https://github.com/dotnet/core/blob/master/release-notes/2.0/2.0-supported-os.md

related https://github.com/dotnet/core/issues/1260

@joshfree link you provided ends with
'Ok, I'm going to close this now as I think the question is answered. We will support this in .NET Core 2.1.'
I don't see reference to adding support for .NET Core 2.0. Just to be clear. I assume the comment by @leecow is what would allow .NET core 2.0 to be used with RHEL6?

https://github.com/dotnet/core/blob/master/os-lifecycle-policy.md is the official entry point for the .NET Core version / OS version support matrix and RHEL6 is included in the 2.0 and 2.1 specific docs.

Will be updated today.

Please note for the future that Red Hat does not support .NET Core on RHEL6 and therefor RHEL6 support should not contain a link to Red Hat Errata policy.
#1844


--tab-width 2 goes crazy with JSX

Input test.js:
function hello() {
return (

hello

);
}

Without any options
prettier test.js

it outputs:

but with
prettier test.js  --tab-width 2

it outputs:

I was expecting the same output from both.
Using version 0.11.0.

Which operating system are you on? I can't reproduce on OSX 10.11.6

I also cannot reproduce it on OSX. And I'm not sure what could cause this. Maybe @rattrayalex you have an idea.

@epeli is it outputting tab characters or spaces? Can you copy/paste the raw inputs and outputs here or a gist?

Ahhh, @epeli are your line breaks CRLF (\r\n) ?
I used a bunch of \n-specific regexes in the JSX processing code.

https://github.com/jlongster/prettier/pull/452 fixed this issue I believe.

@jaysoffian Nope, that PR was all about fixing this the following :)
$ prettier test.js --tab-width 4
test.js: { ● Validation Error:

  Option "tabWidth" must be of type:
    number
  but instead received:
    string

  Example:
  {
    "tabWidth": 2
  }

  name: '',
  message: '\u001b[31m\u001b[1m\u001b[1m●\u001b[1m Validation Error\u001b[22m:\n\n  Option \u001b[1m"tabWidth"\u001b[22m must be of type:\n    \u001b[1m\u001b[32mnumber\u001b[31m\u001b[22m\n  but instead received:\n    \u001b[1m\u001b[31mstring\u001b[31m\u001b[22m\n\n  Example:\n  {\n    \u001b[1m"tabWidth"\u001b[22m: \u001b[1m2\u001b[22m\n  }\n\u001b[39m' }

It seems to be working fine with windows line endings.
cat -e test.js
function hello() {^M$
return (^M$

^M$
hello^M$

^M$
);^M$
}^M$

./bin/prettier.js test.js --tab-width 2 | cat -e
function hello() {$
  return ($

$
      hello$

$
  );$
}$

Closing this for now until @epeli gives us more information.

Sorry for the delay but it seems that the issue is not present anymore in 0.17.0. Awesome tool, thanks!


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target/pull/887 is merged into live branch.
Auto archive content to https://github.com/opstest2/docs-archive-test-target.git

Docs Build status updates of commit c31767f:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error-GitBranchDeletedOrForcePushed] Cannot sync git repo to specified commit because branch Release_Archive_master_2021-06-07-08-34-15 has been deleted or has been force pushed

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Error on resume when temp files (file.tmp.*) present

Resume will fail when temp files are present in the root of the backup directory.  These temp files are the remnants of partially copied files.  Log output:
-bash-4.1$ crunchy-backup-manager --stanza=dev --type=full backup --log-level-console=info
2015-07-24 17:42:15.505 T00   INFO: backup start: type = full
2015-07-24 17:42:15.579 T00   INFO: executing pg_start_backup() with label "crunchy-backup-manager backup started 2015-07-24 17:42:15": backup will begin after the requested immediate checkpoint completes
2015-07-24 17:42:16.057 T00   INFO: archive start:<PHONE_NUMBER>0000040000003D
2015-07-24 17:42:45.250 T00   WARN: aborted backup of same type exists, will be cleaned to remove invalid files and resumed
2015-07-24 17:42:45.250 T00   INFO: cleaning backup tmp path

2015-07-24 17:42:46.180 T00  ERROR: process terminated on signal or exception, 2 threads stopped
substr outside of string at /usr/share/perl5/BackRest/Backup.pm line 149.
 at /usr/bin/crunchy-backup-manager line 13
        main::__ANON__('substr outside of string at /usr/share/perl5/BackRest/Backup....') called at /usr/share/perl5/BackRest/Backup.pm line 149
        BackRest::Backup::backup_file_not_in_manifest('backup:tmp', 'BackRest::Manifest=HASH(0x1eb0d68)', 'BackRest::Manifest=HASH(0x3a5c478)') called at /usr/share/perl5/BackRest/Backup.pm line 231
        BackRest::Backup::backup_tmp_clean('BackRest::Manifest=HASH(0x1eb0d68)', 'BackRest::Manifest=HASH(0x3a5c478)') called at /usr/share/perl5/BackRest/Backup.pm line 662
        BackRest::Backup::backup('/data/pgsql/9.4/data', 1) called at /usr/bin/crunchy-backup-manager line 208
        eval {...} called at /usr/bin/crunchy-backup-manager line 61
 at /usr/bin/crunchy-backup-manager line 260
 at /usr/bin/crunchy-backup-manager line 13
        main::__ANON__('substr outside of string at /usr/share/perl5/BackRest/Backup....') called at /usr/share/perl5/Carp.pm line 45
        Carp::confess('substr outside of string at /usr/share/perl5/BackRest/Backup....') called at /usr/bin/crunchy-backup-manager line 260

Fixed in e488f7b.


fix image

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/reference-architectures/serverless/_images/serverless-event-processing.png
:white_check_mark:Succeeded
View

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


Added script to compare files across adafruit repos

This is the first time in a while I've documented a file like this. However, I do think it looks way better than the way I was previously doing

I don't know if this would be the same thing, but I clone https://github.com/adafruit/Adafruit_CircuitPython_Bundle and keep it up to date. Then I search inside it with ag (like grep -r but better and faster).

@dhalbert That's a good point. The reason I found doing it with this script easier than something like that is that this script outputs them in a format that's really easy to read through and similarly easy to be used by another script or by a user manually using the output for something.


SSH_AUTH_SOCK needs to be white listed when environment is filtered

Please note we will close your issue without comment if you delete, do not read or do not fill out the issue checklist below and provide ALL the requested information. If you repeatedly fail to use the issue template, we will block you from ever submitting issues to Homebrew again.
Please always follow these steps:

[X] Confirmed this is a problem with running a brew command and not brew installing or the post-install behaviour of one or more formulae? If it's a formulae-specific problem please file this issue at the relevant tap e.g. for Homebrew/homebrew-core https://github.com/Homebrew/homebrew-core/issues/new
[X] Ran brew update and retried your prior step?
[X] Ran brew doctor, fixed all issues and retried your prior step?
[X] Ran brew config and brew doctor and included their output with your issue?

brew config
HOMEBREW_VERSION: 1.3.8-65-g6cf38d0
ORIGIN: https://github.com/Homebrew/brew.git
HEAD: 6cf38d0dd87ab215bc5be13bf98f3654531b6ef4
Last commit: 77 minutes ago
Core tap ORIGIN: https://github.com/Homebrew/homebrew-core
Core tap HEAD: f17f64c9aa75957ee7775f6a4d5d32db4a137b6d
Core tap last commit: 13 minutes ago
HOMEBREW_PREFIX: /usr/local
HOMEBREW_DEV_CMD_RUN: 1
HOMEBREW_ENV_FILTERING: 1
HOMEBREW_GITHUB_API_TOKEN: set
HOMEBREW_GIT_CONFIG_DEVELOPERMODE: true
CPU: octa-core 64-bit haswell
Homebrew Ruby: 2.3.3 => /usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/2.3.3/bin/ruby
Clang: 9.0 build 900
Git: 2.13.6 => /Applications/Xcode.app/Contents/Developer/usr/bin/git
Curl: 7.54.0 => /usr/bin/curl
Perl: /usr/local/bin/perl => /usr/local/Cellar/perl/5.26.1/bin/perl
Python: /usr/local/opt/python/libexec/bin/python => /usr/local/Cellar/python/2.7.14/Frameworks/Python.framework/Versions/2.7/bin/python2.7
Ruby: /Users/paulmer/.rvm/rubies/ruby-2.2.0/bin/ruby
Java: 1.8.0_77, 1.8.0_71, 1.8.0_60
macOS: 10.13.1-x86_64
Xcode: 9.1
CLT: <IP_ADDRESS>.1.1508540944
X11: 2.7.11 => /opt/X11

brew doctor
Please note that these warnings are just used to help the Homebrew maintainers
with debugging if you file an issue. If everything you use Homebrew for is
working fine: please don't worry and just ignore them. Thanks!

Warning: "config" scripts exist outside your system or Homebrew directories.
`./configure` scripts often look for *-config scripts to determine if
software packages are installed, and what additional flags to use when
compiling and linking.

Having additional scripts in your path can confuse software installed via
Homebrew if the config script overrides a system or Homebrew provided
script of the same name. We found the following "config" scripts:
  /Users/paulmer/login-config

Warning: Unbrewed dylibs were found in /usr/local/lib.
If you didn't put them there on purpose they could cause problems when
building Homebrew formulae, and may need to be deleted.

Unexpected dylibs:
  /usr/local/lib/libmpg123.0.dylib

Warning: You have unlinked kegs in your Cellar
Leaving kegs unlinked can lead to build-trouble and cause brews that depend on
those kegs to fail to run properly once built. Run `brew link` on these:
  docker
  cpanminus
  php71
  ruby
  cloog

Warning: Some keg-only formula are linked into the Cellar.
Linking a keg-only formula, such as gettext, into the cellar with
`brew link <formula>` will cause other formulae to detect them during
the `./configure` step. This may cause problems when compiling those
other formulae.

Binaries provided by keg-only formulae may override system binaries
with other strange results.

You may wish to `brew unlink` these brews:
<EMAIL_ADDRESS>
To help us debug your issue please explain:

What you were trying to do (and why)
brew update with a tap to a local git repository accessed via ssh

What happened (include command output)
Access to the repository fails because ssh authentication via public/private keys fails
(Paths and system information intentionally omitted [...])

brew update
Red Hat Enterprise Linux Server [...]

Permission denied (publickey,gssapi-keyex,gssapi-with-mic).
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.
Error: Fetching /usr/local/Homebrew/Library/Taps/[...] failed!
Updated Homebrew from 6cf38d0dd to 3f7e63a24.
No changes to formulae.

What you expected to happen
tap should be updated

Step-by-step reproduction instructions (by running brew commands)
Before I give you step-by-step details, let me explain the issue:  When environment filtering is enabled, the SSH_AUTH_SOCK variable is removed from the environment which causes SSH key authentication within git to fail.  The SSH_AUTH_SOCK variable should be whitelisted in bin/brew.

To reproduce:

Create a git repository for a tap, accessible via SSH.  Configure SSH keys to access the repository.
Tap the repository (without environment filtering): HOMEBREW_NO_ENV_FILTERING=1 brew tap repo/name ssh://path/to/repo.  This should access the tap using the configured SSH keys.
Brew update with filtering enabled: HOMEBREW_NO_ENV_FILTERING= brew update.  This will fail because SSH_AUTH_SOCK is filtered from the environment prior to running Git.

Duplicate of https://github.com/Homebrew/brew/pull/3493, but this provides a better justification for whitelisting it.

Ugh, sorry about the duplicate.

Error: Fetching /usr/local/Homebrew/Library/Taps/[...] failed!

Which tap failed?

Tap the repository (without environment filtering): HOMEBREW_NO_ENV_FILTERING=1 brew tap repo/name ssh://path/to/repo. This should access the tap using the configured SSH keys.

Is there a non-SSH URL you can use for this tap?

Depending on the environment, some internal taps might only be accessible via SSH. That's liable to be the case for teams that are hosting their internal taps outside of Github.com proper.

@jonatin Is this the case for you, though?

Yes, it is, but I'm unable to share the full details. I'm stuck with several internal taps that can only be accessed via SSH for the majority of my users.


Introduce caching and minor refactorings

Cache loaded projects and condition evaluation states (controlled by switch below)
Introduce ProjectReader.EnableCaching property that defaults to false (so no surprises in tests, but enabled in Console project)
Fix Project2015To2017.Console.csproj OutputType
Store data source XML elements for future changes (to be able later to reference data source precisely)
Preparation for future PRs (more information to come)
Nuget -> NuGet
Use ProjectReference for Solution model (to reduce future code duplication and simplify semantics)
Add resolved project file path to ProjectReference (to simplify use of ProjectReference in Project creation)
Refactor solution handling in ProjectConverter.Convert (introduce ConvertSolution method)
Use the only solution in current directory if present (to match MSBuild behavior)
Fix indentation according to .editorconfig (tabs instead of spaces, I fixed only files with mixed indent)
Greatly improve XamlPagesTransformation to account for MSBuild.Sdk.Extras advanced features
Add Instance fields for SolutionReader and ProjectReader to reduce memory pressure on huge solutions (non-obligatory)

Behavior changes:

If there is one and only solution in specified directory to convert, use that.
XamlPagesTransformation now filters more elements

Future work (not this PR):

Working analysis for common problems in project configuration
Ability to read (with only FileTransformation enabled) already converted (CPS) projects (no writing of course) to support running analysis again and account for changed line numbers

It's not my place to say for this project, but I found a good article that sounded like a good and efficient way to propose multiple changes like this. It means that maintainers can easily process each change one at a time without you having to do lots of remerging later, just branch of each PR in turn:
https://graysonkoonce.com/stacked-pull-requests-keeping-github-diffs-small/

@hvanbakel review, please?

@hvanbakel I will correct things you've mentioned and force push PR edits. Well, I use Dopamine (regularly) & PascalABC.NET (a couple of times) for XAML tests.
I am thinking of splitting this PR into more PRs too. Will see if I can do that simple enough.

I would expect that to be static, yes. That's around for the lifetime of
the application.
On Sun, Jul 29, 2018, 21:41 Andrew Boyarshin<EMAIL_ADDRESS>wrote:

@andrew-boyarshin commented on this pull request.
In Project2015To2017/Reading/ProjectReader.cs
https://github.com/hvanbakel/CsprojToVs2017/pull/140#discussion_r206013610
:

using static Project2015To2017.Definition.Project;

namespace Project2015To2017.Reading
{
public class ProjectReader
{

public Project Read(string filePath)

public static readonly ProjectReader Instance = new ProjectReader();

Ok, you mean removing Instance field and adding constructors to initialize
some FileInfo Project field? But then there is no point in caching,
unless cache is made static. Is that ok?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/hvanbakel/CsprojToVs2017/pull/140#discussion_r206013610,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AD8FPErRBe-4CXaD6hfe2KztlQQs8YZvks5uLo6AgaJpZM4ViEWh
.

@hvanbakel done. I also had to add .NET Standard 2.0 target for System.Runtime.Caching NuGet package. On older targets (.NET Standard 1.3) old Dictionary solution is used. Only one is compiled at a time.

Merged as is


[Feature] Tags Taxonomy View

Implement a Tags Taxonomy View, so that's it's very easy to see -

Speakers that are associated with a given tag
Upcoming Events (by association of activities) that are associated with a given tag
Past Events (by association of activities), that are associated with a given tag.. may be interested in any resources that are available as a result, for example.

Initial version of this is complete, so now marking as closed. Tags have been adjusted to be "Topics" for consistency across Speakers and Events (Activities).


Add cookie

This PR adds Cookie implementation as discussed in php-http/httplug#98
Fix #15
A few open issues left in the original repository:
https://github.com/php-http/cookie/issues

php-http/cookie#4: Make CookieJar a single class and support Persistence through a separate interface
php-http/cookie#5: Double check matching implementation, I remember I had some issues with it

Thanks for the feedback.

About the expires thing: currently you can pass an expiration parameter in the constructor. Should we have max age instead. Also, this expiration is set for both max age and expires property ATM.
Should we keep both max age and expires and only remove the magic expiration method?

i would allow to set both from the constructor, to make it possible to
model a response precisely.
but remove the magic method.
and phpdoc that expire is http 1.0 and should be avoided.

What about order of arguments: should expires be the last?

:+1: agree to have expires last so it can be ignored when not recording an exact Server response

We could add cookie related helpers here.
One thing I am thinking about:
We could have the cookie parser as is OR provide a compat package around this method:
http://php.net/manual/en/function.http-parse-cookie.php
In general, we could have a pecl_http compatibility layer, but I have never used it before and don't plan in the future, so I wonder if it makes sense.

Let's make cookie parsers in another PR. I think this is good to merge. Checked RFC compliance, but I am not sure about it. I've read the RFC several times, it is not always clear. From what I can understand, everything is OK.

the pecl_http compatibility layer sounds like a good idea but lets move forward. it can always be added later.


Translate  concepts/storage/volume-snapshot-classes/ in Japanese

This is a Feature Request

What would you like to be added

Translate concepts/storage/volume-snapshot-classes/ in Japanese.
Why is this needed
This document is not translated in Japanese.
Comments

Copy a file from en contents to content/ja/docs/concepts/storage/volume-snapshot-classes.md and translate it into Japanese.

/language ja


Pylance allows redefining a variable with a type hint

I would expect that the following program should result in a warning (using the strict mode):
a: int = 1
a = True

In fact, I think it should result in a warning even without a type hint, but it does not.
Using Pylance v2022.11.20 under VS Code 1.72 under Windows 10 Pro.

I realized the Boolean can be converted to int in Python.

Yes, the bool type is a subtype of int, so it is allowed to assign a bool to a variable that has been declared as an int.


Update ipipe_py3.sh

重构已有case，添加trt模型加密的case，优化CI产出

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


feat(webapp): improve document search

The searchbar in the top right corner is not optimal. Better solution is to create a shortcut for fast access (⌘K/ctrlK) and show a popup panel with input field and autocomplete list.
Should be also possible to search for document title for fast access.

Closed with #62


Adds SOCKS5 support via https_proxy ENV var

Fixes #18
Re-enables using the default Go 1.9 environment variables to proxy std library calls via a SOCKS5 proxy
$ https_proxy=socks5://localhost:61011 credhub api -s https://<IP_ADDRESS>:8844 --ca-cert "$CREDHUB_CA" --ca-cert "$UAA_CA"

LGTM; thanks @evanfarrar.

Hit another http.Transport i had ignored (I only manually tested api --skip-tls-validation not login or with cert validation, which end up using both types of client.


Animation using animate.css not working when this plugin is loaded.

If the plugin is loaded on a page where slide is supposed to  animate with animate.css the animation is reset to default slide animation. I used this code to animate:
           animateOut: 'slideOutDown',             animateIn: 'slideInDown',
Does not work with plugin loaded. I don't have to use it on the slider to break the animation.
Thank you for the plugin I love it!
Edit: Using latest Owl carousel 2.0.0-beta.2.4

Hi, works fine for me. Example -> http://codepen.io/gijs/pen/dded8acc210d35774b73c436ef6ef4e3/


[FR] Asterisk-Form OPTIONS Requests

What kind of issue is this?

[X] Feature Request.

Support for the asterisk-form of a request-target.
As specified in https://tools.ietf.org/html/rfc7230#section-5.3 the request-target can take an asterisk-form (*), this is particularly useful when sending OPTIONS requests in order to query the server as a whole, rather than a specific resource.  Currently a url with a path must either be provided (and start with a /) or if none is given a path of / is used, providing no mechanism to query the * resource uri.

Really tough to think through how we’d express this.

Won't fix.


systemd: 247.2 -> 247.3

Motivation for this change
Sync with latest upstream stable release.
Fixes:
1cb171695a sd-device: make TAGS= property prefixed and suffixed with ":"
0e0a165d72 sd-device: keep escaped strings in DEVLINK= property
ca3b974a55 sd-device: use set_strjoin()
041fe7dfee set: introduce set_strjoin()
fc398137fc tools: make update-dbus-docs compatible with Python 3.6
539e67159c man: fix small issue in AllowedMemoryNodes description
eb6910c8ab man: make it clear how systemd calculate the DefaultTasksMax.
ab9f7e1a51 resolved: use reference counting for DnsQueryCandidate objects
91ba2eac4b resolved: minor cleanups
fd76ba69a1 tools: make update-dbus-docs compatible with Python 3.7
c6d30eb104 network: drop wrong flag for neighbor entry
b3465837dd sysusers: flush nscd's caches whenever /etc/{passwd,group} are modified
b7e0ac754e tree-wide: ignore messages with too long control data
a054fe9c89 systemctl: warn when importing environment variables with control characters
288b980fe5 Allow control characters in environment variable values
57bc92bf08 systemctl: print a warning when trying to import a nonexistent variable
64317106ae resolved: fix use-after-free with queries hitting the cache
c97c62ed3d man: clarify what network scopes are
490b9ae9dd rpm: expose $systemd_util_dir also as rpm macro
c77f5629d7 systemctl-edit: Add missing ret_dropin_paths argument in retry path
71630f1187 systemctl-edit: fix abort in find_paths_to_edit()
e12b4112a8 import: mangle untarred OS images after pull-tar, too
d60de7d137 dhcp6: refuse zero length vendor class
a455e20118 dhcp6: refuse zero length dhcp user class
ead71a1a95 network: refuse zero length dhcp user class
5dbb9342a1 dhcp: length of each user class field must be positive
dc9ab43854 journal: send journald logs to kmsg again
f3997dd056 timedate: actually reset system time with new timezone
fc4eae72f8 wifi-util: do not ignore wifi iftype when SSID is not set
3885103672 wifi-util: cleanup header inclusion
b81e441b61 docs: `mesonconf` is not a valid command, `meson configure` is
95ee2c6b48 bpf: do not use structured initialization for bpf_attr
3dcf950663 test-xattr-util: don't insist that /usr supports xattrs
94bb28590b bpf: zero bpf_attr before initialization
6db2ae6618 shell-completion: fix systemctl set/unset/import-environment
d0a124c0af man: improve description of environment block creation
a2f0da2de0 stat-util: don't try to open path on path_is_temporary_fs()
7c63e5ed58 systemctl: have is-enabled return success for aliases when calling into pid1 too
d0b76f0738 man: fix path reference to unit file
1f39070e40 docs: fix the link to boot loader specification
b7db0461a6 network: fix possible memory leak
310fd03e07 resolve: field size in dns resource record may be zero
9401ed294d siphash: introduce siphash24_compress_safe()
5cb414f8c5 fuzzers: set maximum length for several fuzzers
efa8f49344 shared/dns: fix dlopen_idn return code check
4032a13588 man/systemd-nspawn: document hashing machine name for uid base
5dd2b56443 udev: fix memleak
cefb123e8a journal-importer: ignore invalid field at one more place
a580023f1d man/localtime: document default timezone
14475e0e79 man/systemd.netdev: clarify the wireguard AllowedIPs= setting
2a76d510d9 logs-show: refuse data which contain invalid fields
2c53886b4f journal: refuse data which contain invalid fields
b7f69284f1 journal: move journal_field_valid() to journal_file.c
b7171ae4bd test: use modern qemu numa arguments
36bc4a18fd bus-util: improve logging when we can't connect to the bus
a1b1ef65a4 sd-bus: make credential acquisition more graceful
a62421591e sd-bus: 'ret' parameter to sd_bus_query_sender_creds() is not optional, check for it
242fc1d261 network: fix IPv6PrivacyExtensions=kernel handling
2ba904a2e5 network: fix typo
a22eef68c0 dissect: fix root hash signature autodiscovery
6fa5ec5a41 cryptsetup: add support for workqueue options
4275f1c95e test-login: skip consistency checks when logind is not active

Also, nixpkgs-fmt the changes introduced in 494ed4d6ee6f3d3b25a0b5d45f3f9e8dd1c45ccf.
Things done

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

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

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

I added a comment to the top.

If I read correctly the changelog:
242fc1d261 network: fix IPv6PrivacyExtensions=kernel handling

it is now possible to revert the temporary solution of https://github.com/NixOS/nixpkgs/issues/106858

Good catch! I reverted the workaround and successfully ran nixosTests.networking.networkd.privacy.

With #110799 being merged, I pushed some cleanup fixes to remove the now-unused /etc/systemd-mutable/system paths and further simplify the amount of patching.

Aside from the fact that the systemd-mutable/ directory is considered a requirement (or an obstacle if you will) for Dysnomia (that by itself is not something you should consider an impediment for applying this change), it is also used by the Nix process management framework to make deployments of generated systemd units possible, both on NixOS and conventional Linux distributions that have systemd and Nix installed.

I don't think other distributions read from /etc/systemd-mutable/system either, but Dysnomia writes to /etc/systemd/system there. If you want to keep using /etc/systemd-mutable/system, you can still override systemd via systemd.package, to also read from that path, without having to recompile the world - but I don't think it belongs into the "vanilla systemd" we link so many packages with.

If you remove this feature, then there is no longer a mutable directory in which you can store arbitrary systemd units. Managing systemd units can only be done by installing packages in the global Nix profile or the system profile (that themselves are immutable), basically introducing an all or nothing approach when it comes to working with systemd (meaning you always have to use Nix, or you can't change systemd's state).
Although for most the most common NixOS' use cases this is typically fine. I also consider it a useful feature to still have the ability experiment with systemd and deploy experimental systemd units during development, without requiring me to continuously update and redeploy my system configuration. (For example: for certain development projects, I tend to use this directory to regularly check whether my systemd unit is still doing the right thing).

There still is a very mutable location, /run/systemd/system/*, which works for quick testing during development. Some part in dysnomia could also copy things over from some persistent, but mutable location to that place, to avoid having to use a custom systemd package on "dysnomia/nix-processmgmt" systems.

My extra-container utility also requires /etc/systemd-mutable for persistent or auto-starting containers. /run/systemd can't be used in this case.
A NixOS option like systemd.mutable might be a compromise. Let me know if I can help with the implementation.

On 02:21 12.02.21, erikarvstedt wrote:

My extra-container utility also requires /etc/systemd-mutable for persistent or auto-starting containers. /run/systemd can't be used in this case.
A NixOS option like systemd.mutable might be a compromise. Let me know if I can help with the implementation.

Why can't it be used?

Let's move the systemd-mutable discussion to another place. I'll drop the relevant commits from this PR, and will open a new one for that (and link from here).

Why can't it be used?

Because the container service definitions need to persist between reboots.
/run/systemd would be fine for ephemeral containers.

PR is open at https://github.com/NixOS/nixpkgs/pull/112891, and also proposes a solution for persistency across reboots. Let's do the discussion over there.


Add pre-trained of fusion-net's option in config

Add pre-trained of fusion-net's option in config

Sorry, this one also need to be merged.

@rginjapan thanks for your PR. Please only change the fusion-net-part as following. Since other part of the config is not affected by the PR #12. Thanks
  fusion-net:
    name: "fusion-layer-soft"
    requires-grad: true   # only soft-fusion has trainable params
    pretrained: false
    model-path: ""


[Problem]  Malware Detection on VirusTotal 6/71

Describe the issue
The executable exe tests positive for scanning on VirusTotal
The executable msi results clean, but the exe installed from the msi package results positive with 3/69 detections
Some necessary information

System: Windows 11
Version: 2.9.4
Deployment method: releases on GitHub

二进制文件使用 tauri actions 自动构建，你可以尝试自行构建并重新测试，我相信这应该是误报。
workflow: https://github.com/Yidadaa/ChatGPT-Next-Web/blob/main/.github/workflows/app.yml
logs: https://github.com/Yidadaa/ChatGPT-Next-Web/actions/runs/5901771382


Loading a diff from a patch sent via mail

Hey,
I would like to load a patch that was sent via email. I saw that formatting an email is implemented but parsing an email is not, is it? Can I simply use the from_buffer method to load the actual patch into a diff?
Best regards and thanks in advance!!

I didn't know that I can just load the mbox formatted patch the same way as a non-mbox formatted patch. Lesson learned.
So what I'm trying to access is the header information to answer e.g. if the patch was signed off or not. Is there a way to access this already using git2-rs or libgit2? I didn't find a go-to solution in either of those. I would I have to use regex to parse the header?
Cheers