[Security] Implementing a Proper Backup solution to Protect IP in a High Turnover World

Topic Suggestion
Implementing a Proper Backup solution to Protect IP in a High Turnover World
Proposed title of article
Implementing a Proper Backup solution to Protect IP in a High Turnover World
Proposed article introduction
companies are struggling to protect their intellectual properties (IP) from mishaps. How do you recover from an angry employee sabotaging your product or a remote employee compromising your security measures?
In this article, readers will learn why backups are needed and implement a proper backup solution to protect the IP. Also, the article will make a case for backups as a way to protect companies’ IPs and how they can ensure the safety of their products.
Key takeaways

Why are backups needed?
Why not having backup is a bad idea
Implementing a proper backup solution

How can companies protect their IP?

Article quality
This article will help readers understand why backups are crucial and get to know ways to protect companies' IPs to ensure the safety of their products. The article will be straight to the point and unique as this content is not widely published by other sources.
References
N/A

Thank you for submitting your topic. @LeahKui
After some careful consideration, it struck us that this topic may be a bit over-saturated throughout other blog sites and official documentation.
We believe this is the best way for students to build a great portfolio (for potential employers) is by building what does not exist and what can provide the most value.
Please feel free to suggest an alternate topic to explore. 🚀


Cursors: What's the point of tracking the previous cursor?

I'm trying to wrap my head completely around the cursor pagination. The example looks like this:
GET /books?cursor=10&previous=5&limit=5
With each pagination request, you're supposed to pass the current cursor as the new "previous" cursor. But what's the point of keeping track of the previous cursor? I thought maybe it would be so you could quickly get the previous page, but I don't know how that would make sense. Because if you wanted to get the previous page you could pass the "previous" cursor as the new "current" cursor. But after that I think you'd be stuck. Because you don't have access to the previous-previous cursor (2 cursors back). So it's not like you can easily keep going backwards, so what's the point?

this is a few years old now, so this will be closed. If you are still having issues, please reopen the issue.


128 frame scale error:RuntimeError: Calculated padded input size per channel: (0 x 224 x 224). Kernel size: (2 x 4 x 4). Kernel size can't be greater than actual input size

I read the paper and would like to check the model on 128 frame scale since the paper mentioned a larger scale yields a better performance, but I got an error like:
RuntimeError: Calculated padded input size per channel: (0 x 224 x 224). Kernel size: (2 x 4 x 4). Kernel size can't be greater than actual input size

Is that because of the video 2 npz?

Please try to refer to #43 and adjust the num of frames or check the input data.

Close as no update.


bug: assert value is 0 at the end of load_bytecode

Bug Report
At the end of load_bytecode, we need to assert that value is 0
jmp cond if remaining_bytes != 0;

assert value = 0;
let bytecode = cast([fp], felt*);
return (bytecode=bytecode);

@ClementWalter
The fix is trivial (done in PR here) but not sure how to test the fix except with a local test by changing the hint and ensure there is a  raise from the assert.

@obatirou I'm thinking about a way to patch hints during the tests, not implemented yet but I can try to do this today


Remove Windows "Hack"

https://github.com/tsduck/tsduck/blob/3b50845686528e8c99c2a0c2dc14bdc312353ce8/src/libtsduck/dtv/windows/tsTunerGuts.cpp#L475-L480
This should be removed. I'm not sure how you came to add this, but - based on the assumption that this wasn't about a re-used filtergraph but rather happened across separate TSP process runs - this can only be result of an incorrect driver implementation.
When you think this would still be needed, it should at least be optional but not default behavior.

Hi @SS-FF
This date back to a while in the past. I do not remember exactly which was the tuner model, its driver and the version of Windows. But I do remember that there were inconsistent packets right after a stop/start cycle. I lost a long time tracking that issue and those inconsistent packets were finally from the previous stream (I remember having caught a PAT or PMT packet that was obviously from the previous stream).
I agree with you that this is the result of some incorrect implementation, driver or filter, I cannot decide. But this horrible hack was necessary to make things work (and I will not come back on how much I hate this environment...)
So, if I remove this hack, there is a risk that something goes wrong with someone else using the same tuner (by experience, those drivers are never updated and this kind of bug remains forever).
I know that you are chasing tuning time on #671 but come on, this is only 50 ms, 1/20 second, while you are looking for a 1 or 2 seconds improvement.
At best, I can make it optional and actionable on the command line. But users having issues will probably never have the idea to correlate their imperceptible problems with that option.

I'm more concerned about what the .Flush might do rather than the Sleep. (I haven't debugged and analyzed this yet).

So, if I remove this hack, there is a risk that something goes wrong with someone else using the same tuner (by experience, those drivers are never updated and this kind of bug remains forever).

Driver updates are very much dependent on the manufacturer. Many are still releasing regular updates, but yes, others don't.
But this bug would manifest in every other DVB application as well, so I can't really believe that it would still exist.
50ms sounds little - but these things sum up (e.g., BDA device enumeration is expensive and something that I'm going to tweak - another 150ms...).
Finally, let me ask the following question: Would you find a 50ms hack acceptable when it would be in the Linux implementation?
😆

Would you find a 50ms hack acceptable when it would be in the Linux implementation?

I would put it the other way round. Waiting or polling is always ugly.  Similarly, for something as simple as reading data from an input device, if you need to develop something as complicated as DirectShow filter, this is the symptom of something going wrong with the platform. I do not prefer Linux in principle (Linux has its own flaws elsewhere). I just notice by experience and experimentation that doing something simple and reliable is much more complicated on Windows than Linux.

That's what I mean: You blame it on DirectShow, even though the blame is on the driver. You consider the 50ms acceptable based on your attitude against DirectShow. That's why I asked, whether you would make the same decision on Linux: Make all use cases suffer from a 50ms delay just because there exists a single faulty driver where you don't even remember the brand and model..

(I'm just challenging you a bit for fun - it's been a long stint for me 😄 )

Automatically closed after 648 days without update and "close pending" label set.


carla: Fix NSM compatibility

See https://github.com/NixOS/nixpkgs/issues/117781

[x] 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)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 117880 at 87d158a6 run on aarch64-linux 1

1 package failed to build:

carla

Note that build failures may predate this PR, and could be nondeterministic or hardware dependent.
Please exercise your independent judgement.

Probably best to mark meta.platforms to not support aarch64.

done. I'm keeping this in 2 separate commits as it's not really related.

I can confirm that this PR solves the problem I had with Carla+NSM, thanks a lot.

Actually, I'm thinking we should apply similar patches to all executables of carla:
$ grep -ri appname
.carla-wrapped:exec $PYTHON "$INSTALL_PREFIX"/share/carla/carla --with-appname="$0" --with-libprefix="$INSTALL_PREFIX" "$@"
.carla-patchbay-wrapped:exec $PYTHON "$INSTALL_PREFIX"/share/carla/carla-patchbay --with-appname="$0" --with-libprefix="$INSTALL_PREFIX" "$@"
.carla-jack-single-wrapped:exec $PYTHON "$INSTALL_PREFIX"/share/carla/carla-jack-single --with-appname="$0" --with-libprefix="$INSTALL_PREFIX" "$@"
.carla-jack-multi-wrapped:exec $PYTHON "$INSTALL_PREFIX"/share/carla/carla-jack-multi --with-appname="$0" --with-libprefix="$INSTALL_PREFIX" "$@"
.carla-control-wrapped:exec $PYTHON "$INSTALL_PREFIX"/share/carla/carla-control --with-appname="$0" --with-libprefix="$INSTALL_PREFIX" "$@"
.carla-osc-gui-wrapped:INSTALL_ARGS="--with-appname="${0}""
.carla-rack-wrapped:exec $PYTHON "$INSTALL_PREFIX"/share/carla/carla-rack --with-appname="$0" --with-libprefix="$INSTALL_PREFIX" "$@"

superseded by https://github.com/NixOS/nixpkgs/pull/140129


Outputs this when generating an image

Today I launched notepad and when I try to generate an image, it gives this

What should I do ?

colab ? what is the resolution used for the image ?

colab ? what is the resolution used for the image ?
Yes colab .
Strangely after restarting notepad this message doesn't appear anymore for now.


Integrate Pipeline CLI with new Validation framework

Describe the issue
Integrate Pipeline CLI with new Validation framework

@lresende - is this an umbrella for other issues that we might open or can this be closed by the mentioned merged PR?

Once the new validation service is in place, the cli should call it explicitly as it's not integrated with parsing anymore.


ServiceAccount not getting annotations from global annotations

If i use  this
global:
  annotations:
      eks.amazonaws.com/role-arn: arn:aws:iam::xxxx:role/xxxxxxx-pingelm-pap-irsa

It works
But if i use
global:
  annotations:
      eks.amazonaws.com/role-arn: arn:aws:iam::xxxxxxx:role/xxxxxg-pingelm-pap-irsa
  rbac:
    applyServiceAccountToWorkload: true
    generateGlobalServiceAccount: true
    generateGlobalRoleAndRoleBinding: true

Then i get this error
Error: UPGRADE FAILED: error validating "": error validating data: [apiVersion not set, kind not set]
make: *** [pingdevops] Error 1
Look like there is some issue with service account templates which are not able to parse annottaions

It looks like you have a space in your annotation, is that intentional?

Thanks for opening this issue @mk2134226 , I've opened an issue internally and I'll update this issue when we have a fix.

A fix for this was merged into main and will be included in the next monthly release

@henryrecker-pingidentity  Thanks , is there any simple way to add annotation only to service accounts ? rather to all resources ?

Currently there isn't a way built-in to the chart to do this. You can do this with a kustomize post renderer (similar to https://github.com/pingidentity/pingidentity-devops-getting-started/blob/master/30-helm/read-only-filesystem/kustomize/kustomization.yaml) to set the desired annotation.

@henryrecker-pingidentity This limitation also affects services.  For instance, I need to add a traefik servertransport annotation to the service resources created by the helm chart but can only do so by adding the annotation globally to every resource.  While this doesn't immediately cause an issue, it is very confusing and causes stateful set update issues if the annotation ever changes (The annotation is added to the persistent volume templates).
Is it possible to modify the helm chart to allow annotations for resources that do not currently support sourcing values from the values.yaml?
Unfortunately the kustomize workaround doesn't easily apply as I am using a gitops tool to render the tempaltes, argocd in this case.

Hi @corybolar , thank you for the feedback. I'll open an internal ticket to track this request and will update you here when it is available.

