Add UI for creating a subset of an existing dataset

We might want to print an error somewhere when you try to create a subset with the same name as an existing dataset (at least in my test, it just refused to do so without any indication why).

I created #210 to track this suggestion separately;  I think this should be approached for all API calls. @manthey Other than this, do you think this can be merged so we can resolve the console errors in #208?

I created #210 to track this suggestion separately; I think this should be approached for all API calls. @manthey Other than this, do you think this can be merged so we can resolve the console errors in #208?

General error handling is a good approach.


Create verulamschool.txt

@Samueljh1 Pull request merged. Thank you!

Hi, I still cant seem to be able to apply with a verulamschool.co.uk email address.

@Samueljh1 No. Usually one can send a request for the license in about half an hour after the pull request is merged. We are investigating the issue. I am sorry for the inconvenience.


What happened to filterType in MUIDatatableColumnOptions in the Typescript definitions?

I can't find any documentation for the change made in September removing filterType from MUIDatatableColumnOptions in the Typescript types definitions. How is this handled now?

Looks like this issue is missing some information! I got a screenshot of a commit I didn't recognize in my email alerts for this. Could be from a different project or a fork or something. Can you find the commit you're looking for in the commit history here? Or something in the release notes? I reference all PRs that get merged in every release.

Yes, sorry about this. Please delete.


Update http proxy settings after configuring auth

I might be missing something but ran into a use case where we needed to configure http proxy after setting up auth on our ops manager.  It appears that configuring auth is completely skipped when already configured which appears to mean that http proxy settings are also not updated.
+ om --env env/env.yml configure-authentication --config config/auth.yml
configuration previously completed, skipping configuration

Based upon this PR I figure the http proxy would be configured for the ops manager but only the first time that auth is configured: https://github.com/pivotal-cf/om/pull/252  After which point the task appears to skip configuring all configuration?  Maybe I am missing something?

There is a limitation of the OpsManager API, which does not allow reconfiguring of the authentication after initial load -- including http(s)_proxy values. We are discussing with the OpsManager team on how to move forward with this.

There is no way to do this via API for the GUI, at the moment.
I'm going to close this issue, we can revisit it, if/when OpsManager supports reconfiguration or authentication.
If you must to do this on an OpsManager, please contact Pivotal Support as they will be able to assit you further.

Went ahead and opened a pivotal support ticket about the use-case so there's something in the backlog about it: case number 211454


fix: 修复ReadableMap tostring方法兼容性

还有一些问题存在,先别合并

There are many other methods that also use map.toString() and you need to determine if you want to replace them all with readableMap2string

see https://github.com/openimsdk/open-im-sdk-reactnative/issues/32


Merging cells: Colspans [More Flexible Tables Pt.3a]

Allows specifying a colspan to grid.cell and table.cell to have a cell span more than one column.
Rowspans will be in a future PR. This PR will provide the foundations for merged cells, and rowspans would simply be an addition to those.
This PR does not include changes to docs. These were postponed so review doesn't have to take that into account for now.
Part of the third task in #3001.
User-facing API
You can now specify grid.cell(colspan: amount, ...) and table.cell(colspan: amount, ...) as seen below.
#grid(
  columns: 4,
  fill: (x, y) => if calc.odd(x + y) { blue.lighten(50%) } else { blue.lighten(10%) },
  inset: 5pt,
  align: center,
  grid.cell(colspan: 4)[*Full Header*],
  grid.cell(colspan: 2, fill: orange)[*Half*],
  grid.cell(colspan: 2, fill: orange.darken(10%))[*Half*],
  [*A*], [*B*], [*C*], [*D*],
  [1], [2], [3], [4],
  [5], grid.cell(colspan: 3, fill: orange.darken(10%))[6],
  grid.cell(colspan: 2, fill: orange)[7], [8], [9],
  [10], grid.cell(colspan: 2, fill: orange.darken(10%))[11], [12]
)

#table(
  columns: 4,
  fill: (x, y) => if calc.odd(x + y) { blue.lighten(50%) } else { blue.lighten(10%) },
  inset: 5pt,
  align: center,
  table.cell(colspan: 4)[*Full Header*],
  table.cell(colspan: 2, fill: orange)[*Half*],
  table.cell(colspan: 2, fill: orange.darken(10%))[*Half*],
  [*A*], [*B*], [*C*], [*D*],
  [1], [2], [3], [4],
  [5], table.cell(colspan: 3, fill: orange.darken(10%))[6],
  table.cell(colspan: 2, fill: orange)[7], [8], [9],
  [10], table.cell(colspan: 2, fill: orange.darken(10%))[11], [12]
)

Implementation details

CellGrid now stores a vector of Entry, which can either be Entry::Cell(Cell) (a regular cell position in the grid) or Entry::Merged { parent: usize } (this position was merged with the cell at index parent).
In GridLayouter, the function cell_spanned_width is now used to determine the resolved width of a cell. (Before, it would just correspond to the resolved width of the column it was in.) This prompted changes in several locations, including in fill size calculation (in render_fills_strokes, which was changed to not borrow self mutably so we can use that method) and when laying out the cell in various places, such as in measure_auto_rows.
measure_auto_columns was changed to consider colspans. A colspan will only affect the size of the last auto column it spans (which doesn't affect non-colspans, since they only span the column they are in).

For the effect of comparing the width of a colspan at its last auto column with the widths of other cells in that auto column, we take the colspan cell's measured width and subtract its resolved width (through cell_spanned.width), noting that said resolved width doesn't consider the current auto column since it wasn't measured yet. That way, the colspan only expands an auto column based on how much width it actually needs; that is, excluding width already provided by other spanned columns.

I created a function split_vlines which is responsible for taking a vline across a region and splitting it into multiple vline segments to draw so that it doesn't overlap with colspans. The algorithm for colspan detection is in should_draw_vline_at_row. I added unit tests because they were very helpful in detecting bugs. (The idea is to implement similar functions for hlines in the future rowspans PR.)

split_vlines is used in render_fills_strokes, at the drawing vlines step. The automatic vlines start spanning the entire region, and are replaced by an iterator of vline segments.
Without this, strokes would be drawn above colspans when they shouldn't.

Note: In measure_auto_columns, I included a heuristic borrowed from https://github.com/PgBiel/typst-tablex/pull/116 in which colspans which span all fractional columns don't affect the sizes of auto columns at all, since they will already have all extra available space after auto columns are resolved and later resized. This is needed since fractional columns' widths are not known when measuring auto columns, so a colspan over all fractional columns and one or more auto columns would almost certainly cause the auto column to expand much more than it should, leading to unnecessarily zero-sized fractional columns. However, this only applies to finite regions/pages, otherwise the fractional columns would always have zero width. Additionally, it doesn't apply to colspans which span some but not all fractional columns, since then we wouldn't be able to tell whether or not they'd have enough available space later. (See the tests for examples)

This heuristic might be harmful in the case where the table is so large that fractional columns have zero width, in which case the colspan might get too little space available. However, I'd argue that, in that case, the table will look off either way, since some columns will have zero width. So this heuristic probably makes more sense overall.
An alternative is to implement something like tablex's fit-spans option, through which we instruct colspans and rowspans to not affect the sizes of tracks, and force users to consider using that option when tables look off. However, while I am in favor of adding this option (maybe in a separate PR though), I believe it's fine for Typst to prioritize looking good by default when possible.
See examples of tables which would look off without this heuristic here: https://github.com/PgBiel/typst-tablex/issues/56 and https://github.com/PgBiel/typst-tablex/issues/78

Drafting for a bit as I forgot to add some checks for RTL

RTL should be fixed. Should be OK to review now

Merged mergeable cells, yay!


Error consistency

I have VSCode set up to work with Lua and it's amazing. Feels like a real language.
I am having some trouble parsing sol error messages. Most error messages have a format like this:
sol: runtime error: [string "c:\(...)\Leadwerks\Projects\Ultra Project\Scripts\main.lua"]:17: attempt to call global 'amera' (a nil value)
stack traceback:
	[string "c:\(...)\Leadwerks\Projects\Ultra Project\Scripts\main.lua"]:17: in main chunk

But some have a format like this:
sol: runtime error: stack index 1, expected userdata, received sol.sol::detail::unique_usertype<UltraEngine::World>: value is a userdata but is not the correct unique usertype (bad argument into 'std::shared_ptr<UltraEngine::Camera>(std::shared_ptr<UltraEngine::Entity>)')
stack traceback:
	[C]: in function 'Camera'
	[string "c:\(...)\Leadwerks\Projects\Ultra Project\Scripts\main.lua"]:17: in main chunk

And I found another version:
[sol3] An error occurred and has been passed to an error handler: sol: syntax error: [string "c:\(...)\Leadwerks\Projects\Ultra Project\Script..."]:15: '=' expected near 't'
sol: syntax error: [string "c:\(...)\Leadwerks\Projects\Ultra Project\Script..."]:15: '=' expected near 't'

Consequently, it is difficult for me to parse the error, open the correct file, and display an error message.

I am also having trouble with file paths being truncated to 64 bytes. I set LUA_IDSIZE to 2048 in my Lua DLL but it seems to make no difference.

It appears that syntax errors truncate the filename and runtime errors do not.

For the "filename" issue, this is the "chunkname" parameter in sol2. It exists outside of Lua. I didn't even know LUA_IDSIZE existed. I use 64 because it's a modest buffer size, but I can always make it larger.

MAX_INFO_STRING?

This is how I am running scripts. I don't know where the file path is being truncated:
		sol::protected_function_result solresult;
		try
		{
			solresult = VirtualMachine::lua->safe_script(source, s);
		}

I can't figure out why, but I am still getting errors that look like this:
"sol: syntax error: [string \"c:\\Users\\Name\\Documents\\Leadwerks\\Projects\\Ultra Project\\Script...\"]:18: function arguments expected near 'sed'"

It says SOL_ID_SIZE_ is 512.

An error like this results gives the full correct file path:
local framebuffer = Creat eFramebuffer(window);

An error like this gives a truncated (useless) file path:
while window:Clos ed() == false do

It looks like the traceback function of luaL_traceback is doing automatic truncation of the "chunk name" description string, semi-outside of my control. It looks like you will need to manually implement your own traceback handler, similar to the one that exists in sol2, to get more consistent handling of the behavior: https://github.com/ThePhD/sol2/blob/e256012de1b7ac03b54bce9d09f6e17a2c1ad100/include/sol/state_handling.hpp#L76
Either that, or I could rip the luaL_traceback function open and force some different behaviors...

I don't understand, I set LUA_IDSIZE to 2048 in the Lua source code. Shouldn't that be sufficient to avoid the problem?

traceback behaves differently and probably has special rules for how it truncates things to provide the traceback, probably. So you'd need to build a better traceback. (Or I would.)

These guys seem to have dealt with this problem:
https://forums.civfanatics.com/threads/how-to-restore-runtime-error-reports-for-your-lua-code.485607/

I tried just printing the traceback in Lua and it works fine this way:
Print(debug.traceback())

The full path is printed.

I GOT IT!
You know, I had a vague memory of there being another value that had to be modified, from years back.
In llex.c at line 75:
#define MAXSRC          80

Just change that to a bigger value (preferably the same one you have LUA_IDSIZE set to) and the truncation goes away.
There may be other issues in the error format, but I am going to close this topic.


[FEATURE request]: Implmenting openpose_hand

As in the title. It would be somewhat cool if we get the openpose_hand model to use in openpose-editor, so that we can really fine tune those hands when extracting and refining a pose. Template making would then be a breeze. :)

