DE_load_old vs. DE_load_new

What is the difference between DE_load_old and DE_load_new? In the documentation both are the "Total load in Germany in MW" but in 2015 the values are different.

The data are from two different platforms, although both are maintained by ENTSO-E: DE_load_old is taken from the ENTSO-E Data Portal, which was discontinued in 2016 and replaced by the ENTSO-E Power Statistics. DE_load_new is taken from the ENTSO-E Transparency Platform
I had also noted that the reported values differ. In Germany in 2015, the load reported on the Data Portal is on avarage 3.1 GW or 5.5 % higher than that reported on the Transparency Platform, but this difference also changes every hour.
I am not sure where these differences are rooted, but my best guess is that the reason lies with the different reporting deadlines: Values on the Transparency Platform have to be reported "At the latest H+1 after the end of the operating period". For the Data Portal/Power Statistics, the data seems to be published only after the end of every month, which might allow for more accurate metering.
In priciple, the differences could also be an issue of different definitions of "load": The definition applied on the The ENTSO-E Transparency Platform is as follows (while I believe that the Data Portal/Power Statistics use the same definition, this is not 100% clear):

The sum of power generated by plants on both TSO/DSO networks, from which is deducted

the balance (export -import) of exchanges on interconnections between neighbouring bidding zones.
the power absorbed by energy storage resources.

The ENTSO-E Glossary also has this definition:

The consumption corresponding to the short-term average active power absorbed by all installations connected to the transmission grid or to the distribution grid

excluding consumption for pumping of the pumped-storage stations
excluding consumption of power plants auxiliaries
including network losses.

European electricity consumption data is a tricky subject, and neither of the hourly resolution time series are totally accurate. If you want to dig deeper: A former colleague did a comparison of different sources a while ago, not yet including the ENTSO-E Transparency Platform however.
As a final note: The documentation to the Agorameter estimates that the load values from the ENTSO-E Transparency Platform underestimate the actual situation on average by 8799.8 MW in 2017.

Thank you very much for this detailed answer.

Please note that I have changed the notation of the load columns since version 2018-03-13 of the time series data package:
DE_load_old has become DE_load_actual_entsoe_power_statistics
DE_load_new has become DE_load_actual_entsoe_transparency

ENTSO-E had discontinued the Power Statistics in November 2019. An archive of historical data is now available, with the latest hourly load values provided for 31.10.2019.


Fails with Scala triple-quoted string containing quote(s) inside

Consider this Scala snippet:

val txt = """This text has "quoted" text on it""" + otherString

The quotes inside the triple-quoted string should not unquote or confuse the
parsing.

The entire tripe-quoted string should be formatted as if it was a single-quoted
string with nothing fancy on them.

In other words: the quotes inside a triple-quoted string should not need to be
escaped (because, in Scala, they don't need to; that's the point).

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 15 May 2014 at 11:02
Attachments:

tripple-quoted-bug.png

Scala in code-prettify is implemented as an extension lang-scala.js, and you must explicitly include this script in your page, otherwise you get the default generic language handler (C-like lang that does not accept triple-quoted strings)..
Here is an example to show it is highlighted correctly when lang-scala.js is properly loaded:
https://jsfiddle.net/amroamroamro/wdxgwnsz/
Pay attention to the URL in the "External Resources" (the auto-loader script).


self signed improvements

Motivation
This PR does several things.  First it fixes

https://issues.jboss.org/browse/AEROGEAR-3413 (Check that self signed certificates work)
https://issues.jboss.org/browse/AEROGEAR-3574 (Dismiss the dialog on Auth screen)

It also makes the following changes

Remove self signed and pinning docs from the readme and link to docs.aerogear.org
Removes pinning checks from the app

Progress

[x] Remove pinning calls from main activity and authentication screens
[x] Add Dialog that displays when certificates are not correctly configured
[update readme]

Additional Notes
To test this pr configure openshift for the showcase but do NOT configure the self signed certificates.  The app should be navigable and there should be a dialog on the main page which nags that certificates  are not correct.
The app should also work correctly when certificates are imported.

replaced by #59


Access prefix list redundancies

