[No QA] [TS migration] Migrate 'TransactionUtils.js' lib to TypeScript

Details

Migrate 'TransactionUtils.js' lib to TypeScript
Fixed Issues

$ https://github.com/Expensify/App/issues/24893
PROPOSAL: N/A
Tests

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

Make sure the Request Money functionality works as expected
Offline tests

QA Steps

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

Make sure the Request Money functionality works as expected
PR Author Checklist

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

Screenshots/Videos

Web

https://github.com/Expensify/App/assets/23176449/f38c3aed-1861-43ba-80ef-5d81c3bb9148

Mobile Web - Chrome

https://github.com/Expensify/App/assets/23176449/5f55f264-b905-41a4-b3ad-c2b65748b99f

Mobile Web - Safari

https://github.com/Expensify/App/assets/23176449/6ba2b5c4-ec36-4793-862b-aaf91391e958

Desktop

https://github.com/Expensify/App/assets/23176449/cc630135-13de-4f92-bd19-ad558c84dd50

iOS

https://github.com/Expensify/App/assets/23176449/ab44ee1e-9fff-49b4-9a9c-939de543c27b

Android

https://github.com/Expensify/App/assets/23176449/c44a09e4-57ce-43b3-b003-4bb4d107162c

Reviewing...

LGTM! Make sure to check suggestions from Błażej @VickyStash

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

🚀 Deployed to staging by https://github.com/cristipaval in version: 1.3.76-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/mountiny in version: 1.3.76-6 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅

🚀 Deployed to staging by https://github.com/cristipaval in version: 1.3.77-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/mountiny in version: 1.3.77-7 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅


infoWindowClose event not firing

The infoWindowClose event for sebm-google-map-info-window is not firing.

Added: (infoWindowClose)="onInfoWindowClose($event)" to sebm-google-map-info-window
onInfoWindowClose is just a method returning a console log message
The method does not get executed

Please find example in the plunker below:
http://plnkr.co/edit/RoOay1R4o1C8vxANFWNz?p=preview
Try clicking on a marker and then closing the info window.
angular2 & angular2-google-maps version

I met same issue, but nobody answers.

I met the same issue, waiting for solutions.

which version you had this installed?


Make invalid config file error more friendly

When the config file has some invalid fields or whatever the error i am getting is:
$ ./zebrad start
thread 'main' panicked at 'terminal streams not yet initialized!', /home/oxarbitrage/.cargo/registry/src/github.com-1ecc6299db9ec823/abscissa_core-0.5.2/src/terminal.rs:26:5
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
Aborted (core dumped)
$

I ran into this yesterday and it was caused by the Terminal component being constructed twice.

it's almost certainly an issue with the lifecycle on the Terminal component.

I think the problem is that we load the config before the Terminal, so when we run into an error the terminal isn't initialized.  This was because we wanted to be able to configure the Tracing component, when we were using the Abscissa component.  But now we have our own tracing component so we don't need configuration while we're loading the Abscissa components.
I think the fix is to move
        let config = command.process_config(config)?;
        self.config = Some(config);

in application.rs:init() to after
        // Create and register components with the application.
        // We do this first to calculate a proper dependency ordering before
        // application configuration is processed
        self.register_components(command)?;

Then we can also get rid of the self.config = Some(config) and the config.take() lines and just have
        self.register_components(command)?;

        let config = command.process_config(config)?;

        self.after_config(config)?;

Oops, this solution wasn't quite right.


Torrent list design

Removed hover / select brightness changes
All those gradients and brightness filters toggling on and off were getting annoying
Before

After

Progress bar styling
This fixes the inconsistency between different platforms.

Nice work!

LGTM


Thermostat continue its heating cycle whatever the setpoint is,  even when disabled.

Configuration

My VTherm attributes are the following:
hvac_modes:
  - heat
  - "off"
min_temp: 7
max_temp: 23
target_temp_step: 0.5
friendly_name: Thermostat SdB
supported_features: 401
current_temperature: 19.6
temperature: 7
hvac_action: heating
preset_modes:
  - none
  - frost
  - eco
  - comfort
  - boost
preset_mode: frost
is_on: true
hvac_mode: heat
type: null
is_controlled_by_central_mode: true
last_central_mode: null
frost_temp: 7
eco_temp: 18
boost_temp: 21
comfort_temp: 19
frost_away_temp: 0
eco_away_temp: 0
boost_away_temp: 0
comfort_away_temp: 0
power_temp: 15
target_temperature_step: 0.5
ext_current_temperature: 16.05
ac_mode: false
current_power: 400
current_power_max: 9000
saved_preset_mode: frost
saved_target_temp: 7
saved_hvac_mode: null
motion_sensor_entity_id: null
motion_state: null
power_sensor_entity_id: sensor.linky_puissance_inst_no_evse
max_power_sensor_entity_id: sensor.linky_pcoup_marge_evse
overpowering_state: false
presence_sensor_entity_id: null
presence_state: null
window_state: "off"
window_auto_state: "off"
window_bypass_state: false
window_sensor_entity_id: binary_sensor.ouverture_sdb
window_delay_sec: 30
window_auto_enabled: false
window_auto_open_threshold: 3
window_auto_close_threshold: 0
window_auto_max_duration: 30
window_action: window_frost_temp
security_delay_min: 30
security_min_on_percent: 0.1
security_default_on_percent: 0
last_temperature_datetime: "2024-10-11T14:44:19.356048+02:00"
last_ext_temperature_datetime: "2024-10-11T14:35:32.771625+02:00"
security_state: false
minimal_activation_delay_sec: 60
device_power: 560
mean_cycle_power: 0
total_energy: 71.86
last_update_datetime: "2024-10-11T14:49:12.189119+02:00"
timezone: Europe/Paris
temperature_unit: °C
is_device_active: true
ema_temp: 19.58
is_used_by_central_boiler: false
is_over_switch: true
is_inversed: false
keep_alive_sec: 0
underlying_switch_0: switch.switch_rad_sdb
underlying_switch_1: null
underlying_switch_2: null
underlying_switch_3: null
on_percent: 0
power_percent: 0
on_time_sec: 0
off_time_sec: 420
cycle_min: 7
function: tpi
tpi_coef_int: 0.6
tpi_coef_ext: 0.02

I got this problem 3 times with 3 different thermostat this week. ( I have only started to test Versatile Therm this week )
Sometime, the thermostat still continue the heating cycle and never stop.
When it's in this bugged state, I try to put the setpoint to a really low temperature, or set anti frost mode, the thermostat instantely cut off the heating. But a few mn later, I can see the heater is back on.
It's the same if I set it to OFF. Heating continue in cycling state to a previously setted temperature ( on the example below it was 19.5°C )
Here on the screenshot below you can see at the beginning it was already still maintaining a 19.5°C temp on the room with a 18°C setpoint on the thermostat. ( it was already in this bugged state ) .
At 13h20 I set the thermostat to anti-frost mode.
But the heating cycle continue whatever the temperature was set.

Disabling the integration and enabling it don't solve the problem.
But restarting Home assistant does.
Debug log
The related thermosatt is "thermosatt_sdb"
Complete log for the timeframe  here:
home-assistant_versatile_thermostat_2024-10-11T12-39-35.156Z.txt

As I'm in a testing phase for now, there's chance this issue wakes up after I've modifed some Versatile settings. But I can't confirm that yet. Perhaps a home assistant restart each time I change a parameter would avoid this.

The template switch works ok 99% of the time. I've double checked it first before using it.
But in bugued state , even with the thermostat disabled, the zigbee module still continue to receive ON/OFF commands from hass and regulates the temperature like you can see in the screen capture, but locked on a previous setted temp.
There's nothing else that can send switch commands to z2m than versatile on my setup.

However, I was using the normal switch from the pilot wire zigbee device before and got the same issue. I've changed to the template switch as I plan to mods my Lexman module with NodOn firmware ( those have no such switch internally but just climate commands )

I guess the change of parameter have created another VTherm leave the old running in background.
I can't even thing how it is possible due to HA config entry behavior, but it seems to be the case.
Is this issue still present after a restart ? Does this reproduce after each change of parameters ?

That has sense and is coherent with what I've observed. I don't know well enough HASS code structure yet, but I'd like to add some debug trace on the different Versatile functions to track down what's happening in the background.
Will take me a bit of time to be comfortable with your codebase first.

debug trace on the different Versatile functions to track down what's happening in the background

Be careful, the debug mode is very verbose. It could degrade you sd-card if you have one.
I transform this issue into conversation to have a follow up.


Update packages loader

See https://github.com/livebook-dev/livebook/pull/1560#discussion_r1038671759.
When updating to Livebook > 0.7.2 we need is to change this:
https://github.com/livebook-dev/nerves_livebook/blob/7904a83b2ab236c651635aeb45a866ea5687f590/lib/nerves_livebook/dependencies.ex#L48
to:
dependency: %{dep: {app, "~> #{version}"}, config: []},

Thanks for letting me know!!

Completed in #313.


Feature request: Add costs to chargings session

