Allow referring to root context with .

This could be useful if the root context is not a map, see #22.
There are some questions here though because the root context is not a single map but rather multiple scopes containing values. This would require copying the entire context which could potentially be expensive.

Has this feature been released?
How should I use the variables passed from the parent template?
parent:
{% include "text-format" with node.children %}

child:
{% for child in . %}
{% if child.category.type | eq: "Text" %}
{{child.category.text}}
{% endif%}
{% endfor %}

error:
invalid syntax

  --> text-format:1:17
   |
 1 | {% for child in . %}
   |                 ^--
   |
   = reason: expected expression, found member access operator

It hasn't been implemented. You will have to pass node to the sub template right now.
{% include "text-format" with node.children %}

{% for child in children %}
{% if child.category.type | eq: "Text" %}
{{ child.category.text }}
{% endif %}
{% endfor %}

I put some thought into it, there are some outstanding questions:

What is the expected behaviour with multiple scopes of values, does it always refer to the root context or all of the contexts merged?

{% for k, v in object %}
    {% with k | transform as name %}
        {{ . }} <!-- does this have k, v, name, loop merged into the global context? -->
    {% endwith %}
{% endfor %}

What is the expected behaviour when using render_from_fn?


Revert "Continue the Misk to Wisp conversion of feature - this time k…

