Better argument ergonomics

Looking at this with fresh eyes, this whole bit is dumb:

Note the order of options: There are two sets of options we need to pass.
The first concerns the data we're geocoding. We need to say where our database is and what table we're using, and optionally, how to extract a location query.
Then, we need to say what geocoder we're using, and pass in any options needed to initalize it. This will be different for each geocoder we want to use.

Ideally, it should be something like this:
geocode-sqlite nominatum  data.db data \
 --location="{address}, {city}, {state} {zip}" \
 --delay=1 \
 --user-agent="this-is-me"

Maybe this would be ok:
geocode-sqlite
 --location="{address}, {city}, {state} {zip}" \
 --delay=1 \
 --user-agent="this-is-me" \
nominatum data.db data

But the first way is definitely better.

This would probably do a lot to help #2, since doing geocode-sqlite nominatum alone would trigger a help callback.

Closed via #9


fix(amTimeAgo): Get rid of Angular change detection warning

This should hopefully address the Expression has changed after it was checked issue, see #48 for details.

I also need this! :+1:
@urish
BTW travis error is just DifferencePipe.d.ts[4, 1]: Exceeds maximum line length of 120

@josx seems like this doesn't do the trick, as the time won't update automatically after this change. You can follow the discussion in #48 for details. If you do find a way to make it work, I'd love to have this fixed :)


Add support for TTCN-3

Fixes #1732
Changes proposed:

[x] Add
[ ] Delete
[ ] Fix
[ ] Prepare

@ealap I did a fast check on the svg file and to be honest, it is almost okay but need some fixes according to guidelines. The height is like 29px and hence, it doesn't satisfy the 2px margin for every side.
I did a little fix on the svg but too lazy to clone the branch, so I will let the code here:
<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 32 32"><title>file_type_ttcn</title><path d="M12.086,2c-.718,0-.816.015-.377.062a21.978,21.978,0,0,1,2.389.4,16.45,16.45,0,0,1,7.714,3.824,12.314,12.314,0,0,1,3.786,7.4,15.018,15.018,0,0,1,.173,2.622,10.656,10.656,0,0,1-.039,1.3,14,14,0,0,1-.593,2.767,12.573,12.573,0,0,1-2.129,3.965,13.634,13.634,0,0,1-5.43,4A22.264,22.264,0,0,1,11.2,29.889c-.192.019-.348.038-.3<IP_ADDRESS>.7.057,1.146.068v0c5.659.133,10.343-1.764,13.2-5.347a13.86,13.86,0,0,0,1.336-2.031,15.056,15.056,0,0,0,1.4-4.548A14.655,14.655,0,0,0,25.1,7.18a6.935,6.935,0,0,0-.92-1.04,14.645,14.645,0,0,0-5.655-3.214A21.205,21.205,0,0,0,12.086,2Zm6.68,10.8,0,0,.107.071c-.044-.028-.07-.051-.105-.073Zm.<IP_ADDRESS>v0l-.02-.013Zm.2,3.227,0,0,0,.018,0,0Zm-1.543,3.659-.008.016-.006.016.008-.016Zm-6.3,1.037-.025.02-.02.016.027-.018Z" style="fill:#00adee"/><path d="M4.76,23.949a1.5,1.5,0,0,1-.644-.875,2.1,2.1,0,0,1,.434-1.346A4.823,4.823,0,0,1,5.734,21h7.432c0-.089.9-.338.987-1.112.068-.6.437-1.63-.007-1.82C14.01,18.01,11.224,18,11.224,18l-2.29-.019L8.79,17.9a1.053,1.053,0,0,1-.534-1.028,1.889,1.889,0,0,1,.205-1.053,1.59,1.59,0,0,1,.62-.624c.345-.176.33-.175,2.924-.2l2.253-.02.205-.094a1.509,1.509,0,0,0,.81-1.04,2.31,2.31,0,0,0,.015-1.076,1.13,1.13,0,0,0-.7-.689L10.954,12l-3.442-.016-.158-.069a1.315,1.315,0,0,1-.737-1.3,1.751,1.751,0,0,1,.732-1.53L7.5,9l5.2,0c4.048,0,5.184,0,5.283.031a.9.9,0,0,1,.49.385,1.365,1.365,0,0,1,.044.979c-.139.636-.079.964.223,1.217A1.934,1.934,0,0,0,20,12h2a59.766,59.766,0,0,1-3,8s-.506.01-.993.007c-.536,0-.833.53-1.029.971-.209.472-.748,1.769-.956,2.063a2.7,2.7,0,0,1-1.054.991c-.119.043-.426-.009-4.929,0C5.816,24.035,4.881,23.984,4.76,23.949Z" style="fill:#949494"/></svg>

Cheers!

language ids should ONLY be used when a vscode extension exists that declares and use it.

@ealap I did a fast check on the svg file and to be honest, it is almost okay but need some fixes according to guidelines. The height is like 29px and hence, it doesn't satisfy the 2px margin for every side.
I did a little fix on the svg but too lazy to clone the branch, so I will let the code here:
<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 32 32"><title>file_type_ttcn</title><path d="M12.086,2c-.718,0-.816.015-.377.062a21.978,21.978,0,0,1,2.389.4,16.45,16.45,0,0,1,7.714,3.824,12.314,12.314,0,0,1,3.786,7.4,15.018,15.018,0,0,1,.173,2.622,10.656,10.656,0,0,1-.039,1.3,14,14,0,0,1-.593,2.767,12.573,12.573,0,0,1-2.129,3.965,13.634,13.634,0,0,1-5.43,4A22.264,22.264,0,0,1,11.2,29.889c-.192.019-.348.038-.3<IP_ADDRESS>.7.057,1.146.068v0c5.659.133,10.343-1.764,13.2-5.347a13.86,13.86,0,0,0,1.336-2.031,15.056,15.056,0,0,0,1.4-4.548A14.655,14.655,0,0,0,25.1,7.18a6.935,6.935,0,0,0-.92-1.04,14.645,14.645,0,0,0-5.655-3.214A21.205,21.205,0,0,0,12.086,2Zm6.68,10.8,0,0,.107.071c-.044-.028-.07-.051-.105-.073Zm.<IP_ADDRESS>v0l-.02-.013Zm.2,3.227,0,0,0,.018,0,0Zm-1.543,3.659-.008.016-.006.016.008-.016Zm-6.3,1.037-.025.02-.02.016.027-.018Z" style="fill:#00adee"/><path d="M4.76,23.949a1.5,1.5,0,0,1-.644-.875,2.1,2.1,0,0,1,.434-1.346A4.823,4.823,0,0,1,5.734,21h7.432c0-.089.9-.338.987-1.112.068-.6.437-1.63-.007-1.82C14.01,18.01,11.224,18,11.224,18l-2.29-.019L8.79,17.9a1.053,1.053,0,0,1-.534-1.028,1.889,1.889,0,0,1,.205-1.053,1.59,1.59,0,0,1,.62-.624c.345-.176.33-.175,2.924-.2l2.253-.02.205-.094a1.509,1.509,0,0,0,.81-1.04,2.31,2.31,0,0,0,.015-1.076,1.13,1.13,0,0,0-.7-.689L10.954,12l-3.442-.016-.158-.069a1.315,1.315,0,0,1-.737-1.3,1.751,1.751,0,0,1,.732-1.53L7.5,9l5.2,0c4.048,0,5.184,0,5.283.031a.9.9,0,0,1,.49.385,1.365,1.365,0,0,1,.044.979c-.139.636-.079.964.223,1.217A1.934,1.934,0,0,0,20,12h2a59.766,59.766,0,0,1-3,8s-.506.01-.993.007c-.536,0-.833.53-1.029.971-.209.472-.748,1.769-.956,2.063a2.7,2.7,0,0,1-1.054.991c-.119.043-.426-.009-4.929,0C5.816,24.035,4.881,23.984,4.76,23.949Z" style="fill:#949494"/></svg>

Cheers!

thanks! I will try to implement this.

language ids should ONLY be used when a vscode extension exists that declares and use it.

Actually, there is an extension for TTCN-3. It's a syntax highlighter which I made :)

Actually, there is an extension for TTCN-3. It's a syntax highlighter which I made :)
Then why didn't you mentioned it in your OP? The template is there for a reason.

Is there an additional step I need to do to implement this with language ids? I am getting this debug prints and I feel like it's related to your comment.

No. Other than what you did, it's fine, Can't think of reasons for the debug prints you are seeing.

I think it would be caused by this line:
 {
      icon: 'ttcn',
      extensions: [],
      languages: [languages.ttcn],
      filename: true, <-- // This line
      format: FileFormat.svg,
 },

As stated on the comments inside IFileExtension:
"filename?: boolean; // set to true if the extension represents the whole file name."
The extension doesn't represent the whole filename since you have files with some name followed by the .ttcn extension. Also, you should put the other two extension (ttcn, ttcnpp) on the Extensions Array like as: extensions: ['ttcn', 'ttcnpp'].

@KingDarBoja Nice catch there, but you are wrong about adding the extra file extensions, as that will defeat the point of adding the language id. By using language id to associate the icon it's the vscode extension's responsibility to declare the file extensions it supports (like done in the said extension)

@JimiC My fault, didn't checked the provided extension and assumed the vscode-icons to work in another way. Now I learned something new regarding how this vs code extension works.

entioned it in your OP? The templa

Right, I should have updated that. I created that ticket last year before I even started creating the extension that's why I left it blank.

As it turns out the debug print was caused by another extension! Oops! Sorry about that. But at least I get to more understand how the extension works, too. I got rid of the incorrect setting 'filename:true'.


[Task] Finalize YouTubeDNN retrieval model

Problem:
YouTubeDNN retrieval model is the most popular approach for recommendation domains like e-commerce, media, news, which  either suffer from user-cold start problem or where users preferences change a lot over time.
YouTubeDNN differs from other retrieval models like TwoTower and MF, as it represents the user by the sequence of his last interactions, rather than trying to learn embeddings for the user.
YouTubeDNN is going to be the first model that supports sequences of interactions in Merlin Models, providing building blocks that will be useful for future RNN and Transformers support.
Most of the work for full support of YouTubeDNN is done, but there are still some pending tasks.
Goal:

Finish the YouTubeDNN retreival model in order to

Start support sequential interactions in Merlin Models, as expected by many our customers, as a first step towards upcomming support for more advanced sequential and session-based recommendation.
Enable us to provide a more comprehensive benchmark of retrieval models, i.e. comparing YouTubeDNN with MF and Two-Tower.

Constraints:

The YouTubeDNN should inherit from 'RetrievalModel' like MF and Two-tower, so that the evaluation metrics can be computed over all items using TopkIndex.
The current YouTubeDNN implementation uses a popularity-based sampler for sampled softmax, that assumes that the item id was categorified so that the ids are sorted by frequency decreasingly (like done by default by NVTabular Categorify() op)
The target generation is done as a pre-processing step using Pandas (set different cutting points of the past user's interaction and reserve the last item as the target)

Starting Point:

[ ] https://github.com/NVIDIA-Merlin/models/issues/496
[ ] https://github.com/NVIDIA-Merlin/models/pull/473
[ ] https://github.com/NVIDIA-Merlin/models/pull/454
[ ] https://github.com/NVIDIA-Merlin/models/issues/540

The problem statement doesnt' articulate what customer value is being provided

@viswa-nvidia I have refined the problem description a bit to highlight the value of the YouTubeDNN support

The planned tasks don't seem to cover the problem statement. @EvenOldridge , please review

@marcromeyn , what is this task blocked on ?


Better guidance on how to set up XR management in MRTK 2.3.0 and Unity 2019

Describe the issue
I've recently upgraded to Unity 2019.3.3f1 and MRTK 2.3.0. Unity tells me that Windows Mixed Reality is deprecated and will be removed. Since we're starting a new project that may last a while, we want to move to the new XR Management early to be prepared. Unfortunately, I ran into an issue upgrading my WMR based project to the new XR packages.

Even if I use the DefaultXRSDKConfigurationProfile, the app gets locked up when trying to run it on HoloLens 2. It is not clear what the Default profile is referring to: VR, HoloLens 1, HoloLens 2? It seems at least that the default camera is set up with a Skybox, which seems that it's rather set up for VR than for AR.

It would be great to extend the documentation to make it clearer how to upgrade an existing MRTK project to the new XR packages:
https://microsoft.github.io/MixedRealityToolkit-Unity/Documentation/GettingStartedWithMRTKAndXRSDK.html

It would be nice to know what developers should set up for:

Input Helpers
XR Tracking

In addition, I got stuck when trying to update my existing MRTK profiles: A table shows which legacy XR profiles have to be updated with the new XR SDK profiles, however, for somebody who is not super familiar with the nuances of the different types of profiles, it's difficult to know where to exactly find and change them.
https://microsoft.github.io/MixedRealityToolkit-Unity/Documentation/GettingStartedWithMRTKAndXRSDK.html#required-in-mrtk

I find myself in the same situation. I followed the steps in the aforementioned documentation, but when deploying to the HoloLens 2, all my scene content becomes head-locked and I'm not sure what I need to do to get tracking working as before.

Same situation. for unity 2019.3.6f1 with 2.3.0 MRTK, XRSDK.

Even if I use the DefaultXRSDKConfigurationProfile, the app gets locked up when trying to run it on HoloLens 2.

This is currently a known issue with the latest preview WMR package for XR SDK. It's been fixed internally and will ship with the next version. I'll try to get a date for the next version

It is not clear what the Default profile is referring to: VR, HoloLens 1, HoloLens 2? It seems at least that the default camera is set up with a Skybox, which seems that it's rather set up for VR than for AR.

The camera profile allows you to specify the skybox and clearflags per-transparent or opaque device. I'll double check that this is set up correctly. Otherwise, the profile should work across all three.

It would be nice to know what developers should set up for:
Input Helpers
XR Tracking

I'll make this more explicit in the docs. You shouldn't have to do anything with Input Helpers, and I think the same is true for XR Tracking but I'll double check.

A table shows which legacy XR profiles have to be updated with the new XR SDK profiles, however, for somebody who is not super familiar with the nuances of the different types of profiles, it's difficult to know where to exactly find and change them.

Will update this too!

As a temporary workaround, you should be able to regain positional tracking by covering the sensors with your hands, forcing the device to fully lost tracking, then remove your hands to allow it to regain tracking.

Could we get a temporary code fix for this ahead of the 2.4 release?

This is currently a known issue with the latest preview WMR package for XR SDK. It's been fixed internally and will ship with the next version. I'll try to get a date for the next version.

Update! Unity shipped a new preview package with this fix. At the moment, the preview package is waiting on the next version of 2019.3 to be successfully deployable to an HL2. As far as I'm aware, things should align then.

@keveleigh it looks like part of the issues here have been addressed in Unity's latest packages - what still remains on this particular issue?

@wiwei I still want to take the action item of improving the MRTK + XR SDK docs a bit as a result of this feedback.

I find myself in the same situation. I followed the steps in the aforementioned documentation, but when deploying to the HoloLens 2, all my scene content becomes head-locked and I'm not sure what I need to do to get tracking working as before.

I have the same problem with Unity 2019.3.10.f1, MRTK 2.3.0 and Windows XR Plugin 2.1.5 but on HoloLens 1. Is there a fix for this already, and if so, what do I have to update to make this work?

I have the same problem with Unity 2019.3.10.f1, MRTK 2.3.0 and Windows XR Plugin 2.1.5 but on HoloLens 1. Is there a fix for this already, and if so, what do I have to update to make this work?

Are you using the MRTK default XR SDK profiles, or your own custom profiles? If you're using your own profiles, you'll want to make sure GenericXRSDKCameraSettings is present in your camera profile. With either the default MRTK profiles or your own, you'll also want to double check that the XR Legacy Input Helpers package is installed. Windows XR Plugin 2.1.5 should import it as a dependency, but it's good to double check.

Thank you, I was indeed missing the GenericXRSDKCameraSettings in my custom profile :) After adding it, everything works now.

@cgsaxner Excellent! I'll work on updated docs today, since the existing table isn't the most clear as far as what's required

I updated our docs with the new XR Management flow and package requirements. Looking through here, I think all questions are updated either by the new docs or by certain things (Input Helpers and XR Tracking, for example) being removed by the new XR Management version

@brandf If I keep the Default XR SDK Settings on the Toolkit everything seems to work fine. If I change the camera or the input section of the settings, I get "2D" headlocked content. Maybe it helps! Took me two days to figure out.


Update AKS base image to 0.14.0

What this PR does / why we need it: This is a smaller base image (30 GB) to allow for expanded backwards-compatibility w/ pre-configured OS disk sizes. This images includes a reduced Kubernetes hyperkube versions matrix (only 1.7 and above, and only the latest two minor patch versions).
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
If applicable:

[ ] documentation
[ ] unit tests
[ ] tested backward compatibility (ie. deploy with previous version, upgrade with this branch)

Release note:

Update AKS base image to 0.14.0

/lgtm


Dead link in your document

I'm searching help for this:
[Violation] Avoid using document.write()
The site: https://developers.google.com/web/updates/2016/08/removing-document-write
The link "maintaining a set of known services that provide asynchronous loader support" is dead:  https://developers.google.com/speed/docs/insights/UseAsync (404)
Thx!

Thanks for filing the issue, it seems to be a Duplicate of #7832

Yeah, it seems so.


Deploying the smart contracts on custom network

Do you have any instructions/guidance for deploying all the smart contracts on a new network (other that localhost/ropsten/rinkeby/mainnet) such as polygon?
Any help is appreciated.

I'd recommend taking a look at the hardhat docs for polygon configuration. Goldfinch is only deployed currently on ethereum mainnet, and have not deployed contracts to any other chains.
https://docs.polygon.technology/docs/develop/hardhat/

What about rinkeby? That would still help quite a lot. Has goldfinch ever deployed their smart contracts on rinkeby network?

follow up on this. Do you have some suggestions for deploying all the smart contracts on rinkeby/goerli/any other network?
There is a multitude of smart contracts and external dependencies, so it's hard to tell the exact setup that needs to be done so that there are no errors occurring. I've just tried to deploy everything on rinkeby, but it fails heavily...

This is one of the errors that I'm getting:
failed to get chainId, falling back on net_version...
An unexpected error occurred:

ProviderError: Must be authenticated!

  [RINKEBY_CHAIN_ID]: {
    safeAddress: "0x618C20c64cAc5211E099D355ba213790708e7462",
    executor: "0xeF3fAA47e1b0515f640c588a0bc3D268d5aa29B9",
  },

Does the executor address need to change as well or just the safeAddress?

I believe just the safe address because getProtocolOwner returns that field here

You may need to deploy your own gnosis safe on rinkeby to make this work as I think our deploy scripts build a transaction for a gnosis safe but I would give editing that to your EOA first and then use a gnosis safe.

This error literally happens on every single contract, but only the first time. If you run it again, deploying the contract will succeed but the next one will fail with the same error.

to better understand what i mean:
About to deploy Fidu...
----- THE ERROR OCCURRS -----
I run it one more time, I get
About to deploy Fidu...
Fidu was deployed to: 0x85F97dFf419216F65A416032744bB8bCfcA44baA (9.712kb)
About to deploy Pool Tokens...
----- THE ERROR OCCURRS -----
I run it one more time, I get
About to deploy Fidu...
Fidu was deployed to: 0x85F97dFf419216F65A416032744bB8bCfcA44baA (9.712kb)
About to deploy Pool Tokens...
PoolTokens was deployed to: 0x9aB3cfeA6f849a2106b2D3874D6d3Cb8b24cbcdC (2.286kb)
About to deploy TransferRestrictedVault...
----- THE ERROR OCCURRS -----
Any idea what could be the issue? Seems that re-running solves the issue, but you need to re-run couple of times to get rid of all the errors.

When looking at that address that it's saying poolTokens is deploying to it looks like it's using the existing deploys that we did a long time ago on rinkeby. You may need to delete all of the manifests under packages/protocol/deployments/rinkeby so that fresh contracts get deployed.

That solved the problem! Will let you know if I encounter any other errors. Thank you so much.
I'll make sure I document all the deployment process after I finish.

Bumping into
An unexpected error occurred:

Error: ERROR processing /Users/constantindogaru/weavik/mono/packages/protocol/deploy/baseDeploy.ts:
Error: replacement fee too low (error={"name":"ProviderError","code":-32000,"_isProviderError":true}, method="sendTransaction", transaction=undefined, code=REPLACEMENT_UNDERPRICED, version=providers/5.4.5)

This is the most recent blocker.

Also, getting Fail with error 'Contract instance has already been initialized' if I run the script twice.
Borrower was deployed to: 0x1b4F0d7c2d1782f79Ee085f2B04B6930d8F47e9E (10.023kb)
FixedLeverageRatioStrategy was deployed to: 0x643eDB1AEa8b9Ec2C4664606093f6Dee9CAD89BF (6.213kb)
An unexpected error occurred:

See https://rinkeby.etherscan.io/tx/0xe3988196b98aa4c5e1b2ce01ba2f367c920d5758c7d32c1b1fb5cc5e203a3e75.

Another one which happens on line await deployEffects.executeDeferred():
ERROR:
Error: Safe Proxy contract is not deployed in the current network

You may need to deploy your own gnosis safe on rinkeby to make this work as I think our deploy scripts build a transaction for a gnosis safe but I would give editing that to your EOA first and then use a gnosis safe.

You may need to deploy your own gnosis safe on rinkeby to make this work as I think our deploy scripts build a transaction for a gnosis safe but I would give editing that to your EOA first and then use a gnosis safe.

I've tried to use a gnosis safe address from Rinkeby and I get the following error: Fail with error 'Must have admin role to perform this action'
https://rinkeby.etherscan.io/tx/0x17cad6d0badcfaa8323ef8e189172cd0bed1e2dc71fee611bef7dc8242bd93bc
Any idea how to grant permissions?