Hi, thank you very much for this great project!
One feature I am missing is keeping track of my charging costs. I am relying completely on public infrastructure and get charged per session or based on my consumption.
I would like to be able to annotate the automatically recorded charging sessions with the amount I was charged from my charging operator.
From my point of view only one more field per charging session for the total cost would be required in the database. Everything else (price per kWh, ...) can be calculated in the dashboards.
The interface to input the charging cost could probably be located in the same part of the project where also the geo-fence settings are located (port 4000 in default installation).
Have a great time!
toerb

Could I just add to this suggestion - I suspect what will take the most time here would be the UI to update the charging cost, however if we could put in a database field for cost for charging sessions into an upcoming schema change, those of us comfortable with updating it via the SQL interpreter could start logging these values until a UI solution exists. At the moment, I (and likely others) are recording the details elsewhere in anticipation that we might have an interface for it in the future.
At simplest, I would think it would just be a double precision field named cost. I've got a bit of spare time now, so I will see if I can submit a PR for this.


What's the use of annotation notation in webpack development environment [some annotation symbols are not real annotations]

Is it for segmentation, placeholder?


Fix undefined inside CSS code fence on /Source/ and /Download/

Fixes #1846.  A single tiny change:
	getInitialState : function() {
		const brew = this.props.brew;

		if(typeof window !== 'undefined') { //Load from localStorage if in client browser
			const brewStorage  = localStorage.getItem(BREWKEY);
			const styleStorage = localStorage.getItem(STYLEKEY);
			const metaStorage = JSON.parse(localStorage.getItem(METAKEY));

			if(!brew.text || !brew.style){
				brew.text = brew.text  || (brewStorage  ?? '');
				brew.style = brew.style || (styleStorage ?? '');   //  <--- changed this from undefined to '' (empty string)
				// brew.title = metaStorage?.title || this.state.brew.title;
				// brew.description = metaStorage?.description || this.state.brew.description;
				brew.renderer = metaStorage?.renderer || brew.renderer;
			}
		}

This does leave the code fence there, but empty.  I thought leaving the empty code fence would be helpful in showing that there is no additional code, rather than making it uncertain if it exists, but is missing or hidden.  However, possibly in the future there would exist an option to 'hide' the contents of your Style editor from the source?  Though I imagine that "closed source" brews would hide the entire source, rather than just portions.  That's a future discussion in any case.
I am not sure why this only needed a change on the /new/ page, but it works for the tests I did.  I know some unification is going to happen so am not too bothered about it unless someone says something.

I don't see that it actually changes anything as far as fixing #1846

So #1850 does this:

If style tab isn't touched from "new", the example styling doesn't appear, and no code fence block.
If it's changed, the change appears
If it's changed, but entirely deleted, it shows the code fence block but is empty:

This is already the behavior without this PR as far as I can tell. I think we might need to clarify with @ericscheid the steps to reproduce the issue because I can't reproduce it with or without this PR.

Closing, since the issue in #1846 was due to a different cause. Turns out this PR doesn't actually change the behavior of anything since a couple lines down we turn '' into undefined anyway.


Support for vsync

Is it intended that vsync is only supported for Direct3D backends, or is it just temporary? Looking at the source, nothing seems to be done in the other backends?
Thanks for the library.

No, was not the intent, Not sure about OpenGL, but this absolutely should work on Vulkan.

Haven't used Vulkan yet, but though it, like the other non-D3D backends, have code in the Present(Uint32 SyncInterval) implementation for the swap chain, none of them seem to make use of the SyncInterval parameter.
From some searching, it seems it should be possible for OGL, thought it varies a bit by platform, e.g. wglSwapIntervalEXT vs glXSwapIntervalEXT.

Not sure where to post this question - and since it is likely not a bug, but probably me misunderstanding something, I reuse this issue, since it is at least OpenGL related.
I am porting NanoVG to Diligent and have it working in D3D11 and D3D12. But, when I switch to OpenGL I got a problem with the appearance. The can replicate it in D11/D12 by changing the texture format for the render target to one without a stencil, so I am quite sure OpenGL is not properly using the stencil. Looking through Diligent's source I can see it does indeed call glEnable(STENCIL_TEST), but I cannot find where the stencil buffer is set, or how to enable it.

Hi! The stencil test as well as stencil operations are configured in the DepthStencilDesc of the pipeline state. The engine keeps track of the current stencil test state and enables it when needed. Stencil test should work fine in OpenGL because there are some test and also atmospheric light scattering post effect uses stencil a lot.

Double checked at the effect uses D24S8 format, so it should work fine. A potential difference could be that Diligent ignores viewport and scissor when clearing render targets, similar to Direct3D. glClear respects scissor, so this may somehow cause the difference.

@senevoldsen Did you figure out what was the problem?

@DiligentGraphics  No, not yet. Seems NanoVG does not use scissor. I tried outputting the stencil test state using glGetBooleanv direct and it is indeed enabled. I then tried querying the GL_STENCIL_BITS, but was not allowed to query that parameter.
I tried searching the samples and other repositories, and the only actual usage of stencil I could find was the Epipolar Light Scattering in the FX repository: https://github.com/DiligentGraphics/DiligentFX/blob/master/Postprocess/EpipolarLightScattering/interface/EpipolarLightScattering.h#L143 . But this is a "manually" allocated texture that is then the non-default target of a rendering operation https://github.com/DiligentGraphics/DiligentFX/blob/master/Postprocess/EpipolarLightScattering/src/EpipolarLightScattering.cpp#L995 . Whereas I am just using the default RTV and DSV. But I tried copying it and attaching it as the stencil target using SetRenderTargets to the context. But when I do I cannot clear the RTV or DSV since only the default targets can be cleared (at least in OpenGL).
I must admit I am still working on the assumption that the default DSV does not have stencil bits, at least in my case on Diligent's OpenGL backend. This assumption is strenghtened by the fact that if I apply the following  glfwWindowHint(GLFW_STENCIL_BITS, 0);, which requests zero stencil bits, to the original NanoVG demo example using GL3 before creating the window, I get the exact same issue, but when I remove the line, the issue is gone. Looking through Diligent's source I cannot find the actual place where the "raw" opengl stencil buffer is created/applied/requested or stencil bits requested.
At this stage, I am thinking of creating a tiny sample application using OpenGL and some stencil operation to see what happens.

Ah, I see, you are trying to create D24S8 depth buffer in the swap chain. You are right then, in GL backend the depth format is actually hardcoded to 32-bit without stencil. Will fix that:
https://github.com/DiligentGraphics/DiligentCore/issues/new

Sounds great; thank you.

@senevoldsen Try getting the latest Core, the stencil should work now. Note: the stencil will not yet work on Android

Had to change a the calls to resource commands to get it to compile since the interface changed to pointer, but excellent, it works great now :thumbsup:

OK, great! Yes, the API is still in flux little bit, and few more updates are coming.

Don't think a new issue is needed for this little thing: Uint32 Stride           = AutoOffset; think you mean AutoStride, this is in InputLayout.h - they are both the same though, but still.

Looks like a typo, will fix. Thanks for pointing out.


Make use of redirectTo property or method and finaly remove hardcoded '/home' path

Laravel Version: 5.4.17
PHP Version: 7.0.16
Database Driver & Version: mysqld 10.1.21-MariaDB

Description:
Current Laravel 5.4 documentation explains how to use Authentification with Path customization.

When a user is successfully authenticated, they will be redirected to the /home URI. You can customize the post-authentication redirect location by defining a redirectTo property on the LoginController, RegisterController, and ResetPasswordController:
protected $redirectTo = '/';

If you do like it says and define your home URI different than /home you have a problem.
The last place where this URI is left hardcoded is in the RedirectIfAuthenticated middleware.
As we can see from history, it was change from one location to another but none is going to be good while it is hardcoded.
        if (Auth::guard($guard)->check()) {
            return redirect('/home');
        }

Previous mentions of this issue: #17092, #15523, #9634
Steps To Reproduce:

Install Laravel Authentification
Set protected $redirectTo = '/dashboard'; in all Controllers LoginController, RegisterController, and ResetPasswordController.
Log in

Case Nr.1 - Reset password

Click browser back button
Click on forgot password link
Page not found '/home'

Case Nr.2 - Email with reset password information

Open email with password reset information
Click on Reset password link
Page not found '/home'

Case Nr.3 - Log in

Click browser back button
Click on Log in link
Page not found '/home'

Case Nr.4 - Register

Click browser back button
Click on Sign up link
Page not found '/home'

Possible solution:

Make it clear in documentation that you need to manually change the URI in RedirectIfAuthenticated middleware as well.
Make use of redirectTo property or method in RedirectIfAuthenticated middleware as it is currently written in documentation. 👍

The RedirectIfAuthenticated middleware is in App namespace. You have full control over it and can modify.
Since you are suggesting a change in the docs, you should create an issue (and possible PR) in the laravel/docs repo


Add NativeArrayExtension

Add NativeArrayExtension. A collection of extension methods to help developers work with NativeArray<T>s
Tag Alongs

Updated RandomExtension to match anvil extension class convention.
CopyFromSingletonBufferJob - Fixed spelling mistake in docs.

What is the current behaviour?
There are no existing extension methods provided by Anvil.
What is the new behaviour?
Developers now have access to FloodClear() and FloodSet() methods to clear and set the value of a range of elements in a NativeArray<T> efficiently.
Some quick tests show that this implementation is ~8x faster than using a MemsetNativeArray job.
Rough metrics on an array of 262,144 byte elements (ticks)
 - MemsetNativeArray.Schedule: 868
 - FloodSet: 103
 - FloodClear: 72

If needed, in the future, we could create a job that executes these more efficient methods so that multiple clears/sets could be done in parallel but we'll build that feature if/when the need comes up. You'd have to be setting values on many truly massive arrays at once to see benefits.
Discussion: Open to method naming suggestions. Flood* works but I'm not 💯  on it.
What issues does this resolve?
Convenience and performance in flood filling NativeArray<T>s
What PRs does this depend on?

None

Does this introduce a breaking change?

[ ] Yes
[x] No

Merging in. Will deal with namespace fixes in a separate PR.


too verbose output e.g. in install-all-components.bat

Verbosity fixed in
https://github.com/PX4/windows-toolchain/commit/f2bffa8b28562d6e3a10b34ce121b6670388230f
https://github.com/PX4/windows-toolchain/commit/3e559e39b41baee5c0f455e2cf1c3dfcd6b94c84
https://github.com/PX4/windows-toolchain/commit/0fa335a3101d54af829ba861c94f365ab598ca07
Current less verbose output of install-all-components.bat can be seen here:
https://ci.appveyor.com/project/Dronecode/windows-toolchain/builds/20512061


Ship with deployment installer script for Linux, and web publish profiles

Hello,
Even after 3 yrs of ASP-on-Linux, Linux Webhosts support for ASP sites is non-existant/minimal, For e.g. digital ocean has removed their .NET runtime scripts, as a result the adoption is very low.

In the release package a link to an installer for a develop to publish and run ASP sites on Linux hosts is desperately needed, in the pic. below you can see D.O. dropped support
Enhance documentation on support/configuration for linux

Thanks

Ping @DamianEdwards - any thoughts on this?

I asked around and we have some detailed docs on Linux installation scenarios here: https://docs.microsoft.com/en-us/dotnet/core/linux-prerequisites?tabs=netcore2x#supported-linux-versions
This includes lists of pre-reqs, as well as download links.

hi, after spending more than a weekend with digital ocean, host manits and site grund, on trying to see if these templates would work with any of them. They ran into new issues, the support guys were hesistant and not keen, since this was not widely supported and lacked more support articles around it.
Bigger issue was both a well tested secure image that was easy to deploy ASP core apps. I also noticed powershell was not their forte, but bash is not supported and they still had trouble.
End result its still not supported, microsoft's support for Linux is just half baked!!
Please ensure it runs on other native Linux clients easily.


Hook up Migration Card to API

http://homelessness.civicpdx.org/migration needs to be hooked up to the API

For the tree map, that data is simply not available in the API.
For the "12%" number, I don't think that is in the API either. The /migration endpoint lists percentages of various areas that people moved from, but they do not add up to anything near 12%.

The 12% comes from page 36 of PIT draft report 2015. Here's a screencap:

The percentages in the tree map come from page 37 of PIT draft report 2015:

These look correct in http://service.civicpdx.org/homeless/#!/migration/migration_list.

Ok I see, I was just thrown off by the decimal format of the percentages.
Should we make an endpoint for the "homeless when they came here" numbers? It would be cool to get both years in the app. That's also a pretty important number in context of the homelessness issue.

We could calculate the numbers and just round up/down accordingly and add those as a field to the existing table. It looks like those numbers refer to the 224 people in the 2015 count that were homeless when they came.

@outerpress What do you think of this? Obviously the margins need to be tweaked, but I'm trying to standardize the "half donut" chart so that the same one is being used across the site. It's not necessary to have the legend and "year link" show up, but if we fixed the styling is that something you'd want?

Done in #65


(Question) How to log anything that is logged to the console?

If I visit a URL, I want to be able to see what get's logged to the console and see any errors that might show up in javascript running on that page.
How can I log whatever gets logged to the console?
I've looked through the example wiki but I didn't find what I was looking for.

There are two domains for that: Log and Runtime (Runtime.consoleAPICalled); show me what you've tried.

Thx I just found this comment as well which might be helpful: https://github.com/ChromeDevTools/devtools-protocol/issues/5#issuecomment-332223303
Going to try them out and i'll post back here with my results.
As a beginner it's still little confusing to me what a "domain" actually is. From what I understand CDP() returns a client, which has these domains, but I'm looking for a more introductory write up that talks about what a client actually is.

I got what I wanted by following this example with a few modifications
https://github.com/cyrus-and/chrome-remote-interface/wiki/Log-network-requests
/ extract domains     const {Network, Page, Runtime} = client;     // setup handlers     Runtime.consoleAPICalled((entry)=>{         console.log("console api called: " + entry.args.map(arg=>arg.value).join(" "));     });     Promise.all([         Network.enable(),         Page.enable(),         Runtime.enable()     ])
...
When I included Log.entryAdded() it didn't seem to work. I expected to have duplicate copies of the outputs, one from Runtime.consoleAPICalled() and one from Log.entryAdded() but that wasn't the case, i only saw output from the consoleAPICalled() listener.
Also it's a little confusing to me what enable() actually does. In my experience to date I've been able to use Runtime.evaluate() without having enabled Runtime, so I'm assuming enabling is just for turning on the event listeners? The documentation is very concise (which is generally a good thing) but makes it a little harder for beginners to understand exactly what is happening.(https://chromedevtools.github.io/devtools-protocol/tot/Runtime#method-enable).

I expected to have duplicate copies of the outputs, one from Runtime.consoleAPICalled() and one from Log.entryAdded() but that wasn't the case, i only saw output from the consoleAPICalled() listener.

If I recall correctly Log.entryAdded() shows errors and warning from Chrome (e.g., XSS violations and such), not necessarily coming from the console.* API.

In my experience to date I've been able to use Runtime.evaluate() without having enabled Runtime, so I'm assuming enabling is just for turning on the event listeners?

Yeah, that's the usual case: you enable a domain if you want to receive events from it.

The documentation is very concise (which is generally a good thing) but makes it a little harder for beginners to understand exactly what is happening.

I totally agree with you, unfortunately I'm in no way affiliated with Google, for such kind of complaints/suggestions/questions the devtools-protocol repo is a better place.


Begin prototyping support for mentioning a specific user

I've not yet tested to determine if it works in our environment, but as noted on https://github.com/atc0005/go-teams-notify/issues/127, the following curl/JSON recipe can be used to target a specific user when posting to a Teams channel via webhook URL:
curl -X POST -H "Content-type: application/json" -d '{
    "type": "message",
    "text": "Hey <at>Some User</at> check out this message",
    "entities": [
        {
            "type":"mention",
            "mentioned":{
<EMAIL_ADDRESS>                "name":"Some User"
            },
            "text": "<at>Some User</at>"
        }
    ]
}' <webhook_url>

