Some more enhancements of YASA

Here are some suggestions from @epugh

[x] Links back to the legacy Solr admin, so if the users can’t do something in YASA, just pop me back to the legacy interface

[ ] Edit config files in Yasa

[ ] Streaming expressions support

[ ] More understanding of the shape of data, https://github.com/o19s/bearded-wookie

@kezhenxu94 and @epugh I think that as of today, it would seem that the priority is feature parity for that which we would like to keep in Solr. Shaping data is an entirely different effort and seems like it would be another plugin at this point. I'm not saying completely, but there is a lot of complexity there.

@kezhenxu94 and @epugh I think that as of today, it would seem that the priority is feature parity for that which we would like to keep in Solr.

Yes I agree totally, let’s just list the features (in individual issues) that are missing in this project but existed in the default admin and we could do that before this one

Thoughts on how to measure feature parity?  Is it at all useful to try and build an inventory of EVERYTHING solr admin does?   How much in Solr admin could be removed if we rethought what the admin tool does?  I thought about making a list of every feature that is in Solr Admin and comparing to YASA, but not sure how useful that would be...   Or maybe make a giant daunting list!

I thought about making a list of every feature that is in Solr Admin and comparing to YASA, but not sure how useful that would be... Or maybe make a giant daunting list!

@epugh  both are good to me, I'm just wondering the diff feature list, and I'll align them ASAP, I need you buddies to help on that list

I rename the title of this issue to Feature requests of YASA as the checklist is new features compared to the old admin, let's list the diff in another issue, agree?

@MarcusSorealheis @kezhenxu94 I started a spreadsheet, just to see if this works:
https://docs.google.com/spreadsheets/d/13tjLXZezQmnRh2iSquj51a3VAUklUa_e6pVLLVu8sUM/edit?usp=sharing
What are your thoughts on this?  Does going through all of the Solr Admin and listing things at the level of detail that I did provide value?  I only did the main Dashboard, Logging, and Cloud aspects.   I could keep on going listing out all the areas, and then add a column that compares them to what is in Yasa????

@MarcusSorealheis @kezhenxu94 I started a spreadsheet, just to see if this works:
https://docs.google.com/spreadsheets/d/13tjLXZezQmnRh2iSquj51a3VAUklUa_e6pVLLVu8sUM/edit?usp=sharing

The list looks to be what I've been expecting

What are your thoughts on this? Does going through all of the Solr Admin and listing things at the level of detail that I did provide value?

As we're going to reach the feature parity before Solr 9.0, this list gives a big picture (and the TODO list) before that, IMO, the list is valuable in this stage of YASA, and I'll develop YASA according to that list. Thanks @epugh very much

I only did the main Dashboard, Logging, and Cloud aspects. I could keep on going listing out all the areas, and then add a column that compares them to what is in Yasa????

That would be much more appreciated so that I can find out the missing features in YASA and fill the gap

Great!   I'll keep plugging away at it ;-)   And try and update the "Already in YASA" column.
I do worry that focusing too much on parity means we'll miss some opportunities to rethink what an Admin tool really should be!   Let's see how we do.

Hi @kezhenxu94 I hope your year is going well.   I've got some end of year time, if you want me to test out some more featurs to YASA ;-)

Hi @kezhenxu94 I hope your year is going well. I've got some end of year time, if you want me to test out some more featurs to YASA ;-)

Thanks @epugh , hope you’re well. To be honest, there is no much progress recently because of my personal affairs. 😔 I’ll be back to this project when I’m done. I’ll ping you then. Thanks again.

Also, I’d like to see more people joining so that I don’t become the bottleneck of it


Stretching X axis

Using ScatterPlot
I tried to find a way to stretch the X-axis.
When a scatter plot is drawn all of the data are shown inside the plot area.
Now, I would like to stretch the X-axis without resizing the plot object.
It means that a part of the X-axis will be outside the visible area.
It would be perfect if a horizontal slider is shown in order to horizontally pan using the slider.
I have tried resizing the plot object programmatically.
It works as expected.  However, the X-axis label disappears after a couple of resizes.
X-label should appear at the center of the X-axis.

If I would put a plot object in the panel, the horizontal slider will appear when plot object is wider than the panel.
However, is there any other way how to achieve the result I need?

I think you create more trouble with such approach than what you win with it. You stretch your control, so its right side will be unreachable in your UI, as I understand. This will create all sort of issues. Why would you want such unusual usage of a UI control?
If your container where your Plot is sitting, is always visible, just set the Plot as docked, and then the labels will be always visible at any resized step...

https://user-images.githubusercontent.com/51174971/227255511-19766fe1-b17b-4fe1-b314-48f34b232e7f.mp4

I think you create more trouble with such approach than what you win with it

I agree with this overall sentiment...

I have tried resizing the plot object programmatically. It works as expected.

This is good news! It sounds like you have solved your core problem as far as axis limits are concerned.

However, the X-axis label disappears after a couple of resizes.

Maybe we can add an option to left-align the axis label if it's not already in there. I'd be happy to review a PR if you want to try to add this!
This could be achieved by editing all instances of HorizontalAlignment.Center in this file:
https://github.com/ScottPlot/ScottPlot/blob/33fca4cbcb884161d61e05615604b008a0e7aac1/src/ScottPlot4/ScottPlot/Renderable/AxisLabel.cs#L148
Exposing a HorizontalAlignment property that defaults to Center would preserve the present behavior but also allow users to customize that behavior.

It would be perfect if a horizontal slider is shown in order to horizontally pan using the slider.

This is an interesting effect and it seems useful to demonstrate this on an FAQ page or as a demo added to the demo app. I'll leave this issue open until I can put this in.

#2501

Just gonna say that I've been waiting a long time for project 2501 🙃
https://www.youtube.com/watch?v=roUmza6tdjw

Hi @dusko23, I hope this code example helps set you on a patch toward a solution that works for you!

public partial class Form1 : Form
{
    ScottPlot.Plottable.SignalPlot MySignal;

    public Form1()
    {
        InitializeComponent();

        MySignal = formsPlot1.Plot.AddSignal(ScottPlot.DataGen.Sin(1_000, oscillations: 3));
        formsPlot1.Refresh();
    }

    private void hScrollBar1_Scroll(object sender, ScrollEventArgs e)
    {
        double visibleSpan = 100;
        double fraction = (double)hScrollBar1.Value / hScrollBar1.Maximum;
        double scrollableSpan = MySignal.PointCount * MySignal.SamplePeriod - visibleSpan;
        double xMin = fraction * scrollableSpan;
        double xMax = xMin + visibleSpan;
        formsPlot1.Plot.SetAxisLimitsX(xMin, xMax);
        formsPlot1.Refresh();
    }
}

Hi, this works for your case because you are using the signal. I use a scatterplot where the sample period does not apply.
I played further with Plotables Properties and discovered that important property was Missing.
a TAG
Like Plotable.Tag = "anything"
"anything" can be literary anything. It is 100% to the user. Plottable just holds that reference.
That would save a lot of work for further ScottPlot development.
A unique ID is such an example.
The user adds an ID to his class which also holds the unique string.
Or just declare Plotable.Tag = new GUID.ToString (VB notation) if he wants to trade a precious Tag Property for a simple String.
There is no need that ScottPlot to be aware of everything about how ScottPlot can be used.
Tag is an important property that enables keeping user business logic design independent of ScottPlot.
I have read discussion #694
Sedighzade made some good points.
No need to add any Interface. Just a pure Property named Tag which holds a reference to the Object, whatever Object represents.
Instead of populating ListBoxes while adding Plotables to the whole lot, it would be better to have the option to iterate in a loop through all plotables and populate lists as needed regarding scenarios that can be completely SCottPlot independent.
Adding Tag property which holds a reference to the Object, whatever it represents will set ScottPlot development free of many future desires that have nothing to do with ScottPlot Logic as such :-)

I use a scatterplot where the sample period does not apply.

Good point! In this case you can use the scatter plot's GetAxisLimits() method to get the min/max values in every direction

Adding Tag property which holds a reference to the Object

This idea comes up from time to time. It makes sense for GUIs, but not for console applications. Because users create all different types of GUIs, I leave the door open for users to solve this problem in the best suited to the particular GUI framework they are using.

#694
#695
#2114

The recommended way to pair arbitrary data with plot types or controls is to use a dictionary, as described in https://github.com/ScottPlot/ScottPlot/issues/2114#issuecomment-1238760801