…eep the … (#2076)"
This reverts commit 0476091c655b6ecfa8e2a3c16283c189686d5e6c.
This is a breaking change that passes builds but breaks at runtime.

How does it break at runtime?

How does it break at runtime?

@quad Something to do with circular dependencies / transitive dependencies being pulled in with mismatched versions of Misk.
java.lang.NoSuchMethodError: 'boolean misk.feature.FeatureFlags.getBoolean(misk.feature.Feature, java.lang.String)'


Report from publicapis.dev

Name: AniAPI
Category: Anime
Issue: Broken link
This report was sent from publicapis.dev.

Removed.


Warning: separate entrypoint breaks CI pipelines

Hi, not an issue per se, but I wanted to raise a warning for those who (like myself) use this image for Docker-based pipelines like Gitlab CI.
Since the split of entrypoint and cmd pipelines fail because the underlying engine is (probably) running bash -c $MYCOMMANDS as a custom CMD.
Since now we have sonar-scanner as the entrypoint, I get errors like ERROR: Unrecognized option: -c because it's passing it to sonar-scanner.
The solution is to override the image entrypoint in the pipeline, for example by setting (Gitlab CI syntax):
analysis:
  stage: analysis
  image:
    name: newtmitch/sonar-scanner:3.2-alpine
    entrypoint: ['']
  dependencies:
    - tests
  script:
    - sonar-scanner -Dsonar.host.url=$SONAR_URL -Dsonar.login=$SONAR_TOKEN -Dsonar.projectBaseDir=.

I hope it can be useful to others.
Thanks for the image, btw!

Thanks for the feedback, @MatteoJoliveau! I don't use this in a pipeline at all and don't really have the capability to test this out. Is there a better way to separate the CMD and ENTRYPOINT directives in the Dockerfile so it doesn't require such a hacky workaround? I made this change to solve someone else's issue with the way it was oriented before and also to align more closely with Docker's best practices for CMD and ENTRYPOINT usage. Clearly this broke some other stuff (sorry!).

Yeah, probably best to make a new tag for a breaking change like this. When you change the entrypoint it is very likely to break any type of automated usage. Anyway, thanks for publishing the image!

No need to apologize, @newtmitch! It's difficult to please everyone. I think the best approach is the one proposed by @mhodgson. You could make a new Dockerfile without the ENTRYPOINT (you can even use it as a base for the regular one to reuse code!) and tag it with something like 3.2-alpine-ci so that we know it's best suited for automated use.

Hi @newtmitch!, have you planned to add a ci version soon without entrypoint ? Matteo workaround will work for gitlab > 9.4

Agreed. Additionally, I'd recommend not touching old tags and instead increment the version - was using 3.2.0-alpine and wondered why it broke suddenly :P
Here's an output from Jenkins for reference.

docker run -t -d -u 1000:1000 -v $PWD:/root/src -w /home/jenkins-slave/workspace/@2 --volumes-from  -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** -e ******** newtmitch/sonar-scanner:3.2.0-alpine cat
ERROR: The container started but didn't run the expected command. Please double check your ENTRYPOINT does execute the command passed as docker run argument, as required by official docker images (see https://github.com/docker-library/official-images#consistency for entrypoint consistency requirements).
Alternatively you can force image entrypoint to be disabled by adding option --entrypoint=''.

I just refactored the Dockerfile and simplified the repo while doing so. Part of that refactor was moving back to the CMD-based usage rather than the ENTYRPOINT+CMD-based usage, which should simplify the image's usage for use-cases like CI. I've left existing images as-is in order to maintain current usage (and not break things again!) and also created a 4.0-ci tag specifically to address this issue. All image tags 4.1 and later have been updated to use CMD only.
Closing this issue. Thanks for all the discussion here!


Changing Hero Section Colour Gradients

i want to change the colougradient of the hero Section of the website and I guarantee you I become user friendly and attractive also .
and some animation add on the hero page

As a contributor to GSSOC 2024, I'm eager to take on tasks. Could you please assign me this task for completion?


Fee calculation error when spending UTXO with scriptRef

Steps to reproduce:

Create a wallet
Attach a script to a UTXO
Spend the UTXO

This will cause the fee calculation to be incorrect and cannot be submitted on-chain

Example on preview network:
const address =
  "addr_test1qqja0lh0lnpwd7hewzptwpay09mgqm2k0srqsrfp89nya2h3uq0x2xemxfac2lf7axszqezaw2tpahj2d23guffx687sqz8edy";
const utxos = await lucid.utxosAt(address);
const destination = await getRandomAddress(lucid);

lucid.selectWallet.fromAddress(address, utxos);
const tx = await lucid
  .newTx()
  .pay.ToAddress(destination, {
    "20a814aa7a68d16b8d236f21a4dcb1ed1e1b359faff67287da2c33dd626561636f6e":
      1n,
  })
  .complete();

I think I know what's being missed here. I'll get it fixed.

Currently, fee calculation is incorrect for a tx which spends utxos (containing ref scripts) added by lucid's coin section. After giving some though, I believe utxos with scriptRef attached to them should not be selected by coin selection in the first place. They should be spent if explicitly intended by the user using collectFrom. If we go with this approach, the fix to this issue would be very simple otherwise it would require more complex logic at the level of coin selection and fees calculation.
I would like to know your opinions. @solidsnakedev @SynthLuvr

Sometimes we filter out UTXOs with datums from the wallet ourselves to avoid them being spent. I think by default they should be included because there are dapps that attach datums as a "receipt."
Regarding scriptRef, I think it'd be fine to spend it explicitly via collectFrom.

When using collectFrom it's still the same error

This could be due to updated fee structure in Conway (active in preview testnet) where reference scripts now carry additional weight, more details here.

This could be due to updated fee structure in Conway (active in preview testnet) where reference scripts now carry additional weight, more details here.

Yes this is true. The fee for reading from reference scripts was already implemented in https://github.com/Anastasia-Labs/lucid-evolution/pull/195. However, this new issue is related to spending UTXOs with reference scripts attached. I don't think there should be a fee attached for spending reference scripts. However, assuming this stays in cardano-ledger, lucid-evolution will need to be updated to account for this extra fee.

@SynthLuvr thanks for explaining! I haven't looked carefully at this issue, but

However, this new issue is related to spending UTXOs with reference scripts attached.

In my understanding, this additional fee is enforced irrespective of whether you are actually making use of reference script in your tx or not. As long as spend inputs or reference inputs have script attached to them, they would contribute to fees.

In my understanding, this additional fee is enforced irrespective of whether you are actually making use of reference script in your tx or not. As long as spend inputs or reference inputs have script attached to them, they would contribute to fees.

Yes that's correct. I think this was overlooked in the cardano-ledger implementation. Given that this is related to a vulnerability that was exploited, the implementation may have been rushed. It doesn't make sense that this should be contributing to fees, because there's no vulnerability associated with spending data, only referencing data within validation logic.
Having said that, these are the current ledger rules for conway, so if these rules remain as-is, then we have to adapt and add support for the fee.

@SynthLuvr Not sure if I am following properly, but generating script context requires resolving inputs, in particular also mentioning associated script hash (if any) which thus a Plutus script can depend upon, so even if a given input's script is not being exercised for validation, it could be referred by other script.

so even if a given input's script is not being exercised for validation, it could be referred by other script.

That's an interesting point. I guess even if such a scriptRef is being referred by another script (which is witnessing the transaction), the witnessing script will bear the cost of deserializing the scriptRef thereby accounting for the cost via its increased exUnits. The ledger need not presume and charge every scriptRef in inputs being spent.

When using collectFrom it's still the same error

After spending some good amount of time, I was finally able to get this working. Will need to test it further though.

just to clarify the deserialization of the reference scripts (flat encoded) are done by the cardano nodes, and according to the ledger it is mandatory to count all the inputs with reference scripts, even if they're two or more inputs with the same reference script.
Also I think we should do this calculation inside the recursive coin selection, and add the extra cost for each added input https://github.com/Anastasia-Labs/lucid-evolution/blob/2617572fa8a39f4f299119abc94c0f0a8ccd932a/packages/lucid/src/tx-builder/internal/CompleteTxBuilder.ts#L636-L692

There's also a CML update in the work, but I think we're much closer to get it fixed.
https://github.com/dcSpark/cardano-multiplatform-lib/pull/349


Accelerate data science using nvidia rapids

TypeError: '<=' not supported between instances of 'int' and 'str' in for loop

The rapids team provided us with an updated notebook which I switched


layers incorrectly rewrite extensions list

This was found during SDK vkconfig testing.
Launch vkconfig. Inside vkconfig, launch hwcapsviewer, and look at the Extensions tab. Note that there is no VK_KHR_shader_object or VK_KHR_synchronization2 extension reported:

In vkconfig, create a layer configuration with VK_LAYER_KHRONOS_synchronization2 and VK_LAYER_KHRONOS_shader_object forced on.  In their settings, they should both have ☑ Force Enable selected.  Launch hwcapsviewer from inside vkconfig again, and again check the list of extensions reported.

There are two observed issues here, both with the reported list of extensions.  @christophe-lunarg suspects that one or both of the layers   VK_LAYER_KHRONOS_synchronization2 and VK_LAYER_KHRONOS_shader_object fail to rewrite the reported extension list correctly.

VK_KHR_synchronization2 is not listed.
VK_EXT_vertex_attribute_divisor was shown before the layers were activated; it disappeared after the layers were activated.

I went through the various options and reported what I saw in the caps viewer, which differs slightly from what is reported here. This was done on Windows using the latest SDK (1.3.261, which is in development).
The gist is that the properties & features of an extension are only present if the respective structs appear in pNext chains - which depends on the layers adding them.
Because Sync2 doesn't define any properties, sync2 never appears in the Properties->extensions list. It does appear in the features->extensions list, as well as just the raw list of extensions.
On windows, I did not see any different in output based on whether the Force Enabled option was set in either layer.
Note that because my GPU supports sync2, they appear twice in the output.
VulkanCapsViewer_sync2_shader_object_findings.txt

On my machine with current test SDK:

The double reporting does look wrong - but that is probably a separate issue to the one initially reported, that the extension doesn't appear at all unless the layer is enabled through vkconfig (it may appear if the layer is enabled through other mechanisms, but I did not test that).
To respond to the two reported issues: VK_KHR_synchronization2 is not listed in properties->extensions because sync2 has no properties struct. It never appeared no matter what when I tested things. Don't expect sync2 to appear there. In other words, user error with VulkanCapsViewer gives misleading results.

Also - VK_EXT_vertex_attribute_divisor is always removed whenever shader_object is active because they are incompatible. The disappearance is expected.

I recommend closing this issue - both things reported by bob are expected behavior.
Christophes double reporting of sync2 is now in its own issue, #303, to better track things.


No test logging was found with AssemblyQualifiedName, URI, or FriendlyName "junit".

I always get the following error:
dotnet test --no-build -c Release -v n myProject.test.csproj --test-adapter-path:. --logger:"junit;LogFilePath=foo.xml"

No test logging was found with AssemblyQualifiedName, URI, or FriendlyName "junit".

Can anybody give me a hint whats wrong?

@dojo90 Sure thing. One item to confirm is that you have the nuget package Microsoft.NET.Test.Sdk installed. Assuming you do, I would guess the logger isn't being copied into the release folder. I have seen some variation between .Net core / framework in this respect. What type of project are you testing?

After complete cleanup of solution/build server and reinstalling of the nuget package. Everything is now working as expected. I cannot reproduce the error. So don't know what was wrong..
<Project Sdk="Microsoft.NET.Sdk">

  <PropertyGroup>
    <TargetFramework>netcoreapp2.2</TargetFramework>
    <IsPackable>false</IsPackable>
  </PropertyGroup>

  <ItemGroup>
    <PackageReference Include="DeepEqual" Version="2.0.0" />
    <PackageReference Include="JunitXml.TestLogger" Version="2.1.15" />
    <PackageReference Include="Microsoft.NET.Test.Sdk" Version="16.4.0" />
    <PackageReference Include="xunit" Version="2.4.1" />
    <PackageReference Include="xunit.runner.visualstudio" Version="2.4.1">
      <PrivateAssets>all</PrivateAssets>
      <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
    </PackageReference>
    <PackageReference Include="JunitXml.TestLogger" Version="2.1.15" />
  </ItemGroup>
</Project>


Magic strings objectified

Changes

Magic strings collected in single object
Second usecase example in Readme

Thanks for the suggestion, but I don't really see how this improves anything.


Borderlands 2 cursor missaligned when using DSR.

Using DSR 2.25x DL causes the cursor to be missaligned and making it impossible to click certain parts of the menu.
Software information
Borderlands 2 GOTY (Steam)
System information

GPU: RTX 4070 Ti
Driver: 531.79
Wine version: N/A
DXVK version: 2.2

We don't really support Windows, even less so when using driver options that only exist on Windows.

Mouse cursor position really isn't something that we have any control over to begin with.

This isn't something we can solve from our side, sorry.

You could try creating a config file with d3d9.dpiAware = True, maybe that helps. See the README for instructions on how to do that.

You could try creating a config file with d3d9.dpiAware = True, maybe that helps. See the README for instructions on how to do that.

May work, but i'll stick to D3D9 after all, since you can't force Anti-Aliasing with Vulkan/DX11/DX12. And Borderlands 2 needs SGSSAA to look smooth.


Store all locales in the data warehouse

As a Managing editor
I want have data for all content items
So that I can make informed decisions about any content item that I am responsible for
What
Store all locales in the data warehouse. The sandbox and metrics endpoints filter out only the 'en' locale
Why
We are attempting to only store content that has the 'en' locale in the data warehouse. The code that filters out the default locale is not working as intended. We have realised it is better to store everything.
The Trello card is Store all locales in the data warehouse

LGTM 👍
Could you please test in integration before merging?


Environment Setting for this Repo

Hello, could you please give the environment setting for this repo?

Hello, could you please give the environment setting for this repo?
I want to know about data folder location and this code is only for linux


active_callback: Maximum call stack size exceeded

Issue description:
Maximum call stack size exceeded at Object.checkCondition (field-dependencies.js?ver=3.0.33:103)
Version used:
3.0.33
Using theme_mods or options?
theme_mods
Code to reproduce the issue (config + field(s))
$dont_display = '__return_false';

'active_callback' => array(
	array(
		'setting'  => 'other_setting',
		'operator' => '==',
		'value'    => true,
	),
	$dont_display
),

ex. __return_false worked fine like this in a previous version I worked with (3.0.15)
If this behavior is changed, how should we approach multiple active_callbacks which are not depending on an option?

ex. __return_false worked fine like this in a previous version I worked with (3.0.15)

I don't believe this was ever working in previous versions if combined with other items... It can be either a valid callback, or an array or arguments. But not a combo of the 2... Can you please confirm that you had that working on a previous version when using both a callback and an array of arguments?
If yes then we'll need to backtrack this and figure out what changed

Hi @aristath
Yes, I can confirm this was working, I have a theme project which includes 3.0.15 (within the project), where we configured it like this.
We could manage when returning a simple boolean to instead of a PHP function, but that isn't working either.
$conditional_display = x ? true : false;

'active_callback' => array(
	array(
		'setting'  => 'other_setting',
		'operator' => '==',
		'value'    => true,
	),
	$conditional_display
),

I can confirm that this no longer works. In older versions there was a PHP implementation that was handling all field dependencies. However that implementation had a lot of bugs and was causing a lot of issues when using postMessage or when the conditions were too complicated, resulting in server timeouts etc.
That implementation has been removed since v3.0.15, but almost nobody uses mixed PHP conditions and field dependencies so this error was not discovered until now.
The bad news is that we can't really make this work the way it did unless we revert to the previous buggy implementation.
We could make this work, but only if we dropped support for PHP 5.2 since it will require using an anonymous function, but that's easier said than done since there are hundreds of themes out there that use Kirki and we can't drop support for PHP 5.2 without a proper plan.
The good news is that I just pushed a commit that will not throw any errors. However, it does that by removing the extra PHP conditions and only keep the field dependencies since these are implemented via JS. This fix will be included in v3.0.34 which will be released soon.
Alternative solutions:
You could change the active_callback in your theme to use a valid PHP callback.
If you're using PHP 5.3+ you can do it like this using an anonymous function:
'active_callback' => function() {
	$dependency_option = get_theme_mod( 'other_setting', true );
	$other_condition   = is_singular();
	if ( true === $dependency_option && $other_condition ) {
		return true;
	}
	return false;
},
```

If for some reason you need to keep compatibility with PHP 5.2 (which is the unfortunate case for Kirki) then you can just create a function like this:
```php
function custom_function_check_dependency() {
	$dependency_option = get_theme_mod( 'other_setting', true );
	$other_condition   = is_singular();
	if ( true === $dependency_option && $other_condition ) {
		return true;
	}
	return false;
}
```
and then use that function as a callback:
```php
'active_callback' => 'custom_function_check_dependency',
```

Thanks, we'll see how we can change this

The good news is that I just pushed a commit that will not throw any errors.

Errors are a good thing, otherwise we would not know what went/is wrong 🙂


Provide a company page

Provide a company page for users to be able to view postings by a company.
When a user views a job posting or views the posting on the index, search or job posting view the company will have a link to a new company page.

The company page should include:

The name of the company
List any job postings by that company (like it does on the index)
There should include a back button in the top right.
The results should also be paginated
The postings should be available in rss and json like on the index.

Additional Features

Integrate with Glassdoor (if possible) to provide feedback about the company.
Provide a link to the company website (this may not be possible because the postings don't always include a link to the company site)
Add filter for the postings

This might be a little large of a feature for someone else to work on. I can provide clarifications if needed and answer any questions if someone does pick it up.

I'd be able to take a swing at this.

Thanks @jereinhardt that would be great!


cmd/go: go clean <package> downloads modules

#28680 is fixed and go clean --modcache command no longer donwloads modules before cleaning if no argument is given to the command.
Note however 'go clean' accepts packages parameter as an argument(which isn't relevant to -modcache but is still meaningful for other purposes of go clean). If they are given, go clean still downloads the modules and even updates the go.mod file.

$ go help clean
usage: go clean [clean flags] [build flags] [packages]
...
$ export GOPATH=$(mktemp -d)
$ go1.13beta1 clean -testcache -cache -modcache
$ go1.13beta1 mod init scratch
go: creating new go.mod: module scratch
$ go1.13beta1 clean  golang.org/x/text
go: finding golang.org/x/text v0.3.2
go: downloading golang.org/x/text v0.3.2
go: extracting golang.org/x/text v0.3.2
$ cat go.mod
module scratch

go 1.13

require golang.org/x/text v0.3.2 // indirect

And the go clean -modcache <package> behaves differently.

$ go1.13beta1 clean -modcache golang.org/x/text
$ go1.13beta1 clean -modcache golang.org/x/text
go: downloading golang.org/x/text v0.3.2
go: extracting golang.org/x/text v0.3.2
go: finding golang.org/x/text v0.3.2

Probably the best we could do here is to detect that the argument to clean is not in the main module and error out.
Per https://tip.golang.org/cmd/go/#hdr-Remove_object_files_and_cached_files:

If a package argument is given or the -i or -r flag is set, clean removes the following files from each of the source directories corresponding to the import paths: […]

But of course we cannot remove files from the (read-only) module cache.
CC @jayconrod

I have following project:

greeter
|- go.mod: defines module a.b.c
|- *.go

after I run go build it generates a.b.c, after I run go test it generates a.b.c.test.
But when I run go clean, a.b.c and a.b.c.test not removed, while in go 1.12.6 it works fine.

@hitzhangjie, that seems mostly unrelated to the issue reported here. If you are seeing something you believe to be a regression, please file a new issue with complete steps to reproduce it.


Check HOME variable before using it

Fixes #1497

Unfortunately, the logic should be reverse. I myself never remember which way around +x notation works. The fix is in #1551.


Confine dialog scroll to viewport

See the test-dialog page
Before
https://github.com/radix-ui/themes/assets/8441036/f142a81b-295a-4776-80b6-164a17b14a53
After
https://github.com/radix-ui/themes/assets/8441036/037bd2eb-caed-4743-ab97-d3207ae8573e

@kendallstrautman can you think of a case where the UX would be a better fit with scroll confined to the dialog content?
Looking at your example, you might just need an overflow: auto there to re-create the previous behaviour

@vladmoroz I'm not sure if the UX is better with a fixed height dialog, but it's a pattern I do see a lot where someone wants a dialog to feel fixed and not scroll on the rest of the page. I don't think its a blocker, just noting that if folks do have just height set, they'll need to adjust somethings to retain that behavior with the new release.

Interesting, yeah I’d assume that design is more common because it's just trivial to implement compared to the full-height scroll which comes with a few gotchas.
Anyway, doesn't cost us anything to also add overflow: auto on the dialog content to allow fixed-height dialogs, added that:
https://github.com/radix-ui/themes/assets/8441036/6c045599-5237-486a-93f9-edb25a6b82be


How to activate the extensions?

Category

[X] Question
[ ] Bug
[ ] Enhancement

Expected or Desired Behavior
Are there any instructions on how to activate the extensions in this solution? I mean step by step instructions.

If you aren't deploying through the provisioning template, you'll have to do it manually through PowerShell using Add-PnPCustomAction and passing in the information for the Custom Action
https://docs.microsoft.com/en-us/powershell/module/sharepoint-pnp/add-pnpcustomaction?view=sharepoint-ps
You can find the name and the component id inside the provisioning XML around line 900 https://github.com/SharePoint/sp-starter-kit/blob/master/provisioning/starterkit.xml
I have a couple blog posts that show how to add them at the bottom.
https://beaucameron.net/2018/07/19/customizing-the-pnp-starter-kit-for-office-365-part-1-portal-footer/

If you aren't deploying through the provisioning template, you'll have to do it manually through PowerShell using Add-PnPCustomAction and passing in the information for the Custom Action
https://docs.microsoft.com/en-us/powershell/module/sharepoint-pnp/add-pnpcustomaction?view=sharepoint-ps
You can find the name and the component id inside the provisioning XML around line 900 https://github.com/SharePoint/sp-starter-kit/blob/master/provisioning/starterkit.xml
I have a couple blog posts that show how to add them at the bottom.
https://beaucameron.net/2018/07/19/customizing-the-pnp-starter-kit-for-office-365-part-1-portal-footer/

@bcameron1231
Thanks for the info on activating this extension but I am still having issues. I installed the starter-kit by dropping the sppkg file into my app catalog. I can see the web parts on my Modern Comm Site.

I navigated to my Comm site and created a List called "Alerts" based on the specs
I created one item in that list
Using PS I executed the following
Connect-PnpOnline -Url https://mytenant.sharePoint.com/sites/CommunicationSiteModern
Add-PnPCustomAction -Title AlertNotification -Name AlertNotification -Location ClientSideExtensi
on.ApplicationCustomizer -ClientSideComponentId aa8dd198-e2ee-45c5-b746-821d001bb0e1

PS C:> Get-PnPCustomAction
Name              Location                                  Scope Id

AlertNotification ClientSideExtension.ApplicationCustomizer Web   0e5d82eb-df83-4201-a4d9-e045db54be48
When I go to the site now I do not see any notifications. What else am I missing?

I installed the starter-kit by dropping the sppkg file into my app catalog. I can see the web parts on my modern comm site. also created the Alerts list with given spec
When I go to the site now I do not see any notifications. What else am I missing?

@eoverfield @bcameron1231
Thanks in advance

When I look to debug these types of problems, my common approach is to open Chrome dev tools and watch the console log and the network tab.
In the console log I am looking for any specific errors that are being thrown that look to be a part of alerts.
The network tab may be more interesting in the beginning. First, is the alert extension JS being loaded? Something tells me it is, though PS should give you that insight into the custom action being added. If you do see the alerts js being downloaded, you can assume it is being at least initialized. At that point, check to see if the alerts list get request is throwing an error or otherwise providing bad data.


Gaston doesn't watch the imported .less/.css files

While working on the global.less (containing the variables that are imported etc) this is a must have.

fixed in tag 0.9.16


chore: bdd-spring-1574888028 to 0.0.1

chore: Promote bdd-spring-1574888028 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Is it possible to fetch data from txt, json instead of pdf?

@andrew3920029 yes. I tried my side to ingest txt file from local using TextLoader of langchain
You can check my sample code here
https://github.com/tuantranf/gpt4-pdf-chatbot-langchain/blob/feature/vietnamese-insurance-law-qa/scripts/ingest-data-text.ts

This project has support of docx, pptx, csv, txt, etc. With open source chromadb instread of Pinecone.
gpt4-pdf-chatbot-langchain-chromadb


Sort TestClock suspensions on insert rather than in the advance method

Fixes https://github.com/pointfreeco/swift-clocks/discussions/7
Looking better at it, looks like the implementation was technically working, because after getting the first suspension, then it was using advance, so the suspensions were sorted there.
Nevertheless, I think this approach is clearer and safer, so I though it would have been better to open the PR anyway

Hey @danyf90, yeah after looking into this a bit more this morning and trying to get a failing test I came to the same conclusion. It does work as-is.
I'm not sure this approach is necessarily safer or cleaner though. The main point of run is to loop while there is any work in the suspensions queue, and we can pick any element in the array to advance to. We could pick the last or even a random one, and it would all work the same.
I do think, however, that we should definitely get test coverage on this. Would you like to revert your changes and write a test? Or I can do it if you don't have time.

Hey @mbrandonw, makes sense, I'll close the PR for now. Feel free to add the test if you have the time. 🙏
Just wanted to add that I think that if 3 of us misinterpreted the code, then it's probably an indication that the current implementation is not super clear.


Suggestion: document what lay_coefficent does

in the config file comment there is no documentation on what lay_coefficent does, and it appears to be 1.0 for all chickens

In Curseforge.com, the description of this mod claims it is "heavily customizable: change laying rates, items, disable chickens, modify breeding, set drops, ...", so I assumed lay_coefficient would affect the rate at which chickens lay items, acting as a multiplier to their growth stat. However, after some testing, they lay items just as fast with "lay_coefficient": 1.0 and "lay_coefficient": 0.001. It would definitely be a good idea to tell players what this does.

Maybe it is with how I read the example text at the top of the config file, but I understood the "lay_coefficient" as a quantity multiplier for the item the chickens are supposed to drop. So instead of changing how fast the chicken produces an item (which is already altered by the chicken's growth, gain and strength stats), it changes how many items are produced per chicken when they finally drop.

Having a brief look at the code, it looks like lay_coefficient affects the frequency that chickens lay eggs, as a multiplier of "time between lays". So setting it to 5.0 would make chickens wait longer between each lay, and setting it to 0.5 would make chickens lay more frequently. If you're more curious than me in how much the coefficient affects the frequency exactly, and are not satisfied with my lazy "more or less frequently" description, you can look in the EntityChickensChicken.resetTimeUntilNextEgg() method.
This seems to only affect free-range chickens and has no impact on roost chickens. I can only see it referenced when a chicken outside of a roost lays an egg; it is not referenced at all in the roost.


Support for Datalake

Is there an existing issue for this?

[X] I have searched the existing issues

Description
Add is_hns_enabled to the storage account to enable the data lake. Ensure it has Storage Blob Data Contributor on enable and add a data lake gen2 filesystem.
New or Affected Resource(s)/Data Source(s)
3.55
Potential Terraform Configuration
resource "azurerm_storage_account" "storage" {
  ...
  is_hns_enabled           = var.enable_datalake
}

resource "azurerm_storage_data_lake_gen2_filesystem" "file_system" {
  count              = var.enable_datalake ? 1 : 0
  name               = var.datalake_filesystem_name
  storage_account_id = azurerm_storage_account.storage[0].id
}

References
No response

Talk with @jrspinella to review needs for DataLake prior to starting this issue.


Included Upgradeable Contract

This is the approach taken by 0xytocin Labs for making the ERC721A contract upgradeable.

https://github.com/chiru-labs/ERC721A-Upgradeable
We already have upgradeable contracts.


docker-compose containers cross communication

What i am trying to do is to make a request from one container to the an other
So this is my docker-compose file
version: '3'

networks:
  traefik-net:
    driver: bridge

services:
  # The reverse proxy service (Træfik)
  reverse-proxy:
    image: traefik  # The official Traefik docker image
    ports:
      - "80:80"      # The HTTP port
      - "8082:8082"  # The Web UI (enabled by --api)
    volumes:
      - /var/run/docker.sock:/var/run/docker.sock
      - ./traefik.toml:/etc/traefik/traefik.toml
    labels:
      - "traefik.docker.network=traefik-net"
    networks:
     - traefik-net

  auth:
    image: auth
    labels:
      - "traefik.enable=true"
      - "traefik.backend=auth"
      - "traefik.frontend.rule=Host:auth.localhost"
      - "traefik.docker.network=traefik-net"
    networks:
     - traefik-net

  clients:
    image: clients
    labels:
      - "traefik.enable=true"
      - "traefik.backend=clients"
      - "traefik.frontend.rule=Host:clients.localhost"
      - "traefik.docker.network=traefik-net"
    networks:
     - traefik-net

and this is my traefik.toml file
defaultEntryPoints = ["http"]

[api]

[docker]
  endpoint = "unix:///var/run/docker.sock"
  domain = "traefik.localhost"
  watch = true

[entryPoints]
  [entryPoints.traefik]
  address = ":8082"

  [entryPoints.http]
  address = ":80"

What i am trying to do is to make a request from auth container to the clients container
Inside the auth container i execute this commande

wget -qO- --header="Host: clients.localhost" http://localhost/

i get this output

wget: can't connect to remote host (<IP_ADDRESS>): Connection refused

outside the container the commande works just fine.
what can i do to make requests from one container to the other using traefik
thanks for the help :)

Hi! I'm Træfiker :robot: the bot in charge of communication regulation.
Thanks for your interest in Træfik!
We dedicate the issue tracker to bug reports and feature requests only.
If you have questions and need community support, you can join our Slack workspace and reach out to us on the #support channel.
Another option is to use Stack Overflow and tag your question traefik.
Of course, double checking Træfik's documentation is always a good idea :sweat_smile:


source option

Is your feature request related to a problem? Please describe.
When testing it's useful to get an operation's source
import { NoParamQuery } from '../codegen/schema';

describe('with NoParamQuery script child', function() {
  let element;
  beforeEach(async function() {
    element = await getElement();
    element.innerHTML = `<script type="application/graphql">${NoParamQuery.loc!.source.body}</script>`;
    await nextFrame();
  });

  it('sets query property', function() {
    expect(element?.query).to.deep.equal(gql(NoParamQuery.loc!.source.body));
  });
});

Describe the solution you'd like
.graphqlrc.yml
extensions:
  codegen:
    config:
      dedupeOperationSuffix: true
      namingConvention: keep
      documentVariableSuffix: ''
      source: true

    generates:
      test/schema.ts:
        schema: test/graphql/test.schema.graphql
        plugins:
          - typescript
          - typescript-operations
          - typed-document-node

        documents:
          - test/graphql/*.query.graphql
          - test/graphql/*.mutation.graphql
          - test/graphql/*.subscription.graphql
          - test/graphql/*.fragment.graphql

Describe alternatives you've considered
build-time transformers like rollup-plugin-graphql etc.
Additional context
source could be boolean|string[] to provide users with an allowlist of operations to include sources for, if they need source at runtime for some reason but want to lower bundle sizes.
source:
  - NoParamQuery
  - SourcyMutation

Hi @bennypowers !
You can use print to get it for now, as a workaround. If you want, feel free to create a PR for it!


Add VSO Formatter

Add a formatter for Visual Studio Online/Team Foundation Server integration.
Closes #1281.

Looks good to me, thanks @cgwrench! It'll be included in a release in the next few days

@JKillian Great, thanks very much.


added triangular matrix multiplication kernel

Companion to #213, adding a file specifically for the development of this matmul.
Also shows different intermediate kernels on the way towards efficiency.
To give a break from all the maths and indexing in the code, the development of these is described as a story.
Some of the metaphors are stretched quite a bit, so feel free to make adjustments, but I  hope that overall, this might be easier to follow than just "indexing with this formula to achieve coalesced access".
Currently, the reads in the inner loop still cause 2-way bank conflicts, so there is still room for improvement.
Timings on my machine:

time 1.40 ms vs 2.37 ms for CuBLAS

Given that we're doing only half the work, that leaves us still 20% less efficient than cuBLAS.

Wow, you really had a lot of fun with the TriMatlon 😂 😂 😂
The most incredible fusion of art and engineering I've seen yet :D


iOS KeyboardAvoidingView not Entirely Avoiding Keyboard

Description
I'm trying to use a keyboard avoiding view to basically push up the entire app window on iOS (mocking the behavior seen by default on Android). KeyboardAvoidingView is having some trouble by not pushing things up quite enough to make it fully visible, the iOS keyboard's autocorrect menu obscures the very bottom of the contents.
Reproduction Steps and Sample Code
Not really doing anything fancy here, this is actually even simpler code which results in the following screen shots. (The tab bar at the bottom of the app doesn't and isn't supposed to move with the rest of the view. It's intentionally obscured by the keyboard).
<KeyboardAvoidingView behavior="padding" style={{flex: 1, backgroundColor: "#FF0000", justifyContent: "flex-end"}}>
    <PtTextInput theme="dark"/>
</KeyboardAvoidingView>

Solution
Not sure what the problem is. Is the keyboard height not being properly calculated?
Additional Information

React Native version: 0.43.2
Platform: iOS
Development Operating System: MacOS
Dev tools: iPhone 7 simulator, iOS 10.1

Looks like it has to do something with this nav bar at the bottom, if I wrap the entire thing then it correctly lines up.

I'm seeing something similar- I can tell the TextInput is moving up as the keyboard appears, but it's not moving up the entire way, and is completely covered by the keyboard. Same React Native version I believe, using iPhone 6 simulator with iOS 10.2.
@nickcharles could you describe further what you did to make it work? Thank you.
Code:
`  render() {
return (
        <View style={styles.wrapper}>
          <KeyboardAvoidingView style={styles.sendControlContainerOuter} behavior='padding'>
            <View style={styles.sendControlContainerInner}>
              <TextInput style={styles.textBox}/>
            </View>
          </KeyboardAvoidingView>
        </View>
        );

}
}
const styles = StyleSheet.create({
wrapper: {
flex: 1
},
sendControlContainerOuter: {
flex: 1,
justifyContent: 'flex-end'
},
sendControlContainerInner: {
flexDirection: 'row',
justifyContent: 'space-between',
paddingHorizontal: 10,
paddingVertical: 3,
},
textBox: {
borderRadius: 5,
borderWidth: 1,
height: 44,
paddingHorizontal: 10,
flex: 3
}
})`

@llamaluvr I actually never got great closure with this. I ended up doing some fancy stuff that forces iOS to mimic Android behavior by shrinking the available window space when the keyboard pops up.
This involved making my top level view a KeyboardAvoidingView on iOS and just a normal View on Android, rending each using the Platform api.
It felt a little weird at first but I really like it. One less difference between Android and iOS that we have to worry about.

I was seeing the same problem and then came across this post: https://medium.freecodecamp.com/how-to-make-your-react-native-app-respond-gracefully-when-the-keyboard-pops-up-7442c1535580
I'm trying react-native-keyboard-aware-scroll-view as an alternative and it seems to be working so far:
https://github.com/APSL/react-native-keyboard-aware-scroll-view

Seems to be dependent on navbar behavior. I'm using the wix navigator and not drawing under the navbar (view starts at bottom of navbar instead of behind navbar).
Adding navbar height with keyboardVerticalOffset seems to work:
      <KeyboardAvoidingView
        behavior='padding'
        keyboardVerticalOffset={-64}
      >

+1 - with bugs like this, i can not for the life of me take React Native seriously ...

@braco Bravo! That worked perfectly for me (with React-Navigation)

I have my navbar on top so I needed keyboardVerticalOffset={64}
not -64 but +64

With a textInput inside an multiple absolutely positioned element, I had to set the offset to 500+

0.46.1 introduces following fix a975c1e. Hopefully it'll help. Anyone to confirm?

@sobstel  still doesn't work

@braco , Bravo! lifesaver. But I find in the comment that keyboardVerticalOffset

... is the distance between the top of the user screen and the react native view, which may be non-zero in some use cases.

So I guess it has nothing to do with the nav bar or some magic number. I tried on different devices and it turned out fine.

@dmr07 Thank you. You probably just saved me a few hours of trial and error

@sobstel I use 0.46.4 and I still have this bug but because of the suggestion section on top of the keyboard

Stilling seeing this bug.  Hate the thought of hardcoding in a value to fix this

it seems that when we use react-navigation we need to use keyboardVerticalOffset={64} as @skhro87 😢

Use paddingTop instead of marginTop if you have a navbar.

also had to do keyboardVerticalOffset per  https://github.com/facebook/react-native/issues/13393#issuecomment-310431054 albeit with a different value.

keyboardVerticalOffset  's value for iOS worked 64 and on android it did 178. I do not know if it can vary device to device dimensions .
@dmr07 the snippet below taken from https://github.com/jevakallio/react-native-chat-tutorial/blob/master/App.js
        <FlatList
          data={this.state.messages}
          renderItem={this.renderItem}
          inverted
        />
<KeyboardAvoidingView behavior="padding" keyboardVerticalOffset={this.state.navBarHeight} >
<View style={styles.footer}>
            <TextInput
              value={this.state.typing}
              style={styles.input}
              underlineColorAndroid="transparent"
              placeholder="Type something nice"
              onChangeText={text => this.setState({ typing: text })}
            />
            <TouchableOpacity onPress={this.sendMessage}>
              <Text style={styles.send}>Send</Text>
            </TouchableOpacity>
          </View>
        </KeyboardAvoidingView>
      </View>
.....
...
  container: {
    flex: 1,
    backgroundColor: '#fff'
  },
  row: {
    flexDirection: 'row',
    padding: 20,
    borderBottomWidth: 1,
    borderBottomColor: '#eee'
  },
  avatar: {
    borderRadius: 20,
    width: 40,
    height: 40,
    marginRight: 10
  },
  rowText: {
    flex: 1
  },
  message: {
    fontSize: 18
  },
  sender: {
    fontWeight: 'bold',
    paddingRight: 10
  },
  footer: {
    flexDirection: 'row',
    backgroundColor: '#eee',
  },
  input: {
    paddingHorizontal: 20,
    fontSize: 18,
    flex: 1
  },
  send: {
    alignSelf: 'center',
    color: 'lightseagreen',
    fontSize: 16,
    fontWeight: 'bold',
		padding: 10

  }

If I use behavior="position" then textinput disappear completely , if I set
position absolute nothing happens too

If anyone is interested, I found a pretty cool solution that worked for me by wrapping entire view with , eg
<KeyboardAvoidingView behavior="position">
  <View>
  {/* Put everything you got here, including inputs, etc  */}
  </View>
</KeyboardAvoidingView>

You can even put  on top and it's all good

@braco

<KeyboardAvoidingView
behavior='padding'
keyboardVerticalOffset={-64}

that works fine for me!!!
save my day!
Thanks!
but my value is '64' ,not '-64', anyone knows why?

@wellenzhong - top vs bottom?
https://github.com/facebook/react-native/issues/13393#issuecomment-313048502

@alxvallejo - how is your comment helpful?  UIKit native doesn't even contain a KeyboardAvoidingView.  This should have been handled by Apple from day one and the great shame is that devs have to make up their own solution over and over again.  How is this an issue with React Native specifically?

Hey my friend, I don't know u can figurred out but u have to add 'enabled' prop in KeyboardAvoidingView @nickcharles :) You can find in react-native docs in KeyboardAvoidingView
https://facebook.github.io/react-native/docs/keyboardavoidingview.html#docsNav

I recently experienced this issue on an absolute or fixed view. I could be caused by any issue, though one that I found was the miscalculated position, as they (the KeyboardAvoidingView component) will return 0 when using onLayout to get the height of the component. So that's why in some situation, we can use keyboardVerticalOffset to make it work.
Though in some case, when the view within KeyboardAvoidingView has a dynamic height, it doesn't seem possible to fix.
I came up with a different method that supports 'absolute', 'fixed', and doesn't resize child components. You can have a look at react-native-spacer.

<ScrollView>
  <KeyboardAvoidingView keyboardVerticalOffset={64} behavior="position" enabled>
  </KeyboardAvoidingView>
</ScrollView>

Note that the fix with
          keyboardVerticalOffset={
            64 /* see https://github.com/facebook/react-native/issues/13393 for why */
          }

does not work for iPhoneX... I put 84 instead of 64...

I am having the same issue when using react-navigation and a BottomTabNavigator. It accounts for the height of the component it's nested in and not the tab bar, so I have to set keyboardVerticalOffset to at least 78 to have it work on my device. Need to test on other platforms.

it seems that when we use react-navigation we need to use keyboardVerticalOffset={64} as @skhro87 suggested 😢

Dude, you are cool.

Here's the proper sizing for both platforms as well as iPhone X (this is using react-native-navigation). Values should be similar for other navigation libraries as well.
import { Platform } from 'react-native';
import { ifIphoneX } from 'react-native-iphone-x-helper';

const isIOS = Platform.OS === 'ios';

const statusBarHeight = isIOS ? ifIphoneX(44, 20) : 0;
const navBarHeight = isIOS ? 44 : 56;
const headerHeight = statusBarHeight + navBarHeight; // Use this for the keyboardVerticalOffset

Just found useful script for handling iPhoneX (in case you don't wanna add another dependency to your project).
https://gist.github.com/m-tymchyk/a7bd64ff2ee2844a4890de0aa6925fc0#file-is-iphone-x-js

This worked for me using react-native's createStackNavigator. It's slightly hacky but did the job. Basically you make the header absolute positioned and adjust your screen content accordingly.
When the header is absolute positioned, the KeyboardAvoidingView doesn't take it into account.
1. In navigation options for your screen:
First make the header absolute positioned by setting headerTransparent: true.
const InboxStack = createStackNavigator(
    {
        Screen1: {
            screen: Screen1,
            navigationOptions: () => ({
                headerTransparent: true
                // other things you put here
            })
        },
);

2. Adjust your screen style
This part is up to you. Basically, just set your screen paddingTop equal to the height of the header.
import React, { Component } from 'react';
import { View, KeyboardAvoidingView, Platform } from 'react-native';
import { Header } from 'react-navigation';

const behavior = (Platform.OS === 'ios') ? 'padding' : null;

export default class YourComponent extends Component {
render() {
    const headerOffset = Header.HEIGHT;
    <KeyboardAvoidingView behavior="padding" style={{ flex: 1 }}>
         <View style={{ paddingTop: headerOffset, flex: 1 }}>
            // your content here...
          </View>
   </KeyboardAvoidingView>
}
}

For me this worked like a charm:
`
import { Header } from 'react-navigation';
<KeyboardAvoidingView
keyboardVerticalOffset={Header.HEIGHT}
behavior={Platform.OS === 'ios' ? 'position' : undefined}
>
...

