Diffs recognizing less similarity since 4.10

Since version 4.10 of the library, diffs are recognizing less similarity between texts.
In the attached program, under version 4.9, the library correctly recognizes that there is only a 5-character difference between the texts (3 letters + 2 whitespace characters). Under version 4.10, the library reports that a large block of identical text has been deleted and then added.
TestCase.txt
4.9 output:
   1 EQUAL    apple1                                                                           apple1
   2 EQUAL    apple2                                                                           apple2
   3 EQUAL    apple3                                                                           apple3
   4 CHANGE   A man named Frankenstein==oldCHANGE==> abc <==old==to Switzerland for cookies!   A man named Frankenstein
   5 CHANGE                                                                                ==newCHANGE==>xyz<==new==
   6 CHANGE                                                                                to Switzerland for cookies!
   7 EQUAL    banana1                                                                          banana1
   8 EQUAL    banana2                                                                          banana2
   9 EQUAL    banana3                                                                          banana3

4.10 output:
   1 EQUAL    apple1                                                                           apple1
   2 EQUAL    apple2                                                                           apple2
   3 EQUAL    apple3                                                                           apple3
   4 CHANGE   A man named Frankenstein==oldDELETE==> abc to Switzerland for cookies!<==old==   A man named Frankenstein
   5 INSERT                                                                                ==newINSERT==>xyz<==new==
   6 INSERT                                                                                ==newINSERT==>to Switzerland for cookies!<==new==
   7 EQUAL    banana1                                                                          banana1
   8 EQUAL    banana2                                                                          banana2
   9 EQUAL    banana3                                                                          banana3

Admittedly, there are aspects of the 4.10 output that are improved over the 4.9 output.  For example, the fact that line 6 of the 4.9 output is indicated as a CHANGE, but there is no oldLine text and no changes in the newLine text can be confusing.  However, the sacrifice in accuracy in 4.10 is far less desirable.  In 4.9, the line 6 difference is indeed a change, but it's almost like a new tag is needed to indicate a group (?) change to make it clear that the change is a continuation of the line 4 difference.

Sorry, about the late answer:
What about .ignoreWhiteSpaces(true)?


Adding attackSpeed nbt to item

Trying to add generic.attackSpeed to an item using ItemStackBuild.customData() but getting the following error: https://pastebin.com/48iPd5nZ

I have tried the to switch the arguments, that returns same error

you should use attribute modifier methods to edit attributes, not the custom data.

you should use attribute modifier methods to edit attributes, not the custom data.

Also for 1.8?

Also for 1.8?

because its for 14 and later

because its for 14 and later

also, the attack speed came at 1.9

it's not for 1.8

also, the attack speed came at 1.9

it's not for 1.8

oh, guess I should have done some researched before hand :/

oh, guess I should have done some researched before hand :/


Very beautiful project, but here I still couldn't run

Hello, how are you?
First of all I want to thank you for the beauty you are doing, it is very good!
But I followed the steps of installing the project, and here it gave this error:

If anyone knows how to proceed, please :) thanks in advance.

Hi @vitorhugosg,
Thanks for your kind words!
It seems like you're app is having trouble connecting to the database. Did you set it up? If so, is it working in other areas of the application?

I managed here, it was a failure with the connection, I gave a php artisan cache: clear and it worked.
Again a great project, congratulations.

I've been testing here trying to create my first table, but it turns red like this when I click on create. Do you know how I should proceed?
I'm sorry for the newness.

No problem @vitorhugosg, glad you got rid of the connection problem!
The red line you're seeing is because of the model name you are using which is invalid.
If you want to store models in App\Models you can specify that behavior in the config/schematics.php file. You'll see something like this:
    'model' => [
        'namespace' => 'App\\',
        'path' => app_path(), <----- This is the path used for creating and editing
        'paths' => [ <----- These paths are scanned for models
            app_path(),
        ],
    ],

Also, you can't use underscores or special characters in the model name as they are used as classname.

It worked perfectly, thank you very much and again congratulations for the work;)


fix GET /__REMIX_ASSETS_MANIFEST loop #5663

Closes: #5663

[ ] Docs
[ ] Tests

Testing Strategy:

The loop occurs when you run the unstable dev server, make a change, and quickly undo that change. See this for more details

Superceded by #6133
@conundrumer thanks for your work here! We ended up switching to a different dev server architecture that doesn't rely on polling, so this should be fixed once the tagged PR above lands / is released.


Improve integration tests (plugin + element template loading)

What should we do?
With Cawemo SaaS changes we may want to adjust the cloud-connect-plugin tests.
I see a bunch of options:

Amend the integration test, ensure that spinning up Cawemo (self-managed) is clearly documented
Add an automated end-to-end test to verify the cloud-connect-modeler-plugin works
Change the plugin to execute its E2E tests periodically (every night), on supported platforms (currently does not execute on Windows)

Why should we do it?
Ensure we have E2E coverage of Camunda Modeler plug-in capabilities.

I quickly sketched out how we could use the cloud-connect plug-in as an E2E integration test, executed every night :arrow_right: https://github.com/camunda/cloud-connect-modeler-plugin/pull/132.

With https://github.com/camunda/cloud-connect-modeler-plugin/pull/132 how should we continue with this?

Cawemo SaaS changes

Are you referring to Cawemo SaaS being shut down and self-managed being the only option to test against?

I think https://github.com/camunda/cloud-connect-modeler-plugin/pull/132 + it running nightly is fine. We still want to update the documentation (remove the plug-in, or simply link it).


[Feature request] add a mode for gradle to fail on depreciation warnings

I would be interested in an option org.gradle.warning.mode=fail where gradle would fails in case of a depreciation warnings.
This would be useful to fix all depreciation in the build and then enable this mode to prevent any regression on those depreciation from re-entering the codebase.

This feature seems very nice to have, I would be happy if we could have this feature.

i was about to request this feature. besides its obvious usefulness, my motivation for it is that i think it would work around a shortcoming in the way Gradle reports warnings.
right now i've a deprecation warning i can't fix because i've no idea where it is coming from. when i enable --warning-mode=all with --stacktrace or even -S, all i get is an exception chain that is fully within Gradle and Java runtime. there is not a single useful line in there, no mention of the Android plugin code, my plugin code, or build scripts. i don't even know whether i'm to blame. the trace ends with some worker and gives no info on what triggered the work.
i imagine that if an exception is thrown in a worker, it will be marshaled back to the thread blocking on the work (i hope there is one). given that Gradle doesn't seem to track the code that invoked the worker (or is there a mode for that?), possibly the best bet is throwing from the worker and seeing who gets the bad news. just logging the exception is not enough in many cases.
regarding my deprecation, i'll tackle it at some point in the future, i'll find a way. but this common scenario shouldn't be hard to diagnose better. am i missing something?

Feature has been added and will be in Gradle 5.6

hi, thanks for the fix!
unfortunately it doesn't fix my issue here: https://github.com/gradle/gradle/issues/6491#issuecomment-486415747

If fail is activated, the build will fail at the end if any deprecation warnings were reported

failing at the end is useless for me. and the logged exception is completely useless too. i need the whole chain of chained exceptions, which will only happen if the root cause exception is actually thrown and not simply logged.
really, im at a point where the only way for me to go forward is patching gradle myself so it throws and make a test build. is this really the kind of "help" you want to give people to fix deprecations? my only other option is to wait for gradle 6.0 to finally get to see the whole exception chain.
the deprecation warning effort in gradle is great. the reporting of deprecations is seriously broken and invalidates de whole system. this needs fixing asap.
do you need a real life example of this? i can point you to a repo, and you tell me what plugin line trigger the deprecation. because i'm getting zero help from gradle 5.5.
my suggestion is to make "fail" fail fast on the spot, instead of finishing the process with exit(1) if the deprecation warning list is not empty.
if for whatever reason fail as-is is desirable (could be, for a more general view of number of issues) the add a fast failing "abort" option.
something is desperately needed. im serious about the sample repo offer. reporting truly doesn't work


Uncaught Exception:

Hello, i cant add new blocks. I get this error;

[User Error] Uncaught Exception: Object->__call(): the method 'isnew' does not exist on 'TextBlock'
GET /admin/pages/edit/EditForm/field/Blocks/add-multi-class/TextBlock

Line 761 in /var/www/vhosts/erlanda/erlanda/wwwroot/framework/core/Object.php

What version of SilverStripe are you using? And is there a full stack trace? There's no isNew method within this module.

I fix it , the better button module was cousing the problem. Can i please contact with your privatly. Please it is very important. Just need to ask you a question.

Hey men , this site is yours yes ? https://bummzack.ch/
2017-09-14 20:53 GMT+03:00 Roman Schmid<EMAIL_ADDRESS>
What version of SilverStripe are you using? And is there a full stack
trace? There's no isNew method within this module.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bummzack/page-blocks/issues/17#issuecomment-329559830,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AL8K3uf6IOZJLmZlOqAL3ZVdqXIasULPks5siWgvgaJpZM4PX2B8
.

You already wrote me an Email. Feel free to send me your question… if it's about this module, why not ask it here though? Others might have the same question.

https://bummzack.ch/ <- Can you suggest me how to make the same type block menu as you did in your page ? I use your module and i want to make something similar, but your page do not support links to other sites. AS for example i want to make a image block for example name cars it has car image and when user clicks on it , he is redirected on specific page.

Can you please help me out with this? ;/
2017-09-14 21:02 GMT+03:00 Roman Schmid<EMAIL_ADDRESS>
You already wrote me an Email. Feel free to send me your question… if it's
about this module, why not ask it here though? Others might have the same
question.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bummzack/page-blocks/issues/17#issuecomment-329562083,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AL8K3jcosuuYMI0PncuBjrJRljRd4Nzsks5siWoPgaJpZM4PX2B8
.

@xpowert For your use-case you could either create a custom block, where you add DataObjects with Image & Link, or you add a DataExtension to Image, and add an additional Field with a Link to your Images.

already made it. I can add the code sample for other users. Can you help
with design ?
2017-09-15 13:01 GMT+03:00 Roman Schmid<EMAIL_ADDRESS>
@xpowert https://github.com/xpowert For your use-case you could either
create a custom block, where you add DataObjects with Image & Link, or you
add a DataExtension to Image, and add an additional Field with a Link to
your Images.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/bummzack/page-blocks/issues/17#issuecomment-329739014,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AL8K3jwk3CxodnJ4J8lUFBbtFhdNShyjks5siksTgaJpZM4PX2B8
.


Install time

Install time is currently a bit high, mostly due to the ws module. It's not that big of a deal but it does count towards the user experience.
Amok is not well known at the moment, a five-ten minute install may cause a lot of users to kill the process and get back to work instead of trying it out so it's still a concern. (On a low-end laptop, compiling ws has taken up to 10 minutes at times)

