Implementation of Flora and PlantFunctionalType

Description
This PR provides a restructure and review of @AmyOctoCat's implementation of the Flora and PlantFunctionalType classes needed for the demography milestone.
The PR isolates the implementation of those two classes in order to start merging elements of that milestone down into develop rather than having large PRs. The main implementations are taken from #231 and revised.
Code changes:

The validation has dropped the use of json_dataclasses from #231 and dropped back to the parent marshmallow and marshmallow_dataclasses. The validation then happens on Python data objects, which pushes the file handling back into our code, but also makes it really easy to share that validation across different file formats.
The calculation of the q_m and  z_max_prop traits has been pushed back all the way into PlantFunctionalType since they are constant within a type and are now populated automatically within __post_init__.
The Flora class now has three factory methods for creating instances from JSON, TOML and CSV inputs. Each method handles file loading and minor data format alignment and then can use the same shared marshmallow validation schema. The code in #231 had a separate serialisation class, but this functionality has been replaced with the factory methods.

Other changes:

The module paths in #231 have been simplified to pyrealm.demography. This may change but I think we had more nesting than needed.
Tests are implemented for the pyrealm.demography.flora module.
Docs should now build, including the simple canopy sketch (more or less an accidental inclusion).

Fixes #276 (issue)
Type of change

[x] New feature (non-breaking change which adds functionality)
[ ] Optimization (back-end change that speeds up the code)
[ ] Bug fix (non-breaking change which fixes an issue)

Key checklist

[x] Make sure you've run the pre-commit checks: $ pre-commit run -a
[x] All tests pass: $ poetry run pytest

Further checks

[x] Code is commented, particularly in hard-to-understand areas
[x] Tests added that prove fix is effective or that feature works

Codecov Report
Attention: Patch coverage is 98.05825% with 2 lines in your changes missing coverage. Please review.

Project coverage is 95.44%. Comparing base (1f315ba) to head (bae7df5).
Report is 5 commits behind head on develop.

Files with missing lines
Patch %
Lines

pyrealm/canopy_model/model/flora.py
98.05%
2 Missing :warning:

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #277      +/-   ##
===========================================
+ Coverage    95.29%   95.44%   +0.15%
===========================================
  Files           28       29       +1
  Lines         1720     1823     +103
===========================================
+ Hits          1639     1740     +101
- Misses          81       83       +2

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Nice catch @j-emberton - that's a really likely area for errors. I originally had the PFT dataclass without defaults, which would have prevented that failure mode, but this is effectively replacing the existing TModelTraits constants class.
I think it probably makes sense to enforce strict mode when using the factory methods - that's particularly true of CSV, where you'd have to handle NA if you wanted to support partial definitions for some fields. Is there a canned approach to this? My immediate thought is to have a base PlantFunctionalTypeStrict that is used for the loading schema and then a derived PlantFunctionalType as now is the programmatic interface with defaults.

Also, I'm thinking that the PFT dataclass should have frozen=True.

@j-emberton How about the solution in the most recent pushes? I've made the decision to make the file loading strict but retained the version with defaults as a way to get that defined set of default values and get a slightly friendlier programmatic API.

@j-emberton How about the solution in the most recent pushes? I've made the decision to make the file loading strict but retained the version with defaults as a way to get that defined set of default values and get a slightly friendlier programmatic API.

I think that's a simpler fix than anything I was going to suggest

@j-emberton How about the solution in the most recent pushes? I've made the decision to make the file loading strict but retained the version with defaults as a way to get that defined set of default values and get a slightly friendlier programmatic API.

This is a simpler solution than anything I was going to suggest and It adds some good safety rails for the user.

There's apparently conflicts in the ref.bib file. I'll be good to approve once they're fixed.

@j-emberton Conflicts all fixed.


Unable to use in automations due to lack of device enlisting

Checklist

[X] I have filled out the template to the best of my ability.
[X] This only contains 1 feature request (if you have multiple feature requests, open one feature request for each feature request).
[X] This issue is not a duplicate feature request of previous feature requests.

Is your feature request related to a problem? Please describe.
The AC is being listed as an entity only but not as a device. Though I am able to manually control all the aspects of the AC through the dashboard, I am not able to do any automations.
Describe the solution you'd like
List the device and and enlist all the actions separately from the entity.
Describe alternatives you've considered

Additional context

You can call them in automations with climate.room


Ember 2.13.2 Deprecation ember-cli-babel

When running "ember s" on Ember 2.13.2 the first thing i see is the following deprecation
DEPRECATION: Addon files were detected in [directory]/node_modules/ember-breadcrumbs/addon, but no JavaScript preprocessors were found for ember-breadcrumbs. Please make sure to add a preprocessor (most likely ember-cli-babel) to in dependencies (NOT devDependencies) in ember-breadcrumbs's package.json.

Sorry, ignore, just seen that this has been fixed on 0.1.10


Edison

Thanks for this handy recipe. But Carver is being retired, do either of you have an update that works on Edison? I tried to follow as-is, but fails for me in Edison shell (after some delay):
Warning: no access to tty (Bad file descriptor).
Thus no job control in this shell.
TUNNEL
Succesfully opened notebook!
Kill this process to end your notebook connection.
channel_new: internal error: channels_alloc 10010 too big.
channel 4: open failed: connect failed: Connection refused
This was on Mac.
I also tried connecting from Windows using latest putty to get any security fixes, but then with Windows browser I also get (quickly) connection refused along with a simultaneous drop in the putty connection.

I don't have any updated version. On Edison its been convenient enough to just run the notebook just on the login node ssh edison -L 8888:localhost:8888 ipython notebook. You could also try ipython.nersc.gov.

I am not using this script anymore, not even on Carver, I have been using a simpler script, which should work on Edison as well.
I get an interactive node, then I have a notebook script:
#!/bin/bash
ssh -N -f -R $NOTEBOOKPORT:localhost:$NOTEBOOKPORT $NOTEBOOKHOST
ipython notebook --port=$NOTEBOOKPORT --no-browser

then define the variables in my bashrc, of course you need a public IP for this to work.
I often run this from work desktop and then forward port to my laptop if I'm working remotely.

Thanks both, I will use interactive. I think the hint about "public IP" may be a crucial piece of information (this wasn't obvious to me). So for a laptop, connecting via some static server to the cluster, the steps to get something working are something like:

Make a script "sshipython" on the server, with, e.g.:
ssh -L<EMAIL_ADDRESS>ipython notebook --no-browser --port $NOTEBOOKPORT--port-retries=0 || sleep infinity
(the sleep stuff is supposed to start ipython only if it isn't already running and using the port)

Every time you want to use ipython notebook, ssh from the laptop to the server a) forwarding localhost:$NOTEBOOKPORT to $NOTEBOOKPORT and b) running "sshipython".on the server

Then view the notebook at localhost:$NOTEBOOKPORT. This assumes you've set up authorized_hosts appropriately on both the server and the cluster, and to itself on the cluster.
However getting additional python modules working is a can of worms that I've not solved yet (not helped by the fact the Edison shell is csh which module python/2.7-anaconda does not support..)

I think your steps look correct. you can use backticks to format code on github, it makes it easier to read.
you can change your login shell on nim.nersc.gov
I installed my own anaconda in my home.

don't make sshipython on the server.

ssh from your laptop to Edison
you do qsub -I to get an interactive node, then launch my notebook script so you forward the port to your NOTEBOOKHOST (the server).
when the job starts from your laptop launch ssh -L to forward a local port from laptop to the server
connect to localhost:NOTEBOOKPORT on your laptop

@marius311 @cmbant
with the retirement of Carver, I had to switch to using notebooks Edison.
Configuration is quite complex...but I got it working.
See my tutorial:
http://zonca.github.io/2015/09/ipython-jupyter-notebook-nersc-edison.html
if any of you would like to test and send feedback, that would be great.
Still an unsolved issue is that the ccm queue is very slow. I think we should ask NERSC to provide more CCM nodes.


Getting broken manifest generated .. scss files mapping to png, ttf, woff

version 2.2.0
webpack 4.41.2
mini-css-extract-plugin 0.8.0
part of manifest :
cat dist/manifest-5.4.9.json|grep scss
  "nitro-common.scss": "https://www.drivenow.com.au/webdata/nitro/turboiconfont-webfont-dff16c533467fba9f5f98be6509cb3e8.ttf",
  "RecordManager.scss": "https://www.drivenow.com.au/webdata/nitro/outline-icons-ef60a4f6c25ef7f39f2d25a748dbecfe.woff",
  "mobile.scss": "https://www.drivenow.com.au/webdata/nitro/receipt-perforation-3d56433a66ba84d84b19dbd2bb613b88.png",
  "landing.scss": "https://www.drivenow.com.au/webdata/nitro/info-bg-airport-adaf20c33cffe620622980eae7d13a37.png",
  "bootstrap.scss": "https://www.drivenow.com.au/webdata/nitro/glyphicons-halflings-regular-fa2772327f55d8198301fdb8bcfc8158.woff",

I found that webpack-assets-manifest fixed a similar bug https://github.com/webdeveric/webpack-assets-manifest/issues/39 fixed in https://github.com/webdeveric/webpack-assets-manifest/pull/40

A locally patched version applying the same fix appears to fix it :
changing moduleAsset to :
  var moduleAsset = function (loaderContext, module) {
    const emitFile = loaderContext.emitFile;
    loaderContext.emitFile = (name, content, sourceMap) => {
      moduleAssets[name] = path.join(
        path.dirname(name),
        path.basename(module.userRequest)
      );
      return emitFile.call(module, name, content, sourceMap);
    };
  };

and
changing compilation.hooks.moduleAsset to compilation.hooks.normalModuleLoader
results in no scss entries in the manifest and correct values for the ttf files etc..
  "turboiconfont-webfont.svg": "https://www.drivenow.com.au/webdata/nitro/turboiconfont-webfont-3b5e81f74bffde1c174c0dd2e298eac5.svg",
  "turboiconfont-webfont.eot": "https://www.drivenow.com.au/webdata/nitro/turboiconfont-webfont-5c87f74d71b93a54720c240baabee897.eot",
  "turboiconfont-webfont.woff": "https://www.drivenow.com.au/webdata/nitro/turboiconfont-webfont-dc8c4e2db2e54a5d58a3dc58b05be578.woff",
  "turboiconfont-webfont.ttf": "https://www.drivenow.com.au/webdata/nitro/turboiconfont-webfont-dff16c533467fba9f5f98be6509cb3e8.ttf",

I implemented a fork which resolves my issue, but breaks 1 test case

Also experiencing this issue.

I have the same issue with CSS files.
{
  "default.css": "wand.png?62a925af59561ea618a98a477b4a1585",
  "fonts/default.css": "fonts/roboto-v19-latin_latin-ext-regular.woff2",
  "overwrite.css": "ui-icons_333333_256x240.png?548a05af48ef6545db2fd999b12ca937"
}