Need to start prototyping changes to this tool and the library to support that functionality.

For this JSON block:
{
    "type": "message",
    "text": "Hey <at>Some User</at> check out this message",
    "entities": [
        {
            "type":"mention",
            "mentioned":{
<EMAIL_ADDRESS>                "name":"Some User"
            },
            "text": "<at>Some User</at>"
        }
    ]
}

Is the <at>...</at> text needed, or is it just used for formatting? If it's needed, is it needed at both places?
e.g.,

"text": "Hey <at>Some User</at> check out this message",

the first text key/value

"text": "<at>Some User</at>"

the key/value for entities

When attempting to find the minimum required syntax, I found that failing to provide the <at>Some User portion of the message text value resulted in this response:
Webhook message delivery failed with error: Microsoft Teams endpoint returned HTTP error 400 with ContextId MS-CV=z4ufBXevF0iTilVdhweKBA.0..

The HTTP respone code still appears to be 200 however. For a successful submission (and mention of target user), a body response of 1 still occurs. This is when submitting a test message via webhook using the full JSON example noted in the OP.

Some potentially relevant official documentation:

https://docs.microsoft.com/en-us/answers/questions/493820/teams-incomming-webhook-mention.html
https://docs.microsoft.com/en-us/microsoftteams/platform/task-modules-and-cards/cards/cards-format?tabs=adaptive-md%2Cconnector-html#user-mention-in-incoming-webhook-with-adaptive-cards
https://docs.microsoft.com/en-us/powershell/module/azuread/get-azureaduser?view=azureadps-2.0

Example of an ID:

2b450b8e-1db6-42cb-a545-1b05eb8a358b

Example of a User Principal Name (UPN):

<EMAIL_ADDRESS>

Accepting (for now) that this is the minimal JSON payload required to produce a message that mentions a user:
{
    "type": "message",
    "text": "Hey <at>Some User</at> check out this message",
    "entities": [
        {
            "type":"mention",
            "mentioned":{
<EMAIL_ADDRESS>                "name":"Some User"
            },
            "text": "<at>Some User</at>"
        }
    ]
}

I can see how we would populate all values easily enough except for this one:
    "text": "Hey <at>Some User</at> check out this message",

If we accept a raw value directly from the user, do we parse the text and require that ...

 and  are present
that a user-specified placeholder string (e.g., PLACEHOLDER) is present

If the latter, we can search/replace against that placeholder and drop in the username (shown as name in the JSON payload) wrapped with  and  tags.
If the former, we leave the text as-is.
Perhaps the behavior can be configurable:

