[Question] can i adopt the setup if I were to apply it to a jupyter notebook?

I saw this setup in the original blogpost.
There's a public jupyter notebook that I am helping the maintainer bring to life as an interactive dashboard. Met him via Chip Huyen's MLOps discord. Same discord where I learn the terms: ibis and dagster.
https://github.com/dgilmore33/NBA-Dashboard
I am thinking of applying your setup (not all of it, just the essential parts) to the jupyter notebook.
I did notice that you mentioned the use of notebook in EDA

I can add from eda import * in a untitled.ipynb for a classic notebook experience in VSCode, which I prefer when plotting or just saving my work.

in the post.
Is it sensible to apply your setup to a jupyter notebook that needs to become an interactive dashboard? Or it's more of a round peg to square hole?
ChatGPT 4 response
This is what I got when I posed the same question to ChatGPT 4 including the link to the post.

Yes, you can apply the setup from the Ibis Analytics project to create an interactive dashboard using Jupyter Notebook. Here’s a general approach to achieve this:

Develop Your Analytics Pipeline: Use Ibis for data manipulation and analytics within your Jupyter Notebook, ensuring you can interactively develop and test your data transformations and visualizations.
Set Up Streamlit: Streamlit is used for deploying the dashboard. You will need to adapt your Jupyter Notebook code to work within a Streamlit app. This involves organizing your code into functions that Streamlit can use to render the interactive components.
Deploy the Streamlit App: Once your Streamlit app is ready, you can deploy it using Streamlit Community Cloud. This involves setting up your app on GitHub and then deploying it from there with a few clicks.
Integration and Testing: Ensure that the data flow from Ibis to Streamlit is working correctly, and your interactive components function as expected in the Streamlit environment.

This process involves taking your data manipulation and visualization logic from a Jupyter Notebook and adapting it to a Streamlit app, which can then be deployed as an interactive dashboard.

Let me know if there's any obvious misunderstanding from my question, or any obvious flaws from ChatGPT's response.

Is it sensible to apply your setup to a jupyter notebook that needs to become an interactive dashboard? Or it's more of a round peg to square hole?

I'd say it's sensible! I wouldn't necessarily recommend putting all of the code -- ingestion, ETL with Dagster, and postprocessing -- in the DAG, but once you have the data transformed and loaded into whatever database or table format you want it in, using a notebook for the dashboard/visualization piece makes a lot of sense. there are numerous open source notebook -> dashboard app frameworks, or tools like Hex, that make this pretty easy
you could also put the Dagster logic in a notebook, but personally I'd prefer to keep that in Python scripts. regardless, just importing whatever you need in your notebook and running it is fine. as mentioned, when I'm experimenting I'll open up a notebook, run from eda import * that gives me the con to the database with all the tables + sets up plotly, and go from there

sure thing! feel free to ask questions here or in the Ibis project Zulip (chat)

sure thing! feel free to ask questions here or in the Ibis project Zulip (chat)

i joined. Let me know if ok i post my progress here and there

feel free to post in either or both!

Crossposting from https://ibis-project.zulipchat.com/#narrow/stream/405263-general/topic/use.20ibis-analytics.20to.20convert.20jupyter.20notebook

In the learning project, I'm doing with a finance analyst, i easily converted the jupyter notebook to be streamlit compatibel script.
i had to swap out the flask/dash related commands. Apparently, not compatible with streamlit but that's easeily done with the help of chat GPT 4. haha :happy:
First time using streamlit and i think it's awesome.
While waiting for feedback from Danny, the finance analyst, I will switch back to replicating Cody's blogpost.
Exciting times!
Here's the latest streamlit-ready python script from the learning project: NBA-Dashboard/NBA.py at main · simkimsia/NBA-Dashboard (github.com)


Question: how should static paths be generated?

Hi- thanks for open-sourcing this!!
As this example repo does not leverage getStaticPaths, I'm left wondering how one could/should configure locale paths to be included in a static export? Is this yet possible with next-export-i18n for projects using dynamic routes & getStaticPaths?
Said differently, the need is to be able to use locales w/in getStaticPaths. However, when doing so the following error occurs (running npx next build):
Collecting page data  ...Error: Invalid locale returned from getStaticPaths for /[[...slug]], the locale en-US is not specified in next.config.js

