check for existence of command line arguments

check for existence of arguments variable to avoid error when using less.js in embedded interpreter

Please merge, just solved my problem when calling directly from rhino

This would have to be patched somewhere else, the files in dist/ are generated automatically.


Text to voice is not working

Hi, pls tell me how update or edit text to voice in this webapp
Tnks

Hi bro, how can update it?


Server Side Rendering

The idea is to start with a simple server rendering that will render only the main layout (App shell), this includes navbar, footer, hamburger menu and the rest will be lazy loaded (or wait for React to render the components).

gatsby all the way!


BarycentricData: share compile time and run time logic

We need the functionality of BarycentricData for both field and field_t. The former is "literal" i.e. is compatible with constexpr operations, and the former is not. The functions for computing the pre-computable arrays in BarycentricData need to be constexpr and it takes some trickery to share these functions with the non-constexpr setting. Right now everything is more or less duplicated across BarycentricDataCompileTime and BarycentricDataRunTime. There should be a way to share more of the logic.

I believe this issue can be easily resolved now that key classes like Sumcheck and Zermopmorph are separated into Prover and Verifier classes. I think there should be no need to do barycentyric extension on stdlib types.

potential subtle bug in sumcheck - verifier needs to believe that round univariate evaluations are actually evaluations of the round univariates. this doesn't come for free - verifier might need to do barycentric extension to check this.


GRAVITYAI-1251 Update Agenta backend and UI to store and display reasoning for custom metric scores

This PR:

Adds the functionality for agenta-backend to store the reasoning from our LLM as a judge custom webhook metrics.
Supports displaying the reasoning in the UI (see image below) and also returning the reasoning in our CSV files.
Note: a new column is added in the UI only if there is at least one reason given for that metric.

looks good. nice work 👍

test


Lab 3: Introduce  Markdown Functionality

For this feature, I am going to be implementing the  tag within markdown.
Inside markdown, to get a thematic break, the syntax is as follows:
To make this change active, I will modify the romanssg.py file and make small changes to how the .md file is being read. It will look for the specific syntax "---" and write a  tag within html.

Closed by https://github.com/roman-rezinkin/RomanStaticSG/commit/dcae6b470f4c9a012f6c29c1198a7da992eb93a0


Azconfig logging

Fixes #3326
Adds logging to the AzConfig client SDK.

Can one of the admins verify this patch?

I am OK with a central logging API in azure-core. I would encourage you to familiarise yourself with the recently-released Google Flogger API. This is primarily related to the performance concerns I outlined to you earlier today in our sync up. Some useful links:
https://github.com/google/flogger
https://google.github.io/flogger/benefits
https://google.github.io/flogger/anatomy.html

@JonathanGiles Thanks for that link! It was a fascinating read. Learnt something new. :D

@JonathanGiles I second Connie's statement, it was a really good read and I'll start playing around with the concept.


Google scholar as social profile

Closing to keep the theme lean and avoid configuration bloat. The defaults cover the most popular networks, anything else I envisioned being customized by the user.


link_animetion

The link_animetion file contains player animations in a different format to the usual animations found in objects. The headers for these animations are located in gameplay_keep and are already extracted as PlayerAnimations. These contain the frame count and the segment start for the animation data in link_animetion.
At each segment, there is (sizeof(Vec3s) * limbCount + 2) * frameCount worth of s16 limb rotation data. Player has 22 limbs after adding 1 in SkelAnime_InitFlex, so this is 0x86 * frameCount worth of data.
I believe then a suitable form for the link_animetion file's xml is:
<PlayerAnimationData Name="gPlayerAnimData_00D710" FrameCount="51" Offset="0x00D710"/>

Resolved by #200 🎉


Is ctre faster then perl regex ??

I want to convert a perl program to C++ , and since perl is made in C , I've decided to get a C regex library. I found one called pcre made in C , and they said it was very fast. Since perl regex also uses C , should pcre be as fast as perl regex. I don't know , and since my code is full of ctre stuff and still so slow ( I don't know why ) , I've decided to use primitive C.
So the question comes down to :
Is ctre faster then perl regex ???
Also you should open a discussions channel , because I am having to post an issue for this even though its a question

the question you are asking is same as asking which programming language is faster. Only thing CTRE is doing is converting a regex into assembly (with help of compiler), and if your regex is slow, there can be issue in its design, especially around greedy/lazy cycles instead of possessive ones. PCRE library is generally really fast, same as CTRE. In case of CTRE it depends on your compiler and optimisation levels.
I don't want to open a discussion channel as I'm doing this only in my free time. Feel free to start such discussion at includecpp.org's discord where I'm sometimes available.


Add missing ` character to exception ruleset docs

Thanks for the PR!
We are generating the documentation from the source files.
Please update ExceptionRaisedInUnexpectedLocation KDoc and build the project, so the documentation will get generated with the missing ` ;)

Fixed by cfb415e03b788688389902427f5b8a23bc3c7354


[WIP] np.unwrap implementation

With regards to #70. Illustrative colab notebook gist here. I reference cell numbers to clarify examples, but it looks like the gist removes them.
Many thanks to @hawkinsp for guidance this morning.
Some comments on the implementation:

I don't know how folks feel about importing jax.ops in lax_numpy.py
I stuck very closely to the original numpy source, but imagine lines 559-561 could be collapsed into a single call to jax.ops.index
I'm not sure if I was overzealous with type promotions

Some comments on correctness:

From what I can tell, the numpy testing is quite limited, with only two simple tests.

The first test (assert_array_equal(unwrap([1, 1 + 2 * np.pi]), [1, 1])) fails because line 551
p = _promote_to_result_dtype(onp.unwrap, p)

turns p from a DeviceArray to a list of DeviceArrays with 1 element (see colab cell 4). That's unexpected, but didn't dig into it and brute-forced past the issue with
p = _promote_to_result_dtype(onp.unwrap, p)[0]

Resolving this still fails because jax has a precision roughly equivalent to numpy's float32 and this particular example demands precision (see colab cells 5-8)

There is no longdouble dtype in jax

Second test passes fine with the cludgy DeviceArray list hack above

Once the numpy tests pass, I'd love some guidance on implementing the jax test harness for this function. I poked around, but only saw one-liners in the testing for functions like rad2deg and @hawkinsp mentioned the harness might be more like ~10 lines of code.


Sudo not working

Hello. After about a week of leaving my node alone without much change, I opened Putty to make some changes to my node. I found that calling sudo gave me the following errors
sudo: unable to stat /etc/sudoers: Structure needs cleaning
sudo: no valid sudoers sources found, quitting
sudo: unable to initialize policy plugin
I googled and I want to try this solution, but in order to unmount the external hard drive for the raspibolt, i have to disable bitcoind and lnd, both of which require sudo. Which doesn't work. So I'm a little stuck here, can I just go ahead and try the solution? My concern is that I risk damaging the file system further without any backup

No worries, only "blame" you get is making me even more interested in bitcoin and lightning. :)
Welp, things just seem to get worse and worse. my putty couldn't access the pi at all, the window would close the moment i typed 'admin'. so can't even close my channels if i wanted to :( anyway, I've taken the plunge and powered down my pi, sd card is not working after multiple tries, so i got a new one.
Can I ask, does that mean I'll follow the "Raspberry Pi" section of the Raspibolt guide? I find that I am unable to access the 'admin' account any longer, so does that mean to authorization keys I created has to be re-created?

Also, I'm not too sure what you mean by "set the systemd units" What units is it I have to set? And also, under "mounting the external hard disk, there's a warning that says "previous data on this hard drive will be deleted!", how would i mount it back?

Yes, with the new sdcard, you need to set up the operating system from scratch and follow the guide.

Ignore the "format external hdd" part, just mount the existing one.
If you still have your private & public keys for ssh, you can just add this to the "authorized_keys" file again, no need to recreate them.
Install bitcoin & lnd again, but after creating the symbolic link to the external hdd (with ln -s ...) the necessary directories & config files are already there, just check them.
You also need to recreate the "systemd units" (the .service files) for the serives to startup automatically.

Finally finally got everything up and running again, at the moment it seems like no bitcoins were lost. Thank you Stadicus!! :)


Slow Start

Type: Performance Issue
I dont know when it started, and i dont know what i have done to become like this
VS Code version: Code 1.77.3 (704ed70d4fd1c6bd6342c436f1ede30d1cff4710, 2023-04-12T09:16:02.548Z)
OS version: Windows_NT x64 10.0.22621
Modes:
Sandboxed: Yes

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-10750H CPU @ 2.60GHz (12 x 2592)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
undefined

Memory (System)
15.87GB (8.01GB free)

Process Argv
--crash-reporter-id da1c2326-2f36-4a4e-9aba-1fc0f1770db0

Screen Reader
no

VM
0%

Process Info
CPU %	Mem MB	   PID	Process
    0	   106	  3392	code main
    0	    42	  5492	   utility-network-service
    0	   134	  6476	   gpu-process
    0	    88	  8808	shared-process
    0	    67	 11052	     ptyHost
    0	   189	  9892	window [1] (Visual Studio Code)
    0	    69	 15768	fileWatcher [1]
    0	    23	 16372	   crashpad-handler
    0	   142	 20768	extensionHost [1]
    0	    36	 17420	     c:\Users\filip\.vscode\extensions\ms-vscode.cpptools-1.14.5-win32-x64\bin\cpptools.exe
    0	     4	  1852	       "c:\Users\filip\.vscode\extensions\ms-vscode.cpptools-1.14.5-win32-x64\bin\cpptools.exe"
    0	     6	 16260	       console-window-host (Windows internal process)
    0	    91	 20924	window [2] (Issue Reporter)

Workspace Info
;

Extensions (17)

Extension
Author (truncated)
Version

doxdocgen
csc
1.4.0

c-cpp-runner
fra
5.2.1

cmake-language-support-vscode
jos
0.0.9

vscode-dotnet-runtime
ms-
1.6.0

python
ms-
2023.6.1

remote-containers
ms-
0.288.1

remote-ssh
ms-
0.100.0

remote-ssh-edit
ms-
0.84.0

remote-wsl
ms-
0.77.0

cmake-tools
ms-
1.13.45

cpptools
ms-
1.14.5

cpptools-extension-pack
ms-
1.3.0

remote-explorer
ms-
0.4.0

vsliveshare
ms-
1.0.5862

gcc-md
pha
0.0.1

cmake
twx
0.0.17

vscode-lldb
vad
1.9.0

