Incorporate CF standard units package

Is your feature request related to a problem?
At the moment alignment to CF-compliant units (e.g. for the "units" attribute associated with variables) is mostly done manually with an internal pre-defined lists of standards.
Describe the solution you'd like
Link to a CF-compliant units package e.g. using the cfunits package and make sure all input units are converted to the standard using this. This seems to be ultimately lead back to the UDUNITS C library from looking into the CF compliance documentation.
This would need to be checked against e.g. data downloaded from ICOS Carbon Portal to ensure all unicode cases were covered as well.
Describe alternatives you've considered
No response
Additional context
The cfunits package may already be a dependency for the cf checker being used but the UDUNITS C library still seems to need to be installed separately.

Would link in a larger way to #304


DISP for gopro max

Are there any documents on DISP for gopro max?

From the readme:
DISP | Disparity track (360 modes) | frame rate | n/a | 1-D depth map for the objects seen by the two lenses
While the is stored in both the .LRV and .360, it is only used for previewing the LRVs.  LRVs are not stitched, and .360 are.  So the camera has already computed the depth map for stitching near and far objects that cross the stitch line for the .360 files, this is stored as DISP.   DISP is used so the preview more closely matches the full resolution output.  This data doesn't have much other application.


more open source funding available from Gitpod

👋 from Gitpod,
Your open-source project was one of the competition entrants at DevX Conf and there are funds outstanding that can be claimed by the project steward. If you wish to divert the funds to a cause that helps improve diversity in open-source such as Outreachy then that's possible!
Last year was the first year of the DevX Conference, and over USD $10,000 was raised and donated to unpaid volunteers and the maintainers of Open-source projects that underpin our digital infrastructure. This year, thanks to the support of the conference sponsors AtomicJar, CodeSee, JetBrains, slim.ai, Tailscale, WorkOS, okteto and swimm we are pleased to share that we raised more than twice the amount of last year, that is, $22,600 USD. Like last year, attendees voted on how to distribute profits from DevX Conf between Open-source tools maintained almost exclusively by unpaid volunteers.

For more information and a breakdown of how attendees voted to distribute this year’s funds see https://www.gitpod.io/blog/devxconf-2022-wrap

This is in addition to #555

Thanks for letting us know @ghuntley. We need to setup an org account


Implement these UI features for homepage

Refer to this site https://www.scribd.com/home and try to make UI like that.

Add a search box and symbol button
Add only book categories(e.g sci-fi, self-help, spiritual,etc). As shown in the screenshot make use of "card" to list down categories

UI should look clean. Make it in black and white color mostly with standard look and design. Use text-shadows, box-shadows, and many more.. refer to this site for font-size, font-style, button color and design https://www.zestmoney.in/

current ui is like this change this feature and follow good practice of using same color-code and font throughout the project with proper use of text-shadow and styling .
@rohansx i will change the home page to some familiar design and create a PR then we can work on that to improve it and make it best.
please assign me this under GSSOC'23

thanks @rohansx
I'll start working on it soon.

@k-deepak04 You're also a maintainer for this repository from now on

@k-deepak04 Help other contributors with your role.

@rohansx sure i'll take a lead on this
just one question the pr's that i'll be doing will be counted in the GSSOC'23?

@k-deepak04 yes ig so

Now looking like this

@k-deepak04 lgtm

@rohansx I'm also creating a discussion group for this project we can discuss things there.
Like i have few ideas about yhe design of home page we can look there 🤓

@k-deepak04 ping me on twitter

Can I start working on this issue also? @rohansx

@laxanarana1513 You can enhance the UI for homepage

@k-deepak04 have you started contributing?? What abt the social grp you mentioned before for discussion of work?

@laxanarana1513 i have done the basic layout now you can enhance the features

@laxanarana1513 There is no social group created for this but there is a discussion tab where you can discuss things regarding this project

@k-deepak04  where is the discussion grp?

@laxanarana1513 you can find discussion tab on the tab menu of repository.. right next to "Pull requests" tab

hello @rohansx  @k-deepak04  I really want to contribute to this project, can you plese assign me to this. I can improve navbar.

hey @laxanarana1513 are you working on this?

can i also work on this...

hey @laxanarana1513 are you working on this?

Yes I will make the home page. Exactly what to add in Home Page??

@laxanarana1513 you can take the scribd.com as a reference and add your creativity

@laxanarana1513 you can take the scribd.com as a reference and add your creativity

Ohhk. And the website background should be white only right?

@laxanarana1513 yes for now

@laxanarana1513 yes for now

Okk I have started my work

@rohansx I have also modified navbar & footer. Footer was not proper responsive so modified it


staff + approver role can not see the approve the PAF button

User Story
For staff + approver (VP of tech) actions to take block doesn’t show the button to approve the PAF when trying to approve by going into the PAF page.

Related Issue
#3196
#3197

Do you want the button on both the project page and the PAF page or only on one of them?

@frjo i think it's worth having it in both the places

As we are rebuilding the PAFReviewerRoles flow, I think we should work on it after that.

I think it is already fixed. As we checked in testing, every PAF reviewer could see the Update PAF status button?


Webhook RBAC has hardcoded namespace

Bugs should be filed for issues encountered whilst operating cert-manager.
You should first attempt to resolve your issues through the community support
channels, e.g. Slack, in order to rule out individual configuration errors.
Please provide as much detail as possible.

Describe the bug:
https://github.com/jetstack/cert-manager/blob/9757b2aed1144b8d57d74848675ecdcc069942d6/deploy/charts/cert-manager/webhook/templates/rbac.yaml#L34
Namespace is hardcoded to kube-system.  Is this required?
Expected behaviour:
Everything installs into the targeted namespace.  EG: helm install --namespace foo ...
Steps to reproduce the bug:
No steps, just doing a regular install, no custom values or anything.
Anything else we need to know?:
Environment details::

Kubernetes version (e.g. v1.10.2): 1.11.7
Cloud-provider/provisioner (e.g. GKE, kops AWS, etc): kops AWS
cert-manager version (e.g. v0.7.0): 0.7.0
Install method (e.g. helm or static manifests): helm

/kind bug

If this shouldn't be hardcoded, I can send a PR, I assume it should be:
namespace: {{ .Release.Namespace }}

@polothy I concur: this is not correct, I deployed cert-manager in a different namespace but the role binding was created in kube-system so nothing is working as expected.

@polothy actually I think the namespace field should just be removed so that it is created in the release namespace like the rest

hm, actually I think I have another problem, and this is not the reason why, sorry for the noise


line chart

i want to display the point on the graph with out using tooltips="true"  ..any possible plzz reply me

Hi @udaykanth3
do you mean without enabling tooltips? you could use tooltip: {mode='none'} to achieve this.
Hope this answers your question.
Best,
Christoph

NO,automatically i wnt to display the point on the graph itself ...like (3,5) on the graph

thanks for reply ...my question is i wnt to display the (x,y) values on the
graph only...so plz reply me
On Mon, Jun 1, 2015 at 10:11 AM, chaosmail<EMAIL_ADDRESS>wrote:

Hi @udaykanth3 https://github.com/udaykanth3
do you mean without enabling tooltips? you could use tooltip:
{mode='none'} to achieve this.
Hope this answers your question.
Best,
Christoph
—
Reply to this email directly or view it on GitHub
https://github.com/n3-charts/line-chart/issues/249#issuecomment-107303185
.

You need a little hack to achieve this. Please take a look at this example: http://codepen.io/anon/pen/PqWxbM


Change "browser" to "unpkg" in package.json.

This change is

Review status: 0 of 1 files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


feat: simplify negotiation and transfer handling

WHAT
Simplifies the negotiation and transfer handling by switching to the TX EDC's EDR api. Makes exposing of a callback interface and the corresponding configuration redundant.
WHY
EDR engine with single polling on the EDR is faster than two asynchronous processes (negotiation + transfer) with callback.
FURTHER NOTES
Closes #96  <-- insert Issue number if one exists

Since the EDRs should be explicitely deleted (as they would restart transfers until the end of the agreement period), we currently refrain from using them as the main tool for KA.


MODELS_DIR seems to be set to ComfyUI/ComfyUI/models when server started from comfy root dir

WAS Node Suite Error: There is no write access to /media/sd/ComfyUI/custom_nodes/was-node-suite-comfyui or /media/sd/ComfyUI/ComfyUI/models. Write access is required!
MODELS_DIR =  os.path.join(( os.getcwd()+os.sep+'ComfyUI' if not os.getcwd().startswith('/content') else os.getcwd() ), 'models')

The error doesn't seem to indicate that. It indicates there is no write access so it can't save models or anything.

There's no write access because the directory doesn't exist. My models dir is /media/sd/ComfyUI/models but it's looking at /media/sd/ComfyUI/ComfyUI/models

Try pull latest, using comfy paths, as I've been meaning to switch to it anyways. https://github.com/WASasquatch/was-node-suite-comfyui/commit/93d8dadb82b15c278e9a002189df6604f2a06e8b#diff-35652c7be27f406523ce77f22809c807782fa83236d30eed7d170b3859941ddbR52

the complaint went away


Profile button name "Post" should be "Save"

Profile button name "Post" should be "Save"

This issue can probably be closed?


Back-end debugging in Docker

Thank you for this project, it's very useful.
One of the missing things is a description of how to debug back-end services in Docker. I couldn't successfully manage how to debug in VSCode (via ptvsd) or PyCharm (via remote debug option). Probably some Traefik config should be changed in order to allow remote connections to interpreter.

Thanks for the report, I'll take a look into this.

It seems the solution is quite simple: one can add the following lines in main.py:
`import ptvsd
ptvsd.enable_attach(redirect_output=True)`
and then use such VSCode launch.json config as:
{       "name": "Python: Remote Attach",       "type": "python",       "request": "attach",       "port": 5678,       "host": "localhost",       "pathMappings": [         {           "localRoot": "${workspaceFolder}/backend/app",           "remoteRoot": "."         }       ]     }