Hey @ctindogaru I think I found the problem
deployConfig does not use deployEffects for setting the initial config values. Instead it will make the transactions directly with the deployer account.
A quick overview on deployEffects: You can think about it is as a queue of transactions that will all be bundled together and executed as a single transaction. when you call add or changeImplementations it queues a transaction that will then be executed along with every other transaction. I'm actually not sure how the transaction will appear. In our current setup a proposal appears in our Defender instance. I think for it to work you need to have DEFENDER_API_KEY in your environment with a valid api key that can create proposals.
In order to get around this problem you'll need to pass deployEffects to basically every deploy___ function in baseDeploy.ts that doesn't currently take it. For every transaction that happens (besides constructors) you'll need to deployEffects.add( that transaction.
here is an example of what that looks like.


Add Language Name Param to AutocompleteTemplate

Description
Language name was previously added as a parameter only for string autocomplete templates. This change provides this same information to template functions that implement the AutocompleteTemplate interface.
Language name is explicitly supported for the CodeGeeX4 autocomplete template, and is added here in this change. Explicit language awareness could also be added for other templates in a future change, like the holeFillerTemplate used for models without explicit completion support.
Checklist

[x] The base branch of this PR is dev, rather than main
[x] The relevant docs, if any, have been updated or created

Screenshots
[ "For visual changes, include screenshots." ]
Testing
This change specifically targets CodeGeeX4, so modify config.json with the following:
  "tabAutocompleteModel": {
    "title": "Tab Autocomplete Model",
    "provider": "ollama",
    "model": "codegeex4:{quant}"
  },

I'm running the quant 9b-all-q6_K, but should work similarly with other quantizations.

Shoutout to @TheLapinMalin for bringing this to my attention in #1827!


TEST

Context
Changes proposed in this pull request
Guidance to review
Important business

[ ] Does this PR introduce any PII fields that need to be overwritten or deleted in db/scripts/sanitise.sql?
[ ] Does this PR change the database schema? If so, have you updated the config/analytics.yml file and considered whether you need to send 'import_entity' events?
[ ] Do we need to send any updates to DQT as part of the work in this PR?
[ ] Does this PR need an ADR?

NB: Please notify the #twd_data_insights team and ask for a review if new fields are being added to analytics.yml

Hi @saliceti do we still need this?

Hi @saliceti, would it be ok to close this PR now?

Yes, the test was successful. Closing


Forced SourceTableView has significant performance implications

In many ledger entry pages, the forced setting of a key in the SourceTableView property is causing significant performance decreases. Opening a page with the wrong key takes more than a minute instead of less than one second, if the correct key is being used.
For example: in the page 29 "Vendor Ledger Entries", the sorting key is set to "Entry No.". If this page is opened from the vendor card, then the filter is applied to the vender no. and better keys are available. However, the SourceTableView property takes precedence and sorts the filtered results without considering the better secondary key, causing the significant performance impact.
This applies to all ledger entry pages, where a SourceTableView was set, including (but not limited to):

page 20 "General Ledger Entries" (SORTING("G/L Account No.", "Posting Date"))
page 25 "Customer Ledger Entries" SORTING("Entry No.")
page 29 "Vendor Ledger Entries" SORTING("Entry No.")
page 5604 "FA Ledger Entries" SORTING("Entry No.")

Creating a separate page without the key in the SourceTableView is not an option, because hardly any code lines that call the standard page offer an option to overwrite this behaviour. This is also not a feasible approach for such a fundamental problem in so many different pages. One "acceptable" alternative could be the possibility to overwrite the SourceTableView property in a pageextension, but even this does not seem feasibly in this case.
Please remove the unnecessary SourceTableView from these pages.

I wholeheartedly agree - Microsoft should not "force" a key on its customers / partners. In terms of encapsulation the SourceTableView property should be made available for read/write access.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 43051.


Align with Messenger Platform v2.0 release

https://developers.facebook.com/docs/messenger-platform/changelog

Part of 2.2.0 release.


update the index.ts on "How-to Guides" EKS Page

File: themes/default/content//registry/packages/kubernetes/how-to-guides/eks
The code snippet of "index.ts" is obsolete in the following step "Create a new EKS cluster-> Open the existing file index.ts, and replace the contents with the following"
please update the code snippet.

Thanks for letting us know!


[Feature Request]: Add support for pd.get_dummies

What would you like to happen?
Part of #20958
pd.get_dummies is a non-deferred column operation that we could implement for columns with a categorical dtype.
Issue Priority
Priority: 2
Issue Component
Component: dsl-dataframe

Duplicate of #22268


Variable kube-rbac-proxy image and resources in helm chart.

See https://github.com/redhat-cop/proactive-node-scaling-operator/issues/15


shibuqam: provide a basic auth web service to simplify the use of shibuqam

Configuring a shibboleth client is a PITA.
So why not configure it once with an simple auth service that allows servers to delegate their authentication to it?
This is the point of the new server shibuqamoauth.
Please review because I'm not a security expert and I do not want to get to much exploitable flaws.

ping @ppepos @Morriar @xymus

IMHO, instead of reinventing the wheel, it is better to implement an existing authentication protocol (example: OAuth). There are a lot of non-obvious attacks that experts that design these protocols take into consideration. The problem with security is that developers has to care many aspects to make their software secure while the attacker has only to find one vulnerability to break the whole system.
I normally don’t care a lot about security for Nit PRs, but since the service would handle sensitive data about UQAM’s students, I would prefer to play safe.

What I implemented is, in fact, basically the authorization code grant of OAuth2  https://tools.ietf.org/html/rfc6749#section-4.1
I just get rid of:

response_type because there is only a single kind of grant here
client_id because I do not want to code a db of clients. The redir can be seen as a simple client_id. I have also no way to present the client to the user since I do not control the authorization page of uqam/shibboleth (beside, the authentication is done by the reverse proxy).
scope since there is only one kind of access request
authorisation_code because I have no access to anything. I just get minimal information after the successful shibboleth login. Thus I have nothing more to authorise once the user is authentified.

Nevertheless, I just saw a very big and evident flaw in the current protocol. I will update the PR soon. (bonus point for those that also see the flaw)

The authorization code MUST expire shortly after it is issued to mitigate the risk of leaks.
So all data in the redirection URI must be temporary tokens.

beside, the authentication is done by the reverse proxy
In fact, it is done by HTTPS. :)

major reroll. updated the documentation a lot

+1

+1


Stand Alone and Cloud One API Keys could be leaked

Make sure to remove the .csv file generated by the tool for AWS keys and the .yaml config files for Conformity API Keys from the local system after all the migration is done to prevent future leakage of the keys.

I don't know if will be a good idea to remove after the migration, since it can be execute multiples times for several accounts. What we already have that is helpful is a task to clean the legacy account but not to delete the local files, since is easier to the user executing the tool to delete the files I don't know if there is too much value in the tool to do that...
Thoughts? @atiradocc @ronald-bautista

I was thinking about just printing a warning message at the end of each run of the tool that says something like 'The file user_config.yml contains your API Keys, don't forget to delete this after migration.'

I was thinking about just printing a warning message at the end of each run of the tool that says something like 'The file user_config.yml contains your API Keys, don't forget to delete this after migration.'

Make sense to me!

Hi, did we have any update on this?

It has been fixed on the release v0.13.0


Request: Method to customize default texts

Hello and thank you very much for developing this module, I have a small request to be able to customize the texts that are shown in the search boxes such as "State", "Search State", "Search Country" and "Search city" this would allow internationalization correctly these labels and increase the potential of this module. Thank you

Hi @soullest, thank you suggesion. Definately we will add this feature in library. You will receive this feature in next version

Wow! Thanks for your really fast answer, it's much appreciated!

Hi @soullest, now this option is available in csc picker you can use it as shown below
 ///placeholders for dropdown search field
                  countrySearchPlaceholder: "Country",
                  stateSearchPlaceholder: "State",
                  citySearchPlaceholder: "City",

                  ///labels for dropdown
                  countryDropdownLabel: "*Country",
                  stateDropdownLabel: "*State",
                  cityDropdownLabel: "*City",


Support color correction for img2img and inpainting

Fixes #552
Perform image color correction, borrowed from https://github.com/Pyr-000/YetAnotherStableDiffusion/commit/c7bf958c2dfa88fc027690983fee1f877e8827e9
Note that the correction needs to be done against a stable initial image when running img2img or inpainting multiple times. If the correction is done only against the previous image, you end up with everything turning cyan, as in https://github.com/AUTOMATIC1111/stable-diffusion-webui/issues/481
Using tests from that ticket, before correction:

After correction:

Thanks, rebased against development. Seems my test script needed to be changed as it was hitting a new issue: https://github.com/lstein/stable-diffusion/issues/628

#628 is fixed by PR #654, which should go in very soon. I'll run some testing on your PR tomorrow (Sunday) and if all goes well it'll go into dev. Thanks so much for the rebasing!

Thanks! Yeah I saw that as well, I thought that was further artifacts of the diffusion process. Could validate that by skipping the actual processing and seeing if the artifacts appear still?


Improve the upload server.

This pull request depends on the Queue-related pull request sent a few minutes earlier.

Merged directly.


Hello, do you consider making multiple cross-platform functions such as Windows and Mac?

Is your feature request related to a problem? Please describe.
When the monitoring server network cannot find a problem and needs to be monitored for Windows and Mac client client requests, there is no simple and efficient monitoring tool.
Describe the solution you'd like
Can this tool be made into a cross-platform, such as a client monitoring tool on Windows and Mac?
Describe alternatives you've considered
I haven't found a better network exclusion tool like kyanos for the time being.
Additional context
None for the time being.

Thank you for your feedback. This feature is good, I need to do some research on it.

Thank you for your feedback. This feature is good, I need to do some research on it.

Please reply

Thank you for your feedback. This feature is good, I need to do some research on it.

Please reply

Don't worry, I also think this feature is valuable, but it will take some time to implement.

Thank you for your feedback. This feature is good, I need to do some research on it.

Please reply

Don't worry, I also think this feature is valuable, but it will take some time to implement.

Thank you

It looks like we need to integrate this project: https://github.com/iovisor/ubpf.

Thanks, i'll have a look!


Fix for issue #104

Fix for issue #104:
FNH should mark ReferenceComponent as mapped when it is overridden in IAutoMappingOverride

Is anybody here?


Create Kubernetes YAML for Couchbase Server

What this PR does / why we need it:

Contains YAML for creating a StatefulSet
Contains YAML for creating a Headless Service and NodePort Service.

Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Fix for #709
Special notes for your reviewer:
Signed-off-by: yudaykiran<EMAIL_ADDRESS>

Changes Implemented.


Add macro to declare global allocator

Motivation

Future Work

Current dependencies on/for this PR:

main

PR #12   👈

This comment was auto-generated by Graphite.


Fix conversational_retrieval combining with chat history affects the question and answer experience