`

still not working for me

@mi-mazouz without some code people are not able to help you..

<KeyboardAvoidingView
        behavior={Platform.OS === 'ios' ? 'padding' : 'height'}
        style={{flex: 1}}
  >
<View style={[styles.container, {}]}>
      <Text>Hello</Text>
      <Text>Hello</Text>
      <Text>Hello</Text>
      <Text>Hello</Text>
      <Text>Hello</Text>
      <Text>Hello</Text>
      <Text>Hello</Text>
      <Text>Hello</Text>
      <Text>Hello</Text>
      <Text>Hello</Text>
      <Text>Hello</Text>
      <Text>Hello</Text>
      <Text>Hello</Text>
      <Text>Hello</Text>
      <Text>Hello</Text>
      <Text>Hello</Text>
      <Text>Hello</Text>
      <Text>Hello</Text>
      <Text>Hello</Text>
      <Text>Hello</Text>
      <Input placeholder='comment' />
</KeyboardAvoidingView>

@mi-mazouz
behavior={Platform.OS === 'ios' ? 'position' : undefined}

I got an empty view

https://snack.expo.io/r1MCaff7H

sorry it's from react-native-swipper, when I wrap it in <KeyboardAvoidingView..> it's not working well

probably something like this: https://snack.expo.io/S1XgS7fmH ?

@netbull thanks works perfectly, I missed alignItems: 'center'

For me this worked like a charm:
import { Header } from 'react-navigation';

<KeyboardAvoidingView
          keyboardVerticalOffset={Header.HEIGHT}
          behavior={Platform.OS === 'ios' ? 'position' : undefined}
        >
    ...
</KeyboardAvoidingView>

Works perfectly on iPhone 7 (iOS). Doesn't work so well on OnePlus One (Android).
On Android you have to also account for the status bar:
keyboardVerticalOffset={Platform.OS === "ios" ? Header.HEIGHT : (Header.HEIGHT + StatusBar.currentHeight)}

For me this worked like a charm:
import { Header } from 'react-navigation';

<KeyboardAvoidingView
          keyboardVerticalOffset={Header.HEIGHT}
          behavior={Platform.OS === 'ios' ? 'position' : undefined}
        >
    ...
</KeyboardAvoidingView>

Works perfectly on iPhone 7 (iOS). Doesn't work so well on OnePlus One (Android).
On Android you have to also account for the status bar:
keyboardVerticalOffset={
    Platform.OS === "ios" ? Header.HEIGHT : (Header.HEIGHT + StatusBar.currentHeight)
}

probably something like this: https://snack.expo.io/S1XgS7fmH ?

Try this https://snack.expo.io/rkoXmL9wr
Added more text, and more inputs in the middle. Also instead View added ScrollView. And it's a reproducible bug

keyboardVerticalOffset is doing absolutely nothing, and I'm not sure why. There is a view wrapping the keyboardavoidingView, and the keyboardAvoidingView is wrapping the TextInput. Thanks for the help.

While I've been able to use this component several times successfully it's really so blatantly a poor developer experience and needs an overhaul. I'm not saying I have a solution but it almost never works correctly for anyone the first time and requires lots of hacking and trial and error.

This might help someone...
KeyboardAvoidingView was working just fine for me until I changed the UI of my app and introduced SafeAreaView from React Navigation. The only other thing I changed was introducing flex elements nested within KeyboardAvoidingView. I tried the VerticalOffset of {64} and that did work, but no way that would be scalable across various devices and the overhead to manage that is not ideal, just my .02.
My solution was to do as suggested above and replace my wrapping  tag with  and just like that it works as expected; no vertical offset required, behavior set as follows: const padding = (Platform.OS === 'ios') ? 'padding' : null;

@wincod75 Just a note but I think VerticalOffset should work fine cross device because your header size will stay static no matter what device you're using. The PPI of the screen changes but the header from react-navigation is the same regardless of PPI. I believe that the offset shouldn't be for the notch area or the toolbar but for your internal chrome. Your solution is probably the ideal way of dealing with this issue though.


Can't Connect to Azure Hosted Bot

System Information

SDK Language: .NET
SDK Version: 3.9
Development Environment: Azure API

Bot Info

Bot name: BeyondBotEY
Bot handle: beyondbotey
App ID: 75069f6a-5d72-40ab-97fd-2f60d71fde64
Connected channels: Skype, Web

Issue Description
When turn on the auth (without auth everything is all right), the bot seems to connect to remote host (I'm using Bot Emulator), but when I try to send some message, I got [500] Internal Server Error.
Using the Fiddler, I got more details, it seems to be something related to OAuth, {
"message": "An error has occurred.",
"exceptionMessage": "Unauthorized",
"exceptionType": "Microsoft.Bot.Connector.MicrosoftAppCredentials+OAuthException",
"stackTrace": "   at Microsoft.Bot.Connector.MicrosoftAppCredentials.<RefreshTokenAsync
Reading a lot (most time here), I even checked the OAuth authorization using curl -k -X POST https://login.microsoftonline.com/botframework.com/oauth2/v2.0/token -d "grant_type=client_credentials&client_id=APP_ID&client_secret=APP_PASSWORD&scope=https%3A%2F%2Fapi.botframework.com%2F.default"
The answer was fine.
I tried to remove LUIS integration, just to make sure there is no issues related with that. But no success.
The auth method is by Web.Config. Tried to remove BotId, but not working...

Change the vms from Free, Shared and S1. My latests tests were with S1 instances.
I remove the bot and created on, same as application. No success.

What I'm missing? :(

There seems to be a problem with your bot code.  Do you have your code in a repo so we can look?  In the dev portal I see a bunch of errors in the webchat channel.

The errors are related to connection as related, for example "There was an error sending this message to your bot: HTTP status code InternalServerError", same in bot emutator.

I don't have the code in a repo. :(

In Microsoft App Dev, there are other configurations that I didn't set up. In some tutorials they don't exist.
For example, I should set a platform or grant any rule to Graph (I don't think so).

Running local, with auth (AppiId and AppPassword), same error:
Microsoft.Bot.Connector.MicrosoftAppCredentials+OAuthException: Unauthorized ---> System.Net.Http.HttpRequestException: Response status code does not indicate success: 401 (Unauthorized).
at System.Net.Http.HttpResponseMessage.EnsureSuccessStatusCode()
at Microsoft.Bot.Connector.MicrosoftAppCredentials.d__25.MoveNext()
--- End of inner exception stack trace ---
at Microsoft.Bot.Connector.MicrosoftAppCredentials.d__25.MoveNext()

500 error generally means there is a problem with your bot code.

Hi @michelpf
Is your bot deployed to the messaging endpoint configured in dev.botframework.com?  https://YourSiteName.azurewebsites.net/api/messages
There are a few ways to debug locally.  The easiest is to remove the MicrosoftAppId and MicrosoftAppPassword from the web.config, and do not supply them in the emulator.  This method doesn't use BotAuthentication security, and doesn't require a bot registration on dev.botframework.com

I don't know what happend, but I solved in the worst way for developers... Deleted bot account, app account, create new ones, create a new bot app, copy from old app and paste same code in the new app and Voilá, started to work. :(

@michelpf  I'm glad you've got it working now.  Thanks for letting us know.  And, I agree: your resolution method is not something any developer wants to do, ever.  Let's just pretend this didn't happen (since we don't know what actually happened anyway).


Optionally persist "open" state in Sidebar menu on page change

What version of starlight are you using?
0.11.0
What is your idea?
Currently, the menu categories/accordions are built with native <details> and <summary> elements. However, the state of the individual items (opened or closed) is not persisted across page changes/navigation.
So, when navigating to another page, the menu items are always reset to their default state.
It'd be nice to use localStorage to persist the menu state even when navigating around.
Why is this feature necessary?
It's confusing and not understandable to users why this happens. There are many similar applications which keep their menu state. The user expects this to "just work".
Do you have examples of this feature in other projects?
No response
Participation

[X] I am willing to submit a pull request for this issue.

This is possibly a duplicate of #307 talking about scroll position and sidebar state restoration.
No built-in solution has been implemented yet while the various possible solutions are being discussed but the linked issue contains a few workarounds if you want to try them out.

Thanks for pointing me to the other issue. Indeed, it's a duplicate, so I'm closing this one and will participate in the other discussion.


Add transaction history for token and address

:tada: This PR is included in version 2.7.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


CPPKI: Implement signature validation based on chain

Should check that the TS is covered by the validity.

dupe of #3119


ERROR : using Docker for W 17.03.1-ce-win5

Expected behavior
Actual behavior
λ docker-compose start
Starting db ... done
Starting maildev ... done
Starting php ... done
Starting web ... error
Starting pma ... done
ERROR: for web  Cannot start service web: oci runtime error: container_linux.go:247: starting container process caused "process_linux.go:359: container init caused "rootfs_linux.go:54: mounting \"/c/var/www/config/local.dev/httpd/httpd.conf\" to rootfs \"/var/lib/docker/aufs/mnt/f357c654eeef97abfc38073b653f5d46e2bce05c540624e30e02cc2c81f9b6ff\" at \"/var/lib/docker/aufs/mnt/f357c654eeef97abfc38073b653f5d46e2bce05c540624e30e02cc2c81f9b6ff/usr/local/apache2/conf/httpd.conf\" caused \"not a directory\"""
: Are you trying to mount a directory onto a file (or vice-versa)? Check if the specified host path exists and is the expected type
Information about the Issue
Steps to reproduce the behavior

...
...

Sorry, after 3 restart it works !!


support export to pdf in obsidian

envirment:

ubuntu: 22.04
obsidian: v0.14.6
obsidian-kroki: 1.2.2

problem

pictures not show in export pdf

action
write down content in markdown file as below

\```plantuml
[*] --> State1: init
State1 --> State2: action1
State2 --> State1: action2
State2 --> [*]: end
\```

and picture can normaly show in obsidian preview mode
click top-right three-nod button  and choose export to pdf
picture not show in pdf

I've tested this in Macos and the exported diagram in visible in the PDF just fine.
I'll try ubuntu in a bit and let you know what I find.


Explicitly add ostype to  az container create

The  CI build for RabbitMQ failed with the following error :
ERROR: (InvalidOsType) The 'osType' for container group '' is invalid. The value must be one of 'Windows,Linux'.
Code: InvalidOsType
Message: The 'osType' for container group '' is invalid. The value must be one of 'Windows,Linux'.
This PR addresses the error by explicitly specifying the ostype while creating the azure container. The os-type has been defaulted to use Linux as Rabbitmq images do not have a windows variant

The os-type has been defaulted to use Linux for better performance

What does that mean? Rabbitmq images do not have a windows variant as far as I recall so isn't the Linux the only sensible choice?

The os-type has been defaulted to use Linux for better performance

What does that mean? Rabbitmq images do not have a windows variant as far as I recall so isn't the Linux the only sensible choice?

Yes, I have updated the description


When will you release the code?

Thank you for your work. When will you release the code?

Thank you for your interest in our work. We are currently organizing the code and expect to open-source it around next week.


Screen: Warned Persons (EXPOSUREAPP-4511)

Implementation of Key Submissions Card

The primary value and label seem incorrect. The spacing on the bottom seems to be off.

The primary value and label seem incorrect. The spacing on the bottom seems to be off.

The primary value and label seem incorrect. The spacing on the bottom seems to be off.

The primary value and label seem incorrect. The spacing on the bottom seems to be off.


React Native problem: Cannot inject event plugin ordering more than once.

This works perfectly with react-redux 3.x, but with 4.x it got broken.
It works with 3.x:

Clone this repo: https://github.com/alinz/example-react-native-redux
Run it in XCode, and it works perfectly.

With react-redux 4.x, it's broken:

Delete all dependencies in package.json file so next time on npm install the latest will be installed
Install the latest dependencies

cd example-react-native-redux/Counter
npm install react react-native react-redux redux redux-thunk --save

In app/containers/app.js and counterApp.js remove the native suffix after react-redux import
Open XCode project and Run it

open ios/Counter.xcodeproj

You will get an error an Invariant Violation error: please see this screenshot

The problem is definitely with react-redux because if I comment the react-redux import lines and remove all code that depends on either connect or Provider, than it works perfectly.
Thank you,
Peter

From the README:

Until React Native works on top of React instead of shipping a fork of React, you’ll need to keep using React Redux 3.x branch and documentation.


✨ Add support for proxied docker #758

New Vars (if applicable)
DOCKER_HOST host for docker with url
DOCKER_PORT port for docker with url

FYI: @SmartPhoneLover new env vars

Thank you @manuel-rw
I will take it into account when next stable release (latest).


New features

LG TV support: The LG TV DLNA has some special requirements
Subtitle support
Renders list feature

Awesome!
One thing:
I'm using the Standard Codestyle in the project. See https://github.com/feross/standard
Can you format the code in that way?
I'll gladly merge it then.

Yeah, no problem... Yesterday add some more new things... Like use the module as a module by another modules.

nice, sounds cool :)

Now follow the rules :innocent: hahaha
Has the new features and remove some old and unused packages

Thanks!


Cannot read property 'some' of undefined

macOS version: 10.13.6
Kap version: v3.3.0
Steps to reproduce
Was a fresh install on a mac running OSX 10.13.6
Once OSX security allowed the app to run as an unsigned developer, start-up would fail with the following dialogue.

Current behaviour
Kap will not start, error dialogue box is broken and can not be interacted with (e.g. the buttons can not be pressed)
Expected behaviour
Kap to start up
Workaround
No workaround found.

Couldn't really spot how getAudioDevices() isn't returning an array.
https://github.com/wulkano/Kap/blob/35f18894b3c77fe710acf4507d45e920498877cf/main/utils/devices.js#L9
Installed https://github.com/karaggeorge/macos-audio-devices locally and got the following;
apple$ audio-devices
dyld: Library not loaded: @rpath/libswiftCore.dylib
  Referenced from: /usr/local/bin/audio-devices
  Reason: image not found
Abort trap: 6

Sorry about that, fixed in #915
In the meantime you can get it to work by following this:

Requires macOS 10.12 or later. macOS 10.13 or earlier needs to download the Swift runtime support libraries.

That seemed to fix it. Thanks!

@mcvicar just out of curiosity for a different feature we released:
When you got the error dialog, did you click on the collect & report? If so, how did it go?
Or did you just copy the error and create the issue manually?

I couldn't interact with the error dialog at all, and had to create the issue manually.
When the dialog appeared the cursor was "precision select", none of the buttons could be pressed and the text in the toolbar was all messed up.

Thank you for the info! We'll try to fix that


Revert "Connectables-based effect router in mobius-core"

Reverts spotify/mobius#22

Codecov Report

Merging #32 into master will decrease coverage by 1%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master      #32      +/-   ##
============================================
- Coverage     75.17%   74.17%   -1.01%
+ Complexity      243      222      -21
============================================
  Files            67       63       -4
  Lines          1402     1305      -97
  Branches         77       67      -10
============================================
- Hits           1054      968      -86
+ Misses          334      323      -11
  Partials         14       14

Impacted Files
Coverage Δ
Complexity Δ

...om/spotify/mobius/internal_util/ImmutableUtil.java
92.85% <ø> (-2.15%)
5 <0> (-2)

...-core/src/main/java/com/spotify/mobius/Mobius.java
78.94% <ø> (-0.37%)
2 <0> (-1)

Continue to review full report at Codecov.

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


Installing sqlx cli

It seems the README instructions don't currently work:
$ cargo install sqlx-cli
    Updating crates.io index
error: could not find `sqlx-cli` in registry `https://github.com/rust-lang/crates.io-index`

