[Question] failed to call OrtRun(). error code = 1. When I try to load Xenova/pygmalion-350m

I'm getting an error failed to call OrtRun(). error code = 1. When I try to load Xenova/pygmalion-350m. The error is as follows
wasm-core-impl.ts:392 Uncaught Error: failed to call OrtRun(). error code = 1.
    at e.run (wasm-core-impl.ts:392:19)
    at e.run (proxy-wrapper.ts:215:17)
    at e.OnnxruntimeWebAssemblySessionHandler.run (session-handler.ts:100:15)
    at InferenceSession.run (inference-session-impl.ts:108:40)
    at sessionRun (models.js:191:36)
    at async Function.decoderForward [as _forward] (models.js:478:26)
    at async Function.forward (models.js:743:16)
    at async Function.decoderRunBeam [as _runBeam] (models.js:564:18)
    at async Function.runBeam (models.js:1284:16)
    at async Function.generate (models.js:1009:30)

And my Code for running it is this

let text = 'Once upon a time, there was';
let generator = await pipeline('text-generation', 'Xenova/pygmalion-350m');
let output = await generator(text, {
  temperature: 2,
  max_new_tokens: 10,
  repetition_penalty: 1.5,
  no_repeat_ngram_size: 2,
  num_beams: 2,
  num_return_sequences: 2,
});

console.log(output);

I see that OrtRun is something returned by the OnnxRuntime on a failure but have you had success in running the Pygmalion-350m model ?

I'm using a mac with brave browser and chrome 117.

Same on Linux / Chrome 117, not browser related. We had these errors before and sometimes it's "easy" to fix them:
https://github.com/xenova/transformers.js/issues/140#issuecomment-1629879842
Sometimes they crash the entire browser :see_no_evil:
error code = 1 means ORT_FAIL, it couldn't be more descriptive than that :sweat_smile: Sometimes I question the WASM overhead, it makes everything just so much more difficult to debug and I had a project where WASM is actually slower than V8 jitted/optimized JS (which would still be easy to debug).

I didn't spend the time yet to figure out how to use 1.16, for some reason is just doesn't find any backends:

Whereas using this works like a charm:
https://cdnjs.cloudflare.com/ajax/libs/onnxruntime-web/1.14.0/ort.es6.min.js

Ah yes, it defaults to use WASM files served via jsdelivr, which is what the errors indicate. No worries then, I can do further testing for an upcoming release.

For the DynamicQuantizeMatMul, sorry but I do not know much details. Maybe need who write this kernel to take a look.
https://cdnjs.cloudflare.com/ajax/libs/onnxruntime-web/1.16.0/ort.es6.min.js  <-- this seems working on my html when I use <script src="https://cdnjs.cloudflare.com/ajax/libs/onnxruntime-web/1.16.0/ort.es6.min.js"></script> . Can I get a reproduce steps?

@xenova  I haven't tried 1.16.0 but when I try with the unquantized version, it gives an error saying Error: Can't create a session.
Will try with 1.16.0 and see

Not sure if this is caused by the .wasm file not found. you can check network tab in devtool if there is a 404 for the .wasm file.
if you don't use

Let me know if you think the issue is related to the export.

Can I get a reproduce steps?

Yep, I have a simple example project here: https://github.com/kungfooman/transformers-object-detection/
It's a bit hacky because there doesn't seem to be any published ESM build yet.
Because I like to use repo's as "source of truth", I also converted what I needed once from TS to ESM here: https://github.com/microsoft/onnxruntime/compare/main...kungfooman:onnxruntime:main
(but I don't have the time to maintain it rn)
In general the future is ESM + importmap, every browser aims for that nowadays and npm packages follow, for example:
https://babeljs.io/docs/v8-migration (only ships as ESM now)
Would be nice if ONNX ships browser-compatible/working ESM files too (aka don't drop the file extensions).
Thank you for looking into this @fs-eire :+1:

@uahmad235 The repo you linked to does not include ONNX weights in a subfolder called "onnx". You can do the conversion yourself by installing these requirements and then following the tutorial here. Please ensure the repo structure is the same as this one.

Apologies @xenova . My bad. I forgot to mention that i have already converted weights using the given instructions. The converted weights does have a file called onnx/decoder_model_merged.onnx alongwith others.

@uahmad235 No worries :) In that case, could you try run the unquantized version?
const generator = await pipeline('text-generation', '<your_model_id>', {
    quantized: false, // <-- HERE
});

This may be due to a missing op support in v1.14.0 of onnxruntime-web.

Thank you @xenova . I have tried that already following the thread but unfortunately it does not work. It is strange that gpt works but not the danish variant that i am trying to work with.

@uahmad235 Feel free to open up a new issue with code that allows me to reproduce this. It might just be a configuration issue. 😇

Sure. Let me give another try and I can add a separate issue in case it does not work. Thanks for the prompt response :)


Failed to load on 1.8.0

The plugins fails to load on v1.8.0. Compatability Issue? This is my list of active plugins:

Thanks will check it out, I run 1.7.7 myself.

Hi, just updated to Obsidian 1.8.0 and Featured Image works as expected, no issues. Please make sure you are running the latest version 1.1.10 or later. I tried both MacOS and Windows 11. Can it be a conflict with one of your other plugins?
Obsidian version:

My list of activated plugins:


BBGW not booting with Arago images

Hi,
I have created images using Arago build:
The images are:
MLO
U-boot.bin
zImage
dtbs
core-image-base-am335x-evm-20181030062907.rootfs
I have copied the MLO, u-boot.img, uEnv.txt, zImage to the boot partition of the SD
Extract the rootfs to rootfs -partition of the SD
My  uEnv.txt sets the following:
"
bootpart=0:2
bootdir=/boot
bootfile=zImage
console=ttyO0,115200n8
fdtaddr=0x88000000
fdtfile=am335x-bonegreen-wireless.dtb"
But when I insert the SD Card, the board does not boot at all.
Quick help is much appreciated.
Thanks in Advance!
Annapurna.B

@tany5522, Closing, sorry this isn't related to "omap-image-builder", you need to contact the team that developed your Arago build..
Regards,


lambda-controller fails to recognize changes in function code when deploying a new package

Is your feature request related to a problem?

A description of what the problem is. For example: I'm frustrated when [...]

The feature request is a part of solution to the problem arising in the issue #1550. To summarize the issue, when a user creates a lambda function with ACK using a file from s3 to deploy the package and then tries to use a new deployment package, the changes are not reflected.
The problem is arising due to ACK not being able to compare the change in state for Spec.Code (ex: change in Spec.Code.S3Key), which in-turn is issue because ACK Lambda API does not keep record of Spec.Code variables. ACK does not have any record of previous Spec.Code variables, used to create the function and thus when there is any change it fails to recognize it.
The output of ACK Lambda API:
$ aws lambda get-function --function-name test-function --region us-west-2
{
   "Configuration": {
       "FunctionName": "test-function",
       "FunctionArn": "arn:aws:lambda:us-west-2:322962841005:function:test-function",
       "RevisionId": "3ee79e30-bbf0-4a58-9337-b17a52dc4a6e",
        ...
   },
   "Code": {
       "RepositoryType": "S3",
       "Location": "https://awslambda-us-west-2-tasks.s3.us-west-2.amazonaws.com/snapshots/322962841005/test-function-9ac7357e-a87b-489e-ba10-351735be85ee?versionId=r12LlaOpoSHuzgrBk8Gtzx5hSTB4mUob&X-Amz-Security-Token=IQoJb3JpZ2luX2VjEA8aCXVzLXdlc3QtMiJIMEYCIQDkj8WvX%2FglQRAPoNL7hEFV9XVVzUFfbWxcGXm94hUNnAIhAPy73r%2Fjb12oBJn5SpxB8Gnyd4M7UotoNRPCYAS7j2gWKtUECJj%2F%2F%2F%2F%2F%2F%2F%2F%2F%2FwEQBRoMNTAyMjk3MDc2MTYzIgxrbESv%2FpxZYbqy8vwqqQTf4HwOan5Eli7odnwQI%2FdGE7hd6K0CjX2Bw9%2BEq72f9PWoBBeduezJrXAI%2FBw8OYPqiZv6CHNzU1cGmb%2FrCjmngvEIyJa2ROLpZSSuAClC6jX65qXvZB1VLLx2gaXX9ll8RF81OwCAM%2B2RO4M67MLSdv6USRQfFakFFTrOdk%2FrjUmofH%2F%2FTFciojHg8xJU2mqzm%2F4II7VAiEAjg%2BtqwJBEE2VY02usJWB3SAANRQOGBcx%2B0Y11aIlwxWW7ZVpjCdEtB3JvKttHfL8uqebUWa8pKw0iUaDtAf4d0weUG2M%2FUHMTvW4U4snIK6KHFVzDFLLeq3PRntaj%2BZO09dqvK586gnppME37NBUbSQ59yFA7t13tpJfFhoU9IJYGVOM0npKAbmphygJJrS3nhlxyM0Wq2j98STOLL6TKQSOGH1zHaEctqD98LolLTcLA1xgvcH9tiRY6rBjZ3OhWsMBN08rg8JT%2F9tG0VzabyT8nfdA%2FA7mp1kdPImUbnvtiwHte6sGo0nIGXQbTz2ypQs7Hsek%2FPKDJV1rQImwJo%2FNojo5agyC7iQyQKywW0bcpS26j7Haa9uXTurs7z55QqqPJkVONp68x109NGxnJ54gIkXZXmw5H7Kb3jAcw12QWwun3aKAySqdPvn36K%2BPhBGjZmYpWJ7DJIbtxHKIT7c0hli1JhSCTAIAk0PQkrFCtgt1jTxHYmfofoz7JRkSZ%2BhNcuOtzXb8l%2B9B1kIaSMIXxlZ8GOqgBRS%2F3ssabko4pgAC4cBjVPf9ioFzcQ83JnSOTWO%2Fq0jB4S6rhYZ%2FU3kqRn53uwo1nAYwcQSnoEwleT3DRGVV9uGSmPp%2F8jNx1%2BgMJ3G3Di8FqMfmJW4mcQaoFYY7uu0IB3nsoiEsO%2BGrR%2FdpPj2HgA3RvjUS%2FMXWE6%2B4CRSo37LsLnre19yuxqtczc%2BPr4%2BtjtE4OFwykCqS5Igugdspskdfsc%2Bibu31i&X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Date=20230209T230900Z&X-Amz-SignedHeaders=host&X-Amz-Expires=600&X-Amz-Credential=ASIAXJ4Z5EHBQBUTO7MC%2F20230209%2Fus-west-2%2Fs3%2Faws4_request&X-Amz-Signature=e90564a5ad7c878a96981d218313424c431c64756ac095562c39645526a2ba96"
   },
   "Tags": {
       "services.k8s.aws/controller-version": "lambda-unknown",
       "services.k8s.aws/namespace": "default"
   }
}