next.config.js:
const nextConfig = {
  output: 'export',
  // i18n: { locales, defaultLocale },
  reactStrictMode: true,
  ...

Given in the next.config.js that i18n cannot be specified when output: 'export', how can locales be passed in to getStaticPaths?
getStaticPaths from ./src/pages/[[...slug]].js:
export const getStaticPaths = async ({ locales = i18n.locales }) => {
  const paths = [];
  await Promise.all(locales?.map(async (locale) => {
    const entries = await fetchNestedFromCache({
      locale,
      contentTypeId: "pageGeneral"
    });
    entries.forEach((entry) => {
      const as = (entry['@as']?.[0] === '/') ? entry['@as'].slice(1) : entry['@as'];
      const slug = (entry.slug === DEFAULT_HOMEPAGE_SLUG) ? [''] : (as || '').split('/');
      if (slug !== 'undefined' && entry.siteTarget.includes("Org")) {
        paths.push({ params: { slug }, locale });
      };
    });
  }));

  return {
    paths,
    fallback: false
  };
};

Thanks for reading! Cheers

Hi @christopher-theagen,
Thank you for your contribution. If I understood your question correctly, I don't think it is possible.
_

Note that Internationalized Routing does not integrate with output: 'export' as it does not leverage the Next.js routing layer. Hybrid Next.js applications that do not use output: 'export' are fully supported.
_
https://nextjs.org/docs/pages/building-your-application/routing/internationalization

The https://github.com/martinkr/next-export-i18n/ provides a purely client-side solution to workoutound the shortcoming of next.js' i18n combined with output:export.
Cheers,
Martin


Fix few emojis with missing/out-of-order codepoints

Coverage remained the same at 93.125% when pulling b6a3ae011dff5e967f00fc3f827230184a979cd9 on BillyGalbreath:master into 18335af05118b826297b6eb6ce066e97bf6c0c46 on vdurmont:master.

Thanks!


Grafana behind reverse proxy

I am trying to install grafana using the operator, it gets installed, but as I am behind a reverse proxy the UI loads but the links doesn't work.
According to grafana documentation I should use root_url but I don't see a way to change it using the operator.
Thanks

root_url and domain are currently not in the list of options that can be passed to the ini file via the Grafana CR. But it makes sense to add them. I'll try to do this as soon as possible.
As a workaround, once Grafana is deployed, you could manually edit the configmap grafana-config. It's contents are mounted as grafana.ini and you could add the root_url here.

Thanks for the workaraund.

@marcecaro sorry for the long wait, but the 1.2.0 release (https://github.com/integr8ly/grafana-operator/releases/tag/v1.2.0) supports the whole range of Grafana configuration and should allow you to configure a reverse proxy.


Re-run chooser tests if they fail

Sometimes the chooser tests fail spuriously. We should re-run them a few times until they pass.

Is this a chooser test? https://github.com/casey/just/blob/50cd24d37ba696dd74f3b05f1c23b934ef049b10/src/justfile.rs#L429-L450
I've had it fail randomly a couple times with "Text file busy"

@06kellyjac That's interesting, that's not a chooser test. Is that the only test you've seen fail with that error?

Yeah and I couldn't get to fail consistently. It failed when building the latest update using nix with this patching (the original sed on the left): https://github.com/NixOS/nixpkgs/pull/125653/files#diff-85969503e7886ff0bcdf36e5c35be3b9fa6cd334be07e99d9948a4643bcbe90eL38-L43
We patch these shebangs because env doesn't have access to anything in the sandbox.
I thought there might be an issue with the patching of #!/usr/bin/env sh to use bash, which could be used at the same time as another test that uses bash but I don't see why there'd be a "Text file busy" error for two processes trying to run bash :shrug:
I don't have the output of the issue in my terminal anymore so I'll try recreate the error
I'll also move this discussion to a new issue to avoid filling this one with non-chooser test stuff

I haven't seen an error for these in a while, so I think this is fixed somehow 🤷♂


feat: adding context value acrossing input methods

Here is an example from BugBuster:
https://optimism.sunodo.app/0x9cb6c6e904ce6bf3ca6d0002b9629acce74ea89b/graphql :////graphql

deprecated


sweat & tears: Using volumes

using -d is not explained and mentioned
Run the http image -> Run the httpd image
"I did it" is no where mentioned

Fixed!


트리거 매니져 구성

트리거 인덱스 및 정보를 가자고 있는(코드 상에서) 트리거 매니저 필요?

Is this an issue or a question about GridDB Python Client ?
It seems you open some issues in griddb/python_client repository,
but I quess these are other topics

No, I was opened issue wrong page. Sorry.


Initialize a ByteBuffer from a ByteBufferView

Motivation:
As @weissi previously said in (#946), there's no reason not be able to get a ByteBuffer from a ByteBufferView.
Modifications:
I added a public initializer to ByteBuffer taking a ByteBufferView to create a slice of the view's buffer from the same range.
Result:
ByteBuffer will have a public initializer to create a ByteBuffer from a ByteBufferView.
Closes #946

@swift-nio-bot test this please


fix: #874 frontend improvement no try catch

Fix/Remove unnecessary try-catch in components.
Organize some import using better reference.
Create LoadingState store for central loading.
Trigger "startLoading", "stopLoading" and "stopLoadingOnError" at both new request/response HttpInterceptors.
Add loadingLabel at Button component and implement feature for Button to take care of label change when loading.

Looks good to me, thanks!!


Publish MHDS in html

This publication is already converted into markdown. Seems this should be published in html.
https://github.com/IHE/IT-Infrastructure/tree/master/MHDS
Note this does not include conversion to IG publication, but could be done with IG publisher.

https://github.com/IHE/ITI.MHDS

upgraded to IG


Remove duplicate GC routes on summarization

Resolves #9472
We've discovered multiple duplicate routes in our GC outbound routes in our GC blob. This PR's goal is to prevent that from happening and will fix the bug on summarization.
How to replicate the issue:

Have a datastore with 2+ DDSs.
Run GC's collectGarbage during summarization, which the parent datastore adds GC routes from each DDS to itself during the getGCData call.
The summary is submitted to the server.
Reload a new container, which has these newly added routes. This is then passed down from the container runtime to the datastore back down to the child datastores' summarizerNodeWithGC as this.gcData.
Make a change to one of the DDSs but not another.
Re-run GC's `collectGarbage' during summarization
The DDS which did not change returns getGCData with the data passed down from the snapshot containing an extra route.
The parent datastore adds the GC route from that DDS to itself again, which is a duplicate route.
The summary is submitted to the server.

Solution:

When a DDS's gcData is passed back down to it, remove the implicitly added route to its parent datastore.

This relies on Snapshot tests being setup #10015

Any chance you can describe in plain English what is the problem, and how big is it? I glanced through changes, but it's hard for me to get a gist of it quickly :)
I.e., does it get worse over time? How many duplicate entries we get on average? How big impact do you think it will have in terms of size?

Any chance you can describe in plain English what is the problem, and how big is it? I glanced through changes, but it's hard for me to get a gist of it quickly :) I.e., does it get worse over time? How many duplicate entries we get on average? How big impact do you think it will have in terms of size?

This means the GC blob will get bigger overtime. We don't have data yet on the size of GC blobs until 0.59.1000 is integrated. This issue affects all components/datastores with multiple DDSs.
I've updated the comment to include more details. I'll have to get back to you on duplicate entries on average and impact in terms of size.

Do we need to do anything to "reset" existing files?

Will re-summarization of data store fix that data store representation?
Do we need to reset GC version to force full summary (or use any other mechanism to recalc this data)?

Do we need to do anything to "reset" existing files?

Will re-summarization of data store fix that data store representation?
Do we need to reset GC version to force full summary (or use any other mechanism to recalc this data)?

To reset existing files, we need to re-summarize all DDSs with this issue.
I just realized that my current solution does not fix old summaries as it only deletes one route. We can implement a solution that deletes all routes as soon as summarization occurs. There is one edge case we have to consider - if a DDS explicitly adds a route to its parent, then the gcData we would be passing down to the datastore would not be important. The route would automatically be added back. In the future if we make DDSs non-root this will likely need to be reconsidered/remembered.

if a DDS explicitly adds a route to its parent

DDS should always have a link back to data store - if DDS is alive (because someone holds it), then Data store should be alive.
Root DDS means Data store has a reference to it - DDS would not be collected if data store is alive.
Non-root means Data store does not keep it alive - if there are no other references, it should be collected (whenever / if we implement GC for DDSs).

The simplest solution is to use a set instead of an array while processing outbound routes in either direction - sending data up from DDS to container runtime or sending it down from container runtime to DDS. Then explicitly remove the link from the DDS to the data store. This will ensure that there are never any duplicate routes in the GC data.

if a DDS explicitly adds a route to its parent, then the gcData we would be passing down to the datastore would not be correct.

From GC's perspective, it doesn't matter whether a route from DDS to its parent data store was explicitly added by the DDS or implicit added to keep the data store alive. Another thing to consider is if a DDS has multiple handles to the same object, it will have a lot of duplicate outbound routes. GC should remove these duplicates as well before it processes them and stores them in the summary.

In the future if we make DDSs non-root this will likely need to be reconsidered/remembered.

I am not sure I understand this. DDSs being root or non-root has nothing to do with the route back to the data store. That is about having a route from the data store to its child DDSs.

I'd only say - we need to stop the bleeding and fix existing files. If this can be done in one go - fine. If not - consider targeted fix to stop the bleeding (and possibly porting / producing hotfixes).
This is also an area where we need to rethink validation. Ideally, we should find them sooner and via some automated ways. We should consider multiple directions - better telemetry (upfront), better UTs (we should have UTs that keep summarizing 100 times with various changes or not to container and observe metrics), etc.

Given the size of the average GC blob ~ 29 within the last day and an estimation of about 29 routes per blob and an estimation of 76 bytes per route, which is the length of 2 full gc id routes with "/" delimiters, the average expected GC blob size is estimated to be 63916 bytes or ~64 KB.

Looks reasonable to me, but Navin should be the one signing off :)
Is this enough to fix everything, even if some number of data stores would never summarize in a given file (user would not change them)?

Looks reasonable to me, but Navin should be the one signing off :) Is this enough to fix everything, even if some number of data stores would never summarize in a given file (user would not change them)?

Yes, because the GCDataBuilder is used during the getGCData call at the container runtime level, so at that point, any duplicate routes would have been removed - including for duplicate routes we do not already know about.

I believe so. You can validate this with a test to confirm. Also, the snapshot tests which you are working on will also validate this.

I don't think you can update the test snapshots in this PR because it fixes the bug. Maybe you can if you update the snapshots in your local repo with this change. This will add a "src_snapshots" folder (which has duplicate routes) to the snapshots added by https://github.com/microsoft/FluidFrameworkTestData/pull/41 and update the "current_snapshots" to not have the duplicate routes.

Also, please make sure that the snapshot repo changes are merged and you update this PR to point to a commit in the main branch of the snapshot repo.


Fix generation for coreutils

Ignore read json error
Set default timeout for run tests 30

@ladisgin should it be '30' in the settings too? still see there '0' in 2022.10.4


Stuck in "Reconnecting" Loop

I noticed that YakYak was stuck a loop of "Not Connected" and "Reconnecting". I cleared out my login cookie and logged in again, but no dice. If it helps, I am on win10 and the 64 bit client.

Pretty sure this is happening to everyone. I know of at least two other people in different parts of the world experiencing the same problem.
I would suspect Google changed something about how to connect to the hangouts service and hangups needs to be updated...

This is happening to me as well. I love your app guys please fix it so I can continue using it!

Same here. :cry:

yeah=(((

Same here

+1 on Mac

+1 here. console log shows:
2016-01-14 14:17:34 WARN init failed 404 Not Found
connect_failed

For me too. Latest version of hangupsjs is in use already, so I was unable to try a build with a newer version to see if that helped.

Same here on Mac Yosemite.

Indeed, the URL ('https://talkgadget.google.com/u/0/talkgadget/_/chat') hangupsjs uses to connect to Hangouts, does not exist anymore.

A bug has been reported (already) against hangupsjs: https://github.com/algesten/hangupsjs/issues/39

Same here Mac OSX "El Capitan" is there a workaround in the mean time to make it usable?

I'm trying to find if Hangups (the python original hangupsjs is a port from) has the same issue, and if not, what URL they are using to see if it is a simple issue.
If Google changed the API, it's probably not a simple workaround... :(

As explained earlier the endpoint at google.com has been removed. So it doesn't matter what your OS is!
Please wait until the developer find what the new endpoint is.

The python version still works, but it isn't clear to me which endpoint they use :-)

Mentioned in https://github.com/algesten/hangupsjs/issues/39#issuecomment-171648277:

https://github.com/tdryer/hangups still works, so they have the correct endpoint :)

+1 on Mac OSX

+1 on OSX 10.11.2

Working in 1.1.2 -- thanks! :)

gooooood!!

Whatever it is, but we are now in Dec 2016 and what is being described here in this thread is happening again. I can't use YakYak anymore. Same Not Connected, Reconnecting loop. I am on macOS Sierra and YakYak was working up until today.

Try version 1.4.0-beta4, it works for me!
Kind Regards!
Antonio “Toni” Muñoz
Sent from
https://polymail.io/?utm_source=polymail&utm_medium=referral&utm_campaign=signature
On Thu, 01 Dec 2016 at 16:28 Kay Wrobel
<
mailto:Kay Wrobel<EMAIL_ADDRESS>
wrote:

a, pre, code, a:link, body { word-wrap: break-word !important; }
Whatever it is, but we are now in Dec 2016 and what is being described here in this thread is happening again. I can't use YakYak anymore. Same Not Connected, Reconnecting loop. I am on macOS Sierra and YakYak was working up until today.
—
You are receiving this because you commented.
Reply to this email directly,
https://github.com/yakyak/yakyak/issues/111#issuecomment-264202998
, or
https://github.com/notifications/unsubscribe-auth/AD4yBKGsI8lzcBBW-r8MVDN4ccN_Ivl9ks5rDueUgaJpZM4HEgH2
.

yep...the new beta version fixes it...

How do you install 1.4.0 release version on mac?  It doesn't have a link for osx

an official version has been released
see here https://github.com/yakyak/yakyak/releases/tag/v1.4.0

Thanks @averissimo - saw that, but didn't see the osx version - is there a simple way for me to use one of the others - or do they need a conversion of some sort?  I noticed past releases had an osx version.
Thanks

@jskovtoo You would have to build it yourself. I just went through the same because 1.4.0 for OSX was not immediately available. You'd need to install npm on your Mac. I installed it through MacPorts (port install npm4). Then follow the build instructions. I'd say be carefor with the last step (deploy) because it will build versions for all platforms. So you'd want to deploy for darwin-x64.

thanks @kwrobel - I'm not super keen on that, since I'm not real technical and wouldn't want to mess things up - is it something where you can pass over the version you built, or I would need to re-do and build my own?  Thx for the help

@jskovtoo In that case I'd just stick to the binaries that are being released. I was impatient and wanted to get YakYak working again after my older version went into a reconnect loop. It was suggested to use 1.4.0, but no binary was available. So I built it and it works. Of course, with a little patience (as in a day or so), I could have just downloaded the binary when it was released for macOS.

Since a while I download the binaries from https://github.com/saenzramiro/rambox/releases.

I have the same trouble.
On the first login it is stuck at loading contacts.
On second start it is stuck at Connecting or says it can't connect and switching with connecting.
OS:
Windows 10 (64bit)
Program-Version:
yakyak 1.4.0 x64
yakyak 1.4.1 x64
yakyak 1.4.2 x64 and ia32

@klonamy can you please open a new issue with that information?


Add {VERSE}: {Verse} Bitcoin.com Token

This closes https://github.com/maticnetwork/polygon-token-list/issues/418

@Groovytent could you take a look please?

You've added both 'pos' and 'fx' tags. Make sure to add only one 'bridge' tag. If you do not want to make the token bridgable please add the 'noDeposit' and 'noWithdraw' tags.

addressed here: https://github.com/maticnetwork/polygon-token-list/pull/426/commits/2723e1ebf5d560d8faedb003de9db1eba5d648b2

👍

@Groovytent @DONOVA28 @mayanks1211 thanks for merging this, but we've noticed the metadata still is wrong on the contract, who can assist to correct this?
https://polygonscan.com/token/0xc708d6f2153933daa50b2d0758955be0a93a8fec#readContract#F7 we would like this to say VERSE not fxVERSE
https://polygonscan.com/token/0xc708d6f2153933daa50b2d0758955be0a93a8fec#readContract


feat(KONFLUX-2547): Make it possible to configure BuildPipelineSelector

Description
Make it possible to configure BuildPipelineSelector
Issue ticket number and link
https://issues.redhat.com/browse/KONFLUX-2547
Type of change

New feature (non-breaking change which adds functionality)

How Has This Been Tested?
Locally against Stage and will run a CI test I guess.

/test ?

/test load-test-ci-10u-10t

Looks good
Just ran the CI test to verify success (load-test-ci-10u-10t) ..

We can change the below line - return fmt.Errorf("error deleting build pipeline selector %d from %s: %v", i, f.UserNamespace, err)
With return fmt.Errorf("error deleting build pipeline selector %s from %s: %v", bps.Name, f.UserNamespace, err)

Hello Naftaly. Feedback incorporated. Please take a look now.

/test load-test-ci-10u-10t

Hi @jhutar
load-test-ci-10u-10t  has failed
Can you run it on stage from command line?

/test load-test-ci-10u-10t

/test load-test-ci-10u-10t

/test load-test-ci-10u-10t

/test load-test-ci-10u-10t

/test load-test-ci-10u-10t

ci/prow/load-test-ci-10u-10t load test fails as I fail in my PR (https://github.com/redhat-appstudio/e2e-tests/pull/1075)
PR is not recognized in the load test eventhough it actually created
You still get 100 errors of the below type (example) -
"errorCode": 20,
"message": "PipelineRun for applicationName/componentName ci10t-1111-0081-app/devfile-sample-icug has not
been created within 30m0s: context deadline exceeded"

Yes @naftalysh , but it is unrelated issue. It is happening in other PRs that does not have these changes in.

Yes @naftalysh , but it is unrelated issue. It is happening in other PRs that does not have these changes in.

Hi @jhutar
I thought I saw it also in this PR openshift-ci test
If this PR openshift-ci test passed than this comment is not relevant

/lgtm

/approve

/test load-test-ci-1u-1t

/test load-test-ci-10u-10t

/test load-test-ci-10u-10t

Obsoleted by https://github.com/redhat-appstudio/e2e-tests/pull/1163


Adding Spanish translations

Summary
Adding Spanish translations to https://github.com/cicirello/user-statistician/blob/main/src/StatConfig.py
Closing Issues
Closes #36
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Improvements to existing code, such as refactoring or optimizations (non-breaking)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[x] I have read the CONTRIBUTING document.
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

@alanverdugo thank you for contributing

@alanverdugo This just went live with release v1.9.0. :rocket:


Online editing of job manifest

The Lab expressed that it would be nice to be able to edit the manifest of a job directly in the RT dashboard and then redeploy the job with the updated manifest.
Let's discuss pros and cons of this functionality.

I think that's a good idea as long as we're fine with sacrificing Infrastructure-as-Code. The consequences are:

your online changes can be overwritten accidentally by a local deployment.
If the job is wiped out for some reason, nobody really knows what exactly was deployed there, can't recover the exact revision.
The history of changes is hard to track without git.

Nevertheless, If we use common sense, I consider this feature beneficial. We don't need to drop Infrastructure-as-Code completely.
Fast prototyping and temporary experiments could be made online, while keeping in mind that it should get reflected in git after all.

I feel like such jobs should quite obviously be marked in the dashboard as lesser somehow - preferably in bright red so that people don't fix something in a way that isn't properly infrastructure-as-code. I want to nudge people in the direction of not using online changes to get something up and running in production. Something like this (but prettier) I think could work:

It'd be really nice not to have an issue created sometime in the future because someone's jobs got wiped and no one can get them back up and running. A big red warning sign would help, I think.

Well this church needs a heretic.
Alternative proposal:

Stop sending job.yaml from racetrack client
Send instead a command to deloy which contains only instructions for fetching the git remote (including branch)
Obtain the job.yaml from the git remote
Poll the git remote for changes and redeploy automatically when there's new commits on the specified branch

In other words, gitops.
This way we don't have a sync problem.
If someone edits the job.yaml in the dashboard, it gets pushed back to the tip of the branch. Maybe even people will prefer to not edit in the web UI since the development loop becomes tighter.
I'm mainly dissenting to force some deeper thinking on this proposed change, but that doesn't mean I don't belive in this alternate idea.

I think there's a lot of value in being able to throw code onto the wall and seeing what sticks. Having a giant commit history because someone didn't know what they were doing and just tried 20 different things until one worked could be a bother to some people.
My own little heresy that I've been growing in my backyard revolves around implementing proper gitops, but being able to freeze it somehow for a small period of time - or freeze it in some way that's generally disruptive and has to be reconciled before normal operation can continue. I'm not sure about how this would work on a detailed level - but I am sure that I really wanna discourage people from using this web editor to do anything more than testing. Real development should live in git.
It's however an unfinished thought, and I think it'd be a lot more work than slapping an "out of sync" label on the Job - something which I think is valuable regardless.


docs: Clarify access to port 25 is needed for local email delivery.

Self-review checklist

[ ] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[ ] Each commit is a coherent idea.
[ ] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

Merged, and marked as a backport candidate.

Backported in #25994.


[template] Add support for ninemanga(all languages)

Filters are different for each language. If we have dynamic filter generation then we can merge in one


error running getmusic trainning on windows

hello there thanks alot for this awesome project ,i was able to make infereing run on my system but when i try to train my own dataset i get this error
(getmusic) C:\Users\Genesis\anaconda3\muzic\getmusic>python preprocess/to_oct.py example_data/train example_data/processed_train
Traceback (most recent call last):
File "", line 1, in
File "C:\Users\Genesis\anaconda3\envs\getmusic\lib\multiprocessing\spawn.py", line 116, in spawn_main
exitcode = _main(fd, parent_sentinel)
File "C:\Users\Genesis\anaconda3\envs\getmusic\lib\multiprocessing\spawn.py", line 125, in _main
prepare(preparation_data)
File "C:\Users\Genesis\anaconda3\envs\getmusic\lib\multiprocessing\spawn.py", line 236, in prepare
_fixup_main_from_path(data['init_main_from_path'])
File "C:\Users\Genesis\anaconda3\envs\getmusic\lib\multiprocessing\spawn.py", line 287, in _fixup_main_from_path
main_content = runpy.run_path(main_path,
File "C:\Users\Genesis\anaconda3\envs\getmusic\lib\runpy.py", line 265, in run_path
return _run_module_code(code, init_globals, run_name,
File "C:\Users\Genesis\anaconda3\envs\getmusic\lib\runpy.py", line 97, in _run_module_code
_run_code(code, mod_globals, init_globals,
File "C:\Users\Genesis\anaconda3\envs\getmusic\lib\runpy.py", line 87, in _run_code
exec(code, run_globals)
File "C:\Users\Genesis\anaconda3\muzic\getmusic\preprocess\to_oct.py", line 32, in
manager = Manager()
File "C:\Users\Genesis\anaconda3\envs\getmusic\lib\multiprocessing\context.py", line 57, in Manager
m.start()
File "C:\Users\Genesis\anaconda3\envs\getmusic\lib\multiprocessing\managers.py", line 579, in start
self._process.start()
File "C:\Users\Genesis\anaconda3\envs\getmusic\lib\multiprocessing\process.py", line 121, in start
self._popen = self._Popen(self)
File "C:\Users\Genesis\anaconda3\envs\getmusic\lib\multiprocessing\context.py", line 327, in _Popen
return Popen(process_obj)
File "C:\Users\Genesis\anaconda3\envs\getmusic\lib\multiprocessing\popen_spawn_win32.py", line 45, in init
prep_data = spawn.get_preparation_data(process_obj._name)
File "C:\Users\Genesis\anaconda3\envs\getmusic\lib\multiprocessing\spawn.py", line 154, in get_preparation_data
_check_not_importing_main()
File "C:\Users\Genesis\anaconda3\envs\getmusic\lib\multiprocessing\spawn.py", line 134, in _check_not_importing_main
raise RuntimeError('''
RuntimeError:
An attempt has been made to start a new process before the
current process has finished its bootstrapping phase.
    This probably means that you are not using fork to start your
    child processes and you have forgotten to use the proper idiom
    in the main module:

        if __name__ == '__main__':
            freeze_support()
            ...

    The "freeze_support()" line can be omitted if the program
    is not going to be frozen to produce an executable.

We did not test our code on Windows. Perhaps this helps: https://stackoverflow.com/questions/69515321/an-attempt-has-been-made-to-start-a-new-process-before-the-current-process-has-f.
According to this answer, you may need to move 'manager = Manager()' to within the if name = 'main': block.


Trusted claims

Reviewer checklist

[ ] Test coverage: stack test --coverage
[ ] Public API documentation: stack haddock
[ ] Style conformance: stylish-haskell

You should probably review the commits separately, as the Refactoring IndexedModule to include axiom attrs touches a lot of code (and does not do much).
Also, it will probably be untestable until https://github.com/kframework/k/pull/225 is merged

@virgil-serbanuta are you satisfied with the changes?

Do we really want to replicate the trusted attribute of the java backend? wouldn't it be better to simply tell the prover when you prove which claims you want to prove and which claims you want to trust?

@dwightguth I don't have anything against using a different mechanism, but that would have to be designed and work from the frontend towards the backend.  We could separate them in different modules and send two such modules to the backend, but I'm not sure it would be such a great improvement.
Also, languages such as Coq sometimes use something like Admitted to accept a result without proof.

Jenkins: test this please


IOS not working

ringerMode is undefined on IOS 15

Because of the limitations of Apple, as far as I know it's not possible to control ringer mode on iOS devices, so which makes this library an Android-only one. I think this should have mentioned in the README better.

How about accessing the silent mode state & do not disturb state, without controlling these mode?
One of the use case of accessing these state is notify user he's in silent mode probably not able to get notified when notification comes in.

@liho00 Still, as far as I know there is no such API to get status of whether or not device in its silent mode. I guess Apple restricts developers to access it because of privacy concerns. But I think it's possible to access focus feature of iOS devices and this can be implemented.

Hi @liho00, as @kutsan said I don't know a way to support iOS at the moment. If you find a way to do it, we are open to PRs. I'll keep an eye on this topic, thanks!


External links can't be followed

External links are properly rendered in content, but it doesn't seem possible to follow them. In a note, a single click on a link just expands text to show the link markup. In a ToDo, a single click does nothing. In either case, a ctrl + click or other key/mouse combination also doesn't follow the link to display it, e.g., in a browser/Unigraph view, etc.
Windows 11, Unigraph 16d448f

Hi! this has been fixed in the latest commit.


[5.0] Adds "--path" to the migrate:reset command

Hi all,
I've added a --path option to the migrate:reset command.
Usage for this should be similar to the php artisan migrate --path="..." command.
So after doing the above command, you can do this:
php artisan migrate:reset --path="..."
I'm adding this because we have the option of doing a migration from a specified path but have no option to reset the migration from that path itself.
Hoping you consider this pull request.
Thanks
Chris

You change files permissions. Care about that.

@RomainLanz i missed some of those - just figured out that laravel doesn't follow PSR-2 coding style. Thanks for pointing those out.

Laravel doesn't follow PSR-2 now, probably on the future.
Still dot missing on line 182.

Please squash to one commit.

Hi @GrahamCampbell done. Thanks!

@GrahamCampbell now using glob instead of scandir. Glob returns the full path of the files though but using basename did the trick. It even has a handy suffix parameter that can be used to remove the file extension.
Let me know if you're ok with it and I can squash to one commit again.

@GrahamCampbell moved

Thanks.

I'm not sure why this would be needed? migrate:reset should roll back all migrations.

Hi Taylor
Here is an example use case:
Let's say I wanted to install a Laravel package and it has its own migration files. Part of the instructions would be to migrate them using php artisan migrate --path="...".
But let's say we don't need the package anymore and wanted to remove only the migrations from that package without having to touch the main application's tables. Or another scenario would be that there was an updated migration file from the package. We don't want to reset the whole database just for this package.
In this case, we can use php artisan migrate:reset --path="..." and not touch the main tables. It makes sense to enable resetting from the path if we can migrate from a path.

good idea, also why we have only one hardcoded in config emailView for reset password ?
think those problems from one way - you Tailor should think wider about modularity

Going to hold off on this for now.

+1 great feature but please redo it for 5.1

Changes Unknown when pulling f7ce2d5d918e7e9c39a94b86e334ae1f72468c42 on chrisbjr:5.0 into ** on laravel:5.0**.


[utils/gyb_syntax_support] Add collection_element_name for when specifying a Child

This is useful to provide add<ELEMENT_NAME> APIs for SwiftSyntax that are better named
and without having name conflicts.

@swift-ci smoke test

Are you planning to do the actual API name change in follow-up PR?

Are you planning to do the actual API name change in follow-up PR?

Yes, it will be in swift-syntax repo.

How about C++ APIs renamed in https://github.com/apple/swift/pull/24190/files ?

Ah good point, I missed those. Will update PR.

@swift-ci please python lint

@swift-ci please python lint

@swift-ci smoke test

@swift-ci smoke test

@rintaro see latest changes.


Testing xCO2-pCO2/fCO2 conversions

Not sure if you're already aware of this, but I just came across another tool that could be used to validate/compare these conversions - https://github.com/dpierrot/fCO2_Calc

Thanks for finding this! Will look into it.


Added Responsive Landing Page with Dark and light toggle

🛠 Fixes Issue #464

👨💻 Changes proposed

✅ Check List (Check all the applicable boxes)

[x] My code doesn't break any part of the project (Zero Octave-Javascript-Projects).
[x] This PR does not contain plagiarized content.
[x] My Addition/Changes works properly and matches the overall repo pattern.
[x] The title of my pull request is a short description of the requested changes.

📄 Note to reviewers

Hey, project admin as you assign me this issue to work on I have completed the project and created the pr regarding that I hope that you will go through my pull request and merge it.
📷 Screenshots

delete the json file and add your project in the index and cardsdata file too

Hey Abhishek I have changed the code and added my project to project's
index.html file so please go through it and review it thanks 🙂
On Mon, Apr 11, 2022, 7:15 PM Abhishek Kumar @.***>
wrote:

delete the json file and add your project in the index and cardsdata file
too
—
Reply to this email directly, view it on GitHub
https://github.com/ZeroOctave/ZeroOctave-Javascript-Projects/pull/476#issuecomment-1095072515,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWWDINXFQWSNPUTXT2IGODTVEQUHVANCNFSM5TA24RHA
.
You are receiving this because you authored the thread.Message ID:
@.***
com>

@NOiR-07 Please review the PR I have done the changes

you haven't added in the index file of website
for ref: check others code how they did

you haven't added in the index file of website for ref: check others code how they did
@NOiR-07 look at this


Overloads should consider optional parameters and a type union including undefined as identical

TypeScript Version:  2.4.2
Code
class A {
  constructor(); // Error: overload signature not compatible with impl
  constructor(a: number | undefined) {}
}

class A {
  constructor();
  constructor(a?: number) {} // this one works
}

Expected behavior:
I expected the first example to work especially since the type of parameter a is identical in either constructor implementation. I think it is helpful when writing overloads to not use optional parameters in the implementation signature and be explicit about the types that might exist in each position.
Actual behavior:
The first results in the noted error.
Commentary:
The following error may a moral equivalent of the error I am objecting to:
function foo(a: number | undefined) { }
foo(); // error, even though it's "implicitly" passing undefined

but I think this case is different as on the call side the error is justifiable on the grounds that we think it's likely the caller forgot a parameter (even if undefined is an allowable type, you should be explicit).

Duplicate of https://github.com/Microsoft/TypeScript/issues/12400

Automatically closing this issue for housekeeping purposes. The issue labels indicate that it is unactionable at the moment or has already been addressed.


fix: fix backend warnings

This looks like two separate PRs?

Hmm it's all mainly to silence warnings when running the backend.

What would you separate out?

The system-name stuff from the --factory and removing the run server stuff but no worries it's fine

integration tests might actually be doing something?

Did some debugging - this breaks because slices are stored with "modelName" in the database so existing project slices fail to render the model.
INTEGRATION TESTS WORK!

idk how to fix, or if it's worth the name refactor


Brief Thank you and Make Public Feature Request

Wanted to thank you for your work on this; it is the only open source API for Suno I have found easy to get setup and working. I plan to use this to tide me over until they make an API eventually, hopefully. I could not find your donation/tip button on Kofi, feel free to provide something that we can use to compensate you for your efforts.
I was wondering if it is possible (either as it is currently setup or in the future) to, through the API, control whether a song is public as well as retrieve its public link. I use the API for live generation and it would be convenient for people to be able to access the songs they generated live later via the link instead of only via the MP3 I download.
thank you, much appreciated, sorry for any oversight if this is already in place and I simply missed it (or if it is simply not possible to do other than on the Suno site itself)

Hello,
Thank you for your encouraging words! I’m glad the Suno API has been useful to you. Regarding your feature request, it’s not currently available, but I’m open to implementing it in next version (A new method to **return direct mp3 download link**). Do you have any specific ideas or requirements for this feature?
For donations, you can support my work through Ko-fi.
Thanks for your support and suggestions!

Hello,
Thank you for your encouraging words! I’m glad the Suno API has been useful to you. Regarding your feature request, it’s not currently available, but I’m open to implementing it in next version (A new method to return direct mp3 download link). Do you have any specific ideas or requirements for this feature?

Set whether the song will be made public or not once it is generated
If set to public, a link can be output to share it publically (currently we can make links to the audio stream and image by appending the song ID info to those URLs, but there is no way to change a song to public through the API or set it to make songs public by default). Since suno has their own URL structure for public links we can simply append onto that if it is possible to make the songs public once they are generated.

For donations, you can support my work through Ko-fi.

This was noticed but it looks like donations to your KoFi have been disabled, we cannot find any button to actually tip.

Thanks!

Hello @thatTinker ,
Thank you for your suggestion! I've added a new set_visibility() method to the API that allow to set songs as public or private. This feature is now available in the latest release.
You can check out the release here: v1.0.7
And here's the commit that introduced this update: b865838...38302bcf
Example Usage:
from suno import Suno, ModelVersions

client = Suno(model_version=ModelVersions.CHIRP_V3_5)

# Generate Sample Song
clips = client.generate(
  prompt="A peaceful melody reflecting a serene landscape",
  is_custom=False,
  wait_audio=True
)

# Make All Private | Link Only
for song in clips:
  is_public = client.set_visibility(song_id=song.id, is_public=False)
  print(is_public)

Please feel free to test this new feature and let me know if everything works as expected or if there are any further improvements you would like to see.
Thank you for contributing to the improvement of Suno-API!


Change how we select top n children for a set of parents

Using the 'rank() over(..)' window function for ranking and selecting the top n children per parent turned out to be very slow in some cases. With  a 'join lateral' we are essentially encoding a for loop in the queries which seems to perform much better.
This also eases subgraph composition since we now never join two 'real' tables, and therefore do not need to worry about joins crossing subgraph boundaries
Fixes https://github.com/graphprotocol/graph-node/issues/1450

@lutter What's the deployment plan?

Would also be good to know what level your confidence in the implemenation is at.

@lutter What's the deployment plan?

It would be deployed like any other code change by pushing to staging and production.

Would also be good to know what level your confidence in the implemenation is at.

Pretty confident. I would do some more testing of queries in staging once we start rolling this out (or maybe run it against a clone of the production db and try out important real-life queries with @verify)

Addressed the review comments and rebased to latest master

@Jannis I added one more commit that fixes #1438 as that bug was throwing off my testing that compares slow and prefetch execution for the queries we see in a day. Can you have a look at that, too? I am going to rerun the mass query test to make sure that the two really do produce the same results now (that bug was the only thing that was different when I replayed 1.3M of the 4.3M queries production saw on 2/23)

The tests have been running for 12 hours, and churned through 3.3M of the 4.3M queries for 2/23 (126k unique queries) and for each of them slow and prefetch execution returned the same result.

@lutter Let me know if you want to merge this. The test suite actually finished, despite GitHub thinking it got stuck. May need my admin permissions to force the merge.

@Jannis I am waiting to merge this until the large notification change has hit production because I want to see if that changes some of the db characteristics first. I'll merge this shortly after that hits prod


Basic R2 plays hide and seek with HomeKit

Hello,
Using homebridge-config-ui-x v4.41.4 on Windows10 running homebridge-ewelink v8.7.1 plug-in for Sonos BasicR2.
The device disappears from HomeKit after a full day. All efforts removing & reading homebridge to HomeKit result in device being discovered and added only to have it disappear after setup is complete; but if I do the same thing in the morning it works fine.
Additionally if I skip removing & reading HomeKit shows all devices offline while the Ewelink app and Tuya smart app shows devices to be online, even homebridge itself never loses any of the device but HomeKit keeps losing them overnight.
Could you please help ?
Many thanks.

hi @doctornakhare
do you have devices on your homekit network on an older version of ios like 10 or 11?
I have seen before some strange behaviour with devices going “missing” on all devices when users have a device with an old version of ios.
Another question, when the accessory goes missing from homekit, is it still visible as an accessory in the homebridge ui?

Hi @bwp91 ,
Thank you for responding so soon 🙏🏻
No I’m on the latest iOS & yes they are still visible in homebridge Accesories.
I found out my problem ! I think !!
Parents kept turning off the AppleTV , the Tuya lights would still be visible in HomeKit but the sonos would disappear.
Did I guess it correctly ?

I don't necessarily mean your phone with an older iOS version, but any device within your HomeKit network with an older version of iOS, like an old iPad laying around, or one of your parents phones.
If the accessories are still visible in the homebridge UI then this definitely indicates a HomeKit issue rather than a plugin issue.

Yes , I do have an iPad Mini gen 1 just lying around.
If this is the case do what do you recommend ?

Sign off from the said iPad  Or
Is there a possible way to keep it signed in for Music and other purposes and remove it from HomeKit ?

In the icloud settings you should just be able to turn off the 'Home' sync.
Once you've done this, I would suggest restarting your home hub device (apple tv) and also homebridge. And see if this fixes this issue.


Introduce rate limiter into streams cache

Wanted to solicit feedback on this change. (I have not written tests for it yet, so not ready to be merged.)

I thought more about this approach and I think it can have undesired consequences. What I am trying to guard against is multiple threads asking the stream for records at head and get empty results. I believe this approach prevents that, but also has the undesirable side effect that if there are records, one of the threads won't return any. I am working on an alternative solution.


Shell RC files are sources in home directory instead of PWD

Helo,
Some problems with alias ? Alias defined in the upterm is perfect, but alias in .zshrc dysfunction with the ENV.

I can confirm this behavior differs from ZSH, but it seems to me that it's brittle to rely on a shell startup directory in the first place.  Since sessions in upterm always start in the latest PWD, it will be different every time. Is that what you expect?

Thx for ready, i expect it's work like Konsole or other. Because my alias depend of pwd for execute my mix (elixir) task.

Isn't it because $PWD is immediately expanded, resulting in alias testa=echo /home/zack, and then alias testb=echo /home/zack/Projects/Nawo/apps/exfdets? Shouldn't you declare your aliases between simple-quotes instead of double-quotes?


Update the report object structure to include reportFields

Held on https://github.com/Expensify/App/issues/32757
https://docs.google.com/document/d/1LXGXwWsei-Zx2wJMaYBmVe_M8NH7AHJZ5bIqwSec4uo/edit#bookmark=id.b2mfzn46smk

Fields getting added to the policy. Now to figure out how to get them added on the report

Okay, double checked and they should be populating in the OpenReport API call 👍


[1.3] Bridged Device Basic Information Cluster XML has to be updated as per spec

Description
XML has to be updated for below event:

BRBINFO.S.E02(Leave) - FabricIndex field has to be added for the Leave event.

XML Reference:

Specification Reference:

Note: Bridged Device Basic Information Cluster is derived from Basic Information Cluster.
Bug prevalence
Whenever I do this
GitHub hash of the SDK that was being used
https://github.com/project-chip/connectedhomeip/commit/eb2f8f6ca95c181577415bc4b3e7cc240d5b2edd
Platform
raspi
Platform Version(s)
RPI - 4, 8GB RAM
Type
Test Improvement

Looks like https://github.com/CHIP-Specifications/connectedhomeip-spec/pull/5264 was never followed with an SDK issue.

Looks like Basic Information as fixed, so this is just Bridged Device Basic Information.


Make sure entire code base is properly formatted

While testing the new format I noticed that there are A LOT of files that are not correctly formatted but didn't show up because we only look at touched files.  You can repro this by running ./scripts/check.py format tree --fix velox and check the changed files in git.

Originally posted by @assignUser in https://github.com/facebookincubator/velox/issues/9948#issuecomment-2161795278

Sounds reasonable to me. Not sure how we ended up in this situation though; shouldn't CI check for format on everything new PRs are touching?

There was a recent update to the clang-format file that probably covers most of what I saw? https://github.com/facebookincubator/velox/commit/955c071593c410cfe559ba3f51f563af4bad1195

Cc: @kevinwilfong

This is still not done.


[Improvement] Netty Support getLocalShuffleData and getMemoryShuffleData.

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Search before asking

[X] I have searched in the issues and found no similar issues.

What would you like to be improved?
At present, only sendShuffleData uses netty. The three interfaces getLocalShuffleData, getMemoryShuffleData, and getLocalShuffleIndex all support netty, but they have not been enabled yet.
How should we improve?
No response
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

any update on this?

I think we already support this. Closed.


This page is useless without providing the Compass version that can be used

Last Compass version 1.21.0 does not work.
At least provide the Compass version that was used when creating this page.
The screenshots are too old.

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

ID: 3ae63b0b-8c13-36da-c5c9-d126ffbc7077
Version Independent ID: 17b87fc6-88ad-959b-9b11-452844ee2279
Content: Connect to Azure Cosmos DB using Compass
Content Source: articles/cosmos-db/mongodb-compass.md
Service: cosmos-db
Sub-service: cosmosdb-mongo
GitHub Login: @LuisBosquez
Microsoft Alias: lbosq

@stephanebachelier We are checking this and will get back to you.

@stephanebachelier The latest Mongo compass is only compatible with 3.2.
Product Team is working on getting it compatible for version 3.6. Please let us know if you need any further help.

@stephanebachelier This is being assigned to the content author to evaluate and update the documentation as appropriate.

I also cannot access Cosmos DB Mongo DB API(version 3.6) account from Compass

@stephanebachelier , @andresferreira Currently only 3.2 version works with MongoDB compass, the 3.6 server version doesn't yet support it. I updated the note in the doc to clarify in the above PR.
Adding support is under works, we will have an update in a month or so. We will now close this issue.
#please-close


FIR regressor names are incorrect

In http://neurovault.org/collections/1692/fsl_hrf_fir.nidm.ttl, the regressor names include a temporal derivative while they should list all the column names including the FIR basis:

 a nidm_DesignMatrix:,
        prov:Entity ;
    rdfs:label "Design Matrix" ;
    dc:description niiri:b0c84d24-94b9-4458-bfd7-b2d4584276c0 ;
    dct:format "text/csv"^^xsd:string ;
    nidm_regressorNames: "[\"tone_counting\", \"tone_counting*temporal_derivative\", \"tone_counting_probe\", \"tone_counting_probe*temporal_derivative\"]"^^xsd:string ;
    nidm_hasDriftModel:  ;
    nidm_hasHRFBasis: nidm_FiniteImpulseResponseBasisSet: ;
    nfo:fileName "DesignMatrix.csv"^^xsd:string ;
    prov:atLocation "http://neurovault.org/collections/BNRPBNVV/fsl_hrf_fir.nidm/DesignMatrix.csv"^^xsd:anyURI .

This issue was identified by @TomMaullin and discussed with @nicholst.

This has been solved in #103.


Ensure 'service trafficserver stop’ is synchronous under load on redhat

Currently the service trafficserver stop command on redhat might return before ATS has exited.
This is not good as it’s often followed by a start, which then fails.
/etc/init.d/trafficserver does:
        action "Stopping ${TC_NAME}:" killproc -p $TC_PIDFILE $TC_DAEMON
        action "Stopping ${TM_NAME}:" killproc -p $TM_PIDFILE $TM_DAEMON
        action "Stopping ${TS_NAME}:" killproc -p $TS_PIDFILE $TS_DAEMON

and killproc, as defined in /etc/rc.d/init.d/functions, with those arguments essentially does this:
send SIGTERM
wait up to 3 seconds for it to exit
send SIGKILL
wait 0.1 seconds then return an exit status indicating if the process still exists

A SIGKILL signal always causes the death of a process but it’s not instantaneous. A process can take a long time to exit on a busy system for assorted reasons, including flushing dirty buffers to disk.
So if the stop is immediately followed by a start, as it often is, the start may fail with an error like ‘port 80 in use’. This seems to be a common cause of restart failures on busy systems and frustrating manual hand-holding.
Contrast this with the behaviour that /etc/init.d/trafficserver uses when run on Ubuntu… there it’ll wait up to 35 seconds.
(It'll also use SIGQUIT instead of SIGTERM which seems odd).
This PR makes /etc/init.d/trafficserver more reliable, and consistent, on redhat by adding -d 35 to the killproc arguments so it'll wait for the daemons to stop on redhat for about as long as it does in ubuntu. Not perfect, but much better.

[approve ci]

Intel CC build successful! See https://ci.trafficserver.apache.org/job/icc-github/74/ for details.

FreeBSD build successful! See https://ci.trafficserver.apache.org/job/freebsd-github/1641/ for details.

Linux build successful! See https://ci.trafficserver.apache.org/job/linux-github/1537/ for details.

clang-analyzer build successful! See https://ci.trafficserver.apache.org/job/clang-analyzer-github/206/ for details.

Having some issues with this on my F25 box, e.g.
Stopping traffic_cop: pidof: invalid option -- 'p'

Usage:
 pidof [options] [program [...]]

Options:
 -s, --single-shot         return one PID only
 -c, --check-root          omit processes with different root
 -x                        also find shells running the named scripts
 -o, --omit-pid <PID,...>  omit processes with PID

 -h, --help     display this help and exit
 -V, --version  output version information and exit

For more details see pidof(1).
pidof: invalid option -- 'p'

I have not investigated why yet, I'm assuming something odd with how they parse arguments.

Gah! killproc is written to need the -d option to follow the -p option. Sorry!
killproc -p $TS_PIDFILE -d 35 $TS_DAEMON

[approve ci]

FreeBSD build failed! See https://ci.trafficserver.apache.org/job/freebsd-github/1642/ for details.

Linux build successful! See https://ci.trafficserver.apache.org/job/linux-github/1538/ for details.

Intel CC build successful! See https://ci.trafficserver.apache.org/job/icc-github/75/ for details.

The FreeBSD failure is because of the VM being unstable (bloody FreeBSD needs a reboot once a week or so).

clang-analyzer build successful! See https://ci.trafficserver.apache.org/job/clang-analyzer-github/207/ for details.

[approve ci]

FreeBSD build successful! See https://ci.trafficserver.apache.org/job/freebsd-github/1648/ for details.

Intel CC build successful! See https://ci.trafficserver.apache.org/job/icc-github/80/ for details.

Linux build successful! See https://ci.trafficserver.apache.org/job/linux-github/1544/ for details.

Dang it my bad... I'm gonna revert these for now. @timbunce can you please squash the commits into one commit, and push again?

Not sure if it'll let you push again on this PR, but if not, just make a new one. The point is, we don't want broken / non-functional commits in the history, so I reverted this.

clang-analyzer build successful! See https://ci.trafficserver.apache.org/job/clang-analyzer-github/212/ for details.


Update function description

Design is for catalog, but will be the same for functions

We don't have BE API for this yet and is currently not in scope for 0.2


Code folding on []

Hi!
Currently, I can fold the code on { ... }, but not on [ ... ].
In the example below, I can fold $a, but not $b.
I honestly think this should be fixed.

$a = {
  ... // something
};
$b = [
  ... // something
];

Have a good day!

Lack of info, so closing.


Define ArtifactManifest type which collects ArtifactIds for build inputs which produced an artifact.

In the OmniBOR specification, Artifact Input Manifests record the identifiers of the inputs used to produce a software artifact. These identifiers are recorded in lexicographic order and may be stored in two distinct ways:

The full record embedded in the software artifact they describe.
Just the identifier of the manifest itself embedded in the artifact.

Either way, some embedding is happening, and the nature of the embedding is dependent on the type of artifact which is being embedded into.

For ELF files, the embedding involves a specified notes section being added
For text files, the embedding involves comments being added in a specified manner

Note that further embedding extensions can be added, and I believe the goal of the OmniBOR spec in general includes specifying embedding mechanisms for as many popular formats as possible. This is very important given that the exact nature of the embedding impacts identifier generation for the thing being embedded into. For things to be reproducible independently, they must be very carefully specified.
In the Rust code, we probably want to name the type something like ArtifactManifest or InputManifest rather than ArtifactInputManifest just because that name is enormous.
The specific operations for it are going to basically be constructing based on some bundle of inputs for which identifiers are produced, and then having some mechanism to embed in the output artifact.
The goal of the API is to make this as foolproof as possible.

This has actually been done, though embedding mode is not yet implemented.


feat: replace lookmlcodeblock with codedisplay

Adds in codedisplay and addresses sqlTableName as promised.

Gotta merge #27 first

Yeah, you're right. And it does work still. Sorry about that, had a Thursday moment :)


New Demo C9.io

https://agar-io-clone-d3vont3ch.c9.io/
this works like a charm

What's the location?

i don't know

Do we need to login? It asked me to login first.

serious?

Yeah. Cloud9 apparently asks for an account.

I added your server to README.md, @D3VONT3CH  :D

@D3VONT3CH Do we need any config to run it on C9 (like Heroku, OpenShift)? If need, could you make a patch to put these config in server.js please? :D

try now, was my mistake, sorry

no, is the same code.

great! it's work fine now!


[Degradation Pipeline] What's the meaning of 100 PSF kernels

Hi, I notice that you applied PSF kernel from pyblur, however, I can't find any description about these kernels, do you have any ideas about it?

Thanks. I'll try it, though I think it is hard to get response from them, the repo is created 8 years ago.

No worries. You can also visualize those kernels directly if there is no response :)


Extending the block list exporter to get additional information like color, stone type etc. from the mcstructure file

It's a first draft and I've tested it against all available test structures and some of my own. I will refine it in the future but I think it's a good first step.

Added en_US translations for most blocks

Moving the target of the PR to 1.5 develop. this allows us to keep mainline clean of the development bugs for people running from source.


Implementation of passing the OTP value to the UI and Storing it in t…

…he db

Please add method descriptions in the doc comments.


Add a waterfall series type closes #591

V2 of the waterfall series.

This is a big improvement, I like the waterfall series is a simple extension of the bar series.
The algorithm looks OK, but has a couple of issues:

It assumes that the start will always be a total
I'm not keen on the way that the user provides 'totals' as an array.

As an alternative to (2) why not use a visitor pattern?
Let's say we have some data which contains the monthly profits:
{
   { date: 2011-01-01, profit: 4000 },
   { date: 2011-02-01, profit: -2000 },
   ...
   { date: 2011-12-01, profit: 8000 },
}

I'd like to render this as a waterfall with quarterly totals, as follows:
var data = { ... see above ... };

// I'd expect to render this on an ordinal axis, so convert each date into a string ...
data.forEach(function(d) {
  d.month = d3.format(d.date, 'blah');
});

var algorithm = fc.series.algorithm.waterfall()
  .total(function(datum, index, data) {
    if (datum.date.month() % 4 === 0) {
       return "Q" + Math.floor(datum.date.month() / 4) + " total";
    }
  });

var waterfall = algorithm(data);

Here the total function takes a 'callback' with arguments of (1) the current datapoint, (2) its index, (3) the entire array of data. If it returns a value, this is used as the 'x' component and a total is constructed. If not, no total is added.
You can create a default total implementation that always creates a total at the end of the series.
It feels like a bit of an anti-pattern to rely on a function without an explicit return actually returning undefined - might be worth asking @chrisprice for his opinion on this.
As the component has to both read and write the x value, I don't think a value accessor is appropriate. If you look at the spread component, it uses an x value key for the same reason.
I'd also add a property:
fc.series.algorithm.waterfall().startsWithTotal(true);

This would be used to indicate that the first datum in the input array of data is actually a total.

Thanks @owennw a big improvement, a few comments on the algorithm implementation. Also looking at the visual tests, I'm not sure the rendered series looks correct for the input data?
One final thing, the algorithm could really do with some unit tests!

Thanks for the updates - a big improvement, pretty much ready to merge. However on further testing I do think the algorithm is wrong, it doesn't maintain a running total.
As an example:
        var data = [
            { quarter: 'Q1', profit: 4000 },
            { quarter: 'Q2', profit: 2000 },
            { quarter: 'Q3', profit: -1000 },
            { quarter: 'Q4', profit: 1500 }
        ];

This should result in 3 rising bars, for three quarters with profit, and one falling. This is what you actually get:

By changing the algorithm to maintain a total, this gives the correct result:

Once this is fixed, I'm happy to merge

Just awaiting a squash on this one ...

Thanks @owennw  :-)


feat: Adds Parser interface and JsonParser component to declarative framework

What

Adds new Parser interface
Adds new JsonParser component which accepts str, bytes, or bytearray data and yields a dict object.
Resolves https://github.com/airbytehq/airbyte-python-cdk/issues/164

Recommended Reviewing Order:

declarative_component_schema.yaml
parsers.py
model_to_component_factory.py
test_parsers.py

Summary by CodeRabbit

New Features

Introduced a new JsonParser component for flexible data parsing.
Added functionality to create instances of JsonParser through the ModelToComponentFactory.

Bug Fixes

Improved formatting and readability of various field definitions across multiple classes.

Tests

Added tests for the JsonParser to validate its ability to parse JSON data in different formats.

Documentation

Updated copyright notice in the test files.

@coderabbitat
First off, you should ignore all __init__.py files when providing a recommended reviewing order. Additionally, for the context of this repo, you can always ignore including the declarative_component_schema.py file as this is auto-generated based on the declarative_component_schema.yaml file. Additionally, when providing a recommended reading order, it's fine to just include the filename and not the full path (unless there are multiple files with similar names).


Fix setup script permissions

What this PR does / why we need it:
Fixed too wide permissions of the /opt/bin/setup
NONE

/retest

/hold cancel
3 days have passed, removing hold


Multipath interactive mode: websockets doesn't work with Wi-Fi without internet connection

Websockets doesn't work when iPhone has both WiFi and 3G enabled and connected, but WiFi router has no internet connection.
Steps to reproduce

Enable 3G
Enable Wi-Fi
Connect to router Wi-Fi with stable internet connection
Open any test application with Starscream
Get the internet cable out of Wi-Fi router
See websocket is disconnected and never reconnects

Usual standard REST requests that use .intereactive multipath mode works fine.
I found that your issue may be related to usage of c sockets, but seems you don't use POSIX sockets.
https://developer.apple.com/library/content/documentation/NetworkingInternetWeb/Conceptual/NetworkingOverview/CommonPitfalls/CommonPitfalls.html
Is there any way I can switch my connection from Wi-Fi without internet connection to 3G with a good internet connection?
Ideally, it should be done automatically.
Thanks

NSURLSessionStreamTask uses URLSession, a session can use .intereactive multipath mode

This sounds like a reachability issue, which has already been discussed on #401. You will need to implement Apple's Reachability APIs and handle whatever your app's reconnection logic when that happens.


Problem with T180 and cozero complemented

I was wondering why the Odd-even topology (product of discrete countable by two points indiscrete) was showing as not being perfectly normal:
https://topology.jdabbs.com/spaces/S000005/properties/P000015
It is not T6, as it is not even T0, but it should be perfectly normal, as far as I can tell.
The reason seems to be the combination of these two theorems:
T180: Perfectly normal implies cozero complemented
T126: Cozero complemented implies T3.5
The article mentioned in the reference for cozero complemented (https://topology.jdabbs.com/properties/P000061/references), as well as https://www.sciencedirect.com/science/article/pii/S0166864103003766 only mention that notion in the context of Tychonoff spaces.  I assume that's why the definition assumes T3.5.
So it seems we need to change T180 to "T6 implies cozero complemented".
Would you agree?

So this is interesting, according to https://topology.pi-base.org/properties/P000061 the pi-Base thinks that "perfectly normal" https://topology.pi-base.org/properties/P000015 implies T_{3.5} https://topology.pi-base.org/properties/P000006 (by way of co-zero complemented). In fact our definition of cozero complemented assumes T_{3.5} (since it's defined in that context).
This reveals a bug here: https://topology.pi-base.org/spaces/S000004/properties/P000015 as any indiscrete space should be perfectly normal vacuously. So I'll make a PR with indiscrete=>perfectlyNormal and fixing T180 as @prabau mentions. I'll toss in indiscrete=>compact as well. @lyengulalp check my work in that upcoming PR.


added project toml

Travis could protest here, we'll see

Oh hadn't seen https://github.com/SCIP-Interfaces/SCIP.jl/pull/119, closing


Bicep deploy - additional params support

Fixes:
#6159
#7005
#7019
#7024
Changes in the PR include:

Support for additional/missing parameters. Note: we only support parameters of type int, bool and string. Arrays and objects are ignored for now because of the complexity involved
Missing params – these are parameters that are not specified in parameters file and don’t have a default value in bicep file. We will show an input box that will let the user input value for missing params
Params with default values – these are parameters that are specified in bicep file with default value. We will provide an input box, prefilled with default value from bicep file. User can either press enter to use the default or provide a new value. Note: Expression strings are handled differently. We will provide three options here: 1. Use the expression as is 2. Provide a new value 3. Leave blank. Note: In case of (1), value will not be written to parameters file
After iterating through all the parameters, the user can either choose to create a new parameters file, if it doesn’t exist or update an existing file. Note: secure parameters will be sent across to the azure sdk and used in deployment, but will not be written to parameters file
UI changes to show all the .json/.jsonc files from the workspace along with current – None and Browse options
Added telemetry
Added unit tests

Microsoft Reviewers: Open in CodeFlow

@ucheNkadiCode, we discussed about ignoring unused parameters. I think we need to support it.
File with below contents is deployable:
param test string = 'abc'

So for the below param we need to show input box to accept value from user:
param test string

Let me know if you disagree.
@shenglol , thanks for finding the issue. I fixed it here:
https://github.com/Azure/bicep/commit/27c2b1e8f26f8bcaad48f2290a7db62d2867e93b


Angular not rendering

Hey there. This is a very cool service but I'm having a lot of difficulties getting it to render and I'm not sure what's wrong.
Here are some details:
I'm using Angular 1.5.8
Relevant index.html code I've added:
<meta name="fragment" content="!">
<base href="/index.html">

Relevant app.js code:
$locationProvider.html5Mode(true).hashPrefix('!');

I've tried with and without window.prerenderReady, so I assume the problem is something else. For now I've left all if it out to see if Prerender can figure it out.
As far as our backend, we are using varnish which forwards a request to Prerender when ?prerender=1 is in the URL (a param used for testing to emulate googlebot).
Basically the problem is that Prerender seems to be 'half' working. I can tell it's doing something because it's removing script tags, as well as showing the <style></style> tag that angular automatically injects for ng-show, but all the other html is still using <div>{{templating.variables}}</div>.
Does anything about the way I'm using Prerender look wrong?
One last thing, the way were accessing the URL is like this:
https://example.com/featured?prerender=1&_escaped_fragment_=

Have you tried using your own prerender server locally? I'd suggest uncommenting the logger() plugin in the server.js file and then passing the logRequests parameter.
That should show you any javascript errors on the page.
Also, you don't need to check if prerender=1 since you are using the _escaped_fragment_ query parameter.
Post back and reopen if you don't see any javascript errors but still have issues.


Disable -Werror for parentheses-equality warning

This warning fires here:
/home/travis/build/jaagr/polybar/src/utils/command.cpp:135:46: error: equality comparison with extraneous parentheses [-Werror,-Wparentheses-equality]
    } else if ((((*(int *) &(m_forkstatus))) == 0xffff)) {
                ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~

and as this system macro cannot be changed, let's not turn this into an error

Thanks


Redesign: DAO tree view

Overview
Implement DAO Node view using Chakra UI components
Link to documentation
https://www.figma.com/file/uRV3Nfowdzt5CknW1FgUkQ/📚-Fractal-Design-System-|-Base-Components-|-October-2022?node-id=504%3A415

@decent-dao/fractal this will display DAOs which the parent DAO has veto permissions over, not necessarily the DAOs it has created.
For example, if the subDAO votes to remove the veto guard and that proposal is not vetoed, that subDAO would no longer appear in this node tree.


Copy right Update

Hello:)
The website is still under 2023. I would like to update it

Isaw it is an issue


MIME multipart support

We should provide parsing support for dealing with MIME multipart messages. This would be useful for things like file uploads and batching.

Does that mean that asp.net 5 rc1 will contain batch support?

No. Open a new issue here https://github.com/aspnet/home/issues to discuss support for batch.


[Feature request] Hide cursor while typing

https://superuser.com/questions/928839/what-does-the-hide-pointer-while-typing-feature-actually-do
Can we get this as an option in VS Code?

I would really enjoy this feature.

That would be very handy!

Even I would like this feature

Please add this feature

You can use a free 3rd party utility called AutoHideMouseCursor that will not only hide the cursor for vscode, but over the entire OS. Quite handy. I found it after finding this thread. It would basically, just hide the mouse on inactivity for a predefined time.

Dude how is this still not implemented. This is mandatory for any serious code editor! Terrible..

I tried looking into adding this but as far as I can tell there is no way to hide the cursor with Nodejs or Electron.
It would be possible by calling the OS functions to hide/show the cursor, like this package does https://www.npmjs.com/package/hide-cursor. But I dont know the protocol for adding dependencies to this project.
Another possibility is using document.body.style.cursor = 'none'; but when I tried this it would only hide the cursor in areas other than the text editor area (the document I was altering had "workbench" in its name). So maybe someone more familiar with the project will know how to get the correct document to be able to apply the 'none' cursor style.

There was another issue filed with a relevant chromium bug report.
Gif demo of Notepad behaviour regarding mouse cursor:

Originally posted by @alexdima in https://github.com/microsoft/vscode/issues/79915#issuecomment-525338610

vs code, and just plain microsoft windows (this doesn't even work in the OS for tons of users) should figure this out. it's a very common situation for a user to click a spot to edit text and start typing right in the spot they last had their cursor. it's probably in the hundreds of times or more that i obscured a letter as i typed with no way to verify the word(s) without moving my mouse out of the way.
otherwise all <3

You can use a free 3rd party utility called AutoHideMouseCursor that will not only hide the cursor for vscode, but over the entire OS. Quite handy. I found it after finding this thread. It would basically, just hide the mouse on inactivity for a predefined time.

Thanks a lot! On Linux, a similar utility is available called unclutter that hides the pointer system-wide.
$ sudo apt-get install unclutter
$ unclutter -idle 1 -root # for hiding after a one-second delay

AskUbuntu Answers
Ubuntu Manpages
ArchWiki

@manojbaishya True, there is also xbanish if you want to hide the cursor on keyboard input.

@basaran very useful suggestion!!!!!!! This problem make me annoyed for a long time, my God.

Please add this feature to vscode. I use a large, coloured mouse pointer and it effectively hides multiple characters, typically exactly where I want to type. I tried the banish-pointer extension but it somehow corrupts the non-standard pointer.

hello, I use a system level software to do this. And it works well.
https://www.softwareok.com/?seite=Microsoft/AutoHideMouseCursor

Hi @dzgraphbed1 , in fact I tested that too and I was wrong in my previous comment, it is the AutoHideMouseCursor that corrupts the mouse pointer.  I prefer an integrated solution, and only in VSCode.
Apologies to @Gruntfuggly , the banish-pointer extension works fine on VSCode, I just got these two solutions mixed up when testing them.

Add this feature pls

100500

Most Windows apps like Notepad, WordPad, Windows Terminal and even Paint do respect this mouse option:

Why VSCode doesn't? 😭

I would like to see this feature in VSCode. It seems that every text editor has this functionality, and it appears to be a standard feature in these types of applications.

OH boy this is still open, 7 years later

Most Windows apps like Notepad, WordPad, Windows Terminal and even Paint do respect this mouse option:
(image)
Why VSCode doesn't? 😭
@alexdima any chance this will be enabled?

THIS! I was literally going to post this too (with the same screenshot, even!). It's crazy to me that Microsoft don't respect their own Operating System's design! This superfluous setting which has (afaicr) never worked should be removed, or better yet, implemented system-wide, and across all of Microsoft's software. This kind of thing should just be standard.
Slightly unrelated, but It's about time somebody goes through Windows with a fine-tooth comb and makes sure that there aren't all these loose ends from deprecated or never-implemented features lying around all over the place (and while they're at it, get rid of that ERROR_SUCCESS malarkey, too! What an oxymoron!)


Create a GIF to showcase the project features

GIF is very useful in demoing the project feature.
https://github.blog/2018-06-29-gif-that-keeps-on-gifing/
Create one for gitnoter

Done


Fix KMeans++ initialisation slowness

Reference issue

Relates to: #11877
What does this implement/fix?

The squared distances were computed in Python in a suboptimal way with a
complexity of O(n²). It took the overall running time of the function.
They are now computed using  cdist, making the overall algorithm fast.

Using the snippet provided by @Gabriel-p in #11877:
import numpy as np
from sklearn.cluster import KMeans
from scipy.cluster.vq import kmeans2
import time as t

N = 1000
data = np.random.uniform(0., 1., (N, 4))

n_clusters = 50

s = t.time()
centroid, labels1 = kmeans2(
    data, n_clusters, minit='++', iter=10, thresh=1.e-4)
print("scipy:", t.time() - s)

s = t.time()
# KMeans
model = KMeans(n_clusters=n_clusters, init='k-means++', n_init=10, tol=1.e-4,
    max_iter=1000, verbose=0)
model.fit(data)
labels2 = model.labels_
print("sklearn", t.time() - s)

We previously got the following running times:
scipy: 3.926667865371704
sklearn 0.741915225982666

With this fix, we now get:
scipy: 0.016803741455078125
sklearn 0.741915225982666

Awesome work @jjerphan!

Shall I add some tests in for this issue?

Thanks @jjerphan. This code already is covered by tests (I assume, would be good to check). Extending the benchmark in benchmarks/benchmarks/cluster.py to include the init keywords would be nice to make sure we detect future performance regressions.

This code already is covered by tests (I assume, would be good to check).

Yes, there is one test for the initialisation:
https://github.com/scipy/scipy/blob/787efe4d2829a1f9c177aa368e9d73202ee044e4/scipy/cluster/tests/test_vq.py#L245-L259
and several to test reproducible results.

@rgommers: I've separated the benchmark in two classes as only kmeans2 support different initialisation with init. Let me know if this makes sense or if I should change it.

LGTM, merged. Thanks @jjerphan!

@jjerphan I just added to the release notes "Initialization of scipy.cluster.vq.kmeans2 using minit="++" is now much faster". Please edit as you see fit!

Thank you @mdhaber : I've just added a few more words to what you've written.


fix(configs): token validation logic

Closes #2167
📝 Description
Fix token validation logic to check value property recursively.
⛳ Current behavior (updates)
{
  ...
  theme: {
    tokens: {
      colors: {
        primary: {
          value: "#000",
          type: "color", // finalPath is not endWith ‘value'
          description: "The primary color", // finalPath is not endWith ‘value'
        },
        secondary: { value: "#fff" },
        tertiary: {
          value: "#ddd",
          type: "color",
          description: "gray.100", // finalPath is not endWith ‘value'
          extensions: {
            category: "colors", // finalPath is not endWith ‘value'
            type: "primitive", // finalPath is not endWith ‘value'
          },
        },
      },
      ...
    },
    ...
  },
  ...
};

Validating this token outputs error.
🚀 New behavior
No error occurred.
💣 Is this a breaking change (Yes/No):

No
📝 Additional Information

thanks for refactoring!!


Add user journey view for end-to-end SLO reporting

Summary
SLOs are defined at the level of individual systems or services (e.g. a single entity in NR). This is useful for services operators to understand how their service is behaving, but doesn't reflect the end user experience when services are chained together in deep dependency trees.
Desired Behaviour
I would like a user journey view that allows individual SLOs to be associated with a user journey such that I can quickly see and understand:

What are all the services/SLOs required for users to complete a given journey
Which services/SLOs are the weak links that have the biggest impact on the user experience
Are those weak links weak because they are failing to attain their SLO, or because their SLO is set too low in the first place?

As an executive, I want the ability to see the status of my critical user journeys at a glance, whether I'm meeting my across the board, and if not, which service owner I need to talk to.
As a service owner or operator, I want to understand how my dependencies might be negatively impacting my SLOs, and how I might be negatively impacting my upstream services SLOs so I can take corrective action.
Possible Solution
Blameless.com has a basic implementation of this feature. We might be able to redesign the composite view to have a) saved views, b) funnel view of SLOs, and c) lowest attainment, rather than average.

Additional context
We have deep dependency chains. Front end services that are accessed by users with their browser may go half a dozen "hops" into the backend, and the backend services have their own dependencies. These "hops" often cross organizational boundaries and technology stacks, making it difficult to identify the constraint that needs to be improved for overall system performance to improve.
At minimum, I would like a view that makes it obvious how a particular business process functions and how well it is working, but future enhancements might also include a way to "walk the graph" of services/SLOs based on defined relationships.
@khpeet

@andrewseling - do you currently use the "Combine SLOs" view at all? If not, that can probably be repurposed for this case.

I do not, but I agree that it would probably work for this use case with a few upgrades.

From: Keagan Peet<EMAIL_ADDRESS>Sent: Wednesday, October 14, 2020 8:17:42 AM
To: newrelic/nr1-slo-r<EMAIL_ADDRESS>Cc: Andrew Seling<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [newrelic/nr1-slo-r] Add user journey view for end-to-end SLO reporting (#124)
@andrewselinghttps://github.com/andrewseling - do you currently use the "Combine SLOs" view at all? If not, that can probably be repurposed for this case.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/newrelic/nr1-slo-r/issues/124#issuecomment-708362605, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AGHIBOKFZPAZDMPQTFMNMW3SKWJGNANCNFSM4SNCAM4A.

can be closed. Added in https://github.com/newrelic/nr1-slo-r/pull/133


Failed to find configured root

I am trying to use a different line than the default
<files-path name="internal_pictures_files" path="Pictures/" />
to something like
<files-path name="internal_pictures_files" path="images/" />
and it causes app crash and subsequent relaunch.  How should I really use to save to different directory.
FYI: I am a novice.

Hello there, @gnanakeethan. Thanks for giving Shutter a try!
It is no surprise that your app is crashing when you change: <files-path name="internal_pictures_files" path="Pictures/" /> to <files-path name="internal_pictures_files" path="images/" />.
This is because Shutter is hard coded into saving pictures in the Pictures/ directory. Because you are using images/, Shutter throws an error as it can no longer save into the Pictures/ directory.
If you don't mind me asking, why would you like to use the directory images/ instead of the default Pictures/?

Quite right. I actually didn't know that earlier. I actually wanted to save pictures took with different actions (button clicks) to different locations.
Anyhow thanks for clarification. Is it possible to implement? I shall try to contribute if possible.

Quite right. I actually didn't know that earlier. I actually wanted to save pictures took with different actions (button clicks) to different locations.
I never thought about that. Tell me more about why you want to save pictures in different locations. Or, would you be ok with having all of the images saved to 1 location, but they are named a different file name?
I am asking you questions to get a better idea of what you are looking for. Then we can talk about how to implement this into Shutter.
The more information you can tell me about your use case the better help it is to me.

@levibostian I am currently sorting out that issue. Actually it would have been easier if I store images in multiple places. The use case is that, I am building an app in which there are proof of pickup/delivery photos needed in case there is no one to digitally sign using (SignaturePad).
I thought to place the photos sorted by the ID in a specific directory so that I can list them all and check with remote and push if necessary.
Right now using a "_" separator and adding a suffix to the name.  👍

Ah, ok.
I can see how having the ability to store a photo in a specific directory would be helpful.
I created a new issue about adding this feature to the library. If you want, @gnanakeethan, feel free to take that issue and make a pull request to add the ability 😄.
I will close this issue as it has been solved. We can talk about adding the directory feature in this new issue from here on out.
Thanks, @gnanakeethan!


CE-682 - Mobile Refactor - Complaints List

Description
Refactored base layout and the majority of the components hosted in the complaints page to support iPad and eventually smaller hand-held devices. Most of the changes in this PR are limited to the front-end/styling, however a few minor additional pieces were included to support mobile viewports.

Scroll behavior for the complaint table interface updated to target the scrollable containing element.
Leveraged OOTB functionality from React Bootstrap to support filters display on mobile.

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[x] Manually testing the interfaces across Chrome, Firefox, IE and Safari (iOS device)
[x] Updated tests to support some changed naming conventions on some components

Checklist

[x] New and existing unit tests pass locally with my changes

Further comments
There wasn't a small piece-meal solution for this. As mentioned in the description, most of the components on the complaints page, as well as the base layout needed to be adjusted to support smaller devices.
I've tried to make each commit succinct to a given component/area of the application to alleviate review.

Thanks for the PR!
Any successful deployments (not always required) will be available below.
Backend available
Frontend available
Once merged, code will be promoted and handed off to following workflow run.
Main Merge Workflow

Thanks for the PR!
Any successful deployments (not always required) will be available below.
Backend available
Frontend available
Once merged, code will be promoted and handed off to following workflow run.
Main Merge Workflow

Thanks for the PR!
Any successful deployments (not always required) will be available below.
Backend available
Frontend available
Once merged, code will be promoted and handed off to following workflow run.
Main Merge Workflow

@jeznorth - A couple of things:

The map list seems to have dropped off the messaging when there are complaints that can't be mapped

The alignment of the icon in the assign modal seems to be off and there is some weird styling when hovering over the logged in officer

Should the create button have an image to be consistent with the details page?

@afwilcox @jeznorth I have take a look at those cypress test under your branch and there're some findings:

✅ change-complaint-assignee-details-view.cy.ts: has a specific name for the Test Account. If I remove this, the test's successful.
cy.get("#comp-details-assigned-officer-name-text-id").contains("ENV TestAcct").should("exist");
❌ change-complaint-status.cy.ts: "tt-status-icon" is being removed so that's why the test failed
cy.get("svg.tt-status-icon").filter(":visible").click({ force: true });
🟡 complaint-attachments.cy.ts: sometimes the carousel is being clipped, so it's half visible on the screen. Hence, if I remove be.visible then the test is fine.
cy.get("div.carousel.coms-carousel").should("exist").and("be.visible");
❌ zone-at-a-glance-header.cy.ts: "comp-sub-header" is being removed so that's why the test failed.
cy.get("#root > div > div.comp-main-content > div.comp-sub-header").contains("Complaints");
✅ sticky-headers.cy.ts: success to me
❌ zone-at-a-glance-setup.cy.ts: same reason as "zone-at-a-glance-header.cy.ts"


IllegalStateException no outbox_state entry for message

Checklist

[X] I have used the search function to see if someone else has already submitted the same bug report.
[X] I will describe the problem with as much detail as possible.

App version
6.507
Where did you get the app from?
F-Droid
Android version
12
Device model
OnePlus Nord
Steps to reproduce
Unfortunately I don't know, I just updated K-9 from 6.506 to 6.507 and then suddenly I received the message:
Nachrichten konnten nicht gesendet werden IllegalStateException: No outbox_state entry for message with id xyz
I have three different mailboxes configured, so I do not know to which account this message refers to. I also saw that the message id the errer refers to is changing. So there seems to be a general issue I am not able to resolve.
I found two older issues regarding this message, but wasn't able to find a resolution.
I used K-9 for many years now and never had any issues, unfortunately I do not know how to track the issue down, since I dont know which account (of the configured 3 mailboxes) this is refering to.
I am able to send and receive emails via K-9 with all three configured accounts.
I just can't get rid of this IllegalStateException messages.
I seems I am no longer able to delete a message from my mailbox. It disappears from the Inbox for a short time and then reappears again in the Inbox.
The issue seems to start as I started to switch into other folders like: 'send' and 'trash' and refreshed the folder and also scrolling down and click the 'bis zu 50 weitere abrufen' button.
Any help would be much appreciated.
Thanks in advance.
Expected behavior
Deleted messages should be moved to 'trash' folder and should not reappear in Inbox
Actual behavior
Deleted messages won't be moved to trash
Logs
No response

I just saw that in the Folder settings for one account there is a total mess for the folder assignments, I copied this out of the current backup:
        <value key="autoExpandFolderName">Entwürfe</value>
        <value key="archiveFolderName">Papierkorb</value>
        <value key="draftsFolderName">Gesendet</value>
        <value key="sentFolderName">PSN</value>
        <value key="spamFolderName">Rechnung</value>
        <value key="trashFolderName">Urlaub</value>

I never changed the assignment by myself , but I saw that Archive is assigned to Papierkorb (trash) and Drafts to Gesendet(Sent) etc.
I corrected this with hope that this will fix the issue, but unfortunately the issue is also happening with correct folder assignments.

There's a good chance there was a data corruption somewhere. The safest and probably fastest way to get rid of this issue is to uninstall the app, then reinstall and import settings.

Thank you very much for the fast response. Your recommendation fixed the issue. I just found an old backup from 2022 where the folder assignment were correct:
        <value key="autoExpandFolderName">INBOX</value>
        <value key="archiveFolderName">Archiv</value>
        <value key="draftsFolderName">Entwürfe</value>
        <value key="sentFolderName">Gesendet</value>
        <value key="spamFolderName">Spam</value>
        <value key="trashFolderName">Papierkorb</value>

So when did you think could the data corruption happend? Could this be at the time updating from 6.506 to 6.507?
Is there a chance to break things link that?

I don't think it's a bug in the app. We'd get more reports if that was the case. Plus, we haven't changed that part of the code in a while.
Random memory and/or storage corruption is something that happens regularly given the huge number of mobile devices and the different environmental conditions they are operated in. Given that there's a near-infinite number of broken states, it's incredibly hard to identify what exactly went wrong and then fix the data (and possibly code) on the affected device to get to a non-broken state. Uninstalling, then reinstalling the app usually does the trick. Trying to find out what went wrong is usually futile.

Just as a note: Galaxy S23 Ultra - Android 14, version 6.714 (probably updated last night automatically). Same error, I'll reinstall the app.


Corrected Class Name Associated with Unit Test

OCD: TheGetExtensionProperty test -> TheGetExtensionMethod test

@daveaglick ha ha, well spotted!

@daveaglick I have no issues with pulling this in, just one quick suggestion...
Can you change the commit message to provide some further information about what changed, and why?  That way, when we are looking back at the history, we know what was done.
For instance, I would suggest something like this:
`(maint) Changed class name

To clearly indicate the situation under test
The previous class name suggested it was the GetExtensionProperty, but it was GetExtensionMethod`

What do you think?

:thumbsup: No problem - hopefully the new message is good.

@daveaglick thank you for your contribution! :+1:

Lol - I gotta tell you, this one was really tough :smile:

Ha ha, everything counts :smile_cat:


Devise a method for normalizing storage of different units in a packed Long

Similar to how Duration stores a value in either milliseconds or nanoseconds.

This will be particularly helpful for Area and Volume. Their scales are much more steep than other SI units' -- Area scales by 1,000,000 between each order of magnitude, for example. Sacrificing 2 bits of storage capacity allows us to model the following for area with gradually degrading precision:

2.3km² at μm² precision
2.3Mm² at mm² precision
230Mm² at cm² precision
2.3Gm² at m² precision