Thanks, Scott, I see where are you coming from.
I was suggesting to add Tag because it is the standard thing to have at hand. Every Microsoft Windows GUI element has got one. It makes life easier for everyone. In general, I see no reason why you should not add a tag to every object you create. It can not sleep out of control because it already would (it is 100% up to the user only what he does with a tag :-)
The only way I can see a tag could be a problem is if you would decide to provide serialization/deserialization of the Plotables. Imagine binary writing Plotables to sqlite and populating a Plot by reading from sqlite, deserializing, and adding deserialized plotables to Plot. That would be nice for many use cases.
The console application shows the Plot and the end of the day. So it is not GUI less I think. However, Tag might not be used. But that is fine and causes no harm.
Currently, I am using SCottPlot the simple way, just to enable a simple display and analysis of the data shown.
Clicking Plot I perform actions that relate to the data ScottPlot holds and show GUI effects elsewhere. SCottPlot just serves as Event Trigger (MouseMove, MuseUp/Down) in this case.
When creating a more intensive GII with loads of business logic I will create a Class when one item of the class will be Plotable object. The wrapper will be a list of these Classes and the problem of Tag is solved for good. Using Dictionary has its own limits.
So far so good. Excellent work you have done and I like using SCottPlot. Just need more time to get used to the naming of things. Perhaps using ChatGPT4 from side might help :-)
There is one thing I would like to mention/suggest.
It would be good to have the option to define axis objects as separate GUI objects.
To be placed outside the plot object.
These objects will just reference Plot Axis objects while the Axis of originated Plot object could be left hidden.
Could also serve as a slider :-)


Removing unreleased feature

The option has still no effect
closes https://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness/issues/8898
Converted to draft to get a final confirmation

@SerdarSoysal PR has been copyedited and is ready for final review, could you please check and merge? Thanks!

@mkbond007 please review

@tiburd

Option has been finally removed from TAC, can you help me merge? Thanks


Is there a way to have "read" activities logged as well as "write" activities in the activity log (or elsewhere)?

Is this issue resolved or you still need help?
Please provide us with the link to the documentation you are referring to so we can better assist.

It is nt resolved. i opened a new issue with (i hope) the right references:


Breaking change in v3.3.2 with multiple "withComponent"

Reproduction
These codesandbox use the same code with 2 different versions of styled-components.
The result is different.
version 3.2.6 https://codesandbox.io/s/ll4lpv6mnz
version 3.3.2 https://codesandbox.io/s/038xx95oxp
Steps to reproduce
It occured when i use multiple withComponent

Looking over this, I think the current behavior is correct. If you look at the CSS being generated, the application order is:
Button > DefaultButton > A
And the corresponding component order as dictated by your example is:
Button  > DefaultButton > StyledLink > A
Since it is Button, which composes DefaultButton, with an underlying StyledLink, which composes A.
DefaultButton has overrides that cancel out many styles in A.
If you change StyledButton to be this instead:
StyledLink.withComponent(Button)
You arrive at something similar to the original behavior. But I do think what we're doing now is correct.


Actual code that Infinion flew with on  first ZeroPilot flight ever

A few files where #if 0'd, I was just having some trouble with getting them to build.


Allow subject info editing in new style

A lot of this logic can be removed when subject becomes its own container (this is also noted with inline comments). Old methods for modifying subject info are deprecated and will eventually no longer be supported.
Breaking Changes
None
Review Checklist

Tests were added to cover all code changes
Documentation was added / updated
Code and tests follow standards in CONTRIBUTING.md

Codecov Report

Merging #1039 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #1039      +/-   ##
==========================================
+ Coverage   90.69%   90.69%   +<.01%
==========================================
  Files          50       50
  Lines        6800     6804       +4
==========================================
+ Hits         6167     6171       +4
  Misses        633      633


Target Pixel File cut-out example fails for TESS data

I would like to extract Target Pixel Files from calibrated TESS FFI and tried running the How to cut out Target Pixel Files from Kepler Superstamps or TESS FFIs? example from the documentation but it fails. Using the exact syntax as provided in the example
from lightkurve import KeplerTargetPixelFile
from astropy.coordinates import SkyCoord
tpf = KeplerTargetPixelFile.from_fits_images(images=fnames,
                                             position=SkyCoord(257.13700, 24.48958, unit='deg'),
                                             size=(9,9),
                                             target_id='MyCutOut')

leads to the following warning:

.../site-packages/lightkurve/targetpixelfile.py:659: LightkurveWarning: A TESS data product is being opened using the KeplerTargetPixelFile class. Please use TessTargetPixelFile instead.

The warning is triggered as part of KeplerTargetPixelFile.from_fits_images.  Unfortunately, the class TessTargetPixelFile does not have a method called from_fits_images. Will a TESS version of this method be available any time soon?
Run under Ubuntu 16.04, Python 3.6.4 (anaconda), lightkurve version 1.0b23, installed with pip.

Thank you for reporting this issue @mommermi! You are quite right that the cut-out tool in the current version of Lightkurve hasn't fully been prepared for TESS yet.
I expect we will work on this over the coming month.  In meanwhile, you may be interested in trying MAST's new astrocut package which is dedicated to this task.  This new tool also has an online API called TESScut.