Fixes # (issue)
While using the ConversationalRetrievalChain, and specifically applying both condense_question_prompt and combine_docs_chain_kwargs parameters, I followed the official documentation to pass chat_history when invoking this chain. The program runs as expected, but the Q&A experience is less than optimal. I've identified two main issues:
When summarizing the question using condense_question, the response prompt generated by load_qa_chain is treated as 'Human' rather than 'AI'. Currently, I am remedying this by determining whether to pass the question to human_message or ai_message, based on whether chat_history is provided through an external interface.
Upon reviewing the code, I noticed that the answer is first generated by self.question_generator, but in reality, the desired answer initially comes from the prompt in combine_docs_chain_kwargs. However, this response is then re-fed into self.combine_docs_chain, leading to a situation of self-posed questions and self-given answers, which isn't ideal. The final answer from combine_docs_chain isn't what I'm looking for, and I see no need to go through another loop only to end up with an unsatisfactory answer. Thus, I wish to adjust the logic for obtaining the answer in the call and acall methods.
While my implementation might not be the most ideal, it considerably enhances the Q&A experience based on chat history in my project. I hope for official support on this issue and appreciate your help. Thank you!
Before submitting

@hwchase17 Kindly request you to please take a look.


Add section about using Custom Events: GA4 Web

Proposed changes

Add excerpt from Google's docs on when to use custom event
Add {:target='_blank'} to open new browser window when customers click on the link
Changed name of subsection from "Custom Event Naming" to be "Custom Events" instead
Added a couple words in front of the existing paragraph since I removed "Naming" from the subsection heading

Merge timing

ASAP once approved

Hi @sarahrudy - I saw that there was a few changes that were present on this branch before you merged develop into this branch. Did you mean to remove those changes?

Hi @forstisabella - yes! I removed those changes intentionally. It reads as it should now, I believe.


HOSTEDCP-1256: control-plane-pki-operator: add a CSR flow for break-glass creds

In order to acquire break-glass credentials, end users will not have access to the management plane from which they can read certificates. Instead, we expect them to create (by proxy) a CertificateSigningRequest that we will sign, if it is approved.
/cc @sjenning @csrwng @enxebre

/approve
/lgtm
/retest-required
Lets try to get this in before the branch

/cherry-pick release-4.15


Implement remainder verification procedure using probabilistic argument

In what follows we make the assumption that the blow-up factor is a fixed constant equal to $8$.
The procedure that we want to implement, call it verify_remainder, takes as input:

$l$ the length of the final domain i.e. the length of the remainder codeword.
The memory address ptr of the word containing the first two coefficients of the remainder $q_0,q_1$. The remaining coefficients are stored in the subsequent addresses ptr + i for $i\in [1,\frac{l}{2} - 1]$. The remainder polynomial coefficients $p_i$ are stored in the subsequent memory addresses i.e. ptr + i for $i\in [\frac{l}{2}, \frac{l}{2} + \frac{\frac{l}{8}}{2}-1]$. Finally, the challenge $\tau$ is stored in address $\frac{l}{2} +\frac{\frac{l}{8}}{2}$.

Using the above, the procedure computes two quantities:

$$\beta := \frac{\sum\limits_{j=0}^{l-1}\frac{\omega_j}{\tau - \omega_j}q_j}{\sum_\limits{j=0}^{l-1}\frac{\omega_j}{\tau - \omega_j}}$$ where $\omega_j := \omega^j$ and $\omega$ is the inverse of the final domain generator which can be computed from the initial domain generator $g$ as $$g^{N^{nlayers}}$$
$$\alpha := \sum\limits_{i = 0}^{\frac{l}{8}-1} p_i \tau^{i}$$ using Horner's rule. To do this efficiently, it is better to have the coefficients of the remainder polynomial stored in descending order in memory.

Finally, the procedure asserts that $\alpha = \beta$.
To compute $\beta$ efficiently, non-deterministic inversion in the extension field has to be implemented i.e. #591 .

I wonder if we should simplify this a little. Specifically, instead of making length $l$ an input parameter, let's fix the length at 64. So, the procedure could be verify_remainder_64. My thinking is that knowing size of the domain upfront will simplify procedure logic quite a bit. Once we have it working for one size, I think creating procedures for other sizes would not be too complicated and we could create verify_remainder_128 etc. (I don't think we'll need more than a handful of these once we find the optimal size).
So, for remainder of size 64, size of $q$ would be 64, and size of $p$ would be 8 - right?
Another thing I wonder is whether $p$ should be already assumed to be in memory, or whether procedure should "read it in" non-deterministically. And similarly for $\tau$ - maybe it should also be something that the procedure computes internally.
Assuming the above works, the only parameter for verify_remainder_64 would be ptr which holds the address of the first two coefficients of $q$. Then, the procedure would need to do the following:

Non-deterministically get $\tau$ from the advice provider.
Compute $\beta$ using the formula described in the original description.
Get coefficients of $p$ from the advice provider.
Compute $\alpha$ using the formal described in the original description.
Verify that $\alpha = \beta$.
Verify that $\tau$ was provided correctly. To confirm my understanding, $\tau$ is computed by sequentially hashing elements of $q$ and $p$ - right?

Assuming the above is correct, we'll need to add a new decorator the advice injector decorators. This decorator would take domain length $l$ and memory address $i$ as inputs and perform the follwoing:

Interpolate values in mem[i], ..., mem[i + l - 1] into polynomial coefficients.
Injected these coefficients into the advice tape.
Compute $\tau$.
Inject $\tau$ into the advice tape.

The name of decorator could be adv.invntt - but I'm open to other suggestions.

I wonder if we should simplify this a little. Specifically, instead of making length an input parameter, let's fix the length at 64. So, the procedure could be verify_remainder_64. My thinking is that knowing size of the domain upfront will simplify procedure logic quite a bit. Once we have it working for one size, I think creating procedures for other sizes would not be too complicated and we could create verify_remainder_128 etc. (I don't think we'll need more than a handful of these once we find the optimal size).

Yes, I agree but then, if the initial domain size is not fixed, we would have to implement both versions. This is because of what we discussed some time ago which is that for folding factors higher than $2$ the remainder length can take different values. Off the top of my head, I think the number of possibilities is $h$ where $2^h$ is the folding factor. My thinking is that, with the layout I have used for both $p$ and $q$, the delta from a specific procedure to one that is generic is relatively small.

So, for remainder of size 64, size of $q$ would be 64, and size of $p$ would be 8 - right?

That's is exactly right.

Another thing I wonder is whether $p$ should be already assumed to be in memory...

I am assuming that the calling procedure would read the $q$ from the advice tape, followed by $p$ (which is put on the advice tape by a decorator), and lastly $\tau$, and then put these in the configuration I laid out above. My thinking is that this would make hashing, in order to get $\tau$, more straight forward but I like the neatness of your proposal.

To confirm my understanding, $\tau$ is computed by sequentially hashing elements of $q$ and $p$ - right?

That is correct.

Yep, I remember that. So, for folding factor = 4, once we figure out optimal remainder length, we'll only have to implement 2 procedures. E.g., it could be for size 64 and 32, or for 128 and 64 etc.

That's correct.

I actually think this will be non-negligible because if we know the size, we don't need any conditional logic. But if we don't know the size we need to run a while loop and for each iteration we need to increment the counter and perform inequality comparison. This could add up.

I agree, a while loop is unavoidable. Then I am onboard.

Yep, makes sense. I assumed that would be constructed by evaluating individual queries, and it wouldn't come from the advice tape. Then, we'd compute sequential hash of the and compare it to the commitment in the proof. Would this work, or am I missing something?

Here is how I am thinking about it and let's forget about the hashing for this part. The calling procedure (of fri_verify) would read $q$ from the advice tape and lay it out in memory starting from ptr. Then, for each query that we want to verify, at the last step of a query verification we would get an index (i.e. the folded position) in the range $[0,l-1]$ which we can use, together with ptr, to compare the output of the last fold_4 with q[index].
This is equivalent to this part.

Sounds great, I have mentioned this in novifinancial/winterfell#126

I add advice provider for iNTT over qudratic extension field in https://github.com/0xPolygonMiden/miden-vm/pull/598

I implement verify_remainder_64 routine in https://github.com/0xPolygonMiden/miden-vm/pull/644


Document Regex usage

We should show how regex use works. Probably best as an example as opposed to in the readme.

Should be fixed now, new example with regexp in the readme, released in v2.2.1.


feat: add Web Support

Summary
Adding picker support for react-native-web
Test Plan
run yarn install in example-web and run yarn web

What's required for testing (prerequisites)?
What are the steps to reproduce (after prerequisites)?
Compatibility

OS
Implemented

iOS
❌

Android
❌

Web
✅

Checklist

[x] I have tested this on a device and a simulator
[x] I added a sample use of the API in the example project (example/App.js)

This PR is also dependant on RN version being 0.61.5+, so #71 needs to be merged first

@jainkuniya
This will be helpful for project made for react-native-web https://github.com/necolas/react-native-web, which allows using react-native code in the web.
It's also great when making a component library for react-native with storybook .
Support for Picker already existed in the react-native-web, but since picker is being extracted from the core react-native, its web should also be extracted from react-native-web to this repository.
I've made an example-web folder where you can test this component in web.
Here's a screenshot of it

+1 for this feature https://github.com/react-native-community/react-native-picker/pull/76
It's very useful for Expo users

:tada: This PR is included in version 1.6.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Silence build tools to prevent garbage output

When using shelljs' exec function, the stdout output of the executed
process is mangled with the stderr output.
This causes deprecation warnings like util.print: Use console.log instead to appear e.g. in out/remark.min.js when using an up-to-date
nodejs binary (v0.12.2) after running node make minify.
Using the -o output file option solved this problem easily because
it circumventes the broken shelljs output handling.
The same problem arises when running the less compiler.
Trying to upgrade the less compiler to the latest version did remove
the warning, in exchange for a new one:
The compress option has been deprecated. We recommend you use a
dedicated css minifier, for instance see less-plugin-clean-css.html

This message appeared in the generated src/remark/resources.js file
after running node make bundle.
That's easy to remedy by using the --silence option.
Fixes #226

Thanks!


Issue when trying to expand a group of a second level pivot

What version of React-Table are you using?
Your bug may already be fixed in the latest release. Run yarn upgrade react-table!
6.5.3
What bug are you experiencing, or what feature are you proposing?
Nested pivoting is not working. When you assign an array of columns to pivot, you are able to expand the first level, but not the deeper ones.
I found that the problem is in the set method, of utils
`function set() {
var obj = arguments.length > 0 && arguments[0] !== undefined ? arguments[0] : {};
var path = arguments[1];
var value = arguments[2];
var keys = makePathArray(path);
var keyPart = void 0;
var cursor = obj;
while ((keyPart = keys.shift()) && keys.length) {
if (!cursor[keyPart]) {
cursor[keyPart] = {};
}
cursor = cursor[keyPart];
}
cursor[keyPart] = value;
return obj;
}`
When trying to assign value, cursor[keyPart], the cursor[keyPart] is true, instead of an object.
What are the steps to reproduce the issue?

Have a React Table with an array of valid columns assigned to Pivot
Expand a group of the first pivot level. This works well
Try to expand a group of the second pivot leve. This doesn't work

Please refer to this working example. https://react-table.js.org/#/story/pivoting-aggregation


Add i18n support for official website

What is the problem that this fixes or functionality that this introduces? Does it fix any open issues?
Fix #1483
Give a brief summary of what the PR does, explaining any non-trivial design decisions

zh-Hans

fr

Other references

