Remove comment and fix {live,readi}ness path to '/graph'

Remove comment and fix {live,readi}ness path to /graph on servicegraph Deployment template yaml.
This is workaround because servicegraph haven't the health check path.
But the ingress-gce needs the readinessProbe path that returns 200 status, so we should get 'generic JSON serialization' body and 200(http.StatusOK) status.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: sbezverk
Assign the PR to them by writing /assign @sbezverk in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

install/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

ref: #5435

/cc @ymesika

/ok-to-test

@zchee: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
87d6257793e973add279796cb7a75b693bf29cc5
link
/test istio-pilot-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@zchee: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
87d6257793e973add279796cb7a75b693bf29cc5
link
/test istio-pilot-e2e

prow/e2e-simpleTests.sh
87d6257793e973add279796cb7a75b693bf29cc5
link
/test e2e-simple

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@zchee: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
87d6257793e973add279796cb7a75b693bf29cc5
link
/test istio-pilot-e2e

prow/e2e-simpleTests.sh
87d6257793e973add279796cb7a75b693bf29cc5
link
/test e2e-simple

prow/e2e-bookInfoTests-v1alpha3.sh
87d6257793e973add279796cb7a75b693bf29cc5
link
/test e2e-bookInfo-envoyv2-v1alpha3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@zchee: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
87d6257793e973add279796cb7a75b693bf29cc5
link
/test istio-pilot-e2e

prow/e2e-simpleTests.sh
87d6257793e973add279796cb7a75b693bf29cc5
link
/test e2e-simple

prow/e2e-bookInfoTests-v1alpha3.sh
87d6257793e973add279796cb7a75b693bf29cc5
link
/test e2e-bookInfo-envoyv2-v1alpha3

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
87d6257793e973add279796cb7a75b693bf29cc5
link
/test istio-pilot-e2e-envoyv2-v1alpha3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ymesika ping

/retest

@ymesika Yep.
/ endpoint returns 404, will UNHEALTHY status on ingress-gce.
/dotviz endpoint is better than /graph?
Anyway, I think change readiness.path to any endpoint that certaily returns 200 status.

I guess /graph is good enough to re-enable the probes.
Let's wait for the required tests to pass so we can proceed with this PR. Thanks.

I see. Thanks.
BTW, just off-topic, istio maintainer not use ingress-gce? developing uses ingress-nginx?

Not sure I can answer it.
I saw relevant discussions here: #3469 and #3527

@ymesika Thanks polite response and references!! I'll read those issues.

@zchee: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
87d6257793e973add279796cb7a75b693bf29cc5
link
/test istio-pilot-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/approve

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: ymesika, zchee
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: sbezverk
Assign the PR to them by writing /assign @sbezverk in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

install/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/assign @linsun


Add logout button to /private view

Allow user of heroku app to log out.
when they attempt to re-visit the /private content auth should fail.

Done.


Attester slashing 5212

See https://medalla.beaconcha.in/validator/5212
(no blocks proposed, so graffiti not known)

They made these additional deposits https://medalla.beaconcha.in/validators/eth1deposits?q=0x89088e8559cb48887e3f496eAeA6988502944f22
No other blocks proposed for those validators yet either

This user is "AlexSSD7#8774 in Discord" on discord and probably the same @AlexSSD7 from github.
@AlexSSD7 can you help us understand your setup and any details that might have led to this slashing?

It is a Prysmatic Labs' Goerli faucet address. Probably just copied the wrong one and haven't noticed. xD

Sorry, scratch the above.
It's discord user baconBeacon

I just dm'ed baconBeacon to learn more

I spoke with baconBeacon Here's the summary:
It was a teku client running with 1 validator. It did not appear to crash or hang. Commands used to launch:
./teku --network=medalla --eth1-endpoint=http://localhost:8545/ --validators-files=keystore-m_....json --validators-key-password-files=tekupassword.txt --rest-api-enabled=true --rest-api-docs-enabled=true --metrics-enabled --validators-graffiti="poap..."

Logs: teku.zip
It's worth noting that this slashed key was rescued over from a lodestar client. I'm confirming whether teku and lodestar were both running at the same time

The logs look completely normal around slot 688 where they were slashed.  It's not near a restart point and looks like the chain is progressing forward as you'd expect.

Who was running the lodestar? Can we check whether it was still running at that point?

From baconBeacon

Hey, this was a teku client running with 1 validator against a local geth node. Not sure why it was slashed. It did not appear to crash or hang.  ./teku --network=medalla --eth1-endpoint=http://localhost:8545/ --validators-files=keystore-m_....json --validators-key-password-files=tekupassword.txt --rest-api-enabled=true --rest-api-docs-enabled=true --metrics-enabled --validators-graffiti="poap..."
On the same system, prysm and lighthouse are running fine.
Nim and loadstar haven't been able to keep synchronized.

baconBeacon's teku logs
https://gist.github.com/djrtwo/f7c176dd4dcf7be44b58d7eaaee33b43

djrtwoToday at 10:07 AM
Might you be running the same keys on multiple clients? Or are you sure the keys are fully isolated?
baconBeaconToday at 1:56 PM
I don't believe so. I checked the imported keystore folders. They are different validator public keys.


benchmarks

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 2 (0.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-2.3%) to 96.471%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/foapy/order.py
0
2
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-2.3%

Covered Lines:
65

Relevant Lines:
68

💛  - Coveralls


Update collapsed header state on Desktop Trading Page

All collapsed states can be found next to the open states in the Trading Page Design

Animation should be smooth, quick with subtle ease. Message me if you have questions regarding the transition.

I believe @matt-bullock has already reviewed this

I'm not seeing any updates here

Looks good. One request is if the rotation of the arrow could finish at the same time the open/close transition finishes. When closing the header the arrow is still moving after the header closed which is a little distracting.

pushing this last part to 1.14


many tiles are being produced with no data

Two things at play here:

the GeoJSON files derived from the ground-truth datasets are coarse, which causes rough cuts + strange shapes in the resulting clipped image from Planet
high-resolution tiling (zoom 15) simply doesn't capture many pixels, so higher probability of seeing all-0 tiles than perhaps a smaller zoom level

Likely solution is to just not write tiles where (tile == 0/nodata).all() == True

fixed in https://github.com/acannistra/planet-snowcover/commit/d0f3c1401dd8211edf90d44e79dc9027b69cc35d


Preview not working in LUbuntu

I'm running LUbuntu 15.04. Preview does not work with the atom that I downloaded. Which would be the newest .deb release from the website.

Same as Archlinux.

Any hint to a solution?

@roppert , Thanks. It works well.

I'm afraid this is my bad, I wasn't able to test my PR in Linux, sorry.
@vedmalex's solution (#83) looks like a good fix, and is how I should have done it in the first place.


Update Chainguard source

This contains both Chainguard and wolfi.

This will leave existing records orphaned, right?

Yeah, those will have to be cleaned up manually I think.

Is there anything that I can do to help?

Is there anything that I can do to help?


Custom Household Naming and Greeting interface would benefit from being subclassed instead

I have a scenario where I would like to write code to override the formal and informal greeting automation, but I would like to keep the household naming logic as the default. With the interface implementation, all the methods have to be rewritten. It would be nice if we could subclass the default naming class and override only the methods we care to implement.

Hi @dhallsim
We don't provide support via GitHub issues but you are welcome to create an Idea or submit a pull request if you have suggestions for how you could make this better.


Update documentation to reflect the addition of default endpoints

Most of the documentation updates will happen in the swagger.json file which I'm not touching with a ten foot pole.


Variable precision / DGD support

@jorisbontje:

Required for supporting the DGD token, which has 9 decimals instead of the 18 decimals as used by ETH, DAI and MKR.

(ported from  Variable precision / DGD support #130 )

@nmushegian:

If you can bear reading the token standard thread, there's some discussion about "precision" / "decimals" and how to handle that metadata

https://github.com/ethereum/EIPs/issues/20


Doc/heroku

Solid start @raksonibs, thanks! Instead of having the Procfile already in there, I'd rather the doc explains how to add one. Not everybody has a need to deploy to Heroku, so having that there by default seems like a lot of unnecessary file space.
Two other small nitpicks and then we can merge this! Thanks so much!
/cc @grabbou who might have some input about this

Awesome, the doc looks perfect now!
Could you rebase onto master, resolve the conflicts and then we'll land this, thanks @raksonibs :fire:

Hey @mxstbr, I don't have write access to the repo however, so I can't actually push my merge!

Did it myself, I realized that it's actually a bit more involved because we changed the commands recently. Thanks @raksonibs, this is awesome! :fire:

np:) will work on other issues soon. Got a list of features anywhere?

Right up there, the "Issues" tab. :wink:


Where the data is converted into tensor

Like this:
ta = tf.convert_to_tensor(np.array([1,2,3]))
with tf.Session() as sess:
         result= sess.run(ta)

I want to know where the list is converted to tensor.
Is in GPU, or in CPU and then being feeded to GPU ?

Thank you for your post. We noticed you have not filled out the following field in the issue template. Could you update them if they are relevant in your case, or leave them as N/A? Thanks.
Have I written custom code
OS Platform and Distribution
TensorFlow installed from
TensorFlow version
Bazel version
CUDA/cuDNN version
GPU model and memory
Exact command to reproduce

It has been 15 days with no activity and the awaiting response label was assigned. Is this still an issue?


0.25.3 -> 0.26.0 column_* api removed from Row

See docs.rs/rusqlite/0.25.3 vs docs.rs/rusqlite/0.26.0.
Was this intentional without a deprecation warning?

Awesome, I can use that in the mean time.


fp16 preprocessing

7.1 has fp32 <-> int8 preprocessing conversion as untimed. We would like to add fp16 also.

SGTM

change 'May quantize image data from fp32 to int8 and between signed and unsigned' to 'may convert among all the whitelisted data formats' @jveejay

PR merged.


Need custom data onChange of value

I am using react-picky in loop and displaying multiple dropdowns using package, my requirement is i need custom data onChange of value so i can use it to make generic handleOnChange method. Please suggest how can i get custom data related to dropdown

@Puneetbh I'll need an example, I don't get what you mean.

i have array of objects and with loop i am displaying react-picky dropdown by passing options, now i am passing common generic onchange method to in onChange property, inside that i want to know which dropdon valud changed so i can update state accordingly
Like if country changed by user i want to know user changed country then i will update state of country so want to get some additional data whenever user making selection in dropdown