Feat(#536): Graphically display Component and Trait relationships and status

status
Signed-off-by: weiy730<EMAIL_ADDRESS>Description of your changes
Feat(https://github.com/kubevela/velaux/issues/536): Graphically display Component and Trait relationships and status
I have:

[x] Read and followed KubeVela's contribution process.
[x] Related Docs updated properly. In a new feature or configuration option, an update to the documentation is necessary.
[x] Run yarn lint to ensure this PR is ready for review.
[x] Added backport release-x.y labels to auto-backport this PR if necessary.

Special notes for your reviewer

@weiy730 Can you provide a screenshot of the new graph?

@weiy730 Can you provide a screenshot of the new graph?
![1665453458352](https://user-images.githubusercontent.com/105778708/194980431-7b2f8624-7713-4d43-b187-898413e813

da.jpg)

@weiy730 Can you provide a screenshot of the new graph?

@weiy730 Can you provide a screenshot of the new graph?

Is there any problem ？


Python multiple assignment highlighting

Does this issue occur when all extensions are disabled?: Yes

VS Code Version:
Version: 1.75.1
Commit: 441438abd1ac652551dbe4d408dfcec8a499b8bf
Date: 2023-02-08T21:35:30.018Z
Electron: 19.1.9
Chromium: 102.0.5005.194
Node.js: 16.14.2
V8: <IP_ADDRESS>-electron.0
OS: Linux x64 5.17.0-1027-oem snap
Sandboxed: No
OS Version: Ubuntu 22.04

Steps to Reproduce:

Write a multiple assignement in Python, like foo = bar = 9
See that the first variable (foo here) is not highlighted (remains white)

@amikrop can you share a screenshot? I'm not able to reproduce the issue.

@alexr00 sure, lines 1 and 6:

I'm still not able to reproduce the issue, it's likely caused by an extension.

This issue is caused by an extension, please file it with the repository (or contact) the extension has linked in its overview in VS Code or the marketplace for VS Code. See also our issue reporting guidelines. If you don't know which extension is causing the problem, you can run Help: Start extension bisect from the command palette (F1) to help identify the problem extension.
Happy Coding!


Fixes for plugin progress dialog cancel & LR lockup.

This change allows the progress dialog to be cancelled.
It also prevents the plugin from being run twice which locks up LR with a 'Task Completed' dialog box that can't be closed.

how does it prevent it from running twice? Does startAsyncTask use single thread pool or something?

It appears that doing the call to 'checkPhotoAvailability' outside the 'innerContext' was allowing you to trigger the plugin twice. I was only able to do this by mapping a keyboard shortcut to the plugin - sometimes the plugin would take a while to start and I would hit the shortcut again.
I noticed that the metadata dialog plugin didn't do this, and the main difference was the extra code being run before the 'innerContext'.

It appears that doing the call to 'checkPhotoAvailability' outside the 'innerContext' was allowing you to trigger the plugin twice.

gezzz......

turns out the dialog blocks all UI input which is what stops it from opening twice. Move the photo check to inside past the progress dialog opening is the trick needed.

thanks combined the code and merged in.


Move the client lib code to new repositories

Move the code that is still valid for tiki/apps/issues/71 to the client library repositories.

Most of the code will be rewritten. Not applicable


OAPL and CAPL keys conlfict with SW1 and SW0 on paddles

The OPAL and CAPL buffer is a 74LVC04 (back-to-back). It should be a 74LVC06 with open-drain outputs and then tied high.
The reason this matters is that the Gameport uses those two signals as buttons and they pull the signal to ground. So it creates a short situation. Open drain fixes it.

Updated title. Originally though an open-drain driver was needed on the CAPL and OAPL signals from the RP2040. This assumption was incorrect.
First, the Paddles have a 590-ohm pulldown resistor and connect the SWx signal to 5V.
The proposed solution is to go back to 74LVC04 drivers and add a series resistor to the SWx_xPL signals.

This simulation seems to agree this is workable

Instead of 470-ohm resistor, we tested with a diode. Works fine. Also need to add pulldown AFTER the diode!

add schottky and pull-down to 3b


problem INSTALLING  ionic(in cli)

HI , I FOLLOWED INSTRUCTIONS AT http://ionicframework.com/docs/guide/installation.html
BUT I GET THIS RESULTS ;CAN ANY ONE HELP
Microsoft Windows [Version 6.3.9600]
(c) 2013 Microsoft Corporation. All rights reserved.
C:\Users\Sena>npm install -g ionic
C:\Users\Sena\AppData\Roaming\npm\ionic -> C:\Users\Sena\AppData\Roaming\npm\nod
e_modules\ionic\bin\ionic
npm WARN unmet dependency C:\Users\Sena\AppData\Roaming\npm\node_modules\cordova
\node_modules\cordova-lib\node_modules\npm requires<EMAIL_ADDRESS>but will lo
ad
npm WARN unmet dependency C:\Users\Sena\AppData\Roaming\npm\node_modules\cordova
\node_modules\cordova-lib\node_modules\npmconf,
npm WARN unmet dependency which is version 0.1.16
npm WARN unmet dependency C:\Users\Sena\AppData\Roaming\npm\node_modules\cordova
\node_modules\cordova-lib\node_modules\npm requires<EMAIL_ADDRESS>but will l
oad
npm WARN unmet dependency C:\Users\Sena\AppData\Roaming\npm\node_modules\cordova
\node_modules\cordova-lib\node_modules\request,
npm WARN unmet dependency which is version 2.47.0
npm WARN unmet dependency C:\Users\Sena\AppData\Roaming\npm\node_modules\cordova
\node_modules\cordova-lib\node_modules\npm requires<EMAIL_ADDRESS>but will loa
d
npm WARN unmet dependency C:\Users\Sena\AppData\Roaming\npm\node_modules\cordova
\node_modules\cordova-lib\node_modules\semver,
npm WARN unmet dependency which is version 2.0.11
npm WARN unmet dependency C:\Users\Sena\AppData\Roaming\npm\node_modules\cordova
\node_modules\cordova-lib\node_modules\cordova-js\node_modules\browserify requir
es<EMAIL_ADDRESS>but will load
npm WARN unmet dependency C:\Users\Sena\AppData\Roaming\npm\node_modules\cordova
\node_modules\cordova-lib\node_modules\through2,
npm WARN unmet dependency which is version 0.6.3
<EMAIL_ADDRESS>C:\Users\Sena\AppData\Roaming\npm\node_modules\ionic
├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>paus
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>debug@
1.0.4)
├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>se
<EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>stackman@0
.2.2<EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>.3)
├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>reada
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>.2<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>n
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>http-signatur
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>wi
<EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>0<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>archy@0
.0.2<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>.7<EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>└──<EMAIL_ADDRESS>C:\Users\Sena>ionic start todo blank
'ionic' is not recognized as an internal or external command,
operable program or batch file.
C:\Users\Sena>

@edgbk Do you have cordova installed? npm install -g cordova

hmm did install cordova before but since u ask i'll try again(i appreciate your help)
hmm dont know what to make of this problem!
Microsoft Windows [Version 6.3.9600]
(c) 2013 Microsoft Corporation. All rights reserved.
C:\Users\Sena>npm install -g cordova
C:\Users\Sena\AppData\Roaming\npm\cordova -> C:\Users\Sena\AppData\Roaming\npm\n
ode_modules\cordova\bin\cordova
<EMAIL_ADDRESS>C:\Users\Sena\AppData\Roaming\npm\node_modules\cordova
├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS>└──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>properties-parser@0.
2.3<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>.4<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>npm@1
.4.28<EMAIL_ADDRESS>C:\Users\Sena>npm install -g ionic
C:\Users\Sena\AppData\Roaming\npm\ionic -> C:\Users\Sena\AppData\Roaming\npm\nod
e_modules\ionic\bin\ionic
<EMAIL_ADDRESS>C:\Users\Sena\AppData\Roaming\npm\node_modules\ionic
├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>map-
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>debug@
1.0.4)
├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>stackman@0
.2.2<EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>se
<EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>.3)
├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>reada
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>.2<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>n
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>http-signatur
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>wi
<EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>6<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>zip-
<EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>archy@0
.0.2<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>.7<EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>└──<EMAIL_ADDRESS>C:\Users\Sena>ionic start todo blank
'ionic' is not recognized as an internal or external command,
operable program or batch file.
C:\Users\Sena>install -g ionic
'install' is not recognized as an internal or external command,
operable program or batch file.
C:\Users\Sena>npm install -g ionic
C:\Users\Sena\AppData\Roaming\npm\ionic -> C:\Users\Sena\AppData\Roaming\npm\nod
e_modules\ionic\bin\ionic
<EMAIL_ADDRESS>C:\Users\Sena\AppData\Roaming\npm\node_modules\ionic
├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>paus
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>debug@
1.0.4)
├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>stackman@0
.2.2<EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>se
<EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>.3)
├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>.2<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>q
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>tough-cookie@
0.12.1<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>wi
<EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>0<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>archy@0
.0.2<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>.7<EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>└──<EMAIL_ADDRESS>C:\Users\Sena>ionic start todo blank
'ionic' is not recognized as an internal or external command,
operable program or batch file.
C:\Users\Sena>

Its looking like its not letting you hit the ionic command for some reason.
Try this solution

hmm just don't get it !
Microsoft Windows [Version 6.3.9600]
(c) 2013 Microsoft Corporation. All rights reserved.
C:\Users\Sena>npm cache clean
C:\Users\Sena>npm install -g cordova
C:\Users\Sena\AppData\Roaming\npm\cordova -> C:\Users\Sena\AppData\Roaming\npm\n
ode_modules\cordova\bin\cordova
<EMAIL_ADDRESS>C:\Users\Sena\AppData\Roaming\npm\node_modules\cordova
├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS>└──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>properties-parser@0.
2.3<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>through
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>plist@
1.1.0<EMAIL_ADDRESS>C:\Users\Sena>npm install -g ionic
C:\Users\Sena\AppData\Roaming\npm\ionic -> C:\Users\Sena\AppData\Roaming\npm\nod
e_modules\ionic\bin\ionic
<EMAIL_ADDRESS>C:\Users\Sena\AppData\Roaming\npm\node_modules\ionic
├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>paus
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>debug@
1.0.4)
├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>stackman@0
.2.2<EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>se
<EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>.3)
├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>wi
<EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>oauth-
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>q
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>combined-stream
@0.0.7<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>6<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>zip-
<EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>archy@0
.0.2<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>.7<EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS>└──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>C:\Users\Sena>ionic start todo blank
'ionic' is not recognized as an internal or external command,
operable program or batch file.
C:\Users\Sena>ionic start todo blank
'ionic' is not recognized as an internal or external command,
operable program or batch file.
C:\Users\Sena>

Yeah, this is getting me too I believe...
I've been using Cordova with Ionic for about 2 months now... I recently upgraded the Ionic CLI after being prompted and now I no longer have access to Ionic or Cordova (not saying the upgrade is the cause, just that it's the last thing that I recall doing to this system other than developing using it)...
I've since attempted to resolve my issue via a npm cache clear and a reinstall of Cordova and Ionic, but I am still having path issues..
NPM executes without issue or error, but Ionic nor Cordova appear to be in the path anymore (which, if I recall, both are added when installing via 'npm install -g')....
Fun times...

