No way for HandlerFunc to know if the query came over TLS

I know this may be a corner case but I want to know if the ResponseWriter LocalAddr()/RemoteAddr() went over TLS or not.
I can't see a way to get to that information as Server.Net isn't really accessible in that callback and you can map multiple server against the same mux.
Only possible way to do this that I can see is to add a function (for example) ServerNet to ResponseWriter interface (or something similar).
Or maybe any of you have a better idea?

Why should it care?
Does net/http from the std lib have a way to do this?

Does net/http from the std lib have a way to do this?

From what I can see, yes because you can pull it from Request.URL.

Nevermind, found a way.

Happy to be pointed to some code so I can paste it in this issue for
prosperity's sake
On Fri, 10 Jun 2022, 14:47 Jerry Lundström, @.***>
wrote:

Closed #1380 https://github.com/miekg/dns/issues/1380 as completed.
—
Reply to this email directly, view it on GitHub
https://github.com/miekg/dns/issues/1380#event-6785646755, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AACWIW33BFMWI73BNC6BIWLVOM2PJANCNFSM5YNQZS2A
.
You are receiving this because you commented.Message ID:
@.***>

That code is not public but it was kinda easy (and a bit shameful I thought
of it after creating the issue), just map ip:port:tcp/udp to the real
protocol after creating the listener then use that for lookup using
ResponseWriter's LocalAddr().

@jelu couldn’t you use theConnectionState introduced in #728 ?

@chantra Oh hmm, yeah I've could have used that if I wanted to know if it was TLS or not and probably look at negotiated ALPN for DoT vs DoQ, but now with the mapping I can check for any protocol as long as they don't share port and IP protocol :D


RFE: a !welcome command to quickly welcome newcomers

The Fedora Join SIG currently has the "welcome to Fedora" process in place. We:

add newcomers to the pagure fedora-join group
create a new "welcome to Fedora" ticket for them

It's documented in the repo readme here:
https://pagure.io/fedora-join/Welcome-to-Fedora
Currently, when a newcomer comes, one of us manually sends this sort of message to welcome them:

Hello ! Welcome to the community! We have a "welcome to Fedora" process in place to help newcomers quickly learn where/when/why/what/how the community does things. Should we start the process for you? We'll need to you tell us your Fedora account username (you can get one at https://accounts.fedoraproject.org) and then to login to https://pagure.io (our FOSS "github" where we open tickets and host code)."

Example from today:

Would it be possible to create a !welcome command that prints this out please? We'll still create the ticket etc. manually, but this will just make it a little easier for us to welcome folks (instead of us currently typing this by hand). The one argument would be the Matrix username:
!welcome <a matrix username>
zodbot: Hello ...

I guess in the future, we could also extend this to create the ticket etc., but that's not a necessity at the moment.

Hi @ryanlerch , thanks for that.

I think we can limit this to the Fedora Join channel on matrix since we do want to funnel newcomers to the SIG.

Yes, it's probably best as a hidden command. I don't see it being used generally by everyone.


V2 increase polling rate for refreshing display for snigle file scripts

Expected Behavior
faster reload after save.
Actual Behavior
slower reload than V1
Steps to Reproduce the Problem

open single file script
change script
wat for refresh

Specifications
I have asked for similar thing but more complicated regarding usage of livereload.
This request is different. Although for folders and big projects, polling rate should be carefully picked
to avoid spamming the system this is not the case for single file projects.
A different(faster) polling rate should be used for single file projects :)

Version: V2
Platform: all
Environment: (browser)

@hrgdavor probably not what you actually want, but the delay intervals are now dynamically calculated. for small projects, the reload time should be about 1 second. how's that?

yes, dynamic delay is better, and livereload support is underway :)


Add logging

Closes https://github.com/ISISComputingGroup/ibex_bluesky_core/issues/45

Adds logging throughout the library
Adjusts some of the logging setup code as it was accidentally capturing messages from root logger and not quite adhering to normal logging idioms in some places. Also made easier to adjust log levels on the fly in e.g. an interactive session.

To review:

Check logging statements generally make sense
Run a representative scan with INFO logging (default), check you get a reasonable set of messages in the log file.

Reasonable = not too many, not too few - tricky balance but should generally be logging the "main" things that happen during a scan.

Run the same scan with DEBUG logging on all of bluesky/ibex_bluesky_core/ophyd_async (see added docs for how to do this), check that you get much more detailed debug logging which would help to diagnose tricky low-level issues.
Check you don't pick up messages from other loggers (e.g. run logging.getLogger("foo").warn("hello") in pydev console, check that doesn't end up in the bluesky log.
Check tests make sense etc

ibex_bluesky_core needs to be released after merging this so that system tests pick up the changes.

I have completed the review however merging is blocked as the last push was done by me. It now requires a different reviewer to complete the merging business


Unit testing

some way to do unit testing,

all object types
TADIR entries are properly created/deleted
Objects recorded in transports properly
Coverage

okay for now, closing


TimePicker - numbers are in wrong position

Problem description

Steps to reproduce
git clone https://github.com/callemall/material-ui.git
cd /material-ui/examples/webpack-example
npm install
Add TimePicker to app/Main.js as i add here Main.js.txt
Versions

Material-UI: 0.15.0
React: 15.0.1
Browser: Chromium 50.0.2661.102 Ubuntu 14.04 (64-bit)

On Firefox, Google Chrome and Opera its working fine.

#4230


add krustlet to vhd + cse_install

note that this is not invoked in cse_main

passed ci except windows which should be no-op, merging.


Name of the the button "Add custom name" and label in modal window are not the same on Dataset page

Set up
https://platform.dev.odd.provectus.io/dataentities/24027406/overview
Steps to Reproduce

Login with valid data
Go to the Catalog page
Open Datasets
Open some dataset from the list
Tap on "Add custom name" button
Pay attention to the name of the label on the modal window that appears

Expected result
The name of the button and label should be the same. Need some discussion about which exact name should be
Actual result
The name of the button and label are different.
Attachment
https://recordit.co/yuwg1biVFQ

@Anna-EG

@DementevNikita @nstkvl
When we enter a new name, it replaces the original one. So "add" the name is not exactly what we do here. I would like to suggest "Edit name" for the button instead of "Add custom name" and "Edit datasource name" as a title for the modal window.

For button will be "Edit name"
For title of the modal will be the same, because user can edit not only datasource names, but also the names of entities of all types.


Generate multiple galleries from multiple folders of photos

would be great and useful feature!
in this context it would be also worth to think about a non interactive way with a config file instead. This would help to build up automated workflows.
Indeed I already build a simple wrapper around your library to achieve this and used such an ini file:
[DEFAULT]
gallery_root = /home/user/gallery

[Mexico 2017]
description = Una cerveza por favor
image_source = /home/user/Pictures/Mexico2017
background_photo = 2017-11-01_16-09-11.jpg
background_photo_offset = 30

[Dublin 2017]
description = One more Guiness, please
image_source = /home/hille/Pictures/Dublin2017

Yeah, this would be important as well! I have ideas how to address these issues for a major overhaul planned with version 2.0. I need to find time to work on this, though...


Consider removing excerpt, content, & thumbnail from PostItem type

From 2.20.23 slack feedback:
i think excerpt, content, and thumbnail can be removed from the PostItem type? this is all being parsed and will be available in PostContents

Yep, its in the markdown front matter now.


Dataframe functionality: filter dataframe view

The view is derivated from a dataframe via column/row filters.

By default, the view is readonly; if read/write is allowed, the dataframe will create a copy of the underlying dataset and then modify the data.

read/write/filter api similar to pandas df

view:

view inherite from the DataFrame abstract class.

has any number of filters, each filter is associated with a field. The field can be from any dataframe.

the view is rw by default, thus the data changed is traced back to the original field
if the view is readonly, than change of data will trigger a behaviour of copy the referenced field into a new dataframe


Investigate context-sensitive actions

As an Algernon enthusiast, I want to find context-sensitive actions, in order to speed-up my workflow.
Acceptance Criteria

[ ] Research possible solutions & issues
[ ] Build Spike if feasible

I couldn't find anything obvious, I am gonna ask on slack


Rack version bump to 2.2.3 in both main and test-app

Signed-off-by: Andy Tinkham<EMAIL_ADDRESS>What does this PR do?
Bumps rack in both the main Gemfile/Gemfile.lock and the test-app Gemfile.lock to v2.2.3 to close CVE-2020-8184 and CVE-2020-8161 (neither of which directly impacted us).
What ticket does this PR close?
Checklists
Change log

[X] The CHANGELOG has been updated, or
[ ] This PR does not include user-facing changes and doesn't require a CHANGELOG update

Test coverage

[ ] This PR includes new unit and integration tests to go with the code changes, or
[X] The changes in this PR do not require tests

Documentation

[ ] Docs (e.g. READMEs) were updated in this PR, and/or there is a follow-on issue to update docs, or
[X] This PR does not require updating any documentation

Something in puma 3.12.6 breaks the tests for this, so I pulled that version bump back out.


Tre new layout

WIP*: Merge upstream v0.2.0 and add Tampere customizations
Merge upstream changes up until v0.2.0 release to tre branch which is the active Tampere specific development branch.
On top of upstream changes, small Tampere specific customization work is done to meet the Tampere brand requirements.
Merging this will also be the end of the old map based UI that Tampere has used and a move to the same UI that Helsinki and other cities use.

Duplicate of #4 and these changes are already integrated to tre branch in #6. Thanks @JsseL and @blomqma! :+1:


db2mon script throwing error in Ubuntu 20.4 intermiddently