It looks like these are files the css files use. The css itself gets built into one root css file that was defined in webpack.config.js:
module.exports = {
	entry: {
		'index': './index.js',
	},

And in index.js:
import './css/default.css';
import './css/overwrite.css';
import './css/print.css';

Where print.css is not included in the manifest file, which makes (a sort of) sense, because it doesn't reference images.
In other words: this may be intended behavior.

Same here. I have 2 entry points: app.js and vendors.js. I use css-loader and file-loader to copy+version the images/fonts found in my css files to the build directory and rewrite url()-s to them in the final css, then MiniCssExtractPlugin to extract css files.
The file vendors.js imports some css vendor packages and some of them have image references.
Here's the (wrong) generated manifest json:
{
  "/assets/build/index.css": "/assets/build/index.519adaf6.css",
  "/assets/build/index.js": "/assets/build/index.895c45a4.js",
  "/assets/build/index.css.map": "/assets/build/index.519adaf6.css.map",
  "/assets/build/index.js.map": "/assets/build/index.895c45a4.js.map",

  "/assets/build/vendors.css": "/assets/build/ec161a74448ccf2c18beefe8cca2b44a.svg",

  "/assets/build/vendors.js": "/assets/build/vendors.9b920610.js",
  "/assets/build/vendors.css.map": "/assets/build/vendors.4330fee9.css.map",
  "/assets/build/vendors.js.map": "/assets/build/vendors.9b920610.js.map"
}

As it sees, vendors.css has wrong value. It holds the last asset (see the output below) handled by file-loader:

Since I only need the chunks (js and exported css files) in the manifest file, my workaround is the plugin's filter option (see its documentation here):
    filter: (FileDescriptor) => {
      // fix for file-loader imported assets
      return FileDescriptor.isChunk;
    }

However I'm not really sure if it's safe, but all seems working fine:
 "/assets/build/vendors.css": "/assets/build/vendors.4330fee9.css"

I also had this issue when using file-loader and an svg image as background-image in my css. I temporarily fixed it by excluding svgs from the manifest.
new ManifestPlugin({
  publicPath: '',
  filter: (file) => {
    return !file.path.endsWith('.svg');
  },
}),

Is this being worked on at all. It's causing a lot of headaches for me trying to setup a manifest in a new project.

Note: The original post for this issue is likely a duplicate of #167
Hey all! I've taken over maintenance of the plugin and am doing some housecleaning. For Issues over a year old without a reproduction, we're going to go the route of closing them first. However, they're not dead! If the issue is still pending and still a problem, please reply with a reproduction and we'll reopen post-haste.
Please provide a reproduction by choosing one of the options below:

Using the REPL.it plugin reproduction template at https://repl.it/@shellscape/manifest-plugin-repro
Provide a minimal repository link (Read https://git.io/fNzHA for instructions).
Please use NPM for installing dependencies!
These may take more time to triage than the other options.

⚠ ZIP Files are unsafe and maintainers will NOT download them.


The Blacklist was created in my download directory

I'd prefer the blacklist be somewhere in my LibreWolf profile rather than my download directory is it possible to move it?  What about syncing to the cloud?  Is it expected to be in the download directory?

It would be even better if it the blacklist could be shared across all browsers like %APPDATA%\uBlacklist\

Who's fault is that?

The blocklist file will not be created in your download directory unless you press the export button.


[Cosmos] updates for release

updates for release

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-cosmos


Agentic cycle execution

Want to run the crew with a Critic agent in loop. Basically creating two different agents, one prompted to generate good outputs and the other prompted to give constructive criticism of the first agent's output. Can't make it possible. Any pointers?

second task must include 'context' that is output of first task.

That I got, but who takes the output of the second task? Seems like it needs to be a new third task. Also, we want to break this loop conditionally, so will need to create this linking on the fly, but seems to be not possible as Crew is created only once, unless branching is possible with Crew.
Eg.
first_producing_task = Task(...)
first_critic_task = Task(...,context=[first_producingtask])
second_producing_task= Task(..., context=[first_critic_task])

@gdagitrep, if I understand well, you are asking how to make a kind of Quality & Assurance (QA) to say if the task should pass and maybe restart the process again.

Aha, got it, i think yes it can be achieved with third task attached to the first agent. Try with memory=True Perhaps this would obviate the need to pass the content of the first and the context of the second in the last Task (not sure)

@Tavernari Sort of yes, taking inspiration from : https://arxiv.org/abs/2303.17651

@xitex What about the 4th task which needs to review work done by third?

Could you provide a code snippet example to understand how tasks 3 or 4 will help it? I am really curious to understand it.

So the idea is to give opportunity to LLM to iterate on its work, and not just give it a single shot to produce result. Kind of derived from RLHF, but instead of human feedback, we use another agent to provide feedback. Take this example:
Math_agent: Solves math problems, and produces an output. If a feedback is provided, it considers it to improve on the answer.
Reviewer_agent: Reviews the outputs of math problems, and provides feedback if mistakes are made. If no mistakes, output SUCCESS

Now create tasks for these two agents; give the first one a math problem, and the second task will have the problem as well as the output produced by the Math agent, and run them in loop, only breaking out when Reviewer_agent produced SUCCESS.

I think in this case you should use Process.hierarchical so you can manager agent and give a task to relevant agent. Another Consensual Process only planned so for now you can try hierarchical. You can find examples https://github.com/joaomdmoura/crewAI-examples

Related to this manager my issue is in this scenario.
I have these Agents:

manager
writer
translator

Tasks:

get content from source
write a text based on the source

context: 1

outputfile: some path

translate the content to some language

context: 2

outputfile: some path

Review the final content

How to make the task 4. Force execute task 2 if the content is wrong or missing important detail?

https://docs.crewai.com/how-to/Hierarchical/#hierarchical-process-overview
Here in example you can see that manager assign agent to task so your don't need to assign (i don't see code so must say that). To evaluate result manager should know what important detail he must to check so try include this to the task or try to generate some plan (plan task)

Thanks for the return.
Even with GPT-4, the manager was not reliable.
For example, in one of the tasks to generate some content, I put on the expected goal the instruction to restart the task if the result doesn't match the requirements. By the way, it did not respect this rule. The manager wrote that it was wrong and accepted the wrong content instead of restarting the task.
Am I doing something wrong? Should the validation live in another task?

There is little doumentation on the functionality of the manager. Offloading planning to the user would be ok if you incorporated some sort of ability to make training the manager LLM in concepts like https://github.com/allenai/lumos to know the planning break down (is this customizable by the use somehow to their trained planning/action combinations. It would be great to have some integration with a training process for planning / actions if developer is responsible for the LLM manager training. Training of the manager would be critical to tools available. Perhaps ICL of the Manager as mentioned by someone before may implicitly finetune the manager LLM if we had access to it to disclose the tools we want available to agents etc.
Based on the docs it would appear the hierarchy is very flat given the manager evaluates all returned outputs (assuming based on task "expected_output"), so no real hierarchy is feasible (my understanding is the manager it top and all other agents are one level below) unlike say unlike the autogen state transition graphs that can be scripted to whatever state transitions you want and however deep you need and if iteration is allowed). As noted in another blog I read if we are letting the LLM do the planning and allocation, perhaps should we allow the LLM to do the agent creation also (as per autogen (https://microsoft.github.io/autogen/docs/reference/agentchat/contrib/agent_builder/).
Perhspa there are features I am missing that are not documented yet?

There is little doumentation on the functionality of the manager. Offloading planning to the user would be ok if you incorporated some sort of ability to make training the manager LLM in concepts like https://github.com/allenai/lumos to know the planning break down (is this customizable by the use somehow to their trained planning/action combinations. It would be great to have some integration with a training process for planning / actions if developer is responsible for the LLM manager training. Training of the manager would be critical to tools available. Perhaps ICL of the Manager as mentioned by someone before may implicitly finetune the manager LLM if we had access to it to disclose the tools we want available to agents etc.
Based on the docs it would appear the hierarchy is very flat given the manager evaluates all returned outputs (assuming based on task "expected_output"), so no real hierarchy is feasible (my understanding is the manager it top and all other agents are one level below) unlike say unlike the autogen state transition graphs that can be scripted to whatever state transitions you want and however deep you need and if iteration is allowed). As noted in another blog I read if we are letting the LLM do the planning and allocation, perhaps should we allow the LLM to do the agent creation also (as per autogen (https://microsoft.github.io/autogen/docs/reference/agentchat/contrib/agent_builder/).
Perhspa there are features I am missing that are not documented yet?

I have the same impression


Added Other option and input box

Description of changes

Added an Other option to status.js and input box to interests.js
Issue Resolved

Fixes #410

@sabrown84 What issue does this resolve or address?

@sabrown84 Thanks for taking this issue on!  Looks like you've added most of what we need - just need a few more tweaks IMO:

In interests.js, the input element has been added, but it's plain and unstyled right now.  It should be styled to match how inputs look throughout the rest of the UI. I suggest using the FormInput component that wraps input elements, see signup.js for an example.  There should also be some vertical space between the input and the 'Select any disciplines you are interested in', which is currently immediately below it.

I also noticed that even if I entered another language as a text string into the input element, that it wasn't being included in the payload sent to the backend API.  There may be an issue with the event handler: onChange={e => this.props.update(e)} currently on the input element? :

Hi @jjhampton thanks for the feedback. I understand what you mean. This is my first time interacting with React. I actually dreamed up that I wish the input box would only show up if the other checkbox is checked. I'm looking for a way to link the other checkbox to the input box. I researched that there is a way to link the two...I'm just fuzzy on how to implement it.

@jjhampton @sabrown84 I feel like this is the perfect area to use the Select component John just brought in on #475 - thoughts?

@kylemh Agree, multi-select would be nice for something like this. The react-select Select component seems to work well for this.

Any idea why I keep getting this error?

Any idea why I keep getting this message?

Is this an error you only get while on your branch?

Yes, @jmayergit I was trying to test the feature.

I'll fetch your branch and see if I can reproduce

Where are we on this implementation @sabrown84??

Hi @Cooperbuilt,
I keep running into debugging issues when trying to create a dropdown multi-select input option using the Select component from #475 . I was working through it with @jmayergit , but I'm trying to back track a little bit and learn some more about React and how to rework through this. If I'm taking too long I'm happy to follow a new set of eyes and learn from them.

Hey @sabrown84! Not trying to rush you at all, I just wanted to check in and see if you need some additional resources to wrap this up. If you are still interested in working on this, I can work with you on it if you'd like.

@Cooperbuilt I appreciate you offering your time, and I would really like that. Currently, I have shifted my focus to other issues right now and I was going to come back to this one over the weekend to give it my full attention again. It that works for you...can I contact you via Slack to work through it?

@sabrown84 of course! I will be in and out this weekend, but if you can give me some advanced notice on a time, I will do my best.

@Cooperbuilt Great, thank you thank you! I will let you know.

I'm going to close this due to inactivity. Feel free to open an updated PR with fixed conflicts and the requested changes.


docs: Mendable Search Improvements

New pin-to-side (button). This functionality allows you to search the docs while asking the AI for questions
Fixed the search bar in Firefox that won't detect a mouse click
Fixes and improvements overall in the model's performance

Thank you!
@dev2049 @nfcampos

thanks @nickscamara!


Updated validations.py python script

Updated valdiations.py python script.
Fixed the behavior of validate_user function in validation.py
closes: #1
Fixes #<issue_number_goes_here>

It's a good idea to open an issue first for discussion.

[ ] Tests pass
[ ] Appropriate changes to README are included in PR

@googlebot I signed it!

@googlebot I fixed it.

@googlebot I fixed it.


Fix Set Public Name on native

CC @keybase/react-hackers

I think this is related to having the box wrap the scrollview, and outer box not filling the screen properly.
I checked to make sure we aren't doing this anywhere else and it looked fine

@MarcoPolo some eslint errors

Gonna merge for @MarcoPolo so we can make a build, yell if that was a mistake.


we should email submitters when their submission was rejected or deleted

when deleting a submission from an active round, it should email the submitter saying it was rejected.
When changing the status of a submittion to rejected on an already ended round, it should email with a reason.
when deleting a submission from an already ended round, it should not email

seems good to me.

thank you ♥


az acr repository show-manifests is deprecated but acr manifest is in preview

Describe the bug

By deprecating "az acr repository show-manifests" and marking "acr manifest" as preview you're effectively removing any stable and future-proof way to do something as simple as retrieving manifests from a repository.
Please either make "acr manifest" stable or remove the deprecation notice.
Related command
az acr repository show-manifests
az acr manifest list-metadata
Errors
This command has been deprecated and will be removed in a future release. Use 'acr manifest list-metadata' instead.
Command group 'acr manifest' is in preview and under development. Reference and support levels: https://aka.ms/CLI_refstatus
Issue script & Debug output

Expected behavior
Either no deprecation notice or no preview notice
Environment Summary
azure-cli                         2.63.0
core                              2.63.0
telemetry                          1.1.0
Extensions:
account                            0.2.5
log-analytics                      0.2.2
ssh                                1.1.2
Dependencies:
msal                              1.30.0
azure-mgmt-resource               23.1.1
Python location '/opt/azure-cli/bin/python'
Extensions directory '/home/simao/.azure/cliextensions'
Python (Linux) 3.12.5 (main, Aug  9 2024, 08:20:41) [GCC 14.2.1 20240805]
Legal docs and information: aka.ms/AzureCliLegal
Your CLI is up-to-date.
Additional context
No response

Thank you for opening this issue, we will look into it.


Network usage with disabled Telemetry and extensions

Issue Type: Bug
When start VSCode with disabled telemetry and extensions it is try to connect to these hosts:

dc.services.visualstudio.com:443
az764295.vo.msecnd.net:443

How to stop VSCode to connnect to anywhere?
Steps to Reproduce:

start with code  --disable-extensions

VS Code version: Code 1.28.2 (7f3ce96ff4729c91352ae6def877e59c561f4850, 2018-10-17T00:23:51.859Z)
OS version: Windows_NT x64 10.0.17134

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-4510U CPU @ 2.00GHz (4 x 1995)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: enabledvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Memory (System)
15.90GB (12.74GB free)

Process Argv
--disable-extensions

Screen Reader
no

VM
0%

Extensions disabled

My settings to disable the network connections:
    "extensions.autoCheckUpdates": false,
    "extensions.autoUpdate": false,
    "extensions.showRecommendationsOnlyOnDemand": true,

    "workbench.settings.enableNaturalLanguageSearch": false,
    "workbench.enableExperiments": false,

    "telemetry.enableCrashReporter": false,
    "telemetry.enableTelemetry": false,

    "update.channel": "none",
    "update.showReleaseNotes": false,

    "npm.fetchOnlinePackageInfo": false,
    "git.autofetch": false,
    "update.enableWindowsBackgroundUpdates": false,

Thanks for reporting this @crash5!
As per IP address used by AppInsights, dc.services.visualstudio.com:443 is used by the applicationInsights sdk.
If you have telemetry disabled, then the sdk shouldnt have been initialized at all. I will look into it.
az764295.vo.msecnd.net:443 points to the cdn for a blob storage and is used in 3 scenarios:

To get list of malicious extensions that shouldnt be loaded
To get data for the extension recommendations of the kind "This extension is popular among the users of this workspace"
To get list of some UI experiments that we run from time to time.

Based on your settings, you have disabled the second and third feature. I'll have to check if the malicious extension check is being made when no extensions are loaded.

@ramya-rao-a I see two PRs were merged, should this issue be closed?

Yes, I was waiting for a Insiders build with the fix before closing this.
@crash5 With the latest Insiders, when you run code-insiders --disable-extensions --disable-telemetry along with all the settings you have already set to [disable network connections](url
https://github.com/Microsoft/vscode/issues/61646#issuecomment-432811668), you should no longer be  seeing any outgoing network connections.
Can you please give the insiders a try?

Sorry for the late reply. Thank you for your work @ramya-rao-a .

Tested with 1.29.1
Version: 1.29.1 (user setup)
Commit: bc24f98b5f70467bc689abf41cc5550ca637088e
Date: 2018-11-15T19:13:36.375Z
Electron: 2.0.12
Chrome: 61.0.3163.100
Node.js: 8.9.3
V8: 6.1.534.41
Architecture: x64

With --disable-extensions --disable-telemetry it looks fine.
With the settings above and without the --disable-telemetry option still try to connect to dc.services.visualstudio.com:443.

If I'm right:

I can't disable telemetry only from settings I have to use the --disable-telemetry argument to start the vscode?
It's not possible to disable the malicious extensions check from settings. It works only with --disable-extensions option on start?

Ps: The code -h command output does not contains the --disable-telemetry option.

I can't disable telemetry only from settings I have to use the --disable-telemetry argument to start the vscode

Yes. This is true. I have logged #63404 to track this. Please subscribe to that issue

It's not possible to disable the malicious extensions check from settings. It works only with --disable-extensions option on start

Yes, we don't want to have a setting to disable the check as you can understand this is an important check to make if you are using extensions from the marketplace.

The code -h command output does not contains the --disable-telemetry option.

@joaomoreno Any idea why this is so?

Thanks @joaomoreno
I have updated
https://github.com/Microsoft/vscode/issues/63404 with the your note


added refinable fallback_urls

removed non working django 1.10+ middleware compatibility from multilanguage
adjusted multilangauge_switcher urls to refine get_multilang_urls instead of get_urls

thx


Make requires lazy to improve startup times

Addressing #122. Added some simple caching as well, not sure if it's beneficial or not. These changes seem to cut startup times by about 50% on average.
cc @stefanpenner

@rwjblue updated

:shipit:


Jenkins master script

👉 @MiguelRomero
closes #1

Branch name is wrong, should be 1-feature/provision-jenkins
Scripts must be in a scripts folder


Fix accidental exclusion of default values

The inclusion of exclude_unset=True in the JSON producing functions was preventing non-modified defaults (such as type from being included in the JSON output.
This PR fixes that, and also explicitly sets exclude_defaults to False and prevents it from being overridden by the kwargs.

Discussed on call.

OK to merge


Docs fixes

This combines 3 different fixes to the manual:

Make the bcrypt generation more straightforward, and applies a couple of small bug fixes to the getting started instructions regarding this step. This is done for both main and stable.
Fix download link in stable. This takes the fixed link (as of fce5eab4) and applies it to stable.
Adds slashes to all URLs. There's a lot of detail in the commit message for that fix - in essence this fixes relative links if you're linked to the manual, so to reproduce see the installation page and click on "Securing access to the dashboard" in the Pre-requisite section. This should fail now, but the staging docs should pass the same test.

I tried the new steps and worked perfectly, thanks @ozamosi - also checked all the links opened as expected, raised a couple of comments but nothing major/urgent


🎈 11.4.0

🎈 Release
Preparation for 11.4.0 release. This is a big one, it's been a while!
Wait for #877 to go in before releasing. That PR is already on the changelog.
Comparison to 11.3.0: https://github.com/ignitionrobotics/sdformat/compare/sdformat11_11.3.0...sdf11

Needed by

https://github.com/ignitionrobotics/ign-gazebo/pull/1343
https://github.com/ignitionrobotics/ign-gazebo/pull/1387

Checklist

[x] Asked team if this is a good time for a release
[x] There are no changes to be ported from the previous major version: https://github.com/ignitionrobotics/sdformat/compare/sdf11...sdf9
[x] No PRs targeted at this major version are close to getting in: this seems to be on hold https://github.com/ignitionrobotics/sdformat/pull/643
[x] Bumped minor for new features, patch for bug fixes
[x] Updated changelog
[x] Updated migration guide (as needed)
[ ] Link to PR updating dependency versions in appropriate repository in ignition-release (as needed): :warning: TODO

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

Codecov Report

Merging #879 (26c0b10) into sdf11 (2e396c7) will decrease coverage by 0.00%.
The diff coverage is 87.50%.

@@            Coverage Diff             @@
##            sdf11     #879      +/-   ##
==========================================
- Coverage   88.91%   88.90%   -0.01%
==========================================
  Files          73       73
  Lines       11086    11087       +1
==========================================
  Hits         9857     9857
- Misses       1229     1230       +1

Impacted Files
Coverage Δ

src/Model.cc
89.36% <80.00%> (-0.16%)
:arrow_down:

src/parser.cc
86.72% <100.00%> (-0.05%)
:arrow_down:

Continue to review full report at Codecov.

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


Nextcloud updates

Description of changes

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] 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

[x] 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
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 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

[x] Fits CONTRIBUTING.md.

Thanks, will try it out tonight!

FYI I deployed it on my instance and things worked fine

Can we please update the apps when updatimg Nextcloud?

I never remember to do this because I am not using it. I just auto update imperative apps. :/


Intel process infos indexing fix

I believe the issues #58 #56 are resulting from index mismatches between process ids and the infos lists. This is likely to happen when zombie process are detected. Furthermore, if the indices for infos1 and infos2 are not in synch, you may also see issue #38.
I have fixed the issue in the get_powercap_power() and get_rapl_power() functions.

A great, thanks so much!


[CHORE] - Clear out a missed console.log()

What work needs to be performed?
We recently had a very cool pagination feature added but the reviewer (... me) missed that one console.log() was left in from development.
It all works perfectly, but we need to remove one line to stop it dumping to the console every time the pagination is initiated.
Look in: ./src/utils/paginate.js
And remove: console.log(results) from line 36.
That's all.
Additional information
No response

Hi @MizouziE I would like to contribute to this project and handle this issue.

@TeaBizzy thanks for your interest in contributing.
You can start working on it if you wish, @MizouziE will assign it to you ASAP :rocket:

Sorry I'm late, thanks a lot @TeaBizzy !

@MizouziE Friendly reminder we can close this now :smiley:


Add PythonModuleAttribute overload

Related to https://github.com/IronLanguages/ironpython2/issues/587.
This allows the numpy (for .NET) egg (at http://code.enthought.com/.iron/eggs/index.html) to import with ipy.
Note that I think the egg is 32-bit so it needs to run against ipy32. Also, needs binding redirects:
<assemblyBinding xmlns="urn:schemas-microsoft-com:asm.v1">
    <dependentAssembly>
        <assemblyIdentity name="IronPython" publicKeyToken="7f709c5b713576e1" culture="neutral" />
        <bindingRedirect oldVersion="<IP_ADDRESS>" newVersion="<IP_ADDRESS>" />
    </dependentAssembly>
    <dependentAssembly>
        <assemblyIdentity name="Microsoft.Scripting" publicKeyToken="7f709c5b713576e1" culture="neutral" />
        <bindingRedirect oldVersion="<IP_ADDRESS>" newVersion="<IP_ADDRESS>" />
    </dependentAssembly>
    <dependentAssembly>
        <assemblyIdentity name="Microsoft.Dynamic" publicKeyToken="7f709c5b713576e1" culture="neutral" />
        <bindingRedirect oldVersion="<IP_ADDRESS>" newVersion="<IP_ADDRESS>" />
    </dependentAssembly>
</assemblyBinding>

Related links:

http://blog.enthought.com/python/scipy-for-net
https://stackoverflow.com/questions/29397540/how-to-install-numpy-and-scipy-for-ironpython27-old-method-doenst-work
https://github.com/enthought/numpy-refactor

Hi all,
I really need pynum with ironpython 2.7.7. But I have still error while importing clr module.
I try to investigate by using ironpython in debug mode. I get the exception while importing but I don't find the root cause. Can someone give me some idea about that?
Thanks for your help

Can you run with -X:ExceptionDetail?

Yes sur :
"No module named clr\r\n   à Microsoft.Scripting.Runtime.LightExceptions.ThrowException(LightException lightEx) dans G:\Download\ironpython2-ipy-2.7.7\ironpython2-ipy-2.7.7\Runtime\Microsoft.Dynamic\Runtime\LightExceptions.cs:ligne 164\r\n   à Microsoft.Scripting.Runtime.LightExceptions.CheckAndThrow(Object value) dans G:\Download\ironpython2-ipy-2.7.7\ironpython2-ipy-2.7.7\Runtime\Microsoft.Dynamic\Runtime\LightExceptions.cs:ligne 158\r\n   à Microsoft.Scripting.Interpreter.FuncCallInstruction`2.Run(InterpretedFrame frame) dans G:\Download\ironpython2-ipy-2.7.7\ironpython2-ipy-2.7.7\Runtime\Microsoft.Dynamic\Interpreter\Instructions\CallInstruction.Generated.cs:ligne 692\r\n   à Microsoft.Scripting.Interpreter.Interpreter.Run(InterpretedFrame frame) dans G:\Download\ironpython2-ipy-2.7.7\ironpython2-ipy-2.7.7\Runtime\Microsoft.Dynamic\Interpreter\Interpreter.cs:ligne 132\r\n   à Microsoft.Scripting.Interpreter.LightLambda.Run2[T0,T1,TRet](T0 arg0, T1 arg1) dans G:\Download\ironpython2-ipy-2.7.7\ironpython2-ipy-2.7.7\Runtime\Microsoft.Dynamic\Interpreter\LightLambda.Generated.cs:ligne 97\r\n   à IronPython.Compiler.PythonScriptCode.RunWorker(CodeContext ctx) dans G:\Download\ironpython2-ipy-2.7.7\ironpython2-ipy-2.7.7\Languages\IronPython\IronPython\Compiler\PythonScriptCode.cs:ligne 81\r\n   à IronPython.Compiler.PythonScriptCode.Run(Scope scope) dans G:\Download\ironpython2-ipy-2.7.7\ironpython2-ipy-2.7.7\Languages\IronPython\IronPython\Compiler\PythonScriptCode.cs:ligne 72\r\n   à IronPython.Compiler.RuntimeScriptCode.InvokeTarget(Scope scope) dans G:\Download\ironpython2-ipy-2.7.7\ironpython2-ipy-2.7.7\Languages\IronPython\IronPython\Compiler\RuntimeScriptCode.cs:ligne 100\r\n   à IronPython.Compiler.RuntimeScriptCode.Run(Scope scope) dans G:\Download\ironpython2-ipy-2.7.7\ironpython2-ipy-2.7.7\Languages\IronPython\IronPython\Compiler\RuntimeScriptCode.cs:ligne 59\r\n   à Microsoft.Scripting.Hosting.CompiledCode.Execute(ScriptScope scope) dans G:\Download\ironpython2-ipy-2.7.7\ironpython2-ipy-2.7.7\Runtime\Microsoft.Scripting\Hosting\CompiledCode.cs:ligne 83\r\n   à JDSU.Scripting.PythonEngine.ExecuteScript(String Code, String File)\r\nImportError: No module named clr\r\n"

Can you paste inside of triple backticks? Also, how are you running your code?

No module named clr\r\n à Microsoft.Scripting.Runtime.LightExceptions.ThrowException(LightException lightEx) dans G:\Download\ironpython2-ipy-2.7.7\ironpython2-ipy-2.7.7\Runtime\Microsoft.Dynamic\Runtime\LightExceptions.cs:ligne 164\r\n à Microsoft.Scripting.Runtime.LightExceptions.CheckAndThrow(Object value) dans G:\Download\ironpython2-ipy-2.7.7\ironpython2-ipy-2.7.7\Runtime\Microsoft.Dynamic\Runtime\LightExceptions.cs:ligne 158\r\n à Microsoft.Scripting.Interpreter.FuncCallInstruction`2.Run(InterpretedFrame frame) dans G:\Download\ironpython2-ipy-2.7.7\ironpython2-ipy-2.7.7\Runtime\Microsoft.Dynamic\Interpreter\Instructions\CallInstruction.Generated.cs:ligne 692\r\n à Microsoft.Scripting.Interpreter.Interpreter.Run(InterpretedFrame frame) dans G:\Download\ironpython2-ipy-2.7.7\ironpython2-ipy-2.7.7\Runtime\Microsoft.Dynamic\Interpreter\Interpreter.cs:ligne 132\r\n à Microsoft.Scripting.Interpreter.LightLambda.Run2[T0,T1,TRet](T0 arg0, T1 arg1) dans G:\Download\ironpython2-ipy-2.7.7\ironpython2-ipy-2.7.7\Runtime\Microsoft.Dynamic\Interpreter\LightLambda.Generated.cs:ligne 97\r\n à IronPython.Compiler.PythonScriptCode.RunWorker(CodeContext ctx) dans G:\Download\ironpython2-ipy-2.7.7\ironpython2-ipy-2.7.7\Languages\IronPython\IronPython\Compiler\PythonScriptCode.cs:ligne 81\r\n à IronPython.Compiler.PythonScriptCode.Run(Scope scope) dans G:\Download\ironpython2-ipy-2.7.7\ironpython2-ipy-2.7.7\Languages\IronPython\IronPython\Compiler\PythonScriptCode.cs:ligne 72\r\n à IronPython.Compiler.RuntimeScriptCode.InvokeTarget(Scope scope) dans G:\Download\ironpython2-ipy-2.7.7\ironpython2-ipy-2.7.7\Languages\IronPython\IronPython\Compiler\RuntimeScriptCode.cs:ligne 100\r\n à IronPython.Compiler.RuntimeScriptCode.Run(Scope scope) dans G:\Download\ironpython2-ipy-2.7.7\ironpython2-ipy-2.7.7\Languages\IronPython\IronPython\Compiler\RuntimeScriptCode.cs:ligne 59\r\n à Microsoft.Scripting.Hosting.CompiledCode.Execute(ScriptScope scope) dans G:\Download\ironpython2-ipy-2.7.7\ironpython2-ipy-2.7.7\Runtime\Microsoft.Scripting\Hosting\CompiledCode.cs:ligne 83\r\n à JDSU.Scripting.PythonEngine.ExecuteScript(String Code, String File)\r\nImportError: No module named clr\r\n

I run my application and ironpython DLLs in debug mode in VS2017.

I get the error "No module named errno" when I start ipy.exe with DebugV2 configuration in VS.
But it can start, after I get No module named clr" if I type "import clr" in the console.

I continue the debug.
I saw that CLR module attribute is declared like that :
[assembly: PythonModule("clr", typeof(IronPython.Runtime.ClrModule))]
Without plateform specification.
So in function LoadBuiltins (pythonContext.cs) , we don't load CLR module
I certainly missed something.....

No module named errno probably means the standard lib isn't getting loaded.

Yes sure. But why builtin modules are not loaded?

@maxmont It's hard to tell. As @slozier mentioned, it worked for him, so we would need more information on how you build and so forth. Also, we really don't have the resources to support 2.7.7 anymore.

@slide, I understand that you don't want to loose time by supporting old version. But my application is using .NET 3.5 and I have to stay with ironpython 2.7.7 (not to take too much risks)
@solzier, can you provide me 2.7.7 msi generated with patch "Add PythonModuleAttribute overload". I would appreciate a lot. Thanks in advance for your help.

Hi all,
Sorry again, but do you think you will be able to provide 2.7.7 msi generated with patch "Add PythonModuleAttribute overload"?
Thanks again for your help

@maxmont You are aware that the numpy package that you're trying to use requires .NET 4.0, right?

@slozier Yes but it's not a problem. In fact I have a library in .NET3.5 which uses ironpython (but it doesn't use numpy. But my app which is in .NET4.0 uses this library and numpy.

I can't produce an msi, but here's the patched version of IronPython.dll for .NET 3.5.

@slozier Many thanks for your help. I apreciate.
I patch official installation of ironpython 2.7.7 with your DLL.
When I execute ipy.exe I get this error :
"unable to load type 'Microsoft.Scripting.IValueEquality' from the assembly 'Microsoft.Dynamic, Version=<IP_ADDRESS>, ....."

@maxmont IValueEquality only exists in the .NET 3.5 assemblies. The official installation is .NET 4.0 so you tried patching the 4.0 assemblies which obviously won't work. The .NET 3.5 assemblies are under Platforms\Net35...

Hi all, Thanks again for your help. But numpy seems not to be compatible with new version of ironpython. So I adandon this solution.
Instead, I try and validate a commercial solution  : Extreme Optimization which seems to be a high level mathematics library. I have another issue by using it with ironpython. I open a new thread deicated for that.....


invoice2data: 0.3.6 -> 0.4.2

Description of changes
Update invoice2data to version v0.4.2 released on 2023-02-11.
This change also

Installs ghostscript, which is used to convert pdfs to images used as input for tesseract
Uses tesseract 5 instead of tesseract 3
Switches from xpdf (which is also marked as insecure) to poppler_utils, which is the recommended provider of pdftotext according to the invoice2data repo.

Things done

Built on platform(s)

[x] x86_64-linux
[ ] 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

[x] 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/)
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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 215853 run on x86_64-linux 1

1 package built:

invoice2data

Fixed commit message (package is named invoice2data, not invoice2pdf).

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/1831


Add return event

Describe the project you are working on
No response
Describe the feature / enhancement and how it helps to overcome the problem or limitation
This event is a similar implementation of return keyword. Whetever is called, ends the timeline execution.
Adittional information
No response

Do not call it "return", call it "end timeline". Calling it return would actually be extremely confusing otherwise especially if paired with the "Condition" event, so someone could put in a "return" event, expecting it to return back to where the condition was called from, only for it to suddenly end execution.
I still want an equivalent of a Return event though, but call it "Return to Previous Timeline", or simply "Go Back".


Adds support to capistrano multistage completion

Uses shipit because cap is a reserved word, so cap completion doesn't work.
$ shipit |
production staging

$ shipit production |
deploy                       -- Deploy a new release
deploy:bundle                -- Bundle
deploy:check                 -- Check required files and directories exist
deploy:check:directories     -- Check shared and release directories exist
deploy:check:linked_dirs     -- Check directories to be linked exist in shared
deploy:check:linked_files    -- Check files to be linked exist in shared
deploy:cleanup               -- Clean up old releases
deploy:cleanup_rollback      -- Remove and archive rolled-back release
deploy:finished              -- Finished
deploy:finishing             -- Finish the deployment, clean up server(s)
deploy:finishing_rollback    -- Finish the rollback, clean up server(s)
deploy:log_revision          -- Log details of the deploy
deploy:published             -- Published
deploy:publishing            -- Publish the release
deploy:restart               -- Restart application
deploy:revert_release        -- Revert to previous release timestamp
deploy:reverted              -- Reverted
deploy:reverting             -- Revert server(s) to previous release
deploy:rollback              -- Rollback to previous release
deploy:started               -- Started
deploy:starting              -- Start a deployment, make sure server(s) ready
deploy:symlink:linked_dirs   -- Symlink linked directories
deploy:symlink:linked_files  -- Symlink linked files
deploy:symlink:release       -- Symlink release to current
deploy:symlink:shared        -- Symlink files and directories from shared to release
deploy:updated               -- Updated
deploy:updating              -- Update server(s) by setting up a new release

how to use this?

@adisos it wasn't merged into oh-my-zsh yet (@robbyrussell ???). So for now you could use it as a custom plugin, something similar to:
https://github.com/phstc/dotfiles/blob/master/zshrc#L39
https://github.com/phstc/dotfiles/tree/master/oh-my-zsh/custom/plugins/capistrano_custom

Oops, sorry for missing this. /cc @robbyrussell


#1639 QA - vault details - add dollar value for loans card

What UI/UX improvements would you like to be implemented:
Related to 1639
Acceptance Criteria: to display the dollar value of the loan card
Android:

[ ] Loans > Your vaults > Loans sections verify relative dollar value is displayed for loan amount

iOS:

[ ] Loans > Your vaults > Loans sections verify relative dollar value is displayed for loan amount

USD value to be under dToken amount.


Error in webapp/error.log

I saw the below error in my webapp.log. I am unfamiliar with graphite. I am new to a project using it.
Data is not being populated into the charts. Running on Centos 7.3. Is this a known issue?
mv: cannot move â€˜/opt/graphite/storage/.index.tmpâ€™ to â€˜/opt/graphite/storage/indexâ€™mv: : No such file or directorycannot stat â€˜/opt/graphite/storage/.index.tmpâ€™
: No such file or directory

You can safely ignore this error

Seems like maybe it was trying to use single quote and was encoded wrongly?
https://stackoverflow.com/questions/2477452/â€-showing-on-page-instead-of


Tor: Add GUI support for v3 onion service client authorization

Some time ago I opened ticket #307 for having a Tor GUI on Opnsense and a very nice one has been made available. Thanks to the developers.
Since Tor <IP_ADDRESS> is now on Opnsense I think a lot of people would like to get the new v3 onion service client authorization and I believe some changes are required on the GUI to accommodate the new scheme.
You can find a very good summary and links on a Onionshare feature request https://github.com/micahflee/onionshare/issues/697
I also found a nice write-up about actually using this new feature https://gist.github.com/mtigas/9c2386adf65345be34045dace134140b

I did some tests and manually using both V3 addresses and the new client authorization scheme was pretty straightforward.
I used the script mentioned in the gist to generate the proper keys.

 ./onion-svc-v3-client-auth.sh
X25519 Private Key:
3DXT2BT7KC7JMPZSSBQUVDTUOCD4TUECCQ23CVFC46ROWUG3Y5FQ

X25519 Public Key: (give this to the onion service)
2T3JCSCQDKGGXYX3GNNLARG5JKXFPUMAHIK6ZCDTLHC5LD47DE5A

=====
Tor client configuration
=====
Make sure you have ClientOnionAuthDir set in your torrc. In the
<ClientOnionAuthDir> directory, create an '.auth_private' file for the
onion service corresponding to this key (i.e. 'bob_onion.auth_private').
The contents of the <ClientOnionAuthDir>/<user>.auth_private file should
look like:

    <56-char-onion-addr-without-.onion-part>:descriptor:x25519:3DXT2BT7KC7JMPZSSBQUVDTUOCD4TUECCQ23CVFC46ROWUG3Y5FQ

i.e.:
    p53lf57qovyuvwsc6xnrppyply3vtqm7l6pcobkmyqsiofyeznfu5uqd:descriptor:x25519:3DXT2BT7KC7JMPZSSBQUVDTUOCD4TUECCQ23CVFC46ROWUG3Y5FQ

=====
Onion service configuration
=====
Inside the HiddenServiceDir for this onion service, create an
/authorized_clients/ subdirectory and a '.auth' file for the user (i.e.
'alice.auth'). The contents of the <HiddenServiceDir>/authorized_clients/<username>.auth
file should look like:

    descriptor:x25519:2T3JCSCQDKGGXYX3GNNLARG5JKXFPUMAHIK6ZCDTLHC5LD47DE5A

I then added those keys to both servers and clients:
Opnsense side
/usr/local/etc/tor/torrc
HiddenServiceDir /var/db/tor/testme/
HiddenServiceVersion 3
HiddenServicePort 80 <IP_ADDRESS>:80

And then added the authorized key:
cd /var/db/tor/testme
mkdir authorized_clients
chown _tor:_tor authorized_clients
echo "descriptor:x25519:2T3JCSCQDKGGXYX3GNNLARG5JKXFPUMAHIK6ZCDTLHC5LD47DE5A" > test.auth
chown _tor:_tor test.auth

Restarted Tor service and got the new long .onion address from the GUI (or hostname file). For example:
p11lf57qovyuxyzc6xnrppyaaa3vtqm7l6pcobkmyqsiofyeznfu5bcd.onion
It was then very simple to add the right file on the client side:
Client side (Tor Browser)
First edited torrc to add path to the client authentication directory:
echo "ClientOnionAuthDir /home/alphazo/.tor-browser-en/INSTALL/Browser/TorBrowser/Data/Tor/ClientOnionAuthDir" >> ~/.tor-browser-en/INSTALL/Browser/TorBrowser/Data/Tor/torrc
And then add the private key file and updating .onion address of the hidden service provided by Opnsense (without the .onion extension).
echo "p11lf57qovyuxyzc6xnrppyaaa3vtqm7l6pcobkmyqsiofyeznfu5bcd:descriptor:x25519:3DXT2BT7KC7JMPZSSBQUVDTUOCD4TUECCQ23CVFC46ROWUG3Y5FQ" >> /home/alphazo/.tor-browser-en/INSTALL/Browser/TorBrowser/Data/Tor/ClientOnionAuthDir/test.auth_private

This issue has been automatically timed-out (after 180 days of inactivity).
For more information about the policies for this repository,
please read https://github.com/opnsense/plugins/blob/master/CONTRIBUTING.md for further details.
If someone wants to step up and work on this issue,
just let us know, so we can reopen the issue and assign an owner to it.


Download data by cycle and pass

Instead of asking for data between DATE-1 and DATE-2 some users may want data for CYCLE-23 to CYCLE-26.  Expand subscriber to search/download by cycle as well.  This would really be helpful for JASON series missions.  Other users may want data by pass, too.  For missions that don't include pass and cycle appropriate warning messages have to be displayed.

We will begin with cycle based downloads. if multiple passes are specified, only one cycle can be used.

Closed. Will be available in 1.8.0 release. Cycles only, the "pass" based downloads are not added right now.


[Doc-enhancement] OnTeamsMessagingExtensionSelectItemAsync details are not explained in documentation

Documentation don't have details about the purpose and use of OnTeamsMessagingExtensionSelectItemAsync event. So, Can you add details in documentation.
And OnTeamsMessagingExtensionSelectItemAsync does not trigger in mobile. This event is not implemented for mobile. So, Please add note for the same.

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

ID: a4f3b627-c87b-6a19-9c26-7170a99f38ee
Version Independent ID: a9546115-7c5d-39c9-ca8a-3cf3e02ff621
Content: Respond to search command - Teams
Content Source: msteams-platform/messaging-extensions/how-to/search-commands/respond-to-search.md
Product: msteams
GitHub Login: @surbhigupta
Microsoft Alias: anclear

@HunaidHanfee-MSFT - could you please help to provide code snippets for enabling Tap action and OnTeamsMessagingExtensionSelectItemAsync  handling part?


Add contributing guidelines

We should add contributing guidelines for opening issues, submitting pull requests, etc. Example: https://github.com/Opentrons/opentrons/blob/edge/CONTRIBUTING.md

We have implemented this


Sync version of livekit api

Hi, thanks for the good work.
I just wonder can we have a version of sync api of the LiveKit api (together with async api). So that we can call api in some sync framework such as Django or Celery? The 2 version sync and async live in one repo is just like some famous packages such as Redis, Google cloud, OpenAI.

i don't know if this is necessary, because django has support for asynchronous functions https://docs.djangoproject.com/en/4.2/topics/async/

The exception raised from /site-packages/livekit/api/_twirp_client.py:86, in TwirpClient.request(self, service, method, data, headers, response_class)

can you show the snippet of the code where you wrap  client.room.create_room?
In my case i created a function called create_room with async def and used "await client.room.create_room" inside. Then in my django view i used async_to_sync(create_room)() and it works fine.

Could you try moving client.room.create_room to another function, and then use async_to_sync on it? like
async def createRoom(name):     await client.room.create_room(CreateRoomRequest(name=name) )
And then on your task do:
async_to_sync(createRoom)(f"room-{room_id}")
This timeout error could be something else thought

Agreed with @Gabriel-rd2, you could use async_to_sync or other techniques to handle the async lib. We have a small team and will not able to maintain two versions of the library at the moment.

Hi @huynguyen-ff
Have you fixed this error? Please guide for me. Thank you.

@TheNha , because I used django-channel so I created an async worker for me and I move the livekit api into that worker.

@huynguyen-ff Yeppp. Can you share the function, which you have created an async worker.
Thank you very much.

@TheNha I used this one https://channels.readthedocs.io/en/latest/topics/worker.html  because I used websocket channel. You can use any others as long as it support asyncio (async function).

@TheNha I used this one https://channels.readthedocs.io/en/latest/topics/worker.html because I used websocket channel. You can use any others as long as it support asyncio (async function).

I'm using flask, is there a better way to do it? I've tried workaround from Stack overflow. But it doesn't work.


Transactions refactor

What was wrong?
Related to Issue #1021
Cute Animal Picture

I've modified cancun to match the older forks. I can go the other way if that's really what we prefer.


Fixing a calculation error in the scaling recommendation

This should also apply to the 2.3 and tip version of the docs.

my first reaction to this doc change was that it was a bit confusing.
rather than saying X * 2 / 500 i believe it may be more readable to say X / 250.
also, if we're changing the recommendation by double the VMs, is that noted in the release notes for PAS 2.2 if that is a significant change from PAS 2.1 and prior?

@jbayer a little more background on this recommendation change: https://pivotal.slack.com/archives/CEBMUG067/p1543504532228600.
And I agree that the formula is confusing. After talking with @jtuchscherer, we decided to remove the Scaling section on this page all together and replace it with a reference to https://docs.pivotal.io/pivotalcf/monitoring/key-cap-scaling.html, which provides the same info and is up-to-date.

@jbayer Just to add to what @paigecalvert said: The scaling recommendation has always been wrong. The confusion and bad wrong wording came from the fact that a syslog adapter can handle 500 syslog drains. The scaling recommendations didn't take it into account that we have each drain binding serviced by 2 adapter (therefore doubling the number of drains). I am not sure if we should call this out in the release notes.


Allow PM to handle delegates for grouped tables

ProMotion has both a TableScreen class and a GroupedTableScreen class, both of which inherit from TableViewController.
The pm_handles_delegates? method properly refers to ViewController (which both PM::Screen and PM::WebScreen inherit from), but refers directly to PM::TableScreen, excluding PM::GroupedTableScreen. As a result, view_did_load was firing twice on GroupedTableScreens.
The method now checks against TableViewController instead. I also added specs to run the 'proper delegate caller' specs on both TableScreen and GroupedTableScreen.

Nice catch!

:+1:

:+1:


Only track released versions in oldVersions.

This would avoid test failures when a release is in progress.

More fixes are needed, I'm looking.

We used to have logic to allow up to one missing file to account for in-progress releases: https://github.com/apache/lucene/blob/releases/lucene/9.8.0/lucene/backward-codecs/src/test/org/apache/lucene/backward_index/TestBackwardsCompatibility.java#L805-L843. We need to figure out a way to replicate this logic with the new parameterized testing.
It would help if we could have access to an authoritative list of released versions, e.g. via the DOAP file maybe?

We used to have logic to allow up to one missing file to account for in-progress releases: https://github.com/apache/lucene/blob/releases/lucene/9.8.0/lucene/backward-codecs/src/test/org/apache/lucene/backward_index/TestBackwardsCompatibility.java#L805-L843. We need to figure out a way to replicate this logic with the new parameterized testing.

Hahaha that logic is crazy....

It would help if we could have access to an authoritative list of released versions, e.g. via the DOAP file maybe?

This could work, but that would require network access at test time which is not allowed (security manager). Replicating the DOAP file to the repo seems like it defeats its purpose.
I tend to replicate the previous logic.

See this issue for more insights to the "crazy logic": https://issues.apache.org/jira/browse/LUCENE-5936

@RmuirSays: "yes please, lets not do the network. the smoketester can do that. In fact mike already wrote the code."

@uschindler I also have a bias against adding more dependencies on network resources, this is why I was thinking of the DOAP files since we have them in the repository: https://github.com/apache/lucene/blob/main/dev-tools/doap/lucene.rdf.

@uschindler I also have a bias against adding more dependencies on network resources, this is why I was thinking of the DOAP files since we have them in the repository: https://github.com/apache/lucene/blob/main/dev-tools/doap/lucene.rdf.

Oh that's new to me. Cool idea. Parsing with a simple stax parser should work. Unfortunately those are outside of test resources and can't be loaded easily.
I am tempted to add the legacy behaviour back. It's proven to work since 10 years. The code wasn't changed since that old issue.

P.S.: Unrelated to this issue here: we should update the doap file to use https links.

I pushed a new commit that revives the old logic that allowed up to one version missing backward-compatibility data.

With our last change, we may need to change the script that adds the backwards indexes. Or is this now how it worked before?

@uschindler I'm not sure to understand what you mean. The script that adds bw compat indices indeed needs updating to update the oldVersions array when introducing bw compat indices for a new release.

(which is what Simon is doing at #13095)

Thanks!


register DNS for rkt containers

This would probably involve metadata service, but there is no obvious way of setting name/hostname of a container, nor there is any notion of tags. Would be great to get some better understanding of this.

You can set the hostname with rkt run --hostname=...:
      --hostname=""		pod's hostname. If empty, it will be "rkt-$PODUUID"

About tags, can you give an example how you would use them?

About DNS: rkt does not start a DNS server but you can configure the container's /etc/resolv.conf with the following command line parameters:
      --dns=			name servers to write in /etc/resolv.conf
      --dns-opt=		DNS options to write in /etc/resolv.conf
      --dns-search=		DNS search domains to write in /etc/resolv.conf

Is it enough for your use case, or do you need more?
There is a pending issue about getting the DNS information from the CNI plugin instead of the rkt command line parameters but it is not implemented yet: https://github.com/coreos/rkt/issues/2044
If it would be useful for Weave, please say so on the issue above ;)

/cc @steveeJ

@alban ok, so --hostname wasn't documented where I looked (https://github.com/coreos/rkt/pull/2519). It should sufficient.

@errordeveloper
Is there still the need for a new feature in rkt or is --hostname sufficient for your use-case?


svg rendered werid

Description
svg rendered weired

Reproduction URL
https://typst.app/project/ph2HV071Y0tJ1lVojf8ytt
Operating system
Web app
Typst version

[X] I am using the latest version of Typst

The project does not seem to be accessible, can you post the SVG here?

https://typst.app/project/wW5kE31GRsDhL6TcmJ0eIg this can be seen and edited.

Ah I see now... The problem is that your SVG seems to use icc colors, which is currently not supported by the upstream crate. So if you somehow have control over that, you should try to change it so it uses "normal" rgb colors. But a bug nevertheless.

Thank you for your reply! I now see how the problem happens.
BTW, Is there any CLI tool to convert icc to rgb? inkscape seems can't.

Not that I know of, unfortunately. :(

Ah I see now... The problem is that your SVG seems to use icc colors, which is currently not supported by the upstream crate. So if you somehow have control over that, you should try to change it so it uses "normal" rgb colors. But a bug nevertheless.

Is this a duplicate of https://github.com/typst/typst/issues/3781 ?

No, this is a different issue.

@Wongboo et al. Martin from the Inkscape project is currently working on full ICC support for Inkscape. This is an enormous project, but as soon as https://gitlab.com/inkscape/extensions/-/merge_requests/646 is completed it should be trivial to do what you need.


Add a test for Base.showerror [av skip]

Skipping AppVeyor since it's failing anyway.

Okay, apparently I didn't set up the [av skip] thing here. Canceled AV.


Fix windows build

Issue
Stabilize Windows and other distribution builds so that they run everywhere without errors
Things done

Lowered minimal Python version to 3.10.11
Dropped Windows x86 support
Bumped Python version to 3.10 in Windows binaries
Upgraded Tor Proxy bundle in the Windows binaries to the latest (Tor Expert Bundle 13.0.8)
Added Komodo Dash ElectrumX (cipig) production servers in the hardcoded list
Upgraded and fixed a lot of dependency issues

Release built from this branch were tested by various community members


feat: support using mwc token for fabric openai transformer

Related Issues/PRs

#xxx
What changes are proposed in this pull request?
Briefly describe the changes included in this Pull Request.
How is this patch tested?

[ ] I have written tests (not required for typo or doc fix) and confirmed the proposed feature/bug-fix/change works.

Does this PR change any dependencies?

[ ] No. You can skip this section.
[ ] Yes. Make sure the dependencies are resolved correctly, and list changes here.

Does this PR add a new feature? If so, have you added samples on website?

[ ] No. You can skip this section.
[ ] Yes. Make sure you have added samples following below steps.

Find the corresponding markdown file for your new feature in website/docs/documentation folder.
Make sure you choose the correct class estimators/transformers and namespace.
Follow the pattern in markdown file and add another section for your new API, including pyspark, scala (and .NET potentially) samples.
Make sure the DocTable points to correct API link.
Navigate to website folder, and run yarn run start to make sure the website renders correctly.
Don't forget to add <!--pytest-codeblocks:cont--> before each python code blocks to enable auto-tests for python samples.
Make sure the WebsiteSamplesTests job pass in the pipeline.

/azp run

/azp run

Codecov Report

Merging #2070 (a3ec118) into master (dd7ba7c) will decrease coverage by 81.72%.
The diff coverage is 50.00%.

@@            Coverage Diff             @@
##           master   #2070       +/-   ##
==========================================
- Coverage   87.15%   5.43%   -81.72%
==========================================
  Files         306     306
  Lines       16117   16134       +17
  Branches      858     846       -12
==========================================
- Hits        14046     877    -13169
- Misses       2071   15257    +13186

Files Changed
Coverage Δ

...synapse/ml/cognitive/openai/OpenAICompletion.scala
0.00% <ø> (-93.75%)
:arrow_down:

...ure/synapse/ml/cognitive/openai/OpenAIPrompt.scala
0.00% <ø> (-85.49%)
:arrow_down:

...re/synapse/ml/cognitive/CognitiveServiceBase.scala
57.42% <50.00%> (-24.20%)
:arrow_down:

... and 287 files with indirect coverage changes

/azp run

/azp run


Results download fix

Fixes the "download results" error, and the test that should have caught it.

rebased and merged


Add module on serverless (AWS/Lambda)

Investigate create lambda functions in workshop.

Is it work merging this work into https://signalfx.github.io/otelworkshop/
@geoffhigginbottom @hagen-p

Closing this as no longer being worked on


Building Issues (Can Not Handle Multiple Files)

When trying to build the template, the output that I get is:

This prevents any building

The issue is with GenType, not the template


more tags + alphabetical listing

I have added a few tags and listed them all by alphabetical order (label). What do you think?

Looking good and working perfectly for me. I think the one that can better assess this PR is Matt, as he was working on unify tags between OpenCap and AddBiomechanics.

@antoinefalisse I've talked with Matt and we have a question:
What "Right Leg" and "Left Leg" stand for? is is right or leg single squat? Is it standing on a single leg? Or something generic to refer to an arbitrary action with either leg?

@AlbertoCasasOrtiz

Or something generic to refer to an arbitrary action with either leg?
Yes that's what I meant. You could associate it with like squats, or countermovement jumps, or any type of jumps for instance. In rehab/sports, people often distinguish between double-leg and single-leg activities. It would be great if we could capture that in our datasets, and this might be a way to do it. I'm open to suggestions. Maybe we should also add dominant and non-dominant.

something like that if people do it with a single leg.

Sounds good to me. I've talked with Matt and we both think having dominant and non-dominant would be good.
If you add it, I think you can directly merge then

@AlbertoCasasOrtiz see last commit. Feel free to merge.


UI extensions serve in Node

WHY are these changes introduced?
This PR is part of the effort of moving the logic for creating, building, and serving UI extensions from Node to Go.
WHAT is this pull request doing?
This PR Duplicates the serve command from packages/ui-extensions-go-cli in packages/app.  Shopifolk are then opted into the new version unless they specify the RUN_AS_USER=true env var.
This code has a few concerns:

Creating a bundler to build extensions
Creating a Payload Store to maintain the state of all extensions
Creating a web server to respond to fetch requests to /extensions/...
Creating a web socket server at /extensions/ that can be used to update the Payload Store, mediate messages, or inform clients of changes to the Payload Store

I'll explain these concerns in depth now:
Bundler
First, for each extension the bundler uses esbuild to watch for changes to the extension entry file.  E.g: /extensions/[extension]/src/index.jsx.  When ESBuild detects a change:

EsBuild builds the file to /extensions/[extension]/dist/main.jsx
EsBuild informs the Payload Store that of the build status.
The Payload Store then recalculates the data for that extension.
The WebSockets then pass an update message to every client with the updated extension data

Second, for each extension the bundler uses chokidar to watch the /extensions/[extension]/locales/ folder for changes to json files. When a file is added, edited or deleted:

Chokidar informs the PayloadStore that extension has changed
The Payload Store recalculates the data for that extension, including the localization data.

Payload Store
The Payload Store is responsible for keeping the state of extensions up to date.  At a high level it is an event emitter with methods to mutate it's internal state.  Whenever it updates it's internal state it emits events to allow listeners to do what they need to do.  In more detail it is responsible for:

Updating extension data: The websocket server could ask the Payload Store to update an extension.  An example of this would be when a user hovers, or hides an extension in the Dev Console.  Or the Bundler could ask to update the extension.
Updating app data:  Extension points can save arbitrary data to the app data that the Payload Store is responsible for.  In this situation an extension point would send a websocket message with new App Data, and the Payload Store would merge that data with existing data.
Informing Event Listeners on updates: The websocket server listens for when the Payload Store  updates data and informs clients of those updates.
Calculating data: One example of this is calculating the localization for any extension with localization files.  This involves parsing the JSON files on disk and converting those files into the Payload stores internal state.  Arguably this could be the responsibility of the bundler, but the current architecture is quite simple to reason about.

WebServer  for fetch Requests
The HTTP WebServer for fetch requests has the following end points:

/extensions/dev-console returns the UI and assets for the dev console
/extensions/[id] returns the redirect URL to view the extension point for that extension.  That extension point will then connect to the server extensions/[id]/assets/some/asset.js to get the assets for that extension.  Right now this is just the dist/main.js
/extensions/ returns the JSON data for all the extensions the developer has setup

In addition, the following applies to all endpoints

We allow Cross origin requests
We disable caching
We log requests for debugging

Websocket server
The web socket server is responsible for:

Sending the app & extensions state to clients when a client first creates a websocket connection.
Informing clients when there are changes to an extension.  An example of this is when a user edits a localization the end result is that the new extension data is sent via an UPDATE message to every client. Another example of this is when it responds to an UPDATE Message sent by a client.  It does this by asking the Payload Store to update the app or extension data with the data the client passed in the message. Once the Payload Store completes this operation it is the responsibility of the Websocket server to pass the new Payload Store data to all clients via an UPDATE message.
Mediating messages from clients.  Clients can send DISPATCH message.  The websocket server then passes this message on to every client.

How to test your changes?
@dmhenry @pepicrft and @alfonso-noriega have tested each extension manually.  So I'll only add instructions for 2 extensions (Thanks Trish @vividviolet for these)
Checkout

Ensure your dev shop has the following beta force_checkout_one, checkout_one_argo_extensions
Run yarn shopify app dev --path ./fixtures/app.
Click on the extension preview link
Verify that you see an extension rendered inside the order summary
Make some changes to localization files in /fixtures/app/extensions/checkout_ui
Verify that the extension is reloaded and reflect the changes
Make some changes to the extension script in /fixtures/app/extensions/checkout_ui
Verify that the extension is reloaded and reflect the changes

Admin Product Subscription (published and unpublished extension)

Ensure your dev shop has the following beta argo_dev_portal, argo_dev_portal_mobile and the app is installed on your shop
Run yarn shopify app dev --path ./fixtures/app.
Click on the extension preview link
Verify that Web loads with the Dev Console appearing as an iframe at the bottom of the screen
Verify that you are redirect to a product details page
Navigate down to the Purchase Options card and click on Add purchase options
Verify that you can select your app and render the "Create new option" extension point
Make some changes to the extension script
Verify that the extension is reloaded and reflect the changes
Verify the following all work with the Product Subscription extension:

hovering on the Product Subscription extension should highlight that extension
clicking on the hide icon for the Product Subscription extension should make the extension unavailable. If you have
multiple apps installed that powers the Subscription extension then clicking on "hide" should remove your test app from the list
clicking on the link for the Product Subscription extension should navigate to the product details page for the first product if you're not already on the page
clicking on the "refresh" button should force reload the extension

Testing a deployed the extension

Deploy your extension
Make some changes to the extension script
Click on the preview link again to load up the extension
Select your app and render the "Create new option" extension point
Make some changes to the extension script
Verify that the extension is reloaded and reflect the changes
Click the "hide" button on the Dev Console
Verify that you now see the published extension script - local changes should have no effect
Click the "show" button on the Dev Console
Verify that you see your local script rendered

Measuring impact
How do we know this change was effective? Please choose one:

[x] n/a - this doesn't need measurement, e.g. a linting rule or a bug-fix
[ ] Existing analytics will cater for this addition
[ ] PR includes analytics changes to measure impact

Checklist

[x] I've considered possible cross-platform impacts (Mac, Linux, Windows)
[x] I've considered possible documentation changes

@pepicrft  I've updated the PR Description with some details to help testers.  We could also use these as a starting points for any docs we want to add.


Improve Signup and Login Pages

Clear Design: Keep the design clean and straightforward, with easy-to-understand instructions.
Mobile Friendly: Ensure the pages work well on mobile devices for seamless access.
Prominent Buttons: Make signup and login buttons clearly visible and easy to click.
Social Login: Offer the option to sign up or log in using social media accounts.
Password Strength: Include a tool to help users create strong passwords.
Error Handling: Provide instant feedback on incorrect inputs with clear error messages.
Privacy Statement: Clearly state your privacy policy to reassure users.
Forgot Password: Offer a simple process for password recovery.
Accessibility: Ensure the pages are accessible to all users, including those with disabilities.
Continuous Improvement: Regularly update based on user feedback to enhance user experience.

@jaiyankargupta , no need to go with such details
Things for authpages:

Login:

in form, we have email fields

SignUp:

in form, need to have name, username, email, other fields

No need for password

After clicking on button, they'll be redirected to page which shows them to click on code that is emailed to them

@jaiyankargupta , update input label UI, and add login page


Get the milestone data consistently

GitHub stores PR data in both the PR and Issue data structures and the script fetches them at different times. This means that the PR data, which is fetched later, may lag behind. For the milestone detection this may result a PR getting through the milestone filter (which uses the issue data) and then getting displayed with a milestone that should be filtered (if the milestone has been applied in that time), because right now the filtering is on issue.milestone but the page is using pr.milestone.
Fix that by switching the display code to issue.milestone, at list it's consistent, either way it would render correctly on the next run.

@aescolar


I want to get the result body. Is there any way?

I'm using elastic In V5, because the business rewrites the result body, in addition to the standard result, there are other data to be obtained according to the body. What should I do?

You could use the PerformRequest method directly on the client, then deserialise the envelope on your side.


Functions of npm metadata generating for single npm package

Some steps tips for the npm metadata generating functions:
Reorganize the npm content files structure from the input tarball as the input is a bit different from the normal npm contents structure:
Extract the input tarball, relocate the npm tarball file, move the version metadata package.json from the extracted files, then generate package metadata from the version metadata and write it into the proper location.
This covers two kinds of npm standard paths: single path and scoped path, it's more reliable to get the path from the version name meta than the analytic request path.
BTW, for some metadata fields types defined here, which I found differ from the ones in Indy npm, so I've followed the input samples. About the jinja2 template for json, it's a little complex to declare in template, looping deeply, I'll try to use the environment class to render the metas.

@ligangty Thanks for your comments!
For the template or native json generates the file solution,
The class in python needs to be JSON serializable(default is not) when using marshmallow-dataclass(or json) to dump(s), but it only accepts the built-in types: https://docs.python.org/3.8/library/json.html#encoders-and-decoders. So we need to make the class inherited from JSONEncoder or implement the JSONEncoder.default() function during the dump. It will be complex to cover all kinds of data types in the json, so I use dict member of the instance to transform the standard python dict(both for version and package metadata), they could be JSON serializable as the structure in classes are simple enough so far.
Some shortcomings when using the json to generate metadata file directly: it could not customize the data formats like hiding fields of empty values, while it could also be met when using the object mapper in JAVA, so this should not be a big deal from my perspective.
The benefit is obvious that we don't need to edit the template file for json anymore once some fields or fields types changes.
So I leave the updates in the new commit(also merge the two metadata .py), and let me know if there is any issue around here before merging, thanks!


Adding json tags for ShardInfo struct to reuse for docstore

What changed?
Added json tag for ShardInfo for docstore shardManager implementation. TimerFailoverLevels and TransferFailoverLevels are not used as these are not persisted.

Why?

How did you test it?

Potential risks

Coverage increased (+0.3%) to 67.498% when pulling a792fa87f1ec9a5a7b2b70ffac5b6901d58b920e on anish531213:aanish/update-shard-info into 74c2c6910b6fd4194fa0a2b918f4168e0a4a6263 on uber:master.


wrong variable name

What does this PR do?
Changes a variable name which is wrongly referred to in mltrs.py
Collection: [Note which collection this PR will affect]
Changelog

if self.reconstruction_module_accumulate_predictions:

Usage

Variable name is not referred correctly

# Add a code snippet demonstrating how to use this

Before your PR is "Ready for review"
Pre checks:

[x] Make sure you read and followed Contributor guidelines
[x] Did you write any new necessary tests?
[x] Did you add or update any necessary documentation?
[x] Does the PR affect components that are optional to install? (Ex: Numba, Pynini, Apex etc)

[x] Reviewer: Does the PR have correct import guards for all optional libraries?

PR Type:

[ ] New Feature
[x] Bugfix
[ ] Documentation

If you haven't finished some of the above items you can still open "Draft" PR.
Who can review?
Anyone in the community is free to review the PR once the checks have passed.
Contributor guidelines contains specific people who can review PRs to various areas.
Additional Information

Related to # (issue)

Thanks @TimPaquaij, let's do a batch fix of typos in another PR!


Add CI workflows and docs

@dylex sorry to bother you about this again. Could you re-enable Jenkins for this repository as well as for PRs?
I had to start this repository over from scratch because I needed to copy over the git history of this code from its original repository in order to make previous versions of this Julia package installable from this new repository (see https://github.com/JuliaRegistries/General/pull/105961 in case you are curious).


Man caches updating after the tiniest changes due to nixos-version implementation

Describe the bug
Man caches are updated unnecessarily, taking an annoying amount of time during rebuilds.
Steps To Reproduce
Steps to reproduce the behavior:

Be using a flake config
Set documentation.man.generateCaches = true; (or programs.fish.enable = true; which sets the same option indirectly)
Set system.configurationRevision = mkIf (self ? rev) self.rev;
Commit and rebuild
Add a comment in your config
Commit and rebuild, and note the man caches being regenerated, despite absolutely nothing of any consequence changing.

Expected behavior
I would expect the system path not to need to be rebuilt for such a minor change, given that man caches take nontrivial work to generate.
Additional context
This is clearly because nixos-version is created with the system.configurationRevision baked into the script itself. It could be fixed by placing the changing information in a fixed location under /run/current-system and having the nixos-version command read that file to get the data to return.
Metadata
$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 5.15.81, NixOS, 22.11 (Raccoon), 22.11.20221208.04a75b2`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.11.0`
 - nixpkgs: `/etc/nix/path/nixpkgs`

When considering exactly how to go about this, it's probably work taking https://discourse.nixos.org/t/retrieving-nixos-version-derivation-path-from-nix/24539 into consideration.

This issue has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/retrieving-nixos-version-derivation-path-from-nix/24539/3


[404] Page not found: https://nf-co.re/docs/nf-core-tools/api_reference/2.14.1/pipeline_lint_tests

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Browser (please complete the following information):

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

Additional context
Add any other context about the problem here.

This issue also affects module_lint_tests.
To reproduce:

Navigate to https://nf-co.re/docs/nf-core-tools/api_reference/2.14.1/module_lint_tests
Click a link in the main section of the index page e.g. main_nf

Note that the sidebar links to the same pages work as expected. i.e. the actual link https://nf-co.re/docs/nf-core-tools/api_reference/2.14.1/module_lint_tests/main_nf works as expected.
The issue appears to be a bad link generation in the main index page.


Decrease Savestate Size

Run the JSON state through JSON Clean, and bring the state size down from 10000 to 5000?

It's compressed.


Update APT before installing puppet

Installing puppet would fail in with some base boxes. A simple 'apt-get
update' will fix that situation.

Thank you for contributing to openstack-dev/devstack-vagrant!
openstack-dev/devstack-vagrant uses Gerrit for code review.
If you have never contributed to OpenStack before make sure you have read the
getting started documentation:
http://docs.openstack.org/infra/manual/developers.html#getting-started
Otherwise please visit
http://docs.openstack.org/infra/manual/developers.html#development-workflow
and follow the instructions there to upload your change to Gerrit.


add reorderable backgrounds in theme editor

Right now, when you create a background, it will cover over all previous backgrounds (alpha channel included, but moot for colors).
By making them drag/droppable or with up down buttons, controlling this stack would be easy.
In the data layer, themes.theme.*.backgrounds is a list, so that could just be reordered with splices... or just a big set. Or, we could go to the more customary model and use a keyed tree and a linked list (again for the atomicity).

Closing for now... might re-emerge in another project, given time constraints...


No gamma correction on output to default SRGB-capable framebuffer with OpenGL on laptop (Intel graphics)

Looks like Mesa bug: https://bugs.freedesktop.org/show_bug.cgi?id=92759


upgrade mime for improved CJS handling

.cjs is being served as text/plain, which is incorrect. This PR upgrades the mime package as this is fixed in newer versions of it. See https://github.com/mjackson/unpkg/issues/355#issuecomment-1541520209

Looking forward for this PR to be merged, I got the same mime type issue when importing a ".cjs" file.

@mjackson shy bump :sweat_smile:

Fixes #355
Fixes #268

Please merge, would be great :+1:


venv not ignored for Python.

Hello,
I have noticed that thevenv folder is not or no longer ignored when using the Python template. Why is that so? GitHub still creates a .gitignore file where the venv folder is ignored when Python is selected.
Everything under # Environments is not included in the .gitignore file created by gitignore.io:
# SageMath parsed files
*.sage.py

# Environments
.env
.venv
env/
venv/
ENV/
env.bak/
venv.bak/

# Spyder project settings
.spyderproject
.spyproject

To be honest, I'm not quite sure if these folders were ever ignored and I just used the GitHub version. If so, can you add these entries? Thanks

Nevermind. I found this pull request: https://github.com/toptal/gitignore/pull/217
For those interested: the venv related files have been moved to virtualenv.

Since there is a dedicated ignore file for virtualenv, let's deduplicate the default entries from the Python template.


Error when loading screen

Got this when open the app and loading, it won't work with the nabu casa or ip adress

I have same issue, since v68.
Samsung galaxy s9+ / Android 10 b5
HA: 0.104.0b0

I'm o ha  0.103.6, Galaxy s10+ android 10


can 't toggle dark or light mode

aflter clicked the toggle button, it stucked and have no response after clicking,

the reverse is the same
can only be restored after reopening a new page
can visit lodatang.com to verify the problem

There seems to be other problems with your site; it hangs when I try to close the tab. Do you have the repo with a list of commits / changes you've made to the base repo?
I think the problem likely has nothing to do with toggling dark mode and is related to some other change.

only some basic configuration information and icons have been modified

remove this stuff  ， it works


alt+tab needs to open any editor regardless of group

VSCode Version:
Version 1.10.2
Commit 8076a19fdcab7e1fc1707952d652f0bb6c6db331
Date 2017-03-08T14:02:52.799Z
Shell 1.4.6
Renderer 53.0.2785.143
Node 6.5.0

OS Version:
Windows 10

Alt+tab needs to open recently used editor REGARDLESS of group. I understand there is a showAllEditors command that you can map to alt+tab. But that's NOT the same thing.

This issue has already been reported. Please use the search function next time before reporting an issue and/or this time if you are curious about the progress of the issue.

I tried to use the search but couldn't find anything. Would you mind linking the related already reported issue. Thanks.


Unexpected footgun while using Repo.exists?/2

Elixir version
1.17.3
Database and Version
PostgreSQL 14.11
Ecto Versions
3.11.2
Database Adapter and Versions (postgrex, myxql, etc)
0.17.5
Current behavior
I quickly skimmed through Repo.exists?/2's docs the other day and I saw that a keyword list could be provided as a second argument. My mind immediately assumed that an API similar to what Repo.get_by/2 supports was supported, I.e. that something like this could work:
> Repo.exists?(User, id: 1)
true

And it does, but it does not behave that way, what actually happens is that the id: 1 argument I gave the function is just ignored and it checks for the existence of an User instead of an User with id 1.
Now this is clearly my fault for not paying attention while reading, but I ended up shipping a bug to prod due to this mistake (again, my fault). Could a warning be added if keys other than :prefix or one of the Shared options are given to the second arg?
(Sorry if this is not the right place to post this)
Expected behavior
Maybe something like this?
Repo.exists?(User, id: 1, name: "John")
> warning: `[:id, :name]` are not supported as options, only: `[:prefix, ...]` are
> true

This is tricky for us to handle because the options given to Repo.exists? are actually understood and interpreted by the database adapter. And we even have callbacks such as prepare_options that allow users to parse and customize options. So I don't think we can do something to effectively address this. We could special case some keys, such as :id, but it may not be worth it.

Would it help at all if we expanded the description? Or do you think that would that have not prevented the issue from happening?

At the risk of sounding silly, a small admonition or callout with a colored background would've grabbed my attention for sure 😂 . But maybe a text disclaimer in the docs would be good enough too

We could special case some keys, such as :id, but it may not be worth it.

Agreed on probably not being worth it.
I looked for similar issues and couldn't find any in the repo so maybe updating the docs is good enough?
Thanks to both for the quick responses!

Would you like to send a PR giving the admonition text a try? Then we can collaborate and suggest improvements too, if you are not 100% comfortable.


Rename type=part -> type=building_part

Description
Recently agreed upon by schema working - theme=buildings/type=part should be renamed to theme=buildings/type=building_part as the type fields must be unique across the schema. The phrase "part" is too generic.
Reference
Testing
Automated github test
Checklist

[ x] Add relevant examples.
[ x] Add relevant counterexamples.
[ x] Update in-schema documentation using plain English written in complete sentences, if an update is required.
[ x] Update Docusaurus documentation, if an update is required.
[ ] Review change with Overture technical writer to ensure any advanced documentation needs will be taken care of, unless the change is trivial and would not affect the documentation.

Documentation Website
Docs preview for this PR.

Still need to update docs

Solves https://github.com/OvertureMaps/schema-wg/issues/287