Update in case it helps someone else...
I am not sure what's caused this - I've also found a recent thread in the Ionic forums that seems to be related... Everything was fine a few days ago and then seemingly something broke... Windows Update?  Upgrading Node? NPM? Ionic?  No idea...
To resolve this, I had to edit my path as advised here: https://github.com/npm/npm/wiki/Troubleshooting#upgrading-on-windows
Basically, I had to add the Windows user path for NPM modules to my path before the installer inserted path to NodeJS... This raises the question (though not an Ionic question, but a NodeJS question) - Why does the NodeJS installer only add the '\program files (x86)' path when NPM clearly installs modules to the '...<user>\AppData\Roaming\npm...' path?

@mattezell - thanks for sharing that.
Did that take care of it for you?
@edgbk - you might want to try those suggestions Matt made.

Yeah, adding the '...\AppData\Roaming...' npm path to my PATH variable before the NodeJS-installer-added path for '...\Program Files (x86)\nodejs' resolved the '..is not a recognized..' issues I was encountered when trying to run Cordova or Ionic...
I am still scratching my head as to how this happened, but the best that I can figure is that something modified my path, though I have no idea what... I kid you not - I was developing on an Ionic project until about 1am this morning without issue.
Chain of events was that a corp update was pushed that required reboot - run of the mill Windows update, so far as I know...  After reboot, a cmd window was auto-opened upon first login by Windows - I used this cmd window for development all day without issue (lots of "ionic run android -l -c -s" being executed without a problem)...  In the new cmd window, 'ionic' results in the '..is not recognized..' issue first hitting my radar... I then close both cmd windows to go ahead and bite the bullet to research this issue - at which time I realize that all newly opened command windows no longer have access to the Ionic path...
The more that I think about it, I am guessing (just a guess) that whatever mechanism re-opens open applications upon first login after a update-forced reboot perhaps also provides whatever PATH was passed in when it was initially opened, which perhaps somehow contained the \AppData\ path which allowed it to resolve Ionic... Then manually opened CMD windows don't have this PATH passed to them as the actual system PATH had somehow been modified by the upgrade or some other mechanism...
What I don't have any clue about is, if this guess is correct, why didn't just reinstalling NodeJS/NPM via the Windows MSI resolve the issue?  Has something changed with the installer in regards to the paths that it adds to the PATH?  I suppose it's possible that I added this \AppData\ user npm path to my PATH when I first installed this stack 3 months ago, but I definitely do not recall doing such a thing.. Additionally to this point, everything that I read in discussions on this say that the Windows installer should properly add the NPM modules path - but then that begs the question of why the information in the link I shared exists to begin with if this isn't a common problem...

hurrayyyy!!!! (:
seems to be be responding at last.{thanks guys for all the help}
Microsoft Windows [Version 6.3.9600]
(c) 2013 Microsoft Corporation. All rights reserved.
C:\Users\Sena>npm cache clean
C:\Users\Sena>npm install -g cordova
C:\Users\Sena\AppData\Roaming\npm\cordova -> C:\Users\Sena\AppData\Roaming\npm\n
ode_modules\cordova\bin\cordova
<EMAIL_ADDRESS>C:\Users\Sena\AppData\Roaming\npm\node_modules\cordova
├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS>└──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>properties-parser@0.
2.3<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>thr
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>cordo
<EMAIL_ADDRESS><EMAIL_ADDRESS>C:\Users\Sena>npm install -g ionic
C:\Users\Sena\AppData\Roaming\npm\ionic -> C:\Users\Sena\AppData\Roaming\npm\nod
e_modules\ionic\bin\ionic
<EMAIL_ADDRESS>C:\Users\Sena\AppData\Roaming\npm\node_modules\ionic
├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>paus
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>finalhandler@
0.1.0)
├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>stackman@0
.2.2<EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>se
<EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>.3)
├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>m
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>.2<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>n
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>tough-cookie@
0.12.1<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>wi
<EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>6<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>lazy
<EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>pretty-hrtime@0
.2.2<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>.7<EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>└──<EMAIL_ADDRESS>C:\Users\Sena>ionic start todo blank
Creating Ionic app in folder C:\Users\Sena\todo based on blank project
Downloading: https://github.com/driftyco/ionic-app-base/archive/master.zip
[=============================]  100%  0.0s
Downloading: https://github.com/driftyco/ionic-starter-blank/archive/master.zip
[=============================]  100%  0.0s
Update config.xml
Initializing cordova project
Fetching plugin "org.apache.cordova.device" via plugin registry
Fetching plugin "org.apache.cordova.console" via plugin registry
Fetching plugin "com.ionic.keyboard" via plugin registry
Your Ionic project is ready to go! Some quick tips:

cd into your project: $ cd todo

Setup this project to use Sass: ionic setup sass

Develop in the browser with live reload: ionic serve

Add a platform (ios or Android): ionic platform add ios [android]
Note: iOS development requires OS X currently
See the Android Platform Guide for full Android installation instructions:
https://cordova.apache.org/docs/en/edge/guide_platforms_android_index.md.html

Build your app: ionic build

Simulate your app: ionic emulate

Run your app on a device: ionic run

Package an app using Ionic package service: ionic package

For more help use ionic --help or visit the Ionic docs: http://ionicframework.co
m/docs
C:\Users\Sena>

I had the same situation:
I've used ionic without problems for couple of days, than made some changes on my PATH variable due the reinstall of Android sdk. I may deleted or changed some chars there, don't know :)
Anyway, here is my PATH now and Ionic is working:
other vars;C:\Program Files (x86)\Git\cmd;C:\Program Files (x86)\Git\bin;C:\Program Files (x86)\Android\android-sdk\tools;C:\Program Files (x86)\Android\android-sdk\platform-tools;C:\Users\Codrin\AppData\Roaming\npm;C:\Program Files\nodejs
Basically I added C:\Users\Codrin\AppData\Roaming\npm BEFORE C:\Program Files\nodejs


Scripts might be broken

After the change referenced in #7, the datasets are now located in a different repo.
This might break the cron jobs located in the scripts folder and maybe some minor changes might be necessary.

Yep looks like it'll need to pull down the submodule and update the paths to the data files.

Oh ok. I guess it just needs some minor changes to commit from the right repo.

Created a PR to address it ^


added fcm style dataset for c4

@lintangsutawika Thanks for pointing that out. Apparently I made a mistake with my gin config and forgot to add my tasks.py, where I registered the task of causal LM on C4. Basically, instead of the usual
{
        "inputs":
            seqio.Feature(
                vocabulary=t5x.data.vocab.get_default_vocabulary(mode),
                add_eos=True,
                required=False),
        "targets":
            seqio.Feature(
                vocabulary=t5x.data.vocab.get_default_vocabulary(mode),
                add_eos=True)
}

I used the following. So, only thing you have to specify is TASK_FEATURE_LENGTHS={"inputs": 512, "targets": 114}. But if you want to do MLM-LM etc, it'd be something different. I'll fix it asap (maybe not today). You can work on other components meanwhile :)
{
        "targets":
            seqio.Feature(
                vocabulary=t5x.data.vocab.get_default_vocabulary(mode),
                add_eos=True,
                required=False),
}

OK I finally fixed the scalable decoder and added Pile. I probably shouldn't have synced right away, but hopefully this is OK. I changed the style of c4/tasks.py in a way such that objective (e.g. CLM vs. MLM) and dataset (e.g. Pile vs. C4) are separated. This will allow various different combinations without having to write the registration for each of them. Sorry if this was confusing. I think you can implement the FCM-style training as default_fcm_task in t5x/utils.py and add the following at the end of c4/tasks.py:
default_clm_task('c4_eye_fcm', c4_files, jsonl=False)
Or maybe we should have a separate file data/objectives.py that store all the objectives like default_clm_task.

Thanks, I will take a look soon.
Remind me again, we want to train a scalable decoder with FCM style as a benchmark, right?

Yeah scalable decoder with FCM is going to be used as one of the baselines. By the way, main branch causes an error for larger models after a number of gradient updates for an unknown reason, so I'm trying to figure out what's going on.

Can you share the error in a separate issue?
Also, I don't know if it's related but I had trouble once that seemed to be due to network connections between nodes.

Did some sanity check. The features are correct.
In [7]: ex['decoder_input_tokens'][:9]
Out[7]:
array([    0,  8815,  3051,     5,   148,   131, 32099, 32099,   987],
      dtype=int32)

In [8]: ex['decoder_target_tokens'][:9]
Out[8]: array([8815, 3051,    5,  148,  131,   43,   12,  987,    5], dtype=int32)


Make reset and bootloader sequences configurable

