testhost.dll is double written

Description
See https://github.com/KirillOsenkov/MSBuildStructuredLog/wiki/Double write detection and https://www.meziantou.net/detecting-double-writes-in-msbuild-using-the-binlog.htm for info about double writes.
In a binlog from a private repo, I'm seeing testhost is double written.

Steps to reproduce
I didn't try to produce a minimal repro, but basically, the package props (https://github.com/microsoft/vstest/blob/39c7dd12c7ec24d0552513e84d95476f2077ca33/src/package/Microsoft.TestPlatform.TestHost/Microsoft.TestPlatform.TestHost.props) is marking testhost.dll in build/netcoreapp3.1/x64 directory as <CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>. However, MSBuild also adds the testhost.dll from lib directory to ReferenceCopyLocalPaths.
Expected behavior

No double writes. It's a bit discouraged.
Actual behavior

Double writes.
Diagnostic logs

Environment

@drognanar I think you were investigating similar issue in the past, do you remember if it was resolved?

repeating the steps above I no longer see any double writes.


Fix for #1352

Fixes #1352 and increases the lower-bound of DynamicPPL (the recent version includes bug-fixes for predict and generated_quantities).

The tests fail since Libtask_jll does not support Julia 1.6.0 yet - it will be fixed by https://github.com/JuliaPackaging/Yggdrasil/pull/2176 but the PR depends on https://github.com/JuliaPackaging/Yggdrasil/pull/1987 (see https://github.com/TuringLang/Turing.jl/issues/1554 for more details).

Maybe we should run Turing tests with Julia 1.5, here and in DynamicPPL?

Maybe we should run Turing tests with Julia 1.5, here and in DynamicPPL?

Sounds like a good idea to me :+1:

Just for reference: this is now only waiting on packages to get up-to-date with Bijectors.jl and running #1568

This should finally be good to go now, once tests pass :+1:

Pull Request Test Coverage Report for Build 738570650

5 of 5 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.02%) to 78.184%

Totals

Change from base Build 738567682:
0.02%

Covered Lines:
1111

Relevant Lines:
1421

💛  - Coveralls

What in the world. Gibbs is failing again

I'm very confused by that test. Nothing that I did in the fix for MH and linking changed the behavior of MH(), which is NOT linked and uses StaticProposal. But now it results in v2 in the test at https://github.com/TuringLang/Turing.jl/blob/master/test/inference/mh.jl#L149-L152 vanishing, which makes the test fail o.O
Is it okay if I just disable that particular comparison for now and make an issue for it? Really want to get the PR merged, and the only thing holding it back is tests unrelated to this PR failing due to random seed :sweat_smile:

Is it okay if I just disable that particular comparison for now and make an issue for it? Really want to get the PR merged, and the only thing holding it back is tests unrelated to this PR failing due to random seed 😅

Happy to merge this PR as-is and open an issue for the failed test.


Use .tfvars for Terraform variables

Terraform recommends using a .tfvars file for variable values rather than updating a variables tf file.
An additional concern with the existing implementation is that someone contributing to your repo (or a fork of it) may accidentally commit sensitive information in the var.tf file, as this file is already tracked and versioned (it is not in the .gitignore list).

Good point! We'll update the instructions to use .tfvars instead and add it to the .gitignore file. Thank you!


Remove style property from paragraph when no column layout

[question] Is there a possibility to remove style="column-count: 1" from paragraph element when I don't choose layout column?

Fixed in 0.7.11


make build image for nginx the same as django

In the Dockerfile it says:

The code for the build image should be idendical with the code in
Dockerfile.django to use the caching mechanism of Docker.

this PR makes Dockerfile.nginx back in sync with Dockerfile.django again

haha look how good I am with markdown (not)

Was the typo in identical yours too :P


Create vcconnect.txt

Added Varsity College

the college website seems to be www.varsitycollege.co.za - could you please provide some confirmation that vcconnect.co.za also belongs to college? thanks!

Good Morning
What specific details do you require for the confirmation? Will a simple
letter from Varsity College with their letter head a signature suffice?
Thanks
Greg Wright
On Wed, 4 Mar 2015 at 17:18 Elena Ganenkova<EMAIL_ADDRESS>wrote:

the college website seems to be www.varsitycollege.co.za - could you
please provide some confirmation that vcconnect.co.za also belongs to
college? thanks!
—
Reply to this email directly or view it on GitHub
https://github.com/JetBrains/swot/pull/821#issuecomment-77176229.

Good Afternoon
I have still not heard from you in regard to what particular information is required to prove that vcconnect.co.za belongs to varsity college.
Please can you advise me in this.
Thanks
Greg Wright

I'm very sorry for the delay. Yes a confirmation letter should do fine - please send one to<EMAIL_ADDRESS>and i'll add the domain. Thanks for your help!

I've got the letter, thank you - I'm adding the domain then.


Update python-jose's hash.

@johngian r?

This is straight forward and it has been already reviewed here:
https://github.com/mozilla/mozillians/pull/1710


I want to get detail about the payload test

I run the script and get result but have no idea how to find the payload detail. I need look into each payloads resulting to false negative to verify the test result.
Thanks