I'll be rewriting rdbg next week or so, this is likely to go in then.

Another concern is the failing optional dependencies, they'll always happen to windows users and they are the ones most likely not to understand that they aren't failures.

There's talk in ions of merging wsock into core so for now I'll close this for now and re-open for consideration later.


Create policy-as-code.md

Describe your changes
Localize policy as a code in Portuguese
Related issue number or link
(resolves #2378)
Checklist before opening this PR (put x in the checkboxes)

[x] This PR does not contain plagiarism

don’t copy other people’s work unless you are quoting and contributing it to them.

[x] I have signed off on all commits

signing off (ex: git commit -s) is to affirm that commits comply DCO. If you are working locally, you could add an alias to your gitconfig by running git config --global alias.ci "commit -s".

Reviewers Please looks on this PR

Hi @ARYANis0001!
Could you join to our channel in the CNCF slack (#glossary-localization-portuguese) and read the guide to the portuguese speakers contributors? Thanks!
https://gist.github.com/edsoncelio/d7fdec7f24d5aba87e3e517c672e9eb5

@ARYANis0001 I have a common comment on your PRs.
PTAL: https://github.com/cncf/glossary/pull/2605#issuecomment-1815774999


Format SETUP.md nicely

Wrapped lines to match the instructions in exercises, and use the other link syntax to increase readability.

Thanks!


Broken image resize

https://openbenches.org/image.php?id=4e56d3f471b0ea4332759ff36588e8d081bcf9e5
Appears rotated in full view. Resample fails.
PHP Fatal error:  Allowed memory size of 134217728 bytes exhausted (tried to allocate 16384 bytes) in /home/openbenc/public_html/image.php on line 21

Image was over 128MB when uncompressed. Due to a DPI of 180(!).
Need a way to catch before storing. Manually corrected.


Variable $card-border-width should take in $border-width as it's default.

Unlike other border-width properties (btn, input, etc.) which take in the $border-width !default card is set to 1px !default. This means that resetting the global $border-width variable does not change the card. If making this change is approved I can submit a PR.

@afholderman You are right. It was fixed here: https://github.com/twbs/bootstrap/commit/1cfbc2c62df86ed1d243591160634965eb0ef2b7
It will be reflected on the next release.
Thanks for reporting this.


Dead links in Showcases/media_view README

The "Getting Started Guide" and "Media View Documentation" links in the media_view readme lead to dead links.

Thanks for the reporting, Nanette! We're adding the content.

Fixed in the PR: https://github.com/meta-quest/Meta-Spatial-SDK-Samples/pull/5
Waiting for merging.


Added array_wrap method to BoltArrayLocal

If a ufunc is called on a BoltArrayLocal, it should never return an array with shape () (a single element). Standard ndarrays extract the scalar out of these arrays and return it instead. The BoltArrayLocal.__array_wrap__ gives our local arrays the ability to extract out scalar values from ufunc results.

Change looks great, maybe add a test or two to check this? Probably just in test_local_basic

K I actually added the tests to test_local_functional because I'm testing using the sum ufunc. Seemed good to keep that consistent with the spark version

Cool, makes sense!

LGTM, merging in.


Updated ServiceAccount.yaml for Docker auth

Changes
ServiceAccount.yaml needs K8s secret parameter as "imagePullSecrets" not "Secrets" as mentioned in the doc.
Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[ ] Has Docs if any changes are user facing, including updates to minimum requirements e.g. Kubernetes version bumps
[ ] Has Tests included if any functionality added or changed
[ ] pre-commit Passed
[ ] Follows the commit message standard
[ ] Meets the Tekton contributor standards (including functionality, content, code)
[ ] Has a kind label. You can add one by adding a comment on this PR that contains /kind <type>. Valid types are bug, cleanup, design, documentation, feature, flake, misc, question, tep
[ ] Release notes block below has been updated with any user facing changes (API changes, bug fixes, changes requiring upgrade notices or deprecation warnings). See some examples of good release notes.
[ ] Release notes contains the string "action required" if the change requires additional action from users switching to the new release

Release Notes
NONE

PR: Updated ServiceAccount.yaml for Docker auth #7736 will solve the issue mentioned below:
https://github.com/tektoncd/pipeline/issues/4603
I faced this issue in my test and it got resolved with the changes which I have submitted in the above PR.

@ashugcet Thanks for the PR, but it is not accurate though. This documentation is about how to make the tekton "helpers" (entrypoint) put the right "docker" auth ($HOME/.docker/config.json or $DOCKER_CONFIG) content in the right place, in the pod that will need it. And, for this, the tekton code looks for the secrets attached to the ServiceAccount and not the imagePullSecrets. Usually, imagePullSecret on a ServiceAccount is to allow the node to pull an image that needs some credentials to access it ; which is different, and not documented here (I don't remember where / if it's documented though).

@ashugcet Thanks for the PR, but it is not accurate though. This documentation is about how to make the tekton "helpers" (entrypoint) put the right "docker" auth ($HOME/.docker/config.json or $DOCKER_CONFIG) content in the right place, in the pod that will need it. And, for this, the tekton code looks for the secrets attached to the ServiceAccount and not the imagePullSecrets. Usually, imagePullSecret on a ServiceAccount is to allow the node to pull an image that needs some credentials to access it ; which is different, and not documented here (I don't remember where / if it's documented though).

But this doc also says that when we associate this ServiceAccount with either PipelineRun or TaskRun it will work, which it doesn't in cases where the task is to deploy a Docker image from private registry using the secret. I was victim of this confusion and before me also few people have wasted time. May be this should be at least mentioned somewhere so that people do not struggle with starting with Tekton. Thanks.

The doc also says: "When the Steps execute, Tekton uses those credentials to access the target Docker registry." Which is not correct.

Well, we need to define "execute". The imagePullSecret is required when the Pod is scheduled (just after being created). When the Pod (or the container/step) execute, the image(s) is (are) already pulled on the Node, so execute in that sense (and in the doc), is when the process in the Pod is starting (or going to start).
But I do agree, that documentation needs a bit more clarification, I created #7737 👼🏼

The doc also says: "When the Steps execute, Tekton uses those credentials to access the target Docker registry." Which is not correct.

Well, we need to define "execute". The imagePullSecret is required when the Pod is scheduled (just after being created). When the Pod (or the container/step) execute, the image(s) is (are) already pulled on the Node, so execute in that sense (and in the doc), is when the process in the Pod is starting (or going to start).
But I do agree, that documentation needs a bit more clarification, I created #7737 👼🏼.
But as is, that pull-request is not correct as it doesn't work in the example it gives (with those changes, if you want a $HOME/.docker/config.json with the content of the secret, it won't be the case, where it was before the PR).

Understood. Thanks for the clarification.


Proxying components causes template to not render

With a co-located component it's possible to proxy the component and everything works as it should.
import Component from '@glimmer/component';

class MyComponent extends Component {
  world = 'World';
}

const ProxiedComponent = new Proxy(MyComponent, {});

export default ProxiedComponent;

When using a template tag, the template fails to render, no errors are logged.
import Component from "@glimmer/component";

class MyComponent extends Component {
  world = "World";
  <template>
    Hello {{this.world}}
  </template>
}

const ProxiedComponent = new Proxy(MyComponent, {});

export default ProxiedComponent;

In the generated code for the colocated component the setComponentTemplate is applied to the proxy.
export default setComponentTemplate(TEMPLATE, ProxiedComponent);

This doesn't appear to be the case for a template tag component.
static {
    setComponentTemplate(createTemplateFactory(
    /*

Tests {{this.count}}

          <button type="button" {{on "click" this.handleClick}}>Click me</button>

    */
    {
      "id": "Hg3FL3Xg",
      "block": "[[[1,\"\\n    \"],[10,0],[15,0,[30,0,[\"classList\"]]],[12],[1,\"\\n      \"],[10,\"h1\"],[12],[1,\"Tests \"],[1,[30,0,[\"count\"]]],[13],[1,\"\\n      \"],[11,\"button\"],[24,4,\"button\"],[4,[32,0,[\"on\"]],[\"click\",[30,0,[\"handleClick\"]]],null],[12],[1,\"Click me\"],[13],[1,\"\\n    \"],[13],[1,\"\\n  \"]],[],false,[]]",
      "moduleName": "/Users/liam/Work/ember-vite/my-fancy-app/app/components/my-component.gjs",
      "scope": () => [template__imports__],
      "isStrictMode": true
    }), this);
  }

Reproduction repo https://github.com/evoactivity/ember-proxy-template-tag/

Ah i see, that's a reasonable low level thing to want these for.
So, i think we may just need to tweak the object returned from proxy so that the prototype is the same as the class.
And then, if you're storing state in the proxy, i need to think on that for a bit

State has to be stored externally to the module itself, since HMR will rebuild the whole thing and reload it it will all get thrown away if stored locally.
You can see my current implementation here, it's just defined in the component file whilst I poke around to see what works, before I attempt to do anything babel/vite transform related.


Add middleware layer

Is there any plans for adding middleware layer similar to what we have in AoG? It will allow us to add more functionality to the "agent". If nobody is working on this, I can make the changes and send a pull request.

I've submitted pull request #172


"amac" is damaged and ca't be opened

When I try to open it, it says is damaged and recommends me to move it to trash.

Macbook:

@gabbanaesteban
Solution: https://www.macbed.com/macos-high-sierra-gatekeeper/

I'm having the same issue as @gabbanaesteban
The solution offered by you, @jaywcjlove is no longer available apparently... I'm also on 10.14.4

Ok, I managed to fix the error by launching in the console :
xattr -cr /Applications/amac.app

It's working now :+1:

I found the solution in this article

@cristianciofu 👍


Create instructions for Git + Python on Windows

Our buildscripts require Python and Git to work together. Setting up these on Windows can be tricky, especially regarding how / where Python is added to the $PATH environment variable, is / isn't installed for all users, and so on.
We should perhaps find one or two simple, standardized use cases that we document in a step-by-step tutorial.

@wecsam, given your Windows shell skills, do you want to take a stab at this issue?

@aaaaalbert Sure, I'll take a look. Where are the existing buildscripts?

Cool! The build scripts are here, https://github.com/SeattleTestbed/buildscripts/ . The build instructions are here, https://github.com/SeattleTestbed/docs/blob/master/Contributing/BuildInstructions.md .
There are three issues as far as I can see:

Python on Windows needs some extra care so that it sets up all PATHs correctly and can be started from the command line. Otherwise, we cannot run Repy.
Python must also run from within git-bash (which seems to be the most popular way to have a Git CLI on Windows). Otherwise it's a pain to check out all dependencies and build Seattle components for later use (e.g. repy_v2 and seash).
Related to this, the build scripts must be able to call git. On Unixoid systems, we use Popen, but again there seem to be path issues on Windows.

So we will need Windows-aware installation instructions that make the components work together, and perhaps modifications to the buildscripts too.

Unlike writing a command to loop through some files, that's not something that I'll be able to do in two minutes. I'll revisit this after final exams.

Thank you, much appreciated!


Refont of idendity card on my identities page

Description 📹
Current one :
https://app.starknet.id/identities/855076439471

New design :

Figma file : https://www.figma.com/design/S1UKYgWewNqNHZFAaBUilG/🏝-Starknet-ID?node-id=4108-40951&t=wEy6O4uIMpUixad7-1
Refont the domain idendity card : new color, new size content, text colors...etc. You can reuse the card of this issue : https://github.com/lfglabs-dev/app.starknet.id/issues/940
Proposed Actions 🛠
Here’s a checklist of actions to follow for resolving this issue:

Fork and Create Branch:
Fork the repository and create a new branch using the issue number:
git checkout -b fix-[issue-number]

Implement Changes:
[Insert Code snippet if needed with a mardown todo list]

Run Tests and Commit Changes:
Make sure your changes don't break existing functionality and commit with a clear message:
git commit -m "Fix: [Short description of the fix]"

Required 📋
To keep our workflow smooth, please make sure you follow these guidelines:

Assignment: Don't create a pull request if you weren’t assigned to this issue.
Timeframe: Complete the task within 3 business days.
Closing the Issue: In your PR description, close the issue by writing Close #[issue_id].
Review Process:

Once you've submitted your PR, change the label to "ready for review".
If changes are requested, address them and then update the label back to "ready for review" once done.

Testing: Test your PR locally before pushing, and verify that tests and build are working after pushing.

Thank you for your contribution 🙏
⚠ WARNING: Failure to follow the requirements above may result in being added to the OnlyDust blacklist, affecting your ability to receive future rewards.

Can I handle this task?

Can I work on this issue? @Kevils
I am a fullstack and blockchain developer with experience for more than 1.5 years and an active contributor to starkent
My profile: https://app.onlydust.com/u/SudiptaPaul-31
ETA-1Day

I'd be happy to do this.

Mind if I try this one?  I’m a full-stack blockchain developer with over 3 years of experience working with Tailwind, React, TypeScript, and Solidity/Rust.

Would love to tackle this!


[Text Analytics] Add support for AdditionalProperties to TextDocumentBatchStatistics

Reference: https://github.com/Azure/azure-rest-api-specs/blob/main/dev/cognitiveservices/data-plane/Language/analyzetext.json

Closing now that I finally received confirmation from the service team that the "additionalProperties" set to true in the definition of RequestStatistics in the swagger spec is unintentional.


panic when using pre-trained model

What happened?
When running the kepler in K8S with the pretrained model to estimate the process power, kepler pod just go panics after launch.
The models are trained by following kepler model server tekton training process, using the complete run.
Kepler container goes into error just after it started:
<omit>
I0529 02:05:36.588422  690634 exporter.go:175] starting to listen on <IP_ADDRESS>:9102
I0529 02:05:36.588445  690634 exporter.go:181] Started Kepler in 2.243991957s
I0529 02:05:39.594488  690634 exporter.go:457] successfully get data with batch get and delete with 700 pids in 3.298332ms
I0529 02:05:39.914526  690634 estimate.go:139] estimator unmarshal error: json: cannot unmarshal array into Go struct field ComponentPowerResponse.powers of type map[string][]float64 ({"powers": [], "msg": "\"None of [Index(['bpf_cpu_time_us'], dtype='object')] are in the [columns]\"\n"})
I0529 02:05:39.914657  690634 process_energy.go:210] Could not estimate the Process Platform Power
panic: runtime error: index out of range [0] with length 0

