Support opening files with an external application

In Visual Studio, you could open a file directly within another application whereas Visual Studio Code only lets you open with editors within the application. Sometimes you may want to open a file with another application and there's no way to do it with Visual Studio Code without opening the folder in File Explorer and using the "open with" option there.
Visual Studio displays external applications as well designers/editors within the IDE.

Visual Studio Code doesn't.

This extension does it: https://marketplace.visualstudio.com/items?itemName=sandipchitale.vscode-open-with
The key is to launch child process with:
rundll32.exe shell32,OpenAs_RunDLL <path to file>

Oh if an extension already does it that's great and I would recommend installing that extension 👍

I tried to create a PR but ran into layering issues in VS Code. Any guidance where I can implement the Menu registration, command registration and the above snippet I can submit the PR.

@iramos15 Can I get some guidance on where in the vscode this should be implemented. I have implemented -
export function showOpenWithDialog(fsPath: string): Promise<string> {
	return new Promise((resolve, reject) => {
		if (fsPath) {
			let stdout = '';
			const child = cp.spawn('rundll32.exe',
				[
					'shell32,OpenAs_RunDLL',
					fsPath
				],
				{
					cwd: fs.dirname(fsPath)
				});
			if (child.pid) {
				child.stdout.on('data', (data: Buffer) => {
					stdout += data.toString();
				});
			}
			child.on('error', err => {
				reject(err);
			});
			child.on('close', code => {
				resolve(stdout);
			});
		}
	});
}

MenuRegistry.appendMenuItem(MenuId.ExplorerContext, {
	group: 'navigation',
	order: 20,
	command: {
		id: OPEN_WITH_DIALOG_COMMAND_ID,
		title: nls.localize('explorerOpenWithDialog', "Open With Dialog..."),
	},
	when: ContextKeyExpr.and(ExplorerFolderContext.toNegated(), ResourceContextKey.HasResource),
});

CommandsRegistry.registerCommand({
	id: OPEN_WITH_DIALOG_COMMAND_ID,
	handler: async (accessor, resource: URI | object) => {
		const uri = getResourceForCommand(resource, accessor.get(IListService), accessor.get(IEditorService));
		if (uri) {
			showOpenWithDialog(uri.fsPath);
		}
	}
});

I ran into issues related to layering of vscode.

The layering is there to prevent code from being executed in the wrong place. I think this is best to leave to an extension as it is specific to windows and VS Code is cross platform + on web

Thanks!


support multiple pointer devices

The current Input<MouseButton> interface is very nice but implies the existence of a single mouse input device.
In future maybe multiple mice or touch inputs want to be supported (touch having a "backwards compatible" interface with MouseButton maybe?), for that a mechanism to select/retrieve the input device ID would be needed.

Now that bevy has an API for touch input (and since mice and touch are different things anyway), can we adjust this issue (change the title, etc) to be about supporting multiple mice / pointer devices, specifically?

This feels like a surprisingly straight-forward problem to be the oldest open issue on Bevy.

I'm looking into working on this as my first contribution, and I'd like some affirmation that the way I have a sensible plan:

MultiMouseButtonInput: event struct containing a MouseButtonInput and a winit::event::DeviceId
MultiMouseMotion: event struct containing a MouseMotion and a winit::event::DeviceId
MultiMouseWheel: event struct containing a MouseWheel and a winit::event::DeviceId
MultiMouseButton: struct containing a MouseButton and a winit::event::DeviceId
multi_mouse_button_input_system: the equivalent of mouse_button_input_system, but for ButtonInput<MultiMouseButton> (I'd separate these into different systems because they do not require access to any of the same resources or event readers and so they could run in parallel)
Update <InputPlugin as Plugin>::build to include .init_event::<...> for the above events
Update <InputPlugin as Plugin>::build to include .init_resource::<ButtonInput<MultiMouseButton>>
Update <InputPlugin as Plugin>::build to include .add_systems(PreUpdate, multi_mouse_button_input_system.in_set(InputSystem))
Update <InputPlugin as Plugin>::build to include .register_type::<MultiMouseButtonInput>
Add variants to bevy_window::event::WindowEvent for the above events
Add bevy_winit::converters::convert_multi_mouse_button(mouse_button: winit::event::MouseButton) -> MultiMouseButton
Update bevy_winit::state::window_event to emit MultiMouseButtonInput and MultiMouseWheel in addition to the existing mouse events
Update bevy_winit::state::device_event to emit MultiMouseMotion as well as MouseMotion

I think I'm not going to try and update bevy's pointer API's to support multiple pointers associated with different pointer devices, since (I think) those are supposed to imitate the underlying OS's pointer behavior, and most (all?) OS's don't this.

Hey, just so you're aware, winit (our windowing lib) is moving to a pointer abstraction in its next release. So I think we are probably just going to switch to a unified pointer api (and possibly scrap a big portion of the existing bevy_input) instead of making additions to what we already have:

I was not aware, thanks for saving me the trouble, I'll look for another issue that seems doable.


Improve error handling on backend.

Need to return a generic message rather than a big stack trace for most errors.

Hello @R-Sandor Can I work on this issue ?

@pisal-shubham Thanks! For a little more context on the ticket. This really about when the frontend makes a request, sometimes you get these long stack traces rather than a nice short error.


Is Jasypt is having CVE-2022-22965 Spring4Shell Vulnerability as it is using Spring boot 2.5.4 ?

jasypt is having Spring4Shell Vulnerability which is needed to fix.

new release 3.0.5


add function r:any() to match all methods

redo of #17

looks like it doesn't like my test; i'll get that fixed rq

@skwerlman thank you very much! This is amazing. Cheers!

Hi,
POST section seems not working for me, after i put openresty nginx.conf, GET options with
curl http://localhost/hello works for me and give me output as "someone said hello"
curl http://localhost/hello/nur gives output as hello nur, but how should i test the POST part?
I tried several way but please show me how can i test it exactly, i tried below:
curl -X POST --data '{ "comment" = "fascinating"}' http://localhost/app/2/comment
nothing shows
//Regards
Nur


Baselines for: BUG #1265: Fix datawc zoom-in for geographic projections.

@danlipsa I'm about to submit a PR for the new default colormap, it updates EVERY baselines pretty much. Do you mind if we merge this one after?

@doutriaux1 No, I don't. Go ahead. I can regenerate my baselines.

The following look weird:
https://github.com/UV-CDAT/uvcdat-testdata/blob/689fd0a5ca31ed737a33b86821dfd16d09788f57/baselines/vcs/test_vcs_basic_isofill_masked_aeqd_proj.png

Yes, I have an issue about it:
https://github.com/UV-CDAT/uvcdat/issues/2099
For certain aeqd tests, we still get two layers of cells in the front or back. We might be able to fix this with a new proj4.


Add runtime tags

Feature Request
I like the tags feature by Plasmo, and this framework could focus on the general use case
Is your feature request related to a bug?
N/A
What are the alternatives?
set VITE_TAG=test && wxt

# .env.test
VITE_IMPORTANT_ENV="foobar"

I'm not sure as for the output directory
Additional context
wxt --tag test

this would affect:

process.env.VITE_TAG === "test"

Values in.env.test would be populated in import.meta.env
The build output will be chrome-test-mv3/

This already exists in Vite, it's code "modes", and is already supported by WXT
pnpm wxt build --mode=test

It will load the .env.test file as well. Read more about it in Vite's docs: https://vite.dev/guide/env-and-mode.html

There was discussion around adding browser to .env file resolution on discord as well. So you could do something like .env.firefox.production: https://discord.com/channels/1212416027611365476/1224505556346212515/1296857455661420606

Sounds good
So far I attempted to get around it with
import dotenv from "dotenv";

export default defineConfig({
  hooks: {
    ready(wxt) {
      const pathEnvs: Record<string, string> = {
        production: ".env.production",
        development: ".env.local",
        test: ".env.test"
      };
      dotenv.config({ path: pathEnvs[wxt.config.mode] });
    }
  }
});


Remove slider_min/max for Colormaps and add exposure multiplier

EntityEditor takes in "min" and "max" parameters to set the slider range for colormaps as seen in the code below but it doesn't pass the values to the ColorMapInputWidget class nor was there a function to set the slider range in ColorMapInputWidget. So, I've added the function to set the slider range as well as passing the min/max values to that function in EntityEditor.
I think this is the bug referenced in #1727
https://github.com/appleseedhq/appleseed/blob/f051daff605b4ca945659e4942eca8a24e61e908/src/appleseed.studio/mainwindow/project/entityeditor.cpp#L490-L519

So just to be clear about the tasks:

Exposure slider for color maps stays between 0 and 1 -> Revert back
Remove min and max options in exposure and create a Exposure Multiplier scalar input there with a min and max (-64,64 for exposures)
Remove slider_min and slider_max variables and the respective validators

Last question, now that a scalar multiplier has been added to exposure for colormaps in spotlights and lat long edfs, I assume should that multiplier be multiplied with the exposure to get the right value of exposure? So, changes will be required in spotlight and lat long exposure evaluation?

All absolutely correct!

I have made all the changes requested and tested the multiplier setting with the test scenes. I hope this is suitable. Thanks.

In addition to the comments I've made, the same changes must be propagated to other light types.

I've made all the changes according to the comments. However, the other light types and EDFs that have exposure inputs have scalar inputs and therefore don't need a multiplier as their range is already set to (-64,64) as referenced to by this:

Also note that in the case of non-spot light types, exposure cannot be textured so a single Exposure scalar input with the right range is sufficient.

Unless I'm missing something, that should be it hopefully!

I've made all the changes according to the comments

Thanks!

However, the other light types and EDFs that have exposure inputs have scalar inputs and therefore don't need a multiplier as their range is already set to (-64,64)

Right, but there is at least MirrorBallMapEnvironmentEDF to update.
I wonder if it would make sense to add exposure controls to the physically-based environment EDFs (Preetham, Hosek-Wilkie) and to the Sun light.
@est77 @luisbarrancos @juancarlosgzrz Opinions?

Ah sorry I overlooked the MirrorballMapEnvrironmentEDF. I've added exposure and its multiplier to it.

Thank you! Looking good. I'll just alter the commit's title slightly.


Mention how to handle cancel in ConfirmationDialog

Resolves #17460
Although not sure what about the modal_closed mentioned in the issue. The get_cancel() is the proper way to use cancel, but you can do also do
connect("modal_closed", self, "cancelled") #This is emitted when user clicks outside dialog.
get_close_button().connect("pressed", self, "cancelled") #This is emitted when user clicks X.

which are more like "neutral actions" and aren't immanent to ConfirmationDialog.

Thanks!


Broilerplate example please?

I'm trying to figure out how to use tcom-form, without knowing React.  Could you please provide a simple & clean broilerplate example, or a more detailed listing of how to use?  You list dependencies, but I do not know if those are for the browser or to be compiled developer-side.

No examples now?

@sm-g replaced with documentation: https://github.com/gcanti/tcomb-form/blob/master/GUIDE.md

Well, i did found only very simple Working example. Would be nice to see some real app, may be you know good repo for this role?


Add Spark 3.0 EMR Shim layer

closes https://github.com/NVIDIA/spark-rapids/issues/818
This adds a shim layer to support Amazon EMR.
The main issues is they have a different version number and then their FileScanRDD takes a different number of parameters.
Other then that add some tests support and fix a few things that assumed DataSource v1 was enabled.  There is one join test that has a follow up jira because it appears they have a new exec for bloom filters.
I ran all the integration tests successfully on an EMR cluster by specifying --runtime_env emr with the spark-submit command.  It does not need to be built on an EMR cluster because we are using reflection to determine the FileScanRDD parameter count.
I did not run the unit tests against it as there is another jira to setup build and test on EMR and we can do it with that.

build

build

build


[INFRA] Study the approach for testing of the rendering layer

Pending questions

test lib: cypress, .....
how do we identify element in the graph?

Current status

#116 introduce e2e tests
disable on gh workflow on windows because jest didn't stop correctly
we only check that the page is loaded, checking the element on the graph didn't work


Delay ID lookups on IVR apps

We need to do it only after the user has identified themselves as more than a UUID

@imsickofmaps Please review.

:+1:


Breast Cancer Classification with Neural Network | Deep Learning

Deep Learning Simplified Repository (Proposing new issue)
:red_circle: Project Title : Breast Cancer Classification with Neural Network | Deep Learning
:red_circle: Aim : The Breast Cancer Classification project aims to leverage the power of deep learning to develop a robust neural network model for accurately classifying breast cancer. Breast cancer is a significant health issue affecting millions of women worldwide, and early detection plays a crucial role in effective treatment and patient outcomes.
:red_circle: Dataset : https://drive.google.com/file/d/1HQqUCF23fihnE3aCbx2MNPmER9o_If_m/view
:red_circle: Approach : A neural network architecture, such as a Convolutional Neural Network (CNN), is then designed and implemented to capture the intricate patterns and features in the data. The model undergoes rigorous training using backpropagation and optimization algorithms to minimize classification errors. Hyperparameters, including learning rates, batch sizes, and network depths, are fine-tuned to achieve optimal performance.

📍 Follow the Guidelines to Contribute in the Project :

You need to create a separate folder named as the Project Title.
Inside that folder, there will be four main components.

Images - To store the required images.
Dataset - To store the dataset or, information/source about the dataset.
Model - To store the machine learning model you've created using the dataset.
requirements.txt - This file will contain the required packages/libraries to run the project in other machines.

Inside the Model folder, the README.md file must be filled up properly, with proper visualizations and conclusions.

:red_circle::yellow_circle: Points to Note :

The issues will be assigned on a first come first serve basis, 1 Issue == 1 PR.
"Issue Title" and "PR Title should be the same. Include issue number along with it.
Follow Contributing Guidelines & Code of Conduct before start Contributing.

:white_check_mark: To be Mentioned while taking the issue :

Full name : Ashish Kuma Patel
GitHub Profile Link : https://github.com/ashis2004
Email ID<EMAIL_ADDRESS>Participant ID (if applicable):
Approach for this Project :
What is your participant role? GSSOC contributor

Happy Contributing 🚀
All the best. Enjoy your open source journey ahead. 😎

Already present in this repository. Closing this issue as duplicate.


VisualRust.Build.Rustc.ExecuteInner(), error : The system cannot find the file specified

Trying to build any rust project using VS2015 I get the following error:
C:\Program Files (x86)\MSBuild\VisualRust\VisualRust.Rust.targets(53,5): error : The system cannot find the file specified
at System.Diagnostics.Process.StartWithCreateProcess(ProcessStartInfo startInfo)
at System.Diagnostics.Process.Start()
at VisualRust.Build.Rustc.ExecuteInner()
rustc (1.3), cargo, racer are in my path, I tried both the latest build artifact and the stable version.

Looks like a duplicate of #182

@melak47, actually the plugin looks for the intalled rust using the registry, not the PATH variable (at least, that is my understanding), e.g. on my machine I have 2 versions of Rust installed (a stable build and a beta), and the plugin actually uses not the one I have in the PATH :-)
Can you check your registry settings to help debug this? Run "regedt32" , then press "Ctrl+F" and search for "Mozilla Foundation", then under that folder find the first value "InstallDir" and see if that path still exists...
Hope this helps.