Just run tool with --details option (https://github.com/nemesida-waf/waf-bypass#options), e.g. --host='example.com' --details


dire-warning detour if invalid BIP-85 child index

resolves issue #369
This pr adds a detour to Dire Warning whenever a BIP-85 Child Index is outside the valid range of (0, 2**31-1), informing the user and forcing them to "Try Again", instead of the application crashing.
Related to this pr is #370 (handling exceptions which have no exception message), but they are independent of each other.

if this one is merged but not the other one: the bug reported in #369 will be resolved, and the exception handler bug will be disguised until another message-less exception is found.
if the other one is merged but not this one:  users would see a vague "Assertion Error" with embit references, and can start over... guessing what they did wrong... but at least the application won't crash.
if both are merged, users are informed and can try again AND we might learn of exception-type/filename/line-number the next time seedsigner "hangs" for message-less exceptions.

ACK Tested


BUG: Socket connection not caught

Describe the bug
In the vetext forwarder lambda we break the connection to vetext after 6 seconds. The exception is not caught (we incorrectly thought it would be an http.client.HTTPException.
Expected behavior
A broken socket connection just logs the exception and the code continues.

I spent about half a day trying to understand what was going on with this. Essentially this is not throwing an exception. It appears the getrequest method is hanging. The lambda code needs to be updated to use the requests module. requests is a higher-level library for making http requests, whereas http.client is lower level. The higher level library handles the timeout issues for us (setting the timeout parameter does what you would expect). The documentation for http.client's timeout is not as clear.

Spoke with @jakehova and he's going to create a new ticket given we're not progressing on this path of fixing the http.client issues.


Tab key triggers twice in shell

Environment
Windows build number: Microsoft Windows [Version 10.0.19041.508]

Windows Terminal version (if applicable): 1.3.2382.0

Steps to reproduce

Start either cmd, PowerShell or wsl (Ubuntu 20.04) profile (with default settings.json).
Hit Tab ↹ button.
Tab ↹ action is executed by shell twice,

Expected behavior
Shell receives Tab key once, and therefore executes it's action once.
Actual behavior
Shell (bash on wsl, PowerShell or cmd) executes Tab key twice, resulting with two \t characters in vim, or advancing shell's directory autocompletion by two directories.
Notes:

After mapping another key to Tab input code with:
{ "command": { "action": "sendInput", "input": "\u0009" }, "keys": "f2" }
F2 key sends Tab's input code only once.
It is not a problem with keyboard, as the problem appears only in Windows Terminal.

Interesting. Are you using a physical or virtual keyboard? What layout is it?

(I ask because this doesn't seem to reproduce with US (QWERTY) hardware keyboards for anyone on the team.)

It's physical keyboard.
I am using Polish layout, but the problem appears also with English US and UK layout.
Also tried to change system language to English UK, but it did not help.

By any chance, did you try the workaround earlier in #4448, with setting InputServiceEnabled: 1?

InputServiceEnabled: 1 does not help.

Sorry, I do not understand.
InputServiceEnabled: 1 is the default, but you need to enable the "Touch Keyboard and Handwriting Panel Service" service. When you have InputServiceEnabled: 1 and service started, does the keyboard continue working for Terminal?
If so, does the Tab bug still happen?

Thank you for clarification. It solved the issue.
So for anyone in the future setting
HKLM\SOFTWARE\Microsoft\Input:
  InputServiceEnabled: 1
  InputServiceEnabledForCCI: 1

and setting service Touch Keyboard and Handwriting Panel to manual solves the problem.

Thanks.
I will add this to the list of issues caused by disabling random windows services!


Preload

Adding support for responsive preload tags. Will update the linked vue-visual package to point to the main branch once https://github.com/BKWLD/vue-visual/pull/76 is merged, so that should be reviewed first.

I've been using this PR to test https://github.com/BKWLD/vue-visual/pull/76.  Made some tweaks, but just ran a test on PageSpeed insights and it's detecting the <link>!


[feat] [client] Adding-toast-notifications

closes #18
Added toast notifications for successful login and registration or error while authenticating ^^
npm i to download the toastify node package

@Sameersawant77 Thankyou for the PR. I would like to add on into it that can you please specify the toster which is been shown. Like if password is not entered, it should display password not display, or if the password is not correct, it can show wrong password added or if email format is not correct then it can show wrong email format etc etc. same goes for the registration page.

Hey @ojasaklechayt, sure i can the requirements provided the API response passes a message depending on the error

Hey @ojasaklechayt, sure i can the requirements provided the API response passes a message depending on the error

You can check the backend code, I feel the response for wrong password is not there, may you can add up that condition in the backend.

@ojasaklechayt  I have added toast notifications "user not found" and "invalid password" for login and "User already exists" while signup ^^

@Sameersawant77 Great work Sameer, Thanks for the contribution ^^^


BF: fix query to response button for survey-likert

I couldn't make the example for jspsych-survey-likert work, so I tried debugging it and this fixes it. However, I'm very naive to javascript, so please ignore if this is just noise.
Thanks

Thanks for catching these!


Due Date & Submit button overflows the form

->On smaller screens the Due date and Submit button overflows out of the form.
SCREEN-SHOTS For reference
#Screent-Shot-1

#Screent-Shot-1

@Kritika30032002, Kindly assign me this issue, I will solve it as soon as possible.

I want to work on this. Assign it to me


Error after installing in my Meteor application

I keep getting the following error:
node_modules/clipboard/node_modules/babelify/node_modules/babel-core/node_modules/convert-source-map/test/fixtures/map-file-comment-double-slash.css:
missing '{' near line 15:1

@lnmunhoz can you check that?

I was able to get things working in Meteor by uninstalling the npm your package and just copying over the min.js file.  This works great.
Thanks for this amazing package!

I'm going to check it out @zenorocha

@aadamsx What meteor version are you using?


move latest blog:  database, notification services, protocol under nr section

…der node-red section
Description

Related Issue(s)

Checklist

[ ] I have read the contribution guidelines
[ ] I have considered the performance impact of these changes
[ ] Suitable unit/system level tests have been added and they pass
[ ] Documentation has been updated

@sumitshinde-84 MIssing redirect updates?

Done


Parallelize domain eval

This change is

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#821  👈
#820
#819
dev

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @spapinistarkware and the rest of your teammates on  Graphite

Codecov Report
Attention: Patch coverage is 94.73684% with 1 line in your changes missing coverage. Please review.

Project coverage is 92.78%. Comparing base (755db7c) to head (dfa1003).

Files with missing lines
Patch %
Lines

crates/prover/src/core/backend/simd/utils.rs
85.71%
1 Missing :warning:

Additional details and impacted files
@@                     Coverage Diff                      @@
##           spapini/09-05-simd_twiddles     #821   +/-   ##
============================================================
  Coverage                        92.78%   92.78%
============================================================
  Files                               89       89
  Lines                            12165    12185   +20
  Branches                         12165    12185   +20
============================================================
+ Hits                             11287    11306   +19
- Misses                             771      772    +1
  Partials                           107      107

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


LabelledMixin validation of missing labels

If a label is missing on one of our input elements, it's a P1 accessibility violation. We're pretty inconsistent with how we try to notify developers of missing labels -- sometimes we do nothing, occasionally we console.warn in firstUpdated.
This change attempts to centralize the reporting of that error in a way that doesn't initially blow up for all the existing implementations that might be missing labels. It will throw the error but not block rendering, so that we can go through the error logs and find any problem areas.
Once we're confident that all the missing label errors are fixed for a particular component, we can update it so that a missing label actually stops rendering, which will be a strong signal to the developer (or tester) that something is wrong and hopefully they see the console error.
If this approach looks good, I'll do some follow-up PRs to actually adopt the LabelledMixin in more (or all) of our input components. Currently only lists use it.

Going to update this to also handle missing elements referenced by labelled-by, which is currently just console.warn-ing.


Sticky belongs-to doesn't work

Thank's for this amazing addon.
I'm using:
Ember             : 2.9.0
Ember Data        : 2.9.0
jQuery            : 3.1.1
Ember Simple Auth : 1.1.0

I'm have problems with Sticky belongs-to. My project has model chat and model chatMessage. chat hasMany chaMessages and chatMessages belongsTo chat.
model chat.js
import Model from 'ember-data/model';
import attr from 'ember-data/attr';
import { belongsTo, hasMany } from 'ember-data/relationships';
import HasManyQuery from 'ember-data-has-many-query';

export default Model.extend(HasManyQuery.ModelMixin,{
  name: attr('string'),
  date: attr('date'),
  chatMessages: hasMany('chat-message', { async: true })
});

model chatMessage.js
import Model from 'ember-data/model';
import attr from 'ember-data/attr';
//import emojione from 'emojione';
import { belongsTo, hasMany } from 'ember-data/relationships';
import HasManyQuery from 'ember-data-has-many-query';

export default Model.extend(HasManyQuery.ModelMixin,{
  message: attr('string'),
  date: attr('date'),
  chat: HasManyQuery.belongsToSticky('chat')
});

I'm implementing an action like "Read More" to get more items from chatMessages. My API expect a request with params:
https://myapi.com/chats/2/chat-messages?page%5Bnumber%5D=1475591389&page%limit%5D=20
Route chats/show: https://myapi.com/chats/2/chat-messages
model(params) {
    return Ember.RSVP.hash({
      chat: this.store.findRecord('chat', params.chat_id)
    })
}

In this pieace of code above, I'm get the chat user has selected: https://myapi.com/chats/2/chat-messages and because I'm using JSONAPI this already push chatMessages in store.
So when user click in action "readMore" I'm doing this:
actions: {
    readMore(chatId, pageNumber) {
      this._super(...arguments);
      let chat = this.controller.get('model.chat');

      this.set('chat-messages', Ember.ObjectProxy.extend(Ember.PromiseProxyMixin).create({
        promise: this.controller.get('model.chat').query('chatMessages', { page: { number: pageNumber } })
      }));
   }
}

With this, I'm capable to get items from https://myapi.com/chats/2/chat-messages?page%5Bnumber%5D=1475591389&page%limit%5D=20 BUT :sob: it's clear the frist 20 items I'm already get in: https://myapi.com/chats/2/chat-messages
How can I persist this data? I'm following Sticky belongs-to but it's doesn't work. Please, you can help with this? Have any idea?
Thanks!

Apologies, the README on this is a bit unclear. When you query the API to get the next page, the hasMany relationship is cleared and the new data is loaded. This is because query calls reload on the hasMany relationship, which clears and reloads the data.
The way I implemented readMore functionality in my app was to store the records in a separate array. For example, in the component, have a messages property which is an array, and each time you load another page push the loaded records into this array:
  readMore: function(page) {
    var self = this;
    this.get('model.chat').query('messages', {
      page: page
    }).then(function (response) {
      var array = self.get('messages');
      response.forEach(function(message) {
        array.push(message);
      });
    }
  }

@mdehoog It does not work :cry: . It seems that when I print self.get( 'model.chat.chatMessages') the array has already been updated with the response, so it already brings the 20 new items (deleting the 20 previous items).
I also could not (from my component) access: this.get('model.chat'), I had to do: this.controller.get('model.chat').
Do you have any idea what might be happening to update the array before I can pick up the previous items?

'model.chat.chatMessages' will always contain the latest queried messages, because 'chat.chatMessages' is the hasMany array. What you need to do is cache these messages in a separate array after every query is complete, and display this cached array instead of the has-many relationship.

Thank's @mdehoog I'll follow your tip.

The issue with displaying the separate array instead of the canonical hasMany array is it makes subsequent operations tricky (like deleting items). Normally ember-data makes this stuff super simple as long as you stick to just using the canonical hasMany array.


Use our Tty instead of Homebrew's

Changes to the core

[x] Followed hacking.md.

Closes #22836

Thank you!
Sorry that I did not notice that.

No worries! I'm working on converting our modules to the nested style (module Hbc; module Utils; class Tty) vs the compact style (class Hbc::Utils::Tty) to properly scope everything so this doesn't happen with other naming conflicts as we continue to merge in more Homebrew stuff.


gk-deploy fails waiting for pods matching '--selector=glusterfs=pod'

When running gk-deploy it gets to the point where it deploys the glusterfs pods, then it waits for a few minutes before it times out, even though the pods are running. Here's the output from 'gk-deploy -gvy'.
Using Kubernetes CLI.
Checking status of namespace matching 'default':
default   Active    76d
Using namespace "default".
Checking for pre-existing resources...
GlusterFS pods ...
Checking status of pods matching '--selector=glusterfs=pod':
Timed out waiting for pods matching '--selector=glusterfs=pod'.
not found.
deploy-heketi pod ...
Checking status of pods matching '--selector=deploy-heketi=pod':
Timed out waiting for pods matching '--selector=deploy-heketi=pod'.
not found.
heketi pod ...
Checking status of pods matching '--selector=heketi=pod':
Timed out waiting for pods matching '--selector=heketi=pod'.
not found.
gluster-s3 pod ...
Checking status of pods matching '--selector=glusterfs=s3-pod':
Timed out waiting for pods matching '--selector=glusterfs=s3-pod'.
not found.
Creating initial resources ... /usr/bin/kubectl -n default create -f /home/steve/kubernetes/elk/kube-templates/heketi-service-account.yaml 2>&1
serviceaccount/heketi-service-account created
/usr/bin/kubectl -n default create clusterrolebinding heketi-sa-view --clusterrole=edit --serviceaccount=default:heketi-service-account 2>&1
clusterrolebinding.rbac.authorization.k8s.io/heketi-sa-view created
/usr/bin/kubectl -n default label --overwrite clusterrolebinding heketi-sa-view glusterfs=heketi-sa-view heketi=sa-view
clusterrolebinding.rbac.authorization.k8s.io/heketi-sa-view labeled
OK
Marking '<IP_ADDRESS>' as a GlusterFS node.
/usr/bin/kubectl -n default label nodes <IP_ADDRESS> storagenode=glusterfs --overwrite 2>&1
node/<IP_ADDRESS> not labeled
Marking '<IP_ADDRESS>' as a GlusterFS node.
/usr/bin/kubectl -n default label nodes <IP_ADDRESS> storagenode=glusterfs --overwrite 2>&1
node/<IP_ADDRESS> not labeled
Marking '<IP_ADDRESS>' as a GlusterFS node.
/usr/bin/kubectl -n default label nodes <IP_ADDRESS> storagenode=glusterfs --overwrite 2>&1
node/<IP_ADDRESS> not labeled
Deploying GlusterFS pods.
sed -e 's/storagenode: glusterfs/storagenode: 'glusterfs'/g' /home/steve/kubernetes/elk/kube-templates/glusterfs-daemonset.yaml | /usr/bin/kubectl -n default create -f - 2>&1
daemonset.extensions/glusterfs created
Waiting for GlusterFS pods to start ...
Checking status of pods matching '--selector=glusterfs=pod':
glusterfs-kz2zc   0/1       Running   0         6m
glusterfs-tj6qx   0/1       Running   0         6m
glusterfs-zkg59   0/1       Running   0         6m
Timed out waiting for pods matching '--selector=glusterfs=pod'.
pods not found.

I deleted the glusterfs daemonset and tried gk-deploy again, similar result:
daemonset.extensions/glusterfs created
Waiting for GlusterFS pods to start ...
Checking status of pods matching '--selector=glusterfs=pod':
glusterfs-dxfdf   0/1       Running   0         3m
glusterfs-l5bq9   0/1       Running   0         3m
glusterfs-p6982   0/1       Running   0         3m
When I look at syslog I see this:
Aug 23 16:29:42 10 containerd[1294]: time="2018-08-23T16:29:42-04:00" level=info msg="ExecSync for "fecf862c4ae2792b71035876f2f20043224e68b3925fc54a48a1ff3ffdcc0ed4" with command [/bin/bash -c systemctl status glusterd.service] and timeout 3 (s)"
Aug 23 16:29:42 10 containerd[1294]: time="2018-08-23T16:29:42-04:00" level=info msg="Finish piping "stdout" of container exec "a86c2d811ec8f50dcc1032825d35865c56400224034bb21a9410d5287ca54262""
Aug 23 16:29:42 10 containerd[1294]: time="2018-08-23T16:29:42-04:00" level=info msg="Finish piping "stderr" of container exec "a86c2d811ec8f50dcc1032825d35865c56400224034bb21a9410d5287ca54262""
Aug 23 16:29:42 10 containerd[1294]: time="2018-08-23T16:29:42-04:00" level=info msg="Exec process "a86c2d811ec8f50dcc1032825d35865c56400224034bb21a9410d5287ca54262" exits with exit code 3 and error "
Aug 23 16:29:42 10 containerd[1294]: time="2018-08-23T16:29:42-04:00" level=info msg="ExecSync for "fecf862c4ae2792b71035876f2f20043224e68b3925fc54a48a1ff3ffdcc0ed4" returns with exit code 3"
Aug 23 16:29:42 10 kubelet[1491]: I0823 16:29:42.335434    1491 prober.go:111] Liveness probe for "glusterfs-dxfdf_default(fdba7974-a712-11e8-9696-005056aa3804):glusterfs" failed (failure): ● glusterd.service - GlusterFS, a clustered file-system server
Aug 23 16:29:42 10 kubelet[1491]:    Loaded: loaded (/usr/lib/systemd/system/glusterd.service; enabled; vendor preset: disabled)
Aug 23 16:29:42 10 kubelet[1491]:    Active: failed (Result: exit-code) since Thu 2018-08-23 20:27:45 UTC; 1min 56s ago
Aug 23 16:29:42 10 kubelet[1491]:   Process: 67 ExecStart=/usr/sbin/glusterd -p /var/run/glusterd.pid --log-level $LOG_LEVEL $GLUSTERD_OPTIONS (code=exited, status=1/FAILURE)
Aug 23 16:29:42 10 kubelet[1491]: Aug 23 20:27:45 <IP_ADDRESS> systemd[1]: Starting GlusterFS, a clustered file-system server...
Aug 23 16:29:42 10 kubelet[1491]: Aug 23 20:27:45 <IP_ADDRESS> systemd[1]: glusterd.service: control process exited, code=exited status=1
Aug 23 16:29:42 10 kubelet[1491]: Aug 23 20:27:45 <IP_ADDRESS> systemd[1]: Failed to start GlusterFS, a clustered file-system server.
Aug 23 16:29:42 10 kubelet[1491]: Aug 23 20:27:45 <IP_ADDRESS> systemd[1]: Unit glusterd.service entered failed state.
Aug 23 16:29:42 10 kubelet[1491]: Aug 23 20:27:45 <IP_ADDRESS> systemd[1]: glusterd.service failed.

Since glusterd doesn't seem to be reporting error conditions in the journal snippet you get in systemctl status I recommend looking into the glusterd logs for more information. The gluster pod mounts /var/log/glusterfs from the host, so even when the pod exits there ought to be glusterd logs in the dir.

So even though the pods appear running, the status of the gluserd service on the node is 'failed', near as I can tell it fails to start because of this:
[glusterfsd.c:2150:parse_cmdline] 0-glusterfs: ERROR: parsing the volfile failed [No such file or directory]
but I cant figure out how to correct this. Is it telling me it can't find the vol file, or that it can't find the directory to which the vol file refers? My topology file looks like this:
{
"clusters": [
{
"nodes": [
{
"node": {
"hostnames": {
"manage": [
"node0"
],
"storage": [
"<IP_ADDRESS>"
]
},
"zone": 1
},
"devices": [
"/dev/sdc",
"/dev/sdd"
]
},
{
"node": {
"hostnames": {
"manage": [
"node1"
],
"storage": [
"<IP_ADDRESS>"
]
},
"zone": 1
},
"devices": [
"/dev/sdc",
"/dev/sdd"
]
},
{
"node": {
"hostnames": {
"manage": [
"node2"
],
"storage": [
"<IP_ADDRESS>"
]
},
"zone": 1
},
"devices": [
"/dev/sdc",
"/dev/sdd"
]
}
]
}
]
}
I've confirmed that /dev/sdc and sdd exist on each node, the IP addresses above are the IP addresses of nodes. Is there something I'm missing in the topology or one of the template files that's causing this volfile error? I'm running 'gk-deploy -gvy' and I've also tried --no-object

Well I managed to get this deployed. I ran 'gk-deploy --abort' and deleted the daemonset, then I deleted:
/var/lib/heketi
/etc/glusterfs
/var/lib/glusterd
/var/lib/misc/glusterfsd
and ran 'gk-deploy -gvy' again. This time it worked. From other forums I think that deleting /var/lib/glusterd was the key. This may have had left over cruft from a previous standalone glusterfs-server installation.


Add networkpolicy to minio

cc: @lachie83

/ok-to-test

cc @NitishT

This looks good to me.

Just need a bump of a chart version

Looks good overall. Thanks for the PR.

ping @lachie83 good to go?


pausing two or more springs

🐛 Bug Report
In case of two springs or more, pausing just works for the first one. For example if you use the callback function below (animCallback) as click handler, it stops the first animation but the second one continues regardless of the pause state.
To Reproduce
const [pause, toggle]= useState(false)
 const animCallback= useCallback(async () => {
    await setGreen({
      width: 200,
      default: { pause },
      })
    await setBrown({
      transform: "translate(400px, -300px)",
      default: { pause }
    });
  }, [pause, setGreen, setBrown]);

Expected behavior
All animations defined in animCallback should be stoped if puase is true.
Link to repro
Codesandbox: https://codesandbox.io/s/pausing-second-spring-not-working-2mndx?file=/src/App.tsx
Workaround
If you use spring ref and  pause all of them in useEffect it works as expected!
Sandbox: https://codesandbox.io/s/spring-pause-with-ref-e29v7?file=/src/App.tsx
Environment

react-spring<EMAIL_ADDRESS>react v16.12.0

When the pause button is pressed while the setBrown animation is going, your animCallback gets stalled on the setGreen call because pause is true. I think you want something like this:
await Promise.all([
  setGreen({
    width: 200,
    default: { pause },
  }),
  setBrown({
    transform: "translate(400px, -300px)",
    default: { pause }
  }),
]);

Oh, I suppose you want to call setBrown after the setGreen animation is finished?
In that case, you'll want to send pausing/resuming updates separately.
So, remove default: {pause} from your animCB function, and change your onClick handler to do this:
setGreen({ default: {pause} })
setBrown({ default: {pause} })
if (!pause) {
  animCB();
}
toggle(!pause);


You must use Bundler 2 or greater error

ruby version: 2.6.0
ubuntu: 18.04
bundler: 2.0.1
I am not sure wether this is a replica of #6784 or #6937 but it happens for me on the above setup. Weird thing is that when I retry a dozen times, it succeeds but fails again later in another run. This only happens through the CI ( gitlab ).
If I ssh into the server and manually run bundle install.  everything is fine
$ bundle install
You must use Bundler 2 or greater with this lockfile.

Can you try upgrading to ruby 2.6.2 and report back?

with 2.6.2 it still happens. The error message looks different but it is still the same
/usr/share/rvm/rubies/ruby-2.6.2/lib/ruby/2.6.0/rubygems.rb:283:in `find_spec_for_exe': Could not find 'bundler' (2.0.1) required by your /home/gitlab-runner/builds/jZR2y_qx/0/proactive-mobile/android/Gemfile.lock. (Gem::GemNotFoundException)
To update to the latest version installed on your system, run `bundle update --bundler`.
To install the missing version, run `gem install bundler:2.0.1`
	from /usr/share/rvm/rubies/ruby-2.6.2/lib/ruby/2.6.0/rubygems.rb:302:in `activate_bin_path'
	from /usr/share/rvm/rubies/ruby-2.6.2/bin/bundle:23:in `<main>'
	from /usr/share/rvm/rubies/ruby-2.6.2/bin/ruby_executable_hooks:24:in `eval'
	from /usr/share/rvm/rubies/ruby-2.6.2/bin/ruby_executable_hooks:24:in `<main>'

Ok, thanks.
And you definitely have bundler 2 installed there, right?

yes I have, this is the output of gem list bundler
bundler (2.0.1, default: 1.17.2)
bundler-unload (1.0.2)
rubygems-bundler (1.4.5)

:+1:. And you just can't reproduce it consistently, right?
It'd be great if you could try reproduce this on bundler's master. You would need to lock your lockfile to 2.1.0.pre.1, and make sure you have that installed in your CI (bin/rake install:local from a bundler checkout should do the trick, once #7104 is merged :sweat_smile:).

I actually fixed it just now by deleting the local Gemfile.lock and pushing it to the CI again. It recreated the Gemfile and runs fine so far.

Did something change in the lock file?

I think the ruby version. Not sure though..

I think the ruby version.

Interesting...

Not sure though..

I do commit it.. but I think that is why it didn't regenerate it maybe after changing ruby versions?

Oh ok, I figured, since you were not sure about what changed, that you didn't keep that file under version control. Anyways, please report back if this happens again. I'd really like to be able to reproduce this.

sure! will do.


XP-PEN G430 Driver: init at 20190820

Motivation for this change
This driver is an useful application for people who own the XP-PEN G430 drawing tablet, as the tablet does not work without this.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
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 nixpkgs-review --run "nixpkgs-review wip"
[X] 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)
[X] Fits CONTRIBUTING.md.

I'm very sorry if i messed something up, fairly new to this.

When trying to build locally with the change from qt5.qtebase > qtbase i get the following error:
error: anonymous function at /home/ivar/misc/nixpkgs/pkgs/misc/drivers/xp-pen-g430/default.nix:1:1 called without required argument 'qtbase', at /nix/store/lxid8qkmifw0bhyxa14qanvmkv5lkd5d-nixpkgs-20.09pre217039.13e7a3e1127/nixpkgs/lib/customisation.nix:69:16
Not sure how to fix

Rebased, any chance you could just use the squash&merge feature from github? Im not that familiar with git, sorry :frowning_face:

Let me take a look

I’m sorry, this PR is too broken to fix up. Could you manually take your current state and force-push it as one commit to this branch? Or open a new PR if you can’t figure out how to force-push (I admit, git can be a bit confusing).
Github has nice guides that give an introduction on how to use git with it’s interface, maybe you will want to read those.
As a general rule of thumb: don’t merge master back into your pull request branch, it makes changing things after the fact very hard.

Apologies for the trouble, will open up a new PR.


Graceful or Gracefull for worker-statefulset value

Currently going through v8 of the chart (really nice btw) and noticed that we spell graceful as gracefull in the context of the worker statefulset values. I tried looking online to see if there's a dictionary which spells it with two Ls, but some dictionaries didn't recognize it and others mentioned it as being the archaic version of graceful. I wanted to ask before putting up a PR, but is fixing the word to use the more modern common version of its spelling worth doing?
The main impact would be on the two chart values:

.Values.workers.celery.gracefullTerminationPeriod -> .Values.workers.celery.gracefulTerminationPeriod
.Values.workers.celery.gracefullTermination -> .Values.workers.celery.gracefulTermination

If it's better to package this change in the next major version bump, it can wait. Otherwise if it's worth making this change sooner rather than later I can put up a PR for it.

Because renaming is functionally equivalent to removing a value, we will have to wait for 9.0.0.

Also, I love that the misspelling is technically an archaic version of the word, lol!


Advanced Eventual Consistency Options

I'd like to use Titan with a Cassandra cluster distributed across multiple data centers. I must use CL_LOCAL_QUORUM for reads/writes. That means I can't use locking anywhere. As much as possible I want to use Cassandra’s write timestamps to resolve consistency issues.
There are three changes to Titan that I am interested in:

a “single” option for edges.
read access to the Cassandra timestamps for edges and properties.
eventually consistent uniqueness constraint option.

Detailed explanation follows.
 I can allow things to become inconsistent, but there needs to some way to eventually resolve the inconsistency and a way to hide it until its resolved.
Vertex Properties:
If a property is written at the same time across two data centers a conflict will occur.
If I configure my property key like so:
tx.makeKey("foo")
.single(TypeMaker.UniquenessConsistency.NO_LOCK)
.make();
Cassandra will resolve the conflict for me using the write timestamps. This works for me.
Edges:
If I add an edge with a given label between two vertices twice across the two data centers then I may end up with two edges. Titan doesn't allow me to configure the edge as "single" the same way I can with properties. I need to pick a winning edge myself. I'd rather just leave it up to Cassandra's timestamps.
Edge Properties:
With the current Titan edge system, if I update the same edge property from two different data centers then I end up with two new edges, each with a different value. If I could make the edge "single" like I can with properties, then Cassandra would resolve this using write timestamps.
Luckily, I don't have a use case for multiple properties per edge, which would be harder to resolve.
Property uniqueness constraints:
This case is a real pickle. Say we have a vertex property index:
tx.makeKey("foo")
.single(TypeMaker.UniquenessConsistency.NO_LOCK)
.unique(TypeMaker.UniquenessConsistency.NO_LOCK)
.make();
I could add a new vertex with the same foo property value from both data centers at the same time. I'd end up with two entries in the index. How can I resolve this?
Well, anytime I use this index I can check to see if there are ever more than one vertex returned per index key and merge the vertices before returning them to the user. I need some method to resolve conflicts between the vertices though. Happily, Cassandra provides a write timestamp per column that we can use to pick winners for each property (and edge). It gets trickier if one side of the "split" vertex has been deleted, but I believe this can be resolved using tombstones. When deleting properties or vertices, columns must be overwritten with nulls with a TTL instead of deleted. When repairing a “split” vertex, we can use the write timestamps of any null fields to decide whether the delete won or not.
If I'm over (or under) complicating things, your advice would be appreciated.
Thanks.

1 & 2 are fixed. 3 is likely a NO_FIX


Set http basic auth user/password to empty strings

to prevent NullPointerException when using HTTP without setting
authentication credentials.

This change is

@Thor77 Thanks!


[Fix][api] Fix build parameter error of sqlserver when create.

Fix jdbcUrl splicing error of SQLServer.

Looking forward to your submission of a new pr to complete the changes required by @gabrywu

okay.


Rules

Your extensions are easy to use and the instructions are easy to understand.
Whether all rules can be exported. Need your help

@jsbianxiang
Unfortunately, currently, there is no way to import or export rules. However, it is planned to introduce the ability to import and export rules in the near future, making it more convenient for users.


This fix solves 2 issues when the chaincode container is down:

Parallel requests try to start the CC container at the same time & ends
up in a race condition.
If the first request is starting the CC container, the second request
fails giving an error "Premature execution - chaincode is being launched"

By this fix, the second request will wait until the CC container is
launched. This solves both the above issue and user will not see any of
the above errors.

Thanks, mihir.


WIP: start migrating

Thanks soru for attempting to push us into the new world. Naturally I failed to take any notice and ended up doing #626. Hopefully it will all be worth it.


ls command requries glob to list directories

Use @darkr4y noted in issue #1 that trying to list files with ls c:\ fails.  This is because listing contents of a directory with FindFirstFile requires a glob.  The network payload needs to make this more accessible.

I added some error checking in commit e61e9e3 so that the user is guided to the correct input using the ls command:
5  [sea-green]  -> ls c:\
5  [sea-green]  -> To list directories, add a *
5  [sea-green]  -> ls c:\*
5  [sea-green]  ->
---- BEGIN OUTPUT ----
2016-12-18 21:30:13.480                0 $Recycle.Bin/
2019-04-26 23:09:31.094             5980 bar
2019-05-22 15:56:15.515             7680 bind.exe
2016-07-16 11:43:00.546           384322 bootmgr
2016-07-16 11:43:00.546                1 BOOTNXT
. . .


Add LoRA, QLoRA fine-tuning for HF models

Current onsite LLM class uses full parameter fine-tuning which costly. LoRA fine-tuning will require less memory and prevent overfitting by freezing the pretrained weights.

yeah, sounds good

Hey @VictorOdede, I'd like to work on this.

Kindly mention this in the engineering channel on Discord then @mmirman or @cartazio will give you the green light.

Hey @VictorOdede, I'd like to work on this.

@bilal-aamer We certainly won't stop you from working on it :-) but it'd be good to get on a call with @abhigya-sodani to see what is useful to know before starting it!

Sure will do! Did some due diligence already, need some pointers from @abhigya-sodani.

sure bilal lets get on a call soon

If someone is not actively working on this, I can take up this task

Have at it!
On Mon, Aug 14, 2023 at 9:47 AM Hari Vamsi @.***> wrote:

If someone is not actively working on this, I can take up this task
—
Reply to this email directly, view it on GitHub
https://github.com/anarchy-ai/LLM-VM/issues/97#issuecomment-1677349544,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAABBQVSV53J3WQS3P3EZLTXVIT65ANCNFSM6AAAAAA2XRNFKI
.
You are receiving this because you were mentioned.Message ID:
@.***>

https://github.com/anarchy-ai/LLM-VM/issues/7 is also applicable/related to this ticket

Is there any team working on this issue?

if youre confident you can bang something together, have at it! this one of the more complex open tickets, so be honest with yourself! that said, worst case you learn something!


Migrate the docs to SvelteKit

Closes #311. Work in progress.

Replace all the ./ in src/app.html with / and it will fix your styles

Replace all the ./ in src/app.html with / and it will fix your styles

Another alternative you could test would be adding a <base href='/'> tag at the top of your app.html file. That could potentially fix the broken links I mentioned in https://github.com/illright/attractions/issues/311#issuecomment-943636158

Superseded by #349


Added support for multi-column keys for CSV Import.

Closes: #3794
What's Changed

TODO

[ ] Explanation of Change(s)
[ ] Added change log fragment(s) (for more information see the documentation)
[ ] Attached Screenshots, Payload Example
[ ] Unit, Integration Tests
[ ] Documentation Updates (when adding/changing features)
[ ] Example Plugin Updates (when adding/changing features)
[ ] Outline Remaining Work, Constraints from Design

@timizuoebideri1 please fix markdownlint and we should be set

CI failing due to #4299.


[Question] Way to determine whether photo is taken from camera or selected from gallery

Question
Is there any way to determine whether the photo you receive has been taken from a shot with the camera or existing one selected via the gallery after using showImagePicker method?

Sorry we could not respond in time.
Version 2.x.x is not longer supported, please upgrade to 3.x.x.
showImagePicker API is removed


Automate the test plan to run via script based on test plan

Currently, the test plan is being run manually and evaluated.

Automate running test plan via script as manually running it is error-prone.

Depends on: #142

@mpgreg can you link this issue to the PR you have created? @vatsrahul1001 can add more details as required, please?

Some feedback.

Should consider NOT providing 'answer' to vectordb search.  The point of the search is to see which links are retrieved. This will save cost too.  Use near_text instead of with_ask.
Remove duplicates from references.

@vatsrahul1001 @mpgreg can you create the PR for this, please?

@sunank200 as discussed this should not be part of be 28th release, also @mpgreg can you raise the PR of your script whenever you have bandwidth afterwards we can close this

Waiting for @mpgreg to create the PR with the script that he developed

@mpgreg we need to plan this out, let us know if there is any update on this?

Waiting for an ETA on this @mpgreg

@mpgreg This is super important for testing phase 2.5 changes. Any update on this?

@mpgreg can you link the PR to this issue please?


Editor not support Chinese in Safari with iPad/iPhone

codemirror  editor can not support chinese in safari from iPad/iPhone.

Hi,
I am sorry to hear that but I don't see how we could fix that :/


Discrepancy in gpt4o-mini Results on MSMarco Compared to Reported Results

Thank you for providing this excellent benchmark and sharing the evaluation results across various models!
When we tested the MSMARCO task using the evaluation code provided in this GitHub repository, we observed that the results seem to differ significantly from the numbers reported in the paper. Specifically, we used the same model (gpt-4o-mini-2024-07-18) as mentioned, but for the different lengths in MSMARCO, we only achieved results like 67（8k）, 58(16k), 46(32k), 30(64k), and 22(128k).
Could you kindly advise if there's anything extra we need to do in order to reproduce the results from the paper? Currently, we are using the code exactly as provided in this repository.
Thank you!

Hi, thank you for your interest in our work!
Can you please show me the arguments that were saved in your output files and the name of the output file?

Of course! The arguments saved in the output file are as follows:
"args": {
    "config": "configs/rerank.yaml",
    "tag": "eval",
    "model_name_or_path": "gpt-4o-mini-2024-07-18",
    "use_vllm": false,
    "datasets": "msmarco_rerank_psg",
    "demo_files": "data/msmarco/test_reranking_data_k10_dep3.jsonl",
    "test_files": "data/msmarco/test_reranking_data_k1000_dep3.jsonl",
    "output_dir": "output/gpt-4o-mini-2024-07-18",
    "overwrite": false,
    "max_test_samples": 100,
    "num_workers": 4,
    "num_depths": 10,
    "popularity_threshold": 3,
    "shots": 2,
    "input_max_length": 131072,
    "do_sample": false,
    "generation_max_length": 200,
    "generation_min_length": 0,
    "temperature": 1.0,
    "top_p": 1.0,
    "stop_newline": false,
    "seed": 42,
    "no_cuda": false,
    "no_bf16": false,
    "no_torch_compile": false,
    "use_chat_template": false,
    "rope_theta": null,
    "debug": false,
    "count_tokens": false,
    "stop_new_line": true
}

And the filename is msmarco_rerank_psg_eval_test_reranking_data_k1000_dep3_in131072_size100_shots2_sampFalsemax200min0t1.0p1.0_chatFalse_42.json.
Let me know if you need further information!

I re-ran these experiments today, and I got<EMAIL_ADDRESS>on 128k. It appears that the key difference between the arguments is use_chat_template, where it should be True for all API models. (it is set to False by default for running the base models).
You may refer to the script scripts/run_api.sh for reproducing the results on the API models.
Note that you might still see some small differences in the results (up to 1-3 absolute point differences) due to the nondeterministic nature of the APIs.
Hope this helps!

It's strange—I directly used the command bash scripts/run_api.sh, but the results are still quite different. Could you please share your output files so I can check which step may have gone wrong?
You're truly appreciated!

You can find the result files here: https://drive.google.com/file/d/1PDRFhRXn4YcZ5IH9250gC5xe4CixXK4S/view?usp=sharing
What results are you getting from bash scripts/run_api.sh? Let me know if you end up finding the differences, thanks!

Thank you very much! I compared the two and found that the only difference lies in the demos. Since the data.py code uses a hash, this difference is normal (or possibly need an extra hash seed setting). Additionally, there are some fluctuations in the API results. I'm not exactly sure where the difference comes from. I tried using the same inputs as you (including the demos), but the results still vary quite a bit. I suspect the API might be the cause, but I need to check further. If I find anything else later, I’ll be sure to share it with you! Thanks again!

Thanks for catching this! I will update the code with a deterministic hash function to get the demos.

Hi, thank you very much for your attention and response!
I've encountered an issue while aligning the evaluation results for MSMARCO. Specifically, there seems to be a problem with the qrels used in the metric calculations. In the code, this occurs between lines 305 and 321 of data.py.
The logic used here —
data = data.filter(lambda x: x[key] in keys)
keys = random.sample(sorted(keys), min(max_test_samples, len(keys)))

— appears correct. However, the data field data['qid'] is not a unique primary key, meaning that multiple records share the same qid, which causes the filter operation to act more like a random sampler.
More critically, in the get_qrels function, the qrels are assigned as:
qrels[d["qid"]] = {c["id"]: c["label"] for c in d["ctxs"]}

Since d["qid"] is not unique, it seems like later data might overwrite earlier entries, and the overwritten entries are not identical. This would likely lead to issues when computing the results in the calculate_retrieval_metrics function using:
pytrec_eval.RelevanceEvaluator(qrels, {map_string, ndcg_string, recall_string, precision_string, "recip_rank"})

Could you please provide insights or suggestions on how to address this issue?
Thank you once again for your time and for your valuable work!


Bump dependencies

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (db9bf6e) 22.20% compared to head (6bcbf8f) 19.38%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #82      +/-   ##
==========================================
- Coverage   22.20%   19.38%   -2.83%
==========================================
  Files         111      111
  Lines         761      872     +111
  Branches       35       35
==========================================
  Hits          169      169
- Misses        592      703     +111

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


Revert "Principles and Phases of Design Thinking"

Reverts journey4201/design-thinking-intro#3

approve


List SHOW ROLES in info schema table

Add SHOW ROLES to this table: https://www.cockroachlabs.com/docs/v2.0/information-schema.html

From @BramGruneir:

While in there, it would be great to show some more examples.  I find them very lacking. Also, it’s good to know how to link a show within a select. SELECT database_name AS datname FROM [SHOW DATABASES]; for example since it is not obvious at all.


Question on fees

It is to do about the way the fees are collected for the operators.
Hi mate have a question , the fees are added when someone withdraws there coins/tokens from the web wallet, but say for example someone saves there private keys then adds that wallet, for example to there metamask and moves the coins to another wallet, would be that a way for them to get around paying any fees??
Ok see that's a way for people to get around paying the fee because they can export the private key into another wallet like metamask and then move the funds and knowing people in crypto they would catch onto that fast , adding the fee to the trades would get rid of that problem.
So i was wondering if we could have it so the fees can be added to each actual trade , to stop people from doing this ??

this is not possible, alas. We can add options for hide the "show private key" in dropdown menu, optionally. here you need to think

ok so the fee cant be applied to trades ??
well the option to make it so it doesnt show is a option

fee for trades in process. will be later

is there a time frame for this and will it be possible to have the option to hide private key in menu??

May be today or tomorrow. Task created


default configurePhase fails if there is no configure script

This is in current master.
The problem is that if there is no configure script and $dontDisableStatic is not set, or is set to false. Then the configurePhase hangs. This is because the grep -q enable-static $configureScript; becomes grep -q enable-static if $configureScript is not set. Thats just grep waiting on input that never arrives.
The generated configure phase looks as follows:
onfigurePhase is a function
configurePhase ()
{
    runHook preConfigure;
    if [ -z "$configureScript" -a -x ./configure ]; then
        configureScript=./configure;
    fi;
    if [ -z "$dontFixLibtool" ]; then
        find . -iname "ltmain.sh" | while read i; do
            echo "fixing libtool script $i";
            fixLibtool $i;
        done;
    fi;
    if [ -z "$dontAddPrefix" ]; then
        configureFlags="${prefixKey:---prefix=}$prefix $configureFlags";
    fi;
    if [ -z "$dontAddDisableDepTrack" ]; then
        if grep -q dependency-tracking $configureScript; then
            configureFlags="--disable-dependency-tracking $configureFlags";
        fi;
    fi;
    if [ -z "$dontDisableStatic" ]; then
        if grep -q enable-static $configureScript; then
            configureFlags="--disable-static $configureFlags";
        fi;
    fi;
    if [ -n "$configureScript" ]; then
        echo "configure flags: $configureFlags ${configureFlagsArray[@]}";
        $configureScript $configureFlags "${configureFlagsArray[@]}";
    else
        echo "no configure script, doing nothing";
    fi;
    runHook postConfigure
}

``

I  see, this is my fault, introduced recently. I should get to fix it properly within a few days, unless someone beats me to it.

I would fix it like this:
diff --git a/pkgs/stdenv/generic/setup.sh b/pkgs/stdenv/generic/setup.sh
index 102a8f2..c4f9a6a 100644
--- a/pkgs/stdenv/generic/setup.sh
+++ b/pkgs/stdenv/generic/setup.sh
@@ -639,14 +639,14 @@ configurePhase() {

     # Add --disable-dependency-tracking to speed up some builds.
     if [ -z "$dontAddDisableDepTrack" ]; then
-        if grep -q dependency-tracking $configureScript; then
+        if grep -q dependency-tracking "$configureScript"; then
             configureFlags="--disable-dependency-tracking $configureFlags"
         fi
     fi

     # By default, disable static builds.
     if [ -z "$dontDisableStatic" ]; then
-        if grep -q enable-static $configureScript; then
+        if grep -q enable-static "$configureScript"; then
             configureFlags="--disable-static $configureFlags"
         fi
     fi

Would you also test it on your use case?
I think it's better to quote this variable in general, because it is common and even documented to pass space-separated commands in there. The greps should just fail in that case and if won't proceed which seems fine for such cases, and it's certainly better than passing additional unintended parameters to grep (which was happening all the time before).

Fix in staging as 89036ef76a. Reopen if you think this wasn't fixed.


Can't find extension on VS

I'm following the instructions to install this preview extension but had no success, the extension don't show as you can see on these screenshots.

@andrevlins Apologies, but it seems I've let the temporary links to the unofficial feed expire, which might be the cause of your issue.  I've generated new builds/links; please try again and let me know if you run into any other issues.

Forgot to mention the visual studio version:

Unfortunately I cannot use version 17.8 since I am following and trying to collaborate with the .net aspire project and it requires at least 17.9.
I'll try to get time to install 17.8 side by side just to test the extension installation.

I use 17.8 and 17.9 side-by-side for working with Aspire as well.  While perhaps not super convenient, I found the two versions of VS do work well together.  Since there are breaking manifest/APIs changes between 17.8 and 17.9, I don't know viability of supporting both; I'd need to investigate our options there.

Thanks @philliphoff I'll try side-by-side

@andrevlins FYI, I've built/published a new version of the extension that works around the Container Tools issue.

Closing as resolved (with updated links and a workaround for the Container Tools issue).


Sync Recordings with Cryostat

The operator should create Recording objects in response to a user creating a recording in Cryostat outside of the operator (e.g. via web client, HTTP API). Perhaps we can use Cryostat's notification channel to do this.

Having the operator listen on Cryostat's notification channel may not scale if the operator becomes cluster-scoped. We would have to listen on an arbitrary number of WebSockets, one for each Cryostat deployed by the operator. @andrewazores any thoughts on this?

That's true. Perhaps it would only make sense to deploy the Operator at cluster-scope if/when we can also support that deployment scenario in Cryostat itself, so that there is still a 1:1 relationship of Operators to Cryostats. I think that's eventually something we want to support in Cryostat anyway, since deploying n Cryostats for n namespaces is already not the best scalability-wise.

Makes sense. In that case, it should be fine to move forward with using the notification channel in the operator, if that's how we want to handle this.

I think so - this is pretty much exactly the scenario the notification channel exists for. We definitely want the HTTP API and CRD API to reflect the same system state and as close to real-time-synchronized as possible. I imagine as a user it would be very confusing and seem like something was very broken if I could create a recording in the web-client and see it there, even after closing/reopening the tab, trying a new browser, etc., but never see that same recording appear as a Custom Resource. But then if I do it in reverse and create a recording via CR, it would appear instantaneously in the web-client(!?).

I suppose there's still a chance these could fall out of sync, if there's a network issue or the operator pod crashes. Perhaps we can worry about that later though.

Right, that's true. Not too much we can do about a network issue if we don't detect it, but perhaps if we do detect a problem we can do an HTTP poll to re-sync. An initial HTTP poll on startup, after (re-)establishing the notification channel connection, would also help re-sync if the operator crashes.

I encountered a bit of a problem mapping the target listed in the notification to a Pod. In the operator's API, the Recording object needs to reference a FlightRecorder object, which in turn references a target Pod.
The notification contains the JMX service URL, which gives the pod IP:
{"meta":{"category":"RecordingCreated","type":{"type":"application","subType":"json"},"serverTime":1622145272},"message":{"recording":"test","target":"service:jmx:rmi:///jndi/rmi://<IP_ADDRESS>:9096/jmxrmi"}}
I haven't yet come across a way of efficiently mapping this pod IP back to a pod name. Reverse DNS lookup helps a bit for Pods created by Deployments, but not in the general case. [1]
$ nslookup <IP_ADDRESS>
<IP_ADDRESS>.in-addr.arpa	name = 10-217-0-13.quarkus-test.cryostat-operator-system.svc.cluster.local.

[1] https://kubernetes.io/docs/concepts/services-networking/dns-pod-service/#a-aaaa-records-1

https://github.com/cryostatio/cryostat/issues/280
I think an implementation solving the above issue would unblock what you're facing here.

cryostatio/cryostat#280
I think an implementation solving the above issue would unblock what you're facing here.

Possibly, but even the existing alias would help because it's the pod name. I think the problem is more that there's no way to get the ServiceRef from the target ID/JMXServiceURL without looping through them all. For example, the Recording POST handler only knows the target ID it was provided, so it can't provide any more detailed information in the notification without doing a linear search for the ServiceRef each time.

Ah, yes that's also true. I hadn't filed an issue for it but this same deficiency is on my radar, although I'm not sure how best to tackle it yet. We could simply maintain a mapping of service URLs to ServiceRefs for reverse lookup, but that's really just a bandaid fix. A better solution might be to actually and finally decouple targetIds from service URLs - when we detect a new target and create a ServiceRef for it, maybe we should actually generate a unique ID for that target and use that ID to reference it everywhere that we currently expect the service URL as the "targetId". We could then cache these IDs for reverse lookup of ServiceRefs.
This also has the side benefit of not allowing API requests to arbitrary targets, at least not without first going through the API to create a custom target definition, which could do some kind of validation on the provided URL beyond making sure it's a well-formed URL.

Sounds like a good idea. If you were to use the pod name and perhaps the namespace in the target ID for the Kubernetes case, you could even avoid having to cache the mappings since you could just look up the pod directly to recreate the ServiceRef.

The operator's Recording API has been deprecated with #349.


Mutation tracking

who in Israel is tracking mutations of coronavirus and linking the specific mutation (and patient information?) to development of disease state and/or prognosis?

this is also related to distinguishing super-spreaders from others. See attached paper about clusters in Singapore.
three clusters.pdf


Invalid driver file

Hi,
Just downloaded LINQPad 5.36.03.
I get the error message "Invalid driver file" when I try to add the LiteDBPad.lpx file.

I've downloaded and installed under LINQPad 5.36.03 and seems working fine.
Can you rename LiteDBPad.lpx in LiteDBPad.zip and try to decompress it somewhere, just to be sure that file isn't corrupted.

That was the problem.
Sorry for wasting your time.


tools/version.sh: Add the remaining cut to VERSION

Summary
The idea is to have something like EXTRAVERSION. This is useful for
getting RC tags, development tags and for customizing the version. For
example using the tag nuttx-10.0.0-RC0 the VERSION will be
10.0.0-RC0
Impact
I really have no idea, I am still assimilating NuttX. So I am leaving this PR as WIP.
This is a suggestion (also to help me understand everything and do something hands-on), so I will wait for feedback from the maintainers.
Testing
Tested on Linux user-mode:

Thank you @microhobby  for implementing the suggested approach!


🧑💻 :  Improve Frontend UI and Fix Note Deletion Issue

Description:
Current Problems:

Frontend Design:

The current user interface for the note-taking app appears cluttered and lacks visual appeal.
The card layout for notes is not visually distinct enough, making it harder for users to differentiate between individual notes.
The overall layout does not utilize the available space effectively, particularly in terms of note content and the form for adding new notes.

Note Deletion Issue:

After a new note is written and saved, the note is not being deleted or cleared from the input area.
This results in old notes remaining in the input field, which could confuse users or lead to unintentional overwriting of previous notes.

Proposed Solutions:

Frontend Design Improvements:
Card Layout Enhancements:

Make individual note cards more distinct by adding better margins, padding, and shadows to enhance visual separation.
Implement better font hierarchy (e.g., larger titles, smaller descriptions) to improve readability.
Responsive Design Adjustments:
a. Improve the responsiveness of the layout to ensure the app looks good on different screen sizes.
b. Ensure that the "Add Note" button is clearly separated and well-positioned.
c. Color Scheme Refinement:
d. Update the color scheme to make the "Add Note" button more consistent with the overall theme.
e. Use more visually appealing color contrasts to differentiate buttons and text areas.

Note Deletion After Save:

a. Fix the issue where after saving a note, the input area retains the old content.
b. After successfully saving the note, clear the note input field so that users can easily start writing a new note.

Hi @iamrahulmahato,
Can you assign me this issue ?


[Gally]: master <- dev

Automatically created by Git-Ally

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Report imported battles when players join

This was requested by the smogon tournament directors.
Normally when a battle is created, and Config.reportbattles is set to a truthy value, it is reported in lobby or the the room Config.reportbattles specifies. Battles created with /importinputlog are not logged though. This attempts to fix that by calling Rooms.global.onCreateBattleRoom when the battle gets both of its players.

Bumping this @Zarel

Yeah fsr I didn't get the original fix?

Has happened to me before to with reviews, no big deal.


Global Saves&Follows tracking, SavedShowItemRow fragmentification

This PR is for LD-196 and does two things:

Adds tracking

for tapping into the saves tab
for tapping save/unsave

Refactors the SavedShowItemRow component into a fragment container as discussed in 1331

I'm going to leave a few comments inline describing the fragment refactor – please let me know what I can clarify.

Fails

:no_entry_sign:

No CHANGELOG added.

:no_entry_sign:

Missing Test Files:

src/lib/Components/Lists/__tests__/SavedShowItemRow-tests.tsx
src/lib/Scenes/Favorites/Components/Shows/__tests__/index-tests.tsx
src/lib/Scenes/Favorites/__tests__/index-tests.tsx

If these files are supposed to not exist, please update your PR body to include "#skip_new_tests".

  Generated by :no_entry_sign: dangerJS

Looks great Ash!


initial macos support

Compiles, and maybe even runs if luck is on your side.

Very much appreciated, but as you can see, the build system is gonna need some adjustment to support macOS :D and as I mentioned on Discord already, we have no one who owns a Mac, so testing that things work is a bit difficult. I'll leave this open and we can see how feasible macOS support would be.

Thanks for the patch, i'll see if i find time looking into cleaning it up and making to work with azure pipelines, as it does provide necessary host agent to compile the mod on macos. In the meantime, that boost integration broke the other platforms, i will need to check what headers are missing in deboost to make it working with macos (i might have consciously omitted macos related headers when i made the repack).

Thank you very much for your patch again.

I did some cleanups as well as refactored the cmake project files (and some other things).
Our CI should now be able to build and test macOs binaries.
Our deployment CI should be able to produce macOS binaries and pack them along with binaries from other platforms to produce the final release package.
I have only managed to test the qagame binary with etlded, but that's as far as i can go.


Idea for linearization of the Editor grid storage

Converts Editor::grid from a nested Vec<Vec<GridCell>> to a Vec<GridCell> structure.
Does the same thing with Editor::dirty.
I'm opening this pull request just to discuss this as a possible approach. I believe this specific implementation can be improved (maybe by encapsulating a bit more the grid itself? Maybe extract grid and dirty into a separate struct?), but I also think it shows promise.

I really like the idea of encapsulating the grid structure in its own type with access functions to make it easy to understand while still reducing the need for extra allocations.
I've been thinking about the ways in which the Vec of Vec approach is inefficient:

Each Vec if not initialized to a given size, likely has allocated extra space for additions to the array
Size management overhead at the beginning of the Vec (negligible)
Extra bounds checks for both the row and column

Those were the ones I thought of off the top of my head. Can you help me understand further problems? So part of me feels the local change isn't worth the extra minor complexity since the above issues feel small to me. Not to mention, none of them show up on perf measurements as an issue.
Regardless, abstracting the storage to some general purpose 2d datastructure seems completely reasonable and represents the best of both worlds to me. The internals can be whatever is most performant while the externals can expose an interface that makes sense for the domain. This would also have the added benefit that both the cells themselves and the dirty tracking can use the same data structure and have the same api. This way any perf improvements or what have you can be shared etc.
In any case, thanks so much for your continued help. I appreciate the rigor you're lending. Hopefully I can figure out the visual issues youve been running into.

I would be happy to take a stab at abstracting the data structure soonish. You've already done a lot

It's no trouble at all. Neovide is the first vim client that fits my specific needs:

Good hi-dpi support (I use an extremely high resolution screen at work)
Fast rendering at high resolution (same reason as above)
Good font rendering
Doesn't use OpenGL (OpenGL doesn't work at all through VPN on Windows)
Its source code is small enough for me to wrap my head around it

No other client fits those, so even though I have the line ordering issue, I'm already using it as my main driver on Windows. Once the issue with the keyboard modifiers is solved on Linux, I'll start using it there too :)
I don't have much free time (yay kids!), but I'll happily hack on neovide when I can to improve my own life :smile:

As for the reasons for the linearization, there's a couple more.
First, going through the nested Vecs will require extra pointer chasing (with associated cache invalidations), which will hurt performance.
Second, a sizeable chunk of religious zealotry on my part. Neovide's performance does plummet quite quickly under heavy cpu load, so I'm also looking into ways of improving that. And because proper performance analysis is a lot more work than seeing a usual suspect and start shouting "Wiiitch! Bring the torches!"... well, that's how we end up with this pull request.
Still, I agree that splitting the grid into its own structure will be beneficial for both performance and readability. I think I'll give it a try today if I have some time

Still unfinished, but starting to get there

I still have an issue with scrolling, I think I introduced a bug in the extraction. I'll try to look into it today

Ok, scrolling issue solved. PR is looking much better now after the feedback :-)

Looks awesome. This should be a big improvement.


Server-to-Server OAuth not working as expected :(

Traceback (most recent call last):
File "/root/zoom-recording-downloader/zoom-recording-downloader.py", line 325, in
main()
File "/root/zoom-recording-downloader/zoom-recording-downloader.py", line 292, in main
downloads = get_downloads(recording)
File "/root/zoom-recording-downloader/zoom-recording-downloader.py", line 157, in get_downloads
download_url = download['download_url'] + "?access_token=" + JWT_TOKEN
NameError: name 'JWT_TOKEN' is not defined

Working on a fix right now

Fix added to master branch.

I have just now verified!
the new git clone of the repo and the new instruction of using zoom-recording-downloader.conf seems to be great and working great. it's downloading now. Please mark this issue as resloved.


feat: Federating CSS/scss NextJs

Description
Making it easier to use css and scss files without losing the benefits / compatibility with module federation and SSR.
We can federate and share javascript easily between applications, it is particularly challenging with SSR though when trying to federate something like a button, because on the server side rendering currently struggles to process css and scss files that are not in the JS itself.
Css options:

Inline CSS
Normal CSS (external file)
CSS in JS
Styled Components
CSS module
Sass/SCSS/Less
can be adjusted during the implementation

To do:

Prepare MF examples for React with different variants of Options of how to load CSS.
Prepare MF examples for Next with different variants of Options of how to load CSS.
Investigate approaches/best practices on how to share the css between MF.
Analyse how do we register CSS for our bundles when we’re exposing it.

This one is related to this RFC discussion #483

We may want to include Tailwind as one of the examples given its popularity.

Prepared PR with examples for React https://github.com/module-federation/module-federation-examples/pull/2577
As a result of shared/namespaced based css - is css.modules.

Css federation should work, webpack handles side effects. Is the problem you're referring to related to flash of unstyled text? Like the css chunks are not getting SSRd along with the HTML, and end up loading after app boots?

Analyse css including into ssr
Analyse ways how to load it with js off.
prepare comparison table for items above

partial results:

Nextjs SSR and any type of federating styling not possible - federated components in nextjs only ssr: false
React-dom SSR - federated CSS styling available with isomorphic-style-loader (tricky thing with context provider and withStyles - it should be shared from single place)
React-dom SSR  - federated styled-components styling available (https://styled-components.com/docs/advanced#server-side-rendering)
React-dom SSR - combination of CSS and styled components available

future plans:

React-dom and Jss analysis
preparation of separate SSR only examples of combinations

PR created and on Review https://github.com/module-federation/module-federation-examples/pull/2577
there are some CI issues - investigating how code changes impact it


docs: fix simple typo, exluding -> excluding

There is a small typo in README.rst.
Should read excluding rather than exluding.
Semi-automated pull request generated by
https://github.com/timgates42/meticulous/blob/master/docs/NOTE.md

Thank you!


text UI review

Some text changes on text and UI review feedback

Codecov Report

Merging #753 into m_csr will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           m_csr    #753   +/-   ##
=====================================
  Coverage   9.77%   9.77%
=====================================
  Files        492     492
  Lines      12070   12070
=====================================
  Hits        1180    1180
  Misses     10890   10890

Impacted Files
Coverage Δ

...CloudSourceRepositories/CsrCloneWindowContent.xaml
0% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

Thanks


Empty sparse fieldsets

I have a resource (e.g. posts) which has a to-many relationship to other resources (e.g. comments). I do not need any fields of the related resource but their self-links (to fetch them asynchronously on demand). The response should look something like this:
{
  "links": {
    "self": "http://example.com/posts/1?xxx",
  },
  "data": [{
    "type": "posts",
    "id": "1",
    "attributes": {
      "title": "JSON API paints my bikeshed!"
    },
    "relationships": {
      "comments": {
        "links": {
          "self": "http://example.com/posts/1/relationships/comments",
          "related": "http://example.com/posts/1/comments"
        },
        "data": [
          { "type": "comments", "id": "5" },
          { "type": "comments", "id": "12" }
        ]
      }
    },
    "links": {
      "self": "http://example.com/posts/1"
    }
  }],
  "included": [{
    "type": "comments",
    "id": "5",
    "links": {
      "self": "http://example.com/comments/5"
    }
  }, {
    "type": "comments",
    "id": "12",
    "links": {
      "self": "http://example.com/comments/12"
    }
  }]
}

My question is how should the URL for the request look like?
My idea would be to include the comments and then use an empty sparse fieldset to avoid getting any comment fields but just the self link (+id, type).
Hence, it should look something like http://example.com/posts/1?include=comments&fields[comments]=[].
The specification is not very detailed about sparse fieldsets or their relation to links.

I think it would just be http://example.com/posts/1?include=comments&fields[comments]=, since that would make fields[comments] have an empty string as its value which, when comma-split, forms an empty list. It looks a little awkward, certainly, but I think that'd be the way to do it.

Yep - I agree with @ethanresnick. I suppose a non-normative note to this effect could be helpful.

P.S. The long-term fix for this would probably be optional links in resource identifier objects, as we've discussed before. But one thing at a time :)

@ethanresnick I thought about such a solution too and would really appreciate it.
Anyway, thanks for your help.


Remove the title from the cookie message link

It’s unnecessary and doesn’t give an accessibility benefit. Thanks to @trevorsaint for catching this.

Thanks @robinwhittleton, it was copied from the config set for the prototype kit, I'll update it there too.


[Formatting Bug]: Slow on big HTML files

Platform
Linux
Template before formatting
Too long template (for example the ones of Keen Themes or any admin Saas template).

Template after formatting
Too long.

Expected Behaviour
The plugin takes several minutes (sometimes never finishes in a dozens minutes) to complete formatting a huge HTML page.
Relevant log output
No response

To avoid this problem, it is recommended to split huge blade files into components.
https://github.com/shufo/blade-formatter/issues/616
You can use anonymous component to split large blade file.
https://laravel.com/docs/11.x/blade#anonymous-components


Support CocoaPods resource bundles

About

Based on the suggestion from https://github.com/line/line-sdk-ios-swift/issues/76
resource_bundles is recommended by CocoaPods over resources

Note

Since LINE SDK is integrated using dynamic framework, resource will be under separated framework, thus we might not have the same duplicated resource in main bundle issues as describe.
But still resource_bundles is a better way.

@eJamesLin Can we still get the images from the bundle when using CocoaPods? I mean for things from here: https://github.com/line/line-sdk-ios-swift/blob/v5.3/LineSDK/LineSDK/Utils/Helpers.swift#L160-L162

Nice! 👍


Migration tool fails to detect that a method lacking a return statement implicitly returns null.

This came up during @vsmenon's migration of samples-dev/swarm.  See https://dart-review.googlesource.com/c/sdk/+/253702/comments/bffd8796_22f80104.

As of 1c7fe7196dd7a542f7f91cd6ea2ac81e1a13146c, the null safety migration tool has been removed from active development and retired. No further work on the tool is planned.
If you still need help, or you believe this issue has been closed in error, please feel free to reopen.


"Score" response body property is conflicting between documentation and example

This page mentions that the score response property is between 0 and 1, however the example response body shows score=401.

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

ID: b2adf00c-d6f4-165f-8be9-01a879386905
Version Independent ID: 08681003-1f0b-5237-b55c-0c2adc3a4676
Content: Translator BreakSentence Method - Azure Cognitive Services
Content Source: articles/cognitive-services/Translator/reference/v3-0-break-sentence.md
Service: cognitive-services
Sub-service: translator-text
GitHub Login: @laujan
Microsoft Alias: lajanuar

@goamn
I've delegated this to content author @laujan, who will review it and offer his insightful opinions.

Hi @goamn, Thank you for your feedback. We ran the POST request and received the correct response that has now been updated in the documentation. Thank you again!

#please-close


Crash on world start, or creation

Describe the Bug
It crashes when I try to join, or create a world, sometimes without even making a crash report.
Reproduction Steps

Install the mod
Create world
Crash
...

Expected Result
It should work
Screenshots and Videos
No response
Crash Report or Log
https://pastebin.com/ztLq2DGX
Operating System
Windows 10
Mod Version
0.5.1f
Minecraft Version
1.20.1
Other Mods
        Architectury
	Chisels-and-bits
	Cloth Config
	Collective
	Comforts
	Create fabric
	Distant Horizons
	When Dungeons Arise
	FTB Chunks
	FTB Library
	fTB Teams
	FTB Ultimine
	handcrafted
	Hide Experimental Warning
	Indium
        Just Enough Items
	Kiwi
        Moonlight
        Player Animator
        Resoursful lib
	Serious Player Animations
	Snow! Real Magic!
	Snowy Spirit
	sodium
	tectonic
	valkyrien skies
        Eureka
        Clockwork

Additional Context
No response

create_interactive is crashing


Validation of encrypted file.

How could we validate that the encryption was really successful after completion?
I assume such an option (e.g. --validate or --verify) would be a good addition to crypto. The only feasible solution I can see right now, though, is to decrypto the file as a new temporary file and then compare this result with the original input file (maybe by comparing md5/sha256 hash values).
Ok, theoretically it could still be the decryption that is messing up in this scenario, but I don't exactly see a better solution.

GPG errors are raised if there are problems with encryption and the non 0 exit status code indicates a problem.   I think that we need to rely on GPG to handle that.


Manage subscription link

I notice that in the email that contains the invoice, there is a link to the customer portal so that the user can manage their subscription.  I'm just wondering if a similar link can be added to the entitlement info returned by the SDK?

Hi @gezquinndesign, this should be working now. Let us know if you find any issues.

Nice, tested it out and it works perfectly, thank you!  The only thing I'd mention is that I was expecting to see it specifically on the PurchasesEntitlementInfo rather than the CustomerInfo but I guess there's some reasoning in why it was added where it was added...

Hi @gezquinndesign,
happy to hear it works well!
And yes, the current solution might feel a bit cumbersome. We are working on changing this structure for the future. We decided to follow this approach to keep similarity with our other SDKs.
We will revisit this decision in the future, but we need to find a good solution that works both for purchases-js and the other SDKs before proceeding.
I'll close this issue to keep the issues tab clean.
Thanks for all your comments and testing, really appreciated!

Cool, I thought that would be the reason. Thanks for getting this feature out so quickly, much appreciated.


[WIP] Use separate keydb for delegated targets

Fixes issue: part 3 of #1084
To implement TAP 12, this pr adds separate keydbs for each delegating instance. This means that the roles delegated to by root will use one keydb, the roles delegated to by targets will use a different one, and so on.
To do so, this pr introduces keydb.create_keydb_from_targets_metadata to make a new keydb for all roles delegated to by a targets metadata file and adds a parent_role field to the roledb to keep track of what keydb should be used when verifying a role.

This is awesome, thanks, Marina!
I'm a bit ambivalent about continuing the use the global key dictionary, even if isolated by repositories. Ideally, this should be done recursively. I understand we need to make progress, but I'm also hesitant about adding code we will need to eventually rewrite anyway.
It's a difficult problem. Thoughts @lukpueh @joshuagl @JustinCappos ?

@mnm678 would you please kindly mark this as a draft PR until ready? Thanks!

In a recent TAP meeting we agreed that it might be better to not rework keydb, but instead hold off the implementation of TAP 12 until we have better key handling infrastructure, as e.g. discussed in https://github.com/theupdateframework/tuf/pull/1060#issuecomment-660056376. Should we close here, @mnm678?

Yes, I'll close this and we can revisit the TAP 12 implementation in the future.


make SoftAssertsExtension thread-safe

Proposed changes
Fix issue from Allure tg
SoftAssertsExtension should use thread local ErrorsCollector
Checklist

[x] Checkstyle and unit tests are passed locally with my changes by running gradlew check chrome_headless firefox_headless command
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

@dtuchs I have merged this PR, but please look at https://github.com/selenide/selenide/pull/1335


Feature/heroku app json

Adds app.json to streamline the creation of instances on heroku for the purpose of reviewing PRs in a production environment.
Note: the parameter for test script in app.json is set to "python manage.py test" when the actual testing suite is at "pytest". However, since our testing suite is broken this would cause a nuisance with CI testing, and prevent deployment. When the testing suite is resolved this setting should be fixed.

@andylitalo this is in response to PR #35. Roll this update into that PR, and I can more easily create a review version of the app on Heroku that will pull in the existing production DB for testing. PS: i could have manually configured that review app, but this way it'll be easier in the future to test in production environment.


Running the application on a Linux system

Hi,
Thanks for the great work!
I am trying to run this software on a couple of SEM/TEM images on a Linux system.
Could you please help me get started on this? I had a look at the readme but wasn't sure how exactly this can be done.
Thanks!

Sorry for no response for a long time. You will have to install QT on your linux machine and compile it using QT. The attached .pro is a project file for QT, and QT will create cmake or makefile.


refine defs for cell cycle transition regulation terms

e.g.
GO biological process
regulation of G1/S transition of mitotic cell cycle (GO:2000045)
Any cell cycle regulatory process that controls the commitment of a cell from G1 to S phase of the mitotic cell cycle.
-> A signalling pathway that results in the activation of a cyclin-dependent  protein kinase.
I will make a list...

GO:1902806    regulation of cell cycle G1/S phase transition
GO:1902749    regulation of cell cycle G2/M phase transition
GO:1902807    negative regulation of cell cycle G1/S phase transition | is_a
GO:1902808    positive regulation of cell cycle G1/S phase transition
GO:1902751    positive regulation of cell cycle G2/M phase transition
GO:1902750    negative regulation of cell cycle G2/M phase transition
GO:0110030    regulation of G2/MI transition of meiotic cell cycle
GO:0010971    positive regulation of G2/M transition of mitotic cell cycle
GO:0110032    positive regulation of G2/MI transition of meiotic cell cycle
I don't think I missed any. basically all children of
GO:1902806    regulation of cell cycle G1/S phase transition
and
GO:1902749    regulation of cell cycle G2/M phase transition
Just add the "clause"
A signalling pathway that results in the activation of a cyclin-dependent protein kinase.
All cell cycle transition regulation  (G1/S and G2/M) is a  highly conserved signalling pathway, and it always converges on a cyclin-dependent protein kinase.
Adding this clause should prevent this been used for the bzillions of genes (trancription/translation/splicing etc. etc.) which cause a cell cycle phenotype (block), but are not regulation per se.

Any sigalling pathway  that  results in the activation of a cyclin dependent kinase to modulate the transition from G2 phase to M phase of the mitotic cell cycle.

How about: Any sigalling pathway  that  results in the activation of a cyclin dependent kinase to modulate the switch from G2 phase to M phase of the mitotic cell cycle.

@jvhayles
Hi Jacky,
We'd like to make these definitions more precise so that the transition terms only get used for cell cycle regulators.
Can we define the transtion as a "switch"?
Cheers
Val

Also is there a way to be precise about the kinase. I guess we could jsut say Cyclin dependent Kinase 1?  Does that cover all  G1/S and G2/M transition regulators though?

That won't work. In human I think its CDK1,2 and 3. But there are other cyclin dependent kinases like the mediator, TFIIH and TefB associated.
What do people call the CDKs to exclude these ones?
Just "cell cycle CDK" maybe?

whatever you do, please hyphenate "cyclin-dependent"

In a def i would. Here I'm just typing quickly.

apologies
positive regulation
Any sigalling pathway that results in the activation of a cyclin-dependent kinase to modulate the switch from G2 phase to M phase of the cell cycle.
negative regulation
Any sigalling pathway that results in the inhibition of a cyclin-dependent kinase to modulate the switch from G2 phase to M phase of the cell cycle.
regulation
Any sigalling pathway that results in the activation or  inhibition of a cyclin-dependent kinase to modulate the switch from G2 phase to M phase of the cell cycle.

I found more terms than you listed - is this right ?
name: regulation of G2/M transition of mitotic cell cycle
name: negative regulation of G2/M transition of mitotic cell cycle
positive regulation of G2/M transition of mitotic cell cycle
name: regulation of G2/MI transition of meiotic cell cycle
name: negative regulation of G2/MI transition of meiotic cell cycle
name: positive regulation of G2/MI transition of meiotic cell cycle
name: regulation of cell cycle G2/M phase transition
name: positive regulation of cell cycle G2/M phase transition
name: negative regulation of cell cycle G2/M phase transition
name: regulation of cell cycle G1/S phase transition
name: negative regulation of cell cycle G1/S phase transition
name: positive regulation of cell cycle G1/S phase transition
name: regulation of G1/S transition of mitotic cell cycle
name: positive regulation of G1/S transition of mitotic cell cycle
name: negative regulation of G1/S transition of mitotic cell cycle
Thanks, Pascale

Yes I forgot about the meiotic and the grouping terms.


Inference cpu Docker container

I am trying to run the inference cpu docker container however I am running into an issue that it is not setting the BRAIN_MASKING environ variable correctly and therefore not importing brain_extraction function in the image_processing script.
Error: /tmp/zou_2519.log - No such file or directory.
loading nifti files
registration to MNI template space
performing N3 bias correction
performing brain extraction using ANTsPyNet
Traceback (most recent call last):
File "/app/preprocess.py", line 94, in
noelImageProcessor(
File "/app/deepMask/app/utils/image_processing.py", line 440, in pipeline
self.__skull_stripping()
File "/app/deepMask/app/utils/image_processing.py", line 181, in __skull_stripping
prob = brain_extraction(self._t1_n4, modality="t1")
NameError: name 'brain_extraction' is not defined
I am running the Docker image on my Mac OS w M3 processor.
Thanks

Could you please fill out the following information, and the actual command executed?
Your Environment

Version used (e.g. v1.1.1 or commit hash):
Environment name and version [e.g. Docker (noelmni/deep-fcd:v1.1.2), M1/Intel macOS 14.2.1 with Conda (Python 3.7), Linux (Conda 4.11 w/ Python 3.7)]:
Hardware (e.g., 4 vCPUs, 16GB RAM, NVIDIA TITAN RTX w/ 24GB VRAM)
Using GPU or CPU:
Operating System (OS) and version (if not using the Docker image; e.g. Ubuntu 22.04.4 LTS):
OS baremetal or virtualized:

This is the docker command I used except I replaced $PWD w my directory.  CA was the subject ID:
$ docker run --rm -it --init --env OMP_NUM_THREADS=6 --volume=$PWD/io:/io noelmni/deep-fcd:latest /app/inference.py CA T1.nii.gz Flair.nii.gz /io cpu 1 1
Not sure what version this is.  Version is latest?  I don’t know how to tell which docker image version that is pulling.
I am running this on my Mac OS M3 cpu but I am using the docker container which uses ubuntu base image.


WebP support

Is your feature request related to a problem? Please describe.
OnlyM should support displaying webp. Sometimes brothers have brought in several WebP images and aren't aware that these images can't be displayed; as you know websites can optionally serve WebP images instead of png/jpg.
Describe the solution you'd like
WebP images get treated the same as jpg/png.
Describe alternatives you've considered
We run webps through krita, but if they're many images this can take more time than we have.
Additional context
This may be fairly straightforward because OnlyM already bundles ffmpeg.

@flopcat webp and webm compatibility available in the latest pre-release


warning<EMAIL_ADDRESS>Could not find Cargo.lock for /usr/local/cargo/git/checkouts/substrate-contracts-node-....runtime/Cargo.toml, while searching from /tmp/cargo-installFbuv1p/release/build/contracts-parachain-runtime-../out. To fix this, point the WASM_BUILD_WORKSPACE_HINT env variable to the directory of the workspace being compiled.

When I run the following here https://github.com/ltfschoen/XCMTemplate/blob/main/docker/run.sh#L103
cargo install contracts-node --git https://github.com/paritytech/substrate-contracts-node.git --tag ${SUBSTRATE_CONTRACTS_NODE_VERSION} --force --locked

where i'm using versions:
RUST_NIGHTLY="2024-02-17"
NODE_VERSION=v20.12.0
CARGO_CONTRACT_VERSION=4.1.0
SUBSTRATE_CONTRACTS_NODE_VERSION=v0.40.0
it outputs the following error.
   Compiling rococo-runtime v10.0.0
   Compiling contracts-node-runtime v0.40.0 (/usr/local/cargo/git/checkouts/substrate-contracts-node-cf7c16677784d274/17065f3/runtime)
   Compiling contracts-parachain-runtime v0.40.0 (/usr/local/cargo/git/checkouts/substrate-contracts-node-cf7c16677784d274/17065f3/parachain-runtime)
warning<EMAIL_ADDRESS>Could not find `Cargo.lock` for `/usr/local/cargo/git/checkouts/substrate-contracts-node-cf7c16677784d274/17065f3/runtime/Cargo.toml`, while searching from `/tmp/cargo-installFbuv1p/release/build/contracts-node-runtime-5d514765cf8d82a9/out`. To fix this, point the `WASM_BUILD_WORKSPACE_HINT` env variable to the directory of the workspace being compiled.
   Compiling rocksdb v0.21.0
   Compiling kvdb-rocksdb v0.19.0
   Compiling sc-client-db v0.38.0
warning<EMAIL_ADDRESS>Could not find `Cargo.lock` for `/usr/local/cargo/git/checkouts/substrate-contracts-node-cf7c16677784d274/17065f3/parachain-runtime/Cargo.toml`, while searching from `/tmp/cargo-installFbuv1p/release/build/contracts-parachain-runtime-4049840299a089fe/out`. To fix this, point the `WASM_BUILD_WORKSPACE_HINT` env variable to the directory of the workspace being compiled.
   Compiling sc-service v0.38.0

note: it actually outputs each of those warnings three times in a row, i've just removed the duplicates from the logs
that error is the same as the one shown in this old issue https://substrate.stackexchange.com/questions/5182/failed-to-run-custom-build-command-for-contracts-node-runtime-v0-17-0

instead of running
export SUBSTRATE_CONTRACTS_NODE_VERSION=v0.40.0
cargo install contracts-node --git https://github.com/paritytech/substrate-contracts-node.git --tag ${SUBSTRATE_CONTRACTS_NODE_VERSION} --force --locked

if i change it to run the following instead it doesn't generate that error
export SUBSTRATE_CONTRACTS_NODE_VERSION=v0.40.0
git clone https://github.com/paritytech/substrate-contracts-node.git
cd substrate-contracts-node
git checkout ${SUBSTRATE_CONTRACTS_NODE_VERSION}
cargo install --path ./node --force --locked

substrate-contracts-node is published to 'crates.io', so you can just:
cargo install contracts-node --version 0.40.0 --locked


Document Allowance + Compliance Bulk File generation for Inactive Program Codes

Manually Generate using the on-demand bulk file endpoint
If there are any every changes to inactive program code files, manual implementation is needed to go in and generate the inactive programs

@maheese Lets discuss this ticket for internal questions.

Discuss the requirements with the contractors and prioritize.


Set the default pool-overflow to zero

The pool auto-scaling behaviour seems to be creating problems occasionally. It seems best to default to zero and let users set a max-overflow themselves if they wish.

Coverage remained the same at 74.419% when pulling 684fb768b0bcba881373048bc4715b4abef4f90a on sk-default-max-pool-size-zero into 3993cab12b014806cbba682b107df79f80714ab0 on master.


Get rid of type variables in Block Class

var addedBlocks: [Block] = [] //blocks that modify the current block (e.g. two times for a repeat block)
var type: String = "Operation" //types: Operations, Booleans, Numbers
var acceptedTypes: [String] = [] //list of types that can be added to current block (e.g. numbers for a repeat block)

Check that the placeholderBlocks are not required
Add and save attributes on the modifiable block itself
Repeat block needs addedBlocks for now


patches of lumina-fileinfo when the proposed file is not existing or empty

this patch correct 2 issues with non existing or empty files:

lumina-fileinfo file (where file is a not existing file) now return expected values
lumina-fileinfo file.desktop (where file.desktop is an empty or non existing file) now present the correct screen allowing users to enter values of the desktop file

Thanks to last fix, users willing to create a desktop file (via lumina-fm -> new file) in their ~/Desktop folder are able to edit it.

Looks good!


Keep alive

Hello
I am using the library and it is working good except that due to testing phase we don't have lot of pushes some times for more than 6 hours , when this happen always the first push after this we got error
2016-05-03 12:51:24.390  WARN 7084 --- [ntLoopGroup-3-1] c.r.pushy.apns.ApnsClientHandler         : APNs client pipeline caught an exception.
java.io.IOException: Connection reset by peer
        at sun.nio.ch.FileDispatcherImpl.read0(Native Method) ~[na:1.8.0_51]
        at sun.nio.ch.SocketDispatcher.read(SocketDispatcher.java:39) ~[na:1.8.0_51]
        at sun.nio.ch.IOUtil.readIntoNativeBuffer(IOUtil.java:223) ~[na:1.8.0_51]

Then it reconnect and work , is their any way to check if the connection is up before submitting any message or if their is some thing to keep the connection alive on low pushes requests

We introduced automatic sending of PING frames in v0.6.1. Which version of Pushy are you using? How long does it take for connections to close? Do connections close spontaneously, or is it always in response to attempting to send a notification?

I was on version 0.6.0 and just updated to 0.7.1 couple of hours ago , i will keep an eye on it . Based on my monitoring i was getting connection reset only as a response to my attempt to send push .

Okay. I expect the automated pings should resolve this issue; going to close this for now. Please let us know if the issue persists and I'll reopen if necessary.
Thanks!