if a --mention-user-placeholder (example flag name) flag is specified, don't require that  and  are present if the --mention-user-name and --mention-user-id flags are used
if the --mention-user-name and --mention-user-id flags are used without specifying the --mention-user-placeholder flag, require that  and  are present in the user-specified message text string.

When attempting to find the minimum required syntax, I found that failing to provide the <at>Some User portion of the message text value resulted in this response:
Webhook message delivery failed with error: Microsoft Teams endpoint returned HTTP error 400 with ContextId MS-CV=z4ufBXevF0iTilVdhweKBA.0..

Per https://docs.microsoft.com/en-us/microsoftteams/platform/bots/how-to/conversations/channel-and-group-conversations?tabs=json#work-with-mentions:

The text field in the object in the entities array must match a portion of the message text field. If it does not, the mention is ignored.

Note to self:
Look back at https://docs.microsoft.com/en-us/microsoftteams/platform/bots/how-to/conversations/channel-and-group-conversations?tabs=typescript#work-with-mentions later for inspiration regarding handling the text field (duplication of content in entity and the standalone text field).

Accepting (for now) that this is the minimal JSON payload required to produce a message that mentions a user:

The go-teams-notify project currently implements much of the MessageCard format, but not yet the Adaptive Card format.
The JSON payload appears to follow the Adaptive Card format, but the entities element is not nested inside of a msteams element as expected. I also don't see either listed here: https://adaptivecards.io/explorer/

The JSON payload appears to follow the Adaptive Card format, but the entities element is not nested inside of a msteams element as expected. I also don't see either listed here: https://adaptivecards.io/explorer/

https://docs.microsoft.com/en-us/microsoftteams/platform/bots/how-to/conversations/channel-and-group-conversations?tabs=json#retrieve-mentions provides example JSON in that format (trimmed):
{
    "type": "message",
    "text": "Hey <at>Pranav Smith</at> check out this message",
    "entities": [
        {
            "type":"mention",
            "mentioned":{
                "id":"29:08q2j2o3jc09au90eucae",
                "name":"Pranav Smith"
            },
            "text": "<at>@Pranav Smith</at>"
        }
    ],
}

This doesn't appear to be in either of Adaptive Card or MessageCard format. Is this a third format?
Per https://docs.microsoft.com/en-us/microsoftteams/platform/task-modules-and-cards/cards/cards-reference#adaptive-card:

Adaptive, hero, list, Office 365 Connector, receipt, signin, and thumbnail cards and card collections are supported in bots for Microsoft Teams. They are based on cards defined by the Bot Framework, but Teams does not support all Bot Framework cards and has added some of its own.

Perhaps validation is lax and Teams isn't rejecting JSON payloads that don't completely meet the Adaptive Card format?

I'm going to prototype changes to the atc0005/go-teams-notify project involving the creation of a new "sub" package named botapi.

See also atc0005/go-teams-notify#149.
The prototyping work is surfacing enough limitations that I'm strongly leaning towards exposing the teamsClient directly (e.g., goteamsnotify.TeamsClient) instead of publishing versioned API (e.g., APIv2, APIv3) interfaces.

I'm going to prototype changes to the atc0005/go-teams-notify project involving the creation of a new "sub" package named botapi.

See also atc0005/go-teams-notify#149.
The prototyping work is surfacing enough limitations that I'm strongly leaning towards exposing the teamsClient directly (e.g., goteamsnotify.TeamsClient) instead of publishing versioned API (e.g., APIv2, APIv3) interfaces.

This is the direction I went.
Because the "botapi" package will be an incomplete representation of the actual bot api and because it doesn't support all of the features of the MessageCard format, in many ways using the bot api will be a step backwards. If someone opts to generate a user mention (message) using this tool they're implicitly opting out of several existing settings:

message title
message theme color
message trailer (app responsible for generating a message that send2teams delivers)

Granted, this is probably not a big deal; I am unsure which approach is better:

emit warnings when these options are specified alongside user mention(s)
use config validation to reject these flags paired with user mention(s)

I'd ordinarily think of the first approach in order to be backwards compatible, but using the user mention support would be an intentional choice and a good time for the app to surface incompatibilities between the settings.

This suggests that emitting warnings for a mix of incompatible settings would be a better approach as a near-term hotfix.

I'm actually leaning towards emitting errors for incompatible settings. Users will opt into using user mention support and it would be good to explicitly note what options do not work well together.

I think I've got the initial prototype where it needs to be for a test/dev release. Still rough edges and no doc updates yet to reflect the changes.
I also have some more cleanup work to do for the bundled local changes atc0005/go-teams-notify library before I push them upstream (for further polish/iteration). Hoping to get that done before week's end.

I also have some more cleanup work to do for the bundled local changes atc0005/go-teams-notify library before I push them upstream (for further polish/iteration). Hoping to get that done before week's end.

Latest changes have been pushed upstream and local prototype updated to use them.
Aiming for an alpha release of both the prototype of this CLI app and the atc0005/go-teams-notify library by week's end.

Aiming for an alpha release of both the prototype of this CLI app and the atc0005/go-teams-notify library by week's end.

Library update available here:
https://github.com/atc0005/go-teams-notify/releases/tag/v2.7.0-alpha.1


dfsrmig: Typo correction ("dsfrmig")

As pointed out in issue ticket #5459 (Typo: DFSRMIG not DSFRMIG), it should be 'dfsrmig', not "dsfrmig".
Thanks to WhetDawg for noticing and reporting the typo.
Proposed change:

correct the typo "dsfrmig" to 'dfsrmig'

Closes #5459

@illfated : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Property-level integration with reanimated

This PR makes it possible to use reanimated's shared values as properties of react-native-skia components which allows for updates or property animation to happen fully on the UI thread.
Here is what this PR changes:

We provide implementation for a previously empty JsiDomNode::setProp method – this method is used to update individual properties of a "Skia Dom Node"
We update Canvas.ts and Container.ts in order to be able to access view ID from container instance (via getNativeId method) – this is used because the prop updater needs the view ID to request redraw
We update HostConfig and inject reanimated specifi method for filtering shared values. We transform shared values into regular object and also register these shared value to monitor updates and call the prop updater function accordingly.
We add JsiRuntimeAwareStore and update JsiHostObject's function cache to use separate caches per runtime instance. This is necessary to allow host objects to be used in between two runtimes (jsi::Functions created with one runtime won't work on the second runtime). The role of JsiRuntimeAwareStore is also to control the secondary runtime lifecycle and cleanup methods cache when the runtime gets destroyed. Otherwise, the app crashes for example in case of a JS reload, because the functions kept past its runtime lifetime try to access the runtime;'s memory while being destroyed.
We update example app to now use new version of reanimated and add gesture-handler. We then use these two libraries in a new example section which is just a clone of the "Animations" sections with all the samples rewritten with reanimated and gesture-handler APIs.

TODO

[ ] Typescript types needs to be sorted out, currently using shared value as a prop in Skia component is treated as an error, because the property types does not accept a shared value type. We need to find a way to alter that type without listing react-native-reanimated package as dependency
[ ] This change should be tested with web version, AFAIK web bundlers tend to not like conditional require statements that we use in order to not have a hard dependency on react-native-reanimated package.
[ ] This change requires one change in reanimated which is not yet included in a public release: https://github.com/software-mansion/react-native-reanimated/pull/4024

This is huge.
I haven't done a deep dive yet but the diff looks super clean.
This is how we do the reanimated "detection" currently: https://github.com/Shopify/react-native-skia/blob/main/package/src/external/reanimated/useSharedValueEffect.ts#L4
And indeed this makes things like web support quite messy but it's a problem we have already anyways so it's a bit of a separate discussion I think.
I couldn't get the PR to run, after the build, I'm getting the following error. This may be expected because the code in the reconciler seems to have unresolved variables?

Thanks, I pushed these changes to https://github.com/Shopify/react-native-skia/tree/feature/reanimated-integration – will have to close this PR because it isn't possible to update the source branch and will open new one from that new branch

Here it is: https://github.com/Shopify/react-native-skia/pull/1341


DocsArchive-1.0.20272.1Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/jesong/docs-archive-test-source-archive/pull/1 is merged into live branch.
Auto archive content to https://github.com/jesong/docs-archive-test-source-archive.git

Docs Build status updates of commit f9ca7d2:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.redirection.json
:white_check_mark:Succeeded

versioningArchiveSourceFolder/TOC.yml
:white_check_mark:Succeeded
View (sql-server-2014 || azuresqldb-current || azuresqldb-mi-current || azure-sqldw-latest || >= aps-pdw-2016 || >= sql-server-2016 || >= sql-server-linux-2017 || sql-server-previousversions || sqlallproducts-allversions)

versioningArchiveSourceFolder/docfx.json
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Fix custom on runtime upgrade not being called

When the Executive was used through the ExecuteBlock trait, the
custom on runtime upgrade wasn't called. This happened because we forgot
to forward the type and it instead used default type () that doesn't
do anything. This pr fixes it by forwarding the type and also adds a
regression test.

bot merge


npm ci on Windows with Node 8

This isn't a problem with the code, but the Action example I think might have a fault.
It uses a matrix, and windows + Node 8 I think might not have ci yet, because the run fails:
Run npm ci && npm t

Usage: npm <command>