It could be super useful indeed, as I don't know how to edit hands from the PNG file provided by the preprocessor.
@Woisek Do you have a workaround in the meantime, other than blender?

@oudivad
My current workaround is to merge the generated poses from openpose-editor and the pre-processor.
There are two ways:
Either you generate one pose image with openpose-editor, one with the pre-processor and then cut&paste the hands from the pre-processor into the one from openpose-editor, or
you finetune the pose in openpose-editor and then generate images with it until there are hands that fit. Run this image through the pre-processor again to get the complete pose with the matching hands.
The second way tends to be a bit cleaner.

@Woisek Thanks a lot, came across this thread today, and was experiencing with hands ;)
https://www.reddit.com/r/StableDiffusion/comments/11q6e4c/fixing_hands_with_openpose_hand_controlnet_stable/

Funny, I found this thread, also ... :D


Self-hosting v2.16.7 Music player not playing at the correct speed.

After updating to version 2.16.7, the music player no longer plays youtube audio at full speed. I have tested it using different youtube links including:
https://youtube.com/watch?v=4zLfCnGVeL4
https://youtube.com/watch?v=u9Dg-g7t2l4
Note: my upload speed is not an issue (50mb/s)

Little confused why this isn't closed and asked in #1053 . But oh well.
Can you say wich OS your using?
Have you tried maybe re-installing? Maybe the whole OS?
Checked how cpu/memory performance is doing when playing music?
Any logs that might indicate something strange when you play other songs that also happen to be slower played?


Add code files.

This PR introduces the code for handling input data for protein-ligand free energy benchmark calculations. There are classes for handling different aspects of the input:

target (protein target)
ligand (small drug-like molecule which binds to the target)
edge (perturbation/transformation between pairs of ligands)
For all these above classes, there are also corresponding classes handling sets of these:
targetSet
ligandSet
edgeSet.
These classes are initialized from (meta)data stored in yml files.

One thing I notice is that a lot of sample data is included. We should aim to have as little of this as possible, just enough to test the baseline functionality (and do so only once). Could less data serve these testing purposes? Don't need to reproduce much data to ensure the code works as intended.