goroutine 33 [running]:
github.com/sustainable-computing-io/kepler/pkg/model.addEstimatedEnergy({0xc000746400, 0x3d, 0xc00075c820?}, 0x0?, 0x1)
        /workspace/pkg/model/process_energy.go:219 +0xbf0
github.com/sustainable-computing-io/kepler/pkg/model.UpdateProcessEnergy(0xc0005d4000?, 0xc000b88660?)
        /workspace/pkg/model/process_energy.go:145 +0x145
github.com/sustainable-computing-io/kepler/pkg/collector/energy.UpdateProcessEnergy(...)
        /workspace/pkg/collector/energy/process_energy_collector.go:26
github.com/sustainable-computing-io/kepler/pkg/collector.(*Collector).UpdateProcessEnergyUtilizationMetrics(...)
        /workspace/pkg/collector/metric_collector.go:152
github.com/sustainable-computing-io/kepler/pkg/collector.(*Collector).UpdateEnergyUtilizationMetrics(0xc0005d4000)
        /workspace/pkg/collector/metric_collector.go:139 +0x2a
github.com/sustainable-computing-io/kepler/pkg/collector.(*Collector).Update(0xb2d05e00?)
        /workspace/pkg/collector/metric_collector.go:113 +0x65
github.com/sustainable-computing-io/kepler/pkg/manager.(*CollectorManager).Start.func1()
        /workspace/pkg/manager/manager.go:75 +0x7b
created by github.com/sustainable-computing-io/kepler/pkg/manager.(*CollectorManager).Start in goroutine 1
        /workspace/pkg/manager/manager.go:67 +0x65

There are some errors in kepler-estimator container too:
<omit>
failed to get model from request {"metrics":["bpf_page_cache_hit","task_clock_ms","bpf_cpu_time_ms","bpf_net_tx_irq","bpf_net_rx_irq","bpf_block_irq","cpu_cycles","cpu_instructions","cache_miss"],"values":[[0,0,0,0,0,0,0,0,0]],"output_type":"DynPower","source":"acpi","system_features":["cpu_architecture"],"system_values":["Skylake"],"trainer_name":"GradientBoostingRegressorTrainer","filter":""}
get archived model
failed to get model from request {"metrics":["bpf_page_cache_hit","task_clock_ms","bpf_cpu_time_ms","bpf_net_tx_irq","bpf_net_rx_irq","bpf_block_irq","cpu_cycles","cpu_instructions","cache_miss"],"values":[[0,0,0,0,0,0,0,0,0]],"output_type":"DynPower","source":"intel_rapl","system_features":["cpu_architecture"],"system_values":["Skylake"],"trainer_name":"GradientBoostingRegressorTrainer","filter":""}
get archived model
failed to get model from request {"metrics":["bpf_page_cache_hit","task_clock_ms","bpf_cpu_time_ms","bpf_net_tx_irq","bpf_net_rx_irq","bpf_block_irq","cpu_cycles","cpu_instructions","cache_miss"],"values":[[0,0,0,0,0,0,0,0,0],[0,0,0,0,0,0,0,0,0],[0,0,0,0,0,0,0,0,0],[0,0,0,0,0,0,0,0,0],[0,0,0,0,0,0,0,0,0],[0,0,0,0,0,0,0,0,0],[0,0,0,0,0,0,0,0,0],[0,0,0,0,0,0,0,0,0],[0,0,0,0,0,0,0,0,0],[0,0,0,0,0,0,0,0,0],[0,0,0,0,0,0,0,0,0],[0,0,0,0,0,0,0,0,0],[0,0,0,0,0,0,0,0,0],[0,0,0,0,0,0,0,0,0],[0,0,0,0,0,0,0,0,0],[0,0,0,0,0,0,0,0,0],[0,0,0,0,0,0,0,0,0],[0,0,0,0,0,0,0,0,0],[0,0,0,0,0,0,0,0,0],[0,0,0,0,0,0,0,0,0],[0,0,0,0,0,0,0,0,0],[0,0,0,0,0,0,0,0,0],[0,0,0,0,0,0,0,0,0],[0,0,0,0,0,0,0,0,0],[0,0,0,0,0,0,0,0,0],[0,0,0,0,0,0,0,0,0],[0,0,0,0,0,0,0,0,0],[0,0,0,0,0,0,0,0,0],[0,0,0,0,0,0,0,0,0],[0,0,0,0,0,0,0,0,0],[0,0,0,0,0,0,0,0,0],[0,0,0,0,0,0,0,0,0],[0,0,0,0,0,0,0,0,0],[0,0,0,0,0,0,0,0,0],[0,0,0,0,0,0,0,0,0],[0,0,0,0,0,0,0,0,0],[0,0,0,0,0,0,0,0,0],[0,0,0,0,0,0,0,0,0],[0,0,0,0,0,0,0,0,0],[0,0,0,0,0,0,0,0,0],[0,0,0,0,0,0,0,0,0],[0,0,0,0,0,0,0,0,0],[0,0,0,0,0,0,0,0,0],[0,0,0,0,0,0,0,0,0],[0,0,0,0,0,0,0,0,0],[0,0,0,0,0,0,0,0,0],[0,0,0,0,0,0,0,0,0],[0,0,0,0,0,0,0,0,0],[0,0,0,0,0,0,0,0,0],[0,0,0,0,0,0,0,0,0],[0,0,0,0,0,0,0,0,0],[0,0,0,0,0,0,0,0,0],[0,0,0,0,0,0,0,0,0],[0,0,0,0,0,0,0,0,0],[0,0,0,0,0,0,0,0,0],[0,0,0,0,0,0,0,0,0],[0,0,0,0,0,0,0,0,0],[0,0,0,0,0,0,0,0,0],[0,0,0,0,0,0,0,0,0],[0,0,0,0,0,0,0,0,0],[0,0,0,0,0,0,0,0,0],[0,0,0,0,0,0,0,0,0]],"output_type":"DynPower","source":"intel_rapl","system_features":["cpu_architecture"],"system_values":["Skylake"],"trainer_name":"GradientBoostingRegressorTrainer","filter":""}
GradientBoostingRegressorTrainer_1 fail to predict, removed: "None of [Index(['bpf_cpu_time_us'], dtype='object')] are in the [columns]"
<omit>

The complete kepler log can be found here :
kepler.log
The complete kepler-estimator log can be found here:
kepler-estimator.log
What did you expect to happen?
Kepler should be run without any panics
How can we reproduce it (as minimally and precisely as possible)?
run kepler with the kepler deployment configurations below.
Anything else we need to know?
No response
Kepler image tag

kepler: quay.io/sustainable_computing_io/kepler:latest
estimator: quay.io/sustainable_computing_io/kepler_model_server:latest

Kubernetes version

$ kubectl version
Client Version: v1.29.3
Kustomize Version: v5.0.4-0.20230601165947-6ce0bf390ce3
Server Version: v1.29.2

Cloud provider or bare metal

bare metal

OS version