where <command> is one of:
    access, adduser, bin, bugs, c, cache, completion, config,
    ddp, dedupe, deprecate, dist-tag, docs, doctor, edit,
    explore, get, help, help-search, i, init, install,
    install-test, it, link, list, ln, login, logout, ls,
    outdated, owner, pack, ping, prefix, profile, prune,
    publish, rb, rebuild, repo, restart, root, run, run-script,
    s, se, search, set, shrinkwrap, star, stars, start, stop, t,
    team, test, token, tst, un, uninstall, unpublish, unstar,
    up, update, v, version, view, whoami

npm <command> -h     quick help on <command>
npm -l           display full usage info
npm help <term>  search for help on <term>
npm help npm     involved overview

Specify configs in the ini-formatted file:
    C:\Users\runneradmin\.npmrc
or on the command line via: npm <command> --key value
Config info can be viewed via: npm help config

<EMAIL_ADDRESS>C:\hostedtoolcache\windows\node\8.10.0\x64\node_modules\npm

Settled on the following:
        run: npm i -g npm && npm ci && npm t

Can create PR with a doc fix.

Another tip:
        run: npx semantic-release@latest

The @latest release has a fix for using standard GITHUB_TOKEN.
So this is enough, and don't need to create a personal token:
          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
          NPM_TOKEN: ${{ secrets.NPM_TOKEN }}
          NODE_AUTH_TOKEN: ${{ secrets.NPM_TOKEN }}

But, @latest if I understand correctly, is alpha, so might not be too stable.

And thanks for the package, highly appreciated and saved me from so much tinkering!!

Info re @latest: https://github.com/semantic-release/semantic-release/issues/974#issuecomment-525362404


[NETBEANS-1074] maven.jaxws Module Review 2nd Donation

Changed licenses in two files
Added licenseinfo.xml for graphic files

Thanks!


Update in-case-of-crisis-mobile-tutorial.md

Changed Raw Certificate to Base64 Certificate

#sign-off

Am I not the assigned author? Please advise.

Hi @rdsgit,
@jeevansd is the assigned author.

sign-off

@jeevansd , could you approve this merge request?  Thank you.

How do I get this review processed?  Please advise if I'm missing a step here.  This is taking too long to approve.  Thank you.

@rdsgit, we're creating a separate PR for this in our private repo for better content review, the changes will merge within 2-3 working days.
#please-close


[New game]: Sound Wordle Game

🎮 Game Request
In this game, a tune will be played, and the user has to guess the name of the song in 5 guesses.
As stated before, the issue #4406 is the Word Shuffle Game. My game is different. It uses audio and is not an unscrambling of words, it is more of a guessing game. Like a wordle but using audio.
Point down the features

The user will be given 5 chances.
The tune can be played as many times as the user wants.
If the letter is used in the word, it will become yellow.
If the letter is in the right place in the word, it will become green.

Select program in which you are contributing
GSSoC24
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @abckhush !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

/assign

Hey @abckhush ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇


IE broken since beta.2

Use of window.location.origin instead of window.location.host in popup.service breaks IE

Try beta.9

fixed, thanks!


ansible: Update docker role

This two commits adds support for Fedora25, Trusty and Yakkety in addition to the current supported Fedora24 and Xenial.
Fixes #1110

Coverage remained the same at 66.754% when pulling 7935f3586a966fc1b33b2866ca648b77439770eb on albertomurillo:docker into 7c6525a966fb194d40508cfad04757d1f52191ed on 01org:master.

@rbradford Tested and working on ClearLinux, Fedora24, Fedora25, Xenial and Yakkety.
I removed Trusty because we need docker-py to control docker with ansible and docker-py is not available on Trusty. Also I don't think we are aiming to Trusty for ciao

Coverage decreased (-0.009%) to 66.735% when pulling a50ed01307ccf40e347113faad0a6a81f5c7ece0 on albertomurillo:docker into 0423d82376ac08f1d90aca770d3bb7079b077c7d on 01org:master.


Srsi macd

Renamed and tweaked a bit the strategy and a small fix to the SMA file.

Besides the typo, you could also rename the strategy name in backtester.js, too.
Just rename all rsi_macd to srsi_macd:
sed -i 's/rsi_macd/srsi_macd/g' backtester.js

https://github.com/carlos8f/zenbot/blob/master/scripts/auto_backtester/backtester.js#L75

can i fix the typo and re commit the pr? how can i fix it?
this happens a lot... screw the length
On Sat, Jul 8, 2017, 2:33 PM fasheng<EMAIL_ADDRESS>wrote:

Besides the typo, you could also rename the strategy name in
backtester.js, too.
Just rename all rsi_macd to srsi_macd:
sed -i 's/rsi_macd/srsi_macd/g' backtester.js
https://github.com/carlos8f/zenbot/blob/master/scripts/auto_backtester/backtester.js#L75
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/carlos8f/zenbot/pull/369#issuecomment-313856176, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ALhf8mK8uEDCY5krSWuu2Asmyk1Phb6Fks5sL4UWgaJpZM4ORufw
.

And you must edit lib/_codemap.js to export sma function.


"GOTO" for collaborators?

Not sure if this has been discussed - it's not an open issue that I found.
Will there ever be a way to have some way to go to a collaborator cursor? I've found if I open atom-pair in a bigger project, I have no idea where my collaborators are typing. This seems to really restrict what I'm comfortable doing with atom-pair.
Some way to go to where their cursors are, or see what they last did - maybe with a background color on text that you didn't type, during the pairing session?

Thanks for the suggestion!


Better Polish language date shortcuts translation and default date format

I would like to propose better from Polish grammatical point of view language translation for weekdays, months and their shortcuts.
Added default date format.

Thanks @crabonature!


FFI should enable syncing a user's sphere with a gateway

As of #131 , we have a basic FFI for initializing a sphere. In #133 we will extend support to modify the sphere. We should then further extend the FFI capabilities to include syncing to a gateway.

This was completed in #210


Add useful documentation to functions.

This issue can be fleshed out to help keep track of what documentation is needed still.


How to make some routes API's public?

I want to make part of my API's, some routes without authentication and some using simplejwt. How this is possible?

This is not manage by simplejwt, this is manage by django rest framework(drf), did you read this?
in your drf you can protect all by add this in your settings.py

REST_FRAMEWORK = {
    'DEFAULT_PERMISSION_CLASSES': [
        'rest_framework.permissions.IsAuthenticated',
    ]
}

and in your viewset that want not protected (views.py)

from rest_framework.permissions import AllowAny

class ExampleViewSet(viewsets.ModelViewSet):
    permission_classes = [AllowAny]

or by action(example POST to /v1/example/):

from rest_framework.permissions import AllowAny, IsAuthenticated

class ExampleViewSet(viewsets.ModelViewSet):
    permission_classes = [IsAuthenticated]
    def get_permissions(self):
        if self.action == 'create':
            self.permission_classes = [AllowAny]


How to use JWT?

I cannot see jwt in New docs ,I need to implement.

Hey @ibalaji777! 👋
This is more a question than an issue with the framework itself.
Since we want to keep this space clean, please use GitHub Discussion or the Discord Server for this kind of request.


MIL-5: Milestone 1 Group Review

What's up?
In this report, the work we have done so far needs to be reflected. The report should be prepared based on the content of our repository and observations. It will give us an  opportunity to examine the work we have done and the processes & tools we have chosen. Also it is a document for the assessment  & improvement of our project.
To Do

[x] Executive Summary: Summary of the project status and any changes that are planned for moving forward.
[x] List and status of deliverables.
[x] Evaluation of the status of deliverables and its impact on your project plan.
[x] Evaluation of tools and processes you have used to manage your team project.
[x] The requirements addressed in this milestone.
[x] Overall description of responsibilities that are assigned to each member.
[x] A Summary of work performed by each team member (in tabular format).

Deadline
04.11.2022 @22.55
Additional Information
This report requires the effort of all our group members.
Reviewers
@KarahanS @MustafaEmreErengul

All related sub-issues are closed, closing the issue.


Add RegQueryInfoKey

Can be useful for getting various information about keys.

What would you expect the return type to be? It looks like only actual non-performance related information (that doesn't seem relevant given the current package API) that could be returned is the modified time and the user defined class. Since the current package API doesn't let you set or update the class, and AFAICT it's a weird leftover relic that isn't actually used, I'm not too worried about exposing that. I guess the modified time is potentially useful?

What would you expect the return type to be? It looks like only actual non-performance related information (that doesn't seem relevant given the current package API) that could be returned is the modified time and the user defined class. Since the current package API doesn't let you set or update the class, and AFAICT it's a weird leftover relic that isn't actually used, I'm not too worried about exposing that. I guess the modified time is potentially useful?

For my use case I needed lpcSubKeys - the number of subkeys of a given key. I assume that lpcValues and, as you said, lpftLastWriteTime can also be useful.

For my use case I needed lpcSubKeys - the number of subkeys of a given key. I assume that lpcValues and, as you said, lpftLastWriteTime can also be useful.

Interesting, I wouldn't have thought the number of subkeys or values would be valuable except for allocating the array? I guess if you're doing something a like tree-view you would prefer to not need to actually query the subkeys or values, though I'm not sure if there would be much gain after the JS<->C++ overhead, which I understand to be pretty high.
Worth a try at least!

Interesting, I wouldn't have thought the number of subkeys or values would be valuable except for allocating the array? I guess if you're doing something a like tree-view you would prefer to not need to actually query the subkeys or values, though I'm not sure if there would be much gain after the JS<->C++ overhead, which I understand to be pretty high.
Worth a try at least!

tree-view

Exactly :)

JS<->C++ overhead

I'm not sure about that, but at least the complexity can be O(#subkeys) and not O(#subkeys-of-subkeys) (by calling enumKeyNames for each subkey, the only possible solution with the current API).

tree-view

Exactly :)