Used gpt-4o to finish the translation work.
Uploading the i18n folder to Git may not be the best practice. Docusaurus are using Crowdin to build a workflow of translation. Maybe starte with setting a specific API key for translation, we can integrate with Crowdin and streamline the localization process.
i18n is an ongoing process that requires patience and dedication. I welcome more people to join me in this effort to make our documentation accessible to a wider audience.

Thanks @Umpire2018 , this looks nice! But tests are failing, so we'll need to fix those.
Uploading the i18n folder to git might not be best practice, but honestly it seems good enough for now.

I have made attempts to eliminate the following issue:
[WARNING] Docusaurus found broken anchors!

Please check the pages of your site in the list below, and make sure you don't reference any anchor that does not exist.
Note: it's possible to ignore broken anchors with the 'onBrokenAnchors' Docusaurus configuration, and let the build pass.

Exhaustive list of all broken anchors found:

Broken anchor on source page path = /OpenDevin/fr/modules/usage/architecture:
-> linking to #backend-architecture-fr (resolved as: /OpenDevin/fr/modules/usage/architecture#backend-architecture-fr)

However, I was not successful. Nevertheless, this warning does not affect the build.

This is awesome! French works too, at least at a cursory reading at 1:00 a.m. As you say, i18n is an on-going process, it's great to get it started. ❤

Hey @Umpire2018, it seems all check pass. Can you solve the conflict before we merge this PR?

@neubig @yufansong Hi, the conflict is solved and ready to merge.


Improve the dependabot configuration

Description

Generate .github/dependabot.yml, to reduce human strain
Teach GHA to have it run make generate on dependabot branches, so that humans don't need to edit them before merging.

We'll see if the GHA permissions are correct--I haven't tested that.
Related Issues
(none)
Testing
(none)
Checklist

[ ] Does my change need to be backported to a previous release? - dependabot doesn't run on old branches, so no

What backport versions were discussed with the Maintainers in the Issue?

[ ] I made sure to update CHANGELOG.md. - no applicable changes
Remember, the CHANGELOG needs to mention:

Any new features
Any changes to our included version of Envoy
Any non-backward-compatible changes
Any deprecations

[ ] This is unlikely to impact how Ambassador performs at scale. - not a runtime change
Remember, things that might have an impact at scale include:

Any significant changes in memory use that might require adjusting the memory limits
Any significant changes in CPU use that might require adjusting the CPU limits
Anything that might change how many replicas users should use
Changes that impact data-plane latency/scalability

[ ] My change is adequately tested. - no
Remember when considering testing:

Your change needs to be specifically covered by tests.

Tests need to cover all the states where your change is relevant: for example, if you add a behavior that can be enabled or disabled, you'll need tests that cover the enabled case and tests that cover the disabled case. It's not sufficient just to test with the behavior enabled.

You also need to make sure that the entire area being changed has adequate test coverage.

If existing tests don't actually cover the entire area being changed, add tests.
This applies even for aspects of the area that you're not changing – check the test coverage, and improve it if needed!

We should lean on the bulk of code being covered by unit tests, but...
... an end-to-end test should cover the integration points

[ ] I updated DEVELOPING.md with any any special dev tricks I had to use to work on this code efficiently. - no new tricks

[ ] The changes in this PR have been reviewed for security concerns and adherence to security best practices.

v2:
diff --git a/.github/workflows/dependabot.yml b/.github/workflows/dependabot.yml
index 6e4d33142..248631ad1 100644
--- a/.github/workflows/dependabot.yml
+++ b/.github/workflows/dependabot.yml
@@ -12,11 +12,10 @@ name: dependabot
       - 'dependabot/**'

 jobs:
-  repatriate:
+  generate:
     runs-on: ubuntu-latest
-    name: repatriate
     steps:
-      - uses: actions/checkout@v2
+      - uses: actions/checkout@v3
         with:
           fetch-depth: 0
       - name: Install Deps
@@ -38,4 +37,4 @@ jobs:
           fi

           git commit --signoff --message='make go-mod-tidy && make generate'
-          git push origin
+          git push origin HEAD:${{ github.ref }}


Predictive custom service for user defined functions

 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.


Add option to update exchange rate each day

Will be good to add an option to update exchange rates each day, because when a computer has a long uptime of 1-2 weeks without reboot, we've got very old exchange rates in calculations.

Tested, works well - thanks!

Hmm, seems not fully working, after 7 days of Plasma session uptime, I see old exchange rates, but the fresh date of rates in settings (today). Restarted Plasma - and the calculator starts using new rates.
@dschopf Could you please recheck, maybe it doesn't refresh the rates after updating in some places?

I have tried to reproduce this issue with the latest version of the plasmoid using Qalculate! version 4.6 but it was working as expected. Could you provide some details on which version you are using?


How can I access state within renderNavigationBar?

Version
Tell us which versions you are using:

react-native-router-flux v3.37
react-native v0.41.1

Expected behaviour
To be able to use my state inside renderNavigationBar
Actual behaviour
I can't access state
I need to display a button based from a boolean value that is inside my state, but I can't access it because renderNavigationBar is a static method. If someone has worked around this, that would be great.
Thanks.

It's a dirty workaround, but you need to Actions.refresh({ your state here }) to load it in as, props, which you can then access.  There is no cleaner workaround since it is a class method that will not have access to the instance's state.
If you find that to be too funky or inconvenient, I would recommend hiding the navbar and just making your own navbar component to render in instances of your components without this issue.


To access updated State in parent component

The props passed to CarouselProvider renders the initial carousel, but there should be a way to access the updated state in parent component, I faced the issue while implementing accessibility on the carousel, while tabbing if we have passed slides the navigation buttons for example ButtonBack, ButtonNext are not changing there state as they do when we click on them.
There must be an updated state available in parent component.

Hey @Himanshu-uidev ! I hope I understood you correctly, you should be able to see the carousels current state by utilizing the Higher Order Component withStore or Context CarouselContext and I created an example using withStore here. You also said you ran into a bug while tabbing ? Would you mind providing an example for that?


SAK-47640 Rubrics Fix: Total Grade not updating correctly

Fix for #10776
Total Grade for weighted Rubrics:

wasn't being calculated correctly when using Criterion Groups.
wasn't updating when Ratings where added or removed.

I see a JS error, could you confirm?

I see it too, trying to understand the problem.


router: fix regex_rewrite for redirects matched on prefix

Commit Message: router: fix regex_rewrite for redirects matched on prefix
Additional Description:
These changes fixes an issue when using regex_rewrite as a part of a redirect when the path that is matched does not equal the full path. The issue was that only the matched section of the original path (e.g. prefix) was replaced with the substitution of the regex rewrite, while the expected behaviour is that the full path should be replaced.
This solves this by adding a case for these scenarios, i.e. if we have a redirect and use regex_rewrite, replace the full path with the rewritten path.
Risk Level: medium (changes the behaviour of how regex_rewrite and redirects works together)
Testing: Implemented unit tests to ensure the expected behaviour.
Docs Changes: N/A
Release Notes: Added
Platform Specific Features: N/A
Fixes #Issue: #23557
Signed-off-by: Axel Pettersson<EMAIL_ADDRESS>

@wbpcode Fixed!


cmake: Fix MSVCRT override

Fix the CMake code that overrides the /MD flag with the /MT flag for
the Windows demo and loader projects.  The CMake debug and release
variables were set to /MTd and /MT, overriding all existing debug
and release settings.
Now replacing /MD with /MT in the debug and release variables,
without losing other debug/release settings.  Also performing
the MD/MT substitution for all release and debug configurations.

Does this change or David's change cause this warning or was it already there before the change(s)?
Warning	1	warning LNK4098: defaultlib 'LIBCMTD' conflicts with use of other libs; use /NODEFAULTLIB:library	C:\Users\jon\vk\vk\dbuild\loader\LINK	vulkan-1
Other than this potentially new warning, the loader CMake change looks fine to me as far as the specific concern I had with the original patch.

That warning predates both changes.


Wrong dependancy group id in readme

Readme specifies com.crystalgames as the group id whereas the actual group id is net.crystalgames

Thank you for your report, this has been resolved :)

Thank you for your report, this has been resolved :)


[Bug Report] Dialog failed to display the el-image image

Element UI version
2.8.2
OS/Browsers version
win 10 / chrome 版本 73.0.3683.103（正式版本）
Vue version
2.6.10
Reproduction Link
https://codepen.io/pen/?&editable=true
Steps to reproduce
Dialog failed to display the el-image image
What is Expected?
Imagescan be loaded
What is actually happening?
Images are not loaded

Issues without a valid reproduction link will be closed directly.


Add retry limit for chromecast connection

Description:
Connect to chromecast fail will block whole HA startup
_get_chromecast_from_host method accept tries, retry_wait, timeout and blocking parameters. But above code doesn't provided any one of them. timeout default is 30s, and tries default is infinite.
I don't have chromecast, need someone help to test this PR
Related issue (if applicable): fixes #14956
Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#<home-assistant.github.io PR number goes here>
Example entry for configuration.yaml (if applicable):

Checklist:

[ ] The code change is tested and works locally.
[ ] Local tests pass with tox. Your PR cannot be merged unless tests pass

If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

[ ] New dependencies have been added to the REQUIREMENTS variable (example).
[ ] New dependencies are only imported inside functions that use them (example).
[ ] New or updated dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[ ] New files were added to .coveragerc.

If the code does not interact with devices:

[ ] Tests have been added to verify that the new code works.

Tried it locally and it works.


Upgrades to work with Terraform v0.12.20

Info
Following up on #10, this PR uses that fork and then applies the most recent requirement: Updates the type for parameter_write variable so terraform can run without errors.

@maximmi I'm giving this a shot as I'd like to give back to you folks for these awesome modules (even if this one is one of the simplest out there), but I didn't get too far. Honestly, I think my lack of Make + Go knowledge make this an uphill battle, but I'm happy to bang at it if you or @osterman are willing to provide some guidance. 😅
Current status, is that the example works when running locally, but I'm unable to get very far due to the following:

I installed bats since that seemed like an unmentioned prereq. I then ran cd tests/ && make which gave me a number of "not ok" values:

Running tests in  ../
1..10
ok 1 check if terraform is installed
not ok 2 check if terraform code needs formatting
ok 3 check if terraform modules are valid
ok 4 check if terraform modules are properly pinned
ok 5 check if terraform plugins are valid

Test: check if terraform providers are properly pinned
File: provider-pinning.bats
---------------------------------
* provider.aws: version = "~> 2.53"
---------------------------------