cargo install will not select a pre-release version by default, you need to specify it:
$ cargo install --version=0.1.0-beta.1 sqlx-cli


[AI Agent idea] English to Hindi Translator

Description
It will be a nice one to be added in the list
Additional Information
No response

@maitri-vv  if u like it please assign me the issue

@ayan-joshi Hello, she is not our member. If you want to do it, you can enter the command /assign @yourself

/assign @ayan-joshi


argo submit --watch never exited even though all pods got to completed state

Is this a BUG REPORT or FEATURE REQUEST?:
BUG
What happened:
I've started argo submit --watch some_workflow.yaml from a prow presubmit test.
The command printed the workflow info twice and never printed anything else after that (no logs, no steps). The command never exited and hung forever.
Running argo get I see that the workflow status is Succeeded.
What you expected to happen:
I expected the command to exit as soon as the workflow finished. I also expected it to print the steps in the end.
How to reproduce it (as minimally and precisely as possible):
The workflow is private, but i's pretty simple - just builds several docker images in parallel and that's it.
Anything else we need to know?:
Environment:

Argo version:

$ argo version
argo: v2.2.0
  BuildDate: 2018-08-30T08:52:05Z
  GitCommit: af636ddd8455660f307d835814d3112b90815dfd
  GitTreeState: clean
  GitTag: v2.2.0
  GoVersion: go1.10.3
  Compiler: gc
  Platform: linux/amd64