JS<->C++ overhead

I'm not sure about that, but at least the complexity can be O(#subkeys) and not O(#subkeys-of-subkeys) (by calling enumKeyNames for each subkey, the only possible solution with the current API).

Well you only need to call reg.enum*() once for each key, like you would need to call reg.queryInfoKey() once for each key, so it would be only the internal enum overhead, plus whatever GC pressure the extra discarded strings give.
I'm not really against it though, but I don't really want to either get too far away from the underlying API (it is called native-reg, afterall) but also it would be weird to include things like the size of the security descriptor if you can't read it. I'll try some things after my break, and see what works out.

Well you only need to call reg.enum*() once for each key, like you would need to call reg.queryInfoKey() once for each key, so it would be only the internal enum overhead, plus whatever GC pressure the extra discarded strings give.
I'm not really against it though, but I don't really want to either get too far away from the underlying API (it is called native-reg, afterall) but also it would be weird to include things like the size of the security descriptor if you can't read it. I'll try some things after my break, and see what works out.

I was mainly concerned about the internal enum overhead, since the amount of subkeys-of-subkeys can possibly be huge.
In any case, it's just a nice-to-have missing feature I stumbled upon, and I agree it's a bit specific. Let's close it for now, and perhaps revisit it if other users show interest in the future.

I was mainly concerned about the internal enum overhead, since the amount of subkeys-of-subkeys can possibly be huge.
In any case, it's just a nice-to-have missing feature I stumbled upon, and I agree it's a bit specific. Let's close it for now, and perhaps revisit it if other users show interest in the future.

I guess if you're doing something a like tree-view

Here's the toy I was working on, in case you were wondering:
https://marketplace.visualstudio.com/items?itemName=m417z.regedit

I guess if you're doing something a like tree-view

Here's the toy I was working on, in case you were wondering:
https://marketplace.visualstudio.com/items?itemName=m417z.regedit


Update dependencies and rework some tests

Signed-off-by: Maciej Szlosarczyk<EMAIL_ADDRESS>

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please ask for approval from maciej-szlosarczyk. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: ad3d009e7006e290062f985398d360a51127fddb

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: antonsatin
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [antonsatin]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Better foot plant

Makes foot plant work when foot trackers are being used. It's based on https://github.com/SlimeVR/SlimeVR-Server/pull/647

Ready for review & merge!


Introduce flow/packet target, local and netflow targets

other that being able to report NetFlowV5 from agent, you also may want to use NetFlowV5 as an alternative for reporting flows from the agents to analyzer, by that also solving limited network bw (by implementing sampling capabilities of NetFlowV5).

@hunchback from my point of view, expire/update of flow is already a sampling mechanism.

@hunchback Quoted from https://stackoverflow.com/questions/41087206/on-github-whats-the-difference-between-reviewer-and-assignee :

According to official github documentation, Assignee is a person who is working on specific issues and pull requests. It is sometimes confused as a reviewer. It is actually meant to be used with issues rather than pull request so that when we receive a issue we can assign someone to fix it.

run skydive-functional-tests-backend-orientdb

run skydive-functional-tests-backend-elasticsearch

run skydive-scale-tests
run skydive-cdd-overview-tests


EventSpy CANNOT handle custom object

The sencario is coding as follow:
it("when trigger a jQuery custom event", function () {
  var obj = {}
  var spyEvent = spyOnEvent(obj, 'customEvent')

  $(obj).on('customEvent', function (event, data) {
    expect(data.msg).toEqual("It's OK")
    // It works.
  })

  $(obj).trigger('customEvent', {msg: "It's OK"})

  expect('customEvent').toHaveBeenTriggeredOn(obj)
  // Expected event customEvent to have been triggered on [object Object]

  expect(spyEvent).toHaveBeenTriggered()
  // Expected event customEvent to have been triggered on [object Object]

})

So, could you tell me what the correct method with handling event form custom object?

Could anyone prove this suite?

Yes, I see the same issue, with your suite.


Stop writing logs to filesystem

It annoys me that every time i run my application, even without invoking DSPY, i get assertion.log and openai_usage.log writte to filessytem. This is totally unnecessary. Thank you.

Hi @j4k0bk , setting dspy.settings(log_openai_usage=False) should avoid this. However, it's probably a good idea to enable an enable/disable flag in settings for all .log files going forward, since these are getting created upon importing dspy.