I've tried this approach and it's not working for met, at least using the latets ptvsd
any help would be appreciated!

I cannot help with VSCode, but I'd like to leave an instruction on how to do the same thing when using Pycharm. To debug backend app deployed locally using docker-compose, my preferred way is to use Pycharm's remote debugger.
Here is a documentation on how to set it up.
Few caveats that require a little explanation when debugging project deployed in a local docker environment:

In a 'run/debug configuration' window, as a Local host name use localhost. When creating a mapping, use the same mapping that is defined in docker-compose.dev.volumes.yml, which is <path in your host filesystem>/backend/app=/app.
Installation of the pydevd-pycharm~=<version of PyCharm on the local machine>, mentioned in can be added in backend/backend.dockerfile, just add "pydevd-pycharm~=193.6494" to pip install run.
Finally add import pydevd_pycharm pydevd_pycharm.settrace('<IP_ADDRESS>', port=12345, stdoutToServer=True, stderrToServer=True) in backend/app/app/main.py. Replace IP address, with an address of your host. It can be checked using docker inspect <container-id-or-name> | grep Gateway (e.g. docker inspect projectslug_proxy_1 | grep Gateway).
It would be even better to use some env var to enable debugging only in development.
Now you can run your debugger, then docker-compose up and voila!

What's up with this @tiangolo ?
This project is great, but not knowing how to debug it makes development a tedious headache.

Hello!
@tiangolo, i try instruction by @LukGerman, but can't debug backend via pycharm.
I need to put breakpoint to code,
Is it possible?
do you help me?
Thanks a lot!

In light of @sepharg's suggestion, using debugpy which is the replacement of ptvsd(for VSCode users, or Vim/Emacs+DAP users), adapted to the current structure of the project:

Add the following to the settings at backend/app/core/config.py:
   DEBUG_MODE: bool = False
   WAIT_FOR_ATTACH: bool = False

Add  the following dev depenency to backend/app/pyproject.toml, under [tool.poetry.dev-dependencies]:
debugpy = "^1.2"

Add the following code to backend/app/main.py, right before the FastAPI constructor:
if settings.DEBUG_MODE:
    import debugpy
    debugpy.listen(("<IP_ADDRESS>", 5432))
    if settings.WAIT_FOR_ATTACH:
        debugpy.wait_for_client()

Optionally(not related to the debugger), add the argument debug=settings.DEBUG_MODE to the FastAPI constructor to have stacktraces upon API endpoint errors.

Add the port mapping
- "5432:5432"

(or any other external port you wish) to docker-compose.override.yml under services.backend.ports

Add DEBUG_MODE=True to the .env file and optionally WAIT_FOR_ATTACH=True

At this point, you should be able to connect to localhost:5432 via any debugger that supports DAP
For VSCode users, make the backend/app/.vscode/launch.json as follows: (assuming you open the VSCode editor at the backend/app folder)
{
    "configurations": [
        {
            "name": "Python: Remote Attach",
            "type": "python",
            "request": "attach",
            "connect": {
                "host": "localhost",
                "port": 5432
            },
            "pathMappings": [
                {
                    "localRoot": "${workspaceFolder}",
                    "remoteRoot": "/app"
                }
            ]
        }
    ]
}

This has worked for me, using Docker on Windows (WSL2 engine) and VSCode

@nmdanny provided excelent instructions, unfortunately a typo on the port number, 5432 is for postgres, they should have been 5678, the default port for debugpy.
Another minor issue is in the pathmappings, if the  IDE workspace, is the project root folder, should use:
                "localRoot": "${workspaceFolder}/backend/app",
                "remoteRoot": "."

@nmdanny's setup works for me - using docker on Ubuntu 20.04

Thanks @nmdanny and @okz for your solutions. It works for me.
Question for @okz  :
Regarding 5432 being the default port for postgres -
Aren't we using port 5050 for postgres for this project? Why is it blocked then?
I'm newbie here. Sorry if the question is too naive


Added handling of other web resources

Other webresources can now be uploaded/link to dataverse
Other webresoruces will now be shown in webresource list

Thx for your feedback. I changed the method and extended the samrtmatch feature to show webresources in this too

Awesome. The PR looks good. Will work on releasing it in some time.


Request? Dropbox support and/or logging of links processed to file.

Hey. I was wondering if your fork handles (or you intend to add support for) dropbox links and also if it supports saving a log of links encountered to a log file for manual downloading if something isn't supported.
Cheers.

Hey. I was wondering if your fork handles (or you intend to add support for) dropbox links and also if it supports saving a log of links encountered to a log file for manual downloading if something isn't supported.
Cheers.

For dropbox, I'll add it to the to-do list.
For logging links, you can setup something like this in the channel(s)/server(s) configs:
"logLinks": {
    "destination": "log_links",
    "destinationIsFolder": true,
    "divideLogsByServer": true,
    "divideLogsByChannel": true,
    "divideLogsByUser": true,
    "userData": true
},


Add properties for listing stake keys

Issue Number
ADP-721
Overview

[x] Test no rewards => sum of all keys' stake == utxo balance"
[x] Test ours and foreign keys are disjoint
[x] ~Rename getAccountBalance to getCachedAccountBalance and add un-cached fetchAccountBalance~ -> done in #2694 instead
[x] ~Make listStakeKeys call fetchAccountBalance instead of getCachedAccountBalance~ -> done in #2694 instead
[x] Improve integration testing of the rewardBalance field
[x] Tweak API docs

Comments

@rvl @jonathanknowles could any of you you have another look? 🙏
Happy to have a quick call about it if it helps.

Thanks for the comments, closing temporarily while the task is on hold.


Correction: AD Application Secrets only good for 24 Months

https://github.com/MicrosoftDocs/azure-docs/blob/main/articles/marketplace/partner-center-portal/commercial-marketplace-lead-management-instructions-dynamics.md

We recommend this option because you never need to update your username or password to keep getting leads. To use the Azure Active Directory option, you provide the App ID, Application Key, and Directory ID from your Active Directory application.

That is no longer true. Even in Custom, you can only go out 2 years.

The value must be between 11/07/2022 and 11/07/2024.

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

ID: cb2ad729-6bcd-dc00-f472-ed39b10da509
Version Independent ID: 61a120d7-2a43-76d6-01ee-a2f19ce07c6c
Content: Lead management for Dynamics 365 Customer Engagement - Microsoft commercial marketplace
Content Source: articles/marketplace/partner-center-portal/commercial-marketplace-lead-management-instructions-dynamics.md
Service: marketplace
Sub-service: partnercenter-marketplace-publisher
GitHub Login: @urimsft
Microsoft Alias: uridor

@askpatrickw
Thanks for your feedback! We will investigate and update as appropriate.
Since this channel is for driving improvements towards MS Docs, could you provide the URL of the specific documentation that your feedback is applicable to? That way, we are able to make sure your feedback is addressed by the right team

@KrishnaG-MSFT
Please review it

Tagging @durior1 for visibility, review and document enhancement consideration as appropriate. cc:@sadiqhahmed-msft

please remove @durior1 from any place it's listed, and use @urimsft instead