@Aidurber  i got the solution, Thanks

Hi Aidurber,
I have recently downloaded the picky and have used it inside a map function to loop through my dynamic data. In the picky i have set multiselect as true.
How ever on the onchange function am getting only the currently selected value and not the list of values selected for picky.
This is the sample code:
{
sectorsData.map((selectItem, i) => (

))}
in the on change function  i have written the following code :
const selectedOption = (name,value) =>{
setAssetsAddedState(prev => { return { ...prev, [name] : value}});
}
The issue am facing is in the case of multiselect where the value will be a single object containing the current selected item rather than the list of items selected. Could you please help me with the issue. I have tried every possible solution and it does not seem to work.Any help would be really appreciated as am stuck in the issue for sometime.
Where a sample sectorsData:
[
{
"name": "Technology",
"options": [
{ "id": "1",
"name": "AI & Analytics"
},
{
"id": "2",
"name": "Robotics"
},
{
"id": "3",
"name": "IoT"
}
]
},
{
"name": "Sector",
"options": [
{
"id": "1",
"name": "Automotive"
},
{
"id": "2",
"name": "Oil and gas"
}
]
}
]


does not support the stream_option parameter

changelog
parameter does not support the stream_option parameter added in the new version of openai.

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

ID: fadd6778-b11c-d9af-1bd9-e3ed394e0fcb
Version Independent ID: 45436c53-4aae-284b-a6c5-3f47c9342537
Content: Azure OpenAI Service REST API reference - Azure OpenAI
Content Source: articles/ai-services/openai/reference.md
Service: azure-ai-openai
GitHub Login: @mrbullwinkle
Microsoft Alias: mbullwin

@supuwoerc
Thanks for your feedback! We will investigate and update as appropriate.

@supuwoerc
Thanks for your feedback! I've assigned this issue to the author who will investigate and update as appropriate.

Thanks @supuwoerc. You are correct the latest Azure OpenAI preview API does not currently support the new stream_options parameter. Once support is added in a later preview API release we will update the docs accordingly.

#please-close

this is closed. was stream_options added?

this is closed. was stream_options added?
No not yet, we will update the docs when it is added. GitHub Issues will no longer be the feedback mechanism for Azure docs on learn.microsoft.com so unfortunately all issues have to be closed out due to the retirement.


Change tab label to be easy to reference and understand

As a young person I want to be able to easily understand where to look in the Editor for my input and output code.
AKA at the moment the tab name = output, but in some cases for some projects it's an input and an output.
Common terminology used in Editors (terminal, console, shell) are often geared towards adults, and are not necc. very easy for a young person to understand.
Ideally as part of this issue we want to:

[ ] check what other Editors with similar key audiences use (MVP)
[ ] agree a label to use with Tracy that we think is short, but easy to understand and reference (MVP)
[ ] is possible have an icon so there is an additional visual indicator of where we mean that can be referenced in instructional start code/project step by steps (e.g if we supporting Google fonts we could pick an icon from there)

I think this is going to be covered by the html/css and redesign work Max is looking at so marking as closed as covered under separate design work.


Fix a typo in Text Detector

According to what characters are encoded in UTF8, we should count the number of bytes from 0xf8 to 0x100 instead of 0xf80 to 0x100 which is apparently a typo.

Thanks for your contribution to Apache Tika! Your help is appreciated!
Before opening the pull request, please verify that

there is an open issue on the Tika issue tracker which describes the problem or the improvement. We cannot accept pull requests without an issue because the change wouldn't be listed in the release notes.
the issue ID (TIKA-XXXX)

is referenced in the title of the pull request
and placed in front of your commit messages surrounded by square brackets ([TIKA-XXXX] Issue or pull request title)

commits are squashed into a single one (or few commits for larger changes)
Tika is successfully built and unit tests pass by running mvn clean test
there should be no conflicts when merging the pull request branch into the recent main branch. If there are conflicts, please try to rebase the pull request branch on top of a freshly pulled main branch
if you add new module that downstream users will depend upon add it to relevant group in tika-bom/pom.xml.

We will be able to faster integrate your pull request if these conditions are met. If you have any questions how to fix your problem or about using Tika in general, please sign up for the Tika mailing list. Thanks!

This looks like it goes nearly all the way back: https://svn.apache.org/viewvc/tika/trunk/tika-core/src/main/java/org/apache/tika/detect/TextStatistics.java?revision=1367991&view=markup#l79

Thank you @thebsssss


Map seems to reset zoom level every 25 seconds

When I use my map for more than 25 seconds, the map will jump and zoom back out to the "Start" coordinates. This makes me lose my place when looking across large geographical areas.
The only "Map In" event I have set is "Connect" and I am on the latest plugin version 2.22.2. Is there anything I can do to avoid the map resetting itself? This is on Google Chrome. I can attach a video if needed as well.

are you able to share your flow here ? Is the map set to auto pan to points or locked to position ?
Have you checked all messages going to  the map to see if they have any extra commands in ?

Here is my flow for the map. It uses a simple JSON DB node to get the payload data. I can send you a dev version of the actual map I'm using on the web so you can see the behavior later today or tomorrow.
Here my map node:

Heres the Map In node -