$ cat /etc/os-release
PRETTY_NAME="Ubuntu 22.04.4 LTS"
NAME="Ubuntu"
VERSION_ID="22.04"
VERSION="22.04.4 LTS (Jammy Jellyfish)"
VERSION_CODENAME=jammy
ID=ubuntu
ID_LIKE=debian
HOME_URL="https://www.ubuntu.com/"
SUPPORT_URL="https://help.ubuntu.com/"
BUG_REPORT_URL="https://bugs.launchpad.net/ubuntu/"
PRIVACY_POLICY_URL="https://www.ubuntu.com/legal/terms-and-policies/privacy-policy"
UBUNTU_CODENAME=jammy
$ uname -a
Linux onap02 5.15.0-105-generic #115-Ubuntu SMP Mon Apr 15 09:52:04 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux

Install tools

Using manifest

Kepler deployment config

For on kubernetes:
$ KEPLER_NAMESPACE=kepler

# provide kepler configmap
$ kubectl describe configmap kepler-cfm -n ${KEPLER_NAMESPACE}
Name:         kepler-cfm
Namespace:    kepler
Labels:       sustainable-computing.io/app=kepler
Annotations:  <none>

Data
====
CGROUP_METRICS:
----
*
EXPOSE_IRQ_COUNTER_METRICS:
----
true
KEPLER_LOG_LEVEL:
----
5
KEPLER_NAMESPACE:
----
kepler
MODEL_CONFIG:
----
PROCESS_COMPONENTS_ESTIMATOR=true
PROCESS_COMPONENTS_INIT_URL=http://onap01.sh.intel.com/kepler_models/CompleteTrainPipelineExample/intel_rapl/DynPower/Basic/GradientBoostingRegressorTrainer_0.zip
PROCESS_COMPONENTS_TRAINER=GradientBoostingRegressorTrainer
PROCESS_TOTAL_ESTIMATOR=true
PROCESS_TOTAL_INIT_URL=http://onap01.sh.intel.com/kepler_models/CompleteTrainPipelineExample/acpi/DynPower/Basic/GradientBoostingRegressorTrainer_0.zip
PROCESS_TOTAL_TRAINER=GradientBoostingRegressorTrainer

PROMETHEUS_SCRAPE_INTERVAL:
----
30s
CPU_ARCH_OVERRIDE:
----

ENABLE_GPU:
----
true
ENABLE_QAT:
----
false
EXPOSE_CGROUP_METRICS:
----
false
EXPOSE_HW_COUNTER_METRICS:
----
true
BIND_ADDRESS:
----
<IP_ADDRESS>:9102
ENABLE_PROCESS_METRICS:
----
false
MAX_LOOKUP_RETRY:
----
1
REDFISH_PROBE_INTERVAL_IN_SECONDS:
----
60
ENABLE_EBPF_CGROUPID:
----
true
METRIC_PATH:
----
/metrics
REDFISH_SKIP_SSL_VERIFY:
----
true

BinaryData
====

Events:  <none>

# provide kepler deployment description
$ kubectl describe daemonset kepler-exporter -n ${KEPLER_NAMESPACE}
Name:           kepler-exporter
Selector:       app.kubernetes.io/component=exporter,app.kubernetes.io/name=kepler-exporter,sustainable-computing.io/app=kepler
Node-Selector:  <none>
Labels:         sustainable-computing.io/app=kepler
Annotations:    deprecated.daemonset.template.generation: 1
Desired Number of Nodes Scheduled: 1
Current Number of Nodes Scheduled: 1
Number of Nodes Scheduled with Up-to-date Pods: 1
Number of Nodes Scheduled with Available Pods: 0
Number of Nodes Misscheduled: 0
Pods Status:  1 Running / 0 Waiting / 0 Succeeded / 0 Failed
Pod Template:
  Labels:           app.kubernetes.io/component=exporter
                    app.kubernetes.io/name=kepler-exporter
                    sustainable-computing.io/app=kepler
  Service Account:  kepler-sa
  Containers:
   kepler-exporter:
    Image:      quay.io/sustainable_computing_io/kepler:latest
    Port:       9102/TCP
    Host Port:  0/TCP
    Command:
      /bin/sh
      -c
    Args:
      until [ -e /tmp/estimator.sock ]; do sleep 1; done && /usr/bin/kepler -v=5
    Requests:
      cpu:     100m
      memory:  400Mi
    Liveness:  http-get http://:9102/healthz delay=10s timeout=10s period=60s #success=1 #failure=5
    Environment:
      NODE_IP:     (v1:status.hostIP)
      NODE_NAME:   (v1:spec.nodeName)
    Mounts:
      /etc/kepler/kepler.config from cfm (ro)
      /etc/redfish from redfish (ro)
      /lib/modules from lib-modules (ro)
      /proc from proc (rw)
      /sys from tracing (ro)
      /tmp from tmp (rw)
      /usr/src from usr-src (rw)
      /var/run from var-run (rw)
   estimator:
    Image:      quay.io/sustainable_computing_io/kepler_model_server:latest
    Port:       <none>
    Host Port:  <none>
    Command:
      python3.8
    Args:
      -u
      src/estimate/estimator.py
    Environment:  <none>
    Mounts:
      /etc/kepler/kepler.config from cfm (ro)
      /mnt from mnt (rw)
      /tmp from tmp (rw)
  Volumes:
   tmp:
    Type:       EmptyDir (a temporary directory that shares a pod's lifetime)
    Medium:
    SizeLimit:  <unset>
   mnt:
    Type:       EmptyDir (a temporary directory that shares a pod's lifetime)
    Medium:
    SizeLimit:  <unset>
   proc:
    Type:          HostPath (bare host directory volume)
    Path:          /proc-host
    HostPathType:  Directory
   usr-src:
    Type:          HostPath (bare host directory volume)
    Path:          /usr/src
    HostPathType:  Directory
   lib-modules:
    Type:          HostPath (bare host directory volume)
    Path:          /lib/modules
    HostPathType:  Directory
   tracing:
    Type:          HostPath (bare host directory volume)
    Path:          /sys
    HostPathType:  Directory
   var-run:
    Type:          HostPath (bare host directory volume)
    Path:          /var/run
    HostPathType:  Directory
   cfm:
    Type:      ConfigMap (a volume populated by a ConfigMap)
    Name:      kepler-cfm
    Optional:  false
   redfish:
    Type:        Secret (a volume populated by a Secret)
    SecretName:  redfish-4kh9d7bc7m
    Optional:    false
Events:
  Type    Reason            Age   From                  Message
  ----    ------            ----  ----                  -------
  Normal  SuccessfulCreate  29m   daemonset-controller  Created pod: kepler-exporter-9wxsp

Container runtime (CRI) and version (if applicable)

containerd://1.7.13

Related plugins (CNI, CSI, ...) and versions (if applicable)

CNI: kindnet

It seems the trained power model using the CPU time metric exported by Kepler before v0.7 (bpf_cpu_time_us); however, the estimation is called by the new Kepler (with bpf_cpu_time_ms). You may have to retrain the power model with new Kepler version.
I0529 02:05:34.744887  690634 utils.go:86] Available ebpf counters: [bpf_page_cache_hit task_clock_ms bpf_cpu_time_ms bpf_net_tx_irq bpf_net_rx_irq bpf_block_irq cpu_cycles cpu_instructions cache_miss]
...
I0529 02:05:39.914526  690634 estimate.go:139] estimator unmarshal error: json: cannot unmarshal array into Go struct field ComponentPowerResponse.powers of type map[string][]float64 ({"powers": [], "msg": "\"None of [Index(['bpf_cpu_time_us'], dtype='object')] are in the [columns]\"\n"})


Roadmap: V3 Prerelease 2

Following Issues will have to be completed before publishing v3 prerelease 1:

#60
#111

Afterwards the Prerelease 2 will be tested and potentially receive further bugfixes or minor features before being properly released as v3.0.
Note: V3 will drop support for 32-Bit. This is done due to A: 32-Bit being outdated and B: Because slight issues with nvapi can arrise when running a 32-bit version on a 64-bit OS.

All Issues for Prerelease 2 have been Implemented. Building both prereleases (full and micro) now.