From: KrishnaG-MSFT @.>
Sent: Tuesday, November 8, 2022 6:45 PM
To: MicrosoftDocs/azure-docs @.>
Cc: Uri Dor @.>; Mention @.>
Subject: Re: [MicrosoftDocs/azure-docs] Correction: AD Application Secrets only good for 24 Months (Issue #101122)
Tagging @durior1https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fdurior1&data=05|01|uridor%40microsoft.com|cd0ee146dccc40db1d4f08dac1a8b36e|72f988bf86f141af91ab2d7cd011db47|1|0|638035227573851283|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=kaXPAjRYReU%2FTxWrA0hApoQtS9hBa9Q9leiJAtocCao%3D&reserved=0 for visibility, review and document enhancement consideration as appropriate. @.***https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FSadiqhAhmed-MSFT&data=05|01|uridor%40microsoft.com|cd0ee146dccc40db1d4f08dac1a8b36e|72f988bf86f141af91ab2d7cd011db47|1|0|638035227573851283|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=JGfQWUiiDWuZviwmvxqaOAz36SMYcUdO9N1RKWv1HDs%3D&reserved=0
—
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F101122%23issuecomment-1307514157&data=05|01|uridor%40microsoft.com|cd0ee146dccc40db1d4f08dac1a8b36e|72f988bf86f141af91ab2d7cd011db47|1|0|638035227573851283|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=mkNRm2KMWccQeY69SYcHsrz%2BUZxM7Aq6XfGejCoUAQM%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAY4LPO52DTIPTEC473ZIR7TWHJ7UBANCNFSM6AAAAAARZWCM4A&data=05|01|uridor%40microsoft.com|cd0ee146dccc40db1d4f08dac1a8b36e|72f988bf86f141af91ab2d7cd011db47|1|0|638035227573851283|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=EwQvKmatqXofd8ISV7SZkaKZVoWQaYqfN6xPRbhus8o%3D&reserved=0.
You are receiving this because you were mentioned.Message ID: @.***>

@urimsft done

#reassign:urimsft

#reassign:urimsft

#reassign:sudharocks

Hi @askpatrickw, thanks for submitting this issue. We have filed an internal work item to track this issue to resolution and will update the docs accordingly. #please-close


[Orbit] SPEED of Transition Orbit 6

How can we reproduce this bug?

ussing Orbit

What did you expect to happen?
i would like to control the speed of the transition
What happened instead?
just can control data-timer-delay that is time beetwen sliders, but not the time of transition itself
please i need help i think transition time it's too fast, regards.

@Joseluisblom I was trying to figure this out as well, and found the answer here-
http://foundation.zurb.com/sites/docs/motion-ui.html#custom-transitions

@albertvolkman Can you explain what that code looks like?
I'm initializing foundation with this line:
ZDF.globals.$document.foundation();
and when I try
ZDF.globals.$document.foundation({
      orbit : { animation_speed: 300}
      });

sorry, object is not a valid parameter. You must use a string representing the method you wish to invoke

@shushugah you define your transition in your sass file via the Motion UI directions-
https://github.com/zurb/motion-ui/blob/master/docs/transitions.md
And then you add the transition in your constructor. I instantiated mine via attributes, as seen here-
https://github.com/Norstone/norstone.github.io/blob/develop/index.html#L73
and then refer to the first 4 stanzas of this sass file-
https://github.com/Norstone/norstone.github.io/blob/develop/_sass/_modules.scss


Regarding the visualization error issue and the parallelization problem in Maniskill3.

Amazing work! However, I encountered some problems while using it.
The first problem is, if I open more than one environment, i.e. num_env > 1, is it unreasonable for visualization?
The code below can reproduce this issue; the robotic arm does not move in the visualization window, only the gripper moves.
import gymnasium as gym
import mani_skill.envs
envs = gym.make(
    "PickCube-v1", # there are more tasks e.g. "PushCube-v1", "PegInsertionSide-v1", ...
    num_envs=2,
    obs_mode="state", # there is also "state_dict", "rgbd", ...
    control_mode="pd_ee_delta_pose", # there is also "pd_joint_delta_pos", ...
    render_mode="human")
print("Observation space", envs.observation_space)
print("Action space", envs.action_space)
obs, _ = envs.reset(seed=0)
for i in range(100):
    action = envs.action_space.sample()
    obs, reward, terminated, truncated, info = envs.step(action)
    envs.render()

The second problem is that Maniskill3 directly supports multi-environment parallelization, but how does it handle resets in this context? If I open multiple environments and one of them ends (succeeds early), will it reset immediately, or will it continue executing actions after success? Both approaches can ensure that the lengths of the collected trajectories are equal. Is there still a need to use gym.vector.AsyncVectorEnv for re-encapsulation?  Since I am training RL, the details of parallelization are very important.
Thanks for your valuable time.

What do you mean if only the robot arm moves, not the gripper? When I run the code I see the whole robot moving. Note that if yhou have parallel environments there isn't a recommended way to visualize it other than to record videos using the RecordEpisode wrapper (which will record videos in parallel, e.g. if you run the baseline PPO code the evaluation videos are always multiple videos together).
For environment parallelization you should not use any wrappers/async vector envs from gymnasium or other libraries if you want to use GPU simulation. (We still support asyncvectorenv for CPU simulation). To use GPU simulation setting num_envs > 1 will automatically turn it on when using gym.make. To make it look like AsyncVectorEnv, you should use the ManiSkillVectorEnv wrapper which wraps the env created by gym.make into a object that is an instance of gymnasium VectorEnv.
Relevant imports:
from mani_skill.utils.wrappers.record import RecordEpisode
from mani_skill.vector.wrappers.gymnasium import ManiSkillVectorEnv

I highly recommend looking at the PPO code https://github.com/haosulab/ManiSkill/blob/main/examples/baselines/ppo/ppo.py to see how it is used for RL workflows. I will be adding documentation about how to do certain types of ML workflows with ManiSkill soon.

Oh and regarding environment reset, by default the environment created by gym.make does not come with auto reset, so it only resets if you call env.reset. If you wrap ManiSkillVectorEnv around the environment, then you get auto reset (you can also turn it off). There is also an argument for ignoring terminations (terminated = True if environment is success or fail state). If you ignore terminations then resets always happen whenever the environment is truncated (e.g. reaching the max episode steps). If you don't ignore terminations then there is partial resets, where at each step not all environments reset at the same time (which can sometimes speed up RL training).
See https://github.com/haosulab/ManiSkill/blob/main/mani_skill/vector/wrappers/gymnasium.py for the code.

Hi @yeshenpy is this still an issue?


Delete the various servers.

They have been unsupported for a while, so deleting them is the safest course of action, so that people are not tempted to use them.

Can we put a big pointer to the new stuff somewhere visible?

There's already some links in the README: https://github.com/google/certificate-transparency#c-log-server-deprecation-notice

True. Maybe that's enough.

Did update the README a bit, though.


Added CKEditor to custom slide create and edit form.

A full rerun of Gulp is required.
@emanuelschuetze Please test and check the options. Maybe ask @rolandgeider to help to fix the toolbar and the options object and to reinsert 'insertpre' if neccessary.

@normanjaeckel I added a custom ckeditor config in pull request https://github.com/normanjaeckel/OpenSlides/pull/9. After that please rebase on top of master.

Rebased. Ready to review and merge.


Differences with subword-nmt

I have found that loading a fastBPE codes and vocabulary against subword-nmt I get a different result in the bpe codes:
Using fastBPE
hoy quiero que te qu@@ ede &@@ apo@@ s@@ ; a dormir
this song is gonna make you mad

Using subword-nmt
ho@@ y qui@@ ero que te que@@ de &@@ apo@@ s@@ ; a dor@@ mir
th@@ is son@@ g is gon@@ na make you mad

using the same codes and vocabulary, with minimal adaptation in the latter package. My understanding of BPE was that the implementation should be almost the same.
I have asked subword-nmt author as well: https://github.com/rsennrich/subword-nmt/issues/76

[UPDATE]
Considering the new Python API Wrapper this is not necessary anymore, by the way would be interesting to understand those differences!
Thanks a lot!

How large is the dataset on which you learned the BPE codes? I believe in the original implementation they do not merge two BPE splits if the resulting word only appear once in the original corpus. This will make a difference for small training datasets (typically, if the word "gonna" only appears once in your training set), but otherwise there will be no difference.

@glample it's FAIR LASER :)
So codes and vocabulary are
Codes: https://dl.fbaipublicfiles.com/laser/models/93langs.fcodes
Vocab: https://dl.fbaipublicfiles.com/laser/models/93langs.fvocab
root@3f40ea8e2cc4:/tornado_api# head -n10 /root/laser_models/93langs.fvocab
. 87264459
, 78156033
de 19001435
- 13731976
? 13338524
a 13062980
i 8917603
en 8272731
" 8258142
la 7623301
root@3f40ea8e2cc4:/tornado_api# head -n10 /root/laser_models/93langs.fcodes
e n 52708119
e r 51024442
e n</w> 47209692
a n 46619244
i n 44583543
s t 42633672
a r 34974160
o n 31941788
t i 30717853
d e 30509691

@glample it's FAIR LASER :)
So codes and vocabulary are
Codes: https://dl.fbaipublicfiles.com/laser/models/93langs.fcodes
Vocab: https://dl.fbaipublicfiles.com/laser/models/93langs.fvocab
root@3f40ea8e2cc4:/tornado_api# head -n10 /root/laser_models/93langs.fvocab
. 87264459
, 78156033
de 19001435
- 13731976
? 13338524
a 13062980
i 8917603
en 8272731
" 8258142
la 7623301
root@3f40ea8e2cc4:/tornado_api# head -n10 /root/laser_models/93langs.fcodes
e n 52708119
e r 51024442
e n</w> 47209692
a n 46619244
i n 44583543
s t 42633672
a r 34974160
o n 31941788
t i 30717853
d e 30509691

why subword-nmt‘s vocab has label "@@"，but fastBPE has no.

closing this because solved my issue. Not sure about @zpppy question.


Fix #4934: Option in TexturePacker.Settings to specify the InterpolationMode when scaling

As per request in #4934, new option addition in Settings under TexturePacker to specify the array of interpolation modes per scale factor. No breaking changes in the API, and fully backwards-compatible assuming Bicubic (which was the default) in case nothing is being specified.
Contributor agreement has been signed and emailed.

@Tom-Ski @NathanSweet I noticed 1.9.8 is the current now, but the changes are documented under 1.9.7. Do you guys want me to close this pull request and reopen one that puts the changes under 1.9.8?

Thanks for the PR. You can just make a new commit changing it to under 1.9.8 and push. The PR will be updated.
Or if you prefer, you can always remake it the PR entirely.

Gotcha, @Tom-Ski. It conflicted anyway, so I've just edited it to be under 1.9.8. Any chance it's eligible for merge? Small change, plus a 'test case'.

@Velocity- didn't notice your comment, sorry, I broke your PR again :]

Oh no! No worries @Tom-Ski, I've fixed them. Quick! Merge! 😉

@xoppa @MobiDevelop Any of you guys able to merge this PR? I don't want to come across as impatient but I'm trying to help out the developer in the linked issue to help fix it. :)

It's a nice addition, thanks!


[REVIEW] Rabyâ Raghib (@rabraghib)

GitHub Profile Link
https://github.com/rabraghib

Hey, @Pankaj-SinghR, thank you for opening an Issue.
Here are some of my recommendations -

Instead of Avatar, use your picture (It man be for privacy reasons, then ignore this :) )
Add Bio
Banner size can be reduced (totally on you :))
Pin your top 2/4 repos.
your profile looks commit heavy, try to distribute it (Diamond)

If you need any kind of help, ping us here or on Discord server

Hey, @rabraghib, thank you for opening an Issue.
Here are some of my recommendations -

Instead of Avatar, use your picture (It man be for privacy reasons, then ignore this :) )
Add Bio
Banner size can be reduced (totally on you :))
Pin your top 2/4 repos.
your profile looks commit heavy, try to distribute it (Diamond)

If you need any kind of help, ping us here or on Discord server

Hey, @Pradumnasaraf!
for

The banner and the avatar: they will be changed ASAP (whenever I have time to design them)
Pining repos: I am currently documenting some of my projects so I can make them public & pin them
contributions: I absolutely need to do more code reviews, issues, PRs..