The API keeps record of only Code.Repository and Code.Location. We want it to keep record of Code.s3Key, Code.s3Bucket and Code.s3ObjectVersion.
Describe the solution you'd like

A description of what you want to happen.

The main goal is to store the state of s3.Key and s3.Bucket and use it to compare the change in state. We have think of three solutions to save the record for s3 variables.

To include the fields in the status. Means to add it in the API response. This would be a bigger task to add the fields in the API response call.

status:
  s3Key: X
  s3Bucket: Y
  s3ObjectVersion: Z

To store it under metadata in function.yaml, metadata stores information related to the function for k8s. This way the record will be stored on K8s side, and not on AWS side.

metadata:
  annotation:
     s3Bucket: X
     s3Key: Z

To store using tags as an array of {key, value} pairs. This could be done while describing a function in function.yaml, we can add extra tags, which API would return as normal tags but it will be storing the state of Code.s3.

spec:
  name:
  code:
  tags:
    s3Key: X
    s3bucket: Y

This way the information will be stored on AWS side.
Please give your thoughts and ideas on the above mentioned solutions and other ways this issue can be handled.
Describe alternatives you've considered
A description of any alternative solutions or features you've considered.

/cc @jaypipes @RedbackThomson @jljaco @embano1

An alternative for options 2 and 3 is to instead include some sort of CodeHash or CodeSha256, that will be some sort of hash (or any value really) that if it changes, it means that the code needs to be updated. In theory can be created actually calculating the hash of the code, so regardless the actual location, we know that the code changed, so we just call UpdateFunctionCode (In practice, it can be just any value, that it's manually changed when customers change their code).
There's something like this in SAM for the code (to indicate that the code changes for Auto publishing versions) AutoPublishCodeSha256

There's been a discussion last year about loosing the meaning of status from "populated purely by observation". IMHO a good read to weigh some options: https://github.com/kubernetes/community/pull/5842/files
In this case I'd simplify to using hashing and perhaps tracking the value in an annotation (unless any required changes in ACK runtime for custom statuses is an easy one).

Has there been any progress here? We've just started testing lambda-controller and this is a dealbreaker. We can't update our functions after the initial creation.

Hi @jessebye , we are working on a fix for spec.code.s3* changes in https://github.com/aws-controllers-k8s/lambda-controller/pull/88 - however this requires a feature in code-generator that is not supported yet. The problem here is that lambda api caches the code ins s3 and there is no way to compare latest/desired s3bucket, filename etc... we will have to support it by adding a new field spec.code.sha256 that users will have to update to trigger a code update.
If you use ECR images, ImageURI updates works fine.

any update on this ?

Also interested in this and got a question - will we need to set the new sha field to the actual sha of the S3 object? Or will it be just a placeholder for a force-update?

The functionality to support updating deployment package for Lambda function in ACK is available now.

@mhr3 The expected behavior is for you to set the actual SHA256 of the S3 object, however it does work as a placeholder to force-update and will work even if the SHA256 doesn't match.


105 graph anttns

added graph columns to display the annotation messages,
slight change in the indexing.

Code looks good, but I think the original intention of the issue was also to have a datepicker https://eonasdan.github.io/bootstrap-datetimepicker/ for adding new annotations, since it just assumes the present as is. That might require slightly changing a back-end api, but I don't think it should be too much.

Datepicker would be optional, we could assume present by default. That's just to make it so if we do a FB ad campaign and forget to make an annotation, we can add it back in later.

merge develop into this branch before later merge


Add initial event to onAuthStateChange

What kind of change does this PR introduce?
Fixes supabase/gotrue-js#326
What is the current behavior?
There is no event fired unless a sign in occurs, but that is sometimes delayed. So there is no way to know if the user is signed in or not right when the page loads
What is the new behavior?
There is an INITIAL event fired after the library is finished loading and checking for a logged in user.

looks like the failing tests are from gotrue - not your changes. Let me investigate

Adding a bit more food for thought on the above:
Calling .session() or .user() will return null if the session is expired, though it can be refreshed. So checking these when attaching the auth state listener won't work.

Maybe inside .session() or .user() we should check if the loading is finished and wait for that before returning null

Has this change been given more consideration? Would love to see it implemented

https://github.com/supabase/gotrue-js/pull/147#issuecomment-946548042 @gustavohenke
Could this be solved potentially but setting an isInitialized boolean on the client which can be checked at the same time as attaching listeners?
Still not sure if this would be 100% atomic though (ie vs passing in a callback to the constructor).

Thank you for this contribution, however the library has advanced quite a bit.


Fixed downloads link in readme

Hey @amilajack!
Are you ok with making master protected and making code reviews required?

Do you have this option enabled?

If it's not, do you know how can I enable it?

Are you ok with making master protected and making code reviews required?

I'm all for it!

If it's not, do you know how can I enable it?

I dont thinkg it can't be enabled for me. There can only be one admin of a project. I think its fine tho.

@amilajack I made master protected but disabled code review and ci requirements in case you'd need to push a quick fix. I think it doesn't make a lot of sense to disable that button. Let's just use common sense and feel free to merge urgent or small fixes and wait for a review for bigger changes.


Adding class name to meta by default

I have a custom format for my logs. I would like for each log to indicate which class it's from.
var customFileFormat = function(options){
  class_name = options.meta.classname || '';

  return JSON.stringify({
    created_time: new Date().toISOString(),
    message:        undefined !== options.message ? options.message : '',
    level:               options.level,
    class:               class_name
  });
};

Right now, I require the logger.js in every class and log like this:
logger.info( ' logging message ', { classname: 'foo' } );
Is there any way that I don't have to repetitively type the  { classname: 'foo' }

This can be accomplished with using custom formats in<EMAIL_ADDRESS>Please consider upgrading.

Is there anywhere an example, how this has been solved using a custom format?
I can't find a solution anywhere and I'm not able to figure it out myself using custom formats.
Thanks for the help!


Fix #3224: Added mysql unit test

Describe your changes :

Fix #3224: Added mysql unit test

Type of change :

[x] Improvement

Checklist:

[x] I have read the CONTRIBUTING document.
[x] I have performed a self-review of my own.
[ ] I have tagged my reviewers below.
[ ] I have commented on my code, particularly in hard-to-understand areas.
[x] My changes generate no new warnings.
[x] I have added tests that prove my fix is effective or that my feature works.
[x] All new and existing tests passed.

Reviewers

Closing this as it's being tracked in #3623


xfreerdp (v. 2.0.0~dev) corrupts date and time with Windows 7 and 10

xfreerdp (FreeRDP version 2.0.0~dev (git n/a)) corrupts date and time with Windows 7 and Windows 10 while it works well with Windows 2008. Same is true with remmina (Version 1.2.0-rcgit-18 (git rcgit-18)) that relies on the FreeRDP library.
A previous version of xfreerdp (FreeRDP version 1.1.0-beta1 (git n/a), downloaded from the Ubuntu 16.04 repo) handles correctly the date and time with all the above mentioned Windows variants.
See https://github.com/FreeRDP/Remmina/issues/934.

@hanakemt Please be a bit more detailed in your bug reports. I guess you're talking about the drive redirection? That one has been fixed on current master. Could you please confirm?

@akallabeth: Sorry for omitting that detail. Yes, the error effects shared folders (aka drive redirection: "/drive:hanak,/home/hanak").
Good news that it is fixed on the current master. As I am using the remmina ppa (http://ppa.launchpad.net/remmina-ppa-team/remmina-next/ubuntu/dists/xenial/main/) I hope it willalso  be updated soon. Thanks!


@update for NotPixel bot

Any update

I wrote my own bot and not only for HamsterKombat (NotPixel is there, but still in testing mode). Made a full-fledged website and personal office for it.
https://imbot.space
Authorization via Telegram. It is paid with a week trial period (Payment via YooKassa in rubles. If interested in payment not in rubles, please contact me ap@ex9.ru).


FRI Session could not be established

Hello, I'm trying to set up our LBR Med 14 robot, but I cannot establish the FRI session.

I can ping the robot on <IP_ADDRESS>, so I guess there is a connection. But it immediatly shuts down when I try to run the server.
My setup is Humble on 22.04. And I'm running
ros2 launch lbr_bringup bringup.launch.py model:=med14 sim:=false rviz:=true moveit:=false

I saw the error is similar to this issue, but I cannot resolve it in the same way.
Thanks in advance

<IP_ADDRESS>
<IP_ADDRESS>
Wow, that was a fast response

no problem, was just working on something. Can you try to update your netmask?

I updated it to <IP_ADDRESS> and it still failed if that was what you meant.

there is a chance that your FRI version is slightly different

I checked my FRI version and it's 1.15.1.SMedV121-15-0-35. If this is a version you want me to extract, then I'll do it
I tried to open the ports with the command spesified in the issue, but it didn't help

Hmm... I tired to save the version as outlined, however I get the error:

Could this be an issue?

no, this indicates that you'll have to add your robot in the StationSetup.cat -> Topology tab before you do something else

I fixed the issue of the Topology version. Seemed like it was set to Version 2. Setting it to Version 1, made me able to save the FRI version.
I have followed the robot setup. I ran into a few issues with Java versions, since I got errors in the LBRServer file when running on the default jdk. Changed the jdk to jdk1.7.0_80 and the compiler to 1.6, which somehow fixed the issue and I was able to upload the project

so in that case, could it be that the LBRServer java application does not work with your version?

Could be. The error I got was:
The type java.io.ObjectInputStream cannot be resolved. It is indirectly referenced from required .class files

after some Googling it seemed to be a compatibility issue with breaks at Java 1.8 and up. And I don't know what Java version my robot should be running on.

it might make sense for you to try and run KUKA's FRI demos first, i.e. without any ROS 1/2 dependency.
You might want to run the LBRJointSineOverlay java application and the corresponding FRI client side application

I did that now. Seems I get the same error. This is probably an issue with the FRI on the robot. I've contacted KUKA Service to help with this. I'll let you know when this is resolved

Thanks BTW. You've helped me alot!

happy to help, please let me know in case you find a solution!

Hey! I found the solution. The solution was that I had not installed any packages in the Sunrise Project.
When I ran the SineOverlay test, it worked.
However, the adventure did not end there. The LBRServer did not compile because of a compile error "The type java.io.ObjectInputStream cannot be resolved", which was solved by setting the JRE System Library to jdk.1.7.0_80 and the Compiler to 1.6
It has something to do with enum not being supported in some versions of Java.

Thank you for reporting back @adamleon. Okay that is quite interesting and also inconvenient for you, glad you found a solution.
Do you think this is something that could be changed in the code, rather than by changing the compiler?


Top level validation events do not work with<EMAIL_ADDRESS>and higher

vue & vue-validator version
1.0.22, 2.1.1
Reproduction Link

https://jsfiddle.net/tzj8kjc9/
Steps to reproduce
Top level validation events don't work with<EMAIL_ADDRESS>Try to change Vue version to e.g. 1.0.21 and you will see that it works.
Example is from https://github.com/vuejs/vue-validator/blob/dev/example/event/top/index.html
What is Expected?
Top level validation events works.
What is actually happening?
Top level validation events don't work.

Ran into this today as well. For some reason, it was being evaluated as a statement expression, so I had to do @valid="onValid()"

I also have this issue.

Thank you!

release v2.1.2
please try it!
https://github.com/vuejs/vue-validator/releases/tag/v2.1.2


Complete switch to postgres 13.1

Description

Forgot one place where to switch it.
Forgot to set a+x for the docker-entrypoint.sh.
Accidentally downloaded the html page for docker-entrypoint.sh instead of the raw content.

Test plan

docker build && docker run
GHA

If you're wondering why tests didn't catch the previous oopsie, that's because the test infra file wasn't updated.


Update tttLinks.md

Added prod links for design pages

Docs Build status updates of commit 3c9f05e:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

mixed-reality-docs/tttLinks.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Allow setting site-packages root for uv sync instead of using venv's

It would be nice if uv sync (and similar commands) supported the setting site-packages root (for installation of the pep582 type), instead of using venv.
 Now they have venv. This is not great and I don't like it.

Please see https://docs.astral.sh/uv/concepts/projects/#configuring-the-project-environment-path which allows you to change the target of uv sync.
Note that "not great and I don't like it" isn't enough detail for us to actually change things for you; you'd need to explain why a virtual environment is a problem.


'could not find symbol value' in for loop

What version of Delve are you using (dlv version)?
Delve Debugger
Version: 1.0.0-rc.1
Build:

What version of Go are you using? (go version)?
go version go1.9.1 darwin/amd64

What operating system and processor architecture are you using?
macOS Sierra 10.12.6 Macbook Pro 15" 2017

What did you do?
Using https://gist.github.com/dvic/633738f5dcad90cd3be30455cff38c47, I did:

dlv debug .
break main.go:33
continue
app stops at line 33
print i

What did you expect to see?
the value of i (0)

What did you see instead?
Command failed: could not find symbol value for i

When I step into the function runPerson on line 37, then I can print the arguments (i and p)

I can also confirm that it is working when installing with go get -u github.com/derekparker/delve/cmd/dlv. It also outputs the same version as Intellij IDEA. It seems an issue with the brew installed version?

Brew installs the last release which doesn't support go 1.9, IDEA has a build from master.

Ah, I see, thanks, I've created https://github.com/derekparker/delve/pull/988 in case this should be mentioned in the docs.

Fixed by release of 1.0.0-rc.2


Failed test while building from source

After getting the source and running mvn clean install in OWLTools-Parent, I ran into a failure...
Tests run: 2, Failures: 1, Errors: 0, Skipped: 1, Time elapsed: 0.059 sec <<< FAILURE! - in owltools.gaf.io.PseudoRdfXmlWriterTest
testRelationWriter(owltools.gaf.io.PseudoRdfXmlWriterTest)  Time elapsed: 0.008 sec  <<< FAILURE!
org.junit.ComparisonFailure: expected:<...eontology.org/go#GO:[0065007"/>
    <go:is_a rdf:resource="http://www.geneontology.org/go#GO:]2000112"/>
    <go:r...> but was:<...eontology.org/go#GO:[]2000112"/>
    <go:r...>
	at org.junit.Assert.assertEquals(Assert.java:115)
	at org.junit.Assert.assertEquals(Assert.java:144)
	at owltools.gaf.io.PseudoRdfXmlWriterTest.testRelationWriter(PseudoRdfXmlWriterTest.java:61)

It hit this failure then continued to run and sat on Running owltools.gaf.owl.MgiGAFOWLBridgeTest for about 5-6 minutes before totally failing.
Here's the final info:
[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary:
[INFO]
[INFO] OWLTools-Parent .................................... SUCCESS [  0.676 s]
[INFO] OWLTools-Core ...................................... SUCCESS [01:26 min]
[INFO] OWLTools-Annotation ................................ FAILURE [09:31 min]
[INFO] OWLTools-Oort ...................................... SKIPPED
[INFO] OWLTools-Sim ....................................... SKIPPED
[INFO] OWLTools-Web ....................................... SKIPPED
[INFO] Lego ............................................... SKIPPED
[INFO] OWLTools-Solr ...................................... SKIPPED
[INFO] OWLTools-Runner .................................... SKIPPED
[INFO] OWLTools-NCBI ...................................... SKIPPED
[INFO] Golr-Client ........................................ SKIPPED
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 11:22 min
[INFO] Finished at: 2016-11-04T07:44:58-04:00
[INFO] Final Memory: 40M/508M
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-surefire-plugin:2.18.1:test (default-test) on project OWLTools-Annotation: There are test failures.

I need OORT which it looks like doesn't come with the downloadable jars (the easy way). I also looked at #163 which had a similar error, but I am using Java 8. Here's mvn --version:
Apache Maven 3.3.9 (bb52d8502b132ec0a5a3f4c09453c07478323dc5; 2015-11-10T11:41:47-05:00)
Maven home: /Users/Tauber/lib/apache-maven-3.3.9
Java version: 1.8.0_71, vendor: Oracle Corporation
Java home: /Library/Java/JavaVirtualMachines/jdk1.8.0_71.jdk/Contents/Home/jre
Default locale: en_US, platform encoding: UTF-8
OS name: "mac os x", version: "10.11.6", arch: "x86_64", family: "mac"

Blocker for me too.

Will try and look into this today
for now you can get the jar from
https://build.berkeleybop.org/job/owltools/lastSuccessfulBuild/
@kltm - do you know why this is a yellow rather than red on jenkins?

Thanks! @cmungall


v1, how can I focus a select field?

I'm using version 1.0.0-rc.1, I used .focus() to focus the field but it doesn't work.

It's not clear what your tried or your exact goal. Here is a CodePen that will hopefully help you. I put some comments into the JS area to which you'll want to pay attention.

It is not an issue, but a question about how to code, so it should be on stackoverflow, not here #

Personally I don't like overwriting built in functions so I don't think we will implement this. A couple extra lines shouldn't be too bad


New Tractus-X project contributor - Saad Bin Rafiq

Necessary Information
Email address to add<EMAIL_ADDRESS>Committers, that could take over: @oyo @evegufy @ss-nikunj
Eclipse Account
https://accounts.eclipse.org/users/saadanzari

done, you should get an invite soon :)


MP-7294 [BS-8103] NOX - Define and implement custom type "Formula" #82

looks good, small comments

Thanks for feedback @rochar , I've fixed the comments


New update

Nice new update no problems so far keep up the good work 👍👍😁🌟

Dude 🤦♂.
Open issue if you have any issue with the app.

And dude I am just trying to show some support this is the only way I know how.

And dude I am just trying to show some support this is the only way I know how.

Go to discord channel.

Ok


DSPT3 Cai Nowicki Sprint 1 - Dealing with Data

Nice work importing that data!
You've found both ways of accessing a pandas column, well done.
Would it make sense to use any two or more columns to make an additional column?

Load a dataset from Github (via its RAW URL)

You did this flawlessly. Great job!

Load a dataset from your local machine

Nice work again using the colab library

Load a dataset from UCI using !wget

Dealt with some funky data here. Nice.

Stretch Goals - Other types and sources of data

Ummm, just, like, wow! Great work exploring a couple brand new packages and dealing with some difficult stuff. Too Awesome! Keep it up!

Sprint challenge code review:
Part 1 - Load and validate the data

Load the data as a pandas data frame.

Complete.

Validate that it has the appropriate number of observations (you can check the raw file, and also read the dataset description from UCI).

Complete.

Validate that you have no missing values.

Complete.

Add informative names to the features.

Complete.

The survival variable is encoded as 1 for surviving >5 years and 2 for not - change this to be 0 for not surviving and 1 for surviving >5 years (0/1 is a more traditional encoding of binary variables)

Make sure you pay close attention to the details here.

At the end, print the first five rows of the dataset to demonstrate the above.

Complete.

Part 2 - Examine the distribution and relationships of the features

Explore the data - create at least 2 tables (can be summary statistics or crosstabulations) and 2 plots illustrating the nature of the data.

Good job exploring several cross-tabs. What do you see here?
Check out pd.qcut()

Part 3 - DataFrame Filtering

Use DataFrame filtering to subset the data into two smaller dataframes. You should make one dataframe for individuals who survived >5 years and a second dataframe for individuals who did not.

Complete.

Create a graph with each of the dataframes (can be the same graph type) to show the differences in Age and Number of Positive Axillary Nodes Detected between the two groups.

I like the bar graphs.
Part 4 - Analysis and Interpretation

What is at least one feature that looks to have a positive relationship with survival? (As that feature goes up in value rate of survival increases)

Age, or year_of_operation?

What is at least one feature that looks to have a negative relationship with survival? (As that feature goes down in value rate of survival increases)

👍

How are those two features related with each other, and what might that mean?

👍

Great job, Cai!


Set PPC video optimization feature flag to true

This will be the last ticket to go through for the new video optimization experiment. Just set the experiment to true and use this as the QA step for the video optimization bulk.
Acceptance criteria

Experiment set to true
Detailed QA steps for Claudio

Note that we usually set default => true for flags in 1 PR, and then completely remove it in the next PR. In case we need to revert rollout quickly.

To Pascal's comment above, I'll make two prs: one set to true and one to remove it completely

@csossi the PR itself has really helpful QA steps for you :)

To Pascal's comment above, I'll make two prs: one set to true and one to remove it completely

@embarks Note that the second PR is usually done weeks later, after release. Not right afterwards.

I have separate issues for these actually :)
https://github.com/google/web-stories-wp/issues/8168