What is wrong?
In an access prefix list (see https://ethresear.ch/t/account-read-write-lists/285), some prefixes can be unnecessary: If there is a prefix list consisting of 'eth' and 'ethereum', everything that matches 'ethereum' matches 'eth' too. Therefore, 'ethereum' can be safely removed from the list. This can potentially improve efficiency of access checks.
How can it be fixed
Remove such redundancies from access prefix lists in evm.utils.state_access_restrictions.to_prefix_list_form.

I'd like to work on this! Getting started now.

Here's a modified to_prefix_list_form that removes redundant prefixes before appending to o:
+def remove_redundant_prefixes(prefix_list):
+    class Node:
+        def __init__(self, char):
+            self.char = char
+            self.children = {}
+
+    root = Node(None)
+
+    result_list = []
+    for prefix in sorted(prefix_list, key=lambda prefix: -len(prefix)):
+        cur = root
+        created = False
+        for c in prefix:
+            if c not in cur.children:
+                cur.children[c] = Node(c)
+                created = True
+            else:
+                created = False
+            cur = cur.children[c]
+
+        if created:
+            result_list.append(prefix)
+
+    return result_list
+
 def to_prefix_list_form(access_list):
     o = []
     for obj in access_list:
         o.append(sha3(addr) + b'\x00')
         o.append(sha3(addr) + b'\x01')
         o.append(sha3(addr) + b'\x02')
-        for prefix in storage_prefixes:
+        for prefix in remove_redundant_prefixes(storage_prefixes):
             o.append(sha3(addr) + b'\x03' + prefix)
     return o

@adtac mind claiming this on gitcoin.co/explorer real quick?

@owocki claimed with the google form option! (because of the metamask issue we were discussing the other day)

@adtac opening a pull request with your implementation is going to be the best way for us to move this forward.
I'd like to explore is doing this without the Node class as it seems this should work using nested dictionaries.
prefixes = ['a', 'b', 'ab']
# translates to
{
  'a': {'b': {}},
  'b': {},
}

@pipermerriam Is to_prefix_list_form in the source tree yet? If so, I couldn't find it or any related terms when I tried grepping for all kinds of things. Where do I make the change?
And yeah, Node is pointless; removed it and implemented it with plain old dictionaries.
Regarding the complexity: it's O(n log n) (because of the sort), which is linear for all practical purposes in the real world. A naive pairwise search would be O(n^2), so I avoided that. If you feel O(n log n) is too big, lemme know and I'll try to bring it down to linear (although I should note that asymptotic complexity is often misleading; the constants are hidden and a linear solution might be much slower than a log-linear solution).

It can be found in the sharding branch: https://github.com/ethereum/py-evm/blob/sharding/evm/utils/state_access_restriction.py
O(n log n)  is good and close enough to linear for me.

Here's PR merging into sharding: https://github.com/ethereum/py-evm/pull/217

@owocki this issue is complete

@adtac can you claim the bounty at https://gitcoin.co/funding/details?url=https://github.com/ethereum/py-evm/issues/212 via metamask so that i can remit the payment to you?


Use quads for sprites and text

Resolves #111

Codecov Report

Merging #112 into master will decrease coverage by 0.83%.
The diff coverage is 88.88%.

@@            Coverage Diff             @@
##           master     #112      +/-   ##
==========================================
- Coverage   77.37%   76.54%   -0.84%
==========================================
  Files          74       74
  Lines        2290     2281       -9
  Branches      203      203
==========================================
- Hits         1772     1746      -26
- Misses        359      373      +14
- Partials      159      162       +3

Flag
Coverage Δ

#unittests
76.54% <88.88%> (-0.84%)
:arrow_down:

Impacted Files
Coverage Δ

src/standard/sprite/sprite_system.ts
89.79% <ø> (-0.95%)
:arrow_down:

src/standard/text/text_system.ts
89.07% <ø> (ø)

src/shape/polygon.ts
88.23% <83.33%> (-7.66%)
:arrow_down:

src/rendering/texture/texture.ts
90.00% <100.00%> (-10.00%)
:arrow_down:

src/standard/sprite/sprite.ts
88.88% <100.00%> (-2.03%)
:arrow_down:

src/rendering/material/material.ts
59.25% <0.00%> (-37.04%)
:arrow_down:

src/rendering/color.ts
85.71% <0.00%> (-7.15%)
:arrow_down:


Support for Mysql connector

Hi,
Until version 2.0.0 was posible to use Mysql DB for persistence, just adding config properties on compose or application.yaml, but in early releases we got this error message.

Cannot load driver class: com.mysql.cj.jdbc.Driver

please can you add support for mysql db, including mysql connector dependency in pom
Thanks in advance.
Oswaldo

Hello, I think we should add mysql driver as a dependency, so it will be bundled into Docker image...
As you did on issue #307

@ojgutierrez  sounds good :+1:
Feel free to contribute a pull request :rocket:


fuzz/decoder.c: Lower the limits on key checks

These checks still take too long time on clusterfuzz so they are longer than the timeout limit.

This pull request is ready to merge

merged to master, 3.3 and 3.2.
decoder fuzzer doesn't exist on 3.1 or 3.0 branches, we would need a separate PR to merge there
Thanks @t8m


SRVKE-958: [release-v1.9] Cache init offsets results

When there is high load and multiple consumer groups are created and reconciled, we get many dial tcp <IP_ADDRESS>:9092: i/o timeout errors when trying to connect to Kafka.
This leads to increased "time to readiness" for consumer groups.
The downside of caching is that, in the rare case, partitions increase while the result is cached we won't initialize the offsets of the new partitions.
It's based on the existing prober cache

/test 410-test-reconciler-aws-ocp-410

Merging, it passes on 4.10

sounds good!

/cherry-pick release-v1.10

/cherry-pick release-v1.11


Update SUPPORT.md

is this fixing newlines?

Yeah this is the only file that produced this warning for me:
LF will be replaced by CRLF in git


when i click download image, the image is opened in new tab

@JasirMJ would you elaborate this. What browser are you using? Are you clicking the download-icon on the top right corner? I'm unable to reproduce this issue with the latest Chrome, Edge, Safari and Firefox.

I tryed on chrome and mozilla, Not able to download image in single click. Instead of downloading them image is open in another tab,
My backend server is hosted on Aws
And Frontend is hosted on firebase
Image is loaded from backend server to client

Hi there,
Same issue here - in my case my App is here - http://4x4pb.chapterserver2.co.uk/ - if you click an image of a wheel, lightbox shows but download button does not download the image, simply opens the image in the SAME tab. Download attribute is definitely on the link.
Any thoughts?

Apologies... I have now realised that my issue is because the image is on a different domain and that the download attribute only works correctly when on the SAME domain. I'm now using axios to asyncronously get a blob of the image and using that as my large - works fine!

Nice discovery! I did not know that the download only works when on the same domain. This will help to find a solution for the issue.

@aautio, thanks for this lib!
Could you maybe implement a "onClickDownload" param, so we can handle this "cors images download" issue?
Right now, my work around looks like this:
const ModalImageCors = ({ imageUrl }) => {
	const [fetching, setFetching] = useState(false);
	const [url, setUrl] = useState("");

	useEffect(() => {
		setFetching(true);
		if (!imageUrl) return;
		const fetchImage = async () => {
			const res = await api.get(imageUrl, { responseType: "blob" });
			const teste = window.URL.createObjectURL(
				new Blob([res.data], { type: res.headers["content-type"] })
			);
			setUrl(teste);
			setFetching(false);
		};
		fetchImage();
	}, [imageUrl]);

	if (fetching) return null;

	return (
		<ModalImage
			smallSrcSet={url}
			medium={url}
			large={url}
			alt="image"
		/>
	);
};

export default ModalImageCors;

It would be nice if I could put this "turn image into blob" logic on a function and call it when users click "Download" on Modal Image.
Thanks anyway!

Hi, @aautio  are you still supporting this lib? Did I saw no updates in a year, could be the @canove suggestion implemented?

@rootlinux2 yep still supporting this. I haven't yet studied this issue - I guess the blob-approach could work. I'll need to test it first - perhaps later this week.

@aautio thanks, great news! I just clone the repo to see if I can help with any of these issues.

how about if u wanted to title and paragraph with an external link >?

hello, i have the same problem. Has anyone managed to solve it?


Proof of Concept: TrixiParticles on GPUs with CUDA.jl

I spent the last two days building a first prototype for TrixiParticles.jl with CUDA.jl.
The good news:

It works! I can run the oscillating drop simulation successfully on an RTX 3090.

The bad news:

It doesn't work with boundary systems yet.
It's about 30x slower than on the CPU.
It doesn't work with neighborhood search.
It only works for problem sizes smaller than the GPU (#particles < #threads of the GPU). This is because I just set the number of threads to the number of particles instead of using blocks.

Superseded by #474.


text/scanner: Can't properly detect EOF from os.Stdin

What version of Go are you using (go version)?

$ go version
go version go1.4.2 darwin/amd64

What operating system and processor architecture are you using?

OSX 64bit.

What did you do?

I tried to use scanner.Scanner from text/scanner from os.Stdin and I pressed ctrl+D to send a EOF to the program. The program couldn't properly detect EOF. Reference code.

What did you expect to see?

The program should stop immediately I sent the EOF (ctrl+D).

What did you see instead?

The program hung there. Pressing ctrl+D multiple times did stop the program.

A second ^D works. This is always a notorious problem due to lookahead, but it would be good to fix.

:+1: to the fixes


Add 8 missing reborn commands

@razorapid if you can check if there is any error or missing info in the description/examples of this commands please.
.adminrights
chartoint
fnewdir
fremovedir
.leanleftheld
.leanrightheld
.runheld
teamswitchdelay

Looks good to me. Thanks for noticing all those missing commands!


Memory Usage

The difference in memory usage between the LLaMA LLM model and BitNet B1.58 doesn't seem as large as expected, even though they have the same number of parameters. If the LLaMA LLM uses 16-bit precision weights and BitNet uses 1.58-bit weights, the memory usage for LLaMA should be at least 8 times higher. Why is the difference in usage smaller?

I was only focusing on the weights, but I should also consider the memory space used for activations. Since the activations in BitNet B1.58 are 8-bit, the memory savings for internal activations will be limited to 2x. The KV cache is also 8-bit, so the memory savings in that area should also be calculated as 2x.
I would like the author to confirm my assumptions.

The KV cache and the embeddings remain in full precision. For a smaller-size model, they take up a larger proportion of the memory.

@shumingma  Thanks for the clarification.


Building with both JPEGDEC and PNGDEC - compiler warnings

Describe the bug
Compiler warnings
To Reproduce
Create a sketch that includes both of your libraries.
Make sure the Arduino IDE is setup to show you all messages.
Note: I am building for a Teensy.  In the current case for a variant of the Teensy Micromod.
In my current case, I am playing with an Image viewer which displays images that are contained on an SDCard onto the display,
currently it is using JPEGDEC (using your RGB8888), BMP files - using code in sketch, and PNG files.   Currently playing with a WIP Parallel display library for the NT35510 display...   I have been seeing this before but never got around to reporting, but the library that I am currently playing:
https://github.com/KurtE/NT35510_t4x_p
Sketch is in the example: nt35510_tft_picture_view_24bit
You will see a set of messages like:
In file included from D:\github\NT35510_t4x_p\examples\nt35510_tft_picture_view_24bit\nt35510_tft_picture_view_24bit.ino:45:
c:\Users\kurte\Documents\Arduino\libraries\PNGdec\src/PNGdec.h:208: warning: "INTELSHORT" redefined
  208 | #define INTELSHORT(p) ((*p) + (*(p+1)<<8))
      |
In file included from D:\github\NT35510_t4x_p\examples\nt35510_tft_picture_view_24bit\nt35510_tft_picture_view_24bit.ino:42:
c:\Users\kurte\Documents\Arduino\libraries\JPEGDEC\src/JPEGDEC.h:301: note: this is the location of the previous definition
  301 | #define INTELSHORT(p) (*(uint16_t *)p)
      |
In file included from D:\github\NT35510_t4x_p\examples\nt35510_tft_picture_view_24bit\nt35510_tft_picture_view_24bit.ino:45:
c:\Users\kurte\Documents\Arduino\libraries\PNGdec\src/PNGdec.h:209: warning: "INTELLONG" redefined
  209 | #define INTELLONG(p) ((*p) + (*(p+1)<<8) + (*(p+2)<<16) + (*(p+3)<<24))
      |
In file included from D:\github\NT35510_t4x_p\examples\nt35510_tft_picture_view_24bit\nt35510_tft_picture_view_24bit.ino:42:
c:\Users\kurte\Documents\Arduino\libraries\JPEGDEC\src/JPEGDEC.h:302: note: this is the location of the previous definition
  302 | #define INTELLONG(p) (*(uint32_t *)p)
      |
In file included from D:\github\NT35510_t4x_p\examples\nt35510_tft_picture_view_24bit\nt35510_tft_picture_view_24bit.ino:45:
c:\Users\kurte\Documents\Arduino\libraries\PNGdec\src/PNGdec.h:210: warning: "MOTOSHORT" redefined
  210 | #define MOTOSHORT(p) (((*(p))<<8) + (*(p+1)))
      |
In file included from D:\github\NT35510_t4x_p\examples\nt35510_tft_picture_view_24bit\nt35510_tft_picture_view_24bit.ino:42:
c:\Users\kurte\Documents\Arduino\libraries\JPEGDEC\src/JPEGDEC.h:303: note: this is the location of the previous definition
  303 | #define MOTOSHORT(p) __builtin_bswap16(*(uint16_t *)p)
      |
In file included from D:\github\NT35510_t4x_p\examples\nt35510_tft_picture_view_24bit\nt35510_tft_picture_view_24bit.ino:45:
c:\Users\kurte\Documents\Arduino\libraries\PNGdec\src/PNGdec.h:211: warning: "MOTOLONG" redefined
  211 | #define MOTOLONG(p) (((*p)<<24) + ((*(p+1))<<16) + ((*(p+2))<<8) + (*(p+3)))
      |
In file included from D:\github\NT35510_t4x_p\examples\nt35510_tft_picture_view_24bit\nt35510_tft_picture_view_24bit.ino:42:
c:\Users\kurte\Documents\Arduino\libraries\JPEGDEC\src/JPEGDEC.h:307: note: this is the location of the previous definition
  307 | #define MOTOLONG(p) __builtin_bswap32(*(uint32_t *)p)
      |

Note: there are some other warnings as well:
In file included from c:\Users\kurte\Documents\Arduino\libraries\JPEGDEC\src\JPEGDEC.cpp:36:
c:\Users\kurte\Documents\Arduino\libraries\JPEGDEC\src\jpeg.inl: In function 'int JPEGMakeHuffTables(JPEGIMAGE*, int)':
c:\Users\kurte\Documents\Arduino\libraries\JPEGDEC\src\jpeg.inl:1175:37: warning: comparison of integer expressions of different signedness: 'int' and 'unsigned int' [-Wsign-compare]
 1175 |             if (iTable * HUFF11SIZE >= sizeof(pJPEG->usHuffAC) / 2)
      |                 ~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
c:\Users\kurte\Documents\Arduino\libraries\JPEGDEC\src\jpeg.inl: In function 'void JPEGIDCT(JPEGIMAGE*, int, int)':
c:\Users\kurte\Documents\Arduino\libraries\JPEGDEC\src\jpeg.inl:2263:19: warning: unused variable 'ucColMask' [-Wunused-variable]
 2263 |     unsigned char ucColMask;
      |                   ^~~~~~~~~

Expected behavior
No compiler warnings, except my own.
Additional context
There is also a warning in the PNGDEC as well, should probably be its own issue (or punted)
In file included from c:\Users\kurte\Documents\Arduino\libraries\PNGdec\src\PNGdec.cpp:29:
c:\Users\kurte\Documents\Arduino\libraries\PNGdec\src\png.inl: In member function 'void PNG::getLineAsRGB565(PNGDRAW*, uint16_t*, int, uint32_t)':
c:\Users\kurte\Documents\Arduino\libraries\PNGdec\src\png.inl:411:27: warning: 'c' may be used uninitialized in this function [-Wmaybe-uninitialized]
  411 |                         c <<= 1;
      |                         ~~^~~~~
c:\Users\kurte\Documents\Arduino\libraries\PNGdec\src\png.inl:231:13: note: 'c' was declared here
  231 |     uint8_t c, a, *pPal, *s = pDraw->pPixels;
      |             ^

Side question (is there a 24 bit version of this library?)
Library/Sketch mentioned up on the PJRC forum in the thread:
https://forum.pjrc.com/index.php?threads/ili948x_t41_p-a-parallel-display-driver-for-teensy-4-1.72660/page-9#post-347467

If the warnings don't prevent you from building your project, then just ignore them. As far as 24-bit support - no. With both the JPEG and PNG libraries you can easily convert the pixel format in the DRAW callback functions. If you decode a 24-bit PNG image, you will get 24-bit pixels already. For JPEG, 24-bit output is a low priority since you can just use the RGBX32 output.


Every new window I open shows an extension's install page

There's a bug, where after I install a VS Code extension, then I open a new window in VSC with File > New Window, I'll see an extension's install page.
To reproduce: I open the page of an extension in the VS Code Marketplace in Chrome (it was ES Lint at the time), I click Install, it opens up the page in VSC. I click Install on the extension, then Reload, then I close the extension page. Since then, whenever I open a new window, the extension page appears. This stops when I restart VSC.

@joaomoreno Is it could be due to the opening vscode from market place functionality?

/duplicate #34404


Support private submodules (github)

Hi, let me know please how to deploy repository containing private submodules like this
"name":
  "version": "0.0.1",
  "private": true,
  "dependencies": {
    "bluebird": "^2.10.2",
    "classnames": "^2.1.3",
    "efu-core": "git+https://longToken:x-oauth-basic@github.com:OrgName/RepositoryName.git",
    "express": "^4.13.3",
    "jquery": "^2.1.4",
    "jquery-mousewheel": "^3.1.13",
    "lodash": "^3.10.1",
    "malihu-custom-scrollbar-plugin": "^3.1.0",
    "node-uuid": "^1.4.3",
    "path-to-regexp": "^1.2.1",
    "react": "^15.0.2",
    "react-dom": "^15.0.2",
    "slick-carousel": "1.5.9",
    "validator": "^4.2.1"
  },
  "devDependencies": {

Same as here. Best guidance is @suwatch's post: http://blogs.suwatbodin.com/2013/07/deploy-to-windows-azure-using-git-with.html

Would changes here in near future?

Before we talk about changes, did you try going through the guidance in the post?

Yes, I follow the steps.
I stopped here

Simply copy the deployment key (d:\home.ssh\id_rsa.pub) and paste to this user as user key.  Because GitHub only allows one unique key, you may have to delete the deployment key from repo before add as a user key.

So, it means FTP?  And where it's needed to be paste?

Is this way it will work or not?

@suwatch maybe tweak your blog post to make the points above more explicit?

I have updated the blog with link instruction how to copy SSH key.

Simply copy the deployment key (d:\home.ssh\id_rsa.pub) from your webapp (using FTP or Kudu Console) and paste to this user as user SSH key.

BTW, I admitted that I only experimented with submodule (.gitmodules).    In your case, it is npm packages to private git repo and I think this should work the same.   Let us know one way or another.

I've create integration account at GitHub. Added it to organization account.
When setup deployment source - azure don't see organization repositories. Why? (From github I can easy fork)
As a workaround - I've created fork and setup deployment from it.
Follow by you article - after copy publis id_rsa.pub to github account ssh, preliminary - delete deployment key from forked repo. I used following schema -
"packageName": "git+ssh://git@github.com:{Organization}/{Repo}.git",
 ERROR: Repository not found.\r\nnpm ERR! fatal: Could not read from remote repository.\r\nnpm ERR! \r\nnpm ERR! Please make sure you have the correct access rights\r\nnpm ERR! and the repository exists.\r\nnpm ERR! \r\nnpm ERR! \r\nnpm ERR! If you need help, you may report this error at:\r\nnpm ERR!     <https://github.com/npm/npm/issues>\r\n\r\nnpm ERR! Please include the following file with any support request:\r\nnpm ERR!     D:\home\site\wwwroot\npm-debug.log\r\nC:\Program Files (x86)\SiteExtensions\Kudu\55.50526.2249\bin\Scripts\starter.cmd "D:\home\site\deployments\tools\deploy.cmd"

@eyefitu-integration see this page to investigate why you don't see the repos listed.
For the second issue, to isolate from anything npm related, as a test, try just closing the repo in Kudu console using the ssh url. Does that work?

Thank for link - will try.
@davidebbo  you mean cloning repo?

Yes, sorry, typo (corrected). And you can do this in a test folder. e.g. d:\home\TestRepo. This will tell us whether the issue is npm specific, or more generally a problem with ssh key.

When I granted owner permission to integration account - I can to deploy via such url git+ssh://git@github.com:<Organization>/<Repo>.git with tip copy/paste id_pub.rsa and delete deployment key.
Will continue to play with it.
Have some questions now -

how to automate/scale such solution?
Why I need to give integration account owner permission, is it GitHub restriction?
Why it's not working via HTTPS OAuth url like here http://stackoverflow.com/questions/28728665/how-to-use-private-github-repo-as-npm-dependency

I think it's pretty straightforward case to use npm dependencies as private github repos.
Also FYI, some outdated info:
at Github due this https://azure.microsoft.com/en-us/blog/using-app-service-web-apps-continuous-deployment-with-github-organizations/ - they renamed menu item Applications -> OAuth Applications

Will try to answer one by one

how to automate/scale such solution?

You could use the same SSH key for all sites.  On Github, you have to manually setup the user SSH key once.  Then, for each sites, you can propagate the SSH private key across using /api/sshkey.

Why I need to give integration account owner permission, is it GitHub restriction?

For set up the continuous deployment (say thru portal), the account need admin/owner permission since it needs to set up hook and deployment key with Github.   If you do the key manually (like you did above), the account will only need READ permission since it only performs git fetch operation.

Why it's not working via HTTPS OAuth url like here http://stackoverflow.com/questions/28728665/how-to-use-private-github-repo-as-npm-dependency

If you generate the OAuth token and hardcode it with packages json file, it should just work although it may not be wise in term of security (persist and check in security token to a file).

Thanks for answer.
@suwatch  can you please sample of using [/api/sshkey].  (https://github.com/projectkudu/kudu/wiki/REST-API#sshkey) to setup deployment across several sites without using portal?
Don't want to have an integration account with full permissions.
Another one question, does kudu supports private npm registry?

@suwatch can you please sample of using [/api/sshkey]. (https://github.com/projectkudu/kudu/wiki/REST-API#sshkey) to setup deployment across several sites without using portal?

You can find the instruction here.

Another one question, does kudu supports private npm registry?

Does private npm registry mean only reachable from specific machine?  If so, then it may not be possible since the npm install step happens on (and must be accessible from) Azure WebApp VM.

I've tried to use some submodules from private repro with HTTP connection, but it's stucked at "git credential-manager get" and looks like waiting for input. Is it a expected behavior ?

No, it should just fail fast.   We added this code to avoid cred hanging issue.   Could you check if the repo (use Kudu Debug Console) to see if git cred hook exists?

Yes, the hook "git-credential-invalid.sh" exists and the content is the same as the code you mentioned.

Looking at Suwat Bodin's site I get an idea of how to get the SSH keys. but even with the find -name '.ssh' I can not locate a file with an SSH key. Any idea on how i can proceed?
I have a solution that i am trying to deploy. It consists currently of two projects. Everything is in private repositories.

Hi
If the problem persists and is related to running it on Azure App Service, please open a support incident in Azure:
https://learn.microsoft.com/en-us/azure/azure-portal/supportability/how-to-create-azure-support-request
This way we can better track and assist you on this case
Thanks,
Joaquin Vano
Azure App Service


Exactly reproduce 56 layers ResNet on CIFAR10

Test accuracy are 0.9309 and 0.9303 in this patch and Kaiming He's paper, respectively.
The accuracy is the best one of the last 3 epochs (0.930288, 0.930889 and 0.929587), while the original paper select the best one in 5 runs. The dockerfile and log are in: https://gist.github.com/Answeror/f9160145e1c64bb509f52c00014bdb77
The only difference between this patch and Facebook's implementation (https://github.com/gcr/torch-residual-networks and https://github.com/facebook/fb.resnet.torch) are:

The kernel of shortcut with downsampling is 2x2 rather than 1x1. I can't reproduce this accuracy with 1x1 kernel. Note the shortcut does not contain learnable parameters.
I use a BatchNorm after data layer to simulate z-score normalization. Although subtract (127, 127, 127) and divide 60 works equally well.

Some details affect the accuracy:

Z-score normalization of the input.
Weight decay of all parameters (weight, bias, gamma, beta). See comments in train_cifar10_resnet.py for details.
Nesterov momentum
fix_gamma=False in BatchNorm (gamma is necessary because of the weight decay of the conv weight)
Initialization
4 pixel padding

And thanks #1230 (@freesouls) and #1041 (@shuokay) to provide preliminary implementations.
update@2016-06-08
With #2366 and a batch size of 64, I got an accuracy of 0.939704 after 200 epochs on 2 GPUs.
Note, the accuracy is strongly affected by the batch size, the more GPU you use, the smaller batch size should be.
See https://gist.github.com/Answeror/f9160145e1c64bb509f52c00014bdb77#file-resnet-dual-gpu-log for the full log.

@mavenlin @antinucleon

May I ask if you are using cudnn for this experiment? The 1e-5 eps get gives bad_param error. By adding 1e-9(my coworker used that value), it works well now.

@Godricly Thank you for your report. Pending PR #2366 fixed this issue with cuDNN v5 (2e-5 is ok).


SWC-6602: Validate ColumnModel maxSize when editing a view schema based on columns generated from annotations

useState to display warning on runtime

pull defaultAnnotationModel from TableColumnSchemaEditor to locate matching columnModel

On Production

On Feature

Can you also add tests to verify that this warning appears when the form data size < recommended size, and doesn't appear when form data size >= size?


USHIFT-1363: tools to build multiple images for ci

This PR adds a framework for managing "test scenarios". It is possible to independently define images, VMs, and tests, then mix and match them to produce scenarios like "test upgrade and rollback from MicroShift 4.13 to 4.14 on RHEL 9.2". The PR includes a few scenarios as examples. The scripts have been tested on a RHEL hypervisor using remote access similar to what we will do in CI. The next phase of implementation will include some wrappers and integration scripts for linking in to CI, but these scripts are usable as-is for developers to run locally.

/hold cancel


Suspend OA and OA UI for PT

ACs:

Remove OA on PT applications
Remove OA screen for Ministry and Student for PT students

@HRAGANBC I have #3031 to remove access to the Overaward screen for student users.

This is a dup closing


Fix jinja templates to render component and property descriptions as JSON-safe strings

Fixes #2461
This also fixes a recent bug (no corresponding issue currently open) where certain Airflow operators were causing an error popup with JsonDecodeError due to unescaped characters in property descriptions.
What changes were proposed in this pull request?

Converts pipeline descriptions and node comments (in the airflow DAG template) to multi-line strings (""") using the jinja replace filter and escapes any other existing instances of """ to avoid black formatting errors after DAG render.

DAG:

Airflow GUI:

Converts property descriptions (in the canvas properties template) to JSON-safe strings using jinja's tojson|safe filter. This allows un-escaped characters to be caught before descriptions are converted to JSON in the properties API request.

Python operator file (source):

Properties panel:

How was this pull request tested?
A case has been added to an existing test for each scenario above.
This PR will remain in draft form until more manual testing occurs.
Developer's Certificate of Origin 1.1
   By making a contribution to this project, I certify that:

   (a) The contribution was created in whole or in part by me and I
       have the right to submit it under the Apache License 2.0; or

   (b) The contribution is based upon previous work that, to the best
       of my knowledge, is covered under an appropriate open source
       license and I have the right under that license to submit that
       work with modifications, whether created in whole or in part
       by me, under the same open source license (unless I am
       permitted to submit under a different license), as indicated
       in the file; or

   (c) The contribution was provided directly to me by some other
       person who certified (a), (b) or (c) and I have not modified
       it.

   (d) I understand and agree that this project and the contribution
       are public and that a record of the contribution (including all
       personal information I submit with it, including my sign-off) is
       maintained indefinitely and may be redistributed consistent with
       this project or the open source license(s) involved.

Could you make the title string to fit the commit title guidelines


Broken test framework!

This is not a PR to be merged; rather it's a MWE reproducing a terrible, terrible bug.
I don't know the reason, but if we have several await's inside a test, the test framework is not awaiting until the end.
Runt the test added:

via f test test/timer_test.dart
via f test (run all tests)
via IDE
github actions are green

In all cases, the test go up to a certain number, never past 8. And therefore the test passes, because the assertions in the end never run.
Unless I am doing something really stupid, this could be a MASSIVE bug with flame_test (but more likely with flutter_test itself), that compromises our entire test suit and stability.
I'm using flutter stable btw. I will try to repro this with flutter test directly, bypassing flame_test -- I imagine the issue has to come from below as we don't seem to do anything at the flame_test level that could cause this.

Can you reproduce the same thing using flameWidgetTest?

Do you mean widgetTest? that one seems to work, as far as I am testing

I think so, The one that uses GameWidget, and not only the Game Class, I don't quite remember the right method.
This maybe be worth having an issue open, I would enjoy investigating this.

Please do, I am at odds.
I wonder if test simply doesn't support async. Looking at the signature turns out dart is not super type safe wrt to whether lambdas are async or not. Test just expect a lambda that returns dynamic instead of a non-future T. but looking at the code for it, it does await the lambda and seem to be build in a way that would implicate this should work.
Also testWidget (which does explicitly expect a future lambda) just calls test just like we do.
Finally, it seems to be random how long it rusn for; different methods give different results and even the same method varies. I even got it to run to the end and fail once (not sure how).

Closing in favor of #1166


[feat] DOGE Core rewards

Feature Request
Describe the Feature Request
Include in the transaction fee a small reward for running a Core node
Describe Preferred Solution
Each transaction should include a small amount airdropped to people running Core nodes.
Describe Alternatives
No alternatives are known at this time.
Related Code
No code suggested
Additional Context
Many people do not own miners but still want ways to earn DOGE, running nodes in other currencies such as BTT results in rewards.  Running miners is energy-intensive and does not yield a net positive earning unless a high-powered miner is being run.

Rewards will strengthen the node system by incentivising the use of high-speed networks to run nodes.  It will also encourage more people to buy into the coin.


NSTimer needs a running RunLoop;

Check if the current thread's is running if not run the RunLoop until the timer has a chance to fire.

@edingwell any additional information I can provide?


Source command

Closes #331
Created !source command, that supports 3 types of source items: Help command, command, cog. Send embed that has a link to GitHub, help text of the item and when command/cog, are you able to run it in this context.

Before a proper review,
The embed when not found could use some text instead of just being empty like that.

And what's the reason for the two identical links in the title and the footer?

@Numerlor Is this better now? I removed the link from the title and made converter raising BadArgument instead just sending bot repo when invalid argument provided.

These review is now addressed.


[WIP] use pool_names.yml to populate vars in upgrade.yml

pool_names.yml has all tasks those populate vars, so we use
pool_names.yml to populate vars instead of using ceph-osd role.
Also prevent running pool_names.yml more than one time.

Can one of the admins verify this patch?

"ok to test" to accept this pull request for testing
"test this please" for a one time test run
"add to whitelist" to add the author to the whitelist

Can one of the admins verify this patch?

"ok to test" to accept this pull request for testing
"test this please" for a one time test run
"add to whitelist" to add the author to the whitelist

Can one of the admins verify this patch?

"ok to test" to accept this pull request for testing
"test this please" for a one time test run
"add to whitelist" to add the author to the whitelist

upgrade passed on cisdev with this fix

ok to test


Getfasta fixes

Fixes:

FASTA and BED files to test IUPAC processing added.
Print a warning if a FASTA index file is outdated.

Thanks!


Add support for adding support for DLang LSPs

Adds entries for DLang as needed by https://github.com/LunaTheFoxgirl/lapce-dlang

Thanks!


chore: default dataAggregation to high + update docs

Description
As title
Related Issue
If this pull request is related to any issue, please mention it here. Additionally, make sure that the issue is assigned to you before submitting this pull request.
Checklist

[ ] I have read the contributing documentation.
[ ] I signed and signed-off the commits (git commit -S -s ...). See this documentation on signing commits.
[ ] I have correctly attributed the author(s) of the code.
[ ] I have tested the changes locally.
[ ] I have followed the project's style guidelines.
[ ] I have updated the documentation, if necessary.
[ ] I have added tests, if applicable.

Screenshots (if applicable) or Testing Completed
Please add any relevant screenshots or GIFs to showcase the changes made.
Additional Notes
Add any additional notes or context about the pull request here.

Please refer to the CONTRIBUTING.md file for more information on how to contribute to this project.

@nddq Can you update the description to explain the decision making here?


adding support for KML Styles

@JamesMBallard please review
Now that we have KmlStyle for 100.7, this can be added to the sample to closer match the common design
QML sample also was restructured slightly to make the code more declarative

@JamesMBallard great points. Updated with your suggestions to better reflect the object model


[BUG] Allow Native repositories to be configured with. an event mapper.

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
If you try to associate an event mapper with a native repository, using admin command /local-repository/event-mapper-connection, then the request is rejected as we are not a repository proxy. Further more the event mapper is not initialised unless for native repositories.
The reason an event mapper is useful along side a native repository - is that it can feed content from a 3rd party repository directly into the native repository.
Expected Behavior
loosen the logic to allow native repositories to be configured and started with an event mapper
Steps To Reproduce
Configure an in memory repository and issue a post " /local-repository/event-mapper-connection". It will be rejected
Environment
- Egeria:
- OS:
- Java:
- Browser (for UI issues):
- Additional connectors and integration:

Any Further Information?
No response

I suggest you use the RepositoryProxyConfigurationClient for configuring the repository proxy since it limits the api to the methods that are valid for the repository proxy.   See https://egeria-project.org/services/admin-services/overview/
Using the

You can update the local repository mode in the configuration document to switch it to a repository proxy to allow the event mapper to be set up.

I recommend that this issue is closed since:

It is possible to configure a native repository in a repository proxy - it just needs the correct commands to be issued.
Configuring an event mapper in a metadata access store is invalid and so the error message and exception is correct.

It is important that the correct commands are issued so that the local repository mode is set up correctly - otherwise the OMRS does not initialise correctly.  For example, it only sets up an event mapper if the local repository mode says it is a repository proxy.  The admin commands are set up to guide the administrator to use the right commands for the right purposes and provide helper classes to simplfy the configuration for common cases.
For example, the in-memory admin command is a helper command that sets the in-memory repository up in a metadata access store.  It is more than just setting up the in-memory repository connection.  To this end it is documented in the pages that describe setting up a metadata access store and not in the documentation for a repository proxy.
It has taken a lot of work to get the design to this point where there is clarity of what to configure for each type of server.  There is also flexibility in being able to used the repository connectors in different types of server.  There is no need to muddy the API in this way.
If you want to persist in this design change then it needs to be reviewed by the community and it needs to include all of the documentation updates to people can assess the impact of this change on consumability.


Verbose error logging on file resolve error in template.go

It would be useful for the user to get a verbose output when template file resolve gets an error.
For example, a simple file not found error, or path resolution error can be displayed instead of "this path is invalid".
This would avoid new users making mistakes like in #115 .

This can be simply added as a string to template.go#115
color.Red("%v is invalid.\n%v\nPlease input again: ", path, err)


Mistake in Quickstart Tutorial

https://docs.manim.community/en/stable/tutorials/quickstart.html

The explanation says that the rotation is 360 degrees but as can be seen from the corresponding code below, the rotation is actually 180 degrees.
class DifferentRotations(Scene):
def construct(self):
left_square = Square(color=BLUE, fill_opacity=0.7).shift(2 * LEFT)
right_square = Square(color=GREEN, fill_opacity=0.7).shift(2 * RIGHT)
self.play(
left_square.animate.rotate(PI), Rotate(right_square, angle=PI), run_time=2
)
self.wait()
The interpolation of a rotation of 360 degrees results in a complete lack of any animation(can be observed by changing rotation to 2*PI for left_square.

This was meant for ManimCommunity/manim, and should be closed here.


[Invitation] accepted

This should be   NO_CONTENT = 204,
201 ---- > 204

And this !
201 is wrong


Upgrading to golangci-lint 1.48.0 version.

Codecov Report

Merging #42 (59cd753) into main (6e02b9b) will increase coverage by 0.05%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main      #42      +/-   ##
==========================================
+ Coverage   47.31%   47.37%   +0.05%
==========================================
  Files          37       37
  Lines        3462     3462
==========================================
+ Hits         1638     1640       +2
+ Misses       1756     1754       -2
  Partials       68       68

Flag
Coverage Δ

go_tests
47.37% <ø> (+0.05%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

xopjson/jsonlogger.go
58.83% <0.00%> (+0.38%)
:arrow_up:

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


Dynamic tank losing contents

I setup a dynamic tank, that contained about 700 buckets worth of lava, I went to get more lava (was about 80% filled) came back: tank entirely empty. GUI was working just fine, but no more contents(lava) what so ever... nothing hooked up to it, so nothing that could have drained it, it just vanished. I tried to reproduce it in a single player world, but to no success.
running the latest mekanism (just updated this morning to: 1.7.10-<IP_ADDRESS>.jar) the dynamic tank was crafter and setup AFTER I updated, so has nothing to do with the update process, so also no idea if this happened in the previous version. I kinda don't want to use the tanks anymore if it can lose all its contents at random times.. kinda put a big dent in my lava supply.
Running on a MC Server
Mekanism-1.7.10-<IP_ADDRESS>.jar
MekanismGenerators-1.7.10-<IP_ADDRESS>.jar
MekanismTools-1.7.10-<IP_ADDRESS>.jar

There were some multiblock inventory sustaining issues just a few releases back, there's a good chance that was the source of your issue.  You shouldn't run into this anymore with 8.1.7.

The version I am running, and ran when this error occurred is: Mekanism-1.7.10-<IP_ADDRESS>.jar so it doesn't seem like it has been fixed. I made the blocks, and the multiblock AFTER i updated to <IP_ADDRESS>. So it seems this is still an issue?


data sanitising: ip2long32, ip2ulong, long2ip32 (FIXED RESUBMIT of #2152)

Self explanatory. If these functions find themselves trying to convert non-int data (or an x64 int with non-zeros in any bits >32) to dotted IPv4, or non-dotted IPv4 to integer IPv4 values, something's wrong and they shouldn't return a value that looks like they succeeded.
The original PR caused issues with VPN. This was because, to check the presence of any bits beyond #32 were zero (if INT was 64 bits or larger), the operator >>32 was used. Unfortunately this was undefined on x32 platforms. (See https://forum.pfsense.org/index.php?topic=104175 ).
The fix below was tested on x32, on the same thread.
TEST PR'ed IN #2152 (FAILS ON x32):
return ((is_int($ip) && ($ip >> 32) == 0) ? long2ip($ip & 0xFFFFFFFF) : '');
TEST NOW USED (SEEMS RELIABLE ON ALL SYSTEM INT SIZES):
return ((is_int($ip) && ($ip & ~0xFFFFFFFF) == 0) ? long2ip($ip & 0xFFFFFFFF) : '');
Other than this line and a comment, this code is identical to PR #2152

I reverted this because it broke things again. First thing noticed was services_dhcp.php range checking started failing (on 64 bit at least, didn't check 32 bit). Maybe other things, didn't dig much into it beyond that. Please make sure every area that uses these functions still works on 32 and 64 bit before re-submitting.

just wow. Okay, will do.....
What is the best (as in, most reliable) way to test if x32 will work, if the hardware available is x64 or x64 + VM on x64?

@cbuechler - the code is correct.
What it's doing is precisely what it * should * be doing - namely, identifying invalid data and bad calculations in the originating/calling code.
In this case the issue is a bug in services_dhcp.php.  The $input_errors code tries to calculate the end point of the range by using (start_IP) || (~NETMASK).
This method isn't, and never has been, valid code on x64, because on x64 inverting an IPv4 netmask would produces a 64 bit integer with 0xFFFFFFFF in the high bits, which is automatically bad data for any kind of IPv4 conversion function.  So naturally now we're checking for bad data, it gets-correctly-picked up. This code had only worked in the past because - by chance - the high bits are silently ignored or ignored by some other oversight, either in PHP's function long2ip() or in the old version of long2ip32(). That's not guaranteed behaviour and isn't correct behaviour, as any integer >=2^32 should give an error when converting to an IPv4 IP, and any netmask that is logical NOTted on x64 will give such a value.
It should use gen_subnetv4_max() or gen_subnetv4_max to get this value - which does correctly use (~$sn & 0xFFFFFFFF) rather than ~$sn   .....
Not sure how pervasive this logic bug is. Please re-add this code and in case of future issues check for the same problem in calling code.....

Re-adding this code without first (or simultaneously) fixing all of the calls it breaks would be irresponsible. It doesn't matter if it's technically correct, if it introduces breakage it won't be imported so we can trip over the land mines it plants later.

hehe - I was reporting the issue for better ideas as you responded. See redmine https://redmine.pfsense.org/issues/5702


fix: Check child's stderr

Fix issue #74 "qemu-system-aarch64: Failed to put registers after init: Invalid argument".

Hi, thanks for the PR!
This looks like the same issue/fix as in https://github.com/danobi/vmtest/pull/72. Can you please check if that PR fixes your issue? If so, I can cut a release

Yeah, #72 fixes it, too.


Contributing guideline - tests and their dependencies.

CONTRIBUTING.md says:

To run the tests run nosetests in the root of the repository.
They require the rosdistro library, available on Ubuntu with the ROS repositories as python-rosdistro or via pip as rosdistro.

The tests actually need more dependencies than rosdistro. That should be somehow reflected in this text excerpt. Probably linking to .travis.yml for an idea about what to install?
When (after installing dependencies) I run nosetests in the root of repo, nothing happens, no tests are executed (well, noestests tell me 1 test ran, but the runtime is ~0 sec).

Thanks @peci1 for the report. Indeed it looks like we added several tests that require specific modules without adding them to the CONTRIBUTING.mg... Linking to the travis file is a good idea indeed this way we're sure the instructions stay up to date :+1: .

When (after installing dependencies) I run nosetests in the root of repo, nothing happens, no tests are executed (well, noestests tell me 1 test ran, but the runtime is ~0 sec).

Hmm that's surprising, I just tried this in a fresh container and got the full suite of tests running:
# apt upgrade && apt install git dirmngr lsb-release
# sh -c 'echo "deb http://packages.ros.org/ros/ubuntu $(lsb_release -sc) main" > /etc/apt/sources.list.d/ros-latest.list'
# apt-key adv --keyserver hkp://ha.pool.sks-keyservers.net:80 --recv-key 421C365BD9FF1F717815A3895523BAEEB01FA116
# apt update && apt install python-pip
# pip install PyYAML argparse catkin_pkg ros_buildfarm rosdistro nose coverage yamllint unidiff rosdep PyGithub
# git clone https://github.com/ros/rosdistro
# cd rosdistro/
# nosetests -s
[...] # plenty of console output
Ran 6 tests in 107.476s

OK

Do you see any difference with what you tried on your side ?

# cd rosdistro/
# ls
CONTRIBUTING.md  groovy  index.yaml  jade     lunar      releases  ros.key  test
doc              hydro   indigo      kinetic  README.md  rosdep    scripts
# nosetests -s
.
----------------------------------------------------------------------
Ran 1 test in 0.067s

OK

nosetests run in Python 2.7

nosetests version 1.3.7

Unfortunately that is not enough information to help us figure out what your system setup is, can you provide the OS you are running, a pip freeze output as well as the history of commands you run before trying to run the test?

Ubuntu 14.04.5 LTS 64bit, Python 2.7.6.  Runs on Intel i7 Haswell, Dell Latitude E6450.

I tried in trusty and without anything installed it runs fine.
Installing a few of the ros packages you have on your system (python-catkin-pkg and python-ros-buildfarm for example) I got nose to fail (not being found). Removing them from pip, installing it from apt and running nose from /usr/bin/nose worked.
Didnt get to the point where it runs but doesnt find any test yet though.
There is definitely something fishy when you have the same python packages installed from apt and from pip. I don't have much more time to look into it right now so I'd advise you to run it in a container for now or to try installing and run the python3 version of these packages to be sure to avoid weird mix-n-match of packages.
Please keep us updated if you find out where the problem comes from

debug_nose.txt
A log from running nosetests in the rosdistro dir.
This was achived by the following .noserc:
[nosetests]
verbosity=5
with-doctest=1

It seems only the test_url_validity test is picked up.

Found the culprit!
It was the pip package nose-unittest, which installs a plugin that switches the testing to unittest mode. What's weird, according to the documentation I understood that should only happen when you pass "--with-unittest", which I don't do.
I also don't know how did the nose-unittest package get to my system, I don't use it directly anywhere, but maybe uninstalling it broke something.
Could you please test that you can reproduce my problem by installing nose-unittest?

Thanks @peci1 for investigating.
I can reproduce the issue. If I follow the steps in https://github.com/ros/rosdistro/issues/16336#issuecomment-343633404 I can get the tests to run.
If I install nose-unittest and run nosetests again I get the same behavior as you described in https://github.com/ros/rosdistro/issues/16336#issuecomment-343634242. (same thing if I istall the packages via apt but nose-unittest via pip)


feat: add appendix D (observability)

adds small appendix section to spec that defines how particular SDK fields are mapped to the recently merge OTel semantics
this allows providers and hooks to export OTel data consistently for easy interop
adds a couple supporting glossary liks

Moving to draft for now based on some ongoging OTel sem-conv discussion: https://bitbucket.lab.dynatrace.org/projects/DEUS/repos/semantic-dictionary/pull-requests?create&sourceBranch=refs%2Fheads%2Ffeature%2FOA-40566-update-semantic-dictionary-feature-flag-events&targetRepoId=9646


terminate called after throwing an instance of 'csvstream_exception'

When trying to build example with cmake it gives this error
terminate called after throwing an instance of 'csvstream_exception'
  what():  Error opening file: dataset.csv
Aborted (core dumped)

Can any please help !! Thank you.

Can you give this a try and post the output?
$ git clean -xdi
Would remove the following items:
  csvstream_test         example1               example3
  csvstream_test.o       example1.o             example3.o
  csvstream_test.out     example2
  csvstream_test.passed  example2.o
*** Commands ***
    1: clean                2: filter by pattern    3: select by numbers
    4: ask each             5: quit                 6: help
What now> 1
Removing csvstream_test
Removing csvstream_test.o
Removing csvstream_test.out
Removing csvstream_test.passed
Removing example1
Removing example1.o
Removing example2
Removing example2.o
Removing example3
Removing example3.o
$ make test
/Library/Developer/CommandLineTools/usr/bin/make CXXFLAGS="-std=c++11 -pedantic -Wall -Wextra -Werror -Wconversion -Wsign-conversion -O3 -c" \
     \
    csvstream_test.passed
g++ -std=c++11 -pedantic -Wall -Wextra -Werror -Wconversion -Wsign-conversion -O3 -c csvstream_test.cpp -o csvstream_test.o
g++ csvstream_test.o  -o csvstream_test
./csvstream_test > csvstream_test.out 2>&1
touch csvstream_test.passed


Create simple installation scripts

For:

Community
Enterprise

Community: https://github.com/slipstream/SlipStream/blob/master/install/slipstream-community.sh
Enterprise: https://github.com/slipstream/SlipStream/blob/master/install/slipstream-enterprise.sh


config.yaml documentation

Looks like I need a config file somewhere?

We have fixed this bug, please pull the latest version of the code, thanks!


libs: Add aio.api.github

@kfaseela this should be ready for review
its a partial implementation of the github api, but its done in such a way that it should be ~easy to cover more parts as we might require
there are a lot of unit tests, so probs better to focus on the actual code
once this lands we can land #143 which will not require nearly as much code

@kfaseela this one should be ready for final review - i have managed to remove the workarounds that i had


feat: handlersjs-logging

Don't forget to update ci.yml :-)

Don't forget to update ci.yml :-)

@wouteraj should be updated. Can you check if it's correct?


[HUDI-1635] Improvements to Hudi Test Suite

What is the purpose of the pull request
Brings in Hudi test suite enhancements.

Brief change log
- Correcting date partitions to start from 1970
- HiveQueryDAG marshalling not working fix
- RollbackNode to support multiple commits
- Fix testPayloadGeneratorWithTimestamp
- DAG optimization for test suite
- Fix for root node not having dependencies

Verify this pull request
This change added tests and can be verified as follows:

verified by launching hudi-test-suite DAG, with the newly introduced parameters.

Committer checklist

[ x] Has a corresponding JIRA in PR title & commit

[x ] Commit message is descriptive of the change

[ x] CI is green

Codecov Report

Merging #2628 (e395853) into master (f11a6c7) will increase coverage by 17.89%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             master    #2628       +/-   ##
=============================================
+ Coverage     51.54%   69.44%   +17.89%
+ Complexity     3284      363     -2921
=============================================
  Files           445       53      -392
  Lines         20381     1944    -18437
  Branches       2109      235     -1874
=============================================
- Hits          10506     1350     -9156
+ Misses         9003      460     -8543
+ Partials        872      134      -738

Flag
Coverage Δ
Complexity Δ

hudicli
?
?

hudiclient
100.00% <ø> (ø)
0.00 <ø> (ø)

hudicommon
?
?

hudiflink
?
?

hudihadoopmr
?
?

hudisparkdatasource
?
?

hudisync
?
?

huditimelineservice
?
?

hudiutilities
69.44% <ø> (-0.11%)
0.00 <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ
Complexity Δ

...in/java/org/apache/hudi/utilities/UtilHelpers.java
64.53% <0.00%> (-1.17%)
33.00% <0.00%> (ø%)

...g/apache/hudi/exception/HoodieRemoteException.java

.../hadoop/realtime/AbstractRealtimeRecordReader.java

...mon/table/log/block/HoodieCommandBlockVersion.java

.../java/org/apache/hudi/common/util/StringUtils.java

...he/hudi/hadoop/SafeParquetRecordReaderWrapper.java

...org/apache/hudi/cli/utils/InputStreamConsumer.java

...i/hive/SlashEncodedDayPartitionValueExtractor.java

...udi/operator/partitioner/BucketAssignFunction.java

...ache/hudi/cli/commands/ArchivedCommitsCommand.java

... and 383 more

@modi95 can you review this ?

@nbalajee these changes look good to me. Can try doing the following:

Run schema evolution test suite in the docker setup (step-by-step guide available in test-suite readme)
Add instructions to the test-suite readme on how to use schema evolution


npm README shows incorrect install instructions

should probably publish a new version to npm as it currently shows:
npm install stdlib --save

whereas it is correct in the repo README  as per 84c46c9b3e3d8bfa89e3ff2c1ba3281906bca783

fixed. thanks for the catch!


Illegal instruction

I have a strange problem with KMS on my Debian server:

What could be wrong and how to solve this "Illegal instruction" thing?

My guess would be your server doesn't have AES-NI, or has AES-NI explicitly disabled in UEFI/BIOS. What CPU is it? I can at least tell you if AES-NI should theoretically be supported. Otherwise try checking your UEFI/BIOS for an option to enable AES-NI or some other "Enable AES acceleration"-style option.
If your server has a CPU which actually doesn't support AES-NI, and you built the KMS with the RUSTFLAGS to enable it i.e.
export RUSTFLAGS=-Ctarget-feature=+aes

...you will need to disable those. Note that the software fallback it will use instead has not been extensively tested for potential security issues that can arise in a soft AES implementation (e.g. cache timing sidechannels).

Alternatively, you can try to figure out which illegal instruction was executed by disassembling the program with gdb or lldb after it catches the SIGILL. In gdb that'd look something like:
$ gdb tmkms
(gdb) run yubihsm keys list
[...] (keep `cont`inuing until you get to the SIGILL)
(gdb) disassemble

Here is an upstream issue to do runtime detection for AES-NI support:
https://github.com/RustCrypto/block-ciphers/issues/25

I ran into the same issue and it was AES-NI. (The Core2Duo I was using doesn't support the AES instruction set.) Disabling it resolved this issue. (I had others, but unrelated ones.)


drt: separation of drc checking in pin access

Secure CI: Running...
ISPD CI: Running...
Creates the functions isViaViolationFree() and isPlanarViolationFree() to separate Design Rule Checks from the rest of the code.

Maybe better names would be hasPlanarViolations and hasViaViolations? It would require inverting the return to be 0 for no violations.


dts: npcx: add device-tree files of npcx9 series.

This PR can be separated into three CLs to introduce NPCX9 series.

Replaces series-prefix "npcx7-" with family-"npcx-" for npcx dts nodes such as 'espi-vws-map' and 'miwus-int-map'.
Arrange device-tree files of npcx7 series by following steps:
Move device-tree declarations of npcx family to npcx.dtsi.
Leave specific device declarations of npcx7 series to npcx7.dtsi.
Move chip series related mapping tables such as npcx7-miwus-wui-map.dtsi and so on to npcx/npcx7 folder.
Move common device-tree declarations used in each npcx series to npcx-miwus-wui-map.dtsi and so on to npcx folder..
Add device-tree files of npcx9 series.

Then, the folder tree of nuvoton npcx family will be:
dts/arm/nuvoton
        |--npcx
        |    |--npcx7
        |    |    |--npcx7-miwus-wui-map.dtsi
        |    |    |--npcx7-alts-map.dtsi
        |    |    |--.....
        |    +--npcx9
        |    |    |--npcx9-miwus-wui-map.dtsi
        |    |    |--npcx9-alts-map.dtsi
        |    |    +--.....
        |    |--npcx-miwus-wui-map.dtsi
        |    |--npcx-alts-map.dtsi
        |    |--.....
        |
        |--npcx.dtsi
        |--npcx7.dtsi
        |--npcx7m6fb.dtsi
        |--npcx9.dtsi
        +--npcx9m3f.dtsi

LGTM

I set the other comments as "resolved". Thanks for reviewing.

Any suggestion for this PR? Or could we merge it since it got two approvals? Thanks.

One more request. When I tested this code I ran into issues with ecst.py. Could you also add the appropriate chips to CHIPS_INFO in ecst_args.py?

Nuvoton split npcx9 driver implementation into three PRs. The related ecst parameters for npcx9m3f and npcx9m6f are located in the first CL of PR 35440. If you still encounter issues after applying this, could you give me the build log? Thanks.


Feature/create pip package

In order to facilitate running of snowchange in CICD pipeline, I needed to create a pip installable version of the script.
This pull request moved the main script into a package folder and created a setup.py file.
I also published this version to pypi. I am happy to turn over the pypi name, but I definitely need to figure out how to install via pip before I transfer.

Thanks for the PR @trejas , we're just reviewing another PR which will impact this one, but intend to accept this one

Hi @trejas  we've finished merging that other PR now. Would you mind resolving the conflicting files? (should be very minor)

Hey there @trejas, just wanted to follow-up on @jamesweakley question from PR #8. Can you please transfer the ownership of https://pypi.org/project/snowchange/ to James (his pypi username is jamesweakley)? Once that is done we can fine finalize the code changes and get snowchange out there as a pip package! Thanks.

@jeremiahhansen Looks like you had no answer from @trejas. Have you tried the gmail address listed on his GH profile?
I would like to see this tool deployed as a pip-package, but I'm not willing to use a non-official one 😄

I've contacted Traey and he's agreed to do this, I'll comment here again when it's done

Hi guys. Many apologies for the delay. I don't check my messages on github that often. I have sent an invite to add @jamesweakley as an owner of the pypi name. He should have it in his inbox now.
Also, please let me know if there are any modifications that need to be made to here. This pip package is in use at a customer, so please do not delete the deployed versions.

This is great! I can see its already available in pip.   Looking forward to this being mainlined.

Closing this PR as it's a duplicate of PR #8. That code has just been released and the package on PyPI has been updated!


"crypto" not present on Vite

Using Puck with Vite yields an error:
Module "crypto" has been externalized for browser compatibility

A workaround by using the vite-plugin-node-polyfills package (thanks @leweyse 🙏)
We should either document this officially, or patch it.

Related to https://github.com/vitejs/vite/issues/792

Removing the crypto lib in 0.13.1


Upgrade to 10.1.3 (most recent as of June 11, 2015)

Same as @adampope's pull request but just with the most recent version 10.1.3 rather than 10.1.1

I'm worry this corrupts Redactor License. We cannot upgrade redactor assets anymore. Moreover we need to downgrade it to latest free version soon.

Sorry, it's been a while since I looked but I thought the licence issues had been resolved between this Gem and Redactor? We own a licence so can use it in our projects. Are there issues with packaging it in this way?


Blocks don't kill you in platformer mode when crahing onto them

I refuse to tell which device I'm using and which version, you know that by now for sure
https://github.com/famidash/famidash/assets/110625168/e2c8a3ea-91d1-4ee3-ac25-0d98a3e596ae

Thats intentional and an added feature

Thats intentional and an added feature

Oops sorry


Callback in joinExperimentsWithCallback is called multiple times

Integration Method: Manual
Xcode Version: Version 8.0 (8A218a)
Library Version: 3.0.3
Platform: iOS
Device: All
Language: Swift and Objective-C
[] A/B Testing Manually Disabled? (DISABLE_MIXPANEL_AB_DESIGNER)
Description:
Call to joinExperimentsWithCallback with bad network condition (or no network) results in callback called more than once
Is this an intended behaviour? I looked at the method looks like the completion handler block is called at least once when error happens and then called again just before the method exits at the bottom.
Expected Behavior:
I would expect the callback to only be called once in all situations

Thanks for pointing that out! It was a bug where the completion handler would be called an additional time with old information in the case of an error. It's been fixed in https://github.com/mixpanel/mixpanel-iphone/pull/587, and should come out in the next release.


[Feature Request] Allow dot notation in render method

It would be nice to use Laravel's default dot notation when passing views in subfolders to the render() method.
// currently
return Inertia::render('User/Create');

// proposal: like we are used to in laravel
return Inertia::render('user.create')

This would be nice

Blade files are generally lowercase, so I think that makes more sense. Since Javascript components are often title case, I'm not sure this makes sense. I get the desire to be consistent with Laravel, I'm just not convinced adding this extra complexity is worth it.
Thanks for the suggestion either way! 👍

Then what about simply Inertia::render('User.Create')? That way you are consistent with laravel dot notation, but still include the title case of Javascript components...


Does Android kotlin project scanning work properly?

When I scan our project through GitHub Actions scanner I see the following in the scan output:
[02:32:26] INFO     Scanning /github/workspace using plugins ['credscan', 'kotlin', 'depscan']
           INFO     ANDROID_SDK_ROOT or ANDROID_HOME should be set for automatically building android projects
                         SAST Scan Summary
╔════════════════════════╤══════════╤══════╤════════╤═════╤════════╗
║ Tool                   │ Critical │ High │ Medium │ Low │ Status ║
╟────────────────────────┼──────────┼──────┼────────┼─────┼
║ Kotlin Static Analysis │        0 │    0 │      0 │   0 │   ✅   ║
║ Kotlin Security Audit  │        0 │    0 │      0 │   0 │   ✅   ║
╚════════════════════════╧══════════╧══════╧════════╧═════╧════════╝

It doesn't look like it scanned.
Is Android kotlin supported?
Thanks

Hi @donnie-jp,
Yes, Android with Kotlin is supported. However, scan requires the project to be compiled which can be either manual or automatic

Manual: Run the gradle commands before the scan step. Example workflow here - https://github.com/prabhu/AndroGoat/blob/master/.github/workflows/shiftleft-analysis.yml
Automatic: Set the environment variable ANDROID_SDK_ROOT based on GitHub action documentation

I think it might be the below value but please confirm.
export ANDROID_SDK_ROOT=${HOME}/Library/Android/sdk

Thanks. I had to manually compile - just adding the ANDROID_SDK_ROOT didn't work. Probably due to the project structure.
Here's my results:
https://github.com/rakutentech/android-remote-config/pull/43/checks?check_run_id=1319991368

This looks great! Thank you for supporting scan! Please feel free to drop by to our discord channel for any questions or comments.


Updating an existing app resulted in a "forbidden" error

Repro steps:

Create an HTTP triggered function with auth level function key and deploy
Change auth level to anonymous instead and redeploy

Action: azureFunctions.deploy
Error type: Forbidden
Error Message: Forbidden
Version: 0.17.0
OS: darwin
Product: Visual Studio Code
Product Version: 1.33.1
Language: en

Call Stack
Function.module.exports.j._normalizeError extension.bundle.js:332:65127
Object.module.exports.j._processResponse extension.bundle.js:332:60937
$.h [as _callback] extension.bundle.js:332:54789
$.module.exports.$.init.t._callback.t.callback.t.callback extension.bundle.js:238:5462
$.emit events.js:182:13
$.<anonymous> extension.bundle.js:238:18275
$.emit events.js:182:13
IncomingMessage.<anonymous> extension.bundle.js:238:17117
Object.onceWrapper events.js:273:13
IncomingMessage.emit events.js:187:15
endReadableNT _stream_readable.js:1090:12
process._tickCallback next_tick.js:63:19

@asavaritayal this doesn't repro for me. Can you give more details:

I assume this is a Python project?
How did you create the function app? From VS Code or the portal?
Does this happen consistently for this function app?


Variables are cleared once the WAIT task is started

Bug description:
Variables are cleared once the WAIT task is started.
Details:
Conductor version: 3.11.0
Workflow definition:
{
  "name": "Set_Variable_Workflow",
  "description": "Set a value to a variable and wait",
  "version": 1,
  "tasks": [
    {
      "name": "Set_Name",
      "taskReferenceName": "Set_Name",
      "type": "SET_VARIABLE",
      "inputParameters": {
        "name": "Foo"
      }
    },
    {
      "name": "Test_Wait",
      "taskReferenceName": "Test_Wait",
      "type": "WAIT",
      "inputParameters": {
        "duration": "30 s"
      }
    }
  ],
  "restartable": true,
<EMAIL_ADDRESS>  "workflowStatusListenerEnabled": true,
  "schemaVersion": 2
}

Steps to reproduce the behavior:

Create provided above workflow definition
Start a new workflow for the Set_Variable_Workflow definition
Open the workflow execution page
Check the Variables

Actual behavior:
No variables exist.
Expected behavior:
The variable name with the value Foo, created by the Set_Name task exists.

Screenshots:

Hi @vzubyk , Thanks for reporting this. I am able to reproduce this. We will raise a fix for this.

HI @vzubyk , PR for the fix https://github.com/Netflix/conductor/pull/3221


[BUG] Blockchain node should use nonce tracker's nonce in transaction creation

In createSingleTransaction(), nonce tracker's nonce should be used to be consistent.

Fixed in #281


Keyboard Shortcuts not Working

This issue has been moved from a ticket on Developer Community.

[severity:It's more difficult to complete my work]
When trying to use Ctrl + ] to increase indent a line, it does not work (VS 2022 default). Changing the shortcut in the command to something else in preferences also does not work. Decreasing indent does work (Ctrl + [) Using Windows 11, Vs 2022, Version 17.4.4 x64.

Original Comments
Feedback Bot on 1/23/2023, 06:33 PM:
(private comment, text removed)

Original Solutions
(no solutions)

Hi there, if you are looking to increase/decrease line indent, you can try using tab/tab + shift, which will add/remove a tab (or the equivalent spaces) to the selected lines or the current line. Please reopen the issue if that doesn't solve your problem. Thanks!


Use pixels for Line Height

Was wondering why the current Line property comes up as a percentage? Think it would be better to have it display pixels, especially for CSS purposes.
What do you think?

No. Defiantly not.
Especially for modern CSS you’ll need the unit-less value (it’s percent). Pixels are dead ;-)
Even the font size should be rem (based on 16px base font size—mostly standard).
h1
font-family: $font-1
font-size: 2rem
line-hight: 1.2

I agree that more modern CSS practice uses %, em's and rem's, but some projects use pixels because of legacy code.
Forward thinking is definitely important, but it also doesn't allow for existing codebases that clients might already be using.

Hi, how could I generate a line-heigth label at all? Currently if i use the Get Typography tool it don't write any lh - do I have to use a special setting?

@MiMaMuh Looks like the new version removes line height from the "get typography"  feature.

"get typography" feature merged into the "get property", and using "sp", "pt" and "px"


OTBR is broadcasting MLDv2 messags in the Thread network

Describe the bug
OTBR is broadcasting MLDv2 messags in the Thread network. (see the pcap)
To Reproduce

Git commit id: master (8a7c53b9faeeef96a8a7c4fb1de364cf00acf365)
IEEE 802.15.4 hardware platform: Raspberry Pi3 (otbr-agent) + OT RCP

Expected behavior
OTBR should NOT broadcast MLDv2 messages in the Thread network.
Console/log output
Pass_OpenThread_9.2.1_Commissioner_R2.0.0_2020_06_22-16_01_06_Ch-23.pcapng.zip
MasterKey: aabbccddeeff00112233445566778899

Resolved by https://github.com/openthread/openthread/pull/7343


Yarn 2 issues, @vue/test-utils tried to access @vue/compiler-dom

@vue/compiler-dom is not listed as a dependency which causes Yarn 2 to break.
@vue/test-utils tried to access @vue/compiler-dom, but it isn't declared in its dependencies;
This makes the require call ambiguous and unsound.

https://github.com/vuejs/vue-test-utils-next/blob/8a1f348969ffadfe1b35de746f0c27c020ee074c/src/utils/compileSlots.ts#L1
Temporary solution:
# <project root>/.yarnrc.yml
packageExtensions:
  "<EMAIL_ADDRESS>    dependencies:
      "@vue/compiler-dom": "*"

Hi! I have not tried out yarn 2 yet.
Can we just add @vue/compiler-dom as a dependency to fix this?
If you want to send a PR, I will do a release this weekend so you can use VTU with yarn 2.

I am seeing the same issue with npm (I am in a vue2 project):
 ERROR  Failed to compile with 1 error

 error  in<EMAIL_ADDRESS>
Module not found: Error: Can't resolve '@vue/compiler-dom' in 'C:\proj\node_modules\@vue\test-utils\dist'

So I guess @vue/compiler-dom should be added as dependency.
Installing it using npm i -D @vue/compiler-dom seems to solve the issue but I can unfortunately not be 100% sure as I now did run in the next issue (unrelated to this).
@lmiller1990 if you think it's correct to add this, I can submit a PR for this.

Vue 2 was not a monorepo and does not have a @vue/compiler-dom - have you got the right version of test utils? test-utils v1 is for Vue 2, test-utils v2 is for Vue 3 (confusing, yes - Vue 1 never had any testing utils, so that's why).
Either way if it is as simple as adding the dependency here, please go for it! yarn 2 compat would be great.

Ok, I think I was getting confused with the versioning. My fault, sorry.
I am now on Vue2, @vue/test-utils 1 and webpack 4. So far so good, everything works (except I ideally would like to upgrade to webpack 5) but when doing so, I face quite some issues.
Anyway, as I'm on Vue 2, it definitely does not relate to here.
I think I came across this particular issue when trying to upgrade to webpack 5 and fix some of the errors but I unfortunately can't remember what exactly I did.
Probably I tried to use vue-test-utils-next in the Vue2 project (because I thought it might solve some issues) and that was the reason I got this error.
So I think in my case it was just a misconfiguration by myself.
Therefore I would also not just add the dependency as I wrote above.

The issue is still there with yarn version 3.3.1 and @vue/test-utils 2.2.6
How I did it:

created a vanilla vue project with yarn create vite and included the unit test option
after installing packages with yarn I tried to run unit tests with yarn vitest --environment jsdom --root src/
Result: Error: @vue/test-utils tried to access @vue/compiler-dom, but it isn't declared in its dependencies

Looks like it is not fully compatible with yarn plug and play as described in yarn docs. Switching to pnp loose option seems to work around the issue.
My env:
yarn 3.3.1
@vue/test-utils 2.2.6
node 16.19.0
vite 4.0.3
vitest 0.26.2


Loading a DLS file has no effect

Using the example program (sonivoxrender) on the testing midi file (arm-wt-22k/vectors/ants.mid) produces the same output regardless of whether a DLS file is loaded or not.
Or in other words, these two commands produce the same file:
./sonivoxrender ../arm-wt-22k/vectors/ants.mid
./sonivoxrender -d gm.dls ../arm-wt-22k/vectors/ants.mid

Tested DLS files were gm.dls from Windows installation and DLS files mentioned in issue #9.

Thank you for your contributions.  I've decided that is worth to include your fix in the library.
Please add your valid copyright statement to the README.md document at your convenience.


Tables: BulkActions - Fix getSelectedTableRecords query to include filters.

This way the thrashed records will be included in bulk actions if the TrashedFilter is applied.

[x] Changes have been thoroughly tested to not break existing functionality.
[x] New functionality has been documented or existing documentation has been updated to reflect changes.
[x] Visual changes are explained in the PR description using a screenshot/recording of before and after.

This needs to only apply when shouldDeselectAllRecordsWhenFiltered() is true

Thanks! I'll be looking into it, and updating the request.

After your suggestion it still works as expected. There is no way that I can oversee and test all use cases. So thanks again (and your great work). Hopefully this will not break a thing..

Thanks


Archive.org links break if https:// link is saved but http:// link is viewed

Instead of using base_url without http/https scheme, use a form of the url with scheme for archive.org saving and viewing.

Fixed by: 7a81a9f


fields added to package.json

I've added/modified a few fields in the package.json file: repository, keywords and bugs, so they validate with http://package-json-validator.com.
Let me know if these changes are ok and if you would like anything modified with them.

Thanks for the effort, but unfortunately unnecessary. Npm automatically populates the github info from the short string field, which is nicer and the new npm search no longer uses the keywords array in the search.
You may want to report the issues to the validator tool.

Ok, I will check with them.  Thanks.

Awesome! Npm itself uses https://npmjs.org/package/read-package-json to read the file, so that would probably be a good reference implementation. Npm itself will throw errors for invalid fields, warnings for missing / wrong fields on publish or package read and this module is currently clean for that as well, so I don't understand what the validator is validating against, as I would assume it should work like npm.

Looking through the GitHub repo for that validator tool, there at lots of open issues regarding the issues I noted above regarding incorrect validation for npm packages like https://github.com/gorillamania/package.json-validator/issues/29 and https://github.com/gorillamania/package.json-validator/issues/44

I should note that the only non-valid field was the repository url.  keywords and bugs were only at warning level.  There is also a recommendation for the 'homepage' field, which I left alone.
package-json-validator
Results:
{
  "valid": false,
  "errors": [
    "Url not valid for repository: mysqljs/mysql"
  ],
  "warnings": [
    "Missing recommended field: keywords",
    "Missing recommended field: bugs"
  ],
  "recommendations": [
    "Missing optional field: homepage"
  ]
}

Looking at the open issues you provided, it seems that the normalization rules are different between https://www.npmjs.com/package/normalize-package-data and https://www.npmjs.com/package/read-package-json

The repository field in our repo is definitely valid. I liked you to the issue reporting that the validator is incorrectly thinking it is invalid. Also the bugs and homepage URLs are auto populated from the repository short string to their GitHub addresses, which is why we don't need to specify them yet they show up on the npmjs.com site and in the npm repository just fine, so there is nothing to change here, I think you just accidentally stumbled across some bugs in that validator tools, which is why I suggested to file issues there and hopefully they'll get it fixed.

Yes it's valid, it's the validator that is a bit outdated. I hope to get a weekend to give it some love soon :)


fix(behavior_path_planner): revise some content about avoidance in .md

Description

This PR revises some content about  avoidance in README.md and behavior_path_planner_avoidance-design.md
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[ ] I've confirmed the contribution guidelines.
[ ] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

@shulanbushangshu
you need doc: ~ for document PR


OPSEXP-517: Add deprecation notice

Added deprecation notice at the top of the page and fixed markdown warnings.

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


How to correctly make the plugin applicable on a scrollable div?

I revised the code a bit in order to apply it on a scrollable div. It is working correctly until browser resizing.
To reproduce the issues, clicking on one or two nav items (and let the scrolling animation run) first, and then resize your browser. After browser resizing, I can see two issues:

Clicking on a nav item to trigger the scrolling animation. When the scrolling animation finishes, the "current" indicator suddenly changes to another nav item.
When scrolling the content manually, the "current" indicator changes to wrong nav items.

What did I do wrong?
Here is the revised code on jsFiddle.

When you say revised, what did you revise and why?

@davist11 Sorry for not being clear. I had changed the thread title.
I mainly changed the references to document to the scrollable div. And I did that in order to make the plugin applicable on the scrollable div.

Someone solved the issue. Please refer to this Stack Overflow answer.


updated oracleai_demo.ipynb

The outer try/except block handles connection errors, and the inner try/except block handles SQL execution errors, providing detailed error messages for both.
try:
conn = oracledb.connect(user=username, password=password, dsn=dsn)
print("Connection successful!")
cursor = conn.cursor()
try:
    cursor.execute(
        """
        begin
            -- Drop user
            begin
                execute immediate 'drop user testuser cascade';
            exception
                when others then
                    dbms_output.put_line('Error dropping user: ' || SQLERRM);
            end;

thanks for merging my contribution.


BUGFIX cssInterop only triggering when a param is passed

Fixing issue #767: edge cases on prop remapping, to ensure that props are only (re)mapped if a value is passed.
E.g.: <FlatList /> only accept the columnWrapperStyle param when its numColumns param is > 1. With the current implementation of cssInterop, an empty array was passed to all remapped parameters, introducing an unexpected breaking error (Uncaught Invariant Violation: columnWrapperStyle not supported for single column lists), even though no columnWrapperStyle was set.
👉 With this proposed fix, the props are only remapped if a value is passed to the original component.
E.g.:

Calling <FlatList /> with no columnWrapperStyle and no numColumns (default=1) param won't trigger errors anymore.
If a columnWrapperStyle or columnWrapperClassName were to be added, it would be then be remapped
BUT the end user would have also set numColumns > 1 (otherwise there is no point in calling columnWrapperStyle :).

It's my first PR on this repo, so lmk if I missed a step 🤓

@kimchouard I think adding a description with the details of your issue would be helpful :)

@kimchouard I think adding a description with the details of your issue would be helpful :)

Indeed, thanks for raising it 😅 I've created this PR using github.dev (which I just discovered and is a complete 🤯 🤩) and all the details were in the issue #767, but I didn't realize it was not clearly showcased in here.
I've updated the description, hopefully it's more clear now! ;)

@marklawlor FYI I've integrated your latest changes from master to make this PR ready to be merged 🤓

Does this only affect web or should there be a native equivalent to this as well?
I should appreciate a unit test to demonstrate the fix. You can find some in packages/react-native-css-interop/src/__tests__/react-native-components.test.tsx that should how to test components.

It's only affecting Web.
I'll work on a unit test this week, sorry first contribution to this repo 🙏


fix: Fix project name generation from complex URLs

My regex-fu is not strong enough, made it in steps but with comments :( For such small strings, it should not affect the speed much, right?

Actually we'll need to rework the metadata, urlStringToRemote no matter what, as without the port parsing will show the remote as https://bitbucket.org:8888/


Example of an IUserClaimStore

Hello,
I'm looking through the source (EF mainly) and can't find an IUserClaimStore example. Does it implement it? Or is there source repo somewhere else I could find a working example?

IClaimStore isn't one of ours, do you mean IUserClaimStore?

I guess the answer is no, so I'm closing


Support App Names with UUID as Suffix

When using Deploy with Confidence (DwC), cf app names are suffixed with a uuid. This uuid changes with every deployment of our application.
Because of that we have to setup mtx-tool everytime we want to use it with the correct cf app name.
It would be great if this project could be enhanced and also support DwC style apps.
One hurdle is that there can be multiple deployed apps with the same base name, but different uuids at the same time. I guess this should not matter too much because they are then different versions of the same app and are bound to the same services. So the logic could choose any app which starts with the configured app name followed by a valid uuid.
mtx-tool is already capable of handling different app names with suffixes -blue, -green and -live. But I guess this is different, because the suffix in our case is not static. One would probably have to work with a regex to filter the correct app name.

Let's go with a two tier approach. First goal is to support an env variable MTX_APP_SUFFIX that gets mixed in with the hard-coded app suffixes -blue and -green.
Second goal is to support a simple * wildcard in the configured app names, that expands into arbitrary app names, but leaves out the known suffixes if they are found.

@Kkoile When I did the actual implementation, I did not like the wildcard approach so much anymore. I implemented this in a different way now, that I think is more specific to your use-case and slightly less prone to unintended behavior.
The first part with an env variable that can control suffixes works pretty much as we imagined. There was some ordering shenanigans happening, where I had previously gone through all apps and looked for a matching suffix candidate, but since the custom suffix should be checked first it now goes through all suffixes and looks for matching apps instead. This seems to be sensible anyways and lead to fairly intuitive behavior for me.
For the second part, I added a hard-coded suffix -{UUID}, which expands into a UUID regex for the suffix check. I feel  this approach is similar to how I handle the -blue/-green case, by hard-coding them as suffixes to check. Ideally, I want other DwC projects that try the tooling to just work, without thinking a lot about the configuration. I may add a section in the docs on this as well.
Here is a pre-built with the new functionality. Can you try this and see if it works well in practice?
cap-js-community-mtx-tool-0.7.12.tgz

I tested it and it works as expected. I only configured the base name of our application and that it connects to the correct cf application.
Thanks a lot for your effort!


Subtitles

Hey,
I just went through the whole introduction tutorial and I noticed a few mistakes in the spanish subtitles (I'm latin so you can trust me).

Table is tabla instead of mesa
Performance is rendimiento instead of actuación

Those two changes may change a lot the meaning of the phrase.
I hope it helps!

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

ID: bcb9bc55-ac56-d1ab-c465-4e41bfa8f152
Version Independent ID: 44bd9bb3-5aad-31ac-27b6-53e3c0554960
Content: Introduction to DAX - Power BI
Content Source: powerbi-docs/guided-learning/introductiontodax.yml
Service: powerbi
GitHub Login: @davidiseminger
Microsoft Alias: davidi

Thanks for your comments and close read @dnavia. Apologies for the mistakes in the subtitles, we'll let the translators know about them. Much appreciated!


Issue 421 hostnames

@lulf do you have any idea why host in endpoints for address spaces is not set in travis ? It works fine locally and in Jenkins. Maybe some different configuration for Travis machines?
21:53:58,925 [main] INFO io.enmasse.systemtest.bases.TestBase - Address-space successfully created: {name=brokered-shared-1,namespace=brokered-shared-1,type=brokered,plan=unlimited-brokered,endpoint-console={host=null,port=32655},endpoint-messaging={host=null,port=31998}}

@tkratky This is because on kubernetes, routes does not exist. And instead, loadbalancer service types are used instead. However, those do not generate a hostname to use, so hostname will be null. I think that if hostname is null, the rule should be to lookup the service ip of the service field in the endpoint entry and use that.


[5.6] Fix URL validation pattern

When running the Laravel tests with PHP 7.3.0beta1 on Windows, testValidateUrlWithValidUrls() fails with an exception:

preg_match(): Compilation failed: invalid range in character class at offset 1828

The hyphen has to be escaped:
https://stackoverflow.com/q/26208622/4848587
https://stackoverflow.com/q/24764212/4848587
Apparently, this pattern has always been invalid, PHP just never complained.
This is the Symfony fix (where the pattern is from): https://github.com/symfony/validator/commit/ea91e49

Happens the same whit Laravel 5.4 + PHP 7.3.
I think that it will be useful for someone.


WARNING: X second(s) too late at Date

Thanks for creating blackbird! Awesome!
I'm running it locally in a docker container in demo mode right now and when I check the logs in the container I get a lot of these warnings:
WARNING: 2 second(s) too late at 03/17/2017 10:27:42
What does the warning actually mean and (how) can I avoid this?

Hi @mcmunder,
It means that your system is a bit slow. Usually it's because of network latency. You can increase the analysis loop by increasing the parameter GapSec. That should fix the problem.

when increasing GapSec  blackbird could miss opportunities...what other solution do we have? running blackbird on a hosting like ovh with good bandwitch?

@playmobitch Yes, I can confirm that hosting/running it on AWS, Azure or similar solves the problem as well. No need to increase the GapSpec when running it there...


why keep dependency video_player comment?

https://github.com/PonnamKarthik/FlutterHtmlView/blob/da6f11dd2b92afe4ad1a35b3b01e2418740795f0/pubspec.yaml#L14

fixed


[BUG]: Out of memory when training stable diffusion on RTX2070

🐛 Describe the bug
When training stable diffusion, crashed report: CUDA out of memory.
Is it possible to run on RTX2070?
Thanks!
Environment
OS: Ubuntu 20.04
GPU: RTX2070 + 8GB RAM
Python: 3.8
pytorch: 1.11
cuda: 11.3
Colossalai: 0.1.10
Stable Diffusion pre-traind models downloaded from https://huggingface.co/CompVis/stable-diffusion-v1-4/tree/main
train_colossalai.yaml: only modified "data"->"file_path", there are 4 jpg files in size of 256*256 for testing

I don't think whoever wrote that article claiming that 8GB GPU's could easily be used for training has answered any questions about it yet.
That includes their Reddit account they used for spam marketing this release, absolutely zero engagement with the community.

what's your train.yaml, if you want to train on RTX2070, you must decrease your batch size from 64 to > 4.


Could not prepare to run the isolate.

下拉刷新网络请求无反应。
--- translate ---
Drop-down refreshes network requests without response.
Steps to Reproduce

在FlutterViewController上present一个WebViewController。
--- translate ---
On FlutterViewController, present a WebViewController.
[self presentViewController:viewControllerToPresent animated:flag completion:completion];
返回FlutterViewController，刷新网络请求。
--- translate ---
Return to FlutterViewController to refresh the network request.
如果没出现，则重复步骤1、2
--- translate ---
If it does not appear, repeat steps 1 and 2.

Logs
[VERBOSE-2:engine.cc(157)] Could not prepare to run the isolate.
[VERBOSE-2:engine.cc(116)] Engine not prepare and launch isolate.
[VERBOSE-2:FlutterViewController.mm(462)] Could not launch engine with configuration.

bogon:star_admin v7lin$ flutter doctor -v
[✓] Flutter (Channel beta, v0.9.4, on Mac OS X 10.13.6 17G65, locale zh-Hans-CN)
    • Flutter version 0.9.4 at /Users/v7lin/flutter
    • Framework revision f37c235c32 (5 weeks ago), 2018-09-25 17:45:40 -0400
    • Engine revision 74625aed32
    • Dart version 2.1.0-dev.5.0.flutter-a2eb050044

[✓] Android toolchain - develop for Android devices (Android SDK 28.0.3)
    • Android SDK at /Users/v7lin/Library/Android/sdk
    • Android NDK at /Users/v7lin/Library/Android/sdk/ndk-bundle
    • Platform android-28, build-tools 28.0.3
    • ANDROID_HOME = /Users/v7lin/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1136-b06)
    • All Android licenses accepted.

[✓] iOS toolchain - develop for iOS devices (Xcode 10.0)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 10.0, Build version 10A255
    • ios-deploy 1.9.2
    • CocoaPods version 1.5.3

[✓] Android Studio (version 3.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 29.1.1
    • Dart plugin version 181.5656
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1136-b06)

[!] IntelliJ IDEA Community Edition (version 2018.2.1)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
    • For information about installing plugins, see
      https://flutter.io/intellij-setup/#installing-the-plugins

[✓] VS Code (version 1.28.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 2.19.0

[✓] Connected devices (1 available)

Duplicate of #21221

Duplicate of #21221


Underscores to dashes

Just to help make the resulting SVG markup and CSS a bit more idiomatic.
cc: @jamescott

got the :+1: from @jamescott