Wrong version on npm?

This is strange:
$ npm install --save messageformat
$ cat node_modules/messageformat/package.json | grep version
  "version": "0.2.1",
$ cat node_modules/messageformat/messageformat.js | grep @version
 * @version 0.1.7

Version 0.2.1 of the npm package looks like it actually contains version 0.1.7 of messageformat.js.

Probably my inability to use build tools correctly. I'll push something out
asap.
On Sat, Mar 7, 2015, 2:35 PM Michael Jackson<EMAIL_ADDRESS>wrote:

This is strange:
$ npm install --save messageformat
$ cat node_modules/messageformat/package.json | grep version
"version": "0.2.1",
$ cat node_modules/messageformat/messageformat.js | grep @version

@version 0.1.7

Version 0.2.1 of the npm package looks like it actually contains version
0.1.7 of messageformat.js.
—
Reply to this email directly or view it on GitHub
https://github.com/SlexAxton/messageformat.js/issues/90.

Thanks @SlexAxton !

As an update to this issue, since it's been way too long:

The 0.2.1 file contains the 0.2.1 library, but with the wrong version in the header comment. So this isn't as sad as I thought, luckily.
We've got a 3.0.0-0 tag out now that we're not quite ready to publish without docs, but you can install it via tag.
The newest versions make this problem much harder to occur in the future.

I'll see how easy it is for me to check out this old tag and push the comment change as 0.2.2... but just wanted to update this thread for completeness sake.

Went ahead and released an 0.2.2 for good measure - but it has no functionality changes. Thanks!


UX suggestion for netwrok filtering

Hey!
I was very confused testing QF when I tried to filter projects. I looked for projects with a wallet on Mainnet, but all disappear. So I kept adding chain, but they were not visible.
@laurenluz explained that filtering works with an AND condition. However, it was very confusing for me from the donor journey perspective. I think it's more common to chain by chain to see where can we donate. This would mean have an AND/OR condition.

@maryjaf  We deployed it you can test it now

Now when you select multiple networks we use OR condition instead of AND so if a project has a recipient address in one of those chains we will return it
In Staging ENV if you choose
Mainnet we will assume it as Goerli
Celo -->> CELO_ALFAJORES
Optimism -->> Optimism-Goerli

Thanks @mohammadranjbarz I tested and every thing is based on our expectation


Improve missing block finder & retry queue manager

Changes

Missing block finder previously used to query DB for all blocks
Now it attempts to query blocks by range ( batch request ) & processes only missing ones
Retry queue manager only attempts to process missing block for every 3rd request, for a certain block number
👆 potentially reduces #-of RPC calls being made to full node for block, which is probably still not available in node
For implementing above one Redis backed hash table is being used & wrapped attempt count being stored there

Huge performance improvement 🚀
Before  2.6G memory consumption with ~99% CPU usage 👇

After 15M memory consumed with very normal CPU burst 👇

Huge performance improvement 🚀
Before  2.6G memory consumption with ~99% CPU usage 👇

After 15M memory consumed with very normal CPU burst 👇

Above two instances are running on two different machines 😉

Above two instances are running on two different machines 😉

While subscribing to topics i.e. when certain tx between account/ from account/ to account happens or certain event of interest happens, client can provide with addresses in non-checksum format and case insensitive matching to be performed, while checking whether data can be delivered to client or not.

While subscribing to topics i.e. when certain tx between account/ from account/ to account happens or certain event of interest happens, client can provide with addresses in non-checksum format and case insensitive matching to be performed, while checking whether data can be delivered to client or not.


KBFS: Unable to open on Finder due to permission errors

I am seeing an odd one here. I see this error when I open Keybase on Mac's Finder app:
The folder “Keybase (arunsathiya)” can’t be opened because you don’t have permission to see its contents.
I don't remember this happening a few months ago. I did not open Keybase on Finder over the past few months. I also updated to Mojave during this time. Did that cause an issue?
my log id: 00cbae65e06e4e3cee1b2e1c

@arunsathiya it seems like your computer is having resource issues.  Keybase is getting errors like this from the operating system: "too many open files".  This starts immediately when our process starts up, so I don't think it's something Keybase is doing; seems like at the time, your operating system was out of file descriptors.
If it's still happening, please update to the latest Keybase release (released Friday) just in case it helps.

Can I ask where I can download the new version, please? I am not seeing the option to fetch the new version from the app menu or on the Mac menu at the top. 🤔 Thanks.

You can run keybase update check.

Looks like I am on the latest version, and Finder opens just fine now. I shall close this issue. 👍🏽 Thank you for your help.

I am having this problem on OSX 10.13.6 with latest Keybase
Version 5.6.2-20210202191343+d72cc00cd3 (5.6.2-20210202191343+d72cc00cd3)
I check the "I understand that a closed-source kernel extension (FUSE for macOS) will be installed.
I click "Yes, enable" and get a spinning circle, which does complete. But the message  and Yes, Enable never goes away.
are there debug logs I Can provide?

I am having this problem on OSX 10.13.6 with latest Keybase
Version 5.6.2-20210202191343+d72cc00cd3 (5.6.2-20210202191343+d72cc00cd3)
I check the "I understand that a closed-source kernel extension (FUSE for macOS) will be installed.
I click "Yes, enable" and get a spinning circle, which does complete. But the message and Yes, Enable never goes away.
are there debug logs I Can provide?

I’m having the same exact problem on the same OSX version.
Tried the tips&tricks posted before but non of them worked.
Have you found a solution yet @MattWeatherford?

I noticed somewhere in the installation logs a lot that read "Installed for
Mac OS 10.15".  I think the latest version may just not be compatible.
On Feb 12, 2021 5:37 PM, "jiy0"<EMAIL_ADDRESS>wrote:

I am having this problem on OSX 10.13.6 with latest Keybase
Version 5.6.2-20210202191343+d72cc00cd3 (5.6.2-20210202191343+d72cc00cd3)
I check the "I understand that a closed-source kernel extension (FUSE for
macOS) will be installed.
I click "Yes, enable" and get a spinning circle, which does complete. But
the message and Yes, Enable never goes away.
are there debug logs I Can provide?
I’m having the same exact problem on the same OSX version.
Tried the tips&tricks posted before but non of them worked.
Have you found a solution yet @MattWeatherford
https://github.com/MattWeatherford?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/keybase/client/issues/16512#issuecomment-778540731,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJ2XZ4SX7TTDZPXVUQOT5DS6XJT7ANCNFSM4G43MQHQ
.

I noticed somewhere in the installation logs a lot that read "Installed for Mac OS 10.15". I think the latest version may just not be compatible.
…

Thanks Bryan. Do you if there’s still a dmg file for older version available that’s compatible with my OSX version?

I haven't found one.  You can find old versions to compile from source but
haven't had time to try yet.
On Feb 14, 2021 6:52 PM, "jiy0"<EMAIL_ADDRESS>wrote:

I noticed somewhere in the installation logs a lot that read "Installed
for Mac OS 10.15". I think the latest version may just not be compatible.
… <#m_-7831580488143635794_>
Thanks Bryan. Do you if there’s still a dmg file for older version
available that’s compatible with my OSX version?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/keybase/client/issues/16512#issuecomment-778905612,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJ2XZZ24KYNPWEFU7NSUETS7CD6DANCNFSM4G43MQHQ
.

I keep seeing this error after each time the Mac OS updates

I'm having the same issue - it says i don't have permission to view the keybase folder content. Is there any way around this?


Get GitHub tag if the release is empty.

for example, nginx didn't use GitHub release, https://api.github.com/repos/nginx/nginx/releases is empty but we can get the tags from https://api.github.com/repos/nginx/nginx/tags

Thanks for this @travislee8964 , nice one!
Do you think it might be worth making this an explicit choice?
Perhaps we could add a boolean use_tags variable to the config of the GitHub project in the Yaml.
What do you think?

Yeah, it's better to let the user choose whether to enable it or not, I will submit  patch.


get_dicom_or_nifti_paths_and_num naming

Hi,
I just tried to run Comp2Comp and I experienced the following error:
from comp2comp.io.io_utils import get_dicom_nifti_paths_and_num
ImportError: cannot import name 'get_dicom_nifti_paths_and_num' from 'comp2comp.io.io_utils' (/home/jakob/dev/Comp2Comp/comp2comp/io/io_utils.py)

I fixed it by renaming get_dicom_nifti_paths_and_num to get_dicom_or_nifti_paths_and_num.

Hi Jakob,
First of all, thanks for all your work on TotalSegmentator. Please let me know if you have any ideas for Comp2Comp, which as you know, builds on TotalSegmentator.
Sorry for the error! I think we had different names in different branches that we merged in yesterday and that caused the issue. We need to add some unit tests to make sure this kind of thing doesn't happen.
Best regards,
Louis

I tried the contrast_phase classifier and it worked great for a few cases I tried it on. I thought about building something similar, but never found time to do so.
The runtime was quite long for me. I think there could be potential to reduce it. Probably a lower number of radiomics features would also work fine without reducing Accuracy too much. You could check which ones are slow and see if you can remove those. I always use pyradiomics for calculating features. Maybe that is faster. But I did not check that.

Connected @edreisMD who developed this pipeline

Thank you Louis. Hi @wasserth. Glad it was useful for you, thank you so much for the thoughts. We just fixed a bug that was happening in some cases, but this shouldn't reduce the runtime yet.


vivox7颜色显示不对

@Yuriko-wbh 描述需要更具体点, 代码+截图更好

I'm going to close this issue, if this issue is still present, please feel free to reopen it.


Add Lychee Link Checker

This Pull Request is going to implement the Lychee Broken Link Checker action
The action is configured to run on each push and scheduled every seven days.
It will check all links in the README.md and automatically creates issues when it finds broken links.
Please feel free to make any changes to the PR so that it fits your needs.

Thanks a lot for your contribution @tuxinaut :)

@tuxinaut it looks like the Action is failing, can you take a look and send a follow-up PR with the update?

:sob: I will try to have a look at it today.

@dastergon this should hopefully fix the issues  #102


Expand more referenced records in hover when it makes sense?

This is just an idea, but something I find myself struggling with in both ReScript and in TS is whenever a record that's important for something I hover is not expanded, but just referenced by its name. Here's an example:

This shows hovering a variable that contains a function. That function is wrapped in React's callback mechanism, and the first level of the record is expanded (React.callback). But, in this (and quite a few other scenarios), it'd be really helpful if one additional level of records could be expanded. In this particular case, this would mean that Confirm.confirmItem would be expanded.
To be clear of what I mean, this is what I'd like to see:
React.callback<Confirm.confirmItem, unit>

type callback<'input, 'output> = 'input => 'output
type Confirm.confirmItem = {
  text: string,
  callback: unit => unit
}

I realize this might be difficult because where do you stop, how many levels deep do you go, and where do you draw the line given the fact that there could be a large number of records referenced in a single hover. But, maybe there's a common ground for cases that are quite common, like the one I describe?

Let's close it for now -- there are quite a few other issues coming up these days that are uncontroversial.


Issue 116 and 250