Kubernetes version :

$ kubectl version -o yaml
clientVersion:
  buildDate: 2018-08-07T23:17:28Z
  compiler: gc
  gitCommit: bb9ffb1654d4a729bb4cec18ff088eacc153c239
  gitTreeState: clean
  gitVersion: v1.11.2
  goVersion: go1.10.3
  major: "1"
  minor: "11"
  platform: linux/amd64
serverVersion:
  buildDate: 2018-08-16T21:33:47Z
  compiler: gc
  gitCommit: 9b635efce81582e1da13b35a7aa539c0ccb32987
  gitTreeState: clean
  gitVersion: v1.9.7-gke.6
  goVersion: go1.9.3b4
  major: "1"
  minor: 9+
  platform: linux/amd64

Other debugging information (if applicable):

workflow result:

$ argo get build-images-wlvmq --cluster gke_blabla
Name:                build-images-wlvmq
Namespace:           default
ServiceAccount:      default
Status:              Succeeded
Created:             Mon Sep 10 02:20:49 -0700 (7 hours ago)
Started:             Mon Sep 10 02:20:58 -0700 (7 hours ago)
Finished:            Mon Sep 10 02:26:50 -0700 (7 hours ago)
Duration:            5 minutes 52 seconds
Parameters:
  ....

STEP                                  PODNAME                        DURATION  MESSAGE
 ✔ build-images-wlvmq
 └-·-✔ build-a         build-images-wlvmq-750169231   4m
   ├-✔ build-b       build-images-wlvmq-2748166194  2m
   ├-✔ build-c           build-images-wlvmq-4238954511  5m
   ├-✔ build-d   build-images-wlvmq-1958430995  4m
   └-✔ build-e  build-images-wlvmq-3156406827  3m