(1 theme extensions excluded)

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
vserr242:30382549
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263cf:30335440
pythondataviewer:30285071
vscod805:30301674
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
cmake_vspar411:30581797
vsaa593cf:30376535
pythonvs932:30410667
cppdebug:30492333
vsclangdc:30486549
c4g48928:30535728
dsvsc012:30540252
pynewext54:30695312
azure-dev_surveyone:30548225
nodejswelcome1cf:30587006
3biah626:30602489
pyind779:30671433
f6dab269:30613381
pythonsymbol12:30671437
showlangstatbar:30672706
vsctsb:30705553
azdwalk:30687957
pythonms35:30701012
j16hj152:30706079
pythonfmttext:30716742

@1fox4 do you have Mcafee AV installed ? We had a couple of issues reported about the AV slowing down the application (including hanging renderers) with latest updates, refs https://github.com/microsoft/vscode/issues/178962.

yes i have, i can do something to resolve that?
A qua, 19 de abr de 2023, 06:17, Robo @.***> escreveu:

@1fox4 https://github.com/1fox4 do you have Mcafee AV installed ? We
had a couple of issues reported about the AV slowing down the application
(including hanging renderers) with latest updates, refs #178962
https://github.com/microsoft/vscode/issues/178962.
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/180255#issuecomment-1514145423,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A56XDU4NHCVVVHHQ5XNC7R3XB5YNZANCNFSM6AAAAAAXDDBILI
.
You are receiving this because you were mentioned.Message ID:
@.***>

Can you follow the steps in https://github.com/microsoft/vscode/issues/178733#issuecomment-1500812938 and exclude the code.exe binary ?
Merging to https://github.com/microsoft/vscode/issues/178962

it works, thank you
Robo @.***> escreveu no dia quarta, 19/04/2023 à(s)
12:42:

Can you follow the steps in #178733 (comment)
https://github.com/microsoft/vscode/issues/178733#issuecomment-1500812938
and exclude the code.exe binary ?
Merging to #178962 https://github.com/microsoft/vscode/issues/178962
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/180255#issuecomment-1514585174,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A56XDU7DCPD2HSJQLI7HBCDXB7FRVANCNFSM6AAAAAAXDDBILI
.
You are receiving this because you were mentioned.Message ID:
@.***>


Move all of the english words to properties files

So that we can start localizing.

I think @andreadelrio has gone a long way down this path. We'll find omissions when we start to look at other languages.


fix to not filter random ports

This PR is an attempt to fix issue #272. It allows a request with random port to go though the filter.
Signed-off-by: Chanwit Kaewkasi<EMAIL_ADDRESS>

ping @aluzzardi
/cc @technolo-g for testing

LGTM

@technolo-g #310 would replace this PR. I'll close this after #310 get merged.

#310 was merged


'LinearPeptide' object cannot be converted to 'CompoundPeptidoform' error

Dear @douweschulte,
I have been trying out the version 0.9.0a1 you made previously for the panicked thread error, but I ran into an error when trying to annotate a spectrum with a proforma peptide.
TypeError: argument 'peptide': 'LinearPeptide' object cannot be converted to 'CompoundPeptidoform'
This happens when I annotate a RawSpectrum object with a LinearPeptide object and proforma string (all with the python bindings). Do you have any idea what causes this?
Kind regards,
Arthur

With the new alpha update annotation needs a CompoundPeptidoform. This because the peptide model is extended to allow for cross-linked peptides. I could not find any convenient conversion methods so I added some and will be releasing a new alpha version right now.

The automated release to PyPy seems broken. @RalfG do you know these errors? "rustc-LLVM ERROR: out of memory" but only on linux all other platforms work fine. I tried some stuff but honestly do not really know where to start.

I'm afraid the build is actually just running out of memory. Most likely, the other platforms have more RAM available, or their compilers/linkers consume less. Can you reduce the memory usage through any compilation options?

The linux runners have as much of not more RAM allowance so that puzzled me: https://docs.github.com/en/actions/using-github-hosted-runners/using-github-hosted-runners/about-github-hosted-runners#supported-runners-and-hardware-resources. Also I do not know how it uses 16 gb during compilation, I have never seen it reach that high on my pcs (both on windows and linux).

I had a look at this. Locally, I indeed don't see any issue with RAM consumption. I've been trying to reproduce the issue with act, but I didn't get it running yet, I'll have another look later on. But as a temporary workaround, I wonder if adding --jobs 1 to the maturin args would have any effect?

Thanks for the suggestion, above is the regrettably negative result. One thing this highlights however is that it is only the x86 that failed. I would assume that this runs in a 32 bit system image and as such cannot use more than 4Gb ram. With all runs before it only crashed on this one and cancelled the other two midrun.

In that case, doesn't it make sense to just disable the 32-bit build? I wouldn't think a lot of people dust of their 90's or very early 2000's PCs for scientific computing, and especially on Linux, the transition to 64 bit OS was made a while ago, with quite a few distros no longer supporting 32-bit. I'm not sure if it makes sense to invest time into it then. Although from what I observed locally in terms of memory consumption, it's even surprising it ran out of memory on a 32-bit system, assuming a PAE kernel.

That is a very sensible solution. I updated the workflow, which worked. So now I redid the release and lets hope it will work this time.

It took some more wrangling but it worked in the end, v0.9.0-alpha3 is live on pypi as well.

Thanks @douweschulte!


Clientvalidatorfixes

This small change became somewhat large as time went by and things were uncovered :)

Fixed the validation bug where parameters for a method on an interface supplied to NClient were provided in the annotation but not included in the actual declaration. The validation code now detects this scenario and throws the relevant exception.
In order to achieve this the execution pipeline needed a way to gracefully exit if an error was encountered during processing, so a new IPipelineCanceller interface and a basic implementation of it in PipelineCanceller were added, that uses CancellationTokenSource under the hood.
Added generic params to the validation Stubs
Moved clientInterceptorFactory creation to the ctor of the ClientValidator
Added validation code to skip 'hidden' methods on interfaces, added the tests for it

The net48 build picked up some errors specific to that framework version, so I'm closing this PR, will fix them, and open a new one.

The net48 build picked up some errors specific to that framework version, so I'm closing this PR, will fix them, and open a new one.

Hi @reaver330!
Errors is ok :) In the future, you shouldn't close the pull request. You can fix the problem and commit it to the branch. The changes will be displayed here, the pipeline will start again and the tests will pass


delayLoad and delayLoadTime did not work

Hi! At version 4.0.3, the delay load didn't work.
delayLoad: true,
delayLoadTime: 1000

Segment load without page scroll event or Link click.

I'll have a look. I guess nobody really has been using that or watching closely?
Are you a new user of this library, or that option? Wondering how long it's been broken.

Hi @eDubrovsky I think you just need to set the trackPage: true option...the delayLoad functionality is working for me.

I found the problem scenario.
My current setting:
trackPage: true,
trackPageDelay: 50,
delayLoad: true,
delayLoadTime: 1000

In this case, the delay time = trackPageDelay = 50. And script loaded without a page scroll event.
But delayLoadTime and loading script by scroll event no applied in this case.
If I set delayLoad to false, the script loading without delay and works correctly.

Hi @eDubrovsky maybe I'm misunderstanding. Are you saying that:

With the settings listed above
you expected the pageview to not occur until the User scrolls?

I am trying to improve my website's TTI for PageSpeed Insights by Google.
When I set delayLoad: true I am expecting this scenario:

User opens my site, and Segment is NOT loading (it helps with a better result in PageSpeed Insights)
After the user scrolls the page, the Segment loading with delayLoadTime: 1000
After that analytics.page() trigger with trackPageDelay: 50

OK, I think I understand.
You expect to not see the loading occur until either (a) the user scrolls or (b) a page routing change.
I think what I've done in 4.x is more like breaking-change and expected behavior from this issue:
https://github.com/benjaminhoffman/gatsby-plugin-segment-js/issues/37
At this point, what I think I should do is leave things how they are with the options you've chosen, but add an additional option like delayLoadUntilScrollOrRouteChange that if true will do what you're expecting....which was the old actual behavior but not the new current behavior.
How does that sound?

Code back at 3.7.1:
https://github.com/benjaminhoffman/gatsby-plugin-segment-js/tree/897db9ba7f2b0bab91aab3c87bac1682fcf36396
https://github.com/benjaminhoffman/gatsby-plugin-segment-js/blob/897db9ba7f2b0bab91aab3c87bac1682fcf36396/src/gatsby-browser.js
https://github.com/benjaminhoffman/gatsby-plugin-segment-js/blob/897db9ba7f2b0bab91aab3c87bac1682fcf36396/src/gatsby-ssr.js

@newhouse delayLoadUntilScrollOrRouteChange option sounds cool, yep! 🤟

@eDubrovsky can you please install 5.0.0-beta.0 and play around with it a bit to see if it's working correctly?
You can see the new options here:
https://github.com/benjaminhoffman/gatsby-plugin-segment-js/pull/59/files

Actually 5.0.0-beta.1, please.

Yes, it works! Like it! 🤟
p.s.: how can I turn off plugin console.logs?


Forbid unsafe code

Hi. Should this crate use #![forbid(unsafe_code)]? Could we add it now or should some steps be taken before that?

Ignoring that most of the crate(s) is autogenerated, isn't that virtually impossible for FFI crates?

That's what I'd think too but this one doesn't use unsafe code, so I figured I'd ask anyway.

It does use unsafe code in almost every file. Are you maybe in the wrong repo? Or am i misunderstanding something? I'm super out of the loop when it comes to the webkitgtk bindings tbh.

Sorry, two things went wrong here on my part.

I was specifically referring to the webkit2gtk-sys crate, not webkit2gtk. Should have checked the whole repo beforehand.
Although cargo-geiger does mark webkit2gtk-sys as not using unsafe, the keyword does appear many times in the source code. It seems to always be in declarations, so I'm not sure how that'd work with #![forbid(unsafe_code)].

Again sorry for the potential waste of time.


To verify MAC, we need a MAC

Fixes #26106

Checklist

[ ] documentation is added or updated
[x] tests are added or updated

Windows test failure is irrelevant

24 hours has passed since 'approval: done' was set, but as this PR has been  updated in that time the label 'approval: ready to merge' is not being automatically set.  Please review the updates and set the label manually.

Merged to the master and 3.4 branches. Thank you for your contribution.


Add os resources to versioned constants

Move hardcoded os resources json into the versioned constants json, move OsResources into into OsResources module.
Delete os_usage.rs: all logic is now called from methods in VersionedConstants(todo: extract into submodules, currently just a big module).
Delete os_usage_test.rs: these are not post-deserialize validations of OsResources.
the rest are getters and patching things up.

This change is

Codecov Report
Attention: 25 lines in your changes are missing coverage. Please review.