/home/db2inst1/sqllib/samples/perf/db2mon.sh: 49: [: 60: unexpected operator
https://github.com/IBM/db2-samples/blob/master/perf/db2mon.sh

Good catch.  I am surprised you're saying this is only intermittent.  At first glance I suspect the problem is that the == operator isn't technically correct /bin/sh syntax and should just be = instead.
If I'm not mistaken that version of Ubuntu links sh to dash and because of that only implements POSIX compliant  sh syntax, which == isn't.  It's fairly common in other distros to link sh to bash instead, which is why it is usually just fine, since that is acceptable bash syntax.
You could probably easily try that out by making that change locally for yourself.  I'll certainly be curious if that works for you.  You're even welcome to submit a PR to correct it but I'll also work to get that addressed on our end.


Feature Request: Expand/collapse all categories on Budget page

A way to expand all or collapse all categories in the budget display.
(I thought this already existed but can't find it in the settings. Maybe it was a feature in YNAB 4?)

It's still there but has been merged with another feature named Category Solo Mode and Toggle Master Categories which is a drop down option. I recommend you select the Enable Both selection which gives you the feature you're looking for and the new feature which allows you to have just one category open at a time.
I'm closing this Issue as it isn't a defect. Feel free to reopen it if you disagree.

Thanks for the explanation for where to find the feature I wanted.  It's
working great!
Thanks too for closing the issue.
Cheers,
Janice
On Wed, Dec 4, 2019 at 11:51 AM David Baldon<EMAIL_ADDRESS>wrote:

It's still there but has been merged with another feature named Category
Solo Mode and Toggle Master Categories which is a drop down option. I
recommend you select the Enable Both selection which gives you the
feature you're looking for and the new feature which allows you to have
just one category open at a time.
I'm closing this Issue as it isn't a defect. Feel free to reopen it if you
disagree.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/toolkit-for-ynab/toolkit-for-ynab/issues/1730?email_source=notifications&email_token=AASLL43EYEIZ6CGYE2TMKKTQXAC33A5CNFSM4JVNRZDKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEF6JA7A#issuecomment-561811580,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AASLL43OK2IR6BBL556SVNLQXAC33ANCNFSM4JVNRZDA
.


Potential in mesh_navigation for staircase climbing

Hi @mklpiening,
Just wanted to firstly express my gratitude for this new package. It really makes 3D navigation much more accessible to ROS users like me.
I would like to enquire on the potential for using mesh_navigation on multi-level navigation, for instance, in a building where humans would normally ascend or descend via staircases? Correct me if I am wrong, but it seems to me that there cannot be multiple map layers on the same map (when it comes to mapping an entire multi-level building). In addition, I am not sure if we could send a command pose that has a Z (height) component?
Looking forward to hearing from you, thanks!

I would like to enquire on the potential for using mesh_navigation on multi-level navigation, for instance, in a building where humans would normally ascend or descend via staircases? Correct me if I am wrong, but it seems to me that there cannot be multiple map layers on the same map (when it comes to mapping an entire multi-level building).

This is possible if the 3D map (triangle mesh) represents that building! We call this multiple level (multi story building). We use the term multi-layer for multiple cost-layers defined on the mesh surface.

In addition, I am not sure if we could send a command pose that has a Z (height) component?

This is possible too! With the mesh tools package bundle we provide a goal selection too for picking a goal on the mesh surface which of course also uses the z value if you, for example, pick a goal on the second floor, in your example.
Cheers, Sebastian

I would like to enquire on the potential for using mesh_navigation on multi-level navigation, for instance, in a building where humans would normally ascend or descend via staircases? Correct me if I am wrong, but it seems to me that there cannot be multiple map layers on the same map (when it comes to mapping an entire multi-level building).

This is possible if the 3D map (triangle mesh) represents that building! We call this multiple level (multi story building). We use the term multi-layer for multiple cost-layers defined on the mesh surface.

In addition, I am not sure if we could send a command pose that has a Z (height) component?

This is possible too! With the mesh tools package bundle we provide a goal selection too for picking a goal on the mesh surface which of course also uses the z value if you, for example, pick a goal on the second floor, in your example.
Cheers, Sebastian

Regarding stair cases: If the current stack of levels and the current planners could not handle this planning problem well enough, you could develop new layers plugins and / or a new planner plugin.

Regarding stair cases: If the current stack of levels and the current planners could not handle this planning problem well enough, you could develop new layers plugins and / or a new planner plugin.

Thank you @spuetz , I am excited to give this a shot! Can't wait for the dynamic obstacle avoidance feature.

Thank you @spuetz , I am excited to give this a shot! Can't wait for the dynamic obstacle avoidance feature.


Bump serde and serde_json to 0.7.

There were breaking changes in 0.7.0 so this is needed for mime to work with programs that also use Serde 0.7 themselves.

Thanks!

Thank you! Mind publishing a new version so I can update the body-parser PR?


No refresh worker logs when running Cube in production mode

Hi team,
Recently I switched my Cube setup to production mode, and noticed that my pre-aggregations were not getting updated as expected.
There were no logs whatsoever on either the API instance pod or refresh worker pod. As soon as I switched back to dev mode, logs were coming from refresh worker pod.
Is this the expected behaviour, because it would be quite difficult then to debug issues while running Cube in production mode.
This is my values.yaml for Cube
cube:
  image:
    repository: cubejs/cube
    tag: v0.34.40
    pullPolicy: IfNotPresent

  datasources:
    default:
      type: snowflake
      user: CUBE_DEV_USER
      pass: <PASSWORD>
      schema: PUBLIC
      name: UC_DATA_DEV
      snowFlake:
        account: <ACCOUNT>
        region: ap-southeast-1
        role: <ROLE>
        warehouse: CUBE_POC_WH

  cubestore:
    host: cubestore-dev-router.cubestore
    port: 3030

  config:
    preAggregationsSchema: pre_agg_schema
    debug: true
    logLevel: "debug"
    devMode: false
    rollupOnly: true
    pgSqlPort: 15432
    sqlUser: admin
    sqlPassword: admin
    volumes:
      - name: cube-schema
        configMap:
          name: cube-schema
      - name: cube-config
        configMap:
          name: cube-config

debug is set to true
logLevel is set to "debug"

@lvauvillier

Hello @anshjain18, we're using a similar configuration and everything is working fine for us.
Could you try adding cacheAndQueueDriver: cubestore to your configuration section? In development mode, the default setting is memory. This change might generate logs that could help identify the error.

Hi @lvauvillier, thanks would add this.
Though I can see that the default value for this setting is 'cubestore' only.

Yes, the comments in the chart above the value fields may become outdated, as the Cube documentation is frequently updated.
The best is to refer to the official documentation. Most of the environment variables has a direct mapping in the chart values
For example, regarding cacheAndQueueDriver, you can refer to:
https://cube.dev/docs/reference/configuration/environment-variables#cubejs_cache_and_queue_driver


Updated chipgroup and chipgroup tests

Updated chipgroup.py and test-chipgroup.py for the pf4 changes, but retained various features to maintain backwards compatibility.

@digitronik take a look at this?

@rsnyman @jjaquish-rh thanks :tada:


s360 fixes for MixedReality 2020-05-01 and 2020-04-06-preview swagger

MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Changelog
Please ensure to add changelog with this PR by answering the following questions.

What's the purpose of the update?

[ ] new service onboarding
[ ] new API version
[ ] update existing version for new feature
[x] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When you are targeting to deploy new service/feature to public regions? Please provide date, or month to public if date is not available yet.
When you expect to publish swagger? Please provide date, or month to public if date is not available yet.
If it's an update to existing version,  please select SDKs of specific language and CLIs that require refresh after swagger is published.

[ ] SDK of .NET (need service team to ensure code readiness)
[ ] SDK of Python
[ ] SDK of Java
[ ] SDK of Js
[ ] SDK of Go
[ ] PowerShell
[x] CLI
[ ] Terraform
[ ] No, no need to refresh for updates in this PR

Contribution checklist:

[ ] I commit to follow the Breaking Change Policy of "no breaking changes"
[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

[ ] Ensure to check this box if one of the following scenarios meet updates in the PR, so that label “WaitForARMFeedback” will be added automatically to involve ARM API Review. Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs, all “removals” and “adding a new property” no more require ARM API review.

Adding new API(s)
Adding a new API version
Adding a new service

[ ] Please ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If there are following updates in the PR, ensure to request an approval from Breaking Change Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in stable version
[ ] Removing properties in stable version
[ ] Removing API version(s) in stable version
[ ] Updating API in stable or public preview version with Breaking Change Validation errors
[ ] Updating API(s) in public preview over 1 year (refer to Retirement of Previews)

Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.
Please follow the link to find more details on PR review process.

@roytan-microsoft The Track2 Validation errors look like there is more than one CloudError Schema brought into these api versions.  If you use the CloudError definition from common-types: /specification/common-types/resource-management/v1/types.json, this should be resolved.


Vue devtools crashes chrome

Vue devtools version
latest
Link to minimal reproduction
.
Steps to reproduce & screenshots
when you open it with a vue2 project, it will crashes chrome, which is unacceptable.
What is expected?
ok
What is actually happening?
chrome crash
System Info
System:
    OS: macOS 14.2.1
    CPU: (10) arm64 Apple M1 Pro
    Memory: 6.56 GB / 32.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 20.10.0 - /usr/local/bin/node
    Yarn: 1.22.18 - /usr/local/bin/yarn
    npm: 8.11.0 - /opt/homebrew/bin/npm
    bun: 1.0.1 - ~/.bun/bin/bun
    Watchman: 2<IP_ADDRESS> - /opt/homebrew/bin/watchman
  Browsers:
    Chrome: 120.0.6099.234
    Safari: 17.2.1

Any additional comments?
No response

Could it be related?

https://github.com/vuejs/devtools/issues/2103

Sounds similar to what I'm seeing - left a comment also on https://github.com/vuejs/devtools/issues/2103

Duplicate of #2103


Reuse LocalAlluxioCluster to speedup REST tests

Merged build finished. Test FAILed.

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

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/6884/
Test PASSed.

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/6896/
Test PASSed.

(Please update the PR description before merging)

alluxio-bot, merge this please


Fix 3.0

This PR is intended to fix #36.
I've restored the original directory structure, but put the translations into device/ to be able to add them from the editor. I also added the missing lightmap to the test scene and made a change to achievements.gd, since has_singleton() is no longer a valid function, though I'm not sure if has_setting() is equivalent.
I have not touched the scripts in tools/, and I'm not sure if they have been tested with Godot 3.0 yet.
EDIT: I will squash the commits before it's considered for merge. Not 100% sure what happened there.
EDIT 2: I'm actually not sure how to solve this. My local commit log looks fine. I tried to rebase this branch onto the last commit of master, but git is putting these old commits in before that one, so it just explodes.

After looking into it, I'm pretty sure this happened because I merged the changes from upstream into my fork, instead of using force pull/push to get it in sync.
I think I can fix this by cherry picking the last two commits from this branch onto master (after fixing its commit history) and then just squash them into one and then force push the branch to my origin/fix-3.0.

Great job !


Mjolnir trace IR, Constants support, better type routing

Use the Mjolnir package's trace function to get our forward IR and leverage its type inference and constant propagation.
Support inverting primitives that have a constant as an argument -> no dependence on a parameter.
Route the types of variables to better match the function method. For example:
before we had invert + where output is a Float64.
Now, we say invert + where input is Float64, Int64.
Minor changes to primitives to reflect these changes.
Corresponding test for constants

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@8b29a4d). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master       #9   +/-   ##
=========================================
  Coverage          ?   55.86%
=========================================
  Files             ?        7
  Lines             ?      145
  Branches          ?        0
=========================================
  Hits              ?       81
  Misses            ?       64
  Partials          ?        0

Continue to review full report at Codecov.

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


Wandering Trader Spawns Lag Server

Describe the bug
A clear and concise description of what the bug is. Also include any error messages
The spawning of wandering traders seems to take  up a lot of server resources.
To Reproduce
Steps to reproduce the behavior:
When I test wandering trades I have a contraption that spawns 15 traders at a time so I can get a good idea of the trades and ensure the heads are all present in the config. So spawn 15 traders at a time through dispensers and redstone.
Expected behavior
You should notice tps drop a few points and the traders take a while longer to die in lava with some visual "hiccups"
Screenshots
If applicable, add screenshots to help explain your problem.

Before Spawning

After Spawning

Additional context
Add any other context about the problem here.

Fixed in the next version 🙂


agent not respecting schedule?

Occasionally my agents report that they have not checked for 22-25 minutes despite thir schedule being every 10 or five minutes. what could cause this? thanks
image from the agents view
http://imgur.com/a/PqPyZ

Hi @JesseJams,
is it possible that many agents are queued to run and thus delay the execution of that agent? You can check that in the "Account -> Job Management" page.

Hi Dsander, That was indeed the issue, a bunch of jobs queued up.
Thanks!


Update patch_csv.sh to work with bundle images

The bundle image has no runtime so docker run fails.
This patch does not address all the steps required to
process bundle images.The csv will need to be renamed
manually to include the version information.
Signed-off-by: N Balachandran<EMAIL_ADDRESS>

/lgtm


Add pipeline to $lookup stage

Q
A

Type
feature

BC Break
yes

Fixed issues
#1863

Summary
Added a pipeline lookup functionality that helps a lot in the futures developments,

I discovered that "foreignField" and "localField" is not mandatory when using "pipeline" in the $lookup stage. In fact it dropped the performance in our case so we had a query running for about 4 seconds. Without foreignField and localField the query runs in 250-300ms and the result set was the same.
Also it would be great if you can add an option to set variables using "let".

@Ravetracer Thanks for your comment. Changes pushed to PR.

@khaperets thanks for the PR! It seems that the test you've added is failing: https://github.com/doctrine/mongodb-odm/actions/runs/3067090011/jobs/4971970883#step:12:1783. The rest of errors seems to be unrelated, we'll investigate them. Once the test is green please fix coding standards, this can be done by running vendor/bin/phpcbf command. If possible, please also squash your commits so there's just one.

@malarzm I've done everything, thank you for the feedback.

@khaperets we've managed to get checks back to shape. May I ask you to rebase your PR atop newly created 2.5.x branch?

@malarzm done

@malarzm Added compatibility with MongoDB 5.0

Merged manually in https://github.com/doctrine/mongodb-odm/commit/7ebc038973587222c89de11cad4367c2185776f9. Thanks a lot @khaperets!


Crashed whit command "/time set" on server

Crashed whit command "/time set day" on server

Can you attach the loga from the crash? Should be in the file latest.log

I uploaded a possible fix of the problem (i think).
I can't tell for sure if the same issue affected you unless i get more details.


Player times out before joining any server

Whenever I try to join a server through my locallly hosted geyserconnect, instead of joining the server, it times out after around 5 seconds. Looking at the logs while I'm joining shows geyserconnect beginning to start geyser, then the player timing out, then geyser starting after 10 seconds. This is probably caused by me not running it on the most powerful pc, but there should still be a way to not time the player out until geyser has actually started.

I just pushed a change that should hopefully fix that.
https://github.com/GeyserMC/GeyserConnect/commit/5c439cb2a358db871be4c1bc88ac1e79c71b9a20


Check for at least one non-zero bytes for buildin account

Problem
Alternative for https://github.com/anza-xyz/agave/pull/298
check for at least one non-zero bytes in the first 8 bytes for buildin account.
Summary of Changes
Fixes #

We no longer need this fix because we have reverted deprecating executable feature from v1.18.
https://github.com/anza-xyz/agave/pull/337


[Homework] Make Shader API code independent to OpenGL code.

At the end of one of the episodes The Cherno have given us a "homework" which is to seperate the OpenGL code from Shader class to make it platform independent. Since I haven't seen him touch that topic for a few episodes I think I should finish the homework and keep Shader in line with the other components.
Also this fix the problem mentioned in PR #72: added a missing semicolon in LayerStack.cpp

Sorry for the spaces - tabs mismatch.
I'm using 4 spaces indent in my local workspace, and everything should be formatted (indent) correctly. The mismatch may be because of me editing files directly on github web, which the editor is set to tabs by default, and I didn't noticed that.

As mensioned in #52, premake could resolve this.

Just a note to myself and maybe to cherno: A place on the github wiki; one page that list a possible solution for the homework assignment. So others can make a pull request (with homework flag) to propose a change, and a good answer can be picked to put up in the wiki for future references

As The Cherno have implemented Shader abstraction, this PR become obsolete and need to be closed.


Remove some unneeded deps

With Heroku-20 stack, we are exceeding the 500 MB dyno slug limit, so
we try to remove some deps here

Just got our limit to be 550 MB after talking to Heroku support, so going to close this


[Question/Feedback]: Is the Azure Monitor Alerts for quotas in the AMBA roadmap?

Check for previous/existing GitHub issues

[X] I have checked for previous/existing GitHub issues

Description
Will the azure monitor alerts for quotas be included in AMBA? https://learn.microsoft.com/en-us/azure/quotas/monitoring-alerting

Hi @dmrellan we are actively speaking with the team who are working on the quota feature for Azure and discussions are ongoing , nothing committed yet


Accept credential Offer is not working.

previously it was fine we just needed to pass cred_ex_id of the credential offer but now it is not working.
it expects some kind of options attribute i figured it out it needed some extra things that credential doesn't contain.
It would be great if it can be used as we used in the previous version.

I thought the problem was with accept offer rather than accept proposal

I would think you would want to take care with taking the cred_def_id from the proposal.  That data element is from the holder and may not align with what the issuer is able/willing to offer.  Obvs if the cred-def-id is not one the issuer owns, they can't issue such a credential, so that check is a minimum.

The method we're talking about is acceptProposal (thus indicating we want to take over the values from the proposal). If you don't want to take these values from the proposal you can either start from an offer (offerCredential) or create a counter offer (negotiateProposal)

I thought the problem was with accept offer rather than accept proposal at least that's what the title of this issue says

You're right, I misread. This is probably another issue then. @ChSonendra can you add some more context on the issue? Looking at the API you should be able to call it like this:
    await this.agent.credentials.acceptOffer({
      credentialRecordId: credentialRecord.id,
    })

okay so when we have a credential offer and if we try to accept it by calling
so previously we used call this function as   await agent.credentials.acceptOffer(cred_ex_id);
but in recent version that alpha-92 we have another problem this time this function is expecting another type of argument that is options "async acceptOffer(options) { function_body }" and this option argument should contain three fields that is comment, autoAcceptCredential holderDid and credential_exchange id. ideally it should work fine with cred_ex_id as it worked before. even if we pass everything we are getting this error  [AriesFrameworkError: Could not determine errorName of indyError undefined].
but i really and everyone would like to accept the credential offer just by calling acceptOffer with cred_ex_id.

Looking at the code, the acceptOffer method takes one argument in the credentials module:
  public async acceptOffer(options: AcceptOfferOptions): Promise<CredentialExchangeRecord> {
}

And AcceptOfferOptions only requires a single attribute - credentialRecordId.
My understanding is that this was changed because credential definition id is an Indy specific concept, and this API is now used in V2 by non-Indy users.

Now I notice in that method we have some new code that was added last week to fix a bug with the Indy DID:
  const indyDid = getIndyDidFromVerficationMethod(verificationMethod)

      const requestOptions: RequestCredentialOptions = {
        comment: options.comment,
        autoAcceptCredential: options.autoAcceptCredential,
        holderDid: indyDid,
      }

      const { message, credentialRecord } = await service.createRequest(record, requestOptions)

Could there be an issue somewhere in this code?

I am debugging this piece of code and will tell the outcomes here shortly.

@NB-MikeRichardson
I passed this parameter to agent.credentials.acceptOffer() parameter
const options = {
credentialRecordId : item.id,
comment: "accept offer",
autoAcceptCredential : true
}
and this thing worked fine and state is set to
credential-received at holder and
credential-issued at issuer's end.
So the problem now is it is not completing the flow that is sending a acknowledgement after receiving the credential.
for confirmation of the issue i tried it many times even with a fresh app, but its not working.

closed by mistake.

@ChSonendra I think the autoAcceptCredential is not a boolean but takes an enum:
export enum AutoAcceptCredential {
  // Always auto accepts the credential no matter if it changed in subsequent steps
  Always = 'always',

  // Needs one acceptation and the rest will be automated if nothing changes
  ContentApproved = 'contentApproved',

  // Never auto accept a credential
  Never = 'never',
}

Can you try that?

Thankyou @NB-MikeRichardson  issue is now resolved and we have to pass
const options = {
credentialRecordId : cred_ex_id,
comment: "accept offer",
autoAcceptCredential : "always"
}

this kind of argument to acceptOffer instead of just a cred_ex_id and it works fine.
So thank You once again.

@ChSonendra what have you set for autoAcceptCredential on the agent config?

@TimoGlastra Maybe we should add a warning log when AutoAcceptCredential.Always is set at on the agent config. Like
autoAcceptCredentails is set to "Always". It is not recommended for production use, and should only be used for development.
A warning might be a bit too high, so maybe an info log.

@TimoGlastra  Right, So Issuer can change the content at the time of issuing of credential, thanks for mentioning this. I will try to make space to add an option for asking user to autoAcceptCredential in the App itself.


[OBS Control] Start virtual camera

Extension – OBS Control
Author:
It would be great if the plugin would have a command to toggle (start or stop) the virtual camera as well.
Describe the feature and the current behavior/state.
Currently the plugin only toggles the stream
However I always use the virtual camera so adding a command to toggle the virtual camera would be great.
Who will benefit with this feature?
People who use obs in other ways. e.g. for video conferences were an external camera input is fed through obs and then into ms teasm, slack, etc.
Any Other info.
The obs-cli supports this feature, so I think it should be available.

@pernielsentikaer any chance of this being added?

I think we have to ask @Yukaii as the author if he could look into this 🙂 I'm not really into OBS myself 🫤

It should be pretty straight forward to do. So we need @Yukaii to react? I am happy to send a PR to add this feature.

Well, @Yukaii is the author which needs to accept the incoming PRs but it seems like a good feature, I don't see a problem making it better 🙂


Concatenate multiple FMUs with NN

Problem Description:
I want to know if FMIFlux can currently chain multiple FMUs and NNs together. For example, I have constructed a circuit system in OpenModelica that includes a voltage source and a resistor. Using OpenModelica, I split this system into three subsystems (voltage source, resistor, and ground), and exported each as an independent FMU. I then tried importing these three subsystem FMUs into OpenModelica and connecting their inputs/outputs for simulation. Unsurprisingly, the simulation results were the same as the original circuit system.
Diagram as follow, from left to right, it sequentially shows the Voltage Source FMU, Resistor FMU, and Ground FMU:

Now, I want to know two things:

Is it possible to achieve the same functionality described above in FMIFlux or other FMI packages?
Is it possible in FMIFlux to chain NNs and multiple FMUs together? For example, I want to replace the original linear resistor with an NN and use training data to simulate a nonlinear resistor.

Current Attempts:
I found that FMIFlux currently supports using Parallel to chain multiple FMUs and NNs (via Chain). However, after looking into it in detail, I discovered that the Parallel function allows multiple FMUs to run concurrently and produce results. We can then concatenate the results or perform other operations like summation. However, this is different from sequentially chaining multiple subsystem FMUs as I intend.
If there is any misunderstanding on my part, please correct me. Thank you.

Exactly, there are two to different operation modes:

"co-simulation" which uses different solvers to solve the models independently of each other and syncs signals after every time step
"model integration" which actually combines the two ODE systems (or ME-FMUs) into a single one and then solves the combined system.

As you correctly noticed, the parallel example simulates the FMUs in CS-mode.
From the text, it sounds like you want to do "model-integration", but importing FMUs in OpenModelica and simulate them together sounds like co-simulation to me....
Can you specify:
(a) if you want to operate in CS or ME mode?
(b) if ME: if you want to use the same solver for both FMUs (model integration) or different solvers for every FMU.
(c) do you want to connect states/state derivatives or inputs/outputs
In any case I think there is a solution for your problem, but it is easier to discuss if we further specify.
Best regards!
PS: I convert this to discussion :-)


Unset ANDROID_HOME in workflow to fix Android SDK loading error

Attempt to fix "ERROR: Analysis of target '//:test/com/google/javascript/jscomp/disambiguate/AmbiguatePropertiesTest' failed; build aborted: Android SDK api level 32 was requested but it is not installed in the Android SDK at /usr/local/lib/android/sdk. The api levels found were [35, 34]. Please choose an available api level or install api level 32 from the Android SDK Manager." - https://github.com/google/closure-compiler/actions/runs/12412367428/job/34652003730

For some additional context -
unsetting ANDROID_HOME stops some Bazel code from trying to load the Android SDK.
It seems like something changed in our GitHub CI setup, where previously ANDROID_HOME was unset & now it's set?
There might be a better fix but 1) we already did this in the past for a different build script - https://github.com/google/closure-compiler/commit/f1faba688ddcddacdcfa3df370eb40bb6848afea (for Google's internal CI) and 2) we definitely don't need the Android SDK.

