computedFrom doesn't observe lists

In other words.
@computedFrom('filter', 'list')
get filteredList() {
  // filter the list
}

list.splice(index, 1); // filteredList doesn't recompute

there are any workaround this?

one workaround would be to write this.list = this.list.slice(0).splice(index, 1) instead of this.list.splice(index, 1),

@andrevlins So this is actually a bad design practice. Observing lists via splice is working, and modifying a list with another variable through value converters is working. Thus if you want a computed list you should bind the list and add a value convert to filter it.
However, I think we're agreed that this should eventually work, too, @jdanyow?

@davismj I agree, the best way to do this would be with a value converter.
There are other enhancement requests for @computedFrom so I think we should leave this open and revisit when we refactor that component.

related to #149

Using computed from with checked.bind to a list also doest not work
app.ts
import { computedFrom } from 'aurelia-framework';

export class App {
	selectedOptions = [];
	options = [1, 2, 3, 4];

	@computedFrom("selectedOptions")
	get selectedItemsSeparetedWithComma(): string {
		return `${this.primeiroNome} ${this.sobreNome}`;
	}
}

app.html
<template>

      <input type="checkbox" model.bind="item" checked.bind="$parent.selectedOptions" >${item}

${selectedItemsSeparetedWithComma}

</template>

selectedItemsSeparetedWithComma is never fired when selectedOptions change.

Using @computedFrom with any array mutation will not work.

@davismj how can I make it work?

I recommend creating a value converter. Try looking at this article by Jeremy Danyow  for more information on value converters.

@davismj thanks!

@jdanyow, any progress with @computedFrom('someArray') and array mutations? I stumbled upon issue with someArray.push(someElem).

@jdanyow, I might be totally wrong as I'm not familiar with binging logic, but could my gut feeling be correct, that this problem might be related to this line that uses ExpressionObserver, but not CollectionObserver

@atsu85 - the binding system observes property changes- if you put <input value.bind="foo.bar.baz"> in an html template or in @computedFrom('foo.bar.baz'), the binding system will observe the foo property for changes, foo's bar property for changes and bar's baz property for changes. It doesn't check "is the value an array? yes? also observe the array for mutation".
Perhaps we could do that in the @computedFrom use-case or maybe we need an explicit instruction to do so- something like: @computedFrom('foo.bar.baz', { collection: true, expression: 'cart.items' })

@jdanyow I'm for an explicit option. We should be able to differentiate between property change and changes to the value.

@jdanyow,

Perhaps we could do that in the @computedFrom use-case or maybe we need an explicit instruction to do so- something like: @computedFrom('foo.bar.baz', { collection: true, expression: 'cart.items' })

I understand, that if I'd need to use @computedFrom("bindableArray.bar.baz") instead of @computedFrom("bindableArray"), then there might be a need to explicitly instruct that CollectionObserver should be used, but when one of the computed properties is actually a array, then I'd expect that computedFrom decorator detects that bindableArray is array and it then additionally uses CollectionObserver, so it could also detect array modifications (such as .push(newElem)) in addition to replacing array instance (that is implemented right now).
Here is an example of my use-case:
<template>
  ${computedGetter.value}
<template>

with
  @computedFrom("bindableArray", "bindableId")
  get computedGetter() {
    // return item from `this.bindableArray` by `this.bindableId`
  }

@atsu85 As mentioned above, I haven't seen a case where this couldn't be solved with a value converter, which is much more efficient. So instead you might have:
<template>
  ${bindableArray | filter: bindableId}
</template>

export class FilterValueConverter {
  toView(array, id) {
    // return item from `array` by `id`
  }
}

I've even written a small library that does this for you here: http://foursails.github.io/bouncer/

@jdanyow As the person who originally opened this issue, I recommend we close it, as I don't see this as a valuable enhancement. What do you think?

@davismj, my example was a lot simplified from real use-case. I also need the result of computedGetter in other methods of the class.

Is it intended that the value converter (jsonValueConverter) in this gist does not update when I change the array?
https://gist.run/?id=82c48de63159f1b1d39c2658ab748bf7

Even if the official solution is "use a value converter", an error message would be much more valuable than the current approach of "try it in futility for an hour, google it and find this issue." Can @computedFrom do a typeof check to see if the observed property is an array, and throw an error if so?

Value converters don't work at all for this use case.  I don't understand why it was so strongly recommended nor why those who should be more in the know didn't object when it was.

Observing array length with @computedFrom("array.length") works, is it officially supported?

Yep! That is a completely viable strategy. See https://github.com/aurelia/binding/blob/master/src/observer-locator.js#L159


Switch-Case for HasRole

added HasRoleSwitchDirective

This is not equal to HasRoleDirective since one might distinguish between users with multiple roles. Let's assume user A has roles "ADMIN" and "USER" and user B has only role "USER". With the previous directives you can either display something for A (*hasRole="['ADMIN']") or for A and B (*hasRole="['USER']"). If an user interface is required that supposed to display text for either A or B, it's simply not possible. With this directive, there is an implicit order of preferred elements to be displayed (switch-case-break).
Example 1

You are Admin!

You are User!

You are Unexpected!

You are Unapproved!

User A will see: "You are Admin!"
User B will see: "You are User!"
Example 2

You are User!

You are Admin!

You are Unexpected!

You are Unapproved!

User A will see: "You are User!"
User B will see: "You are User!"
=> The order is important.

Depends on #11

Oh I cant wait to merge this in, superb addition!


Make some changes in links

I don't really find this necessary.

This will look great.
On Mon, Jul 12, 2021, 4:12 AM Two @.***> wrote:

I don't really find this necessary.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/real2two/dashactyl/pull/24#issuecomment-877871874,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASM6DFP4T7CV4QEVNZDPBLLTXIM6LANCNFSM47ZFQHVQ
.


UAV policy enhancement: Add "last attached to" and "tags" to the UAV policy

Policy Template Name
The request came from Jim @ CIBC who is interested in seeing the 2 fields: last attached to and tags in the UAV policy.
Issue Type
Feature Request
Use Case (Feature Request -or- New Policy Template Request Only)
[Describe the use case for this feature request or new policy template request]
Policy Template version (Bug Only)
1.4
Code Snippet (Bug Only)
[Please provide any relevant code snippets (ie. datasource,escalation,policy, RCL, etc.) as it relates to the bug]
Error Messages (Bug Only)
[Provide any error messages returned (if applicable)]

There is no way to know what a volume was last attached to.
Also the only tags that are supported for volumes (currently) are RightScale level tags.
Therefore, the "last attached to" part of the issue will not be implemented.
And, the "tags" part of the issue will only reflect RightScale tags.

Cool thanks.

closing, already updated


Implement sub function for substitutions

I need to validate lists of cell gates for flow cytometry. Most gates consist of a protein name and an expression level,  e.g. "CD4+". I will have a list of protein names, so I can use in(protein.Synonym) to match the "CD4" part. But I need to ignore the "+".
So I'm proposing a sub function, similar to Python's re.sub(pattern, match, string): sub(pattern, match, expression), e.g. sub("(\+|-)*$", "", in(protein.Synonym)).
I wrote a first pass at this, but ran into a bug with the grammar #26.
It might be better if the pattern used slashes instead of quotes, but that would require a change to the grammar. E.g. sub(/(\+|-)*$/, "", in(protein.Synonym)).