I'd prefer if VisualRust first checked PATH for rustc/cargo, and only use the registry as a fallback.

@retep998, I usually don't add them to PATH, when I'm building from the command line I just explicitly call cargo from stable or beta or nightly to know which one I'm building with... maybe it would be better if VisualRust let you select from all the installations it discovers? (eventually) (though I don't know what was the original rationale for not using PATH)

@yacoder I did indeed have an old 1.3 nightly in the registry which no longer existed on disk, removing it resolves the error!

I think using registry entries is a less surprising way to discover install paths. If we are getting a support for non-registry install paths it should be part of #172.
Closed by #189.


Migrate to null safety

Description
All dependencies are migrated to null safety. So we should migrate this package to null safety.
Resources

How to migrate Dart packages to null safety
Announcing Dart null safety beta

I will do when null safety will out of beta :hammer:

@Golapadeog Null safety is out of beta :) Since Flutter 2 it's stable 👍

@Golapadeog I created a PR: #5. Please check everything

Waiting for this to update our application to Flutter 2.0!
It would be awesome to have it ASAP. Thank you for your great work :)

@Golapadeog I created a PR: #5. Please check everything

Thanks for your PR, I check it as soon as possible :ok_hand:

Waiting for this to update our application to Flutter 2.0!
It would be awesome to have it ASAP. Thank you for your great work :)

An update will arrive at the end of the week :rocket:

#5 is merged :)

@Zeswen @nilsreichardt
:tada: The new version of string_similary with null safety compatibility is avialable https://pub.dev/packages/string_similarity
:bulb: Please note that null can be used by methods to compare string and doesn't throw exception

null / null = 1.0
null / "a" = 0.0
"a" / null = 0.0

:pencil: the documentation has been updated accordingly


Trigger events before and after a simulation run

Please describe what you would like the feature to accomplish.
When implementing a custom simulator I'd like to add code that is invoked before and after the simulator is run on some operation.
Describe the solution you'd like
public class MySimulator : SimulatorBase {
  public MySimulator() : base(...) {
    OnSimulationStart += SomeCustomStartAction;
    OnSimulationEnd += SomeCustomEndAction;
  }
}

Describe alternatives you've considered
So far the same can be achieved by overriding the Run method, but it's not very concise.
public override Task<O> Run<T, I, O>(I args)
{
    SomeCustomStartAction();
    var result = base.Run<T, I, O>(args).Result;
    SomeCustomEndAction();
    return Task.Run(() => result);
}

Additional context
N/A

Hi @msoeken
I am curious about the use case for this. Do you mind sharing?

Hi @bettinaheim, some concrete examples are:

Generating output files after a simulation run (see #187)
Resetting internal state before a simulation run (see #189)


Efficient pickling

In addition to mongo storage, especially for batch jobs, one should store snapshots with a fast and efficient format, probably the binary format of scala pickle.

Not clear this is needed - file storage written but not deployed. Should revisit of akka-reactivemongo does not work well.


Test64D failure on checking hiprtcLinkCreate API in oroInitialize

Hello.
I tried to run Test64D app on Linux x64 system, but I got the error:
$ gdb dist/bin/Debug/Test64D
...
(gdb) run
...
Test64D: ../contrib/hipew/src/hipew.cpp:470: int hipewHipInit(): Assertion `hiprtcLinkCreate' failed.
Program received signal SIGABRT, Aborted.

(gdb) backtrace
#0  0x00007ffff7af234c in __pthread_kill_implementation () from /usr/lib/libc.so.6
#1  0x00007ffff7aa54b8 in raise () from /usr/lib/libc.so.6
#2  0x00007ffff7a8f534 in abort () from /usr/lib/libc.so.6
#3  0x00007ffff7a8f45c in __assert_fail_base.cold () from /usr/lib/libc.so.6
#4  0x00007ffff7a9e116 in __assert_fail () from /usr/lib/libc.so.6
#5  0x0000555555562844 in hipewHipInit () at ../contrib/hipew/src/hipew.cpp:470
#6  0x00005555555629db in hipewInit (flags=1) at ../contrib/hipew/src/hipew.cpp:487
#7  0x000055555555655a in oroInitialize (api=ORO_API_HIP, flags=0) at ../Orochi/Orochi.cpp:105
#8  0x000055555555b138 in main (argc=1, argv=0x7fffffffe3e8) at ../Test/main.cpp:31

And hipew.cpp:470 is
https://github.com/GPUOpen-LibrariesAndSDKs/Orochi/blob/ac7f0ab7f537d6724fd6e9228f0539391e730230/contrib/hipew/src/hipew.cpp#L470
which is checking the existence of API named hiprtcLinkCreate.
So I checked the name in my .so file:
readelf /opt/rocm/hip/lib/libamdhip64.so -s --wide | grep hiprtc
   413:<PHONE_NUMBER>2c7f40  1084 FUNC    GLOBAL DEFAULT   10 hiprtcAddNameExpression@@hip_4.2
   437:<PHONE_NUMBER>2c9250  4267 FUNC    GLOBAL DEFAULT   10 hiprtcCreateProgram@@hip_4.2
   520:<PHONE_NUMBER>2c89e0  1511 FUNC    GLOBAL DEFAULT   10 hiprtcCompileProgram@@hip_4.2
   547:<PHONE_NUMBER>2c86a0   832 FUNC    GLOBAL DEFAULT   10 hiprtcGetCode@@hip_4.2
   590:<PHONE_NUMBER>2c8380   792 FUNC    GLOBAL DEFAULT   10 hiprtcGetProgramLog@@hip_4.2
   599:<PHONE_NUMBER>2c6ae0   331 FUNC    GLOBAL DEFAULT   10 hiprtcGetErrorString@@hip_4.2
   601:<PHONE_NUMBER>2c76d0   610 FUNC    GLOBAL DEFAULT   10 hiprtcVersion@@hip_4.2
   603:<PHONE_NUMBER>2c8fd0   638 FUNC    GLOBAL DEFAULT   10 hiprtcGetLoweredName@@hip_4.2
   612:<PHONE_NUMBER>2c7940   758 FUNC    GLOBAL DEFAULT   10 hiprtcGetCodeSize@@hip_4.2
   616:<PHONE_NUMBER>2c7c40   758 FUNC    GLOBAL DEFAULT   10 hiprtcGetProgramLogSize@@hip_4.2
   690:<PHONE_NUMBER>2c6c30  2716 FUNC    GLOBAL DEFAULT   10 hiprtcDestroyProgram@@hip_4.2

no hiprtcLinkCreate found. Any ideas?
thanks in advance.

hip-runtime-amd 5.2.3-1
https://aur.archlinux.org/cgit/aur.git/tree/PKGBUILD?h=hip-runtime-amd

Linux 5.17.4

Orochi ac7f0ab7f537d6724fd6e9228f0539391e730230

Thanks for bringing this up. Sadly we don't have the hiprtcLink* APIs available on Linux yet, but it should come eventually. I'll get back to you here in case we have a way to bring those to Orochi.

Thanks. I send PR to modify README to inform Linux users about it.

I think a better way could be to use an older commit of orochi. Can you try this commit? https://github.com/GPUOpen-LibrariesAndSDKs/Orochi/tree/d78fb813e6ca19309313119052462ef8882c9908

Yes using an older version of Orochi is the right solution as linux hip is just behind.


Just a question : add event markers within a timeline

Hi
I need to add event markers on specific dates within my timelines.
Each timeline is a 'project' on its own row, and each project will have maybe 10 key events that I'll indicate with a diamond - on rollover a tooltip will show the named event.
Do you think this should be fairly easy for me to figure out and build ?
I'm not a JS expert but not a beginner either.
I can always send you my hacky code when it's done!
I looked at vis.js but it seems bad at overlaying multiple timelines.
ALSo - on your demos, mousewheel only moves the chart to the left - reverse mousewheel does not move it back ?
Thanks!

I'd start by adding a markers array to the data property of the timeline and then work from there. It shouldn't be that complicated, but I guess you'll need some time to grasp how the plugin works.
Just fork the project and give it a try. As you said, PRs with new features are always welcome.
The bug with the mouse wheel has been moved to #5, thanks for pointing that out.


Minor typo and question

the very minorest of issues: typo in the README for the API Method, under How to use: requierd -> required.
If I were to use this EM to send an API call that required basic authentication, I'd guess that I'd include the  'Authorization: Basic ...' header in the API Header configuration. If you happen to have done that and know the exact syntax to use, I'd love to see it.

I'll make sure the next bugfix or feature update fixes the typo, thanks for pointing it out!
I swear I thought I had used it with a basic authentication but I can't find an example. The one that I was thinking was actually "Api-Key: <>" I tried looking for a sample API that uses basic auth and couldn't find one. If I had one, I'd test and get the right syntax for ya. And if I find one, I'll update you!


Just udpating the typenames brushstart and brushend

The names for 4.0 are now start and end, so the documentation is correct. Please see the release notes for more.

Ahhh...my fault.  Still in 3 and didn't pay attention to where I was.  Thanks!


Command tracking should show glyphs next to rows and flash lines when navigating

Follow up for https://github.com/Microsoft/vscode/issues/45435
cmd+up/down on macOS should show some indication of the line that was scrolled to.
This will need some upstream changes.

if i have a few empty lines such as this:

pressing cmd+up appears to do nothing

@chrisdias yeah I would have liked to get this in but was a bit randomized this iteration.

Blocked on https://github.com/xtermjs/xterm.js/issues/1852

Brought this to UX sync with a focus border outline that fades to 0 opacity over 2 seconds. But after it occurred to me that navigating the a symbol in the editor is the exact idea we're trying to get across which flashes a focus border briefly without animating. So going with that to align, we can change both if we want to tweak it:

Using 350ms:
https://github.com/microsoft/vscode/blob/12e6b0b3043f19d0cf1730eb3c213d3930310053/src/vs/editor/contrib/gotoSymbol/browser/goToCommands.ts#L214-L222


An issue with piping to request

I'm trying to update tar to a recent version however in my particular situation it's not working as expected.
I'm packing some directory and then sending it to couchdb using cradle but for some reason I have a Write after end error.
Here is the code that I'm using:
const pathlib = require('path');
const cradle = require('cradle');

const dirToPack = pathlib.join(__dirname, 'some-files');
const CONN_DEFAULTS = {
    cache: false,
    raw: false,
    port: 5984,
    hostname: 'localhost'
};
const connection = new cradle.Connection(CONN_DEFAULTS);
const DB_PACK = 'dbnode-testrepo';
const db = connection.database(DB_PACK);
const tar = require('tar');
const readable = tar.create({
    prefix: dirToPack
}, [dirToPack]);
const writtable = db.saveAttachment({_id: 'test'}, {
    name: 'archive.tar',
    'Content-Type': 'application/x-tar',
}, (err, reply) => {
    if (err) {
        console.error('ERR', err);
    }
    console.log('REPLY', reply);
});
readable.pipe(writtable);

to test it you must install cradle and tar and have a couchdb on your localhost.
The output I see is this:
ERR Error: write after end
    at write_ (_http_outgoing.js:625:15)
    at ClientRequest.write (_http_outgoing.js:620:10)
    at Request.write (/home/sel/Documents/spotme/test-tar/node_modules/request/request.js:1501:27)
    at Yallist.pipes.forEach.p (/home/sel/Documents/spotme/test-tar/node_modules/minipass/index.js:263:40)
    at Yallist.forEach (/home/sel/Documents/spotme/test-tar/node_modules/yallist/yallist.js:154:8)
    at Object.emit (/home/sel/Documents/spotme/test-tar/node_modules/minipass/index.js:263:20)
    at Object.write (/home/sel/Documents/spotme/test-tar/node_modules/minipass/index.js:99:17)
    at Object.source.on.chunk (/home/sel/Documents/spotme/test-tar/node_modules/tar/lib/pack.js:344:20)
    at emitOne (events.js:116:13)
    at Object.emit (events.js:211:7)
REPLY undefined

Previously we ware using<EMAIL_ADDRESS>with tar.Pack and it was working just fine. After the update we have this error and saved file is empty. Also if I redirect the stream to a file on a file system it works fine.
Any advice would be appreciated.

I believe that this was fixed a while back.  Can you try on the latest version?  Istr the issue was emitting close before end, causing some stream implementations to assume the stream was finished.  (Logically, this still irks me, since close is about the underlying resource, not about the stream of data, but we live in the world, not in our ideals.)

I've just tested it again with the recent version of tar and the problem is still there.
What I did is simply:

copied some files to ./some-files directory,
then copied the provided script to a.js
created a database in couchdb: dbnode-testrepo
run:

npm install cradle
npm install tar

cradle was installed at version 0.7.1 and tar at version 4.4.10
run: node a.js and the output was:

ERR Error: write after end
    at write_ (_http_outgoing.js:622:15)
    at ClientRequest.write (_http_outgoing.js:617:10)
    at Request.write (/home/sel/Documents/spotme/tmp/node_modules/request/request.js:1500:27)
    at Yallist.pipes.forEach.p (/home/sel/Documents/spotme/tmp/node_modules/minipass/index.js:270:40)
    at Yallist.forEach (/home/sel/Documents/spotme/tmp/node_modules/yallist/yallist.js:154:8)
    at Object.emit (/home/sel/Documents/spotme/tmp/node_modules/minipass/index.js:270:20)
    at Object.write (/home/sel/Documents/spotme/tmp/node_modules/minipass/index.js:102:17)
    at Object.source.on.chunk (/home/sel/Documents/spotme/tmp/node_modules/tar/lib/pack.js:345:20)
    at emitOne (events.js:116:13)
    at Object.emit (events.js:211:7)
REPLY undefined

So the issue does not seem to be fixed yet.
As a workaround I just store the file on fs and then pipe it from there to couch.


TODOs (v0.1.3)

[x]  English support
[ ] tocbibind

English support: df44d98a9470858a69dbc05a7b9240f56f402c8d

tocbibind is a bad idea.


[Master] rollback all dedicated fcp when got exception in dedicating fcp

With attach, there would be multiple fcp to be dedicated. Previously if
one fcp dedicate failed, only the current fcp will be rollback.
This PR will also rollback the previously dedicated fcps.
Signed-off-by: dyyang<EMAIL_ADDRESS>

@bjhuangr please help to take a look

lgtm


TypeScript compilation error related to @types/express

With a brand new module, that only has @google-cloud/logging-bunyan and bunyan installed, I get this error message:
> tsc -p .

node_modules/@google-cloud/logging/build/src/middleware/express/make-middleware.d.ts:16:49 - error TS7016: Could not find a declaration file for module 'express'. '/Users/beckwith/Code/node_modules/express/index.js' implicitly has an 'any' type.
  Try `npm install @types/express` if it exists or add a new declaration (.d.ts) file containing `declare module 'express';`

16 import { Request, Response, NextFunction } from 'express';
                                                   ~~~~~~~~~

Installing @types/express made the bad message go away.  We should probably be distributing those types along with the package.

Fixed on googleapis/nodejs-logging#376.


add dependabot to kubectl-validate

What type of PR is this?
/kind feature

What this PR does / why we need it:
Add dependabot to kubectl-validate.
Dependabot is used to keep your dependencies up-to-date. We have been using it in Kueue and JobSet for at least a year now.
Its provides a weekly PR update of your dependencies. K8s dependencies are tough due to the api changes so those are usually done manually. But it can be useful for other sig dependencies.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?

NONE

Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.:

Hmm. Looks like this may not work with go 1.20..

Hmm. Looks like this may not work with go 1.20..

Turns out there was a CI issue. Pushed a fix to upgrade to go 1.21 and updated go.mod.

Rebased and this PR is now ready for review.

This is a great addition to help track k8s patches, thank you!
/lgtm
/approve


is there any way to create flutter project targeted only to android and IOS even if web support is enabled??

I just wanted to know is it possible ?
OR
I think this is good add-on for flutter and sometimes i need this

sorry i didn't see flutter config options
i found how to do it
You can disable web support in flutter config. use flutter config --no-enable-web to disable and flutter config --enable-web to enable. There are several other options as well which you can explore by running flutter config


fuels build throws Dynamic require of "@fuel-ts/script" is not supported error

Run the following command:
git clone https://github.com/Props-Labs/fuel-src721-localnode.git && cd fuel-src721-localnode && pnpm install && mkdir sway-programs && forc init --path sway-programs && pnpm fuels init -o ./src/types -w ./sway-programs && pnpm fuels build

It should throw the following error:
Dynamic require of "@fuel-ts/script" is not supported

@petertonysmith94 found that after removing this line that the problem goes away, so this is a non-issue.


[AST] BridgeObject uses ReferenceCounting::Bridge even without ObjC interop

@swift-ci please smoke test Linux

... which means we aren't testing this at all. yay

@swift-ci please smoke test


Compatibility with Oasis Core 21.2

use oasis_core 21.2 version
oasis-core-rosetta-gateway 1.1.1
rpc:block
block:3610570
post body
{
"network_identifier": {
"blockchain": "Oasis",
"network": "53852332637bacb61b91b6411ab4095168ba02a50be4c3f82448438826f23898"
},
"block_identifier": {
"index": 3610570
}
}
Returned result
{
"code": 14,
"message": "unable to get transactions",
"retriable": true
}

The current version of the Rosetta gateway (v1.1.1) has only been tested with Oasis Core 21.1.x. Can you reproduce the issue with that version?

There is no problem with oasis core 21.1. X
oasis core 21.2 version have problems
You try block:3610570

There are also many block problems, 3610570 is just one of them

This is expected due to some breaking event changes in Oasis Core 21.2. As mentioned, Rosetta gateway v1.1.1 targets Oasis Core 21.1 and if you need to use Rosetta, you should currently use Oasis Core 21.1 until a new version of the Rosetta gateway is released. We'll use this issue to track the necessary changes.

Actually what you describe may be caused by the issue fixed in https://github.com/oasisprotocol/oasis-core/pull/4033. The fix is planned to be released as part of Oasis Core 21.2.2, but will require a node re-sync from scratch since the problematic events have been incorrectly stored.

I used oasis core 21.2.7 and resynchronized the nodes
But there will be problems.
Can I only use 21.1.X?

@lougon, the latest master of Rosetta Gateway should be compatible with Oasis Core 21.2.x and we'll release a new version of Rosetta Gateway (1.2.0) that will be compatible with it soon.


notifications

Hey there
By creating comments that include @username you're inadvertently sending out a lot of github notifications to users who share the same name on github & hn
Could you come up with a different system to backup your HN conversations, pls?
Thanks!

Even worse would be sending out GitHub notifications to users who are NOT the same on HN. That didn't happen to me, but it's plausible.

True


Bug: NPM install returns "unable to resolve dependency tree"

Hi. I'm trying to add Ohif-viewer to my existing React project. When I try to install it via NPM I get the following error:

React version is 18.2.0.
NPM version 9.2.0
Is anybody experiencing this or aware of a workaround?
I'm aware I could use the --force or --legacy-peer-deps flags to try and bypass this, but they both generate a very high number of red and yellow warnings.
Thanks.

:warning: Reports we cannot reproduce are at risk of being marked stale and
closed. The more information you can provide, the more likely we are to look
into and address your issue.

I am getting the same error now any solutions ??

I am getting the same error now any solutions ??

@SaiChandra2896  I couldn't find any, unfortunately.

Please review the latest code in the master branch. I am confident that this issue has been resolved. If it persists, kindly reopen the issue with updated details.
Try viewer-dev.ohif.org instead of viewer.ohif.org
Our viewer.ohif.org is deployed from release branch while viewer-dev.ohif.org is our master  branch
Read more about branch explanations here https://docs.ohif.org/development/getting-started#developing


app/tproxy.md: fix reference link

www.dazhuanlan.com is likely a content farm, which collect articles across the web, but don't provide the source link. I found the real source by search the article title on Google.

Thanks for your work

Thanks for your work


Sometimes, no matter how I format the markdown table, it just won't render. Why is that?

Describe the bug
Are there any specific formatting requirements for markdown tables in memos?
| Directory    | Description                                                                           |
|--------------|---------------------------------------------------------------------------------------|
| /            | Root directory, the starting point of the file system.                                |
| /bin         | Essential binary files directory, like `ls`, `cp`, etc.                               |
| /boot        | Bootloader files directory, including kernel images and boot configs.                 |
| /dev         | Device files directory, representing hardware devices like `/dev/sda`.                |
| /etc         | Configuration files directory, storing system-wide configuration files and scripts.   |
| /home        | User home directories, storing personal files and configurations, like `/home/alice`. |

Steps to reproduce

create new thoughts
input

| Directory    | Description                                                                           |
|--------------|---------------------------------------------------------------------------------------|
| /            | Root directory, the starting point of the file system.                                |
| /bin         | Essential binary files directory, like `ls`, `cp`, etc.                               |
| /boot        | Bootloader files directory, including kernel images and boot configs.                 |
| /dev         | Device files directory, representing hardware devices like `/dev/sda`.                |
| /etc         | Configuration files directory, storing system-wide configuration files and scripts.   |
| /home        | User home directories, storing personal files and configurations, like `/home/alice`. |

save

The version of Memos you're using.
v0.22.5
Screenshots or additional context
No response

Try the blow
| Directory    | Description                                                                           |
| -------------- | --------------------------------------------------------------------------------------- |
| /            | Root directory, the starting point of the file system.                                |
| /bin         | Essential binary files directory, like `ls`, `cp`, etc.                               |
| /boot        | Bootloader files directory, including kernel images and boot configs.                 |
| /dev         | Device files directory, representing hardware devices like `/dev/sda`.                |
| /etc         | Configuration files directory, storing system-wide configuration files and scripts.   |
| /home        | User home directories, storing personal files and configurations, like `/home/alice`. |

Based on this https://github.com/usememos/memos/issues/3514#issuecomment-2147334370 each line needs a space between the text and the | and your header divider did not have the required spaces

Try the blow 尝试以下内容
| Directory    | Description                                                                           |
| -------------- | --------------------------------------------------------------------------------------- |
| /            | Root directory, the starting point of the file system.                                |
| /bin         | Essential binary files directory, like `ls`, `cp`, etc.                               |
| /boot        | Bootloader files directory, including kernel images and boot configs.                 |
| /dev         | Device files directory, representing hardware devices like `/dev/sda`.                |
| /etc         | Configuration files directory, storing system-wide configuration files and scripts.   |
| /home        | User home directories, storing personal files and configurations, like `/home/alice`. |

Based on this #3514 (comment) each line needs a space between the text and the | and your header divider did not have the required spaces基于此 #3514 (comment) ，每行都需要在文本和 | 之间留一个空格，并且您的标题分隔符没有必要的空格

This solves my problem, but I still think memos should be more flexible with this kind of grammar.


The plugin upi_pay uses a deprecated version of the Android embedding.

The plugin upi_pay uses a deprecated version of the Android embedding.

There are issues and PR already tracking this. Please don't create duplicate issues.


[WIP] chore(e2e): ephemeral env setup

Description
Please explain the changes you made here.
Which issue(s) does this PR fix
https://issues.redhat.com/browse/RHIDP-4991
https://issues.redhat.com/browse/RHIDP-5059
https://issues.redhat.com/browse/RHIDP-5024
PR acceptance criteria
Please make sure that the following steps are complete:

[ ] GitHub Actions are completed and successful
[ ] Unit Tests are updated and passing
[ ] E2E Tests are updated and passing
[ ] Documentation is updated if necessary (requirement for new features)
[ ] Add a screenshot if the change is UX/UI related

How to test changes / Special notes to the reviewer

/e2e tests

/test e2e-tests

/test e2e-tests

/test e2e-tests

/test e2e-tests

/test e2e-tests

/e2e-tests

/test e2e-tests

/test e2e-tests

/test e2e-tests


[10.x] Sort conflict section in composer.json alphabetically

Sort the conflict section alphabetically, like other sections in composer.json

I would suggest sending this to master branch.


Question with opcodes

We dont have to edit any opcodes currently with this version right?, since its default are set for 5.55 hotfix? Only change the opcodes on .json file when there is a new path or update?

Yes. Is something not working?

No no i was just asking because the new version had a seperate kr and jp server json files.

People were being confused about which section of previous configuration file, and there were issues with JSON string escapes, so I've made them into separate files.

People were being confused about which section of previous configuration file, and there were issues with JSON string escapes, so I've made them into separate files.

Thanks alot. So only change the opcodes in my server's json file when there is an ingame patch or update right? If so, are you no longer providing new releases in accordance with new patches? If we manually have to change the opcodes? Thank you

I will update the files when patches come out, but it might be not asap, and finding opcodes is for people who don't want to wait for me. This way, people can share opcode definition file easier - they can just upload, and for users, they can download and just paste it into the addon directory.

Thank you !!


Union Find Disjoint Sets Implementation In C++

This issue is about Union Find Disjoint Sets

[x] I searched or browsed the repo’s other issues to ensure this is not a duplicate

[x] Issue Title is in form [Algo/DS Name] in [Language] (ex:- BubbleSort in Java)

[x] I will send my PR only after I'm assigned to this issue by any maintainer

[x] Assign this issue to me (I want to work on this)

[x] I ensure that I am not already assigned to 2 or more issues

Kindly assign this to me.

You have been assigned for this issue. Get Started!


Changes to add bfloat16 support for image rotation

This are changes to add bfloat16 to image rotation. When models are run in mixed bfloat16 precision, models that use image transformation would perform much better with image rotation in bfloat16.

@cantonios, Thanks for the review. I was not able to find a testcase in the framework, (that I could test and enable) for bfloat16. Do you want me to add a python testcase or C++ testcase for this?. If you can give me a reference testcase it will be great. I tried updating and running this testcase tensorflow/python/ops/image_ops_test.py (only forImageProjectiveTransformV2) . It fails for different versions of protobuf (latest : 4.23, and based on comments in failure: 3.19, 3.20).

Yes, modifying the existing ImagProjectiveTransformV2 python test is fine.  I don't understand your comment about protobuf issues... the test is currently enabled, so should be passing.  It sounds like a configuration error on your end.

Hi @jojivk73 Any update on this PR? Please. Thank you!

@gbaned , Additions under internal review. Will upstream soon. Thanks


Unsupported OS release: debian

I'm getting the message
"Unsupported OS release: debian"
when trying to install Zulip.
My system is Debian Linux 12.6
Kernel and CPU | Linux 6.1.31-sun50iw9 on aarch64
and I tried both the 9.1 and the Git master.

Anyway to get help here on GitHub? Not really looking to join yet another community.

We don't generally provide support for things that might be user error on GitHub; it's a less efficient workflow.
This (scripts/lib/supported-os) is the script that is failing. I suspect you're not actually running Debian 12.
#!/usr/bin/env bash
#
# This script serves only to verify that the OS is a supported
# version, before we attempt to rely on that version in
# upgrade-zulip-stage-3

if [ -f /etc/os-release ]; then
    os_info="$(
        . /etc/os-release
        printf '%s\n' "$ID" "$VERSION_ID"
    )"
    {
        read -r os_id
        read -r os_version_id
    } <<<"$os_info"
fi

case "$os_id $os_version_id" in
    'debian 12' | 'ubuntu 22.04' | 'ubuntu 24.04')
        exit 0
        ;;
    *)
        exit 1
        ;;
esac

Showing us the contents of /etc/os-release may be instructive.


Replace dep on dry-configurable with a simple Configuration mod

This removes dependency on dry-configurable as dry-container is a lower-level gem and it shouldn't depend on dry-configurable, which is a higher-level gem. Originally, this wasn't the case, but dry-configurable has grown into a quite advanced configuration system (and will most likely continue to evolve).
Configuration needs of dry-container are very basic, they don't justify the usage of dry-configurable. Notice that dry-container is one of the oldest dry-rb gems and throughout the years its config (3 config values: namespace separator, registry and resolver) have been always the same.
Stuff works the same now:
irb(main):001:1* class MyContainer
irb(main):002:1*   include Dry::Container::Mixin
irb(main):003:0> end
=> MyContainer
irb(main):004:0> MyContainer.config.namespace_separator = "/"
=> "/"
irb(main):005:0> mc = MyContainer.new
=> #<MyContainer:0x00007f9a68962a30 @_container={}>
irb(main):006:0> mc.namespace("foo") { |foo| foo.register("bar", :bar) }
=> #<MyContainer:0x00007f9a68962a30 @_container={"foo/bar"=>#<Dry::Container::Item::Callable:0x00007f9a689df990 @item=:bar, @options={:call=>false}>}>
irb(main):007:0> mc["foo/bar"]
=> :bar

Currently, dry-configurable's features propagate to dry-system, and someone may use them. Maybe dry-system's container should continue to have them. Other than this, 👍

Yeah so dry-system's extends its container with Dry::Configurable and the gem depends on dry-configurable explicitly. So this is not a problem (luckily 😅).

@timriley are you OK with this?

@solnic sorry! Less time at the computer while I'm on holidays. I'll try check it out tomorrow.

@solnic Don't forget to remove dry-configurable from project.yml! Otherwise the change in the gemspec will get overridden.


Fix 921 SNLE SNRE code duplication

Fix for #921.
Moved append_simulations to base.py. There is some method specific arguments given to the nle_nre_apt_msg_on_invalid_x function, but purely for logging.
So I added an algorithm argument in base.py and overwite it for the specific method in the super() call.
The nle_nre_apt_msg_on_invalid_x function makes append_simulations specific to SLNE, SNRE. So an alternative would be to have an intermediate object between NeuralInference and the estimators to make this clear?

pyright is failing because of vi_flows type issues.
pytest is passing locally, so it is safe to merge this.


bullet: 401 Unauthorized

Hi, I want gun the bullet with your library.
But I cannot gun bulets becouse Unauthorized error occued.
My System is CentOS release 6.5 (Final) on Vagrant.
I installed with go get -u github.com/mitsuse/bullet , and export BULLET_ACCESS_TOKEN=MY_TOKEN wrote in ~/.bash_profile
Do I need some other settings ?
Anyway, Thanks !

Ok, I can gun the bullet as follows.
BULLET_ACCESS_TOKEN=MY_TOKEN bullet send -l http://example.com
Is this golang environmental problem?

@eccyan Sorry for my late reply.
I have two questions:

What version of go did you use to compile bullet?
Was ~/.bash_profile correctly loaded when auth failed? The Failure may be caused because the old access token (just old or wrong) is loaded.

Thanks!


Improvements for krm-functions/lib/test

Add a slightly modified version of upstream testhelpers.RunGoldenTests(), that automatically creates the _actual_output.yaml.


[Desktop] List Card scroll option is not available

Hi, I am using a List card to show details of different search results, the card is getting rendered properly but I'm not able to scroll the item on desktop as there is no scroll bar available for the card.
I'm able to scroll this using the mouse scroll button.
please help
here is the card payload
ListCard.txt
Desktop view :

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

ID: 3fb4394d-25b1-8752-3996-fe3a2934732d
Version Independent ID: 6d391381-89d0-117c-27cc-00043bd26a36
Content: Cards Reference - Teams
Content Source: msteams-platform/task-modules-and-cards/cards/cards-reference.md
Product: msteams
GitHub Login: @laujan
Microsoft Alias: lajanuar

@Sujit-Gaikwad , This is by design, There is mouse scroll in the list card.

@Sujit-Gaikwad , This is by design, There is mouse scroll in the list card.
@Trinetra-MSFT ,
I didn't get you completely please correct me if I'm wrong,

do you mean that as per list card design we should use mouse scroll as we won't get scroll bar
there.
If yes then,
How users who are working on laptop and doesn't have mouse suppose to scroll the card.

or  2. you mean that there is property for list card using that we can add scroll bar to it or
if I am using any related property then please can you highlight it.
Regards,
Sujit Gaikwad


Clearly separate legacy artifact resolution path from modern path

DefaultLenientConfiguration implemented VisitedArtifactSet, the modern artifact resolution path. We now implement a separate DefaultVisitedArtifactSet to implement the modern path and create a new LegacyVisitedArtifactSet interface, which DefaultLenientConfiguration implements, to handle the legacy artifact resolution path.
This allows us to remove the dependency spec parameter from VisitedArtifactSet, making it clear that the modern path does not support filtering by dependency spec.
Furthermore, we add new interfaces to ResolverResults, clearly separating the modern results from the legacy results.
Reviewing cheatsheet
Before merging the PR, comments starting with

❌ ❓must be fixed
🤔 💅 should be fixed
💭 may be fixed
🎉 celebrate happy things

@bot-gradle test this

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build

@bot-gradle test apt

I've triggered the following builds for you. Click here to see all build failures.

AllPerformanceTestsReadyForRelease build

@bot-gradle test and merge

The merge queue build has started. Click here to see all failures if any.

The merge queue build has failed. Click here to see all failures.

@bot-gradle test this

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build

@bot-gradle test and merge

The merge queue build has started. Click here to see all failures if any.

The merge queue build has failed. Click here to see all failures.

The merge queue build has started. Click here to see all failures if any.

The merge queue build has started. Click here to see all failures if any.

The merge queue build has failed. Click here to see all failures.

The merge queue build has started. Click here to see all failures if any.


Allow the assumption that options is always an object.

Fixes issue where we are attempting assign the value to an undefined member of options when options has not yet been assigned.

https://github.com/atularen/ngx-monaco-editor/issues/59

fixes https://github.com/atularen/ngx-monaco-editor/issues/59


Update list.json

added openTCT/zotct_utils

cool, thanks
What is "Open TCT"? suggest linking in the readme
but anyhow, I'll go ahead and merge it


New version: Initex.YogaDNS version 1.44

Pull request has been created with komac v2.1.0 :rocket:

Resolves #141681

Microsoft Reviewers: Open in CodeFlow

Automatic Validation ended with:

2024-03-02 07:26:51.775   Process exit code: 2

(Automated response - build 857.)

Hi @fancybody,
The installation is unattended, but installs a driver which isn't unattended:
Unfortunately, installer switches are not usually provided for this situation. Are you aware of an installer switch to have the driver silently install as well?
(Automated response - build 859.)

Automatic Validation ended with:

2024-03-04 12:00:02.424   Process exit code: 2

(Automated response - build 861.)

@wingetbot waivers Add Validation-Unattended-Failed

Completing validation.


getPermalinkBaseUrl() must not add the scheme

The APP_URL configuration setting in Laravel is supposed to include the scheme.
For instance in the .env.example file :
APP_URL=https://mydomain.com

See https://github.com/laravel/laravel/blob/master/.env.example for instance and https://github.com/laravel/laravel/blob/master/config/app.php#L55 too.
So in the Twill module controller, getPermalinkBaseUrl() must not add the scheme in front of the APP_URL config of Laravel :
https://github.com/area17/twill/blob/master/src/Http/Controllers/Admin/ModuleController.php#L1388
This breaks the permalink href (with a double scheme, such as http://https://mydomain.com)
The method should be something like :
        return Config::get('app.url') . '/'
            . ($this->moduleHas('translations') ? '{language}/' : '')
            . ($this->moduleHas('revisions') ? '{preview}/' : '')
            . ($this->permalinkBase ?? $this->moduleName)
            . (isset($this->permalinkBase) && empty($this->permalinkBase) ? '' : '/');

We're now talking about this line https://github.com/area17/twill/blob/master/src/Http/Controllers/Admin/ModuleController.php#L1500

I guess this is fixed by https://github.com/area17/twill/pull/651


Research about cert manager issue

Research

Research and figure out how to solve the following cert manager error when running on AWS EKS

ACME server URL host and ACME private key registration host differ

Seemingly it works when running Kubernetes on other infrastructures, but more tests are needed


Clarify that CDC / changefeeds are not intended to be a low latency pub sub mechanism

Our CDC docs say that it

provides efficient, distributed, row-level change feeds into a configurable sink for downstream processing such as reporting, caching, or full-text indexing.

Although it's (perhaps?) implied by the text above, we should say explicitly that

CDC is not a low latency pub sub mechanism
CDC is for high-latency data export from the database to a data warehouse

This came up via internal discussion, so it's possible that some users will need this clarified / made more explicit as well.

@lnhsingh please feel free to unassign if you are no longer the CDC person!

@rmloveland - nope, still me. thanks for filing!


Enable CORS support via the application

Allow a way to enable CORS support.

Released in v0.1.11


wefit_admin.js:1 Uncaught Error: Sprockets::FileNotFound: couldn't find file 'foundation' with type 'application/javascript'

When I add the gem in my engine, gemspecs as follow
 s.add_dependency "rails", "~> 5.0.0", ">= <IP_ADDRESS>"

  s.add_dependency 'haml'
  s.add_dependency 'haml-rails'
  s.add_dependency 'sass-rails'
  s.add_dependency 'uglifier', '>= 1.3.0'
  s.add_dependency 'jquery-rails'
  s.add_dependency 'foundation-rails'

my_engine.js as follow
//= require jquery
//= require jquery_ujs
//= require foundation
//= require_tree .

$(document).foundation();

I setup engine.rb as follow
initializer 'WefitAdmin precompile hook', group: :all do |app|
      root_foundation_gem = Gem.loaded_specs['foundation-rails'].full_gem_path
      Dir.glob("#{root_foundation_gem}/**/").each do |path|
        app.config.assets.paths << path
      end

      app.config.assets.precompile += %w(my_engine/my_engine.css my_engine/my_engine.js)
    end

When i run with development mode, my site can load css file, but cause error when load js file.
I check foundation gem directory and see foundation.js.es6. So i think maybe the file is not precompiled.

Facing a similar issue here


[Feature] Build renewal/replacement request modals

Notion ticket link

Renewal/Replacement Request modals

Implementation description

Created Edit modals (EditDoctorInformationModal, EditPaymentDetailsModal, EditReasonForReplacementModal) used in the request renewal/replacement flows

Notes

There is no form submission logic and validation is handled by Chakra itself. Error state will likely be addressed post-MVP.
Left a few TODO comments in the code that should be addressed at a later time

Checklist

[x] My PR name is descriptive, is in imperative tense and starts with one of the following: [Feature],[Improvement] or [Fix],
[x] I have run the appropriate linter(s)
[x] I have requested a review from the RCD team on GitHub, or specific people who are associated with this ticket

@OustanDing Thanks for all the great feedback! I think the PR is ready for another review!
Also addressed some feedback from Charmaine which was adding Dividers and changes to capilization


improvement Improve fe high concurrent query performance

Proposed changes
Issue Number: close #7935
Problem Summary:
Describe the overview of changes.
Checklist(Required)

Does it affect the original behavior: (No)
Has unit tests been added: (No)
Has document been added or modified: (No Need)
Does it need to update dependencies: (No Need)
Are there any changes that cannot be rolled back: (No)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

optimized statistic, 5k connection query 'select 1'
before 561ms 262ms 15.56k
after    406ms 130ms 32.2k


Forgot to tag 3.1.0 in git

Just a note that it seems you forgot to tag the git repo with the 3.1.0 release.

https://github.com/devinus/poison/releases/tag/3.1.0


Fixing The Withdraw Flow

Check out from https://github.com/webb-tools/webb-dapp/pull/806
Summary of changes

Add logging service by @AhmedKorim
Refactoring redundant / duplicated deposit/transfer/withdraw codes from dApp
Update the relayer for consuming new API
Got the deposit (same-chain and cross-chain) working
Got the withdrawal (same-chain and cross-chain) working with or without relayer
In the withdraw flow, display the switch chain button when the balance on the active chain is 0. Otherwise

Auto switch to an available chain that has a balance greater than 0 and if the user only has one available chain
Or display a list of available chains to switch into to withdraw

In the deposit flow, auto-switch to the wrapping function if a user does not have a balance
Display the user's balance on the withdrawal flow
Change the delay of the amount tooltip in the withdrawal flow

Reference issue to close (if applicable)

Related https://github.com/webb-tools/webb-dapp/issues/841
Closes https://github.com/webb-tools/webb-dapp/issues/835
Related https://github.com/webb-tools/webb-dapp/issues/840
Related https://github.com/webb-tools/webb-dapp/issues/838
Closes https://github.com/webb-tools/webb-dapp/issues/794
Closes https://github.com/webb-tools/webb-dapp/issues/817
Related https://github.com/webb-tools/webb-dapp/issues/797

Code Checklist

[x] Tested
[x] Documented

Demo
https://www.loom.com/share/2bc0f71b7ef040a285425d85c9e7fdc4

Adding changes:

In the withdraw flow, display the switch chain button when the balance on the active chain is 0. Otherwise

Auto switch to an available chain that has a balance greater than 0 and if the user only has one available chain
Or display a list of available chains to switch into to withdraw

In the deposit flow, auto-switch to the wrapping function if a user does not have a balance
Display the user's balance on the withdrawal flow
Change the delay of the amount tooltip in the withdrawal flow

Linked issues:

Related https://github.com/webb-tools/webb-dapp/issues/841
Closes https://github.com/webb-tools/webb-dapp/issues/835
Related https://github.com/webb-tools/webb-dapp/issues/840
Related https://github.com/webb-tools/webb-dapp/issues/838
Closes https://github.com/webb-tools/webb-dapp/issues/794
Closes https://github.com/webb-tools/webb-dapp/issues/817
Related https://github.com/webb-tools/webb-dapp/issues/797

@AtelyPham great work as usual! A couple things I noticed to be addressed.

Update container header to read "Wrap and Deposit Completed" once the transaction is successful or if it fails indicate "Wrap and Deposit Failed".
The user currently needs to select the copied note checkbox in order to enable "New Transaction" button, this shouldn't be the case as the user would have already made that selection when initiating the deposit.
Update the transaction card header to read "Transaction Completed" instead of "Transaction Processing" once the tx is completed successfully, the inverse would be "Transaction Failed" when it has been unsuccessful.
Lastly, and this could be done in a subsequent PR the transaction card remains present until the user selects the "Dismiss" button. We should update this to disappear upon the users next operation (e.g. clicks "New Transaction") or disappear after X amount of time.

See below screenshot.

Transaction Processing Component @monaiuu @AtelyPham @dutterbutter
https://www.figma.com/file/H43lYnTCOAPjO9kTrrepe1?node-id=1200:22610#339222045

On Withdrawal page:

Deposit flow review: https://www.loom.com/share/84852dc4da514957847fbc912886608a

Chain switching issues: https://www.loom.com/share/84852dc4da514957847fbc912886608a

Resolved the display comments @dutterbutter. I'm working on "Auto switching to wrapping function when a user has no balance" on the deposit flow. Little stuck with state and side effect, I'll continue to investigate

We have "Auto switching to wrapping function when a user has no balance" on the deposit flow.


How to render static imagery map image with pins

[Enter feedback here]
I would request you to please provide me details as how can I get a static satellite imagery map image with pins embedded on it. I can only get pins with static basic/hybrid etc image, but my requirement is to get an satellite image with pins on it.

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

ID: 9aeb145b-1372-1e2a-c10a-2d430d72b4f5
Version Independent ID: c59a5271-fd5d-1950-21fb-9822124bf6a4
Content: Render custom data on a raster map
Content Source: articles/azure-maps/how-to-render-custom-data.md
Service: azure-maps
GitHub Login: @philmea
Microsoft Alias: philmea

@vinod83k Thank you for the inquiry. We are investigating your request I also suggest you present the same question to Stack Overflow and Microsoft Q&A, where this question is better suited. However, I have assigned this to an engineer to be investigated.

Adding @rbrundritt for further insights.
If a layer is set to hybrid  It returns an image containing borders, roads, and labels, and can be overlaid on other tiles (such as satellite imagery) to produce hybrid tiles. Please see Render - Get Map Image

Currently the static map image service does not support satellite imagery. @outinyman who is the PM for the service, has this on the feature request list.

Thanks for the confirmation, Ricky.
@vinod83k Hope that answers your question. Do let us know if you have any further queries.

@AshokPeddakotla-MSFT can you please check with team that is there any way by which we can get a bigger tile image of satellite image instead of just 256x256. Actually we need a bigger or better resolution image to render on pdf. This small size image is of no use then because on rendering it on pdf the image gets distorted. Please suggest.

There isn't other size available at this time. It is however possible to download multiple tiles and stich them using the tile schema: https://docs.microsoft.com/en-us/azure/azure-maps/zoom-levels-and-tile-grid?tabs=csharp In fact, if your image needs less than 15 tiles, this would work out cheaper than using the Map Image service.

@rbrundritt  The feature which are we are trying to achieve is to render Azure Maps on pdf using wkhtmltopdf C# wrapper.
Also, I would like to provide you the different approaches we tried to achieve it but not been able successful.

We tried to use Azure Map REST Api's to get different map images i.e., satellite map image and overlayed it with hybrid map image (with pins). The problem we are facing here is that the satellite image size is 256x256 and when we use it to render on pdf and stretch it we get distorted image which does not look well.
As wkhtmltopdf lib can convert html to pdf, we tried using the Azure Maps web sdk to be used on html and then we shall create pdf using the rendered html. Here we are getting an error from html that the 'Map' object provided by atlas.min.js is undefined and so map is not rendered on html.
We even tried creating pdf from html by nodejs app as well. Here first of all we faced a challenge that Azure App service does not support phantamjs (html rendering engine) because of the sandbox environment with limited permissions. Then we thought of running this nodejs app in VM, but here when it comes to rendering map in pdf, we faced the similar challenge with Azure Maps web sdk. It was giving us error of 'Map' object is undefined.

It will be really helpful if you can provide us some examples of rendering satellite map image with pins. Thanks !!

Ok, some comments;
For #1, there are several REST API's, there are "Map imagery tile" API, which returns the 256x256 map tiles you mentioned. There is also the "map image" API which returns map images of larger size, but doesn't currently support satellite imagery. It is possible to use the satellite tiles and stich them together to make a larger image.
For #2, I'm not familiar with wkhtmltopdf, but understand what you are trying to achieve. The map uses WebGL, so to print the map, you have to change one of the map settings. Here is a code sample: https://azuremapscodesamples.azurewebsites.net/index.html?sample=Export Map as Image Alternatively, if you are having too many issues using this SDK due to WebGL, you could use an older open source map control such as leaflet which uses DOM elements for map rendering. It won't be able to render as much data as the Azure Maps control, but would likely be easier for an HTML to image converter to use. Here is a sample of using Azure Maps with leaflet: https://azuremapscodesamples.azurewebsites.net/index.html?sample=Azure Maps Raster Tiles in Leaflet JS

Considering the following co-ordinates and parameters to get the satellite imagery:
Latitude = -12.07685064;
Longitude = 26.41362781;
defaultZoom = 17;
tileSize = 256;
Following is the code to calculate the tileX and tileY.
int tileX, tileY;
var pixelXY = AzureMaps.TileMath.PositionToGlobalPixel(new double[] { Longitude, Latitude }, defaultZoom, tileSize);
AzureMaps.TileMath.GlobalPixelToTileXY(pixelXY, tileSize, out tileX, out tileY);
After this I am trying to generate the following url for satellite image:
https://atlas.microsoft.com/map/imagery/png?api-version=1.0&subscription-key=[subscription-key]&style=satellite&zoom=17&x=75152&y=69965
And this returns the following image:

But in actual, when trying to pin the above co-ordinates on web page using web sdk, the image is different.

I have tried multiple combinations but not able to get the center of the satellite image as of same of web sdk image. Can you please help on this?

Attached is a fairly complex code sample I put together for a customer a while back. It takes in a GeoJSON file and style information defined in a JSON file and then generates a static map image of just about any size (massive images will likely run into memory issues).
[
AzureMapsRenderingService.zip
](url)
To change custom images, add your png image to the images folder in the CustomRenderingService project. Then add the iconName property of the JSON style. If you want to modify how it's positioned, add an entry into the IconImageConfig.cs file.
I haven't made this an open source project yet as I have a couple of larger changes I want to make (use SkiaSharp for rendering instead of System.Drawing)

@rbrundritt , Thanks for all your help. I have finally being able to render map using leaflet js. The other solutions did not worked for me.
Meanwhile, I was just trying to figure out as how we can get the best optimal zoom level based on the location co-ordinates available to render on map. Do you have any reference for this ?

If you are using leaflet and have more than one point in your data set, calculate the bounding box of the data (min/max of lat/long) will be good enough for most scenarios. From there, you can use the maps fitBounds function to have it automatically zoom and center the map for you. I recommend adding a padding of 20 or 30 pixels to account for the size of your icon.


HmIP-eTRV shows Heating while Off

I have noticed a small display error if i turn off the heating. In the old integration lovelace shows off, here it stays on Heating.

Also, i have noticed, that if the window is opened, I have to press off two times to really turn off the thermostat. After the first time, it goes from Automatic 12°C to Heating 12°C.  But thats the same with the old integration, so maybe it's a CCU limitation.

Good Point.

Turn off issue should be fixed with latest release.

Yes, now it is showing the state correctly.  Thanks


Do not scroll during debugging session

While debugging python, while jumping thought lines it auto scrolls vertically. So the line the debugger is in stays at the center.
I would prefer to keep the scroll position and, instead, just move the highlight to the next break position.
https://github.com/daveleroy/SublimeDebugger/assets/109356759/cc938727-3298-4de4-bd8c-7ddeaa5fa19a

view.show is now used to show the current line


modify Caliper Reader to use new Frame object

@bhatele @tgamblin: Any comments or feedback on this PR? For the readers, should the dataframe columns match what is going into the frame attrs dict?


(PDOC-63) Code refactoring, fix up, and lots of new functionality.

This work is a re-implementation of Puppet strings to clean up the code, fix a large number of bugs, migrate to the latest version of YARD, and add additional functionality like documenting custom functions written in the Puppet language, ability to document Puppet function overloads (using the create_function 4.x API) and switch to using Markdown as the default format for docstring text (note: this means Markdown text in YARD comments and tags and not a change to document with Markdown itself).
This work makes some breaking changes with respect to the JSON output so that it outputs all the data it has on the Puppet-related YARD code objects.  See the updated documentation for the new schema.

👍 thanks a TON, @peterhuene !!!!!

@peterhuene Hi!  The Vox Pupuli project currently uses puppet-strings from github.  I was wondering if there might be many breaking changes before the next release and if a new release is perhaps imminent?
Many thanks,
Alex

There were a few breaking changes, the biggest one was the yardoc action was renamed to generate.  If you were invoking using the default action (e.g. puppet strings), then this shouldn't affect you; otherwise puppet strings yardoc needs to change to puppet strings generate.
The other big breaking change is the update to the JSON schema; this affects the --emit-json and --emit-json-stdout options, so if those are not used, it shouldn't impact you.
From the UI perspective, this work migrates Puppet Strings to YARD 0.9.5, which appears different from the much older version of YARD that Puppet Strings was previously using.  However, the work fixed a lot of small issues in the generated HTML, so hopefully the HTML output is now more polished.

Thanks for the clarification, @peterhuene. I know this might be a bit optimistic, but you do you have an approximate window for "very soon"?

I'd personally (i.e. I can't comment officially, which may set unmet expectations) hope for within a week or two, but unfortunately I have no control over the release process.


Please allow building with GHC 8.4.1 (base-<IP_ADDRESS>, template-haskell-<IP_ADDRESS>)

The build seems to work fine when the constraints on base and template-haskell are relaxed.

I'm not on GHC 8.4.1 yet, and it appears that Travis CI doesn't have GHC 8.4.1 installed either. I could do a blind patch to update the dependencies, but I'd feel a lot better if I knew that the CI system does the building & testing for me. I'll wait a bit for the Travis guys to catch up, maybe it won't be long.


Unable to delete the plugin from plugin menu option

Description
After installing the OSC from Build#96, delete option does not appear due to nauge plugin OSGI error.
Expected Behavior
Actual Behavior
deletion button on the plugin should appear.
Steps to Reproduce

Deploy the OSC Build#96.
Go to Plugin Menu
Delete option for the plugin does not appear due to nauge plugin error message is too long.

Additional Information
Environment

OSC Version/Build: OSC Build#95.
Cloud Provider Info:
Plugin Info:
Other:

Status

[ ] Reproduced
[ ] In Progress
[ ] Validated     =>    Verified in Version/Build:

[x ] Validated     =>    Verified fix in Version/Build: v0.8 Build#123

This issue is still happening on later versions of OSC. For this to repro though one needs a defective plugin.

[x] Validated     =>    Verified fix in Version/Build: v0.8 Build#125


fix: login problem in fullscreen mode

fixes #747

:tada: This PR is included in version 1.1.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


[JVM] Fudge some exploding tests

Not needed anymore -- see https://github.com/rakudo/rakudo/issues/4732.


Change messaging around Windows

/cc @eseidel @sethladd

There are other bugs too, but rslgtm.


Display name hueristics for table and columns

Use display name hueristics annotation to determine the display name transformation e.g. "My_Table" -> "My Table" or "my_table" -> "My Table".
We probably need to define a chaise config, a new annotation at the schema and table level on how to derive the display name instead of annotating every columns.. Assigning this to @robes and @karlcz to come up with the annotation syntax.

@robes I've drafted annotation support for this in 2015:display.name_style

@karlcz The only issue I can see is that the concept of a "title case" or at least the implementation of a title case may be language dependent. That is, is this an English only feature? Would we know how to implement this for any other language? We have a couple options, either we say this is a feature that works for model element names that are use the Latin-1 charset. Other option, we deprecate this feature and have all our databases updated to use a column name exactly as they want it to be displayed. Personally, (although I think I created this monster) I am not fond of this behavior in our apps... :(
I would actually prefer to deprecate this feature. (I think)
Since we've now added so much to this annotation why don't be just update it from tag:misd.isi.edu,2015:display to tag:isrd.isi.edu,2016:display too?

@robes Let's assume that this feature will only work for Latin-1 charset (which is pretty much all our deployments).

I think the spec should be open-ended.  We should not have to talk about something like Latin1 here. The concept of word-splitting and casing are language specific but well understood.  Our implementation may be best effort.  It is OK to ignore an annotation if an implementation doesn't know how to handle it...

I expect we'll split words on whitespace, hyphen, and underscore.  A better implementation might apply language-specific heuristics.
I expect we'll call toUpperCase() or toLocaleUpperCase() on the first codepoint in each word after splitting.  I think these methods are supposed to be quite general and should do no harm, right?

Does the underline_space mean just simply convert each underline to a space?
Or it means split string on whitespace, hyphen, and underscore? So a__-_b would become a b.

I think it means the Javascript behavior "string".replace(/_/g, " ")

So shouldn't it be something like str.replace(/[_\-]/g, ' ').replace(/\s\s+/g,' ') to include multiple spaces and hyphens too?

No we are not replacing anything except _ ->   for the
underscore_space mode.
Word-splitting is for the purpose of identifying word boundaries to
apply title-casing.  You do not replace or modify the word separators
but use them to identify the beginning-of-word characters that should
have toUpperCase() applied.

A couple comments on module._determineDisplayName ands its usage, as it stands now.

the display name's for all model elements will be precomputed. In most of our scenarios where we are only displaying a subset of model elements that means that we are doing preprocessing that could be avoided.
does this function belong in utils and attached to module or could it just be a local function within the ermrest.js script? the answer depends on whether it ever gets used outside of ermrest.js.

This function is used in Schema, Table, and Column class. It is called in their constructors to determine their display names.
I thought the purpose of utilities.js is to extract the functions that are used in multiple places in ermrest.js file. If this is not the purpose, it can just be a local function in ermrest.js. _toTitleCase and _underlineToSpace should be local functions in ermrest.js too.

#73 implemented this and #82 added the unit tests for it.


Update Skeleton color to use "Grey-100" variable token to align to developed component.

Background
Land titles service team (Graeme) noted that the skeleton loader was a bit visually "heavy" when applied in context. On further investigation, it was discovered that the design specification is using a heavier grey (grey-300) than the grey that is specified in the web components ui-components libraries.
Acceptance criteria

[x] Update Figma component to use "grey -100"
[x] Update Figma design documentation into "new format"

I have updated the documentation in Figma to include the correct colour (#f1f1f1 - grey-200) as is used in Storybook.
Also updated the documentation in Figma to new format.


Adding support to load libraries (Once) and save time in the deploy…

…ment process

The user will have to update only one file config.gradle.
Previously, the user had to touch 2 files: MANIFEST.MF & config.gradle
No more hardcoded password in the gradle files
Added gradle files to load in the brick some files
Added a simplelogger.properties example
Added new configuration

I have hopefully finished the refactoring. I have a new review point:
jvmFlags += "-noverify "
jvmFlags += "-XX:TieredStopAtLevel=1 "
jvmFlags += "-Djava.security.egd=file:/dev/./urandom"

You have commented with these flags that they should increase performance. You can uncomment them now in config.gradle and try them live.
Regards,
Jakub

Hi @JakubVanek,
installing the libraries from the installer:
I saw this messages:
> Task :installJavaLibraries
Strict host key checking is off. It may be vulnerable to man-in-the-middle attacks.

ev3dev#12| --- Running command: "echo -e "maker" | sudo -S /home/robot/java/installer.sh javaLibs" ---
ev3dev#13|
ev3dev#13|##############################
ev3dev#13|# EV3Dev-lang-java Installer #
ev3dev#13|##############################
ev3dev#13|
ev3dev#13|Platform detected: ev3
ev3dev#13|
ev3dev#13|[sudo] password for robot: 2018-12-02 08:27:50 URL:https://jitpack.io/com/github/ev3dev-lang-java/ev3dev-lang-java/2.4.16/ev3dev-lang-java-2.4.16.jar [115531/115531] -> "/home/robot/java/libraries/ev3dev-lang-java-2.4.16.jar" [1]
ev3dev#13|2018-12-02 08:27:50 URL:https://jitpack.io/com/github/ev3dev-lang-java/lejos-commons/0.7.2/lejos-commons-0.7.2.jar [108826/108826] -> "/home/robot/java/libraries/lejos-commons-0.7.2.jar" [1]
ev3dev#13|2018-12-02 08:27:51 URL:http://central.maven.org/maven2/net/java/dev/jna/jna/4.5.2/jna-4.5.2.jar [1484022/1484022] -> "/home/robot/java/libraries/jna-4.5.2.jar" [1]
ev3dev#13|2018-12-02 08:27:51 URL:http://central.maven.org/maven2/org/slf4j/slf4j-api/1.7.25/slf4j-api-1.7.25.jar [41203/41203] -> "/home/robot/java/libraries/slf4j-api-1.7.25.jar" [1]
ev3dev#13|2018-12-02 08:27:51 URL:http://central.maven.org/maven2/ch/qos/logback/logback-classic/1.2.3/logback-classic-1.2.3.jar [290339/290339] -> "/home/robot/java/libraries/logback-classic-1.2.3.jar" [1]
ev3dev#13|2018-12-02 08:27:51 URL:http://central.maven.org/maven2/ch/qos/logback/logback-core/1.2.3/logback-core-1.2.3.jar [471901/471901] -> "/home/robot/java/libraries/logback-core-1.2.3.jar" [1]
ev3dev#13|2018-12-02 08:27:51 URL:http://central.maven.org/maven2/org/slf4j/slf4j-simple/1.7.25/slf4j-simple-1.7.25.jar [15257/15257] -> "/home/robot/java/libraries/slf4j-simple-1.7.25.jar" [1]
ev3dev#13|FINISHED --2018-12-02 08:27:51--
ev3dev#13|Total wall clock time: 3.0s
ev3dev#13|Downloaded: 7 files, 2.4M in 0.9s (2.66 MB/s)
ev3dev#13|find: 'jar': No such file or directory
ev3dev#13|find: 'jar': No such file or directory
ev3dev#13|find: 'jar': No such file or directory
ev3dev#13|find: 'jar': No such file or directory
ev3dev#13|find: 'jar': No such file or directory
ev3dev#13|find: 'jar': No such file or directory
ev3dev#13|find: 'jar': No such file or directory

What is it?
ev3dev#13|find: 'jar': No such file or directory
ev3dev#13|find: 'jar': No such file or directory
ev3dev#13|find: 'jar': No such file or directory
ev3dev#13|find: 'jar': No such file or directory
ev3dev#13|find: 'jar': No such file or directory
ev3dev#13|find: 'jar': No such file or directory
ev3dev#13|find: 'jar': No such file or directory

Hmmm, I have added the jar program but haven't added a jar symlink. It is for generating the class list for AppCDS.

Verified all points & discussed


Roll Flutter Engine from 38a6ef7548f5 to 6561adf5ad87 (1 revision)

https://github.com/flutter/engine/compare/38a6ef7548f5...6561adf5ad87
2024-05-11<EMAIL_ADDRESS>Roll Skia from b6186e1b3995 to 709deddd904f (1 revision) (flutter/engine#52727)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Fix install destination with directory prefixes

In addition to #128, the install destination should be fixed to make sure the directory prefixes are preserved.
For example, when the extension name is foo/bar, the target shared library bar.so will be installed in lib directory in the current master branch in spite of it should be in lib/foo.
Moreover, I've added some examples for the case with directory prefixes.

Thanks.
I've merged.


Improve FCM.hasPermission() on Android

Verify implementation of

https://github.com/dpa99c/cordova-diagnostic-plugin#isremotenotificationsenabled

And if it would be interesting to internalize into this plugin.

Thanks to @bwojtyca for mentioning this plugin.

@andrehtissot  hi! Any updates?  We can help with PR if it is need

It's implemented and it'll be available in the next update.


Rundi command mode options

Command mode options for rundi.nvim

Make :rundi command to trigger rundi
Add option for extra arguments to be passed which overrides default config for build-system/language and global rundi configs.
Extra args should not be persistant.
Maybe :rundi set(extra flag) or :rundi-set(different command) command to override/set configs.

imma work on this pls :)

arigato

can i get the dependency list for working on this ?

There are no dependencies, just plain lua

I'm still working on it, give me some more time

!assign @ninsid711

add me in again pls

@neb0lle

!assign @ninsid711

!assign @ninsid711

!assign @ninsid711

!deassign

!deassign

Hey can I please get assigned to this?


Live Instance migration with storage getting failed

ISSUE TYPE

Other

COMPONENT NAME
Migration along with storage

CLOUDSTACK VERSION
<IP_ADDRESS>

CONFIGURATION
Single Management server
3 Hypervisor host: RHEL 8.6 with KVM
Basic network configuration with Shared network option is selected during zone configuration
PowerFlex as Primary storage
OS / ENVIRONMENT
Management Server: RHEL 8.6
Hypervisor: RHEL 8.6 with KVM
SUMMARY
Individually I'm able to migrate volumes from one storage pool to other pool. However when I select migrate storage during VM migration its giving error.
STEPS TO REPRODUCE
After complete setup like CloudStack, PowerFlex side, adding pool etc
create instance from CloudStack UI with data disk.
select instance: click on migrate instance to another host, select target host, select migrate with storage and select different storage pool for each disk, click okay.
EXPECTED RESULTS
instance should migrate to target host along with the volumes/disk migration as specified in the above steps.
ACTUAL RESULTS
its giving migration failed error with Error message: Migrating a volume online with KVM from managed storage is not currently supported

@Punithahs I am not sure how complete powerflex support is (pertaining to basic zones particularly)
As per the message what you are trying to do is not supported. You can try stopping the VM and then migrating it.

Hi,
in the below link it talks about migration however its bit confusing whether they talking only about storage migration or VM with storage migration. In my case VM with Storage migration is not working.
https://docs.cloudstack.apache.org/en/<IP_ADDRESS>/adminguide/storage.html#migrating-a-instance-root-volume-to-a-new-storage-pool

Hi @Punithahs, migrating a VM to another host with the storage is not supported for managed storage like PowerFlex - live VM migration, check the second note.
The link you shared is for migrating the PowerFlex's Root volume of an instance.
You can probably change this as a feature request.

@harikrishna-patnala could you check and advise if the current PowerFlex storage plugin supports live VM migration with storage, or just the live storage migration of powerflex volumes (across different powerflex storage pools?).

I see there is a specific check for this work flow, which restricts the volume migration when we do VM migration along with storage,
https://github.com/shapeblue/cloudstack/blob/82f7abddb3010d2d0c0d0b1622602d06d2a32016/engine/storage/datamotion/src/main/java/org/apache/cloudstack/storage/motion/StorageSystemDataMotionStrategy.java#L2379-L2381
but I don't see a reason to keep this restriction as we are already allowing the volume migration individually.
@sureshanaparti do you know any known issue with this sort of migration ? I feel we can remove this check and fix this case.


Eloquent custom class cast cache problem

Laravel Version: 7.15.0
PHP Version: 7.4.5
Database Driver & Version: N/A

Description:
Eloquent is caching the result of custom class cast calls. The cache key is the attribute name.
There is only one cache that is used for both original and dirty attributes.
As a result:

the value initialized by getOriginal or getAttribute will always be returned until the cache is reset,
any changes might be overwritten by the original value when mergeAttributesFromClassCasts is called from HasAttributes::getAttributes.

Steps To Reproduce:

Create a model with a custom class cast property
Set a value
Persist the property with syncOriginal
Change the property value to a null or non null value
Call getOriginal: $classCastCache is initialized with the original value
Call getDirty again: cache will be persisted by the HasAttributes::mergeAttributesFromClassCasts in HasAttributes::getAttributes

// Always returns the first fetched value
$user = new MyUser([
    'prop' => MyEnum::VALUE,
]);
$user->syncOriginal();
$user->prop = MyEnum::OTHER_VALUE;
dump($user->getOriginal('prop'), $user->prop);
// Expected: MyEnum::VALUE, MyEnum::OTHER_VALUE
// Actual: MyEnum::VALUE, MyEnum::VALUE

// Dirty changes lost
$user = new MyUser([
    'prop' => MyEnum::VALUE,
]);
$user->syncOriginal();
$user->prop = null;
dump($user->prop, $user, $user->getOriginal('prop'), $user->prop);
// Expected: null, MyEnum::VALUE, null
// Actual: null, MyEnum::VALUE, MyEnum::VALUE

Possible Fixes
We might create two caches: one for original values and one for changes.
We could also not use the cache with getOriginal.
I did the following in my BaseModel:
public function getOriginal($key = null, $default = null)
{
    $previousCache = $this->classCastCache;
    $this->classCastCache = [];

    $result = parent::getOriginal($key, $default);

    $this->classCastCache = $previousCache;

    return $result;
}

Share your cast class.

class MyEnum implements Castable
{
    public const VALUE = 1;
    public const OTHER_VALUE = 2;
    public $value;
    public function __construct($value = null)
    {
        $this->value = $value;
    }

    public static function castUsing()
    {
        return new MyEnumCast(static::class);
    }
}

class MyEnumCast implements CastsAttributes
{
    protected $enumClass;

    public function __construct(string $enumClass)
    {
        $this->enumClass = $enumClass;
    }

    public function get($model, string $key, $value, array $attributes)
    {
        if ($value === null) {
            return $value;
        }

        if ($value instanceof MyEnum) {
            return $value;
        }

        return new $this->enumClass($value);
    }

    public function set($model, string $key, $value, array $attributes)
    {

        $value = $this->get($model, $key, $value, $attributes);

        return [$key => $value->value ?? null];
    }
}

Fixed by: https://github.com/laravel/framework/commit/b20125d7bf270bcd6cc651114512a2dc7f182a96
Can you try this patch in your own application?

Thanks for the quick feedback.
This solves the problem when the key is given to getOriginal().
However, the following still fails:
$user = new MyUser([
    'prop' => MyEnum::VALUE,
]);
$user->syncOriginal();
$user->prop = MyEnum::OTHER_VALUE;

dump($user->prop, $user->getOriginal(), $user->prop);
/*
MyEnum {#1443
  +value: 2
}
array:1 [
  "prop" => MyEnum {#1443
    +value: 2
  }
]
MyEnum {#1443
  +value: 2
}
*/

I will make a slight change for this; however, it's important to note that if you pull a record from the database and hit getOriginal() with no arguments, you will get the raw, original attributes. If you have a custom class cast that maps to multiple attributes only the attributes it maps to would be included in the original array because there is no corresponding "column" for the class cast attribute if it serving as a "virtual" attribute of sorts.

OK thanks.

Will be fixed in next patch release.


Look into GO:0046488 phosphatidylinositol metabolic process & children

While reviewing regulation of phosphatidylinositol 3-kinase activity' & children for https://github.com/geneontology/go-ontology/issues/25383, I noticed that there are pathways involving various phosphatidylinositol that we dont have in GO. Some were annotated to GO:0048015 phosphatidylinositol-mediated signaling  (which maybe we should keep, but add appropriate children?), others to regulation of phosphatidylinositol 3-kinase activity, and maybe some calcium signaling.
It would be very helpful to review annotations to GO:0046488 phosphatidylinositol metabolic process & children to see if these annotations should be rehoused and if we need new GO terms.

Papers that could use some reannotation:

PMID:18723082
PMID:14676841
PMID:15157668
PMID:18694560


make demo not work

What happened?

demo is not work.
$ make demo
kubectl apply -f ./config/crd
error: the path "./config/crd" does not exist
make: *** [demo] Error 1

How can we reproduce it?

run make demo
What environment did it happen in?
Crossplane version:

thanks


trafficpolicy: Remove conditional always evaluating to true

Removing if !routeExists { since this will always evaluate to true.

Affected area:

New Functionality      [ ]
Documentation          [ ]
Install                [ ]
Control Plane          [ ]
CLI Tool               [ ]
Certificate Management [ ]
Networking             [ ]
Metrics                [ ]
SMI Policy             [ ]
Security               [ ]
Tests                  [ ]
CI System              [ ]
Performance            [ ]
Other                  [ ]

Please answer the following questions with yes/no.

Does this change contain code from or inspired by another project? If so, did you notify the maintainers and provide attribution?

@michelleN would you be so kind to take a look at this? Was routeExists meant to be flipped to true at some point (which seems to be the error condition)?

@michelleN would you be so kind to take a look at this? Was routeExists meant to be flipped to true at some point (which seems to be the error condition)?

Codecov Report

Merging #2250 (0480b04) into main (268b512) will increase coverage by 0.20%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #2250      +/-   ##
==========================================
+ Coverage   58.16%   58.36%   +0.20%
==========================================
  Files         145      145
  Lines        6377     6375       -2
==========================================
+ Hits         3709     3721      +12
+ Misses       2663     2649      -14
  Partials        5        5

Impacted Files
Coverage Δ

pkg/trafficpolicy/trafficpolicy.go
100.00% <100.00%> (ø)

pkg/configurator/validating_webhook.go
78.98% <0.00%> (+10.14%)
:arrow_up:

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 268b512...0480b04. Read the comment docs.

Codecov Report

Merging #2250 (0480b04) into main (268b512) will increase coverage by 0.20%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #2250      +/-   ##
==========================================
+ Coverage   58.16%   58.36%   +0.20%
==========================================
  Files         145      145
  Lines        6377     6375       -2
==========================================
+ Hits         3709     3721      +12
+ Misses       2663     2649      -14
  Partials        5        5

Impacted Files
Coverage Δ

pkg/trafficpolicy/trafficpolicy.go
100.00% <100.00%> (ø)

pkg/configurator/validating_webhook.go
78.98% <0.00%> (+10.14%)
:arrow_up:

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 268b512...0480b04. Read the comment docs.