Note: the Build & Test for this PR did fail, but it failed later on than the current CI runs are doing.
The problem it's hitting now is that we added a Java 21 API usage to the compiler & our Bazel Java rules aren't setup to use Java 21 yet.
But without this PR, we don't even get to the point of seeing the Java 21 error.

I'm going to import this and get it submitted.


Type narrowing doesn't work when using variables inside brackets property accessors

TypeScript Version:  3.9.2

Search Terms: type narrowing, type guard, control flow analysis
Code
const students: Record<number, {name: string | null, age: number}> = {
    1: { name: 'John', age: 5},
    2: { name: 'Joseph', age: 6},
    3: { name: null, age: 6},
};

if (typeof students[5]?.name === 'string') {
    students[5].name; // The type of name is string, correct ✅
}

const id = 5; // Type of id is 5

if (typeof students[id]?.name === 'string') {
    students[id].name; // The type of name is (string | null) 🤔
}

Expected behavior: The type of students[id].name inside the body of the second if statement should be string, without null.
Actual behavior: The type of students[id].name inside the body of the second if statement is string | null.
Playground Link: link
Related Issues:
Maybe related? https://github.com/microsoft/TypeScript/issues/9998

Duplicate #10530. Type narrowing does not occur for indexed access forms e[k] where k is not a literal.


