Failed to create shader - waveform-compute.glsl failed

When running on a VM hosted on macOS Catalina. Guest OS is Ubuntu 18 LTS, the same result using either Parallels or VMWare Fusion. OpenGL versions under Parallels is OpenGL 2.1, on VMWare, it's OpenGL 3.3
ERROR: Failed to create shader (of type 37305)
ERROR: Compile of shader shaders/waveform-compute.glsl failed:
�����
Shader source: #version 430

//The output texture (for now, only alpha channel is used)
layout(binding=0, rgba32f) uniform image2D outputTex;

//Voltage data
struct data_point
{
	float x;		//x pixel position (fractional)
	float voltage;	//y value of this sample, in pixels
};

layout(std430, binding=1) buffer waveform
{
	data_point data[];
};

//Global configuration for the run
layout(std430, binding=2) buffer config
{
	uint windowHeight;
	uint windowWidth;
	uint memDepth;
	uint alpha_scaled;
};

//Indexes so we know which samples go to which X pixel range
layout(std430, binding=3) buffer index
{
	uint xind[];
};

//Maximum height of a single waveform, in pixels.
//This is enough for a nearly fullscreen 4K window so should be plenty.
#define MAX_HEIGHT		2048

//Number of columns of pixels per thread block
#define COLS_PER_BLOCK	2

layout(local_size_x=COLS_PER_BLOCK, local_size_y=1, local_size_z=1) in;

//Interpolate a Y coordinate
float InterpolateY(vec2 left, vec2 right, float slope, float x)
{
	return left.y + ( (x - left.x) * slope );
}

/*
NEW IDEA
Multiple threads per X coordinate (say, 32 - 1 warp)
Parallel fetch base[i+z] and atomically increment local memory

Each local has a 2D shared array
Assuming 96 KB shared memory, we can fit a total of 24K float32 temp pixels
Assuming 2K max line height, that's up to 12 pixels of width per local
*/

//Shared buffer for the local working buffer
shared float g_workingBuffer[COLS_PER_BLOCK][MAX_HEIGHT];

void main()
{
	//Abort if window height is too big, or if we're off the end of the window
	if(windowHeight > MAX_HEIGHT)
		return;
	if(gl_GlobalInvocationID.x > windowWidth)
		return;

	//Save some constants
	float alpha = float(alpha_scaled) / 256;

	//Clear column to blank in the first thread of the block
	if(gl_LocalInvocationID.y == 0)
	{
		for(uint y=0; y<windowHeight; y++)
			g_workingBuffer[gl_LocalInvocationID.x][y] = 0;
	}
	barrier();
	memoryBarrierShared();

	//Loop over the waveform, starting at the leftmost point that overlaps this column
	uint istart = xind[gl_GlobalInvocationID.x];
	vec2 left = vec2(data[istart].x, data[istart].voltage);
	vec2 right;
	for(uint i=istart; i<(memDepth-1); i++)
	{
		//Fetch coordinates of the current and upcoming sample
		right = vec2(data[i+1].x, data[i+1].voltage);

		//If the current point is right of us, stop
		if(left.x > gl_GlobalInvocationID.x + 1)
			break;

		//If the upcoming point is still left of us, we're not there yet
		if(right.x < gl_GlobalInvocationID.x)
		{
			left = right;
			continue;
		}

		//To start, assume we're drawing the entire segment
		float starty = left.y;
		float endy = right.y;

		//Interpolate if either end is outside our column
		float slope = (right.y - left.y) / (right.x - left.x);
		if(left.x < gl_GlobalInvocationID.x)
			starty = InterpolateY(left, right, slope, gl_GlobalInvocationID.x);
		if(right.x > gl_GlobalInvocationID.x + 1)
			endy = InterpolateY(left, right, slope, gl_GlobalInvocationID.x + 1);

		//Sort Y coordinates from min to max
		int ymin = int(min(starty, endy));
		int ymax = int(max(starty, endy));

		//Push current point down the pipeline
		left = right;

		//Fill in the space between min and max for this segment
		for(int y=ymin; y <= ymax; y++)
		{
			g_workingBuffer[gl_LocalInvocationID.x][y] += alpha;
		}

		//TODO: antialiasing
		//TODO: decimation at very wide zooms
	}

	//Copy working buffer to RGB output
	for(uint y=0; y<windowHeight; y++)
		imageStore(outputTex, ivec2(gl_GlobalInvocationID.x, y), vec4(0, 0, 0, g_workingBuffer[gl_LocalInvocationID.x][y]));
}

INTERNAL ERROR: failed to load waveform compute shader, aborting
    This indicates a bug in the program, please file a report via Github
Aborted (core dumped)