Verified in QA


Collector refactoring and multithreaded search

This is a rather large refactoring of the collector API.
This change include no change to the index format, but breaks user API.
The big picture is, implementing a collector now requires to implement 2 Traits.
They both share a Fruit associated types that represents the result type of the collector. (what you are trying to compute)
The Collector and the SegmentCollector.
The SegmentCollector does the actual computation over segments, and Collector acts as a factory of SegmentCollector and implements the logic to merge fruits.

collector::chain is dropped and is replaced by just passing tuples of collectors.
search(...) returns the fruit directly. There is no need to get the result from the collectors.

This unlocks multithreaded search, simplify the user API, and opens the door to distributed search (provided the fruit you are working with a serializable.)

@hntd187 Added you because this refactoring is quite important for Toshi. It creates a simple solution to implementing sharded search.

Call for advise @jason-wolfe , @drusellers : as pointed out by @drusellers, it might not be a good idea to create the thread pool at every single query.
The trouble here is: we could let the user pass his own thread pool, but if so, we should have an object wrapping rayon's thread pool to avoid exposing rayon's object and create a dependency hell.
Alternatively, we could also have a thread pool at the scale of the Index, and use it by default.
The trouble then is that we could imagine a world where people want to run different queries on different thread pool and possibly single threaded too.
What's your take on this?