not ok 6 check if terraform providers are properly pinned
# (in test file test/.test-harness/test/terraform/provider-pinning.bats, line 17)
#   `[ $status -ne 0 ]' failed
not ok 7 check if terraform code is valid
# (in test file test/.test-harness/test/terraform/validate.bats, line 19)
#   `[ $status -eq 0 ]' failed
#
# Initializing provider plugins...
# - Checking for available provider plugins...
# - Downloading plugin for provider "aws" (hashicorp/aws) 2.53.0...
#
# The following providers do not have any version constraints in configuration,
# so the latest version was installed.
#
# To prevent automatic upgrades to new major versions that may contain breaking
# changes, it is recommended to add version = "..." constraints to the
# corresponding provider blocks in configuration, with the constraint strings
# suggested below.
#
# * provider.aws: version = "~> 2.53"
#
# Terraform has been successfully initialized!
#
# You may now begin working with Terraform. Try running "terraform plan" to see
# any changes that are required for your infrastructure. All Terraform commands
# should now work.
#
# If you ever set or change modules or backend configuration for Terraform,
# rerun this command to reinitialize your working directory. If you forget, other
# commands will detect it and remind you to do so if necessary.
not ok 8 check if terraform-docs is installed
# (in test file test/.test-harness/test/terraform/terraform-docs.bats, line 5)
#   `which terraform-docs' failed
not ok 9 check if terraform inputs have descriptions
not ok 10 check if terraform outputs have descriptions
make: *** [module] Error 1

The above led me to running the following and I got the following errors:

make terraform/validate:

Error: Error parsing /Users/gowiem/Workspace/Cloned/terraform-aws-ssm-parameter-store/main.tf: At 2:11: Unknown token: 2:11 IDENT var.enabled

make terraform/lint:

xargs: illegal option -- -
usage: xargs [-0opt] [-E eofstr] [-I replstr [-R replacements]] [-J replstr]
             [-L number] [-n number [-x]] [-P maxprocs] [-s size]
             [utility [argument ...]]
Error running fmt: In examples/complete/.terraform/modules/kms_key/examples/complete/main.tf: At 2:12: Unknown token: 2:12 IDENT var.region

Stopped there as something seems off. Is it possibly trying to lint / validate using v0.11 Terraform instead of v0.12?

I also attempted the following and am getting the corresponding error:

cd tests/src && make
Error:
make: *** No rule to make target `/usr/bin/go', needed by `/Users/gowiem/Workspace/Cloned/terraform-aws-ssm-parameter-store/test/src/.gopath/src/terraform-aws-ssm-parameter-store'.  Stop.

Anyway, I'm stopping here as I'm a little out of my depth on your folks setup, but if it seems I'm just missing some key steps then please point me in a direction and I'll work to give it another shot. Thanks folks!

@Gowiem this is your PR/hard work so i'm just going to share some learnings...
bats
aside from ensuring bats was installed, i:

installed terraform-docs
ran terraform fmt
updated main.cf with (pin providers):

# main.tf
terraform {
  required_version = "~> 0.12"

  required_providers {
    aws = "~> 2.58"
  }
}

provider "aws" {
  region = var.region
}

added region var as part of that:

# variables.tf
variable "region" {
  type        = string
  description = "AWS region to target"
  default     = ""
}

rm /tmp/terraform-modules-XXXXXXXXXXX.txt (module-pinning.bats leaves it behind)

that got bats happy:
➜ make all
Running tests in  ../
1..10
ok 1 check if terraform is installed
ok 2 check if terraform code needs formatting
ok 3 check if terraform modules are valid
ok 4 check if terraform modules are properly pinned
ok 5 check if terraform plugins are valid
ok 6 check if terraform providers are properly pinned
ok 7 check if terraform code is valid
ok 8 check if terraform-docs is installed
ok 9 check if terraform inputs have descriptions
ok 10 check if terraform outputs have descriptions
Running tests in  ../examples/complete
1..5
ok 1 check if terraform is installed
ok 2 check if terraform code needs formatting
ok 3 check if terraform modules are valid
ok 4 check if terraform plugins are valid
ok 5 check if terraform code is valid

terratest
from the looks of the Makefile it's intended to run in an Alpine Dockerfile in a CI/CD pipeline,
so my hacks were likely better solved another way...but to get it running i:

made sure dep was installed
linked it into GOBIN
specified GOEXE when running the test (GOEXE=/usr/local/bin/go make)
updated example to pass in region since module has a var for it now:

# examples/complete/main.tf
module "store" {
  source = "../../"