Thank you for your recommendations! I will definitely take them into consideration!

Looks great, some thoughts...

pin 2 or 4 repos (not 3)
latest blog post section seems empty
years go back very far, is this a hack? suggest removing


ptrauth.h is missing

Ran into this while building libffi. It has support for pointer authentication on Arm64, but it needs ptrauth.h to build successfully with it.

This will be fixed with the LLVM 19 upgrade; Clang ships this header.


Don't use 0 for "fresh leaf".

Closes https://github.com/dmlc/xgboost/issues/4314

Also https://github.com/slundberg/shap/pull/520 .


Token moved to secure object inside session

ember-simple-auth made a change to the session that moved the token into a sub object called "secure"

I see this PR is superseded by Issue #49 - closing.


add option to verify command to include similarity hashes

@mielliott I took a stab at implementing tlsh and tlsh+tika hashes.
For now, it look like this:
$ cat algae.zip | preston hash -a sha256
hash://sha256/175bdab3638a4fd16d58845a768eba8e76bd01c9cc0f3afc100670d8740214f2
$ cat algae.zip | preston hash -a tlsh
hash://tlsh/423c0388b0e3aff488b022c03d5705ac5dd3dddc6ef199cbd6a661a69569cc85b2d
$ cat algae.zip | preston hash -a tika
hash://tika/574fd07be0a8a2ed5117b2aca254279bbc63a80356d8bf8de935c10f94975374d0c

where the algae collection comes from https://github.com/bio-linker/organization/wiki/TLSH-Example .
ltsh - is the hash without the T1xxx prefix, where the T1 is the hash version and xxx encodes file properties. Only the buckets are included
tika - is a ltsh hash applied after letting tika extract text from the content.
I've also added the option to the verify command to make it easy to traverse the an entire graph.

Please re-open if issues remain.


Output format field error when accepting input numbers from the user

Describe the bug

Version

Browser

[ ] Electron distribution
[ ] Chrome
[ ] Safari
[ ] Firefox
[x] Edge

OS

[ ] macOS
[x] Windows
[ ] Ubuntu

To Reproduce
Steps to reproduce the behavior:

Open Composer.
In a dialog prompt the user for a number.
In the Output format field in the drop down menu select Write an expression.

Select the fx tab, then in the drop down menu select Conversion->int.

This produces the following format =int(value).

Also an error is issued.

Expected behavior
The Output field format should be int(this.value). No equal sign and value prefixed with this.  See next screenshot.
Screenshots

Additional context
Please see documentation PR: https://github.com/MicrosoftDocs/composer-docs-pr/pull/568.

https://github.com/microsoft/BotFramework-Composer/pull/7722 should fix that

@mewa1024 Actually this is not something to do with using the equals symbol.
"int(this.value)" evaluated to a string whereas "=int(this.value)" evaluated to a number which our compiler complained.
The core reason for this is, the schema for outputFormat property in our current codebase uses ref to expression which expects a return type of string

Essentially, @lei9444 PR linked here checks if the type is of expression and checks if there is a oneOf in the schema. If there is none we can assume that the return type need not be forced to a string and it could return number or other data types.


Crashes with 'Gdk-Message: Error flushing display: Broken pipe' sometimes when the page, or an element on the page changes.

OS: EndeavourOS (linux)
Napture Version: 1.2.2
Log:
(.:29384): Gtk-WARNING **: 17:28:50.576: Unknown key gtk-modules in /home/xande/.config/gtk-4.0/settings.ini

(.:29384): Adwaita-WARNING **: 17:28:50.642: Using GtkSettings:gtk-application-prefer-dark-theme with libadwaita is unsupported. Please use AdwStyleManager:color-scheme instead.
Attempting to navigate to https://github.com/face-hh/dingle-frontend/index.html...
Unknown head element: meta
Unknown head element: meta
Attempting to navigate to https://github.com/face-hh/dingle-frontend/styles.css...

(.:29384): Gtk-WARNING **: 17:28:52.841: Theme parser error: <data>:3:5-8: No property named "gap"

(.:29384): Gtk-WARNING **: 17:28:52.841: Theme parser error: <data>:5:5-14: No property named "direction"

(.:29384): Gtk-WARNING **: 17:28:52.841: Theme parser error: <data>:6:5-16: No property named "align-items"

(.:29384): Gtk-WARNING **: 17:28:52.841: Theme parser error: <data>:54:5-10: No property named "width"

(.:29384): Gtk-WARNING **: 17:28:52.841: Theme parser error: <data>:55:5-11: No property named "height"
Attempting to navigate to https://github.com/face-hh/dingle-frontend/main.lua...

Gdk-Message: 17:28:58.200: Error flushing display: Broken pipe

Weird, despite running the project on Endeavour multiple times I haven't seen that issue. Are you running X11 or wayland?

Weird, despite running the project on Endeavour multiple times I haven't seen that issue. Are you running X11 or wayland?

Wayland, KDE Plasma 6.0.5.

Wayland is probably the issue in that case

Wayland is probably the issue in that case

I guess I'll try X11.

Yep, Wayland was the issue.

https://docs.gtk.org/gtk4/wayland.html ?


Update maintenance-notifications.md

Fixed the target audience for email notifications. Also added more clarity and links for Service Health alerts.

@cynthn

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


Same name cars issue, can you  use ID-CarName or CarName (ID) on the dropdown, it's only showing 1 car

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
2 cars, same name, dashboard only showing 1 on selection to pick from
Expected Behavior
Have 2 selections on dropdown, even if same name
Steps To Reproduce
Open dashboard, try to select vehicle, must have 2 with the same name
Relevant log output
none

Screenshots

Additional data
none
Type of installation
Docker
Version
1.28.5

So I edited the teslamate cars table and chagned the names and added a 1 and a 2 on respective car ID rows, it shows both and I can pick from them.  Dashboard for some reason is using the car name to pick what to show?  Should I post this elsewhere?
NOTE:  After some time, it reverted back to old name and  I could no longer pick from the lsit.  Is there a way where I can update the select statement so I can include [id] as the display name?

Dropdown shows both:

So I edited the teslamate cars table and chagned the names

The name is retrieved by the API.
It must therefore be defined on the vehicle to be retained.

So I edited the teslamate cars table and chagned the names

The name is retrieved by the API.
It must therefore be defined on the vehicle to be retained.
You can search "rename my tesla" in Google :)

That is what I’m complaining about.
That is correct.  I named both cars the same.   Same names, but different IDs, so the dashboard should be able to tell apart both cars, instead it is “merging” them both and only lets me pick 1.

Given, that the images show2 a Custom Dashboard, we cannot really help.

.. and I provided a simple workaround above.

Given, that the images show2 a Custom Dashboard, we cannot really help.

This is on all dashboards, I just chose the custom one since it is the one I use the most.  They all work the same, they show it is querying the ID and name but it’s using the name as the “value” for the drop-down.
Here are screenshots of the stock Teslamate dashboard.
Both cars the same name (not renaming on database)

Name changed in the database

.. and I provided a simple workaround above.

I appreciate you trying to help and letting me know I can rename the car names, but that's not what I am reporting and not what would solve my (and possibly others) problem.  I believe (being a web developer) that the value of the drop-down should be the car id, something that is unique, not the name that could be the same and that the user would want to call them the same (unless there is somehow a limitation of what you can set the value of the drop-down to.)