The bcf tool has built-in support for the reset and bootloader sequences implemented by the Hardwario Tower Core Module. These sequences use the DTR and RTS signals on the on-board FTDI chip to reboot the MCU or to switch it into the bootloader mode.
I use a similar approach on the LoRa Module for firmware development purposes. My reset and bootloader sequences are different since the corresponding pins on the TypeABZ module are directly connected to DTR and RTS signals (unlike in Core Module): https://github.com/hardwario/lora-modem/wiki/LoRa-Module-Firmware-Update
This PR makes it possible to use the bcf tool to flash firmware into the Tower LoRa Module with an external FTDI converter and only minimal HW modification in the module (boot and reset pins on the debug connector directly connected to DTR and RTS). It adds support for per-module reset and bootloader sequences selectable on the command line with the new parameter -m. By default (if omitted), the parameter is set to "Core". With -m LoRa, different reset and bootloader sequences compatible with the LoRa Module will be used.

Nice work,
but if I can move the module argument as optional with the default value set, this implementation will break some of our scripts using bcf as a library.

I am dropping this PR because it is no longer needed. With a slightly modified electrical connection, one can use an unmodified bcf version to flash firmware into the LoRa Module.


.NET Standard / Core support?

I'm having trouble finding any statements on whether you support, or plan to support, .NET Standard / Core moving forward.
I'm getting a compatibility warning in my project though, so I'm guessing Z.ExtensionMethods currently is only for Full Framework, is that right?

How is this project going? :)

Keen to see support for .NET Standard 2 :-)

Hello @kinetiq , @Ricky-G ,
My developer made the change a few weeks ago but unfortunately, I got some health issue, so a lot of free requests has been paused like this one.
I will try to review his code next weekend and make a release from it.
Best Regards,
Jonathan

Thanks - really looking forward to this!

Since 4-5 months have passed, I was wondering if this is going to happen or if we should just give up on z for future versions of .net?

Hey, that's great! Thanks for the heads up!


Manage the identities for cached selector policies via identitymanager

Previous incremental PRs have required the policy distillery to reference-count the users of an identity policy, and tie the adds/removes of identities for which we cache selector policy to the endpoint (by, for instance, inserting when the selectorpolicy is found to be nil inside the endpoint, and removing when the endpoint leaves). This PR replaces that with instead triggering the caching of selector policy based upon the identitymanager lifecycle. This is a bit cleaner, requiring less management, and reducing interdependency between the distillery and endpoint packages.

This change is

test-me-please

test-me-please

Coverage increased (+0.04%) to 41.619% when pulling caf2fb568845da4e04cd2bcb09c8d9771e7c043f on joestringer:submit/distillery-lifecycle into 21f462c5ebf3c37be401a5d296fe6cc40c96d5be on cilium:master.

test-me-please

The only agent-related changes are unit-testing changes, merging.


enhance modularity of variantinterpretation workflow

These changes improve the usability of the pipeline as a sub-pipeline by:

factoring out VARIANTINTERPRETATION_CORE from VARIANTINTERPRETATION
moving the pipelines scripts to their modules resources/usr/bin

This is because the multiqc configuration causes issues with a calling pipeline, e.g. wrong path to assets/.
Also, a calling pipeline might want to run multiqc in their own fashion.
The scripts under bin are visible to the whole pipeline, but not usable when the pipeline is called as a sub-pipeline/module.
The moduleBinaries under their modules resources/usr/bin are better/narrower scoped and usable to the calling pipeline.
PR checklist

[x] This comment contains a description of changes (with reason).
[ ] If you've fixed a bug or added code that should be tested, add tests!
[ ] If you've added a new tool - have you followed the pipeline conventions in the contribution docs
[ ] Make sure your code lints (nf-core lint).
[ ] Ensure the test suite passes (nextflow run . -profile test,docker --outdir <OUTDIR>).
[ ] Usage Documentation in docs/usage.md is updated.
[ ] Output Documentation in docs/output.md is updated.
[x] CHANGELOG.md is updated.
[ ] README.md is updated (including new tool citations and authors/contributors).

I fixed some general errors in calling the new subworkflow.

Suggestion for clearer structure and even more modularization:
Now, VARIANTINTERPRETATION_CORE is a subworkflow within the VARIANTINTERPRETATION workflow, which leads to this long nested structure of the main workflow call: CIOABCD_VARIANTINTERPRETATION:VARIANTINTERPRETATION:VARIANTINTERPRETATION_CORE
Instead, we could outsource the multiQC part in VARIANTINTERPRETATION as its own subworkflow named, e.g., VARIANTINTERPRETATION_MULTIQC and call it in main.nf
So the main.nf would first call the VARIANTINTERPRETATION pipeline (the current VARIANTINTERPRETATION_CORE) and then the VARIANTINTERPRETATION_MULTIQC subworkflow.
So the process would look like this:
CIOABCD_VARIANTINTERPRETATION:VARIANTINTERPRETATION_CORE:
CIOABCD_VARIANTINTERPRETATION:VARIANTINTERPRETATION_MULTIQC:

This would still preserve your changes, as the multiQC parts are not within the VARIANTINTERPRETATION_CORE workflow stays the same.

When running the pipeline it does not find the python scripts in the resources directories, I am not sure how to fix this:

ERROR ~ Error executing process > 'CIOABCD_VARIANTINTERPRETATION:VARIANTINTERPRETATION:VARIANTINTERPRETATION_CORE:CHECKVCF:VCFCHECKS (HD789-mutect2)'

Caused by:
  Process `CIOABCD_VARIANTINTERPRETATION:VARIANTINTERPRETATION:VARIANTINTERPRETATION_CORE:CHECKVCF:VCFCHECKS (HD789-mutect2)` terminated with an error exit status (127)

Command executed:

  check_vcf.py \
      --meta_id HD789-mutect2 \
      --vcf_in HD789-DNA.mutect2.head.vcf.gz \
      --bcftools_stats_in HD789-mutect2.bcftools_stats.txt \
      --warnings_out HD789-mutect2_warnings.txt \
      --check_chr_prefix --check_vep_annotation --check_MNPs --check_gVCF --check_other_variants --check_multiallelic_sites --check_FILTERs

Command exit status:
  127

Command output:
  (empty)

Command error:
  INFO:    Converting SIF file to temporary sandbox...
  .command.sh: line 3: check_vcf.py: command not found
  INFO:    Cleaning up image...

I wanted to propose the simpler code change first (nested), but sure, i too thought it might belong into CIOABCD_VARIANTINTERPRETATION.
Sry about the errors, i had to move/redo some things and must've lost/missed them.
The python scripts arent found since i've moved them away from $projectDir/bin and i recognized late that scripts in $moduleDir/resources/bin aren't atm. available to the top workflow/module.
https://github.com/nextflow-io/nextflow/issues/5288#issue-2507585985
One mediocre solution is to have them all duplicated in both $projectDir/bin and $moduleDir/resources.


Duplicated code and got functional build for .net standard

Sorry, I'm unfamiliar with github and did not mean to make that request.


platform dashboard

Issue:

bcgov/entity/issues/

Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the namex license (Apache 2.0).

Temporary Url for review: https://strr-platform-dev--pr-225-bpj0znwh.web.app

Looks pretty good, I would just double check the typing and default method on the $strrApi calls

Temporary Url for review: https://strr-platform-dev--pr-225-bpj0znwh.web.app


[Event Request] Codeunit 8611 "Config. Package Management" - InsertPackageRecord

Dear ALAppExtensions team,
we currently have issues with importing Rapid Start Package. Therefore we would like to request a couple of events to be added to the Codeunit 8611 "Config. Package Management". Requested changes are shown in the attachment to this post and marked using tags as such:
// S/ Event Request
OnBefore...
// E/ Event Request
Requested Events:

OnInsertRecordOnBeforeInsertRecRef(RecRef, ConfigPackageRecord, IsHandled)
OnModifyRecordDataFieldsOnBeforeRecRefInsert(RecRef, ConfigPackageTable, IsHandled);
OnModifyRecordDataFieldsOnBeforeRecRefModify(RecRef, ConfigPackageTable, RecordsModifiedCount, IsHandled);

ConfigPackageManagement.txt
Thanks.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Add an install script

ERROR: Package 'llm-vm' requires a different Python: 3.9.6 not in '>=3.10'

I was trying to install Anarchy but faced some issues with the Python version.
I tried conda create -n myenv python=3.9.6 but it did not work.

Make project requirements clearer
Add an install script.