[#7] Update SDK to v1.0.0-rc.4

There are some problem:

build for platform other than linux/amd64 (e.g. "linux/arm/v6", "linux/arm/v7", "linux/arm64","linux/ppc64le","linux/s390x")
missing vendor folder
lint issue in upstream (using [string](https://github.com/nspcc-dev/distribution/blob/7bbddbf67d0c2609c0f1f18234505d79c9b660e9/registry/storage/linkedblobstore.go#L309) as key in context value)

close #7
Signed-off-by: Denis Kirillov<EMAIL_ADDRESS>

Related to https://github.com/nspcc-dev/tzhash/issues/32


Mock server causing client to panic

I tried running the mock server
[a-dot ~/repos/ch-go/internal/cmd/ch-bench-server (main)]$ go build
[a-dot ~/repos/ch-go/internal/cmd/ch-bench-server (main)]$ ./ch-bench-server
starting with chunk of 1.6 MB

And then tried running the ch-write-bench-faster benchmark. (I had to add one line to specify port 9001 instead of the default 9000):
[a-dot ~/repos/ch-bench/ch-write-bench-faster (main)]$ go build
[a-dot ~/repos/ch-bench/ch-write-bench-faster (main)]$ ./ch-write-bench-faster
Error: decode block:
    github.com/ClickHouse/ch-go.(*Client).Do.func4
        /home/a-dot/go/pkg/mod/github.com/!click!house/ch-go@v0.44.1/query.go:640
  - decode block:
    github.com/ClickHouse/ch-go.(*Client).decodeBlock
        /home/a-dot/go/pkg/mod/github.com/!click!house/ch-go@v0.44.1/query.go:219
  - info:
    github.com/ClickHouse/ch-go/proto.(*Block).DecodeBlock
        /home/a-dot/go/pkg/mod/github.com/!click!house/ch-go@v0.44.1/proto/block.go:254
  - field id:
    github.com/ClickHouse/ch-go/proto.(*BlockInfo).Decode
        /home/a-dot/go/pkg/mod/github.com/!click!house/ch-go@v0.44.1/proto/block.go:44
  - read:
    github.com/ClickHouse/ch-go/proto.(*Reader).UVarInt
        /home/a-dot/go/pkg/mod/github.com/!click!house/ch-go@v0.44.1/proto/reader.go:92
  - read:
    github.com/ClickHouse/ch-go/proto.(*Reader).ReadFull
        /home/a-dot/go/pkg/mod/github.com/!click!house/ch-go@v0.44.1/proto/reader.go:66
  - read next block:
    github.com/ClickHouse/ch-go/internal/compress.(*Reader).Read
        /home/a-dot/go/pkg/mod/github.com/!click!house/ch-go@v0.44.1/internal/compress/reader.go:117
  - data size should be 0 < 913600073 < 134217728:
    github.com/ClickHouse/ch-go/internal/compress.(*Reader).readBlock
        /home/a-dot/go/pkg/mod/github.com/!click!house/ch-go@v0.44.1/internal/compress/reader.go:46

But then I get this panic. How do I use the mock server?

I took a closer look at the mock server and realized it's not really a mock server.


Systemparser conversion bugs and lifetime criticality

List of changes:

Added __pycache__  directory to gitignore
Fixed conversion from int16_t to uint16_t for VID and PID
Changed the return value from  const char*  to char* with deep copy
Updated constructor list of utils

So far only the ESP32-S2 Basic example is updated.

Great idea, looks very clean.


matrix question added in Amazon

Given a square matrix, turn it by 90 degrees in an anti-clockwise direction without using any extra space.

Please refer to the contributing guidelines.


I am using html to write html5, please tell me how to use freetype

Issue details
I am using html to write html5, please tell me how to use freetype
Reproduction steps/code
I need to do chat with gdx, I can't know the text in advance, so I have to use the Chinese font library.
Version of LibGDX and/or relevant dependencies
1.9.8
Stacktrace
//Please provide the stacktrace if applicable

Please select the affected platforms

[x] HTML/GWT

use the forums/irc/discord for help

A wild lestat

Freetype is not supported on gwt.


Nest database support

Create table for nests
Load nest fences from table

Unfinished

Current status:
nest branch - golbat creates the nest table, and will read nests from it
It collects and categorises the pokemon; however this really needs to be in hour long buckets (perhaps using time.now() % 3600 as the key) dropping off old ones so that we have multi-hours worth of data which expire - as sample sizes are too small currently.
It needs to pick the pokemon with the highest %age; but not if the pokemon is > 50% spawns in that hour [this would indicate spotlight or community day]
then it just needs to write it to the db in a batch rather than logging it

I will not be working on this branch any further


HBASE-23556: Minor ChoreService Cleanup

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
3m 51s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

-0 :warning:
test4tests
0m  0s
The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ master Compile Tests _

+1 :green_heart:
mvninstall
6m  2s
master passed

+1 :green_heart:
compile
0m 22s
master passed

+1 :green_heart:
checkstyle
0m 28s
master passed

+1 :green_heart:
shadedjars
5m  5s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 21s
master passed

+0 :ok:
spotbugs
0m 51s
Used deprecated FindBugs config; considering switching to SpotBugs.

+1 :green_heart:
findbugs
0m 49s
master passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
5m 30s
the patch passed

+1 :green_heart:
compile
0m 23s
the patch passed

+1 :green_heart:
javac
0m 23s
the patch passed

+1 :green_heart:
checkstyle
0m 26s
hbase-common: The patch generated 0 new + 3 unchanged - 1 fixed = 3 total (was 4)

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
shadedjars
5m  1s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
hadoopcheck
17m 19s
Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.

+1 :green_heart:
javadoc
0m 20s
the patch passed

+1 :green_heart:
findbugs
0m 55s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
2m 57s
hbase-common in the patch passed.

+1 :green_heart:
asflicense
0m 12s
The patch does not generate ASF License warnings.

56m 51s

Subsystem
Report/Notes

Docker
Client=19.03.5 Server=19.03.5 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-927/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/927

JIRA Issue
HBASE-23556

Optional Tests
dupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile

uname
Linux 9a7c84929202 4.15.0-66-generic #75-Ubuntu SMP Tue Oct 1 05:24:09 UTC 2019 x86_64 GNU/Linux

Build tool
maven

Personality
/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-927/out/precommit/personality/provided.sh

git revision
master / a580b1d2e9

Default Java
1.8.0_181

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-927/1/testReport/

Max. process+thread count
259 (vs. ulimit of 10000)

modules
C: hbase-common U: hbase-common

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-927/1/console

versions
git=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

In the future can we leave some more time on these kind of refactoring reviews? especially when it involves changes that can impact correctness (removing synchronization) or gc (removing proactive nulling out of references)?


Add PATCH support for upload on Android

Hi! Thanks for the great library.
This PR enables support for PATCH requests on Android.
I submitted the PR on the 0.9.7 branch since this feature already works perfectly on iOS, which caused us some confusion as our code didn't seem wrong but didn't work on Android...
Let me know what you think! Maybe I missed something.

Now that I take a look at the iOS code it looks like it is the same there (https://github.com/wkh237/react-native-fetch-blob/blob/b98143d5d888f49abad43d50990c980ac3a7b4f7/src/ios/RNFetchBlobReqBuilder.m#L90) so I'm not sure why it works on iOS?

Thanks for the PR, I'll add some test case for insure IOS and Android both working properly for PATCH requests : )

Thanks!


Three level derived identity (against master)

A test case and some work towards a solution for https://hibernate.atlassian.net/browse/HHH-5764
See related pull request #751

I rebased against master and removed my non-test commits. I'll later take a crack at seeing if my 4.x changes will work against 5.x now. (Last year, they didn't.)

So, looks like my changes for 4.x do indeed work against 5.x. Sweet.
I've added them here (after fixing conflicts).
(There is one failing test -- GetAndIsVariantGetterTest.testAnnotations, but I'm pretty sure it's not related to my changes)

Awesome, thanks Matt!  I will make this the last one I look at before I finally get some sleep tonight :)
BTW, GetAndIsVariantGetterTest passes currently on master and 5.0 branches.  So the problem would certainly seem to be in your changes.  I'll take a look when I pull this in unless you get to it first

There are some really good tests in nestedcompositekeystest.zip attached to HHH-5764 using entities with @EmbeddedId and @IdClass that have fairly complex multi-level associations. When I ran them using Matt's pull request for 4.3, they all succeeded. I've moved them into hibernate-core unit tests and I'm ready to push once Matt has everything sorted out on master. Thanks Matt!

@sebersole GetAndIsVariantGetterTest.testAnnotations() fails for me on 5.0 and master (even after a clean build). Not sure why. So I don't think that test failure is due to my changes.
I'll try to get the other changes done here in the next day or two.

I'd like to understand the reason for that (maybe it is indicative of some
real platform/system specific bug).  Can you send me/us your JVM, OS, etc
info?
On Fri, Oct 30, 2015 at 12:45 PM Matt Drees<EMAIL_ADDRESS>wrote:

@sebersole https://github.com/sebersole
GetAndIsVariantGetterTest.testAnnotations() fails for me on 5.0 and
master (even after a clean build). Not sure why. So I don't think that test
failure is due to my changes.
I'll try to get the other changes done here in the next day or two.
—
Reply to this email directly or view it on GitHub
https://github.com/hibernate/hibernate-orm/pull/752#issuecomment-152598971
.

from groovy --version:
Groovy:       2.3.10
Ant:          Apache Ant(TM) version 1.9.3 compiled on December 23 2013
JVM:          1.8.0_60 (Oracle Corporation 25.60-b23)
OS:           Mac OS X 10.10.5 x86_64
more details:
https://gist.github.com/mattdrees/0eb3b9243fa50a4b381b

created a jira for you to track this: https://hibernate.atlassian.net/browse/HHH-10242

I added a commit that removes the abstraction.
However, I noticed that there's another bit of code in InFlightMetadataCollectorImpl that does a topological sort, as well: processFkSecondPassesInOrder(). I tried altering it to use the DependentSecondPass mechanism, and it seemed to work out well.  At the very least the tests pass. My work is located here: https://github.com/mattdrees/hibernate-orm/commit/5bbb7f7f37395d49481b10342e1cd4379e3c97b9. I'm guessing my sorting method is slower than the one I removed, but I think it's easier to understand.
Do you want to go with removing the abstraction? Or expanding its use?

Personally I am all for readability as long as the perf tradeoff is not
significant
On Mon, Nov 2, 2015, 5:46 PM Matt Drees<EMAIL_ADDRESS>wrote:

I added a commit that removes the abstraction.
However, I noticed that there's another bit of code in
InFlightMetadataCollectorImpl that does a topological sort, as well:
processFkSecondPassesInOrder(). I tried altering it to use the
DependentSecondPass mechanism, and it seemed to work out well. At the very
least the tests pass. My work is located here: mattdrees@5bbb7f7
https://github.com/mattdrees/hibernate-orm/commit/5bbb7f7f37395d49481b10342e1cd4379e3c97b9.
I'm guessing my sorting method is slower than the one I removed, but I
think it's easier to understand.
Do you want to go with removing the abstraction? Or expanding its use?
—
Reply to this email directly or view it on GitHub
https://github.com/hibernate/hibernate-orm/pull/752#issuecomment-153195080
.

do you have a test somewhere that has hundreds (or thousands) of entities & relationships? I don't know of a good way to test the performance tradeoff.

You had mentioned the performance angle ;)
If we are unsure, then for now let's just keep that algorithm for CopyIdentifierComponentSecondPass processing.  We can look at processFkSecondPassesInOrder later

Well, I really doubt it's a problem, but I just don't have a good way to verify that.
So yeah, let's just look at processFkSecondPassesInOrder later.
Anything else that you (or @gbadner ) want me to do here? squashing or whatnot?

Looks good to me.  Still waiting to hear from Gail wrt the question I asked
her on the Jira.  If she does not get to it or does not answer this week
I'll just apply it.
On Wed, Nov 4, 2015 at 6:35 PM Matt Drees<EMAIL_ADDRESS>wrote:

Well, I really doubt it's a problem, but I just don't have a good way to
verify that.
So yeah, let's just look at processFkSecondPassesInOrder later.
Anything else that you (or @gbadner https://github.com/gbadner ) want
me to do here? squashing or whatnot?
—
Reply to this email directly or view it on GitHub
https://github.com/hibernate/hibernate-orm/pull/752#issuecomment-153915747
.

Applied upstream.  Thanks Matt!

 And thanks for applying this!


Fedora: Some tests fail for s390x arch

Since we got the arm test issues squared away I wonder if we could also see what's going on with s390x.
Because there's so much output I won't paste it all here but you can open the build log and search for "%check" to jump to where the tests are run.
https://kojipkgs.fedoraproject.org//work/tasks/910/76780910/build.log

Here's the first relevant failure in the log:
23/111 Test  #25: OpenEXRCore.testReadDeep ..................Subprocess aborted***Exception:   0.14 sec
tempDir = '/var/tmp/OpenEXR_k0Rol9/': 24
=======
Running testReadDeep
  generating deep scanline file '/var/tmp/OpenEXR_k0Rol9/randomtempdeep.exr' compression 0
  --> done
Core Test failed: packed.size() == (sampcount[N-1]) * bps
           file:/builddir/build/BUILD/openexr-3.1.2/src/test/OpenEXRCoreTest/deep.cpp
           line:456
       function:void testReadDeep(const string&)

I haven't got time to dive into this, but is this a historical preservation effort? Or am I misunderstanding that s390x is the last of the old 360 mainframes? If this is historic iron, I'm just wondering if some 32bit/64 bit detection code fell off a cliff and is trying to build 64 bit OpenEXR on a 32 bit machine (or vice versa, was there ever a 64 bit s390x?)

Thanks for that explanation! I had no idea :)
One more question, did OpenEXR work on s390x until recently, and then broke, or has it always failed?