{
    "id": "26ec8c9037ba430d",
    "type": "DataOut",
    "z": "b4afb7c4b5e96840",
    "collection": "e8195ef9d497d22c",
    "name": "",
    "topic": "/backlog",
    "error": true,
    "x": 2600,
    "y": 2060,
    "wires": [
        [
            "a76ffa9d0c3a756f"
        ]
    ]
},
{
    "id": "46b52538b96149c4",
    "type": "change",
    "z": "b4afb7c4b5e96840",
    "name": "",
    "rules": [
        {
            "t": "set",
            "p": "datapath",
            "pt": "msg",
            "to": "/",
            "tot": "str"
        }
    ],
    "action": "",
    "property": "",
    "from": "",
    "to": "",
    "reg": false,
    "x": 2410,
    "y": 2060,
    "wires": [
        [
            "26ec8c9037ba430d"
        ]
    ]
},
{
    "id": "a76ffa9d0c3a756f",
    "type": "split",
    "z": "b4afb7c4b5e96840",
    "name": "",
    "splt": "\\n",
    "spltType": "str",
    "arraySplt": 1,
    "arraySpltType": "len",
    "stream": false,
    "addname": "",
    "x": 2750,
    "y": 2060,
    "wires": [
        [
            "8c0c06b85d4cfd1d"
        ]
    ]
},
{
    "id": "d71291081e035d11",
    "type": "worldmap in",
    "z": "b4afb7c4b5e96840",
    "name": "",
    "path": "/backlog",
    "events": "connect",
    "x": 2130,
    "y": 2060,
    "wires": [
        [
            "46b52538b96149c4"
        ]
    ]
},
{
    "id": "8c0c06b85d4cfd1d",
    "type": "worldmap",
    "z": "b4afb7c4b5e96840",
    "name": "",
    "lat": "44.3458545",
    "lon": "-71.4627607",
    "zoom": "9",
    "layer": "OSMG",
    "cluster": "",
    "maxage": "",
    "usermenu": "show",
    "layers": "show",
    "panit": "false",
    "panlock": "false",
    "zoomlock": "false",
    "hiderightclick": "false",
    "coords": "none",
    "showgrid": "false",
    "allowFileDrop": "false",
    "path": "/backlog",
    "overlist": "DR,CO,RA,DN,HM",
    "maplist": "OSMG,OSMC,EsriC,EsriS,EsriT,EsriDG,UKOS,SW",
    "mapname": "",
    "mapurl": "",
    "mapopt": "",
    "mapwms": false,
    "x": 2900,
    "y": 2060,
    "wires": []
},
{
    "id": "e8195ef9d497d22c",
    "type": "json-db-collection",
    "name": "",
    "collection": "backlog",
    "save": true
}

]`

ok - looks ok so far...
Can you add a debug node just after the database set to send data to the console.log and then cut/paste and example of the data you get back (Between      so I can import it (not a picture). Thanks

After watching the full msg object debug node and my map page, I noticed what seems to be the issue. It seems the "sessions" are refreshing every 25 seconds, each time the exact same but with a new "session id". Ideally, each session could be extended to keep the same session ID for a custom time like 10 minutes/1 hour/etc.

Here is the copied msg object. You can also see the example of the zoom out here on this dev clone I set up (I removed all customer PII data): https://dev-nodered.netafyengine.com/backlog/
I can also give you access to the Node-RED login as well if it'll help
backlog : msg : Object object payload: array[82] [0 … 9] [10 … 19] [20 … 29] [30 … 39] [40 … 49] [50 … 59] [60 … 69] [70 … 79] [80 … 81] topic: "backlog" _sessionid: "5934ee0d-144c-4efb-83cf-19b2aac71f93" _sessionip: "<IP_ADDRESS>, <IP_ADDRESS>" _msgid: "da8f54e1c8859856"

This is most odd - I can only think it is something that your hosting provider is doing to cause it to drop the connection every 25 seconds... certainly there is no such timeout in the code that I know of.

Interesting. In this case, I am running Node-RED on Google Kubernetes Engine. I will move my NR Map server to a consistent host VM on Google Compute and see if that resolves the issue.

I ended up figuring out the solution here - For people hosting Node-RED on GKE Kubernetes, you need to follow the steps here- https://cloud.google.com/kubernetes-engine/docs/how-to/ingress-features#1.16-gke.3+ to change from the default 30-second refresh.
You must create a "BackendConfig" object and attach it to the Node-RED service, and within the BackendConfig object, set the timeout limit to be higher than 30 seconds. 30 seconds is the default setting for a refresh if no BackendConfig is set up. This only really affects you if you're using Node-RED Map or Dashboard.

Interesting... wonder why they do that ? but hey - glad you fixed it


release: 0.2

Mostly improving speed, testability and architecture.

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


it is not a trait

For bug reporting only! If you're posting a feature request or discussion, please ignore.
Expected Behavior
Please describe the behavior you are expecting.
Current Behavior
Please describe the current behavior?
Failure Information
Please help provide information about the failure if this is a bug.
Steps to Reproduce
Please provide detailed steps for reproducing the issue.

Step 1
Step 2
...

Context
Please provide any relevant information about your setup. This is important in case the issue is not reproducible except for under certain conditions.

Package Version: vX.Y.Z
Laravel Version: vX.Y.Z
PHP Version: vX.Y.Z
Using a toolset (Docker, Laradock, Vagrant, etc.):

Failure Logs
Please include any relevant log snippets or files here.

I guess this issue can be safely closed.


Service principal & client secret

I can understand that a service princpal is created on the background to pull images from acr. But how the client id & secret is passed into AKS deployment yaml file  to pull the image from acr?

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

ID: 9180e07f-236a-20b4-1c38-66a1bad9c73d
Version Independent ID: 148ae764-76b7-da53-a897-b865e722946f
Content: Integrate Azure Container Registry with Azure Kubernetes Service
Content Source: articles/aks/cluster-container-registry-integration.md
Service: container-service
GitHub Login: @mlearned
Microsoft Alias: mlearned

@shankarsundaram The service principal which is created in the background is not only for pulling images. AKS will use the same service principal for creating the internal/external load balancer, additional disks etc.
Service principal creation is one step in the ARM template and that resource values are fed to the subsequent AKS creation step.
For ACR authentication, Secret is not used. Instead the flag --attach-acr in the command az acr update adds the role assignment.
This role assignment assigns the acr pull role for the service principal used by AKS.
After the role assignment, pods in AKS can pull the images.
You can assign this role manually as well.
Let me know if you need more info

Thanks I'm good.


Add an optional field conversationId in message to support querying messages sent by the sender and recipients in a conversation quickly (disabled by default)

Prerequisites: https://github.com/turms-im/turms/issues/924


robux counter and topbar buttons kind of broken

roblox seems to have made a site update that broke the robux counter and the buttons on the top left of the screen. theyre probably not meant to be looking like this

windows
firefox
<IP_ADDRESS>
1366x768

It should be fixed now

yeah it works now

although another thing i just noticed was that when you open the menu on the left the three lines button square doesnt go away

maybe it's fixed now?

yeah its pretty much fixed now i guess


dynamic assertions?

Hi, and thanks for supporting this library. I have experience with Zend ACL and was comparing this with it.
Does this support the ability for a permission to be determined dynamically? For example, how would I only allow a user with the "editor" role to edit a "document" resource only if he is the document creator?
https://docs.zendframework.com/zend-expressive-authorization-rbac/v1/dynamic-assertion/
Thanks!

Hello @scottybe
None of us is familiar with Zend ACL.
But answering your question I believe that the answer should be "Yes".
But I have no idea what would be the JS code for that.
Sorry. There is not much help you can get from us.


java.net.UnknownHostException: postgres - after migrating container to new host

I migrated all the docker volumes with the data and am in the process of migrating the containers - but can't get bitbucket to connect with the db.
I have a postgres docker container and a bitbucket app container both have
--network="bitbucket_default"
I added these three to the bitbucket container start up command
--env="DB_HOST=bitbucket_db" \
--env="PG_HOST=bitbucket_db" \
--env="DOCKER_WAIT_HOST=bitbucket_db" \

But they didn't suffice/fix the issue - Thanks
I can connect to the postgres container - using a small container with the psql client on the same network - so clearly I am just not passing the right ENV vars to bitbucket - can you help?
HERE IS THE Java (console) error - it's trying to talk to an host called postgres that does not exist..
JAVA start up error
org.postgresql.util.PSQLException: The connection attempt failed.
at org.postgresql.Driver$ConnectThread.getResult(Driver.java:401)
at org.postgresql.Driver.connect(Driver.java:259)
....
Caused by: java.net.UnknownHostException: postgres

Are you able to ping postgres in the bitbucket container??


Add class variable USE_UNIQUE_DUPLICATE_SUFFIX

Hello, I have the following use case:
a) I have an object with a string field (uuid) that is stored in a 36 character field.
b) I want to use django-clone to create copies of it (including the uuid), however the current implementation append the UNIQUE_DUPLICATE_SUFFIX field and hence the value is larger than the max field size and the save() call fails.
This PR adds a class variable USE_UNIQUE_DUPLICATE_SUFFIX to disable this behavior.

@all-contributors please add @andresp99999 for bugs.


Allow filtering/excluding a combination of event types

Is your feature request related to a problem? Please describe.

My specific issue is since we've started using signals, the "compact" view has become a lot more cluttered. What I want to see is "everything besides signals", but you can only filter out by 1 event type at a time. Also, the compact view shows child workflows but doesn't let you filter to those. You can only filter to child workflows in the history view.
Describe the solution you'd like

Ideally the workflow events/event type filters would show all types and allow selecting a combination of them. I think a typical filter pattern is to have check boxes next to each item, a "select all" button, and an "only" option next to each type. But really just checkboxes would solve this.
Describe alternatives you've considered

Additional context

https://github.com/temporalio/ui/pull/1806


Wrong value in config.son for grpc_tls parameter

SUMMARY
Upgrade from consul 13.x.x to 14.x.x failed due issue in config.json file
ISSUE TYPE

Bug Report

COMPONENT NAME
config.json
ANSIBLE VERSION

ansible [core 2.15.12]
executable location = /opt/homebrew/bin/ansible
  python version = 3.11.8 (main, Feb  6 2024, 21:21:21) [Clang 15.0.0 (clang-15<IP_ADDRESS>.5)]<EMAIL_ADDRESS>  jinja version = 3.1.

COLLECTION VERSION

No need it is role issue, wrong parameter reference
CONFIGURATION
n/a
OS / ENVIRONMENT

Ubuntu 22.04 and 18.04
STEPS TO REPRODUCE
Just run current role with default parameters with consul version 14.x.x

EXPECTED RESULTS

grpc_tls port has proper value
ACTUAL RESULTS

"ports": {
        "dns": 8600,
        "grpc": -1,
        "grpc_tls": "<IP_ADDRESS>",
        "http": 8500,
        "https": 8501,
        "serf_lan": 8301,
        "serf_wan": 8302,
        "server": 8300
    },

PR with fix https://github.com/ansible-collections/ansible-consul/pull/606


Fix size_t related compiler warnings

For Windows 64-bit build using MSVC, the fact that sizeof(long) != sizeof(size_t) would generate a fair bit compiler warnings.
Using size_t consistently is a better approach than using long and size_t interchangeably.
For printing size_t variables using printf(), It is also better to take advantage of C11 "%zd" if possible.

Coverage decreased (-0.6%) to 95.561% when pulling ec1b6318e4d31e8dc7c62ef5bf9b63e0379cdc4b on kiyolee:master into a740f15c17ef7956b1493d8d1f800896c1e60bad on akheron:master.

should submit PR with branch other than master.


[RFC] Configurable Staleness for Search queries

Current Behavior:
Currently, when a document is updated in a shard and a refresh occurs, the entire corresponding request cache is invalidated. This is done to ensure that the response a user receives will never be stale, regardless of whether the request goes through the cache or directly to the shards.
Challenge:
Even with Tiered Caching, where we can cache more heavily, we are still at the mercy of the refresh interval of the underlying index. If there are subsets of requests that are okay with a certain level of staleness, there is no way to achieve this without creating a separate index, which adds to the cost and complexity.
Use Case:
I am an OpenSearch admin in a cloud SaaS enterprise that manages customer support data. I provide OpenSearch as a service to my engineering teams. There is an index called "Tickets" that contains all the relevant data related to customer support tickets.
All the data in my OpenSearch cluster is multi-tenant, with an account_id field in every document that is used to filter queries.
There are two services making search queries to this index:
The "Views Search" service needs the most current data possible.
The "Dashboarding" service is okay with data that is up to 1 minute stale, as per their SLA, but they want the searches to be very fast.
Proposed Solution:
What if we expose a parameter called stale= (e.g., stale=30s) that a customer can include in their search query? This would signal to us that the data can be cached for the specified time period, regardless of any refreshes or document updates that may occur.
On each refresh, we would check if the cached query has exceeded the specified staleness threshold. If so, we would delete the entry from the cache and recompute the query, caching the new results for the next stale time period.
Benefits:
Cost Savings: Users can avoid creating multiple indices or setting up a separate data store (like Redis) for caching.
Improved Performance: More cache hits would lead to reduced compute requirements.
Flexibility: Customers can choose the appropriate staleness level for their use case, effectively prioritizing their queries.
Increased Product Stickiness: This solution would increase the reliance on OpenSearch, making it a more integral part of the overall system.
Related component
Search:Performance

[Triage - attendees 1 2 3 4 5 6 7 8 9 10]
@kiranprakash154 Thanks for creating this RFC, looking forward to seeing how this is addressed


Add missing space in CLI help text.

Before:
$ code --help
Visual Studio Code 1.92.0

Usage: code [options][paths...]

After:
$ code --help
Visual Studio Code 1.92.0

Usage: code [options] [paths...]

Hopefully ya'll don't mind fly-by PRs. I just saw this when I was looking for a flag in the help text and figured I could drop a quick patch.
I did not do any testing myself.


bug: custom element readonly prop always converted to readOnly

Related to https://github.com/solidjs/solid/issues/594 - basically you cannot have a custom element with readonly prop, as it is always converted to readOnly, which is required for input element, but readOnly is not part of HTMLElement spec, only HTMLInputElement.
https://playground.solidjs.com/anonymous/3385e30a-71ab-478f-8307-4752b1729eee
Aliasing readonly to readOnly should be made only for  tag.

I hear you. Right now all aliases are global so would take some rework

I can make a PR, including changes related to #211.

Fixed with #213


RBS Calculator

This PR adds RBS calculator functionality to poly.
What needs to be done:

[x] Add datasets.
[x] Add ability to generate folded mRNA structure for datasets.
[x] Add ability to generate potential values for the RBS model.
[ ] Add calculation functions needed for v1 RBS model.

[x] LinearFold (basic functionality complete in PR #146, code cleanup pending)
[x] Gibbs Free Energy (basic functionality complete in PR #161, code cleanup pending)
[ ] Co-folding

[ ] Add go notebooks to visualize effect of generated variables on actual protein translation initiation rate (needed to understand relationship between variable and output).
[ ] Add ability to 'curve-fit' and calculate coefficients for generated variables that will be used in RBS model.
[ ] Add 'model regression' ability that figures out the optimal set of parameters that lead to the highest correlated prediction results.
[ ] Finally, create a RBS v1 calc.
[ ] Create a workflow so that people interested in improving the RBS calc. model can easily test their own hypothesis (this will include refactoring code, documentation, etc.).

@v-raja how is this software going?

Alright, so this is a very important function, and I'd like to move this along. So here is what my plan is:

Just the calculation of numbers is sufficient (in comparison to making a model to fit for a particular protein). We can figure out more code at a later time point.
The workflow for improving the RBS calculator we can add at a later time point. Would make for a great PR.

What we want right now is high quality code committed to main branch. I'm going to start a review and we're gonna really push on this + synthesis fixer getting pushed through. Closing the other PRs for each individual function, we'll just do it all at once here.


Angular Meteor Collection API doc

Added the return value for the save as discussed in https://github.com/Urigo/angular-meteor/issues/383

Thanks @Tallyb !


Configuring hyperparameter space for Auto* Models

Description
As of now, users can pass a hyperparameter space using config argument which overrides the existing parameter grid. A new option update_config can be added so that users can make minor modifications in the default space.
Use case
Quite frequently the default hyperparameter space is good enough, but sometimes users may want to control it in a minor way. For example, user may want to fix the context length to a specific value or may want to restrict the number of layers. As of now, passing a dictionary corresponding to only that field is behaving as if that's the entire search space along with default values of other parameters, but users intention is to restrict only few specific dimensions in the default space.

To do what you want, you can access the default_config  attribute of the Auto* models, and make changes:
import AutoNHITS
config = AutoNHITS.default_config
config['learning_rate'] = tune.choice([1e-2, 1e-3])

This isn't clear from the documentation though, so we'll include a statement about this in the hyperparameter optimization guide.

Thank you.
For a interactive use, I think that is fine. Currently I am using these in automated pipelines through the sktime wrapper I added a few weeks back, where the model object itself is not exposed publicly to end users. So an ability to configure itself during initialization will be easier I feel.
This is the relevant adapter PR, if you want to take a look at the flow: https://github.com/sktime/sktime/pull/5962.

Do you have an example of such an automated pipeline? The linked PR doesn't talk about Auto* models and I don't see an example piece of code of an automated pipeline in that PR.

I have, but not readily as a public code but only for my office work.
The linked PR only introduces the wrapper, and users/developers are encouraged to define/contribute their own classes inheriting from it for different other models from neuralforecast.
This is a sample code for the general use of interfaced models:
# creating model instance configuring the hyperparameters
model = NeuralForecastRNN(freq="A-DEC", futr_exog_list=["ARMED", "POP"], max_steps=5)

# fitting the model
model.fit(y_train, X=X_train, fh=[1, 2, 3, 4])

# getting point predictions
model.predict(X=X_test)

Now, if it was NeuralForecastAutoRNN, as a user I may like to fix context length at 14, based on may be some specific data domain. I want to still search for best combinations of other hyperparameters, so I'd like to use:
NeuralforecastAutoRNN(update_config={"context_size": 14})
# or tune.choice([14]) if 14 causes any issues

As of now, users have to know that there's a private attribute _forecaster which they can configure themselves later. We can make it public, but I believe it goes against sklearn/sktime principle to modify public properties after definition, as model.config will no longer be in sync with model._forecaster.default_config.

Thanks - Weighing the pros and cons for now we will update the docs to demonstrate the ability to access the attribute to attain the default config.
For the sktime wrapper I'd also suggest to indicate it in the docs; it's fairly easy and the additional complexity with adding a potentially conflicting additional input (for example, what happens if update_config and config are both specified?) is something we would like to avoid.

updating the default config didn't worked for me I tried doing same mentioned above by @elephaint, accessing the dictionary keys and updating the values for AutoLSTM. Opening an issue regarding it

updating the default config didn't worked for me I tried doing same mentioned above by @elephaint, accessing the dictionary keys and updating the values for AutoLSTM. Opening an issue regarding it

When updating the docs we found that it requires a deepcopy to properly copy the config. See the updated comment above.


Resolve compilation warnings for Elixir v1.15+

Changes
Resolved the compilation warnings about deprecated function usage:

Logger.warn/1 was deprecated in Elixir v1.15 and replaced with Logger.warning/2.

Notes
This library doesn't specify what versions of Elixir it works with -- if this PR is merged then it will require Elixir v1.11 or above.

Thank you!

This library doesn't specify what versions of Elixir it works with -- if this PR is merged then it will require Elixir v1.11 or above.

Elixir version is specified here
https://github.com/fuelen/ecto_erd/blob/295d2d3db8d87ae618fb788fe47d44d9d1ec0823/mix.exs#L20


[ALLUXIO-2788] replace Throwables.propagate with RuntimeException

replace Throwables.propagate with RuntimeException.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/14496/
Test PASSed.

Hey @Pennywpp, do you mind signing the CLA? You can sign it here: http://cla.alluxio.org/

@Pennywpp  could you sign the contribution license agreement?


Check for unexpected species contaminants

Especially with low-input non-human samples, a useful QC step would be to screen for unexpected species (e.g. human, ecoli)

https://github.com/csawye01/nf-core-demultiplex/blob/d336df6dc08caed44505e286f9a47e386986d50c/main.nf#L53
https://www.bioinformatics.babraham.ac.uk/projects/fastq_screen/

I suspect a k-mer based approach is like to be the most efficient way of screening for contaminants in a way where you can pass a relatively unbiased and large database of organisms.
@olgabot Have there been any comparisons made between kraken2 and sourmash? They seem to have similar applications? kraken2 is written in C++ and is quite rapid whereas sourmash is written in Python with some optimisation Im assuming?

I use kraken2 in the nf-core/bacass pipeline to check for potential contamination prior to doing the assembly, that works quite nicely though you, of course, need to specify a database still.

If (in addition to rRNA removal as suggested in #227) an optional step would be added that would even remove all reads from a particular species, e.g. human, than this pipeline might be able to also efficiently analyze human metatranscriptomics.

I am myself not involved in metagenomics of human samples but environmental samples, so my ideas have to be taken with a little caution.
The simplest solution would be using the host genome and forward all unmapped reads (use mapper of choice) for analysis. However, non-host sequences similar to the host could be lost as well in the process. This could be minimized by using strategies such as KRAKEN2 on relevant references (e.g. human + bacteria and remove all that are annotated as human) or DIAMOND (e.g. on whole Ensembl).
Here is an example where contaminant reads were removed by bowtie mapping (in the tool KneadData) to focus on the endogenous E. coli strain.

The merge above closes this. If there are other tools besides Kraken that people want to see specifically, the --contaminant-screening structure allows for their future addition


etcdserver: print out correct restored cluster info

Before this PR, it always prints nil because cluster info has not been
covered when print:
2015-10-02 14:00:24.353631 I | etcdserver: loaded cluster information
from store: <nil>

After:
2015-10-02 14:01:53.320350 I | etcdserver: loaded cluster information
from store: {ClusterID:7e27652122e8b2ae Members:[&{ID:ce2a822cea30bfca
	RaftAttributes:{PeerURLs:[http://localhost:2380
		http://localhost:7001]} Attributes:{Name:default
ClientURLs:[http://localhost:2379 http://localhost:4001]}}]
		   RemovedMemberIDs:[]}

@yichengq Can we make the format better? Just print out the member id?

The commit message reformat the sentence. The actual one looks like this:
2015-10-02 14:01:53.320350 I | etcdserver: loaded cluster information from store: {ClusterID:7e27652122e8b2ae Members:[&{ID:ce2a822cea30bfca RaftAttributes:{PeerURLs:[http://localhost:2380 http://localhost:7001]} Attributes:{Name:default ClientURLs:[http://localhost:2379 http://localhost:4001]}}] RemovedMemberIDs:[]}

I would like it to print more data because initial cluster info is quite important for log read and debug. The member representation could be improved though.

@yichengq I am fine if you want to switch it to debug mode. For info, we just need ids probably. Also you can separate this to multiple lines if you really want to see it.

I really want to see it in the log. One reason is that I could find out all cluster info and changes from reading the log, which makes the debug easier.
The new output:
15:17:25 etcd2 | 2015-10-02 15:17:25.291755 I | etcdserver: recovered cluster information from the underlying store
15:17:25 etcd2 | 2015-10-02 15:17:25.291772 I | etcdserver: loaded member 8211f1d0f64f3269 [http://<IP_ADDRESS>:12380] to cluster ef37ad9dc622a7c4
15:17:25 etcd2 | 2015-10-02 15:17:25.291779 I | etcdserver: loaded member 91bc3c398fb3c146 [http://<IP_ADDRESS>:22380] to cluster ef37ad9dc622a7c4
15:17:25 etcd2 | 2015-10-02 15:17:25.291786 I | etcdserver: loaded member fd422379fda50e48 [http://<IP_ADDRESS>:32380] to cluster ef37ad9dc622a7c4
15:17:25 etcd2 | 2015-10-02 15:17:25.291792 I | etcdserver: loaded the cluster version 2.2

Addressed all comments.

LGTM


JS: Fix OrderedPutCache for browser

Fixes #2395

@cmasone-attic PTAL

THis LGTM, but something happened on Jenkins. It looks like the npm installation over there can't find a whole bunch of the required modules or something. Not sure if it's a Jenkins issue or some odd side effect of this patch (likely the former?)


docs: Maintain menu position during navigation

closes #1864

@Amelie-V @davidberenstein1957  I've done some testing by saving the sidebar position between pages, it's a small jump on page change. Take a look and we can comment
https://docs.argilla.io/en/docs-maintain_menu_position/

LFTM!

@leiyre cool


Frameworks/link to guides

One set of framework guides to rule them all.

Thx. I haz no merge power here, so help me @odoe, you're my only hope.


Image drag and drop on web composer

Fixes https://github.com/bluesky-social/social-app/issues/1257
Adds the ability to insert images by drag-and-drop.
The drop trigger covers the whole screen so you don't have to go very far to add the image.
The overlay modal has a box-shadow to compensate for reduced opacity on the backdrop, because having the backdrop opacity be the same as the composer modal seems to make it too dark.

Is there a reason behind the text/plain handling? I'm a bit confused because I don't remember having to deal with that sort of thing

I forget exactly why that got done -- I seem to remember there's some flow that gets recognized as a text-link getting pasted. But it does mean that pasting an image URI gets added automatically as an image if it can.


Added 'only' as a word with no plural

Coverage remained the same at 100.0% when pulling 52ad609c6c40785f4c1b35b348f42570955ea2ac on Wildhoney:master into 298eb67f19369dd30207c0817eda4f0045c6c1a7 on blakeembrey:master.


[stable/locust] Using an Ingress with the v0.9.10 returns 404

Hello,
I'm installing this chart on one of our clusters. Unfortunately, when enabling the ingress the ingress controller returns the standard NGINX 404 Not Found error page, rather than routing correctly to the service. Removing the asterisk from the path fixes it. I was hoping there was an ingress.hosts[].path variable I could set in the values file, but it is hardcoded in the template.
The path was changed from / to /* in #127, and I see it was changed in order to fix a bug related to static files.
I'm pretty new to helm and I'm not sure if this is a problem with the chart or with the cluster. We definitely have used asterisks in paths before, on other clusters. This cluster is new to me, but the thing I know is different from the others is that it has 2 Ingress Controllers. (I also had to set the kubernetes.io/ingress.class annotation for it to work at all.)
For now, I can just install v0.9.9 and it works fine. But I was hoping it could be made configurable, or perhaps just provide some pointers to determine whether it's a cluster problem.
If you decide to make it configurable, I'm happy to submit a PR.
Please let me know if you need additional information.
Thank you.

I'm using an EKS Cluster and AWS load balancer controller, so I need to set the /* path.
It seems to be a problem due to environmental differences.

But I was hoping it could be made configurable

I think this way is better.
locust/templates/master-ingress.yaml
...
    {{- range .Values.ingress.hosts }}
    - host: {{ .host | quote }}
      http:
        paths:
          # configurable
          - path: {{ .path }}
            backend:
              serviceName: {{ $fullName }}
              servicePort: 8089
    {{- end }}
...

I'm using an EKS Cluster and AWS load balancer controller, so I need to set the /* path.

Ah I see. Yeah with nginx-ingress (probably the most common ingress controller), the * functions differently.
I will make a PR to fix.

@recfab let me know if chart version 0.9.11 works for you.
@linzhengen you will need to change the path value to suit your env/ingress controller

Yes, version 0.9.11 works for me.
Thanks!


Is it possible to add custom User Tokens for regular expressions?

Is it possible to add custom UserTokens for regular expressions?
For example: \F
For float numbers:
+.123
-1.5e-10
3.14

Or you can give an example, or where to look, how to do this.

CTRE is C++ code, so "yes", but not as part of the actual regex grammar...
#define RX_FLOAT "whatever you want here"
auto res = ctre::match<RX_FLOAT " plus " RX_FLOAT " equals " RX_FLOAT>(teststring);
Don't like defines? you can also get it done by using a compile literal time string ( a bit more advanced than the one in CTRE so you can use + operators).
And you can also split up the regex in multiple ctre calls which may result in better code because the compiler only needs to generate the float pattern match once, which depending on the regex it may or may not do on it's own.  This may even end up with more readable C++ code and not require the C++ dev to also be fluent in regex, which may be an issue with really complex regexes.   Also... repeating a complex sub-pattern multiple times into a larger pattern may cause quite slow compilation because of multiple rounds in the optimizer.


[Q] oneOf with null value generates unexpected property Type

I have a json-schema part like:
"permanentNumber": {
  "oneOf": [
    {
      "type": "string",
      "minLength": 1,
      "maxLength": 2,
      "pattern": "^[0-9]{1,2}$"
    },
    {
      "type": "null"
    }
  ],
  "description": "The permanent number."
}

It uses the oneOf to either support an actual value or null (as the property is required in the schema).
So below cases are both valid:
{
  "permanentNumber": "33"
}

{
  "permanentNumber": null
}

Now jsonschema2pojo generates the property as an Object where I want it to be a String:
@JsonProperty("permanentNumber")
@JsonPropertyDescription("The permanent number.")
@NotNull
public Object permanentNumber;

Does this make sense and is there a way to generate a String instead?

I changed the schema to
"permanentNumber": {
  "type": ["string", "null"],
  "minLength": 1,
  "maxLength": 2,
  "pattern": "^[0-9]{1,2}$",
  "description": "The permanent number."
}

which does seem to do the trick.

This project doesn't support oneOf I'm afraid. We have a simple fallback where if we see a union type like ["string", "null"] we will take the first schema as the type, which allows for most use-cases where you are just using a union to express something optional.


Update formatting and grammar in UG

Reordered some portions and made the formatting more consistent

Codecov Report

Merging #252 (274a71a) into master (3299d3b) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #252   +/-   ##
=========================================
  Coverage     68.56%   68.56%
  Complexity      711      711
=========================================
  Files           112      112
  Lines          2510     2510
  Branches        306      306
=========================================
  Hits           1721     1721
  Misses          677      677
  Partials        112      112

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Schema empty

:( everything is starting fine but the schema seems to be empty and the gui is thorwing an error
{
"errors": [
{
"message": "Names must match /^[_a-zA-Z][_a-zA-Z0-9]*$/ but "Map«string,object»" does not."
}
]
}

Hi. This is an internal graphql.js error, pointing out that type name is wrong. I actually doubt that it would work with Swagger either


feat: update schema, add provider key + transform

this pr updates the asset schema to the schema below.
poster and sources are derived data so I didn't add them to the stored state yet.
to generate this derived data, transformer's were added alongside the provider files.
not in the same file as the provider, because the front-end needs to pull this in.
I tried to add some more logic to assets.ts and re-use it where was possible.

export interface Asset {
  status: 'sourced' | 'pending' | 'uploading' | 'processing' | 'ready' | 'error';
  originalFilePath: string;
  provider: string;
  providerSpecific?: {
    [provider: string]: { [key: string]: any }
  };
  blurDataURL?: string;
  poster?: string;
  sources?: AssetSource[];
  size?: number;
  error?: any;
  createdAt: number;
  updatedAt: number;

  // Here for backwards compatibility with older assets.
  externalIds?: {
    [key: string]: string; // { uploadId, playbackId, assetId }
  };
}

do we need/want the CLI to migrate existing assets over? Even if it's not a big deal now, do we want to set the precedent that we'll do so?

it crossed my mind as well, I thought to keep it for the v1.
externalId's is used only in the transformers as a fallback so it's pretty minimal code to stay backwards compatible


Add demote command feature

Describe your changes
Added DemoteCommand class
Created demote command feature
Issue ticket number and link
Closes #106
Checklist before requesting a review

[x] I have performed a self-review of my code.
[x] If it is a core feature, I have added thorough tests.

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

Files with missing lines
Coverage Δ
Complexity Δ

...va/seedu/address/logic/commands/DemoteCommand.java
100.00% <100.00%> (ø)
9.00 <9.00> (?)

.../seedu/address/logic/parser/AddressBookParser.java
100.00% <100.00%> (ø)
14.00 <0.00> (+1.00)

...eedu/address/logic/parser/DemoteCommandParser.java
100.00% <100.00%> (ø)
3.00 <3.00> (?)

... and 2 files with indirect coverage changes


mini.animate prevents zz from centering when at top/end of file

Contributing guidelines

[X] I have read CONTRIBUTING.md
[X] I have read CODE_OF_CONDUCT.md
[X] I have updated 'mini.nvim' to latest version

Module(s)
mini.animate
Description
With  and  rebound to zz and zz scrolling does not function properly.
Depending on the version of neovim this problem has two variants:
0.8.3
When I press  in the center of a file, everything works great. But if you are at the very top/bottom of a file the first <C-d> will not center the screen. The next press will attempt to scroll, fail, jump back to the current position, but this time with the screen centered. Continuing down will now succeed for the rest of the file.
0.9.0
<C-d> and <C-u> fail, always returning to the original cursor position.
Neovim version
0.8.3 and nightly 0.9.0-dev-857+g2c5906b55
Steps to reproduce

Rebind  and  to center the buffer with zz

vim.keymap.set("n", "<C-d>", "<C-d>zz")
vim.keymap.set("n", "<C-u>", "<C-u>zz")

Hit C-d at the top of a long file.
Hit C-u at the bottom of a long file.

Expected behavior
The buffer should scroll down and center the screen when  is pressed from the top of a file.
Alternatively, it should scroll up and center the screen when  is pressed from the bottom of a file.
Actual behavior
0.8.3
The buffer scrolls down but does not center the screen. The next time  is pressed the screen starts to scroll down, then jerks back up and ends up with the cursor back at the same location, but the buffer is now centered. After this point, scrolls succeed as normal.
https://user-images.githubusercontent.com/454563/218840867-888d99e6-e8c8-40dc-99d5-2b01a297eb18.mp4
0.9.0
The screen scrolls but snaps back and the cursor returns to the original position. If you rapidly press  and get the command in before the buffer scrolls down to 1, you successfully scroll a bit.
https://user-images.githubusercontent.com/454563/218840823-580471b1-e448-4a7c-85a0-b1c8b1b179b4.mp4

Thanks for a detailed issue!
It seems like symptoms of asynchronous nature of scroll. When you execute <C-d>zz mapping, what happens is Neovim registers <C-d> and 'mini.animate' starts scroll animation (which also progressively changes cursor position for a smoother effect). But right after it start animating, there is a zz command which is registered as another scroll event which takes precedence (thus trying to center the screen).
Unfortunately, having multiple scrolling commands in a mapping is not really supported. The best way to overcome this right now is by following this advice. Could you try remapping it as written there and tell if this makes things better?

Ah, not sure how I missed that issue, thanks!
Adding the execute_after version did solve my problem.
  vim.keymap.set(
    'n',
    '<C-d>',
    [[<Cmd>lua vim.cmd('normal! <C-d>'); MiniAnimate.execute_after('scroll', 'normal! zvzz')<CR>]]
  )
  vim.keymap.set(
    'n',
    '<C-u>',
    [[<Cmd>lua vim.cmd('normal! <C-u>'); MiniAnimate.execute_after('scroll', 'normal! zvzz')<CR>]]
  )

Given how common the <C-d>zz rebind is maybe explicitly using this remap as an example in a section after Installation in the readme for Remapping animated commands or something of that nature would help future users.
Thanks again!

Adding the execute_after version did solve my problem.

Great!

Given how common the <C-d>zz rebind is maybe explicitly using this remap as an example in a section after Installation in the readme for Remapping animated commands or something of that nature would help future users.

This is already in the help file. But as you are not the first one with this kind of problem (and solution, even), this might be a good idea.
Closing then.


⚠ Bofrost.at has degraded performance

In 6d84f72, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1237 ms

Resolved: Bofrost.at performance has improved in 60633f8 after 6 minutes.


Redirecting to newly inserted post

Hi,
I'm wondering how I can redirect to post I've created in my onSuccess callback. Is there a way to use this._id to redirect?
Cheers!

Should be in this.docId.

thanks a bunch Aldeed! Pleasure to use your stuff btw :+1:


Feature request: Hierarchical alert receivers based on time

Alertmanager can repeat sending alert to receiver or multi-receivers. But it cannot re-send the same alert to other receivers like supervisor after a timeout period. If the same alert exists more than set time, it will be sended to sub-receiver to solve.
This feature avoids bothering everyone and delaying to solving alerts.
sample
route:
  group_by: ['alertname', 'cluster']
  group_wait: 30s
  group_interval: 5m
  repeat_interval: 12h
  receiver: default-receiver
  routes:
  - match:
      alertname: test
    receiver: receiver1
    timeout: 1h
    sub_receiver: receiver2

This looks more like the responsibility of an incident management system (eg Pager Duty, OpsGenie, ...).
You could still emulate something similar with continue:
routes:
- match:
    alertname: test
  receiver: receiver1
  continue: true
- match:
    alertname: test
  receiver: receiver2
  group_wait: 1h


Conty without using an image?

I'm wondering about modifying Conty to not use a squashfs image, but just a plain directory structure or a filesystem of my choosing that I mount (E.g. zfs dataset).

This is possible. I added this feature to the custom_mnt branch.
So download this conty-start.sh, download utils.tar.gz, combine them:
cat conty-start.sh utils.tar.gz > conty.sh
chmod +x conty.sh

And then you can use it with your custom mount points (or directory structures) like this:
conty_custom_mnt=/path/to/your/mnt ./conty.sh some_command

I added the feature to the latest release.


Synonym additions

This came in via the GO-Help desk from SIB
Name: Paula Duek
E-mail<EMAIL_ADDRESS>

Could "Midbody ring" be added as synonym of "Flemming body" (PMID: 19242118;
PMID: 18329369)? DONE

I suggest to remove "P-body" as synonym of GO:0010494 ("cytoplasmic stress
granule") as "P-body" (GO:0000932) and "stress granule" (GO:0010494) are
distinct entities. DONE

It would be useful to create the parent term "Cytoplasmic body" that will
have as child GO:0010494 and GO:0000932, and will be used in cases in which
protein localization is not well defined as in
http://www.proteinatlas.org/learn/dictionary/cell/cytoplasmic+bodies

I have proposed this general def: "Cytoplasmic body is a collective term for distinct foci inside the cytoplasm such as processing bodies (P-bodies) or stress granules.
Upon staining, cytoplasmic bodies appear as distinct round spots in the cytoplasm."
Currently,  these two ids are decedents of cytoplasmic ribonucleoprotein granule
The possible definition of the term suggested from the URL protein atlas site has nothing in it to indicate an RNA component.
(the next highest term is “intercellular non-membrane-bounded organelle”
Have emailed Dr. Duek for advice

Dr. Duek replied:
I re-contacted HPA and they have experimental evidences for their "cytoplasmic bodies" localising to one of the granules listed under GO:0036464 "cytoplasmic ribonucleoprotein granule”. So there is NO need to create an additional GO term as HPA "cytoplasmic bodies" can be mapped to GO:0036464 "cytoplasmic ribonucleoprotein granule”.
Sorry for contacting GO about this, hope this did not create inconveniences
Best regards,
Paula
I am closing the ticket


Allergens refactor and document

Description
For classes concerning Allergens, I have added javadoc comments. This should make it easier for other developers to understand if working on or with these classes and methods.
Related issues and discussion
N/A (Did not feel necessary to create, no functionality added or changed).

@Karljoones Great, I'll resolve the merge conflicts.

re: Travis - a test failed for OBF.


Fix kubernetes manifest

depends_on not working with kubernetes_manifest its an open issue
as a workaround we can use kubectl_manifest
https://medium.com/@danieljimgarcia/dont-use-the-terraform-kubernetes-manifest-resource-6c7ff4fe629a

The dependency is not fixed, it's a known limitation CRD must exist before CR creation, detailed discussion: https://github.com/hashicorp/terraform-provider-kubernetes/issues/1367


New pipeline 2024-01-09

This pipeline does not make use of a shingle analyzer, which addresses latency issues https://github.com/wellcomecollection/catalogue-api/issues/734.
This PR also changes the default cluster to be a 4gb CPU-optimized one, and uses CPU-optimized hardware for both reindexes and normal operation. It also resolves https://github.com/wellcomecollection/catalogue-pipeline/issues/2463.

Not blocking for merge, but what metrics did you use to validate that the latency is caused by the shingling (it seems very likely from the context and linked conversations that it is, but is there a smoking gun somewhere that makes it very clear it's this specifically?).

Good question @kenoir - I used the Kibana search profiler with a query that I knew was very slow. It showed (a) uniform high latency across all shingle-analyzed fields that was (b) distributed across a huge number of shingled terms, so I guessed that the shingling was the problem.
I then added this to the relevance tests and confirmed that it failed with a timeout, removed the shingles from the analyzers and saw that it started passing.


[Feat]:Added pagination functionality for the issue #8

#8
I have added the pagination functionality below the table and set the maximum number of rows per page to 8, anyone can change it to whatever number is required. Some styling for the container div was also changed.
if anything is found broken in the code or need help with please let me know.

Instead of the arrow, can it be page number "1", "2"?

It can be numbers  but I found this to be more scalable. We can include page numbers in between the arrows.

There seem to be no conflicts and the restyled test is failing because I changed the container's CSS a little but nothing to worry here the code works fine.

There seem to be no conflicts and the restyled test is failing because I changed the container's CSS a little but nothing to worry here the code works fine.

Yes, ok.

Thanks for your contribution @piyushyadav1617 .

Welcome! ping me if more features are needed.

I am thinking of adding the numbers to the pagination in between the arrows today. Shall I open a new issue?

It's already added.
Thanks,
Kendall Does Coding
Portfolio
https://cdc-aims.com/click?redirect=https%3A%2F%2Fkendalldoescoding.gq&dID=1680603500688&hashId=060de6478a47358f5315814fdd49f493dff92988d90dca1ba25e30911a8a60f18777e7&linkName=Portfolio
Github
https://cdc-aims.com/click?redirect=https%3A%2F%2Fgithub.com%2FKendallDoesCoding&dID=1680603500688&hashId=060de6478a47358f5315814fdd49f493dff92988d90dca1ba25e30911a8a60f18777e7&linkName=Github
Youtube
https://cdc-aims.com/click?redirect=https%3A%2F%2Fkendalldoescoding.gq%2FcodingYT&dID=1680603500688&hashId=060de6478a47358f5315814fdd49f493dff92988d90dca1ba25e30911a8a60f18777e7&linkName=Youtube
On Tue, 4 Apr 2023 at 13:31, Piyush Yadav @.***> wrote:

I am thinking of adding the numbers to the pagination in between the
arrows today. Shall I open a new issue?
—
Reply to this email directly, view it on GitHub
https://github.com/KendallDoesCoding/youtubers-birthdays/pull/14#issuecomment-1495521620,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUGJUTNVMYXPSZS6PBAIGJ3W7PINLANCNFSM6AAAAAAWNFRTYQ
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

Yes! I saw.


Can't find '../test/testWriter.js' from bin/microformat-node

The likely cause is that test/testWriter.js was removed in: https://github.com/glennjones/microformat-node/commit/c44e1da9e77131f21d875a1b1b2b5bd64ff2f069
It means that bin/microformat-node refuses to start now though.
Not sure what function it played though so can't provide an easy PR to fix it unfortunately.

The test/testWriter.js is no longer needed after the recent changes to the microformat test suite. It was use to update the mocha tests.  Sorry I just forgot to remove it from bin/microformat-node  I will do that now.

Guess this one can be closed then?


Adding tooth:teeth

This looks good but can you squash into a single commit?

Thanks for contributing!


Add retries to the downloads tasks

Downloads might fail due to temporary network issues. At the moment this makes a deployment fail entirely, which is pretty bad for user experience.

As discussed with @leopaul36 if one encounters download issues, they should better fix the root cause.


IGNITE-23138 Fix TxWithKeyContentionSelfTest (Cache 12 group)

Thank you for submitting the pull request to the Apache Ignite.
In order to streamline the review of the contribution
we ask you to ensure the following steps have been taken:
The Contribution Checklist

[ ] There is a single JIRA ticket related to the pull request.
[ ] The web-link to the pull request is attached to the JIRA ticket.
[ ] The JIRA ticket has the Patch Available state.
[ ] The pull request body describes changes that have been made.
The description explains WHAT and WHY was made instead of HOW.
[ ] The pull request title is treated as the final commit message.
The following pattern must be used: IGNITE-XXXX Change summary where XXXX - number of JIRA issue.
[ ] A reviewer has been mentioned through the JIRA comments
(see the Maintainers list)
[ ] The pull request has been checked by the Teamcity Bot and
the green visa attached to the JIRA ticket (see TC.Bot: Check PR)

Notes

How to Contribute
Coding abbreviation rules
Coding Guidelines
Apache Ignite Teamcity Bot

If you need any help, please email<EMAIL_ADDRESS>or ask anу advice on http://asf.slack.com #ignite channel.

https://ci2.ignite.apache.org/buildConfiguration/IgniteTests24Java8_Cache12/8080567
https://ci2.ignite.apache.org/buildConfiguration/IgniteTests24Java8_Cache12/8080691
https://ci2.ignite.apache.org/buildConfiguration/IgniteTests24Java8_Cache12/8080699
https://ci2.ignite.apache.org/buildConfiguration/IgniteTests24Java8_Cache12/8080701


Clarify documentation for Scope - User

Please clarify in the text on this page when settings "Scope to user", state whether that scoped user means the modified by field of the record must match user that is set as the Flow Connection for the Dataverse. Does this mean the flow will not trigger if the user used in the connection to dataverse is not the user currently making the modification. Or will the flow on this dataverse record trigger for every edit regardless of the dataverse connection user
[Enter feedback here]

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

ID: 94c84a31-4f63-6ee1-ad15-096cae44cb41
Version Independent ID: de8cf5bc-b78b-a88e-696f-7755e3ce22f0
Content: Trigger flows when a row is added, modified, or deleted - Power Automate
Content Source: articles/dataverse/create-update-delete-trigger.md
Service: flow
GitHub Login: @MSFTMan
Microsoft Alias: Deonhe

Thanks for contacting us. The Scope heading describes the ownership scope for Users. For additional information, contact Support at: https://flow.microsoft.com/support/ Closing this issue.


How to add a pthread library in the build

Team,
Nor so much an issue really, but more a request for help:
In my wakaama build I add various functions, and one of them is using pthread and sqlite3.
I added the following in the wakaama.cmake:
add_link_options(
-lpthread
-lm
-lsqlite3
)
This is accepted, though the final CMakeFiles/lwm2mclient_tinydtls.dir/link.txt which contains the actual compile arguments shows that the libraries are at the start of the cc, see this excerpt:
==> link.txt <==
/usr/bin/cc   -lpthread -lm -lsqlite3 CMakeFiles/lwm2mclient_tinydtls.dir/lwm2mclient.c.o ....
This result in missing functions in pthread etc.
If I edit the link.txt and move the -lpthread -lm -lsqlite3 to the end of the command, I have a perfect build.
What can I do to have these libraries always at the end in the compile command ?
Where in the wakaama.cmake, or other input file can I control this ?
Many thanks,   - Dirk Gooris

FMPOV, that's a question for the cmake forum or help.
add_link_options
###################################################
Add options to the link step for executable, shared library or module library targets in the current directory and below that are added after this command is invoked.
add_link_options( ...)
This command can be used to add any link options, but alternative commands exist to add libraries (target_link_libraries() or link_libraries()). See documentation of the directory and target LINK_OPTIONS properties.
###################################################
So, try to use target_link_libraries instead.
(But I hove not verified, that it works ;-), I leave that to you.)

Thanks for these directions. It tried this:
I updated this file :   examples/client/CMakeLists.txt  with target_link_options() in the next lines:
    add_executable(lwm2mclient_tinydtls ${SOURCES})

    set_target_properties(lwm2mclient_tinydtls PROPERTIES DTLS "tinydtls")
    target_compile_definitions(lwm2mclient_tinydtls PRIVATE LWM2M_CLIENT_MODE LWM2M_BOOTSTRAP LWM2M_SUPPORT_SENML_JSON)
    target_sources_wakaama(lwm2mclient_tinydtls)
    target_sources_shared(lwm2mclient_tinydtls)

    target_link_options(lwm2mclient_tinydtls PRIVATE LINKER:-lpthread -lm -lsqlite3)

This will also generate a faulty cc command.  See the very top of the link.txt file:
CMakeFiles/lwm2mclient_tinydtls.dir/link.txt
/usr/bin/cc -Wl,  **-lpthread -lm -lsqlite3**  CMakeFiles/lwm2mclient_tinydtls.dir/lwm2mclient.c.o  <other .o files>
And this build results in missing libraries of pthread and sqlite3, because the -l options are not take at the start of the cc.
Again, if I move the -lpthread -lm -lsqlite3 to the end of the link.txt and run this as a copy/paste, it compiles without any error.
I am not sure its a cmake issue really. The previous versions of Wakaama accepted this command in the same CMakeLists.txt:
   TARGET_LINK_LIBRARIES(lwm2mclient pthread m sqlite3)
The result of that function/macro resulted in the right syntax of cc, see the libraries at the end of the cc command:
   /usr/bin/cc     CMakeFiles/lwm2mclient.dir/lwm2mclient.c.o <and all the other .o's>  -o lwm2mclient -rdynamic -lpthread -lm -lsqlite3
In general terms to build Wakaama against an application; what would be the preferred way to link these type of libraries otherwise ?  Or should I open a question to cmake how to control the position of the libraries when using  target_link_options() ?
Many thanks again, regards.

Good news,
I mixed up the use of target_link_options() with the suggested target_link_libraries()  by Achim Kraus.
It works now. Here are the changes I made in examples/client/CMakeLists.txt
# Client without DTLS support
add_executable(lwm2mclient ${SOURCES})
target_compile_definitions(lwm2mclient PRIVATE LWM2M_CLIENT_MODE LWM2M_BOOTSTRAP LWM2M_SUPPORT_SENML_JSON)
target_sources_wakaama(lwm2mclient)
target_sources_shared(lwm2mclient)

# Add the libraries that our application uses
target_link_libraries(lwm2mclient pthread m sqlite3)

# Client with DTLS support provided by tinydtls
add_executable(lwm2mclient_tinydtls ${SOURCES})

set_target_properties(lwm2mclient_tinydtls PROPERTIES DTLS "tinydtls")
target_compile_definitions(lwm2mclient_tinydtls PRIVATE LWM2M_CLIENT_MODE LWM2M_BOOTSTRAP LWM2M_SUPPORT_SENML_JSON)
target_sources_wakaama(lwm2mclient_tinydtls)
target_sources_shared(lwm2mclient_tinydtls)

# Add the libraries that our application uses
target_link_libraries(lwm2mclient_tinydtls pthread m sqlite3)

Question answered and problem solved - thanks Achim :-)
Regards, - Dirk Gooris

If it's solved, please close the issue.


Fixes #1004: Support Swagger tags

Closes https://github.com/getinsomnia/insomnia/issues/1004

Adds support for Swagger tags which groups requests into folders

Also provides support for multiple tags in a single request: it will create duplicate requests to list them in multiple folders

A request with multiple Swagger Tags (i.e. Insomnia folders) is possible and displaying multiple times is expected as seen in this Swagger UI issue
Sample below is a modified json file where

Update an existing pet has tags of pet and store and
Add a new pet to the store has no tags

What happened to this pr? seems to have never made it in?

Hi @mawiseman 👋 I've imported https://petstore.swagger.io/v2/swagger.json and correctly puts the requests in different folders (each folder represents the tag on the request) so it looks like it has been merged.
Do you have a URL / file where it doesn't work? That could help us figure out what's wrong 👍

These are my Insomnia details if it helps as well 😃
Version: Insomnia Core 2020.4.1
Release date: 9/23/2020
OS: Darwin x64 19.6.0
Electron: 9.1.1
Node: 12.14.1
V8: <IP_ADDRESS>-electron.0
Architecture: x64
node-libcurl: libcurl/7.69.1 OpenSSL/1.1.1g zlib/1.2.11 brotli/1.0.7 libidn2/2.1.1 libssh2/1.9.0 nghttp2/1.41.0

Thanks for looking into this @thewheat
I have attached a cut down version of the swagger file I am trying to import.
swagger.json.txt
It was generated using the Swashbuckle library for dotnet core: https://github.com/domaindrivendev/Swashbuckle.AspNetCore
You can see it is valid when pasting it into the online editor: https://editor.swagger.io/
Here are my Insomnia details
Version: Insomnia Core 2020.4.1
Release date: 9/23/2020
OS: Windows_NT x64 10.0.18363
Electron: 9.1.1
Node: 12.14.1
V8: <IP_ADDRESS>-electron.0
Architecture: x64
node-libcurl: libcurl/7.69.1-DEV OpenSSL/1.1.1d zlib/1.2.11 WinIDN libssh2/1.9.0_DEV nghttp2/1.40.0

Ah @mawiseman I see that your file says it is an OpenAPI specification and not a Swagger one (Insomnia currently has a different import for each of those and the one I modified was Swagger 😅

Swagger definition in the Petstore example

Did a test and if you add tags in the root definition it does correctly create the folders

Digging further, from https://github.com/Kong/insomnia/issues/2408 it looks like the beahviour of needing to add it to global scope is expected. Not ideal but if anything, I think that issue would be best for any further discussion 👍

Thanks @thewheat!


Version 0.18 release

@charlotteis made codes of conduct a requirement. @ExcaliburZero made the language option case-insensitive.
Updated the readme and changelog to reflect this.
:tada:

:tada:

@M-Zuber Wait don't merge yet! Just had to add he compiled code. It's v0.18.1 now

Not that we owe semver anything, but what mechanism do you use that forces the increment?

never mind dumb me. npm right?

Correct! Not dumb. I go npm minor|patch -m "blah blah"

Can I merge btw?
Maybe the steps for deployment/release should be written up somewhere?

Sure, go ahead and merge.
And they should be yes, especially since once we switch to an org I'm hoping you can also make releases :tada: if you want to anyway!
I'll write something up now and make another PR.
One of the ways my current job does PRs is to encourage others to do the merge as it means they've probably had to have checked the code is a-ok. I think we should do it that way.

One of the ways my current job does PRs is to encourage others to do the merge as it means they've probably had to have checked the code is a-ok. I think we should do it that way.

I agree with this. Will check if there is a way to enforce that the merger can not be the PR author.

And they should be yes, especially since once we switch to an org I'm hoping you can also make releases :tada: if you want to anyway!

Since we have decided on the previous item, then anyone who is a collab should be able to make a release - as it is not final until the PR is merged.
The npm story does need to be though out though.
Maybe the merger is the one that pushes to npm?
Otherwise if a PR is made but not merged, you have lost that version on npm. (there is a trick to publishing betas that can not be seen on the site, remind me and I'll tell you later)

Oh, and maybe the @ forkability account on twitter should be telling people about the release?

Oh, and maybe the @ forkability account on twitter should be telling people about the release?

On it. Will reply to other thing in a min.

Will check if there is a way to enforce that the merger can not be the PR author.

Ah, I dunno if we wanna make it a requirement. Just in case!

Since we have decided on the previous item, then anyone who is a collab should be able to make a release - as it is not final until the PR is merged.

Hmm, maybe. Publishing a release is quite a big deal though! I mean, it can make or break someone's dependent code. Quite a huge responsibility.

Maybe the merger is the one that pushes to npm?

This sounds like a good plan, as long as the merger has npm publish access.
I gotta go teach a keen coder to get their Python environment set up now - I'll get around to the publishing guide soon! Maybe tonight or tomorrow morning. Ugh.
Real life ¯_(ツ)_/¯


[Feature request] Transform specific frontmatter key(s) into image ESM exports

I would like to convert this:
```markdown
---
title: "Cool title"
description: "Cool description"
coverImage: "./cool-pic.jpg"
coverImageAlt: "Cool alt"
---
```

into something like this:
export { default as coverImage } from "./cool-pic.jpg"

It has overlap with https://github.com/remcohaszing/remark-mdx-frontmatter. Perhaps it should be another plugin?

Thanks for your proposal!
I see why this would be useful. I also see the similarities with remark-mdx-frontmatter and remark-mdx-images, but I don’t see how it fits in either of them.
I think it’s best to create a separate plugin for this. I imagine something like this?
import { compile } from '@mdx-js/mdx'
import remarkMdxFrontmatterExports from 'remark-mdx-frontmatter-exports'

const { result } = compile(file, {
  remarkPlugins: [
    [remarkMdxFrontmatterExports, { names: ['coverImage'] }]
  ]
})

I’m closing this issue as it’s not going to be implemented here.


Build logs sometimes failing to push to the UI

Hi,
I'm seeing an issue where the build logs don't seem to be populating in UI sometimes in my deployment of BinderHub, but they are still showing up in the kubernetes pod. The logs in the UI seem stuck at "Waiting on the build to start" step for a long time. There doesn't seem to be any errors related to this in the binder or build pods.
Has anyone experienced this?

I can't think of the last time when I've seen this :-/
OTOH networking issues or the build pod watcher/reflector losing track of the build pods that are running is the reason for this.
Are you running master (or near to it) of binderhub?

Does the user eventually get redirected to their running pod or does it get stuck at "waiting for build pod to start"? If the latter happens that would point towards the pod reflector not working properly.
Just remembered that we did recently have a problem on mybinder.org where build logs wouldn't get streamed. This turned out to be some internal networking issue and "went away by itself". We first noticed it because kubectl logs <podname> had stopped working. It printed an error message about the kubernetes master not being able to create a ssh tunnel to the node.
https://github.com/jupyterhub/mybinder.org-deploy/blob/master/docs/source/incident-reports/2019-02-20-no-logs.md is the report on that

Hi! Yes, i am using the latest development release helm chart: binderhub-0.2.0-64bb3c7. I believe it's sync'd to the latest master.
Sometimes the build logs resume from the middle or after the code reaches the launcher. I've seen the launcher fail due to timeouts as well the first and second time after waiting 300 seconds to spawn a session. Other times, it gets stuck completely and i never see the user session launch. I have seen it working end to end correctly sometimes though.
Thank you for the pointer, I was suspecting some issue with networking as well, I'll dig deeper into it.
kubectl logs <podname>  do still work for me which is why this was confusing since I don't see any errors in the logs. Let me know if you have any other pointers!

If kubectl logs works and there are timeouts I'd do what you are doing: investigate the networking.
Where is your kubernetes cluster hosted or is it one you installed yourself/base metal? I'm not super familiar with lower level k8s things but maybe there is a helm chart or something that runs a kind of stress test for the networking in the cluster. Thinking of memtest86 but for networking (not sure if that exists but it would be great if it did)

I'm trying this out on Azure.

Hi :wave: This is an old issue which doesn't sound like a BinderHub bug so I'm closing it. If you have more questions please use the Jupyter community forum https://discourse.jupyter.org/.


Demo Program

If possible can you add demo program

@alancting, do you have a demo app for this to share as an example?

Hi all, apologize for the late response.
I'm now working on this right now. Hopefully i can make it by this weekend. 🙏

Thank you very much, really appreciate! 🙏
On Wed, 7 Jul 2021, 11:54 Alan Si, @.***> wrote:

Hi all, apologize for the late response.
I'm now working on this right now. Hopefully i can make it by this
weekend. 🙏
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/alancting/react-cas-client/issues/7#issuecomment-875464661,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AICTYFE5XKKM7FVN3WH3CI3TWQP55ANCNFSM42SDOSDQ
.

HI all, please find the example at alancting/react-cas-client-example. Cheers!


Driver Environments - Simple Cases

I think the framework should make the simple cases simple in regards to the driver environments.  Instead of forcing the users to create their own driver environments (implementations of IDriverEnvironment), out of the box, we should provide at least the basic drivers with no settings.
For instance, I would like to see:  (ignore the fact that the browser is not the best one...)
var session = new Session(new InternetExplorer());
var session = new Session();
This should work for the major browsers:  Internet Explorer, Chrome, Firefox, PhantomJS, etc.

Implemented as part of 1.0.10 release.

Could we also add the ability to have it maximize the window and default an implicit wait (say 5 seconds) and allow those who use it to pass a TimeSpan if they want to override that period?

This has been added as part of the 1.0.11 release.


DRILL-5880: Marking as failed

@agirish please review

@cchang738
This PR disables 3 failing tests related to DRILL-5880. Please review

+1. Please add comments to DRILL-5880 that these three tests need to be enabled after bug fix. Thanks.

+1


MARKDOWN!!

I think it would be so cool if this could render markdown in the terminal. It is kind of hard to read some of the stuff, and I would love if it could use something like glow or mdcat. Thanks!

just use on of the existing UIs? they all can show proper markdown.

@sebastian-philipp  Not as efficient. What if i dont need a whole UI?

You can create a simple Python script like this
import subprocess
import re
import sys

command = sys.argv[1:]

def colorize_markdown(text):
    # Define markdown syntax patterns and their corresponding color codes
    patterns = {
        r'\*\*(.*?)\*\*': '\033[1m' + r'\1' + '\033[0m',  # Bold
        r'\*(.*?)\*': '\033[3m' + r'\1' + '\033[0m',  # Italic
        r'`(.*?)`': '\033[4m' + r'\1' + '\033[0m',  # Inline code
    }

    # Apply each pattern
    for pattern, color_code in patterns.items():
        text = re.sub(pattern, color_code, text)

    return text

# Start the subprocess with line buffering
process = subprocess.Popen(command, stdout=subprocess.PIPE, bufsize=1, universal_newlines=True)

# Loop over the lines from the subprocess's stdout
print()
for line in process.stdout:
    line = line.strip()
    print(colorize_markdown(line))

# Wait for the subprocess to finish
process.wait()

Look at how I use it: https://github.com/esteban-elias/markdown-ollama

Would love some flag when running that enables it (or perhaps disables it)
ollama run llama3.2 --md

YESSSSSS 100000%

I think this might be refactoring displayResponse and some other minor changes. Anyone have any insight before I attempt a PR?


[CI] Relocate netlify script to /docs directory

@nehaLohia27 can we have this file https://github.com/meshery/meshery/blob/master/script.sh moved to the /docs directory? // @Nikhil-Ladha
Originally posted by @leecalcote in https://github.com/meshery/meshery/discussions/3210#discussioncomment-944626

Hi @nehaLohia27, will you consider taking assignment of this issue? // @Nikhil-Ladha

@leecalcote Can you give me write access to review PRs?

@includestdlibrary Thanks for the PR.

@leecalcote Can you give me write access to review PRs? PR #3365 is still open . I approved the changes also.

Write access is limited to Maintainers only :)

okay :)


Prototype supporting file info

Uses same StringLit as Print
Do not merge yet, this is just a prototype.
This PR provides fileinfo as double quoted strings before circuits, modules, ports, and statements. I think we should discuss how we want file info to actually look. In this current implementation, file info goes before the firrtl text and it looks something like this:
<EMAIL_ADDRESS>circuit Test :
<EMAIL_ADDRESS>module Test :
<EMAIL_ADDRESS>input clk : Clock
<EMAIL_ADDRESS>input reset : UInt<1>

<EMAIL_ADDRESS>reg x : UInt<32>, clk with :
      reset => (reset, UInt(0))

<EMAIL_ADDRESS>x <= add(x, UInt(1))

<EMAIL_ADDRESS>when eq(x, UInt(20)) :
<EMAIL_ADDRESS>stop(clk, UInt(1), 0)

Personally I find that unreadable. An alternative would be to put the fileinfo on the previous line:
<EMAIL_ADDRESS>circuit Test :
<EMAIL_ADDRESS>  module Test :
<EMAIL_ADDRESS>    input clk : Clock
<EMAIL_ADDRESS>    input reset : UInt<1>

<EMAIL_ADDRESS>    reg x : UInt<32>, clk with :
      reset => (reset, UInt(0))

<EMAIL_ADDRESS>    x <= add(x, UInt(1))

<EMAIL_ADDRESS>    when eq(x, UInt(20)) :
<EMAIL_ADDRESS>      stop(clk, UInt(1), 0)

Still difficult to read, but with syntax highlighting it might be okay. I think putting the file info after the firrtl text is better:
circuit Test<EMAIL_ADDRESS>
  module Test<EMAIL_ADDRESS>    input clk : Clock<EMAIL_ADDRESS>
    input reset : UInt<1><EMAIL_ADDRESS>
    reg x : UInt<32>, clk with<EMAIL_ADDRESS>      reset => (reset, UInt(0))

    x <= add(x, UInt(1<EMAIL_ADDRESS>
    when eq(x, UInt(20<EMAIL_ADDRESS>      stop(clk, UInt(1), 0<EMAIL_ADDRESS>
With proper syntax highlighting I think this is the most readable.

Yeah, I think after is probably the best

We should also discuss how we want the file info to look. I used string literals because it was the easy to implement using the existing StringLit implementation, but perhaps this is not the right thing to do. We could use brackets<EMAIL_ADDRESS>and colon as the conventional "append" [TestFileInfo.scala<EMAIL_ADDRESS>Also, what characters should be allowed in file info? The same characters as StringLit? Should people be able to escape tabs, and eventually escape any arbitrary character? Currently string lit just supports a very restricted subset of ASCII since I thought it would be better to be strict now and add support later rather than the other way around.

Replaced by #166


Viewing method implementation for $ and $$ doesn't work as expected

I love this feature of being able to view the implementation right in the documentation!
There is a bug where the documentation for $() is actually showing the implementation for $$(), and the documentation for $$() shows undefined.
This is because the code that finds the source looks for the method name on the $ object and the $.sources object.
This works fine for all of the other methods, but for these in particular, it doesn't work:
$["$"];  // => The $.$() method, also known as $$()
$["$$"]; // => Undefined

Gah, yeah, I should special case those. Thanks for the tip!!


CM-311 Switch expression with multiple cases with a same result in C#

pull request closed to check for edited code and make a new PR