  region = "us-east-1" # added

made sure all my AWS env vars were set for test to find credentials
changed region in examples/complete/provider.tf to us-east-1 (got a region error as-is, should this be a var somehow?)
updated examples/complete/maini.tf kms_key.alias (i already had one, should that be a random string in tests?)

then it was all go test related, you gave me a good start. here's what i ended up with:
package test

import (
	"strings"
	"testing"

	"github.com/gruntwork-io/terratest/modules/terraform"
	"github.com/stretchr/testify/assert"
)

// Test the Terraform module in examples/complete using Terratest.
func TestExamplesComplete(t *testing.T) {
	t.Parallel()

	terraformOptions := &terraform.Options{
		// The path to where our Terraform code is located
		TerraformDir: "../../examples/complete",
		Upgrade:      true,
	}

	// At the end of the test, run `terraform destroy` to clean up any resources that were created
	defer terraform.Destroy(t, terraformOptions)

	// This will run `terraform init` and `terraform apply` and fail the test if there are any errors
	terraform.InitAndApply(t, terraformOptions)

	// Run `terraform output` to get the value of an output variable
	output := terraform.Output(t, terraformOptions, "map")
	result := strings.Contains(output, "Amazon")
	// Verify we're getting back the outputs we expect
	assert.Equal(t, true, result)
}

that got terratest happy:
--- PASS: TestExamplesComplete (57.94s)
PASS
ok      terraform-aws-ssm-parameter-store       59.188s

observation
this was a fun learning experience, and i'm not sure how far we should stray from the intent of the PR with this update... i do have some concern over putting passwords/secrets (even examples) in source control. it might be worth another PR to re-work this (and anything like it) to inject secrets via the environment (12F/HashiCorp best practice).
hth, let me know if you have any questions!

well if you guys are doing all this, then I guess I should try to add the KMS + s3 options too, if you are ok with that @Gowiem

@maximmi This should be ready for 👀 . I got everything running locally and all bats + terratests pass.
@deadlysyn Thanks for the pointers! Some of them definitely helped, but I also had some weird environment issues going on which were holding me back. Primarily:

I installed the original bats initially, which is massively out-of-date. This led me astray a bit.
I also was using Mac OSX's default Bash which is v3.2.57 and that was causing havoc once I finally did get onto bats-core. Upgrading to bats v5 got me fixed there.

As part of trying to get this working I ran into some other issues with the CP build-harness and test-harness. Put up PRs for those issues:

https://github.com/cloudposse/build-harness/pull/208
https://github.com/cloudposse/build-harness/pull/206
https://github.com/cloudposse/test-harness/pull/11

Haha it has been a fun few hours 😅
@jamengual, not sure what you're referring to regarding KMS + S3 option (would love a link), but go for it!

@Gowiem thank you for your contribution! Great job! I will take a look on this closely next week

/codefresh run test

/codefresh run test

/codefresh run test

/codefresh run test


? comparison test

The ? comparison triggers the following error :
,PATTERN-57 Cannot use _ before a comparison
I'm trying to do it just like in the docs :
u: ( _~propername?~bands )

https://github.com/bwilcox-1234/ChatScript/blob/master/WIKI/ChatScript-Advanced-User-Manual.md#more-comparison-tests----
When not using the _, the build passes, but the rule doesn't seem to match anything.
Thanks !

doc change because it is not legal, do u: (_~propername '_0?~bands)

Thanks !
Is there a way to do this without capturing the word ?
Something like : ~propername?~bands without the _.

yes. it was just the test AND capture that was illegal

Indeed without the _, the build passes.
Yet, I'm running version 7.54, and the sentence you love ski
matches : u: (you love ~tool) and u: (you love ~winter_sports)
but doesn't match u: (you love ~tool?~winter_sports)
Am I doing anything wrong ?


Reenable working clfswm

Motivation for this change
I'm interested in trying out Common Lisp window managers, specifically CLFSWM. It appears that this window manager was never moved over to using quicklisp, as StumpWM (another Common Lisp window manager) was.
Things done
Added CLFSWM through quicklisp.
Added the 'clfswm' nixos module back in.
Updated the 'clfswm' nixos module to use the quicklisp CLFSWM, and not the normal (broken) CLFSWM package.

[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"
[x] 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.

@SuperSandro2000 Regarding all your comments clfswm.nix: all .nix files in the quicklisp-to-nix-output directory are automatically generated by quicklisp-to-nix (see README.txt in lisp-modules, as well as the "quicklisp-to-nix" derivation in lisp-packages.nix in the same directory). quicklisp-to-nix uses a templating library to produce these .nix files, which uses double single-quoted strings. I'm not familiar enough with this implementation to know if there's a fundamental reason for this, or if the original author just happened to think this was a better idea, but I can say the template file has used these double single-quoted strings since this was originally implemented in March 2017.
If this is a problem, we should update quicklisp-to-nix and regenerate all packages in quicklisp-to-nix-output, which seems like it should be its own issue. Perhaps the original author (@7c6f434c) should weigh in?

@SuperSandro2000 Regarding all your comments clfswm.nix: all .nix files in the quicklisp-to-nix-output directory are automatically generated by quicklisp-to-nix (see README.txt in lisp-modules, as well as the "quicklisp-to-nix" derivation in lisp-packages.nix in the same directory). quicklisp-to-nix uses a templating library to produce these .nix files, which uses double single-quoted strings. I'm not familiar enough with this implementation to know if there's a fundamental reason for this, or if the original author just happened to think this was a better idea, but I can say the template file has used these double single-quoted strings since this was originally implemented in March 2017.
If this is a problem, we should update quicklisp-to-nix and regenerate all packages in quicklisp-to-nix-output, which seems like it should be its own issue. Perhaps the original author (@7c6f434c) should weigh in?

Didn't know that. You are right that it should be changed in the generator.

Didn't know that. You are right that it should be changed in the generator.

I am not even the original author of the generating scripts…

I am not even the original author of the generating scripts…

@7c6f434c

I am not even the original author of the generating scripts…

 At any rate, you have the most commits against files in the quicklisp-to-nix directory, so I figured you would be the best person to weigh in on whether or not there were any blocking issues.
Cheers for the merge!

@7c6f434c

I am not even the original author of the generating scripts…

 At any rate, you have the most commits against files in the quicklisp-to-nix directory, so I figured you would be the best person to weigh in on whether or not there were any blocking issues.
Cheers for the merge!

@7c6f434c

I am not even the original author of the generating scripts…

 At any rate, you have the most commits against files in the quicklisp-to-nix directory, so I figured you would be the best person to weigh in on whether or not there were any blocking issues.

Ah, it does reuse my old template from the earlier generator
I am more or less sure that using " or '' based on presence of " in the upstream descriptions is not worth it, and then it is more escaping vs more abuse of '' in the code that is not supposed to be read, so I am not convinced " is even better here.

@7c6f434c

I am not even the original author of the generating scripts…

 At any rate, you have the most commits against files in the quicklisp-to-nix directory, so I figured you would be the best person to weigh in on whether or not there were any blocking issues.

Ah, it does reuse my old template from the earlier generator
I am more or less sure that using " or '' based on presence of " in the upstream descriptions is not worth it, and then it is more escaping vs more abuse of '' in the code that is not supposed to be read, so I am not convinced " is even better here.


Document how to work with SELinux in enforcing mode

This line is missing from https://docs.mattermost.com/install/install-rhel-8.html:
sudo restorecon -R /opt/mattermost/
It should come after the line saying
sudo chmod -R g+w /opt/mattermost

Thank you for letting us know, @nigelhorne! Would you be open to updating the doc and opening a PR for it?


Using libkrun-efi as open source alternative to Apple's Virtualization framework

The new EFI flavor turns libkrun into an open-source alternative of Virtualization.framework on macOS.  It brings some excellent additions, such as a Vulkan-capable virtio-gpu, which allows you to offload compute workloads to the host's GPU, and an extended virtio-fs implementation. Podman 5.2 is taking advantage of this: https://linuxiac.com/podman-5-2-enhances-macos-vms-with-gpu-support/. Would be nice to add this VMM backend in addition to qemu and Virtualization.framework.

libkrun uses Hypervisor.framework the same as QEMU. Adding a new backend is tremendous effort. Why is it not possible to port the vulkan-capable virtio-gpu to QEMU instead? Is there any documentation on how they are doing it?

@lurenJBD Xen and KVM are both Linux-only virtualization technologies


Xamarin iOS Support

Since there's a couple of changes after last month pr of #280 , here's a new pr for Xamarin support #276 , the idea is to add Xamarin iOS, Mac and Android support.

Add Xamarin iOS support for Yoga, by providing a wrapper for the native lib.
Add Xamarin iOS Unit testing project
Adds a YogaKit shared implementation in c#.
Adds a YogaKIt iOS implementation.
Adds Yoga/YogaKit iOS sample application

Facebook.Yoga.iOS.Tests.exe : 169 ms
Tests run: 114 Passed: 114 Inconclusive: 0 Failed: 0 Ignored: 0

@splhack updated

Thank you for updating! Have you seen #322 new YogaKit PR? With my #327, I think we can just use C# YogaNode for @property (nonatomic, readonly, retain) YGLayout *yoga. we can remove YogaWidth and other methods. So maybe we could use something like
root.Yoga.AlignItems = YogaAlign.Center;
root.Yoga.Width = 100;
root.Yoga.MarginTop = 10;

Seems #322 was reverted..

@rmarinho yup, due to an internal issue. Nothing wrong with the PR itself. Working on getting it back in today or tomorrow.

@rmarinho @splhack is this ready to be merged?

@emilsjolander not yet, this refers old YogaKit impl. maybe we can split this into two part, Xamarin iOS project files (like #278), and new YogaKit support.

Sure

Rebase and updated!

We changed a couple of things in the merge commit. Aligned directory name (Xamarin.iOS to iOS) with others (Windows and Mac), removed AssemblyInfo, added copyright in C# code and create a new .gitignore.

Ok, had a pr coming to that directory change! thanks!

<3


Do I have to make manually 'category' for this theme?

Hello, I currently setup my blog with this theme.
My github url is: https://github.com/beomi/beomi.github.io
and my github page is surelly https://beomi.github.io
It seems works fine with github page but not on category tab.
When I click category card, then it shows just 404 page.
But when I locally serve jekyll, there isn't error at all. (it shows category posts perfectly.)
What's the difference for local serving and github serving for Category?
and what I should do for solving this?

Check #23 !


not working with current wine

hello im having this problem on blackarch:
python2.7 ./lnk2pwn.py -c config.json -o ~/hackthebox/<IP_ADDRESS>/files/privscalation/lnk2pwn/

                 ,-*
                (_).lnk
           -----------------
               <lnk2pwn>

    Malicious Shortcut(.lnk) Generator

                             [tommelo]
                                v1.0.0

[+] Checking wine installation
[+] wine status: OK
[+] Setting up the workspace
[+] Creating the output folder
[+] Generating the shortcut
0025:err:ntoskrnl:IoCreateDriver failed to insert driver L"\\Driver\\WineHID" in tree
0025:err:ntoskrnl:IoCreateDriver failed to insert driver L"\\Driver\\WineHID" in tree
0025:err:ntoskrnl:IoCreateDriver failed to insert driver L"\\Driver\\WineHID" in tree
0025:err:ntoskrnl:IoCreateDriver failed to insert driver L"\\Driver\\WineHID" in tree
0025:err:ntoskrnl:IoCreateDriver failed to insert driver L"\\Driver\\WineHID" in tree
0025:err:ntoskrnl:IoCreateDriver failed to insert driver L"\\Driver\\WineHID" in tree
0038:fixme:ver:GetCurrentPackageId (0x33fe24 (nil)): stub
0038:fixme:msvcrt:__clean_type_info_names_internal (0x1e2709b0) stub
0009:fixme:ver:GetCurrentPackageId (0x33fe24 (nil)): stub
[+] Renaming .lnk to a fake extension: .txt
[+] Generating the UAC bypass VBScript
[+] Moving files to the output folder
Traceback (most recent call last):
  File "./lnk2pwn.py", line 126, in <module>
    main(cli_args)
  File "./lnk2pwn.py", line 101, in main
    executor.execute(args)
  File "/home/rek2/hackthebox/<IP_ADDRESS>/files/privscalation/lnk2pwn/cmd/generatecmd.py", line 240, in execute
    self.delegate(args, json_config=json_config)
  File "/home/rek2/hackthebox/<IP_ADDRESS>/files/privscalation/lnk2pwn/cmd/command.py", line 113, in delegate
    self.handler(args, kwargs)
  File "/home/rek2/hackthebox/<IP_ADDRESS>/files/privscalation/lnk2pwn/cmd/generatecmd.py", line 189, in __generate
    os.rename(tmp_lnk_rtlo, lnk_file)
OSError: [Errno 18] Invalid cross-device lin

I also try with the steam wine version changing the env $PATH but same result.
[ blackarch lnk2pwn ]$ wine --version
wine-3.19

[ blackarch lnk2pwn ]$ uname -a
Linux rek2system 4.18.16-arch1-1-ARCH #1 SMP PREEMPT Sat Oct 20 22:06:45 UTC 2018 x86_64 GNU/Linux

Hey @ReK2Fernandez thanks for reporting this issue!
Yeah, depending on wine wasn't a good solution when I started developing this tool, I've seen some other users reporting issues with wine in different linux distros.
I'm currently working on a C version of this tool to remove the wine dependency, meanwhile, would you mind testing the script using a Kali Linux vm?
By the way, I'm not sure if this tool is going to help you to privesc a hackthebox machine, though.
Bypassing the UAC using the trusted binary "fodhelper" only works if the logged user is part of the admins group.
Let me know if you can get this to work using a Kali Linux VM and also if the privesc works using lnk2pwn.

hello @tommelo thanks. I ended up writing my own using golang, with win32/com libs etc, that worked for what I needed, this was for a horizontal escalation not vertical so I really did not need that part, only the part of run something. if I had used yours I will had prob commented that piece.
Thanks for the response, maybe next time if you have a C version with more options I will add it to my goto tools. thanks.


[AutoPR storage/resource-manager] Adds canFailover boolean flag under storage account's GeoReplicationS…

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/4532

This PR has been merged into https://github.com/Azure/azure-sdk-for-go/pull/3371


rest-api: move config modify rest api handler into rest-api cell

Currently, the REST API handler that handles config modifications is implemented in the Daemon struct.
This commit moves the handler and all it's dependencies into the rest api cell.

/test

rebased to main to resolve conflicts.

/test

rebased on top of main to resolve conflicts

/test

Rebased on top of main to resolve conflicts with https://github.com/cilium/cilium/pull/33190.
@ldelossa Gentle ping :)

/test

removing review-request for @ldelossa (PTO) - sig-agent is already covered by the review of @joamaki anyway.


Dockerfile build fails: base image missing

Hi Daniel,
Building the docker from utils/env/Dockerfile fails due to missing base image  nvidia/cuda:11.4.0-base-ubuntu20.04.
I could build a docker for BiaPy after pulling a TF2 installed nvidia-base image.
Could you kindly test if the existing Docker build works at your end?
Happy to share my Dockerfile via pull request if it would be of benefit.
Best,
Samia

Hello,
They have deleted that base image so you can use nvidia/cuda:11.4.3-base-ubuntu20.04 instead (see avaiailable tags here).
Best regards,
Dani

This works. Thanks.


can not use rate and pitch for Voice ?

this code is from website doc Introduction:
`from elevenlabs import VoiceSettings
settings = VoiceSettings(speaking_rate=0.8)
audio = generate("Welcome to our podcast!", voice="Amy", settings=settings)
`
However,actually it can not use!

Incorrect docs, no such parameters yet.

This is fixed in the latest SDK (v1.0.0b0). Please reopen if the issue still persists!


Create an exception for the eslint rule regarding server actions

Using server action, we can pass functions as props in server components.
The example below is taken from the Next.js server action post.
export default function Page() {
  async function createInvoice(formData: FormData) {
    'use server'

    const rawFormData = {
      customerId: formData.get('customerId'),
      amount: formData.get('amount'),
      status: formData.get('status'),
    }

    // mutate data
    // revalidate cache
  }

  return <form action={createInvoice}>...</form>
}

The lint rule is triggered at the <form action={createInvoice}>...</form> part of the example above.

Functions can only be passed as props to Client Components. Add the "use client" directive at the top of the file to use it.eslint(react-server-components/use-client)

package version

"next": "14.2.4"
"react": "^18"
"eslint": "^8"
"eslint-plugin-react-server-components": "^1.2.0"

PRs welcome! Otherwise I'll try to get to this sometime soon.


Auto completion not working

Auto completion is not working with the default "init.example.el" config.
Tested on macOS 10.12.5 and ubuntu 17.04.

Could you offer any other information? Like errors or steps to reproduce? What mode are you invoking auto-completion from?

Steps:
cd ~/.emacs.d
cp init.example.el init.el  # maybe edit init.el
make install
make compile       # optional, may take a while```

then open emacs, edit a file (lua, python or C) and no autocompletion suggestion is shown.

Even I can vouch for this, I thought this was only an issue on my system.
Yes, I have libclang, and installed irony server from within emacs.
After looking into the config for sometime, I saw that you have mapped C-SPC to show suggestion in insert mode, so only then the suggestions are shown, Is this expected? or not? I searched online about on how you may have disabled the suggestions list to appear as text typed, but couldn't find any info. Thanks

leojrfs, are you sure that you have libclang installed on your system? and ran the command 'irony-install-server' from within emacs?

Ah, I see what you mean.
@vsr625 Yes, this was intentional (and a matter of personal preference). I prefer to invoke it only when I need its help. You can reverse this with:
;; these are the defaults (before I changed them)
(setq company-idle-delay 0.2
      company-minimum-prefix-length 3)

@leojrfs

python needs setuptools, e.g. pip install setuptools;

C needs irony-mode (which needs irony-server: M-x irony-install-server)

lua doesn't need anything, but company-lua (the plugin that gives you autocompletion in lua) is very primitive. As in it only completes from a static list (for the Lua 5.1, 5.2, 5.3 and Love2D APIs). It won't complete custom functions or variables (frustrating, I know). For instance, try autocompleteing raw, you should get:

Unfortunately, that's the best we can hope for until someone writes a more comprehensive lua completion library.

Thanks for replying, that worked exactly as expected, now I get suggestions as I type.
One last question, in cc mode, I am getting suggestions only over the variables and functions I define.
Is there a way for getting suggestions for snippets and basic c++ language stuff like for main,while,if?

If you want it to complete library functions Irony-mode is smart enough to follow included headers, e.g. #include <iostream>, then try to complete std::co.
But for language constructs like main/while/if, no, irony-mode won't give you that. But I have a good alternative: snippets. My snippets library has snippets for if, for, while, etc (and others). You just need it somewhere yas-snippet-dirs can see them.
But you want those to be included in the completion results, don't you? That's where company-yasnippet comes in. You do need to add it to c++-mode's company backends, but that should give you what you want.

Though, I should add: I'm a bit hazy on the exact format for company-backends to get company-irony-c-headers, company-irony and company-yasnippet to cooperate. It could be '(company-irony-c-headers company-irony company-yasnippet) or '(company-irony-c-headers company-irony :with company-yasnippet) or '(company-yasnippet :with company-irony-c-headers company-irony).
Sorry I'm uncertain, but just in case you run into trouble.

@hlissner
I've added:
(setq company-idle-delay 0.2
      company-minimum-prefix-length 3)

to init.el, but i still don't have completion. is it supposed to go there?
Sorry, im trying emacs for the first time, with your config...

Not exactly. The module is lazy loaded, so any settings in init.el will be overwritten later.
A slight modification will make it work:
(require 'company)
(setq company-idle-delay 0.2
      company-minimum-prefix-length 3)

It'd be better if you made these changes in modules/completion/company/config.el, but the above will work just as well to get you started.

Still no completion with
(require 'company)
(setq company-idle-delay 0.2
      company-minimum-prefix-length 3)

in init.el.
is there any way that i can see if this is being read?

Oh wait, I recall you ran make compile, right? Emacs will prefer the compiled file over the uncompiled one. You'll need to run make compile again (or make clean to get rid of compile files altogether -- compilation is optional, you shouldn't be using it while you're making changes to the config).

same situation.
just ran make clean and make compile.
But i just found that one module is failing to compile:

Then forgo compilation entirely and just do make compile-lite, this will only compile the core files. I'm aware of that error and will push a fix for it tomorrow.

still no completion, even did a clean install:
git clone https://github.com/hlissner/.emacs.d ~/.emacs.d
cd ~/.emacs.d
cp init.example.el init.el

added the following to init.el:
(require 'company)
(setq company-idle-delay 0.2
      company-minimum-prefix-length 3)

And then:
make install
make compile

Very strange. Could you run DEBUG=1 make doctor and paste its output here?
On May 22, 2017, 2:57 AM +0200, Leo Soares<EMAIL_ADDRESS>wrote:

still no completion, even did a clean install:
git clone https://github.com/hlissner/.emacs.d ~/.emacs.d cd ~/.emacs.d cp init.example.el init.el
added the following to init.el:
(require 'company) (setq company-idle-delay 0.2 company-minimum-prefix-length 3)
And then:
make install make compile
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub (https://github.com/hlissner/.emacs.d/issues/77#issuecomment-302975750), or mute the thread (https://github.com/notifications/unsubscribe-auth/AAfLo-hztZzxiJkavIdfjgeW3kMZjHVtks5r8N1fgaJpZM4NhqwK).

DEBUG=1 make doctor
DOOM Doctor
Running Emacs v25.2.1
Compiled with:
  NOTIFY ACL LIBXML2 ZLIB
uname -a:
  Darwin colossus.home 16.6.0 Darwin Kernel Version 16.6.0: Fri Apr 14
  16:21:16 PDT 2017; root:xnu-3789.60.24~6/RELEASE_X86_64 x86_64

----

====
Have some debug information:

  + Attempt to load DOOM: success!

All three of your suggestions worked, thanks for taking time to reply

@vsr625 It's how company cycles through company-backends. In a C++ buffer company-backends might contain:
((company-irony-c-headers company-irony)
 company-capf company-yasnippet)

In which case, your completion results will be computed like so:
(company-irony-c-headers + company-irony) || company-capf || company-yasnippet
So yasnippet won't get a turn unless all its predecessors fail. And by "fail", I don't mean "no results found", I mean more like "this major mode isn't my jam" or "I'm not in the mood".

@leojrfs I assume you still have
(require 'company)
(setq company-idle-delay 0.2
      company-minimum-prefix-length 3)

Scroll to the bottom of init.el. Make a new line, and:

type (define-. Press C-SPC if you haven't gotten a completion popup yet.
If you don't, press M-x and run global-company-mode. Pay attention to the minibuffer. It should say "Global Company mode enabled".
2.1. If it says enabled, try step 1 again (from scratch)
2.2. If it said "Global company mode disabled", that means your changes in init.el aren't being read for some reason. Try adding this to your init.el:

(message "Hello world")

Then restart Emacs and run M-x view-echo-area-messages and see if "Hello World" appeared in the log.
If it does work, let me know what step you were on.

Completion works on emacs lisp, but it doesn't on lua.

The only thing I can think of is that Emacs cannot see the lua executable. Try this: M-x eval-expression RET (executable-find "lua") and see what it says. If it returns blank, then that's the problem.

Though, I should add: I'm a bit hazy on the exact format for company-backends to get company-irony-c-headers, company-irony and company-yasnippet to cooperate. It could be '(company-irony-c-headers company-irony company-yasnippet) or '(company-irony-c-headers company-irony :with company-yasnippet) or '(company-yasnippet :with company-irony-c-headers company-irony).

Can you tell me the proper way to change the backends inside private init.el?

@vsr625 All set! settings should be done from your private config.el file. They won't exist at the time your private init.el is loaded.
;;; private/{user-login-name}/config.el
(set! :company-backend 'python-mode '(company-anaconda))
;; multiple modes at once
(set! :company-backend '(css-mode scss-mode) '(company-css company-yasnippet))

Note:  (set! :company-backend MODES BACKENDS) will overwrite past settings for that mode.

@hlissner Did you test it? Because for me it's not working, before your latest commits it was working fine with (after! company (set! :company-backend ..)), in my config.el file.
I don't know why but it stopped working now.
If I use this, in a (after! company block) inside config.el, it works correctly.

Yea, now it works, but only if it's inside a (after! company) block, but that's fine anyway. Thanks for looking into it though.

Hello,
same issue for python autocomplete. To activate it I need to manually disable and re-enable global-company-mode. It's tedious.

same situation.
just ran make clean and make compile.
But i just found that one module is failing to compile:

where did you run make compile and make clean?
I cannot do this in .emacs.d folder.

same situation.
just ran make clean and make compile.
But i just found that one module is failing to compile:

where did you run make compile and make clean?
I cannot do this in .emacs.d folder.


fecap.br - Fundação Escola de Comércio Álvares Penteado (FECAP)

@xudre edu.fecap.br domain is already in this repository. If you wish to add fecap.br domain, please explain why, and rename the file to lib/domains/br/fecap.txt. Please ensure that you are familiar with this repository rules located at the bottom of this page before submitting a new request or modifying your old one. Thank you.

@philipto {at}fecap.br is the domain for school teachers and students, the already added {at}edu.fecap.br it is used by the college teachers and students. There are these differences here at the institution. Mine, as school teacher, is<EMAIL_ADDRESS>for example.

@xudre Thank you for the clarification. Pull request merged.

@philipto I didn't rename the path of the file before you merged, how can I proceed now? Revert the merge and post a new one?


[Bug] Better discord crashes

Describe the Bug

Better discord error message upon discord start explaining that better discord has crashed
To Reproduce

Expected Behavior

everything working like normally without an error message
Screenshots

Discord Version

Additional Context

Thanks for letting me know ,that’s right have great day

On 5 Apr 2021, at 1:08 AM, fflixer @.***> wrote:

Describe the Bug
Better discord error message upon discord start explaining that better discord has crashed
To Reproduce
Expected Behavior
everything working like normally without an error message
Screenshots
Discord Version
Additional Context
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Removed all themes I had installed and performed a fresh install:
> discord -vvv
Discord 0.0.14
(node:24836) [DEP0005] DeprecationWarning: Buffer() is deprecated due to security and usability issues. Please use the Buffer.alloc(), Buffer.allocUnsafe(), or Buffer.from() methods instead.
Starting app.
Starting updater.
[Modules] Modules initializing
[Modules] Distribution: remote
[Modules] Host updates: enabled
[Modules] Module updates: enabled
[Modules] Module install path: /home/sfranke/.config/discord/0.0.14/modules
[Modules] Module installed file path: /home/sfranke/.config/discord/0.0.14/modules/installed.json
[Modules] Module download path: /home/sfranke/.config/discord/0.0.14/modules/pending
[Modules] No updates to install
[Modules] Checking for host updates.
[Modules] Host is up to date.
[Modules] Checking for module updates at https://discord.com/api/modules/stable/versions.json
[Modules] No module updates available.

Unexpected crash report id length
Failed to get crash dump id.
Report Id:

Unexpected crash report id length
Failed to get crash dump id.
Report Id:
[WebContents] crashed... reloading

Unexpected crash report id length
Failed to get crash dump id.
Report Id:

Unexpected crash report id length
Failed to get crash dump id.
Report Id:
[WebContents] double crashed... RIP =(

Rename your existing BD directory and reinstall.

Thank you!
I did rename ~/.config/discord/0.0.14/modules/discord_desktop_core/injector/betterdiscord and reinstalled which fixed the issue!


Weird issue when running npm run test

Getting a weird error when running the test suite on specific Mac machines with python3 http.server

Can't reproduce anymore, will close for now


Versions in gauge -v should reflect nightly builds

Currently, gauge -v lists all the plugins which could either be a released build or a nightly build. This list should show up the difference.

Gauge shows the nightly versions of plugins and that of Gauge too.


babel "transform-object-rest-spread" can't load plugin

Module parse failed: /home/me/code/projects/web/casino/common/reducers.js Unexpected token (42:6)
You may need an appropriate loader to handle this file type.
|     const { message } = payload;
|     return {
|       ...state,
|       list: [ ...state.list, message.id ],
|       entities: { ...state.entities, [message.id]: message }
 @ ./src/store.js 4:0-41
 @ ./src/main.js
 @ multi react-hot-loader/patch webpack-hot-middleware/client<EMAIL_ADDRESS>!style-loader!css-loader!normalize.css/normalize.css ./main.js

I've tried to add transform-object-rest-spread and stage-3 to package.json.
babelConfig.presets.push({
  plugins: [
    path.resolve(__dirname, '../node_modules/babel-plugin-transform-object-rest-spread/lib/index.js'),
  ],
});

to webpack.config, no results. Can you help me with this, please?

Maybe you need to include folder with reducers.js here

@frenzzy you're right! Thanks, this solved my problem.


Add externally built plugin example #enhancement

I'd prefer to keep my domain specific question types in a separate package so I don't have to build and keep in sync surveyjs.

I may provide an example if I decide to go this route.

@kbrown Hello! I will try to create examples for that task.

@kbrown I've added examples:
http://surveyjs.org/examples/knockout/custom-question.html
http://surveyjs.org/examples/react/custom-question.html
Could you check it, please?