This addresses issues #116 and #250. Principle work was done by @MohammedElsayyed and offered up in PR #215. I've just given it a bit of polish and returned the RemoteLiveWebCache class for backwards compatibility.

@kris-sigur Thanks for thinking about backward compatibility. Good point.


Cannot read property 'trim' of undefined

https://github.com/sebhildebrandt/systeminformation/blob/master/lib/network.js#L220
https://sentry.iobroker.net/share/issue/b5f7dc58f42c4d3392d0b9da0fef9f85/
TypeError: Cannot read property 'trim' of undefined
  File "D:\iobroker\Dieter\node_modules\iobroker.info\node_modules\systeminformation\lib\network.js", line 220, col 49, in parseLinesWindowsNics
    let linesNicConfig = nconfigsections[i].trim().split('\r\n');
  File "D:\iobroker\Dieter\node_modules\iobroker.info\node_modules\systeminformation\lib\network.js", line 259, col 19, in null.<anonymous>
    resolve(parseLinesWindowsNics(nsections, nconfigsections));
  ?, in runMicrotasks
  File "internal/process/task_queues.js", line 95, col 5, in processTicksAndRejections

@Apollon77 ... problem found and solved. Version 5.11.11 just released.

Thank you!


classHandler and errorsContainer as functions does not work correctly

http://parsleyjs.org/doc/annotated-source/defaults.html#section-16
http://parsleyjs.org/doc/annotated-source/defaults.html#section-18
After 3538ae49b6e8888da2f2588035da2ab47c58620a.
Please, go back

this.options.errorsContainer(this) or
this.options.errorsContainer.call(this) and update annotated-source.

Oups :bomb: sorry for this.
Fixed & released.
Thanks!


Currences

On the store I set up the currency - EUR.
But in the Cart the price was in RUB. For example 1000 RUB (which means 14 EUR).
Totally at the https://commerce.coinbase.com/charges/ the price was 14 RUB.

Hi @kabetov, are you still seeing this issue? Can you provide some more details?

Hello @guacamoli Yes, please https://docs.google.com/document/d/1rcYSv3QROikCbDjRMMuhyR-N0_eovUunognKx_-H1gI/edit?usp=sharing

@kabetov, thanks again for reporting the issue and providing details! We've pushed a fix for the issue and will be updating the plugin soon. Thanks!

@guacamoli please inform when I can test it)

Hello @kabetov we just published a new release with the fix: https://github.com/coinbase/coinbase-commerce-opencart/releases/tag/v1.0.1


Core Dump just after mining a block

Is there an existing issue for this?

[X] I have searched the existing issues

Current behaviour
while using SRI pool+jds+jcc+tproxy from git dcaa92dbed49976a5e174941903cf799c68d8eb6, just after finding the block 38172, got a Core Dump crash
Expected behaviour
not crashing
Steps to reproduce
cd bitcoin-sv2-tp-0.1.4
./bin/bitcoind -testnet4 -sv2 -sv2port=8442 -debug=sv2 -conf=/data/bitcoin-sv2/bitcoin.conf -datadir=/data/bitcoin-sv2

with /data/bitcoin-sv2/bitcoin.conf :
server=1
[main]
[test]
[testnet4]
rpcbind=<IP_ADDRESS>
rpcbind=::1
rpcbind=<IP_ADDRESS>
rpcallowip=<IP_ADDRESS>
rpcpassword=****
rpcuser=****
[signet]
[regtest]

Relevant log output
2024-08-06T00:46:49Z Flushed fee estimates to fee_estimates.dat.
2024-08-06T00:47:41Z Saw new header hash=00000000f7298f2400c3514fa2a68a68a77ead56cb15064bfbd85130998987e0 height=38171
2024-08-06T00:47:41Z Saw new cmpctblock header hash=00000000f7298f2400c3514fa2a68a68a77ead56cb15064bfbd85130998987e0 peer=7
2024-08-06T00:47:41Z UpdateTip: new best=00000000f7298f2400c3514fa2a68a68a77ead56cb15064bfbd85130998987e0 height=38171 version=0x20000000 log2_work=70.407958 tx=747408 date='2024-08-06T01:02:07Z' progress=1.000000 cache=0.6MiB(2642txo)
2024-08-06T00:47:41Z CreateNewBlock(): block weight: 6344 txs: 3 fees: 1384 sigops 410
2024-08-06T00:47:41Z [sv2] Send 0x71 NewTemplate id=1130 to client id=1
2024-08-06T00:47:41Z [sv2] Send 0x72 SetNewPrevHash to client id=1
2024-08-06T00:47:41Z CreateNewBlock(): block weight: 6344 txs: 3 fees: 1384 sigops 410
2024-08-06T00:47:41Z [sv2] Send 0x71 NewTemplate id=1131 to client id=2
2024-08-06T00:47:41Z [sv2] Send 0x72 SetNewPrevHash to client id=2
2024-08-06T00:47:41Z [sv2] Received 0x73 RequestTransactionData from client id=2
2024-08-06T00:47:41Z [sv2] Send 0x74 RequestTransactionData.Success to client id=2
2024-08-06T00:47:59Z New block-relay-only v1 peer connected: version: 70015, blocks=38171, peer=704
2024-08-06T00:48:53Z [sv2] Received 0x76 SubmitSolution from client id=2
2024-08-06T00:48:53Z Saw new header hash=000000000000005aad1da1c2646a65e34c94089d7a51a9fe1ae3118169f18c82 height=38172
2024-08-06T00:48:53Z UpdateTip: new best=000000000000005aad1da1c2646a65e34c94089d7a51a9fe1ae3118169f18c82 height=38172 version=0x26b4c000 log2_work=70.408134 tx=747412 date='2024-08-06T00:47:41Z' progress=1.000000 cache=0.6MiB(2644txo)
2024-08-06T00:48:53Z CreateNewBlock(): block weight: 5059 txs: 3 fees: 1063 sigops 405
2024-08-06T00:48:53Z [sv2] Send 0x71 NewTemplate id=1132 to client id=1
2024-08-06T00:48:53Z [sv2] Send 0x72 SetNewPrevHash to client id=1
2024-08-06T00:48:53Z [cormorant] MarkUnusedAddresses: Detected a used keypool item at index 0, mark all keypool items up to this item as used
2024-08-06T00:48:53Z CreateNewBlock(): block weight: 5059 txs: 3 fees: 1063 sigops 405
2024-08-06T00:48:53Z [sv2] Send 0x71 NewTemplate id=1133 to client id=2
2024-08-06T00:48:53Z [sv2] Send 0x72 SetNewPrevHash to client id=2
2024-08-06T00:48:53Z [cormorant] AddToWallet 2936ea56ed56d5e561de4f05ad8da99dba5f52cbcc905ec3452e68704fb3519e  new Confirmed (block=000000000000005aad1da1c2646a65e34c94089d7a51a9fe1ae3118169f18c82, height=38172, index=0)
2024-08-06T00:48:53Z [sv2] Received 0x73 RequestTransactionData from client id=2
2024-08-06T00:48:53Z [sv2] Send 0x74 RequestTransactionData.Success to client id=2
2024-08-06T06:22:07Z Bitcoin Core version v27.99.0-g44c8fa05bb16e1f7a00c8787703ef183a4b7cf51 (release build)
2024-08-06T06:22:07Z Script verification uses 3 additional threads
2024-08-06T06:22:07Z Using the 'sse4(1way),sse41(4way),avx2(8way)' SHA256 implementation
2024-08-06T06:22:07Z Using RdSeed as an additional entropy source
2024-08-06T06:22:07Z Using RdRand as an additional entropy source
How did you obtain Bitcoin Core
Pre-built binaries
What version of Bitcoin Core are you using?
sv2-tp-0.1.4 => v27.99.0-g44c8fa05bb16e1f7a00c8787703ef183a4b7cf51
Operating system and version
Ubuntu 23.10
Machine specifications
No response

Can you add -loglevel=sv2:trace to the configuration? That'll give me a more precise view of the last thing that happened. Also -logtimemicros will give you more accurate timestamps.
In your log I'm seeing RequestTransactionData.Success as the last thing before the crash. But it's not clear if the crash happened there or a bit later.

Yes I can and I will, but reproducing takes me multiple days... Hitting a block even on T4 is not immediate;)

I can confirm with sv2-tp-0.1.9 , I mined a block this night without crashing, so looking at the changelog, you have fixed this issue in 0.1.9.

Thanks for testing and contrast on the testnet4 block!


Question: How to specify custom set of proteins

Hi,
A couple of questions as a new user:
I'd like to run this from the command line non-interactively, when using the docker version is there any way to run it without it pausing on this expecting user input? some equivilent of a "/Y" or "-Y" option
In order to search Thermo .raw files, you must agree to the above terms. Do you agree to the above terms? y/n
My main query is about how to input custom protein sequences to search:
I've got a list of amino acid sequences and I am trying to use MetaMorpheus to search mas-spec data for matches.
I'm trying to use the MetaMorpheusVignette-selected-examples as a guide on how to input data.
Do I understand correctly that the uniprot-cRAP-1-24-2018.xml.gz is for potential contaminants?
And that the uniprot-mouse-reviewed-1-24-2018.xml.gz is for the sequences you're actually interested in searching for?
I've attempted to make my own equivalent of the "reviewed" xml file using this format but with multiple  blocks:
<?xml version='1.0' encoding='UTF-8'?>
<uniprot
	xmlns="http://uniprot.org/uniprot"
	xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://uniprot.org/uniprot http://www.uniprot.org/support/docs/uniprot.xsd">
	<entry dataset="Custom-1">
		<name>ABC.123</name>
		<protein>
			<recommendedName>
				<fullName evidence="5">ABC.123</fullName>
			</recommendedName>
		</protein>
		<sequence length="42" mass="4705">
MTPRSALPKQFRVHFGTQLGTEQSNVWFGIPSGVFFRISALV
</sequence>
	</entry>
</uniprot>

Running it against the tutorial example data it seems to run without complaint or error but it seems to run too fast, even with a few hundred sequences it blinks through each stage of the search fast enough that I don't think it's actually searching and I don't know how to check what sequences MetaMorpheus has successfully loaded.

yes you can modify your installation so that it doesn't pause to ask about the thermo license. You need to find the folder where MetaMorpheus is installed. That directory will contain the following folders and files.

You will need to edit the settings.toml in a text editor.  You can use that or not as you see fit.
To confirm that metamorpheus is using your sequences, open the AutoGeneratedManuscriptProse.txt file in the Task-1SearchTask folder. This file will tell you which file was used as the sequence file and how many sequences were used. In your case, do a search with only your list of sequences and see if this document matches your expectations.

Also, feel free to send me your sequence file and i will test it myself<EMAIL_ADDRESS>

Unfortunately I've still had no luck with this.
I've tried to create a minimum file with a single entry but the result files just seems more confusing.
<?xml version='1.0' encoding='UTF-8'?>
<uniprot xmlns="http://uniprot.org/uniprot" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://uniprot.org/uniprot http://www.uniprot.org/support/docs/uniprot.xsd">

<entry dataset="Swiss-Prot">