Comparison is base (e74a992) 70.04% compared to head (7430754) 69.12%.
Report is 1 commits behind head on gilad/use-versioned-constats-for-cairo-os.

Files
Patch %
Lines

crates/blockifier/src/versioned_constants.rs
80.24%
12 Missing and 4 partials :warning:

...lockifier/src/execution/syscalls/hint_processor.rs
84.09%
7 Missing :warning:

crates/native_blockifier/src/py_validator.rs
0.00%
2 Missing :warning:

Additional details and impacted files
@@                              Coverage Diff                              @@
##           gilad/use-versioned-constats-for-cairo-os    #1393      +/-   ##
=============================================================================
- Coverage                                      70.04%   69.12%   -0.92%
=============================================================================
  Files                                             60       58       -2
  Lines                                           7709     7541     -168
  Branches                                        7709     7541     -168
=============================================================================
- Hits                                            5400     5213     -187
- Misses                                          1880     1898      +18
- Partials                                         429      430       +1

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


Multiple command buses

Hi !
First of all thank you for the lib and bundle !
I want to set multiple command buses like explained in the README with:
tactician:
    commandbus:
        default:
            middleware:
                - tactician.middleware.command_handler
        queued:
            middleware:
                - tactician.middleware.queued_command_handler

But I got the following error:
  [League\Tactician\Bundle\DependencyInjection\Compiler\UnknownMiddlewareException]
  Unknown middleware with service id "tactician.middleware.queued_command_handler".

What do I miss?

Hello !
I guess we need to improve the README ;)
It was just an example to show you can have different command_handler but by default only tactician.middleware.command_handler is available

Ok thank you, I suspected a little :)

Hi @notFloran, thanks for opening this issue and sorry for the confusion!
I've tried to clarify this here: https://github.com/thephpleague/tactician-bundle/commit/f6a336d053d061bcb8128bb10203f562103ca97b
Please let me know if that helps or feel free to open a PR if you have any suggested edits. Thanks! :)

Hi @rosstuck !
Thank you, is better like that!  With the custom id some.middleware.service.to.call.the.remote.accounting.app I understand right away that it is up to me to define the service.


Changes for complying with docker 1.10

Changes include:

Changing Accept header version
Adding GetCapabilities API for IPAM

The actual diff(excluding the Godep changes) as such is not huge and can be found here:
https://gist.github.com/DivyaVavili/47b89c14e2792ae4a6f7

@DivyaVavili Changes look good Thanks.
router := mux.NewRouter()

s := router.Headers("Accept", "application/vnd.docker.plugins.v1.1+json").

s := router.Headers("Accept", "application/vnd.docker.plugins.v1.2+json").
Methods("POST").Subrouter()

You might want to change the above to something like:
s := router.Methods("POST").Subrouter()
Otherwise, netplugin will not work with docker 1.9..
I'm guessing sanity failed for the same reason..


Questions about DOMTagger - XPath & html

Hi Pascal.
For some reason I cannot get XML parsing to work (see below), and only some of my html tags are working.
What am I doing wrong?
	<preParseHandlers>
	<tagger class="com.norconex.importer.handler.tagger.impl.DOMTagger" parser="xml">
 		<dom selector="/html/body/div[1]/div[1]/div[2]/div/div/section/section/div/div[3]/span/div/div[1]/ul/li[2]/div/div[2]/div[4]" toField="p_test" />
	</tagger>
	<tagger class="com.norconex.importer.handler.tagger.impl.DOMTagger" parser="html" >
 		<dom selector=".productTile__itemTitle___3B03g div" toField="p_name" />
 		<dom selector="[data-automation-id=ingredients] div" toField="p_ingredients" />
 		<dom selector=".productTile__imageLinkOld___2SLBT" toField="p_similar_products" />
 		<dom selector="img.prod-HeroImageCarousel-image" toField="p_images" />
 		<dom selector="NutritionFacts__nutritionFacts___18C6B" toField="p_nutrition_table" />
	</tagger>

Other than suggesting you try with parser="xml" for both taggers, I cannot tell just looking at this snipper.  Can you share your full config with a sample page to reproduce the issue?

Does this help? https://gist.github.com/svanschalkwyk/5fe70144e486ea5f0bdaaf4599863dc2
These don't workt:
<dom toField="p_prod_image" selector="[data-tl-id=ProductPage-primary-image]" extract="outerHtml"/>

<dom toField="p_serving_size" selector="[class^=NutritionFacts__ssRowTwo___] h3" extract="outerHtml"/>
 		<dom toField="p_servings_per" selector="[class^=NutritionFacts__ssRowOne___]" extract="outerHtml"/>
 		<dom toField="p_calories" selector=".NutritionFacts__calories___vQPc7" extract="outerHtml"/>
 		<dom toField="p_nutrition_table" selector="[class^=NutritionFacts__nutrientTable___]" extract="outerHtml"/>

Those fields are not appearing in the html - I need to investigate further.
Thanks.

There is something I don't understand: The inspect function of Chrome shows different html from the connector-downloaded html files. There are vastly different classes in the downloaded html.
What am I not understanding?

Could it be that the site you crawl is JavaScript-driven? I.e., lots of the content is pulled via Ajax calls?   If so you may need a different strategy for crawling.  Either crawl the API that JavaScript is calling or use a browser-based approach.  Version 2.x of the HTTP Collector relies on PhantomJS for a JavaScript interpreter.  See PhantomJSDocumentFetcher.   Unfortunately, PhantomJS is no longer being maintained and it has trouble with latest Javascript specs so your mileage may vary.  Upcoming version 3 has popular browser support but it is not out yet.
Please create a new ticket for new questions not related to this one (DOMTagger).

Hi Pascal
I have been using PhantomJS. I didn't realise it had issues already.
Thanks!


Links open on default browser instead of Twitter app.

For some reason, when I open a link under this type of link, it will never open the mobile Twitter app. Instead, it will always open on the browser version of Twitter. Is there any possibility to be implemented to have them open the Twitter app instead of the browser?

Duplicates #616


feat: upgrade to swipl wasm 3.4.0 (including better regex support)

Also adds the string regex tests from #278

:tada: This PR is included in version 4.24.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release
v4.24.0

Your semantic-release bot :package::rocket:


mdb bootstrap project - Invalid React Hook call error

Hi there,
I have created a React project using create-react-app and successfully deployed it to AWS Amplify.
Next I have installed the mdbootstrap.com mdbreact package and added some content to my App.js file.
The page and components load up with no problems when running npm start and browsing to https://localhost:3000
However, the version running here https://main.d19qlnqhs16f85.amplifyapp.com throws an error in the browser inspector console. It is citing an Invalid React Hook error.

I followed the link to the React website and followed some of the troubleshooting there. I have<EMAIL_ADDRESS>and only have one copy. I have installed the linting tool and don't seem to be getting any errors on an npm run build.
Is there anything else I can try?
Thanks

The troubleshooting guide from the link in the error has a guide for adding debug logging. This printed out true for me so I had to try something else. In the end an npm dedupe worked.

npm ls react

+--<EMAIL_ADDRESS>| `--<EMAIL_ADDRESS>`--<EMAIL_ADDRESS>
npm dedupe

+--<EMAIL_ADDRESS>| `--<EMAIL_ADDRESS>deduped
`--<EMAIL_ADDRESS>
Sorted :-D


Not seeing local notifications

Running sim ios9, I see console logs being thrown and if the app is open a toast notifcation is visible, but I am unable to see any local notifications. Any tips?
Thanks

I am having the same problem. Will there be an updated version for iOS 9 or do I need to look into it by myself? I need this functionality for a project I am currently working on.

Support for iOS9 merged into master #82. Take a look and see if it works now.

I tried @Mojo90's fork yesterday, which did not work. But I will try again with the merged pull request.

I just tried the latest version, console log says "creating notification", but no notification appears. It doesn't matter if the app is open or closed. iPhone 6s on iOS 9.0.2.

Sorry @freinbichler  I didn't test Local Notifications for this Plugin cause I am using Katzers Local Notification Plugin as I do need update some things before firing local Push... (https://github.com/katzer/cordova-plugin-local-notifications)

Hello @Mojo90 , the local notification works fine for me in iOS but I've encountered an error when I open the notification:
SyntaxError: Unexpected identifier 'Domain'. Expected ')' to end a argument list.
I've looked everywhere in my code and I don't see any syntax error and it only happens when I open the app from the notification drawer, else it works fine. Any idea what caused this?

:+1:

@derek1988ec I am seeing a similar error

@derek1988ec Not really. I only get this Error if sth with my cordova html/javascript-Code is wrong. Maybe there is an error? Didn't had a look what happens if a local notification is sent. And I assume that I didn't changed the code of sending local notification for this Plugin. What I could imagine that there is an Error regarding Swift 2 that it adds sth. weird or so...

@Mojo90 yea I thought the same thing too. And usually for syntax error, most of the case that the compiler will catch it and/or error will be thrown when the app launches for the first time. That's why I find it weird that it only thrown this kind of error when I tap on the notification from the notification drawer...