executor logs:

$ kubectl logs <failedpodname> -c init
$ kubectl logs <failedpodname> -c wait

workflow-controller logs:

$ kubectl get pods -l app=workflow-controller -n kube-system --cluster gke_
<nothing>

This is over 6 months old. Please re-open if still an issue.


Show how to store IRCompiler inside IRObject for reduced function complexity

This is a pure excerpt from #13 that merges into #12.  The former PR will be merged separately without clobbering #12 with too many changes (specifically reverting the Arc system from #11 to the original lifetimes).

Note that this ended up being super inconvenient, but also "wrong" now that we have a separation between IRCompiler and the loaded library. This is effectively reverted in #15, see that PR for more info.


Not compatible with cython 3.0

As it says above, I get compile errors when installing through pip. Mostly implicit conversion is happening between types.

Fixed by #246


rename pg component to paragraph component

Fixes #47

Thank you! Before merging this, we need to make sure, that users get informed about this breaking change. Either we provide backwards compatibility (leaving the pg component as is - next to the new paragraph component - for the next releases and add a warning when being used) or communicate this breaking change explicitly. I would prefer a structured and documented approach including terminated backwards compatibility. What do you think?

Great thinking from your side. I have re-added the pg core component including tests and the docs, now mentioning its deprecation and linking the "new" paragraph component.
Maybe we should introduce a deprecation warning from the 0.7 relase on and remove it completely at the 0.8 release?