<name>ZSWM2_MOUSE</name>
<protein>
<recommendedName>
<fullName>E3 ubiquitin-protein ligase Zswim2</fullName>
</recommendedName>
</protein>
<gene>
<name type="primary">Zswim2</name>
</gene>

<sequence length="631" mass="71793" >
MLRGGCKASEKRRHLSESLSWQQDQALSSSIYLLRQIGPTGFLLKEEEPEKGDFRVLLGN
PHECSCPTFLKRGELCKHICWVLLKKFKLPRNHESAFQLGLTEGEINDLLRGIHQVQAPQ
LRASDETAQVEEDGYLKQKDINAGDICPICQEVLLEKKLPVTFCRFGCGNNVHIKCMRIL
ANYQDTGSDSSVLRCPLCREEFAPLKVILEEFKNSNKLITISEKERLDKHLGIPCNNCNQ
LPIEGRCYKCTECVEYHLCQECFDSCCHSSHAFASREKRNQRWRSVEKRSEVMKYLNTEN
EGEAKPGCFQEKQGQFYTPKHVVKSLPLLMITKKSKLLAPGYQCRLCLKSFSFGQYTRLL
PCTHKFHRKCIDNWLLHKCNSCPIDRQVIYNPLIWKGIATDGQAHQLASSKDIACLSKQQ
EPKLFIPGTGLVLKGKRMGVLPSIPQYNSKVLTTLQNPSDNYQNITMDDLCSVKLDNSNS
RKLVFGYKISKQFPTYLKNPTTGQTPSQTFLPSLPHKNIICLTGRESPHIYEKDHIGQSQ
KTSRGYEHINYNTRKSLGSRLRQHKRSSALSSEDLNLTINLGTTKLSLSKRQNNSMGKVR
QKLGHPPRRPAYPPLQTQNAALSLIMQGIQL
</sequence>

</entry>

<entry dataset="Swiss-Prot">

<name>ZSWM2_MOUSEXX</name>
<protein>
<recommendedName>
<fullName>E3 ubiquitin-protein ligase Zswim2XX</fullName>
</recommendedName>
</protein>
<gene>
<name type="primary">Zswim2XX</name>
</gene>

<sequence length="87" mass="8872" >
MDVFMKGLSKAKEGVVAAAEKTKQGVAEAAGKTKEGVLYVGSKTKEGVVHGVATVAEKTK
EQVTNVGGAVLTDVPSCTSAQFQCAQS
</sequence>
</entry>

</uniprot>

the log shows it seems to read in the 1 sequence but then reverts to zero, I can't seem to get it to recognise the second sequence at all:


FeaturesToGPX Toolbox Not Loading in Arc

Dear Kevin,
Thanks so much for creating this script!  It's such a great idea and will make it so much easier to do this in Arc.  I'm having trouble getting FeaturesToGPS to show up on my Arc ToolBox and might have missed something.  I downloaded your python script and and .tbx files and added them to the ArcGIS "Toolboxes" program folder but when I import the toolbox, the tool isn't showing up in a usable way.  If it's easier to explain over email, you can find me at sms493 at cornell dot edu.
Thanks again!
Sheila

Sorry, I am just seeing this now. To use the tool, you have 2 options. You can deploy them with the setup, which will make a new toolbox inside your ArcToolbox. Alternatively, if you simply download the zip file and extract, you can navigate using the Catalog window to that location and run the tool from there. Either the samples.tbx or go into the FeaturesToGPX folder and run that specific tool.

I cannot seem to load the FeaturesToGPX tool, or rather the part I was looking for.
As I understood the documentation, you can convert a line/point shape to a GPX file, which is what I was looking for.
However, when I load the tool into Arc, I can only load GPX files into Arc, which is useless to me.
This is what I see when I open the Add-In Controls tool to drag the toolset to my toolbar, just "GPX Import":

Where is the FeatureToGPX that I was looking for?

@GeoCanuck this is a tool, not an add in. As such just navigate to the folder you unzipped to in the Contents window and open the toolbox.  Or open the ArcToolbox, right click and add which will allow you to navigate to the toolbox.

I have never added tools before, so am not sure how it is done.
I just followed installation instructions, but now cannot find them to check.
However, all I got was the wrong tool. Even though the downloaded zipfile contents say FeaturestoGPX, I just seem to get GPXToFeatures


Modals have incorrect bottom padding with empty modal actions container

Describe the bug

How to reproduce
Steps to reproduce the behavior:

Go to https://codesandbox.io/s/cds-modal-actions-padding-b1fl7t
You will need to open the preview in a dedicated window. Click "Open in New Window"
Open modal 1 to see the correct padding, modal 2 for the incorrect padding

Expected behavior
Both modal 1 and 2 should look identical.
Versions
Clarity project:

[x] Clarity Core
[ ] Clarity Angular/UI

Clarity version:

[ ] v5.x
[x] v6.x

Framework:

[ ] Angular
[x] React
[ ] Vue
[ ] Other:

Framework version:
React 18
Additional notes
Why rendering an empty CdsModalActions in the first place?
We need empty actions because we display the action button conditionally. We initially wrapped the conditional around the entire container (instead of just the button) as a workaround, but another issue kicked in: when the modal is opened and the actions container is meant to render later, it simply... does not appear.
I opened https://github.com/vmware-clarity/core/issues/188 to report this behavior as well.

cds-modal-actions is an html element. If instead of a component, you were to render

 you wouldn't expect it to behave any differently than it does now. Yes, you're going to get a gap between the body and the actions when you include the cds-modal-actions element, just like if you used a div. They're both HTML Elements.
Looping @colinreedmiller in to chime in, but I don't think this is something we'll fix (I will try and fix the linked issue, however).


Code clean up/pep8 clean up

Description
This Branch is just a small clean up of the DS code base to meet PEP8 standards on most .py files.
Changes made were contained in the /app and  /data_generators folders.
Added TODO calls for bug Fix branches in the future.
Fixes # (None)
Loom Link
Type of change
Please delete options that are not relevant.

[x] Bug fix
[ ] New feature
[ ] This change requires a documentation update

Checklist:

[x] My code follows PEP8 style guide
[x] I have removed unnecessary print statements from my code
[x] I have made corresponding changes to the documentation if necessary
[x] My changes generate no errors
[x] No commented-out code
[x] Size of pull request kept to a minimum
[x] Pull request description clearly describes changes made & motivations for said changes

This is the most beautifully-written and legible code I've ever seen; excellent job

Blessed be the PEP8 rules.

The code looks cleaner and more readable. Great job

The code follows PEP8 guidelines and is definitely easier to read. Fantastic work!

Nice clean code!


Key Pair are visible for client

Hi AWS,
The affected JavaScript file contains sensitive information like client secrets that should not be placed in client-side code as they are discoverable without any authentication.
Can you please fix it ? https://aws.amazon.com/solutions/implementations/voicemail-for-amazon-connect/

The file name(s) https://github.com/amazon-connect/voicemail-for-amazon-connect
Where the API Keys are exposed in the output of the cloudformation call.

The file is in the portal bucket  that is sharing the Key Pair
under the folder static/js/

Hi there!
The key you are seeing is the API Key for API Gateway and not used for authentication. The solution does not store any secrets client-side and uses Cognito for all authentication. The authentication is done through the API Gateway Lambda authorizer found here.
The API Gateway Key is there so the solution can follow the best practice of implementing  a Usage Plan (set initially to rate limit the API requests, 50/second and a burst of 500), and they are generated by APIG as part of the deployment as per best practices.
We wouldn't recommend that you remove the APIG Key, but since this an open source project, you do have the option of modifying it for your own needs. If you wish to do that, we recommend using the development branch which has the latest changes, and has a simplified build process as noted here.


feat(searchpost): update pattern search same company name for post endpoint

Description
adjust existing endpoints where company name is getting inserted with the same pattern as used for search of company names.
Changes occur in below endpoints:
POST: api/administration/invitation
GET: api/administration/registration/applicationsWithStatus?page=0&size=15
GET: api/administration/registration/applications?page=0&size=4
POST: api/administration/registration/network/partnerRegistration
POST: api/registration/application/{applicationId}/companyDetailsWithAddress
Why
Pattern harmonization across all endpoints where the company name is entered.
Issue
N/A Ref: CPLP-3564
N/A Ref: CPLP-3562
Checklist

[x] I have followed the contributing guidelines
[x] I have performed a self-review of my own code
[x] I have successfully tested my changes locally
[x] I have added tests that prove my changes work
[x] I have checked that new and existing tests pass locally with my changes
[x] I have commented my code, particularly in hard-to-understand areas

Besides the endpoints you adjusted I found two additional endpoints.
Please add the validation there as well:
POST: api/administration/registration/network/partnerRegistration Post: /api/registration/application/{applicationId}/companyDetailsWithAddress

Changes done

Frontend PR was already merged:
https://github.com/eclipse-tractusx/portal-frontend/pull/403

after a short discussion with julia there were some app / service endpoints which came up:
POST: /api/services/servicerelease/addservice PUT: /api/apps/appreleaseprocess/15507472-dfdc-4885-b165-8d4a8970a3e2 POST: /api/apps/appreleaseprocess/createapp POST: /api/services/servicerelease/addservice
please add the validation for thoses endpoints as well. the field which should be validated here is the provider

changes done

@AnuragNagpure please add the adjusted endpoints to the ticket in jira as well.


Add space to between stream name and down arrow

The distance between the stream name and the down arrow in the compose box dropdown looks too small. We should try increasing it.
Note that the compose box redesign in #24878 proposes a different arrow icon. However, the down arrow matches the down arrow for settings dropdowns, so we probably want to hold off on updating the design for now.

@zulipbot claim


[Service Bus] peekMessages to have maxMessageCount mandatory

This PR updates the peekMessages() method to take maxMessageCount as a mandatory parameter based on the concusion in the discussions in #9742

/azp run js - servicebus - tests

/azp run js - servicebus - tests

/azp run js - servicebus - tests

Changelog? Not changelog worthy?

Made the below changes:

Changelog updated in https://github.com/Azure/azure-sdk-for-js/pull/9798/commits/28fd4c0ee611f049709735402ed2f66c882f52cd
If no maxMessageCount is given, default to 1 in https://github.com/Azure/azure-sdk-for-js/commit/e6dd038f769835420110138b3bb64aa9018b4d2d


using both of "collection" and "schema" attribute for autoform

I have one "Ads" collection in mongoDB, and too many schema in following format:
AdsBaseSchema = new SimpleSchema({
  _parentId: {
    type: String,
    optional: true
  },
  title: {
    type: String,
    label: "Title",
    max: 200
  }
  description: {
    type: String,
    label: "Description",
    optional: true
  }
});

but they are different is some fields.
and I want to use them for insert autoform. like this:
{{> quickForm schema="AdsBaseSchema" id="insBaseAds" type="method" meteormethod="insBaseAds"}}

whit this method:
insBaseAds: function(doc) {
    Ads.insert(doc);
    this.unblock();
}

this approach works correctly!
but its hard for me to use this approach for all of my schema!
I want to ask you: "is this possible for meteor to use something like below autoform, and use just one autoform (form generator) for all schema?"
{{> quickForm collection="Ads" schema="AdsBaseSchema" id="insertAds" type="insert"}}