For posterity I tried building 3.1.5 with s390x to see if anything had changed. With the caveat that I used qemu emulation as the builders are down for maintenance, the following tests fail:
The following tests FAILED:
	 25 - OpenEXRCore.testReadDeep (Subprocess aborted)
	 60 - OpenEXR.testCompression (Subprocess aborted)
	 93 - OpenEXR.testRgba (Subprocess aborted)
	 94 - OpenEXR.testRgbaThreading (Subprocess aborted)
	 96 - OpenEXR.testSampleImages (Subprocess aborted)
	 98 - OpenEXR.testSharedFrameBuffer (Subprocess aborted)
	103 - OpenEXR.testTiledRgba (Subprocess aborted)

Build logs can be provided if needed.

Thanks for the update, it appears these are the same issues as in the OP. There will be a point release soon, perhaps it makes sense to re-run the tests on top of tree instead of 3.1.5, or wait for the point release, but bearing in mind that to my knowledge nothing has changed with respect to support for big-endian systems. After testing a new release, addressing the remaining issues might require s390x community involvement ~ PRs would certainly be welcome.

@sharkcz, you did set me up with remote access to a s390 system, but I never succeeded in getting any time to further investigate. I did get connected to the machine (back in 2021), but didn't spend enough time with it to narrow the problem. I'd love to find someone with more bandwidth to investigate further.
@hobbes1069 , can you share the build logs for the failures? Similarly, I'm not sure they'll be all that informative, but I'd at least like to see what they say.

I've kicked off a build just now, you can use the link for the build.log once it's complete.
https://koji.fedoraproject.org/koji/taskinfo?taskID=97575052
I also asked for some assistance on the Fedora developer mailing list. Hopefully they'll have some time to poke around.

WIth 3.1.9, the same tests fail on s390x plus those mentioned in #1460.


Added accuracy graphs for datacenter - open

deployed: https://anandhu-eng.github.io/inference_results_v4.0/

Closing this as this code is now in this repository.


use keydown events instead of keypress events.

Add space character to keydown class.

This addresses issue #29 and issue #27


Реализация обратной связи

Codecov Report

Merging #46 (9b4fa92) into main (5c90f0b) will decrease coverage by 0.09%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main      #46      +/-   ##
==========================================
- Coverage   98.91%   98.82%   -0.10%
==========================================
  Files          35       42       +7
  Lines         832      935     +103
==========================================
+ Hits          823      924     +101
- Misses          9       11       +2

Impacted Files
Coverage Δ

test/cyberdas/utils/hash_type.py
88.63% <0.00%> (-4.55%)
:arrow_down:

test/cyberdas/config.py
100.00% <0.00%> (ø)

test/cyberdas/routes.py
100.00% <0.00%> (ø)

test/cyberdas/exceptions.py
100.00% <0.00%> (ø)

test/cyberdas/models/user.py
100.00% <0.00%> (ø)

test/cyberdas/models/queue.py
100.00% <0.00%> (ø)

test/cyberdas/models/session.py
100.00% <0.00%> (ø)

test/cyberdas/models/__init__.py
100.00% <0.00%> (ø)

test/cyberdas/resources/login.py
100.00% <0.00%> (ø)

test/cyberdas/resources/slots.py
100.00% <0.00%> (ø)

... and 34 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 e33d594...9b4fa92. Read the comment docs.


multiprocessing market files

Could someone provide an example to get around the GIL? I run into "TypeError: cannot pickle 'builtins.File' object" with both joblib and pathos.

nvm it works if I open the files in the subprocess so just use the path as the parameter instead of the file object


Jade Autocompile