Export smaller icons at increased weight?

One of the tricks when drawing smaller versions of icons is to increase line thickness.
So I wondered if it's possible to render the small icons at higher weight?
https://lmnt.me/blog/1024x1024.html

Good idea. I'll take a look.

I have a feeling specifying exact size for the icon will make things sharper.
24pt
rather than 23.854pt
sort of thing


Update README.md

Project URL
https://github.com/Zolang/Zolang
Category
Tools
Description
A programming language for cross platform code generation - Swift, Kotlin, Python etc.
Why it should be included to awesome-ios (mandatory)
Great for sharing logic between mobile platforms - iOS and Android - but also tools and automation scripts.
It allows for improving iOS code by removing boilerplate code that isn't platform specific.
Checklist

[X] Only one project/change is in this pull request
[X] Has unit tests, integration tests or UI tests
[X] Addition in chronological order (bottom of category)
[X] Supports iOS 9 / tvOS 10 or later
[X] Supports Swift 4 or later
[X] Has a commit from less than 2 years ago
[X] Has a clear README in English

          2 Warnings

:warning:
Please update the Pull Request title to contain the library name

:warning:
Found 2 link issues, a project collaborator will take care of these, thanks :)

Link issues by awesome_bot

Line
Status
Link

1688
301
https://github.com/VirgilSecurity/virgil-foundation-x redirects tohttps://github.com/VirgilSecurity/crypto-x

2628
301
http://holko.pl/2016/01/05/typed-table-view-controller/ redirects tohttps://holko.pl/2016/01/05/typed-table-view-controller/

  Generated by :no_entry_sign: Danger


Remove ExpressibleByArgument conformance from Optional

Description
This drops the ExpressibleByArgument conformance for Optional. This conformance doesn't quite make sense, and allowed property declarations like this:
@Option(default: 0) var count: Int?

The default value meant that count would never be nil, and therefore shouldn't be optional.
Detailed Design
This change adds initializers specifically for when an Option's or Argument's value is optional, without a default parameter as well as deprecated versions that include a default parameter to guide clients to the correct declaration syntax.
Documentation Plan
The new initializers are documented; related initializers have been updated. There is no change required in the guides.
Test Plan
Existing tests exercise the functionality and verify the largely source-compatible nature of the change.
Source Impact
While this is technically source-breaking, it should have minimal impact in regular usage. All the declaration sites for arguments and options with optional values are still supported, with deprecation warnings for declarations that combine a default value and an optional type. If a property is declared as optional but does not have a default value, there won't even be a deprecation notice.
Code that uses Optional's conformance to ExpressibleByArgument, however, will break without warning. Users can work around this by adding an Optional-specific code path, or by other means.
Checklist

[x] I've added at least one test that validates that my change is working, if appropriate
[x] I've followed the code style of the rest of the project
[x] I've read the Contribution Guidelines
[x] I've updated the documentation if necessary

@swift-ci Please test

@swift-ci Please test

@swift-ci Please test


⚠ Bofrost.be has degraded performance

In 2df9aa2, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1089 ms

Resolved: Bofrost.be performance has improved in b1e94b8.


Run task in external terminal

Issue Type: Feature Request
Just like the launch.json supports the ability to host the application, i would like to be able to host my tasks in an external console window. Some console workloads i have are to heavy for xTerm.js and the ability to accidentally kill one of those processes when killing terminal processes with the trash can button is a little bit annoying.
VS Code version: Code - Insiders 1.28.0-insider (cd9d71a31f731baf17330a84448c3efdeabc873f, 2018-09-17T05:20:06.620Z)
OS version: Windows_NT x64 10.0.17134

This will have the downside that we will not be able to capture any output and detect problem markers in the output.


KAFKA-2588 ReplicaManager partitionCount metric should actually be re…

…plicaCount

test this.

@granthenke Could you update the documentation as well?