@drusellers added an executor.

@hntd187 Nice catch, thanks.

Question/suggestion re: Schema API since you are changing from SchemaBuilder::default to Schema::Builder
The last call to schema_builder is schema_builder.build(), which is a bit too verbose IMHO.
This PR will break the API. Additionally, the examples folder includes user-facing code, which will form users' impression of the project/codebase, it would be great to discuss a change like to the schema::build() calls.
We can change the schema::build to something like finish, complete or finalise, eg.
schema_builder::finish()
Alternatively, we can s/schema_builder/schema in all examples, which will make the last call site of schema::build() concise and to the point.
Feel free to disregard, if you think it's a minor nit-pick. The rest of the PR looks great - thanks for the continuous work

@petr-tik I readded the Default implementation to make sure that we stay backward compatible.
I have no idea what you mean about the verbosity of schema_builder.build().
I don't care about build vs finish.... Either way I don't think this is worth the change.

As I said above, feel free to disregard - I felt it was minor


Fix for retrieval of content by type when rebuilding Algolia index and blocks list value getter

The current code that is getting a list of content by type is not (always) working, this is an Umbraco bug which causes the following call: var contentItems = ctx.UmbracoContext.Content.GetByContentType(contentType) to return an empty list for some content types (as far as I know, it is connected to types that are composed of other types). In order to avoid this problem, I switched to using GetPagedOfType from ContentService.
As a side effect, since the new code makes use of IContent and not IPublishedContent, we can remove the extra code that was added in this PR: https://github.com/umbraco/Umbraco.Cms.Integrations/pull/140 by @gardarthorsteins
It is also worth mentioning that the removed code had an extra bug in AlgoliaSearchPropertyIndexValueFactory's KeyValuePair<string, string> GetValue(IPublishedProperty property, string culture): it used property.GetSourceValue(culture) in order to get a value to index but as contrary to GetIndexValues from IPropertyIndexValueFactory, it did not return correct data to index for the data type of Umbraco.BlockList which is another reason to get rid of the new method of GetValue for IPublishedProperty. Additionally, the method introduced some code duplication.
As the mentioned PR from @gardarthorsteins attempted to fix performance issues, the new version is handling them by making use of an injected PropertyEditorCollection in AlgoliaSearchPropertyIndexValueFactory so there is no need to make multiple calls for data types and their editors from IDataTypeService. I tested the performance on indexing over 9000 nodes which is taking on average 12 seconds on my local machine so around 1.3 millisecond for entry

Hi @AndyButland , would you mind checking out this PR? The other PR (https://github.com/umbraco/Umbraco.Cms.Integrations/pull/154) that you just accepted makes use of the code here in this PR so it'd great to have it merged, it also fixes this issue: https://github.com/umbraco/Umbraco.Cms.Integrations/issues/149

Hi @piotrkrajewskicn ,
I have prepared a new version of Algolia to be shipped out today, that contains some refactoring of the converters, and I have manually added some of your changes.
I will update here once the new version is on NuGet.
Thank you,
Adrian

Version 2.0.0 has been released, and it contains a refactoring for the converters using collection builder (more details here).
I have manually picked changes you've recommended.
Thank you for your contribution.
Adrian


Close while compiling

What is your question?
when compiling for the first time, what happends if i need to stop it and close the cmd

it stops either way


zlib: unexpected end of file

Hey! Thanks a lot for helping degit live on.
There's an issue that happens both in degit as well as tiged which can get quite annoying, so I'll open it here too if you don't mind.
From @natemoo-re:

We're using degit to power create-astro and a number of users have reported that the CLI throws the following error:
zlib: unexpected end of file

It looks like a workaround is to clear the cached files in ~/.degit, but this is confusing because we're explicitly setting cache: false.
I'd be happy to contribute a PR to fix this if I can narrow down the problem!

Duplicate of: https://github.com/Rich-Harris/degit/issues/313

Yes, PRs are always welcome here. :) I can also look into this. But I make no promises :D
I have a ticket here that might be related: https://github.com/tiged/tiged/issues/36

Fixed with #36
You can try noCache: true or on the command line --no-cache.
Left cache setting as is as not to break existing functionality. It is however deprecated and will be removed in next major version. Use offlineMode: true or --offline-mode instead.

update from #36:

I'm unhappy how this works currently.
Will do a patch version hopefully out today. Technically it all works. But the way tiged parses the opts is not nice. Hence my edited previous post (offlineMode to offline-mode and noCache to no-cache). I definitely want it to work as offlineMode and noCache in the programmatic use.

Will keep this ticket closed and work done on #36. Just thought I should let people know.

I take that back. This functionality is quite broken until I fix ticket #36.

Fixed it. Just use --disable-cache argument or programatically disableCache: true


mv, cp: add support for --update=none

#4627 cover mv/update
test still failing because of diff in err messages

i know it is the draft but don't forget the tests please :)


Adds support for site app catalog for app get, list, add, remove and deploy

Adds support for site app catalog for app get, list, add, remove and deploy solving part 1 from #405
This will allow users to deploy and remove apps with skip feature deployment and I will complete the rest of the commands (install, retract, uninstall, upgrade) in the following weeks.
@waldekmastykarz, please do not forget to give the credit to @andrewconnell  :)

Thanks! Will definitely do!

Machine! I'll have a look shortly. Thank you! 👏

Makes sense and it will be more clean.  @waldekmastykarz,  I can do that shortly.

@waldekmastykarz, siteUrl is replaced now with appCatalogUrl in the last commit https://github.com/pnp/office365-cli/pull/653/commits/6554ca12f4b698499e1b6693fe60c4d1480b1dbf

Coverage remained the same at 100.0% when pulling 6554ca12f4b698499e1b6693fe60c4d1480b1dbf on VelinGeorgiev:alm-appcatalog-site into 52349d3e700b9a0e769c736588bb7dcbcfe6e7a7 on pnp:dev.

Going through your latest changes, I realized something pretty confusing as a result of my suggestion to change siteUrl to appCatalogUrl. In other commands, where we already had appCatalogUrl it was meant to point to the tenant app catalog site, not the actual app catalog list. In the context of a site collection app catalog however, this is confusing, because the name appCatalogUrl indicates, as you correctly assumed, the URL of the catalog list, not the URL of the site. But, because the app catalog is always called AppCatalog, there is no point in including it in the URL and we don't need it for anything (we're actually removing it from the URL). So thinking about it, I'd say, we should go back to your original suggestion and use siteUrl instead.
Terribly sorry for the trouble and not thinking it through originally. I'll adjust it myself when merging the PR as I don't want to cause you any more work again, but just wanted to let you know why things will be different in the merged code and avoid any surprises.

Thank you @VelinGeorgiev!

@waldekmastykarz I had to do small refactoring to make sure the right site has the right accessToken. As a result of my refactoring:

I've created an app-base.ts class with shared app methods.
I've moved one method from SpoCommand to the app base class since it was used only by the app commands
Search query to find app catalog in the getTenantAppCatalogUrl method is replaced by client.svc query .This removes edge cases where the app catalog is not indexed or has set NoCrawl to true. Having that in place removes the need of additional prompt when search does not find the app catalog and I removed it.

Thanks! Have you considered using the REST API to get the tenant app catalog URL as I described here: https://blog.mastykarz.nl/get-sharepoint-tenant-app-catalog-url-rest-api/?

@waldekmastykarz , I just pasted what we had in the tenant get appcatalog command, but I can change it, should not be a problem. Would you like me to change it in the app catalog command as well. I can create a shared method and use it for both.

If you could first verify, that it indeed works when the app catalog has not yet been indexed, then let's do that indeed.

@waldekmastykarz , here is what I have found.
Search, Client.svc and OData do not have advantage over each other. Approx. time for all is like 5-10 mins after the app catalog is specified.

Search requires crawler to index the app catalog first.
Client.svc and the OData services try to get data from the tenant settings and work in a similar way, but there seems to be a cache expiry interval per web api instance.