Jade file needs to be enabled per file-level using format like the below.
{
"jade.enabled": false,
......
"path": {
"index.jade": {
"jade.enabled": true
},
.......
}

This requires Jade module. If you haven't, install it via "npm install jade."

@hanwiz How can i add autocompile functionality? Do i need to build this plugin with this changes by my self?

At this point, you need to download my repository zip file and add it
manually.
https://github.com/hanwiz/jade-brackets
Also you need to install two plugins using "npm install jade" "npm install
fs-extra".
Han
On Wed, Mar 2, 2016 at 2:22 AM, thecosss<EMAIL_ADDRESS>wrote:

@hanwiz https://github.com/hanwiz How can i add autocompile
functionality? Do i need to build this plugin with this changes by my self?
—
Reply to this email directly or view it on GitHub
https://github.com/ForbesLindesay/jade-brackets/pull/24#issuecomment-191124939
.

@thecosss Here is my compiled file. Alternatively, you can download my repo and compile them. Use the link and install it from your bracket extension manager.
jade.zip

@hanwiz ok, i've installed your zip, but autocompile doesn't work.

@thecosss Did you install jade and fs-extra module? If you haven't, install it via "npm install jade" "npm install fs-extra"

I've installed it to project folder and globaly, nothing works. Console says "[brackets-jade-node] failed to run jade Unable to load one of the modules: C:/Users/Cosss/AppData/Roaming/Brackets/extensions/user/jade/node/JadeDomain, reason: Cannot find module 'jade'"

https://github.com/hanwiz/jade-brackets/blob/master/jade 3.zip
Would you try this and let me know?
On Thu, Mar 10, 2016 at 1:45 AM, thecosss<EMAIL_ADDRESS>wrote:

I've installed it to project folder and globaly, nothing works. Console
says "[brackets-jade-node] failed to run jade Unable to load one of the
modules:
C:/Users/Cosss/AppData/Roaming/Brackets/extensions/user/jade/node/JadeDomain,
reason: Cannot find module 'jade'"
—
Reply to this email directly or view it on GitHub
https://github.com/ForbesLindesay/jade-brackets/pull/24#issuecomment-194719413
.

@hanwiz Yes this time it works! Thank you very much.
Maybe you'll be interested in making preprocessor out of Twig? I don't really like Jade syntax...


Report the correct WIN size

Report the correct WIN size
Description
A FreeRTOS+TCP user Wolfgang Schur, reported that under some rare circumstances, the reported value of WIN ( the available space in the TCP r4ecption window) is not correct.
In fact, it would be a bit behind: the last read from a socket would not be taken into account, so the reported WIN size sometimes wat too small.
The WIN size would be stored in the TCP socket field called ulRxCurWinSize.
In this patch, the field ulRxCurWinSize will be removed, and WIN will be calculated at the moment it must be send to the peer.
Under certain circumstances, this patch will make the communication a bit more efficient.
Checklist:

[x] I have tested my changes. No regression in existing tests.
[x] My code is Linted.

In the previous version, ulOutstanding would become non-zero in case there are bytes missing: the difference between the highest received (ulHighestSequenceNumber) and the highest acknowledged (ulCurrentSequenceNumber).
I believe that was not correct. The reported "Win" (windows size) always begins at "Ack".
I tested the change on both an STM32F and on a Xilinx platform.

More detail about the proposed change, as well as related lab testing:
The problem was that WIN is derived from a variable 'xTCP.ulRxCurWinSize', That variable is only calculated within 'prvSendData()'. The variable is used ( being read ) in 'prvTCPReturnPacket()' in order to report the current WIN size. There are 5 functions that can call 'prvTCPReturnPacket()' :
prvSendData()  ( OK )
xTCPSocketCheck()
prvTCPSendPacket()
prvTCPSendRepeated()
prvTCPPrepareConnect()

When prvTCPReturnPacket() is not called by prvSendData(), the value of 'xTCP.ulRxCurWinSize' may run behind. Lab testing shows that happening form prvTCPSendRepeated().
It is proving difficult to write test code that shows the merits of the patch unconditionally. There is a lot of randomness in the flow of TCP communication: the occupation rate of the network, the quality of the transport layers, speed of the sender, speed of the receiver. For example, sending 10 MB of data through a LAN to a FreeRTOS+TCP device, it might happen 10 times that the reported WIN size is not running behind. This is causing unnecessary delays.
The other problem with the current code is : the WIN size has to start with 'ulCurrentSequenceNumber' (ACK), and not with 'ulHighestSequenceNumber'. This was not correct:
pxSocket->u.xTCP.ulRxCurWinSize = pxTCPWindow->xSize.ulRxWindowLength -
    ( pxTCPWindow->rx.ulHighestSequenceNumber - pxTCPWindow->rx.ulCurrentSequenceNumber );

The difference only becomes apparent when there is data missing in the reception window, and so it will affect connections that run through a bad connection.

/bot run checks


docs: Adapt links

They seem to have been copy/pasted from PhpMetrics' README. I removed the link to the contributing guide because I could not find an equivalent.

Oh, I see, it was my mistake. Thanks for your help!


unable to implement Individual column searching (select inputs) in my table

hi all,
I am trying to implement "Individual column searching (select inputs)" https://datatables.net/examples/api/multi_filter_select.html but unable to do.
The <tfoot> don't work / make as select list.
Can anyone help me. my php file is attached, I am using codeigniter.
students_list.zip

Hi, Sorry but this is not the place to ask meybe at https://stackoverflow.com/ as this is not related to the template itself.


added time details for student review to exam summary

Checklist

[x] I tested all changes and all related features with different users (student, tutor, instructor, admin) on the test server https://artemistest.ase.in.tum.de.
[x] Client: I followed the coding and design guidelines.
[x] Client: I added multiple integration tests (Jest) related to the features (with a high test coverage)
[x] Client: I documented the TypeScript code using JSDoc style.
[x] Client: I added multiple screenshots/screencasts of my UI changes
[x] Client: I translated all the newly inserted strings into German and English

Motivation and Context

resolves https://github.com/ls1intum/Artemis/issues/1923
In the exam summary students could not see the time when they could review and create complaints. The date/times are shown now, also a tag that shows that the review is open.
Description

Steps for Testing

check the exam summary of a student. When the exam is over (and the instructor set review times) you are able to see the review time, and when you are in the review time you can see an "review is open" tag.
when the exam is not over yet, you should not be able to see the info about the review in the heading.
-->

I don't really like the date format. Why do you have a - in it?
Can we use a standardized date format? I think the artemisDate pipe also support a format with date + time

I changed it, now it only uses one pipe. (see the new screenshot)


🚀 Feature Request: Detect whether code is being run in Cloudflare Workers (or Node.js)

Describe the solution
Add some kind of macro to detect whether the code, that is currently being run, is running in Cloudflare Workers.
For example with esbuild:
return esbuild.build({
  define: {
    IS_CLOUDFLARE_WORKER: 'true'
  }
})

I've been using this for my custom Cloudflare Worker bundling (see github.com/brillout/build-worker/tree/build-worker.mjs#L20-L22).

In the meantime: this https://community.cloudflare.com/t/how-to-detect-the-cloudflare-worker-runtime/293715 works.

I believe the Navigator API is the preferred way to do this
if (navigator.userAgent === 'Cloudflare-Workers') {
  // logic
}

I don't think wrangler would inject any compile time value that would indicate a workers environment. @WalshyDev's suggestion would be like the right way to do this. Closing this issue.

It doesn't seem to work: I get typeof navigator === "undefined".

I just tried this in a worker:
export default {
  async fetch() {) {
    return new Response(JSON.stringify(navigator));
  }
}

and in the browser, I see:
{"userAgent":"Cloudflare-Workers"}

I just tried again using<EMAIL_ADDRESS>and by inserting console.log(typeof navigator) somewhere in the worker code and I still get typeof navigator === "undefined". Maybe it's because I'm using wrangler.toml#node_compat?

For anyone else that comes across this, the navigator useragent method appears to be the Cloudflare recommended way of determining cloudflare worker runtime: https://developers.cloudflare.com/workers/runtime-apis/web-standards/#navigatoruseragent

For future readers, navigator.userAgent requires the global-navigator compatibility flag, or a compatibility date after 2022-03-21: https://developers.cloudflare.com/workers/configuration/compatibility-flags/#global-navigator


Update DefaultAzureCredential Exception Message

Update the java defaultAzureCredential exception message to match the new one that is in the .NET SDK.
https://github.com/Azure/azure-sdk-for-net/pull/9821
https://github.com/Azure/azure-sdk-for-net/pull/9958

/cc @catalinaperalta @schaabs

pr merged


Bug in audio loss computation?

Hi @TaoRuijie,
You have an audio classifcation loss in https://github.com/TaoRuijie/TalkNet-ASD/blob/85b37afc32bedcadd8b5dfbc7f69298a76de279b/talkNet.py#L36, and the labels that you use there are the ASD labels (which basically say if the voice matches the face or not). However, it's not possible to predict just on the basis of audio which face is speaking since there is no concept of speaker classes that's uniform/consistent throughout the dataset. Shouldn't the labels be voice-activity labels instead, i.e. one should first check if for the particular frame any of the visible faces is an active speaker and if that's the case, assign the frames a label of 1 and 0 otherwise?

Yes this setting is same to the baseline paper (AVA dataset paper by Google)
I also tried to remove nlossA and nlossV, the results are similar.


[TEST] test_0_storage_network.py::test_storage_network failing on 1.4

What's the test to develop? Please describe
The test test_0_storage_network.py::test_storage_network failing on CI with 1.4-head
# teardown
        disable_spec = api_client.settings.StorageNetworkSpec.disable()
        code, data = api_client.settings.update('storage-network', disable_spec)
        assert 200 == code, (code, data)
        snet_disabled, (code, data) = setting_checker.wait_storage_net_disabled_on_harvester()
        assert snet_disabled, (code, data)
        snet_disabled, (code, data) = setting_checker.wait_storage_net_disabled_on_longhorn()
>       assert snet_disabled, (code, data)
E       AssertionError: ("No annotation 'k8s.v1.cni.cncf.io/network-status' on pod", {'apiVersion': 'v1', 'id': 'longhorn-system/instance-mana...'https://<IP_ADDRESS>/api/v1/namespaces/longhorn-system/pods/instance-manager-230bda33ca02e64a1b2e5a6e4cca5599'}, ...})
E       assert False

_          = (200,
 {'apiVersion': 'k8s.cni.cncf.io/v1', ...

Additional related test cases failing:

TestImageWithStorageNetwork::test_delete_image::teardown
TestVMWithStorageNetwork::test_enable_storage_network_with_api_stopped_vm::teardown
TestVMWithStorageNetwork::test_enable_storage_network_with_cli_stopped_vm::teardown

Additional context:
Why are other storage test cases are not failing like TestImageWithStorageNetwork::test_create_image_by_file, TestImageWithStorageNetwork::test_download_image, TestImageWithStorageNetwork::test_delete_image

LH update annotation k8s.v1.cni.cncf.io/networks as Storage Network configured https://github.com/longhorn/longhorn-manager/blob/8fadbfbeb537ea8fbbc957b69e1e6b243cb806af/controller/backing_image_data_source_controller.go#L845-L853
LH manager doesn't rely on such annotation, if there is no annotation k8s.v1.cni.cncf.io/network-status, it directly retrieves pod IP by pod.Status.PodIP

We will update our test logic as following

Check networks instead when enabling storage-network .
Should NOT have networks  when storage-netowrk is diabled.


Where is the code?

Where is the source code apart from the attached zip files with each release? Do you use Gitlab or something?

see README Please


fix: no_optional should remove Optional

TODO:

[ ] Test

Codecov Report
Patch coverage: 100.00% and project coverage change: -0.04% :warning:

Comparison is base (55c0d81) 85.39% compared to head (2e548f9) 85.35%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #73      +/-   ##
==========================================
- Coverage   85.39%   85.35%   -0.04%
==========================================
  Files          24       24
  Lines        1239     1236       -3
==========================================
- Hits         1058     1055       -3
  Misses        181      181

Files Changed
Coverage Δ

crates/cli/src/commands/add.rs
94.59% <100.00%> (ø)

crates/cli/src/commands/install.rs
96.10% <100.00%> (-0.15%)
:arrow_down:

crates/package_json/src/lib.rs
95.58% <100.00%> (ø)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


fix: translation issues

Description
Fixes some translation issues and general cleanup

please open a new pr and include only related commits, I could merge this if there was only translation issues

@eraygundogmus What is unrelated about this pr? All the changes are related to the translation?

@eraygundogmus What is unrelated about this pr? All the changes are related to the translation?

changes on the footer section is not related to translation, also changing the user interface

Ahh, let me revert that

@eraygundogmus What is unrelated about this pr? All the changes are related to the translation?

changes on the footer section is not related to translation, also changing the user interface

Fixed the mobile issue. (I think my changes are related, so I didnt revert them)

Member

oh. you are right.


fixed overlapping masonry items issue

unloaded images throw off masonry layouts and cause item elements to overlap
used imagesLoaded to to solve this issue and updated bower dependencies
updated dummy app to illustrate the problem (remove imagesLoaded dependency to replicate the problem)

Thanks for the PR! I'll take a look at this later tonight.

Hey! Did you have a look at it yet?

+1, works for me.

Actually, this does not work when the items change.

Thanks for this, and sorry it took me so long to merge! Much appreciated!


More feed Option

What if I want to load more than 12 images?

You cannot load more than 12 with this script
Check this https://github.com/jsanahuja/jquery.instagramFeed/issues/31


callback function name is required

https://github.com/gsuitedevs/apps-script-oauth2/blob/fdd3d0cb150574d67421c61c9644f7da73d2c4ef/samples/TwitterAppOnly.gs#L27
I was trying to make use of this and I already have the github service running in the script so I went ahead and changed the words out that I needed to but the code I am running and the sample both wont work.
I get an error
Error: Callback function name is required. (line 45, file "Utilities", project "OAuth2")
Your example does not have a callback function, so I tried adding my own. After I do this I am then told I need an authorization base url, which I cannot seem to figure out.
What should I do to get this sample working?

turns out I had added something to the sample that made it not work, and my credentials accidentally had a space at the end making my request never authenticate.


Issue with "keepMirroringOnWrite = YES" - a black screen in video

Hi, rFlex
In last version of SCRecorder if I activated mirroring in writing:
_recorder.keepMirroringOnWrite = YES;
then I see a black screen in wrote video
Please, resolve it

Same problem here.

i also have the same issue, anyone know how to fix it?

Shipped fix for this issue.
Please, review. @rFlex


Fixed Unit tests and allowed Pester 4.0.3 usage - Fixes #76

Removed requirement on using Pester 3.4.6 because Pester bug fixed in 4.0.3.
Fixed unit tests for MSFT_xDiskAccessPath resource to be compatible with Pester 4.0.3.

This change is

Codecov Report

Merging #83 into dev will not change coverage.
The diff coverage is n/a.

@@        Coverage Diff        @@
##           dev   #83   +/-   ##
=================================
  Coverage   90%   90%
=================================
  Files        5     5
  Lines      600   600
=================================
  Hits       540   540
  Misses      60    60

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 69446d4...ce7ddb6. Read the comment docs.

@johlju - I don't suppose you'd mind reviewing this one for me if you get a chance? It is pretty small and just fixes some legacy issues with the unit tests.

Just minor comment. Don't need to change it for this PR.

Reviewed 2 of 3 files at r1, 1 of 1 files at r2.
Review status: all files reviewed at latest revision, 1 unresolved discussion.

Tests/Unit/MSFT_xDiskAccessPath.tests.ps1, line 772 at r2 (raw file):

                It 'the correct mocks were called' {
                    Assert-VerifiableMocks

This does not work if you don't have -Verifiable on the mocks (?).

Comments from Reviewable

Thanks heaps for reviewing @johlju - I really appreciate it! I'll merge!

Review status: all files reviewed at latest revision, 1 unresolved discussion, all commit checks successful.

Tests/Unit/MSFT_xDiskAccessPath.tests.ps1, line 772 at r2 (raw file):
Previously, johlju (Johan Ljunggren) wrote…
This does not work if you don't have -Verifiable on the mocks (?).

Good point! :grin: But I have got -verifiable set on the mocks that should actually be called - see from line 738 above. I don't set -verifiable on mocks that should not be called (e.g. ones I test for Assert-MockCalled -Times 0). Does that make sense? :grin:

Comments from Reviewable

@PlagueHO Ah, I missed that. Then it is correctly done, otherwise Assert-VerifiableMocks would fail when the mock wasn't called. 😄 Should have looked at the code a little farther up 😉 .


Add --no-optional-locks to git status command

I came across this issue in combination with tpope/vim-fugitive as described here: https://github.com/tpope/vim-fugitive/issues/1624#

Thanks!

This causes endless errors since my git does not support this flag.
I can't upgrade my git to a version that supports it (i tried using apt-get to latest today).

@sagiegurari @feddischson I pushed a new PR #176.
Does it solve your problem?

looking at the code, i think it should fix it. thanks

@Xuyuanp also works in my case, thanks

merged


New share screen needs a way to return to project from beneath call to action

Feels weird not to have a "done" button or something below

vs

design is different now


[Project Submission]CID-Management.md

submission to the community spotlight repo

Thanks for the submission! Cheers


fix(docs): Add hyperlinks to skore's classes and functions in code snippets

In the scikit-learn documentation, in the examples, you can click on a class or method and arrive in the documentation of it, for example:
https://github.com/user-attachments/assets/5f5528ab-aac7-4a6c-bb56-0314a877dcd4
This is a nice feature that skore should have as well.

Loic's answer:

This is a sphinx-gallery feature, probably start with https://sphinx-gallery.github.io/stable/configuration.html#add-intersphinx-links-to-your-examples + look at what scikit-learn doc/conf.py for inspiration
but our skore doc already points to scikit-learn and but not our own functions


Consolidate unit filter options for Razor

Description
Consolidate Razor options in "Unit" filter box to display as (Razor HQ, Razor 1st Flight, Razor 2nd Flight, etc.)
Details
Basically the goal here is to bring it more in line with how Warden HQ and their 3 sections are displayed in the filter
For example. when "Razor 2nd Flight" is selected, then the below results should show up

Flight Commander, both Section Leaders, all remaining pilots in that flight

I will be submitting a Roster Change form to have all Section Leaders' roles changed from Pilot to Section Leader
Notes
If any questions or concerns, see CX Neb

Filter options will not be consolidated. They are based off the roster data and is needed to put pilots in their correct position on the ORBAT and to determine where they are. If you want to filter a Razor unit, you can click the T icon to switch to text search and manually type in Razor 2 to see the information you would like to see.


Weird peerDependency issue.

I'm trying to update my app to<EMAIL_ADDRESS>but getting this error from npm i:
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>|| ^0.6.0 || ^0.7.0 || ^0.8.0-b || ^0.9.0 || ^0.10.0 || ^0.11.0 || ^0.12.0 || ^0.13.0 but none is installed. You must install peer dependencies yourself.

I had forked the repo, and I see correct peerDependency statement in package.json,including<EMAIL_ADDRESS>and version of<EMAIL_ADDRESS>Could be there an issue with publishing?

Is there any movement about new release?


Use an app factory pattern for Response Operations UI

Motivation and Context

Makes it easier to spin up an instance (or instances) of the Flask application on demand. Useful for testing and less dependency on setting everything up at import-time.
What has changed

Changed to use a reproducible app factory pattern (better late than never!).
How to test?

Unit tests 🍰
Links

TBL#138
http://flask.pocoo.org/docs/1.0/patterns/appfactories/

 Python services to use app factory pattern (5)

Looks bigger than it is ;)

@jcox-dev have you pushed to CF to check that the procfile still works?


Image API not working

I use Image api to create serving URL from blobkey (java)
     ServingUrlOptions option = ServingUrlOptions.Builder.withBlobKey(blobKey);             final String servingUrl = imagesService.getServingUrl(option);
and the serving Url I've got from this function is in local ip like

http://<IP_ADDRESS>:20000/_ah/img/C--I1_h_34B30lGEfHNGLQ==

and the image is unable to be served by the appscale server.
Is there any example on how to use image api or create serving url on appscale? This function works perfectly on appengine.
Thank you

It looks like there's a bug in getServingUrl() while picking the correct IP and port for the serving URL.
You can work around this by using Blobstore serve() call for now.
Thank you for reporting the issue.


Authentication config won't work

Despite defining the security scheme for JWT in my Tspec configuration, the output in the JSON file doesn't reflect this correctly
Tspec Configuration
const tspecParams: Tspec.GenerateParams = {
        specPathGlobs: ["./src/**/*"],
        // tsconfigPath: "./tsconfig.json",
        outputPath: "./src/config/openapi.json",
        openapi: {
            title: "Tspec API",
            version: "0.0.1",
            securityDefinitions: {
                jwt: {
                    type: "http",
                    scheme: "bearer",
                    bearerFormat: "JWT",
                },
            },
        },
        debug: true,
        ignoreErrors: true,
    };

    const openApiSpec = await generateTspec(tspecParams);

API Specification:
export type CommentApiSpec = Tspec.DefineApiSpec<{
    security: "jwt";
    paths: {
        "/comment/:postId": {
            get: {
                summary: "Get comment by post id";
                path: { postId: Tspec.UuidString };
                responses: { 200: CommentType[] };
        };
    };
}>;

Resulting JSON Output (part of):
  "info": {
    "title": "Tspec API",
    "version": "0.0.1"
  },
  "openapi": "3.0.3",
  "paths": {
    "/comment/:postId": {
      "get": {
        "operationId": "CommentApiSpec_get_/comment/:postId",
        "tags": [],
        "summary": "Get comment by post id",
        "security": [
          {
            "jwt": []
          }
        ],
      },
    },
  },

I checked that this code generates output correctly
But I can't find any Authorization button at all in swagger ui page

Hi, @seoes.
Thank you very much for taking the time to report the issue and for your interest in our project.
As @digitalcrafted has already tested it, I have also tested it and can confirm it works well in the latest tspec version 0.1.114.

I have also added this to the example book-advanced (commit: c573d83324e085b02dd573ee85a3727abab3f977).
Please update your tspec version and try again. If the issue persists, let us know.


Failure of Black Test in CI testing in github

Description of the bug:
I opened a pull request to add some documentation files. However, during CI testing on GitHub, some .py files failed the black test (see image below and this link
for the full log description), despite passing when I ran these tests on my Linux machine. Note that I didn't make any changes to these Python files in the PR.

Expected behavior:
All files should pass the CI testing.

There seems to be a difference between the version of Black installed locally (23.12.1) and the version used in the CI environment (24.2.0), despite both using the same command pip install ".[test]" !!

I don't understand your point.  The new version of black==24.2.0 should be different than black==23.12.1, else there's nothing to release.   But, yes, for black==24.2.0 the files which passed with the previous version of black==23.12.0 are no longer compliant.  So, the solution is to upgrade black and rerun on it on all files in the repo with black . to reformat those files which require it.

Did you verify that the files were reformatted?  How would you see these changes in the Pull Request?

When I checked the log of the Black formatter, it is written "36 files would be left unchanged", but these files were already passing the CI test. However, it didn't provide any information about the other 8 files that failed the test. so, I assumed that nothing has changed. I will see if I can run the Black formatter before the Black testing in ci.yml file. If not, I will apply the black . command to all files.

The CI log doesn't mean that files in the repo (associated to the PR) have changed.

From today's meeting the way forward in resolving this issue is to do the following:

[ ]  Remove pytest-black  and --black from the pyproject.toml file
[ ]  Add a run a step to run black during CI like you did previously (ci.yaml)
[ ]  Add a checkbox in the Pull Request template for the reviewer to check the CI log to evaluate whether any files had to be changed or not

Let me know if something is unclear.

Done. I also removed pytest-cases to avoid encountering this error during pytest test:

I am not sure whether the solution to remove pytest-cases is the correct one. Could you please open a new ticket to track this new problem?

Actually, I checked and I'm not using pytest-cases to set up test cases, rather I am using pytest.mark.parameterize, so it's okay to remove it.  But, it's odd that removing it resolves this conflict.


Check modified values when filtering store.

Based on John's original PR, per plan by API review meeting.
NOTE --  John mentioned some further concern about also applying this to StoreFilterField.  I think it would behoove us to go ahead and provide the same logic there.
NOTE -- If we decide not to take this PR, there is a nice refactor for DRY in the switch statement that we should take in any case.
Hoist P/R Checklist
Pull request authors: Review and check off the below. Items that do not apply can also be
checked off to indicate they have been considered. If unclear if a step is relevant, please leave
unchecked and note in comments.

[X] Caught up with develop branch as of last change.
[X] Added CHANGELOG entry, or determined not required.
[X] Reviewed for breaking changes, added breaking-change label + CHANGELOG if so.
[X] Updated doc comments / prop-types, or determined not required.
[X] Reviewed and tested on Mobile, or determined not required.
[X] Created Toolbox branch / PR, or determined not required.

If your change is still a WIP, please use the "Create draft pull request" option in the split
button below to indicate it is not ready yet for a final review.

Pull request reviewers: when merging this P/R, please consider using a squash commit to
collapse multiple intermediate commits into a single commit representing the overall feature
change. This helps keep the commit log clean and easy to scan across releases. PRs containing a
single commit should be rebased when possible.

@haynesjm42 are you still planning to review this / are we still planning to take any changes here?  Thanks much, just looping through some of the open PRs to catch up.

CR: John


Hash Lookup and Linked Event based Evidence

Summary

Concisely describe the issue and your solution:
Link to existing issue (type "#" to check):
Specify the tests done so far:

Solutions (Optional)
Please describe the details of your solution if it might be too complicated for reviewers.

Codecov Report

Merging #123 into develop will decrease coverage by 1.27%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             develop     #123      +/-   ##
=============================================
- Coverage      45.03%   43.75%   -1.28%
+ Complexity      1375     1346      -29
=============================================
  Files            182      183       +1
  Lines          10035     9940      -95
  Branches        1286     1275      -11
=============================================
- Hits            4519     4349     -170
- Misses          5005     5093      +88
+ Partials         511      498      -13

Impacted Files
Coverage Δ
Complexity Δ

...eid/service/impl/engine/EvidenceServiceEngine.java
0.00% <0.00%> (-33.34%)
0.00% <0.00%> (-1.00%)

...transportation/pdf/impl/PdfTransportationImpl.java
46.51% <0.00%> (-13.96%)
46.00% <0.00%> (-16.00%)

src/main/java/com/webank/weid/util/WeIdUtils.java
71.42% <0.00%> (-3.18%)
25.00% <0.00%> (-2.00%)

...ebank/weid/suite/encode/CipherEncodeProcessor.java
48.42% <0.00%> (-3.16%)
9.00% <0.00%> (-1.00%)

.../webank/weid/service/impl/EvidenceServiceImpl.java
65.04% <0.00%> (-6.39%)
22.00% <0.00%> (-27.00%)

.../main/java/com/webank/weid/util/DataToolUtils.java
64.23% <0.00%> (-0.54%)
127.00% <0.00%> (+1.00%)

...om/webank/weid/protocol/base/EvidenceSignInfo.java
75.00% <0.00%> (ø)
4.00% <0.00%> (?%)

...rvice/impl/engine/fiscov2/WeIdServiceEngineV2.java
79.23% <0.00%> (+3.38%)
44.00% <0.00%> (+3.00%)
:arrow_down:

.../weid/protocol/response/ResolveEventLogResult.java
85.71% <0.00%> (+28.57%)
4.00% <0.00%> (+1.00%)
:arrow_down:

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 7613a96...3633e43. Read the comment docs.

This is now in #130