Thanks for the patch. I'm going to close this for now since it requires a KIP, but please consider reopening if you (or anyone else) has time to submit a proposal. (The change does seem reasonable to me for what it's worth.)

…plicaCount

…plicaCount

…plicaCount

…plicaCount

…plicaCount

…plicaCount


Release binaries

Hey,
Please release binaries so noobs like me don't have to go through all the steps
Regards,
Karthik.

Good idea @leovarmak, I'll compile and generate the binaries and make it available for download in GitPages.
But you generated the binary and it worked?
Thank you!

I haven't. I was too lazy 🙈
Like usually there are binaries provided in the releases section.

@leovarmak I created the executables and made it available for download in Git Pages here mariorodeghiero.com/git-cheatsheet
Thank you for the idea.


Is there a way to start a non-autoplaying player programmatically?

See our example code and the youtube iframe api reference:
https://github.com/troybetz/react-youtube/blob/master/example/example.js#L30-L32
https://developers.google.com/youtube/iframe_api_reference#Playback_controls


Python 3 support?

With Ansible having a technical preview of Python 3 support, will this module be updated to support that as well?

Haven't investigated yet, but yes I plan to update when I get some time to test, if you get to it first please update the thread with your findings.

Test output:
(py3-ansible) [mittell:...nsible/ansible-sn-inventory]$ ./now.py | jq 'del(._meta) | keys[]'                                    (master) "esx_server" "linux_server" "os_x_server" "solaris_server" "storage_server" "windows_server" (py3-ansible) [mittell:...nsible/ansible-sn-inventory]$ ansible --version                                                       (master) ansible <IP_ADDRESS>   config file =   configured module search path = Default w/o overrides   python version = 3.6.2 (default, Jul 17 2017, 16:44:45) [GCC 4.2.1 Compatible Apple LLVM 8.1.0 (clang-802.0.42)]

Test output:

(py3-ansible) [mittell:...nsible/ansible-sn-inventory]$ ansible --version (master)
ansible <IP_ADDRESS>
config file =
configured module search path = Default w/o overrides
python version = 3.6.2 (default, Jul 17 2017, 16:44:45) [GCC 4.2.1 Compatible Apple LLVM 8.1.0 (clang-802.0.42)]
(py3-ansible) [mittell:...nsible/ansible-sn-inventory]$ ./now.py | jq 'del(._meta) | keys[]' (master)
"esx_server"
"linux_server"
"os_x_server"
"solaris_server"
"storage_server"
"windows_server"
(py3-ansible) [mittell:...nsible/ansible-sn-inventory]$ ansible -i now.py esx_server -m ping --list-hosts                      (master✱)
hosts (4):
<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>

Regression test for Python 2.7.3:

(py3-ansible) [mittell:...nsible/ansible-sn-inventory]$ ansible --version                                                      (master✱)
ansible <IP_ADDRESS>
config file =
configured module search path = Default w/o overrides
python version = 2.7.13 (default, Jul 18 2017, 09:17:00) [GCC 4.2.1 Compatible Apple LLVM 8.1.0 (clang-802.0.42)]
(py3-ansible) [mittell:...nsible/ansible-sn-inventory]$ ansible -i nowpy3.py esx_server -m ping --list-hosts                   (master✱)
hosts (4):
<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>

Tested confirmed functional on Ansible 2.3.2 w/Python 2.7.13 and Python 3.6.2.


Inaccurate Code Coverage on UDP server app

I am testing a UDP server, not Rails, which seems to have very low code coverage (42%), but when I investigated I found many lines that must have been exercised were marked as not covered. At first, I thought it was a coveralls problem. Then I tried codeclimate's coverage, the number was exactly the same. So I boiled it down to simplecov and also got the same coverage. I experimented with placement of lines of code in my env.rb (cukes) but the result was always the same. I tried moving the server start from the hooks to within the SimpleCov.start but it would not run, and all tests failed.
I'm sure based on the other issues with similar symptoms, there must be a workaround. I'd love to try things.
Submission checklist:
ruby 2.2.2p95 (2015-04-13 revision 50295) [x86_64-linux] (thought I get this no matter which version of ruby I use)
Generated by simplecov v0.11.2 and simplecov-html v0.10.0 using features
coveralls (0.8.13)
simplecov (0.11.2)
simplecov-html (0.10.0)
codeclimate-test-reporter (0.5.0)
Rakefile:
require 'bundler/gem_tasks'
require 'cucumber/rake/task'
require 'simplecov'
Cucumber::Rake::Task.new(:features) do |t|
  t.profile = 'default'
end
task default: [:features]

I also tried .simplecov, but no affect.

@dmcnulla Please move the require 'simplecov'at the top of your cucumber env.rb, rake test tasks spawn subprocesses for actually running the tests and hence what happens there becomes invisible to simplecov if you load it in a Rakefile. See also #340.
Closing this, feel free to reopen if the issue persists.

@colszowka I've moved require 'simplecov' to the very top of the env.rb, removed it from Rakefile, but I still get the same coverage percentage. Could it be anything else?

Not sure exactly what might be wrong there without digging into the code, but I'd suspect that your test process (where simplecov is running) is separate from the one where your test server is running (although https://github.com/dmcnulla/ntp/blob/master/features/support/hooks.rb seems its the same, but I didn't look into how starting works in this case)
The issue is comparable to running a ruby webserver process, then hitting that server with HTTP request from a test suite - simplecov will not see the lines hit by the server process, just the querying logic in your test suite. You'd have to make sure you load simplecov in the server process, too, and set a distinct SimpleCov.command_name 'Test Server' test suite name for the merging then.

You are right, it starts in the scenario hook. It ultimately starts in the control class like this:
   def start port = DEFAULT_PORT
      fork do
         Process.setsid
         NTP::Server::Base.new(port || DEFAULT_PORT).start
      end
      read(@client_in, 5)
   end

I will try to make a clean branch using only simplecov (master branch uses coveralls) tonight in case you wanted to run it locally. That would be about 11-12 hours from now.

I created a new branch with just simplecov.
Results showing low coverage percentage.
It's the same percentage as the coveralls & codeclimate attempts.

I've been doing more experiments. I tried moving my start into the simplecov start (nothing would run, or I'd get 0 metrics). I tried coco, and the coverage results are the same as with simplecov.
I think I have a similar problem to rails caused by the forked process, but without it being similar enough to rails to use the same solution. I think that I need to drop the test coverage for now. Thank you for looking into the matter.


MethodMatcherException: Data provider mismatch

TestNG Version
<IP_ADDRESS>
Expected behavior
Test works like it has for years
Actual behavior
Test parameter wiring fails with:
org.testng.internal.reflect.MethodMatcherException:
Data provider mismatch
Method: testLongEncodeLength([Parameter{index=0, type=long, declaredAnnotations=[]},
Parameter{index=1, type=int, declaredAnnotations=[]}])
Arguments: [(java.lang.Integer)0,(java.lang.Integer)1]

Looks like testng forgot how to automatically widen int to long.
Is the issue reproductible on runner?

[ ] Shell
[ ] Maven
[ ] Gradle
[x] Ant
[ ] Eclipse
[ ] IntelliJ
[ ] NetBeans

Test case sample
    @Test(dataProvider = "lengths")
    public void testLongEncodeLength(long value, int expected) {
     ...
    }

    @DataProvider(name = "lengths")
    public Object[][] genLengths() {
        return new Object[][] { { 0, 1 } };
   }

@nitinverma Any idea for a fix?

@juherr - I have tried fixing this issue. Can you please help take a look ?

pardon me, Im on a vacation.
fix is a sugar syntax for numbers. strict syntax works even today; 0L would have indicated user intension.

return new Object[][] { { 0L, 1 } };

Error msg was load and clear asking for a long/Long Arguments: [(java.lang.Integer)0,(java.lang.Integer)1]
@krmahadevan plz verify if need any changes in error/warns given to the user.

Are you saying this doesn't need to be fixed?? That's crazy talk.
(a) This has worked fine in TestNG for years; this bug is a regression.
(b) Java programmers expect automatic widening conversions.
Either (a) or (b) alone is reason enough to fix this; both together is that much more compelling.