So I did conda create -n myenv python=3.10 but it did not work. Everytime I checked the version using python3 --version it kept showing Python 3.9.6
since I was using VS code, I found that if I changed the interpreter path and restarted VS code it finally worked.

Are these the same steps you followed?
(base ) $ conda create -n myenv python=3.10
(base) $ conda activate myenv
(myenv) $ python --version

Note how base turns into myenv upon activating the environment
`

Hello!
I would like to work on this

You are free to work on it @Lioncat2002, but we can't assign it to you until you are a full contributor as per #contributing-guide in discord.

Ahh cool @mmirman
So I was thinking of possible solution for this, and a simple one would be to make a bash script ig?
The users can directly run the script to get the repository work.

What's the goal of this issue?

@VictorOdede Is this issue solved?

<EMAIL_ADDRESS>


Bug Fix: Run<EMAIL_ADDRESS>- Failed

Related Issues or Pull Requests
Bug Fix: Run<EMAIL_ADDRESS>Brief explanation of the Problem statement
Added up the auto_assign.yml file to assign roles automatically for reviewers and assignees. This file will resolve the check error in the Pull Request.
Solution to the problem statement
By adding the auto_assign.yml file in the repository
Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.

File(s) Added / Modified

.github/auto_assign.yml

How to test:
You test it after merging the pull request.

Screenshots:
Tested on KamalDGRT/event-aggregator-demo

Updated AkashM398 bro as a reviewer

shipped it


pyautogui.write('@') doesn't work

When you write pyautogui.write('@') it doesn't work, it doesn't give an error or anything but it just doesn't type @, like it's supposed to. And when you write pyautogui.write('test@') for example it just writes 'test' without the @.

in my case it's writing double quotes, I think it's related to the keyboard encoding being used.

in my case it's writing double quotes, I think it's related to the keyboard encoding being used.

in my case it's writing double quotes, I think it's related to the keyboard encoding being used.

I've just tried it with the single quotes and it doesn't work either.

in my case it's writing double quotes, I think it's related to the keyboard encoding being used.

I've just tried it with the single quotes and it doesn't work either.

I'm revisiting this and trying to come up with a solution. The problem is that that shows up depends on keyboard configuration set by the operating system. For example, press('@') will "press" the shift key and the press the 2 key. But on Russian keyboards, this produces the " double quote character due to the layout of Russia keyboards.
So I need to go through all the different keyboard configurations and fix this.


Enable/disable solder paste per footprint

KiVar shall be able to switch solder paste application per footprint.
For this, the FOOTPRINT.Get/SetLocalSolderPasteMarginRatio() shall be used.
As this shall be mapped to a bool property, the original value must be saved. Therefore, a negative offset value shall be used.
As the resulting clearance is the sum of the PasteMarginRatio and the PasteMargin (which can be a fairly high positive value), there shall be a safety margin to make sure the effective solder paste amount (per pad) is really reduced to zero. Also, there can be an additional boundary check for reasonable PasteMargin values.
Using the Offset Value:
KiVar shall classify a footprint as with paste if the ratio value is in the (usual) range of -10.0 to 10.0 (i.e. -1000% to 1000%) or without paste if the ratio values is in the range -10010.0 to -9990.0 (i.e. -1001000% to -999000%). If the ratio value cannot be classified, an error shall be thrown.
Enabling/disabling paste is then done by adding/subtracting the 10000.0 (i.e. 1000000%) offset.
Idea: Maybe the offset can use some "signature", such as<PHONE_NUMBER> or just 866582, representing the concatenation of the decimal ASCII codes for KIVAR or VAR, respectively.
The bool property identifier to be used shall be s (for "solder paste enabled", as p is already claimed by "in position file").
There shall be a note in the docs that this property will not be applied to "SMD Aperture" type pads.
This issue is related to #52, which was about "completely" removing footprints, although it seems that the real use-case was just enabling/disabling solder paste per footprint.

RFC:
Any suggestions that are more elegant than changing the PasteMarginRatio as described above?

Implemented on main branch: 80adb3c640b8ccee56e6973eacbc188e4c76a0d3


Add the option to have persistent switches

Add new GPIO switch class - PersistentRPiGPIOSwitch which will restore the switch state after Home Assistant restart.
As part of the change move the core switch to use async functions
This is based on my work in the the original branch: https://github.com/thecode/ha-rpi_gpio/pull/266

Didnt get any feedback so will try to submit it here: https://github.com/jdeneef/ha_gpiod/pull/15/


members: add change notification service component

related to https://github.com/inveniosoftware/invenio-communities/issues/557

closing, it was decided not to do it through change notificiations


fix(taskframework): job is not started  in resource idle node when deployed multi-nodes

What type of PR is this?

type-bug
What this PR does / why we need it:
job is not started  when multi nodes has enough resource
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

Additional documentation e.g., usage docs, etc.:

I can not accept your solution:

StartPreparingJob is a daemon job on every node, we'd better not disable it on a specific node
we have already got rateLimiter MonitorProcessRateLimiter, why does it work?
Will other nodes be affected if you releaseAcquiredTrigger in ResourceDetectJobStore?

I can not accept your solution:

StartPreparingJob is a daemon job on every node, we'd better not disable it on a specific node
we have already got rateLimiter MonitorProcessRateLimiter, why does it work?
Will other nodes be affected if you releaseAcquiredTrigger in ResourceDetectJobStore?

in cluster model,  all quartz node seize trigger, and be fired exact only once,  if one node  with no resource acquire trigger and fired it, the job will not be started in StartPreparingJob in QuartzSchedulerThread while loop  and  other node with enough resource will not acquire trigger

I can not accept your solution:

StartPreparingJob is a daemon job on every node, we'd better not disable it on a specific node
we have already got rateLimiter MonitorProcessRateLimiter, why does it work?
Will other nodes be affected if you releaseAcquiredTrigger in ResourceDetectJobStore?

in cluster model, all quartz node seize trigger, and be fired exact only once, if one node with no resource acquire trigger and fired it, the job will not be started in StartPreparingJob in QuartzSchedulerThread while loop and other node with enough resource will not acquire trigger

as we discused before, in cluster model, each node should do StartPreparingJob

I can not accept your solution:

StartPreparingJob is a daemon job on every node, we'd better not disable it on a specific node
we have already got rateLimiter MonitorProcessRateLimiter, why does it work?
Will other nodes be affected if you releaseAcquiredTrigger in ResourceDetectJobStore?

in cluster model, all quartz node seize trigger, and be fired exact only once, if one node with no resource acquire trigger and fired it, the job will not be started in StartPreparingJob in QuartzSchedulerThread while loop and other node with enough resource will not acquire trigger

I can not accept your solution:

StartPreparingJob is a daemon job on every node, we'd better not disable it on a specific node
we have already got rateLimiter MonitorProcessRateLimiter, why does it work?
Will other nodes be affected if you releaseAcquiredTrigger in ResourceDetectJobStore?

in cluster model, all quartz node seize trigger, and be fired exact only once, if one node with no resource acquire trigger and fired it, the job will not be started in StartPreparingJob in QuartzSchedulerThread while loop and other node with enough resource will not acquire trigger

so, why doesn't we rely on MonitorProcessRateLimiter but disable the StartPreparingJob?


When is cpu kv cache supposed to be used?

In my testing with many concurrent users, I see what appears to be changing of requests from Running to Pending. I've never seen Running change to Swapped, nor have I seen cpu kv cache at anything other than 0%.
INFO 02-28 21:48:18 metrics.py:161] Avg prompt throughput: 0.0 tokens/s, Avg generation throughput: 44.9 tokens/s, Running: 2 reqs, Swapped: 0 reqs, Pending: 3 reqs, GPU KV cache usage: 82.1%, CPU KV cache usage: 0.0%

I have tried setting swap_space to a value equal to vram and that does not make a difference.

mark

Closing as duplicate: https://github.com/vllm-project/vllm/issues/2853


WebApi child object Modelstate not evaluated

I'm trying to set up FV as metioned here
The validation appears to work ok for the top level objects but everything underneath never appears to be validated.
I know I could specify the SetValidator etc methods and new up the validator within but that seems to be against the whole point of using IOC.
From debugging I can see that WebApi never appears to ask the factory for a validator of the child types so it never botheres to validate them.
I've reproduced it in a project here
See the ValuesController post.
Sending
{
	Test: "a",
	Bar: { Pepsi : "j"}
}

Only triggers an error on "Test2" being null when it should also validate the Bar.Pepsi < 3
Any ideas?

I'm pretty sure you'll need to use SetValidator for this (which is the recommended way of doing child-validation with FV)


Crash while joining to the map

Hello dear devs.
I was changed ws protocol to win tcp, and ive got crash while joining to the map, i think this hack is outdated. Do you plan to update it for current versions of csgo or not?

@certeas can you help me about injection

@certeas can you help me about ws protocol to win tcp ?

@certeas you would need to update CInput structure offsets (in_thirdperson, cam_angles, cmd etc)
input pointer is stored in interface pointers array (0x229C5C90) with offset 0x2C (index 11). GetUsercmd as well as GetVerifiedCmd are not inlined and really easy to update, the correct fix is provided in the latest commit.
Gl on updating other stuff, feel free to contribute!
@nstify changed ws to wintcp in connection routine, you need achieve request handling yourself. All required functions and libraries are already presented and linked

@violanes thank you sir.


What is the hyper-parameters for SOP+ in table.2?

Hi, I noted that in table.2, the backbone for SOP+ is Res34.
However, in the table A.4 (table of hyper-parameters), it is shown that all Res34 have lambda_C = lambda_B = 0. It really confused me since SOP+ with  lambda_C = lambda_B = 0 is simply SOP.
Am I make something wrong? Or the SOP+ in table.2 is in fact simply SOP? Or there are some others hyper-parameters differing from table A.4?

What's more, is SOP+ consistently better than SOP in your experiments or SOP+ may perform worse sometimes?
It would be appreciate if you could share some experience on it.

Nice catch! It was a mistake and for the architecture for Table 2 we use preactresnet18. We will update the paper. Thank you and sorry for the confusion. I believe when the dataset is highly imbalanced adding the regularization could be worse but we didn't run any experiments to confirm this.


[hot-fix] Fetching SCFile

Without this fix, it always returns self.data, which is nil by default (unless storeDataAfterFetch was set to YES)

Already pushed to master to fix a support issue, but still check please.

LGTM :) Thanks


[feature]: allow configuration of the kubernetes client via operator settings

Is your feature request related to a problem? Please describe.
No response
Describe the solution you would like
the operator settings should support some way of configuring the kubernetes client.
Additional Context
No response

/related #544


Change the way Property Types are named, fixing bugs in the process

This is the result of a few scenery_generator bugfixes, and likely the cause of a few more bugs :)
Relevant scenery issues that were logged in scenery_generator:

https://github.com/OpenWhere/scenery_generator/issues/25
https://github.com/OpenWhere/scenery_generator/issues/23

One more issue that was not logged but is addressed by this request was that the Ec2NetworkIface didn't have the Boolean properties AssociatePublicIpAddress and DeleteOnTermination

i think we should merge and start using the new auto gen classes


Not able to change bibliography style

I am using the template provided on Overleaf (v4.3). I have tried various ways based on the documentation to change the bibliography style to plain, but it doesn't work. Either no bibliography is printed, or it is printed in default style. I am compiling the project on Overleaf and it is set to use XeLaTeX. I have tried "recompile from scratch", which should delete everything rerun the whole process.
I am also using bibend=bibtex, but not sure whether it is actually being used. I have even tried putting the style file plain.bst in the project folder.

I solved the problem by commenting all bibliography related lines in the .cls file and then using the normal way of specifying a bibliography file and the bibliography style:

\bibliographystyle{plain}
\bibliography{reference}

Now I can print bibliography in the plain or any other style. I can also change the bibliography file if required.


feat: modified docker helper for ACR for jxboot-helmfile-resources

Hi @chrismellard. Thanks for your PR.
I'm waiting for a jenkins-x member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign garethjevans
You can assign the PR to them by writing /assign @garethjevans in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

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

Yes! I was adding an imagePullSecret to work around this problem, I'll try right now without it :)

@hervelemeur for now you'll also need to specify this Kaniko image in your pipelines gcr.io/jenkinsxio/kaniko:0.0.4 until I can get the helper merged in to kaniko project


Some natural/* and man_made/* translations

Translation for some useful keys for natural/* and man_made/* nodes.

Please submit these translation corrections via Transifex, our localizations management tool. See this guide for instructions. Thank you!

Also, I noticed that you included some translations for things that aren't actually presets in iD yet (adit, monitoring_station, volcano).  Adding them as translations won't make them available as presets - but they would be good additions.
You can open up a new GitHub issue to have presets added, or you can try to add them yourself.  If you do want to try this, take a look at the presets README which explains where all the preset files are stored - it's as simple as just adding the new files and running make.

Thanks, I will try to do both.
Regards


json loads string dump instead of object

Lines 746 and 747 use json.dumps(), which results in the error "TypeError: string indices must be integers". I think these should use json.loads() instead.

Fixed in 1.5.1


can't write to docker volume / persistence of secrets.yaml

A simple docker run onionpipe:latest <IP_ADDRESS>:80~80@test fails with ``mkdir /.local: permission deniedbecause onionpipe will try to save the secrets.yaml file to/.local/share/onionpipe/secrets.yaml: https://github.com/cmars/onionpipe/blob/804fcbbcc08e62b6d512c26f6dfebdb435ac8fac/app/app.go#L45 This is because $HOMEis/` for the non-existing user 1000 inside docker.
I think the best fix here is to create a user inside the dockerfile, perhaps with home directory set as /data.
Similarly the nextcloud example fails because command: --secrets /var/lib/onionpipe/secrets.json app:80~80@nextcloud https://github.com/cmars/onionpipe/blob/804fcbbcc08e62b6d512c26f6dfebdb435ac8fac/examples/nextcloud/docker-compose.yml#L12 tries to access var/lib/onionpipe but it does not have rights to do so.
I am not sure how to best fix this. Volumes are only writable by root. Perhaps the volume should be under /data/.
There is also an issue of persistence here. Running docker run onionpipe:latest --secrets /data/secrets.json <IP_ADDRESS>:80~80@test works (because user 1000 is allowed to write to /data/), but any subsequent run will use a new anonymous volume.
I think the best way here is to declare /data as a VOLUME in the dockerfile.
Any other ideas? I will create a pull request.