You could try writing a helper for your template which returns a schema name dynamically, like so:
{{> quickForm collection="Ads" schema=schema id="insertAds" type="insert"}}
Template['myTemplate'].helpers({  schema() {  //Write your logic here  return "adsBaseSchema";  } })

thanks. it works


Bicep --what-if changes always detected on existing site config

Bicep version
0.6.1
I would like to implement in pipeline dynamic check of --what-if changes on bicep deployment. The problem is there are couple places that bicep is always detecting as change:
Result of what-if:
  ~ Microsoft.Web/sites/xyz [2021-03-01]
    + properties.siteConfig.localMySqlEnabled:   false
    + properties.siteConfig.netFrameworkVersion: "v4.6"

  ~ Microsoft.Web/sites/xyz [2021-03-01]
    + properties.metadata: [
        0:

          name:  "CURRENT_STACK"
          value: "dotnetcore"

      ]

I tried to put netFrameworkVersion and localMySqlEnabled to both Microsoft.Web/sites (siteConfig section) and Microsoft.Web/sites/config separately/at same time,  it always detects these configs as a change.
About CURRENT_STACK=dotnetcore, its necessary to put becase otherwise it's creating App Service with null Stack Settings. That setting is also always detected as a change.
Please for ideas and help.
Thanks.

Hello, any update on this? Thanks!
Reproduction: https://github.com/Azure/azure-cli/issues/24163

In our case we always have this modification:
  ~ Microsoft.Web/sites/***************** [2022-09-01]
    + properties.siteConfig.localMySqlEnabled:     false
    + properties.siteConfig.minTlsVersion:         "1.2"
    + properties.siteConfig.netFrameworkVersion:   "4.6"
    + properties.siteConfig.use32BitWorkerProcess: false

It does not matter that those properties are explicitly defined in the template with those values (in an attempt to find a workaround).

I'm getting something very similar when using Pulumi. I suspect the same root cause.
Partial output from a pulumi preview --diff:
~ siteConfig: {
    + localMySqlEnabled  : false
    + netFrameworkVersion: "v4.6"
  }

I am also seeing this issue. This is my particular output and explicitly defining all options doesn't resolve the problem.
  ~ Microsoft.Web/sites/********************* [2024-04-01]
    + properties.siteConfig.ftpsState:                              "Disabled"
    + properties.siteConfig.ipSecurityRestrictions: [
        0:

          action:    "Allow"
          ipAddress: "LogicApps.AustraliaEast"
          name:      "LogicAppsAustraliaEastTraffic"
          priority:  100
          tag:       "ServiceTag"

        1:

          action:    "Deny"
          ipAddress: "<IP_ADDRESS>/0"
          name:      "DenyAllOtherTraffic"
          priority:  200

      ]
    + properties.siteConfig.localMySqlEnabled:                      false
    + properties.siteConfig.minTlsVersion:                          "1.2"
    + properties.siteConfig.netFrameworkVersion:                    "v8.0"
    + properties.siteConfig.publicNetworkAccess:                    "Enabled"
    + properties.siteConfig.scmIpSecurityRestrictionsDefaultAction: "Deny"
    + properties.siteConfig.use32BitWorkerProcess:                  false


Can't guess how to fix this.

I'll look into in a few hours.

Can't guess how to fix this.
—
Reply to this email directly or view it on GitHub.

I wasn't able to reproduce this. Do you have some screenshots or some build log?

Attaching screenshot.

Awesome! Can you expand and copy one of the errors?
man. 1. jun. 2015 kl. 13.51 skrev Valeriy Van<EMAIL_ADDRESS>
[image: xcode screenshot]
https://cloud.githubusercontent.com/assets/1630974/7912147/96fb5b9c-086d-11e5-947d-31d132192ed8.jpg
—
Reply to this email directly or view it on GitHub
https://github.com/hfossli/AGGeometryKit/issues/29#issuecomment-107414208
.

I am not able to reproduce this. Please reopen if you experience this again!


Failed to resolve dependencies using package.json or import_map.json

18 hours ago, my package deployment/publishing worked as expected, but not now.
Failed to publish @slainless/prodeskel-ws at 0.1.13: failed to build module graph: Module not found "file:///dist/core/typia".

No big commit from my end since the last successful build so it's probably from jsr.
Recent build (failed): https://github.com/slainless/prodeskel-ws/actions/runs/9789189029
Last successful build: https://github.com/slainless/prodeskel-ws/actions/runs/9777157205
Comparing both commits: https://github.com/slainless/prodeskel-ws/compare/0272ef9..4775e95

OK, I thought adding import_map.json might help, but nope. It still failed to resolve my dependencies.

Also starting to see this - no changes to jsr.json, package.json (apart from the version!), or any strange new imports.
https://github.com/inngest/inngest-js/actions/runs/9798531607/job/27057242163

I'm brand new at jsr.io and tried to publish my first package, but it failed with similar error.

Related https://github.com/denoland/deno/issues/24430

Good news: We pushed a fix and this should be resolved now 👍


Feature/issue 618 new data transfer interface

This PR implements https://github.com/smartdevicelink/sdl_android/issues/618
This PR is ready for review.
Summary

Implements IVideoStreamListener and IAudioStreamListener in StreamPacketizer
Implements IVideoStreamListener in RTPH264Packetizer
Adds SdlProxyBase.startVideoStream() and startAudioStream()
Renames SdlProxyBase.pauseH264(), resumeH264(), endH264(), pausePCM(), resumePCM() and endPCM()
Old public methods of SdlProxyBase are marked as deprecated
Updates createOpenGLInputSurface() to use startVideoStream() and IVideoStreamListener

Risk
This PR adds and deprecates some APIs in SdlProxyBase.
The code inside SdlProxyBase.createOpenGLInputSurface() is also affected by the refactoring work.
Testing Plan
Unit test cases are added for the packetizers.
Video streaming is tested with our proprietary app that uses SdlProxyBase.createOpenGLInputSurface(). (Note that createOpenGLInputSurface() now uses startVideoStream().)

@joeygrover Please kindly review.

Codecov Report

Merging #623 into develop will decrease coverage by 0.09%.
The diff coverage is 17.47%.

@@             Coverage Diff              @@
##             develop     #623     +/-   ##
============================================
- Coverage      40.54%   40.44%   -0.1%
- Complexity      2399     2408      +9
============================================
  Files            331      334      +3
  Lines          14518    14682    +164
  Branches        1526     1549     +23
============================================
+ Hits            5886     5938     +52
- Misses          8388     8487     +99
- Partials         244      257     +13

Impacted Files
Coverage Δ
Complexity Δ

...com/smartdevicelink/proxy/rpc/ImageResolution.java
81.81% <0%> (-18.19%)
6 <0> (ø)

...devicelink/streaming/AudioStreamingLPCMParams.java
0% <0%> (ø)
0 <0> (?)

...martdevicelink/streaming/VideoStreamingParams.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...martdevicelink/proxy/rpc/VideoStreamingFormat.java
85.71% <0%> (-14.29%)
7 <0> (ø)

.../com/smartdevicelink/SdlConnection/SdlSession.java
12.18% <0%> (-0.32%)
15 <0> (ø)

...smartdevicelink/streaming/AudioStreamingCodec.java
0% <0%> (ø)
0 <0> (?)

...martdevicelink/streaming/AudioStreamingParams.java
0% <0%> (ø)
0 <0> (?)

...n/java/com/smartdevicelink/encoder/SdlEncoder.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...n/java/com/smartdevicelink/proxy/SdlProxyBase.java
5.91% <0%> (-0.19%)
6 <0> (ø)

...m/smartdevicelink/streaming/RTPH264Packetizer.java
72.95% <45%> (-0.42%)
29 <2> (+1)

... and 6 more

Continue to review full report at Codecov.

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


Bar Chart not rendering on web page.

I am getting an error with the Bar Chart after deploying a web page to Server 2012 R2.
Unhandled exception at line 437, column 9 in http://localhost:61100/PartechTrans/ScriptResource.axd?d=z48y4oai1GgZ_a25brgQ9zD7rMfezY4sEp5Qnn37-rlZ89uRTV-Wsg-lM4_ZNzOWQ2VmVspM40ER72YqFszMf_2pESipSyouigdyVNX1wTvW4j2EqGJHpzdepIG3HgYn0&t=ffffffffcfeeeb28
0x800a138f - JavaScript runtime error: Unable to set property 'innerHTML' of undefined or null reference
Everything works perfectly on the DEV PC.
From any web browser, accessing the page there is only a line across the page where the control should render. The data is being retrieved and is identifiable in View Source, but the control is not present.
this line is where the error halts execution, when attempting to view the page on the test server within VS.
ln 437 =   this._parentDiv.innerHTML = svgContents;
I have an image attached.
This is a fully deployed web site that is working, this was just adding a new page. The control is on an aspx page which has 2 master page levels. The first provides the navigation panel, and the top level parent master page provides the site header and other logo and login controls.
https://www.devexpress.com/Support/Center/Attachment/GetAttachmentFile/bd33d5f7-7082-11e6-80bf-00155d62480c

title was an error, I don't seem to be able to edit it.

Hi @htm-git,
Thank you for posting this issue.
I tried to reproduce it using the information you provided but did not manage.
Would you please clarify the following:

The AJAX Control Toolkit version you are using.
The site root URL and page URL with BarChart.
The browser(s) you are using.

A sample project that illustrates this issue would be also of great help to quickly identify the problem.

toolkit is ver 16.1.1
The site is test server on our local LAN. On that server I have installed VSExpress2012 for debugging issues like this.
When the site is accessed from any PC on our LAN using the internal URL everything works accept the control renders in the browser (EDGE) as a straight line where the graph should appear.  The data is listed in the page source as indicated above. This site tests a large number of functions from web services to SQL connectivity, its been online for 5 years, upgraded from Server 2008 to 2012 R2.
Tested the page in Chrome, Firefox, and IE11 as well.

using the internal URL

Would you please provide this URL? If it contains sensitive data, remove it so I'm able to understand the structure of the project.
Also, please provide a markup for BarChart that is used to display it.

the url is internal.
http:\aspdemosvr2
It is resolved internally by DNS.

Are you using Master Pages?

Hi @htm-git ,
Yes I'm using Master Pages, but cannot get the result similar to yours.
I've attached a sample project to make sure we run the same code in our environments:
BarChartTest.zip
Please deploy this project to your IIS and check if the problem persists.
Also, I noticed a ctl00_ prefix in the control ID.
Would you please execute the this.get_id() method while debugger is at line 7 and show me the result?

I hope this is what you are looking for. Please let me know if you need something else.
And my control has 2 levels of Master Pages, if they makes a difference in your test project. I will try to set that up now and past back soon.

@htm-git thank you for that info.
I was able to reproduce this error by adding the following attributes to the system.web/pages element in Web.config:
<pages controlRenderingCompatibilityVersion="4.0" clientIDMode="AutoID">

This forces ASP.NET to generate ctl00_ prefix that is removed by the script, making getElementById() call invalid.
Here is a sample project which illustrates this issue: BarChartTest2.zip
If you have these settings applied in your project, please remove them and see the result.
I look forward to your response.

I have some issues running your project on my dev server. And am still working to test on DEV PC. I believe you created it with .Net 4.5 and my environments are both still 4.0.
I believe my first concern would be to make this a page level change, as I am sure you could image that such a global change would have an impact on the overall sites operation.
Do you know how I can remove these at the page level itself?

Removing just the clientIDMode="AutoID" attribute from the web config <pages property corrected the display of the bar chart. I am investigating the impact on the remainder of the site.

OK, this means that for .NET 4.0 projects, only the ClientIDMode affects ID generation.
You can set ClientIDMode to Predictable in code-behind for a particular page:
Page.ClientIDMode = ClientIDMode.Predictable

or particular control:
BarChart1.ClientIDMode = ClientIDMode.Predictable

You can also set it for a page using the @Page directive:
 <%@ Page ClientIDMode="Predictable" %>

For more information about ClientIDMode, please refer to this MSDN article: https://msdn.microsoft.com/en-us/library/system.web.ui.clientidmode(v=vs.100).aspx

Thank you for all of your help, I think this is resolved.

@htm-git Not yet. We provided you with a workaround, but the code that caused this error still exists.
At the moment, we are evaluating if it was added for some special purpose and whether it is safe to remove it.

Here is an update, I made the code changes as described to the web config. The page renders correctly in Edge, FF, Chrome, but NOT IE 11.

That's strange, because setting ClientIDMode to Predictable fixes my sample application for all browsers, including IE 11.
Would you please show the page rendered markup in IE 11?

Well, I retested and first time I got a break on the SVG non compatible browser error message, but now it is working in IE 11.

Well, I retested and first time I got a break on the SVG non compatible browser error message, but now it is working in IE 11.

Well, I retested and first time I got a break on the SVG non compatible browser error message, but now it is working in IE 11.


Chore - update travis tests to use node 0.12 instead of 0.11

Done with 32a4a0d3d0696c73704a36bcf06f81fd6fa023a4


Camera returns null image path on the Android device

[x] I have read the Getting Started section
[x] I have already searched for the same problem

Environment

Technology
Version

Flutter version
3.16.3

Plugin version
6.0.0

Android version
13

iOS version

macOS version

Xcode version

Google Chrome version

Device information: Samsung Galaxy A22
Description
My app is using the flutter_inappwebview plugin to open the url in the InAppWebview widget. The url loads the page which is developed in the Python. Webpage uses the ”Django-image-uploader-widget” to capture the image or pick image from the gallery of the both Android & iOS devices. Everything working fine in the iOS. But in the Android device, after capturing the image, local image path is returned as null.
Expected behavior:
On Android device, when the "Django-image-uploader-widget" opens the camera and captures the image, then app should receive an image path to show the image on the webpage.
Current behavior:
On the Android device, when the "Django-image-uploader-widget" opens the camera and captures the image, local image path is returned as null. However gallery option is working fine.
Steps to reproduce

Open the url in the InAppWebView widget.
After the page loads, tap on the image uploader view, select the camera.
Capture the image.
Image does show up on the webpage inside the inappwebview. Image path returned as null.

Images
Added Permissions in the Android Manifest file

Added provider tag with the meta-data in the Android Manifest file.

Added provider path in the xml

added the logs to check the path of the returned image

This is how my Inappwebview widget looks like

Stacktrace/Logcat
Case: When selecting image from gallery/files
Android onActivityResult: image path: content://media/picker/0/com.android.providers.media.photopicker/media/1000001581
Case: When capturing image from camera
Android onActivityResult: image path: null

Your provider seems not correct, the android:authorities value should be as follow:
<provider
    android:name="com.pichillilorenzo.flutter_inappwebview_android.InAppWebViewFileProvider"
    android:authorities="${applicationId}.flutter_inappwebview_android.fileprovider"
    android:exported="false"
    android:grantUriPermissions="true">
    <meta-data
        android:name="android.support.FILE_PROVIDER_PATHS"
        android:resource="@xml/provider_paths" />
</provider>


Copy text from JavaScript string variabvle when my app triggers it to do so

Is it possibble tocopy text to clipboard from a JavaScript variable holding a string of text and to have it happen when my app calls a function to do it instead of requiring a button or DOM element?
I have a bookmarklet which converts a GItHub file URL into a Rawgit.com URL.  I want to have the JS in the bookmarklet add that generated URL to the users clipboard.

You can copy a JavaScript variable holding a string by using the imperative API: https://github.com/zenorocha/clipboard.js#advanced-options
Due to security restrictions imposed by browser vendors, user interaction is required in order copy/cut to work. A function call or simulated click event using JavaScript does not work as this would enable clipboard poisoning.


fix: set @angular peerDependencies version to >= 8.0.0 =< 9.0.0

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


branch pushes events: these are my expectations

I just had a conversation w/ @lachmanfrantisek about builds of pushes to a branch, these are my expectations how it's suppose to work:

a push to any branch in the upstream repo triggers p-s

if packit.yaml on that branch has this: {jobs: [{job: build, trigger: commit}]}, the build is triggered

Franta says that these need to be defined on master branch for all other branches - that's not intuitive to me, also some repos may not even have a master branch (hello ansible).

Franta says that these need to be defined on master branch for all other branches - that's not intuitive to me, also some repos may not even have a master branch (hello ansible).

No, that was not what I meant..;-)
If we build in each branch with a valid config with present job config. It will run for the build for push to any branch that starts from master after we add that config: e.g. stable, upstream->upstream PR branches. Mostly all branches present in the upstream repository.

By now, it works like this:

we push a new commit to the branch a
we load the config from that commit
if a is configured in the job config, we trigger the build

(If a=master, you don't need to define branch in config.)

so it works as I expected then, right? :D

so it works as I expected then, right? :D

I hope so..;) You only need to specify that branch in the config.
Example:

https://github.com/packit-service/hello-world/commits/the-build-branch
master does not contain config with branch build
the-build-branch does contain the config:- job: copr_build
  trigger: commit
  metadata:
    branch: the-build-branch

nice, works! https://github.com/packit-service/docker-py-source-git/commit/d4d8dbd739c6c0db073e1579570168b8eeec08c1

but still, I feel like the metadata -> branch: ... is redundant

Let's use packit as an example in the situation where we don't force to specify a branch (your proposal):

We will add this job to config and merge to master. (1. master build)
We will create a new PR and merge it. (2. master build, still works well)
We will move the stable branch to the HEAD commit of master (stable build)
Release-bot will create a new branch for release PR (0.10.2 build for each commit in that PR, in case of PR builds, we will build two times for each commit.)

Do we want to have all these builds done automatically? I think it's better to specify only the relevant branches to avoid lot of builds that are not needed.

How can one disable builds for the master branch?
Wouldn't be better to have builds for branches disabled by default for all branches, except when explicitly enabled with metadata.branch?

Wouldn't be better to have builds for branches disabled by default for all branches, except when explicitly enabled with metadata.branch?

That's how it works for now: you need to specify branch to enable builds. The exception with master is that you don't need to add metadata.branch but you still need to define job. (=> You are manually turning master-builds on by adding job-config without metadata.branch.)

How can one disable builds for the master branch?

By doing nothing. (No config with commit trigger.)
Using metadata.branch if you want to build in another branch.

Looks like we (=I) need to document this behaviour at least.

How can one disable builds for the master branch?

By doing nothing. (No config with commit trigger.)
Using metadata.branch if you want to build in another branch.

I see, thanks for the explanation! So for the commit trigger, the master branch is the default currently.
I was actually thinking whether it would make sense to have "on any branch" be the default instead. This is the way it's done in GitLab pipelines, and then they use only/except to narrow this. But then I remembered that the drawback of this is that it requires explicit action to limit the number of builds, and many times people don't even notice that these builds are running for every branch, which is kinda a waste of resources.
One more question: could we make the default be "the default branch" instead of "master"? This would be a more generic solution as not all Git projects have their default branch set to "master".

I was actually thinking whether it would make sense to have "on any branch" be the default instead. This is the way it's done in GitLab pipelines, and then they use only/except to narrow this. But then I remembered that the drawback of this is that it requires explicit action to limit the number of builds, and many times people don't even notice that these builds are running for every branch, which is kinda a waste of resources.

Exactly. We can maybe add support (opt-in) for all later. Some alias? Or regex support (e.g. *)?

One more question: could we make the default be "the default branch" instead of "master"? This would be a more generic solution as not all Git projects have their default branch set to "master".

If we can get that info via API, that would be nice. (OGR implementation needed.)

adding this to the Tue arch meeting, I'm lost

We have discussed this and concluded we wanna keep this explicit: if you wanna use branch builds, you need to explicitly list the branch name in the config.


Update is needed for country processing of Turkish spellings for Türkiye

Overview
The official spelling of Turkey has been updated by the Turkish Government to "Türkiye" (also, Turkiye without the umlaut is possible). These spellings are not currently included in our codebase and thus return a parsing error when a user's list uses Türkiye/Turkiye/TÜRKİYE/TURKİYE as the country name. List # 2349 is an example of this occurrence.
Expected Behavior
When a list has any of the current English "Turkey/TURKEY" or Turkish spellings of the country "Türkiye/Turkiye/TÜRKİYE/TURKİYE" it should be accepted and processed as a reference for the country.
Actual Behavior

Lists with "Türkiye/Turkiye/TÜRKİYE/TURKİYE" in the country column are resulting in a parsing error. List # 2349 is an example of this occurrence. "Turkey" also needs to continue being supported, in addition to the other variations.

All instances of "Turkey" on the front end should also be changed to "Türkiye."

Steps to Reproduce

Log-in to OS hub admin dashboard
navigate to the desired list. It has been rejected due to the parsing error. rejected lists
select list
note parsing error
download the submitted file
Note that the country value is TÜRKİYE
MacBook Pro, Opera Browser

Demo
Additional context

Thanks for wiring up a detailed issue for this, @vrwOAR. This came up in a discussion with @mariel-oar and @KlausGPaul and @KlausGPaul mentioned potentially taking ownership of this issue and creating the pull request to make the required code update.

@KlausGPaul , if you have code for this, could you add it as a comment here?

Marked as client action needed until code created so far has been added

@KlausGPaul - just checking in on when the code will be added?

@obrienad @jwalgran , we don't have code to contribute for this one.

This looks good to go. I've tested / verified the following things on staging.opensupplyhub.org:

The country that shows up in the country search is now "Türkiye" - you can search for it with and without the accent.
"Turkey" is no longer in the country list
On profiles, "Türkiye" is the value that is appended to the end of all address fields
Submitted a list with 3 facilities - one with Turkey, one with Türkiye, one with Turkiye - all were accepted and correctly processed in the system as "Türkiye"
Instances where "Turkey" shows up in a facility name, address (other than the appended value at end of address), or in any of the extended fields is not translated.

cc: @open-apparel-registry/project-managers @hlennett (i dont think Irem has github, so tagging Hannah too)


network - add screen to change network outside of wallet context

What kind of PR is this?:

/kind feature
What this PR does / why we need it:

To provide an option to user to switch network during create wallet and restore wallet flow

Which issue(s) does this PR fixes?:

Fixes #566
Additional comments?:
Replace bottom modal used in mock UI with a screen due to technical difficulty
Developer Checklist:

[x] Read your code changes at least once
[x] Tested on iOS/Android device (e.g, No crashes, library supported etc.)
[x] No console errors on web
[x] Tested on Light mode and Dark mode*
[ ] Your UI implementation visually matched the rendered design*
[x] Unit tests*
[x] Added e2e tests*
[x] Added translations*

@kyleleow we need to have a different alert message when you're out of wallet context since you're already on the onboarding screen.

Added german translation from Bella


Delete $template.json

Team background:
The Nexus consists of 4 core teams with more than 5 years of experience on the Internet.
Jonatan - Founder and CEO has more than 17 years of experience in finance and corporate management, working with global investment banks.
Stepanus Brian - one of the founders.
He has extensive experience in the IT industry and specializes in customer relationship management. Then he built an amazing QR mobile app company.
Cristian Baher - Marketing Manager
As an independent trader in the information technology industry, he has expanded the company from an insignificant starting point to suppliers throughout the region
.
Albert Lucius - Community Manager
He is also the fourth Industrial Revolutionary Committee of the 21st Institute of the People, a policy that advises think tanks. He is also the head of education at the Korea Financial Technology Association.
Basic project information:
NEXUS is a decentralized platform. As a medium for shopping at online stores, several business partnerships will fill the market for encryption and support for special wallet support.
More Nexus information
Official website: http://nexusworld.us
Telegram: https://t.me/nexus_world
Twitter: https://twitter.com/nexusOfficer
Recommended gas limit: 90000

https://github.com/consenlabs/token-profile/pull/2969#issuecomment-437576138

Incorrect information submitted, please go through our guide and check again.


added api for tools in documentation

Motivation
Added an API display functionality for the nwb_conversion_tools.tools in the documentation.
Checklist

[x] Have you checked our Contributing document?
[x] Have you ensured the PR description clearly describes the problem and solutions?
[x] Have you checked to ensure that there aren't other open or previously closed Pull Requests for the same change?
[x] If this PR fixes an issue, is the first line of the PR description fix #XXX where XXX is the issue number?

@CodyCBakerPhD , can we get this to build automatically?

@bendichter Webhook should be working now

@CodyCBakerPhD awesome, how did you do that?

can we reformat the docstrings so these look better?

otherwise, lgtm!

@CodyCBakerPhD awesome, how did you do that?

It just had to be re-synced on the 'Integrations' tab of the ReadtheDocs admin page

can we reformat the docstrings so these look better?

For docstrings we need to think about how it looks when displayed both via the some_function? command as well as how the API autodoc renders it.
What we could do is try to setup the conf.py to convert certain parts of function docstrings read via automodule to parse with .. code-block:: python instead of as text.
That might take some tinkering; in the meantime, we could hard code it directly into the docstrings (which render as rst) but that would then make the in-console display look less nice.

@CodyCBakerPhD how about this?

with code:
https://github.com/catalystneuro/nwb-conversion-tools/blob/7ed60c26a4db33c240555d82f7ca8193b139e82d/nwb_conversion_tools/datainterfaces/ecephys/blackrock/blackrockdatainterface.py#L94-L103

@bendichter Yeah, that looks good for both approaches!
BTW do you know how to fix the black-style formatting in the readthedocs? From #422 I thought it was supposed to apply to the blue box (function definition), but it seems to be working on the 'parameters' (docstring) level at the moment. I think you mentioned it might need a specific sphinx version? If so maybe we can try pinning that version in the requirements-rtd.txt, which is the last thing to install on a readthedocs build

Apparently the webhook doesn't re-build when you update the branch from master.
Heres the unhidden wep link: https://nwb-conversion-tools.readthedocs.io/en/documentation_add_api_for_tools/api/tools.spikeinterface.html
Changes specific to this PR LGTM

let me update the rest of the metadata sections first


React.PropTypes deprecated. Use prop-types package instead

Using React.PropTypes is deprecated. Added prop-types package and fixed references to React.PropTypes. Fixes issue #24

👍


Object browser unable to access source and object

Does anyone face error message prompt while access object browser as attach here? Is it due to authority?

Trying to open an object (like a *PGM in your example) will result in this error being displayed.
There is an extension that has not been released yet (but a prerelease is downloadable here: https://github.com/codefori/vscode-ibmi-fs/releases) that will allow you, for some types, to display and edit objects when applicable.


Incorrect Metadatacore.rdf references

https://github.com/iofoundry/ontology/blob/84e4c12b2cddbe1bc70bf88037d83ac72d33f00e/core/Metadatacore.rdf#L35
should be:
https://spec.industrialontologies.org/ontology/20230120/core/Metadatacore/

Also:
https://github.com/iofoundry/ontology/blob/84e4c12b2cddbe1bc70bf88037d83ac72d33f00e/catalog-v001.xml#L8
should read:
https://spec.industrialontologies.org/ontology/core/Metadatacore/

The same applies to:
https://github.com/iofoundry/ontology/blob/84e4c12b2cddbe1bc70bf88037d83ac72d33f00e/MetadataIOF.rdf#L5
https://github.com/iofoundry/ontology/blob/84e4c12b2cddbe1bc70bf88037d83ac72d33f00e/MetadataIOF.rdf#L19
https://github.com/iofoundry/ontology/blob/84e4c12b2cddbe1bc70bf88037d83ac72d33f00e/MetadataIOF.rdf#L35


SSH agent initialization postStart event fails workspace if $HOME is not writable

Description
In container images where the $HOME directory is not writable (such as the go-toolset image, which has $HOME=/opt/app-root/src/) the init-ssh-agent-command-... postStart event will fail.
This is due to the fact we assume $HOME/ssh-environment is writable when doing ssh-agent | sed 's/^echo/#echo/' > $SSH_ENV_PATH, however this is not always the case:
$ ssh-agent |  sed 's/^echo/#echo/' > $SSH_ENV_PATH
bash: /opt/app-root/src/ssh-environment: Permission denied

Maybe we should wrap the entire ssh-agent intialization command with a (...) || true so that regardless of wether a specific step of the ssh-agent initialization fails, the workspace will start up. This is the approach taken for the init-persistent-home preStart event.
How To Reproduce

Set up an SSH key with a passphrase
Create a workspace using the go-toolset image:

kind: DevWorkspace
apiVersion: workspace.devfile.io/v1alpha2
metadata:
  name: plain-devworkspace
spec:
  started: true
  routingClass: 'basic'
  template:
    components:
      - name: web-terminal
        container:
          image: registry.access.redhat.com/ubi9/go-toolset:1.19.13-4.1697647145
          memoryRequest: 256Mi
          memoryLimit: 512Mi
          mountSources: true
          command:
           - "tail"
           - "-f"
           - "/dev/null"

Check the workspace status, it should have failed: oc get devworkspace -n $NAMESPACE

Expected behavior
The workspace should succceed to start up. Whether the automatic SSH passphrase provisioning functionality works is another topic (maybe we should set SSH_ENV_PATH=/tmp/ssh-environment instead of SSH_ENV_PATH=$HOME/ssh-environment?)
Additional context
Upstream Che Issue

@AObuchow I have the same issue but that's not related to $HOME not being writable, in my case, the second container is distroless so it doesn't have sh or true. Any postStart event will fail in that container. But the init-ssh-agent-command is injected in both containers.
Some ideas:

There should be a documented attribute to opt-out from post start events
If there are multiple components the event should be added for the first component only
If mountSources: false there should be no need to add the init-ssh-agent-command-... postStart event

That's my DevWorkspace with 2 containers:
apiVersion: workspace.devfile.io/v1alpha2
kind: DevWorkspace
metadata:
  name: outyet-dw
spec:
  contributions:
    - components:
        - container:
            env:
              - name: CODE_HOST
                value: <IP_ADDRESS>
          name: che-code-runtime-description
      name: che-code
      uri: https://eclipse-che.github.io/che-plugin-registry/main/v3/plugins/che-incubator/che-code/latest/devfile.yaml
  started: true
  template:
    components:
      - container:
          image: ghcr.io/l0rd/outyet-dev:latest
          memoryRequest: 2G
          memoryLimit: 10G
          cpuRequest: '1'
          cpuLimit: '4'
          mountSources: true
        name: dev
      - container:
          cpuLimit: 500m
          image: ghcr.io/l0rd/outyet:latest
          memoryLimit: 128Mi
          mountSources: false
        name: outyet
    projects:
      - git:
          remotes:
            origin: https://github.com/l0rd/outyet.git
        name: outyet
        sourceType: Git

@vinokurig, your PR does not fix the use case above. I would appreciate it if you could address that as well.

@l0rd

There should be a documented attribute to opt-out from automatically injected post start events

I think this is not related to the current issue and can be done separately.

If there are multiple components the event should be added for the first component only

We can not be sure that the dev component will always be first in the list.

If mountSources: false there should be no need to add the init-ssh-agent-command-... postStart event

Why not to handle the ssh passphrase for such components?

@l0rd Thank you for the feedback on this issue, it's always appreciated to see how you're using DevWorkspace Operator :)
In general, I think your issue is outside the scope of the current issue (as it seems more related to postStart events in general, rather than the DWO-injected SSH agent initialization postStart event). I'll go ahead and create a new DWO issue for your topic.
I agree with @vinokurig's responses to your suggestions. An opt-out attribute for automatically injected postStart events would probably be best for container images where running postStart events is impossible.
I inspected the ghcr.io/l0rd/outyet:latest image (which is based on golang:onbuild) and saw that it does have sh but it's located in /usr/bin/sh instead of /bin/sh which is what DWO expects.
This suggests another improvement to DWO's postStart events: maybe we can use command (which is POSIX-compliant) to determine how to invoke sh if is available on $PATH?
Unfortunately, if someone uses a scratch image that does not even have command, the postStart event will still cause the container to fail. In this case, the best solution would probably be to allow users to opt-out of the DWO-injected postStart events.

We can not be sure that the dev component will always be first in the list.

@vinokurig The convention is to consider the first component the dev component. We contribute the editor in the first component, not in every component. There should be an attribute to specify a component that is not the first one as the dev component.

Why not to handle the ssh passphrase for such components?

@vinokurig isn't the SSH passphrase mainly used for git authentication? If that's not you can ignore my comment

I inspected the ghcr.io/l0rd/outyet:latest image

@AObuchow you have inspected the wrong image. In the meantime I have changed ghcr.io/l0rd/outyet:latest base image to make my scenario work (it used to work and that's what I am going to demo at KubeCon in a couple of weeks so I fixed it with bigger but working image). But I had updated the example above with the distro-less image ghcr.io/l0rd/outyet@sha256:b83e158687d6cb3d7ae46382be1e4fbb8eb3572f3423a9c3c9beae6cd55cc0e8.
In general I think that #1307 was a bad idea because there is a high risk to break existing workspaces (we just found 2 container images where the command failed, there may be more). And no matter the value of the new feature implemented, avoiding breaking existing workspaces should be the priority. Also we should avoid, at the DWO level, to modify a DW applied by a user. The dashboard does an automatic generation of DW, so why not adding the post start event there?

@ibuziuk I am reproducing it on developer sandbox, without any specific config on my side.

@l0rd @ibuziuk @AObuchow
To summarize: from the discussion I see next issues:

$HOME directory is not writable, the related pull request addresses the particular issue.
sh might not be available. This might affect all workspaces regardless the presence of ssh key configmap.
Do not add the init-ssh-agent post start event to all containers except the first one.

I believe that [2] and [3] issues should be addresses in a separate pull requests

@vinokurig sure, I am perfectly fine with a separate PR, of course.

@vinokurig Thank you for summarizing the current state of things.

I am going to create an issue for [2]

I'm happy to create an issue for this since it's past your working hours in your current timezone. I will also start investigating potential solutions to solve this issue.
Regarding [3]:

Do not add the init-ssh-agent post start event to all containers except the first one.

Though I agree with @l0rd's comment that the convention is to consider the first component as the main tooling component:

The convention is to consider the first component the dev component. We contribute the editor in the first component, not in every component. There should be an attribute to specify a component that is not the first one as the dev component.

We had an issue CRW-6614 where on the DevSandbox the first component in the devfile was not always the tooling component. I'm still confused as to why this was the case, as no reproducer devfile was left in that Jira and there's only screenshots that show this repo which does have the tooling container as the first component.
However: when an SSH key & passphrase is provided via the automount configmap/secret mechanism, it is mounted to all container components (not just the tooling component). Thus, it seems natural to also allow for automatic SSH passphrase provisioning with the ssh-agent in all container components.
So I do not think we should only provide the ssh-agent postStart event to the first container component.

We had an issue CRW-6614 where on the DevSandbox the first component in the devfile was not always the tooling component. I'm still confused as to why this was the case, as no reproducer devfile was left in that Jira and there's only screenshots that show this repo which does have the tooling container as the first component.

Using the attribute controller.devfile.io/merge-contribution: true/false should allow you to control in which component the editor is contributed.

Thus, it seems natural to also allow for automatic SSH passphrase provisioning with the ssh-agent in all container components.

Ideally, that may be right. But considering that it can generate a workspace startup error, making using Che / Dev Spaces impossible, I would be very cautious. You should find a way to ignore any error in the passphrase provisioning or execute only when it's necessary and providing an option to opt out.
I understand that's not an easy task 🤷 and I don't have any magical solution. However, considering the maturity of Che / Dev Spaces, any new feature that introduces risks of breaking existing workspaces (with no workaround) should be avoided.

According to the kubernetes documentation there is no way to ignore the post start event failure:
If either a PostStart or PreStop hook fails, it kills the Container.
Although we can increase the stability of the  workspace start with next solutions:

Do not add the post start event if the ssh secret is not defined

The ssh key secret that maps the keys to the container filesystem, may have another from conventional name, so we can not detect them.
If the ssh key has a passphrase set up, the workspace start will continue to fail due to a vulnerable container.

Add the post start event only to the first container component.

I agree with @AObuchow

However: when an SSH key & passphrase is provided via the automount configmap/secret mechanism, it is mounted to all container components (not just the tooling component). Thus, it seems natural to also allow for automatic SSH passphrase provisioning with the ssh-agent in all container components.
So I do not think we should only provide the ssh-agent postStart event to the first container component.

Have an additional configuration to opt-out from all the init-ssh-agent post start event.

This overcomplicates the flow and would be not very obvious for users.

Ideally, that may be right. But considering that it can generate a workspace startup error, making using Che / Dev Spaces impossible, I would be very cautious. You should find a way to ignore any error in the passphrase provisioning or execute only when it's necessary and providing an option to opt out.
I understand that's not an easy task 🤷 and I don't have any magical solution. However, considering the maturity of Che / Dev Spaces, any new feature that introduces risks of breaking existing workspaces (with no workaround) should be avoided or reverted in my opinion.

+1 Agreed. Allowing this feature to modify the DevWorkspace unconditionally (and potentially break existing workspaces) was something I should have avoided.

Do not add the post start event if the ssh secret is not defined

The ssh key secret that maps the keys to the container filesystem, may have another from conventional name, so we can not detect them.

If the ssh key has a passphrase set up, the workspace start will continue to fail due to a vulnerable container.

Agreed, unfortunately the workaround/fix I currently have provided only resolves the issue for Che/DevSpaces specifically.

Have an additional configuration to opt-out from all the init-ssh-agent post start event.

This overcomplicates the flow and would be not very obvious for users.

Instead of opt-out, I think an opt-in to enable the SSH agent feature (as a devfile & devworkspace attribute) might work. This is something I propose in the long term solution idea of this issue. The Che dashboard would be responsible for setting this attribute. For non-Che users, they would unfortunately have to consult the DWO documentation to discover this attribute, but at least their workspaces would not potentially fail from having this feature enabled by default.

Currently, @ibuziuk and I are thinking of guarding the injection of the SSH agent postStart event under the DWOC's config.enableExperimentalFeatures field until we can sort out how to handle this feature in the most graceful way.

given that the feature is risky I believe it would be a really good move to enable it based on the DWOC config.
on UI we can also articulate that this is experimental - https://github.com/eclipse-che/che-dashboard/pull/1249/files
@l0rd @vinokurig @dmytro-ndp thoughts?


Text to speech functionality missing on android

hey bro. I face the same question.    you can use the 0.5.1 version. As I know, the tts support is removed at 0.5.2 or higher.

No description provided.

hey bro. I face the same question. you can use the 0.5.1 version. As I know, the tts support is removed at 0.5.2 or higher.


Some versions of phpdbg are ignoring the __debugInfo magic method

Describe the bug
For some PHP versions, the phpdbg executable provided by this action is ignoring the __debugInfo magic method.
To Reproduce
I've created a repo to reproduce the issue: ezzatron/setup-php-action-repro
But the gist of it is this PHPUnit test, which passes on equivalent Travis CI builds, (and weirdly, only under PHP 7.3 on GitHub Actions?):
<?php

use PHPUnit\Framework\TestCase;

class ReproTest extends TestCase
{
    public function testDebugMagicMethod()
    {
        ob_start();
        var_dump(new ClassA());

        $this->assertRegExp('/\["a"]=>\s+string\(1\) "b"/', ob_get_clean());
    }
}

class ClassA
{
    public function __debugInfo() {
        return ['a' => 'b'];
    }
}

Expected behavior
The tests should pass, like this Travis build: https://travis-ci.org/ezzatron/setup-php-action-repro/builds/618399139
At the moment, they fail like this GitHub Actions build: https://github.com/ezzatron/setup-php-action-repro/commit/cf465a6517a7fd06811997914fe54807e045e235/checks?check_suite_id=333972681
Screenshots

Additional context
I'm the author of Phony, a PHP mocking library. I use the __debugInfo() magic method so that if a developer uses var_dump() on one of Phony's mocks (or something containing a mock), it doesn't produce thousands of lines of output about internal Phony classes. I discovered this bug because I have a test for this behaviour in Phony's test suite.

@ezzatron Thanks for reporting this.
Fixed in #94

No worries. Thanks for the speedy fix!


Unable to deploy my website

Describe the bug
An unknown error occured while running an action. :\
Link to the Bolt URL that caused the error
https://bolt.new/~/sb1-qsmj6q
Steps to reproduce
Just try to deploy
Expected behavior
Successful Deployment
Screen Recording / Screenshot

Platform
Browser name = Chrome
Full version = <IP_ADDRESS>
Major version = 129
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
performance.memory = {
  "totalJSHeapSize": 74673762,
  "usedJSHeapSize": 68371054,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Username = j-q-khan
Chat ID = 58b7f6591b1a

Additional context
Error:
An unknown error occured while running an action.

Thanks for the report and patience during this beta period! Tracking all deployment related bug fixes in issue #499


Handle conflicting attribute names #230

Summary
resolves #230
Description
This PR adds a support for importing attributes with conflicting names by prefixing them with attribute. string. For example, to import value to attribute named as productType the header should be attribute.productType.
Todo

Tests

[x] Unit
[x] Integration
[ ] Acceptance

[ ] Documentation

Coverage decreased (-0.03%) to 80.029% when pulling 1a1e1825ec0919a0611c88043507dd4e71c4aacd on 230-conflicting-attribute-names into d4cc3d0f4eca3544d865cd30f80424bef23924a9 on master.


Always copy to "/" (root). Fixes #1954

As seen in https://github.com/testcontainers/testcontainers-java/pull/1814, we introduced a regression in 1.12.1.
This PR makes the copy API always target the root and then uses the full path in the archive we send to the daemon.

Good catch - thank you. I'll hit the merge button.


How to set glue dynamically when tests are executed using Jenkins

Previously plugin had a possibility to set glue in pom like this:
<glue>${glue}</glue>
It worked fine. Now I am trying to use newer version (5.0.0) since I want to get parallelizm-by-feature option, but I am forced to set glue like this:
<glue><package>glue1</package><package>glue2</package></glue>
Which doesn't allow to set glue dynamically. If I set it like I did previously I am getting Unable to parse configuration of mojo com.github.temyers:cucumber-jvm-parallel-plugin:5.0.0:generateRunners for parameter glue: Cannot assign configuration entry 'glue' with value 'steps.advisor.general,steps.advisor.insurance,steps.advisor.loan,hooks' of type java.lang.String to property of type java.util.List
Is there a way how I can set glue as an option from the command line? I can not set all the glues in pom since I would get duplication of step definition errors in this case. So each test suite defines its own set of glues to use

have you tried to set cucumber.options with --glue glue1,glue2 ?

The goal of maven is to make builds portable and repeatable. Setting parts of the configuration dynamically goes against this.
You can either configure different executions of the cucumber-jvm-parallel-plugin in different maven profiles. These can then be activated from the command line by using -Pmy-tests or  -Pmy-other-tests.
Or you can configure multiple executions of the cucumber-jvm-parallel-plugin with different configurations. This will generate runners for each configuration. All generated runners will then be executed in the test phase.

Launch with maven paramater -Dcucumber.glue="glue1,glue2" will not works?

it looks like -Dcucumber.glue="glue1,glue2" works. Thank you @johny2000uwb


location of data/models/unet16_binary_20/model_0.pt

Hello, I was looking to run through the demo notebook, but received a FileNotFound error when trying to load model_path = 'data/models/unet16_binary_20/model_0.pt'. Does the model need to be trained first to generate these files/directories? Thanks!

You need to download them from https://drive.google.com/open?id=13e0C4fAtJemjewYqxPtQHO6Xggk7lsKe


Can't publish messages  with QoS 'ExacltyOnce' with MQTTv5

When I publish a message after connecting with MqttProtocolVersion.V500 and setting the QoS to MqttQualityOfServiceLevel.ExactlyOnce, my HiveMQ Server disconnects the client immediately with the error message

Sent a PUBREL with payload.

If I switch the protocol to MqttProtocolVersion.V311, I can publish messages with MqttQualityOfServiceLevel.ExactlyOnce without the error.
My guess is that there is something wrong with the conversion of the PUBREL message to a MQTTv5 message-type.
Here is my code for connecting:
var options = new MqttClientOptionsBuilder()
                        .WithProtocolVersion(MqttProtocolVersion.V500)
                        .WithClientId("id")
                        .WithCredentials("user", "pass")
                        .WithTcpServer("localhost", 1883)
                        .Build();

await this.client.ConnectAsync(options);

Here is my code for publishing:
var message = new MqttApplicationMessageBuilder()
                .WithTopic("topic")
                .WithQualityOfServiceLevel(MqttQualityOfServiceLevel.ExactlyOnce)
                .WithPayload("payload")
                .Build();

await client.PublishAsync(message);

As I said, with .WithProtocolVersion(MqttProtocolVersion.V311) everything is fine.
Since the problem seems to occour in the PUBREL, the other two QoS's are also fine.
I am using the Client from the NuGet 3.0.0 in a Xamarin Forms Project.
The broker is a locally installed HiveMQ Community Edition version 2019.1.
Any help is greatly appreciated :)

Hi,
thank you for reporting that issue. Do you may have more information from HiveMQ? I am wondering what the "payload" may mean. MQTTv5 has some additional data in that packet in comparison to v3 but the term payload is strange.
Best regards
Christian

I found the issue and fixed it. I will release a new version soon. Until that please use V311 instead.

Hi Christian,
thank you very much for your fast response and fix!
I'm looking forward to the next release.
Best regards
Philipp


Automated cherry pick of #1172: fix: 1. project add user/group rpc paramter error 2. secgrouprule 403 3.

Cherry pick of #1172 on release/2.10.0.
#1172: fix: 1. project add user/group rpc paramter error 2. secgrouprule 403 3.

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.10.0@55c813c). Click here to learn what that means.
The diff coverage is 0%.

@@               Coverage Diff                @@
##             release/2.10.0   #1173   +/-   ##
================================================
  Coverage                  ?   7.87%
================================================
  Files                     ?     364
  Lines                     ?   57157
  Branches                  ?       0
================================================
  Hits                      ?    4503
  Misses                    ?   52254
  Partials                  ?     400

Flag
Coverage Δ

#aFlag
7.87% <0%> (?)

Impacted Files
Coverage Δ

pkg/cloudcommon/policy/resources.go
0% <ø> (ø)

pkg/cloudcommon/db/sharablevirtual.go
0% <0%> (ø)

pkg/compute/models/secgrouprules.go
3.65% <0%> (ø)

pkg/util/rbacutils/rbac.go
71.29% <0%> (ø)

Continue to review full report at Codecov.

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

/lgtm
/approve


fix: Optimize reflows on the applications list page

Description

Optimize reflows on the applications list page which is happening due to isVerticalEllipsisActive helper function.

Fixes #16012
Type of change

Performance improvement

How Has This Been Tested?

Took a performance profile before and after making this change. Could see a difference in rendering time.

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

/ok-to-test sha=d2bce74

/ok-to-test sha=86c3723

/ok-to-test sha=24120c3


Minecraft bot specifies if message is a reference or not

Added a small thing that will specify the user they are referencing (if they are)

Why does this change all single quotes to double quotes and why does it add semicolons?


Can't run

2023-08-16 08:20:22 INFO     discord.client logging in using static token
 * Serving Flask app ''
 * Debug mode: off
Traceback (most recent call last):
  File "/Users/Moosems/Desktop/tkinter-bot/tkinter-bot/main.py", line 35, in <module>
    main()
  File "/Users/Moosems/Desktop/tkinter-bot/tkinter-bot/main.py", line 30, in main
    bot.run(TOKEN)
  File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/discord/client.py", line 860, in run
    asyncio.run(runner())
  File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/asyncio/runners.py", line 190, in run
    return runner.run(main)
           ^^^^^^^^^^^^^^^^
  File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/asyncio/runners.py", line 118, in run
    return self._loop.run_until_complete(task)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/asyncio/base_events.py", line 653, in run_until_complete
    return future.result()
           ^^^^^^^^^^^^^^^
  File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/discord/client.py", line 849, in runner
    await self.start(token, reconnect=reconnect)
  File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/discord/client.py", line 777, in start
    await self.login(token)
  File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/discord/client.py", line 609, in login
    raise TypeError(f'expected token to be a str, received {token.__class__.__name__} instead')
TypeError: expected token to be a str, received NoneType instead

Do you have an .env file in your project root directory with the keys required to run a discord bot?

Nope.

Any docs on how to set that up?

I think this should cover: https://www.writebots.com/discord-bot-token/


Some tweaks to improve alignment in table with checkbox

If a checkbox does not have a description (even if it is empty), layout will be broken as checkbox won't be aligned correctly

UPnP checkbox looks better with description instead of help

This commit fixes the two issues

Thanks for the improvement.


Date Format Error

why nlog not respect my format ?
fileName="${basedir}/Texto/${date:dd-MM-yyyy}.json"
"attribute name="time" layout="${date:dd-MM-yyyy-HH-mm}"
and he still writes "time": "2015-08-26 12:12:46.3999"...
why ?

ok works fine now
${date:format=dd-MM-yyyy HH:mm:ss.fff}
Thx


Last version bug

I don't know why but after about 40 mins bot give this, and won't work
https://i.gyazo.com/442351a55c44094b5decce4e92e700d7.png

I used 100 bots and 5 tabs

Checking the files of the bot, maybe its because now i should use
cd AGAR-XPBOT && node FacebotsWithRestart.js and not anymore Facebots.js? tell me now im testing

https://gyazo.com/d9a846e816a9423b1620f78c11f71ad4
@camilleeyries so i try to start with facebotswithrestart  but i get error, windows 10

Everithing will be fixed if you delete node_modules folder and start Install Dependencies (Windows).bat

Ok gonna test this out now, im doing 100 bots with 5 prompt opened, this could cause fails? @hassanila97

Yes, if you're using one pc only...   it's not depending on the speed of the pc but by the connections limit...    You can run 4 tabs and it should be ok..    but you can also run 100 tabs in many vps and you get lvl 100 in 3/4 days

Yeah i know how this works, so if my net fail anyways the app will restart after some time ( didnt check ) so i dont need to worry? If it crashs i lower the prompt to 4

Yes it will auto-restart on fail

Ok very fast support, maybe tomorrow i post a video on YouTube, idk i dont want childs without brani could use this because moss of ppl dont know how get cookies:)

Yep... Btw I'm not the dev and the dev that made it work with multiple fb accs is @camilleeyries but the real dev that made 98% of the code is (pulviscriptor (GitHub)) [https://github.com/pulviscriptor/agario-client]

@skillize pls don't make any YouTube video. I Want to start my channel !


Fix compiler warnings exposed by extra flags

Note that this change makes use of the AX_APPEND_COMPILE_FLAGS macro
from the "Autoconf Archive"1; the list of warnings is taken from the
AX_COMPILER_FLAGS_CFLAGS macro. I've not directly used the latter
macro, as it enables dubious warnings such as
-Wdeclaration-after-statement.
Besides some minor tweaks to silence the warnings, the code changes in
this commit also fixes one definitive bug uncovered by the warnings,
namely the use of size_t instead of ssize_t when calling
mq_receive.

Note that this change has room for improvement (IMO):

It introduces a build dependency on autoconf-archive, when
building from git (not when building from a release tarball). This
could be fixed by including the code for AX_APPEND_COMPILE_FLAGS
in the repo, probably after cleaning it up and simplifying it a
bit. I could do that, if it's deemed desirable.

The list of warnings should be considered more carefully, but this
should be a reasonable starting point.

Thank you for your contribution.
However I have 2 remarks.

Your patch raises an autoconf error on my computer:

configure.ac:14: error: possibly undefined macro: dnl
      If this token and others are legitimate, please use m4_pattern_allow.
      See the Autoconf documentation.
autoreconf: /usr/bin/autoconf failed with exit status: 1

It would be simpler to add something like:
mq_CFLAGS = \
    -Wall \
    -Wextra \
    -Wundef \
    ...

I accept the part of your patch that modifies src/mq.c. Could you please split your commit in 2 commits: one for the src/mq.c, that I will accept, and one for the autotools part, that is still being discussed?
(In fact I do not know if I will be able to merge only 1 commit out of 2 of a pull request)

Thank you for your contribution.
However I have 2 remarks.

Your patch raises an autoconf error on my computer:

configure.ac:14: error: possibly undefined macro: dnl
      If this token and others are legitimate, please use m4_pattern_allow.
      See the Autoconf documentation.
autoreconf: /usr/bin/autoconf failed with exit status: 1

Interesting; I just checked that the code I pushed (commit dce85a86f7cb9) does not cause that error on my machine (autoconf 2.69, automake 1.16.1, m4 1.4.18), and autoreconf -i works fine here, albeit I seem to recall encountering this error during development of the patch. What autotool versions do you use? Anyway, this issue is expected to go away in the split-out, cleaned-up version of the buildsystem changes (see below).

It would be simpler to add something like:
mq_CFLAGS = \
    -Wall \
    -Wextra \
    -Wundef \
    ...

It would, but the idea of using the AX_* machinery is that it checks for available compiler flags, and only enables flags that are actually supported by the chosen toolchain, so this works for different toolchains (e.g., GCC and clang), as well as for different versions of, say, GCC. You should always get the maximum subset of warning flags supported by the toolchain.

I accept the part of your patch that modifies src/mq.c. Could you please split your commit in 2 commits: one for the src/mq.c, that I will accept, and one for the autotools part, that is still being discussed?

I'll split off the build system changes into their own PR, so they can be considered separately.

(In fact I do not know if I will be able to merge only 1 commit out of 2 of a pull request)

You could do that easily on the git level, but cherry-picking commits doesn't map well (or at all) to the Github model of pull requests, as far as I'm aware.
[ BTW, as a somewhat-related note, you could make use of the rebase option for merging PRs to obtain a simpler resulting git history, but people have different preferences here, just thought I'd mention it ].

1. Your patch raises an autoconf error on my computer:

Do you have autoconf-archive installed? This could be the source of the error (autoconf errors are notoriously unhelpful).

@goeb I've now updated this PR to only contain the code changes. I'll submit another PR with the build system changes when I've come around to polish the changes, including not requiring the autoconf-archive build dependency.

I have manually merged your commit on src/mq.c.

Do you have autoconf-archive installed?

No. After installing autoconf-archive, the error disappeared.


Use url_for to resolve path instead of hard coding it

Using url_for is useful if httpbin is served under a prefix path.  For
example, we can serve httpbin under http://example.org/httpbin.
Previously, we have to replace all hard coded paths, which produces
many diffs, and it makes hard to upgrade httpbin from source.  This
commit uses url_for to resolve correct path even if prefix path is
used.  For digest auth, we have to prepend request.script_root to
request.path to pass authentication.

I'm on board with this up until the change to httpbin.1.html in the templates. I'm not sure that renders the way you expect it to but I'll pull this down tonight and test it locally to make sure it does.
Thanks @tatsuhiro-t!

Thank you for quick response.  I tested both settings: running httpbin directly and with prefix path "httpbin" using DispatcherMiddleware, and both worked fine, but yeah, there is plenty of space I'm missing something.

Looks good to me. Thanks @tatsuhiro-t