glscopeclient requires OpenGL 4.3 to get compute shaders. Is there any way to get newer in a virtual machine? GL 4.3 has been around since 2012 so I would expect it's pretty well supported.
My understanding is that OSX's OpenGL version is stuck at 4.1 forever since Apple deprecated OpenGL for some silly reason, so you probably won't be able to run it natively. Do you have a Linux or Windows computer you can try it on?  Apple clearly has zero interest in cross platform accelerated graphics and I'm not about to rewrite the whole tool with a different graphics API, so for the time being I'm pretty sure OSX support is just not possible.
Closing this issue since there's nothing wrong with glscopeclient, the problem is on Apple's end. Feel free to hop in the IRC channel (#scopehal on Freenode) to discuss possible workarounds etc.


Does Mxnet for windows support multi-device training?

I read Mxnet\dmlc-core\tracker\dmlc_tracker\mpi.py, and don't know how to install mpirun for windows.
I tried MS-MPI, it has mpiexec but not mpirun.

This issue is closed due to lack of activity in the last 90 days. Feel free to reopen if this is still an active issue. Thanks!


GC EARB 2017-12-14 Open First Whitepaper DRAFT

GC EARB 2017-12-14 02 Open First Whitepaper DRAFT.pptx

I got the following trying to open this in LibreOffice

It was able to open. Some of the layout is messed up but still readable.  Saving it as PDF would preserve original formating.

Thanks for bringing this to my attention. I'll try saving it as a PDF as well to have both formats.


Are there any plans to add a wrapper for Julia language?

What language and solver does this apply to?
Julia
Describe the problem you are trying to solve.
Use Julia to call OR-Tools.
Describe the solution you'd like
Describe alternatives you've considered
Additional context
Add any other context or screenshots about the feature request here.

No. We will accept contributions. But I have no time for that effort.

Currently wrapper for Python, Java and .Net are provided thanks to SWIG which can generate them.
SWIG doesn't support Julia, could be done by https://github.com/JuliaInterop/Cxx.jl

MathOpt should soon get a C API, which should be useful to get a Julia wrapper. It won't be have the same level of support as the existing Python, .Net, and Java wrappers, though.


DEV: Move extensions into reloadable patch.

Otherwise, the extensions are lost in development each time Rails
autoloader is activated.

Thanks @martin-brennan


Pull files/folder from specific branch

Hello,
Via https://github.com/actions/checkout/issues/483 I arrived here. I'm interested in pulling (checking out) a specific file from another branch. But in the instructions I don't see anywhere how to specify the branch to pull from.
If I already checked out the main branch, can I then pull specific files from another branch (in the same repo) using this action? If so, how could I do that?
(I tried following these instructions but they don't work in an action:)

https://github.com/Bhacaz/checkout-files/pull/6 addresses this.
You may try using our fork https://github.com/makers99/checkout-files in the meantime.
        uses<EMAIL_ADDRESS>


Add _binary_exists and use in github-cli completion

gh was also used as an git alias, and caused _command_exists to not fail as expected
Closes #1684

/lgtm
/approve
works on my machine!

I agree with @cornfeedhobo's comment - that looks like a good addition.
Outside of that, the _binary_exists function is a nice new feature!

@cornfeedhobo @nwinkler I added some tests for _binary_exists and created a new issue about the system completion
If the CI is green, I will merge this PR 😄


DNN module, Net class

It seems we have no implementation of GetUnconnectedLayers method for the Net class in the DNN module.
Is there anyway we can go around this problem?

Thanks. I added GetUnconnectedLayers by  https://github.com/shimat/opencvsharp/commit/adcb449609e71ecf0abd4c617c5c4016b0a144e9


Easy reflection type traversal utils

What problem does this solve or what need does it fill?
Currently, it is non-trivial to search for and patch a specific type nested deep inside a dyn Reflect.
What solution would you like?
Add an iter_children (name bikesheddable) iterator function on Reflect that returns a ChildrenIter (name bikesheddable) over all direct children. ChildrenIter will be an enum with the same same variants as ReflectKind, but it also holds a reference to the iterator of the erased data trait.
e.g. Struct(IterFields<'a>)
This is needed for Reflect to still be object-safe.
It should also exist an iter_children_mut, which is blocked on #14846
What alternative(s) have you considered?
Writing that myself.

Thinking about it, might it not be just easier to put that whole "get by index" stuff directly into Reflect?

Personally, I don't think this should be on Reflect itself. This sounds like it would be best done as either an extension trait or utility function. We could potentially take inspiration from syn's visit module.


no update

I use the adapter for my TV. Unfortunately, it only updates when I start the mobile app. The update interval in the adapter is set to 10 seconds.
What is the reason for this?

same here.
Just smartthings.0.28df1303-e3c2-0b43-bbf0-ba6eea3090ee.status.switch.switch.valueis working fine all the time


sveltekit error: Internal server error: Cannot find module 'codemirror' imported from '/node_modules/svelte-codemirror-editor/CodeMirror.svelte

Downloaded the latest sveltekit today. Imported your script and followed your sveltekit instructions, which worked great. I did have to install @codemirror/lang-javascript separately, which was something I didn't see mentioned in your README. When I run the package I am getting the error in the title. Your index.js looks right and when I hover over Codemirror in VSCode it shows the correct import name. Any ideas why this is happening?

I was able to get this to work by adding codemirror to the project. One thing I would also point out using sveltekit is that you can install all the dependancies as dev dependancies. None of them need to be added as dependancies. Also, you don't need to do anything different for this package to work in vite.config.ts.

@Blumed do you remember anything more you did to get this to go away?  I have this in my vscode and it's driving me batty.

@Blumed do you remember anything more you did to get this to go away?  I have this in my vscode and it's driving me batty.

I did end up getting it to work. Here is the file I am initializing it in. https://github.com/Blumed/make-bookmarklets/blob/main/src%2Flib%2Fcomponents%2Feditors%2Fjs-editor.svelte


Configuration the server for calls on Flutter App

I have issues for the application , I need to make a call on the server and I cant handle it

I need to make a meeting with mentors please

I need to parse JSON with Dio package or HTTP package

@data-charya please look into this

?? Can we talk on Discord or WhatsApp

@data-charya

Can I contribute to this project after GSSoC? and I can right now still contribute?

@vigneshshettyin


jQuery triggers

Is it possible to have an update to fire jQuery triggers? When selecting a value for a RadioButtonGroup, our page isn't catching the event. It should be as simple as adding ".trigger('click')" to the end of the Radio (and maybe Select) groups.

Hi @erbaker . I don't quite follow what you are asking. Could you please provide a bit more detail on the scenario you are trying to automate with Seleno? Are you using ASP.Net MVC5?

Hi @mwhelan ,
We have radio buttons on our page, and a jQuery event listener for onClick() on the radios. When seleno selects a radio button, it's setting the attribute (.attr('checked', true)) which does not fire the onClick() event of our radio buttons.
I can submit a PR as an example, and it might make more sense. Let me know.
Ed


Use correct meta item in rich text.

Prevent incorrect static cast and unnecessary meta_hover_ended/started
events.
Fixes #25645

Thanks!


Chrome debs from which we extract flash fail to download

Issue description
@aszlig, it seems that the deb files we extract flash from don't seem to be there anymore. The mirror.pcbeta.com DNS entry does not even exist.
Steps to reproduce
$ nix-build -E 'with import <nixpkgs> { config.chromium.enablePepperFlash = true; }; chromium'
these derivations will be built:
  /nix/store/vzmrqcj2jiik7zcwarczxg4cy1bn944x-google-chrome-stable_51.0.2704.103-1_amd64.deb.drv
  /nix/store/x64xp98v7jdv8hl4bjd9n0kxa5anng8n-chromium-binary-plugins.drv
  /nix/store/6aqb6c4w8rx3ihq8f5zp54ni3l4vlc2q-chromium-51.0.2704.103.drv
building path(s) ‘/nix/store/ywyi3jwy43iqzi1vxgm7s4wa5i2kab41-google-chrome-stable_51.0.2704.103-1_amd64.deb’

trying https://dl.google.com/linux/chrome/deb/pool/main/g/google-chrome-stable/google-chrome-stable_51.0.2704.103-1_amd64.deb
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
curl: (22) The requested URL returned error: 404

trying http://<IP_ADDRESS>/chrome/pool/main/g/google-chrome-stable/google-chrome-stable_51.0.2704.103-1_amd64.deb
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
curl: (22) The requested URL returned error: 404 Not Found

trying http://mirror.pcbeta.com/google/chrome/deb/pool/main/g/google-chrome-stable/google-chrome-stable_51.0.2704.103-1_amd64.deb
Warning: Transient problem: timeout Will retry in 1 seconds. 3 retries left.
Warning: Transient problem: timeout Will retry in 2 seconds. 2 retries left.
Warning: Transient problem: timeout Will retry in 4 seconds. 1 retries left.
curl: (6) Couldn't resolve host 'mirror.pcbeta.com'

trying http://repo.fdzh.org/chrome/deb/pool/main/g/google-chrome-stable/google-chrome-stable_51.0.2704.103-1_amd64.deb
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   185  100   185    0     0    317      0 --:--:-- --:--:-- --:--:--   348
  0     0    0     0    0     0      0      0 --:--:--  0:00:01 --:--:--     0
curl: (22) The requested URL returned error: 404
error: cannot download google-chrome-stable_51.0.2704.103-1_amd64.deb from any mirror
builder for ‘/nix/store/vzmrqcj2jiik7zcwarczxg4cy1bn944x-google-chrome-stable_51.0.2704.103-1_amd64.deb.drv’ failed with exit code 1
cannot build derivation ‘/nix/store/x64xp98v7jdv8hl4bjd9n0kxa5anng8n-chromium-binary-plugins.drv’: 1 dependencies couldn't be built
cannot build derivation ‘/nix/store/6aqb6c4w8rx3ihq8f5zp54ni3l4vlc2q-chromium-51.0.2704.103.drv’: 1 dependencies couldn't be built
error: build of ‘/nix/store/6aqb6c4w8rx3ihq8f5zp54ni3l4vlc2q-chromium-51.0.2704.103.drv’ failed

Technical details

Nixpkgs version: 2fa9bd5

See also #14961.

Duplicate of #14961 as a version bump should fix this (though we should probably remove mirror.pcbeta.com altogether)


Add codes highlighting

I've added codes highlighting, used some files from default theme "landscape", and import those files into 'theme-styles.styl'
Then we can enable highlighting in '_config.yml' (in the root of blog directory). And we don't need to use code blocks, just write codes following for example:
```cpp
#include <iostream>
using namespace std;
int main()
{
    cout<<"Hello world!"<<endl;
    return 0;
}
```
And it will display:
#include <iostream>
using namespace std;
int main()
{
    cout<<"Hello world!"<<endl;
    return 0;
}

Great, I'll add some information about the feature to the README.


xfce.automakeAddFlags: Drop

All the remaining references to it are not necessary and can be cleaned up.
(I am trying to upstream the remaining gio-unix-2.0 workarounds to upstream)
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

@ofborg build xfce.xfce4-dict xfce.xfce4-pulseaudio-plugin


Runner no longer works; fails at "Copy files back from the VM" with "The process '/bin/bash' failed with exit code 2"

Tried updating to @v0.2.8 to no avail. Any suggestions?

Also tried with sync: sshfs. Same result, other than nothing logged under the "Copy files back" step

Apologies, this was an issue on our side, just didn't find the correct error in the log


docs: update  deposit/withdrawal docs

fixed 3 tiny typos

I think this document is a bit out of date and a bit unclear, we should link directly to Godwoken's documentation to avoid redundancy
I have some suggestions for changes for this document, we can change it to Dependencies of Godwoken Bridge UI.

a well-defined outline
using mermaid for the diagram directly, GitHub can render it, the png is difficult to contribute

The example outline:

Diagram

deposit
withdraw
fast withdraw

Details

Deposit

Pre-require

link to Godwoken deposit
link to deposit lock args molecule definition

Withdraw

Pre-require

link to Godwoken withdraw
link to Godwoken RPC gw_submit_withdrawal_request
link to Godwoken withdrawal request molecule definition

notes:

how to calc account_script_hash
how to calc owner_lock_hash

Data Fetching/Config

balance - indexer
token list - force bridge
deposit/withdraw list - scanner API

Others

Omnilock and MetaMask

references

v0: https://github.com/nervosnetwork/godwoken/blob/0.10-rc/docs/life_of_a_godwoken_transaction.md
v1: https://github.com/nervosnetwork/godwoken/blob/develop/docs/deposit_and_withdrawal.md
scanner API
ForceBridge token list


Learn git & github page updated

Description

Category

[ ] Documentation
[x] Resource Addition
[ ] Codebase
[x] User Interface
[ ] Feature Request

Related Issue

Fixes #351
Checklist

[x] I have gone through the CONTRIBUTING guide
[ ] The name of the resource is spelled correctly (if applicable)
[x] The link to the resource is working (if applicable)
[ ] The resource is added in the correct format (if applicable)
[x] I have tested changes on my local computer (if applicable)

The text is not visible in the boxes. Please change it to 'white' or some other color that makes these resources visible.


Replace xmlhttprequest module with xhr2

Fixes #44 by replacing use of the Node xmlhttprequest module with xhr2.
This only affects those running purescript-affjax on Node, not in the browser. They will need to install the xhr2 package as a local npm dependency instead of xmlhttprequest.
This is therefore a breaking change for them. However, we could preserve backwards compatibility by adding an extra couple of lines to still use the xmlhttprequest package as a fallback if xhr2 isn't found.

Looks like the test failed on Travis - I'm all for this if it fixes xhmlhttprequest's deficiencies though!

Hmm I thought the Travis failure is because of  #48? Getting the same behaviour as reported in #47. However I'll try to work out what's going on there.

Ah yes, you might well be right there! Although part of the problem there is Travis wasn't failing even though the tests were. If you have the time to try and figure it out I'd much appreciate it. Otherwise I'll run the tests locally and merge if it all seems okay.

Yeah will look into the travis build now but it might take a little while. The tests all passed on my machine (both in Node and in the browser)

OK the problem was a race condition in the test command node ./test-server.js & node tmp/test.js: the tests started running before the test server was fully set up. Adding a sleep 2 in there before running the tests fixed it.
I don't know if that was the problem causing #48 or if it's something else so will continue looking in to that.
I can squash the commits for merging if wanted.

Awesome! Thanks very much, and your comments on #48 make sense. I'd agree this is definitely a solution for that too.

Thanks for the merge!


Fix typos

Fix typos

@lucemans All good, will close this in favor of another PR with just the FAQ https://github.com/ensdomains/docs/pull/248


Ace Card counted as one and not one or eleven

When I play the BlackJack Game on my Flipper Zero on Xtreme and I get an Ace Card it always counts only as one value and not one or eleven.

I have to check the code they have, last update I made for this game only counted the ace as one when you're over 21


limit 0 tests with TPCH, taking out failing tests

So in summary

1 new test suite with 15 tests have been added (TPC-H queries with limit 0). Verified the test ID and tests
1 test previously failing with verification issue (due to Decimal casts) was fixed.
Some tests were renamed - nothing much to verify here.

Looks good. +1

Abhishek,
Really appreciate you spelling out what is done, I should do it next time.
Thank you for the review,
Vicky.


Incorrectly parses Windows file paths

It seems that the parsing at the moment strips away drive letters in paths on Windows. An example coverage file might look like this:
SF:C:\workdir\usr\share\julia\stdlib\v1.12\nghttp2_jll\src\nghttp2_jll.jl
LH:0
LF:0
end_of_record

And then after parsing the path is given as \workdir\usr\share\julia\stdlib\v1.12\nghttp2_jll\src\nghttp2_jll.jl, i.e. it is missing the drive letter C:.
vtjnash identified https://github.com/friedemannsommer/lcov-parser/blame/8a980f27039fd41ba6188d37888b1ec9331ec04d/src/parser.ts#L229-L230 as the likely culprit.

@davidanthoff Thanks for pointing that out. This is an oversight on my part.
I will implement a fix for this and let you know when it's published.

@davidanthoff The issue has been fixed in version 4.0.1, thanks for reporting it.


Question: Is shading for jackson still relevant?

I can't see why we need to do shading on the jackson dependency.
Can someone explain this to me please?
Thanks.

A long time ago, around the start of the project probably when I was the sole user, I experienced dependency version conflicts with Jackson. This was in a codebase that had Jackson on the classpath both via Testcontainers/docker-java and Dropwizard. I don’t recall the versions I’m afraid.
I imagine it’s quite possible that the transitive dependencies have converged on compatible versions, so that in practice a conflict would be unlikely nowadays. However, I think I’d prefer not to expose people to that risk all of a sudden.
Is this something you’re just curious about, or has the shading of Jackson caused you some problems?
Sent with GitHawk

Hi @rnorth
Thanks a lot for taking the time to reply!
This is something I was curious about. When I saw the codebase, I couldn't see any reason for shading jackson. It would increase the size of the library too and introduce duplication.
From what I could see, docker-java (3.1.5) and dropwizard (1.3.14) have similar versions of jackson with only a minor version difference - 2.9.8 and 2.9.9 respectively.

Our rule of thumb - shade everything that:

can be shaded (read - does not have native code)
does not appear in our public API

We do it to avoid version conflicts.
Also, docker-java and dropwizard are not the only consumers of Jackson (given it's popularity).
Did you have any issues with it, or it was just out of curiosity?

Hi.
It was just out of curiosity.
Thanks for the explanation

Cool, let’s close then!
Thanks @ali-nutmeg
Sent with GitHawk


Hide ImmediatelyFast and Presence footsteps debug info

https://modrinth.com/mod/immediatelyfast
https://modrinth.com/mod/presence-footsteps

Also, is there a way to minify the NoiceRouter T text? Maybe give it some new lines?

@tyrannus00

ImmediatelyFast Buffer Pool can be hidden in the config file.

👍🏽

I just implemented compat with Presence Footsteps, so the next release will have this feature

Thanks!!


Rule for groovy: migration from org.codehaus.groovy to org.apache.groovy.

With the release of groovy 4 the group id has changed from org.codehaus.groovy to org.apache.groovy.
Any chance this could become a rule in this plugin?

Could you clarify what you would expect the plugin to do in such a case?
The conflict detection is already provided by Groovy in the published metadata:
     "capabilities": [
        {
          "group": "org.codehaus.groovy",
          "name": "groovy",
          "version": "4.0.21"
        },
        {
          "group": "org.apache.groovy",
          "name": "groovy",
          "version": "4.0.21"
        }
      ]

What we could add to the plugin is some standard resolution strategy. For example, always select the highest version automatically if there is a conflict. But maybe this is not helpful, because libraries relying on Groovy 3 will most likely break if we "just" update the dependency to for.

We currently observe the following when a 3rd party library depends on an older version of groovy.
   > Could not resolve org.codehaus.groovy:groovy:2.4.21.
     Required by:
         project :A > project :B > project :C > org.jfrog.artifactory.client:artifactory-java-client-services:2.17.0
      > Module 'org.codehaus.groovy:groovy' has been rejected:
           Cannot select module with conflict on capability 'org.codehaus.groovy:groovy:2.4.21' also provided by [org.apache.groovy:groovy:4.0.21(groovyRuntimeElements)]

In this particular case project :C has verified that it also functions when substituting the transitive dependency to groovy coming from the artifactory client library.
But we'd like to avoid having to adapt the published metadata for that project though, as technically it should not force downstream projects to depend on groovy 4.
Project :A does depend on groovy 4 though, so it would be convenient to automatically resolve the conflict using the plugin in this case.

Thanks for elaborating. This is a specific situation in your setup, right? In the general case, silently updating to Groovy 4 most likely breaks 3rd party libraries because of the changed package.
The plugin has a DSL for defining your own strategy in a compact way. I would suggest you do this in your Gradle setup just after you applied the plugin:
jvmDependencyConflicts {
    conflictResolution.select("org.codehaus.groovy:groovy", "org.apache.groovy:groovy")
}

See the docs for more details.


Model adaptor bug fix

fix the glibc version bug
fix the input args name mismatch bug

@XinyaoWa Please have a code review.

@xuechendi Please help to merge, thank you~


Remove nsenter workaround

Docker 1.12 and further don't need nsenter hack. This patch removes
it.  Also, it bumps the minimal version to 1.12.

Closes #776

ci check this

This seems to work for me:
#!/bin/bash
{{ docker_bin_dir }}/docker run
--cpu-shares={{ kubelet_cpu_limit|regex_replace('m', '')  }}
--memory={{ kubelet_memory_limit|regex_replace('Mi', 'M') }}
--name=kubelet
--net=host
--pid=host
--privileged
--restart=on-failure:5
-v /etc/cni:/etc/cni:ro
-v /etc/ssl:/etc/ssl
-v /opt/cni:/opt/cni:ro
-v /sys:/sys:ro
-v /var/run:/var/run:rw
-v /var/lib/kubelet:/var/lib/kubelet:shared
-v {{etcd_config_dir}}:{{etcd_config_dir}}
-v {{kube_config_dir}}:{{kube_config_dir}}
-v {{ docker_daemon_graph }}:/var/lib/docker:rw
{{ hyperkube_image_repo }}:{{ hyperkube_image_tag}}
./hyperkube kubelet
$@

If you host mount the /etc/ssl dir, then it will probably fail on redhat/CentOS due to invalid symlinks. I had this issue in the past for kubeadm, but I can't dig up the issue right now as I'm on my mobile atm.
I'd assume mounting that one is not required as hypekube has valid ca certificates installed. Or does kargo install something there?

@holser Unfortunately you have dns related CI failure. If you feel it's a flake, I can retrigger the build.

According to logs it's not related to this PR. Please retrigger CI.

So, I tried this in my environment.  KVM instances running ubuntu16.04 and it comes up but the net checker calls show intermittent DNS replies.  components come and go.  I noticed that the DNSmasq logs were showing that the 150 limit was being hit.  I upped the limit to 15000 and while DNSmasq doesn't show the error anymore, I still intermittent DNS issues.  I'm not sure what is going on beyond that.  This is the defaults of dnsmasq_kubedns and docker_dns.

You may want to ignore me.  I don't run the system from cluster.yml and that could be my problem.

Great work, @holser!


Add AllowClients Field to packets

Adds a new AllowClients field to the packet structure. This field can be set during the OnMessage event hook, allowing an embedding server to selectively deliver messages to one or more clients within one or more topics based on their client id. For example:
// Add OnMessage Event Hook
server.Events.OnMessage = func(cl events.Client, pk events.Packet) (pkx events.Packet, err error) {
  pkx = pk

  // Example of using AllowClients to selectively deliver/drop messages.
  // Only a client with the id of `allowed-client` will received messages on the topic.
  if pkx.TopicName == "a/b/restricted" {
    pkx.AllowClients = []string{"allowed-client"} // slice of known client ids
  }

  return pkx, nil
}

If AllowClients is nil or not set, it is ignored and the message is delivered as normal.
All unit tests passing, paho compatibility tests passing

Codecov Report

Merging #14 (fc61cc9) into master (fb13648) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master      #14   +/-   ##
=======================================
  Coverage   98.96%   98.96%
=======================================
  Files          18       19    +1
  Lines        2020     2027    +7
=======================================
+ Hits         1999     2006    +7
  Misses         19       19
  Partials        2        2

Impacted Files
Coverage Δ

server/internal/packets/packets.go
100.00% <ø> (ø)

server/internal/clients/clients.go
98.92% <100.00%> (ø)

server/internal/utils/utils.go
100.00% <100.00%> (ø)

server/server.go
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Incompatability With Django 1.8

djangotoolbox doesn't work on django 1.8.
After the work that was done by @kavdev in #56  it seems like most of the errors are generated from Django moving around their db modules and packages significantly in this release.
Can we get #56 merged in so we can work on porting to Django 1.8?

Sorry it took so long for me to get around to this, but #64. Please give it a shot and let me know if I missed anything.

There are some imports, that does not work in 1.8, what must be:
#base.py
from django.conf import settings
from django.db.backends.base.features import BaseDatabaseFeatures
from django.db.backends.base.operations import BaseDatabaseOperations
from django.db.backends.base.base import BaseDatabaseWrapper
from django.db.backends.base.client import BaseDatabaseClient
from django.db.backends.base.validation import BaseDatabaseValidation
from django.db.backends.base.introspection import BaseDatabaseIntrospection

#creation.py
from django.db.backends.base.creation import BaseDatabaseCreation

Not sure why you're getting an error, but those imports are indeed correct. See here

I've committed more changes since you've looked at it though; there might have been a bug that should now be resolved.

In master branch are another imports https://github.com/django-nonrel/djangotoolbox/blob/master/djangotoolbox/db/base.py#L5
In django 1.7.x it worked, because all imports were in init file https://github.com/django/django/blob/stable/1.6.x/django/db/backends/init.py
But in 1.8 this file is empty, so all imported classes are in another location https://github.com/django/django/blob/stable/1.8.x/django/db/backends/init.py
In my last comment I wrote how it must be imported

The pull request I mentioned (#64) was created to fix all off the issues you've mentioned. If you want to test the new changes, you need to checkout the incoming changeset.

Fixed in #64


Data output from MYSQL to template

Could you tell me please as to withdraw from MYSQL data in pattern? (view.php)
You need to display the name of the channel. How can I do that?

I don't understand the question. Please explain better.

I don't understand the question. Please explain better.

I don't want to use Vue.
Me need a template to display the title of the article and the link of the article (In the database is the

Submission - Title

and

Submission - Slug

How to display the name in the template (submission.view)? I try

{{  $submission->title }}

in view template, but i get error

ErrorException (E_ERROR) Undefined variable: submission

Thank you.


update activation_flow.md title for testnet

https://github.com/threefoldtech/tfchain_activation_service/blob/master/docs/activation_flow.md
this docs for testnet
Creating an Account on TFChain Devnet    this title need to change to Creating an Account on TFChain Testnet

not relevant anymore


More support for seed offsets

It would be nice to have more support for seed offsets, for example when creating an new seed or viewing a seed of an owned wallet

for example when creating an new seed

Added in 13c3e9b8e2b8f7a3e5ed08d5a19f98dee4cbda4e.

viewing a seed of an owned wallet

Wallets based on 14 or 16 word seeds will show the seed offset in seed dialog. I'm aware of a bug that prevents the seed offset from showing for 25 word seeds, hoping to fix this before the next release.


[App Service] az functionapp deployment github-actions: Add linux powershell support and fix publish profile bug

Related command
az functionapp deployment gihub-actions
Description
Minor improvements to the functionapp github-actions command.
Testing Guide
Manual testing since we cannot hardcode github tokens
History Notes
[App Service] az functionapp deployment github-actions: Add support for linux powershell runtimes
[App Service] az functionapp deployment github-actions: Fix issue where publish profile would not be populated before the github action was run

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

App Service

@haroonf Please do not add a list symbol under the History Notes. I have updated it for you

@haroonf Could you add some tests for those PR change?

@haroonf Could you add some tests for those PR change?

These changes impact github-actions commands, which we can't write tests for since we require a GitHub Authentication Token.


fix: refactor JSX namespace to React.JSX

Try out this version of the Hiro Wallet - download extension builds.

Description
This is a small change to migrate from the deprecated global JSX space to React.JSX.
I noticed we get a lot of these warnings:

It's not a serious warning but its nice to clean them if we can so more important ones are more noticeable
Type of Change

Refactor

Is there any way to turn this particular deprecation warning into an error now it's fixed?

Is there any way to turn this particular deprecation warning into an error now it's fixed?

I'm not sure but I will check it out. There probably is


(Bug) MqttCommunicationException on ConnectAsync

Version 2.1.3 has a connection bug it seems.
A request to send or receive data was disallowed because the socket is not connected and (when sending on a datagram socket using a sendto call) no address was supplied
at MQTTnet.Implementations.MqttTcpChannel.WriteAsync(Byte[] buffer)
at MQTTnet.Core.Serializer.MqttPacketWriter.WriteToAsync(IMqttCommunicationChannel destination)
at MQTTnet.Core.Serializer.DefaultMqttV311PacketSerializer.SerializeAsync(MqttConnectPacket packet, IMqttCommunicationChannel destination)
at MQTTnet.Core.Serializer.DefaultMqttV311PacketSerializer.SerializeAsync(MqttBasePacket packet, IMqttCommunicationChannel destination)
at MQTTnet.Core.Adapter.MqttChannelCommunicationAdapter.d__5.MoveNext()
This occurs with and without TLS enabled.  The only thing I can gather so far is that the connection fails (via MqttClient.cs line 61) before the packet is sent (at MqttClient.cs line 74).
Can anyone else confirm this issue?
For the record, my MqttClientOptions are:
MqttClientOptions opts = new MqttClientOptions
{
    ClientId = Guid.NewGuid().ToString(),

    UserName = "flight|eedovmDCY48yEZtlXPdJwfeayNK5",
    Password = "...",

    Server = "ssl://mqtt-prod.airmap.io",
    Port = 8883,

    CleanSession = true,
    KeepAlivePeriod = TimeSpan.FromSeconds(15),
    DefaultCommunicationTimeout = TimeSpan.FromSeconds(2)
};

Hi, please try without the "ssl://" prefix. Such a prefix is not supported in the library because the new "UseTsl" flag is used for this.

Yep, that'd do it.  Perhaps a case to allow ssl:// prefixes (if it's detected, automatically set the UseTLS flag).

If you would like to use it you can still create a pull request and add this. But you need to check the options because ssl:// and "UseTsl=false" does not make sense.


Added jruby-head

Show that the newest JRuby version is passing.

Looks like there was a SEG Fault when running the 'rbx-2' ruby version.
warning: parser/current is loading parser/ruby21, which recognizes
warning: 2.1.6-compliant syntax, but you are running 2.1.0.
warning: please see https://github.com/whitequark/parser#compatibility-with-ruby-mri.
Inspecting 23 files
...The Rubinius process is aborting with signal: SIGSEGV```

Looks like there was a SEG Fault when running the 'rbx-2' ruby version.

I've noticed that a few times recently. I'm not sure what's going.


Error CSS Stylesheet

Apart from the XML-parser problem I am having problems with the css stylesheet.
Since I am not sure if this problem relates to the XML, I am posting this as a separate issue.
In the electron dev log I am getting error messages such as:

Error on loading stylesheet: modules/MMM-RBB-Weather/font-awesome5.css

Appreciate any help.
Raspbian 10 buster
MM 2.26.0
nodejs 18
Thanks in advance.

Hi, are your sure this is an error related to the module? It seems that the same error appears for your css/custom.css, so the problem is maybe something more general?

No, not completely sure.
It simply seemed odd to me that the file the module needs (font-awesome5) should be in the module folder but isn't there.
Or does the module refer to the systems CSS files?

Have you looked into the module folder to check the file? According to the screenshot, the problem are not missing files, but wrong MIME types.
The module doesn't refer to the system CSS files.

Yes, I checked the module folder, there is no such file (font-awesome5.css):

xxx@raspberrypi:~/MagicMirror/modules/MMM-RBB-Weather $ ls -al
insgesamt 600
drwxr-xr-x   9 pi pi   4096 Feb  5  2022 .
drwxr-xr-x   7 pi pi   4096 Apr  8  2022 ..
-rw-r--r--   1 pi pi   2488 Jan 23  2022 CHANGELOG.md
-rw-r--r--   1 pi pi    724 Jan 23  2022 .eslintrc.json
drwxr-xr-x   8 pi pi   4096 Feb  2 22:12 .git
drwxr-xr-x   2 pi pi   4096 Jan 23  2022 .github
-rw-r--r--   1 pi pi     60 Jan 23  2022 .gitignore
-rw-r--r--   1 pi pi   3159 Jan 23  2022 IconMapper.js
-rw-r--r--   1 pi pi   1080 Jan 23  2022 LICENSE.txt
-rw-r--r--   1 pi pi    679 Jan 23  2022 Logger.js
-rw-r--r--   1 pi pi   1146 Jan 23  2022 MMM-RBB-Weather.css
-rw-r--r--   1 pi pi  10284 Jan 23  2022 MMM-RBB-Weather.js
-rw-r--r--   1 pi pi   5127 Feb  5  2022 node_helper.js
drwxr-xr-x 465 pi pi  20480 Feb  2 21:03 node_modules
-rw-r--r--   1 pi pi   1475 Feb  2 20:26 package.json
-rw-r--r--   1 pi pi 488363 Feb  2 21:03 package-lock.json
-rw-r--r--   1 pi pi  10189 Jan 23  2022 README.md
drwxr-xr-x   2 pi pi   4096 Jan 23  2022 templates
drwxr-xr-x   5 pi pi   4096 Jan 23  2022 test
drwxr-xr-x   2 pi pi   4096 Jan 23  2022 translations
-rw-r--r--   1 pi pi    365 Jan 23  2022 .travis.yml
drwxr-xr-x   3 pi pi   4096 Jan 23  2022 vendor

Here is the error message from electron log, which was pointing me to the missing file.

Any help on this one, please?

Will close this issue now after it started working again.
I reseted (git reset --hard) set nvm explicitly to 18 (nvm use 18) and reinstalled the module. For some reason there were two npm versions installed (18 and 20).
Now it works again.
Thanks anyway for trying to help a bit.


Provide centroid for each brain region

Each brain region should have a property reflecting its centroid in physcial coordinates. This feature was suppressed when moving to siibra-python 0.2x for being migrated to a maintenance module, and needs to be re-introduced.

Has been reintroduced with 66178ffa1912570b07216db46faf714def5acb36 and is part of  recent releases as Region.spatial_props(), as well as since d1c8871093a0829d9c8faaca85b093fb3203f533 as Region.centroids().


[API Improvement] : Dockerize the OnlineBookSales_API #125

🎯 Title and Issue Number
[API Improvement] : Dockerize the OnlineBookSales_API #125
Closes #125
🗒 Checklist:

[Y] I have mentioned the issue number in my Pull Request.
[Y]  I have gone through rules of when PR will get merged.
[Y] I have commented my code, particularly in hard-to-understand areas
[Y] I have created a helpful and easy to understand README.md
[Y] I have followed proper file structure.
[Y] I have gone through

Thank you for contributing!❤
Don't forget to show some support by ⭐ the repository!

Hello @sanjay14073, thank you for your pull request! We will review it shortly. 😇❣
Your contributions help us grow and improve our project. Please make sure you have read and followed our CONTRIBUTING GUIDELINES.
Your efforts are appreciated, and we value your participation in our community. Don't forget to ⭐ our repository!
Happy coding!✨

@Puskar-Roy i have raised the pr could you look into it once


Fixed: Messages with flags are displayed with contra code

Details
Changed the emoji font order
Fixed Issues
$ https://github.com/Expensify/App/issues/13070
PROPOSAL: https://github.com/Expensify/App/issues/13070#issuecomment-1328321538
Tests

Use Chrome on Windows (or on any OS as long as Segoe font is installed)
Send a flag emoji
Verify that the flag is displayed correctly and not the country code

[ ] Verify that no errors appear in the JS console

Offline tests
Not Applicable
QA Steps

Use Chrome on Windows (or on any OS that as long as Segoe font is installed)
Send a flag emoji
Verify that the flag is displayed correctly and not the country code

[ ] Verify that no errors appear in the JS console

PR Author Checklist

[X] I linked the correct issue in the ### Fixed Issues section above
[X] I wrote clear testing steps that cover the changes made in this PR

[X] I added steps for local testing in the Tests section
[X] I added steps for the expected offline behavior in the Offline steps section
[X] I added steps for Staging and/or Production testing in the QA steps section
[X] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[X] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[X] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[X] I included screenshots or videos for tests on all platforms
[X] I ran the tests on all platforms & verified they passed on:

[X] iOS / native
[X] Android / native
[X] iOS / Safari
[X] Android / Chrome
[X] MacOS / Chrome
[X] MacOS / Desktop

[X] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[X] I followed proper code patterns (see Reviewing the code)

[X] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[X] I verified that comments were added to code that is not self explanatory
[X] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[X] I verified any copy / text shown in the product was added in all src/languages/* files
[X] I verified any copy / text that was added to the app is correct English and approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[X] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[X] I verified the JSDocs style guidelines (in STYLE.md) were followed

[X] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[X] I followed the guidelines as stated in the Review Guidelines
[X] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[X] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[X] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[X] I verified that if a function's arguments changed that all usages have also been updated correctly
[X] If a new component is created I verified that:

[X] A similar component doesn't exist in the codebase
[X] All props are defined accurately and each prop has a /** comment above it */
[X] The file is named correctly
[X] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[X] The only data being stored in the state is data necessary for rendering and nothing else
[X] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[X] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[X] All JSX used for rendering exists in the render method
[X] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[X] If a new CSS style is added I verified that:

[X] A similar style doesn't already exist
[X] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG)

[X] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[X] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[X] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web
https://user-images.githubusercontent.com/16493223/204869056-de22fbe5-0a72-434e-bace-82d8bf7646ad.mp4

Mobile Web - Chrome

Mobile Web - Safari

Desktop

iOS
Not Applicable

Android
Not Applicable

@sobitneupane Updated.

Thank you for testing @sobitneupane, I do not have my dev environment set up on a windows machine so I'll be leaning on you for this one

@sobitneupane Can you check if you have Noto Color Emoji font installed
I tested on Linux where I have both Segoe and Noto, it was using Segoe at first since it was listed first, after the change it's using Noto and the issue has been fixed on my side.
If you don't have Noto Color Emoji it will fallback to Segoe and if so it's expected to still get the flags as countries codes.
It may be worth checking the Windows version, perhaps Noto was added on a new newer build

@s77rt I had not used the device from last few years.

Do we have any other fonts present in older versions to fallback?

@sobitneupane I think Windows does support Noto Color Emoji but it's not installed per default.

Do we have any other fonts present in older versions to fallback?

The fallback is Segoe but it does not have flags support, no other font that i know of can be used as another fallback
Maybe we should include a @font-face declaration for this font to be collected from an external source
https://github.com/Expensify/App/blob/main/assets/css/fonts.css

I think it makes sense to still merge given it still fixes the issue on an entire OS. There is a deploy blocker now though (easy fix)

I didn't see this issue on ubuntu 18.04 and 22.04. I tested sending flags on staging from chrome.

@sobitneupane That's because you don't have Segoe UI Emoji font installed, and it's falling back to Noto.
On a system with both fonts, it will use Segoe which is wrong.
With this PR, on a system with both fonts, it will use Noto

As per @s77rt, this will fix the issue in one specific case.

On a system with both fonts, it will use Segoe which is wrong.

I am not sure if we want to move with this solution or wait for the one which solves the issue for all the cases. I will wait for confirmation from @stitesExpensify before moving forward with the testing.

Maybe I'm misunderstanding here. @s77rt is Segoe UI Emoji installed by default on any widely used OS? If you need to manually install it on both Ubuntu and Windows, I don't think it is worth fixing.

@stitesExpensify Segoe is installed per default on every Windows.
On Mac, it's not.
On Linux, it depends, I don't remember explicitly installing Segoe font but I think i just happen to install Microsoft fonts on the installation process of the os
So:

Windows: The issue will be fixed once you install Noto fonts (if this PR is merged)
Mac: The issue is fixed per default but will you will face it once you install Segoe fonts  (unless this PR is merged)
Linux: Installation is pretty much custom so it depends on what fonts you have

My apologies, my comment should have said:

Is Noto Color Emoji installed by default on any widely used OS? If you need to manually install it on both Ubuntu and Windows, I don't think it is worth fixing.

Based on your last comment, it seems like this PR won't fix anything for the vast majority of users, only those who specifically download fonts.

@stitesExpensify Yes, i guess that's correct

Closing this out per this comment https://github.com/Expensify/App/issues/13070#issuecomment-1349027136


Auto assign when create a task in a DM

Details

When creating a task in a DM, we want to auto-assign for a participant in this DM
Fixed Issues

$ https://github.com/Expensify/App/issues/19048
PROPOSAL: https://github.com/Expensify/App/issues/19048#issuecomment-1551694235
Tests

Login with Expensifail account
Open a DM
Click on plus icon in this DM --> Assign task
Enter title, description and go to the next step
Verify that assginee that appears is a participant of this DM
Open any other chat that isn't a DM
Repeat steps 3 - 4 again
Verify that no assignee is auto assigned

[x] Verify that no errors appear in the JS console

Offline tests

Same above
QA Steps

Login with Expensifail account
Open a DM
Click on plus icon in this DM --> Assign task
Enter title, description and go to the next step
Verify that assginee that appears is a participant of this DM
Open any other chat that isn't a DM
Repeat steps 3 - 4 again
Verify that no assignee is auto assigned

[] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web
Screencast from 09-06-2023 16:00:13.webm

Mobile Web - Chrome
https://github.com/Expensify/App/assets/129500732/dd5301d5-297f-45f7-9b27-230cbcbfe74e

Mobile Web - Safari
https://github.com/Expensify/App/assets/129500732/2d8e1947-7801-47b6-953b-8aa3fbc2a188

Desktop
https://github.com/Expensify/App/assets/129500732/98ddec84-fbc7-4f0c-90bf-635d48dc673e

iOS
https://github.com/Expensify/App/assets/129500732/e7ff31d7-8b3e-4a79-aa98-c73a22fd277f

Android
https://github.com/Expensify/App/assets/129500732/ebc4de9f-5e56-41c4-abcb-f649fdb338d9

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/MariaHCD in version: 1.3.28-0 🚀

platform
result

🤖 android 🤖
failure ❌

🖥 desktop 🖥
failure ❌

🍎 iOS 🍎
failure ❌

🕸 web 🕸
failure ❌

🚀 Deployed to production by https://github.com/AndrewGable in version: 1.3.28-5 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Removed depracated functions in mongo 3.4, added Oplog lag and oplog sync metrics

I've removed the background flushing and heap size metrics which have been deprecated in mongo 3.4 as mentioned in issue #1
Further to this I have added a couple of checks which we consider essential, these are checks which look at the oplog sync and oplog lag. I was trying to make these work so they followed your code style/conventions, but I was unable to get the cursor to work (I always hit a StopIteration, even when I checked that the cursor was alive as per the docs: http://api.mongodb.com/python/current/api/pymongo/cursor.html) so I ended up doing this in the only way I could get it to work. Maybe you can improve my code to make this work and keep it in the same style your original code worked.

Further to my previous PR I have added my other checks, this includes a check for fsync lock, hidden and priority. These are useful when doing maintenance on mongodb in our environment.

@Rumbles, thanks for your contribution.


I get a second row of column headers added when clicking the back button

I use tablefilter on a table of tickets.  There are links in the table.  When I click a link to go to a ticket and then click back to go back to the tablefilter-ed list the tf column headers are duplicated.
Is there a way to stop table filter loading if its already loaded?

Hi @cjohnsonuk,
It is probably not a full page navigation happening in there, probably something like a partial load (SPA routing?).
Not sure the headers are provided by your HTML, weird. What is your TableFilter configuration?


Fight Caves Spawn Predictor Release - v1.0.0

Utilizes the logic discovered on Fight Caves Rotations to determine where the fight caves NPCs will spawn. Review README for further information on information and logic applied.

Don't use enabledByDefault = false, it's confusing for users: https://github.com/damencs/spawn-predictor/blob/c07c847f71f10a7b69ca3f6b7d3141c39ad3f035/src/main/java/com/spawnpredictor/SpawnPredictorPlugin.java#L68

Don't use enabledByDefault = false, it's confusing for users: https://github.com/damencs/spawn-predictor/blob/c07c847f71f10a7b69ca3f6b7d3141c39ad3f035/src/main/java/com/spawnpredictor/SpawnPredictorPlugin.java#L68

@Nightfirecat - my fault, I forgot again, lol.


SSL Protocol

In LogListDataSourceFactory, is there a reason why SSL is specifically used here and not TLS ?
sslContext = SSLContext.getInstance("SSL")

Wondering the same since LogListDataSourceFactory class is marked by Veracode as insecure due to the use of the SSL protocol. The recommendation to remove this warning is to replace the protocol with TLS (TLS 1.2 preferred). Is there a  reason to maintain SSL?

This is a very good question. I'm also not entirely sure why this hadn't been raised in any of the security scans the library has been in that I'd seen. 🤷
While it may say SSL, SSLv3 support was dropped since Java SE 7u75 so it will usually be using TLS as a minimum anyway. That being said, the min Android SDK is API 19 (Android 4.4), and I'm not sure on its supported protocols.
It's worth noting that the log list data is signed with a public/private key pair so the risk comes down to man in the middle attack pushing stale data which the library protects itself against given that log-list.json the file contains a timestamp.
Given the library makes as few assertions of what platform it is running on I cannot easily enforce a precise protocol (or cipher suite), but will expose OkHttp's ConnectionSpec to allow this to be configured through LogListDataSourceFactory.createLogListService. The default is MODERN_TLS.


It is downloading for a long time, but still 0% How can I fix it?

I was using this so good for a long time. But today, this problem happened.
So how could I fix it?

Hard to say, can you try with a different video URL? Same story? Try upgrading your ffmpeg version, can't promise that will change anything but is worth a shot.

Was this solved @ASHREDpikachu?

Closing as inactive


python3Packages.thinc: 7.4.1 -> 7.4.2

Motivation for this change
Changelog:
https://github.com/explosion/thinc/releases/tag/v7.4.1
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@ofborg build python37Packages.spacy python38Packages.spacy

Result of nixpkgs-review pr 102998 run on x86_64-darwin 1

6 packages built:

python37Packages.spacy
python37Packages.textacy
python37Packages.thinc
python38Packages.spacy
python38Packages.textacy
python38Packages.thinc


when installing druid on k8s

When installing druid on top of k8s, we set middlemanager and historical to come up on the same node. After that, when proceeding with the volume setting, will additional settings be required?
For example, set the same path of the volume

each pod acts as its own machine, as long as you have a zookeeper they will register themselves. Look at the Druid operator, it makes the process much more straightforward. Notice that having multiple replicas under one service will only act as one server and could cause problems.


Collector configuration doc is missing new service.telemetry subsection

What needs to be changed?
Update collector configuration doc with new service.telemetry section
What is the name + path of the page that needs changed?
https://opentelemetry.io/docs/collector/configuration/#service
Additional context
With #4069 in opentelemetry-collector there is now a new subsection, telemetry, allowed under service.  The docs should be updated with details on all the new configuration allowed under the telemetry subsection.

@TylerHelmuth, would you be open to sending in a PR? If so, just ping me and I'll review it.

@TylerHelmuth, would you be open to sending in a PR? If so, just ping me and I'll review it.

I'll need to brush up on the new telemetry config code in the collector but I can take a shot.

If you need a primer, this is how you can configure it:
receivers:
  otlp:
    protocols:
      grpc:

exporters:
  logging:

processors:

extensions:

service:
  extensions:
  pipelines:
    traces:
      receivers: [otlp]
      processors:
      exporters: [logging]
  telemetry:
    logs:
      level: debug
      initial_fields:
        service: my-instance
    metrics:
      level: detailed

The code with all the available options can be seen here:
https://github.com/open-telemetry/opentelemetry-collector/blob/7666eb04c30e5cfd750db9969fe507562598f0ae/config/service.go#L35-L111

@jpkrohling The only things I'm not 100% on is if the default value for metrics.address, which I believe is :8888 means that the collector by default sends itself metrics on port 8888.  I noticed most configurations of the collector explicitly list <IP_ADDRESS>:8888 instead, but isn't that the same as the default value?

Indeed, the default port is 8888. With the example I gave previously, metrics can be scraped from localhost:8888/metrics. I'd say most configs explicitly setting the metrics port to :8888 are just a result of copy/paste without further consideration.


allow overriding replication factor for 'test'

in our prod environment, we use simplestrategy with >1 replication factor.
seems like test and prod are a bit tied to uber's environments, but we can do what we want with this PR by using "test" (actually prod) and a custom replication factor.

Coverage remained the same at 100.0% when pulling b2078c7768571a54b903d998ff67dcb30a128d70 on Dieterbe:test-override-replication-factor into 975982bc79f255490cc111993cc3ddcc636f0927 on jaegertracing:master.

Please make sure all commits are signed

Coverage remained the same at 100.0% when pulling 5b6388224880bc8cded54b45bf6c35480fa1fe8b on Dieterbe:test-override-replication-factor into 3eddf478f098dbff9cec02451dec7c5a83bcebe0 on jaegertracing:master.

Coverage remained the same at 100.0% when pulling 5b6388224880bc8cded54b45bf6c35480fa1fe8b on Dieterbe:test-override-replication-factor into 3eddf478f098dbff9cec02451dec7c5a83bcebe0 on jaegertracing:master.


Unexpected error: nil pointer dereference

When no tailnet defined.

Hi,
Can you provide some more details on how to reproduce this error?

Sure, this is my steps:
1. Start ionscale with oidc and no tailnets defined
2. Login to tailscale using SSO/OIDC
3. Error raised

Hi,
I'm not able te reproduce the error. What versions of ionscale/tailscale are you using?


Fix MicroModal.close now closes the modal based on the ID passed

This is the initial pass at fixing #112 . Let me know if anything needs a tweak to be a better fit for the project :)
It also enables development to be done on a Windows environment with the addition of the cross-env library.

I also ran into that bug. The documentation states that a modal can be closed by its id but it's not the case.
Also, the activeModal variable is set in the the show function. It should be set in the showModal function instead. This way a modal instantiated with the data-micromodal-trigger could also be closed programatically.

Reference #113

Thank you @roebuk. Appreciate you writing the tests as well. Cheers!

I still get an error...
TypeError: null is not an object (evaluating 'activeModal.closeModalById')

I am using Turbolinks.
import MicroModal from 'micromodal'

document.addEventListener('turbolinks:load', function () {
  MicroModal.init()
})

document.addEventListener('turbolinks:before-cache', function () {
  MicroModal.close('modal-id')
})

The issue linked to this has been closed but we're having the exact same problem, either there is still an issue or the fix never made it into the latest version for download from the micromodal website.

Yes, it’s not released yet. Closing a few other issues as well. Will update soon.

Is this issue has been fixed?

@ksromero I still have this problem.

HI @ghosh ! When this PR will be released ?

i'm still having this issue, has anyone got a working alternative?

any update?


add unified_mode true

Description
Adds cookbook compatibility with Chef 17
Issues Resolved
#484
Check List

[X] All tests pass. See TESTING.md for details.
[X] New functionality includes testing.
[X] New functionality has been documented in the README if applicable.

Released as: 8.0.0


Add support for Homebrew

It would be nice to be able to install SwiftCov using hombrew.

See https://github.com/Homebrew/homebrew/pull/40332 for some backstory as to what we did to try to support Homebrew, and how I eventually gave up because Xcode 7 now generates Swift test coverage information.
Feel free to submit a patch to homebrew for this though.

OK so you are not longer maintaining this?

I'll let @kishikawakatsumi answer that.

@guidomb I'll continue to maintain this project. Because I think Xcode coverage feature is not enough in some situations (e.g. Send coverage report service, HTML report, branch coverage, and so on), and for my interest.
I also would like to support to install via Homebrew, but it is difficult because I do not know the cause of the test to fail. I would be happy if you could help to solve it.


In the marker gene heat map the cluster labels are not in the correct sorting order

In the Single Cell Expression Atlas the cluster labels are not in the correct sorting order in the marker gene heat map.
Steps to reproduce:

Open the following experiment: https://www-test.ebi.ac.uk/gxa/sc/experiments/E-MTAB-10448/results/cell-plots
Go to the Marker genes tab

Here is the full URL:  https://www-test.ebi.ac.uk/gxa/sc/experiments/E-MTAB-10448/results/marker-genes
Expected result:
The cluster labels are sorted naturally.
Actual result:
The cluster labels are not sorted naturally. Look at the screen shot.

I can also reproduce it in our local env with this experiment:
http://localhost:8080/gxa/sc/experiments/E-GEOD-99058/results/marker-genes

It looks like it is not a sorting order error. If we select the same k value then the page refresh itself correctly and the cluster names will be prefixed with Cluster. After that the sorting order of the cluster names is correct.
This might be related to this previous ticket: Marker genes shown in marker gene plot on first landing not consistent with gene shown when single cluster view selected that might not be fixed entirely.


fix: update policy ctrl sha and operator go version for psirt

https://github.ibm.com/IBMPrivateCloud/roadmap/issues/40261
https://github.ibm.com/IBMPrivateCloud/audit-policy-controller/pull/40

@lauriet-ibm ready for review


Added vertical_min, vertical_max (MLDB-1663)

+1

Test failures are related to MLDB-1634 and will pass on master once merged.


🔧 chore: added command for blog contribute

Added two new functions to simplify contribution

Add Author from GitHub: AddUser function GitHub users as authors to a YAML file.
It fetches user data from GitHub API and updates the YAML data.
Create Blog:  creates markdown or directory structures for blog posts.
Automatically generates file/directory names based on the provided title and structure.
Updates markdown file with title, author, and creation date.

btw: Tính ngồi viết 1 bài blog dạng handbook hoặc cheatsheet của thằng docker nhưng... effort đó đã để ngồi tạo cái này huhu 🥲
🥹


Improve the configuration of the embedded OrientDB

We need to resolve the warnings when the embedded OrientDB instance is launched. At the moment, we're getting these warnings during the build.
[INFO] Running org.carlspring.strongbox.services.MavenChecksumServiceTest
23:39:24.192 13-02-2018 | INFO  | main                 | DataServiceConfig                                  | Start Embedded OrientDB server [remote:<IP_ADDRESS>:2424/strongbox].

2018-02-13 23:39:24:988 WARNI MaxDirectMemorySize JVM option is not set or has invalid value, that may cause out of memory errors. Please set the -XX:MaxDirectMemorySize=15896m option when you start the JVM. [OMemory]
2018-02-13 23:39:27:728 INFO  [OVariableParser.resolveVariables] Error on resolving property: ORIENTDB_HOME [orientechnologies]
2018-02-13 23:39:27:729 SEVER ODefaultServerSecurity.loadConfig() Could not access the security JSON file: /config/security.json [ODefaultServerSecurity]23:39:28.483 13-02-2018 | DEBUG | main                 | DataServiceConfig                                  | Create database strongbox

@carlspring also I suggest to make configurable parameter for OrientDB memory usage limitation which is -Dstorage.diskCache.bufferSize if I remember correctly.

Yes, I agree.

@sbespalov ,
Should this task now be closed?

@carlspring no, this warning still appears in log.

Closing this, as it will become obsolete with #1649 ( #1676 ).


Fh 2919

JIRA
https://issues.jboss.org/browse/FH-2919
Changes

update the Jenkinsfile to match the file currently used for demo purposes

@odra @wei-lee

👍

@wei-lee I made the change suggested. Not sure what is the best way to test this. Can i do it in OSM4?

@odra could you take a look at the last commit when you get some time. Not sure if is correct. It is about updating the fhconfig.json file with the FH_CONFIG_CONTENT param

👍

@odra pushed the changes for the archive. Thanks for the help with solving the issue. @wei-lee could you take another look and approve changes if you are happy.

Successful build for debug and release using this Jenkinsfile here

@wei-lee added the output as suggested


Fixed spelling

Updated spelling from clinking to clicking

[ ] I have read freeCodeCamp's contribution guidelines.
[ ] My pull request has a descriptive title (not a vague title like Update index.md)
[ ] My pull request targets the master branch of freeCodeCamp.
[ ] None of my changes are plagiarized from another source without proper attribution.
[ ] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

👍 LGTM
You can check off the checkboxes by adding x to [] to finish off your PR
[] -> [x]


Add runVerify, runAll, and runAllVerify helper methods in ScriptInter…

…preter

Coverage increased (+0.2%) to 90.488% when pulling 73d439f81ad4da06d7b93dda74323b669a710884 on Christewart:interpreter_run_verify into 409e489937485395b533fdd48e973f50e4ece188 on bitcoin-s:master.


AsyncTask Hunting

Improve AsyncTasks memory allocation through monitoring and code changes.
Currently all async tasks are inner classes with implicit references to its activity.
Read more here

First step has been made, saving some leaked activities cancelling AsyncTasks execution when Activity#onPause happens.
Changes: https://github.com/alabeduarte/mypodcasts-android/commit/740fcab500f0dd18bd61a6510e548fcabb1b5d4d


HAOS warning

Hi there,
Recieved the folowing error after the latest upgrade.
The integration weatherapi is using the deprecated forecast attribute.
Please report it to the custom integration author.

I’m receiving the same warning. HA 2023.9.0, Supervisor 2023.08.3, OS 10.5, WeatherAPI integration v1.1.0

This requirement is new as of 2023.9.0. You should not see any functional shortcoming till 2024.3.0 .. I will take a look at it sometime.


Add python code format check

As we will add more Python code, I think we should consider setting up a formatter tool, and also add the Python format check into CI.

Closed due to finish


Can I classify Chinese text please?

Do I need to change anything please? Thank you.

Sorry, just saw the language option.


remote RT dotnet not match local dotnet-isolated

Hi,
I got this error while i tried to deploy isolated process func into one of my func app:
The remote runtime "dotnet" for function app "rmbsfunc" does not match your local runtime "dotnet-isolated".
my question is how can i create func app of which the runtime is dotnet-isolated, when i tried to create one on azure portal, i didn't see .net(isolated) in runtime option.

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

ID: 76840d73-f300-0ad5-b9f5-4cfc366ad28e
Version Independent ID: 1f79962e-2c4c-04d0-a74c-7d6ecd34ff78
Content: Create a C# function using Visual Studio Code - Azure Functions
Content Source: articles/azure-functions/create-first-function-vs-code-csharp.md
Service: azure-functions
GitHub Login: @ggailey777
Microsoft Alias: glenga

Thank you for your feedback! We will review and update as appropriate.

You can't yet create a C# function app that runs in isolated process in the Azure portal. You must instead use something like the Azure CLI. Visual Studio Code also lets you create an isolated process app in Azure.


Add walkthrough for how to use VS extension to develop Linux enclave apps

This walkthrough uses v0.5 of the extension, and OE SDK built from the master branch

Not clear in this workflow where the compiler is being set. Is it already clang or does the user have to specify it?

It's already set to clang automatically in the generated project.

Re "The only critical issue that affects semantics/security is to make sure that the correct compilation flags are used to build an enclave."   Fixed.  (no doc changes)

bors r+


Configure build environment for Web-CAT.

Thanks! I'm testing this now in my own Vagrant to make sure the build environment it creates can do the build.

When I run this, I get:
BUILD FAILED
/vagrant/web-cat/Web-CAT/build.xml:287: /vagrant/web-cat/Web-CAT/${servlet.jar.dir} does not exist.

Total time: 13 minutes 5 seconds

seems like there is one small-ish problem left. Can you test it on a fresh virtual machine, and see if you get the same error?
For what it's worth, I made a few small changes. Here they are:
diff --git a/install.sh b/install.sh
index d08380b..bad2b2e 100755
--- a/install.sh
+++ b/install.sh
@@ -13,13 +13,13 @@ if [ ! -f /usr/bin/javac ] ; then
 fi

 # install WebObjects
-[ -f WOInstaller.jar ] || curl -# -O http://wocommunity.org/tools/WOInstaller.jar
+[ -f WOInstaller.jar ] || ( curl -# http://wocommunity.org/tools/WOInstaller.jar -o WOInstaller.jar.tmp && mv WOInstaller.jar.tmp WOInstaller.jar )
 wodir=/Library/WebObjects/Versions/WebObjects543
 sudo mkdir -p $wodir
-[ -d $wodir/Library/Frameworks/JavaXML.framework ] || sudo java -jar WOInstaller.jar 5.4.3 $wodir
+[ -d $wodir/Library/Frameworks/JavaXML.framework ] || ( sudo java -jar WOInstaller.jar 5.4.3 $wodir || (sudo rm -rf "$wodir" ; exit 1) )

 # install Wonder Frameworks
-[ -f Wonder-Frameworks.tar.gz ] || curl -# -O -L https://jenkins.wocommunity.org/job/Wonder/lastSuccessfulBuild/artifact/Root/Roots/Wonder-Frameworks.tar.gz
+[ -f Wonder-Frameworks.tar.gz ] || ( curl -# -L https://jenkins.wocommunity.org/job/Wonder/lastSuccessfulBuild/artifact/Root/Roots/Wonder-Frameworks.tar.gz -o Wonder-Frameworks.tar.gz.tmp && mv Wonder-Frameworks.tar.gz.tmp Wonder-Frameworks.tar.gz)
 sudo tar xfz Wonder-Frameworks.tar.gz -C $wodir/Library/Frameworks

But I don't think those have anything to do with it.
In fact, I have a request -- can you add a .travis.yml file, so we can verify that it builds by looking at the travis-ci.org log?
Thanks!

servlet.jar.dir error is expected and it happens during javadoc generation

So .war is built properly - javadoc is an issue that needs eventually be fixed but it is not critical for now

Oh, okay. Does it cause the script to exit 1? If so, can you fix things so it doesn't cause the script to exit with a failure status code?
Feel free to add a temporary hack like literally mkdir ' /vagrant/web-cat/Web-CAT/${servlet.jar.dir}'.


Definite group

Hello. There is a suggestion to add the generated contacts to a specific group.

I don't see any "Groups" facility in the standard Android Contacts app in my Android 9 phone.  Is it a new feature in later versions of Android, maybe ?

I see in the Contacts app I can select two or more contacts and say "link" them.  Is that what you mean ?

Hello. I am a Russian speaker and it is a little difficult for me to explain, I apologize.
I attach a screenshot that shows an additional Group field. I can create groups myself and add contacts to them. I use a non-standard Android Contacts app, but this is also present in the standard app.

Have I been able to clarify the situation?

My Contacts app, and I think I am using the standard one, does not have Groups.  I looked through the screens.

My Contacts app, and I think I am using the standard one, does not have Groups. I looked through the screens.

On Google Contacts this work, like "family", "school", "collage"


added to readme: how to retrieve messages from multiple email addresses

Clarified how to retrieve messages from multiple email addresses in README.

:+1:


fix plantuml preview links in readme

added ~1 to the link as the fallback images told me to.

@markushedvall this looks good to me


fixes #7 by giving a link to ruby EOL and clarifying what versions of ruby we support

this looks quite weird in the diff so you might want to just look at whole file.

@eheydrick updated.


use build trains for the tag

So previously we were using a fastlane action to grab the last build number, then increment that to do a tag. This doesn't work, cause out build number it 2016.01.30 or whatever, so it would always return 2017. Now we count the builds via spaceship.

works as expected now

:+1:

  | 1 Error
------------- | ------------
    :no_entry_sign: | Developer Specific file shouldn't be changed

  Generated by :no_entry_sign: danger

ObjC code looks good to me beside those logic issues.
So to be clear, the goal is that we (admins) will get the dev VC in App Store builds as well?

Yeah, this makes it easy to see useful details ( git sha, push notification ID, dev A/B options, etc )

The builds will always fail now, this is cause I have a merge commit - meaning danger takes the entire merge also into account.
I wonder if there's a way to say I want the merge between these two branches, excluding things that already exist in the firs branch?

I’m not 100% sure I understand the scenario (replicatable git commands for me to locally see what this is about and how Danger knows what code to check would be helpful), but if I do get it right, maybe you just want the real diff, e.g. git diff branch1 branch2 ?

so, here's the code: https://github.com/danger/danger/blob/master/lib/danger/scm_source/git_repo.rb#L9-L12 - which I think should be the same as what you're talking about - but I need to do some danger PR merging to make a lot of this right

Yeah, looks right, I’m not sure then what happens with a merge, does this happen with a rebase as well?

Hrm, good q. Unsure, will put it on my TODO sometime.

This worked.

:+1:


fix: DBTP-1282 allow for explicit settings of PUB_PATH_LIST

Addresses DBTP-1282
Previously setting a PUB_PATH_LIST explicitly to '/' would throw an error when the NGINX container spun up. See ticket for details.
Updated the entrypoint.sh for the image to allow for explicit setting of a PUB_PATH_LIST to '/'

Checklist:
Title:

[x] Scope included as per conventional commits
[x] Ticket reference included (unless it's a quick out of ticket thing)

Description:

[x] Link to ticket included (unless it's a quick out of ticket thing)
[ ] Includes tests (or an explanation for why it doesn't)
[ ] If the work includes user interface changes, before and after screenshots included in description
[ ] Includes any applicable changes to the documentation in this code base
[ ] Includes link(s) to any applicable changes to the documentation in the DBT Platform Documentation (can be to a pull request)

Tasks:

[ ] Trigger the pull request regression tests for this branch and confirm that they are passing

Need to run the regression pipeline still, this runs with the below PRIV_PATH_LIST:
PRIV_PATH_LIST: /ipfilter/
So won't validate that the bugfix was successful so that will need some inc manual testing.

Regression test run: https://eu-west-2.console.aws.amazon.com/codesuite/codebuild/763451185160/projects/platform-tools-test/build/platform-tools-test%3A1f540d4e-4ed4-42d9-b1de-efb0f16c8110/?region=eu-west-2.
This was with my Nginx image, with the default of PRIV_PATH_LIST set to '/'


Update for numpy int deprecation.

Replace all np.int instances with just int.

Mentioned by @thusser in the astroplan slack channel. Thanks!

Thanks for getting on this so quick @wtgee! I've just added a commit that fixes the style checks.


Issue with webpack-dev-server compression

As stated in the documentation SSE encounters problem when undergoing compression. The proposed flushAfterWrite: true is not effective for webpack-dev-server compression (which is used in create-react-app). To resolve this, one can set the 'Cache-Control' response header to 'no-cache, no-transform' (at least while not in production). I wasn't sure how to alter response headers using expresse, so I uninstalled.

Thanks a lot for signaling that. This will be taken in account in a future version. Also, you should be able to change the response headers in the middleware with res.header('Cache-Control', 'no-cache, no-transform'), there are no particular restrictions.


Add 46-:ri9 (Sylvalum Night) by Hiroyuki Sawano

Song Name
46-:ri9 (Sylvalum Night)
Artist Name
Hiroyuki Sawano
Source
Youtube
Youtube Link
cDbh3lLi1zy
Direct File Link
No response
Song ID
507637
Start Offset [ms]
0
Extra
Created from Auto Nong.
Created for level: 96228427

reject
invalid YouTube link


Regenerate /etc/issue to show IP address changes

https://github.com/rancher/os/issues/1176#issuecomment-250033346

how did you solve it?

I don't think we did yet


cargo-expand: 1.0.56 -> 1.0.59

Diff: https://github.com/dtolnay/cargo-expand/compare/1.0.56...1.0.59
Changelog: https://github.com/dtolnay/cargo-expand/releases/tag/1.0.59
Description of changes

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 241768 run on aarch64-darwin 1

1 package built:

cargo-expand


Wireshark Test with GitHub Actions

error message
qt.qpa.xcb: could not connect to display
qt.qpa.plugin: Could not load the Qt platform plugin "xcb" in "" even though it was found.
This application failed to start because no Qt platform plugin could be initialized. Reinstalling the application may fix this problem.

Available platform plugins are: eglfs, linuxfb, minimal, minimalegl, offscreen, vnc, xcb.

https://github.com/novogrammer/wireshark-lua-api-declarations/commit/ffb910990110a651c22b4c97cfa5491b300e36aa
https://github.com/novogrammer/wireshark-lua-api-declarations/actions/runs/2479751793


Error: Superclass has no getter named 'serverApi' with --preview-dart-2

I have a mixin
abstract class FirebaseDaoMixin implements DaoBase {
  @override
  ServerApiFirebase get serverApi => super.serverApi as ServerApiFirebase;
}

an abstract base class required to be implemented by the mixin
abstract class DaoBase {
  DaoBase(this.serverApi);

  final ServerApi serverApi;

and a class where I use it
class UserDaoFirebase extends UserDao with FirebaseDaoMixin {
  UserDaoFirebase(ServerApi serverApi) : super(serverApi);

  void foo() {
    serverApi.bar();
  }
}

compiler message: lib/src/server_api_firebase/firebase_dao_mixin.dart:11:44: Error: Superclass has no getter named 'serverApi'.
compiler message:   ServerApiFirebase get serverApi => super.serverApi as ServerApiFirebase;
compiler message:                                            ^^^^^^^^^

Works fine without --preview-dart-2
 (backoffice) $ flutter doctor -v
[✓] Flutter (Channel dev, v0.1.9, on Mac OS X 10.13.3 17D102, locale en-AT)
    • Flutter version 0.1.9 at /Users/zoechi/flutter/flutter
    • Framework revision 70f60b17d7 (2 days ago), 2018-03-05 08:55:10 -0800
    • Engine revision 97b22348c8
    • Dart version 2.0.0-dev.32.0.flutter-ee15c8eb68

This
abstract class FirebaseDaoMixin implements DaoBase {

should be this
abstract class FirebaseDaoMixin extends DaoBase

@mraleph I don't think this is valid. Perhaps with supermixins enabled but that doesn't work with dart2js
The class where the mixin is applied to extends DaoBase
class UserDaoFirebase extends UserDao ...

Yes, you can't use super-mixins anywhere outside of Flutter - and in Dart 2 super calls are only well-formed if class has appropriate super class.
So I think you can't write the code that would be both dart2js and Flutter compatible.

@mraleph I updated the comment. Actually I use super-mixins. I had to change the implementation for Angular to not use that mixin. Sorry, I got a bit confused.
But the code works fine without --preview-dart-2 and in Dart1 and seems legit to me.

Dart 1 does not report and error for "unbound" super calls. It is a static error in Dart 2.
/cc @lrhn Lasse, am I explaining it correctly?

@mraleph is almost correct, and tools are probably mostly correct too. I doubt anyone gets everything right.
Looking at:
abstract class FirebaseDaoMixin implements DaoBase {
  @override
  ServerApiFirebase get serverApi => super.serverApi as ServerApiFirebase;
}

This should cause a warning in Dart 1.
The super.serverApi invocation should cause cause a warning because the super-class interface does not contain a serverApi member. From the ECMA Dart spec, section 16.7.3:

Let Sstatic be the superclass of the immediately enclosing class. It is a static type warning if Sstatic does not have an accessible (6.2) instance member named m unless Sstatic or a superinterface of Sstatic is annotated with an annotation denoting a constant identical to the constant @proxy defined in dart:core.

Implementing DaoBase doens't help. It does not introduce any constraint on the actual superclass that the mixin will be applied to. It only affects the interface of the FirebaseDaoMixin class, not its static superclass, which is the one that is the actual constraint.
The static warning that MrAleph is probably thinking of is the case where the static super-class declares the super-invoked member, but it is abstract. In that case, there is no warning even though executing the code as written will surely fail.
Now, this is only a static warning, so you might not get it if you only run the VM (or if some other tool forgets to show it).
In Dart 2, super-mixins are removed. You can't write a super-invocation in a class used to derive a mixin. You can't do a super-invocation where the super-class doesn't have a concrete implementation that you are targeting.
If this is Flutter code, we have provided a way to keep using a variant of the Dart 1 super-mixin behavior in Dart 2 only until we provide a better alternative, and then it should be using the rules for that variant. That one would still require the static supertype to declare the member, and since we don't do warnings any more, this is a Dart 2 + Flutter-mixins compile-time error (as the compiler correctly tells you).

@mraleph @lrhn
Thanks for the feedback.
Should the issue be closed?

I guess this can be closed. All tools are working as expected, the only problem is that the Flutter compilation fails to provide static warnings in Dart 1 mode.
I can't tell whether that's because they just don't run the analyzer, or because the analyzer fails to report the warning.


Case study: 2D->3D challenges

UPDATE: my team has migrated to ThreeStudio where we just ran similar experiments, after much optimization. https://github.com/threestudio-project/threestudio/pull/166
In https://github.com/ashawkey/stable-dreamfusion/pull/293 I added new challenges for 2D->3D. I'll track here some of the early results and progress.
Initial conclusions:

the estimated depth and normals are often wrong and result in severe errors that are unrecoverable in later phases, even when using an outrageously high number of denoising iterations.

baby_pheonix_on_ice

Results (end of phases1, 2, 3)
Result after phase 1 (zero123):
https://github.com/ashawkey/stable-dreamfusion/assets/5090329/da8acc0b-4d8c-4c2b-a104-ff669480f80d
Phase 2:
https://github.com/ashawkey/stable-dreamfusion/assets/5090329/bd971c6f-5a71-4944-afcb-0ad5beff53d1
Phase 3:
https://github.com/ashawkey/stable-dreamfusion/assets/5090329/4a246392-9211-4bfc-99b6-8d5ae7937bc9
Conclusion: I suspect that the estimated depth and/or normals are wrong, and are keeping the reconstruction stuck in a local minimum.
beach_house_1

The reconstruction failed early in phase 2.
Phase 1 result:
https://github.com/ashawkey/stable-dreamfusion/assets/5090329/4edbd7f0-bf3f-4b2b-bbfd-a828e00d628e
Phase 2 partial result:
https://github.com/ashawkey/stable-dreamfusion/assets/5090329/94f0d2b6-5751-411c-ae29-8fa37ca31cfd
beach_house_2

The reconstruction failed early in phase 2.
Phase 1 result:
https://github.com/ashawkey/stable-dreamfusion/assets/5090329/58a2c6dd-1de8-44ce-af11-435fbff302c0
Phase 2 partial result:
https://github.com/ashawkey/stable-dreamfusion/assets/5090329/b995a289-62f6-4ed9-9b38-44dc68f68ccc
bollywood_actress

Phase 1 result: WARNING: looks gruesome!
https://github.com/ashawkey/stable-dreamfusion/assets/5090329/687ef521-d1c4-46ae-a72f-320cc424d5ef
Zero123 really struggles with this poor lady.. presumably because it's out of its training distribution...
Phase 2 result:
https://github.com/ashawkey/stable-dreamfusion/assets/5090329/3cc6dc0a-cd0b-4bbb-8265-af22cf7c311d
Phase 3 failed due to insufficient VRAM.
church_ruins

Phase 1:
https://github.com/ashawkey/stable-dreamfusion/assets/5090329/2b30b9dc-1493-47b8-8ead-d8043128c7f0
The results of this phase look plausible, but have lots of holes.

Phase 2:
https://github.com/ashawkey/stable-dreamfusion/assets/5090329/cdcb8bcf-e829-4951-a2a6-77addc4f5116
the depth of the car has shortened, the side and back are now completely unrecognizable.
mona_lisa

This is an extra-hard challenge, since the Monal Lisa is well-known to be flat.
Phase 1:
https://github.com/ashawkey/stable-dreamfusion/assets/5090329/0a9b1144-333d-4311-86d7-a2454cb760ec
A mirror image is in the back.
Phase 2:
https://github.com/ashawkey/stable-dreamfusion/assets/5090329/50b011af-6d3c-4288-9257-b35f95ddef5b
The nose has emerged but the back image remains.
Phase 3 wasn't run since Phase 2 was already problematic.

UPDATE: my team has migrated to ThreeStudio where we just ran similar experiments, after much optimization. https://github.com/threestudio-project/threestudio/pull/166. I will close this issue.

Hi, how to achieve Phase2 and Phase3, could you share the corresponding code？


VSCode-Web is blocked in mainland China

VSCode-Web (vscode.dev) is blocked in mainland China and cannot be accessed. But turning on VPN can open it.

Visual Studio Code Information:
Version: 1.67.1
Commit: da15b6fd3ef856477bf6f4fb29ba1b7af717770d
Date: 2022-05-06T12:37:52.619Z
Browser: Mozilla/5.0 (Windows NT 6.1; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/101.0.4951.64 Safari/537.36 Edg/101.0.1210.47

Is that means we can`t login in VS Code with Microsoft account in mainland China?being troubled by this problem.

vscode.dev is still in preview. Once GA, we'll have to address the China firewall. Until then, there's not much we'll do.

I suggest making a VSCode-Web mirror on Github Pages because Github Pages is not blocked now.

Such as vscode-web-cn.github.io ("vscode-web-cn" is my other Github account). But it can't work.


LVictoria

closes #294

Almost there @LVictoria159753  😄
2 more steps:

can you remove the angle bracket (<>) so that the issue gets linked? eg. closes #294
re-request a review from us so we know when to take another look (top right)
Then we 👍 and 🚢 this thing!

Hello Guyandtheworld as well as Bedrock-Adam,
Thank you so much for your comments! I just edited the angle bracket for the phrase, closes #294. I hope I did that right, but please let me know if there is anything else that I need to fix!