The search approach in some cases can index the app catalog faster, but in is not guaranteed. This is because there seem to be a cache on the front ends servicing the OData and Client.svc services that has to be invalidated per web api instance. When I hit the OData enpoint I randomly get NULL or valid url meaning that I am hitting different api instances some of which have not invalidated the cache. As result if the rest apis are scaled to 3 api instances the cache can be invalidated to 2 of 3, but if  you hit the 3rd you get NULL result.
Anyway, we are fine in 5-10 minutes after the app catalog is specified.
Please let me know if I should change it to use the REST endpoint?

Thanks for the detailed findings @VelinGeorgiev. Let's switch to REST for consistency and simplicity. Sorry for the last minute additional work this is causing 😊

No worries @waldekmastykarz. I wanted to simplified anyway.

In the deploy, get and remove commands, why not use the existing appCatalogUrl option for specifying the URL of the site collection app catalog? Right now we have two options for the same purpose but one exclusive to tenant app catalog and the other to the site collection app catalog. Using the appCatalogUrl option for all cases and defining the scope using the scope option would be easier, don't you think?
If we'll go with that approach, then I'd suggest consistently using the appCatalogUrl option in all app commands.

@waldekmastykarz, I am confused now. Have we not discussed that up in the same conversation? See: https://github.com/pnp/office365-cli/pull/653#issuecomment-439182529
Could you please give me two examples (code snippets) of how the get command would look like when scope sitecollection and when scope is tenant?

Sorry, my bad. Seems like I changed my mind without considering what we discussed. Let's keep everything as-is. Sorry for the confusion.

@VelinGeorgiev you are right and that's what I've been thinking as well. What confused me initially was the piece related to removing the list name from the URL but now I get that it's meant more as a precaution in case someone included it. Either way, it would make perfect sense to have one option to specify the app catalog URL no matter if it's tenant or site collection and then use the scope option to specify what kind of app catalog the URL is pointing to.
As I mentioned, I've done some fixes on your PR. It's not merged in the dev branch yet, as I noticed some tests to be broken and which I was just fixing. Since I'm working on the PR, I could either take what you suggested above and change in the commands or I could push the temp branch I'm working on right now and you could apply the fix there (with or without fixing the broken tests). I caused enough trouble so far going back and forth, so I'd be totally fine with fixing the mess myself 😊

@waldekmastykarz, additional suggestion:
Another area of change can be to completely drop the tenant app catalog url as option because you can have only one active at a time and we can auto-discover it. Then the would not be a need to have --appCatalogUrl, but just --siteUrl option when the scope is site.
Here is my very aggressive suggestion:
Tenant:
o365 spo app list

Sitecollection:
o365 spo app list --siteCatalog https://contoso.sharepoint.com/sites/velindev

or
o365 spo app list --siteCatalog https://contoso.sharepoint.com/sites/velindev/AppCatalog

I'd prefer to keep the appCatalogUrl for two reasons:

dropping it means a breaking change, means a major release, which I'd prefer to avoid for such a relatively small change
when you build a script, discovering the URL of the app catalog once and passing it, speeds up the script, also it's a necessity in case you have a new tenant and can't get the URL from the API

Good points. Let's then NOT go so extreme. :)


Section titles are not hi-lighted

When using language-gfm, titles such as:
## Title get syntax hi-lighting whereas with this package does not.

Can you post a screenshot showing the difference? And can you tell me what syntax-theme you are using?

My apologies. I did not read that only certain syntax-themes worked.
I was using Nebula, but It works with a supported syntax theme such as Pubster

No problem. If you point the author of a syntax-theme to this repo, it should be trivial for them to add support for this package.

I'll do that.


Making the tests pass on single GPU platforms.

These tests fail on single GPU machine (which I guess is most devs setups).
This adds a simple device count check before running the test.

Alternatives:

Use a feature flag (seems overengineered).
Add #[ignore] (but then we're discarding those tests basically).
Use and environment variable.

I feel like the proposed approach is simple enough we actually get the testing run on valid platforms.
If there's a need for much more multi GPU tests, then maybe the solution should evolve.

@coreylowman

Yep great minimal solution, thanks 👍


fix: Editing via Pencil can now change auto-scheduled date

Description
Allows a scheduled date that was set automatically to be modified using the edit icon.
Motivation and Context
Fixes #1300
How has this been tested?

Added unit tests for the logic.
Verified that the scheduled date can be modified using the edit button
Verified that there are no regressions when modifying a task using the edit dialog

Types of changes

Changes visible to users:

[x] Bug fix (prefix: fix - non-breaking change which fixes an issue)
[ ] New feature (prefix: feat - non-breaking change which adds functionality)
[ ] Breaking change (prefix: feat!! or fix!! - fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation (prefix: docs - improvements to any documentation content)
[ ] Sample vault (prefix: vault - improvements to the Tasks-Demo sample vault)

Internal changes:

[ ] Refactor (prefix: refactor - non-breaking change which only improves the design or structure of existing code, and making no changes to its external behaviour)
[x] Tests (prefix: test - additions and improvements to unit tests and the smoke tests)
[ ] Infrastructure (prefix: chore - examples include GitHub Actions, issue templates)

Checklist

[x] My code follows the code style of this project and passes yarn run lint.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] My change has adequate Unit Test coverage.

Terms

[x] My contribution follow this project's contributing guide
[x] I agree to follow this project's Code of Conduct

Awesome. Thank you. Will review and release this as soon as possible.

@mvdkwast Fabulous. Thank you very much indeed for fixing that so quickly, and adding the tests! 🎉 ❤

Thanks for post-processing @claremacrae :)

I updated Tasks to 1.18.1, closed and re-opened Obsidian, and can confirm that editing scheduled date with the pencil icon works now. Thank you both.

Oooh - thanks for checking @bayradberry !


Create a new parameter \Phi to distinguish from the previous \phi?

It's a little confusing on page 41. Having \phi represent two things will likely cause confusion in the meeting.
If we switch then we should edit the SRG report's \Phi to \phi (page 44), to avoid real confusion.
Might involve a few changes, so thought I'd open for discussion.
And since the epsilon's in (5) and (7) are not the same thing, I suggest appending a ' to one of them. I thought the one in (5) since only appears once.