We've merged a change to main to allow more fine-grained control over labels and annotations across various resources created by the chart. It'll be included in the next release.
Annotations and labels can now be set per-product for RBAC resources, Workloads, Pod templates in workloads, Services (External and ClusterIP), and HPA. Global values can also be set, such as to set an annotation for all services created by the chart for example.
Here is an example of some of the new annotations that can be set with this change (labels can be set in the same way):
global:
  annotations:
    globalAnnotation: val

  workload:
    annotations:
      globalWorkloadAnnotation: val
    statefulSet:
      annotations:
        globalSSAnnotation: val
    deployment:
      annotations:
        globalDepAnnotation: val

  rbac:
    generateGlobalServiceAccount: true
    generateGlobalRoleAndRoleBinding: true
    serviceAccountAnnotations:
      globalServiceAccountAnnotation: val
    roleAnnotations:
      globalRoleAnnotation: val
    roleBindingAnnotations:
      globalRoleBindingAnnotation: val

  services:
    annotations:
      globalServiceAnnotation: val
    clusterServiceAnnotations:
      globalClusterServiceAnnotation: val

pingdirectory:
  enabled: true
  annotations:
    pdAnnotation: val
  workload:
    annotations:
      pdWorkloadAnnotation: val
    statefulSet:
      annotations:
        pdSSAnnotation: val
  rbac:
    generateServiceAccount: true
    generateRoleAndRoleBinding: true
    role:
      rules:
      - apiGroups: [""]
        resources: ["secrets"]
        verbs: ["get", "list"]
    serviceAccountAnnotations:
      pdServiceAccountAnnotation: val
    roleAnnotations:
      pdRoleAnnotation: val
    roleBindingAnnotations:
      pdRoleBindingAnnotation: val
  services:
    annotations:
      pdServiceAnnotation: val
    clusterServiceAnnotations:
      pdClusterServiceAnnotation: val

@henryrecker-pingidentity  when is new helm chart  getting released , its more than 1 month now , i am waiting for new chnages

@mk2134226  We've just released version 0.11.0 which includes the new changes - https://helm.pingidentity.com/release-notes/currentRelease/


Fix resource-locker-operator repo name