@archiecobbs does the comment (https://github.com/cbeust/testng/issues/1181#issuecomment-253276860) any where said this fix was unneeded? we need error/warm messaging inline with this fix.

It seemed to be implying the user was at fault for not specifying 0L instead of 0.

my bad should not have written from a beach side having distractions close by.

@nitinverma - What error message are you talking about ? Can you please help elaborate ?
The reason why i didnt feel that an error/warning message was needed here was because

This used to work fine in the earlier versions of TestNG
This seemed counter-intuitive to me when compared to Java's behavior because Java allows widening conversions.

Please let me know if I am missing something obvious here.

@krmahadevan no issues, i'll verify myself soon.


improve fresh start error handling and logging

Special handling for error message that should not be shown on fresh sync.
Add error return to several functions, including dcrsqlite.NewDB.
Fix returned height from GetBlockSummaryHeight when db is empty.
Do not log scanning progress for block 1 when just did genesis.

Any 1 review would be good to confirm nothing is broken.


large dockerfile size

Describe the bug
Dockerfile size is too large for the prokeject its 2.23 Gb it needs to be reduced for less build time and optimisation
To Reproduce
Steps to reproduce the behavior:
build the image using cmd docker build . -t image-name
and then check size ..
Expected behavior
we should alter dockerfile to change itssize
Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

@dhruvpatidar359 I can update this dockerfile and reduce to to 1.25 gb . i hv worked already on this

sure , go ahead . Assigning to you


fix: add code-splitting and ssr-code-split app

Context
The current implementation introduces unnecessary dependencies into the _app or _document. Because  1 function is used to support both. This results in having code that's needed for the server in the client as mentioned in this issue.
What's done?
The API is backwards compatible to support current usage
The functions withAppEmotionCache and augmentDocumentWithEmotionCache are split into separate files so they could be implemented individually. This enables to only import and use the code related to them in the files where it's actually needed. And reduces the _app bundle size by over 50%.
Result
Suggested implementation _app bundle size:

Current implementation _app bundle size:

I've added the analyze script in both apps so they're easier to compare.

Thank you for your effort, I'm trying it out

Ok @michielmetcake,
My bad,
Your implementation does indeed drastically reduce the bundle size.
I would like to rework the API but I released a cadidate that include your fix that you can use right now:  4.6.2-rc.1
Thanks for your work!

Fixed in 4.7 by dependency injection: Fixed in 4.7 by dependency injection: https://docs.tss-react.dev/migration-guides/v4.6-greater-than-v4.7
Thanks for your work!


Development of an application using excel for design of a double rotor axial flux motor

https://www.youtube.com/watch?v=-LM2Bx0HGkQ

Can you upload the excel file again? Thanks

Welcome to use tmp.link upload service.
Upload Complete.
Download Page: https://ttttt.link/f/66557455961db
Thank you for using our services.
password:1963678409

Can you upload the excel file again? Thanks

Can you upload the excel file again? Thanks

Sorry, I can't Access the Excel file, this repo is build for video saving only.  you can visit https://www.comprogexpert.com to download :)

Welcome to use tmp.link upload service.
Upload Complete.
Download Page: https://ttttt.link/f/6656c9eebfea2
Thank you for using our services.
password:1963678409

Auto-closing issue


Create Report: The top N populated countries in a region where N is provided by the user.

AB#20

As a Customer
I want the top N populated countries in a region where N is provided by the user
So that the data can be used and accessed quickly and efficiently


Can mobile clients use the WireGuard app?

I really love Tailscale, except that the control server is not open source and that the iOS app is a huge battery drain. It does not implement on demand capabilities.
Is it possible for a solution like this to generate the proper WireGuard config file to use with the WireGuard native iOS app?
So I have a finite amount of clients, say 10. I only have 2-3 that are truly mobile. Is a control server "check in" required to make it work? I'm unfamiliar with exactly how the mesh overlay is created and continuously updated for all other clients. I'm assuming a static WireGuard config would not be able to know if/when another client changes networks (dynamic ip, roaming, etc.)
As a consolation, I wonder if a static WireGuard config file could be downloaded, but options added to just connect to one "relay" server all the time and get its "mesh connectivity" from the relay, since it won't have constant updates from the control plane.

Sorry for the bug label. I meant this only as a question of capabilities.

Relay servers in Tailscale are implemented as DERP servers which are over TCP, plus there is a question of network policies, NAT traversal/magicsock, MagicDNS, the list grows longer with every Tailscale feature release.
So it's unlikely this is possible but I might be wrong.
If you've only got 10 clients I'd recommend just sticking with vanilla Wireguard to be honest as once you strip out all of the above that's basically what you're left with in terms of feature set.


Missing or reset data

First of all I would like to thank you for OrmLite as it was and is very helpful for migrating my app from using custom cursors to full-fledged object-relational mapping.
Issue Description
Sadly I have some users reporting missing data. These reports come far and few between and so far I was not able to identify common denominators between users. They stretch from several years ago until yesterday when one user reported that some data is going missing sporadically on his Galaxy S9 Dual.
There are no batch tasks that delete data and the only way a DELETE is called is when users delete data manually, one by one. I have checked my CRUD operations and am rather sure that this is some kind of meta-issue affecting OrmLite, the database file or something OS-related. I can only guess wildly since I am not able to reproduce this issue.
Maybe someone has experienced something like that and could help me point in the right direction because I am running out of ideas.
Thank you!
Information
build.gradle
implementation "com.j256.ormlite:ormlite-core:5.1"
implementation "com.j256.ormlite:ormlite-android:5.1"

Initialization
public class DatabaseHelper extends OrmLiteSqliteOpenHelper {

    public DatabaseHelper(Context context) {
        super(context, "app.db", null, 1);
    }

    @Override
    public void onCreate(SQLiteDatabase sqLiteDatabase, ConnectionSource connectionSource) {
        try {
            // Calling for every entity class:
            TableUtils.createTable(connectionSource, <entity class>);
        } catch (SQLException exception) { ... }
    }
}

public class Database {

    private static Database instance;

    public static Database getInstance() {
        if (instance == null) {
            instance = new Database();
        }
        return instance;
    }

    private DatabaseHelper databaseHelper;

    private Database() {}

    public void init(Context context) {
        databaseHelper = OpenHelperManager.getHelper(context, DatabaseHelper.class);
    }

    public DatabaseHelper getDatabaseHelper() {
        return databaseHelper;
    }
}

Hi @Faltenreich ,
We have experienced the same issue with database corruptions. When Database corruption is detected, Android's default behaviour is to remove that database completely (see DefaultDatabaseErrorHandler). This behaviour can be overridden if you supply your own implementation of the DatabaseErrorHandler.
To make it possible to provide our own implementation of the  DatabaseErrorHandler new constructor has been introduced in the PR #120 .
There are multiple possible reasons for database corruptions see howtocorrupt.
The plan for our APP is to have a daily database backup job running and restoring the last good known database if corruption is detected in our own DatabaseErrorHandler implementation.

Hi @yuriy-blinov,
this sounds frightening!
Thank you for posting your fix which is really interesting. Backing up the whole database on a daily basis seems like a really big step to me. In my scenario, not the whole database is wiped but only a few dataset entries. Maybe this is not the same problem as you experienced?


feat: add transporter config

What type of PR is this?
feat
Check the PR title.

[x] This PR title match the format: <type>(optional scope): <description>
[x] The description of this PR title is user-oriented and clear enough for others to understand.

(Optional) Translate the PR title into Chinese.
添加 transporter 配置
(Optional) More detail description for this PR(en: English/zh: Chinese).

en: Add OnAccept and OnConnect for transporter
zh(optional): 为 transporter 添加 OnAccept 和 OnConnect 方法
Which issue(s) this PR fixes:

https://github.com/cloudwego/hertz/issues/428

In netpoll library, OnAccept is called after connection accepted but before adding it to epoll. OnConnect is called after adding it to epoll. The difference is that onConnect can get data but OnAccept cannot. If you'd like to check whether the peer IP is in the blacklist, you can use OnAccept. In go net, OnAccept is executed after connection accepted but before establishing tls connection. OnConnect is executed after establishing tls connection.
在 nepoll 中，OnAccept 会在连接 accept 后，加入 epoll 监听之前调用，这个时候可以获取到对端 ip 但是不能读取数据；onConnect 会在加入 epoll 监听之后使用。在 go net 当中，onAccept 会在连接 accept 之后，转换为 tls 连接之前（如果有的话）调用，onConnect 会在转换为 tls 连接之后调用


collection-repeat and ion-item adding 1-2px at bottom

Hi,
I am using collection-repeat and ion-item to display list and added swipe event.
I am seeing 1 -2px difference when I swipe left. Please see below image.
code pen example: http://codepen.io/ajayparsana/pen/xwEoPO

Greetings @ajayparsana!
My sensors indicate that you need to update your issue through our custom issue form. We are now requiring all issues to be submitted this way, to ensure that we have all of the information necessary to fix them as quickly as possible.
Click Here To Update Your Issue
I will have no choice but to close this issue if it is not resubmitted through the form. Please fill out the rest of the form, so that I may use my friendly robot powers to assist you.
Thank you!

Also collection-repeat doesn't work well with the item-text-wrap option:
With collection-repeat:

With ng-repeat:

Hello! Are you still experiencing this issue?

I have fixed this issue by overriding style in parent.


Euclidean’s theorem

please accept my pull request

Give this repo a star and follow me for follow back 😊


Video Not Available

Describe the bug
Attempts to download anything results in "Video Not Available" error.
Attempting a simple command on a "Slo Mo Guys" video
/opt/projects/mytube/venv/bin/python3
>>> import pytubefix
>>> VIDEO = "https://www.youtube.com/watch?v=2wiMOUIx_5o"
>>> yt = pytubefix.YouTube(str(VIDEO), use_oauth=True, allow_oauth_cache=True)
>>> yt.title
'Video Not Available'

Also, tried it without auth, thinking perhaps just my account was getting rate limited or something.
/opt/projects/mytube/venv/bin/python3
>>> import pytubefix
>>> VIDEO = "https://www.youtube.com/watch?v=2wiMOUIx_5o"
>>> yt = pytubefix.YouTube(str(VIDEO))
>>> yt.title
'Video Not Available'

Expected behavior
I would expect it to download videos as its been doing. This suddenly happened last night.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Red Hat Enterprise Linux release 9.3 (Plow)
Python Version 3.11.5
Pytubefix Version 2.0.0 (intentionally locked at that version)

Additional context
This is how I'm implementing your project into mine

https://github.com/thisguyshouldworkforus/mytube/blob/main/TEMPLATE.py
https://github.com/thisguyshouldworkforus/mytube/blob/main/libs/functions.py

try version 3.0.0

At my ending it works properly:
video_id = '2wiMOUIx_5o'
url = f'https://youtube.com/watch?v={video_id}'

yt = YouTube(url)
duration = yt.length
print(f'duration:      {duration} seconds')

title = yt.title
print(f'title:      {title}')

Output:
duration:      968 seconds
title:      Superman’s “Bullet vs Eyeball” Shot at 200,000 FPS -  The Slow Mo Guys

Tested at Windows, Python 3.9, Pytubefix 3.0.0 final release.

The "Video Not Available" issue is resolved in v3.0.0, but as mentioned here (issue #51) any version above 2.0.0 there is now an AttributeError on yt.publish_date
>>> (yt.publish_date).strftime
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
AttributeError: 'str' object has no attribute 'strftime'

This bug does not exist in v2.0.0

try:
from pytubefix import YouTube

video_url = 'https://www.youtube.com/watch?v=jDwVkXVHIqg&t=25s'

yt = YouTube (video_url)

publish_date = yt.publish_date
print(publish_date)


gui: include pathgroup name information when searching for pathends

Resolve #5882
It seems like the manual check we were doing was wrong. There's an option to include the PathGroup(s) name(s) on the search so we can get only the paths that belong to that group.

Will it change the behavior of cases that didn't have issues before?

Looks ok to me but I'll let @eder-matheus merge once his question is answered.

@eder-matheus @maliberty

Will it change the behavior of cases that didn't have issues before?

It shouldn't. group_names == nullptr for all cases just like before. We only use this option when generating the report from Charts.
I can add a comment to make it clearer if desired.


Dotnet runner and DIffSharp module

Work that has been done (by @novikov-alexander):

Dotnet runner is implemented.
BA objective differentiation using DiffSharp is implemented.
Unit tests for runner and for BA processing are added.

Grpahs
Note: the new version of DiffSharp module is called DiffSharpModule in these graphs.
Note: the old version of DiffSharp is marked as incorrect because it produces output files in a wrong format.

DiffSharpModule also looks generally faster than DiffSharp.  Is that right?  If so, that's great!

Is this because of BASparseMatrix.cs?  Interesting, and valid, as the AD itself doesn't need to pass through it.


Mac Catalyst target build fails

macOS Version(s) Used to Build
macOS 13 Ventura
Xcode Version(s)
Xcode 14
Description
When I attempt to build this project for Mac Catalyst on macOS Sonoma 14.0 with Xcode 15.0, build fails with the following failure:
.../AudioKitUI/Sources/AudioKitUI/Controls/ADSRView.swift:113:15 'prepareForInterfaceBuilder()' is unavailable in Mac Catalyst
Note that in this GitHub bug report form I had to select macOS 13 and Xcode 14 because it doesn't have options for macOS 14 or Xcode 15. But the issue is on macOS 14 and Xcode 15.
Crash Logs, Screenshots or Other Attachments (if applicable)
No response

I'm seeing this crash. I guess it needs to be fixed on the AudioKitUI repo first.

This should be fixed in the latest version using AudioKitUI 0.3.6. Right click on the package > update package. Let me know how it goes!

If this is still an issue feel free to reopen it.


Fixes #8644.

Q
A

Is bugfix?
yes

New feature?
no

Breaks BC?
no

Tests pass?
yes

Fixed issues
#8644

It seems to be a conflict in CHANGELOG.md, but I'm unable to resolve it myself.

No worries.

https://github.com/yiisoft/yii2/commit/33dfddd0c2b019470243d100b9db59f58ad8367b Merged! Thank you!

Great! Thanks to you!


Chevron showing even after using NavigationAction.foo({type: "reset"})

Version
Tell us which versions you are using:

react-native-router-flux v3.36.0
react-native v0.34.0

Expected behaviour
Chevron is hidden when no scenes exist to go back to
Actual behaviour
Chevron shows, and clicking it does nothing
Steps to reproduce
my router looks like:
<Router>
        <Scene key="logIn" component={LogIn} title="Home" initial hideNavBar/>
        <Scene key="userRoleSelection" component={UserRoleSelection} title="Choose Role" />
        <Scene key="addressSelection" component={AddressSelection} title="Select Address" />
        <Scene key="compostItemsSelection" component={CompostItemsSelection} title="Select Composting Items" />
        <Scene key="composterSelection" component={ComposterSelection} title="Select Composter" />
        <Scene key="composterConfirm" component={ComposterConfirm} title="Confirm Composter" />
        <Scene
          key="tabbar"
          tabs={true} >
          <Scene key="home" title="Home" fontIconName="home" icon={TabIcon} >
            <Scene key="homepage" component={UserHome} title="Home" />
          </Scene>
          <Scene key="learning" title="Learning" fontIconName="lightbulb-outline" icon={TabIcon}>
            <Scene key="educationLibrary" component={EducationLibrary} title="Learning" />
          </Scene>
          <Scene key="rewards" title="Rewards" fontIconName="star" icon={TabIcon}>
            <Scene key="rewardsList" component={Rewards} title="Rewards" />
          </Scene>
          <Scene key="contributions" title="Contributions" fontIconName="library-books" icon={TabIcon}>
            <Scene key="contributionLog" component={ContributionLog} title="Contributions" />
          </Scene>
        </Scene>
      </Router>

I call NavigationActions.tabbar({type: 'reset'}); from within the scene with key compostItemsSelection, and I get a chevron on tabbar scene

Also not sure if related but navigating between tabs then seems to freeze the scroll view on the first tab. 🤔

this happens to me also in some cases (in others it works don't know why).
I circumvented it somehow using
Actions.tabbar({ key: 'tabbar', type: 'reset', onBack: null, hideBackImage: true }); but would also like to know why that happens!

Thanks for the tip @tmaximini I'm going to try that.


Provide An Example For az apim api import

If I want to import a swagger file into the API Management with terraform I can do it like this:
https://www.terraform.io/docs/providers/azurerm/r/api_management_api.html
resource "azurerm_api_management_api" "example" {
  name                = "example-api"
  resource_group_name = azurerm_resource_group.example.name
  api_management_name = azurerm_api_management.example.name
  revision            = "1"
  display_name        = "Example API"
  path                = "example"
  protocols           = ["https"]

  import {
    content_format = "swagger-json"
    content_value  = file("../build/swagger.json")
  }
}

I can't work out how to do this from the documentation using the az cli.
I'm not even sure if this section is doing the same thing as this.
The page says "Import an API Management service instance."
What is a "service instance"?
Can you clarify the docs and provide a useful example please.

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

ID: 86f27e34-66e0-1d08-f734-26ce29c56c36
Version Independent ID: d1a421eb-807c-c0bd-cee3-a5aed7335625
Content: az apim api
Content Source: src/azure-cli/azure/cli/command_modules/apim/_help.py
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

routing to appropriate team


add(components): Myna UI

Name
Link

Myna UI
https://mynaui.com

Open source UI Components and Marketing Elements made with Tailwind CSS.

[x] My item is in the right category
[x] My item is logically grouped below similar items
[x] My item's name and description respects the conventions of the list
[x] My item is awesome
[x] My item is in line with the Tailwind brand usage guidelines
[x] I have read and followed the contribution guidelines

Please resolve merge conflicts.

Done @joshmanders


Load more results in Select2 Pagination using specified URL from JSON Response

I’m working with Edamam API and I was able to load the first page. Now I’d like to load the next page. I have added the infinite scroll but it is loading the same first-page result over and over again.
This is my AJAX;
`$("#searchIngredients").select2({
// multiple: true,
width: '100%',
ajax: {
theme: 'bootstrap',
headers: {
'X-CSRF-TOKEN': '{{ csrf_token() }}'
},
url: "{{ route('searchFoodAPI') }}",
type: "post",
dataType: 'json',
data: function (params) {
return {
search: params.term,
page: params.page
};
},
processResults: function (response, params) {
params.page = params.page || 1;
                var select2Data = $.map(response.results, function (obj) {
                    obj.id = obj.text;
                    return obj;
                });

                return {
                    results: select2Data,
                    pagination: {
                        more: params.page
                    }
                };
            },
            cache: false
            },
            placeholder: 'Search for Ingredient',
            minimumInputLength: 2,
    });`

But my JSON response provided "_links" object.
"_links" => array:1 [ "next" => array:2 [   "title" => "Next page"   "href" => "https://api.edamam.com/api/food-database/v2/parser?session=44&ingr=oil&app_id=******&app_key=******&nutrition-type=cooking" ]

This appears to be more of a usage question than a bug report or common feature request.  As we continue to keep the Select2 issue tracker geared towards these two areas, bug reports and common feature requests, we are redirecting usage questions to other communities such as the mailing list, IRC, or Stack Overflow.

Better if you delete this on Github.
I posted this issue but none of them responded. I removed it from my project because it's a waste of time.


Sigmoid and Tanh Box Tensor

@purujitgoyal Looks good and almost ready to be merged. You might have to do pull --rebase to get the latest changes from dev/main into your branch before we can merge it.


FMS Photo Detail requirements

Add the four photo detail requirements as a list onto the "Upload Photographs" text to be visible to user and
Add these requirements as a reminder to user on the frontpages where applicable - Fossil Site, Collection Event, Fossil Sample, SAP

We will need to override the card for this to allow the addition of boilerplate text to the Photo Gallery Card.


Reorder habitat build resource to cleanup the studio before we try running the build

This Pull Request was opened by Chef Automate user Elliott Davis

removes cleanup property

Ready to merge? View this change in Chef Automate and click the Approve button: https://delivery.chef.co/e/chef/#/organizations/Delivery-Build-Cookbooks/projects/habitat-build/changes/9b04853d-af17-4acc-8327-0aacb7086912

@phiggins got that fixed up


3.5 Provide documentation files on the assertions provided for ingestion.

Corresponds with https://github.com/nih-cfde/IDG/issues/46.

Progress on this issue should be logged via comments, including specific tasks, and related deliverables including code and data.


Module names are confusing

Having Overture is weird. I'd either make it Core.Prelude or Foundation. I'd probably make all the other modules Foundation.Number etc.

what about just Overture -> Core ? Foundation is a bit long to type, and also keeping the distinction could be useful in the future. for example having a clear distinction between Core part of foundation, and WIP/Unstable/Extra parts.

Sure, Overture is a horrible name, Core is quite reasonable and has a strong relationship with the rest of the package. That said, I've never found module name length to be a real problem when writing modules, what is a problem is ambiguity - both two packages providing the same module and it not being clear which package a module comes from. I still think Foundation.* would be a good name for everything to start with, but your call. Alternatively, perhaps rename the package as Core, but that overloads against GHC-Core.

so Overture is now Core in 7626da031c00ada351a760589ea736d2b12bb81c


[Feature] Support @keyframes for animation

Hello,
It seems there is not way to create and use @keyframes for animation. Could you take a look at this feature?

Hi, @macaron-css/core uses vanilla-extract's @vanilla-extract/css as the core layer, on top of which macaron's bundler integrations and static extraction take over, so currently for quite a few base features the docs haven't been ported over to macaron.
Keyframes do exist though, and you can see the docs + examples over here https://vanilla-extract.style/documentation/api/keyframes/#keyframes


jsonwebtoken.PrematureJwtException ... a difference of 666 milliseconds. Allowed clock skew: 0 milliseconds

Hi,
I have three connectors in three companies. I will call them A, B and C the DAPS is in C company but in diferent machine
connector of A as consumer VS connector C as provider--> works
connector of A as provider VS connector C as consumer--> it works
connector of B as consumer VS connector C as provider--> works
connector of B as provider VS connector C as consumer --> works
connector A as provider VS connector B as consumer --> not working
I get the following error:

Company A is behind 2 firewalls, I don't know if that has anything to do with those 666 milliseconds.
Can the connector be configured so that the JWT validation leaves some more time?

As connector A has worked with connector C, I guess that the two firewall setup of Company A will not be the reason for the failure.
It seems that the request you made happened before the JWT`s validity time.

Current time --> 2023-05-12T07:15:35Z (07:15:35,335)

JWT valid time from 2023-05-12T07:15:36Z

The error reason could be the time defined for both machines, please check that both machines are synchronized with the same time.
If this is not the reason of the error, it will be neccessary the help from the DSC and DAPS developers.

Yep, It seems that the machine that has the connector A had the timesyncd service unconfigured (port 123 of ntp.ubuntu.com blocked by some of the firewalls), we have reconfigured it and although the connectors do not use the same NTP server it seems that the error has been fixed.
Thanks!!!! @jfernandezsqs

Great to hear that!!
As it has been solved I will close the issue.


Allow changing tekton hub catalog name

Changes

Allow changing tekton hub catalog name

Submitter Checklist

[x] ♽  Run make test lint before submitting a PR (ie: with pre-commit, no need to waste CPU cycle on CI
[x] 📖 If you are adding a user facing feature or make a change of the behavior, please verify that you have documented it
[x] 🧪 100% coverage is not a target but most of the time we would rather have a unit test if you make a code change.
[x] 🎁 If that's something that is possible to do please ensure to check if we can add a e2e test.
[x] 🔎 If there is a flakiness in the CI tests then don't necessary ignore it, better get the flakyness fixed before merging or if that's not possible there is a good reason to bypass it. (token rate limitation may be a good reason to skip).

@siamaksade Any chance we get this implemented ... we did our best here ..

/retest

@spuranam thanks for the PR :) The team is looking at it

Thanks @kcloutie this looks good 👍🏻
we have some infra issues with our test cluster, let's wait it come back...
@vdemeester it's not as easy at it seems since anything with a slash is detected as a file inside repo, but let's discuss this in a new issue (as well as the use case)

/ok-to-test

the error is a random pull fluke from the vale image:

/retest

and it's green! 🚀
Thanks for your contribution @kcloutie 🙇🏻


Please add clear links to release and pre-release extension in marketplace

The "download extension" links in the marketplace for at least the  Python and Pylance extensions often point at pre-release versions.  When this happens, there's no obvious link to download the most recent release version.  Please add such a link and clearly label them when the latest version is a prerelease version.
To compound this problem, VS Code does not offer an update button when an outdated prerelease extension is installed but the next full release is not out.  For example, installing prerelease Pylance v2023.11.13 via a script on VS Code 1.83.1.  It is not the latest pre release version but is newer than the most recent release version.  There is no update button letting me know that a newer release versiion is available and no "switch to release" version button either .
This is an issue for enterprises that block automatic updates and instead periodically write scripts to upgrade extensions.

Hi @rl-utility-man
Thank you for contacting the VS Marketplace Team. Could you please try downloading extension versions from the 'Version History' tab?
If this does not meet your requirements, please let us know.

Thanks @madhurivadaligithub!  No, the version history tab on Marketplace does not make what many users want to do clear or easy.
Pylance is a good example.  If I install the default version using the VS Code UI, I will get the latest "release version" -- which is also clearly marked on GitHub.  But the version history tab does not even convey the existence of a release version that's not the latest one.  The user has to do research some place outside of the marketplace to be able to use the marketplace correctly and the default download link on the marketplace is to a prerelease version while the default download link inside VS Code is to a release version.  That inconsistency is troubling.
I would like:

the default download behavior to be the same on the main Marketplace download link as VS Code.  You can either point that link to a release version or add pre-release and release download links and
pre-release and release versions to be clearly tagged as such in the version history tab.
(in general, I was astonished that Marketplace behaves differently from GitHub and VS Code)

Many thanks for a very useful resource.  I appreciate the huge amount of effort from a big team and vibrant community that went into it.

Thanks @rl-utility-man for sharing the detailed information, we will investigate and get back to you with a response as quickly as possible.
Regards,
VS Marketplace Team


Publish to AWS: I could not deploy to an existing cloud app

In Publish to AWS, the Existing Targets list did not show my project. I expected to see it there.

Just set image tag as "latest" and point to ECR browsed from the list myapp.api (is the name)
Toolkit: <IP_ADDRESS>
Visual Studio: 17.7.6

I encountered the same behavior and compared a working case to a non-working case.   I noted that the non working case was using Linux 2023 as opposed to Linux 2... Linux 2023 environment is new default behavior, via the AWS create environment wizard.   I tried creating a new environment using ".NET Core running on 64bit Amazon Linux 2/2.6.2" and that worked.   I also tried creating an environment via the AWS toolkit and noted that the created environment was Linux 2, supporting my suspicion that this version of the AWS toolkit is not supporting Linux 2023.
Hope this helps somebody

Hi, can you please provide details about the project you are trying to deploy, and what you were expecting to see as a deployment target in the Publish to AWS experience?

The project is a .NET 6.0 WebAPI written in C#.   In the past we had created an Elastic Bean Stalk environment running on the .NET Core running on 64bit Amazon Linux 2 platform, we'd been using this in various forms over the past year, having deployed to it multiple times, using the AWS Toolkit's "Publish to AWS" function.     We recently rebuilt it and unknowingly built the new Linux 2023 platform.   The AWS Toolkit's "Publish to AWS" function failed to list the Linux 2023 environment as a target.
Having noticed the difference in the platform I tried building another environment identical (AFAIK) excepting the Linux 2 platform.   The AWS Toolkit's "Publish to AWS" function did list the newly created Linux 2 platform.
I'm happy to provide any further details, as needed.

Hello @coanpape we are tracking the issue you reported in https://github.com/aws/aws-toolkit-visual-studio/issues/373
If that issue does not address your concern, please create a new issue and provide relevant details for us to further investigate it.
@acmesoft-arg we have not heard back from you so we will go ahead and mark this issue duplicate of the one indicated above.
If you have specifics you can share, please create a new issue and describe the project you are working with, and the deployment you were expecting to see.

excuse me for late response:  in my case the project is a asp.net core 7.0 web api, I tried to deploy to North Virginia, to the ECR to avoid any complexity for some reason publish to existing target never shows.


Correct documents.

sample/make_dict/README.md

Makes a index file of UniDic lex.csv.

sample/transfer_trains/README.md

At first, this program asks you a departure station and time, and a arrival


Shorthand dropdown and @ for suggestions

Code should be refactored into a series of  at least two components:

A generic 'portal' component that corresponds to the dropdown.
An autocomplete component that uses the portal component.

We can then instantiate two of the autocomplete components, one for shorthand and one for the inline helpers.

The @ for suggestions works as expected and the dropdown also works as expected. Selecting an option from the drop down via typing and pressing space, via enter for selection, and via mouse click works.
I noticed that on mouseover for the dropdown there is an error in the console that "updateCurrentShorthandMatch" is not a function. There's a typo in MyEditor.jsx on line 692. "updateCurrentShorthandeMatch" should be "updateCurrentShorthandMatch"
I fixed it and pushed it to your branch

Thanks for the catch.
If there isn't anything else coming to mind @gregquinn2001  or @cjduffett I'm going to push it.

Dylan


Tech Debt: Certbot shouldn't use a fork of the main repository

###Description of the Tech Debt###
Certbot jobs are currently failing because of missing permissions on the certbot service account.
We are using a fork of the certbot chart that already addresses part of that issue: https://github.com/BCDevOps/certbot/commit/6836588e58ccaa1e691de610716bbb8b11c7246d
This is not too time-sensitive, since the certificate update actually happens, and the error happens after.
Todo:

update our stack to use the latest certbot version: https://github.com/BCDevOps/certbot
make sure we don't run into the permissions issue again, and if so, submit a PR to fix it

###Tech Debt Triage#
The purpose of our technical debt triage process is to analyze technical debt to determine risk level of the technical debt and the value in tackling that technical debt.
Risk Value Scoring:

Level
Value

High
3

Medium
2

Low
1

Technical Debt - Risk Types
Level
Value

Business Area Risk - Risk of business area visibility / damage to user experience

Developer Fault Risk - How likely will this tech debt cause a future error related to coding on top of it

System Fault Risk - Risk of system errors or application downtime

Time Scale Risk - Compound risk effect if left alone. How much more difficult to fix or dangerous will this become over time?

Time Sink Risk - How much will this tech debt slow the development process down

TOTAL SCORE:

Waiting on new certificate, but will become high priority once we have the new cert. Pinning it now for visibility.


Rename dir to workdir

In the section of the Cargo.toml file where spin-test looks to build the test component, the configuration for the directory to run the command in is called dir. However, for spin build the equivalent configuration is called workdir. We should change spin-test to expect the same key name as spin build.

Quick note: We will need to update the documentation example at /spin/v2/testing-applications.md when this lands.


Make NiceComponents available via Jitpack

We'll need to make this repo public in order to do so, but then we can start playing around with it in our projects

Done! Instructions for installation have been added to the README as well.


Demo site is broken in Firefox

I opened https://material.preactjs.com/ in Firefox and… it’s kind of broken. Clicking on the home drawer doesn’t do anything and there is no error in the console.
Opening a link such as https://material.preactjs.com/component/button directly will result in 1/4 of the page having the actual content, and 3/4 of it is blank.

Is there any update? Its still broken in Firefox

Once 2.0 is published it will be fixed

I can confirm the problem is still there

Yes bei assured I know that I'm using Firefox myself, this will be fixed with 2.0