Also, if you are trying to import dspy in a Read-only file system (i.e. AWS Lambda) you wouldn't even be able to import it
  File "/function/.deps/dspy_ai-2.4.0-py3-none-any.whl/dspy/__init__.py", line 1, in <module>
    import dsp
  File "/function/.deps/dspy_ai-2.4.0-py3-none-any.whl/dsp/__init__.py", line 1, in <module>
    from .modules import *
  File "/function/.deps/dspy_ai-2.4.0-py3-none-any.whl/dsp/modules/__init__.py", line 1, in <module>
    from .azure_openai import AzureOpenAI
  File "/function/.deps/dspy_ai-2.4.0-py3-none-any.whl/dsp/modules/azure_openai.py", line 7, in <module>
    handlers=[logging.FileHandler("azure_openai_usage.log")],
  File "/usr/local/lib/python3.11/logging/__init__.py", line 1181, in __init__
    StreamHandler.__init__(self, self._open())
  File "/usr/local/lib/python3.11/logging/__init__.py", line 1213, in _open
    return open_func(self.baseFilename, self.mode,

I agree here, we shouldn't be writing those files unless they're required

This also should be fixed with #684
Logging is set up globally, with config for output type, path, and log level.


[Android] Fix Crosswalk failed to build debug version.

Because isLoading api doesn't exist in |m_frame|.

@rakuco , @kenchris would you help to review the patch? thanks.

This is broken upstream as well. I've left a comment in https://codereview.chromium.org/904113002

@rakuco , This PR doesn't need on master now, It's fixed by upstream. Would you help to revert it for master branch? otherwise, it would cause build failed on debug version.
Thanks.

Don't worry, this has been taken care of automatically by the rebase process.


Add the missing colored insect hotel parts as wavefront obj format models

Models are designed with SolidWorks and color structure is added using Blender.
The origin is shifted to the center of the surface.

Perfect, thanks!


ENH: added metadata flag to contain maximum deviation in series reader

ENH: aded metadata tag for maximum deviation from expected slice spacing in ImageSeriesReader.
@seanm

PR Checklist

[ ] :no_entry_sign: Makes breaking changes
[ ] :no_entry_sign: Makes design changes
[ ] :no_entry_sign: Adds the License notice to new files.
[ ] :no_entry_sign: Adds Python wrapping.
[X] :no_entry_sign: Adds tests and baseline comparison (quantitative).
[ ] :no_entry_sign: Adds test data.
[ ] :no_entry_sign: Adds Examples to ITKExamples
[ ] :no_entry_sign: Adds Documentation.

Refer to the ITK Software Guide for
further development details if necessary.

Maybe we should also get rid of the boolean version we recently added with https://github.com/InsightSoftwareConsortium/ITK/pull/1391 ?  Math::AlmostEquals is so strict that I suspect all clients will want/need this newer way.


Bring escape-html into labs/ssr?

I've been experimenting with getting LitElements to render in Deno and am mostly able to get it to work (importing with Skypack URLs). The only blocker as far as I can tell is the usage of the Node.js 'module' module for the sake of requiring 'escape-html'.
I can understand not wanting to take on the maintenance burden of officially supporting Deno, but you could pretty easily unofficially support it just by bringing escape-html into the codebase. It's an old module that hasn't been updated in a long time and is a single small function which is unlikely to need changes in the future, so I think it's a good candidate for vendoring.
Happy to send a PR if you're agreeable.

This would likely help solve https://github.com/lit/lit/issues/1983 as well.


Updated Add- and Remove-eDiscoveryCaseAdmin topics per PM and support escalation

Docs Build status updates of commit 75ea4c7:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/Add-eDiscoveryCaseAdmin.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Remove-eDiscoveryCaseAdmin.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


pipelinewise-target-postgres fails with ModuleNotFoundError: No module named 'azure.storage.common'

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/2024
Originally created by @DouweM on 2020-05-14 21:56:54

As reported by Zafar Toshpulatov on Slack:

Hello everyone,
I am trying to add a new loader "pipelinewise-target-snowflake" to Meltano (this target is not known to Meltano).
I installed pipelinewise-target-snowflake with the following command:
meltano add loader pipelinewise-target-snowflake
Meltano created virtual environment for target and I configured it as suggested. But when I run etl process like:
meltano elt mytap target-snowflake --job_id=mytap-to-snowflake

I am getting the following error from target:
ModuleNotFoundError: No module named 'azure.storage.common'
It doesn't see the module. I checked the virtual env directory of target .meltano/loaders/target-snowflake/venv/lib/python3.6/site-packages/ that module exists.
Before I used this target in standard singer commands and never had a problem. Is there problem between Meltano and Azure?
Is it possible to create manually virtual env and install target, without using Meltano? If possible, does it communicate with Meltano?

View 2 previous comments from the original issue on GitLab


✨ feat: supported actor-likes command (#700)

1. Description

1.1. Checklist

[ ] The title of my PR starts with a Conventional Commit prefix (fix:, feat:, docs: etc).
[ ] I have read the Contributor Guide and followed the process outlined for submitting PRs.
[ ] I have updated/added tests for ALL new/updated/fixed functionality.
[ ] I have updated/added relevant documentation in docs and added dartdoc comments with ///.
[ ] I have updated/added relevant examples in examples.

1.2. Breaking Change

[ ] Yes, this is a breaking change.
[ ] No, this is not a breaking change.

1.3. Related Issues

Codecov Report

Merging #701 (d165614) into main (d2734ea) will increase coverage by 6.75%.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##             main     #701      +/-   ##
==========================================
+ Coverage   80.50%   87.26%   +6.75%
==========================================
  Files         225        7     -218
  Lines        2729      157    -2572
==========================================
- Hits         2197      137    -2060
+ Misses        532       20     -512

see 218 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


With HMR-hot module reloads when I use global: true I get You've already defined a global store is there any way to suppress this

I opened a PR to fix the issue PR: https://github.com/zalmoxisus/mobx-remotedev/pull/45

Thanks for the contribution! It's in 0.3.6.


GpuParquetScan fails with NullPointerException during combining

fixes https://github.com/NVIDIA/spark-rapids/issues/7688
The underlying bug here happens with the combining multithreaded reader when the ignoreMissingFiles in Spark is used and the first file read by the combining reader was missing.   Essentially when we ignore missing files, the metadata comes out to be empty (read schema is null) and if that mixes in with metadata from blocks where we just do row counts() and the ignore missing files one was the first one we hit when combining files, we use that bad metadata to our combined results when we should have used the metadata from one of the blocks with row counts.  Here the fix is to just make sure to use the metadata from one of the blocks with row counts.  This only happens when we have mixed ignored files with files where we just get row count.  If all ignored it takes a different code path.
Test for this was kind of difficult due to the timing. To make Spark ignore missing files you have to first do the spark.read.parquet to get the metadata when all the files are there, and then remove the file before you do an action on that dataframe.

this was supposed to be against 23.04, updated it


Broken for fish 2.7.0

After update fish shell to latest version 2.7.0 I noticed the fisher command stop working.
This happens in 10 different servers running Centos 7 and 2 machine under Ubuntu 16.04
❯ fisher up2m 17s 746ms
fish: argparse: command not found...
/usr/share/fish/functions/math.fish (line 7):
    argparse -n math --stop-nonopt --min-args=1 $options -- $argv
    ^
in function “math”
	called on line 2062 of file ~/.config/fish/functions/fisher.fish
	with parameter list “1511450765784 - 0”

in function “__fisher_get_epoch_in_ms”
	called on line 17 of file ~/.config/fish/functions/fisher.fish

in command substitution
	called on line 15 of file ~/.config/fish/functions/fisher.fish

in function “fisher”
	called on standard input
	with parameter list “up”

fish: argparse: command not found...
/usr/share/fish/functions/isatty.fish (line 2):
    argparse -n isatty $options -- $argv
    ^
in function “isatty”
	called on line 279 of file ~/.config/fish/functions/fisher.fish

in function “fisher”
	called on standard input
	with parameter list “up”

Hmm maybe something changed in isatty or else?

Yep I get
~/.config/fish/functions/fish_prompt.fish (line 240): function: Unknown option '-q'
~/.config/fish/functions/fish_prompt.fish (line 240):
function type -q -a name -d "Check if a function or program is type -q."
^
from sourcing file ~/.config/fish/functions/fish_prompt.fish
	called on standard input

in command substitution
	called on standard input

       functionfunction - create a function
        -

   Synopsis
       function NAME [OPTIONS]; BODY; end

function: Type 'help function' for related documentation

function type -q -a name -d "Check if a function or program is type -q."
  type "$name" ^/dev/null >&2
^
from sourcing file ~/.config/fish/functions/fish_prompt.fish
	called on standard input

in command substitution
	called on standard input

I haven't upgraded to fish 2.7, so I guess that's why I haven't bumped into this. Do you have a solution? Because I don't imagine this must be so hard to fix.
Hi @faho, do you know if isatty changed or what may be the cause of this error?

Hi @faho, do you know if isatty changed or what may be the cause of this error?

isatty now uses the new argparse builtin to parse its arguments - it's basically our answer to getopt. A number of functions have been changed to use it.
This looks like the old fish (2.6.0?) is picking up the new functions.
@agarzon, @askl56: This should just solve itself once you close the old sessions or e.g. exec fish.

Thanks @faho the exec fish worked for me. Peace !

Just updated my ArchLinux system, got 2.7.0, rebooted my computer, but no luck, same error from @askl56 .
exec fish makes no diff.
Any other suggestions? Thanks!

@faho Perhaps there is something else? I am going through: https://github.com/fish-shell/fish-shell/releases/tag/2.7.0 but no luck! 🤔

@rdlu: You're getting the "fish: argparse: command not found..." message, right?
That's basically impossible with fish 2.7.0 since it's a builtin. And I can confirm that Arch hasn't e.g. patched it out or something since the official arch package works for me.
So what I'm assuming is that you have an old fish somewhere else in your $PATH that is started. Try fish --version - if that doesn't print "fish, version 2.7.0" you have some cruft you need to remove (see command -s fish).

Oh, wait, you said the same as @askl56? I.e. function: Unknown option '-q' and function type -q -a name -d "Check if a function or program is type -q."?
In that case, check your ~/.config/fish/functions/fish_prompt.fish - there should be some broken syntax in there. 2.7.0 tightened the function builtin against wrong options, and it seems something is setting a function with a wrong option.

Well, I have to mention that I did nothing, and suddenly fisher is working back in all servers, no update, command or logout was executed.

Still an issue here.

I haven't amended or changed fish_prompt at all from the default so confused as to why others are seeing different behaviour.

@askl56 I still haven't upgraded to fish 2.7, if you find a solution a PR is welcome.

@askl56 Can't you just debug it? fisherman is literally one file, trace the problem to the function that's causing the problem.  The solution must be like 1 byte or something.

> fisher ls
find: ^: unknown primary or operator

I get this error with fish, version 2.7.1-1096-ga9e9af5c

I'm running fish 2.7.1 with fisherman 2.13.2 without issues.

Fish 2.7.1-1096-ga9e9af5c is a nightly build

@Markcial thanks for pointing that out, after reinstalling fish 2.7.1 (stable via homebrew) fisher now works again.
Confirmed working on 2.7.1 (stable)


modules_linux: Add support for builtin kernel modules

Previously, kernel modules names were read from the /proc/modules only which prevented k8s linux distribution with kernel modules builtin into the kernel to pass the validation of required kernel modules by Cilium.
This patch adds support for systems with builtin modules by trying to read the kernel module names from following files additionally:

/lib/modules/<kernel-version>/modules.builtin
/usr/lib/modules/<kernel-version>/modules.builtin
/usr/lib/debug/lib/modules/<kernel-version>/modules.builtin

Fixes: #23863
Add support for builtin kernel modules

Please ensure your pull request adheres to the following guidelines:

[x] For first time contributors, read Submitting a pull request
[ ] All code is covered by unit and/or runtime tests where feasible
Unable to test on my current machine
[x] All commits contain a well written commit description including a title,
description and a Fixes: #XXX line if the commit addresses a particular
GitHub issue.
[x] If your commit description contains a Fixes: <commit-id> tag, then
please add the commit author[s] as reviewer[s] to this issue.
[x] All commits are signed off. See the section Developer’s Certificate of Origin
[x] Provide a title or release-note blurb suitable for the release notes.
[x] Thanks for contributing!

I mistakenly made it as draft. Please publish the PR. :)

Should I rebase? The one of the pipelines is failing (perhaps due to 0.6 gateway api upgrade).

Yes, please. The ConformanceGatewayAPI failure doesn't look like a flake and seems fixed on master.

/test

@gentoo-root

IMO, it's more reliable to query subdirectories of /sys/module

I thought that also. The problem with /sys/module is that only appears when at least has a version or at least one parameter and it is not reliable source due to The conditions of creation in the built-in case are not by design and may be removed in the future.
Since the /lib/modules is getting already mounted due to cilium's btf (findVMLinux function) - I see no problem here.

@TheAifam5 could you please squash the last commit into the first? Since it's not merged yet we can just have the
correct code in the original commit.

@NikAleksandrov Understood. I will do it today, around 19-23:00 GMT+1.

/test

Should I rebase it again?

Should I rebase it again?

No. In general it's best not to rebase unless 1) you're going to push a change anyway, or 2) there is a merge conflict, or 3) a maintainer asks you to. When rebasing all tests need to be restarted.
I'll triage CI results now and will rerun or mark ready to merge accordingly.

Thanks for fixing this @TheAifam5!