VOLUME sounds good! I wonder if Docker buildkit features might support even more secure storage options for secrets but being able to run things seems like a good start.
Looking at this now, I wonder if there's a proper XDG package I could have used for the file location... might not have avoided this particular issue but could allow more flexibility in other contexts.


postgresql  the table name has blank space，run starrocks-migrate-tool migrate error

the table name has blank space ,eg: " itlh1", when  run the migrate tools
$ ./starrocks-migrate-tool
2023/08/17 17:05:04 source/pgsql.go:124 ERROR: relation "public.itlh1" does not exist (SQLSTATE 42P01)
[131.892ms] [rows:0] SELECT "table_name","table_schema","table_catalog",pg_table_size(table_schema || '.' || table_name) as data_length,pg_indexes_size(table_schema || '.' || table_name) as index_length FROM "information_schema"."tables" WHERE table_type='BASE TABLE' and table_schema not in ('information_schema', 'pg_catalog') ORDER BY TABLE_SCHEMA asc, TABLE_NAME asc
panic: ERROR: relation "public.itlh1" does not exist (SQLSTATE 42P01)
goroutine 1 [running]:
main.main()
main.go:39 +0x959

select
table_name,
table_schema,
table_catalog,
pg_table_size(table_name1) AS  data_length,
pg_indexes_size(table_name1) as index_length
from
(
select
table_name,('"' || table_schema || '"."' || table_name || '"') as table_name1,table_schema,table_type,table_catalog
from
information_schema.tables
where
table_type = 'BASE TABLE'
and table_schema not in ('information_schema', 'pg_catalog')
) t
order by
TABLE_SCHEMA asc,
TABLE_NAME asc

@huangfeng1993 Could you have a look?


Implement CSSNumericValue.parse()

Add proper tokenization and parsing for CSSNumericValue.parse().

I think we might have come at a point where it might be a good idea to split off the CSSOM polyfill into its own repo with its own tests? I mean, feels like this could become its own library.

I think we might have come at a point where it might be a good idea to split off the CSSOM polyfill into its own repo with its own tests? I mean, feels like this could become its own library.

It still limited to implementing the parts needed for this library.. But I’ve been thinking we should start by moving the implementation into its own folder to make it more explicit that the code is meant for polyfilling parts of the typed cssom.
As it is right now it could be tempting to use internal functions directly without going through the public api when implementing scroll driven animations. Then we would end up with too tight a coupling.
I can create a refactoring PR once the open PRs are merged.