I understand what @edgararroyo is asking for here, because I just encountered this exact situation.
I had a vehicle with the name CAR (not really its name, but the name doesn't matter). I took that vehicle to Tesla and traded it in as part of a purchase of a new vehicle. I think the name of the original vehicle still suits the new vehicle so I also name it CAR.
Now, in the Teslamate settings UI both vehicles are displayed as tabs at the top of the screen, both the same name. Data collection for both vehicles works as expected.
However, in the Grafana UI the dropdowns for selecting a vehicle no longer work because of this query:
https://github.com/teslamate-org/teslamate/blob/cabb474504f915a7e3e2ff83735230f08ed06410/grafana/dashboards/drives.json#L1509-L1531
I currently just run Teslamate in a container and don't know enough to know what the correct way to "fix" this is but the problem is that SELECT name AS __text means only a single entry ends up in the dropdown. This could be fixed by something like SELECT CONCAT(id, name) AS __text, but I think that having everyone's grafana change the dropdown value change would be jarring and the wrong fix.
Hopefully this illustrates the issue so that someone who is familiar with all this can come up with a real fix. For now I'm just going to manually edit my cars table directly since the old vehicle will no longer cause the names to be updated since it doesn't show up in my Tesla account anymore.

I could modify the Cars Query to show a counter next to the Cars if two or more are named the same *without performance impact. Should it be done? @JakobLichterfeld

I could modify the Cars Query to show a counter next to the Cars if two or more are named the same *without performance impact. Should it be done? @JakobLichterfeld

Could it show Car name - [last four or six of VIN]?

Yes, works as well

I like the VIN idea, better to track.  Count (or number) could change unless you plan on using the car id and not just adding a sequential number.

@edgararroyo - could you try changing the variable query for car_id to the one below and post a screenshot of the result.
i'll update all dashboards while working on #2736
select
	id as __value,
	case when count(id) over (partition by name) > 1 then concat(name, right(vin, 6)) else name end as __text
from cars
order by display_priority asc, name asc, vin asc

Should it be done?

As we do have now at minimum two users reporting the same thing, we can change it with the lowest priority, as it is not an issue but an UX improvement.

Could it show Car name - [last four or six of VIN]?

I also vote for this idea

Can you add a dash between name and right(vin, 6)?   concat(name, '-', right(vin, 6))
Otherwise it works as intented, only add the last 6 VIN if more than 1 of the same name.


Dispatching to a second pool from a job in one pool is buggy

extern crate rayon;

use std::sync::Arc;
use std::{thread, time};

fn main() {
    let pool = Arc::new(rayon::ThreadPoolBuilder::new().num_threads(1).build().unwrap());
    let pool2 = Arc::new(rayon::ThreadPoolBuilder::new().num_threads(1).build().unwrap());
    for x in 1..10 {
        let pool2_c = pool2.clone();
        pool.spawn(move || {
            println!("start job {}", x);
            pool2_c.install(move || {
                thread::sleep(time::Duration::from_secs(1));
            });
            println!("end job {}", x);
        });
        thread::sleep(time::Duration::from_millis(20));
    }
    thread::sleep(time::Duration::from_secs(12));
}

I would expect the spawned jobs to run one at a time, because the pool they're spawned into has only one thread. i.e. the output should be something like:
start job 1
end job 1
start job 2
end job 2
...

However, the actual output I get is:
start job 1
start job 2
...
start job 9
end job 9
end job 8
...
end job 1

which means the spawned tasks are running concurrently. This makes no sense to me. Note that if I do 1-second sleep inside the spawn task directly, instead of installing it into pool2, then I get the expected behaviour.

This output order doesn't necessarily mean that the jobs are running concurrently, they could also be running in a nested fashion on the same thread. E.g. the call to pool2_c.install might start executing pending work of the pool that owns the current thread.

This is intentional, per #449, to make sure that invoking pools in a circle (A -> B -> A) won't deadlock.
I can see how your case is surprising, but the way I would think of install is that it blocks the current thread of execution in an abstract sense (the job is blocked), but the underlying thread pool is not blocked.
Does that semantic work for you?  For your example, if you really wanted those jobs to be executed in order, I would just move the for loop into the spawn so it's explicitly sequential.  Of course, I'm sure your real use case where you noticed this is more complicated...

Ok, that makes sense. I'll close this issue. I can probably figure out how to update my original code to work with this semantic.

To expand upon this - I think the bug as filed was basically expecting the thread pool to provide additional guarantees when the number of threads was specified as 1. In Gecko terms, you wanted AbstractThread/nsISerialEventTarget [1]. But Rayon doesn't make those guarantees (considering the number of OS threads to be an implementation detail), and instead offers the semantics of nsIEventTarget/nsIThreadPool.
So it sounds like you want something like TaskQueue [2], which provides serial semantics on top of an underlying thread pool. It's conceivable that something like that could be built on top of (or into) rayon.   In C++/XPCOM these semantics are critical for thread-safety, whereas they obviously aren't in Rust/Rayon. But it sounds like there still may be some value there for maintaining program invariants, so it's worth thinking about.
[1] https://searchfox.org/mozilla-central/source/xpcom/threads/AbstractThread.h
[2] https://searchfox.org/mozilla-central/source/xpcom/threads/TaskQueue.h


Distribute cargo-ament-build as a PyPI package

Distributing cargo-ament-build as a crate is fine, but makes it more difficult for the colcon extensions to be properly package since they require cargo-ament-build, but we can't specify that dependency anywhere. This complicates things more when uploading the extensions to PackageCloud.
https://github.com/PyO3/maturin seems to support packaging Rust crates as PyPI packages (thanks @stelzo for the link). cargo-ament-build could be compiled natively via cargo-dist and then packaged as a script in the scripts folder inside the PyPI archive. This might be bending the rules of what a PyPI package is, so any feedback would be great.

@jhdcs @luca-della-vedova @maspe36 @stelzo, thoughts?

Off hand, I don't see anything wrong with shipping cargo-ament-build as a PyPI package. In fact, one of the maintainers of maturin has done exactly this a number of times

cargo-deny
cargo-xwin
cargo-deb

Considering the ROS ecosystem, this seems like a great way to offer symmetry for users.
Our instructions could look something like this
pip3 install colcon-cargo colcon-ros-cargo cargo-ament-build

Now to @luca-della-vedova's point, maybe it doesn't make sense to even write this component in Rust. It would probably be a pain to maintain something like cargo-manifest ourselves though.

Now to @luca-della-vedova's point, maybe it doesn't make sense to even write this component in Rust. It would probably be a pain to maintain something like cargo-manifest ourselves though.

Fair enough, there are toml crates but not really any cargo.toml crate, my hunch is that it wouldn't really be a ton of work since this crate is fairly simple but I admit haven't looked at it much.
My point was really to get support for ROS users a bit more straightforward, we should be able to release both colcon-cargo and colcon-ros-cargo to the buildfarm "soon" :tm: so users can install them through apt.
However, debian packages can only depend on other debian packages. Assuming we get this on rosdep, releasing it on pypi will be more of a side step (users can install it through rosdep install) not in the direction of full support (it still won't be installed when users do apt install python3-colcon-ros-cargo), we will still need full Rust support on the buildfarm to release this which has a much more unclear state right now.

My point was really to get support for ROS users a bit more straightforward, we should be able to release both colcon-cargo and colcon-ros-cargo to the buildfarm "soon" ™ so users can install them through apt.

Correct me if I'm wrong, but the colcon extensions are not released to the ROS buildfarm. They are built locally and then uploaded to Dirk's packagecloud (https://packagecloud.io/dirk-thomas/colcon). So in our case, we'd add support for maturin in cargo-ament-build, upload it to PyPI and then also generate Debians (I guess with stdeb?).
AFAIK, packagecloud does not build packages, just stores them, so there's no worry about it supporting Rust or not. Given that the PyPI package will already contain a binary version of cargo-ament-build (built via cargo-dist and saved as part of the release on GitHub), we can just upload such package as is to packagecloud.

I've submitted #13 to add support for ARM builds with cargo-dist and Buildjet, but I don't really know how maturin and cargo-dist work together, or does maturin replace cargo-dist entirely? @stelzo do you know more about this?

Correct me if I'm wrong, but the colcon extensions are not released to the ROS buildfarm. They are built locally and then uploaded to Dirk's packagecloud (https://packagecloud.io/dirk-thomas/colcon). So in our case, we'd add support for maturin in cargo-ament-build, upload it to PyPI and then also generate Debians (I guess with stdeb?).
AFAIK, packagecloud does not build packages, just stores them, so there's no worry about it supporting Rust or not. Given that the PyPI package will already contain a binary version of cargo-ament-build (built via cargo-dist and saved as part of the release on GitHub), we can just upload such package as is to packagecloud.

Ah yes sorry indeed colcon is built locally and uploaded to packagecloud. I am still worried about distributing compiled binaries with Pypi / packagecloud though. Currently all the packages that are distributed are pure Python packages hence we don't need to worry about which platform they are running on. If we distribute debians with binaries / Pypi packages with precompiled binaries how would that work with different architectures? I.e. as you brought up in your last comment about supporting aarch64, I don't see it being currently done in the colcon packagecloud (although I imagine it should be possible to distribute multiple debian packages depending on the architecture) and I'm not sure how it can be done in Pypi either

If we distribute debians with binaries / Pypi packages with precompiled binaries how would that work with different architectures? I.e. as you brought up in your last comment about supporting aarch64, I don't see it being currently done in the colcon packagecloud (although I imagine it should be possible to distribute multiple debian packages depending on the architecture) and I'm not sure how it can be done in Pypi either

PyPI supports wheels, which are platform and architecture-dependent binary packages, maturin takes care of building the binary part and uploading it to PyPI as the appropriate wheel. See for example https://pypi.org/project/ruff/#files, Ruff is a Python linter written in Rust, but distributed as a PyPI package that can be installed via pipx / pip.

I've submitted https://github.com/ros2-rust/cargo-ament-build/pull/14 as a test, it generated a bunch of wheel files, for example the one I'm attaching (I'm not sure if you everyone can see the log files from the CI run). Unzip it and then install the whl file with pipx and it'll be available in your PATH:
$ pipx install ./cargo_ament_build-0.1.8-py3-none-manylinux_2_17_x86_64.manylinux2014_x86_64.whl
  installed package cargo-ament-build 0.1.8, installed using Python 3.11.2
  These apps are now globally available
    - cargo-ament-build
done! ✨ 🌟 ✨

$ cargo ament-build
Error in cargo-ament-build

Caused by:
    the '--install-base' option must be set

$ which cargo-ament-build
/home/esteve/.local/bin/cargo-ament-build

I've submitted https://github.com/ros2-rust/cargo-ament-build/pull/13 to add support for ARM builds with cargo-dist and Buildjet, but I don't really know how maturin and cargo-dist work together, or does maturin replace cargo-dist entirely? @stelzo do you know more about this?

Maturin uses cross-compilation and reaches much more targets. Probably needed for PyPI. The binaries are somewhere in the pipeline but there is no easy integration for both tools in the same project. Ruff does use the binaries from maturin in cargo-dist but it does not look very easy nor maintainable imo.
I would keep the tools the way you initialized them, building the binaries multiple times with the respective tool. Maturin publishes to PyPI and cargo-dist to GitHub when pushing a tag. I ran the changes (without buildjet because I it is not set it up for me) and it worked.


Show a list of rare words before listening to the episode

More advanced listeners could review the rarest words before starting to listen to the episode, ideally with translations, so that they can then later listen without having to look at the screen to use the vocab helper.
We can utilize this word frequency list to determine which words are more unique to the current episode:
https://github.com/marcomzp/easygerman

Hallo Manuel und @bagilevi
I have created a python script to make this enhancement.
I would love to include it in this repository.

Hallo Manuel und @bagilevi
I have created a python script to make this enhancement.
I would love to include it in this repository.

Hey @aliraeisdanaei
I think this pretty independent, so I don't really want to add it to this repository, but I added it as a related project on the wiki

Hey @aliraeisdanaei
I think this pretty independent, so I don't really want to add it to this repository, but I added it as a related project on the wiki


Error when running "/rcnn/script/additional_deps.sh"

Description
I tried to run "/example/rcnn/script/additional_deps.sh". However I get the following error:
/usr/lib/gcc/x86_64-linux-gnu/4.9/../../../../lib/libcblas.so: undefined reference to ATL_zger2u' /usr/lib/gcc/x86_64-linux-gnu/4.9/../../../../lib/libcblas.so: undefined reference to ATL_cger2u'
/usr/lib/gcc/x86_64-linux-gnu/4.9/../../../../lib/libcblas.so: undefined reference to ATL_zger2c' /usr/lib/gcc/x86_64-linux-gnu/4.9/../../../../lib/libcblas.so: undefined reference to ATL_cger2c'
/usr/lib/gcc/x86_64-linux-gnu/4.9/../../../../lib/libcblas.so: undefined reference to ATL_dger2' /usr/lib/gcc/x86_64-linux-gnu/4.9/../../../../lib/libcblas.so: undefined reference to ATL_sger2'
collect2: error: ld returned 1 exit status
Makefile:406: recipe for target 'bin/im2rec' failed
make: *** [bin/im2rec] Error 1
~/repos/mxnet/python ~/repos/mxnet ~/repos/mxnet/example/rcnn
...
Using /usr/lib/python2.7/dist-packages
Finished processing dependencies for mxnet==0.12.1
~/repos/mxnet ~/repos/mxnet/example/rcnn
~/repos/mxnet/example/rcnn
im making cython
cd rcnn/cython/; python setup.py build_ext --inplace; rm -rf build; cd ../../
running build_ext
skipping 'bbox.c' Cython extension (up-to-date)
skipping 'cpu_nms.c' Cython extension (up-to-date)
skipping 'gpu_nms.cpp' Cython extension (up-to-date)
cd rcnn/pycocotools/; python setup.py build_ext --inplace; rm -rf build; cd ../../
Compiling _mask.pyx because it changed.
Cythonizing _mask.pyx
Error compiling Cython file:
...
siz h,
siz w,
siz m,
uint* cnts,
void rlesInit( RLE **R, siz n )
void rleEncode( RLE *R, const byte *M, siz h, siz w, siz n )
^
_mask.pyx:42:39: Expected ')', found '*'
Traceback (most recent call last):
File "setup.py", line 36, in
ext_modules=cythonize(ext_modules)
File "/opt/tiger/ss_lib/python_package/lib/python2.7/site-packages/Cython/Build/Dependencies.py", line 667, in cythonize
cythonize_one(*args[1:])
File "/opt/tiger/ss_lib/python_package/lib/python2.7/site-packages/Cython/Build/Dependencies.py", line 731, in cythonize_one
raise CompileError(None, pyx_file)
Cython.Compiler.Errors.CompileError: _mask.pyx

@apache/mxnet-committers: This issue has been inactive for the past 90 days. It has no label and needs triage.
For general "how-to" questions, our user forum (and Chinese version) is a good place to get help.

@YeWenting are you still facing the same issue? It seemed this issue is more related to the package your are using. @nswamy can you add this as 'blas'?

@nswamy when you add the labels you may want to remove the "Needs Triage" label :)

To prevent this type of issue from arising again, #11373 removed all cython modules in rcnn example completely.

Closing this after merging #11373


How to remove orphan endpoints?

I have a few "orphan" endpoints which were created with an older build of this module. I don't seem to be able to delete them through TFS, and there is no obvious way to do it via PowerShell.

I will ping PMs from the VSTS product team for help.

@mrichman, can you try updating it once and then attempt delete? let us know if you are still facing the issue

@RoopeshNair I'm unable to update or disconnect:

@Vincent<EMAIL_ADDRESS>– can you please take a look?
[addTag:deployment;dri;v3]
From: Mark Richman<EMAIL_ADDRESS>Sent: Saturday, September 23, 2017 1:47 AM
To: DarqueWarrior/team<EMAIL_ADDRESS>Cc: Roopesh Nair<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [DarqueWarrior/team] How to remove orphan endpoints? (#22)
@RoopeshNairhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Froopeshnair&data=02|01|roopesh.nair%40microsoft.com|ae1efd62027f466da5be08d501f6d1cb|72f988bf86f141af91ab2d7cd011db47|1|0|636417081947863554&sdata=qP%2F%2FsqsC4hZXxYOs8TFVLSLs6Q1vNWHH3UiK2BUrTkY%3D&reserved=0 I'm unable to update or disconnect:
[image]https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fuser-images.githubusercontent.com%2F11228%2F30762788-63b68e68-9fb1-11e7-9df9-7a94cafcf00b.png&data=02|01|roopesh.nair%40microsoft.com|ae1efd62027f466da5be08d501f6d1cb|72f988bf86f141af91ab2d7cd011db47|1|0|636417081947863554&sdata=uSbl6AqOussKlFEsu4rijDbkhrywvenaZ9Cu5PpkIGU%3D&reserved=0
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FDarqueWarrior%2Fteam%2Fissues%2F22%23issuecomment-331550175&data=02|01|roopesh.nair%40microsoft.com|ae1efd62027f466da5be08d501f6d1cb|72f988bf86f141af91ab2d7cd011db47|1|0|636417081947863554&sdata=%2FToUa8y0ihLHmERFP5cC3p2At7e%2BStHio5p6kcir1fQ%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FABTpcDUZ6I3hSxIkMRozbQmMCuAiA8f5ks5slBWggaJpZM4Pcd2P&data=02|01|roopesh.nair%40microsoft.com|ae1efd62027f466da5be08d501f6d1cb|72f988bf86f141af91ab2d7cd011db47|1|0|636417081947863554&sdata=%2FwHzugrPzNYFl8DfNQf%2FK6AoNdeli0s0%2BxbMi01IdXg%3D&reserved=0.

Have you tried using the Remove-VSTeamServiceEndpoint method in the module? Originally called Remove-ServiceEndpoint before the VSTeam rename.

I just tried this:
Remove-VSTeamServiceEndpoint -id 23dbd458-b3ce-40a0-8a77-47b9adc95486 -ProjectName Student -Verbose
Where that GUID is the resourceId parameter in my TFS URL: http://tfs:8080/tfs/EcoSys/Student/_admin/_services?resourceId=<xxxx>
The command failed, but the output indicates success:
VERBOSE: DELETE
http://tfs:8080/tfs/EcoSys/Student/_apis/distributedtask/serviceendpoints/23dbd458-b3ce-40a0-8a77-47b9adc95486?api-version=3.0-preview.1 with 0-byte payload
VERBOSE: received -1-byte response of content type

Deleted service endpoint 23dbd458-b3ce-40a0-8a77-47b9adc95486

However, the endpoint still remains.

Url looks good. In keeping with the documentation.
It would be interesting to try and see what the actual response code from that request is. It's supposed to be 204 by the looks of it.
Just to confirm, what version of TFS are you running? It appears that that version of the Api is only available from TFS 2017 onwards. But then again, if I try Get-VSTeamServiceEndpoint here (we're on 2015) I get the standard http error page spat out. I would expect the Remove to do likewise.

@ObsidianPhoenix I'm running TFS 2017 RTM (15.105.25910.0).
The -Verbose flag doesn't indicate the HTTP response code.

Hey @RoopeshNair has anyone from the PG had a chance to take a look at this?

I just pinged the team again on this.

@mrichman, It will be great if we can have a Skype call to debug the issue. Can you please share your Skype Id with us if possible?
Can you please try to delete the endpoint via Rest API using PostMan or similar rest api tool? The url will be similar to
http://tfs:8080/tfs/EcoSys/Student/_apis/distributedtask/serviceendpoints/23dbd458-b3ce-40a0-8a77-47b9adc95486?api-version=3.0-preview.1
HTTP Method : Delete
Please post the response of the  rest call here in case  you get some error.

@niadak Sure, but I'm not available this week for a Skype call. I can try the delete method and reporting my findings.

@niadak Calling DELETE on that URL returns HTTP 204 (No content) with the following response headers:
ActivityId →f9deb713-e32e-4756-81b6-92ae66507e11
Cache-Control →no-cache
Date →Tue, 28 Nov 2017 16:22:19 GMT
Expires →-1
Lfs-Authenticate →NTLM
P3P →CP="CAO DSP COR ADMa DEV CONo TELo CUR PSA PSD TAI IVDo OUR SAMi BUS DEM NAV STA UNI COM INT PHY ONL FIN PUR LOC CNT"
Persistent-Auth →true
Pragma →no-cache
Server →Microsoft-IIS/10.0
X-AspNet-Version →4.0.30319
X-Content-Type-Options →nosniff
X-FRAME-OPTIONS →SAMEORIGIN
X-Powered-By →ASP.NET
X-TFS-ProcessId →607be308-045b-4bee-ac37-b5d54909a752
X-TFS-Session →f9deb713-e32e-4756-81b6-92ae66507e11
X-VSS-E2EID →f9deb713-e32e-4756-81b6-92ae66507e11
X-VSS-UserData →e71d123d-590b-49e7-a136-431ed7965741:mrichman

@mrichman , Thanks for sharing your findings. Please make a GET call and see if the end point is successfully deleted or not. Share the response with us. Ideally you should not get any object.
http://tfs:8080/tfs/EcoSys/Student/_apis/distributedtask/serviceendpoints/23dbd458-b3ce-40a0-8a77-47b9adc95486?api-version=3.0-preview.1
HTTP Method : Get

@niadak No, I still get the same response to a GET:
{"data":{"subscriptionId":"717dbfc1-958c-4a09-ba32-5fce93bbb8d6","subscriptionName":"100022-Flame_University","creationMode":"Automatic"},"id":"23dbd458-b3ce-40a0-8a77-47b9adc95486","name":"delete me","type":"azurerm","url":"https://management.azure.com/","createdBy":{"id":"e71d123d-590b-49e7-a136-431ed7965741","displayName":"Mark Richman","uniqueName":"CMC\\mrichman"},"authorization":{"parameters":{"serviceprincipalid":"","serviceprincipalkey":null,"tenantid":"a38ff870-9c86-43df-9e6b-6b01f206a398"},"scheme":"ServicePrincipal"},"isReady":false,"operationStatus":{"state":"Failed","statusMessage":"The given key was not present in the dictionary. For troubleshooting refer to link."}}

I'm not sure how the DELETE would have worked anyway. I never supplied an auth header. Shouldn't I have to do that from Postman, curl, etc.?

Hi @mrichman
Can you try the solution suggested in this link

Please mail us at<EMAIL_ADDRESS>if you face this issue again.


Clip rendering into a CGContext by a CGPath

Depends upon #1267 for its geometry retrieval machinery; is influenced by #1268 insofar as composition.

Done in #1368 .


Assembler: Need a way to get pointer to code location.

Brought about with https://github.com/FEX-Emu/FEX/pull/1640, Need a way in the assembler to either get the pointer to the current  Assembler's cursor position for branching to, or a way to get a label's bound position.
Either way, need the pointer.

Pushed a few buffer access functions in this commit: https://github.com/lioncash/biscuit/commit/a7e520f9d288cf1a0ef2fc8fc23f27b512b2a9b5

That works! Thanks!


Write a file header on each new file created

I need a file header to be written each time a new file is created. I don't see a way to do this, especially on rotate. Will add a PR with a suggested solution.

https://github.com/natefinch/lumberjack/pull/51

I'm curious what the use case is for this?  What do you want to write in the header?

In my case I want to write in a CSV format for easily reading the log into a analytics tool. For this I need a header for what each column represent. However I figured there might be other reason why you might do this and it could be useful for others.

So, upon thinking about this... I think your specific use case is too specific.  The tool you're importing data into should give you the option of setting column names.  I don't think this feature is useful in enough cases to merit an API change.

Had to think and experiment quite a bit, but I have a solution now which will not require the CSV header. Thank you for considering the request.


Setting up the Cypress with first test

You can add it in the .eslintignore! If the error comes from eslint of course :)

You can add it in the .eslintignore! If the error comes from eslint of course :)

.eslintignore updated :)

Can you pass your PR as ready to review ?


Wrong version error for libnetcdf

Currently, I am trying to convert a shapefile to geojson via the command line.
I'm working from these instructions:
http://ben.balter.com/2013/06/26/how-to-convert-shapefiles-to-geojson-for-use-on-github/
install gdal and then run ogr2ogr -f GeoJSON -t_srs crs:84 VTS_Geometry.geojson VTS_Geometry.shp
Here is the error:
dyld: Library not loaded: @rpath/libnetcdf.11.dylib
  Referenced from: /Users/aschharwood/anaconda2/bin/ogr2ogr
  Reason: Incompatible library version: ogr2ogr requires version 12.0.0 or later, but libnetcdf.11.dylib provides version 11.0.0
Abort trap: 6

I should note I've gotten this same error with a number of other GIS packages. Ill post again when I remember where I was bumping into this.
Your help is much appreciated!
Asch

The latest version of libnetcdf, <IP_ADDRESS>, provides SOVERSION 11.  I'm not sure where the dependency on a version 12.0.0 is coming from, as that is not something Unidata has released.  You might contact the ogr2ogr (I'm unfamiliar with the program or I would try to provide more help, sorry!) developers and see if they can provide some guidance.

okay. thanks!
On Fri, Dec 9, 2016 at 2:59 PM, Ward Fisher<EMAIL_ADDRESS>wrote:

The latest version of libnetcdf, <IP_ADDRESS>, provides SOVERSION 11. I'm not
sure where the dependency on a version 12.0.0 is coming from, as that is
not something Unidata has released. You might contact the ogr2ogr (I'm
unfamiliar with the program or I would try to provide more help, sorry!)
developers and see if they can provide some guidance.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Unidata/netcdf-c/issues/337#issuecomment-266108453,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJSO1oBSYoZdsGS4j0gNZwpWhmTuLJlSks5rGbMFgaJpZM4LJD2o
.

--
Asch Harwood
Mobile:<PHONE_NUMBER>
Skype: Aschharwood


Update deprecated exponentially_longer method

This method is deprecated in Rails 7.1 and will be removed in Rails 7.2.
https://github.com/rails/rails/blob/7-1-stable/activejob/CHANGELOG.md#rails-710rc1-september-27-2023

@vassyz There is already a PR open for this https://github.com/DFE-Digital/dfe-analytics/pull/89

@ericaporter my bad, didn't think to look at the open PRs 🤦


Investigate and use syncthing if appropriate

Its good but to work nice needs to installs making it less than simple. Actively developed though. Will use it and hope its improved before someone needs to use again.


Strip authorization from headers on redirects

When follow redirects is set to true, sensitive headers are forwarded to hosts that the sender did not anticipate.
I've drafted a quick fix here: https://github.com/myfreeweb/httpotion/pull/134, an extension would be to mimic curl's --location-trusted flag and give users the option.
See a similar issue w/ more in depth explanation here:
https://github.com/chef/chef/issues/6984

Thanks! I've merged #134. But note that httpotion is sort of deprecated, I myself am using Tesla with hackney backend now :)


Pipeline steps base image

Currently Kale uses the KFP default base image when executing pipeline steps.
The alternative could be to use the NotebookServer image.
A user running a Notebook and testing some code, is working on the env defined by the Notebook Server image. If we snapshot and mount the workspace to preserve the user environment, then it is reasonable to run the steps on the same image where the user develops the notebook.
It is not enough to use the default Kale Notebook Server image, as users might extend this image with custom images that add some custom packages.
The approach explained in #3 could be used to retrieve the image of the current container when Kale is executed in the NotebookServer.

@elikatsis  what do you think about the following approach to retrieve the docker image used from inside the Notebook Server:

Use env variable HOSTNAME to filter the current pod from kubectl get pods -o json output
Use env variable NB_PREFIX (the string after the last /) to filter the notebook server container, and then its image.

Is it reliable to use NB_PREFIX as the name of the current container? You can see how I would do this in #3

@StefanoFioravanzo,
I would propose using Kubernetes Client to get all the required information, instead of calling CLI commands.
Please take a look at @klolos'  script in this commit, retrieving Notebook's PVCs, as an example on Kubernetes API usage.

@elikatsis nice work there. I think we could just extend the script with a function _get_container_base_image and include this python script in the utils folder.
If you do that, then I can update PR #10 to just call that function.


nixos/iso-image: add an option to disable BIOS boot

Description of changes
This change adds an option to disable legacy BIOS boot support for ISO images. The implementation uses syslinux package that currently cannot be cross-compiled from non-x86 platforms, e.g. aarch64-linux system.
Things done

Built on platform(s)

[x] x86_64-linux
[x] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)

Cross-compiled from aarch64-linux and and successfully booted the resulting ISO using Ventoy.

23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

@RaitoBezarius @alyssais gentle ping, are there any updates on this?


Fix grammar

Thank you for finding the mistake and providing an appropriate fix. I highly appreciate your help!


TLS config for ccass_builder can be improved

TLS config, i.e. root_cert, client_cert and client_key are set using CHAINCODE_AS_A_SERVICE_BUILDER_CONFIG env in the Peer docker-compose file as same as peername. And then pass the values into connection.json by ccass_builder commands. The templated connection.json is really convenient now for multiple peers support.
But in deployment, peername is static, while root_cert, client_cert and client_key will not be known until enrolled from CA. So I think these three values should be set through directly modification of connection.json before install chaincode package.

It implements the directly modification connection.json.


TypeError at /login/

Django Debug Toolbar version: 2.2
With the following view:
from django.contrib.auth.views import LoginView

class PublicLoginView(LoginView):
    redirect_field_name = None

Getting the following error:
Environment:

Request Method: GET
Request URL: http://(removed)/login/

Django Version: 2.2.2
Python Version: 3.6.9
Installed Applications:
['django.contrib.admin',
 'django.contrib.auth',
 'django.contrib.contenttypes',
 'django.contrib.humanize',
 'django.contrib.sessions',
 'django.contrib.messages',
 'django.contrib.staticfiles',
 'django.forms',
 'capture_tag',
 'django_celery_beat',
 'django_celery_results',
 'debug_toolbar']
Installed Middleware:
['debug_toolbar.middleware.DebugToolbarMiddleware',
 'django.middleware.security.SecurityMiddleware',
 'django.contrib.sessions.middleware.SessionMiddleware',
 'django.middleware.common.CommonMiddleware',
 'django.middleware.csrf.CsrfViewMiddleware',
 'django.contrib.messages.middleware.MessageMiddleware',
 'django.middleware.clickjacking.XFrameOptionsMiddleware',
 'django.contrib.auth.middleware.AuthenticationMiddleware',
 'global_login_required.GlobalLoginRequiredMiddleware']

Traceback:

File "/home/django/srv/current/venv/lib/python3.6/site-packages/django/core/handlers/exception.py" in inner
  34.             response = get_response(request)

File "/home/django/srv/current/venv/lib/python3.6/site-packages/django/core/handlers/base.py" in _get_response
  145.                 response = self.process_exception_by_middleware(e, request)

File "/home/django/srv/current/venv/lib/python3.6/site-packages/django/core/handlers/base.py" in _get_response
  143.                 response = response.render()

File "/home/django/srv/current/venv/lib/python3.6/site-packages/django/template/response.py" in render
  106.             self.content = self.rendered_content

File "/home/django/srv/current/venv/lib/python3.6/site-packages/django/template/response.py" in rendered_content
  83.         content = template.render(context, self._request)

File "/home/django/srv/current/venv/lib/python3.6/site-packages/django/template/backends/django.py" in render
  61.             return self.template.render(context)

File "/home/django/srv/current/venv/lib/python3.6/site-packages/django/template/base.py" in render
  171.                     return self._render(context)

File "/home/django/srv/current/venv/lib/python3.6/site-packages/django/test/utils.py" in instrumented_test_render
  95.     template_rendered.send(sender=self, template=self, context=context)

File "/home/django/srv/current/venv/lib/python3.6/site-packages/django/dispatch/dispatcher.py" in send
  175.             for receiver in self._live_receivers(sender)

File "/home/django/srv/current/venv/lib/python3.6/site-packages/django/dispatch/dispatcher.py" in <listcomp>
  175.             for receiver in self._live_receivers(sender)

File "/home/django/srv/current/venv/lib/python3.6/site-packages/debug_toolbar/panels/templates/panel.py" in _store_template_info
  91.                 key_values = sorted(context_layer.items())

Exception Type: TypeError at /login/
Exception Value: '<' not supported between instances of 'NoneType' and 'str'

This only happens when redirect_field_name is set to None. If I remove it or change it to a string things work as expected and there aren't any errors.

The issue here is that LoginView has a key-value pair in it's context with the redirect_field_name as a key. You can override the get_context_data method in PublicLoginView or find an alternate way to achieve your goal.
class PublicLoginView(LoginView):
    redirect_field_name = None

        def get_context_data(self, **kwargs):
            context = super().get_context_data(**kwargs)
            if self.redirect_field_name is None:
                context.pop(self.redirect_field_name, None)
            return context

Why are you setting redirect_field_name to None? Maybe we can find a different way to solve that for you.

@tim-schilling thanks, we'll look at modifying the context. What we've done for now is set redirect_field_name to an unused/absurd string to keep DDT from throwing errors due to the sorted() use and setting redirect_field_name to None.
The use case for removing the redirect field is to help curb some auth-related redirect loops for certain views by disabling the "next" feature of the login view.

IMO, this should be considered incorrect use of LoginView and therefore closed. Template variable names are expected to be strings, so using None would be API misuse.
@merit-mthompson one option would be to override get_redirect_url to always return ''.
https://github.com/django/django/blob/f1f24539d8c86f60d1e2951a19eb3178e15d6399/django/contrib/auth/views.py#L69-L80

Django has a test specifically for None which leads me to believe it's at least allowed: https://github.com/django/django/blob/cc7c16af98e4c9a7146c248c8567aac7a50e0fbb/tests/auth_tests/test_mixins.py#L156
Feel free to close, we're working around it. The fix would be to change how things are being looped/sorted in DDT at https://github.com/jazzband/django-debug-toolbar/blob/master/debug_toolbar/panels/templates/panel.py#L91. Maybe pop None items out before sorting.

Thanks for linking to that test. I agree we should support this if Django does.
I think we could sort by repr() as a workaround. Are you interested in submitting a PR with a test?

Potential fix in #1385

@jdufresne thank you!


error :Failed to send message. Server closed the connection unexpectedly.

According to your instructions, I added the api-key and successfully accessed the page, but when I asked a question, I was able to reply to the answer and an error message would also appear. Is this normal? I'm building with docker-compose from the latest branch on your git, thanks

Hello, this is a bug. I have fixed it in the new version. Please pull the latest image again.

ok Thanks

Hello, I used your docker-compose under Run services on github to start, but when I entered the account admin password password, Request failed, please try again.

Trying to log in to http://localhost:9000/admin will return a page not found

Thank you very much for your feedback! ❤
This was an image build error caused by forgetting to remove comments in the Dockerfile. I have updated the image and also the docker-compose configuration example in the Readme file.
You can copy the docker-compose configuration again and pull the latest image by running the command docker-compose up --pull always.
手忙脚乱，老兄海涵

ok 3Q  哈哈哈

请问这个bug解决了吗？我添加了API-Key依然是返回这个报错

你能看到容器日志吗？可以的话截个图
On Thu, Mar 23, 2023 at 4:48 PM duanyunlun @.***> wrote:

请问这个bug解决了吗？我添加了API-Key依然是返回这个报错
—
Reply to this email directly, view it on GitHub
https://github.com/WongSaang/chatgpt-ui/issues/9#issuecomment-1480798003,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALAX6FAHNI7XXDILXC7DEXDW5QE6PANCNFSM6AAAAAAVLRMWPI
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

在 wsig-server 添加一个环境变量 DEBUG=True ，就会输出错误日志
On Thu, Mar 23, 2023 at 5:17 PM Rafi Wong @.***> wrote:

你能看到容器日志吗？可以的话截个图
On Thu, Mar 23, 2023 at 4:48 PM duanyunlun @.***>
wrote:

请问这个bug解决了吗？我添加了API-Key依然是返回这个报错
—
Reply to this email directly, view it on GitHub
https://github.com/WongSaang/chatgpt-ui/issues/9#issuecomment-1480798003,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALAX6FAHNI7XXDILXC7DEXDW5QE6PANCNFSM6AAAAAAVLRMWPI
.
You are receiving this because you modified the open/close state.Message
ID: @.***>


Fix StorageKeys.service_and_creds so it properly parses keys with multiple credentials

This PR fixes a bug. For keys that contain multiple credentials, for example, an app_id and an app_key, .service_and_creds was just returning the first one.

For some reason the Travis build didn't appear in the PR.
https://travis-ci.org/3scale/xcflushd/builds/208155930


Github Actions - Merge Queue

If you haven't already, please read through our contributing guidelines before opening your PR
What is the purpose of this PR?
Closes #931.
How did you implement your changes
Adds merge queue, removes python Dependabot.
Remaining issues
N/A.

I think we’ll need to tweak readthedocs as well? Did you take a look at what’s required there?

@ngreenwald What did we need to tweak for readthedocs? It seems to be building properly. Is it the older package requirements that it uses?

Test failed because out of disk space. You can also see that it's currently waiting for the RTD to build.

@ngreenwald Tried to restart the CI workflow in the merge queue, however there was no response coming from the workflow request. After 12 min of no responses it kicked the PR off the merge queue. Alex's latest 2 run also did not start successfully.

I merged this in, so at least the GA part of the merge queue will be working. RTD doesn't know to look for a merge queue. There's a description in the link I posted in the issue about what to do for non-GA integration.
Separately, we'll need to free up space on GA


Improve logging of command STDOUT and STDERR

To assist with debugging why new or complex scripts aren't working as expected, we need better visibility into STDOUT and STDERR for all commands run.

Looking into this now

Looking into this now

Awesome - thanks Jim!


feat(storage): initialize item with default value

@veryCrunchy Why do you need to re-initialize a value? What's your use-case?

@aklinker1 Since I seem to have trouble explaining things in a way that's clear and easy to understand, I've let ai sum it up for me;
Your explanation captures the essence of reInit quite well. Here’s a refined version that maintains clarity and emphasizes the key benefits:

The reInit method simplifies the process of resetting and synchronizing stored values by leveraging the initialization logic defined within the item itself. Here's why this is beneficial:

Streamlined State Management:

By defining the initialization logic directly within the item, you can easily manage state changes. For instance, when a user logs in with a new account, you can call reInit to update the stored user ID without manually handling the reset logic elsewhere.

Seamless Synchronization:

If your stored data needs to align with external sources (such as a database or API), reInit ensures that the stored value is updated according to the latest external data. This avoids discrepancies and keeps your application state in sync with external changes.

Encapsulation of Initialization Logic:

Instead of having separate functions for resetting or initializing values, the logic is encapsulated within the item definition. This simplifies your codebase by reducing the need for additional imports and separate reset functions.

Example Use-Cases

User Session: Resetting user-related data when a new account is logged in.
Feature Flags: Updating feature flags based on configuration changes or external settings.

By using reInit, you streamline state management and ensure that your stored values remain consistent with their intended initialization logic.

This explanation underscores the convenience and effectiveness of reInit in managing and synchronizing data.

Example Use-Cases

User Session: Resetting user-related data when a new account is logged in.
Feature Flags: Updating feature flags based on configuration changes or external settings.

This is what I was looking for, thanks 😄
Neither of these cases make sense to me. Both are easily solved by just calling setValue instead of putting things like network requests inside the init option...
From an code organization perspective, think of the storage items as your persistence layer, not the service layer. You don't put business logic in repos, you put business logic in services.
For example, for user session account management, this is how I would implement it at a high level:
const user = storage.defineItem<User>('local:user');
const preferences = storage.defineItem<Preferences>('local:preferences')

async function login(username, password) {
  const newUser = await fetchUser(username, password)
  const newPreferences = await fetch preferences(newUser);

  await user.setValue(newUser)
  await preferences.setValue(newPreferences)
}

Or feature flags fetched from an API:
const featureFlags = storage.defineItem('local:feature-flags', {
  fallback: {...},
})

scheduleJob('every 30m', async () => {
  const latestFlags = await fetchFeatureFlags();
  await featureFlags.setValue(latestFlags);
})

In both these cases, set value works just fine, and the code is organized with all related logic together. I don't see how reInit would simplify this code or make it more organized.

One potential use-case I see is analytics session ids. Having a way to quickly regenerate a new ID at the beginning of a session would be useful. But in that case, I'm either using the browser.storage.session or sessionStorage; the first, init would work as-is and the second isn't extension storage.

@veryCrunchy I already have a PR open for this change, minus reInit. I would hold off on doing more work until you convince me the feature is necessary. https://github.com/wxt-dev/wxt/pull/827

@veryCrunchy I merged my PR, putting together a big release and wanted to include that one. Feel free to keep this open and add reInit if you feel strongly about it :)