This will be released in v1.14.0. I've added the backport label given that it fits the backporting criteria and shouldn't cause backport conflicts (that code is rarely touched). So unless the backporter hits issues, it should also be in the next v1.13 release.

Awesome, thanks!

Any idea when there will be another 1.13 release or the 1.14 release?


Memory leak when training the network

Dear Authors,
I tried to train the network, but the memory usage on GPU keeps increasing with every iteration.
My specs:
torch: 1.4.0
torchvision: 0.5.0
cuda: 10.1
Steps to reproduce:

Pull the repo
Changed the data path in the config to my local dataset
Download pretrained backbone from voxelpose repository
set GPUS: '0' in config
Data preprocessing runs (resizing images)
Run the training: python run/train.py --cfg configs/panoptic/jln64.yaml

I tried to pinpoint the exact spot where the memory leak occurs, but it seems like it happens in different places in the network. The major increase seem to happen:

i the resnet backbone, when 1st pass out of 5 goes through self.layer1
after that a small one in encoder-decoder part of the pose-net.

I use GPUtil for checking, the memory, and put it in the following places:
`        if views is not None:
input_heatmaps = torch.stack([self.backbone(view) for view in views], dim=0)
else:
input_heatmaps = torch.stack(input_heatmaps, dim=0)
print("after self.backbone", GPUtil.getGPUs()[0].memoryUsed)
batch_size = input_heatmaps[0].shape[0]
    # human detection network
    proposal_heatmaps_2d, proposal_heatmaps_1d, proposal_centers, \
                          bbox_preds = self.pose_net(input_heatmaps, meta)
    print("after self.pose_net", GPUtil.getGPUs()[0].memoryUsed)
    mask = (proposal_centers[:, :, 3] >= 0)

    # joint localization network
    fused_poses, poses = self.joint_net(meta, input_heatmaps, proposal_centers.detach(), mask)
    print("after self.joint_net", GPUtil.getGPUs()[0].memoryUsed)
    # compute the training loss
    if self.training:
        assert targets is not None, 'proposal ground truth not set'
        proposal2gt = proposal_centers[:, :, 3]
        proposal2gt = torch.where(proposal2gt >= 0, proposal2gt, torch.zeros_like(proposal2gt))

        # compute 2d loss of proposal heatmaps
        loss_2d = F.mse_loss(proposal_heatmaps_2d[:, 0], targets['2d_heatmaps'], reduction='mean')

        # unravel the 1d gt heatmaps and compute 1d loss
        matched_heatmaps_1d = torch.gather(targets['1d_heatmaps'], dim=1, index=proposal2gt.long()\
                                           .unsqueeze(2).repeat(1, 1, proposal_heatmaps_1d.shape[2]))
        loss_1d = F.mse_loss(proposal_heatmaps_1d[mask], matched_heatmaps_1d[mask], reduction='mean')

        # compute the loss of bbox regression, only apply supervision on gt positions
        bbox_preds = torch.gather(bbox_preds, 1, targets['index'].long().view(batch_size, -1, 1).repeat(1, 1, 2))
        loss_bbox = F.l1_loss(bbox_preds[targets['mask']], targets['bbox'][targets['mask']], reduction='mean')

        del proposal_heatmaps_2d, proposal_heatmaps_1d, bbox_preds

        # weighted L1 loss of joint localization
        joints_3d = torch.gather(meta[0]['joints_3d'].float(), dim=1, index=proposal2gt.long().view\
                                         (batch_size, -1, 1, 1).repeat(1, 1, self.num_joints, 3))[mask]
        joints_vis = torch.gather(meta[0]['joints_3d_vis'].float(), dim=1, index=proposal2gt.long().view\
                                         (batch_size, -1, 1).repeat(1, 1, self.num_joints))[mask].unsqueeze(2)
        loss_joint = F.l1_loss(poses[0][mask] * joints_vis, joints_3d[:, :, :2] * joints_vis, reduction="mean") +\
                     F.l1_loss(poses[1][mask] * joints_vis, joints_3d[:, :, ::2] * joints_vis, reduction="mean") +\
                     F.l1_loss(poses[2][mask] * joints_vis, joints_3d[:, :, 1:] * joints_vis, reduction="mean") +\
                     2 * F.l1_loss(fused_poses[mask] * joints_vis, joints_3d * joints_vis, reduction="mean")

        loss_dict = {
            "2d_heatmaps": loss_2d,
            "1d_heatmaps": loss_1d,
            "bbox": 0.1 * loss_bbox,
            "joint": loss_joint,
            "total": loss_2d + loss_1d + 0.1 * loss_bbox + loss_joint
        }
    else:
        loss_dict = None
    print("after lossess block", GPUtil.getGPUs()[0].memoryUsed)
    # confidence score
    fused_poses = torch.cat([fused_poses, proposal_centers[:, :, 3:5].reshape(batch_size,\
                             -1, 1, 2).repeat(1, 1, self.num_joints, 1)], dim=3)
    return fused_poses, poses, proposal_centers.detach(), loss_dict, input_heatmaps`

Readings are as follows 1st iteration:
after self.backbone 2274.0
after self.pose_net 1680.0
after self.joint_net 1680.0
after lossess block 1680.0
2nd iteration
after self.backbone 2630.0
after self.pose_net 2640.0
after self.joint_net 2640.0
after lossess block 2640.0
3rd iteration
after self.backbone 2880.0
after self.pose_net 2890.0
after self.joint_net 2890.0
after lossess block 2890.0
4th iteration
after self.backbone 3132.0
after self.pose_net 3140.0
after self.joint_net 3140.0
after lossess block 3142.0
And it goes on like this until OOM.
Did you experience similar problems?
Thank you for your help.

Thanks for your interest in our project. We have fixed the bug in the previous commits. Please check out the newest version and let me know if there is any problem.

Thank you!
Looks like it works now.

Thank you! Looks like it works now.
By the way - can you tell what was the problem? I spent some time debugging this memory leak, and couldn't find the cause.

Hello, I want to ask some problem. Would you face with the problem that when the model starts training,the computer operates slowly and quit after it warning that the bounding box is not sufficiently large for few times?


Markdown formatting

Some of the headings weren't showing up correctly on Github so I updated the md files to get that working.
Code snippets also have some formatting around them for legibility.

Looks good. Thank you @noogn You can squash and merge it

Thanks, @ButenkoT!


Document --transpile-only flag for seed script

Thanks to Chris for mentioning that https://github.com/prisma/docs/pull/2787
--transpile-only see https://github.com/TypeStrong/ts-node#transpilers

In transpile-only mode, we skip typechecking to speed up execution time.

Chris mentions that

Failing to use the --transpile-only flag in some contexts is a catastrophic mistake that consumes massive amounts of RAM when typechecking the seed script(s) unnecessarily, in addition to transpiling them, thereby completely crashing Node in any container with limited RAM i.e. Docker, Heroku, etc. when trying to seed.

After discussing with @aqrln we came to the conclusion that we would like this to be documented but would not make it the default command like in the original PR.
This issue is then about mentioning ts-node's flag --transpile-only on seeding page https://www.prisma.io/docs/guides/database/seed-database#seeding-your-database-with-typescript-or-javascript

Maybe this would do
ts-node does transpiling and typechecking by default, typechecking can be disabled with `--transpile-only` if you want to have a faster and less resource (CPU/RAM) intensive seeding script.

Oh, I do like that wording, @Jolg42 . I didn't see it until after I edited Chris' PR... with my own wording. I might go back and add yours in.
I'm closing this Issue since we agreed to use Chris' [edited] PR.


Programmatically open select2

Hi,
I am trying to open the select2 on 'shown'.
Here is the code:
$(".edit_in_place").on('shown', function(e, editable) {
$(".input-medium").select2("open");
});
The shown and open events are correctly triggered but the dropdown menu doesn't display properly. The search bar appears in the top left corner of the window as you can see on the screenshot. After scrolling a bit, the dropdown puts itself together but if I click on another editable element things start to get wild (a video would be required to show the crazy behavior). Everything works fine if I open the dropdown manually though. I am not sure what I am doing wrong.
Help would be greatly appreciated :).

Thank a lot for Vitalets! I have the same problem today (About top left corner). I am only beginner in the web-programming and can't solve this problem during whole day.  And only in this topic I found decision of the problem. It is work!

If you don´t like the timeout() this works as well:
$.when(
    selectElement.select2();
).done(function(){
     selectElement.select2('open');
});


vue-r-mask did not work with Vuetify text field

Hello,
I'm trying implement masking with v-text-field of vuetify Js but did not work.
It is work well in Input Type as below
<input name="name" v-model="masked_value" v-mask="pattern"  placeholder="Choose email">
I'm trying below code with vuetify its not working.
<v-text-field
        name="name"
        v-model="masked_value"
        :label="label"
        :placeholder="placeholder"
        outlined
        dense
        hide-details="auto"
        :class="classes"
        v-mask="pattern"
      >
      </v-text-field>

Please give me any solutions.

vue-r-mask works only with native html tags. If you want attach vue-r-mask to wrapper around native input you need event propagation to that wrapper.


TAJO-1556: "insert into select" with reordered column list does not work.

fixed a bug in buildProjectedInsert.

Thanks for your work.
I've looked over your patch. It looks good to me, but will be great if you add a test for query execution in addition to the existing planner test.