I can create a refactoring PR once the open PRs are merged.

Yeah, sounds like a good plan.
Which PRs specifically – other than this one – definitely need to be merged?

Which PRs specifically – other than this one – definitely need to be merged?

I was wrong. I looked through the other PRs, and it looks like none of them are touching the cssom implementation, so we should be fine after this PR is in. We could do it as part of this PR as well, if that's preferable.

I think it’d be best to do this refactor as part of a separate PR.

@bramus Thank you for the review 🙏
I think I have addressed all your remarks, and have rebased onto master.

Thanks!
Pinging @flackr to take a look before merging. I think this one should go in before #221.

Wow, this is a lot! I'm a bit concerned about the size increase. We'll have to think about what parts of this could be removed for a reduced build. Also, do we skip this parsing code if the browser supports CSSStyleValue.parse?

To be clear, I'm ecstatic to see such a precise implementation following the spec steps! I just think the extra kilobytes will be too much for some sites, and we'll want to have a light version of the polyfill where you have to construct the CSSOM value instead of parsing it from a string.

Wow, this is a lot! I'm a bit concerned about the size increase. We'll have to think about what parts of this could be removed for a reduced build. Also, do we skip this parsing code if the browser supports CSSStyleValue.parse?

To be clear, I'm ecstatic to see such a precise implementation following the spec steps! I just think the extra kilobytes will be too much for some sites, and we'll want to have a light version of the polyfill where you have to construct the CSSOM value instead of parsing it from a string.

Yeah, I know it's a lot 😅, and by all means we don't have to use it. It's been an interesting exercise :)
The parsing code should only be executed in browsers without CSSStyleValue.parse() support (Looking at you Firefox), but it does add 2.1kB to the compressed build.
If we had a build system that could build two targets, we could build one with a full implementation of CSSStyleValue.parse() and a light build with with a simple or no implementation of CSSStyleValue.parse(). That simple implementation could be limited to supporting CSSUnitValues.
The current solution though (that I'm at fault for), supports some math functions but can fail in more complex calculations. And there's no clear definition of what it will support or not. It also fails in browsers without regex lookahead support (Safari < 16.4).


[Misc]Add cacheblend hyper-parameter configurations

Add cacheblend hyper-parameter configurations on recompute ratio(blend_recompute_ratio) and minimum token number to trigger cacheblend(blend_min_tokens).
This PR resolves #190

Closing because changes included in this PR


Wrong completion suggestions in annotation

Not any of those suggestions are relevant. And the right suggestions are not in the list of suggestions.
This makes libraries that heavily use annotations much harder to explore, than other dart libraries.

I saw this test in the analyzer_server source, and this test is passing.
  test_Annotation_local_constructor_named_param() {
    addTestSource('''
class A { A({int one, String two: 'defaultValue'}) { } }
@A(^) main() { }''');
    computeFast();
    return computeFull((bool result) {
      assertSuggestArguments(namedArguments: ['one', 'two']);
    });
  }

So that seems like there is already logic for this in place?

haha I actually opened this same issue two months ago :P
https://github.com/dart-lang/sdk/issues/24296
and was then fixed by @danrubel
but it doesn't seem to work for some reason in the editor
not in dartpad, atom or webstorm

maybe interesting, I get different results if I put the annotation class in the same file, that could maybe explain why that test pass

this test would also fail:
  test_Annotation_imported_constructor_named_param() {
    addSource('/libA.dart', 'library libA; class A{A({int one}){}}');
    addTestSource('import "/libA.dart"; @A() main() { }');
    computeFast();
    return computeFull((bool result) {
      assertSuggestArguments(namedArguments: ['one', 'two']);
    });
  }

I think this can be closed, the remaining part of this issue, is the junk that is provided, but that is essentially this issue:
https://github.com/dart-lang/sdk/issues/23992
@danrubel


changes have been made

Thank you, great job!

Thanks!
Great workshop!  Did many people get to complete the exercise?
Hope so!  Learning was definitely taking place 😊
On Wed, Dec 5, 2018, 5:16 PM Nathan<EMAIL_ADDRESS>wrote:

Thank you, great job!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ndanielsen/intro-cli-git-github/pull/6#issuecomment-444712211,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQ3u7SyxQ4hSeOyCbWOoHNkt4ZT0XlAwks5u2G_XgaJpZM4ZCAen
.


Add in new metrics to existing workflows

Add in new metrics and plotting to existing workflows that evaluate mrna-prot correlation across all datasets and simulated data.

@LifeWorks I'm nearly done with this, but can you point me to the code that does the heatmaps?

I will push it into main soon.

I think this is nearly done, in the ipynb - just need to collect files and plot in a single script.

Currently, the combine_results.R can generate the heatmaps. We don't need extra python code to plot heatmap. The pipeline would be just the same as the correlation plot

oops, now I know what you mean. Because of the simulated data is not using the cptac data, so it can not be directly used.

The combine_results.R should be ready to use now. Just need to change the parameters in cwl files to match the plot-fig docker image and cwl.


Remove fqdn configuration for azad-kube-proxy from aks-core and eks-core

Removes the ability to some how change something that should be standard for all.

Depends on #867 to be merged first.

This PR is seriously outdated. We don't support EKS anymore. Closing.


Create 0392_is_subsequence.java

File(s) Modified: 4-Is-Subsequence
Language(s) Used: java
Submission URL: _https://leetcode.com/submissions/detail/819227305/_

I don't think a separate nested directory is needed for the neetcode_all problems, they can be added in the same folder as the current java solutions.

I have moved my code to the same folder that has java solutions like you said.

Thanks, @tharunkanna14!


Find a way for npm to have right access to package.json and lifecycle scripts only read-only access

This would prevent the npm worm from working without loss of functionality.

Maybe with dind?

Lifecycle scripts are run as child processes of npm

https://github.com/npm/npm/blob/1310b5f17bf7d279083142724005a5b385df50e0/lib/run-script.js#L158-L166
https://github.com/npm/npm/blob/1310b5f17bf7d279083142724005a5b385df50e0/lib/utils/lifecycle.js#L231

They are doing a sh -c

Ok, idea:

Run npm in a container (with read-write access to package.json)
Put in the path a version of sh that starts an inner-container with essentially the same rights, but read-only access to package.json.

The security implications of --privileged are unclear at best.

Need:

Docker image that contains node + docker for npm. This images runs npm with

--privileged
a special sh command that is available in $PATH and takes precedence over the default one. It runs the sh of the inner docker image

(Inner) Node Docker image to be used inside the previous container
*

For dind to work it needs to be started as a daemon (run -d). This means that a daemon needs to be started. npm commands can be docker execs into this container.
However, this is problematic as it means volume bindings are set when the daemon starts and cannot change.

Volumes can change using nsenter.
So the daemon can start without access to any dir (except maybe for globals) and an npm command would:

mount the host $PWD to a predictable location
perform the npm command inside the daemon
unmount the directory

Let's do this step by step:

[ ] Make dind work on my machine
[ ] Make dind work with rw on a file in the outer container and ro in the inner one
[ ] Try changing a volume in a dind container dynamically

[ ] Verify the inner container sees the new content as well

[ ] Make dind work in a container with node + npm
[ ] Write an exploit that works on Linux (pizza-party doesn't): a package.json that has a script that modifies package.json. Prove that npm version patch works, but this npm script does not

Done


Make releases selected bi-directional

What feature would you like to have?
At the moment, we require the "from" version to the base one, and the "to" version to be the target one. If you select them the other way around, the webapp won't display any changelog comparator.
We should make "from" and "to" bi-directional, so the greater version works as target and the other one a base, no matter in which order you set them.
What problem will this feature solve?
Some users are confused about not getting the proper changelog comparison when entering the target version in "from" select, and the base version in "to" select (an example here).
Do you want to help make the feature?
Yes
Is there anything else we need to know?
No response

I like this idea! This will make it easier to compare between versions even if you use them the "wrong" way around.


Add support for token ca cert when joining to k8s cluster

So far token ca cert was not used with kubeadm join command (skip flag was used).
It's not recommended and may lead to security breach.
This commit utilize token ca cert as described in kubernetes documentation to provide
secure way of joining nodes ot cluster.

all commits must go through https://review.opencontrail.org/ for this repository


New Label: Sipgate CLINQ

New label for the App Sipgate CLINQ, a softphone client for sipgate.de.

Thank you!