What do you mean by notification drawer? Do you mean Notification Center?
But nevertheless currently I have little time to investigate this, sorry :-( we are in development of new App-Version and in preparation for Web Summit from 2.-5.11. in Dublin...

yea apology @Mojo90, it's the notification center. I always get it messed up between android and ios. Anyway I do apologize to disturb you for your busy time. I'm looking into the problem now and will update here if I find any solution to it

Notifications work for me. However, I am testing on simulator iPhone 6, iOS 9.0.

@Mojo90 @tsubik I found the error, it's from this line of code in geofenceplugin.swift
let js = "setTimeout('geofence.onNotificationClicked("+ data +")',0)"
the 'data' output 'ErrorDomain' as 'Error Domain'. Not sure why.

@derek1988ec did you fix this?

I can now confirm that notifications are working, the problem was that I was not asked for permission to display notifications. Deleting and installing a fresh build fixed it.

Please feel reopen if problem is not solved.


Settings window wrong displayed after setting the taskbar back to the bottom

I tried the application, setting the taskbar to the top. Now, after setting it back to the bottom, when I open Windows' Settings, it will be shifted when maximized as if the taskbar was on top.
I've already deleted the .exe file from the application and tried rebooting my PC.

It suddenly fixed itself.

Hi there, thank you for reporting the issue!
It is possible that not all settings can be correctly reset after killing and launching the 'explorer.exe' process. For some things a full restart might be needed. I think this is the cause for the issue you described.


Update Singer Spec description to be clearer

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/2663
Originally created by @tayloramurphy on 2021-03-22 15:27:16

A lot of work could be done to simplify https://github.com/singer-io/getting-started/blob/master/docs/SPEC.md#singer-specification to make a "human readable" version. This would help adoption so that folks could have a high-level takeaway but then dive in deeper if they need to.

View 4 previous comments from the original issue on GitLab


BPARules : Add data category for columns does not work properly for Latitude and Longtitude

Hello,
from my point of view the rule Add data category for columns does not work properly for Latitude and Longtitude columns.
I believe the issue is related with its bracket definition.
I suggest to update the rule definition by following way:
(     string.IsNullOrWhitespace(DataCategory)     or     DataCategory.ToLower() == "uncategorized" ) and (     (         (         Name.ToLower().Contains("country")         or          Name.ToLower().Contains("continent")         or         Name.ToLower().Contains("city")         )         and DataType == "String"     )     or     (         (             Name.ToLower() == "latitude"               or              Name.ToLower() == "longitude"         )         and         (             DataType == DataType.Decimal              or              DataType == DataType.Double         )      ) )
I did some testing on my test dataset and it has been working fine. But I will appreciate any feedback from your side.
Regards,
Jiri

This portion of the rule as it is catches columns named 'latitude' or 'longitude' which are of decimal or double datatype and do not have a data category. It appears you have added a clause that assesses if the data category shows as 'uncategorized' but this value would be blank if it is uncategorized. No one would intentionally mark this as 'uncategorized'. Power BI Desktop shows a column as 'uncategorized' if the data category has not been set but the actual .bim file does not show 'uncategorized' so looking for 'uncategorized' would not be helpful.

Hello,
thank you for your response. You are right that no one should intentionally mark this as 'uncategorized'. But I am working with the scenario when someone has changed the Data category by mistake. PowerBI Desktop is filling the data category in .bim model with this value when you set the Uncategorized Data Category in model manually. By default this value is not filled in .bim model. But let's try following example.
I have a column State. In Power BI Desktop I change the Data category to a wrong value, e.g. City.  In the .bim model TabularEditor now shows the Data Category as "Uncategorized". I know this case is rare, but it is a rule and I believe it could follow most of the possibilities. And this was a small change to the rule definition.
However this change in the rule is not necessary and it does not take effect with the longitude and latitude issue.
The origin rule does not take effect for the longitude and latitude columns at all. I made a change in parenthesis command ordering. I attached a zip file with the sample .bim model. It contains only a one table with Langitude and Longitude column. Both have Data Category set. Unfortunately origin rule still shows these columns as violating this rule. My update of the rule does not.
langitude_longitude_example.zip


.scons_env.json logging improved

Many more types are now serialized
Any remaining unserialized types now have the same pprint wrapper SCons utilizes
File guaranteed to export as utf-8; allows ensure_ansi to be disabled

Oh that is... A lot of failed runs
Maybe the defines are different from the latest SCons versus 3.0? Gonna have to look into this further, whoops

Could you clarify what's your use case for this, what the output looks like before and after, and whether this impacts build time?

Sure thing!

use case: In local testing, I noticed that the env.Dump() method from SCons 4.0 had a pretty option in addition to the json option that's very similar to what we use — the former was quite a bit more verbose because it ended up natively serializing more types. At first this was just pretty-printing the non-serialized types, which technically worked, but it ended up as long strings for a lot of the types and I really wanted the proper json representation instead. So I dug into a lot of the classes that weren't serialized but really should be & implemented simple logic to have them display in a human-readable format. This is all to the end of making this debugging file much more usable for debugging
diff: https://gist.github.com/Repiteo/45a92567fea3ea3ea01d6cdb0f6c419f
build time: methods.dump() before ≈ 0.00102s, after ≈ 0.00203s. Ended up being roughly twice as long, but the original method was already extremely fast, so it safely remains a drop-in-the-bucket for overall build time

Added a few more serializations, and made all conversions one-liners: https://gist.github.com/Repiteo/388bbdf82ff7c4cd5d4d5e37e6bdfaa5
Also, the build time might be even less impacted than I thought. I'm actually getting ranges from 10-20ms with no real consistency from what I can find, so that range may very well be inherent to the function regardless of what all has been added

Better parsing logic was added to SCons upstream, so this is no longer needed.


Made documentation less ambiguous about classes

added example for loading a different config based on platform

After comments answered, :+1:


[grid] Remote data cannot selectNone

Hi, the first thanks to this plugin :dancer:
I have a problem, when i use grid with remote data. And I cannot use selectNone function.
Plz help me, thanks u.

does it work from JS console? If you do
w2ui.grid.selectNone()

Hi, this is my JS console

The problem resolved. Thanks= you!


Implement VSC dev container

See
See https://marketplace.visualstudio.com/items?itemName=ms-vscode-remote.remote-containers and https://code.visualstudio.com/docs/devcontainers/tutorial if needed

No longer needed


fix: add disconnect vCenter Server prior to host restart

In order to have a good experience with our community, we recommend that you read the contributing guidelines for making a pull request.
Summary of Pull Request
Fix add a disconnect from vCenter Server prior to ESXi host reboot.
Type of Pull Request

[x] This is a bug fix.
[ ] This is an enhancement or feature.
[ ] This is a code style / formatting update.
[ ] This is a documentation update.
[ ] This is a refactoring update.
[ ] This is a chore update
[ ] This is something else.
Please describe:

Related to Existing Issues

Closes #22
Test and Documentation Coverage

[x] Tests have been completed (for bug fixes / features).
[ ] Documentation has been added / updated (for bug fixes / features).

Breaking Changes?

[ ] Yes, there are breaking changes.
[x] No, there are no breaking changes.

Also update the change log and the build number in the psd file.

Examples:
Increment the version to v1.1.0 and the first build. Result: <IP_ADDRESS>0.
    # Version number of this module.
    ModuleVersion = '<IP_ADDRESS>0'

# Release History

## v1.1.0 (Not Released)

Bugfix:

- Added a disconnect from vCenter Server prior toan ESXi host reboot`. [GH-36](https://github.com/vmware/powershell-module-for-vmware-cloud-foundation-certificate-management/pull/36)

## [v1.0.0](https://github.com/vmware/powershell-module-for-vmware-cloud-foundation-certificate-management/releases/tag/v1.0.0)

> Release Date: 2023-05-30

Initial availability of the PowerShell module for VMware Cloud Foundation Certificate Management.


[KeyVault] Onboard KeyVault Round 1

This is the first in a series of PRs which will onboard the KeyVault data plane commands. This PR:

Updates management plane commands to enable certificates
Adds majority of commands (and test scenarios) for keys and secrets
Includes first draft of the Python convenience layer (which does NOT include the authentication piece)

Note that the client was generated in autorest and will be removed in favor of direct import once the Python SDK is approved and released.
@yugangw-msft for the changes to profile and @derekbekoe for the changes to the management plane commands. Anyone feel free to drop feedback on the commands themselves. :smile:

keyvault group
Group
    az keyvault: Safeguard and maintain control of keys, secrets, and certificates.

Subgroups:
    certificate
    key
    secret

Commands:
    create       : Create a new Key Vault.
    delete       : Delete a Key Vault.
    delete-policy
    list         : List Key Vaults within a subscription or resource group.
    set-policy
    show         : Show details of a Key Vault.
    update       : Update properties of a Key Vault.

keyvault set-policy
Command
    az keyvault set-policy

Arguments
    --name -n      [Required]: Name of the key vault.
    --object-id              : A GUID that identifies the principal that will receive permissions.
    --resource-group -g      : Proceed only if Key Vault belongs to the specified resource group.
    --spn                    : Name of a service principal that will receive permissions.
    --upn                    : Name of a user principal that will receive permissions.

Permission Arguments
    --certificate-permissions: Space separated list. Possible values: all, get, list, delete,
                               create, import, update, managecontacts, getissuers, listissuers,
                               setissuers, deleteissuers, manageissuers.
    --key-permissions        : Space separated list. Possible values: all, encrypt, decrypt,
                               wrapKey, unwrapKey, sign, verify, get, list, create, update, import,
                               delete, backup, restore.
    --secret-permissions     : Space separated list. Possible values: all, get, list, set, delete.

keyvault key group
Group
    az keyvault key

Commands:
    create        : Creates a new, named, key in the specified vault.
    delete        : Deletes the specified key.
    list          : List keys in the specified vault.
    list-versions : List the versions of the specified key.
    set-attributes: Updates the Key Attributes associated with the specified key.
    show          : Retrieves the public portion of a key plus its attributes.

keyvault key create
Command
    az keyvault key create: Creates a new, named, key in the specified vault.

Arguments
    --name -n       [Required]: Name of the key.
    --protection -p [Required]: Specifies the type of key protection.  Allowed values: hsm,
                                software.
    --vault-name    [Required]: Name of the key vault.
    --disabled                : Create key in disabled state.
    --expires                 : Expiration UTC datetime  (Y-m-d'T'H:M'Z').
    --not-before              : Key not usable before the provided UTC datetime  (Y-m-d'T'H:M'Z').
    --ops                     : Space separated list of permitted JSON web key operations. Possible
                                values: encrypt, decrypt, sign, verify, wrapKey, unwrapKey.
    --size                    : The key size in bytes. e.g. 1024 or 2048.
    --tags                    : Space separated tags in 'key[=value]' format. Use "" to clear
                                existing tags.

keyvault key set-attributes
Command
    az keyvault key set-attributes: Updates the Key Attributes associated with the specified key.

Arguments
    --name -n    [Required]: Name of the key.
    --vault-name [Required]: Name of the key vault.
    --enabled              : Enable the key.  Allowed values: false, true.
    --expires              : Expiration UTC datetime  (Y-m-d'T'H:M'Z').
    --not-before           : Key not usable before the provided UTC datetime  (Y-m-d'T'H:M'Z').
    --ops                  : Space separated list of permitted JSON web key operations. Possible
                             values: encrypt, decrypt, sign, verify, wrapKey, unwrapKey.
    --tags                 : Space separated tags in 'key[=value]' format. Use "" to clear existing
                             tags.
    --version -v           : The key version. If omitted, uses the latest version.

keyvault secret group
Group
    az keyvault secret

Commands:
    delete        : Deletes a secret from the specified vault.
    list          : List secrets in the specified vault.
    list-versions : List the versions of the specified secret.
    set           : Sets a secret in the specified vault.
    set-attributes: Updates the attributes associated with the specified secret.
    show          : Gets a secret.

keyvault secret set
Command
    az keyvault secret set: Sets a secret in the specified vault.

Arguments
    --name -n    [Required]: Name of the secret.
    --value      [Required]: The value of the secret.
    --vault-name [Required]: Name of the key vault.
    --content-type         : Type of the secret value such as a password.
    --disabled             : Create secret in disabled state.
    --expires              : Expiration UTC datetime  (Y-m-d'T'H:M'Z').
    --not-before           : Key not usable before the provided UTC datetime  (Y-m-d'T'H:M'Z').
    --tags                 : Space separated tags in 'key[=value]' format. Use "" to clear existing
                             tags.

keyvault secret set-attributes
Command
    az keyvault secret set-attributes: Updates the attributes associated with the specified secret.

Arguments
    --name -n    [Required]: Name of the secret.
    --vault-name [Required]: Name of the key vault.
    --content-type         : Type of the secret value such as a password.
    --enabled              : Enable the secret.  Allowed values: false, true.
    --expires              : Expiration UTC datetime  (Y-m-d'T'H:M'Z').
    --not-before           : Key not usable before the provided UTC datetime  (Y-m-d'T'H:M'Z').
    --tags                 : Space separated tags in 'key[=value]' format. Use "" to clear existing
                             tags.
    --version -v           : The secret version. If omitted, uses the latest version.

Issue #1029 is preventing this from passing the CI.

cc/ @pomortaz for review and feedback (authentication will be incorporated in a later PR)

@tjprescott the endpoint for KeyVault in china is https://vault.azure.cn

@tjprescott: I have worked out why your recent build failed - https://travis-ci.org/Azure/azure-cli/jobs/165952645.
In src/command_modules/azure-cli-keyvault/setup.py, you should list the packages you want to be included in the packages array in the setup(...) function.
You are getting the error ImportError: No module named 'azure.cli.command_modules.keyvault.keyvaultclient' because azure.cli.command_modules.keyvault.keyvaultclient is not in this list of packages.
See this other setup.py file as an example - https://github.com/Azure/azure-cli/blob/master/src/command_modules/azure-cli-network/setup.py#L51.

@derekbekoe you're a genius! Thank you 😄


Replace WPF context menu with VS context menu

Replace WPF right-click context menu on Editor Bar with a Visual Studio native context menu.
Reasons:

provides consistent UI;
reflects VS theme;
VS icons are tied to theme and dark themed VS will render white icons on silver WPF context menu.

Seems doable by creating a new VS context menu, but also has a drawback: menu item accelerators are not visible (pressing Alt activates the main menu and closes the context menu).
Looks better than standard WPF menu:

Since the menu items are VS commands, there's a possibility to include them in menus and keyboard customization, but I'll probably prevent that as it maybe out-of-scope of this extension (?).

[ ] Decide whether to allow customization of context menu commands.


sortBy and IS routing

What is the best way to use sortBy and instantsearch.js routing?
What I'm trying is to have urls like &sort_by=price:desc filtering results and preselect sortBy widget option

You’d just have to set routing: true when initializing instantsearch like this: https://github.com/typesense/showcase-songs-search/blob/39b3a402da0d0df4cda458daeb3909e030e3ba58/src/app.js#L158
More details here.

I'm using https://www.algolia.com/doc/guides/building-search-ui/going-further/routing-urls/js/#seo-friendly-urls so it is not that simple :) There is createURL, parseURL, stateToRoute and routeToState works fine, but tricky with sortBy widget

Ah yes, SEO-friendly URLs were a little not-so-straightforward to setup with InstantSearch/js, so I haven't ventured into it yet. Let me know if anything in their docs work with Algolia but not Typesense. That would indicate an issue with this adapter and I can take a closer look.


help after a recent update, the ( StaticData::getPaintKit(item).name.c_str() "c_str()" ) function stopped working

help after a recent update, the function ( StaticData::getPaintKit(item).name.c_str() "c_str()" ) stopped working after update (https://github.com/danielkrupinski/Osiris/commit/bd775b8731d8c99f32e9b57bf23c6dae0004d814)
error C2039: "c_str": is not a member of "StaticData::ItemName".

 AAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA


Pipe access

const references = ({Model, references}) => R.pipe(
  R.prop('model'),
  R.when(R.is(String), R.pipe(
    R.prop(__, this.sequelize.models),
    relations(Model)
    )
  )
)(references);

How to access to pipe arguments without arrow wrapper function
Like this:
R.pipe(
  R.prop('rawAttributes'),
  R.map(references)
);

Sorry, I ignored this when I first saw it because I didn't know what you were asking, told myself I'd come back to it and promptly forgot.
Could you add some more explanation of what you're trying to do and what is not working?

I allready found answer, thanks.


Adding global styling for a tags

This PR adds global styling to set the color of any a tags used in Varnish 2.0, adds the link default and contrast colors to the extended palette, and adds a linkContrast class to apply to a tags for contrast styling.

i think you need to base off the theme, or else they get hardcoded to the default values in the theme file

makes sense for dynamically changing values, i wasn't sure if it mattered in this case since the paletteExtended values for default and contrast are hard-coded as is but this is a good practice point either way, changed it :)

im assuming it works :-P

yes, did test this with different scenarios 👍


crude oil deposit

crude oil deposit not showing up

first of all, thanks for posting your issue sparrow!! can you elaborate on the scenario in which you cant find a crude oil deposit? for me, i

have create, fabric api and create diesel generators fabric restitched rel3 installed
create a new world
type /cdg oil locate

fly to the chunk coordinate 5 1
right click with the oil detector item

without cheats, you would have to manually scan every chunk by right clicking with the oil detector. you can enable chunk borders with F3+G
in my survival playthrough of a bunch of create mods, i also have no issue with finding oil deposits, although admittedly, good ones seem a little rare


CAMEL-14590 Camel-Pulsar consumer calls unsubscribe on doStop/doSuspend, potentially deleting subscription

In the PulsarUtils stopConsumers method all the consumers call unsubscribe(). This causes the subscription to be deleted if this call comes from the last consumer. The PR makes the unsubscribe call configurable so that unwanted subscription deletion can be prevented.

Lgtm but please add the jira issue number to the commit message and run the build with source check profile enabled, thanks

I ran with sourcecheck enabled locally and didn't see any messages or changes to the files (not entirely sure what I should be looking for?). Am I good or do you see a glaring issue?

There should be some warnings at the beginning of any


russian jsc corrected

SInce about 2014 there is no such type of business entity like "Закрытое акционерное общество" ("Closed joint-stock company"). Correct version "Акционерное общество" ("Joint-stock company").

@mfperminov Thanks!


ANDROID-72 FAQ search

Also screenshots are missing - this is UI change. And improve the pull request name, mention JIRA task there.


Update Secrets Documentation

This is a...

[x] Feature Request
[ ] Bug Report

Study this Concept topic: Secrets. Much of the content is task-like. See how much of the task-like material is already covered in existing Task topics. For task-like content that is not already covered, create new Task topics, and remove the task-like content from  Secrets. In the Task topics, if possible, create steps that the reader can actually do. Put the remaining conceptual material into the Concept template. Edit the content to comply with our style guide. Check for accuracy. Here's an existing Task topic: Distribute Credentials Securely Using Secrets. Look at the overall body of topics related to Secrets. Do we have the right set of topics? Have we covered all the important points?. Is there overlap that could be reduced?

/remove-lifecycle stale
/lifecycle frozen
/reopen

/language en

/priority important-longterm

/triage accepted

https://github.com/kubernetes/website/issues/25962 is along the same lines

https://github.com/kubernetes/website/issues/25962 is along the same lines

Also see issue #4410

Also see issue #4410

/sig apps
/sig security

/assign

I'll use this doc to plan my work (and check if improvements are really needed)

Based on an initial analysis, the overarching scope of work is:

Update Secret concept
Create new tasks:

Allow containers to connect over SSH by mounting SSH keys as Secrets
Distribute multiple Secrets to Pods that share common templates
Hide files in Secrets from normal listing commands (why?)
Restrict Secret access to individual containers in Pods
Edit or replace a Secret

Mark a Secret as immutable

Update existing tasks:

Pull an Image from a Private Registry
Distribute Credentials Securely Using Secrets
Distribute Credentials Securely Using Secrets

Hyperlink all the things (see Existing Secret-related topics for a starting point)

I've reached out to sig-security with an invitation to comment on the Scope of Work in the doc. I'll leave that open for 1 week and will action the scope of work in that state starting on Thursday, May 26th. I'll do smaller PRs where possible to avoid XXL reviews.

/milestone clear

/milestone clear

Hide files in Secrets from normal listing commands

That wasn't how I read the existing example. I thought the intent was to show how you can take confidential config that an app expects to find in a hidden file (ie, the name begins with .) and manage that with a ConfigMap. What we're competing with here is the rather unwise alternative of baking the config into your container image, or possibly something complicated with sidecars.

That wasn't how I read the existing example. I thought the intent was to show how you can take confidential config that an app expects to find in a hidden file (ie, the name begins with .) and manage that with a ~ConfigMap~ Secret. What we're competing with here is the rather unwise alternative of baking the config into your container image, or possibly something complicated with sidecars.

So something like "Manage sensitive configurations using ConfigMap Secrets" with a real-world ish example of that

I haven't touched this in a bit but I will continue to work on it.
/triage accepted

https://github.com/kubernetes/website/issues/42235 has a specific comment relating to this issue.

Also see https://github.com/kubernetes/website/issues/32564


Add FindEmailCommand

#79
Previously, there is only the general FindCommand that returns persons whose names, addresses, emails, tags and statuses contains the keywords.
Let's add a more specific command to display search results of a certain attribute, in this case returns persons whose emails fit the keywords.

Codecov Report
Base: 64.30% // Head: 64.74% // Increases project coverage by +0.43% :tada:

Coverage data is based on head (f69a92c) compared to base (18b5137).
Patch coverage: 91.66% of modified lines in pull request are covered.

:exclamation: Current head f69a92c differs from pull request most recent head 6467166. Consider uploading reports for the commit 6467166 to get more accurate results

Additional details and impacted files
@@             Coverage Diff              @@
##             master      #80      +/-   ##
============================================
+ Coverage     64.30%   64.74%   +0.43%
- Complexity      433      448      +15
============================================
  Files            94       97       +3
  Lines          1471     1495      +24
  Branches        159      162       +3
============================================
+ Hits            946      968      +22
- Misses          488      489       +1
- Partials         37       38       +1

Impacted Files
Coverage Δ

.../seedu/address/logic/parser/AddressBookParser.java
77.27% <0.00%> (-3.68%)
:arrow_down:

...odel/person/LocationContainsKeywordsPredicate.java
85.71% <ø> (ø)

...s/model/person/EmailContainsKeywordsPredicate.java
85.71% <85.71%> (ø)

...seedu/address/logic/commands/FindEmailCommand.java
100.00% <100.00%> (ø)

...u/address/logic/parser/FindEmailCommandParser.java
100.00% <100.00%> (ø)

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Fix "personal-settings.md" broken link

A broken link to "personal-settings.md" now points to the correct page.
Solves #299

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

See my comment https://github.com/SenseNet/sensenet.github.io/issues/299#issuecomment-538680780


fix #1578 and some improvements

Add shell support for ssh connection, so that remote script can be started with user environment.
Minor fixes,

Fix gpu_metrics_collector to support pyenv. As pyenv will create one more process, so that original pgrep code always got extra processes, and cannot start gpu_metrics_collector.
Fix NASUI failure on dev-install-node-modules, to create subfolder every time.
Add .gitignore items for dev-install.
Add node --watch for nni_manager for better dev experience.

dev env test passed.

suggest to merge into a dev branch to run integration-test-remote  and integration-test-remote-windows pipelines first

suggest to merge into a dev branch to run integration-test-remote and integration-test-remote-windows pipelines first

@chicm-ms
what's process to create a dev branch, and how to enable those tests on dev branch?

Create #2370 as a dev branch. please continue review there.


feat(Daily Challenge LeaderBoard): Added the DailyChallengeLeaderboard

In this PR

Added Daily Challenge Leaderboard

Codecov Report

Merging #8 (3c43a36) into main (475fb37) will decrease coverage by 0.32%.
The diff coverage is 27.77%.

@@             Coverage Diff              @@
##               main       #8      +/-   ##
============================================
- Coverage     51.16%   50.84%   -0.33%
- Complexity      249      251       +2
============================================
  Files            74       76       +2
  Lines          1288     1306      +18
  Branches        101      101
============================================
+ Hits            659      664       +5
- Misses          606      619      +13
  Partials         23       23

Impacted Files
Coverage Δ

...er/server/daily_challenges/DailyChallengeEntity.kt
0.00% <0.00%> (ø)

...acter/server/user/public_user/PublicUserService.kt
5.47% <0.00%> (-0.50%)
:arrow_down:

...leaderboard/DailyChallengeLeaderboardController.kt
75.00% <75.00%> (ø)

...racter/server/user/public_user/PublicUserEntity.kt
100.00% <100.00%> (ø)

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


Enhance HttpStatus Class with Improved Error Handling and Immutability

Improved error messaging for unrecognized HTTP status codes and added verification test case.
Added @immutable annotation to HttpStatus class for enhanced immutability.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
30

Relevant Lines:
30

💛  - Coveralls


#203 test migration to selene 2.x version

Migrate and actualize tests from tests_from_past/integration selene v1 to tests/integration. selene v2.
Relates #203

Migrate and actualize tests from tests_from_past/integration selene v1 to tests/integration. selene v2.
Relates #203

Ready for review


Prevent return extra userData

Describe your changes
If there is a difference between local and cached data, we trigger a cache refresh; as we remove users from the local store and potentially from the remote IDP, we need to switch the source of truth to the local store to prevent unwanted endless cache for cases where the removal from the IDP fails or for cases where the userDeleteFromIDPEnabled got enabled after the first user deletion.
This also aligns with the change in the warmup method in PR #1147
Issue ticket number and link
Checklist

[x] Is it a bug fix
[ ] Is a typo/documentation fix
[x] Is a feature enhancement
[ ] It is a refactor
[ ] Created tests that fail without the change (if possible)
[ ] Extended the README / documentation, if necessary

It is unclear what was the issue

It is unclear what was the issue , could you please describe in the ticket?

updated, have a look


🐛  Entering the tab key in the text editor does not indent the line but tries to go out to the next field on firefox

Entering the tab key in the text editor does not indent the line but tries to go out to the next field on Firefox

Also happens on Chrome (Version 74.0.3729.157 (Offizieller Build) (64-Bit))

Switching to the next input is the browsers default behavior. We need to implement a custom handler for Tab Key events. An example on how to do that can be found here: https://jsfiddle.net/2wAzx/13/

This was fixed in #578 and will be part of the release 3.2.0 which will be available shortly


backend/epoll: implement eventfd wakeup notification

Tries to mimic what happens in backend/kqueue.
Closes #4

Added note, without this, trying to use any thread scheduled operation (like file io) will be very slow (up to 100ms).


WIP - 1 service account loading

connects to #1
This is a work in progress to add the service account logic.
The end goal should be, we first check to see if there is a .kube/config available, if not, then check for the service account stuff, if not, then fail.

Coverage decreased (-9.4%) to 90.566% when pulling e3ff483ac7eb5b624ee7e5426ff71987b4b7bf14 on lholmquist:1-service-account-loading into 77a2d3114819f6dbfe4faa557bb7d3f3f3fc513e on bucharest-gold:master.

Coverage decreased (-9.4%) to 90.566% when pulling d0429df1d2401c8ff2d0bfb4b0d163745bc1601a on lholmquist:1-service-account-loading into 77a2d3114819f6dbfe4faa557bb7d3f3f3fc513e on bucharest-gold:master.


Add TCP Servers Transports support

What does this PR do?
This PR introduces TCP servers transports support.
Motivation
Fixes https://github.com/traefik/traefik/issues/7803
More

[x] Added/updated tests
[x] Added/updated documentation

Additional Notes
Co-authored-by: Antoine Couchard<EMAIL_ADDRESS>

@sdelicata Awesome, I was just struggling with this very problem. When will there be an update Docker image - experimental or otherwise - in order to try it out?


convert byte strings in python3 to fix #145

This is a simple fix to make the CLI work with python3. I took the code from one of the tests. It fixes #145.

Sorry for late reply. Tested this now and it works fine, so merged, execpt that I removed one comment as wasn't really correct and shouldn't be needed anyway. Thanks!

@bfredl @fwalch @tarruda Was this published to pypi?

@justinmk I'm tagging a release.

I tagged but @tarruda needs to push to PyPI.


Migrations no longer work correctly

Link to reproduction
No response
Payload Version
3.0.0-beta.70
Node Version
22.5.1
Next.js Version
15.0.0-rc.0
Describe the Bug
Changes introduced in PR #7320 changed the migration function structure, making it unable to compile.
If left unchanged, you get the following:
Type error: Property 'execute' does not exist on type 'LibSQLDatabase<Record<string, never>> | PostgresDB'.
  Property 'execute' does not exist on type 'LibSQLDatabase<Record<string, never>>'.
  2 |
  3 | export async function up({ payload, req }: MigrateUpArgs): Promise<void> {
> 4 | await payload.db.drizzle.execute(sql`

I've attempted to create a new migration, to see what changed, however it still leads to a broken file, shown below:
Type error: Property 'db' does not exist on type 'MigrateUpArgs'.
  2 |
  3 | export async function up({ db, payload, req }: MigrateUpArgs): Promise<void> {
> 4 | await db.execute(sql`

Reproduction Steps

Attempt to run any migration from before 3.0.0-beta.68 inclusive.
Attempt to create a new migration through payload migrate:create example.

Adapters and Plugins
db-postgres

Hey @dinckelman, thank you for reporting this. I cannot believe we didn't catch this before releaes.
I've published 3.0.0-canary.6d066c2 that has the fix for you and anyone else that needs this immediately. @HarleySalas
You will have to install all the @payloadcms modules with the exact same version:
"payload": "3.0.0-canary.6d066c2",
"@payloadcms/db-postgres": "3.0.0-canary.6d066c2",
....etc

Cheers!


[Global] Revisit open menu on hover

We DON't have to change the current behavior, just creating an issue in case this comment got slip through the cracks.

@rusackas I'm finding a case where hovering to show menu might have become an inconvenience:
https://user-images.githubusercontent.com/335541/108138715-6439ed80-7073-11eb-8233-e56ef5a99322.mp4
When saving chart, I wanted to click on the "Save" button, but if I move the mouse too fast, it will trigger open the hover menu and when I moved back to the position of the "Save" button and click, I would accidentally click on the menu and lost all my changes to the chart---which is quite annoying.
I'm wondering if we can:

Increase the size of the run button area---it could use a little bit more margin around the buttons.
Move SQL lab to the second menu item so the hover menu doesn't overlap with the buttons.
Add a delay to open the menu on hover?

to avoid users accidentally clicking on the hovered menu item. We can do one or all of these if they make sense to you.
cc @junlincc @mihir174
Originally posted by @ktmud in https://github.com/apache/superset/issues/12025#issuecomment-780207617

@ktmud
All make sense, what i was thinking is to  move run button to the bottom, move save to top right corner, once we simplify and reorganize the controls?

In the long term, I'm a fan of @junlincc 's proposal... just moving to SIP-34's layout.
In the meantime, there are probably other places one might overshoot with the mouse and run into similar issues.
Option 2 is reasonable enough, but I'm not sure it will solve the problem in other areas.
Option 3 is intriguing to me... we could do it with JS, or we might be able to do it with a little CSS like this little demo I just scratched together.

@rusackas CSS delay looks cool. I did some testing and felt 250~300ms is the most comfortable.

"When creating/editing a chart, the save button lines up underneath a drop down to navigate to SQL Lab. The hover area that expands the drop down is pretty large. If a user (me) wanted to save their work on the chart (I did), they might inadvertently trigger the hover to expand the drop down and click through to the SQL editor losing their work on the chart (doh! it happened to me).
It's obviously not a huge deal. Maybe the hover could only be over the text instead of the area or the save button could be moved or a warning could allow the user to cancel navigating away from a chart with unsaved changes."
cc @steejay we have an open ticket to track this issue. ^

For what it's worth I would prefer to replace "open on hover" with "open on click". Open on click is a pretty common pattern in other apps and would prevent accidental menu opens. Hope this is the right place to share this kind of feedback/opinion.


problem: free(): invalid pointer

If I import cv2 first and import pyanime4k, then python exits with error mesage as followed:

free(): invalide pointer

but strangely, if i import pyanime4k first and import cv2 then, everything goes well.
I just install pyanime4k accross pip and I test it on WSL2 of Win10.
This error can be avoid by import pyanime4k first, but I still open an issue for some one encounters the same issue.

Interesting, I tested it in Windows10, no mater importing pyanime4k first or cv2 first, everything goes well, but in manjaro Linux, it happened.

Well, after more testing, It will only happen in Linux which installed pyanime4k by pip. I recompiled libac.so by GCC 10 in Manjrao Linux, everything goes well.
Linux pip version was compiled by a old version of GCC in Ubuntu 14.04, this is for making sure it can run in most of Linux distributions , but it can also lead to some compatibility problems. If it is possible, please compile libac.so by yourself for best compatibility.

thanks~


Add example script illustrating use of temperature measurement with noise

Tickets addressed: bsk-680
Review: By commit
Merge strategy: Merge (no squash)

Description
This scenario is an exact replica of scenarioAttitudeFeedbackRW. The only difference lies in the fact that this scenario uses the motorThermal module for gathering true temperature data from reaction wheels and tempMeasurement module for adding noise into the temperature readings. The results and performance are expected to closely resemble those of the original scenario, with JitterSimple set to false and RWVoltageIO set to true.
Verification
Did clean build, ran scenario without issue. Clean documentation build.
Documentation
Added scenario documentation.
Future work
None

I combined all the changes in the script file into one commit and split a commit into four for each file.


pad up to 90m of zero deviations after site change

Last night we saw an example of a site change followed by lots of positive deviations (likely meal-related, but not all excluded as such) that caused autosens to detect its maximum allowed resistance, and subsequently caused oref0 to be a bit too aggressive with SMB.  This change makes autosens pad up to 6 additional zero deviations after a site change, so it'd be less likely to detect resistance based on too-little data.

Unless someone wants to explicitly test this, I'll go ahead and merge this to dev for real-world testing there.


RTC's Logo Not Appearing

Recent change has made load times much quicker but now the RTC Logo is not appearing for me
Mobile screenshot:

This is because one of the script tags that I set to defer isn't loading in time. It also caused the visual bug of all of the committee images not appearing one at a time.


Add merge implementation for DataFrame and as free function

What do these changes do?

These changes contains merge implementation for inner and left types of join, some fixes of documentation for methods, changing the logic regarding axis parameter for _join_index_objects method.

[x] passes flake8 modin
[x] passes black --check modin
[x] signed commit with git commit -s
[x] Resolves #1204
[x] tests added and passing

@devin-petersohn , just a friendly reminder to review. For now most of cases with left and inner types of join are covered.

@devin-petersohn , thanks you for suggestions! The comments are addressed.

@YarShev you will need to rebase on master for CI to run.


Fixing a test failing on Mac and Linux

Replacing non-ASCII character with equivalent Unicode character escape sequences

:shipit: when builds are green (and ask mode approval)

Mac is red due to a failure @cesarbs is looking at. I coded the fix on Mac and tested it there. It's been approved by @Eilon


x/pkgsite: linkify vulnerability aliases

The "Aliases" section for vulnerability reports should make CVE and GHSA IDs into links.
We should also stop adding these links to the "References" section of the OSV data by default. We do want to manually include advisory links on occasion, but we shouldn't put a CVE/GHSA link in the references just because the advisory aliases reference it; that's redundant and it's better to generate the link at display time if desired.

...actually the CVE/GHSA IDs are already links at HEAD, so just need to stop putting links in the refs section.


Optimized All display

As more and more browser versions become available, the screen displaying all browser versions is pretty full.
I would recommend to implement smart collapse of "all" browser versions, only showing version that have any change in support.
For example: search "calc" and browser "Chrome" would show:
4 - not supported
19 - supported with prefix webkit
26 - supported
The blocks could optionally show also ending number like 4-18, 19-25, 26-57, release date of smallest version and total usage % of the whole block.
I hope you got the idea.

Something very close to this idea has now been implemented in the "Show all" view. Thank you!


No dekstop icon saved in finished gameimage

With the latest gameimage there is no desktop icon saved in the gameimage. If this is by design how can I inject a desktop icon into the gameimage?

Hi @Blad3forc3 , thanks for reporting this issue. It is still working as before on my side, did you leave both boxes checked at the end of the wizard?

You are correct Ruan. I had the top one of the two unchecked.

Just one question on this, can you inject an icon into an already created flatimage?

Yes, you can use fim-desktop for that.

I'm trying to inject an icon into an existing flatimage i created and it errors with this..
Program exited with error: Could not determine enum entry from './CC4.PNG'

Hi Ruan,
Could you write the exact command to inject a missing desktop icon for a flatimage?

Hi @Blad3forc3 , it's /app.flatimage fim-desktop setup ./desktop.json, where the desktop.json file is:

{
  "name": "MyApp",
  "icon": "/path/to/icon.png",
  "categories": ["Game"]
}

Yes, that is the help message of flatimage versions previous to 1.0.0.

I would recommend updating if possible.


Update Workplace SSO setup instructions

The authentication settings for the Workplace app have moved. The instructions at https://docs.microsoft.com/en-us/azure/active-directory/saas-apps/workplacebyfacebook-tutorial?fbclid=IwAR0voe6ab-NFGez60-B1viLWhFIlaB3m21ppbU8iM26-FMLcGmg2SyarQIc need to be updated.
Please change steps 8 and 9 to read:
8. In the Admin Panel, go to the Security section and click on the Authentication tab
9. Under Authentication, select Single-Sign On (SSO) to reveal the detailed settings

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: changtom sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


Assignment 4 Part I: Test Set Error is Lower than the Training Set Error

In Part 1 of the Assignment I adjusted the Hyper Parameters to simplify the model, in order to not overfit the model to the training set.
My Hyper Parameters

However, in my case the test set error is lower than the training set error, rather than the other way around.
My Error Rate

In all other examples, the training set error is more accurate (lower) than the test set. Am I doing something wrong?
Other Example of Error Rate

I have same problem on both parts of the assignment. This is quite counterintuitive.

Hey guys, these area great questions and valid concerns. The curves I drew in class represent the typical case, where performance over the test set will be somewhat worse than on the training set. This makes sense because, since the algorithm is learning from the training set it is always inherently biased to perform better on that set than on one it's never seen before.
However, this is definitely not a rule, and from time to time you do see cases where you actually get better performance on the test set. This usually doesn't mean anything specific about your model setup, and does not imply poor performance. However, it is more likely to happen with smaller data sets, such as the ones we are using for demonstrations in the class. This is because any given model is more likely to 'guess right' on a small set of samples even if the model is not that good.
In the real world you would not typically use such advanced algorithms for such small sets of data, and are thus less likely to see this issue in real world cases where you might be using several orders of magnitude more data.
Hope this helps!

Yep! Thanks!


🌐 Add Japanese translation for docs/ja/docs/tutorial/sql-databases.md

Relates to https://github.com/tiangolo/fastapi/issues/1572
This PR translates tutorial/sql-databases.md.
I am Japanese and I love FastAPI,  so I translated it.
This is my first pull request for this repository, so I apologize if there are any problems. I would appreciate it if you could check it.

Thanks for your review @komtaki !
As all the comments make sense, I've fixed them in [1e7df12].(https://github.com/tiangolo/fastapi/pull/5058/commits/1e7df12e526ce4196874f7179578781b8ce1f8b7)

📝 Docs preview for commit a8022e6064cebc460641079404eff645b4f84659 at: https://649a1931d1bbbc12fe60f919--fastapi.netlify.app

As this PR had requested changes to be applied but has been inactive for a while, it's now going to be closed. But if there's anyone interested, feel free to create a new PR.


ENH: SurfaceSpatialObject templated over point type

By templating over point type, derived surface classes can specify new point types with additional meta data. This is the same design patter used for TubeSpatialObjects.
Also, updated the normal computation to work for 2D data, instead of just 3D data, and marked old code as Legacy.

This replaces pr #2672


Move internal methods to FactoryBot::Internal

We introduced FactoryBot::Internal in https://github.com/thoughtbot/factory_bot/pull/1262 so we could handle inline sequences in https://github.com/thoughtbot/factory_bot/pull/1258 without adding additional public methods on FactoryBot.
I suspect there are quite a few class methods on FactoryBot that don't need to be public, and could be deprecated and then moved to FactoryBot::Internal.

@aledustet should we close this now?


Do they really come back?

Hello unam. today when i was reading about the infamous XXS attack on the web panned, i see one weird miner adding him self to my panel and my miners started being offline. i immediately realized the situation and put my website down. then i upgraded to the newer version u provided.
I took the action immediately, but he managed to take ~90Kh/s in less than 1 minutes. My question is, i upgraded the panel and every thing is backing to the normal condition but not the miners he took. I see u mention in this issue, the miners will back on the next restart. however i use your very old miner(V 3.1.0) and i'm not sure version 3.1.0 have the feature or ability to do this. because the attacker changed the API URL and Remote URL  to his own address. Is there any chance they will back home?

Yes all versions of the miner go back to their builder configuration when they start, any configuration changes done while it's running (through either the "Remote Configuration" or web panel) are only in memory and never saved anywhere.

Yes all versions of the miner go back to their builder configuration when they start, any configuration changes done while it's running (through either the "Remote Configuration" or web panel) are only in memory and never saved anywhere.

OK thanks very much. And for information to another miners, this time when he add his miner, he done his best to cover the action. like he wasn't sent weird CPU name. instead he fill very accurate info and send some H/s. as a result every thing seems normal until i see my number of miners and recognize a new one :smiley: . take a look the screenshot

if u need to investigate more, i will share the full panel files.
Thanks. have a nice time. :+1:


Speed up Cassandra cluster boot.

By setting "auto_bootstrap" to false (only acceptable for new clusters),
multiple nodes can join in a shorter period of time. This significantly speeds up startup.

Just one comment. LGTM.

Removed the *2. Thanks.


crates.io package does not work with the example

The example cargo_completion works when cloning this repo, but when using the crates.io package it fails to compile.
To reproduce, clone this repo and make the following changes:
git diff
diff --git a/cargo_completion/Cargo.toml b/cargo_completion/Cargo.toml
index 7ee4c76..2fb70ad 100644
--- a/cargo_completion/Cargo.toml
+++ b/cargo_completion/Cargo.toml
@@ -10,7 +10,7 @@ keywords = ["cli", "completion", "autocomplete", "cargo"]
 license = "MIT OR Apache-2.0"

 [dependencies]
-shell_completion = { path = "../shell_completion" }
+shell_completion = "0.0.1"

 [[bin]]
 name = "_cargo_completion"

Building the example will generate the following error (truncated):
cargo install --force --path cargo_completion
  Installing cargo_completion v0.0.1 (/home/jk/dev/rust/shell_completion/cargo_completion)
    Updating crates.io index
   Compiling shell_completion v0.0.1
   Compiling cargo_completion v0.0.1 (/home/jk/dev/rust/shell_completion/cargo_completion)
error[E0432]: unresolved import `shell_completion::CompletionSet`
 --> cargo_completion/src/main.rs:1:62
  |
1 | use shell_completion::{BashCompletionInput, CompletionInput, CompletionSet};
  |                                                              ^^^^^^^^^^^^^
  |                                                              |
  |                                                              no `CompletionSet` in the root
  |                                                              help: a similar name exists in the module: `CompletionInput`

Am I doing something wrong, or is the crates.io package out-dated?

Thanks for the report!
I've pushed a 0.0.2 version to crates.io, and confirmed cargo build -p cargo_completion builds.


Support publishing npm module with "ts" files

I hope the TypeScript to support publishing npm module with *.ts files.
I don't mean that publishing npm module only with *.ts files, but also with pre-compiled *.js files. If a developer, using the npm module, is not using the TypeScript but pure JS, only pre-compiled *.js files would be utilized. If the developer is using the TypeScript compiler, *.ts files would be used instead of pre-compiled *.js and *.d.ts files.
Anyway, if the npm module is published with *.ts files and let users to compile those *.ts files by themselves, there would be two benefits. The first benefit is the exact-debugging and the second is the performance enhancement.
1. Exact Debugging
In nowadays, many TypeScript users are using ts-node or source-map-support to debug their TypeScript codes exactly. When error occurs, those libraries point the exact line number of *.ts file. Those library also adding breakpoint on *.ts files possible.
However, if an error comes from the external npm module, it's hard to debug because the error message would point the line number of *.js file. In such reason, adding breakpoint on external npm module is also very hard for users.

If the external npm module is using async/await feature and compiled with polyfill, debugging the external npm module without *.ts files are almost impossible.
var __generator = (this && this.__generator) || function (thisArg, body) {
    var _ = { label: 0, sent: function() { if (t[0] & 1) throw t[1]; return t[1]; }, trys: [], ops: [] }, f, y, t, g;
    return g = { next: verb(0), "throw": verb(1), "return": verb(2) }, typeof Symbol === "function" && (g[Symbol.iterator] = function() { return this; }), g;
    function verb(n) { return function (v) { return step([n, v]); }; }
    function step(op) {
        if (f) throw new TypeError("Generator is already executing.");
        while (_) try {
            if (f = 1, y && (t = op[0] & 2 ? y["return"] : op[0] ? y["throw"] || ((t = y["return"]) && t.call(y), 0) : y.next) && !(t = t.call(y, op[1])).done) return t;
            if (y = 0, t) op = [op[0] & 2, t.value];
            switch (op[0]) {
                case 0: case 1: t = op; break;
                case 4: _.label++; return { value: op[1], done: false };
                case 5: _.label++; y = op[1]; op = [0]; continue;
                case 7: op = _.ops.pop(); _.trys.pop(); continue;
                default:
                    if (!(t = _.trys, t = t.length > 0 && t[t.length - 1]) && (op[0] === 6 || op[0] === 2)) { _ = 0; continue; }
                    if (op[0] === 3 && (!t || (op[1] > t[0] && op[1] < t[3]))) { _.label = op[1]; break; }
                    if (op[0] === 6 && _.label < t[1]) { _.label = t[1]; t = op; break; }
                    if (t && _.label < t[2]) { _.label = t[2]; _.ops.push(op); break; }
                    if (t[2]) _.ops.pop();
                    _.trys.pop(); continue;
            }
            op = body.call(thisArg, _);
        } catch (e) { op = [6, e]; y = 0; } finally { f = t = 0; }
        if (op[0] & 5) throw op[1]; return { value: op[0] ? op[1] : void 0, done: true };
    }
};

2. Performance Enhancement
When developing and publishing an npm module with TypeScript, compability should be consired. In my case, because of the compability, I always compile TypeScript files as es5 target and commonjs mode with --downlevelIteration option.
However, if the TypeScript supports publishing npm module with *.ts files, such compile options can be determined by users. As you know, NodeJS is optimizing the async/await features whenever beeing updated. Therefore, when using async/await feature in NodeJS environment, es2020 target compiled *.js file is faster than es5, who does not use the async/await statements and utilizes polyfill method instead.
Like the above example case, higher version's target and mode can take the performance enhancement. Therefore, if users can determine compile options by themselves freely, the performance can be enhanced.

TypeScript does not control what you choose to publish to npm. You can publish .ts files if you wish, and require that they be built by the top-level TypeScript project. You could either have them required directly, or rely on a module setup using the tsc -b flag.

Turn on "sourceMap": true, so that when you are viewing source code or debugging, it will point you to the exact line of code in your .ts file.
You can try adding then debugging my package here:
https://www.npmjs.com/package/httyp
It's written in .ts, but compiled down to .js. However when you use debugging tools you will be redirected to the .ts file.
You can also use my npm template for typescript to test it out:
https://github.com/aelesia/npm-template


<IP_ADDRESS> - "Exception: Index was out of range" when selecting a subscription for a topic

When selecting a subscription for a topic I get the following exception:
Exception: Index was out of range. Must be non-negative and less than the size of the collection.
Parameter name: index

Regards

@vascofernandes any chance you could provide steps to build a topic that would reproduce this issue?
Also, what build of the sbexplorer are you using?

Version <IP_ADDRESS> from the screenshot

@vascofernandes given that the latest is 3.0.4, could you test with the latest and co firm of the issue is still happening? Thanks.

The topics and subscriptions are added with C# using
Assembly Microsoft.ServiceBus, Version=<IP_ADDRESS>,
ServiceBus.v1_<IP_ADDRESS>\lib\net40-full\Microsoft.ServiceBus.dll
However, with <IP_ADDRESS>, even if I delete all Entities, manually create a new topic (named Test), and try to manually add a subscription (by clicking Create Subscription), if fail with the same exception.
I did try the latest, 3.0.4.
It connects successfully but is does not show a list of topic/queues. There is no sub-tree to expand.

Are you connecting to Windows Server SB?

I installed SB on my windows 8.1 machine and did "Create a New Farm".
No problems there. All running fine.
Get-SBFarmStatus shows that everything is running fine.

I will try build locally <IP_ADDRESS> and debug it.

SB library for WSSB is much older than for Azure SB. Can't tell you what's the issue here.
You could pull the codes down and debug it. Suspect something in the client library has changed, while WSSB has not been updated.
@paolosalvatori any ideas?

The OneDrive link for <IP_ADDRESS> is dead.

Try this link for the source code of 2.1.3: https://1drv.ms/u/s!AmUKNmugpNsJrgKSItfhDqF4sx2z
Ciao
Paolo

I have the same issue in version 3.1.4
I can connect to a ServiceBus / Eventhub using a connection strng, but I cannot start a listener.

Hi
Please try the last version of the tool.
Ciao
Paolo

Hi,
I have the exact same issue on the current latest version (3.4.24) while trying to create a listener on a Azure Service Bus.
I'm using the binaries from the zip file of the release.
Is there a workaround? Something I need to install first?

Could you please send me the connection string and tell me on which entity you are trying to create the listener? You find mu email address in the about form. This week I'm pretty busy. Expect a delayed answer.

I have tested on a VM under Windows 2012 R2 and it's working...
My workstation is under Windows 10 1607 but in french whereas my VM is in english...
Weird...

this is also happening in 3.4.24 for me

connected with a azure connection string. this happens when trying to view the subscriptions.

maybe it makes sense to re-open this issue

I close the issue, because I can't repro it. When clicking a subscription I don't get any error.

I am also seeing this issue, I can run on a Win 7 Pro machine with no issues, but on my Win 7 Enterprise laptop I see this "Index was out of range" exception.  It occurs with both my production and test Service Bus connection string.  Has someone got to the bottom of this one?

Hi @WarrenKin I've no way to repro this problem. If you have Visual Studio installed on that machine, it would be great if you could debug through the application and tell me exactly which method call fails. Sorry about that.

Hi Paolo,
The CalculateLastColumnWidth is the method that crashes, it seems the object passed into the method is null:
<EMAIL_ADDRESS>This is the statement where it finally complains:
<EMAIL_ADDRESS>Please let me know what else you require….
Warren
From: Paolo Salvatori<EMAIL_ADDRESS>Sent: Mon 22 May 2017 14:40
To: paolosalvatori/ServiceBusExplorer<EMAIL_ADDRESS>Cc: Kinley Warren, Beetham<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [paolosalvatori/ServiceBusExplorer] <IP_ADDRESS> - "Exception: Index was out of range" when selecting a subscription for a topic (#48)
Hi @WarrenKinhttps://github.com/warrenkin I've no way to repro this problem. If you have Visual Studio installed on that machine, it would be great if you could debug through the application and tell me exactly which method call fails. Sorry about that.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/paolosalvatori/ServiceBusExplorer/issues/48#issuecomment-303102860, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AbhrM5Eczx6TbCE5qjm_fTpRRWoLsr7oks5r8ZAggaJpZM4JKPC9.

Hi @WarrenKin I don't see the images :(

Paolo, I've edited my previous comment and attached the VS clip.

Hi @WarrenKin in which version of the tool do you see the problem? The latest or 2.1.3? And in which class? Thanks! HandleQueueControl or HandleSubscriptionControl? Thanks!

The class is HandleSubscriptionControl and the version I am using is 3.0.4.

Can you please check the latest version? Thanks!

Do you mean this:

I don't recognize that view. I mean the latest version of the code or the latest release you can download from GitHub .

I see what you mean now, I have downloaded the latest code from GitHub and the issue is now fixed, thank you so much.  I thought I had the latest version 3.0.4, I'll have to check where I found this download because I am sure it said it was the latest and it was only about 1 month ago, and I see you're now at 4.0.89. Thanks again - it's a brilliant tool.

Thanks @WarrenKin :)


Ignore auto generated files

What do you think about ignore the generated files like docs, analysis.json, etc?
Easier to see the PRs.

It would be a nice improvement if these auto-generated files do not appear in PRs.
Currently they are part of the repo because its GitHub pages are set up to render the content of the /docs folder.
If there is any way to keep GitHub pages up-to-date but exclude the autogenerated files from PR reviews, let's do that.
One idea how to do that was to use the gh-pages branch (https://github.com/vaadin/vaadin-router/issues/95) instead of /docs. That would require some build scripting magic to make sure the yarn docs command still works.


git-vanity-hash: fix build and execution

Build has been broken in Hydra: https://hydra.nixos.org/build/256865980/nixlog/2.
Potential cause seems to be an old dependency, with a possible workaround sent upstream.
Description of changes
I left a PR to switch dependencies and work around the issue in https://github.com/prasmussen/git-vanity-hash/pull/2.
Things done
~Marking as broken with comments documenting the issue.~
Change the fetched rev of the package to pick up the upstreamed solution.

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.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
[x] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Thanks for the review @SuperSandro2000!


Change module name to dreamsheets

This change is a part of productising this repo, and includes an update
to the README.
Change-type: major
Signed-off-by: Lucian Buzzo<EMAIL_ADDRESS>

@balena-ci

@balena-ci I self certify!

@balena-ci I self certify!

@somombo This catches me out all the time, self-certify fails with an error if you're not the original author. You can approve in the top right though

@balena-ci I self-certify!