Register R12 requires special encoding in modrm bytes - Avoid using it to simplify x64 code gen

Register R12 requires special encoding in modrm byte - Avoid using it to simplify x64 code gen.
0:  45 8b 3b                mov    r15d,DWORD PTR [r11]
0:  45 8b 3c 24             mov    r15d,DWORD PTR [r12]
For reason not yet understood, a move operations with R12 as indirect source results in a 4 byte instruction.
Switch the register mapping to avoid R12.

More background on this:
R12 and RSP use the same register encoding (excluding the bit in REX).
R13 and RBP use the same register encoding (excluding the bit in REX).
emit_modrm_and_displacement correctly handles encoding R13 as an EBP relative addressing, but doesn't do RSP relative addressing correctly. This results in malformed instruction.
https://github.com/iovisor/ubpf/blob/221bbe075e7f461efe7c3c066964c2ea9b57a93f/vm/ubpf_jit_x86_64.h#L125


User is not able to delete / modify a property when it was created with a space in the name

@rkarputs commented on Tue Jul 21 2020
This is a...

[ ] Feature request
[ ] Regression (a behavior that used to work and stopped working in a new release)
[x] Bug report
[ ] Documentation issue or request

Description
When user created a property in the Data Mapper with a space in the name, e.g. 'new test', he is not able anymore to modify its Name / Value / Value Type, all changes just cannot be applied after saving. Also user cannot delete this property.
Steps to reproduce

Go to the Data Mapper.
Click on the '+' button in the Properties tab to create a new property.
Specify property name with a space, e.g. 'new test'.
Click the 'Confirm' button.
Try to modify the property (name/value/value type).
Try to delete the property.

Actual result
Nothing happens when trying to modify/delete the property. Changes are not applied.
Expected result
User is able to modify/delete the property. Or if spaces using is forbidden, user should not have the ability to use them.

@stale[bot] commented on Sat Oct 24 2020
This issue has been automatically marked as stale because it has not had any activity since 90 days. It will be closed if no further activity occurs within 7 days. Thank you for your contributions!

@stale[bot] commented on Thu Jun 10 2021
This issue has been automatically marked as stale because it has not had any activity since 90 days. It will be closed if no further activity occurs within 7 days. Thank you for your contributions!

@zregvart commented on Fri Jun 11 2021
@igarashitm one to consider as well

Hey @igarashitm - so what specifically do we want to support?  The latest UI doesn't allow a space in a property name.
Are we saying users can enter
new test
or
'new test'
and do we really want to add this ability (thinking of parsing issues due to spaces in property names appearing in conditional expressions).

I think it's OK to bun a field name with space. I wonder what happen for constants though. The constant value would want those special characters in some usercases. It would need a name apart from value for constants.

I think created constants only makes sense for the UI mappings - not for conditional expressions.  In other words, the user would always simply embed the constant into the expression since the constant value and name are the same.
So - should we close this issue since there is no supported property names the contain spaces?

true for expression, but still a problem for non-expression mapping isn't it?

Since you can go from mapping details to the expression I don't want to have different rules for properties in each scenario.  Basically I was suggesting no spaces in propeprty names anywhere.

I'm now confused, no space in property names sounds OK. What I'm worrying is no space in constant value is too restrictive. And the solution would be to separate constant name from constant value.

And then no space in constant name is also OK.

Ok - that makes sense.  I'll change the title of this issue to reference the fact that we're adding names to constants (no spaces in names) and we'll allow spaces in constant values.

Yep, sounds good!

Current dialog:

Now looks like this:


Not really an ISSUE but a notice about query() function

When using $database->query() function, it is important to know that an INSERT statement has to be written WITHOUT the trailing semicolon!
So,
$database->query("INSERT INTO table VALUES (1, 2);");  WILL NOT WORK,
but
$database->query("INSERT INTO table VALUES (1, 2)"); will work flawlessly.
Thank you for creating such a great tool, and please consider updating the documentation with this small addendum, you might save someone a few hours next time :)

Where the columns? INSERT INTO table VALUES (1, 2). My query works with and without a semicolon.
(Use exec() for INSERT/UPDATE queries.)

And @catfan I really didn't want to create an issue. But there are some really too much of grammar mistakes in the documentation. I know that your main language is not English. But still, please take care of the grammar. In some places, we couldn't even understand what you are trying to say.
If possible, I will be happy to help you out

@smartclash Great! It's very appreciated if you can help it out. You can send me the correct content via email, so that I can update for it :)


Make using RPCs in cast CLI easier

Which components does the task require to be changed? (think hard pls)
sncast
Description
We should allow saving the RPC URL so we don't have to use profiles for using this - we're not always in a project scope, and it seems excessive, if you only want to do a few transactions in a day.
We could also provide some free default options (set and forget) which would be configured in installation, if no config is detected on disk. Those can be re-configured later.
We should allow storing a few URLs and setting a default, which will be used if no other RPC ID was specified.
This means we would have to store it in a file, in the installation directory, preserving it when migrating versions.

Marking as blocked by #2221

This should be addressed by the defaults in global config #2221

RPC URLs are poorly validated, this can be addressed in this or any other issue

Not creating issue for now, this doesn't seem that impactful. We can create one in the future.


[experimental] [WIP] jobs framework

EXPERIMENTAL
inspired by sourcehut: https://git.sr.ht/~sircmpwn/builds.sr.ht
ideas behind the jobs framework: https://hackmd.io/CPkdhdJQQnmy6uO0um6ABA#CI-requirements

Codecov Report

Merging #233 (08b983c) into master (76d64bd) will increase coverage by 0.23%.
The diff coverage is 91.45%.

@@            Coverage Diff             @@
##           master     #233      +/-   ##
==========================================
+ Coverage   83.67%   83.91%   +0.23%
==========================================
  Files          31       33       +2
  Lines        2414     2536     +122
==========================================
+ Hits         2020     2128     +108
- Misses        394      408      +14

Impacted Files
Coverage Δ

quetz/jobs/runner.py
80.85% <80.85%> (ø)

quetz/tasks/workers.py
95.79% <96.29%> (+0.05%)
:arrow_up:

quetz/jobs/models.py
100.00% <100.00%> (ø)

...ons/a3ffa287d074_case_insensitive_channel_names.py
68.75% <0.00%> (-6.25%)
:arrow_down:

quetz/testing/fixtures.py
96.87% <0.00%> (-2.35%)
:arrow_down:

quetz/tasks/mirror.py
91.85% <0.00%> (+0.26%)
:arrow_up:

Continue to review full report at Codecov.

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


Changed version to 1.0.0

Bot is functional(?) So it should be 1.0.0?

Doesn't matter, it's not an npm package. I don't do any versioning currently.


VALUE of blood type, birth, and citizenship?

VALUE of blood type, birth, and citizenship?
shares: lahat naman tayo iisa ang pinagmulan, kung tiningnan natin ito sa mahabang panahon;
--> halimbawa: SUMERIA, MESOPOTAMIA
--> adds: Lolo and Lola may NOT be the SAME as that of another household;
--> adds: NEED for QUICK MONEY is cause of NEED to be of same blood;


android: logcat if connectivity does not succeed in example apps

Signed-off-by: Jose Nino<EMAIL_ADDRESS>

Is there a non-interactive mode of adb logcat that's being used here?

Is there a non-interactive mode of adb logcat that's being used here?

Totally forgot that logcat is interactive. Going to look more into this.


Labels for solar thermals in buildings should be updated

For buildings, solar thermal panels cannot supply 100% of hot water demand (as for households), but at most 13% of space heating demand. See the input statement buildings_space_heater_solar_thermal_share:
# Solar thermal panels can provide a maximum of 13% of year heat demand. Setting slider at 100% means that 13% of heat demand comes from solar thermal and 87% from the main heating technologies in buildings.

- query =
    EACH(
      UPDATE(EDGE(buildings_space_heater_solar_thermal, buildings_useful_demand_for_space_heating_buildings_present), share, (USER_INPUT()*0.13)/100.0),
      UPDATE(EDGE(buildings_space_heater_solar_thermal, buildings_useful_demand_for_space_heating_buildings_future), share, (USER_INPUT()*0.13)/100.0)
      )

The frontend of the Space heating section for buildings however has the following structure:

There are two issues here:

[ ] The group label states "space heating and hot water" for the solar thermal panels, which in this case is applied incorrectly and should be fixed
[ ] There should a label stating "of potential" for the solar thermal panels in addition to the "% of heat demand" label, to clarify to user that they do not apply to the total heat demand

Solved in PR 4245


Problem display on phone