@beckyjackson suggested an alternative syntax that I like even better, but might be more of a pain to implement: sub(s/(\+|-)*$//g, in(protein.Synonym))


Serializable attribute seems optional

"An exception is thrown if you attempt to serialize but the type doesn't have the SerializableAttribute attribute"
I just created a simple example to test xml serialization and it all went well without exception being thrown, although I didn't mark my test class with Serializable attribute. It appears that it is optional, not mandatory.

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

ID: 2dd80e70-d456-580a-86c0-451f933cc066
Version Independent ID: 53894d4f-36fd-9b68-99fe-cfdbb7759beb
Content: Serialization (C#)
Content Source: docs/csharp/programming-guide/concepts/serialization/index.md
Product: dotnet-csharp
Technology: csharp-fundamentals
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

Also it appears that xml serializer ignores NonSerialized attribute and it expects XmlIgnore.

Thanks for writing these up @zerocoins
This article should be updated to improve the distinctions between XML and binary serialization.
I've added this it to our backlog to update when we next update this area.


[bugfix-feature/v1-053] Implemented toggle menu

This major update includes a few features:

Added functions for toggle menu:
Updated menu items according new design;
Rewrote style props for new toggle switcher;
Fixed minor bugs(shadows, lines etc.;
Implemented toggle logical for each block our SPA;
Removed unless functions for checking menu tabs and  replace it on 1string func
Other updates.

@Julia-Maliutsina if we open Courses List Page in figma we will see, that arrow isnt in the middle. I saw that some pages seems have middle position but page was cut, or its mistake?


sofa.verify() is space-sensitive

The verification of valid entries is sensitive for space characters, e.g.
import sofar as sf
sofa = sf.Sofa('FreeFieldHRTF')
sofa.verify()
sofa.SourcePosition_Units = 'degree,degree,metre'
sofa.verify()

This error occured for a sofa file created with the ITA-Toolbox in Matlab. Let's check wether die AES standard requires space characters after the commas, so it could be either adjusted in sofar or ITA-Toolbox.

This is funny, because the answer is, sofar and ITA are wrong. This is what AES69-2020 says on page 22:
"In case of multiple units, writing applications shall separate the units by a comma and a space [...]. Reading applications should accept unit lists separated by a comma symbol [...] or a space symbol."
So I will change the verification in sofar and ITA should also be changed...

@sikersten: can you upload the SOFA file or verify yourself that the issue is solved in https://github.com/pyfar/sofar/tree/bug/compliance_with_AES69_multi_units

It works :)

Great, I will wait a few days with publishing the path because Marco also found something and wanted to open an issue.


Error on ExtModules w/ same defname, diff. ports

Adds a high form check to ensure that external modules that have the
same defname and same parameters also have exactly the same ports.
Note that this is a simple check that the ports match exactly. If ports are reordered, then this will still fail. Also, this if the same defnames have different parameters, then this check can rightly be a false negative. The underlying Verilog may be parameterizing the width of ports based on the parameters meaning that any check here is going to be insufficient unless the Verilog is examined.
This came up as part of a discussion here: https://github.com/freechipsproject/firrtl/pull/1731#issuecomment-652771527.
Contributor Checklist

[nope] Did you add Scaladoc to every public function/method?
[x] Did you update the FIRRTL spec to include every new feature/behavior?
[x] Did you add at least one test demonstrating the PR?
[x] Did you delete any extraneous printlns/debugging code?
[x] Did you specify the type of improvement?
[x] Did you state the API impact?
[x] Did you specify the code generation impact?
[x] Did you request a desired merge strategy?
[x] Did you add text to be included in the Release Notes for this change?

Type of Improvement

new feature/API

API Impact

Circuits containing extmodules with the same defname and different ports will result in the circuit being rejected.
Backend Code Generation Impact

None.
Desired Merge Strategy

Squash: The PR will be squashed and merged (choose this if you have no preference.

Release Notes

Reviewer Checklist (only modified by reviewer)

[ ] Did you add the appropriate labels?
[ ] Did you mark the proper milestone (1.2.x, 1.3.0, 1.4.0) ?
[ ] Did you review?
[ ] Did you check whether all relevant Contributor checkboxes have been checked?
[ ] Did you mark as Please Merge?

Rebased. I can update this with @albert-magyar's request for the second check, though.

Rebased. I can update this with @albert-magyar's request for the second check, though.

Sounds good. Maybe you could add some comments as to why the port width has to match if the parameters are the same and why it is ok to diverge if this isn't the case.
Also have you thought about the edge case where one ExtModule declares a port as Reset  and the other as UInt<1> (same for Clock etc.). Should that be a warning or an error?

Sure, I can add those comments. It's not super clear what the check is as written.
Good thought on the edge cases. What do you think about the following?

Comparing AsyncReset and UInt<1> is an error
Comparing [AsyncReset, UInt<1>] and Reset is not an error
InferResets invalidates CheckHighForm because it may enable now checks of type (2) are checks of type (1)
Comparing Clock with anything other than Clock is an error
Comparing UInt<n> vs. UInt<m> for an extmodule that has no parameters is an error
Comparing UInt<n> vs. UInt<m> for an extmodule that has parameters is not an error (could be a warning, but this is a common case and we don't have a way to turn off the warning)

@ekiwi-sifive: Updated with the logic above.
Possibly worth an @albert-magyar look, too.

The checks I added conflict with the test introduced in https://github.com/freechipsproject/firrtl/pull/1087.
Specifically, the test is something like two modules with the same defname, same ports, different widths, and different types for ports (one is a UInt and one is an SInt). This PR changes CheckHighForm to reject this.
Example circuit that is now rejected:
circuit Foo:
  extmodule Bar:
    output out: UInt<1>
    defname = Bar
    parameter A = 0

  extmodule Qux :
    output out: SInt<1>
    defname = Bar
    parameter A = 1

  module Foo:
    inst bar of Bar
    inst qux of Qux

@jackkoenig: Do you have any thoughts on if these are equivalent or different?
It seems problematic to have them the same as connecting to one vs. connecting to the other will be different for a UInt in Foo vs. an SInt in Foo.

@seldridge I think your check is correct and the test from #1087 is wrong. The test only passes at the Verilog-level because we emit SInts as unsigned in Verilog. While changing that behavior would be tricky, I think it's right to error here.

@jackkoenig: Sounds good. I added a commit to the head of this patch series that updates the test.

I think this is a 1.2.x backport. @ekiwi-sifive: do you have any objection to that?

I think this is a 1.2.x backport. @ekiwi-sifive: do you have any objection to that?

I think it might break some not-strictly-correct but currently working firrtl which is why I think this might be a breaking change the should only go into 1.4. Feel free to convince me otherwise though.

I think it might break some not-strictly-correct but currently working firrtl which is why I think this might be a breaking change that should only go into 1.4. Feel free to convince me otherwise though.

I agree with @ekiwi-sifive, we could try to backport a version that only catches cases that would already be broken in Verilog but dunno if it's worth it.

It is changing the API of what was previously accepted FIRRTL and it does seem odd to break that on a minor release. 1.4 it is. Can always backport later if a good reason presents itself. Thanks for the review and feedback.


Get Exception Login in variable

How can i get exception message in login?
if i use wrong username or password and i do:
// Retrieve the profile
$profile = $pokemonGoApi->getProfile();
print_f($profile);

result is:
Fatal error: Uncaught exception 'NicklasW\PkmGoApi\Authentication\Exceptions\AuthenticationException' with message 'Your username or password is incorrect. You have 4 attempts left before you will be locked out of your account for 15 minutes.'
how to get $message = 'Your username or password is incorrect. You have 4 attempts left before you will be locked out of your account for 15 minutes.' ?
Thanks to all!

This may be a good read for you: https://secure.php.net/manual/en/language.exceptions.php
So, basically you want to try to get-the-Profile() and catch a possibly occuring AuthenticationException. If you catch one, you want to get-it's-Message():
$profile = null;
try {
  $profile = $pokemonGoApi->getProfile();
} catch (\NicklasW\PkmGoApi\Authentication\Exceptions\AuthenticationException $authEx) {
  $message = $authEx->getMessage();
}

You won't run into a fatal error when you catch the exception, so you'd probably want to terminate your application yourself (in the catch block, you only enter it in case of errors).

Thanks DrDelay! But doesn't works!
My code:
......
$config = new Config();
$config->setUser("RIGHT_USER");
$config->setPassword("WRONG_PWD");
$config->setProvider(Factory::PROVIDER_PTC);
$manager = Factory::create($config);
$application = new ApplicationKernel($manager);
$pokemonGoApi = $application->getPokemonGoApi();

$profile = null;
try {
  $profile = $pokemonGoApi->getProfile();
} catch (NicklasW\PkmGoApi\Authentication\Exceptions\AuthenticationException $authEx) {
  //$message = $authEx->getMessage();
  print_r($authEx->getMessage());
}

i get blank page!

But you get no fatal error ... so you run in the catch block (or there is no error, but the credentials look wrong :-D ) -> debug it. var_dump($authEx). \Exception::getMessage() returns a string, don't know how print_r works with strings, use echo/print instead.

i don't know why, but this 3 solutions
print($authEx);
var_dump($authEx);
echo $authEx;

(used one by one) inside catch returns blank!

I'm sorry to say it, but I shouldn't have to tell you about basic coding and debugging a PHP program. It's not part of this repository.
Empty page you say, even with var_dump. var_dump is guaranteed to output something, even if it is only NULL. So you probably have an error somewhere that prevents execution but is not rendered on the screen. Turn on all errors in your development environment, if you don't know how put this at your programs entry point:
error_reporting(E_ALL);
ini_set('display_errors', true);


Error while installing python tools on atom, windows platform

python-tools was unable to find your machine's python executable.
Please try set the path in package settings and then restart atom.
If the issue persists please post an issue on https://github.com/michaelaquilina/python-tools/issues/new
Error: spawn python ENOENT

I believe this was fixed with the latest version of python tools. Could you tell me what version you are currently using?

Hello I answer you to late but my Python installation is Private.
Thank you your Question by and i hope you find another connection for yoour Python.


[Bug] How to disable «Clipboard» pane?

When I edit my GIF, appears Clipboard pane, although I do not want it. Clipboard pane just prevents me.

What should I do to Clipboard pane does not always appear, but only when I want it: Edit → Show Clipboard. I did not find in the settings, how can I disable Clipboard pane.

Windows 10.0.14393
ScreenToGif 2.2.160907

Thanks.

That's strange. :O
This panel should only appear when clicking Ctrl + C or Cltr + X

@NickeManarin, thanks for comment! In mouse mode of QTranslate program are emulated pressing Ctrl+C.

I disable mouse mode, and the problem no longer for me.

Oh!
That explains it. Thanks for the feedback, even so I should make an option to show (or not) the panel when copy/cut.


epic: enable /audio/speech

Problem
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Success Criteria
A clear and concise description of what you want to happen.
Sub Issues
Additional context
https://platform.openai.com/docs/api-reference/audio/createSpeech

Link: https://github.com/janhq/nitro/issues/59

Done


envoy: Add configurable initial fetch timeout

Add Helm option 'envoy.initialFetchTimeoutSeconds' and Cilium CLI option 'proxy-initial-fetch-timeout' (seconds), defaulting to 30 seconds. Envoy default is 15 seconds, which may go by while we wait for the endpoints to have regenerated during restart.
This can be used to avoid Envoy warning logs when the initial fetch is expected to take longer as we wait for the endpoints to regenerate.
Fixes: #32824
Added Helm option 'envoy.initialFetchTimeoutSeconds' (default 30 seconds) to override the Envoy default (15 seconds).

/test

/test


Implement Stepper horizontal back and forth navigation buttons (just needs tests)

@andryfailli provided a patch to implement Stepper horizontal back and forth navigation buttons, it just needs tests before we can land it: https://github.com/flutter/flutter/pull/11165
See also #11133.

Looks like the PR for this issue was closed. Should we close this issue?

@sethladd The issue was opened because the PR was closed.

Steppers in general don't have a test yet ?

Closing this, as #23310 formalized customization options for stepper controls.


False error on static property

JSHint 2.13.5
esversion: 11
JSHint appears to report a false error on static properties.

Class properties must be methods. Expected '(' but instead saw '='.

Code Example: YouTubePatchCollection.user.js
Code snippet
class YTP {
  static observer = new MutationObserver(this.onNewScript);

  static _config = {};

  static isObject(item) {
      return (item && typeof item === "object" && !Array.isArray(item));
  }
...

Thanks for taking the time to write a report! The JavaScript language feature you are looking for is called "class fields," and it has not yet been implemented in JSHint. We already have an issue to track that feature at gh-3139, so I'm going to keep the discussion focused there and close this issue.


All badges in a row are animated if only one of then should be updated

Hi,
Good package, thx for that!
I have an issues with that (from your example):
      actions: <Widget>[
        _shoppingCartBadge(_counter),
        _shoppingCartBadge(_counter2),
      ],

If _counter is increased both badge are animated. My expectation is that only the first badge gets an animation. Any way to avoid that?

Hi @udiedrichsen. Thanks a lot.
Try a new version of the package (1.0.3). The bug should be fixed there.
The fix should work only for Text or Icon passed into badgeContent.
If you have better fix feel free to suggest or create PR.

Thx, it's fixed!


Using Video instead of 2DTexture

Dear Keijiro,
Thank you for sharing this <3
I want to switch the standard image background with a video but I can't seem to find the right way.
I am not the best programmer out there.
But I think I have to switch 2DTexture with the RawImage which has a render texture of a video.

Could you help me out here please.
Thanks in advance!

I'd recommend using the BodyPixSample project as a starting point rather than NNCam.
https://github.com/keijiro/BodyPixSample
It uses ShaderGraph, so you can easily modify the features.

I close this issue as answered. Please reopen it for further problems.


Error in translation

I found that in Convert to CSV / Excel doesn't match the Spanish translation with the button below

Closed by #62


[BUG] [OPENSEARCH] [AWS] - Error while deleting snapshot on S3

Describe the bug
We are saving a custom snapshot to S3 on AWS. For this purpose, we set up a custom s3 repository and create a policy that takes a daily snapshot and saves it to s3.
We don't have any problem with the save process, but the delete process always throws an error. It's a strange error because we see that the previous snapshot was being deleted, but the process fail.
The error is:
{
    "message": "[2024-06-22T23:01:04Z]: Caught exception while deleting snapshot [daily-snapshots-2024-06-15t23:00:42-vouq72uv].",
    "cause": "[2024-06-22T23:01:04Z]: groupSize must be greater than 0 but was -15"
}

We communicate with our aws support and they said that there isn't a problem with s3 configuration.
Related component
Storage:Snapshots
To Reproduce

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
We hope that the snapshots will be deleted and the process will be marked as successful.
Additional Details

Open Search Version 2.11

S3 repository

Open Search Snapshot policy

Creation/Deletion Status

I believe that error message is coming from here, which suggests there is a bug here. Even if the cause is some sort of misconfiguration, the error should be something more meaningful. @cuasso Can you get us the full stack trace from the logs to help pin this down?

[Storage Triage - attendees 1 2 3 4 5 6 7 8 9 10 ]
@cuasso : As @andrross  also said , stack trace would be helpful to proceed .


How can I store the audit data in a JSON format to allow other programs to consume it?

The library doesn't store Json by design. It would make it very hard to support custom ecto types with the rollback feature.
If you need the versions in Json format, use the history function together with a Json view and offer it as an API.


Fix support for ARM64 macOS python wheels

This PR address #1573 and contains the following changes:

Update the setup.py build script to support cross compiling on macOS, heavily based on this PR https://github.com/pybind/cmake_example/pull/48 from @henryiii
Build universal wheels, containing both arm64 and x86_64 architectures as it seems to be the recommended way, see https://cibuildwheel.readthedocs.io/en/stable/faq/#apple-silicon
Build all external dependencies instead of missing ones to be sure to compile with the proper architecture flags on macOS
Migrate to the latest cibuildwheel

For the current 2.1.1 release (and 2.1.0), the arm64 wheel is broken. Maybe one way of dealing with that without a new release would be to delete these wheels only and push the universal ones instead. PyPI doesn't allow overwriting already published wheels (even after deletion) and having both arm64 and universal wouldn't work because pippick the closest match possible, here the arm64 if you are using a M1 mac.
Also not clear why I needed 13 escape characters to correctly preserve the arm64;x86_64 architecture flags in cmake ExternalProject, it interact with cmake list separator but if someone have a better way feel free to jump in!

Also not clear why I needed 13 escape characters to correctly preserve the arm64;x86_64 architecture flags in cmake ExternalProject, it interact with cmake list separator but if someone have a better way feel free to jump in!

Below are some questions / ideas:

With this proposal, the line is now part of all the findXX() methods, did you manually check on your machine that -DCMAKE_OSX_ARCHITECTURES="arm64;x86_64" works on command line?
As the goal is to have something like "arm64;x86_64", did you try to change setup.py code to produce the right string (i.e. removing the 'string replace' line in all the findXX() methods)?
Did you try in all the findXX() methods to only add -DCMAKE_OSX_ARCHITECTURES="${CMAKE_OSX_ARCHITECTURES}" or -DCMAKE_OSX_ARCHITECTURES:string=${CMAKE_OSX_ARCHITECTURES} without the 'string replace' line?

With this proposal, the line is now part of all the findXX() methods, did you manually check on your machine that -DCMAKE_OSX_ARCHITECTURES="arm64;x86_64" works on command line?
As the goal is to have something like "arm64;x86_64", did you try to change setup.py code to produce the right string (i.e. removing the 'string replace' line in all the findXX() methods)?

Yes it works and is effectively what is currently happening in setup.py, it does call CMake from a subprocess using the variable set just like you showed unless I'm missing something. This is done here. Setup.py already use the correct string if I understand your meaning, just that because it contains a semicolon, it is interpreted as a list separater in the ExternalProject stuff, the main OCIO project is not affected.

Did you try in all the findXX() methods to only add -DCMAKE_OSX_ARCHITECTURES="${CMAKE_OSX_ARCHITECTURES}" or -DCMAKE_OSX_ARCHITECTURES:string=${CMAKE_OSX_ARCHITECTURES} without the 'string replace' line?

Not luck with that, however I found a cleaner way by doing string(REPLACE ";" "$<SEMICOLON>" ESCAPED_CMAKE_OSX_ARCHITECTURES "${CMAKE_OSX_ARCHITECTURES}"), I'll push that if that sounds good to you.

string(REPLACE ";" "$<SEMICOLON>" ESCAPED_CMAKE_OSX_ARCHITECTURES "${CMAKE_OSX_ARCHITECTURES}")

Great. That's way better to understand the change.

Trying to force push with the same commit amended to fix the CI builds always being cancelled after I manually re-launch the jobs.


change master to main references

🚀 Pull Request
Description
After the renaming of the iris/master branch to iris/main, this pull-request changes all relevant master references to be main within iris.

Consult Iris pull request check list

isort is misbehaving under nox with pip... I'll take a look at that :+1:

isort is misbehaving under nox with pip... I'll take a look at that :+1:

We've suffered due to isort's latest version changing under our feet. The intent to make pre-commit's versions king - discussed in #4181 - would avoid this happening in future. Either:

@bjlittle's preferred solution of scraping the versions from the pre-commit file before installing the lint packages
OR
The alternative that I have recently proposed for cf-units that just uses pre-commit for all linting.

@trexfeathers Awesome, thanks 👍


build: bump angular

@angular/cdk        18.0.6 -> 18.1.0
@angular/cli        18.0.7 -> 18.1.0
@angular/core       18.0.6 -> 18.1.0
@angular/material   18.0.6 -> 18.1.0
PR Checklist

[ ] Unit Tests have been added for new changes
[ ] API tests have been updated if applicable
[ ] All commented code has been removed
[ ] If you've added a dependency, you've ensured license is compatible with Apache 2.0 and clearly outlined the added dependency.

What are you changing?

Anything the reviewer should know when reviewing this PR?
If the there are associated PRs in other repositories, please link them here (i.e. open-amt-cloud-toolkit/repo#365 )

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

GitHub release
v3.10.2

Your semantic-release bot :package::rocket:


Pigs drop hogchops.

Describe the bug
Pigs drop hogchops, which I assume is because hoglins and pigs use the same loot table, which means their drops are the same.


Set only the border of the block code

Eliminating the border of the inline code can make it more fused with the normal text without being obtrusive.
Before:

After:

The screenshots look identical to me. Perhaps you can focus a pair of screenshots on a narrower border area you are referring to.
Also conflicts need to be fixed, so rebase onto master.


Analysis/Batch API Failing

in the analysis logs I am seeing:
{
  "name":"camshaft",
  "hostname":"a11804d63940",
  "pid":1,
  "level":30,
  "msg":"analysis:limits_error",
  "time":"2019-02-19T16:52:08.563Z",
  "v":0
}

saw something similar in #3

I haven't experienced this personally, but it looks like you're hitting a per user or per org limit that's set in the metadata database. The issue is closed so presumably you solved it, but if it was me I'd be looking at the rake tasks that update settings, like rake cartodb:db:set_unlimited_table_quota.
@sabman - If you get time, I'd be interested to know what your resolution turned out to be.

@nballenger thanks for the tip I think it was something like that.

@nballenger how are you going? I wanted to let you know since we have been running carto in production in a microservices environment (think docker, docker swarm and kubernetes) with backups and monitoring for our clients.
After going through the pain of setting up carto in a high performance and high availability environment we have decided to make our solution available as a service and are happy to announce the launch of http://cartohost.com
We are still in beta an sign up still requires some manual setups.
Ping me if you wanna talk more.
Shoaib Burq
CTO SpacialDB UG, Berlin
twitter.com/sabman

Hi, @sabman -- this looks really interesting. The project I'm working on (which I think you saw, maybe) is at:
https://github.com/ruralinnovation/multi-svc-cartodb
I'm consulting for a client interested in self-hosting Carto, and given the level of frustration I've seen generally with doing that, I've been working on putting together some open source ops documentation around running multi-container Carto for HA / production.
I'd love to chat some time--just sent you a message on LinkedIn with my contact info.
Nick

@sabman FYI, turns out it's not an analysis limits thing at all, not sure if you resolved it, but looks like it's that the geocoder isn't ready:
https://github.com/CartoDB/camshaft/issues/365


error

Issue Type: Bug
An update or installation of VS Live Share failed due to a corrupted download. Please uninstall and reinstall the extension to resolve. See https://aka.ms/vsls-corrupted-install for more details.
I have this error, I have tried re install the extension and don´t solve it.
Extension version: 1.0.2427
VS Code version: Code 1.47.1 (485c41f9460bdb830c4da12c102daff275415b53, 2020-07-13T22:42:53.386Z)
OS version: Windows_NT x64 6.1.7601

System Info

Item
Value

CPUs
Intel(R) Core(TM) i3-2310M CPU @ 2.10GHz (4 x 2095)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: unavailable_offprotected_video_decode: unavailable_offrasterization: unavailable_offskia_renderer: disabled_off_okvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_off_okwebgl: enabledwebgl2: unavailable_off

Load (avg)
undefined

Memory (System)
3.91GB (0.61GB free)

Process Argv

Screen Reader
no

VM
0%

Hi @cctunjanod! I'm sorry you are hitting this issue. This is a duplicate of #3124, please refer to this issue where a solution and some workarounds are discussed, I hope you are able to get the extension to work again easily.


Feat: restore the "backend: 'WebAudio'" option

Short description
Resolves #3272
I've restored the backend: 'WebAudio' option.
Implementation details
There's a new class WebAudioPlayer that can be used as a drop-in replacement for a HTMLMediaElement. It emulates an audio element via a Web Audio buffer source node.
Checklist

[x] This PR is covered by e2e tests
[x] It introduces no breaking API changes

In what sense? Nothing is changed in this regard, Web Audio has always been used for decoding. This PR is only about the playback mechanism.


Update httproute.md to add link for v1.SectionName

What type of PR is this?
/kind documentation
What this PR does / why we need it:

Rules- Define a list of rules to perform actions against matching HTTP requests. Each rule consists of matches, filters (optional), backendRefs (optional), timeouts (optional), and [name][name] (optional) fields.

In this line, the link called name is not added.
Which issue(s) this PR fixes:
Fixes #
Does this PR introduce a user-facing change?:
NONE

/ok-to-test

/approved

/approve

Thanks!
/lgtm


Feature/xrd postgres

I was thinking we could go through this pull request at 4pm cst. Works with naming conventions and provisions resources in GCP.

Just added commit of what we talked about earlier


Introduce fake_token endpoint on mock server

Depends on d7a18321d9444bc1790fd41f087acf62a226d343 which was already merged to master
Introduces new dependency to spring-xsuaa-test
Adds an endpoint to mock server to retrieve fake tokens

@nenaraab As mentioned yesterday.

@mwinklerdev


[CS2103T-F12-4] myStudents

myStudents helps tuition center admins manage details of the students, tutors and classes. It is optimized for CLI users so that frequent tasks can be done faster by typing in commands.

Codecov Report

Merging #88 (261971f) into master (77a32bf) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #88   +/-   ##
=========================================
  Coverage     72.15%   72.15%
  Complexity      399      399
=========================================
  Files            70       70
  Lines          1232     1232
  Branches        125      125
=========================================
  Hits            889      889
  Misses          311      311
  Partials         32       32

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Failed to upload the binary file (in progress 99%) on the S2I page

Describe the Bug
构建镜像器上传99%失败，就卡弹出失败，报500错误
Versions Used
KubeSphere:v3.2.0
Environment
CentOS 7.9 / 1 masters:  24cpu/128g; 2 nodes: 24cpu/128g
How To Reproduce
Steps to reproduce the behavior:

Expected behavior
没有通过ingress，直接使用的nodeport的端口访问的。

/kind support

ks-apisesrver logs

https://user-images.githubusercontent.com/47712570/171999048-f5e8e958-19e6-48d9-a3e5-a26d52c707ad.png

This feature belongs to the DevOps project. The corresponding code lines are here.
So, we need the log output of devops-apiserver instead of ks-apiserver.

------------------ 原始邮件 ------------------
发件人:                                                                                "kubesphere/ks-devops"                                                                                @.>;
发送时间: 2022年6月8日(星期三) 下午2:58
@.>;
<EMAIL_ADDRESS>主题: Re: [kubesphere/ks-devops] Failed to upload the binary file (in progress 99%) on the S2I page (Issue #648)
This feature belongs to the DevOps project. The corresponding code lines are here.
So, we need the log output of devops-apiserver instead of ks-apiserver.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

------------------ 原始邮件 ------------------
发件人:                                                                                "kubesphere/ks-devops"                                                                                @.>;
发送时间: 2022年6月8日(星期三) 下午2:58
@.>;
<EMAIL_ADDRESS>主题: Re: [kubesphere/ks-devops] Failed to upload the binary file (in progress 99%) on the S2I page (Issue #648)
This feature belongs to the DevOps project. The corresponding code lines are here.
So, we need the log output of devops-apiserver instead of ks-apiserver.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

As you can see from the log output. This problem was caused by missing bucket s2i-binaries in the minio. You could create it if you didn't find it.

Creating s2i-binaries is like ok

It looks like you have resolved this problem. I'm going to close this issue.
/close


Fixed the sample code for csvBuffered

Issue #26 was opened and closed with a fix being mentioned, but not provided.

Nice! Thanks for doing this. I'm sure the sample out there was from a previous version long ago. :+1


No button to edit project from /admin/projects/X

it's a bit frustating, to edit a project i can't simply click on 'projects' in the top bar and edit one
i need to click on build, choose a  build, click on edit job and then i will see the menu to edit other properties of a job :/

Here is button:

Interesting i don't have it...
and i'm using latest version (7.9.1)

There is a check current_user.can_manage_project?. So can you manage that project?

normally i'm "admin" in gitlab so my guess was that i could edit any project
let me check my "role" in that specific project

@vsizov, where were you looking? I don't see an attempt to display the link in 7-9-stable's /app/views/admin/projects/show.html.haml.

@andynu thanks =)

We reworked UI in 7.10 and I think this problem will not confuse you anymore. Feel free to reopen if this issue still exist for you.

no problem, thanks


Remove test certs the smart agent bundle

cc: https://github.com/signalfx/splunk-otel-collector/issues/860


test: repair the prespecialized-metadata tests on ASi

Introduce llvm_src_root and llvm_obj_root to mirror swift_src_root
and swift_obj_root.  This allows us to repair the paths used in the
tests without having to hardcode the build layout into the tests.

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

CC: @nate-chandler @drexin

@swift-ci please clean test

Neat!


Pwm API

What is missing in this API?
Can this API be implemented for different devices?

I'm wondering how useful are the get_period and set_period methods get it comes to building other abstractions, or in even in generic programming. cf. https://github.com/japaric/cortex-m-hal/issues/3#issuecomment-307512820

I could not implement either of the PWM traits for the PCA9685 driver because all the methods are infallible. I did not feel like starting another digital::v2 debacle so I've just dropped it.
About set_period/get_period I think it would be cumbersome to implement since it depends on the "real world" frequency on which everything is running, which may be simply an external input and thus subject to external change plus configurable through a prescaler or power configuration or so.
Even when solving this, the driver will need to calculate an approximation to a combination of prescaler and/or other configs and I feel this is something that the driver user would rather determine themselves so that the authoritatively provided frequency is followed exactly.
Additionally, choosing a Time type seems difficult because of precision and conversion issues.

@eldruin i believe the fallible component is resolved, do you still have concerns with the current implementation?

Indeed. I believe I could implement a split yielding PwmPins in the PCA9685 driver using the embedded-hal 1.0.0 alpha. I will have a try at this soon and see how it works. It will probably be similar to the pcf857x I/O expander driver.
Regarding set_period/get_period for the Pwm trait, my concerns about the frequency calculation still stand.

Can you describe your API concerns in a separate issue for better visibility so we can close this ticket and avoid the impression PWM still doesn't exist? 😅

Sure. I filed https://github.com/rust-embedded/embedded-hal/issues/226. This can be closed now.

Thank you!


Feat/viewUserFiles

You can now view another user's files 😳
As long as you're admin (or superAdmin) tho

Great PR! Not sure if administrators should be able to view other administrator's files though.


Container does not work with Docker for Mac

Does the container also work with Docker for Mac? The README.md states that it is only tested with Fedora so far.
When running the container, the DB1 does not start. This is the extract of my log:
[2018-01-30 15:16:13] stage3: Run service storage_volumes
[2018-01-30 15:16:13] stage3: Run service databases
Traceback (most recent call last):
  File "/usr/opt/EXASuite-6/EXAClusterOS-6.0.2/libexec/exainit.py", line 79, in <module>
    main(sys.argv)
  File "/usr/opt/EXASuite-6/EXAClusterOS-6.0.2/libexec/exainit.py", line 71, in main
    run(args[1])
  File "/usr/opt/EXASuite-6/EXAClusterOS-6.0.2/libexec/exainit.py", line 45, in run
    serv.run_result = serv.run()
  File "/usr/opt/EXASuite-6/EXAClusterOS-6.0.2/exainit/tasks/database.py", line 146, in run
    self.start(db)
  File "/usr/opt/EXASuite-6/EXAClusterOS-6.0.2/exainit/tasks/database.py", line 80, in <lambda>
    return lambda db, *a, **kw: self._wrappercall(getattr(db, name), *a, **kw)
  File "/usr/opt/EXASuite-6/EXAClusterOS-6.0.2/exainit/tasks/database.py", line 86, in _wrappercall
    raise RuntimeError("Could not call function %s(*%s, **%s): %s,%s" % (repr(func), repr(args), repr(kw), repr(ret), repr(val)))
RuntimeError: Could not call function <built-in method start of libDWAd_py.exa_db object at 0x7f0840115340>(*(), **{}): 1,'Could not start database: system does not have enough active nodes or DWAd was not able to create startup parameters for system'
[2018-01-30 15:16:14] root child 68 returned with state 1.

I also tried to build the image with the provided Dockerfile on my Mac. The image build is successful, but it does not run:
docker run --detach --privileged --stop-timeout 120 --name exasol myexasol
WARNING: IPv4 forwarding is disabled. Networking will not work.
0985f1bbc833d77803982e580ffb59c0257aa787e80545d19378a4f1da62ac66
docker: Error response from daemon: OCI runtime create failed: container_linux.go:296: starting container process caused "exec: \"/usr/opt/EXASuite-6/EXAClusterOS-6.0.6/devel/docker/exadt\": stat /usr/opt/EXASuite-6/EXAClusterOS-6.0.6/devel/docker/exadt: no such file or directory": unknown.

@TribuneX most probably your docker is set on default settings, which do not allow more then 2GB to be allocated per container.
go to docker for mac -> advanced settings -> memory -> something greater than 2 GB
then restart the docker
And voila!

@TribuneX were you building the exasol docker container from scratch using this repo and the Dockerfile?  It seems that it uses a placeholder tar.gz file in the repo which doesn't have anything in it, yet when pulling the image from Docker Hub, it is much bigger (>1GB).  I had the same error you did when trying to create container using the Dockerfile in this repo, but once I changed to using a pre-fab image from Docker Hub, it worked fine.  A Dockerfile with only FROM exasol/docker-db:6.2.1-d1 in it runs correctly.

Back then I used the Docker Hub image. But thanks for pointing this out.


modal-open not being removed from body after closing modal

components: Modal
reactstrap version #6.5.0
import method es
react version #6.6.3
bootstrap version #4.1.3

What is happening?
The modal-open CSS class is not being removed from <body>when I close a modal. Clicking on the header's "x" button (or any other means of closing the modal) causes the page to be unscrollable.
What should be happening?
The modal-open style should be removed from the page when all modals are closed.
Investigation
Tracing through the Modal code in Chrome devtools, what I saw was two Modal instances being constructed but only one Modal instance is actually being used. The first Modal instance is being constructed by React but is never used after the constructor runs.  It's being thrown away.  None of its lifecycle methods are run. The second Modal instance is being used normally and its lifecycle methods fire, but when it closes, Modal.openCount is still nonzero so the modal-open style is not removed from the <body>.
Looking at Modal.js code, I saw a problem which explained the behavior above. The Modal constructor contains a side effect: a call to init() which sets modal-open on the <body>.  The constructor (and hence init()) is being called twice, but destroy() is only being called once.
I was able to resolve the problem by forking reactstrap and moving the call to init() from the end of the constructor to the beginning of componentDidMount:
class Modal extends React.Component {
  constructor(props) {
    super(props);

    this._element = null;
    this._originalBodyPadding = null;
    this.getFocusableChildren = this.getFocusableChildren.bind(this);
    this.handleBackdropClick = this.handleBackdropClick.bind(this);
    this.handleBackdropMouseDown = this.handleBackdropMouseDown.bind(this);
    this.handleEscape = this.handleEscape.bind(this);
    this.handleTab = this.handleTab.bind(this);
    this.onOpened = this.onOpened.bind(this);
    this.onClosed = this.onClosed.bind(this);

    this.state = {
      isOpen: props.isOpen,
    };

    // removed
    // if (this.props.isOpen) {
    //   this.init();
    // }
  }

  componentDidMount() {
    // added
    if (this.props.isOpen) {
      this.init();
    }

    if (this.props.onEnter) {
      this.props.onEnter();
    }

    if (this.state.isOpen && this.props.autoFocus) {
      this.setFocus();
    }

    this._isMounted = true;
  }

React advises against side effects in constructors. I suspect the behavior above may be one reason for this recommendation.  Apparently, React can construct instances and throw them away without running any lifecycle methods!
I'm happy to prepare a PR to move Modal initialization into componentDidMount().  But there may be other reasons that I'm not aware of to use this non-recommended side effect in the constructor.
Does anyone know the history of why init() is being called in the constructor instead of in componentDidMount where side effects and DOM manipulations are recommended?
Steps to reproduce issue
Repro steps are difficult at the moment because the problem shows up in the middle of a fairly complicated project. To display a modal, my app pushes data into an array that's stored in React Context (the new 16.x kind, not the deprecated old context). Another part of the app will notice the change to Context and will transform each element in the array into one or more modals.  When a modal is closed, the onClosed event handler will remove the data from the array in Context which will cause the modal to be unmounted.
I can pull this logic out of my app to provide a simpler repro, but doing this will take some time so (per discussion above) I wanted to understand more about the reasons for initializing in the constructor before going through the trouble of creating a repro. Hope this is OK.
Error message in console
(none-- it's UI behavior, not an error, that's the problem)

Code
See above-- this behavior is deeply nested inside my app, so it's non-trivial to pull it out. I can do it if needed, though.

+1, I just ran into this today.  Any update?

@nibblesnbits,
Yes I came across this issue.
Had to downgrade.  The issue is that init() is being called in the life cycle event while the modal is taking itself down.
It might be that its async with the animation so maybe try turning the fade off. Or in your close function just change Modal.openCount = 0; yourself.
Basically the modal count is always 1.

I see.  Maybe I'll have time to do a PR this weekend for that.  Here's hoping.

With react strict mode turned on class is almost never removed in development mode as strict mode may invoke constructor twice.

BTW, it looks like this issue should be fixed by PR #1495 which was just merged.

FYI, v8.0.1 was just released. It contains #1495 which should solve this problem. Closing.

I am using reactstrap with typescript. I am using v8.0.1, still I am facing the issue.

I checked with updated version 8.0.1. The issue still there. It is something due to very quick update in its isOpen prop. I am using Modal in my Loader component. My Loaded component show/hide quickly.

This issue is still happening for me too, when my component's constructor initialises this.state = { isOpen: true };, and this state property is passed along into <Modal isOpen={this.state.isOpen}>...
I fixed it by initialising my own component's state to being false in the constructor, then setting it to true in componentDidMount. Then it seems the Modal knows it's added the class to the body, and therefore remembers to remove it again.

I'm still having this issue on v. ^8.2.0, does somebody have a workaround? For now I'm going to remove the modal-open class by force using: document.body.classList.remove('modal-open');

8.4.1 the issue is still here. I don't know how to reproduce. It happened after I recently upgraded. The project is big and several months old.

@saysmaster I also had this problem with project ~2 years old, beginning with reactstrap v8.3.0 the classname modal-open wasn't removed after modal submit. Version 8.2.0 didn't have this problem. I found the problem and solve it, maybe it will also help to you.
After hours of debugging and reviewing the reactstrap code and changes introduced between 8.2.0 and 8.4.1 I found the cause - it's because of changes introduced with https://github.com/reactstrap/reactstrap/commit/310b061. However, from my understanding, it's not a bug in the reactstrap but it's the consequence of wrong usage of the Modal component.
Description of what was hapenning in my code:

Situation: There's a component with Modal, this Modal has inside other component with some form. When the form is submitted, onSubmit in the form component calls two functions from the parent:

modal close (ie. setModal that sets state of isOpen in parent to false),
refresh of some ancestor component (that has this modal component as children).

Thanks to setModal, we set the prop isOpen to false on Modal component, but in the meantime the refresh from the parent component causes the Modal to unmount from DOM. Then the Modal is not able to successfully end the operation for close (reactstrap Modal function onClosed), instead of this the componentWillUnmount method of modal starts on unmount. This method is now not able to completely remove the modal (mainly modal-open attribute on body) because it would remove this attribute only if this.props.isOpen === true, but this.props.isOpen is set thanks to setModal to false.
In the versions <= 8.2.0 my code worked because the componentWillUnmount method was based on the state: this.state.isOpen === true in modal method componentWillUnmount was considered to be true thanks to the fact that propagation of props changes to the state (in method onClosed) was interrupted by method componentWillUnmount that saves this situation because this.state.isOpen === true is true.
Impact of this behaviour can also be observed on the modal close animation - when onClosed does not correctly end, modal is unmounted, forced to close and there is no animation on close.

TLDR description:
Modal is forced to unmount, modal-open is not removed. In previous versions the modal method componentWillUnmount corrected this behaviour (probably unintentionally). Versions >=8.3.0 introduced one important bugfix https://github.com/reactstrap/reactstrap/commit/310b061 that removed this unintentional helper that corrected the behaviour and let the bug in the code appear.
Solution:

The form in the modal had a prop with function for refresh of the ancestor components that was called after the form submit. This function caused the modal to unmount before the modal-open could be removed. This function can't be called from the children component, it should be called in the modal prop onClosed. Then, onSubmit of the inner form will only call the function that sets isOpen prop on modal to false.
Thanks to this change the modal is now able to close completely (without unmount interrupting the closing process). After that, the onClosed function can be processed, the modal can be unmounted and the refresh from some ancestor component can occur.

The only problem with that logic is that this bug existed before 310b061 was created so it's not possible for 310b061 to introduce it. I believe there are multiple factors in play. Both cases are race conditions which makes them difficult to reproduce and difficult to determine there was more than one scenario.
The difference between state and props is that state is async, leading to issues when the modal unmounts before the state has been updated to reflect the value of the prop which is what 310b061 aimed to fix. With props, the cleanup code run synchronously which means the cleanup happens before other code runs which would unmount the component.
But based on your finding, I believe the issue lies with how onClosed is supposed to be trigger when props.isOpen is false. In that case, CSS transition group is set to trigger it on onExited, but it doesn't seem to be happening when CSS transition group is unmounted.
Currently, here are the possibilities during unmount:

STATE
PROP
Reason
Has body class
close Triggered?

false
false
not open, not opening
No
No

true
false
open, but closing next render
Yes
No

false
true
closed, but opening next render
Yes
Yes

true
true
open (not closing)
Yes
Yes

Based on that truth table, maybe we can just check if either state.isOpen or prop.isOpen is true in order to trigger close during unmount?

@TheSharpieOne nice overview, looks good to me. The check of state.isOpen or prop.isOpen is true should solve all these problems with modal-open attribute completely, including the problem I was writing about and also won't break the fixed one (https://github.com/reactstrap/reactstrap/issues/1626) :)

Fixed my problem for moment...
import React from 'react';
import ReactDOM from 'react-dom';
import { Provider } from 'react-redux';
import { Modal } from "reactstrap";
import 'normalize.css/normalize.css';
import 'alertifyjs/build/css/alertify.min.css';
import 'alertifyjs/build/css/themes/bootstrap.min.css';

import store from '@/redux/store';
import fetchInitialData from '@/redux/fetchInitialData';
import Root from './screens/Root';
import '@/components/PdfViewer/pdfJsPageChange';
/** import  Playground from '@/playground'; */

// store.subscribe(() => console.log(store.getState()));
fetchInitialData();

/** Hotfix css-modal */
Modal.prototype.componentWillUnmount = function() {
  if (this.props.onExit) {
    this.props.onExit();
  }

  if (this._element) {
    this.destroy();
    if (this.props.isOpen || this.state.isOpen) {
      this.close();
    }
  }

  this._isMounted = false;
};
/** Hotfix css-modal */

ReactDOM.render(
  <Provider store={store}>
    <Root />
  </Provider>
  , document.getElementById('ROOT')
);

This was still an issue for me when I implemented a loading modal in a Layout component shared across all pages. My Modal's isOpen prop was set to read an isLoading prop stored in Redux state. modal-open was not being removed from body, even on reactstrap 8.4.1.
Solution: I added an effect in my modal to forcefully remove the modal-open class on body.
const LoadingModal = (props) => {
  const { isLoading } = props;

  useEffect(() => {
    if (!isLoading) {
      document.querySelector('body').classList.remove('modal-open');
    };
  }, [isLoading])

  return (
    <Modal isOpen={isLoading}>
        Loading....
    </Modal>
  );
}

@AKhoo the fix is merged in master but not released yet
@TheSharpieOne is there any ETA for the release?

Is there any ETA for the release? I expected a patch release already since the last release was months ago...

I was able to resolve this issue with the following CSS:
body:not(.modal-open) {
  overflow-y: scroll !important;
}

My issue was the class being removed but the CSS somehow staying. So, when the modal is not open I'm forcing it to allow scrolling, otherwise the modal's css will apply.


Can't create breakpoint with vscode version 1.52

VSCode Version:
OS Version:

Steps to Reproduce:

Use Native debug  to debug cpp with vscode version 1.52, can't create breakpoint on the editor side bar.
Change to vscode version 1.51, it works and the breakpoint works perfectly.

OK, it's because the extension doesn't give the obvious language supported by this extension like contributes.breakpoints in package.json. Modify the extension and show the language.identifiers. Or you can enable the breakpoint for all files like https://github.com/microsoft/vscode/issues/112288#issuecomment-743456329

This issue has been fixed in the latest VS Code stable (1.52.2). There is no need to patch the package.json.


Fix: Fix image loading on node style properties

It fixes the following issue:
orb.data.setDefaultStyle({
  getNodeStyle(node) {
    return {
      imageUrl: "https://image.com/image.png",
    };
  },
});

orb.data.setup({ nodes, edges });

orb.view.render();

const node = orb.data.getNodesById(0);
node.style.imageUrl = "https://image.com/new-image.png"

Orb knows about default style and it will check each default style when applied to new nodes. This means it will check for values in imageUrl and imageUrlSelected in order to get those images in the browser cache. The problem is that the line where node.style.imageUrl is set, Orb doesn't know about it, so the image is never loaded in the browser cache, not it will be rendered.
So a Graph that is connected to the ImageHandler service that fetches images is now using the node's callback in order to figure out if there are new image URLs that should be fetched. After each new image that is downloaded, Orb will call render on its own to rerender the image that is available in the browser cache.

Not completely sure what's going on here. From what I understand, you're adding a callback to assign the image when it's available and also perform error handling. Maybe some comment would be more helpful.
I'm approving so it's not blocked.

Sorry, I haven't added a description. You can check it now. It is actually a decent fix :D


Overloads shouldn't gain @deprecated tags of other overloads in quick info

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

VS Code Version: 1.67.2
OS Version: Windows 11 Pro 21H2, also reproducible on macOS Monterey 12.4

Steps to Reproduce:

Create a file that contains a list of exported, consecutive, overloaded functions, with a single @deprecated tag at the top of the first one.

api.d.ts
/**
 * @deprecated Use the new method instead.
 */
export function getArrayLength(arr: string[]): number;
export function getArrayLength(arr: number[]): number;
export function getArrayLength(arr: boolean[]): number;

Create a file that imports the overloaded function

test.ts
import { getArrayLength } from "./api";

getArrayLength(["1"])
getArrayLength([1])
getArrayLength([true])

Only the first function is striked-through, the other two overloaded functions aren't, even though they are all marked as @deprecated when hovering over them.

Either the bottom two overloaded functions shouldn't be marked as @deprecated when hovering over, or all three functions should be striked-through.

This is working as intended. If we made all of them deprecated, there would be no way to convey that you want someone to use a different overload instead of a different function entirely, so we had to opt for a more granular approach.

@DanielRosenwasser In that case, the hover text shouldn't show @deprecated for all of them.

Agreed, I think I missed that. Sorry about the incorrect triage.

I think we can exclude only @depracated tag, however, does it make sense to show other tags/docs for overloaded signatures from the root declaration? Maybe we should inherit docs, however, tags seem to add confusion in the following cases.
/**
 * comment
 *
 * @param {string[]} a
 */
export function a(a: string[]): number;
export function a(a: number[]): number;
export function a(a: number[], b: number[]): number;

a/*1*/([1], [1])
a/*2*/([1])

/cc @DanielRosenwasser @RyanCavanaugh @sandersn

@a-tarasyuk one of our oldest issues tracks that: #407
I don't think anybody's made a complete proposal, but I have to admit I haven't paid attention in several years.

@sandersn @DanielRosenwasser I've made a PR (#50904) which filters out the @deprecated tag from being passed on to other signatures of an overloaded function. Please let me know your feedback on it 😄


Support radix

Feature request - full radix support
Subtasks:

Convert bynary to any radix (hex, fixed-pint, floating point, double, any other?, enums?, Labels?)
Show proper format in all places (values col, bus signals, other?)
Use it in analog waveform
Add user-interface to select the radix for each lines

For analog waveforms, it would be nice to separate e.g. traces that carry voltage information from those that carry current information. These usually need to the scaled differently, and it would be nice if these "domains" could be distinguished by e.g. their background color.


PR checks for fabric8-ui running e2e smoke test

Diff in PR check looks OK. Ready to be merged. FYI @joshuawilson

what does this change?
How much time does it add to the build?

@joshuawilson after this PR is merged, when you create a PR to the repository https://github.com/fabric8-ui/fabric8-ui it will do whatever it was doing so far and additionally, it will start two new Jenkins jobs (one with e2e tests for beta features and one for released features) - similarly to what we do now for PRs to the repository saas-openshiftio.


Adds call workflow documentation.

@timvandervoornmx , thanks for the review!
I did not use the term "sub-workflow" for two reasons:

We use terms "called microflow" and "calling microflow" for the microflow editor.
"sub-microflow" appears in the UI as a term in the microflow editor, otherwise we use "called microflow" and "calling microflow".
CC: @PaulJongenMx, what terms should we use in this document?


getting cmake args

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


Dynamic GHC

This is an archlinux related issue, since the entire haskell ecosystem distributed to archlinux is in dynamic linking. Currently, tests of hie-bios, ghcide, HLS , or maybe new projects depending hie-bios will fail immediately in archlinux, because ghc sessions created by hie-bios are not in dynamic for some cases. Personally, I use three kinds of cradle: stack, cabal, and directly ghc on standalone .hs file. For stack, the environment is isolated if we don't tell stack to use system ghc, so it works well; and for cabal, we can add:
program-default-options
  ghc-options:
    -dynamic

in ~/.caban/config, to let cabal call ghc in dynamic globally; and for directly ghc, I think there is no way except passing dynflags in hie.yaml.
Haskell developers using archlinux can write hie.yaml, or use project-local cabal file to make sure that hie-bios works as expect and thier projects work fine, despite it would be tedious to write hie.yaml everywhere. However, for tests I mentioned at first, providing hie.yaml files for every test case is less reasonable, and even in distribution packaging, this is a disaster. Similar issue occurred in doctest, our workaround is to patch source code, @felixonmars :
https://github.com/archlinux/svntogit-community/blob/39a3b66baee8c233ca227a5d969b41cac112630b/trunk/PKGBUILD#L21
to let doctest work with system ghc, not only in tests, but also in runtime. This is hacky, so my opnion is that hie-bios should give a flag to control whether passing -dynamic to cabal, or dynamicTooMkDynamicDynFlags directly to ghc. Alternatively, without a flag, hie-bios just need to guarantee that if it was compiled dynamically, sessions created should be dynamic as well in default (for cabal and direct ghc).

Your unlikely to be happy using the Haskell packages from the Arch repo there are a huge number issue with Arch's approach. I personally use ghcup on Arch, but I hear nix works too. I wish they would stop shipping hackage packages.

Closed by #286 and #287 (hopefully)


pan automation

an option for panning automation for the selected bird - activated via params

cool. I think it would be good to address this by adding lfos for pan. and while we're at it for other params too.

this was added via LFOs


cell-content > "vuecal__no-event" with splits

Hi,
First of all, thank you for this wonderful component!
it must have been a huge job!
I am having difficulty changing the text of "vuecal__no-event" when the columns are slit
I put this in the component but splits disapear when I do that :(
<template v-slot:cell-content="{ cell, view, events, goNarrower }">     Aucun RDV </template>
Is this a bug or is it coming from me ?

Thank you very much for your help, I learn more every day :)
Amazing component !


[GUI-NX] v-guinx-18.01.22 -  Buttons lock up and the screen goes blank

I'm submitting a ...

[x] bug report
[ ] feature request
[ ] support request

Do you want to request a feature or report a bug?

R:. Report a bug

What is the current behavior?

R:. When switching between device and template buttons sometimes in table or map settings, buttons of the type ADVANCE, TEMPLATE AND DEVICE, as well as selecting device or template do not work, they all lock, only back button is available for user.
OBS:. PROBLEM ONLY HAPPENS IF THERE ARE DEVICE AND TEMPLATE ADDED IN THE PLATFORM.

If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem

1° Add template and device
2° Add a table or map widget
3° General screen insert name and description, click on the NEXT button
4° Device screen, alternate a few times between the Device and Template buttons until it locks and there is no more possibility of this movement.

5 Select a device or model, click on the BACK button.

NOTE: EVEN IF YOU TRY TO MOVE FORWARD AGAIN TO THE NEXT ATTRIBUTES SCREEN, THE BUTTON WILL BE LOCKED.
YOU MUST GO BACK TO THE GENERAL SCREEN AGAIN AND GO FORWARD TO THE DEVICES SCREEN, ONLY THEN THE FORWARD BUTTON WILL BE AVAILABLE.
6° Click on the BACK button, the GENERAL screen must be displayed again, with the device or template already selected, and click on the NEXT button.
7° Device screen should appear, click on the NEXT button.
8° Click on the NEXT button on the ATTRIBUTES screen
9° Click on the button NEXT on the FILTERS screen
10° Click on the NEXT button on the SUMMARY screen
11° White screen should appear

Please tell us about your environment:

Version: v-guinx-18.01.22
Environment: [docker-compose]
Operating system: [Ubuntu 16.04]

NX GUI Tested Version: Version: DEV-220816 (OK)


Error: The security token included in the request is invalid

Describe the bug
When running serverless deploy using the plugin in Linux (Ubuntu 20.04.1) I get the following error and I can't deploy to the environment:
Error:
The security token included in the request is invalid

To Reproduce
Steps to reproduce the behavior:

install serverless-better-credentials plugin using npm
add the plugin at the top of the plugins section in serverless.yaml file
run export AWS_SDK_LOAD_CONFIG=1
run aws sso login
run serverless deploy
I get the following error:

Error:
The security token included in the request is invalid

Expected behavior
Serverless App get deployed in the AWS environment.
Screenshots
 serverless deploy
Running "serverless" from node_modules

Deploying my_app to stage sandbox (us-east-2)
✔ serverless-better-credentials: credentials resolved from config ini profile: AWS_PROFILE (sandbox)
Using Python specified in "runtime": python3.8
Packaging Python WSGI handler...

✖ Stack my_app failed to deploy (9s)
Environment: linux, node 16.14.2, framework 3.20.0 (local) 3.19.0v (global), plugin 6.2.2, SDK 4.3.2
Credentials: Local, environment variables
Docs:        docs.serverless.com
Support:     forum.serverless.com
Bugs:        github.com/serverless/serverless/issues

Error:
The security token included in the request is invalid

Desktop (please complete the following information):

OS: [Ubuntu]
Version [20.04.1]
Serverless Version [3.20.0]

Additional context
Can this be an issue related to Linux OS?

Interesting -
Something which stands out to me is the line credentials resolved from config ini profile - that means that your ~/.aws/config file's section under the profile [sandbox] has a valid configuration for loading directly from the config file (i.e. a key, secret and maybe session token). Because of this, the plugin is using those credentials (which seem to be invalid) instead of using SSO.
On the assumption that you do want to use this plugin to handle your SSO, can you make sure your config only has the following sections under that profile (from https://docs.aws.amazon.com/cli/latest/userguide/cli-configure-sso.html):
[profile my-dev-profile]
sso_start_url = https://my-sso-portal.awsapps.com/start
sso_region = us-east-1
sso_account_id =<PHONE_NUMBER>11
sso_role_name = readOnly
region = us-west-2
output = json

Let me know how you get on 😄

Not sure why the plugin didn't work in my end, but I ended up using this npm package that solved my issue.

No worries - the best I can do is that it still works for me on our legacy stack.
Although- probably worth mentioning that the aws-cdk isn't as bad as it used to be, and supports SSO out of the box.


Required Status Check

What does this pull request do?
Creates the status check test / all which can be set as required status check.
This requires the matrix items to fail in case of failing tests.

Related issues

https://github.com/elastic/observability-robots/issues/1571

@basepi
I noticed that windows checks do not return a failing exit code. not sure if this is also true for the docker runs.
Can you help me accomplish this?


CudaText pkg: disable 'gtk3'

Describe the bug
CudaText pkg has support for gtk2, gtk3, qt5.
Lazarus GTK3 is NOT ready! Any big app will be broken with gtk3 (crashes, paint bugs).
https://github.com/NixOS/nixpkgs/blob/master/pkgs/applications/editors/cudatext/default.nix#L27

There is nothing to disable, as far as there is no cudatext-gtk3 package exposed in all-packages.nix:
  cudatext-qt = callPackage ../applications/editors/cudatext { widgetset = "qt5"; };
  cudatext-gtk = callPackage ../applications/editors/cudatext { widgetset = "gtk2"; };
  cudatext = cudatext-qt;

Possibility to override widgetset to gtk3 exists for those who still wants to try gtk3. But nevertheless gtk3 is not an option available from all-packages.nix.


[DOC] flagd provider docs

Change in the documentation
Create documentation that explains how a new provider can be developed. This can be referenced by new and existing flagd providers to ensure a consistent experience across technologies.
Areas to include:

Responsibilities of a flagd provider
Where to find the protobuf definitions
How to generate a client library using Buf
Caching recommendations
Recommended environments variables
How to handle failure conditions
Design considerations

I would also add a link to the general provider recommendations/FAQ: https://docs.openfeature.dev/docs/reference/concepts/provider#faq
Does Recommended environments variables mean standard ENV vars and config overriding?
Otherwise, this sounds good.


Does not work on a child portal

On any give DNN installation with more than one portal, this module will fail in any child portal.
The error is: "Message":"Specified page is not in this site."
When calling the API, the portal id needs to be sent in order for DNN to get the correct context.

Hi Vicenç - will pick up your issue upon request from Ernst Peter.


ran machine-translate

What is the purpose of the change

ClickUp Task
ClickUp Task URL
Brief Changelog

Testing and Verifying

Documentation and Release Note

Waiting on this, have some more adjustments to translations coming

I think we should just run all the re-translations at the end.


Tracability and origin

Feature request: a mechanism to document the origin of the data retrieved from the connectors.
For example this could be based on the same pattern as Entity and EntityKind, for every piece of data, associate some trace/origin/source information.

Duplicate of https://github.com/enola-dev/enola/issues/214


feat(generators): add provde in option angular client

Hello, super happy with these changes, thank you for this!
Just 1 question:
I don't really see how the removal of generateQueryFormDataFunction is related to this. Was it just a clean-up on the side of something which was not actually used?

@kfrederix it's unused code that I cleanup


Build failing for React-native 0.69.0

Hello, i am upgraded my react native and started to get this error, not sure whats the cause, thanks.
PS C:\Users\Abdullah\Desktop\Programming\company\apps\AppName\AppName-0.69.0> npm run android

><EMAIL_ADDRESS>android
> react-native run-android

info Running jetifier to migrate libraries to AndroidX. You can disable it using "--no-jetifier" flag.
Jetifier found 2265 file(s) to forward-jetify. Using 8 workers...
info JS server already running.
info Installing the app...

> Configure project :lottie-react-native
WARNING:Software Components will not be created automatically for Maven publishing from Android Gradle Plugin 8.0. To opt-in to the future behavior, set the
Gradle property android.disableAutomaticComponentCreation=true in the `gradle.properties` file or use the new publishing DSL.

> Configure project :react-native-reanimated
Native libs debug enabled: true
Android gradle plugin: 7.1.1
Gradle: 7.3.3
building Reanimated2
WARNING:Software Components will not be created automatically for Maven publishing from Android Gradle Plugin 8.0. To opt-in to the future behavior, set the
Gradle property android.disableAutomaticComponentCreation=true in the `gradle.properties` file or use the new publishing DSL.

> Configure project :shopify_react-native-skia
react-native-skia: node_modules/ found at: C:\Users\Abdullah\Desktop\Programming\company\apps\AppName\AppName-0.69.0\node_modules
react-native-skia: React Native version: 69
react-native-skia: Is Source build: false
react-native-skia: Prebuilt dir: C:\Users\Abdullah\Desktop\Programming\company\apps\AppName\AppName-0.69.0\node_modules\@shopify\react-native-skia\android\build/react-native-0*/jni

Deprecated Gradle features were used in this build, making it incompatible with Gradle 8.0.

You can use '--warning-mode all' to show the individual deprecation warnings and determine if they come from your own scripts or plugins.

See https://docs.gradle.org/7.3.3/userguide/command_line_interface.html#sec:command_line_warnings
6 actionable tasks: 6 up-to-date

FAILURE: Build failed with an exception.

* Where:
Build file<EMAIL_ADDRESS>line: 178
* What went wrong:
A problem occurred evaluating project ':shopify_react-native-skia'.
> Expected directory 'C:\Users\Abdullah\Desktop\Programming\company\apps\AppName\AppName-0.69.0\node_modules/react-native/android' to contain exactly one file, however, it contains more than one file.

* Try:
> Run with --stacktrace option to get the stack trace.
> Run with --info or --debug option to get more log output.
> Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 22s

error Failed to install the app. Make sure you have the Android development environment set up: https://reactnative.dev/docs/environment-setup.
Error: Command failed: gradlew.bat app:installDebug -PreactNativeDevServerPort=8081

FAILURE: Build failed with an exception.

* Where:
Build file<EMAIL_ADDRESS>line: 178
* What went wrong:
A problem occurred evaluating project ':shopify_react-native-skia'.
> Expected directory 'C:\Users\Abdullah\Desktop\Programming\company\apps\AppName\AppName-0.69.0\node_modules/react-native/android' to contain exactly one file, however, it contains more than one file.

* Try:
> Run with --stacktrace option to get the stack trace.
> Run with --info or --debug option to get more log output.
> Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 22s

    at makeError (C:\Users\Abdullah\Desktop\Programming\company\apps\AppName\AppName-0.69.0\node_modules\execa\index.js:174:9)
    at C:\Users\Abdullah\Desktop\Programming\company\apps\AppName\AppName-0.69.0\node_modules\execa\index.js:278:16
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async runOnAllDevices (C:\Users\Abdullah\Desktop\Programming\company\apps\AppName\AppName-0.69.0\node_modules\react-native\node_modules\@react-native-community\cli-platform-android\build\commands\runAndroid\runOnAllDevices.js:109:5)
    at async Command.handleAction (C:\Users\Abdullah\Desktop\Programming\company\apps\AppName\AppName-0.69.0\node_modules\@react-native-community\cli\build\index.js:192:9)
info Run CLI with --verbose flag for more details.

Possibly related to #624

Hey @chrfalch This is turning out to be a blocker for building it on android. Any idea, how we can resolve this issue?

@quicksilverr  replace
def rnAAR = fileTree("${nodeModules}/react-native/android").matching({ it.include "**/**/*.aar" }).singleFile
with
def buildType = "debug"     tasks.all({ task ->         if (task.name == "buildCMakeRelease") {             buildType = "release"         }     })     def rnAarMatcher = "**/react-native/**/*${buildType}.aar"     def rnAAR = fileTree("${nodeModules}/react-native/android").matching({ it.include "**/**/*.aar" }).singleFile

@hungnq139 Please could you share which file we need to do this in?

@chrfalch Yes, I also have a similar experience. It fails in CMake while compiling, I double checked, I do have CMake installed in android studio.

Hey @chrfalch I tried adding your suggestion, but this is the error that I'm facing while, running yarn android
Error: Command failed: ./gradlew app:installDebug -PreactNativeDevServerPort=8081
[CXX1300] CMake '3.10.2' was not found in SDK, PATH, or by cmake.dir property.
[CXX1301] - CMake '3.18.1' found in SDK did not satisfy requested version.
[CXX1300] CMake '3.10.2' was not found in SDK, PATH, or by cmake.dir property.
[CXX1301] - CMake '3.18.1' found in SDK did not satisfy requested version.

FAILURE: Build failed with an exception.

* What went wrong:
A problem occurred configuring project ':shopify_react-native-skia'.
> [CXX1300] CMake '3.10.2' was not found in SDK, PATH, or by cmake.dir property.

I have installed CMake from android studios -> sdk manager

Great that it works. The Android build system for building C++ code in React Native is (in my opinion) still a bit immature, like which version of CMake to use etc. We're super happy for PRs and contributions, so go ahead @quicksilverr!

@thespacemanatee - this project will show you how to set up CMake appropriately: https://github.com/brentvatne/try-skia - notice the eas-build-pre-install hook in package.json and pre-install.sh
cc @Kudo / @chrfalch - we should coordinate on making this work out of the box on eas build, this was a bit of an oversight on our part

@brentvatne Thank you for the repo. Had to make sure to chmod +x the script as well otherwise it would through permission error on EASS build servers.

@chrfalch is cmake 3.10.2 here is a hard requirement? is it possible to remove the line to make it more flexible?
supposedly AGP could download required cmake automatically. someone came across similar issues as well. not sure whether it's license issue or what.

It shouldn't be a hard requirement - I've created an issue to follow up on this one: #739


Failed to load plugin when use custom sync in ios 17

Thanks again for your hard and great work!!
Abstract
Failed to load plugin when use custom sync to sync plugin from other device in ios 17
Expected behaviour

sync plugin and config data correctly

Actually happened

The plugin I select to apply sync shows in the Community plugin setting page, But when launch it, it shows failed to load plugin.
If I download the plugin in market, the config data can be use correctly. So I think maybe the bug is because of the uncomplete sync of plugin file?

Reproducing procedure

Click sync once in Window device customize sync setting.
choose the plugin I need in Ios device cutomize sync setting. select MAIN and Data , Click apply all.
The plugin shows in Community plugin setting page, but can't be launched.

Sorry for the intrusion.
I don't use Obsidian Memos, but is there any way that Obsidian Memos files could be corrupted?  In the past, I couldn't enable plugins if the plugin configuration files were 0 bytes.
In the official version of Obsidian Memos, v1.9.7 appears to be as follows. It may be different in the unofficial version v2.0.12.

main.js 757 KB
manifest.json 267 Bytes
styles.css 277 KB

That's about it. I apologize if I missed the point.

Hello， Thanks for you reply!
In fact this pic capture is only a example, all the plugins I sync failed
to be launched in ios. But the configure data can be sync correctly
becausef when I download the plugin in community market after sync, the
configure can be same with my windows.
On Thu, Oct 19, 2023 at 9:48 PM dlphnkck @.***> wrote:

Sorry for the intrusion.
I don't use Obsidian Memos, but is there any way that Obsidian Memos files
could be corrupted? In the past, I couldn't enable plugins if the plugin
configuration files were 0 bytes.
In the official version of Obsidian Memos
https://github.com/Quorafind/Obsidian-Memos, v1.9.7 appears to be as
follows. It may be different in the unofficial version v2.0.12.

main.js 757 KB
manifest.json 267 Bytes
styles.css 277 KB

That's about it. I apologize if I missed the point.
—
Reply to this email directly, view it on GitHub
https://github.com/vrtmrz/obsidian-livesync/issues/306#issuecomment-1771029666,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFHXYTPIRTBSTASTGLIND4LYAEVUFAVCNFSM6AAAAAA6E4P456VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONZRGAZDSNRWGY
.
You are receiving this because you authored the thread.Message ID:
@.***>

Thank you for opening the issue!
Reproduced! Some files could not be saved in the storage.
I will address it later!

same problem! js and css files in plugins folders  are not synced


Corteza tour doesn't open for new user

Description
corteza.tour is always set to false which prevents the modal from opening
This breaks CY test

Cypress tests are fine. Closinig


Syft always replaces 'latest' version with digest, even when passed explicitly with --source-version

What happened:
When generating SBOMs (at least for the CycloneDX XML format) for images with the 'latest' tag, the 'version' element in the SBOM gets assigned the value of the image digest instead of the value 'latest'. This causes regressions in our automation pipelines, where the value 'latest' is expected for the image versions. This even happens when the 'latest' version is explicitly passed via the --source-version argument. (Not sure exactly when this was introduced, the issue remained unnoticed for at least a few months)
I understand that the 'latest' tag is typically volatile, given that the 'latest' tag will likely point to different images over time. However, in our process we regenerate the 'latest' SBOMs for the 'latest' images automatically, and in addition we have to cater for some images where the 'latest' tag is not properly used. Our current workaround is to set a dummy version-token as the 'source-version', and then post-process the SBOM with sed, replacing the token with the actual tag (e.g., 'latest').
What you expected to happen:
I would expect that the version element in the SBOM reflects the version/tag of the image as specified via the image argument. E.g., when I invoke 'syft scan myimage:latest', explicitly specifying 'latest', I expect that the SBOM will also include 'latest' as the version. Only when I invoke syft without specifying an image tag (e.g., 'syft scan myimage'), and syft automatically infers 'latest' internally, I think it makes sense to use the digest instead.
Also, when passing 'latest' explicitly as the 'source version', I would definitely expect that to be honored, and not getting replaced with the digest.
A command-line argument that would prevent the replacement of 'latest' by the digest could be an alternative solution.
Steps to reproduce the issue:
This can be reproduced by generating an SBOM for the 'latest' Ubuntu image:
syft -q scan -o cyclonedx ubuntu:latest > ubuntu_latest.bom.xml

Inspecting the SBOM will show the digest for the version instead of 'latest', e.g.:
    <component bom-ref="66f265fe5d67425c" type="container">
      <name>ubuntu</name>
      <version>sha256:19022ba720af5cb90755e0ed007dd9729c76db322e17dd292cc993ae0b8a020d</version>
    </component>

The same happens when passing 'latest' as version explicitly:
syft -q scan -o cyclonedx --source-name "ubuntu" --source-version "latest" ubuntu:latest > ubuntu_latest.bom.xml

Anything else we need to know?:
I believe the code that causes the replacement of 'latest' with the digest can be found here:
https://github.com/anchore/syft/blob/25f5c6729f02342c2a442b6d208e69b2a6661624/syft/source/stereoscopesource/image_source.go#L63
Subsequently being called, below, e.g.:
https://github.com/anchore/syft/blob/25f5c6729f02342c2a442b6d208e69b2a6661624/syft/source/stereoscopesource/image_source.go#L87
I.e., it treats the value of 'latest' as if the version has not been set.
Environment:

Output of syft version:
Application: syft
Version:    1.13.0
BuildDate:  2024-09-24T13:28:58Z
GitCommit:  01de99b25304ec95197c00b21d698f127b31a887
GitDescription: v1.13.0
Platform:   linux/amd64
GoVersion:  go1.22.7
Compiler:   gc

OS (e.g: cat /etc/os-release or similar):
NAME="Ubuntu"
VERSION="20.04.6 LTS (Focal Fossa)"
ID=ubuntu
ID_LIKE=debian
PRETTY_NAME="Ubuntu 20.04.6 LTS"
VERSION_ID="20.04"
HOME_URL="https://www.ubuntu.com/"
SUPPORT_URL="https://help.ubuntu.com/"
BUG_REPORT_URL="https://bugs.launchpad.net/ubuntu/"
PRIVACY_POLICY_URL="https://www.ubuntu.com/legal/terms-and-policies/privacy-policy"
VERSION_CODENAME=focal
UBUNTU_CODENAME=focal

@spiffcs - thanks for the reply! I understand your point. Respecting the source-version flag would be an adequate solution I think.


Updates splitview to conform to new teaser requirements

And adds malariaday to frontpage
Turns out we already had basically the component we needed. I massaged it to meet our requrements.

closes #649

Tested on devices

[ ] Desktop 💻
[ ] Mobile 📱

Tests

[x] All tests are running ✔
[x] Test are updated 🧪
[ ] Code Review 👩💻
[ ] QA 👌

⏲ Time spent on CR:
⏲ Time spent on QA:

@EliseAndresen You need to use preview for the frontpage to test the new teaser :) When the article is published, we can merge this.


read buffer length is wrong

reported via email.
--- copied ---
Hello,
I'm not sure if this is a bug or not, but thought I would report it as it was problematic for me.
I have been using IETF hybi-00 with iOS & Safari (iPhone, iPad, Safari browser).  Up until Chrome 14, chrome also worked.  I recompiled and deployed on different servers the hybi-08 version to talk to chrome, but it did not work for me in some instances.  I sent a lot of data through the websocket, usually bursts of 200k-250k which are read by the backend socket and then relayed to the front end client in this code:
mod_websocket.c
handler_t _handle_fdevent(server *srv, void *ctx, int revents) {
int b = 0;
handler_ctx *hctx = (handler_ctx *)ctx;
mod_websocket_frame_type_t frame_type;
char readbuf[UINT16_MAX + 1];
&n
bsp;  ssize_t siz;
What I found was that (only for hybi-08, hybi-00 did not have this problem for some reason) if the packet was full, for me this is 65536 bytes, the Chrome string object given to websocket onmessage would have null bytes in it.  This led to data corruption in the application layer.  The following change solved the problem for me:

char readbuf[UINT16_MAX + 1];

char readbuf[UINT16_MAX];

With readbuf 1 byte smaller, my problems went away.  I did not debug further to find the exact cause of this, it's possible its a problem on my client code, but I expect its a bug in mod_websocket.  Hopefully this is helpful.  I would file a bug if I verified the problem further, but since it is working I am back to other projects.

fixed.


add margin to space out nav-bar options on mobile screen

Solved this issue: https://trello.com/c/PXEgjQ2f/79-navigation-menu-on-mobile-issue
See if you agree or if a better spacing is required

There's issues with the current page yellow underlining looking bad on mobile. It's also a spacing issue. Can this be part of this PR?


ERR_CONNECTION_REFUSED

Hi,
After log in, Nginx returns this error on the https://****.com/index URL : ERR_CONNECTION_REFUSED
Do you have an idea of the problem ?
Thanks in advance.
My Nginx .conf file is:
server
{
    #listen [::]:80;
    server_name ****.com www.****.com;
    index index.php default.php index.html index.htm default.html default.htm;
    root  /usr/share/nginx/html/public/;

    #error_page   404   /404.html;

    # Deny access to PHP files in specific directory
    location ~ /(wp-content|uploads|wp-includes|images)/.*\.php$ { deny all; }

    # Don't Log the access log of robots.txt or favicon.ico
    location = /robots.txt { allow all; log_not_found off; access_log off; }
        location = /favicon.ico { allow all; log_not_found off; access_log off; }

    # security headers
    # add_header X-Frame-Options "SAMEORIGIN" always;
    # add_header X-XSS-Protection "1; mode=block" always;
    # add_header X-Content-Type-Options "nosniff" always;
    # add_header Referrer-Policy "no-referrer-when-downgrade" always;
    # add_header Content-Security-Policy "default-src 'self' http: https: data: blob: 'unsafe-inline'" always;
    # add_header Strict-Transport-Security "max-age=31536000; includeSubDomains; preload" always;

    location = / {
        rewrite ^(.*)$ /index last;
    }

    location / {
        proxy_http_version 1.1;
        proxy_set_header Connection "keep-alive";
        proxy_set_header Host $http_host;
        proxy_set_header Scheme $scheme;
        proxy_set_header X-Real-IP $remote_addr;
        proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for;
        if (!-e $request_filename) {
            proxy_pass http://<IP_ADDRESS>:9501;
        }
    }

    # assets, media
    location ~* \.(?:css(\.map)?|js(\.map)?|jpe?g|png|gif|ico|cur|heic|webp|tiff?|mp3|m4a|aac|ogg|midi?|wav|mp4|mov|webm|mpe?g|avi|ogv|flv|wmv)$ {
        expires 7d;
        access_log off;
    }

    # svg, fonts
    location ~* \.(?:svgz?|ttf|ttc|otf|eot|woff2?)$ {
        add_header Access-Control-Allow-Origin "*";
        expires 7d;
        access_log off;
    }

    # ACME-challenge
    location ~ /.well-known {
        allow all;
    }

    # . files
    location ~ /\.(?!well-known) {
        deny all;
    }

    access_log  /usr/share/nginx/html/bht.is.log;

    listen 443 ssl; # managed by Certbot
    ssl_certificate /etc/letsencrypt/live/bhtorrents.com/fullchain.pem; # managed by Certbot
    ssl_certificate_key /etc/letsencrypt/live/bhtorrents.com/privkey.pem; # managed by Certbot
    include /etc/letsencrypt/options-ssl-nginx.conf; # managed by Certbot
    ssl_dhparam /etc/letsencrypt/ssl-dhparams.pem; # managed by Certbot

}```

Did you start the ridpt application and listen at port 9501 ?
RidPT is based on swoole and is not a php-fpm application.
And I advise you to use UNIT3D (3.x) at this time. It's based on Laravel and since this pr https://github.com/HDInnovations/UNIT3D-Community-Edition/pull/1369 , UNIT3D tracker Behavior like RidPT's tracker.
More, RidPT is still on early development, It is not suitable for use in a production environment.


Add a reload command

like:/qss reload
A lib that might be of use:https://github.com/Ghost-chu/SimpleReloadLib

It's a bit hard for me to understand what exactly that library is doing... Do you just intend to reload the configuration-file? Or should the whole .jar-file be reloaded? The former would be easily doable with my config-mapper, without any additional libraries. The second option seems a bit dangerous to me, ^^".

Just thought might use it.
reload the configuration-file
To reload the entire jar file, I can use PlugMan

Exactly, I also thought about PlugMan. I'm going to implement reloading here first, and if you're happy with it, I'm going to copy the idea over to the parser.

Please check the latest release. You should now be able to reload the configuration-file at will; note that this command also affects all currently open GUIs in real time, and can be dispatched from the console or in-game.

Works fine for me.

Great! I will go about adding the same functionality to the parser too, alongside another little feature, and notify you in the corresponding issue once things are ready to be tested too.


Update GitHub Actions workflows to latest

What does this implement/fix? Explain your changes.
Changes from commits:

Update actions/checkout to v4
Always use latest theos/theos upstream
Remove TODO to use a matrix for release workflow

A matrix would be more difficult to manage because all the artifacts would need to be uploaded and then downloaded to attach to the release

Migrate to creating GitHub Releases using the official CLI

The actions/checkout action is no longer maintained
Also migrate to workflow specific-tokens instead of an access token stored in Secrets

Migrate to latest step output API
Switch test workflow to use same steps as release workflow

Using the a build workflow closer to the release workflow means we're testing the code closer what will be deployed

Does this close any currently open issues?
No
Any relevant logs, error output, etc?

Tag deploy from this PR: https://github.com/leptos-null/orion/actions/runs/9142424939
Successful test run: https://github.com/leptos-null/orion/actions/runs/9142279869
Failing test run: https://github.com/leptos-null/orion/actions/runs/9142386514

Where has this been tested?
Operating System: macOS 11
Platform: macOS
Target Platform: iOS

The CI for this pull request is failing (as intended) currently. #35 should fix these.

Wouldnt keeping the theos version fixed be better to prevent a theos update breaking orion builds?

Wouldnt keeping the theos version fixed be better to prevent a theos update breaking orion builds?

Yes, usually it's a good idea to version-pin dependencies, however theos is designed to be highly backward compatible.
This is also a theos repository.

let's merge #35 first and make CI green again, will review this after that.