I reduced the sample data substantially by deleting unused files and using only a subset of ligands. (commit https://github.com/openforcefield/PLBenchmarks/pull/3/commits/6dc9ef58afcfee31ff6d8847c2f213527d291bb5)


Posting blogs to the ContentMine, Winnower, etc.

Steph,
This is a licence to nag me to get my blog posts onto the ContentMine site and the Winnower. Perhaps occasional F2F meetings will help.

@GrahamSteel I'm transferring some issues over to you, let me know if you need any help with anything.


Look into potential security vulnerability

Github found the following potential security vulnerability:

The jquery-ui dependency defined in package-lock.json has a known moderate severity security vulnerability in version range < 1.12.0 and should be updated.

Jep, I know. Nothing to be done about it for the current version, as upping jquery uit causes problems with other jquery plugins. Pushing this forward to the next rewrite, as that one won't contain jquery

The latest version is mostly up-to-date with all dependencies


Show help when are no arguments

Before
https://travis-ci.org/ichigotake/chromeos-apk/builds/47321197
$ ./chromeos-apk

/Users/ichigotake/Development/ichigotake/chromeos-apk/chromeos-apk.js:31
    throw new Error('Please provide a path to an APK file...');
          ^
Error: Please provide a path to an APK file...
    at module.exports (/Users/ichigotake/Development/ichigotake/chromeos-apk/chromeos-apk.js:31:11)
    at Object.<anonymous> (/Users/ichigotake/Development/ichigotake/chromeos-apk/chromeos-apk:3:29)
    at Module._compile (module.js:456:26)
    at Object.Module._extensions..js (module.js:474:10)
    at Module.load (module.js:356:32)
    at Function.Module._load (module.js:312:12)
    at Function.Module.runMain (module.js:497:10)
    at startup (node.js:119:16)
    at node.js:906:3

After
https://travis-ci.org/ichigotake/chromeos-apk/builds/47321006
$ ./chromeos-apk

  Usage: chromeos-apk <path_to_apk_file ...>

  Options:

    -h, --help          output usage information
    -V, --version       output the version number
    -t, --tablet        Create a tablet version
    -s, --scale         Enable application window scaling
    -n, --name [value]  Extension display name

Sorry, I made mistake.
Don't pass this block.
  if (!apk) {
    throw new Error('Please provide a path to an APK file...');
  }

If accept this pull request, I think don't need that block.

I see...
chromeos-apk$ ./chromeos-apk

  Usage: chromeos-apk <path_to_apk_file ...>

  Options:

    -h, --help          output usage information
    -V, --version       output the version number
    -t, --tablet        Create a tablet version
    -s, --scale         Enable application window scaling
    -n, --name [value]  Extension display name

$ ./chromeos-apk -t

/Users/ichigotake/Development/ichigotake/chromeos-apk/chromeos-apk.js:36
    throw new Error('Please provide a path to an APK file...');
          ^
Error: Please provide a path to an APK file...
    at module.exports (/Users/ichigotake/Development/ichigotake/chromeos-apk/chromeos-apk.js:36:11)
    at Object.<anonymous> (/Users/ichigotake/Development/ichigotake/chromeos-apk/chromeos-apk:3:29)
    at Module._compile (module.js:456:26)
    at Object.Module._extensions..js (module.js:474:10)
    at Module.load (module.js:356:32)
    at Function.Module._load (module.js:312:12)
    at Function.Module.runMain (module.js:497:10)
    at startup (node.js:119:16)
    at node.js:906:3

Don't really see the point of adding help on a CI build... Won't really help to debug anything.  I'm curious about the modification to chromeos-apk.js though.  What exactly is this supposed to do?  And why a length of 2?  Thanks for the clarification.

Also if you don't want this block: '''  if (!apk) {
throw new Error('Please provide a path to an APK file...');
}''' can you please remove it from the PR, and rebase it into one commit.  Thanks

Couldn't we merge the 2 versions? Maybe something like:
if (!apk) {
    program.outputHelp();
    process.exit(0);
  }


Unable to use 1password when asked for password in TFE and desktop wallet

Whenever there is an opportunity to enter a password the browser recognises that and drops a 1passord icon in the password text box.
When user clicks on the icon the window closes
Expected to be able to select 1password password and it dropping into the text box.

Note from Dex... On mainnet this effects nothing, on testnet it effects hosted wallet only, so popping in the icebox for now


site.url missing from schema

also should probably be listed as required: https://github.com/the-grid/apis/issues/63

So to set a custom URL, do we have to set both domain and url?
To unset a custom domain, do we have to set domain to null and generate url from repo?

Where in the code can I find this?
Where in the docs should I be able to RTF docs?

(Flowhub recruitment potential: Answer API issues with screenshots and links to the graph...)
https://app.flowhub.io/#project/apis/apis_graphs_apis

site.url is a read-only property that the back-end provides for convenience when you read site data.

To set a custom URL, just set domain?
Can we specify https?
To unset a custom domain, just set domain to null?

@forresto

Yes
Not at the moment, maybe in the future (though user may possibly do this with service like Cloudflare)
Not supported yet


"Upload your own files" in cookbook/quickstarts /Video.ipynb Gives error 404

Description of the bug:
In quickstarts/Video.ipynb file under "Upload a video to the Files API" section, link "Upload your own files" is redirecting to examples/Upload_files.ipynb which gives error "404 - page not found".

Actual vs expected behavior:
No response
Any other information you'd like to share?
No response

same issue +1

@SGCODEX, "Upload your own files" link has been updated. Thank you for reporting this issue.


Improve styling of the form and the footer.

Hi @garimasingh128 . Thanks for opening issue. What kind of styling issue are you facing in the form and Footer?

I was just thinking if we could add a colour scheme to whole website as now it looks quite random. We could also add a detailed footer and a form that follows the design.

Yes, ur right.
There is no color scheme decided yet and the footer just consists of icons.
I am assigning this issue to you and would like to see your creativity.
And join the discord server.


Feature 3.7/remove blacklist contents

Scope & Purpose

Migrate this branch to use test-definitions.txt

So far we would have the OskarTestSuitesBlockList to stop removed testsuites to be executed in PR runs.
Since now the Branch has the test-definitions.txt the contents of the Blacklist can simply be removed from the test definition,
and the blacklist be emptied.

3.7 is EoL.


video max duration

how can i set videoMaximumDuration using this library
as i need 1 minute video only from camera/photogallery

No there is currently no option like that in the library


Update to Go 1.19

Codecov Report

Merging #277 (05c99ae) into master (79f921f) will increase coverage by 0.29%.
The diff coverage is 82.35%.

@@            Coverage Diff             @@
##           master     #277      +/-   ##
==========================================
+ Coverage   82.97%   83.26%   +0.29%
==========================================
  Files          69       68       -1
  Lines        1950     1942       -8
==========================================
- Hits         1618     1617       -1
+ Misses        231      224       -7
  Partials      101      101

Impacted Files
Coverage Δ

antireplay/noop.go
100.00% <ø> (ø)

internal/utils/net_listener.go
0.00% <0.00%> (ø)

ipblocklist/files/http.go
61.53% <0.00%> (ø)

ipblocklist/files/mem.go
88.88% <ø> (ø)

ipblocklist/firehol.go
79.20% <ø> (ø)

ipblocklist/noop.go
100.00% <ø> (ø)

mtglib/events.go
100.00% <ø> (ø)

mtglib/internal/obfuscated2/server_handshake.go
60.00% <0.00%> (ø)

mtglib/proxy_opts.go
89.65% <ø> (ø)

network/load_balanced_socks5.go
100.00% <ø> (ø)

... and 32 more

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


mat-radio-button is deselected when the value passed to setValue() is the same

Reproduction
Steps to reproduce:

https://stackblitz.com/edit/angular-slemqj-x62ebw?file=app%2Fradio-overview-example.html

Expected Behavior
What behavior were you expecting to see?
When I dynamically set the value for a form control (even if the value is the same) I expect the radio-button to be selected.
Actual Behavior
What behavior did you actually see?
If the radio-button value="1" is currectly selected and I use setValue() control method to setValue to "1" I expect it to remain selected, but it is actually deselected.
Environment

Angular: 5
CDK/Material: 5
Browser(s): edge
Operating System (e.g. Windows, macOS, Ubuntu): windows

i have to submit a stackblitz example with latest version of angular. I didn't realize this was version 5.


Remove deep_space patch

Code that depends on gravity_bridge in external Cargo.toml files will not be able to correctly resolve the right version of deep_space with the setup using the patch directive in the workspace Cargo.toml file. This reverts that behavior and specifies the specific version of deep_space in each crate.

This patch was needed for some the crypto.com was using


Improve error handling

Closes #116

Raise a 404 if pipeline doesnt exist
Show descriptive error message in frontend
Redirect to all pipelines if specific pipeline doesn't exist

todo:

[x] investigate duplicate errors
[x] add backend test
[x] add frontend tests

idea: perhaps display a persistent popup with choices "Update pipelines" and "Back to all pipelines"
also added a loading indicator for when metrics are being refreshed

This could also close #131 , right? :)

This could also close #131 , right? :)

yep, it does. forgot to mention it here


Aeotec Multisensor 6 temperature threshold parameter reported incorrectly

Home Assistant release with the issue:

0.78.3
Last working Home Assistant release (if known):
Unknown
Operating environment (Hass.io/Docker/Windows/etc.):

Hassio, pi3b
Component/platform:

https://www.home-assistant.io/docs/z-wave/
Description of problem:
For a newly connected Aeotec Multisensor 6 zwave device the default temperature threshold value (parameter 41) displays as 20, but if something is written to the value (e.g. 10, or 5) after a short delay it displays 1310976 as the set value. I have tried this with both secure and insecure connections. I have tried and found this behaviour with two new devices.
All other parameter values display correctly when edited.
I’m pretty sure this is just a parameter value display bug as the device behaves as expected for the value entered.
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):

Traceback (if applicable):

Additional information:

Could you provide some logs please?
(OZW_Log.txt and homeassistant.log)

There was nothing relevant in the homeassistant.log but I can supply it if you need it. Below is a snippet from the ozw log. Setting parameter 41 to 5 (0.5°C) then to 2 (0,2°C):
`2018-09-29 14:27:50.960 Info, Node003, Value::Set - COMMAND_CLASS_CONFIGURATION - Temperature Reporting Threshold - 41 - 1 - 5
2018-09-29 14:27:50.960 Info, Node003, Configuration::Set - Parameter=41, Value=5 Size=4
2018-09-29 14:27:50.960 Detail, Node003, Queuing (Send) ConfigurationCmd_Set (Node=3): 0x01, 0x0f, 0x00, 0x13, 0x03, 0x08, 0x70, 0x04, 0x29, 0x04, 0x00, 0x00, 0x00, 0x05, 0x25, 0x11, 0x80
2018-09-29 14:27:50.961 Detail, Node003, Queuing (Send) ConfigurationCmd_Get (Node=3): 0x01, 0x0a, 0x00, 0x13, 0x03, 0x03, 0x70, 0x05, 0x29, 0x25, 0x12, 0x8d
2018-09-29 14:27:50.962 Detail,
2018-09-29 14:27:50.962 Info, Node003, Sending (Send) message (Callback ID=0x11, Expected Reply=0x13) - ConfigurationCmd_Set (Node=3): 0x01, 0x0f, 0x00, 0x13, 0x03, 0x08, 0x70, 0x04, 0x29, 0x04, 0x00, 0x00, 0x00, 0x05, 0x25, 0x11, 0x80
2018-09-29 14:27:50.973 Detail, Node003, Received: 0x01, 0x04, 0x01, 0x13, 0x01, 0xe8
2018-09-29 14:27:50.973 Detail, Node003, ZW_SEND_DATA delivered to Z-Wave stack
2018-09-29 14:27:51.260 Detail, Node003, Received: 0x01, 0x0b, 0x00, 0x04, 0x00, 0x03, 0x05, 0x31, 0x05, 0x05, 0x01, 0x2d, 0xeb
2018-09-29 14:27:51.260 Detail,
2018-09-29 14:27:51.260 Info, Node003, Received SensorMultiLevel report from node 3, instance 1, Relative Humidity: value=45%
2018-09-29 14:27:51.260 Detail, Node003, Refreshed Value: old value=45, new value=45, type=decimal
2018-09-29 14:27:51.260 Detail, Node003, Changes to this value are not verified
2018-09-29 14:27:51.260 Detail, Node003, Notification: ValueChanged
2018-09-29 14:27:51.311 Detail, Node003, Received: 0x01, 0x07, 0x00, 0x13, 0x11, 0x00, 0x00, 0x22, 0xd8
2018-09-29 14:27:51.311 Detail, Node003, ZW_SEND_DATA Request with callback ID 0x11 received (expected 0x11)
2018-09-29 14:27:51.311 Info, Node003, Request RTT 349 Average Request RTT 188
2018-09-29 14:27:51.312 Detail, Expected callbackId was received
2018-09-29 14:27:51.312 Detail, Expected reply was received
2018-09-29 14:27:51.312 Detail, Message transaction complete
2018-09-29 14:27:51.312 Detail,
2018-09-29 14:27:51.312 Detail, Node003, Removing current message
2018-09-29 14:27:51.312 Detail,
2018-09-29 14:27:51.312 Info, Node003, Sending (Send) message (Callback ID=0x12, Expected Reply=0x04) - ConfigurationCmd_Get (Node=3): 0x01, 0x0a, 0x00, 0x13, 0x03, 0x03, 0x70, 0x05, 0x29, 0x25, 0x12, 0x8d
2018-09-29 14:27:51.321 Detail, Node003, Received: 0x01, 0x04, 0x01, 0x13, 0x01, 0xe8
2018-09-29 14:27:51.321 Detail, Node003, ZW_SEND_DATA delivered to Z-Wave stack
2018-09-29 14:27:51.364 Detail, Node003, Received: 0x01, 0x07, 0x00, 0x13, 0x12, 0x00, 0x00, 0x05, 0xfc
2018-09-29 14:27:51.364 Detail, Node003, ZW_SEND_DATA Request with callback ID 0x12 received (expected 0x12)
2018-09-29 14:27:51.364 Info, Node003, Request RTT 51 Average Request RTT 119
2018-09-29 14:27:51.364 Detail, Expected callbackId was received
2018-09-29 14:27:51.443 Detail, Node003, Received: 0x01, 0x0e, 0x00, 0x04, 0x00, 0x03, 0x08, 0x70, 0x06, 0x29, 0x04, 0x00, 0x14, 0x01, 0x00, 0xb0
2018-09-29 14:27:51.443 Detail,
2018-09-29 14:27:51.443 Info, Node003, Response RTT 130 Average Response RTT 978
2018-09-29 14:27:51.443 Detail, Node003, Initial read of value
2018-09-29 14:27:51.443 Info, Node003, Received Configuration report: Parameter=41, Value=1310976
2018-09-29 14:27:51.443 Detail, Node003, Expected reply and command class was received
2018-09-29 14:27:51.444 Detail, Node003, Message transaction complete
2018-09-29 14:27:51.444 Detail,
2018-09-29 14:27:51.444 Detail, Node003, Removing current message
2018-09-29 14:27:51.444 Detail, Node003, Notification: ValueChanged
2018-09-29 14:27:52.182 Detail, Node003, Received: 0x01, 0x09, 0x00, 0x04, 0x00, 0x03, 0x03, 0x80, 0x03, 0x64, 0x15
2018-09-29 14:27:52.182 Detail,
2018-09-29 14:27:52.182 Info, Node003, Received Battery report from node 3: level=100
2018-09-29 14:27:52.182 Detail, Node003, Refreshed Value: old value=100, new value=100, type=byte
2018-09-29 14:27:52.182 Detail, Node003, Changes to this value are not verified
2018-09-29 14:27:52.182 Detail, Node003, Notification: ValueChanged
2018-09-29 14:27:52.261 Detail, Node003, Received: 0x01, 0x0c, 0x00, 0x04, 0x00, 0x03, 0x06, 0x31, 0x05, 0x03, 0x0a, 0x02, 0x3b, 0xf6
2018-09-29 14:27:52.261 Detail,
2018-09-29 14:27:52.261 Info, Node003, Received SensorMultiLevel report from node 3, instance 1, Luminance: value=571lux
2018-09-29 14:27:52.261 Detail, Node003, Refreshed Value: old value=575, new value=571, type=decimal
2018-09-29 14:27:52.261 Detail, Node003, Changes to this value are not verified
2018-09-29 14:27:52.261 Detail, Node003, Notification: ValueChanged
2018-09-29 14:27:52.403 Detail, Node003, Received: 0x01, 0x0b, 0x00, 0x04, 0x00, 0x03, 0x05, 0x31, 0x05, 0x1b, 0x01, 0x00, 0xd8
2018-09-29 14:27:52.403 Detail,
2018-09-29 14:27:52.403 Info, Node003, Received SensorMultiLevel report from node 3, instance 1, Ultraviolet: value=0
2018-09-29 14:27:52.403 Detail, Node003, Refreshed Value: old value=0, new value=0, type=decimal
2018-09-29 14:27:52.403 Detail, Node003, Changes to this value are not verified
2018-09-29 14:27:52.403 Detail, Node003, Notification: ValueChanged
2018-09-29 14:28:03.432 Detail, Node003, Received: 0x01, 0x0c, 0x00, 0x04, 0x00, 0x03, 0x06, 0x31, 0x05, 0x03, 0x0a, 0x02, 0x37, 0xfa
2018-09-29 14:28:03.432 Detail,
2018-09-29 14:28:03.432 Info, Node003, Received SensorMultiLevel report from node 3, instance 1, Luminance: value=567lux
2018-09-29 14:28:03.433 Detail, Node003, Refreshed Value: old value=571, new value=567, type=decimal
2018-09-29 14:28:03.433 Detail, Node003, Changes to this value are not verified
2018-09-29 14:28:03.433 Detail, Node003, Notification: ValueChanged
2018-09-29 14:31:19.722 Info, Node003, Value::Set - COMMAND_CLASS_CONFIGURATION - Temperature Reporting Threshold - 41 - 1 - 2
2018-09-29 14:31:19.722 Info, Node003, Configuration::Set - Parameter=41, Value=2 Size=4
2018-09-29 14:31:19.722 Detail, Node003, Queuing (Send) ConfigurationCmd_Set (Node=3): 0x01, 0x0f, 0x00, 0x13, 0x03, 0x08, 0x70, 0x04, 0x29, 0x04, 0x00, 0x00, 0x00, 0x02, 0x25, 0x16, 0x80
2018-09-29 14:31:19.722 Detail, Node003, Queuing (Send) ConfigurationCmd_Get (Node=3): 0x01, 0x0a, 0x00, 0x13, 0x03, 0x03, 0x70, 0x05, 0x29, 0x25, 0x17, 0x88
2018-09-29 14:31:19.724 Detail,
2018-09-29 14:31:19.724 Info, Node003, Sending (Send) message (Callback ID=0x16, Expected Reply=0x13) - ConfigurationCmd_Set (Node=3): 0x01, 0x0f, 0x00, 0x13, 0x03, 0x08, 0x70, 0x04, 0x29, 0x04, 0x00, 0x00, 0x00, 0x02, 0x25, 0x16, 0x80
2018-09-29 14:31:19.732 Detail, Node003, Received: 0x01, 0x04, 0x01, 0x13, 0x01, 0xe8
2018-09-29 14:31:19.733 Detail, Node003, ZW_SEND_DATA delivered to Z-Wave stack
2018-09-29 14:31:20.135 Detail, Node003, Received: 0x01, 0x07, 0x00, 0x13, 0x16, 0x00, 0x00, 0x29, 0xd4
2018-09-29 14:31:20.136 Detail, Node003, ZW_SEND_DATA Request with callback ID 0x16 received (expected 0x16)
2018-09-29 14:31:20.136 Info, Node003, Request RTT 412 Average Request RTT 261
2018-09-29 14:31:20.136 Detail, Expected callbackId was received
2018-09-29 14:31:20.136 Detail, Expected reply was received
2018-09-29 14:31:20.136 Detail, Message transaction complete
2018-09-29 14:31:20.136 Detail,
2018-09-29 14:31:20.136 Detail, Node003, Removing current message
2018-09-29 14:31:20.136 Detail,
2018-09-29 14:31:20.136 Info, Node003, Sending (Send) message (Callback ID=0x17, Expected Reply=0x04) - ConfigurationCmd_Get (Node=3): 0x01, 0x0a, 0x00, 0x13, 0x03, 0x03, 0x70, 0x05, 0x29, 0x25, 0x17, 0x88
2018-09-29 14:31:20.146 Detail, Node003, Received: 0x01, 0x04, 0x01, 0x13, 0x01, 0xe8
2018-09-29 14:31:20.146 Detail, Node003, ZW_SEND_DATA delivered to Z-Wave stack
2018-09-29 14:31:20.208 Detail, Node003, Received: 0x01, 0x07, 0x00, 0x13, 0x17, 0x00, 0x00, 0x07, 0xfb
2018-09-29 14:31:20.209 Detail, Node003, ZW_SEND_DATA Request with callback ID 0x17 received (expected 0x17)
2018-09-29 14:31:20.209 Info, Node003, Request RTT 72 Average Request RTT 166
2018-09-29 14:31:20.209 Detail, Expected callbackId was received
2018-09-29 14:31:20.285 Detail, Node003, Received: 0x01, 0x0e, 0x00, 0x04, 0x00, 0x03, 0x08, 0x70, 0x06, 0x29, 0x04, 0x00, 0x14, 0x01, 0x00, 0xb0
2018-09-29 14:31:20.285 Detail,
2018-09-29 14:31:20.285 Info, Node003, Response RTT 148 Average Response RTT 563
2018-09-29 14:31:20.285 Detail, Node003, Refreshed Value: old value=1310976, new value=1310976, type=int
2018-09-29 14:31:20.285 Detail, Node003, Changes to this value are not verified
2018-09-29 14:31:20.286 Info, Node003, Received Configuration report: Parameter=41, Value=1310976
2018-09-29 14:31:20.286 Detail, Node003, Expected reply and command class was received
2018-09-29 14:31:20.286 Detail, Node003, Message transaction complete
2018-09-29 14:31:20.286 Detail,
2018-09-29 14:31:20.286 Detail, Node003, Removing current message
2018-09-29 14:31:20.286 Detail, Node003, Notification: ValueChanged
2018-09-29 14:31:33.581 Detail, Node003, Received: 0x01, 0x0c, 0x00, 0x04, 0x00, 0x03, 0x06, 0x31, 0x05, 0x03, 0x0a, 0x02, 0x1f, 0xd2
2018-09-29 14:31:33.581 Detail,
2018-09-29 14:31:33.581 Info, Node003, Received SensorMultiLevel report from node 3, instance 1, Luminance: value=543lux
2018-09-29 14:31:33.581 Detail, Node003, Refreshed Value: old value=549, new value=543, type=decimal
2018-09-29 14:31:33.582 Detail, Node003, Changes to this value are not verified
2018-09-29 14:31:33.582 Detail, Node003, Notification: ValueChanged`

Ok, this is either a problem with the device itself, or the settings of the config parameters in OZW.
2018-09-29 14:27:51.443 Info, Node003, Received Configuration report: Parameter=41, Value=1310976
Check your manual and compare to https://github.com/OpenZWave/open-zwave/blob/88a9e07b2fd60dc0a43cd36ac142f7bcf2129813/config/aeotec/zw100.xml#L58-L63
The important ones are type= and size=
This issue is already present at their repo: https://github.com/OpenZWave/open-zwave/issues/1332


TextField: reset error onBlur

resets error onBlur to remove any previous errors in the TextField,
which is preventing submission.
closes inveniosoftware/react-invenio-deposit#439

@jennur this should fix the issue, can you have a look and let me know if you can think of any other problem that this change could trigger?


Insert node at nth position in the linked list

Related Issue

#97

Proposed Changes

Problem and code in the file
Readme (Contains Problem description and example)

Checklist

[x] ✅ My code follows the code style of this project.
[x] 📝 My change requires a change to the documentation.
[x] 🎀 I have updated the documentation accordingly.
[x] 👀 I have read the CONTRIBUTING document.
[x] ✨ I have added tests to cover my changes.
[x] 🚩 All new and existing tests passed.
[x] 🌟 ed the repo

This pull request contains solution of the issue #97 ( Insert a new node at nth position of the linked list)

It fixes the issue which was created by me for addition of a problem in L-linked list section

It contains a readme file which contains Problem description and a file which contains the code for the given problem with proper description and comments.

It has passed all the testcases and i have attached screenshots of two testcases as well

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to change)

Output Screenshots
Test case 1

Test case 2

Hey @DlVYANSH  , make sure your Pull Request is descriptive and complete the checklist , star the repo and other fields are missing .
I would recommend you to go through this tutorial video of the project here ,thanks.

@Lakhankumawat sir I also want to work on this issue. Can I work on it?

@Lakhankumawat sir when you are going to review this pr again? It's been 2 days now since i have updated my pr. Kindly review this

@n4i9kita kindly review it again, I have made changes in the code as you have suggested

Really nicely done.
LGTM!

@DlVYANSH make a readme, and then add the problem description there.

@DlVYANSH make a readme, and then add the problem description there.

Kindly work on this and then its good to go.

@miraj0507 @n4i9kita i have added the readme. Kindly review it

You need to follow particular guidelines for your README , please follow
these guidelines click here
also you can see how your README should look like see here
for more please go through these here

No RESPONSE ??

No RESPONSE ??

Sir i am working on the readme. Actually i don't know how make it so i am learning

@n4i9kita ma'am i have added the readme file kindly review it

Uncheck all the checks which do not apply to you in checklist

@all-contributors please add @DlVYANSH code ,


removing ORCA dependency, bumping to v0.2.8

Removing ORCA dependency to fix issue 54 and recommending bump to v0.2.8

Thank you for the the pull request!


Add Botamp to the list of projects using Facebook Messenger

Coverage remained the same at 99.097% when pulling 69d03bac435e80d096762c1ffeebaceb47840155 on botamp:master into ce8bd7b5ec13d55e8f90223a13e88685d791bf6c on hyperoslo:master.

Cool, @ousmanedev! :sparkles: :cake: :sparkles:


Scanning does not work

There is some issue during compilation. Need to debug with checkify and debug prints.
This could allow faster compilation (though it's not too slow right now).

It now works fine


Suggestion: Recover HP breakpoint

I dont know if " Recover HP " as now working optimally (wasting time little maybe?).
For example If I get hit by stronger enemy and auto recoverHP is not fast enough for healing me to 100%, then I get to safe zone for healing even 1% of missing HP.
Maybe is good to set some breakpoints like dont get to safe zone if health is not under 80-90%? after enemy.
this is only to consider whether this hp recovery feature working optimally

this is probably marginal and there are certainly much more important improvements on the way,
it just seemed to me that healing is done quite often :D


linking to a pdf specific page with '#'

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots and Source
If applicable, add screenshots to help explain your problem.
You can help speed up fixing the problem by either

providing a simple reproduction
linking to your Quartz repository where the problem can be observed

Desktop (please complete the following information):

Quartz Version: [e.g. v4.1.2]
node Version: [e.g. v18.16]
npm version: [e.g. v10.1.0]
OS: [e.g. iOS]
Browser [e.g. chrome, safari]

Additional context
Add any other context about the problem here.

closing as duplicate of #1166


[UPD]

updated presentation

👷 Deploy Preview for cerulean-blini-883dc7 processing.

Name
Link

🔨 Latest commit
32aaa2a2674d3c243393d0e5d590664faad3a6b7

🔍 Latest deploy log
https://app.netlify.com/sites/cerulean-blini-883dc7/deploys/625b3690655d3000086fa2b3


npm WARN config production Use --omit=dev instead.

When I build Central, I see the following warning:
Step 14/22 : RUN npm clean-install --production --legacy-peer-deps
 ---> Running in 6203f5e5415a
npm WARN config production Use `--omit=dev` instead.

Should we be doing something different? Is there a way for us to address this warning so that it's not shown during the build?

@matthew-white what command are you using to build central?  Which branch/commit are you on?

Hello!

what command are you using to build central?

docker-compose build

Which branch/commit are you on?

next

Also re-creatable with docker-compose build service.

production

Default: null
Type: null or Boolean
DEPRECATED: Use --omit=dev instead.

Alias for --omit=dev
-- _https://docs.npmjs.com/cli/v9/using-npm/config#production_

It's official!  I'll open a PR to fix.

Closed by #358.


readOnly flag effectively disables model markers

I've stumbled upon an issue that seems to have been present since at least v0.14.x and most probably pertains to older versions as well (v0.14 is the farthest I could go back without webpack refusing to compile the bundle due to some missing polyfills etc.).
Setting readOnly either in the editor's constructor or during an update effectively disables model markers and it just feels counter-intuitive to assume this behaviour to have been intentional / by design. It rather appears to be a bug. This issue is currently blocking our team, so it would be immensely appreciated if somebody could clarify why that is and a possible workaround.
Thanks in advance
**monaco-editor version:**at least 0.14 - 0.22.3 (0.14 is the oldest version I can get to run so it's possibly also an issue for older versions)
Browser: FF, Chrome (newest versions as of now)
OS: Linux (but most probably also reproducible on other platforms)
Playground code that reproduces the issue:
const editor = monaco.editor.create(document.getElementById("container"), {
	value: "function hello() {\n\talert('Hello world!');\n}",
	language: "javascript",
    readOnly: true,
});

monaco.editor.setModelMarkers(editor.getModel(), 'analysis-tool-x', [{
    startLineNumber: 2,
    endLineNumber: 2,
    startColumn: 5,
    endColumn: 15,
    severity: monaco.MarkerSeverity.Error,
    message: 'axivion error',
}]);

Ok, I see this is a duplicate to #311. setting renderValidationDecorations did the job.


terraform validate with / with out variable hooks not working.

Hello,
Frist, thank you so much for the shared pre-commit code. i am referencing this original repo in pre-config yaml but somehow both validation with / without variable ones are not working as expected. see below.
λ cat .pre-commit-config.yaml
- repo: git://github.com/antonbabenko/pre-commit-terraform
  rev: v1.7.3
  hooks:
    - id: terraform_fmt
    - id: terraform_validate_no_variables
    - id: terraform_validate_with_variables

λ pre-commit install
pre-commit installed at C:\Cloud\Github\pre-commit\tf-test3\tf-test3.git\hooks\pre-commit
λ git status
modified:   main.tf
λcat main.tf
provider "aws" {}
resource "aws_s3_bucket_test1" "b" "e" "f" {
  bucket1                   = "my-tf-test-bucket-1"
  acl     = "private"
}
provider "aws" {}
resource "aws_s3_bucket_test2" "b" "c" "d" {
  bucket1 = "my-tf-test-bucket-2"
  acl                 = "private-public"
}

λ git commit -m "main.tf add"
Terraform fmt............................................................Failed
hookid: terraform_fmt
Files were modified by this hook. Additional output:
main.tf
Terraform validate without variables.....................................Passed
Terraform validate with variables........................................Passed
We can see here that fmt working as expected but somehow validation hooks are not catching the issues in main.tf file ( which shouldn't go through validation passed ).
Really appreciate help on this.

There is only one terraform_validate hook when using Terraform 0.12 and v1.15.0 which has been released lately.
Closing this issue.


Drop Keyword suffix from TokenSpecSet

Since we started modelling keywords as a single RawTokenKind there hasn’t been an ifKeyword etc. anymore. Instead we always refered to them as Keyword.if or .if everywhere. We should be consistent about this in TokenSpecSet as well, which also shouldn’t have a Keyword suffix on the cases.

@swift-ci Please test

@swift-ci Please test

@swift-ci Please test

@swift-ci Please test macOS

@swift-ci Please test Windows


Data missing from ohare (plenario) dataset

There is no data in the ohare hourly precipitation dataset from early 1997 through 2000.

I think I found all places where there was more than 24 consecutive hours missing. There were 7 spots in total, 6 were < 1 week, but one was the gap described above:
tm = (rain_df.index.astype(np.int64) // 10**9)
large_gaps = np.where(np.diff(tm) < -3600*24)[0]
rain_df.iloc[sorted(np.hstack((large_gaps, large_gaps+1)))]['Unnamed: 0']

datetime        integer index (?)
--------------------------
2016-08-01 00:51:00    440
2016-07-26 22:51:00    441
2016-07-01 00:51:00    192
2016-06-24 23:51:00    193
2016-06-01 00:51:00    852
2016-05-28 22:51:00    853
2016-05-01 00:42:00    715
2016-04-27 23:51:00    716
2016-04-01 00:14:00    579
2016-03-28 23:51:00    580
2016-03-01 00:51:00    481
2016-02-27 00:51:00    482
2000-01-01 00:56:00    210
1997-02-28 23:56:00    211
Name: Unnamed: 0, dtype: int64

@kbrose Thanks for catching this! I just checked plenario again, and it definitely looks like they're missing that chunk on the API. I'm downloading the full datasets (both back to around the 1940s) for O'Hare and Midway now from NOAA's Local Climatological Data, and we can see if that fills in the gaps

@kbrose I just uploaded the data to the Drive (links below). I haven't gotten the chance to fully look through it and see if there are any missing chunks, but at first glance the main gap you mentioned is in there.

full_midway_noaa.csv
full_ohare_noaa.csv

I ran the same numpy code to check on the updated dataset if there are any gaps (large_gaps = np.where(np.diff(tm) < -3600*24)[0]) and it didn't return any rows for O'Hare or Midway. I also reduced it to large_gaps = np.where(np.diff(tm) < -3601)[0] to see if there were any gaps at all greater than an hour and it didn't find any for either.
Looks like these are much more complete so I'll close this, and here are some links to versions cleaned up a bit and reduced just to HOURLYPrecip, location, and datetime:

ohare_full_precip_hourly.csv
midway_full_precip_hourly.csv


fix: Language system detect

Description
Fixes for language detection/selection

[x] - add language selector
[x] - remember choice, add device default
[ ] - confirm design

About # (link your issue here)
#4065
How Has This Been Tested?
Please describe the tests that you ran to verify your changes.
Checklist:

[ ] PR title matches follow: (Feature|Bug|Chore) Task Name
[ ] My code follows the style guidelines of this project
[ ] I have followed all the instructions described in the initial task (check Definitions of Done)
[ ] I have performed a self-review of my own code
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] I have added reference to a related issue in the repository
[ ] I have added a detailed description of the changes proposed in the pull request. I am as descriptive as possible, assisting reviewers as much as possible.
[ ] I have added screenshots related to my pull request (for frontend tasks)
[ ] I have pasted a gif showing the feature.
[ ] @mentions of the person or team responsible for reviewing proposed changes

awaiting design confirmation of @patpedrosa

@L03TJ3 in the case we definitely need to restart app, I suppose we need to display some red label under the selectbox
e.g. "This change requires the app the be restarted".  Needs to be discussed with @patpedrosa , a new PR could be opened for it
This one is ready to merge


Removed race condition in tests

Since the AlarmClock doesn't sleep for real in the tests, there is a race condition in checking the AlarmClock for expiration. After an AC is constructed, it kicks off a thread to do the sleeping. If the thread kicks off quickly, it will "sleep" and report that it is expired before we check that value on the very next line of the tests.
Instead, we just check to make sure that the AC variables were set properly, and then wait for it to expire.

:+1:

@john-gress @KjellKod  updated by removing the fake sleep from that long timeout test.

:+1:

I don't see the race but this works too


TSP Stipple ordering for drawing output

First of all, great project. Pretty great speeds and beautiful output. I work on a pen plotter project called the WaterColorBot and have been looking for a fast stipple with SVG output for pen plotting to integrate into RoboPaint, and this is great.
The old standard has been StippleGen, a great program unfortunately limited by its implementation in both number of stipples and general performance.
One of a few things missing before I can attempt to use the output of this app is a "Traveling Salesman Problem" circle/path ordering of the stipples. Attempting to draw the current output with the given path order shows the paths to be in basically random order, requiring huge pen moves between each circle, resulting in a var more inefficient toolpath than one where the TSP ordering has been done.
Might it be possible to add path ordering like this? I've considered adding some kind of top->bottom left->right naive ordering as an after-process, but it would be more efficient to make it part of the creation step. Thanks!

Hi! Thanks for using (or trying to anyways!) the application. I would be willing to do the same naive reading order ordering (left->right, top->bottom) that you are suggesting, but actually creating an optimal ordering based on the related TSP solution is outside the current scope of the project (contributions welcome though!). Now to find the time :)


./scripts/run_checks.sh fails locally

Running autoformatting.
Skipped 2 files
Autoformatting complete.
Running type checking.
src/envs/doors.py:430: error: Call to untyped function "norm" in typed context
src/envs/doors.py:537: error: Call to untyped function "norm" in typed context
Found 2 errors in 1 file (checked 131 source files)
Type checking failed! Terminating check script early.

1.21.2


[v2] Directive definition won't check  type

Version and Environment Details
Operation system: MacOS
IDE name and version: PHPStorm
Plugin version:  alpha-3
Expected Behaviour
Seems like the plugin won't check the type for directive's arguments?
directive @rules(apply: String) on ARGUMENT_DEFINITION | INPUT_FIELD_DEFINITION

# directive @rules(apply: [String!]!) on ARGUMENT_DEFINITION | INPUT_FIELD_DEFINITION
# ↑ This is the correct definition for `@rules` directive.

type Mutation {
    createUser(
        email: Email @rules(apply: ["email"]) # should throw error here because `apply` is an array!
        password: Password
    ): User
}

Works in 3.0.0.


Tweak plugin output message for run rebar3 upgrade

Bug Description
Output from the plugin states run rebar3 upgrade, but these days you have to either run upgrade <dep> or upgrade --all. Don't know if you wanna be so specific, though.
Expected Behavior
Output from the plugin should state run rebar3 upgrade --all | <dep>.
Additional Context
Using the latest plugin version

I originally left it vague on purpose 😛 … it's a feature.
Jokes aside, I don't mind either way.

I'll try to find a bit of time to update it. Is the propose above Ok with you?

Sure. I don't really care much about it. Change it to whatever you want, man.

Good stuff. I'll try to do a twofer and solve both issues I opened.


Reading binary format instead of base64 for browser version

I am trying to use this in the browser.  When I try to read, if I try to convert to base64, it works fine.  I can get it to work in the base64 format using option {base64: true} , but if I try to use plain binary, it throws error.  I don't think passing {'binary' : true} in options does anything.

Can confirm that binary didn't do anything for me either - so I switched over to base64 too (but deadlines meant I didn't have time to follow up, then forgot).

@poushy - 'binary' is the default mode, and you're right... passing {'binary': true} will have no effect. What to do depends on the type of data you have. If you've made an xhr request to a server to pull down a spreadsheet, it's most likely you're dealing with base64 - so you can just pass {base64: true} as the options and it should work.
This simple example demonstrates sending and receiving spreadsheets to/from a node service: Manual Test


Add storage volume to container

Hi, my workflow requires adding a data pool as a volume to my containers, e.g.:
docker run -v "path/to/data/:/data" ...

However, I could find anywhere in your documentation if this is possible. Is there a section about this or is it currently not supported? If it's currently not supported are there any plans to add it?
Any help will be greatly appreciated!

Yes, I was also a bit confused about how to do this. It would be great if an additional note could be added to the documentation to explain this, please.
Many thanks,
Matt

Will add a section to the README about code and data volumes.

Many thanks, Beberg. Much appreciated. I managed to do this manually but wasn't sure whether there was a different approach that I should be using.
docker run --name nvidia_data_science_stack -v ~/Data/notebooks:/home --gpus all --rm -it -p 8888:8888 -p 8787:8787 -p 8786:8786 data-science-stack:2.5.0
This maps the directory, but the JupyterLab is set to use the notebook_dir of /notebooks, which I couldn't seem to override. Appreciate your help with this.
Thanks,
Matt


AYON: Prepare functions for newer ayon-python-api

Changelog Description
Newer ayon python api will add new filtering options or change order of existing. Kwargs are used in client code to prevent issues on update.
Testing notes:

Current functions should work as did before
They should also work with ayon-python-api from this PR https://github.com/ynput/ayon-python-api/pull/116/files

I see this now (not sure if it has anything to do with this PR though):


[Keys] Sign partial identifyTx

We require that an IdentifyTx be communicated out-of-bounds to a participant which should then broadcast it to the network it wishes to join. Therefore a node should have the ability to partially sign an identify transaction for passing on.

We'll probably also need the ability to push this partially signed tx to a running node.


chore(test): add unit test for mem table spill

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?

Checklist

[ ] I have written necessary rustdoc comments
[ ] I have added necessary unit tests and integration tests
[ ] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[ ] My PR contains breaking changes. (If it deprecates some features, please create a tracking issue to remove them in the future).
[ ] All checks passed in ./risedev check (or alias, ./risedev c)
[ ] My PR changes performance-critical code. (Please run macro/micro-benchmarks and show the results.)

[ ] My PR contains critical fixes that are necessary to be merged into the latest release. (Please check out the details)

Documentation

[ ] My PR needs documentation updates. (Please use the Release note section below to summarize the impact on users)

Release note
If this PR includes changes that directly affect users or other significant modifications relevant to the community, kindly draft a release note to provide a concise summary of these changes. Please prioritize highlighting the impact these changes will have on users.

Generally LGTM.
But I think #[cfg(feature = "enable_test_epoch")] is still enabled in the unit test of streaming? If so, can we still test spill? When it is enabled, we are not able to include spill offset in EpochWithGap.

Solved.


Bar height - Is it possible increase the height of the bar?

I'd like to know if is it possible increase the height of the bar.
From this:

To this:

Loking at RNCSliderNativeComponent.web.tsx I could verify that it has trackHeight, but not for Android/iOS.

Does anyone confirm this?
Thank you for now.

I am also looking for this same feature

Hello @gabrielbarth!
The feature you requested is already reported as a feature request in #85.
Let me close this one as a duplicate.


Where is the docker container data mounted to on my system?

In run.sh, I see:
--volume $ROOT/data:/data \

on line 54, however when I run echo $ROOT, it outputs an empty line.
My end goal is to have this data be stored on an external drive, however I can't find this data folder on my SD card at the moment.
Also just wanted to mention how helpful I find this project - it's great!
Any help would be appreciated. Thanks!

Hi @jacobhq, sorry for the delay - the $ROOT variable gets set inside the run.sh script to the directory in which run.sh is located in (your jetson-containers directory)
https://github.com/dusty-nv/jetson-containers/blob/2d6187b00eaad34a4a51bf1e088baf4a600faa09/run.sh#L4
(the $ROOT variable won't persist outside of run.sh, which is maybe why you can't print it?)
Anyways, yes the standard location for the model/data cache is in your jetson-containers/data directory, and it would be easiest to just clone jetson-containers to your SD card - but you can change that data directory to wherever you want on your device (but it should always be mounted to the same place inside the container under /data)
run.sh is just a helper script and will print out the docker run command that it invokes - you can either edit the script to change this command, or ditch run.sh completely and just run your own docker run command (both ways are shown in the package readme's in the jetson-container docs)
Also, just to point it out - your jetson-containers repo does not store the containers themselves (those container images are stored by the docker daemon elsewhere, in what's known as the docker data root).  Both your jetson-containers repo and docker data root can be moved around independently.

Ok, thanks - that makes sense! So my understanding is that the data directory is not stored in the docker data root. I've moved the docker root to an external SSD following the steps outlined in setup.md. To do the same with the data directory, should I just clone the repo to the SSD or is there a better way (like just changing the location of the data directory mounted to the container)? If so, how would I do that?

You can literally just move (or re-clone) your jetson-containers repo to your desired location and start the container again, and by virtue of that $ROOT variable it will start the container next time to mount /data to your updated jetson-containers location.

Great, thank you!


[Issue]:The page of Gallery is not in autogenstudio  0.1.3 ?

Describe the issue
The page of Gallery is not in autogenstudio  0.1.3!   why?
Steps to reproduce
No response
Screenshots and logs
No response
Additional Information
No response

Missing the Gallery dearly...

Same here


Store properties are undefined in component called with Route via react-router

I've tried with the decorators and via import statement for the component called with Route component (HomeComponent) from react-router.
The parent component works completely fine, I'm using decorators and I'm able to access the properties of my store (only in the top component that is not called via Route).
No error is thrown, I can access my store but the @observables are undefined.
My store:
class AuthenticationStore {

    @observable isAuthenticated = false;

    /** more code here **/

}
const authenticationStore = new AuthenticationStore();
export default authenticationStore;

Component called via Route:
@withRouter @inject('authenticationStore') @observer
class HomeComponent extends Component {

    constructor(props){
        super(props);
    }

    render(){
        let { classes } = this.props;
        const {authenticationStore} = this.props;
        //can access, but isAuthenticated is always undefined
        console.log(authenticationStore.isAuthenticated);
        return (

Home

        )
    }
}

export default withStyles(styles)(HomeComponent);

entry point component:
class App extends Component {

    render(){
        return (
                <Router>
                    <Provider {...stores}>
                        <RefluxMainComponent/>
                    </Provider>
                </Router>

        );
    }
}

Route called in my top component (RefluxMainComponent):
<Route key={0} path="/" component={()=><HomeComponent/>} exact={true}/>
What am I missing?

A reproduction and much more details :)
Op do 24 mei 2018 om 17:10 schreef rottenoats<EMAIL_ADDRESS>
I've tried with the decorators and via import statement for the component
called with Route component from react-router.
The parent component works completely fine, I'm using decorators and I'm
able to access the properties of my store.
No error is thrown, I can access my store but the @observables
https://github.com/observables are undefined.
My store:
class AuthenticationStore {
@observable isAuthenticated = false;

/** more code here **/

}
const authenticationStore = new AuthenticationStore();
export default authenticationStore;
What am I missing?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/mobxjs/mobx-react/issues/485, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABvGhAmx7PR5K9ahwYd5aPZ0r0P8p6Cgks5t1s1agaJpZM4UMcYo
.

Dont inject, just use the imported class in you component.
import AuthenticationStore from '@libs/AuthenticationStore';
...

render(){
const {isAuthenticated} = AuthenticationStore;

in javascript seems all class are singletons.
Im I right?

@mweststrate I'll provide a reproduction in a bit.
@kikoseijo doesn't work, I already tried that.

Hmmmm, I created a general same setup in codesandbox and it worked.
https://codesandbox.io/s/91q23369kr
In my environment, I'm using Symfony4 with webpack-encore. Would it have anything to do with my configuration? Otherwise, I have really no idea where to go with this.
I'll keep thinking about it, maybe I forgot to include something in the codesandbox.
Here is a snippet of my webpack.config.js:
// webpack.config.js
var Encore = require('@symfony/webpack-encore');

Encore
// the project directory where all compiled assets will be stored
    .setOutputPath('public/build/')

    // the public path used by the web server to access the previous directory
    .setPublicPath('/build')

    // will create public/build/app.js and public/build/app.css
    .addEntry('app', './assets/js/app.js')

    .addStyleEntry("calendar", "./node_modules/react-big-calendar/lib/css/react-big-calendar.css")

    .configureBabel(function(babelConfig) {
        babelConfig.plugins = [
            "transform-object-rest-spread",
            "transform-class-properties",
            "transform-decorators-legacy",
            "transform-runtime",
        ]

        // no plugins are added by default, but you can add some
        // babelConfig.plugins.push('styled-jsx/babel');
    })
    // enable source maps during development
    .enableSourceMaps(!Encore.isProduction())

    // empty the outputPath dir before each build
    .cleanupOutputBeforeBuild()

    // show OS notifications when builds finish/fail
    .enableBuildNotifications()

    .enableReactPreset();

// export the final configuration
module.exports = Encore.getWebpackConfig();

Decorators plugin should be first
Op do 24 mei 2018 20:28 schreef rottenoats<EMAIL_ADDRESS>
Hmmmm, I created a general same setup in codesandbox and it worked.
https://codesandbox.io/s/91q23369kr
In my environment, I'm using Symfony4 with webpack-encore. Would it have
anything to do with my configuration? Otherwise, I have really no idea
where to go with this.
I'll keep thinking about it, maybe I forgot to include something in the
codesandbox.
Here is a snippet of my webpack.config.js:
// webpack.config.js
var Encore = require('@symfony/webpack-encore');
Encore
// the project directory where all compiled assets will be stored
.setOutputPath('public/build/')
// the public path used by the web server to access the previous directory
.setPublicPath('/build')

// will create public/build/app.js and public/build/app.css
.addEntry('app', './assets/js/app.js')

.addStyleEntry("calendar", "./node_modules/react-big-calendar/lib/css/react-big-calendar.css")

.configureBabel(function(babelConfig) {
    babelConfig.plugins = [
        "transform-object-rest-spread",
        "transform-class-properties",
        "transform-decorators-legacy",
        "transform-runtime",
    ]

    // no plugins are added by default, but you can add some
    // babelConfig.plugins.push('styled-jsx/babel');
})
// enable source maps during development
.enableSourceMaps(!Encore.isProduction())

// empty the outputPath dir before each build
.cleanupOutputBeforeBuild()

// show OS notifications when builds finish/fail
.enableBuildNotifications()

.enableReactPreset();

// export the final configuration
module.exports = Encore.getWebpackConfig();
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mobxjs/mobx-react/issues/485#issuecomment-391814468,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABvGhDPMsyZYDpKQvKbKPMcl6CT9FCLhks5t1vvTgaJpZM4UMcYo
.

@mweststrate can't believe it was just that. Thanks. Spending hours trying to figure this out.

When I change the language by the user's interaction, it works well but I need to try it more. I remember that I did not always translate all the texts. For the moment, such an error no longer happens.

Thanks, @mweststrate and @davidgolden, just moving the following to the top of the plugins array in the babel.config.js solved the problem for me.
["@babel/plugin-proposal-decorators", { legacy: true }], ["@babel/plugin-proposal-class-properties", { loose: true }],


Update boinc to 7.8.2

After making all changes to the cask:

[x] brew cask audit --download boinc is error-free.
[x] brew cask style --fix boinc reports no offenses.
[x] The commit message includes the cask’s name and version.

Thanks for the PR!


Attribute "border_color" has already been defined

Dependency

compile 'de.hdodenhof:circleimageview:1.3.0'

Getting error when building:

Error:(2) Attribute "border_color" has already been defined
Error:(2) Attribute "border_width" has already been defined

I don't have these values in my values/colors.xml or anywhere else.
This problem occured when adding this library to my gradle file.

This will be fixed after merging #58. You could build that branch yourself until I'll upload a new build to maven central.

Ok thats great! Thanks.

I got the same problem. The source was the fact that I also had in my Gradle file the library Pkmmte/CircularImageView. I guess Pkmmte also defined those values

Fixed in master: 8e1e781.

Hi, first off, thanks for this great library! May I ask when will this be pushed to the maven central?

I just pushed v2.0.0 with this fixed.


Issue #498

issue

Can I help you with a review? May be separate to several pull requests/branches/commits?


Cleaning up old jobs

Hi,
Is there anyway of cleaning up old jobs from the Mongo DB?
The UI displays the last 20 jobs but nowhere can I find any mention of what happens to old jobs.
I have many jobs in the DB to the point where Mongo is now refusing to start because of insufficient disk space.
Apart from removing job documents by hand from Mongo, is there anything else I can do?
If not, why keep them if we can't view them? And maybe a config parameter would be good to have, don't you think?
Thanks

Good idea, maybe even with a default value where it cleans up jobs after they are x old and there are more then y.

Until it's possible to configure how many jobs the GUI can display, I'd say the default value should be the maximum number of jobs that are displayed (i.e. 20). What's the point in keeping more if you can't view them?
If there are people out there that view the data directly from the Mongo DB, the limit should not be applied if the number of jobs is greater than the limit itself. A log message would be preferable with maybe an extra option to force Strider to delete the old jobs.

There are many old jobs, and query all of them cost a lot time.

Removing job documents by hand

Lightweight :P
> db.jobs.count();
10661

Those 10k jobs don't produce any noticeable effect on performance. If you think you're having trouble with 126 jobs, the count is not the problem.

Agreed.
The only reason I brought this up is because my build process produces a lot of output and therefore takes up a lot of space.

Maybe someone can suggest something that can be run in the mongo shell from a cron job? Then that could be a workaround for the time being.

Like a prune that removes the oldest jobs beyond n jobs.

Something like this should do it:  mongo localhost:27017/strider-foss  /tmp/clean_old_jobs.js
where /tmp/clean_old_jobs.js is:
var MAX_JOBS = 20
projects = db.projects.find({}, { 'name': true }).toArray()
var nb_deleted = 0
var i, projects_len
projects_len = projects.length
for (i = 0; i < projects_len; i++) {
   var cursor = db.jobs.find({ 'project': projects[i]['name'] }, { 'created': true }).sort({ created: true })
   var jobs = cursor.toArray()
   var jobs_len = jobs.length
   for (j = 0; j < jobs_len - MAX_JOBS; j++) {
     db.jobs.remove( { _id: jobs[j]['_id'] } );
     nb_deleted++
   }
}
print("Deleted " + nb_deleted + " jobs")

We could add that as a command to strider-cli

Excellent idea!

Implemented in the cli, see https://github.com/Strider-CD/strider-cli#prunejobs
Available in master of Strider. Let me know if there are any issues.

@knownasilya Very awesome!


Make DistributedCp/Mv not retry

Reason: overwrite has been broken. Overwrite hasn't appeared to have
worked since at least 2.0 or perhaps further back.
Both the createDirectory and createFile calls in MigrateDefinition is
not overwriting version.
Thus, retrying is pointless because retries will always fail with the
message of being unable to overwrite.
pr-link: Alluxio/alluxio#10665
change-id: cid-5d9b1144501a4aba5256a6fb2ec6f1686d20c43a

@LuQQiu

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/9038/
Test PASSed.

@LuQQiu : did you want to merge this? Not merge this?

ping @LuQQiu @ns1123 is this still necessary?

@bradyoo is there a reason this is to 2.1 and not master?

Close this PR for now, thanks for providing the fix


Update README.md

Lists m2m tags interface

Thanks @dimitrov-adrian! Just a small change & then we're good to go 👍


Add information about licensing

It would be also great to add information about licensing.
PS: Project was added on the Android Arsenal. If you would like to support AA, you could add this badge in the README file (I will really appreciate it):
[![Android Arsenal](https://img.shields.io/badge/Android%20Arsenal-android--icon--copier-brightgreen.svg?style=flat)](https://android-arsenal.com/details/1/1325)

All done! thanks!


Able to list project for a short time even after it gets deleted.

acorn version v0.5.0-rc3-23-g34eb244+34eb2440
Able to list project for a short time even after it gets deleted. acorn project rm should wait for the project to be removed completely before returning success.
 % acorn project rm hello1
hello1
 % acorn project
NAME                          DEFAULT   DESCRIPTION
acorn.io/sangee2004/acorn
acorn.io/sangee2004/mytest1
acorn
hello1           ( hello1 is still listed)
hello2
hello3
p2                            *
% acorn project
NAME                          DEFAULT   DESCRIPTION
acorn.io/sangee2004/acorn
acorn.io/sangee2004/mytest1
acorn
hello2
hello3
p2

I don't see this as a blocker, but leaaving it for @ibuildthecloud to look at in case he has a quick fix

https://github.com/acorn-io/acorn/pull/1111

Tested with acorn version v0.5.0-rc3-48-g7e333e6+7e333e6f
acron projects done right after project is deleted does not list the project that was deleted.


[F] Fix undefined behavior due to incorrect usage of isprint

Fixes a part of problems from https://github.com/seleznevae/libfort/issues/20

Pull Request Test Coverage Report for Build 627

4 of 4 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 78.944%

Totals

Change from base Build 621:
0.0%

Covered Lines:
4334

Relevant Lines:
5490

💛  - Coveralls


How can I receive the full AD object with all the attributes not just the few from the examples?

Hi guys,
I'm using this package right now and it works perfectly.
The one thing I'm missing is to receive the full user object(s).
I want to have more attributes then just the few from the example (samAccountName, mail, ...).
Is this possible or will it get added in a future update?
Am I just missing something? :D
Thank you guys! Great work at maintainers!

Hi @SelfDevTV , did find how to receive the full user object?


an example of using formly with advanced layout build with flex-layout and angular-material rather than bootstrap

I'm submitting a ...  (check one with "x")
[ ] bug report => search github for a similar issue or PR before submitting
[ ] feature request
[ x ] support request

an example of using formly with advanced layout build with flex-layout and angular-material rather than bootstrap

@aitboudad while solution you provided works well with fields, when I wanted to add flex-layout to  fieldGroups so that elements below act accordingly (Ex: fieldGroup has fxLayout attribute) by using wrapper.
Problem is, wrapper wraps fieldGroup therefore whole group is just one elment, can't affect fields below.

Note: the @angular/flex-layout example has been updated which use a better way by relying on a custom field type, here is an example https://stackblitz.com/edit/ngx-formly-flex-layout

@aitboudad
In the example you posted, you set up fxLayout, how can I specify the size for each fxFlex field?
In case it is not possible, where this type is located in the repository, for me to create a custom one, based on it


Performance questions

Incredible work! I was trying to see what kind of overhead there is to call a wasm function from Python.
I am using WSL2 on Windows, with a recent Fedora, CPython3.10.
Re-using the exact gcd.wat from the examples, it looks like any call has a "cost" of about 30μs.
The code I used is a simple timer to compare performance:
import wasmtime.loader
import time
from math import gcd as math_gcd
from gcd import gcd as wasm_gcd

def python_gcd(x, y):
    while y:
        x, y = y, x % y
    return abs(x)

N = 1_000
for gcdf in math_gcd, python_gcd, wasm_gcd:
    start_time = time.perf_counter()
    for _ in range(N):
        g = gcdf(16516842, 154654684)
    total_time = time.perf_counter() - start_time
    print(f"{total_time / N * 1_000_000:8.3f}μs")

This returns about:
   0.152μs  # C code from math.gcd
   0.752μs  # Python code from python_gcd
  31.389μs  # gcd.wat

Note that I tested this with an empty "hello world", and the 30μs are still there.
I am wondering about 2 things:

is this overhead inevitable and linked to the design, or are there ways to reduce it?
I noticed in the gcd.wat code that input parameters are expected to be i32, but the code does not fail when parameters exceed 2**32, so I was wondering how this works

Not much attention has been paid to performance in the bindings in this repository, so this isn't altogether unsurprising. I have never personally written high-performance Python and consequently probably made a ton of mistakes when writing these bindings.
If you're looking for performance though I would recommend the Rust bindings rather than the Python bindings, as there we have indeed focused on performance and the overhead is significantly smaller.

Thanks, I was actually wondering if using wasm was a possibility for Python libraries that rely on C/C++/Rust, as you would only need to build the one file, then read it using wasmtime-py, instead of generating many platform wheels, which can be a headache. So I wanted to compare performance as a first step.
Thanks a lot for your response, I will close the issue.

That's definitely an intended use case for bindings like this, and I'll emphasize again that the cost here isn't intrinsic to these bindings or Python, it's probably just that I don't know how to write high-performance Python. PRs are of course always welcome for improvements as well.

I've been doing some experiments and identifying bottleneck
the first one which is solved now is moving data in and out
this was demonstrated by moving large data (several MB of images)
the next obvious bottleneck is that there seems to be a fixed context switch cost between python and WASM or vice versa
I can demonstrate this bottleneck by considering the following cdb_djp_hash.c
it's a simple 32-bit string hash
and estimated the cost of find hashes of strings of different length, 13, 26, 130, 1300
that is 1x 2x 10x and 100x the work
a loop of 100x
and the shocking finding was all this took the same time ~40ms regardless of the length of the string
on the other hand the pure python and the c version the time was proportional to the length of the string
the pure python started faster than wasm for the 13 byte string with 1.7ms (vs ~40.0 ms in WASM)
but for the 1300 string the pure python  took 170ms (vs. ~40.0 ms in WASM)
this indicates that most of the 40ms is a constant cost of switching regardless of the actual work done inside the loop
I've some commented profiling to identify exactly where is this waste of time happening
but did not have enough time to continue
https://github.com/muayyad-alsadi/wasm-demos/blob/main/cdb_djp_hash/cdb_djp_hash.py#L88

Re-using the exact gcd.wat from the examples, it looks like any call has a "cost" of about 30μs.

I think I fixed that in #137
@alexprengere would you please test my approach and report how much performance gain did it achive

I've was able to eliminate the 40ms wasted time
here is the code
# gcd_alt.py
import ctypes

from wasmtime import Store, Module, Instance, WasmtimeError
from functools import partial

from wasmtime import _ffi as ffi
from wasmtime._func import enter_wasm
from wasmtime._bindings import wasmtime_val_raw_t

store = Store()
module = Module.from_file(store.engine, './gcd.wat')
instance = Instance(store, module, [])

def func_init(func, store):
    ty = func.type(store)
    ty_params = ty.params
    ty_results = ty.results
    params_str = (str(i) for i in ty_params)
    params_n = len(ty_params)
    results_n = len(ty_results)
    n = max(params_n, results_n)
    raw_type = wasmtime_val_raw_t*n
    func.raw_type = raw_type
    def _create_raw(*params):
        raw = raw_type()
        for i, param_str in enumerate(params_str):
            setattr(raw[i], param_str, params[i])
        return raw
    func._create_raw = _create_raw

_gcd_in = instance.exports(store)["gcd"]
func_init(_gcd_in, store)

def gcd(a, b):
    raw = _gcd_in._create_raw(a, b)
    raw_ptr_casted = ctypes.cast(raw, ctypes.POINTER(wasmtime_val_raw_t))
    with enter_wasm(store) as trap:
            error = ffi.wasmtime_func_call_unchecked(
                store._context,
                ctypes.byref(_gcd_in._func),
                raw_ptr_casted,
                trap)
            if error:
                raise WasmtimeError._from_ptr(error)
    return raw[0].i32

print("gcd(6, 27) = %d" % gcd(6, 27))

and here is the benchmark
import wasmtime.loader
import time
from math import gcd as math_gcd
from gcd import gcd as wasm_gcd
from gcd_alt import gcd as wasm_gcd_alt

def python_gcd(x, y):
    while y:
        x, y = y, x % y
    return abs(x)

N = 1_000
for gcdf in math_gcd, python_gcd, wasm_gcd, wasm_gcd_alt:
    start_time = time.perf_counter()
    for _ in range(N):
        g = gcdf(16516842, 154654684)
    total_time = time.perf_counter() - start_time
    print(total_time)

and here is the result
0.0002523580042179674 # math_gcd
0.0014094869984546676 # python_gcd
0.043804362998344004  #  wasm_gcd
0.005873051006346941  # wasm_gcd_alt <-------- my WASM

we have 3 types of performance bottlenecks

overhead of passing large data #135 #81 #134 which is fixed, merged  and released as part of 7.0.0 release
overhead of calling a small wasm function from python #137 #139 which is fixed not yet merged. in this case the function itself does not take time (ex. gcd of two integer or simple string hash like cdb hash). there is a constant time wasted when calling a wasm function from python regardless of what's inside the function
overhead of calling python from wasm. the loop is inside wasm and the small function lives in python

I've addressed the first two, and I'll create a ticket for the last one with details


fix misspell 'instance' word

misspell in error message

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Mingjun Liu seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.

Current coverage is 91.61% (diff: 100%)

Merging #881 into master will not change coverage

@@             master       #881   diff @@
==========================================
  Files           114        114
  Lines         16631      16631
  Methods           0          0
  Messages          0          0
  Branches       3896       3896
==========================================
  Hits          15237      15237
  Misses          672        672
  Partials        722        722

Powered by Codecov. Last update e1614b5...3d4a7a2


chore: update pkg.pr.new gh action

Description

This PR adds pkg.pr.new  as dev dependency, we need to register ni here: https://github.com/apps/pkg-pr-new (I don't have admin privileges on ni repo)
Linked Issues
Additional context

/pkg-pr-new

We can also remove the dev dependency, removing it... dont merge yet


More understandable formatting of alert data in logs

What type of PR is this?
Cleanup
What this PR does / why we need it?
Minor change. Improves the formatting of the alert data by ensuring that the struct's fieldnames are specified. It's not perfect, but it makes the log slightly more interpretable to know what fields mean what.
Goes from this:
alert="{firing map[alertname:AnotherWatchdogManagedNotificationSRE alertstate:firing namespace:openshift-monitoring openshift_io_alert_source:platform prometheus:openshift-monitoring/k8s send_managed_notification:true severity:info] map[description:test] 2022-03-13 04:45:16 +0000 UTC 0001-01-01 00:00:00 +0000 UTC https://blahblah }"

to this:
alert="{Status:firing Labels:map[alertname:AnotherWatchdogManagedNotificationSRE alertstate:firing namespace:openshift-monitoring openshift_io_alert_source:platform prometheus:openshift-monitoring/k8s send_managed_notification:true severity:info] Annotations:map[description:test] StartsAt:2022-03-13 04:45:16 +0000 UTC EndsAt:000
1-01-01 00:00:00 +0000 UTC GeneratorURL:https://blahblah Fingerprint:}"

Codecov Report

Merging #30 (5deef79) into master (b978c44) will not change coverage.
The diff coverage is 66.66%.

@@           Coverage Diff           @@
##           master      #30   +/-   ##
=======================================
  Coverage   24.56%   24.56%
=======================================
  Files          13       13
  Lines         639      639
=======================================
  Hits          157      157
  Misses        472      472
  Partials       10       10

Impacted Files
Coverage Δ

pkg/handlers/webhookreceiver.go
40.76% <66.66%> (ø)

/lgtm


Catch Ctrl+C on SC invoke password entry

What current issue(s) does this address, or what feature is it adding?
Ctrl+C on password entry when invoking a SC breaks neo-python
How did you solve this problem?
try-catch
How did you make sure your solution works?
manual test
Are there any special changes in the code that we should be aware of?
no
Please check the following, if applicable:

[ ] Did you add any tests?
[X] Did you run make lint?
[X] Did you run make test?
[X] Are you making a PR to a feature branch or development rather than master?
[ ] Did you add an entry to CHANGELOG.rst? (if not, please do)

It was setup for master branch instead of dev. If you can resolve the conflicts we should be good.

didn't check dev :P
https://github.com/CityOfZion/neo-python/blob/development/neo/Prompt/Commands/SC.py#L216

ah I see. I thought this could have been a missed try/catch


tests: arch_interface: test early serial on x86

test function arch_printk_char_out() on x86.
Signed-off-by: Ying ming<EMAIL_ADDRESS>

Sorry for replying late. The reason why I add this test is because arch interface of x86 is in our Fusa scope. So our goal is to test all the funtion in related source code.
I haved move this testcase to "tests/arch/x86/info".

Close this PR, @Zhaoningx will take over this job.


Extract TAT-C Application

Now that TAT-C has been listed on PyPI and more easily installed for downstream applications, we should extract the source code and supporting configuration for the web application to a different repository.

After extracting TAT-C application, investigate whether we can relax upper bound on Python version (currently <3.11).


Upgrade jest to 29

I enabled the old snapshot format, to verify nothing's breaking.  It could be turned off if we want to use the new snapshot format, which is a bit clearer, I think.

📦 Published PR as canary version: 2.0.8--canary.71.da93500.0

:sparkles: Test out this PR locally via:
npm install<EMAIL_ADDRESS># or
yarn add<EMAIL_ADDRESS>

:rocket: PR was released in v2.0.8 :rocket:


Can't get code example on Fabric to work

I am attempting to implement code checks into my custom structure to have more control over the spawning of my structure but no matter what I do I can't get it to run. I've followed your code example with the Sky Structures but when I get to editing my json file in the worldgen/structure folder and remove use_expansion_hack like it states in your json, the game crashes because it's not provided. When I return the key and open a new world, the checks that I did in the code seemingly are ignored.
I poured through vanilla structure code and tried searching up other solutions online but I can't seem to get my structure json to be untied to the key requirements that minecraft has set and instead use my own custom code with the codec. I'm not sure if 1.20.4 is just very different from 1.20.2 in this way or I am just doing something very wrong. Any help would be appreciated.

The codec, and by extension the code, is copied straight from your SkyStructures.java class for now just to get things working, with the exception of making sure the height is below -30 in the extraSpawningChecks() just to verify the method is working. I can send my json files, logs don't have anything to say about my structure spawning, at least nothing that I could find, but I can still send those if you'd like.
{
  "type": "minecraft:jigsaw",

  "start_pool": "mythos-amythologymod:fates_mausoleum/start_pool",

  "size": 1,

  "max_distance_from_center": 80,

  "biomes": "#mythos-amythologymod:has_structure/fates_mausoleum",

  "step": "surface_structures",

  "start_height": {
    "type": "minecraft:uniform",
    "min_inclusive": {
      "absolute": -25
    },
    "max_inclusive": {
      "absolute": 40
    }
  },

  "use_expansion_hack": false,
  "spawn_overrides": {},
  "values": [

  ]
}

I couldn't tell if the code was working so I made the start_height what it is now and checking if it is below -30 and only letting it spawn if so just to verify the code is working. The structure still spawns above -30. I added this check because at first I was changing the startY integer in the getStructurePosition() method and noticing no changes either. I want to eventually get it so that most of the information is generated in code rather than json files, particularly the height, but I'm getting no where.

The type there is pointing to the vanilla jigsaw class. You want it to point to your own registered structure type in order to run your code instead

I don't believe it is possible to know the magnitude of the discontent I feel for myself for continually overlooking that line. I appreciate and thank you for your swift responses.


node - Cannot find module

I cannot seem to use this package in my vue project.
I also run typescript, which is something that this package has no support for.
33:38 Cannot find module 'vue-chat-emoji'.
> 33 | import { VueChatEmoji, emojis } from 'vue-chat-emoji'

Reproduction

$ yarn add vue-chat-emoji
$ yarn install && yarn serve
33:38 Cannot find module 'vue-chat-emoji'.

This works:

import { VueChatEmoji, emojis } from 'vue-chat-emoji/dist/vue-chat-emoji.esm.js'

Thanks. I will work towards that.

@vestusola one of the binary files (dist/*.esm|umd|min.js) should be set as the main prop so this wouldn't be neccessary.

@vestusola  This is fixed, works well now.
However, latest release is broken:

@vestusola This is fixed, works well now.
However, latest release is broken:

Is it via import or using the CDN link?
If you import the package, remember to require the stylesheet file

@vestusola  CDN link is broken (404) and im using import, i simply updated package.json to use the latest release rather than my fork, and this happened
i believe its unrelated to the stylesheets since it works on my fork which is using 1.1.26

@PatricNox I tried using the latest version 1.1.27 and it works well on my side.

@PatricNox I tried using the latest version 1.1.27 and it works well on my side.

I mean the CDN link

@vestusola  Ah!
Since I use typescript, I had to import the stylesheet by manually copying the css contents.
I forgot to do this on the new release, my bad.
Everything works flawlessly, amazing!


Uncaught TypeError: Cannot read property 'postMessage' of undefined

We got this sporadically. We're using hosted fields integration. The setup looks like following:
try {
    braintree.setup( paypalBraintreeData.token, "custom", {
        id: checkoutFormID,
        hostedFields: paypalBraintreeData.hostedFields,
        onReady: onReady,
        onError: onError,
        onPaymentMethodReceived: onPaymentMethodReceived
    } );
} catch (e) {
    onError(e);
}

Even wrapping the setup doesn't catch the exception, so I thought this happens async. This happens after the setup and onReady is not invoked yet. I scanned braintree-2.26.0.min.js for any line that has postMessage call and one is missing the try-catch block. It's around this:
return e===!1?!1:void i.postMessage(e,n)}

Since this is working intermittently, I try to reproduce it on several client sites and got one that I could reproduce consistently by throttling the internet connection too. Let me know if you want more detail on that as I couldn't share the URL publicly.

Hi @gedex, thanks for reporting.
Could you contact our support team at<EMAIL_ADDRESS>with the steps for reproduction?

Closing. Reopen if the root cause exists within our SDK.

@mrak I'll come up with simplified test case for this. It turned the issue happened sporadically on test site. One possible culprit is on how we setup and tear down the integration multiple times due to the re-rendering, via AJAX, HTML section where the fields are displayed.

@mrak did you ever get an answer on this? I'm using the Drop-in UI and experiencing this sporadically as well.

We are using the Drop-in UI and also experiencing this sporadically.
TypeError: Cannot read property 'postMessage' of undefined
  at e.reply.e.reply(/js/braintree-2.29.0.min.js:4:9545)
  at MessageEvent.<anonymous>(/js/braintree-2.29.0.min.js:5:2128)
  at MessageEvent.r(/js/braintree-2.29.0.min.js:4:29823)
  at p(/js/braintree-2.29.0.min.js:4:10118)
  at h(/js/braintree-2.29.0.min.js:4:9952)

@Enalmada Can you provide some reproduction steps?
In particular, which browsers (and browser versions) are you seeing this with?

I just see it in logs after I setup sentry.io javascript monitoring but here is everything I can think of:
We are using pjax between pages (doing setup and teardown between ajax page loading).  Seems to be consistent with other reports above.  Sentry says it is usually happening on chrome.  So far seems to always be on device: HTC 2PST2.

Test server is available: https://webtest.relationup.com/credits

Just fyi this problem can happen on safari ios10 too and look like this...as a google hook for any others that happen to be searching when they experience it themselves.
TypeError: null is not an object (evaluating 'i.postMessage')
  at reply(/js/braintree-2.30.0.min.js:4:9545)
  at ? (/js/braintree-2.30.0.min.js:5:2220)
  at r(/js/braintree-2.30.0.min.js:4:29828)
  at p(/js/braintree-2.30.0.min.js:4:10123)
  at h(/js/braintree-2.30.0.min.js:4:9953)
  at this(raven.js:372:34)

Was there ever any resolution on this? Still seeing it now.

@isaachinman What version of braintree-web are you using?

@crookedneighbor Sorry for not posting that originally:
"braintree-web": "2.30.0"

Version 2.31.0 includes a change to our post message library that may fix this. Please upgrade and report back if you're still having the issue.

@crookedneighbor Is there an accurate list of releases somewhere? I am fine with upgrading to the latest 2.x version but need to be 100% sure there aren't any breaking changes.

@isaachinman Since 2.x only included the built minified code, we no longer push those changes to this Github repo. Instead, we push them to the repo we use to manage bower releases, which does include the changelog. You can find it here: https://github.com/braintree/braintree-web-bower/blob/2.x/CHANGELOG.md
Can you provide exact steps to reproduce the issue?


All data analysis must be done in Python?

I can code in python but I'm more a R programmer. So, can I contribute with R scripts or must I translate everything to python?

You can use R. Check issue #73 and this PR  for ongoing analysis:  https://github.com/datasciencebr/serenata-de-amor/pull/52


Custom Auth Module for authentication for stackstorm using keycloak

I have developed a custom authentication backend called opscruise_keycloak  for integration with StackStorm. This backend leverages the Keycloak Python library to seamlessly connect with Keycloak servers. During regular StackStorm installations, the integration of this custom backend proves successful, ensuring smooth authentication processes.
However, when deploying StackStorm using Helm charts, a challenge arises. Helm deployments lack the flexibility to accommodate updates or configurations for custom authentication backends, posing a hurdle in seamlessly incorporating our opscruise_keycloak package.
I need help to make my package come up by default with the helm-based deployment.
Also, I wanted to make my package publically available for all the Stackstorm users as an open-source package.

I transferred this from the helm chart repo, because this is a more general discussion.
A new auth backend would be great!
Have you signed the CLA?
Are you willing to open source the code under the Apache 2 license (the only license we use in the StackStorm org)?
Do you have a link to the code that we can review, and then perhaps transfer it under the StackStorm org?
For now, if you want to use an alternate auth backend with k8s, you need to use a custom docker image that adds the relevant python packages. Then use the helm chart values to adjust st2.conf settings to use that new auth backend.


Update cdb.h

Useless since last commit

https://github.com/radare/sdb/commit/f7f1010dbc1ed7c1cb952f989753b96c1fafc123#diff-5a92056954d424b70a53bbbb17477e94L104/
ssize_t not needed anymore


[Removal]: 3 of 39 Ex Preveza, Greece relocated to Bulgaria

Hotspot Name
Massive Graphite Albatross
Hotspot b58 Address
116xS2H5Tfndied8Hhot7LbJZALosjsu9pd3PRKPx1A2uqH8gJC
Discord Handle
No response
Hotspot Manufacturer
SyncroB.it
Removal Reason
Device has been updated to reflect its actual location.
Modifications
No
Extra forwarders
No
Extra antennas
No
Additional Information
Refer to 1186Mo7vFig86LiQznq4HVxgyDzSkF2G8V8cJXT7JxfADeDaoRz ,a device located in the nearby town, as a proof of witnessing.

Please provide photo and GPS location of the new setup.


fix loading more

Fix trigger loadNext on reaching bottom after the first time

@hazantip I'm sorry I missed this earlier. It's the current version not working for you?

@ankeetmaini version until 29.09.16 not working as well as i need, and in my project i using with my fix;

@Hazantip actionTriggered  needs to be set true as otherwise if the user scrolls to the bottom again while the next function is already executing it'll call it again. The check exists to stop duplicate invocation of next function which might be an API call.
I'm closing this PR. Can you raise an issue or give me a JSBin so that I can debug why it's not working for you?
Please check the demos in the demos folder.