We saw https://prow.svc.ci.openshift.org/view/gcs/origin-ci-test/logs/periodic-org-sync-redhat-cop/309 failing with the following message after https://github.com/redhat-cop/org/pull/53 was merged.
msg":"Configuration failed: failed to configure redhat-cop team container-cop repos: [failed to configure redhat-cop child team operators-wg repos: [failed to update team 3299468(operators-wg) permissions on repo resource-operator-locker to read: status code 404 not one of [204], body: {\"message\":\"Not Found\",\"documentation_url\":\"https://developer.github.com/v3/teams/#add-or-update-team-repository\"}]]"}

Looks like the name of the https://github.com/redhat-cop/resource-locker-operator repo was put in the config wrong.
/cc @sabre1041 @oybed

https://prow.svc.ci.openshift.org/?job=periodic-org-sync-redhat-cop is green again


Reproduce SSIM/MSE MAD Competition example from paper

MAD Competition is working now, but in order to be certain about it, we want to synthesize some images that match examples from the MATLAB code (though they won't be identical, should be in the ballpark)
Additionally, I found a weird issue with the example in the Simple_MAD notebook: when using po.add_noise to generate the initial image, the image generated would always lie along the forward or reverse diagonal (e.g., from base image [.5, .5] to [.6, .4]), which gives you L1 and L2 loss contours such that the circle is completely circumscribed within the square:

In this case, MAD Competition (with the parameters set up in that notebook) completely failed to find any solution for it. Need to think about both why there appear to be such limited possible values and why MAD Competition has trouble here, but for now that means going back to the earlier way of adding noise. Should maybe add option to specify initial image?

Try to replace current "two loop" method with a penalty in the loss with respect to the other model.  While refactoring, make a single instantiation generate a single image.

Currently (and with #136), MADCompetition does not allow user to specify that starting image. Should change that, right? Since we want them all to be based on the same initial image (rather than different initial images, each with same initial noise)


NTLM Proxy Authorization support

Hi,
I read that "chainedProxyAuthorization" function supports only Basic proxy authorization,
Is there a thought of adding support of NTLM proxy authorization as well?
Thanks.

I'm not planning to implement, but it has been talked about and pull requests are always welcome. My only requirement would be that it is done in a platform-independent way.


We need a way to put commands and events in other files

I managed to implement putting commands in other files in yabluzo, but it's far from ideal and other people are on their own.

I have no idea on how to design this. Any input and suggestions from others is appreciated

I got an idea: we provide a function that loads commands from a specific file.
This function uses import() to get the default export which should be a function that we pass the instance of client as the only argument.
Then the function should return an object satisfying CommandHandlerMap, which will have all of its properties assigned to client.commands

I'm also gonna implement a function to load all commands from a directory

This feature works perfectly really. I do have one thing I want to add though:
I don't want to bot to crash because of one module being faulty.
What do we do instead? Implement a warning system, sending a message when the help command is ran.
Like this we can tell people some module is faulty (provide the filename perhaps? do you agree with this @mybearworld?)

I got an issue once with yabluzo where some module caused the entire thing to just not do anything at all. I had to delete every module one by one and update compiled js files over ftp to tell users I had to disable a part of yabluzo.
I don't want anyone to have to deal with that.
How about we add a verbose mode which can also be triggered by setting an environment variable?

sending a message when the help command is ran.

Wouldn't the console be a better place here? The bot creator would see it instantly, rather than being confused for a bit, and then remembering they can run the help command to check for invalid modules.

I'd do both incase the developer doesn't see one of the two.

It seems that the types are not really working that well.
I'm talking about these:
export type ModuleInitializeFunctionReturnType = Promise<CommandWithName | CommandMap | void> | CommandWithName | CommandMap | void;
export type ModuleInitializeFunction = (client: Client) => ModuleInitializeFunctionReturnType;

The first one causes problems with the autocomplete (see the screenshots):

The problems come from the support for asynchronous initialize functions, which is causing all kinds of weird stuff with typescript like this.
I'm trying to come up with a way to fix this, without removing support for asynchronous intialize functions.
If you're using this feature, ignore the type errors for now, as the library still handles the output of these functions correctly.

I think I should be able to fix the type errors but not the autocomplete as it's technically corrrect.
You'll have to ignore the promise-related stuff

I found a solution:
ModuleInitializeFunction is now either a function that takes the client argument and returns ModuleInitializeFunctionReturnType, or a function that also takes the client argument, but returns Promise<ModuleInitializeFunctionReturnType>.
To get rid of the Promise-related stuff, put after your return object satisfies CommandWithName, like this:

These changes will be committed soon.


Download Test Deck

Received the error below when trying to download test deck.
Cypress encountered an internal error while building this measure. Please contact the Cypress team at https://github.com/projectcypress/cypress/issues, and include the text below:
invalid date
Measure Test Information
Measure: Appropriate Testing for Children with Pharyngitis
HQMF ID: 40280382-5ABD-FA46-015B-499F4310386A
CMS ID: CMS146v6
Provider Name: Jordan, Marcus
Provider NPI:<PHONE_NUMBER>
Provider TIN: 666904271
Provider CCN: 309649
View Patients
Get Known Good Result
C1 and C3
QRDA Errors
--
Reporting Errors
--
Submission Errors
--
Warnings
--
Execution Date
--
C2 and C3
QRDA Errors
--
Reporting Errors
--
Submission Errors
--
Warnings
--
Execution Date
--
start

@moclarke Thanks for getting in contact with us! Can you provide some additional information:

The version of Cypress you were using when you encountered this error?
How the Cypress instance you were using is hosted (VM, AMI, custom install, Cypress Demo Server, etc.)?
What the settings were for the test you created (Measure Bundle used, which Measures were selected, Certification Types selected, etc.)

That should help us debug the issue if need be.
Thanks!
Michael O'Keefe

I don't see the name O'Keefe too often.  That is my maternal grandmother's
name.
Anyway,
We are on Version 3.2.3
The instance is a VM install
We are certifying for 2015 Ambulatory and Inpatient, C-1 thru C-4, 2017
bundle for 2018 reporting.  Below are the measures selected for our custom
list:
CMS55v6       Median Time from ED Arrival to ED Departure for Discharged ED
Patients
CMS111v6      Median Time from ED Arrival to ED Departure for Admitted ED
Patients
CMS104v6     Discharged on Antithrombotic Therapy
CMS71v7       Anticoagulation Therapy for Atrial Fibrillation/Flutter
CMS72v6       Antithrombotic Therapy By End of Hospital Day 2
CMS105v6     Discharged on Statin Medication
CMS107v6     Stroke Education
CMS102v6     Assessed for Rehabilitation
CMS108v6     Venous Thromboembolism Prophylaxis
CMS190v6     Intensive Care Unit Venous Thromboembolism Prophylaxis
CMS113v6     Elective Delivery
CMS9v6         Exclusive Breast Milk Feeding
CMS31v6       Hearing Screening Prior to Hospital Discharge
CMS26v5       Home Management Plan of Care Given to Patient/Caregiver
CMS53v6       Primary PCI Received Within 90 Minutes of Hospital Arrival
CMS125v6     Breast Cancer Screening
CMS130v6     Colorectal Cancer Screening
CMS165v6     Controlling High Blood Pressure
CMS159v6     Depression Remission at Twelve Months
CMS131v6     Diabetes: Eye Exam
CMS122v6     Diabetes: Hemoglobin A1c (HbA1c) Poor Control (>9%)
CMS139v6     Falls: Screening for Future Fall Risk
CMS164v6     Ischemic Vascular Disease (IVD): Use of Aspirin or Another
Antiplatelet
CMS127v6     Pneumococcal Vaccination Status for Older Adults
CMS69v6       Preventive Care and Screening: Body Mass Index (BMI)
Screening and Follow-Up Plan
CMS147v7     Preventive Care and Screening: Influenza Immunization
CMS2v7         Preventive Care and Screening: Screening for Clinical
Depression and Follow-Up Plan
CMS138v6     Preventive Care and Screening: Tobacco Use: Screening and
Cessation Intervention
CMS347v1     Statin Therapy for the Prevention and Treatment of
Cardiovascular Disease
CMS156v6    Use of high-risk medications in the elderly- 1
CMS146v6    Appropriate testing for children with pharyngitis 2
CMS117v6    Childhood immunization status 4
CMS136v7    FollowUp Care for Children Prescribed (ADHD) Med  6
CMS75v6      Children who have dental decay or cavities 8
CMS166v5    Use of imaging studies for low back pain 5
CMS123v6    Diabetes  Foot Exam
I have removed the offending measure from our list in order to move
forward, but we will have to circle back and test it eventually.
Thank you,
Maureen Clarke
On Mon, Sep 17, 2018 at 1:05 PM, okeefm<EMAIL_ADDRESS>wrote:

@moclarke https://github.com/moclarke Thanks for getting in contact
with us! Can you provide some additional information:

The version of Cypress you were using when you encountered this
error?
How the Cypress instance you were using is hosted (VM, AMI, custom
install, Cypress Demo Server, etc.)?
What the settings were for the test you created (Measure Bundle
used, which Measures were selected, Certification Types selected, etc.)

That should help us debug the issue if need be.
Thanks!
Michael O'Keefe
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/projectcypress/cypress/issues/1037#issuecomment-422152020,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ApBa3rgXqjFKnj8aESbZtU3_797QYd-Bks5ucACmgaJpZM4WblUE
.

--
Maureen Clarke
Clinical Applications Coordinator
MAC Group 100
808-295-6594
<EMAIL_ADDRESS>

Hello,
I'm not sure from this what could be causing the issue. Can you post this information to https://oncprojectracking.healthit.gov/support/secure/CreateIssue!default.jspa under the "Cypress" project? That will allow us to better triage and fix the issue.
Thank you!
Michael

Hello,
After talking with one of the other developers, we've seen an issue like this before that was resolved when the user re-created the test with the measure. Can you try re-creating the test, and see whether it succeeds?
Thank you,
Michael

@moclarke I'm going to close this issue as resolved. If you continue to observe this issue, please feel free to post to the JIRA tracker linked above.


Use official jamm

According to the comment mark
// Swap to use the officially published version of 0.4.x once available
// instead of relying on a community published copy. See
// https://github.com/jbellis/jamm/issues/44 for additional details.

Offical jamm 0.4.0 has been released in July 2023: https://mvnrepository.com/artifact/com.github.jbellis/jamm/0.4.0
Please add a meaningful description for your change here

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI or the workflows README to see a list of phrases to trigger workflows.

Java PreCommit stuck test org.apache.beam.runners.fnexecution.logging.GrpcLoggingServiceTest. testMultipleClientsFailingIsHandledGracefullyByServer not related

assign set of reviewers

Java PreCommit stuck test org.apache.beam.runners.fnexecution.logging.GrpcLoggingServiceTest. testMultipleClientsFailingIsHandledGracefullyByServer not related

Doesn't look like this is normally flaky - https://github.com/apache/beam/actions/workflows/beam_PreCommit_Java.yml?query=event%3Aschedule - so lets get a green signal before merging (kicking off a retry)

GrpcLoggingServiceTest

The rerun is green
I saw the timeout test observed before: #14474 it has higher probability when gradle cache is disabled. Similarly cron jobs appear to be less flaky than the PR that changed java core code.

Also manually tested a dataflow wordcount 2024-01-19_11_04_21-10729576904645109680, checked the worker log there is no warning about jamm, and the job succeeded
merging for now
(there is a followup that build container locally see error that command --provenance does not exist, Add a fix in #30051)


Retrieve master page info in Shape object

Hi @dave-howard,
I need to access the master page info of a Shape in order to access to their attributes (name, master_base_id, etc..)  (AWS-complex-stencils-shapes.zip), as I can see on the code, this master shape is retrieve from here:

But then it access the child_shape[0] for retrieve it as master_shape (I don't understand why), in this part I lost track of the page master attributes that I need (name, master_base_id, etc...)
As you can see, the child_shape[0].text attribute is not informed, then when trying to access shape.text, the attribute self.master_shape.text will be empty.
I need to access to the value (AWS Step Functions workflow) in the only shape of the vsdx file, should it be updated how to retrieve the master page info in order to do that?
Thank for your help.
Cordially,
Juan.

Hi @dave-howard ,
I just added this comment to the PR:
https://github.com/dave-howard/vsdx/pull/48#issuecomment-1195136994

Hi @jmgarcia-iriusrisk,
I think Shape.text returning an empty string in this case is correct - if we change the behaviour of Shape.text to return value of first child Shape with non-empty text, then that will break some other use cases (i.e. Page.find_shapes_by_text() method)
That said - if it would be useful to have a method of a Shape that returns all text, or first text value.
In the example above - if you get shape by id 7 and check it's text property - you'll get the expected text - but the other peer and patrent shapes have an empty text so that's what gets returned
I've added a new test in da2c7e1a that includes your sample file
Dave

Hi  @dave-howard,
Thanks for your answer and your time, our problem is that we need to go throug all shape's of a page and retrieving their text.
But with the file test_master_multiple_child_shapes.vsdx , in this case, how could we get the text inside the shape (it could have a custom one modified by the user):

Hi @dave-howard , sorry for insists, but this is a lacking feature we need to implement in our code, What would be the best approach for retrieving this custom text inside the component?
Cordially,
Juan.

Hi @dave-howard! thx for answering.
We implemented the following logic for retrieve the text for the master shape:
def get_master_shape_text(shape: Shape) -> str:
    if not shape.master_shape:
        return ""

    result = shape.master_shape.text.replace('\n', '')
    if not result:
        result = get_child_shapes_text(shape.master_shape.child_shapes)

    return (result or "").strip()

def get_child_shapes_text(shapes: [Shape]) -> str:
    if not shapes:
        return ""
    return "".join(shape.text for shape in shapes).replace('\n', '')

Thanks for the update also - I can see how that would work fine for what you are doing. I'll close this issue.


Update documentation in README to include better examples

Codecov Report

Merging #64 (8d8f63e) into master (398558e) will decrease coverage by 0.10%.
The diff coverage is 58.33%.

@@            Coverage Diff             @@
##           master      #64      +/-   ##
==========================================
- Coverage   91.65%   91.54%   -0.11%
==========================================
  Files          53       53
  Lines        2623     2627       +4
==========================================
+ Hits         2404     2405       +1
- Misses        219      222       +3

Impacted Files
Coverage Δ

pyminion/bots/big_money_ultimate.py
34.37% <0.00%> (ø)

pyminion/bots/big_money.py
50.00% <50.00%> (-5.56%)
:arrow_down:

pyminion/bots/base_bot.py
37.33% <100.00%> (ø)

tests/conftest.py
100.00% <100.00%> (ø)

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 398558e...8d8f63e. Read the comment docs.


Install elasticsearch with werf failed, without werf ok

Before proceeding

[X] I didn't find a similar issue

Version
1.2.177
How to reproduce
helm repo add bitnami https://charts.bitnami.com/bitnami

werf helm install --atomic -n elasticsearch my-elasticsearch bitnami/elasticsearch --version 19.4.2  -f value.yaml

value.yaml
master:
  replicaCount: 1
  resources:
    limits:
      cpu: 500m
      memory: 2Gi
    requests:
      cpu: 500m
      memory: 256Mi

data:
  replicaCount: 1
  resources:
    limits:
      cpu: 500m
      memory: 2Gi
    requests:
      cpu: 500m
      memory: 256Mi

coordinating:
  replicaCount: 1
  resources:
    limits:
      cpu: 500m
      memory: 2Gi
    requests:
      cpu: 500m
      memory: 256Mi

metrics:
  enabled: true

Result
log werf by install elasticsearch
https://gist.github.com/patsevanton/663027a4040057011f743f3142d039f3
Expected result
Install elasticsearch with werf passed
Additional information
install without werf
helm install --atomic -n elasticsearch my-elasticsearch bitnami/elasticsearch --version 19.4.2  -f value.yaml
WARNING: Kubernetes configuration file is group-readable. This is insecure. Location: /home/user/.kube/config
WARNING: Kubernetes configuration file is world-readable. This is insecure. Location: /home/user/.kube/config
NAME: my-elasticsearch
LAST DEPLOYED: Tue Oct 11 21:10:11 2022
NAMESPACE: elasticsearch
STATUS: deployed
REVISION: 1
TEST SUITE: None
NOTES:
CHART NAME: elasticsearch
CHART VERSION: 19.4.2
APP VERSION: 8.4.2

-------------------------------------------------------------------------------
 WARNING

    Elasticsearch requires some changes in the kernel of the host machine to
    work as expected. If those values are not set in the underlying operating
    system, the ES containers fail to boot with ERROR messages.

    More information about these requirements can be found in the links below:

      https://www.elastic.co/guide/en/elasticsearch/reference/current/file-descriptors.html
      https://www.elastic.co/guide/en/elasticsearch/reference/current/vm-max-map-count.html

    This chart uses a privileged initContainer to change those settings in the Kernel
    by running: sysctl -w vm.max_map_count=262144 && sysctl -w fs.file-max=65536

** Please be patient while the chart is being deployed **

  Elasticsearch can be accessed within the cluster on port 9200 at my-elasticsearch.elasticsearch.svc.cluster.local

  To access from outside the cluster execute the following commands:

    kubectl port-forward --namespace elasticsearch svc/my-elasticsearch 9200:9200 &
    curl http://<IP_ADDRESS>:9200/

get pod
kubectl get pod -n elasticsearch
NAME                                       READY   STATUS    RESTARTS   AGE
my-elasticsearch-coordinating-0            1/1     Running   0          3m39s
my-elasticsearch-data-0                    1/1     Running   0          3m39s
my-elasticsearch-ingest-0                  1/1     Running   0          3m39s
my-elasticsearch-ingest-1                  1/1     Running   0          3m39s
my-elasticsearch-master-0                  1/1     Running   0          3m39s
my-elasticsearch-metrics-944c9685b-8w486   1/1     Running   0          3m39s

Full log elasticsearch install by helm
https://gist.github.com/patsevanton/32619ebad0087372842f6999c2e03817

This is due to readinessProbe failing. We track all the probes. Usually solved by fixing the probe (e.g. increasing failureThreshold).


Add metacritic score parsing

Add metacritic score parsing from IMDB movie pages.
Note: I was not able to test this. I think this should work, but the HTML class is defined as follows in IMDB:

I'm not sure if the class statement on line 325 needs to include all three classes to make the match.
Motivation for changes:
Detailed changes:

Addressed issues:

Fixes # .

Implemented feature requests:

Feathub #XX.

Config usage if relevant (new plugin or updated schema):
paste_config_here

Log and/or tests output (preferably both):
paste output here

To Do:

[ ] Stuff..

Combined into #2142 Closing.


Sidekiq::Status doesn't play well with Sidekiq Enterprise's encryption feature

When I call perform_async on an encrypted worker it hits the branch in api.rb (379 is a print line I added) that anonymizes things:

This caused the value [encrypted data] to be encrypted and stored in redis. When the job is run, it appropriately decrypts the data and passes that literal string "[encrypted data]" to the method.
This plugin calls display_args which then calls down into the base implementation of display_args.
Making my jobs that need encryption bypass calling include Sidekiq::Status::Worker has fixed the issue for me.
I don't know the middleware stack well enough to know if there is a fix for this or the "solution" is simply not using status workers but I thought I should file something to help others out.

Have you tried adjusting the order of the middleware? It's not guaranteed, but by flipping the order, they might be able to play nicely with each other.


Add in the Galactic release page.

Signed-off-by: Chris Lalancette<EMAIL_ADDRESS>This is so that I can put a note about PR https://github.com/ros2/geometry2/pull/210 into it.

Please reference the new page from https://index.ros.org/doc/ros2/Releases/.


17 pop merge resample

Description

4 files (3 completely new and requirements.txt updated):

notebooks/population/grid_pop_experiment.py: this is purely a development notebook and does not require review.
requirements.txt: updated with necessary package requirements.
src/transport_performance/utils/raster.py: added function to merge and resample raster data.
tests/utils/test_raster.py: unit tests for transport_performance/utils/raster module.

Fixes #17.
Motivation and Context

Added functionality to the utils module to merge and resample rastered data.
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

How Has This Been Tested?

Development notebook used to build methods and functions. QA-ed manually and visualised results in plots. Added unit tests in tests/utils/test_raster.py, using dummy data in pytest fixtures. All unit tests run and pass locally.
Test configuration details:

OS: macOS
Python version: 3.9.13
Java version: 11
Python management system: conda/pip

Advice for reviewer

notebooks/population/grid_pop_experiment.py was purely a development notebook and does not require review for this PR.
Checklist:

[X] My code follows the intended structure of this project
[X] I have performed a self-review of my own code
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have made corresponding changes to the documentation
[X] My changes generate no new warnings
[X] I have added tests that prove my fix is effective or that my feature works
[X] New and existing unit tests pass locally with my changes
[X] Any dependent changes have been merged and published in downstream modules

Additional comments

None

CI runners are failing because geos needs to be installed before pip installation packages (it's a requirement for cartopy). Raised an issue to capture this #32

Added lines to python-package GitHub actions workflow to install geos before the python dependencies (requirement for cartopy). Looking through the log messages of the macOS and Linux runners the installation of cartopy now seems to be OK. So this PR should now also close #32 (PR has been updated to reflect this).

Codecov Report
Patch coverage: 87.50% and project coverage change: +87.50 :tada:

Comparison is base (1442bba) 0.00% compared to head (e4e7f9c) 87.50%.

Additional details and impacted files
@@           Coverage Diff            @@
##           dev      #30       +/-   ##
========================================
+ Coverage     0   87.50%   +87.50%
========================================
  Files        0        1        +1
  Lines        0       32       +32
========================================
+ Hits         0       28       +28
- Misses       0        4        +4

Flag
Coverage Δ

unittests
87.50% <87.50%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

src/transport_performance/utils/raster.py
87.50% <87.50%> (ø)

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

Added a comment in the python-package workflow to make it clear why geos installation is needed. Removed placeholder.py in utils since this isn't needed any more. The PR is up-to-date with these comments and it is now ready for reveiw.


Master

The current implementation starts the animation regardless if it was
running or not when the app goes to the background and then back to the
foreground.
This fixes the issue.

@hactar  You're totally right. Thanks.


Add nil checks to mumble text message handling

This hopefully fixes #1293. Hard to tell though, as I was unable to reproduce the issue.

:+1:


Bug when using Component in iOS

When using the component in iOS, version 0.1.7 causes a strange behavior. If you reload a page with the component, the content cannot be scrolled down within "vue-pull-to"-component. You have to scroll up a little bit and then you can scroll down. In version 0.1.6 this behavior did not occur. Maybe you can fix this...

Try updating version 0.1.8

Thank you for your quick support. I tried the new Version, but it still works only at second scroll-attempt. 🤔

Am 17.07.2019 um 05:31 schrieb stackjie<EMAIL_ADDRESS>Try updating version 0.1.8
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

I broke the first scrolling situation and found that scroll events are not triggered under ios. When I changed transform to none, everything was fine.
Find related issues in Stack Overflow:
https://stackoverflow.com/questions/28288710/cant-scroll-inside-a-div-after-applying-webkit-transform-in-safari


when is the first release?

Hi, thanks for your interest, but we're honestly not in a position to accurately estimate that right now.

Maybe. I guess we'll see. Your guess is as good as mine at this point. :heart:

Thanks everyone for your concern.
Atom remains key to GitHub. Our product roadmap is set, and the team will continue all of their work.


Some links are broken

In http://bamos.github.io/latex-templates/, some links are broken like the "source" link https://github.com/bamos/latex-templates/tree/master/templates/cheatsheet

Thanks for the report @idf - I refactored this a while ago in https://github.com/bamos/latex-templates/commit/5a51b3db4c43b7dfba1469b8dc7c5d850bc0bcc4 and forgot to regenerate the static site. Fixed!
I also just added this and all of my other github pages to my nightly bad link checker.


add testIds for pincard and vc activation

@PoojaBabusing can we use testID as "warningMsg" for binding and removing? Other testID are common in both the overlay so that it will help QA to automate as single locator


Correctly serialize readonly properties

Json Schema does not specify "readonly" on definition properties
Swagger specifies it as "readOnly"
Newtonsoft.Json and NJsonSchema serialize IsReadOnly as "readonly"
Fix it by remapping "readonly" to "readOnly"

Maybe we should fix this in NJsonSchema?

https://github.com/NJsonSchema/NJsonSchema/commit/80ee9989c77839b3e8312f894320eff911adb8d7

http://json-schema.org/latest/json-schema-hypermedia.html#rfc.section.4.4

@rsuter then we should fix it in Newtonsoft.Json too?

https://github.com/JamesNK/Newtonsoft.Json/pull/1280
Thanks!

Yes, but im not the maintainer of this repo... :-)

Ok we could revert your pr as soon as njs is updated...


POMS campaigns refactoring

This PR includes a refactoring of the POMS configurations which now use the includes statement to avoid repeated settings.

@FNALbuild run build test

:hourglass: The following tests have been triggered for 23b1e12a468bb607613e7243d531bff83304d594: build  (Build queue has 1 jobs)

:sunny: The tests passed at 23b1e12a468bb607613e7243d531bff83304d594.

Test
Result
Details

merge
:white_check_mark:
Merged 23b1e12a468bb607613e7243d531bff83304d594 at 0f0a154cf94a9023f345bc145922c6c9f8eb17a7

build (prof)
:white_check_mark:
Log file. Build time: 14 min 24 sec

ceSimReco
:white_check_mark:
Log file.

g4test_03MT
:white_check_mark:
Log file.

transportOnly
:white_check_mark:
Log file.

POT
:white_check_mark:
Log file.

g4study
:white_check_mark:
Log file.

cosmicSimReco
:white_check_mark:
Log file.

cosmicOffSpill
:white_check_mark:
Log file.

ceSteps
:white_check_mark:
Log file.

ceDigi
:white_check_mark:
Log file.

muDauSteps
:white_check_mark:
Log file.

ceMix
:white_check_mark:
Log file.

rootOverlaps
:white_check_mark:
Log file.

g4surfaceCheck
:white_check_mark:
Log file.

FIXME, TODO
:white_check_mark:
TODO (0) FIXME (0) in 0 files

clang-tidy
:white_check_mark:
0 errors 0 warnings

N.B. These results were obtained from a build of this Pull Request at 23b1e12a468bb607613e7243d531bff83304d594 after being merged into the base branch at 0f0a154cf94a9023f345bc145922c6c9f8eb17a7.
For more information, please check the job page here.
Build artifacts are deleted after 5 days. If this is not desired, select Keep this build forever on the job page.


Pull request for snmp-mibs-downloader

For travis-ci/travis-ci#4345.
Ran tests and found no setuid bits.
See https://travis-ci.org/travis-ci/apt-whitelist-checker/builds/72054264

Sorry, but this test will have to be re-run. Will open a new one later.


Provide the trained models for various scenarios.

Is your feature request related to a problem? Please describe.
In the previous DeepRank repo we were providing the pre-trained models.
These are missing here. It would improve the usability of the code.

Is your feature request related to a problem? Please describe.
In the previous DeepRank repo we were providing the pre-trained models. These are missing here. It would improve the usability of the code.

Do you mean this folder? If yes, for providing the same pre-trained models we should re-train them, in order to save them again in a format which is compatible with DeepRank2 API.

Yes - and also the models of DeepRank-GNN. Those models are needed is users want to use it for scoring their own models as we describe in the original papers.

We are reviewing and cleaning the open issues on deeprank2, this one is a very nice to have, but we don't have the resources to pick it up at the moment. Just a heads up in case you want/can to assign this to someone in the lab @LilySnow


Telegram passport support

Hello,
I would love to ask if you are planning to add support for telegram passport since it has been released at version 4.0?
Thanks in advance.

Hey @odeadglaz!
Yes, we want to provide full support to the Telegram Bot API from the library, therefore supporting Telegram passport is in our plans. However, I can't say anything about when we'll work on it...
For the moment, I have created an issue to keep track of this feature request and to make it visible to any possible contributor interested in working on it -> https://github.com/kotlin-telegram-bot/kotlin-telegram-bot/issues/95
Thanks for you question!


090916 work

Do you have a summary of what is actually in this PR?  It looks like it is pulling in from an incredibly old set of changes and merging them.  It might be best to re-base or re-checkout your branch and then commit your two changes that you authored.  Right now, we can't pull this in as is.

Hey Shannon - it looks like this is trying to merge your changes into MASTER not into next-version, can you resubmit the PR to next-version instead.  Thanks!


Update vizviewer to use arbitrary port

Codecov Report

Merging #155 (67fb137) into master (73d7b9a) will decrease coverage by 2.27%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##            master     #155      +/-   ##
===========================================
- Coverage   100.00%   97.72%   -2.28%
===========================================
  Files           22       22
  Lines         2202     2198       -4
===========================================
- Hits          2202     2148      -54
- Misses           0       50      +50

Impacted Files
Coverage Δ

src/viztracer/viewer.py
100.00% <100.00%> (ø)

src/viztracer/patch.py
80.45% <0.00%> (-19.55%)
:arrow_down:

src/viztracer/report_builder.py
94.11% <0.00%> (-5.89%)
:arrow_down:

src/viztracer/viztracer.py
94.28% <0.00%> (-5.72%)
:arrow_down:

src/viztracer/simulator.py
96.74% <0.00%> (-3.26%)
:arrow_down:

src/viztracer/main.py
97.32% <0.00%> (-2.68%)
:arrow_down:

src/viztracer/code_monkey.py
98.51% <0.00%> (-1.49%)
:arrow_down:

src/viztracer/prog_snapshot.py
98.88% <0.00%> (-1.12%)
:arrow_down:

Continue to review full report at Codecov.

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


can't change feast assignment?

I am proofreading some chants but I can't seem to change the feast assignment to anything but "------"--and if it already is that, I can't change it at all! See below (this is https://cantusdatabase.org/edit-chants/672486?pk=673348&folio=012r)

re-opening with low priority--I don't think it's necessary to be backwards-compatible for every academic with a coal-burning computer, but if there is an easy way of doing so that would be nice. (I do like editing on my desktop, even if it is from 2013...)

Fair enough. I wonder whether django-autocomplete requires some feature that's only present in new browsers, and uses simple <select> elements as a fallback when that feature is missing.


Dev (#206)

Add cid-checker for hyperspace

Update buildspec environment

ga tag for hyperspace

wallwby helm removed

updated build version

be.increased HEIGHT_ALARM_ALLOW_DIFF to 160

Update buildspec.yaml

Increase allowed diff

add variable ONLY_COMPRESSED to values s3SortCaching

Feat/network switcher (#203)

network switcher

network switcher - small fix

Feat/network switcher (#203) (#204)

network switcher

network switcher - small fix

fe.define network from host

ci version 0.1.29

Lets close it - dont merge


Feature Request: Use ASP.NET Core services

In ASP.NET Core there are services for almost everything, e.g. detecting controllers and so on. I think it is better to use these services to find controllers and actions. This means that the documentation also shows all the special adjustments that the user made (e.g. controllers in another assembly, other prefixes, special actions, ...).
You can also get rid of this ugly config: typeof(Startup).GetTypeInfo().Assembly

This would be a complete rewrite of the Swagger generator

I know, but it would be compliant to asp.net core and there already options to show or hide a controller for example.
http://stackoverflow.com/questions/38368750/dotnetcore-is-apiexplorer-supported-and-how-to-use-it

Closed due to duplicate, see referenced epic.


145-Create ReadMe files for YOCO intergration-Tersia

Naming Rules
Name your PR like this: ISSUENUMBER-TITLE-YOURNAME
Description
Write a summary of your change. Explain why you have made it.
Related to
Make sure you include the issue number with a hash sign # so Github can link this PR to the right issue, like this:
Fixes #145
📸: Screenshots

Kindly attach screenshots if available
Checklist:

[x] My code follows the style guidelines of this project
[x] I have carefully reviewed my own code
[x] I have commented my code
[x] I have updated any documentation

You may want to remove the MIT License from the readme


Text cutoff at the bottom of the connect screen (chrome)

Describe the bug
"Block and ignore" is currently being cut off:

I don't have that?
Can you still scroll down?

There are no scrollbars but I could scroll the bottom message into the view.

I think scrollbars are hidden by CSS.
it fits in my screen, but maybe we can remove some spacing a bit to make it a bit smaller that it fits?


Swift runtime failure: Double value cannot be converted to UInt16 because the result would be greater than UInt16.max

This line: https://github.com/migueldeicaza/SwiftTerm/blob/main/Sources/SwiftTerm/iOS/iOSExtensions.swift#L15
causes a crash sometimes on iOS 14 with the error: Swift runtime failure: Double value cannot be converted to UInt16 because the result would be greater than UInt16.max
It seems to be a precision issue:
(lldb) p red
(CoreFoundation.CGFloat) $R3 = (native = 0.99990183115005493)
(lldb) p green
(CoreFoundation.CGFloat) $R4 = (native = 1.0000687837600708)
(lldb) p blue
(CoreFoundation.CGFloat) $R5 = (native = 0.99987989664077759)

Whoa, fascinating!  How did you get that green value 1.0000687?

Oh, I see, before iOS 10, the values could only be between 0.0 and 1.0, after iOS 10, if the values are within that range, they are in the sRGB gamut, otherwise, they are outside that range.
Given that these are RGB values for the terminal, I am going to clamp those.

Fixed in f1716ef7f5a6bf40e8c835052831d681bbb232b3

That makes sense, thanks.


fix: Add ability to process address complete in emails.

Copied from #862
This should allow Emails to process the Address Complete components output.
Fixes : https://github.com/cds-snc/platform-forms-client/issues/4419

I found a bug when testing your code. Once this is fixed (I will wait for you to confirm that my suggested fix works) we should be good to merge ;)

Good catch!~ Sorry about that. :)


Fix path typo in REST Protocol link

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Remove passing storage to unneeded places

This PR has too many things related to #18, trying to separate it progressively more confusing, so I desided to join them and test internally in tandem


Observables returned by the client when streaming may drop events

Currently, the implementation of IObservable for streaming fires events to all subscribed observers, but does not buffer or store any events for replaying. That means that there is a race between when the Invocation is sent to the server and when the Observer subscribes to the Observable. Any items streamed down before the invocation is sent may be lost. We may want to consider adding some replay/buffering or triggering the invocation on the subscription, rather than on the Stream call itself.

This idea sounds a lot like ring buffer from the old SignalR. I wonder if it is possible not to stream to users who subscribed after streaming started or potentially give the user the option between not streaming and streaming partial results. If you think about stock ticker why would it be useful to stream old results? Also, in this case streaming may never end so storing and replaying all this data is going to be costly while newly joined clients need only the latest data.

IMHO these should be separated into hot/cold streams similar to hot/cold observable in RX

also wondering if Bart De Smet's work on rx cloud streaming services may be useful 😎

@grahamehorner Yep, this is a traditional Hot/Cold Observable problem. I think that SignalR Hubs would always be Cold Observables, in that the Client is structured such that they receive every item sent by the server, without races. If the server wishes to allow Clients to subscribe to Hot Observables via a Hub method, that Hub method would return a new Cold Observable over that Hot Observable that closes over the desired stream. Trying to build a primitive to distinguish between the two gets complicated and is one of those things where we can't easily do it in a default way that any app would want, so it's up to the App to determine the lifetime of a Client's subscription by building an Observable that represents exactly what will be sent to that Client when they request a stream.
@moozzyk The buffering I'm talking about is on the Client, to ensure that any events received in between the Invocation being sent and the Subscribe method being called on the IObservable returned by the client can be replayed. I definitely don't want to add any server buffering.

@anurse - thanks for clarification - it now makes sense.

We're going to use ReadableChannel<T> for now, which should resolve this. Channel has an AsObservable which can produce an observable.


fix: Incorrectly-cached formatting

Every time I try to make a PR that changes packages/kit it tries to reformat the paths-base/svelte.config.js file. I figured this was some weird mis-cached formatting issue, so I ran turbo lint --force to bypass caching and see if there were any other mis-formatted things. Looks like there was one in adapter-vercel.
Note that running turbo check --force also fails, but I couldn't figure out what was causing it and I didn't have time to dig into it any further.
If this is some weird configuration issue on my end, feel free to just close the PR and forget about it.
Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

[ ] It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs
[x] This message body should clearly illustrate what problems it solves.
[ ] Ideally, include a test that fails without this PR but passes with it.

Tests

[x] Run the tests with pnpm test and lint the project with pnpm lint and pnpm check

Changesets

[x] If your PR makes a change that should be noted in one or more packages' changelogs, generate a changeset by running pnpm changeset and following the prompts. All changesets should be patch until SvelteKit 1.0

This PR prompted further questions that made me realize we have a broken lint step: it runs format before linting, so any issues that are fixed automatically by format go undetected in CI.
I pushed a change to drop the unwanted format from lint, but added an autolint script that keeps the old behavior since that's useful during dev.

I had to merge in master since, case in point, it introduced another missed lint error.


Firebase Messaging is not working with device_info_plus

When i try to use firebase_messaging, then it gives that error:
( So, because watkit depends on both device_info_plus ^3.2.2 and firebase_messaging ^8.0.0-dev.14, version solving failed.)
How to fix that? Waiting for answers. Thanks

 Because firebase_messaging >=8.0.0-dev.14 <9.0.0-1.0.nullsafety.0 depends on firebase_messaging_platform_interface ^1.0.0-dev.10 and no versions of firebase_messaging_platform_interface match >1.0.0-dev.10 <2.0.0, firebase_messaging >=8.0.0-dev.14 <9.0.0-1.0.nullsafety.0 requires firebase_messaging_platform_interface 1.0.0-dev.10.
And because firebase_messaging_platform_interface 1.0.0-dev.10 depends on plugin_platform_interface ^1.0.0, firebase_messaging >=8.0.0-dev.14 <9.0.0-1.0.nullsafety.0 requires plugin_platform_interface ^1.0.0.
And because device_info_plus >=3.2.0 depends on device_info_plus_platform_interface ^2.3.0 which depends on plugin_platform_interface ^2.0.0, firebase_messaging >=8.0.0-dev.14 <9.0.0-1.0.nullsafety.0 is incompatible with device_info_plus >=3.2.0.
So, because watkit depends on both device_info_plus ^3.2.2 and firebase_messaging ^8.0.0-dev.14, version solving failed.
pub get failed (1; So, because watkit depends on both device_info_plus ^3.2.2 and firebase_messaging ^8.0.0-dev.14, version solving failed.)

Issues with third-party packages need to be reported to the developers of those packages.
Although in this case, it looks like the problem is just that you are using an old version of firebase_messaging.


Add support for filtering posts by tag slug, bump NuGet version

This pull request adds support for filtering by tag slug to the ListPosts() and ListPostsAsync() methods. Also fixed the issue where lack of trailing slashes in the URLs were causing 301 redirects.


Ask a Question

Epic
As a user, I want to ask the database a question in natural language and receive a response in both natural language and data received so that I may understand the results, as well as copy the data, if necessary
User Stories

[x] #6
[x] #7

Finished in https://github.com/warrenwallis/cs130-project/pull/21.


Add an EventTable type that supports the Tables interface

Both Kaplan-Meier and Nelson-Aalen compute the same set of basic counts at each unique time prior to computing their respective quantities of interest. The counts have a notably table-like format, so much so that they can implement the Tables.jl interface with minimal effort.
As a follow-up, I'll refactor KaplanMeier and NelsonAalen to use EventTable under the hood and clean out all the stuff that's now redundant with the EventTable machinery.
Credit for the idea of integration with Tables.jl goes entirely to Tyler Beason, author of #25, who has been added as a co-author of this commit.

Codecov Report

Merging #46 (954a5a8) into main (c48d5c9) will decrease coverage by 2.18%.
The diff coverage is 91.56%.

@@            Coverage Diff             @@
##             main      #46      +/-   ##
==========================================
- Coverage   99.58%   97.39%   -2.19%
==========================================
  Files           6        6
  Lines         240      307      +67
==========================================
+ Hits          239      299      +60
- Misses          1        8       +7

Impacted Files
Coverage Δ

src/Survival.jl
100.00% <ø> (ø)

src/eventtimes.jl
92.63% <91.56%> (-7.37%)
:arrow_down:

src/cox.jl
99.21% <0.00%> (-0.07%)
:arrow_down:

src/estimator.jl
100.00% <0.00%> (ø)

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 c48d5c9...954a5a8. Read the comment docs.


GDNative: Make godot_int an int64_t

Redo of the change in cf8c679a23b21d6c6f29cba6a54eaa2eed88bf92
that caused a build issue, with the extra change needed to
'godot_arvr_blit'.
Thanks @Brettink for the fix to the bindings.
CC @godotengine/gdnative

@akien-mga the hero Godot needs.

No problem, happy to help!

@touilleMan Does this fix #34254?

seems fine to me

@akien-mga Yep this should do the trick ;-)
I think #35794 is still relevant (though I should review it a bit) given it documents the signature of multiples functions providing uint8/32 and float types


Incorrect highlighting for negation pseudo-class

IIRC this issue was introduced with the most recent updates.

Thanks!


feat: Add support for the iCON QCon Pro G2

Hey @bluesound7,
thanks for coming over to GitHub for this! This is still in a draft state, but I already finished the surface layout. Here's an incomplete list of things that are yet to be done:

[ ] Use only 6 display chars per channel, so there's some padding between the channels
[ ] Investigate the encoder assign button order – does it match the default Cubase MCU order?
[ ] Conditionally disable the color manager
[ ] Update the auto detection port names

I have attached the build output from commit 0fdf5e9 in this zip file. The folder structure needs to be icon/qcon-pro-g2/icon_qcon-pro-g2.js then...
With all the buttons properly aligned for your device, it might be easier for you to judge what's working and what isn't. Can you compile a list of things that don't work as expected? I'm running out of time for today, so I might ask you some more questions in the next couple of days when I get to this again :slightly_smiling_face:

@bjoluc that works great!
I'm wrapping up for the day and have a couple gigs this weekend and will be able to get to things in between also so that works great
So the next steps, download the zip file, load up and test, report whats working/not working - is there anything I'm missing?
Also, where are the instructions again on how to properly load the script? I just finished moving 2 weeks ago and the timing is perfect for collaborating on this project again, but I need to refresh my memory on loading script and other basics now that the main parts of the move have settled down :)

So the next steps, download the zip file, load up and test, report whats working/not working - is there anything I'm missing?

At the moment I think that's it, except that I only need to know what's not working :stuck_out_tongue_winking_eye:

Also, where are the instructions again on how to properly load the script?

I think the readme of this repository has the gist of it, though the directory names and file name need to be icon/qcon-pro-g2/icon_qcon-pro-g2.js for this version of the script now. Given that you managed to try out the original script some while ago, I'm confident you'll find what you need – proper (setup) documentation for various script flavors is another point on the list...
Cheers and have some fun gigs!

So I was able to plug this up and get some notes started :)
(attached is the image of the cubase overlay that comes with the qcon)
https://europe1.discourse-cdn.com/steinberg/original/3X/f/d/fde989807f89594b79b426ba0528c217de13842f.jpeg
NOTES:
-flip button: what is this supposed to do? (all faders go to zero when pressed)
-name/value doesn't do anything
-master fader: adjusts the control room red fader instead of the master fade
-I would like to control the master fader, however the
-vst button: it seems to reset the volume when pressed on a regular track (seems to make other tracks update on occasion when I press a second time)
-master button: seems to select the master fader on mixconsole 1
-write button: doesn't do anything
-left and right don't seem to work on anything
-solo button: seems to nudge the play head forward
-F1-F8 are these customizable and if so, how and are there any recommendations for uses of these? (f8 opens channel settings)
-LAYER 1: F8 opens the midi remote mapping assistant
-LAYER 2: F8 opens the channel settings (I would prefer this to be without a modifier button for faster workflow if possible/alternate the other f8, perhaps the focus quick controls on the layer 2)
-is there a way that the cubase mixer can scroll the right when using the track fader control to the right?
-lastly, extra credit:
this is a crazy question, I have an xr-18 and use the x air edit software, is it possible that a midi device could be created to control the software with the icon, i have mapped some simple steps with a loupedeck but its not user friendly enough to find myself actually using it.
-additionally: is there any other functionality that can be added on different pages that can customize the experience after the standard mcu items are implemented? seems the midi devices have options for more flexibility of uses

attached is the image of the cubase overlay that comes with the qcon

I'm a bit confused by the overlay: According to the product images (based on which I created the script surface), the only button that's labeled differently on the overlay is the Shift button ("Write", although there's another write button one row above). Am I missing anything?

-flip button: what is this supposed to do? (all faders go to zero when pressed)

Like in the default MCU mapping, it assigns the faders to the encoder parameters.

-name/value doesn't do anything

Does the corresponding button on the device surface in Cubase light up when you press the Name/Value button, i.e. is the MIDI mapping correct?

-master fader: adjusts the control room red fader instead of the master fade -I would like to control the master fader, however the

Working as intended as per the Readme (last point under "Miscellaneous"). Feel free to change it in the config.

-vst button: it seems to reset the volume when pressed on a regular track (seems to make other tracks update on occasion when I press a second time)
-master button: seems to select the master fader on mixconsole 1

VST and Master are mapped to MixConsole undo & redo like the Readme says.

-write button: doesn't do anything

Are we talking about the Write button next to VST and Master? I suspect it's mapped to the MCU "Shift" button. Do any other (shiftable) buttons light up while you press it?

-left and right don't seem to work on anything

Do they work in the default Cubase MCU setup?

-solo button: seems to nudge the play head forward

Whoops, seems I forgot to map the button. Still curious why it would do anything else instead?

-F1-F8 are these customizable and if so, how and are there any recommendations for uses of these? (f8 opens channel settings)
-LAYER 1: F8 opens the midi remote mapping assistant
-LAYER 2: F8 opens the channel settings (I would prefer this to be without a modifier button for faster workflow if possible/alternate the other f8, perhaps the focus quick controls on the layer 2)

You can customize/override all the F button mappings, as well as almost all other button mappings in the MIDI Remote Mapping Assistant – that's one of the great MR benefits: The device is yours to map! You can consider the script a mapping suggestion striving to be as close to the default as useful.

-is there a way that the cubase mixer can scroll the right when using the track fader control to the right?

Interesting idea. Is there a key command to do so? I might look into this as an optional (configurable) feature.

-lastly, extra credit: this is a crazy question, I have an xr-18 and use the x air edit software, is it possible that a midi device could be created to control the software with the icon, i have mapped some simple steps with a loupedeck but its not user friendly enough to find myself actually using it.

I'm not sure if I get the question: It's not related to this script, is it?

-additionally: is there any other functionality that can be added on different pages that can customize the experience after the standard mcu items are implemented? seems the midi devices have options for more flexibility of uses

Most certainly. There might be a lot to enhance with all MCU devices. The intention here is to use the same mapping across all MCU devices but make that mapping as useful as possible while staying as close to the default MCU mapping as reasonable. So if you have general MCU mapping ideas, feel free to voice them – ideally in a separate GitHub discussion or issue :slightly_smiling_face:

Hi @perivar,
I just noticed you created a fork a while ago dedicated to the QCon Pro G2! Seems that you put a lot of effort into that already, so I hope this PR doesn't render that void :man_facepalming: My goal with this project is to rename it to cubase-mcu-midiremote and support more devices in the long term. Guessing that you own a QCon Pro G2, would you like to test this and contribute here? No probs if you prefer to work on your fork instead, that's how OS works :slightly_smiling_face:
Cheers!

Hi,
Yes I'm happy to test the pull request.
Just a question: I tried to modify your code to avoid having so many
polyfills added since I thought it might affect the performance, as well as
making the final file so hard to read.

I have not verified the code I submitted into my fork just now, but will be
testing if the rollup build process still works. If so it avoids most of
the core-js polytfills.
I will get back to you when i have tested your pull request.
Best Regards,
Per Ivar
On Sun, Jun 4, 2023 at 10:48 PM bjoluc @.***> wrote:

Hi @perivar https://github.com/perivar,
I just noticed you created a fork a while ago dedicated to the QCon Pro
G2! Seems that you put a lot of effort into that already, so I hope this PR
doesn't render that void 🤦♂ My goal with this project is to rename it
to cubase-mcu-midiremote and support more devices in the long term.
Guessing that you own a QCon Pro G2, would you like to test this and
contribute here? No probs if you prefer to work on your fork instead,
that's how OS works 🙂
Cheers!
—
Reply to this email directly, view it on GitHub
https://github.com/bjoluc/cubase-xtouch-midiremote/pull/7#issuecomment-1575706880,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAHGCFFHYKQMWMTZVDEH2TTXJTYA5ANCNFSM6AAAAAAYYOQSVI
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hi. have now just tested the new build on the QCon G2.
Overall its amazing you have developed this. Some really smart mappings I like (Quick controls etc.).
The issues I notes were:

I could not make the build work on windows due to some issues with the regex code in tsup.config.ts
I have not checked in detail why it fails, but to make it work I replaced the scriptConfig code with;

let scriptConfig = "";
if (configFileContents) {
const scriptConfigMatches = /BEGIN JS\n([\s\S]+)/.exec(configFileContents);
if (scriptConfigMatches) {
console.debug(Succesfully parsed config file ...);
scriptConfig = scriptConfigMatches[1];
if (devicesConfiguration) {
scriptConfig = scriptConfig.replace('devices: ["main"]', devices: ${devicesConfiguration});
}
}
}
This made npm run build work, but the script did not work and was not detected in Cubase.

Since there is no CONFIGURATION section at the top, I needed to move the line var config = CONFIGURATION below the CONFIGURATION section

The script is looking for the wrong ports, to fix it I replaced
.expectInputNameEquals("X-Touch").expectOutputNameEquals("X-Touch");

with
.expectInputNameContains('iCON QCON Pro G2')
.expectOutputNameContains('iCON QCON Pro G2')
in order to support the fact that the firmware version number is often added to the port name.
After these changes the script started working.
However after around 10 minutes the script crashed and the remote device became unresponsive.
The only way to make it work again, was to restart Cubase.
Nothing in Cubase, except the connection to the QCon failed.
This does not happen when using the "standard MCU" connection, so it has something to do with the script.
Possibly the problem is that there will be a crash if the midi remote manager debug window is open?!
I will test further.
Thanks again - great work.

Another comment based on the comments from bluesound.
All buttons does in fact seem to work correctly, this includes Name/Value, Beats etc.
However, there does not seem to be any mapping for the buttons you can reach using Layer 2 on the QCon - only the EDIT button, which is reached using Layer2+F8.
So Layer2 and F1-F7 does not seem to exist in your code.
I believe these are called VIEW_* in the standard MCU mapping. I.e. Note 62 to 68. (69 is not used I believe, rather 51 which is EDIT)
Another weird thing, which I originally thought was your code, was that when clicking SHIFT only the LEFT button lights up, and no one else. It does seem this is a firmware issue, and I have contacted ICON support.
When I was checking this clicking all the buttons, the device got unresponsive again and crashed.
Still don't know what does that - could it be the timer?

I'm a Qcon Pro G2 user, and surprisingly the Behringer Xtouch script works like a charm on my surface. The only thing I miss compared to the Mackiecontrol is the adaptive jogwhell speed. When you scroll fast, it scrolls fast, when you scroll slowly it is more precise. In your script the jog is locked at the grid settings. Is there on option that could change that behaviour ? (sorry if I am on the wrong place to ask this)

Just a question: I tried to modify your code to avoid having so many polyfills added since I thought it might affect the performance, as well as making the final file so hard to read.

I definitely have, yes. Initially, I started without any polyfills, just plain ES5. During that process, I got reminded how crazy dumb ES5 is :joy: I became defiant very quickly and started using Babel, later switching to SWC via tsup. There are enough limitations in the current MIDI Remote API already, so at least I didn't want to cope with language limitations. The same thing holds for polyfills – I was just too annoyed by reinventing the wheel and I didn't care too much about script size. Luckily, I couldn't detect any speed differences with SWC and polyfills vs without them.
Re readability: I didn't expect anyone to read / work on the build output other than changing port names or some button mappings, which I think is pretty doable right now. Well, not pretty, but definitely doable :sweat_smile:

I have not verified the code I submitted into my fork just now, but will be testing if the rollup build process still works. If so it avoids most of the core-js polytfills.

If you manage to get your plain ES5 build to work without wasting too much time, I would be keen to know if there are notable performance differences between both versions. Otherwise, don't bother – life's short :sweat_smile:
Also, thanks for the feedback! I hope I'll be able to reply in a timely manner :upside_down_face:

Hey @perivar,
I just fixed (1) and (2), thanks for these!

However after around 10 minutes the script crashed and the remote device became unresponsive.
The only way to make it work again, was to restart Cubase.

I've heard that some times, yet could never reproduce it :grimacing: I've also seen other script authors complain about this in the Steinberg forum, so I tend to blame it on Cubase. If only we knew some reliable repro steps. Then again, we currently don't get any response even on simple, easily reproducible issues raised on the forum :confused:

In addition to the comments below found that since the LCD scripts only use 6 characters, it does not seem to ever clean all the 7 characters for the channels.

Good one, how could I not think about this :man_facepalming: :sweat_smile:

So Layer2 and F1-F7 does not seem to exist in your code.
I believe these are called VIEW_* in the standard MCU mapping. I.e. Note 62 to 68. (69 is not used I believe, rather 51 which is EDIT)

Oh, I didn't know Layer2 actually works with F1-F7, so I didn't create surface elements for the "virtual buttons". Thanks!

Another weird thing, which I originally thought was your code, was that when clicking SHIFT only the LEFT button lights up, and no one else. It does seem this is a firmware issue, and I have contacted ICON support.
When I was checking this clicking all the buttons, the device got unresponsive again and crashed.
Still don't know what does that - could it be the timer?

Oh no. No idea why this might happen, but would you mind commenting out all the buttons that should light up when pressing shift and uncomment them one at a time to see if receiving any or a combination of these midi notes causes the crash?

I think I addressed everything I could so far. Also, no need to build this yourself: The build GitHub action uploads the built scripts as an artifact for each commit, from where you can download them as a zip file (when you're logged in). For instance, here are the scripts for commit afbe9fd :+1:

Thanks for testing @perivar! I'm glad it works for you. Also, thanks for providing the repro test script for the shiftable button lights issue.
Just to make sure, does this

When I was checking this clicking all the buttons, the device got unresponsive again and crashed.

still happen – with either your test script or the real thing?
I noticed the Edit button can only be reached via Layer2, so maybe it's expected behavior that it doesn't light up? For the other buttons you list, I cannot think of any reason why they wouldn't light up. Let's hope this will be fixed in a firmware update.
My task list still has

Investigate the encoder assign button order – does it match the default Cubase MCU order?

I think the labels refer to a legacy Cubase MCU mapping and the Readme doesn't fit for them currently. But if you compare them to the original MCU (or X-Touch for what it's worth) encoder assign button names, do they appear to be ordered correctly? It's a bit annoying that the EQ button is labeled "Pan" on the surface and "Send" is labeled "Inserts", etc. but that's also a problem with the default Cubase MCU mapping, right? Not sure how to best handle this.

When I was checking this clicking all the buttons, the device got unresponsive again and crashed.

Did not experience this over many hours so I hope it’s fixed!

I noticed the Edit button can only be reached via Layer2, so maybe it's expected behavior that it doesn't light up? For the other buttons you list, I cannot think of any reason why they wouldn't light up. Let's hope this will be fixed in a firmware update.

I’m confident this is a firmware issue.

Investigate the encoder assign button order – does it match the default Cubase MCU order?

As far as I can see the mapping works the same and works fine.
Well done !

Thanks @perivar! I'm going to merge this now so we can get a release out soon :rocket:
@bluesound7 If you have some time to try this out, feel free to respond here anyway or open discussions / issues for individual problems. Cheers!


Mismatch in output columns

Hi,
I am seeing an issue with the final output of the temperature_score.calculate() step, where the actual timeframe length for a counterparty's target is not aligning correctly with the value in the time_frame column. The example I have supplied shows that we have a 7-year timeframe for a companies S1S2 target, which according to the methodology should be classed as a mid-term timeframe, but it is being allocated under short term in the output.
The inputs we are supplying to the model should be correct and have all the fields required to run the tool. I believe the issue is occurring around the prepare_targets() step.
Is this a known issue and if so, is it in the pipeline to be rectified?
sbti_temperature_tool.xlsx

Hi @mountainrambler
Thanks for your response.
I had to create a  mock-up of the data as the actual data is confidential, and must have made typo with the reduciton_ambition column - apologies!
I believe I have identified what was going on with our data, however it has raised another couple of questions.
In short, the cause of the mismatch I was seeing with the timeframe value allocation was down to how these are set in the model. I was under the impression that these values (SHORT, MID, LONG) were allocated by taking the difference between the base year and target year for a counterparty, whereas the tool takes the current calendar year the model is being run in and subtracts this from the target year instead.

This meant that as a result of the difference between target year and current year was <4, we were being returned SHORT when we were expecting MID as the overall target length is more than 4.
However, correct me if I am mistaken but the docstring here seems to contradict what is in the methodology, where is states:

"Scores will be based on the ambition over the timeframe of the target (base year to target year),
and not just on the forward-looking portion (current year to target year)."

Is this being accounted for despite the current year being used to subtract from the target year, as we would expect a counterparty with 7 year target timeframe to be allocated MID?
We have data from 2021 e.g. that we are planning to run through the model, making 2021 our 'current' year rather than 2023, so this is just making us wonder if we are using the tool as expected and if this will be providing us the correct temperature scores for our counterparties.
Any advice on this would be appreciated!

Hi,
Yes the score is calculated using target year - base year which you can see in temperature_score.py on lines 262&263:
return target[self.c.COLS.REDUCTION_AMBITION] / float(             target[self.c.COLS.END_YEAR] - target[self.c.COLS.BASE_YEAR]
However, this is different from the time frame where the score is placed. The time frame always considers the time that is left of the target validity, so if there are only three years left to the end date, the target will be short term, although the score is calculated based on the ambition of the orignal time span of the target.
Hope that helps.
/Peter


No HTML generated

The images and mipmaps for the world (and night mode) are rendering, as well as the JSON, but there is no HTML file being generated to actually view the world and render the zoomable map.
Mode                 LastWriteTime         Length Name
----                 -------------         ------ ----
d-----        2024-02-13   6:54 PM                light
d-----        2024-02-13   6:54 PM                map
d-----        2024-02-13   6:53 PM                processed
-a----        2024-02-13   6:54 PM             12 entities.json
-a----        2024-02-13   6:54 PM           9689 info.json

This is using MinedMap-2.1.0-i686-pc-windows-msvc on Windows 11, exporting a 1.20.1 map (with Create as the only functional mod installed that affects worldgen)

The HTML is static, it can be found in a separate "viewer" download on the Github release page. The generated files need to be put in the data subdirectory of the unpacked viewer.
A mode that also copies the HTML into the right place with the other files could be added as part of https://github.com/neocturne/MinedMap/issues/38 (as we will need to include the HTML files in the generator for that anyways).

Thanks, I figured that out an hour or two later. I do understand why the platform invariant files are packaged separately... but your instructions could be much clearer. I'll post a pull request with my proposal for clearer steps in the README.md.


Automated smoketest failed in "Debugging JavaScript autodetects program attribute for launch.json"

After merging PR #32050 the automated smoke test failed with this error:
2017-08-07T07:19:07.5367300Z   1) Smoke Test Suite Debugging JavaScript autodetects program attribute for launch.json:
2017-08-07T07:19:07.5394730Z      SyntaxError: Unexpected token } in JSON at position 385
2017-08-07T07:19:07.5419360Z       at JSON.parse (<anonymous>)
2017-08-07T07:19:07.5439110Z       at JavaScriptDebug.<anonymous> (out/areas/javascript-debug.js:41:31)
2017-08-07T07:19:07.5454090Z       at Generator.next (<anonymous>)
2017-08-07T07:19:07.5467850Z       at fulfilled (out/areas/javascript-debug.js:8:58)
2017-08-07T07:19:07.5481780Z       at process._tickDomainCallback (internal/process/next_tick.js:135:7)

Further investigation showed that commit #a9ebfd8814d71e4e92af776dcbf133535b3e81e4 contained the culprit in debugConfigurationManager.ts:

After the user selects a debug environment, the content of the launch.json is not passed as an IContent but just as a string. As a consequence the 'value' attribute is undefined and the indexOf doesn't exist.

@isidorn I've added a workaround. Please review and/or replace by a proper fix.

Looks like a proper fix to me, thanks


Aftman error: Incorrect function. (os error 1)

Trying to follow instuctions on readme.
When I try to use aftman to add a tool I get error messages like so:
$ aftman.exe add rojo-rbx/rojo
Tool rojo-rbx/rojo has never been installed before. Install it? [y/n] Aftman error: Incorrect function. (os error 1)
$ aftman install
Tool rojo-rbx/rojo has never been installed before. Install it? [y/n] Aftman error: Incorrect function. (os error 1)
dougb@DougPC MINGW64 ~
On Windows using GitBash shell.

Super strange, I've never heard of an "Incorrect function" OS error! Something to look into.

Seems likely related to https://github.com/console-rs/console/issues/35.

had the same problem running it through git bash. powershell worked fine though.


Add badges

This PR will add the following badges:

View rendered README.md

Because it's just a README update, I'll merge directly when the checks turn green.


Change format of ChatBot parameters

Currently, all of the parameters passed to a chat bot are also passed to each of the chat bot's adapters regardless of whether or not that adapter uses that parameter. This setup also makes it a bit unclear as to which adapter an extra parameter is used for. As ChatterBot grows, there might also be future issues with parameter name conflicts between multiple adapters that expect the same argument.
Proposed parameter change:
ChatBot(
    storage_adapter={
        'class_name': 'my.storage.AdapterClass',
        'database_name': 'my_database'
    },
    logic_adapters=[
       {
           'adapter_class': '.some.adapter.ClassName',
           'api_key': 'my_api_key_string'
        },
        {
           'adapter_class': '.some.adapter.ClassName',
           'api_key': 'my_api_key_string'
       }
    ]
)

@gunthercox  please have a look at PR, let me know your feedback.


Fixed link cost calculation

Fix for link cost calculation into TopologyInstance for the metric "utilization".
It was an erroneous copy-paste from the link-speed case.

@MicheleIessi thanks for fixing this!


podcast:chapter Start Time Type Update

At @tomrossi7 's recommendation I'm proposing a change from using the lesser known Normal Play Time type to use seconds since the start of podcast instead represented in integers. This is a simpler way of representing a time within a podcast and will be simpler for clients to adopt.
This also aligns with the type used by the itunes tag: <itunes:duration>

Ugh. I just realized we have had some issues with people needing to use a fraction of a second. You may want to let the host provide a number as either int or float. The aggregator could do with it what they want?

Yes, let's make sure the spec aligns with what's happening in the real world already.  Good catch.


No "Open file" dialog pops up when executing "Download Remote Settings..." command from command palette

OS: All
Azure Resource Extension Build Version: 20220421.32 (unified)
Azure Function Extension build Version: 20220421.8 (unified)
Regression: Previous Build
Repro Steps:

Create a Function App.
F1 -> Azure Functions: Download Remote Settings... -> Select a resource group -> Select a Function App.
Check whether the "Open file" dialog pops up or not.

Expect:
The "Open file" dialog pops up.

Actual:
No "Open file" dialog pops up, it will navigate to "Select App Setting" option.

More Info:

This issue also reproduces for "Azure Functions: Upload Local Settings..." command.
This issue does not reproduce on the previous Azure Function unified build 20220421.1.

Fixed by #3141


Why exactly one alias?

Why does the method PassSigningUtil.firstAlias require exactly one alias? I am trying to generate p12 files, but they are always generated with two aliases.

I had the same problem. In my case It worked only specifying as alias the name of the key inside the .p12 certificate.

There are several options for specifying the alias:
alias(String) - if the p12 file contains more than one alias and the privateKey and signingCertificate aliases are the same
signingCertificateAlias(String) - specify the signing certificate alias
privateKeyAlias(String) - specify the private key alias


[AWS] Add pod default object to inject kubeflow user IAM role

We want to have generic IAM role for user to connect to some AWS services like ECR and S3.
Instead of using credentials, we can service account with IAM policies to achieve this.
This story depends on https://github.com/kubeflow/kubeflow/issues/4403

PR https://github.com/kubeflow/kfctl/pull/225 is merged and we can close issue in kfctl project.


[v1.9] daemon: store Cilium's configuration in a file

#16017 -- daemon: store Cilium's configuration in a file (@aanm)

Once this PR is merged, you can update the PR labels via:
$ for pr in 16017; do contrib/backporting/set-labels.py $pr done 1.9; done

test-backport-1.9

all failures are know flakes?


fix(WebSocketSubject): prevent early close

Description:
This prevents the WebSocketSubject from prematurely closing the underlying websocket before all subscriptions have completed.
Related issue (if exists):
ReactiveX/rxjs#1827

Coverage remained the same at 96.472% when pulling b884ba00ed8b70004170cf9b1a351f5d7bbfbe55 on deontologician:websocket_keep_open into 4f65b03ce6908a934602d619993fa31ec49a2cbf on ReactiveX:master.

I think additional test case can be added for this behavior changes. (check stub socket.close being called only if condition mets)

Alright, test added. I've confirmed it fails on master and succeeds on this branch (on master the second B next is not received)

Coverage remained the same at 96.472% when pulling 543131c5e3dc90a2bee96c945d73a616800ae305 on deontologician:websocket_keep_open into 4f65b03ce6908a934602d619993fa31ec49a2cbf on ReactiveX:master.

Has anyone had a chance to check this out?

Coverage remained the same at 96.71% when pulling a77b7074d2844a302e3f38c3aef9446d4bf90102 on deontologician:websocket_keep_open into 8f0dc014a8cc9b7a639be7ba69396d07eee37bae on ReactiveX:master.

For me this change looks ok, need other reviewer's approval as well.

LGTM

Thanks!


LibWeb: Remove associated transitions when clearing transitions

Fixes crash on https://nextjs.org
Fixes #1481

After #1442, auth0.com also broke but this change fixes it as well.

This PR has also fixed a crash on github


[WFU] Fix for artist name titles and status dot

Fixes https://github.com/artsy/eigen/issues/2303 and https://github.com/artsy/eigen/issues/2306

Fails

:no_entry_sign:
No CHANGELOG added.

Warnings

:warning:
Could not find corresponding stories for these components:
 - [] `export class Notification extends React.Component`

  Generated by :no_entry_sign: dangerJS

LGTM!


[Question] Failed to create a graph when using MySQL storage for hugegraph

Problem Type (问题类型)
configs (配置项 / 文档相关)
Before submit

[X] 我已经确认现有的 Issues 与 FAQ 中没有相同 / 重复问题 (I have confirmed and searched that there are no similar problems in the historical issue and documents)

Environment (环境信息)

Server Version: 1.0.0 (Apache Release Version)
Backend: mysql
OS: xx CPUs, xx G RAM, Ubuntu 2x.x / CentOS 7.x
Data Size:  xx vertices, xx edges

Your Question (问题描述)
我想配置多图，然后参考官网（https://hugegraph.apache.org/cn/docs/config/config-guide/#5-多图配置）     但是init时一直出错。
详细更改的配置如下：
gremlin-server.yaml

rest-server.properties

新增的图配置文件

mysql.properties改动内容：

问题：
1、init时，提示graphs这个选项应该是一个文件夹，但是官网指导是一个map结构

2、当我把graphs换成目录时（/etc/hugegraph/conf/graphs），又提示其他异常

Vertex/Edge example (问题点 / 边数据举例)
No response
Schema [VertexLabel, EdgeLabel, IndexLabel] (元数据结构)
No response

Thanks for the feedback, the server config doc is outdated & misunderstand users,
refer here, and @chakkk309 do u have the latest doc?

Hi, @LiJie20190102, Your modification is correct, the reason for the problem is because there is no mysql library, you can refer to #2175 and #2196
CC @imbajin

Hi, @LiJie20190102, Your modification is correct, the reason for the problem is because there is no mysql library, you can refer to #2175 and #2196
CC @imbajin

mysql doc also need update（and need update the config template）
meanwhile，the multi-graph doc is also outdated😢

Hi, @LiJie20190102, Your modification is correct, the reason for the problem is because there is no mysql library, you can refer to #2175 and #2196
CC @imbajin

mysql doc do need update for jar dependency（and need update the config template）
meanwhile, the multi-graph doc is also outdated/wrong 😌

i can help do this.


Add cache cleanup ability

I was thinking that if this proves difficult, I could simply do a 'touch'
to the cache files as used, which would then show up with mtime.
On Thu, Jun 1, 2023, 1:49 PM Jonas Eberle @.***> wrote:

@.**** commented on this pull request.
In autoortho/aoconfig.py
https://github.com/kubilus1/autoortho/pull/182#discussion_r1213490370:

@@ -45,6 +45,43 @@ def eq(self, other):
return NotImplemented

+def clean_cache(cache_dir, size_gb):
+

log.info(f"Cleaning up cache_dir {cache_dir}")

target_gb = max(size_gb, 10)
target_bytes = pow(2,30) * target_gb

cfiles = sorted(pathlib.Path(cache_dir).glob('**/*'), key=os.path.getmtime)

I assumed it from what I am used with Linux stat on my noatime file
systems...
You are right - we should check the other platforms. I've dug a bit into
the python source an and I think the syscalls are here
https://github.com/python/cpython/blob/main/Python/fileutils.c#L1233-L1284:
so fstat() on Unix / GetFileInformationByHandle on Windows. But I can not
find an assertion that atime is always >= mtime, it's totally up to the
file system. So my assumption was wrong. Maybe this?
⬇ Suggested change

cfiles = sorted(pathlib.Path(cache_dir).glob('**/*'), key=os.path.getmtime)

cfiles = sorted(pathlib.Path(cache_dir).glob('**/*'), key = lambda filePath: max(os.path.getatime(filePath), os.path.getmtime(filePath)))

(Python syntax is not my home turf so I might be missing a nicer way)
—
Reply to this email directly, view it on GitHub
https://github.com/kubilus1/autoortho/pull/182#discussion_r1213490370,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABVEDLGSRN5AUBXWA5G3GLDXJDI4PANCNFSM6AAAAAAYWCQP6Y
.
You are receiving this because you authored the thread.Message ID:
@.***>


[HOLD for payment 2024-01-17] [$500] Web - IOU & Task - User is redirected to previous page when double clicking outside RHP editor

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.15-4
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:

Go to request details view of IOU/task report
Click on any field to open RHP
Double click outside RHP

Expected Result:
User stays on the same page
Actual Result:
User is returned to previous page
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/f3b17a68-aa2b-4920-b5d4-5aa8f3076c91
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01b870d525be430c1d
Upwork Job ID:<PHONE_NUMBER>358630912
Last Price Increase: 2023-12-21
Automatic offers:

tienifr | Contributor | 28070388

@thesahindia plz review the proposals above.

Sounds good! @tienifr you are assigned, please feel free to create a PR for us to review 🙂

@thesahindia can you please provide details on why @tienifr 's proposal was selected above others that were posted before? Thx
@blimpich can you please review the PR?

@thesahindia can you please address my question above?
https://github.com/Expensify/App/issues/33460#issuecomment-1874734025
@thesahindia can you also plz complete the regression test steps?
Contributor: @tienifr paid $500 via Upwork
Contributor+: @thesahindia owed $500 via NewDot

Thanks @thesahindia !

$500 approved for @thesahindia