I already changed the \Phi to \phi in the SRG recommendation (figuring that the difference wasn't intentional).
I've struggled with the old/new meaning of the \phi and indeed it's confusing, so using \Phi would be fine. The epsilon's in (5) and (7) have different meaning in terms of the impact on the model, but in terms of parameters that we report in Table E.1 with names starting with "AgeSel_P3_Fishery.1._DEVadd_1991", they are identical. Nevertheless, since we never connect E.1 to the equations, adding a ' makes sense to separate them, especially if it's consistent with a change in the \phi notation.
Note that \phi also occurs frequently in the sections on bridging and sensitivity analyses among other areas.
I can help with this tomorrow if you wish.

Note that I never added this figure (previously posted to https://github.com/cgrandin/hake-assessment/issues/323#issuecomment-361778573)

The simple code to create it is in https://github.com/cgrandin/hake-assessment/blob/master/doc/r/r-functions/selectivity-transformation-notes.r and could be augmented or captioned to note that the old phi/epsilon was in the vertical dimension, while the new Phi/epsilon' is in the horizontal dimension.

I think we're done with this (I changed a bunch yesterday, making sure to keep \phi when referring to 2017 model). The figure above can be shown if anyone requests more explanation.


Make mocks independent at subtest level

Summary
Some of our tests use shared mock instances across subtests within a test suite. This could cause expectations set from subtest A still leave open in subtest B if it's not fulfilled in subtest A. This is due to a limitation from testify that before/after test hook is not available for subtest. Currently, we avoid that issue by ensuring the number of expectations matches the actual executions using .Once(). But a similar issue still could happen in parallel tests.
The second issue is in a complex function that makes many calls from dependencies, we had to set the expectations for all executed calls in each subtest resulting in the code being not DRY and making the test harder to read/understand.
Proposed solution
In testify v1.8.2, before and after test hooks are now supported for subtests. We can use this to ensure every subtest starts with empty expectations.
To also fix the second issue as well as for the parallel testing, mocks need to be decoupled from the suite struct and can be initialized at anytime when needed.
Proposing following approach:
package appeal_test

type mock struct {
	mockRepository *appeal.Repository
	mockXService   *appeal.XService

	service *appeal.Service
}

func newMock() mock {
	m := mock{
		mockRepository: new(mockRepository),
		mockXService:   new(mockXService),
	}
	m.service = appeal.NewService(m.mockRepository, m.mockXService)
	return m
}

func (m *mock) setSuccessExpectations() {
	// always expect mock.Anything in params and return success values
	m.mockRepository.EXPECT().Create(mock.Anything, mock.Anything).Return(nil)
	m.mockXService.EXPECT().Create(mock.Anything, mock.Anything).Return(nil)
}

type AppealTestSuite struct {
	suite.Suite
	mock // shared mock
}

func (s *AppealTestSuite) SetupSubTest() {
	s.mock = newMock() // reinitialize mock for each sub test
}

// usage example
func (s *AppealTestSuite) TestCreate() {
	s.Run("repository returns an error", func() {
		s.mockRepository.EXPECT().Create(mock.Anything, mock.Anything).Return(errors.New("error"))
		_, err := s.service.Create(context.Background(), appeal.CreateRequest{})
		s.Error(err)
	})

	s.Run("test specific case", func() {
		s.setSuccessExpectations() // set common expectations

		// only specify expectations related to the tested case:
		s.mockXService.EXPECT().Create(mock.Anything, mock.Anything).Return(errors.New("error"))

		_, err := s.service.Create(context.Background(), appeal.CreateRequest{})
		s.Error(err)
	})

	s.Run("parallel tests", func() {
		testCases := []struct {}{
			// ...
		}

		for _, tc := range testCases {
			tc := tc
			s.Run(tc.name, func() {
				s.T().Parallel()

				independentMock := newMock()
				// set expectations for independentMock
				// ...
				//_, err := independentMock.service.Create(context.Background(), appeal.CreateRequest{})
			})
		}
	}
}

decoupled mocks from suite struct into an independent struct
restart mocks in every subtests s.mock = newMock()
for parallel tests, use an independent mock instead of a shared one
introduced setSuccessExpectations() to avoid writing common expectations multiple times across subtests

Note: currently trying out this approach for appeal service as a POC, will raise the PR sson

func (m *mock) setSuccessExpectations() {
	// always expect mock.Anything in params and return success values
	m.mockRepository.EXPECT().Create(mock.Anything, mock.Anything).Return(nil)
	m.mockXService.EXPECT().Create(mock.Anything, mock.Anything).Return(nil)
}

I don't think we should have something like this which always expects only mock.Anything. We can try to abstract common expectations, but that should not come at a cost of proper expectation checks.


no initial delay for discovery

don't wait 10 minutes (initial delay) but start DNS discovery immediately
PR description
Fixed Issue(s)
replaces #3

not needed any more since this change is done in Besu


#2 change ceil to floor

Coverage remained the same at 100.0% when pulling 6dceca81713f381e8225918f61a6accc67033424 on nathanblogs:issue-2 into 4f21bfbb8bad39fde106c67fb6bb7bef72067992 on HealthEngineAU:master.

Coverage remained the same at 100.0% when pulling 6dceca81713f381e8225918f61a6accc67033424 on nathanblogs:issue-2 into 4f21bfbb8bad39fde106c67fb6bb7bef72067992 on HealthEngineAU:master.

Coverage remained the same at 100.0% when pulling 6dceca81713f381e8225918f61a6accc67033424 on nathanblogs:issue-2 into 4f21bfbb8bad39fde106c67fb6bb7bef72067992 on HealthEngineAU:master.

That's quite annoying that coveralls sent 3 messages with the same comment, all from only one commit. I wonder if its a misconfiguration on our end or a bug in coveralls.
Anyway, looks pretty good to me as is but what do you think about also subtracting 60 from the input seconds. The default expiry of credentials from STS is 15 minutes so I think that is still reasonable to only cache for 14 minutes.
Can you also update the description of this PR to reference the issue, so it will close when this is merged


migrate part of painting to nullsafety

Description
NNBD migration for a part of painting
Related Issues

dart-lang/sdk#40146

Tests
I added the following tests:
None
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I signed the CLA.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Did any tests fail when you ran them? Please read Handling breaking changes.

[x] No, no existing tests failed, so this is not a breaking change.
[ ] Yes, this is a breaking change. If not, delete the remainder of this section.

[ ] I wrote a design doc: https://flutter.dev/go/template Replace this with a link to your design doc's short link
[ ] I got input from the developer relations team, specifically from: Replace with the names of who gave advice
[ ] I wrote a migration guide: https://flutter.dev/go/breaking-changes-template Replace this with a link to a pull request that adds the migration guide to https://flutter.dev/docs/release/breaking-changes

/cc @dnfield Since you're very familiar with the image caching logic, can you take a loo at this as well?

PTAL

This commit appears to be breaking a test: https://cirrus-ci.com/task/5991685194776576

I just saw but I really don't understand what's happening :-( I haven't changed the assert pointed at https://cirrus-ci.com/task/5991685194776576?command=main#L1707 and I had no issue in the PR CI :-/

Let's roll it back for now to fix the tree: https://github.com/flutter/flutter/pull/62868
Does this reproduce locally?

Does this reproduce locally?

Yes. The question is why textDirection is now null???

Looks like the test is supposed to fail and we just need too adjust the expectation: https://github.com/flutter/flutter/blob/49dc9b1227bb78df7c03a78b2a65e34d017b0e15/dev/automated_tests/flutter_test/print_user_created_ancestor_expectation.txt

(line numbers probably changed)

--> https://github.com/flutter/flutter/pull/62872 potential fix, still running locally to verify.


iOS/iPadOS Webclip icons

Add support for Web Clip icons within iOS/iPadOS when adding Transmissionic to home screen.

Hi Sorry, I attempted to create a pull request to add the above functionality (see title) but I think I hit a problem. Apologies.  Let me know if this is something I can do.
Thanks

Hi, no problem I am not very familiar with GitHub as well. This happened because you tried to merge the dev branch to the main.
I think you need to fork the dev branch, make your changes on the fork, and then create a pull request to the dev branch from your fork.


[17.09] Overlay fix for transient IP reuse

Backport of https://github.com/docker/libnetwork/pull/1935 for 17.09.
To get a clean cherry-pick, I also included https://github.com/docker/libnetwork/pull/1752, which brings no change in behaviour, so looked safe to include.


Mylesson5

Thanks

Thanks for you😃
بتاريخ ٢١/١٢/٢٠١٧ ١:٤٦ م، كتب "Mahmoud Samir Fayed" <
<EMAIL_ADDRESS>
Merged #450.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


[APPM-1582] Add integration test for "configurable subscription model"

Reference: https://wso2.org/jira/browse/APPM-1582

Not related to IoT server, hence closing the issue.


test suite, selector api doc, and wish list

@aliirz
I've added a short section on the README listing the supported selectors (and future selectors, maybe...), and created a tiny test suite.html to exercise them.  There's patched assert method in the suite as well.  No 3rd party test lib used.
On the README there's a link to the test suite viewable as a rawgit page ~ you'll need to modify the href to point to aliirz instead of dfkaye

Wow Thank you. i am gonna test and merge this tonight

I get the error Uncaught TypeError: Illegal invocation on the line
    !!ok || (typeof fn == 'function' || (fn = console.log)) && fn('fail: ' + msg);

When trying to run the test suite.

I've tried another pull request or update ~ not sure it made it.
If it didn't, just change line 32 to this:
!!ok || fn && fn('fail: ' + msg) || console.log('fail: ' + msg);

Fixed. Cheers!


[BUG]  error upon installing

Acknowledgement of preliminary instructions

[X] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.
[X] I have thoroughly looked through the available Wiki articles and could not find a solution to my problem.
[X] I am using the latest version of Bloxstrap.
[ ] I did not answer truthfully to all the above checkboxes.

Bloxstrap Version
v2.1.4
What problem did you encounter?
it  has an error as soon as i try to install what it has to offer and i need a fix for it so it does not happen again
Bloxstrap Log
[Bloxstrap_20241213T213455Z.log](https://github.com/user-attachments/files/18132092/Bloxstrap_20241213T213455Z.log)

Update to the latest edition, where did you even get 2.1.4 lol


DraftJS with s3 image upload

Hello,
Is it possible to use this editor for the following;

h1-h6 styles, bold, italic and links buttons (Appears so)
Allowing images to be added, and aligned but the image should be uploaded to an s3 bucket as part of the saving process

If not, is there any other editors that work with ReactJS and could do this? My stack is NextJS + Material-UI + Hapi.
Many thanks... I did look through the awesome draft repo, but I couldnt determine where images went.

You need to write your own handler for image uploads or use some of the community plugins.
In reality, if you're looking for a more robust solution for a text editor, you should check out other packages, as this one is barely maintained and requires lots of configuration to make it work.

Check out https://www.draft-js-plugins.com/, they seem to have a plug-in for adding images. I haven't used them myself, but the Alignment + Resize + Focus + Drag'n'Drop + Drag'n'Drop Upload Example section sounds like what you need.

Hey @bonbonio. I made a quick plugin for draft-js called draft-js-aws-amplify-s3 that may help you.

Hey @bonbonio. I made a quick plugin for draft-js called draft-js-aws-amplify-s3 that may help you.

I’ll take a look this weekend! Awesome news though!

@bonbonio Great. It's super young so don't be too harsh. :stuck_out_tongue_closed_eyes:


Documentation for Data Request Issue Form

Issue/ticket: https://github.com/NASA-IMPACT/active-maap-sprint/issues/724
Updated issue form: https://github.com/MAAP-Project/Community/blob/master/.github/ISSUE_TEMPLATE/data_request.yaml
Creating a notebook for the Docs site to show users how to request new datasets using the updated issue form.

I think this will close out the last open ticket for Objective 4 once this gets approved

Thanks, y'all!


Allow multiple IO callbacks or throw

Currently, multiple IO callbacks are ignored, only the first one will work. #86 fixes this to be consistent, but silently ignoring should be changed to either throw exceptions or call all handlers and not ignore them.

I agree that adding multiple listeners to the same resource makes little sense and should not happen in a real world program :+1:
Throwing an exception in this case would be the obvious choice, does anybody feel like filing a PR? :shipit:


fetch latest dictionary

update fetch_dictionary.sh

download latest Sudachi dictionary
take version and dict_type as arguments (default: latest / core)

No, the downloaded dictionary (at least in CI) should be fixed because the CI tests the tokenization against fixed results.

No, the downloaded dictionary (at least in CI) should be fixed because the CI tests the tokenization against fixed results.

I missed that point.
We can do either of

make the default dictionary-version of fetch_dictionary.sh fixed
let CI scripts call fecth_dictionary.sh with fixed version

I prefer the latter since it's more explicit.


Polymail on Linux?

Hi all. I successfully created a 'nativefied' Polymail, however the only thing lacking were the system notifications, which are pretty desirable for it being a mail client. I have tried the slew of configuration parameters, to no avail, so continuing to just run it in a browser tab still.
OS: Pop!_OS 20.04 LTS
Has anyone had luck with Polymail on Linux with closer full functionality?

@phodamentals

Please fill the information requested by the issue template
What else apart from notifications is missing?


Fix logic flaw in multi tag handling

Example:

"Hide Tags": "tag1,tag2"
"Initial display state": "Show"
"Display preference": "Hide"

If the visitor only has "tag2", the element won't be hidden since not matching "tag1" triggers the "break" of the whole loop, rather than just this aspect.

Finally got back to putting a test in place to capture this, have resolved in 1644238 to resolve compiled conflicts


[WIP] deepin.deepin-manual: init at 2.0.19

Motivation for this change
Add the deepin-manual package.
About packaging deepin for NixOS: https://github.com/NixOS/nixpkgs/issues/59023
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Building is failing with the messages:
[...]
Scanning dependencies of target update-search-db
[ 47%] Generating _update-search-db
/build/source/build
QSqlDatabase: QSQLITE driver not loaded
QSqlDatabase: available drivers:
Failed to open search db: "/build/source/build/src/search.db"
QSqlQuery::exec: database not open
[...]
-- Installing: /nix/store/fpyjyaqyn21s3bm5211zxbh1b54qbw2p-deepin-manual-2.0.19/share/deepin-manual/web_dist/qwebchannel.js
CMake Error at src/cmake_install.cmake:145 (file):
  file INSTALL cannot find "/build/source/build/src/search.db".
[...]

Still important.

Packaging of the Deepin Desktop Environment has been cancelled (https://github.com/NixOS/nixpkgs/issues/94870).
Closing.


Enable IntertBraces in clang-format

Description
Enable expand brackets clang-format for better readability.
https://github.com/stackrox/collector/pull/1851

So the lint workflow is now broken because this line:
https://github.com/stackrox/collector/blob/3f10453b255cba73f85a5b678eb9ec663c87d062/.github/workflows/lint.yml#L21
Is trying to install packages using pip on a system that has python managed by the OS (this is actually a good thing, a lot of systems have been broken by pip installing packages over the actual system ones).
Additionally, the pre-commit/action workflow is now in maintanance mode and they suggest using pre-commit-ci instead, but that looks like a tool that fixes commits and is not really what we want, we want to check PRs, we don't want to have additional commits pushed on us.
I suggest we change the pre-commit/action line for the following block instead:
      - name: Run pre-commit
        run: |
          python3 -m venv .venv
          source .venv/bin/activate
          python3 -m pip install pre-commit
          pre-commit run --all-files

Not as convenient as a prebuilt action, but it's only 4 lines of bash, so 🤷🏻♂.

So the lint workflow is now broken because this line:
https://github.com/stackrox/collector/blob/3f10453b255cba73f85a5b678eb9ec663c87d062/.github/workflows/lint.yml#L21
Is trying to install packages using pip on a system that has python managed by the OS (this is actually a good thing, a lot of systems have been broken by pip installing packages over the actual system ones).
Additionally, the pre-commit/action workflow is now in maintanance mode and they suggest using pre-commit-ci instead, but that looks like a tool that fixes commits and is not really what we want, we want to check PRs, we don't want to have additional commits pushed on us.
I suggest we change the pre-commit/action line for the following block instead:
      - name: Run pre-commit
        run: |
          python3 -m venv .venv
          source .venv/bin/activate
          python3 -m pip install pre-commit
          pre-commit run --all-files

Not as convenient as a prebuilt action, but it's only 4 lines of bash, so 🤷🏻♂.

Let's try it.

Let's try it.

Well it sorta worked, looks like the clang-format step failed, maybe we need to rebase this PR so the changes you did to the braces are picked up? Also, could you add --show-diff-on-failure --color=always to the pre-commit command in the lint workflow? That should help see what the differences actually are.
There's also a lint error I was not expecting here:
https://github.com/stackrox/collector/blob/3f10453b255cba73f85a5b678eb9ec663c87d062/integration-tests/container/perf/scripts/init.sh#L143-L146
That block seems unreachable because the previous if block does exit  0 on both of its branches. @Stringy, can we remove the block or should we move it above the previous one?

Discussed offline and decided I'll be taking over the PR, @SimonBaeumer sorry your small change got so out of hand 😅

Looks like konflux is gonna time out, but we don't really care about it, so I'm merging as is.


Add vision and finetuning experiments; extend pretrained checkpoints experiment to trigrams

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: luciaquirke:x: davidojYou have signed the CLA already but the status is still pending? Let us recheck it.


Migration: DB deadlock

while try to migrate from gitea ( 1.14.0+dev-63-g26d6c1530) to gitea (1.14.0+dev-44-gf0fe5683f) a db deadlock ocure (mysql)
source repo: https://try.gitea.io/abcKing/GitNex

Dude!! Seriously this is a bad bad bad issue report!
I presume you mean that you were trying to migrate the repository https://try.gitea.io/abcKing/GitNex from try.gitea.io to a local server and the local server deadlocked? Or do you mean that try deadlocked?
I'm guessing you meant your local server's database is MySQL? (I note try uses MySQL too)
Do you have any logs around this point?
Are you sure it's a DB deadlock and not some kind of http loop or other code deadlock? MySQL should have a deadlock breaker meaning that at least one of the deadlocking queries/transactions should get broken.
Have you repeated the procedure with:
``ini
[database]
LOG_SQL=true
...

to get the SQL logs for this?

@zeripath try uses sqlite, so it's easy to let the DB deadlock.

I'll shuld be more specific :sweat: - but do not have logs at the moment, so I only can gess ...
... I'll work on this issue - and if I konw more I'll let you konw

no worries - it's just that if a user had made this report we'd be all stuck scratching our heads!

@zeripath I have an error leged :tada: - thanks for the error screen on migrations :)
Error 1062: Duplicate entry '+1-13037-0-1--1' for key 'UQE_reaction_s'

so I guess: since source repo is already a migrated repo: api returns some entrys who make no sence
the question is how to treat them: drop/show as GhotUser/ ?!?


blog support

simplcommerce is good cms But needs this features:
1- Blog  and forum
2- Strong multivendor
3- page builder like wordpress
4- better loading speed
5- full document and tutorial
6-  calculation system
7- black friday system
8- realtime search
9- more filter option like woocommerce
11- mobile app with react native

SimplCommerce is not a CMS its an eCommerce solution, but it has a basic CMS module out of the box, some of what you suggested are suitable, and some are not

At the moment SimplCommerce does not have blog and forum support which would be ideal for an ecommerce platform such as adding posts where users can land when clicked on an email link of a newsletter.
2. About your complain of strong multivendor, i have also thought about that and it was a headache to me. On my store, I want users to become a vendor by themselves instead of the admin making them become one an in that way, they can create warehouses of their own. Anyway I'm developing a workaround to this where i'm building that as a seperate module. Hope this works out. Or I'll rather implement it in the source code. You can also do so if you can.
4. I'm also greatly concerned about the loading speed of the application as its relatively slow. I don't know why and .Net Core is not known for this kind of behaviour. The angular aspect also does not give a good user experience when loading content. I'm just trying to get to the source code and implement that for myself.
9. I'm also seriously disturbed about filter options. this gives the user more friendly browsing experience.


What do we need 1.23 for?

bytecount will currently build with Rust 1.20, but will no longer be able to benchmark with criterion (I have a PR open that I'll need to extend to make criterion optional so we can at least test.
So I wonder: What do you need the current version for?

Criterion.rs uses the handlebars crate to generate HTML. That, in turn, uses pest, which requires 1.23.
I didn't realize this until after I released 0.2, but the (unwritten) policy has always been that Criterion.rs is only tested on the latest stable release. Now that it's become a problem for some users, I'll probably look at changing that policy going forward.
I'm not sure what the usual compatibility policy in the Rust ecosystem is. What would you suggest?

Perhaps split the data collection and report generation in two? This would also allow us to e.g. plot timelines over multiple versions in the future.

That's definitely in the long-term plan, yes.
In the short term, I suppose I could make the HTML reports a default Cargo feature in 0.2.1. That way, you could at least disable it.

That'd be great for CI. Thanks.

In the short term, I suppose I could make the HTML reports a default Cargo feature in 0.2.1. That way, you could at least disable it.

While I wholly support this goal, please note that introducing default features is technically a breaking change for anyone using default-features = false. (In this case, I would expect the impact to be minimal, because it seems criterion doesn't have default features so far, so nobody should be using default-features = false.)

Technically that's true, but I'm not too concerned about breakage with this change - no public API would change so the only way this could break code is if someone had disabled default dependencies and written scripts that expect the existence of the HTML reports, which seems pretty unlikely.

Thank you! I've updated my branch to test this, it appears to work fine.


lets allow the http transport to be supplied by the caller

Can one of the admins verify this patch?

ok to test

[test]


Dependency error in Account Service

Error: Error creating bean with name 'accountService': Unsatisfied dependency expressed through field 'UserRepo'; nested exception is org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'userRepository': Invocation of init method failed; nested exception is org.springframework.data.mapping.PropertyReferenceException: No property firstName found for type User! Did you mean '_firstname'?

Now fixed. Get user by first name also OK now. Available on service-a.


Reflect metadata only for included tables

Reflecting all tables is unnecessary when include_tables is provided. It's time-taking and prone to errors.

i believe https://github.com/hwchase17/langchain/pull/2216/files fixes it

i believe https://github.com/hwchase17/langchain/pull/2216/files fixes it

Yes, it does


Close button

Close button is not showing

For me the close button is not showing up too. Any idea what I'm doing wrong ? I use ImageViewer as a pod.

Sorry @MDittmann the pod version is a bit old.
The close button should be there in the pod version but the image on the button is tied to hardcoded image names (close_highlighted and close_normal). See https://github.com/MailOnline/ImageViewer/blob/4.0/ImageViewer/Source/Extensions/UIButton.swift#L98
Waiting for @RuiAAPeres to release a new version to Cocoapods as only he has access to it atm.
Alternatively you can specify the repo as the source of the pod, something like:
pod 'ImageViewer', :git => 'https://github.com/MailOnline/ImageViewer.git'


allow for execution without a control

Describe what enhancement you'd like to see.
We currently allow for MACS2 to run without a control - may have situations were a control is not available or it failed sequencing. Valuable data can still be obtained for such a scenario. Because of this, an adjustment should be made to allow for running without a control

When needed, we run the pipeline by including a dummy IgG control in the sample matrix (as a sample entry = row and listed as the control for each sample). It is included in the MultiQC report and the DeepTools plots, but otherwise output is separate from that of the true samples.


feat: allow visibility modifiers in struct definitions

Description
Problem
Part of #4515
Part of #4775
Summary
Allows marking structs as pub or pub(crate). Also will now report unused structs.
Additional Context
Let me know if some of the structs I marked as pub shouldn't actually be pub. I didn't mark some as pub, I think only the ones used for tests.
Documentation
Check one:

[ ] No documentation needed.
[x] Documentation included in this PR.
[ ] [For Experimental Features] Documentation to be submitted in a separate PR.

PR Checklist

[x] I have tested the changes locally.
[x] I have formatted the changes with Prettier and/or cargo fmt on default settings.

Actually, the unused warning isn't super correct right now. If you have struct Foo then impl Foo it will consider Foo as being used. That said, the warning will never give a false positive, and this PR's intention is to enable visiblity. We can improve the unused warning later on.


Error loading local .tflite model in Android release build when Expo Updates are enabled.

I noticed that ze release build didn't load ze model correctly in Android when loading models using
  const model = useTensorflowModel(
    require("../assets/ai-models/ze-most-amazing-model.tflite"),
  );

I created a minimal project using just react-native-fast-tflite and ze release builds were working fine. So to check what's causing ze error, I kept on rebuilding ze release app adding one dependency at a time.
After 2 days I came to ze conclusion that something in expo updates is causing ze Android release build to have error loading ze local model.
It gives the error message
Error: java.lang.IllegalArgumentException: Expected URL scheme 'http' or 'https' but was 'file'
My app.json
{
  "expo": {
    ...,
    "extra": {
      "eas": {
        "projectId": "my-real-project-id"
      }
    },
    "updates": {
      "url": "https://u.expo.dev/my-real-project-id"
    },
    "plugins": [
      "expo-router",
      [
        "react-native-vision-camera",
        {
          "cameraPermissionText": "$(PRODUCT_NAME) needs access to your Camera.",
          "enableMicrophonePermission": true,
          "microphonePermissionText": "$(PRODUCT_NAME) needs access to your Microphone."
        }
      ],
      [
        "react-native-fast-tflite"
      ],
      [
        "expo-updates",
        {
          "username": "my-username"
        }
      ],
    ]
  }
}

After removing ze expo.updates property from app.json, I was able to get ze model loading properly in ze release build but expo updates may not work using this workaround.
Is there any way to have ze local model load fine along with expo-updates in android release builds?

Hi @mrousavy,
Could you please take a look at this issue, do let me know if I would need to provide mode information.

hey - hm not sure, file should be supported as of the latest version

actually maybe i forgot something - try again with 1.5.0 i just released it


Remove executable file flag for non-executables

Fixes #1899

Refer to this link for build results (access rights to CI server needed):
http://jenkins.terasology.org/job/TerasologyPRs/295/
Hooray Jenkins reported success with all tests good!


relay.NewNodeDefinitions.TypeResolve type changed

$ go run main.go
# github.com/graphql-go/relay/examples/starwars
../relay/examples/starwars/schema.go:121:14: cannot use func literal (type func(interface {}, graphql.ResolveInfo) *graphql.Object) as type graphql.ResolveTypeFn in field value

I'm making a PR for this issue.
Thanks

Ah, never mind.
I found another PR from here.


Add permutations generator

Closes #171.

We don’t want d3.permute to have a radically different semantics if only a single argument is passed. That’s very confusing. So, this needs a new name. I propose d3.permutations.
I’m not sure I see this function reaching the bar of usefulness to merit inclusion in this library. Do you have some practical, applied examples?
Given that this is derived from third-party code rather than an original contribution, this would require including the Google Collections Apache license. Also, that algorithm is a little complicated, and is it worth the complexity? I feel it would be simpler and easier to write a fresh implementation of the simpler Heap’s algorithm such as
function* permutations(array) {
  array = Array.from(array); // defensive copy and convert to array
  let i = 0, n = array.length, index = new Uint32Array(n).fill(0);
  yield array.slice(); // defensive copy
  while (i < n) {
    let j = index[i];
    if (j < i) {
      if ((i & 1) === 0) j = 0;
      let t = array[j];
      array[j] = array[i];
      array[i] = t;
      yield array.slice(); // defensive copy
      ++index[i];
      i = 0;
    } else {
      index[i] = 0;
      ++i;
    }
  }
}

I guess if it’s that small, there’s not much harm in including it… but I’d still appreciate some applied examples.
Another consideration is whether we want to implement an optional r argument, and a d3.combinations method at the same time. For reference, here’s a port of itertools by Jacob Rus, and the Python itertools API reference for itertools.permutations and itertools.combinations. Of course if we’re going to base on implementation on Python’s itertools then we’ll likewise need to add a copy of the PSF license.


Feature request: Add ENV vars alternative to set parameters

Hello.
Thank you for the pgbouncer_exporter. I was surprised that there is a new one maintained by prometheus-community.
As I see, kingpin supports environment variables in addition to already existing command-line flags.
https://github.com/alecthomas/kingpin/blob/master/_examples/chat1/main.go#L11
So I think it would be nice to add this. I could try to do PR if needed.
P.S. Also there is no information about command-line flags in Readme.md, so I have to download binary and check -h output. It might be good to add it there I suppose.

Nevermind, found that it was already proposed https://github.com/prometheus-community/pgbouncer_exporter/issues/24


What about null values ?

Unlike other libraries, like igorw/get-in for example, Parkour\Access does not handle null values, and consider them as non-existent.
Thus, we cannot

use has to know if an empty key exists
use update to update null values, to set a default value for example

This issue is due to the use of isset instead of is_array and array_key_exists.
Is this a voluntary omission or is it a bug ?

This is something I didn't really think about, as the current implementation fits my use cases.
It is however a good point, maybe an additionnal parameter on those methods could turn on/off handling of null values?


added Hochschule Flensburg

Fachhochschule Flensburg changed to Hochschule Flensburg and their domains changed.

@goggelz Pull request merged. Thank you!


GH-36255: [C++] Add benchmarks for "if_else" kernel on lists

Rationale for this change
"if_else" kernels kernels with list input can have their perf improved at the cost of code specialization. I'm working on this problem and would like to introduce benchmarks that measure how it behaves currently.
What changes are included in this PR?
Only the new benchmarks and addition of items_per_second metric on the others.
Are these changes tested?
By running the benchmarks.
Are there any user-facing changes?
No.

Closes: #36255

@pitrou


train model file give me 'Model file has wrong file format!'

I got 'Model file has wrong file format!'
when using docker:
docker run -v torontobooks_bigrams.bin:/sent2vec/pretrained_model.bin -it keyphrase-extraction
that i try two files torontobooks_bigrams.bin ,wiki_unigram.bin
but they give me the same error!
thanks in advance

Hello,
Could you try without relative path ? i.e. docker run -v full_path/torontobooks_bigrams.bin:/... -it keyphrase-extraction ?


Using the same protocol to connect to another server after disconnecting

When disconnecting from a server/Client, the protocol is used again to connect to another server/Client however the connection hangs and eventually "Connection closed." is received.
I'd prefer to keep the protocol open otherwise Mojang blocks my attempts to log in again due to spam-prevention. Is there a certain method or something I have overlooked?

For various reasons, protocol instances cannot be reused, like java sockets. This is not likely to change. You can, however, pull the access token from a dead protocol instance using the available method and use it to initialize a new instance.


Implement a "normalize by default" strategy

Consider an option to normalize by default, unless otherwise specified. Add a new constructor option for normalization, as well as in the toString method.

Done


Vulnerability Report in ResultEntries excludes vulnerability name

From here https://github.com/eedevops/shipyard/blob/dev/checker/checker.go#L172-L185
It can be seen that we are just adding the Feature information. However, we should also add other fields, the most important being the Name in the Vulnerability object as they do here https://github.com/eedevops/shipyard/blob/dev/checker/checker.go#L146
Thanks!

From what I can tell they are nested: Report -> [ ] Features, each Feature -> [ ] Vulnerabilities, each Vulnerability  -> Name.
So you would have:
for f := range Report.Features
      for v := range f.Vulnerabilities
           fmt.Printf(v.Name)

So we do add the names, but in a more nested, complicated manner.

What I'm looking for is the CVExxxx code. That's actually not being added.

FIxed here https://github.com/eedevops/shipyard/commit/6f64199ef03523ba44234146b71295d1a67923a4


Query multiple tables

In this project, I added some new tables and added foreign key columns to their existing animal table to model one-to-many relationships. Then, I used JOIN to query the data.
Screenshot of queries:

What animals belong to Melody Pond?

List of all animals that are pokemon (their type is Pokemon).

List all owners and their animals, remember to include those that don't own any animal.

How many animals are there per species?

List all Digimon owned by Jennifer Orwell.

List all animals owned by Dean Winchester that haven't tried to escape.

Who owns the most animals?

Hi @MarcoDiaz2000,
While you made a great effort in this project, unfortunately, I cannot proceed to review your code.
Invalid Code Review Request
You have submitted a project, that is invalid because your project seems to have been reviewed but I do not find or see the review in the PR
Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

Invalid Code Review Request does not count into the code reviews limit.

The first reviewer forgot to put the review.
I have already informed in the new revision request that the reviewer forgot to put the revision.
In the new request I report this event.
But now the second reviewer tells me that he cannot review because he cannot find the comments of the first reviewer.
I hope that the next reviewer can help me, and do me the favor of being able to review my project, or advise me on how to proceed.
Thanks


Add custom options for SelectWithFilter

Main branch
I will merge all my child PRs in this one and build a demo with it at the end

Codecov Report

Merging #865 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #865      +/-   ##
==========================================
- Coverage   99.75%   99.75%   -0.01%
==========================================
  Files         276      276
  Lines        7008     7007       -1
  Branches     1717     1716       -1
==========================================
- Hits         6991     6990       -1
  Misses         17       17

Impacted Files
Coverage Δ

src/components/select/SelectWithFilter.tsx
100% <ø> (ø)
: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 481470f...b0d1a3b. Read the comment docs.


wrong Lint/AssignmentInCondition

def p
  111
end
if b
  do_something
elsif a = p  <===
  do_something(a)
end

It trigger Lint/AssignmentInCondition and said:
Assignment in condition - you probably meant to use ==.

Clearly, use == is wrong.

Clearly, use == is wrong.

I don't think so.
Why do you think this is clearly wrong?

Since i want to use a new val not compare the val, the cop output is not right.

Also, if use elsif(a = p), this cop will disappear. I mean the cop output may mislead.

The unclear error message has been fixed on master branch by #4605, but hasn’t been released yet.

Also, if use elsif(a = p), this cop will disappear.

That sounds like a bug in Lint/AssignmentInCondition


error when trying to make the site work

Traceback (most recent call last):
File "app.py", line 14, in
with open("config.yaml") as f:
FileNotFoundError: [Errno 2] No such file or directory: 'config.yaml'
tried to create an empty config.yaml there is an other error
File "app.py", line 17, in
app.config['SECRET_KEY'] = config['SECRET_KEY']
TypeError: string indices must be integers

Ok I just made an installation tutorial that goes in depth here