(PDB-1508) Deprecate HSQLDB [WIP]

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://jenkins.puppetlabs.com/job/platform_puppetdb_intn-sys_pr/1349/

this looks fine to me aside from the logging issue

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://jenkins.puppetlabs.com/job/platform_puppetdb_intn-sys_pr/1350/

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://jenkins.puppetlabs.com/job/platform_puppetdb_intn-sys_pr/1353/

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://jenkins.puppetlabs.com/job/platform_puppetdb_intn-sys_pr/1354/

OK, once this passes tests, likely ready for another look.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://jenkins.puppetlabs.com/job/platform_puppetdb_intn-sys_pr/1355/

The semantics of this look pretty good. Some of the symbol naming was pretty confusing to me though, and took me awhile to puzzle out.

@mullr may have addressed your concerns with that push

looks good now :+1:

@rlb this still seems to log twice to the console and not at all to the log

sorry rlb, meant @rbrw

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://jenkins.puppetlabs.com/job/platform_puppetdb_intn-sys_pr/1371/

^ transient

this looks good to me pending the change to not-any?

@wkalt ok, I changed it to not-any, added a comment about why that's OK, and adjusted the message to presume less about the situation.

@rbrw this needs a rebase

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://jenkins.puppetlabs.com/job/platform_puppetdb_intn-sys_pr/1376/

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://jenkins.puppetlabs.com/job/platform_puppetdb_intn-sys_pr/1383/

@pljenkinsro retest this please

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://jenkins.puppetlabs.com/job/platform_puppetdb_intn-sys_pr/1384/


Remove rust-navigation-item style

AFAICT the only code that generated nodes with this class was removed in
9826ca0f1ed6b373ff49f182a5e44f8fe25ba89f. I noticed the issue because
/rust-logo.png shows up as one of the top requested URLs, but is a 404.

That is ... a while ago :sweat_smile: excellent catch, thanks!


Add Publisher::can_loan_msgs

When can we have this one merged?

I'll merge it now, just wanted to give others a day or so to review.

OK, many thanks :)


URL category lookup

Hi. I think the feature "URL category lookup" is not available anymore. There is no option Search under Endpoints menu.

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

ID: fbaaa90e-6975-ee29-335a-545cb6a55986
Version Independent ID: 75c93c17-4952-a13f-ed60-d2baefd0337e
Content: Web content filtering
Content Source: microsoft-365/security/defender-endpoint/web-content-filtering.md
Product: m365-security
Technology: mde
GitHub Login: @Dansimp
Microsoft Alias: dansimp

@ediemerson-br Thank you for your feedback. I was not able to locate the option as well. I have removed the section via PR#https://github.com/MicrosoftDocs/microsoft-365-docs/pull/8251
Once the author approves, the changes will be live.
Thanks
Sri

@msbemba Thank you very much for the contribution and updating the documentation with PR. @ediemerson-br Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Replace special characters with ASCII replacements. Fixes #269.

I'm not sure if I should add tests, would love to though! I just couldn't find tests that test similar functionality. Existing tests all seem to run fine.
Basically I simply ported the Laravel 5.7 Str::slug() functionality which supports replacing special characters to ASCII ones. I incorporated the changes for allowing dots.
Feedback/suggestions/anything is welcome!

@bhulsman This is great! I added some tests, and in the process, realized that in some cases, the filterInvalidCharacters method was filtering out international characters before they could be transliterated. So I removed that (which isn't necessary anyway, since it's covered by the second preg_replace in slug() anyway), and moved the ascii() call to cleanOutputPath() so that it covers all circumstances.
Take a look and let me know if you have any thoughts, then I'll merge this in!

Ah nice! I did not realize the slug method was already filtering those characters. Tests cover the special character cases from what I can think of, looking good!

@bhulsman Thanks for your contribution!


Maui SearchBar changes

Adds the new Properties
Removes NamedSize from the font modifier as it is deprecated and will be deleted in NET 7 . See https://github.com/dotnet/maui/pull/5088

I'm noticing we let people set the cursor position and selection length but they can't listen for the user changing the cursor or the selection. This is a problem since Fabulous will override whatever the user is doing on each update.
Would be nice to make those modifiers both setter and event listener like the text/textChanged modifier.
static member inline cursorPosition(position: int, onPositionChanged: int -> 'msg)


Make alle static text translatable

Hi,

i.e.  prev/next buttons text should be translatable via configuration.
"Image" i.e.

Tx :)

hi @sprungmarker, thanks for raising the ticket. Sounds like a good enhancement, though won't be high on the priority list if I can be completely honest.
For anyone interested in updating these kinds of areas, until we can push our an update our recommendation would be to grab the unminified JS, modify those words yourself directly into the JS.

thx :)


Null Pointer Exception fix

Pull Request Description
This pull request closes #1613
Acceptance Test

[x] Building the code with mvn clean install -Dintegration.tests still works.
[x] Running mvn spring-boot:run in the strongbox-web-core still starts up the application correctly.
[ ] Building the code and running the strongbox-distribution from a zip or tar.gz still works.
[ ] The tests in the strongbox-web-integration-tests still run properly.

Questions

Does this pull request break backward compatibility?

[ ] Yes
[x] No

Does this pull request require other pull requests to be merged first?

[ ] Yes, please see #...
[x] No

Does this require an update of the documentation?

[ ] Yes, please see strongbox/strongbox-docs#{PR_NUMBER}
[x] No

@anki2189 This is working for me, awesome thanks for the fix.

Thanks for fixing this and thanks for testing it! :)


Use newer const fn features

Existing methods made const fn:

~AsciiStr::len()~ (done in #95)
~AsciiStr::is_empty()~ (done in #95)
AsciiStr::as_str()
AsciiStr::as_bytes()
AsciiStr::trim()
AsciiStr::trim_left()
AsciiStr::trim_right()
AsciiChar::from_ascii_unchecked()
AsciiChar::as_printable_char()
~AsciiChar::is_digit()~ (postponed)

New methods:

AsciiStr::new([AsciiChar])
AsciiStr::from_ascii_bytes()
AsciiStr::from_ascii_str()
AsciiChar::try_new(char)

Also switch back to && and || in many methods, ~and panic!() in AsciiChar::new()~ (postponed).
This increases MSRV ~all the way to 1.57~ 1.56.

1.57 is quite recent.
Dropping the AsciiChar::new() and AsciiChar::is_digit() improvements bumps it down to 1.56, which is the 2021 edition release.

Lowered MSRV to 1.57 by replacing casts through raw pointers with transmute.

Would it be possible to cut a new release onto crates.io with this? It'd be really useful to be able to use some of these in a const context.


Add library versioning using inline namespaces

Using two versions of this library in the same project causes segfaults. This is expected, but could be avoided by using inline namespaces for library versioning.
For example Google Benchmark (from v2 branch) uses this library in version 3.0.1, my program uses version 3.3.0. This causes crashes.
A solution would be to wrap all code with version specific namespaces to disambiguate them, like this:
namespace nlohmann {
    inline namespace json_3_3_0 {
    }
}

I am not sure how exactly to approach this issue. Will this break existing code? Will two such versioned libraries be able to co-exist?

Maybe related: #813-

Usage of inline namespaces does seem to be low, but I'd risk a guess this is due to people not being familiar with them. I haven't heard of any problems with them.
Applying them would be quite simple. Just change all namespace nlohmann { to namespace nlohmann { inline namespace json_3_3_0 {.
The results would be as follows:

All symbols (as seen by the linker) will have json_3_3_0 added to them. This will guarantee no mismatch of symbols from different versions, and ability of different versions of the library to coexist.
json_3_3_0 namespace will not change anything from the user's point of view. No code changes will be necessary in clients. ADL will work as it does now. User specialized templates inside nlohmann namespace (like adl_serializer) will work without any changes.
User's will be able to target specific version of the library by explicitly importing from json_3_3_0 namespace.

Note: I do not suggest to put many versions of json library in one header. While it's possible, I don't see much of a benefit in that.

A short introduction to inline namespaces is here: https://foonathan.net/blog/2018/11/22/inline-namespaces.html

Google Benchmark library intended to (but didn't yet) rename nlohmann namespace in it's copy of json library to benchmark in order to avoid conflicts with user code. Inline namespaces will solve this problem without need of renaming anything.

I will try to prepare a PR for this.

Hello, I would like to know which symbol causes the crash in the first place, was it a subtle API breaking change? ODR violation caused by an API compatible but ABI breaking one?
I'm supporting the idea of having a single inline namespace for major versions (this is what I've always seen in libraries, e.g. ranges-v3, libcxx). I'm skeptical about having one namespace for each version (minor and patch included). But for your specific use-case, you want to have ABI-related inline namespaces as mentioned in the blog post you linked.
I would rather see something like:
namespace nlohmann {
inline namespace v3 {
inline namespace abi_v2 {
class stuff {};
}
}
}

I've never used ABI inline namespaces though, you should take it with a grain of salt

The crash was in simple json.dump(). There was no API incompatibility I believe - pure ABI break.
I propose introducing inline namespaces specifically to guard against ABI incompatibility.
As for introducing two inline namespaces (one for API changes, one for ABI changes) I'm not sure it's needed. It would be difficult to track ABI changes. Using full release version for namespace name is the simplest solution in this case.
A compromise would be to use this scheme:
namespace nlohmann {
inline namespace v3 {
inline namespace v3_3_0 {

But personally I would keep just the "v3_3_0" namespace.

One of the initial purposes of inline namespaces is to allow users to explicitly choose an old behavior when a new major version is introduced:
// imagine v4 removed the deprecated stream operations
nlohmann::v3::json j;
std::istream& s = /*...*/;

s >> j;

With your proposed change, this is not possible anymore, since there is a single namespace that is bumped on every release (major, minor and patch).
Having the two inline namespaces solves that, v3 will stop becoming inline when v4 is released, but users can still explicitly use it.
As for the always-inline one, whether it is abi_vX or the vX_Y_Z is more of a stylistic choice than something else.

That being said, I wonder why benchmark does not hide json's symbols, IIRC it's used internally so it should not be exposed to its consumers...

I'd like to avoid having an ABI inline namespace to be honest

Is this implemented? It says in the changelog but I can't find the associated PR or code.

It was not implemented. The issue is mentioned in the generated ChangeLog, because it was closed since the previous version.

I'd like to see this too.
Unless one promises to not break ABI within a major version, the namespace needs to contain the full version name, but I don't see a problem with this (except one mist must remember to always update it consistently).
Ironically, the fact that it is so easy to integrate this library into your project via copy paste, is exactly what makes it more likely that you'll end up with conflicting versions.


Tags

It would be very convenient to be able to attach tags to a benchmark. Each tag should become a column in  the report. This would allow easier sorting, filtering, charting in Excel.

Ok, it is very simple to add with help of ResultExtenders. What do you think about:

How it should look?
What should we print in the tags columns?
/cc @mattwarren

May be like this
[Benchmark(Tags = new { Kind = "Simple", Type = "int", RangeCheck = "yes"})]
public void SimpleIntWithRangeCheck()

[Benchmark(Tags = new { Kind = "Simple", Type = "long", RangeCheck = "yes"})]
public void SimpleLongWithRangeCheck()

[Benchmark(Tags = new { Kind = "Unrolling", Type = "int", RangeCheck = "no"})]
public void LoopUnrollingIntWithoutRangeCheck()

@omariom, unfortunately, we can't keep instances of reference types in attributes.

Yeah this seems like a nice use-case for ResultExtendes. Although as has been pointed out, we need to decide how the Tags can be expressed in attributes first.
But once that's done, I guess the output could be something like this:

Method
AvrTime
Error
Kind
Type
RangeCheck

SimpleIntWithRangeCheck
102.8345 ns
0.0000 ns
Simple
int
yes

SimpleLongWithRangeCheck
171.6548 ns
0.0000 ns
Simple
long
yes

LoopUnrollingIntWithoutRangeCheck
207.6533 ns
0.0000 ns
Unrolling
int
no

I can suggest to keep tags in a string which will be parsed. E.g. Tags = "Kind=Simple;Type=int;RangeCheck=yes". It looks a little ugly, but I think, there is no nice way to do it via attributes.

:+1:

@omariom, I'm writing new API for BenchmarkDotNet. Here is an implementation of the tags: https://github.com/PerfDotNet/BenchmarkDotNet/tree/f149cb14c864e1b09d2f71e3bf558f9bb59bf77b#columns What do you think? Is it ok for you?

@AndreyAkinshin
The change is big. I need to try it first.

Regardless of the change, there is a good rule I believe  every API should follow: "Simple things should be simple, complex things should be possible".

there is a good rule I believe every API should follow: "Simple things should be simple, complex things should be possible".

This got me thinking a bit and I'm worried we've lost a bit of simplicity with the code in the "refactoring" branch
Compare the old way of specifying Benchmarks to run on the CLR and Mono:
[BenchmarkTask(runtime: BenchmarkRuntime.Clr)]
[BenchmarkTask(runtime: BenchmarkRuntime.Mono)]
public class Framework_StopwatchVsDateTime

with the new way using Config classes, where you need to declare an entire class (if I'm missing something and there is already an easier way let me know):
[Config(typeof(Config))]
public class Framework_DateTime
{
    private class Config : ManualConfig
    {
        public Config()
        {
            Add(Job.Clr, Job.Mono);
            ...
        }
    }
}

Whilst I think the new way is great and gives a lot of power and flexibility in more advanced situations, I worried it makes things harder for more simple scenarios?
Can we do something to make simpler scenarios easier to write? For instance is there a way that you could still specify it all in attributes for a basic set-up like CLR and Mono or x86 and RyuJIT and then only drop down to using a Config class if you need to create more complex set-ups?

For now, you can write something like this:
[Config("jobs=clr,mono")]
public class Framework_DateTime

Looks pretty easy for me. Look at the information about the command style for configs in the refactoring README.

Looks pretty easy for me. Look at the information about the command style for configs in the re-factoring README.

Yeah your right, that's pretty easy.
Definitely a case of needing to RTFM on my part ;-)

I think that configuration with parsable strings might be just too error prone.
Maybe we could combine [Flags] enums with  and Attributes where AllowMultiple = true?
Some sample code

I think that configuration with parsable strings might be just too error prone.

Yeah that's a good point, "magic strings" are always tricky

Why not preserve (somehow) BenchmarkTask and add BenchmarkTaskConfig?

Ok, here are some points:

New "magic strings" completely match to command line arguments, so, you can copy-paste your config between the Config attribute and console. Also, it allows you to set complex config in a short and human-readable form (e.g. jobs=LegacyJitX64,RyuJitX64 columns=min,max). You always can copy a desired config from examples in Readme and improve it with a few symbols.
Old BenchmarkTask (aka Job after the refactoring) is too complex to develop and extend. For now, we have a toolchain that can't be easily added as an attribute property. I want to add a lot of new Job's properties. E.g. I want to add Runtime Settings and it will look ugly as attribute arguments because of CLR metadata capabilities. It is very hard to use predefined jobs (or other kind of BenchmarkDotNet extensions), inherit it, or use several jobs at once ([Flags] enum is not a silver bullet; e.g. you can't ask for LegacyJitX86,LegacyJitX64,RyuJITx64 at once because RyuJIT doesn't have x86-version). And you can't use static object instances, you should make constants. E.g. I want to add additional statistics columns via attributes. How I can possible do it? I should make a separated constants per statistics (min,q1, median, mean, q2, max, confidence intervals and so on) and match each of them to real code. In the refactoring branch, it is easy to define a new column (in the core or as a 3rdParty extensions) with help of an labmda expression and use it everywhere.
New object style allows you to implement any logic that set you config (e.g. Tags) in a nice way. Yes, you should write several lines instead of a single attribute. On the other hand, you can do anything you want. And it is really easy to read and write such configs.
New "command style" and "object style" are full-featured systems. By design, you can express anything in both styles. Of course, we can add a Job attribute as an alternative way to create some configs. But it will be limited, you can't express anything via attributes. Why we should do it? In a simple scenario, we can use "command style" for simple configs (like jobs=AllJits). In a complex scenario (when we want to write some additional code), we can use "object style". It is very hard to design limeted-featured additional attributes that will be really useful.

I really want to make the BenchmarkDotNet API simple. I spent a lot of time trying to create a design that would fit for all tasks. New config system is the best design that I have. I wrote benchmarks last two weeks with help of new API, and it is really simple to use it.
Anyway, I'm ready to discuss any suggestion about alternative way to set configs.

@AndreyAkinshin

Ok, here are some points:

Convinced! )
The last question. Will it be possible to do the following?
[MyBenchmarkConfig(warmCount: 4)]
class BenchmarkManualSerialization
{
}

public class MyBenchmarkConfigAttribute : ConfigAttribute, IConfig
{
    private IConfig manualConfig = new ManualConfig();

    public MyBenchmarkConfigAttribute(int processCount = 3, int warmCount = 3)
        : base(typeof(MyBenchmarkConfigAttribute))
    {
        // Somehow add processCount and warmCount to manualConfig
    }

    IEnumerable<string> IConfig.GetColumns()
    {
        return manualConfig.GetColumns();
    }

    // Implement the rest of IConfig..
}

It is an interesting idea. Now we have the following source code:
[AttributeUsage(AttributeTargets.Class)]
public class ConfigAttribute : Attribute
{
    public IConfig Config { get; }

    public ConfigAttribute(Type type)
    {
        Config = (IConfig)Activator.CreateInstance(type);
    }

    public ConfigAttribute(string command)
    {
        Config = new ConfigParser().Parse(command.Split(' '));
    }
}

I can define IConfigAttribute:
    public interface IConfigAttribute
    {
        IConfig Config { get; }
    }

    [AttributeUsage(AttributeTargets.Class)]
    public class ConfigAttribute : Attribute, IConfigAttribute
    {
        // ...

So, anyone who implements IConfigAttribute can be ConfigAttribute. @omariom, what do you think?

@AndreyAkinshin
:+1: I would just call it IConfigSource.

@omariom, done: https://github.com/PerfDotNet/BenchmarkDotNet/blob/120b7c8bb3720c3a48a26840fcec71c30ab5c7b7/BenchmarkDotNet.Samples/Intro/IntroConfigSource.cs

:+1: :+1:

Implemented in v0.9.0.


Sandeep/issu_657 resolved#657

"I addressed issue #657, which was related to the 'database/challenges/css.json' file. The issue stemmed from missing 'title' fields in some JSON data, causing errors when using 'item.title.toLowerCase()' in line return ${item.title.toLowerCase()}.includes(searchItem.toLowerCase());. To resolve this, I added the missing 'title' fields to the 'database/challenges/css.json' file. This correction has fixed the problem, and the code now functions as expected."

please visit new PR #660

please visit new PR #660


TraceX - The App must be enabled to support multiple connector endpoint for one BPN provided by the discovery

Description
At the moment TraceX only works properly if just 1 EDC is configured for a company / BPN. In real life this scenario won't work, because companies will have multiple EDCs.
Example
[
{
"bpn": "BPNL000000000CX4",
"connectorEndpoint": [
"https://EDC-Cof-Test-01.cofinity-x.com",
"https://EDC-Cof-Test-05.cofinity-x.com",
"https://controlplane.preprod.cofinity-x.com"
]
},
{
"bpn": "BPNL0000000017BL",
"connectorEndpoint": [
"https://controlplane-trace-x.preprod.cofinity-x.com"
]
}
]
Impact

Additional information

[x] I'm willing to contribute to this feature

@wiegerterik you think of data cosumption, correct?

@jzbmw & @wiegerterik
Please have a look here : https://github.com/eclipse-tractusx/item-relationship-service/issues/214
We use the same client library as irs.

@ds-mwesener
Could we please check if this is already working in the current setup.

Decision: Might be already resolved by IRS and provided IRS Client library. This will be double-checked until  21.03.2024 by Cofinity-X.

This topic should be resolved already. Please recheck and get back to us if it is still a problem. Thank you.


Transaction.prove and Transaction.send result method chaining

Hi o1js community and maintainers. I'm new to this library + zkApps in general, so please take my thoughts with a grain of salt.
I'm contemplating the following code.
let tx = await Mina.transaction(id, () => { /* ... */ })
await tx.prove()
let pending = await tx
  .sign([key])
  .send()
await pending.wait()

Could we enable a fully-chained experience?
await Mina
  .transaction(id, () => { /* ... */ })
  .prove()
  .sign([key])
  .send()
  .wait()

I suppose prove and send would need to return promise subtypes (ProveResult and SendResult respectively).
type ProveResult = Promise<(Proof<ZkappPublicInput, undefined> | undefined)[]> & {
  sign: (additionalKeys?: PrivateKey[] | undefined) => Promise<Transaction> & {
    send(): SendResult
  }
}

type SendResult = Promise<Mina.PendingTransaction> & {
  wait: (options?: {
    maxAttempts?: number | undefined;
    interval?: number | undefined;
  } | undefined) => Promise<Mina.IncludedTransaction>
}

Some follow-up thoughts + questions:

this would be non-breaking
are there reasons to keep these steps non-chainable?
a given promise subtype could contain a reference to the step/promise off of which it was chained, incase a direct reference is required

to me this seems to add complexity without a clear benefit. With prettier formatting your non-chained example even uses less lines of code than the chained one

Apologies for not clearly articulating the benefits:

1 await instead of 4 awaits.
No need to define (and therefore come up with) variable names for (A) the tx and (B) the pending counterpart. This would be especially useful in scopes that create/prove/sign/send/wait multiple txs, as devs wouldn't need to spend time considering names for variables that are not subsequently referenced.
Legibility gains

This last benefit I listed is subjective... but I do believe the chained/fluent equivalent would be––for most developers––more legible/preferable.
I can close this issue if you still disagree. Thank you for hearing me out + considering this approach.

That utility would work great! Alternatively, we could create dedicated promise subclasses, as to avoid bloating the return type (the intersection of Promise<T> with the mapped type might be a bit intimidating).
const a = Mina.transaction(id, () => { /* ... */ }) // `CreateTransactionPending`
const b = a.prove() // `ProoveTransationPending`
const c = b.sign([key]) // `SignTransactionPending`
const d = c.send() // `SendTransactionPending`
const e = d.wait() // `Promise<FinalizedTransaction>`

The downside of dedicated subclasses: it obfuscates thenability... the Pending postfix should mitigate this downside. Still a risk. The return signature you suggested is clearer, although more verbose. What's preferable?

The return signature you suggested is clearer, although more verbose

It would only be that verbose in the source code, in the compiled type it would be more similar to your suggestion and fine I think (better than a custom class)

Agreed: mapPromise also allows the proxied method tsdocs to flow through. Quick question re. implementation:
I'm looking at the overload signatures of src/lib/mina/transaction.ts#transaction––how would you recommend modifying these overloads? My sense was that we would rely on inference from mapPromise, but that doesn't seem to be an option given how it's currently set up.

I'm looking at the overload signatures of src/lib/mina/transaction.ts#transaction––how would you recommend modifying these overloads?

Maybe the return of mapPromise should be made its own type MapPromise<T, K> and that used?

Hey @harrysolovay do you want to create a PR? this looks like a great start!

The src/bindings submodule is failing to clone on my machine

It's just very big so might take a while

Just wanted to follow up. I intend to take a pass at this soon.
I have been second-guessing mapPromise. The promise subclassing approach may be more straight-forward. Currently, PendingTransaction, IncludedTransaction and RejectedTransaction––stages in the desired chaining DX––are the result of using the Pick utility type and intersecting on a common Transaction type. This + mapPromise might start to break down in some cases. For instance...

await Mina
  .transaction(id, () => { /* ... */ })
  .prove()
  .sign([...privateKeys])


Rails 5 fixes

Not necessarily specific to Rails 5, but togglv8's dependency on logger gem causes selenium to fail due to Selenium::WebDriver.logger.level not being an integer
addresses https://github.com/kanet77/togglv8/issues/25
also fixed specs to run with

Oj 3.7.12
current toggle time_entries api (which only sometimes returns 'guid')

Coverage increased (+1.1%) to 91.94% when pulling 534272a15c3c160ffec5c708f2f1b5a74e0d7a06 on RepairShopr:rails-5-fixes into ef3167688340909b913994079537cfe02edb8fa9 on kanet77:master.


Fix ERC1155 chaincode indeterminism caused by iterating maps in Go

Hi. I am the person who contributed ERC1155 in #463 . I noticed a problem with the chaincode. I fall into the trap of using map data structure in Go when developing a Fabric chaincode. The functions BatchTransferFrom and BatchTransferFromMultiRecipient in the ERC1155 chaincode sometimes give the error ProposalResponsePayloads do not match. This happens since iterating maps in Go is not deterministic. As a solution, I copied the keys of the map and sorted them and iterated over the sorted keys.
Signed-off-by: Baran Kılıç<EMAIL_ADDRESS>

@kilicbaran Thank you, but it looks like you need to rebase on latest main branch.

Nevermind, I was able to merge it!


Handle ctrl+v in textbox

Hello!
I am trying to restrict user to type some characters into textbox. I use TextInputEvent in this case and everything works fine until user use ctrl+v to paste text. Ctrl+v fires 4 events (KeyDown Ctrl, KeyDown V, KeyUpCtrl, KeyUp V), TextInputEvent is not fired, and I can't check text added into textbox and I can't prevent invalid characters from being added.
Maybe you can help me with this problem. Thanks in advance.

Our current TextBox implementation checks for keyboard mappings and triggers its internal Paste implementation which calls HandleTextInput directly.
We should probably trigger routed TextInputEvent instead.

It's possible to handle PastingFromClipboard event or disable CanPaste.

@maxkatz6 how can we disable CanPaste? The property is readonly
I also tried to remove the event handler for PastingFromClipboard event but that's not possible too
How an we prevent any copy/pasting in a textbox?

I also tried to remove the event handler for PastingFromClipboard event but that's not possible too

You don't remove even handler, but instead you handle it and set EventArgs.Handled property to true.
That's how handling event "cancellation" is normally done with routed events.
Sometimes you need to handle even with tunnel strategy (so it won't be handled by other controls first), but in this case I don't think you need.

@maxkatz6 how can we disable CanPaste? The property is readonly

You don't remove even handler, but instead you handle it and set EventArgs.Handled property to true.
That's how handling event "cancellation" is normally done with routed events.

Yesterday I tried :
`
            textbox.AddHandler(PastingFromClipboardEvent, textbox_PastingFromClipboard, RoutingStrategies.Tunnel);

            private static void textbox_PastingFromClipboard(object? sender, RoutedEventArgs e)
            {
                e.Handled = true;
            }

`
It happens that the tunnel version of PastingFromClipboardEvent is never raised, maybe it's intended?
So today I tried with the bubble version of the event and it worked fine.
And I understand the purpose of CanPaste now, thank your for your answers !


Duribility Config

Minecraft version
1.19.2
LibX version
1.19.2-4.2.8
Simply Tools version
1.19.2-1.2.0
Forge version
43.2.0
The latest.log file
https://pastebin.com/8wLrw4tQ
Issue description
Durability config file does not accurately affect tools actual durability.
I have the durability set to 10x the normal durability however the tools just end up breaking at around 2120 (Also tried at 5x and it breaks at around 900) remaining durability (iron). So even though the number is displaying a higher durability the tool itself breaks far before ever reaching 0. As such, changing the durability seems to little to nothing for the tools. I experienced this with a Diamond Hammer as well but can't remember when it broke. I also have not tried this with the excavators, only the hammers.
Steps to reproduce
1.) Change the durability
2.) Use the hammer tool
3.) Watch it break before actually hitting 0 durability.
Other information
No response

Ignore this bug, I am a dingus and forgot to properly set the server config. Closing.


✨ Add more iFrame attributes

🤖 Generated by Copilot at 0d46caf
Summary
🛠📝🖼

Add permission switches and allow attribute to IFrameTile widget. This lets the user customize the security and functionality of the embedded iframe in the dashboard.

Oh we are the coders of the IFrameTile
We make it more flexible with every compile
We add some switch properties to set the permissions
And use the allow attribute for the iframe conditions

Walkthrough

Add new properties to IFrameTile widget to configure iframe permissions (link)
Create allowedPermissions array to store the enabled permissions for the iframe (link)
Set the allow attribute of the iframe element to the joined string of allowedPermissions (link)

is this still a draft ?

Yes. Still not sure if this really works how I want it to


Support for hosted Gitlab

:clipboard: Description

It would be great if there were support for passing in credentials to access hosted Gitlab solutions. It can be something similar to how users can override Github's information for private and hosted solutions.
Something like the following:
GITLAB_URL (yml: public.services.gitlab.baseUri)
GL_TOKEN (yml: private.gl_token)
GL_CLIENT_ID (yml: private.ghl_client_id)
GL_CLIENT_SECRET (yml: private.gl_client_secret)

Hi! Is this request related to the existing GitLab badge? That badge doesn't use the API, rather it scrapes GitLab's pipeline status badge. In a typical private GitLab, how does auth work on that badge URL? Is it possible to make an authenticated request?

Is this request related to the existing GitLab badge?

Hi @rokpu-wish, could you clarify if this request is about the existing GitLab badge?

Feel free to respond at some point and we can reopen this!


adding more flexible approaches to jira start/stop/review/done & jira done improvements

This is a proposal to #32 in order to make possible use those very helpful commands and still keep it flexible.
Also #34

Erro message:

Hello?

:sob:

I'm sorry but I'm having long days at work. I'll try to review them tonight.

As you are running out of time I pushed everything so you can review it all into one PR and optimize the use of your time ;)
Also including changes mentioned in #34

I will release a new version tomorrow. I will notice you in this PR.

Thank you!

0.5.0 is live! :)

Thanks! :beer:


Update DG for cleardebts and deletedebts

Codecov Report
Base: 69.93% // Head: 69.93% // No change to project coverage :thumbsup:

Coverage data is based on head (3ad269b) compared to base (1b6fa63).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #324   +/-   ##
=========================================
  Coverage     69.93%   69.93%
  Complexity      717      717
=========================================
  Files           106      106
  Lines          2428     2428
  Branches        322      322
=========================================
  Hits           1698     1698
  Misses          609      609
  Partials        121      121

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


docs: Documentation is not responsive on mobile

Summary
While trying to readup on sailsjs docs in my free time. I tried acessing it on my device and found out it isn't responsive on mobile which makes it harder to read
Node version: N/A
Sails version (sails):  V1
ORM hook version (sails-hook-orm): N/A
Sockets hook version (sails-hook-sockets): N/A
Organics hook version (sails-hook-organics): N/A
Grunt hook version (sails-hook-grunt): N/A
Uploads hook version (sails-hook-uploads): N/A
DB adapter & version (e.g. sails-mysql@5.55.5): N/A
Skipper adapter & version (e.g. skipper-s3@5.55.5): N/A

Hey @ace-builds, this issue has been resolved. Fun fact, I documented the process in this thread on X


Add specifics of what issues may be encountered

Make the readers aware that usage of read replicas can reduce risk of issues Make the recommendation of forking and testing more than a cautionary call out

#263


Ctrl-C not working

A python program, which does not determine in the prolog part,
cannot be stopped by entering Ctrl-C.

#!/usr/bin/python
from pyswip import Prolog
prolog = Prolog()
prolog.assertz("""
f(X) :-
X #> 0,
f(X)
""")
for result in prolog.query("f(X)"):
print(result["X"])
Python 3.9.1
pyswip                        0.2.10
SWI-Prolog version 8.2.3 for x86_64-linux

Did you try:
prolog.query("f(X)", catcherrors=False)
?
On Mon, Dec 28, 2020 at 8:04 AM pampelmuse76<EMAIL_ADDRESS>wrote:

A python program, which does not determine in the prolog part,
cannot be stopped by entering Ctrl-C.
#!/usr/bin/python
from pyswip import Prolog
prolog = Prolog()
prolog.assertz("""
f(X) :-
X #> 0,
f(X)
""")
for result in prolog.query("f(X)"):
print(result["X"])
Python 3.9.1
pyswip 0.2.10
SWI-Prolog version 8.2.3 for x86_64-linux
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/yuce/pyswip/issues/115, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB3QJLFYXPP4OFDC4NYWZRDSXCUCHANCNFSM4VMGBDLQ
.

Yes, with catcherrors=False and catcherrors=True.
Neither works.

kill  is working (it determines the program)


Remove KPO and Kyverno

CAPZ uses default-apps to install the security-bundle, which causes conflicts with collection. We shouldn't push Kyverno and KPO to collection anymore.

Please wait for this PR. We can merge this after bumping default-apps-azure for MCs, which is blocked by 1.25 upgrades. This is on my list.

Yeah, I realized this is only on gaggle and that it would break 😄 Won't merge yet.

@tuladhar I couldn't follow the last changes. Did we upgrade default-apps for all CAPZ MCs to the version that has security-bundle? If so, we can merge this.

@tuladhar I couldn't follow the last changes. Did we upgrade default-apps for all CAPZ MCs to the version that has security-bundle? If so, we can merge this.

@erkanerol No, we haven't, we only upgraded till v0.6.0 with Teleport.

It seems @AverageMarcus  upgraded gaggle to fix something else. https://github.com/giantswarm/giantswarm-management-clusters/commit/f3d745eae720a3dd336fbcff3f1339b6d78594b8
Let's keep gaggle broken for a few days. I hope we will sort it out.

Sorry, did I break something?
I needed to bump the observability-bundle to fix some of the issues with the E2E and conformance tests.

@AverageMarcus It is a complex topic. Don't worry about it. The customer asked us to support 1.25. We added security-bundle to default-apps to support 1.25 for WCs and WCs are OK now. However, we couldn't finish MC issues and we were avoiding upgrading defualt-apps for MCs. There is a conflict between collection apps and default apps now.

Ah sorry :(  I needed the prom stack fixed on the MC.


Updated Nuxt and related modules.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Support for stderr redirection to stdout

rm('file').stderr('&1') should work
On that note,
rm('file') >= '&1' should also work

Fixed!


Broken build

Describe the bug
hugo is unable to build the project.
To Reproduce
hugo --minify -t hugo-theme-shell

Expected behavior
compile correctly
Logs
hugo -t hugo-theme-shell

gives this error:
ERROR render of "page" failed: "/home/roozbeh/proj/rsharifnasab.github.io/themes/hugo-theme-shell/layouts/_default/baseof.html:3:8": execute of template failed: template: _default/single.html:3:8: executing "_default/single.html" at <partial "head.html" .>: error calling partial: "/home/roozbeh/proj/rsharifnasab.github.io/themes/hugo-theme-shell/layouts/partials/head.html:6:7": execute of template failed: template: partials/head.html:6:7: executing "partials/head.html" at <partial "partials/style.html" .>: error calling partial: "/home/roozbeh/proj/rsharifnasab.github.io/themes/hugo-theme-shell/layouts/partials/style.html:32:25": execute of template failed: template: partials/style.html:32:25: executing "partials/style.html" at <$themeData.themes>: can't evaluate field themes in type []interface {}

and
hugo -D

gives this:
Start building sites …
hugo v0.122.0+extended linux/amd64 BuildDate=unknown

WARN  found no layout file for "html" for kind "home": You should create a template file which matches Hugo Layouts Lookup Rules for this combination.
WARN  found no layout file for "html" for kind "page": You should create a template file which matches Hugo Layouts Lookup Rules for this combination.
WARN  found no layout file for "html" for kind "taxonomy": You should create a template file which matches Hugo Layouts Lookup Rules for this combination.

                   | EN
-------------------+-----
  Pages            |  3
  Paginator pages  |  0
  Non-page files   |  0
  Static files     |  2
  Processed images |  0
  Aliases          |  0
  Sitemaps         |  1
  Cleaned          |  0

Total in 9 ms

Additional context
It seems that when i download the the theme, it does not contains the #55 change, and therefore it's broken.

I could not reproduce the problem in my environment.
Could you please attach the repository where the problem is occurring?

@rsharifnasab If you're using the theme as a Git submodule, maybe you need to update the submodule?

That was something on my side. Sorry for the inconvenience.


Add aria labels to GitHub profile button and link.

Current dependencies on/for this PR:

main

PR #2   👈

This comment was auto-generated by Graphite.


VSL-182:  Define styles in scss - Button, Link

Description

This pr defined and applied the default link style and the following button styles:

primary
secondary
tertiary
transparent
border
nav

two button sizes:

large (default)
small (use with .is-small class)

with & without Icon

without (default)
with (use with .with-icon class)

Demo / Test Result

A brief walkthrough
https://user-images.githubusercontent.com/7423845/189189721-1ebc891a-1617-4046-ac36-a3911f04ef1c.mov

Some of the buttons from the new features are not styled correctly, for example:
Error modal:

Success Modal for sending NFT:

Success Modal for sending Tokens:

@KsenijaZivkovic updated!


Port opening error

I am trying to connect the pi to the GT-511C3. However, I get the following error with your sample code.
init GT511-C3 at 9600 baud...
events.js:161
      throw er; // Unhandled 'error' event
      ^

Error: Port is opening
    at SerialPort.open (/home/rohin/fingerprint/node_modules/gt511c3/node_modules/serialport/lib/serialport.js:160:24)
    at _combinedTickCallback (internal/process/next_tick.js:67:7)
    at process._tickCallback (internal/process/next_tick.js:98:9)
    at Module.runMain (module.js:607:11)
    at run (bootstrap_node.js:422:7)
    at startup (bootstrap_node.js:143:9)
    at bootstrap_node.js:537:3

Note: When I try to open the same port /dev/ttyS1 in python (pyserial) or even through screen. The port opens fine.
I was unable to install serialportv2 (as per your package spec), I had installed serialport v4.0.7 instead.
Would that be causing this?
May be helpful -
node -v
v7.6.0

Any help is appreciated. Thanks!

Update: no port error when I run the following
var SerialPort = require('serialport');
var port = new SerialPort('/dev/ttyS1');

port.on('open', function() {
  port.write('main screen turn on', function(err) {
    if (err) {
      return console.log('Error on write: ', err.message);
    }
    console.log('message written'); // this log message runs => port opened w/o error

  });
});

// open errors will be emitted as an error event
port.on('error', function(err) {
  console.log('Error: ', err.message);
})


Mobile Navbar Improvement

Describe the bug
Navbar in mobile version is not well responsive we have to make it responsive has responsive
To Reproduce
Steps to reproduce the behavior:

Go to '360 screen resolution'
Click on 'Mobile menu'
Scroll down to '....'
See error

Expected behavior
https://github.com/Priyaaa1/StartConnect-Hub/assets/109230049/6ec63d98-35e0-4943-9cde-480b6bfb1430
Screenshots

Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Hi. Under GSSOC'24, I would like to work on this issue.
Please Assign me this task.


Check srcAccessMask in transitions from PRESENT_SRC_KHR

From the vkAcquireNextImageKHR section of the spec:
The vkCmdWaitEvents or vkCmdPipelineBarrier used to transition the image away from VK_IMAGE_LAYOUT_PRESENT_SRC_KHR layout must have dstStageMask and dstAccessMask parameters set based on the next use of the image. The srcAccessMask must include VK_ACCESS_MEMORY_READ_BIT to ensure that all prior reads by the presentation engine are complete before the image layout transition occurs.

Fixed

@Tony-LunarG what do you want to do about that edge case you raised in email, where the layout doesn't change?

I wasn't sure enough about the optimization to do it.  Are the src and dst access masks meaningless if the layout isn't changing?  If so, I'll add the optimization to this request

Cherry picked to master


Correction in title

[Enter feedback here]
Is the title  "Circuits SKUs from 50 Mbps to 10 Gbps" correct ?
I guess I should  say "Circuits SKUs from 1 Gbps to 100 Gbps" for the Express Direct Services.

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

ID: 84695fdf-03f1-a3d4-2bec-b17e06102119
Version Independent ID: 008b4912-3d0a-4fff-577a-d9a3e4dcc9c7
Content: About Azure ExpressRoute Direct
Content Source: articles/expressroute/expressroute-erdirect-about.md
Service: expressroute
GitHub Login: @duongau
Microsoft Alias: duau

@Anders-Marius, Thanks for reaching out. We will investigate and update this thread.

@Anders-Marius, Appreciate your patience. The doc has been edited accordingly and it would take some time for the changes to get reflect over the doc.
We are closing this issue for now. If there are further questions regarding this matter, please reply and we will gladly continue the discussion.


Acceptance criteria for sub manager

Acceptance Criteria
Need Eric's help defining:

[ ] something about ingestion/scoring program interaction?
[ ] volume for ingestion program?
[ ] hidden_directory things?

General

[ ] stdout/stderr saved

[ ] storage location is private bucket
[ ] prediction step stdout/stderr and ingestion stdout/stderr saved
[ ] scoring step stdout/stderr and ingestion stdout/stderr saved

Submission adim tab

[ ] admin tab only viewable if you are comp admin/collaborator
[ ] admin tab has submission management tab
[ ] submission table defaults to whole competition
[ ] table should be filterable

[ ] by phase
[ ] by submission status

[ ] table should be searchable

[ ] by file name
[ ] by 'owner' (person who made the submission)

[ ] the data reflected in the table should be downloadable by CSV w/ a button
[ ] the submission for a specified phase should be able to be re-run in bulk

[ ] this should return how many submission have been re-run

[ ] The following functionality should be executable per single submission:

[ ] Rerun submission
[ ] Delete submission
[ ] cancel submission

[ ] Should only be able to cancel a submission that is running (i.e. status is NOT finished or failed)

[ ] the owner, phase, and actions columns should only be viewable in the table in the admin tab

Submission modal
General modal

[ ] Able to download submission zip
[ ] able to download output from prediction step
[ ] placeholder to eventually download output from scoring step (not currently available)
[ ] placeholder to eventually download private output from scoring step (not currently available)
[ ] can view prediction step stdout/stderr and ingestion stdout/stderr
[ ] can view scoring step stdout/stderr and ingestion stdout/stderr

Admin specific modal

[ ] modal admin tab only viewable if submission row clicked from table in submission manager admin tab
[ ] can view and manually change scores

[ ] cannot change computed column scores
[ ] this works with leaderboard w/ multiple columns
[ ] this works with competition w/ multiple leaderboards

@ckcollab I've bulked out the acceptance criteria for the submission manager, but I have a section at the top of things from the SoW that don't really make sense to me, (still... even though I feel like I should by now...) so I'll need your help creating the acceptance criteria for those details.

I think hidden dir handling is fine for now, let's go forward with this


imagegen: disktutils: Fix parted/partprobe error code.

Hi.
Merge Checklist

All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)

[x] The toolchain has been rebuilt successfully (or no changes were made to it)
[x] The toolchain/worker package manifests are up-to-date
[x] Any updated packages successfully build (or no packages were changed)
[x] Package tests (%check section) have been verified with RUN_CHECK=y for existing SPEC files, or added to new SPEC files
[x] All package sources are available
[x] cgmanifest files are up-to-date and sorted (./cgmanifest.json, ./toolkit/tools/cgmanifest.json, ./toolkit/scripts/toolchain/cgmanifest.json, .github/workflows/cgmanifest.json)
[x] LICENSE-MAP files are up-to-date (./SPECS/LICENSES-AND-NOTICES/data/licenses.json, ./SPECS/LICENSES-AND-NOTICES/LICENSES-MAP.md, ./SPECS/LICENSES-AND-NOTICES/LICENSE-EXCEPTIONS.PHOTON)
[x] All source files have up-to-date hashes in the *.signatures.json files
[ ] sudo make go-tidy-all and sudo make go-test-coverage pass
[x] Documentation has been updated to match any changes to the build system
[x] Ready to merge

Summary
I tried to build CBL Mariner 2.0 ISO on top of 455142ea03c1 because I want to test something related to BTF and kernel BTF related CONFIG_* were added in 2.0.
I was successful to build the ISO but it sadly failed when trying to install it on a virtual disk within qemu.
The errors were related to parted and partprobe which return code != 0 while actually doing what they should do (you can find more information about it in the source code).
I agree this PR does not address the root cause of the problem but I had to be a pragmatic about it.
I mainly opened it to inform you about this problem and share a temporary solution to other people which could face it.
If you see any way to improve this PR, feel free to share it.
Change Log

Modify diskutils.go to check if parted and partprobe actually did what they are supposed while not relying only on their return code.

Does this affect the toolchain?
The toolchain was not modified.
Test Methodology
I tested this PR installing succesfully a CBL Mariner 2.0 with qemu and running it:

Best regards.

This bug was introduced when upgrading parted to v3.4. That upgrade has since been reverted. I believe the underlying parted issue is addressed with this modification from this thread  Testing this fix out now with parted 3.4

Hi.
Thank you for your reply.
I am not really sure the issue you linked addressed this problem since the mentioned fix seems to be about atari labels and mariner does not use such partition type.
Can you please indicate me on top of which commit should I rebase to be OK regarding this problem?
And also, where can I find the rpms.tar.gz corresponding to this commit? I would like to avoid rebuilding all the packages if possible.
Best regards and thank you in advance.

Hi.
I changed the based branch from 2.0 to main.
Sadly, I did not test this change against main, I will do it and come back later with confirmation it works or modification to ensure it works.
Best regards.

Hi.
Thank you for your reply. I am not really sure the issue you linked addressed this problem since the mentioned fix seems to be about atari labels and mariner does not use such partition type.
Can you please indicate me on top of which commit should I rebase to be OK regarding this problem? And also, where can I find the rpms.tar.gz corresponding to this commit? I would like to avoid rebuilding all the packages if possible.
Best regards and thank you in advance.

Hi! sorry for the late reply here. I believe you're right, tested out the patch and iso issue persists. I will keep looking into it. In terms of the upgrade reversion, here is the commit and here  is the link to the internal pipeline artifcats to grab the rpms.tar.gz.

Hi.
No problem, I understand.
Regarding the patch I sent, this is clearly not the best solution and I sent to help someone who would be in the same situation than me.
I think the good way would be to bump parted version (or a patch to it) to fix this strange behavior but this is quite easy to say and more complicated to actually fix it.
Anyway, I think I will wait until stable 2.0 is released to continue my tests.
During this time, if you need some help you can ping me.
Best regards.

Just closing the loop on this - @MaxBrodeurUrbas identified the root cause and applied a patch as part of a new version upgrade to parted v3.4. https://github.com/microsoft/CBL-Mariner/pull/2133

Perfect! I also did not have this bug when I tested on 2.0-stable.


Is it possible to use this with Gulp, Browsersync etc?

What are you trying to achieve with browsersync // gulp? the live-compiling or live-serving?


Breaks frontmatter in daily notes when 'day planner heading' is empty in settings

Please check that this issue hasn't been reported before.

[X] I searched previous Bug Reports didn't find any similar reports.

Expected Behavior
Usually, when I 'drag and drop' on the timeline, day planner appends the text to the end of my daily note, not affecting anything else in my daily note.
Thanks so much for the help!!!!!!

Steps to reproduce
Picture 1: My daily note when I have just created it, using a template.

Picture 2: When I drag and drop in the timeline one time!
Which Operating Systems are you using?

[ ] Android
[ ] iPhone/iPad
[ ] Linux
[X] macOS
[ ] Windows

Obsidian Version
1.7.5
Possible solution
I believe the problem has to do with it adding '***' at the start of my daily note! Messing with front matter!

PS - I love your plugin, it has helped me so much!!!

UPDATE! SOLVED IT!
Just had to add
Day Planner
to my daily notes template!! Thanks!!!

Just add it to the template!

I have the same problem too.
Besides, it messes up embedded file links when drag and drop. I have underscores "_" in linked file names. A backslash is added before the underscore, it becomes "_"  and the links are all invalid.

Yes, I also encountered the same problem.

@yanqing-song, this is a different issue. Please create another one and add to it the markdown that gets broken on edits.

A temporary fix is disabling automatic sorting of entries on edits.

Fixed in 0.26.2

@yanqing-song, this is a different issue. Please create another one and add to it the markdown that gets broken on edits.

issue #635


Fixed out-of-tree build

Use the original tflite CMakeLists.txt instead of the tflite.cmake file.
Added external delegate as an option to the tflite CMakeLists.txt file
Fixed using local tensorflow tree through FETCHCONTENT_SOURCE_DIR_TENSORFLOW
cmake flag.
Use git apply command instead of git am to avoid creating commit in the
tensorflow tree.

Please review the changes and approve if possible.  You can now use local tensorflow tree to build. Also the external delegate is added as any other delegate to the tensorflow library. There is no need to add external delegate source files or include files to the application. Replaced git am with git apply. The git am command creates a new commit which is not always desirable, especially if you are using the local tree. Removed tflite.cmake which was customized copy of the original tflite build file. The customized copy is no longer needed and it must be maintained in sync with the original. Using original file removes this need.

@xwin would you mind adding one section in the readme to explain FETCHCONTENT_SOURCE_DIR_TENSORFLOW
I think this is a good option to let the user reuse their local TensorFlow instead of always download it from the internet.
Thanks

I will add a section on this and submit an amended pull request.

Extended cmake section with the description of the variable. Please review.


Moved .done files into a subdirectory

No review needed.

coverage: 25.0%. remained the same
when pulling 4c63f6cead4f008fb561349bcc1f7904f943d85a on andy/done-directory
into fb99cff87a3ee3d19b8644456074e5dcc558cd3c on master.


regulation classes

Hi there,
I'd like to request some high-level regulation classes.  I don't see these as particularly useful for functional annotation of genes/gene products; rather, these are motivated by the task of annotation of text with GO classes.  There are several requests herein, but I thought it'd be more comprehensible to aggregate these, as they're related.
First, mirroring GO:'biological regulation', which subsumes regulations of biological processes, functions, and qualities, it'd be really useful to also have GO:'positive biological regulation' and GO:'negative biological regulation' that analogously subsume positive/negative regulations of biological processes, functions, and qualities.  Though there are currently only positive/negative regulations of biological processes, it'd be much easier to annotate all mentions of positive/negative biological regulation rather than figure out which ones are positive/negative regulations of biological processes specifically.
Then, building off of these, I'd like to suggest GO:'biological activation' and GO:'biological inhibition' as subclasses of GO:'positive biological regulation' and GO:'negative biological regulation', respectively, as these are very common in biomedical text.  Other subtypes of positive regulation (e.g., acceleration, initiation, increase (in level of)) and negative regulation (e.g., delay, termination, decrease (in level of)) would be great as well, but aren't as important as the first two.
There are some followup suggestions as well, but I'll first wait for your responses to these... :)
Thanks!
Mike
Reported by: mbada
Original Ticket: geneontology/ontology-requests/10900

The synonyms  'activation' and 'inhibition' already exist for the BPs positive regulation of biological process and negative regulation of biological process; hopefully this is sufficient, we will not create the activation and inhibition BPs, this has led to a lot of misannotation in the past, since it's hard to define and especially, characterize experimentally.


BUG: not possible to delete storage on forked chain

Expected Behavior
Using a fork of mainnet, ganache should be able to set storage variables to 0.
Current Behavior
Ganache is able to set storage variables to any value except 0. If an attempt is made to set the storage variable to 0, it will skip the instruction and continue execution as if successful.
Possible Solution
unknown
Steps to Reproduce (for bugs)

Deploy a contract with non-zero storage value
Fork into ganache
Attempt to set storage value to zero
Use a getter or event to check that it failed

Context
Forked chains should support setting storage values to zero.
Your Environment

Version used: 6.4.2, v6.4.4-beta.0
NodeJS Version: [ ] 6.x, [ ] 7.x (unsupported), [x] 8.x, [ ] 9.x
Operating System and version (include distro if Linux): macOS

I ran into this same issue today and it is really confusing and makes the fork feature unusable for me.

I believe the bug is in how ForkedStorageBaseTrie works: It will just pass put(key, value) through to https://github.com/ethereumjs/merkle-patricia-tree/blob/master/src/baseTrie.js#L85, which in turn will delete the node if the value is 0. Afterwards, queries of that key will fall back to an upstream lookup since keyExists returns false: https://github.com/trufflesuite/ganache-core/blob/develop/lib/utils/forkedstoragetrie.js#L3
How does this usually work? Do you add an issue in ganache-core for that?

My previous analysis was wrong, but I verified now that the following is actually what's causing this bug. The problem is indeed with ForkedStorageBaseTrie, in a way.
The EVM implementation will call the .del(key) method on the storage trie for SSTORE 0 <address>. This causes the key to be removed from the trie, thus the next lookup for that key will cause an upstream lookup due to the logic in https://github.com/trufflesuite/ganache-core/blob/develop/lib/utils/forkedstoragetrie.js#L35, thus the value is effectively "reset" to whatever the value was on the upstream chain.
I believe the correct fix would be to maintain a set of deleted keys in ForkedStorageBaseTrie, add to that set in .del() and remove from that set in .put(). .get() and .keyExists() can then tell the difference between "the key was explicitly deleted in the fork" or "the key was never set in the first place"

I confirmed this by using the following workaround to forkedstoragetrie.js:
ForkedStorageBaseTrie.prototype.del = function(key, callback) {
  // Put an explicit value of 0 so such that no upstream storage lookup is used
  // for later queries on this key
  this.put(key, Buffer.alloc(1, 0), callback);
};

This is not a correct fix probably however, if the EVM implementation assumes there to be no 0 values in the trie, which it probably does.


Language switcher uses the wrong logo for Ruby

We're using the Rails logo, not the Ruby logo -- can we switch this out ASAP?
Here's the list of logos:
https://github.com/GoogleCloudPlatform/gcloud-node/tree/gh-pages/src/images

Should be ok now in deployed sites (might need to clear the cache). I'll send a PR to update the logo for future sites.
BTW, our landing page might benefit a thorough check to see that we are aligned with other gcloud-* projects.

The other sites logos are good -- shouldn't need anything else.
Going forward, we should start exporting our doc data to the standard
format so we don't have to maintain our own gh-pages stuff :
https://github.com/GoogleCloudPlatform/gcloud-common/tree/master/site

Going forward, we should start exporting our doc data to the standard
format so we don't have to maintain our own gh-pages stuff :

Providing a json-based doc instead of what we have now in gh-pages (which
includes among others Javadoc, and standard maven project information) would be a mistake
in my mind. Providing it in addition is probably fine (though I would expect Java people to be
more comfortable and prefer the javadoc over other doc formats).

Yes, this is not instead -- but in addition.


Allow checking for updated devel packages with --check and --devel

It would definitely be cool if pacaur could support checking for updates to devel packages with some set of options, maybe --check and --devel.  This would be especially easy and quick when the clean option is set to false for devel packages, as all that would be needed is to check to see if the repository needs updating.

Do you mean actually downloading all --devel sources and updating pkgver()? This is actually never done before displaying the prompt because it is a slow operation, and eventually not that important when you do your regular update with --devel.

Yes, that is exactly what I mean.  I agree that it is not that important, but it would be kind of nice to have.  Also, it is not extremely slow when cleandevel is set to false.

True, it might be worth it if cleandevel is set to false. I'll have a look shortly.

I don't know if I should open a new issue for this, as it could be a solution or addition to this issue:
Could you add an option that, if you use --devel and a package is already up-to-date, skips that package? So you can update git packages, even if pkgver() hasn't changed, without updating already up-to-date packages.
If there is already an option to do that, please tell me and ignore my request ;).

@benutzer193: The --needed option, used along with the --devel option should do what you want.

@benutzer193 Yes, --needed is what you are looking for.

@aereaux  @rmarquis  thank you very much!

@aereaux @rmarquis I second that, wonderful feature, thanks!

Also need to fix the output alignment:

Done is ce7cf4f58ae36fc6886340b66aee2e73312a2871. As this is unsurprisingly slow, the --needed flag is required, otherwise the standard behavior is used.


Delta addr calculation when last packet emitted by trace encoder was format 0 subformat 1 packet

Delta address = current instruction address (that needs to be reported) - last reported address
Consider the following example:
The trace encoder emits the following packets:

Packet 2 reporting address X
Packet 0 subformat 0 reporting address Y using jump target cache index
Packet format 1 wants to report address Z

Packet format 1 wants to report address Z using delta address, how will the delta address be calculated:
Z - X OR Z - Y?
(where X, Y, and Z are full addresses)
According to the trace spec:
In delta address mode, addresses are encoded as the difference between the actual address of the
current instruction and the actual address of the instruction reported in the previous packet that
contained an address.

The packets that do not report address are:

format 3 subformat 2
format 3 subformat 3
format 1 (no address, branch map)
format 0 subformat 0 (no address, branch count)

Based on the answer to the above question, the list would also include format 0 subformat 1 packets.

Muhammad,
It will be Z - Y.
Iain
From: Muhammad Saadi Abbasi @.>
Sent: 22 December 2024 08:30
To: riscv-non-isa/riscv-trace-spec @.>
Cc: Subscribed @.***>
Subject: [riscv-non-isa/riscv-trace-spec] Delta addr calculation when last packet emitted by trace encoder was format 0 subformat 1 packet (Issue #182)
Delta address = current instruction address (that needs to be reported) - last reported address
Consider the following example:
The trace encoder emits the following packets:

Packet 2 reporting address X
Packet 0 subformat 0 reporting address Y using jump target cache index
Packet format 1 wants to report address Z

Packet format 1 wants to report address Z using delta address, how will the delta address be calculated:
Z - X OR Z - Y?
(where X, Y, and Z are full addresses)
According to the trace spec:
In delta address mode, addresses are encoded as the difference between the actual address of the
current instruction and the actual address of the instruction reported in the previous packet that
contained an address.
The packets that do not report address are:

format 3 subformat 2
format 3 subformat 3
format 1 (no address, branch map)
format 0 subformat 0 (no address, branch count)

Based on the answer to the above question, the list would also include format 0 subformat 1 packets.

Reply to this email directly, view it on GitHubhttps://github.com/riscv-non-isa/riscv-trace-spec/issues/182, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ALQOPSVAWSCUTOUECTTHH232GZ2ABAVCNFSM6AAAAABUBNS3WGVHI2DSMVQWIX3LMV43ASLTON2WKOZSG42TINJUHE3DENY.
You are receiving this because you are subscribed to this thread.Message ID<EMAIL_ADDRESS>

OK, that makes sense.

Could a clarification be added into the current batch of PRs?  It's not perfectly clear that "the previous packet that contained an address" includes packets that include an "index" field instead of an explicitly named "address" field.


Updating Vmware references in Horizon samples

Updating Vmware references in Horizon samples
PR for the same - https://github.com/euc-oss/euc-samples/pull/5

PR #6 merged


Admin module Oracle add operation_ record_ Log table processing

// Describe your PR here; eg. Fixes #3489

Make sure that:

[ ] You have read the contribution guidelines.
[ ] You submit test cases (unit or integration tests) that back your changes.
[ ] Your local test passed mvn clean install -Dmaven.javadoc.skip=true.

Codecov Report

Merging #3490 (bc8dbad) into master (177cdf9) will decrease coverage by 0.40%.
The diff coverage is 88.88%.

:exclamation: Current head bc8dbad differs from pull request most recent head 286dfd4. Consider uploading reports for the commit 286dfd4 to get more accurate results

@@             Coverage Diff              @@
##             master    #3490      +/-   ##
============================================
- Coverage     62.87%   62.46%   -0.41%
+ Complexity     5544     5504      -40
============================================
  Files           834      834
  Lines         23172    23172
  Branches       2098     2095       -3
============================================
- Hits          14570    14475      -95
- Misses         7266     7351      +85
- Partials       1336     1346      +10

Impacted Files
Coverage Δ

...in/mybatis/oracle/OracleSQLPrepareInterceptor.java
0.00% <0.00%> (ø)

...che/shenyu/admin/config/DataBaseConfiguration.java
100.00% <100.00%> (ø)

...plugin/logging/rocketmq/LoggingRocketMQPlugin.java
22.72% <0.00%> (-72.73%)
:arrow_down:

...ache/shenyu/plugin/grpc/cache/GrpcClientCache.java
75.00% <0.00%> (-25.00%)
:arrow_down:

...controller/ShenyuClientHttpRegistryController.java
77.77% <0.00%> (-22.23%)
:arrow_down:

...ogging/rocketmq/body/LoggingServerHttpRequest.java
0.00% <0.00%> (-20.00%)
:arrow_down:

...che/shenyu/sync/data/http/HttpSyncDataService.java
68.29% <0.00%> (-15.45%)
:arrow_down:

...yu/sync/data/http/refresh/AbstractDataRefresh.java
70.00% <0.00%> (-13.34%)
:arrow_down:

...ruptor/RegisterClientServerDisruptorPublisher.java
52.94% <0.00%> (-11.77%)
:arrow_down:

...a/org/apache/shenyu/common/utils/VersionUtils.java
67.85% <0.00%> (-10.72%)
:arrow_down:

... and 40 more

Continue to review full report at Codecov.

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


fix broken docs link

Our readme is getting a little longer. How about using a badge for this? Like this one:

Our readme is getting a little longer. How about using a badge for this? Like these ones:

Edit: If we specify which Python versions our analyzer supports, we can use this one (currently says 3 but it takes the version from the package on PyPI):

https://packaging.python.org/guides/distributing-packages-using-setuptools/#classifiers

Like this much better than adding a section i nthe docs, thanks @omri374 ! I would go with the first option, since we did got some attempts with python 3,9 and I wanted to highlight the we support 2.6 3.7 3.8

@omri374 Will it be rendered in mkdocs?

@omri374 Will it be rendered in mkdocs? Or are we talking mentioning only in the main readme not in the docs?

@SharonHart I only added it to the main readme, not the github pages index file


Translate Java Debugging Section to Spanish

Signed-off-by: Eddú Meléndez<EMAIL_ADDRESS>

@marcosnils @ManoMarks thanks. PR has been updated.

@eddumelendez you squashed the commits and now I can't see what you changed exactly :(.

@marcosnils sorry for the squash. I have added a new commit with the fix for the last comment.

@eddumelendez @mano LGTM

Thanks @eddumelendez!


Can't create comments from checkout pull request

Extension version:
VSCode Version: 1.41.1
OS: Windows 10

How I can't add comments:

Go to PR side panel
Click "checkout pull request" from context menu
Open local version of file
Add comment (plus buttons)
Click "Add comment"
Get error (see screenshot)

How I can add comment:

Open file from PR side panel.
I can add comments, they appear online.

I would prefer first option for less visual culter (red, green no wrap lines and so on).

Thanks for reporting this @Vallek! I'm having trouble reproducing this issue so far, I'll see if I can add some more logging to narrow down the problem. Do you consistently see this problem?

Hi! I don't have a lot of repos to test it, but it never works with two repos I'm working with. I think it used to work some time ago.
Could it be some settings?

I think this is a duplicate of https://github.com/microsoft/vscode-pull-request-github/issues/1339


Emit generic type definition EETypes

Set the expected flag and skip emitting fields that don't make sense for
generic type definitions.

LGTM

Why is the CI not running?


Cannot add new relationships between Entities

Describe the bug
When adding a new relationship between entities, it's not possible to save the results.  The page hangs and the new relationship is never saved.
To Reproduce
Steps to reproduce the behavior:

Edit a publication, going to the "Relationships" tab
Click "+Add" next to Authors and attempt to add a new Author
Select a local author who is already in the system.
Click "Save".
Behind the popup, you'll see the author has appeared.  But the popup never closes.  It hangs.
Click away from the popup and it will finally close.  It looks like the Author may be added.
Now reload the page in your browser. The new Author disappears. It was not added.

NOTE: If you try the same process but attempt to import a new Author from an external source, the imported Person Entity will be created. However, it will not be linked to your Publication.
Expected behavior
Obviously this should work like in 7.x.

I retested this issue in the sandbox and found, that if you hit save on the itempage as well, it works for me. I did the following:

Edit a publication, going to the "Relationships" tab
Click "+Add" next to Authors and attempt to add a new Author
Select a local author who is already in the system.
Click "Save".
Now, the popup hangs for me as well
So, click elsewhere on the page (the popup will close)
Click save on the top of the relationship page
Reload the page -> The related entities are still there.

Still, I see the issue with the hanging popup window, which is not closing itself.

Thanks @Leano1998 for the additional test!  I think I realize where I went wrong.  I forgot your step 7.  I didn't go back and click "Save" at the top of the relationship page.  I had assumed the "Save" button on the popup modal was the only button I had to click.
So, this is working. But the behavior of saving a new relationship is slightly confusing & the popup window never closes itself.

I looked a bit into this and think it might be related to this older issue #1371
If I add the following line (which has been removed in this Commit), the window closes for me:
        setTimeout( () => {
          this.submit.emit();
          this.modalRef.close()
        },1000);

edit-relationship-list.component.ts (L352)

But I don't fully understand, why it has been removed.

Thanks @Leano1998 for working to track that down.  I don't know the answer to your question, but it seems like the code you referenced was removed in #1329 by @atarix83 .  This seems to have been over 3 years ago now, so it may be difficult to track down why that change was made.
If you want to simply create a PR to add the fix that works for you, that may be the best approach.  That way we can get some additional testers to ensure it also works for them.

Okay, I'll create a PR!


Configure Secure Cookies via Environment variable

Be able to affect whether or not dotCMS will serve Secure Cookies (https://dotcms.com/docs/latest/cookie-use#Properties) via environment variables

In my mind, almost every variable in dotmarketing-config.properties should be overwriteable via environmental variables.

On dotCMS we have interpolation for dotmarketing-config.properties
You can do something like
property1={env:SomeEnvironmentKey}
property2={sys:SomeSystemKey}

As you can see the first one is to get the value from the environment and the second from the system, the reference is here: https://dotcms.com/docs/latest/changing-configuration-properties#References

@jdotcms That helps for my case, since I'm using the dotcms docker image as a base and building on top of it I can add those references into my dotmarketing-config.properties or dotmarketing-config-ext.properties files, and then I can turn on/off secure cookies depending on the environment without the extra overhead of having to put together a dockerize template of those files.
@wezell  For people who want to just use the vanilla dotcms docker image, or just want to do less work, I think your opinion makes sense. The dotmarketing-config.properties, by default, could use string interpolation/references and the docker image provides all of the sensible default values for all of the environment variables.


Categories of PIDs

This was raised in PR #479, in this comment:
"to 3) up to @apdavison ; from my side of course can this be added to whatever we see fit. Question: should we not rather introduce categories of PIDs for the different RPs/RPVs to more fluently add new PIDs when needed? @olinux would that be dynamically adopted in the KG system?"
I'm adding it as an issue to keep it open since I am about to merge the PR and this is the only open discussion point. Whether or not the ID from the PR is also added to model versions, can be discussed at a later point. If we conclude to add those categories, it can be fixed in parallel with the update.

@openMetadataInitiative/openminds-developers I though about this a bit more and I'm not sure if it is worth introducing different categories of PIDs for the different research products. I would suggest we have one category for PIDs and allow them for all RPs/RPVs as selection (even if they are not used by all of them)


'Partials' should be recognized as invalid

Overview
Input 'f<EMAIL_ADDRESS>parses as username = 'f' and domain = 'oo'. We can recognize this case by checking if the parse result field partials is true.

Closed by #7


Move the theme to furo, and add some markdown examples.

What do these changes do?
as titled.
Related issue number
Fixes N/A

It shouldn't affect current "stable" documentation (a.k.a. https://graphscope.io/docs) and should be safe to merge.


Pushing site_cmp3 statistics to prometheus pushgateway

Monitoring consistency check jobs with newer and more suitable technologies is essential. Thus, the pull request implemented allow to a part of the consistency check (site_cmp3) pushing metrics to a prometheus pushgateaway.
For the implementation, the MetricManager from Rucio was used, in order to align the monitoring of consistency check with the monitoring of other CMS Rucio relevant tools.
This implementation can easily integrate other parts of consistency check in the future, allowing even more detailed monitoring.

@dynamic-entropy changes are applied.

What is scan_type? why is a command being passed to the prometheus script?
Isn't it supposed to simply scrape the result jsons and push to the push gateway?

It is not a function call. site_cmp3 is handled like a string inside the python script. However, I changed it so there is no more confusion.
In details, site_cmp3 is the name of the script that includes the command. site_cmp3 triggers the required subprocesses for consistency check on sites merged volume. Different scripts that trigger different subprocesses are used for the unmerged volume of the sites and RAL site. I believe that allowing prometheus script to take the script name as an argument (scan_type), will help to easily integrate in the future the rest of consistency check in the monitoring


Javascript in

What happens:

An error message notification pops up: "Request has failed: HTML language features (extension)"
The bottom drawer pops open with error details (see below)
Sometimes I can continue working; other times, basic editor features no longer function, requiring restarting VSCode.
However, once the error begins happening, it often keeps happening every few seconds, each time popping open the bottom drawer, again necessitating restart.
For more details, please see the error output below.

Under what circumstances?
While editing javascript within the <script> tags in a mixed JS/CSS/HTML document.  Specifically, I've noticed it happens when I pause for a moment while typing, so the JS doesn't make sense yet.  For instance, writing if (condition) {...}, if I haven't written the condition yet -- BUT, un/fortunately the error doesn't occur in 100% of such instances.
How frequently?
Multiple times a day.  Usually not right away.
But as noted above, once it begins, there's a 50/50 chance it will keep happening every few seconds, requiring restart.
When did it start?
Began 2-4 weeks ago.
Does it happen without extensions enabled?
I wish I could answer this right now, but it usually doesn't begin until a while into a session, and I rely on extensions to work, so I'm not sure how best to test that.
Version info
OS X 10.13.6
VSCode Version: 1.40.1
Commit: 8795a9889db74563ddd43eb0a897a2384129a619
Date: 2019-11-13T16:47:44.719Z
Electron: 6.1.2
Chrome: 76.0.3809.146
Node.js: 12.4.0
V8: 7.6.303.31-electron.0
OS: Darwin x64 17.7.0
Error details
[Error - 12:06:12 PM] Error while resolving completion proposal: Debug Failure. Invalid cast. The supplied value [object Object] did not pass the test 'isCaseClause'. Error: Debug Failure. Invalid cast. The supplied value [object Object] did not pass the test 'isCaseClause'.     at Object.cast (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/typescript.js:1445:25)     at getContextualType (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/typescript.js:102990:50)     at getCompletionData (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/typescript.js:103240:51)     at getSymbolCompletionFromEntryId (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/typescript.js:102844:34)     at Object.getCompletionEntryDetails (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/typescript.js:102884:36)     at Object.getCompletionEntryDetails (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/typescript.js:125916:35)     at Object.doResolve (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/html-language-features/server/dist/htmlServerMain.js:1:541001)     at /Applications/Visual Studio Code.app/Contents/Resources/app/extensions/html-language-features/server/dist/htmlServerMain.js:1:491955     at Immediate._onImmediate (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/html-language-features/server/dist/htmlServerMain.js:1:548528)     at processImmediate (internal/timers.js:439:21)
Installed extensions  (not all of these are enabled, but I didn't see a way to filter for that):
2gua.rainbow-brackets
albert.TabOut
albymor.increment-selection
Anjali.clipboard-history
arthurwhite.white
Atishay-Jain.All-Autocomplete
benjavr.jshintrc-generator
bierner.comment-tagged-templates
cdes.wrap-with-template-string
chunsen.bracket-select
closedcontour.very-smart-select
cmstead.jsrefactor
CoenraadS.bracket-pair-colorizer-2
dbaeumer.jshint
dbaeumer.vscode-eslint
donjayamanne.githistory
eamodio.gitlens
ecmel.vscode-html-css
eg2.tslint
esbenp.prettier-vscode
formulahendry.auto-rename-tag
formulahendry.code-runner
fosshaas.fontsize-shortcuts
gebilaoxiong.vscode-flatland-icon-theme
gerane.Theme-Github
Gruntfuggly.activitusbar
Gruntfuggly.scoper
Gruntfuggly.todo-tree
Gruntfuggly.ubertab
gtwsky.oolory
HookyQR.minify
huizhou.githd
ispushkin.pushkin-is-white-theme
jasonlhy.vscode-browser-sync
jeremyrajan.browsersync
josee9988.minifyall
letrieu.expand-region
lukapetrovic.image-resizer
mhutchie.git-graph
morissonmaciel.typescript-auto-compiler
mrmlnc.vscode-autoprefixer
ms-vscode.vscode-typescript-tslint-plugin
msjsdiag.debugger-for-chrome
olback.es6-css-minify
ow.vscode-subword-navigation
plibither8.remove-comments
pranaygp.vscode-css-peek
pustelto.bracketeer
ritwickdey.LiveServer
ryu1kn.partial-diff
ryuta46.multi-command
sashaweiss.bracket-jumper
saviorisdead.Theme-GitHubCleanWhite
Shan.code-settings-sync
SirTori.indenticator
sohamkamani.code-eol
sporiley.css-auto-prefix
streetsidesoftware.code-spell-checker
thomaspink.theme-github
toba.vsfire
usernamehw.errorlens
v4run.transpose
vincaslt.highlight-matching-tag
VisualStudioExptTeam.vscodeintellicode
vittorioromeo.expand-selection-to-scope
vmsynkov.colonize
WallabyJs.quokka-vscode
webreflection.literally-html
xyz.local-history
yuichinukiyama.vscode-preview-server

Update: I've working with extensions disabled today, and I am not experiencing the problem.
I suspect (but am not sure) that the problem extension might be "ErrorLens" (usernamehw.errorlens), because I believe it's the only one I installed around the time the problem started happening.


Microsoft Security Advisory CVE-2024-30046 | .NET Denial of Service Vulnerability

URL(s)
https://github.com/dotnet/aspnetcore/issues/55714
Description
I'm using the .net 6.0 to develop the software, it was scanned a security issue named CVE-2024-30046 using scan tool, but I found that the affected software only includes .net 7.0 and .net 8.0, see the https://github.com/dotnet/aspnetcore/issues/55714, do you know whether this issue will affect .net 6.0 or not? Thanks

Adding @wtgodbe since you handled the issue on the apnetcore side. Do you know if this also affects .NET 6.0?

This issue does not affect .NET 6.0

Thanks!
@leon-tliang3 I'm going to close this as this issue does not affect .NET 6.


remove redundant babel dependency

Motivation
Remove babel for it already included by metro
Test Plan
pass all current ci.
Related PRs
none
Release Notes
[GENERAL] [INTERNAL] [MISC] - remove redundant dependency.

But it is required by jest/preprocessor.js, does it safe to make such dependency implicit?

It's not here in the first place. It's added in solving ci when upgrade to babel 7. I think since metro has all the babel packages, there is no need to declared it multi times.

But it's here at least today (2018-06-17): https://github.com/facebook/react-native/blob/15f84420f0c53f53f11a93b609385ca741379898/jest/preprocessor.js#L15
File configured to be used by jest: https://github.com/facebook/react-native/blob/15f84420f0c53f53f11a93b609385ca741379898/package.json#L21-L24
And jest can run against react-native working copy:
https://github.com/facebook/react-native/blob/15f84420f0c53f53f11a93b609385ca741379898/package.json#L124-L126
So it is clearly dev dependency of RN. The fact that @babel/core required by metro do not remove this dependency, imho. But may be RN devs can manage dependency between RN, jest and metro consistently (but they can't), then ok, it's redunant.

When use babel 6, it's there too. Check the git history.


Stopping and starting the EC2 instances (master and nodes)

Hi,
This is more a question than an issue.
Can I shutdown/start the EC2 instances without any bad surprise? What are the consequences I should be aware of? Of course, I could try on my own but I've already got some projects on my OpenShift installation.
I am asking because of following concern: OpenShift installation done by this project generates monthly costs of around 500$. There are periods of time where we're NOT using the installation and, to save some money, we would like to stop the EC2 instances.
Is that recommendable? Many thanks and cheers,
christian

@dwmkerr,
Exactly as mentioned by @ribeaud, we are doing POC and I've installed 3.9v successfully and it's up and running and we were able to login web UI. However, after shutdown and bring up, found that, unable to access Origin console URL https://master-node-public-IP:8443 with some error like firewall issue. I've checked inside master node, all services up and running and even getting nodes/pods information, but URL is not working. It should work because all private IPs remain same.
Appreciate if you can do the needful to get rid of it.
Regards,
Vasu


Forward X-Request-ID header to Notification call

Helps trace the original call that caused the notification service invocation

Ike Plugins (test-keeper)
Thank you @aslakknutsen for this contribution!

It appears that no tests have been added or updated in this PR.
Automated tests give us confidence in shipping reliable software. Please add some as part of this change.
If you are an admin and you are sure that no test is needed then you can use the command /ok-without-tests as a comment to make the status green.
For more information please head over to official documentation. You can find there how to configure the plugin - for example exclude certain file types so if PR contains only them it won't be checked.

@aslakknutsen snapshot fabric8-wit image is available for testing.  docker pull docker.io/fabric8/fabric8-wit:SNAPSHOT-PR-1988-1

Codecov Report

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

@@           Coverage Diff           @@
##           master    #1988   +/-   ##
=======================================
  Coverage   64.89%   64.89%
=======================================
  Files         148      148
  Lines       13630    13630
=======================================
  Hits         8845     8845
  Misses       3956     3956
  Partials      829      829

Impacted Files
Coverage Δ

notification/notification.go
12.85% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

@xcoulon the last modified once were only removed in auth? Seems flaky atm
--- FAIL: TestSuiteWorkItemLinkType (0.27s)
    --- FAIL: TestSuiteWorkItemLinkType/TestList (0.12s)
        --- FAIL: TestSuiteWorkItemLinkType/TestList/not_modified_using_IfNoneMatch_header (0.01s)
        	work_item_link_type_testing.go:240: invalid response status code: got 200, expected 304
        	golden_files_test.go:53: mismatch of actual output and golden-file /tmp/go/src/github.com/fabric8-services/fabric8-wit/controller/test-files/work_item_link_type/list/not_modified_using_ifmodifiedsince_header.res.headers.golden.json:
        		 {
        		  "Cache-Control": [
        		    "max-age=2"
        		  ],
        		  "Content-Type": [
        		    "application/vnd.api+json"
        		  ],
        		  "Etag": [
        		    "0icd7ov5CqwDXN6Fx9z18g=="
        		  ],
        		  "Last-Modified": [
        		    "Mon, 01 Jan 0001 00:00:00 GMT"
        		  ]
        		}

        		github.com/fabric8-services/fabric8-wit/controller_test.testableCompareWithGolden
        			/tmp/go/src/github.com/fabric8-services/fabric8-wit/controller/golden_files_test.go:123
        		github.com/fabric8-services/fabric8-wit/controller_test.compareWithGoldenAgnostic
        			/tmp/go/src/github.com/fabric8-services/fabric8-wit/controller/golden_files_test.go:51
        		github.com/fabric8-services/fabric8-wit/controller_test.(*workItemLinkTypeSuite).TestList.func6
        			/tmp/go/src/github.com/fabric8-services/fabric8-wit/controller/work_item_link_type_blackbox_test.go:215
        		testing.tRunner
        			/usr/lib/golang/src/testing/testing.go:657
        		runtime.goexit
        			/usr/lib/golang/src/runtime/asm_amd64.s:2197

@aslakknutsen I opened https://github.com/fabric8-services/fabric8-wit/issues/1990 to fix the test failure you reported above.


Got wandb network(ssl error) error even when I run wandb in offline mode

I Got wandb network(ssl error) error even when running in the offline mode.
I really don't know why this bug can occur!

I would also check if you have a proxy configured that doesn't have trusted SSL certificates.  You can check for proxy configuration by running env | grep PROXY to see if any PROXY environment variables are set.

Leslie commented: And can you also upgrade your version of wandb
Can you try curl -v https://google.com as well as curl -v ``https://api.wandb.ai/graphql

I upgrade wandb to 0.12.5 version. And, there is indeed an error when curl -v https://api.wandb.ai/graphql:

I would also check if you have a proxy configured that doesn't have trusted SSL certificates. You can check for proxy configuration by running env | grep PROXY to see if any PROXY environment variables are set.

I have checked that no PROXY environment variables are set.

We are closing this due to inactivity, please comment to reopen.


CSP registers style-src conflict with cookiepunch.js

I have a problem to use CSP-Header with this package.
It gives me this error:

In the code the error is that it doesnt have a nonce value the dynamically added stylesheet.
Can we maybe define this in the settings or other possibility?
  function s(e) {
                    var t = document.createElement("style")
                      , o = e.attributes || {};
                    if (void 0 === o.nonce) {
                        var i = n.nc;
                        i && (o.nonce = i)
                    }
                    if (Object.keys(o).forEach((function(e) {
                        t.setAttribute(e, o[e])
                    }
                    )),
                    "function" == typeof e.insert)
                        e.insert(t);
                    else {
                        var a = r(e.insert || "head");
                        if (!a)
                            throw new Error("Couldn't find a style target. This probably means that the value for the 'insert' parameter is invalid.");
                        a.appendChild(t)
                    }
                    return t
                }

@adrian-cerdeira Hey, I also tried to use CSP-Headers in Neos in combination with this package.
I started using https://github.com/LarsNieuwenhuizen/Nieuwenhuizen.ContentSecurityPolicy where the last version is over 6 years old now, so it isn't working anymore. I'm trying to repair the package at the moment and later there will be a working version of the package.
I was able to add nonce-Values to Tags where needed but the value was removed. This is an ongoing issue in Klaro, which is used by CookiePunch. There is an open issue and PR since a long time now in Klaro: https://github.com/klaro-org/klaro-js/issues/441
So hopefully the issue in Klaro will be resolved soon and I'll be able to publish the fixes for the Nieuwenhuizen.ContentSecurityPolicy package and then we can easily use CSP-Headers in Neos.
But in this package here there is no issue with CSP and nonces directly so I think that the issue can be closed @fheinze

@t-heuser Thanks for the information, keep me updated 😃


feat: add MemoryCell structure

At the moment, in the Memory structure, the data field is a vector composed of elements of type MaybeRelocatable.
https://github.com/keep-starknet-strange/cairo-zig/blob/747dc4a492d0f35e4894edb026f6273c7805d0a7/src/vm/memory/memory.zig#L31-L36
In the future, it may be desirable to add other information in addition to only a MaybeRelocatable type. In particular, I find the boolean flag is_accessed introduced by the Rust implementation quite useful in this context. I therefore propose the implementation of a MemoryCell structure which will initially contain the fields:

maybeRelocatable of type MaybeRelocatable
isAccessed of type boolean

This structure can then be extended and improved. In this same issue, it will also be necessary to replace the MaybeRelocatable type in the data vector of the Memory structure by the MemoryCell structure thus created.
Here are the Rust implementation details here.

Oh, seems interesting, I will take this feat

Oh, seems interesting, I will take this feat

Assigned!

@lana-shanghai I can take this if @Deecellar isn't working on it anymore. Should be lots of new issues want to get pr through for fib program deadline.

have this almost finished I'm getting some errors in my Memory.markAsAccessed function that I think could be because of passing my value and not reference? Should i open a pr or can you check my local branch.  https://github.com/oxlime/cairo-zig/tree/feat/add_MemoryCell
Changes to Memory requires a lot of testing changes so just want to make sure this is the right direction.


Not able to edit articles

I am not able to edit articles. When I edit and click Save an error comes up saying PublishedAt field is required
The browser console logs are:
2.69355630.chunk.js:1 {err: c}err: cerrors: Array(1)0: "components.Input.error.validation.required"length: 1__proto__: Array(0)inner: Array(1)0: cerrors: ["components.Input.error.validation.required"]inner: []message: "components.Input.error.validation.required"name: "ValidationError"params: {path: "publishedAt", value: Tue Aug 18 2020 11:00:00 GMT+0530 (India Standard Time), originalValue: w, label: undefined}path: "publishedAt"type: "required"value: Tue Aug 18 2020 11:00:00 GMT+0530 (India Standard Time) {}stack: "ValidationError: components.Input.error.validation.required↵    at https://himu-blog.herokuapp.com/admin/main.2c92ad92.chunk.js:1:4389329↵    at https://himu-blog.herokuapp.com/admin/main.2c92ad92.chunk.js:1:4388769↵    at async Promise.all (index 0)"__proto__: Errorlength: 1__proto__: Array(0)message: "components.Input.error.validation.required"name: "ValidationError"path: undefinedtype: undefinedvalue: Seo: {id: 8, metaTitle: "Himu's first Blog", metaDescription: "A glimpse of what wonderful pieces of surpises are going to come in the upcoming days", shareImage: {…}}category: {name: "tech", id: 2}content: "# My First Blog!↵↵Let's get started!   I did a fresh install, edited an existing article, and saved with no problem.  I also created a new article, then edited that article with no problem.
Could you give steps to reproduce the problem from a fresh install?  Also could you provide your OS, browser, and node version?

Hello!
As we have not received any new or updated information in over 14 days we are marking this issue/discussion as closed. Should you have new information please feel free to respond and we can consider reopening it.
For others finding this closed issue and you have updated information, please open up a new bug report and simply reference this closed bug report so that we can get any new information you may have. If you have questions please refer to the contributor's guide on opening issues.
Thank you and have a great day!


Migrate the RISC-V entry point to global_asm!.

Building libtock_runtime on RISC-V no longer requires an external toolchain! We use global_asm! instead.
This runs into a LLVM bug where it fails to parse a comment after a mv instruction unless the mv instruction has a semicolon after it, so I added a semicolon. I'm not sure where to put a descriptive comment, though.
I have not migrated the ARM entry point to global_asm!. When I tried, it generated significantly different code from the external toolchain, and I have no way to test if the new code works. Perhaps it's a Thumb 1 vs. Thumb 2 thing? I'll leave debugging that to someone else.

Oops, I forgot to update the GitHub workflows -- they shouldn't need an external assembler anymore! Also, we can now test compilation for a RISC-V target in the Mac OS workflow.
I'll push a commit with the updated workflows when they're ready.

@kupiakos Does this resolve the issues that ti50 is having with building the entry point?

@kupiakos Does this resolve the issues that ti50 is having with building the entry point?

I don't currently have access to hardware to do a full test, but it does appear to build correctly.

bors r+