Hello
Bootstrap 4.1.1
I thinks a bug exists but I don't be sure.
Actually, i see on my phone two img and on my desktop one img
I should display
-> on my phone only the img "HeaderBackground_ht_mobile" with the text "test phone"
-> and on the desktop, only the img "HeaderBackground_ht_desktop" with the text "test desktop".
Thanks a lot for your response.
This is my codeplay
https://www.codeply.com/go/YZdc0YqTaB
`

Just use one of the display utilities https://getbootstrap.com/docs/4.1/utilities/display/

Sorry
I say well the docs but I always thinks this bug exists


Enhance Local Password and Key Management Security

The current implementation of password and private key handling stores decrypted private keys in localStorage, which poses a significant security risk:
auth.ts:
nuxtClientInit() {
  const decryptedPrivKey = ref<string | null>(localStorage.getItem(AUTH_LOCAL_STORAGE_DECRYPTED_KEY));
  if (decryptedPrivKey.value) {
    try {
      this.privKey = decryptedPrivKey.value;
      this.pubKey = getPublicKey(Buffer.from(this.privKey, 'hex'));
      this.authMethod = AuthMethod.LOCAL;
    } catch (err) {
      console.warn('Error setting local key from local storage: ', err);
      this.delete();
    }
  }
  watch(() => this.privKey, (newVal) => {
    localStorage.setItem(AUTH_LOCAL_STORAGE_DECRYPTED_KEY, newVal || '');
  });
},

Here are suggested fixes:

Avoid storing decrypted private keys:

Keep decrypted keys in memory and never persist them to localStorage.
Use Vue's reactive state or composables to handle keys temporarily.

Use Web Crypto API for Encryption:

Encrypt sensitive data before storing it.
Derive keys securely using a passphrase with PBKDF2 or Argon2.

Implement a TTL (Time-to-Live):

Add an expiration mechanism for encrypted keys stored in localStorage.

Keeping the unencrypted private key material in memory only is indeed a good idea. However take in consideration that a new private key management scheme will have to be introduced in the upcoming releases and the current one will be phased out. So I won't recommend putting too much time in modifying the current scheme for that.
You can more about the new scheme here: https://mostro.network/protocol/key_management.html

Ok. Gonna check in detail.


Shay/booking

Describe your changes
seed.js:
Added a bookings array with sample booking data
Added bookingData array that maps over the bookings array and assigns each booking to a user and a property
Booking schema, logic, routes:
Updated schema with pre save hook
Updated controller with consistent error handling and validation
Created booking routes with validation and authorisation
auth.js:
Modified protect middleware to correctly set req.user based on decoded token
Tested all routes - user, property, booking - to ensure middleware still valid
Example json object to create a new booking:
{
    "user": "user-id",
    "property": "property-id",
    "startDate": "2024-12-08",
    "endDate": "2024-12-11",
    "status": "Pending"
}

Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)

What kind of feedback do you need?
Please check everything and test routes locally
Checklist:

[x] I have performed a self-review of my code
[x] I have recommended where to start and how to proceed with the review
[x] I have done a git pull from DEV to ensure this PR is up to date

Tested locally and all working fine :)


NuGet.org allows all images, including badges, from the following trusted domains to be rendered (but doesn't)

this page indicates that the following domains are allowed including travis-ci, except the travis-ci domain listed on that pages is api.travis-ci.<org/com> but the images used for the build status badges are not api.travis-ci.com but simply travis-ci.com which means nuget blocks the badge on any documentation package readme's (including mine as shown below).
Image from github readme.md

image on nuget which should be pulling from github readme.md but has travis-ci build image blocked

@nullfx Thank you for providing more information. api.travis-ci.<org/com> are supported early with badge.
Example:
Thanks for pointing out that they also support travis-ci.com.  We will update to travis-ci.org/com to support more scenarios.

has this been pushed to production yet? guessing not since nuget.org still not showing the build status for travis-ci.org while trying to update documentation pointing to readme with https://travis-ci.org as the badge source for "build status" which also doesn't show up on the main website:

It would work. please let me know if you have further questions.

brilliant. it wasn't showing up earlier. that was when i went to try and update the documentation (and saw what I just posted) but its showing now. thanks.


Problem with non-English characters

Nice component, and I found it suit very well to my need. However I found a small bug with the version I installed with npm.
I get this error when input with non-english characters"
Error in ./TagInputForm class TagInputForm - inline template:2:4 caused by: Cannot read property 'focus' of undefined
And it auto-separates the word into many pieces, for example: I type "đất" and it shows 4 tags: d, đa, ât, ất
When I tried your demo on Webpackin, there is no problem.
Thank you very much!

Hello,
It appears to be OK when I remove the separatorKeys attribute from
<tag-input [ngModel]="entry.tags"
		separatorKeys="[13, 188]"
		 formControlName="tags">
</tag-input>

Is it considered a bug and need to be fixed?
Anyway, thank again!!!

Hi @manhlinhnguyen,
Hmm, I don't seem able to reproduce it! If you don't need the separatorKeys attribute, I'll close for now, but reopen if needed :)


Content behind modal is swipable, causing user confusion

When the modal is open, if you accidentally swipe to the sides the content behind the modal navigates to other screens.
Is there anyway to prevent this?

Weird, do you have any padding on the border ? Not sure to understand


Add --noImplicitOverride flag to tsc/webpack

Need to add override to those methods that do so and then add the flag that will cause an error if it's not used.  This is important in terms of directing code readers to look at the documentation for the interface.

Actually override isn't relevant for implementing interfaces.


Fix unsubscribe button

Episodes are coming back undefined when we try to call destroyRecord on the podcast https://github.com/mozilla/high-fidelity/blob/ember-cli-conversion/app/models/podcast.js#L26-L31

The fix for this is a little hacky right now (https://github.com/mozilla/high-fidelity/commit/ff40adb0e1af27dcb58d4e0e8fe3d812a57f1c64), but the issue was caused by trying to get a large amount of records at once. The response gets split up as outlined here: https://github.com/emberjs/data/issues/651
So even though the application knows there are, say, 80 episodes to delete, it only makes it through the first half before erring out. Will ask around for a more appropriate solution to this.


Changed client param

Codecov Report

Merging #88 into master will not change coverage.
The diff coverage is 0%.

@@           Coverage Diff           @@
##           master      #88   +/-   ##
=======================================
  Coverage   92.42%   92.42%
=======================================
  Files           2        2
  Lines          66       66
=======================================
  Hits           61       61
  Misses          5        5

Impacted Files
Coverage Δ

index.js
90.19% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

Coverage increased (+1.5%) to 92.424% when pulling 00e70ef4caf67d0e7195260a4eafaad91f385932 on MartinKamenov:MKamenov/fix into 777d7db94f82ec402e7758af1549818c07d55747 on matheuss:master.


phenotype section, allele filters

single allele phenotype:
Suggest:
deletion/null expression
sequence alteration
wt (overexpression)
?
multigene phenotype
multiple deletions
sequence alteration + deletion
multiple sequence alteration ?
plus a text box filter to type in any gene name (or a drop down) to filter on any genotypes with said gene.

This has the "not required to go live" label and is on the public release milestone.  Is it essential?

not essential to go live...can e one of the first tasks after the query builder.
However we do need to be able to do "allele type" int he query builder so I kept this in the milestone list initially. Can remove, and can be done early on after public release.

We want this, but it can be a post-release enhancement...

I think it would be nice if, when we make these changes, we change the query builder to match (as much as possible).
So if we can filter by allele type in the annotation tables so should be able to do the same thing in a phenotype query.
This isn't essential but I like consistency.


Opening a native window fails due to missing method

When opening a native window an error is thrown because ffiLibraryName is not present in the class GlutinGL.

The method is added to the class using a trait. Seems to be caused by pharo-project/pharo#3158

So, do you compile the method explicitly in the meantime?

yes, I compiled it explicitly and pushed


Refactor BlockDAG to track two tips - the headers-only tip (light sync) and the block bodies tip (full sync)

Mining flowchart:

State sync - overview of downloading windows of the block DAG

State sync - overview of the two different ingestion pipelines for headers and block bodies, and how it interacts with the state machine

Block DAG new design:


create a template for RAG Dev Bootcamp 2024

Note to self:
Seems like after we merge this, I will need to do this step:

Update the product repo backend/workspace-templates.yaml to point to the new template added here after being merged.

would have been nice if pipeline handled it for me automatically ...

I will follow up to fix the CI and prevent merging if it is busted

fix lint in master. please rebase

fix lint in master. please rebase

@gvspraveen is this blocking merge from your perspective?

fix lint in master. please rebase

@gvspraveen is this blocking merge from your perspective?

Well nevermind - I just rebased


Add highlighting for set_from_resource

In the release of i3 v4.13, a new command set_from_resource was added that allows i3 variables to be set from values in the X resource database.

Hello Mattmahn,
I won't be able to add this feature in the coming months. In the meantime, you can always make a PR.
Cheers


'Embedded' versions of all container classes

There should be an embedded version for each dynamic container class that never does heap allocation, but instead has the maximum capacity as a template argument similar to the StaticArray class. This would be especially be useful for small arrays that should still behave like a dynamic container.

thought: to prevent template-bloat the actual storage should be kept in a separate ArrayBuffer<TYPE,SIZE> class, and all container classes should accept such a ArrayBuffer class in their constructor:
class MyClass {
...
    ArrayBuffer<MyItem,64> itemBuffer;
    Array<MyItem> items = Array<MyItem>(itemBuffer);
...
hmm this looks mighty ugly...
};


Link to comparison article

I believe this page would be improved by linking to Hybrid Identity directory integration tools comparison when referring to ongoing Microsoft support for MIM Synchronisation.
[Enter feedback here]

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 720992cd-e2bc-d993-ef47-70205afe92ee
Version Independent ID: ea9a8093-2861-0dc3-0c78-f11b1c944579
Content: Support update for Azure AD Premium customers using Microsoft Identity Manager
Content Source: MIMDocs/support-update-for-azure-active-directory-premium-customers.md
Product: microsoft-identity-manager
GitHub Login: @EugeneSergeev
Microsoft Alias: esergeev

The following issue is being closed due to the age. If this is still an issue, please re-submit this. Thank you.
#please-close


Large funded-accounts.json causes invalid chain spec

When using a funded-accounts.json file with 100k accounts, the following error can be observed.
Not sure what is going on. I personally know that JS has a string length limit of 512MiB above the JS tooling becomes worthless, but we are currently far from that limit.
Error: Invalid chain spec raw file generated.
    at /snapshot/zombienet/dist/providers/k8s/chain-spec.js:102:19
    at Generator.next (<anonymous>)
    at fulfilled (/snapshot/zombienet/dist/providers/k8s/chain-spec.js:5:58)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)

Hi @bernardoaraujor this is still needed for k8s provider right? I can check if needed.
Thanks!

hey @pepoviola thanks for checking in on this.
I did some tests zombienet v1.3.52 and it seems that there were some improvements.
I was able to successfully load up to 100k prefunded accounts, which was bigger than the 72k from my last attempt last year.
I'm not sure what the exact limit is, but it failed againafter trying 300k.
Anyways, I'd say this is not really high priority right now, since the current work of @bredamatt with Versi uses a different method for loading prefunded accounts.
I'll close the issue for now, and re-open in the future if we need to improve this again.

Great, thanks @bernardoaraujor